how install ngx spped with last nginx  nginx/1.11.5

how install ngx spped with last nginx  nginx/1.11.5
thank you

Do you run into a specific problem with nginx 1.11.5?
Did you try the guide here?: https://developers.google.com/speed/pagespeed/module/build_ngx_pagespeed_from_source

not working at all i install it lot of times but now i don't ander stand what hapend
thank's i will see on google thank's

@twins16 have you been able to get ngx_pagespeed up and running? Can we close this?

It'll be great if you can put the version of pagespeed on the README and on it's site. Right I want to build it on my centos 7.2 droplet but I can seem to find the version of the master zipball I downloaded here.

I think @twins16 tried to install nginx with ngx_pagespeed module following the "Automated Install".
The title is a bit disturbing, because it is not really an "Automated Install" as a apt install nginx would be.
In fact, it only compiles both sources (which is a great improvement btw), on the other hand, you need to configure and created all files / folders, such as NGINX systemd service file to be able to start nginx server or /sites-available/ /sites-enabled/ foldder and so on.

latest pagespeed only is working with nginx until 1.13.3 ....
I already tried 1.13.4 and 1.13.5 ... dont install...

@arielesp see https://github.com/pagespeed/ngx_pagespeed/issues/1472


Issue #647 : Moving ehcache core services out of service tag

Documentation still needs work here. My previous comments were about adding the information that the high level element (with an s) is a collection of elements that have specific requirements.
It contained a type (each spelled as eache) that has been interpreted by adding ehcache in front of the description - but it still makes no sense.
So to be very specific:

A <default-serializers> element represents a collection of Serializers configured at CacheManager level.
Ehcache Serializer requires serializer type and a fully qualified class name of the Serializer.

needs to become:

A <default-serializers> element represents Serializers configured at CacheManager level.
It is a collection of serializer tags that require a type and the fully qualified class name of the Serializer.

And the same inside the XSD.
To be applied to the other documentation that relates to a collection of elements as well.


error trying to run with source code

Describe your bug here.
everytime i try to run it, it says ccould not find haxelib "linc_luajit", does it need to be installed?
Command Prompt/Terminal logs (if existing)
C:\Users\lol\Downloads\FNF-PsychEngine-0.5.1\FNF-PsychEngine-0.5.1> Could not find haxelib "linc_luajit", does it need to be installed?

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
nope

thanks


new config history created on each build

we have JobConfigHistory plugin installed
https://wiki.jenkins-ci.org/display/JENKINS/JobConfigHistory+Plugin
and every build of each multibranch project makes the jobconfighistory to save new configuration in the history
though there are no changes in the config

also created issue in jobconfighistory-plugin tracker https://issues.jenkins-ci.org/browse/JENKINS-30089

Using Jenkins JIRA issue


refactor(helm): allow for loading values from common directory for all environments

Ticket T326643
This PR is preparation for DRYing up our helmfile usage as described in: https://github.com/helmfile/helmfile/blob/main/docs/writing-helmfile.md#release-template--conventional-directory-structure
It does multiple things:

Define a default release that all other release definitions inherit from. This default loads values from both a common directory as well as the per-env directory. If a file is not found, a warning is printed, but value resolution will continue. The per-env values take precendence, objects are deep-merged, arrays are not merged, but replaced
Rename values files so that they all match the <release>.values.yaml.gotmpl pattern and can be picked up automatically
Remove the unused values for local/nginx-ingress. ingress-nginx is what is currently in use.

Currently, there are no common values defined so this PR is a no-op when it comes to diffing and applying.
I tested diffing this against local and staging where no differences could be found as expected.
Connects #235 and #236

I'm still a little bit unsure with the missingFileHandler feature, because the behaviour without it brings a nice failsafe with it. But since we want to introduce these common files anyway real soon I don't think it's a 'real' issue.

In case it makes you feel safer about merging and releasing this, we could also use the branch of this PR as the base branch for all PRs that actually deduplicate values, then merge this PR to main once all values have been migrated. This probably forces me to rebase this branch a couple of times, but I guess that's ok too.
I'd be fine with both approaches in any case.

I'm still a little bit unsure with the missingFileHandler feature, because the behaviour without it brings a nice failsafe with it. But since we want to introduce these common files anyway real soon I don't think it's a 'real' issue.

Another thing we could do is set the missingFileHandler back to Error and create an empty file in common for each entry.
This would give us:

pro: everything works the same way everywhere
pro: deployments fail early in case you do something wrong
con: lots of empty files for no obvious reason, which might be confusing in some situations

Thanks for exploring the options here, I think for now it's okay to proceed with this as it is and once we have migrated all common config we can figure out how we want this to behave


i18n(ko-KR): create components-reference.mdx

Description (required)

create components-reference.mdx

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
reference/components-reference.mdx
Localization added, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Fast pika publishing

Proposed changes:

fixes https://github.com/RasaHQ/rasa/issues/4193

Status (please check what you already did):

[x] made PR ready for code review
[ ] added some tests for the functionality
[ ] updated the documentation
[x] updated the changelog
[x] reformat files using black (please check Readme for instructions)

@wochinge

Should we also skip certain acknowledge events to make it even faster?

what do you mean?

Should we also skip certain acknowledge events to make it even faster?
what do you mean?

I read something that you can omit certain acknowledge events to make the messages faster, but can't find it atm. However, that's probably not important for this iteration.


Problem installing Mass Action Scheduler

Tried to install Mass Action Scheduler in sandbox and got this error:

(Mass_Action_Configuration__c-Mass Action Configuration Layout) The ActionId specified was invalid for ActionType QuickAction.
Mass_Action_Configuration__c-Mass Action Configuration Layout: The ActionId specified was invalid for ActionType QuickAction.

(Mass_Action_Log__c-Mass Action Log Layout) The ActionId specified was invalid for ActionType QuickAction.
Mass_Action_Log__c-Mass Action Log Layout: The ActionId specified was invalid for ActionType QuickAction.

Hi @kentallman, Release 2.0 is now available. You can install it from here.
Kindly try out the new version and see if that resolves your issue.
Thanks!

Will do. Thanks Doug.
Ken Tallman
6512 Consulting Group
6512consulting.comhttp://6512consulting.com
970-903-8818
“We make a living by what we get, but we make a life by what we give.”  Winston Churchill
On Jan 2, 2019, at 9:08 AM, Doug Ayers <notifications<EMAIL_ADDRESS>wrote:
Hi @kentallmanhttps://github.com/kentallman, Release 2.0https://github.com/douglascayers/sfdx-mass-action-scheduler/wiki/Upgrading-to-Release-2.0 is now available. You can install it from herehttps://github.com/douglascayers/sfdx-mass-action-scheduler#packaged-release-history.
Kindly try out the new version and see if that resolves your issue.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/douglascayers/sfdx-mass-action-scheduler/issues/48#issuecomment-450904638, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AcFBc5AbxgJL59R4AkNygHS9QE3YnjUJks5u_NlggaJpZM4XINQK.


Feature/fix issue string size table fields

Fix Incorrect string size on table fields #23

@HieiOne Thank you for the pull request! I need to make a couple of changes before I can merge it in. Can you please check the "Allow edits from maintainers" box on the pull request so that I can make the changes directly to your branch?

@caleblanchard thank you for your fantastic work on this feature, I see that the check is marked for this pull request and you added 2 commits.
Is there something else you need from my side?


updated with links to TN overviews

am covered under google CLA

Closing as these links where included in commit https://github.com/google/TensorNetwork/commit/414f15b2782f9615ca3a4bbf16e9139b3c4a442d


Forward certs and Authorization

First, apologies for mixing two branches together.
This pull request covers the forwarding of certificate information to back-end servers when using Traefik as an SSL termination point.  We have services that need to authorize users, however we are using Traefik as the entrypoint into the system due to it's Marathon integration.  It creates several X-Headers that pass along the Base64 encoded cert, the IDN, the SDN, and the IP address/hostname of the system making the request.
This also covers a start to an authentication module that I know as being discussed in a ticket.  I created an interface and two basic modules: none and basic HTTP.  This was necessary because we wanted Traefik to talk to a proprietary service that stores user information and authenticate accordingly.  The interface assumes that the authentication modules are configured using JSON as it was a better choice that TOML in terms of describing the rules we are using.  Even for basic authentication it made things much simpler.
These changes have been running on a live system for several weeks and appear to be working as they should.

Closing as no activity


Make heap more space efficient

PR #163 and #170 made environment/heap management more efficient by first using hashes of UPLC values to index heap indices on the environment and second by turning the environment into a map of identifier to stack of heap indices. However, we still have an exponential growth of the heap. We need to investigate why and how this is happening. On why, most likely due to the semantics of values that carry their own environment. On how, investigate the use of standard techniques for scope handling essentially by withdrawing unused entries from the heap whenever its encompassing scope terminates.

So here are the two attempts I made but still unsuccessful.

