Support 163cn.tv links

Link: http://163cn.tv/uf2FY2

Do you happen to know how to clean up music.163.com URLs? Is it sufficient to only keep the id parameter, or are there anything else to keep as well?

Thanks! I decided to not touch the URL hash for now and only deal with the tracking parameters for y.music.163.com on mobile.


resource/github_organization_webhook: Avoid spurious diff

Similar to #133 the same bug appears in org webhook, which has pretty much the same schema, so I refactored some of the functionality to reuse it.
Here's a test failing prior to the patch:
TF_ACC=1 go test ./github -v -run=TestAccGithubOrganizationWebhook_ -timeout 120m
=== RUN   TestAccGithubOrganizationWebhook_basic
--- PASS: TestAccGithubOrganizationWebhook_basic (3.22s)
=== RUN   TestAccGithubOrganizationWebhook_secret
--- FAIL: TestAccGithubOrganizationWebhook_secret (1.38s)
	testing.go:518: Step 0 error: After applying this step, the plan was not empty:

		DIFF:

		UPDATE: github_organization_webhook.foo
		  configuration.secret: "********" => "VerySecret"

		STATE:

		github_organization_webhook.foo:
		  ID = 44243925
		  provider = provider.github
		  active = true
		  configuration.% = 4
		  configuration.content_type = json
		  configuration.insecure_ssl = 0
		  configuration.secret = ********
		  configuration.url = https://www.terraform.io/webhook
		  events.# = 1
		  events.2342231791 = pull_request
		  name = web
		  url = https://api.github.com/orgs/terraformtesting/hooks/44243925
FAIL
FAIL	github.com/terraform-providers/terraform-provider-github/github	4.633s

After patch
TF_ACC=1 go test ./github -v -run=TestAccGithubOrganizationWebhook_ -timeout 120m
=== RUN   TestAccGithubOrganizationWebhook_basic
--- PASS: TestAccGithubOrganizationWebhook_basic (2.23s)
=== RUN   TestAccGithubOrganizationWebhook_secret
--- PASS: TestAccGithubOrganizationWebhook_secret (1.21s)
PASS

@appilon Moved the schema into its own file, PTAL.


terminated node blocks node-selector scans upon initial scan

What steps did you take and what happened:

adjusted deploy/helm/values.yaml with tolerations, longer timeouts (20min)
installed helm in AWS cluster (~35 nodes) via helm upgrade --install trivy-operator ./deploy/helm   --namespace trivy-system   --create-namespace
intial scans start
1 node selector job failed as unschedulable (believed to be the case as the scheduled node was terminated in the time between installing the operator and working through all other nodes first)
as only 1 concurrent node-selector job, was configured, the whole scan could not complete as it was blocked

What did you expect to happen:
trivy should terminate the node-selector job if the node does not exist anymore. It seems like it creates a list of nodes upon startup and just works the list off independent on whether node still exists.
Anything else you would like to add:
Please take the issue with a grain of sault as I resolved it by re-installing and could not fully confirm the behavior.
[Miscellaneous information that will assist in solving the issue.]
Environment:

Trivy-Operator version (use trivy-operator version): 0.16.3
Kubernetes version (use kubectl version): 1.24.16-eks
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc):  Linux <IP_ADDRESS>-microsoft-standard-WSL2

@xopham I was unable to reproduce it , can this issue be closed?  is it resolved after re-install (following to your comment)

@chen-keinan fine with me. Think it can be handled if it pops up again by choosing tolerations accordingly


Spelling and dead links

Fixed spelling and deadlinks. Did not add content.

Unclear if BS+MS link is bad or just seasonal.
http://learningtechlab.pitt.edu/ may be broken.

I've noted various pages needing updates.

@agrattan0820 Wiki is getting some love before school starts ❤
Would you mind merging for me? It looks like I'm not authorized.

