Ensure that line-length and doc-length behavior is consistent with Flake8

See: https://github.com/charliermarsh/ruff/issues/1784#issuecomment-1426597334

On further investigation, it seems that our current behavior is consistent with Flake8.

@charliermarsh

On further investigation, it seems that our current behavior is consistent with Flake8.

https://github.com/charliermarsh/ruff/blob/6049aabe27ee536d40c8cf6f446f475d1a937318/ruff.schema.json#L1339-L1340
As far as I can see, that is not the case. It does not work for docstrings, nor single comments (isolated or at the end of a line).
I have several E501 Line too long warnings when using comments, that flake would ignore. I would not even mind, but since there is no auto-fix for this, it makes this a unwelcomed hassle and I could not find a way to change it or turn it off for just the comments.
I am using the ruff 0.0.267.
My pyproject.toml file
...
[tool.ruff]
line-length = 88
...

[tool.ruff.pycodestyle]
max-doc-length = 180
...

Example file
import numpy as np # this is a long comment increasing the length of this line over 88 characters

# this is a very very very very very very very very very long single line comment with a length over 88 characters.

def func():
    """this is a very very very very very very very very very long doc string with a length over 88 characters."""
    print(np.array([1,2,3]))

Warnings
Untitled-1.py:2:89: E501 Line too long (97 > 88 characters)
Untitled-1.py:4:89: E501 Line too long (115 > 88 characters)
Untitled-1.py:7:89: E501 Line too long (114 > 88 characters)
Found 3 errors.

Also note that Ruff doesn't enable the W rules by default:
❯ ruff foo --select W505
foo/foo.py:3:101: W505 Doc line too long (115 > 100 characters)
foo/foo.py:6:101: W505 Doc line too long (114 > 100 characters)

And that Flake8 also doesn't flag end-of-line comments as "doc lines":
❯ flake8 foo.py --select W505 --max-doc-length 100 --isolated
foo.py:3:101: W505 doc line too long (115 > 100 characters)
foo.py:6:101: W505 doc line too long (114 > 100 characters)

Do you mind sharing your Flake8 configuration? In my experience and testing, Flake8 flags all of those line as E501:

Excuse me, I overlooked the Error in my ignore list. You are right.


Game Dev Roadmap

Hi @rishicds
Issue Title
Game Development Design for GDSC RCCIIT

[x] I have provided the issue title.

Info about the Related Issue
The goal is to design and develop the Game Development section for the GDSC RCCIIT community.

[x] I have described the aim of the project.

Name
Devansh Vishwa

[x] I have provided my name.

GitHub ID
Please mention your GitHub ID.
dev129

[x] I have provided my GitHub ID.

Identify Yourself
GSSOC-EXT

[x] I have mentioned my participant role.

Closes
Enter the issue number that will be closed through this PR.
*Closes: #449 *

[x] I have provided the issue number.

Describe the Add-ons or Changes You've Made
Give a clear description of what you have added or modified.
I have contributed to the design of the Game Development section for GDSC RCCIIT, enhancing the layout, style, and structure to improve the user experience.

[x] I have described my changes.

Type of Change
Select the type of change:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Describe how your changes have been tested.
I tested the design changes by viewing them in different screen sizes and ensuring responsiveness, as well as ensuring that the visual elements align as expected.

[x] I have described my testing process.

Checklist
Please confirm the following:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.

Hi @rishicds , please review my issues and PR.


S4 | Create a process-image/:id route

Story

As an API, I should accept incoming requests by id and find their pending or complete process
So that I can check the status on them

Acceptance Criteria

a image-api/routes/process.js file that exports an express router
Has a /process-image/:id GET route that for now, only res.json({id: 1, status: 'pending'})

@RayA98


Add support for service accounts

Summary
With 1Password Service Accounts launched we should add support for configuring a service account with op-js.
Use cases

Any use case of the CLI via op-js that doesn't want to use biometrics or otherwise wants to perform actions not tied to a specific individual.
In op-js own codebase, we can move to SA-based integration tests.

Proposed solution
Much like Connect env vars, allow the user to set a service account token via env var: OP_SERVICE_ACCOUNT_TOKEN.
Note: 1Password CLI only supports a subset of commands with service accounts. It may be worth trying to figure out a solution that wraps each command in a check, if a service account is being used, if the command supports it.
Is there a workaround to accomplish this today?
Don't think so.

Setting process.env['OP_SERVICE_ACCOUNT_TOKEN'] actually works for me.

Hey @dreusel I tried to connect using the service account with process.env but it did not work. Can you please share your example in detail? Thanks

Addressed in https://github.com/1Password/op-js/commit/bf5f9971535de0bf9ab80e4cbe60e17cbe85fdc0


Support optional input/output files

Description
This PR introduces optional input/output files.
For some tasks in biometric computing, the number of input/output files varies case by case. Therefore, this PR adds a new feature called optional files. Once a file is marked with option: True, users do not need to register the files before creating tasks. The C, Python, Rust SDKs have been updated accordingly.
Fixes # (issue)
Type of change (select or add applied and delete the others)

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] API change with a documentation update
[ ] Additional test coverage
[ ] Code cleanup or just sync with upstream third-party crates

How has this been tested?
Checklist

[x] Fork the repo and create your branch from master.
[x] If you've added code that should be tested, add tests.
[x] If you've changed APIs, update the documentation.
[x] Ensure the tests pass (see CI results).
[x] Make sure your code lints/format.

Please rebase to the latest master for CI checking.

Please rebase to the latest master for CI checking.

Done.


accept references to payloads in put requests

Describe the feature
many builders require their input to be owned, which means that in order to handle a possible retry under failure, the caller must make a copy that is most likely going to be thrown away. This is very wasteful, especially in high throughput applications and when writing large payloads.
Use Case
here's an example of writing records to kinesis. The batch here has to be fully owned, containing the payloads. Which means we need to clone it just incase we need to retry it.
                PutRecordsRequestEntry::builder()
                    .data(Blob::new(data))
                    .partition_key(common_partition_key)
                    .build()
                    .unwrap(), // test code

...
        let putter = kinesis
            .put_records()
            .stream_name(&name)
            .set_records(Option::Some(batch));

        let mut success = false;
        let mut attempts = 0;
        while !success {
            // annoying that we have to clone the data to be able to retry
            let results = putter.clone().send().await.unwrap(); // test code
            let failures = results.failed_record_count;
            if failures.unwrap_or(0) == 0 {
                success = true;
            } else {
                tracing::debug!("RETRYING A WRITE TO THE KINESIS STREAM {name}");
                // we may have dupe records now, how exciting! it is possible to
                // only resend the ones that failed, but it's not worth it here.
                attempts += 1;
                sleep(Duration::from_secs(1)).await;
                if attempts >= 10 {
                    panic!("Too many retry attempts writing to kinesis stream {name}");
                }
            }
        }

Proposed Solution
I don't think builders are a very idiomatic way of doing a lot of this in the rust ecosystem, although it seems to be the AWS way. If there were no builders, or struct objects, we would be able to say that the methods could simply take referencse rather than owned objects. However, I doubt we'll see builders done away with... so it would be good enough if select responses returned the input parameter instead of completely consuming. That behaves effectively the same as taking a reference.
Or take Rc wrappers of everything in the builders... or push that to the smithy layer and use Rc wrapped bytes in the Blob.
Other Information
No response
Acknowledgements

[ ] I may be able to implement this feature request
[ ] This feature might incur a breaking change

A note for the community

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue, please leave a comment

Thanks for the issue.

I don't think builders are a very idiomatic way of doing a lot of this in the rust ecosystem, although it seems to be the AWS way.

I'd argue that builders are fairly common in Rust but that's neither here nor there. The use of builders in the Rust SDK stems in part from dealing with model evolutions and being able to provide strong backwards compatibility guarantees.

I don't think there is much we can do here but I would ask why you are implementing retries on top of the SDK which already retries requests?

If there is a retry on top of the Kinesis PutRecords call, it is inadequate to handle the transient failures that occur when the stream is being scaled in or out. For my very particular usecase, it would be sufficient for the SDK to simply try harder at retrying a PutRecords but in general it would be nice if there was a way to avoid completely consuming the input as there are many usecases where holding onto the data after sending it to an AWS endpoint is required.

Actually, on reflection, the PutRecords batch call can't be retried automatically, because it supports partial success... so the retry would only work up until a partial success. I would imagine all endpoints with partial success are therefore susceptible to this same problem.
In this particular case, I think what is happening is that within the batch, the puts to shards that are being rescaled are failing, but the puts to shards that are fine are succeeding. A more efficient retry user-level logic would retry only the puts that failed the first time, but it's sufficient to retry everything. Regardless, to be able to avoid data loss, one must copy all the data and throw it away it most cases, because the builder consumes the input.


Provide Configuration Time insights to Build Scans

How long did each script take to evaluate?
How long did each plugin take to evaluate?
How much did each script/plugin contribute to configuration time overall (including afterEvaluate and similar hooks)
What network requests were made during configuration time? (e.g. dependency resolution at configuration time, plugin resolution, apply from: someUrl etc.)
How much file I/O occurred during configuration time? (e.g. walking directory trees at config time instead of at execution time)

Also: garbage collection pauses, and other interesting GC metrics.


Converts a documented values.yaml to MD format

Co-Authored-By: Guang Ya Liu<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sdake
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
Assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

values.tar.gz
An example values.yaml file to which this code processes.

