How to use list of regex for filter

Its a word blacklist not a partial word blacklist...
time to start matching every word that contains the letter
i dont see why you dont use regex instead?
Originally posted by @buthed010203 in https://github.com/jonas747/yagpdb/issues/544#issuecomment-547662279

How would I implement this? I have created a list of blacklisted words but in filter it only has "Words from blacklist" or "Matching regex" when I select "Matching Regex" it has a single line for input. Is it possible to use a list as regex?

Please read carefully through the linked issue, as there is a solution given already.
Moreover, regular expressions function in a way which allows for very high customization, which is however linked with some "having to learn it".
You can get a more definite answer on the support server, issues on this repository should be kept for real issues with the bot, not for general questions.

You can always just use a custom command with a regex of .* and process multiple regexes inside of the command. Alternatively you can just use pipes, hi|hello would match either hi or hello

Thank you for the responses. After chatting on the discord server it seems I will just use a word list. ironically used regex to reformat my regex file to plain text :laughing:


oshi version upgrade ?

is there a particular reason to use that old version of oshi ?
<version.com.github.dblock>1.5</version.com.github.dblock>
v2.6.2 is available but I didn't try it.
(v1.5 complains about my raspberry, nothing blocking for my demo, just the following error in the logs many times)
2016-06-22 14:32:12,488 ERROR [org.hawkular.agent.monitor.protocol.EndpointService] (Hawkular WildFly Agent Full Discovery Scan-1) HAWKMONITOR010049: Could not access resources of endpoint [PlatformEndpointService[Endpoint[platform]:[null]]]: org.hawkular.agent.monitor.protocol.ProtocolException: java.lang.IllegalArgumentException: Processor number (4) must be less than the number of CPUs: 4
at org.hawkular.agent.monitor.protocol.platform.PlatformDriver.fetchNodes(PlatformDriver.java:60)
at org.hawkular.agent.monitor.protocol.platform.PlatformDriver.fetchNodes(PlatformDriver.java:38)
at org.hawkular.agent.monitor.protocol.Discovery.discoverChildren(Discovery.java:72)
at org.hawkular.agent.monitor.protocol.Discovery.discoverChildren(Discovery.java:105)
at org.hawkular.agent.monitor.protocol.EndpointService.discoverChildren(EndpointService.java:251)
at org.hawkular.agent.monitor.protocol.EndpointService$1.run(EndpointService.java:204)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
at org.jboss.threads.JBossThread.run(JBossThread.java:320)
Caused by: java.lang.IllegalArgumentException: Processor number (4) must be less than the number of CPUs: 4
at oshi.software.os.linux.proc.CentralProcessor.(CentralProcessor.java:125)
at oshi.software.os.linux.LinuxHardwareAbstractionLayer.getProcessors(LinuxHardwareAbstractionLayer.java:81)
at org.hawkular.agent.monitor.protocol.platform.OshiPlatformCache.getProcessors(OshiPlatformCache.java:187)
at org.hawkular.agent.monitor.protocol.platform.OshiPlatformCache.discoverResources(OshiPlatformCache.java:438)
at org.hawkular.agent.monitor.protocol.platform.PlatformDriver.fetchNodes(PlatformDriver.java:53)
... 9 more

I know of no reason why we can't update Oshi.

Just did a quick test - immediately got compile errors when upgrading. So some work would need to be done.

Wow, they completely changed the API - specifically around the CPU stuff. This would require some work to upgrade.

move to JIRA : https://issues.jboss.org/browse/HWKAGENT-115


fix duplicate file in queue

When I drop two files then I drop two others files when the first queue not started, the plugin upload two files first then four files because the files in the first queue are status UploadStatus.Queue

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Mathieu Morchipont seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Fargate] Monitor control plane in EKS/Fargate

Investigate what can be scraped regarding the control plane clarifying if the current implementation of nri-kubernetes is capable to do so.

[x] In case we cannot scrape it modify nri-kubernetes to do so. Keep in mind we want to avoid scraping duplicate data

Then once possible:

[x] Modify manifest/chart configuration to help retrieving such data in Fargate
[x] Check if is there any overlap in the hybrid solution.

I created a "POC", it is indeed possible to scrape the control plane.

The issue is that the code is not ready to perform the scraping from a slave node.
Moreover we currently do not have any leader election in place, therefore the only way to do it now would be to ask the user to add a flag to a SINGLE instance forcing nri-kubernetes to ignore the master label and scrape the control plane
In EKS there is currently no solution that do not involves weird configuration/labels.

In an hybrid scenario currently it would be not an issue since in EKS the control plane metrics are not scraped

Closing for now since we do not have a way to scrape such data and it is not requested for feature parity with EKS


establish line sink as default driving function for 2D wfs

Having a focused point source as the default driving function in the 2-dimensional case is imho not very intuitive, since the line sources are used as secondary source. Therefore, I would suggest to use the model of the line sink as the default driving function.

This problem emerges from some general naming inconsistency. In all other cases we have always the same driving function for 2D and 3D as there should be no difference between the mathematical solutions. The problem with focused sources is that we have only the name "focused source" instead of "focused line source" and "focused point source". I think this is ok, as it is the way it is handled in theory. Nonetheless, I wanted to avoid to have a distinction between 2D and 3D inside the driving function and that is the reason why the default driving function is always a focused point source.
I agree, that this is not very intuitive and we should change it. The problem with your current implementation is that the code for the line source is repeated for the 2D and 3D case. It is not very nice, but maybe it is ok, and we simply live with it?

The same driving functions for 2D and 3D is maybe concept, which we shall reassessed. Having e.g. a driving function for a virtual point source in a pure 2D scenario, where we have secondary line sources, does not make much sense imho: It is impossible in my view to create the amplitude decay 1/r of a point source with line sources having a decay of 1/sqrt(r).
I agree that the repetitions of the code is not very nice. One option could be to change the order of the if-clauses (first the driving functions, then the dimensionality). Second option may be to combine dimensionality and driving functions into one string and use a switch directive, which is able to handle multiple strings in one case

Yes, it is correct, it is impossible to create a point source in a 2D setup and it is also impossible in most of the 3D setups to create a line source. The idea behind having those driving functions in the toolbox is that people can see themselves.

I found a way to incorporate this a little bit easier.
Now, we have this only for monochromatic sound fields. Could you implement it also for the time domain?

I have added no also the time domain version of the line sink focused source for 2D.
@fietew: could you please test.

Where does this formula come from? see http://sfstoolbox.org/#equation-d.wfs.fs.ls

I guess it comes from http://sfstoolbox.org/en/latest/#equation-d.wfs.ls and should be its time inversed version. But we should remove v and replace it with x0-xs as for the D case. Do you agree?

I have corrected the documentation and going to merge this now. If there are still some errors, please create an issue and we fix it in the next release.


overall township roadmap

Just to lay out the rough plans I've had for township and hopefully to gather some input, here's the roadmap I've had in mind.
Suggestions are welcome!
Work so far

township-auth, township-access, and township-token  provide basic functionality for authenticating and authorizing users
township-accounts is a wrapper around the auth, access, and token modules that makes getting started a little easier, and provides some extra helper methods
most of this work came out of building some node projects and wanting an alternative to auth0 & passportjs

Near-term goals

complete the api request handlers in this repo
continue setting up postmark for transactional emails like resetting password, or decide on another service/approach
make sure the way JWT tokens are being used is reasonable (something I'm still learning about)
write the docs!

Long-term goals

create libraries that simplify creating user profiles, organizations, teams, and related authorization/permissions & transactional email
create township.someTLD as a service that provides

APIs for auth, profiles, organizations, and teams
integrations with other services (e.g. associating slack channels with teams, github orgs with township orgs, etc)

use the org/team functionality as basis for various projects

awesome, sounds good. I also wanna make this module have a CLI tool that does stuff like this for user management for sysadmins https://github.com/staticland/staticland-api/tree/master/bin


Parse labels with single quotes

The regex was only stripping double quotes, it
should also support singles.


Fix false positive serverspec checks on Swift and Ceph Nodes

Ceph and Swift nodes do not have a /backup/ directory. It should not
therefore complain if it doesn't exist, but if it does exist, it should
check for the appropriate mode, etc.

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

ok to test

retest


python-pex - Usage example?

Currently there is no usage example for python-pex plugin.
I've being trying to use it without success; A ./gradlew build outputs the following error:
> Task :buildPex FAILED
Processing /root/git/pex-example-tentative
Building wheels for collected packages: pex-example-tentative
  Running setup.py bdist_wheel for pex-example-tentative: started
  Running setup.py bdist_wheel for pex-example-tentative: finished with status 'done'
  Stored in directory: /root/git/pex-example-tentative/build/wheel-cache
Successfully built pex-example-tentative
/root/git/pex-example-tentative/build/deployable/bin/pex-example-tentative.pex
Traceback (most recent call last):
  File "/root/git/pex-example-tentative/build/venv/bin/pex", line 11, in <module>
    sys.exit(main())
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/pex/bin/pex.py", line 620, in main
    pex_builder = build_pex(reqs, options, resolver_options_builder)
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/pex/bin/pex.py", line 573, in build_pex
    pex_builder.add_distribution(dist)
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/pex/pex_builder.py", line 317, in add_distribution
    dist_hash = self._add_dist_zip(dist.location, dist_name)
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/pex/pex_builder.py", line 280, in _add_dist_zip
    wf.install(overrides=self._get_installer_paths(whltmp), force=True)
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/wheel/install.py", line 268, in install
    if self.parsed_wheel_info['Root-Is-Purelib'] == 'true':
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/wheel/decorator.py", line 17, in __get__
    val = self.wrapped(inst)
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/wheel/install.py", line 238, in parsed_wheel_info
    return read_pkg_info_bytes(self.zipfile.read(self.wheelinfo_name))
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/wheel/decorator.py", line 17, in __get__
    val = self.wrapped(inst)
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/wheel/install.py", line 232, in zipfile
    self.verify(vzf)
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/wheel/install.py", line 385, in verify
    record = zipfile.read(record_name)
  File "/usr/lib64/python2.7/zipfile.py", line 931, in read
    return self.open(name, "r", pwd).read()
  File "/root/git/pex-example-tentative/build/venv/lib/python2.7/site-packages/wheel/install.py", line 437, in open
    ef = zipfile.ZipFile.open(self, name_or_info, mode, pwd)
  File "/usr/lib64/python2.7/zipfile.py", line 957, in open
    zinfo = self.getinfo(name)
  File "/usr/lib64/python2.7/zipfile.py", line 905, in getinfo
    'There is no item named %r in the archive' % name)
