Check that envvars are valid identifiers when using encrypted strings

When Secretary decrypts something it outputs a scriptlet that is sourced into the env, to overwrite encrypted env vars with decrypted ones. Meaning that env vars with encrypted substrings like rabbitmq.password won't work since they contain dots. This change ensures the problem is caught with a user readable error

Current coverage is 55.36%

Merging #8 into master will increase coverage by +0.92% as of a3c2ff9

@@            master      #8   diff @@
======================================
  Files            8       8
  Stmts          461     466     +5
  Branches        70      70
  Methods          0       0
======================================
+ Hit            251     258     +7
+ Partial         31      30     -1
+ Missed         179     178     -1

Review entire Coverage Diff as of a3c2ff9

Powered by Codecov. Updated on successful CI builds.


🚀 [Feature] Build Admin Interface Coin Creation Management

🚀 Feature
As Supporter we need to create coins for multiple users, for this we have a page with every user or a filtered list.
In this list we need:

Select every User that gets a coin creation,
A user that has >= 1000 coins creation for the selected month needs to be fade out / not selectable
Select one date for all creations,
Enter a message for all creations,
Enter one amount for all creations,
Each User gets a own message
Each User gets a own amount
Edit and Delete a coin creation

ToDos:

[ ] Create Coin Creation Management Page
[ ] Use User Search Component / slots for specific user coin creation fields (Select, Message, Amount) \
[ ] Coin Creation Form Component / Date, Message, Amount for all creations \
[ ] Create a Coin Creation Transaction Page
[ ] Coin Creation Transaction Component / Edit & Delete \

work an this https://github.com/gradido/gradido/issues/1124


Documentation - sticky connection setting moved

https://docs.opnsense.org/manual/how-tos/multiwan.html#configure-load-balancing
It states that Stick Connections can be enabled in
System->Settings->Miscellaneous
But it looks like it has been moved to
Firewall->Settings->Advanced

Fixed


Logs: No way to differentiate stdout vs stderr

There is no apparent way to differentiate stdout lines from stderr (the messages are not header-ed with a type, length, etc like in the docker API)

This has been fixed.


Add SPDX license expressions to the output

Today we only display license expressions and license data using ScanCode LicenseDB keys. We should also display these with an SPDX license expression.

Could be done as a part of https://github.com/nexB/scancode.io/issues/569? Since we have new fields for SPDX expressions in packages/resources.

@AyanSinhaMahapatra perfect!

Following the toolkit v32 upgrade, we now have SPDX version of the license expressions on the model and in the UI/results.


[NavigatorIOS] Navigator overlay NavigatorIOS bar

Is it possible for a Navigator component to overlay NavigatorIOS navigation bar? I can't make the modal transition of the JS navigator to overlay the native navigation bar.

This is an example of the issue. Looks confusing to open a modal an still have a navigation with a back button that does not close the modal.

I would try putting the NavigatorIOS component inside of the Navigator (it might not work because it takes extra work to set up UIViewControllers inside of arbitrary views but worth a shot).

yeah, you probably just don't want to mix these...but passing to Eric in case he's thought about this more.

Yeah, combining navigator with the native navigator is not very well supported. But I suspect it will work if you put the Navigator outside of NavigatorIOS

@alvaromb NavigatorIOS has a navigationBar and a view. When you nest Navigator inside it, it is being drawn into the view. Then NavigatorIOS draws the navigationBar on top of the view. Hence the view has no way to come on top of the navigationBar.
Please try the suggestion by @ericvicenti and share here!

Thank you for your replies. I'll try that as soon as I can @paramaggarwal, but I'm facing a tremendous amount of work right now and this has been removed from my top priority list. I expect I'll be able to try this on Friday.

I've switched from NavigatorIOS with a Navigator to display the modal to a full Navigator navigation.
However, if I set a navigationBar in the parent Navigator, it stays on top of the modal-Navigator.
This is the render code for the main Navigator:
return (
<Navigator
    navigationBar={
        <Navigator.NavigationBar
            style={styles.navBar}
            routeMapper={MnmNavigationBarRouteMapper}
        />
    }
    initialRoute={{
        title: this.state.selectedTab,
        component: MnmPublished,
        passProps: {section: this.state.selectedTab}
    }}
    renderScene={(route, nav) => {
        var props = route.passProps;
        props.navigator = nav;
        return React.createElement.bind(this)(
            route.component, props
        );
    }}
    configureScene={(route) => {
        if (route.sceneConfig) {
            return route.sceneConfig;
        }
        return Navigator.SceneConfigs.FloatFromRight;
    }}
/>
);

This is how I open another Navigator (the "modal" component):
this.props.navigator.push({
    index: 1,
    component: MnmCommentsContainer,
    sceneConfig: Navigator.SceneConfigs.FloatFromBottom,
    passProps: {entryId: entrada.id}
});

And this is the code for the "modal" component:
render() {
    return (
        <Navigator
            renderScene={(route, navigator) => (
                <MnmComments entryId={this.props.entryId}/>
            )}
            initialRoute={{
                title: 'Comentarios'
            }}
            navigationBar={
                <Navigator.NavigationBar
                    routeMapper={MnmNavigationBarRouteMapper}
                    style={styles.navBar}
                />
            }
        />
    );
}

This last navigationBar is the one that has the red background. What I would like to achieve is  a stacked (right-to-left) navigation similar to the iOS native one and open a modal-like transition from one of the previously navigated views (something which I think is quite common in iOS app navigation). That modal should overlay the content that's behind. I would also like to prevent the right-to-left transition in the root navigationBar if possible.

Another thing that bugs me is that setting the content to flex: 1 with the custom navigationBar displays the content a couple of points behind the navigation bar, preventing the first element to be correctly displayed.

This is intentional. You should use content insets or top padding if you don't want that effect.

Thanks @ide! I've realised about that a while ago so I decided to close the issue :smile:

From this conversation I can't find solution
May be somebody will help
Here what I have

But I need Red one should overlay Green one NavBar

@ButuzGOL you can wrap your views in two <Navigator /> components. Take my pseudocode as an example:
<Navigator ref='topNavigator'
  render={() => {
    <Navigator ref='innerNavigator' render={ ... } />
  }}
/>

Then you use innerNavigator to make push-like transitions from right to left with a navigation bar. When you want to open your red modal and overlay the other views, use a Navigator.SceneConfigs.FloatFromBottom transition and make the push from topNavigator instead of innerNavigator.
Let me know if you have any trouble setting up this :smile:

@alvaromb thanks for comment
but I made it using http://facebook.github.io/react-native/docs/modal.html#content
So you can add there Navigator and it works like I expected

Cool @ButuzGOL! Modal it's a very nice component but keep in mind that if you render some kind of picker component there, chances are that you're not going to get it rendered because of this https://github.com/facebook/react-native/issues/2628 bug.

@alvaromb what if you set navigationBarHidden to true when you push your Navigator or modal schene for NavigatorIOS? would that work?

Haven't tried that @IlyasM, but honestly the nested <Navigator /> is the way to go.


Add support for older IntelliJ versions

I have a dead end because the runConfiguration extension for NodeJS was changed leaving it with same name and we can't limit the dependency to a specific nodejs version so we have to either support pre-2023 or post-2023 :|


Refactor out legacy content