@gyliu513 can you confirm your OK with merging this PR as per requirements in:https://github.com/istio/istio.github.io/pull/1497#issuecomment-396301038

/assign @linsun
/assign @geeknoid

@geeknoid - would like to integrate this somehow into the build system.
The workflow would be something like

Pull istio/istio
Run MD script on istio/istio/install/kubernetes/helm/values.yaml
generate MD files from that

barring that as an option, we can manually update.  If this approach is taken, could you offer up some idea of how to get the md file included in the helm.md file without vi :r?
Cheers
-steve

@sdake: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: sdake
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
Assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

Deploy preview for preliminary-istio failed.
Built with commit 9de3b45bafbda9b87e7eb6a7c9fd16b8d506b488
https://app.netlify.com/sites/preliminary-istio/deploys/5b278ea911b73b6e0e26c655

Closer - still incomplete as values.yaml hasn't been doc-stringified.

@mandarjog can you provide a review of the python code?
TIA
-steve

@sdake: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

G R O A N


Several polynomials

Description
Changed functionality so that the relevant functions work with a list of polynomials rather than a single one, as intended by the general interface. Tested that this works and things like swapping of proofs of values in those lists is detected.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (master)
[ ] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[ ] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

@Antonio95 note to just format the files you touch :)
we can make a separate PR to ark-poly-commit that does a fmt/clippy of the rest

