add VEX file with vulnerabilities information to SBOM

Dear project owners,
We are a group of researchers investigating the usefulness of augmenting Software Bills of Materials (SBOMs) with information about known vulnerabilities of third-party dependencies.
As claimed in previous interview-based studies, a major limitation—according to software practitioners—of existing SBOMs is the lack of information about known vulnerabilities.  For this reason, we would like to investigate how augmented SBOMs are received in open-source projects.
To this aim, we have identified popular open-source repositories on GitHub that provided SBOMs, statically detected vulnerabilities on their dependencies in the OSV database, and, based on its output, we have augmented your repository’s SBOM by leveraging the OpenVEX implementation of the Vulnerability Exploitability eXchange (VEX).
The JSON file in this pull request consists of statements each indicating i) the software products (i.e., dependencies) that may be affected by a vulnerability. These are linked to the SBOM components through the @id field in their Persistent uniform resource locator (pURL); ii) a CVE affecting the product; iii) an impact status defined by VEX. By default, all statements have status under_investigation as it is not yet known whether these product versions are actually affected by the vulnerability. After investigating the vulnerability, further statuses can be affected, not_affected, fixed. It is possible to motivate the new status in a justification field (see https://www.cisa.gov/sites/default/files/publications/VEX_Status_Justification_Jun22.pdf for more information).
We open this pull request containing a VEX file related to the SBOM of your project, and hope it will be considered.
We would also like to hear your opinion on the usefulness (or not) of this information by answering a 3-minute anonymous survey:
https://ww2.unipark.de/uc/sbom/
Thanks in advance, and regards,
Davide Fucci (Blekinge Institute of Technology, Sweden)
Simone Romano and Giuseppe Scaniello (University of Salerno, Italy),
Massimiliano Di Penta (University of Sannio, Italy)

maybe nice, but as soon as there is a library change within this project (as done today) this file is useless and out-dated.
As long as there is no automated way on updating this file together with the SBOM it wont be part of the project and users can use all other existing software out there that automatically extracts current vulnerabilities from given SBOM files as OWASP Dependency-Track or Anchore Grype (or others) do.

Thanks for your feedback, @sseide.
You’re right that VEX and SBOM should stay aligned. A VEX can easily be added to the deployment pipeline—see our script for reference.
On the other hand, this study aims to gauge how well VEX info is accepted and what potential OSS developers see in it (if any).
We believe creating and managing a VEX is best handled by the project's maintainer, not downstream users.
VEX includes project-specific vulnerability data, like impact_status and justification, to help track vulnerabilities in the context of a specific project independently of the tool used to extract the vulnerabilities.


Clear socket.waiting before invoking callbacks

If we pause() and then later resume() a consumer within a 'done' callback, we can get into a situation where the consumer doesn't actually resume because sendToBroker bails out early (because broker.socket.waiting is still true while the 'done' callback is being invoked, so the fetch request gets thrown away).
To work around this I've had to use setImmediate(() => consumer.resume()) to schedule the resume() to occur after the socket.waiting flag has been reset. It works well enough, but it's not very intuitive.
This change simply moves the socket.waiting reset to occur before we fire off the decoder & associated callbacks. I think this is pretty low risk, but perhaps you can think of a reason why we wouldn't want to do this.
Something else to consider: it can be somewhat surprising that requests to brokers are simply "thrown away" if the waiting flag is set: at a minimum, a log message before the continue in Client.sendToBroker would be great (I can add this if you like, it's easy enough).I fFeel like a more "correct" approach would be to fire the callback before that same continue so callers can at least notice that something went screwy, but that seems like a larger change.

@hyperlink any thoughts on this one? The last part RE: requests thrown away is especially concerning since it means callbacks passed to sendToBroker may never get invoked. (Probably strictly a separate issue -- happy to open a separate ticket for that if it helps.)

@thomaslee I am not too familiar with the history of this code. And I don't understand why it's necessary to have a dedicated socket for fetches that blocks requests using the wait flag. Do you know? Or maybe @haio or @crzidea could shed some light on this?
I have tried in another branch  to do away with this behavior and I've ran into some concerning intermittent test failures in travis.
I will kick off the build again and merge this if it's ok.

Thanks very much for landing this. 🙂

And I don't understand why it's necessary to have a dedicated socket for fetches that blocks requests using the wait flag.

I actually didn't realize there were two sockets at play. Only thing I can think of is that on low-volume topics those fetches could be quite slow (since callers can tell the broker to wait some user-specified amount of time for messages to arrive) ... perhaps the original implementors were concerned about other (non-fetch) requests being sent while a fetch was in flight and "blocked"? My instinct is this shouldn't actually be a problem thanks to correlation IDs etc. -- a "blocked" fetch shouldn't delay other requests like topic metadata requests etc. -- but I don't know enough about the protocol to  say for certain.

I have tried in another branch to do away with this behavior and I've ran into some concerning intermittent test failures in travis.

I'll take a look when I get a sec, but I fear I may not be much help. Still trying to get my head around some of the basics ... the interactions between different layers of the inheritance hierarchy on the consumer side  still hurt my head a bit 😃 .

It is a long time since I wrote the first version of this module, so I can't remember everything clearly.
But it seems that longpolling socket can't fetch broker info while longpolling socket is working, so it's necessary to have a dedicated socket to fetch broker info. Actually, I'm not sure about this.


Unable to run examples on ILI9341

Howto run the exampe on the newly added ILI9341?
This does not work ?
python clock.py --display ili9341 --interface spi --width 320 --height 240

Do you get an error? Does anything happen on the screen? It's hard to know what to advise when you ask vague questions

At first my luma.examples was old and also had issue with my wiring
I updated my examples and re-checked my wiring, and switched to python3
I am able to run the examples now thank you.


Rewrite installation pages to mention auto-installers as the recommended installation method

We're making r2modman, Viper, and VTOL the recommended way of installing Northstar over manual installation.
This change affects the main installation page as well as pages for other platforms like Linux and SteamDeck, and pages reference installation.

i would say this is resolved

Good catch. Closing for now ^^


Showing black on initial recorded video frame

Sometimes, the recorded video has black initial frame. Any clue why?

I figured out the issue. The appendBuffer is writing audio ahead of the video. As a workaround, I don't do startWriting until video is given....
`   func appendBuffer(sampleBuffer: CMSampleBuffer!, isVideo: Bool) {
    if CMSampleBufferDataIsReady(sampleBuffer) {
        if self.assetWriter.status == AVAssetWriterStatus.Unknown {
            if isVideo {
                print("Writer: startWriting")
                let startTime = CMSampleBufferGetPresentationTimeStamp(sampleBuffer)
                self.assetWriter.startWriting()
                self.assetWriter.startSessionAtSourceTime(startTime)
            }
        }
        if isVideo {
            if self.videoInputWriter.readyForMoreMediaData {
                print("Writing video...")
                self.videoInputWriter.appendSampleBuffer(sampleBuffer)
            }
        }
        else {
            if self.audioInputWriter.readyForMoreMediaData {
                print("Writing audio...")
                self.audioInputWriter.appendSampleBuffer(sampleBuffer)
            }
        }
    }
}`

I will check that. Thank you so much for your return ! It really helps.

No problem. Being able to pause and resume recording would be great feature. Great work by the way.

@remirobert Great work on this library 👍 Any chance you know when you'll update the above?

@MFaarkrog I did, it already implemented. Please update your pod.
Do you still have the issue ?

@remirobert
Hey this issue still persists. I checked your code and missing was && isVideo in this line by @roceller swift if self.assetWriter.status == AVAssetWriterStatus.Unknown && isVideo which fixes the problem for me atleast.

@remirobert Sorry for the delayed response. Yeah, I still have the issue in the project I'm working on.
I haven't tried otymartin's fix yet, though.

It's fixed, check the next commit with swift 3 update


If you have a gripe with a pool, and more specifically with a Stratum that sends out work units which apparently got rejected, bring it up with your pool operator. This is a bug tracker which coincidentally has nothing to do with shares being accepted or rejected. That’s all stratums work.


Added test for GetOrphanTasks

What changed?
Added tests

Why?
Increase code coverage

How did you test it?

Potential risks

Release notes

Documentation Changes

I don't think this test adds anything to our code base. Please, focus on the functionality that is actually used. This looks like a test purely for coverage :(

Pull Request Test Coverage Report for Build 018fa436-b062-4053-8e48-158b3af95147
Details

0 of 0   changed or added relevant lines in 0 files are covered.
112 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-0.05%) to 69.211%

Files with Coverage Reduction
New Missed Lines
%

service/history/shard/context.go
1
69.08%

service/history/task/transfer_standby_task_executor.go
2
86.21%

service/matching/tasklist/task_list_manager.go
2
76.48%

common/task/fifo_task_scheduler.go
2
87.63%

service/history/execution/mutable_state_util.go
2
78.52%

service/history/execution/mutable_state_builder.go
9
77.86%

service/history/execution/mutable_state_builder_methods_timer.go
14
67.03%

common/persistence/execution_manager.go
22
83.74%

service/history/execution/mutable_state_builder_methods_activity.go
58
59.02%

Totals

Change from base Build 018fa25a-37ac-4e2f-9de0-0090786e76c2:
-0.05%

Covered Lines:
101950

Relevant Lines:
147303

💛  - Coveralls


Start tsserver with noEmit

Issue Type: Bug
There is a similar issue #70014 but feel like it is necessary to reopen it
Steps to Reproduce:

Create an empty folder and open it with VSCode
Create tsconfig.json

{
	"compilerOptions": {
		"allowJs": true,
	},
	"include": [
		"./main.js",
	]
}

Create empty main.js

VScode would show an error in tsconfig.json sayingCannot write file 'path/to/main.js' because it would overwrite input file . Following #70014 and adding outDir: path/to/out  or  noEmit: true could shutdown that error. However, this doesn't make sense to me because:

--outDir could be add to tsc build script
If a bundler such as webpack is used, it would take care how emitted content should be processed

In neither cases should it prevent VSCode from linting JS files.
IMO, as linting itself doesn't actually emit files, we could start tsserver with noEmit: ture to solve this issue.
 Please file a bug upstream

Although this is likely by-deign now that I think about it a bit more. TS Server has no way of knowing you will use --noEmit so it shows an error assuming you will just run tsc directly. This is correct in the majority of use cases
Just add "noEmit": true in the tsconifig to suppress this

Does Typescript team also maintain the typescript-language-features plugin? It seems to me that the plugin could override these tsconfig settings when starting tsserver.
would like to dig it deep if have some time.
Thanks in advance.


updated with "My Changes!"

updated

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

just trying to get a shirt

Fair enough
On Sun, Oct 21, 2018, 12:01 PM Aditya<EMAIL_ADDRESS>wrote:

@ezioda004 commented on this pull request.
Hi,
Thanks for this pull request (PR).
Unfortunately, we are marking this PR invalid for hacktoberfest. We
believe it does not follow the recommended guidelines:
Quality Standards
In line with Hacktoberfest value #2 (Quantity is fun, Quality is key), we
have provided examples of the quality standards we encourage. This applies
mainly to beginners.

PRs that are automated e.g. scripted opening PRs to remove
whitespace / optimize images.
PRs that are disruptive e.g. taking someone else's branch/commits
and making a PR.
PRs that are regarded by a project maintainer as a hindrance vs.
helping.
Something that's clearly an attempt to simply +1 your PR count for
October.

Learn more at https://hacktoberfest.digitalocean.com/details
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/pull/25358#pullrequestreview-166790646,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AqGKdIKaXVklx-GfRu86C9tZHwSjt1DHks5unJpwgaJpZM4Xyhqm
.


Typescript tsserver support for regular Javascript

I work on several projects, some in Javascript and some in Typescript. Since Typescript's tsserver can support regular Javascript files, it would be nice to have it enabled in ALE's LSP for Javascript as well.
Although there's no type-checking, there will be completion, and :ALEGoToDefinition.

I think you can actually use type checking to with some comments. I don't know if tsserver has support for that.
I think this can be done.

This would finally complete my move from vscode to vim 😆 can't wait!

Any updates on this? I see it was removed from the Version 1.9 milestone, but not added to the V 2.0 project

I haven't worked on it. If you want it badly enough, create a pull request for it.

I'm taking a look at this but pretty new to vimscript. It seems to me the only difference would be the inclusion of the '--checkJs' flag for the javascript files. Perhaps, also a check to verify the correct version of tsserver. I can try to throw something together for this

I was able to add completion using the existing tsserver implementation we have for typescript.
https://github.com/mlwarner/ale/commit/b4b38027fbdf1b6602241f7a9560f2ca02072639
But I had a few questions before going forward.

The --checkJs flag is on the compiler which I can't pass directly to tsserver. We get these from the config https://github.com/Microsoft/TypeScript/wiki/Standalone-Server-(tsserver)#project-system Are we actually using the project config right now? As far as I can tell, we just pass an empty root.

If we can't specify the --checkJs flag directly, should we bundle finding the correct project root change into this task?

checkJs is probably something you can configure in tsconfig.json, which tsserver will load.

Ahhh I see. Tsserver actually looks up the Project tsconfig even though we don't specify a ProjectRootPath. I was able to verify my change works properly by changing tsconfig compile options (with/without checkjs). I'll try to add a test or so to this and open a pull request

I think that for javascript, the project root should be defined by jsconfig.json https://code.visualstudio.com/docs/languages/jsconfig

PR #1653 just loads the TSServer for JavaScript files, the same way it does for TypeScript files.

Implemented by @chaucerbao


Creating ip_address_list using complex example fails

When creating an ip_address_list using the complex example in the documentation (https://github.com/DimensionDataResearch/dd-cloud-compute-terraform/blob/development/v1.2/docs/resource_types/address_list.md) fails with status code 400 (INVALID_INPUT_DATA) error.
ddcloud_address_list.test_list: Creating...
address.#:             "" => "3"
address.0.begin:       "" => "<IP_ADDRESS>"
address.1.begin:       "" => "<IP_ADDRESS>"
address.1.end:         "" => "<IP_ADDRESS>"
address.2.network:     "" => "<IP_ADDRESS>"
address.2.prefix_size: "" => "24"
ip_version:            "" => "IPv4"
name:                  "" => "TestAddresses"
networkdomain:         "" => "ebbd338f-a965-4106-8fa6-8e0ecbdcc1a3"
Error applying plan:
1 error(s) occurred:

ddcloud_address_list.test_list: Request to create IP address list 'TestAddresses' failed with status code 400 (INVALID_INPUT_DATA): At least one ipAddress or childIpAddressListId must be provided.

Looking at the log files, I see that the request generated does not include any information from the address maps:
2016/12/06 17:50:30 [DEBUG] plugin: terraform-provider-ddcloud: 2016/12/06 17:50:30 Invoking 'POST' request to 'https://api-au.dimensiondata.com/caas/2.2/63255a5a-622b-42a4-850e-56a4c3e7e00e/network/createIpAddressList'...
2016/12/06 17:50:30 [DEBUG] plugin: terraform-provider-ddcloud: 2016/12/06 17:50:30 Request body: '{"name":"TestAddresses","description":"","ipVersion":"IPv4","networkDomainId":"ebbd338f-a965-4106-8fa6-8e0ecbdcc1a3","ipAddress":null,"
childIpAddressListId":null}'
Version information
$ terraform --version
Terraform v0.7.13
$ terraform-provider-ddcloud --version
terraform-provider-ddcloud v1.2.0-alpha1 (33515c8e674a81765f2c4cabff55d23f11365ebc)
$ uname -a
Linux cxs-de-mgmt01.cxlab.local 3.10.0-327.36.3.el7.x86_64 #1 SMP Thu Oct 20 04:56:07 EDT 2016 x86_64 x86_64 x86_64 GNU/Linux

Thanks! Sorry for the inconvenience - I'll look into this ASAP.

Hi @mgoldsmi - I've found and fixed the problem (thanks for reporting it). I'll get a new build out shortly that includes this fix.

Ok, should be fixed in v1.2.0-alpha3.
I've also updated our acceptance-test suite to ensure that won't happen again.

Thanks for the quick turn around. Can now create dd_address_list with network address and mask. However the #v1.2.0-alpha3 build appears to have reverted the new ddcloud_server.image (#66) change.
$ terraform plan
There are warnings and/or errors related to your configuration. Please
fix these before continuing.
Errors:

ddcloud_server.vpn-server: image must be a single value, not a list

$ terraform --version
Terraform v0.7.13
$ terraform-provider-ddcloud --version
terraform-provider-ddcloud v1.2.0-alpha3 (7a1232cb1a389036c8bc9ffe76244cd3d048124c)

Actually that's correct - it is a single value, let me check the docs to see if they are up to date...
On 7 Dec. 2016, at 1:09 pm, mgoldsmi <notifications<EMAIL_ADDRESS>wrote:
Thanks for the quick turn around. Can now create dd_address_list with network address and mask. However the #v1.2.0-alpha3 build appears to have reverted the new ddcloud_server.image (#66https://github.com/DimensionDataResearch/dd-cloud-compute-terraform/issues/66) change.
$ terraform plan
There are warnings and/or errors related to your configuration. Please
fix these before continuing.
Errors:

ddcloud_server.vpn-server: image must be a single value, not a list

$ terraform --version
Terraform v0.7.13
$ terraform-provider-ddcloud --version
terraform-provider-ddcloud v1.2.0-alpha3 (7a1232chttps://github.com/DimensionDataResearch/dd-cloud-compute-terraform/commit/7a1232cb1a389036c8bc9ffe76244cd3d048124c)

You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://github.com/DimensionDataResearch/dd-cloud-compute-terraform/issues/72#issuecomment-265334694, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABkezDd7GVQ57uQgbAJuKzabeqbzgbOhks5rFhVqgaJpZM4LFFeH.
itevomcid

Sorry, I think you may have been caught out by an early change we made where the image was of the form image { name = "xxx" type="yyy" } but after user feedback we simplified it to just image = "xxx" (and optionally image_type="yyy").
If you want a working example, take a look here.

(again, sorry for the confusion / inconvenience - we're pretty sure we have the final schema locked down now so there shouldn't be any further breaking changes)

All good. Correction is working for me.


Delete From workspace feautre in File menu of editor not working

Description:
Delete From workspace feautre in File menu of editor not working

It hangs here

Suggested Labels:
Editor,Bug
Affected Product Version:
SP-4.0.0-Beta3
OS, DB, other environment details and versions:
Ubuntu/Linux, Firefox 57, Chrome 63
Steps to reproduce:
1)open a terminal and cd into <SP_HOME>/bin directory and issue./editor.sh to start the editor
2)Launch the editor
3)Click File->Open and select any siddhi app from workspace to open it
4)Try to delete it with Ctrl+D or File-> Delete File
5)Click Delete again on the delete from workspace window
The Delete Windows Hangs as follows

