Download Pluralsight Subtitles

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

I use this command for downloading course subtitles in Pluralsight:

youtube-dl --username USERNAME --password PASSWORD --all-subs URL --skip-download

But I receive this :
[pluralsight:course] unity-shader-graph-introduction: Downloading JSON metadata
[download] Downloading playlist: Introduction to Shader Graph in Unity
[pluralsight:course] playlist Introduction to Shader Graph in Unity: Collected 19 video ids (downloading 19 of them)
[download] Downloading video 1 of 19
[pluralsight] Downloading login page
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

But when I use this command, I can download videos:

youtube-dl --username USERNAME --password PASSWORD URL

What should I do to fix this problem ?

which python version you're using? We've been identified that issue happening with some particular python versions. Try using version 3.7 or 3.8.
Also -all-subs feature is not available on pluralsight yet, it's just downloading english subtitles.
I've opened a Pull Request for that but it hasn't been merged yet https://github.com/ytdl-org/youtube-dl/pull/24832

@jaimebl It was 3.6.2 I just upgraded it to 3.8.2 but still the same problem.
What should I use instead of --all-subs ?

@ATHellboy With python 3.8.2 you shouldn't get that 403 error message(just tried myself same version), double check you're using that version in the logs and also that your account is not banned by logging in manually with the browser.
There's no way to get all subs currently, you can check out my pull request here if you want to try it https://github.com/ytdl-org/youtube-dl/pull/24832
However I wouldn't use --all-subs as it is now because you'll get Too many requests error message eventually and get banned, I'd better use --write-sub --sub-lang en,es instead and list the languages that you want to get

