Interactive graph for Active Cases per state in Deep Dive

In the Deep Dive Page, Graphs are available for total cases reported state wise. However, there is no graphs for Active Cases.
We can have a "States - Active Case" Graph on the pattern of "States - Total Cases " graph currently available.

I would like to work on this. Kindly assign to me

if its not done please assign it to me

/claim

@jeremyphilemon @shuklaayush  I thought a pie chart is a good option for this. Any thoughts or suggestions?


refactor(AMD/CommonJS exported module): Export module name

When the module is imported using AMD/CommonJS, it is expected to expose its name instead of the
Angular module object. This is because angular.module() expects the dependencies as string instead
of actual module objects.
Fix #326
Breaking change: code that relies on the module object to be exported (accessing the name
via .name for example) will break, since the name is now directly returned.

[Current coverage][cc-pull] is 100%

Merging [#360][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #360   diff @@
====================================
  Files           1      1
  Lines         811    811
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits          811    811
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last updated by [6e8b022...fad069d][cc-compare]
[cc-base-branch]: https://codecov.io/gh/angular-slider/angularjs-slider/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/angular-slider/angularjs-slider/compare/6e8b02201ee1ec4cc0e32ea7b8ae16330ff8c1aa...fad069d51dd55b907d86d55116f79ac946587705
[cc-pull]: https://codecov.io/gh/angular-slider/angularjs-slider/pull/360?src=pr

👍

Released under 5.0 version ;)

Merci Valentin ! ;)
Thanks for being that reactive ! 👍


[KARAF-6963] First send SIGTERM, only SIGKILL if not shutting down nicely

PR for issue: https://issues.apache.org/jira/browse/KARAF-6963
Try first a regular SIGTERM to shutdown the daemon process.
If that doesn't result in the process stopping within 10 seconds, follow with the SIGKILL signal.
It's verifying in 1 second intervals whether the process is still running, so that the 'stop' request won't be blocking too long.

Rebundling our code with Karaf and with this fix applied, the exit status is now 'clean':
root@vagrant:/opt/instana# systemctl stop instana-agent
root@vagrant:/opt/instana# systemctl status instana-agent
● instana-agent.service - "Instana(tm) agent."
   Loaded: loaded (/lib/systemd/system/instana-agent.service; enabled; vendor preset: enabled)
   Active: inactive (dead) since Wed 2020-12-16 10:27:35 UTC; 3s ago
  Process: 14272 ExecStop=/opt/instana/agent/bin/karaf stop (code=exited, status=0/SUCCESS)
  Process: 14180 ExecStart=/opt/instana/agent/bin/karaf daemon (code=exited, status=0/SUCCESS)
 Main PID: 14180 (code=exited, status=0/SUCCESS)

Bump.
I haven't seen any updates. I think the build error is not correct or at least not related to this change, but cannot re-trigger.
Would be nice if I could get some feedback, to move this forward.
Many thanks.

Thanks for the PR. I'm reviewing it now.


In 0.16.x, Set as heading 1 was added to core

It looks like these related shortcuts can now be removed from this plugin.

Unfortunately these are not yet equivalent. The core functionality does not support toggling, which ObsidianTweaks does. Heading toggling is something I use myself, so I won't remove that feature unless it is fully superseded by the core functionality.
Thanks for the heads up though!


Move caesar cypher interactive

Move interactive from Encryption Coding chapter

Plaintext

<textarea id="caesar-plaintext" class="form-control" rows="5"></textarea>
<button id="caesar-encrypt" type="button" class="btn btn-primary btn-block" style="margin-top:10px;">Encrypt</button>

Rotation Amount

<input id="rotation-input" type="number" min="1" max="26" value="1" onkeydown="return isNumber(event,this);" class="interactive-input">

Ciphertext

<textarea id="caesar-ciphertext" class="form-control" rows="5"></textarea>
<button id="caesar-decrypt" type="button" class="btn btn-primary btn-block" style="margin-top:10px;">Decrypt</button>

Pretty sure @Jamesernator has done this one?

The Caesar interactive in the encryption chapter doesn't currently work properly for me. Should it? It currently will encode one text (or decode one text), but can't decode or do a new encoding until the page is reloaded.

This was fixed in 2.0.1


"Grow Me" button does not work

Issue Info

Info
Value

Platform
ios

Platform Version
11

Masonry Version
1.0.2

Integration Method
cocoapods

Issue Description
Masonry iOS Examples--> Update Constraints --> tap "Grow Me" button.

It did not work.

Just try below:
make.width.equalTo(@(self.buttonSize.width));//.priorityLow();
make.height.equalTo(@(self.buttonSize.height));//.priorityLow();

@murphy-yu

@implementation MASExampleUpdateView

- (instancetype)init {
    if (self = [super init]) {
        self.growingButton = [UIButton buttonWithType:UIButtonTypeSystem];
        [self.growingButton setTitle:@"Grow Me!" forState:UIControlStateNormal];
        self.growingButton.layer.borderColor = UIColor.greenColor.CGColor;
        self.growingButton.layer.borderWidth = 3;

        [self.growingButton addTarget:self action:@selector(didTapGrowButton:) forControlEvents:UIControlEventTouchUpInside];
        [self addSubview:self.growingButton];
        [self.growingButton makeConstraints:^(MASConstraintMaker *make) {
            make.center.equalTo(self);
            make.width.equalTo(@(self.buttonSize.width)).priorityLow();
            make.height.equalTo(@(self.buttonSize.height)).priorityLow();
            make.width.lessThanOrEqualTo(self);
            make.height.lessThanOrEqualTo(self);
        }];
    }

    self.buttonSize = CGSizeMake(100, 100);

    return self;
}

+ (BOOL)requiresConstraintBasedLayout
{
    return YES;
}

// this is Apple's recommended place for adding/updating constraints
- (void)updateConstraints {
    //according to apple super should be called at end of method
    [super updateConstraints];
}

- (void)didTapGrowButton:(UIButton *)button {
    self.buttonSize = CGSizeMake(self.buttonSize.width * 1.3, self.buttonSize.height * 1.3);
    [self.growingButton updateConstraints:^(MASConstraintMaker *make) {
        make.width.equalTo(@(self.buttonSize.width)).priorityLow();
        make.height.equalTo(@(self.buttonSize.height)).priorityLow();
    }];

    // tell constraints they need updating
    [self setNeedsUpdateConstraints];

    // update constraints now so we can animate the change
    [self updateConstraintsIfNeeded];

    [UIView animateWithDuration:0.4 animations:^{
        [self layoutIfNeeded];
    }];
}

@end


toTxMethod returning "[object Object]" value for the dest field in method.args.dest

When decoding a transaction like so:
  const decodedTxn = decode(rawTransaction, {
      metadataRpc: this._material.metadata,
      registry: this._registry,
    })

We were able to do something like decodedTxn.method.args.dest.id but now the value of dest in version 1.5.2 is "[object Object]" therefore our code breaks since it's expecting an object not a string.
Can this change introduced in this PR to toTxMethod be reverted? It works with the implementation in 1.5.1

@lcovar Sorry for the delayed response as I am in the process of moving and out this week.
That being said, @jarcodallo is on it and checking out the issue, but I will prioritize some time to help him out and diagnose it. The  reason for that PR and the isNaN check was in reaction to some upstream changes in p.js.

@lcovar Which method are you using to construct the TX?

methods.balances.transferKeepAlive is one of them @TarikGul

So I am not able to reproduce your issue.
Using 1.5.2 against 0.9.16 as well as a 0.9.15-1 polkadot dev node, when decoding the values I wasn't able to get "[object, Object]". Below I pasted my exact responses for decodedTxn.method.args for all types of constructed tx's.
// Unsigned tx
{
  dest: { id: '14E5nqKAp3oAJcmzgZhUD2RcptBeUBScxKHgJKU4HPNcKVf3' },
  value: '90071992547409910'
}

// signing payload
{
  dest: { id: '14E5nqKAp3oAJcmzgZhUD2RcptBeUBScxKHgJKU4HPNcKVf3' },
  value: '90071992547409910'
}

// signed tx
{
  dest: { id: '14E5nqKAp3oAJcmzgZhUD2RcptBeUBScxKHgJKU4HPNcKVf3' },
  value: '90071992547409910'
}

Are you possibly having a conflict of polkadot-js dependencies. 1.5.2 uses:
polkadot/api 7.7.1
polkadot/util-crypto 8.3.3

@TarikGul
We are resolving to these version:
"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>  version "1.5.2"

"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>  version "7.8.1"

"<EMAIL_ADDRESS>"<EMAIL_ADDRESS>  version "8.4.1"

However, I am unsure about this if condition here https://github.com/paritytech/txwrapper-core/blob/main/packages/txwrapper-core/src/core/method/toTxMethod.ts#L46
The !Number.isNaN() check will return true even if jsonArgs has a value of { id : 'asdfasdf'}
Before the if condition:

After:

That condition may need to be updated.

@TarikGul @jarcodallo ^