Resolved in https://github.com/wso2/carbon-analytics/pull/894


check "extras" not just "deps" ?

would it make sense to check everything that's in "compat", not just the subset that's also listed in "deps"?  the stuff in "extras" seems not to be checked.  see https://github.com/GiovineItalia/Compose.jl/issues/374

Hello, is there anything I can bump/help with to get this implemented?

I think with #458, a single-line change to loop over ["deps", "weakdeps", "extras"] instead of just ["deps", "weakdeps"] would resolve this issue.

yeah looks like that's it. I'll try to open another PR when #458 gets merged. (Not sure if adding more stuff to that PR would be good)


Refactor to Enforce Principal Name Sanitization

Goal: make sure all principal names are sanitized using the SanitizeName function.
Proposed solution:

Make the Principal struct unexported so that its initializer cannot be called outside a wrappers_interface package which just includes wrappers.go. Move the implementations of the specific wrappers into a new package (like transparentReleaseWrappers).
Make a factory method for constructing principals that runs sanitization like the following:

func ConstructPrincipal(contents String) {
   return principal{ Contents: sanitizeName(contents) }
}

This way, the only way external packages can make a principal is by calling the constructor.
If making the principal struct unexported does not work, another way to enforce this could be to introduce a SanitizedName type produced by SanitizeName(), and make the Principal initializer take one of these.