This branch (https://github.com/runtimeverification/plutus-core-semantics/tree/hash-terms) is unsuccessful attempt to optimize the size of environment/heap. The main changes are:
The heap is now a map whose keys point to stacks of values.
Application hashes only the term part of a value, not the environment. Hence, different environments for the same term will produce a collision. For this reason we use a stack on the heap.
The problem is lookup. Simply fetching the most recent value for a given identifier both from the environment and the heap is not enough. It seems to me that since the environment is "distributed" among the various values, it would be necessary to fetch a particular hash for the identifier in the environment and not simply to look at the top of the stack.
So here is another unsuccessful attempt to optimize the environment/heap. https://github.com/runtimeverification/plutus-core-semantics/tree/ref-count The idea is to record the binding during application time and undo it after the application. Since there may be many references to heap cells, we keep a counting of these references for each heap key. When there are no more references to a cell, it is removed both from the heap and the environment.

Since there is no heap anymore (since PR #245) this issue now makes no sense.


Prepare release notes for v1.4.10

Wanted to open this to get the convo started on a new 1.4.x release as seems like there's enough to justify a new patch. Noticed @dmcgowan has opened all of the previous patch bump PRs so feel free to steal my branch and re-open 😋
As far as I can see, there's currently two outstanding PR's towards 1.4 that we might want to wait for.

https://github.com/containerd/containerd/pull/6014
https://github.com/containerd/containerd/pull/6010

cc @kevpar

Oh sweet https://github.com/containerd/containerd/pull/6014 was merged, updating.


JSON payload mismatch for presentation callback example

Hi Folks,
My team and I have recently started using the new GA Verified ID API and came across a difference in what's being returned from the API and what the docs say:
Actual:
{
  "requestId": "bae6704d-73a5-47c8-9d1c-83b2328e6c79",
  "requestStatus": "presentation_verified",
  "state": "guid-abc-state",
  "receipt": {
    "id_token": "...",
    "vp_token": "...",
    "state": "..."
  },
  "verifiedCredentialsData": [
    {
      "issuer": "did:ion:issuer",
      "type": ["VerifiableCredential", "TrueIdentity"],
      "claims": {
        "claim1": "value",
        "claim2": "value"
      },
      "credentialState": { "revocationStatus": "VALID" },
      "domainValidation": { "url": "https://did.woodgrovedemo.com/" }
    }
  ],
  "subject": "did:ion:subject"
}

Expected/Docs example
From the section starting "the following example demonstrates a callback payload..."
{
  "requestId": "e4ef27ca-eb8c-4b63-823b-3b95140eac11",
  "requestStatus": "presentation_verified",
  "state": "92d076dd-450a-4247-aa5b-d2e75a1a5d58",
  "subject": "did:ion:EiAlrenrtD3Lsw0GlbzS1O2YFdy3Xtu8yo35W<SNIP>…",
  "issuers": [
    {
      "type": [
        "VerifiableCredential",
        "VerifiedCredentialExpert"
      ],
      "claims": {
        "firstName": "Megan",
        "lastName": "Bowen"
      },
      "domain": "https://contoso.com/",
      "verified": "DNS",
      "authority": "did:ion:….."
    }
  ],
  "receipt": {
    "id_token": "eyJraWQiOiJkaWQ6aW<SNIP>"
  }
}

I was able to repro this just now against the endpoint https://verifiedid.did.msidentity.com/v1.0/verifiableCredentials/createPresentationRequest with the payload below.  Just let me know if you need any additional data.
Request Payload:
{
  "includeQRCode": true,
  "callback": {
    "url": "{{callbackUrl}}",
    "state": "guid-abc-state"
  },
  "authority": "{{authority}}",
  "registration": {
    "clientName": "Entra Verified ID Postman"
  },
  "includeReceipt": true,
  "requestedCredentials": [
    {
      "type": "TrueIdentity",
      "purpose": "Test",
      "acceptedIssuers": [
        "{{authority}}"
      ],
      "configuration": {
        "validation": {
          "allowRevoked": true,
          "validateLinkedDomain": true
        }
      }
    }
  ]
}

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

ID: c4fcc0e1-7256-01f1-3b8a-a8cf536ffd8e
Version Independent ID: 7e80041b-7f0f-f86a-a121-a1dbac207830
Content: Specify the Request Service REST API verify request - Microsoft Entra Verified ID
Content Source: articles/active-directory/verifiable-credentials/presentation-request-api.md
Service: decentralized-identity
Sub-service: verifiable-credentials
GitHub Login: @barclayn
Microsoft Alias: barclayn

@mtakemoto Thanks for posting. I have added the content author of this document to review the request.
@barclayn FYI.


Add Last-Event-ID header

@viktorklang, I was hoping for you to contrbute some tests for this incredibly hard change ...


More docs improvements

More notes on node requirements:  memory, storage (RethinkDB can take advantage of SSDs), file systems supported
New short section on BigchainDB and Smart Contracts
Added a high-level explanation of conditions (with help from @diminator )
Added short subsection on BigchainDB Integration with Other Blockchains
Added note about how one can't currently do queries via the HTTP API. I suggested they use the Python Server API for prototypes, for now
Moved "Topic Guides" later in the Table of Contents
Added a Quickstart page with instructions to setup a stand-alone node for learning or experimenting
Added a new short page about Example Apps (pointing to the bigchaindb-examples repo)
Added the setting of the replication factor to the instructions for cluster node setup
Refactored the setup instructions into four cases (all linked from the Introduction page):

stand-alone node for learning & experimenting (Quickstart)
set up and run a federation (points to the page about how to set up a cluster node, which got simplified quite a bit)
set up to contribute to the development of BigchainDB Server (points to the CONTRIBUTING.md file, which also got revised)
deploy a cluster on AWS

[Current coverage][cc-pull] is 88.27%

Merging [#393][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #393   diff @@
==========================================
  Files            18         18
  Lines          1262       1262
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1114       1114
  Misses          148        148
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [36fec60...6647d42][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bigchaindb/bigchaindb/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bigchaindb/bigchaindb/compare/36fec60d74b070388903fd184dc46e7b4fc7d5c9...6647d42829f5c3a18dbe9cb89e434fd664bbe39c
[cc-pull]: https://codecov.io/gh/bigchaindb/bigchaindb/pull/393?src=pr

:+1:

:+1:


$min/$max on long column does not work

If you use the $min or $max operator with a long value, it is safely cast to an int somewhere in the process.
Especially when using timestamps in milliseconds it's only possible to use dates near 1970.

Hi,
check the v3 branch :)

Cool, thanks for the fix! Are you planning to release it soonish?


Build wheels with cibuildwheels on GitHub Actions

Fixes #175
Ref #688
Closes #668
Closes #685
Fixes #441 and pretty much most issues that mention
 sql.h: No such file or directory
This changes default version for checkout from 4.0.0-unsupported to PEP 440 compliant 4.0.dev0, which is needed by wheels.
Needs PyPI keys for automated uploads.
Following the guide from https://cibuildwheel.readthedocs.io/en/stable/setup/#github-actions

This is probably ready, but blocked by https://github.com/pypa/cibuildwheel/pull/872

Appveyor is failing the Windows Python 3.5 tests, but it is time to drop support for 3.5 anyway, I think.  fi'm going to merge this.


build: use repository field in Cargo.toml

To allow Crates.io , lib.rs and the Rust Digger to link to it. See also the manifest.

@szabgab please push an empty commit so we can have semantic commit title from the pr


[Suggestion] Make PM in F3 available when player's muted.

Right now if you're muted, you can still recieve PMs but you can't send any. Why not make it so that muted people can still not use chat (ofc) but be able to PM with a friend in F3?

F3 was disabled during mute because muted players would use it in retaliation and secretly start spamming others.
If someone really wants to talk to a friend, they can use Skype/Steam


newly increased a switch parameter to Get-ChildItem, pass through (or not) a directory symbolic link

Q:How to find any ps1 file?
A:(Get-ChildItem / -Recurse -File ).Extension -match 'ps1'
but this command is circling in centos.
in my test,powershell exhausted memory,killed by kernel.
ls -la /proc/1
#return:
lrwxrwxrwx.   1 root root 0 5月  11 20:52 cwd -> /
#1958
@daxian-dbw
@iSazonov
@SteveL-MSFT
@jeffbi
@jaraco
Steps to reproduce
Get-ChildItem /proc -Recurse
#Dead loop
Get-ChildItem / -Recurse
#3697 create a trap for users, who use this command
Environment data
centos 7.3 1611
Name                           Value

PSVersion                      6.0.0-beta
PSEdition                      Core
BuildVersion                   <IP_ADDRESS>
CLRVersion
GitCommitId                    v6.0.0-beta.1
OS                             Linux 3.10.0-514.16.1.el7.x86_64 #1 SMP Wed Apr 12 15:04:24 UTC 2017
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0
reporter
By the chinese powershell study & research group. QQ group No. is 183173532
in 'vscode' or other open source project,in version release notes,bug reporter should be thank.
So I left a group name.

"Get-ChildItem /" read and convert to psobjects in memory all file names from file system!

I would like to exclude the proc directory, but powershell can not do it.
see #3762

@iSazonov And then some. Since Get-ChildItem follows symlinks any directory that has a symlink pointing to it will have its contents enumerated at least twice: once for the actual directory and again for each symlink that points to it.
The /proc directory is particularly problematic in that each of the process subdirectories has a symlink to the root directory, which will eventually get back to /proc guaranteeing an endless loop.
I think in the fix for #1958 I should have added additional code that prevented following symlinks for anything but the path specified with the cmdlet's -Path parameter.

@jeffbi It seems we already have old Issue for symlink loops.

@iSazonov :
about symlink loops,i check again,not found.
i found jeffbi talk remove-item symlink loops in #3674.
@jeffbi :
in my test memory full,the additional code work?
endless loop test command：
(Get-ChildItem / -Recurse -File -ErrorAction SilentlyContinue ).Extension -match 'ps1'

Closed via #3780


JDK 17 crashes when running Payara with APM agent

Please provide a brief summary of the bug
When deploying an application to Payara 5.2022.3 with the AppDynamics agent, the JVM crashes.
#
# A fatal error has been detected by the Java Runtime Environment:
#
#  SIGSEGV (0xb) at pc=0x00007ff25a01ca50, pid=178, tid=279
#
# JRE version: OpenJDK Runtime Environment Temurin-<IP_ADDRESS>+1 (<IP_ADDRESS>+1) (build <IP_ADDRESS>+1)
# Java VM: OpenJDK 64-Bit Server VM Temurin-<IP_ADDRESS>+1 (<IP_ADDRESS>+1, mixed mode, sharing, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
# Problematic frame:
# V  [libjvm.so+0x82fa50]  java_lang_String::create_from_symbol(Symbol*, JavaThread*)+0x20
#
# Core dump will be written. Default location: /opt/payara/glassfish/domains/domain1/config/core.178
#
# If you would like to submit a bug report, please visit:
#   https://github.com/adoptium/adoptium-support/issues
#

Please provide steps to reproduce where possible
No response
Expected Results
No crash
Actual Results
It crashes.
I found a similar bug report on the Jetbrains JDK 17: https://youtrack.jetbrains.com/issue/JBR-4697/Generic-SIGSEGV-at-libjvm-javalangStringcreatefromsymbol
What Java Version are you using?
OpenJDK 64-Bit Server VM Temurin-<IP_ADDRESS>+1 (<IP_ADDRESS>+1, mixed mode, tiered, compressed oops, compressed class ptrs, g1 gc, linux-amd64)
What is your operating system and platform?
Alpine Linux 3.16.2, running in Docker.
How did you install Java?
From base Docker image eclipse-temurin:<IP_ADDRESS>_1-jdk-alpine
Did it work before?
It works with JDK 11 (base image eclipse-temurin:11.0.15_10-jdk-alpine)
It works when the AppDynamics agent is disabled, or when running AppDynamics with option `-Dappdynamics.agent.disable.retransformation=true`, so it seems related to class transformation.

Did you test with other Java versions?
No response
Relevant log output
No response

fargate-jdk-17-crash.log

@karianna Yes I have, and they provided me with a workaround to limit the class transformations done by the agent.
But as I understand, the agent does not contain any native code, so whatever it does, the JVM should not crash as a result of its operation, right?


Update init.py

Hi @B4K2 ! Thanks for opening a pull request. Looks like this is intended to address #208 - currently @siwamsingh is working on that in #229 . I'm going to close this PR in favor of that one.

ohh OK thank you i an new to this github so i didnt know but thanks

@B4K2 currently i am not able to work on this issue feel free to contribute to this issue.


Doxygen instructions are incorrect

Required Info

Operating System & Version
macOS 10.12

Build System
Doxygen 1.8.12 (via brew)

The instructions for doxygen documentation state to run it via doxygen doc/Doxygen_API/Doxyfile. I believe this is incorrect as per https://github.com/IntelRealSense/librealsense/blob/a2488ebb04449da73220558e3ac91cfd5650438b/doc/Doxygen_API/Doxyfile#L784.
Suggested solutions:

Update Readme.md to state

cd doc/Doxygen_API/
doxygen Doxyfile

Update Doxyfile to

INPUT                  = include \
                         include/librealsense \
                         doc \
                         src \

You are welcomed to submit your fix in form of a pull-request.

@dorodnic, I appreciate the invitation, but considering my low level of familiarity with doxygen and librealsense, as well as my inability to test on other platforms, I will leave project docs to the maintainers. However, I can attest that both methods worked on macOS 10.12!


Blob Aggregation follow ups

remove bufferToString2
Remove usage of localStorage from gatesAllowPacking once done with feature
Enable write part of feature, update tests and snapshots for snapshot tests.

Waiting for SPO, so moving it to Next for now.


Bugfix/577

Coverage decreased (-0.1%) to 71.429% when pulling 8dbcb21d3631f9d7db31bf1b2ab2299dbf7ff5e0 on radiolips:bugfix/577 into 7ff29906e431cc8bcec09ae2afbcaa74a57809fb on troolee:develop.


[BugFix] Fix mv memory leak bug in QeProcessorImpl

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #15700
Problem Summary(Required) ：

executor.handleDMLStmt() registers the queryInfo into QeProcessorImpl. But there is no place to unregister it in PartitionBasedMaterializedViewRefreshProcessor.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

[FE PR Coverage Check]
:heart_eyes: pass : 1 / 1 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/scheduler/PartitionBasedMaterializedViewRefreshProcessor.java
1
1
100.00%
[]

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4


Support passing container tag/digest on Kubevirt Conformance plugin

What this PR does / why we need it:
When deploying Kubevirt and conformance tests from nightly builds,
if you dont pass a correspond tag of the nightly build to Kubevirt test suite container-tag flag
it will default to latest tag.
This could cause unexpected behavior on tests affecting conformance results.
As you run a specific build of the conformance plugin that relays on that build images to be used,
but 'latest' tagged images are being used instead including VM's containerdisk's images.
This PR set default container-tag to conformance binary from the time it was built,
it will make sure that the images that being used by are actually those from the nightly build.
And supports overriding the container-tag through new env var.
This PR also updates conformance plugin docs and makefile rule.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
This PR is needed in order to run Kubevirt conformance tests on kubevirt/kubevirtci
SRIOV provider check-up lane https://github.com/kubevirt/project-infra/pull/1453 https://github.com/kubevirt/kubevirtci/pull/645
Release note:

Kubevirt Conformance plugin now supports passing image tag or digest

/cc @phoracek @rmohr

/retest

@rmohr could you please take another look?


Problem with CSS library components, inconsistent behavior

Summary
The behavior related to CSS Library components available since Summer20 is inconsistent.
Doing a source:retrieve only retrieves the ".css" file
Doing a source:deploy fails if the css file is not with a ".js-meta.xml" file (surely to at least specify the API version)
Doing a mdapi:convert will produce only the ".css" file which then can't be deployed.
Definitely linked to #563 which is marked as solved in release notes (49.13.0) but ticket still open in issues list
Steps To Reproduce:
Repository to reproduce: dreamhouse-lwc
Expected result
needed files are produced for retrieve/pull/convert so that it can be deployed
Actual result
retrieved/converted component can't be deployed as-is.
Additional information
@oclif/plugin-autocomplete 0.1.5 (core)
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-help 3.2.0 (core)
@oclif/plugin-not-found 1.2.4 (core)
@oclif/plugin-plugins 1.9.0 (core)
@oclif/plugin-update 1.3.10 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/sfdx-diff 0.0.6
@salesforce/sfdx-trust 3.4.3 (core)
alias 1.1.2 (core)
analytics 1.12.1 (core)
config 1.1.8 (core)
generator 1.1.3 (core)
odaseva-dx 0.0.1 (link) C:\GitRepos\odaseva-dx
salesforcedx 49.13.0 (core)
├─ templates 49.10.0 (core)
├─ @salesforce/sfdx-plugin-lwc-test 0.1.7 (core)
├─ salesforce-alm 49.12.1 (core)
├─ custom-metadata 1.0.10 (core)
└─ apex 0.1.0 (core)
sfdx-cli 7.75.0 (core)
OS and version: windows 10 (1903)
node 12.18.4

Thanks @KevinGossentCap.  We'll fix asap.

Hello @shetzel , will the problem be solved soon?
This impacts all my projects and others have this problem as well (https://github.com/forcedotcom/cli/issues/717)
David

Same for me, this bug is very penalizing and prevents us from working efficiently.

I'm trying to get the fix for this into the next latest-rc. It's very close, and I'll let you know when you can try it in the RC.
Note that "Doing a source:deploy fails if the css file is not with a ".js-meta.xml" file" is expected behavior.  The meta file is required.  The other usecases should definitely work.

The fix for this will be in the release this Thursday.  salesforcedx v50.6.0, sfdx-cli v7.81.0.  You can try it early using the release candidate.  sfdx plugins:install salesforcedx@latest-rc

The fix for this will be in the release this Thursday. salesforcedx v50.6.0, sfdx-cli v7.81.0. You can try it early using the release candidate. sfdx plugins:install salesforcedx@latest-rc

Thank you very much, i will test as soon as it is available

The fix is in CLI v7.81.0 and salesforcedx v50.6.0


drag the layout, help me！

Boss, ask me, I am dragging the layout now. I found that the width can be adaptive, but the height is as much as it is. The width will change when the window size changes, but the height will not change. Is it something I wrote wrong or the design is just In this way, my layout is ok on the layout page now, but my full-screen preview will be reduced by half. I have searched a lot of documents and can't find it, and there is no communication group or community, so I can only try my luck here and see what the big guy will do. I won't see it, please reply if you see trouble, thank you (exhausted)!

The same problem bothers me now, have you solved it?

You can dynamically calculate the row height, record the current width after the page is rendered, and assign a value after scaling as the page width changes.


Getting translation issue in forget password page for not found email.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
2.x
Issue Description
While sending the correct email in the forget password input field. So it is showing notification and showing translation issues in the notification.
Preconditions

Admin should not be logged in.

Steps to reproduce

Go to 'Admin panel'
Click on 'forget password'
Send the correct email
Click on the send button
See the error

Expected Result
It should be shown proper notification.
Actual Result
it is showing the translation issue.
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


Check if CSP is available

This is causing tests to fail for apps that use the gem but not
on Rails 5.2, which is where CSP is introduced as a feature.

Follow up Trello card to address the overall issue.

@alex-ju thanks! Do you think this needs a Changelog entry?

@alex-ju thanks! Do you think this needs a Changelog entry?

It doesn't affect the components 'API' or how they are being displayed, so I'd say it's not necessary, but feel free to add if you want to.


create_meta_info saves yaml now

use yaml instead of comment blocks
refs: https://github.com/gilbitron/Raneto/issues/18

Excellent. Thanks.
Ready to merge after the minor package.json conflict is resolved from previous merge
Also, can you speak towards if this is a breaking change?
I am able to create new documents in YAML with the right metadata and the old files load seemingly fine, but according to what I see in the code, old documents would need to be converted

The comment blocks documents are still loaded via Raneto-Core, but when/if they are resaved, they are saved as yaml. You can take a look at https://github.com/gilbitron/Raneto-Core/blob/master/src/raneto.js#L70-L101
When a major version bump happens, the switch/case can be removed to drop support for the comment block.

OK, sounds good and makes sense.
Please resolve merge conflicts to your satisfaction so I can merge it in


Validated content doc fixes

According to the comment of https://github.com/ansible/validated-content-discussion/issues/16, changing the collection documentation.

Thanks for the review @sabre1041, you can proceed with the repository name changing.


Added a file size check in checkImageCache

As I was developing my application, I found that if an image failed to download for whatever reason (network offline, for example), the image was saved as a zero byte file. This lead to blank images in my UI.
I added a check to checkImageCache() that makes sure the cached file is greater than 0 bytes. If it is zero bytes, it throws an error to trigger another download attempt.

Does this mean the image will continuously attempt to download should the network be down ?
Maybe we should implement a network availability check?

Maybe we should use http://facebook.github.io/react-native/releases/0.34/docs/netinfo.html
Actually I think its best to do so. We can add a state property networkAvailable
That way should network ever change.. the networkAvailable property will be updated.. Only if networkAvailable is true should a download be performed.. otherwise a local file or ActivityIndicator should be loaded.
This would save power and usage from continuously calling on the network to attempt to download while the network isnt available.
Thoughts?

I think you're right. It seems that it would fruitlessly attempt to download at each new render. I will admit that I am not 100% sure that network connectivity was the reason for the 0 byte files. I just noticed that this was happening after I had been toggling network connectivity while using the iOS Simulator.
Either way, I think adding a network availability check is a smart move. I'll see if I can get around to adding this enhancement.

@nbolender  I reverted this commit as I'd rather check for the network to be available.. will see if I can look into this later this week or next weekend.

@nbolender added initial NetInfo support to master.


Duplicate Hubspot Forms with unique identifying names

What does this pull request do? Explain your changes. (required)

Specific updates (required)

Duplicate Hubspot Forms with unique identifying names

[Unknown] forms were Non‑HubSpot Forms, I created two login and stripe forms on the HubSpot side, integrated these forms since Alex didn't want to disable the Non‑HubSpot Forms function on the Hubspot side.
Fixed /register and /login client side validation
Fixed /forgot-password and /reset-password client side validation

Does this pull request close any open issues?
Fixes:
Bug: Duplicate Hubspot Forms with unique identifying names
#839
Screenshots (optional):

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.

Are these by chance related to this change, or perhaps it will be addrssed upon deploy. Just highlighting those for reference.

No, the above Non-Hubspot Forms are not related to this change, also we need to check it on the HubSpot side after this pr deploy to the production.


Scmi usb

PR contains the patches to support scmi protocol that is necessary to restart doma.

not relevant - not required for the project any more


Do two IR LEDs in different positions to create more robust signal?

To amplify (double) a signal, does it make sense to have the following configuration?

Benefits?

Stronger signal by a factor of two
More consistent light source (error cancel out I THINK)

What about the following to get some signal from 135 and 45?

Benefits:

Slightly stronger signal for low-density and high-density cultures

Another interesting configuration:

Note that the current software only allows for a single angle per PD:
[od_config.sensor_to_adc_pin]
od_angle_channel=135,0|135,3

A PD in the future could have multiple angles. How do we represent this? Part of the solution might be to expand the config syntax, see #125.
Does order matter? No

While this is an interesting idea, we are not pursuing it.


Feat(VETS-CPC-335): Dockerfiles Error Fix For All Services

Sorry Christine I was too lazy to commit after each service but the fix is there enjoy everyone!

Changing to jdk 8 for the billing service fixed their failing service


ReferenceError: Element is not defined

/react-router-dom/cjs/react-router-dom.js:209
current: PropTypes.instanceOf(Element)
ReferenceError: Element is not defined
Version: 4.4.0-beta.7

Fixed by #6607

Just merged that in. Should be released soonish.

Is this fix in new version?
I can't see it fixed in version 4.4.0-beta.7 when install it with npm.

It's NOT FIXED in  4.4.0-beta.7!!!

The issue is that rollup removes unused code. It considers adding const Element = typeof Element === "undefined" ? function () {} : Element; as unused and unneccesary for the compiled source. The issue eminates from this line:
    PropTypes.shape({ current: PropTypes.any })

Using PropTypes.any seems to trigger a PropTypes.instanceOf(Element) validation call. Since Element is undefined, it crashes.  The solution is to either create a custom validation function to define the shape of the current prop or do a simple patch where Element is utilized and remains declared in the compiled output:
  const Element = typeof Element === "undefined" ? function() {} : Element;
  const innerRefType = PropTypes.oneOfType([
    PropTypes.string,
    PropTypes.func,
    PropTypes.instanceOf(Element), // by adding this line, rollup won't remove the "Element" declaration
    PropTypes.shape({ current: PropTypes.any })
  ]);

@timdorr we're 2 weeks on, any chance of a beta.8 that fixes this?


Migrate viewlet-level progress indication to view-level

Refs: #92135
Now that we can support view-level progress indication, there doesn't seem like much reason to have viewlet-level progress.
This can also address the concerns here: https://github.com/microsoft/vscode/pull/92136#pullrequestreview-371744519
/cc @bpasero @sbatten

@eamodio I guess one problem is that extensions can still use API to report progress on the viewlet container right? I forgot about that when I suggested this, so we may not easily be able to remove this.

Yeah, we won't be able to remove it, but we could deprecate it to encourage view-level use.
I've added an API proposal here for extensions: https://github.com/microsoft/vscode/issues/92421

Since we really only expose ProgressLocation.SourceControl (and it is kind of special), I don't think its worth doing anything here.


Fix bug #2000 broken error messages (output contained variable name as string)

Type

[x] Bug Fix

Related Issues?
Fixes #2000
What is in this Pull Request ?
Fixes error messages when user provides invalid input

Thanks @alirobe !


this.field() not working as expected

Hello,
I try to set up the real name based on firstName and lastName with the help of autoValue:
  "profile.firstName": {
    type: String,
    index: 1,
    regEx: /^[a-zA-Z-]{2,25}$/
  },
  "profile.lastName": {
    type: String,
    regEx: /^[a-zA-Z-]{2,25}$/,
    index: 1
  },
  /**
   * @TODO - this is undefined undefined
   */
  "profile.name": {
    type: String,
    autoValue: function () {
      var name = this.field("profile.firstName").value + ' ' + this.field("profile.lastName").value;
      return name;
    }
  },

But it just not work as expected, I only receive undefined undefinedas values. Any idea on this? Thanks a lot!

Acutally I'm not sure if this issue belongs to collection2 because I try to set properties for Meteor.userscollection..

Your autoValue would need to be much more complex to handle update modifiers, etc. Usually I use a collection helper to generate the full name as needed.
As for why they are undefined, you'll need to post your insertion code. Are you just using Accounts.createUser?

Hi Eric,
yes I use Accounts.createUser and there the this.fieldis undefined. I'm wrapping now the function making sure that the this.field()function is actually set correctly. Now it works, but do you have any best practice for this matter?
Cheers and Thanks for the quick reply


subscriber crashed when deserialize msg which is send by foxglove_bridge

Description
push button, in studio remote control panel, schema is Twist.subscriber crashed
Stack trace (most recent call last):
[decodec-1] #12   Object "/home/admin/5v5r/fsinstall/nov_codec/lib/nov_codec/decodec", at 0xaaaae95184e3, in
[decodec-1] #11   Object "/lib/aarch64-linux-gnu/libc.so.6", at 0xffff823bfe0f, in __libc_start_main
[decodec-1] #10   Object "/home/admin/5v5r/fsinstall/nov_codec/lib/nov_codec/decodec", at 0xaaaae9518b57, in main
[decodec-1] #9    Object "/opt/nros/lib/librclcpp.so", at 0xffff828fad83, in rclcpp::executors::SingleThreadedExecutor::spin()
[decodec-1] #8    Object "/opt/nros/lib/librclcpp.so", at 0xffff828f42fb, in rclcpp::Executor::execute_any_executable(rclcpp::AnyExecutable&)
[decodec-1] #7    Object "/opt/nros/lib/librclcpp.so", at 0xffff828f2fcb, in rclcpp::Executor::execute_subscription(std::shared_ptr<rclcpp::SubscriptionBase>)
[decodec-1] #6    Object "/opt/nros/lib/librclcpp.so", at 0xffff828f208b, in
[decodec-1] #5    Object "/opt/nros/lib/librclcpp.so", at 0xffff828f1ee3, in
[decodec-1] #4    Object "/opt/nros/lib/librcl.so", at 0xffff8233183b, in rcl_take_loaned_message
[decodec-1] #3    Object "/opt/nros/lib/librmw_cyclonedds_cpp.so", at 0xffff81f7bcd3, in
[decodec-1] #2    Object "/opt/nros/lib/librmw_cyclonedds_cpp.so", at 0xffff81f7b5a7, in rmw_deserialize
[decodec-1] #1    Object "/opt/nros/lib/librmw_cyclonedds_cpp.so", at 0xffff81f88b07, in
[decodec-1] #0    Object "/opt/nros/lib/librmw_cyclonedds_cpp.so", at 0xffff81f88a84, in
[decodec-1] Segmentation fault (Address not mapped to object [(nil)])

Version:
latest
Platform:
aarch64
Steps To Reproduce

1.change rmw to cyclonedds + iceoryx and use loaned api
Expected Behavior
subscriber got msg,and work correctly

change rmw to cyclonedds + iceoryx and use loaned api将 RMW 更改为 CyclonedDS + IceoryX 并使用借出的 API

I have the felling that it is related to the RMW. Does it work when you use the default RMW?我听说它与RMW有关。当您使用默认 RMW 时，它是否有效？

it worked if I turn off loaned api,but I have to use loaned api for big msg

btw @sgf201 your issue is referencing many other issues because it's not properly formatting. You might want to put it in a code block

@sgf201 is this still an issue for you?

are the button msg senders of bridge using rclcpp to send msg？

Yes, we use rclcpp for that. The relevant code is
https://github.com/foxglove/ros-foxglove-bridge/blob/e37a3e1e309c1a6255e3df5a86324e002dab65e1/ros2_foxglove_bridge/src/ros2_foxglove_bridge.cpp#L628
and
https://github.com/foxglove/ros-foxglove-bridge/blob/e37a3e1e309c1a6255e3df5a86324e002dab65e1/ros2_foxglove_bridge/src/ros2_foxglove_bridge.cpp#L711-L712
Can you spot any issues with that?

@sgf201 I'm going to close this issue since this seems to be related to a middleware issue.

change rmw to cyclonedds + iceoryx and use loaned api

Judging from this, it also seems like you have to make changes to foxglove bridge to use the loaned API which it does not by default.

The root cause has been identified. When the bridge is using the generic publisher to forward messages, it does not use the publish_as_loaned interface for the original loan-based messages.
  explicit PlayerPublisher(       std::shared_ptr<rclcpp::GenericPublisher> pub,       bool disable_loan_message)     : publisher_(std::move(pub))     {       using std::placeholders::_1;       if (disable_loan_message || !publisher_->can_loan_messages()) {         publish_func_ = std::bind(&rclcpp::GenericPublisher::publish, publisher_, _1);       } else {         publish_func_ = std::bind(&rclcpp::GenericPublisher::publish_as_loaned_msg, publisher_, _1);       }

@sgf201 could you give #314 a try and check if it works?


Add native OTEL ES exporter

This PR reimplements OpenTelemetry Elasticsarch exporter. Only span exporter/writer is implemented the query still uses the current ES storage implementation.
Notable changes:

new span writer implementation versioned in exporter/jaegerelasticsearch
used ES client is https://github.com/elastic/go-elasticsearch
OTEL data model is directly transformed to Jaeger ES model
writer uses ES Bulk API
batching and retries capabilities will be used as OTEL processors (batch, queued_retry)
ES client is abstracted and specific implementation is created based on the supplied ES version or derived from ES ping API at startup. This allows us to easily add support for new ES versions
added integration tests

Other changes to the codebase:

storage integration test is more generic - it can be executed from any package

TODOs:

[x] ES clients tests
[x] enable integration storage test on travis
[ ] ES exporter should probably accept its own params struct and not config.Configuration

Considerations:

OTEL batch processor can overflow configured max batch size (units) https://github.com/open-telemetry/opentelemetry-collector/issues/1140
OTEL batch processor does not have configurable batch size in bytes https://github.com/open-telemetry/opentelemetry-collector/issues/1139
OTEL queued retry processor cannot retry a single index unit - only a part of ES bulk API can fail. https://github.com/open-telemetry/opentelemetry-collector/issues/990
when OTEL queued retry is used the retried spans will be serialized multiple times.

Performance
From the perf tests I have done the new writer performs well (with batch processor). It seems to be slightly faster than the current all-in-one with similar settings.
docker run -it --rm -e "ES_JAVA_OPTS=-Xms2g -Xmx2g" -p 9200:9200 -p 9300:9300 -e "http.host=<IP_ADDRESS>" -e "discovery.type=single-node" --name=elasticsearch docker.elastic.co/elasticsearch/elasticsearch-oss:6.8.4

Jaeger OTEL all-in-one with ES exporter and batch processor
processors:
  batch:
    send_batch_size: 1000
    timeout: 200ms

2020-06-17 10:05:26 INFO  Main - Expected number of spans 100000, actual 100000 stored in 39s
2020-06-17 10:24:00 INFO  Main - Expected number of spans 100000, actual 100000 stored in 39s
2020-06-17 10:27:46 INFO  Main - Expected number of spans 300000, actual 300000 stored in 105s
2020-06-17 10:31:17 INFO  Main - Expected number of spans 300000, actual 300000 stored in 96s
2020-06-17 10:37:41 INFO  Main - Expected number of spans 300000, actual 300000 stored in 111s
2020-06-17 10:55:03 INFO  Main - Expected number of spans 300000, actual 300000 stored in 115s
2020-06-17 11:04:25 INFO  Main - Expected number of spans 300000, actual 300000 stored in 164s
2020-06-17 11:10:55 INFO  Main - Expected number of spans 300000, actual 300000 stored in 121s

Jaeger all-in-one --es.bulk.size=10000000000000, --collector.queue-size=300000
2020-06-17 11:20:35 INFO  Main - Expected number of spans 300000, actual 300000 stored in 181s
2020-06-17 11:24:38 INFO  Main - Expected number of spans 300000, actual 300000 stored in 166s
2020-06-17 11:29:54 INFO  Main - Expected number of spans 300000, actual 300000 stored in 172s
2020-06-17 11:49:06 INFO  Main - Expected number of spans 300000, actual 300000 stored in 165s

The code is in the final state. The missing item from TODO:

ES exporter should probably accept its own params struct and not config.Configuration

it's not critical. The query configuration options are not exposed in the OTEL config. I will consider doing refactoring in a follow up PR.


Gradle wrapper validation

https://blog.gradle.org/gradle-wrapper-checksum-verification-github-action
action doc: https://github.com/marketplace/actions/gradle-wrapper-validation#add-a-new-dedicated-workflow

I should add this to jib-extensions repo too.

Curious if this validates all wrapper files?

Seems yes.

This action will verify that any and all gradle-wrapper.jar files in the repository match an official SHA-256


Create an official image in docker hub

With the docker file created we could have an official and automated build in monitoringartist namespace.
Would be very nice. Thank you!

Actually, I prefer GitHub pages version to be used by the public. There is a minimum user cases, where Docker image will be useful. It's only JS/HTML/CSS, so even nginx is not required = users can clone the project and open index.html in the browser directly.
If you really need ready public Docker image, then Zapix is already available as integrated XXL extension in https://github.com/monitoringartist/zabbix-xxl.


add erdii as approver and reviewer to OWNERS

Summary

Change Type

Check List Before Merging

[ ] This PR passes all pre-commit hook validations.
[ ] This PR is fully tested and regression tests are included.
[ ] Relevant documentation has been updated.

Additional Information

/ok-to-test


[Campaign] Overflowing campaign name creates unnecessary white space on the right side of the page

Title
[Campaign] Overflowing campaign name creates unnecessary white space on the right side of the page
Platform / OS
Windows 10
Browser
Chrome
Description
If user created a very long campaign name (max 80 char) the name will overflow and create unnecessary space on the right side of the page
Steps to Reproduce

Create a new campaign with very long campaign name
Visit the campaign page after successfully creating the campaign. Sample page --https://dev.sleepbus.org/campaign/test-lenght-wwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwwww
Notice that overflowing campaign name and page can be move to the right side

Expected Behavior
Make necessary cut-off for wrapping the text so that overflow will not be an issue or limit the number of characters that can be inputted on the text field.
Additional Info

Branch: 54789c348427efcfa22edd86964eb9b398ea178e
dev Build: https://circleci.com/gh/sleepbusorg/sleepbus.org/169

creating an update and comment with too much characters also makes unnecessary white space on the right side of the page. Probably best to limit the number of characters and put a proper border where text should begin to wrap.
https://dev.sleepbus.org/campaign/test-campaign-a-2

won't fix


auth0 errors

ncc version: 0.0.4
Source:
var AuthenticationClient = require("auth0").AuthenticationClient;

var auth0 = new AuthenticationClient({
  domain: "{YOUR_ACCOUNT}.auth0.com",
  clientId: "{OPTIONAL_CLIENT_ID}"
});

crashes with:
Use of eval is strongly discouraged, as it poses security risks and may cause issues with minification
(node:30620) UnhandledPromiseRejectionWarning: Error: Illegal reassignment to import 'commonjsHelpers'
    at error (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:3460:30)
    at Module.error (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:13349:9)
    at MemberExpression$$1.disallowNamespaceReassignment (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:11372:26)
    at MemberExpression$$1.deoptimizePath (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:11340:18)
    at AssignmentExpression.bind (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:9920:19)
    at ExpressionStatement$$1.NodeBase.bind (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:9183:23)
    at IfStatement.NodeBase.bind (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:9183:23)
    at IfStatement.bind (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:10807:31)
    at Program.NodeBase.bind (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:9179:31)
    at Module.bindReferences (/usr/local/lib/node_modules/@zeit/ncc/node_modules/rollup/dist/rollup.js:13303:18)
(node:30620) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:30620) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

This is a bug in rollup-commonjs caused in node_modules/formidable/lib/incoming_form.js with the following code:
if (global.GENTLY) require = GENTLY.hijack(require);

which is converted to:
import * as commonjsHelpers from 'commonjs-helpers';
if (commonjsHelpers.global.GENTLY) commonjsHelpers.require = commonjsHelpers.global.GENTLY.hijack(require);

I think the best fix would be to have the CommonJS plugin entirely ignore assignments when doing dynamic require replacements to avoid this bug.
Working on the PR now.

PR https://github.com/rollup/rollup-plugin-commonjs/pull/359

(and added to the #merges fork)

With the merge made, the remaining issue is a cycle bug.


docs: Add AFM section

Landing here after the lightning talk by @MarcSkovMadsen at PyData Paris 2024 :)
Just wanted to say having some kind of "standard" for portable widget code would indeed be really useful :+1:
For reference, a similar idea was discussed a few years ago at the Jupyter Widgets community workshop in London (https://github.com/jupyter-widgets/team-compass/issues/16), which led to some experimentation in https://github.com/jupyterlab/richoutput-js.


Reworking of App

It was discussed yesterday that we could benefit from having an app which could control the bot and also display a feed from a camera as well as the environment constructed from lidar.
Currently we have the framework of an app from previous years robots.  The task is to take an modify this app to display this new information.
This task is included as an end of summer milestone goal

I'd like to take on the task of routing the live camera feed to the app.

An update to this issue: I have successfully been able to stream video from my laptop's webcam (using python and openCV) to an android app over the internet and I believe this can be replicated with the BBB running python code to stream the video to the android app.
However this is still under progress since there are various smaller issues that need to be resolved; the most important of them being the delay in displaying the video on the app (about a second).
If anyone's done something like this before, I'd appreciate some ideas on how to minimize this delay. I will restrain from spilling all the technical details here for the sake of conciseness.

If anyone's done something like this before, I'd appreciate some ideas on how to minimize this delay. I will restrain from spilling all the technical details here for the sake of conciseness.

I've done a lot of performance stuff generally, but nothing specific to video/streaming/Android/etc.
Again, just generally speaking, the first step is to quantitatively describe the sources of the delay. Which steps of the stack are taking a long time? What resource are they short on (CPU, RAM, HDD/SDD backing store, network throughput, network latency, ...)? Fix the lowest hanging fruit and re-measure.
Hope that helps. If you give some of the relevant tech details, I might be able to help more.

I will restrain from spilling all the technical details here for the sake of conciseness.

This is exactly the right place to spill all of the technical details :)

@dfarrell07 That does help a lot. I'm going to run some tests later at different points in the stack to see which takes the most time.
@AhmedSamara haha fair enough:
So the stack goes like this
Server side:

 Python grabs frames from laptop's webcam using openCV
 compresses the frames to jpeg format
sets up a tcp socket and waits for client
 Once client has connected send the size of the image in bytes
 Waits for client confirmation
 Once client confirms, send the image data in bytes
 Waits for client confirmation, then loops back to step 4.

 Client side:

 Sets up basic stuff
 Creates a tcp socket and attempts to connect to server.
 Upon connection and receiving the image size, send confirmation.
 Reads all of the bytes from the socket's input stream.
 Sends confirmation to server.
 Gets the reference of the ImageView object (ImageView is an android class to display pictures)
 Constructs a bitmap from the data received and sets the ImageView reference to it.
 Loops back to step 3

I hope that this addresses most of the technical details.
Now regarding the delay in the stream, I have two suspicions as to what could cause it:

I'm not pausing anywhere when I'm reading the images and sending it over the network. Maybe this is causing overhead at the client side, with the thread trying to update the ImageView too rapidly.

ImageView may not be the conventional way to stream video as it's more suited for images. Unfortunately android does not support m-jpeg as a video format so we might need to switch over to h.264 or something like that.

Is the delay better on different wifi networks?
As I understand it, the delay would probably happen at client.step 4, the
"recieve step" would last for as long as is required to send the entire
message each time.
I don't know if the delay is necessarily a thing we can solve in code (at
least not without adding an order of magnitude of complexity).
IP cameras are a thing, and they're specifically made for people with high
real-time requirements for video surveillance. We'd have to recreate
whatever they do to actually get zero delay.
Could you go ahead and post your code? (It doesn't matter if it's not done,
in-progress projects are perfectly appropriate for github).
On Tue, Jul 21, 2015 at 10:28 PM, Vijay Thiagarajan <
<EMAIL_ADDRESS>wrote:

@dfarrell07 https://github.com/dfarrell07 That does help a lot. I'm
going to run some tests later at different points in the stack to see which
takes the most time.
@AhmedSamara https://github.com/AhmedSamara haha fair enough:
So the stack goes like this
Server side:

Python grabs frames from laptop's webcam using openCV
compresses the frames to jpeg format
sets up a tcp socket and waits for client
Once client has connected send the size of the image in bytes
Waits for client confirmation
Once client confirms, send the image data in bytes
Waits for client confirmation, then loops back to step 4.

Client side:*

Sets up basic stuff
Creates a tcp socket and attempts to connect to server.
Upon connection and receiving the image size, send confirmation.
Reads all of the bytes from the socket's input stream.
Sends confirmation to server.
Gets the reference of the ImageView object (ImageView is an android
class to display pictures)
Constructs a bitmap from the data received and sets the ImageView
reference to it.
Loops back to step 3

I hope that this addresses most of the technical details.
Now regarding the delay in the stream, I have two suspicions as to what
could cause it:

I'm not pausing anywhere when I'm reading the images and sending it
over the network. Maybe this is causing overhead at the client side, with
the thread trying to update the ImageView too rapidly.

ImageView may not be the conventional way to stream video as it's more
suited for images. Unfortunately android does not support m-jpeg as a video
format so we might need to switch over to h.264 or something like that.

—
Reply to this email directly or view it on GitHub
https://github.com/IEEERobotics/bot/issues/370#issuecomment-123537639.

--
Ahmed Samara
Computer Engineering

If you are not committed to the solution you are already using, my team working on the Firefighting Drone Challenge was able to stream video from a webcam, through the BBB, over wi-fi, last semester. We used gstreamer, and were able to tweak it to only a 300ms delay. That took about 40 hours to get up an running. Derek Molloy also lays out another way to do it in his book Exploring Beaglebone.
I didn't work on that part personally, but if you are interested in discussing it, I might be able to get my teammate in next week. We also did a write up for the class.

@kvijay1995 - Super quickly, it sounds like there are quite a few RTTs (round trip times) per image frame. One RTT would typically be ~10-20ms at best (just latency, throughput delay from the large image).

@AhmedSamara I haven't tried it on different wifi networks so I wouldn't know if that's the reason. I'll look into IP cams and try to replicate their methodology. And yes I'll post my code on github.
@PaladinEng I think this is the right time to be open to ideas so that would be great. I looked into gstreamer and it looks like both android and linux are supported so that's a good sign.
Do you happen to know which video codec you guys used (h.264, mjpeg, etc.)?. I believe Derek Molloy uses v4l2 (Video4Linux2) in conjunction with avconv to stream video. That would be great if I can talk to your teammate; only problem is I won't be in for another 2 weeks but I can do anytime after. Would it be possible to meet him then? In the meantime I can do some more research about this.
@dfarrell07 - Yes that's true; I'll try to eliminate those acknowledgement messages from the client and see if that helps.
Thank you guys for your input! :)

I am wondering what resolution and framerate are you sending the video over? Having a 1080/720  at 30fps is taxing. The question to ask then is, do we actually need this fidelity or can we use a lower quality? Changing these might help a bit.
One possible way of lowering the latency is to reduce the payload of each packet. Sending big packets may be slower than sending many lesser packets. This can be done by Fragmenting/Splitting up each packet into a certain max size.
A solution for removing step four as pointed out by Ahmed could be to merge the size of the jpeg and data together or just send them without the confirmation, as order in TCP is maintained (assuming the same connection). I think that the confirmation in step seven can also be discarded safely. Unless the confirmations are there for other precautions.

@kennychuang Right now the resolution I'm sending over is 480p (640*480) and the framerate is not being controlled right now. I am planning to control the frame rate as we move forward however.
I haven't looked into fragmenting the packets yet but for reference each image I'm sending right now is no more than 90,000 bytes. I'll try that too to see if it helps.
I've already removed all of the confirmation messages as suggest by you, Ahmed and Daniel. I'll commit the revised code later tonight. Thanks for you input Kenny

Can someone post an update for this?  Its still an active and ongoing issue in real life but the issue is out of date
@mynameis7 can you describe the issue you were seeing with the app?  It looked really good the other night, but I remember it couldn't collect data?
@kvijay1995 are you still having an issue with delay in video processing?

Yea, there's still some delay in the video processing. From my talk with Alwyn, we came to a conclusion that there will always be a delay in this since the beagle bone hardware is not designed for realtime video processing. We can certainly optimize the video streaming process, but the question is how relevant or useful would this be for our end goals since this might turn out to be quite time consuming.

I'll try fragmenting too to see if it helps.

I'm not sure I'd recommend that route. Packet size optimization for perf is a very complex problem, without quite a bit more background/study I think it's a poor use of time.

we came to a conclusion that there will always be a delay

+1, it's about figuring out which properties you need (latency, jitter, throughput) and how optimized they need to be.

question is how relevant or useful would this be for our end goals since this might turn out to be quite time consuming.

Very important question to keep in mind, it does smell a bit like a time sink/rabbit hole.


Documment available REST API URLs in Shoal Server README

We need to document the new server URL that @colsond is adding is his pull request in the shoal server readme file. https://github.com/hep-gc/shoal/blob/master/shoal-server/README.md

@igable Can I close this or do you think further documentation is needed?


When will the training code be released?

Excuse me , When will the complete code (including training code)be released ? Thank you!

+1

+1

@joanChen0212 @zhangxiao696 @why529913 Thank you for your interest and great patience in our work! We are glad to announce that the training code has recently been updated! Welcome to use and leave comments!


Undefined method on attr_reader should not cause panic

 class Foo
   attr_reader("bar")
 end

 a = Foo.new
 a.bar = "fuz"

Ruby:

13sandbox.gb:9:in <main>': undefined method bar=' for #Foo:0x007ff1f610dac0 (NoMethodError)

Goby:

Internal Error: interface conversion: vm.Object is *vm.RClass, not *vm.RObject
panic: interface conversion: vm.Object is *vm.RClass, not *vm.RObject [recovered]
panic: interface conversion: vm.Object is *vm.RClass, not *vm.RObject [recovered]
panic: interface conversion: vm.Object is *vm.RClass, not *vm.RObject [recovered]
panic: interface conversion: vm.Object is *vm.RClass, not *vm.RObject

But the following outputs as expected:
 class Foo
 end

 a = Foo.new
 a.bar = "fuz"

Goby:

17sandbox.gb:6:in <main>': undefined method bar=' for #Foo:0x007ff0cf118b50 (NoMethodError)

Ruby:

goby 1 $goby 17sandbox.gb
UndefinedMethodError: Undefined Method 'bar=' for

This behavior didn't show on my machine. And I added test for these cases https://github.com/goby-lang/goby/commit/143f126af4783e2cd50b9f9c64783c3e715c5c7b, it passed both my machine and CI.
Also, if I run it in file, it returns expected message too.

I'll look into it.

As you notified, this time works fine with that.
I close the issue.


Make participants checklist 2 column

As a user I dont want 4Minitz to waste my precious screen realestate so that I can see more valuable info at once.

[ ] If screen is "desktop" width and we have >7 participants make participants list 2 column
[ ] On smartphones stick with one column!
[ ] Check manually on major browsers and different windows witdth: IE11, Edge, current FF, current Safari, current Chrome

No E2E needed!
Hint:
https://css-tricks.com/guide-responsive-friendly-css-columns/
http://stackoverflow.com/questions/36747969/why-does-column-count-break-bootstrap

Timeboxed: 1d


run_all_experiments: update project summary data on-the-fly

Prior to this commit project summary stats were generated only when all experiments had compled, i.e. all benchmarks running should terminate. This causes issues when, for some reason, some benchmarks may not terminate properly or terminate at all. This was observed on some large-scale experiments where 10-20 benchmarks did not finish, and, consequently the full run_experiment_all didn't terminate proper. However, in this case we stil want project summary stats for the 1000+ other benchmarks that have been analyzed.
This fixes it by updated benchmarks stats after each completed benchmark run.

/gcbrun exp -n dk-ft-102 -m vertex_ai_gemini-1-5 -b minor-for-ci -i

First experiment is looking good -- as well as all locals

/gcbrun exp -n dk-ft-105 -m vertex_ai_gemini-1-5 -b from-test-small -i

second experiment using test migration benchmarks is also looking good with results updating during run


update is not a function

Are you submitting a bug report or a feature request?
bug
What is the current behavior?
I have many working <Form /> components in a react-native (v0.60) app - as soon as I add decorators prop to Form, i get this error, "update is not a function":
decorators={[
            createDecorator({
              field: 'department', // when the value of department changes...
              updates: {
                // ...set field value
                subDepartment: -1,
              },
            }),
          ]}

What is the expected behavior?
For the decorators to work as expected.
What's your environment?
"final-form": "^4.18.1",
"final-form-calculate": "^1.3.1",
"final-form": "^4.18.1",
"final-form-calculate": "^1.3.1",
Other information
Looks to be the culprit here, but not sure why: https://github.com/final-form/final-form-calculate/blob/bac6a750b7de1c1614996207d0e42ac3958c402d/src/decorator.js#L39
The stack trace only points to the line where <Form /> is declared in the error:

Here is a simple reproducible code snippet one could paste into the App.js file with a new react-native init rffDemo project from the cli:
import React, { Component } from 'react';
import { Text, TextInput, View } from 'react-native';
import createDecorator from 'final-form-calculate';
import { Field, Form } from 'react-final-form';

export default class App extends Component {
  render() {
    return (
      <Form
        onSubmit={() => {}}
        decorators={[
          createDecorator({
            field: 'name', // when the value of department changes...
            updates: {
              // ...set field value
              last: 'something',
            },
          }),
        ]}
        render={({ handleSubmit }) => (
          <View>
            <Field
              name="name"
              render={({ input }) => (
                <TextInput value={input.value} onChangeText={input.onChange} />
              )}
            />
          </View>
        )}
      />
    );
  }
}

I'm a tard. I was copy-pasting an old codesandbox from erikras and not using the latest API for final-form-calculate :-(
I need to pass a function to the field i'm updating.


refactored RequireNot to OrElse

Hi!
First of all season's greetings everybody! :-)
This pull request is intended as a first attempt to cover an idea John has expressed in a private conversion while I contacted him for help on issue #70.
John was suggesting the following:

Actually I think we should delete requires/requiresNot and solve the problem in a different way:
By making an orElse operator on Options.
Then we can explicitly denote which options are mutually incompatible with each other: a | b | c (orElse).
We can already denote which options require each other using "cons": (a :: b) | c would mean, we require a and b, or we require c. One of the two, but not both.
Basically the requires / requiresNot way of solving this problem is hard because it requires non-local solution.
A compositional solution is preferrable.
So we should just delete requires / requiresNot and add OrElse.

Introducing || worked out wonderfully and feels much more intuitive than the requires/requiresNot operators. I've added a new subtype OrElse, removed  Requires/RequiresNot and added some tests in this pull request.
More work is required to properly render the error messages.

Should I add an arity 4, 5 and 6 version of fold also?

Thank you! 🙏

Thank you! 🙏


Improve the shadow price analysis

Constrain the biomass formation to be only very small (not 0.99 of maximum).

The option to change the ratio of max yield and the upper bound when calculating shadow price has been added to the script now called "calculate_shadow_prices.py".
The analysis for several different yield factors is yet to be done (takes a few days).


example data processing warning using google colab

Hello,
The following warning is issued when processing one of the .xml from the example data:
Processing: paperetl/file/data/0.xml
/usr/local/lib/python3.10/dist-packages/paperetl/file/tei.py:35: XMLParsedAsHTMLWarning: It looks like you're parsing an XML document using an HTML parser. If this really is an HTML document (maybe it's XHTML?), you can ignore or filter this warning. If it's XML, you should know that using an XML parser will be more reliable. To parse this document as XML, make sure you have the lxml package installed, and pass the keyword argument features="xml" into the BeautifulSoup constructor.
soup = BeautifulSoup(stream, "lxml")
Any clue how to avoid/correct that?
Thanks a lot.

I am using the colab notebook.

You can ignore it like this:
import warnings
from bs4 import XMLParsedAsHTMLWarning

warnings.filterwarnings("ignore", category=XMLParsedAsHTMLWarning)

Thanks.
But "using an XML parser will be more reliable" the message says.

Feel free to fork this project and try. It doesn't work in the tests I've run.

Closing this issue as the question has been addressed.


Make IndexingPolicyEqualityComparer public

Is your feature request related to a problem? Please describe.
We have code that uses attributes to mark properties on objects as indexable. We use these attributes to generate an IndexPolicy that represents all indexes needed by our applications. At startup, we fetch the container's indexing policy from the server and compare the existing policy with the generated one to see if it needs to update the container and start a new index transformation.
We noticed that an equality comparer already existed in the SDK, but is internal for tests.
Describe the solution you'd like
Make IndexingPolicyEqualityComparer (and related classes) publicly accessible as a top-level class.
Describe alternatives you've considered
Continue to use our own comparer, use reflection to access the class internal to the SDK, or copy the code directly into our project.
Additional context
None.

So-far its a single ASK, we will revisit making it public contract later.


All icons disappeared after update

Just updated Atom to 1.10.2 (includes Nuclide, added some other packages in the same update step). And all icons have disappeared. Not a showstopper. But annoying nonetheless.

Please open a new issue and completely fill out the provided issue template.  Thanks.


Do not install json header when json functions is disabled

When you disable JSON functions, headers are still installed, which is misleading for dowstream project.

Hi @AlexandreBossard
Thanks for the report. I'd appreciate it if you could review #114 to confirm it does what you expect before I merge it.

Looks good to me, thanks!


[Feature Request] Documenting error code mapping

Currently in our repo, it is not well documented what an error code mapping (errmap) is. This leads to confusion when some of the tools ask the user for one.
We should consider

[ ] Adding comments or a README file to move-core-types, explaining what an errmap is.
[ ] Adding a README to errmap gen to explain how the generation process works

For the first part, wouldn't it be better to put the description as a comment in errmap.rs instead of a README file?


AresCoin Enterprise

https://github.com/RabbitFinanceProtocol

Hi @Areskings, please clarify your issue or question. Thank you

Areskings,
Atom is still alive. This should be bug or feature, not promotions and trash. Thank you.


Updating AngularJS View when Scope has changed using $watch

Hi, I am using AngularJS 1.6 along with AngularFire 2.3.
I'm a bit stuck ATM...

For some reason when I make a change in the Firebase DB, and I log the scope.. it is reflecting the new data, however the changes aren't making it into the angular view.
var furryPostsRef = firebase.database().ref('stories/' + storyId + "/correspondents").orderByChild('published').equalTo(true);
var fbObj = $firebaseObject(furryPostsRef);
fbObj.$watch(function(){
//do stuff
$scope.stuff = fbObj;
console.log($scope.stuff)
//everytime a change happens on the Firebase server, This is logging correct
});

Not exactly sure where I;m going wrong here.
Thanks

forgot password reset link , see the video link completely solved
https://youtu.be/daGZO6YAkRg


Path not correctly built when using offline fonts

Describe the bug
When using the option cdnCaching in quartz.config.ts font files are correctly downloaded and visible in public/static/fonts/ after building locally. But with a baseURL like https://my-site.com/digital-gardenthe path is omited in the URL: Font files are not found as Quartz looks for them in https://my-site.com/static/fonts/. The partdigital-garden` is missing.
To Reproduce
Steps to reproduce the behavior:

Go to quartz.config.ts
Change cdnCaching to true
Change baseURL to your site and add a path
Publish the site on GitHub e.g. Locally the path part of the baseURL is omited anyway and the error is not reproducable.

Expected behavior
The chosen fonts being loaded from https://my-site.com/digital-garden/static/fonts
Desktop (please complete the following information):

Quartz Version: v4.2.3
node Version: v21.5.0]
npm version: v10.5.0
OS: Manjaro Linux
Browser: Chromium