@NathanBarta I changed the permissions for the "pittcsc/officers" group to have Admin permissions (which you're a part of) for this repo, let me know if that allows you to merge or not

@agrattan0820 appears so, thanks!


Peripheral pins defined in BoardConfig.h are not getting ASSIGNED_TO_ADDON on a fresh board

Prerequisites
Please check the following before posting an issue / bug report.

[x] I am running the latest version of the firmware found HERE
[x] I have checked the documentation HERE and found no answer
[x] I checked to make sure that this issue has not already been filed HERE
[x] I am reporting the issue to the correct repository

Context

Firmware Version: 0.7.6
Name of device: any
Link to where you purchased the device: n/a
Is this a custom build?: no
Operating System: n/a
Browser: n/a

Expected Behavior
"Peripheral Mapping" pins defined in BoardConfig.h can't be used in the "Pin Mapping" screen using the Web Config.
Current Behavior
On a freshly flashed board with a display and/or a USB Host defined in the BoardConfig.h those pins are not blocked in the "Pin Mapping" screen and can be assigned to a button.
Saving on the "Peripheral Mapping" screen returns to the expected behavior.
Steps to Reproduce

Nuke-flash a board using firmware with display and/or USB Host defined in the BoardConfig (for example RP2040ABB or Haute42).
Go to "Pin Mapping" in the Web Configurator.
Observe that pins assigned to the display or USB Host in BoardConfig.h are not blocked.
Go to "Peripheral Mapping".
Click "Save".
Go back to the "Pin Mapping" screen.
Observe that pins are properly displayed as "Assigned to addon".

Hi @jakubkoziol ,
This has already been addressed and is fixed in main.  You will need to either compile from there or wait for our v0.7.7 release which should be coming soon.

@TheTrainGoes
Compiled as of commit bcf73abc44cc4a573934e23750d2a2bc47f2e7fa I'm still seeing this behavior.


Exercises: Initial Idea List

The Manual is being revised to facilitate workshops, part of which requires the addition of exercises to the Template, which is used by students during the workshop. PR 46 in the Manual contains a list of ideas, which can be added here when the workshop is completed.
Issues can be used to track ideas for new exercises to create. Label exercises is created to help filter Issues.

Adding ideas here for exercises for local build:

a getting started exercise that concisely indicates the preparation steps and where to find them. should be able to complete independently, as this page could be sent as preparation material for a workshop
include one for the missing pages in the toc that requires a refresh. A second one for when the book needs to be cleaned and rebuilt (e.g., add a bunch of title: xxx to existing pages in toc.
using teachbooks: build and serve

Freek's exercises, along with a cheat sheet: https://tud-seed.github.io/betasteunpunt/main/JB/Online_werken.html
Other ideas for exercises:

markdown crash course (key things in one page - e.g., use the things for freek's cheat sheet). could also have them activate a page in the toc that includes a bunch of syntax errors that break the markup, but not the build
notebook crash course
cell tag and download link replacer
debugging challenges: we set up things that break (toc problems, incorrect file names, missing/extra subdirectories in path, etc. need some that break book build, and other that only have missing pages.

Can we leave open? I imagined this would collect things post workshop too

And added ideas for exercises as separate issues: https://github.com/TeachBooks/template/labels/exercises


YTC just not working? (Firefox)

I'd been using YTC for about a month, now, but about 2 days ago it just stopped working. I tried uninstalling then reinstalling, both the version on the mozilla addon site and on this github site, and neither has worked. I've tried disabling my other addons, and that didn't help either. There's no gear to change settings where it should be beside my Youtube username, and when I try to go from my addon manager to the settings, it just loads a regular Youtube page. Is there anyone who knows how to fix this, or even what might be wrong?

To start, completely remove all versions (addon and/or userscript) currently installed in Firefox, clear the cache, then close and reopen the browser. Additionally, you can create a new profile in Firefox. The goal in this step is to make sure that no part of YTC is present in your profile before re-installing. Having multiple copies of the script installed can cause problems.
Then, install the current developer build, either the userscript or the addon. Make sure you install only one. If the configuration menu still does not appear, then there is an issue specific to your build of Firefox.

It's still not there, so I guess it's Firefox.

I am currently using Firefox v34.0.5 with both userscript and addon latest YTC developer version 461, and it is working correctly.
What Firefox version are you using?

34.0.5

Did you try installing the developer version?
https://github.com/YePpHa/YouTubeCenter/wiki/Developer-Version

The developer version seems to be working for me. Still no idea what happened to the other version. I'm going to put it down to some kind of script on my end that still messed things up despite my disabling it. I suppose I'll stick with the developer version and pray.

It's probably because you were using the outdated v2.1.0, I just checked that version and the settings' button wouldn't show up for me either.
Glad to see that ErdeFB managed to help you out. If there isn't anything else then could you close the issue? It's the "Close issue" button to the left of the green button below the reply box. The less issues are open, the less work the addon author will have.

I was using the 2.1.7 version. I tried with the version on the Mozilla Addon site, which I believe was the 2.1.0 version, but I was originally using the one on the YTC Github. The issue wasn't really solved, I guess, but I'll close it anyway.


Exception calling "ProcessRecord" with "1" argument(s)

Hello,
I followed the instructions and installed using the command
Install-Module -Name LoopbackAdapter -MinimumVersion <IP_ADDRESS>

I then tried running the example script
Import-Module -Name LoopbackAdapter
New-LoopbackAdapter -Name 'LoopbackAdapter1'
New-LoopbackAdapter -Name 'LoopbackAdapter2'

I then get this error in PowerShell ISE:
Exception calling "ProcessRecord" with "1" argument(s): "The value Ignore is not supported for an ActionPreference variable. The provide
d value should be used only as a value for a preference parameter, and has been replaced by the default value. For more information, see
 the Help topic, "about_Preference_Variables.""
At line:165 char:5
+     $__cmdletization_objectModelWrapper.ProcessRecord($__cmdletizatio ...
+     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [], ParentContainsErrorRecordException
    + FullyQualifiedErrorId : NotSupportedException

Hi @Ruslebiffen - I gave this a try in ISE on Windows 10 build 16299 and it did work as expected:

What version of Windows is this on? Does it work in a normal PowerShell console (not ISE)? There are some different behaviors in ISE than a normal console.

Hi, thanks for the reply.
I've got the identital versions as yourself.
I think there might be an issue with WMI on my computer, which in turns results in an error when calling Get-NetAdapter, so I guess the fault lies here, and not in the LoopbackAdapter script.


[Event Request] CU99000834 Purch. Line Reserve

Hello,
can we please add
`
[IntegrationEvent(false, false)]
local procedure OnTransferToPurchLineOnBeforeCheckEntries(var OldPurchLine: record "Purchase Line"; var NewPurchLine: Record "Purchase Line")
begin
end;
procedure TransferPurchLineToPurchLine(var OldPurchLine: Record "Purchase Line"; var NewPurchLine: Record "Purchase Line"; TransferQty: Decimal)
var
OldReservEntry: Record "Reservation Entry";
ReservStatus: Enum "Reservation Status";
begin
if not FindReservEntry(OldPurchLine, OldReservEntry) then
exit;
    OldReservEntry.Lock;

    NewPurchLine.TestItemFields(OldPurchLine."No.", OldPurchLine."Variant Code", OldPurchLine."Location Code");

    for ReservStatus := ReservStatus::Reservation to ReservStatus::Prospect do begin
        if TransferQty = 0 then
            exit;
        OldReservEntry.SetRange("Reservation Status", ReservStatus);
        // --add
        OnTransferToPurchLineOnBeforeCheckEntries(OldPurchLine, NewPurchLine);
        // -- add
        if OldReservEntry.FindSet then

`
This will allow to perform some custom validation before transfer.
Thanks,
Bartek

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


cp-connector: flush buffered bytes when shutting down nats event source

It is possible that clients using the cp-connector lib will miss to send out pending messages to the event broker, e.g. when shutting down. The nats event source implementation of cp-connector already provides a Stop() method that can be used to disconnect from the event broker, however before doing it, it should make sure that pending messages are flushed before closing the connection. (see nats Flush()/FlushWithContext(). Further eventsource::stop() shall be called when canceling the controlplane component.
Acceptance Criteria
When shutting down the cp-connector library, pending messages are flushed before actually closing the connection.

Resolved by https://github.com/keptn/go-utils/pull/496


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_5bv4r8jaux/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_5bv4r8jaux/clone/recipe/meta.yaml:26: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_5bv4r8jaux/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_5bv4r8jaux/clone/recipe/meta.yaml:38: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_5bv4r8jaux/clone/recipe/meta.yaml:38: missing_dev_url: The recipe is missing a dev_url
WARNING: /tmp/abs_5bv4r8jaux/clone/recipe/meta.yaml:38: missing_description: The recipe is missing a description
ERROR: /tmp/abs_5bv4r8jaux/clone/recipe/meta.yaml:38: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Add additional logging to the Send step

User Story
As a developer, I would like to analyze the poor performance of the send step to come up with some mitigation strategies
Description/Use Case
The Send step is currently very slow (on average each invocation takes 60-120 seconds) and is by far the slowest part of the system.  We would like to add better logging/metrics to begin to diagnose it
Risks/Impacts/Considerations

Dev Notes

This might be something that can get plugged in; either the Jar or itself or micrometer on its own

Acceptance Criteria

[ ] Queries against events/metrics/logs can be written in azure to show a breakdown of duration by receiver, transport type, number of items and type of report (HL7 or FHIR)
[ ] A follow up ticket is written to analyze the queries after they have run in production

@brandonnava to discuss it at prod sync


feat: Deploy dashboard-share-settings

What this PR does / Why we need it:
Allows dashboard-share-settings to be deployed
Special notes for your reviewer:
Currently, this only has a proof-of-concept implementation
Does this PR introduce a user-facing change?
Yes, dashboard-share-settings can be deployed

@arthurpitman can you also put a dashboard share settings config in the integration test resources?
https://github.com/Dynatrace/dynatrace-configuration-as-code/tree/main/cmd/monaco/integrationtest/v2/test-resources/integration-all-configs/project


Why depreciated?

I get that you depreciated it in favor of the tool Microsoft build (for their own Linux VM's). But do you might know of their solution can also be installed on other VM's not hosted in Azure?
Because that would be a great advantage you have over their solution.

Check out this project - https://github.com/CyberNinjas/pam_aad I think it does way better job than this implementation - it supports user creation, integration with OpenVPN and much more.


feat: gnosis support, part 2

Changes included in this PR:

Gnosis asset listing
transaction history

Asset listing:

Not sure about the lack of receive address, looking into it

Also USDC balance missing:


Captcha on YouTube

Checklist

[* ] I'm reporting a feature request
[ *] I've verified that I'm running youtube-dl version 2019.09.28
[ *] I've searched the bugtracker for similar feature requests including closed ones

Description
Hello,
Since a few days, I can no longer download youtube videos with youtube-dl because a captcha is requested by the site. I tried to fill it with my web browser, it works, but in command line, always the same problem (however I have the same IP).
Do you have a solution, tutorial on how I could bypass the captcha without change ip or proxy? I am currently blocked.
I saw on another post that it was possible to do this with a cookie file, but there are no details and it dates back to several years ago...
Thank you.
Curl: https://pastebin.com/Jj5xXSqd

Sorry for the interruption. We have been receiving a large volume of requests from your network.

To continue with your YouTube experience, please fill out the form below.

<form action="/das_captcha" method="POST">

<input type="hidden" name="next" value="/watch?v=XTO4Q8bebJ4">
<input type="hidden" name="session_token" value="QUFFLUhqa0pSSElYd1RlNW9xSVViSF9mUWZ2cmFjU0dhd3xBQ3Jtc0tuczB5bEoxeEtxdkRSTmZZa2lYOGI4Vk1oOHNIYXJHQmhIdTVPdE9Zcm5zT3VNMWUxV1BuenhjYXd0R1p6X1RlamhEMXhvSWxWZ2p6bnF1c1VkUjdQZGpDYUhmb3hXdm5TZld5MThEYUVITGY1T1ZNc2NhTUVhdlVuX0lqVVY2NVM1dkhrZjZIQXJVN0VFeE5LQlAzMVJOWWFFWUE=">
<input type="hidden" name="action_recaptcha_verify2" value="1">
<input type="submit" value="Submit">
</form>

Log:
youtube-dl https://www.youtube.com/watch?v=XTO4Q8bebJ4 --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'https://www.youtube.com/watch?v=XTO4Q8bebJ4', u'--verbose']
WARNING: Assuming --restrict-filenames since file system encoding cannot encode all characters. Set the LC_ALL environment variable to fix this.
[debug] Encodings: locale ANSI_X3.4-1968, fs ANSI_X3.4-1968, out ANSI_X3.4-1968, pref ANSI_X3.4-1968
[debug] youtube-dl version 2019.09.28
[debug] Python version 2.7.13 (CPython) - Linux-4.9.0-9-amd64-x86_64-with-debian-9.11
[debug] exe versions: ffmpeg 4.1.2, ffprobe 4.1.2, phantomjs 2.1.1
[debug] Proxy map: {}
[youtube] XTO4Q8bebJ4: Downloading webpage
[youtube] XTO4Q8bebJ4: Downloading video info webpage
WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests
WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests
WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests
WARNING: unable to download video info webpage: HTTP Error 429: Too Many Requests
ERROR: XTO4Q8bebJ4: YouTube said: Unable to extract video data
Traceback (most recent call last):
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 796, in extract_info
ie_result = ie.extract(url)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 530, in extract
ie_result = self._real_extract(url)
File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/youtube.py", line 1836, in _real_extract
'YouTube said: %s' % unavailable_message, expected=True, video_id=video_id)
ExtractorError: XTO4Q8bebJ4: YouTube said: Unable to extract video data

Pass cookies with solved captcha.

Pass cookies with solved captcha.

Hello, thank you for your message. Do you have any idea how to retrieve the cookie with the captcha? If the cookie is retrieved from another ip address, will it work? Because for downloading youtube video, I use ip addresses of different outgoing ip (basically to avoid this blockage).
Thank you


Add extra sidecar containers config for Keda operator

Add configuration parameters for specifying extra sidecar containers and extra init containers. Implements #607
Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)
[x] README is updated with new configuration values (if applicable) learn more
[x] A PR is opened to update KEDA core (repo) (if applicable, ie. when deployment manifests are modified)

Fixes #

@JorTurFer Sure, I've added it to the test run yaml

Thanks for your contribution!


db-demo-webtest

I hoped you will supply 2 commits: one for the empty project .... one for the real test ... in one Pull request.

if the build is happy, I'll be glad too


Could preserveCaps add {} in front of Caps in export?

I have selected the "All words with capitals" under "Preserve capitalization", I found that the export title, while preserve the Cap in the tex file, without the {} BiberLaTex in some style (e.g. APA) would turn that into sentence cases. I tried very hard to look at the code in Better BibLaTeX.js and attempt to figure this out myself but I am not really a programming person so I failed to make such changes myself.... I only figure out it is something related to preserveCap :(
Could this be done?
Thank you very much.

Can you give me an example of what you get and what you want it to look
like?

Oh and you really don't want to look at Better BibLaTeX.js, it's generated from a coffeescript source, although the caps preserving is done in the generic translator scaffolding

Thanks a lot! I have actually modified all the titles in my zotero database with corrected sentence case over 400 publications, e.g. Children's peer relations: A quantitative review.
e.g. A database for writing English
could the export of title be:
title = {Children's peer relations: {A} quantitative review}
title =  {A database for writing {E}nglish}
(Cause APA style in biblatex force all title to Sentence case and removed all my hard work on corrected sentence case... )
Thanks so much!

I see, cause I saw this: https://github.com/zotero/translators/pull/477/files and thought may be I can change the corresponding lines myself... too naive. :P

Ah, no, that used to be a valid approach. I do try to track the behavior of the Zotero's Bib(La)TeX translator where it makes sense, but the sources have diverged substantially a long time ago.
Functionally, wouldn't declaring \DeclareFieldFormat[article]{apacase}{#1} do what you want?
\documentclass{article}
\usepackage[UKenglish]{babel}
\usepackage[style=apa,backend=biber,natbib=true] {biblatex}
\DeclareLanguageMapping{british}{british-apa}

\DeclareFieldFormat[article]{apacase}{#1}

\usepackage{filecontents}
\begin{filecontents}{\jobname.bib}
@article{A01,
  author = {Author, A.},
  year = {2001},
  title = {The LaTeX companion},
  journaltitle = {Some journ.}
}
\end{filecontents}
\addbibresource{\jobname.bib}

\begin{document}
Lorem ipsum \autocite{A01}?

\printbibliography
\end{document}

OMG :exclamation: that's work perfectly! I am sorry I asked such silly questions and thanks so much for your suggestions. I have read the biberlatex manual and could not figure this out myself. No words can express my gratitude! Thanksx10000!  :bow:

It's not a silly question at all, and it's my pleasure. Don't think that I knew all this arcane stuff from memory -- I had to dig into the biblatex-apa source to see whether it could be done at all. The biblatex manual doesn't mention this, you couldn't have known.
I suspect also that \DeclareFieldFormat{apacase}{\MakeSentenceCase{#1}} would change the titles to sentence case everywhere (which is what it looks like you prefer) without you having to sentence case them manually in Zotero.

You are so nice!
I actually used \DeclareFieldFormat{apacase}{\MakeSentenceCase{#1}} but it did too well a job on sentence case everything, so I will have to add the {} in the export file from Zotero manually...  So glad now this is solved with a simple {#1} !

Again, my pleasure. I'm trying to be a little more conservative these days about what behavior I build into the translators, both for reasons of maintainability and in order not to overwhelm the user with a plethora of options; in hindsight something like fancy urls should have been handled the same. I hope you don't mind me solving it this way.

I totally agree with your programming philosophy! :+1:


Pass specific version to fe_download

It's a good idea for reproducibility of tests to always refer to a specific version here, avoid using master since that can change arbitrarily.

Coverage remained the same at 92.308% when pulling 4365992d241a20d39d52a4928ba351efedb1629e on tkelman:patch-1 into 1ab6342e33d6429280a4671085682a34ea184651 on JuliaFEM:master.


Fix DICS rank handling

DICS handing of the rank parameter was completely broken. It was changed from a single number to something more complicated, but the code was not updated appropriately.
Still needs unit test to cover this case, but here at least is a fix for the code.

I wonder if fixing this would allow us to remove the reduce_rank=True from here:
https://mne.tools/dev/auto_examples/inverse/plot_dics_source_power.html?highlight=dics#compute-source-power-using-dics-beamformer

Spoiler alert: it doesn't. I guess this makes sense since the reduce_rank has to do with the 3-orientations rank as opposed to the n_ch rank / whitening.

The computations have not changed. It's just that setting the rank parameter to anything other than None was broken.

Thanks @larsoner! I really appreciate you picking this up.


Reduce PEP-8 errors

See https://github.com/Senzing/g2-python/issues/123

Pull requests:

https://github.com/Senzing/g2createproject/pull/22


Remove additional DOM elements and CSS file

It would be easier just to add container with image. No additional empty DIVs etc.
Also I thought it could be good to customize blur radius by data attribute

What do you think about extracting dominant color from image if it's unsupported browser and creating shadow based on that color?

That's a good idea, but do you think it will be possible to manipulate shadow-radius in the DOM?

Use the default config would be better, and enlarge the shadow using calc() look like this:
backgroundShadow: '0 25px calc(${blurValue}px + 10px) -10px rgba(${dominantColor}, .7)


SQLiteException: duplicate column name: Id

 void Start()
    {
        // 1. Create a connection to the database.
        // The special ":memory:" in-memory database and
        // URIs like "file:///somefile" are also supported
        var db = new SQLiteConnection($"{Application.persistentDataPath}/MyDb.db");

        // 2. Once you have defined your entity, you can automatically
        // generate tables in your database by calling CreateTable
        db.CreateTable<Player>();

        // 3. You can insert rows in the database using Insert
        // The Insert call fills Id, which is marked with [AutoIncremented]
        var newPlayer = new Player
        {
            Name = "gilzoide",
        };
        Debug.Log("before insert Id=" + newPlayer.Id);
        db.Insert(newPlayer);
        Debug.Log("after insert Id=" + newPlayer.Id);
        Debug.Log($"Player new ID: {newPlayer.Id}");
        // Similar methods exist for Update and Delete.

        // 4.a The most straightforward way to query for data
        // is using the Table method. This can take predicates
        // for constraining via WHERE clauses and/or adding ORDER BY clauses
        var query = db.Table<Player>().Where(p => p.Name.StartsWith("g"));
        foreach (Player player in query)
        {
            Debug.Log($"Found player named {player.Name} with ID {player.Id}");
        }

        // 4.b You can also make queries at a low-level using the Query method
        var players = db.Query<Player>("SELECT * FROM Player WHERE Id = ?", 1);
        foreach (Player player in players)
        {
            Debug.Log($"Player with ID 1 is called {player.Name}");
        }

        // 5. You can perform low-level updates to the database using the Execute
        // method, for example for running PRAGMAs or VACUUM
        db.Execute("VACUUM");
    }

Hello, I found a problem. After building and running the above example on an Android device, starting from the second launch, an exception is thrown: “SQLiteException: duplicate column name: Id.”.
Editor is OK.   v1.1.0

Hey @MrTony1987, thanks for the report.
That's really weird 🤔 How is your "Player" class defined?

Hey @MrTony1987, thanks for the report. That's really weird 🤔 How is your "Player" class defined?

Hello, my code only contains the content in the Start method, with nothing else. You can freely create a MonoBehaviour script that includes this Start method and attach it to a GameObject. Then, print the log to the screen, and run it multiple times on an Android device to check the logs.

Well, you didn't really put the definition of Player in your code in this issue. I'm assuming you are using the same code that is present in this repository's README file.
public class Player
{
    [PrimaryKey, AutoIncrement]
    public int Id { get; set; }
    public string Name { get; set; }
}

I'll try to reproduce on an Android build here.

Also, tip for next time: use "```cs" to highlight code blocks with C# syntax =]

Is there any update on this? I also face the same problem.

No updates from me, at least. In my latest test, I got the "Id" column in the "CREATE TABLE" statement by lowering the managed code stripping level. It's possible that the attributes are being stripped from the builds, we can try to preserve them or something like that (reference documentation here).
Unfortunately I won't have time to look into this for a couple of weeks.

I had the same issue with the iOS platform also.
SQLiteException: duplicate column name: LevelName
  at SQLite.SQLite3.Prepare2 (System.IntPtr db, System.String query) [0x00000] in <00000000000000000000000000000000>:0
  at SQLite.SQLiteCommand.ExecuteNonQuery () [0x00000] in <00000000000000000000000000000000>:0
  at SQLite.SQLiteConnection.Execute (System.String query, System.Object[] args) [0x00000] in <00000000000000000000000000000000>:0
  at SQLite.SQLiteConnection.MigrateTable (SQLite.TableMapping map, System.Collections.Generic.List`1[T] existingCols) [0x00000] in <00000000000000000000000000000000>:0
  at SQLite.SQLiteConnection.CreateTable (System.Type ty, SQLite.CreateFlags createFlags) [0x00000] in <00000000000000000000000000000000>:0
  at GameDataManager.LoadDataModel () [0x00000] in <00000000000000000000000000000000>:0
  at GameDataManager.Awake () [0x00000] in <00000000000000000000000000000000>:0

The table structure like this:
public class WaveDataModel
{
    // Remove the static dbPath and replace it with a method
    private static string dbFileName = "EnemyWave.db";

    public static string GetDbPath()
    {
        string dbPath = Path.Combine(Application.streamingAssetsPath, dbFileName);
        return dbPath;
    }

    public string LevelName { get; set; }

    public int WaveNumber { get; set; }
}

Loading logic:
    private void LoadDataModel()
    {
        // Open a connection to the SQLite database
        using (SQLiteConnection db = new SQLiteConnection(WaveDataModel.GetDbPath()))
        {
            var result = db.CreateTable<WaveDataModel>();
            WaveDataArray = db.Table<WaveDataModel>().ToArray();

            string LevelName = SceneManager.GetActiveScene().name;
            WaveDataArray = WaveDataArray.Where(wave => wave.LevelName == LevelName).ToArray();
            Debug.LogFormat("Load enemy wave done, {0} waves in total.", WaveCount);
        }
    }

Ok, found the culprit. Unity managed code stripping is removing the SQLite.SQLiteConnection.ColumnInfo.Name property setter, which causes SQLite-net to ignore it as a valid and thus always returning null for column names. This affects the ORM in several places.
Adding a [Preserve] or [RequiredMember] attribute to this property fixes the issue. I'll open a PR soon with the fix and release the fixed version, I'll keep you posted.

There it is, this issue was fixed and is available in version 1.1.1
Thanks again for the bug report!
Cheers \o/


Update sources

:robot_face: Updating sources to the latest version.
Commits touching OCaml packages:
Diff URL: https://github.com/NixOS/nixpkgs/compare/4a01ca36d6bfc133bc617e661916a81327c9bbc8...57bfdfae1747fa296400ab0558eb9f8ab5916118

🐑


Openssl asynchronous support

Openssl 1.1.x provides asynchronous support.
Now there are two problems：
1 There are some errors when the program is compiled；
2 How to use asynchronous mode in sslsplit?
I look forward to your reply.Thank you.

Check open issues and if none match, open a new one with the exact compile failure.
What problem are you trying to solve? SSLsplit already uses OpenSSL in an asynchronious way, through libevent2_openssl. There might be benefits of migrating both libevent2_openssl and SSLsplit to the new async OpenSSL API, but I don't expect any user-visible changes that would result in. Closing this issue.


Collect runtime statistics about number of agents

Users can / should specify a different database for the statistics, including on a different host. This is advisable because the statistics tables could become very large, and operations on the statistics database could become expensive in terms of CPU cycles, RAM, and I/O bandwidth, both for the python app and for the database server. There are no references between the normal tables and the statistics tables, and the statistics tables will not be needed for normal operations of pyfarm.

I have not done a deep dive on the code yet, this is in progress after all, but overall the design seems like it would work well.  One question I have though is if you've considered using something like statsd/graphite to handle the stats and graphs?  The approach being taken here is better for a number of reasons I think (nicer display, more control over how we/where the data is displayed, does not require extra services, etc), I just though I might throw the suggestion out there in case you where thinking you'd rather have something more dedicated for logging and rendering statistics.

One question I have though is if you've considered using something like statsd/graphite to handle the stats and graphs?

No, I've never heard of either of those.

They might be worth taking a look at.  Statsd and graphite work well together and, at least in my own experience, are a good solution if you don't know what all the stats you'll ever want are in advance.
With statsd you don't have to predefine the stats you want to log or how they're going to be stored since the application handles both of these cases for you.  With graphite, you also don't have to implement functions to draw the graphs yourself and many of the commonly used graphing functions are all already provided for you.
It does add a bit to the setup but it would probably be more flexible in the end.  There's some docs here on a complete setup if you're at all interested.  It's part of a series and goes into quite a bit more depth than I have here.
In the long run, I think statsd/graphite or something like it would probably help quite a bit when it comes to understanding a farm's overall performance especially around the scheduler.  But I don't know if this is where we should start adding or if you think it's not worth the extra setup steps.  It still wouldn't be a requirement, much like you're starting with here, so smaller farms could do without (API calls won't fail if the server is down).

I've had a look at graphite and statsd, and they really do look interesting.
However, the added complexity that system brings with it seems a bit too much for now. Maybe we can add support for that later – shouldn't be too hard on the pyfarm-side of things – but for now, I'd prefer to merge this statistics branch instead.

Alright, well thanks for taking to time to have a look.  Hopefully we can eventually integrate graphite/statsd eventually but for now I don't have a problem with merging this.


Add geofencing

Add geofencing

As said on the Domoticz forum. I would very much like to see this feature in an upcoming release.


[Fix]: add convert script for Votenet

Fix issue #145 by  converting the keys in the checkpoints of votenet models.

This is still an issue. When I run python tools/test.py configs/votenet/votenet_16x8_sunrgbd-3d-10class.py checkpoints/votenet_16x8_sunrgbd-3d-10class_20200620_230238-4483c0c0.pth --show --show-dir ./data/votenet/show_results, I get the message:
/home/eric/miniconda3/envs/open-mmlab/lib/python3.7/site-packages/mmcv/cnn/bricks/conv_module.py:107: UserWarning: ConvModule has norm and bias at the same time
  warnings.warn('ConvModule has norm and bias at the same time')
The model and loaded state dict do not match exactly

unexpected key in source state_dict: bbox_head.conv_pred.0.conv.weight, bbox_head.conv_pred.0.conv.bias, bbox_head.conv_pred.0.bn.weight, bbox_head.conv_pred.0.bn.bias, bbox_head.conv_pred.0.bn.running_mean, bbox_head.conv_pred.0.bn.running_var, bbox_head.conv_pred.0.bn.num_batches_tracked, bbox_head.conv_pred.1.conv.weight, bbox_head.conv_pred.1.conv.bias, bbox_head.conv_pred.1.bn.weight, bbox_head.conv_pred.1.bn.bias, bbox_head.conv_pred.1.bn.running_mean, bbox_head.conv_pred.1.bn.running_var, bbox_head.conv_pred.1.bn.num_batches_tracked, bbox_head.conv_pred.conv_out.weight, bbox_head.conv_pred.conv_out.bias

missing keys in source state_dict: bbox_head.conv_pred.shared_convs.layer0.conv.weight, bbox_head.conv_pred.shared_convs.layer0.conv.bias, bbox_head.conv_pred.shared_convs.layer0.bn.weight, bbox_head.conv_pred.shared_convs.layer0.bn.bias, bbox_head.conv_pred.shared_convs.layer0.bn.running_mean, bbox_head.conv_pred.shared_convs.layer0.bn.running_var, bbox_head.conv_pred.shared_convs.layer1.conv.weight, bbox_head.conv_pred.shared_convs.layer1.conv.bias, bbox_head.conv_pred.shared_convs.layer1.bn.weight, bbox_head.conv_pred.shared_convs.layer1.bn.bias, bbox_head.conv_pred.shared_convs.layer1.bn.running_mean, bbox_head.conv_pred.shared_convs.layer1.bn.running_var, bbox_head.conv_pred.conv_cls.weight, bbox_head.conv_pred.conv_cls.bias, bbox_head.conv_pred.conv_reg.weight, bbox_head.conv_pred.conv_reg.bias


Tasks documentation

more and more tasks are coming up. Is there any central place where all the tasks will be documented?
So far i have seen some Tasks having a link to furhter help documentation. Is that the way it should go?

We will be adding more scenario based markdown docs (and videos) on the front page of the repro.  Also, tasks will optionally carry a task specific markdown document in each folder.  We started with a handful and we have an event planned to document the rest.


chore: configure stale bot

We are moving from the stale action to Stalebot. It allows for issues to be pinned so they are never stale.
I'll be going through our issues to mark those.
I've changed the messaging and the timings; please review and let me know if you want any changes.

I got an email to install Stale. Let me know what needs to be done.

It's already added, once this pr merges it's active :)

Seems good to me. I presume the syntax is right. Merge when you think its good to go.


Migrate powershell docker scripts to docker@2

Reason for Change:

We are having intermittent failures when running docker powershell script in windows. Migrating to docker@2 is an attempt to fix this.
Even if this does not resolve the issue, still migrating to a native task is an improvement for pipelines
Issue Fixed:

Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests

Notes:

/azp run

/azp run

/azp run

/azp run


Upgrade to 1.13

Hi!
Is there a way to upgrade an existing server (1.12) to the next version (1.13)?

If you left VERSION at the default of LATEST, then the container will automatically download the latest 1.13 version on the next container startup.


feat: add mapTypeId prop

Hi @phamtung1, first I want to thank you for this lib! :+1:
I needed mapTypeId prop in one of my projects so I added it. I never contributed to open-source so I decided that this would be a great start :)
About map types
There are four types of maps available within the Maps JavaScript API. In addition to the familiar "painted" road map tiles, the Maps JavaScript API also supports other maps types.
The following map types are available in the Maps JavaScript API:

roadmap displays the default road map view. This is the default map type.
satellite displays Google Earth satellite images.
hybrid displays a mixture of normal and satellite views.
terrain displays a physical map based on terrain information.

[source]

@nikolicstjepan : Hi, thanks for the PR, sorry because I did not reply sooner. I did not use github much. I will review your PR asap :)

Thanks for your support. Your PR have been merged. I will update the example soon :)

Thanks for your support. Your PR have been merged. I will update the example soon :)


Remove LoadDistrictConfig, refactoring into PerDistrict

Who is this PR for?
developers
What problem does this PR fix?
There's some older per-district config that's in LoadDistrictConfig, this consolidates into into PerDistrict.  This PR doesn't change how individual fields are read from the yaml - they're still not expressed in PerDistrict and so the code doesn't have information about which values are missing because the district doesn't support, or missing in a way that's a bug.

selfie


CI pipeline broken, Maven fails to build project

maven.yml is our CI testing configuration file. A .yml file needs to have a goal. It currently uses a default process to set the location of the pom.xml but fails because there is no pom.xml in that location. Because it cannot locate the pom.xml file it fails to build the project. See the red "x" on the top of the table displaying the repos code and click on any failed build to see details.
The problem has been a real pain to understand. This developer has had the same problem, but the solution proposed is incorrect and already existed in the .yml file I created from the beginning.
Any insight into the problem would be appreciated. I've tried many tweaks and spent hours looking at documentation for GitHub Actions, Maven, YAML files, pom.xml files etc.

Figured out how to give a file path to the "run mvn" command, debugged the file path, and removed an optional dependency caching command from mavel.yml. Now we have successful builds but keeping this issue open until I have JUnit building, and running automated tests. Only then can we really say that we have CI implemented correctly.


sometimes CI failed because harborcluster-sample-harbor-harbor-notarysigner-authority not ready

https://github.com/goharbor/harbor-operator/pull/583/checks?check_run_id=2400534324

kubectl describe harbor-operator

harbor-operator log

kubectl -n cluster-sample-ns get cert -o wide

kubectl -n cluster-sample-ns get certificaterequests -o wide

can't reproduced locally

Same root cause with #533


Appearance of Zotero item type "thesis" in output-html

Hi Michaël,
I believe there is a bug with rendering of bibliographic items of type "Thesis" in the output-html. Please see the TF-C item Sarungga: https://erc-dharma.github.io/tfc-nusantara-epigraphy/workflow-output/html/DHARMA_INSIDENKSarungga.html
Here, we include in the apparatus the edition published in the thesis of Nurul Fadilla Budiana with siglum N:
<bibl n="N">      <ptr target="bib:NurulFadillaBudiana2021_01"/>      <citedRange unit="page">47-50</citedRange>                     </bibl>
The short title NurulFadillaBudiana2021_01 is already present in the Zotero Web Library. However, the reference does not render properly in the output, but only the page numbers appear:

By contrast, the journal article by Goenawan Sambodo et al. appears fine. Is this indeed a problem with the output transformation, and if so, can it be fixed?
Thanks,
Jarrah

Dear Jarrah,
This is due to the way bibliographic entries are currently pulled from Zotero. Entries are looked up by tags, not by short name. Thus, for your reference to work, you need to assign it a tag NurulFadillaBudiana2021_01 in Zotero.
That said, you are right to expect entries to be referenced by their short name. I will address that eventually, but this requires some care.
Best,
Michaël

Thanks Michaël. I have gone ahead and made the change in Zotero. @.***: I have also addded the thesis type, though I haven’t seen this item so please expand/correct as needed, and corrected capitalization of the title.)
Le 30 sept. 2023 à 05:31, Michaël Meyer<EMAIL_ADDRESS>a écrit :
Dear Jarrah,
This is due to the way bibliographic entries are currently pulled from Zotero. Entries are looked up by tags, not by short name. Thus, for your reference to work, you need to assign it a tag NurulFadillaBudiana2021_01 in Zotero.
That said, you are right to expect entries to be referenced by their short name. I will address that eventually, but this requires some care.
Best,
Michaël
—
Reply to this email directly, view it on GitHubhttps://github.com/erc-dharma/project-documentation/issues/229#issuecomment-1741725170, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAGMAEY5UZMVOCPKRFUSJ6DX47RIBANCNFSM6AAAAAA5NKIPDE.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Dear @michaelnmmeyer, thank you for this! I will double check the item @arlogriffiths


Pass values to jailed code

I'm sure I'm missing something obvious, but how do I go about passing an object of key / value pairs to the jailed code?
For example, if I had something like:
{
  firstName: John,
  lastName: Doe
}

How would I go about accessing firstName and lastName within plugin.js? My original guess was that I'd add the object to the 2nd property of jailed.Plugin(pathToPlugin, { scope: { firstName: 'John', lastName: 'Doe' } }) but it didn't work.
I looked at the web-banner example in the repo, but the bad / good image names are hardcoded into plugin.js. I need those values to be dynamic.

Thanks for the example. It looks to be working.
Serializing the data was my first thought, but thought there should be a better solution (such as what you provided).
Thanks again.


#6156 Removed dockerignore. Addded cleaning up after install.

rm: cannot remove '.DS_Store': No such file or directory
rm: cannot remove '.idea/': No such file or directory

i got these errors.  probably best to use -f option, and put all arguments into one cmd.

i confirmed the version is displayed correctly in the container 👍


100% CPU Usage

Gocron is using 100% of one core, 100% of the time.
You don't even have to schedule anything, just calling gocron.Start() will do it.
Any thoughts?

This is related to: https://github.com/jasonlvhit/gocron/issues/3
The fix was merged with my repo but the usage is a bit different.
As it now requires
<- gocron.Start()

Yes, we merged in @Claudiu improvements. That should fix your problem.
go get -u "github.com/jasonlvhit/gocron"

should fix the issue.

Has there been any additional progress on this? I'm still getting 100% CPU usage when using this library unfortunately.

have you pulled the latest version? you could also try the v1 branch.

On Mar 15, 2016, at 2:59 PM, Dan Hardman<EMAIL_ADDRESS>wrote:
Has there been any additional progress on this? I'm still getting 100% CPU usage when using this library unfortunately.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub https://github.com/jasonlvhit/gocron/issues/7#issuecomment-196973594

I believe so. I'll give it a go again to make sure. How do I get the v1 branch?

git checkout v1.0

It was the latest version that I was getting the issue with. Just trying v1 now. Having to change some code around for it

I'm getting error: sch.Start() used as value wherever I use <-sch.Start()
//Monitor checks the heating schedule at 1 minute intervals
func (s *Schedule) Monitor() {
	sch := gocron.NewScheduler()
	sch.Every(1).Minute().Do(s.checkSchedule)
	<-sch.Start()
}

in the v1.0 branch start doesn't block or return anything. I haven't had a chance to update the docs though. From scheduler.go:
// Scheduler keeps a slice of jobs that it executes at a regular interval
type Scheduler interface {
	// Start starts the scheduler
	Start()

	// IsRunning returns true if the job  has started
	IsRunning() bool

	// Stop stops the scheduler from executing jobs
	Stop()

   //  other methods ...
}

Just found out it was likely not gocron causing the 100% CPU usage. I jumped to conclusions as I remembered seeing this issue! My bad. All is good


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2024-06-12T15:32:41+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
53317ea86bdb8799a2c29e261e71906b19bc83e4
link
unknown
/test integ-security-multicluster

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-sigs/prow repository. I understand the commands that are listed here.

/retest


olly trace: ingest custom events

In 5.1 programmers will be able to add other runtime events. This PR modifies olly trace so that it includes custom events of common types (unit, int, span).
Conditional compilation was used to make it work on 5.0.

How did you test this? With 5.1.0~alpha2, opam pin . gives me the following error building the dependent package core:
[ERROR] The compilation of core.v0.16.0 failed at "dune build -p core -j 7".

#=== ERROR while compiling core.v0.16.0 =======================================#
# context     2.1.3 | macos/arm64 | ocaml-base-compiler.5.1.0~alpha2 | https://opam.ocaml.org#7b30d632
# path        ~/.opam/5.1.0~alpha2/.opam-switch/build/core.v0.16.0
# command     ~/.opam/opam-init/hooks/sandbox.sh build dune build -p core -j 7
# exit-code   1
# env-file    ~/.opam/log/core-97555-cfb0d4.env
# output-file ~/.opam/log/core-97555-cfb0d4.out
### output ###
# should be applied to '()'; using '(struct end)' is deprecated.
# [...]
# File "_none_", line 1:
# Warning 73 [generative-application-expects-unit]: A generative functor
# should be applied to '()'; using '(struct end)' is deprecated.
#
# File "_none_", line 1:
# Warning 73 [generative-application-expects-unit]: A generative functor
# should be applied to '()'; using '(struct end)' is deprecated.
#
# File "_none_", line 1:
# Warning 73 [generative-application-expects-unit]: A generative functor
# should be applied to '()'; using '(struct end)' is deprecated.

<><> Error report <><><><><><><><><><><><><><><><><><><><><><><><><><><><><><><>
+- The following actions failed
| - build core v0.16.0
+-
- No changes have been performed

This failure on arm64 and mac is odd -
Uncaught exception:

  (Invalid_argument
   "Float.iround_nearest_exn: argument (nan) is too small or NaN")

FWIW, I was seeing the same error on #16 as well, it wasn't introduced by this PR. I have access to an arm64 machine, will try to reproduce this locally.

@kayceesrk I didn't test the 5.1 alpha so no idea why core is not building there.
My tests were done on a 5.0 compiler with the custom events patch: https://github.com/TheLortex/ocaml/tree/custom-events-5.0-rc1.

@kayceesrk I don't see any errors on 5.1.0~beta1. Maybe we should activate a GH Actions job for 5.1.

Maybe we should activate a GH Actions job for 5.1.

Sounds good to me.

Is this PR good to be merged @TheLortex @Sudha247?


Un-exclude JCK-8 targets with failures not related to setup / infra

This PR un-excludes jck8 targets with failures not related to setup/infra issues. Test issues corresponding to these excludes are listed in the table in backlog/issues/500 . This PR will make sure we are running the maximum set of tests at all times.
Signed-off-by: Mesbah-Alam<EMAIL_ADDRESS>FYI @JasonFengJ9

Sanity check: Grinder/15858

Sanity check: Grinder/15858

Is this the right grinder? It only runs target jck-runtime-api-java_lang-invoke, not those affected by this PR.

Is this the right grinder? It only runs target jck-runtime-api-java_lang-invoke, not those affected by this PR

Yes it is the right grinder. The point of that grinder is to ensure this PR didn't break anything. Those that are affected by this PR will be run in jck.extended builds.

I was hoping to see a grinder with un-excluded tests w/o setup related failures.

I was hoping to see a grinder with un-excluded tests w/o setup related failures.

We can run grinders with targets that this PR un-excludes, but I believe that is not necessarily within the scope of this PR.
This PR simply un-excludes the tests that had been previously excluded to have a 'green' run of JCK8 extended.
These un-excluded targets will then run as part of the extended weekly, and as the next step, we then can focus on fixing any test issues (the table in backlog/issues/500 contains the list of test issues open).

Do we have people actively working on the fix of these test targets? If not, these tests will continue falling in weekly runs which may not be ideal.

Do we have people actively working on the fix of these test targets?

We need some help from infra to fix some potential machine issues like backlog/issues/302.
That said, the original motivation for creating this PR stemmed from a slack discussions I had had with @JasonFengJ9; in short: We need to revisit the excludes done in https://github.com/adoptium/aqa-tests/pull/2487 and un-exclude the targets that had not been excluded for known infra issues like infrastructure/issues/5254 - that are causing test failures across all JDK versions.
As a second step, we need to then either fix the associated test failures (e.g. backlog/issues/302), or try and disable specific sub-tests that cause the failure. Excluding top level targets disables thousands of sub-tests under each target - causing loss of test coverage.

We need to revisit the excludes done in #2487 and un-exclude the targets that had not been excluded for known infra issues like infrastructure/issues/5254 - that are causing test failures across all JDK versions.

Can we start identifying the setup related failures via grinder runs and only un-exclude the green targets? I feel it is better to keep existing weekly tests green such that a regression failure appears in a straightforward way?

I feel it is better to keep existing weekly tests green such that a regression failure appears in a straightforward way?

That was the original reason why https://github.com/adoptium/aqa-tests/pull/2487 was done.
Can we please decide if we want a 'green' build or 'maximum test coverage'? @JasonFengJ9

Can we start identifying the setup related failures

A list of issues have already been identified (ref: 'issue' column in the table in backlog/issues/500). Among those issues, potential setup/test related issues are:
backlog/issues/302
backlog/issues/507
backlog/issues/485
backlog/issues/488

Closing this PR without merging. We will focus on the issues listed above instead. Once an issue is resolved, we can then un-exclude its target. Confirmed with Jason via Slack.


[MB-13917] pass from value to View Orders link on view allowances page

Jira ticket
Summary
This PR updates #11086 so that when a TIO starts on the payment request page and navigates to view orders -> view allowances -> back to view orders -> on close they are taken back to the payment request details page
Verification Steps for the Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[x] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.
Setup to Run the Code

Instructions for starting storybook
Instructions for starting the MilMove application
Instructions for running tests

How to test

Access the office app
Login as a TIO
Select a payment request that has a Pending Status (these show up in the queue as "Payment Requested"
On the payment request tab, scroll to find the "View Orders" link for the payment request and click it
Click on View Allowances.
Click on View Orders
Close
Ensure that clicking X takes you back to the previous page (payment request details tab)

See this slack message for more context
Frontend

[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, Edge).
[ ] There are no new console errors in the browser devtools.
[ ] There are no new console errors in the test output.
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Warnings

:warning:
This PR does not include changes to unit tests, even though it affects app code.

Messages

:book:
:link: MB-13917

  Generated by :no_entry_sign: dangerJS against 141a1027c3b8f208c8cb57a39cfbd6fd8dc2a2cb


Menu bar text too large?

I opened VSCode today and found that the menu bar text was much larger than usual.

Comparison shot:

VSCode Version 1.8.1
Windows 10, Version 1607, Build 14393.693
Changing the DPI compatibility setting for code.exe did not fix this. Neither did running with all extensions disabled. Ctrl-+ and Ctrl-- zoom every other aspect of the UI including the sidebar, but not the top menu bar.
I'd guess this has to do with Electron, seeing as how Chrome had a similar size increase in its UI a little while back. I fixed that with the command flags "/high-dpi-support=1 /force-device-scale-factor=1", but obviously those flags don't work here.
It's such a little thing, but it's driving me nuts.
Edit: Uninstalling and reinstalling did not fix the problem. Building from source, however, did.

Insiders build does not have the bug. Closing. Thanks!


Function, Parameters, ReturnType

TypeScript Version:  3.8.3
(Not tested against Nightly; as the playground doesn't work with Nightly for some reason for now...)

Search Terms:
Function, ReturnType, Parameters
(Extremely hard to find something reliable because of the word "Function"...)
Code
const a: Function = () => ({})
type B = ReturnType<typeof a>
type C = Parameters<typeof a>

Expected behavior:
ReturnType and Parameters accepting Function global type.
Actual behavior:
Compiler complaining Function not being compatible with ReturnType constraint (...args: any) => any
Type 'Function' does not satisfy the constraint '(...args: any) => any'.
  Type 'Function' provides no match for the signature '(...args: any): any'.

Playground Link:
https://www.typescriptlang.org/play/?target=99#code/MYewdgzgLgBAhgLhgMQK5mFAluGBeGACgEp8A+IgbwF9iAoKATwAcBTGAIXxgCVWpUAJzAAVFqwA8TNiABm8Mg3EwAwtwAKcQXAC2-VoIhTxchUA
Related Issues:
Not really, see "Search Terms" for explaination

I was also unable to cook up any good searches.
What type would you want B and C to be here?  any?

This is the intended behavior since, as noted, there is not any obvious answer to the question "What parameters does a accept".

@nmain hum; nah I guess
const a: Function = () => ({})
type B = ReturnType<typeof a>
// {}
type C = Parameters<typeof a>
// []

@RyanCavanaugh an empty array I guess ?

@RyanCavanaugh ; check this out :
const a : () => void = () => {}
type A = Parameters<typeof a>
type B = ReturnType<typeof a>

https://www.typescriptlang.org/play/?target=99&ssl=1&ssc=1&pln=3&pc=30#code/MYewdgzgLgBAhjAXDAFAShgXgHwwG4gCWAJlqhjjAN4C+AUFAJ4AOApjAIJkAKcATnAC2rKKz4QAPEzYgAZvGwMW7AEJkASiICufMABVlU5XIVA
This works without any compilation error; and I doubt my first use-case is an intended behavior; because in that last example; Parameters = []; ReturnType = void which seems logical

Ok just found why;
Inside lib.es2015.core.d.ts; the Function interface does not embed a "callable" form.
See this : https://www.typescriptlang.org/play/?target=99#code/JYOwLgpgTgZghgYwgAgGIFcQLMA9iVKXAWwBlgAjZAbwChlkB6AKmfoeeQGE4AbXgM7IwACxQxM2PCAA0yAegoCcYdDhABzYWPkAHCAmAxgEACbJcFAFYGwyGLijaUo4EIBufdClwxn9yRx8OTgQc1EUAX1DYzNkOCgoOABPe0d-BI10YghwIV9-CSwgkAA6dmROAAFdBLhibTcAQSgtABUdSxtsYVxkChR0ATi4IQjG-OtbcoZK5Bq6hsyWpNSm+Qg7Aszs3LAxvoHkWoFh8L7xoql8GY5GdjhdXV5kgApXAQAuNEDpOQ+WhpvqFkiFWisUgB+YEgZIAShhyQA3LR2Cw2LNODx+EI4MgcqJcOZtiALFNsHIFEoVGpQFpQrgIk4urY0k5xgh0Ik9mTumBbnMFkkGgDWsgOj5yXYwIdBmd4mMdJzueBedMKtVasL4uDEilkOteG4tn4djk8r1+igTvKZf4CSIiQLmPcGAg+Lx3iI3N8MMU-hNAYi5KVQ8s9clEQBtAC6CPisJRaNYGrQ6TxGmA7lyAX9wWQCCgEDgkFx-VwmHMVxK2hLyBEo38AnqKAoRNSjfGjmAmZAfFz1zKqYlEzVPQK4zblYHNbcCoEuEMJbiAHdgKIm9EjCZiVKQmF652dFEDNu4qB18B+1qW5AoAJnfMbyLvQJAQbSSyenaV96ECJ-A+ZAAGsIGSFdHHMd1SSLGBoGQUABGAUwXB0EAIBXGdpEfIV6h1DQITWZAjWUCxTVaXYLTtI4bTiO1xnQzDq2wjVXX6UBTC9H0fjzWRA1aYNkFDUpw1WaM40RJMGHRZAACVNi5EAy2UKA6WQItdCLYZwBLaQyPiLCbkqNiZQAZTAVTNFeeMVLpKTjiIGUwGSfRJPYIs4FMfAXmI3INFEb4QGyAYoHsxhGGQAA5fAAFplFCUwEnMCAAA9ICU6QBAeCjzX2Ny3Q9aBfV+fAUQAX1RUA73gJA0GAFKzD9QdCBIcgqFS9LTCEJqShashKBoNEIraAAJABRNAAEkAA1kHG2SxvYV5hMyL4E2SWN4xBcrUWc-Q5ISkger0gBeZBrOQE6AD5kHcXBkNoPaUHWM6AAVFk2aABAAHlkw7iGO-ArselyUAAIUuuSFKgEA2lB37-sBkBgae7hIfe4VPvvb6kZR0HkAAEUh+TVBhuH9BxkrkZB-aJrej67x+pG+ravH9tQYnodh+HmaIfqKDZlAAHF0YZr6cfqxqqZZyhBbmznSe5inUEl0xeda2WgA

Function is meant to represent any value for which v instanceof Function is true, with no further restrictions.  So it has no known call signatures.  If you know you have something that has a particular signature, type it as that.
Function values are granted a signature of (...args: any[]) => any when you attempt to call one for back compat reasons, but that signature doesn't participate in other things.  If you want that signature, you can explicitly type your function as that too.

@nmain OK thanks :) I understand.
I'll make a Callable<T extends Function = Function> helper; but in my opinion;

Function values are granted a signature of (...args: any[]) => any when you attempt to call one for back compat reasons

I'm curious, where is that code actually ?

Function is meant to represent any value for which v instanceof Function is true, with no further restrictions. So it has no known call signatures. [...] that signature doesn't participate in other things

That's curious IMHO. Especially the fact every duck-typing insights are coded inside lib.es2015.core.d.ts EXCEPT the main - basic - feature of a function : being callable. Am I missing something ? oO


Safer key storage in the browser

The decentralized architecture of Atomic Data dictates that changes to information (Commits) are signed using the private key of an Agent (user). This means that, if we want to edit things from the browser, the browser app needs to sign commits using that private key.
Attack risks of current implementation:

Thief could wait until user leaves desk, go to agent settings, copy the secret / mail it / send it / photograph it. This could be preventable by making the secret only copy-able on creating the Agent, and locking it afterwards.
Thief could check localStorage on same browser, even after the session.
Thief could navigate the JS state / the react tree and find the private key value

Possible solutions
I want both a good UX, and a safe solution. Some thoughts:

Storing the key anywhere where JS has access (localstorage, indexedDB), means that all libraries of Atomic Data Browser have to be trusted.
The CryptoKey browser API seems interesting. It saves to indexedDB, but in a way that the key itself cannot be retrieved - it can only be used for signing things. I don't think it supports ed25519 signatures, which kind of sucks, because that's what I'm using in this project and the rust implementation of atomic data.

Some resources to check out::

https://pomcor.com/2017/06/02/keys-in-browser/
https://www.w3.org/TR/WebCryptoAPI/
https://developer.mozilla.org/en-US/docs/Web/API/Web_Crypto_API
https://www.w3.org/TR/IndexedDB/
https://www.boxcryptor.com/en/blog/post/building-an-app-with-webcrypto-in-2016/
https://gist.github.com/saulshanabrook/b74984677bccd08b028b30d9968623f5
https://blog.engelke.com/2014/09/19/saving-cryptographic-keys-in-the-browser/

Curve25519 (including ed25519) support is currently being tested in Chrome and being worked on in FireFox. If this is implemented, we could probably switch to the web.crypto API


Footer and margins on homepage

On the homepage with larger screens and shorter content, the footer appears at the end of the body rather than the end of the page.
Additionally, the header container is compacted right up against the navigation. A margin might help here to give it more breathing room.

I am not sure I understand you comment. Shouldn't the footer appear at the end of the body? Placing it at the bottom of the screen would appear far removed from the rest of the page.
As for the header, The main idea is to have compact navigation to leave space for the rest, but I'm happy to see some examples of what you mean.

Fixed in release B.4.1


merge package from arlac77/npm-package-template

package.json

chore(scripts): cover@#overwrite c8 --temp-directory build/coverage ava && c8 report -r lcov --temp-directory build/coverage
chore(package): add xo from template
chore(package): add nyc from template
chore(package): set $.ava.require='esm' as in template
chore(package): set $.ava.files='tests/-test.js,tests/-test.mjs' as in template

Coverage remained the same at 97.619% when pulling<PHONE_NUMBER>91fd4ca14669bdba12721fc9842db4 on template-sync-1 into 2bbe91d384f851fbc90dca1d2c58e3099e316ba6 on master.


Added cardano-client-<IP_ADDRESS>

From https://github.com/intersectmbo/ouroboros-network at 624085b7a84f099b3f669cda61c380e87fd5d0a7

[x] checked with build-with-chap.sh in ouroboros-network


pod container Gemini Connection refused

pod manager 50052 is not up
[root@k8s-node4 yaml]# kubectl log pod-i-1 -c run
log is DEPRECATED and will be removed in a future version. Use logs instead.
main before
run
2020-04-28 07:27:15.822120 Gemini E/ attempt 1: Connection refused
2020-04-28 07:27:25.822452 Gemini E/ attempt 2: Connection refused
2020-04-28 07:27:35.822711 Gemini E/ attempt 3: Connection refused
[root@k8s-node4 yaml]# kubectl get pod
NAME      READY   STATUS    RESTARTS   AGE
pod-i-1   1/1     Running   0          44s

python 3.65
LD_LIBRARY_PATH=/usr/local/lib64:$LD_LIBRARY_PATH
resource-config.txt 文件中的client1 应该与launch-command.py --name保持一致
resource-config.txt client个数 为实际个数


Update readme.md: added SpotX to list of adopters

We use circe in pretty much every scala project here at SpotX.
Thank you for this awesome library!

Codecov Report

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

@@          Coverage Diff           @@
##           master    #910   +/-   ##
======================================
  Coverage    85.6%   85.6%
======================================
  Files          76      76
  Lines        2459    2459
  Branches      157     157
======================================
  Hits         2105    2105
  Misses        354     354

Impacted Files
Coverage Δ

...d/src/main/scala/io/circe/NonEmptySeqDecoder.scala
91.66% <0%> (-8.34%)
:arrow_down:

...rc/main/scala/io/circe/numbers/BiggerDecimal.scala
91.25% <0%> (-0.63%)
:arrow_down:

...les/core/shared/src/main/scala/io/circe/Json.scala
78.84% <0%> (ø)
:arrow_up:

...rc/main/scala/io/circe/shapes/SizedInstances.scala
92.85% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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

Great to hear, thanks @dcastro!


[Carousel-v3.9.0-Android] The item gets stuck when dragged left

Is this a bug report, a feature request, or a question?
Bug

Have you followed the required steps before opening a bug report?

[x] I have read the guidelines regarding bug report.
[x] I have reviewed the documentation in its entirety, including the dedicated documentations :books:.
[x] I have searched for existing issues and made sure that the problem hasn't already been reported.
[x] I am using the latest plugin version.
[x] I am following the issue template closely in order to produce a useful bug report.

Have you made sure that it wasn't a React Native bug?
Yes

Is the bug specific to iOS or Android? Or can it be reproduced on both platforms?
Android
Is the bug reproductible in a production environment (not a debug one)?
Yes

Environment

React: 16.11.0
React native: 0.62.1
react-native-snap-carousel: 3.9.0

Expected Behavior

No item stuck (previous version behavior)
Actual Behavior
The item gets stuck when dragged left.
This behavior is introduced by this commit archriss@9a42e4eef52e4c18bf8f177b1a0350de4a5e0c6b

Reproducible Demo
https://snack.expo.io/@ahce/a66df8

Steps to Reproduce

Start expo example in Androird device
Drag the item to the left

Can you try the shiny new beta version and let me know how it works for you?
Click here to learn everything about the upcoming v4!

Can you try the shiny new beta version and let me know how it works for you?
Click here to learn everything about the upcoming v4!

@bd-arc The bug continues in v4

@ahce After looking at your code, there is no bug: you're actually defining a itemWidth superior to sliderWidth, which just can't work.
Try inverting the two!

@ahce After looking at your code, there is no bug: you're actually defining a itemWidth superior to sliderWidth, which just can't work.
Try inverting the two!

With the previous behavior, it was possible to drag the last item (similar to iOS), if itemWidth <= sliderWidth this does not happen.

@ahce The plugin is not meant to display items with a size superior to the one of the carousel. I think this was more of an unintended side effect.
What is your use case and what are you trying to accomplish with the plugin exactly?

@ahce The plugin is not meant to display items with a size superior to the one of the carousel. I think this was more of an unintended side effect.
What is your use case and what are you trying to accomplish with the plugin exactly?

That the last item can be dragged, when it's released it's centered.
Check this example
Also the paging doesn't update correctly (with fast dragging on real devices).
I had to go back to v3.8.4.
If reverse this  commit, works fine.


Update gsp/forecast/{gsp_id} route

Detailed Description
We can update for gsp/forecast/{gsp_id} route to set only_forecast_values=True. Will need to update the documentation with this.
Context

https://github.com/openclimatefix/uk-pv-national-gsp-api/issues/200, check that no users are using the soon to be removed routes

Possible Implementation

remove parameters historic and only_forecast_values and assume only_forecast_values=True
add extra route which is gsp/{gsp_id}/forecast and point old route to it
Update documentation

@rachel-labri-tipton Ive split the tidy task out here for the gsp/forecast route


Fix bugs

[ ] Show each role requirement on a new line
[] Throws parser exception: Unexpected argument for command "":  when argument is passed to non-argument commands
[] Add maximum of 50 workers per role to role requirement
[] Throw duplicate roles detected error for shift-add and shift-edit
[] Change worker phone number to accept only Singapore phone numbers (8 digits, starting with 6, 8 or 9)
[] Change AssignCommand to detect and throw error when duplicate workers is being assigned

Codecov Report

Merging #204 into master will decrease coverage by 0.09%.
The diff coverage is 52.50%.

@@             Coverage Diff              @@
##             master     #204      +/-   ##
============================================
- Coverage     69.27%   69.17%   -0.10%
- Complexity      852      859       +7
============================================
  Files           137      137
  Lines          3033     3066      +33
  Branches        416      424       +8
============================================
+ Hits           2101     2121      +20
- Misses          809      821      +12
- Partials        123      124       +1

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/mcscheduler/commons/core/Messages.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...heduler/logic/commands/WorkerAvailableCommand.java
27.45% <0.00%> (ø)
9.00 <0.00> (ø)

...a/mcscheduler/model/assignment/WorkerRolePair.java
76.00% <ø> (ø)
11.00 <0.00> (ø)

.../java/mcscheduler/model/shift/RoleRequirement.java
90.90% <ø> (ø)
21.00 <0.00> (ø)

src/main/java/mcscheduler/ui/ShiftCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

src/main/java/mcscheduler/ui/WorkerCard.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...main/java/mcscheduler/logic/parser/ParserUtil.java
57.50% <22.22%> (-5.36%)
26.00 <0.00> (ø)

...va/mcscheduler/logic/parser/McSchedulerParser.java
68.08% <100.00%> (+18.08%)
18.00 <0.00> (+6.00)

src/main/java/mcscheduler/model/worker/Phone.java
81.81% <100.00%> (+1.81%)
7.00 <1.00> (+1.00)

src/main/java/mcscheduler/model/worker/Worker.java
96.55% <100.00%> (ø)
27.00 <0.00> (ø)

Continue to review full report at Codecov.

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


After Updating where-used list: "Error from RPY_DYNPRO_INSERT: 6"

Hello,
we are using abapGit 1.86.0 and we are facing an error on importing a package which we exported with abapGit. After importing the package from the online repository by pressing 'PULL', we are getting this error message Error from RPY_DYNPRO_INSERT: 6 after the step where abapGit updates the where-used list of the objects. Import and export were done on the same system.
We are running on NW 731 SP 15. The export has been created with folder logic PREFIX in the repository settings and it's imported with folder logic FULL. No other options set in the repository settings. Our package contains a registered customer namespace like /EXAMPLE/<package_name>.
Thanks for your support!
Best Regards

try setting a breakpoint at https://github.com/larshp/abapGit/blob/06097f96142e655b46729b1e32339b24ff33020f/src/objects/zcl_abapgit_objects_program.clas.abap#L372 and check SY messages

https://github.com/larshp/abapGit/issues/1174 might be relevant, check if function module RS_SCRP_CHECK_FIELD_POSITIONS gives an error

I've forwarded this to my colleague and I will get back to you quickly with more information. Thanks for your support!

According to the sy message variables we found out that there is an error thrown:
The field <field_name> doesn't exist in active version of table <our table>.
The missing field is a field of one of our own tables. We have to find out if this field is called in the source code or on the dynpro somewhere.
Function module RS_SCRP_CHECK_FIELD_POSITIONS seems fine but some call after it seem to set the sy-subrc to 6. SAP Logon is crashing with C parser thread fail popup during debugging so we are not able to analyse it right now.
Our developer will analyse this in detail next days. So far, I can't provide more details.

there might be some issues in the deserialization sequence, so <our table> does not exist or is not active when the dynpro is inserted. It should have been imported, check the status in SE11 and try activating manually, then retry the import

If you are using the long report, it might help with the "C parser" error if you install the many classes instead.
RS_SCRP_CHECK_FIELD_POSITIONS: added https://github.com/larshp/abapOpenChecks/pull/682 to check for this in the future

I checked see11/se16: The table does exist and it is active. The mentioned field is missing.

@GiantCrocodile any news?

I've contacted our developer regarding this and asked for news. I'm sorry for the late reply @larshp - seems like I missed the notification e-mail.

It looks like an issue in the exported objects. I will close this issue now due to the fact that it's most likely a mistake on our side. Thanks for your support @larshp!


Using C# and Visual Studio Code produces weird references in the VSCode editor

Godot version
4.1.1 stable
System information
Windows 11
Issue description
The problem is maybe not related to Godot directly, but Visual Studio Code is an open source tool used by a lot of people so I decided to post it here anyway.
The problem is in intelissense, it generates references to files I do not use.
Please see image below:

So there is a variable called "_velocity" and I use it once. There is 1 real reference to this variable in my code.
But as can be seen from the image, there are 4 additional reference to "_velocity" in some \Godot.SourceGenerators\blabla... files.
This applies to almost every variable that I have not just "_velocity", so if I add a new varible to class I will get the same behaviour:

I assume those files do not concern us directly but are rather used by the engine itself.
Is there a way to disable the references to those files ?
I tried via VScode Preferences->Settings-> files.exclude but I failed to get rid of them.
Steps to reproduce
See above.
Minimal reproduction project
N/A

I assume those files do not concern us directly but are rather used by the engine itself.

Yeah, that generated source code is used to communicate with the engine. But the code is still part of your project, even if it's generated, and you still have access to it. So the references are just as real as references in your own code.

Is there a way to disable the references to those files ?

Not that I know of, but it's a common request (see https://github.com/godotengine/godot/issues/77128). This would be VSCode's responsibility though, they should already be able to differentiate between generated and non-generated files.
I'll close this issue since there's nothing we can do on our side.


Fix update-solana-dependencies.sh to work on osx

Bump deps to 1.10.0 and tweak update-solana-dependencies.sh to work on osx.

For context: this is currently needed to run scripts/build-downstream-projects.sh in the monorepo or it will pull the solana crates from crates.io

Hmm, scripts/build-downstream-projects.sh works for me on my mac. I did have to install gnused, though.
I also usually use a patch like this to run against my spl clone:
diff --git a/scripts/build-downstream-projects.sh b/scripts/build-downstream-projects.sh
index a2181ebe4..7c499a58f 100755
--- a/scripts/build-downstream-projects.sh
+++ b/scripts/build-downstream-projects.sh
@@ -60,9 +60,9 @@ example_helloworld() {
 spl() {
   (
     set -x
-    rm -rf spl
-    git clone https://github.com/solana-labs/solana-program-library.git spl
-    cd spl
+    # rm -rf spl
+    # git clone https://github.com/solana-labs/solana-program-library.git spl
+    cd /Users/xxx/solana-program-library

     ./patch.crates-io.sh "$solana_dir"

Oh I missed that build-downstream-projects.sh bumps the version through ./patch.crates-io.sh. Then just the extended regex fix should be sufficient.
And yes installing gnused would probably work, but turning into ERE makes it work with BSD sed as well, and removes a footgun both @dmakarov and I wasted some time today 😅

As long as it doesn't break the monorepo CI, I'm all for it :)

I've tried the regex with GNU sed on ubuntu 20.04 so I'm reasonably sure it should work (famous last words)

The SPL project's Cargo.toml is patched by
However, that doesn't help on my system at least. For example, I see these messages when cargo test is started by cargo-test-bpf (same happens when cargo build runst)
/Users/dmakarov/.rustup/toolchains/1.57.0-x86_64-apple-darwin/bin/cargo test --manifest-path /Users/dmakarov/work/git/solana/target/downstream-projects/spl/stateless-asks/program/Cargo.toml --features test-bpf -- --test-threads=1
warning: Patch `solana-account-decoder v1.10.0 (/Users/dmakarov/work/git/solana/account-decoder)` was not used in the crate graph.
Patch `solana-banks-client v1.10.0 (/Users/dmakarov/work/git/solana/banks-client)` was not used in the crate graph.
Patch `solana-banks-server v1.10.0 (/Users/dmakarov/work/git/solana/banks-server)` was not used in the crate graph.
Patch `solana-bpf-loader-program v1.10.0 (/Users/dmakarov/work/git/solana/programs/bpf_loader)` was not used in the crate graph.
Patch `solana-clap-utils v1.10.0 (/Users/dmakarov/work/git/solana/clap-utils)` was not used in the crate graph.
Patch `solana-cli-config v1.10.0 (/Users/dmakarov/work/git/solana/cli-config)` was not used in the crate graph.
Patch `solana-cli-output v1.10.0 (/Users/dmakarov/work/git/solana/cli-output)` was not used in the crate graph.
Patch `solana-client v1.10.0 (/Users/dmakarov/work/git/solana/client)` was not used in the crate graph.
Patch `solana-core v1.10.0 (/Users/dmakarov/work/git/solana/core)` was not used in the crate graph.
Patch `solana-logger v1.10.0 (/Users/dmakarov/work/git/solana/logger)` was not used in the crate graph.
Patch `solana-notifier v1.10.0 (/Users/dmakarov/work/git/solana/notifier)` was not used in the crate graph.
Patch `solana-program v1.10.0 (/Users/dmakarov/work/git/solana/sdk/program)` was not used in the crate graph.
Patch `solana-program-test v1.10.0 (/Users/dmakarov/work/git/solana/program-test)` was not used in the crate graph.
Patch `solana-remote-wallet v1.10.0 (/Users/dmakarov/work/git/solana/remote-wallet)` was not used in the crate graph.
Patch `solana-runtime v1.10.0 (/Users/dmakarov/work/git/solana/runtime)` was not used in the crate graph.
Patch `solana-sdk v1.10.0 (/Users/dmakarov/work/git/solana/sdk)` was not used in the crate graph.
Patch `solana-stake-program v1.10.0 (/Users/dmakarov/work/git/solana/programs/stake)` was not used in the crate graph.
Patch `solana-transaction-status v1.10.0 (/Users/dmakarov/work/git/solana/transaction-status)` was not used in the crate graph.
Patch `solana-version v1.10.0 (/Users/dmakarov/work/git/solana/version)` was not used in the crate graph.
Patch `solana-vote-program v1.10.0 (/Users/dmakarov/work/git/solana/programs/vote)` was not used in the crate graph.
Check that the patched package version and available features are compatible
with the dependency requirements. If the patch has a different version from
what is locked in the Cargo.lock file, run `cargo update` to use the new
version. This may also occur with an optional dependency that is not enabled.

So on my system, SPL is not built using the local clone of solana repository. Instead the version downloaded from crates.io is used always.

@dmakarov are you sure build-downstream-projects.sh is building a fork of spl that includes this patch? It'll remove the existing tree and do a fresh clone of main, so you need to tweak the script to test the patch. This is what I did:
diff --git a/scripts/build-downstream-projects.sh b/scripts/build-downstream-projects.sh
index a2181ebe4..f4b8461af 100755
--- a/scripts/build-downstream-projects.sh
+++ b/scripts/build-downstream-projects.sh
@@ -61,7 +61,8 @@ spl() {
   (
     set -x
     rm -rf spl
-    git clone https://github.com/solana-labs/solana-program-library.git spl
+    # git clone https://github.com/solana-labs/solana-program-library.git spl
+    cp -a $HOME/src/solana/solana-program-library spl
     cd spl

     ./patch.crates-io.sh "$solana_dir"
@@ -101,6 +102,6 @@ EOF
 }

Where $HOME/src/solana-program-library includes this patch.

never mind. the patch works perfectly.


Crash with discover and TVMaze

Hi, there is an issue in latest version (1.2.400) with the discover plugin and TVMaze, here is the error: http://pastie.org/10606946
There is a workaround, you need to add ignore_estimations: yes to your discover plugin configuration.
If you need anything to help solving this I'm willing to help!

This should have been auto closed. Weird. Should be fixed now.

My commit was a temporary fix, we should still fix the lookup functions to raise a LookupError any time they don't return a result.


Dropwizard / Docker example can't find task definition

When I run mvn -Pawseb -Dbeanstalk.versionLabel= beanstalk:create-environment I get back:
[WARNING] FAILURE
org.apache.maven.plugin.MojoExecutionException: Something went wrong in while waiting for the environment setup to complete : No ecs task definition (or empty definition file) found in environment

I understand that it's looking for the Dockerrun.aws.json but it's obviously not looking in the right place.

Why are you setting versionLabel to empty?

Because you said to here: http://stackoverflow.com/questions/27185671/elastic-beanstalk-docker-and-continuous-integration
;)

Because you said to here: http://stackoverflow.com/questions/27185671/elastic-beanstalk-docker-and-continuous-integration
;)

I think the problem was that I ran package without the awseb profile, so the zip wasn't properly created. I'll assuming this is the case and close this ticket.

I just ran into this issue and discovered that this page is currently the only search result for "No ecs task definition", so here's some more detail: My problem was that my Dockerrun.aws.json file got renamed inside the zip file for the uploaded version. A clearer error response from AWS would be "Dockerrun.aws.json not found".

I just ran into this too - in my case it's because I'd created the zip on a mac by downloading an existing app version, editing, rezipping (from a terminal) and reuploading. In downloading the zip from AWS my mac had set a com.apple.quarantine extended attribute on all of the files, which (I guess) made ECS ignore them. Removing that attribute with xattr -c prior to zipping did the trick.

Are your SolutionStacks multicontainer? If so, it leaves the CloudFormation realm and goes into ECS realm :)
And btw, the original solutionStack mask from the archetype seems to allow it to be picked. Try this instead:
<beanstalk.solutionStack>64bit Amazon Linux 201* v* running Docker *</beanstalk.solutionStack>

I ran in to the No ecs task definition as well. I tried to add a task_definition.json in ecs settings. But every time I run my task, it would stop immediately, I think it's an issue with my task_definition.json because I don't know how to write the configuration for node.js application in that.

thanks to @powdahound
The issue is that the source-bundle is been uploaded to S3 in a wrong directory instead of been in the root directory.
.
└── some_files
└── more files
└── Dockerrun.aws.json
Instead it should be;
.
└── Dockerrun.aws.json
└── some files


Documentation Edits

Added a table of contents to the README, to make it easier to navigate.
Edited some titles of the README's sections.
Simplified the Build Dependencies section.
Inserted commands into coding blocks for easy copy/paste.
Gave the "Documentation" section a hotlink to the respective folder.
Added a table for the python dependencies.

I am very sorry, however this is no longer needed with the big doc PR I did a while ago. If you feel, I missed something, please re-open or create a new PR.


Remove deploy cairo0

@karol-bisztyga I think this can be merged now

done in https://github.com/software-mansion-labs/cairo/pull/68


dependencies always resolved as latest

It seems dependencies are always resolved as latest.
E.g. git:2.4.0 depends (amongst others) on matrix-project:1.4
This is also echoed when running the script with arg git:2.4.0
However, when it comes to the actual download of the dependency, it always takes the latest version as the download function is only called with one argument, which is the plugin name - so the download function falls back to latest.
One could argue that plugins should be compatible, so using a newer version should be safe. However that is only true for the API the plugin provides, but not for the API it uses itself.
This means, even though the latest version of matrix-project, say 1.6 is compatible, hence will not break the older git plugin 2.4.0 that uses it: the latest version of matrix-project in turn might internally rely on a newer Jenkins API.
Probably in my Docker file I've set git explicitly to version 2.4.0, because I know this is compatible with the Jenkins image I'm inheriting in my Dockerfile!
If suddenly the newest version of the matrix-project dependency is fetched, say 1.6, this might not be compatible with the Jenkins version I'm using...

This is intended and was changed to match the Jenkins UI plugin manager, see https://github.com/jenkinsci/docker/issues/327
If you need a specific version you can use a explicit version

Hi carlossg / all,
I understand the point that the UI plugin manager behaves like this and read #327.
However, isn't the scenario I've described a potential big issue?
To me the solution of always taking "latest" sounds like a kind of workaround to the issue described in #327 - why should a clean installation of dependencies cause errors?
This sounds rather like the plugins have not maintained clean dependencies / some dependencies they have cause warnings. Indeed I have seen such warnings in the past during startup. But to my understanding this is something that should be fixed in the plugin dependencies, not in the installer.
And with a recent version of Jenkins and the plugins it seems these errors are gone, at least for my use case / particular set of plugins, when I fetch the exact dependencies, not latest, I cannot see any warnings during startup anymore.
I do think explicitly setting the versions is quite cumbersome, from a small set of 10 base plugins you wanna install you easily end up with 50 plugins in total that come from the dependencies and for which you're suddenly forced to give the explicit version...

you just need to explicitly set the version of the plugins that a re a problem in latest version


List Simulate() methods for lazy people searching to find them quickly

A heuristic is to search on the page for doubleClick. I did this and assumed it was not supported (because click() was listed, and doubleClick() not).

:+1:


Switching tabs reloads view

Is there a way that I can program the tab views so that it does not load if I'm going back to a tab?
For example, I have a table view that gets populated from a network call on tab 1, I then navigate to another tab and navigate back to tab 1, I find that it reloads the view, it makes a network call and loads loads the table view as though it dismounted the view.

Sorry, the tab views aren't related to the BottomNavigation.
If you're using react-navigation, look through their issues.
If not, SO is a great place for questions.
Hint: Load and save your fetched data in a data structure independent from your view, e.g. a redux store or something similar. That way your data isn't lost if the component gets unmounted in the meantime.


So bad

Any consideration to enhance Chinese?

Sure, we are working on it. Once getting it done, we will let you know.


dep ensure --update fortio.org/fortio

to get 1.2.1 (small bug fix over 1.2.0) also to confirm it's easy/working to upgrade fortio

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ldemailly
If they are not already assigned, you can assign the PR to them by writing /assign @ldemailly in a comment when ready.
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:

OWNERS

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

/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ldemailly
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:

OWNERS [ldemailly]

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
86836812f284aaa85aaa952aa92f343aa6860227
link
/test istio-unit-tests

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.

/lgtm
/approve

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ldemailly, sebastienvas
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:

OWNERS [ldemailly]

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

/woof

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
86836812f284aaa85aaa952aa92f343aa6860227
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
86836812f284aaa85aaa952aa92f343aa6860227
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.

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
86836812f284aaa85aaa952aa92f343aa6860227
link
/test istio-unit-tests

prow/istio-pilot-multicluster-e2e.sh
86836812f284aaa85aaa952aa92f343aa6860227
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.


Failed in adding vertex to elastic graph

It can load the vertex and edge data to graph from my ES store, but I cannot add vertex to it. My code is as follows:
BaseConfiguration conf = new BaseConfiguration();
conf.addProperty("providers", "D:/work/gdb/docs/conf");
UniGraph graph = UniGraph.open(conf);
GraphTraversalSource g = graph.traversal();
graph.addVertex(T.id, "3", T.label, "software", "name", "gremlin");
After reading the code, I found in the StaticPropertySchema, there is the following method:
public Map<String, Object> toFields(Map<String, Object> prop) {
Object o = prop.get(this.key);
if(o == null || o.equals(this.value)) return Collections.emptyMap();
return null;
}
In the method, the key and value is always the V(1) vertext, and  my prop of new vertext
is not equal to V(1) props, so this method always return null. And so my vertex will never
be added to ES store.
Anyone can help give an explanation on it?
@seanbarzilay

@zxnblake can you provide you mapping file?


ZCL_ABAPGIT_TIME: Unit tests fail on NW 752 SP04 dev

UTC difference is being determined by TZON_GET_OFFSET with same import parameters two times but with different results:

Oh...of course summer and winter time..


Add quasistatic 2D hand-ball examples.

A global change I made in PR is renaming /irs_lqr to /algorithm, which makes importing easier. For example, under the old naming convention, /examples/quasistatic/dynamics_gradient_worker.pyneeds to import from irs_lqr.irs_lqr.quasistatic_dynamics, but if I'm running it at the repo's root, it's going to confuse the package name irs_lqr with the subpackage irs_lqr which has now been renamed to algorithm. I used PyCharm's refactoring to make the changes, so it should not have broken your code. Let me know if there's any problems!

Oop - before the review, let's discuss this global change. The convention that I am used for python packages is that the root of the github folder (irs_lqr) gets added to PYTHONPATH, and import irs_lqr should automatically import the folder that was renamed algorithms.
Some examples:
https://github.com/numpy/numpy has numpy folder within repo, and import numpy imports the folder.
https://github.com/cvxpy/cvxpy has cvxpy folder.
https://github.com/openai/gym has gym folder.
Maybe it has to do with how things get packaged into pip later on? I am not sure :) I am fine with this change if it's causing lots of trouble with the IDE, but would prefer to keep irs_lqr folder.
The import should call irs_lqr.quasistatic_dynamics because the high-level irs_lqr should be on the path.

Reverted renaming /irs_lqr to /algorithm based on slack discussion.


Try to break things with random .gitmodules

Reproed here https://github.com/youtube/cobalt_sandbox/pull/424