Changed GPU, Case, and Motherboard sections

Native Spanish speaker. I have made some changes to help with readability as well as changed word use and some grammar, the content is intact conceptually.

[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

LGTM!


Column mode selection behavior is inconsistent with user expectations

VSCode Version: 1.5.1
OS Version: Ubuntu 14.04, Windows 7/10, probably all others

Most programs with column mode (visual studio/notepad++/kdevelop/etc) have the following user experience:

hold the keyboard shortcut/toggle column mode.
click somewhere. This is the selection 'start' anchor.
drag somewhere. This is the selection 'end' anchor.

However, in visual studio code, the process is:

click somewhere. This is the selection 'start' anchor.
hold the keyboard shortcut/toggle column mode
drag somewhere. This is the selection 'end' anchor.

IMHO this is unintuitive for two reasons:

It goes against all the training we have from other dev tools we've been using much longer.
Usually, I don't think about where I left my cursor when I decide to do an operation. I'm thinking about what I want to do. I find that whenever I use code's column mode, I'm always unpleasantly surprised to find that the cursor is not where I expect it and isn't updated to my click.

Additionally, while it's technically the same number of steps, method 1 is much more fluid as the holding/clicking/dragging can be done in a smooth motion, rather than having to position the cursor as a distinct and thoughtful first step.
Could you please change the cursor selection to method 2?

Duplicate of #5402.

@nholthaus Don't forget to up vote the original issue
@glen-84 Thanks for finding the original issue


Better :messages log

Here's a proof of concept for Atom's notifications that get rid of the spammy user experience of Atom's notifications. It would be cool to render Vim's :messages log in this manner:
https://github.com/atom/notifications/issues/29

Cool design, I hadn't seen it before, I like the idea a lot! It's nice to be able to opt-in to getting barraged with all the details. Would make a lot of sense to integrate in terms of messages, errors, etc.
I believe this RFC will make it easier to get at the messages:
https://github.com/neovim/neovim/pull/5686

This PR https://github.com/neovim/neovim/pull/7466 looks like it could help us here too-


App closes unexpectedly

OS: Win 10
Scout-App Version: 2.18.16
I've been using this program for the past couple of days but it's closing out of nowhere when I have it running in the background, otherwise, it seems to work well. Please look into the issue asap, as it's incredibly annoying when you're troubleshooting code and realize errors aren't being fixed because one of your apps, which should be stable, is no longer running.

@spercy16

I cannot reproduce this, which means it may be something project, code, or system specific. Can you narrow down what could be causing it?

See: https://github.com/scout-app/scout-app/issues/286

Can you try on the latest code from master. There have been several bug fixes that are not yet released that may resolve the issue. If so, report back so I know this issue is already fixed in the codebase.

See: Running Latest guide

Had been having this issue for a while, now. Different machines, different installations, they all did this for me for some reason. The app would:

always succeed the first time, right after clicking the "run" button.
fail around 60% of the times, even when saving the same, valid file over and over again.
play error sounds, but wouldn't show them in the gui.
crash with about a 10% chance upon saving, never hearing any sound beforehand.

Applying this commit made all of these issues disappear!


Catch exception error but without http error code

Document say that when http response error would return statusCode (404, 304, 500,...)
i.e,
fetch('GET', url, headers)
      .then((response) => {  // when response status code is 200
         ...
      })
      .catch((errorMessage, statusCode) => {   // Status code is not 200
                            ^^^^^^^^^^^^
          ...
      }

But I always get undefined for variable:  "statusCode".
Additional Information:
react-native:  0.29.2
react-native-fetch-blob: 0.9.4
iOS: 8.X & 9.X

@leotu , thank you for the information, I'll try to add the missing part and update the status in this thread : )

@leotu , after some investigation I think this is a design error on IOS, a response with status code other than 2xx should not cause a promise rejection. Therefore I've changed the implementation and published a new version 0.9.5-beta.3. Please try install this version and you should be able to handle the request status code directly in then block instead of catch block.
RNFetchBlob.fetch('GET', 'http://www.example.com')
.then((res) => {
    let statusCode = Math.floor(res.info().status)
})

Thank you !

Ok, I will try it.
Catch http status code is very important that I need to control my cache design.
i.e, use Etag & If-None-Match communicate with "Images Web Server".
for example:  return 404 for using default image, 304 for using local cache image.

@leotu , thanks for your information 👍 Specifying path in config, that will force the response body write to the location regardless the original one, when the response is 304 the response body is empty so the original file becomes an empty file.
Perhaps you can try use the following workaround with 0.9.5-beta.3

import RNFB from 'react-native-fetch-blob'
const {fetch, fs} = RNFB

RNFB.config({
    fileCache : true
})
.fetch('GET', url, headers)
.then((res) => {
    let status = Math.floor(res.info().status)
    switch(status) {
        // replace the existing file with the new one
        case 200 :
        fs.mv(res.path(), existPathFile).then(() => {  callback(res.path()) })
        break
        // resource not modified
        case 304 :
        callback(res.path())
        break
        // hanlding HTTP request error
        default :
        callback(defaultImageSrc)
        break
    }
})
// handle network and socket errors
.catch((err) => { ... })

It works.
"0.9.5-beta.3"
Thanks.

res.info().status is useful but not documented. Could you add it to the README.md as a comment? thanks!


"dep-bin-cargo-miri-test" is only created on the second cargo miri run

To reproduce, run rm target -rf && cargo miri run -- "" && cargo miri run -- "" in the test-cargo-miri directory:
   Compiling byteorder v1.3.4
   Compiling cargo-miri-test v0.1.0 (/home/r/src/rust/miri/test-cargo-miri)
   Compiling issue_1691 v0.1.0 (/home/r/src/rust/miri/test-cargo-miri/issue-1691)
   Compiling cdylib v0.1.0 (/home/r/src/rust/miri/test-cargo-miri/cdylib)
   Compiling issue_1705 v0.1.0 (/home/r/src/rust/miri/test-cargo-miri/issue-1705)
   Compiling issue_1567 v0.1.0 (/home/r/src/rust/miri/test-cargo-miri/issue-1567)
    Finished dev [unoptimized + debuginfo] target(s) in 0.78s
     Running `/home/r/.cargo/bin/cargo-miri target/x86_64-unknown-linux-gnu/debug/cargo-miri-test ''`
0x01020304
main

   Compiling cargo-miri-test v0.1.0 (/home/r/src/rust/miri/test-cargo-miri)
    Finished dev [unoptimized + debuginfo] target(s) in 0.01s
     Running `/home/r/.cargo/bin/cargo-miri target/x86_64-unknown-linux-gnu/debug/cargo-miri-test ''`
0x01020304
main

Notice how in the second run, it is still doing Compiling cargo-miri-test. If we do a third run, it does not do this any more.
Running with CARGO_LOG=cargo::core::compiler::fingerprint=trace shows that cargo is missing a file called dep-bin-cargo-miri-test, which (as of right now on my platform) it expects in target/x86_64-unknown-linux-gnu/debug/.fingerprint/cargo-miri-test-44bcc890eaab31dc. After the first run, that folder contains
bin-cargo-miri-test  bin-cargo-miri-test.json  invoked.timestamp

Only after the second run does it also contain the expected dep-bin-cargo-miri-test.
This is not a terribly big deal since the Compiling step for a binary that Miri will run does literally nothing, but it's still strange noise in the log, and it would be good to figure out what is going on.
@ehuss could you give some guidance, in particular about who is supposed to create that file and why it might only happen on the second attempt here?

Ah, I think I understand what is happening. I found a comment in cargo:
//! After `rustc` exits successfully, Cargo will read the dep info file and
//! translate it into a binary format that is stored in the fingerprint
//! directory (`translate_dep_info`). The mtime of the fingerprint dep-info
//! file itself is used as the reference for comparing the source files to
//! determine if any of the source files have been modified (see below for
//! more detail). Note that Cargo parses the special `# env-var:...` comments in
//! dep-info files to learn about environment variables that the rustc compile
//! depends on. Cargo then later uses this to trigger a recompile if a
//! referenced env var changes (even if the source didn't change).

What happens is that the .d file has not actually been created yet when cargo "built" the binary; it will only be created when the binary is actually run. On the second cargo miri run, the .d file exists from the previous run and now cargo finds it and translates it to its binary format.
I see two paths forward:

Do nothing. The rebuild is harmless in the sense that no work is repeated.
When cargo "builds" the binary, run rustc with --emit=dep-info. That would create the required .d file. I am not sure how using just dep-info is supported and how expensive it is.

I am not sure if emitting just dep-info is supported and how expensive it is.

It's supported:
$ $(rustup which miri) --emit=dep-info -o /dev/stdout - <<<'macro_rules! empty { () => (include_bytes!("/dev/null")) } const _: &[u8] = empty!();'
/dev/stdout: /dev/null

/dev/null:

But unfortunately looks like it has to expand macros, which could be slow.

I'm wondering if creating a stub .d file just like .rlibs and .sos works, Cargo will run the final "binary" anyway.

Hm... a step that doesn't do anything arguably has no dependencies, so yeah that could work.^^


Fix internal server error and false positive when creating a server

Fixes: #6779
Fixes: #6781
With this fix there is no internal server error when creating a server without a profile key in v2 and v3
The other fix is when creating a server with multiple profiles in v4, if any of the profiles do not belong to the same CDN it will now fail as expected.

Which Traffic Control components are affected by this PR?

Traffic Ops

What is the best way to verify this PR?

Build and test the following in TO:
POST request to /api/2.0/servers with no profile key in the request body
{
    "cachegroupId": 8,
    "cdnId": 2,
    "domainName": "ga.atlanta.kabletown.net",
    "hostName": "atlanta-edge-01",
    "interfaceMtu": 1500,
    "ipAddress": "<IP_ADDRESS>",
    "ipGateway": "<IP_ADDRESS>",
    "ipNetmask": "<IP_ADDRESS>",
    "ipIsService": true,
    "interfaceName": "eth0",
    "physLocationId": 2,
    "statusId": 3,
    "typeId": 11,
    "profileId": 13
}

Should succeed with no error:
HTTP/1.1 200 OK
{
    "alerts": [
        {
            "text": "Server created",
            "level": "success"
        }
    ],
    "response": {...}
}

POST request to /api/3.0/servers with no profile key in the request body
{
    "cachegroupId": 8,
    "cdnId": 2,
    "domainName": "ga.atlanta.kabletown.net",
    "hostName": "atlanta-edge-01",
    "interfaces": [
        {
            "ipAddresses": [
                {
                    "address": "<IP_ADDRESS>",
                    "gateway": "<IP_ADDRESS>",
                    "serviceAddress": true
                }
            ],
            "name": "eth0"
        }
    ],
    "physLocationId": 2,
    "statusId": 3,
    "typeId": 11,
    "profileId": 13
}

Should succeed with no error:
HTTP/1.1 201 Created
{
    "alerts": [
        {
            "text": "Server created",
            "level": "success"
        }
    ],
    "response": {...}
}

POST request to /api/4.0/servers with multiple profiles, where one profile belongs to a different cdn than the server (regardless of order)
{
    "cachegroupId": 8,
    "cdnId": 2,
    "domainName": "test.net",
    "hostName": "test-01",
    "interfaces": [
        {
            "ipAddresses": [
                {
                    "address": "<IP_ADDRESS>",
                    "gateway": "<IP_ADDRESS>",
                    "serviceAddress": true
                }
            ],
            "name": "eth0"
        }
    ],
    "physLocationId": 2,
    "statusId": 3,
    "typeId": 11,
    "profileNames": ["ORG_MSO", "TRAFFIC_STATS", "CCR_CIAB"]
}

Should return an error:
HTTP/1.1 400 Bad Request
{
    "alerts": [
        {
            "text": "CDN id '1' for profile 'TRAFFIC_STATS' does not match Server CDN '2'",
            "level": "error"
        }
    ]
}

PR submission checklist

[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

can you make a better title for the PR @ericholguin to describe what this PR does? thanks

LGTM.


⚠ Processors has degraded performance

In 4840d49, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 124 ms

Resolved: Processors performance has improved in 1feec76.


[DOCFIX] Update to correct short version

@aaudiber This is a fix to display the correct version for the docs on alluxio.org.

@calvinjia

LGTM, thanks.

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/11527/
Test PASSed.

Merged build finished. Test PASSed.


feat(barcode-scanning): add web support

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] The changes have been tested successfully.
[x] A changeset has been created (npm run changeset).
[x] I have read and followed the pull request guidelines.

Try it
For my and you convenience here is build: capacitor-mlkit-barcode-scanning-6.2.0.tgz
You can install it in your project like this:
npm i @capacitor-mlkit/barcode-scanning@https://github.com/user-attachments/files/17894704/capacitor-mlkit-barcode-scanning-6.2.0.tgz

Hi @iBobik, we are currently not planning to add web support to these plugins. :/ This is because these are plugins for the ML Kit SDK and we will only add web support when the ML Kit SDK officially supports the web. Feel free to upvote this suggestion. However, we are already working on another solution. More updates coming soon! Thanks for your contribution though! I will close this PR for now.

Hmm, better to hear it on that another unmaintained PR (which is there for months) before I spend a days on it debugging it with the current version.
However I need it working now, it works in my project, so I will maintain this branch until I find something else what will implement all 3 platforms in one plugin.


Support podcast:chapters (phase 1)

The Podcasting 2.0 XSD defines the podcast:chapters element.

Add two meta tags - episode_chapter_file and episode_chapter_type.
Add a "default chapter type" to the podcast feed options, and use it when episode_chapter_type is missing.

Fixed via #9


[iOS] Support for Clipping and Shadow

BackgroundKit should support clipping and shadows as on Android:

Sneak peak from clipping-elevation branch:

Sneak peak from clipping-elevation branch:
Android supports clipping for different corner radius on each corner with elevation

By a900cab0e90ef807f78074e62d45c08ae30fe637 iOS supports clipping for different corner radius on each corner with elevation

Fixed by a900cab0e90ef807f78074e62d45c08ae30fe637


Volume sharing not working ..

I followed this commands
On Swarm Manager Machine

docker run --restart=unless-stopped -d  -v /run/docker/plugins/:/run/docker/plugins/ -v /shareddata:/shareddata cwspear/docker-local-persist-volume-plugin

docker volume create -d local-persist -o mountpoint=/shareddata --name=test2

docker run -itd -v test2:/data/ ubuntu

docker exec -it 1df1ae82a71abd08077c451d470d2401ea5275a14cf7e6508ba02817f233f72e bin/bash

Inside Container

ls

data
srv
...
..
So now it showing inside the container .

cd data

touch text.txt

Ctrl+p+q

On Swarm Manager Machine

ls /shareddata

This gives empty list . basically it not working as you mentioned. please give the example , How to do it properly. The concept it very nice.

It definitely does work. I don't know if it has something to do with the way you have Swarm set up?

Closing due to inactivity... Still not sure why people are trying use this plugin with Swarm. Feel free to comment if you think there's more to this.


ENH: stats: add skewed Cauchy distribution

Reference issue
Identical to gh-7702 (git diff d1894489c06fc530a8f7e2d4272f3ce3cde12c4d) aside from commit history. Creating a new PR to avoid force-pushing to original author's repo after git surgery.
Closes gh-7437
Closes gh-7702
What does this implement/fix?
Adds the skewed Cauchy distribution.

LGTM.
Arpack failure is inrelated, merging. Thanks Matt, @adivijaykumar

LGTM.
Arpack failure is inrelated, merging. Thanks Matt, @adivijaykumar

Thanks all.  I added a note about this to the release notes.

Thanks all.  I added a note about this to the release notes.

Thanks @mdhaber for seeing this through, and apologies for not looking at this after I opened the PR ages ago :smile:

Thanks @mdhaber for seeing this through, and apologies for not looking at this after I opened the PR ages ago :smile:

No problem. Just curious - how did you get interested in the distribution?

No problem. Just curious - how did you get interested in the distribution?

If I remember correctly, I thought I needed the distribution for one of my university projects. Turned out that I really didn't, but thought it would be a good idea to start a PR since there was an open issue to add this distribution.

If I remember correctly, I thought I needed the distribution for one of my university projects. Turned out that I really didn't, but thought it would be a good idea to start a PR since there was an open issue to add this distribution.

Thanks for reviewing @ev-br!

Thanks for reviewing @ev-br!


Update dd-trace-api, dd-trace-ot to 0.98.0

Updates

com.datadoghq:dd-trace-api
com.datadoghq:dd-trace-ot

from 0.91.0 to 0.98.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.datadoghq" } ]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

Superseded by #543.


perf: slotted classes

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[x] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Closes #500

Checklist

[x] Code follows style guide (ruff)
[x] Tests added
[x] Documented the changes

If you have comments or can explain your changes, please do so below.
Opening as draft as I am trying to figure out drawbacks of this (if any)

cool - tests pass, there don't seem to be downsides...shall we ship it?

Closing this for now as it is fairly outdated. Will come back in the future if we consider this a good investment


how to integrate airflow dags with azure files?

I would like to know how do we integrate airflow with azure files ?
I tried to follow what's suggested here to create a PV -- https://docs.microsoft.com/en-us/azure/aks/azure-files-dynamic-pv and then changed the dags-data pvc to point at this but the pods were not loading ?
How do we work this out with azure files ? do you have an approach please advice thanks.
PS: the issue we have is azure storage class doesn't support ReadOnlyMany/ReadWriteMany access modes.
@thesuperzapper -- please suggest

I am facing this error

MountVolume.SetUp failed for volume "pvc-a85214b0-3264-46a7-9d48-b2b9dcac4931" : mount failed: exit status 32 Mounting command: systemd-run Mounting arguments: --description=Kubernetes transient mount for /var/lib/kubelet/pods/5231e52c-b642-4042-b808-5d954cdd17b0/volumes/kubernetes.io~azure-file/pvc-a85214b0-3264-46a7-9d48-b2b9dcac4931 --scope -- mount -t cifs -o actimeo=30,mfsymlinks,file_mode=0777,dir_mode=0777,vers=3.0,<masked> //fc0c947e33845492e8b8a07.file.core.windows.net/kubernetes-dynamic-pvc-a85214b0-3264-46a7-9d48-b2b9dcac4931 /var/lib/kubelet/pods/5231e52c-b642-4042-b808-5d954cdd17b0/volumes/kubernetes.io~azure-file/pvc-a85214b0-3264-46a7-9d48-b2b9dcac4931 Output: Running scope as unit: run-rd32e7216e4f84b5fae239474993c49e1.scope mount error(2): No such file or directory Refer to the mount.cifs(8) manual page (e.g. man mount.cifs)

@gowdra01 I don't understand the issue you are having, please follow "Option 2a" or "Option 2b" under the "How to store DAGs?" docs.

sorry I can t use those ReadOnlyMany or ReadWriteMany.. its not supported so I have to use azurefile storage class. but unfortunately I am getting the above error. although the pvc is successfully created but the scheduler are throwing the errors

@gowdra01 I can see here that AzureFile supports ReadOnlyMany and ReadWriteMany.
Make sure your PersistentVolumeClaim specifies accessModes to include ReadOnlyMany.


RENAME INDEX doc

Documents RENAME INDEX.

This change is

CircleCI passes for this locally. The broken links in the report seem like a hiccup in availability.


Page references for citations should be auto-formatted according to CSL

Description
Background:
Currently both cite() and the shortcut syntax (@something) take a supplement that can be used for anything. The supplement will commonly be used for page references, however, at the moment the syntax of that reference is left to the user to implement.
Different reference styles have different syntax for page numbering, i.e.:

pp. 1
p. 1
1

Request:
Overall, this is a request for this syntax to be handled by Typst and done according to the CSL style specified (which, I believe, has the syntax specified).
This will remove the responsibility for the user to hard-code page reference syntax.
An aside: I wonder if the @ citation can take an integer or range to make one of the most common uses very simple. I.e. @something[1] and @something[1-10].

See also:

#1139

Use Case
Less friction for an extremely common component of writing.
Less errors by users in their writing.

Keep in mind that next to pages, also sections could be enumerated. Biblatex can detect this already.


Add filter files extension

Feel free to dismiss if you don't find it useful, but I'm using it a lot at my day to day job :)

Looks interesting, but this is not the place to have every GitHub related extension. I would instead recommend you submit it to https://github.com/stefanbuck/awesome-browser-extensions-for-github ;)

No problem ! looks like its already there :)