cc @aarnphm
probably here: https://github.com/jackyzha0/quartz/blob/141dd3b51f59aabc8ef3506d040f7360a86f26d0/quartz/plugins/emitters/componentResources.ts#L231
we probably need to take into account baseUrl here

probably here:
https://github.com/jackyzha0/quartz/blob/141dd3b51f59aabc8ef3506d040f7360a86f26d0/quartz/plugins/emitters/componentResources.ts#L231
we probably need to take into account baseUrl here

Hmm we need to also format the string with the baseUrl I assume.


Move all jb-main settings into repository

It isn't hard to solve conflicts for them when we merge the upstream branch, and it significantly simplifies project setup.
Note that gradlew and gradlew.bat are modified (the additional section with env. variables).
Pros:

we can clone only androidx
we can setup auto tests on github actions
no more desync issues (config is one version, and repo is another)
no manual configuration in IDEA. works almost out of the box

no manual configuration in IDEA. works almost out of the box

Does "almost" mean that the only thing we need to do is to donwloadAndroidSdk?

should it be merged into jb-main or in sync-androidx/1.2.1 ?

Yes, it should be jb-main. Thanks!


Added Howard Community College

Howard Community College is located in Columbia, Maryland, USA
https://www.howardcc.edu/about-us

@mchowcc I am sorry, the domain howardcc.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