[x] create placeholder for legacy mock content /data-legacy
[x] include readme in legacy folder, explaining where content came from
[x] move non-British regions to legacy folder
[x] move non-complex categories to legacy folder
[x] remove britain-complex content that doesn't follow page model
[x] fix quick-links to avoid missing pages
[x] create single non-complex category (to check we are walking multiple categories
[x] copy (fixed) britain-complex content to britain-legacy
[x] rename britain-legacy content to make it unique
[x] expect data folder to contain these folders:

[ ] britain1
[ ] britain_complx
[ ] britain_legacy
[ ] introduction
[ ] PlatformData
[ ] QuickLinksData

[ ] check above runs through dita converter (update from Robin's branch)

Done.


Update mapintegratedvuer to 1.4.2.

This update fix an issue with the minimap on the flatmapviewer.
A separate hot-fix branch has been created here - https://github.com/nih-sparc/sparc-app-2/tree/v1.3.0_hot-fixes
A heroku instance with Portal v1.3.0 + the viewer update can be found herr - https://alan-wu-sparc-app.herokuapp.com/
Bug:

Fixed:

failing tests are unrelated


v0.18.0

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.6 out of 7 committers have signed the CLA.:white_check_mark: mattkasun:white_check_mark: abhishek9686:white_check_mark: afeiszli:white_check_mark: alphadose:white_check_mark: Aceix:white_check_mark: 0xdcarns:x: dependabot[bot]You have signed the CLA already but the status is still pending? Let us recheck it.


Use k8s keychain to auth against IaaS

@cppforlife
Azure:
I chose to provide client id and secret however the config can also be provided with an assigned managed identity ID
(The flag comes from the library https://github.com/vmware-tanzu/carvel-imgpkg/blob/f7fe9601fcb35e5cbaac88783381137ddef707ab/vendor/github.com/vdemeester/k8s-pkg-credentialprovider/azure/azure_credentials.go#L44)
cat <<EOF >/tmp/azure
{
        "aadClientId": "client",
        "aadClientSecret": "secret"
    }
EOF

/imgpkg pull -i registry-subdomain.azurecr.io/some-image:latest -o /tmp/lol --azure-container-registry-config=/tmp/azure

SUCCEEDS

As part of this PR, logs from iaas providers have been suppressed.
After https://github.com/vmware-tanzu/carvel-imgpkg/pull/241 has been merged, it would be useful to print out IaaS logs if the --debug flag is set.


Improve Accessibility

Nice component!
Consider adding accessibility features, especially since this component uses div elements instead of the more self-descriptive table elements.
https://www.w3.org/TR/2017/NOTE-wai-aria-practices-1.1-20171214/#grid
Here's a presentation I recently attended that put this topic front and center in my mind:
https://youtu.be/txdDJNgqVlw?t=1821

I welcome any and all PR's to implement this :)

This feature/issue has been tagged as one that will likely be fixed or improved in the next major release of React-Table. This release is expected to be completed in the next 6 - 8 months. Please see #1143 for more detail. If you believe this feature can be easily fixed, we invite you to fork the project, attempt the fix, and use it until the latest version is available. If the fix is simple enough, we will also consider it as a PR to the current version.


v21.2.0-alpha.1 Release Notes

Prep date: tbd
Release date: Aug 30, 2021
SHA candidate: tbd
Release Manager: tbd
Issue: tbd
Docs To Do:

[ ] Run script to generate release notes
[ ] Update docs site
[ ] Edit release notes - Additions/changes/omissions to the release notes should be left in this issue's comments
[ ] Once PR is merged / release notes are live, notify @meagan-g in the #releases Slack channel
[ ] Email releases@ with link to published release notes
[ ] Generate release notes issues (cc @jseldess)

Closing this issue, as this release got pushed back. Will rename the subsequent issues.


Fix: Multisig details

Fix:

Show appropriate block explorers in Multisig Details
Removed unused functions

New:

Finally made a standalone Popover with explorers that can wrap any DOM element
Created 2 functions to produce explorer link for account / extrinsic (with tests)
Added Wallet types similar to Account types
WalletConnect wallet have runtime property isConnected that is calculated only once in model
2 new components WalletCardLg WalletCardMd, they do not include explorer popover, if you need so use component composition and provide required data (explorers, matrix, custom actions)
ExplorerLink component

Task linked: CU-862kcvq47 Show Multisig wallet details

Task linked: CU-8692y90xq WalletCard

Task linked: CU-86930u98c Multisig details. Multisig address is displayed in your wallets instead of the signer's address.


Have a problem with applying css in serenity

Hi
I would like to align the following one vertical line but the default is like following. This default design is with right to left languages.

Thank you for help

https://serenity.is/demo/BasicSamples/StaticTextBlock#!1

In xyzForm.cs:
....
[HalfWidth]
[DisplayName("")]
[Serenity.StaticTextBlock(Text = "", IsLocalText = true, IsHtml = true)]
public String divBlank { get; set; }
[HalfWidth]
public String FullName { get; set; }
....

Thank @luongnv6
but i need something else. I want to view each field in full with [FullWidth]. It works very well in left to right languages like english but when I use switch the language to right to left langues like persian or arabic then the with is automatically changed to HalfWidth or even smaller but not full width.
Thank you


CLE Unit Tests

Codecov Report

Merging #53 (71e9710) into develop (abb792a) will decrease coverage by 2.26%.
The diff coverage is 96.66%.

@@             Coverage Diff             @@
##           develop      #53      +/-   ##
===========================================
- Coverage    80.76%   78.49%   -2.27%
===========================================
  Files           62       69       +7
  Lines         1731     1837     +106
===========================================
+ Hits          1398     1442      +44
- Misses         333      395      +62

Impacted Files
Coverage Δ

Modules/Sources/CV-SwiftUI/CLE/CLE.swift
100.00% <ø> (ø)

Modules/Sources/CV-SwiftUI/CV.swift
0.00% <ø> (ø)

Modules/Sources/Networking/APIClient.swift
84.61% <ø> (+7.69%)
:arrow_up:

...dules/Sources/TCADependencyKeys/APIClientKey.swift
0.00% <ø> (ø)

Modules/Sources/TestHelpers/Stub.swift
46.15% <ø> (-3.85%)
:arrow_down:

...ests/CollectionView/CollectionViewModelTests.swift
100.00% <ø> (ø)

Modules/Sources/TestHelpers/APIClient.swift
50.00% <75.00%> (ø)

Modules/Sources/TestHelpers/Stubs.swift
100.00% <100.00%> (ø)

Modules/Tests/CV-SwiftUITests/CLETests.swift
100.00% <100.00%> (ø)

...sts/CV-UIKitTests/CLE/CLEViewControllerTests.swift
100.00% <100.00%> (ø)

... and 16 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Versioning scheme

The versioning scheme described in the README didn't work out quite as well as I had hoped. The current version scheme assumes that the first two parts of the version are dedicated for Kafka compatibility while the rest of the parts describe SKC versioning. Since the Kafka project decided to make additional incompatible changes in 0.10.1, we'd have to track the extra third part of the version.
Figure out a versioning scheme for SKC that tracks both Kafka compatibility and the client compatibility.

As an example, we could use a versioning scheme like this:
$MAJOR.$MINOR.$PATCH-k$KAFKA

where:

MAJOR, MINOR, and PATCH describe the SKC versioning
KAFKA describes the Kafka compatible version

Examples:
1.2.3-k0.9  => compatible with Kafka 0.9
1.2.3-k0.10 => compatible with Kafka 0.10
1.2.3-k0.10.1 => compatible with Kafka 0.10.1

Note that JARs use lexicographical ordering after the dash, so -k0.9 takes precedence over -k0.10. This could be solved by naming 0.9 compatible version as -k0.09 instead. However, if you ever run into a situation where you pull dependencies for multiple Kafka versions, you should probably standardise to a single Kafka version anyhow.

I think we also need to figure out how far are we going to extend Kafka support.


Resource selector

In case of a "rolling upgrade" of camel-k, where you have multiple version of the camel-k operator running and you want to move integrations from one operator to the another by steps, it may be useful to have an option to configure what resources an operator is suppose to watch by providing a label selector.
i.e. by having a selector resource on the integration platform:
selector:
    camel.apache.com/operator.version: 1.2.3

/cc @astefanutti @nicolaferraro

It seems highly correlated to #1943, if not a duplicate.

yep,  I'm getting old ...


Water sensors PYE (1724914) INDIGO Tyler

no data for a while asking tyler to donwload again.

all available data downloaded at the end of the season.


implementing orders endpoint

[x] LagTime
[x] Reports

@drbarzaga

@pedrohma Please can you create one ticket in the Walmart MWS Project for this PR and complete this information


Intrusive singly-linked list

Forgive my ignorance. What does // + Drop mean?
Like https://github.com/hawkw/alarm/blob/master/intruder_alarm/src/doubly/mod.rs#L58

@amanjeev It means I was considering adding a Drop bound on Linked, realised it wasn't necessary, commented it out, and then forgot to remove it. :)

If you think its worth something, should I open a PR? Please let me know if you have suggestions for further improvements.

@amanjeev if the implementation is done and the tests pass, please open a PR!

@amanjeev if the implementation is done and the tests pass, please open a PR!

Opened https://github.com/hawkw/alarm/pull/35 😨 😃


Dependency Updates

An effort to update all the dependencies in the Uploader to their most recent releases. A large number of them were patch/minor releases and required no code changes. A handful of the major version revision updates also didn't require code changes.
chai update changes comparisons for Error objects from checking that the message property matched to being === referential equivalence, forcing some strange modifications to tests.
removed the d3 dependency as it was only being used to create a test fixture and nothing else (replaced with lodash version of generating the fixture)
used a number of codemods to handle the react updates and the lodash updates
react-router and redux-form both had semi-complete rewrites that severely changed the paradigm of how each of them function and came with a lot of code modification. luckily we were only using redux-form in one place so far and the react-router migration was pretty straightforward (outside the use of react-router-redux, which is arguable as to whether or not we should, but it was still easier to maintain current patterns than to remove it for now)
It's going to be easier to review this one commit at a time.

 #492 ⁃  [uploader] update lodash from 3.x to 4.x

Pinging for review(s). @pazaan can probably adequately review most of the updates, but it will probably be useful to have @cbwebdevelopment to review the React/Redux specific commits like the redux-form and react-router updates along with the react codemod updates. (redux-form: https://github.com/tidepool-org/chrome-uploader/pull/528/commits/bc1eac23c654978b709756efc0d252982434a334, react-router: https://github.com/tidepool-org/chrome-uploader/pull/528/commits/89f6dffcad95ec40fc923edda9a30ad7cd9a9fcb, codemods: https://github.com/tidepool-org/chrome-uploader/pull/528/commits/77e636fab63173cddf5d4de9857d708b67b20833, https://github.com/tidepool-org/chrome-uploader/pull/528/commits/db539a747861d7ed01d0ff283ca54cffe1d8c318, https://github.com/tidepool-org/chrome-uploader/pull/528/commits/c4ab3fc8facb459c9fd549f3a283404de14e0845)


Only apply the abbr substitution in paragraphs

Trello card
https://trello.com/c/BA9vQXqT/2841-restrict-acronym-expander-to-body-copy-only
Context
Any potential abbreviation with a parent element other than  will be
skipped

@peteryates I've checked the following examples and they look good to me:

Ways to train flags
Blue h2 headings (eg on /what-is-a-pgce)

I won't approve this ticket in case you want to get tech eyes on this though - but I'm happy for it to be merged when it's had that tech review


Text Encoder LR Scale for Stable Diffusion Training

Hello! This is a suggested implementation to scale the magnitude of the Text Encoder's learning rate during Stable Diffusion training sessions.
So, I'm another user of the stable diffusion community, and, like many others, I really enjoy training using Prodigy. However, as we know, many people come here to find a solution to the overtraining of the TE, since Prodigy doesn't accept different LRs.
Just so you know, I've been using Kohya's scripts and training LoRAs for SD 1.5.
After analyzing how Prodigy receives and handles tensors, I came up with a trick to reduce the LR of the TE during the .step().
Btw, correct me if I'm wrong, please.
I based this on the premise that TE tensors will always be 2D, so their shape will always have two dimensions.
As the tensors from Kohya come in two groups, using all(len(p.shape) == 2 for p in group["params"]) allowed me to determine which group of tensors belongs to the TE.
With that information, I created a boolean and used an if-else statement for the weight update:
            is_text_encoder = all(len(p.shape) == 2 for p in group["params"])

            ...

                ### Take step
                # and here we apply the new te scale to the weight update
                if is_text_encoder:
                    # scale down the weight update for TE due his higher sensitivity
                    p.data.addcdiv_(exp_avg, denom, value=-(dlr * te_scale))
                else:
                    # standard weight update for other parameters
                    p.data.addcdiv_(exp_avg, denom, value=-dlr)

My approach will only influence the optimizer if the user sets the te_scale in the optimizer's extra arguments before starting the training. Otherwise, te_scale will be 1, and nothing will change in the weight update.

Hi Lucas, thanks for preparing this pull request to improve the performance on SD. I like the idea of supporting different scales for different parts of a network, but ideally we'd want Prodigy to have parameters that make sense for all applications, not just SD. For other networks that may make this hyperparameter a distraction. Unfortunately, I cannot come up with a solution that would be easy to use from kohya-ss/sd-scripts, it seems that your solution makes a lot of sense in that context, though it's not related to the original purpose of this repository.
@adefazio, do you have any thoughts on this?

@konstmish hello! Thanks a lot for the response! I totally get it, and even if it can’t be officially implemented, the people in charge of the SD trainers' repositories can look into finding a way to add it in a custom way, if they think it could be useful. 😊


[Gemma2] Enable Gemma2 Inference on Gaudi

What does this PR do?

This PR continues the work done to enable Gemma2 on Gaudi here https://github.com/huggingface/optimum-habana/pull/1280.
Single Card
python3 run_generation.py --model_name_or_path google/gemma-2-9b --batch_size 1 --use_kv_cache --max_new_tokens 100 --use_flash_attention --use_hpu_graphs --bf16
11/25/2024 11:49:11 - INFO - __main__ - device: hpu, n_hpu: 0, bf16: True
11/25/2024 11:49:11 - INFO - __main__ - Model initialization took 12.165s
11/25/2024 11:49:11 - INFO - __main__ - Graph compilation...
Warming up iteration 1/3
11/25/2024 11:49:12 - INFO - __main__ - Time to first token =<PHONE_NUMBER>9954153ms
Warming up iteration 2/3
11/25/2024 11:49:14 - INFO - __main__ - Time to first token = 18.381826997938333ms
Warming up iteration 3/3
11/25/2024 11:49:15 - INFO - __main__ - Time to first token = 14.948340995033504ms
11/25/2024 11:49:15 - INFO - __main__ - Running generate...
11/25/2024 11:49:16 - INFO - __main__ - Time to first token = 15.03746800153749ms
11/25/2024 11:49:17 - INFO - __main__ - Time to first token = 14.983528999437112ms
11/25/2024 11:49:18 - INFO - __main__ - Time to first token = 15.013975997135276ms
11/25/2024 11:49:19 - INFO - __main__ - Time to first token = 14.970359003200429ms
11/25/2024 11:49:20 - INFO - __main__ - Time to first token = 15.033622999908403ms

Input/outputs:
input 1: ('DeepSpeed is a machine learning framework',)
output 1: ('DeepSpeed is a machine learning framework that enables training of large-scale deep learning models on a single GPU or across multiple GPUs. It is designed to be easy to use and highly scalable, making it a powerful tool for researchers and practitioners working with large-scale deep learning models.\n\nDeepSpeed is built on top of PyTorch, a popular deep learning framework, and provides a set of tools and libraries that make it easy to train large-scale models. It includes features such as zero-shot inference, which allows models to be',)

Stats:
---------------------------------------------------------------------------------
Input tokens
Throughput (including tokenization) = 92.2688307826808 tokens/second
Memory allocated                    = 19.41 GB
Max memory allocated                = 19.42 GB
Total memory available              = 94.62 GB
Graph compilation duration          = 3.9148149259999627 seconds
---------------------------------------------------------------------------------

Multiple Cards with DeepSpeed
python3 ../gaudi_spawn.py --use_deepspeed --world_size 8 run_generation.py --model_name_or_path google/gemma-2-9b --batch_size 1 --use_kv_cache --max_new_tokens 100 --use_flash_attention --use_hpu_graphs
11/25/2024 17:25:15 - INFO - __main__ - device: hpu, n_hpu: 8, bf16: True
11/25/2024 17:25:15 - INFO - __main__ - Model initialization took 42.190s
11/25/2024 17:25:15 - INFO - __main__ - Graph compilation...
Warming up iteration 1/3
11/25/2024 17:25:17 - INFO - __main__ - Time to first token = 1137.7861259970814ms
Warming up iteration 2/3
11/25/2024 17:25:18 - INFO - __main__ - Time to first token = 19.362977996934205ms
Warming up iteration 3/3
11/25/2024 17:25:19 - INFO - __main__ - Time to first token = 13.938947005954105ms
11/25/2024 17:25:19 - INFO - __main__ - Running generate...
11/25/2024 17:25:20 - INFO - __main__ - Time to first token = 13.15586299460847ms
11/25/2024 17:25:21 - INFO - __main__ - Time to first token = 12.668373994529247ms
11/25/2024 17:25:22 - INFO - __main__ - Time to first token = 12.446482003724668ms
11/25/2024 17:25:23 - INFO - __main__ - Time to first token = 11.886986998433713ms
11/25/2024 17:25:24 - INFO - __main__ - Time to first token = 14.2978059957386ms

Input/outputs:
input 1: ('DeepSpeed is a machine learning framework',)
output 1: ('DeepSpeed is a machine learning framework that enables training of large-scale deep learning models on a single GPU or across multiple GPUs. It is designed to be easy to use and highly scalable, making it a popular choice for training large-scale models such as GPT-3 and BERT.\n\nDeepSpeed is built on top of PyTorch, a popular deep learning framework, and provides a set of tools and libraries that make it easy to train large-scale models. It includes features such as zero-shot learning, which allows models to',)

Stats:
-----------------------------------------------------------------------------------
Input tokens
Throughput (including tokenization) =<PHONE_NUMBER>9390004 tokens/second
Memory allocated                    = 5.95 GB
Max memory allocated                = 5.97 GB
Total memory available              = 94.62 GB
Graph compilation duration          = 4.5140280299965525 seconds
-----------------------------------------------------------------------------------

@libinta @emascarenhas @nedo99 this is a PR to replace the original https://github.com/huggingface/optimum-habana/pull/1280 with fixes on style

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.

@Luca-Calabria It seems there is still an issue with the code format.
Can you also add a test here adding the model output there please?

@Luca-Calabria It seems there is still an issue with the code format.
Can you also add a test here adding the model output there please?

@regiss I fixed the format issue and added test case for Gemma2

@Luca-Calabria I just pushed a fix for the text generation test. Can you rebase your branch on main and also add the model output to check the model accuracy here please?

@Luca-Calabria I just pushed a fix for the text generation test. Can you rebase your branch on main and also add the model output to check the model accuracy here please?

@regisss I rebased the PR and added the output check to test suite.

@Luca-Calabria The code quality check probably failed because of a new version of ruff, can you run
pip install -U ruff

to make sure it's updated?

These are the issues found running make style. I think they came from one rebased step because 'modeling_starcoder2.py' is not something I modified in this PR

@regisss please try again

@Luca-Calabria  I only see single configuration (batch1, output token100). Could you also clarify what other configurations have been run with performance data (You can provide more details in HS ticket)?   @libinta

@Luca-Calabria I only see single configuration (batch1, output token100). Could you also clarify what other configurations have been run with performance data (You can provide more details in HS ticket)? @libinta

@jiminha I set up just a single configuration because I intend this test case as a non-regression check between new code versions, and this is also the "standard" configuration used for the others models.
I think I didn't catch you question: Do you mean we should check other configs here? or you are asking what are the benchmarking setup for Gemma2 Inference?


Update jests "Matchers" interface

closes https://github.com/FormidableLabs/enzyme-matchers/issues/318
the Matchers type of definitelytyped's jest package have been updated with https://github.com/DefinitelyTyped/DefinitelyTyped/pull/39243, which causes a compilation error since the interfaces don't match anymore.
See: https://github.com/testing-library/jest-dom/issues/152 or https://github.com/styled-components/jest-styled-components/pull/270
This PR changes the type to match the new one.

as far as i can tell, for custom matchers passed to toMatchSnapshot assertions. not a 100% on that though


dinkum compat initial idea

replace the skill class import and get dinkum skills loading under ovos-core

just a proof of concept, dinkum's fate is obvious now and we wont support it


PIXI.Rectangle should have intersects method, but doesn't?

Here's a rectangle I made, note the methods it has do not include intersects, but the pixi.js documentation lists it as a method it should have. Am I doing something wrong?

But then this other version of the doc doesn't have intersects, so I'm confused. Both seem to be v6.2.2?

Hi @Heilemann,
Please file an issue in the Pixi.js repo, this is related to Pixi itself (not this library).
I'll close this issue, please let me know if you think we should reopen it.
Thanks!


Failed to save slice when filters field is empty

Superset failed to save slice/chart when filters were empty. I got around it by setting an artificial filter with some_field IS NOT NULL where I knew some_field will not be null.
Expected results
Expect slice/chart to be saved successfully.
Actual results
"Failed to save slice" in GUI, but only after opening the Save modal again. No feedback at all until the modal is opened.
Logs:
Traceback (most recent call last):
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/flask/app.py", line 2447, in wsgi_app
    response = self.full_dispatch_request()
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/flask/app.py", line 1952, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/flask/app.py", line 1821, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/flask/_compat.py", line 39, in reraise
    raise value
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/flask_appbuilder/security/decorators.py", line 109, in wraps
    return f(self, *args, **kwargs)
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/superset/utils/log.py", line 224, in wrapper
    value = f(*args, **kwargs)
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/superset/views/core.py", line 781, in explore
    datasource.name,
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/superset/views/core.py", line 886, in save_or_overwrite_slice
    form_data.get("adhoc_filters", [])
  File "/home/fedora/superset/venv/lib64/python3.7/site-packages/superset/views/core.py", line 862, in remove_extra_filters
    return [f for f in filters if not f.get("isExtra")]
TypeError: 'NoneType' object is not iterable
eval - with context - False, False

How to reproduce the bug
I am not sure why this does not happen every time. I know that I was using the Time series table chart when it happened. It happened both with physical and virtual datasets.
Environment
(please complete the following information):

superset version: 1.1.0
python version: 3.7.10

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Working as Expected in the latest version of Superset 2.1.0.
Loom video:-
https://www.loom.com/share/e021697ce8054ec789ae1a58e3f3823f


Support string escaping

We should support string escaping in the grammar language:

Add \ escaping to the STRING rule in langium-grammar.langium
Process escaped characters in the convertString method of DefaultValueConverter
Adapt the syntax highlighting config of the grammar language to support escaped characters

Discussed in https://github.com/langium/langium/discussions/884

Originally posted by ballcoach12 January 26, 2023
The following rule generates an error in the grammar editor:
terminal S_BYTE_CHAR_STR returns string:
 	'\'' S_BYTE_CHAR_VALUE* '\''
 ;

The second backslash causes the following error: Unexpected character value: ->\<-.   Do I need to escape this character somehow?

See also #797, which is about including escaped characters in the generated syntax highlighting. Shall we try to detect whether the terminal rule supports escaped characters? Or add that to syntax highlighting without checking?


How to implement wall switches?

But because Siri is great for activating special scenes or disable specific lights but not so much for quick all lights on/off actions, I am trying to equip my former 230 volts switches with small battery-driven transmitters. Their signals should be received from a raspberry pi which is running HAP-NodeJS.
But thats the problem, I think the StatelessProgrammableSwitch service is what I need here? As I was not able to get any code running with this service, I looked for reference implementations, but found nothing. So I tried the MotionSensor accessory as a possible substitute, which comes with HAP-NodeJS, but even this code seems not to work?
I tried the GarageDoorOpener to make sure it's not a general problem, which worked well.
My questions are:

Are there any known issues with the MotionSensor?
Has anybody a working example of the StatelessProgrammableSwitch?

I would be thankful for any help/code/ideas :)

Lukas
Take a look at the homebridge plugins available, there should be several available that could be used as base for what you are trying to do.

But because Siri is great for activating special scenes or disable specific lights but not so much for quick all lights on/off actions, I am trying to equip my former 230 volts switches with small battery-driven transmitters. Their signals should be received from a raspberry pi which is running HAP-NodeJS.
But thats the problem, I think the StatelessProgrammableSwitch service is what I need here? As I was not able to get any code running with this service, I looked for reference implementations, but found nothing. So I tried the MotionSensor accessory as a possible substitute, which comes with HAP-NodeJS, but even this code seems not to work?
I tried the GarageDoorOpener to make sure it's not a general problem, which worked well.
My questions are:
Are there any known issues with the MotionSensor?
Has anybody a working example of the StatelessProgrammableSwitch?
I would be thankful for any help/code/ideas :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I use them extensively in homebridge-hue.  Not the simplest plugin, though.
You need to expose an accessory with a single ServiceLabel service and a Stateless Programmable Switch service per button.  Note that you can only use them to trigger HomeKit automations.  You need a Home hub (AppleTV, HomePod, or mains-powered iPad) for the automations.

@NorthernMan54 Thanks I'll have a look:) homebride-hue seems to be a good starting point
@ebaauw I've read trough your code and tried to extract the parts for the switch, I came up with the following:
var Accessory = require("../").Accessory;
var Service = require("../").Service;
var Characteristic = require("../").Characteristic;
var uuid = require("../").uuid;

var lightswitchUUID = uuid.generate("hap-nodejs:accessories:" + "LightSwitch");

var lightswitch = (exports.accessory = new Accessory(
  "Light Switch",
  lightswitchUUID
));

lightswitch.username = "C1:2E:3F:D2:5E:FA";
lightswitch.pincode = "031-45-154";

lightswitch
  .getService(Service.AccessoryInformation)
  .setCharacteristic(Characteristic.Manufacturer, "Default Manufacturer")
  .setCharacteristic(Characteristic.Model, "0.1")
  .setCharacteristic(Characteristic.SerialNumber, "001");

lightswitch.on("identify", function(paired, callback) {
  console.log("Identify: fake light switch");
  callback();
});

const labelService = new Service.ServiceLabel("Light Switch");

labelService
  .getCharacteristic(Characteristic.ServiceLabelNamespace)
  .setValue(Characteristic.ServiceLabelNamespace.ARABIC_NUMERALS);

lightswitch.addService(labelService);

const SPSService = new Service.StatelessProgrammableSwitch(
  "Light Switch Button",
  "Button"
);

SPSService.getCharacteristic(Characteristic.ProgrammableSwitchEvent).setProps({
  minValue: Characteristic.ProgrammableSwitchEvent.SINGLE_PRESS,
  maxValue: Characteristic.ProgrammableSwitchEvent.SINGLE_PRESS
});
SPSService.getCharacteristic(Characteristic.ServiceLabelIndex).setValue(1);
SPSService
  .getCharacteristic(Characteristic.ProgrammableSwitchEvent)
  .updateValue(false);
lightswitch.addService(SPSService);

But unfortunately after a few moments the switch dissapears from homekit with the following log output:
EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP request: /pairings +181ms
HAPServer [C1:2E:3F:D2:5E:FA] HAP Request: POST /pairings +0ms
HAPServer [C1:2E:3F:D2:5E:FA] Removing pairing info for client +0ms
Accessory [Light Switch] Unpairing with client DBB219C6-BA46-4F56-92F5-B87999722136 +1ms
EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP Response is finished +1ms
EventedHTTPServer [::ffff:<IP_ADDRESS>] Client connection closed +140ms
EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP connection was closed +0ms
EventedHTTPServer [::ffff:<IP_ADDRESS>] HTTP server was closed +0ms

Any ideas whats going on here?

Looks like HomeKit has banished the accessory for having an illegal characteristic value.  You should only set the value of the Programmable Switch Event to Characteristic.ProgrammableSwitchEvent.SINGLE_PRESS.  And only when the button is pressed.
You might need to wait a couple of weeks before HomeKit will again accept the accessory.  Or change the username.


refactor all controllers with yacht

What type of PR is this?
What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (e2ffa5f) 13.60% compared to head (ecba2f8) 13.10%.
Report is 114 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #757      +/-   ##
==========================================
- Coverage   13.60%   13.10%   -0.50%
==========================================
  Files          66       76      +10
  Lines        7492     8673    +1181
==========================================
+ Hits         1019     1137     +118
- Misses       6391     7453    +1062
- Partials       82       83       +1

see 50 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Allow a custom host to be set

Some of the requests we make from the server need to overwrite the host header (e.g. for ‘apis’ which were originally written with a CDN in mind and we're just making requests that act like a CDN) and node-fetch currently chokes if you try to set a custom host header, check out e98f22b3ba8350a37c6f3c8917df56655e5c4fc7, run npm test and you'll get this output:-
  1) node-fetch should allow setting of custom host:
     TypeError: undefined is not a function
      at Agent.createSocket (_http_agent.js:186:39)
      at Agent.addRequest (_http_agent.js:166:23)
      at new ClientRequest (_http_client.js:154:16)
      at exports.request (http.js:49:10)
      at /Users/mandrews/sandboxes/node-fetch/index.js:83:13
      at new Fetch (/Users/mandrews/sandboxes/node-fetch/index.js:44:9)
      at Fetch (/Users/mandrews/sandboxes/node-fetch/index.js:32:10)
      at Context.<anonymous> (/Users/mandrews/sandboxes/node-fetch/test/test.js:763:10)
      at callFn (/Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runnable.js:251:21)
      at Test.Runnable.run (/Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runnable.js:244:7)
      at Runner.runTest (/Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runner.js:374:10)
      at /Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runner.js:452:12
      at next (/Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runner.js:299:14)
      at /Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runner.js:309:7
      at next (/Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runner.js:248:23)
      at Immediate._onImmediate (/Users/mandrews/sandboxes/node-fetch/node_modules/mocha/lib/runner.js:276:5)
      at processImmediate [as _immediateCallback] (timers.js:358:17)

I'm not entirely happy with my hack but it seemed the least intrusive way to make this work :disappointed:.  Very happy to tweak the implementation if you can think of a better way.

I don't think the travis failure is a real failure :confused:

I am trying to understand this issue:

So node.js http/https module can't handle a host header that's an array (fair enough), which our Headers interface are using by default when you set a header.
And you need to send test request to domain A while using a host header of domain B.
And this is a server-side only feature, because fetch on client-side shouldn't allow users to set custom host header (it's a forbidden header).

All True?
An alternative solution would be to use String instead of Array when storing headers internally (unless the input is an Array), so headers.raw() return host header as a string by default.
Any other case that might warrant such a change? If not, perhaps this hack has a lower risk of breaking things...

Yes all true…

Yes, it appears so — at least on the versions of Node I'm using (v0.12.x).  Actually I couldn't find any docs suggesting that array was supported (although it seems to be in most cases).
Yes.
Yes that's correct, but this is equivalent to node-fetch's ability to allow a custom user agent to be set so I don't think it would be wrong for node-fetch to support this.

An alternative solution would be to use String instead of Array when storing headers internally (unless the input is an Array), so headers.raw() return host header as a string by default.
Any other case that might warrant such a change? If not, perhaps this hack has a lower risk of breaking things...

I couldn't think of a use case where you might want to set multiple values for the same header, but as you say, since it's supported already fixing it for the host case only lowers the risk.  I'm really not that familiar with http……
A quick glance at the spec suggests the following:-

Multiple message-header fields with the same field-name MAY be present in a message if and only if the entire field-value for that header field is defined as a comma-separated list [i.e., #(values)]. It MUST be possible to combine the multiple header fields into one "field-name: field-value" pair, without changing the semantics of the message, by appending each subsequent field-value to the first, each separated by a comma. The order in which header fields with the same field-name are received is therefore significant to the interpretation of the combined field value, and thus a proxy MUST NOT change the order of these field values when a message is forwarded.

I wonder if that's what node's http(s) request module does internally:-
headers: {
  'my-header': [ 'a', 'b' ]
}

=>
curl … -H 'my-header: a, b'

^^ That does seem to be the case…
webserver.js
require('http').createServer(function(request, response){
        console.log("my-header: " + request.headers['my-header']);
        console.log("my-other-header: " + request.headers['my-other-header']);
}).listen(8080);

test.js
var req = require('http').request({
        host: 'localhost',
        port: 8080,
        method: 'POST',
        path: '/',
        headers: {
                'my-header': ['a','b'],
                'my-other-header': ['a, b']
        }
});
req.write('');
req.end();

Output of webserver:-
my-header: a, b
my-other-header: a, b

So I think it would be safe If I changed my implementation to something like:-
options.headers = Object.keys(options.headers).map(function(header) {
    return options.headers[header].join(', ');
  });

Does that look ok to you?

ok that code doesn't work hold on……

                options.headers = headers.raw();

                options.headers = Object.keys(options.headers).reduce(function(previousHeaders, header) {
                                previousHeaders[header] = options.headers[header].join(', ');
                                return previousHeaders;
                        }, {});

Not sure how to feel about your last snippet, to me your original PR is better, as it's easier to figure out why we make such change :)
I have searched around on github and found a few users already using the internal _headers store and expect Array type on each header, so I think your hack is the best way forward.

okay let's go with that :)

Although my last snippet doesn't change anything about the internal _headers store …… I'm proposing putting that into index.js just before the options object gets passed into send.

@matthew-andrews got it, i was refering to my alternative solution to change Headers. Also looping around the all headers just to workaround host header seem like a bad deal.

ahh ok :)

looping around the all headers just to workaround host header seem like a bad deal

:+1: to this
Thanks for your help!

Definitely won't want to change the headers interface — I think it currently matches the spec perfectly:-
https://developer.mozilla.org/en-US/docs/Web/API/Headers/get
https://developer.mozilla.org/en-US/docs/Web/API/Headers/getAll

v1.3.1 release include this workaround.

thanks!


Segmentation fault when using make_batch_reader

Hi!
I am trying to read a non-Petastorm parquet file in pytorch but just get the error:

Segmentation fault (core dumped)

The code I am using is from examples/hello_world/external_datasets/pytorch_hello_world.py:
`
with DataLoader(make_batch_reader(file_path_to_my_parquet_file)) as train_loader:
    sample = next(iter(train_loader))
    print("id batch: {0}".format(sample['id']))

`
Any idea why this might be happening?
Thanks,
Janhavi

Good chance it is because of this: https://issues.apache.org/jira/browse/ARROW-5130. We saw it happening with pyarrow 0.13.0. It will be fixed in the next release of pyarrow. For now, I would recommend downgrading to pyarrow 0.12.1 if possible.

Downgrading the pyarrow version worked. Thanks!


remove hard version requirement for ffmpeg

AFAIK FFmpeg has a rather stable API so this shouldn't be a problem until FFmpeg 5 is released


[Feature-10363][server] S3 Resource center supports bucket customization

Modify the configuration. Sort out HDFS and S3 configurations to distinguish them
Modify the configuration comment to clarify the configuration meaning
Modify official documents to add missing configuration descriptions and clarify configuration meanings and usage
Modify the configuration file in e2E to the correct definition, dolphinscheduler-e2e/dolphinscheduler-e2e-case/src/test/resources/docker/file-manage/common.properties
Modified code to support bucket customization
Check that the bucket on personal permission, Otherwise, the startup fails
S3 increase support resource. Storage. Upload. Base. The base path path
Change S3Utils implementation style to be consistent with HadoopUtils

close #10363

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #10433 (c4e19a0) into dev (f0131d5) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev   #10433      +/-   ##
============================================
- Coverage     40.53%   40.53%   -0.01%
- Complexity     4772     4773       +1
============================================
  Files           877      878       +1
  Lines         35698    35756      +58
  Branches       3969     3970       +1
============================================
+ Hits          14470    14492      +22
- Misses        19785    19821      +36
  Partials       1443     1443

Impacted Files
Coverage Δ

.../org/apache/dolphinscheduler/common/Constants.java
80.95% <ø> (ø)

...che/dolphinscheduler/common/utils/HadoopUtils.java
5.88% <0.00%> (ø)

.../apache/dolphinscheduler/common/utils/S3Utils.java
0.00% <0.00%> (ø)

...olphinscheduler/plugin/task/api/TaskConstants.java
0.00% <ø> (ø)

...lphinscheduler/rpc/common/AbstractRpcCallBack.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
48.21% <0.00%> (ø)

...egistry/zookeeper/ZookeeperRegistryProperties.java
53.48% <0.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 f0131d5...c4e19a0. Read the comment docs.

@caishunfeng @zhongjiajie PTAL. Thanks.

merging...


Change android assets names

Motivation
https://github.com/facebook/react-native/issues/16263
Test Plan
First run the android emulator

git clone https://github.com/vovanmozg/react-native-test-android-assets-paths.git
cd ./react-native-test-android-assets-paths
yarn
patch node_modules/react-native/local-cli/bundle/assetPathUtils.js android_filenames.patch
cd android && ./gradlew assembleRelease

You should see image in the center of test app screen.
Release Notes
[CLI] [BUGFIX] [./local-cli/bundle/assetPathUtils.js] - Change android assets names representation

Something seems wrong with your PR.
Did you recently rebase your changes? You probably just need 67ba7d5937d643ca654cfed2712192ee8a41596c.

@charpeni I rebased changes, please check it.

Can we use a sha1 hash using node's crypto module instead to avoid the extra dep? That's what we use for hashes in metro-bundler https://github.com/facebook/metro-bundler/blob/b08ce130fdd8bd455ec2f0528a597ac986866de4/packages/metro-bundler/src/lib/TransformCaching.js#L139.

@janicduplessis I think it might be used node's crypto module. Filenames must be unique and this looks suitable.

@janicduplessis

Can we use a sha1 hash using node's crypto module instead to avoid the extra dep?

Unfortunately, no.
getAndroidResourceIdentifier is used in react-native/Libraries/Image/AssetSourceResolver, so that it runs inside mobile JSCore, not in node environment
Also, we can't use node.js core modules inside JSCore 'as is' without any mubo-jumbo-workarounds.
https://medium.com/@aakashns/using-core-node-js-modules-in-react-native-apps-64acd4d07140
There are many related issues here, but none of them are yet resolved in general.
https://github.com/facebook/react-native/issues/1871
https://github.com/facebook/react-native/issues/6253
So I see only three ways here (to resolve this issue, of course):

fix related issues to use node.js core modules (a long way, but best of all)
For ex., https://github.com/facebook/react-native/issues/1871#issuecomment-235667345 is one workaround which looks not so bad as many others are

use new external dependency in react-native

copy-paste any sha1 / md5 / etc as inplace function / nearby import and maintain it (not so good as new external deps, but almost same way)

Last (and worse of all) way - leave it "as is". Deeply nested component structure sometimes adds PITA to bundle/deploy process.
Thoughts?

Warnings

:warning:

:lock: package.json - Changes were made to package.json. This will require a manual import by a Facebook employee.

  Generated by :no_entry_sign: dangerJS

This PR has been open for a long time without resolution. We just moved this code to https://github.com/react-native-community/react-native-cli so please submit your PR there.


Linux font rendering issue in dialogues

issue description
The font/text in dialogues is rendered incorrectly or one could say "fuzzy". When the app is rebuild (for example by resizing the window) the next time the text is rendered correctly.
recording
https://user-images.githubusercontent.com/15329494/184888627-816c0351-49b1-4df2-9b8a-6b67dd4d8f3d.mp4
minimal code example
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return const MaterialApp(
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key});

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: TextButton(
            onPressed: () => showDialog(
                  context: context,
                  builder: (context) => const AlertDialog(
                    title: Text('Title'),
                    content: Text(
                        'text text  text  text  text  text text  text  text  text  text  text  text'),
                  ),
                ),
            child: const Text('Open Dialog')),
      ),
    );
  }
}

