Follow-up from "Bschubert/backport log missed cache"

See original issue on GitLab
In GitLab by [Gitlab user @tristanvb] on Aug 29, 2018, 18:22
The following discussion from !752 should be addressed:

[ ] [Gitlab user @tristanvb] started a discussion:

I realize this is already in master, and is probably the best thing to do for bst-1.2 branch now that we're so close to release, however, this has problems and should ideally be done differently, but it will require some additional support from the core.
The main problem I can see here is, if all of the artifact servers contacted did not find the artifact, then the entire task should be marked as SKIPPED.
SKIPPED also usually indicates that an element in a queue is SKIPPED, which should cause the summary and status bar to show the element as skipped as well.
This patch will only emit SKIPPED messages, but will not treat the element as SKIPPED in the summary.

I think an interesting approach to resolving this properly is to add some semantic for the queues to raise a special exception in their processing method, allowing the main job harness to treat the entire task as SKIPPED accordingly and bookkeep it properly.

In GitLab by [Gitlab user @tristanvb] on Aug 29, 2018, 18:26
[Gitlab user @BenjaminSchubert] could you investigate this ?

In GitLab by [Gitlab user @BenjaminSchubert] on Aug 29, 2018, 18:30
Sure, I'll give it a look this week!

In GitLab by [Gitlab user @BenjaminSchubert] on Aug 30, 2018, 09:36
assigned to [Gitlab user @BenjaminSchubert]

In GitLab by [Gitlab user @Qinusty] on Aug 30, 2018, 16:39
mentioned in merge request !765

In GitLab by [Gitlab user @Qinusty] on Aug 30, 2018, 16:55
I appear to have unknowingly submitted an MR for this before seeing the issue. As it is assigned to you [Gitlab user @BenjaminSchubert], can you take a look at my MR https://gitlab.com/BuildStream/buildstream/merge_requests/765 and see if it covers all of the bases.
Apologies for not spotting this issue sooner.

In GitLab by [Gitlab user @Qinusty] on Aug 30, 2018, 17:21
assigned to [Gitlab user @Qinusty] and unassigned [Gitlab user @BenjaminSchubert]

In GitLab by [Gitlab user @Qinusty] on Sep 20, 2018, 13:16
closed


Document LoRa Basics Station CUPS forwarding

Summary

We should document LoRa Basics Station CUPS forwarding - the why and the how.
Why do we need this ?

In order to show users how to redirect gateways to other Gateway Servers using CUPS. This is very relevant for the remote control of the gateway.
What is already there? What do you see now?

Generic description of CUPS and LNS.
What is missing? What do you want to see?

A new section for CUPS redirection.
How do you propose to document this?

Explain situations in which this is applicable.

This applies to gateways using the LoRa Basics Station forwarder, which are configured to do CUPS with a The Things Stack instance.
To users which want to move their gateway connections from one instance (for example TTSCE) to another (for example, TTSC).

Explain how to achieve this:

One needs to register the gateway in the target instance (i.e. TTSC if you migrate from TTSCE).
Then one needs to create an LNS key in the target instance. The LNS key is a gateway API key with linking rights.
Then one needs to set the LNS key from the target instance in the source instance (i.e. TTSCE if you migrate to TTSC).
Optional: invalidate the LNS key in the source instance, either by deleting it or by removing the linking right.
Finally, the gateway server address of the target instance must be set as gateway server address in the source instance.
Yes, these are exactly the same steps as the CUPS setup tutorial, but the keys are in different servers.

Upon the next CUPS request, the gateway will reconnect from the old instance to the new instance. Normally a CUPS request is sent every 24 hours, but there is another way to speed this up:

Starting from v3.28.2, changing the gateway server address will force the gateway to reconnect. This is not instant - it takes on average 10 minutes after the change occurred.
If one invalidates the source LNS key, Basic Station will automatically do a CUPS lookup before the 24 hour window, during which it will pick up the new gateway server address and LNS key (which now point to the target instance).

I heavily recommend testing this between staging1 and TTSC(E), with a gateway which is not a TTIG. Most gateways on staging1 are CUPS enabled, so feel free to ask for details.
Can you do this yourself and submit a Pull Request?

Can review.

@nejraselimovic: Please pick this up when you have time.

Hey, I've been working on testing this, I'm using RAK2245 gw, trying to migrate from TTSS to TTC, and I have some questions:

You haven't mentioned this in steps above but I had to change TC_KEY variable (that carries LNS key) in gateway config to the new LNS key value (of the target instance), but I haven't had to change CUPS_KEY or CUPS_URI (source instance values). Is this expected? I'm a bit confused by this and term "remote gateway control", i.e. my thoughts are, if I can change TC_KEY, I could also change URI and not mess around on TTS. Or changing TC_KEY shouldn't be required here? I think I'm missing something here but not sure what.
I disabled forwarding uplinks to Packet Broker, but I still see them being forwarded on Live data. Do you know why?

You haven't mentioned this in steps above but I had to change TC_KEY variable (that carries LNS key) in gateway config to the new LNS key value (of the target instance), but I haven't had to change CUPS_KEY or CUPS_URI (source instance values). Is this expected? I'm a bit confused by this and term "remote gateway control", i.e. my thoughts are, if I need to change TC_KEY, I could also change URI and not mess around on TTS. Or changing TC_KEY shouldn't be required here? I think I'm missing something here but not sure what. If changing TC_KEY shouldn't be required, then I'm not sure what else I could be doing wrong.

As a general remark, CUPS itself deals with the LNS configuration - both the credentials (the API key in our case), and the address (the gateway server address) are provided to the gateway as part of CUPS, and the gateway updates them and stores them 'while doing CUPS'.
In general when I configure a gateway using Basics Station I leave the LNS (i.e. TC) address and credentials empty, and provide only CUPS settings (the URL, and the API key). The gateway is then expected to retrieve the settings via CUPS. So yes, I believe the TC_KEY and TC_URI shouldn't be provided here in the first place - it should be all CUPS only.

I disabled forwarding uplinks to Packet Broker, but I still see them being forwarded on Live data. Do you know why?

There are two behaviors here:

Gateway settings are retrieved only when the gateway connects, and no changes are done while the gateway is connected.
A gateway will not be disconnected due to settings changes immediately - there is a delay of up to 10 minutes until the gateway reconnects.

A corollary of the above is that after you change the settings, some time will have to pass until the gateway disconnects, then upon connecting the new settings will be used and you can expect the packets to no longer be forwarded.


PIA training for team Privacy SME

Describe the issue
We have a new Privacy focused team member, Gary H, and we want to connect him with the right people that can help the team address our learning debt.
Additional context

Assume the course material can be done by oneself

Definition of done

Connect with Dayna B to discuss privacy courses
Connect with Mindy A. our ministry CITZ IMB Privacy SME that is working on the GCP PIA and get up to speed on what she has done/is doing (you will be collaborating together)
Take the training if possible in this sprint (we don't know enough to properly estimate, so we're going to guess)
Cap to a 3 effort for this sprint

Guessing this is blocked, Gary any updates?

Guessing this is blocked, Gary any updates?

Dayna and Mindy recommended courses to take

Check the dates, place your name to reserve a spot, make tickets for future sprints


Invalid Icon annotations in MultiBody.

Reported by dhedberg on 6 Dec 2013 11:22 UTC
Multiple models in the MultiBody library have invalid Icon annotations. The textString attribute of the Text primitives contains the macro "%=r". According to the Modelica language specification this is not a valid macro, it should probably be "r=%r".
The following models have this error:

Modelica.Mechanics.MultiBody.Parts.FixedTranslation
Modelica.Mechanics.MultiBody.Parts.BodyShape
Modelica.Mechanics.MultiBody.Parts.BodyBox
Modelica.Mechanics.MultiBody.Parts.BodyCylinder
Modelica.Mechanics.MultiBody.Visualizers.FixedShape2

Migrated-From: https://trac.modelica.org/Modelica/ticket/1376

Modified by dietmarw on 6 Dec 2013 15:35 UTC

Comment by beutlich on 28 Apr 2014 20:16 UTC
attachment:T1376.patch summarizes the proposed changes.

Comment by beutlich on 16 May 2014 20:10 UTC
Resolved by 00f237d11976a81d827dcbfe5ec1ed94d7aea116 and cfbf5eedf67a9ad1986814934ca4809c780fecec.


pyflakes

Adds pyflakes. Just treating everything as errors (severity 1), at present.

Looks fine on binder.


not authenticated

i would think i could skip authentication by providing a VAULT_TOKEN, but this doesn't seem to work.
$ safe status
You are not authenticated to a Vault.
Try safe auth ldap
 or safe auth github
 or safe auth token

could it be masked by a silent cert validation error?
$ safe vault status
Error checking seal status: Get https://sdp-vault-dev:27003/v1/sys/seal-status: x509: certificate is valid for vault, not sdp-vault-dev

$ vault capabilities /
Capabilities: [root]

Are you sure you have a $VAULT_TOKEN in your environment, and not a .vault-token file in your home directory?

@automaticgiant Have you checked into this? Thanks 👍

no, sorry. hopefully next week.

no problem @automaticgiant, just wanted to check in :)

Any update @automaticgiant ?

Sorry, no. I've switched teams. Feel free to investigate or close.
Sent from my Samsung SM-N920V using FastHub

Thanks!


[UI] Extension: Add Clilium logo SVG

Signed-off-by: Abhishek kr<EMAIL_ADDRESS>Description
Add cilium svg
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Create as many versions as needed. Let's just ensure there's no duplication and that the purpose for each type is clearly defined.


Separate workflow into own repo

The CLI should be separated into its own github repo, made into a python package and docker image.

Would then also be good to move all issues with tag 'workflow' into that repo.

Actually just kidding this should be closed when this repo imports from other repo for any workflow related things.

Closed with #136


docs(plugin-ideal-image): mention plugin changes type of imported images

Included information about changes to the types of imported support images

Pre-flight checklist

