Support exposing the Audience of a Broker

Fixes #3520

Proposed Changes

Support exposing the Audience of a Broker based on flag IsOIDCAuthentication

/cc @creydr @Leo6Leo

Thanks for the PR @gunishmatta, please check linter errors https://github.com/knative-extensions/eventing-kafka-broker/actions/runs/7521296763/job/20481144990?pr=3600

@pierDipi  I have fixed the formatter, can you please approve the workflow run?

/ok-to-test

@gunishmatta Thanks for the PR! As we can see from the error log of the failing tests, we can see there problems with your code. Hope this information give you some hint!
containerID: containerd://783e4cd89babfca38b1dec1b2fafc1e65d30ccbe2ca0c444a3f64891c2d6bee3
      image: sha256:47ff6e6f1028017bd4e6fefb9b5046f630f6913d16a7750dbd91da68804e6bb7
      imageID: gcr.io/knative-boskos-80/eventing-kafka-broker-e2e-img/7587/kafka-controller-409c76cb00d7d7c46941c7e4c366e4a6@sha256:cfb93cd819ad68831e609a727e848822f10413ed458f53e58ef473c5e19b215d
      lastState:
        terminated:
          containerID: containerd://783e4cd89babfca38b1dec1b2fafc1e65d30ccbe2ca0c444a3f64891c2d6bee3
          exitCode: 2
          finishedAt: "2024-01-15T12:39:41Z"
          message: "cileKind(0xc00007d4a0, {0x2591bc8, 0xc001805620}, 0xc000fbaa80)\n\tknative.dev/eventing-kafka-broker/control-plane/pkg/reconciler/broker/broker.go:271
            +0x2836\nknative.dev/eventing-kafka-broker/control-plane/pkg/reconciler/broker.(*Reconciler).ReconcileKind.func1()\n\tknative.dev/eventing-kafka-broker/control-plane/pkg/reconciler/broker/broker.go:90
            +0x25\nk8s.io/client-go/util/retry.OnError.func1()\n\tk8s.io/client-go@v0.28.5/util/retry/util.go:51
            +0x30\nk8s.io/apimachinery/pkg/util/wait.runConditionWithCrashProtection(0x2202727?)\n\tk8s.io/apimachinery@v0.28.5/pkg/util/wait/wait.go:145
            +0x3e\nk8s.io/apimachinery/pkg/util/wait.ExponentialBackoff({0x989680,
            0x4014000000000000, 0x3fb999999999999a, 0x4, 0x0}, 0xc000fbab98?)\n\tk8s.io/apimachinery@v0.28.5/pkg/util/wait/backoff.go:461
            +0x5a\nk8s.io/client-go/util/retry.OnError({0x989680, 0x4014000000000000,
            0x3fb999999999999a, 0x4, 0x0}, 0x2202727?, 0x5?)\n\tk8s.io/client-go@v0.28.5/util/retry/util.go:50
            +0xa5\nk8s.io/client-go/util/retry.RetryOnConflict(...)\n\tk8s.io/client-go@v0.28.5/util/retry/util.go:104\nknative.dev/eventing-kafka-broker/control-plane/pkg/reconciler/broker.(*Reconciler).ReconcileKind(0x6?,
            {0x2591bc8?, 0xc001805620?}, 0xc000fbab98?)\n\tknative.dev/eventing-kafka-broker/control-plane/pkg/reconciler/broker/broker.go:89
            +0xb5\nknative.dev/eventing/pkg/client/injection/reconciler/eventing/v1/broker.(*reconcilerImpl).Reconcile(0xc0002654a0,
            {0x2591bc8, 0xc0018055c0}, {0xc0016cbf60, 0xd})\n\tknative.dev/eventing@v0.39.1-0.20240111175334-3cbddd640f53/pkg/client/injection/reconciler/eventing/v1/broker/reconciler.go:251
            +0x5f9\nknative.dev/pkg/controller.(*Impl).processNextWorkItem(0xc0005e8a20)\n\tknative.dev/pkg@v0.0.0-20240111013350-bc230ae58d14/controller/controller.go:542
            +0x4ad\nknative.dev/pkg/controller.(*Impl).RunContext.func3()\n\tknative.dev/pkg@v0.0.0-20240111013350-bc230ae58d14/controller/controller.go:491
            +0x59\ncreated by knative.dev/pkg/controller.(*Impl).RunContext in goroutine
            365\n\tknative.dev/pkg@v0.0.0-20240111013350-bc230ae58d14/controller/controller.go:489
            +0x349\n"
          reason: Error
          startedAt: "2024-01-15T12:39:09Z"

Hello @gunishmatta,
can we help you somehow on this?

@creydr  sorry was busy with work during the week, will debug it today

/retest

@creydr  can you please approve workflow, have added tests and fixed issues locally.

/retest-required

/retest-required

Hi Team @pierDipi  @creydr , Please give a final review to the PR and provide the feedback.
Thanks a lot
Looking forward to contribute more to knative community.

Thanks for the feedback, I will be working on the feedback, you can expect an update within a few days.

Hey @gunishmatta,
can I help you somehow on this?
I am working on exposing the audience from a KafkaChannel in #3622. Maybe this helps you a bit.

HI @creydr very sorry, was occupied with work commitments last week, I will try to get out time and update the PR before this weekend.
Thanks for the support.

@gunishmatta Hey Gunish, are you being blocked by any issue? anything we could help you with on this PR?

@gunishmatta Since it seemed you were occupied with other work and there was not much left to do, I added the missing bits from my comment in 6d9b33d839855e1653c839bc809f7a43a5acc575 (and run gofmt & goimports 5f7bd8bee9537a496a3d2a11f7db7a11daf5d8b6). I hope that's ok.
@Leo6Leo looks OK from my side. Can you recheck (as I committed to this PR now too :smile: )
/assign @Leo6Leo

/test channel-reconciler-tests-ssl

/test reconciler-tests-namespaced-broker

@Leo6Leo can you PTAL

/lgtm

/test reconciler-tests

/retest-required

/test reconciler-tests-namespaced-broker


Auto add project reference based on path

This tool seems like its perfect for exploratory testing of projects or doing quick experiments to test out an in development feature.
To smooth over that use case, I think it would be valuable to automatically (or with a magic command) add a project reference to whatever project is in the current path. An initial version of this could be to just do a quick check on startup if there is a csproj file in the current directory and just reference it.
Some possible issues:

Maybe the project would have to be built first? I don't know how this system handles referencing csproj files.
The .net version likely has to match up. This could be fixed by just trying and silently failing (or with a quick message explaining the problem)

This capability is something we're looking at enabling for .NET Interactive generally (including for notebooks): https://github.com/dotnet/interactive/issues/890.
If the scenarios you have in mind aren't captured there, please add your thoughts.


Switch to pip-compile and update requirements

This PR creates a new requirements.in file which can be used with pip-compile to create a new requirements.txt file.
It updates all the packages to their current version and also removes manually including dataclasses and typing_extensions as they're already included in current Python versions.

Are there any objections to this PR?


No default secret key in Workflow

Verifying the project includes trying to package and sign the project, therefore the Key for signing needs to be imported into the workflow

Siging the files is not really necessary for checking the integrity of the project.


BUG: scipy.ndimage.zoom produce wrong output with zoom factor of 1

Describe your issue.
Hey,
I am using scipy.ndimage.zoom for 3d images. Based on the input parameters, I might need to zoom or not. So, I expect the function to act as an identity function if the zoom factor is one. You can see the code and the output.

Reproducing Code Example
import numpy as np
from scipy.ndimage import zoom
import matplotlib.pyplot as plt

size = 100
mask = np.zeros((100, 100, 100))
center = np.multiply(size,(0.5,0.5,0.5))
print(center)
radius = size/8
for x in range(size):
    for y in range(size):
        for z in range(size):
            if (x - center[0]) ** 2 + (y - center[1]) ** 2 + (z - center[2]) ** 2 < radius ** 2:
                mask[x, y, z] = 1

zoomed_mask = zoom(mask, (1,1,1))

# Plot the original and zoomed masks in 3d
fig = plt.figure()
ax = fig.add_subplot(121, projection='3d')
ax.voxels(mask, edgecolor='k')
ax.set_title('Original Mask')
ax = fig.add_subplot(122, projection='3d')
ax.voxels(zoomed_mask, edgecolor='k')
ax.set_title('Zoomed Mask')
plt.savefig('mask.png')

Error message
There is no error message. See the output image.

SciPy/NumPy/Python version and system information
Build Dependencies:
  blas:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT=0 DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= ZEN MAX_THREADS=64
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.27
  lapack:
    detection method: pkgconfig
    found: true
    include directory: /usr/local/include
    lib directory: /usr/local/lib
    name: openblas
    openblas configuration: USE_64BITINT=0 DYNAMIC_ARCH=1 DYNAMIC_OLDER= NO_CBLAS=
      NO_LAPACK= NO_LAPACKE= NO_AFFINITY=1 USE_OPENMP= ZEN MAX_THREADS=64
    pc file directory: /usr/local/lib/pkgconfig
    version: 0.3.27
  pybind11:
    detection method: config-tool
    include directory: unknown
    name: pybind11
    version: 2.12.0
Compilers:
  c:
    commands: cc
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  c++:
    commands: c++
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  cython:
    commands: cython
    linker: cython
    name: cython
    version: 3.0.10
  fortran:
    commands: gfortran
    linker: ld.bfd
    name: gcc
    version: 10.2.1
  pythran:
    include directory: ../../tmp/pip-build-env-mnl4e8vy/overlay/lib/python3.10/site-packages/pythran
    version: 0.15.0
Machine Information:
  build:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
  cross-compiled: false
  host:
    cpu: x86_64
    endian: little
    family: x86_64
    system: linux
Python Information:
  path: /opt/python/cp310-cp310/bin/python
  version: '3.10'

IIUC the problem is that zoom with a factor of 1 adds numerical noise.
@grlee77 @stefanv @lagru Is it important for zoom with a factor of 1 return the input unmodified, or is a small amount of noise acceptable?


Fix changelog release dates

I hereby submit my entry for commit message of the year :smile:

haha, thanks!


api.helix.channels.startChannelCommercial should be a POST and the method is not defined in the request

Hey there-
The helix API startChannelCommercial won't work because the method is defaulting to GET. It [should be a POST].(https://dev.twitch.tv/docs/api/reference#start-commercial). I've also temporarily modified my local copy of the library to include the method parameter in this call.
As seen here: https://github.com/d-fischer/twitch/blob/7721e45cfce2498dc93ac5b914a25c6e9bbf261e/packages/twitch/src/API/Helix/Channel/HelixChannelApi.ts#L90
At the time of issue creation
async startChannelCommercial(broadcaster: UserIdResolvable, length: CommercialLength): Promise<void> {
		await this._client.callApi({
			type: TwitchApiCallType.Helix,
			url: 'channels/commercial',
			scope: 'channel:edit:commercial',
			jsonBody: {
				broadcaster_id: extractUserId(broadcaster),
				length: length
			}
		});
	}