Marathon collector

Output from marathon is in the format:
{
  "version": "3.0.0",
  "gauges": {
    "api.mesosphere.marathon.core.event.impl.stream.HttpEventStreamActorMetrics.number-of-streams": {
      "value": 0
    },
    ...
  },
  "counters": {
    "GarbageCollector.totalGcs": {
      "count": 11652
    },
    ...
  },
  [other stuff]
}

I addressed most of the comments, however the for loops with range and the if statements with := and variables used outside the if block probably cannot be simplified

Line 160 prefixes all metrics with marathon. when building the metric.Metric


Android BookSearch App Demo

Just add a link to the project, you don't have to add the entire codebase.


Numbering reset in markup. No way to enforce my own numbering.

Declarative markup in composer has  - 3. …, it renders in the bot as 1. ....  It shows up as \3.  I want it to show as 3. …
Workaround - drop the '.'
 You can set textFormat to plain to disable Markdown rendering.


Improve ut class name

Description

Move class RestTemplateProxyCustomizerConfiguration from main source folder into test source folder
Rename test class name from XXXTest to XXXTests

All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
spring-cloud-azure-autoconfigure

Closing it, changed in this PR https://github.com/Azure/azure-sdk-for-java/pull/32612


remove unused kms key

The idea behind adding this KMS key was to use it to encrypt
secrets before kicking off automation.  It seems like we never
actually used it.