Okay, I have now undone the fmt/clippy on all files other than src/ligero/*. Instead of revert, I did reset --soft (to HEAD~1) thinking this would lead to the same, only to realise the one difference is that with revert you can push normally afterwards (because you have only added commits) whereas with reset you have to force-push because the last commit in remote no longer exists. I hope this is okay in this instance - I know force-pushing must be used with care.
For the record, I can't find a way to apply fmt/clippy only to a given path. Some internet answers falsely propose things like cargo fmt -- src/ligero/mod.rs, but this doesn't work. It is possible these commands only apply to the current directory and subdirectories so that if one happens to want formatting/clipping exacly there one can cd and call the command - I haven't tested this, however.


Progress bar API

See https://www.electronjs.org/docs/latest/tutorial/progress-bar
I see two options for an API here.

Electrons route, a Window method to set the current progress. The problem here is that Windows is the only platform capable of a progress bar per window.
An App method to set the progress for the entire app, with an overload to provide the window (or a secondary method on window to set it per window).

Either way it is confusing on some platforms.

I like the Electron route. If one platform requires Window, we might as well make other platforms think the same way, even if it will make no difference for them.

Makes sense, what is the standard for attribution in this repo? After looking at Electrons way of doing this I cannot imagine a scenario where the implementation does not look nearly identical to theirs.

Electron is MIT:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

I don’t think our use will count as “substantial use”, but you can always leave a link pointing to their source next to impl


Add basic exlusion to build matrix

I took a crack at exclusions for the build matrix per my conversation with @bradrydzewski and this is a first shot.
Please bear with me as I've never written code in go before.
I also added unit tests that test also the contents of the build matrix instead of only testing the correct size.
Please let me know what you think.

 You should sign our Contributor License Agreement in order to get your pull request merged.

LGTM

does it possible to update this pr?

@bradrydzewski maybe something we want to have?


fixed ACE

📊 Metadata *
Fixed Arbitrary Code Execution
Bounty URL: https://www.huntr.dev/bounties/1-pip-tensorflow/
⚙ Description *
TensorFlow is an end-to-end open source platform for machine learning. It has a comprehensive, flexible ecosystem of tools, libraries, and community resources that lets researchers push the state-of-the-art in ML and developers easily build and deploy ML-powered applications. It is vulnerable to Arbitrary Code Execution.
💻 Technical Description *
Exploitable to Arbitrary Code Execution due to unsafe yaml.load in its python keras module.
🐛 Proof of Concept (PoC) *

🔥 Proof of Fix (PoF) *

👍 User Acceptance Testing (UAT)
just changed to safeload .
🔗 Relates to...
same disclosure as https://github.com/418sec/huntr/pull/1533

👋 Hello, @tensorflower-gardener - @B3EF has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.
Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.
If you have any questions or need support, come and join us on our community Discord!
@tensorflower-gardener & @B3EF - thank you for your efforts in securing the world’s open source code! 🎉

👋 Hello, @tensorflower-gardener - @B3EF has opened a PR to us with a fix for a potential vulnerability in your repository. To view the vulnerability, please refer to the bounty URL in the first comment, above.
Ultimately, you get to decide if the fix is 👍 or 👎. If you are happy with the fix, please write a new comment (@huntr-helper - LGTM) and we will open a PR to your repository with the fix. All remaining PRs for this vulnerability will be automatically closed.
If you have any questions or need support, come and join us on our community Discord!
@tensorflower-gardener & @B3EF - thank you for your efforts in securing the world’s open source code! 🎉

@yongtang - attaching for reference.

@yongtang - attaching for reference.


Notebook UX Explorations

Summary
This issue covers the ux explorations for our Notebook native support.
Backlog

[x] :running_man: 1. Cell toolbar
[x] :running_man: 2. Cell progress and status
[x] :running_man: 3. Output toolbar
[x] :running_man: 4. Drag and drop
[x] :running_man: 5. Folding indicators and folding UX
[x] :running_man: 6. Keyboard and cell focus
[x] :running_man: 7. Changing cell type
[ ] :running_man: 8. How does the current focus indication go together with SCM decorations?
[ ] What does a minimap for notebooks look like?
[ ] Overall polish: spacing, color, ...
[ ] Notebook diffing
[ ] Global notebook state and actions

Designs
1. Cell toolbar
Code Cells

Text Cells

2. Cell progress and status

3. Output toolbar
Switcher for output renderers

4. Drag and drop

5. Folding cells

6. Keyboard and cell focus
Code Cells

Text Cells

7. Changing cell type

8. Focus indicator & SCM decorations
TBD

when we will get this new jupyter notebook?

These initial explorations have been completed, please see our Notebook UX Feedback issues for an up-to-date status on notebooks.


react-native-webview  iOS Swipe back with Vue Router causes a blank page

Bug description:
I have a react-native project using Vue and Vue Router, with swipe gestures enabled I am unsure if this issue is coming in from the Vue Router or webview.
The gestures are enabled with a plugin using this line
<WebView
                    useWebKit
                    ref={webview => (this.webview = webview)}
                    style={{ flex: 1, width: global.sw }}
                    source={{ uri: 'xxx', headers: { 'Cache-Control': 'no-cache' } }}
                    automaticallyAdjustContentInsets={true}
                    decelerationRate="normal"
                    scrollEnabled={true}
                    javaScriptEnabled={true}
                    domStorageEnabled={true}
                    onMessage={this.invoke.listener}
                    nativeConfig={{ props: { flex: 1, width: global.sw } }}
                    onLoad={syntheticEvent => {
                        const { nativeEvent } = syntheticEvent
                        console.log(`onLoad nativeEvent.url: ${nativeEvent.url}`)
                    }}
                    onLoadProgress={({ nativeEvent }) => {
                        console.log(`nativeEvent.progress: ${nativeEvent.progress}`)
                    }}
                    onLoadStart={syntheticEvent => {
                        // update component to be aware of loading status
                        const { nativeEvent } = syntheticEvent
                        console.log(`onLoadStart nativeEvent.loading: ${nativeEvent.loading}`)
                    }}
                    onLoadEnd={syntheticEvent => {
                        // update component to be aware of loading status
                        const { nativeEvent } = syntheticEvent
                        console.log(`onLoadEnd nativeEvent.loading: ${nativeEvent.loading}`)
                    }}
                    onError={syntheticEvent => {
                        const { nativeEvent } = syntheticEvent
                        console.log('onError WebView error: ', nativeEvent)
                    }}
                    onHttpError={syntheticEvent => {
                        const { nativeEvent } = syntheticEvent
                        console.log('onHttpError WebView received error status code: ', nativeEvent.statusCode)
                    }}
                    onContentProcessDidTerminate={terminate => {
                        console.log('webView onTerminate')
                        this.webview.reload()
                    }}
                    originWhitelist={['*']}
                    onNavigationStateChange={this.handleWebViewNavigationStateChange}
                    allowsBackForwardNavigationGestures={true}
                    allowFileAccessFromFileURLs={true}
                    allowFileAccess={true}
                    allowUniversalAccessFromFileURLs={true}
                    allowingReadAccessToURL={'file://'}
                    showsHorizontalScrollIndicator={false}
                    showsVerticalScrollIndicator={false}
                />`

If I go one page, to the next, then swipe back. The previous page appears in the view and slides in as expected.
However, if I scroll even a few pixels before moving to the next page. When swiping back, the previous page appears white with no content. The current page slides out as expected Its only once the slide transition is finished that the previous page pops onto the screen
If I remove the scrollBehaviour function in my router it works correctly
js scrollBehavior (to, from, savedPosition) {
This function can be empty, or return various positions, it all causes the slide on the non active page to be white.

The issue is with the Safari browser. There are multiple solutions listed here


c2w-net with mapdir

Is it possible to pass --mapdir /mnt/share::./share argument to c2w-net?
I've tried to modify the code to include mapdir argument, but wasmtime refuses to recognize --mapdir as a valid argument unless it's the only argument used.
Using wasmtime 21.0.1.

The problem is actually on the wasmtime's side: https://github.com/bytecodealliance/wasmtime/issues/3936


Allow more granual CM_IGNORE_WINDOW

I would like to ignore a specific website (password manager) from clipmenud for that I would need more than to ignore the window name.
This should be possible, right? As there is more information stored in window properties than the application (in this case the string firefox).
% xprop | grep -i firefox
WM_CLASS(STRING) = "Navigator", "firefox"
WM_ICON_NAME(COMPOUND_TEXT) = "New Issue · cdown/clipmenu — Mozilla Firefox"
_NET_WM_ICON_NAME(UTF8_STRING) = "New Issue · cdown/clipmenu — Mozilla Firefox"
WM_NAME(COMPOUND_TEXT) = "New Issue · cdown/clipmenu — Mozilla Firefox"
_NET_WM_NAME(UTF8_STRING) = "New Issue · cdown/clipmenu — Mozilla Firefox"

Would this be something you are willing to add to clipmenu?

I don't understand why clipmenu only returns the string "Firefox" and not what xprops shows me. Could it be that clipmenu does not check the type. With a bit of chatGPT (as this type of code is way out of my comfort zone) it suggested that the window name might be a XA_COMPOUND_TEXT and clipmenu does not deal with that?

You can use clipctl disable to handle this. Extending CM_IGNORE_WINDOW for every single prop is extremely unwieldy, sorry.

Why do you think clipmenud sees something different? We request _NET_WM_NAME with prop UTF8_STRING.

Because I tested it. For this page, xprop gives me _NET_WM_NAME(UTF8_STRING) = "New Issue · cdown/clipmenu — Mozilla Firefox" where as if when I run clipmenud (with an added printf("Window Title: %s\n", win_title);) and copy something on this page it only ouputs the string Firefox. Where did all the other stuff go?

Do you have an idea why this is happening?

Sorry bothering you but I'd love to hear back from you if possible :)


Flickering

How do we avoid the terrible flickering on the styled toolbar buttons when resizing the form?

Are you using VCLStyles or in "non-styled" Windows?

I just compiled the toolbar demo program.

On my PC, flicker is visible only using VCL styles. The Flickering issue is also present in the standard toolbar... Probably caused by a WM_ERASEBACKGROUND sent to components in Styled mode... I'll investigate.

确实闪烁 闪的很厉害

Please check if problem persist with latest 3.2.0 version


Adds framework for using Impala C++ UDFs within Ibis

Addresses #262 and #195

Closing and resubmitting due to branch links changing


Dependency updates + linter related issue/todo

get to fortion 1.63.3 as well as all the other dependabot open PRs

lgtm, thx


Add "event" to glossary

This is an Enhancement Request

What would you like to be added

Add "event" entry to the Kubernetes Reference Glossary.
Why is this needed
events are referenced several times in other definitions, but it is not itself defined.

/assign
I would like to give this a try

Awesome, thanks @anubha-v-ardhan!


Issue with k8s.io/docs/concepts/workloads/controllers/jobs-run-to-completion/

This is a...

[ ] Feature Request
[x] Bug Report

Problem:
Running example following instructions to the letter:
File job.yaml:
apiVersion: batch/v1
kind: Job
metadata:
  name: pi
spec:
  template:
    metadata:
      name: pi
    spec:
      containers:
      - name: pi
        image: perl
        command: ["perl",  "-Mbignum=bpi", "-wle", "print bpi(2000)"]
      restartPolicy: Never
      backoffLimit: 4

Create the job:
$ kubectl create -f ./job.yaml

Output:
error: error validating "./job.yaml": error validating data: unknown object type schema.GroupVersionKind{Group:"batch", Version:"v1", Kind:"Job"}; if you choose to ignore these errors, turn validation off with --validate=false

Proposed Solution:
Provide working example tested on the latest version of the product or provide hits for supported versions.
Page to Update:
https://kubernetes.io/docs/concepts/workloads/controllers/jobs-run-to-completion/

Kubernetes Version:
$ kubectl version
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.0", GitCommit:"6e937839ac04a38cac63e6a7a306c5d035fe7b0a", GitTreeState:"clean", BuildDate:"2017-09-28T22:57:57Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.5", GitCommit:"17d7182a7ccbb167074be7a87f0a68bd00d58d97", GitTreeState:"clean", BuildDate:"2017-09-18T20:30:29Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Morning @pranasblk - thank you for the report!
I ran into this issue myself, and it's a bug with Kubernetes-CLI version 1.8 that has also been reported at https://github.com/kubernetes/kubernetes/issues/53309. There are a couple of work-arounds listed in the reporting bug that can be used until this issue is resolved. I recommend you subscribe to the bug listed here to get updates if you're interested.
Since this isn't an issue with the documentation directly, I'm going to go ahead and close this bug (basically in favor or https://github.com/kubernetes/kubernetes/issues/53309, which will track this issue more effectively), but please re-open if there's a documentation issue we can resolve.


Error: TypeError: e._v is not a function

Hi,  loving your work on this plugin - thanks for all the hard graft!
I'm seeing a console error - basically the same as #181.   I've updated vue to 2.1.8 as suggested and I've also pulled down the latest beta release (2.0.0-beta.13).
  Thanks

Aaaand, five minutes later it works.  Sorry about that - turned out I hadn't run a proper yarn upgrade but rather update vue within my package.json and some older vue dependencies were causing the error to occur.  A yarn upgrade did the trick.
Sorry and thanks again for the hard work!

Thank you @FourStacks, you saved my day. yarn update and gulp did the trick.

I have the same problem. I am using laravel 5.3. Can you help me?

Hey @legreco, best advice I can give (based on how I solved things) is to make sure you're pulling in the most recent beta as per the docs for the v2 version in your package.json. Delete your node modules folder then pull everything back in with 'yarn' (or npm install if you use npm). Hopefully that should get you back on the right track

After doing that it works!!!  Thank you a lot!! :)

👌


create native ops

[x] file access write files
[x] file access read directories etc...
[x] file access watch files for changes
[x] webserver
[ ] websocket server
[x] osc
[ ] kinect? - no macos etc
[ ] curl?
[ ] gltf npms stuff converter? gltf-pipeline ? compress? meshpack?
[ ] render html to images?
[ ] ffmpeg

https://www.npmjs.com/package/ffmpeg

screenshots: https://www.geeksforgeeks.org/how-to-take-screenshots-in-electronjs/

npm install electron-screenshot-service

[ ] my ip adress

[ ] save textures to hdd ?

[ ] home directory/downloads etc...

[x] ableton link


perf: ⚡ mark toList calls as non-growable

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

24 of 24 (100.0%)  changed or added relevant lines in 9 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
1325

Relevant Lines:
1325

💛  - Coveralls


Fix links and wording in glossary.mdx

Description (required)
Fix links for CI purposes, and fix some wording in the present definitions.
Please ignore this, I will remake this PR later!!!
Related issues & labels (optional)

Suggested label: soon, this will get the glossary label, right?

OK, clearly this system is getting ridiculous.  😅
Please see my note in the Discord thread about how we're going to proceed with the glossary! 💜


Reduce microbatch size

I was going through old issues, and we discussed lowering the micro batch size to 1 a while back in #678 to make the finetuning defaults more compatible with more models and GPUs out there. This will of course increase the runtime, but given that VRAM is usually the bigger bottleneck, I think it's a worthwhile change. Especially now that we have the config files in the config_hub, we also have good recommendations for hparams that work well for specific models so I think lowering resource needs is maybe preferred here.
Fixes #678

I agree and updated everything accordingly.

@rasbt My suggestion was meant as a follow up. The PR was totally fine as it was.

haha ok, in this case I am happy to revert.


hummock: block decode function receives impl Buf

Currently, the block decode function receives Bytes, which requires the underlying buffer must be continuous. It's optimized when the block is not compressed, but not convenient for common usages.
To solve it, the block decode function should receive impl Buf for common usage, and can also receive a continuous buffer optimization.

Checked the code and found that the non-continuous stream interface is not good for checksum calculation, I've tried to introduce ring buffer in foyer to solve this.


Use cache to speed up build & checks?

see https://github.com/AustinScola/mypy-cache-github-action

we do not do this

lets try again...


feat: addition of client code corresponding to packages customeradministration, authorizationmanagement, organization

Background
A new package, customeradministration was recently added to NerdGraph (few months ago), after which significant customer interests were received to have these under the purview of the New Relic Go Client, in order to be able to perform these queries in Go. This was conveyed to the team maintaining the Go Client (New Relic Internal: see this Jira ticket for more details), as a result of which work was logged to have the package added to the client.
In addition to the above, further interests were received in terms of PR contributions, in order to have a few mutations from the authorizationmanagement and organization packages also included in addition to customeradministration, in order to enable a few additional functions for customers via the Go Client (to manage organizations, grants, and more).

PRs
#1108
#1109
(formerly #1097 #1098 #1099 #1100)

This PR is a consolidation of all of the above -

addition of support to use endpoints within the customeradministration package on NerdGraph to perform queries
addition of support to use few endpoints within the packages authorizationmanagement and organization to manage grants and organizations
addition of test cases to cover most of the above

Tests
Tests have been added to all relevant code changes, however, a few of these endpoints added have been excluded (owing to reasons listed here - New Relic Internal), to be added as a follow up to this PR after necessary evalutation.
Things to Note
This PR also proposes the addition of two new secrets, INTEGRATION_TESTING_NEW_RELIC_ORGANIZATION_ID and INTEGRATION_TESTING_NEW_RELIC_AUTHENTICATION_DOMAIN_ID as it might not be a good idea to have these included in the code. These secrets have been added to this repository, and require to be added to local environment in order to test most test cases written in this PR.

Codecov Report
Attention: Patch coverage is 20.36082% with 309 lines in your changes are missing coverage. Please review.

Project coverage is 37.82%. Comparing base (3ff4ee2) to head (33b8b20).
 Consider uploading reports for the commit b7750b1 to get more accurate results

Files
Patch %
Lines

...stomeradministration/customeradministration_api.go
0.00%
238 Missing :warning:

...orizationmanagement/authorizationmanagement_api.go
0.00%
47 Missing :warning:

...authorizationmanagement/authorizationmanagement.go
0.00%
9 Missing :warning:

...g/customeradministration/customeradministration.go
0.00%
9 Missing :warning:

pkg/organization/organization_api.go
86.95%
4 Missing and 2 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1126      +/-   ##
==========================================
- Coverage   38.84%   37.82%   -1.03%
==========================================
  Files          86       92       +6
  Lines        5612     6044     +432
==========================================
+ Hits         2180     2286     +106
- Misses       3266     3589     +323
- Partials      166      169       +3

Flag
Coverage Δ

unit
37.82% <20.36%> (-1.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

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

⚠ A Note About a Limitation These Changes Include
(a general note, for anyone visiting this PR)
(As on 16 April, 2024)
In the customeradministration package, few types which correspond to fields whose values would need to match either a string, or be contained in a list of strings comprise Eq string json:"eq,omitempty" and `In []string `json:"in", as seen in customeradministration > types.go. Since the field Eq has an omitempty tag next to it, while the In field does not, failures when trying to use the Get methods in this package, with an error stating that these types are expected may be observed.
This is not a limitation in the New Relic Go Client or with the tool Tutone which we use for code generation; instead, it has been observed that this occurrence is because of the schema definition of these fields in NerdGraph, with In as [ID!] and Eq as [ID], which implies that In is a mandatory field with a list of IDs. Though this can be deselected in the NerdGraph UI, since an identical schema exists within the Go Client, the aforementioned failures are seen as In is expected to be populated.
It may be noted that adding an omitempty tag next to these In fields in the Go Client could solve the problem, however, the idea is to engage our friends over at the API team in a conversation to see if this scenario with NerdGraph could be made better, so the Go Client can tune into those changes, without having to deviate from the schema seen on NerdGraph. This shall be an ongoing effort, however, in the interim, in order to avoid the aforementioned error with the specified datatypes comprising Eq and In, a workaround could be to include the value you would like to specify in the Eq field in the In field instead, embedded in a list of strings. An example of this would be the following - if you would like to specify
			AuthenticationDomainId: &MultiTenantAuthorizationGrantAuthenticationDomainIdInputFilter{
				Eq: authenticationDomainId,
			},

in order to avoid the aforementioned error, the workaround would be to add the same variable authenticationDomainId in a list of strings to In instead, as this is currently behaving as a required field.
			AuthenticationDomainId: &MultiTenantAuthorizationGrantAuthenticationDomainIdInputFilter{
				In: []string{
					authenticationDomainId,
				},
			},

The following datatypes in customeradministration > types.go have been observed to display this behaviour and require the aforementioned workaround.

Type
Path in NerdGraph

MultiTenantAuthorizationGrantAuthenticationDomainIdInputFilter
customerAdministration > grants > filter! > authenticationDomainId

MultiTenantAuthorizationGrantGroupIdInputFilter
customerAdministration > grants > filter! > groupId

MultiTenantAuthorizationGrantScopeIdInputFilter
customerAdministration > grants > filter! > scopeId

MultiTenantAuthorizationRoleGroupIdInputFilter
customerAdministration > roles > filter! > groupId

MultiTenantIdentityUserGroupIdInput
customerAdministration > users > filter! > groupId

MultiTenantIdentityUserIdInput
~

OrganizationOrganizationCreateJobIdInput
customerAdministration > jobs > organizationCreateAsyncResults > filter! > jobId

UserManagementGroupIdInput
~

UserManagementUserIdInput
~


Added docCount property on aggregations and size option to search

I updated my fork using the Github interface fetchstream and looks like your master repository lost some commit from last week... I'm not sure to understand why. Maybe you know?

I'm not sure about why it become diverged from the main repo.
The change looks good, but could you also update the changelog and pubspec?

I just did it. Thank you for the review!

Great, thanks! I'll publish this shortly.

Thank you for the merge and for the fast publish every time!


User.bot not working correctly

Returns false for many bots that show the "BOT" tag in the app.

any more data for this? like under what circumstances this happens?

I'm not really sure. It seems to be random, even for the same bot. It changes between restarts.

Can confirm: http://i.imgur.com/how4PWM.png
Although this doesn't fix for me when restarting. Still says false for the same bots.

Can you check if this happens under each condition:

The user was online before the bot started
The user went offline => online after the bot started
A server member was edited (roles, nickname) before coming online

Should be fixed in 9.0.2


GEODE-3710 Make CodecAlreadyRegisteredForTypeException a RuntimeException

GEODE-3710 Make CodecAlreadyRegisteredForTypeException a RuntimeException.
If we throw this exception, the SerializationCodecRegistry will
already be in an invalid state, so it doesn't make sense for anyone to
catch it.
Signed-off-by: Sarge<EMAIL_ADDRESS>Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[n/a] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

@kohlmu-pivotal I had a talk with @PivotalSarge and I think we'd rather make the register method private. It won't work at present, because there's only one serialization type available and that's registered whenever the geode-protobuf JAR is loaded (always), and... YAGNI. Or, we can add the method to the public API when it's usable.


Fixed image preview for subdomainless (without CDN) applications

Basically we tried to parse maps_api_template incorrectly. So, right now we replace username from that string and it works perfectly in any situation, w/o subdomain.
Fixes #6274.
REVIEWER: @javierarce, you was my mate changing this stuff some time ago :D.

retest this please

Deploying in ded02.

:+1: code looks fine


How to use options asynchronous?

Hi,
I'm dealing with some async data for a select. Before I was calling the form like a component was working just fine because I pass a props to the component form with the data resolved, now, I move it into a router address, and now the form is the component itself.
This is how looks my code to bring the options
        {
          key: 'fk_artist',
          type: 'select',
          required: true,
          options: this.artistList,
          templateOptions: {
            label: this.$tc('artworks.form.fk_artist'),
            classes: {
              'form__select': true
            }
          }
        },

Now since the created () function is way to late and the data is asynchronous the form render without the options. There is a way to wait for the options or to pause the form render until all my async calls are done?

Hmm, you may have to use watchers or computed properties to do this. Maybe have a 'loaded' flag that conditionally displays the form. When your data is loaded, populate the fields and then switch the loaded flag on or something. Because all the options are returned from data() I don't think you'll be able to connect responsive data to them.

or, alternatively, you could have all your fields as a computed property, which will respond to async updates.


Automatic update of resolver list

Hi, do you have any plans to automatically update the resolver list as dnscrypt-osxclient does?
alterstep/dnscrypt-osxclient@a364d88326b806807b98a799f0f237af9b88b989
(for example when downloading the latest release tar, you end up with an outdated/not working resolver list.)

Problably not, as this is very dependant on the operating system/distribution and on how and where the software was installed. So, that's better left to package maintainers.
That said, it's super simple to do. This is how dnscrypt-osxclient does it:
#! /bin/sh

RESOLVERS_UPDATES_BASE_URL=https://download.dnscrypt.org/dnscrypt-proxy
RESOLVERS_LIST_BASE_DIR=/tmp
RESOLVERS_LIST_PUBLIC_KEY="RWQf6LRCGA9i53mlYecO4IzT51TGPpvWucNSCh1CBM0QTaLn73Y7GFO3"

curl -L --max-redirs 5 -4 -m 30 --connect-timeout 30 -s \
  "${RESOLVERS_UPDATES_BASE_URL}/dnscrypt-resolvers.csv" > \
  "${RESOLVERS_LIST_BASE_DIR}/dnscrypt-resolvers.csv.tmp" &&
curl -L --max-redirs 5 -4 -m 30 --connect-timeout 30 -s \
  "${RESOLVERS_UPDATES_BASE_URL}/dnscrypt-resolvers.csv.minisig" > \
  "${RESOLVERS_LIST_BASE_DIR}/dnscrypt-resolvers.csv.minisig" &&
minisign -Vm ${RESOLVERS_LIST_BASE_DIR}/dnscrypt-resolvers.csv.tmp \
  -x "${RESOLVERS_LIST_BASE_DIR}/dnscrypt-resolvers.csv.minisig" \
  -P "$RESOLVERS_LIST_PUBLIC_KEY" -q &&
mv -f ${RESOLVERS_LIST_BASE_DIR}/dnscrypt-resolvers.csv.tmp \
  ${RESOLVERS_LIST_BASE_DIR}/dnscrypt-resolvers.csv

Pretty straightforward.

@jedisct1 I see, thanks!


Update scala3-library to 3.1.3

Updates org.scala-lang:scala3-library from 3.1.1 to 3.1.3.
GitHub Release Notes - Version Diff - 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!

Files still referring to the old version number
The following files still refer to the old version number (3.1.1).
You might want to review and update them manually.
README.md
src/main/paradox/index.md

Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala3-library" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "@monthly" },
  dependency = { groupId = "org.scala-lang", artifactId = "scala3-library" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains, commit-count:1

Updated manually to 3.3.1


Remove redis cache

Fixes #589

Coverage: 82.838% (-4.4%) from 87.219% when pulling b8bb30c2043103c47c9af1412e8a78d43631b182 on remove-redis into c084239a5ddaea29f73cf39224cb3e83b1d7b0e6 on main.


Styling Classes

Create classes for:

Objectives
Project Tips
Activities

Use the Google Doc for ideas.

@Makosak we can mark this as closed, right?


hibernation: include machines in undesired state when hibernating/resuming

Currently when hibernation is in progress, the controller checks if all
machines for the cluster are in desired state (stopped when hibernating
and running when running). If all machines are not in that state, the
controller returns and retries after some timeout.
This means that there is no information from the controller about the
progress made and machines in mis-matched state.
With this change, when machines are in mis-matched state, we update the
Hibernating condition to include a message listing those machines.
xref: https://issues.redhat.com/browse/HIVE-1564
/assign @joelddiaz
/cc @2uasimojo

get rid of the stopMachines func entirely, and use checkClusterStopped (possibly renamed? ensureClusterStopped?) instead, since it's now effectively a superset anyway

Yes, that's true. but i didn't want to change the code flow for this PR that makes sure condition messages are clearer. Removing that code will affect the unit tests in a way i was not comfortable handling here. maybe in a separate pr.

adjust the message to say something like, "Stopping cluster machines. The followng are not yet stopped: X,Y,Z"

the condition reason is already Resuming/Stopping and with that context, including "Stopping cluster machines" just seems more duplication imo. But i don't mind including that if that makes it easy for users.

Updates lgtm, thanks @abhinavdahiya.

the condition reason is already Resuming/Stopping and with that context, including "Stopping cluster machines" just seems more duplication imo.

(Not asking for changes, just trying to explain my thinking. Feel free to ignore.)
It's in the initial message. I was thinking about the flow where, as a user, I'm watching the Condition: If I see it transition from
Stopping / True / "Stopping cluster machines"
to
Stopping / True / "Some machines have not yet stopped: X,Y,Z"
...I might think there's a Problem.
Put another way, if we're worried about redundancy, perhaps the initial message shouldn't say anything :)
But this is probably just me still being uncomfortable with how Conditions are structured.

/lgtm

Great UX improvement, thanks!

/test e2e
infra timeout


fix: ignore unstash coverage errors

What does this pull request do?

It adds a couple of catchError steps to avoid failure when the stash does not exist
Related issues
closes https://github.com/elastic/apm-agent-python/issues/913

:green_heart: Build Succeeded

Expand to view the summary

Build stats

Build Cause: [Pull request #914 opened]

Start Time: 2020-09-02T11:13:03.976+0000

Duration: 28 min 2 sec

Test stats :test_tube:

Test
Results

Failed
0

Passed
10927

Skipped
8506

Total
19433

Steps errors
Expand to view the steps failures

Name: Shell Script

Description: ./tests/scripts/docker/run_tests.sh python-2.7 django-1.11

Duration: 4 min 46 sec

Start Time: 2020-09-02T11:19:00.617+0000

log

Name: Shell Script

Description: ./tests/scripts/docker/run_tests.sh pypy-2 none

Duration: 3 min 2 sec

Start Time: 2020-09-02T11:19:14.679+0000

log

Name: Shell Script

Description: ./tests/scripts/docker/run_tests.sh pypy-3 none

Duration: 3 min 12 sec

Start Time: 2020-09-02T11:19:05.667+0000

log

Name: Shell Script

Description: ./tests/scripts/docker/run_tests.sh pypy-3 mysqlclient-newest

Duration: 1 min 11 sec

Start Time: 2020-09-02T11:31:07.021+0000

log


Update contributing.md file

I know that the contributing.md file already does a great job listing this project's essence. I thought we could update it with steps on how to help new contributors do stuff like fork, clone, commit their changes, and push back to this repo.
Should in case this issue is approved, I would like to be assigned to it.

This is a great issue @Njong392! Would appreciate you working on this. Thanks!

Hey Emy! Could you put in there that the page will be constantly evolving and please don't get hurt if someone's contribution gets updated/modified/deleted at some point in the future? Maybe that if they really like some part they can publish their own fork somewhere?

Definitely! This is an important note :+1:


Functions for timestamps

Currently, when doing aggregations, timestamp of the resulting value is the first timestamp of the bucket.
But when datapoints are not made at regular intervals, then this does not provide necessary the best output, for example, when you want to visualize aggregated datapoints and you want a representative of a bucket. I would suggest that there is a way to specify a function when aggregating on how you want to aggregate the timestamps as well:

first: the first timestamp of all datapoints in the bucket
mean: the average of all timestamps in the bucket
last: the last timestamp of all datapoints in the bucket

I find this still relevant.

I find this still relevant.


Newtonsoft.Json.JsonException Unexpcted JSON token at start of NPM catalog data

My issue is that if I load https://registry.npmjs.org/-/all directly into my browser I get nothing.
Other URL's work such as https://registry.npmjs.org/-/all/static/all.json
So, as seen in the output below, it fails to update my catalog on first opening.
I am on VS 2015, using the latest build from 10/5/2016 for Node JS Tools.
Here is the output of the Install Output:
Registry url: https://registry.npmjs.org/
Your package catalog database needs to be upgraded. Please wait while we execute a full refresh.
Deleting C:\Users\s2cc\AppData\Local\Microsoft\Node.js Tools\NpmCache
Path to npm: C:\Program Files\nodejs\npm.cmd
Downloading package cache to C:\Users\s2cc\AppData\Local\Microsoft\Node.js Tools\NpmCache\all_packages.json
Downloading list of packages from https://registry.npmjs.org/-/all
Download complete - updating database
Reading 0 bytes from C:\Users\s2cc\AppData\Local\Microsoft\Node.js Tools\NpmCache\all_packages.json (10/6/2016 10:40:00 AM)
Newtonsoft.Json.JsonException: Unexpected JSON token at start of NPM catalog data
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.<>c__DisplayClass6_0.b__0()
at SQLite.SQLiteConnection.RunInTransaction(Action action)
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.ParseResultsAndAddToDatabase(TextReader reader, String dbFilename, String registryUrl)
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.<>c__DisplayClass23_2.b__2()
at System.Threading.Tasks.Task.InnerInvoke()
at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.d__23.MoveNext()
Download or parsing failed - deleting C:\Users\s2cc\AppData\Local\Microsoft\Node.js Tools\NpmCache. Try refreshing the catalog again. Otherwise restart Visual Studio, and try again.
Deleting 16d830f2-3508-4aa5-9f4b-1c2362a73887
Newtonsoft.Json.JsonException: Unexpected JSON token at start of NPM catalog data
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.<>c__DisplayClass6_0.b__0()
at SQLite.SQLiteConnection.RunInTransaction(Action action)
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.ParseResultsAndAddToDatabase(TextReader reader, String dbFilename, String registryUrl)
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.<>c__DisplayClass23_2.b__2()
at System.Threading.Tasks.Task.InnerInvoke()
at System.Threading.Tasks.Task.Execute()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
at Microsoft.NodejsTools.Npm.SPI.NpmGetCatalogCommand.d__23.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.NodejsTools.Npm.SPI.NpmCommander.NmpCommandRunner.d__2.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
at Microsoft.NodejsTools.Npm.SPI.NpmCommander.d__12.MoveNext()

So, I think I fixed it myself.
I used the commands below:
SET http_proxy=[MY PROXY]
SET https_proxy=[MY PROXY]
After doing this and deleting the NpmCache folder, the issue seems to have resolved itself.

There is more simple way: you need to add all required packages to 'package.json'.

Can you please explain how you fixed this?
I'm having exactly the same error now. I have tried all the possibilities but couldn't fix it.
Can you please help?

@bala41288 If you are using Visual Studio then you can create a package.json file like this
On Saving it should by default download the required packages that you have placed under dependencies object. If you find any issues, then go to the npm>"dependencyName" & npm>dev>"dependencies" and right click on them and click on "Install missing npm Package(s)". You can see in the below screenshot that i have "gulp" in red which means its successfully installed and "gulp-protractor" in yellow that means it's not installed.

Now, right click on the missing package (i.e., in the yellow) and select "Install missing npm Package(s)". You should now see that npm module is also installed and turned into red as shown in the below image


Checks on sanity of folds should be improved

No check is performed concerning empty folds during the training-validation-test split

The folders concerning validation and test for each split are now checked for emptiness and possibly fixed moving one group from training


Search-ADAccount.md - TimeSpan parameter - Undocumented Behaviour

Previously undocumented behaviour/interaction with ms-DS-Logon-Time-Sync-Interval
Usage of AccountInactive & TimeSpan has the undocumented behaviour of incorporating the ms-DS-Logon-Time-Sync-Interval  attribute value interval [seen with the default 14 days]
Likely would've similar impacts on DateTime parameter - but didn't've opportunity to confirm
PR Summary

PR Checklist

[x] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[x] Summary: This PR's summary describes the scope and intent of the change.
[x] Contributor's Guide: I have read the contributors guide.
[ ] Style: This PR adheres to the style guide.

@microsoft-github-policy-service agree


Problem when combine slidesPerView and CenteredSlides. First and last item can't be selected

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/timeitem-7fn8zp
Bug description
I combined several parameters in swiper, the most prominent are slidesPerView:3, centeredSlides:true and direction: "vertical", I'm trying to simulate a "native clock" from iOs to web. The thing is that although I add the property "centeredSlidesBound:true" what happens is that the first item and the last one of the array are visible but they can't be selected/activated.
Example:
  const swiperOptions = {
      height: 150,
      slidesPerView: 3,
      centeredSlides: true,
      spaceBetween: 0,
      centeredSlidesBounds: true,
      direction: 'vertical'
  }

<Swiper onSlideChange={handleSlideChange} {...swiperOptions}>
      {data.map((item, index) => (
        <SwiperSlide key={index}>
          <TimeItem time={item.time} isSelected={index === activeIndex} />
        </SwiperSlide>
      ))}
</Swiper>

Expected Behavior
it should be possible to activate the first and last items of the array.
Actual Behavior
The first and last items are visible, but cannot be activated.
Swiper version
9.4.1
Platform/Target and Browser Versions
macOs Chrome
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[X] I'm willing to open a PR

You need to disable centeredSlidesBounds, it won't let to set active slides on slides before /after slidesPerView/2


Worker-based containers not showing in dashboard or containers screen

Description
Worker-based  containers are not showing in dashboard, or containers tabs. The services tab shows the correct amount of services, as does the cluster visualisation tab.
Steps to reproduce the issue:

Start service
docker service create --name nginx --publish 80:80 --replicas 5 nginx
Navigate to portainer and view dashboard, container, services, and swarm visualisation to see where containers are shown / not shown.

Screenshot (Google drive) of issue
https://drive.google.com/open?id=16Mkb2-aERYVNy4OlEhq7QkDyPopsOxLU
Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
If it isn't a configuration issue, then I would expect a few fixes:

Dashboard has drop down to select between all nodes in a swarm cluster
Containers tab to show all containers regardless of location, including label of host name where containers reside.

Technical details:

Portainer version: 1.15.2
Target Docker version (the host/cluster you manage): Docker version 17.09.0-ce, build afdb6d4
Platform (windows/linux): ubuntu 16.04.3 LTS
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):

docker service create --name orchestration --replicas 1 --constraint 'node.role == manager' --network ingress --publish 443:9000 --mount type=bind,src=/var/run/docker.sock,dst=/var/run/docker.sock  --mount type=bind,src=/home/ubuntu/certs,dst=/certs portainer/portainer --ssl --sslcert /certs/portainer.crt --sslkey /certs/portainer.key -H unix:///var/run/docker.sock

Target Swarm version (if applicable):
Browser: Chrome Version 62.0.3202.94 (Official Build) (64-bit)

Yeah it's a known issue and a docker api limitation for the moment
It's like when you run docker ps on your manager node, you see only containers on that node
You can have a look at #461 for more details
For the moment you can't have a global view of all your containers, but you can add the worker endpoint to portainer, when switching to this endpoint, you will see worker containers

Thomas, is this something you can work on to create a workaround under our engagement model? If so, drop me an email...
Sent from my iPad
On 26/11/2017, at 7:04 AM, Thomas Krzero <notifications<EMAIL_ADDRESS>wrote:
Yeah it's a known issue and a docker api limitation for the moment
It's like when you run docker ps on your manager node, you see only containers on that node
You can have a look at #461https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_461&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=ZCIgjrZdTmDIXqKd4E5L5awE28r7nzmgrbYhp2zulgE&s=JX1V7RXPucMn4e1UG7G77X-xaNNMqMRrnIuCOMfYzbw&e= for more details
For the moment you can't have a global view of all your containers, but you can add the worker endpoint to portainer, when switching to this endpoint, you will see worker containers
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_portainer_portainer_issues_1442-23issuecomment-2D346974474&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=ZCIgjrZdTmDIXqKd4E5L5awE28r7nzmgrbYhp2zulgE&s=ngYTfKT5v0DOkHGky_f3P0Of7kAqZVW1QJoYPhJWOTg&e=, or mute the threadhttps://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AWGrlSYbj1w9snEewYljdSVJbOSGzPMyks5s6Kr9gaJpZM4Qqk82&d=DwMFaQ&c=euGZstcaTDllvimEN8b7jXrwqOf-v5A_CdpgnVfiiMM&r=0fx0h4vB56iTLpw2McH1ZD6TqG_QGpbggVOB-PfMJpM&m=ZCIgjrZdTmDIXqKd4E5L5awE28r7nzmgrbYhp2zulgE&s=R0z2qrbYmuGstZJLP-I3QuV_N8hh0z1hj7sJmI2CqsI&e=.

Indeed, duplicate of #461


[enhancement] Option to hide Art in browser/sheet

An option to hide the art of a creature/NPC in the browser and sheet.
My suggestion would be two or three options for hiding. completely no art visible, silhouette, or maybe a custom "unknown" image.
I can see this being useful for Creatures/NPCs the party has heard of, but not actually seen yet.

I pushed out v1.0.2 that adds Avatar Filters as a feature along with module settings to set the default filters desired for the different types of entries in the Bestiary. 🤖 ✌


Running any experiment

When I try to run any experiment from the downloaded code, I directly get the error:
AttributeError: 'CSkipLSTMCell' object has no attribute 'check_forward_input'
I looked into the custom_cells.py and other files (with the base class) can't see find where it is initialized in.

I used the master version of Pytorch, to develop this code. The function 'check_forward_input' from the class RNNCellBase, at the file torch/nn/modules/rnn.py is included at the 0.4 version of Pytorch. You can comment the lines at the code where 'check_forward_input' appears, if you want to make it work. They are only asserts to check if the dimensions of the Tensors are correct.


Typescript declarations missing

It seems that the typescript declarations have gone missing from ReSub.
I'm trying to use with ReactXP example and both the compiler and VSCode are complaining about missing definitions.
Version 1.0.1 gives this issue while version 1.0.0 doesn't.
Thanks for your work :+1:

I've submitted a PR that fixes this issue.
I guess that, if needed, discussion could continue there.

Interesting -- don't we need to fix the index.js path to the actual js files as well if this is the case?

Okay looked into this some more and had to fix several things.  1.0.2 published and 1.0.1 unpublished.  Thanks for letting us know so quickly!  Sorry about the messup!

Thanks!
I hadn't thought about this other file, my bad :-1:
I've updated to the new version and it's working fine :fire:


Some updates for the front-end and back-end

[x] Providers data getter function should be added ( getting plan info )
[ ] Provider login passphrase mechanism
[x] Data of total subscription and providers in the contract for showing
[x] Events should be added
[ ] Subscription renew functionality should be implemented
[ ]  Add tests for the codes of this issue

Data of total subscription and providers in the contract for showing

will be handled by events

renew test are added in this commit


The Pirate Bay failed with error 500

Error message: The server responded with a Cloudflare challenge, which is not supported by RSS-Bridge! If this error persists longer than a week, please consider opening an issue on GitHub!
Query string: action=display&bridge=ThePirateBay&q=vostfr&crit=search&cat=&format=Atom
Version: git.master.c4896c7
A possible fix may be to integrate this project https://github.com/KyranRana/cloudflare-bypass into rss-bridge.

Thanks for reporting this issue :+1:
Unfortunately TPB is offline. Try https://thepiratebay.org/

A possible fix may be to integrate this project https://github.com/KyranRana/cloudflare-bypass into rss-bridge.

This has been discussed in #725 where Cloudflare detection was added to RSS-Bridge. Due to legal reasons we won't accept PRs to bypass Cloudflare. That being said, your case is a false positive. The detection is very weak right now and can certainly be optimized in the future. The correct error message for your query should be something along the lines of HTTP/1.1 522 Origin Connection Time-out (this is the actual server response).
Here is what Cloudflare has to say about the error:

A 522 error occurs because CloudFlare could not make a TCP connection to your origin server before the attempts timed out. This means that CloudFlare was unable to send the HTTP request to the origin because a network connection to the origin server could not be established.
-- https://wordpresscheat.com/fixing-cloudflare-error-522/

If I remember correctly, TPB has been taken down for good and may only be accessible via the darknet, which obviously is out of scope of this project (you may install a proxy if that is what you want). Alternatives and more info here: https://gbhackers.com/the-pirate-bay-torrent-alternative-2018/
You can check its status here: http://www.isitdownrightnow.com/thepiratebay.org.html

The site has been unreachable for more than 40 days.

The same applies to KAT.

Oh, I see... So the only way to keep TPB bridge would be by using a web tor exit node (like tor2web or onion cab) that does not block TPB... (but then, we would be dependent of a third party)

Yes, that's assuming TPB is actually accessible via the darknet, which I haven't tested (and don't intend to do). There are also some news on the Internet about it being shutdown permanently due to financial problems:

The torrent site owner said: “The torrent site economy is in a bad state. Profits are very low. Profits are f***** compared to previous years.
-- https://www.express.co.uk/life-style/science-technology/981576/Pirate-Bay-shut-down-torrents-website-servers-down-outage

I guess people don't want to pay for a service that provides contents they didn't want to pay for in the first place... kind of ironic if you ask me :smile:
Seriously though, we have seen torrent sites shutting down one by one over the years, increasingly even with countries putting laws in place against such sites (especially those that primarily share contents illegally - like TPB and KAT). This is exactly why those sites moved into the darknet. Read this for KAT.
I'm thinking of removing both bridges in the next release. Of course, you can still maintain a copy on your own instance and make use of a proxy to access the sites (if they are still available that is). This is naturally highly discouraged - they went under the radar for reasons.
Maybe we need to add alternative sites, here are a few honorably mentions: https://fossbytes.com/best-legal-torrents-websites/

Confirmed, pushing changes.

Closing because PR was merged.


display link information in mobile view

Description
current behavior :

when in the mobile view all the information on the link disapears, like the expiration and creation date:

expected behavior :

users should be able to see details of the link by clicking on the dropdown then "info".

Screenshots
No response
Additional information
No response

That's awesome idea!
Go for it @ymoukhli 🚀

Hello @origranot,
I did add an "Info" button in mobile view only. Upon clicking on it, it will display all the missing information

should we stick to this, or is there any room for improvement.

Hello @origranot,
I did add an "Info" button in mobile view only. Upon clicking on it, it will display all the missing information

should we stick to this, or is there any room for improvement.

Hey @ymoukhli,
That's looks great, although I am not sure this is the correct way to do it UX wise.
Maybe we can create a toggle button (near the 3 dots) to show/hide information about the link?
I had an idea to show a modal with the relevant information but I think it's even worse UX.
What do you think?

sure thing, I ll prepare some examples.

Hey @ymoukhli, are you still working on that issue?

@origranot I am interested to work on this part, can you please assign to me ?

@origranot I am interested to work on this part, can you please assign to me ?

Awesome, I assigned you 😀

#809 Solved here


Change require name to xlsx

Coverage remained the same at 95.132% when pulling 60eef6b2d201d2fd26fe23529755676c96dc4875 on JulienKode:patch-1 into e167fec8fbd0de203b1e80c1e9d124629a2c5935 on SheetJS:master.

@JulienKode this module shouldn't be used.  The repo is available for historical reasons, but current and future projects should be using xlsx instead.  The repo is https://github.com/sheetjs/js-xlsx .  That being said, the module is still available on NPM and the README is technically correct.
Short summary: at the beginning the XLSX logic was separate from XLS thanks to licensing concerns -- XLSX is described in ISO/IEC 29500 and XLS is described in a series of technical specifications covered under the Microsoft Open Specifications Promise.  At the beginning, js-xlsx was MIT licensed and js-xls was Apache2.  Maintaining two separate stacks was unsustainable, especially with the glue code required for browsers to pick the correct parser for a file, so we coordinated with the external contributors to do a license shift and merged all of the code under the js-xlsx banner with an Apache2 license.

Sorry, I have mingled these two
Thank you for these explanation


[WFLY-7730] Fix of undertow/application-security-domain removing

https://issues.jboss.org/browse/WFLY-7730
Fix of type in RemoveHandler.

Can one of the admins verify this patch?

This is ok to test

You'll need to create a PR against the EAP 7.x branch that references the corresponding JBEAP issue to get it merged.

Will keep in wildfly-security-incubator/ladybird only for now to prevent future conflicts.


Color Thief v2.0 is inc

Sorry, should not have creeated this one,  probably pressed something extra. See this issue instead https://github.com/CrackerakiUA/ngImgCropFullExtended/issues/86


Fix secondary index issue: cannot find tuple after updating

This PR fixed the secondary index issue: cannot find tuple after updating, that is described in the comment of #384 (but #384 is still there).

Coverage decreased (-0.1%) to 54.777% when pulling 65674d7860f6c995f58dd652a0bf3a1887bc309e on Michael-Tieying-Zhang:master into 5ff5ee3d4f88f85160640ab38a338cbec3809d65 on cmu-db:master.

I don't understand what problem this is solving? Can you make a separate issue with a more complete example?

Pls see #390

@Michael-Tieying-Zhang The fix is good. Can you write a test case for that? You can use SQLTestsUtil to write a sql test. See how I wrote the sql test for min/max aggregation here:
https://github.com/cmu-db/peloton/tree/master/test/sql

Coverage increased (+0.1%) to 55.005% when pulling f93351a5584d646da7a4b163423ca5ecc7d5e1ce on Michael-Tieying-Zhang:master into 5ff5ee3d4f88f85160640ab38a338cbec3809d65 on cmu-db:master.

Coverage decreased (-0.04%) to 54.865% when pulling f93351a5584d646da7a4b163423ca5ecc7d5e1ce on Michael-Tieying-Zhang:master into 5ff5ee3d4f88f85160640ab38a338cbec3809d65 on cmu-db:master.

@malin1993ml Done


Step listAWSAccounts() truncates account list

Description
Step listAWSAccounts() truncates account list
Steps to Reproduce
Use listAWSAccounts() in a pipeline, where your AWS Organizations has more than 20 accounts.
withAWS(role: MBA.role, roleAccount: MBA.account) {
  accounts = listAWSAccounts()
}
println("Number of accounts: ${ accounts.size() }")
accounts.each {
  println("${it.status}\t${it.id}\t${it.name}")
}

Expected behavior: [What you expected to happen]
Should print number of actual accounts and list all of them.
Actual behavior: [What actually happened]
Number of accounts reported: 20.  Accounts listed: some 20 of them.
Appears plugin does not honor AWS API pagination (NextToken) and loop to retrieve all accounts, only the first set returned.   Plugin not usable for environment with more than 20 accounts.
Environment
Jenkins-Version: 2.141
Java-Version: OpenJDK 64-Bit Server VM 1.8.0_181
Plugin-Version: 1.31
Master/Slave Setup: Master only

This is correct. I will try to create a fix for this in the next days.

This is release with 1.32


onMobSpawn handling slow

Version 3.0 is lagging my server, spiking mspt pretty high and sometimes even up to 20000 stopping my server.
https://spark.lucko.me/p9JQ8mbCCr
I'll try to add more timings with higher mspt spikes if I can catch them.

I'm pretty sure there is a memory leak somewhere there, since it's also messing with my GC, reverted to 2.6.1 and all issues were gone.

Not only that, but the problem also caused the server creatures to be unable to reproduce.
Sheep can't have children, for example
You can't break a chicken with an egg
不仅如此，这个问题也造成了服务器生物无法生育的问题。
比如说羊无法生子
鸡蛋砸不出鸡
QAQ


[HUDI-6055] Fix input format for bootstrap tables

Change Logs
Push down instantiation of table schema resolver to the point when it's actually needed. This helps in avoiding some cycles reading commit metadata unless needed.
Impact
None.
Risk level (write none, low medium or high below)
low
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@yihua @xiarixiaoyao thanks for reviewing. As mentioned in the comments, the unnecessary instantiation is already removed and we don't need lazy listing in Hive file index impl for now. So, I am going to close the PR.


Patch neither glibc nor glibc@* using patchelf

Patching the dynamic linker of Glibc breaks it.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

See PR

https://github.com/Homebrew/homebrew-core/pull/92329

Review period will end on 2022-01-04 at 00:00:00 UTC.

Thanks @sjackman!

Review period skipped due to critical label.


Scope status widget (sidebar) – CSS issue

The widget appears when we use scopes for index page:

Original AA widget:

P. S. Thanks for your GREAT job! I love your theme!

#15 tested locally. Looks like it is:
:not(#search-status-_sidebar_section) {
      .panel_contents {
        padding: 0;

        > div {
          padding: 15px;
        }
      }
    }

Fixed. eef40f622791b55279ff8c1d8e6f9f7527c65747


NTR: EGAD pathway

New term: EGAD pathway
Definition: The protein catabolic pathway which selectively extracts ER-resident membrane proteins exported to the Golgi and endosomes for degradation by cytosolic proteasomes. It begins with phosphorylation of the ER-resident membrane protein, which triggers export of the protein from the ER to the Golgi and endosomes, followed by polyubiquitination by the Dsc E3 ubiquitin ligase complex and extraction of the ubiquitinated target, and ends with proteasomal degradation.
reference: https://pubmed.ncbi.nlm.nih.gov/31368600/
placement: is_a child of GO:0010498 proteasomal protein catabolic process

+[Term]
+id: GO:0140624
+name: EGAD pathway
+namespace: biological_process
+def: "The protein catabolic pathway which selectively extracts ER-resident membrane proteins exported to the Golgi and endosomes for degradation by cytosolic proteasomes. It begins with phosphorylation of the ER-resident membrane protein, which triggers export of the protein from the ER to the Golgi and endosomes, followed by polyubiquitination by the Dsc E3 ubiquitin ligase complex and extraction of the ubiquitinated target, and ends with proteasomal degradation." [PMID:31368600]
+is_a: GO:0010498 ! proteasomal protein catabolic process
+property_value: term_tracker_item https://github.com/geneontology/go-ontology/issues/21085 xsd:anyURI
+created_by: pg
+creation_date: 2021-03-12T10:41:58Z
+


Hide part of the message

We are working with a product that places rather large statements in the log file. In most cases I am only interested in the second part of the message. It would be great if we could hide a part of the debug statements.
For example:
2015-11-21 01:48:34.411 DEBUG        c.a.i.portal.model.tx.FlowContext SessionId="" userId="" projectName="" projectVersion="" currentPageName="" - [pushActiveInstance] Activating instance for flow: NameOfTheFlow, instance: [TmsInstance instance=[TxEntityInstance [profile:EntityInstance (null,8df6eebe-fb03-4518-ac24-184b1d18650e,EntityName)]]]
In this case I would like to hide: SessionId, ProjectName, ProjectVersion, UserId -> I would use these to filter my log, they are not relevant to debugging but they do take op a lot of space in my window.

@djeedjay : we could add a 'hide' filter to hide part of the message...
for 'wildcard/regex' it could hide everything it matches, in without regex-groups show only what is in groups?


Can't use float32

type Company struct {
	ent.Schema
}

func (Company) Fields() []ent.Field {
	return []ent.Field{field.String("name"),
		field.String("address"),
		field.String("address2"),
		field.String("city"),
		field.String("country"),
		field.Float32("account"),
		field.Float32("account2"),
		field.Bool("is_enabled"),
		field.Time("created_at").
			Default(time.Now),
		field.Time("updated_at").
			Default(time.Now),
	}
}

// Edges of the Company.
func (Company) Edges() []ent.Edge {
	return nil
}

produces:
value.Float32 undefined (type *"database/sql".NullFloat64 has no field or method Float32)

I feel that there has a bug in entc/gen/template/dialect/sql/decode.tmpl  line 102, because  package database/sql/sql.go dose not have sql.NullFloat32, so we can't use value.Float32 to generate code, should use value.Float64.
I don't know if what I said is right. If there is any mistake, please correct it.

type Company struct {
	ent.Schema
}

func (Company) Fields() []ent.Field {
	return []ent.Field{field.String("name"),
		field.String("address"),
		field.String("address2"),
		field.String("city"),
		field.String("country"),
		field.Float32("account"),
		field.Float32("account2"),
		field.Bool("is_enabled"),
		field.Time("created_at").
			Default(time.Now),
		field.Time("updated_at").
			Default(time.Now),
	}
}

// Edges of the Company.
func (Company) Edges() []ent.Edge {
	return nil
}

produces:
value.Float32 undefined (type *"database/sql".NullFloat64 has no field or method Float32)

U can modify float32(value.Float32) to float32(value.Float64) in project/ent/company.go to solve this problem temporarily.

Thanks for reporting this issue @kryptodirect. #381 should solve this.


regexp search / replace would be useful

just wrap around regexp -- add a flag -- shouldn't be hard.

regexp would be nice.

done


Add support for add or remove device from collection

#19
Ready for Review

@sarahhenkens can you please review when you have a moment?

Looks good, ready to merge

Looks good, ready to merge 👍


Added function comments to the CSP Queue function csp_queue_create_static

The function csp_queue_create_static does not appear on the libcsp docs, so I added comments to the function

Thank you for your feedback, I have made the requested changes

Thank you for your feedback, I have made the requested changes

You should avoid using revert commit 😅
First, drop your last commit (the revert one):
git reset --hard 7590e10e072f376b00e14c94af1d40b5933defd3
Then, modify the function parameter back:
git add include/csp/arch/csp_queue.h
Finally, amend and push:
git commit --amend
git push --force
This way, you'll keep only one commit.

Should I have used git rebase -i instead?

You can squash them and keep the commit message of 7590e10e072f376b00e14c94af1d40b5933defd3

I fixed it, thank you


Update drawer font size

Fixes Themes 71 .

Changes proposed in this Pull Request:

Update drawer font size to 14px

Looks like one read me wasn't updated for some reason. Hero read me.

also this warning on render, not sure if this is only in this branch.
vendors~main.e5b9f99a0ef1b19485b2.bundle.js:128286 Warning: Cannot update during an existing state transition (such as within render). Render methods should be a pure function of props and state.
in Unknown (created by Context.Consumer)
in WithTheme(Component)
in WithTheme(Component) (created by Preview)
in div (created by Context.Consumer)
in Side (created by FlexBar)
in div (created by Context.Consumer)
in Styled(div) (created by FlexBar)
in div (created by SimpleBar)
in div (created by SimpleBar)
in div (created by SimpleBar)
in div (created by SimpleBar)
in div (created by SimpleBar)
in div (created by SimpleBar)
in SimpleBar
in Unknown (created by Context.Consumer)
in Styled(Component) (created by ScrollArea)
in ScrollArea
in Unknown (created by Context.Consumer)
in Bar (created by FlexBar)
in div (created by FlexBar)
in FlexBar
in Unknown (created by Context.Consumer)

@jeffvg I think that's something coming from the storybook internals...I've seen it before. Let's log it as a separate issue and investigate it unless it's caused by the changes in this PR


docs(website): correct link to faucet

thanks for the contribution @ameneth1 , i just updated the scope of the title to docs(website) otherwise looks great


std::cout flags not resetted before doctest output

Description
Not sure if this counts as a bug, but it took me a while to figure out what's going on. When modifying std::cout with e.g. std::cout << std::hex << std::setfill('0'), the testcase printer also writes in that format.
E.g. this example:
#define DOCTEST_CONFIG_IMPLEMENT_WITH_MAIN
#include <doctest.h>
#include <iostream>
TEST_CASE("a") {
    for (size_t i = 0; i < 1234; ++i) {
        REQUIRE(i >= 0);
    }
    std::cout << std::hex << std::setfill('0');
}

Running ./app prints
[doctest] doctest version is "2.3.1"
[doctest] run with "--help" for options
===============================================================================
[doctest] test cases: 000001 | 000001 passed | 000000 failed | 000000 skipped
[doctest] assertions: 0004d2 | 0004d2 passed | 000000 failed |
[doctest] Status: SUCCESS!

So all the numbers are printed in hex and with leading 0.
Boost has a handy class ios_base_all_saver (from here):
class ios_base_all_saver
{
public:
    typedef ::std::ios_base  state_type;

    explicit  ios_base_all_saver( state_type &s )
        : s_save_( s ), a1_save_( s.flags() ), a2_save_( s.precision() )
        , a3_save_( s.width() )
        {}

    ~ios_base_all_saver()
        { this->restore(); }

    void  restore()
    {
        s_save_.width( a3_save_ );
        s_save_.precision( a2_save_ );
        s_save_.flags( a1_save_ );
    }

private:
    state_type &                s_save_;
    state_type::fmtflags const  a1_save_;
    ::std::streamsize const     a2_save_;
    ::std::streamsize const     a3_save_;

    ios_base_all_saver& operator=(const ios_base_all_saver&);
};

Instantiating something like this at startup and calling restore() before printing anything would do the trick

doctest version: 2.3.1
Operating System: Linux
Compiler+version: g++ 8.2.0

Good find. I cannot guarantee that I'll evaluate this soon and it might not go into the next version - I'm buried in work. At least I fixed the other bug since it's more serious. But thanks for reporting!

No problem, it's easy to work around

This is not a bug. It's important - because doctest does not provide a facility for setting those flags. What if I want the printing to be in hex?

Yep, this should be considered a feature, not a bug. If you are testing code that messes with std::cout you can simply redirect doctest output via Context::setCout.

@Saalvage : Actually, on second thought - while it may be useful for the test printer to be configurable, it should not be as an accidental side-effect of the code before it's invoked. It would make more sense for there to be ios flags for the printer which one can set explicitly. So perhaps this should be reopened.


metadata tags added for sharing on Facebook

Pre-Flight checklist

[ ] Screenshots and design review for any changes that affect layout or styling

[ ] Desktop screenshots
[ ] Mobile width screenshots

[ ] Testing

[ ] Code is tested
[ ] Changes have been manually tested

What are the relevant tickets?

Closes https://github.com/mitodl/ocw-hugo-themes/issues/464

What's this PR do?

Adds metadata tags for sharing on facebook

How should this be manually tested?

Copy the URL of course deployed on netlify on this PR. (either course or course-v2)
Test the page on Facebook sharing debugger
Repeat the above steps with multiple/different course pages
OR:
Checkout this branch
Build any course
View page source
Verify that all the Facebook Open Graph tags are present in the head with correct values/content.

Screenshots (if appropriate)
Shared on Facebook (view mode is only me so this is not visible to public)

@pdpinch Do we also need to use facebook insights?
If yes, then we'll be needing our app_id.

doc

@pdpinch Do we also need to use facebook insights?

I don't think so. OCW doesn't have an app_id.

og:image property doesn't seem to have the correct image URL for courses.
I'll open up a follow-up PR for its fix.


Fix anchor linking

With the site menu, anchor linking works but hides the title:

With this change it should show the title of the section:

THIS IS UNTESTED and unfortunately I did the commit on top of the other PR - so for this one, only last commit matters.

@OXINARF It looks very sensible :-) I haven't got time to test this until probably the weekend, but I'd be happy for this to go live based on your images and the implementation.

... and I haven't looked at the other changes!

@hamishwillee I think there's no rush, testing it first is a good idea. I also only tested in Chrome and IE11 using the developer tools - we might want to see if it works at least in Firefox too.

Superseded by https://github.com/ArduPilot/ardupilot_wiki/pull/1745


New RWpod logo is broken in Firefox

Open rwpod.com in firefox and check the logo.
My user agent string: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:54.0) Gecko/20100101 Firefox/54.0

Thanks, fixed. Looks like https://github.com/svg/svgo "optimize" something in svg, which firefox cannot handle.


[Search v2.4] [Auth] Reduce the limit of SearchForReports

Implement this section of the doc

@luacmartins do you need me to add a C+ or is this going to be handled by eng only?

This is internal work only, so you can unassign yourself form this issue if you'd like

Got a draft PR up. This is still on hold for https://github.com/Expensify/App/issues/49122

just a heads up that I'm going to be ooo until 9/30. I'm not going to assign a BZ teammate since we're working on the PR/discussing the issue. If you need someone before Monday please reach out in Slack for a volunteer

I'm back from ooo - looks like we're still on hold for https://github.com/Expensify/App/issues/49122 so gonna keep this set to weekly

Hold for - https://github.com/Expensify/App/issues/49122

Ah sorry I just saw your message that I can unassign myself so I'll do that. Feel free to add me if you need anything related to BZ team - @luacmartins

Still pending deprecating Chat Finder

App PR is merged. We just need to wait on deploy to prod and we can continue work here

This is done.