See also: https://github.com/project-oak/transparent-release/pull/52#discussion_r845596657
via @rbehjati  : check the naming convention for constructors in go as well as making the Contents in the struct unexported (see also the linked thread for more details).


Dev Team Innovation and Planning (FY24 Q4)

Benefit
The final sprint of the quarter is meant to be an Innovation and Planning (IP) Iteration. Benefits include:

Allow time for Innovation
Dedicate time to PI Events (I&A and PI planning)
Integrate the Complete Solution (presentation of work of all teams on the train)
Advance Development Infrastructure
Enable Continuous Learning

Possible tasks include:

Pay down technical debt

app and data dependencies and updates
updating CIs

Other modernization tasks
Cloud migration
"Work on whatever with whomever"
Alignment of best practices

Acceptance Criteria
Possible Tasks

[ ] https://github.com/agrc/raster/issues/169
[ ] Consider https://github.com/step-security/harden-runner this quarter or another
[ ] Revisit the reactfire npm conflicts
[ ] Some action workflows are not completing since their required checks aren’t running
[x] Push swapper to pypi and automate builds https://github.com/agrc/swapper/issues/19
[x] https://github.com/agrc/skid/pull/8
[x] Migrate to actions/create-github-app-token
[ ] Migrate to Artifact Registry

[ ] geocode-job
[ ] unclaimed property
[ ] roadkill
[ ] masquerade
[x] parole
~[ ] cambiador~

[ ] Palletjack: https://github.com/agrc/planning-queue/issues/274
Masquerade:

[ ] https://github.com/agrc/planning-queue/issues/91
[x] https://github.com/agrc/planning-queue/issues/281

[x] Upgrade Pro version on Forklift machine
Skid readme's

[ ] https://github.com/agrc/uorg-skid/issues/3
[ ] https://github.com/agrc/dabs-skid/issues/4
[ ] https://github.com/agrc/wmrc-skid/issues/33
[ ] https://github.com/agrc/broadband-speedtest-skid/issues/5

[ ] Bump agrc/get-next-version-action and agrc/release-composite-action to Node JS v20

Notes
No response
Risks
No response
Issue Reference
refs #311

Copied from #311, before moving more of the Acceptance Criteria tasks to here, and closing 311.

Sprint 6

[ ] https://github.com/agrc/raster/issues/169 (@jacobdadams ?)

Sprint 6 Notes (Steve)

Atlas improvements

Started to replace reactfire from atlas
Started to incorporate design system elements
Started to restructure the UI

Design system improvements

Evaluated building blocks for design system and chose to migrate to react-aria
Created sherlock component
Created Header component
Created Footer component
Created Social component
Started Button component
Started Input component
Started Dartboard component building on Button and Inputs. Stuck on react hook form integration.

API Client improvements

Migrated to an improved build system with vite
Created universal builds for intel and arm based systems
Added a beta channel for testing new features

gis.utah.gov improvements

Evaluated broken link checker and created list of links to triage

UIC improvements After finding out UIC was in production

Fixed a bug with admin notifications
Added a link to a help doc that was created
Fixed a bug with database columns not matching input size limits


Create a top like SusQL command.

Create a CLI tool that displays labeled energy consumption similar to the top command.

Resolved with the following merged pull request. https://github.com/sustainable-computing-io/susql-operator/pull/17


Installation fails on platforms without multiprocessing

On the latest version of requests in pypi, requests requires the multiprocessing module to install correctly, even though it is not used in the installation, only in the unit tests. This means it is no longer possible to install requests on platforms that lack multiprocessing support (such as Android or IoT devices)
Expected Result
Requests is installed and available for use
Actual Result
Installation fails with the following:
Downloading/unpacking requests
  Downloading requests-2.17.3.tar.gz (123Kb): 123Kb downloaded
  Running setup.py egg_info for package requests
    Traceback (most recent call last):
      File "<string>", line 14, in <module>
      File "/tmp/build/requests/setup.py", line 11, in <module>
        from multiprocessing import cpu_count
    ImportError: No module named multiprocessing
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):

  File "<string>", line 14, in <module>

  File "/tmp/build/requests/setup.py", line 11, in <module>

    from multiprocessing import cpu_count

ImportError: No module named multiprocessing

----------------------------------------
Command python setup.py egg_info failed with error code 1
Storing complete log in /sdcard/com.hipipal.qpyplus/.pip/pip.log

Reproduction Steps
From a platform without multiprocessing:
pip install requests

System Information
For obvious reasons, requests.help is not available, but I've tried to piece together the information that it would produce:

platform.system()   Linux
platform.release()   3.14.29
implementation     CPython
implementation_version   2.7.2
ssl.OPENSSL_VERSION_NUMBER   268443727L
requests_version   2.17.3

Appears to be since e6e3589, I'll see if I can do a fix for it

Thanks for this report. It should be entirely possible to pull the multiprocessing call into a tighter scope.


CO2-Ampel board support

Thank your for your great job.
I would like to use WiFiWebServer library on CO2-Ampel board with information as below:
SAMD21 + WINC1501B (WiFi101)
/ Needed for WINC1501B (WiFi101) library
// --------------------------------------
#define WINC1501_RESET_PIN   (12u) // reset pin
#define WINC1501_WAKE_PIN    (11u) // wake pin
#define WINC1501_CHIP_EN_PIN (10u) // enable pin
#define WINC1501_INTN_PIN    (13u) // int pin
#define WINC1501_SPI         SPI
#define WINC1501_SPI_CS_PIN  (14u) // cs pin
https://github.com/mariolukas/Watterott-CO2-Ampel-Plus-Firmware
https://learn.watterott.com/breakouts/co2-ampel/
I tried the following settings with many compilation errors
include <WiFiWebServer.h>
#define USE_WIFI_NINA       false
#define USE_WIFI101           true
Is it possible to make a slight modification on WiFiWebServer library to support CO2-Ampel board running WiFi101 ?
Thank you

Hi @philippesikora
The CO2-Ampel board has been implicitly supported, because it's using SAMD21 and WiFi101 library.
You have to follow these steps

Install the CO2-Ampel core by following the instruction in Compile your own traffic light version from the sources
Select the board CO2-Ampel from Arduino IDE
Compile and upload

This is the compile result as well as the illustration

Illustration

Compile result =>OK

In file included from /home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/AdvancedWebServer.ino:39:0:
/home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/defines.h:76:4: warning: #warning Using WiFi101 using WiFi101 Library [-Wcpp]
   #warning Using WiFi101 using WiFi101 Library
    ^~~~~~~
In file included from /home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/defines.h:348:0,
                 from /home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/AdvancedWebServer.ino:39:
/home/kh/Arduino/libraries/WiFiWebServer-1.3.0/src/WiFiWebServer.h:55:4: warning: #warning Use SAMD architecture from WiFiWebServer [-Wcpp]
   #warning Use SAMD architecture from WiFiWebServer
    ^~~~~~~
In file included from /home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/defines.h:348:0,
                 from /home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/AdvancedWebServer.ino:39:
/home/kh/Arduino/libraries/WiFiWebServer-1.3.0/src/WiFiWebServer.h:117:4: warning: #warning Use WiFi101 from WiFiWebServer [-Wcpp]
   #warning Use WiFi101 from WiFiWebServer
    ^~~~~~~
In file included from /home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/defines.h:348:0,
                 from /home/kh/Arduino/khoih-prog_working/WiFiWebServer_GitHub/examples/AdvancedWebServer/AdvancedWebServer.ino:39:
/home/kh/Arduino/libraries/WiFiWebServer-1.3.0/src/WiFiWebServer.h:141:4: warning: #warning SENDCONTENT_P_BUFFER_SZ using default 4 Kbytes [-Wcpp]
   #warning SENDCONTENT_P_BUFFER_SZ using default 4 Kbytes
    ^~~~~~~
/home/kh/.arduino15/packages/co2ampel/hardware/samd/1.0.7/libraries/WiFi101/src/WiFi.cpp: In member function 'char* WiFiClass::firmwareVersion()':
/home/kh/.arduino15/packages/co2ampel/hardware/samd/1.0.7/libraries/WiFi101/src/WiFi.cpp:343:7: warning: '%d' directive writing between 1 and 3 bytes into a region of size between 1 and 5 [-Wformat-overflow=]
 char* WiFiClass::firmwareVersion()
       ^~~~~~~~~
/home/kh/.arduino15/packages/co2ampel/hardware/samd/1.0.7/libraries/WiFi101/src/WiFi.cpp:343:7: note: directive argument in the range [0, 255]
/home/kh/.arduino15/packages/co2ampel/hardware/samd/1.0.7/libraries/WiFi101/src/WiFi.cpp:356:10: note: 'sprintf' output between 6 and 12 bytes into a destination of size 9
   sprintf(_version, "%d.%d.%d", rev.u8FirmwareMajor, rev.u8FirmwareMinor, rev.u8FirmwarePatch);
   ~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Sketch uses 59052 bytes (22%) of program storage space. Maximum is 262144 bytes.
Global variables use 3788 bytes of dynamic memory.

Enjoy and be safe,
Good Luck,

Thank you for your quick reply.
I used AdvancedWebServer example, select the board CO2-Ampel from Arduino IDE, compile  (set my ssid[ ] & pass[ ] ) and upload successfully to the board
I got the following result on serial monitor:
Starting AdvancedServer on SAMD MKR1000 with WiFi101 using Wifi101 Library
WiFiWebServer v1.3.0
Connecting to WPA SSID: sceslaw
HTTP server started @<PHONE_NUMBER>
I cannot see my IP address