This was reverted in commit 7a1a688 due to deployment error.
INFO: done
DEBG: putting object to S3:
{
  "Bucket": "organization-formation-***",
  "Key": "state.json"
} (*** = MasterAccount)
ERROR: unable to put object to s3 (bucket: organization-formation-***, key: state.json) (*** = MasterAccount)
The provided token has expired.
ExpiredToken: The provided token has expired.
    at Request.extractError (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/services/s3.js:718:35)
    at Request.callListeners (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:38:9)
ERROR: Task ItKmsKey execute failed. reason: The provided token has expired.
The provided token has expired.
ExpiredToken: The provided token has expired.
    at Request.extractError (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/services/s3.js:718:35)
    at Request.callListeners (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:106:20)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:38:9)
    at Request.<anonymous> (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:690:12)
    at Request.callListeners (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:116:18)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/sequential_executor.js:78:10)
    at Request.emit (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:688:14)
    at Request.transition (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:22:10)
    at AcceptorStateMachine.runTo (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:14:12)
    at /home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/state_machine.js:26:10
    at Request.<anonymous> (/home/runner/work/organizations-infra/organizations-infra/node_modules/aws-sdk/lib/request.js:38:9)
ERROR:
ERROR: ==========================
ERROR: Stopped performing task(s)
ERROR: Following tasks failed:
ERROR:  - Task ItKmsKey
ERROR: ==========================
ERROR:
ERROR: Number of failed tasks 1 exceeded tolerance for failed tasks 0.
Error: Process completed with exit code 1.


[4.0] Serialization: Handle possibly-null serialized xrefs for optional protocol witnesses.

Explanation: The deserialization code for protocol conformances failed to account for null witness xrefs deserialized to represent an optional protocol requirement with no witness in the conformance.
Scope: Any code using optional protocol requirements crashes when deserialized.
Issue: SR-4850, rdar://problem/32134722.
Risk: Low, bug fix.
Testing: Swift CI, compatibility suite, tests from radar and Jira

@jrose-apple Mind reviewing for Swift 4.0?

@swift-ci Please test

@swift-ci Please test source compatibility

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - f25bf0430f29414144f043688385c46d77d7fcbe
Test requested by - @jckarter

@swift-ci Please test

@swift-ci Please test

@swift-ci Please test source compatibility

@swift-ci Please test source compatibility


Are celpy Environments thread safe?

Hello!
I was wondering if a celpy Environment instance could be shared amongst threads, or if we should create a new instance when we want to create a runner for a new expression.
I tried to look through the docs and source code but didn't find any reference to this.
One thing I've noticed though is that, when we call the program method, the code is doing an intermediate assignment to an instance variable of the environment, before returning, which I believe can be problematic in terms of thread preemption.
        self.runnable = runner_class(self, expr, functions)
        return self.runnable

Nonetheless I might have missed something and would like to get your feedback on this use case.
Thank you very much!

There was no design intention to be thread-safe. There are no internal threads.
An example of using threading.local() to manage storage for an Environment instance would be helpful.


Adapt update to other kinds of workload object

The path to find image references is hard-wired to .spec.template.spec.{containers,initContainers}[*].image (in effect; YAML path is not actually used). At least CronJobs have a different structure, so will fail to be updated.
The simplest thing to accomodate CronJob would be to dispatch on the Kind, using a different path-traversing func.
To update HelmRelease will need a deeper change; I'll have to look at the code in flux and see if I can adapt that to kyaml.

CronJobs now supported via #21.

Closing this, as the setters implementation (which can be used with any workload, or indeed any YAML that has apiVersion and kind fields) is now the only implementation used. The simple update mechanism was removed in #27.


Last AxisValue in chart is cropped

https://goo.gl/photos/89bWTSMsHczwu7x1A
I've set AxisValues by
            for (int i = 0; i < 6; i++)             {                 axisValues.Add(new AxisValue(i).SetLabel($"{DateTime.Today.AddDays(-6+i).ToString("d MMM")}"));             }             axisValues.Add(new AxisValue(6).SetLabel(Activity.GetString(Resource.String.today)));

I was having this issue as well and the only solution that I found was to make the labels tilted. I think the method is axisName.setLabelsTilted(true)

Thanks, I think it's good idea
But I chose another variant - I skip last element in hotisontal labels


[ICE: 20] High relayer fees estimation - UI malfunction?

Describe the bug
A user sent 0.005 ETH from Linea to Ethereum Mainnet and paid 0.069938ETH as relayer fee:
https://connextscan.io/tx/0xc5152f6af5ac9b8d778b381310ffbd406c507241f457e3b653e83deebc7c7325

but when the txn arrives it costs 0.02413873 ETH
https://etherscan.io/tx/0x3c2c562b11c8bd36eed9658969b86ab5aeae50e44a71774c24034e34cd96c176

Relayer fees were overestimated?.
Also, there is another issue, in cases like this (where Fees are greater than estimate received) a pop-up should like this should appear:

But seems that the user was able to bridge, so i guess that something didn't work as intended in our UI.
Ticket:
https://discord.com/channels/454734546869551114/1234846071642132480
@oncall

@just-a-node need to check it.

I'm seeing a relayer fee of .0699 ETH coming back from sdk.sdkUtils.getTransfers({ transactionHash: '0xc5152f6af5ac9b8d778b381310ffbd406c507241f457e3b653e83deebc7c7325' }), so I think the UI is displaying the data correctly.
Is the SDK call incorrect perhaps?

Any update? @regexpressyourself
We should definitely block transactions where the relayer fee exceeds the sent amount in value, and also display the message like @MinistroDolar said.

@0xanedi Did a bit of digging with Eddie on Friday, not sure what exactly happened in this case.
It looks like the SDK is returning the correct value. It also looks like the UI blocks users from bridging when fees are greater than the value.
I have been unable to reproduce a situation where we allow the user to bridge when fees exceed the amount.

Are we seeing any more examples of users being able to bridge an amount less than the fees incurred?

Not 100% sure that it is the same: https://app.intercom.com/a/inbox/qge2vd8z/inbox/conversation/675?view=List

connext_2024_05_05_675.txt

Based on our attempts at troubleshooting, I would advise keeping this open for the next ~3 days to monitor for additional cases. It's unclear if the case above is the same. If nothing rolls in, let's close.

Based on our attempts at troubleshooting, I would advise keeping this open for the next ~3 days to monitor for additional cases. It's unclear if the case above is the same. If nothing rolls in, let's close.

What resolution should we take regarding the user? Something went wrong in her case. She is quite upset and has been insisting on a response from the team for several days. https://discord.com/channels/454734546869551114/1234846071642132480

Refund processing? @ekbainova to determine this

Did we fix it with refactoring updates? @just-a-node @jwtong


XSS vulnerability stored in the publish blog module of Jfinal_cms V5.1.0