KeyError: "There is no item named 'pex_example_tentative.dist-info/RECORD' in the archive"

Since there is no info on google or issue here about this message I assume this is a configuration problem rather than a bug. But I'm unable to troubleshot this any further without help; I'm willing to submit a PR with an fully working example once I've figured this out.
I'm attaching below the full project I've used to reproduce the issue above.
pex-example-tentative.zip
Any pointers to a solution or further troubleshoot steps are appreciated.

That's because we need a version to produce the deployable artifact.
Simply add version = '0.0.1' below the applied plugin in build.gradle and it will work. :-)
We also recommend using python-cli plugin, instead of plain python-pex.
The pex plugin is the engine under the hood of all other plugins that produce deployable artifacts (cli, web-app, ...), but is rarely used in its direct form. There are exceptions of course, but if you need a command line tool, it's best to use the cli plugin. You can always define your own wrapper for it (the small program that calls pex with right entry points) if the default one doesn't suit you. Let us know if you need further help with customizations.
I agree, we need to put more substantial examples as a part of documentation here.


Fix: sorting and add news tag filter

This would fix sorting matter. It will be based on date+time with ISO 8601.

Found a small style bug.

Now it is okay for review


Fix Typo in spelling of linear search on home page

Fix Typo in the spelling of linear search on home page @ankiiitraj can you brief on this?

Closing against #3


bower_components

Good day,
I want to install a component that's only available through bower: http://timetablejs.org/
How can I get it to work in this project and make Timetable available in a component?

Hello!
This is a good question.
First of all this is not angular module/component/directive, so be careful if you want to use it with Angular.
I talked with many angular developers and I never found a guy that is using bower. I think that npm is the 'standard de facto' for Angular2.
My template, but also all other templates that I found on github are using npm.
The best thing that you can do is to open an issue on timetable's github page to ask for an npm release.
npm is official, bower is an alternative.
Yesterday, I published a library to npm (not on bower) and it is extremely simple.
If you need support while opening an issue on timetable's page, link this issue.

Oh wait, there is an open issue about this problem https://github.com/Grible/timetable.js/issues/1

Thank you for your reply.
Ooh I see,  so I won't be able to use it if it's not installed through npm? If I can use it can you please direct me on how to get it to work?
Also, let's say I wanted to use lodash (installed through npm)  in a component? How can I use it in a component?

Yes, all templates/skeletons use npm so you have to wait for an npm release.
I used underscorejs with angularJS 1 and I created a factory to expose "_" to angular in a standard way.
I never tried to use lodash in a component, but only inside an Angular2 unit test.
Check here for an example: /blob/4b0885779745e7c94e20a011c8edada442a0ac16/src/app/common/services/contact.service.spec.ts.
If you have problems with the previous example, you can try to use lodash as localstorage/sessionstorage, accessing the "window" object, for instance window._.isError();.

Thank you so much for your help.  I'm going to look into it.


JacksonJsonNodeJsonProvider throws class cast exceptions

https://github.com/zalando/logbook/blob/bc4afe1b12215f32197449b54ae8f96e6c44be55/logbook-json/src/main/java/org/zalando/logbook/json/JsonPathBodyFilters.java#L92
This JacksonJsonNodeJsonProvider throws class cast exceptions:
class com.fasterxml.jackson.databind.node.ArrayNode cannot be cast to class java.util.List (com.fasterxml.jackson.databind.node.ArrayNode is in unnamed module of loader 'app'; java.util.List is in module java.base of loader 'bootstrap')
Resulting that the body is never transformed when there is one rule applied that handels arrays. Once the exception is throw the original body is returned (and this line also hides the exception pretty well...)
https://github.com/zalando/logbook/blob/786791737dfcf8e83daa7152e381836eeda254cd/logbook-json/src/main/java/org/zalando/logbook/json/JsonPathBodyFilters.java#L112-L113
Issue is that the expected type of (T) would be List but JacksonJsonNodeJsonProvider on createArray() is returning an ArrayNode which is not castable to List.
A solution would be to use JacksonJsonProvider instead of JacksonJsonNodeJsonProvider

Where is this cast to a List happening?

The cast itself is implicit and part of JsonPath (and maybe not much to do about it):
https://github.com/json-path/JsonPath/blob/3f4900d6e7361cd21f0327c828dabf83fd188227/json-path/src/main/java/com/jayway/jsonpath/internal/JsonContext.java#L131

then:
https://github.com/json-path/JsonPath/blob/3f4900d6e7361cd21f0327c828dabf83fd188227/json-path/src/main/java/com/jayway/jsonpath/JsonPath.java#L225
then:
https://github.com/json-path/JsonPath/blob/3f4900d6e7361cd21f0327c828dabf83fd188227/json-path/src/main/java/com/jayway/jsonpath/JsonPath.java#L752

the line it starts is here:
https://github.com/zalando/logbook/blob/786791737dfcf8e83daa7152e381836eeda254cd/logbook-json/src/main/java/org/zalando/logbook/json/JsonPathBodyFilters.java#L58

Sounds like a bug in json-path then. Maybe there was a change in Jackson which caused this incompatibility?

I got the same error. The class cast exception occurs when a node is deleted that does not exists.
The following test fails:
@Test
void test(){
BodyFilter filter =  JsonPathBodyFilters.jsonPath("$.invalid").delete();
System.out.println(filter.filter("application/json","{"key_1":"value 1","key_2":"value 2"}"));
}
I my opinion a node that does not exsists should  be ignored (withoud exception).

In the meantime, as a workaround I've used the JsonBodyFilters instead of the path:
JsonBodyFilters.replaceJsonStringProperty(Set.of("fieldName"), CONTENT_REPLACEMENT_TEXT);

Also this log can be supressed by adding this to your logback.xml for now:
    <!-- quiet down the logging for logbook json filters -->
    <logger name="org.zalando.logbook.json.JsonPathBodyFilters" level="INFO"/>


Second refactoring of container names

Currently we have most container and image names renamed to our new format "gcb_ ..."
However the wordpress-offfical-data and similar repositories still use legacy container names. Reason being, these names are used as hostnames for HTTP access or hostname based networking.
So we cannot actually use underscores in our container names.
In turn we would like to move all image names to underscore based naming (which should already be done).
And move all container names to the SAME name as the image, only with dashes.
This must also be documented in the documentation somewhere that our example applications  follow this logic.

https://docs.green-coding.org/docs/measuring/containerizing-applications/
https://docs.green-coding.org/docs/contributing/example-applications-contribution/


