☂ Upstream Flutter issues affecting Zulip

This tracking issue collects notes on potential work in Flutter upstream that would benefit the (new) Zulip mobile app. See also:

#1079

Some of these are much more important to us than others. They're organized by where the work would happen, not by importance.
Many of these have corresponding issues in our tracker with the "upstream" label. Conversely, all issues with that label should appear either below or in the sibling issue #1079.
Framework

WidgetSpan double-applies text scaling: https://github.com/flutter/flutter/issues/126962
In Zulip the main symptom is on @-mentions: #735
Especially affects a11y, at large font size: one @-mention can easily occupy 1/3 of the screen.

(TODO upstream issue)
RawAutocomplete should gain a "controller" for customizability.
#226

(TODO upstream issue)
Richer Flex (aka Row and Column), with more CSS flexbox features like flex-basis vs flex-grow vs flex-shrink. (This is one of the few things we miss from React Native: its Yoga layout engine has those features.)
We probably need something like this for:
#282
#433

https://github.com/flutter/flutter/issues/117061
#638

https://github.com/flutter/flutter/issues/57609

https://github.com/flutter/flutter/issues/52207
The Zulip impact is that if you try the "scroll to end" button when scrolled a long way back in history, the result is very choppy — like ~1fps — until it reaches the end.

(TODO upstream issue)
#69

(TODO upstream issue)
InputDecoration should gain a helper field, to generalize helperText in the same way as error vs. errorText and label vs. labelText.
We'd use this on the "server URL" field in the login flow.
https://github.com/zulip/zulip-flutter/issues/109#issuecomment-1990971435
https://github.com/zulip/zulip-flutter/issues/109#issuecomment-1992745149

Engine

The Color API is in flux; to accommodate it, we're currently (since 2024-08) relying on versions of package:color_models and package:flutter_color_models from a branch maintained by a member of the Flutter engine team, instead of released versions.
This isn't a stable situation and we hope the API settles soon.
https://github.com/zulip/zulip-flutter/issues/918

(TODO upstream issue)
We want to have scripts that run as a CLI (at dev time) and import our model code.

E.g., #190
where we wound up with a real hack of putting the logic in a "test", run with flutter test, and having a wrapper script around that to accept a more reasonable CLI and run flutter test. (flutter test has a fine CLI for its actual job; but a terrible one for running an arbitrary tool as a script, which it's not intended for.)
Potentially also: #617
This is a well-known problem in the community. The trouble is our model code imports package:flutter/foundation.dart for things like ChangeNotifier and dart:ui for things like Color; the former imports the latter; and dart:ui is available only in the Flutter SDK, not dart run.
One existing proposal is to move some definitions to a package outside the Flutter SDK. Not sure if that will succeed in solving the problem.

https://github.com/flutter/flutter/issues/148026
Has a proposed API from Hixie which would solve it; needs implementation.
The symptom in Zulip is:
https://github.com/zulip/zulip-flutter/issues/500

Skia / SkParagraph

Emoji small on iOS: https://github.com/flutter/flutter/issues/28894

(TODO upstream issue)
Padding for text spans, to let us match Zulip web's styling for code spans.
An overlapping issue: flutter/flutter#30541

Flutter on Android

https://github.com/flutter/flutter/issues/137760
Symptoms in Zulip:
https://github.com/zulip/zulip-flutter/issues/546
(There might be another upstream issue underlying that Zulip issue thread, too.)

Flutter on iOS

(TODO upstream issue)
https://github.com/zulip/zulip-flutter/issues/591

Packages (first-party)

package:video_player bugs on iOS:
https://github.com/flutter/flutter/issues/126760
https://github.com/flutter/flutter/issues/139107
Because of these, for now we don't try to play video in the message list; we only play if the user opens in lightbox.
Details here: https://github.com/zulip/zulip-flutter/issues/656

Tool

We want to specify in our tree what Flutter version to use:
https://github.com/flutter/flutter/issues/95472
https://github.com/zulip/zulip-flutter/issues/15
The design space here is a bit tricky: we want the pinned version to be automatic, but also preserve our low friction for developing upstream changes in tandem with changes in our tree.

(TODO upstream issue)
After upgrading a package, running its CLI with flutter pub run still gets the old version. Running with dart run gets the new version, and works around the problem.
For Zulip this shows up when running pigeon:
https://github.com/zulip/zulip-flutter/issues/622

https://github.com/flutter/flutter/issues/123977
Greg, for one, finds this a pretty substantial source of friction in developing changes to the Flutter tree.

flutter_test

flutter/flutter#123189
This is the reason many of our tests have to end with debugNetworkImageHttpClientProvider = null; — even tests that don't otherwise mention that variable. With this issue fixed, a line setting a variable like that one to some non-default value could be paired immediately with an addTearDown to reset it.

Integration tests

We'll want to write some integration tests. The current state of writing those seems challenging:
https://github.com/flutter/flutter/issues/142021
though happily it looks like there's active work going on right now.
In the future once we actually sit down to writing these, we may have more specific thoughts on what we want to see.
https://github.com/zulip/zulip-flutter/issues/757

(Posting a first comment, to have the option to edit it as a home for related information in the future.)


[SUGGESTION] Add Saplings

Except for Spruce and Birch, which leaves does not have tint applied

Interesting...I'll think about it!


authorize is possibile only by GET and not by POST

as specific OIDC https://openid.net/specs/openid-connect-core-1_0.html#AuthorizationEndpoint authorization endpoint MUST support the use of the HTTP GET and POST methods.
But if you try a POST request AAC responds with a 404:
{
"timestamp":<PHONE_NUMBER>342,
"status": 404,
"error": "Not Found",
"message": "No message available",
"path": "/oauth/authorize"
}

close with #425


Remove playground auto-join during signup

Related to #2080
What does this PR do?
Remove option to auto-join playground group on signup. This seemed to confuse people.
Links to related issues
Checklist

[ ] added a test, or explain why one is not needed/possible...
[x] no unrelated changes
[x] asked someone for a code review
[x] joined chat.foodsaving.world/channel/karrot-dev
[x] added an entry to CHANGELOG.md (description, pull request link, username(s))

Yeah, that's some almost-unused logic there! I'll comment on the issue instead ;)


Ability to hide host name is lost in 8.1.8

Describe the bug
I switched recently to 8.1.8 and find out that Zabbix hosts names on graphs with several hosts are no longer hidden, if only one host is selected.
Expected behavior
I believe it was someone's feature request, to always show hostname, but there are cases when it is, in fact, inconvenient. At least it could be an option.
Looks like replaceAlias or setAlias in that case is of no help, since when several hosts are selected, it is OK to show their names.
Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

8.1.8
5.0.18
x.x.x

+1
Overrides that were set on "fields with name" now require rewriting, also..
Will need to downgrade, I think!

Grafana v8.0.0 & v8.3.3,
Grafana-Zabbix Plugin 4.2.4
It appears to me that host name is always shown regardless of single or multiple hosts are selected. Could it be optional? :)


Streamline Maven Migration

Users planning to migrate from Maven to Gradle need to be equipped with capable tools that provide a smooth transition. Well known concepts in the Maven world need to translate to Gradle to ensure compatibility with existing Maven infrastructure. Appropriate user guides and samples need to be established to educate users about migration strategies.
Expected Behavior
Maven projects can be easily translated to Gradle projects without a lot of manual intervention. Users are guided through the transition by appropriate documentation. Existing shortcomings of Gradle's interoperability with Maven infrastructure is minimized.
Current Behavior
The Gradle init task can help to get started with Maven migration but lacks in features. Manual intervention is necessary. There's little information about how Maven concepts translate to Gradle. Migration of large Maven projects to Gradle is not trivial and requires intricate knowledge about both ecosystems. Specific Maven features are simply not supported by Gradle.
Context
Over time Maven users will have a look at the Gradle ecosystem to see if there are better solutions to the problems they are facing. The easier the tooling and documentation for a migration, the more likely it is for users to start a prototype.
Possibly interesting words from Eric

I see some quick wins here:

Improving the init task to make it even easier for people to migrate without manual intervention.
Heavily promote the Maven migration guide to the community and harvesting feedback. I can't even find the guide when googling for it.

Personally, I'd like to see the BOM support in Gradle though that's a personal opinion and requires some deep thinking.

Some things mentioned here have been done (BOM/platform support, improvements to gradle init, updates to the migration guide). Since this issue is no longer tracking any current or planned work, I am closing it.


Rewrite RunnableResolver

It seems this should be the primary goal for 0.0.8. Let's say the current resolver... misbehaves. If you know what I mean. And it's practically the first thing a Novika user interacts with! It must be super obliging. I burned myself out quite heavily due to resolver misbehaving already -- was bitten by it!
Linked to: #69.
Anyway.