info about my environment
flutter
frederik@karbon:~$ flutter --version
Flutter 3.1.0-0.0.pre.2294 • channel master •
https://github.com/flutter/flutter.git
Framework • revision 370fc483e3 (vor 6 Stunden) • 2022-08-16 08:56:07 +0200
Engine • revision 280e9a7bec
Tools • Dart 2.19.0 (build 2.19.0-100.0.dev) • DevTools 2.16.0
Hardware
Processor: 12th Gen Intel(R) Core(TM) i7-1260P x16
Memory: 31 Gb
Graphics: Intel Corporation
Disk Capacity: 954 GB
System
OS: Ubuntu 22.04 (64-bit)
Kernel version: 5.15.0-46-generic
GNOME version: 42.2.0
Windowing System: wayland

@jonahwilliams, the issue seems to be caused by https://github.com/flutter/engine/pull/34115 and is likely related to RasterCache. The following hack helps to avoid the rendering artifact:
diff --git a/flow/raster_cache.h b/flow/raster_cache.h
index f3dbac5b7e..ba91cac388 100644
--- a/flow/raster_cache.h
+++ b/flow/raster_cache.h
@@ -152,9 +152,9 @@ class RasterCache {
   static SkRect GetDeviceBounds(const SkRect& rect, const SkMatrix& ctm) {
     SkRect device_rect;
     ctm.mapRect(&device_rect, rect);
-#ifndef SUPPORT_FRACTIONAL_TRANSLATION
+// #ifndef SUPPORT_FRACTIONAL_TRANSLATION
     device_rect.roundOut(&device_rect);
-#endif
+// #endif
     return device_rect;
   }

@jonahwilliams Correct in what sense? It's not a fix proposal but It was just a quick test to see which part of https://github.com/flutter/engine/pull/34115 may cause the rendering issue.

Oh I see, I'm sorry I thought that diff was meant to apply to ToT.

I'd be surprised if that dialog entry were in the raster cache, that said we might be using that method elsewhere.

Oh I see, I'm sorry I thought that diff was meant to apply to ToT.

My bad, I forgot to mention the little detail that I had d30c460362 checked out after bisecting. Interestingly, RasterCacheUtil::GetDeviceBounds() does no longer have SUPPORT_FRACTIONAL_TRANSLATION.

So I suspect what is happening here, is that if you happen to draw at something like 0,0 or another pixel aligned boundary, and your DPR is a whole number (like 1.0), then the effect of pixel snapping the layers does what you'd expect and pixel aligns your text.
I believe if you draw your text at something that isn't pixel aligned already, you'll still look pixelated. Worse, with the pixel snapping the engine used to do, its not really possible to compute where you would be laid out so that you can pixel align yourself.
An approach to solve this problem could be something like a "smart" canvas that aligns certain drawning operations (like text) to physical pixel boundaries when painted. That would avoid the problems that the layer based pixel snapping caused like instability in animations and unpredicatable layout.

Here's a couple of screencasts @Feichtmeier shared earlier where things shift around a bit when being hovered:
https://user-images.githubusercontent.com/140617/184940435-e6c76c01-dbed-4c6e-9dc1-2e95ec335354.mp4
https://user-images.githubusercontent.com/140617/184940390-d456e415-9d3e-43a5-9416-c9e3ff5577d9.mp4

hmm, that looks different

Hi @Feichtmeier, thanks for filing the issue. Unfortunately, I am unable to reproduce that issue on the latest stable or the master channel.
https://user-images.githubusercontent.com/31410839/185326638-44c2b0d5-aa8e-4cfa-9001-0a77a5328f3a.mp4
Labeling it as a bug since the ubuntu team was able to reproduce it.

flutter doctor -v (Linux)
[✓] Flutter (Channel stable, 3.0.5, on Ubuntu 20.04.4 LTS 5.13.0-48-generic, locale en_GB.UTF-8)
    • Flutter version 3.0.5 at /home/mahesh/Development/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision f1875d570e (5 days ago), 2022-07-13 11:24:16 -0700
    • Engine revision e85ea0e79c
    • Dart version 2.17.6
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/mahesh/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at:
      /home/mahesh/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2021.2)
    • Android Studio at /home/mahesh/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683
    • Flutter plugin version 67.1.2
    • Dart plugin version 212.5744
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1)
    • IntelliJ at /home/mahesh/.local/share/JetBrains/Toolbox/apps/IDEA-U/ch-0/221.5921.22
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.68.0)
    • VS Code at /usr/share/code
    • Flutter extension version 3.44.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 20.04.4 LTS 5.13.0-48-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 102.0.5005.115

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