We can easily resolve this by just including the method parameter as well as POST.
Fix
async startChannelCommercial(broadcaster: UserIdResolvable, length: CommercialLength): Promise<void> {
		await this._client.callApi({
			type: TwitchApiCallType.Helix,
			url: 'channels/commercial',
                        method: 'POST',
			scope: 'channel:edit:commercial',
			jsonBody: {
				broadcaster_id: extractUserId(broadcaster),
				length: length
			}
		});
	}

Fixed in 0fee74df but forgot the "closes" statement. Also released with 4.4.12. Thanks for reporting!


Linux Mint - Login screen is bugged

Question
Answer

Java Version
1.8.0_171

Operating System
Linux Mint 18.3 Sylvia

The login screen is bugged and doesn't show the username & password fields. See below for a screenshot.

https://github.com/runelite/runelite/wiki/Disable-Hardware-Acceleration


@overload autocompletion issue

---`SERVER/CLIENT`
---@param eventName string
---@param callback function
---@overload fun(self: Events, eventName: '"Level:Loaded"', callback: fun(levelName: string, gameMode: string))
---@overload fun(self: Events, eventName: '"Level:LoadingInfo"', callback: fun(screenInfo: string))
function Events:Subscribe(eventName, callback) end

When I type Events:Subscribe("Level:LoadingInfo", I want the only correct suggestion for that, which is  fun(screenInfo: string). Is there any way to archieve that? Just wanna know.

I have a similar issue, where when I select the overload I want, somehow the first one is selected. Even if it obvious, that the first one is not the overload I want (like when I'm typing an argument that should not exist in the current overload).
I would like to be able to select the overload and then it stays there until it's either obvious that its not the correct one anymore or when I select a different one.
Demo:

Have supported


Various patches + careers page remake

Apparently the patches were on a different PR (already merged) please ignore that

wait i forgot to put the deadline in mobile view


proxy configured as environment variable in the containers

From support email:
We see the below error even when use http proxy configured as environment variable in the containers. Can you advise?
Thanks
???
2018/09/26 12:22:50 [error] 7#7: *33 connect() failed (111: Connection refused) while connecting to upstream, client: <IP_ADDRESS>, server: , request: "GET /proxy/www.reactome.org/DiagramJs/diagram/diagram.nocache.js?v=3.8.2 HTTP/1.1", upstream:

More from the user on support:

Can you help me with this question from IT, on the NGINX configuration for “webapp”, I see “/proxy/” is configured to forward the requests to https://proxy.opentargets.io/ which is again a NGINX proxy hosted in front of the production API servers by OpenTargets team. In our case can this be configured to point to our local instances of “rest-api” application?
If you could get more info on what is the functionality of this particular redirection it would help us to configure a fully functional app."
Can you help @mkarmona @ElaineMcA ?

Known challenges with proxy for reactome, which will be addressed as part of FE POC rewrite.


adding hf_transfer

https://github.com/EleutherAI/lm-evaluation-harness/issues/1399
Should give a 2-5x download boost (given that you sit in a datacenter) https://github.com/huggingface/hf_transfer - I have seen no issues with lower bandwidth.
Should I add it as extra or in the main deps?

Thanks for the PR. I think this should be an optional dependancy.

@lintangsutawika Alright, that sounds like the way to go.

Done that!


Release version v0.9.0

What does this PR do?:

Release changes for AI Software Templates v0.9.0.
Which issue(s) this PR fixes:

https://issues.redhat.com/browse/DEVAI-185
PR acceptance criteria:
Testing and documentation do not need to be complete in order for this PR to be approved. We just need to ensure tracking issues are opened and linked to this PR, if they are not in the PR scope due to various constraints.

[ ] Tested and Verified

[X] Documentation (READMEs, Product Docs, Blogs, Education Modules, etc.)

How to test changes / Special notes to the reviewer:

Needs https://github.com/redhat-ai-dev/ai-lab-template/pull/44 to be merged before this PR can be opened.


Handle component names that contain other component names, causing some false matches.

https://github.com/supermedium/aframe-watcher/commit/a0718ad94e5f67dc37f83f30e7b4f7624245b8b7


How does vetur relate to VSCodeBeautify

Operating System (+Version): Windows 10 x64 1809
VS Code Version: 1.31.1
beautify Version: 1.4.9

NOTE: Please provide code snippets instead of screen shots. Your issue needs to be replicated, and we need the failing code (text) to do this.
Provide the settings you use:
(VS Code workspace and user settings, .jsbeautifyrc, .editorconfig)
eg.:
VS Code:
// A path to a file, or an object containing the configuration options for js-beautify. If the .jsbeautifyrc file exists in project root, it overrides this configuration.
  "beautify.config": null,

  // List of paths to ignore when using VS Code format command, including format on save. Uses glob pattern matching.
  "beautify.ignore": [],

  // Link file types to the beautifier type
  "beautify.language": {
    "js": {
      "type": [
        "javascript",
        "json",
        "jsonc"
      ],
      "filename": [
        ".jshintrc",
        ".jsbeautifyrc"
      ]
    },
    "css": [
      "css",
      "less",
      "scss"
    ],
    "html": [
      "htm",
      "html"
    ]
  }

"vetur.format.defaultFormatter.html": "js-beautify-html",
  "vetur.format.defaultFormatterOptions": {
    "js-beautify-html": {
      "wrap_attributes": "auto"
    },
    "prettyhtml": {
      "printWidth": 100,
      "singleQuote": false,
      "wrapAttributes": false,
      "sortAttributes": false
    }
  },

Action performed
Press the shortcut key to format automatically
Expected results
.vue file was formatted smoothly.
Actual results

The vetur extension has nothing to do with the beautify extension. It looks like it uses the same backend as we do here. To use beautify as your formatter for vue add "vue"to the "html"array under the "beautify.language" option. This will not pick up the settings in the "vetur.format.defaultFormatterOptions" as they're unrelated.

This question is void with the formatter selector capability was introduced into VS Code.


Add helpful information to the SETUP.md

The contents of the SETUP.md file gets included in
the README.md that gets delivered when a user runs the exercism fetch
command from their terminal.
At the very minimum, it should contain a link to the relevant
language-specific documentation on
help.exercism.io.
It would also be useful to explain in a generic way how to run the tests.
Remember that this file will be included with all the problems, so it gets
confusing if we refer to specific problems or files.
Some languages have very particular needs in terms of the solution: nested
directories, specific files, etc. If this is the case here, then it would be
useful to explain what is expected.

Thanks, @tejasbubane for suggesting that we add this documentation everywhere.
See exercism.io#2198.

I've added the requested comments to the SETUP.md file.
Unfortunately, testing this showed that the content was not included in README.md (deleted existing files and ran exercism fetch).

This is working for me. Steps to confirm:

Delete my hamming problem for this track.
Run exercism fetch
Open README.md; it has the setup contents right before the source.

I also got it if I fetched a particular problem, e.g. exercism fetch plsql raindrops.

Ah, works now - strange. When I first tried the above steps, only the Setup section was included. No idea why and how. Closing this issue, then.


⚠ AGEG website (prod) has degraded performance

In 2a6e559, AGEG website (prod) (https://www.ageg.ca/) experienced degraded performance:

HTTP code: 200
Response time: 4266 ms

Resolved: AGEG website (prod) performance has improved in d7b941f.


Handle editor variants inside EditorContainer & align editor event listener types

Description
The Sourcecast & Sourcereel workspaces make use of the SourcecastEditor instead of the Editor. To allow for this, both Workspace and MobileWorkspace take in an optional customEditor prop of type JSX.Element, which is rendered instead of the normal editor if the prop is present.
This is however quite problematic as the creation of SourcecastEditor is already done before the Workspace/MobileWorkspace component is called, as opposed to the Editor which is created in Workspace/MobileWorkspace. In order to support multiple editor tabs for both the Editor and the SourcecastEditor without having to duplicate similar logic, EditorContainer was updated to make use of a discriminated union as its prop. I initially tried playing around with generics to get something like this:
export type EditorContainerProps<T> = Omit<T, keyof EditorTabStateProps> & {
  renderEditor: (editorProps: T) => void;
  editorTabs: EditorTabStateProps[];
};
....
const EditorContainer = <T extends EditorTabStateProps>(props: EditorContainerProps<T>) => {
  const { editorVariant, editorTabs, ...editorProps } = props;
  // TODO: Implement editor tabs.
  const editorTabProps: T = {
    ...editorProps,
    ...editorTabs[0]
  };
  return renderEditor(editorTabProps);

Unfortunately, TypeScript is unable to infer the type of editorTabProps correctly here unless I perform type casts (which are evil). On top of that, using generics results in all of the parent states also becoming generic, which is rather ugly as even the top-level WorkspaceState requires a type variable to represent the editor variant.
By making use of discriminated unions, we can remove the customEditor prop in Workspace and MobileWorkspace. Regardless of which editor variant should be used, EditorContainer will be rendered. This allows the future multiple editor tabs logic to work for both kinds of editors, as well as makes the handling of both editor variants consistent.
Additionally, I also aligned the event listener prop type signatures between SourcecastEditor and Editor so that enableMobileKeyboardForEditor can be used for both editor variants in MobileWorkspace.
Part of the refactoring for #2176.
Type of change

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

How to test
There should be no change in behaviour.

Since there is no change in behaviour, I have targeted the master branch for this PR. This is so as to reduce the probability of merge conflicts as much as possible.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 23 (8.7%)  changed or added relevant lines in 8 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 32.933%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/commons/workspace/Workspace.tsx
0
1
0.0%

src/pages/academy/sourcereel/Sourcereel.tsx
0
1
0.0%

src/pages/githubAssessments/GitHubAssessmentWorkspace.tsx
0
1
0.0%

src/pages/playground/Playground.tsx
0
1
0.0%

src/pages/sourcecast/Sourcecast.tsx
0
1
0.0%

src/commons/sourceRecorder/SourceRecorderEditor.tsx
0
2
0.0%

src/commons/mobileWorkspace/MobileWorkspace.tsx
0
4
0.0%

src/commons/editor/EditorContainer.tsx
2
12
16.67%

Files with Coverage Reduction
New Missed Lines
%

src/commons/mobileWorkspace/MobileWorkspace.tsx
1
0.8%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
4359

Relevant Lines:
12309

💛  - Coveralls


[Improve] Optimize the HeartbeatMsg class

Description
In the previous improvement[1], it was mentioned that for the DataProxy node, the reported heartbeat information contains multiple sources, each source has a separate {ip, port, protocolType} information pair, and other information is the same, so the report information of DataProxy needs to be carried in a List for this part, instead of a comma-separated field string, which is not easy to expand.

https://github.com/apache/inlong/issues/6234

InLong Component
InLong DataProxy
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Since the implementation of https://github.com/apache/inlong/pull/9512 has been completed, modifications to this issue will now begin.

finished


[WIP] Add a BackgroundCorrector class

This PR aims to add a BackgroundCorrector class which provides a simple way to correct a light curve by utilizing the pixel time series data contained within the target's own Target Pixel File.  Specifically, this corrector is intended to be used on TESS target pixel files provided by TESSCut.
BackgroundCorrector builds upon RegressionCorrector by correlating the light curve against a design matrix composed of the following elements:

A background light curve to capture the dominant scattered light systematics.
Background-corrected pixel time series to capture any residual systematics.
Splines to capture the target's intrinsic variability.

All credit for developing this method belongs to @christinahedges!
Example use:
import lightkurve as lk
corrected_lc = lk.BackgroundCorrector(tpf).correct()

HI @barentsen! I Think this is a very good idea. Presumably, this is coding up a technique like what @christinahedges used in her RegressionCorrector demo to generate the design matrix. That methods looks to work quite well.
In wonder if what we really want is instead of "pixelcorrector", "cbvcorrector", & etc... we want "pixelDesignMatrix" and "cbvDesignMatrix" classes which generate a design matrix and independently of the method use to fit the design matrix to the data. We then use the RegressionCorrector class to apply one of many potential fitting techniques.
I am curious how you intend to use the splines. Would a Gaussian Process work better? Also, chance correlations between a stellar signal and systematics makes this tricky in some situations. Looking forward to see what you come up with!

@jcsmithhere Yes!  It definitely makes sense to have a nice library of re-usable functions or classes that create various flavors of design matrices.
We should brainstorm about the API some more this week!  My thinking has been that it makes sense to subclass Corrector for different methods, because it's a convenient way to encapsulate custom-tailored diagnose() methods.  For example, PixelCorrector could have a diagnose_pixels() method to elucidate which pixels in a TPF contributed most strongly to the noise model, and e.g. CBVCorrector could have a diagnose_cbvs() method visualizing the contribution of individual CBVs.
The PixelCorrector idea is definitely not fool-proof against chance correlations between astrophysics and systematics.  I'd love to hear your thoughts on fighting that!

This current WIP does not yet include any priors (i.e. regularization) on the regressors.
Here's my plan: if the target aperture mask includes 5 pixels, and the background mask selects e.g. 100 background pixels, I find that setting prior_mu equal to 5/100 for the background pixel time series appears to limit overfitting very well without needing PCA.  This of course makes sense because the scattered light is additive and roughly equal in each pixel.
@christinahedges Does that make sense to you? Can we think of a similarly-principled way to determine prior_sigma?

My thinking has been that it makes sense to subclass Corrector for different methods, because it's a convenient way to encapsulate custom-tailored diagnose() methods.

I agree with this, custom Correctors are great. In the future we should worry about having too many of them, but I don't think we're there yet. As both @barentsen and @jcsmithhere have said, we need design matrix generators e.g. lk.design_matrix.get_pixel_dm. I'm waiting to do these until we have the CBVCorrector stuff working, because we need some version of lk.design_matrix.get_cbv_dm.

Here's my plan: if the target aperture mask includes 5 pixels, and the background mask selects e.g. 100 background pixels, I find that setting prior_mu equal to 5/100 for the background pixel time series appears to limit overfitting very well without needing PCA. This of course makes sense because the scattered light is additive and roughly equal in each pixel.

I like this logic, can you make a MWE to show that this works? If this is a Corrector class, you can set the priors in it because it auto-generates the DesignMatrix objects, so fixing these priors to something reasonable like this is ideal.

I am curious how you intend to use the splines. Would a Gaussian Process work better?

GPs would be better, we don't have the capability in lightkurve at the moment but I have some working examples of how we would add this. @barentsen and I intend to work on this in the near future.

Also, chance correlations between a stellar signal and systematics makes this tricky in some situations.

Chance correlations should be reflected in the uncertainties of the fit. There is a way in RegressionCorrector to propagate errors, and if the two signals are truly degenerate, this should show up in the spread of allowable models. This is a not thoroughly tested feature, and I'd like to generate a tutorial/notebook on it in the future.
@barentsen do you feel this is a generic "background" corrector, or is this specific to TESS FFIs. For example, how does this work on a rolling band target from Kepler/K2. I find BackgroundCorrector to be misleading, because the "Background-corrected pixel time series" are actually removing systematics that are not due to the background but due to other systematic effects (e.g. focus change, motion). I am more comfortable with ≌TESSFFICorrector, but I would like to know your thoughts on this.

The current implementation does not seem to work well with small masks (i.e. where the light curve systematics are dominated by spacecraft motion).  This makes me think we may want to add some simple centroid vectors?
For example, this produces a very poor light curve:
tpf = lk.search_tesscut("AC Pic", sector=13).download(cutout_size=(21, 21))
mask = tpf.create_threshold_mask(threshold=1000)
lk.BackgroundCorrector(tpf, aperture_mask=mask).correct().plot()


Format of Uploaded Files

Hi,
I couldn't find any docs anywhere telling what kind of file format cog (and Replicate API) expects to receive in the input of the prompt. The NodeJS example only includes ... and I don't if python adds any extra headers with open function.
const output = await replicate.run(
  "meronym/speaker-transcription:12483517b558629508e03bed77e9c46c6f6e5756715af2b3a4f741f70be45575",
  {
    input: {
      audio: "..."
    }
  }
);

output = replicate.run(
    "meronym/speaker-transcription:12483517b558629508e03bed77e9c46c6f6e5756715af2b3a4f741f70be45575",
    input={"audio": open("path/to/file", "rb")}
)

Could you help me figure out what's the expected way to upload files using this library?

I usually pop the web console on the UIs to see. It seems that the website has some sort of file upload functionality and running the prediction passes the uploaded file's URL to it. I'm doing images myself, but I'm guessing this is also the case for audio.

https://replicate.com/docs/reference/http#predictions.create:~:text=Files should be passed as data URLs or HTTPS URLs.
Ooh, data URLs.

Data URLs worked for my image case.
It's documented in the API, but it would be nice if the examples also reflected this for files.

For what it's worth, I've been banging my head against the wall for several hours trying to pass it file objects. This information should definitely be a lot more prominent. At very least, just put a dummy URL in the examples instead of an ellipsis.

@maticzav @nelsonjchen @jmcannon Thanks for your patience on this. I just opened https://github.com/replicate/replicate-javascript/pull/63, which adds an example of how to pass a file input. Please take a look and let me know if you have any questions.
As I mentioned in that PR, we may want to add an affordance in the library to simplify this process. If you have any ideas for what that should look like, I'd love to hear them. Thanks!

Looks great - I think I'd also make it clear that you can just pass it an http url, as well. In my case, that saved an extra step and was super convenient. Maybe it's obvious to others, but I've been working with lots of streams and blobs lately, so it wasn't immediately obvious to me that I could provide a remote uri.


messages are not transmitted from telegram to the game

Hello, I encountered a problem that messages from the telegram chat are not transferred to the game. But messages come from the telegram game.
Telegram Bridge v0.4.4
Current Purpur Version: 1.21.1-2291-bba87bf (MC: 1.21.1)*

You are 33 version(s) behind
Download the new version at: https://purpurmc.org/downloads
Previous: 1.21-2280-930003b (MC: 1.21)

Could you post the logs?

The plugin does not write logs, and there is silence in the server console when I send a message from a telegram

Are there any messages at startup?
Can you reproduce this with only tgbridge installed?

Yes, there are messages from the plugin that it has been launched successfully, there are no errors or warnings from it
I've now run some tests.

I launched a server on Windows 10 with one lpagin - It works.

I launched a server on Windows 10 with the entire set of plugins - it works

I launched a server with one plugin on Linux (where the plugin does not work) - it does not work

I launched a server with all the plugins on Linux (where the plugin does not work) - it does not work

windows:
version 21h2 (19044.2130 )
java version "21.0.2" 2024-01-16 LTS
Java(TM) SE Runtime Environment (build 21.0.2+13-LTS-58)
Java HotSpot(TM) 64-Bit Server VM (build 21.0.2+13-LTS-58, mixed mode, sharing)
Llinux server:
Ubuntu 22.04.4 LTS
openjdk version "21.0.4" 2024-07-16 LTS
OpenJDK Runtime Environment Temurin-21.0.4+7 (build 21.0.4+7-LTS)
OpenJDK 64-Bit Server VM Temurin-21.0.4+7 (build 21.0.4+7-LTS, mixed mode, sharing)

Works for me on Ubuntu 22.04.5, Temurin-21.0.4+7 and Purpur 1.21.1-2324-ver/1.21.1@e12a4de.
Did you use the same bot for both servers? Maybe you didn't disable group privacy

God, I'm sorry for wasting your time, the problem was on my side. The bot did not have access to messages.
Thank you for your assistance, it’s not every day that I set up telegram applications

No problem :)


Added API for setting cipher suites in TLSConfiguration

Motivation:
Addressing #207 for adding an API for setting a cipher suite while also preserving the string based declaration.
The NIOTLSCipher struct provides the common IANA cipher names instead while preserving the string based route.
Modifications:
Added NIOTLSCipher struct and added a constructor for client and server to set a new array of cipher in TLSConfiguration.
Added test cases in TLSConfigurationTest.
Result:
Additional API for #207.

Looks like I need to rebase here as there is a conflict and APIs that I am no longer aligning with.  I will rebase re-open this PR for additional feedback.

You can rebase and then force push to this branch to avoid needing to reopen the PR, if that's helpful


[FR] Configuration file

It will be very easy for regular user to have configuration file,
where can be set some basic settings.
ex.
web_port : 8080
dev : hiddev1

yes for sure. will add that in the next release. thanks!

btw, have you tried the app with your inverter? does it work? what brand/model inverter do you have?

Inverter is mapped to hiddev1
ups remap and take hiddev0
/dev/usb# ll
total 0
drwxr-xr-x  2 root root    100 юли 16 15:47 ./
drwxr-xr-x 24 root root   4440 юли 16 18:28 ../
crw-------  1 root root 180, 0 юли 16 15:47 hiddev0
crw-------  1 root root 180, 1 юли 11 07:50 hiddev1
lrwxrwxrwx  1 root root      7 юли 16 15:47 seups0 -> hiddev0

Inverter is EASun smv iii 5kva with have ton of clones, Voltronic protocol
Panels are in a shadow from few hours

v1.2 released with ability to specify web port & usb device path via appsettings.json file.
and also you can now customize sunlight period in settings tab.

v1.2 released with ability to specify web port & usb device path via appsettings.json file. and also you can now customize sunlight period in settings tab.

Update this info also in README.md


Prepare 0.25 release

Docs and version number updates for upcoming release.

Merging. But before we release i'll do a commit to remove some weight we are carrying and not using anymore. Let you know


allow ordered dict export to yaml

Currently, in utils.yamlio, dictionaries exported via convert_to_yaml use a hard-coded list to control the ordering of keys in the output file. This has two drawbacks (1) the list has to be updated when new parameters are added and (2) only the outer dictionaries ordering can be controlled, the ordering of inner dictionaries will not follow the order list.
This PR adds support for exporting ordered dictionaries to yaml in utils.yamlio.  For example, the following:
inner_dict = OrderedDict()
inner_dict['z'] = 'had'
inner_dict['a'] = 'a'
inner_dict['f'] = 'little'

outer_dict = OrderedDict()
outer_dict[10] = 'marry'
outer_dict['inner'] = inner_dict
outer_dict['a'] = 'lamb'

yamlio.convert_to_yaml(outer_dict, my_file.yaml)

Will have the following yaml:
10: marry

inner:
    z: had
    a: a
    f: little

a: lamb

Note that if a ordered dictionary is provided (as in the current example), its ordering will override the hard-code ordering the yamlio file.

Coverage decreased (-0.2%) to 94.127% when pulling ba03b34215fc0aee04f07c543c77da4542d4b4c9 on AZMAG:odict_to_yaml into a14b287b0d4835acdc460252bb06f545b458b4df on UDST:master.

Great idea Scott - I'll review this later today

Coverage decreased (-0.2%) to 94.129% when pulling aa9406fd8719e9b5edeaa016690f58a9c3b0b6f9 on AZMAG:odict_to_yaml into a14b287b0d4835acdc460252bb06f545b458b4df on UDST:master.

Coverage decreased (-0.3%) to 94.036% when pulling 542f926a688589d14129cd6ce17fbf3abf3a0609 on AZMAG:odict_to_yaml into a14b287b0d4835acdc460252bb06f545b458b4df on UDST:master.

Coverage increased (+0.04%) to 94.334% when pulling 058a06f3f6c5d693147cfd06d28d08dcaabf4ebb on AZMAG:odict_to_yaml into a14b287b0d4835acdc460252bb06f545b458b4df on UDST:master.

Coverage increased (+0.04%) to 94.334% when pulling 9abd5f7351df19b653c9f1191250284b635e0284 on AZMAG:odict_to_yaml into a14b287b0d4835acdc460252bb06f545b458b4df on UDST:master.


Change cmdlet name to Get-CertificateData

Cmdlets are always singular

This has been changed in version 0.0.10


Consider having differing prefixes for IDHash to make it clearer for the user

Currently, id hashes start with "0x" should consider a different beginning for clarity sakes. Also move off of storing 0x of hash in db

Decided not to implement, but rexex checking on fields is now much better #107


[SpeedDial] Fix broken aria reference issue

[x] I have followed (at least) the PR section of the contributing guide.

Fixes https://github.com/mui-org/material-ui/issues/19205

Let's also update the PlaygroundSpeedDial.tsx as per https://github.com/mui-org/material-ui/issues/19205#issuecomment-807701164

@oliviertassinari added test. Let me know if it requires any changes.


Question about prev_label_ids creation

Hi,
concerning the creation of the prev_label_ids token type ids: I see in the code that the index of a table-question pair in a batch is used to set them. If it's the first example in the batch, set them to zeros, if the index is > 0, then set them to the calculated label_ids of the previous table-question pair of the batch.
But: is it possible that this is only an approximation? Because questions do not have an equal length, then the previous label ids are not simply equal to the label ids of the previous example? Or am I missing something here? Maybe I can make it more clear using an example:
Suppose that we have the following 2 examples in a batch:

tokenized example 1
[CLS]
what
is
his
name
[SEP]
his
name
is
Niels
and
he
is
24
years
old
[PAD]

label_ids
0
0
0
0
0
0
0
0
0
1
0
0
0
0
0
0
0

tokenized_example 2
[CLS]
how
old
is
he
really
[SEP]
his
name
is
Niels
and
he
is
24
years
old

prev_label_ids of example 2 (set to label_ids of example 1)
0
0
0
0
0
0
0
0
0
1
0
0
0
0
0
0
0

Then the prev_label_ids of example 2 (which are set to the label ids of the first example), do not align with the tokens.
And second question: for datasets such as WTQ, WikiSQL, should they always be set to zeros, as they're not conversational?

The code you point to is the TF conversion that happens before batching.
At this point we will have a sequence of questions that all refer to the same table.
The code should map a cell coordinate (row and column index) to the right indices in the sequence.
You are right in that this position might change because it depends on the length of the question.
The code that maps from cell to index is here:
def _get_all_answer_ids_from_coordinates(
    column_ids,
    row_ids,
    answers_list,
):
  """Maps lists of answer coordinates to token indexes."""
  answer_ids = [0] * len(column_ids)
  found_answers = set()
  all_answers = set()
  for answers in answers_list:
    for column_index, row_index in answers:
      all_answers.add((column_index, row_index))
      for index in _get_cell_token_indexes(column_ids, row_ids, column_index,
                                           row_index):
        found_answers.add((column_index, row_index))
        answer_ids[index] = 1

  missing_count = len(all_answers) - len(found_answers)
  return answer_ids, missing_count

Does that help?

We also have a piece of code that does this based on TF examples.
This is required during prediction when we want realistic prediction for a sequence we need to do this on the fly (because the prev label ids depend on the previous prediction):
The code is here

At this point we will have a sequence of questions that all refer to the same table.

Yes I understand that the questions of a batch all refer to the same table. And I am using the same function that you quote in my implementation.
So the answer to my question is... yes? It is possible that the prev_label_ids are not set correctly, due to the fact that questions have different lengths? Wouldn't performance increase if you set the prev_label_ids based on len(question_tokens) of both the current and previous question?  This can easily be achieved in the convert function, by keeping a dictionary that maps an index of a question in the batch to its number of question tokens (including [CLS] and [SEP]).
Suppose this dictionary is called index2len_question_tokens. Each time you tokenize a question, you can store its length into this dictionary, as follows:
text_tokens = self._tokenize_extended_question(question, table)
index2len_question_tokens[index] = len(text_tokens)

Then you can replace
prev_answer_ids = get_answer_ids(interaction.questions[index - 1],) (from this line)
by
prev_answer_ids = [0]*index2len_question_tokens[index] + get_answer_ids(interaction.questions[index - 1],)[index2len_question_tokens[index - 1]:]

I think the current code should work correctly.
The important part is that this inline function:
    def get_answer_ids(question):
      if self._update_answer_coordinates:
        return _find_answer_ids_from_answer_texts(
            column_ids,
            row_ids,
            tokenized_table,
            answer_texts=[
                self._tokenizer.tokenize(at)
                for at in question.answer.answer_texts
            ],
        )
      return _get_answer_ids(column_ids, row_ids, question)

Captures column_ids, row_ids.
So we will use the column and rowl coordinates from the previous question, but the positions in the serialized sequence will be computed for the current query.
Does that make sense?
Can you otherwise give a test case where the ids are currently incorrect?

Oh ok, I understand now! Thanks for the clarification.


remove upcoming important changes from del RN

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: remove upcoming important changes from del RN https://harness.slack.com/archives/C06FDKGFV1U/p1713830510916069?thread_ts=1713782480.523249&cid=C06FDKGFV1U

Jira/GitHub Issue numbers (if any): ______________________________

Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6626feab65330c5f8efed51b--harness-developer.netlify.app


Added C++ book "More C++ Idioms"

What does this PR do?
Add resource(s): a new C++ book
For resources
Description
Describes a lot of functional patterns used in the C++ language.
Why is this valuable (or not)?
This book is quite unknown, but shouldn't be. It has great and quite unique mini-tutorials on how to use idioms that knowledge of is often taken for granted when someone mentions some solution.
How do we know it's really free?
Its on wikibooks, it has been authored there.
For book lists, is it a book? For course lists, is it a course? etc.
It is a book, it's self contained and while it contains stubs for expanding into new idioms, it has value in the form it currently is.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


White Wolf Mountain Fairy Mushroom Teleport crossed out/hidden in Kandarin, despite being able to teleport there.

You're able to teleport to the White Wolf Mountian teleport with the fairy mushroom in Kandarin, it takes you to the Catherby Fruit Tree patch, this is located in Kandarin, but it's crossed out/hidden as a teleport using this plugin currently.

I realised it's also crossed out with this plugin disabled, so perhaps this is a Jagex issue.


fix: added repository to pubspec.yaml and refactored

Hi amazing developers,
I noticed that there is no link to the development repository. Therefore, we added a link to the repository in pubspec.yaml.
Also, there were many other warnings about unnecessary imports, so we refactored those that could be fixed.

Hello! Thank you for the PR! Sorry to take too long, I've been maintaining voluntarily alone.


Failed: error connecting to db server: no reachable servers

Failed: error connecting to db server: no reachable servers .

I got error while connecting to server. where do i give the authorizing and connecting

This can be solved by manually grabbing the latest version of mongotocsv.js from the Git repo and then providing hostname, username, and/or password parameters. See issue #3 :)


Tags on Jupyter Notebooks with raw cells

After #51 is merged, it should be possible to use tags in jupyter notebooks using raw cells, but we need to make sure the tags are correctly rendered in the notebook output.

This is possible with nbsphinx (see the documentation for details)


How to deal with an input image of an odd size?

The network automatically learns the offset: https://github.com/wkentaro/pytorch-fcn/issues/87,
so you don't need to deal with odd size of image.


DMA recvchannel stop working after use DMA.recvchannel.transfer(output_buffer)

It seems like my dma recvchannel stop running after i run DMA.recvchannel.transfer(output_buffer)
I'm using .hwh and i think my .bit is ok because dma works fine in sdk.

By the way i'm using zcu104

The .bit file i'm using is like this

The error in python is like this

The code i'm using is like this

I think the execution order matters. See our example at: https://github.com/Xilinx/PYNQ-HelloWorld/blob/master/boards/ZCU104/resizer/notebooks/resizer_PL.ipynb
where you want to do something like
dma.sendchannel.transfer(in_buffer)
dma.recvchannel.transfer(out_buffer)
 resizer.write(0x00,0x81) # start
 dma.sendchannel.wait()
 dma.recvchannel.wait()

I think i have tried every combination of execution order, but still the recvchannel stop running after i strat transfer action.

The zynq setting is like this, i didn't enanble GPIO at the first version.

Hey did you ever figure this issue out? If I use wait_async() or no wait at all it completes the process but the output buffer doesnt have all correct values. If I use wait() it gives me the same error you have

you mean you can't use wait() function but you can still get correct data in buffer?
after I failed with python interface I switch to petalinux

Would you be able to print the output of pl2ps.read(0x30) and pl2ps.read(0x34) after the wait fails? It looks like for some reason the DMA engine is erroring out and shutting down but I don't understand why.
Only things that springs to mind is whether TLAST is being asserted on the correct beat - if more data is sent than the DMA is expecting that could cause the error you are seeing, or if the physical_address is getting assigned incorrectly.

I solved my DMA issue by looking into the parameters of the IP and adjusting the width of the buffer length (default 14) to 17. Progress! My new issue is data loss that appears to be a pattern. My IP uses 8 bit data streaming(I’ve tried char and ap_int<8>) but it only streams all of the correct values If the dtype is int64 when I initialize the in_buffer on jupyter notebook. If I set it to int32 it skips every other(places a 0), if it’s int16 it skips two shows two, int8 skips 4 shows 4
Example data coming in:
[1,2,3,4,5,6,7,8,9]
Int8 output
[1,2,3,4,0,0,0,0,9]
Int16
[1,2,0,0,5,6,0,0,9]
Int32
[1,0,3,0,5,0,7,0,9]
Int64 shows them all correctly

I am surprised to see the buffer length of 14 did not raise any warning / error message for you. Did you use tcl or hwh file for your overlay?

just tcl at the moment. My Z1 board is still using pynq 2.2. I was just recently made aware of the hwh when I gained access to an ultra96 running the pynq 2.4 image. I plan on reflashing my Z1 to 2.4 when I get home.
I am trying to get a basic 4 layer MLP using 5bit quantized weights for mnist 28x28 handwritten digits.
The first hidden layer has 100 neurons so the first multiplication is 784x100 5 bit numbers (switched to 8bit since I do not know how to set the dtype for the cma.numpy array to 5)
After wondering why my transpose worked on the host but not in the hardware I noticed the patterned data loss I mention in my previous post

So, why 17, is there any reason behind it?
And, I'm wondering will it be the problem of my address width setting?
I look into the resize example of ZCU104, it shows that the address width is set to 32

@eleICoto This is the buffer length width, so 17 means your DMA buffer can be as big as 2^17 = 128K bytes. For resizer example that is set to the maximum value, 4GB.

@theDweeb Better to post in another thread on pynq support forum. We are discouraged to discuss issues here.:)

Ok sorry. The pynq forums do not load for me (or any Google groups for that matter)
yes 784*100 at 8 bit values = 78400 bytes. log2(78400) is 16.25 so I rounded up to 17.
I should clarify that I am not doing your resizer but had similar issues (I just googled my problem and this thread popped up)
When using 2.4 without hwh I get a big warning. with hwh I receive no warning.
thanks again!

@yunqu Alright got it, depends on buffer size.
So do you have any idea on address width?

@eleICoto See my previous post

@yunqu Emmmm, I'm sorry, previous post?

@theDweeb So the wait() function is working correctly?

We have new support forum so you do not have to use google forum any more. If you have further questions, please post here: https://discuss.pynq.io/c/support


Bug 2233071: rebase: bump k8s.io dependencies to v0.26.7

Bumping the k8s.io dependencies to v0.26.7. The main intention is to update the client-go library dependency for kube 1.27+, where aggregated discovery is enabled. With client-go lower than v0.26.4 in some cases it is possible that the discovery response comes with the malformed response with nil GVK leading to Crash Loop Back Off state.
Upstream kubernetes issue:
kubernetes/kubernetes#116603

/lgtm


Figure out adding apps to Dock

Figure out a way to add installed applications into the Dock.
Applications are installed via brew cask install whatever, but they're not added to the Applications folder or the Doc. How would I do? [research ensues]

Not implementing. I really don't feel inclined to have them on the dock anyway, it's all iterm2 and spotlight.


New version: Git.Git version 2.47.1 (2.47.1.windows.1)

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

There is also an arm64 version which should be added:
https://github.com/git-for-windows/git/releases/download/v2.47.1.windows.1/Git-2.47.1-arm64.exe

/azp run

@wingetbot run

This package installs and launches normally in a Windows 10 VM.

@stephengillie The PR is not merging.

Sorry for the delay. The issue preventing merging has been declared as fixed, and my attempts to reopen it have failed, so I do my best to catch these and not make waves.


Undefined is not a function

Hi guys,
Buttons in the _renderContent function doesn't respond to events.
Take a look at this
` return (

Provider Name
{section['Provider'].name}

Provider Phone
{section['Provider'].phone}

Service Type
{section['Service'].name}

Status
{section['Schedule'].status}

  <View>
  <Button small style={styles.button} onPress={() => {
        this.setModalVisible(true);
      }}>
  {
      section['Schedule'].status = 'pending' ?
          <Text style={styles.buttonText}>Mark as Complete</Text>:
          <Text style={styles.buttonText}>Marked as Complete</Text>
  }
  </Button>
  </View>
  </View>
);`

When i click the button it says Undefined is not a function. Even when i tried to use this.setState
Note: Buttons outside the accordion responds.

Set your Accordion renderContent like this
renderContent={(section, i, isActive) => this._renderContent(section, i, isActive)}
And then in your _renderContent function you can do:
this.setModalVisible.bind(this, true)
Hope it helps

Works great. Thanks!


DCOS-50310: replace momentjs

Testing
Running npm run analyzebundle should no longer show moment.js in the vendor bundle
Start dcos-ui with the Soak cluster
Click around and make sure relative times look the same as they do in master
Here are some places to look:
Service debug tab
Job run history table (go to Jobs tab and click a job)
Deploy modal
Trade-offs
English locale is slightly different. Instead of saying a minute ago, it says 1 minute ago. We can extend the default English locale provided by date-fns if we don't like this
Dependencies

Screenshots

It looks good, but as @natmegs correctly pointed out, we should remove the console.logs first.
Also, I prefer "1 minute ago" to "a minute ago".
Another thing I noticed is that here we show "half a minute" instead of the exact number of seconds. This might be bad if some people want to know the exact number.
Here:

master:

@GeorgiSTodorov - good catch on the formatting for < 1 minute.

legit integration test failure

👀 - we need to merge this to make plugins ci green again

@juliangieseke 🤔 it was green without the PR

fixed 🤞
we need to fix our graphql implementation, finding an error is hard when errormessage gets swallowed somewhere…

@juliangieseke 🤔 it was green without the PR
no, plugins CI had a bug and the Run actually tested master instead of the PR 🙈 fix is in my #3767 PR

When using this with fixtures it breaks, because Dateutil.strToMs is called with 0 (where it expects a string).
Checked because i anticipated a huge performance gain on the services table (which to date is fully occupied with momentjs trying to parse date-strings). And indeed it seems to be faster by a lot! 🎉

extended early exit in strToMs 🙈

I ran this against soak and everything LGTM, but I agree with Brandon's request for changes related to lingui

https://github.com/dcos/dcos-ui/pull/3754#discussion_r271043098

I'm pretty sure this is associated with this feature prettier/prettier#4717 and not a configurable option. Let's keep the new formatting and I'll add a commit in order to unblock this PR

:tada: This PR is included in version 2.73.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

@juliangieseke 	@mperrotti 	we should have a (very small) follow up here for https://github.com/dcos/dcos-ui/pull/3754#pullrequestreview-221173174. a prettier-ignore will do: https://prettier.io/docs/en/ignore.html#jsx.

brandon fixed it: https://github.com/dcos/dcos-ui/pull/3754#discussion_r271043098

Thank you @juliangieseke and @brandonc for the commits 🙏

@juliangieseke i think that the translation key now looks a little odd containing a space. if i got it correctly prettier mandated this on us which does not feel right, thus the mention of prettier-ignore in the above comment.
if nobody else minds the space, we can ignore me on that matter. :)


Fix go.uuid api break change

see: https://github.com/satori/go.uuid/issues/66 AND https://github.com/satori/go.uuid/issues/84

Please get this merged, latest gocelery is unable to run
# github.com/gocelery/gocelery
vendor/github.com/gocelery/gocelery/uuid.go:7:18: not enough arguments in call to uuid.Must
	have (uuid.UUID)
	want (uuid.UUID, error)

Already fixed


Prompt for pass needs to be more dynamic.

Other sftp-clients will continue to prompt for password, until keyboard-interactive ends and user is logged inn.
This is not the case for this package, and makes issues for people with custom login-protocols.
In my case, this disables the possibility to use this package on servers where 2FA is activated.
The normal login-procedure for a standard ssh/sftp-server is:

Username:
Password:

However at servers with 2FA, the procedure is similiar to this:

Username:
Password:
Verification code:

Password can still be auto-filled, but remote-ftp needs to prompt the user for all of the unknown lines in keyboard-interactive. Verification-code need to pop up as a prompt that the user needs to fill in.
It appears that remote-ftp ignores the "Verification-code:" Text and assumes it was wrong password, thus failing with wrong password/all auth tried error.
I suggest that promptForPass should have the function of prompting whenever, and not only assume its the password its requesting.
Read more about 2FA here https://www.digitalocean.com/community/tutorials/how-to-protect-ssh-with-two-factor-authentication

Hmm, good feature request, but I don't think Remote-FTP's current ssh module (https://github.com/mscdex/ssh2) supports this... If you see a possibility, feel free to submit a PR :)


memory usage 1

Do preliminary work in checking places where the claimtrie uses memory, verify that the empty nodes are the main problem, check if there is some quick optimization that will improve memory usage.
After this is done, create new issues in this epic for the rest of the work.

closing this in favor of new issues :
https://github.com/lbryio/lbrycrd/issues/90
https://github.com/lbryio/internal-issues/issues/117


lone zero block in tar.lz4

Some of tar files produced with pipeline-16.10 trigger a warning while being unpacked:
canned/2013-05-05/dns_consistency.0.tar.lz4
tar: A lone zero block at 20380

canned/2016-04-01/http_requests.16.tar.lz4
tar: A lone zero block at 131740

canned/2017-01-01/facebook_messenger.0.tar.lz4
tar: A lone zero block at 280

Content of tarfile is OK, but root cause of the warning deserves being dug out.

Obsoleted


Removing source to cloud patching from 08-01-preview branch

ARM (Control Plane) API Specification Update Pull Request
PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

[1] ARM review queue (for merge queues, see [4])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] https://aka.ms/azsdk/support/specreview-channel
[3] List of SDK breaking changes approvers in pinned Teams announcement
[4] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[ ] New API version. (Such PR should have been generated with OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[x] Other, please clarify:

removing a feature from a new API version that has not been released yet

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to Step 2, "ARM Review", for this PR.

Breaking changes review (Step 1)

If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

ARM API changes review (Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
Getting help

For general PR approval workflow, see the diagram at the top of this comment.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

@rkmanda Hi! Sorry for tagging you here. This PR is opened to revert the changes for the source to cloud patching resource from this unreleased api version. We will open a new PR once we know the API version the patching resource will be targeting to. I know it is near the release date for the 08-01 api version. Could this be expedited by any chance? Much appreciated!


Prevent nested transactions with *DB.Transaction

Make sure these boxes checked before submitting your pull request.

[x] Do only one thing
[x] No API-breaking changes
[x] New code/logic commented & tested

What did this pull request do?
This will prevent creating nested transactions while calling *DB.Transaction(...) multiple times.
Only fist call will create transaction. Rest will just reuse it.

Could this be used in .Begin() for those that don't always use .Transaction?

It is probably not useful put it into .Begin(). Because Because even so you have to call .Commit() only once at the end.
Only solution that I can think of is putting also counter into *DB, which would count number of .Begin() calls and then ignored counter-1 calls to .Commit(). But I would say that it is easier to just use .Transaction() (and more error prone).

I find this interface to be a bit confusing, but I don't use it - so take my feedback with a grain of salt. I think this changes the semantics of the function a bit - when you write the function block as a programmer, you sort of assume that at the end of the function block, it'll be committed since that's the behavior in most cases.
Only in the nested case is this not true if I'm reading it correctly, and you can imagine issues arising if the "nested" transaction has an RPC call after.
In my company's codebase, we panic on these nested transactions in tests/removing ways to have nested transactions, and while I'm not saying that it is the ideal behavior, it forces our developers to make sure that they are aware of what data is committed or not.


SyntaxError: Unexpected token . (outdated Node.js)

When I run the thing it says this:
if (this.spectateTarget?.isRemoved) this.spectateTarget = null;
^
SyntaxError: Unexpected token .
at Module._compile (internal/modules/cjs/loader.js:721:23)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)
at Module.load (internal/modules/cjs/loader.js:653:32)
at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
at Function.Module._load (internal/modules/cjs/loader.js:585:3)
at Module.require (internal/modules/cjs/loader.js:690:17)
at require (internal/modules/cjs/helpers.js:25:18)
at Object. (my downloads\MultiOgarII-master\MultiOgarII-master\src\Server.js:13:16)
at Module._compile (internal/modules/cjs/loader.js:776:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:787:10)

What version of node are you using? Have you tried making sure it's up to date?

What version of node are you using? Have you tried making sure it's up to date?

let me check

v10.16.0

Yeah I'm on v15.3.0 and it's working fine for me. Try updating node.

k

thanks it works

but also agario might have removed the way to connect to servers with ip adresses

Yeah, currently this server doesn't work with the official client.


Alpha release touchups

closes #111
:clap: :champagne:

added units to creation forms to close #65

Added version printing to banner like so:

Maybe add the release to the sentry_sdk init in sfa_dash/__init__.py.
Something like
release=f'sfa-dash@{__version__}'


Not able to pull image for kubelet

Sometimes when I create clusters with acs-engine kubelet service not able to start. What I see from systemd logs is docker can not pull hyperkube image (gcr.io/google_containers/hyperkube-amd64:v1.5.3)
Apr 07 14:19:07 k8s-agentpool1-35482160-1 systemd[1]: Started Kubelet.
Apr 07 14:19:07 k8s-agentpool1-35482160-1 docker[5274]: Unable to find image 'gcr.io/google_containers/hyperkube-amd64:v1.5.3' locally
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: v1.5.3: Pulling from google_containers/hyperkube-amd64
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 5040bd298390: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 6b1b1be60d9d: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: f163e8430603: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: e3a2e612099f: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: ae1cc9eb5881: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: e3592413a73f: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: a6576fd6cbda: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 4426b151b520: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 5c17a00e67cd: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 3ad0a968bece: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: b5975036b156: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 3579abf9ff23: Pulling fs layer
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 3579abf9ff23: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: ae1cc9eb5881: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: e3592413a73f: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: a6576fd6cbda: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 4426b151b520: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 5c17a00e67cd: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: 3ad0a968bece: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: b5975036b156: Waiting
Apr 07 14:19:08 k8s-agentpool1-35482160-1 docker[5274]: e3a2e612099f: Waiting
Apr 07 14:19:39 k8s-agentpool1-35482160-1 docker[5274]: /usr/bin/docker: error pulling image configuration: Get https://storage.googleapis.com/artifacts.google-containers.appspot.com/containers/images/sha256:08fdc47202632891dbd15c1a1d25c2ff97afcc87a08e296cd43045619f0863e1: dial tcp <IP_ADDRESS>:443: i/o timeout.
Apr 07 14:19:39 k8s-agentpool1-35482160-1 docker[5274]: See '/usr/bin/docker run --help'.
Apr 07 14:19:39 k8s-agentpool1-35482160-1 systemd[1]: kubelet.service: Main process exited, code=exited, status=125/n/a
Apr 07 14:19:39 k8s-agentpool1-35482160-1 systemd[1]: kubelet.service: Unit entered failed state.
Apr 07 14:19:39 k8s-agentpool1-35482160-1 systemd[1]: kubelet.service: Failed with result 'exit-code'.

and I do not see agent nodes in cluster (there should be 10 agents, all of them have same issue):
kubectl get nodes
NAME                    STATUS                     AGE
k8s-master-30197476-0   Ready,SchedulingDisabled   9m
k8s-master-30197476-1   Ready,SchedulingDisabled   8m
k8s-master-30197476-2   Ready,SchedulingDisabled   8m

I'm able to connect to google.com:443 from this node:
nc -v google.com 443
Connection to google.com 443 port [tcp/https] succeeded!

but not able to connect to <IP_ADDRESS>:443:
nc -v <IP_ADDRESS> 443
nc: connect to <IP_ADDRESS> port 443 (tcp) failed: Connection timed out

so this does not look like network issue. And the only my guess, assuming that sometimes clusters work fine, is that maybe some azure's public ips are blocked by public docker repositories (could it be?).
anyway, here is public ip of this particular node: <IP_ADDRESS>. Hope it will help.

Thanks for reporting this. We are also looking at this internally. I don't have latest info on status though. @anhowe @amanohar Can you guys update here when you make progress?

Hi @vmakhaev , can you run "curl ifconfig.co" to get external IP of the VM where you are seeing the repro and let us the know the IP.

@vmakhaev, can you try again? The issue has been resolved now.

@amanohar external IP of the VM was <IP_ADDRESS>. I will return to this in few days and let you know if some thing goes wrong. Btw, what was the problem?

A live site incident last week impacted a subset of customers resulting in the symptoms described in this issue. The problem has been identified and was resolved Friday April 7 afternoon PDT

I hit the same issue today. some nodes fails to pull image for hours.
Apr 14 22:31:52 k8s-agentpool2-62235045-4 docker[5090]: Unable to find image 'gcr.io/google_containers/hyperkube-amd64:v1.5.3' locally
Apr 14 22:31:54 k8s-agentpool2-62235045-4 systemd[1]: Time has been changed
Apr 14 22:31:54 k8s-agentpool2-62235045-4 systemd[1]: apt-daily.timer: Adding 9h 17min 45.323201s random time.
Apr 14 22:31:54 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 4h 2min 6.451293s random time.
Apr 14 22:31:54 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 50min 9.283398s random time.
Apr 14 22:31:59 k8s-agentpool2-62235045-4 systemd[1]: Time has been changed
Apr 14 22:31:59 k8s-agentpool2-62235045-4 systemd[1]: apt-daily.timer: Adding 8h 36min 53.343869s random time.
Apr 14 22:31:59 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 3h 23min 56.086923s random time.
Apr 14 22:31:59 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 4h 13min 715.593ms random time.
Apr 14 22:32:04 k8s-agentpool2-62235045-4 systemd[1]: Time has been changed
Apr 14 22:32:04 k8s-agentpool2-62235045-4 systemd[1]: apt-daily.timer: Adding 11h 12min 53.892521s random time.
Apr 14 22:32:04 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 1h 29min 25.093203s random time.
Apr 14 22:32:04 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 3h 49min 5.841404s random time.
Apr 14 22:32:09 k8s-agentpool2-62235045-4 systemd[1]: Time has been changed
Apr 14 22:32:09 k8s-agentpool2-62235045-4 systemd[1]: apt-daily.timer: Adding 1h 13min 35.363241s random time.
Apr 14 22:32:09 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 1h 24min 3.335082s random time.
Apr 14 22:32:09 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 5h 14min 13.203605s random time.
Apr 14 22:32:14 k8s-agentpool2-62235045-4 systemd[1]: Time has been changed
Apr 14 22:32:14 k8s-agentpool2-62235045-4 systemd[1]: apt-daily.timer: Adding 3h 1min 52.329893s random time.
Apr 14 22:32:14 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 5h 10min 26.933088s random time.
Apr 14 22:32:14 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 4h 23min 49.798948s random time.
Apr 14 22:32:19 k8s-agentpool2-62235045-4 systemd[1]: Time has been changed
Apr 14 22:32:19 k8s-agentpool2-62235045-4 systemd[1]: apt-daily.timer: Adding 2h 40min 44.149317s random time.
Apr 14 22:32:19 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 1h 57min 41.599536s random time.
Apr 14 22:32:19 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 3h 22min 49.898867s random time.
Apr 14 22:32:23 k8s-agentpool2-62235045-4 docker[4994]: time="2017-04-14T22:32:23.117122719Z" level=error msg="Error trying v2 registry: Get https://gcr.io/v2/google_containers/hyperkube-amd64/manifests/v1.5
Apr 14 22:32:23 k8s-agentpool2-62235045-4 docker[4994]: time="2017-04-14T22:32:23.117184550Z" level=error msg="Attempting next endpoint for pull after error: Get https://gcr.io/v2/google_containers/hyperkube
Apr 14 22:32:23 k8s-agentpool2-62235045-4 docker[4994]: time="2017-04-14T22:32:23.117221868Z" level=error msg="Handler for POST /v1.24/images/create returned error: Get https://gcr.io/v2/google_containers/hy
Apr 14 22:32:23 k8s-agentpool2-62235045-4 docker[5090]: /usr/bin/docker: Error response from daemon: Get https://gcr.io/v2/google_containers/hyperkube-amd64/manifests/v1.5.3: dial tcp <IP_ADDRESS>:443: i/o
Apr 14 22:32:23 k8s-agentpool2-62235045-4 docker[5090]: See '/usr/bin/docker run --help'.
Apr 14 22:32:23 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Main process exited, code=exited, status=125/n/a
Apr 14 22:32:23 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Unit entered failed state.
Apr 14 22:32:23 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Failed with result 'exit-code'.
Apr 14 22:32:23 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Service hold-off time over, scheduling restart.
Apr 14 22:32:23 k8s-agentpool2-62235045-4 systemd[1]: Stopped Kubelet.
Apr 14 22:32:23 k8s-agentpool2-62235045-4 systemd[1]: Starting Kubelet...
Apr 14 22:32:23 k8s-agentpool2-62235045-4 ebtables[5119]: Bridge table: nat
Apr 14 22:32:23 k8s-agentpool2-62235045-4 ebtables[5119]: Bridge chain: PREROUTING, entries: 0, policy: ACCEPT
Apr 14 22:32:23 k8s-agentpool2-62235045-4 ebtables[5119]: Bridge chain: OUTPUT, entries: 0, policy: ACCEPT
Apr 14 22:32:23 k8s-agentpool2-62235045-4 ebtables[5119]: Bridge chain: POSTROUTING, entries: 0, policy: ACCEPT
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: Chain PREROUTING (policy ACCEPT)
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: target     prot opt source               destination
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: Chain INPUT (policy ACCEPT)
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: target     prot opt source               destination
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: Chain OUTPUT (policy ACCEPT)
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: target     prot opt source               destination
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: Chain POSTROUTING (policy ACCEPT)
Apr 14 22:32:23 k8s-agentpool2-62235045-4 iptables[5123]: target     prot opt source               destination
Apr 14 22:32:23 k8s-agentpool2-62235045-4 systemd[1]: Started Kubelet.
Apr 14 22:32:23 k8s-agentpool2-62235045-4 docker[4994]: time="2017-04-14T22:32:23.460190467Z" level=error msg="Handler for POST /v1.24/containers/create returned error: No such image: gcr.io/google_container
Apr 14 22:32:23 k8s-agentpool2-62235045-4 docker[5127]: Unable to find image 'gcr.io/google_containers/hyperkube-amd64:v1.5.3' locally

(If you use triple backticks, newline, paste, newline, triple backticks, then you will get slightly nicer formatting there)
We're working on fixing this (cc: @dmitsh), but in the meantime... we should look to see if this is the same root cause as last time and raise alarms as needed (cc:  @amanohar).

Thanks for the tips. Update the formatting. I still have one machine failed to pull the hyperkube image.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: Time has been changed
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: apt-daily.timer: Adding 3h 27min 7.979891s random time.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: snapd.refresh.timer: Adding 4h 46min 55.952799s random time.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 docker[4994]: time="2017-04-15T00:39:51.264921872Z" level=error msg="Not continuing with pull after error: Error pulling image (v1.5.3) from gcr.io/google_containers/hyp
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Main process exited, code=exited, status=125/n/a
Apr 15 00:39:51 k8s-agentpool2-62235045-4 docker[8870]: 5b1f17435ed5: Error pulling layer metadata
Apr 15 00:39:51 k8s-agentpool2-62235045-4 docker[8870]: eb0c136b8d12: Error pulling image (v1.5.3) from gcr.io/google_containers/hyperkube-amd64, endpoint: https://gcr.io/v1/, Failed to download json: Get https:
Apr 15 00:39:51 k8s-agentpool2-62235045-4 docker[8870]: eb0c136b8d12: Error pulling image (v1.5.3) from gcr.io/google_containers/hyperkube-amd64, Failed to download json: Get https://gcr.io/v1/images/5b1f17435ed
Apr 15 00:39:51 k8s-agentpool2-62235045-4 docker[8870]: /usr/bin/docker: Error pulling image (v1.5.3) from gcr.io/google_containers/hyperkube-amd64, Failed to download json: Get https://gcr.io/v1/images/5b1f1743
Apr 15 00:39:51 k8s-agentpool2-62235045-4 docker[8870]: See '/usr/bin/docker run --help'.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Unit entered failed state.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Failed with result 'exit-code'.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: kubelet.service: Service hold-off time over, scheduling restart.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: Stopped Kubelet.
Apr 15 00:39:51 k8s-agentpool2-62235045-4 systemd[1]: Starting Kubelet...

Can you get me the whole log? sudo journalctl -u kubelet | nc termbin.com 9999 will spit out a URL with the contents of the kubelet log. I need to see why the gcr pulls are failing. I'm wondering if the same livesite thing as last week is occurring now (sporadically) ?

I cannot repo the issue these days. All the nodes reported to the master after the Azure template is deployed..

Sounds like this can no longer be reproduced. Feel free to reopen @vmakhaev.


Capitalize Operator design pattern

Software design patterns, like Operator, seem to be proper nouns and
therefore should be capitalized.
For example, see the classic design pattern book:
Design Patterns: Elements of Reusable Object-Oriented Software
https://www.amazon.com/dp/0201633612
Also fix all instances in a second commit.

@dfarrell07 please check this page again: https://submariner.io/deployment/subctl/
"subctl helps to automate the deployment of the Submariner operator, thereby reducing the possibility of mistakes during the process."

@dfarrell07 please check this page again: https://submariner.io/deployment/subctl/
"subctl helps to automate the deployment of the Submariner operator, thereby reducing the possibility of mistakes during the process."

Good catch @nyechiel, fixed!


fix: use relative url for submodule

Use a relative URL for importing the dependencies submodule.
The top-level part of the URL is the same between repos, so need not be duplicated,
and can be inferred from the URL of the parent repo.
Tested by building locally and via CI.

bors try

bors merge


ENH: stats.lmoment: add function to calculate sample L-moments

Reference issue
gh-19460
What does this implement/fix?
The idea of fitting distributions to data with method of L-moments has been tossed around for a while, and gh-19460 brought it back up again. This PR adds a function to compute sample L-moments, which is an essential step of the method of L-moments.
Additional information
No rush here; I wrote the essential part of the code in https://github.com/scipy/scipy/issues/19460#issuecomment-1793547120 so I figured I might as well submit a PR.
This needs an email to the mailing list before merge. (Done 3/11/2024.)
I can see arguments for the name being l_moment instead of lmoment. If there is strong support for changing it, that's OK with me.

AlI vote for the name lmoment.
The argument order should not have a default value. Explicitly supplying the order makes the code more readable while adding no burden to the writer. And yes, the moment argument of the moment function should read order; but that would be a breaking change.
The axis argument should have default value None, but I guess we’ve passed the point of no return as moment function has axis=0 by default. None would have been a least-surprise choice, and in C layout working on the 0th axis is really poor performance.
The sorted argument is very useful!
The standardize argument should not be there, and in particular should not default to True, because higher order L-moment over second L-moment is analogous to skew and kurtosis of conventional moment, but skew and kurtosis are not “standardized” moments and in particular they should not be the concern of the moment function.

Btw I come across this Python package lmo (BSD license) which deals with L-moments and which appears to be well written. Could be used as a reference.

The argument order should not have a default value.

The is a justifiable perspective. Please keep in mind that the choice is subjective; other qualified people have thought about the same thing and come to a different conclusion. stats.moment has a default value, so I included it, but I don't care too strongly either way as long as we realize it is not a matter of "right" and "wrong".

And yes, the moment argument of the moment function should read order; but that would be a breaking change.

This can easily be changed in a backward compatible way with _rename_parameter, and I'd be happy to do so if there is some support for it.

The axis argument should have default value None, but I guess we’ve passed the point of no return as the moment function has axis=0 by default. None would have been a least-surprise choice, and in C layout working on the 0th axis has the worst performance in terms of memory locality.

Before breaking changes are allowed by a SciPy 2.0 release, scipy.stats as a whole has passed the point of no return in the choice of axis=0 - almost every reducing function has axis=0 as the default.  Again, I would caution against the use of "should"; this is a matter of opinion. I've written about this a few times, but here are the main reasons for each possibility:

axis=0: best for pandas DataFrames
axis=None: same as NumPy
axis=-1: like axis=0, assumes slices are independent, but faster for C arrays
I've advocated for axis=-1 in the past, but that's beyond the scope of this PR.

The sorted argument is very useful!

Great!

The standardize argument should not be there, and in particular should not default to True, because higher order L-moment over second L-moment is analogous to skew and kurtosis of conventional moment, but skew and kurtosis are not “standardized” moments and in particular they are not the concern of the moment function.

Skewness and kurtosis are known as "standardized moments".
Higher order L-moment ratios are also known as "standardized L-moments".
This may or may not change your opinion on whether the standardize argument should be included, but did you have other rationale?

Thanks for the reply @mdhaber !
The _rename_parameter thing sounds cool! The reason why I think there should not be a default order is because one would say “the sample mean is xxx” but would not say “the sample moment is xxx” — which moment? Therefore the signature of the function should reflect this.
For axis, pandas defaults to 0 because it is natural for panel data, where the columns often mean very different things (e.g. age, weight, height) so it doesn’t make sense to average them. Consistent with this, pandas stores its data by columns. Since numpy copes with both Fortran and C conventions, None looks a reasonable choice. I also find None the least surprising for multidimensional array of data of the same nature.
Thanks for pointing out the term of “standardized moments”, I haven’t heard of them before. I mentioned Python lmom because I find it well written from the looking (though I have not verified its calculations). I find your code good enough for computing the sample L-moment. But in case you want to add more L-moment related functionalities, such as L-moment estimator or more sophisticated statistics, I guess that package could save you much time from reinventing the wheel :-)
Hosking’s R function samlu by default returns the 1st and 2nd L-moment together with the 3rd and 4th L-ratio. It may well suit R users’ expectation, but I’m not sure it would be a good choice for scipy as I see scipy more toward providing lower-level building blocks that are easily reusable and composable. Higher level “off-the-shelf” solutions can leave to say statsmodels. (My experience with statsmodels is that when it provides the functionality it’s really nice, but if you want something extra then it’s hard to reuse the existing code and I sometimes have to work out from scratch. Hence my desire for “lower-level building blocks”.)

@rlucas7 I wonder if I could interest you in reviewing this one?

@rlucas7 I wonder if I could interest you in reviewing this one?

I definitelt will but I won't be able to look until the coming weekend.

Thanks for being willing to review!

Thanks @rlucas7, resolved the failures (caused by hasty merge).

Thanks @rlucas7. I resolved the failures (caused by hasty merge).

thanks @mdhaber no worries, I'm on on long weekend/vacation this weekend so I likely won't be able to take a look until the weekend after this one.
I'd like to build the branch and do some poking to check for edge cases etc.
Hopefully that's ok w/your timeline on this one and feel free to ping me later next week.

@rlucas7 I just fixed merge conflicts so I thought I'd ping you. Thanks for your help!

Thanks @jorenham. To facilitate further cycles of review, please click "Resolve conversation" when the comment is no longer needed. Please leave those that require action from me or input from @rlucas7 unresolved.

I dont't think I can; I'm not a maintainer here

Ok, please add an 😄 emoji to the ones I can resolve.

It apparantly became a 👍🏻, is that ok too?

Hey @rlucas7, thought I'd ping you again here. Think we can get this into the next version of SciPy?

Oops, the last time we ran tests was a while ago, and this is failing a relatively new test. I'll work on that now.


Update azure-monitor-glossary.md

@TomHickling : Thanks for your contribution! The author(s) have been notified to review your proposed change.

There's a merge conflict, so I'm going to create a different pull request that incorporates this change. #please-close


Template error in admin/repo/list

template: admin/repo/list:33:39: executing "admin/repo/list" at <.Id>: Id is not a field of struct type *models.Repository

I think #170 has resolved this bug.


[Question] How cat to set auto focus mode off.

Dear. Pedro
I have a question.
I has used the RtmpCamear2 API for streaming broadcasting in OpenGl Surface view.
I want to auto focus mode off but can not to set  the auto focus off.
It does not supported in Camera2ApiManager.java
How can set to auto focus mode off at front and rear camera?
I has set false value of use feature in android manifest .xml
    android:name="android.hardware.camera"
    android:required="false"

    android:name="android.hardware.camera2"
    android:required="false"

    android:name="android.hardware.camera.autofocus"
    android:required="false"

I added this feature to my last commit:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/commit/f27cf4c372e9a6ceca04a90b0570d9362c1dedfd

Hi. Pedro
I have applied the last commit for camera auto focus setting.
but , auto focusing still working .
auto focus.zip

Hi , Pedro.
thanks for your advice.
Auto focus setting working well  which has apply at the camera capture session callback function.
private void startPreview(CameraDevice cameraDevice) {
try {
final List listSurfaces = new ArrayList<>();
Surface preview = addPreviewSurface();
if (preview != null) listSurfaces.add(preview);
if (surfaceEncoder != preview && surfaceEncoder != null) listSurfaces.add(surfaceEncoder);
  cameraDevice.createCaptureSession(listSurfaces, new CameraCaptureSession.StateCallback() {
    @Override
    public void onConfigured(@NonNull CameraCaptureSession cameraCaptureSession) {
      Camera2ApiManager.this.cameraCaptureSession = cameraCaptureSession;
      try {
        CaptureRequest captureRequest = drawSurface(listSurfaces);
        if (captureRequest != null) {
          cameraCaptureSession.setRepeatingRequest(captureRequest,
              faceDetectionEnabled ? cb : null, cameraHandler);
          Log.i(TAG, "Camera configured");
        } else {
          Log.e(TAG, "Error, captureRequest is null");
        }
        // auto focus disable when started streaming
        **disableAutoFocus();**
      } catch (CameraAccessException | NullPointerException e) {
        Log.e(TAG, "Error", e);
      } catch (IllegalStateException e) {
        reOpenCamera(cameraId != -1 ? cameraId : 0);
      }
    }

And recover camera auto focus when the close camera
public void closeCamera(boolean resetSurface) {
resetCameraValues();
...
....
if(!isAutoFocusEnabled()){
enableAutoFocus();
}
}

Hi.Pedro.
I have a question.
Can I auto focus off only the front camera and keep the auto focus setting on the Back camera?

The only way to do it is switch camera to front camera and disable autofocus in the next line:
rtmpCamera1.switchCamera();
if (rtmpCamera1.isFrontCamera()) rtmpCamera1.disableAutoFocus();
else rtmpCamera1.enableAutoFocus();

I will fix autofocus disable before startPreview in next commit.


Fix the decode_ptypstring() function

The previous implementation of decode_ptypstring() was incorrect assumed the 8 most significant bits always are zeroes. This new implementation fixes it.

Thanks so much for the commit..

published in crates as well.
Once again thanks so much for your commit :)


Summarize users "getting to value"  (downloads) on the analytics reports

We have the answers to many of these questions buried in the report. Let's extract some answers to key questions in a summary at the top o the report.
In the last month:
Traffic Summary

x many users came to the site x many times.
x many users came to the data browser x many times.

Download / Export Summary

x many users requested or performed, x many downloads
x many users requested or performed x many matrix downloads.
x many users requested or performed x many DCP-generated matrix downloads.
x many users requested performed x many contributor matrix downloads.
x manny users downloaded or requested alignment files (BAMs)
x manny users downloaded or requested sequencing files (FASTQs)
x many users downloaded or requested metadata.

Project Specific Downloads

x many users downloaded x many project files
x many users downloaded x many project matrices.

Cross Project (Bulk ) Downloads
1.
To be continued and refined

Obsolete.


addresses #682, extends PaginationRequest as MessageListRequest adding run_id

MessageListRequest as extension of PaginationRequest like VectorStoreFileListRequest

@dotnet-policy-service agree company="Sigmund Software"

excuse some of the merge fumbling plz, I work all Azure Devops and am still finding my way with Git.
Summary of changes is MessageListRequest:PaginationRequest which adds run_id and use of that object in in plce of PaginationRequest

you don't, that's an accidental change.
On Tuesday, November 19, 2024 at 10:14:47 AM EST, Tolga Kayhan @.***> wrote:
@kayhantolga commented on this pull request.
In .gitignore:

\ No newline at end of file
+.idea
+/OpenAI.SDK/Betalgo.Ranul.OpenAI.csproj

why do we need to add /OpenAI.SDK/Betalgo.Ranul.OpenAI.csproj to ignore?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you modified the open/close state.Message ID: @.***>


Refactoring 🛠(WIP)

The script is over 700 lines right now and it's a pain to maintain.
The goal of this PR is to break the script into functions and somewhat refactor everything to make it simpler and easier to understand.

Too much conflicts, I'll start from scratch


Use a polyfill for WeakMap for older browsers

Currently writing my unit tests, and came accros an issue with callback-proxy.js.
PhantomJS 2.1.1 (Mac OS X 0.0.0) ERROR
  ReferenceError: Can't find variable: WeakMap
  at webpack:///~/fine-uploader-wrappers/callback-proxy.js:19:0 <- index.js:27812

I know it's not an issue per se, because PhantomJS don't know WeakMap (I'm currently looking for a polyfill), but since fine-uploader is supposed to be compatible with IE8+, it might be a good idea to add a polyfill when it's not available for a given browser.

A few things:

React Fine Uploader and all newer libraries in this org that wrap Fine Uploader or extend it should only be targeted at newer browsers: IE11 and up.
I'll be removing support for all browsers older than IE11 in fine-uploader in v6.0.

I stopped using phantom a long time ago, it's too much of a hassle. Better to test against real browsers anyway.
Time to move forward and leave old browser behind!


Sometimes apear

After relocate on bombay-12 I began to get this error sometimes.
Same code as my, but from example, only data of wallet and amount, denoms and etc changes.
   msgs: [
     new MsgMultiSend(
       [
         new MsgMultiSend.Input(keys.test1.accAddress, "1000000uluna"),
         new MsgMultiSend.Input(keys.test2.accAddress, "1000000uluna"),
         new MsgMultiSend.Input(keys.test3.accAddress, "1000000uluna"),
       ],
       [output, output, output]
     ),
   ],
 });

 const signatures = await Promise.all(
   ["test1", "test2", "test3"].map(async (keyName) => {
     const key = keys[keyName] as MnemonicKey;
     const acc = (await client.auth.accountInfo(key.accAddress)) as Account;

     tx.account_number = acc.account_number;
     tx.sequence = acc.sequence;

     return key.createSignature(tx);
   })
 );

 const stdTx = new StdTx(tx.msgs, tx.fee, signatures, tx.memo);

 await client.tx
   .broadcastSync(stdTx)
   .then((result) => {
     console.log(result);
   })
   .catch((err) => {
     console.error(err.message);
   });
} ```

Full error message: rpc error: InvalidArgument desc = account sequence mismatch, expected 161, got 160: incorrect account sequence: invalid request

Can you drop the code that produce this error

I have the similar error I guess. It arises spontaneously, and there is no pattern. This was not the case on tequila This only happens on bombay and quite often
const msg = new MsgSend(accAddress, self.receiverTerraWallet, amount)
        const tx = yield self.terraClient.wallet.createAndSignTx({
          msgs: [msg],
          memo: self.for,
        })
const txResult = yield self.terraClient.terra.tx.broadcast(tx)

after the broadcast I sometimes get signature verification failed: please verify account number (69745) sequence (97) and chain-id (bombay-12) unauthorized

@dagogodboss All works fine, but sometimes this error appear in random moments.
const tx = await wallet.createTx({
msgs: [new MsgMultiSend(_msgs, [...multiSendOutoutData])],
feeDenoms: ['uusd'],
gasPrices: '0.15uusd',
});
const signatures = await Promise.all(
  ['im', 'spare'].map(async keyName => {
    const key = keys[keyName];
    const acc = await client.auth.accountInfo(key.accAddress);

    tx.account_number = acc.account_number;
    tx.sequence = acc.sequence;

    return key.createSignature(tx);
  })
);

const stdTx = new StdTx(tx.msgs, tx.fee, signatures, tx.memo);

const response = await client.tx
  .broadcastSync(stdTx)
  .then(result => {
    return { result };
  })
  .catch(error => {
    return { error };
  });

The Guideline for solving sequence mismatch problem:

Use one node for broadcasting the tx. Do not use public infrastructure because it has load balancer before the cluster
Save sequence in local variable and use it for making sequential transactions within a block.


conda environment.yml broken?

Currently, all CIs using conda with our environment.yml are broken because of unsatisfiable dependencies. It has something to do with traits, which is available as 5.2.0, but some other package (pyface?) wants >=6.0.0. I tested this locally, but everything works fine, so maybe the problematic packages have already been updated upstream. I can't restart Travis jobs for some reason, maybe @sappelhoff can you do it in #8037?
Related issue: I noticed that pyqt 5.9.2 gets installed automatically as a dependency by conda, but then we overwrite it with a newer version of PyQt5 available in pip. This could be problematic and might explain issues related to PyQt that pop up from time to time.

https://travis-ci.org/github/mne-tools/mne-python/builds/710508815

Now the first jobs have a different error:
ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: -n

There doesn't seem to be a -n argument (pytest -m "${CONDITION}" --tb=short --cov=mne -n 1 ${USE_DIRS}).

-n is used by the pytest-xdist plugin. It's not installed?

Looks like not. Where is the requirements file that contains this package?

@sappelhoff could you maybe also restart the failing Azure job?

done, but I can only re-run the failed ones. Can only the PR owner or an admin re-run? I think you and I should have same permissions @cbrnr

The failed ones are OK for now. I didn't find the restart button in Travis (although I was logged in and I know where the button usually is - now there is just a "More options" button). I haven't figured out how to restart Azure jobs, I'm logged in and now I see only a "Cancel" button for the running job.

I haven't figured out how to restart Azure jobs, I'm logged in and now I see only a "Cancel" button for the running job.

you do it from the GitHub UI, go to "details" and then there is an expanded dropdown menu on the left

This page? I don't have a dropdown menu.

This is the one I mean:

and it shows handles when you can restart something. you can't do anything for running or succeeded jobs with simple maintainer permissions.

Fixed upstream.