Note that I'm working on this, it's just that it's pretty hard to do properly. And I can't do anything else, because my other ideas kind of rely on a proper resolver implementation.
The latest (12th or 13th, I can't remember!) revision was a rewriting system, which seems to be the best choice when it comes to the way I should implement the resolver.
It didn't quite work though; that is, it will work the same as the current implementation, but I'm not just rewriting, I'm not that stupid! I'm rewriting to enable new features, and to fix the permission system! Minus that, everything's fine. With new features and permissions, not really. It'd probably take a few more weeks as I learn more advanced rewriting techniques (I am particularly interested in graph rewriting now, although I don't know whether it's going to end up being useful).
I guess I could brute-force the solution and then refactor it to something more manageable... Who knows?


feat: add icon support to the wireplumber module

Adds basic icon support for the wireplumber module.
This can be achieved by using {icon} in the format config and pairing it with the format-icons config as well.
Example:
"wireplumber": {
    "format": "{volume}% {icon}",
    "format-icons": ["", "", ""]
}

Adding because #1811 indicated it wasn't supported.

May Linus Torvalds bless you.


CreateECSService.ps1 container does not have a container port 8080 defined

Doing the workshop on Windows 10 developer workstation.
Module-2
Creating a Service with Fargate
Creating a Service Linked Role for ECS
When run CreateECSService.ps1 get this error:
New-ECSService : The container MythicalMysfits-Service did not have a container port 8080 defined.
I don't know if it is related but earlier in the module I applied this fix from issue 132:
https://github.com/aws-samples/aws-modern-application-workshop/issues/132

Go the same issue and fixed it by going to Amazon ECS -> Task Definitions -> mythicalmysfitsservice -> mythicalmysfitsservice:1 -> Create New Revision -> Configure via JSON and change:
"portMappings": [],
to:
"portMappings": [
{
"hostPort": 8080,
"protocol": "tcp",
"containerPort": 8080
}
],
Than click save and ran again CreateECSService.ps1 script.

See my reply in https://github.com/aws-samples/aws-modern-application-workshop/issues/132#issuecomment-829209534
After I fixed that and ran RegisterECSTaskDefinition.ps1 again I got a valid task definition and CreateECSService.ps1 succeeded, too.


Allow a view to be re-created on rotation

Different views may be used for landscape and portrait, allow a way to cause the different views to be inflated.

Closing as fixed.


Plans for logging?

I don't understand how you can cope without working logging 😄
Could we add some logging configuration, at lease so that when DEBUG is set log output goes somewhere useful? Ideally (for me) the web server console? I think we also need production logging of unexpected events (so probably warn and above) to help diagnosing failures.
I could set something up, but I don't feel I really understand Django's interaction with the standard Python log framework so you probably wouldn't like the result...
Just adding
if DEBUG:
    # will output to your console
    logging.basicConfig(
        level = logging.DEBUG,
        format = '%(asctime)s %(levelname)s %(message)s',
    )

to tfc_web/tfc_web/settings_dev.py seems to come close to meeting my immediate needs.

PR #106 sets up some default Django logging for tfc_web that sends output to Gunicorn's console and hence (thanks to run.sh) to /var/log/tfc_prod/gunicorn.log. this is truncated each time Gunicorn is restarted with run.sh, and we probably need something better.
nginx logging also needs attention. Current plan is to get the sites-{available,enabled} scripts to direct logging to /var/log/nginx/{access,error}.log which Ubuntu's logrotate will rotate for us.
Current testing this on tfc-app4.


Update onyx

Details
Updates Onyx
Context for removing Onyx from webpack includeModules - https://github.com/Expensify/App/issues/9163#issuecomment-1137363084
Fixed Issues
$ https://github.com/Expensify/Expensify/issues/211233
Tests

Run npm i and verify that the app still works

[x] Verify that no errors appear in the JS console

QA Steps

Launch the app and verify that it works

[ ] Verify that no errors appear in the JS console

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @AndrewGable in version: 1.1.67-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @Julesssss in version: 1.1.67-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Setting timeout with variable data

Hi.
I'm trying to set the timeout with a value that should come from a function in $scope, but in that momento of the app loading, nothing like $scope of anything else exists yet.
How could I do this? Is it even possible?
Thanks.

Having the exact same issue. I need to pull the idle and timeout data from an external service depending on the client logged in. Any help on this would be great!

@alexandroscgb @adamfollett - Correct, you can't do this on config while angular is bootstrapping itself (you can only manually set the values in config and only providers can be injected into a configuration block)
However, after initial config, you should be able to use setTimeout() or setIdle() on the Idle service. You should be able to do that in any service, controller, or factory (e.g., immediately on/after login) that needs access to $scope (I would recommend a service).
See: https://github.com/HackedByChinese/ng-idle/issues/206#issuecomment-315958885


when jumping among pages, styles lost

Bug report
Describe the bug
I found this bug in with-ant-design example. but I think it's not a bug of example, it's a bug of next.js.
In this example, there is only one page, so it works fine. but, when built one more page, and jump between pages, you can find styles lost.
To Reproduce

run the with-ant-design example properly (by npm run dev). It looks good:

build one more page (named: other), and create a link between them. here is other.js source:

import Layout from '../index.js'
import Link from 'next/link'
import { Select, Button } from 'antd'
const Option = Select.Option
export default () => (
    <Layout>
        <div style={{ marginTop: 100 }}>
            <Link href="/"><Button>Back to main</Button></Link>
            <Select>
                <Option value={null}>请选择</Option>
            </Select>
        </div>
    </Layout>
)

nothing special

at main page, click the link to other, and then, click the Back to main to go back. then you can see the styles are not corrent:

actually, all of those components which haven't been imported by the other.js page are lost.
and when this occurs, you can do nothing to get the styles back, but only by restart npm run dev
Expected behavior
I have tried more UI framework such as 'zent', and got into the same problem. so I think it's not a bug of antd
I hope the styles can work properly, wherever they were imported.
System information

OS: Windows 10
Version of Next.js: 6.1.1

but I think it's not a bug of example, it's a bug of next.js.

Pretty sure it is a bug in the example. As Next.js doesn't handle styles besides styled-jsx, and antd doesn't use styled-jsx.

i think the problem you describe is related with the custom document component, here are two closed issues about that: https://github.com/zeit/next.js/issues/251 and https://github.com/zeit/next.js/issues/1035

@naojamg noop, there is no custom document in with-ant-design example

@timneutkens Thank u for your hard work! I upgraded the next.js and plugin to the lateset canary version, then this issue has gone. I think this issue has been resolved

Great, will close this then.

Hi @wslx520, I have next ver 6.1.1 and next-css ver 0.2.0 but I still have the issue with style disappearing. How did you fix it?

@qbf75196  I updated dependencies to latest canary version:
"dependencies": {
    "@zeit/next-css": "^0.2.1-canary.1",
    "antd": "^3.3.1",
    "babel-plugin-import": "^1.1.1",
    "next": "^6.1.1-canary.3",
    "react": "^16.0.0",
    "react-dom": "^16.0.0"
  }

and no changes in next.config.js:
/* eslint-disable */
const withCss = require('@zeit/next-css')

// fix: prevents error when .css files are required by node
if (typeof require !== 'undefined') {
  require.extensions['.css'] = (file) => {}
}

module.exports = withCss()

Above code is based on with-ant-design example as I said.
may help u


Hide 'run_system_tests.py' from nose, no matter how called.

LGTM (pending Travis)
Thanks for doing this.


sentimentAnalysisTasks opinion mining is string even though swagger specifies boolean in 3.1

Input:
{
"tasks": {
"sentimentAnalysisTasks": [
{
"parameters": {
"opinionMining": true
}
}
]
},
"analysisInput": {
"documents": [
{
"id": "1",
"text": "The park was clean and pretty. The bathrooms and restaurant were not clean.",
"language": "en"
}
]
}
}
Output:
{
"error": {
"code": "InternalServerError",
"innerError": {
"requestId": "33b6e341-5cfd-4a24-a014-183f81d08252"
},
"message": "Internal server error"
}
}

https://msazure.visualstudio.com/Cognitive Services/_workitems/edit/10123726

Fixed. need to enable tests again


Add new vinbud version that includes items

Thanks to @davidarnarsson and his wonderful scraper we are in the business of all items offered in the liquor store.

I thought the same thing when I wrote it and I'm down for changing it to vinbud.is. When I was typing it up I just thought since tldr.is is the datasource it made sense, even if it's not the original datasource.
I think I'll change it for this.

LGTM


What url for vue.esm-browser.js should I put into the import map?

I'd like to fix this error (I try to use compile()):
Runtime compilation is not supported in this build of Vue. Use "vue.esm-browser.js" instead.
 at <Repl>

Remove `runtime.` in the `Import Map` tab.


doc: input: add a dedicated GPIO keyboard documentation

This driver supports few different hardware configurations, add a document page showing some of them next to some schematics snippets
https://builds.zephyrproject.io/zephyr/pr/66450/docs/services/input/gpio-kbd.html

BTW: this is a really nice doc page! <3

https://github.com/zephyrproject-rtos/zephyr/pull/67308 for the followups.


Specifying POST body content-type

I was using requests to submit POST requests to a third-party website when I noticed that the files parameter can also accept a three- or four-element tuple that denotes body content-type (and another header?) (https://github.com/kennethreitz/requests/blob/master/requests/models.py#L120) instead of just two as noted in the docs.
Is there a reason this is not yet mentioned in the docs or can I expect this interface may break in future releases?
I'm on Requests version 2.0.1, by the way :).

hola caracola

Sounds great! I'll get right on it once I feed my cat.


Modify leaf segment key generator related functions.

Changes proposed in this pull request:

modify the name of some static final attribute
modify the constructor.
add "final" to the related test class.
modify methods: generateKey(),initLeafSegmentKeyGenerator(final String leafKey)
,increaseIdWhenLeafKeyStoredInCenter(final String leafKey).

Pull Request Test Coverage Report for Build 8589

10 of 11 (90.91%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 60.634%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-orchestration/sharding-orchestration-core/src/main/java/org/apache/shardingsphere/orchestration/internal/keygen/LeafSegmentKeyGenerator.java
10
11
90.91%

Totals

Change from base Build 8587:
0.0%

Covered Lines:
8410

Relevant Lines:
13870

💛  - Coveralls


A bad version id detected in the configuration page

Describe the bug
In the configuration page of the autoupgrade module, we have a checklist.
When we have an old module, the first checklist is wrong.
We have "The 1-click upgrade module is up-to-date (your current version is v4.5.1)" with a right icon, which is wrong, the latest version is 4.6.0, so the module is not up-to-date.
Thanks!
Screenshots

Server (please complete the following information):

Web Server: Apache
PHP Version: 7.1
Module Version 4.5.1

ping @Quetzacoalt91

Can you please check your file config.xml, and remove all existing config_<iso_code>.xml? We rely on these files to check the current version.
On the other side, this test is always green, because we must now handle to different versions of the same module. One for PS < 1.6, and the other for 1.6 & 1.7.

Hi @Quetzacoalt91,
I don't have any config_<iso_code>.xml
In my file config.xml, I have this:
<?xml version="1.0" encoding="UTF-8" ?>
<module>
	<name>autoupgrade</name>
	<displayName><![CDATA[1-Click Upgrade]]></displayName>
	<version><![CDATA[4.5.1]]></version>
	<description><![CDATA[Provides an automated method to upgrade your shop to the latest version of PrestaShop.]]></description>
	<author><![CDATA[PrestaShop]]></author>
	<tab><![CDATA[administration]]></tab>
	<is_configurable>1</is_configurable>
	<need_instance>1</need_instance>
	<limited_countries></limited_countries>
</module>

I installed the 1-click upgrade befor the release of v4.6.0., which is v4.5.1.
No, after the release of the new version v4.6.0, in the checklist of this module, it should net be displayed "up-to-date".
What do you think?
Thanks!


Add Markdown Plus to Downstream projects

Markdown Plus is a markdown editor with extra features. The frontend is open source under MIT license. It provides apps for OS X and Windows for purchase.

Thanks! Sorry about that I missed your editor its a great one!


Implement webview proxy for cross platform

What does this PR do?
Adds internal function to consume web view events for cross-platform SDKs
Additional Notes

Review checklist (to be filled by reviewers)

[x] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[x] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[x] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Codecov Report

Merging #1290 (f9dee9e) into develop (dde54aa) will increase coverage by 0.12%.
The diff coverage is 86.36%.

@@             Coverage Diff             @@
##           develop    #1290      +/-   ##
===========================================
+ Coverage    81.85%   81.97%   +0.12%
===========================================
  Files          361      361
  Lines        12835    12854      +19
  Branches      2148     2151       +3
===========================================
+ Hits         10506    10537      +31
+ Misses        1665     1664       -1
+ Partials       664      653      -11

Impacted Files
Coverage Δ

...oid/src/main/kotlin/com/datadog/android/Datadog.kt
77.36% <0.00%> (ø)

.../main/kotlin/com/datadog/android/_InternalProxy.kt
89.66% <90.48%> (+9.66%)
:arrow_up:

...android/log/internal/logger/TelemetryLogHandler.kt
85.71% <0.00%> (-14.29%)
:arrow_down:

...droid/rum/tracking/FragmentViewTrackingStrategy.kt
75.00% <0.00%> (-3.85%)
:arrow_down:

...n/com/datadog/android/v2/core/SdkInternalLogger.kt
88.89% <0.00%> (-2.78%)
:arrow_down:

...id/rum/internal/ndk/DatadogNdkCrashEventHandler.kt
91.00% <0.00%> (-2.00%)
:arrow_down:

...android/v2/core/internal/net/DataOkHttpUploader.kt
94.94% <0.00%> (ø)

.../android/rum/internal/monitor/DatadogRumMonitor.kt
89.42% <0.00%> (+0.53%)
:arrow_up:

...ndroid/telemetry/internal/TelemetryEventHandler.kt
71.09% <0.00%> (+0.78%)
:arrow_up:

.../kotlin/com/datadog/android/v2/core/DatadogCore.kt
85.59% <0.00%> (+0.85%)
:arrow_up:

... and 6 more


New CodeCov Token for CDCGov/phdi

What type of help do you need?

[ ] Question
[ ] New Repo
[ ] Delete Repo
[ ] User Membership (please make sure new members are familiar with the CDC open practices and set up their profile with name and org info to help people collaborate with them)
[x] Other

I was told I could request a Codecov access token here. Our repo is https://github.com/CDCgov/phdi. We already have a CODECOV_TOKEN secret but it must no longer be valid. I get error - 2024-12-10 19:55:52,761 -- Upload failed: {"message":"Repository not found"} when I try to reimplement Codecov in our Github actions.

Hmm, I thought I passed over a token before, and when logging into codecov, I see the following:

I'll send a one-time link with the token to your CDC email.

That fixed it, thank you!

Oh, just to add a note, I don't think there's a time expiration on that token (at least I don't see anywhere where it says it expires). This might be a one-time event from Codecov.


Fragmented execute -vvv output (output.txt is correct though)

I'm not sure why but execute -vvv out for following test's out is fragmented. Content of /var/tmp/tmt/run-058/plans/execute/data/repro/output.txt is correct.
require:
- javapackages-bootstrap
test: $(rpm -ql javapackages-bootstrap | grep /xmvn$) --version

tmt run execute -vvv shows:
test: /smoke
                out: Warning: Permanently added '[<IP_ADDRESS>]:10025' (ED25519) to the list of known hosts.
                out:
out: [1m
                out: Re
                out: d
                out: Ha
                out: t XM
                out: vn
....

However plans/execute/data/repro/output.txt contains expected content (cat -A)
^[[1mRed Hat XMvn 3.6.3 (3.6.3)^[[m$
XMvn home: /usr/share/javapackages-bootstrap$
Java version: 11.0.11, vendor: Red Hat, Inc., runtime: /usr/lib/jvm/java-11-openjdk-<IP_ADDRESS>.9-5.fc34.x86_64$
Default locale: en_US, platform encoding: UTF-8$
OS name: "linux", version: "5.11.12-300.fc34.x86_64", arch: "amd64", family: "unix"$

This might be caused by the change which fixed the blocked ssh connections:
https://github.com/psss/tmt/blob/10d87087fcb5c3f8c4c3e20d2bf222c0adfd3e41/tmt/utils.py#L333
If the output is slow the output check will make another loop. Probably one more reason to use #763 to handle the output?

@lachmanfrantisek @psss I think so, that these threads could fix this issue theoretically. But there were no reason to continue with this thread model of processing output of commands as there were bigger risk to cause hidden some issue. But code will be much cleaner then and probably fix many of out of sync issues.


docs: add information about pii classification feature

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

Comparison is base (2bc13d9) 89.53% compared to head (a55590d) 89.53%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1517   +/-   ##
========================================
  Coverage    89.53%   89.53%
========================================
  Files          194      194
  Lines         6346     6346
========================================
  Hits          5682     5682
  Misses         664      664

Flag
Coverage Δ

py3.8-ubuntu-22.04-pandas
89.53% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Vectorize ELBO computation over num particles

Resolves #791

looks good! do you have any basic speed comparisons, at least on the test examples?

@jpchen - This will be many orders of magnitude faster than sequentially sampling traces num particles times.
e.g. for test_conjugate_gradients.py 103 m.s. for 10k particles when they are parallelized vs. more than 2 mins otherwise.
I intend to do some comparisons w.r.t. hand parallelization which is what we should be comparing against. I did not observe any significant difference for most models, but there are cases when we end up adding up to 20% more time as compared to hand parallelization. It is still very fast though (all in milliseconds vs seconds or mins).

question: i guess as the code stands we expect the user to know when auto-vectorization will break (e.g. in cases with stochastic control flow etc.)?

Yeah, we'd have to do a bit of static analysis for automatic validation

question: i guess as the code stands we expect the user to know when auto-vectorization will break (e.g. in cases with stochastic control flow etc.)?

That's true. I think if an iarange does not specify a size arg or if the size arg is not consistent with the sample site's observed batch shape, we should throw an error in broadcast poutine itself (by having some kind of a strict mode under which this operates). That will ensure that we are are not doing something odd or silently failing in those situations.

The failure is due to the observations library unable to download MNIST, and is unrelated. I am fine with merging this PR as is, once tests pass. Will address the issue of slowness in .expand, and adding more warnings separately.


[milvus-4.1.18] Adding support for ScaNN index

What this PR does / why we need it:
In order to ensure proper coordination and support for Scann index we need this fix. A new Scann index is essential for enhancing performance, scalability, customisation and competitiveness which is missing in older version of attu.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [mychartname])
[ ] PR only contains changes for one chart

/assign @zwd1208 kindly validate the PR.

@Rachit-Chaudhary11 Thank you for providing this patch! Before we cloud merge it, please sign-off your commit https://github.com/zilliztech/milvus-helm/pull/61/checks?check_run_id=21896425998.

@Rachit-Chaudhary11 Thank you for providing this patch! Before we cloud merge it, please sign-off your commit https://github.com/zilliztech/milvus-helm/pull/61/checks?check_run_id=21896425998.

@haorenfsa I have added the sign-off to the commit. If there's anything else you need from me, please let me know.

Hi, @Rachit-Chaudhary11 The DCO still not pass

please follow the instruction to make sure the DCO is pass.
https://github.com/zilliztech/milvus-helm/pull/61/checks?check_run_id=21967637385

@haorenfsa apologies. I have created another PR - https://github.com/zilliztech/milvus-helm/pull/66. Closing this.


Check for untyped URIs extended

Signed-off-by: mereolog<EMAIL_ADDRESS>Description
This extends the check for untyped URIs on those that occur in triples in the property or subject positions.
Note that this shouldn't be merged until #1707 is fixed - otherwise we will get errors in PROD.
Fixes: #1706
Checklist:

[X] I'm familiar with the FIBO developer quide. My contribution meets all the requirements described there.
[X] My contribution follows the principles of best practices for FIBO.
[X] My changes have been reconciled with latest master and no merge conflicts remain.
[X] This PR is related to exactly one issue. The issue is referenced by using a GitHub keyword such as "fixes", "closes", or "resolves".
[X] Hygiene tests have been applied by a PR with "(WIP)" in title.
[X] The issue has been tested locally using a reasoner (for ontology changes).

@merelog One of the recent tests added missed the prefix for XML Schema Datatypes - you might take a look at that and correct it either in that issue resolution in an additional pull request, or in this one.

#1707 is fixed and merged so this branch needs to pull in latest master to avoid its own hygiene failures since it's looking at the unfixed triples


UX improvements for creating new leave request

As per customer request:

It would be really handy if after selecting start date  in
calendar, calendar for end date would jump to same month

The point in description is going to be true only if START date is later then END one.

I think the request means the following:
If it's April currently and I want to add a leave request for October I select e.g. 10th of October as a start date and the end date gets set to 10th of October automatically so that I don't have to set the end date to October manually.

Hi @jekuno,
This exactly how the feature was implemented recently.

On 4 Apr 2017, at 23:58, jekuno<EMAIL_ADDRESS>wrote:
I think the request means the following:
If it's April currently and I want to add a leave request for October I select e.g. 10th of October as a start date and the end date gets set to 10th of October automatically so that I don't have to set the end date to October manually.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.

All right! I thought you meant to reject the issue. ;)


Wrong value of config('auth.defaults.guard') when you're logged with another guard.

Laravel Version: 5.5.21
PHP Version: 7.1.11
Database Driver & Version:  mysql 7.1.11

Description:
When I'm logged with another guard than the default guard and I try to get the default guard value with config('auth.defaults.guard') it returns the current guard value (the guard which I'm currently logged in) rather than the real default guard value.
For example, if I have two guards 'web' and 'admin' and the default guard is 'web' and I'm logged with 'admin' when I make config('auth.defaults.guard') it returns 'admin'.
Steps To Reproduce:
Try to get the default guard value with config function when you're logged with another guard.

Yes laravel switched the default driver used to the current used guard.

I think that when I call the config function it returns the config value.

Hello, I don't much about guards but I have followed this steps for a web and admin guards and it works perfectly for me, maybe you are missing the configuration of some Laravel Files, checkout this video : https://www.youtube.com/watch?v=iKRLrJXNN4M

How Do I suppose to get the default guard if I'm logged with another guard?

This is expected behavior. Laravel will switch the default guard to the current one used.

I don't see how this is expected behaviour. It's confusing.
Default should be default auth guard as per config file, no matter what is the current ACTIVE guard on the route.

Definitely going to need a better explanation here. Honestly baffled it's been closed.

"Default" does not mean "current,"
If you manually set a config value, that's the value expect to be returned.
This very predictably leads to the obvious question: "How do I find the actual default value?"


Publish on PyPI

[x] Built on top of DFTracer changes (most updated branch)
[x] Tested CI via testpypi.org on Polaris
[x] Tested Unet3D and Cosmoflow on Polaris via testpypi.org

Also, why there is no CI-tests running?

Also, why there is no CI-tests running?

CI will work when this branch is merged into dependency/dftracer -- because #208 is targeted to be merged on main.

Also, why there is no CI-tests running?

CI will work when this branch is merged into dependency/dftracer -- because #208 is targeted to be merged on main.

It is better to trigger this when new commit is pushed. Because we have to know whether it pass the tests or not before merging, right?

Also, why there is no CI-tests running?

CI will work when this branch is merged into dependency/dftracer -- because #208 is targeted to be merged on main.

It is better to trigger this when new commit is pushed. Because we have to know whether it pass the tests or not before merging, right?

Current CI script (below) works only when the PR is targeted towards main or dev -- this is usually to avoid wasting CI time when the PR is for a side branch.
on:
  pull_request:
    branches: [main, dev]


Regarding 'Full Screen' option in style guide

Wondering if the text for this should say 'Toggle Sidebar' rather than 'Full Screen' to make it clearer?
'Full Screen' makes it sound like it's another size option, like 'Large Desktop', but it isn't really. E.g. If I have 'Small Desktop' selected and then click 'Full Screen', the device width is still set at 'Small Desktop' but it just hides the sidebar.

@paulgoodfield Now added in version 3.1.3. Commit found here: https://github.com/towardstudio/websitedocumentation/commit/934b9d0dd08a5362f06b33ea84cc8ed9f0f5e50d
To change in existing projects, you'll need to manually do the updates in the commit


fix typo (was uknown)

Correct a minor typo in scanner.c.

This seems to have gotten fixed at some point.
Thanks for reporting it. Closing.


Anyone want to help with a friendly fork of this project?

I see there is a lot of backlog that isn't handled, and I think it would be nice to keep this tool alive.
I've mostly moved on to ripgrep (and will often use ack when speed is not critical) but there are places I prefer to use ag mostly because it works due a balance of the features, speed, and portability offered.  I still use ag on AIX, thought rust support in AIX landed in 1.67.0, so I might switch, but regardless, there is a good amount of work that can saved here.
I'd like to make a fork of this project, but I'd also like to clearly state that I don't want to "take it over" or really do much new development on it ... so what I propose is a new project that will operate as a friendly fork, with the following goals - roughly in order of importance - and setting expectations up-front:

No new features - The fork would focus only on fixes and improving compatibility and portability, at least to start.  Everything it touches should be able to be easily merged back to this upstream.  No major refactoring!

Automate testing - First focus must be on getting some kind of CI/CD infrastructure going that supports testing with the included tests.

Catch-up - Merging any pending PR's that are a) clearly correct, b) solve real problems, and c) pass tests.

Bug-fixes - Tracking down and (hopefully fixing) the open issues that have a clearly reproducible tests (i.e. https://github.com/ggreer/the_silver_searcher/issues/1243) which could be git bitsect'ed.

Consolidation - Merging in vendor patches (Debian, Red Hat, Gentoo, Ubuntu, maybe others?) or other forks fixes, so these distributions and other projects won't need to do local patching (but only where such patches are clearly appropriate and low-impact.)

Bus factor >1 - I would guess doing this as a GitHub organization with a few others is best, so if I get hit by a bus or give up, the project wouldn't die or have to move again.

I know GitLab CI/CD very well, but I wouldn't want to run my own runners for this potential new project, and the free tier is limited.  It would quality to get a free Ultimate upgrade, so perhaps it's something to look into, but GitHub actions might make more sense.  I'd prefer GitLab but would like to keep this fork on GitHub since that's where this upstream project is.
Anyone have suggestions, maybe want to help? @extrowerk @JFLarvoire @aswild ?

Hey Jeffery, I'm honored to be apparently prominent enough to be included here :)
I think you've got noble goals and a good plan laid out here; it'd be nice to see ag live on as a usable portable tool. I hacked at ag a bunch because it was a tool I used all the time and it gave me a ton of experience learning autoconf/automake and working on a C project of nontrivial but manageable size.
That said, I've pretty much entirely migrated to ripgrep (except in cygwin) and don't expect to have the time or motivation to continue actively maintaining an ag fork. I may be available to help with or answer questions about particulars, if needed.
If you (or anyone, really) would like to adopt or port anything from my fork, you're certainly welcome to it. Consider all my commits in https://github.com/aswild/the_silver_searcher to be released under the Apache 2.0 license. Attribution, e.g. by preserving git author info when cherry-picking or a line in changelogs, would be appreciated but not required.
Cheers,
Allen

#1257 is now a grave issue for Debian and every derivative Bug#999962 silversearcher-ag: depends on obsolete pcre3 library.  I'm thankful to @aswild for solving it in his fork :smile:
@johnsonjh, @JFLarvoire, do you think you'll pick up the baton in the next six months, or should we all start writing ripgrep compatibility wrappers, while falling back to ACK on platforms that don't have great Rust support?

@sten0 Hi there!
In short, yes, I do think so, especially in light of this.  I'll see what plan I can come up with in, say, another week or so.

@sten0 Hi there!

Hi @johnsonjh! :smile:

In short, yes, I do think so, especially in light of this.

Thank you, it's sincerely appreciated.

I'll see what plan I can come up with in, say, another week or so.

How did it go?

Sorry, been busy..
I see the libpcre3 was removed from Debian sid, so, I guess we better get going soon, huh?  I'm wrapping up this project now, thankfully, better late than never, so I can visit this soon.

@hholst80 I haven't yet, but it's still on my radar to do.  Once we get a new version of DPS8M released, I should have some time to look at it seriously.


[FIX] fix broken link

Description
Fix broken link
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[ ] Attached issue to pull request
[ ] Changelog entry
[ ] Type annotations are present
[ ] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)
[ ] Sufficient test cases (reproduces the bug/tests the requested feature)
[ ] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches master in f7f689b97a13c180150904a05f2cf7f80fa4ed13

Processing #6507.

Processing #6506.

Processing #6505.


Move all rewrite handling logic into the rewrites module

Description
This implements the same changes to rewrites as have been implemented for redirects in #348. As suggested, I merged the identical code so that it is only present in the redirects module now.
There are some changes to the handler: request fields can no longer be borrowed because we are now modifying the request. While I meant to implement these changes anyway, this unfortunately introduces a merge conflict with #351.
There are also some behavior changes due to the fact that destination is always being processed as a URL now and not merely in the redirect scenario:

Previously, internal rewrites kept the query string unchanged. Now the query string from the destination overrides the original query string if present. This actually matches the behavior of nginx rewrites. The impact should be limited because the only place where query strings currently matter is directory listings, and existing configurations shouldn’t have destinations with query strings anyway.
It is now possible to do an internal rewrite to a full URL which includes a different host name. IMHO that’s a useful feature because it allows defining virtual hosts that are normally only reachable via rewrite – so you can have a subdirectory mapped to a different root (similar to the nginx alias directive). Configuration example:

[advanced]

[[advanced.rewrites]]
source = "/test/{**}"
destination = "http://internal.local/test/$1"

[[advanced.virtual-hosts]]
host = "internal.local"
root = "/usr/srv/test"

This might be worth documenting. There are caveats however. First, it’s a good idea to rewrite to the same path and merely change the host name like I’ve done above, otherwise redirects when accessing directories will be a mess (this is generally an issue with rewrites). Second, someone who knows the name of the internal virtual host will still be able to access it by mapping this host name to the server’s IP address explicitly. So one should not expect anything under that host’s root to be hidden just because rewrites only map to some of the files there.
Related Issue
This is another step towards fixing #342, also fixes the rest of #347.
How Has This Been Tested?
Integration tests pass, some additional unit tests have been added as well. Functional testing on Linux looks fine.

JFYI @palant I will check the PR in the next hours but maybe you want to join us in Discord to also coordinate better reviews, tasks, time, etc.


feat: make channels optional

Description
Part of https://github.com/asyncapi/spec/issues/661.
This PR removes the required prefix from the channels field at root level of the document.
Related issue(s)
https://github.com/asyncapi/spec/issues/661

cc @derberg @fmvilas as you both are code owners.

Is this not a breaking change? 🤔

@jonaslagoni

Is this not a breaking change? 🤔

If we switch optional to required then yes, but switching required -> optional won't break any AsyncAPI document.

Is this not a breaking change? 🤔

It will not be backward compatible but it will be forward compatible. I think we should also make it clear in the spec, that we aim for forward compatibility.

Looks good to me. Let's wait for @derberg.

see https://github.com/asyncapi/spec/issues/688

I thought twice about it and think this one is a breaking change and should be postponed to v3.

I'll have a look to see if there is anything else that needs to be pulled out of 2.3 release branches

As per https://github.com/asyncapi/spec-json-schemas/pull/146#issuecomment-1016290582, we won't include this change in 2.3.0 but rather in 3.0.0.

/dnm


Scan now skips ignored fields when setting the values of a struct field

In the case a field of a struct was explicitly ignored by gorm (i.e. using the attribute gorm:"-"), scan would attempt to populate the ignored field with data coming from the next field.
In some cases this would lead to a crash due to incompatible data types while in other cases this would populate the field with the wrong data.
Possibly related to issue #1826

Codecov Report

Merging #2685 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #2685      +/-   ##
=========================================
+ Coverage   79.49%   79.5%   +0.01%
=========================================
  Files          24      24
  Lines        3458    3460       +2
=========================================
+ Hits         2749    2751       +2
  Misses        609     609
  Partials      100     100

Impacted Files
Coverage Δ

scope.go
87.03% <100%> (+0.03%)
: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 81c17a7...762298b. Read the comment docs.


Document Redshift SSL

connect_args={'sslmode': 'prefer'} or verify-ca , verify-full
User can send this through connect args . But thiught we should capture as additional param and add to connect args

If the parameter sslmode is set to verify-ca, redshift connector will verify that the server is trustworthy by checking the certificate chain up to a trusted certificate authority (CA).
If sslmode is set to verify-full, redshift-connector will also verify that the server host name matches its certificate. The SSL connection will fail if the server certificate cannot be verified. verify-full is recommended in most security-sensitive environments.
In verify-full mode, the cn (Common Name) attribute of the certificate is matched against the host name. If the cn attribute starts with an asterisk (*), it will be treated as a wildcard, and will match all characters except a dot (.). This means the certificate will not match subdomains. If the connection is made using an IP address instead of a host name, the IP address will be matched (without doing any DNS lookups).
if a root CA file exists, the behavior of sslmode=require will be the same as that of verify-ca, meaning the server certificate is validated against the CA. Relying on this behavior is discouraged, and applications that need certificate validation should always use verify-ca or verify-full
Lets document this in a section of redshift cc @ShilpaVernekar

cc @codingwithabhi, please add this when working on the redshift docs 🙏


Sidebar Support?

Thanks for contributing this helpful component. When I attempt to use with sidebar st.sidebar.Datalist, I get an exception stDatalist() is not a valid Streamlit command. Is sidebar supported in some way?

Hi. You can add components to the sidebar by using with, discussed in this Streamlit Forum discussion and this blogpost entry.
Just tried it and it should work by writing:
with st.sidebar:
    value = stDatalist('Title', data)

Hope that helps!


Fix #13474 by partially reverting #12883

What does the pull request do?
Fixes issue #13474, where Tapped events are not generated by ListBoxItems or their children from Touch based pointer events.
What is the current behavior?
Currently when a ListBoxItem (or child control) is tapped with a touch device, the ListBoxItem re-raises the PointerReleased event on the owner ListBox, overwriting the event source with the ListBox.  The Tapped gesture recognizer is looking for PointerPressed and PointerReleased events with matching sources, so this change of source prevents Tapped events from being raised.
What is the updated/expected behavior with this PR?
ListBoxItem no longer re-raises the event from Touch input, allowing it to bubble up to the ListBox (and Tapped gesture recogniser) normally.  This re-enables Tapped events from Touch input, and also prevents the probably unintended consequence of the same event arguments bubbling up twice.
How was the solution implemented (if it's not obvious)?
Partially reverts #12883, removing the changes to ListBoxItem where it re-raises the PointerReleased event on the owner ListBox.  I couldn't figure out why the event needed to be raised rather than bubbling up, as nothing I could find to the AutoCompleteBox checked what the source of the event was or whether it was handled or not.  I couldn't reproduce #12848 after my changes, so I don't think I've re-broken the AutoCompleteBox focus issues.
In the course of setting up the failing unit test, I ran into an issue where different parts of the framework make different assumptions about Touch input.  Specifically the TouchTestHelper generates Touch events without assigning mouse buttons to them, but every other part of the framework seems to assume that Pointer events are by default LeftMouseButton events.  This means that using the TouchTestHelper to unit test the change didn't work, because Gestures is checking if the PointerReleasedEvent has InitialButtonPress == LeftMouseButton even for touch input, and thus wouldn't generate Tapped events.  I elected to manually generate the Pressed and Released events in the unit test rather than touch other parts of the framework to avoid unintended consequences.  I also wasn't sure where the unit test should go, but elected for with the ListBox unit tests rather than Gestures or TouchDevice as it's very specific to ListBoxItem.
Checklist

[X ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/avalonia-docs with user documentation

Breaking changes
No breaking changes I'm aware of.
Fixed issues
Fixes #13474

@dotnet-policy-service agree company="Silmetra Ltd"

On non-desktop touch devices(android, ios), tapping an item in an autocomplete box doesn't close the list and focus on the box, with this PR. Could you confirm this works on android?

On non-desktop touch devices(android, ios), tapping an item in an autocomplete box doesn't close the list and focus on the box, with this PR. Could you confirm this works on android?

You're right, it doesn't close the drop down on Android.  I'll spend some time today trying to figure out why, and how to fix it without breaking tapped events.

Putting the Event re-raise back and testing on Android more, I'm getting some strange results.

On the emulator, typing into the box and selecting something closes the popup and focuses the textbox as expected.
On a physical device with bluetooth keyboard, the same result.
On a physical device with no keyboard, the popup closes, but focus is usually lost.  But not always.

On another note, connecting a Bluetooth keyboard to the physical device while the soft keyboard is showing is causing the ControlCatalog to throw System.ArgumentException: 'Attempt to call InvalidateArrange on wrong LayoutManager.'
That's a different issue obviously.

OK, I've figured out the particular oversight that caused the problems on Android.  I neglected to consider that marking the event handled would (of course) prevent it bubbling up to the ListBox.  By changing the ListBoxItem change to only mark the event handled from a mouse it seems to have fixed the AutoCompleteBox, at least as far as I can tell, and Tapped events still occur.  I'm running the Unit Tests now to make sure I haven't broken anything else.

New version has the ListBoxItem not mark the PointerReleased event as handled if it came from a non-mouse device.  It seems to work, but I'm leery of handling mouse and touch differently, which is what started me working on this in the first place.  An alternative is simply to have the ListBoxItem not mark PointerReleased as handled full stop, but I don't know the wider implications of that.

Third version.  Always sets handled, but backs the source up before raising the event again, and restores the original source after.  Allows Tapped events to be generated, and seems to not break AutoCompleteBox.

requesting review @emmauss

Any chance of this getting backported?  Or is it slated for an imminent release?


Expand the path of the --file argument

brew bundle --file=~/Brewfile didn't work because the ~ was not expanded.
Now we expand the whole path so that people can pass more idiomatic paths to
brew bundle.
About tests: I couldn't find any tests for the functions in utils.rb. I don't know what this project's policy on tests for the functions in that file is (maybe I missed something!) but if you'd like, I can add unit tests for at least Bundle.brewfile.

Thanks!


-bash: catalyst: command not found

Dear Catalyst Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: MacOS
Python Version: Python 3.6.8 :: Anaconda, Inc.
How did you install Catalyst: conda

Now that you know a little about me, let me tell you about the issue I am having:
Description of Issue
After installed catalyst by anaconda, and conda activate catalyst，
when I try to check catalyst --version, the terminal told me:
-bash: catalyst: command not found
I don't know why.  Please help me.
There's no catalyst file in <anaconda3/envs/catalyst/bin>.
...
Sincerely,
$ whoami

Not on the catalyst team, but maybe try:
source activate catalyst
this should change the -bash to (catalyst)
then give this a shot:
catalyst --version
hope it helps!

@milanhpatel
Thank you for your help.
But I have try source activate catalyst and the terminal will show that I am in the env 'catalyst',  and still throw the error -bash: catalyst: command not found.
I found that there's no file name catalyst  in  <anaconda3/envs/catalyst/bin>.

the same issue, but catalyst/bin is in its place, which unfortunately does not change the result:
command not found: catalyst
Also installed via conda

I had issues getting catalyst properly in PATH with my conda installs but had luck with a pip install. My coworker had luck with pipenv, while both of us had issues getting the conda install to properly function (different issues however).
This is not necessarily a solution but I would attempt a different install route if you have only tried conda so far.


Update dev dependencies

bors r+

Codecov Report

Merging #146 (08e49a1) into prīmum (91f779e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           prīmum     #146   +/-   ##
=======================================
  Coverage   78.00%   78.00%
=======================================
  Files          92       92
  Lines        2950     2950
=======================================
  Hits         2301     2301
  Misses        649      649

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 91f779e...08e49a1. Read the comment docs.


Methods defined in openxr_platform.h  such as XrGraphicsRequirementsOpenGLESKHR have no impemention in openxr_loader.so

Hi  rpavlik:
I find methods defined in openxr_platform.h  such as XrGraphicsRequirementsOpenGLESKHR have no implementationin openxr_loader.so. when i call these method,i encount a Compile  error msg: undefined reference to 'xrGetOpenGLESGraphicsRequirementsKHR'.
should i implement these methods in loader_core.cpp or  somewhere.
thanks very much

Extension functions are only available for lookup through xrGetInstanceProcAddr at runtime.

Extension functions are only available for lookup through xrGetInstanceProcAddr at runtime.
I get that ,thanks very much

So this is by design, that extension functions are not exported from the loader. However, it is confusing since the prototypes are available by default from the header - we'll address this part of it shortly.


Where could I find a lang spec to highlight my code in editors like gedit ?

I assume you've already done such a job, could you tell me where to find it ?
Anyway, this is a great stuff

https://github.com/wavded/gedit-coffeescript

thx !


ui: Allow editing VM and template settings

Description
This PR fixes the issue seen when attempting to update template / VM settings where in users aren't unable to change the existing value.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[X] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[X] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?
Post Fix:

@blueorangutan ui

@Pearl1594 a Jenkins job has been kicked to build UI QA env. I'll keep you posted as I make progress.

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6262 (SL-JID-1379)


Portainer Setup with DFP

Description

Hi Viktor,
First of all thank you for this awesome project!
I would like your help with some setup or configuration I might be doing wrong. So I am trying to use Portainer with DFP (see command below) and even though I define ports that it can be accessible on, I can still get to it without the port in the url. For example
https://portainer.companydomain.com works as well as https://portainer.companydomain.com:7442
Is there a way I can make it to not respond to https://portainer.companydomain.com. I only want
https://portainer.companydomain.com:7442 to be accessible since we have that port locked down to our corporate IP.
docker service create \
    --name portainer --publish 7442:9000 \
    --mount type=bind,src=/var/run/docker.sock,dst=/var/run/docker.sock \
    --mount type=bind,src=/home/centos/portainer,dst=/data \
    --network app --network proxy --label com.df.notify=true \
    --label com.df.distribute=true --label com.df.serviceDomain=portainer.companydomain.com \
    --label com.df.servicePath=/ --label com.df.port=9000 --label com.df.xForwardedProto=true --label com.df.redirectWhenHttpProto=true \
    --constraint 'node.hostname==ip-192-168-24-172.eu-west-1.compute.internal' \
    portainer/portainer \
    -H unix:///var/run/docker.sock

I read your comment on another issue to use subdomain instead of paths and that is why I am using serviceDomain as an label.
Steps to reproduce the issue:

Start portainer with the docker command above

Describe the results you received:
This makes both https://portainer.companydomain.com  and https://portainer.companydomain.com:7442 accessible
Describe the results you expected:
I want it to be restricted or accessible only at https://portainer.companydomain.com:7442
Additional information you deem important (e.g. issue happens only occasionally):
Additional environment details (AWS, VirtualBox, physical, etc.):
AWS - EC2 - centos-release-7-4.1708.el7.centos.x86_64
Docker Version
Client:
 Version:	17.12.0-ce
 API version:	1.35
 Go version:	go1.9.2
 Git commit:	c97c6d6
 Built:	Wed Dec 27 20:10:14 2017
 OS/Arch:	linux/amd64

Server:
 Engine:
  Version:	17.12.0-ce
  API version:	1.35 (minimum version 1.12)
  Go version:	go1.9.2
  Git commit:	c97c6d6
  Built:	Wed Dec 27 20:12:46 2017
  OS/Arch:	linux/amd64
  Experimental:	false

Let me know if I can provide any additional information.
Thank you!

FYI here is my portainer config:
# Rolls out Portainer
# -- Make sure Docker Flow Porxy exists and the network "proxy" also exists
# -- (Optional) Store Data on CINDER Remote Storage, make sure the plugin is installed.
version: '3'

services:
  portainer:
    image: portainer/portainer:1.16.1
    networks:
      - net
      - proxy
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock
      - portainer-data:/data
    deploy:
      mode: replicated
      replicas: 1
      placement:
        constraints:
          - node.role == manager
      labels:
        - com.df.notify=true
        - com.df.distribute=true
        - com.df.port=9000
        - com.df.serviceDomain=cloud.example.com
        - com.df.httpsOnly=true
        - com.df.httpsRedirectCode=301
        - com.df.letsencrypt.host=cloud.example.com
        -<EMAIL_ADDRESS>networks:
  net:
  proxy:
    external: true
volumes:
  portainer-data:

@remy-tiitre  can you elaborate a bit more on http-keep-alive
what might happen if one does not set it or what problem does this error covers?

Well, I took this information from https://portainer.readthedocs.io/en/stable/faq.html. Theres an HAProxy example and remark: Note: http-keep-alive must be set for both frontend and backend.

It says
Note: http-keep-alive must be set for both frontend and backend
The question is where is - com.df.connectionMode=http-keep-alive applied frontend, backend or both

Environment variable CONNECTION_MODE is applied to the frontend. The service-specific parameter connectionMode (or label com.df.connectionMode) is applied to the backend.
@remy-tiitre Did you set both?

Thank you for the info. A bit off topic if I set CONNECTION_MODE it will be applied to all frontends right? Is it planned to have it also on service level@

Frontend settings cannot be on service level since a single frontend (single port) is used for multiple backends (services). That being said, it should be enough to set connection mode only on the backend. It's like an overwrite of frontend settings.

I didnt set ENV variable so backend only. I think it defaults to http-keep-alive on frontend.

The project moved to https://github.com/docker-flow/docker-flow-proxy. Please recreate the issue there if it's still valid.


can't import anything from package

importing from the package (its init.py) doesn't work.
I guess it misses an from .dispatchonvalue import *

OK, I've change it so all of these work:
import dispatchonvalue
from dispatchonvalue import *
Is this working the way you want it to?

you need from .dispatchonvalue import * so that it doesn't try to import the package itself

or just put the whole code in init.py


Feature request: tooltip delay

As it is, tooltips show up (or begin to show up) immediately on mouse over. I propose adding a delay attribute to the Tooltip component that will delay showing by that many milliseconds.

hello, @rluders @multiwebinc, I  would love to solve this issue.

@robbyevans Are you still able to resolve this?

@tulup-conner yes I am available

I'd also love to see this solved with a delay prop or something like trigger="hover:delay-500". Without the ability to configure a delay, this component is useless for my use cases at least. Thnx.


[BUG] User's bot shows up in Old Bots with No buttons. Bot does not show as repaid on Etherscan

Bot type: Curve Healthy Bot
Bot Transaction:  https://etherscan.io/tx/0xfae2916fb18b30fee7503457bd7d94fd1248a507bfaa80ba05623efb03c12b5a
Bot:

@webelf101 this seems related to the CW upgrade on Friday?

Fixed with: https://github.com/VolumeFi/palomabot/pull/462


[+] CMake support

Hi DialogSemi team,
SmartSnippets Studio is very convenient to use but requires a lot of computer resources and doesn't allow to build a firmware with GitHub actions, Bitbucket pipelines, etc in a convenient way.
There is one quite cool docker container for CI builds but it requires you to download Studio every time when you want to build your firmware. It takes a lot of runner's minutes and may lead to going out of CI/CD budget.
Because a typical DA1469X project has complex organization (forced includes, python scripts for setup and downloading firmware and more) it's quite difficult to set up a make/CMake project by yourself. This example may be helpful for new users.

Hi Egor,
Thanks a lot for sharing. This is very interesting. This is actually something we had a look into. There's been some work for our SDK6_example repository on the same topic we're just about to share.
Our intention was to port the work we've done on SDK6_example to SDK10_example. It's a little lower priority for us because we already have the the project build automated based on Smartsnippets studio.
We will most probably want to reuse the work you've done but at the same time align that with the structure we use for our other repository. So please bear with us for a moment while we figure out how we address that.

Hi blauret,
Thank you for response. Let me know if I can help with something else.


load_all_datasets.sh now expects the path to dplace-data as argument

closes #452

Codecov Report

Merging #453 into master will decrease coverage by <.01%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master     #453      +/-   ##
==========================================
- Coverage   93.92%   93.92%   -0.01%
==========================================
  Files          24       24
  Lines        1565     1563       -2
==========================================
- Hits         1470     1468       -2
  Misses         95       95

Impacted Files
Coverage Δ

dplace_app/load.py
96.8% <100%> (-0.06%)
:x:

dplace_app/loader/society.py
74.5% <80%> (ø)
:white_check_mark:

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 1e1eba2...6e15e44. Read the comment docs.


web: don't highlight endpoint link after clicking

fixes #2380

this doesn't seem right. there are other elements of the ui that need this

i would probably leave the existing css as-is, and add a rule like: this so that
a.resourceInfo-value {
user-select: auto;
}


Problems with vue

Some parts of vue depend on each other, greenkeeper always tries to bump them separately which always fails, see https://github.com/WeakAuras/WeakAuras-Companion/issues/21

i'm not a vue user, so i don't know its specific details, but would it be helpful to define the versions that need to change together, so that greenkeeper sends them in the same PR? if so, it might be worth sending a PR to https://github.com/greenkeeperio/monorepo-definitions

Oh interesting, will take a look, thanks

I'm closing this because https://github.com/greenkeeperio/monorepo-definitions/pull/27 is merged now.
Thanks to @Stanzilla


Update changelog for version 0.13.0

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6406

Relevant Lines:
7114

💛  - Coveralls


Update how-to-run-example-android.md

Fixing some bugs I encountered during running the android makes.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1906. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.

INFO: You can read if there are misspelled characters at our misspelling check report. Please read http://ci.nnstreamer.ai/nntrainer/ci/repo-workers/pr-checker/1906-202205130932310.51682996749878-972159117dc2e2ce68fb919e598c4b3eab59055e/report/misspelling_result.txt.

To contributor, We have used 'Signed-off-by:' notation by default to handle the license issues, that result from contributors. Note that 'Is there a Signed-off-by line?' is important because lawyers tell us we must have to it to cleanly maintain the open-source license issues even though it has nothing to do with the code itself.

I'll fix and resubmit.

Thanks for the fix! It is much better if you follow the "https://github.com/nnstreamer/nntrainer/blob/main/docs/contributing.md" to fix the CI Fails. It says you missed the "Signed-off-by"

Yes I fixed it in #1907, that is why I made another PR.
I will close this one.


MWPW-140535 allow for option to floating cta to stay closed

Allow for option to not auto-open the floating cta toolbox
Resolves: MWPW-140535
To test this, you'll need to override the object you get in the utils.js file on line 1875. The floatingCTAData?.[device] must contain the values 'multifunction-button closed'
Test URLs:

Before: https://main--express--adobecom.hlx.page/express/create/flyer?experiment=ccx0098%2Fchallenger-1
After: https://MWPW-140535-closed-option-floating-cta--express--adobecom.hlx.page/express/create/flyer?experiment=ccx0098%2Fchallenger-1

Codecov Report
Attention: 18 lines in your changes are missing coverage. Please review.

Comparison is base (b54528c) 31.75% compared to head (974707c) 31.64%.

Files
Patch %
Lines

...locks/multifunction-button/multifunction-button.js
0.00%
9 Missing :warning:

express/blocks/shared/floating-cta.js
14.28%
6 Missing :warning:

express/scripts/utils.js
0.00%
3 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #544      +/-   ##
==========================================
- Coverage   31.75%   31.64%   -0.12%
==========================================
  Files         124      125       +1
  Lines       23810    23932     +122
==========================================
+ Hits         7561     7573      +12
- Misses      16249    16359     +110

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

Should we add a cursor: pointer at the option icon for consistency in the CTA? (if not, ignore my comment)

I think it's a good idea, but it's new functionality. Tonight we just want to get this option rolled out so we can start a test on production


sha1sum: unrecognized option: status

I updated my Dockerfile to use the <IP_ADDRESS> tag. It's working great, but I ran into an issue when using streamsets stagelibs install some individual stages (I needed to do this since the Dockerfile has been updated to install core tarball instead of all)
FROM streamsets/datacollector:<IP_ADDRESS>
RUN  /opt/streamsets-datacollector-<IP_ADDRESS>/bin/streamsets stagelibs -install=$STAGELIBS

sha1sum: unrecognized option: status
BusyBox v1.24.2 (2016-05-31 09:35:34 GMT) multi-call binary.

Usage: sha1sum [-c[sw]] [FILE]...

Print or check SHA1 checksums

        -c      Check sums against list in FILEs
        -s      Don't output anything, status code shows success
        -w      Warn about improperly formatted checksum lines
Failed! running sha1sum --status -c stage-lib-manifest.properties.sha1 in /tmp/sdc-setup-5

I was able to work around this by editing the file to replace the --status flag with -s:
RUN   sed -i -e 's/run sha1sum --status/run sha1sum -s/g'  /opt/streamsets-datacollector-<IP_ADDRESS>/libexec/_stagelibs

Related JIRA

+1

Fixed, new images available on docker hub.


New Logo

@sr6033 on slack asked me to get someone to design a better logo. I kind of like our current one with the black background but suggestions for improvement or even the redesign of the current logo our welcome.
cc @IamRaviTejaG your thoughts?

I'd suggest a redesign of the logo.

So @sr6033 wants us to get a professional to design our new logo. We would need to pay for that. I think if all of us chip in with a little money we can get this done. Anyone who wants to contribute comment on this thread. Let's also discuss possible designers for the logo over here.
We should also integrate the Mozilla Campus Club logo into ours.
cc @p-society/2019-batch @p-society/2020-batch

How much amount will be needed? And do we really need professional designer, all our college poster were designed by college students only at the time of Advaita.

@sr6033 asked me to get professional help for the logo on slack. But if a student comes up with a really good design I think that would be great.

Is it necessary to integrate mozilla campus club logo with it? (I mean to ask does mozilla expects/asks us to do so?)
Think from my perspective, we are P-Soc, the programming club of IIIT-Bh. P-Soc is our identity. We can say that we partnered with mozilla campus clubs, or ACM Student Chapter, or any other chapters/body we open in future, for perks and benefits. Like these bodies are sort of supporting body to PSoc, right? I want us and others to recognize ourselves as P-Soc, an entity of our own. We are not ACM IIIT Bh, we are not Mozilla IIIT Bh, not primarily, we are P-Soc, right? That's my perspective. I want that we remain primarily as P-Soc. Yes, we should try to get help from as many bodies as possible, ACM, Mozilla, IEEE, etc, but when its done and all, we recognize ourselves as P-Soc. And hence I would go against the idea of integrating mozilla campus club's logo with P-Soc logo.

Good point by @hdidwania and I completely agree with that , even if we are integrating with Mozilla Club , keep it as another part, and lets keep the identity of P-Soc . In the long run ,its important to have unique society of our college. We may be submitting reports to ACM or Mozilla ,that can be formulated easily, but lets keep P-Soc as individual body.

are not Mozilla IIIT Bh, not primarily, we are P-Soc, right?

We kind of are now. But its okay if you dont want the Mozilla logo integrated with ours that was just an idea. I don't think its compulsory for mozilla campus clubs to change their logo.
I'm going to wait for other people to comment on logo ideas and see where it takes us. In the end the decision will be unanimous.

Maybe we can have our own unique p-soc logo along with it at the bottom we can write.
Programming Society IIIT-Bh
A Mozilla Campus Club

Maybe we can have our own unique p-soc logo along with it at the bottom we can write.
Programming Society IIIT-Bh
A Mozilla Campus Club

Yes, we can do this.

@palash25 @arijeetsat @hdidwania We can also contact uttariyo saha(etc or eee branch). He is a professional logo designer. May be he can help us in this.

Yup, lets see if we can get some contacts with other students as well.

Can someone contact him or he is on Github can someone invite him to be a part of the discussion on this thread?

@palash25 @arijeetsat @hdidwania @shivank01
We can contact Soumya Ranjan Bishi bhaiya. He's an alumni and is a professional designer. Advaita's logo (VA) is his design. I'll contact him regarding the P-Society logo. Will be posting updates here.

I can talk with him. I am working with him here in param. But we are a lot busy. I don't think he can find out time.

@sr6033 Please talk to him and try your best to convince him. He is really really good at designing, so we can expect a really good logo design from him.

Whatever decision you all make I am cool with that.

For the new logo , I have talked to Manish Ray of CE , he is very proficient at this, in some days he will be giving us some sample, lets wait for it. 👍

Any updates @arijeetsat ?

Will contact him again, any specific idea you people want in the logo?

It has been already discussed on this thread

Nothing about the design is mentioned, I mean does anybody have any particular design thing in mind like what shape/image should be there, that has to be added?

I don't but one thing you can do is give the guy these two images that have been posted and tell him to take inspiration from those.

Shape of the logo: Something that can fit into a circle of square
Design style : Cartoony/Doodly, Colorful

Can someone throw up any more suggestions for the logo? Maybe I can whip up one..

@p-society/2019-batch @p-society/2020-batch :point_up:

Can someone throw up any more suggestions for the logo? Maybe I can whip up one..

Read the whole thread and discuss your design on this thread :+1:  @Ashugupta01

okay...

I don't see any progress from @Ashugupta01 on this and @sr6033 has already shown us a glimpse of the new logo so I am closing this in favour of that
@Ashugupta01 sorry about this but if you are interested in working with us there will be a lot more opportunities to work with us. Like designing posters for our upcoming sessions or Advaita.
If you are interested in any such work DM me on slack around August :+1:


py2pack failing on demosh

py2pack errors out on demosh:
https://github.com/BuoyantIO/demosh
$ y2pack generate -t opensuse.spec demosh python-demosh.spec
Traceback (most recent call last):
  File "/usr/bin/py2pack", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/usr/lib/python3.11/site-packages/py2pack/__init__.py", line 436, in main
    args.func(args)
  File "/usr/lib/python3.11/site-packages/py2pack/__init__.py", line 301, in generate
    fetch_data(args)
  File "/usr/lib/python3.11/site-packages/py2pack/__init__.py", line 347, in fetch_data
    urls = args.fetched_data['urls']
           ~~~~~~~~~~~~~~~~~^^^^^^^^
KeyError: 'urls'

This is because a wrong usage, the correct way of using fetch is:
$ py2pack generate -t opensuse.spec -f python-demosh.spec demosh

Without the -f, the filename is interpreted as the version parameter and with that py2pack is unable to find anything in pypi.

Oh, sorry, my bad. Must have picked the wrong old command from my bash history... :-(


Integrity Constraint Violation issue

We seem to be getting a lot of these errors popping up in our FeedMe logs
Message	[Forced] Events: CDbCommand failed to execute the SQL statement: SQLSTATE[23000]: Integrity constraint violation: 1062 Duplicate entry 'event/junior-youth-theatre-year-5-en_gb' for key 'craft_elements_i18n_uri_locale_unq_idx'

It does look like the task then successfully completes all of its steps, but then doesn't close itself off and sticks as a 'Failed Task' in Craft.

See attached log from the last couple of days for your reference: feedme.txt
Many thanks

Are you able to send a Help request? Really need a bit more information if possible!

Oh yes, sorry. I forgot you introduced that in v2! I've just pushed one through for you.


More than one frames as decoder input?

I'm trying to make a modified version of Tacotron, in which the decoder receive more than one previous-frames to generate r frames in output. Something like changing "next_inputs" function in helpers.py. for TacoTrainingHelper to:
next_inputs = self._targets[:, time - hp.receptive_field:time, :]
which other parts of the model should be changed?
Is it possible to keep the rest of the decoder as much as possible?

Sorry but I have no idea of how to do it. I appreciate you to show me any sample code for keeping the track of previous decoder steps (more than one time steps).


Added setAttachment and getAttachment to WebSocket interface

Description
This allows a WebSocket connection object to have any kind of data associated.
You can now store information of each connection/user and access it when onMessage or onClose is fired.
Usage example
public void onOpen(WebSocket conn, ClientHandshake handshake) {
    User user = new User(conn);
    conn.setAttachment(user);
}

public void onMessage(WebSocket conn, String message) {
    User user = conn.getAttachment();
    // ...
}

Related Issue
#485 Create key/value map in WebSocket
Motivation and Context
I need to store information about each connection, id, name, etc.
I started by open an issue, then @marci4 replied there was an open discussion about my problem here: #485. After reading all it was said about a possible implementation, I did some performance tests between implementing an HashMap or creating a getter and a setter to an attachment of type <T>, and this one was the most efficient by far, with tens and billions of elements.
How Has This Been Tested?
I'm using this feature now, and it's working great. I have a User.java that I use as the connection attachment, with many variables and some logic, everything is working as expected.
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hello @promatik,
thank you very much for your pull request.
Do you have any tests which you can include for this feature?
I really like your approach, but would like to wait on some feedback from the rest of the community, if this is fine for you (waiting for one week or so).
Greetings
marci4

Hi @marci4, just added the javadoc since tag.
I'm not used to do tests, I don't know what kind of tests could I do for this feature.
Regards!

Hello @promatik,
thank you very much for your changes.
That is fine, I will add a simple JUnit test for this feature :)
Greetings
marci4


MATH: Is a 99.9999% accurate 1-layer addition model possible?

Our 2-layer addition model achieved 99.9999% accuracy using the TriCase/TriAdd approach.
Our 1-layer addition model only achieved 99% accuracy. It never learnt the TriCase/TriAdd approach.
But the 2-layer model only uses the attention heads in one layer.
Perhaps 1-layer addition model cant learn TriCase/TriAdd but perhaps the accurate 2-layer model can be pruned and retrained to give an accurate 1-layer addition model.
This ticket covers:

Migrate the existing "insert mode 1, 2 & 3" code from the VerifiedArithmeticTrain Colab to a new file train.py in the python library
Strengthen the code to handle inserting a 2-layer model into a 1-layer model (just prune off the 2-layer data).
Use VerifiedArithmeticTrain  to train a new model "ins1_add_d6_l1_h3_t15K_s572091" in the normal way - inserting the (pruned) model "add_d6_l2_h3_t15K_s372001"
In this ticket record the AvgFinalLoss and FinalLoss of the new model (as calculated by the Colab).
From the Colab temporary files, download the new model files ins1_add_d6_l1_h3_t15K_s572091.pth and ins1_add_d6_l1_h3_t15K_s572091_train.json. Give these files to PQ to load to https://huggingface.co/PhilipQuirke/VerifiedArithmetic/
Use VerifiedArithmeticAnalyse to run the "99.9999% accuracy" test.
In this ticket record the results (as calculated by the Colab).

Done


STT0003: Parse metadata from Catalog (5)

Description
The metadata for all the departments exist in various formats in their catalog Google Sheets. We need scripts to parse those sheets and get all the metadata associated with audio files.
Completion Criteria
Get the metadata in the TSV/CSV file ready to be consumed on evaluation of the model.

Implementation Plan

Subtasks

[x] set google sheet api and authentication
[x] divide the main google csv files into departments wise csv file
[x] script to parse google sheet and return df of google sheet
[ ] script to merge the deparment csv files with corresponding google sheet
[ ] merge all departments csv to main file.
[ ] write test cases

additional subtask of creating a meta data dictionary containing the columns to be added in main csv file
for each department google sheets.

i have to reformat the google sheets by deleting unnecessary columns which do not contribute to the data information gain.

split the deparment csv file into different parts if the file name id format differs.


feature: Add race for unit tests

Checklist:

[ ] Tests for the changes have been added (for bug fixes / features / refactoring)

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@69f503f). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #119   +/-   ##
=======================================
  Coverage        ?   11.28%
=======================================
  Files           ?        2
  Lines           ?      195
  Branches        ?        0
=======================================
  Hits            ?       22
  Misses          ?      172
  Partials        ?        1

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 69f503f...7618a9e. Read the comment docs.


Support getting all objects from bucket

:+1:

@guy-br I think it does not return all objects in the bucket if you specify Delimiter: / in s3.ListObjectsInput. So you might have to define s3.ListObjectsInput separately in the getObjsFromS3 function.
Cloud you add GET_ALL_OBJ_IN_BUCKET option in README.md as well?

@pottava what i meant was adding an env_var to get all pages in a directory and that is why i left the Delimiter. ill make changes to support both options and of course add GET_ALL_OBJ_IN_BUCKET to README.md as well. tnx for your work.

@guy-br I got it, it seems awesome. Thanks for your great idea and implementing features.

@pottava i have updated the name of the var from "GET_ALL_OBJ_IN_BUCKET" to "GET_ALL_PAGES_IN_DIR" for better understanding the meaning. also updated README.
thank you again.

@guy-br I've reviewed the commit. Looks great. So thank you for your awesome work, again!


Bug: Not all e-mail being threaded anymore

Hi there,
at least on one of my machines not all e-mail from the same sender with the same subject are being grouped/threaded anymore, so I get e.g. not a stack of 20 e-mails in one, but 20 separate e-mails. Latest 2.0.8 on Windows so far, using IMAP. Most e-mails do get threaded, though it is definitely more than one thread not appearing as such.
This was different up to date with N1, with the exact e-mail threads (notifications for a website) being threaded beforehand.

Thanks for the report @enoversum ! We'll take a look


WIP: Use a guard and not .error? method for sensu_ctl

I don't remember the reason for trying to use .error? to check the
output of the shell exec. What matters though is that the configure is
run once when needed and not every time once it's already configured.
This change switches to using a guard and a method to compare the
configured backend url.
The shortcoming of this approach is that as resource is currently
written it supports a username and password property, however we do not
have an implementation that can safely handle reconfiguring these
values so they will only work the first time the resource is used.
Fixes #63

Pull Request Checklist
Is this in reference to an existing issue?
General

[ ] Update Changelog following the conventions laid out here

[ ] Update README with any necessary configuration snippets

[ ] Cookstyle (rubocop) passes

[ ] Foodcritic passes

[ ] Rspec (unit tests) passes

[ ] Inspec (integration tests) passes

New Features

[ ] Added a Testing Artifact as either an automated test or a manual artifact on the PR.

[ ] Adedd documentation for it to the README.md

Purpose
Known Compatibility Issues

@majormoses just a start here to experiment with an alternate approach

This one still needs work and isn’t ready to be included for #80 yet


mesh export without color

i run the mesh export script and get the mesh of stage3. However, the mesh did't have any color. pls tell me how to add the color on mesh?

I haven't tried exporting meshes and textures with materials yet. The export code is from threestudio, and you can find more information and assistance in its repo.


Homework1 submission

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

Your choice of tables are accurate, but you did not mention the relationship between them i.e 1 to 1 or 1 to many.


[34475] Emoji 13.0

Summary
Update emojis to 13.0, there is a new category and some of the old ones have been renamed
emoji 13.0 changes
Ticket Link
34475
Screenshots

Release Note
updated emojis and emoji categories to standard 13.0

Additional Notes
there are a few emojis failing, my guess is that most of those reuse the image from another one. The list of emojis failing are:

png
link
name

1f9d1-200d-1f91d-200d-1f9d1
https://emojipedia.org/people-holding-hands/
People holding hands

1f469-200d-2764-fe0f-200d-1f48b-200d-1f468
https://emojipedia.org/emoji/👩❤💋👨/
Kiss: Woman, Man

1f469-200d-2764-fe0f-200d-1f468
https://emojipedia.org/emoji/👩❤👨/
Couple with Heart: Woman, Man

1f468-200d-1f469-200d-1f466
https://emojipedia.org/emoji/👨👩👦👦/
Family: Man, Woman, Boy, Boy

2640-fe0f
https://emojipedia.org/emoji/♀/
Female Sign

2642-fe0f
https://emojipedia.org/emoji/♂/
Male Sign

2695-fe0f
https://emojipedia.org/emoji/⚕/
medical sign

0023-fe0f-20e3
https://emojipedia.org/emoji/%23⃣/
Keycap Number Sign

1f1e6-1f1e8
https://emojipedia.org/emoji/🇦🇨/
Flag: Ascension Island

1f1e7-1f1fb
https://emojipedia.org/emoji/🇧🇻/
Flag: Bouvet Island

1f1e8-1f1f5
https://emojipedia.org/emoji/🇨🇵/
Flag: Clipperton Island

1f1e9-1f1ec
https://emojipedia.org/emoji/🇩🇬/
Flag: Diego Garcia

1f1ea-1f1e6
https://emojipedia.org/emoji/🇪🇦/
Flag: Ceuta & Melilla

1f1ed-1f1f2
https://emojipedia.org/emoji/🇭🇲/
Flag: Heard & McDonald Islands

1f1f2-1f1eb
https://emojipedia.org/emoji/🇲🇫/
Flag: St. Martin

1f1f8-1f1ef
https://emojipedia.org/emoji/🇸🇯/
Flag: Svalbard & Jan Mayen

1f1f9-1f1e6
https://emojipedia.org/emoji/🇹🇦/
Flag: Tristan Da Cunha

1f1fa-1f1f2
https://emojipedia.org/emoji/🇺🇲/
Flag: U.S. Outlying Islands

@Willyfrog note that I think the plan was to finish the emoji picker UI that was partly implemented here (with a skin tone selector and some UI updates): https://github.com/mattermost/mattermost-webapp/pull/7234
Not sure if that helps with issue # 2 you identified here: https://community-daily.mattermost.com/core/pl/6xux1nsc4fnnxdcu645w1nf8xh
@matthewbirtch can you help with the missing icons as identified by Guillermo?

@matthewbirtch can you help with the missing category icons as identified by Guillermo ^ in his post?

@Willyfrog will you be updating icons as part of this PR or a separate PR to update the entire emoji picker? There are a few conversations going around in a few different places right now and I'm having a hard time following

@matthewbirtch can you help with the missing category icons as identified by Guillermo ^ in his post?

@Willyfrog will you be updating icons as part of this PR or a separate PR to update the entire emoji picker? There are a few conversations going around in a few different places right now and I'm having a hard time following. cc @esethna

I wasn't planning on updating the UI more than needed. If there is a conversation going on about revamping the icons, then I can leave them as they are right now.
The only things to keep in mind regarding those conversations are that the category names have changed and that there is a new category.

@matthewbirtch @Willyfrog let's do the emoji picker UI updates in a separate PR so this one can just be focused on updating the emoji set.

I moved the images to the first two commits so it can be easier to review this change. Use the range of commits https://github.com/mattermost/mattermost-webapp/pull/8070/files/be44116b136b11da6fb221449f57a74b57ffbc18..3ebcc70ee702f74885b42e2e6c8cc7822ddbfb51 to do the review without all the noise from png/css/jsx autogenerated changes

Thanks! @jgilliam17 see above. Propose we wait to test the skin tones once we submit the PR for updating the emoji picker UI with the skin tone selector cc// @Willyfrog

Code looks good, but I'm wanting to test a few different things. I've tested all the Unicode emojis on Mac/Firefox, and there's a few of the newer ones that don't support skin tones.

Are those ones known issues? I also want to test these emojis on Windows as well, but I'll be doing that tomorrow.

Code looks good, but I'm wanting to test a few different things. I've tested all the Unicode emojis on Mac/Firefox, and there's a few of the newer ones that don't support skin tones.

Are those ones known issues? I also want to test these emojis on Windows as well, but I'll be doing that tomorrow.

Weird, I can see the pinching ones with skin tones with mac and firefox. I'm on big sur and I'm guessing you are on catalina?

@hmhealey updated, can you check if you can now see the images correctly?

Weird, I can see the pinching ones with skin tones with mac and firefox. I'm on big sur and I'm guessing you are on catalina?

In case we didn't already discuss that elsewhere, that's because it's using the system font which includes the new emojis. You'll be able to tell when it's falling back to the system font because it's rendering the emojis as text instead of a span with a background image set like our emojis.

updated, can you check if you can now see the images correctly?

Ninja and pinched fingers are correct now, but some of the later ones like woman feeding baby are missing still. I think those might be from the ZWJ sequences file we discussed earlier.

@hmhealey I think it still has problems with some of the emoji sequences, but now it is using the files directly instead of generating random sequences

@Willyfrog @hmhealey is this PR blocked or can we push forward on it and get it merged?

We switched to a new library which should help us deliver new emojis better. As soon as I have the a PR with it I'll close this one. I'm currently using it for reference

Closing in favour of https://github.com/mattermost/mattermost-webapp/pull/8220


Provide Asyncio Support

Would it be possible to add a separate asyncio implementation (or replace)? It should be possible since most of the code is just using sockets.

Feel free to make a pull request to support that.


Iphone can not connect to the vpn

Iphone 8 ios 12.2
I download the client application, and import the ovpn file. But when I click the button, it shows that "network is unavailable. Please try to connect later with active network. "
I open the setting -> vpn, and click the connect button there, after I release the button, it toggles back to close position and fail to connect the vpn.
How can I solve this problem?

That's a problem with your client, this issue tracker is not the right place to get support for that.


About & Resources Part

Add resources in website
text-align in about section on Right Side

Please assigned me @shriya030

@SamikshaSankar For the first part, link resources in the navbar to the tags page, and do 2nd part as it is.

@SamikshaSankar any update?

@SamikshaSankar any update?

@SamikshaSankar ?

@SamikshaSankar any update?

@SamikshaSankar ?

Sorry for late reply... within 5 days I will be completed this issue


Create sunmoon.txt

Sunmoon university
https://www.sunmoon.ac.kr/
https://lily.sunmoon.ac.kr/

wrong request. Pull request it again.  (kr -> ac.kr)


Does operationRef allow linking to external 2.0 definitions?

In link definitions, operationRef can refer to an operation in an external OpenAPI definition:
operationRef: 'https://na2.gigantic-server.com/#/paths/~12.0~1repositories~1{username}/get'

Is the external OpenAPI definition supposed to be in the 3.0 format or can it be 2.0?

While working on OASComply the idea of 2<->3 links came up and was roundly rejected by everyone, plus this issue hasn't seen further discussion since 2017.  I think we can safely say no to major-version-crossing references :-)


Navbar ui

Sorry currently we are not planning to change anything form the landing page please focus on main project