@lcovar Thanks for writing the above, it helps a lot. In an isolated, pure JS enviornment I am able to reproduce it (being that yes passing in an object to !Number.isNaN() such as {id: 'asdf} will return true and is also not intended). I am currently writing out test coverage for toTxMethod (something long past due), and working to see what kind of proper sanitization I can get implemented in there.
I'll be working throughout the weekend to get this sorted.

@TarikGul thanks for looking into this! Confirmed that 1.5.3 fixed our issue.

@TarikGul thanks for looking into this! Confirmed that 1.5.3 fixed our issue.

Rad, glad to hear! Thanks again for the patience 🙏


Incorrect types being passed to ISPyB

During testing on i03 it was found that we were passing incorrect types to ISPyB in some cases:

When converting json using dataclasses_json numbers come back as numpy floats, ISPyB wants them as primitive floats
beam_position, xtal_snapshots and snapshotOffsetXPixel were being set as lists rather than single items

Note these have been fixed in https://github.com/DiamondLightSource/python-artemis/tree/local_i03_testing_changes but need tidying up

This requires more testing on the beamline, I couldn't reproduce it offline.


Preparing for release 3.4.3

Updated package.version
Updated CHANGELOG

Pull Request Test Coverage Report for Build 101

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

Totals

Change from base Build 101:
0.0%

Covered Lines:
308

Relevant Lines:
308

💛  - Coveralls


Compile Remove with a wildcard removes more files than it should

From @bording on May 5, 2017 23:20
I'm not entirely sure where this bug is best filed, but since I'm using the new project system to reproduce it, I figured I'd start here.
I've got a NuGet package that brings in source files via the contentFiles mechanism. I need to be able to exclude certain files, so I added a Compile item with a wildcard pattern to remove them. That works correctly, however it seems to also remove files that aren't part of the pattern as well.
I've pushed up a repo that demonstrates the problem here: https://github.com/bording/GlobbingProblem
If you look at the project file, you can see the following:
 <ItemGroup>
    <Compile Remove="$(NuGetPackageRoot)**\Audit\*.cs" />
  </ItemGroup>

That removes the 5 files in the Audit folder as expected, but it also removes ConfigureEndpointLearningTransport.cs, which doesn't match the pattern at all.
Before remove:

After remove:

I've experimented with different wildcard patterns, and I can't see any discernible pattern to explain the behavior.
For example:
  <ItemGroup>
    <Compile Remove="$(NuGetPackageRoot)**\EndpointTemplates\*.cs" />
  </ItemGroup>

Removes files from the Pipeline and Recoverability folders as well.
Copied from original issue: dotnet/project-system#2134

See also https://github.com/FakeItEasy/FakeItEasy/pull/1103 for another example. This is really weird. Looks like a serious bug in the globbing algorithm.

@AndyGerlicher Can you please triage this?

This may be a bug in ImmutableList<T>.Builder. If I do the following replacement, everything works:
                //listBuilder.RemoveAll(itemData => items.Contains(itemData.Item));

                var itemDatasToRemove = listBuilder.Where(itemData => items.Contains(itemData.Item)).ToList();

                foreach (var itemToRemove in itemDatasToRemove)
                {
                    listBuilder.Remove(itemToRemove);
                }

It's quite hard to write a unit test for this. I'll try and reduce the repro and open a bug against corefx, and then go ahead and check this fix in.

Worked around in #2162


Modify all pipelines to list signal names in a uniform location

Changing signal names from their wip_ to their public versions turns out to be an important part of the release schedule, so let's make it easier to catch these updates in PRs. Maybe params.json.template? Or a separate file?

[x] safegraph
[x] ght
[x] emr-hosp
[x] combo cases/deaths
[x] quidel covid tests
[ ] ~doctor-visits~ [blocked?]
[ ] ~quidel flu tests~ [blocked; wait for merge to main]
[ ] ~jhu~ [blocked; wait for geo revision]
[ ] ~usafacts~ [blocked; wait for geo revision]
[ ] ~combo nmf_day_doc_fbc_fbs_ght~ [blocked; not in this codebase]
[x] python template
[ ] ~fb-survey~ [blocked; wait for merge to main]
[ ] corona data scraper (cds)

Will not complete; moving away from wip names to a staging server.


Disable cron schedule

We are disabling cron schedule, to make it easier to reorganize the repo layout.

Not nedded


Add new version for proactive detection config

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-libraries-for-java

Automation for azure-sdk-for-node

Automation for azure-sdk-for-go

@aviled can you fix the error reported by CI (see previous comments) ?

Automation for azure-sdk-for-java

Looking in Eric Chong @ericc1103 owner of workbooks_api-json spec to address ARM violation.
ARM-Violation: "Properties of a PATCH request body must not be required. PATCH operation: 'Workbook_Update' Model Definition: 'WorkbookResource' Property: 'location'"
'WorkbookResource' model inherits from ‘Resource’ (for which we marked location as required), this model is used for both PATCH and PUT. It is required that all the properties of model representing PATCH operation has to be optional. This also indirectly means service should not enforce any requiredness in the PATCH payload.
In-order to fix this, we need to define a new model representing Workbook PATCH payload (with all properties optional). The PATCH payload model looks something like WorkbookUpdateParameters as shown below. But please note that only you know what are the PATCH-able properties of workbook so you will need to update (by adding or removing properties) the below model accordingly.
    "WorkbookUpdateParameters": {
        "properties": {
            "kind": {
                "type": "string",
                "description": "The kind of workbook. Choices are user and shared.",
                "enum": [
                    "user",
                    "shared"
                ],
                "x-ms-enum": {
                    "name": "SharedTypeKind",
                    "modelAsString": true
                }
            },
            "tags": {
                "type": "object",
                "additionalProperties": {
                    "type": "string"
                },
                "description": "Resource tags"
            },
            "properties": {
                "x-ms-client-flatten": true,
                "description": "Metadata describing a web test for an Azure resource.",
                "$ref": "#/definitions/WorkbookPropertiesUpdateParameters"
            }
        },
        "description": "The parameters that can be provided when updating workbook properties properties."
    },
    "WorkbookPropertiesUpdateParameters": {
        "description": "Properties that contain a workbook.",
        "properties": {
          "name": {
            "type": "string",
            "description": "The user-defined name of the workbook."
          },
          "serializedData": {
            "type": "string",
            "description": "Configuration of this particular workbook. Configuration data is a string containing valid JSON"
          },
          "version": {
            "type": "string",
            "description": "This instance's version of the data model. This can change as new features are added that can be marked workbook."
          },
          "workbookId": {
            "type": "string",
            "description": "Internally assigned unique id of the workbook definition."
          },
          "kind": {
           "description": "Enum indicating if this workbook definition is owned by a specific user or is shared between all users with access to the Application Insights component.",
            "x-ms-client-name": "SharedTypeKind",
            "default": "shared",
            "type": "string",
            "enum": [
              "shared",
              "user"
            ],
            "x-ms-enum": {
              "name": "SharedTypeKind",
              "modelAsString": true
            }
          },
          "tags": {
            "type": "array",
            "items": {
              "type": "string"
            },
            "description": "A list of 0 or more tags that are associated with this workbook definition"
          },
          "userId": {
            "type": "string",
            "description": "Unique user id of the specific user that owns this workbook."
          },
          "sourceResourceId": {
            "type": "string",
            "description": "Optional resourceId for a source resource."
          }
        }
      }

I have updated mine to make PATCH operation of properties as not required. The PR is @ https://github.com/Azure/azure-rest-api-specs/pull/2950

@ericc1103 thanks, your PR is for new preview api-version. The issue here is with 2015-05-01 version of workbooks. I replied to your email and clarified, please take a look.

Closing due to inactivity, please reopen/comment when you're back to this. Thanks.


[BUG] AutoCommissioner requires ConnectMaxTimeSeconds present, but it is only required present with WI | TH, not ET

Problem:

During common commissioning flow, DeviceCommissioner::PerformCommissioningStep reads many attributes to determine features of the device
It was found during refactoring of Network Commissiong cluster that all examples, even those that runtime declare ET feature, have the ConnectMaxTimeSeconds attribute included, which is forbidden since the conformance on it is WI | TH

This is a significant non-conformance.
Example on all-clusters-app:
[2023-11-02 13:00:12.749039][TEST][STDOUT][MatterTest] 11-02 13:00:12.743 INFO
[2023-11-02 13:00:12.749053][TEST][STDOUT]Problem: ProblemSeverity.ERROR
[2023-11-02 13:00:12.749064][TEST][STDOUT]    test_name: test_spec_conformance
[2023-11-02 13:00:12.749075][TEST][STDOUT]    location:
[2023-11-02 13:00:12.749086][TEST][STDOUT]        Endpoint: 0,
[2023-11-02 13:00:12.749097][TEST][STDOUT]        Cluster:  49 (0x31) NetworkCommissioning,
[2023-11-02 13:00:12.749109][TEST][STDOUT]        Attribute:2 (0x02)
[2023-11-02 13:00:12.749120][TEST][STDOUT]    problem: Attribute 0x02 is included, but is disallowed by conformance. Conformance: WI | TH, implemented features: ET
[2023-11-02 13:00:12.749132][TEST][STDOUT]    spec_location:
[2023-11-02 13:00:12.749143][TEST][STDOUT]
[2023-11-02 13:00:12.749153][TEST][STDOUT][MatterTest] 11-02 13:00:12.743 INFO
[2023-11-02 13:00:12.749166][TEST][STDOUT]Problem: ProblemSeverity.ERROR
[2023-11-02 13:00:12.749178][TEST][STDOUT]    test_name: test_spec_conformance
[2023-11-02 13:00:12.749189][TEST][STDOUT]    location:
[2023-11-02 13:00:12.749200][TEST][STDOUT]        Endpoint: 0,
[2023-11-02 13:00:12.749211][TEST][STDOUT]        Cluster:  49 (0x31) NetworkCommissioning,
[2023-11-02 13:00:12.749222][TEST][STDOUT]        Attribute:3 (0x03)
[2023-11-02 13:00:12.749233][TEST][STDOUT]    problem: Attribute 0x03 is included, but is disallowed by conformance. Conformance: WI | TH, implemented features: ET
[2023-11-02 13:00:12.749244][TEST][STDOUT]    spec_location:

Where does AutoCommissioner require this to be present?
If this attribute is not present, then info.network.eth.minConnectionTime will never get set.  Which is fine, because it's also never used, as far as I can tell.


Implemented OS switch so macOS builds use CPU version of TensorFlow.

Fixes #313.
Tested to work as expected with builds on macOS and Ubuntu.

There are no TensorFlow binaries for macOS ARM at all, this is only for macOS x64.


feat: add Hono test client (RPC)

Author should do the followings, if applicable

[x] Add tests
[x] Run tests
[x] yarn denoify to generate files for Deno

What this PR do?
By using the hc test client, we can easily write tests. This utility enables efficient integration and unit testing by allowing manipulation of the Hono application object rather than making actual HTTP requests.
Usage Example
it('test', async() => {
  const app = new Hono().get('/search', (c) => c.jsonT({ hello: 'world' }))
  const res = await hc(app).search.$get()

  expect(await res.json()).toEqual({ hello: 'world' })
})

Hi @hagishi!
Cool! This is one of the things we want to do with RPC-mode. Introducing the testing helper makes sense.
But, there's one thing we should consider. The function name hc might be confusing because hono/client has a function with the same name. I haven't come up with a perfect name yet, but something like appToClient() might be better. Any thoughts?

Hi @yusukebe,
Thank you for the review.
I agree that some parts could indeed be confusing.
Following the example of frameworks like FastAPI,
I think a clearer naming convention like testClient(app) would be better.
What do you think?"

Hi @hagishi !
testClient is simple and good! Let's go with it.

Hi @hagishi
Thanks! Merging it now.


USD file

Hi, is there a way to save out a USD file?
I want to try some scene level pre-processing step in 3D (like running other 3D encoder).

Yes, feel free to check out the "Saving Options" in this link: https://docs.omniverse.nvidia.com/isaacsim/latest/gui_tutorials/tutorial_intro_usd.html


Cannot differentiate between disabled and enabled options for git branch

Cannot differentiate between disabled and enabled options for git branch after right-clicking the branch. Below are the screenshots of the default dark and mocha theme.

Yeah, I see the outline for the enabled options. Thanks for the update!!! :)

@mddheeraj Kindly give v1.2.1 a try. I believe this is now fixed. Thanks

Thanks @djflan!!! It got fixed!!!!

Thanks @djflan!!! It got fixed!!!!

Glad it's working for you. I know some of the colors are still off on the git buttons.
Working to correct that. I'll need to check the VS Code port to see how they might be handling buttons like that.


Initial hw1 for Niket Parikh (niketnp2)

I will review your code.

The code, in general, is well-written. Good use of list comprehension.
I tested the grid world algorithms, and they work fine. A couple of additions to the codes:

The alpha and epsilon plots should be added for the grid world and pendulum problems.
The learning curves for the pendulum problem should be added.
Maybe remove some redundant comments.

Thank you for the review John! I've included this in my latest push.


invalid Bech32 prefix; expected cosmos, got terra: invalid request

"Build transaction error. Status 400 - {'code': 3, 'message': 'invalid Bech32 prefix; expected cosmos, got terra: invalid request', 'details': []}: failed to create transaction !, Error",

It looks like you sent a tx with terra address to cosmos.

and like I mentioned in #126, i can't guarantee it works as you want. use their own sdk please.

It looks like you sent a tx with terra address to cosmos.

Can it be done via the core.ibc module? I don't see a demonstration.

@Vritra4 I want to transfer atom via terra sdk . because they are based on cosmos SDK

@SheydaAshoorian i don't know why do you want but keys from this sdk won't work because HRP is fixed to 'terra'.


Derivation of the Elm Protocol should generate only valid elm code

Due the fact that the library elm-bridge is out of date in some aspects, the generated code can not successfully compiled via elm-make.
A import does not expose a certain function anymore which causes the problems. Also it might be a good idea, to execute elm-format on the generated output file.

writeFileUtf8 "fromtend/Protocol.elm" . unlines . filter (not . contains "(:=)") . lines $ elmProtocolModule

Compiler error is fixed.
Might still be worth it the time to execute elm-format automatically after code generation


Create Handshake Full Node Template

This template allows anyone to add a node to the peer to peer handshake DNS network. handshake.org

Please add this to the README.md at the root of the repository. Thank you.

@FliKites ping ...  ^^

Ok. I'll get this done soon. Thanks for the ping.  ---- On Sat, 08 Apr 2023 13:59:19 -0400  @.***  wrote ----
@FliKites ping ...  ^^
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

@88plug @andy108369 This is good to go. Let me know if their are any issues.

@FliKites please add the last bit - signedBy attributes, e.g. https://github.com/akash-network/awesome-akash/blob/b720cd5/MySQL/deploy.yaml#L29-L32
Thank you!


Maintenance: Java Edition 1.21.4

the garden awakes

1.21.4 released. i'll update this when i have time. right now i am really busy.


Error: /usr/local/opt/usbmuxd is not a valid keg

brew install --HEAD usbmuxd
and then log error
Error: /usr/local/opt/usbmuxd is not a valid keg

This is not related to Flutter.
I guess https://github.com/libimobiledevice/usbmuxd would be the right place to report.


Atom doesn't open. Error: Module version mismatch. Expected 47, got 46.

Prerequisites

[x] Can you reproduce the problem in safe mode?
[x] Are you running the latest version of Atom?
[x] Did you check the debugging guide?
[x] Did you check the FAQs on Discuss?
[x] Are you reporting to the correct repository?

Description
I've updated Atom to the last version and it doesn't open. Instead of it, when I tried to open Atom it opens like the web console of Firefox where I can see all the HTML elements and use debugger. I believe it is the debugging tool of Atom.
I am using an Arch Linux distribution and I built myself Atom for the installation. In other words, I downloaded the git repository and built Atom.
Attached is the log.
-1461595807126.txt
Steps to Reproduce

Open Atom

Expected behavior:  Open Atom
Actual behavior:  Open Atom debugger?
Versions
Atom    : 1.7.2
Electron: 0.36.8
Chrome  : 47.0.2526.110
Node    : 5.1.1
apm  1.9.2
npm  2.13.3
node 0.10.40
python 3.5.1
git 2.8.0g.

Did you try to build Atom from source at some point?
What OS are you running into this on?

Sorry, I thought I indicated that:
I built myself Atom to install it because I have a Arch Linux distribution. So, I followed the instructions to do it by downloading the git repository. I will, update my question to add the OS.

Can you try running script/clean and then script/build again?

No it didn't work. I have uninstalled everything and tried again. The problem still the same.

You should be able to fix this by going to your local Atom repository and:

Delete the node_modules directory
script/clean
script/build

No, it didn't work either. I add the script/build output:
install.txt

There is something hanging out somewhere that you didn't completely delete when you followed my steps. You need to perform a 100% clean build. My only recommendation at this point is to delete anywhere Atom is installed, delete any Atom packages you have installed, delete the Atom repository completely and start over from the beginning.

I deleted everything. Atom was unistalled and I builded atom from zero. I got installed Atom beta, but it crashes when it is open. The normal version of atom was not installed as expected.

The reason that this error message shows up is because some native modules were compiled against an older version of Chromium than the one that is running.

Well, I've uninstalled Atom again, following the same instructions posted here
I've downloaded the source code, not using the git command but the .tar.gz that it is found in Git. I've compiled everything. Now, I have the normal version of Atom 1.7.3 but it crashes when it is open. So, the error of Expected 47, got 46, has been solved. But a new problem has appeared.

I have the same problem now when trying to build Atom from the latest master commit (ad24aae).

(pardon the screenshot, I can't copy paste from the dev tools console)

The above steps worked for me.


Enable comment-on-pr and bump doozer

Follow up to https://github.com/openshift-eng/doozer/pull/761

/hold

https://github.com/openshift-eng/doozer/pull/765 has merged and I've updated it here
We are good to merge now


Support for Logitech G Pro X Superlight 2

The Logitech G Pro X Superlight 2 is different from the existing models (e.g. the older non-2 model) and other similar Logitech devices. Specifically it implements only the extended adjustable DPI feature (0x2202, for X, Y and lift-off-distance/LOD configuration), extended report rate feature (0x8061) as well as a new profile format that is very different from the existing profiles in order to support the data for the aforementioned features. None of these features are currently implemented in libratbag.
This pull request is a draft/work in progress containing the required support for the HID++ 2.0 features needed for this device. I am publishing this draft pull request in order to get any feedback, please disregard any obvious or unnecessary debug/work in progress code changes.
Implemented WIP support:

Feature 0x2202 - Extended Adjustable DPI

Handling of X/Y/LOD fields, using the current DPI singular value from profile structures to populate both X/Y and defaulting LOD to "Medium"

Feature 0x8061 - Extended Report Rate
Support for onboard profile type 0x6

Extended DPI and report rates
Power save timeouts
Mouse button mappings

Things that are not implemented/tested:

Address remaining TODO comments included in this pull request
Configuration over the wireless receiver (currently plug it in via USB to use ratbag to edit profiles)
DPI for Y/LOD
Support for onboard profile type 0x6

Non-mouse button mappings
Profile names (UTF-16 encoded)
Determine remaining unknown fields in the profile format
LEDs? (device does not have any, but the format structure has them defined)

Piper

This pull request intends to resolve #1572 and #1519.

What's the status on this PR?

No progress, but it is still functionally useful in this WIP state. I intend to finish this but have not had the time recently.

Thanks for your effort! :heart:


Accessing ctx.query directly breaks the object

What version of Elysia.JS is running?
<EMAIL_ADDRESS>bun 1.1.7
What platform is your computer?
Linux 6.4.9-arch1-1 x86_64 unknown
What steps can reproduce the bug?
import { Elysia } from "elysia";
import assert from "node:assert";

const hostname = "<IP_ADDRESS>";
const port = 8089;
const app = new Elysia();

// using this request:
// http://localhost:8089/foo/bar?foo=bar&bar=baz
app.get("/foo/bar", async (ctx) => {
  console.log(ctx.query);
  // request with just the console.log above (commenting out the console.logs below)
/*
Listening on <IP_ADDRESS>:8089
{
  query: undefined,
}
*/

  // when you add the following three console.log s:
  // console.log(JSON.stringify(ctx.query));
  // console.log(ctx.query.foo);
  // console.log(ctx.query.bar);
/*
Listening on <IP_ADDRESS>:8089
{
  query: undefined,
  "query)": undefined,
  foo: "bar",
  bar: "baz",
}
{"foo":"bar","bar":"baz"}
bar
baz
*/
});

app.listen({
  hostname,
  port,
});
assert.ok(app.server);

console.log(`Listening on ${app.server.hostname}:${app.server.port}`);

What is the expected behavior?
For ctx.query to actually be a correct object containing all of the query parameters in the request.
What do you see instead?
See repro comments.
Additional information
Taking a glance at the codebase, it appears to be because elysia tries to be "smart" with figuring out what the route handler wants to access, and codegens around it. Why? Why is this a thing??? I understand that it may be for "performance reasons" but in place of it you open up a GIGANTIC can of worms causing issues such as above because people can't expect their code to work as it normally does within normal javascript execution. I am very much okay taking a perf hit if it means that the library I use doesn't try to be fancy with metaprogramming like this, potentially exposing my code to critical error paths.

You can disable aot (the codegen) as a workaround:
new Elysia({ aot: false })

Thanks, I wasn't aware of that option. It's still a pretty nasty surprise learning that this was a thing Elysia did, especially given how this appears to not be the only bug relating to it. I would feel a lot more confident in Elysia if it had it disabled by default, instead letting the user opt into it in the docs, after warning about its potential volatility.

Should be fixed with 346134b alongside #656 and #658 published under 1.0.23. Let me know if it works on your end.

Works for me, thnx


Pixi debug env

Fixes #
Description

Since the latest pixi release, support has been built in for multi environments, as described here.
This changeset adds an environment for local interactive python. This makes it a lot easier to make plots in VSCode, as you can produce them in the ipykernel. Without this, plots need to be made in the debugger, which is notoriously annoying to get to work with Matplotlib, saved to disk.
To install:
pixi install --environment interactive
pixi shell --environment interactive
pixi run install
Now open VSCode, create a script, press CTRL+A to select all and press SHIFT+ENTER to run the code in the interactive kernel.
Checklist

[ ] Links to correct issue
[ ] Update changelog, if changes affect users
[ ] PR title starts with Issue #nr, e.g. Issue #737
[ ] Unit tests were added
[ ] If feature added: Added/extended example

This probably should be mentioned in the documentation as well. Maybe in Developer section.

This probably should be mentioned in the documentation as well. Maybe in Developer section.

I extended the developer docs.


Text references wrong API calls from example.

The text and/or links reference the wrong API calls made by the example code.
NetAdapterCx requires the client driver to set the following capabilities:

Data path capabilities. The driver calls NetAdapterSetDataPathCapabilities to set these capabilities. For more information, see Network data buffer management.

Link layer capabilities. The driver calls NetAdapterSetDataPathCapabilities NetAdapterSetLinkLayerCapabilities to set these capabilities.

Link layer maximum transfer unit (MTU) size. The driver calls NetAdapterSetDataPathCapabilities NetAdapterSetLinkLayerMtuSize to set the MTU size.

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

ID: 06dbac05-7cce-0362-4665-1ce20afdf09a
Version Independent ID: 37b84f23-9f4e-b841-4af0-9d4ecbbcef57
Content: Device and adapter initialization - Windows drivers
Content Source: windows-driver-docs-pr/netcx/device-and-adapter-initialization.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @aviviano
Microsoft Alias: amvivian

@grimbeaver These links are updated now. Thanks again!


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes.

Bicep or Terraform?
Terraform
Module Classification?
Pattern Module
Module Name
avm-ptn-alz
Module Details
This module has been published in the TF registry in November 2023, yet doesn't seem to have an associated Module Proposal, with track record of ownership and approval. Its source code is available here.
I'm creating this issue retrospectively and assigning it to @jaredfholgate, as according to the module index, he's the owner of this module.
CC: @PmeshramPM, @prjelesi, @mbilalamjad
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
jaredfholgate
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

Hi @jaredfholgate,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR

Reopening. @jaredfholgate, when you're ready, please respond to my above message by repeating the quoted line.

I'm the owner of the module... do we really need this?

Hmm, in the module index, Jared's name is captured currently. I've pinged you with some more context and details internally. Thanks!

I'm unsure why I am flagged as the owner in the index? Given Matt has done 99% of the work, he should have that glory. :)

Thanks for confirming! I'm not sure who captured the record incorrectly, but there's no trace of this module even being proposed; hence I submitted this issue after the fact. This shows why we all need to follow the process we defined together, so that mistakes are hard to impossible to make. 👍
As I caught this, I'm happy to fix the wrong record, just need a handshake from @matt-FFFFFF.

Hi @matt-FFFFFF,
Thanks for requesting/proposing to be an AVM module owner!
We just want to confirm you agree to the below pages that define what module ownership means:

Team Definitions & RACI
Shared Specification (Bicep & Terraform)
Module Support

Any questions or clarifications needed, let us know!
If you agree, please just reply to this issue with the exact sentence below (as this helps with our automation 👍):
"I CONFIRM I WISH TO OWN THIS AVM MODULE AND UNDERSTAND THE REQUIREMENTS AND DEFINITION OF A MODULE OWNER"
Thanks,
The AVM Core Team
#RR


Swap selected text with clipboard

Description of the new feature / enhancement
When editing, having the ability to swap selected text with the content of the current clipboard. After the swap, the selected text now contains the clipboard, and the clipboard contains the previously selected text.
Scenario when this would be used?
When editing .xml files, swapping two specific values at different locations would be helpful.
Key sequence before beforethis feature request;
place cursor at first location
drag to select text
ctrl-c
place cursor at second location
ctrl-v
at current location, drag to select text (to be moved to first location)
ctrl-c
place cursor back at first location
ctrl-v
Key sequence after this feature request;
place cursor at first location
drag to select text
ctrl-c
place cursor at second location, drag to select text
ctrl-s (the new feature swapping!)
place cursor at first location
ctrl-v
Supporting information
Two steps less and practical.

I like the idea of this as I work with a bunch of XML files, but would it not make more sense for the ctrl-s command mentioned to just swap the text? My alternative would be:
• Highlight 1st piece of text
• Hold Ctrl and highlight 2nd piece of text
• Press Ctrl + Win + s to swap the location of the 2 highlighted test pieces (Ctrl + s is usually a save shortcut)
A more robust option would be the following (prevents issues around apps not allowing multi-select):
• Press Ctrl + Win + s to activate swap
• Highlight 1st piece of text
•Tooltip or GUI window displays "Power Switcher: "[1st text item]" selected"
• Highlight 2nd piece of text
• Test swaps places
• Tooltip/pop-up confirms swap with option to undo
•Optional settings:

Before completing the swap, confirmation box pop up with:

Swap location of:
Line 1: "[1st text item]"
Line 2: "[2nd text item] "
[OK]                 [Cancel]

Option for cursor and page scroll to return to location for 1st text selected after completion of swap

This would reduce the key strokes further, prevent errors where the 2nd text item has been incorrectly highlighted, and prevent the need to add the pieces of text to the clipboard.


Add wheel, flightstick, throttle types from SDL data

Update data fetch script
Add types to device filtering

Going to just add these manually for now since the pattern isn't an easy fit for our existing parsing rules and they don't change frequently


fix: block swipe if in wrong direction

Fixes #295
This PR blocks the swipe if the user starts swiping in the wrong direction allowing the user to select text within the vaul. Unfortunately this fix will only work for pointers. It was not possible to fix it for mobile because I realized that the problem comes from the react-remove-scroll lib that Radix uses under the hood to block scrolling.
Also forwards some events from the DialogPrimitive.Content props
Before
https://github.com/emilkowalski/vaul/assets/48808846/62a062b7-579f-4e54-93ed-7235f423f788
After
https://github.com/emilkowalski/vaul/assets/48808846/1e54e09a-dba3-4011-9a9e-49a1f42b23c6

@emilkowalski I see this is merged to main, but not deployed to npm. Right now this is blocking people in my production app using vaul from copying text. Do I need to fork vaul and deploy it myself, or is a release right around the corner?

@emilkowalski I'll be fine with a canary/beta release candidate so I can use it before you deploy a minor/patch release


iOS - Distance - Center icon overlaps with compass icon

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.2-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Launch New Expensify app
Go to FAB > Submit expense > Distance
Use two fingers to rotate the map

Expected Result:
Center icon will not overlap with compass icon
Actual Result:
Center icon overlaps with compass icon
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[x] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/778a221d-284c-49cc-89fd-d17e7b2870ad

View all open jobs on GitHub

@trjExpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

Hm, @Expensify/design I'm not sure on this. Do we literally just cover the compass icon in the background or should it not be there at all?

Ah interesting, that only shows up if you rotate the map I bet?

Ah interesting, that only shows up if you rotate the map I bet?

Yup you're right. I had never seen that compass before but I just tested out rotating the map and that's when it shows up. Womp.

Goooot it. So both of these UI elements need to live together in the same view then conceivably?

It looks like we can also just move the compass or disable it entirely. From here:

So I like the idea of maybe moving the compass to the bottom right or something? Curious what @Expensify/design thinks.

Got it. Bottom right, above the info button? We can pay a retainer to get rid of all the Mapbox stuff at the bottom at some point in the future.
I think we should keep the compass, as I think it's pretty standard when turning the map to reorientate it.

Yeah, that's what I was thinking. And the info button would typically always show because the default view is not to rotate the map.

Sounds good!

Works for me, but I'll defer to Shawn!

I like it beneath the scale bar—it does feel kinda related and tucks up in there nicely. Will also wait for Shawn to weigh in 🙂

Oh I really like that too, let's do it

Let's do it!

@Krishna2323's alternative solutions option 2 using compassPosition to change the compass position LGTM!
also, result is accepted by the design team.
🎀 👀 🎀 C+ reviewed

Hi! I am a new developer on expensify.
I understand the bug

@Krishna2323's proposal LGTM.

@Krishna2323 please let me know when PR is ready. thanks!

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x] [@ahmedGaber93] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/40759
[x] [@ahmedGaber93] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/40759#issuecomment-2244596368
[x] [@ahmedGaber93] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x] [@ahmedGaber93] Determine if we should create a regression test for this bug. Yes
[x] [@ahmedGaber93] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Go to FAB > Submit expense > Distance
Android only - Look at the map view Verify compass icon is displayed by default on the top left corner and there is no scale bar
iOS only - Use two fingers to rotate the map and Verify compass icon is displayed after rotate on the top left corner and there is no scale bar