The very old WiFi101 library uses the medieval way to return IP address in uint32_t, such as
uint32_t localIP();

whereas the modern library returns IPAddress, such as
IPAddress localIP();

The<PHONE_NUMBER> in uint32_t is IPAddress(192, 168, 7, 179) or <IP_ADDRESS>
You have to use a function to convert from uint32_t to IPAddress to modify WiFi.localIP() in
Serial.print(F("HTTP server started @ "));
Serial.println(WiFi.localIP());

If you need help to find the function to convert from , ask in Arduino Forum, etc.


[$500] Chat - User paste 2 email together, the part after @ is not considered as url & shown as link

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.3.94-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause-Internal Team
Slack conversation: @
Action Performed:

Go to https://staging.new.expensify.com/
Tap on a report
Paste a email id twice without space together and send it eg: Dghg<EMAIL_ADDRESS>
Expected Result:
ry.com must be displayed as url.
Actual Result:
After com, Dghg is considered as email and shown as url. But ry.com is not shown as url.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019e814827368ea6c0
Upwork Job ID:<PHONE_NUMBER>051106816
Last Price Increase: 2023-11-01

I don't know if there's an optimal way to handle this, if a user pastes nonsense like this I think the current behavior is what I'd expect.

I agree with @erquhart, this looks like the expected behavior. This is how Slack handles it as well, see below:

In step 3 of Action Performed above GitHub actually shows the entire thing as 1 URL. The question is what do you want to achieve functionally here?
Do you want the two email addresses to appear as one and be clickable?
Or do you want some logic that can assess this and identify that it's actually two or more addresses joint together and then split them up?
In scenario 1 that is not a valid email addresses since it contains two @ signs and it will act as a faulty URL. But at least it's clickable as a whole and the user's original text is not altered.
In scenario 2 you have two separated addresses that both qualify as valid email addresses and are clickable, but it means changing the user's original content, which might not be what you want.
If you could clarify a bit more what the functional idea behind the expected result is, that would help to assess whether this is a good idea and then also how to suggest fixing it.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01677298654b2ba748

Okay, so just parse it as if it's a link, noted. I would like to fix it, but I think your rules state that I have to provide a solution here first before pushing a proposal on Upwork, is that correct? If so, I'll have to peek quickly through your codebase and provide a more detailed solution. Please let me know.

Do you want the two email addresses to appear as one and be clickable?

@NisargDeveloper @AlienistSolution No. The expected result is listed in the description.

I would like to fix it, but I think your rules state that I have to provide a solution here first before pushing a proposal on Upwork, is that correct? If so, I'll have to peek quickly through your codebase and provide a more detailed solution. Please let me know.

There's no proposal for an issue process on Upwork. You will get the Upwork offer after you've been assign to the issue.

Proposal
Please re-state the problem that we are trying to solve in this issue.
When two emails are pasted in with no space between them, the first email address is treated as a link, except the first email address domain is concatenated to the beginning of the second email address. There is a trailing @ symbol after the link. The request is for the domain after the trailing @ symbol to be styled as a link.
What is the root cause of that problem?
The user pasted two email addresses with no space in between.
What changes do you think we should make in order to solve the problem?

I do not believe a change should be made. If you change your message parsing to try and create an optimal outcome for this, I would fully expect new bugs to occur because of it. The requirements also don't account for the fact that the initial email address being styled as a link here is incorrect, as it concatenates the beginning of the second email address to the TLD of the first.
Also, as another commenter mentioned, Slack handles this the same, so you're in good company.
What alternative solutions did you explore? (Optional)
None, I recommend not making changes to fix this specific case.

@mollfpr Thanks. Will Wait then

Hey I was wondering if you still need a help

@mollfpr please review proposal 🙇♂
@NisargDeveloper we're still accepting proposals!

I already gave the proposal... I
I think we have to parse the whole input as an URL. Then it should work fine.
There is one more Library in React/ React Native we can use which parses everything into a link.
Let me know if you want to solve this issue. THanks

@NisargDeveloper Please post the proposal in the issue, not in the Upwork.
@dylanexpensify I think the proposal from @erquhart is we do nothing. I am curious about the benefits of the expected result.

Proposal
Please re-state the problem that we are trying to solve in this issue.
When pasting two email it is not showing as link
What is the root cause of that problem?
It looks like we are not parsing whole email address as an input. That is the reason it is causing an issue.
What changes do you think we should make in order to solve the problem?
We have to store in input field and once we get the data we have to parse it as email address. Right now it seems it is happening by default.

What alternative solutions did you explore? (Optional)
We can use React/React native library to solve this problem.

Got it - @shawnborton @Expensify/design what do we think?

I kinda think we should do nothing?
Echoing others, I don't really see what we gain by updating this—and "updating" is the key word here to me, I don't think we're "fixing" anything by changing it.
I don't like the idea of trying to discern that the user actually meant to paste two separate emails, that feels way too complicated and error prone. And if we just make it a link, and the user clicks on Dghg<EMAIL_ADDRESS>they're just going to be taken to nothing because the URL is invalid?
I think this is a case of "the user made a mistake and needs to fix it, and the system is behaving how I'd expect it to behave if I made that mistake" Just my 2 cents! 🤷♂

@mollfpr any chance this counts as my proposal being accepted 😬

@erquhart I'm afraid not, there are no changes you propose and there will be PR to work on.

Fair enough - but I will say, that incentivizes contributors to just find a way to get a bugfix proposal accepted, rather than help drive conversations at the product/design level. Not doing something is a really valuable decision. But I digress.

Proposal
Please re-state the problem that we are trying to solve in this issue.
When pasting two email it is not showing as link
What is the root cause of that problem?
It looks like we are not parsing whole email address as an input or developing the regex in such as way that accepts two emails. That is the reason it is causing an issue.
What changes do you think we should make in order to solve the problem?
We have to store in input field and once we get the data we have to parse it as email address. Right now it seems it is happening by default.

What alternative solutions did you explore? (Optional)
We can use React/React native library to solve this problem.

@mollfpr @dylanexpensify should this be closed, proposals are still coming in

I believe we can close this.


Sort vulnerability processing order by date, most recent ones first

Date-like fields:

published_date
last_modified_date
patch_date
cve_id

Fixed in 99f8d7d52c4c225f4fa50aaeb661177f04f090aa.


Have you had any success getting a response from /search-entries with 'password' as source?

I'm on day 2 of trying to use this API and I think I've realized that the source I need to use to convert English to Italian is 'password', but I can't seem to get it to work. for example: '/search-entries?source=password&text=sweet&language=en' doesn't return any results.
Do you have any idea what I might be doing wrong? I'm having trouble finding any other relevant information on this API. My google search brought me to your .NET client, so I'm hoping you might know.
Here's the full request in C#:
var request = new HttpRequestMessage
{
	Method = HttpMethod.Get,
	RequestUri = new Uri("https://lexicala1.p.rapidapi.com/search-entries?text=sweet&language=en&source=password"),
	Headers =
	{
		{ "X-RapidAPI-Key", "api-key" },
		{ "X-RapidAPI-Host", "lexicala1.p.rapidapi.com" },
	},
};
using (var response = await client.SendAsync(request))
{
	response.EnsureSuccessStatusCode();
	var body = await response.Content.ReadAsStringAsync();
	Console.WriteLine(body);
}

Hi Stephen, I am not familiar with the password dictionary, I've only used the global dictionary. But I also see that there have been some changes to the API, the base URL was changed from https://dictapi.lexicala.com to https://lexicala1.p.rapidapi.com and uses an API key now instead of basic auth (username + password) (but the old api still works, so I never noticed it)
So I it will take me some time to rewrite some stuff to update to the new api.
Having said that, using the old apiI also don't get any search results when searching in the password dictionary, so I think your code is be correct, but there is some other issue why it's not working. So I suggest to contact Lexicala support for this (and let me know if they have an answer for you :) )

Thanks for the response! I'll ask them how to get search results in the Password dictionary and if I hear anything I'll try to remember to let you know

As I was writing my email this morning about the Password dictionary it just started working. I'm not sure what changed since I used the same parameters, but give it a try with the new api base url and let me know if it works for you too

Yes same here, it works now with the old api version. So I'll close this issue then


[Ansible] option to keep easyrsa "context directory"

it would be useful to keep the easyrsa directory (https://github.com/kubernetes/contrib/blob/master/ansible/roles/kubernetes/files/make-ca-cert.sh#L74) around on the client side in order to generate additional client certificates.

@imsky The latest changes keep the ca keys for apiserver and etcd, so new certificates can be generated.


Consider creating a minimal summary file inside the image

Just a random idea...
It might be useful for rootfs.sh to create a plan text file inside the rootfs directory. That file could potentially be used for debugging and would include some basic details about the image such as:

Base distro name image created from (fedora, clearlinux, etc).
Base distro version image created from (27, 12345, etc).
Timestamp of when the rootfs was created
Something like date '+%Y-%m-%d.%H:%M:%S.%N%z'
Version of osbuilder used to create the image
Build type (default or custom maybe where the latter would denote that extra packages had been added to the basic image [maybe for debugging]).with lots of extra packages?)
et cetera

As for the format of this file, if we decide to do this, I vote for YAML :)
/cc @sboeuf as the agent could potentially use this?

Any thoughts on this @sboeuf, @bergwolf?
Two main thoughts:

Create outside the images in /usr/share/kata-containers.
This would be a little like /boot/config-$(uname -r) and would allow the data collection script to include the image metadata in its output.

Tweak the agent to read the image metadata generated by osbuilder and display details in its initial announcement message iff debug is enabled.
That would require us to vendor YAML into the agent of course.

@jodh-intel I am fine with adding such a file inside the image. The agent should not error in case it cannot read from this file though, because we cannot enforce that an image has to contain such a file.

Thanks @sboeuf. And I agree about the agent not erroring.