[✓] Flutter (Channel master, 3.1.0-0.0.pre.2346, on Ubuntu 20.04.4 LTS 5.15.0-41-generic, locale en_GB.UTF-8)
    • Flutter version 3.1.0-0.0.pre.2346 on channel master at /home/mahesh/Development/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision adda088bcc (4 hours ago), 2022-08-17 22:59:07 -0400
    • Engine revision a77ead45a8
    • Dart version 2.19.0 (build 2.19.0-108.0.dev)
    • DevTools version 2.16.0

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /home/mahesh/Android/Sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /home/mahesh/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 10.0.0-4ubuntu1
    • cmake version 3.16.3
    • ninja version 1.10.0
    • pkg-config version 0.29.1

[✓] Android Studio (version 2021.2)
    • Android Studio at /home/mahesh/.local/share/JetBrains/Toolbox/apps/AndroidStudio/ch-0/212.5712.43.2112.8609683
    • Flutter plugin version 67.1.2
    • Dart plugin version 212.5744
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] IntelliJ IDEA Ultimate Edition (version 2022.1)
    • IntelliJ at /home/mahesh/.local/share/JetBrains/Toolbox/apps/IDEA-U/ch-0/221.5921.22
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.69.2)
    • VS Code at /usr/share/code
    • Flutter extension version 3.46.0

