RFE: Flag to change the number of Helm Version Histories we keep for Charts deployed by Rancher

What kind of request is this (question/bug/enhancement/feature request):
Enhancement
Description
At the moment we hard code the number of helm releases we store by default in a cluster for system charts at 10:
https://github.com/rancher/rancher/blob/master/pkg/controllers/managementuser/helm/common/common.go#L34
For clusters with limited resources or at the opposite end with lots of deployments, this assumption can cause instability
We should give customers the option to finetune this value so that they can either have more or less to meet their needs, probably as a flag in cluster.yml so it can be set for specific tenants in any customer environments at a cluster level.
Globally is less preferred but I appreciate that is where the helm controller lives.
We should also lower the default from 10 to 5 so any soft recommendation we make has a less adverse effect (most customers would not use more than 1-2 rollbacks during troubleshooting anyway) and those customers who use this feature extensively can then choose to increase it.

@maggieliu @catherineluse This needs to be added to the support kb, release notes and best practices guide.

rancher/rancher:master-head 76c26fb

Rancher HA install
Deploy downstream cluster
Helm v2

Check /v3/settings/helm-max-history and see the default set to 10
Deploy an app from a Helm v2 catalog and upgrade 10 times
Check ConfigMaps for the project where the app was deployed and see 10 ConfigMaps for each version of the app
Change /v3/settings/helm-max-history to 2 and upgrade the app again
Check ConfigMaps and see 2 ConfigMaps listed for the app version history
Upgrade again and see only 2 ConfigMaps listed with the appropriate versions

Helm v3

Check /v3/settings/helm-max-history and see the value set to 2, change it to 4
Deploy an app from a Helm v3 catalog and upgrade 5 times
Check Secrets for the project where the app was deployed and see 5 secrets for each version of the app
Change /v3/settings/helm-max-history to 2 and upgrade the app again
Check Secrets and see 2 secrets listed for the app version history
Upgrade again and see only 2 secrets listed with the appropriate versions

Boundary testing:

negative values (-1) and values above max int<PHONE_NUMBER>) are accepted -- it doesn't break anything functionally, and seems to set to the default 256 but I haven't confirmed the exact value. We should probably validate this value though.

The default value for helm-max-history is 10 but it was suggested to be 5 in the opening issue

Re-opening for visibility on these items

Helm handles the value ranges, 0 and negative count as unlimited history, and it will handle anything up to max int determined by the architecture (so either int32 or int64)
The default can not be changed as that would be a breaking change and not backwards compatible. If a user has 10 and is expecting 10, changing the default to 5 would cause 5 revisions to be deleted the next time they updated the app and would probably not know why.


Punycode warning

As a developer of VZCode, I'm irritated by the following warning:
[0] (node:468142) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
[0] (Use `node --trace-deprecation ...` to show where the warning was created)

this shows up when you run npm run dev.
Here are the files that reference "punycode" within node_modules:
grep -rh "punycode" /path/to/search

<EMAIL_ADDRESS>./ngrok/bin/ngrok
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./tr46/index.js
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>./typescript/lib/typingsInstaller.js
./typescript/lib/tsserver.js
./typescript/lib/typescript.js
./whatwg-url/lib/url-state-machine.js
./@esbuild/linux-x64/bin/esbuild
./esbuild/bin/esbuild
./ufo/README.md
./ufo/dist/index.d.mts
./ufo/dist/index.d.ts
./ufo/dist/index.d.cts
./.vite/deps/typescript.js
./.vite/deps/typescript.js.map
./rollup/dist/shared/rollup.js
./rollup/dist/es/shared/node-entry.js

Some thoughts from ChatGPT:
To get rid of the deprecation warning for the punycode module in your npm run dev script, you can follow these steps:

Identify the Dependency:
Identify which of your dependencies is using the deprecated punycode module. From your grep results, it looks like several packages reference punycode.

Check for Updates:
Check if there are updates available for these dependencies. Often, newer versions will have resolved such deprecations. You can use npm outdated to see if updates are available.

Update Dependencies:
Update the dependencies to their latest versions if they have addressed the deprecation issue. You can update them manually in your package.json or use npm update <package-name>.

Alternative Libraries:
If the dependency is not updated or maintained, consider replacing it with an alternative that does not use punycode.

Shim or Polyfill:
As a temporary workaround, you can use a shim or polyfill to suppress the warning, though this is not a long-term solution.

Here's a detailed approach:
Step 1: Check for Updates
Run the following command to see if any updates are available for the dependencies:
npm outdated

Step 2: Update Dependencies
Update your dependencies to their latest versions. For example, if you need to update ngrok, you can run:
npm install ngrok@latest

Step 3: Review Dependency Tree
Use npm ls punycode to see which package is using punycode:
npm ls punycode

Step 4: Replace or Patch
If the package using punycode is not maintained, you can look for alternatives. If you find that a specific package (e.g., ngrok) is not updated and causing the issue, you can:

Look for an alternative package that provides the same functionality.
Fork the repository and remove the usage of punycode if it's a small and critical package.

Example: Fork and Fix (for an example package)

Fork the repository of the package using punycode.
Clone your fork and make the necessary changes to replace punycode with a modern alternative like punycode.js.
Publish your fork to npm or use a git URL to use your fixed version in your project.

Step 5: Suppress Warnings Temporarily
If updating or replacing is not an immediate option, you can suppress the warning temporarily by using the --no-deprecation flag in your npm run dev script. Add the following to your package.json scripts section:
"scripts": {
  "dev": "node --no-deprecation ./node_modules/.bin/your-dev-command"
}

Replace your-dev-command with the actual command you use for development.
By following these steps, you can address the deprecation warning for punycode in your development environment.

The option to just ignore it and not output the warning is pretty appealing.
I don't think the usage of "punycode" within our dependencies will go away any time soon.

More thoughts from ChatGPT:
Punycode is a way to represent Unicode characters using only the limited subset of ASCII characters, which makes it suitable for use in domain names. It's particularly useful for internationalized domain names (IDNs) that include non-ASCII characters.
How Punycode Works
Punycode encodes a Unicode string into a form that can be used as part of an ASCII domain name. For example, the domain name "münchen.com" (with a German umlaut) would be encoded as "münchen.com".
Why Punycode?
The Domain Name System (DNS) originally only supported ASCII characters. To enable the use of international characters (e.g., accented letters, Chinese characters), a mechanism was needed to represent these characters in a way that DNS could handle. Punycode was introduced as part of the Internationalized Domain Names in Applications (IDNA) standard to achieve this.
Example
Here’s an example of how a string with non-ASCII characters is encoded and decoded using Punycode:

Original Unicode String:

münchen

Punycode Representation:

xn--mnchen-3ya

Usage
Punycode is often used behind the scenes in applications that need to handle international domain names. Developers typically use libraries to encode and decode Punycode as needed.
Deprecation
The punycode module in Node.js was used to handle these conversions, but it has been deprecated in favor of userland alternatives. This means that you should now use a standalone Punycode library instead of the built-in Node.js module. The deprecation indicates that the core Node.js team wants to reduce the number of built-in modules and encourage the use of community-maintained libraries.
Alternatives
You can use the punycode.js library, which is the userland alternative:
Installation
npm install punycode

Usage
const punycode = require('punycode');

// Encode a Unicode string to Punycode
const encoded = punycode.encode('münchen');
console.log(encoded); // Output: mnchen-3ya

// Decode a Punycode string to Unicode
const decoded = punycode.decode('mnchen-3ya');
console.log(decoded); // Output: münchen

In summary, Punycode is a mechanism for encoding Unicode characters as ASCII, primarily for use in domain names, and the Node.js punycode module is deprecated in favor of standalone libraries like punycode.js.


unicorn commands are use 'sudo' if Debian?

issues #8 #31 #53
already have a PR #37
but this version is upgraded. point is detect 'Debian' and then use sudo
TESTED

local: Macbook
remote: Debian

TODO:

test with other linux
enhance 'detect Debian' regexp

:+1:

@vforvova Thanks for your vote ;)
i inspired your pull request. Thanks :)

@hiphapis thank you! I will use your fork since now! ;)

Any chance on getting @hiphapis pull request merged? At the moment this plugin simply doesn't work with debian jessie.

@hiphapis Have you tried your patch on a recent version of debian jessie? If I'm executing uname -a on my machine, it will return Linux lvps91…de 3.16.0-042stab108.7 #1 SMP Wed Jul 15 15:49:08 MSK 2015 x86_64 GNU/Linux and won't match your regular expression.

Closing. Already merged https://github.com/capistrano-plugins/capistrano-unicorn-nginx/pull/55


Work smoothly on mobile devices

I was testing ep_comments on mobile devices (iPad and iPhone, for now) and noticed some functionalites were not working properly there (clicking to open/close the comment modal, adding comments, etc).
I'm planning to work on this, so wanted to know if anyone has any concerns about it. It seems that http://jquerymobile.com/ could help us, but to be honest I just gave a fast look on it.
Concerns about this? Or about using jQuery Mobile?
I'll work on a separated branch this time, to avoid conflicts if this takes too long to finish.