If we decide to add a debug console, we'll also need the following for a debug image:
build-type: debug

or if we decide to only create a single image which includes the ability to create a debug console:
build-type: default
capabilities: debug-console


Inline suggestion

Details

This PR updates the suggestion box to appear right next to the cursor, making it work like inline autosuggestion. Now, the emoji suggestion box only pops up when typing. It won't appear if you move the cursor with a mouse or finger.
Fixed Issues

$ https://github.com/Expensify/App/issues/16078
PROPOSAL:
Tests
Testing Emoji Suggestions:

Open the chat interface.
Type an emoji trigger, starting with :sm followed by at least two more characters (e.g., :smile).
Verify that the suggestion box appears adjacent to the cursor.
Move the cursor to a new position in the text field.
Again, type an emoji-like text with at least two characters following the :sm prefix.
Confirm that the suggestion box appears next to the cursor as expected.

Testing Mention Suggestions:

Open the chat interface.
Initiate a mention by typing the @ symbol followed by any text (e.g., @username).
Ensure the suggestion box pops up next to the cursor.
Reposition the cursor within the text field.
Type another mention, starting with the @ symbol.
Check that the suggestion box is correctly displayed adjacent to the cursor.

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/48593211/af4f6926-cc8a-454a-9244-cfa668b97266

Android: mWeb Chrome
https://github.com/Expensify/App/assets/48593211/fe5a5673-7e83-468e-80d1-338fc471a79b

iOS: Native
https://github.com/Expensify/App/assets/48593211/c9d81e17-1c1f-4781-bc9d-e0a3d4b5a29d

iOS: mWeb Safari
https://github.com/Expensify/App/assets/48593211/33cb3bda-064d-453e-81d0-3544676dc8a2

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/48593211/2144b498-3b29-4e76-b569-aff52a00b918

MacOS: Desktop

❗ ❗ ❗This PR contains https://github.com/Expensify/App/pull/35149. So, don't mind it while reviewing. Thanks!

@perunt I think something is wrong with mention suggestion
https://github.com/Expensify/App/assets/59907218/50cba2a8-a9fa-48d1-b467-86f4a6933e20
https://github.com/Expensify/App/assets/59907218/274449a3-7036-40e1-843d-c71fa3d43648

Hmm, does this issue occur every time?
I have tried several times but couldn't reproduce the issue.
Just to clarify, this PR should not affect the mechanics of inserting emojis/suggestions. Also, I've just pulled the latest changes from the main branch. We've got some components switched over to TypeScript, so I had to make a bunch of type adjustments. Could you give it another go now?
https://github.com/Expensify/App/assets/48593211/ec47ae73-e3ab-4c19-b8f9-a9716e604c23

It is fixed. I will finish reviewing tonight

In safari after selecting emoji or mention suggestion the cursor is going to the end of the text rather than after emoji or suggestion
https://github.com/Expensify/App/assets/59907218/99163953-e20d-4440-ac02-c6e831eee2f1
The same is happening in ios but it is a bit hard to reproduce since suggestion itself is not showing up consistently
https://github.com/Expensify/App/assets/59907218/ed7350c2-5ac5-45b5-85f4-8b57d4a3757a

@perunt Any updates?

I'm closing this PR as it has become outdated. We changed our approach a bit and have now expanded react-native-keyboard-controller to include the changes we previously patched to react-native. The new PR is here


Feat:subscribe to news letter in the footer

Describe the feature
Adding Subscribe to newsletteer in the footer
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@Suchitra-Sahoo Can you please assign me the issue


[WIP] [ALLUXIO-2647] Split Client module

https://alluxio.atlassian.net/browse/ALLUXIO-2647
TODOs:
Update docs
Determine how to name hierarchy

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14040/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14050/
Test PASSed.

Merged build finished. Test PASSed.

please resolve merge conflicts

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14057/
Test PASSed.


ScriptBlock.InvokeWithContext cannot set PSCmdlet

Steps to reproduce
InvokeWithContext lets you set a lot of scope intrinsic variables like $_, $this, $input but it cannot set $PSCmdlet.
Expected behavior
{ $PSCmdlet }.InvokeWithContext($null, [psvariable]::new('PSCmdlet', 'testing'))
# testing

Actual behavior
{ $PSCmdlet }.InvokeWithContext($null, [psvariable]::new('PSCmdlet', 'testing'))
# no results

Environment data
Name                           Value
----                           -----
PSVersion                      7.2.0
PSEdition                      Core
GitCommitId                    7.2.0
OS                             Microsoft Windows 10.0.18362
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

The use case for this is any Use-* type command with a body script block. Think something like this:
Use-Location './project' {
    dotnet build
}

I was trying to avoid having to set some temporary variables for scope intrinsics but PSCmdlet in particular doesn't seem to work.

I think this is because this automatic variable serves to process object parameters. $args and $PSBoundParameters don't work out either.

$args is set by the third (omitted in the examples) parameter.$PSBoundParameters is another that would ideally also work but currently does not. Note that this API already has some code paths that explicitly enable the setting of some other scope intrinsic variables like $this, $input, $_/$PSItem.
It makes sense why it doesn't work, but it would be useful if that were changed.


Fix #871: Adjust aggregator-rules input_pattern match greediness

... to support numeric matching after captured field. Fixes bug #871 .
Before, an input regexp of "collectd.mycluster-<location>\d+.app.requests" in aggregation-rules.conf, with a hostname of "mycluster-paris105" would capture "paris10" as the location, instead of paris. With a large cluster this would create a whole lot of aggregates, one per 10 nodes.
This patch makes the <location> match not greedy, letting the user-supplied \d+ side to be greedy and eat the 105 away fully, leaving "paris" as the location.

@deniszh It seems that pyhash does not support py27 anymore which results in failing travis builds. We should drop py27-pyhash or pin pyhash to some older version.

@piotr1212 : I'm voting for drop, but pinning also OK for me.
@hessu : thanks for fix, LGTM!


[in_app_purchase_storekit] Re-add reference to tests to the Mac project after bots are upgraded to Mac 14

https://github.com/flutter/packages/pull/7812
Even thought StoreKitTest is technically a mac 11 framework, the bulk of the testing features, such simulating network errors and things like that are mac 14 only. Since our fleet is part mac 13 and part mac 14 right now, the tests only pass when being run on a mac 14 bot.
We should re-add reference to tests to the Mac project after bots are upgraded to Mac 14. They just need to be referenced in place, since mac and ios share the same tests.

Added to https://github.com/flutter/flutter/issues/148870


log in to facebook account

I cannot log in to my facebook account because two-factor authentication is on and the hacker has all three options. so even if i have already changed my email and password, i cannot log in to my account. how to log in to my account without two-factor authentication options. And i have already submit my ID since my account was created based on our products name called 'Pladco Jerrycan'. It is not a person name.

Please contact Facebook support https://www.facebook.com/help/1216349518398524
This is the wrong channel.


[bug]:  flutterfire configure always add GoogleService-Info.plist to project.pbxproj even it is already exists

Is there an existing issue for this?

[x] I have searched the existing issues.

CLI Version
0.3.0-dev.18
Firebase Tools version
12.7.0
Flutter Doctor Output
[✓] Flutter (Channel stable, 3.13.9, on macOS 14.1.1 23B81 darwin-x64, locale en-ID)
• Flutter version 3.13.9 on channel stable at /Users/husainazkas/fvm/versions/stable
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision d211f42860 (4 weeks ago), 2023-10-25 13:42:25 -0700
• Engine revision 0545f8705d
• Dart version 3.1.5
• DevTools version 2.25.0
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
• Android SDK at /Users/husainazkas/Library/Android/sdk
• Platform android-33, build-tools 33.0.2
• Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
• All Android licenses accepted.
[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
• Xcode at /Applications/Xcode.app/Contents/Developer
• Build 15A507
• CocoaPods version 1.13.0
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2022.3)
• Android Studio at /Applications/Android Studio.app/Contents
• Flutter plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
🔨 https://plugins.jetbrains.com/plugin/6351-dart
• android-studio-dir = /Applications/Android Studio.app
• Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
[✓] VS Code (version 1.84.1)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.76.0
[✓] Connected device (3 available)
• sdk gphone x86 64 (mobile) • emulator-5554 • android-x64    • Android 13 (API 33) (emulator)
• macOS (desktop)            • macos         • darwin-x64     • macOS 14.1.1 23B81 darwin-x64
• Chrome (web)               • chrome        • web-javascript • Google Chrome 109.0.5414.119
[✓] Network resources
• All expected network resources are available.
• No issues found!
Description
I'm working with flavors (and ios schemes).

Steps to reproduce
Just run the script then it will adds GoogleService-Info.plist to project.pbxproj every time run.
Expected behavior
Only add GoogleService-Info.plist to project.pbxproj if no exists OR update existing reference. It means, it should not duplicate like unexpected in the description.
Screenshots
No response
Additional context and comments
Besides that, why I install dart pub global activate flutterfire_cli 0.3.0-dev.18 --overwrite then I run flutterfire_cli --version it shows 0.3.0-dev.16 ? I think you may forgot to run generator version, LOL (see https://github.com/invertase/flutterfire_cli/blob/flutterfire_cli-v0.3.0-dev.18/packages/flutterfire_cli/lib/version.g.dart)

Thanks for the report, I have observed this behaviour and will get a fix out for it 👍


No file created

Tried the plugin on an interactive grid, but there is no export file created.
I can see in the debug the process plugin is executed succesfully, but there is no file created.

If you enable the APEX debug mode the plugin should return a log of its operation instead of the xlsx-file, but even that only works if the plugin is really executed. Are you sure that the plugin is executed at process point "Before Header"

That was it, before header.
Now i'm getting this error (nb. it's a large report with lots of columns).
Starting at 02-11-2018 11:35:26
owner: ...
Oracle Database 11g Enterprise Edition Release <IP_ADDRESS>.0 - 64bit Production
APEX <IP_ADDRESS>.08
APEX to XLSX version 0.800
DUTCH_THE NETHERLANDS.WE8MSWIN1252
APX50;APX50;SYS$USERS;an00s232
installed on:<PHONE_NUMBER>3815
num1: 4FF2-6344-67FF-F9A8
num2: F91F-8E38-52EE-495C
no licence key provided
785650297B9F59E7E053E8001EAC4E01
use default template
looking for report regions
found Artikelen Interactive Grid NATIVE_IG
trying primary report
my query
select .......
new sheet added
ORA-00972: Identificatiesymbool is te lang.
ORA-06512: in "SYS.DBMS_SQL", regel 1199
ORA-06512: in "APEX2XLSX", regel 1885
Finished at 02-11-2018 11:35:26

