[feature]: Add Outgoing Custom TLVs to HTLC Interceptor

Is your feature request related to a problem? Please describe.
Users of the HTLC interceptor can receive custom records for the incoming HTLC, but do not have the ability to set custom records on the outgoing HTLC.
Describe the solution you'd like
Update the user-provided HTLCInterceptorResponse to allow interceptors to set CustomRecords for the outgoing HTLC.
Describe alternatives you've considered
There is no clear alternative other than upgrading the interceptor.
Additional context
This will be used to trial-run HTLC endorsement (and local reputation) as a mitigation for channel jamming attacks. Endorsement is a key part of this scheme, so propagation of this signal (using an experimental TLV) for testing purposes is important.

Fixed in #8660


Changing when expression resets the keybinding

Steps to Reproduce:

Open Keyboard Shortcuts Editor
Change the keybinding of keybindings.editor.resetKeybinding command to Backspace
Change the when expression of the same command, by right click and select Change When Expression...
Press Backspace

🐛 Keybinding gets reset

It is not a bug. Because the default when context is empty. User has to set the when context.


Addnote

What issue is this solving?
Solves 3/4 request in #41
Description
The new command cc!addnote [user] [note] takes a user mention or ID and stores note in the db.
It logs: timestamp, user id, moderator id, note to table user_notes in the db.
Command checks:

User has privileges to create a note
A target for the command is provided (a user)
Target is not an Admin, Mod, SU
Target is not self
A note was provided

Command provides feedback to the command caller in the channel the command was called with an embed message that provided details on target user, note content and timestamp.
Any helpful knowledge/context for the reviewer?

Is a re-seeding of the database necessary? No idea what this means
Any new dependencies to install? No
Any special requirements to test? Admin/Mod/SU

(e.g., admin perms, alt account, etc.)

Please make sure you've attempted to meet the following coding standards
✅  Code has been tested and does not produce errors
✅ Code is readable and formatted
✅ There isn't any unnecessary commented-out code

:warning: THIS PR WILL BREAK THE BOT IF MERGED WITHOUT ADDITIONAL WORK
This command uses a new table called user_notes. We need to make that table available in production or this will most likely crash the piñata

Created issue #95 so let @Vic-ST know that we need that table made

I sure did. git pull and git install was the first thing I did

@Victoria-DR good catch. I just pushed a fix for this. Right after it checks if a note was provided, it will also check if note.length > 255


Smooth scrolling doesn`t work in safari

Describe the bug
code :
 const scrollToText = () => {
    scrollTo({ behavior: 'smooth', top: window.outerHeight });
  }

...
<div onClick={()=>scrollToText()} className="bottom down-arrow">
                <Icon
                  name="angle double down"
                  style={{ fontSize: '2rem' }}
                ></Icon>
button
</div>

When you press the button in Safari Browser,smooth scrolling doesn`t work
But it can work properly in Chrome.
Expected behavior
it works

OS: MacOS 11.2
Browser: Safari 14.0.3 (166<IP_ADDRESS>.4)
Version: ^3.0.0-beta.4

Hey @laizeh, were you able to figure this out?


cannot install metaQC package

RT

This is because Silvia has not pushed her package to git yet. I will write an email asking her to do that. Thanks.


Absolute URL 'xref:foo' cannot be used for a RelativeUrl.

Starting with 1.0.0-beta.36 I get the following error:
[WARN] Content/Process » CacheDocuments » RenderContentProcessTemplates » ExecuteIf » RenderMarkdown » [C:/foo.md => foo.html] Exception while rendering Markdown: Absolute URL 'xref:foo' cannot be used for a RelativeUrl.

Corresponding markdown code:
[Foo](xref:foo)

The same happens with [contact me](mailto:invalid@email.address).

The issue is with this.
https://github.com/statiqdev/Statiq.Framework/blob/753b639d3bf0a0b1c642a0481e186ec4851dfaa5/src/extensions/Statiq.Markdown/MarkdownHelper.cs#L54
LinkGenerator.TryGetAbsoluteHttpUri will only work if it's https or http. I'm not sure why that is.

@JoshClose, I think the problem is RelativeUrl.IsRelative(link) returns true but LinkGenerator.TryGetAbsoluteHttpUri returns false, so the condition ends up being false and it doesn't call this:
https://github.com/statiqdev/Statiq.Framework/blob/753b639d3bf0a0b1c642a0481e186ec4851dfaa5/src/extensions/Statiq.Markdown/MarkdownHelper.cs#L56
Since it doesn't, it assumes it is a relative link since there is no if (!RelativeUrl.IsRelative(link)) check before it goes into the relative link code starting with:
https://github.com/statiqdev/Statiq.Framework/blob/753b639d3bf0a0b1c642a0481e186ec4851dfaa5/src/extensions/Statiq.Markdown/MarkdownHelper.cs#L60

Yes. I'm going to submit a pull request with this change.
if (!RelativeUrl.IsRelative(link))
{
    LinkGenerator.TryGetAbsoluteHttpUri(link, out string absoluteUri);

    return absoluteUri ?? link;
}

Since we know it's not relative, if it can't get an absolute URI, just return the original link.


custom matching functions

I want to be able to define a boolean function that can be used in searches.
The function will have a string as an argument and return a boolean (you know... like a boolean function). If the result of the function is true then we match if it is false then we backtrack.
set divisibleBy3 to predicate
    (at least 1 digit)                          -- it might be nice to control the input with a search query
begin
    set result = 0
    for c in input
        result += toDigit(c)
    end
    return result % 3 == 0
end

This will match on number strings that are divisible by 3

This is done with the patterns


Check return code cvmfs_config at the end of installation

The return code of cmvfs_config setup should be checked in the action as it is not the last command in the setup-cvmfs.sh script.

I just realized there are situations like this
Reading package lists...
E: Failed to fetch http://cvmrepo.web.cern.ch/cvmrepo/apt/dists/bionic-prod/InRelease  403  Forbidden [IP: <IP_ADDRESS> 443]
E: The repository 'http://cvmrepo.web.cern.ch/cvmrepo/apt bionic-prod InRelease' is not signed.
Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package cvmfs
Reading package lists...
Building dependency tree...
Reading state information...
E: Unable to locate package cvmfs-config-default
# cvmfs default.local file installed with cvmfs-contrib/github-action-cvmfs
CVMFS_CLIENT_PROFILE='single'
CVMFS_QUOTA_LIMIT='15000'
CVMFS_USE_CDN='yes'
Run cvmfs_config setup
sudo: cvmfs_config: command not found

which the action just passes through, while it should fail.

and I think we should re-tag this into v2


bug: unable to listFocusUp programmatically

Check the following:

[X] I have checked the behavior after setting "vscode-neovim.neovimClean" in VSCode settings.
[X] I have read the vscode-neovim docs.
[X] I have searched existing issues.

Neovim version (nvim -v)
NVIM v0.9.5 Build type: Release LuaJIT 2.1.1710088188     system vimrc file: "$VIM/sysinit.vim"   fall-back for $VIM: "/opt/homebrew/Cellar/neovim/0.9.5/share/nvim"  Run :checkhealth for more info
Operating system/version
Sonoma 14.4.1 (23E224)
Describe the bug
I am not able to trigger the action "list.focusPageUp" (or down) from my init.lua
Steps To Reproduce
in my init.lua I have created a mapping for + in normal mode which does vscode.call("list.focusPageUp") but this does result in nothing happening in vscode.
Expected Behavior
I would expect it to move as if I typed C-u. I debugged with some print statemens but I could not find any issues with my code. I also tried with action() (async).
How can i trigger that action ?
Note: I CAN trigger a map "+" to "" in neovim, but that is not the same action as the one performd by vscode. My intention is to trigger the same action the vscode-neovim plugin does

What is list.focusPageUp supposed to do in the context of an editor? It's only for vscode lists like navigation menu or quickfix.


Xamarin PCL support

Great work here.  I am working with Xamarin Forms which allows us to develop for iOS, Android, and Windows Phone.
It seems that Windows Phone doesn't support System.Net.Sockets.
Do you have any plans to find a way around that?
Thanks, Mitch

I hadn't really considered it, but a quick search found this project for supporting sockets from a PCL.  I'd want to explore it a bit more before taking a dependency though.
Out of curiosity - what is the use case that NTP is needed on a phone app?    Or something else?

I spent a bit of time on this tonight.  The sockets-for-pcl library will work nicely, though I need to rework some of the internals fo proper async implementation.  I should be able to have a new version out in the next couple of days.

@mj1856 maybe this will help.  https://github.com/MitchMilam/NTPClient

Sorry for the delay, this is still on my radar but I've been tied up with a few other things.

Since the world has changed a bit since this was opened, I'm going to close this issue.
The current 2.0.0 version targets .NET Standard 1.3, which is compatible with Xamarin.iOS 10.0 and Xamarin.Android 7.0, according to the chart here.
Thanks.


Non convex bank card scanning, there is no good way to do it

HI, card.io .
Non convex bank card scanning, there is no good way to do it

By "convex" do you mean "embossed" (i.e., three-dimensional)?
If so, this is a known issue. See, for example, https://github.com/card-io/card.io-iOS-SDK/issues/20
We do not anticipate any improvements here in the near future, unfortunately.


Add user travel functionality and improved galaxy map

Description

Improves the galaxy map according to #82
Implements a ordable point type with associated weight that can be sorted etc.
Implements a basic eventbox for writing messages to user by listening to events
Implement A* and utility functions for basic travel functionality.

Alternate Designs
N/A
Benifits
N/A
Possible Drawbacks
N/A
Applicable Issues
#83 #68

Codecov Report

Merging #84 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #84   +/-   ##
=======================================
  Coverage   97.27%   97.27%
=======================================
  Files           3        3
  Lines         147      147
=======================================
  Hits          143      143
  Misses          4        4

Impacted Files
Coverage Δ

src/main.rs
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


RateLimit.GetCredentialsRateLimits null reference exception