Do we agree 👍 or 👎

Payment summary as follows:

$250 to @ahmedGaber93 for C+
$250 to @Krishna2323 for the fix

Paid!


libxsmm_barrier_release segfaults when being called without initializing barrier

A call to libxsmm_barrier_release segfaults when calling it for a
a) correctly created barrier but
b) not initialized barrier (libxsmm_barrier_init was never called)
Desired behavior: libxsmm_berrier_release tracks if barrier was initialized and if not frees only the data allocated in libxsmm_barrier_create

bad wording in commit message, let's keep this open and discuss the fix

I am closing this since this has been discussed and no further issue is know so far. I will look into improvements of the barrier wrt different platforms and µarchs, but that is a different issue.


Tkinter error

I can see that tkinter is installed to this python client but when run I get the error below (program worked fine on widows pc)

this is my code:
import time
from tkinter import *
from tkinter import ttk
if name == 'main':
print('3DS.py brings Python to your Nintendo 3DS!')
print('starting up program in 3')
time.sleep(1)
print('2')
time.sleep(1)
print('1')
time.sleep(1)
root = Tk()
frm = ttk.Frame(root, padding=10)
frm.grid()
ttk.Label(frm, text="Hello World!").grid(column=0, row=0)
ttk.Button(frm, text="Quit", command=root.destroy).grid(column=1, row=0)
root.mainloop()