It would help if you can send me (by mail?) the generated "my query" from the log file

Thanks for the update, the ORA-00972 error is solved.
But now I run into a new issue, I have a column in the grid of type "SQL expression" and this results in the error :
ORA-00904: "COLUMN1": ongeldige ID
ORA-06512: in "SYS.DBMS_SQL", regel 1199
ORA-06512: in "VIRGO.APEX2XLSX", regel 1889
PS. Reinstalling the plugin does not recreate the SQL function, so I have to remove the function manually.

If have extended the function to handle that too. And changed the plugin to reinstall the function when a higher version of the plugin is installed (and run again afterwards)

Thanks, both are working. Still have one issue with the solution,but not sure if this is an issue specific for me.
It uses the column name from the report as the query alias for the SQL expression. But this column is also a DB column, resulting in a ORA-00918 column ambiguously defined. The reason is that I changed the db column to a SQL expression.  In my opinion not an unrealistic scenario to do.
Also I'm wondering how to implement the "Before header" proces on my page. If I use a button condition it fires always. If I use a Request condition it never fires.

I always use Request condition

But make sure that yout button is making a request, that value is rather "hidden", at the "Target" popup

Thanks for the explanation. I was doing a submit, not a redirect.
Now only have the issue of the "ORA-00918 column ambiguously defined.".
Probably can work around it, but like said before, seems like something other people can run into quite easily.

I disagree, using a SQL-expression doesn't seem to be used that much, and certainly not giving that SQL-expression the same name (id) as an existing column. But besides that (disagreement), that issue is fixed too.

Thanks, issue solved.


HMAC implementation does not properly handle keys greater than digestmod's block-size

Problem or feature statement
Larky's HMAC differs from python3's implementation of HMAC when the length of the key is greater the digestmod's block-size. As per HMAC specs (RFC 2104):

The definition of HMAC requires a cryptographic hash function, which
we denote by H, and a secret key K. We assume H to be a cryptographic
hash function where data is hashed by iterating a basic compression
function on blocks of data.   We denote by B the byte-length of such
blocks (B=64 for all the above mentioned examples of hash functions),
and by L the byte-length of hash outputs (L=16 for MD5, L=20 for
SHA-1).  The authentication key K can be of any length up to B, the
block length of the hash function.  Applications that use keys longer
than B bytes will first hash the key using H and then use the
resultant L byte string as the actual key to HMAC. In any case the
minimal recommended length for K is L bytes (as the hash output
length).

Testing scenarios
This was discovered using HMAC with a SHA1 hash function and a key greater than 64, which is SHA1's block size.
When the key length is less than or equal to 64, it works fine. You should see identical responses from both Larky and local python3 when running the equivalent of this script in Larky:
import hmac
from hashlib import sha1

key = b'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434ddabede0f3b482cd9aea9434d'
msg = b'hello world'
hmac.new(key, msg, sha1).hexdigest()

# supposed to be 85a5308073975a335e69381949fa33e269ff3b1a

However, once the key length surpasses 64, you start to see discrepancies:
import hmac
from hashlib import sha1

key = b'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434ddabede0f3b482cd9aea9434dt'
msg = b'hello world'
hmac.new(key, msg, sha1).hexdigest()

# supposed to be 6a176496acfd81da8d2d88514a4ec453a0028fbc, but is different in Larky

As the spec above states, when the key is greater than the block-size of the hash function, we should first apply the hash function to the key itself and use the resulting digest as the key. If we try that with the above example, it still yields the same result:
import hmac
from hashlib import sha1

key = b'aaf4c61ddcc5e8a2dabede0f3b482cd9aea9434ddabede0f3b482cd9aea9434dt'
key_hashed = sha1(key).digest()
msg = b'hello world'
hmac.new(key_hashed, msg, sha1).hexdigest()

# results to 6a176496acfd81da8d2d88514a4ec453a0028fbc, same as above, but is different in Larky

Hi @akuchibotla - this has been fixed with an accompanying test https://github.com/verygoodsecurity/starlarky/commit/a7d78971dddf487ed67f9d30a8c8fa54d6207711 by this commit: https://github.com/verygoodsecurity/starlarky/commit/020fde286b4fca6e16cbbb44b81fd3584833f07a. Should be released shortly /cc @aslepakurov


Add Overview to README.md

Capture some information about the high-level behavior and proper usage of this library.
I tried to leave out details that are implied by "OAuth2.0 Authorization Code Grant with PKCE", but I think the query-string read-and-erase behavior is unusual and unique to this library, so worth explaining.
Also the refresh behavior, which is different from traditional flows where a third-party back-end privately stores a refresh token on behalf of a user.

Closing out remaining PRs.


package.metadata.requires-dist in uv.lock is incomplete for last added packages.

package.metadata.requires-dist in uv.lock isn't updated with the version specifier on add. only subsequent add, lock, or sync calls add it.
➜ uv --version
uv 0.3.3 (Homebrew 2024-08-23)

➜ uv add ruff
Using Python 3.12.5
Creating virtualenv at: .venv
Resolved 2 packages in 190ms
   Built project @ file:///private/tmp/project
