Test failures on Alpine bot in CI

The following tests have been failing consistently in the alpine bot in the ci:
not ok 31 - tty_pty
# exit code 134
# Output from process `tty_pty`:
# Assertion failed in test/test-tty.c on line 369: 0 == uv_tty_init(&loop, &master_tty, master_fd, 0)

not ok 101 - udp_multicast_join6
# exit code 134
# Output from process `udp_multicast_join6`:
# Assertion failed in test/test-udp-multicast-join6.c on line 135: r == 0

not ok 112 - pipe_getsockname_abstract
# exit code 134
# Output from process `pipe_getsockname_abstract`:
# Assertion failed in test/test-pipe-getsockname.c on line 191: memcmp(buf, abstract_pipe, sizeof abstract_pipe) == 0

not ok 192 - poll_bad_fdtype
# exit code 134
# Output from process `poll_bad_fdtype`:
# Assertion failed in test/test-poll.c on line 607: 0 != uv_poll_init(uv_default_loop(), &poll_handle, fd)

not ok 194 - socket_buffer_size
# exit code 134
# Output from process `socket_buffer_size`:
# Assertion failed in test/test-socket-buffer-size.c on line 52: value == 10000 || value == 20000

I tried to reproduce the issue in an alpine docker image on a Debian Jessie host with no luck. All the tests passed correctly.
After getting access to the Alpine bot and with the help of @jbergstroem we found out that some syscalls were behaving differently on the docker image than on the bot and that the reason for that could be that the docker image is using the Debian kernel whereas the Alpine bot, that is running on a SmartOS LX Zone, uses whatever kernel it uses, and they behave differently.
As an example: in the poll_bad_fdtype test, polling on a directory is not supported on Linux, but apparently is supported on the Alpine running on the SmartOS. To confirm this theory, we started a Debian Jessie instance on a SmartOS zone, and the same tests failed (well, all of them but the tty_pty).
In case this explanation is correct, should we do something about it? The options I can think of:

Do nothing. It's a tier 2 platform so it doesn't have to be run on the CI.
Ask the nodejs build group to move the bot to some other container platform that behaves correctly.

/cc @libuv/collaborators @libuv/sunos

Ask the nodejs build group to move the bot to some other container platform that behaves correctly.

I'd do this.  Alpine is based on musl so I guess it's a good idea to have it around.

+1 to moving to another container

Another +1 for moving.  LX Zone is based on an ancient version of KVM, not surprising it's buggy.

I'll look at moving it; joyent is the only platform that gives us alpine out of the box :( It'd be pretty awesome if we had a sponsor that gave us the possibility to provide a iso.

@geek, @chorrell: Are there any plans of improving the lx zone in smartos?

@bnoordhuis

LX Zone is based on an ancient version of KVM

LX zones have nothing to do with KVM, they are a fundamentally different approach to virtualization. You might want to do some research about them before making any recommendation in this thread.
@santigimeno @jbergstroem There are bugs in LX, and the LX maintainers usually fix them fairly quickly. I will forward that info to the LX maintainers, and I'll report their assessment of the problem.

Don't get your panties in a bunch.  KVM-based or not, it's still crappy and we have better things to do than chase down bugs in the platform.
@jbergstroem What would you need?  I can probably get IBM to donate some resources.

@bnoordhuis can we upload custom install iso's to softlayer? we're hitting limit at our spend there but i reckon two minimal setups would fit.

@mhdawson Do you happen to know the answer to Johan's question?

Afaik here's a list of the supported OS'es: http://www.softlayer.com/software. There's this but yeah, I don't think that's going to be a way to get alpine going: https://knowledgelayer.softlayer.com/procedure/import-image

I also found this https://knowledgelayer.softlayer.com/procedure/mounting-local-iso-ipmi which has some instructions for mounting an iso.  So I'm thinking it would be possible.  However, I'm thinking it might be easier though to use something like ubuntu running docker, I think that was are original plan for alpine.

@jbergstroem Yes, lx-brand is under active development and the engineers are regularly working to improve it. I'll pass on the relevant failures to our engineers to take a look. It's quite possible this was already fixed but the instance being used is on an older platform.
I think, maybe a year ago, you and I had a discussion about whether to use the lx-brand image for libuv testing. I'm pretty sure it was in a PR or issue either under build or LTS but I can't find it for the life of me. We also chatted on IRC :). It's so long ago that it probably doesn't matter at this point but I expressed some concern about testing libuv under lx-brand since the test would more than likely hit some lx-brand related quirks.
Ultimately the take away was that I was going to produce a KVM (non lx-brand) image that would be available in the Joyent Public Cloud that could be used instead. There's no firm ETA for that, but it's on my roadmap. I think for the short term I would go with what @mhdawson suggests and use Ubuntu + Docker. That seems like the quickest solution and given that one of the main use cases for Alpine is to use it in the docker-node image, I think that works out nicely.

@chorrell just checked our logs and yeah; the consensus was that the best scenario was using alpine on baremetal [at joyent]. I will look at expanding our "alpine through docker" testing instead.

Also, when we were discussing this was mainly about node where tests basically passed for this specific combo.

That sounds about right.