Happy for you to fix but plz avoid jqm.  It's really bloated and will req another http request that I want to avoid
----- Reply message -----
From: "Luiza Pagliari"<EMAIL_ADDRESS>To: "JohnMcLear/ep_comments"<EMAIL_ADDRESS>Subject: [ep_comments] Work smoothly on mobile devices (#67)
Date: Mon, Jun 29, 2015 22:00
I was testing ep_comments on mobile devices (iPad and iPhone, for now) and noticed some functionalites were not working properly there (clicking to open/close the comment modal, adding comments, etc).
I'm planning to work on this, so wanted to know if anyone has any concerns about it. It seems that http://jquerymobile.com/ could help us, but to be honest I just gave a fast look on it.
Concerns about this? Or about using jQuery Mobile?
I'll work on a separated branch this time, to avoid conflicts if this takes too long to finish.
—
Reply to this email directly or view it on GitHubhttps://github.com/JohnMcLear/ep_comments/issues/67.

Hi lpagliari, did you ever make any progress on adding a comment to an existing comment on mobile?
I have a system built already with Annotator JS but would love to jump completely over to etherpad.

No, I've started to work on it but did not finished, and later mobile devices were not our top priority on the project.

Hi @lpagliari, I know you worked on the comment icon portion, and I've run into an issue with the comment icon itself where it is not triggered on a tablet (i.e. there is no way to open a comment dialogue via icon on an iPad). I've tried converting the 'click' events in ep_comments_page/static/js/index.js to 'touchstart click' and I see you already have them that way in ep_comments_page/static/js/commentIcons.js.
Maybe i'll convert them to 'touchend click' and see if that helps. I'll let you know. Thanks!

No, @DanielBickley. Initially I was developing a plugin that would display a popover every time user selected a text (and that would listen to touchstart), but I didn't have time to finish it before moving to another direction.
Your idea of adding touchstart to click seems correct to me, I believe it would enable adding comments on iOS.

@lpagliari thanks for your help. As it turns out, the clicks were being respected on the iPad, only the measurements on the text area width were off ~80px for me (i.e., the comment block on the right was actually covering the icons making them unclickable). I adjusted widths in the media query a bit and iPad is working.

Awesome!
On Wed, Jul 27, 2016 at 12:48 PM, DanielBickley<EMAIL_ADDRESS>wrote:

@lpagliari https://github.com/lpagliari thanks for your help. As it
turns out, the clicks were being respected on the iPad, only the
measurements on the text area width were off ~80px for me (i.e., the
comment block on the right was actually covering the icons making them
unclickable). I adjusted widths in the media query a bit and iPad is
working.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ether/ep_comments/issues/67#issuecomment-235628855,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAzDIoAM04jntkRAkp7aUuizP76mIIWRks5qZ32zgaJpZM4FOdS6
.

I think this is resolved but if not please comment here and I will re-open.


data: sadath-12

json added to start contributing

json added

I optimised the search page using useCallback with debounce instead of useEffect

I dont see my name as a part of people in the community ??

I dont see my name as a part of people in the community ??

Have you applied to join?

I dont see my name as a part of people in the community ??

Have you applied to join?

I added a first pr with json as my details joined discord and followed in the github thats it what else?

I dont see my name as a part of people in the community ??

Have you applied to join?

I added a first pr with json as my details joined discord and followed in the github thats it what else?

Yeah now i see i raised a issue

I dont see my name as a part of people in the community ??

Have you applied to join?

I added a first pr with json as my details joined discord and followed in the github thats it what else?

Yeah now i see i raised a issue

Great 👍

moving sadath-12.json from root to data dir

I am closing this Pull Request as there are too many changes. Please create new Pull Requests


PDF quality is bad, it's unreadable and swiping experience is not good

For some reason pdfs look quite blurry

Quality problem is fixed in 1.0.5. The new version has just uploaded.
I'll investigate swiping problem. thanks.

Nice!
How should I proceed with adding a progress message because now there is a delay (white window) before pdf is fetched.

Nice!
How should I proceed with adding a progress message because now there is a delay (white window) before pdf is fetched.

I've updated README. You can set statusListener to show preparing status.
pdfView.statusListener = object: PdfRendererView.StatusCallBack {
override fun onDownloadStart() {
loading.visibility = View.VISIBLE
}
override fun onDownloadSuccess() {
// downloading progress is not supported yet. will be added soon.
loading.visibility = View.GONE
}
override fun onError(error: Throwable) {
loading.visibility = View.GONE
}
}

Nice! Thanks for ur support
On Wed, 12 Sep 2018 at 17:39, jidogoon<EMAIL_ADDRESS>wrote:

Nice!
How should I proceed with adding a progress message because now there is a
delay (white window) before pdf is fetched.
I've updated README
https://github.com/jidogoon/PdfRendererView/blob/master/README.md. You
can set statusListener to show preparing status.
pdfView.statusListener = object: PdfRendererView.StatusCallBack {
override fun onDownloadStart() {
loading.visibility = View.VISIBLE
}
override fun onDownloadSuccess() {
// downloading progress is not supported yet. will be added soon.
loading.visibility = View.GONE
}
override fun onError(error: Throwable) {
loading.visibility = View.GONE
}
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jidogoon/PdfRendererView/issues/1#issuecomment-420695251,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOh6F73x0F_zyGZhPHR8Dx9IO1Lae6Qrks5uaSrJgaJpZM4Wi8oF
.

--
Med vänlig hälsning/Best regards,
Eimantas Urbonas, Android Developer
Greenely AB
Mobile: +46 (0) 76 43 70 509
<EMAIL_ADDRESS>www.greenely.com

Also what does it mean
/// downloading progress is not supported yet. will be added soon.
As I see after downloading is finished event is triggered. Do you mean that reporting status in terms of live percentage is not available?

Also what does it mean
/// downloading progress is not supported yet. will be added soon.
As I see after downloading is finished event is triggered. Do you mean that reporting status in terms of live percentage is not available?

Yes. That's what I mean. I just released new version and updated README.

Great!
I also noticed more problems:
I have a multiple pdf with many files. If I zoom into one of these pdf then I can not scroll down anymore on remaining PDF's.
Also in the general overview when I keep scrolling down through a file with few pages the whole scrolling animation seems lagging.
Maybe you should try opening a pdf with google docs viewer and compare it against yours, would be easier to find these problems ?

Great!
I also noticed more problems:
I have a multiple pdf with many files. If I zoom into one of these pdf then I can not scroll down anymore on remaining PDF's.
Also in the general overview when I keep scrolling down through a file with few pages the whole scrolling animation seems lagging.
Maybe you should try opening a pdf with google docs viewer and compare it against yours, would be easier to find these problems ?

As you said, I have to improve behavior on swipe, scrolling and zoom. So I already made a feature using google docs viewer. It is now working only for Kitkat. As you suggested, I'll add option using google docs viewer even when It's not kitkat. I'll notify you when the option added.

Great!
I also noticed more problems:
I have a multiple pdf with many files. If I zoom into one of these pdf then I can not scroll down anymore on remaining PDF's.
Also in the general overview when I keep scrolling down through a file with few pages the whole scrolling animation seems lagging.
Maybe you should try opening a pdf with google docs viewer and compare it against yours, would be easier to find these problems ?

As you said, I have to improve behavior on swipe, scrolling and zoom. So I already made a feature using google docs viewer. It is now working only for Kitkat. As you suggested, I'll add option using google docs viewer even when It's not kitkat. I'll notify you when the option added.

would be really nice, thanks!

@jidogoon  when can you add this flag for loading via google docs viewer?

@eurbon I've had some problems in development env. so I couldn't do something for new version. I just released new version. v1.0.7.

@jidogoon  getting error: duplicate value for resource 'attr/showDivider' with config ''.
Message{kind=ERROR, text=error: duplicate value for resource 'attr/showDivider' with config ''., sources=[/Users/eimaurbon/.gradle/caches/transforms-1/files-1.1/appcompat-v7-28.0.0.aar/64cba8d69b0a6af81138ccd3ee31c5b8/res/values/values.xml:1304:5-69], original message=, tool name=Optional.of(AAPT)}


Solr entrypoint code causes crashloopbacks

Describe the bug
A change introduced with https://github.com/amazeeio/lagoon/pull/1978 causes some issues with escaping
To Reproduce
Steps to reproduce the behavior:

Start any solr image that has a custom solr config
it will fail with:

Found old non lagoon compatible dataDir config in solrconfig.xml:
  <dataDir>${solr.data.dir:}</dataDir>
sed: bad option in substitution expression

Expected behavior

Solr containers with custom configs should start without any problems

Mitigations*

This only affects solr containers that have custom solr configs added during the build step, containers that use the upstream amazeeio/solr:X.X directly are not affected
customers that experience issues can use the previous lagoon base image versions: amazeeio/solr:X.X-v1.6.0 if they need deployments to succeed.

v1.7.1 has been released that fixes this issue


add udp reachable check

add integration test for udp

Checklist

[x] make test-all (UNIX) passes. CI will also test this
[x] unit and/or integration tests are included (if applicable)

Description of change

In order to solve https://github.com/aelsabbahy/goss/issues/542

For what it's worth, I think the whole fixes #542 works in the comments too. I'll see if this comment ends up closing it when the ticket is merged

@weakliwi I'll review and merge this sometime tomorrow.

Sorry for running late on the merge of this.
As long as there are no merge conflicts you don't have to keep merging or rebasing master.
I'll try to get this done tomorrow or first thing Monday.

Sorry for running late on the merge of this.
As long as there are no merge conflicts you don't have to keep merging or rebasing master.
I'll try to get this done tomorrow or first thing Monday.

OK. Get it

Looks great, thanks!

@pedroMMM I was wrong, it didn't auto mark #542 as closed. Closing it manually.


Java8 code optimizations

@mfussenegger Please review

Current coverage is 81.06% (diff: 62.96%)

Merging #4358 into master will decrease coverage by 0.09%

@@             master      #4358   diff @@
==========================================
  Files          1306       1306
  Lines         44249      44217    -32
  Methods           0          0
  Messages          0          0
  Branches       7029       7027     -2
==========================================
- Hits          35910      35844    -66
- Misses         5692       5720    +28
- Partials       2647       2653     +6

Powered by Codecov. Last update 6679553...dfcd517


UI controls have unexpected behavior compared with demo

In the demo the controls such as play/fwd, rwd seem to "depress" however, when using it, the play seems to scale very largely for a moment and play; while, the seek buttons dont have any visual indication that you've pressed the buttons.
https://github.com/LunatiqueCoder/react-native-media-console/assets/75731066/ad30b986-c6b9-4b9a-9df7-a27d10112f09

I can easily add some opacity feedback when the button is being pressed by using the <Pressable /> API from react-native. I don't understand how/why the the play button scales largely, it's definitely not implemented by us, and also why the <TouchableHighlight /> doesn't work like in the demo anymore. Anyway, will be looking into it :/

@LunatiqueCoder thank you, I appreciate you looking into this!

has there been any update to this? @LunatiqueCoder

@LunatiqueCoder thank you for the response! I appreciate it and totally understand.


Understanding Uninitialized Variables

Understanding Uninitialized Variables
"Instructions
Initialize the three variables a, b, and c with 5, 10, and "I am a" respectively so that they will not be undefined."
If you set var c = "I am a" it will not pass as it is looking for var c = " I am a String!"
Chrome
windows 10
Surface Pro 3

Disregard I did not read fully

Please use the Help chat room for getting challenge related help. Happy Coding!


Version 1.57 (All resources are now embedded in EXE and  all settings are now stored in INI)

Changelog:
1.57

Optimized code a little bit. Now application uses 200 Kb less RAM
Now all image resources are embedded in EXE
1.56
Fixed a crash when GraphicsConfig.exe couldn't be found
Fixed a crash when Config file has wrong or corrupted entries
1.551
Fixed a bug that prevented config file to be readable in some cases
Renamed a few variables
1.55
More code refactoring
Settings are stored in *.ini file now instead of AppData\Local

Oh that's some nice changes there! Especially like how the images are embedded inside the .exe now. That is something I always wanted to do. How did you achieve this? Via the resource manager?
Also there's a merge conflict with the README.md file again. Make sure you resolve that proper so we can merge it.

Ah! Problems with README.md. How unoriginal, Mr GitHub! I will fix them :-)
So how I embedded images?

In Solution Explorer mark every image as "Embedded Resource" instead of
"None"
Now you have to call them in code in a little bit different way (look and
you will see how I did that).
If you are still confused - that may help:
http://www.csharp411.com/embedded-image-resources/

On Mon, May 11, 2020 at 4:11 PM Dennis S<EMAIL_ADDRESS>wrote:

Oh that's some nice changes there! Especially like how the images are
embedded inside the .exe now. That is something I always wanted to do. How
did you achieve this? Via the resource manager?
Also there's a merge conflict with the README.md file again. Make sure
you resolve that proper so we can merge it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-626692325,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F35YQWFOENUWZ3KHWHLRQ72P3ANCNFSM4M54D3WA
.

Are you busy?
I hope you are ok and just have no time to look at this small project :-)
On Mon, May 11, 2020 at 8:47 PM Igor Sandu<EMAIL_ADDRESS>wrote:

Ah! Problems with README.md. How unoriginal, Mr GitHub! I will fix them :-)
So how I embedded images?

In Solution Explorer mark every image as "Embedded Resource" instead of
"None"
Now you have to call them in code in a little bit different way (look
and you will see how I did that).
If you are still confused - that may help:
http://www.csharp411.com/embedded-image-resources/

On Mon, May 11, 2020 at 4:11 PM Dennis S<EMAIL_ADDRESS>wrote:

Oh that's some nice changes there! Especially like how the images are
embedded inside the .exe now. That is something I always wanted to do. How
did you achieve this? Via the resource manager?
Also there's a merge conflict with the README.md file again. Make sure
you resolve that proper so we can merge it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-626692325,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F35YQWFOENUWZ3KHWHLRQ72P3ANCNFSM4M54D3WA
.

Oof yeah I am sorry. I am currently in the finishing phase of my Master Thesis paper and have to divert all my time into it since my due date is the 26.05 and my Profs send me some feedback that I have to take into consideration and rework my paper. 😬
Once I am done with this I'll come back to this 😀

Oh Ok! Don't bother with that :-)
Good luck with studying!
On Thu, May 14, 2020 at 3:57 PM Dennis S<EMAIL_ADDRESS>wrote:

Oof yeah I am sorry. I am currently in the finishing phase of my Master
Thesis paper and have to divert all my time into it since my due date is
the 26.05 and my Profs send me some feedback that I have to take into
consideration and rework my paper. 😬
Once I am done with this I'll come back to this 😀
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-628616415,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F34CJQADQAL6OXQE27DRRPTELANCNFSM4M54D3WA
.

Alright I am back in business now that I finished my Thesis paper 😁 . Gotta review this today!

Other than this your changes look pretty nice and work pretty well! Once you fixed the above issue @IgorTheLight we should be able to merge this.

Oh thanks for reply!
I'll fix it tomorrow. My refactoring probably goes wrong at some lines :-)
I hope your study project went fine!
On Mon, May 25, 2020 at 5:52 PM Dennis S<EMAIL_ADDRESS>wrote:

Other than this your changes look pretty nice and work pretty well! Once
you fixed the above issue @IgorTheLight https://github.com/IgorTheLight
we should be able to merge this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-633608536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F3ZUSNFE4VN3AVUOZSDRTKA4NANCNFSM4M54D3WA
.

I fixed the problem!
Please confirm that it's fine now.
On Mon, May 25, 2020 at 5:52 PM Dennis S<EMAIL_ADDRESS>wrote:

Other than this your changes look pretty nice and work pretty well! Once
you fixed the above issue @IgorTheLight https://github.com/IgorTheLight
we should be able to merge this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-633608536,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F3ZUSNFE4VN3AVUOZSDRTKA4NANCNFSM4M54D3WA
.

@IgorTheLight Alright, i'll test it ASAP. BTW I see you wanted to implement Winter Assault support but reverted. What happened? 😀
Another idea could be to add support for Dawn of War Game of the Year edition as in supporting vanilla DoW too.

True - I'm implementing Winter Assault support.
The reason why I reverted the changes is because I want you to accept my
pull request (1.57).
So as you did just that (thank you btw :-) ) I will continue to develop WA
support. Actually - it worked just before I reverted it.
About supporting the original version. Well... I probably add it also! Just
for "100% complete experience".
On Tue, May 26, 2020 at 5:59 PM Dennis S<EMAIL_ADDRESS>wrote:

@IgorTheLight https://github.com/IgorTheLight Alright, i'll test it
ASAP. BTW I see you wanted to implement Winter Assault support but
reverted. What happened? 😀
Another idea could be to add support for Dawn of War Game of the Year
edition as in supporting vanilla DoW too.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-634079901,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F32F7YKLHSCE4N7LZL3RTPKOLANCNFSM4M54D3WA
.

Hi!
Winter Assault mod support is working! Original Dawn of War mod support is
probably working - I not tested it yet. Mod Manager will ignore original
files if Winter Assault is in the same directory (for now). I will improve
that in the near future :-)
Also I'm preparing new "kinda big" release - 1.60
New features:

Winter Assault mod support - already working
Original mod support - testing
Original and WA LAA patching - working (without any additional work from
me)
NEW advanced Start option - High Priority + CPU threads restrictions. It
could improve performance on some systems. If not - it's OFF by default so
no harm is done :-)
More information here:
https://www.moddb.com/members/psykotenshi/blogs/how-to-improve-performance-on-dawn-of-war-soulstorm
The last thing is not working - options don't want to apply. I'm figuring
what's the problem.
I will not make new pull request for now - everything is not tested at all
so 1.57 must be much more stable.
If you are interested I could upload a .NET Core version of DoW Mod
Manager. I'm not very pleased by it for now - GUI Designer is buggy,
application CAN'T be 32-bit and also it takes more RAM (part of the "64-bit
application only" problem). If you are curious - no, I'm not planing to
replace .NET Framework for .NET Core - I see no point in that. It's Open
Source and cross platform but none of that would benefit our little Mod
Manager :-)

On Tue, May 26, 2020 at 9:24 PM Igor Sandu<EMAIL_ADDRESS>wrote:

True - I'm implementing Winter Assault support.
The reason why I reverted the changes is because I want you to accept my
pull request (1.57).
So as you did just that (thank you btw :-) ) I will continue to develop WA
support. Actually - it worked just before I reverted it.
About supporting the original version. Well... I probably add it also!
Just for "100% complete experience".
On Tue, May 26, 2020 at 5:59 PM Dennis S<EMAIL_ADDRESS>wrote:

@IgorTheLight https://github.com/IgorTheLight Alright, i'll test it
ASAP. BTW I see you wanted to implement Winter Assault support but
reverted. What happened? 😀
Another idea could be to add support for Dawn of War Game of the Year
edition as in supporting vanilla DoW too.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-634079901,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F32F7YKLHSCE4N7LZL3RTPKOLANCNFSM4M54D3WA
.

That's excellent news! Yeah I know psyko tenshi well, he's part of my discord moderators 😁 . Support for Winter Assault and base game is also appreciated.

I'm glad to hear that! :-)
I think I could show my result in a few days. But if you are really bored -
you may build project using my last commit. I mean - if you are really
bored!
Also - do you know someone from Ultimate Apocalypse team? If yes - do they
care about new (hopefully not bugged) features?
On Wed, May 27, 2020 at 7:21 PM Dennis S<EMAIL_ADDRESS>wrote:

That's excellent news! Yeah I know psyko tenshi well, he's part of my
discord moderators 😁 . Support for Winter Assault and base game is also
appreciated.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-634778275,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F37LLCP3KDFQS4HVJLTRTU4YZANCNFSM4M54D3WA
.

@IgorTheLight I am a former UA team member. You may know me as Cosmocrat from Moddb. This tool is still demanded in the community and any changes appreciated 😀 .

Wow! That's nice to hear!
About my progress - game threads optimizations are working now! For some
reason not process affinity, nor process priority of the game couldn't be
changed right after it starts. So I have to wait a few seconds after launch
to change it. And of course users wouldn't know about that and probably
would close Mod Manager after starting a game so I had to use
System.Threading to ensure loyalty... I mean... to ensure that process
priority and affinity WILL change no matter what :-)
Original DoW mods are probably supported but I could find them so... :-)
Stable release will be available in a few days.
On Thu, May 28, 2020 at 6:12 PM Dennis S<EMAIL_ADDRESS>wrote:

@IgorTheLight https://github.com/IgorTheLight I am a former UA team
member. You may know me as Cosmocrat from Moddb. This tool is still
demanded in the community and any changes appreciated 😀 .
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Fragjacker/DoW-Mod-Manager/pull/5#issuecomment-635412069,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APM5F3ZTVAZDWPRX2FQMCFTRTZ5PNANCNFSM4M54D3WA
.