fix: broken tests in add-axes-to-a-visualization-challenge-in-d3 (Iss…

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #35554
Fixed the Broken Tests in Add Axes to a Visualization Challenge in D3 opened by @thecodingaviator to resolve the issue #35554.

The files of this PR look fine, it'll still need to be tested locally though

Thank You! @cmccormack for approving it.

Thanks for contributing to the freeCodeCamp repository. Congratulations on your first PR! 🎊
Look forward to more contributions from you! ✈

Thank You so much! @thecodingaviator


Export enums as const

Currently, enums as generated as the following:
export enum Suit {
  club = "club",
  diamond = "diamond",
  heart = "heart",
  spade = "spade",
}

This results in the following JavaScript:
var Suit;
(function (Suit) {
    Suit["club"] = "club";
    Suit["diamond"] = "diamond";
    Suit["heart"] = "heart";
    Suit["spade"] = "spade";
})(Suit = exports.Suit || (exports.Suit = {}));

With this PR, enums are now generated as the following:
export const /*enum*/ Suit = {
  club: "club",
  diamond: "diamond",
  heart: "heart",
  spade: "spade",
} as const;
export type Suit = typeof Suit[keyof typeof Suit];

Which results in this JavaScript:
exports.Suit = {
  club: "club",
  diamond: "diamond",
  heart: "heart",
  spade: "spade",
};

By exporting the additional type, compatibility is preserved for any consumers that are using the existing enums as types.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Are there any backwards compatibility concerns around this?

I added the type export to allow for using these generated consts as types.

In the example, it shows a double export of the name Suit, and was wondering if this is valid?
export const /*enum*/ Suit = { // <- Suit export 1
  club: "club",
  diamond: "diamond",
  heart: "heart",
  spade: "spade",
} as const;
export type Suit = typeof Suit[keyof typeof Suit]; // <- Suit export 2


Check for new Projects who have added an OP address - September 2023

We should do another query and find the list of projects that have added OP address to do another distribution. Related to https://github.com/Giveth/giveth-dapps-v2/issues/2637

is it urgent?

is it urgent?

No not, urgent  - mohammad knows how to do it and can handle it when he's returnd from vacay

hey team! Im gathering the metrics of our progress on this, could you tell me how many projects with an OP address we have? (I assume only verified projects right?)

If its urgent we can ask @CarlosQ96 to take a look. Mohammad is on vacay

I think we can wait for @mohammadranjbarz! And actually would be great to have both categories:
of Giveth Projects with an OP address (verified and not verified).
Verified Projects with an OP address
Thanks!

@oyealmond  @divine-comedian
Non verified projects that have OP addresses ( 170)
non_verified_projects_with_OP_address_2023-09-25.csv
Verified projects that have OP addresses ( 114)
verified_projects_with_OP_address_2023-09-25.csv

Updated a new entry into the spreadsheet and used a formula to detect projects we already distributed to -
https://docs.google.com/spreadsheets/d/1pZ7SUtFcf6sNZ3A8fZXBvtDGAn_w21qD3WOaB4baDOE/edit?usp=sharing

awaiting multisig tx to pass this
https://app.onchainden.com/safes/oeth:0x4d9339dd97db55e3b9bcbe65de39ff9c04d1c2cd/transactions/0xc0eabbeac359e8aeb51fead70875c1b0db85beed85311c66b701a3a8dd52aee6

how ofter will we be doing these checks? I expect we'll get a bunch more in the next week or so because we're doing a big comms/outreach push to try to get projects to add their OP address to enter the QF round.
I think we should do a check next week again.

how ofter will we be doing these checks? I expect we'll get a bunch more in the next week or so because we're doing a big comms/outreach push to try to get projects to add their OP address to enter the QF round.
I think we should do a check next week again.

@mohammadranjbarz does them manually - you can check the original issue referenced


Enhance class search

Optimize class search with an index
Allow manipulation of classes with special characters in their name (+, &...)
Allow partially qualified class name (e.g. mutable.Seq)
Also fixes #503

I have absolutely no idea of why in Scala 2 there are no breakpoints inside A+B$$amp$amp$#x in file RuntimeEvaluatorTests.scala:898
I launched the same program in a debug session and placed breakpoints on the whole method. Never reached them


WIP: packaging fixups

Not yet ready for review/merging
To do:

[ ] re-enable appveyor after rename to hvplot
[ ] someone created a c-f recipe for holoplot; need to have that renamed/create new one
[ ] fill in rest of to-do items...

Closing for now. Can always reference this if we want to use any part of this later.


[MM-48225] Reduce Padding/Margins from Files Tab that was extending into filter icon

Summary

This pull request reduces the padding/margins to make space for the Messages/Files tab on the Search Page. Without reducing these spaces the Team Icon was getting above the sort icon you can see it in the following screenshots.
Feel free to let me know if I need to update more things.
Ticket Link

Github Issue
Jira Issue
Checklist

[X] Has UI changes

Device Information
This PR was tested on:

iPhone SE (3rd Generation) - iOS 16.4
Nexus S - API 25

Screenshots

iOS
Before:

After:

Android
The app is crashing on opening in my Android emulator, so I'll post pictures shortly when I solve this.
Release Note

Reduces the padding/margins on the Files Tab in the Search Page

Hello @panoramix360,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@panoramix360 The idea was to move the text below the buttons, so the buttons don't take that much space (keep in mind they may look like they fit just fine in English, but other languages may have longer words for messages and files, so we prefer to fall on the safe side).

ok @larkox! I'll do it, since I saw that there was another issue to move the results below, I thought that somebody was doing it already.
I'll update the PR then :)

@panoramix360 just in case, can you share that other issue? I would like to verify that :smile:

of course! in the Jira issue you mention an open PR that seems to move the search results below the tabs.
So I figured this could be already taken care of.

@panoramix360 That PR is stale. Feel free to continue it, or merge it into your PR, or any other thing. If you can get that one through the finish line we would really appreciate it :smile:

hey @larkox, take a look, do you think that's what you guys intended?

hey! I manage to run on Android and it's working as expected.
I edited the PR description to include screenshots from Android.
I notice that the files that I updated don't have unit tests for them, so I didn't create them for this.
In regards to e2e, the coverage is for all the screens? If you can explain what are Mattermost's strategy for tests in this case, I can try to do something.

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

hey @enahum!
I created the translations like you asked. Can you review it?
I used the useIntl hook, I don't know if it's the correct way but it was the examples that I found on the project.
Let me know if I need to do anything else!

thank you for your explanation @enahum!
Didn't know that the project had a component called  for these.
I'll keep that in mind for future PRs.
I updated like you requested, let me know if it's correct right now!

hey @abhijit-singh!
I removed the spacing, can you check again? :)

please resolve conflicts

hey @larkox and @enahum!
Sorry for not explaining my reasoning for changing/removing the marginTop.
I'm aware that the component is used in other places, but I was cautious about it.
Before changes
The file.tsx <File> component was adding a margin-top of 10 to have spacing between the Files not only inside the Files result but when it's used to display files inside a body of a message. See screenshots below:

These above are the two places where the <File> component is used.
Then, I searched through the project where the <File> component was being used:

Not only on the results-related components, the one we are interested in changing but on the <Files> component as well.
So, searching for where the <Files> component is being used, I found the following:

You can notice that it's used inside a body post.
After changes
At first, I was cautious about removing the marginTop from the <File> component because of what I described above.
But since it's not so good for the File component to have a marginTop when we want to use it in different use cases, I decided to remove the marginTop and make sure the other places (despite the search screen) would still maintain the same look.
That's why I inserted a marginTop inside the <Files> component, with this we fix the margin when the Files are used inside a body of text (or in other places in the future) and I can use the Separator inside the FlatList of the Search screen without conflicting with the marginTop of the <File> component that was there before.
The separator is great because it adds spacing just between the renderItem components and not on the top or the bottom, so the first element can be in place instead of 10 points downward.
Conclusion
With this change, we adjust the spacing on the File tab in the Search screen since there is no margin on the top of the results.
Let me know if there are other solutions or your thoughts about it.
Thank you!

How does it look in a channel on a message that has multiple attachments with different types videos/images/documents/etc

I needed to make a small adjustment right now, on this line of files.tsx:
let container: StyleProp<ViewStyle> = items.length > 1 ? styles.container : undefined;
It was not adding the marginTop when the items.length > 1 (I don't really understand this if statement)
Since the marginTop was always on the <File> component, I added on the <View> wrapping the <File> component.

Take a look at these examples:

More examples that I could find:

I solved the conflicts too.

Building app in separate branch.

@panoramix360 Thank you very much for your contribution!


Update cs.coffee

Finally done all translations :)

Great work, @Martin005! Thanks!

@Imperadeiro98 Thanks :)


changed flood function in stream-example, for loop to '<=' from '<' s…

updated flood function in stream-example.js so that it will send a null chunk to trainstream.write(). This triggers finish event emitter in write. Original caused a nodejs stream3 error because nodejs object mode true doesn't allow for null data writing.
Example would throw an error, but the change allows the for loop to send undefined to trainsteam.write, causing the chunk to be false triggering the finished event emitter.

Does this fix the behaviour of streams?

This doesn't change the behavior of the current brianjs stream implementation. What it does is show how to use the current trainstream. The way trainstream was written It will emit a finish event when there's no more data. I notice that it's manually emitting 'finish', as opposed to calling write.end() which then triggers a 'finish' event.
I suppose it was originally done this way since the use case is not trying to append anything to the end of the stream. If you think it's necessary I could see about updating the train stream to use the write.end() methodology.

Whichever is the recommended use, I'm fine with.

I think using the end() method would be best as its seems to be the nodejs recommended way. I'll see if I can make the change tonight without effecting anything else in the trainstream implementation. If successful I'll submit a new pull request. If not I'll report back the issue.

👍

managed to run a few test during lunch and just now. I now see why the finish event is being emitted. as well as why the write.end() wasn't used.
Writable streams don't emit an 'end' event. Calling write.end() closes the stream, the way trainstream was written it's handling the write.end() event in the floodcallback() function. So to be able to trigger the event at the correct circumstance it needed to be emitted.
There's no need to change the trainstream implementation people just need to be aware that sending undefined is being handled in the floodcallback which will re iterate training if number iterations or error conditions haven't been met. This should be good to go and the example functions as intended with the purposed change.

A documentation change/addition then?

What about a gotchas section? The stream-example bug is more of a gotcha if someone codes how the example is setup. The trainstream isn't broken it's just handling a false value for you, so adding '<=' is crucial to getting the stream to behave as intended.
Also I've been messing around with different training techniques, basically training the net with different data formats and lengths. I noticed that inputs need to be larger or equal to ouput lengths when dealing with Arrays. But output lengths did not need to be the same length. I also observed some other gotchas like having the length of the first output being one and the rest being length two will cause an output of length one when testing data on a trained network. This is interesting if you're testing data like language or audio files, etc. Since the data length won't be the exact same.  I'm still playing around with the inputs and outputs. But I still haven't tested my solution, which is padding inputs and expected outputs so that they're the same length as the maximum input, and  outputs are the same length as the maximum output.
Have you experienced this at all? It came up because I was getting NaN when tests data inputs of different length and having outputs of different lengths. Also you can't feed an array of input and receive an object as an output. And vice versa. So maybe an expansion on the docs which describes how the data may need to be formatted to get something meaningful. I imagine padding the output could cause issues with  training the net becuase the padding would mess up the weights and bias. Padding the input I don't see much of an issue. I could work on a gotcha section that covers the training data formatting issues and the stream-example issue.

I have not experienced that, but a gotchas section would be fantastic.

Any further input here?

Yes, I've been working on training the network to do a form of language classification. I know there's "natural" for node the language classification. But they store stop words in an array, as well as store the key phrases in the array.
I figured brainJS can do the same thing it's a matter of how to do it.
Initially I started off by making objects with arbitrary weights that I was assigning in the input that seemed cumbersome if a person needed to build a data set. Or the user would have to build an API that weights the words for them, I figured that may not work since the words importance is based on the context.
So I moved to reading the data with a stream the stream returns the data in a buffer in numbers 0-255.
My approach was then have a data set that has a sentence input(string) and then a few key words related to the sentence e.g. is the sentence a question, declaration, or a request, is it about time, location, and what significant words were used.)
I was somewhat successful in doing this an this is where the gotchas come into play.
Gotcha #1 if you build an index file and run it in node you have to wait for the training session then see what it gives you if you put in a few sentences at the end to check/ run(). I wanted to test a bunch of sentence to see where the classification was working and if changing the sentence structure would yield different results.
Gotcha #2
I decided to build a small javascript object that read the data from a stream, formatted the strings into an array of numbers (divided by 255, because the streams maximum is 255) so they'd be between 0 and 1.
Gotcha #3 arrays that vary significantly in length will cause brainjs to return NAN on the output. This goes for different length inputs (I noticed this when the inputs were a difference of 5 in length. And the outputs difference were 3 in length ). So I fluffed the data by appending 0.00 to the end of arrays that were short up to the maximum input length and the same for the output.
Gotcha #4 if the decimal on a number is 0.005 ± then the letter it generates after being converted will be off by 1 to 3 indexes (eg. "q" may end up as "n"). So part of the javascript object was dedicated to holding onto the network and sending it test strings to see which words it recognized and could classify.
Gotcha #5 is about network initialization states and dataset.
Test #1

