Why does the use_inheritance option not apply inheritance?

Please describe a problem.
I have a swagger definition heavily dependent on inheritance in different models. However when setting use_inheritance to true in build.yaml inheritance is not applied to models.
According to this changelog use_inheritance was removed. It is also said in the changelog that use_inheritance was removed because Inheritance is supported by copy-pasting parent fields.
Describe the solution you'd like
Could this be elaborated on further since it seems unclear? Also if use_inheritance was removed an update to the documentation would be good since it's still present there. Also it would be good if there were additonal docs how to apply inheritance if not through the use_inheritance field

Hi @boriszv
Actually, nice catch. We removed this option from generator because swagger supports cases, when class A extends B and B extends A. So we decided just to copy-paste all fields from base class.
I will create PR to update Readme.

@boriszv  Do you have any issues with inheritance?

@boriszv Do you have any issues with inheritance?

@Vovanella95
Actually, yes
My swagger spec looks something like this:
{
    "x-generator": "NSwag v<IP_ADDRESS> (NJsonSchema v<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>))",
    "swagger": "2.0",
    "info": {
        "title": "**omitted**",
        "description": "**omitted**",
        "contact": {
            "name": "**omitted**",
            "url": "**omitted**",
            "email": "**omitted**"
        },
        "license": {
            "name": "**omitted**",
            "url": "**omitted**"
        },
        "version": "1.0.0"
    },
    "host": "**omitted**",
    "schemes": [
        "https"
    ],
    "paths": {
        "/EntityNamees": {
            "post": {
                "tags": [
                    "EntityNamees"
                ],
                "operationId": "EntityNamees_AddEntityName",
                "consumes": [
                    "application/json-patch+json",
                    "application/json",
                    "text/json",
                    "application/*+json",
                    "application/xml",
                    "text/xml",
                    "application/*+xml"
                ],
                "parameters": [
                    {
                        "name": "EntityNameDto",
                        "in": "body",
                        "required": true,
                        "schema": {
                            "$ref": "#/definitions/EntityNameForCreateDto"
                        },
                        "x-nullable": false
                    }
                ],
                "responses": {
                    "200": {
                        "x-nullable": true,
                        "description": "",
                        "schema": {
                            "type": "file"
                        }
                    }
                }
            }
        }
    },
    "definitions": {
        "EntityNameForCreateDto": {
            "allOf": [
                {
                    "$ref": "#/definitions/EntityNameForManipulationDto"
                },
                {
                    "type": "object"
                }
            ]
        },
        "EntityNameForManipulationDto": {
            "type": "object",
            "properties": {
                "field1": {
                    "title": "field1",
                    "type": "string",
                    "maxLength": 250,
                    "minLength": 0
                },
                "field2": {
                    "title": "field2",
                    "type": "string",
                    "maxLength": 250,
                    "minLength": 0
                }
            }
        }
    }
}

And when I generate the client EntityNameForCreateDto doesn't have any fields due to no inheritance or copying of fields being applied
@JsonSerializable(explicitToJson: true)
class EntityNameForCreateDto {
  EntityNameForCreateDto();

  factory EntityNameForCreateDto.fromJson(Map<String, dynamic> json) =>
      _$EntityNameForCreateDtoFromJson(json);

  static const fromJsonFactory = _$EntityNameForCreateDtoFromJson;
  static const toJsonFactory = _$EntityNameForCreateDtoToJson;
  Map<String, dynamic> toJson() => _$EntityNameForCreateDtoToJson(this);

  @override
  int get hashCode => runtimeType.hashCode;
}

@JsonSerializable(explicitToJson: true)
class EntityNameForManipulationDto {
  EntityNameForManipulationDto({
    this.field1,
    this.field2,
  });

  factory EntityNameForManipulationDto.fromJson(Map<String, dynamic> json) =>
      _$EntityNameForManipulationDtoFromJson(json);

  @JsonKey(name: 'field1', includeIfNull: false)
  final String? field1;
  @JsonKey(name: 'field2', includeIfNull: false)
  final String? field2;
  static const fromJsonFactory = _$EntityNameForManipulationDtoFromJson;
  static const toJsonFactory = _$EntityNameForManipulationDtoToJson;
  Map<String, dynamic> toJson() => _$EntityNameForManipulationDtoToJson(this);

  @override
  bool operator ==(dynamic other) {
    return identical(this, other) ||
        (other is EntityNameForManipulationDto &&
            (identical(other.field1, field1) ||
                const DeepCollectionEquality().equals(other.field1, field1)) &&
            (identical(other.field2, field2) ||
                const DeepCollectionEquality().equals(other.field2, field2)));
  }

  @override
  int get hashCode =>
      const DeepCollectionEquality().hash(field1) ^
      const DeepCollectionEquality().hash(field2) ^
      runtimeType.hashCode;
}

extension $EntityNameForManipulationDtoExtension
    on EntityNameForManipulationDto {
  EntityNameForManipulationDto copyWith({String? field1, String? field2}) {
    return EntityNameForManipulationDto(
        field1: field1 ?? this.field1, field2: field2 ?? this.field2);
  }
}

Also could you please clarify one thing for me. Is copying of fields that you suggested here and in the changelog, done by the library, or is it something that should be done on the Api/Swagger spec? Because that's something I cannot afford due to having hundreds of models in an existing Api.

@boriszv  OK I reopened issue and will check it


Generated mocks.json file doesn't show data

The mocks.js file I generated according to the content of the readme file has no requests data.
I have tried many methods to solve this problem.
If anyone knows the solution, please help to reply. Thank you very much for your help!
The following is my specific operation:

After downloading this project locally, execute the command:
npm install playwright-request-mocker -D
npm install @playwright/test

After recording the js script with playwright, directly change the file name to the suffix of spec.js