[Wait for #1828] [Tensor Pool] Add expose/persist concept

Dependency of the PR

[Pending commit: #1828]

Commits to be reviewed in this PR
[Network Graph] Split max_exec_order
This patch split max_exec_order to three part.

graph_exec_end: graph execution end
backward_iter_end: end node of backward operation
forward_iter_end: end node of the forward operation

Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>
[ExecOrder] Add exec order control and fix inference
This patch add exec order control and reduce memory overhead of saving
inference result
exec order cotrol has two part

Additional exec order. This enable graph to add some exec order.
Expose bit. This enables request to be visible after the end of the
execution.

Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>
[Tensor Pool] Add expose/persist concept
This patch add expose/persist concept to the tensor pool and manager.
When a tensor is exposed, this means that the tensor is gauranteed to
remain valid max_exec where max_exec is the value passed along allocateTensors(max_exec);
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

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


High CPU load for streaming data

Hey,
I built this streaming data chart (with 2.x-beta) and experienced a high CPU load.

The chart shows 1000 data points and calls update() @30fps. This results in 80-90% CPU load for a core @3.5Ghz.
I think this is heavy load for drawing some points, isn't it?

Definitely a bug. There were some comments on the performance in #1868
I'm thinking that as soon as 2.0.0 goes out the door, we will do a lot of investigation into improving performance. Some things we are considering: rendering to of different canvas and then simply copying bits to the main canvas.
If possible, can you drop your sample in a JSfiddle? It would give us a starting point to test with.

Sure,
JSFiddle: https://jsfiddle.net/jkhr7vby/2/
and here a Chrome CPU profile of my local page:
streaming-chart.zip

Seems like chart.js really needs the whole 33ms for calculating the new data for the 1000 points. Here is a FlameChart for 1 update

Thanks for the details! This will make it much easier to track down. I think we should be able to optimze some time out of the scale updates (first 8ms) and we can definitely improve the performance of the drawing. It really shouldn't take 20s to draw.

@etimberg I could catch ~7ms from a total of 34ms for update() (in chrome) just by optimizing some often used functions. Replace for..in with for, removed slice of arguments because V8 cannot optimize functions that slices the arguments array etc..
diff --git a/src/core/core.helpers.js b/src/core/core.helpers.js
index 8607dd0..6ce875e 100644
--- a/src/core/core.helpers.js
+++ b/src/core/core.helpers.js
@@ -11,23 +11,31 @@

 	//-- Basic js utility methods
 	helpers.each = function(loopable, callback, self, reverse) {
-		var additionalArgs = Array.prototype.slice.call(arguments, 3);
 		// Check to see if null or undefined firstly.
 		if (loopable) {
-			if (loopable.length === +loopable.length) {
-				var i;
+			var len = arguments.length;
+			var additionalArgs = [];
+			var i;
+			for(i = 4; i < len; i++) {
+				additionalArgs.push(arguments[i]);
+			}
+
+			if (Object.prototype.toString.call(loopable) === "[object Array]") {
+				len = loopable.length;
 				if (reverse) {
-					for (i = loopable.length - 1; i >= 0; i--) {
+					for (i = len - 1; i >= 0; i--) {
 						callback.apply(self, [loopable[i], i].concat(additionalArgs));
 					}
 				} else {
-					for (i = 0; i < loopable.length; i++) {
+					for (i = 0; i < len; i++) {
 						callback.apply(self, [loopable[i], i].concat(additionalArgs));
 					}
 				}
 			} else {
-				for (var item in loopable) {
-					callback.apply(self, [loopable[item], item].concat(additionalArgs));
+				var keys = Object.keys(loopable);
+				len = keys.length;
+				for (i = 0; i < len; i++) {
+					callback.apply(self, [loopable[keys[i]], keys[i]].concat(additionalArgs));
 				}
 			}
 		}
@@ -48,7 +56,12 @@
 		return objClone;
 	};
 	helpers.extend = function(base) {
-		helpers.each(Array.prototype.slice.call(arguments, 1), function(extensionObject) {
+		var len = arguments.length;
+		var additionalArgs = [];
+		for(var i = 1; i < len; i++) {
+			additionalArgs.push(arguments[i]);
+		}
+		helpers.each(additionalArgs, function(extensionObject) {
 			helpers.each(extensionObject, function(value, key) {
 				if (extensionObject.hasOwnProperty(key)) {
 					base[key] = value;

I think there will be more, but this needs some time to search for :) and the biggest effects will be the points you mentioned. There are many Object and Array loops and concatenations in the library, maybe there is a way to reduce some of them, but I did not understand the full code base yet ;)

Great job! That's over 20% :smile:

@etimberg replacing the apply() function in each() with call() (call is ~80% faster) takes it to ~19ms. The each() function is never called with additional args in the current codebase and therefore apply() is not needed. Nearly 50% now, but the CPU usage is still high.

:+1:

@mathiask88 I tested your diff. It worked great! The only change I had to make was the outer if.
if (helpers.isArray(loopable)) {

} else if (loopable) {

}

I'd be happy to merge a PR for this if you have one.

PR is out, but I think only the canvas draw calls are causing the high CPU load :(

:( what size is your canvas? If you're on Windows and using Chrome, trying resizing the canvas to be smaller than 256px * 256px. Chrome will render small canvases with hardware rendering. I'd be curious to see what that does to the CPU usage.

It is exactly the same on a 200x100px canvas. I also tested to prerender the element points one time and then just do drawImage() for every draw(), but that was not faster either.

Joke aside, I got the update time down to ~15ms now (for this test case 1000 points, no lines, no animation, 30fps) I will open another PR to discuss some small changes. Chrome uses hardware acceleration also for a large canvas. My CPU usage is ~30% lower than 2 days ago and GPU is ~9% . But still far far away of getting this working on mobile :D Maybe it is too ambitious.

Nice! I'm sure WebGL would be possible, not sure how hard it is. Might make a nice extension in the future.

Currently I'm at 8ms :) PR will come some time soon.

That's great, can't wait to see the PR

@etimberg Maybe it is possible to implement pixi.js as renderer. It is a WebGL renderer with canvas fallback and a high level API comparable with the canvas API (https://pixijs.github.io/docs/PIXI.Graphics.html#lineTo)

I implemented pixi.js temporarily and I think it is too high level. The high object count takes ages to render if they have to be created every frame.

Closing since I think this was adequately resolved.

I believe this is still a problem right? I'm seeing a cpu usage around 50% (on a I5 6600K OC @ 4.6Ghz) for real time streaming of just 2000 datapoints with animations disabled. A new point is added every 100ms. That is a little worse than what I'm seeing with highcharts (without boost module). Using timestamps for the x axis {x:1518836329 , y: 1000} - dunno if it's moment which is highly ineffective at converting this or what?

@sorenhoyer would be interesting if you can share your chart, we could investigate reasons why it's slow.

@simonbrunel thanks. I just put up a repo of my tests: https://github.com/sorenhoyer/chart-js-vs-highcharts-live-data-performance-test/ - Chart.js example in master and Highcharts in the highcharts branch.
It's my first time using Charts.js so it's also highly likely that I miss something. :)

@simonbrunel  Here's an example: https://github.com/sorenhoyer/chartjs-timeseries-streaming-test/blob/master/chartjs-timeseries-streaming-test.html
A bit more realistic (npm install & npm start): https://github.com/sorenhoyer/chart-js-vs-highcharts-live-data-performance-test/
I need to be able to render 2000+ points (1 new every second) without the CPU going berserk. Right now I'm not seeing what I would have expected. Eventhough chart.js uses canvas it doesn' seem to do better than Highcharts which is svg. Is this the full performance we're seeing with chart.js or am I doing something wrong? :)

In your first example, animations are not disabled because animation: false needs to be in the options and not under the dataset (I guess it's a typo). When disabling animations in this example and updating data every second (not every 100ms), CPU is way more relax.
Time seems spent in drawing the 2000 points (not the line) and updating the time scale layout. In your React example, there is also almost half of the time spent by the mobx::get module (but I'm not sure what is it).
I agree that Chart.js needs optimizations but I don't really have solutions for you right now (neither time to investigate deeper these days). Not drawing points seems to make performance much better but it may not be a solution for you. Also, why updating the chart every 100ms if you want to add new value every second?

Good points @simonbrunel - thanks for looking into it.
After removing animations it actually performs better than highcharts (highcharts may have more settings to tweak and a little more gfx enabled) with the boost module enabled.
Of course both charts can probably be tweaked more, so just wanted to say that chart.js does indeed seem to perform quite well. My mistake!
Keep up the good work.

Okay my examples sucked. Just came across this. Considering Chart.js is canvas based it shouldn't be 10x slower than plain canvas, dygraphs and d3 canvas: https://github.com/signalfx/charting-library-comparison


[SPARK-19734][PYTHON][ML] Correct OneHotEncoder doc string to say dropLast

What changes were proposed in this pull request?
Updates the doc string to match up with the code
i.e. say dropLast instead of includeFirst
How was this patch tested?
Not much, since it's a doc-like change. Will run unit tests via Jenkins job.

Test build #73705 has started for PR 17127 at commit f3f5fd1.

Test build #73705 has finished for PR 17127 at commit f3f5fd1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73705/
Test PASSed.

Merged into master, thanks.


Use the new HTML  tags to simplify the 's of all the pages

Currently there is a lot of boilerplate included in the  of each page. Instead of copying and editing this for every page, it would be nice if this could be included from one file.

This has now been added!


Pluggable datasets

Currently it is possible to plug you own model for it to be evaluated. This can be done by providing a --t2t_usr_dir parameter to t2t-trainer, as explained in the readme, without needing to modify tensor2tensor itself.
However, the same is not possible with datasets. In order to provide your own dataset, you need to modify tensor2tensor itself, as described data generators README.
It would be nice to have a mechanism for custom datasets that is analogous to that for models.
If it is seen as something good but it's not to be implemented in the near future, are contributions in that direction welcome?

Thanks @noe. Yes, we're working on making datasets/problems pluggable.

I believe that the trainer, as well as the datagen, is not 100% "pluggable".
The data generator needs to modify _SUPPORTED_PROBLEM_GENERATORS.
The trainer may also need to add an entry to problem_hparams.PROBLEM_HPARAMS_MAP.
My "solution" is to have my own copies of t2t-trainer and t2t-datagen with minimal changes.
So, I start training by "python my_trainer.py --t2t_usr_dir......." (in Bash on Ubuntu on Windows).
Luckily the google t2t team has made both these modules very small.
To keep my needed trainer change down to one line, I call a simple helper in my usr dir.
It means I can have my own variant Transformer model and my own problem data, without any hassle in keeping tensor2tensor up to date on my platform.

@jgevershed Wouldn't something like this work to address this without needing a forked t2t-datagen/t2t-trainer (if support was added for it, of course)?
@register_problem("image_mnist")
def mnist_problem():
  return (lambda: <generator>, lambda: <generator>)

Currently models, hyperparameter sets, and modalities can be registered without editing the T2T codebase by passing the --t2t_usr_dir command line flag. Data generators and problems cannot currently be registered in that way; however, we're working on unifying data generators and problems and adding them to the registry so that it becomes easy to plug in your own dataset & problem without editing the T2T codebase.

In v1.0.14, users can now register datasets/problems by defining a subclass of Problem and decorating it with @registry.register_problem. See algorithmic.py or wmt.py for examples.

From what I  see in the updated code of t2t-trainer, there is no equivalent to --t2t_usr_dir, which would be needed to load custom datasets from outside t2t codebase. As the issue is closed, I understand that this is not to be done, is that correct?

Ah, you're right. Oversight on my part. Will have it fixed shortly.
On Jul 17, 2017, 1:23 AM -0700, Noé Casas<EMAIL_ADDRESS>wrote:

From what I see in the updated code of t2t-trainer, there is no equivalent to --t2t_usr_dir, which would be needed to load custom datasets from outside t2t codebase. As the issue is closed, I understand that this is not to be done, is that correct?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or mute the thread.


[tasks/pipeline] Updated schedule tasks

What does this PR do?

Allow to manage other repositories (such as buildimages)
Fix pretty print error for activate schedule (the returned object is a dict, not a gitlab api object)

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Npm: Handle .tar.gz extensions for path dependencies

Dependabot has support for loading file dependencies for npm, and when
these files are larger than 1mb, we have some logic in place to retry
using the Git Data API.
We ran into a case where this would surface a Blob too large error,
indicating that the retry via the Git Data API wasn't working as expected,
and it turns out it's because we did not support the .tar.gz extension
in our code.
Previously this would cause us to assume this was a nested package.json,
(see here) so we'd tack package.json onto the
filename, which in turn would cause a 404, which in turn would cause us to
retry without package.json attached, only this time using
_github_repo_contents instead of _fetch_file_content_from_github,
which does not have logic to retry using the Git Data API.

Makes sense! I like the constant idea so we don't have to manage the list in both places but it also doesn't seem like it will change very frequently.

Yeah I think in this case maybe being able to see the values in the place where they are used is more valuable


Misleading onMinPages options when cutting PDF before predicting

Prerequisites

[x] Reproduced the problem or exposed a new need
[x] Checked the github existing issues

Description
When requesting a prediction, pageOptions allows to specify a onMinPages parameter to remove pages from a document only if the number of pages is above the value. However, when the file has less pages that the onMinPages value and pageIndexes has more indexes than onMinPages, we get the following error :
The total indexes of pages to cut is superior to the total page count of the file (3)

Steps to Reproduce

Use the following pageOptions :

pageOptions: {
  operation: PageOptionsOperation.KeepOnly,
  pageIndexes: [0, 1, 2, -1, -2, -3],
  onMinPages: 10,
}

Request a prediction for a PDF that has less than 6 pages

Expected behavior:
It should not cut the document, not throw an error and send the request.
Actual behavior:
We get the following error :
The total indexes of pages to cut is superior to the total page count of the file (3)

Reproduces how often:
100%
Versions
Version 4.14.1
Additional Information
I guess the solution would simply to invert the conditions in the extractPages method so we directly return the file if the number of pages is below onMinPages. I can open a pull request if needed.

Fixed in #297


[GUI] Prototype GraphTools and Graph Webapp /w Cytoscape

Graph-GUI Prototype
Creates visual graphs using ControlFlow.cfg and  http://js.cytoscape.org/ and renders them in a browser for analysis. Very early stage, but maybe useful nevertheless.

emma-language

compiler.tools.GraphTools.renderGraph -- Tree => Tree to be inserted into compiler pipelines (example see below). (Currently) Renders the data dependency graph as an effect to a JSON file to be consumed by GraphServlet

emma-gui

HttpServer -- Receives the location of JSON-Graphs as the first program argument (e.g. /tmp/emma/graphs/)
GraphServlet -- exposes /graph/, renders a HTML website that  renders the graphs using.

Usage (for now):
(assuming compiler.GraphTools is in scope)
Render Graphs from a Pipeline
  /*private */ val renderToTemp = GraphTools.renderGraph(java.nio.file.Paths.get("/tmp/emma/graphs")) _

  /*override*/ val liftPipeline: u.Expr[Any] => u.Tree =
    compiler.pipeline(typeCheck = true)(
      LibSupport.expand,
      Core.lnf,
      renderToTemp("transitiveclosure-post-lnf"),
      Comprehension.resugar(API.bagSymbol),
      renderToTemp("transitiveclosure-post-resugar"),
      Core.inlineLetExprs,
      Comprehension.normalize(API.bagSymbol),
      Core.uninlineLetExprs,
      renderToTemp("transitiveclosure-final")
    ).compose(_.tree)

Web-Server
Start HttpServer with command line argument /tmp/emma/graph. Graphs will be served at http://localhost:8080/graph (by default).
TODO: Proper integration, proper endpoint, proper consumer

This resolves #274.


win11 support

Hello!
I've been using deej for a couple of years now for my win10 PC. Now I would like to use it on my win11 laptop as well.
Either it is not possible to use deej with version 0.9.10 on win11, or I am doing something wrong.
I don't know how much effort is involved, but is it possible to support win11?
Many thanks in advance

Update:
Now, after restarting my laptop, it actually does work. Apparently I did something wrong but my sliders are now working.


Not getting updates on socket write?

Essentially I use:
var stream = ss.createStream();
            ss(socket).emit('file-stream', stream, {name: page.data.user.upload.name, size: page.data.user.upload.size});
            var blobStream = ss.createBlobReadStream(page.data.user.upload.name);
            var size = 0;

            blobStream.on('data', function(chunk) {
                size += chunk.length;
                console.log(Math.floor(size / file.size * 100) + '%');
            });
            blobStream.pipe(stream);

on the client
and
ss(my_socket).on('file-stream', function (stream, data) {
      console.log(data);
      stream.pipe(fs.createWriteStream(__dirname+'/temp/'+data.name));
 });

on the server...
it seems to create the file, but then proceeds to not do anything - any ideas?

@Ewan-Walker - Did you ever figure this out? I have the same problem at the moment. I have nearly the same code. It works in one case for me, but not the other. The readStream seems to be working perfecly, but the writeStream creates the file and never gets anything past that.

@rBurgett How did you solve this issue? Currently in the same situation


Update CSAF website with additional 2.0 resources

Update CSAF website with additional 2.0 resources (including links to tools, current prose, examples, etc.)
Update the navigation bar to be more intuitive and organized.

Addressed in https://github.com/oasis-open/csaf-documentation/commit/79efdcbd6159938994a14da06480096fb69842d9


Strategy for allowing admins to control resource usage

Right now, HaaS allows consumers/schedulers to consume as many nodes as they'd like with no restrictions.
We probably want to have some sort of quota system. This start off really simple (have a per-project default and allow admin override) or be more complex (like allowing for bursting).

Close this since nobody seems interested in this during the last two years. Please reopen if you think we need this.

Hm, not sure I agree with closing this going to re-open while we discuss the fate of the issue. We probably will want some way of facilitating managing resource usage, though ideally it would be in-line with HIL's "mechanism, not policy" philosophy. But right now afaik we don't even have ideas about how an administrator might do this.

I'm also going to change the title to reflect the broader concern.

I think we will want to setup quotas at some point, so I wouldn't close this just yet.

sure, long-term issue then.


Public API

Before a first somewhat stable release the public API has to be well defined.

[x] clean up the public API

[x] #808
[x] #809
[x] #852

[x] #598

[x] #744

[x] add feature flags for different API levels (also see #215)
[x] #261
[x] describe API usage in book
[x] #206
[x] #265
[x] #438
[x] #738
[x] #791

Our work here is done.


Merge feature/azure-spring-cloud-4.0 into main branch.

Merge feature/azure-spring-cloud-4.0 into main branch.

https://github.com/Azure/azure-sdk-for-java/pull/27951

Merged, closing the issue now.


Fix build

Fix gio build issue

Add a screenshot/gif showcasing the feature.

Updated

Crop the screenshot


Serialization/DeSerialization of objects with super large field fails

We know that delayed_jobs serializes jobs with YAML and save them to db for later handling, that works fine for most of the cases, however, we encountered a problem today when there is a parameter object with super long field.
The YAML is cut short right in the middle of the long string which means there is opening quotation mark without closing quotation mark. YAML parser cannot handle it, and delayed_jobs will crash immediately.
The error message is:
(): found unexpected end of stream while scanning a quoted scalar at li
ne 34 column 14
It turns out that the handler field hits the TEXT size limit of 65536, and we changed the field to MEDIUMTEXT which fixes the problem.
Just wondering whether delayed_job can truncate the fields when serializing them to YAML file so we can switch the handler field back to TEXT to save disk space.

+1...


Allow Vault backend to be pattern based

I cant seem to find a way to do this, nor does it look like its possible from looking at the code. However I would like it to be possible to make the vault backend have an option to be pattern based similar to the git backend.
This would allow  for a number of different scenarios.

Currently if I enable multiple backend types in my config server(vault and git) and I do not pass the vault token I am unable to access any properties from the server, a missing token exception is thrown. If I was able to add a pattern to match based off of the profile I could then have apps access the config server without the token and be able to get their properties. Apps that need to access the vault backend as well could just add the vault profile and specify the token. This would allow for the use of the same server for apps that don't require vault access as well as apps that do.

this would allow for setting up multiple vault backends , just like the ability to have multiple git repos.

Is there a way to do this currently ? If not, is this something that is worth implementing?

I feel like the Vault token issue was reported somewhere else, but I cant seem to find the issue right now....

Allowing the config server to ignore failures and continue processing in the composite scenario should be a separate issue.
@warroyo the work-around is to allow the apps access to vault, but have nothing there for them to read.


Add "Reviewed by you" filter for pull requests

This includes pull requests that you approved, requested changes or commented on. Currently such pull requests are not visible in any of the filters on /pulls, while they may need further action like merging, or prodding the author or reviewers.
Especially when working with a large team on a repository it's helpful to get a full overview of pull requests that may need your attention, without having to sift through the complete list.

In #16074 a number of other filters are proposed. I think for the home page it makes sense to keep the number of filters limited, both for performance and to avoid clutter. Adding more filters to the Type menu in issue lists on the other hand does not have those problems, so I imagine more could be added there.
And so a question is, what is a good set of default filters for the home page, and why should this one be on there? For me the reason is that this makes the pulls that need my attention complete. Having pulls already listed under "created by you" categorized more finely useful, but I'm usually quite aware of the state of my own pulls. Having pulls filtered by review state regardless of my involvement also is useful, but perhaps would work best to have only in the Type filter menu in a repository.

I'm not sure what you mean by util.OptionalBool?
An alternative here would be to instead add a filter Reviewed by you, which would include any pull requests where you approved or requested changes. On GitHub that filter also include pull requests that you commented on, which seems good.
If we do that, the filters together would include all the pull requests that you interacted with, which is the main thing I'm after. It also makes #22926 less important for me, though for issues it would still be nice to somehow see issues that I commented on somewhere.

util.OptionalBool is a type that can be true, false, or Not Present, so a ternary boolean.
That way you differentiate between included, excluded, and not set.
I guess that's what lunny meant.

ping LG-TM bot


Why is there a difference between Python scipy.interpolate.griddata and Matlab griddata ?

My issue is about numerical difference between Python scipy.interpolate.griddata and Matlab griddata .
Python Reproducing code:
url = 'https://cloud.ifremer.fr/index.php/s/sgnpHrP8olviOUv/download'
import wget
import os
outdir = '/tmp/'
filename = '/tmp/issue_griddata_python_matlab_linear.pkl'
if os.path.exists(filename):
    os.remove(filename)
filename = wget.download(url,outdir)
import pickle
fid = open(filename,'rb')
data = pickle.load(fid)
fid.close()
print(data.keys())
from scipy.interpolate import griddata
import numpy as np
from scipy.stats import mode
k = data['k'].astype(np.float64)
t = np.radians(data['t'].astype(np.float64))
x = data['x']
y = data['y']
d = data['input_mat']
d = d.astype(np.float64)
X = np.tile(x,[len(y),1]).squeeze().T
Y = np.tile(y,[len(x),1]).squeeze()

NTH = 72
ta =<PHONE_NUMBER>5
a1 = np.radians(ta)
a2 = t
dif = np.arctan2(np.sin(a2-a1),np.cos(a2-a1))
strr = np.argmin(abs(dif))
idd = np.mod(np.arange(strr,strr+NTH),NTH)
print(idd,len(idd),d.shape)

kx = (np.tile(k,[len(t),1]).T*np.tile(np.cos(t),[len(k),1]))
ky = (np.tile(k,[len(t),1]).T*np.tile(np.sin(t),[len(k),1]))
pts = []
for xx in range(kx.shape[0]):
    for yy in range(kx.shape[1]):
        pts.append((kx[xx,yy],ky[xx,yy]))
pts = np.array(pts)

new_pts = []
for xx in range(X.shape[0]):
    for yy in range(X.shape[1]):
        new_pts.append((X[xx,yy],Y[xx,yy]))
new_pts = np.array(new_pts)
interpmethod = 'linear'
#interpmethod = 'cubic'
#interpmethod = 'nearest'
rescale_d = False
D = griddata(points=pts,values=d[:,idd].ravel(),xi=new_pts,method=interpmethod,rescale=rescale_d) #here I use the scipy linear instead of v4 matlab option
print(D.shape,pts.shape,new_pts.shape)
D = np.reshape(D,X.shape,order='A')

The matrix returned by scipy can have up to 3% difference with the one given by Matlab12. see attached figure: https://cloud.ifremer.fr/index.php/s/ibs4KkYboZt9Wu1
Is there an explanation?
Was scipy.interpolate.griddata meant to mimic Matlab's one?
Do someone know a workaround to get exact same result?
Scipy/Numpy/Python version information:
1.3.0 1.16.4 sys.version_info(major=3, minor=7, micro=3, releaselevel='final', serial=0)

Was scipy.interpolate.griddata meant to mimic Matlab's one?

There is no requirement that they match.
If you do find either an explanation or a workaround, please share it. Preferably as a pull request for the griddata documentation.
Meanwhile, I'll close the issue; feel free to discuss of course.


Specify payee to PayableDAppFactory

This specifies the payee of a PayableDAppFactory, instead of depositing the money in the contract itself.
The transfer is now from the deployer to the payee.
The withdraw method was removed.
I believe this will pave the way better for the possible revenue share mode using Payment Splitter.

The problem is who is allowed to change the payee.
If the payee is a PaymentSplitter set by the PayableDAppSystem, the owner of PayableDAppFactory should not be allowed to change the payee (the PaymentSplitter).

@guidanoli how do you feel about @felipeargento ’s concern about payee immutability?

@guidanoli how do you feel about @felipeargento ’s concern about payee immutability?

I don't know how I feel about giving this power to the owner of the PayableDAppFactory contract. By default, I prefer contracts to be as trustless as possible. If you want to change the payee, deploy a new PayableDAppFactory contract via the PayableDAppSystem contract. Want to assign a completely different contract as the payee? Deploy your own flavor of PayableDAppSystem that allows an arbitrary pee.
This last bit could even be a feature: allow the user to decide whether they want to deploy a PayableDAppFactory with a PaymentSplitter contract as payee, or an arbitrary address instead. These could be two distinct functions.


dcarpente/stream_read_buffer_fix

Our usage case uses an IStream implementation that is marshalled across apartments. Because of the way buffers are copied, IStream::Read rewrites the input buffer with null characters even if no bytes are read.
FFMPEG (or at least the OGG container parser) expects FileStreamRead to leave the input ptr unmodified when bytesRead == 0. Since behavior on the buffer for IStream::Read is not defined when bytesRead == 0, we do an additional copy in a temporary buffer to guarantee the behavior FFMPEG expects.

@dcarpente,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

We made a change to how IMFByteStream initializes the buffer cross process in OS code, so we don't need this change any more. We should still cherry-pick part of the 531169d change in another pull request.

Thanks. Cherry-picked to master


Undo, Restore state, Generate code from GUI interactions

Generate code from GUI interactions; State restoration & Undo

https://news.ycombinator.com/item?id=26384396 (bamboolib author)

https://github.com/mito-ds/monorepo
https://github.com/tkrabel/bamboolib
https://github.com/man-group/dtale#predefined-filters
from https://docs.trymito.io/getting-started/overview-of-the-mitosheet :

Generating Pandas Code
For each edit you make to the Mitosheet, Mito generates pandas code below that corresponds to this edit, and puts this code directly below the mitosheet in the next code cell.
Rerunning an analysis
When you run mitosheet.sheet(), Mito will automatically generate a unique ID to store the set of edits make to this mitosheet. This ID will appear as an automatically generated analysis_to_replay parameter to the mitosheet.sheet() function call.
As long as you pass this analysis_to_replay parameter to the mitosheet.sheet() call, Mito will attempt to replay that analysis to the mitosheet. Replaying an analysis means applying the same edits that you did in Mito again.
Since Mito will try and apply the same edits when an analysis_to_replay parameter is passed, differently structured datasets might make these edits invalid and Mito will error. For example, if you change the location of the file that you imported in an analysis, and then attempt to replay this analysis, it will fail (as it can no longer find the file to import).
If you want to start a fresh mitosheet, simply make a new mitosheet.sheet() call in a new code cell.

With RTC, who click what in order to replay is probably somewhat helpful:

https://jupyterlite.readthedocs.io/en/latest/howto/configure/rtc.html
https://github.com/jupyterhub/zero-to-jupyterhub-k8s/issues/2488#issuecomment-1452569524
https://github.com/jupyterlite/jupyterlite/issues/464

git
GDrive,

https://github.com/jupyterlab/rtc

Generate code from GUI interactions; State restoration & Undo
* https://news.ycombinator.com/item?id=26384396 (bamboolib author)

  * https://github.com/mito-ds/monorepo
  * https://github.com/tkrabel/bamboolib
  * https://github.com/man-group/dtale#predefined-filters
  * from https://docs.trymito.io/getting-started/overview-of-the-mitosheet :

Thanks a lot! This feature sounds cool, I'll try to make it work.

This has been on my wishlist since discovering pygwalker - glad to see this issue opened.
I haven't clicked through on all the links above, but I'd love if we could just point to a file store:
gwalker = pyg.walk(df, store="myfile.json")

In the UI there would then be a save button (optionally auto-save) that saves the current config to this file.
When restarting the notebook kernel and rerunning the cell, the UI would then be repopulated with saved state. Undo/Redo is nice-to-have.
My utopia would be that this stored file/dashboard can then also be easily be imported into a standalone Graphic Walker instance, so the dashboard can then be shared with non-analysts in the team.
In this case, the user should be able to replace the data source with one that's available on the standalone Graphic Walker.

Hi @jorisroovers, the latest version of pygwalker supports you to export the chart config in the UI, you can be repopulated by this way.
Pygwalker team is also exploring some more friendly ways to export and repopulated recently.

@jorisroovers  I agree with your suggestion, we will add this feature in next version.

@jorisroovers  Hi, this feature has been added.
You can try the pre-release version first.
pre-release 0.2.0: here
Also look forward to your feedback, Thanks♪(･ω･)ﾉ

Thanks!


update readme.md file

#4
I have update README.md file

@Ayushparikh-code please have a look on this issue

and please add labels here also and my previous PR to. Thanks


fix: fix code error in quick_start.md

fix code error in example code.

Thanks, we didn't sync a commit last week that fixed this, it is now synced. Closing this.


system interrupts process spikes upon search

Issue Type: Performance Issue
Hi guys, I started noticing this yesterday while doing a global search on my project in vscode.
I've searhed on this project millions of times, nothing different. But yesterday I noticed it being VERY slow.
When I checked my processes, I saw that, during the search (and only during the search) the System Interrupts process just stayed at 100%.
As soon as the search ends (which takes way longer than normal) CPU usage for that process goes down again to like 5% total.
I'm on Windows 10 Version 10.0.18363.778.
The only other bit of info I can provide is I updated Windows recently ,these are the most recent updates.
https://support.microsoft.com/en-us/help/4549951/windows-10-update-kb4549951
I'm only reporting this to you guys since the ONLY trigger for this System Interrupts process CPU usage is VSCode search...
VS Code version: Code 1.44.2 (ff915844119ce9485abfe8aa9076ec76b5300ddd, 2020-04-16T16:36:23.138Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8650U CPU @ 1.90GHz (8 x 2112)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
15.93GB (6.71GB free)

Process Argv
--disable-gpu --disable-gpu --disable-gpu-compositing

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	    68	  3632	code main
    0	    23	  1820	   utility
    0	    60	  5132	   gpu-process
    1	    69	 14644	   window (Issue Reporter)
    0	    13	 15744	   electron-crash-reporter
    0	   229	 17092	   window (landing_page_settings_serializer.rb - SnipInt - Visual Studio Code)
    0	     6	  1324	     console-window-host (Windows internal process)
    0	    14	  2188	     watcherService
    0	     5	  6348	       console-window-host (Windows internal process)
    0	    67	  4264	     C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell.exe
    0	     5	 17616	       "C:\HashiCorp\Vagrant\bin\vagrant.exe" ssh
    0	    57	 11260	         ruby C:\HashiCorp\Vagrant\embedded\gems\2.2.5\gems\vagrant-2.2.5\bin\vagrant ssh
    0	     7	 12492	           C:\WINDOWS\System32\OpenSSH\\ssh.EXE<EMAIL_ADDRESS>-p 2222 -o LogLevel=FATAL -o Compression=yes -o DSAAuthentication=yes -o IdentitiesOnly=yes -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -i C:/Users/thata/projects/SnipInt/.vagrant/machines/default/virtualbox/private_key
    0	    36	  8572	     searchService
    0	    66	 10520	     C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell.exe
    0	     5	 11912	     console-window-host (Windows internal process)
    0	    46	 21952	     extensionHost
   13	    65	 19056	   shared-process

Workspace Info
|  Window (landing_page_settings_serializer.rb - SnipInt - Visual Studio Code)
|    Folder (SnipInt): 15727 files
|      File types: cache(8596) json(1785) rb(1438) slim(303) coffee(261)
|                  js(243) scss(136) vue(112) png(83) map(66)
|      Conf files: package.json(1) tsconfig.json(1);

Extensions (10)

Extension
Author (truncated)
Version

erb
Cra
0.0.1

gitlens
eam
10.2.1

vsc-material-theme
Equ
32.6.0

vsc-material-theme-icons
equ
1.1.4

html-slim-scss-css-class-completion
gen
1.7.4

vscode-colorize
kam
0.8.16

VS-code-vagrantfile
mar
0.0.7

remote-wsl
ms-
0.44.2

vetur
oct
0.24.0

slim
sia
0.1.2

(1 theme extensions excluded)

I'm not sure about "system interrupts", do you see other processes using high CPU during that time too? rg.exe?

hey @roblourens thanks for following up on this.
Hmm so you know what... I could not replicate the System Interrupts high CPU usage (although I have observed it several times ONLY when searching in VSCode).
However, what I can CONSITENTLY replicate is VERY SLOW searching, and yes, high rg.exe usage or, for some reason, high Windows Defender usage... but, ONLY when searching in VSCode? 🤔
I wouldn't think this was a VSCode issue EXCEPT the search is VERY slow and it the high cpu usage only happens when I search in VSCode? 🤔
I recorded a video here for you to see. In the video I do 3 searches... you'll notice they are very slow, BUT rg.exe only kicks in on the 2nd search.
https://www.loom.com/share/f5e4e5b0e417467aa9423e9768370f60
Any ideas? 🤔

Yeah, Windows Defender likes to kick in while searching, and slow it down significantly. You might have some luck if you exclude your dev folder in Windows Defender.

hey @roblourens thanks so much... just a question. What is rg.exe and why would I have not seen this behaviour before do you think? The weird thing is, it's never happened before? Any ideas?
thanks again for your help!

rg.exe is the process that actually does the search. It should be using high CPU, the fact that it's not shows that it's being throttled by Windows Defender. Unfortunately I have no idea why Defender does what it does, you are not the only person to report an issue like this and I wish there was some rhyme or reason to it, sorry.


Docs: Put try/except in loop to check if tx is valid

The docs now better-account for the fact that there's a small time delay before a transaction is in a valid block (and hence considered valid).
This resolves #209
This resolves #174

Codecov Report

Merging #244 into master will not impact coverage.

@@           Coverage Diff           @@
##           master     #244   +/-   ##
=======================================
  Coverage   98.93%   98.93%
=======================================
  Files           9        9
  Lines         188      188
=======================================
  Hits          186      186
  Misses          2        2

Continue to review full report at Codecov.

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


Roll engine 7fc22d118e17..f7c6c6d92a12 (10 commits)

https://github.com/flutter/engine/compare/7fc22d118e17...f7c6c6d92a12
git log 7fc22d118e17..f7c6c6d92a12 --first-parent --oneline
2020-04-01<EMAIL_ADDRESS>Use Metal on iOS by default. (flutter/engine#17431)
2020-04-01<EMAIL_ADDRESS>Roll src/third_party/dart 021fa0fbd410..0f89df0c00e7 (4 commits) (flutter/engine#17438)
2020-04-01<EMAIL_ADDRESS>If the application says there is a new texture but does not provide one, reuse the last texture. (flutter/engine#17437)
2020-04-01<EMAIL_ADDRESS>[fuchsia] Fix bug when applying scale. (flutter/engine#17436)
2020-04-01<EMAIL_ADDRESS>Roll src/third_party/skia 65d3ad953957..7a3358c80b6e (20 commits) (flutter/engine#17435)
2020-04-01<EMAIL_ADDRESS>[fuchsia] Re-enable shadows. (flutter/engine#17380)
2020-04-01<EMAIL_ADDRESS>Roll src/third_party/dart e8c4aed700f5..021fa0fbd410 (17 commits) (flutter/engine#17434)
2020-03-31<EMAIL_ADDRESS>Update FlDartProject to new path format (flutter/engine#17302)
2020-03-31<EMAIL_ADDRESS>GPU to raster rename, patch 2 (flutter/engine#17428)
2020-03-31<EMAIL_ADDRESS>Fix iOS builds on Xcode 11.4 for new enum values added in iOS 13.4. (flutter/engine#17429)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Commit queue failed; closing this roll.


Set parameter

Is there a way to set a parameter as well?

Currently I am using VitoWIFI compiled with Arduino to read and write to my Viessmann Heatpump.

Hi @Momo-GitHub-1
Setting parameter is not supported and I do not have plans to extend it currently.
If you want use esphome to set values as well, check this PR https://github.com/esphome/esphome/pull/4453


Create Java code sample for custom regex

To be linked from https://cloud.google.com/dlp/docs/creating-custom-infotypes-regex#regex-example
Fixes #issue
It's a good idea to open an issue first for discussion.
[yes] I have followed Sample Format Guide
[yes] pom.xml parent set to latest shared-configuration
[nothing new] Appropriate changes to README are included in PR
[nothing new] API's need to be enabled to test (tell us)
[nothing new] Environment Variables need to be set (ask us to set them)
[yes] Tests pass (mvn -P lint clean verify)
(Note- Checkstyle passing is required; Spotbugs, ErrorProne, PMD, etc. ERROR's are advisory only)
[yes] Please merge this PR for me once it is approved.

PTAL: Consider fixing the exception - most DLP samples have stopped catching exceptions.
It's best to tell developers what might happen and how to mitigate if it does.

Thanks for the reminder, I've removed the catch block and


"Support Zulip" should point to Support the Zulip project help page

Instead of linking to GitHub Sponsors, the "Support Zulip" gear menu option should link to https://zulip.com/help/support-zulip-project (note: not a relative link), which covers additional ways to support Zulip.
This should not be deployed before ac63c79.

Hey @alya I'll like to work on this. I was a Zulip contributor a long time back and would love to start contributing again.
This looks like a simple task to get started again.

@zulipbot claim

To prevent someone else from trying to pick this up and redo it, let me just post a quick update here.
The PR is ready its waiting on some other dependent issue to be merged after which this can easily be merged, and we can close the issue.

@zulipbot claim

@Rahul-Retharekar Just wanted to let you know that, as the previous comment states, I've already opened up a PR that fixes this issue


Set hex color format to lowercase

Reference Issues/PRs
What does this implement/fix? Explain your changes.

Set all hex color format to lowercase

Any other comments?

Codecov Report

Merging #132 (c06bc16) into dev (4de5e89) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               dev      #132   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          438       438
  Branches        68        68
=========================================
  Hits           438       438

Impacted Files
Coverage Δ

samila/functions.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Export CSV with field aliases instead of field names in Manager

Describe requirements
We would like Manager to export the CSV with field aliases rather than the field names in column headings. Experience Builder Table widget does that

Alternatives
No response
Other Relevant Info
No response
Impact
No response

Ready to test after next build

verified in devext


Dynamic Sprite Font

I think it could be useful to dynamically render more characters as requested. In languages with a lot of characters (Chinese, Japanese, etc) it can make more sense to render characters as needed, instead of having to determine everything upfront.
This would probably work by creating some kind of queue to render characters off the main thread as requested, and then updating the SpriteFont's Texture each frame if there are more characters that have finished being rendered. If you draw a character that hasn't been rendered yet, it could add it to the queue and then just show nothing until a future frame. If you want to avoid a few frames of text "pop in" there could be some way to request characters ahead of time.
I'm not sure if this should just be part of SpriteFont, or a different DynamicSpriteFont class entirely. It needs to be more complicated (ex. rendering characters off the main thread, updating the main texture on the main thread each frame, etc).
Alternatively, if this feels overly complex for the framework, this could just be entirely left up to the end user. SpriteFont already has a way to add user-created characters.

I'm also potentially interested in adding MSDF fonts, which I have working nicely in some non-Foster projects. But they require including a C++ library, and don't work with a lot of fonts without modification.

(M)SDF fonts sound interesting. Raylib supports normal SDF font rendering, and I believe it just uses stb SDF rendering. It may be lower lying fruit/smaller dependency than full MSDF.

yeah SDF fonts are much easier to add ... could be worth dropping those in along with a default shader for them.

I implemented a first pass of this. SpriteFont now dynamically updates internal texture pages as characters are requested, so you no longer need to feed in what characters you need ahead of time. I'm sure there's issues, but it's functional!


[Question] How to record locally

I understand we have API setLocalFileAbsolutePath. But I am not sure how to use it to locally save video to file.

mProfile.setVideoQuality(videoquality)
.setAudioQuality(audioquality)
.setEncodingSizeLevel(StreamingProfile.VIDEO_ENCODING_SIZE_QVGA)
.setStream(stream)
.setLocalFileAbsolutePath(Util.getVideoOutputPath() + "/recording.mp4")
.setSendingBufferProfile(new StreamingProfile.SendingBufferProfile(0.2f, 0.8f, 3.0f, 10 * 1000));

If I use the above then streaming happens but recording to local file does not happen. if I comment ut setStream(stream), then we get IOError saying rtmp is null.
Please guide.

In the logs I see
09-25 16:56:24.272 30956-32748/in.voola.android.staging I/CameraStreamingManager: startStreaming mRecordingEnabled=false,mIsPreviewReady=true,mIsOnlyAudioStreaming=false
so I assume there has to be a way to set mRecordingEnabled=true

Looking for help...Anyone used this?

Sorry for the late replay as the Dragon Boat Festival. We are focusing on the stability of the sdk，thus the feature you requested haven't implemented.
In fact, it is very easy to do that if use pili streaming cloud service. pili will store the streaming data into Qiniu cload storage, and provide the api to covert the streaming data to mp4 or other types.
At present, the locally recording had been failed after removing the ffmpeg dependency. I'm considering add it back now.

"Sorry for the late replay as the Dragon Boat Festival". Did not know about it. Hope you enjoyed the festival
"In fact, it is very easy to do that if use pili streaming cloud service". This is one option, but the easiest is to save it locally, so that users do not have to download it and they can record the moment even if there is no internet
"I'm considering add it back now." - Great news, thanks a lot.

Sorry. I'm afraid i have no time to implement the recording locally.
But i have an idea that the sdk can provide the callback which take the h.264/aac encoding datas as the parameter, and you can use mp4parser to do it in the callback.
What do you think of the idea?

That sounds fine to me. But I guess you might have to elaborate it a bit
about the exact callback that PILI would expose and how exactly I can use
that data. A small example would greatly help.
On Oct 8, 2015 7:38 PM, "jpxiong"<EMAIL_ADDRESS>wrote:

Sorry. I'm afraid i have no time to implement the recording locally.
But i have an idea that the sdk can provide the callback which take the
h.264/aac encoding datas as the parameter, and you can use mp4parser
https://github.com/sannies/mp4parser to do it in the callback.
What do you think of the idea?
—
Reply to this email directly or view it on GitHub
https://github.com/pili-engineering/PLDroidCameraStreaming/issues/51#issuecomment-146557020
.

Will you be also exposing API that we can use similarly for adding our own filters in upcoming version?

Any estimate when can we expect this feature?

A quick ping to check if there is any estimated time for the next update release

Is this possible in the new 1.4.1?

What is the recommended way to record byte[] frames in OnPreviewFrame to mp4 file?

Still waiting for your response on how to record locally in v1.4.1

Is recording possible in 1.4.1?

Is recording possible in 1.4.3?

We thought the locally recording is not good idea for live streaming. There is no plan to support that.

The IOS eqivalent of this library (PLCameraStreamingKIt) allows to record locally. So do you still think it is not required in Android version?


Switch not responding to props change

When the props (selected , disabled) changes after a Switch component is initiated , it does not change its shape accordingly, thus I cannot create a controlled Switch.
The props only affects the initial render of a Switch.
A quick example to reproduce the bug:
https://codesandbox.io/s/jvkx6zoj75
This might be caused by the upstream as stated here:
https://github.com/material-components/material-components-web/tree/master/packages/mdc-switch#initially-disabled-switch
Maybe we should use MDCSwitchFoundation.setDisabled and MDCSwitchFoundation.setChecked?

Thanks for the repro.. will look into it

I've already created a PR addressing this #1190

@prateekbh the pr is ready to be merged

taking a look now


Npm script rule forces unnecessary duplication

The pattern I'd like to lean into, is to have npm always 'drop in' to wireit and exploit its dependency support, meaning named npm script targets are mostly cruft I have to scroll up and down and maintain.
Experimenting with the below package.json config I received the error at bottom, requiring that there should be a matching named npm script target for every entry in wireit for a reason I don't understand. Honestly this seems totally unnecessary and will just lead to duplication, scrolling and errors for no reason that I can see (I am ready to be educated).
I would rather have this requirement removed, at least for the top-level package in a monorepo, and probably for all packages (logic would look in both scripts and in wireit). Having the targets ONLY defined in the wireit property is preferred in some or many cases.
In this context I am also desperate for features around https://github.com/google/wireit/issues/23 to allow wildcarding package paths, (package directories) and defining upstream packages (package dependencies that live in the same repo) so that relations can be maintained in a terse way with minimal manual duplication.
You can see how e.g. the wireit.test.dependencies has to be manually maintained, (to duplicate information already in the root package.json workspaces and monorepo folder structure) and within example:test the upstream dependencies of build ( "packages/example-consumer1:test", "packages/example-consumer2:test" ) have to be manually maintained (to duplicate information already in package.json dependencies).
Not requiring named npm scripts is another way to avoid pointless duplicate info. Would it be good to remove this code (both from wireit and from people's package.json).
  "scripts": {
    "qa": "wireit",
    "qa:watch": "npm run qa --watch"
  },
  "wireit": {
    "test": {
      "dependencies": [
        "packages/example:test",
        "packages/example-consumer1:test",
        "packages/example-consumer2:test"
      ]
    },
    "lint": {
      "command": "eslint --color --cache --cache-location .eslintcache .",
      "files": [
        "**/*.ts",
        ".eslintignore",
        ".eslintrc.cjs"
      ],
      "output": []
    },
    "qa": {
      "$comment": "'test' forces build on a per-project basis",
      "dependencies": [
        "test",
        "lint"
      ]
    }
  },

[github/starter]$ npm run qa

><EMAIL_ADDRESS>qa
> wireit

❌ package.json:29:5 Script "test" not found in the scripts section of this package.json.
        "test": {
        ~~~~~~
❌ package.json:34:5 Script "lint" not found in the scripts section of this package.json.
        "lint": {
        ~~~~~~

This has been implemented in https://github.com/google/wireit/pull/653


Update status with music you are listening to

Skype has a feature where it updates your status with the music you are listening to on iTunes. Be nice to give the option for iTunes or Spotify.

I don't think that yakyak currently supports the status


How to rename the layout id '__layout'

What problem does this feature solve?
I want rename the layout div's id name , but I can't find solution from nuxt.js docs site.
What does the proposed changes look like?
maybe can in nuxt.config.js setting
globals: [
layoutId: 'customId'
]

This feature request is available on Nuxt community (#c9749)

What would be the purpose of that feature?

The easiest way would be to override /template/App.js by creating a /app/App.js file in your source root and copy the content from the original App.js but remove the domProps: {         id: '__layout'       } part.


Urdu Artificial Text Detection: Train and evaluate a CNN to detect presence of Urdu Text on the screen. Briefly mention if the CNN is a valid family of models to use for this task with your reasoning.

Field
Description

About
A short Description about project

Github
Your Github name

Email

Label
Update request

Define You

[ ] Hacktober Fest Contributor

Is your feature request related to a problem? Please describe.

Describe the solution you'd like...

Describe alternatives you've considered?

Approach to be followed (optional):

Additional context

I would like to work on this task. We can use object localization using a simple CNN or a ResNet.


About us video

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

Original: https://www.esri.com/en-us/about/about-esri/asia-pacific
Before: https://main--esri--aemsites.hlx.live/en-us/about/about-esri/asia-pacific
After: https://about-us-video--esri--aemsites.hlx.live/drafts/asimon/about-asia-pacific

Not sure if it has to do with this PR, but the "Contact your local ESRI office" button should have a longer transition for the hover effect.
Besides that, PR looks good to me.


UnicodeEncodeError when downloading songs

I have a lot of German, Japanese and Korean songs including a lot of special characters in my library and am currently trying to automatically download them to my machine. The english songs pass fine but the rest seems to fail:
Traceback (most recent call last):
  File "/home/backup/backup-scripts/virtualenv/bin/gmsync", line 11, in <module>
    sys.exit(main())
  File "/home/backup/backup-scripts/virtualenv/lib/python3.4/site-packages/gmusicapi_scripts/gmsy$
c.py", line 150, in main
    mmw.download(songs_to_download, template=cli['output'])
  File "/home/backup/backup-scripts/virtualenv/lib/python3.4/site-packages/wrapt/wrappers.py", lin
e 561, in __call__
    args, kwargs)
  File "/home/backup/backup-scripts/virtualenv/lib/python3.4/site-packages/gmusicapi_wrapper/decor
ators.py", line 22, in wrapper
    return function(*args, **kwargs)
  File "/home/backup/backup-scripts/virtualenv/lib/python3.4/site-packages/gmusicapi_wrapper/music
manager.py", line 215, in download
    for result in self._download(songs, template):
  File "/home/backup/backup-scripts/virtualenv/lib/python3.4/site-packages/gmusicapi_wrapper/music
manager.py", line 176, in _download
    os.makedirs(dirname)
  File "/home/backup/backup-scripts/virtualenv/lib/python3.4/os.py", line 237, in makedirs
    mkdir(name, mode)
UnicodeEncodeError: 'ascii' codec can't encode character '\xfc' in position 40: ordinal not in range(128)

Using python3.4 in virtualenv

Please always post full tracebacks; it usually helps to know where in the code things happen.
I believe '\xfc' is actually an umlaut (ü) in the latin-1 encoding. What locale is set on your system?

Apologies for the lack of information. The script runs automatically on a backup system and the stack trace was exposing too much information about the setup. I was in a rush yesterday and didn't have time to filter some things out. I updated the initial post 👍
But good news, the locale was indeed the issue! The debian it was running on had locales incorrectly setup. After re-generating and setting the correct locales the problem disappeared.
Thanks for the fast answer!


ipodtouch0218: Blacklist<PHONE_NUMBER>(?#NO NorAm)

ipodtouch0218 requests the blacklist of the number<PHONE_NUMBER>(?#NO NorAm). See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Cow in Charcoal HQ


fmWorkMate: overwrites the current chosen FileMaker when it starts

When fmWorkMate starts, on of the selected FileMakers gets overwritten. (Need to check which one)

Fixed in next version


Fix failing features for graphs

We noticed in #514 that the specs were failing on CI.  While investigating what seemed to be an unrelated change we noticed that the specs were also failing on master.  It seems that something changed in the google charts api that our JS testing infrastructure (poltergeist and phantomjs) didn't like.  This PR upgrades those components and changes how we invoke the charts API.
There's more details in each commit.

Thank yooooouuuuu @h-lame 👻  📉  Tests pass now so I think this is 🍏  but we mostly paired on this so it would be good for someone else to review too.

👻 📉 ⌚ ⭐ 📈


Swift Package Manager Integration

Moved files around according to Swift Package Manager guidelines and added Package.swift for integration with Swift Package Manager.

Thanks for the pull request @jjb3rd, just that minor comment and also could you update the source_files inside the pod spec so that the travis CI build still passes?
After those changes could you squash your comments and then this is good to go :ok_hand:.

How do I squash the comments?

@jjb3rd Check https://github.com/getpelican/pelican/wiki/Git-Tips#squashing-commits


Add age subcommand

Fixes #2098
RELEASE_NOTES=[ENHANCEMENT] Add age subcommand
Signed-off-by: Dominik Schulz<EMAIL_ADDRESS>

Can this be considered stable enough to migrate to? I'd like to retire my Password Store GPG Key and migrate to age.
However if something happens and the format changes I'll wait.