@jaimebl I think you are right about the Python version which it uses.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://app.pluralsight.com/library/courses/unity-shader-graph-introduction/table-of-contents', '-u', 'PRIVATE', '-p', 'PRIVATE', '--write-sub', '--sub-lang', 'en,es', '--verbose']
[debug] Encodings: locale cp65001, fs mbcs, out cp65001, pref cp65001
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: none
[debug] Proxy map: {}
[pluralsight:course] unity-shader-graph-introduction: Downloading JSON metadata
[download] Downloading playlist: Introduction to Shader Graph in Unity
[pluralsight:course] playlist Introduction to Shader Graph in Unity: Collected 19 video ids (downloading 19 of them)
[download] Downloading video 1 of 19
[pluralsight] Downloading login page
ERROR: Unable to download webpage: HTTP Error 403: Forbidden (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\common.py", line 627, in _request_webpage
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\YoutubeDL.py", line 2238, in urlopen
  File "C:\Python\Python34\lib\urllib\request.py", line 470, in open
  File "C:\Python\Python34\lib\urllib\request.py", line 580, in http_response
  File "C:\Python\Python34\lib\urllib\request.py", line 508, in error
  File "C:\Python\Python34\lib\urllib\request.py", line 442, in _call_chain
  File "C:\Python\Python34\lib\urllib\request.py", line 588, in http_error_default

How can I change it ? Although I don't know why the version is 3.4.4. (I use youtube-dl on Windows if it helps)

@ATHellboy I think for now, the easiest way to get this working is grabing the cookies as I list in this other comment:
https://github.com/ytdl-org/youtube-dl/issues/24895#issuecomment-617307358

@jaimebl I just exported cookies by EditThisCookie which you mentioned in that comment and put it into text file and put it in the folder which I execute youtube-dl there.
Then using this commend :

youtube-dl --cookies cookies.txt https://app.pluralsight.com/library/courses/unity-shader-graph-introduction/table-of-contents --write-sub --sub-lang en,es

But still there is an error:
Traceback (most recent call last):
  File "__main__.py", line 19, in <module>
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\__init__.py", line 474, in main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\__init__.py", line 441, in _real_main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\YoutubeDL.py", line 419, in __init__
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\YoutubeDL.py", line 2329, in _setup_opener
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\utils.py", line 2762, in load
  File "C:\Python\Python34\lib\http\cookiejar.py", line 2005, in _really_load
http.cookiejar.LoadError: 'cookies.txt' does not look like a Netscape format cookies file

What do I do wrong ?
It is because of this ? => 'cookies.txt' does not look like a Netscape format cookies file.

@ATHellboy  try to do this https://github.com/ytdl-org/youtube-dl/issues/24895#issuecomment-617324141 , i had the same problem , if you see in your track you still use Phyton 3.4 with youtve-dl even you already installed Phyton 3.8
(your track )
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://app.pluralsight.com/library/courses/unity-shader-graph-introduction/table-of-contents', '-u', 'PRIVATE', '-p', 'PRIVATE', '--write-sub', '--sub-lang', 'en,es', '--verbose']
[debug] Encodings: locale cp65001, fs mbcs, out cp65001, pref cp65001
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362

Ah god, yes you have to manually edit one line of those cookies, look for _sdsat_Target in your cookies file and add this after Subscription info: (Notice there's a tab at the beginning of the following snippet)
	InTerm:true|FullLibrary:true|Free:true|Trial:false|UserType:individual

So it will look something like this
app.pluralsight.com	FALSE	/	FALSE<PHONE_NUMBER>	_sdsat_Target Subscription Info	InTerm:true|FullLibrary:true|Free:true|Trial:false|UserType:individual

@jhondbv Thanks. It really helps. It works but just for the first time, I interrupted it then try it again and still get that 403 error.
@jaimebl Thanks, it works.
Just your last statement maybe confusing for some people.
Should put InTerm:true|FullLibrary:true|Free:true|Trial:false|UserType:individual in the next line after comma and put comma again at the end of this line.

@jaimebl I get this error, what the hell:
[pluralsight] 4dd0c174-ed1c-4968-b05f-72e01d1b5ed4-0: Waiting for 2 seconds to avoid throttling
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.```

Although I downloaded all this course videos before.

@ATHellboy 😄 can you paste your full log?

@jaimebl
C:\Program Files (x86)\VideoDownloader>youtube-dl --cookies cookies.txt https://app.pluralsight.com/library/courses/unity-shader-graph-introduction/table-of-contents --write-sub --sub-lang en,es
[pluralsight:course] unity-shader-graph-introduction: Downloading JSON metadata
[download] Downloading playlist: Introduction to Shader Graph in Unity
[pluralsight:course] playlist Introduction to Shader Graph in Unity: Collected 19 video ids (downloading 19 of them)
[download] Downloading video 1 of 19
[pluralsight] de51c857-ca7d-49fa-8c14-4846faaee0b1-0: Downloading JSON metadata
[pluralsight] de51c857-ca7d-49fa-8c14-4846faaee0b1-0: Downloading mp4-high-widescreen viewclip graphql
[pluralsight] de51c857-ca7d-49fa-8c14-4846faaee0b1-0: Downloading mp4-high-widescreen viewclip JSON
[pluralsight] de51c857-ca7d-49fa-8c14-4846faaee0b1-0: Waiting for 4 seconds to avoid throttling
[pluralsight] de51c857-ca7d-49fa-8c14-4846faaee0b1-0: Downloading captions JSON
WARNING: es subtitles not available for 043bd1cb-098d-4e32-a0f8-fdafd5e66eee
[info] Writing video subtitles to: Course Overview-043bd1cb-098d-4e32-a0f8-fdafd5e66eee.en.srt
[download] Destination: Course Overview-043bd1cb-098d-4e32-a0f8-fdafd5e66eee.mp4
[download] 100% of 3.95MiB in 00:04
[download] Downloading video 2 of 19
[pluralsight] 4dd0c174-ed1c-4968-b05f-72e01d1b5ed4-0: Downloading JSON metadata
[pluralsight] 4dd0c174-ed1c-4968-b05f-72e01d1b5ed4-0: Downloading mp4-high-widescreen viewclip graphql
[pluralsight] 4dd0c174-ed1c-4968-b05f-72e01d1b5ed4-0: Downloading mp4-high-widescreen viewclip JSON
WARNING: Unable to download JSON metadata: HTTP Error 403: Forbidden
[pluralsight] 4dd0c174-ed1c-4968-b05f-72e01d1b5ed4-0: Waiting for 2 seconds to avoid throttling
ERROR: No video formats found; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

@ATHellboy Are you able to get to the website with your browser?

@jaimebl Yup

@ATHellboy try adding this argument on your youtube-dl command

-f "best[height<=720]/worst[height>720]"

something like : >youtube-dl --cookies cookies.txt --write-sub --sub-lang en,es --verbose -f "best[height<=720]/worst[height>720]" https://app.pluralsight.com/library/courses/unity-shader-graph-introduction

@ATHellboy You're probably having some issue with your authentication, if you try without the cookies parameter I bet you will get the same log response.
youtube-dl https://app.pluralsight.com/library/courses/unity-shader-graph-introduction/table-of-contents --write-sub --sub-lang en,es --verbose
Try setting your cookies again 🤔


fix: skip the checking only for xxx_test not xxx_test_yyy

Hi, I am no longer interested in maintaining this PR because the other tool revive seems to satisfy my need.


HTTP Error 503: Service Unavailable

The script worked great for upto ~28000 images and then I started getting HTTP Error 503: Service Unavailable. Restarting the script after a day or two downloaded another ~32000 images before throwing the 503 error again.
Couldn't exactly figure out the cause or fix for this issue.  If so, is there a way to get authorization for more?

I fixed it here https://github.com/macro-dadt/google-images-download/blob/master/google_images_download/google_images_download.py

@BigRedT can you give me the complete query that you were using? Were the images skipped on getting 503s or did the script break? If it is latter, I will need the query to reproduce the issue from my end.

@hardikvasa it wasn't a single query. I was executing multiple single-word queries in parallel using starmap function in the python multiprocessing module and downloading 20 images per query. Those 60000 images actually correspond to 3000 queries, which actually might be the limit on free queries allowed by Google per user per month.
Here are some snippets from my parallelized implementation. Each process would call the following function with a different word:
def downloader(word,args):
    response = google_images_download.googleimagesdownload()
    search_arguments = {
        'keywords': word,
        'limit': args.images_per_word,
        'safe_search': True,
        'output_directory': args.outdir,
        'usage_rights': 'labeled-for-noncommercial-reuse-with-modification',
        'color_type': 'full-color',
        'type': 'photo',
    }
    print('-'*80)
    print(f'Downloading images for word: {word}')
    print('-'*80)
    absolute_image_paths = response.download(search_arguments)[word]

The launch code looks like:
def main():
    args = parser.parse_args()

    # Create output directory if it doesn't exist
    # Results of queries are saved in subdirectories of this output directory
    io.mkdir_if_not_exists(args.outdir)
    print(args.outdir)

    # Load a dict where the keys are the words to be queried in Google
    vocab = io.load_json_object(args.vocab_json)
    vocab_list = list(vocab.keys())
    with Pool(40) as p:
        p.starmap(downloader, product(vocab_list,[args]))

I am also struck with the same error. I have a script running every 10 minutes to fetch for a single image each for different URLs. I successfully downloaded for 4500 odd images but it is continuously giving me error on the server machine.
I am in a dire need for a workaround.

Yea seems to be like server is throttling on the request rate. @BigRedT @rajexp can you give me a complete error stack trace if you still have it? That will help me build a custom retry mechanism within the library to avoid failures.

Closing this thread for now. let me know if this issue pops up again and i will investigate it further :)


Frequency DAILY not handled in checkDate

The DAILY_FREQUENCY in MXLCalendarEvent is never used, meaning that RRULES like this "RRULE:FREQ=DAILY;UNTIL=20190629T110000Z" don't work. Is there any known reason for this @ramonvasc ?

The DAILY_FREQUENCY in MXLCalendarEvent is never used, meaning that RRULES like this "RRULE:FREQ=DAILY;UNTIL=20190629T110000Z" don't work. Is there any known reason for this @ramonvasc ?

This is actually a bug, thanks for reporting. I'll try to fix it as soon as I have some spare time.

I have a fix for this ready. Will put it up as soon as https://github.com/ramonvasc/MXLCalendarManagerSwift/pull/6 is resolved.

The library has been updated with the fix from @minhd-gh.

https://github.com/ramonvasc/MXLCalendarManagerSwift/pull/7 has the fix.


Reduce Load Times by Leveraging Nice API

The application load times of Nice leaves a lot to be desired. When the web app loads it makes seven calls to the ADN API and one call to the Nice API. This should really be reduced as much as possible, and one way to do this might be to allow a person to choose whether their timeline is processed locally, or through the Nice API. Processing through Nice would mean passing the ADN Token from the client through Nice to collect the appropriate data whenever collecting post-related data. Non-post-related calls could be made directly to the API.
This would of course be an option that people would opt into rather than an enforced change, and ADN Tokens would not be stored on the Nice API server anywhere.
Advantages of Doing This

local post filtering would no longer be required (beneficial for less powerful devices)
less bandwidth would be required (beneficial for less-fat pipes)
some avatar processing could be done by the Nice API (compressing the account avatars as much as possible to save on bandwidth)

Disadvantages of Doing This

Everything runs through the Nice API server, which will force the move to a more powerful server sooner
There may be questions about privacy, which is something I've explicitly tried to keep transparent
The Nice API becomes a necessary component to some people's interaction on ADN (downtime will be less acceptable)

To make the move or no ...

Moving this from Question to Enhancement. I want this in place by 1.0.0.

This is now functioning for a number of key functions.
NOTE: The Nice API Proxy will only be used if a person chooses to use it. By default the system will continue to process posts locally.
Closing Ticket.


Ctrl-C does not work when using with alternative layouts (e.g. Dvorak)

Which operating system does the issue occur on? Mac, if I remember things correctly, this works on linux just fine.
If on linux, are you using X11 or Wayland?
Short description:

When I am using the standard QWERTY layout, everything works fine.
When using dvorak, Ctrl-C stops working

The config is here: https://github.com/aignas/dotfiles/blob/master/alacritty/alacritty.yml.symlink

Could you share the output of alacritty --print-events at the specific time you press ^C?
Also, just to make sure your keymap works properly, does Ctrl + the key that's C on QWERTY work with dvorak too? Does typing in dvorak work at all?

C on QWERTY is J on Dvorak and I am not sure if I am not seeing any mixed behaviour, as the key does not behave exactly as it does when using Qwerty.
 Not just Ctrl+C? Alacritty doesn't bind copying to Ctrl+C by default since that's usually sending a SIGINT signal.
If Ctrl+Shift+C doesn't work, you should remap the key binding in your config to Ctrl+Shift+I. Looking at the output of --print-events, it looks like the I key is reported to Alacritty for you.

Sorry for the delay in replying. I don't mean to use Ctrl-c for copying, I mean Ctrl-c for doing the keyboard interrupt to exit a program. When I press Ctrl-c in Dvorak (Physical Ctrl-i), alacritty missinterprets it as Ctrl-i and not Ctrl-c. However, TMUX is receiving the correct key combination when using the prefix (Ctrl-b in my case).
Since I am using Dvorak and QWERTY interchangably (Dvorak with the laptop keyboard, QWERTY with external keyboards), it means that alacritty only works as intended when using an external keyboard.

Looks like this is a duplicate of https://github.com/jwilm/alacritty/issues/458. That issue mostly mentions the Command key, but it should still be the same issue.


Can you specify what are n and k here?

https://github.com/kamyu104/LeetCode/blob/1ad86d344f12384e7cd97d7d1ffb41f1784d403f/Python/combination-sum.py#L1
Can you please specify what n and k are in the time and space complexities here?
Also if possible can you also add add comments like you added here, so as to make it more clear how we got there. Seems like finding time complexity for this problem is not very straight forward.
Thanks for maintaining this awesome repo.

k is the max length of each combination as problem describes.
For a combination, we may have at most n choices for each number, so there is at most n^k combinations with k-length. Thus, it's O(k *n^k).
Currently,  I only added detailed comments for harder problems as I think. Most problems are too trivial to add, I prefer to let learners to figure out it by self.

For your statement:

For a combination, we may have at most n choices for each number, so there are at most n^k combinations with k-length. Thus, it's O(k *n^k).

Since every number can have max. n choices and there are max k numbers, we would get
n * n * n * ....k times which would give n^k.
But why did we multiply this by k then to make it k * n^k?
I think it should be p * n ^k where p is the total combinations we produced, k is the max length of each combination and n is the total number of candidates.

n^k is the max total combinations we could produce. Each combination is with at most length k.
Time complexity would be "the max length of one combination (k) * the max count of total combinations (n^k)". It's O(k * n^k)  .


Fix: seg fault when execute some solutions with extreme larger size

The seg fault is caused by the precision lose when converting large int32 into fp32 for remapping workgroup ID from 1D to 3D.
Below command would hit seg fault if without this PR.
./hipblaslt-bench -f matmul --a_type f8_r --b_type f8_r --c_type bf16_r --d_type bf16_r -m 128256 -n 40960 -k 8192 --transA T --transB N --compute_type f32_r --algo_method all -i 1 -j 1

[----------] Global test environment tear-down
[==========] 50058 tests from 13 test suites ran. (723014 ms total)
[  PASSED  ] 50058 tests.
hipBLASLt version: 1200


How should I stop training?

After 10000 steps to train the mode, the training progress continued. How should I stop it? Or how should I change in the code to change this situation?

You can stop the training progress anytime you want by Ctrl+C.
Or if you want to stop after 10000 steps, simply modify this line  as follows:
- while not coord.should_stop():
+ while step < 10000 and not coord.should_stop():

Thank you, that would be very grateful to me.


Chomp whitespaces in chart templates.

Templates in Istio chart should chomp whitespace where possible.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: morvencao
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

install/OWNERS [morvencao]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@morvencao: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/e2e-bookInfoTests-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/e2e-bookInfoTests-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/e2e-bookInfoTests-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-bookInfoTests-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/e2e-mixer-no_auth.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-bookInfoTests-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-mixer-no_auth-mcp.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-bookInfoTests-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-dashboard.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-bookInfoTests-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth-mcp

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-bookInfoTests-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-dashboard

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
7d12b34bb43b0b19c5ae4b61ba61bbbaeaf4b27a
link
/test istio-pilot-multicluster-e2e

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-bookInfoTests-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-simpleTests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@morvencao: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-unit-tests

prow/istio-pilot-e2e-envoyv2-v1alpha3-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3-mcp

prow/e2e-bookInfoTests-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-bookInfoTests-envoyv2-v1alpha3

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth

prow/e2e-mixer-no_auth-mcp.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-mixer-no_auth-mcp

prow/e2e-dashboard.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-dashboard

prow/e2e-simpleTests.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test e2e-simpleTests

prow/istio-pilot-multicluster-e2e.sh
9b8daad1c47d6152ea8c376ea4c52d43728e2e83
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Okay to merge this in master.


Improve terminology and syntax around iterators

The term "iteration" suggests that something happens in sequence but that is a false assumption in lyra since the default behavior for an "iteration" is to apply the activities in parallel. I would therefore suggest that we use the term "repeat" instead.
Another improvement is to combine the current "style" key (can be set to "each", "eachPair", "range", or "times") and the "over" key (holds the value), and rename the 'variable(s)' to 'as', such that:
iteration: {
  style: 'each',
  over: [
    ['<IP_ADDRESS>', false, 'primary'],
    ['<IP_ADDRESS>', true, 'secondary']
  ],
  variables: ['ip', 'backup', 'role']
}

// Or for an example using times
iteration: {
  style: 'times',
  over: 5
}

becomes:
repeat: {
  each: [
   ['<IP_ADDRESS>', false, 'primary'],
   ['<IP_ADDRESS>', true, 'secondary']
  ],
  as: [ip, backup, role]
}

// Or for an example using times
repeat: {
  times: 5
}

The YAML (where "style" is currently named "function", doh!) the equivalent would change from:
iteration:
  function: each
  over:
    - ['<IP_ADDRESS>', false, primary]
    - ['<IP_ADDRESS>', true, secondary]
  variables: [ip, backup, role]

# Or for an example using times
iteration:
  function: times
  over: 5

into:
repeat:
  each:
    - ['<IP_ADDRESS>', false, primary]
    - ['<IP_ADDRESS>', true, secondary]
  as: [ip, backup, role]

# Or for an example using times
repeat:
  times: 5

The term collect came up as a replacement for repeat. It also resonates well with the term into which could be introduced to name the actual collection of outputs. Consider:
collect: {
  each: [
   ['<IP_ADDRESS>', false, 'primary'],
   ['<IP_ADDRESS>', true, 'secondary']
  ],
  as: ['ip', 'replica', 'role'],
  into: 'loadBalancers'
}

Closed by #252, lyraproj/puppet-workflow#20, and lyraproj/ts-workflow#5.


BHV-21381:InputSample: Removed unnecessary accumulation of directionality in style at the time of creation

Issue:
In Input, directionality property in style is getting appended twice at the time of creation.
Fix:
No necessity of calling  "this.valueChanged" twice in create function of Input.js, as it is already been invoked from "this.placeholderChanged" function. Removed "this.valueChanged" call from create function.
Enyo-DCO-1.1-Signed-off-by: Rakesh kumar<EMAIL_ADDRESS>

Hi @rakesh25 can you clean-up this PR and only include the requisite changes you've made instead of these other existing commits? Thanks!

Closing this PR so to include only the requisite changes other than exisiting commits..Will create a new PR.


[requires.io] dependency update on master branch

Coverage remained the same at 98.187% when pulling 5fba77cd69b98789d82f79c0aa5fe6f64766d813 on requires-io-master into 9675e91ac97188a604430420dd46a03324b9524c on master.


v1.0.3 does not work with feign-core 10.x.x

Upgraded my projects to JDK 11 and as a result I had to upgrade also the Spring Cloud dependencies to the Greenwich.RELEASE. The Greenwich.RELEASE upgrades the feign libraries to 10.1.0, however the feign-annotation-error-decoder, still uses 9.5.0.
The Response class in the latest feign-core library introduced a check that is breaking the ExceptionGenerator in the feign-annotation-error-decoder
The check is:
checkState(builder.request != null, "original request is required");
and this is the broken code:
static {
        Map<String,Collection<String>> testHeaders = new HashMap<String, Collection<String>>();
        testHeaders.put("TestHeader", Arrays.asList("header-value"));

        TEST_RESPONSE = Response.builder()
            .status(500)
            .body((Response.Body) null)
            .headers(testHeaders)
            .build();
    }

The issue is replicable through your unit tests, if the parent version is bumped to 10.2.1
Fix is a one-liner in the static block of ExceptionGenerator

Nice! Thanks! Are you ok to make the change, I'll review it and we try to
push it out?
On Tue, 19 Feb 2019, 15:51 Karl<EMAIL_ADDRESS>wrote:

The issue is replicable through your unit tests, if the parent version is
bumped to 10.2.1
Fix is a one-liner in the static block of ExceptionGenerator
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/OpenFeign/feign-annotation-error-decoder/issues/11#issuecomment-465159670,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANOpit2bOZoX27S1gTAfDZ_jyBCXPbeFks5vPA9mgaJpZM4bCzu1
.

Sure. Just tell me where to commit

The usual way to contribute is:

Fork the repo to your own side
Make the changes (and tests if necessary)
Create a PR against the repo inside the OpenFeign org
I'll review it, provide comments (if any - given your suggestion it might
be a simple one-liner)

I'll merge it if all good, and then hit the release button that will upload
it to maven Central - and job done :)
Does that sound fair?
On Thu, 21 Feb 2019, 09:37 Karl<EMAIL_ADDRESS>wrote:

Sure. Just tell me where to commit
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/OpenFeign/feign-annotation-error-decoder/issues/11#issuecomment-465909874,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANOpivCPahbhcKVGFB29EihmgZ9xMJX2ks5vPlq0gaJpZM4bCzu1
.

Very :) PR done. Thanks.

released with 1.1.0 - should be in maven-central shortly. was compiled with openjdk8 (as opposed to oraclejdk8 - which is what I expected it to be compiled with - but... should still work. 1.1.1 will be compiled with oraclejdk8).

is it normal that it's still not visible by this time?
https://mvnrepository.com/artifact/io.github.openfeign/feign-annotation-error-decoder

No... I looked for it. Turns out Travis cancelled the build that pushes it.
Since then, I've released a new version (1.1.1) and that didngo through and
should show up shortly.
Unfortunately... Due to a race condition, the first build was a Java 11
compiled build. If you pull it and it's java11 (and you need java8) let me
know - I'll release a 1.1.2 with Java 8 (I've removed the race at build
time issue... So it's always java8 that actually gets deployed/built/etc).
Thanks!
On Fri, 1 Mar 2019, 11:29 Karl<EMAIL_ADDRESS>wrote:

is it normal that it's still not visible by this time?
https://mvnrepository.com/artifact/io.github.openfeign/feign-annotation-error-decoder
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/OpenFeign/feign-annotation-error-decoder/issues/11#issuecomment-468619312,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANOpiquqnaDtJ8SCiAHyu9VsD-39jkt7ks5vSQDzgaJpZM4bCzu1
.

@karlmuscat I just pushed 1.1.2 - compiled with java 8 (1.1.1 turns out, in maven central, is compiled with java 11 as I just confirmed). There's no difference between those two. It won't "show up" for a few hours in search results since those take time to get updated, but if you pull it into your project it should work now. Thanks, and sorry for the mess!

First check your dependency by using mvn dependency:tree
Use the below dependency Management for spring-boot to download the suitable versions.
	<dependencyManagement>
	<dependencies>
		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-dependencies</artifactId>
			<version>${spring-cloud.version}</version>
			<type>pom</type>
			<scope>import</scope>
		</dependency>
	</dependencies>
</dependencyManagement>

I am using Java 10, cloud.version is Finchley.SR2 and sprinb-boot:2.2.0 and spring-cloud-starter-openfeign :2.1.2.RELEASE. and this combination worked for me to fix the issue.
Acctual problem was 10.x.x feign-core  was not working only io.github.openfeign:feign-core:jar:9.7.0:compile was working


Use the user id as the default subject identifier.

At the moment default subject identifier (in OIDC and SAML) used is username. However after onboarding user id to user store manager level, we are now in a position to use the user id as the default subject identifier. In the same time OIDC recommends a locally unique and never reassigned identifier as the subject identifier. So user id in the product matches with this definition and it makes more sense to have this as the subject identifier.
With this improvement,

[x] backend implementation needs to be changed to use the user id as the subject identifier instead of username.
[ ] frontend needs to be improved to reflect this change.

Below are the identified places that use the username as the subject identifier in OIDC related functionalities.
OAuth 2 introspection response
sub
      OPTIONAL.  Subject of the token, as defined in JWT [RFC7519].
      Usually a machine-readable identifier of the resource owner who
      authorized this token.
username
      OPTIONAL.  Human-readable identifier for the resource owner who
      authorized this token.

Current sample response
{
   "aut":"APPLICATION_USER",
   "nbf":1626852504,
   "scope":"openid",
   "active":true,
   "token_type":"Bearer",
   "exp":1626856104,
   "iat":1626852504,
   "client_id":"i9mrKqGVZCjzMWzKZljr0XaD_SMa",
<EMAIL_ADDRESS>}

While the username is already there according to the spec, we can additionally add the user id of the user now.
OIDC Id token
sub
      REQUIRED. Subject Identifier. A locally unique and never reassigned identifier within the Issuer for the End-User,
      which is intended to be consumed by the Client, e.g., 24400320 or AItOawmwtWwcT0k51BayewNvutrJUqsvl6qs7A4.
      It MUST NOT exceed 255 ASCII characters in length. The sub value is a case sensitive string.

Current sample id token
{
  "isk": "688e079e07c56423d0523d185a5dbc207f4ca4b45ccb1f317bf875bad4d2b981",
  "at_hash": "gj5-oIAymbY--q0OjMCFew",
  "aud": "i9mrKqGVZCjzMWzKZljr0XaD_SMa",
  "c_hash": "3e2SySfxXb0-NlQCN9vQ0g",
  "sub": "admin",
  "nbf":<PHONE_NUMBER>,
  "azp": "i9mrKqGVZCjzMWzKZljr0XaD_SMa",
  "amr": [
    "BasicAuthenticator"
  ],
  "iss": "https://localhost:9443/oauth2/token",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "sid": "60f9ac54-66d6-4efa-80e5-870a44dfb15d"
}

Since the current sub is not the correct value according to the spec, we can change it.
Userinfo response
The sub Claim in the UserInfo Response MUST be verified to exactly match the sub Claim in the ID Token; if they do not match, the UserInfo Response values MUST NOT be used.
Current sample userinfo response
{
   "sub":"admin"
}

This needs to be aligned with the sub in the id token this needs to be changed to user id.
JWT access token
   sub  REQUIRED - as defined in Section 4.1.2 of [RFC7519].  In case of
      access tokens obtained through grants where a resource owner is
      involved, such as the authorization code grant, the value of "sub"
      SHOULD correspond to the subject identifier of the resource owner.
      In case of access tokens obtained through grants where no resource
      owner is involved, such as the client credentials grant, the value
      of "sub" SHOULD correspond to an identifier the authorization
      server uses to indicate the client application.  See Section 5 for
      more details on this scenario.  Also, see Section 6 for a
      discussion about how different choices in assigning "sub" values
      can impact privacy.

Current sample
{
  "sub": "admin",
  "aut": "APPLICATION_USER",
  "aud": "i9mrKqGVZCjzMWzKZljr0XaD_SMa",
  "nbf":<PHONE_NUMBER>,
  "azp": "i9mrKqGVZCjzMWzKZljr0XaD_SMa",
  "scope": "openid",
  "iss": "https://localhost:9443/oauth2/token",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "jti": "b615657e-c778-4204-bcd4-d1000aaf7488"
}

Related to https://github.com/wso2/product-is/issues/8901

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Error when attempting to scaffold Courses

When attempting to scaffold Courses as instructed here, I get the following error in VS:

Using dotnet I get:
Building project ...
Finding the generator 'razorpage'...
Running the generator 'razorpage'...
Attempting to compile the application in memory.
Could not get the reflection type for DbContext : ContosoUniversity.Data.SchoolContext
   at Microsoft.VisualStudio.Web.CodeGeneration.ActionInvoker.<BuildCommandLine>b__6_0()
   at Microsoft.Extensions.CommandLineUtils.CommandLineApplication.Execute(String[] args)
   at Microsoft.VisualStudio.Web.CodeGeneration.ActionInvoker.Execute(String[] args)
   at Microsoft.VisualStudio.Web.CodeGeneration.CodeGenCommand.Execute(String[] args)

Also logged here: https://github.com/aspnet/Scaffolding/issues/1055

Document details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d0026f69-7e5d-0d3e-5586-16ecc0656ff0
Version Independent ID: 3cb264da-2ff1-1256-572f-aa4ebca6e255
Content: Razor Pages with EF Core in ASP.NET Core - Read Related Data - 6 of 8
Content Source: aspnetcore/data/ef-rp/read-related-data.md
Product: aspnet-core
Technology: aspnetcore-data
GitHub Login: @tdykstra
Microsoft Alias: riande

I had forgotten that I had applied a VS analyzer code change suggestion to convert a switch statement to a switch expression in Pages/Students/Index.cshtml.cs. The code exactly as per the tutorials does not have this problem. I have updated aspnet/Scaffolding#1055 accordingly. Closing this issue.


Large Send Offload (LSO)

This LSO topic should be expanded so it is similar to the other topics where they are explained when this would be beneficial and when it would not be e.g. recommendations for different workloads, and trade-offs for use/non-use

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 57d9f2cd-4171-80c3-8e5a-d0cf4218c3e2
Version Independent ID: e33109ce-190e-dbc5-788b-d1dbea209187
Content: High-performance networking
Content Source: WindowsServerDocs/networking/technologies/hpn/hpn-hardware-only-features.md
Product: windows-server
Technology: networking
GitHub Login: @eross-msft
Microsoft Alias: lizross

We hugely appreciate your contribution, feedback, and support to our technical documentation.
We’ve phased out GitHub Issues on this repository as the feedback mechanism for content and have replaced it with a new feedback system. For more information see, Content User Feedback. As part of this rollout, we are migrating your issue where our team will use your comments to improve the documentation experience. Unfortunately, this does mean you won't be able to access this issue after the migration is complete.
All issues will be migrated by Thursday 29th August 2024.
Once again, thank you for your valuable feedback, we hope you’ll continue to contribute using our new feedback system.


Add function for lowercase uniqueId

First wanted to just add arguments to existing UniqueId function, but since go does not support default values I decided to create a new function instead.

Thanks for your contribution! I'm not sure we would accept this as is though given that it is just as easy and straightforward to get lowercase unique id with strings.ToLower(random.UniqueId()) (which is what we do at Gruntwork).
We also have https://github.com/gruntwork-io/gruntwork-cli/blob/master/random/random.go#L18 in gruntwork-io/gruntwork-cli for more complex unique id generation.
I think we are sufficiently covered between these approaches, so will be closing this PR. Please let us know if they don't work for you!

Ah of course, both options makes much more sense. =)
Thanks/sorry for taking your time.


add TelnetHandlerAdapterTest to branch 3.0

What is the purpose of the change
add TelnetHandlerAdapterTest to branch 3.0
see more detail from #8257
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #8267 (1091cda) into 3.0 (e321324) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.0    #8267      +/-   ##
============================================
- Coverage     64.06%   63.91%   -0.15%
+ Complexity      313      312       -1
============================================
  Files          1073     1073
  Lines         45229    45200      -29
  Branches       6816     6776      -40
============================================
- Hits          28974    28889      -85
- Misses        13044    13093      +49
- Partials       3211     3218       +7

Impacted Files
Coverage Δ

...org/apache/dubbo/common/serialize/ObjectInput.java
33.33% <0.00%> (-33.34%)
:arrow_down:

...rg/apache/dubbo/common/serialize/ObjectOutput.java
66.66% <0.00%> (-33.34%)
:arrow_down:

...ng/transport/dispatcher/all/AllChannelHandler.java
75.86% <0.00%> (-20.69%)
:arrow_down:

...ng/transport/dispatcher/WrappedChannelHandler.java
47.82% <0.00%> (-15.22%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
50.00% <0.00%> (-11.54%)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
61.36% <0.00%> (-9.10%)
:arrow_down:

...apache/dubbo/rpc/protocol/dubbo/FutureAdapter.java
36.00% <0.00%> (-8.00%)
:arrow_down:

.../dubbo/rpc/protocol/dubbo/DecodeableRpcResult.java
52.00% <0.00%> (-8.00%)
:arrow_down:

...mmon/threadpool/support/fixed/FixedThreadPool.java
80.00% <0.00%> (-7.50%)
:arrow_down:

.../remoting/transport/netty4/NettyClientHandler.java
61.01% <0.00%> (-6.78%)
:arrow_down:

... and 49 more

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 e321324...1091cda. Read the comment docs.


[Doc] Explain all property values in email adapter Config

Description:
Not fulfilling checklist items:
Help and Documentation - Is necessary documentation provided?
Related task:
Notify subscribers about new API version
Issue:
Only mail.smtp.from , mail.smtp.user, mail.smtp.password property values have descriptions. Add descriptions for other properties as well.
Doc Link:
[1] https://docs.wso2.com/display/AM210/Enabling+Notifications
Suggested Labels:
UX, Improvement
Affected Product Version:
2.1.0

@RakhithaRR , shall we address this issue for 3.0.0, 3.1.0 and 3.2.0. We can add the details in the configure file and point off from the respective page [2]
[1] https://apim.docs.wso2.com/en/latest/reference/config-catalog/
[2] https://apim.docs.wso2.com/en/latest/learn/design-api/api-versioning/enabling-notifications/#enabling-notifications

This issue can be closed as the docs have been updated with the above PRs.


Make shell script safer

and more readable
Please consider running shellcheck in CI

CC @MarkTiedemann please review

I initially added shellcheck to the CI with https://github.com/denoland/deno_install/pull/23. I think it got removed when migrating to Github Actions.
In any case, I already have a PR open for re-adding it, see: https://github.com/denoland/deno_install/pull/121
I think this MR can be closed as a duplicate. @szepeviktor, please feel free to suggest changes to #121.

@MarkTiedemann :) This PR would be my suggestion :)

@szepeviktor Sorry for the rather harsh comments.
Again, feel free to add to https://github.com/denoland/deno_install/pull/121.
There's also a few open issues. If you need help with any of them, please let me know. I'd be happy to support if necessary.
Hope you have a good day. :)


Personal items

no merge conflicts - this will add "personal items" list and also get some things to display on itinerary cards


Database Cleanup

Our database schema leaves a lot to be desired:

[ ] Tables need to be renamed from e.g. YBoardPost to post. We should see to it that models reflect this as well and allow for a prefix applied to them by configuration. Easiest way to achieve this is by eliminating all tableName() methods.

[ ] Normalization. A lot of tables have bogus fields that can be evaluated through relations. e.g. a Post knows to which Thread it belongs. But it also knows its relation to a forum, although said relation is already set through the Thread. This furthermore contains:

[ ] Restoring natural primary keys. A good example is the YBoardVote table: It's consisting of three separately indexed fields, while it should only be a composite primary key over member_id and choice_id;

[ ] using more appropriate field types. A TINYINT(4) isn't exactly a clever choice for a field that is only ever used to contain a boolean value;

[ ] renaming things for a more expressive and consistent nomenclatura. Examples:

forums → boards

topics → threads

choice → poll_option

logtopic → user_has_read

[ ] Consolidate or break up tables. Users and Members got a 1:1 relation, yet they aren't identical. A member is a profile, really. It should also contain many of the fields that are now part of the User table. In general, we should not rely at all on what a User provides as they are meant to be interchangeable between applications. OTOH, there's a lot of fields in the Post table, keeping track of the last change. These should rather go into their own table to allow for change tracking (w/o versioning). The "profile" URLs in the Members table are worthy for an EAV setup, IMHO;

[ ] Write out tables to disk. Global settings and ranks are much more of a configuration thing and should be reached that way: Through the configuration, not the database. Per-user settings are a different story, though;

[ ] eliminate counting and statistics fields. These are a pain and prone to cause all kind of havoc, including but not limited to synchronization issues. There are some performance implications so we should proceed with special care here;

[ ] remove bogus keys, look for missing ones. Also: Remove all indexes that are left prefixes of others. At least in MySQL/MariaDb, these are totally bogus.

I will let you do this ad have my opinion once you fix the matters but here are my few notes:

Tables need to be renamed from e.g. YBoardPost to post. We should see to it that models reflect this as well and allow for a prefix applied to them by configuration. Easiest way to achieve this is by eliminating all tableName() methods.
Can you explain where the bolded part will be retrieved from? Configuration table? And how will we apply it to th model? I mean will it be installation choice or just a fixed prefix?

logtopic → user_has_read
I have problem with this table (I even raised it to Bbii developer). With time and increased members it grows to huge monster. So I think we need better way to track users.

Write out tables to disk. Global settings and ranks are much more of a configuration thing and should be reached that way: Through the configuration, not the database. Per-user settings are a different story, though;
Are we going to have Global settings Table? I'm thinking that we should and then make application component called settings (preferences or whatever) that will help us access them application-wise. As per user settings, am not sure if you have any suggestion on how to store them.

As for User table, I believe letting user have his own login system is a must. That means Some user information pertaining to authentication, have to be decoupled from user profile. That will enable user to implement his own version of login system yet be able to seamlessly integrate into forum. I have no specific design in my head right now and would like to hear what you think of it.

@DaSourcerer what do you think of my comments? I think we need to push this too

I will take over the task of
Tables need to be renamed from e.g. YBoardPost to post. We should see to it that models reflect this as well and allow for a prefix applied to them by configuration. Easiest way to achieve this is by eliminating all tableName() methods.
This will affect Model names as well as table names. So I will make Upgrade migrations
Do you have any inputs @DaSourcerer as to any alternative to this?

I will take on this this week


out node with zigbee2mqtt groups

Hi,
It seems that the out node has problems handling zigbee2mqtt groups.
It spits the following error :
TypeError: label.split is not a function
That doesn't prevent the node to do its work though

Hi 0uro,
Thank you for raising the issue, but I'm pretty sure you are in the wrong GitHub repo since this project does not have an out node. I think the project you are looking for is: https://github.com/andreypopov/node-red-contrib-zigbee2mqtt
The names are quite similar, so this confusion occurs from time to time. Therefore I'm closing this issue for now - feel free to comment if I'm wrong :-)

You're right, I'm sorry


feat: Add log retention setting, use JSON logger and add logs to the dashboard

Issue #, if available:
Description of changes:

Add descriptions to all Lambdas to simplify finding them
Add configurable retention of logs
Use JSON logging (Lambda setting + replace print to use the Logger)
Add the JSON logs to the dashboard
Started to review logs and remove what could be sensitive

Replace the log_event setting so it does not log headers that might contain security related information
Remove pre-signed url from the log for file upload.

Testing
Ran all the integration tests

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

this is a really great improvement


dotnet not fond

the term dotnet is not recognized as the name of a cmdlet, function, script file, or operable program.
Running windows 10

What does that mean?

i think dotnet is not supposed to run as a standalone command on the terminal , is that what you are trying to do ?

Trying to run windows 10 as a 16 bit system to run old pc game on it.

Seems like you might have got it removed from the ENVIRONMENT ; probably by uninstalling something old.. You may also want to try your hands here again ..https://dotnet.microsoft.com/download

Trying to run windows 10 as a 16 bit system to run old pc game on it.

Seems like you might have got it removed from the ENVIRONMENT ; probably by uninstalling something old.. You may also want to try your hands here again ..https://dotnet.microsoft.com/download

the term dotnet is not recognized as the name of a cmdlet, function, script file, or operable program.
Running windows 10

Seems like you might have got it removed from the ENVIRONMENT ; probably by uninstalling something old.. You may also want to try your hands here again ..https://dotnet.microsoft.com/download

Specifically, you have to Repair the Visual C++ 2015 Redistributable (both x86 and x64) via the "Programs and Feature" Window (a.k.a. Add/Remove Programs)

Hi
And thanks
I fix it now what?
On 2020-02-06 13:26, HEMANT KUMAR wrote:

Specifically, you have to Repair the Visual C++ 2015 Redistributable
(both x86 and x64) via the "Programs and Feature" Window (a.k.a.
Add/Remove Programs)
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
Links:
[1]
https://github.com/dotnet/core/issues/4226?email_source=notifications&email_token=AOPGFTOG7XBBRCZMZYKDYKTRBRI4JA5CNFSM4KQRAHGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOELAITNY#issuecomment-583043511
[2]
https://github.com/notifications/unsubscribe-auth/AOPGFTJ2TKPCDYSBMV5DZYTRBRI4JANCNFSM4KQRAHGA

Did you install dotnet already? Did you make sure to open the command line after you installed dotnet in your system?

No response in 11 days. Closing.


Unknown character set

Laravel Version
11.0.6
PHP Version
8.3.4
Database Driver & Version
MariaDB-1:11.3.2
Description
I created a fresh new laravel project and I'm trying to run php artisan migrate.
This gives me the following error:
Syntax error or access violation: 1115 Unknown character set: 'utf8mb4¾' (Connection: mariadb, SQL: select table_name as name, (data_length + index_length) as size, table_comment as comment, engine as engine, table_collation as collation from information_schema.tables where table_schema = 'xxx' and table_type in ('BASE TABLE', 'SYSTEM VERSIONED') order by table_name)
This is in my ENV:
DB_CONNECTION=mariadb
DB_HOST=localhost
DB_PORT=3306
DB_DATABASE=database
DB_USERNAME=root
DB_PASSWORD=root
Steps To Reproduce
Create a new project and run php artisan migrate

Please provide very detailed steps to reproduce this. Exact commands and choices made when running install commands. Thanks!

 Has nothing to do with Laravel.
I had sqlite disabled in my php, this threw an error while running
composer create-project laravel/laravel:^11.0 example-app
Enabled it and all the problems are gone


DocFX

DocFX will be the new doc generator. However, this does not work with Core projects (yet)
https://dotnet.github.io/docfx/tutorial/docfx_getting_started.html#3-use-docfx-integrated-with-visual-studio

https://github.com/icsharpcode/SharpZipLib/commit/b2948ab646ea0675a15bd413c84571ce696eb8a6
https://github.com/dotnet/docfx/issues/1752

First version is online https://icsharpcode.github.io/SharpZipLib/help/

Is there anything that needs doing with this now?

From my point of view - no.

Since #542 this is now built and updated on every release. Closing this.


Time picker does not update internal value after manual change and clicking outside input

https://jsfiddle.net/tj2Lo4z7/
Your Environment

flatpickr version used: 4.6.9
Browser name and version: Chrome 95 / iOS Safari / Android Chrome
OS and version: MacOS / Windows / Android / iOS

Issue
When manually changing the time value in the input field, instead of the time pickr element and clicking outside the input field instead of using Tab, the value will be reverted.
Element used
Using a pure time picker
flatpickr("#{$id}", {
    "locale": "nl",
    "allowInput": true,
    "dateFormat": "H:i",
    "enableTime": true,
    "noCalendar": true,
    "time_24hr": true
});

Steps to reproduce
Works:

Click input field
Change time by typing in input field (not pickr element)
Press Tab
Time input field stays changed and timepicker will show adapted time when opening it again

Does not work:

Click input field
Change time by typing in input field (not pickr element)
Click or touch (on touchscreen/phone) outside input and pickr element
Time input field reverts to value before manual change and timepicker will show original time when opening it again

Weird situation
A pure datepicker does not appear to have this behaviour.
flatpickr("#{$id}", {
    "locale": "nl",
    "allowInput": true,
    "altFormat": "d-m-Y",
    "altInput": true
});

With pure dates it works well with either clicking outside input field or using Tab. So it's related to the case that time enableTime is used
Human Workaround
On Windows / MacOS: Press Tab (but annoying to
On Android: None possible as you automatically touch outside element to move
Technical workaround
Still unknown

Related #2531

Any news on this?

This bug was introduced in https://github.com/flatpickr/flatpickr/pull/2541 as it works on previous commit. It looks like setDate is calling setHoursFromDate https://github.com/flatpickr/flatpickr/blob/fffc4136d855c525e828e7e16c744489349d035c/src/index.ts#L318 which overrides values for hour/minuteElement that are used in setHoursForInputs https://github.com/flatpickr/flatpickr/blob/fffc4136d855c525e828e7e16c744489349d035c/src/index.ts#L221

Any fix for this?


Feature/nextui v2

Implemented breaking change in upgrading to NextUI V2.
Absolute nightmare and I'm 99% certain i've made a complete clusterfuck of it. Ideally like to get this deployed before we do any further changes such as implementing Bun.
@rsbrowne / @mountclear - Please feel free to make any changes against this by comment or by additional PR to the branch.

yep yep will try to look into this maybe tomorrow

This will also close issue #56

Will also close #141

let me know what you think?

@mountclear That looks a lot better. Let's get your changes added in.


Support tag, markdown, and compact outputs

This PR adds support for:

Markdown output
Printing unchanged resources
Printing tag changes
Printing compact output

This PR also updates all dependencies to the latest and a go mod tidy run as well.
None of these changes are default, the default behavior is the original behavior, with the exception that we only print a category if it has entries.
One key behavior of the --show-tags switch that I'm not sure how to communicate to end users, is that when there is a tag only change, it explicitly does not show up in "Update" if it is only a tag change and no other change to that resource is made. This is intentional, as the purpose of this is to clean up our terraform plan outputs so that engineers do not miss actual resource changes when our tags change for every resource (we use extremely specific version tagging on every resource for lifecycle, versioning, and security tracking purposes, any repository code change will re-tag all resources within a terraform plan).
If these changes are outside of the scope of this tool, do let me know and I am happy maintain my own fork.
There is a demo.json attached to exercise every aspect of the tool.
New switches:
$ ./tftools summarize --help
Get a summary of terraform/terragrunt output plan (created|updated|destroyed...)

Usage:
  tftools summarize [flags]

Flags:
  -c, --compact          Use compact formatting
  -h, --help             help for summarize
  -m, --markdown         Use markdown formatting
  -t, --show-tags        Show resources with tag changes
  -u, --show-unchanged   Show resources with no changes

Examples:

./tftools summarize --show-tags --show-unchanged --compact <../demo.json
🟢 Create:
+ aws_iam_policy.created
+ aws_iam_policy.tainted
🔄 Update:
~ aws_iam_policy.changed
~ aws_iam_policy.tag_and_change
💣 Destroy:
- aws_iam_policy.destroyed
- aws_iam_policy.tainted
#⃣ Tag/Untag:
# aws_iam_policy.tag_and_change
# aws_iam_policy.tag_only
🔹 Unchanged:
• aws_iam_policy.unchanged
• random_string.version

$ ./tftools summarize --compact <../demo.json
🟢 Create:
+ aws_iam_policy.created
+ aws_iam_policy.tainted
🔄 Update:
~ aws_iam_policy.changed
~ aws_iam_policy.tag_and_change
~ aws_iam_policy.tag_only
💣 Destroy:
- aws_iam_policy.destroyed
- aws_iam_policy.tainted

Markdown output:
$ ./tftools summarize --show-tags --show-unchanged --compact --markdown <../demo.json
🟢 Create:

✅ aws_iam_policy.created
✅ aws_iam_policy.tainted

🔄 Update:

⚠ aws_iam_policy.changed
⚠ aws_iam_policy.tag_and_change

💣 Destroy:

🔴 aws_iam_policy.destroyed
🔴 aws_iam_policy.tainted

#⃣ Tag/Untag:

#⃣ aws_iam_policy.tag_and_change
#⃣ aws_iam_policy.tag_only

🔹 Unchanged:

🔷 aws_iam_policy.unchanged
🔷 random_string.version

Hey @danielewood ! Thank you so much for the PR! Looks very nice the new output :) Let me try the new changes and let's merge it! Why not. I will do it ASAP! See you!


ctr run -t intermittent failures

I am getting intermittent loss of output with ctr run -t. If you run a shell not ls it will just exit.

I can't replicate on a new Fedora box, only on this Ubuntu 17.04 box.

I'm running on a 16.10 Linux deathstar 4.4.0-77-generic #98-Ubuntu SMP Wed Apr 26 08:34:02 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux and cannot repo on master.

This is due to the events.  After some debugging I found that the EXIT event is sometimes  sent before the START causing the waitContainer to return thus exiting.
Here are the events from a "good" session:
TYPE      ID        PID       EXIT_STATUS
CREATE    foobar    9547      0
START     foobar    9547      0
EXIT      foobar    9547      0

Here they are from a "bad" session:
TYPE      ID        PID       EXIT_STATUS
CREATE    foobar    9599      0
EXIT      foobar    9599      0
START     foobar    9599      0

I'm looking in the event service to get a fix.

Actually after some more debugging the events are fine.  The process is actually exiting.

After some debugging I found that the EXIT event is sometimes sent before the START causing the waitContainer to return thus exiting.

I think https://github.com/containerd/containerd/issues/871 would help clients deal with events being emitted out of order.

@ehazlett I actually just got the events in the wrong order, I'll try to see if I can reproduce and figure out why

@mlaventure did you get them out of order with my PR from #879?  I think the ordering is right as the container was actually exiting before since the stdout fd was closed.  With #879 I haven't seen them out of order.

@ehazlett yes, it's kinda hard to reproduce, it's a scheduling issue with the goroutines sending the respecting events in the shim
You PR do indeed help a lot. But there's still the possibility of a tiny race

created https://github.com/containerd/containerd/pull/882 to address that race

Ah ok that makes sense. I'll try your PR as well. Thanks!!
On May 19, 2017 18:32, "Kenfe-Mickaël Laventure"<EMAIL_ADDRESS>wrote:

created #882 https://github.com/containerd/containerd/pull/882 to
address that race
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/containerd/containerd/issues/856#issuecomment-302825960,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP6Igd9CFeq4MCp1U2ZC8CfBU2AR0wsks5r7hiLgaJpZM4Naw9n
.


Refactor the use of PlatformStateContext, NerdletStateContext, and AutoSizer

The current implementation on those components falls outside of our best practice. We need to resolve that.

:tada: This issue has been resolved in version 0.5.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How do I get this section bad links

The links in the How do I get this? section which go to other documentation pages are broken.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 7ed7ade2-b6c8-83db-2d20-e87ee7a3cb5d
Version Independent ID: d0591610-bebd-22c9-aaae-d4c1c3ef2ff7
Content: Overview of the new Microsoft 365 compliance center
Content Source: microsoft-365/compliance/microsoft-365-compliance-center.md
Service: o365-seccomp
GitHub Login: @robmazz
Microsoft Alias: robmazz

@bryceroney  - Thank you for submitting feedback.
I will get this issue over to the Microsoft 365 writing team for investigation.
Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@bryceroney  - thanks for the feedback, publishing the link fixes now.


⚠ ClamAV has degraded performance

In 85fac9f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 76 ms

Resolved: ClamAV performance has improved in 455e842.


Is it possible to realize some stop condition?

It looks like yours solution is fastest in Python ODE tasks.
In solve_ivp() we have events which allow to implement stop condition during ODE process solving.
Is it possible to find/implement something similar in NumbaLSODA?
It could be useful if I dont know real parameter's range of integration but know where it should be stopped during entire integration process.
Thanks.

Nope. The code does not have any build-in event finding. It would be possible to implement, but I think it would be a lot of work. The best way might be to re-write solve_ivp in C++:  https://github.com/scipy/scipy/blob/master/scipy/integrate/_ivp/ivp.py#L156

Not that it means much coming from a random, but I'd love to see support on this. Just food for thought. :+1:


Manual 0.21.4 release

Checklist

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

@conda-forge-admin, please rerender

@ijpulidos another set of eyes on this would be great!


Release 1.11.3

Hi! I need the changes from https://github.com/jhy/jsoup/issues/406 for my project. Do you have a timeline for when you're planning on releasing 1.11.3?

How about... now! :)

yay, thanks @jhy! :)


Fixes ViewAction for TabLayoutActions.getSelectedItem

fixes #21
I couldn't find any existing test case for TabLayout. Let me know if you want me to add them.

Hey @malvinstn. Thanks for the contribution. Could you please add UI test to cover this case?

I've added the test cases.

@malvinstn please rebase on master after #23 will be merged

@malvinstn you can rebase on mater now

@malvinstn thanks for contribution. Assigned to 3.0.4 release


Review date

Description of problem
Often tenders request a cms that assists editors to review their content. Especially when a site has hundreds of pages, it is very easy to forget that you wrote an article that is relevant until last year.
Description of solution
What would solve this problem is to have a checkbox 'remind me' where you can select a user and a date. On that date, an email generated to the email address of that user incorporating the link saying "As per your request we remind you that you might want to check the content of the following page".
The place for this would be the module settings. This way, the option applies to all modules, not just HTML.
Additional context
Maybe a seperate issue but the next thing would be to have an overview with 4 columns:
Page name - Module name - review date - email address, sorted by review date.
The place for this would be the Content part in the Persona bar (IMO)

Tycho,
thank you for the idea, but this would not properly fit to page settings - if you consider blog or article modules, where this would needed to be applied on item level, while others would need this on module level. The proper object to attach this would be the DNN "ContentItem" and a new module interface but this would require module developers to implement this interface within their modules

I can imagine that technically, you'd want it that way. As long as I am able to manage the date and the email address on a spot that makes sense to me as a content manager, I'm a happy camper :-)

Hey guys, I am trying to cleanup the backlog and move discussions into the forums and keep only actionable issues that we can assign some foreseeable milestone here on github. So the quesiton is: Is anyone planning on developing this idea, if so what would be a reasonable delivery milestone (Dnn 10, Dnn 11) ? If nobody plans on actually implementing this, then I would suggest moving this to the forums to keep the backlog cleaner.

@sleupold is there a way to move this forward? I mean, I can relate to the remark it should be on content item. Could it be an option to have a hook on contentItem level and the UI as described in the HTML module?

If it helps to make this more actionable, I created a mockup / screen on :
https://xd.adobe.com/view/ddfbe213-ae39-4eaf-5c73-26e877cd4267-abca/
It can be done in 2 fields in the advanced settings of a module.

I am kind of leaning on the same opinion as @sleupold where this should be left to the actual module implementation and not a general setting enough of the overall Dnn module settings since the scope of this could vary widely by module purpose...

So, can we scope it to the html module?
Outlook voor iOShttps://aka.ms/o0ukef downloaden

Van: Daniel Valadas<EMAIL_ADDRESS>Verzonden: Thursday, October 17, 2019 9:56:33 PM
Aan: dnnsoftware/Dnn.Platform<EMAIL_ADDRESS>CC: Tycho de Waard<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [dnnsoftware/Dnn.Platform] Review date (#2670)
I am kind of leaning on the same opinion as @sleupoldhttps://github.com/sleupold where this should be left to the actual module implementation and not a general setting enough of the overall Dnn module settings since the scope of this could vary widely by module purpose...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dnnsoftware/Dnn.Platform/issues/2670?email_source=notifications&email_token=ACFLXHPVONCEXNRFAQFI5KLQPC7PDA5CNFSM4HDZL6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBRKPOI#issuecomment-543336377, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACFLXHNORMYT2EW3NJOPTSLQPC7PDANCNFSM4HDZL6MA.

Yes, I would see that more appropriate IMO.

there are plans to improve workflow within DNN platform.
However, IMO, the benefit of a workflow is much higher for a list of contributions (like Blog, Announcements, News) than a singleton like HML

The common usecase I have in mind is regular content on sites of insitutions. Their content needs to be reviewed periodically. If your are responsible for the correctness of content this is a musthave. Yes, it would be great inarticles etc as well but that would be for other repos and vendors. Html module is a great first step
Outlook voor iOShttps://aka.ms/o0ukef downloaden

Van: Sebastian Leupold<EMAIL_ADDRESS>Verzonden: Friday, October 18, 2019 12:08:16 AM
Aan: dnnsoftware/Dnn.Platform<EMAIL_ADDRESS>CC: Tycho de Waard<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Onderwerp: Re: [dnnsoftware/Dnn.Platform] Review date (#2670)
there are plans to improve workflow within DNN platform.
However, IMO, the benefit of a workflow is much higher for a list of contributions (like Blog, Announcements, News) than a singleton like HML
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/dnnsoftware/Dnn.Platform/issues/2670?email_source=notifications&email_token=ACFLXHKZFCSYADJX7UGK7JLQPDO5BA5CNFSM4HDZL6MKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBRVUAI#issuecomment-543382017, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACFLXHJWTVCFS5JTTDDIN4TQPDO5BANCNFSM4HDZL6MA.

Soooo, do we keep this issue because that would be part of the HTML module or we close this in favor of another issue at https://github.com/DNN-Connect/CKEditorProvider because it would fall under the provider ?

I don't think it would be a CKeditor issue.
If I combine my wish and the feedback from @sleupold, I would envision a generic solution. In that respect it would be functionality that can or can not be utilized by a module.
Also important, this way it can be utilized by vendors as well.
On order to get this 1 step further: would it be on the level of 'my work' or somewhere in the settingstab?

Ok, I am gonna put this a as new feature issue then.


v2.3.0

Adding bulk POST endpoint for decodeScript

Pull Request Test Coverage Report for Build 1703

0 of 0   changed or added relevant lines in 0 files are covered.
91 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.05%) to 76.005%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/rawtransactions.js
30
83.43%

dist/routes/v2/slp.js
61
54.25%

Totals

Change from base Build 1689:
0.05%

Covered Lines:
1766

Relevant Lines:
2172

💛  - Coveralls

:tada: This PR is included in version 1.14.7 :tada:
Your semantic-release bot :package::rocket:


CASMCMS-7995: Fix the Artifactory upload paths

Summary and Scope
In v2 of the jfrog CLI, the default value for the --flat option changed from true to false. If --flat is true, then files are uploaded to the exact target path and their location in the source file system is ignored. If --flat is false, then files are uploaded to a target that includes their parent directories on the source file system.
This was resulting in Python modules and helm charts being uploaded into an incorrect location. For example, Python module archives and wheels were uploaded to a dist subdirectory in the pip repository and thus could not be found by pip.
Fix this problem by specifying --flat true on jfrog rt upload calls to keep the prior behavior.
Issues and Related PRs

Resolves CASMCMS-7995

Testing
Tested on:

GitHub Actions
Local python package install

Test description:
Examined artifactory to see that helm charts and python modules get published in the expected locations.  Checked that I could install the new version of the python module with pip.
Risks and Mitigations
No known risks. Fixes a build publishing problem.
Pull Request Checklist

[x] Target branch correct
[x] Testing is appropriate and complete, if applicable
[x] Is a new version being released? Update https://github.com/Cray-HPE/cray-product-catalog/wiki/CSM-Compatibility-Matrix
[x] HPC Product Announcement prepared, if applicable

Oops, needs a changelog entry. Adding that now.

Here is the python module in its correct location (note that it's not longer in a dist subdirectory): https://artifactory.algol60.net/ui/repos/tree/General/csm-python-modules%2Fsimple%2Fcray-product-catalog%2Fcray-product-catalog-1.6.0.dev64.tar.gz
Here is the helm chart in its correct location (note that it's no longer in a charts/.packaged subdirectory): https://artifactory.algol60.net/ui/repos/tree/General/csm-python-modules%2Fsimple%2Fcray-product-catalog%2Fcray-product-catalog-1.6.0.dev64.tar.gz

Here's the output of my test showing that the python module can now be installed from the location where it's published in artifactory:
C02ZH00ALVCJ: ~$ pyenv virtualenv 3.8.8 CASMCMS-7995
Looking in links: /var/folders/9d/70zmtx7s0j3cjk_b0_4t8wt4000ycc/T/tmpjle2z5op
Requirement already satisfied: setuptools in /Users/haasken/.pyenv/versions/3.8.8/envs/CASMCMS-7995/lib/python3.8/site-packages (49.2.1)
Requirement already satisfied: pip in /Users/haasken/.pyenv/versions/3.8.8/envs/CASMCMS-7995/lib/python3.8/site-packages (20.2.3)
C02ZH00ALVCJ: ~$ pyenv shell CASMCMS-7995
(CASMCMS-7995) C02ZH00ALVCJ: ~$ pip list
Package    Version
---------- -------
pip        20.2.3
setuptools 49.2.1
(CASMCMS-7995) C02ZH00ALVCJ: ~$ PIP_EXTRA_INDEX_URL=https://artifactory.algol60.net/artifactory/csm-python-modules/simple/ pip install cray_product_catalog==1.6.0.dev64
Looking in indexes: https://pypi.org/simple, https://artifactory.algol60.net/artifactory/csm-python-modules/simple/
Collecting cray_product_catalog==1.6.0.dev64
  Downloading https://artifactory.algol60.net/artifactory/csm-python-modules/simple/cray-product-catalog/cray_product_catalog-1.6.0.dev64-py3-none-any.whl (15 kB)
...
Installing collected packages: pyyaml, pyasn1, pyasn1-modules, cachetools, six, rsa, google-auth, oauthlib, charset-normalizer, certifi, urllib3, idna, requests, requests-oauthlib, websocket-client, python-dateutil, kubernetes, attrs, pyrsistent, zipp, importlib-resources, jsonschema, cray-product-catalog
Successfully installed attrs-21.4.0 cachetools-5.0.0 certifi-2021.10.8 charset-normalizer-2.0.12 cray-product-catalog-1.6.0.dev64 google-auth-2.6.6 idna-3.3 importlib-resources-5.7.1 jsonschema-4.4.0 kubernetes-23.3.0 oauthlib-3.2.0 pyasn1-0.4.8 pyasn1-modules-0.2.8 pyrsistent-0.18.1 python-dateutil-2.8.2 pyyaml-6.0 requests-2.27.1 requests-oauthlib-1.3.1 rsa-4.8 six-1.16.0 urllib3-1.26.9 websocket-client-1.3.2 zipp-3.8.0
(CASMCMS-7995) C02ZH00ALVCJ: ~$ pip show cray_product_catalog
Name: cray-product-catalog
Version: 1.6.0.dev64
Summary: Cray Product Catalog
Home-page: https://github.com/Cray-HPE/cray-product-catalog
Author: Hewlett Packard Enterprise Development LP
Author-email: None
License: MIT
Location: /Users/haasken/.pyenv/versions/3.8.8/envs/CASMCMS-7995/lib/python3.8/site-packages
Requires: jsonschema, pyyaml, kubernetes
Required-by:


[FLINK-28919][table] Add built-in generate_series function.

What is the purpose of the change
Add built-in generate_series function.

Thanks for contribution
It would be great to have filled PR template
i also left some comments


[core] Can only load shaders from file

since LoadShaderCode has been moved into rlgl there is no way to load a shader except from a file
while rlLoadShaderCode can be used this only passes back an id, and you then have to manually add in the "default" shader locations, which is a bunch of needless boiler plate and code that should only be in one place (in case it later changes)
putting back an ability to load from memory shader code, also means the code can be loaded in any manner and you have the opportunity to pre-process the shader code (for example you might want to add debug code to a shader temporarily, without needing to hack your shaders)
also means you have to include "config.h" for some defines, which doesn't seem right!

@chriscamacho Good point! Added function!

thx :+1:


Fix plain Postgres synchronous replication mode

Since synchronous_mode was introduced to Patroni, the plain Postgres synchronous replication has been no longer working.
The issue occurs because process_sync_replication always resets the value of synchronous_standby_names in Postgres when synchronous_mode is disabled in Patroni.
This commit fixes that issue by setting the value of synchronous_standby_names as configured by the user, if that is the case, when synchronous_mode is disabled.
Closes: #3093
References: PAT-254.

I'm looking at the failing unit tests.
I opened this as a draft more to get your thoughts about the implementation path.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.862%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
13765

Relevant Lines:
13784

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

6 of 6 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.862%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
13765

Relevant Lines:
13784

💛  - Coveralls