I'm using RateLimit.GetCredentialsRateLimits(credentials, true) to get rate limits for the given credentials.
Sporadically I get null in return. After having a look at the library code I can see multiple possibilities when this method can return null.
    private ICredentialsRateLimits RefreshCredentialsRateLimits(ITwitterCredentials credentials)
    {
        var tokenRateLimits = GetTokenRateLimitsFromTwitter(credentials);
        _rateLimitCache.RefreshEntry(credentials, tokenRateLimits);
        return _rateLimitCache.GetCredentialsRateLimits(credentials);
    }

This method is not thread safe and between setting rate limit cache and returning it back to caller other thread might clear it. Plus GetTokenRateLimitsFromTwitter method can return null.
What would be best strategy to obtain credential's rate limits?

I will investigate and get back to you.

So I just verified and this cannot be a thread issue because the RateLimitManager has a per thread injection strategy. More information to come...

So I reviewed the code, made some changes but I could not find anything wrong about it. The only thing that I can think of is that you tried to get the RateLimits too often and it ended up returning null.
Would you please share your code?
Also please note that <IP_ADDRESS> has improved the accuracy of the cached information which are now updated every time a RateLimited with the value returned in the headers.

Please provide some feedback before the end of the day as I would like to release <IP_ADDRESS> tonight.
Otherwise I will have to postpone this to release 1.0.

I have nothing special in terms of querying rate limits apart from I'm doing it in parallel and rate limits for the same credentials might be requested concurrently. Thanks for looking into this.

Ok. The code changes in <IP_ADDRESS> might help you but I cannot guarantee that without having proper testing over your code.
I have on my side worked out with multithreaded application accessing the RateLimit for the same credentials without any problem.
I am therefore closing this issue for now.
Please let me know if you still encounter the same problem.

Okay. I will update and keep an eye on this issue. Thanks again.


Manual cooldown inputs via command aren't formatted properly in Hydatos

Description
When using an echo command or just simply inputting cooldown info in chat from the eureka tracker, NMs are properly set on cooldown but the formatting doesn't change. It's supposed to be formatted in a smaller font and color changed to red (or that's what happens in other zones). I do use a custom css file but it edits unrelated parts of the overlay:
#label-container {
	font-size: .75rem;
}

.text.icon {
	font-size: 1rem;
}

Additional information

Sorry, thanks for the bug report.  I'll try to get this fixed this weekend.  I probably broke it when changing the CSS formatting for duels.


view ActionBar

its possible to show ActionBar of view inside tab?

check the demo-angular project to have a better idea


Remove version: REALiX.HWiNFO version 7.70

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/hwinfo/files/Windows_Installer/hwi_770.exe/download
OK
(Automated message - build 923)

URL: https://sourceforge.net/projects/hwinfo/files/Windows_Installer/hwi_770.exe/download
Status Code: 200
(Automated message - build 903)

Close with reason: Duplicate;


Freeze_support() error