To get the network close to classifying the sentence correctly you need a large dataset. I start with 80 inputs and outputs, with 20,000 iterations, three layers [150,88,200], and a threshold of 0.005 or 99.5% accuracy.
This performed poorly but would meet the threshold before the iterations completed. The issue was it could classify questions if the word "what" was present in the sentence but it didn't recognize other questions words like who, , when, where, and how.
Test #2
I figured it's a complex task I'm asking of the network it may need more nodes and layers.
So I left the data set the same and upped the layers and nodes to see if additional abstractions on the networks Mart could figure it out. [250,380,180,222] unfortunately this caused the network to not meet the threshold and go over the iterations. Only reaching 0.138 error. Which didn't classify anything correctly.
Test #3
Instead of adding more layers and more iterations I upped the dataset. To 100 in 100 out (now these are different classifications within the dataset not just one type like question)
This resulted in a smaller error 0.108. this sort of classified questions but the answer was un-readable becuase the letters were so far off of that they should be.
I thought surely I'm onto something though because the error decreased with a larger data set.
Test #4
I then upped the dataset to 250 in and 250 out. I left the initialization the same as the prior test. The larger dataset and large node size cause these test to run around an hour maybe a little more.
This yielded an error threshold of 0.0138, which had the same problems as the above test the answers generated were un-readable since the letters were so far off. Still more of the answered yielded were visible to a human as "questions" which was represented onscreen as "pueshhsen" or "rwd'tsm$" there multiple types of messed up answers but these two were close.
Test #5
I thought well surely the dataset caused the answers to get close maybe I should up the nodes and layers again. [408,300,280,370,403,238,321] these are arbitrary choices. I left the datsset as 250in and 250 out.
This resulted in the error threshold increasing back to approximately 0.136 which I thought was coincidental to the earlier error when I increased the nodes and layers.
This cause the training session to last 6-7 and you could see in the end of the training that it was hung up trying to decrease the threshold only making 0.0000002 increments of progress for every 100 iterations.
Test #6
By this point I figured the issue was the test data and it needed to be much greater (image recognition for human written numbers had datasets in the thousands...) So embarked on training more data. I did 300 in and 300out. This lowered the error to 0.08 increased the testing time to about 8 hours
Test #7
Then again I upped it to 400 in and 400out
This yielded an error threshold of 0.0108. And took 12 hours (not sure why). But it maxed out the iterations.
Test #8
Upped it to 508 in and 508 out ( happen to come up with 8more sentences...) I then decreased the nodes and left the layers the same.
[208,100,250,170,203,238,121] . This caused the error to increase back to 0.185 and again was in un-acceptable. Also this maxed out the iterations counts.
This trained for about a day and a half 36 hours.
Test #9
I went back to increased nodes and left the dataset the same and went with [408,300,280,370,403,238,321]  nodes and layers.
The increased nodes and the increased dataset was maxing out the iterations and heck I figured this was an issue so I bumped the iterations to 30,000.
This allowed the error threshold to get back to 0.0176 but it maxed out the iterations this took 48hours to finish training.
Test #10
Left everything the same and bumped iterations again to 40,000 and viola I made it to 0.00499 error threshold.
This is when I found out that the 0.005 differe in a decimal will cause the letters to be off by one.  It was closer to classifying and could determine if things were a question or a declaration. They weren't spelled correctly at all but you could see that the letters were only off by one or two indexes.
Test #11
I bumped the iterations to 50,000 and the threshold to 0.0005.
This maxed out the iterations but yielded an error threshold of 0.00208, this classified all the question words and some of the special words like color, time, you, and my. It classified most all the sleep and alarm classifications like alarm, bed and timer. Again the error wasn't small enough so almost every classification had a single letter some times two that were off by one index.
This also last 56 hours for the training session.
Granted I don't have a powerful machine dedicated to this so that may have been a big issue with the training session. Had I used a more powerful machine it may have taken significantly less time.
All in all I think the I biggest issue is having a large enough dataset for this task. Is it possible definitely since the question words were being classified as questions. This can be useful as you can then take the answers and send those to another nueral net that can maybe set a timer or set an alarm, or if it's a general question Google something. Maybe you need to know your next meeting. The other networks could check and return the final answer.
Now we need to figure out how to best format these test cases into a gotcha section.
I'm thinking initialization, dataset size, and data formatting. Since initialing the network a certain way would allow you to get better results, having a large enough dataset for something this complex is also needed ( and I still think I need to add more to the dataset I was using). And then data formatting was an issue e.g. having the input areay lengths the same and the output array lengths the same was important to yield actual numbers and not get NAN.

A breathtaking ride!  Thanks for sharing!  Do you plan on sharing any of your code?  Please keep up the good work.  I'm working on recurrent neural nets in brian.js at the moment, and have been dabbling in taking advantage of multithreading so that training can be done a great deal faster, a minimum of 5 times and at the maximum around 700 times faster.

Yea I'll work on posting my code this afternoon (hopefully) I'm going to be MIA through Thanksgiving with no internet or phones :) :D  I should be able to post it though since its already built out and only needs to be place on a repo. Maybe during lunch hmmmm.

Quick point: The input and output indices do not have to be related.  Say if you had an input that varies of 1000 floating point numbers between 0 and 1 and the output you are trying to find something with a narrow output, simply decrease the outputs indices.  Example:
var input = toIndices('What are all the letters in the world?');
var idealOutputs = {
  what: 0,
  who: 0.2,
  when: 0.5,
  where: 0.7,
  how: 1
};

You effectively could have one output, which would substantially improve training time.

Too, I share your sentiment with the natural library (https://github.com/NaturalNode/natural/blob/5bbff4217b26cfaa1082cded2ae00e34b48c0bc6/lib/natural/util/stopwords.js#L25).  It is clever, but doesn't scale for multiple languages... unless you built a neural net that could modify the natural library...

I initially did it that way. But then I wanted special words to arise like time,alarm, color, or timer this increased the output size. Also I was having trouble on how best to number them if im looking for a certain word?
Or classifying the sentence as a declaration.
The issue arises when you have array format with different output lengths as opposed to object format. I'm sure we could adapt the object I created to use object format as that seemed to be the cleaner and easier way of getting the data I wanted. I couldn't come up with a decent way of handling the data output.
The array format I chose does the classification by converting the output back tona string.
There's definitely room for improvement...

Fyi I wasn't able to find time before leaving to push to a git repo. Will have to be Monday.

Still excited to see results!

Was able to upload my code I kept it separate from the brain.js files since its an object that I created and wasn't very fond of how I set it up.  I added a readme.md file its not very explanatory but I'm hoping the index-stream.js file is enough to understand what's happening. Will work on adding screenshots soon.
https://github.com/Kalevera/brainjs-classify-example

A lot has happened since you opened this issue, the recurrent neural net is now working.  Now, I'm told (and do believe) that there are many better types of networks that would suffice for training similar to what you are wanting, but it may be the case that the recurrent neural net (specifically the LSTM or GRU) would be more ideal, or perhaps even the perfect fit here, other than the ANN (which I thought was just a standard Feedforward net until today, say hello to my little friend: https://github.com/harthur-org/brain.js/blob/1b92c81b0a5187712015d1eff7c941c50fce5a70/lib/neuralnetwork.js#L138) that is the default neural net in Brain.js, may be more ideal.
The reasons are that you want a somewhat flexible input and a specific output, a true recurrent neural net does just this, it doesn't get close, it gives you exactly that, sort of like a translation system.  Also, I feel that the input/output with a recurrent neural net is somewhat misunderstood...
Expounding:
The recurrent neural net is fed one character at a time, building up a memory of context, until the whole input is within the network, also you give it the expected result, or output...  Read that again and return here.  You read that correctly, the input and output are sent INTO the network in its training phase as the input.  The network is trained to know the difference by a specific index that is only used when separating an input from an output.  Then when predicting, or running the output, you simply feed the network the input followed by the separation character, and then run the prediction.  Now it may take some time to get training down, but most likely MUCH less time than with a feedforward neural net.
The reason it takes less time?  The neural net knows the entire state of the neural net, values can sort of intercommunicate with each other, and the network has context, whereas the ANN is more static mathematical calculations.
I'm working on the training method now, which should accept your inputs just as they are in the library you posted, hang tight, I'll see if I can get it ready in a day or two.
I'm still sort of wrapping my head around the whole thing, and even some data scientists (http://karpathy.github.io/2015/05/21/rnn-effectiveness/) are as well, but hopefully this has helped enlighten the path a little.
How well are you results by the way?

updated my readme.md to include the screenshots i took.

Very nice work and dedication!  I did have some time to add a pseudo input/output to the rnn's, if you'd like to try them: https://github.com/harthur-org/brain.js/tree/rnn-train-pattern
You'd do something like this:
import LSTM from '../../src/recurrent/lstm';
import Vocab from '../../src/utilities/vocab';
var net = new LSTM({
  vocab: Vocab.allPrintableSeparated()
});
net.train([
  {
    input: 'hi',
    output: 'mom!'
  }, {
    input: 'hello',
    output: 'dad!'
  }
]);

//rnn prediction
console.log(net.run('hi'));
console.log(net.run('hello'));

Something I think too that needs addressed that could really help here and elsewhere is an auto adjustment of learning rate based off error rate.  Also, the error rate from rnn is different from that of the ANN, not sure honestly what the base value should be, but this should be enough for you to play with.

Also, any suggestions for simplifying the api would be fantastically appreciated.

Closing for the time being.  If this sparks up more conversation, and becomes something merge-worthy, I can reopen.


Improve logo for Twitter cards

In reference to https://github.com/pypa/warehouse/issues/3112, literally solely changed the image used as twitter does not support svg images.

I'll work on fixing that test, thought it would be a quick one liner fix, I should have tested previously.

Hi, this is the problem:

Image is not cache busted. This is, what I believe to be the solution as the tests pass when I add it:

but I cannot push changes to the manifest.json, so I suppose I'd need to wait for that to happen.

@jMuzsik Yeah, sorry for the confusion. The manifest.json file is generated automatically when compiling our static assets. Most of our static assets live in a separate private repository for... reasons, so the actual assets in ./warehouse/static/images are just placeholders.
You should just create a dummy image, name it twitter.jpg, and put it in that directory for this PR.
I went ahead and created one for you to save you the trouble:

For one reason or another, jpg did not cache in the manifest.json so I converted the image to png, in which case it worked.

@jMuzsik Hmm, the actual image is currently a .jpg, so we'll either need to either a) figure out why it didn't work or b) change it.
Was there an error? Or did it just not appear in the manifest? I'm guessing it's because the extension is not listed in the includeFilesInManifest argument in Gulpfile.js, we might just need to add it.
Probably the best to do this for the future when this might happen again, unless there is some reason why we can't use .jpgs (not totally sure why @nlhkabu made the original image a .jpg, perhaps she can chime in here).