I am faculty, not student. I am teaching Python courses, and I would like to recommend PyCharm to my students.
So, I need to tell them to create a GitHub account?
Thanks,
Marjeta
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Philip Torchinsky @.>
Sent: Thursday, August 15, 2024 3:46:12 AM
To: JetBrains/swot @.>
Cc: Cedilnik, Marjeta @.>; Mention @.>
Subject: Re: [JetBrains/swot] Added Howard Community College (PR #22616)
You don't often get email from @.*** Learn why this is importanthttps://aka.ms/LearnAboutSenderIdentification
CAUTION: This email originated outside of Howard Community College.
Use caution when responding, clicking links, or opening attachments.
@mchowcchttps://github.com/mchowcc I am sorry, the domain howardcc.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Programhttps://education.github.com/discount_requests/application. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).
—
Reply to this email directly, view it on GitHubhttps://github.com/JetBrains/swot/pull/22616#issuecomment-2290832106, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKRKWVDKV2MIUBMFYSIDVWTZRRMEJAVCNFSM6AAAAABMRFWUFSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJQHAZTEMJQGY.
You are receiving this because you were mentioned.Message ID: @.***>

@mchowcc I am sorry, the domain howardcc.edu is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

@philipto
I have applied for GitHub Educational Benefits, and got approved 4 days ago. But when I try to apply for JetBrains license, it says: "We are sorry, GitHub has not granted your account a GitHub Student Developer Pack, please apply with other methods."


Fix broken wiki link

Current link (https://wiki.hyperledger.org/projects/Fabric) gives a 404

Build seems to be failing consistently for unrelated reason, please re-trigger when your Travis build is fixed.

Thanks for the contribution! However, this is a read-only mirror. please read the contribution guide. Also, feel free to join us on chat!


JS: collection.fetch() is used for synchronous requests

In src/js/oseam-views-tracks.js:
initialize : function() {
  [...]
  this.collection.fetch({wait:true});
  [...]
}

This works not (always) as expacted: fetch is an asychonous operation and can not be used to wait until the data is received. This manifests sometimes on my computer: the lists for the license or ships are empty.
From the backbone manual (http://backbonejs.org/#Collection-fetch)

Note that fetch should not be used to populate collections on page load

Another example and a possible solution: http://stackoverflow.com/questions/11842090/backbone-fetch-collection-on-page-load

Any progress on the issue ?

Sorry my free time is very limitied in this weeks.
#143 is a possible solution for this. Please have a second look and some more testing at it.
I have not checked whether this kind of synchronous requests is in other files too.


fix inline code highlight

This change is

Actually, this sentence is a little out of date. The following can be edited as well:
models/* and tool/alm-cli/*


group: add DescribeGroups()

This PR adds DescribeGroups(), which list up the available groups.
Added CLI as well.
Usage:
$ infrakit group describe-groups
ID
aws-example

Signed-off-by: Akihiro Suda<EMAIL_ADDRESS>

Current coverage is 69.58% (diff: 8.33%)

Merging #260 into master will decrease coverage by 1.78%

@@             master       #260   diff @@
==========================================
  Files            19         19
  Lines          1233       1269    +36
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            880        883     +3
- Misses          276        308    +32
- Partials         77         78     +1

Powered by Codecov. Last update 173ad78...f4ad6ad

Looks great, thanks for the patch!  Just a heads up - i'll probably follow up to rename some things (we should really call this describe and rename describe to describe-update).

Yes, I was thinking of that.
Maybe "ls"?

@AkihiroSuda

Maybe "ls"?

Ah, yes - that's even better for the CLI command.


Handle error when baseDirectory doesn't exist

I updated my blog and I had a little error because I have to use __dirname on the baseDirectory.
Also I removed some unused packages and moved underscore to the devDependencies, what do you think about using lodash instead.


Events not fired on new tab

It seems that events are not fired when you are inside a folder connected with a virtualenv (vf connect) and you open a new tab (by default on Mac OS it opens with the current path).
As you can see on the demo below, the virtualenv gets activated, but the postactivate function that's linked to virtualenv_did_activate  event does not execute.
This is my config.fish
eval (python3 -m virtualfish auto_activation)
function postactivate --on-event virtualenv_did_activate
    set -gx HELLO "Hello World!"
end

And here is a little demo:
https://streamable.com/ez6nh
Thanks.

That's strange; I've just tested this and it works fine for me.
What OS version, version of fish (echo $FISH_VERSION) and virtualfish (echo $VIRTUALFISH_VERSION) are you using? (For comparison, mine is macOS 10.12.6, Fish 2.6.0, virtualfish 1.0.5.)

Hi. Thanks for the support. It's still happening, as you can see in this new recording: https://streamable.com/zdpi8
macOS 10.12.6
Fish 2.6.0
Virtualfish 1.0.5
Maybe something wrong with the auto-activation plugin? Thanks.

What happens if you move the event handler above the eval line?

Yep, that fixed it 🎉

Awesome!
I'll leave this ticket open for now, because we probably need to document this behaviour.


August 2023 Patch Release

Codecov Report

Merging #6331 (b113505) into dev (81d34b4) will decrease coverage by 4.24%.
Report is 252 commits behind head on dev.
The diff coverage is 18.96%.

Flag
Coverage Δ

msal-angular
?

msal-browser
85.53% <ø> (-0.94%)
:arrow_down:

msal-common
83.30% <ø> (-1.24%)
:arrow_down:

msal-core
?

msal-node
79.29% <ø> (-4.10%)
:arrow_down:

msal-node-extensions
40.61% <18.96%> (-35.03%)
:arrow_down:

msal-react
94.24% <ø> (-0.45%)
:arrow_down:

node-token-validation
?

Files Changed
Coverage Δ

...ions/msal-node-extensions/src/utils/Environment.ts
20.51% <0.00%> (-22.08%)
:arrow_down:

...b/msal-browser/src/app/IPublicClientApplication.ts
42.42% <ø> (-1.33%)
:arrow_down:

...ib/msal-browser/src/app/PublicClientApplication.ts
67.44% <ø> (-27.89%)
:arrow_down:

...er/src/broker/nativeBroker/NativeMessageHandler.ts
86.29% <ø> (+0.33%)
:arrow_up:

lib/msal-browser/src/cache/AsyncMemoryStorage.ts
80.76% <ø> (+2.99%)
:arrow_up:

lib/msal-browser/src/cache/BrowserCacheManager.ts
89.14% <ø> (-2.57%)
:arrow_down:

lib/msal-browser/src/cache/BrowserStorage.ts
87.50% <ø> (-3.41%)
:arrow_down:

lib/msal-browser/src/cache/CryptoKeyStore.ts
80.00% <ø> (-3.34%)
:arrow_down:

lib/msal-browser/src/cache/DatabaseStorage.ts
34.00% <ø> (-4.54%)
:arrow_down:

lib/msal-browser/src/cache/MemoryStorage.ts
100.00% <ø> (ø)

... and 64 more

... and 166 files with indirect coverage changes


Initialize current_labels to empty list if no labels exist in pr_revi…

User description
…ewer.py

Type
bug_fix

Description

Fixes an issue where current_labels was not initialized to an empty list if no labels were present, potentially causing errors in subsequent operations.
Ensures that current_labels is always a list, improving the robustness of label handling.

Changes walkthrough
Relevant filesBug fix

pr_reviewer.pyFix Uninitialized `current_labels` in PR Reviewer

pr_agent/tools/pr_reviewer.py
Initializes current_labels to an empty list if no labels exist.  Adds a debug log to display the current labels.

+2/-0

✨ PR-Agent usage:
Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

/review auto_approve


src: shows dependencies using npm range version

Connects to #207
Also part of #226 (warning messages)

Coverage increased (+0.007%) to 90.632% when pulling 4f154758518586359ae587437e1ff85c7d71cc60 on helio-frota:cleanup into a9de780175f9841b2fc188387d291b389ad718bd on bucharest-gold:master.


Making sure data is archived even if the host app has crashed

Hi, we noticed that some of our users don't have any super-properties
After investigation, here is a scenario that caused the problem:
Host application crashes, mixpanel looses all the data since it keeps all the properties in memory in-between app being backgrounded / terminated.
A possible solution: not to remove the archived plist files after reading them on startup but it would only preserve last serialised state not the latest set one, which is not desirable when dealing with sensitive user data.
Proposed solution: become one of the chain of the exception handlers and archive data upon exception

hey @alex-hofsteede! Can you review this one please? It's a short one I promise :smile_cat:

:+1:

@zats are you using this in production? seems like we may be missing a few events and wondering if this is the cause

@cmcewen we are using a slightly modified version since we didn't want to change the source.
But we actually got an email from mixpanel saying that proposed solution is not production safe (without any technical details) and they won't probably merge it.
We're still relying on this approach, but you should use your best judgment.
I'd love you if or anyone for that matter can suggest reasons why this code is not production-safe

We're working on a modified version internally that will resolve this. Just need to finalize some details with the folks at Crashlytics.

great, I'm looking forward to it! if there is a beta version, I'd love to test it :+1:

@zats Sure thing, I'll reach out next week.

This will be resolved with #306

Hi, Actually It's not resovled.
It should be use signal handler not a exception handler.
exception handler can handle specific situations such as segmentation fault, invalid instruction, system abort.
plz reopen


Add rendered example of all shapes

On the shapes page, it would be great if you showed rendered examples for every shape, with the name of that shape as the text inside the shape.

@DeflateAwning good point! I'll just copy this test case over: https://github.com/terrastruct/d2/blob/master/e2etests/testdata/stable/all_shapes/elk/sketch.exp.svg


Allow assertion methods to be run after the test

Currently, if an assertion fails, the test fails right away. If there are multiple assertions within a test, then after the first assertion fails, the rest of the assertions are not run.
The new Assertion class resolves this a bit, allowing for multiple assertions to be collected, with the final assertion done at the end of the test. The problem with this is that this final assertion needs to be done at the end of every test method.
For example, if the test class has the following 2 test methods and corresponding assertions (pseudocode), then you can see that the "assertion.doFinalAssert()" operation is being done explicitly at the end of each test.
@Test
public void test1() {
Assertion assertion = new AssertionImpl();
assertion.assertSomething();
assertion.assertSomethingElse();
assertion.doFinalAssert();
}
@Test
public void test2() {
Assertion assertion = new AssertionImpl();
assertion.assertSomething();
assertion.doFinalAssert();
}
It would be nice if this "assert.doFinalAssert()" is in a separate method that will be called automatically right after the @Test is completed. So the above code can result in something like this.
public void SomeTest {
private Assertion assertion;

@BeforeMethod
public void setup() {
    assertion = new AssertionImpl();
}

@Test
public void test1() {
    assertion.assertSomething();
    assertion.assertSomethingElse();
}

@Test
public void test2() {
    assertion.assertSomething();
}

@AfterTestMethod {
    assertion.doFinalAssert();
}

}
Notice the introduction of the new @AfterTestMethod annotation. If such an annotation is added, then I think it's better to re-purpose @AfterTest to @AfterTests and @AfterTest will then be this "@AfterTestMethod". If that is not possible, then @AfterTestMethod to me makes sense.
This @AfterTestMethod is different than @AfterMethod because if the assertion is placed in the @AfterMethod, then it currently results in a configuration failure instead of a test failure. The point of this is to run the method(s) with @AfterTestMethod right after the test, but the result will be part of the test.

This can be solved using SoftAssert and leveraging IHookable implementation. Here's a sample
import org.testng.IHookCallBack;
import org.testng.IHookable;
import org.testng.ITestResult;
import org.testng.Reporter;
import org.testng.annotations.Test;
import org.testng.asserts.SoftAssert;

public class SomeTest implements IHookable {
    private static final String ASSERTION = "assertion";

    @Override
    public void run(IHookCallBack callBack, ITestResult testResult) {
        SoftAssert softAssert = new SoftAssert();
        testResult.setAttribute(ASSERTION, softAssert);
        callBack.runTestMethod(testResult);
        softAssert = (SoftAssert) testResult.getAttribute(ASSERTION);
        softAssert.assertAll();
    }

    @Test
    public void firstTestMethod() {
        SoftAssert asserts = getAssert();
        asserts.assertEquals(1, 2, "Number equality test");
        asserts.assertTrue(true, "boolean test");
        asserts.assertNull("", "null check");
    }

    @Test
    public void secondTestMethod() {
        SoftAssert asserts = getAssert();
        asserts.assertTrue(false, "boolean test");
        asserts.assertEquals(1, 1, "Number equality test");
        asserts.assertNotNull("", "null check");
    }

    private SoftAssert getAssert() {
        ITestResult result = Reporter.getCurrentTestResult();
        return (SoftAssert) result.getAttribute(ASSERTION);
    }
}


autojump: error: argument -i/--increase: invalid int value: 'work'

OS: macOS High Sierra
zsh: 5.5.1 and the latest master oh-my-zsh code (Add fzf plugin (#6910))
I found that when I write autojump after fasd in .zshrc file (like plugins=(fasd autojump))
The command j return error like "autojump: error: argument -i/--increase: invalid int value: 'work'"
When writing autojump before fasd, the command j working fine.
Before I git pull my oh-my-zsh config to the latest, plugins=(fasd autojump) working well.

I finally found that is "alias j='zz'" which in fasd.plugin.zsh overrided the command j defined by autojum.zsh.
Delete fasd in plugins would solve the problem, but I have no idea how to use them both yet. (Although delete alias j='zz' which added by https://github.com/robbyrussell/oh-my-zsh/pull/3860 would work, but it may not be a good idea). Perhaps the author of https://github.com/robbyrussell/oh-my-zsh/pull/3860 may give some solutions

You can run unalias j or put it in your zshrc file. I don't understand how that crashes autojump though...


CI: For colima, modify CPU count and memory

What is the problem that this fixes or functionality that this introduces? Does it fix any open issues?
https://github.com/lima-vm/lima/issues/2454
Give a brief summary of what the PR does, explaining any non-trivial design decisions
Applied the suggestion from https://github.com/lima-vm/lima/pull/2455#discussion_r1660548765

If you haven't seen it yet, this error is in the logs: "unknown flag: --cpus"


Update ru_ru.lang

This translation has been significantly updated and now corresponds to the current English translation;
Eliminated stylistic errors of the previous translation. The new translation is simpler and devoid of many piercing errors, such as tautology;
the wording of the new translation more clearly reflects the intention of the English translation;
Some successful formulations of the old translation have been retained.

 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.

Thanks! I'll take a look when I have time.

Hello, have you watched it yet?

No, sorry, I didn't have time to look at translations.