It is reported by @oksanaguba that when using multiprocessing with v.2.1.0. There were multiple freeze_support() errors. I'm able to reproduce this issue with standalone e3sm_diags env and e3sm_unified.
    self._launch(process_obj)
    self._launch(process_obj)
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 42, in _launch
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 42, in _launch
    self._launch(process_obj)
    self._launch(process_obj)
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 42, in _launch
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 42, in _launch
    prep_data = spawn.get_preparation_data(process_obj._name)
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 143, in get_preparation_data
    prep_data = spawn.get_preparation_data(process_obj._name)
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 143, in get_preparation_data
    prep_data = spawn.get_preparation_data(process_obj._name)
    prep_data = spawn.get_preparation_data(process_obj._name)
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 143, in get_preparation_data
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 143, in get_preparation_data
    _check_not_importing_main()
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 136, in _check_not_importing_main
    _check_not_importing_main()
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 136, in _check_not_importing_main
    _check_not_importing_main()
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 136, in _check_not_importing_main
    _check_not_importing_main()
  File "/lcrc/soft/climate/e3sm-unified/base/envs/e3sm_unified_1.3.1/lib/python3.7/multiprocessing/spawn.py", line 136, in _check_not_importing_main
    is not going to be frozen to produce an executable.''')

RuntimeError:
        An attempt has been made to start a new process before the
        current process has finished its bootstrapping phase.

        This probably means that you are not using fork to start your
        child processes and you have forgotten to use the proper idiom
        in the main module:

            if __name__ == '__main__':
                freeze_support()
                ...

        The "freeze_support()" line can be omitted if the program
        is not going to be frozen to produce an executable.

issue duplicated


Error with tofile-function

There is a problem caused by the tofile function. When trying to write out a vtk file, Abaqus crashes without any error notification. It seems to be somehow linked to the Abaqus and numpy versions used. Error occurs on a system with numpy 1.6.2 and Abaqus 2018.
A workaround has been implemented to check for the numpy version. If numpy is 1.6.2 a simple write command is used instead of the tofile function.

If it crashes, this sounds like an abaqus issue. No program should ever just "crash".
Besides that, not much can be done without an MWE and a hint on what's wrong.

Closing for lack of feedback. Feel free to reopen at any time.


Ask for confirmation on checkin

Prompts user to confirm when checking in out of scanning time

What's the motivation for this?

They changed the API to allow check-in outside of the scanning period, so they wanted the apps to allow that somehow.


Theme

its hard to identify active tab on vs code default theme
Please redesign active tab style (put underline of some bright color or something like that), so that we can easily identify active tab without diverting attention.
Thankyou : )

Thanks for the feedback, you can try other themes that might better suite your preferences or you can add this to your user settings to set a border on the active tab:
"workbench.colorCustomizations": {
    "tab.activeBorder": "#007acc",
  }

Theme name: Dark+
On Thu, 12 Aug 2021, 11:44 pm Renan Martins, @.***>
wrote:

What is the name of the default theme you are referring to? I tried many
different themes and they all have a contrasting color for the active tab.
Have you tried using the "High Contrast" theme?
[image: Screen Shot 2021-08-12 at 2 41 11 PM]
https://user-images.githubusercontent.com/58703271/129251105-0a61ae65-9cd1-42ed-8830-8b755f167da3.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/130670#issuecomment-897881994,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQIIZ4W44CAQDKGBBL4Y6SDT4QJA7ANCNFSM5CA6JHBQ
.

Thanks for the feedback, you can try other themes that might better suite your preferences or you can add this to your user settings to set a border on the active tab:
"workbench.colorCustomizations": {
    "tab.activeBorder": "#007acc",
  }

Thankyou so much it worked for me 👍


Changes for piman.py and make_zipapp.sh.

Documentation changes for piman.py: Fix the commands that are used to call the server and explain that multiple pis can be restarted in one command.
Documentation changes for make_zipapp.sh: Add the command and warn that sudo should not be used when calling make_zipapp.sh.
Code changes for make_zipapp.sh: Add code to warn the user that sudo has been used and exits the process.

Changes look good


run on edit

A --run-on-edit option should enable continuous graph evaluation as it is edited. Also a DFLOW_RUN_ON_EDIT env var should be added and documented.

This issue will import a run on edit implementation https://github.com/fibo/dflow/issues/60

added autorun input checkbox https://github.com/fibo/dflow/commit/c45fce353323bf955e1b6eeef222b2afe8611d4f


RTL Admin Panel Template

Hi , would you please make RTL version of admin dashboard panel ?

Working on it #789

I am not much familiar with RTL designs, could use some help figuring out where to place what.
Could you help me on Slack?

Hi , first tnx for your attention.
1 - sidebar place on right edge od screen
2 - text alignment and directions right to left
3 - tables columns right to left
i think laravel 5.3 boilerplate rtl version is great , but if you guys do
same job that would be aweesomeee .
Again thank you for everything .
On Tue, Feb 21, 2017, 13:05 Mark Topper<EMAIL_ADDRESS>wrote:

I am not much familiar with RTL designs, could use some help figuring out
where to place what.
Could you help me on Slack?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/the-control-group/voyager/issues/513#issuecomment-281291464,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANtQDjxuj0aFIqiT8tzuCm-9_dQHqLPuks5req_KgaJpZM4LdySY
.

What about the login page, would the sidebar needs to be moved to the left side there?

I have got some progress now, please see the pull request at #789 and come with feedback. Thanks 🎉

@marktopper i think login page is ok , coz the sidebar is already on right 👍

I have already moved it around, does that seems like a problem @sinaboromand?

Why RTL feature not merged with Voyager 0.11.5 Master @marktopper ?

It's not ready for a release yet.

i'm sorry , but when it will be ready for a release @marktopper ? because i want to add this feature in a project .

Soon I hope, however if you want to use it now you can update the composer.json file to use dev-feature/rtl-support and then run composer update.
If you do, please let me know any issues you found so I can fix it, you are also more than welcome to do some pull requests with fixes and optimizations for it.

@marktopper : Thank you mark ^^, and if i found any issues i will let u know ;)

Closing as duplicate of #789.  Please use #789 for further discussion

Hi @marktopper . I appreciate your hard work on the rtl version of voyager and I just want to point out that there are a package called rtlcss. The creator but a lot of time on it and it can handle all the hard work of converting the css to RTL. I tired it out with bootstrap 4 and it worked perfectly. You can find every thing in rtlcss.com. Thanks.

This issue has been closed for a year now....

Yea, I know 😀

It is perfectly converted to RTL

@marktopper  thank you sooooooo much

@marktopper But I have some problems
the most important is that in RTL mode, the user drop down menu does not work

Also All uploaded files are like this photo and the path is not set correctly

@v01061374, please open your own issue. This has been closed for over a year now


WIP: fixing slow cgi mode

This is currently VERY draft-state.
Goals:

Make CGI mode multi-threaded (WIP)
Reuse the PHP context (similar to FPM/worker mode) so it doesn't have to be recreated from scratch each request. (not started)
Allow specifying a maximum number of requests before restarting. (not started)

This PR intends to start a number of threads that handle non-worker requests. Like FPM/worker mode, it reuses the PHP context from the previous request to cut down on latency.
fixes #836

Good start! Maybe could we get rid of the thread pooling library and just start a fixed number of threads by ourself (each thread will consume the same channel, as you started to implement).

FYI: this hasn't been forgotten. My wife has had surgery, and I've been doing daddy + day job. She should be recovered in the next two weeks or so.

Good luck to your wife and you! There is no urgency. I can take a look if you want.


Definition for rule 'react/static-property-placement' was not found

When I try to use static-property-placement rule, I get the following error

Definition for rule 'react/static-property-placement' was not found

Use like this
...
"react/static-property-placement": 2
...

Version in package.json
"eslint": "^5.16.0",
"eslint-plugin-react": "^7.12.4",

In node_modules no file with the description of this rule, but in github this file exist - static-property-placement.js
Is this some kind of mistake or has this rule not been released yet?

It's not been released yet. Check the docs on the latest tag, not on master.

Can you notice about that in readme.md and rules/static-property-placement.md ?

@nzvtrk since that’s how every GitHub project works (master can always be potentially unreleased), and since doing so would be a burden since we’d have to remove that notice for every tagged release, no.

This rule seems no to work for me in version 7.13.0.
I have the following config for it:
"react/static-property-placement": [
      "error",
      "static public field"
    ]

but no error is shown

@lostm1nd that code has defaultProps as a static public field - what error would you expect?

@ljharb my bad. I was misled by the name to think it will enforce placement inside class bodies but now I saw it states otherwise.
Any suggestion how to get the static field at the top of the class?

@lostm1nd that's a different rule - https://github.com/yannickcr/eslint-plugin-react/blob/master/docs/rules/sort-comp.md


IMAP Idle Not Working

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App
Thunderbird for Android
App version
8.0
Where did you get the app from?
Google Play
Android version
13
Device model
Sony Xperia 1iii
Steps to reproduce

Even with IMAP IDLE refresh set at every 2 minutes, it does not synchronize and pull in new email for over 5 minutes. With K-9 Email it was instant.
Expected behavior
With K-9 with IMAP IDLE refresh set at every over 10 minutes, synchronization and pulling down new emails it was instant.
Actual behavior
Now it takes about 5 minutes for manual refresh (by swiping down on Inbox) before synchronization happens.
Logs
No response

What type of permissions do you have granted to Thunderbird? Do you have the alarms and notification permission granted for example? Are you also experiencing this with K-9 Mail 8.0?

Thunderbird has permissions for everything.

I just want to pitch in here with my experiences. Fort new, IMAP IDLE has always dropped working sooner or later. I usually have to open the app and force a refresh to get it going again. It doesn't detect that the IMAP connection has been closed.

Did you enable push for the inbox under Manage folders → Inbox → Enable Push?
If you did and it still doesn't work, please record a debug log and attach it here. See https://github.com/thunderbird/thunderbird-android/wiki/LoggingErrors

I would like to offer a debug log, but I cannot trigger this problem at will. I havent looked closer to when the connection is lost; I just force a refresh. I have two email programs configured for this exact problem. The other app uses a poll scheme and is reliable, but usually somewhat delayed. K-9 can be nearly instant, or doesn't show emails arriving at all.
And yes, push is enabled; always has been.

Now it takes about 5 minutes for manual refresh (by swiping down on Inbox) before synchronization happens.

Check my issue #8079 Manual checking for new messages sometimes takes up to 3 minutes Isn't it the same?
I can reproduce it almost always when travel and walk the city, using public transport and subway. I have already captured many logs.

Yes, that is what I'm getting.  It will sync and pull down new messages every 5 minutes, even though I've set IDLE connection refresh to 2 minutes. Push/IMAP Idle worked perfectly when it was K-9 Mail.

How did you set the Account / Fetching mail / Folder poll frequency? To Never or some time period?

I set it to every 15 minutes, which is the shortest interval. Would a different setting make push mail work like in K-9?

I think it somehow conflicts with the push notifications. I had the value 3 hours, now I am testing Never.

@cketti Is there any way to check whether the push notification receiver is running? I can make it stop working almost every day on two different devices.
Steps:

Connect to home Wi-Fi (for hours, push notifications works)
Connect to mobile Internet (push notifications works)
After 10 minutes connect to Wi-Fi (push notifications works)
After 10 minutes connect to mobile Internet (push notifications stops working)

Since that the push notifications no longer work. You can manually check for the new messages only by the swipe down on Inbox, but when a new email arrives, nothing happens. I enabled the detailed logging, there was no record at the time when the new email was sent and supposed to be received.
The only way to restore the notifications is to go to Settings / Applications / Thunderbird, push the Force Stop button and start the Thunderbird again.
I can reproduce it many times with the 8.1 version.

The issue persists in 8.2 too.

Thank you very much @kewisch & @cketti
IManage folders → Inbox → Enable Push has made it work. This has been an anoyance since migrated from K-9. Thank you very much!

So, should I open a new ticket regarding IMAP IDLE not working?
I have IDLE refresh at 24 minutes (default) and push is enabled.
But I have the frequency for fetching emails set to never. I figured IMAP IDLE supercedes this, right?

No thank you. IMAP Idle now works as it did in K-9. I did not know that I also had go to [Settings] > [Manage folders] > [Inbox] > [Enable Push]

Ok, but IMAP IDLE still doesn't work for me. Must be a different reason.

Did you try the suggestion from  @kewisch & @cketti ?
[Settings]  →  [Manage folders] → [Inbox] → [Enable Push] like on the screenshot ?

I never had it disabled.

These are screenshots of my other related settings if that helps...

As mentioned, Folder Poll Frequency (I see it in Danish: frekvens for hentning af mails) is set to never, and I have now increased the IDLE connection renewal to 60 minutes from 24 minutes (just tried it at 6 minutes and that wasn't a success).
I don't think this has ever worked for be, and I am now using a different email provider (same problem).
I already had the Alarms $ Reminders enabled. But thanks for the reminder.
Due to this being unreliable, I have configured Samsungs own email program that polls, just to be sure to get a (delayed) notification that I have unread emails.


Request: Editing Accessories and Persistent data

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

I was trying to remove a device from a Plugin. It was using the accessories/cachedAccessories to store them after they where discovered. It sucked removing them manually.
Describe the solution you'd like:

A simple way to edit cachedAccessories and persistent data would be great.
Describe alternatives you've considered:

Editing it manually but it was annoying
Additional context:

This feature already exists. To use it you must be running your setup using hb-service.
Upper right drop down menu -> Homebridge Settings -> Remove single cached accessory.
If you don't see the option, you aren't using hb-service.

This feature already exists. To use it you must be running your setup using hb-service.
Upper right drop down menu -> Homebridge Settings -> Remove single cached accessory.
If you don't see the option, you aren't using hb-service.


can not use in android 安卓不能用啊

Sorry I don't provide Android support as of now. You can try to find other packages for that.


But if I, a human being who is within the organization, pushes the same changes, CI should run just fine. And if that passes we'll merge.

Is it not possible to give the dependabot the ability to run CI using WIF like we were before?


Too many goroutines in gopark state

Description
We have implemented a single Kafka producer using Sarama, which multiple goroutines utilize to produce messages to Kafka. However, we have observed an excessive number of goroutines in the gopark state. What could be the possible cause for this?
Below is the image of goroutine dump:

Versions

Sarama
Kafka
Go

v1.40.1
2.8.1
1.18

So from the displayed goroutine image it looks fairly normal. Your producer is writing to one or more topics totalling 2093 partitions. Sarama dedicates a parititionProducer to each partition. Each of these has a dispatch() loop which is waiting on an input channel to receive messages that you'd like it to send. When that input channel is empty they will block on the chanrecv waiting for input and the Go scheduler will park the goroutine until it needs to be woken when the channel receives input.


Are there any plans on the Maven support?

Hello!
Thanks a lot for the repository. This project is using the code from here. Since there is no Maven packaging (pom.xml), they have to copy/paste it.
Are you planning to add the pom.xml to this repo?

Hi,
Nice to hear that someone is using my code. I didn't have any plan to add maven support but I will consider it. I think in your case even a git submodule would work.

Yeah, but it's not so fancy and flexible as the Maven artifact :)   Thanks for your response, please let me know if you need some assistance with the Maven support, I could create a PR with the pom.xml

It would definitely help if you could create a PR with a pom.xml.


Fix failing BQ file loads test for Py37

Fixes #23264

Run Python 3.7 PostCommit

Run Python 3.8 PostCommit

Run Python PreCommit

Run Python 3.7 PostCommit

Run Python 3.8 PostCommit

Run Python 3.7 PostCommit

Run Python 3.7 PostCommit

This looks fine, but I don't really understand how this fixes the issue. Can you explain that in the comments of the test?
It's not really clear why we would instantiate the pipeline once with the class instead of for each test.


0.8.1 China fixes

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
These changes fix several issues:
Spatial CLI for China was changes and the "domain" flag was deprecated. The flag was removed from one place in the code and from two setup scripts.
An issue in cloud deployment where the upload snapshot http request was sent with a missing header in the case of China.
The DeploymentLauncher was using the west refresh token when deploying to China.
We previously relied on Spot to run the auth process for us when launching locally. The Spot process ends before finishing the auth process in the case of China. This meant we would continue with out deployment without being authed and run into errors. We know explicitly run the auth process on cloud and local deployment to ensure this does not happen. This does not seem to incur any significant time cost as attempting to auth whilst already authed will quickly finish.
Primary reviewers
@m-samiec @mironec

Are these the same changes that went into the 0.6.5 rc? If so, I would prefer to merge 0.6.5 rc into 0.8.1 rc

This PR fixes the same issues as the 0.6.5 PR however some of the code is quite different. The local and cloud deployment code changes have some bigger differences where as the deploymentlauncher is the same between them.

If this conflicts when we merge release into preview, accept preview's version


Compile with clang, gcc5 (MLDB-1405)

Fixes the warnings that come from the other compilers and make MLDB compile with any of them.  It found some consequential bugs including those in MLDB-1409, MLDB-1410 and that in binding_contexts.cc.

+1


Product Question

Hey Imoji team,
Imoji is really awesome in general! Thank you for the service and your work.
I was wondering if you had any intention to provide a borderless (or rendering option) version again, now that it is been removed since changing the SDK to work only with the REST API.
There are times when having the classic white border significantly enhances things... such as inside a chat, or using it as an emoji in the classic sense.
However, when applying imoji's to images and videos, the border is a very poor effect in comparison.
Please do some tests yourselves, don't just take my word for it, I challenge you to get 10 Images or Videos, and place a white bordered imoji vs a borderless version on them.
Here is an example off your website: http://imoji.io/assets/images/sdk/web_sdk_create.png, of using the white bordered version to replace peoples heads.
The effect of having a border make it looks like a singular object, playful & fun, and therefore more emoji like (which as I said, is great inside something like chat).
I would say that in comparison, when someone is using the imoji to do something like replace a head, put a hat on, or put an object in their mouth, etc (which looks like a very large use case, based on the ratio of peoples heads to random objects, when looking through your most popular results, there are LOTS of heads), the effect of having the head look more realistic, and without the border, makes for an infinitely better image/video
You can even draw a parallel example when looking at Snapchat's new live filters, they have tried to blend the filer/overlay into the image as much as possible, rather than making it severely stick out on top.
A bunch of our users have noticed this and we now have to remain on your old SDK, which I am sure is going to end up unsupported soon.
Thanks

Hi Steve, thanks for the feedback and big apologies for the late reply here.
For what it's worth, we moved the rendering of the images to the server for speedier rendering. In the process we lost the ability for SDK users to customize the border and shadow properties. We currently support bordered and unbordered rendering in IMImojiRenderingOptions. We will look to add more flexibility via a server driven Rest API soon.
Thanks!


buy a domain name and rename the package and the groupid

org.simpleflatmapper
will not rename the package, there are currently no org.sfm package that I could find and it's currently the San Francisco media archive.

need to raise sonatype ticket

https://issues.sonatype.org/browse/OSSRH-11232


[Modal] refs not available in modal

I have a TextInput with a ref set, and it is not available in this.refs:
The code is:
    <Modal
      animated={true}
      transparent={true}
      visible={this.state.workHerePromptVisible}>
          //...snip...
          <TextInput
            style={someStyle}
            ref="username"
          />
          //...snip...
    </Modal>

I am trying to access this.refs.username in a handler

Which React Native version are you using? I have modals with refs and I can access them.
<Modal animated={true}
  transparent={false}
  visible={this.props.visible}>
  <View style={styles.modal}>
    <ElementForm ref='elementForm' />
    <Button onPress={() => {
      // Accessible ref with nested refs
      const elementForm = this.refs.elementForm.refs.form.getValue()
    }}>
      Save
    </Button>
  </View>
</Modal>

No response from OP, one to close? @brentvatne

Works for me as well.

Looks like others couldn't repro this one, closing.

Thanks for checking @grabbou!

I ran across this as well in 0.11.0. Accessing refs directly from within the modal works, but you can't access them from the outside.
Using the above example as a starting point:
render: function(){

console.log(this.refs)
// outputs: Object {theModal: Modal}

console.log(this.refs.theModal.refs)
// outputs: Object {}

return (
<Modal
  ref='theModal'
  animated={true}
  transparent={false}
  visible={this.props.visible}>

  <View
    ref='theView'
    style={styles.modal}>

    <ElementForm ref='elementForm' />
    <Button onPress={() => {
      const elementForm = this.refs.elementForm.refs.form.getValue()
     // this works
    }}>
      Save
    </Button>
  </View>
</Modal>
)

}

I'm having this issue (using 0.40.0), any idea on what's causing this and how to fix? I also notice re-renders of the modal content ever time the modal is set to visible. Code:
<View style={style.wrapper}>
        <Modal
          visible={this.state.singleTaskVisible}
          style={style.singleTaskModal}
          animationType="slide"
        >
          <Swiper
            ref={(component) => {
              this.swiper = component;
            }}
            index={tasks.indexOf(activeTaskId) > -1 ? tasks.indexOf(activeTaskId) : 0}
          >
// ...etc

The ref is only available after the modal is set to visible.

same here
 <Modal
          visible={this.state.singleTaskVisible}
          style={style.singleTaskModal}
          animationType="slide"
        >
          <TextInput
            ref={(component) => {
              this.swiper = component;
            }}
            index={tasks.indexOf(activeTaskId) > -1 ? tasks.indexOf(activeTaskId) : 0}
          /></Model>
<View><TouchableOpacity onPress></View>
..etc

I was not able to get the focus of the textinput from onPress event.

@mkonicek can you check this out.

Sorry i solved this by using onshow prop inside react-native and added focus to the textinput. since the ref for textinput is not available outside the modal.
this solution is only for my use case..


Make build system

This PR does a couple of things:

Further decouples the two repos
Wraps as much of the "scripty" build steps as possible in Make which gives us nice control over the execution DAG
Adds clean and sterile build steps to reset back to known clean state when things go haywire
Cleans up quite a bit of bash stuff to make failures obvious and aid debugging
Adds some healthchecks and stricter deps in the compose file to make dockerDev.sh more stable

Testing procedure

Checkout this branch
Run make clean (you can see what it does in the root Makefile first)
Run ./dockerDev.sh  (this automatically runs make and sets up all projects in the repo)
Run make sterile (note this will nuke local-data too, and will make clean as well)
Run ./dockerDev.sh again

Feedback
Hit me with all types of feedback. Do you vibe with this? Anything that is non-obvious and need more docs? Note that I haven't yet gotten to the README.md.

NOTE: this depends on a companion PR in nodes-web not to break the frontend builds

some issues around perms, investigating

@hubsmoke odd that your entire local-data directory is owned by root :thinking: Did you run dockerDev.sh with sudo?

@hubsmoke odd that your entire local-data directory is owned by root :thinking: Did you run dockerDev.sh with sudo?

nope


Fix issues with kerberos and non NTLM domains

This is an issue from discord.
The problem is if we use flags like --use-kcache we need to use kerberos. Though, some checks are still done with if self.args.kerberos and not if self.kerberos which would be correctly set to true when using --use-kcache. Therefore, in non-NTLM environments the authentication does not work, because for example:
if self.args.kerberos:
    self.host = self.hostname

is not triggered before, when actually self.kerberos by the self.args.use_kcache flag.
This PR should fix edge case issues, like these:
NTLM is not supported
nxc smb winterfell.north.sevenkingdoms.local --use-kcache -> fails (because of the aforementioned issue)
nxc smb winterfell.north.sevenkingdoms.local --use-kcache -k -> succeeds

An issue with delegate also ?

An issue with delegate also ?

We didn't test it, but it likely would be. If you wouldn't have NTLM and delegation. Just added it to be sure

Ah hmm maybe we should just change self.args.kerberos as well... For example laps login sets self.args.kerberos which would now break. Or we change the line in laps to self.kerberos

Should be fixed now

I don't think the issue is resolved yet

This issue has been resolved.
Please make sure to use 66da98c commit.

All working as intended:


Update documentation

@brittainhard This should be good to go. Dev guide and API guide have been unlinked for the time being, so I've added #393 to track the most pertinent outstanding documentation.


Recognizer Options of separate Hammer instances are mixed up (not separated)

Example:
http://codepen.io/anon/pen/bdYQaa
Expected behavior:
The first div is supposed to recognize Hammer.DIRECTION_ALL, the second one should recognize Hammer.DIRECTION_HORIZONTAL.
Actual behavior:
Both instances use Hammer.DIRECTION_HORIZONTAL.
When you inspect both instances in your JS debugger (Firebug, chrome dev tools, ...), you'll see, that the options of the panrecognizers of both instances are set to direction=6 (HORIZONTAL).
If you remove all the JS-code from the second instance, then the first instance is correctly set to Hammer.DIRECTION_ALL. Somehow setting this option on the second instance does also set this option on the first instance.
EDIT
Updated example. Now it outputs actual panrecognizer direction options of both instances
EDIT2
Update example. I've found, that the options-Object of two recognizer-instances are shared (identical objects). See output DIV in example. The options-objects are identical though the recognizer objects are not.

I've created a bugfix for this issue. It ensures that the options are copied into a new object in the recognizer constructor. This fixes the actual issue.
The deeper problem is in the Manager() constructor. This line:
this.options = merge(options, Hammer.defaults);
copies the default-options into the newly created Hammer instance. But merge() only creates shallow copy of the defaults-object. This way, the default-options for the default-recognizers within these Hammer.defaults are not copied but linked to and thus shared between all recognizers.
So additionally replacing merge() in the constructor with a function that performs a deep copy would be recommended to avoid similar problems else where.

The deeper problem is in the Manager() constructor. This line:
this.options = merge(options, Hammer.defaults);
copies the default-options into the newly created Hammer instance. But merge() only creates shallow copy of the defaults-object. This way, the default-options for the default-recognizers within these Hammer.defaults are not copied but linked to and thus shared between all recognizers.

@arschmitz sounds like we may leak options modifications between manager instances (not just recognizer instances), will investigate and PR fix if so.

+1. Would like this to be merged. I have multiple instances of Hammer and need different directions listened. My temporary fix is to listen to all swipe directions but it's not ideal (especially for old devices).
Thanks in advance


Start the move from typescript to web assembly

While no good wasm scrypt implementation exists (the only one I found didn't work for me, which is probably caused by it using an older version of rust and wasm-pack. It's also just possible that the fork didn't actually fix the problem that made original incompatible with web assembly), there might be web assembly versions of some of the functions used by scrypt like pbkdf2 or perhaps salsa20 that might be modified to only use core with 8 rounds. So at least partial transition might be possible now, even if full one will have to wait until I or someone else makes something that works.

Thanks to the work of skdltmxn there is now a nice build script for wasm-pack that works with deno that was used for std/hash wasm implementation.
The performance was improved by A LOT. A quick benchmark with standard parameters changed from ~650ms to ~180ms. I'll try to run some more benchmarks later, but the performance gains are really big.
And it still doesn't need any privileges to run!

Also, this works! https://twitter.com/_denorg/status/1274720537038766084


Renaming user account locks owner out of project

Describe the bug
If the owner of a Ghidra project renames their user account, they will be locked out of the project.
To Reproduce
Steps to reproduce the behavior:

Rename user account.
try to open project.
Error stating that the project does not belong to you.

Expected behavior
Ghidra should have some other way of detecting whether or not a user the owner of a project. With the current method, a user who is not the owner is able to gain write access to a project simply by changing their account name.
Environment (please complete the following information):

OS: macOS 10.15.6
Java Version: 14.0.1
Ghidra Version: 9.1.2

In this situation you must edit the project property file (project.prp) with a text editor and change your user name associated with the OWNER attribute.  This is an XML file and is located at the root of the Ghidra project directory (*.rep)


Implement load testing in CI

This is my intent (choose one)

[ ] I want to report a bug
[ ] I want to request a feature or change
[x] I want to provide or change a feature

The problem
At the moment we can only manually see performance regressions for our production servers.
Proposed solution
We should implement load/stress testing as part of our CI workflow and make sure that the values stay inside a certain threshold / don't change too much between changes.
We can use https://github.com/mcollina/autocannon to run the tests and for example store the values inside the travis build cache, which will be persisted between runs and would allow us to monitor and report changes between builds.

Not a priority right now, we will revisit this at a later time.


support configuring the config file path from vscode settings.

Hey team, this is currently a show-stopper for us. Any guidance on where to start hacking to get this working?

@IWANABETHATGUY is taking a break from a huge neck pain.

To start hacking:

build instructions: https://oxc-project.github.io/docs/contribute/vscode.html
js glue: https://github.com/oxc-project/oxc/blob/main/editors/vscode/client/extension.ts
Rust part: make this configurable https://github.com/oxc-project/oxc/blob/2794064eef4c6b57cfc9f58d18ed7e2730a994a4/crates/oxc_language_server/src/main.rs#L336
Vscode part: figure out how to add a UI for accepting a path configuration

for how to add a config in vscode extension: you could refer to https://code.visualstudio.com/api/references/contribution-points#contributes.configuration,

Thanks both of you - really good pointers. Will try my luck this weekend.

Released in https://github.com/oxc-project/oxc/releases/tag/oxlint_v0.2.6


How to load a open-sourced model without 'value head'

Hi, I am trying to load a reward model (the final layer is reward_head) without 'value head', how to achieve that?
In addition, I am not clear with https://github.com/OpenRLHF/OpenRLHF/blob/main/openrlhf/models/model.py#L187 , which will create a random initialized value head. Does it create a random initialized value head all the time?

Please refer to the args:
https://github.com/OpenRLHF/OpenRLHF/blob/e1205adaf163e28ed58400ed84618bc01c3c4c37/openrlhf/models/model.py#L34
and
https://github.com/OpenRLHF/OpenRLHF/blob/e1205adaf163e28ed58400ed84618bc01c3c4c37/openrlhf/models/model.py#L35

Thanks for your reply.
Does it mean that I can set value_head_prefix="reward_head", and setattr(self, value_head_prefix, nn.Linear(config.hidden_size, 1, bias=False))  will first search the parameters of reward_head from the model. If the parameters doesn't exist, it will initialize it with nn.Linear(config.hidden_size, 1, bias=False).
The main misunderstanding lies in that I don't know the logic of setattr.


Allow undefined errors in executeHandler

The catch of the try-catch in the JobExecutor's executeHandler espects an error with a message.
If a handler rejects with undefined (return Promis.reject()) this leads to a TypeError which could prevent further jobs to be executed.
Instead of
      result = { status: ExecutionStatus.failed, handlerResult: (e as Error).message };
one could write
      result = { status: ExecutionStatus.failed, handlerResult: (e as Error)?.message };
or something similar. I can open a PR to change this.

Good catch (pun very much intended 😄 )!
Consult the CONTRIBUTING guidelines and open a PR!

Fixed in 2.1.0


cssanimation leRotateXZoomIn sequence  not work

using any class my all html code display on screen

What did you mean? :)


Hide sensitive properties from toString()

It is in some cases useful to mark some property in JSON schema definition as a sensitive (or confidential) and thus making it hidden from generated toString() method.
Generator already contains a feature "toStringExcludes" which is global property specifying everything to be excluded from toString generation by name.
It would be advantage to have a definition of what to exclude inside the JSON schema. Idea would be to have it similarly to already existing feature excludedFromEqualsAndHashCode.

Attempt to bring the feature: https://github.com/joelittlejohn/jsonschema2pojo/pull/1557


Add editing cart functionality, remove navigate to cart for products

Please review the following changes:

Added editing cart quantity functionality
Removed navigate to cart for add to cart button

The editing cart quantity functionality currently needs debugging, an error occurs when changing quantity but upon refreshing the page, the quantity changes and persists in the database.

Decrementing also currently increments the quantity.


qq.s3.XhrUploadHandler._getLocalStorageId is never called

Type of issue [REQUIRED]
bug
Uploader type [REQUIRED]
S3
Bug details [DELETE EVERYTHING IN THIS SECTION IF THIS IS A FEATURE REQUEST]
Fine Uploader version [REQUIRED]
5.10.1 and previous
Browsers where the bug is reproducible [REQUIRED]
Chrome , Firefox and Safari
Operating systems where the bug is reproducible [REQUIRED]
OSX 10.11
Detailed explanation of the problem [REQUIRED]
I wanto to generate an localStorageId that keeps track of the full s3 key instead of the filename only, but i've noticed that _getLocalStorageId is never overridden even if in  qq.s3.XhrUploadHandler there's this code
_getLocalStorageId: function(id) {
                var baseStorageId = super_._getLocalStorageId(id),
                    bucketName = upload.bucket.getName(id);

                return baseStorageId + "-" + bucketName;
            }

Please include code that reproduces whatever issue you are having.

i'll setup a demo as soon as possible

Here it's http://labs.e-tip.net/
it's just the demo code. the only modification i've done is
at line 10697 of s3.fine-uploader.js
_getLocalStorageId: function(id) {
                var baseStorageId = super_._getLocalStorageId(id),
                    bucketName = upload.bucket.getName(id);
                    qq.log(baseStorageId + "-" + bucketName);
                return baseStorageId + "-" + bucketName;
            }

and at line 5526
_getLocalStorageId: function(id) {
            var formatVersion = "5.0",
                name = getName(id),
                size = getSize(id),
                chunkSize = chunking.partSize,
                endpoint = getEndpoint(id);
            qq.log(qq.format("qq{}resume{}-{}-{}-{}-{}", namespace, formatVersion, name, size, chunkSize, endpoint));
            return qq.format("qq{}resume{}-{}-{}-{}-{}", namespace, formatVersion, name, size, chunkSize, endpoint);
        },

As soon i add a file i can see in the console that localStorageId is always returned by function at line 5526 because i can see no -nucketName appended

What specific issue are you seeing? Is something broken or not functioning correctly? For example: "The resume feature is not working if x, y, and z" If so, please provide specific steps needed to reproduce a noticeable issue.

Hi, i've modified my test suite.
i've added a second uploader and modified the uploader code
for first uploader i've added to objectProperties object the following key function
key : function(fileID){
    return "folder1/"+uploader.getUuid(fileID)+"."+qq.getExtension(uploader.getName(fileID));
}

and for second uploader
key : function(fileID){
    return "folder2/"+uploader.getUuid(fileID)+"."+qq.getExtension(uploader.getName(fileID));
}

Steps to reproduce the issue

Start uploading a file larger than 6 MB to uploader1 so chunk upload is enabled
upload endpoint is this http://fineuploader-test.s3.eu-central-1.amazonaws.com/folder1/59950077-0b84-4ca6-9f5a-d242fda00f2e.mov
After upload is started refresh the page so upload is interrupted
Drop the same file in the second uploader and you'll see that the file is uploaded in folder1 even if key function points to folder2.
This happens because the localStorageID is calculated not considering that the key can be different for the same filename-filesize keys ( that is the actual localStorageId calculating method )

Thanks for the explanation. Can you open up a pull request with your fix so we can discuss that further?

I'll do as soon as i understand why qq.XhrUploadHandler._getLocalStorageId is called instead of qq.s3.XhrUploadHandler._getLocalStorageId

I'll try to write a fix and open up a pull request. By the way the main issue is that the add function is called in qq.XhrUploadHandler contest so handler. is kind of qq.XhrUploadHandler instead of qq.s3.XhrUploadHandler. as soon as i fix this issue my solution would be to check if objectProperties.key is a function and if yes check if returns a kind of string that contains /. if yes strip out the filename and use also that part as localStorage key. Does that sound good for you ?

There is a lot if inheritance between the modules, so all may. It be as it seems. I plan to look at this closer soon.

yep, actually the main issue i'm having is due to inheritance between s3 and non s3 version, i came out with too many methods to override

One approach is to simply append the key name onto the generated localStorageId in the S3-specific module. In fact, I wouldn't suggest making any changes outside of s3.xhr.upload.handler.js. The only potentially complex part here is that key determination may be an async operation. Currently, that key is first resolved when the upload begins, but that would have to change here. Also, any change to the local storage ID would potentially invalidate all existing localStorageId records, and that is not acceptable in my opinion. So code would need to be added to distinguish between the current v5.0 localStorageIds and this new version (5.?).

I agree with you that invalidating all actual storageIds is not acceptable.
i've played today with sources to try appending some custom code to the localStorageId that is generated in 3.xhr.upload.handler.js but that function is never called. I have to play some more. Thanks for checking into this for me :)


Updated the plans container css class

Making it scrollable. This way, for a project with a significant number of plans, the view will have a limited space for the plan selection.
Plans tab on update/create project view:

Donate view:

LGTM :+1:


quick pick input goes away when creating new resource group using keyboard

use the mouse to click on the "deploy" button on the extension explorer header

use the keyboard to answer the following questions

an existing subscription

create new function app

functionapp name

create new resource group

result: you dont get the next prompt where you can enter the name of the resource group.

then, eventually, after clikcing around for a bit, then you get prompted. not exactly sure what causes it to happen or if its like a long time and i just keep missing it

happens in stable and in insiders...

I'm unable to repro this on both stable and insiders using the extension from the market.

@fiveisprime  I wonder if it has something to do with timing.  i'm on a slow hotel connection when this happens.  maybe try tethering your phone and see if you can repro?

I was able to repro by using a similar name: aaaa-aaaa-aaaa-aaaa. Something about that format makes it slow to suggest a resource group name.

@chrisdias I sent out a PR with a fix for this. Should be included in the next release.
The problem occurs because you have multiple hyphens in your function app name (it breaks our 'related name' generation for resource groups, which don't support hyphens). As a workaround, use a name without multiple hyphens


Release case conversion as a separate crate

Hey, I'm running into a tricky issue (https://github.com/poem-web/poem/issues/362) with another project where the snake casing behavior is a bit different between serde and the backend it uses, inflector. In our project we're using both Poem and serde, e.g. Poem for the API, serde for storage. I'm exploring ways to fix this, I figure one is to make Poem use the same case conversion behavior as serde (which would be more standard in general). For this though, the case conversion code (https://github.com/serde-rs/serde/blob/master/serde_derive/src/internals/case.rs) would either need to be exposed from within serde_derive, or just pulled out into a separate crate.
I wonder if you think either of these options make sense? There are other ways around this issue, but this seems to be one of the nicer ones, plus I imagine others might want this case conversion library for other reasons too.
Thanks!

That file is published to https://docs.rs/serde_derive_internals/0.26/serde_derive_internals/attr/enum.RenameRule.html.

Okay awesome, thanks a lot, I'll take a look at using that.


Add wallet CTA for legacy staking

A user today pointed out that there's no way to view existing stakes or claim them upon maturity when visiting https://ousd.com/earn until a wallet is connected. We should throw some kind of call-to-action in there for the user to connect the wallet.

done in #1060


[secrets][test] fix test that assumes map order

What does this PR do?
Test assumed map order so would be flaky, let's use a DeepEqual validation to check for real.
Motivation
Flaky test.
Additional Notes
Anything else we should know when reviewing?

The issue was related to the order of values in the slice (the list of location for a handles). DeepEqual don't sort slice before comparing them, each element has to be the same at the same index.
I removed DeepEqual since it hides the difference between what was expected and what was actually return by the code.


sess.run(tf.global_variables_initializer()) is slow after installing 1.0.0.

The issue
sess.run(tf.global_variables_initializer()) intermittently takes a long time since installing 1.0.0. I believe this also happened on my Python 2 0.12 install, but only after installing 1.0.0 for Python 3. This particular run was in Python 2, and the initializer took 32 seconds - see bottom of this post (and the attached cprofile output) for more detail.
Strangely, it seems like the more I run the same file, the faster it runs.
What related GitHub issues or StackOverflow threads have you found by searching the web for your problem?
None.
Environment info
Operating System: Ubuntu 16.04
Installed version of CUDA and cuDNN:
(please attach the output of ls -l /path/to/cuda/lib/libcud*):
ls -l /usr/local/cuda/lib64/libcud*
-rw-r--r-- 1 root root   560184 Oct 27 16:26 /usr/local/cuda/lib64/libcudadevrt.a
lrwxrwxrwx 1 root root       16 Oct 27 16:26 /usr/local/cuda/lib64/libcudart.so -> libcudart.so.8.0
lrwxrwxrwx 1 root root       19 Oct 27 16:26 /usr/local/cuda/lib64/libcudart.so.8.0 -> libcudart.so.8.0.27
-rwxr-xr-x 1 root root   394472 Oct 27 16:26 /usr/local/cuda/lib64/libcudart.so.8.0.27
-rw-r--r-- 1 root root   737516 Oct 27 16:26 /usr/local/cuda/lib64/libcudart_static.a
-rwxr-xr-x 1 root root 60696704 Oct 27 16:31 /usr/local/cuda/lib64/libcudnn.so
-rwxr-xr-x 1 root root 60696704 Oct 27 16:31 /usr/local/cuda/lib64/libcudnn.so.5
-rwxr-xr-x 1 root root 60696704 Oct 27 16:31 /usr/local/cuda/lib64/libcudnn.so.5.1.3
-rw-r--r-- 1 root root 59715990 Oct 27 16:31 /usr/local/cuda/lib64/libcudnn_static.a

If installed from binary pip package, provide:

A link to the pip package you installed: sudo pip install tensorflow-gpu on 2/21/17
The output from python -c "import tensorflow; print(tensorflow.__version__)". 1.0.0

If possible, provide a minimal reproducible example (We usually don't have time to read hundreds of lines of your code)
from __future__ import absolute_import
from __future__ import division
from __future__ import print_function
import time
from collections import deque

import numpy as np
print('Importing tensorflow')
import tensorflow as tf
from tensorflow.examples.tutorials.mnist import input_data

xavier = tf.contrib.layers.xavier_initializer
batch_norm = tf.contrib.layers.batch_norm

tf.set_random_seed(1)

flags = tf.app.flags
FLAGS = flags.FLAGS
flags.DEFINE_string('data_dir', 'data/', 'Directory for storing data')
print('Reading data.')
mnist = input_data.read_data_sets(FLAGS.data_dir, one_hot=True)
initialization_variance = 0.1

print('Creating a session.')
sess = tf.Session()

def fully_connected_layer(inputs, output_size, scope, nonlinearity=True):
  with tf.variable_scope(scope):
    input_size = inputs.get_shape()[1].value

    weights = tf.get_variable('w',
                              [input_size, output_size],
                              initializer=tf.constant_initializer(0.0))

    bias = tf.get_variable('b',
                           [output_size],
                           initializer=tf.constant_initializer(0.0))

    multiplied = tf.matmul(inputs, weights)

    return tf.nn.elu(multiplied + bias) if nonlinearity else multiplied + bias

print('Setting up the model.')
input_images = tf.placeholder(tf.float32, [None, 784])
labels = tf.placeholder(tf.float32, [None, 10]) # labels
learning_rate_placeholder = tf.placeholder(tf.float32, name='learning-rate')

layer_a = fully_connected_layer((input_images), 50, 'a')

for i in range(5):
  layer_a = batch_norm(fully_connected_layer(layer_a, 50, str(i)))

layer_b = fully_connected_layer(layer_a, 10, 'b', nonlinearity=False)

predictions = tf.nn.softmax(layer_b)

cross_entropy = tf.reduce_mean(-tf.reduce_sum(labels * tf.log(predictions), reduction_indices=[1]))
train_step = tf.train.AdamOptimizer(learning_rate_placeholder).minimize(cross_entropy)

print('Initializing variables.')
sess.run(tf.global_variables_initializer())

What other attempted solutions have you tried?
None.
Logs or other output that would be helpful
Some output from python2 -m cProfile -o out.profile 1-mnist-simple.py. This is a call to sess.run(tf.global_variables_initializer()).

cprofile-output.profile.zip

Is it the first call, or can it be the second call? If it's the first call, it could have something to do with CUDA generating PTX, which happens the first time op is executed and is slow (and mega-slow if you are on nfs, set CUDA_CACHE_PATH=/non_nfs_dir). Whether or not PTX has to be generated depends on whether the binary was built with compute capability of your card
Does the problem disappear if you revert to older version? Does it happen with CPU-only TensorFlow?

Is it the first call, or can it be the second call?

There's only one call to sess.run - for initializing the variables. If you're talking about running the script as a whole, so far it has been kinda unpredictable.

If it's the first call, it could have something to do with CUDA generating PTX, which happens the first time op is executed and is slow (and mega-slow if you are on nfs,

I'm running this all on my own machine.

set CUDA_CACHE_PATH=/non_nfs_dir). Whether or not PTX has to be generated depends on whether the binary was built with compute capability of your card

It's a GTX 1060, so that's compute capability 6.1 I guess.

Does the problem disappear if you revert to older version? Does it happen with CPU-only TensorFlow?

I switched to 0.12, CPU version: tensorflow-0.12.0-cp27-cp27mu-manylinux1_x86_64.whl. Waaaaay better! After that, I tried 0.12, GPU version: tensorflow_gpu-0.12.0-cp27-cp27mu-manylinux1_x86_64.whl. This returned to the same problem.

Also around the same time I started noticing the problem, I installed opencv for Python 3. I guess that uses CUDA, so maybe that is related?

Running opencv with gpu tensorflow causes crashes on my machine, there's some kind of interference, I have to disable opencv GPU using export OPENCV_OPENCL_RUNTIME=

@chrisranderson what I meant was trying to compare against a GPU version from a different version. GPU version is known to be slow on first kernel launch (up to a minute) when your architecture compute capability is not included

I tried export OPENCV_OPENCL_RUNTIME= before running tensorflow-gpu again, but it didn't seem to have an effect. I don't know what you mean by your second comment.
Slowness is expected, but is 30 second for a network like the one I posted normal? It's pretty small. It seemed like initialization used to be quite fast. Is there something else I need to try with opencv?
Thanks for your help.

The "up to a minute" slowness I refer to is what happens when CUDA starts PTX compilation and is independent of your computation size. If that's your bottleneck, then the solution is compile with your architecture since the official binary only comes with support for ancient cards (I usually compile with 6.1 support, some wheels are here -- https://github.com/yaroslavvb/tensorflow-community-wheels). I'm not sure how to tell if it's indeed the PTX compilation that's causing the slowness, maybe @zheng-xq knows  some tricks.

If it is indeed PTX compilation, you can look at the timeline, like this:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/python/client/timeline_test.py#L44
The first GPU kernel takes a really long time, without a good reason.

Okay, here are the results of the profile. I looked at it in chrome://tracing/, but wasn't able to make sense of it.
out.profile.zip

also you could try recompiling with support 6.1 architecture (just enter 6.1 during configure step)

Anything else I should be aware of besides the stuff in here?
https://www.tensorflow.org/install/install_sources
On Thu, Feb 23, 2017, 7:07 PM Yaroslav Bulatov<EMAIL_ADDRESS>wrote:

also you could try recompiling with support 6.1 architecture (just enter
6.1 during configure step)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/7755#issuecomment-282182584,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFNVls58QDYGuQPVwGT4R1Q0wRBk14_8ks5rfjtkgaJpZM4MIEM0
.

Starting the process of building from source right now.

FWIW, I had the same slow startup problem with the 'latest-gpu' docker image and a GFX1080.
Moving over to 'latest-devel-gpu', which is compiled with 6.1 architecture support, fixed it.

I had this same issue with my own docker container, even while using the 'latest-devl-gpu' container as @afri mentioned.
However, @chrisranderson, your comment about opencv made me realize what the problem was. You're right, compiling opencv with cuda support caused the slow startup for me.
By compiling opencv with -D WITH_CUDA=OFF, the problem is fixed, and my models run almost instantly.

I built from source, and I've also tried 0.12 again. Still very slow. @neil454 thanks for the reply, I'd like to give that a shot.

@neil454 I think that did the trick, thank you!

@chrisranderson Is this fixed, or is there a code fix warranted?

@girving I'd call it "band-aid fixed". Fully fixed would mean I could do whatever I wanted with opencv and not have it affect TensorFlow, I think. I have no idea why this happens or what the situation is, so maybe it's actually opencv's fault, or nvidia's fault or something.

Alas!  I'm going to close, but with sadness.  Let us know if a clearer picture ever emerges.


Service status analytics

Perform some basic service status analytics (availability, API requests to different services, in/out)

Done in https://github.com/0xJuancito/inlens-upptime


bug(go): Using Aspects causes panic: Call using *awscdk.IConstruct as type awscdk.IConstruct

:bug: Bug Report
Affected Languages

[ ] TypeScript or Javascript
[ ] Python
[ ] Java
[ ] .NET (C#, F#, ...)
[X] Go

General Information

JSII Version: v1.28.0 <!-- Output of jsii --version -->
Platform:  18.04.1-Ubuntu

What is the problem?

I am trying to use Aspects with the go CDK and I get the below error when running synth:
panic: reflect: Call using *awscdk.IConstruct as type awscdk.IConstruct

goroutine 1 [running]:
github.com/aws/jsii-runtime-go.Invoke(0x6e8de0, 0xc00009a270, 0x70535b, 0x5, 0xc0002a7ec8, 0x1, 0x1, 0x6900c0, 0xc0000a68a0)
        /home/ubuntu/environment/cdk/cdk-project-go/vendor/github.com/aws/jsii-runtime-go/runtime.go:216 +0x2b7
github.com/aws/aws-cdk-go/awscdk.(*jsiiProxy_App).Synth(0xc00009a270, 0x0, 0x70992e, 0x11)
        /home/ubuntu/environment/cdk/cdk-project-go/vendor/github.com/aws/aws-cdk-go/awscdk/awscdk.go:395 +0xb5
main.main()
        /home/ubuntu/environment/cdk/cdk-project-go/cdk-project-go.go:52 +0xfa
exit status 2

I was able to reproduce the issue with the default app that is created via cdk init --language=go app
Here is my code:
type Aspect struct{}

func (a *Aspect) Visit(node awscdk.IConstruct) {
        awscdk.Annotations_Of(node).AddInfo(jsii.String("Hello"))
}

type CdkProjectGoStackProps struct {
        awscdk.StackProps
}

func NewCdkProjectGoStack(scope constructs.Construct, id string, props *CdkProjectGoStackProps) awscdk.Stack {
        var sprops awscdk.StackProps
        if props != nil {
                sprops = props.StackProps
        }
        stack := awscdk.NewStack(scope, &id, &sprops)

        // The code that defines your stack goes here

        // as an example, here's how you would define an AWS SNS topic:
        awssns.NewTopic(stack, jsii.String("MyTopic"), &awssns.TopicProps{
                DisplayName: jsii.String("MyCoolTopic"),
        })

        return stack
}

func main() {
        app := awscdk.NewApp(nil)

        myStack := NewCdkProjectGoStack(app, "CdkProjectGoStack", &CdkProjectGoStackProps{
                awscdk.StackProps{
                        Env: env(),
                },
        })

        aspect := &Aspect{}

        awscdk.Aspects_Of(myStack).Add(aspect)

        app.Synth(nil)
}

Verbose Log

@all-contributors add @corymhall for bug

Thanks for reporting this! I have been able to reproduce this, and have identified a fix for that. I'll submit a PR shortly (a draft, though - as I still have to write a test that covers the previously broken case).


barchart library

First of all, thanks for this fantastic resource and tutorial -- your hard work on this is highly appreciated and looking forward to the next articles.
A quick question though: did you already publish the widget or demo Shiny app with source code generating the stacked bar chart below the gauges?

thanks for the kind words! at this stage we did not yet release that part of the code base. However, I can send you some code for that particular chart if you want (or a variation on this chart). What precisely do you have in mind? FYI, we will cover this chart (and others) in depth in the next tutorial, which should be out within a few weeks. In addition to what you have seen you then can also send updates to an already initialized chart via a proxy object e.g. changing the selected x or y range, removing series, adding series, or changing e.g. to a stacked bar chart.

Thank you very much for your prompt and kind reply, Herman!
If you can share the related code in a gist or e-mail, that would be very helpful. Although I look forward to the detailed tutorial, now I try to replicate the below barchart -- which includes the overall counts on the left y axis as bar charts and the group percentages on the right y axis with lines:

Hi,
I'll put something together for you and send you an update tomorrow, ok?
best H

That would be very cool, thank you very much!

Hi,
I just provided you access to a private GTIHUB repo that contains a small app I made for you demonstrating the c3.js line/bar example, including the brush.
You can install FrissC3Chart_1.0.0.tar.gz and run the app example in app.R, which are both in the FrissC3Chart folder.
The code for the R part is here:
https://github.com/FrissAnalytics/shinyJsTutorials2_preview/blob/master/FrissC3Chart/R/FrissC3Charts.R
while the code for the js part is here:
https://github.com/FrissAnalytics/shinyJsTutorials2_preview/blob/master/FrissC3Chart/inst/htmlwidgets/FrissC3LineBarChart.js
note we use underscore.js (see http://underscorejs.org/) to perform debouncing i.e. to make sure that when we move the brush we don’t overflow R with too many events to process.
See http://shiny.rstudio.com/articles/building-inputs.html for some more context on that (there is no mention of underscore there, we just use it because of convenience).
This example will be explained in more detail in the upcoming tutorial, which will take a few weeks to come out.
I suggest you study the R and js code a bit to see what we do here (the code documentation for now is a bit weak as we didn’t plan on releasing this just yet).
Obviously the brush min and max date info can be used to filter other charts, which we’ll do in the upcoming tutorial. This includes sending information from R to c3.js to zoom e.g. the area chart in our example.
Kind regards, Herman
PS1: please let me know when you have copied the content, then I’ll delete this repo.
PS2: RSTUDIO still needs to upgrade our account, so the showcase demo could be down for a day or so.
PS3: see my RSTUDIO Stanford shiny dev con user talk to get some more background in case you missed that:
https://www.rstudio.com/resources/videos/friss/
From: Gergely Daróczi<EMAIL_ADDRESS>Sent: zaterdag 14 mei 2016 23:07
To: FrissAnalytics/shinyJsTutorials<EMAIL_ADDRESS>Cc: Herman Sontrop<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [FrissAnalytics/shinyJsTutorials] barchart library (#4)
That would be very cool, thank you very much!
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHubhttps://github.com/FrissAnalytics/shinyJsTutorials/issues/4#issuecomment-219251603

Thank you very much, I've just cloned the repo and found the sources extremely useful & self-explanatory -- your kind and extremely quick help are highly appreciated!


quantitative metrics like Loop Habits？

Can we add quantitative metrics like Loop Habits? And it supports quantitative display imported from the loop?

Hi @JiangLongLiu , thanks for reaching out.
That's a great suggestion, and I will definitely implement this feature.

Hi @JiangLongLiu , thanks for reaching out.
That's a great suggestion, and I will definitely implement this feature.

Thank you very much! Please have a good day every day! ❤

this is closed right?

@ocean-haiyang Not yet, this feature requires a new habit type to record additional information, such as how many hours are spent today.


Button to pruge all intro marker for a selected library

First: Thanks for your work and making it available for everyone!
It would be great to have a button to pruge all intro marker for a selected library. My usecase would be to re-create all detected intros by plex. My library is really old and Plex got a bit better at detecting intros since the first release and I would like to re-detect all intros.

Definitely something I can look into. I unfortunately haven't had a ton of time to work on this lately, but it shouldn't be too difficult to implement. If you don't want to wait for a UI option and are comfortable running SQL directly against your database, running the following should work, where LIBRARY_ID is the id of the library you want to remove all markers for (you can use the --list command from the command line scanner to find the id):
DELETE FROM taggings
WHERE taggings.id IN (
	SELECT t.id AS id FROM taggings t
	INNER JOIN tags ON tags.id=t.tag_id
	INNER JOIN metadata_items ON metadata_items.id=t.metadata_item_id
	WHERE tags.tag_type=12 AND metadata_items.library_section_id=LIBRARY_ID
);

I'd definitely recommend shutting down PMS and backing up your db before trying it though.

 I was wondering if maybe you would be interested in revisiting this enhancement.  I'm in a similar boat where I would really like to wipe all of the markers from a particular library and start fresh without potentially destroying my database with my clumsy fingers, hah.
Though, I did see Plex say that community generated cloud markers (like the movie markers) are coming to TV soon, so it might be worth holding off actually doing anything until that rolls out as the analysis after a purge would likely be significantly faster.

Yes, I can look into this ~soon. One potential issue that I haven't looked into at all is that Plex might be caching the markers somewhere else in addition to the database (e.g. the XML files it also generates in the data directory). I've noticed that when I wipe out markers for a given episode and then reanalyze it season, the markers are added ~instantly instead of the usual wait for Plex to actually analyze the the files. If that's the case, it might not be as valuable to add to this application, though it may still be useful to have a way to "start fresh" with a section.

I think I've noticed this in the past, and the only way I could get the markers to successfully purge from a season and actually re-scan properly was to do a full Plex Dance.  So yeah, maybe they're stored somewhere else as well, I'm not sure.

The latest beta attempts to address this, giving you the option to completely remove all markers for a given library (both user- and Plex-generated). It might still run into the issue described above if your goal is to completely start the generation from scratch, but it will at least get you one step closer.

Closing as this is now in the non-beta release v2.2.0.


Sort by priority not wokring

Also make the setting the priority as mandatory

can you assign me this issue under IWOC?

is this good?

create pr then only i will be able to check or post the video

Uploading To-Do List and 6 more pages - Profile 1 - Microsoft Edge 2024-02-04 21-33-38.mp4…
i have attach the video with this comment

Uploading To-Do List and 6 more pages - Profile 1 - Microsoft Edge 2024-02-04 21-33-38.mp4…

Sort by priority working?

Yes ma'am
On Mon, Feb 5, 2024, 14:31 Kritika Gupta @.***> wrote:

Sort by priority working?
—
Reply to this email directly, view it on GitHub
https://github.com/Kritika30032002/To-Do-List-Application/issues/353#issuecomment-1926507146,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AX5QX7FID7IJBZ6EBU2FQLTYSCNYBAVCNFSM6AAAAABCMCAKOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRWGUYDOMJUGY
.
You are receiving this because you were assigned.Message ID:
@.***>

show video


why I can not use request in the api.js

just like :
request('http://www.google.com', function (error, response, body) {
if (!error && response.statusCode == 200) {
console.log(body) // 打印google首页
}
})
add in the api.js

For anyone running into this or similar issues, make sure you de-register the service worker after switching from the "production" to "development" mode.
You can do that in Chrome in dev tools -> Application -> Service Workers


Deprecations and Removals in Chrome 69

:+1:

:+1:

:tada: This has been pushed live to https://developers.google.com/web/


boto EndpointConnectionError in assume_role()

My graphics colleague @helloeujin got a botocore.exceptions.EndpointConnectionError when running clokta --profile pub.
The error happened inside clokta/role_assumer.py, line 58, in assume_role(). It seems it was trying to connect to a url in the form "https://sts.[long alphanumeric string].amazonaws.com/"
Any ideas? She is using SMS for two factor, if that matters. I've pasted a screenshot in #aws-automation with the full error stack (I'm wary of pasting it in this public repo just in case).
Thanks!

This was resolved offline


WIP: DA1469x crypto driver

Let https://github.com/apache/mynewt-core/pull/2175 merge and then merge the top change.

Let #2175 merge and then merge the top change.

Just merged it!

I am under the impression that mbedtls_aes_setkey_enc and mbedtls_aes_setkey_dec do the same thing, so maybe the code could be moved into a single function? Nice catch on the error return, the reason for only having _enc is that we only use CTR-mode for flash and image encryption and the CTR code in mbedTLS does not check return values!

I am under the impression that mbedtls_aes_setkey_enc and mbedtls_aes_setkey_dec do the same thing, so maybe the code could be moved into a single function? Nice catch on the error return, the reason for only having _enc is that we only use CTR-mode for flash and image encryption and the CTR code in mbedTLS does not check return values!

made a common function.

LGTM
The return (ret == AES_BLOCK_LEN ? 0 : -1); looks a bit weird to me, because I am very used to return (ret == AES_BLOCK_LEN) ? 0 : -1;, but given the low precedence of the ternary conditional it should be fine!

done.


Move "master" to "main"

Other related projects use "main", so I think, it makes sense to follow the same logic here. What do you think @db-operator/admins?

[x] Move master content to main
[x] Update github actions

It's done, now main is the default branch, but master is still there, just in case


support w:color

I have a need to be able to look at the color of the text in a paragraph.
For example, I have a test question:

Which theory describes the entire earth, made of both living and non-living parts, as single system?
reda.  Gaia theory
b.  Evolution theory
c.  Law  of inheritance
d.  Law of production
e.  None of the above

I want to be able to see that the color is red (value ff2600). I tried to look through the code and stumbled on the CT_rPr class. It has a tag for w:color, but when I tried to set up an @property it said AttributeError: 'CT_RPr' object has no attribute 'color'.
I am happy to take another stab at adding my own attribute, but I need a hint if I am going to get any farther.

Hi Erik, @virajkanwade has this working in a pull request here: https://github.com/virajkanwade/python-docx/commits/feature/styles
I'll be getting this into a minor release here probably in a couple weeks now that I've got the 0.8.0 release out. The API will change a little bit based on some discoveries we made working on this but it will be quite close.
Let me know if you need more to go on :)

I was able to cherry pick the color part of the pull request and successfully install python-docx with that commit into a virtualenv. I can query the color using paragraph.style.font.color, but the color for every paragraph in the document reads (0, 0, 0) instead of (ff, 26, 0) - red for some paragraphs.
Thanks for the pointer, it opened up a new avenue. If you see anything I should be doing, let me know. I get the sense that I am looking at color from the wrong object.

Hmm, doesn't sound quite right, I think what you're looking for is run.font.color.
Most runs won't have any directly applied color, and should return None in that case, not (0, 0, 0) which is a valid color (black).
All directly-applied character formatting is applied at the Run level. The reason a paragraph style has a font property is it applies both paragraph and character formatting.
When a run doesn't have a specific value, i.e. returns None, that doesn't mean it doesn't have a color, it just means it doesn't have a directly-applied one. In this case it inherits the value from the style hierarchy, which is several layers deep, ending at the document defaults. We don't have functionality built-in yet to find the "effective" value (by traversing the style hierarchy for a run or whatever), mostly because it's a big job and we'll need to get further along on some infrastructural bits before that will be feasible.

Ah ha! That did it. Now I can find text that has the font color ff2600 using the pull request above. Now I have another conundrum, background color. Any advice is much appreciated. Thank you so much for the help so far. It has been very helpful.

Added Font.color in release 0.8.3 today, Feb 19, 2015. User docs are here: http://python-docx.readthedocs.org/en/latest/user/text.html#font-color
If you submit your new question on background color in a new issue I'll be happy to respond.


@defaultValue with an array as a default value

Is it possible to have an array after the @defaultValue tag?

Yes, it is a documentation block so you could put an English sentence as well

Worth linking the doc page: https://tsdoc.org/pages/tags/defaultvalue/


Adding use_masked attr to new GPT-2 YAMLs

We just merged in some new GPT-2 YAMLs, but they lack the use_masked_lm property. This serves to add this to the YAMLs, so BERT can get merged in.

Ah, I just started rebasing and a ton of problems appeared, so this seemed quicker
Basically, adding the GPT-2 YAMLs made PR Gate fail all of a sudden (due to missing a value), and it was quicker to add this PR rather than rebase off of dev + fix it in the BERT PR!


SQL: Add test for concurrent JDBC queries.

@fjy FYI you can restart a build without closing/reopening PR: http://stackoverflow.com/a/17624403/648955

@leventov yes i Know but it is easier to close/reopen

You should probably be on the travis page anyway though, to check if the failure was a legitimate issue with the patch or not.


Feat: add Device Restore

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Add device restore for model mesh in gizmo & grid

Codecov Report
Patch and project coverage have no change.

Comparison is base (e567f89) 22.19% compared to head (0f25765) 22.19%.

:exclamation: Current head 0f25765 differs from pull request most recent head ed64687. Consider uploading reports for the commit ed64687 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #211   +/-   ##
=======================================
  Coverage   22.19%   22.19%
=======================================
  Files          11       11
  Lines         473      473
  Branches       70       70
=======================================
  Hits          105      105
  Misses        364      364
  Partials        4        4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Final metadata changes

Final changes to metadata fields (to be confirmed during the data call on Monday 11th of September):
Changes in origin metadata fields

[ ] Rename dataset_url_main -> url_main. For simplicity, and to avoid the confusing word "dataset".
[ ] Rename dataset_url_download-> url_download. Idem.
[ ] Rename dataset_title_owid -> title. This will be the title of the snapshot.
[ ] Rename dataset_description_owid-> description. This will be the description of the snapshot, which in most cases will coincide with the title of the data product, but not always.
[ ] Rename dataset_title_producer-> data_product_title. This will be the title of the data product to show in data pages. If not given, it should fall back to title.
[ ] Rename dataset_description_producer-> data_product_description. This will be the description of the data product to show in data pages). If not given, it should fall back to description.
[ ] Rename citation_producer-> citation_full. This field is not necessarily written by the producer, and is the only place where we can host a full citation).

Changes in indicator's presentation metadata fields

[ ] Rename producer_short -> attribution_short. Given that it could also be a dataset name, in the same way that we have attribution_short at the origin level.
[ ] Rename topic_tags_links -> topic_tags. For simplicity, and because they are not links.

Changes to transfer to grapher

[ ] When creating new charts and data pages, we could construct the subtitle as '{description_short} Measured in {unit}'.
[ ] By default, indicator's presentation's producer_short (to be renamed attribution_short) should automatically be filled out by concatenating the origin.attribution_short of all the origins in an indicator.

Thanks for putting thought into this and for the summary, Pablo! Overall, I agree with all the modifications.
Minor comment:
From your definitions, it seems like title and description would be required and data_product_title and data_product_description would not and fall back to the formers when not specified. Do you prefer this over the opposite? E.g. encouraging users always to set data_product_title and data_product_description; and if the snapshot != data product, they need to specify title and description. I don't feel strongly about any of these, but maybe you have thought about this?

@pabloarosado I'm starting work on renaming and will take care of all three lines - ETL, MySQL and Data Page. Let me know if you have any other additions.

Hi @Marigold I'm going to update now the description above to include the changes we discussed yesterday at the data call.


AutoGen playground: Language for specifying agents

Relavant: https://pewekar.github.io/AutogenAppGenerator/


Added styling prop "styleIndicator" to change the styling of indicator Icon. Used it wherever styles.indicator was used.

There was an Icon component that I needed to style as per my need but, there was no prop to add additional styling to that component. So, I've added that prop.

I am gonna approved and publish. Thank you for the contribution.