There is a stored XSS vulnerability in JFinal_cms 's publish blog module. An attacker can insert malicious XSS code into the keyword field. When the user views the content of the article in the foreground, the malicious XSS code is triggered successfully.
payload: " onmouseover="alert(document.cookie)

Successfully executed malicious XSS code:

Fix for this issue #39 is available in pull request #44https://github-com.translate.goog/jflyfox/jfinal_cms/pull/44


cant overwrite saved presets

Describe the bug
When trying to overwrite a existing prewrite by saving a new gear set with the same name, nothing happens when pressing the "Save Gear Set" button.
To Reproduce

Save any gear set
Alter the gear of the set
Make sure the name under "Gear Set Name" is the same and press "Save Gear Set"
See error: Old gear set doesn't get overwriten

Expected behavior
The old gear set should be overwritten, making the giving the gear set color on the gear set chooser to the right (as the gear equipped in the sim corresponds to the gear in the gear set).
Sim Links and Screenshots
https://gyazo.com/a46367529902ccf7b1a0e8cdcd823016 Have gear set "Babbelush" equiped
https://gyazo.com/777e5f8f6e28a452ace0a1b1e0557206 Switched neck from "Charge of the Demon Lord" to "Charge of the Eredar", button on the right for the gear set "Babbelush" stops being green as it is no longer the same as the gear selected.
At this point, clicking "Save Gear Set" does nothing.
Desktop (please complete the following information):

OS: Windows 10, downloaded "wowsimwotlk-windows.exe.zip" from "https://github.com/wowsims/wotlk/releases"

Also getting this issue. The console error is:
Uncaught DOMException: Failed to execute 'replaceChild' on 'Node': The node to be replaced is not a child of this node.     at nl.addSavedData (http://localhost:3333/wotlk/bundle/individual_sim_ui-53b011db.chunk.js:1:351859)     at HTMLButtonElement.<anonymous> (http://localhost:3333/wotlk/bundle/individual_sim_ui-53b011db.chunk.js:1:354649)

should be fixed in newest release


Move Custom to Input

See https://github.com/PistonDevelopers/piston/issues/1322
Currently, it is not possible for window backends to push custom events, since they are expected to return Input and not Event. This was a design error that appeared under a previous refactoring.
By moving Custom to Input, the problem should be solved.

A following up question, is there any plan for supporting raising events at application level? Which means a non window backend specific mechanism for event pushing. Since it can be a common situation that the event is not input/hardware related and may vary over application code. The current solution for event handling is more like a input-driven style, it does well on applications that updates rely on inputs, but not really much on others.

Closed in favor of https://github.com/PistonDevelopers/piston/issues/1324
@PoYen000227 please open up an separate issue if you want to discuss raising events at application level.


roundstart soup sensors to cabbage

About the PR

makes jumpsuits start with suit sensors set to coordinates
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

tweak: Suit sensors now start set to coordinates mode.

these are intentionally randomized so that turning them off doesn't instantly make you valid or suspicious.

sovl destroyed

this is intentional and has been part of game design for long time


How to turn the switch on/off indivudually

Hi!
This is actually not an issue. I would like to ask how to actually turn the switch on/off individually with your example.

Sure not a problem.  In the example I made, the switch works as followed:  I have a member variable switchValue that I declared in the constructor, along with a bind function that is tied to a function that will change the state as it is toggled:
  constructor(){     super();     this.onValueChange = this.onValueChange.bind(this);     this.state = {switchValue: false, loggedIn: false};   }
Next, the declaration of the switch in SettingsList.Item, all I need to do is pass a hasSwitch={true} to tell the class that this item will be a switch-type item, along with all the properties attached to it.
            <SettingsList.Item               icon={                   <Image style={styles.imageStyle} source={require('./images/airplane.png')}/>               }               hasSwitch={true}               switchState={this.state.switchValue}               switchOnValueChange={this.onValueChange}               hasNavArrow={false}               title='Airplane Mode'               titleStyle={{fontSize:16}}             />
The property 'switchState' is tied to my current state of switchValue, and the property 'switchOnValueChange' is your callback, in which i tied to the function we created a bind for earlier.  The example is very simplistic, and only changed the state (which is what's tied to the UI's switch state ON/OFF), but you can extend upon that function to do whatever it is that your switch wants to do in that callback function.
I hope that helps!


Add params to URL in IFrame chart type

Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if any
[x] I have reproduced the issue with at least the latest released version of superset
[x] I have checked the issue tracker for the same issue and I haven't found one similar

Superset version
0.24.0
Expected results
Need to append the URL with a parameter list, so that URL is dynamically generated on the basis of the filter
Actual results
Not able to add parameters to the URL string.
the json for the iframe chart shows "{"url_params": {},
But how do we access this from the Superset front end?
Steps to reproduce
I have a table with a list of domain names, I would want to open the URL as mentioned in the table

Is there any solution for this?

bump

bump

@ranggarppb I second your bump


electron app

Hello,
Can you please add a scaffolding or a way to integrate the project with electron (maybe using  the already existing electron-vuetify ?

It is a very interesting feature. Feel free to fork and give a go. I have no time to try that now, but I will definitely look into it later. If you have sth done, please feel free to create a PR.


Figure out how to make this thing redistributable

Initially we'll plan on everything being configurable via environment variables


Install borg is broken (again)

--------------------------------------------------------------------------------
Installing Domino Borg Backup integration
--------------------------------------------------------------------------------

/tmp/install/install_domino.sh: line 718: /tmp/install/domino-startscript/install_borg: No such file or directory

install_borg was part of the start-script. Cause this part has moved, the install_borg is missing.
As workaround I copied install_borg from start-script to $INSTALL_DIR/domino-startscript

Thanks for the hint!
I had to rework the start script integration, because the repository is now also used for the HCL official image.
So I just included the needed components into the project.
I missed the Borg installer.

just added the file again. but it looks like more is missing! the whole borg directory isn't included.
those files are needed for the integration!
can you explain, why it is working for you by just copying the install_borg file?

No, it's not working, i thought it.
Borg is installed, but the backup scripts (from extra/backup/borg) are missing.

updated the development branch with a fix. all files are back. and I moved the place where it is hooked in to work with the external start script installer plug-in (don't worry about this part. this is more for special cases).
all the files are back!  next time please do an end to end test, before proposing a solution for an issue ;-)
let me know if this works!

Hi Daniel,
Unfortunately I have to reopen the issue.
In the develop branch, the build works for domino but not for traveler.
--------------------------------------------------------------------------------
Running Traveler silent install
--------------------------------------------------------------------------------

cp: cannot stat '/local/notesdata/IBM_TECHNICAL_SUPPORT/traveler/logs/TravelerInstall.log': No such file or directory

And this regardless of whether the -borg option was used or not.
./build.sh domino 12.0.1 FP1
./build.sh -from=hclcom/domino:12.0.1FP1 traveler 12.0.1FP2

The traveler build works in the main branch. But, as is known, not with the -borg option.

Ooops!
But this has nothing to do with the Borg option. Can you create a separate issue for each problem in future to keep track, please.
There was a change in the way the notes.ini is packed into the install data tar file.
The notes.ini is now renamed to notes.ini.instal before creating the tar to ensure we can unpack the tar completely in one go on updates without the risk overwriting a productive notes.ini.
Stay tuned. I am working on a fix...

Thanks, the build issue is also fixed.

Hi Daniel,
I guess Borg is not your friend ;-)
After switching to CentOS Stream 9 and glibc 2.34, borg is not working again.
Even if I call borg info . inside the container, the following error message appears:
[9710] Error loading Python lib '/tmp/_MEINyen0F/libpython3.9.so.1.0': dlopen: libcrypt.so.1: cannot open shared object file: No such file or directory
Same error message at the command: borg --version
The latest Borg release is 1.2.4 (apart from the 2.x beta) and the release download contains a borg-linuxnew64 (built on Debian 11 "Bullseye" with glibc 2.31).

This is a moving target. I took a quick look. What is broken is the OpenSSL dependency.
Right now default borg is installed from the GitHub repository by just copying the binary. The binary does not work for CentOS Stream 9. Also the latest binary does not work when you just copy it.
I checked the CentOS Stream 9 repository and they now ship a current version.
So setting the default back to use what is in the Linux distribution would make sense.
I just modified the software/current_version.txt locally and tested.
If you don't specify a version, it will set the version to the version from current_version.txt.
Without an entry in this file, the installer uses the borg version in the distribution.
This seems to be the best choice today and this will be the new standard.
The other option you have is to use -borg=yes instead of just -borg. this will also override the default version in current_version.txt.
I have changed the default and pushed the change to the develop branch right now.
Please let me know if this works for you.
Daniel

the hashes are sha256 hashes. this is the same for all software packages.
I added this new option at some point to be independent from what is in the distributions.
turns out with OpenSSL dependencies and running and building on different distributions, this isn't a good idea any more.
Also troubleshooting it, isn't a good idea. We should use what borg includes into the distributions. the version is up to date.
By the way CentOS Stream 9 is at:
OpenSSL 3.0.7 1 Nov 2022 (Library: OpenSSL 3.0.7 1 Nov 2022)

The error message isn't even a direct Borg error message. It's a Python dependency.

Without an entry in this file, the installer uses the borg version in the distribution.
This seems to be the best choice today and this will be the new standard.

This is really a great implementation.
It works without the entry in the current_version.txt
Thanks!


Serbian

Thanks a lot @Jason3S for creating script-based commands.
This PR fixes the cspell language for the serbian sample and updates the README to explain in more detail how sr, sr-Cyrl and sr-Latn work.

@ttasovac,
Thank you.
Maybe list the commands:

I added the commands. By the way, @Jason3S, should I now also change "preview" to "false" in package.json?


Changed nbt.py to use logging module instead of print

The accelerated NBT module information best serves the user when
it can be prioritized appropriately.  By using the logging module,
the user can be informed of the state of the module while debugging
their code but not bothered with it when log levels are set higher.

To me, this seems like I good idea (not just with nbt.py with the entire project and MCEdit).
The only problem is projects not using logger and never seeing any output at all, however making them use logger is probably a good thing :)
Any objections/comments? If not, I'll merge tomorrow.