Prepared 2 packages in 695ms
Installed 2 packages in 2ms
 + project==0.1.0 (from file:///private/tmp/project)
 + ruff==0.6.2

➜ cat uv.lock | grep requires-dist
requires-dist = [{ name = "ruff" }]

➜ uv add polars
Resolved 3 packages in 5ms
   Built project @ file:///private/tmp/project
Prepared 2 packages in 139ms
Uninstalled 1 package in 0.51ms
Installed 2 packages in 2ms
 + polars==1.5.0
 ~ project==0.1.0 (from file:///private/tmp/project)

➜ cat uv.lock | grep -A 3 requires-dist
requires-dist = [
    { name = "polars" },
    { name = "ruff", specifier = ">=0.6.2" },
]

➜ uv sync
Resolved 3 packages in 16ms
Audited 3 packages in 0.05ms

➜ cat uv.lock | grep -A 3 requires-dist
requires-dist = [
    { name = "polars", specifier = ">=1.5.0" },
    { name = "ruff", specifier = ">=0.6.2" },
]

Thanks! Will take a look, looks wrong.

Oh this actually does make sense (we update the pyproject.toml after the resolution, to set the lower-bound specifier). Difficult to fix but we definitely need to.


License?

Hi! You don't have a license on your code. Is it openly licensed?

I've added a MIT license.

Thanks!


Added performance note on sprites to README

Looks good to me

:+1:

Hi, could you do a PR on the develop branch? Thanks :+1:

Sorry! #149


Updated CLA instructions

The CLA instructions were outdated.

Hello @thinkingserious,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Unhandled issue when installing package

This problem (see below) occurs on two separate machines.
Desktop i5 Win7 x64
Laptop i7 Win10 x64
Whenever installing the package from within atom, I got the error below.
Uninstalling the package, restarting atom then retry did not solve the problem.
However after 5 unsuccessful attempts the install succeeded on the laptop, but never on the desktop.
Installing<EMAIL_ADDRESS>failed.

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: EPERM, open 'C:\Users\UserName\AppData\Local\Temp\apm-install-dir-116113-15448-x244dc\node_modules\atom-beautify\node_modules\jscs\node_modules\babel-jscs\node_modules\babel-core\node_modules\core-js\modules\core.log.js'

I meet this problem too when I install it.
Installing<EMAIL_ADDRESS>failed.Hide output…

events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: ENOENT, open 'C:\Users\username\.atom\packages\atom-beautify\node_modules\handlebars\dist\amd\handlebars\decorators\inline.js'

Installing<EMAIL_ADDRESS>failed.Hide output…
EPERM, unlink 'C:\Users\username\.atom\packages\atom-beautify\node_modules\typescript-formatter\node_modules\commandpost\node_modules\es6-promise\lib\es6-promise\asap.js'

All these errors look like dependency related failures.  I have been seeing this a lot lately and I am not sure why dependencies are failing to come in.
Try this from the Windows command line: apm install atom-beautify

Did reinstalling the package solve the problem for you guys?

Eventually after several attempts it works at some point (same package), but I have no clue what causes the install to work at some point...

Did you restart / reload Atom at any point? I find issues like this is almost, if not, always a Node.js dependency caching issue and Atom does not clear the package's cache properly. So I make a habit of quickly reloading Atom after updating.

It is quite likely that I tried to solve the problem by restarting Atom,
but I don't remember if the install succeeded right after that.
2016-03-10 7:37 GMT+01:00 Glavin Wiechert<EMAIL_ADDRESS>
Did you restart / reload Atom at any point? I find issues like this is
almost, if not, always a Node.js dependency caching issue and Atom does not
clear the package's cache properly. So I make a habit of quickly reloading
Atom after updating.
—
Reply to this email directly or view it on GitHub
https://github.com/Glavin001/atom-beautify/issues/803#issuecomment-194696141
.


ADBDEV-6139 Fix clang compiler errors

Fix clang compiler errors
This patch fixes multiple errors produced by clang:

-Wimplicit-fallthrough was fixed by adding generic macro
-Wunused-function
-Wunused-variable and -Wunused-but-set-variable
-Wvla-cxx-extension
Unknown "asm" keyword
-Wdeprecated-declarations in third party headers was fixed by disabling this error for particular headers

Also, this patch adds an ability to build gpdb in a docker container using clang

What about creating a new dockerfile for clang? We need to check compilation only. It's redundant to run make install and mocking tests two times.

The fix will be split into several PRs


Does it work with MongoDB ?

I'm thinking about using forem in application which is based on MongoDB. Will it work without additional changes?

Do we have a Mongoid Patch or fork available? for forum to work ?

The answer is still no.

On 9 Jan 2015, at 15:34, Ronak<EMAIL_ADDRESS>wrote:
Do we have a Mongoid Patch or fork available? for forum to work ?
—
Reply to this email directly or view it on GitHub.

Hi, I know this is a bit old but to anyone who was asking for a mongoid version of forem, I've been working on a separate engine modeled after forem for mongoid. Keep in mind, this isn't forem or a fork of forem (although it does borrow a lot of it's code):
https://github.com/NJayDevelopment/mongoid_forums/
It's not 100% complete (I'm open to PRs), but I think that if you were looking for a forum system for mongoid that was similar to forem, this would be fairly good. Here is a demo app (note: this is not the one in test/dummy) if you were curious to see how far along the project is. Feel free to make an account:
http://mongoid-forums.herokuapp.com


Added a configuration property for canvas update Timeout, defaults to 0

Resolves #3

Hello! Please don't forget me! :)


Interactive linter and HTML: "no start tag"

I am posting this issue despite that I am aware that it might not be an issue directly attributable to the Interactive Linter plugin. When examining the following piece of HTML code:
<form name="tle-select" ng-controller="tleSelectCtrl">
    <input type="radio" ng-model="i.tle_select" value="manual">manual</input>
    <input type="radio" ng-model="i.tle_select" value="auto">auto</input>
</form>

... Interactive Linter returns the following two errors (one per line) as a result of the inspection:
Tag must be paired, No start tag: [ </input> ]	 <input type="radio" ng-model="i.tle_select" value="manual">manual</input>
Tag must be paired, No start tag: [ </input> ]	 <input type="radio" ng-model="i.tle_select" value="auto">auto</input>

I have also attached a screenshot of Brackets Release 1.3 experimental build 1.3.0-16022, under Debian 8 "Jessie".
Any ideas?

hey @rtp-calpoly ! I was looking over your code and I think the issue is that input is a void element. You shouldn't be closing it.  This is from W3.org

A void element is an element whose content model never allows it to have contents under any circumstances. Void elements can have attributes.

The following is a complete list of the void elements in HTML:

area, base, br, col, command, embed, hr, img, input, keygen, link, meta, param, source, track, wbr

You can find what I quoted in section 4.3
http://www.w3.org/TR/html-markup/syntax.html

@andrewnaumann : yes, you are right. I did not know that the HTML specification permitted that kind of "relaxation" over the XHTML specification. I have changed my code to a self-closing input tag linked to a label:
<form name="tle-select" ng-controller="tleSelectCtrl">
    <input id="manual-radio" type="radio" ng-model="i.tle_select" value="manual" />
    <label for="manual-radio">manual</label>
</form>

Now it is working perfectly. Thanks for the help.


Remove bad Registry spellings of Organisation

The Registry previously allowed multiple spellings of 'organization' through use of the CKAN API. These have all been consolidated into a single spelling. As such, references to alternative spellings have been removed.

Nice work!


Wrong type for TypeMapping custom.

Hey guys,
How are you?
I have found an error when generate code with a typeMapping.
In my case I have a few class that I need to keep originally for compatibility reasons.
So that in this point I have added in typeMapping properties:
typeMapping = [ 			"Long": "java.lang.Long", 			"DateTime":  "java.time.LocalDateTime", 			"Date": "java.time.LocalDate", 			"BigDecimal": "java.math.BigDecimal", 			"DateTimeRangeFilter": "com.example.graph.filter.DateTimeRangeFilter", 			"DateRangeFilter": "com.example.graph.filter.DateRangeFilter", 			"LongRangeFilter": "com.example.graph.filter.LongRangeFilter", 	]
In my schema I put the types DateRangeFilter, DateTimeRangeFilter and LongRangeFilter in other folder so that the generation not found this definitions. Is important to clarify that the next input contains the reference to this classes:
`input DocumentFilter {
userId: Long!
# Creation date.
date: DateTimeRangeFilter

# Start date of period billed
dateFrom: DateRangeFilter

`
As next step, I generate the code running generateJava, the code is generated in right way, but the method toString() used on serialize is:
`
public String toString() {
java.util.LinkedHashMap<String, Object> entries = new java.util.LinkedHashMap<String,Object>();
entries.put("userId", userId);
    entries.put("date", date == null ? null : "\"" + date + "\"");
    entries.put("dateFrom", dateFrom == null ? null : "\"" + dateFrom + "\"");
    entries.put("dateTo", dateTo == null ? null : "\"" + dateTo + "\"");

.....
`
As you can see the fields date, dateFrom and dateTo are serialized like a String but it should be an Object.
I have reviewed the code of generator and I believe to detect the error

the function isInputString check if the className is present in typeMapping:

For this reason the toString function has this form:
        entries.put("dateFrom", dateFrom == null ? null : "\"" + dateFrom + "\"");
but the correct way would be:
        entries.put("dateFrom", dateFrom);
because it is an object.
Someone can help me or say me how achieve that toString has this format?
Thanks!

@aestepa-meli  please check the latest version. Looks like this issue has been solved

Please reopen if this is still an issue. Closing it for now.


Make benchmark optional in daily_returns plot

make the benchmark optional

see my issue https://github.com/ranaroussi/quantstats/issues/276

Since this library no longer seems to be maintained, we have created a duplicate of it that we plan to maintain and improve over time. If you'd like to use our version check it out here: https://github.com/Lumiwealth/quanstats_lumi
Please feel free to fork and send your pull request to this new library, we will be actively monitoring the library and approving PRs


ci: cache yarn packages

It seems not to be a effect way to cache deps

https://stackoverflow.com/questions/61010294/how-to-cache-yarn-packages-in-github-actions

Besides, I don't prefer cache for build tests. It hides some problems which should be exposed when building. "Build" should be a "pure function".


Only use Rails subgems we actually need

This came out of a subgem of Rails (activestorage) using a dependency
that was yanked, blocking us from bundling our dependencies. It should
make our app leaner and reduce the surface area of these sorts of things
happening a tiny bit.
We still need to do something about govuk-date-fields because that is
still pulling in all of Rails, but this is a first step.

Split out individual gems in Gemfile
Update govuk-components to new version that doesn't depend on all of
Rails
Update mimemagic to new version

Are we even using gov-uk-date-fields at all?

Are we even using gov-uk-date-fields at all?
Sadly yes, although my thinking is we should get rid of them as the form builder supports date fields anyway 🤷♀

We just probably never got around to it?

Are we even using gov-uk-date-fields at all?
Sadly yes, although my thinking is we should get rid of them as the form builder supports date fields anyway 🤷♀

We just probably never got around to it?

Ah ok I can see it now. Yeah let's get rid ASAP

Review app deployed to https://teaching-vacancies-review-pr-3166.london.cloudapps.digital


feature: support for ActiveSupport::MemCacheStore

Hello,
This PR adds support for ActiveSupport::Cache::MemCacheStore, which is Rails' wrapper around Dalli.
Because of Dalli lack of support for race_condition_ttl, we're stuck with MemCacheStore... hence this PR!

Hmm, there's a compatibility issue with Rails 3.x's ActiveSupport::Cache::MemCacheStore...
Working a fix, I hope to ship it tomorrow.

Hmm, there's a compatibility issue with Rails 3.x's ActiveSupport::Cache::MemCacheStore...
Working a fix, I hope to ship it tomorrow.

Thanks for working on that.

I've investigated this a bit, and there's an inconsistency in Rails 3.x, with ActiveSupport::Cache::MemCacheStore.
When using the Rails 3.2 (ActiveSupport::Cache::MemCacheStore +  MemCache backend), the following returns 0:
Rails.cache.delete("foo")
Rails.cache.increment("foo") # => 0

Whereas with every other cache stores, such as Dalli, it returns nil.
I'm not sure it's a good idea to deal with that specificity in Rack::Attack, but since Rails 3.2 is end-of-life, I don't think I can fix it there either.
I could create a new MemCacheProxy class to that effect, but it's only going to be useful for Rails 3.x projects with the MemCache backend... any opinion or alternative idea :) ?

I've addressed the issue by creating a dedicate MemCacheProxy class for the ActiveSupport::Cache::MemCacheStore + MemCache backend case.
The original reason for this PR is still solved, and now everything should work well even for Rails 3.2!

+1

@elhu Very cool. Didn't realize dalli didn't support race_condition_ttl! So I agree it's valuable to support MemCacheStore.
I'll review this as soon as I can (probably in the next couple weeks.)
If you can report success running this patch in a production application, that'd be helpful testing.

I'm on vacation this week, but I can start testing things out on a production app starting the week after that.
I'll adjust the PR if I find any issues, and report here if I don't find any.

I've brought my branch up to date with master, and fixed a couple more things.
I am also running the patch on a live web-app, and have no problems to report so far.

@ktheory did you get a chance to review this?
Once the PR is OK for you, let me know, I'll squash my commits together to avoid clutter in the master tree :smiley:.

I had to revert my refactoring of Rack::Attack::StoreProxy.build, MRI 2.0.0 and JRuby do not behave like MRI 2.1.x and 2.2.x when calling Kernel.const_defined? with a nested constant name.

@ktheory Is there anything more you would like to discuss regarding this PR?

@elhu Looks good! :cake:
I'll aim to publish a new release by the end of the week.

Great, thanks!

Just shipped v4.4.0: https://rubygems.org/gems/rack-attack/versions/4.4.0

Brilliant, thank you very much!


[FEATURE] Add replica name to the backup status

Make it easier to find which replica is doing the backup.

Validation: PASSED

Create a pod using Longhorn volume
write some data to the volume
Create a snapshot
Create a backup, snapshot should be created, and backup progress should be visible in percentage
In Volume Details page, Snapshots section, hovering over snapshot will show replica name that is used to create the snapshot. (PASS)


Backer in Readme has invalid SSL certificate

The anime-dude has invalid cert which causes chrome to put out a scary message; Someone might be trying to steal your digits. This is also an advert btw.
Suggestion
Remove link until cert is fixed.
Let backer know certs point to something else.
https://pronama.co.jp/?utm_source=opencollective&utm_medium=github&utm_campaign=reactiveui

Fixed, thanks.


Add underscores to hide some imports

Closes #972.
The behavior with this change:
>>> import shiny.express as sx
>>> sx.[TAB]
sx.annotations        sx.display_body()     sx.is_express_app(    sx.suspend_display(   sx.wrap_express_app(
sx.app                sx.display_decorator  sx.output_args(       sx.ui

I still see shiny.express.annotations and shiny.express.app. I'm not sure if we can get rid of the former, but maybe the latter?

I agree about app, so I made the change and pushed.


"Float" holograms

Our development team is doing a medical project. When you insert your foot into the dummy and start to look at it from different sides, it creates the feeling that the model in the dummy "swims". Is there any way to solve this problem?

does it have a collider? see https://developer.microsoft.com/en-us/windows/mixed-reality/Hologram_stability.html

I believe you're in need of an anchor.

This could also be an effect caused by inter-pupillary distance configuration where it is not correct for a given specific user. You may also notice that this "swimming" effect may be different depending on the user. Check this repo: https://github.com/Microsoft/HoloLensCompanionKit/tree/master/KinectIPD for automatic configuration. I also think doing the "learn gesture" tutorial in HoloLens sets the IPD.

It could also be caused by a low fps, if your model/shader are resource intensive.

In my experience, @StephenHodgson is right.  Your shaders have to be optimized or use the Holotoolkit shaders and your poly counts should be low to avoid lag.

The exact same swimming is noticeable in the standard Holograms app, although Hololens issues 60 fps

Microsoft Mixed Reality Documentation for Spatial Anchors

Spatial anchors stabilize their coordinate system near the anchor's origin. If you render holograms more than about 3 meters from that origin, those holograms may experience noticeable positional errors in proportion to their distance from that origin, due to lever-arm effects. That works if the user stands near the anchor, since the hologram is far away from the user too, meaning the angular error of the distant hologram will be small. However, if the user walks up to that distant hologram, it will then be large in their view, making the lever-arm effects from the faraway anchor origin quite obvious.

Please reopen if your question was not answered.


[FEATURE] UI: Only allow to upgrade to the default engine image when the setting automatically upgrade volumes' engine to default version is enable

If the setting Concurrent Automatic Engine Upgrade Per Node Limit is bigger than 0, only allow the user to upgrade to the default engine image. Even if there are other engine images available, we don't allow users to upgrade to a different engine image other than the default one.
More specifically, if the setting Concurrent Automatic Engine Upgrade Per Node Limit > 0,  filter the dropdown selection for upgrade engine to only include the default engine image. If the volume already has the default engine image, the dropdown selection should be empty thus the upgrade engine action is disabled

Notice the backend hasn't been merged yet. PR at https://github.com/longhorn/longhorn-manager/pull/818

Notice the backend hasn't been merged yet. PR at https://github.com/longhorn/longhorn-manager/pull/818

The backend has been merged at https://github.com/longhorn/longhorn-manager/pull/818

The backend has been merged at https://github.com/longhorn/longhorn-manager/pull/818

@PhanLe1010 Can you confirm this UI change is ready to be validated?

@yasker Yes, It is ready to be validated.
Should I move it to merged column?

Verified with Longhorn-master 02/24/2021
Validation - Pass

With the auto engine upgrade disabled - It is allowed to upgrade the engine to a engine image which is not default.
With the auto engine upgrade Enabled - Upgrade to only default engine image is allowed using UI. The dropdown doesn't show engine image other than the default one.


Add hatsu to COMMUNITY.md

https://github.com/importantimport/hatsu

Thanks a very useful project! thank you


add parsers to compare and plot benchs

This patch aims to generate graphs from the benchmark results.
For this, we need to add:

dynamo/compare.py - Rewrite part of the Compare and Table classes (from torch.utils.benchmark.utils.compare) to be able to parse into a CSV format to be able to use the results in other tools, as Pandas.
dynamo/plot.py - functions to prepare the table, and perform the plot of the results.

For example, benchmarking the geometry.transform.rescale function ($ python runner.py --save-graphs )
Config file:
global:
  batch_sizes: [16]
  resolutions:
    [
      64,
      512
    ]
  threads:
    [
      1,
      8
    ]
  import_from: 'config'

geometry.transform.rescale:
  factor:
    [
      [0.5, 0.5],
      [1.5, 1.5]
    ]

Torch compare results
[-------------------------------------------- geometry.transform.rescale --------------------------------------------]
                             |  kornia_cpu  |  kornia_cuda  |  dynamo_kornia_cpu  |  dynamo_kornia_cuda  |  opencv_cpu
1 threads: -----------------------------------------------------------------------------------------------------------
      [16, 64, [0.5, 0.5]]   |      266.1   |      16.4     |          267.2      |         16.5         |     135.9
      [16, 512, [0.5, 0.5]]  |    27334.6   |     159.2     |        26912.4      |        159.1         |    6728.7
      [16, 64, [1.5, 1.5]]   |     1703.0   |      16.1     |         1698.0      |         16.1         |     451.5
      [16, 512, [1.5, 1.5]]  |   176466.8   |     523.9     |       176901.1      |        523.3         |   28915.0
8 threads: -----------------------------------------------------------------------------------------------------------
      [16, 64, [0.5, 0.5]]   |       89.3   |      16.5     |           93.9      |         16.4         |     135.9
      [16, 512, [0.5, 0.5]]  |     2878.4   |     159.3     |         2882.0      |        159.1         |    6736.4
      [16, 64, [1.5, 1.5]]   |      380.8   |      16.1     |          383.2      |         16.1         |     460.6
      [16, 512, [1.5, 1.5]]  |    28722.3   |     524.2     |        28646.6      |        523.4         |   28918.5

Times are in microseconds (us).

The dataframe before plot
optimizer                          opencv  eager_cpu  dynamo_cpu  eager_gpu  dynamo_gpu
label
8 threads\n[16, 64, [1.5, 1.5]]     460.6      380.8       383.2       16.1        16.1
8 threads\n[16, 64, [0.5, 0.5]]     135.9       89.3        93.9       16.5        16.4
8 threads\n[16, 512, [1.5, 1.5]]  28918.5    28722.3     28646.6      524.2       523.4
8 threads\n[16, 512, [0.5, 0.5]]   6736.4     2878.4      2882.0      159.3       159.1
1 threads\n[16, 64, [1.5, 1.5]]     451.5     1703.0      1698.0       16.1        16.1
1 threads\n[16, 64, [0.5, 0.5]]     135.9      266.1       267.2       16.4        16.5
1 threads\n[16, 512, [1.5, 1.5]]  28915.0   176466.8    176901.1      523.9       523.3
1 threads\n[16, 512, [0.5, 0.5]]   6728.7    27334.6     26912.4      159.2       159.1

Graphs
For the graph time, small bars are better.
For the comparison graphs, bigger bars are better. And bar smaller than 1, means that it is slower than the comparison item
Time graph

Ratio with OpenCV

Ratio with eager CPU

Ratio with eager GPU

TODO:

add an option on runner to save the graphs after run

@johnnv1 apart from functional do we support adding classes/modules ?

@johnnv1 apart from functional do we support adding classes/modules ?

If I understand correctly, I don't think so, unless we create a wrapper function for each case

i got the following error with the runner

This is from the plot case, probably the dynamo GPU cases has failed so we don't have any sample, so when trying to compute on the dataframe, this occurs


Alert Plugins of Special Case Exceptions Suppressing Deprecation Warnings

Whether or not we choose to move forward with any sort of "strict mode" to display these warnings regardless, we should alert the plugin authors of potential problems with the "special cases" in DefaultConfiguration that do not emit warnings for behavior that should be considered deprecated.
This ticket is just to remember to do that after the 8.2 RC 1 release.  It should be moved to 8.2 RC 2 or 8.3 RC 1 immediately after either of those milestones are created.

Follow up with KMP and AGP teams

It would be great to use #25810 for this

Both teams are aware


Bolting on batch mode

Hey there, claviska, as suggested, I'm working on some changes to submit to you. Next, I'll incorporate CLI arguments for transformation type(s) and arguments.
batch demo so far

👍

Hey @claviska
I've got for you, I think, a good starting point.
https://gist.github.com/paxperscientiam/a80d7b60361c786afeba
What do you think? It should probably take a second argument for "fit_to_height".

What would the second argument be for?

What would the second argument be for?

For specifying the height of the thumbnails.

Sorry, thought you were referring to a second argument in the fit_to_height() method. Gotcha :)

It's been awhile, and I think any work on batch image processing should probably remain at the discretion of the user. One reason I think we should refrain from incorporating this in the actual library: users may not be using a local file system (could be FTP, SFTP, Amazon S3, Dropbox connector, etc.).
Closing because it's ultimately out of scope for the library itself.


Trim non-prod firewall instances to save cost

Describe the Issue
There is a substantial amount of non-production instances that can be cleaned up and deleted. Try to get closer to the baseline cost
Acceptance Criteria

[ ] Review instances in the non-production environments
[ ] Identify instances that can be removed or decreased
[ ] Trim unnecessary spend and measure $ reduction if possible

Warren: Probably taken by Prabhu or Warren

https://github.com/BC-Gov-PaaS-Platform-Services/aws-ecf-cloudwatch-alarms/pull/11

Reduced firewall gateways and rsyslog clusters to 1 instance is both forge and assembly