[ ] I have read the Contributing Guidelines on pull requests.
[ ] If this is a code change: I have written unit tests and/or added dogfooding pages to fully verify the new behavior.
[ ] If this is a new API or substantial change: the PR has an accompanying issue (closes #0000) and the maintainers have approved on my working plan.

Motivation

Test Plan

Test links

Deploy preview: https://deploy-preview-_____--docusaurus-2.netlify.app/
Related issues/PRs

Thanks, some users will find it useful

will do


chore: Add test for nyc --no-clean.

Verify that running nyc command1 && nyc --no-clean command2 causes
results results to be merged.

Coverage increased (+0.2%) to 96.833% when pulling b174a38c70a92d72c710e3a2d7e747a8e3066110 on coreyfarrell:test-no-clean into ca37ffa74c146b77c5fa9dc3411901e96e8c3a17 on istanbuljs:master.

Coverage increased (+0.2%) to 96.833% when pulling b174a38c70a92d72c710e3a2d7e747a8e3066110 on coreyfarrell:test-no-clean into ca37ffa74c146b77c5fa9dc3411901e96e8c3a17 on istanbuljs:master.

using snapshots for these types of tests
@coreyfarrell and I were talking about this on slack... I will try to dedicate some time to this again. I have a few branches of WIP but I keep ending up expanding the scope because it requires quite a few changes to properly use tap's test api...


DownloadFile alias

I do a lot of downloading of external files in my build scripts.
This adds a new HttpAliases class with (so far) a single DownloadFile(string url, DirectoryPath outputPath) alias which does exactly what it looks like.
Eventually there could be more HTTP related aliases here such as DownloadString

Whoops, didn't mean to base this off of the mono support... Will resend.


Add upgrade management components cmd

Issue #, if available:
Description of changes: Add "upgrade management-components" subcommand to allow users to upgrade their management components in management cluster.
Testing (if applicable): Manual test against docker and cloudstack. 1) upgrade management-components 2) upgrade the field of eksaVersion of a cluster object and observed that the clusters have been upgraded correctly. Notes, there is an issue with cluster-api 1.6.0 upgrade that requires restarting cluster-api controller to make the upgrade successful.
Documentation added/planned (if applicable):
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Bug: Can't use query size script to fix file sizes when using S3 (B2)

What happened?
When running docker compose exec zipline yarn scripts:query-size I get:
/zipline/node_modules/minio/dist/main/transformers.js:149
      var e = new errors.S3Error(message); // S3 Error code.
              ^
S3Error: Not Found
    at getError (/zipline/node_modules/minio/dist/main/transformers.js:149:15)
    at /zipline/node_modules/minio/dist/main/transformers.js:161:14
    at DestroyableTransform._flush (/zipline/node_modules/minio/dist/main/transformers.js:91:10)
    at DestroyableTransform.prefinish (/zipline/node_modules/readable-stream/lib/_stream_transform.js:129:10)
    at DestroyableTransform.emit (node:events:513:28)
    at prefinish (/zipline/node_modules/readable-stream/lib/_stream_writable.js:611:14)
    at finishMaybe (/zipline/node_modules/readable-stream/lib/_stream_writable.js:620:5)
    at endWritable (/zipline/node_modules/readable-stream/lib/_stream_writable.js:643:3)
    at Writable.end (/zipline/node_modules/readable-stream/lib/_stream_writable.js:571:22)
    at IncomingMessage.onend (node:internal/streams/readable:705:10) {
  code: 'NotFound',
  amzRequestid: null,
  amzId2: null,
  amzBucketRegion: null
}

Node.js v18.15.0
error Command failed with exit code 1.

Version
upstream (ghcr.io/diced/zipline:trunk)
What browser(s) are you seeing the problem on?
No response
Zipline Logs
No response
Browser Logs
No response
Additional Info
I'm using Backblaze's implementation of S3.

#363 Does this PR (latest commit as of now), fix any of the issues here?

Nope, after pulling and restarting my container I’m still getting the same error when I try to run this script.

Reopened, until we see a response from @NebulaBC - hope the fix works!

@NebulaBC, has the following commit (3ff2153) fixed your issue?


Support subject variable of when expression

As documented a Kotlin when expression's subject can also be assigned to a variable as in the following test case:
    @Test
    void subjectVariable() {
        rewriteRun(
          kotlin(
            """
              val x = when (val response = listOf(1, 2) as Collection<Int>) {
                  is List -> "l"
                  else -> ""
              }
              """
          )
        );
    }

Currently this parses but produces the wrong result when serialized again.

The val statement could be nested in the K.When's J.ControlParentheses, I am however unsure if that would be a good idea. The LST nesting would then look like:
K.When
  J.ControlParentheses
    K.StatementExpression
      J.VariableDeclarations
        J.VariableDeclarations$NamedVariable

What do you think, @traceyyoshima ? FWIW, in the FIR model the FirWhenExpression directly owns a FirVariable as the subject variable.

I've pushed a commit to allow parsing (and printing) sources using this construct. That leaves us more time to decide on the LST model.

What do you think, @traceyyoshima ?
FWIW, in the FIR model the FirWhenExpression directly owns a FirVariable as the subject variable.

I agree with your decision, a K.StatementExpression is fine for now. We should focus on idempotent printing so that we have tests in place before making major changes. I suspect we'll be able to assess how to approach implementing a data-flow (only exists for J) compatible K tree after idempotent prints are accomplished.
I worry about the time and complexity to achieve idempotent prints if we handle all the compiler de-sugaring without tests in place. Since the backend of the Kotlin compiler that occurs after the FIR is converted to the IR (additional transformations to the tree to work on the JVM) is complicated.

@traceyyoshima I am also wondering right now how this all relates to Java's pattern matching switch expression. Possibly it can be mapped directly to that? Although I don't know right now if Java has an equivalent to the subject variable.

AFAICT Java's switch expression doesn't have any equivalent for the subject variable. So I guess we must stay with K.When.

@traceyyoshima If we were to change K.When#selector from ControlParentheses<Expression> to ControlParentheses<TypedTree> (or ControlParentheses<J>) then it would also work when a subject variable is declared.
Using ControlParentheses<J> is probably safer. I assume there is a good reason that Expression doesn't extend TypedTree, but I don't know what it is. But there are a few types in K which implement only Expression and not TypedTree. This should probably be fixed at some point, but I don't see this as urgent.
I will prepare a PR. The only thing that changes is that then there is no longer any intermediate K.StatementExpression element below the J.ControlParentheses.


Refactoring/podcast testing

This pull request aims to add some testing for the Podcast Controller and service as well as to refactor any of the code that may change to fit the test cases
Acceptance Criteria:

[x] Write tests for the PodcastController and podcastService
[x] Refactor existing code to be more in line with test expectations

Waiting for the other podcast pr to go through before merging this one


fitBounds issue calling couple times in a row.

When you call two or more times in a row the method map.fitBounds something inside gets broken. The values of the projection are OK, but the tile layers, overlays and animations were paused and continues with the old offset. I propose a debouncing method on fitBounds, that was my solution.
Best regards.

This is a known issue. Did you try master version?

I'm resolving an issue for production in a project. Only stable releases are allowed. I'll try it later

See also #2021. The fix was backported to stable branch recently, try that out too.


InvalidStateError

(Emailed in):

We are getting this error when trying to use the Download all Attachments app, please advise.
Error: InvalidStateError: Failed to read the 'responseText' property from 'XMLHttpRequest': The value is only accessible if the object's 'responseType' is '' or 'text' (was 'arraybuffer').
We are using Chrome. It happens on all tickets and all users. The only change we made recently was moving from one domain to a new domain. We went from redacted to redacted.
I have already tried to remove the app and reinstall with the same results.

See #11


Adds Watches on NADs

Adds to watch NADs so that the controller reconciles when the configured NAD changes.

Actually I am not sure we need it, the situation I tested was to deploy without the configured NAD to exist. When creating the NAD the controller did not reconcile after the timeout and realized that there is a warning logged that it returns an error and a non-zero result.
The issue for this is that we return the RequeueAfter and an error [1].  When the reconciler returned both a non-zero result and a non-nil error. The result will always be ignored if the error is non-nil and the non-nil error causes reqeueuing with exponential backoff [2], which can be long. So the RequeueAfter is ignored right now.
We could just change to not return the error in this case and the RequeueAfter is not ignored. (we should check this for other situations as well. seems some operators do that a lot) Or remove the RequeueAfter and add this watch, but that might be to much because in general when there is a change on the NAD, like different IP or some other detail it would not be reflected in the annotation, so a reconcile won't be useful. Manual interaction to delete the deployment would be required in that case, or some further logic to delete the deployment automatically, which I am not sure right now we want to have.
[1] https://github.com/openstack-k8s-operators/placement-operator/blob/main/controllers/placementapi_controller.go#L609
[2] https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/reconcile#Reconciler

OK, if the issue is the exponential back-off and not a missing reconcile (just a severely delayed one) then I suggest to drop the error from the return, keep the requeueafter, and log the error before return.


Final issue topic + versioning

In this issue, discuss any last minor details that can quickly be resolved. If there's nothing notable, get ready to make a versioned release, and choose the scope: v1.0.0 or v0.1.0, perhaps something different? I will be making the release and closing the issue by the deadline of 6pm if nobody else does it before me.

Since version is in form MAJOR.MINOR.PATCH, v1.0.0 is a better option

I see, but I think this shouldn't be considered a full release because it doesn't have all the features we planned it to have. Also, optionally alpha/beta can be used to signify it's current status. My vote is on "alpha 0.1.0"

I see, but I think this shouldn't be considered a full release because it doesn't have all the features we planned it to have. Also, optionally alpha/beta can be used to signify it's current status. My vote is on "alpha 0.1.0"

I see no problem with not having all features that we wanted, but I can agree with your option

We could also just not make a manual release. Its not part of the assignment.

We could also just not make a manual release. Its not part of the assignment.

And there is no reason to wait for decision on making a manual release, as the assignment requires a link to GitHub to be submitted

We could also just not make a manual release. Its not part of the assignment.

And there is no reason to wait for decision on making a manual release, as the assignment requires a link to GitHub to be submitted

The links are already submitted. Submitted it this morning.

It is true that this is not required, however in my honest opinion it would be much nicer and tidier if we wrapped up the project in a meaningful way, it doesn't need much work, and it might even be worth half a point.

It is true that this is not required, however in my honest opinion it would be much nicer and tidier if we wrapped up the project in a meaningful way, it doesn't need much work, and it might even be worth half a point.

Maybe annotate the latest automatic release with a version number then.

I ended up naming it alpha v1.0.0 by mistake, but I think it's fine this way :smile:


creating embedding for categorical feature?

In the embed.py file, we compute the sum of the value embedding, position embedding and temporal embedding. Suppose for one of my time series features I have a categorical variable (e.g. I am trying to predict the sales of 10 shops and I have a variable "shop" that takes values among 0,1,2,3,4,5,6,7,8,9 i.e. it is a categorical variable).
Does it make sense if I define categorical embedding module in the embed.py file and calculate the categorical embedding for my variable "shop" and sum it along with value embedding, position embedding and temporal embedding? Thank you @zhouhaoyi

embed.py   你可以在这个文件中增加一个类别嵌入模块  然后计算分类变量的嵌入  并将其与值嵌入  位置嵌入和时间嵌入相加  得到最终嵌入

Hello, I want to ask you the question of categorical variable. Did you try it according to your idea? What was the result

When multiple stores are combined, the dates in the sample data become 1,1,1,2,2,2; I don't know if we can still use time features like this

@puzzlecollector Hello, Can you look at your modified source code


Build and publish Electron packages automatically

…using GitHub Actions

https://github.com/marketplace/actions/electron-builder-action

https://github.com/marketplace/actions/electron-builder-action

This has been deprecated/archived

Another time


chore(fluentd): remove fluentd_pre and fluentd_new

Remove because it is outdated and the single fluentd setup is easier to use.
Ref: https://github.com/dsaidgovsg/terraform-modules/pull/255

Will classify this as a breaking change, since it removes TF modules that potentially others could have already use.


Remove can-i-get-a-british-passport Smart Answer

Added on 7th Mar 2012 - ae39fa485477592718df060b49f054c691788c41
Last updated on 7th Mar 2012 - 1f27b1d19dad7e86ed161bcce49c7fb321930cfe
Never published
No associated test

Gwen and Alex have confirmed that this can be removed.

My motivation for opening this pull request is to run the tests and to see whether people spot anything else that can be removed in this branch.

LGTM. I can't see anything else which can be removed.

:shipit:


object/sftp: unescape username and password

reference https://github.com/juicedata/juicefs/discussions/2962

Codecov Report
Base: 58.76% // Head: 58.85% // Increases project coverage by +0.09% :tada:

Coverage data is based on head (d3ae71f) compared to base (2731d9e).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2981      +/-   ##
==========================================
+ Coverage   58.76%   58.85%   +0.09%
==========================================
  Files         141      141
  Lines       24605    24607       +2
==========================================
+ Hits        14458    14483      +25
+ Misses       8263     8246      -17
+ Partials     1884     1878       -6

Impacted Files
Coverage Δ

pkg/object/sftp.go
51.58% <100.00%> (+0.30%)
:arrow_up:

pkg/meta/tkv_mem.go
86.61% <0.00%> (-0.71%)
:arrow_down:

pkg/vfs/handle.go
85.56% <0.00%> (ø)

pkg/meta/redis.go
69.79% <0.00%> (+0.04%)
:arrow_up:

pkg/chunk/cached_store.go
73.89% <0.00%> (+0.18%)
:arrow_up:

pkg/meta/sql.go
64.07% <0.00%> (+0.23%)
:arrow_up:

pkg/vfs/reader.go
78.99% <0.00%> (+3.57%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Package for debian

would be nice to get some help with this

I am no debian user, but here is what I found:
Other bitmap fonts:

M+
Dr. Manop Thai fonts
Bitmap Mule
JIS X (kanji)
Unifont

For mosts, it is /usr/share/fonts/X11/misc/<fontname>.pcf.
This blog post said that a debian package is built by running the command dpkg -b ~/tewi-font/ tewi-font_<version>_all.deb.
~/tewi-font:

The directory structure to be installed in the debian system of the user from its root dir.  For unifont, it is:
usr
└── share
    ├── doc                             <- not required?
    │   └── xfonts-unifont
    │       ├── changelog.Debian.gz
    │       ├── changelog.gz
    │       ├── copyright
    │       └── README.gz
    └── fonts
        └── X11
            └── misc
                ├── unifont_csur.pcf.gz
                ├── unifont.pcf.gz
                └── unifont_sample.pcf.gz

debian-binary: for xfonts-unifont a file with a single line: 2.0.

DEBIAN directory with in it a few files that may be built using helpers (5.- Building the Debian files skeleton):

control.  For xfonts-unifont:
Package: xfonts-unifont
Source: unifont
Version: 1:7.0.06-1
Architecture: all
[...]

md5sums,

Post-install scripts if needed.

This would be great! I use Testing, and if I can help out in any way, please let me know. I can attempt to build a package if you would like.

Hello everyone,
I've packaged it for debian and made a repository: https://github.com/helvetie/dpkgs.
The package itself is solid, but the apt repository might not be — I'm tempted to add multi-arch packages, which might change the hierarchy and make your apt complain ; I'm not sure.
Still, you'll always be able to get the debianized sources from the master branch, and I won't touch the apt repo carelessly.
Thank you!


Fixed delete test!!

So we now just select all and hit back space. Hacky but it works.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #13   +/-   ##
=======================================
  Coverage   77.97%   77.97%
=======================================
  Files          17       17
  Lines         336      336
  Branches       17       17
=======================================
  Hits          262      262
  Misses         69       69
  Partials        5        5

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 fcd1e5a...4391308. Read the comment docs.


Flaky: Various failures due to PastHorizonException from DB

Context

We saw a sharp uptick in failures when bumping the node to include the Allegra era. There were PastHorizonExceptions from the DB in many of the failures.
Slack thread: https://input-output-rnd.slack.com/archives/GBT05825V/p1607339108011300
Test Case
Multiple. A lot of the times: migration tests of big wallets.

Failure / Counter-example
First we se these in the normal test output:
PastHorizon {pastHorizonCallStack = [("runQuery",SrcLoc {srcLocPackage = "ouroboros-consensus-<IP_ADDRESS>-3JXm5ogVkRThJO19RF1Tj", srcLocModule = "Ouroboros.Consensus.HardFork.History.Qry", srcLocFile = "src/Ouroboros/Consensus/HardFork/History/Qry.hs", srcLocStartLine = 426, srcLocStartCol = 44, srcLocEndLine = 426, srcLocEndCol = 64}),("interpretQuery",SrcLoc {srcLocPackage = "cardano-wallet-core-2020.11.26-Eht2UPkACzpCQC56U01U3O", srcLocModule = "Cardano.Wallet.Primitive.Slotting", srcLocFile = "src/Cardano/Wallet/Primitive/Slotting.hs", srcLocStartLine = 368, srcLocStartCol = 15, srcLocEndLine = 368, srcLocEndCol = 34}),("interpretQuery",SrcLoc {srcLocPackage = "cardano-wallet-core-2020.11.26-Eht2UPkACzpCQC56U01U3O", srcLocModule = "Cardano.Wallet.DB.Sqlite", srcLocFile = "src/Cardano/Wallet/DB/Sqlite.hs", srcLocStartLine = 856, srcLocStartCol = 25, srcLocEndLine = 856, srcLocEndCol = 82})], pastHorizonExpression = Some (ELet (ERelSlotToEpoch (EAbsToRelSlot (ELit (SlotNo 1083)))) (\x0 -> EPair (ERelToAbsEpoch (EVar x0)) (ESnd (EVar x0)))), pastHorizonSummary = [EraSummary {eraStart = Bound {boundTime = RelativeTime 0s, boundSlot = SlotNo 0, boundEpoch = EpochNo 0}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 20s, boundSlot = SlotNo 80, boundEpoch = EpochNo 1}), eraParams = EraParams {eraEpochSize = EpochSize 80, eraSlotLength = SlotLength 0.25s, eraSafeZone = StandardSafeZone 16}},EraSummary {eraStart = Bound {boundTime = RelativeTime 20s, boundSlot = SlotNo 80, boundEpoch = EpochNo 1}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 60s, boundSlot = SlotNo 280, boundEpoch = EpochNo 2}), eraParams = EraParams {eraEpochSize = EpochSize 200, eraSlotLength = SlotLength 0.2s, eraSafeZone = StandardSafeZone 60}},EraSummary {eraStart = Bound {boundTime = RelativeTime 60s, boundSlot = SlotNo 280, boundEpoch = EpochNo 2}, eraEnd = EraEnd (Bound {boundTime = RelativeTime 220s, boundSlot = SlotNo 1080, boundEpoch = EpochNo 6}), eraParams = EraParams {eraEpochSize = EpochSize 200, eraSlotLength = SlotLength 0.2s, eraSafeZone = StandardSafeZone 60}}]}

and finally:
 src/Test/Integration/Scenario/API/Shelley/Migrations.hs:214:15:
  1) API Specifications, SHELLEY_MIGRATIONS, SHELLEY_MIGRATE_01_big_wallet -  migrate a big wallet requiring more than one tx
       expected: Status {statusCode = 202, statusMessage = "Accepted"}
        but got: Status {statusCode = 500, statusMessage = "Internal Server Error"}

       from the following response: Left (DecodeFailure "Something went wrong")

       While verifying (Status {statusCode = 500, statusMessage = "Internal Server Error"},Left (DecodeFailure "Something went wrong"))

  To rerun use: --match "/API Specifications/SHELLEY_MIGRATIONS/SHELLEY_MIGRATE_01_big_wallet -  migrate a big wallet requiring more than one tx/"

  src/Test/Integration/Scenario/API/Byron/Migrations.hs:277:15:
  2) API Specifications, BYRON_MIGRATIONS, BYRON_MIGRATE_01 -  migrate a big wallet requiring more than one tx
       expected: Status {statusCode = 202, statusMessage = "Accepted"}
        but got: Status {statusCode = 500, statusMessage = "Internal Server Error"}

       from the following response: Left (DecodeFailure "Something went wrong")

       While verifying (Status {statusCode = 500, statusMessage = "Internal Server Error"},Left (DecodeFailure "Something went wrong"))

  To rerun use: --match "/API Specifications/BYRON_MIGRATIONS/BYRON_MIGRATE_01 -

Resolution
Two options:

Lower test parallelism in CI (DONE)
Ensure only one or a few chain-sync connections are used for all wallets

QA

Solved through

Lower test parallelism in CI


looks like it isn't making tables in database, is there any way to force make it ?