Also fwiw I spun up an alpine-3 lx-brand instance and ran the tests (per https://github.com/libuv/libuv#running-tests) and I got the same failures except for 31 - tty_pty. My instance is more than likely on a newer platform (joyent_20161110T013016Z) than the one you have in you CI setup which probably explains why I got ok 31 - tty_pty
I'll follow up with our engineers about those failures.

I raised the issue (RE the failing libuv tests on lx-brand) with our engineers. I don't have an ETA or time frame yet as to when they'll be able to dig into this though.

Can you describe the setup @chorrell you are using?

I'm using an lx-brand Alpine Linux 3.5 instance (release 20161222) in the Joyent Public Cloud. Specifically, an instance in the us-east-3b DC. That is our "beta" DC that get's updated on a daily basis and will always get the latest patches and updates.
An lx-brand instance is basically a SmartOS container pretending to have a Linux kernel. There are better explanations though:
http://www.slideshare.net/bcantrill/illumos-lx
http://www.slideshare.net/mrbruning/lxbrand
I suspect that the libuv test failures are either a result of a bug in the kernel emulation or the specific tests rely on something that's not supported yet under lx-brand.
 Output will be in /var/log/mdata-user-script.log
./gyp_uv.py -f make
make -C out
./out/Debug/run-tests

O you do work at joyent... is there a possibility to get a box to play around with?
I wrote the socket_buffer_size test and it is failing. My guess is that it just doesn't allow to set a new buffer size or the default maximum settings are smaller than what is being set. However, it is hard for me to debug because I do not have a box running with smartos.

@chorrell Thanks for explaining that.  Personally I'd rather have a real Linux kernel under the hood, to rule out any possible emulation layer problem in our CI environment.  That way we can focus on the real problem instead of wondering if there is an issue in the emulation layer.
My 2 cents.

For now, docker is the best we can get. The current instance is lacking dependecies so I need to fix that first.

@saghul Yes, that was the same conclusion @jbergstroem and I came to last year (see https://github.com/nodejs/build/pull/476#issuecomment-243144629).
I think the short term goal is to use docker for the Alpine tests makes the most sense. Eventually you should be able to use a KVM Alpine image once I can make that available. I don't have a timeline for that though. Alpine is pretty different from CentOS, Debian etc, so it looks like I might need to come up with a new build process for it.

@txdv Sure, I can probably work something out. I'm usually on IRC in #node-build (chorrell)

@chorrell @jbergstroem Thanks for working on making this better! ❤

I think this can be closed now that the alpine bot has been removed.


"New Terminal with Profile" doesn't work when there's no opened tabs

Cannot select any of the profiles if there's no opened tabs on "New terminal with profile" from upper left corner - on Termius 1.0.81 with Windows 10 1903.

Please capture DevTools console logs at the time of the issue: https://github.com/Eugeny/terminus/wiki/Debug-logs

There's no console error output....
It's just when you start the program with no tabs opened, by clicking on "open terminal with profile", none of the drop-down items can be selected.

Can confirm - if all tabs are closed and the welcome screen is shown you can open the drop-down but clicking on the individual profiles has no effect

Can you check whether this happens with a freshly created profile?


kh and kd as elementary functions

Changes
The iapws_G704_kh and iapws_G704_kd functions might turned into elementary functions in order to improve performance when looping over a range of temperatures.
Currently, those two functions work only on scalar inputs for temperature. If a range of temperatures needs to be computed a loop is needed in the calling the procedure.
Code modification
iapwsG704.f90
This change would imply that the functions:

iapws_kh
iapws_kd
iapwsG704_kh_water
iapwsG704_kh_heavywater
iapwsG704_kd_water
iapwsG704_kd_heavywater

will become subroutines that need to accept an assumed-shape array of rank-1 for the temperature T(:) and an assumed-shape array for the result kh(:).
iapws_capi
By cascading those modifications to the C API, they would imply that the functions:

iapws_capi_kh
iapws_capi_kd

would also become subroutines that need also to accept an assumed-shape array of rank-1 for the temperature T(:) and an assumed-shape array for the result kh(:).
The C header needs to be modified:

turn all functions into void functions
change scalar T to a pointer *T
add a pointer for the result *kh
add a scalar for the size of the array size_t n

Python Wrapper
The calling python functions will not change. The changes needs to be done in the C extension on how to handle input data coming from Python.

send reference for scalar inputs
send buffer pointer for array inputs.

Tested with elemental functions. No performance gain.


Refactor cd to cwd?

We use cd for current directory, but in CLI it means change directory. Maybe we should switch to cwd.
For convenience, both should be available for some versions using a deprication warning.

I think that it worth to think about it. I think it sometimes also depends on the function itself. Hence, we need to check if cd always is used as current working directory (cwd) .

working_directory seems more clear, as cwd could confuse with the cwd of python.


Bad card margin when title is not used

If I don't want to specify the title, the card will have a bad margin and "merges" with the previous.
Bad behavior w/o title:

Expected behavior:

Unfortunately the developer isn't really actively maintaining this card and the header has never really worked properly. I use custom:card-mod as a workaround for this problem with the style parameter like so:
  - type: 'custom:weather-card-chart'
    style: |
      .card-header {
        padding: 20px 16px 10px;
        line-height: 32px;
      }
    title: Weather
    weather: weather.smartweather_home
    sun: sun.sun
    mode: hourly
    temp: sensor.smartweatherudp_temperature
    wind: sensor.smartweather_wind_gust


Fix - Issue #696

Fixes an issue with the User Profile Sync Service

This change is

Codecov Report

Merging #698 into dev will decrease coverage by <1%.
The diff coverage is 57%.

@@          Coverage Diff          @@
##             dev    #698   +/-   ##
=====================================
- Coverage     90%     90%   -1%
=====================================
  Files        105     105
  Lines      10069   10072    +3
  Branches       1       1
=====================================
- Hits        9158    9141   -17
- Misses       910     930   +20
  Partials       1       1

@NikCharlebois Test are failing. Can you please fix them?

One comment after a quick review:
For other resources we had to switch from the TypeName property to GetType().Name, because the TypeName was language specific and the GetType().Name wasn't. Please make sure you test this change on a non-English installation or keep using the GetType().Name method/property.

Linking to issue #696

@ykuijs ok, I wasn't aware of the culture issues with service apps typename. I flipped back to .GetType() and provided the proper service type name instead.

Just one typo. Can you also please update the Changelog.md file?

Reviewed 1 of 2 files at r3.
Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.

Modules/SharePointDsc/DSCResources/MSFT_SPUserProfileSyncService/MSFT_SPUserProfileSyncService.psm1, line 275 at r3 (raw file):

                if ($null -eq $ups)
                {
                    throw [Exception] ("No user profile service Service Application was found " + `

Has the word Service twice

Comments from Reviewable

Fixed

Review status: 1 of 2 files reviewed at latest revision, 1 unresolved discussion.

Modules/SharePointDsc/DSCResources/MSFT_SPUserProfileSyncService/MSFT_SPUserProfileSyncService.psm1, line 275 at r3 (raw file):
Previously, ykuijs (Yorick Kuijs) wrote…
Has the word Service twice

Fixed

Comments from Reviewable

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


python-packages: update elasticsearch-1.4.0 to elasticsearch-1.6.0

The signature is fixes... thank you for your patience :sweat_smile:


Update build creation endpoint

Updates build creation endpoint to no longer use the deprecated repos route, and also adds support for token-only build creation. Closes #16

This has been tested agains production and is ready to go!


Remove bandit lint tool

I'm not convinced this has demonstrated its benefit to the code base.
When introduced, it changed one assert usage to a raise -- that's it --
which doesn't seem all that interesting of a change. Beyond that, this
tool requires many useless "nosec" comments for each suprocess call,
which I find distracting.
The tool itself adds overhead -- even if small -- to all contributor's
test runs and CI. IMO, we're better off without it.
Contributor checklist

[ ] Provided the tests for the changes. (n/a)
[ ] Gave a clear one-line description in the PR (that the maintainers can add to CHANGELOG.md on release). (n/a)
[ ] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

In practice, since the introduction, do you feel this has caught a single legitimate security issue? If so, do you have any details?
Any thoughts on why more Python projects on using this tool?

Hi there, that is an interesting one.
I presume we all agree that security is important and needs to be taken seriously. As I understand the argument, we are questioning if bandit contributes to this goal in a meaningful way. The hypothesis, it only adds a slight overhead, without any measurable outcome.
I would like to propose another way to look that this. I would suggest adding a single comment that alerts both author and reviewer to potential security implications. Following that thought, I do not agree that this tools adds overhead, but shines a spotlight on code that needs additional attention to provide another layer of security.
Moreover, I believe that the argument can not only include the past, but must consider the future. Just because we didn't have any security issues in the past doesn't mean we don't have any in the future. We must also consider, that bandit has prevented potential security risks in the past without our knowledge.
In conclusion, I love to delete useless code, just like the next developer, but in this particular case I must disagree. bandit pulls its weight and rather sacrifice a contributors time over security.

Yes, we all agree with the goal, but this tool strikes me more as theater rather than something of substance.
Any reasonable code review should take security issues into consideration whether bandit is in use or isn't.

@jdufresne bandit is a linter. Just like any other linter, it's an assistive technology. It should help reviewers and implementers not oversee rookie mistakes. The reviewer should pay attention to whether the ignores are fair, linter-wise. Of course, they need to also take into account other things but w/o automations like this, it adds more mental overhead to the process.

Yes, I understand what the tool is and its intent. I still think it isn't a very useful for this project as I don't find it to be effective at achieving its goals and I don't agree that it reduces mental overhead. I think it increases it. I had to learn what "nosec" means. I had to investigate what the tool is all about. :shrug:
Since its introduction, it has created two concrete changes:

Changed assert statements into explicit exceptions. This is a horizontal move and is really an opinion on style more so than anything security related. Yes, I'm aware assert can be optimized away.

Forced every usage of subprocess to include a noisy # nosec comment. This I consider theater and an annoyance. This isn't making anyone rethink the use of subprocess. If someone were to analyze the project's usage of they would see that there isn't anything alarming about its use.

That's it. Reading through the rest of its rules, I don't find them interesting for the purposes of this project.

@jdufresne this sounds a lot like "since it didn't help me, there's no way it would help anybody else". I don't think everybody who faced a legitimate linter failure and fixed it would be posting about the success on an issue tracker.

Forced every usage of subprocess to include a noisy # nosec comment. This I consider theater and an annoyance. This isn't making anyone rethink the use of subprocess.

This again is an instance of "thinking annoys me therefore everybody must turn their brains off". This I consider theater and an annoyance. It really does make other people rethink the use of subprocess, just not you. And this is fine. One of the conclusions could be that any use of subprocess should be put in a separate module/subpackage and abstracted away from the rest of the code. This separate place would need to be watched more closely and probably implement some safeguards. It would be the only place having the "annoying" ignore comments but would have to have really comprehensive comments describing the rationale. And for the rest of the project, it would simply be forbidden to use subprocess directly. This way of separation of concerns can be a great positive impact of the bandit tool if used wisely.

If someone were to analyze the project's usage of they would see that there isn't anything alarming about its use.

And that is why those comments are in place. Too bad that they don't have the motivation written alongside, though.

"since it didn't help me, there's no way it would help anybody else"

...

"thinking annoys me therefore everybody must turn their brains off"

No, this does not describe my position or how I think. With all due respect, I find the tone of this last comment a bit insulting.

I presented this PR, as a frequent contributor, in good faith, in an attempt to improve the tooling around the project and ease contributions for myself and others. I understand you disagree and that is fine, but these accusations like "thinking annoys me" and "therefore everybody must turn their brains off" aren't helpful or welcome.

@jdufresne your contribution is much appreciated, regardless of my opinion on this matter.
I believe an argument has been made and there isn't much to add, that furthers the cause without risking being hurtful.
At the end, this seems to boil down to a difference in values and people will come to a different conclusion even if presented with the same evidence. Therefore, I would advise we leave it like that for now.
@jdufresne it is up to you to withdraw you pull-request, but my advice: It's probably not worth the effort and will make you unhappy, which I don't wish on anyone.

@jdufresne I apologize for making you feel that way. I guess what I was trying to communicate is that there's a useful impact of bandit that may be perceived subjectively depending on personal preferences but nevertheless, there's a way of making it less annoying (and I personally think that it's a good practice) by quarantining any potentially dangerous invocations in a separate location and disabling rules that everyone agrees on as misleading. This would also improve the structural aspect of the app (and its maintainability).
P.S. Calling things theatrical and diminishing their value is far from being constructive and it is what felt insulting to me in the first place. I think that's why overreacted. Apologies again.

Closing as this is going nowhere and clutters the list of open PRs. Usually changes that are likely to be controversial are better to be discussed with other cores before being proposed. I made similar mistakes myself and I know the feeling of investing time and not being able to merge my patch due to failure to gather support.
To everyone involved, be cool and don't assume bad intentions, sometimes the language/culture barrier can be an extra challenge we have to face.


question: adding unrar support

hi, is there any chance you would be able to add unrar support for unix?
i have a source for media but most comes rared so need to auto unpack before script runs
Thanks

Use NZBGet.

I use private torrent source for media
Deluge has an auto unpack plugin but it works on torrent complete same as the script so I believe they interfere with each other
Sent from my Samsung device
-------- Original message --------
From: M4X-StL<EMAIL_ADDRESS>Date: 04/11/2015  21:21  (GMT+00:00)
To: mdhiggins/sickbeard_mp4_automator<EMAIL_ADDRESS>Cc: ptowns188<EMAIL_ADDRESS>Subject: Re: [sickbeard_mp4_automator] question: adding unrar support (#361)
Use NZBGet.
—
Reply to this email directly or view it on GitHub.

Best thing to do would be to just let whatever download manager (CP, SB, Sonarr, SR) handle the unraring, rather than try to reinvent the wheel since all of these programs support it, and just disable converting post-deluge and let it be done right before tagging.

I believe Sonarr and CouchPotato both have quality archive management, if you disable the convert option with respect to deluge (or whatever downloader) and just let the conversion take place as part of the postCouchPotato.py process you should be good. Gonna close this for now.


Hide Default Configuration

Is there an existing issue for this?

[X] I have searched the existing issues

The question
How do i hide the white block inside the editor (This is TheHTML) ?
i managed to hide from the toolBar by setting  showCodeBlock: false, but did figure out how to hide it from the editor.
this is my code:
Column(
children: [
QuillSimpleToolbar(
controller: controller,
configurations: const QuillSimpleToolbarConfigurations(
showCodeBlock: false,
),
),
const SizedBox(height: 10),
QuillEditor.basic(
controller: controller,
configurations: const QuillEditorConfigurations(minHeight: 300),
),
],
)

I have run into this before #2073
I assume you are importing/pasting text from a source that is providing HTML.
In my cases, the paste process converted
 to the code block attribute that appears as your white box.
Since I don't use HTML and the status of pasting HTML is being deprecated, I have not pursued this further.
@CatHood0 knows more about this than I do. Can you confirm that this is what you are doing?

I'll first add a new version for flutter_quill_delta_from_html to allow plain paste from html nodes and then we will need to add a new configuration for this in the editor. Give some time to make this.

This is the html i'm trying to display inside the editor:
"This is the HTML section of the email body."
it does not include
and this is how i convert it:
_quillCtrl = QuillController(
  document: Document.fromDelta(HtmlToDelta().convert("This is the HTML section of the email body.")),
  selection: const TextSelection.collapsed(offset: 0),
);

We know it. But,this is the default behavior. As soon as possible this will be changed

This is the HTML section of the email body.

Do you want to remove the white box? In this case, remove the  tag and you should not have a problem. It seems that the Quill editor is working correctly and showing the  tag as a code-block (block attribute).
Do you want to show the box as a different color and inline? In this case you want to convert the  tag to an inline code-block (inline attribute). It is confusing that there are 2 attributes for code blocks and they have very different properties.

@AtlasAutocode
Yes i want to remove the white box.
i was just confused of this box because i've been using this package for a while now, and it's the first time i see this box (for the same html content).
i guess they made some changes recently,
Anyway, Thank you for your help, I appreciate it.


Add a tool script to help check for unused runners

This pull request includes:

[X] adequate testing for the new functionality or fixed issue
[X] adequate documentation informing people about the change such as

[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Not sure if this should be run on PRs or not, but it should help in keeping Schutzfile cleaned up.

Hrm, looks like this made things go sideways. But I'm not sure why...

We can always assume that centos and centos-stream are the same thing. I hate how the ID in os-release for CentOS Stream is centos.

Rebased to prevent OCI private key leaking. Apologies :(

Rebased on main, currently it is showing:
rhel-8.9
rhel-9.3
as unused. Is this on accident? Or is there something other than .gitlab.yml using these?

Fells like this script should run periodically as part of our CI?

@bcl would you mind resolving the conflict, so that we can merge this? Thanks in advance.


How to remove FOLLOW:GITHUB FEED in the footer

I don't like FOLLOW:GITHUB FEED in the footer and i want to remove them. However, i don't know how to deal with it. Could someone help me and give some necessary details? Thank you so much.

There are two sources for these:

The "Follow" text and the social media icons (i.e., "GitHub" in the above example, are produced when a relevant social media username is supplied in _config.yml. So in your case, removing "academicpages" as the username in your configuration should disable that showing.
The second item, "Feed" is an RSS feed that allows people to subscribe to updates on your site. At this time there's not an easy way of disabling that without editing the code. I've created issue #1936 to address that and should have the code updated in the next couple days.

If RSS is disabled and the user does not have a GitHub or Bitbucket account, the "Follow" line in the footer is automatically removed by the template.

Thanks a lot !

这些有两个来源：

“关注”文本和社交媒体图标（即上例中的“GitHub”）是在 中提供相关社交媒体用户名时生成的_config.yml。因此，在您的情况下，删除配置中的用户名“academicpages”应禁用那个表现。
第二项“Feed”是 RSS 提要，允许人们订阅您网站上的更新。目前，没有一种简单的方法可以在不编辑代码的情况下禁用它。我已经创建了问题允许用户在页脚中禁用 RSS feed  #1936来解决这个问题，并且应该在接下来的几天内更新代码。

Thanks a lot !

There are two sources for these:

The "Follow" text and the social media icons (i.e., "GitHub" in the above example, are produced when a relevant social media username is supplied in _config.yml. So in your case, removing "academicpages" as the username in your configuration should disable that showing.
The second item, "Feed" is an RSS feed that allows people to subscribe to updates on your site. At this time there's not an easy way of disabling that without editing the code. I've created issue Allow users to disable RSS feed in the footer #1936 to address that and should have the code updated in the next couple days.

Thanks a lot !


Possible to implement TranslocoLoader per scope?

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[x] Support request
[ ] Other... Please describe:

Current behavior
Is it possible to implement a scoped loader for each lazy loaded feature module I have? I realize you can do es6 import loaders, but I would like to implement a separate TranslocoLoader for each lazy loaded feature module.
We are implementing Transloco on a massive mono-repo application where each feature module is managed by a separate team of developers with translations pulled from different backend systems.
Expected behavior
To be able to implement a separate TranslocoLoader per provider scope.
In a perfect world I could do this:
// my-feature.loader.ts
import { Injectable } from '@angular/core';
import { TRANSLOCO_LOADER, TranslocoLoader } from '@ngneat/transloco';
import { of } from 'rxjs';

@Injectable({ providedIn: 'root' })
export class MyFeatureLoader implements TranslocoLoader {
  constructor() {}

  getTranslation(langPath: string) {
    return of({ something: 'is' });
  }

}

export const myFeatureLoader = { provide: TRANSLOCO_LOADER, useClass: MyFeatureLoader };

// my-feature.module.ts
...
@NgModule({
  imports: [CommonModule, TranslocoModule],
  providers: [{ provide: TRANSLOCO_SCOPE, useValue: { scope: 'myFeature', myFeatureLoader } }],
})
export class MyFeatureModule {}

Minimal reproduction of the problem with instructions
For bug reports please provide the STEPS TO REPRODUCE and if possible a MINIMAL DEMO of the problem, for that you could use our stackblitz example
What is the motivation / use case for changing the behavior?

Environment

Angular version: 7.2.5

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [ ] Chrome (iOS) version XX
- [ ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX

For Tooling issues:
- Node version: 12.12.0
- Platform: Mac

Others:

I'm currently having to manually load the translations within the constructor. Would prefer to have the ability to implement a TranslocoLoader per feature module. I tried the inline loader route but could never get it to work.

Hi @wesleygrimes you could did you try using the inline loader with useFactory?
export const scope = function(http: HttpClient) {
  const loader =  ['en', 'es'].reduce((acc, lang) => {
    acc[lang] = () => http.get(
      'path/to/translation',
      { params: { fromObject: { lang } } }
    );
    return acc;
  }, {});
  return {scope: 'my-scope', loader: loader}
}

@NgModule({
  imports: [TranslocoModule],
  providers: [
    {
      provide: TRANSLOCO_SCOPE,
      deps: [HttpClient],
      useFactory: loader
    }
  ],
  declarations: [YourComponent],
  exports: [YourComponent]
})
export class FeatureModule {}

This works, almost, but I have to return a Promise not an Observable. Thanks!


Fix weird display with negative bar chart values (Issue #9)

See closed PR https://github.com/nnnick/Chart.js/pull/669 for more details.
This one was created to sync up with the dev branch, instead of master.
Working example at https://jsfiddle.net/ahrex/jy8nnaem/

We've recently announced a feature complete status for v1.x and will only be accepting bug fixes for v1.x. Be sure to take a look at v2.0-dev and it's new features that are either finished or currently under development.  If you see your feature is still not implemented, feel free to submit another PR to the v2.0-dev (or later) branch for your feature.

Sorry, but that the negative values are not working only on Bar Charts in 1.x is a bug. Please add this as bugfix to 1.x. I tested it and it's working fine.
Currently there is no chance to use it, because v2 is not stable and in 1.x it is not working! That's very bad!

+1 For iCodr8

if it's really just not working for bar charts, it's definitely a bug. +1 @iCodr8

+1 @iCodr8

+1 @iCodr8

+1 @iCodr8


Suggestion for the README: SQL Connection

Since SQLcl is running within the Docker host it's easy to see if works. Moreover SQL Developer is running outside so there is not the same challenges. It would be nice to have on the README a session about how to use in SQL Developer
Also, thanks for the project :)

Hi @cvrnogueira,
Thanks a lot for using these images!
Unfortunately, due to the nature of this being a Docker, Podman, Kubernetes, etc. container, there are many different ways to expose access to the container, and then there are many different tools to connect to an Oracle Database.
Documenting all the different variations would make quite a comprehensive documentation which I am not really qualified to do.
For the most part, all of these should behave as if this is just another Oracle Database running on the local environment, with the exception that port 1521 has to be exposed and not be blocked by a firewall or similar.
For SQL Developer-specific questions, I know that https://twitter.com/thatjeffsmith has always had an open ear for questions and and is eager to help.


Dummy implementation of language server and vscode extension [phab:D1744]

Created by djrenren at 2017-06-28 01:36:26:
This change sets up the scaffolding to build a kythe-integrated language server

djrenren wrote at 2017-06-28 01:37:25:
Feel free to add anyone who might be relevant to the CL. Also I'm just seeing the no newline warnings from Phabricator. Should I fix that?

danielmoy-google wrote at 2017-06-28 17:36:43:

! In (earlier comment), djrenren wrote:
Feel free to add anyone who might be relevant to the CL. Also I'm just seeing the no newline warnings from Phabricator. Should I fix that?

RE: newlines
Generally yes, code should have newline at the very end.
That said I think google style guides specifically say that humans shouldn't bother making review comments about missing newlines.  Might as well fix it though, 'cus both phab and eventual import into google3 will complain about it until it's fixed.

djrenren wrote at 2017-06-28 20:56:13:

Add newlines to the end of files

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
nit: whitespace

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
nit: two spaces between content and comments (also: doesn't JSON normally forbid comments?)

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
"the our"?

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
nit: long line

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
nit: long line, here and elsewhere

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
nit: Kythe

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
This didn't work here (or in the other README).

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
Mention install here and in the other README.

zarko commented inline (context lost during migration) at 2017-06-28 21:42:39:
Got this error, approximately:
Property publisher is mandatory and must be of type string.

djrenren wrote at 2017-06-28 22:03:25:

Address nits and fix compile error due to publisher name

djrenren commented inline (context lost during migration) at 2017-06-28 22:34:11:
Yeah, but vscode uses it for their config files so they extended it to allow comments :/

zarko accepted this revision at 2017-06-28 23:16:08

zarko closed via commit bb2215d16f6211acd8f582107bb6cb9d05cc20e0 at 2017-06-28 23:16:43


Flutter web project firebase hosting error: webOnlyScheduleFrameCallback

I am getting this error after building project and hosting it using firebase hosting.
I tried building project  using flutter build web and webdev build and got the same error using both.
Here is the inspect element console log:
s_helper.dart:1340 Uncaught Exception: webOnlyScheduleFrameCallback must be initialized first.
    at Object.d (https://website.firebaseapp.com/main.dart.js:578:3)
    at uI.ds (https://website.firebaseapp.com/main.dart.js:30556:9)
    at Dl.ds (https://website.firebaseapp.com/main.dart.js:21372:7)
    at Dl.th (https://website.firebaseapp.com/main.dart.js:21368:57)
    at Object.eval (eval at Kc (https://website.firebaseapp.com/main.dart.js:710:8), <anonymous>:1:53)
    at Dl.c9 (https://website.firebaseapp.com/main.dart.js:23282:29)
    at Dl.xk (https://website.firebaseapp.com/main.dart.js:15638:3)
    at https://website.firebaseapp.com/main.dart.js:6494:307
    at Hu.a (https://website.firebaseapp.com/main.dart.js:1356:71)
    at Hu.$2 (https://website.firebaseapp.com/main.dart.js:11964:23)

Here is flutter doctor -v output
[✓] Flutter (Channel dev, v1.10.15, on Mac OS X 10.14.6 18G95, locale en-IN)
    • Flutter version 1.10.15 at /Users/development/flutter
    • Framework revision fbabb264e0 (9 days ago), 2019-11-02 01:36:07 +0300
    • Engine revision 8ea19b1c76
    • Dart version 2.6.0 (build 2.6.0-dev.8.2 bbe2ac28c9)

[!] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at /Users/Library/Android/sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin
        usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install:
        sudo gem install cocoapods

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.4)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 35.3.1
    • Dart plugin version 183.6270
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1343-b01)

[✓] IntelliJ IDEA Community Edition (version 2017.3.5)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 29.0.1
    • Dart plugin version 173.4700

[✓] Connected device (2 available)
    • Chrome     • chrome     • web-javascript • Google Chrome 78.0.3904.87
    • Web Server • web-server • web-javascript • Flutter Tools

! Doctor found issues in 2 categories.

My pubspec.yaml is:
name: flutter_website
description: An app built using Flutter for web

environment:
  # You must be using Flutter >=1.5.0 or Dart >=2.3.0
  sdk: '>=2.3.0-dev.0.1 <3.0.0'

dependencies:
  flutter_web: any
  flutter_web_ui: any

dev_dependencies:
  build_runner: ^1.7.2
  build_web_compilers: ^2.0.0
  pedantic: ^1.0.0

dependency_overrides:
  flutter_web:
    git:
      url: https://github.com/flutter/flutter_web
      path: packages/flutter_web
  flutter_web_ui:
    git:
      url: https://github.com/flutter/flutter_web
      path: packages/flutter_web_ui

Please suggest if I am doing something wrong from my side or is it a bug.

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

@arpit-it the flutter_web package is deprecated since support has been merged back into the main project. To create a flutter application targeting the web, see the instructions at flutter.dev/web


Golang Tensorflow v2.2.0 fails install

System information

OS Platform and Distribution (Macos catalina 10.15.4):
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):
TensorFlow version:
Python version:
Installed using virtualenv? pip? conda?:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

Describe the problem
Provide the exact sequence of commands / steps that you executed before running into the problem
Following https://github.com/tensorflow/tensorflow/blob/master/tensorflow/go/README.mdhttps://github.com/tensorflow/tensorflow/blob/master/tensorflow/go/README.md
go generate github.com/tensorflow/tensorflow/tensorflow/go/op
fails with failed to process "api_def_VarHandleOp.pbtxt": Attribute allowed_devices not defined in base api for VarHandleOp exit status 1 go/src/github.com/tensorflow/tensorflow/tensorflow/go/op/generate.go:18: running "go": exit status 1
go test github.com/tensorflow/tensorflow/tensorflow/go passes
building or running go code using tensorflow then proceeds to fail with the following output
go: finding github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto latest go: finding github.com/tensorflow/tensorflow/tensorflow/go/core latest go: finding github.com/tensorflow/tensorflow/tensorflow/go latest go: finding github.com/tensorflow/tensorflow/tensorflow latest build command-line-arguments: cannot load github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto: cannot find module providing package github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto
the code run is the example that prints tf library version from https://www.tensorflow.org/install/lang_go
Any other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.

FROM golang:1.13 AS builder WORKDIR /app COPY libtf22.tar.gz /app/libtf22.tar.gz RUN tar -C /usr/local -xzf libtf22.tar.gz RUN ldconfig
Running in Docker with the above Dockerfile, where libtf22.tar.gz is a libtensorflow build from from the v2.2.0 tag. (Which seems to work, if i use that tar.gz file and install tf 2.1 via go get, and run the tf_hello.go example, i get hello from version 2.2).
doing go get -d github.com/tensorflow/tensorflow/tensorflow/go
Produces
package github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto: cannot find package "github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto" in any of: 	/usr/local/go/src/github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto (from $GOROOT) 	/go/src/github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto (from $GOPATH)
the files are downloaded to /${GOPATH}/src/github.com/tensorflow though
Skipping Step 2 build the TensorFlow C Library
cd ${GOPATH}/src/github.com/tensorflow/tensorflow ./configure bazel build -c opt //tensorflow:libtensorflow.so
As it has already been build from, following: https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/lib_package/README.md
go generate github.com/tensorflow/tensorflow/tensorflow/go/op in linux container still fails with:
failed to process "api_def_VarHandleOp.pbtxt": Attribute allowed_devices not defined in base api for VarHandleOp exit status 1 /go/src/github.com/tensorflow/tensorflow/tensorflow/go/op/generate.go:18: running "go": exit status 1

Okay managed to get go generate to work, by doing
cd ${GOPATH}/github.com/src/tensorflow/tensorflow
git checkout tags/v2.2.0
go generate github.com/tensorflow/tensorflow/tensorflow/go/op
This has now placed a lot of generated protobufs in tensorflow/go/vendor/github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto
Now doing
go mod tidy in my src directory seems to understand that there the core_protos for go exists.
Outputting the following:
go: finding github.com/tensorflow/tensorflow/tensorflow/go/core/framework/tensor_shape_go_proto latest go: finding github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto latest go: finding github.com/tensorflow/tensorflow/tensorflow/go/core/framework/types_go_proto latest go: finding github.com/tensorflow/tensorflow/tensorflow/go/core/framework latest go: finding github.com/tensorflow/tensorflow/tensorflow/go/core latest go: finding github.com/tensorflow/tensorflow/tensorflow/go latest go: finding github.com/tensorflow/tensorflow/tensorflow latest
But doing
go run main.go
or
go build main.go
Seems to try and download the module and fails with the following output
go: finding github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto latest go: finding github.com/tensorflow/tensorflow/tensorflow/go/core latest go: finding github.com/tensorflow/tensorflow/tensorflow/go latest go: finding github.com/tensorflow/tensorflow/tensorflow latest build command-line-arguments: cannot load github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto: cannot find module providing package github.com/tensorflow/tensorflow/tensorflow/go/core/core_protos_go_proto

@JacobSMoller Have you found a solution for this? I have the exact same issue.

@ConverJens Sorry didn't get a notification for you writing me.
Yes not a nice solution. But i did make it work...
Here is a public bucket with working libtensorflow 2.2 for unix and osx (If you haven't got them already)
https://console.cloud.google.com/storage/browser/vml-tf-lib?forceOnBucketsSortingFiltering=false&project=dev-vml-cm
The next step is the not so nice step...
`FROM golang:1.13 AS builder
ARG<EMAIL_ADDRESS>ARG GO_SRC_TF_PATH=/go/src/github.com/tensorflow/tensorflow/tensorflow/go/vendor/github.com/tensorflow/tensorflow/tensorflow/go/core/
WORKDIR /build-lib
RUN apt-get update && apt-get install -y --no-install-recommends ca-certificates autoconf automake libtool curl make g++ unzip curl
Install gcloud to get gsutil
RUN wget https://storage.googleapis.com/pub/gsutil.tar.gz  && tar xfz gsutil.tar.gz -C $HOME && rm gsutil.tar.gz
ENV PATH ${PATH}:/root/gsutil
Install libprotoc
RUN wget https://github.com/protocolbuffers/protobuf/releases/download/v3.11.4/protoc-3.11.4-linux-x86_64.zip
RUN unzip -o protoc-3.11.4-linux-x86_64.zip -d /usr/local bin/protoc
RUN unzip -o protoc-3.11.4-linux-x86_64.zip -d /usr/local 'include/*'
RUN rm protoc-3.11.4-linux-x86_64.zip
Install libtensorflow
RUN gsutil cp gs://vml-tf-lib/libtensorflow-unix.tar.gz .
RUN tar -C /usr/local -xzf libtensorflow-unix.tar.gz
RUN ldconfig
RUN rm libtensorflow-unix.tar.gz
WORKDIR /build-lib
Tensorflows repo is a little broken, so to continue ignore this download error.
RUN go get -d github.com/tensorflow/tensorflow/tensorflow/go/op; exit 0
WORKDIR /go/src/github.com/tensorflow/tensorflow/tensorflow
Checkout the 2.2.0 release in the downloaded source
RUN git checkout tags/v2.2.0
WORKDIR /build-lib
run the go generate script located in the downloaded source code, to generate go protos
RUN go generate github.com/tensorflow/tensorflow/tensorflow/go/op
WORKDIR /app
COPY go.sum /app/go.sum
COPY go.mod /app/go.mod
RUN go mod download
make the core folder in the go mod tensorflow path
RUN mkdir -p ${GO_MOD_TF_PATH}/core
RUN cp -r ${GO_SRC_TF_PATH}/core_protos_go_proto ${GO_MOD_TF_PATH}/core/core_protos_go_proto
RUN cp -r ${GO_SRC_TF_PATH}/framework ${GO_MOD_TF_PATH}/core/framework
Not needed for serving.
RUN rm ${GO_MOD_TF_PATH}/core/core_protos_go_proto/autotuning.pb.go
RUN rm ${GO_MOD_TF_PATH}/core/core_protos_go_proto/conv_autotuning.pb.go
COPY tftimer/tf.go /app/tf.go
RUN go build tf.go
Make runner as pr.
FROM debian:bullseye AS runner
WORKDIR /app
RUN apt-get update && apt-get install -y --no-install-recommends ca-certificates
COPY --from=builder /app/tf /app/tf
COPY --from=builder /app/doc /app/doc
COPY --from=builder /usr/local /usr/local
RUN ldconfig`
So to make it work i do the
go get -d as suggested in go documentation
Then do go generate
then I use go mod download to get the tensorflow package.
And finally copy the generated protos over from the downloaded source code.
Not nice in anyway. But haven't been able to find a better way.
And then copy

@JacobSMoller Thanks for the detailed answer! I will try it the week after next and let you know how it goes!

@JacobSMoller Thanks for the hint. I tried it, but failed with
Step 23/38 : COPY go.sum /app/go.sum
COPY failed: stat /var/lib/docker/tmp/docker-builder720470318/go.sum: no such file or directory
where do you get the go.sum and go.mod files from?

They are generated when using go modules.
https://blog.golang.org/using-go-modules

Fixed as part of:
https://github.com/tensorflow/tensorflow/commit/e5e495db7bee77cd0fd5dda3b06bd743cbcf1ef8
Note that it requires from master:
bazel build -c opt //tensorflow/tools/lib_package:libtensorflow
tar -xzvf the package in /usr/local
go generate github.com/tensorflow/tensorflow/tensorflow/go/op

@jhseu This as far as I can tell, still doesn't enable users to do go get github.com/tensorflow/tensorflow/tensorflow/go
and simply run things.
When using go modules I still need to generate the protobufs in $GOPATH/src/github.com/tensorflow/tensorflow And then move the generated protobufs in to $GOPATH/pkg/mod/github.com/tensorflow/...
Also doing the go generate command from master now returns a new error:
2020/07/27 13:07:50 failed to process "api_def_BatchFunction.pbtxt": Attribute enable_large_batch_splitting not defined in base api for BatchFunction exit status 1 go/src/github.com/tensorflow/tensorflow/tensorflow/go/op/generate.go:18: running "go": exit status

@JacobSMoller Yeah, go get doesn't work for now. I made a pull request to fix it, but there were some objections to supporting both the bazel and generated protobufs to make it work.
For that error: it occurs when there's a version mismatch between libtensorflow.so and the TF ops in the source code. See my command above for building libtensorflow.so.

@jhseu alright thanks :)
So no road map for having a go gettable Tensorflow isntallation :)?

I've spend a good day and a half truign to get this installed on my Ubuntu machine. Tried most things mentioned above, and have built everything from scratch. After installing Protobuf, it's not able to generate the protobufs

@silverark How are you installing protoc? It looks like you're missing the default protos from your installation. Typically they're in /usr/local/include after installation, like so:
jhseu0:~$ locate any.proto
/usr/include/google/protobuf/any.proto

Thanks @jhseu. Turns out I had a different version of proto hidden in my ~/local
Finally managed to get it working but only without module. If I use go mod in my project it downloads the version into pkg which is still broken. I'm currently at the stage @JacobSMoller is describing.


update maintainers

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

update/followit-update-maintainers

@conda-forge-admin, please rerender


Override MimePull lib version

SpringBoot 2.7.0 pulls 1.10 version of below lib which is compiled with Java11 and hence causes errors when running on Java8

What's mime pull used for?  Is the version you're proposing compatible with Spring Boot 2.7?

Jersey-media-multipart has dependency on mimepull lib. However, newer version of dependency is forced via SpringBoot 2.7.0 BOM. See this link for issue details - https://github.com/spring-projects/spring-boot/issues/31133
This explain why they added it in 1st place - https://github.com/spring-projects/spring-boot/issues/14924
And this say that it is redundant and might not be needed as part of SpringBoot BOM - https://github.com/spring-projects/spring-boot/pull/31145
Instead of removing dependency altogether, its safe to downgrade it to latest version compiled with Java8. Ran Bot post changes on both Symphony and MS Teams and they ran fine.


CI M6 test is broken

The e2e test is broken for the token shielding M6 demo:
http://vm-web3-build.scs-ad.scs.ch:8080/job/substraTEE/job/substraTEE-M6-master_nightly/

is this related to #175 ?

caused by #176


Problem with PFFacebookUtils isLinkedWithUser

I have 2 Parse accounts, let's say X and Y. I've signed up them using 'signUpInBackgroundWithBlock' after setting username and pass. Then I login with X and link it with FB using 'PFFacebookUtils linkUserInBackground:[PFUser currentUser]'. After it I logout and login with user Y. I'm trying to connect to FB, and it returns error (Another user is already linked to this facebook id. (Code: 208, Version: 1.9.1)) as it should normally do. But after relaunching application, 'PFFacebookUtils isLinkedWithUser:[PFUser currentUser]' returns YES for user Y, although Parse.com shows that user X is connected to FB and Y is not. Only after relogging into Y, it shows that it's not connected to FB.
Any chances you know what's the problem?
Thanks!

Hey @alexvolchek, any chance you can put the version of the SDK you are using?
1.10.0 from last week has a fix for exactly this issue.
Try it out and let me know how it goes.

@nlutsenko I've updated to 1.10.0 and now it works fine, thanks!


Add more code examples

Is your feature request related to a problem? Please describe.
It's not a real problem of library itself. I would say it's more about documentation. Sometimes it's better to see real example, than reading a thousands line of text, so, please, add more real code examples.
Describe the solution you'd like
I would like to see next examples:

usage of preload API (component, group);
usage placeholder option;
tab/drawer examples;

Describe alternatives you've considered
I think it would be good to have all examples in one place (let's say in Example application) and at the same time have separate Snacks for each case (these snacks can be added to documentation).

I agree, I found the lib great, but I tried to use it in the RN with typescript I don't know if it worked or not

@Victorcassiano it also works in TS. Why do you have doubts that it doesn't work?

@Victorcassiano também funciona em TS. Por que você tem dúvidas de que não funciona?

That's where it is, I did the correct steps that the doc indicates, but it doesn't have a counter to know if it really worked or not.
to check I used this.
import { investigate } from 'react-native-bundle-splitter/dist/utils';
console.log('Bundle Info: ', investigate().loaded);
1992 appeared loaded, does that mean it worked?

@Victorcassiano try to wrap some of your screens in register HOC. After that check the output from the investigate function. If you did everything correctly, then you will not see your screens in loaded. If so, then everything works correctly :)

will it work for functional component also ? I have created the screens and navigator as functional component.

will it work for functional component also ? I have created the screens and navigator as functional component.

@cosmic1234 yes, it should work with functional components as well 👍 Let me know if you encounter any issues!


Make or Document how to use CallbackFuntion with Layers/extensions.

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
I tried to follow Datadog lambda integration which is using Layers to provide some libraries.
I was not able to achieve that with CallbackFucntion due to packaging issues.
After trying CodePathOptions (which could be documented better #pulumi/pulumi-hugo/issues/1504) I hit the hard stop not being able to supply correct TS compiler options in tsconfig.json.
Based on this article I learned that I need to set paths in tsconfig.json however I failed to find out how to do this with Pulumi.
It would be good if the callback function could support this or document that its is not possible to use layers.
Affected area/feature

In case others find this issue, I've posted an example of this here: https://github.com/pulumi/pulumi-hugo/issues/1504#issuecomment-1979545206 (while we work on getting the docs updated to cover this).


docs(website): clarify which commit types auto bump by default and how to change this config

Thanks @emily-zall, sorry for the delay.


Add Unlinked Games to GUI

Add support for unlinked games to the GUI

@Valentin-Metz this one is ready for review


modified readme order

Config.js must exist or npm start will fail.  Re-ordered the steps in the readme to create config.js before testing the app.

btw, issues are not enabled for this repo

Looks good, and thanks! I'll do the same for the Firebase example.

Good call - enabled!
On Mon, Nov 13, 2017 at 2:46 PM Jeremy Keen<EMAIL_ADDRESS>wrote:

btw, issues are not enabled for this repo
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/voice-iot-maker-demo/pull/1#issuecomment-344035693,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALLryPKtZUo8VGMTkOxHylPdnjDTInXks5s2JybgaJpZM4QcTOg
.

--
Mike Dory
<EMAIL_ADDRESS>732-690-8357


Refactor tests from Enzyme to React Testing Library for fieldList

Description
This issue is related to refactoring the fieldList.test.tsx file.
We've recently been migrating our tests from Jest + Enzyme to Jest + testing-library/react to support a full migration to React 18.
You may refer to this previous PR beginning the migration for code examples - https://github.com/dcos-labs/ui-kit/pull/771.
Acceptance Criteria
Refactor the test to use React Testing Library instead of Enzyme (which is no longer supported with React 18).
More information: Migrate from Enzyme
How to Contribute
Please review the contributing documentation beforehand.
Steps to Contribute:

Fork the repository.
Clone it to your local system.
Create a new branch.
Make your changes. Make sure that tests are still passing. To update snapshot tests, run npm run test -- -u and include the updated tests in your commit.
Commit your changes.   For example, refactor: <component_name> tests.
Push up your changes. In GitHub, click the compare & pull request button to begin drafting your pull request.
Fill out the pull request form and attach the "ready for review" label when complete.

Hi @nataliepina! I would love to pick this up


CosmosDB Mongo Cluster Support

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

CosmosDB

Please merge with latest main branch to resolve conflicts

I synced the latest master.


"Invalid argument(s): Cannot find executable forflutter/bin/cache/artifacts/engine/ios-release/gen_snapshot_x86_64.".

Oops; flutter has exited unexpectedly: "Invalid argument(s): Cannot find executable for flutter/bin/cache/artifacts/engine/ios-release/gen_snapshot_x86_64.".
A crash report has been written to flutter_05.log.
This crash may already be reported. Check GitHub for similar crashes.
https://git.io/Jf23W
To report your crash to the Flutter team, first read the guide to filing a bug.
https://flutter.dev/docs/resources/bug-reports
Create a new GitHub issue by pasting this link into your browser and completing the issue template. Thank you!
https://git.io/Jf23l
Command PhaseScriptExecution failed with a nonzero exit code

Hi @nguyentrunghieutcu
can you please provide your flutter doctor -v and flutter run --verbose?
Also, to better address the issue, would be helpful if you could post a minimal code sample to reproduce the problem
Thank you

@nguyentrunghieutcu
please follow up on https://github.com/flutter/flutter/issues/57859, closing as duplicate,
If you disagree, please write in the comments and I will reopen it


Modified FakeTime to allow advances in time.

What type of PR is this?
/kind feature

What this PR does / why we need it:
This feature modifies FakeTime to simulate advancement in time. This aids the testing of wait() within reqcounter, to support multiple sleep/wake cycles on time advancements. In order to simplify logic, this implementation ONLY supports a single sleeping goroutine for testing. Therefore FakeTime should not be used to test multiple request counters at once. If this behavior is necessary for future tests, a different structure can implement such logic.
Which issue(s) this PR fixes:
Part of #358

Special notes for your reviewer:
This PR blocks the actual use of the new FakeTime in #372
Does this PR introduce a user-facing change?
NONE

Modified FakeTime to allow advances in time.

cc: @listx @amwat @justaugustus @kubernetes-sigs/release-engineering

Er, I should have placed a hold. This is still WIP. @tylerferrara can you create a revert?


Build setup tweaked - having es, cjs and umd files now

fixes #58
Changes:

eslint update
some new tools got used like rollup, prettier etc
using src as module entry point as no es6+ features are used (except the export ofc)
lib is built from src as commonjs file (main entry point)
umd builds are also built into dist

Due to nature of changes this probably should get released as new major and I thought I will also include 2 breaking changes along the way (I can ofc revert any of them if you want to):

removed deprecated functionalities
dropped support for ie8 (removed bind "polyfill")

If you could add back the bind polyfill, I will merge this PR.

Brought this back.
I've also changed node's version on travis to the latest, version 4 was needed for some dev deps but also npm4 is needed for publication (I've added prepublish which tests that to prevent publications on older npms which wouldn't run prepare script) so I've just made it latest. I don't think testing this on 0.12 was giving any extra guarantees anyway so I consider this bump not an issue - let me know if you think otherwise.

Ok, I would prefer to test on 0.12 because it doesn't support Promise, but I think it's fine.
Thanks again for all this work. I will look to create a new version in the next couple days

No problem, those were really simple adjustments needed to make the whole thing work


Unsupported challenge

Here is my log:
failed to connect: failed to create session: unsupported challenge: decode('4KKhWL8g8lkI5gtpjL7cUmay8jCHidMKNsmF2xQgRL7TpKh07xAEqZJ8TlNy6bME8lvqmogknRsVIa6RppN3SYXg1XbReRINkH0c'); function decode(message) {var offset = ((69 * 87) + (89 * 96 * 39 + 97) * 62); console.log("Offset derived as:", offset); return _.replace(message, /./g, function(char, position) {return String.fromCharCode((((char.charCodeAt(0) * position) + offset) % 77) + 48);});}
exit status 1

It currently works fine--I just ran it and succeeded.
When did this happen? I had to add something to safeval.pw this morning to fix a bug which Kahoot introduced with their latest "challenge". That update was around 11AM EST this morning (about 3 hours ago). If you tried before then, I'd expect it to have failed.

@unixpickle Could you join my discord? I made a way to make flood more easy. And I flood for people using a discord bot

@LordBunny if you use this tool or a related tool to flood kahoot, then your "discord" (whatever that is) depends on safeval.pw. Thus, a bug in safeval is a bug in your system.

Safeval.pw?

safeval.pw is the website that kahoot-hack uses internally to connect to Kahoot.

It's being blocked by my firewall for web-advertisements

Then there is no way to run kahoot-hack besides using a server to run the hack or having someone else do it.

In an attempt to fix the firewall issue, I have a question:
When you attempt to load the website with https (go to https://safeval.pw), does it work then? I remember some firewalls a few years ago not being able to deal with https domains; I don't know if that is still the case.


video = createCapture(VIDEO); is not working

async function setup() {
myCanvas = createCanvas(640, 640);
video = createCapture(VIDEO, videoReady);
video.size(640, 480);
video.hide();
}
this is my setup function.
but video(webcam) is not working.
how do i make working?

Hey, thanks for the report! I'll take a look on the weekend.

Well, this is very strange, I went to fix the other issue and saw this opened one from so many months ago.
I remember very clearly spending some hours on this and reaching the conclusion that webviews inside VSCode can't access the camera, I have no idea why I didn't close this issue, my apologies.


The Value viewer panel does not remember by which column is data sorted

System information:

Windows 10
DBeaver version community 21.3.5
Additional extensions

Connection specification:

sqlite
odbc sqlite driver
Do you use tunnels or proxies (SSH, SOCKS, etc)?  no

Describe the problem you're observing:
The Value viewer panel does not remember by which column is data sorted
Steps to reproduce, if exist:
-in Database navigator, select table
-in Data view, click on 'modelid' column (a column that has foreign keys, )
-in Value viewer panel, data from models table is displayed
Value viewer has two columns, 'values' and 'description. Sorted by 'values' by default
I click on 'description' and data in value viewer is sorted by description. Good.
back In data view, I click on another column, then again on the 'modelid'
data in value viewer is again sorted by values.
expected situation is that data in value viewer is sort by 'description', just as it was last sorted.
suggested way to fix:
remember last sorting  etting of 'value viewer', or,
add another column in define criteria tab, where users can set sorting of 'value viewer'
('define criteria' setting is saved between clicking on different columns in data view.)
Why is this needed:
Sometimes you add another model, and when viewing/editing data in the main table, from the linked table, it is easier to have data in the linked table sorted alphabetically.
Thanks.
Include any warning/errors/backtraces from the logs

Fixed.
Also I chganged the way how we mark current value (now it is bold font).
Also value can be changed with double click or Enter now (previously it was single click and it produced a mess sometimes)


Restrict matplotlib

matplotlib>=3.5 with pygimli=1.2.5 results in a TypeError: int() argument must be a string, a bytes-like object or a number, not 'slice' for some plotting commands. Restricting therefore matplotlib.

As the problem has been fixed with pygimli>=1.3 and the error is not related to the tutorials but pg in general, we could rather limit any pg<1.3 to mpl<3.5 so that the tutorials will work with both pyGIMLi=1.2 or 1.3.
On the other hand, all tutorials should be merged into pygimli.org so that the transform tutorials stay as historic marks.


EliminateRowCol not work well for zero rows ?

Hi all
I have a stiffness matrix, which has many zero rows. I want to replace these rows with 1.0 at diagonal. But I found that EliminateRowCol and EliminateRowColDiag didn't work well. I want to know why and how to fix it.
Below is a MWC.
#include <iostream>
#include "mfem.hpp"

using namespace std;
using namespace mfem;

int main()
{
    Mesh mesh(5, 5, Element::TRIANGLE, true, 1.0, 1.0);

    int p_order = 1;
    H1_FECollection h1_fec(p_order, mesh.Dimension());
    FiniteElementSpace h1_space(&mesh, &h1_fec);
    int ndofs = h1_space.GetVSize();

    ConstantCoefficient zero(0.0);

    BilinearForm blf(&h1_space);
    blf.AddDomainIntegrator(new DiffusionIntegrator(zero));
    blf.Assemble();
    blf.Finalize();

    SparseMatrix blf_mat(blf.SpMat());

    Array<int> attr2dofs(ndofs);
    for (int i=0; i<ndofs; i++)
    {
        attr2dofs[i] = i;
    }

    Vector diag(ndofs);

    blf_mat.GetDiag(diag);
    diag.Print(cout << "diag (before eliminate):\n");

    for (size_t i=0; i<attr2dofs.Size(); i++)
    {
        blf_mat.EliminateRowCol(attr2dofs[i], Matrix::DIAG_ONE); // by this way or below way
//        blf_mat.EliminateRowColDiag(attr2dofs[i], 1.0);        // not working well neither
    }

    blf_mat.GetDiag(diag);
    diag.Print(cout << "diag (after eliminate):\n");
}

The issue here is that blf_mat does not have any entries (because of the zero diffusion coefficient) -- you can see that by calling blf_mat.PrintInfo(cout);. When I switch to positive diffusion coefficient, this works fine. You can also tell the BilinearForm to store the zeros (from the element matrices) in the SparseMatrix with:
   const int skip_zeros = 0;
   blf.Assemble(skip_zeros);
   blf.Finalize(skip_zeros);

and then you will get the expected behavior.

Thank you @v-dobrev . It works well.


Validation fixes

Category

[x ] validation fix

Docs Build status updates of commit 8364376:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/schema/_authschemename-element-formsauthcredentials-complextypesps15xsdsearchset2.md
:warning:Warning
View
Details

docs/schema/_authsubmissionmethod-element-formsauthcredentials-complextypesps15xsdsearchset2.md
:warning:Warning
View
Details

docs/scenario-guidance/Multi-geo-capabilities.md
:white_check_mark:Succeeded
View

docs/scenario-guidance/User-information.md
:white_check_mark:Succeeded
View

docs/schema/_authschemename-element-formsauthcredentials-complextypesps15xsdsearchset2.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

docs/schema/_authsubmissionmethod-element-formsauthcredentials-complextypesps15xsdsearchset2.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 3c37070:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/schema/_authschemename-element-formsauthcredentials-complextypesps15xsdsearchset2.md
:warning:Warning
View
Details

docs/schema/_authsubmissionmethod-element-formsauthcredentials-complextypesps15xsdsearchset2.md
:warning:Warning
View
Details

docs/scenario-guidance/Multi-geo-capabilities.md
:white_check_mark:Succeeded
View

docs/scenario-guidance/User-information.md
:white_check_mark:Succeeded
View

docs/schema/_authschemename-element-formsauthcredentials-complextypesps15xsdsearchset2.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

docs/schema/_authsubmissionmethod-element-formsauthcredentials-complextypesps15xsdsearchset2.md

Line 2, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Global effort to fix build validation errors
@VesaJuvonen
The Content & Learning team is fixing build validation errors on docs.microsoft.com for the rest of H2. This will eliminate potential accessibility, security, and usability issues. This PR includes only build validation fixes and does not change other content. Please review within five business days and merge or comment in the PR with any changes you'd like to see. Thank you!


Output: Fix malformed <a> tags in the <head>

Context

There can be some mangled <script> tags turned into <a> on WP.com VIP.
Summary

This PR detects incorrect <a> tags in the HTML output and turns them into actual <script> tags as expected.
Relevant Technical Choices

Since it is not possible to fix this with the sanitizers, and since the root cause with the KSES logic is unclear, this workaround lives in the HTML class on purpose.
To-do

Testing

User-facing changes

N/A
Testing Instructions

[x] This is a non-user-facing change and requires no QA

This PR can be tested by following these steps:

TBD

Reviews
Does this PR have a security-related impact?

No
Does this PR change what data or activity we track or use?

No
Does this PR have a legal-related impact?

No
Checklist

[x] This PR addresses an existing issue and I have linked this PR to it in ZenHub
[ ] I have tested this code to the best of my abilities
[x] I have verified accessibility to the best of my abilities (docs)
[x] I have verified i18n and l10n (translation, right-to-left layout) to the best of my abilities
[x] This code is covered by automated tests (unit, integration, and/or e2e) to verify it works as intended (docs)
[x] I have added documentation where necessary
[x] I have added a matching Type: XYZ label to the PR

Fixes #9297

I've sent this to the partner for review and will provide an update once I know whether it works as expected.


[mieb] Fix Flickr30k I2T and T2I

Fixes #1434.

The image to text ratio should be 1:5
Remade the datasets for I2T and T2I
Ran these for the following models (in the format T2I / I2T)

openai/clip-vit-base-patch32
- CLIP benchmark: 0.834 / 0.949
- Ours: 0.829 / 0.951

openai/clip-vit-base-patch16
- Jina paper: 0.856 / 0.962
- Ours: 0.854 / 0.970

jinaai/jina-clip-v1
- Jina paper: 0.890 / 0.965
- Ours: 0.890 / 0.961

Notes:

CLIP benchmark
Jina CLIP paper

Checklist

[x] Run tests locally to make sure nothing is broken using make test.
[x] Run the formatter to format the code using make lint.

perfect. Thanks!!


Variable input in "value" field for color treshold

Would it be possible to make the "value" input for the color threshold variable? Now I can only enter fixed numbers, and I need to change my card when values change.
Before:
color_threshold:
- value: 5
color: blue
- value: 12
color: red
Request:
color_threshold:
- value: input_number.min_price
color: blue
Or
- value:  >
{{input_number.max_price}}
color: red
Or
- value:
entity: input_number.min_price
color: red
Thanks!

Oehh yes I'd like to see this too. This aids in auto scaling for e.g. energy prices


getoODEsFromModel crashes

The following script crashes
import tellurium as te
import roadrunner
r = te.loada("""
S1 -> S2; k1*S1;
k1 = 0.1;
S1 = 1; S2 = 0
""")
print (te.getODEsFromModel(r))
File "C:\Tellurium-Winpython-3.7\python-3.7.7.amd64\lib\site-packages\tesbml\libsbml.py", line 98, in _swig_getattr
raise AttributeError("'%s' object has no attribute '%s'" % (class_type.name, name))
AttributeError: 'Species' object has no attribute 'boundary_condition'
Problem with libsbml or something else?

This also failed for me, which turned out to be because 'boundary_condition' is not... anything.  The correct way to get that attribute is 'getBoundaryCondition()'.
I have fixed this in the develop branch of tellurium, but you should be able to fix it yourself, too.  From Spyder, when you get the error:
File "D:\Tellurium-Winpython-3.7\python-3.7.7.amd64\lib\site-packages\tellurium\utils\misc.py", line 250, in toString
if not self.model.getSpecies (index).boundary_condition:
Click on the filename, and change boundary_condition to 'getBoundaryCondition()':
        if not self.model.getSpecies (index).getBoundaryCondition():

That works thanks. Sounds like it never worked

I know!  I guess that function was never tested.


How to send raw sticker / send sticker with custom packname or author?

How to send raw sticker / send sticker with custom packname or author? (on Multidevice)

You can try to use a generated image from wa-sticker-formatter, using the webp format

You can try to use a generated image from wa-sticker-formatter, using the webp format

Do you think it's feasible to make an implementation using wa-sticker to have this function by default in wa-js?


fix(cosmos): handle Tx fee in

Found while testing https://github.com/shapeshift/web/pull/169. It is not caused by that PR and is a bug currently happening in all environments, where we let the user continue with the flow of sending an amount in the range of (max wallet amount - 0.0035 - max wallet amount), only to be blocked from continuing at confirm step.
Description
This handles the Tx fee in the <Stake /> view.
It subtracts the fee from the amount when making a percent send (on max, but also other percentages if the user has enough balance to send a Tx with (selected percentage - Tx fee), but doesn't have enough for (selected percentage + Tx fee) which could be the case if a user has a low balance of ATOM e.g 0.0038 ATOM which is only barely more than our arbitrary average fee).
It also adds validation on user-input amount, which will error out if amount + Tx fee is greater than total balance, vs. amount greater than total balance currently.
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A
Risk
Since this could produce regressions in 0-balance wallets, tested it out and it produces a consistent result as we have over the app:

There are no other risks that I'm aware of, tested with a wallet with a low balance and all user-input amounts as well as percentages (with subtracted Tx fee if needed) are still working.
Testing

On Cosmos Staking, click Max
The amount in the input should be wallet balance - average Tx fee (0.0035 ATOM)
Manually input some amount that's within the wallet balance, but not enough to broadcast a Tx (e.g wallet balance - 0.003459 ATOM)
The Continue button should be in "Insuffient Funds" state, but the input shouldn't change as we don't modify user input except for valid number/precision sanitizing
Test with a 0 balance wallet
The fiat/crypto input should be -0.0035 ATOM, with the continue button in "Insufficient Funds" state

Screenshots (if applicable)

@gomesalexandre - i'm a little confused/curious re the screenshots.
are we saying that this will show negative values by design? if so @DiggyDiggy2 @reallybeard thoughts?

@gomesalexandre - i'm a little confused/curious re the screenshots.
are we saying that this will show negative values by design? if so @DiggyDiggy2 @reallybeard thoughts?

This one is a little hard to follow but no, we dont want negative values

are we saying that this will show negative values by design?

Precisely, that's consistent with the other places in the codebase where we do something similar with gas fees deduction:

Following @DiggyDiggy2's comment, happy to not display negative values in that case. Without negative numbers, that brings more questions to the table however:

When using percentages, do we want to get a percentage of the max amount - gas fee if it results in a valid Tx? (If the final number is less than 0.000001 ATOM, it won't be a valid Tx to broadcast) instead?
If the final number after gas deduction results in an invalid Tx, do we want to zero-out the input, or rather show the original intented amount instead (if greater than 0)?
Do we want to tackle the negative numbers in trade/send for consistency?

are we saying that this will show negative values by design?

Precisely, that's consistent with the other places in the codebase where we do something similar with gas fees deduction:

Following @DiggyDiggy2's comment, happy to not display negative values in that case. Without negative numbers, that brings more questions to the table however:
1. When using percentages, do we want to get a percentage of the max amount - gas fee _if it results in a valid Tx_? (If the final number is less than 0.000001 ATOM, it won't be a valid Tx to broadcast) instead?

2. If the final number after gas deduction results in an invalid Tx, do we want to zero-out the input, or rather show the original intented amount instead (if greater than 0)?

3. Do we want to tackle the negative numbers in trade/send for consistency?

@shapeshift/product Are we happy to spec this one as an issue, so I can close this and for housekeeping purposes and reopen a clean one that's matching an A/C?

are we saying that this will show negative values by design?

Precisely, that's consistent with the other places in the codebase where we do something similar with gas fees deduction:

Following @DiggyDiggy2's comment, happy to not display negative values in that case. Without negative numbers, that brings more questions to the table however:
1. When using percentages, do we want to get a percentage of the max amount - gas fee _if it results in a valid Tx_? (If the final number is less than 0.000001 ATOM, it won't be a valid Tx to broadcast) instead?

2. If the final number after gas deduction results in an invalid Tx, do we want to zero-out the input, or rather show the original intented amount instead (if greater than 0)?

3. Do we want to tackle the negative numbers in trade/send for consistency?

@shapeshift/product Are we happy to spec this one as an issue, so I can close this and for housekeeping purposes and reopen a clean one that's matching an A/C?

I agree we shouldn't show negative numbers.
Sounds like we may need a way to handle min amounts on sends for different types assets.
Ideally it should zero out the field if the amount is less than the minimum amount e.g. 0.0000001 for ATOM.
Maybe this is something we can add as an override on the asset service? The default would be 0, if the asset has a minAmount we use that instead.

Closing as this will be taken over in https://github.com/shapeshift/web/issues/1966 to implement this in a consistent way.


Fix unhandled exception when tooltask process exits at a certain moment during cancelation

Fixes #3028
I can't really test this. I do see from inspection of the Process code that ProcessName getter will only throw in the situation where the process has exited.

curl: (56) LibreSSL SSL_read: SSL_ERROR_SYSCALL, errno 54
dotnet-install: Cannot download: https://dotnetcli.azureedge.net/dotnet/Sdk/2.1.300-preview2-008046/dotnet-sdk-2.1.300-preview2-008046-osx-x64.tar.gz
dotnet-install: Downloading legacy link: https://dotnetcli.azureedge.net/dotnet/Sdk/2.1.300-preview2-008046/dotnet-dev-osx-x<IP_ADDRESS>0-preview2-008046.tar.gz
curl: (22) The requested URL returned error: 404
Failed to install stage0
/Users/dotnet-bot/j/w/Microsoft_msbuild/master/innerloop_OSX10.13_CoreCLR_prtest/build/build.sh: line 245: /Users/dotnet-bot/j/w/Microsoft_msbuild/master/innerloop_OSX10.13_CoreCLR_prtest/build/../artifacts/.dotnet/2.1.300-preview2-008046/dotnet: No such file or directory

@dotnet-bot test osx10.13 build for coreclr

@rainersigwald do we merge now?


Fresh installs fail due to Pydantic

Describe the bug
Running launch-user.bat for a fresh install after commit bc00bf2de9a24aa75a15f24f6b74a0c6b6cf81f2 does not succeed. Hits a Pydantic error and fails.
After testing other versions, it appears to present in nearly all versions. But it changes where it pops up per version.
Reproduction
git clone https://github.com/ddPn08/Radiata.git
cd Radiata
./launch-user.bat
If commit is beyond bc00bf2de9a24aa75a15f24f6b74a0c6b6cf81f2 it will fail.
Expected behavior
WebUI launches and allows for interaction
System Info
Python 3.10
Windows 11
Versions: 9403f4a0efd6b34f09717e84f4e8c998be43a418, fcdf5d97441459a30ff951cf442a270be850ed02, a32b7b2680da5b7584c60b7159186bc891b900f5, 7f6f2427e9a121c2b751da1e2c9b671588b3b836, bc00bf2de9a24aa75a15f24f6b74a0c6b6cf81f2
RAM: 48 GB
GPU: RTX 3090
Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

The error details are incomplete.
Please paste all of the actual log output.
Or #102 may resolve the issue.

Merged #102. Try updating and starting again.

Issue is resolved after updating. Other errors are in relation to the tensorrt library so I won't bring them up here.


JSON Format should include the resolution

The dimensions can be determined from an I frame - taking the Square root of the array , and assuming the image is square
However some expressive pixels can render as all P frames. In which case the width needs to be known to render correctly.
Many implementations will have a fixed size that matches the ExpressivePixel being rendered.
This is not the case if you use a display that is larger than the animation.
It would be simpler just to put the dimensions in the JSON

Great idea, this will be in the next build of the Windows App


Getting Started with Spring Boot : Hello World Grettings Application

A sample application using Spring Boot for anyone who is willing to learn. It is the most basic Hello World application.
A highly in demand language which can be used to create Microservices!
Attaching the files as request

I really didn't understand the whole codebase. Can you elaborate on what this is all about and why there are 17 files related to this?

Basically it is a application which lets you create API's using spring boot. Spring Boot makes it easy to create stand-alone, production-grade Spring based Applications that you can "just run". This application is a starter app that anyone can take a reference when they are trying to learn spring boot. Here you follow a MVC architecture where there are model, views and controllers each designed for different purposes.

In nutshell
Controller is something that interacts with the user/other spring boot application. It exposes a REST endpoint for you. Service/ServiceImpl is where business logic goes. DTO/Entity is your POJO classes which is used only and only to transfer data between layer. They do not have any business logic what so ever.
Please refer to the readme file for links and other info.

Here this code creates a REST GET endpoint which will return a greeting to user. (This can be changed for actual business logic later as per specific use)
GET /greet/{username}
You need POSTMAN to invoke this. But since it was a GET request i used chrome. Other REST Requests will require POSTMAN

Unnecessary dependencies included.

@YashSaxena9 may i know which dependencies are extra here? As far as i can see there are literally two dependencies. One is for testcase and one is for spring web.


Create mainflux-lite - Compact Monolith

It would be great to remove NATS and bundle mainflux-http-server and mainflux-core-server in one monolith binary. This will greatly simplify things for development and testing (prototyping) deployments.
We could also remove InfluxDB from this binary and replace it by MongoDB for time-series data - so that we have just one DB with two collections (devices and data).

Done: https://github.com/Mainflux/mainflux-lite
Now we can develop directly on single binary ;)

InfluxDB still has to be removed (or at least be configurable). I propose 3 MongoDB collections: devices, channels, data - where data would be something like this: http://blog.mongodb.org/post/65517193370/schema-design-for-time-series-data-in-mongodb


Add E2E tests to test error reporting on UI for invalid PipelineRun definition

adds E2E tests for gitea and github to ensure that if user supplies invalid PipelineRun definition, PAC should report errors on UI.
it was created as bug on JIRA but I confirmed it is working as expected.
https://issues.redhat.com/browse/SRVKP-5776

/test

/test