Here is a couple images, for whatever reason, jpg never got into the manifest.json. It doesn't exactly make sense, I was quite confused for a bit of time not knowing what was going on

Tried altering the places that appeared reasonable to add jpg to but still it does not work. Cache busting error still occurs when running the tests because it does not go into the manifest.json. The jpg always minified though.

@jMuzsik Sorry if this is a little confusing. Are you checking the manifest.json in your local repo, or inside the static container? If the former, are you re-running gulp dist after making the changes to the gulpfile?
I added twitter.jpg to warehouse/static/images/, made the following changes:
diff --git a/Gulpfile.babel.js b/Gulpfile.babel.js
index e77754c..298ad58 100644
--- a/Gulpfile.babel.js
+++ b/Gulpfile.babel.js
@@ -215,6 +215,7 @@ gulp.task("dist:manifest", () => {
         ".ttf",
         ".otf",
         ".png",
+        ".jpg",
         ".ico",
         ".js",
       ],
@@ -246,6 +247,7 @@ gulp.task("dist:compress:br:generic", () => {
     path.join(distPath, "fonts", "*.svg"),

     path.join(distPath, "images", "*.png"),
+    path.join(distPath, "images", "*.jpg"),
     path.join(distPath, "images", "*.svg"),
     path.join(distPath, "images", "*.ico"),
   ];

running gulp dist correctly put twitter.jpg in manifest.json:
{
  "css/font-awesome.css": "css/font-awesome.628aab96.css",
  "css/warehouse.css": "css/warehouse.468a3572.css",
  "css/font-awesome.css.map": "css/font-awesome.css.4741d860.map",
  "css/warehouse.css.map": "css/warehouse.css.8db81ae8.map",
  "fonts/FontAwesome.otf": "fonts/FontAwesome.b5744b64.otf",
  "fonts/fontawesome-webfont.eot": "fonts/fontawesome-webfont.1fb8a705.eot",
  "fonts/fontawesome-webfont.svg": "fonts/fontawesome-webfont.912ec66d.svg",
  "fonts/fontawesome-webfont.ttf": "fonts/fontawesome-webfont.e1dd9adb.ttf",
  "fonts/fontawesome-webfont.woff": "fonts/fontawesome-webfont.a7bbeb58.woff",
  "fonts/fontawesome-webfont.woff2": "fonts/fontawesome-webfont.68560c51.woff2",
  "js/warehouse.js": "js/warehouse.b60a2f5f.js",
  "js/warehouse.js.map": "js/warehouse.js.0cd523c9.map",
  "js/vendor/zxcvbn.js": "js/vendor/zxcvbn.9cf6916d.js",
  "images/blue-cube-small.png": "images/blue-cube-small.c34cf0bf.png",
  "images/blue-cube.svg": "images/blue-cube.e6165d35.svg",
  "images/favicon.ico": "images/favicon.6a76275d.ico",
  "images/history-line.png": "images/history-line.ded09064.png",
  "images/logo-large.svg": "images/logo-large.87e8a3ef.svg",
  "images/logo-small.svg": "images/logo-small.972290d5.svg",
  "images/twitter.jpg": "images/twitter.21ac1c93.jpg",
  "images/white-cube-small.png": "images/white-cube-small.95381ba2.png",
  "images/white-cube.png": "images/white-cube.9f6d32b9.png",
  "images/white-cube.svg": "images/white-cube.8c3a6fe9.svg"
}

@di - no particular reason to use a .jpg. It was just the example I found in the twitter docs.  If it makes any difference, I am happy to provide a .png instead?
according to the docs:

JPG, PNG, WEBP and GIF formats are supported

@nlhkabu I think it's fine but we'll see if @jMuzsik is able to get this resolved on his end with the JPG.

Ah, sorry for taking up some of your time for this beginner problem. Thanks for the info. I was not running gulp dist, but thought that the automatic recreation that occurred in make serve was doing this for me already.
Thanks very much!

@jMuzsik Could you make the image in this PR the full-resolution image I included in the comment above? As it is, the placeholder image is too small to actually be accepted by Twitter, so testing this locally in the future with something like ngrok would fail.

Thanks @jMuzsik!

Thanks for all the help!

:tada:


Fix 'uninstall' target not removing icons on linux

As described in documentation %(Extension) already contains a dot.
https://learn.microsoft.com/en-us/visualstudio/msbuild/msbuild-well-known-item-metadata

Thanks for the fix!


Discussion - Rule 3. Use formatting and favour clarity

Would it make sense to give explicit examples of good practice and bad practice, perhaps in a contextualised way, eg show a colour highlighted git diff going from bad practice to good practice with a comment or a git commit line explaining the change in terms of the rule applied? Or maybe link to a supporting git repo where a scrappy Dockerfile has been revised into a best practice example?
When mentioning:

put each dependency on it's own line, makes it easier to spot changes in version control
split up an instruction (especially relevant for RUN) when you have to scroll to see all of it

a naive reader may misinterpret this instruction and put lots of things on separate lines each with its own RUN command, which would break layering?
So the instruction:

don't worry about image size, clarity is more important (i.e. no complex RUN instructions that remove files after using rightaway)

is problematic when it comes to writing Dockerfiles that build "efficient" images?

have commands in order of least likely to change to most likely to change, it helps readers and takes advantage of build caching

Would it make sense to have a section at the start of the paper that describes the anatomy of a Dockerfile, and perhaps also situates it in a workflow (Dockerfile -> image -> container).

Only switch directoryies with WORKDIR {-}

[typo - directoryies]
So in terms of best practice, is there something here about identifying not just which directory you are in and how to change it, but how to select appropriate USER's for running certain commands?

I personally don't think that "efficiency" in the images matters for reproducible computing environments. Rule 10 gives some advice how not to constantly re-build the image, but other than that I think a data scientist will spend way more time thinking and running a workflow than she will spend waiting for an image to build.
Re. "anatomy of a Dockerfile": good point, tried to do the minimum only in https://github.com/nuest/ten-simple-rules-dockerfiles/commit/e482b86a1fa97d1c038df9992ffb75d54a9aba21 and https://github.com/nuest/ten-simple-rules-dockerfiles/commit/d08dc3e9cf820a5f47566d4d33a400c75df2d95a
IMO splitting a RUN command does not "break" layering, it might even improve it. Still, tried to clarify.
Since you digressed from the "one discussion issue per rule", I'll close this one, feel free to re-open if important comments are not yet addressed.

Re: splitting things across RUN commands - it adds the the number of layers doesn't it? Which means it can also affect the amount fo time/bandwidth required to download an image.
If you have a build step with an rm tidy step, if you finish the RUN with the  && rm ... command, the thing that's removed is not in the layer, whereas if you do RUN ..build bits... then RUN rm ..., you do download the stuff in the first layer and you then delete it from the second?
Of course, my naive understanding of how the implemented mechanics of docker containers actually work could be completely wrong!


User deletion is broken with multiple auth providers

Now that multiple auth providers have landed, attempting to delete one's account fails:
backend_1  |   File "/venv/lib/python3.9/site-packages/anyio/_backends/_asyncio.py", line 754, in run
backend_1  |     result = context.run(func, *args)
backend_1  |   File "/app/./app/logins.py", line 735, in get_deleteuser
backend_1  |     token = models.FlathubUser.generate_token(db, user)
backend_1  |   File "/app/./app/models.py", line 32, in generate_token
backend_1  |     table.delete_hash(hasher, db, user)
backend_1  |   File "/app/./app/models.py", line 275, in delete_hash
backend_1  |     hasher.add_string(account.google_userid)
backend_1  | AttributeError: 'NoneType' object has no attribute 'google_userid'

This is because the hash generator for delete requests assumes everyone has an account of every type.

#143 is probably this same issue

I'm not sure if I've set this up wrong, but the Github deletion flow works fine but for both Google and Gitlab I'm still getting:
backend_1  |   File "/app/./app/models.py", line 104, in delete_user
backend_1  |     delete(GithubRepository).where(GithubRepository.github_account == gha.id)
backend_1  | AttributeError: 'NoneType' object has no attribute 'id'


feat(cmd): add flag that allows to read blob's data from file

It would be nice to have a flag in submitCmd that will allow to get blob's(or multiple blobs) data from the file instead of cli

hey @vgonkivs i'm working on a logic for this, may i ask is there a standard procudure for testing this type of command locally. Something like on Celestia-app repo there is a note on how to test the command so i was wandering if there is one here ?

hi @vgonkivs, can i work on this ? I think i have a functioning version for this feature

Hey @sontrinh16. Yes, sure. Thank you!


Add Sass files and config rollup

Build repo and rollup and distribute CSS

Looks good to me as well, I tried it with the Angular lib, and both SCSS and CSS works as expected


Migrate to use upstream bufferization for all the backends.

This commit also removes IREE bufferization pass.

There are HUGE regression on vulkan side. I probably do something wrong.
MobileNetV2 on pixel 4: 26 ms -> 5xx ms.

Nice! Thanks for doing this! I'd expect there are regressions (especially on the SPIR-V side).
I think the best way forward is to land this PR as a branch in IREE. This will allow Lei to surface/fix issues with the bufferization on the branch. I can help maintain this branch.

SG, I'll upload the change to a branch in IREE.

Move to https://github.com/google/iree/pull/9211 . There is a migrate-bufferization branch in IREE repo now.


Add Pre-Compiled Binary

Compiling the Code takes minutes. Is adding a pre-compiled binary an option?

@srechner Any updates on this?


[Android] Support the feature of proxy in Crosswalk

Crosswalk for android will update proxy when a PROXY_CHANGE intent is received,
but it’s need system permission to send the broadcast, there is also a workaround with
reflecting the incompatible api, but it’s too complex to maintain those codes, so
implement the new API in XWalkView to set proxy for images/text resources.
BUG=XWALK-6769

@lincsoon @xuxin @wuhengzhi PTAL.

LGTM @rakuco

The questions that must always be asked on every single chromium-crosswalk pull request and which have not been addressed here: have you tried to upstream this in any form? If not, why not? What's the maintenance plan here, are we expected to carry this patch forever?

cc @darktears

have you tried to upstream this in any form? If not, why not?

This feature is requested by one of our customers. They came to us because they couldn't realize their idea with existing solutions. First, there's no proxy configuration for Android WebView, the developer can redirect the reqeust for specific url but there's nothing they can do about video streaming, which is just what the customer wants. Second, although chromium supports proxy settings, but it's not available on Android.

What's the maintenance plan here, are we expected to carry this patch forever?

As you can see, we just added some new methods in a class instead of modifying original logic. As long as this class is at work, we want to carry on this feature as a good differentiation from default WebView.


Broken links in README.md files

Some links to the typst templates appears to be relative (?) but do not dereference correctly when used from the GitHub web ui. This change changes those links to be absolute. Suggest using squash+merge when/if merging.

Similar to https://github.com/quarto-ext/typst-templates/issues/1

Thanks !


Fix/make ap work

What does this pull request change?
Why is this pull request needed?
Issues related to this change

closes #4

blocked by https://github.com/equinor/dm-core-packages/pull/16/files
after that is merged, this pr can update package versions to the latest


iOS Landscape Rotation

Description of Change

Register when the device is rotated by the OnElementPropertyChanged property of Height and Width.
When rotated set the bounds of the screen.
When rotated set the camera orientation of the capture connection.

Bugs Fixed
This previously was a bug I raised myself but was understandably changed to a feature-request.
https://github.com/xamarin/XamarinCommunityToolkit/issues/894
API Changes
Added:

Added method in FormsCameraView.ios.cs: GetViewOrientation - Returns the AVCaptureVideoOrientation of the device.
Added method in FormsCameraView.ios.cs: SetCameraOrientation - Sets the previewLayer.Connection.VideoOrientation to the AVCaptureVideoOrientation from GetViewOrientation and then assigned the previewLayer.Connection to captureConnection.

Changed:

In method InitializeCamerain FormsCameraView.ios.cs added the line previewLayer.Connection.VideoOrientation = GetViewOrientation(); to set the correct orientation when the camera is being initialized.
In CameraViewRenderer.ios.cs - Added cases to switch in method OnElementPropertyChanged to check when Height & Width are changed ti trigger Control.SetBounds(Element.Width, Element.Height); & Control.SetCameraOrientation();

Behavioral Changes
Rotating the device from landscape to the opposite landscape flips the camera view as the renderer still doesn't register the difference in rotation change. Rotating from portrait to landscape, then to portrait and landscape again works fine.
Images are captured in landscape also.
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description) - Works in CameraView sample app
[x] Rebased on top of main at time of PR
[x] Changes adhere to coding standard

@JoelPickin, can you rebase your PR against develop?

@pictos have done now. Thanks.

@pictos have done now. Thanks.

something went wrong.
it seems you changed base branch on Github but didn't rebase in git

@pictos have done now. Thanks.

something went wrong.
it seems you changed base branch on Github but didn't rebase in git

Sorry about this. I thought I had rebased correctly. I am having some issues building at the moment. I have tried pulling down a fresh version to look at trying again, but I am getting a lot of UWP errors.

I'm going to close this PR and add my changes in a new branch and push that up as I'm not sure what the develop/rebase has done and feel it would be better starting fresh from the most recent.

New PR https://github.com/xamarin/XamarinCommunityToolkit/pull/1224


use node 14 in update docs

⚠ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.
Please include a minimal reproducible example
when opening a new issue.


Review the SymPy lecture notes page

I have a first draft of:
https://moorepants.github.io/learn-multibody-dynamics/sympy.html
It needs a look over for details. Feel free to open a pull request to improve this file:
https://github.com/moorepants/learn-multibody-dynamics/blob/master/sympy.rst

Final draft is live.

Everything it's clear and working correctly
The only probably not important remark is that one box in warnings in differentiating section is shifted to the right

The only probably not important remark is that one box in warnings in differentiating section is shifted to the right

Yeh, I noticed that too. It is some issue in the css between the theme and the jupyter-sphinx. Maybe I just need to avoid code cells in the warning/note boxes for now.
Interestingly this is what it looks like on the live website:

But it looks fine locally on my computer as I'm developing it:

I'll have to dig into the css to figure out why it is different on the live version.

Is it possible to fit the decimal places inside the box or is it an error like the above issue?

Gonna leave this one as is and call it done.


404 error in Utility classes

There's a link in the Utility classes page under CSS display that leads to a 404 error
browser: Chrome
page: http://v4-alpha.getbootstrap.com/components/utilities/#text-alignment
link: http://v4-alpha.getbootstrap.com/components/layout/responsive-utilities/
Edit: I just figured out what it should link too
http://v4-alpha.getbootstrap.com/layout/responsive-utilities/

Fixed in #20557.


bump requirements & install build-essential

Issue this PR fixes (if applicable): #185
Description:

Installs build-essential, because some pypi packages need compiling.
Fixes error when CherryPy==4.0.0 is installed that doesn't have cheroot as a requirement.
Updates requests from old old version.
Bumps patch versions of pyalsaaudio, webrtcvad - this should be only bugfixes, so no issue here.
PR against dev as this should be tested a bit by someone else than me and I don't want to do a release just because of this - let's do a release with this and the AVS audio streaming. We can do a release tomorrow if this and the streaming get tested.

Checklist:

[x] Update CHANGELOG.md
[x] PR Location (usually master for hotfixes, dev for general development)

@alexa-pi/team Please test, I'd like to release this fix this weekend with the AVS streaming. Thank you.

Now it throws a new error when it runs pip:
Leyendo la informaciÃ³n de estado...
Traceback (most recent call last):
  File "/usr/bin/pip2", line 9, in <module>
    load_entry_point('pip==1.5.6', 'console_scripts', 'pip2')()
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 356, in load_entry_point
    return get_distribution(dist).load_entry_point(group, name)
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2476, in load_entry_point
    return ep.load()
  File "/usr/lib/python2.7/dist-packages/pkg_resources.py", line 2190, in load
    ['__name__'])
  File "/usr/lib/python2.7/dist-packages/pip/__init__.py", line 74, in <module>
    from pip.vcs import git, mercurial, subversion, bazaar  # noqa
  File "/usr/lib/python2.7/dist-packages/pip/vcs/mercurial.py", line 9, in <module>
    from pip.download import path_to_url
  File "/usr/lib/python2.7/dist-packages/pip/download.py", line 25, in <module>
    from requests.compat import IncompleteRead