✓ Starting...
(node:58) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
(Use node --trace-warnings ... to show where the warning was created)
✓ Ready in 386ms
[ERROR] [DB] Error while getting setting: Error: Table 's21_cs_website.sa_mutes' doesn't exist
[ERROR] [DB] Error while getting the admin: Error: Table 's21_cs_website.sa_admins' doesn't exist
[ERROR] [DB] Error while creating sa_admins: Error: Table 's21_cs_website.sa_admins' doesn't exist
[ERROR] [DB] Error while creating tables: Error: Table 's21_cs_website.sa_admins' doesn't exist
[ERROR] [DB] Error while getting all servers: Error: Table 's21_cs_website.sa_servers' doesn't exist
An aria-label or aria-labelledby prop is required for accessibility.
An aria-label or aria-labelledby prop is required for accessibility.
[ERROR] [DB] Error while counting admins: Error: Table 's21_cs_website.sa_servers' doesn't exist
[ERROR] [DB] Error while getting all bans: Error: Table 's21_cs_website.sa_bans' doesn't exist

getting same error

Use this: https://github.com/CSSPanel/Plugin/blob/main/Database/database_setup.sql
Or install the plugin onto your server instead :D

Issue is resolved when you install the plugin first


How to train basicvsr on vimeo90k with LMDB?

Hi, the disk quota is 1 million files per person, so I have to use LMDB for training. However, I met some errors and there seems no template for LMDB training. Can you help me with it?

Could you provide us with the error log?

@ckkelvinchan this is a feature request instead of a bug report 😄

I paste the error log as well as config file

I paste the error file as well as config file

Did you encounter any problems when running EDVR with LMDB?

Did you encounter any problems when running EDVR with LMDB?

Sorry, I did not run EDVR with LMDB. Is my path to LMDB files correct?

Did you encounter any problems when running EDVR with LMDB?

Sorry, I did not run EDVR with LMDB. Is my path to LMDB files correct?

I am afraid not. You should follow the instruction here to generate the lmdb files. The filename should end with .lmdb.

Closing due to inactivity. Please reopen if there are any further problems.


esp32: add support for esp-idf bootloader, XIP execution and linker script updates

This commit is part of the initial ESP32 Development work.

@sylvioalves could you please rebase this on top of the latest master and force push?

@sylvioalves could you please rebase this on top of the latest master and force push?

Sure. However, there too many check errors that I need to handle, most with code style.

@sylvioalves could you please rebase this on top of the latest master and force push?

Sure. However, there too many check errors that I need to handle, most with code style.

Thanks! I fixed the GitHub action that posts the manifest update comment now.

@carlescufi Build check won't pass until this PR gets merged.

PR

I've commented on that PR, there's something wrong with the commit set (8K) in there, you probably need to rebase it.

Hi, hal_espressif repo is in place and supporting this PR.
Any extra reviews are welcome.

@nashif, for the latest update:

python env is not necessary anymore. It means west does not need reinstalling.
fixed windows cmake issues regarding the python clean up
updated getting started document describing OSes configuration

For esp32 bring up, one's need only to run
west update
west espressif update
and export necessary environment variables as described in the getting started tutorial.

@ExtremeGTX @dcpleung, can you please also review?

@ExtremeGTX @dcpleung, can you please also review?

@ExtremeGTX @dcpleung, can you please also review?

Unfortunately I'm on vacation now, I will come back on 11 Jan. and will review the changes if it is not yet merged.

@ExtremeGTX @dcpleung, can you please also review?

Unfortunately I'm on vacation now, I will come back on 11 Jan. and will review the changes if it is not yet merged.

@nashif @carlescufi @mbolivar-nordic I have seen the slack channel talking about west espressif/sof update regarding submodules and binaries. WIll that have any impact on this PR? How can we move on on this?

@nashif @carlescufi @mbolivar-nordic I have seen the slack channel talking about west espressif/sof update regarding submodules and binaries. WIll that have any impact on this PR? How can we move on on this?

@pabigot this new push includes:

added esp32_rom prefix in all zephyr's calls. Future esp releases already have this esp_rom prefix and we shall update linker file soon.
Squashed 3rd commit (fix clock) into 1st commit so that DTS changes won't take place as it introduced a "bug".

@pabigot this new push includes:

added esp32_rom prefix in all zephyr's calls. Future esp releases already have this esp_rom prefix and we shall update linker file soon.
Squashed 3rd commit (fix clock) into 1st commit so that DTS changes won't take place as it introduced a "bug".