you cannot use tkinter with 3ds.py

Dang :/ thanks for letting me know!

imagine if you could though that would be cursed XD


DOC: add required scipy version to kwarg

Closes: https://github.com/scipy/scipy/issues/14444
What does this implement/fix?
Improves the docstring by adding the minimum scipy version required to use a new kwarg.

@tylerjereddy Maybe could be backported as relevant to 1.7.

ok, I added a backport label


Compatibility with Piwik 3

see piwik/piwik#10397

As discussed I will merge now and apply changes later


HHH-18960 Reverted mistakenly commited change

HHH-18960
Revert changes that were only for testing, not for commit :-(

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.

Thanks.


[New Model]: PartClassification 1.0.0

Model Description
This model is used as a shared aspect, that integrates classification properties into the models of the industry core. It provides the possibility for data providers to chose which classification standards for components or parts to use and therefore enables the network to have a broader acceptance towards the various classification standards used in the industry. Moreover, multiple standards can be used for one part these range from E-Class, to ISO, IEC, or STEP.
The requirement for such an shared aspect was identified due to the Industry Core efforts for harmonization, interoperability, and shall be integrated in all part related aspect models.
Relevant Standards
https://catena-x.net/fileadmin/user_upload/Standard-Bibliothek/Update_PDF_Maerz/8_Semantics/CX_-_0044_ECLASS_v_1.0.1.pdf
Example Data
{
"partClassification": [
{
"classificationStandard": "IEC",
"classificationID": "61360- 2:2012 ",
"classificationDescription": "Standard data element types with associated classification scheme for electric components "
}
]
}
MS1 Criteria

[ ] The proposed aspect model does not exist already
[ ] The proposed aspect model does not extend an existing aspect but introduces completely new functionality
[ ] The proposal references a Catena-X use case
[ ] Relevant standards are mentioned/linked
[ ] A first draft of the data structure is provided in an example
[ ] A modeling steward has been assigned

modeling_team

The proposed aspect model does not exist already -> OK
The proposed aspect model does not extend an existing aspect but introduces completely new functionality -> OK
The proposal references a Catena-X use case -> OK
Relevant standards are mentioned/linked -> OK
A first draft of the data structure is provided in an example -> OK
A modeling steward has been assigned -> @catroest


💡[FEATURE]: Improving the  UI of patient portal

Login UI of patient portal is not good, I want to work on it. @varshith257  and @Anishkagupta04  Can you assign me under GSSOC 24?

Give the layout you want to add

Something similar to this matching the theme of the current webpage, I implemented this style in my project

please assign me this issues  i can improve it

i  can make the login portal in that style


Fix err check for metadata json unmarshal

Description
Fix err check for metadata json unmarshal
Why is this needed
It is needed to see error if hw data is invalid.
Fixes: #
How Has This Been Tested?
Local setup.
How are existing users impacted? What migration steps/scripts do we need?
Users currently cannot see if there is an error related to metadata extraction.
Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @mfranczy thank you for your contribution. @thebsdbox is the person when it comes to actions!

@gianarb fair enough, but I don't have permissions to choose person as a reviewer (after the PR is open), maybe i missed something when I opened a PR. Sorry then.

No problem! I know. I left a message just to explain why I changed the
reviewer in this case!! Thanks a lot!!
On Tue, Mar 23, 2021 at 1:04 PM Marcin Franczyk @.***>
wrote:

@gianarb https://github.com/gianarb fair enough, but I don't have
permissions to choose specific person as a reviewer (after the PR is open),
maybe i missed something when I opened a PR. Sorry then.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tinkerbell/hub/pull/34#issuecomment-804846718, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMOAOZTOU6GCWXJATRRX5TTFB7TXANCNFSM4ZU7X4NQ
.


Poor performance using ARGB_8888

This issue continues the discussion from commit a3dcfbc.
By default this view uses RGB_565 to reduce memory usage and improve performance. When using ARGB_8888, under certain combinations of image size, screen size and screen density, rendering can become very slow. Simple gestures like pan and zoom cause large GPU spikes and frame rendering times in the hundreds of milliseconds even on a powerful device.
Testing with various combinations of settings has suggested the key limitation is not the total amount of image data drawn on the canvas (though this is a factor) but the size of the largest single bitmap.
RGB_565 will remain the default for now, and the view caps image density at 320dpi (retina), so most developers will not be affected by this. For those using ARGB_8888 and preferring to sacrifice image density for faithful colour reproduction, the density can be capped lower by calling setMinimumTileDpi(240). This should eliminate the problem for almost everyone, but may be noticeable on images with fine lines, e.g. maps and building plans.
Possible solutions
Render large tiles as segments
Reducing the maximum tile size to lower than the default, e.g. 2048px, causes much longer loading times but reduced rendering times. This implies that by rendering large tiles with 4 separate calls to drawBitmap for 4 segments of the bitmap, it may be possible to preserve loading performance and reduce rendering time, despite the total amount of image data written to the canvas being the same.
Convert to SurfaceView
This may help the view respond better to touch gestures but it's not clear whether it would improve rendering performance. Could be a breaking change for subclasses.
Convert to OpenGL
Converting to OpenGL would be a major change but may greatly improve performance. Could be a breaking change for subclasses.

I have tried rendering tiles as segments, but this performed no better than default. Using a matrix to draw a part of the bitmap is no faster than drawing the whole bitmap. Any workaround for this using smaller bitmaps would make loading much slower so this is a non-starter.
On a more positive note, API 26 added Bitmap.Config.HARDWARE and this gives far better results than ARGB_8888. It's worth trying this when it's available.

I have committed an experimental and very buggy SurfaceView implementation to the surface-view-experimental branch. Under certain circumstances this performs better than master, but in others it is slower.
SurfaceView canvas does not support hardware acceleration until API 26, and with this enabled, performance is far worse. It turns out that forcing software rendering for the view vastly improves performance for very large bitmaps that are displayed without subsampling, while quite noticeably reducing performance for smaller and subsampled bitmaps. In other words, software rendering is slower but lacks the large bitmap performance cliff edge seen with hardware rendering.
These are my results:
Bitmap config: ARGB_8888
Image: JPG photo 4912x3264
Screen: 1080x1920 landscape
Device: Nexus 5X
View master

Software canvas

Paging: VERY FAST
Pan/zoom: FAST

Hardware canvas

Paging: VERY SLOW
Pan/zoom: VERY SLOW

SurfaceView surface-view-experimental

Software canvas

Paging: MODERATE
Pan/zoom: MODERATE

Hardware canvas

Paging: VERY FAST (after redundant invalidation fix)
Pan/zoom: VERY SLOW

In conclusion, it's not worth migrating to SurfaceView because better results can be achieved by calling setLayerType(LAYER_TYPE_SOFTWARE, null) on the view. For now it's up to developers to do this manually if they're displaying large bitmaps and want them displayed at the highest possible resolution.
If I can work out where the performance crossover is, this could be done automatically, although that could cause unexpected rendering behaviour in subclasses.
OpenGL is still an option.

There's some great information about Bitmap.Config.HARDWARE here https://twitter.com/commonsguy/status/894983637585858560

Seems like it would risky to adopt that as a solution, given switching to software rendering produces a decent result and works on the other 99.7% of devices. It does make me wonder what will happen if I try to load lots of tiles into OpenGL textures and hit a limit.
@franciscofranco I know you're using 240dpi now, but if you get the chance could you try forcing software rendering with a much higher DPI to see if you get the same result I did?

Sure, I'll try that right now! Hold on.

Also, to add more fuel to the fire, setting both bitmap decoder and region bitmap decoder to 8888 makes things even worse, even something as low as 160dpi there's some stutter when scaling up/down during pinch to zoom or double tap to zoom.

This is the opposite behaviour to what I've observed. Calling setLayerType(LAYER_TYPE_SOFTWARE, null) on the view definitely makes it perform a lot better with a large unsampled bitmap at >320dpi, and there's some stutter but it's not bad considering my test image is about 60Mb as an 8888 bitmap.
It's hard to see how a large image wouldn't be subsampled if you set the quality to 160dpi, and I've certainly never seen poor performance with a subsampled image. Enabling debug will show you if the image has been subsampled - it will have a magenta border and the "ISS [x]" label.
As you have a Nexus 5X it's worth trying that in case this is hardware dependent.

I'll re-test everything on the Nexus 5X. With setLayerType(LAYER_TYPE_SOFTWARE, null) performance was destroyed on a Pixel. Even tested it with the native display dpi. Anyway I'll keep you posted of my tests, I want to help with this as much as I can.

On my N5X with the default hardware layer, 320dpi and 8888 there's the usual horizontal viewpager scroll jank, but zooming in/out + panning is 10x smoother. With a software layer, same setup it's smoother (though not perfect) horizontal viewpaget scroll, but zooming in/out + pan is completely destroyed. It seems software rendering is way better on the 5X compared to the Pixel in this specific case. 160dpi definitely subsamples, and there's still some jank, but it's tolerable, on the Pixel is destroyed. So with the 5X I can confirm your observations.

If pan+zoom are very slow with N5X/8888/>320dpi/software but fast with hardware, that's the opposite of my observations.
OpenGL may be the last hope for a universal fix. I don't think I'll have time to attempt it for a while, there are other projects I need to spend my time on.

Like game, It's vital to keep a low draw count.
When hasMissingTiles is true, ssiv will draw a lot, so I think I'd better develop a simple algorithm to avoid re-drawing overlapping bitmaps.
And what about a dynamic minimumTileDpi? For  example when scale>0.5, simply calculate scale = (220/averageDpi) * scale; instead of scale = (minimumTileDpi/averageDpi) * scale;, that will make the layer of smallest sample size apear in lager scale level.
By the way RGB_565 is slower than RGB_8888 in my device although it do save memory.


Have Criterions specify whether to "maximize best checkpoint metric"

Currently one is required to manually decide whether the validation metric should be maximized with --maximize-best-checkpoint-metric. This should be defined by the Criterion instead.

Shouldn't the criterion define a default metric, which can be overridden by --maximize-best-checkpoint-metric?  For example if I want to maximize bleu...


Do you use "fuse" optimization?

To complete the optimization pipeline I adapted opeo-maven-plugin to make fuse optimization possible. I even added the integration test for the fuse optimization.
But as I see, you don't use fuse optimization anymore. Is it correct?

@yegor256 Could you help here, please?

@volodya-lombrozo we use ineo:staticize now, not fuse, because fuse doesn't produce any performance effect

@yegor256 Thank you


⚠ Fox Sports has degraded performance

In bb2f738, Fox Sports (https://foxsports.com) experienced degraded performance:

HTTP code: 200
Response time: 2331 ms

Resolved: Fox Sports performance has improved in 18c955d.


How to understand the figure 4

Hi, I found that the symbols in the figure 4 in the paper are similar to Candlestick chart in the stock market, but I don't know how to understand it, could you please describe it?

This a box plot. You can read more about that kind of plot.  Link


Add part 2 of the tutorial: CLI Contact Manager

This PR mostly addresses two important things:
The first is a project that uses this library to create a small contact manager using the command line. There is both the running code in the examples and a tutorial under docs/tutorial/contact_manager.md explaining the hows and whys.
You can use it by doing cm help to display how to use the command.
Regarding the project, this is what is contained on each folder:

contacts/domain: The types of the application, such as Contact, CliCommand and Flag, that address the in memory representation of a contact in Scala, the command to execute in the CLI and the different update flags for the update command, respectively.

contacts/core: Here is the ContactManager algebra. This uses shellfish to add, remove and update the contacts on a particular file. It also has methods to search uses by their name, email and phone number.
Regarding the implementation, it stores the contacts on a custom encoding that's basically every contact field divided by a |, and stores them line by line.

contacts/cli: This class has the functionality to interact with the user in the Cli, it prints to the stdout and takes user input to execute the commands; the Prompt object has functionality to parse the user input in form of flags and subcommands to convert them to instructions to the Cli.

contacts/app: This is basically the entry point of the application.

Any changes or doubts about the code are very welcome!

Changes done!
Also, the tutorial is broken until #88 gots merged.

Suggestions committed!

Changes done! Shall merge this PR or close it and add the changes to Catscript @TonioGela ?


⚠ Turnos 24x7 has degraded performance

In 85bb03e, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 975 ms

Resolved: Turnos 24x7 performance has improved in 80ac8ce.


missing docs for the pricing policy

https://github.com/threefoldtech/tfchain/commit/4976e7dbbfdb3b9a5116a3610276a480ffd0adfb


Extend ec2-bootstrap concurrency to host

@jfieber @gregoryfischer I'm interested in your thoughts on this one.
We could extend concurrent: true to the host and change how ec2-bootstrapping works a bit. Instead of writing to /usr/bin/porter_bootstrap directly porter could run these scripts and leverage knowledge of ordering constraints or lack thereof to increase the speed that ec2 initialization occurs

I would enthusiastically support the ec2 hook contract being: output of hook is something porter can save in a file, chmod u+x, and then fork/exec, with appropriate environment variables and std{in,out,err}.  Generating a fragment of a shell script is brittle, and promotes unexpected cross-hook interactions.


Fix externalizing config

:tada: This PR is included in version 0.3.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[CH-223]Support TakeOrderedAndProjectExec

What changes were proposed in this pull request?
Support running TakeOrderedAndProjectExec in native engine.
How was this patch tested?
unit tests
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

The final plan looks like

@rui-mo please check what is wrong in compiling velox
-- Found folly: /usr/local
CMake Error at velox/CMakeLists.txt:58 (message):
  Velox library not exists:
  /opt/gluten/ep/build-velox/build/velox_ep/_build/release/velox/vector/arrow/c/libvelox_arrow_bridge.a
Call Stack (most recent call first):

@rui-mo please check what is wrong in compiling velox
-- Found folly: /usr/local
CMake Error at velox/CMakeLists.txt:58 (message):
  Velox library not exists:
  /opt/gluten/ep/build-velox/build/velox_ep/_build/release/velox/vector/arrow/c/libvelox_arrow_bridge.a
Call Stack (most recent call first):

Velox backend needs this PR https://github.com/oap-project/gluten/pull/637, which will be merged soon. Maybe you can rebase after that, thanks.

@rui-mo I don't find the failed ut in the velox logs. Could you check it?


Update EstimationParameters.py

"beth" is correct.  See SolvingMicroDSOPs.pdf, which uses the Hebrew letter beth (or bet) as a multiplicative shifter for an age-varying sequence of discount factors.

Oh yeah!

Yeabin: Can you do another commit reverting beta --> beth?  Then I can merge.

Of course. Check it please.


Components diagram in Docs

Reversed the direction of all "Search" arrows. Looks kinda weird with all arrows leading to Indices. Maybe we should also reverse arrows form Shards to DataDources?
Made a gap for less ambiguity.


Add link to commit in build details

As title, this fixes #580
Screenshot:

So, I renamed GitRepoConfig to GitHubRepoConfig and added a link_to_remote_commit there, so we avoid implementing the different cases. (I forced push on my branch so we lost those comment, sorry 🐕 )


Getting started guide: Step "cargo run" (after step "cargo update color") fails

cargo run
   Compiling color v0.0.1 (https://github.com/bjz/color-rs.git#2fe276d1)
   Compiling hello_world v0.0.1 (file:///home/rj/Documents/rust/hello_world)
src/main.rs:8:23: 8:42 error: the trait `core::fmt::String` is not implemented for the type `color::hsv::Hsv<f32>`
src/main.rs:8   println!("HSV: {}", red.to_hsv::<f32>());
                                    ^~~~~~~~~~~~~~~~~~~
note: in expansion of format_args!
<std macros>:2:42: 2:75 note: expansion site
<std macros>:1:1: 2:77 note: in expansion of println!
src/main.rs:8:3: 8:44 note: expansion site
error: aborting due to previous error
Could not compile `hello_world`.

/cc @bjz

Cheers, will address.

We derive Show - perhaps the example should use {:?} ?


update how to persist enabling ipv6 forwarding across reboot

Description
fixes this https://github.com/kubernetes/website/issues/48403
Issue
Closes: https://github.com/kubernetes/website/issues/48403
Current

Proposed

/lgtm


Can't Compile Modules?

I feel like I'm missing something obvious, but it doesn't appear to compile:
~/w/gemacs (master) [1]> go mod init gemacs
go: creating new go.mod: module gemacs
go: to add module requirements and sums:
	go mod tidy
~/w/gemacs (master)> go mod vendor
go: gemacs imports
	github.com/gdamore/tcell/encoding: no required module provides package github.com/gdamore/tcell/encoding; to add it:
	go get github.com/gdamore/tcell/encoding
go: gemacs imports
	github.com/gdamore/tcell/termbox: no required module provides package github.com/gdamore/tcell/termbox; to add it:
	go get github.com/gdamore/tcell/termbox
go: gemacs imports
	github.com/glycerine/verb: no required module provides package github.com/glycerine/verb; to add it:
	go get github.com/glycerine/verb
go: gemacs imports
	github.com/nsf/tulib: no required module provides package github.com/nsf/tulib; to add it:
	go get github.com/nsf/tulib
~/w/gemacs (master) [1]> go install
vendor/github.com/gdamore/tcell/encoding/all.go:18:2: cannot find module providing package github.com/gdamore/encoding: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/colorfit.go:18:2: cannot find module providing package github.com/lucasb-eyer/go-colorful: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/cell.go:18:2: cannot find module providing package github.com/mattn/go-runewidth: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/encoding.go:21:2: cannot find module providing package golang.org/x/text/encoding: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/simulation.go:21:2: cannot find module providing package golang.org/x/text/transform: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/encoding/all.go:21:2: cannot find module providing package golang.org/x/text/encoding/charmap: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/encoding/all.go:22:2: cannot find module providing package golang.org/x/text/encoding/japanese: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/encoding/all.go:23:2: cannot find module providing package golang.org/x/text/encoding/korean: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/encoding/all.go:24:2: cannot find module providing package golang.org/x/text/encoding/simplifiedchinese: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)
vendor/github.com/gdamore/tcell/encoding/all.go:25:2: cannot find module providing package golang.org/x/text/encoding/traditionalchinese: import lookup disabled by -mod=vendor
	(Go version in go.mod is at least 1.14 and vendor directory exists.)

My understanding is that the vendor/ folder is used to hard pin dependencies. In this case, it appears they don't include enough dependencies.

Yes. That's my experience too. I tried to build gemacs last month but could not get it to build. My attempts at vendoring many years ago seem to have been insufficient. So yes, it is broken under modules; it was written before modules and I've not had the time to update it to work with modules.
PR to fix certainly welcome.

It might be worth exploring if the keybindings for https://github.com/zyedidia/micro can be made sufficiently emacs like for your fingers. It appears micro is much more maintained than gemacs (well anything would be, as I've not needed gemacs for anything).

I've made the latest version module compatible. You should be able to build it now. I put a note on the README.  You'll need go1.23.3 or later (the latest as of this writing) because I could not figure out how to get go mod to not update it to the latest. grrrr... ). Anyway, if you cannot build, @valexandersaulys , please let me know.

closing as of v1.1.1


Improve README.md

This fixes some minor grammatical errors in the README of the project, and adds more explanations to some sections of the README.

Nice! please do also change the "issues" link on the last line to point to this new repo-location (under the Testausserveri organization)
And also while you're at it, Could you add a part to the "Hosting" section where you describe the necessary permission-scopes that are required on-invite in order for the bot to be able to initialize application commands on the hosts guild-of-selection.
Here's a picture of the necessary scopes:

Otherwise this looks good, and it's definitely an improvement over the previous one. :)

Also, wouldn't it make sense to call "vahti"s something else, such as "watchers"? Would make more sense for non-finns.

@romeq I don't think so, The choice to use the word "Vahti" was quite deliberate on my part. And also, we do not support other languages than Finnish anyways, so I don't really see a point in not using the term + all of the end-users (AFAIK) are Finns.

If the whole project doesn't support any other language than finnish, why are we even speaking english in the first place? :p

It's good practice since most of the "technical" world is run in English. In my opinion it would be more difficult to talk about code in Finnish because I (and the Finnish language) lack(s) the terminology for a lot of things.

Fair enough.

An .example.env could be handy to add, as it currently takes a bit of time to copy the variable keys by hand to a new file.
Roni has developed an useful tool by name env-exampler to generate the file from a .env file, so just run npx env-exampler and push the changes.

@ahnl Sounds good. Though, this pull request was for README.md as shown in the title.
Could that possibly be made in another pull request, just to keep things organized?

How ever you want to. Imo its related to documentation, so can keep in this PR as well.

Is there some way this merge could be redirected to the main branch instead of next? I think it makes no sense to merge this to the next branch since it only consists of improvements to the documentation and doesn't document any features that are exclusive to the next branch.

Changed to target main

This now looks good to me. I'll be merging this shortly and adding the requested .env.example in another commit.


cycle stats based on proposal types

cycle stats needs to be separated out for each type of proposal

Tried this, but didn't make a difference and actually had a negative impact.
This is a lot of work and not sure it makes sense to do.
Will be fixed by using the current deployed vote power and comparing to that


WebIDL Binder？

Export the lvgl library as a directly callable JS file.

Would it be a JS binding for LVGL?


RF=1 and locking

Just from a quick look it seems to me you are not taking the distributed nature of Cassandra into consideration here.
First of all tracking the state in an RF=1 key space makes little sense when you loose a node and basically loose the state of your evolutions.
Second I am unable to quickly spot a line where you lock on the evolution id, I would expect to use at least a lightweight transaction on insert or update before starting to execute the evolution as today a lot of services do not start up on a single node.

You are right, this adapter is kind of an MVP in context of a specific project.
As for the the replication factor: Well, yes that should be a configuration parameter. RF=1 is good for development (as you usually only have one node there), in production it should be higher of course ... we adapted it to our needs and cluster size via the opscenter.
Locking is more tricky as there are different ways (and requirements) to implement this in a cluster. I did not wanted to enforce a specific strategy, instead locking is intentionally left open by overriding the "CassandraEndpointConfig.executeWithLock" function.
In our project we configured cassandra via etcd, which can be used for locking as well.
A pure cassandra solution would probably be a CAS operation on a row with expiration.


WIP: Allow specify headers to remove in response and config struct change

What?

Fixes the configuration syntax for router.http_rewrite.responses.add_headers_if_not_present  to match https://github.com/cloudfoundry/routing-release/pull/128#issuecomment-436753733
Implements #129: Operator can specify headers that should be deleted from gorouter responses

An explanation of the use cases your change solves
See #129
Instructions to functionally test the behavior change using operator interfaces (BOSH manifest, logs, curl, and metrics)
Configure with:
http_rewrite:
  responses:
    remove_headers:
    - name: X-Vcap-Request-Id

Expected result after the change
After configuring a manifest property with X-Vcap-Request-Id and deploying, responses should not include X-Vcap-Request-Id
Current result before the change
Currently gorouter returns the HTTP header X-Vcap-Request-Id
Links to any other associated PRs
https://github.com/cloudfoundry/gorouter/pull/232
Checklist

[x] I have viewed signed and have submitted the Contributor License Agreement

[x] I have made this pull request to the master branch

[ ] I have run all the unit tests using bin/test

[ ] I have run CF Acceptance Tests on bosh lite

Thank you for the quick turnaround on this! Please send us a matching PR for routing-release.

I have run all the unit tests using bin/test

Got one error, but I do not think it is related
[Fail] Proxy Backend Connection Handling when a TLS handshake occurs when the server cert does not match the client when MaxConns is > 0 [It] prunes the route
/home/keymon/workspace/routing-release/src/code.cloudfoundry.org/gorouter/proxy/proxy_test.go:979

Ran 150 of 150 Specs in 20.807 seconds
FAIL! -- 149 Passed | 1 Failed | 0 Pending | 0 Skipped

I have run CF Acceptance Tests on bosh lite

I do not have an environment and release ready to test. Could do tomorrow or so.

I am considering if it is worth it to make the AddHeaderIfNotPresentRewriter ensure that the headers are right, or just rely on what is using this helper to build the http.Header using the methods to ensure cannonical headers.
Likely I will do the later.
diff --git a/proxy/utils/headerrewriter.go b/proxy/utils/headerrewriter.go
index 22c4a3f..b72f9a3 100644
--- a/proxy/utils/headerrewriter.go
+++ b/proxy/utils/headerrewriter.go
@@ -14,9 +14,12 @@ type AddHeaderIfNotPresentRewriter struct {
 }

 func (i *AddHeaderIfNotPresentRewriter) RewriteHeader(header http.Header) {
-       for h, v := range i.Header {
-               if _, ok := header[h]; !ok {
-                       header[h] = v
+       for h, values := range i.Header {
+               if header.Get(h) == "" {
+                       header.Del(h)
+                       for _, v := range values {
+                               header.Add(h, v)
+                       }
                }
        }
 }

I am considering if it is worth it to make the AddHeaderIfNotPresentRewriter ensure that the headers are right, or just rely on what is using this helper to build the http.Header using the methods to ensure cannonical headers.

Added tests to ensure the handlers do the right thing.

Your changes should be in Routing Release 0.183.0. Thanks for your contributions!


Should include LICENSE header comments

Richard Stallman himself has warned of the dangers of executing closed-source javascript.
While this repository does include a "license" (dedication to public domain, whatever), there is no copyright notice included in the html, css, or javascript of the webpage.
This could lead users of the webpage to not realize their rights.
Secondly, I think you'll find that this page would best be distributed under the AGPL. I believe that there are few enough contributors that relicensing should be possible.
We have been warned of the dangers of proprietary SaaSS offerings, and I would hate to think that due to a non-copyleft license, this service might one day be hosted by a SaaSS company like Amazon to the user's detriment.
If you wish to keep it public domain, the fsf recommends CC0 instead btw.

So add license headers and relicense to AGPL?
@clsr Is that fine with you?

It's public domain, you don't actually have to ask me to relicense.
While I generally disagree with the way AGPL works (software licenses
should put restrictions on developers, not users), I do understand the
problem it tries to solve. Feel free to relicense my contributions to
AGPL.

You actually bring up a good point though @clsr; since it's already public domain, I don't think it's possible to put it back under copyright (or, in this case, copyleft).
The cat's already out of the bag so to speak.
It seems like the only way this project can fully preserve user's essential rights is through a full, properly licensed, full reimplementation.
For now, I'll make a PR which includes suitable header-comments so users know it's public domain...
but I think the only way to fully preserve user's rights would be a reimplementation that's licensed correctly from the get-go.

It certainly is possible to relicense it. The current version of the
code will remain available in the public domain, but any future updates
would be licensed under the new and more restrictive license.


Avoid variable declaration if SDO client is disabled

If USE_CSDO is set to 0, compiler complains about unused variable csdo in CONodeProcess(). Fix that.

Thank you very much for your contribution. I've just checked your changes and want to merge them.
Could you change the merge target branch of your pull request to embedded-office:develop, please? I try to follow the git workflow as good as possible.

Rebased and updated target branch.


Provide a link template for external logs in Argo UI

Summary
A way of providing a templated link for externally-hosted logs, visible in the Argo UI for task and workflow logs
Motivation
Logs for GKE containers exported to Stackdriver, accessible via a link. Example:
https://console.cloud.google.com/logs/query;query=resource.type%3D%22k8s_container%22%0Aresource.labels.pod_name%3D%22argo-workflow-5hjbp%22?project=myproject&query=%0A
Template might be
https://console.cloud.google.com/logs/query;query=resource.type%3D%22k8s_container%22%0Aresource.labels.pod_name%3D%22{{pod_name}}%22?project=myproject&query=%0A - Argo doesn't know what myproject is, so it would likely need to provided as part of deployment automation effectively hardcoded into the template
Proposal
How do you think this should be implemented?

Configuration option for a templated external log link that is interpolated and displayed as a clickable link in the Argo UI.
Probably best a key and value, key is what is display as the link text - might want to say explicitly "View in Stackdriver" or "View in Elasticsearch" or provide a non-English text
Also good to be visible in kubectl get pod output?

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

This feature is already supported:
https://argoproj.github.io/argo/links/
   links:
      - name: Example Workflow Link
        scope: workflow
        url: http://logging-facility?namespace=${metadata.namespace}&workflowName=${metadata.name}
      - name: Example Pod Link
        scope: pod
        url: http://logging-facility?namespace=${metadata.namespace}&podName=${metadata.name}


Please verify that spec api artifacts for 2.1.0 are available on maven central...

Is the Jenkins job that handled the 2.1.0 spec api release pushing to Maven central?
Maven repo only seems to have 1.1.1
CC @lukasj + @Tomas-Kraus

there's no 2.1.0 right now. It must go through ballot first.

same issue here ;-) will it be releasd soon ?

this is done


Clean up deployment workflow

This PR aims to simplify both development and deploy workflows by making the frontend app easier to run and reducing the differences between what runs in local dev and what gets deployed.
Notable changes:

PUBLIC_URL has been removed; the frontend is now agnostic to where it gets deployed (meaning it uses domain-less URLS like /static/js/main.12a34bc.js to fetch required resources). This means you can build one artifact and deploy it anywhere, rather than needing a different artifact for each server you're deploying to (including localhost).
The Docker image now includes a default nginx.conf file. This file is suitable for production use (though it's HTTP, so it should be used along with a TLS-terminating reverse proxy). In a future PR I'll update the Helm charts in the deploy repo to remove the nginx config file that's inlined there, and use this one instead.
Cleaned up a bunch of old scripts and snippets that were only used when the frontend used to be served from Github Pages.

The changes here go along with some changes to the Django app configuration here: https://github.com/OSMCha/osmcha-django/pull/719

No worries, thanks for the review! I'm going to hold off on merging this until OSMCha/osmcha-django#719 is also approved. After that I'll work on the required changes to osmcha-deploy as mentioned in that PR.


__gnat_mulv64 missing on arm-elf ZFP

When building against a ZFP runtime on arm-elf, the 64 bit multiplication is missing:
Link
   [link]         main.adb
/opt/GNAT-Pro-20.2_arm-elf/bin/../lib/gcc/arm-eabi/8.3.1/../../../../arm-eabi/bin/ld: build/example/rflx-spdm-response.o: in function `rflx__spdm__response__set_field_value':
rflx-spdm-response.adb:(.text.rflx__spdm__response__set_field_value+0x3bd2): undefined reference to `__gnat_mulv64'
collect2: error: ld returned 1 exit status
gprbuild: link of main.adb failed

This is probably caused by the U64 type used in the Extract and Insert functions. A solution could be to only use a 64-bit integer, if it is strictly required, and use a 32-bit integer instead. A simple fix could be to change the definition of U64 in the RFLX_Arithmetic package to mod 2**32.
@senier Could you please try if that fixes the issue? Do we want to test that scenario (arm-elf + ZFP) in the CI?

@senier Could you please try if that fixes the issue?

Apparently, not completely. I had to replace a mod 64 reserved field type by two 32 bit fields in the spec (which was expected), but the 32 bit types still causes problems:
   [Ada]          rflx-spdm-request.adb
rflx-spdm-request.adb:739:07: instantiation error at rflx-rflx_generic_types.adb:198
rflx-spdm-request.adb:739:07: value not in range of type "U64" defined at rflx-rflx_generic_types.ads:54, instance at rflx-rflx_types.ads:14
rflx-spdm-request.adb:739:07: "Constraint_Error" would have been raised at run time

-- rflx-rflx_arithmetic.ads
type U64 is mod 2**32 with
   Annotate => (GNATprove, No_Wrap_Around);
-- rflx-rflx_generic_types.ads:54
subtype U64 is RFLX.RFLX_Arithmetic.U64
...
type Reserved_32_Base is mod 2**32 with
   Annotate => (GNATprove, No_Wrap_Around);
...
--  rflx-spdm-request.adb:739:07
function Extract is new RFLX_Types.Extract (RFLX.SPDM.Reserved_32_Base);

Makes no sense to me...

I'm unable to reproduce the linker error. I tried using GNAT Pro 20.2 and 22.0 for arm-elf with zfp/light for one of our integration tests, but didn't get the linker error.
   for Target use "arm-eabi";
   for Runtime ("Ada") use "light-cortex-m4";  -- with 20.2
   for Runtime ("Ada") use "zfp-stm32f4";      -- with 22.0

Is there anything else I have to do to trigger the error?

Here is what I was using (you have to comment out the with Arith64 workaround in main.adb).

I can reproduce the issue in your example with 20.2 and zfp runtime. The issue does not occur with 22.0 and light runtime.

The "value not in range" error was caused by the Compile_Time_Error pragma in RFLX_Generic_Types, where 64 must also be replaced by 32:
      pragma Compile_Time_Error ((if Value'Size = 32 then
                                    U64 (Value'First) /= U64'First or U64 (Value'Last) /= U64'Last
                                 else
                                    U64 (Value'Last) - U64 (Value'First) /= U64 (2**Value'Size) - 1),
                                 "Value must cover entire value range");

After fixing that, there were still linker errors (e.g., due to a calculation in Field_Size), which are caused by the Bit_Length type in the RFLX_Builtin_Types package. By default, it requires a size of 34 bit:
   type Length is new Natural;

   type Bit_Length is range 0 .. Length'Last * 8;

After changing that type, all linker errors are gone.
To support arm-elf with zfp the following has to be done:

[ ] Reduce size of RFLX_Builtin_Types.Bit_Length to 32 bit
[ ] Determine maximum size of scalar types used in messages
[ ] Reduce size of RFLX_Arithmetic.U64 to 32 bit, if no scalar type greater 32 bit is used in any message
[ ] Add test for this scenario

Must be user-provided.


Support 'is' prefix in proxy class to have boolean getters

In case your entities using booleans and you prefer to use the 'is' prefix instead of the 'get' the generated proxy will ignore it. The PR modifies the proxy method filter and enables 'is' prefixes.

Great, thanks!


Implement to fetch data of sponsors

Issue

close #178

Overview (Required)

Fetch data from DataSource.
Add ViewModel, Repository to flow data to UI.
Not Implement UI.

Sorry for late 🙇
I'll apply design with Compose, is the design completed? -> #179
Or should I draft and implement a design? 👀

Now, the design is added!

Thanks for the quick response to add the design 🙏


Merge https://github.com/kubernetes-sigs/cluster-api-provider-vsphere:release-1.5 (f4a58d1) into master

This PR merges upstream's https://github.com/kubernetes-sigs/cluster-api-provider-vsphere:release-1.5 (f4a58d1) into master.
Check this comment for this CAPI provider bump testing status.

/retest

/retest

/retest

For coordination with other CAPI components:
/hold

/approve


feat: next 15

✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog

Update to Next 15
Rename next.config.js to .ts
Rename env.js to .ts
Update next-auth to v5 (beta)

Todo:

[x] Test NextAuth with Next 15

How useful really is this considering the cjs limitations?

How useful really is this (ts config) considering the cjs limitations?

I don't like seeing js files in my src folder. With ts next config env.js can be env.ts. I don't think it changes anything else.
next.config.ts should be more useful in create-t3-turbo. I think we can remove jiti dependency there, when we update to next 15 there.

Are you sure this works? t3-env is an ESM only package which is called out to be incompatible

Are you sure this works?
I tried multiple configuration, everything seems to work. Dev, Build, LSP all work. I get an error if for env variables are missing.

Try doing something esm in it, it will crash
This works:
// app/env.js
import { createEnv } from "@t3-oss/env-nextjs";
import { vercel } from "@t3-oss/env-nextjs/presets"
import { z } from "zod";

console.log(import.meta.filename)

export const env = createEnv({
  extends: [vercel()],
  server: {
    FOO: z.string()
  },
  experimental__runtimeEnv: {
    FOO: "bar"
  }
});

// next.config.js
import "./app/env.js"

export default {
  /* config options here */
};

Change file extensions to .ts. It crashes:

oh, ok. I reverted the next config changes. We can add them later when the ESM thing is fixed.

Can @t3dotgg @c-ehrlich and @nexxeln please review the code please?? I think it would be useful to have it implemented already

Just tested your changes, everything looks good @juliusmarminge


ValueError: setting an array element with a squence

hello there, i'm getting the following error:
ValueError                                Traceback (most recent call last)
<ipython-input-45-0601c7d807e8> in <module>
----> 1 clf.fit(X_train_vec, y_train_lil)
/mnt/storage/clfiles/resources/applications/toolkit/anaconda/3-v5.2.0/lib/python3.6/site-packages/skmultilearn/embedding/classifier.py in fit(self, X, y)
    113         y = self._ensure_input_format(y)
    114
--> 115         y_embedded = self.embedder.fit_transform(X, y)[1]
    116         X_y_embedded = self._concatenate_matrices(X, y_embedded)
    117
/mnt/storage/clfiles/resources/applications/toolkit/anaconda/3-v5.2.0/lib/python3.6/site-packages/skmultilearn/embedding/openne.py in fit_transform(self, X, y)
    142         param_dict[dimension_key] = self.dimension
    143         self.embeddings_ = embedding_class(**param_dict)
--> 144         return X, self._embedd_y(y)
    145
    146     def _init_openne_graph(self, y):
/mnt/storage/clfiles/resources/applications/toolkit/anaconda/3-v5.2.0/lib/python3.6/site-packages/skmultilearn/embedding/openne.py in _embedd_y(self, y)
    170             if len(row) > 0 else empty_vector
    171             for row in (y.A if isinstance(y, np.matrix) else y)
--> 172         ]).astype('float64')
    173
    174
ValueError: setting an array element with a sequence.

i know what the valueerror means, but i have no idea where it comes from (except that it must happen in the _embed_y function). i have an input matrix X, which are tfidf scores of several thousand documents. i have a binary label vector of size 50, where the 1 represents that a label is present and a 0 represents the absence of a label, respectively. i converted both matrices to the linked list data format. nevertheless i get the error above. any clues why that might be?
here is some more of my code:
--> prepare data
X = df.text
y = df.drop(['text'], axis=1)
X_train, X_test, y_train, y_test = train_test_split(X, y, test_size=0.1)
X_train.shape, X_test.shape, y_train.shape, y_test.shape
>>>((8982,), (999,), (8982, 50), (999, 50))

--> converstion binary
y_test_enc = y_test.as_matrix().astype(float).tolil()

--> conversion to lil

y_train_sparse = csr_matrix(y_train_enc, dtype=int)
y_test_sparse = csr_matrix(y_test_enc, dtype=int)

X_train_lil = X_train_vec.tolil()
X_test_lil = X_test_vec.tolil()
y_train_lil = y_train_sparse.tolil()
y_test_lil = y_test_sparse.tolil()

for clarification, these are the dimensions of X_train_lil, X_test_lil, y_train_lil and y_test_lil:
(<8983x50000 sparse matrix of type '<class 'numpy.float64'>'
 	with 6006985 stored elements in LInked List format>,
 <998x50000 sparse matrix of type '<class 'numpy.float64'>'
 	with 673669 stored elements in LInked List format>,
 <8983x50 sparse matrix of type '<class 'numpy.int64'>'
 	with 38792 stored elements in LInked List format>,
 <998x50 sparse matrix of type '<class 'numpy.int64'>'
 	with 4336 stored elements in LInked List format>)

--> setup of skmultilearn
from skmultilearn.cluster import LabelCooccurrenceGraphBuilder
from skmultilearn.embedding import EmbeddingClassifier
from sklearn.ensemble import RandomForestRegressor
from skmultilearn.adapt import MLkNN

graph_builder = LabelCooccurrenceGraphBuilder(weighted=True, include_self_edges=False)
openne_line_params = dict(batch_size=1000, order=3)
embedder = OpenNetworkEmbedder(
    graph_builder,
    'LINE',
    dimension = 5*y_train.shape[1],
    aggregation_function = 'add',
    normalize_weights=True,
    param_dict = openne_line_params
)

clf = EmbeddingClassifier(
    embedder,
    RandomForestRegressor(n_estimators=10),
    MLkNN(k=5)
)

clf.fit(X_train_lil, y_train_lil)

predictions = clf.predict(X_test_lil)

in my opininon, the dimensions are correct. does anybody have an idea, where this might come from?

I have the exact same error and data set up (TF idf transformed data, multiple labels).
`Pre-procesing for non-uniform negative sampling!
epoch:0 sum of loss:4.424449801445007
epoch:1 sum of loss:4.447336554527283
epoch:2 sum of loss:4.350230276584625
epoch:3 sum of loss:4.3726524114608765
epoch:4 sum of loss:4.394627273082733
epoch:5 sum of loss:4.384531915187836
epoch:6 sum of loss:4.352884590625763
epoch:7 sum of loss:4.348439514636993
epoch:8 sum of loss:4.3353071808815
epoch:9 sum of loss:4.30299586057663
TypeError                                 Traceback (most recent call last)
TypeError: only size-1 arrays can be converted to Python scalars
The above exception was the direct cause of the following exception:
ValueError                                Traceback (most recent call last)
 in
C:\ProgramData\Anaconda3\envs\skmulti\lib\site-packages\skmultilearn\embedding\classifier.py in fit(self, X, y)
113         y = self._ensure_input_format(y)
114
--> 115         y_embedded = self.embedder.fit_transform(X, y)[1]
116         X_y_embedded = self._concatenate_matrices(X, y_embedded)
117
C:\ProgramData\Anaconda3\envs\skmulti\lib\site-packages\skmultilearn\embedding\openne.py in fit_transform(self, X, y)
142         param_dict[dimension_key] = self.dimension
143         self.embeddings_ = embedding_class(**param_dict)
--> 144         return X, self._embedd_y(y)
145
146     def _init_openne_graph(self, y):
C:\ProgramData\Anaconda3\envs\skmulti\lib\site-packages\skmultilearn\embedding\openne.py in _embedd_y(self, y)
170             if len(row) > 0 else empty_vector
171             for row in (y.A if isinstance(y, np.matrix) else y)
--> 172         ]).astype('float64')
173
174
ValueError: setting an array element with a sequence.
`

same error here.


fix aria:Link disabled color when skin contains multiple sclass

This commit fixes the following issue: when different sclass with different colors for the disabled state are defined for the aria:Link widget, all disabled links were using the same color for the disabled state, regardless of the sclass property.

bee9cc2 is fine for me

@fbasso Thank you for your review


Mishandling of comments in ARM assembly

I was referred to make this bug report by Discord support after seeing this behavior in Discord, which uses highlight.js.
When highlighting ARM assembly, # comments and // comments are not treated as comments but instead as live code. This leads to incorrect highlighting that can span the entirety of the code, see the included screenshot.
Example (from Discord)

Please include raw Sample text here not just an image.

  //=================== YOUR CODE GOES IN THE SECTION BELOW ===================//

    # check for and handle zero
    orr r1, r0, #128            // r1 = r0 | 128
    cmp r1, #128                // if r1 == 128 {
    bne notZero                 // .
    bl zeroQuidFP2float         //          zeroQuidFP2float() }
    b end                       // else {

notZero:                        // .
    # extract fields from quidfp
    and r1, r0, #128            //          r1 = r0 & 128     // sign
    and r2, r0, #112            //          r2 = r0 & 112     // exponent
    and r3, r0, #15             //          r3 = r0 & 15      // mantissa

    # adjust for larger bias (add 32-bit bias - 8-bit bias << 4)
    add r2, r2, #1984           //          r2 = r2 + 1984

    # move fields into appropriate location for IEEE-754 single-precision fp
    lsl r1, r1, #24             //          r1 = r1 << 24
    lsl r2, r2, #19             //          r2 = r2 << 19
    lsl r3, r3, #19             //          r3 = r3 << 19

    # combine into r0
    orr r0, r1, r2              //          r0 = r1 | r2
    orr r0, r0, r3              //          r0 = r0 | r3 }

end:                            //  return r0

  //===========================================================================//

Correctly highlighted by the ARM VSCode extension by @dan-c-underwood
Incorrectly highlighted by highlight.js in Discord

And how are # comments to be distinguished from numbers (#24) and label references ({begin: '[=#]\\w+' }  //label reference)?

Playing with the VSCode extension, # is only counted as the start of a comment if it's the first non-whitespace character on a line.


allow access to worker nodes from operator host

Signed-off-by: Ali Mukadam<EMAIL_ADDRESS>

This will allow ssh access to worker via bastion host and bastion service via operator.if bastion_enabled is set to false still the ingress rule will be created for bastion subnet. in future if bastion_enabled set to true it can be used.

The control variables have been renamed btw. It's now create_bastion_host