Actually, if your project doesn't use the logging module, you get lots of this:
INFO:../pymclevel/nbt.pyc:Accelerated NBT module loaded.
INFO:../pymclevel/materials.pyc:Loading block info from <open file '../pymclevel/minecraft.yaml', mode 'rb' at 0x30aaae0>
INFO:../pymclevel/materials.pyc:Loading block info from <open file '../pymclevel/classic.yaml', mode 'rb' at 0x30aaae0>
INFO:../pymclevel/materials.pyc:Loading block info from <open file '../pymclevel/classic.yaml', mode 'rb' at 0x30aaae0>
INFO:../pymclevel/materials.pyc:Loading block info from <open file '../pymclevel/indev.yaml', mode 'rb' at 0x30aaae0>
INFO:../pymclevel/materials.pyc:Loading block info from <open file '../pymclevel/pocket.yaml', mode 'rb' at 0x30aaae0>

So projects that use pymclevel are probably using the logging module in self-defense... :-)
Thanks for liking the idea!
Jack.

Well, I'll be a monkeys uncle. In it goes.


Fix gitCredHelper function for go module usage

Since the =~ is a bash extension, it requires double brackets
to be evaluated properly. Using only single brackets results in
a syntax error.
Could also be updated to use:
if printf "${t}" | grep -Eq ":" ; then

if POSIX compatibility is important. Without this update trying to
build with go modules and private repositories fails:

}; gitCredHelper get: 46: [: MY_GITHUB_TOKEN: unexpected operator

Thanks!


How to create weekly log using Winston daily rotate?

How to create weekly log using Winston daily rotate?
Originally posted by @sanket2351 in https://github.com/winstonjs/winston/issues/10#issuecomment-925089059

How to create weekly log using winston


Matrices: Intro-6, * operation between matrix and vector has a mistake

Multiplication of the vector and matrix (in the same order), result is given as vec2(7, 10) which is wrong and must be changed to vec2(5, 11) - as below
vec2 u = vec2(1, 2) * m;  //u = vec2(7, 10)
instead it must be,
vec2 u = vec2(1, 2) * m;  //u = vec2(5, 11)
The operation executed is dot product.
Similarly, multiplication of the matrix and vector (in the same order) result is given as vec2(5, 8) which is wrong again and must be changed to vec2(7, 10) - as below
vec2 v = m * vec2(1, 2);  //v = vec2(5, 8)
instead it must be,
vec2 v = m * vec2(1, 2);  //v = vec2(7, 10)

+1
I found now this problem (following the course as student), good to know that you already reported and solved.
Please, merge this fix!

+1
Can someone give this a merge? Confused me a bit working through it today.


Cannot add repos to TTC_REPOS if the path contains a space

Hello 👋 Loooove this app so much!!! 💖
Currently, it seems like you cannot add any paths that contain a space to the TTC_REPOS environment variable.

Oh yeah you're totally right. I use the space as delimiters. I'm gonna add a "depth" argument soon so that you can just use the parent director you instead

<EMAIL_ADDRESS>now has a TTC_REPOS_DEPTH argument, so I'm going to optimistically mark this as closed (otherwise i'd have to switch from spaces to commas and break all the existing users 😭)

Thanks, I think that will help a lot of people!
In my case, the space is quite early on in the path. It looks something like this: /home/wilhelm/Google Drive/folder/folder/folder/folder/actualProject
So I think it would take forever to check every folder in my Google Drive for .git directories.
If a character that's typically not allowed in paths was to be the delimiter, would that still break things for existing users? Maybe a character like |

Oh! Good news: this is only broken because I wrote dumb code, and the instructions actually tell you to use commas and not spaces. Gimme a sec.

ooo 😁
To clarify: I meant keep commas as the delimiter in the environment variable, but use a more sepcial character to split here: https://github.com/notwaldorf/tiny-care-terminal/blob/0e7d6ae65953bc8348406f65b70d0bbe60a1e384/config.js#L14

Well, that's not the problem, this is the problem: https://github.com/notwaldorf/tiny-care-terminal/blob/master/standup-helper.sh#L33
the shell script takes "everything that wasn't passed to -d or -m" to be directory names, and because they're space separated, it's fine. If they're separated by a different delimiter, I have to do a bash hack to get around that (like split the string. everything in bash is a hack 😅)

same problem here


Use Yarn in circleci

Update circle.yml to persist Yarn cache, and use Yarn to install deps

Solved by #26


[master-next] swift-llvm-opt: update for createTargetMachine API

What's in this pull request?

Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

The relocation model is now passed as an optional.  Explicitly cast the value to
an Optional before the call.

CC @gottesmm

@vedantk committed a similar patch to master-next.  Abandoning.


Baseline - Refactor GCP check stack

[ ] Create README.md
[ ] Add Guardrail links
[ ] Add mod links
[ ] Refactor to be in sync with baselines

Repository: https://github.com/omerosaienni/tdk
Branch: baseline-refactor-gcp-check-stack


[Enhancement]: support SSH contexts

Module
Core
Proposal
Similar to https://github.com/testcontainers/testcontainers-java/issues/5837 but I am thinking of using SSH context also provide the necessary capability using SSH to provide tunnels to the remote services for testing.
Also similar to https://github.com/testcontainers/testcontainers-node/issues/169 which was closed for inactivity and I am looking for a Java solution.
The SSH tunnel requirement should not be mandatory unless we expose ports and have to access the exposed ports.  This is to simply test if the service comes up and passes their healthcheck.
This will allow me to have a Docker machine outside of my laptop to run my services and free up resources on my laptop.
I am pretty sure it's not working with the context as I use because when I execute the test and do docker ps  after I use docker context use [remote]
I have also tested setting the DOCKER_HOST environment variable for the test and it still uses the local one.

Please check https://github.com/testcontainers/testcontainers-java/pull/5039#issuecomment-1039041020


python3Packages.molecule{,-docker}: init

Motivation for this change
Package Molecule from Ansible.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@grahamcOfBorg build molecule molecule-docker

@GrahamcOfBorg build molecule molecule-docker

With the latest molecule version the molecule-docker checkPhase is starting but the tests are still failing:
  ImportError: cannot import name 'lru_cache' from 'molecule.util' (/nix/store/w8wl95k9a0r3fspyg4pahn1yy8g1gpv3-python3.9-molecule-3.5.2/lib/python3.9/site-packages/molecule/util.py)

  ----------------------------------------------------------------------
  Ran 1 test in 0.007s

  FAILED (errors=1)
  Test failed: <unittest.runner.TextTestResult run=1 errors=1 failures=0>
  error: Test failed: <unittest.runner.TextTestResult run=1 errors=1 failures=0>

@GrahamcOfBorg build molecule molecule-docker

@ilpianista Thank you., it looks good now. Can you just squash the fixup commits?

@FRidh @jonringer friendly ping, thank you

ansible-compat is still failing to build and the packages are now out-of-date
I'm a bit tired of seeing this still not merged (no offence intended), so I've updated them, fixed the test and rebased unto master. Here's the branch with my changes.

@ilpianista Why close the pull request? You can pick up the commits and force push here. I could do that myself but I'd rather you verify the packages actually work first.

@rnhmjoj I'm sorry, but I thought that you were going to open a new PR from your branch.

@rnhmjoj it looks like I cannot re-open this PR. However, molecule does work fine! Thank you

Yes, it looks like github doesn't like if you delete the branch on your fork. Can you open a new PR?


Create EC2 instance with nonexistent security group raises exception instead of giving an appropriate error