Add content to the file, as shown below:
//file: test02Mock.spec.js
const { test, expect } = require('@playwright/test'); const { useNetworkRecordMocks } = require('playwright-request-mocker'); test('test02Mock', async ({ page }) => { const requests = await useNetworkRecordMocks(page, {  identifier: 'test02Mock',  recordRoute: "my_login_test_url" }); await page.goto(my_login_test_url);

The rest of this file is the page operation of the web UI, such as adding data after logging in. eg:
  await page.click('[placeholder="account"]'); await page.fill('[placeholder="account"]', '2000M000'); await page.click('[placeholder="passwd"]'); await page.fill('[placeholder="passwd"]', '123456'); await page.click('button:has-text("login")')

run command in terminal :
set PWDEBUG=console
npx playwright test ./src/test02Mock.spec.js

At this point, the playwright inspector window will open, manually execute it according to the operation of spec.js, and then click resume in the inspector window.

The .mocks.json file is displayed in the same directory for a period of time, and only one line is displayed when opening this file:
{"requests":[]}

I manipulate the contents of the spec.js file in the browser, and the F12 developer mode displays multiple pieces of HAR data normally.

So, I don't know why the auto-generated mocks.js has no data. If anyone who knows can let me know, I would be very grateful!

@ysq123qwe i'm also experiencing the same issue. Which playwright/test version are you using?

Hi，I am using version 1.21. And i didn't disable the service worker.
------------------ 原始邮件 ------------------
发件人:                                                                                "kousenlsn/playwright-request-mocker"                                                                                @.>;
发送时间: 2022年5月16日(星期一) 晚上10:03
@.>;
<EMAIL_ADDRESS>主题: Re: [kousenlsn/playwright-request-mocker] Generated mocks.json file doesn't show data (Issue #13)
@ysq123qwe i'm also experiencing the same issue. Which playwright/test version are you using?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


[BUG] make install command under windows don't build the \target\release\spk.exe binary

Describe the bug
make install command under windows don't build the \target\release\spk.exe binary and generate error by trying to copy and not finding the binary.
To Reproduce
Steps to reproduce the behavior:

Run make install under windows
Get the error copy /B C:\Users\p.llerena\workspace\github.com\doubleailes\spk\\target\release\spk.exe "C:\PROGRA~1\spk\bin" /Y the specify file is not found.

Expected behavior
make install should build --release spk

This PR https://github.com/spkenv/spk/pull/1113 fix the issue

https://github.com/spkenv/spk/pull/1113


btadv: sensor location & accelerometer adverts

Add accelerometer adverts and sensor location to btadv
Closes #2907

You think this is ready now?

I think it'll be ready within the week, waiting on feedback in #2907 to confirm it works well

This is ready now - there's an issue with connecting over BLE when the app's running that I've not found the cause of yet, but I'll move that to a separate PR.

after turn on the sensors, ble connections from bangle.js web ide/apploader will always fail while the app is on, connections from phone has no problem.

Could it be that changing the ble advert prevents the apploader & ide from connecting, as they're looking for a uart service? Although that should still be advertised.
I'll try to capture uncaught exceptions, save them to storage and do a post mortem on the app

Thanks! I'll merge then

Could it be that changing the ble advert prevents the apploader & ide from connecting, as they're looking for a uart service?

Yes... the apploader/IDE looks for UART service, or a name it recognises like Bangle.js - if both of those get removed then it won't see it... You could check with NRF Connect and see what it sees?
Maybe just add a on('kill',...) handler that resets advertising when the app exits?

Carrying on from this, I've attempted to capture what's happening when btadv prevents BLE connections from being established to it. I suspect it's trying to cram too much into the BLE advertising packets which is resulting in garbage.
Issue
Summarised in #2907:

after turn on the sensors, ble connections from bangle.js web ide/apploader will always fail while the app is on, connections from phone has no problem

Exception Logging
No error logs appear except those mentioned in #2907, about not being able to update services until restart:
Error: Can't update services until BLE restart

Stashing this code here for non-IDE collection of logs
{
  let f = require("Storage").open("uncaught.log", "a");

  process.on(
    'uncaughtException',
    e => {
      console.log("uncaught: ", e);
      f.write((new Date).toISOString() + " " + e + "\n");
    });
}

External Diganosis
I notice my watch advertises a HID service (0x1812), which isn't set in btadv or any app, for that matter:

Which leads me to the above mentioned idea that I've put the NRF chip into a broken state, seeing as a hard reset is required to be able to connect to my watch. I'll see about altering btadv / reverting some more recent changes to see if I can undo this and restore its advertising functionality.


"tested" data for Washoe County, NV is impossible

Location, date, and short issue description
Washoe County, NV (FIPS 32031) has impossible values for "tested" for many (all?) dates.  In many cases, the delta in "cases" from day-to-day is greater than the delta in "tested", which is impossible.  Since the county official website doesn't seem to report "total tests" or similar, I suspect the "tested" data is just bogus.
I don't know how you normally deal with this, but perhaps it's worth either contacting the county or blacklisting the "tested" data (and maybe re-check it every week or two?).  Or if this is outside your scope, we can just continue to blacklist it on our side.
Source
The invalid "tested" entries seem to come from the underlying arcgis dataset that powers the county website dashboard, but the county website dashboard doesn't actually show "total tested" anywhere, so I think the data is probably garbage.
File
https://coronadatascraper.com/timeseries-byLocation.json (but will show up in any of the timeseries datasets for Washoe)
Issue details
The delta in "tested" from day-to-day is impossible, since it's often less than the delta in "cases".  E.g.:

"2020-05-03": {
"cases": 977,
"tested": 903,
...
},
"2020-05-04": {
"cases": 988,
"tested": 906,
...
},

There were 11 new confirmed cases, but only 3 tests? This is one example, but all of the "tested" data is likely wrong (see screenshot below).
Snippet/screenshot
At https://covidactnow.org/ we are tracking "test positive rate" and for Washoe County, the current data looks like this:

Everything over 100% is impossible, and the rest is pretty suspect too.

@jzohrab @appastair can y'all look into this? Thanks!

I see what you mean, @mikelehen - the data is definitely off...although consistently being updated. :woman_shrugging:
{
"OBJECTID": 144,
...
"reportdt":<PHONE_NUMBER>000,
"confirmed": 1100,
"recovered": 529,
"deaths": 39,
"active": 532,
"tested": 1035,
...
"reportedtPT": "5/11/2020 1:37 PM",
...
}

The latest daily press release reports 14,029 tests.
I've attempted to contact them via social media to see if they'll add the hidden, erroneous field to the dashboard. If that fails, I can attempt to contact them through another channel.
In the meantime, I'm not sure what the done thing is. Wait for correction or remove the field, @lazd ?

I'm removing the tested field, it's bogus.

Removed in PR https://github.com/covidatlas/coronadatascraper/pull/1045.
Thanks @mikelehen !


Problem running Alamofire on device running iOS 7

Has anyone been able to run Alamofire.swift on a device running iOS 7?
For me, it's crashing with:
dyld: Library not loaded: @rpath/Alamofire.framework/Alamofire
Referenced from: /var/mobile/Applications/A59259D8-0AF7-43AD-AC21-2A0530B26B09/Stayful.app/Stayful
Reason: image not found

Add a Copy Files Phase in your applications Build Settings.
Set the Destination to "Frameworks" and add the .framework.


Add a "Flow" diagram

A picture to try and tell the flow through appsody to production. This is intended for an "overview" level, so naturally has some simplifications.
Fixes #273

@chilanti Agreed....I'll try and reflect this....thanks for the review

Thanks for the review & merge.


fix: Unnamed NFTs on Moonriver

Thank you for your contribution to the KodaDot NFT gallery.
👇 _ Let's make a quick check before the contribution.
PR type

[x] Bugfix
[ ] Feature
[ ] Refactoring

What's new?

[x] PR closes #4023
[ ] Requires deployment <rubick/magick_issue>
[ ] <brief_description_of_what_I've_added>

Before submitting Pull Request, please make sure:

[x] My contribution builds clean without any errors or warnings
[x] I've merged recent default branch -- main and I've no conflicts
[x] I've tried to respect high code quality standards
[x] I've didn't break any original functionality
[x] I've posted a screenshot of demonstrated change in this PR

Optional

[ ] I've tested it at </rmrk/collection/26902bc2f7c20c546a-1FVG7>
[ ] I've tested PR on mobile and everything seems works
[ ] I found edge cases
[ ] I've written some unit tests 🧪

Had issue bounty label?

[x] Fill up your KSM address: Payout

Community participation

[x] Are you at KodaDot Discord?

Screenshot

[x] My fix has changed something on UI; a screenshot is best to understand changes for others.

The property of Unnamed NFT is like this:

pay 50 usd

😍 Perfect, I’ve sent the payout
💵 $50 @ 44.92 USD/KSM ~ 1.113 $KSM
🧗 Caiv9TbPz68q5dC8EcHu5xKYPRnremimGzqmEejDFNpWWLG
🔗 0x3b50ed493ecb3bfb3a201389e54adf2db931fa9cb2f5cd10beac2f1ef9e80f48
🪅 Let’s grab another issue and get rewarded!
🪄 github.com/kodadot/nft-gallery/issues


No boilerplate copyright notice

As in Copyright [yyyy] [name of copyright owner], which, as far as I know, is required to apply the Apache 2.0 license.

I don't think that it is strictly necessary, see

http://www.apache.org/dev/apply-license.html#copy-per-file
https://softwareengineering.stackexchange.com/a/270098

I have added a License section in the README, though: https://github.com/sharkdp/diskus#license


Hang on Disabled Windows Notifications

Expected Behavior
Disabling Windows 10 notifications should not hang node-notifier.
Actual Behavior
Running node-notifier on Windows 10 clients with notifications results in a hang.
Repro
The simplest repro is to disable Windows 10 notifications as shown below and run npm run example in the repo.
I'm trying a scenario like:
try {
  const notifier = require('node-notifier');
  notifier.notify({
    title: 'Title',
    message: 'message',
    wait: true
  });
} catch {
  // Notifications may be disabled in OS, causing an error.
  // node-notifier may not installed.
  // Silently fail.
}

I'd just simply like to catch exceptions and silently continue.
However, if notifications are disabled on Windows 10:

This example will hang forever on the command line:
λ node not.js
Notifications are disabled
Reason: DisabledForUser Please make sure that the app id is set correctly.
Command Line: G:\git\ooui-dp\node_modules\node-notifier\vendor\snoreToast\snoretoast-x64.exe -pipeName \\.\pipe\notifierPipe-f80dfe8f-ff7d-42ce-becd-0a2e340a9d59 -m message -t Title

I tried searching the docs and existing issues for a way to "detect" whether notifications are disabled, but I haven't found anything. I'd also like to suppress error output like this if possible. In any case, it doesn't seem like the call to node-notifier should hang like this.
Tools
node-notifier: 7.0.1
node: 12.18.0
OS: Windows 10 Pro 1909

You can't catch any exception due to the right execution of node-notifier. Seems the developer have choosen to print an "error" message about disabled notifications system . Also, for sure, I'm right with you that is an useless info.

I've done a little more investigation. It appears in this case snoretoast never writes to pipeName arg, which means the named pipe never has a data event and therefore never has end called. This means the named pipe server keeps running which is what's causing the hang.
I have a local fix that passes back the server instance to toaster.js so that it can make sure the server is closed. I'm not sure if it's the best fix but I can make a PR.

Some more findings: The error output is generated in snoretoast but bypasses standard stdio constructs entirely. It doesn't even come through on stdio via Node. It appears the reason is due to snoretoast's use of AttachConsole, which prevents this error output from feeding back into node-notifier.

I've entered a snoretoast bug as well.

I have a PR that doesn't totally fix the issue (due to snoretoast's behavior) but at least resolves the hanging issue. I think it's important to fix with some kind of change at least temporarily to prevent a permanent hang in certain use cases.


Backport 'HBASE-27646 Should not use pread when prefetching in HFilePreadReader…

… (#5063)'

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 38s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
4m  2s
branch-2.5 passed

+1 :green_heart:
compile
2m 29s
branch-2.5 passed

+1 :green_heart:
checkstyle
0m 34s
branch-2.5 passed

+1 :green_heart:
spotless
0m 44s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 36s
branch-2.5 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 38s
the patch passed

+1 :green_heart:
compile
2m 27s
the patch passed

+1 :green_heart:
javac
2m 27s
the patch passed

+1 :green_heart:
checkstyle
0m 34s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
18m 39s
Patch does not cause any errors with Hadoop 2.10.2 or 3.2.4 3.3.4.

+1 :green_heart:
spotless
0m 42s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
1m 35s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

39m 29s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5124

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux a6488dfca35f 5.4.0-1093-aws #102~18.04.2-Ubuntu SMP Wed Dec 7 00:31:59 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / c0760d39dc

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 10s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
4m 22s
branch-2.5 passed

+1 :green_heart:
compile
0m 57s
branch-2.5 passed

+1 :green_heart:
shadedjars
5m 35s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 30s
branch-2.5 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 54s
the patch passed

+1 :green_heart:
compile
0m 55s
the patch passed

+1 :green_heart:
javac
0m 55s
the patch passed

+1 :green_heart:
shadedjars
5m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 26s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
233m  7s
hbase-server in the patch passed.

260m 18s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5124

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux cfffe424fadd 5.4.0-144-generic #161-Ubuntu SMP Fri Feb 3 14:49:04 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / c0760d39dc

Default Java
Eclipse Adoptium-11.0.17+8

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/testReport/

Max. process+thread count
2797 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2.5 Compile Tests _

+1 :green_heart:
mvninstall
4m  0s
branch-2.5 passed

+1 :green_heart:
compile
1m  2s
branch-2.5 passed

+1 :green_heart:
shadedjars
6m  8s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 44s
branch-2.5 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 33s
the patch passed

+1 :green_heart:
compile
1m 11s
the patch passed

+1 :green_heart:
javac
1m 11s
the patch passed

+1 :green_heart:
shadedjars
6m 24s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 38s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
241m 29s
hbase-server in the patch passed.

271m 30s

Subsystem
Report/Notes

Docker
ClientAPI=1.42 ServerAPI=1.42 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5124

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 56b05dbdaadc 5.4.0-137-generic #154-Ubuntu SMP Thu Jan 5 17:03:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2.5 / c0760d39dc

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/testReport/

Max. process+thread count
2431 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5124/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Avoid using Assert.DoesNotContain due to performance problems

See xunit/xunit#1543

Codecov Report

Merging #427 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master     #427   +/-   ##
=======================================
  Coverage   17.54%   17.54%
=======================================
  Files         213      213
  Lines      122828   122828
  Branches    11833    11833
=======================================
  Hits        21546    21546
  Misses     100423   100423
  Partials      859      859

Flag
Coverage Δ

#2017
17.54% <100%> (ø)
:arrow_up:

#Debug
17.54% <100%> (ø)
:arrow_up:

Impacted Files
Coverage Δ

src/TraceEvent/TraceEvent.Tests/DispatchTests.cs
40.17% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


New version: XK72.Charles version 4.6.6

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,
This manifest adds Apps and Features entries that aren't present in previous PR versions. These entries should be added to the previous versions, or removed from this version.
(Automated response - build 876.)


Fixes .test() !!!

!!

Cleaned up the path for model vs not vs best weights
load state directly instead of trying to re-init model.

Next PRs will be about refactoring even more once 0.8.5 is live

Hello @williamFalcon! Thanks for updating this PR.

In the file pytorch_lightning/overrides/data_parallel.py:

Line 66:19: E702 multiple statements on one line (semicolon)


Update post-editor.tsx

Add links to "Supports Markdown" that lead to https://remarkjs.github.io/react-markdown/.
This helps the user understand what kind of markdown is supported, consider making a custom page for this later down the road.
This is to help users to figure out what formatting is supported.

I am unable to post changes to this pr, will close it, redo it and open a new pr.


Use standard cache folder instead of __dirname for venv and mitmproxy scripts

Putting the venv and mitmproxy scripts into our node_modules folder (relative to __dirname) is unfortunately quite fragile. We've already had a fair amount of breakage due to this and while reviewing #17, I once again noticed this breaking.
For some bizarre reason, __dirname is /home/benni/coding/JS/tweasel/cyanoacrylate/dist/src, even though that folder doesn't exist (the built index.js is in dist, deleting that and the Parcel cache didn't help).
I propose that we instead put this stuff into the system cache folder using global-cache-dir. That feels more idiomatic, anyway.
Additional benefit: If we want to distribute a single executable at some point, there won't be a node_modules folder that we can write to, anyway.
What do you think, @zner0L?

One thing to consider: It's plausible that someone might delete this cache directory without re-running the postinstall script, which would break the module. This could already happen with the current implementation, but I feel like it's more likely if we use a cache directory.
Not too hard to work, around though: We can just check whether the necessary files exist and run the postinstall script otherwise.

Yeah, that seems reasonable to me. I am just not sure where we would put this check. In startAnalysis?

I am just not sure where we would put this check. In startAnalysis?

Yeah, that seems reasonable to me.

Actually, I went with startTrafficCollection(), since startAnalysis() isn't async and we only need those dependencies for the traffic collection.

This change has fixed the CLI for m.e

For some reason, Parcel breaks os.platform (it removes the import and replaces it with an undefined identifier, very helpful). But process.platform works.

Windows just really wants to annoy you, doesn't it. Of course, PATH entries are delimted by ; instead of :. -.-


feat(docs-site): cookbook onboard - AI assistant (#17862)

Resolves https://github.com/taikoxyz/taiko-mono/issues/17862

ref. the issue for reason


Fog probability

Add Fog Probability
This is a prerequisite for local current conditions
I did simplify calculated fog probability to not include temperature trend data since that is not readily available but it does not have a huge effect not utilizing temperature trends.
Snow Probability will be next and then should have all requirement for current conditions.
I think I got all the appropriate files updated, but I have a nagging feeling that I forgot something.

Thanks Glenn,
I will merge and test it out.

Hi Glenn,
I had to make some modifications, but there is now a dev container which creates this sensor. Can you have a run down on this, and see if this produces the right sensor and the right value?

When done, please also make an update to the README file.
Is this btw the probability for fog for the next 24 hours or what is the timeframe?

Yes, temperature is air temperature (I'm not sure how I missed that since I copied and pasted from my running script, then converted from java-script to pyhton).
This is current conditions for fog; although for my personal testing when at 27% fog probability there will be fog on my way to work usually within a few miles of leaving my home.
I will update the readme and test the dev build (I will be about 10 hours before I can do this).  Hopefully, I can get the Snow Probability to you before the end of your weekend.

I updated ReadMe and will do that PR once I update to the newest DEV build and verify everything.

@briis It is off by a lot when conditions are right.  I need to see what the issue is.  Both calculations were zero (0) until the same time and then started diverging.

OK, I let you dig around. 😁


Save as: incorrect file extension for .R files

This has been referenced here: https://github.com/Microsoft/vscode/issues/28425 and https://github.com/Microsoft/vscode/issues/62054#issuecomment-434149025
Steps to Reproduce:
Try to "save as" on a file with file extension ".r"
Do you want to fix by self? (I hope your help!)
Yes
(If yes,) what kind of help do you want? (e.g. Which file should I fix, Survey (related documents)
Should be easy according to https://github.com/Microsoft/vscode/issues/62054#issuecomment-434149025

Feel free to reopen if you still encounter this problem.


Allow access to variant platform prop for codec2

Changes include:

Add vendor_media_platform_prop to aafd.
Provide get_prop avvess to codec2_hal_default

Tests Done:

Ran CtsMediaEncoderTestCases android.media.encoder.cts.VideoEncoderTest

Tracked-On: OAM-121126

one of the dependent pr cannot be Rebase and Merged; please fix the issue and commit your changes once again to run EB.


Introduce EksekResult class

This introduces a new class EksekResult that would handle the output as suggested in #4. Most of the functionality works exactly the same, but there's a run method now for Eksek which returns an EksekResult in the end.
This exposes the following methods (which work similar to the previous ones):

stdin
stdout
exit_code
success?
success!

⚠ Heads up:

success! will throw the EksekError as before in case not successful
success! will return self (the EksekResult) in case everything is fine, thus making it possible to combine this with all the other methods
Unfortunately a combined stdin+stderr result would not work anymore like this (any ideas about that??)

Todo

[x] Remove getters for stdout and stderr from ExecResult and add to attr_reader
[x] Rename run method (maybe keep an alias?)
[x] Remove initialisation with nil of stdout and stderr in Eksekuter
[x] Update Readme

Oeh, let's not forget to update README.md after!

@jleeothon I added some changes regarding your feedback. When testing, however, I saw that passing the hash as ":env" results in an exception coming from Open3.popen3:
     Failure/Error: @stdin, @stdout, @stderr, @wait_thr = Open3.popen3(cmd, opts)

     ArgumentError:
       wrong exec option symbol: env
     # ./lib/eksekuter.rb:50:in `spawn_process'
     # ./lib/eksekuter.rb:12:in `run'
     # ./lib/eksek.rb:25:in `method_missing'
     # ./spec/eksek_spec.rb:73:in `block (2 levels) in <top (required)>'

This happens in master as well as in this branch... Something is missing here I guess. 🤔


Don't show game tab when not playing an mp seed.

Game tab should be hidden if the current seed has webConn: false in it

Done


Fast but no exceptions

Add a compiler flag ENTITAS_FAST_BUT_NO_EXCEPTIONS
Potential code to exclude:
Checks on entity

Add
Remove
Replace
Get
Retain
Release

Checks on Pool

onEntityReleased

and maybe more...

See #61


[Bug] Text on some Buttons is cut off

Describe the bug
This is a small visual issue.
If you want to close Notepads, but there are still one or more unsaved files,
it asks you whether to "Save and quit" or "Discard and quit".
The text / label on these buttons is cut off on the right side.
How much of is cut off seems to depend
on the size of the window (see screenshots below).
This doesn't happen for the buttons in the specific dialogue
for saving changes when trying to close the files individually.
In the screenshots below you can see the german version,
because I couldn't find Language Preferences in Settings > Advanced.
So this may only happen in the german version.
To Reproduce
See description above.

Screenshots

Desktop:

OS: Windows 10 Pro 1909 18363.836, Windows 10 Home 2004 19041.264
Version: v<IP_ADDRESS>

Additional context
This is independent from Disyplay Scaling Settings in Windows (100%, 125%, 200%, ...).

What language is that?

It's the german version.
Since I couldn't find Language Preferences in Settings > Advanced, I couldn't check if it also happens in other languages.
So this may only happen in the german version.

@WalterWolf49 Can you help take a look?

The word "und" used to be the "&" character for that exact reason. @milotype changed that with #467

This is how it looks in Swiss German (de-CH) and how it looked prior to #467.

"und" instead of "&" does look better imho.
@JasonStein can the dialog be bigger to fit the additional space needed?

This is how it looks in Swiss German (de-CH) and how it looked prior to #467.

"und" instead of "&" does look better imho.
@JasonStein can the dialog be bigger to fit the additional space needed?

I don't think we can make it bigger and I don't feel like we should do that. "&" seems more reasonable to me. Feel free to send out PR. I can do it if you don't have time. Just let me know.

I'm no programmer, so I don't have a solution, but I think it is important to adapt the code, in order to make it work for other languages as well.
A defined number of characters for strings is bad for any translation!!
In this case for German, an alternative could be, to at least use the "+" character.
Speichern + beenden
The "&" character is not a common German "expression" for the word "and" – except, for instance in names of legal agencies and advertising agencies, who like to pretend being international.

I'm no programmer, so I don't have a solution, but I think it is important to adapt the code, in order to make it work for other languages as well.
A defined number of characters for strings is bad for any translation!!
In this case for German, an alternative could be, to at least use the "+" character.
Speichern + beenden
The "&" character is not a common German "expression" for the word "and" – except, for instance in names of legal agencies and advertising agencies, who like to pretend being international.

This is a design problem not a coding problem. Having a long width button makes no sense from design perspective. That's why button has an predefined max width. This is done by the MSFT design team not me which further proves my point here.

I'm no programmer, so I don't have a solution, but I think it is important to adapt the code, in order to make it work for other languages as well.
A defined number of characters for strings is bad for any translation!!
In this case for German, an alternative could be, to at least use the "+" character.
Speichern + beenden
The "&" character is not a common German "expression" for the word "and" – except, for instance in names of legal agencies and advertising agencies, who like to pretend being international.

This is a design problem not a coding problem. Having a long width button makes no sense from design perspective. That's why button has an predefined max width. This is done by the MSFT design team not me which further proves my point here. If you need to explain more, you should put strings into descriptions. For buttons, they should be short in length and more straight forward.

I'm no programmer, so I don't have a solution, but I think it is important to adapt the code, in order to make it work for other languages as well.
A defined number of characters for strings is bad for any translation!!
In this case for German, an alternative could be, to at least use the "+" character.
Speichern + beenden
The "&" character is not a common German "expression" for the word "and" – except, for instance in names of legal agencies and advertising agencies, who like to pretend being international.

Take a look at how MSFT is handling the translation here in Paint 3D:

"Speichern" is good enough here.

Take a look at how MSFT is handling the translation here in Paint 3D:

"Speichern" is good enough here. Since this dialog will only pop up when user click on "Exit"
button, so there is no need to tell user "beenden" here at all.

I don't know if my opinion is of any help here, but
I agree with @JasonStein and I would also very much prefer the simple version of just "Speichern" and "Nicht speichern", as commonly seen in other german applications instead of using "und"s, "&"s or even "+"s.

As mentioned, I'm not a programmer. I'm a translator.
Design or code, it's al the same to me ...
The translatable string specifies: <value>Save All & Exit</value>, just as the other one <value>Discard & Exit</value> (Actually the German would have to be even longer in the first case: Alles speichern + beenden)
From a translators point of view, please don't expect translators to shorten translations to a certain number of characters, nor to check the display of any string in the application, especially, as there is no comment/information for translators about the strings length!!
So in this case, maybe the source string needs to be changed!?
Whatever you do, please do keep in mind, that translations do not and can not follow (generally english) string lengths (and IMO, there is no specific reason for it).
Example:
en: Backup (noun) – 6 characters
hr: Sigurnosna kopija – 17 charaters
I'm just trying to help ...

As mentioned, I'm not a programmer. I'm a translator.
Design or code, it's al the same to me ...
The translatable string specifies: <value>Save All & Exit</value>, just as the other one <value>Discard & Exit</value> (Actually the German would have to be even longer in the first case: Alles speichern + beenden)
From a translators point of view, please don't expect translators to shorten translations to a certain number of characters, nor to check the display of any string in the application, especially, as there is no comment/information for translators about the strings length!!
So in this case, maybe the source string needs to be changed!?
Whatever you do, please do keep in mind, that translations do not and can not follow (generally english) string lengths (and IMO, there is no specific reason for it).
Example:
en: Backup (noun) – 6 characters
hr: Sigurnosna kopija – 17 charaters
I'm just trying to help ...

This is exactly why I call it out since I started this project. I am expecting translators to do the research and validation here not me. Otherwise I will just use Google Translate here.

Sorry, I haven't read that. No offense, please do proceed as you like and excuse me if I generated a problem ...
The issue is actually a general problem ... expecting, that translators – who are often/mostly not programmers – should check/compare apps on systems they might not even use.

As mentioned, I'm not a programmer. I'm a translator.
Design or code, it's al the same to me ...
The translatable string specifies: <value>Save All & Exit</value>, just as the other one <value>Discard & Exit</value> (Actually the German would have to be even longer in the first case: Alles speichern + beenden)
From a translators point of view, please don't expect translators to shorten translations to a certain number of characters, nor to check the display of any string in the application, especially, as there is no comment/information for translators about the strings length!!
So in this case, maybe the source string needs to be changed!?
Whatever you do, please do keep in mind, that translations do not and can not follow (generally english) string lengths (and IMO, there is no specific reason for it).
Example:
en: Backup (noun) – 6 characters
hr: Sigurnosna kopija – 17 charaters
I'm just trying to help ...

I think we can all agree, that translations for user interfaces can't be done literally most of the times. A lot of times the meaning of the original needs to be slightly altered to fit some of the restrictions that are out of our control. By doing that, translations can end up more or less descriptive than the original, depending on word lengths in the specific language.
Since other people may have run into the same problem at some point, taking inspiration from their solutions can help. I think the existing solution @JasonStein proposed ("Speichern", "Nicht speichern") is the one most german users will immediately recognize and be familiar with, even if the original english strings may have a slightly different meaning.
"Speichern" and "Nicht speichern" are by the way already the same pair of german strings, that are used if one wants to close an individual unsaved file, but not closing the application at the same time. Choosing those as a fix would even help to unify the user experience.
I am very grateful to all of the author(s) and contributors of this project and I'm happy to also help improving the experience for everyone. Thank you!

As mentioned before, it's not an issue about literal translation. Most (good) translators translate semantically and where needed, OS system specifically anyways. Unfortunately, programmers of apps often expect, that translators should/need/have to think about/abide by diverse lenghts of strings ...
As a non-programmer, I nevertheless believe it should be possible, to change/addapt the font size of a (translated) string, when the translated string doesn't fit the length of a source string. In the case that there are multiple buttons in a dialog, the button with the longest string would be decisive for the font size, no matter what the "defined" stylesheet for buttons may be ... but I don't know how/if this is possible to realize ...
IMO, this is a general translation/localisation issue. It's not a Notepads specific issue ...

"Speichern" and "Nicht speichern" make sense, so I created a PR for those translations.

Why don't you just make width of buttons flexible to text content?

Why don't you just make width of buttons flexible to text content?
And maybe also check how german version of regular Notepad looks.

I can but read above replies first.
It will become unnecessarily bulky if I blinded increase the maxWidth and by the way, the default max width of button in ContentDialog is predefined by the MSFT design team which I am following.


[Feature]: properly mark breaking changes in release notes

Use Cases or Problem Statement
Latest release 0.94.0 missed adding a migration note.
Category
category:other
Object type(s)
No response
Proposal
While it's great to have a link to the migration notes of the specific version bump, I suspect it's a manual process, and thus error prone.
I'd like to suggest

to always add a static link to https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/MIGRATION_GUIDE.md , no matter if the version bump has or not breaking changes.
to VISUALLY mark which points in the "What's new" section is breaking either via an emoji or a separate section. If you're adhering fully to convention commits, then simply rename the PR from "feat" to "feat!" and it will be marked accordingly

How much impact is this issue causing?
Medium
Additional Information
No response
Would you like to implement a fix?

[ ] Yeah, I'll take it 😎

This broken release process is getting very old very fast.

Hey @sramshaw-yolabs. Thanks for reaching out to us.
What do you mean by the broken release process?

@sfc-gh-asawicki  Broken may have been a strong word.  The Disclaimer describes how this project is being treated as pre 1.0.0 experimental.  But the reality is it has been the provider for over 5 years now.  While I do understand Snowflake officially took it over at some point (can't recall when, but at least 1-2 years ago?) it is extremely cumbersome on your users when bug fixes, new features, and breaking changes all merged into one release path.
Users want to take advantage of bug fixes and new features, but it is extremely impactful on us when we have repeated sudden work injections to address numerous breaking changes with the only other choice being to pin to a version that may have bugs or lack features we want to use.
As you can see from the terraform registry this provider is extremely well used with over 1 million downloads this month alone.
Maybe you are close enough to 1.0 now that this is all moot point, but I've honestly lost count at how many times I've had to inject work to address the breaking changes that imo should have been forked to a pre-1.0.0 release path months or even years ago.
If at all possible I would request for you to consider the impact on users and slate all of the remaining breaking changes for the 1.0.0 release.  This would let people plan for the work to cutover to 1.0.0 as opposed to being surprised every week as to whether there are breaking changes that need to be addressed or not.
Provider Downloads All versions
Downloads this week
288,962
Downloads this month
1.0M
Downloads this year
7.1M
Downloads over all time
17.7M

Thank you for reaching out.

Hey @sramshaw-yolabs. Thanks for the elaborate answer.
After reaching V1, we will definitely change the release process to strictly adhere to semantic versioning (breaking changes will require a major bump).
We have considered the alternatives with the 0.x versions and concluded that we want to avoid a big-bang V1 version. We will change the provider step-by-step with a sequence of breaking changes. It still allows our users to make a big-bang migration themselves by staying on the current version until V1 is released.
It's completely fine to pin to a current version and migrate afterward. I would even say that automatically bumping minors in the experimental (0.x projects) is risky and should be avoided because they can always contain breaking changes.
You are right that pinning to a version limits the use of the newer features/bug fixes. Currently, almost all are added in the context of the redesigned resources, so they still require migrations. It is also possible to set up a separate deployment with the newer provider version to manage only the newly added features.
For the 1M downloads, we have received only three complaints (including yours) about the breaking changes in the 0.x version. We can still reconsider our options. I'll talk about this with the whole team, and we will get back to you. Until then, please pin the version to avoid the biweekly/triweekly migrations.


Add links for PPP

Codecov Report

Merging #365 (890b36d) into master (f37e388) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff            @@
##             master     #365   +/-   ##
=========================================
  Coverage     48.54%   48.54%
  Complexity      525      525
=========================================
  Files           114      114
  Lines          2758     2758
  Branches        297      297
=========================================
  Hits           1339     1339
  Misses         1333     1333
  Partials         86       86

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


#487176 added robots.txt implementation

Added robots.txt implementation, services for sitercore-jss/sitecore-jss-nextjs, sample for create-sitecore-jss

Description / Motivation

Testing Details

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

@matkovskyi You'll need to add site.js and site.d.ts files at the source root for the site submodule to work (causing the build errors). See layout.js and layout.d.ts for example.

@matkovskyi You'll need to add site.js and site.d.ts files at the source root for the site submodule to work (causing the build errors). See layout.js and layout.d.ts for example.

@matkovskyi One more thing required: You'll need to add site.d.ts to the tsconfig.json exclude.
FYI, you can check the build locally for individual packages or entire monorepo using yarn build.


Feature/rad 275 update job status

I think it would be also good to update docs (at least describe job statuses and transition between them).


Improve JetBrains.gitignore styling

Reasons for making this change:
Improving style. This PR does not change any ignore rules.

@akelsch thanks!


Setting and Referer

All pleasant time of day! :)
I have two problems which I cannot solve, I decided to ask here.

How can I set the referer? ( Specify the URL from which the browser allegedly came )
Specify titles for User-Agent.
And so I needed to install in the browser these two values do not tell me how to do it?
In their tests, I use here such code:
<?php namespace Facebook\WebDriver; use Facebook\WebDriver\Remote\DesiredCapabilities; use Facebook\WebDriver\Remote\RemoteWebDriver; require_once('vendor/autoload.php'); $host = 'http://localhost:4444/wd/hub'; $capabilities = DesiredCapabilities::chrome(); $driver = RemoteWebDriver::create($host, $capabilities, 50000,50000); $driver->get("http://example.com/"); $driver->manage()->deleteAllCookies(); $driver->manage()->addCookie(array( 'name' => 'cookie_name', 'value' => 'cookie_value', )); $cookies = $driver->manage()->getCookies(); $driver->quit(); ?>

I am pretty sure that Selenium does not allow you to manually do either of those things. The developers of Selenium are pretty adamant about not adding in support for those types of things unfortunately.

Maybe this can be done through the "command line" browser?

You can eg. install some Firefox extension to spoof the referer. But this is not supported directly with Selenium - see discussion in https://github.com/seleniumhq/selenium-google-code-issue-archive/issues/141

Hi all I know that question is already closed but I found a way to install User-Agent to Firefox
$profile->setPreference('general.useragent.override','String User Agent or Firefox php web driver');
In General, write it if someone will need :-)


Flexible header

(This PR is based on the "painter" branch, as I did some zIndex reordering there... so if/when  #122 gets merged, the changed files for this PR will be much cleaner...)
Hey,
When investigating issue #83 I tried to clean up issues people have been having with the header and its positions. Here are the changes I did:

I completely removed the broken fixedHeader='absolute' and replaced it with fixedHeader='flexible', which I believe should be the new default and works like this: https://youtu.be/Z2hEpSW-Yj8
I noticed that we had two different scroll/zoom modes. First if fixedHeader was fixed, the component took control of the scrolling and zooming behaviours. It waited for scroll/touch events and moved the scroll canvas accordingly. If fixedHeader was anything else, we let the user's browser manage the scrolling/zooming/pinching of the scroll canvas and then we reacted to it. While this made the scrolling faster, rendering elements on the canvas after the scroll was much slower (see comparison here around second 21). So I removed the special case for the "non-fixed header", which also caused issues such as #83. Now we always control the scroll and everything should be visually smoother.
I added a demo that shows the flexible scroll mode.

Any comments?
I guess once this and the painter branch get merged, it's time to do a new minor release. :)

@mariusandra hey thank you very much for this PR. It contains now some painter code and the header. I focuse here only on the header.

There is a merge conflict righ now. Easy one. But if you don't feel like here is my branch with it. Just pull from it. signalwerk:flexible-header
for the prop fixedHeader I would not call the new option flexible. I would call it sticky because it's more or less like the [sticky positioning](* https://drafts.csswg.org/css-position/#sticky-pos ) in the new css proposal. Example in Chrome

other than thta all good for me. thank you for your work

Great, thanks for this! I'll merge it later today and release a new minor version. And good idea with renaming flexible to sticky. It does make more sense!


Track taps on the iOS promo space

[cc: @ashfurrow @brainbicycle]
Submitting analytics tracking for the home screen promo space on iOS. Comments within the code.
[Jira]

Designs for the promo space can be found here.

:rocket: PR was released in v1.16.0 :rocket:


Add default option for title label

What type of PR is this? (check all applicable)

[ ] Refactor
[x] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Other: Replace this with a description of the type of this PR

Description
Allows a default value for when the currently selected window is non-existent
Documentation (check all applicable)

[ ] This PR requires changes to the Wiki documentation (describe the changes)
[x] This PR requires changes to the documentation inside the git repo (please add them to the PR).
[ ] Does not require documentation changes

Thanks for the PR! Looks good.


tulipindicators - Financial Technical Analysis Indicators

Website:
https://tulipindicators.org/
GitHub:
https://github.com/TulipCharts/tulipindicators

I created a pull request at: https://github.com/fffaraz/awesome-cpp/pull/532

Thanks!


Handle Events

MILESTONES

Handle events in a React app.
Use React life cycle methods.
Use states and props.

@AmaduKamara noted


fix: onPageSelected not called on ios

Summary
this fixes an issue where onPageSelected callback is not fired on ios when it should
Test Plan
how to reproduce the bug:

render pager with 2 pages
swipe from page index 0 to 1
after running self.previousIndex = currentIndex - 1;, self.previousIndex will contain 0
now call ref.current?.setPage(0). That triggers  the native listener and if(_previousIndex != _currentIndex) from line 177 will be false and so [weakSelf.eventDispatcher sendEvent... is not called and so onPageSelected callback is not fired in JS

the test plan followed the steps and the bug can no longer be reproduced
What's required for testing (prerequisites)?

ios simulator

Compatibility

OS
Implemented

iOS
✅

Android
❌

Checklist

[x] I have tested this on a device and a simulator
[ ] I added the documentation in README.md
[ ] I updated the typed files (TS and Flow)

Thank you @vonovak for your contribution into pager repo 🎉

https://github.com/callstack/react-native-pager-view/releases/tag/v5.1.4


Pagination theme prop

Summary of the changes
Replace appearance prop with theme and monochrome props on pagination to make use of light and dark
theme colours
BREAKING CHANGE: Appearance prop removed from pagination component and replaced with theme and
monochrome
Related issue
Working towards #1214
Checklist
General

[x] Changes to docs package checked and committed.
[x] All acceptance criteria reviewed and met.

Testing

[x] Relevant unit tests and visual regression tests added.
[x] Visual testing against Figma component specification completed.
[x] Playground stories in React Storybook up to date, with any prop changes and additions addressed.
[x] Compare performance of modified components against develop using Performance addon in React Storybook.

Accessibility

[x] Accessibility Insights FastPass performed.
[x] A11y unit test added and yields no issues.
[x] A11y plug-in on Storybook yields no issues.
[x] Manual screen reader testing performed using NVDA and VoiceOver.
[x] Manual keyboard testing for keyboard controls and logical focus order.
[x] Correct roles used and ARIA attributes used correctly where required.
[x] Logical heading structure is maintained, and the HTML elements used for headings can be changed to fit within the wider page structure.

Resize/zoom behaviour

[x] Page can be zoomed to 400% with no loss of content.
[x] Screen magnifier used with no issues.
[x] Text resized to 200% with no loss of content.
[x] Text spacing increased as per the WCAG 1.4.12 success criterion with no loss of content.

System modes

[x] Browser setting 'prefers reduced motion' tested. No animations or motion visible whilst this setting is on.
[x] Windows High Contrast mode tested with no loss of content.
[x] System light and dark mode tested with no loss of content.
[x] Browser support tested (Chrome, Safari, Firefox and Edge).

Testing content extremes

[x] Min/max content examples tested with no loss of content or overflow.
[x] All prop combinations work without issue.
[x] Tested for FOUC (Flash of Unstyled Content) in both SSR (Server-Side Rendering) and SSG (Static Site Generation) settings.
[x] Controlled and uncontrolled input components tested.
[x] Props/slots can be updated after initial render.

@GCHQ-Developer-530 I’m a bit curious, why some PRs are against v3 develop and not develop?
The cherry pick pr only works if you use develop as the commit branch.

Some PRs are only raised against v3 develop as they include breaking changes that we don't want to deploy to the customer yet. The cherry pick PR should take the work that's been done in develop (bug fixes, accessibility errors etc. that we want the customer to have now) and put that in v3 develop so that they're in sync.

Okay, thanks for explaining. The strategy is then to force push the
v3/develop branch when due? I imagine there will be a lot of merge
conflicts, just so you know.
tir. 5. nov. 2024 kl. 15:28 skrev GCHQ-Developer-530 <
@.***>:

@GCHQ-Developer-530 https://github.com/GCHQ-Developer-530 I’m a bit
curious, why some PRs are against v3 develop and not develop?
The cherry pick pr only works if you use develop as the commit branch.
Some PRs are only raised against v3 develop as they include breaking
changes that we don't want to deploy to the customer yet. The cherry pick
PR should take the work that's been done in develop (bug fixes,
accessibility errors etc. that we want the customer to have now) and put
that in v3 develop so that they're in sync.
—
Reply to this email directly, view it on GitHub
https://github.com/mi6/ic-ui-kit/pull/2713#issuecomment-2457325578, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AD7V5NKRXE366VZCCW7JGMDZ7DIZ3AVCNFSM6AAAAABRGRHAEKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINJXGMZDKNJXHA
.
You are receiving this because you commented.Message ID:
@.***>


Error: Input does not meet YAML 1.2 "Core Schema" specification: no-coverage-ran

Error: Input does not meet YAML 1.2 "Core Schema" specification: no-coverage-ran
Support boolean input list: `true | True | TRUE | false | False | FALSE`

Have the same issue

Using getBooleanInput was forcing the field to exist... even with {required: false}.  So just updated my examples to show that as a required field now


Typo: _setVadility

Hi! I've noticed you have a typo in few places in the code: _setVadility instead of _setValidity. @

Thanks, fixed in code and readme


cannot get streetlights to work

Hello,
i'm on debian
No LSB modules are available.
Distributor ID: Debian
Description:    Debian GNU/Linux 9.9 (stretch)
Release:        9.9
Codename:       stretch

$ npm --version
6.11.3

$ node --version
v10.16.0

Tried to follow the tutorial on streetlights
Could you please help in finding what i could possibly have missed, or done wrong?
I'm new to npm dev, so i created a local package as below
$ cat package.json
{
  "name": "streetlights",
  "version": "1.0.0",
  "description": "streetlights AsyncAPI.org tutorial",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "asyncapi-generator": "^0.16.0",
    "asyncapi-node-codegen": "^1.3.2",
    "mqtt": "^3.0.0"
  }
}

having the asyncapi.yaml
$ cat asyncapi.yaml
asyncapi: '2.0.0'
info:
  title: Streetlights API
  version: '1.0.0'
  description: |
    The Smartylighting Streetlights API allows you
    to remotely manage the city lights.
  license:
    name: Apache 2.0
    url: 'https://www.apache.org/licenses/LICENSE-2.0'

servers:
  mosquitto:
    url: localhost:1884
    scheme: mqtt

channels:
  light/measured:
    publish:
      summary: Inform about environmental lighting conditions for a particular streetlight.
      payload:
        type: object
        properties:
          id:
            type: integer
            minimum: 0
            description: Id of the streetlight.
          lumens:
            type: integer
            minimum: 0
            description: Light intensity measured in lumens.
          sentAt:
            type: string
            format: date-time
            description: Date and time when the message was sent.

whether i install packages locally installed or globally with the 'npm -g install' with
ag asyncapi.yaml nodejs -p server=mosquitto

i'm getting this error
$ ag asyncapi.yaml nodejs -p server=mosquitto
Something went wrong:
Error: Invalid AsyncAPI document
    at parse (/usr/lib/node_modules/asyncapi-generator/node_modules/asyncapi-parser/lib/parser.js:93:11)
[
    {
        "dataPath": ".servers['mosquitto']",
        "keyword": "additionalProperties",
        "message": "should NOT have additional properties",
        "params": {
            "additionalProperty": "scheme"
        },
        "schemaPath": "#/additionalProperties"
    },
    {
        "dataPath": ".servers['mosquitto']",
        "keyword": "required",
        "message": "should have required property 'protocol'",
        "params": {
            "missingProperty": "protocol"
        },
        "schemaPath": "#/required"
    },
    {
        "dataPath": ".channels['light/measured'].publish",
        "keyword": "additionalProperties",
        "message": "should NOT have additional properties",
        "params": {
            "additionalProperty": "payload"
        },
        "schemaPath": "#/additionalProperties"
    }
]

finally, i tried to generate the docs both in markdown and html, but this fails too :
ag -o docs asyncapi.yaml html
Something went wrong:
Error: Invalid AsyncAPI document
    at parse (/usr/lib/node_modules/asyncapi-generator/node_modules/asyncapi-parser/lib/parser.js:93:11)
[
    {
        "keyword":"additionalProperties",
        "dataPath":".servers['mosquitto']",
        "schemaPath":"#/additionalProperties",
        "params": {
                "additionalProperty":"scheme"
        },
        "message":"should NOT have additional properties"
    },
    {
        "keyword":"required",
        "dataPath":".servers['mosquitto']",
        "schemaPath":"#/required",
        "params":{
                "missingProperty":"protocol"
        },
        "message":"should have required property 'protocol'"
     },
     {
        "keyword":"additionalProperties",
        "dataPath":".channels['light/measured'].publish",
        "schemaPath":"#/additionalProperties",
        "params":{
                "additionalProperty":"payload"
        },
        "message":"should NOT have additional properties"
     }
]

tried also with the streetlights.yml and ran successfully with with command
$ ag streelights.yml nodejs -p server=production
Done! ✨

Oops. We mistakenly left the scheme property in the server object. Please, rename it to protocol. I'll also fix it now on the website. Thanks!

I fixed it on the website and confirmed it works. Thanks for reporting.


feat(cheatcodes): add returndata to expectCall and AccountAccess

Component
Forge
Describe the feature you would like
It'd be nice if it was possible to specify returndata in expectCall and/or if it was provided in the AccountAccess struct from recorded state diffs, as these would allow more granular testing of the interactions between contracts.
Additional context
Think these are the primary related bits:
https://github.com/foundry-rs/foundry/blob/874ea8cd3f8bf3545d210efd7b9f4c41ff36218a/crates/cheatcodes/src/inspector.rs#L1218
https://github.com/foundry-rs/foundry/blob/874ea8cd3f8bf3545d210efd7b9f4c41ff36218a/crates/cheatcodes/src/inspector.rs#L1109

@d1ll0n could you please explain how this would be different than just asserting returned data? thank you!


include stddef.h for size_t

I've got compiler error because there is no known size_t in aes.h

Hi @ilkondr - good catch, thanks for reporting.
Will merge 👍


Hyperterm stuck at opening index.html

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Arch Linux x64
Hyper.app version:  1.1

Issue
Shows the following:
running in prod mode
electron will open file:///usr/lib/hyper/resources/app/index.html

and gets stuck there forever

See #1418

Thank you @aabmass 🙌


feat: Update "Swap to ZETA" tutorial to "Swap to Any Token"

issue: #180

Swap tBNB(BSC Testnet) -> sUSDC(Sepolia Testnet) and no withdraw
npx hardhat interact \
--contract 0x7fe0A7f3Fb57BD69b2210dd075a44246d9Fc2eDD \
--network bsc_testnet \
--amount 0.01 \
--target-token 0xcC683A782f4B30c138787CB5576a86AF66fdc31d	 \
--recipient 0x6093537Aa6C8C8bf4705bda40aC9193977208B39 \
--withdraw false

🚀 Successfully broadcasted a token transfer transaction on bsc_testnet network.
📝 Transaction hash: 0x356bf7aefb823783ffe92eea79957d0f682c2eecf1ba4b0edccf5b6c33e3c76d

npx hardhat cctx 0x356bf7aefb823783ffe92eea79957d0f682c2eecf1ba4b0edccf5b6c33e3c76d
✓ CCTXs on ZetaChain found.

✓ 0x24845bd91ad63abadb3231a88472f4efa73f0a3120c30f54805d06289cef923b: 97 → 7001: OutboundMined (Remote omnichain contract call completed)

Swap tBNB(BSC Testnet) -> sUSDC(Sepolia Testnet) and withdraw
npx hardhat interact \
--contract 0x7fe0A7f3Fb57BD69b2210dd075a44246d9Fc2eDD \
--network bsc_testnet \
--amount 0.1 \
--target-token 0xcC683A782f4B30c138787CB5576a86AF66fdc31d	 \
--recipient 0x6093537Aa6C8C8bf4705bda40aC9193977208B39

🚀 Successfully broadcasted a token transfer transaction on bsc_testnet network.
📝 Transaction hash: 0x29a294ebb021c5860bce4f071d64530f3387e161268c790717f4d03c13f19e86

npx hardhat cctx 0x29a294ebb021c5860bce4f071d64530f3387e161268c790717f4d03c13f19e86
✓ CCTXs on ZetaChain found.

✓ 0x7177cd44d4487d9799bb5d976bfd13a5b614d7a86b87017c3ad6bb87e7109300: 97 → 7001: OutboundMined (Remote omnichain contract call completed)
✓ 0x33999e5e4f739fa2f230cbc5aecc0c7f61901696e9fa8c3dd1cd703466266ce6: 7001 → 11155111: PendingOutbound (ZRC20 withdrawal event setting to pending outbound directly) → OutboundMined (ZRC20 withdrawal event setting to pending outbound directly : Outbound succeeded, mined)

pls review @fadeev @andresaiello

@lukema95 https://github.com/zeta-chain/toolkit/releases/tag/v10.0.0 has been released, which includes bytesToBool:
https://github.com/zeta-chain/toolkit/blob/d952cd7b67526c218e07030263d0d5413ab0e64c/contracts/BytesHelperLib.sol#L54-L63
If you can import it from the toolkit that'd be great!

@lukema95 https://github.com/zeta-chain/toolkit/releases/tag/v10.0.0 has been released, which includes bytesToBool:
https://github.com/zeta-chain/toolkit/blob/d952cd7b67526c218e07030263d0d5413ab0e64c/contracts/BytesHelperLib.sol#L54-L63
If you can import it from the toolkit that'd be great!

Looks like it hasn't been uploaded to the npm warehouse yet.

@lukema95 I'm looking into it.

@lukema95 v10.0.0 has been published on npm:
<EMAIL_ADDRESS>

@lukema95 v10.0.0 has been published on npm:
<EMAIL_ADDRESS>
done

@andresaiello please, review.


No means to set /SetPlatformServerAdminPassword with ConfigurationTool

No option/parameter found to set the ConfigurationTool with the command switch /SetPlatformServerAdminPassword to set/reset the ServerAdminPassword in Set-OSSServerConfig

Hi @SmartPlugins ,
We didnt had the /SetPlatformServerAdminPassword to the configuration tool cause you can set the password like:
Set-OSServerConfig -SettingSection 'ServerConfiguration' -Setting 'PlatformServerAdminPassword' -Value 'whatever'

and then use:
Set-OSServerConfig -Apply -InstallServiceCenter

Just a remark. To set the password you need to add the switch -InstallServiceCenter on the -Apply mode.
Is this enough for your use case?


issue-1090 using mocked instance of CronJobSchedulerService in s-w-c …

…tests

next fixes to #1090

Thanks! :)


Setup.py does not include project requirements

setup.py should have a section called  install_requires that includes all the dependencies of the project. In this case
    install_requires=["requests >  2"]

requests > 2 in this case is just an example. This should be the version of requests that your package will not work if it is older than. Requests changed quite a bit in the 2.0 release so > 2 should work, but I have not tested it.
You should never lock a package dependencies to a specific version unless you absolutely have to. So don't do requests == 2.23.1 or anything like that.

thanks


Ellint coverage

The purpose of this PR is to hit some missing ellint_1() and ellint_2() lines via adding tests and also to annotate some lines discovered that are not intended to be covered.

Better, good enough to merge but we will finish even more later


create higher level base64 functions

We should also provide base64_to_binary(const char* input) -> std::vector<uint8_t>, that does calculation of safe size and allocate memory internally. Or maybe something like base_to_binary(const char* input, cont: &Container) and static_assert that the Container has method resize. (credit: @WojciechMula)

I was thinking a little about API. My generic proposal is providing a convenient wrapper that would work incrementally. I mean: user provides partial data (like input buffer when reading from file) and output buffer of fixed size. Using for decoding would be something like (for decoding):
auto decoder = Base64Decoder::new();

std::string input;
input.resize(32 * 1024);

std::string output;
output.resize(16 * 1024);
while (/**/) {
      // read a few kilobytes data from into `input`

      const size_t bytes_stored = decoder.decode(input.data(), input.size(), output.data(), output.size());
      // bytes_stored will never be greater than output.size()

      write (output.data(), bytes_stored)

      if input file reached EOF {
          while (decoder.pending_output()) {
              const size_t bytes_stored = decoder.flush(output.data(), output.size());
              write (output.data(), bytes)
         }
      }
}

Of course this flexibility is at cost of performance, but my gut feeling is that if somebody want to process data in chunks, than problem is likely I/O bound.

@WojciechMula I'm ping you later today as I have a major upgrade to the base64 support, with a slightly improved API.

Please see https://github.com/simdutf/simdutf/pull/382


Error summary links scrolling past the form legend

Summary of problem
Error summary links scroll past the form legend, to see the legend, the users need to scroll up
Steps to replicate

Scroll to the bottom of a content page and from the advice feedback section ('Did this advice help?') select No
This will open a form - submit the form without filling in the form (we want to see the error message appear)
When the error summary appears, click on a link (in the example case 'Select an option')

Expected behaviour
When clicking on a link from the error summary, this should scroll the user to the legend before the input so that the user is aware what needs to be adjusted in their input
Actual behavior
When clicking on a link from the error summary, the page scrolls to the input, past the legend which means that the user would need to scroll up to check what they need to input
Context

url - content pages, example: https://www.citizensadvice.org.uk/immigration/staying-in-the-uk-if-youre-from-the-eu/

screenshot

actual

expected

browser / os / device - any

This actually exists in Jira: https://citizensadvice.atlassian.net/jira/software/c/projects/NP/boards/208?modal=detail&selectedIssue=NP-2423&search=error summary

This has been fixed


Modal not clearing/setting correct state after close combined with NavigationManager.NavigateTo

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
When using Modal and using Show and Close methods. After close, with 'await', then using: NavigationManager.NavigateTo("/Counter");
The scollbar is locked on the next page. Scrollbar keeps state: overflow: hidden and body keeps: class="modal-open"

Expected Behavior
After call .Close() on Modal, I would expect body-section to be "released" from Modal-state.
I could be using it all wrong. Would very much like to hear your opinion.
Steps To Reproduce
BlazorApp2.zip
Start the application.
Navigate to: /   (Home)
Scroll to bottom and Click the button; "Show dialog..."

Dialog will then show for 3 seconds.
Page will then redirect to "/Counter"

You will notice, that scrollbar is locked and if you inspect the page, you will find body have attributes like:

Hope this makes sense, let me know if you need further information.
Exceptions (if any)
No response
.NET Version
8
Anything else?
No response

@HarpunDK Hi
private async Task ShowDialog()
{
    await modalRef.Show();
    await Task.Delay(3_000);
    await modalRef.Close();

    await Task.Delay(300);
    NavigationManager.NavigateTo("/Counter");
}

you can add await Task.Delay(300) before navigation.
i will move the delay into Close method in the next release.

Thank you, I will try it out right now!

Sorry for the inconvenience. That did the trick.

@HarpunDK hi. i had release 8.2.4-beta01
<Modal @ref="modalRef" OnCloseAsync="OnCloseAsync">
    <ModalDialog IsCentered="true" IsDraggable="false" IsScrolling="true" ShowCloseButton="false" ShowFooter="false" ShowHeaderCloseButton="false" ShowHeader="false" Size="Size.Large">
        <BodyTemplate>
            GRRRRRR

            GRRRRRR

            GRRRRRR

            GRRRRRR

            Some content here - we will wait 3 seconds.
        </BodyTemplate>
    </ModalDialog>
</Modal>

@code
{
    private Modal modalRef = default!;

    private async Task ShowDialog()
    {
        await modalRef.Show();
        await Task.Delay(3_000);
        await modalRef.Close();
    }

    private Task OnCloseAsync()
    {
        NavigationManager.NavigateTo("/Counter");
        return Task.CompletedTask;
    }
}

recommendation use OnCloseAsync callback on Modal.
Because the Close method will trigger the client javascript Close method, which has a transition animation delay of approximately 150 milliseconds and triggers the OnCloseAsync callback after the animation ends.


Use \Closure instead of callable

callable is a problematic type, because it's scope dependent. It can therefore not be used for typed properties, but \Closure can be used as a safe replacement, especially with first-class syntax introduced in PHP 8.1: (...).
Resolves #14.

I'm undecided about this – callable can be problematic, but it's not a subtle bug if used incorrectly, it becomes quite obvious. It also seems in PhpStorm we are able to type parameters as, e.g., callable(string):void, whereas the same is not possible with Closure.

I guess we'd have to introduce separate psalm types to allow for the call signatures?

Have to agree with @trowski here, that I'm undecided about this. But I'm also fine with giving this a try and see how it works out during 0.x before making a stable decision.

Let's give this a try then. :+1:

After talking to @kelunik, we realized Psalm does support the same syntax for declaring Closure params/return types. We can always roll back to callable without a big impact to users.


tree-view .gitignore /folder/**/ pattern mismatch

If you have a .gitignore file in your repo root, and you want to ignore sub-directories inside a folder, the following pattern works in git, but Atom shows in tree-view that ALL items in "folder" are ignored (dark grey), instead of only the sub-folders.
    /folder/**/

-OR-
    folder/**/

I've ran the following command to verify what is being ignored by Git, and it showed only items inside the sub-folders are being ignored:
   find . -type f  | git check-ignore --stdin

-hence the specific pattern matching in the tree-view is ignored. I do contract & expand tree folder item to refresh after saving the .gitignore.
I hope this gets fixed to save someone else the trouble for gawking in disbelief about "working" gitignore commands on StackOverflow, when only the tree-view highlighting was lying.

Thanks very much for the report!
Would you mind reopening this issue in the https://github.com/atom/tree-view/ repository? We use multiple repositories to keep the work items tied to the code they apply to. This helps the maintainers spend more time working directly on Atom and less time triaging issues.
Also, please use the issue template when opening issues - the information and format of the template is super helpful for us when triaging issues.
Thanks for meeting us half way 👍


CMP: Adding solution, project, and filters for Visual Studio 2015 UWP compilation

Well, if you work on DirectXTK as well, this should look familiar.
Same solution as there:

Hello all,
First off, thanks for the work of this project. I have used it for HoloLens development and it saved me a ton of time and effort (and taught me a lot).
I maintain my local overwrite of the UWP solution (overwriting v141 with v140) and thought that other people may want to do UWP development in VS2015.
I realize that this change will break people's downstream projects that include the .vcxproj, so an alternative could be to leave DirectXTK_Windows10.vcxproj unchanged (2017) and add DirectXTK_Windows10_2015.vcxproj
Thoughts?
Adam

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

I now see that VS2017 is the recommended IDE for HoloLens dev. Aborting! Sorry.


how to align the text to keep it in the middle

Report
What did you do?
ℹ I am trying to make an OTP Screen page in iOS i have achieved all the functionality just i got one issue that i want it keep the text in the middle of the textfield
What happened instead?
ℹ it starts from the exact corner of the text field
Xcode Version: 9.2
SkyFloatingLabelTextField Version: updated version
Method of Integration: CocoaPods | Carthage | Manual : CocoaPods

Hey @abhijitpp can you give some more details on this? The component is an extesions of UITextField so any method that works for centering text in a UITextField should be applicable

sky ignores the alignment property of the base textfield.

It should appear like this:
https://storage.googleapis.com/spec-host-backup/mio-design%2Fassets%2F1V4TTJr8n_fpZkdk4qQIqXB9vJhMegre3%2Ftextfields-filled-motion2.mp4
(Watch vertical centering)
Thank you!

@abhijitpp ,
Update the function as below in SkyFloatingLabelTextField class -
fileprivate func updateTextAligment() {
if isLTRLanguage {
textAlignment = .center
titleLabel.textAlignment = .center
} else {
textAlignment = .right
titleLabel.textAlignment = .right
}
}

Same issue here. Can't align the text to right of text field. Can anyone help?


Simplify computing force of gravity

Currently, the system for computing gravitational force is fairly complicated with like three different constants involved. Right now, it uses Newton's formula for force due to gravity, F = G * (Mm)/(r^2). This has the nice property where the gravity increases the closer the attracted object is to the attracting object (and can be seen in the simulation too if you squint your eyes a bit!). However, besides just being complicated, it's also not too realistic. We mostly experience the same 9.8 m/s^2 at the surface. For our simulation too, we'll choose an appropriate constant force and use that, which should also simplify things a bit.

donezo!


Notebook changedetect

Note: checklist below only relevant if committing to the develop branch:
Proposed changes
Update to change detection notebook for overhaul.
Checklist (replace [ ] with [x] to check off)

[X] Notebook created using the DEA-notebooks template
[X] Remove any unused Python packages from Load packages
[X] Remove any unused/empty code cells
[X] Remove any guidance cells (e.g. General advice)
[X] Ensure that all code cells follow the PEP8 standard for code. The jupyterlab_code_formatter tool can be used to format code cells to a consistent style: select each code cell, then click Edit and then one of the Apply X Formatter options (YAPF or Black are recommended).
[X] Include relevant tags in the final notebook cell (refer to the DEA Tags Index, and re-use tags if possible)
[X] Clear all outputs, run notebook from start to finish, and save the notebook in the state where all cells have been sequentially evaluated
[x] Test notebook on both the NCI and DEA Sandbox (flag if not working as part of PR and ask for help to solve if needed)
[x] If applicable, update the Notebook currently compatible with the NCI|DEA Sandbox environment only line below the notebook title to reflect the environments the notebook is compatible with

Fantastic notebook, @caitlinadams ! I've run this notebook on the NCI and updated the compatible environments comment to reflect this.  Good to push to develop IMO

Perfect, thanks @cbur24! Would you mind merging when you're ready?


[BugFix] fix lake pk compaction invalid memory access

Why I'm doing:
When do compaction and load state, get_each_segment_iterator can't use a stat in a stack. otherwise it will access invalid memory.
What I'm doing:
get_each_segment_iterator with gloabl stat.
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

use once flag like local update compaction state?

PR was manually closed.

use once flag like local update compaction state?

But we need to load compaction state one segment by one segment for saving memory.

PR was manually closed.


Fixes firmware version for H1 (G1)

Incorrect input registers for Master, Manager, Slave firmware versions on H1 (G1), holding register values are correct.
See https://github.com/nathanmarlor/foxess_modbus/issues/599

Thanks!


Merge updates from develop_1

Sorry, but this pull request was created by mistake, so I have closed the pull request.

No problem.  I've been too busy with other projects and work to look at this.  If you do wind up making changes and want to push them, provided everything is working, feel free.


Examples are too simplistic

Questions that I had after reading this.  (I can imagine lots of people asking these)

What if "WeatherForecast" has a property that's another class.  Do I need to list it as well or will the compiler automatically include it?
What if I'm actually serializing a list of WeatherForecasts?
If it's falling back to Reflection, is there a way for me to know?  I'd like to make it so that a maintainer of my code doesn't accidentally do something that compromises performance and there being no way for them to know that they've made this mistake.

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

ID: ee53c3dc-df79-82ef-ce25-3233ff292c55
Version Independent ID: 558903e9-8173-464e-ff8b-7744419ad7e4
Content: How to use source generation in System.Text.Json
Content Source: docs/standard/serialization/system-text-json-source-generation.md
Product: dotnet-fundamentals
GitHub Login: @tdykstra
Microsoft Alias: tdykstra

What if "WeatherForecast" has a property that's another class. Do I need to list it as well or will the compiler automatically include it?

What if I'm actually serializing a list of WeatherForecasts?

Generally, you only need to specify types that will be passed to the serializer at the root level as [JsonSerializable]. There's no need to specify data types of properties as well. If it's a list of WeatherForecasts to be processed at the root of the serializer, you'd want to specifiy [JsonSerializable(typeof(List<WeatherForecast))].
The only other types that need to be specified are runtime types that are declared as object, in the case of polymorphic serialization. For example
public class WeatherForecast
{
    public object Data { get; set; }
    public List<object> DataList { get; set; }
}

[JsonSerializable(typeof(WeatherForecast))]
[JsonSerializable(typeof(bool))] // runtime type for member declared as object needs to be specified.
[JsonSerializable(typeof(int))] // runtime type for member declared as object needs to be specified.
public partial class WeatherForecastContext : JsonSerializerContext { }

WeatherForecast wf = new() { Data = true, DataList = new List<object> { true, 1 } };
JsonSerializer.Serialize(wf, WeatherForecastContext.Default.WeatherForecast);

If it's falling back to Reflection, is there a way for me to know? I'd like to make it so that a maintainer of my code doesn't accidentally do something that compromises performance and there being no way for them to know that they've made this mistake.

The source generation feature does not fall back to reflection, by design. If there's a type for which there's no source-generated metadata (i.e. not specified as [JsonSerializable]), the serializer will throw a NotSupportedException at runtime with an appropriate message.

Thanks, @layomia! I'll update the article.

Thanks for the hard work here - I was able to figure out all that @layomia reported for myself without too much trouble - the whole package is very well designed and most things are pretty intuitive to guess through.  Updating the article would definitely be great as the example, as I read it, left me wondering whether it would work at all for my use case.


Update flow.rst

This patch makes searching for TODO comments with current ticket number an explicit step in the development workflow.

@ktlim would you be able to review this?


Unsatisfiable dependencies for platform linux-64

I am planning to submit a package to bioconda, which failing and got the following error. Any suggestion on how to fix it?
conda_build.exceptions.DependencyNeedsBuildingError: Unsatisfiable dependencies for platform linux-64: {"streamlit-tags[version='>=1.1.9']"}

Can you add this command in too?
conda config --add channels gagan3012

Following is my meta.yml file for Bioconda recipe. Should I pass the channel info after the script section?
{% set version = "0.2.0" %}
{% set name = "GuideMaker" %}
{% set sha256 = "c9cb6e35844209259159ee0530390d837748678538b6a9ed7e71645ba51d57be" %}
{% set user = "USDA-ARS-GBRU" %}

package:
  name: '{{ name|lower }}'
  version: '{{ version }}'

source:
  url: https://github.com/{{ user }}/{{ name }}/archive/v{{ version }}.tar.gz
  sha256: {{ sha256 }}

build:
  number: 1
  noarch: python
  entry_points:
    - guidemaker=guidemaker.cli:main
  script: python -m pip install --no-deps --ignore-installed .

requirements:
  host:
    - pip
    - python >=3.6
    - biopython==1.76
  run:
    - pip
    - python >=3.6
    - biopython==1.76
    - biopython==1.76
    - numpy >=1.13
    - pybedtools==0.8.2
    - nmslib>=2.0.6
    - pandas>=1.0.0
    - pyyaml>=5.4.1
    - regex==2020.11.13
    - altair
    - streamlit>0.80.0
    - pytest>=4.6
    - pytest-cov
    - streamlit-tags>=1.1.9
    - pdoc3

test:
  commands:
    - {{ name }} --version | grep '{{ version }}'
    - {{ name }} --help

about:
  home: https://github.com/USDA-ARS-GBRU/GuideMaker
  license: 'CC0 1.0'
  license_family: 'PUBLIC-DOMAIN'
  license_file: LICENSE.txt
  summary: 'GuideMaker: Software to design gRNAs pools in non-model genomes and CRISPR-Cas systems'

extra:
  recipe-maintainers:
    - arivers
    - ravinpoudel
  identifiers:
    - biotools:GuideMaker
    - doi:10.5281/zenodo.4849258

Is it possible to put streamlit-tags on Conda-forge channel?  Following is the text from bioconda. https://bioconda.github.io/contributor/guidelines.html
There is currently no mechanism to define, in the meta.yaml file, that a particular dependency should come from a particular channel. This means that a recipe must have its dependencies in one of the following:

conda-forge channel
bioconda channel
default Anaconda channel

I have already created a PR for it waiting for their response

Thanks. I will send my PR after streamlit-tags became available at Conda-forge

Any updates on your PR?

https://github.com/conda-forge/staged-recipes/pull/15164

Should I be using following to install now?
conda install -c conda-forge streamlit-tags

Yes once the pr is merged

Hello,
Streamlit_tags is on conda forge
conda install -c conda-forge streamlit-tags

Thanks @gagan3012
However, I am still getting the following error. Appreciate any suggestion and help.
conda_build.exceptions.DependencyNeedsBuildingError: Unsatisfiable dependencies for platform linux-64: {"streamlit-tags[version='>=1.2.6']"}
More at:
https://app.circleci.com/pipelines/github/bioconda/bioconda-recipes/47452/workflows/4f9004d5-f9fa-4f3f-85fa-6ecc1904c3c7/jobs/156637
My setup.py looks as following:

"""setup.py: python package setup for GuideMaker
"""

from setuptools import setup
import versioneer

requirements = [
    # package requirements go here
    'biopython==1.76',
    'numpy >=1.13',
    'pybedtools==0.8.2',
    'nmslib>=2.0.6', # note 2.1.1 does not pass the build in ubuntu
    'pandas>=1.0.0',
    'pyyaml>=5.4.1',
    'regex==2020.11.13',
    'altair',
    'streamlit>0.80.0',
    'pytest>=4.6',
    'pytest-cov',
    'streamlit-tags>=1.2.6',
    'pdoc3'
]

setup(
    name='guidemaker',
    version=versioneer.get_version(),
    cmdclass=versioneer.get_cmdclass(),
    description='GuideMaker: Software to design gRNAs pools in non-model genomes and CRISPR-Cas',
    license='CC0 1.0 Universal (CC0 1.0) Public Domain Dedication',
    author='Adam Rivers',
<EMAIL_ADDRESS>    url='http://tinyecology.com',
    long_description=open('README.md').read(),
    packages=['guidemaker'],
    entry_points={
        'console_scripts': [
            'guidemaker=guidemaker.cli:main'
        ]
    },
    install_requires=requirements,
    python_requires='>=3.6',
    test_suite='pytest',
    tests_require=['pytest'],
    keywords='CRISPR-Cas',
    include_package_data=True,
    zip_safe=False,
    classifiers=[
        'Programming Language :: Python :: 3.7',
        'Programming Language :: Python :: 3.8',
        'Development Status :: 3 - Alpha'
    ]
)

I see I think if you rename it to streamlit_tags it should be fixed

Thanks!!!


add Option#filter

Adds Option#filter method

Thanks @raveclassic. To all: are there any new additions which should go in 0.5.4?

@gcanti Yep, I'm going to open another PR in a minute


Add files via upload

Lab4: IBM Quantum Challenge Fall 2022
Name:
Score:
Checklist before sharing the notebook to the community

[ X] Notebook is in the format Name_Score_Fall22_Lab4.ipynb
[X ] Check for import statements

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add more interesting case studies of successful designers and design work

The affects several chapters. During our reading sessions, we have encountered several places where the AI is trying to be inspiring by reference projects like Fedora or Element OS as case studies for great design. Those are just a little too big and abstract, and there are no good links to actually tell those stories and show how design made a difference. Let's gather some better examples.
What would be nice to have:

Stories of individual designer (or teams) about their journeys
Case studies of specific design projects (rebranding, restyling, user research, adding a new feature...)
Projects with a good design process
Overall successful projects due to design

Chapters for this content:

Chapter 1: Open Source -> The Impact of Open Source
Chapter 2: Understanding the Designer's Role -> Success Stories
Chapter 11: Case Studies

There might be more opportunities to sprinkle this content throughout.
If you have such a story to tell, or know someone who does, or know of a good case study we can reference, please share.

This also applies to the "Success stories" sectioning in chapter 2.

May i suggest that the process for adding these is an online form as well as submitting a PR? or possibly a discussion forum topic in this github repo or maybe a thread in discord?
I think the reason I ask is a form might give some structure e.g. title, description (with word limit), add up to X number images etc.
a discussion section might also identify where my case study doesn't make sense before it goes live.


[SPARK-43541][SQL] Propagate all Project tags in resolving of expressions and missing columns

What changes were proposed in this pull request?
In the PR, I propose to propagate all tags in a Project while resolving of expressions and missing columns in ColumnResolutionHelper.resolveExprsAndAddMissingAttrs().
Why are the changes needed?
To fix the bug reproduced by the query below:
spark-sql (default)> WITH
                   >   t1 AS (select key from values ('a') t(key)),
                   >   t2 AS (select key from values ('a') t(key))
                   > SELECT t1.key
                   > FROM t1 FULL OUTER JOIN t2 USING (key)
                   > WHERE t1.key NOT LIKE 'bb.%';
[UNRESOLVED_COLUMN.WITH_SUGGESTION] A column or function parameter with name `t1`.`key` cannot be resolved. Did you mean one of the following? [`key`].; line 4 pos 7;

Does this PR introduce any user-facing change?
No. It fixes a bug, and outputs the expected result: a.
How was this patch tested?
By new test added to using-join.sql:
$ PYSPARK_PYTHON=python3 build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite -- -z using-join.sql"

and the related test suites:
$ build/sbt -Phive-2.3 -Phive-thriftserver "test:testOnly org.apache.spark.sql.hive.HiveContextCompatibilitySuite"

Merging to master. Thank you, @cloud-fan for review.


Version bump

Hi,
just wondering if this repo is still actively maintained?
If possible, I would like to see some version bumps, i.e., grafana >= v6.7
There are even some open pull requests for this, i.e., #392 and #394
What do you think?

I just cut a new release v26.3.0


add a bulk API for setting key/values (index)

add a bulk API to set a list of key/values at once

Coverage increased (+0.01%) to 95.145% when pulling 20cde9f37fd2a0bc784c9fd7d7a3fcf34ce8762d on hendrikmuhs:indexer-bulk-add into 97f25a9e9f442cd05fac7487a13f1d06b982d00a on KeyviDev:master.


Add Kotlin-faker example

issue #33
Prints fake name, address, phone number, coffee information and rick & morty information.
Also prints name, address and phone number with the locale set to ES

Looks good to me!
Obrigado  @larissayasin !


How can I use Apex/Naked domain instead of www (sub domain name)?

The stack work perfect but how can I use Apex/Naked domain instead of www (sub domain name)?

I guess this is what you are looking for: https://github.com/aws-samples/amazon-cloudfront-secure-static-site/pull/30

Thank @ultimaweapon . Closed