@tejlmand @nashif  please revisit.  (Did we loose "re-request review" functionality in github?  I couldn't find how to do that.)

@tejlmand @nashif  please revisit.  (Did we loose "re-request review" functionality in github?  I couldn't find how to do that.)


Alanren/selectbox accessibility

fix for the accessibility defect: http://sqlbuvsts01:8080/Main/SQL Server/_workitems#_a=edit&id=11512553

not sure how the extra commits is showing up in this PR, I will recreate one


faster ascii check

with 4096, 50k iterations, Skylake-EP
# before:
validate_ascii_fast(data, N)   :  0.102 cycles per operation (best)    0.118 cycles per operation (avg)
# after:
validate_ascii_fast(data, N)   :  0.086 cycles per operation (best)    0.101 cycles per operation (avg)

I think this is correct... unless a zero byte is considered not valid ASCII.

Also, rdtsc counts reference cycles, not core clock cycles, so it's not really a correct benchmark. For comparing two implementations it's okayish (assuming they don't affect the core clock speed differently).

Just realized the unit test doesn't test validate_fast_ascii, so I haven't actually tested this. Need to run right now, but will come back to this.

The benchmark is correct on a properly configured server. It is definitely not correct on any random machine.
I’ll wait for your test. Thanks.

You can easily disable OS-controlled frequency scaling, but I'm not sure that it's possible to totally override the throttling internal to the CPU that's governed by thermal/electrical limits? Maybe you could peg the CPU to some low frequency (here just LFM, but AVX2 base freq if you make a 256-bit version) and correct for the difference between that true frequency, and the base frequency that rdtsc counts at?

Code was fine, added a test.

Great code.

Name added to AUTHORS.

Cool, thanks, and thanks for that commit.

I also expect Linux counters to have more overhead. So I would only use them for large tasks.

On my test machine, Linux counters agree exactly (less than 1% error) with rdtsc on sizeable arrays. See 654e660

That is probably true of the SSE implementation today.  It may not continue to be true if you move to higher-power AVX/AVX2/AVX-512 instructions.  The CPU can and does downclock to meet thermal limits under high AVX load ("Workloads using Intel AVX instructions may reduce processor frequency as far down as the AVX base frequency to stay within TDP limits").
I think @zbjornson's concern has merit, for benchmarking an eventual AVX implementation.

@cemeyer
Fair point, but some things to take into account...

 So I go by what people (ideally me) can measure.
I have never been able to measure any slowdown with AVX.
As my blog post By how much does AVX-512 slow down your CPU? A first experiment indicates, I have not even been able to measure it with AVX-512.

So though I agree with everything you guys wrote, I would urge some caution. Can you measure these slowdowns on benchmarking servers?
Note that if someone wants to collaborate with me on this, I can make available servers (by ssh).

@pcordes do you want to post your measured frequency asm test on SKL-EP with AVX and AVX-512, or mind if I post it?

@lemire your blog post says you tested on an Intel Xeon W-2104 CPU @ 3.20GHz.
That CPU is special for 3 reasons:

It doesn't support turbo so max freq = baseline all the time.
It only has one 512-bit FMA unit.
It's a quad-core (maybe based on the same ring-bus interconnect as regular Skylake-client)?

Using the port5 FMA may be what triggers throttling, so CPUs without the port5 FMA unit may never do the full AVX512 throttling.  (IDK if anyone's done better experiments to figure out exactly what triggers throttling).
AVX512 throttling only seems to go down to the rated baseline sticker clock speed, i.e. 3.2GHz for your CPU.  The higher clocks are for the common case where there's thermal / power headroom, not a guarantee, I don't think.
I'm not totally clear on this point, but I think that AVX512 can only slow your CPU down to the same minimum frequency as hitting TDP limits from running heavy 256-bit FMA code on all cores, and that frequency is the sticker frequency that a CPU advertises itself as.  i.e. the sticker frequency is a guaranteed baseline unless your CPU seriously overheats from inadequate cooling, and a different kind of throttling kicks in.
There should be a Stack Overflow chat archive where Mysticial was telling me about overclocking his i9-79xx SKX desktop.  It didn't turn up in some quick googling, but I do remember he said it had a configurable AVX512 frequency in the BIOS, but I don't remember if there was also an AVX frequency below the max turbo for workloads with no FMA or no AVX at all.

@pcordes Note that I always configure testing servers so that they have turbo disabled.

Note that I always configure testing servers so that they have turbo disabled.

That's useful for microbenchmarking, especially if you want to use RDTSC instead of perf counters, and what you want to look at for tuning / profiling.
But in real life, people run code with turbo and power-saving enabled, so it's also useful to know how fast different implementations run in wall-clock time.  e.g. in software that doesn't otherwise use AVX512 for anything, it's useful to know how an AVX512 implementation compares against an AVX2 implementation, at their respective max clock speeds.
Or with all cores loaded at whatever clock speed the CPU manages...
This is the kind of thing people need to take into account when deciding which version to use in their program on their server.  Although really that's so specific that most people should just benchmark themselves, because the % of time that different programs spend in these functions will be very different.  But it would still be interesting to see how AVX2 vs. AVX512 stacks up on the same hardware, without any artificial clock-speed limitations for either.

@pcordes
I agree with you. What I want to stress is that one needs to measure these things. Otherwise, we risk taking decisions not based on engineering facts but rather based on stuff we read on the Internet ("AVX-512 slows down your server"). That's no good. Also, Intel's documentation should not be trusted. One needs to measure and verify.

@pcordes I see you live in Halifax. I was once a professor at Acadia University nearby, and I still have friends in Halifax. Nova Scotia is the one place in the world where "brown or white bread?" does not mean what I've come to expect.


[#843] Fix Migration008 to use jsonpb Marshaling

Data stored in the DB is mashalled as a JSON-serialized protobuf and not
as binary data. This migration was failing when attempting to migrate
real data created.

Coverage remained the same at 31.68% when pulling 1b0aee7c90b29fbed721a1e8539bb06c73dba06c on 843-fix-migration-unmarshal-jsonpb into 324d82a920b75355b2a24ea57955e080def71ff7 on 843-dispute-notifications-2.


Add stub ResolveFrameworkReferences target

This will enable the VS project system to call this target to populate the Frameworks node in the solution explorer.  Microsoft.NET.Sdk will override this target (see https://github.com/dotnet/sdk/pull/3355) to provide ResolvedFrameworkReference items as appropriate.

@drewnoakes @davkean

Any idea when we'll see this inserted somewhere for integration from the project system side?


App doesn't start

Short description of the issue
App doesn't start, crashes.
Issue type
Bug
What is the expected behaviour?
App starts, no crash
What is actually happening?
When trying to start the app from Xcode on a appleTV4 via a remote debug session, the app doesn't start, it crashes.
I was also able the reproduce the behavior with the appleTV simulator, but not consistent.
Log in debug window:
`dyld: Library not loaded: @rpath/libswiftAVFoundation.dylib
Referenced from: /var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/TVGemist
Reason: no suitable image found.  Did find:
/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib: code signature invalid for '/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib'
/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib: code signature invalid for '/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib'

/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib: stat() failed with errno=1
/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib: code signature invalid for '/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib'

/private/var/containers/Bundle/Application/EF3FD657-E27B-4977-A079-B71F704D368D/TVGemist.app/Frameworks/libswiftAVFoundation.dylib: stat() failed with errno=1`

Steps to reproduce
Get latest code from GitHub, follow installation instructions, (change team, app id), press play button.
Extra information
Version you are using: 0.0.5
tvOS version: most recent, no updates available
Xcode version: 9.4

I can't reproduce the issue. I suspect this might be related to the NPO API's not responding, or responding with some invalid or missing JSON values. I have seen the NPO change their response a couple of times, which would break the abstraction, but I can't reproduce the issue you're describing at the moment. Is it working for you now?

Nope not working. I got an update of Xcode to 9.4.1 today and I was now able to create a screenshot of the error:

It still works fine for me. I do sometimes experience a crash after the first start (which I think might be caused by the logging framework), but the second start always works. You could try clearing out your derived data and see if that helps? I use this bash method:
# clean derived data
cdd() {
    while true; do
        if [ -d "~/Library/Developer/Xcode/DerivedData" ]; then
            rm -rf "~/Library/Developer/Xcode/DerivedData"
        else
            echo ".removed derived data"
            break
        fi
    done
}

Alternatively try ⌘-shift-k within xcode.

⌘-shift-k in Xcode fixed the issue. Thanks.


Reply to introduction in intros channel

Main feature contribution: DevilBot will now watch the intros channel and reply to user's introductions
Supporting contributions:
Chat Post Message now receives the tread timestamp as an Option param. Now able to receive all the replies from a message
So this is working but there are some things that I know there is probably a better way to do in rust. Specifically, I am not sure if the way I added an optional parameter to the request in chat_post_message.rs was the best way. I also think that message_event_handler.rs is getting cluttered with logic. I was thinking I should create a handler for messages in specific channels where it is necessary
@jtmichelson
closes #51

Codecov Report

Merging #56 (7aef99f) into main (b45385f) will increase coverage by 0.38%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##           main     #56      +/-   ##
=======================================
+ Coverage      0   0.38%   +0.38%
=======================================
  Files         0      13      +13
  Lines         0     257     +257
=======================================
+ Hits          0       1       +1
- Misses        0     256     +256

Impacted Files
Coverage Δ

resources/src/aws/dynamo.rs
0.00% <ø> (ø)

resources/src/commands/buns.rs
0.00% <0.00%> (ø)

resources/src/commands/introduction_reply.rs
0.00% <0.00%> (ø)

resources/src/commands/onboard_user.rs
0.00% <0.00%> (ø)

resources/src/commands/ping.rs
0.00% <0.00%> (ø)

...ources/src/event_handlers/message_event_handler.rs
0.00% <0.00%> (ø)

...src/event_handlers/reaction_added_event_handler.rs
0.00% <0.00%> (ø)

resources/src/main.rs
2.32% <ø> (ø)

resources/src/slack/chat.rs
0.00% <0.00%> (ø)

resources/src/slack/client.rs
0.00% <0.00%> (ø)

... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add count of multiple winners

If enabled "Multiple Winners" a Count behind the Name how often the Winner won would be great.

fixed in af22e0cc054e46921bb12fb00b440afbc6a637ab
old:

new:


added stringToInteger

Lots of edge cases in this one. My algo looked in good shape till they went crazy with edge cases.

Sounds familiar


[BUG] new user here, but the download just installs without asking anything and won't startup

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
the installer download just shows a window saying installing, the app is then installed, according to windows 11, but starting it doesn't do anything.
I even tried the portable version, never starts anything.
Expected Behavior
show me the programm
Steps To Reproduce
No response
Environment
- Mod Assistant Version:351
- OS:Windows 11
- Virtualized: No

Anything else?
No response

Huh.  yep, sure does.  I missed a character on a tiny bit of code that only ever runs the very first time MA starts.
All fixed, you can grab it here (it can just get installed over what is already there) : https://github.com/FSGModding/FSG_Mod_Assistant/releases/tag/v3.5.2

wow, quick response and thanks a lot.

wow, quick response and thanks a lot.


Fix code coverage reporting

Docu:

https://github.com/codeclimate/test-reporter/blob/master/examples/php_examples.md

it has fixed itself already


remove  break in select

remove  break in select

What are you done just now? Why repeat sending?
------------------ 原始邮件 ------------------
发件人<EMAIL_ADDRESS>发送时间: 2019年12月17日(星期二) 上午10:06
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [crawlab-team/crawlab] remove  break in select (#369)
remove  break in select
You can view, comment on, or merge this pull request online at:
  https://github.com/crawlab-team/crawlab/pull/369
Commit Summary
remove  in select
File Changes
 M     backend/services/log.go     (2)

Patch Links:
https://github.com/crawlab-team/crawlab/pull/369.patch
https://github.com/crawlab-team/crawlab/pull/369.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

What are you done just now? Why repeat sending?
…
------------------ 原始邮件 ------------------ 发件人<EMAIL_ADDRESS>发送时间: 2019年12月17日(星期二) 上午10:06 收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS>主题: [crawlab-team/crawlab] remove break in select (#369) remove break in select You can view, comment on, or merge this pull request online at:   #369 Commit Summary remove in select File Changes M backend/services/log.go (2) Patch Links: https://github.com/crawlab-team/crawlab/pull/369.patch https://github.com/crawlab-team/crawlab/pull/369.diff — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or unsubscribe.

I think you are receiving this message because you have subscribed / watched Crawlab repo. Please unsubscribe watching this repo to stop receiving this message.


Support AWS Kinesis as event broker

Description of Problem:
RabbitMQ, Kafra are good choices, but some architectures already rely on Kinesis.
Overview of the Solution:
Implement the support for Kinesis, use Kafka as the template for the implementation
Definition of Done:

[ ] Tests are added
[ ] Feature is documented

Thanks for the suggestion @alexissmirnov! @ricwo you know more about this, could you comment on whether this makes sense?

Hi @alexissmirnov, thanks for your suggestion. Yes, I think Kinesis as an additional event broker would be a worthwhile addition. Would you be interested in making a contribution?


Compression middleware should implement http.CloseNotifier

This change allows http.Handlers which expect http.ResponseWriter to implement http.CloseNotifier to work transparently with the compression middleware.
If the underlying writer does not implement http.CloseNotifier, we return a channel that never receives a value. The semantics here is that the client never disconnects before the request is processed, and this is as close to the default behavior as we can get (i.e. when CloseNotify() is not even called to check whether the client has disconnected).
(I couldn't find any existing tests for middleware/compress.go) so I didn't add any, but please point me to the correct place if I'd missed it.)

The semantics here is that the client never disconnects before the request is processed, and this is as close to the default behavior as we can get (i.e. when CloseNotify() is not even called to check whether the client has disconnected).

Hmm this explanation is not very clear. Let me try this again:
Returning a channel that never returns is closest to the "default" behavior. http.Handlers that do not use the http.CloseNotifier interface are effectively saying that they do not care when the client disconnects early, so a channel that never returns models that behavior.


Can't restore a google account

I am doing some migrations. All was good until I hit a google_xxx account. I haven't been able to locate the error. Is it possible? Links to documentation would be great.
Note: I am raising it here because these value came out of a backup and then I tried to restore and can imagine other people having this problem. In practice what happens is that the non external accounts (ie cognito) restore but it is very hard to notice this.
My assumption is that either we need to strip values of external accounts on restore or setup the pool differently. The former seems more probable because I don't/can't change my pool without deleting it.
UPDATE: Looking at linking cognito user to federated accounts it would seem that it is not surprising we can't out of the box.
General Settings > Attributes

has email as required.

Federation > Google > Attribute mapping

email
email_verified
name
given_name
family_name
sub (--> username)

Here's my script, sample json from a backup and the error message.
➜  cognito-migrate node ./node_modules/.bin/cbr restore \
>      --file to_add.json \
>     -r xxxx \
>     -p default \
>     --pool XXXXXXX
✔ Users imported successfully to XXXXXXX

[
  {
    "Username": "google_100051227669222736914",
    "Attributes": [
      {
        "Name": "sub",
        "Value": "93085109-16ac-4997-9680-0b61dbd4771d"
      },
      {
        "Name": "identities",
        "Value": "[{\"userId\":\"100051227669222736914\",\"providerName\":\"Google\",\"providerType\":\"Google\",\"issuer\":null,\"primary\":true,\"dateCreated\":1609832494628}]"
      },
      {
        "Name": "email_verified",
        "Value": "true"
      },
      {
        "Name": "name",
        "Value": "J T"
      },
      {
        "Name": "given_name",
        "Value": "J"
      },
      {
        "Name": "family_name",
        "Value": "T"
      },
      {
        "Name": "email",
        "Value"<EMAIL_ADDRESS>      }
    ],
    "UserCreateDate": "2021-01-05T07:41:34.635Z",
    "UserLastModifiedDate": "2021-01-05T07:41:34.635Z",
    "Enabled": true,
    "UserStatus": "EXTERNAL_PROVIDER"
  }
]

cognito-migrate node ./node_modules/.bin/cbr restore \
>      --file to_add.json \
>     -r xxxx \
>     -p default \
>     --pool xxxxxx
✔ Users imported successfully to xxxxxx

cognito-migrate/node_modules/aws-sdk/lib/protocol/json.js:52
  resp.error = util.error(new Error(), error);
                          ^

InvalidParameterException: Cannot modify the non-mutable attribute identities
    at Request.extractError (cognito-migrate/node_modules/aws-sdk/lib/protocol/json.js:52:27)
    at Request.callListeners (cognito-migrate/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (cognito-migrate/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (cognito-migrate/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (cognito-migrate/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (cognito-migrate/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at cognito-migrate/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (cognito-migrate/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (cognito-migrate/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (cognito-migrate/node_modules/aws-sdk/lib/sequential_executor.js:116:18) {
  code: 'InvalidParameterException',
  time: 2021-06-18T06:40:27.805Z,
  requestId: '90f09f62-7b7e-412f-a3c4-eb6296e2f727',
  statusCode: 400,
  retryable: false,
  retryDelay: 64.47731523692524

@toddb Did you manage to find a workaround for this? I'm also struggling with the same issue. Thanks

I have just looked at the scripts/project I used for the migration. It was all pretty awful and too embarrassing to post here! It is undocumented with commented out code and looks for practical purposes one-off (although looking at it I kept running it over a period of time of the migration). I have put in some effectively pseudo code to help. It was painful.
From what I can tell.

I decided not to sync the google accounts into the destination environment
if people return they will be added (this works with my application design)

Good luck.
#!/usr/bin/env bash

set -e

# generate restore json files for each environment
node ./node_modules/.bin/cbr backup -r ap-southeast-2 -p default --pool ap-southeast-2_src --dir .
node ./node_modules/.bin/cbr backup -r ap-southeast-2 -p default --pool ap-southeast-2_dest --dir .
# diff the two files to return on non-google with an email address (note: not using phone)
node ./diff.js
# add new addresses without google
node ./node_modules/.bin/cbr restore \
    --pwdModule ./cognito-migrate/pwd_module.js \
    --file to_add_cognito.json \
    -r ap-southeast-2 \
    -p default \
    --pool ap-southeast-2_dest

const fs = require('fs');

export function readPool(poolId) {
    const data = fs.readFileSync(__dirname + '/ap-southeast-2_' + poolId + '.json');
    return JSON.parse(data);
}

const src = readPool('ap-southeast-2_src');
const dest = readPool('ap-southeast-2_dest');

let emailPredicate = x => x.Attributes.filter(x => x.Name === 'email')[0].Value;

const prodEmails = dest.map(x => emailPredicate(x));

const nonGoogleAccounts = src.filter(x => !x.Username.includes('google'));

const toAddCognito = nonGoogleAccounts.filter(x => !prodEmails.includes(emailPredicate(x)))

fs.writeFileSync(__dirname + '/to_add_cognito.json', JSON.stringify(toAddCognito));


doc: misc enhancements to contributors

Description of issue or feature request:
Here are a few suggestions to enhance our Instructions for Contributors on RTD.
Current behavior:

No mention of Secure Systems Lab development guidelines
Too extensive (and prominent) section about DCO
Too extensive (and outdated/opinionated?) virtualenv description
Very extensive description of test logging/verbosity config

Expected behavior:

Link to SSLab development guidelines
Shorten DCO section (and optionally move down in doc) (see e.g. in-toto Instructions for Contributors for inspiration)
Just recommend usage of and link to existing virtualenv documentation
Shorten description of test logging/verbosity config

Shorten description of test logging/verbosity config

#1724 is going to make tox run a reasonable approximation of CI tests so we should suggest that as the way to run tests.
Then a short mention on how to run specific tests and the options (-v for logs, --dump for repository dumps when available)


fix(panic): fix panic when censoring utf-8

Summary
https://github.com/TwiN/go-away/issues/43
Modified indexToRune to correctly count runes up to the given byte index. Adjusted Censor function to handle multi-byte characters properly, ensuring accurate index calculations during the censoring process. Updated incrementation of currentIndex to consider byte length, facilitating correct string iteration.
Checklist
Tested and/or added tests to validate that the changes work as intended, if applicable.
Updated documentation in README.md, if applicable.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.52% :tada:

Comparison is base (8607c2b) 99.47% compared to head (cf3403d) 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##           master       #64      +/-   ##
===========================================
+ Coverage   99.47%   100.00%   +0.52%
===========================================
  Files           1         1
  Lines         191       191
===========================================
+ Hits          190       191       +1
+ Misses          1         0       -1

Files Changed
Coverage Δ

goaway.go
100.00% <100.00%> (+0.52%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

This is great, @TwiN what needs to be done to get this merged in?

@shinroo Sorry about the delay!

@shinroo Sorry about the delay!

No worries, thanks for the quick action. Great library you've got here!


Replace "/" with "|" for initial animation node names.

This PR  replaces "/" with "|" for the initial node name.  A "/" is not allowed in the node name, so when an animation which includes "/" in its name is added to a state machine, the add fails.  The "/" in the animation name is easily encountered by loading an animation library which builds the animation name as library_name/animation_name.
With the above replacement.  An animation with a "/" in the name can be added.  The initial name for the node is simply replaced.
Fixes #60959

The solution with replacement name seems a bit hackey to me.
I think the better way to go is to change the internal path of the StateMachine to not use ”/”, or to store the animation name separately from the internal path.

I'll look into alternatives.  From what I could see the name is already split out.  The node is passed a ref to the actual animation so the replacement is just changing the name of the node,  not the actual animation. The animations in the player stays the same, just new nodes gets the new naming scheme. The animation it is associated with is still the same because of the ref.  It just ensure there isn't an invalid name when the animation is initially added.

You are able to rename the node afterwards just fine (assuming the AnimationTree isn't active....but that seems to be a separate pre-existing issue).  Renames on alpha8 and my build behave the same with the exception that initial node name prevents invalid "/".  Looks at other parts of the code it looks like "." is also bad to have in a name.  You can't rename a node to have a "." for example.

If it passes a reference to the actual animation to the AnimationStateMachine, then there is no reason for the AnimationStateMachine to disallow "/". If there is no problem when that limitation is removed, then just remove the limitation.

I'll keep understanding/investigating it.

I got a chance to look into this a bit more.  The node uses the animation name as the initial name for the node.  The node has a reference to the actual animation as mentioned in an earlier comment.  Right now, you can't use the animation because of the / in the name.
Allowing / in node names isn't easy.  The / is used to split up paths between states.  The nodes are also exposed as properties which are also split up by / so it starts to sound like a bad idea to allow / in node names.  I could be done, but I don't think there is much benefit to doing it and would need a better justification.
The solution I've liked the most so far is to just split the animation name on / and take the last component.  For example, "AnimationLibrary/Walk" would become "Walk" when added.  You can still find the full animation name on the properties for the node.  I don't believe this breaks anything, since you can't have / in existing names (it would already be broken).  It fixes the issue when an animation has a / in its name and is then used as the initial name for the node.
Thoughts? I can push a branch and update the PR if that sounds acceptable.

I propose this solution
replace / to _

Use String.validate_node_name, like in Node.set_name.

Use String.validate_node_name, like in Node.set_name, if it has to do with Node (I'm not familiar with AnimationTree). Otherwise, we can still just remove invalid characters, this is consistent with other cases.

Nice.
@tdaven valdation fix
	String base_name = animations_to_add[p_index];
	int base = 1;
	String name = base_name.validate_node_name();
	while (state_machine->has_node(name)) {
		base++;
		name = base_name + " " + itos(base);

This looks to me like an importer problem. Would it not make more sense to change the importers to disallow / in animation names or just rename them to something else? (like _ as mentioned above)

This looks to me like an importer problem. Would it not make more sense to change the importers to disallow / in animation names or just rename them to something else? (like _ as mentioned above)

And what about the importer?
"/"
"/" - separator between library and animation.

Superseded by #62350.


[UI] [iOS] aliases not visible in Dark mode

device iphone xs
ios 14.3
Issue: when changing UI to Dark Mode aliases font do not alter colour

Fixed.
Thanks for reporting.

Fixed.
Thanks for reporting.


Add --locked to the cargo install for wit-abi.

This tells cargo to use wit-abi's checked-in Cargo.lock file, using the
versions of dependencies specified in that file.
This fixes the immediate error in WebAssembly/wasi-tools#2.

@alexcrichton Would you mind reviewing this? It just adds --locked to the cargo build to ensure that we build with the version of wit-bindgen currently known to work with wit-abi.


Use Hamcrest assertions instead of JUnit in  microprofile/messaging/core (#1749)

Part of #1749
I've already signed OCA. It's an automation problem.

[ ] Create backport 2.x

/oca-checked

/trigger


class_sz implementation for cibxkappa

Included in the soliket/cibxlensing/ folder are the likelihood and theory files that I'm using that use class_sz. This doesn't require any extra custom packages (e.g. regular cobaya is fine). The theory class just replaces the CLASS object in SOLikeT with a class_sz object and creates methods to access the specific spectra of interest.
This is from a fork that hasn't been synced with the main SOLikeT release in a while, so there are other differences too, but the above folder contains the relevant parts.

Thank you @Yogesh3. Let's plan this integration together with other halo model codes (ccl/ferrara/...)


(Sticky Weekly):  extended journal boundaries

https://discuss.logseq.com/t/show-week-day-and-week-number/12685

https://github.com/stdword/logseq13-full-house-plugin/discussions/6

[ ] alpha version (for progress only)
[ ] Strict about Week of Year (Support ISO-8601 for Europe)
[ ] beta version

[ ] As a board to display the number of days remaining until the scheduled date,
https://qiita.com/kyota/items/806da61fb8fffa34b695#今日から指定日数まで何日か計算

https://discord.com/channels/725182569297215569/775936939638652948/1079397607807402076

extended journal boundaries

Cancel


Update custom_server.md

Compatibility changes

playerCountLimit (v<IP_ADDRESS>) to maxPlayers
a2sQueryEnabled  and steamQueryPort  (v0.9.9) to a2s adress and port
Remove autoJoinable (Console shows the following error, nowhere found anything about autoJoinable. If you find something, please improve it.)

09:33:55.521    BACKEND   (E): Param "#/game/autoJoinable" is not allowed and must be removed.
09:33:55.521    BACKEND   (E): Reference in schema: "#/properties/game"

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Update controller-gen version v0.2.5

What problem does this PR solve?
What is changed and how does it work?
Check List
Tests

Unit test
E2E test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has Go code change
Has CI related scripts change
Has Terraform scripts change

Side effects

Breaking backward compatibility

Related changes

Need to update the documentation

Does this PR introduce a user-facing change?:

NONE

Codecov Report

Merging #359 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #359      +/-   ##
==========================================
- Coverage      61%   60.95%   -0.06%
==========================================
  Files          56       56
  Lines        3347     3347
==========================================
- Hits         2042     2040       -2
- Misses       1145     1146       +1
- Partials      160      161       +1

Impacted Files
Coverage Δ

pkg/utils/selector.go
51.65% <0%> (-0.83%)
:arrow_down:

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 1c6a077...ea490cf. Read the comment docs.

/merge


Add OpenTracing support for observability

The Jaeger tracer is implemented for now and it is very easy to add Zipkin but I am leaning to Jaeger more as it is a replacement of Zipkin. Until our users ask for Zipkin, we are not going to spend time on it; however, a user's contribution is welcome.
Skywalking implementation is another one that we want to do but it has more features than the OpenTracing API. Need more time to learn and it is very popular in China.
https://github.com/networknt/light-4j/issues/167

I will take a look Skywalking implementation

 Add OpenTracing support for observability


Fix Listen() bug #212

Added forceUpdateDisplay Controller property, which when true, will ignore the active check on inputs, thereby allowing code to update the value, even when active.

Thanks for these PRs, and sorry for the delay. I'll review this as soon as I can.


how add jquery and jquery-ui to nuxt3 plugin?

Describe the feature
im using nuxt.config way , but work not good.any other way.
script:[
{src:"https://code.jquery.com/jquery-3.6.0.js"},
{src:"https://code.jquery.com/ui/1.13.2/jquery-ui.js"},
]
Additional information

[X] Would you be willing to help implement this feature?
[X] Could this feature be implemented as a module?

Final checks

[X] Read the contribution guide.
[X] Check existing discussions and issues.

Would you provide a reproduction? 🙏

What are you trying, that isn't working?


Newsletters: add 68 (2019-10-16)

[x] Check for updated RCs
[x] Check for YouTube videos made public

ACK ea04b57d3e983bc552f09a673a1b14d29668b7e1
Just need to check latest RCs before merge.

Does the newsletter summary also need to be changed now that LND is a release, rather than a release candidate? It currently reads:

This week's newsletter requests testing of several release candidates, relays an update on the taproot proposal, describes a proposed increase...

Perhaps change to:

This week's newsletter announces the newest release of LND, requests testing of release candidates for Bitcoin Core and C-Lightning, relays an update on the taproot proposal, describes a proposed increase...

ACK

ACK
(pushed a small casing change to Coldcard and squashed)
Apologies for the late review!

Software from LND, C-Lightning, and Eclair
Enough to make @harding lose his hair!
Please make sure nothings out of whack
Review the taproot BIPs and provide feedback
Everything is broken, lots couldve gone wrong
Despite all of that, Bitcoins still going strong
As always, TY to @harding for authoring and the last minute updates and to @jnewbery @adamjonas and @moneyball for the reviews and feedback


New Control: Split Panel

This may already be possible, but I'm missing it:
I want a control that lets me build paneled UIs like UI Catalog more easily:

But with arbitrary horizonal and vertical panels. It's a very common console GUI idiom.
I should be able to resize the panels with either the mouse or keyboard (with well defined shortcut key idioms that would be consistent across all Terminal.Gui apps).
TAB should navigate gracefully between panels.
I should be able to add and remove panels dynamically at runtime.
Panels should have (optional) Titles (like shown above).
Unlike above, the border between panels should be a single column (or row for horizontal panels).

We'll need this for Wizard. But in that case, there's another requirement:

Borders & titles of the panels should be optional, except for a single border between panes

Like this:

Fixed by #2258


How to go fullscreen on orientation change like facebook

I want to make video fullscreen on orientation change just like facebook in the Toro facebook. The playing video must go full screen on screen in landscape mode.

@shikha7763 I saw the behaviour of Facebook app. I think they put another layer of View (start new Activity for example) for that. I will think about how I could easily gain that UX.

it will be great help if you add this feature...

see the player  fullscreen behaviour https://github.com/lipangit/JieCaoVideoPlayer

@jyh149129 please checkout latest release code base, in facebook sample (release/v2.3.0 branch)


How to achieve continuous dialogue

How to achieve continuous dialogue

working on this, will keep you updated

I have completed it.

wow. awesome!
would you mind help to push a PR ?


Unable to search for Teams as Author

There are a few threads with similar questions. I've tried all the suggestions but still running into issues.  Ive checked permissions, storage accounts, urls, etc.
I've side loaded the Author App in one team, and side loaded the user app into another team i'm a member of both teams.  I'm not able to see any Teams/groups as an author.

Hi @gruhle. I'm getting the same experience that I cant see the Teams Groups as an author, however I can see distribution groups. Did you manage to get this sorted?

You need to install the end user app into a Team for it to be listed. Do this under manage team|apps|upload custom app. You can check the TeamData table in the Azure Storage account for any teams listed in there.

I followed @bungleweed 's suggestion and my TeamData table is empty.  I cannot search for Teams or Groups.
When trying "Send in chat to everyone" I get this error:

Errors
Failed to prepare the message for sending:The orchestrator function 'SendQueueOrchestrator' failed: "The activity function 'DataAggregationTriggerActivity' failed: "Specified argument was out of the range of valid values. (Parameter 'Recipient count should be > 0. Value: 0') Actual value was .". See the function execution logs for additional details.". See the function execution logs for additional details.

Hello, having the same issue here. Any updates please?

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!


Hover tooltip module name is monospace once again

The line separator is also moved below the function signature to split regions between the docs. This is very similar to how IntelliJ displays tooltips. Adding an additional separator between the module name and function signature currently has rendering issues.
Fixes #4594
Alternative to #4615
@kjeremy @Veetaha
Note that I have semantic coloring disabled so ignore any differences due to that.

bors r+


1.2 release

Is your feature request related to a problem? Please provide an existing Issue # , or describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you are proposing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[x] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #89 (f5c0000) into 1.2 (b3059e7) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                1.2      #89      +/-   ##
============================================
- Coverage     72.03%   72.02%   -0.01%
+ Complexity     2948     2947       -1
============================================
  Files           376      376
  Lines         18723    18723
  Branches       1432     1432
============================================
- Hits          13487    13486       -1
  Misses         4650     4650
- Partials        586      587       +1

Impacted Files
Coverage Δ

...performanceanalyzer/collectors/StatsCollector.java
92.48% <100.00%> (ø)

...ceanalyzer/rca/store/rca/hotshard/HotShardRca.java
86.13% <0.00%> (-1.00%)
:arrow_down:

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 b3059e7...f5c0000. Read the comment docs.


Setting up development environment results in  circular import errors

Bug/Issue Description:
Get circular import errors below when trying to run main.py to modify code and to debug in IDE.
ImportError: cannot import name 'ndarray' from partially initialized module 'numpy' (most likely due to a circular import) (./DVR-Scan/venv/lib/python3.8/site-packages/numpy/__init__.py)
Required Information:
git clone https://github.com/Breakthrough/DVR-Scan.git
python3 -m build
This builds successfully and produces the dvr-scan files however when I try to run the code directly so I can attach a debugger I get the following errors.
source venv/bin/activate
pip install -r requirements.txt
PYTHONPATH=. python dvr_scan/__main__.py
Produces error
ImportError: cannot import name 'ndarray' from partially initialized module 'numpy' (most likely due to a circular import) (./DVR-Scan/venv/lib/python3.8/site-packages/numpy/__init__.py)
Same thing happens in VSCode and PyCharm
Expected Behavior:
Execution of dvr-scan.
Computing Environment:

OS: [e.g. Windows, Linux (Distro: Ubuntu, Mint, Fedora, etc...), OSX]: Linux Ubuntu 20.04
Python Version: [e.g. 3.6 or 3.6.6]: 3.8
OpenCV Version: [e.g. 3.4.1]

pip list installed
Package               Version
--------------------- --------
appdirs               1.4.4
attrs                 22.1.0
build                 0.9.0
click                 8.1.3
exceptiongroup        1.0.0
iniconfig             1.1.1
numpy                 1.23.4
opencv-contrib-python <IP_ADDRESS>
opencv-python         <IP_ADDRESS>
packaging             21.3
pep517                0.13.0
pip                   20.0.2
pkg-resources         0.0.0
platformdirs          2.5.2
pluggy                1.0.0
pyparsing             3.0.9
pytest                7.2.0
scenedetect           <IP_ADDRESS>
screeninfo            0.8.1
setuptools            44.0.0
tomli                 2.0.1
tqdm                  4.64.1

I could have missed some instructions but only found a little snipped on the "Downloads" page on setting up a development environment.
https://dvr-scan.readthedocs.io/en/latest/download/

To install from source, download and extract the latest release to a location of your choice, and make sure you have the appropriate system requirements installed before continuing. DVR-Scan can be built by running the following command:

python -m build

This should be fixed in the next release, thanks!


Integration::new(): allow any type implementing HasRawDisplayHandle

Previously, to create an Integration an event loop must have been passed as the first argument.
The only place where the event loop is used is in Integration::new(), where it's uset to create an egui_winit::State, which accepts a reference to any type implementing HasRawDisplayHandle.
By implementing this change, winit Windows can be used to create an integration as well, which can be useful when an event loop isn't available.
To implement this change, the raw-window-handle had to be promoted to a crate dependency

Thanks for the PR!
I'll check it out this Sat/Sun after work, so you'll have to wait a little longer.

Sorry for the delay!
I will check the code changes and merge them as they look good!
Thanks for the contribution!

Publish on 0.6.0!
https://crates.io/crates/egui-winit-ash-integration/0.6.0


nwg-hello: 0.2.0 -> 0.2.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for nwg-hello is: GTK3-based greeter for the greetd daemon, written in python
meta.homepage for nwg-hello is: https://github.com/nwg-piotr/nwg-hello
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.1 with grep in /nix/store/3nq94fify067pknpd9fy17vlam9f81ry-nwg-hello-0.2.1
found 0.2.1 in filename of file in /nix/store/3nq94fify067pknpd9fy17vlam9f81ry-nwg-hello-0.2.1

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

nwg-hello

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3nq94fify067pknpd9fy17vlam9f81ry-nwg-hello-0.2.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nwg-hello https://github.com/r-ryantm/nixpkgs/archive/38a6431bf8b6a13c9cf8905973d3082e7fd26bd0.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/38a6431bf8b6a13c9cf8905973d3082e7fd26bd0#nwg-hello

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3nq94fify067pknpd9fy17vlam9f81ry-nwg-hello-0.2.1
ls -la /nix/store/3nq94fify067pknpd9fy17vlam9f81ry-nwg-hello-0.2.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

nwg-hello
nwg-hello.dist

Maintainer pings

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for nwg-hello is: GTK3-based greeter for the greetd daemon, written in python
meta.homepage for nwg-hello is: https://github.com/nwg-piotr/nwg-hello
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.2 with grep in /nix/store/pfbl72cycnqz2vxv0dy7i50fwsc24dbw-nwg-hello-0.2.2
found 0.2.2 in filename of file in /nix/store/pfbl72cycnqz2vxv0dy7i50fwsc24dbw-nwg-hello-0.2.2

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

nwg-hello

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/pfbl72cycnqz2vxv0dy7i50fwsc24dbw-nwg-hello-0.2.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nwg-hello https://github.com/r-ryantm/nixpkgs/archive/c8568ce5535448d9c71bda1efb05a50f824b625a.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/c8568ce5535448d9c71bda1efb05a50f824b625a#nwg-hello

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/pfbl72cycnqz2vxv0dy7i50fwsc24dbw-nwg-hello-0.2.2
ls -la /nix/store/pfbl72cycnqz2vxv0dy7i50fwsc24dbw-nwg-hello-0.2.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages built:

nwg-hello
nwg-hello.dist

Maintainer pings


Fix jtsync: Do not remove all labels on update and notify members only on status change

The script returns if the label already exists. (See line 156)


SW-2159 Skrim

apply skrim to dialogs with destructive actions

example:

One question — what's a skrim? 🤔

I believe it's the modal backdrop .. which blocks the content behind, we are making it a tad blurred.

Thanks @karthikbtf, TIL. I wasn't familiar with the term and also hadn't ever heard it in the context of UI so I had no idea what it was.


feat: add send event when app move to background

Description

add background task for sending request when app move to background.
change event log to prettier format.

General Checklist

[x] Added new tests to cover change, if needed
[x] Build succeeds using Swift Package Manager
[x] All unit tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[x] PR title conforms to conventional commit style
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report
Patch coverage: 97.14% and project coverage change: +0.03 :tada:

Comparison is base (3dfe309) 94.54% compared to head (4a05b4d) 94.57%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #21      +/-   ##
==========================================
+ Coverage   94.54%   94.57%   +0.03%
==========================================
  Files          33       33
  Lines        1669     1697      +28
==========================================
+ Hits         1578     1605      +27
- Misses         91       92       +1

Impacted Files
Coverage Δ

...pendency/Clickstream/Analytics/EventRecorder.swift
98.41% <96.77%> (-0.58%)
:arrow_down:

...ckstream/AWSClickstreamPlugin+ClientBehavior.swift
96.05% <100.00%> (ø)

...ndency/Clickstream/Analytics/AnalyticsClient.swift
100.00% <100.00%> (ø)

...Clickstream/AutoRecord/AutoRecordEventClient.swift
93.18% <100.00%> (+0.05%)
:arrow_up:

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


[vx.dev] implements #138

[vx.dev] This PR implements #138, created by vx.dev.

[vx.dev]: This Vue component represents a high-fidelity prototype of a product page for a web3 official website. The page includes sections for a hero image with a call to action, a company introduction, services offered, showcased projects, client logos, an about us section, and a contact form. The footer includes copyright and a signature by the creator. The images are placeholders from Unsplash and should be replaced with actual project and client images. The design uses a dark theme with blue accents for buttons and links.

添加一些动画效果，滚动时背景也添加下错位显示效果

[vx.dev]: To meet your new requirements, I will enhance the existing code with some animation effects and a parallax scrolling effect for the background images. Here's the updated Vue component:
In this updated code, I've added CSS animations for a fade-in effect on the text and button elements within the hero section. Additionally, I've implemented a simple parallax effect on the background image that moves at a different speed than the scroll, creating a sense of depth. The .parallax class is applied to the background images, and the @keyframes parallaxScroll animation is used to create the parallax effect.
Please note that for a fully functional parallax effect that responds to the user's scroll actions, JavaScript would be required to dynamically adjust the background position based on the scroll position. However, since the instructions specify no additional script, I've provided a simple CSS-based animation to simulate the effect.

[vx.dev]: Based on the provided code and the new requirements, I will enhance the parallax effect to create a more dynamic background as the user scrolls. I will also ensure that the main content of the page is present and that the parallax effect is applied correctly. Here's the updated Vue component:
In this update, I've wrapped the background image in a .parallax-wrapper to control the overflow and positioning. The .parallax class is now applied to the image itself, which is positioned absolutely within the wrapper to create the parallax effect. The height of the .parallax-wrapper is set to 96vh to ensure it covers most of the viewport height, enhancing the parallax effect as the user scrolls.
Please note that the actual parallax scrolling effect that changes as the user scrolls the page cannot be achieved with CSS animations alone. It requires JavaScript to dynamically adjust the transform property of the .parallax element based on the scroll position. However, since the instructions specify no additional script, I've provided a CSS-only solution that simulates the effect on page load. For a true parallax effect, you would need to implement a scroll event listener in JavaScript.


Bloom's example for fast linear regression

Reference R code updated: https://github.com/jbloom22/DASH/blob/master/dash.r

Let me know if you want me to finish this one up @jvmancuso, seems like it's only minor things left!

should be good for review @mortendahl

Let's merge this guy up, great work so far! I'll create a few issues that we can pick up later.


Add a discord link to collective/host's pages

User story
User feedback:

Is there any way to put the discord link on the homepage of our host? Like there is a link to the website?

Best solution for this problem
Add it either here in the Info section of settings:

OR
As a dropdown option in the 'Contact' button
Goal
More options for people who want to contact a host/initiative

good points and questions to raise @Betree...
I hadn't heard of Mattermost until now, but I know that Discord is gaining popularity as a tool amongst a lot of different groups right now (in the states at least, also in Europe from All4Climate's feedback.
Would it be possible to have many options available in the settings and then let the page admin select which 3 they want to be displayed? For example, github is popular within the OSS community, but not so relevant to OCF's hosted groups- sometimes group admin's will get confused and create a github so that it can be displayed on their profile page, but it's likely not something they will ever use, so it just sits on their page for no real reason.

Closing in favor of a more generic approach: https://github.com/opencollective/opencollective/issues/5097


[#562][DotNet] Add Azure Storage tests to the Test Scenarios pipeline

Addresses # 562

Note: This PR depends on the PRs # XXX, # XXX and # XXX.

Description
This PR updates the Test Scenarios pipeline to add support for the new Azure Storage tests.
Detailed Changes

Added new documentation for the Integration/Dotnet project.
Added getStorageAccountConnectionVariables.yml file to gather the StorageAccount connection string.
Added Choose conditions to the root Directory.Build.props to handle preview versions.
Updated the integration.yml file including the new Azure.Storage job to handle all the Storage namespace tests.
Removed all TestCategories from CardAction.

Testing
The following images show the tests working as expected in the pipelines.

/promoted 585


Select API for CAMARA Meta-Release September

Problem description
I wonder about the list of KYC API which should be elected for Meta Release of September.
From what I understand in our discussions, few contributors/MNOs won't implement KYC Fill-In API according to local privacy policy.
Do you think this API should be elected to the September Meta Release ?
BR
Gilles

Hi @GillesInnov35 ,  all,
Thanks very much, Gills, for raising this issue.
We hope KYC Fill-in API will be included in the September Meta Release.  Are there any contributors/MNOs thinking of implementing KYC Fill-in API?
Best regards,
Toshi

Can we include Scoring #85 in the next release?
Thanks,
Kevin.

Thanks Kevin
This would be our proposal as well, to include KYC match v0.2 with Scoring #85 scope in the next meta release, and aligned with the appropiate commonalities
Regards,
Clara

Hi all,
KDDI would like to include KYC Fill-in (v0.1) in the next Fall24 meta release.
Thanks.
Toshi

Closed, as ageed at the SP meeting on 2024-10-01.


fix(Jenkinsfile): set git_branch to env.BRANCH_NAME

Our logic for discerning the git branch was operating on the assumption that it would either be remotes/origin/master or a PR branch.  In fact, if the HEAD commit on the master branch is tagged, it will resolve to tags/<tag>.
As the logic above is used in many spots, this changes over to relying on env.BRANCH_NAME, which will be master even if tagged.
Tested by porting this change manually to the master job to success: https://ci.deis.io/job/Deis/job/workflow-cli/job/master/173/
(With previous code, CI was executing logic on the assumption that it was a PR - kicking off pr test job, not building certain binaries, etc. - now we see it following the intended master path)
Fixes https://github.com/deis/workflow-cli/issues/276

Current coverage is 72.47% (diff: 100%)

Merging #277 into master will not change coverage

@@             master       #277   diff @@
==========================================
  Files            59         59
  Lines          4072       4072
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2951       2951
  Misses          797        797
  Partials        324        324

Powered by Codecov. Last update fb5e41c...18d1195

Actually, realizing this isn't particular to codecov.  Anywhere we check the git branch in a binary either 'remotes/origin/master' or <PR> needs to be updated...

So with this change, what is the expected workflow when we make a PR to workflow-cli?
If it's a matter of us uploading our reports to codecov as remotes/origin/master, would it be easier to strip remotes/origin/ before uploading?

@bacongobbler PRs will still work as they have been (you can see the job triggered by this PR here).
This just changes to using the Jenkins Pipeline-provided env.BRANCH_NAME for resolving the branch name (it will be master for master and PR-<number> for a PR) instead of using the git cli to do so.