My team encountered this when moving from an old version of localstack to a newer one. Apparently in an older version, we could just pass in a non-existent security group as part of the configuration for an EC2 instance request, but in the newer version moto actually looks up the security group.
This line:
https://github.com/spulec/moto/blob/9fc64ad93b40ad179a828be19e6da940e7256e02/moto/ec2/models/security_groups.py#L572
will return the security group found or None if nothing was found. But this line:
https://github.com/spulec/moto/blob/9fc64ad93b40ad179a828be19e6da940e7256e02/moto/ec2/models/instances.py#L482
will try to access the security group ID from the security group object, without realizing the possibility that the security group may not exist, and instead we have a None object in the list.
The call stack we got (from localstack 0.14.5 (which seems to be using moto 3.1.12 if I'm reading that right), but I checked and the relevant lines in moto's master branch don't seem to have changed) is:
2022-09-19T06:47:22.130:WARNING:localstack.utils.server.http2_server: Error in proxy handler for request POST http://diamond-localstack:4566/: 'NoneType' object has no attribute 'id' Traceback (most recent call last):
  File "/opt/code/localstack/localstack/utils/server/http2_server.py", line 185, in index
    result = await run_sync(handler, request, data)
  File "/opt/code/localstack/localstack/utils/asyncio.py", line 85, in run_sync
    return await loop.run_in_executor(thread_pool, copy_context().run, func_wrapped)
  File "/opt/code/localstack/localstack/utils/threads.py", line 39, in run
    result = self.func(self.params, **kwargs)
  File "/opt/code/localstack/localstack/utils/asyncio.py", line 30, in _run
    return fn(*args, **kwargs)
  File "/opt/code/localstack/localstack/services/generic_proxy.py", line 993, in handler
    return modify_and_forward(
  File "/opt/code/localstack/localstack/services/generic_proxy.py", line 534, in wrapper
    value = func(*args, **kwargs)
  File "/opt/code/localstack/localstack/services/generic_proxy.py", line 614, in modify_and_forward
    listener_result = listener.forward_request(
  File "/opt/code/localstack/localstack/services/edge.py", line 170, in forward_request
    result = do_forward_request(api, method, path, data, headers, port=port)
  File "/opt/code/localstack/localstack/services/edge.py", line 225, in do_forward_request
    result = do_forward_request_inmem(api, method, path, data, headers, port=port)
  File "/opt/code/localstack/localstack/services/edge.py", line 249, in do_forward_request_inmem
    response = modify_and_forward(
  File "/opt/code/localstack/localstack/services/generic_proxy.py", line 534, in wrapper
    value = func(*args, **kwargs)
  File "/opt/code/localstack/localstack/services/generic_proxy.py", line 614, in modify_and_forward
    listener_result = listener.forward_request(
  File "/opt/code/localstack/localstack/http/adapters.py", line 51, in forward_request
    response = self.request(request)
  File "/opt/code/localstack/localstack/aws/proxy.py", line 43, in request
    return self.skeleton.invoke(context)
  File "/opt/code/localstack/localstack/aws/skeleton.py", line 153, in invoke
    return self.dispatch_request(context, instance)
  File "/opt/code/localstack/localstack/aws/skeleton.py", line 165, in dispatch_request
    result = handler(context, instance) or {}
  File "/opt/code/localstack/localstack/aws/forwarder.py", line 61, in _call
    return fallthrough_handler(context, req)
  File "/opt/code/localstack/localstack/services/moto.py", line 109, in proxy_moto
    status, headers, content = dispatch_to_moto(context)
  File "/opt/code/localstack/localstack/services/moto.py", line 139, in dispatch_to_moto
    return dispatch(request, request.url, request.headers)
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/core/responses.py", line 205, in dispatch
    return cls()._dispatch(*args, **kwargs)
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/core/responses.py", line 315, in _dispatch
    return self.call_action()
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/core/responses.py", line 405, in call_action
    response = method()
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/ec2/responses/instances.py", line 83, in run_instances
    new_reservation = self.ec2_backend.add_instances(
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/ec2/models/instances.py", line 607, in add_instances
    new_instance = Instance(
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/ec2/models/instances.py", line 160, in __init__
    self.prep_nics(
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/ec2/models/instances.py", line 468, in prep_nics
    group_ids.extend([group.id for group in security_groups])
  File "/opt/code/localstack/.venv/lib/python3.10/site-packages/moto/ec2/models/instances.py", line 468, in <listcomp>
    group_ids.extend([group.id for group in security_groups])
AttributeError: 'NoneType' object has no attribute 'id'

I'm going to apologize here because normally I would try to dig into more information, compare the actual behaviour of AWS in this specific instance and such, but I'm under some time pressure at the moment and can't get into that right now -- but I didn't want to leave this without at least opening the issue here for your awareness. We have a workaround (basically, deferring upgrading the localstack version for now) so I can't stay on this at the moment.
Thanks!

Thanks for raising this @cheshirex!

Looks like AWS returns this error:
{'Code': 'InvalidGroup.NotFound', 'Message': "The security group 'sg-bbfe0e' does not exist in VPC 'vpc-...'"}


Display database name

Hello
is it possible to display the dbname between the connection name and the schema in the toolbar. or in the property of the script worksheet?
kind regards

issue 3426.docx

hello
thanks you for the information.it's perfect


ca is required in the validate_server.json file, but not mentioned in Setup steps

I'm trying to generate a self-asigned cert for testing purpose, but didn't seem to be working.
I got the The server certificate is invalid. when running the validation step.
I also noticed the mention of CSR in Patricks article: https://dev.to/patrickdemers6/fleet-telemetry-streaming-data-from-your-tesla-2i97. Is this step required?
Many thanks in advance 🙏

This can be closed. Basically, these are different things:

the private and public key pair - this is used to authorise commands sent to the vehicle
the private, cert and ca created for mTLS for - this is to build secure connection between the vehicle and the fleet telemetry server.


Dynamic imports in NextJS project in Turbo monorepo dont transpile correctly

What version of Turborepo are you using?
1.0.13
Describe the Bug
I'm not sure where to put this issue, as it might have something to do with several different packages (yarn workspaces, next-transpile-modules, turbo, country-flag-icons) but I ran into the issue when trying out Turborepo with NextJS, so i'm putting it here. Maybe at a minimum can get routed to the right place this way. (Cross posted it to NextJS https://github.com/vercel/next.js/issues/32559)
On a Turborepo monorepo, if I try to use a dynamic import with a specific library my site depends on, country-flag-icons, NextJS tries to import a typescript definition file and then predictably fails. If i import it directly without the lazy loading, it works, and my project without the monorepo structure also has it working, so i guess it might have something to do with the yarn workspaces setup.
Expected Behavior
It doesn't try to load a definition file and instead tries to load the actual code.
There's also a separate issue i experienced which might be related, which is that for the amplitude-js library, i was also doing a dynamic import to lazy-load that library like so:
const amplitude = await import("amplitude-js");
amplitude.getInstance(); // ...

without the monorepo, this works just fine, but with the monorepo, i need to access the default export explicitly or else it fails, saying getInstance doesn't exist:
const amplitude = (await import("amplitude-js")).default;
amplitude.getInstance(); // ...

not sure if that's related but seems theres something wonky happening with dynamic imports, both with and without the next/dynamic component.
To Reproduce
Repro repo: https://github.com/osdiab/repro-monorepo-import-definition

create a turbo repo with npx create-turbo@latest
add country-flag-icons as a dep of the app
use next/dynamic to lazy load a flag from country-flag-icons
Run yarn dev and look at the site, it fails and says it's trying to load a typescript definition file

note that it worked A) when i just used NextJS without turborepo and B) if i don't load it lazily

@osdiab experiencing the same issue with amplitude-js... how did you manage to fix it? adding .default does the trick but did you manage to get it working without it?

@aalises in case it wasn't clear from the merged PR, it seems he changed the configuration of the turborepo starter so that these work as expected out of box


Failing to download https://www.bbc.co.uk/iplayer/live/bbcnews

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.04.17. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2019.04.17

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones
[ x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
C:\Users\aaron\Videos>dl -vv https://www.bbc.co.uk/iplayer/live/bbcnews
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-vv', 'https://www.bbc.co.uk/iplayer/live/bbcnews']
[debug] Encodings: locale cp1252, fs mbcs, out cp850, pref cp1252
[debug] youtube-dl version 2019.04.17
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17763
[debug] exe versions: ffmpeg N-91407-g3c4af577aa, ffprobe N-91407-g3c4af577aa
[debug] Proxy map: {}
[bbc] bbcnews: Downloading webpage
ERROR: no suitable InfoExtractor for URL https://www.bbc.co.uk/programmes/None
  File "__main__.py", line 19, in <module>
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\__init__.py", line 474, in main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\__init__.py", line 464, in _real_main
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\YoutubeDL.py", line 2008, in download
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\YoutubeDL.py", line 807, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\YoutubeDL.py", line 868, in process_ie_result
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\YoutubeDL.py", line 830, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\YoutubeDL.py", line 624, in report_error
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmp1tqz5pcu\build\youtube_dl\YoutubeDL.py", line 586, in trouble
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
ERROR: no suitable InfoExtractor for URL https://www.bbc.co.uk/programmes/None
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

Normal non-live BBC programmes download okay.


Support region for deploy.sh.

When the region is specified other than the default one, the current deploy.sh failed when creating a bucket or cloudformation for the go template.

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


Mark code in markdown-content.md

What kind of changes does this PR include?

New or updated content

Description

Uses our new code snippet inline marking feature to highlight important parts of the code blocks.

I was thinking we could add highlights to the second astro.config.mjs. Maybe the import line and the remarkplugin line. What do you think?

I was thinking we could add highlights [...]. Maybe the import line and the remarkplugin line. What do you think?

I agree! Added green insert markings for both lines as suggested. :) Did anything else catch your eye or are we good to go?

I was thinking we could add highlights [...]. Maybe the import line and the remarkplugin line. What do you think?

I agree! Added green insert markings for both lines as suggested. :) Did anything else catch your eye or are we good to go?

Looks good to me!


feat(tactic/refine_struct): add refine_struct to use goal tags

for refining structures

Ideas for future direction:

I'd like to have an optional := expr after have_field that would allow you, if desired, to specialize for a given structure.

If possible, when Lean 4 comes out, I'd like to merge it with refine from core.

I just rebased this PR. It's ready when you are


correct typos

I fixed some typos in Concurrency/PartialAsyncTask.swift.
Please check out the changes.

cc. @ktoso @kavon
Forgot to tag you guys. Thanks in advance :)

@swift-ci please smoke test

Thanks :)

@Achoo-kr Please read our contributing guidelines before submitting changes, regardless of how simple they are. If you have time before this gets auto-merged, please consider amending your commit message to start with a capital letter and include the file name that contains the typo fixes.

@AnthonyLatsis
I didn't know that. Thanks for noticing!


Added quantlib functions to quant folder

okay, I just removed my old fork, created a new one and then added my quantlib modules; now I"m submitting this pr. I was able to pass the tests locally

Thanks for your comments, I'll move/rename the test file and update the
requirements files, maybe tonight I'm away from my computer now.
On Mon, May 17, 2021, 6:02 PM Graeme Holliday @.***>
wrote:

@Graeme22 commented on this pull request.
In src/quant/test_models.py
https://github.com/Graeme22/tastyworks-cli/pull/13#discussion_r633920202
:

@@ -0,0 +1,24 @@
+"""Apply unit test to models"""
+from . import models as sc

+def test_model():
probably best to name the file test_quant.py
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Graeme22/tastyworks-cli/pull/13#discussion_r633920202,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEY7CLDA73WJF2NQLXYWCZLTOGOBDANCNFSM45BJOJWA
.

Should I edit the same commit and force-push it or should I make a new
commit?
On Mon, May 17, 2021, 5:55 PM Graeme Holliday @.***>
wrote:

@Graeme22 commented on this pull request.
In src/quant/test_models.py
https://github.com/Graeme22/tastyworks-cli/pull/13#discussion_r633917678
:

@@ -0,0 +1,24 @@
+"""Apply unit test to models"""
+from . import models as sc

+def test_model():
this should be in the tests directory at the top of the project
hierarchy, that way it'll get run with pytest.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Graeme22/tastyworks-cli/pull/13#pullrequestreview-661486223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEY7CLDNEYNNPUA4E6TO4ODTOGNGLANCNFSM45BJOJWA
.

Should I edit the same commit and force-push it or should I make a new
commit?
On Mon, May 17, 2021, 5:55 PM Graeme Holliday @.***>
wrote:

@Graeme22 commented on this pull request.
In src/quant/test_models.py
https://github.com/Graeme22/tastyworks-cli/pull/13#discussion_r633917678
:

@@ -0,0 +1,24 @@
+"""Apply unit test to models"""
+from . import models as sc

+def test_model():
this should be in the tests directory at the top of the project
hierarchy, that way it'll get run with pytest.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Graeme22/tastyworks-cli/pull/13#pullrequestreview-661486223,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEY7CLDNEYNNPUA4E6TO4ODTOGNGLANCNFSM45BJOJWA
.

A new one is usually better, and will retrigger the tests.

okay @Graeme22 , I made those changes. you have to review them and merge the pr, correct? let me know if there is anything else.


Update contentful code for new service interface

@lauraluiz @schleichardt
Could you have a look?
I've updated code to meet new requirements.
I am updating README.md file.

Besides the comment, I think the rest is good to go 👍

@lauraluiz so I will merge.
Should I assign PR to you when I am updating PR according to your comments, like I did before?
I am planning to integrate this service to my sunrise project.
It has every functionality which are required for now.

@dariuszor sorry, the update was fine.
Regarding to Sunrise, I am currently updating the Cms behaviour from it. You won't be able to use it until I finish. I plan on finishing the changes today, then I'll create a snapshot version you can use.

@lauraluiz did you update the CMS behaviour already? Please let us know when it's finished.

@dariuszor the new CMS structure is being used in Sunrise already, with version 0.8.2-cms-update-SNAPSHOT. In order to use CMS from a controller, you have to implement the interface WithCmsPage. It is used already in HomeController, if you want an example.
Regardless of that, an open issue is that this CMS service is not being released yet and the common stuff needs to be removed from one of the sides (now we have 2 CmsService and so on). Are you guys working on some temporary solution for that already or are you expecting us to do this part?

@lauraluiz we haven't been working on a temporary solution here yet and would expect you guys to do that. Until when could you finish it?


fix: ensure pre-commit uses args

running cascgen pre-commit some-filter/raw-bundles/thing fails -- imo it should allow args as cascgen generate does.
So, here we go.

Hi @vetsin, thanks for the PR.
It might be better to allow the generate command to run in its own shell with its own variables. However, I can see the benefit in passing the filter. How about passing the args onto the command with something like this?
diff --git a/utils/generate-effective-bundles.sh b/utils/generate-effective-bundles.sh
index 23f0669..d4314ad 100755
--- a/utils/generate-effective-bundles.sh
+++ b/utils/generate-effective-bundles.sh
@@ -1020,7 +1020,7 @@ createTestResources() {

 runPrecommit() {
     PRE_COMMIT_LOG=/tmp/pre-commit.check-effective-bundles.log
-    $0 generate > "$PRE_COMMIT_LOG" 2>&1
+    $0 generate "${@}" > "$PRE_COMMIT_LOG" 2>&1
     # if we:
     # - ran without recreating the plugin catalogs (DRY_RUN=1)
     # - find changes to effective plugins directories
@@ -1157,7 +1157,7 @@ case $ACTION in
         ;;
     pre-commit)
         processVars "${@}"
-        runPrecommit
+        runPrecommit "${@}"
         ;;
     generate)
         processVars "${@}"


Persist measure geometries and rework layer options, map move improvements

Enhancement for measure control: allow geometries to persist.
After the measuring finished, the measured geometries are shown on the map overlay until stop is called or a new measurement is started.
In the existing version the geometry is hidden after the current measuring has been finished.
This option can be disabled.
Missing functions were added to layer options, new classes were added for these options and typed functions in OLFactory to create the according layer options for a given source instance.
It is now possible to catch map movements while they occur, not only after the map has finished moving.

Hello Sebastian,
please provide some tests and examples for new features.


Avoid extra conditional checks with function params

Assigning null and then doing x || this.x means you end up doing an extra check that's not really needed. You always have this.state and this.props, and both _updateScrollLeftForScrollToColumn and _updateScrollTopForScrollToRow are never called with falsy values. (If it were possible for em to get called with falsy values, you'd probably wanna skip the = null in the function param instead.)

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

Merging [#239][cc-pull] into [master][cc-base-branch] will decrease coverage by <.01%

@@             master       #239   diff @@
==========================================
  Files            30         30
  Lines           952        948     -4
  Methods          85         85
  Messages          0          0
  Branches        206        202     -4
==========================================
- Hits            913        909     -4
  Misses           39         39
  Partials          0          0

Powered by Codecov. Last updated by [6876cb2...a11459e][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bvaughn/react-virtualized/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bvaughn/react-virtualized/compare/6876cb27a89b626ccb5c4227b7d3739550880657...a11459ec5bef4d8b3646be79c4f0d2b62c80731b
[cc-pull]: https://codecov.io/gh/bvaughn/react-virtualized/pull/239?src=pr


Add header for Fair2

The type of this PR is: feat
This PR resolves FX-2208
Description
Adds the header for the new Fair2 view displaying a fair's main image, logo/icon, title, description and a link to see more information that navigates to a to-be-built /info view. Also wraps the subcomponents of Fair2 into a FlatList to enable scrolling.
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)

Looks good! I'm gonna merge this in to work off of 😊 . #mergeongreen


fixed iskcon event card styling

I have fixed the iskcon event card styling. Please review it and merge it.

Pls do mention the issue number this PR closes.
Can be Merged @akshitagupta15june

This is issue #198 .


Code for fine-tuning subnets for a downstream task

Hi, I can't find the code used to fine-tune a subnet to a downstream task. Specifically, I want to reproduce the results for CIFAR-100. Is the code for that available somewhere?

You can reffer to classification test in https://github.com/open-mmlab/mmselfsup/tree/openselfsup. But you need to do some change to run with iteration based runner not epoch based runner

After many days of trial and error, I cannot reproduce your results for CIFAR-100. Would you mind sharing the code? The link you shared does not contain the exact code you used I suppose.

After many days of trial and error, I cannot reproduce your results for CIFAR-100. Would you mind sharing the code? The link you shared does not contain the exact code you used I suppose.

GAIA-ssl/configs/classification/cifar100/downstream_task_finetune_cifar100.py

Thank you very much, that config clears up a lot of things for me! :)

I successfully reproduced the results for CIFAR-100 with the provided config. However, how you found the subnet provided in that config is not clear to me. I find that the computed distance is not relatively small, so from a random sample of only a few subnets one could already find a better one. Would you be so kind to share the config for finding that subnet as well? Maybe I'm doing something wrong.

I successfully reproduced the results for CIFAR-100 with the provided config. However, how you found the subnet provided in that config is not clear to me. I find that the computed distance from the provided subnet to the supernet is quite average; from a random sample of only a few (2-3) subnets I can already find a better one. Would you be so kind to also share the config for finding the provided subnet? Maybe I'm doing something wrong.

Yes. you are right, we also found this when I want extend journal version. The domain really affects a lot.  In the small scale downstream, the better usage of this supernet is training subnets on downstream and search on its validation performance directly.

Actually I find that the smallest subnet usually has the lowest distance. Independent of the domain. Could you explain why that is?

I indeed don't use the smallest subnet to do a pilot experiments.  However, I check the architecture searched in COCO and cityscapes. Not smallest subnet has the lowest distance.  Do you find this phenomen in some classification datasets?  I am sorry I am not clear about this. Maybe the smalles subnet and the large subnet are always sampled during training because of sandwich rule?

Specifically for dense distance I find this actually.

On CIFAR-10 and a custom dataset I have.

I am sorry I really have no idea for this. What's your opnion? We can discuss it here.

I'm not sure yet. I did find some problem in your code. For dense distance, you compute the distance between q and k. You use the forward method of dynamic_moco with mode='extract'. You specify extract_from as encoder_q or encoder_k. However, in extract mode, it always uses self.backbone and does not use the extract_from parameter. So you're actually not computing the similarity of relative relations between student and teacher as described in the paper.

I'm not sure yet. I did find some problem in your code I think. For dense distance, you compute the distance between q and k. You use the forward method of dynamic_moco with mode='extract'. You specify extract_from as encoder_q or encoder_k. However, in extract mode, it always uses self.backbone and does not use the extract_from parameter. So you're actually not computing the similarity of relative relations between student and teacher as described in the paper, or am I missing something here?

Yes, you are very right. It's my fault to release a wrong version. This released part about dense prediction is indeed wrong. I have fixed it. Sorry for that.

You can use cityscapes dataset to validate the results in the paper. It only has about 3k images, you can verfity the searched architecture in 1G~2G FLOPs group.

Thanks again. I was wondering. If dense distance is also cosine distance, how come the values are much larger than 1? Doesn't cosine distance have to be in the range [-1,1]?

Thanks again. I was wondering. If dense distance is also cosine distance, how come the values are much larger than 1? Doesn't cosine distance have to be in the range [-1,1]?

Only the relation similarity distance is distance_metric == 'kl' is valid, because the dimension of feature may be wrong between subnet and the largest network. The code for distance_metric == 'cosine' is wrong because that's a dot product of two unnormalized vector


Change Logger arguments async and writer

This PR splits the async_writer argument into two arguments: async and writer. The async argument is the boolean flag for controlling whether the logger is asynchronous or not. The writer argument is for providing a custom writer object. Its possible a custom writer object may not be asynchronous, so the async_writer label is a bit misleading. in that use-case.
This PR also simplifies the Logger implementation by collapsing the logging and async_writer objects to just writer.

@dazuma We would really appreciate your input on this.

Superseded by #919.


metadata editor not working from my end

I can replicate a freeze in WebKit (sometimes?) when adding Creator.


any exmaple?

any exmaple for telling people how to use?

Yes, I'm going to write full documentation soon.

But the basic installation process has described in project README file.


Add and configure postgres

I rule and have completed some work on Case Manager that's ready for review!
Start the process to roll out postgres by installing pg gems and constructing a migration.
This is pretty much the bare minimum we can do to postgresify, and is intended to separate out the devopsy stuff from actual app work. Everything in here is basically a noop - it adds a postgres instance and a table that is not set up to get used yet. So pretty safe, from a risk perspective.
I added a hobby postgres db to test this on in sandbox. I am not sure how to do the db:create / db:migrate as part of deployment, or if we should do that. Likely when we roll this out for real it should be on grown-up paid dbs though.
This pull request makes the following changes:

install and configure pg in code
set up a table (papertrail's versions table) to prove migration works
configure CI/docker

no view changes
It relates to the following issue #s:

Bumps #2072

For reviewer:

Adjust the title to explain what it does for the notification email to the listserv.
Tag this PR:

feature if it contains a feature, fix, or similar. This is anything that contains a user-facing fix in some way, such as frontend changes, alterations to backend behavior, or bug fixes.
dependencies if it contains library upgrades or similar. This is anything that upgrades any dependency, such as a Gemfile update or npm package upgrade.

If it contains neither, no need to tag this PR.

promoting to ready for review!

I rebased this to be sure it wouldn't get too far out of sync - still looking good.
A question we should resolve before we merge this - should we be doing UUIDs in here rather than serial ints?

@xmunoz did you get the migration to run locally? I'm having issues but think it might be that I haven't configured postgres properly

Codecov Report

Merging #2075 (d1c930c) into main (268bedf) will decrease coverage by 0.15%.
The diff coverage is 25.00%.

@@            Coverage Diff             @@
##             main    #2075      +/-   ##
==========================================
- Coverage   96.31%   96.15%   -0.16%
==========================================
  Files          78       79       +1
  Lines        1843     1846       +3
==========================================
  Hits         1775     1775
- Misses         68       71       +3

Impacted Files
Coverage Δ

app/models/application_record.rb
0.00% <0.00%> (ø)

app/controllers/application_controller.rb
97.43% <100.00%> (ø)

Continue to review full report at Codecov.

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

Okay, uuids are turning out to be a big nevermind as standard issue. I'm going to keep pgcrypto enabled but revert the rest of it.

Okay, here's my plan:

merge this (not at 2am though)
deploy to stage and get that configured; if possible, figure out a way to autorun migrate via buildpack or whatever
spin up pg:hobby (paid heroku instances) on all our instances
deploy to funds with same configuration as stage
probably bump our heroku app.json to spin up a pg instance too for a bit

@xmunoz did you get the migration to run locally? I'm having issues but think it might be that I haven't configured postgres properly

Nope, I did not. We might consider adding a Github Action that brings up a dev environment to test every PR.

@xmunoz that's an error I associate with not having run rails db:create db:migrate - e.g. seeds can't populate into the database because the postgres isn't set up and migrated. Is docker-compose run --rm web rails db:create db:migrate working right, and then yielding the seed error?

@xmunoz that's an error I associate with not having run rails db:create db:migrate - e.g. seeds can't populate into the database because the postgres isn't set up and migrated. Is docker-compose run --rm web rails db:create db:migrate working right, and then yielding the seed error?

That fixed the error, and I was able to bring up a full local environment. Just make the documentation change and this should be good to go.

Oh also, do you have a plan to address the failing checks? Given that this is an intermediary change, it's probably fine that coverage decreases temporarily until the full transition is implemented.

yeah, I'm not sweating the coverage drop here (-0.16%?) - makes sense that it would happen given the nature of this PR, but I can address that if you like.
Re-requested a review from you in light of the docker-compose docs change - great catch on that @xmunoz , thank you very much.

(The coverage flag is codecov being sad about application_record.rb not being tested, which does not feel worth resolving to me.)

I'm gonna ship DARIA updates late tonight to clear out the queue and then probably get this deployed tomorrow (pending an approval review from either @xmunoz or @lomky ). I like where this is winding up quite a bit tbh.


Add :class:~.OpenGLImageMobject

Changelog / Overview

Motivation

Make ImageMobject use opengl shaders.
Explanation for Changes

Ported over some code from manimgl with some adjustment to make it manimce-compatible.
Also brought over a bugfix for transparent background from manimgl to the image shader: commit here
Doesn't work, only half of the png is rendered, and when zooming in some more weird behaviour with the movement of the image is observed:

becomes:

via
class a(Scene):
    def construct(self):
        a = ImageMobject("day_texture.png")
        self.add(a)
        self.interactive_embed()

Documentation Reference

N/A
Testing Status

Further Comments

Checklist

[ ] I have read the Contributing Guidelines
[ ] I have written a descriptive PR title (see top of PR template for examples)
[ ] I have written a changelog entry for the PR or deem it unnecessary
[ ] My new functions/classes either have a docstring or are private
[ ] My new functions/classes have tests added and (optional) examples in the docs
[ ] My new documentation builds, looks correctly formatted, and adds no additional build warnings

Reviewer Checklist

[ ] The PR title is descriptive enough
[ ] The PR is labeled correctly
[ ] The changelog entry is completed if necessary
[ ] Newly added functions/classes either have a docstring or are private
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings

With the most recent commit, a full square is rendered, although there are multiple issues I noticed.

There's some weird interaction going on in the center of the image when moved.

https://user-images.githubusercontent.com/70682032/127563382-1b2abfe7-6f0b-4728-a8aa-936af0a4a750.mp4

When writing to a movie using --write_to_movie it produces an error:

[mov,mp4,m4a,3gp,3g2,mj2 @ 0000022f4f3272c0] moov atom not found
[concat @ 0000022f4f31de80] Impossible to open 'file:C:/Users/laith/Documents/GitHub/hydro_manim/media/videos/mkt/1080p60/partial_movie_files/a/1491410813_1875215358_2043878721.mp4'
C:\Users\laith\Documents\GitHub\hydro_manim\media\videos\mkt\1080p60\partial_movie_files\a\partial_movie_file_list.txt: Invalid data found when processing input

2.1. However, when opacity is set to not be the default value of 1, (i.e. 0.99), it renders properly with --write_to_movie
2.2. Also, the scene is written to movie normally regardless of opacity if --disable_caching is passed.

Currently not possible to produce the correct dimensions of the image with this setup 🤔. Here is the image with ImageMobject:

Currently not possible to produce the correct dimensions of the image with this setup 🤔. Here is the image with ImageMobject:

This is because the width/height setters have a rescale_to_fit(..., scale=False) function that makes the width/height match the height/width respectively resulting in a 1:1 aspect ratio. Setting it to True instead will allow you to set the ImageMobject's width/height and have a rectangular texture. This needs to be refactored in some manner.

There's some weird interaction going on in the center of the image when moved.

I'm not exactly sure why this is happening, but my intial guesses are that it's an issue with the vertex shader and the calculations from the line gl_Position = get_gl_Position(position_point_into_frame(point)); -- Actually, this is the case. The perspective_scale_factor stuff in that shaders/include/get_glPosition.glsl is buggy and probably shouldn't be incorporated into the shader logic in the first place (it should be a part of the Camera's View/Projection matrices).

So I would not recommend merging this because this commit modifies a shader that is used by pretty much every Mobject (manim/renderer/shaders/include/get_gl_Position.glsl) but I commit this now to demonstrate the problem code I referred to in the previous comment:

The perspective_scale_factor stuff in that shaders/include/get_glPosition.glsl is buggy and probably shouldn't be incorporated into the shader logic in the first place (it should be a part of the Camera's View/Projection matrices).

This code should not have the weird warping issue.
We should be using model/view/projection matrices as uniforms within the shaders instead of this perspective_scale_factor magic. In reality, this may mean doing some other refactors first in order to properly have ImageMobjects in ManimCE. I'm not sure if ManimGL has working ImageMobjects; if so, there shader pipeline is probably different... but I would recommend fixing this after ManimCE properly uses MVP matrices.

I think this is already merged with #2534 closing this PR. Feel free to reopen if this is not the case