ImportError: cannot import name IncompleteRead

@frisco82 Please test now.

The install is working now and it did finish the authorization.
I need to fix the mic configuration as default is not working to test everything, but the installation is working for me.

@frisco82 Awesome, great to hear that!

I have installed this and verify that it is working as well.
As reported by @frisco82 the mic configuration took some work.  One of two things was necessary:
1.) change allow_unlisted_input_device to true
or
2.) copy the exact line listed from the error message into it: default:CARD=Device instead of just default
Considering this is intended to "just work" on a Raspberry Pi, I'd like to recommend changing the config.template.yaml to reflect one of those two scenarios.
Otherwise, thumbs up!

@maso27 Dope, merging!
What do you mean one of? Is the device is in the list, you don't need to allow ... Please post the list and your portion of the config. Thank you.

Sorry, I'm away from the computer now, but what I mean is that config.yaml says "default" as the input device.  However, the app fails if you leave it set like that. You have to expand the definition to what I wrote above.
Or, you can get past it by enabling allow_unlisted_input_device.
But the app is being too restrictive so it will error out without adjustment from the default.

Still unrelated to this PR, but in order to expand on the above, here's what I get when I start main.py out of the box on a rPi2:
2017-03-05 04:14:04 CRITICAL: Your input_device 'default' is invalid. Use one of the following:
null
default:CARD=Device
sysdefault:CARD=Device
front:CARD=Device,DEV=0
surround21:CARD=Device,DEV=0
surround40:CARD=Device,DEV=0
surround41:CARD=Device,DEV=0
surround50:CARD=Device,DEV=0
surround51:CARD=Device,DEV=0
surround71:CARD=Device,DEV=0
iec958:CARD=Device,DEV=0
dmix:CARD=Device,DEV=0
dsnoop:CARD=Device,DEV=0
hw:CARD=Device,DEV=0
plughw:CARD=Device,DEV=0

I'm using a simple USB microphone and the 3.5mm jack for output.

@maso27 Ah, I see. Well, it is nearly impossible to do it right due to the various audio HW out there. See #190 - hopefully that makes it better. Also, with #180, it's gonna change, so I wouldn't waste much time on this.


AttributeError: 'WebSocketProtocol' object has no attribute 'transfer_data_task'

When the client disconnects just after setting self.application_state = WebSocketState.CONNECTED, subsequent ws.close() will fail at .venv/lib/python3.11/site-packages/websockets/legacy/protocol.py:784 as transfer_data_task will not be set up.
I suppose it should be guarded by if hasattr(self, "transfer_data_task"): as it is in other places.

Thank you for your report.
It shouldn't be possible to reach this line in close() before connection_open() completes successfully. I'd like to understand why this happens and, if applicable, to fix the root cause rather than just patch the symptom.
Could you provide a bit more details, please?