[✓] Connected device (2 available)
    • Linux (desktop) • linux  • linux-x64      • Ubuntu 20.04.4 LTS 5.15.0-41-generic
    • Chrome (web)    • chrome • web-javascript • Google Chrome 103.0.5060.134

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

@maheshmnj Is it a VM?
I haven't been able to reproduce the issue in a VM but the issue does easily reproduce on two different laptops with native Ubuntu 22.04 and 22.10 installations. One of the laptops has a high DPI screen and the issue was at first only visible on an external display with 100% scale but not on the built-in display with 200% scale. Setting the built-in display scale to 100% to get DPR=1 made the issue visible on that display as well.
https://user-images.githubusercontent.com/140617/185346430-858105f1-42ba-4556-89f1-69a43c41aa33.mp4
main.dart
```dart
import 'package:flutter/material.dart';
void main() => runApp(const MaterialApp(home: MyHomePage()));
class MyHomePage extends StatefulWidget {
const MyHomePage({super.key});
@override
State createState() => _MyHomePageState();
}
class MyHomePageState extends State {
@override
void initState() {
super.initState();
WidgetsBinding.instance.addPostFrameCallback(() => _showTestDialog());
}
Future _showTestDialog() {
return showDialog(
context: context,
builder: (context) => const AlertDialog(
content: Text('Text', style: TextStyle(fontSize: 48)),
),
);
}
@override
Widget build(BuildContext context) {
return Scaffold(
body: Center(
child: TextButton(
onPressed: _showTestDialog,
child: const Text('Open Dialog'),
),
),
);
}
}
</details>

@maheshmnj Is it a VM?

No, Ubuntu is installed on a physical machine. Could be an issue with device resolution, I will see if changing resolution makes a difference.

Toggling overlay guidelines or baselines at runtime forces the content to get re-rendered correctly.
Set debugPaintBaselinesEnabled = false on app start-up and toggle Show baselines on from DevTools:

Start-up
Toggle

Set debugPaintBaselinesEnabled = true on app start-up and toggle Show baselines off from DevTools:

Start-up
Toggle

FYI, https://github.com/flutter/flutter/issues/107733 might be a duplicate.

@jonahwilliams What about something like this? Before, RasterCacheUtil::GetDeviceBounds() would round out when fractional translation was disabled. That logic has been now removed which is presumably fine for all other uses of RasterCacheUtil::GetDeviceBounds(), but RasterCache probably should still round out so that an integer-sized texture covers full pixels on all edges, right?
diff --git a/flow/raster_cache.cc b/flow/raster_cache.cc
index 8a4e46d58d..69d2b62f15 100644
--- a/flow/raster_cache.cc
+++ b/flow/raster_cache.cc
@@ -34,6 +34,7 @@ void RasterCacheResult::draw(SkCanvas& canvas, const SkPaint* paint) const {

   SkRect bounds =
       RasterCacheUtil::GetDeviceBounds(logical_rect_, canvas.getTotalMatrix());
+  bounds.roundOut(&bounds);
 #ifndef NDEBUG
   // The image dimensions should always be larger than the device bounds and
   // smaller than the device bounds plus one pixel, at the same time, we must
@@ -68,6 +69,7 @@ std::unique_ptr<RasterCacheResult> RasterCache::Rasterize(
   SkRect dest_rect =
       RasterCacheUtil::GetDeviceBounds(context.logical_rect, context.matrix);
   // we always round out here so that the texture is integer sized.
+  dest_rect.roundOut(&dest_rect);
   int width = SkScalarCeilToInt(dest_rect.width());
   int height = SkScalarCeilToInt(dest_rect.height());

For example, for bounds like [-52,011993,-19,500000, 930,687988x477,000000] (XYWH), the rounded-up size is 931x477 but it needs the size rounded-out to 932x478 to cover all pixels.

I pushed a fix proposal: https://github.com/flutter/engine/pull/35538

This is issue is not present in Flutter 3.13+, I am pretty sure in saying it is fixed :)


feat: Support node loaders

Add --register option.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

11 of 52 (21.15%)  changed or added relevant lines in 3 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 16.72%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/perf-insight/src/findFiles.mts
1
2
50.0%

packages/perf-insight/src/app.mts
10
29
34.48%

packages/perf-insight/src/runBenchmarkCli.mts
0
21
0.0%

Files with Coverage Reduction
New Missed Lines
%

packages/perf-insight/src/runBenchmarkCli.mts
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
143

Relevant Lines:
924

💛  - Coveralls


LeNet5 and VGGNets cannot be exported to ONNX format

LeNet5 network (described here https://github.com/QDucasse/nn_benchmark/blob/master/nn_benchmark/networks/quant_lenet5.py) fails to export to ONNX from within the provided Docker container. The trace stack is the following:
Traceback (most recent call last):
  File "nn_benchmark/core/exporter.py", line 72, in <module>
    bo.export_finn_onnx(lenet5, (1,3,32,32), export_path_lenet5)
  File "/workspace/brevitas/brevitas/onnx/__init__.py", line 103, in export_finn_onnx
    torch.onnx.export(module, input_t, export_path, **torch_onnx_kwargs)
  File "/opt/conda/lib/python3.6/site-packages/torch/onnx/__init__.py", line 25, in export
    return utils.export(*args, **kwargs)
  File "/opt/conda/lib/python3.6/site-packages/torch/onnx/utils.py", line 131, in export
    strip_doc_string=strip_doc_string)
  File "/opt/conda/lib/python3.6/site-packages/torch/onnx/utils.py", line 369, in _export
    strip_doc_string)
RuntimeError: ONNX export failed: Couldn't export Python operator <python_value>

Defined at:
@torch.jit.script_method
def forward(self, value: torch.Tensor) -> torch.Tensor:
    value = self.forward_impl(value)
            ~~~~~~~~~~~~ <--- HERE
    return value

Graph we tried to export:
graph(%0 : Float(1, 3, 32, 32),
      %feature_extractor.0.weight : Float(6, 3, 5, 5),
      %feature_extractor.3.weight : Float(16, 6, 5, 5),
...

Full stack trace here: https://pastebin.com/Z5t4JU5D

The same type of error is noticeable when trying to export VGG (11/13/16/19 - https://github.com/QDucasse/nn_benchmark/blob/master/nn_benchmark/networks/quant_vggnet.py):
Traceback (most recent call last):
  File "nn_benchmark/core/exporter.py", line 74, in <module>
    bo.export_finn_onnx(vgg11, (1,3,32,32), export_path_vgg11)
  File "/workspace/brevitas/brevitas/onnx/__init__.py", line 103, in export_finn_onnx
    torch.onnx.export(module, input_t, export_path, **torch_onnx_kwargs)
  File "/opt/conda/lib/python3.6/site-packages/torch/onnx/__init__.py", line 25, in export
    return utils.export(*args, **kwargs)
  File "/opt/conda/lib/python3.6/site-packages/torch/onnx/utils.py", line 131, in export
    strip_doc_string=strip_doc_string)
  File "/opt/conda/lib/python3.6/site-packages/torch/onnx/utils.py", line 369, in _export
    strip_doc_string)
RuntimeError: ONNX export failed: Couldn't export Python operator <python_value>

Defined at:
@torch.jit.script_method
def forward(self, value: torch.Tensor) -> torch.Tensor:
    value = self.forward_impl(value)
            ~~~~~~~~~~~~ <--- HERE
    return value

Graph we tried to export:
graph(%0 : Float(1, 3, 32, 32),
      %features.0.weight : Float(64, 3, 3, 3),
      %features.1.weight : Float(64),
...

Full stack trace here: https://pastebin.com/A9kC03jm

In order to export the different models, I modified the Docker initialization by adding my repository (https://github.com/QDucasse/nn_benchmark) in the mix and run it from there. I ran the script in https://github.com/QDucasse/nn_benchmark/blob/master/nn_benchmark/core/exporter.py
The quantized version of the MobilenetV1 can be exported without any issues with the same script and the corresponding network

Hey Quentin, thanks for reporting this. Since these are Brevitas examples, we'd like the export to be covered by the FINN test suite so let's start by adding export tests for these networks similar to this:
https://github.com/Xilinx/finn/blob/dev/tests/brevitas/test_brevitas_cnv.py
Could you make a new fork from the finn dev branch and add tests similar to the one above?

I'm closing this issue due to inactivity. Please feel free to reopen, or discuss further on the FINN gitter channel on https://gitter.im/xilinx-finn/community


Fix admin monitoring margin

Before:

After:

Fomantic CSS:
.ui.table {
  margin: 1em 0;
}
.ui.table:first-child {
  margin-top: 0;
}

Because the table is not the first child of its parent, that margin removal did not work. Fix by moving other children after the table.

:rocket:


s3 documentation error

https://github.com/seaweedfs/seaweedfs/wiki/Production-Setup
Setup S3 API
Follow Amazon S3 API to generate a json config file, to assign accessKey and secretKey for different identities, and give read/write permissions to different buckets.
Start s3 together with filer. This avoids the setup for s3 to support multiple filers.
weed filer -s3 -s3.config=<config.json> -port=8333
The endpoint is http://<s3_server_host>:8333.
Correct writing
weed filer -s3 -s3.config=<config.json> -s3.port=8333

fixed. You can actually modify wiki page directly.


Introduce soap action error handling, part 2.1

Description
This this part 2.1 of PR series which suppose to add handling of soap failures.
In scope:

add temporary test which should cover invalid response from sii.
response co-product creation via static helpers

Fixes RND-80626
Type of change

[ ] Bug fix.
[ ] Feature.
[x] Consolidation.

Checklist

[ ] Is breaking change.
[ ] Documentation updated.
[ ] Tests included (please specify cases).

@KaliCZ tested?

@TashaBubo Can you actually exclude PRs from this repository from your board completely?


Flipboard for iOS

Can we have the same api used for flipboard for iOS.
We are searching for it from past one year. But couldn't find the best one like flipboard.
Regards,
Kartheek

it would be helpful for me as well
:+1:

:+1:

Hello, I use UIPageViewController.
http://www.techotopia.com/index.php/An_Example_Swift_iOS_8_UIPageViewController_Application
I hope that help you


chore: update docs with links

Thanks, James!

:rocket: This pull request is included in v5.30.2. See Release 5.30.2 for release notes.
Using Knip in a commercial project? Please consider becoming a sponsor.


Tokens balances and transactions not showing

Hi,
A few days back, my Defichain wallet encountered a "node is disconnected error".
Since then I tried to 1. reindex (failed) and also 2. reinstall the latest wallet.
At present, after fully synchronized (100%), I still cannot see my token balances and transactions
Pls kindly advise further
Thanks
App Log -2021211.zip

Hi @S555O , are you on v2.8.1? can you try RESCAN via settings.

Yes, have tried this method before and just did it again. Not showing addresses still

Hi @S555O , can you try adding your address back manually? Toggle off the Automatically generate new address and paste your address in.

May I where can I access this page? Under settings?
Where can I find out my addresses?

I went on to restore my wallet on another laptop and it worked. Thanks for your help

Hi @S555O , thanks for the update. Glad your issue has been resolved. You may also switch to mobile Light Wallet, does not require running a full node wallet. You may ask the community on how to use / port over / etc. on DeFiChain Telegram (EN). Thank you.


Change log level type

Why

Using string can be easier to match with other services.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.56%. Comparing base (b8a19be) to head (2ed75b4).

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   98.56%   98.56%
=======================================
  Files           9        9
  Lines         415      415
=======================================
  Hits          409      409
  Misses          4        4
  Partials        2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Try to send ERROR messages to the apropriate promise/future instead of letting it bubble up the io_service

Route most ERROR messages  not just CALL to the correct promises/futures. If they can't be passed to a sensible place at least pass the error message inside the thrown exception.
Not sure about using protocol_error in https://github.com/crossbario/autobahn-cpp/compare/master...Bigpet:errorhandling?expand=1#diff-20840b85274d7d0fe386dfa20530582fR972 but I was too lazy to think of something more fitting.

Thanks for that patch!


Begin phase 0 spec implementation

Primarily all of the "helper" functions listed in the phase 0 spec

NOTE: CI failed due to "unused functions" warnings from clippy -- going to merge as there is more on top of this PR but want to keep this PR from getting any bigger


feature: prohibit the IPv6 addr for supernode peer IP

Signed-off-by: Starnop<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

fixes #1034
Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1076 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #1076      +/-   ##
==========================================
- Coverage   47.15%   47.15%   -0.01%
==========================================
  Files         117      117
  Lines        7176     7170       -6
==========================================
- Hits         3384     3381       -3
+ Misses       3527     3526       -1
+ Partials      265      263       -2

Impacted Files
Coverage Δ

pkg/netutils/netutils.go
58.15% <0%> (ø)
:arrow_up:

cmd/supernode/app/root.go
68.85% <0%> (-0.34%)
:arrow_down:

supernode/daemon/mgr/scheduler/manager.go
22.6% <0%> (+0.68%)
:arrow_up:

cmd/dfdaemon/app/root.go
67.85% <0%> (+0.81%)
:arrow_up:

Continue to review full report at Codecov.

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


Crib component DOM structure constraints from Bootlint

Thus you can code your directives such that it's impossible to construct things which would violate the constraints.
See https://github.com/twbs/bootlint
Probably not all that interesting yet until you add more complex directives, like for e.g the grid system.

Yep, seems that way.


LCFS - Organization information is visible on the Organization Details page

Description:
Add organization information to Organization Details page
Display the following elements for both BCeID and IDIR users:

Legal name of organization
Operating name of organization (if different than legal name)
Telephone number of organization
Address for service, including postal address in B.C. and email address
Address in B.C. at which records are maintained (if different than address for service)

Display the following elements for BCeID users only:

A disclaimer below the address fields that says, "Email<EMAIL_ADDRESS>to update address information."

Wireframe: https://preview.uxpin.com/59145260c7d8cc48969ac79401cc03d84843c3d9#/pages/164329808
Purpose and benefit to user:
Provides information about the organization. This information will then be displayed within a compliance report (future work) as this info is required by the legislation
Acceptance Criteria:

[ ] Given I am a , When I am , then
[ ] Given I am a , When I am , then

Development Checklist:

[ ] A
[ ] B
[ ] C

Notes:

Hey team! Please add your planning poker estimate with Zenhub @AlexZorkin @hamed-valiollahi @jig-patel @kevin-hashimoto @prv-proton


Static typing

Static typing via mypy and the typing module.

Initialized in commit https://github.com/timothy-makobu/githubstars/commit/613f6aa1a98603fbb788396e5aa961a74e959250


Controller targeting multiple namespaces

Does Sloth support monitoring multiple namespaces?
I see there are two options

specify the namespace, in this case only the specified namespace will be monitored
do not specify the namespace, in this case the default will be to monitor all namespaces

Hi @mihaidraghici98
you are correct, those are the only options, those are the most common use cases, and adding more not-so-common use cases would be complex and harder to maintain, and isn't worth it in general, very few use cases need what you say.
Normally, if you need to target different namespaces with different Sloths, you would create N sloths and target SLOs using labels so each of them only reads the CRs that you want.
Best,


v1.16 Backports 2024-10-09

[ ] #35267 (@aanm) :warning: resolved conflicts

:information_source: Hit minor conflicts with https://github.com/cilium/cilium/commit/0e9740b7d152c54d2c029474f61fe47a0e5dc9a4 (".github/workflows: change GH runners"); simply accepted combination.

[ ] #35236 (@aanm)
[ ] #35069 (@chancez)
[ ] #35288 (@oneumyvakin)
[ ] #35301 (@hox)

PRs skipped due to conflicts:

#35150 (@jrajahalme)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 35267 35236 35069 35288 35301

/test-backport-1.16


CLI: Update Navigation app template

Update the template that will be used by react-native init --template navigation. The goal is to make it easier for people to get started by demonstrating a few very basic concepts such as navigation, lists and text input.
Test plan (required)

@facebook-github-bot shipit-ninja


Integrating Google Signin with my own backend server

Is that possible to integrate googlesingin with our own backend server? .

I suggest to consult Google's official documentation about that:

iOS
Android

@jozan yep i've done that and it works perfectly. i


fix(upgrade): skip to check engine upgrade path

Which issue(s) this PR fixes:

Issue longhorn/longhorn#7539
What this PR does / why we need it:
Skip to check the engine upgrade path if the upgrade version check flag is disabled.
Special notes for your reviewer:
Additional documentation or context

There are a few things to change after the engine upgrade version enforcement was introduced, which increased 	CLIAPIMinVersion to 8 from 3, which is the version used in 1.5, 1.4, and 1.3.

Add back the VA creation in the upgrade path like what we did in 1.4 -> 1.5
Add a new field incompatible to engineer image status and remove the original incompatible state, which will make CheckEngineImageReadiness incorrectly check the image readiness.
Update the code using the EngineImageStateIncompatible1 to use the new status filed incompatible.

/datastore/longhorn.go#L1746-L1761
func (s *DataStore) CheckEngineImageReadiness(image string, nodes ...string) (isReady bool, err error) {
	if len(nodes) == 0 {
		return false, nil
	}
	if len(nodes) == 1 && nodes[0] == "" {
		return false, nil
	}
	ei, err := s.GetEngineImageRO(types.GetEngineImageChecksumName(image))
	if err != nil {
		return false, errors.Wrapf(err, "failed to get engine image %v", image)
	}
	if ei.Status.State != longhorn.EngineImageStateDeployed &&
		ei.Status.State != longhorn.EngineImageStateDeploying {
		logrus.Warnf("CheckEngineImageReadiness: engine image %v is in state %v", image, ei.Status.State)
		return false, nil
	}

@mantissahz after fixing this, update https://github.com/longhorn/longhorn/blob/master/enhancements/20230815-engine-upgrade-enforcement.md. Also, use separate commits to make the implementation clear if needed.

Tested with the latest implementation:
After upgrading v1.4.x volume attaching to a workload to 1.6.x, we can live/offline upgrade the volume engine to the master-head engine image.
One warning operation:

After the upgrade, detach the v1.4.x volume and then it should not re-attach v1.4.x volume without upgrading the engine.

What's the reason for this warning operation?


Added export capabilities for uint64 type

Export will not be enabled by default, but depending packages can just use the NumpyEigenConverter for that.

@HannesSommer looks ok?

@HannesSommer Good to merge?

test this please

yes, now it looks good.


Strict mode problem

asn1x509-1.0.js defines subjectKeyIdentifier twice which produces the following warning in Node:
Warning: asn1x509-1.0.js:1810
        'subjectKeyIdentifier': '<IP_ADDRESS>',
        ^^^^^^^^^^^^^^^^^^^^^^
Duplicate data property in object literal not allowed in strict mode Use --force to continue.

Removing one of the occurrences fixes it.

Thanks for the report. I'll fix it next update.

The 'jsjws' have been merged into jsrsasign 4.8.0 and it was released. Please use this. Regards.
https://github.com/kjur/jsrsasign/


How to know the parsed digits is an integer?

i.e. there is no '.' or 'e' in the literal.

Why, what's the use case? You don't, this information is not exported by the parser as it's not very useful.

Why, what's the use case? You don't, this information is not exported by the parser as it's not very useful.

If you write, say, a CSV or JSON parser, you may want to be able to know about the data type. E.g., in some systems "1.0" and "1" are to be treated differently. It is a niche use case, one where you need to determine the type dynamically, but it is not  unreasonnable. Note that you would like the integer value (if it is an integer value) to be returned as an integer type so that you parse integers that would not fit in a floating-point type.

If you write, say, a CSV or JSON parser, you may want to be able to know about the data type. E.g., in some systems "1.0" and "1" are to be treated differently. It is a niche use case, one where you need to determine the type dynamically, but it is not  unreasonnable. Note that you would like the integer value (if it is an integer value) to be returned as an integer type so that you parse integers that would not fit in a floating-point type.

Closing this for now as it's outside of this crate's functionality.
(As a workaround for your use case - you could always check if there's a ., e or E in the source byte string, it should be very fast compared to the time spent actually parsing the float)

It seems likely that one could build a derived crate which implements this functionality.


[2pt] Create navigation

Create navigation menu that allows users to open all of the pages you created.

Pull request for this task


'ui' is not very convenient

'ui' is not very convenient - make it possible to export / import settings via json file

So a export/import command for ui. Can be done. Will see in future release. It is not a must to be a json for now since is are few options. When are geting more I wil add to json.


[bot] treewide: remove unused 'inherit' in let blocks

Motivation for this change
clean up

What does the [bot] in the commit title mean? Why do you put it there?

1.1 is a property of a PR, it is not a property of the commit. 1.2 does not seem useful.
Automatic changes may require manual adjustment (as was the case with meta), and for one-off automatic changes it is not important to know whether they were adjusted or not. Tree-wide changes typically are like this: scripted, often not fully tested, and occasionally edited.
It is confusing to label automatic changes with bot because the latter implies that there is an automatic agent (a bot) that runs continuously or periodically and performs the action without human interaction.


add_sp_ra_delete_tips

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
All new code requires tests to ensure against regressions

Description of the Change

Alternate Designs

Benefits

Possible Drawbacks

Applicable Issues

add SL&RA; delete tips

有冲突

duplicated with https://github.com/lenage/language-coursescript/pull/32

see @micanzhang comment


Initial DPCM safe mouse code

The reason its a draft right now is there's still an issue with timing that I need to work out (you'll see at the top of the screen that the parallax is jittery)
But all of the button inputs should be working, please test if i'm missing anything.
Also I cheated to clear up enough space for the new code by moving some code in reset level into the $C000 bank. To hack around that, I made it so we can disable dpcm banking when loading it. In this case it didn't matter since DPCM isn't playing except in practice mode, so i just check for practice mode and skip playing dpcm samples during loading. So sue me. Move something else outta the fixed bank if you want me to not do this 😛 Or you can embrace the awesome hack and move more of the reset level code into a dpcm bank. :)

Unfortunately your autopsy report ideas of our music seem outdated - literally every song uses DPCM and we have 2 DPCM banks to fit all the shit in
what we DO have is a separate SFX bank that isn't a fixed one

My reading of the code was informed of dpcm banking, the only reason this works is because we stop music when you die and restart a level (except for the practice music, which i hacked around as I mentioned). The reason it couldn't go into a bank at $a000 is because the code already switches out $a000 multiple times for loading various things when restarting the level, so we can't put the loading code into a slot in $a000 and expect it to work.

Pushed the update so that palettes are loaded directly instead of through a pointer.
Its just barely not enough time yet to fit all updates in each frame, so i still delay palette writes.


[prim_lfsr] Add option to specify custom output permutation

In some places in OpenTitan (e.g. alert handler ping timer and OTP controller partition check timer), we've added a custom state output permutation in order to break linear shifting patterns. This patchset absorbs this functionality into prim_lfsr such that it can be reused more conveniently. The feature is turned off by default, hence existing instances in OpenTitan should not be affected.
I have updated the DV testbench in order to cover this feature, while in the corresponding FPV testbench this is currently disabled. @udinator let me know if you think we need to test this better.
The LFSR usage in OTP and the alert handler has been updated as well in order to leverage this new feature. @cindychip, does that in any way affect the existing DV environment of the alert handler?
CC: @vogelpi @mwbranstad for your info, since there are a couple of LFSRs in your designs.

Thanks @udinator  yeah I will add the FPV support for StatePermEn to make sure the coverage is 100%

Cool, thanks @cindychip and @udinator! Merging this in once @tjaychen and/or @cdgori gave their blessings ;).

@msfschaffner - just getting back to this. I am pretty sure I would not need this function. I looked at the otp usage and see how it works, but can you tell me why this is useful. thanks!

Hey @mwbranstad. Since LFSRs produce (mostly) linearly shifted output bits, it can be beneficial in some cases to additionally "whiten" the output by passing it through an add-on function such as a lightweight block cipher. This permutation is basically just a cheap alternative of doing that.


Update MD file on new branch 2017-11-13_03-27-53

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/content1.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.


👷 Set Node version to LTS

Updating pipelines, Dockerfile's and .node-version file
Recommend using fnm for versioning locally on your developer machine.

Should we include engine strict >= 20 in package.json in root for good meassure, i.e. when dev doesn't have have a node manager?


add optional phone number field to request a demo page

Please add phone number field to request a demo page. Please route to "phone" in Zoho.
https://www.zesty.io/demos/


Length of String is wrong.

NodeGit uses substring with bytes length comes from strlen but JavaScript requires characters length.
So non-ascii string will be broken.
https://libgit2.github.com/libgit2/#HEAD/type/git_diff_line
git_diff_line->content_len : Number of bytes of data
NodeGit:
https://github.com/nodegit/nodegit/blob/v0.5.0/lib/convenient_line.js#L71
Some other implementations:
https://github.com/libgit2/rugged/blob/v0.23.3/ext/rugged/rugged_diff_line.c#L68
https://github.com/libgit2/libgit2sharp/blob/v0.21.1/LibGit2Sharp/Core/EncodingMarshaler.cs#L129
https://github.com/libgit2/objective-git/blob/0.8.8/ObjectiveGit/GTDiffLine.m#L22
Thx.

Good catch!

I wrote a following ugly fix in the layer of JavaScript.
But I think this issue should be resolved in that of C++.
--- a/convenient_line.js	2015-11-13 23:47:56.299338415 +0900
+++ b/convenient_line.js	2015-11-13 23:48:19.766911535 +0900
@@ -67,8 +67,22 @@
 * @return {String}
 */
 ConvenientLine.prototype.content = function() {
-  return this.raw.content()
-    .substring(0, this.raw.contentLen()).replace("\n", "");
+  function countBytes(chr) {
+      return (encodeURIComponent(chr).replace(/%../g,"x").length);
+  }
+
+  var rawContent = this.raw.content();
+  var rawContentLen = this.raw.contentLen();
+  var trueContentLen = 0;
+
+  rawContent.split('').some(function(chr) {
+    rawContentLen -= countBytes(chr);
+    trueContentLen += 1;
+    if (rawContentLen == 0)
+      return true;
+  });
+
+  return rawContent.substring(0, trueContentLen).replace("\n", "");
 };

 NodeGit.ConvenientLine = ConvenientLine;

@watiko would you mind PR'ing this? The problem with implementing this in C++ is that we'd have to figure out way to generate that code in our generators. I'd argue that you should be using the JS convenience wrappers/functions unless you really know what you're doing and want that freedom.
Also, could you cache this value? We should need to recalculate this stuff since it's immutable.

I did not mean it for PR'ing. It is only ploof of concept that this issue can resolve in the layer of JavaScript. And I don't know the structure of NodeGit very much so I couldn't write patch correcty sorry.

Closed via #844


iOS just returns to app on composeTweet

I've implemented composeTweet() in Android and successfully populated a tweet in the Twitter app (works great!)--but on the iOS version I get a "Connect your account" screen opened in the Twitter app, which when I press "Connect" just takes me back to my app. No error is logged.
I suspect it has to do with referencing my callback URL, but I don't know where/how to do that; also, Twitter's newer migration docs have some functions for the appDelegate, but I'm not sure how to include those in an appDelegate.m
Any suggestions? TIA

Never mind, got it...for anyone who encounters the same thing, I added:
- (BOOL)application:(UIApplication *)app openURL:(NSURL *)url options:(NSDictionary<NSString *,id> *)options {   return [[Twitter sharedInstance] application:app openURL:url options:options]; }
to AppDelegate.m below the didFinishLaunchingWithOptions block, in addition to the
[[Twitter sharedInstance] startWithConsumerKey:@"GfjhkblahblahblaheQkf1Re" consumerSecret:@"Toha54Poi4Df9rsablahblahblahblahHKOGWxawRfgt"];
that I had already put inside that block. I had also earlier added:
- (void)showFromViewController:(UIViewController *)fromController completion:(TWTRComposerCompletion)completion{}
but this didn't seem to do anything; it's still in there, though, so who knows what finally made it work!


Failed to fetch all entries in block filter

I am unable to fetch all entries using a block filter in Web3.py
Expected Behavior
Block information to be retrieved from the block filter.
Current Behavior
The following exception:
eth_newBlockFilter
eth_getFilterLogs
TypeError: c.getAllResults is not a function
    at o.d.getFilterLogs (/usr/lib/node_modules/ganache-cli/build/cli.node.js:2:3924080)
    at Timeout._onTimeout (/usr/lib/node_modules/ganache-cli/build/cli.node.js:2:3922360)
    at ontimeout (timers.js:466:11)
    at tryOnTimeout (timers.js:304:5)
    at Timer.listOnTimeout (timers.js:267:5)

Steps to Reproduce
Start ganache-cli with
ganache-cli -d -p 7545

Write the following python code:
from web3 import HTTPProvider, Web3

web3 = Web3(HTTPProvider('http://localhost:7545'))
block_filter = web3.eth.filter('latest')

for block in block_filter.get_all_entries():
       print(block)

Your Environment

Version used<EMAIL_ADDRESS>Version of Web3py: web3==4.4.0
Version of Python: 3.6
NodeJS Version: v9.11.2
Operating System and version (include distro if Linux): Ubuntu 17.10

Duplicates trufflesuite/ganache#879. Normally I'd prefer to track in ganache-core but since there's already activity against this issue we'll keep tracking it there.


Add a flag to allow a custom authorized_keys file for the uptermd

Hi,
I wanted to ask if there is an option to use an authorized_hosts in the proxy to regulate the connections
for the upstreams ? So that you can regulate who can create an upstream.
If not here is an implementation that can be used for something like this.

It looks like this feature does not exist yet.


Remove "siglum" column on "Browse Chants"?

e.g., https://cantusdatabase.org/source/123723/chants/
The "Siglum" column will be the same for every row in the table, and it will be the same as the Source in the sidebar. Given that the table is pretty cramped, it could probably be removed.

This is a good question. The siglum also appears in some similarly redundant places, like the results for searching a given manuscript, and the full inventory. I think in the case of the full inventory it is there by request, though, since it helps show what you are looking at without scrolling to the top. "Browse chants" may also be incorporating some built-in redundancy in this way--but maybe it's making things cramped for no reason! I will ask.

Confirmed that this can go!
From Debra:
_That would definitely clean up the page.
It must be a hold-over from the view (in Drupal) where you see chants from different sources in different searches. I think Django is much more flexible about all of the views, like what you can and don't have to show._
So let's take this out!


API management

Could you provide more details on the API mgmt implementation?  From the endpoint URL I'm assuming that Bluemix IBM API Connect is being used, however I don't see the API connect service provisioned under my Bluemix account.  Is some other instance of the service being used?  How can I use my provisioned instance API Connect in Bluemix?
Thanks in advance.

Hello @nieleyde
API management is handled using the experimental API gateway feature:
https://github.com/openwhisk/openwhisk/blob/master/docs/apigateway.md
The API gateway used will be dependent on the OpenWhisk provider's implementation. I'm not sure what happens on IBM Bluemix, I'll ask a colleague to let you know.

Hi @nieleyde currently in Bluemix API Connect service is not being used.
What's being used in Bluemix is what @jthomas mentioned a cluster of instance of the containerhttps://github.com/openwhisk/openwhisk-apigateway

Thanks for the info.