Can you confirm that you talking about a client implemented with websockets?
Can you provide the full stack trace? Ideally, debug logs too? (See https://websockets.readthedocs.io/en/stable/topics/logging.html#configure-logging)
Do you have a minimum example that produces this behavior?

Plus one here:
2024-05-01 10:11:59,863 - ERROR    - Task-368  - uvicorn/protocols/websockets/websockets_impl.py - run_asgi             - Exception in ASGI application

Traceback (most recent call last):
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/uvicorn/protocols/websockets/websockets_impl.py", line 240, in run_asgi
    result = await self.app(self.scope, self.asgi_receive, self.asgi_send)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/uvicorn/middleware/proxy_headers.py", line 69, in __call__
    return await self.app(scope, receive, send)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/fastapi/applications.py", line 1054, in __call__
    await super().__call__(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/applications.py", line 123, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/middleware/errors.py", line 151, in __call__
    await self.app(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/middleware/cors.py", line 77, in __call__
    await self.app(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/middleware/exceptions.py", line 65, in __call__
    await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
    raise exc
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
    await app(scope, receive, sender)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/routing.py", line 756, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/routing.py", line 776, in app
    await route.handle(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/routing.py", line 373, in handle
    await self.app(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/routing.py", line 96, in app
    await wrap_app_handling_exceptions(app, session)(scope, receive, send)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/_exception_handler.py", line 64, in wrapped_app
    raise exc
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/_exception_handler.py", line 53, in wrapped_app
    await app(scope, receive, sender)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/routing.py", line 94, in app
    await func(session)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/fastapi/routing.py", line 348, in app
    await dependant.call(**values)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/src/myapp_backend/api_v1/routes/device.py", line 68, in websocket_endpoint  # this is my application
    async with ws_connection(ws, serial, timeout=1) as messages:
  File "/usr/lib64/python3.12/contextlib.py", line 217, in __aexit__
    await anext(self.gen)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/src/myapp_backend/api_v1/routes/websocket.py", line 80, in ws_connection
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/websockets.py", line 203, in close
    await self.send(
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/websockets.py", line 97, in send
    await self._send(message)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/starlette/_exception_handler.py", line 50, in sender
    await send(message)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib/python3.12/site-packages/uvicorn/protocols/websockets/websockets_impl.py", line 337, in asgi_send
    await self.close(code, reason)
  File "/data/myapp-yocto/myapp-appl/myapp-backend/.venv/lib64/python3.12/site-packages/websockets/legacy/protocol.py", line 788, in close
    await self.transfer_data_task
          ^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'WebSocketProtocol' object has no attribute 'transfer_data_task'. Did you mean: 'transfer_data_exc'?

This has to do with how uvicorn embeds websockets. It was integrated before I provided a good API for that use case — namely the Sans-I/O implementation. It's done by using private APIs and it doesn't respect perfectly the logic & invariants that websockets relies on. This is hard and not very valuable to fix.
Now the good API for embedding exists and it should be adopted in uvicorn: https://github.com/encode/uvicorn/issues/1908
That's the proper fix really.

PS: I don't know if that's the explanation of the first report; I never got details on that one. It's the explanation of the second report, where I have a stack trace.


[Feature]: Calendar control - events to control which month is visible in Calendar

Is this request for a new component, or additional functionality for an existing component?
Additional functionality for existing component - Calendar control
Have you validated that your desired feature or component is not already in the backlog?
YES
Describe the component and functionality you would like to see in the starter kit
A clear and concise description of what you want to happen.
For control Calendar
Would it be possible to extend this with 2 additional input events:

go to today's month (to preselect today's month when control is used in date picker custom control)
go to month of selected date (to assure that when end user manually changes date - then closes date picker - and re-opens calendar - we can start from month where we have our selected date / not from month where end user left control during last browsing session)
I would really appreciate those 2 events to be added to Calendar control - this would assure fully functional Date Picker can be built using Creator Kit Calendar control (this control would not have issues that Dataverse for Teams Fluent UI Date Picker has)

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
N/A
Additional context
Add any other context or screenshots about the feature request here.
N/A

Hi @PRusakPL,
Thanks for the suggestions regarding Calendar Control to build a fully functional Date Picker.
As Modern Controls (in preview) are natively available in Powerapps, we highly recommend you to try the DatePicker control.
Please let us know if it works for you.

Yes, we will continue to support creator kit and address every functional bug that we encounter based on the capacity.
Just so you know, Modern controls supports localization and is worth trying out.

One of the biggest issues with the Modern controls is that they do not
support theming. You cannot change colors, fonts, any visual aspects of the
controls. For me, that eliminates them from use entirely.
Regards,
Steven Perry
@.***
On Thu, Jul 13, 2023 at 8:53 AM Ramakrishnan Raman @.***>
wrote:

Yes, we will continue to support creator kit and address every functional
bug that we encounter based on the capacity.
Just so you know, Modern controls supports localization and is worth
trying out.
[image: image]
https://user-images.githubusercontent.com/11135056/253281441-db72e0aa-65be-47f4-940a-4f8abc170cfa.png
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/powercat-creator-kit/issues/302#issuecomment-1634192587,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNR6PWG5QYPFPU6X7ZSVL3XP7VUHANCNFSM6AAAAAAVTTPF7Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi @Ramakrishnan24689,
I see on your screenshot that you had to change browser language to force whole Maker environment + control to adopt Localization settings => many times (almost always) I would like to change Localization on the fly (dynamically) Modern controls do not offer this / Creator kit control they do (see below Language property)

I also agree with @sperry1625 - no theming + staying in preview (so for now we cannot use them on production) - https://learn.microsoft.com/en-us/power-apps/maker/canvas-apps/controls/modern-controls/overview-modern-controls
Is there a chance you will invest in Creator Kit Fluent UI Calendar control more ? (delivering those 2 extra events support + fixing localization in supported languages - https://github.com/microsoft/powercat-code-components/tree/main/Calendar/Calendar/loc)

One of the biggest issues with the Modern controls is that they do not support theming. You cannot change colors, fonts, any visual aspects of the controls. For me, that eliminates them from use entirely. Regards, Steven Perry @.***
…
On Thu, Jul 13, 2023 at 8:53 AM Ramakrishnan Raman @.> wrote: Yes, we will continue to support creator kit and address every functional bug that we encounter based on the capacity. Just so you know, Modern controls supports localization and is worth trying out. [image: image] https://user-images.githubusercontent.com/11135056/253281441-db72e0aa-65be-47f4-940a-4f8abc170cfa.png — Reply to this email directly, view it on GitHub <#302 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFNR6PWG5QYPFPU6X7ZSVL3XP7VUHANCNFSM6AAAAAAVTTPF7Q . You are receiving this because you are subscribed to this thread.Message ID: @.>

One of the biggest issues with the Modern controls is that they do not support theming. You cannot change colors, fonts, any visual aspects of the controls. For me, that eliminates them from use entirely. Regards, Steven Perry @.***
…
On Thu, Jul 13, 2023 at 8:53 AM Ramakrishnan Raman @.> wrote: Yes, we will continue to support creator kit and address every functional bug that we encounter based on the capacity. Just so you know, Modern controls supports localization and is worth trying out. [image: image] https://user-images.githubusercontent.com/11135056/253281441-db72e0aa-65be-47f4-940a-4f8abc170cfa.png — Reply to this email directly, view it on GitHub <#302 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AFNR6PWG5QYPFPU6X7ZSVL3XP7VUHANCNFSM6AAAAAAVTTPF7Q . You are receiving this because you are subscribed to this thread.Message ID: @.>

New Theming system will be available in upcoming weeks as stated in this blog post.
https://powerapps.microsoft.com/en-us/blog/june-2023-updates-for-modernization-in-power-apps/
This will give more flexibility to change colors and fonts.

I saw that, it will provide the same theming available today, select from several fixed OOTB themes. We use custom themes for all of our apps. I know custom theming is on the board, but it doesn't have a date yet. Until it has custom theming we cannot use them.

@Ramakrishnan24689
Theming - all clear about modern controls (thanks for references)
Can you please also work with Calendar localization that can be dynamically changed in runtime (not needing to change browser language)? Calendar is special control that is very hard for Low Code makers to localize (there must be pro code component with this feature enabled)
Please remember that in original issue I reported also lack of 2 input events:

go to today's month (to preselect today's month when control is used in date picker custom control)
go to month of selected date (to assure that when end user manually changes date - then closes date picker - and re-opens calendar - we can start from month where we have our selected date / not from month where end user left control during last browsing session)

I would really appreciate those 2 events to be added to Calendar control - this would assure fully functional Date Picker can be built using Creator Kit Calendar control (this control would not have issues that Dataverse for Teams Fluent UI Date Picker has)


Remove the default value for the integer-simple flag.

Using a default seems to require users to explicitly override the flag if they
want to enable it. Leaving it off allows Cabal to automatically figure out the
proper value of the flag, which is handy, especially when text is at the bottom
of a long dependency chain.

This change breaks the build of several packages that depend on text. integer-simple is not on hackage and so cabal build will fail. Cabal does not correctly infer the value for this.


created po_instance table

fixes #133

This says "fixes #133" which is about modifying inventory_items.  Is that possibly linking to the wrong issue?

It looks like the file name folio_reporting.po_instance.sql is incorrect.  It should probably be po_instance.sql.

Sorry it is #113 Develop derived table po_purchase_orders_instanceshttps://github.com/folio-org/folio-analytics/issues/113
From: Nassib Nassar<EMAIL_ADDRESS>Sent: Monday, December 14, 2020 10:41 AM
To: folio-org/folio-analytics<EMAIL_ADDRESS>Cc: Natalya Pikulik<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [folio-org/folio-analytics] created po_instance table (#232)
This says "fixes #133https://github.com/folio-org/folio-analytics/issues/133" which is about modifying inventory_items. Is that possibly linking to the wrong issue?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/folio-org/folio-analytics/pull/232#issuecomment-744523724, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO5MI7T375FA7ZU27XUPMODSUYWXFANCNFSM4U25EH2A.

This PR closes #113.  Please fix the incorrect file name (see above) before we do the merge.


ftputil init at 3.3

it works locally on python2, not sure why python3 is failing

You can disable the build on anything except python2, see other packages.

all green now

Travis passed, thanks.


fix markdown display in pull request preview

turns out ###-style headers need to be rewritten to use *


Labels in time scale will overlap in some cases

Version
5.0.0
Steps to reproduce
Test case:
/test/timeScale.html
/test/tooltip-link.html
Make the window narrow.
We should not depends on splitNumber. It should fit those cases automatically.

🙄🙄

We can not update echarts to version 5.x because of described problem above.
Please pay attention close to this matter.

We were waiting on version 5.1 mainly due to this problem and we are just disappointed.
We can not update echarts to version 5.x until you fix it. Please pay attention close to this matter.

Fixed in https://github.com/apache/echarts/pull/15712


Provide some sort of data structure for resources

It would be nice to have some sort of data structure for easily storing and passing around shared resources, similar to for example https://docs.rs/legion/0.4.0/legion/struct.Resources.html

I recommend using a struct. If you absolutely require dynamic typing for some reason, consider the resources crate. I'm not aware of any upside to baking something like that into hecs.

The resources crate seems like a good option for my use case. Would be nice to mention these options in the docs somewhere. The resources crate is kinda hard to find when googling (at least I couldn't find it).


SSR Nuxt Example (Nuxt 1.0.0-rc3) BROWSER_BUILD now browser

This line here in your SSR example prevents using the plugin:
https://github.com/surmon-china/vue-quill-editor/blob/master/nuxt-ssr-example/nuxt-quill-plugin.js#L4
I believe this should now be if (process.browser) { ...

See: https://github.com/nuxt/nuxt.js/releases/tag/1.0.0-rc3

@silverbackdan
Yes, is the version of nuxt.js updated, you can submit a pr, very welcome!

Sorry for the delay:
https://github.com/surmon-china/vue-quill-editor/pull/73


Add ubuntu1404-20170728

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

Replaced by 0821 image


[Feature request] Support for powerline in swaybar

Powerline is a status line plugin with support for a modded i3bar, among other things. It does not support mainline i3bar. The main requirement, as far as I know, is to have support for background colors and fallback fonts (the latter of which might be in swaybar already).

Edit: Fallback fonts are not even required, as Powerline can use themes that do not use their special characters

Fallback fonts are supported in swaybar but I don't know about background colors. Powerline is kinda what I'm doing with my status bar in sway but mine has only has the arrow tips in it

Swaybar has support for background colors through the i3bar protocol.
To do the arrows like that, swaybar would need to support per block separators with different fg/bg color by extending the i3bar protocol.
An alternative would be to add support for another protocol which is not aware of blocks like the protocol used in LemonBoy/bar (Which is the preferred way to do a powerline status bar in i3, it seems).

This is slightly irrelevant, but does swaybar support arbitrary input like i3blocks?

@ke7ofi you can pass it an arbitrary line of text if that is what you mean? You won't get colors that way though.

Feel free to keep having a discussion, but I'm not sure there's something actionable here so I'm going to close the issue.

@SirCmpwn If someone were to submit a pull request that implemented support for powerline, do you think it would be appropriate to add that feature to swaybar?

It depends on what's necessary to support it. What would have to change?

I am not 100% sure as I have not gone through the code (here is the old fork of i3bar that supports it).
This issue on
the Powerline github has some details on how it was implemented. Looks like
it was mostly adding a key to the json input for background-color and
making sure it actually changed the background color for that text.
On Thu, Mar 17, 2016 at 10:24 PM, Drew DeVault<EMAIL_ADDRESS>wrote:

It depends on what's necessary to support it. What would have to change?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/SirCmpwn/sway/issues/511#issuecomment-198184575

--
Andrew Conrad
<EMAIL_ADDRESS>

That sounds like it's supported by i3-gaps and falls under the scope of https://github.com/SirCmpwn/sway/issues/307

Oh, wow... I never saw that issue. It does look like it should work, if
Powerline was modified. Thank you for pointing me in the right direction.
On Thu, Mar 17, 2016 at 10:35 PM, Drew DeVault<EMAIL_ADDRESS>wrote:

That sounds like it's supported by i3-gaps and falls under the scope of
#307 https://github.com/SirCmpwn/sway/issues/307
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/SirCmpwn/sway/issues/511#issuecomment-198186074

--
Andrew Conrad
<EMAIL_ADDRESS>


[FEATURE REQUEST] track progress of BulkInsert w/ SqlRowsCopied

Is your feature request related to a problem? If so, please give a short summary of the problem and how the feature would resolve it
Sometimes, bulk inserts take a long time. It would be nice to be able to track the progress.
Describe the preferred solution
System.Data.SqlClient includes a SqlBulkCopy.SqlRowsCopied Event to allow users to do this on the .net side. It would be nice to bring this to the Java side as well.
https://docs.microsoft.com/en-us/dotnet/api/system.data.sqlclient.sqlbulkcopy.sqlrowscopied?view=netframework-4.8
Describe alternatives you've considered
There are other ways to track progress, but this would be the way to keep it consistent w/ the .net library.

Hi @vslee, the team will look into what can be done.


aishell reproduce

Hi,hiro,thank you for you rwork！
I'm reproducing the aishell result now,but i can't get it.Here is the conf:
conf: conf/asr/mma/lc_transformer_mma_subsample8_ma4H_ca4H_w16_from4L_64_128_64.yaml
lm_conf: conf/lm/rnnlm.yaml
And i have some doubts:
1、I check the result folder after training and find only model.epoch-8 in LM folder, when i use the LM model in the score.sh,the result is worse than not using LM model!
2、I change the LM to transformer_xl,but it only runs on CPU.
3、I add the specaug in the conf2,it raise a error in trasformer.py:line526,shape'[1,24,54,54]' is invalid for input of size 34992.
I don't konw whether i do someting still not realizing or i do something wrong or this is a bug.If you see my issue,please point out,i'm really curious,thank you.

The specaug conf is conf/data/spec_augment_speed_perturb_transformer_subsample8.yaml.

I make a mistake,i copy the exactly same conf from librispeech,after i choose the hie-added conf in the aishell,problem 1 is solved,although i still not know why.
Because i don't want to run score.sh twice for editdistance-with-lm and  editdistance-without-lm,i change the conf metric accuracy to editdistance,then raise problem 3,i'm finding the reason.
As for problem 2,i haven't try yet.


defaultCountry:auto is not triggering change onload

this means there is no way how to extract country/dial code get by auto function
fix should by easy, just add "this.telInput.change();" in the end of "_ready: function() {..."

Thanks for reporting this. Can you explain your use case?

I don't think it makes sense to trigger "change" in this case as we're still initialising the plugin. Instead I think we should support the .done() callback. See https://github.com/Bluefieldscom/intl-tel-input/issues/168.

yes you are right, I had a small javascript callback added to change extracting telcode to different field - is there better way how to do that? - send number(without telcode) in one field and telcode in another?
thanks

If you set nationalMode to true (this is the default in the latest version of the plugin), users are expected to just enter their national numbers (no international dial code). You can then extract the dial code for the currently selected country using getSelectedCountryData (see readme).


electron-builder configuration CLI overrides

Hi there,
I just started using this Github action--extremely useful, thank you for it. One piece that I found missing for my use case is the ability to pass a configuration override (e.g., -c.extraMetadata.foo=bar) directly to the electron-builder build command.
This PR accomplish that in a generic way. From a consumer's workflow yml file, a user can now pass a config_overrides object as a JSON string to this action, which will transform it into a properly formed -c.x.y.z=value config CLI arg. For example, passing:
config_overrides: >
  {
    "extraMetadata": {
      "version": "4.3.2-snapshot",
      "foo": "bar
    },
    "linux": {
      "target": "deb"
    }
  }

will be passed through to the electron-builder build command as -c.extraMetadata.version=4.3.2-snapshot -c.extraMetadata.foo=bar -c.linux.target=deb.
Let me know what you think! Thanks for the consideration,
Gabe

So is this PR still move on? it's really a lifesaver.

Any news on this? I need to overwrite the electron-builder config as well.

This can now be achieved with the args option introduced in v1.5.0.


Grading workflow enhancements

This issue is a placeholder for thinking about further bells and whistles for the grading workflow.
For example, if we want to grade the students on the quality of their commit messages, or their PRs, we could add a fields to the rubric table that allow us to specify, for any given rubric item, whether there should be a link in the "grade workflow" that opens a "drop down/close up" part of the web page (similar to the drop down/close up) thing for teams that reveals the commit history, or the list or PRs, or the issues, or whatever is relevant.
We would want to come up with a list of things that could be included in the drop down, and give the user the ability to specify those in the rubric item by key, or by clicking a checkbox, or whatever seems easiest.

I like this idea. I think you're saying that we could provide an empirical basis for our ratings by annotating GitHub repos directly?


Maven build does not bubble up the error

Description
Using Maven with SAM build does not include failure reason in log
Steps to reproduce
Modify the template in any way that makes it invalid, like add FOO to the top
Observed result
2019-03-12 12:57:36,085 [  45935]   INFO - .services.lambda.LambdaBuilder - /usr/local/bin/sam build --template /Users/alfredbr/IdeaProjects/JavaLambda/template.yaml --build-dir /private/var/folders/q5/cfv4k63519vcllnnpwz5zsj0wnjvck/T/lambdaBuild6
2019-03-12 12:57:37,073 [  46923]   INFO - .services.lambda.LambdaBuilder - 2019-03-12 12:57:37 Building resource 'HelloWorldFunction'
2019-03-12 12:57:37,503 [  47353]   INFO - .services.lambda.LambdaBuilder - 2019-03-12 12:57:37 Running JavaMavenWorkflow:CopySource
2019-03-12 12:57:37,526 [  47376]   INFO - .services.lambda.LambdaBuilder - 2019-03-12 12:57:37 Running JavaMavenWorkflow:MavenBuild
2019-03-12 12:57:39,667 [  49517]   INFO - .services.lambda.LambdaBuilder - Build Failed
2019-03-12 12:57:39,667 [  49517]   INFO - .services.lambda.LambdaBuilder - Error: JavaMavenWorkflow:MavenBuild - Maven Failed:

Expected result
2019-03-12 12:57:39,667 [  49517]   INFO - .services.lambda.LambdaBuilder - Error: JavaMavenWorkflow:MavenBuild - Maven Failed:  should include reason for failure
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: Mac
sam --version: 0.13.0

Add --debug flag to command you are running

Should be fixed in next release, closing this issue.


Space between numeric and scale

Is there a way to format output with a space between the numeric and the scale, e.g. 123 MB?

+1

@Bilge now there's one :smile: let me know if you like it