Daily update

Automated changes by create-pull-request GitHub action

Build failed because of:
No cached entry, and can not fetch: arcadyan-technology-corporation. Can't resolve stock ticker 3596; please manually add a "stock_ticker" key to landscape.yml or set to null
No cached entry, and Services4-IT has issues with logo: services4-it.svg, SVG file embeds a png. Please use a pure svg file
No cached entry, and Linklogis Inc has issues with logo: linklogis_inc.svg, SVG file embeds a png. Please use a pure svg file
No cached entry, and Banma Information Technology has issues with logo: banma_information_technology.svg, SVG file embeds a png. Please use a pure svg file
No cached entry, and Open Sourcing Mental Illness, LTD has issues with twitter: https://twitter.com/osmihelp, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
No cached entry, and Sollers College has issues with twitter: https://twitter.com/SollersEdu, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for Shiv Shakti Math: https://twitter.com/MathShakti
Empty twitter for CSIRO/DATA 61: https://twitter.com/CSIROnews
Empty twitter for Zhejiang Dahua Technology Co., Ltd: https://twitter.com/dahuasecurity
Empty twitter for TYNYBAY Inc: https://twitter.com/tynybayinc
Empty twitter for Truepic, Inc.: https://twitter.com/truepicinc
Empty twitter for Tectonic Labs Ltd.: https://twitter.com/cloud_ridge
Empty twitter for SVA System Vertrieb Alexander GmbH: https://twitter.com/SvaGmbh
Empty twitter for Surge: https://twitter.com/surgetech
No cached entry, and ReadMe has issues with twitter: https://twitter.com/readmeio, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for Rancher Federal: https://twitter.com/RancherFederal
Empty twitter for plural: https://twitter.com/plural_sh
No cached entry, and JVC KENWOOD Corporation has issues with twitter: https://twitter.com/JVCKENWOOD_SR, 401 - {"request":"/1.1/statuses/user_timeline","error":"Not authorized."}
Empty twitter for Enterprise DB Corporation: https://twitter.com/EnterpriseDB
Empty twitter for eBaoTech International Pte Ltd: https://twitter.com/Ebaotech
Empty twitter for China Telecommunications Corporation: https://twitter.com/chinatelecom1
No cached entry, and ATS (a KGPCo Company) has issues with twitter: https://twitter.com/KGPCo1, 404 - {"errors":[{"code":34,"message":"Sorry, that page does not exist."}]}
the project does not use a crunchbase, but a crunchbase https://www.crunchbase.com/organization/rootchain-technology-co-ltd is present for a Rootchain Technology Co., Ltd
Arcadyan either has no crunchbase entry or it is invalid


Form improvements

Neoletter instance validation is still missing. I first need to get a signal by the team, that we can require the length of the ID.

REVIEW (3a1c450)
Also added a (now approved) validation for "Neoletter instance".


Grjones use setnx for locking

#60 but add's extra documentation and countdown back to AlreadyQueued exception.

Pull Request Test Coverage Report for Build 191

8 of 10 (80.0%) changed or added relevant lines in 1 file are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-1.9%) to 96.753%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

celery_once/backends/redis.py
8
10
80.0%

Files with Coverage Reduction
New Missed Lines
%

celery_once/helpers.py
1
97.5%

celery_once/backends/redis.py
2
92.73%

Totals

Change from base Build 189:
-1.9%

Covered Lines:
149

Relevant Lines:
154

💛  - Coveralls

Steps to remove existing keys for 1.0 -> 2.0 migration.
NOTE: These are untested, use at your own risk.
This will list all celery-once key's (starting with the prefix qo_) then delete them...
redis-cli KEYS "qo_*" | xargs redis-cli DEL


Configure Watcher when to delete old PipelineRuns and TaskRuns - retention time

Feature request
We would like to be able to configure how long the PipelineRun and TaskRuns will stay on the cluster till deleted. It would be great to configure a retention time.
Use case
We start a lot of Pipelines with Tekton and when they fail we want to analyze them. We would like to have the option to let them for one day on the cluster or so, that also the user of the pipeline has the chance to navigate through PipelineRuns and TaskRuns with the standard Tekton Pipeline Ui.

@manuelwallrapp  this is what the -completed_run_grace_period Watcher's flag does:
- -completed_run_grace_period
        - 24h # or whatever value you consider more suitable
        ```

Is that what you were looking for or would you like to configure the retention for each run (say, via an annotation)?

Yeah I found it as well shortly after I posted the question but forgot to answer it here. Thank you. Diving into the code helps.


Fix how we handle various errors from the extension

https://github.com/penumbra-zone/web/assets/1121544/de6001fa-0d69-43bc-9619-f3d3566efe12
There were a number of issues with how we were handling errors coming from the extension. This PR fixes the following:

Show a nicer message when the user is not logged into the extension (closes #349).
Show a nicer message when the user closes the approval popup without approving or denying (closes #519).
Show a nicer message when the user denies authorization (re-fixes #517, which recently had a regression)

@grod220 you mentioned in #349 that login detection would require adding an Std method. Not sure what that means — did you mean an RPC method? If so, I went with a different approach here, simply throwing a Code.Unauthenticated error from the extension if the user is not logged in, rather than proactively querying from the webapp. This seems better, since the right time to check if the user is logged in is at the moment the tx approval request is being sent to the extension. If we query earlier from the web app, it's possible that the user would log out from the extension between when we made the query and when we send the tx approval request.


bootstrapJython task is always executed

=== build.gradle ===
buildscript {
repositories {
maven {
url "https://plugins.gradle.org/m2/"
}
}
dependencies {
classpath "gradle.plugin.com.jetbrains.python:gradle-python-envs:0.0.8"
}
}
apply plugin: "com.jetbrains.python.envs"
envs {
bootstrapDirectory = new File(buildDir, 'pythons')
envsDirectory = new File(buildDir, 'envs')
minicondaVersion = 'latest'
packages = ["pip", "setuptools"]
}
=== gradle output ===
:build_conda_envs SKIPPED
:bootstrapJython
Performing silent installation
10 %
20 %
30 %
40 %
50 %
60 %
70 %
80 %
Generating start scripts ...
Installing pip and setuptools
90 %
Ignoring indexes: https://pypi.python.org/simple
Collecting setuptools
Collecting pip
Installing collected packages: setuptools, pip
Successfully installed pip-7.1.2 setuptools-19.2
100 %
Congratulations! You successfully installed Jython 2.7.1b3 to directory /Users/ahutalla/projects/xdn-gateway/build/pythons/jython.
Collecting virtualenv
Using cached virtualenv-15.0.2-py2.py3-none-any.whl
Installing collected packages: virtualenv
Successfully installed virtualenv-15.0.2
You are using pip version 7.1.2, however version 8.1.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
:build_jython_envs SKIPPED
:create_files SKIPPED
:build_envs UP-TO-DATE
:assemble UP-TO-DATE
:check UP-TO-DATE
:build UP-TO-DATE

@aputs  I think that build_env task calls jython_envs_task https://github.com/JetBrains/gradle-python-envs/blob/e8534ed908dce501a863c6c4a54dc3378d94434b/src/main/groovy/com/jetbrains/python/envs/PythonEnvsPlugin.groovy#L240 .

@aputs run build_conda_envs instead of build_envs


Publish extension to open-vsx

With this PR we will publish vscode-tekton to open-vsx

Codecov Report

Merging #423 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #423      +/-   ##
==========================================
- Coverage   73.36%   73.34%   -0.02%
==========================================
  Files         122      122
  Lines        8555     8555
  Branches     1609     1609
==========================================
- Hits         6276     6275       -1
- Misses       2279     2280       +1

Impacted Files
Coverage Δ

out/src/tkn.js
63.10% <0.00%> (-0.13%)
:arrow_down:

Continue to review full report at Codecov.

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


readme: use mermaid for 4 stage pipeline diagram

Before:

After:

Sounds like a cool tool. I think I prefer keeping the diagram in ascii but thank you!


[BEAM-3083] Sets sdk_harness_container_images property for all Dataflow Runner V2 jobs

Currently this is only set for x-lang jobs.
We can set this for all jobs now that corresponding Dataflow API is stable.
This will make container setup for x-lang and non-x-lang jobs for runner V2 behave in a similar manner.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python 3.8 PostCommit

R: @angoenka @ananvay

Run Python PreCommit

Run Python PreCommit

Thanks.


Building with -DSECURITY=ON increases memory usage also in non-secured applications [5082]

Hi,
I built Fast-RTPS master branch using the CMake argument -DSECURITY=ON
Then I ran the HelloWorldExample (which does not use security).
The problem is that the memory usage is considerably higher than if I would have built Fast-RTPS with -DSECURITY=OFF.
The difference in memory is mainly caused by the fact that

Additional secured end-points are created together with the usual non-secured end-points by EDPSimple::initEDP method
A third object of type CDRMessage_t is created in each RTPSMessageGroup

I got the following numbers running HelloWorldExample subscriber with FastRTPS v1.7.2
Resident set size computed using ps or top

RSS Security ON:   11.3MB
RSS Security OFF:  9MB

The difference increases with the number of pubs/subs created in each participant.
I think that the decision at the build time should not cause any additional memory allocation.
Moreover I think that, from the ROS2 point of view, having both secured as well as not secured end-points inside a node is not useful, as secured processes will not be able to see/communicate non secured ones.
However, maybe this feature is required for other Fast-RTPS implementations.

This was fixed on #479 which was released with 1.9.0, so I think it may be closed


Config for setting saturation and brightness

Default (proposal):

nodes

saturation: 100%
brightness: 40%

edges

saturation: 0%
brightness: 40%

Default (discussed and decided):

nodes

saturation: 100%
brightness: 37%

edges

saturation: 0%
brightness: 62%

Done with d9d6ffa1f22c77f19152398a6f0dc72f3750d3a1. Thanks!


feat: support array of path with name (sync all method)

@niftylettuce @dead-horse Any time to review this PR?

is it really worth to add this usage? I'd like to keep it simple.

So you mean array of path is supported without a specific name, but not supported with name, I do think it's inconsistent. Or the name feature should be removed.


Distribute roots.pem with Python package

Inspired by #3805
I've been using system certs in a hacky way via:
https://gist.github.com/dhermes/bbc5b7be1932bfffae77
An OS X user filed https://github.com/dhermes/gcloud-python-bigtable/issues/21 and I tried to address with another hack:
https://github.com/dhermes/gcloud-python-bigtable/commit/d16fa35c4070445d38a5103914c1c207487eb7ed
However, I'd like to be able to create ssl_client_credentials without having to know where the roots are stored. Currently just to make an request to a secure URI I have to do:
from grpc.beta import implementations
root_certificates = 'PEM contents loaded from some file'
client_credentials = implementations.ssl_client_credentials(
    root_certificates, private_key=None, certificate_chain=None)

/cc @nathanielmanistaatgoogle @jgeewax @tseaver

If you pass None as the root_certificates, do you find that the "system default" root certificates are found and used? The documentation for that parameter does read The PEM-encoded root certificates or None to ask for them to be retrieved from a default location., doesn't it?

Fingers crossed. Checking now :grinning:

In [8]: implementations.ssl_client_credentials?
Signature: implementations.ssl_client_credentials(root_certificates, private_key, certificate_chain)
Docstring:
Creates a ClientCredentials for use with an SSL-enabled Channel.

Args:
  root_certificates: The PEM-encoded root certificates or None to ask for
    them to be retrieved from a default location.
  private_key: The PEM-encoded private key to use or None if no private key
    should be used.
  certificate_chain: The PEM-encoded certificate chain to use or None if no
    certificate chain should be used.

Returns:
  A ClientCredentials for use with an SSL-enabled Channel.
File:      /usr/local/lib/python2.7/dist-packages/grpc/beta/implementations.py
Type:      function

Can I get a RTFM @dhermes?

Unfortunately there is still an issue
In [1]: from gcloud_bigtable import Client

In [2]: client = Client(project='my-project')

In [3]: client.start()
E1201 14:54:31.379212042   13770 file.c:81]                  Could not open file /usr/share/grpc/roots.pem (error = No such file or directory).
E1201 14:54:31.379231520   13770 security_connector.c:581]   Could not get default pem root certs.

I installed the core via apt-get install libgrpc-dev on Ubuntu 14.04 and then just pip install grpcio==0.11.0b1.

@jboeuf and @ctiller: what's the user experience supposed to be here? How is it that @dhermes should come to have a /usr/share/grpc/roots.pem file (or for the core to find an acceptable file elsewhere)?

@dhermes Is that output spat out and the client goes on to eventually find something and work (i.e. it's just being verbose) or is that actually a failure?

Good call. I'll try to execute something.

In [1]: from gcloud_bigtable import Client

In [2]: client = Client(project='my-project', admin=True)

In [3]: client.start()
E1201 15:04:23.975698434   14455 file.c:81]                  Could not open file /usr/share/grpc/roots.pem (error = No such file or directory).
E1201 15:04:23.975719329   14455 security_connector.c:581]   Could not get default pem root certs.
E1201 15:04:23.976557546   14455 security_connector.c:581]   Could not get default pem root certs.
E1201 15:04:23.977202667   14455 security_connector.c:581]   Could not get default pem root certs.
E1201 15:04:23.977640730   14455 security_connector.c:581]   Could not get default pem root certs.

In [4]: from grpc.framework.interfaces.face.face import NetworkError

In [5]: try:
   ...:     result = client.list_zones()
   ...: except NetworkError as exc:
   ...:     print 'Failed'
   ...:

Failed

In [6]: exc.code
Out[6]: <StatusCode.INVALID_ARGUMENT: 3>

In [7]: exc.details
Out[7]: 'Failed to create security connector.'

@dhermes: it seems like a safe assumption but just to draw it out explicitly: that's a code sequence that works perfectly well if you explicitly provide a non-None root_certificates on channel creation?

Yup. I hard code the system paths used on Debian Linux and OS X (with the one on OS X installed with homebrew's version of openssl).

Could the python packaging script do something similar to what nodejs did here in order to include the roots included in the grpc source tree (in etc/)?
See https://github.com/grpc/grpc/pull/3846/files

@jboeuf I'm confused - I thought we were trying to have the user be able to load from a default location? How does that link to distributing more files?

It seems to be a hard problem. The certifi project (https://certifi.io/en/latest/) just ships their own certs file rather than trying to make an OS specific mapping to default locations. There is some discussion of this on the Python bug tracker.

@dhermes The last message appears to say that they weren't going to distribute (back in January of this year). They went and distributed later on? Before I try to do some digging, do you already have other relevant discussion links handy to reconcile that last bit and what appears to be the case now?

The Python bug tracker (i.e. python-dev) != the open source project certifi.

The specific comment I linked to was about managing a list of default system paths to certs (which BTW doesn't help on Windows, or OS X, since the OS X path in the thread no longer is shipped with OS X):
cert_paths = [
    # Debian, Ubuntu, Arch, SuSE
    # NetBSD (security/mozilla-rootcerts)
    "/etc/ssl/certs/",
    # Debian, Ubuntu, Arch: maintained by update-ca-certificates
    "/etc/ssl/certs/ca-certificates.crt",
    # Red Hat 5+, Fedora, Centos
    "/etc/pki/tls/certs/ca-bundle.crt",
    # Red Hat 4
    "/usr/share/ssl/certs/ca-bundle.crt",
    # FreeBSD (security/ca-root-nss package)
    "/usr/local/share/certs/ca-root-nss.crt",
    # FreeBSD (deprecated security/ca-root package, removed 2008)
    "/usr/local/share/certs/ca-root.crt",
    # FreeBSD (optional symlink)
    # OpenBSD
    "/etc/ssl/cert.pem",
    # Mac OS X
    "/System/Library/OpenSSL/certs/cert.pem",
    ]

The reason I mentioned certifi is because that group of people gave up on managing system paths and just ships a certs file in their source code. This is the same thing being suggested here. So either shipping it with libgrpc core or ship it with the language specific libraries is the equivalent here. (I'm certainly out of my depth in discussing what is secure or correct, I was just trying to explain the things I've seen.)

@soltanmm: we have a list of roots in the grpc source tree that is suitable for publicly trusted certs:
https://github.com/grpc/grpc/blob/master/etc/roots.pem
So what I would be inclined to do is to ship this with the python package as opposed to depend on the roots installed (or not) on the different platforms. Once you have it shipped with the python package, you can do python magic to resolve the path and set the right env var. This is what nodejs did and it looks like it's working well for them. @murgatroid99 can you confirm?

Yes, nodejs did that. But nodejs also ships with the C core bundled instead of linking against it, so I have no expectation that our users would have the certificates installed in /usr/share/grpc/.

That is another little hitch - we require the core package to already be installed. Seeing as how the core determines the expected paths (or, seemingly, not), shouldn't it then be the core package that distributes the required list?

In that case yes, the roots should come for the installed core package and everything should be working seamlessly thanks to the magic that @nicolasnoble put in the path computation of the default roots file path.
What's going on right now if you pass None to the roots arg of the SSL creds? It might just work...

@jboeuf See my comments above (1, 2)

ah yes sorry...
@nicolasnoble, are you the owner of the core package? Does that include the roots? If so, where are they :)? Thanks!

Hmm, so now that we're going to be distributing the core with the Python installation, the certs should somehow be distributed with the installation?
There'd be the (minor, easily worked around) hitch that by design Python packages are not able to install to arbitrary locations. This brings a question: is it a problem thus that the users would never see the certs on their computers in any standard location?

FWIW I recently discovered the httplib2 also distributes its own certs and allows the user to override if they so choose.

3 proposals to move forward in the new core embedded world in order of my
preference:

See if there's a Python library that can dig out the system certs for us
(maybe we get lucky)
Bundle the certs as data with the extension
Bundle the certs as code with the extension

All would need a hook for users to override the defaults, and that hook to
be very clearly documented.
We should probably have a well known HTTPS address for our users to fetch
them also. I could even imagine providing an API to fetch whatever we think
is latest.
On Fri, Dec 18, 2015, 11:39 AM Masood Malekghassemi <
<EMAIL_ADDRESS>wrote:

Hmm, so now that we're going to be distributing the core with the Python
installation, the certs should somehow be distributed with the installation?
There'd be the (minor, easily worked around) hitch that by design Python
packages are not able to install to arbitrary locations. This brings a
question: is it a problem thus that the users would never see the certs on
their computers in any standard location?
—
Reply to this email directly or view it on GitHub
https://github.com/grpc/grpc/issues/4224#issuecomment-165879405.

FYI, this is how we got the etc/roots.pem in the tree.

Get the certs from Mozilla:
https://hg.mozilla.org/mozilla-central/raw-file/tip/security/nss/lib/ckfw/builtins/certdata.txt

Convert them to PEM using
https://github.com/agl/extract-nss-root-certs.

I'm going to do a refresh shortly (it's been one year or so).

@ctiller @nicolasnoble should this be in the upcoming 0.12 release?

@soltanmm is the story on this any different since the merge of #4457?

Fixed in 6d2ef17.


Deprecate Table component

The Table component has been deprecated in favor of MUI X Data Grid in combination with useDataGridRemote. See docs for further information.

I've basically deprecated every part of the public API that's in the src/table folder. This includes some features (e.g, FilterBar) without a direct equivalent for Data Grid. Are we okay with this?

I've basically deprecated every part of the public API that's in the src/table folder. This includes some features (e.g, FilterBar) without a direct equivalent for Data Grid. Are we okay with this?

I think we are okay with this. I guess for new projects the plan is to generally use the filter dialog of the DataGrid.
And since it's only deprecated (without a plan to remove it) it can still be used if necessary.

Be careful when putting links into source code, they get outdated so easy

You're right! I've removed the links from the deprecations in the source code. They don't provide very helpful information anyway. Would be different if there were some sort of migration guide.


Fix the issue #95: Adding email addresses does not work

allauth needs this field "action_add" which will be sent to server with empty value

Seems to work now: https://django-allauth-ui.hodovi.ch/accounts/email/
Thank you!


Newly submitted apps are 404 for some period of time

I've noticed that newly added apps on the "New" page are sometimes 404 for a day or so after being added. It'd be good to avoid this by hiding them until the page is actually available.

This should be gone - worst case needs 15 min for the cache to refresh


How to setup dataset

I cannot figure out how to setup my training data and how to adjust train_svd.yaml. As captions are not needed for SVD i only have a dataset with videos.
videos
|----- video1.mp4
|----- video2.mp4
Like this i have a videos folder, but setting video_dirto "/videos" and "json_path" to "blip_generated.json" does not work. I get this error
Traceback (most recent call last):
  File "D:\MyStuff\Programming\Python\AI\animate-anything\utils\dataset.py", line 182, in load_from_json
    with open(path) as jpath:
         ^^^^^^^^^^
FileNotFoundError: [Errno 2] No such file or directory: 'blip_generated.json'
Non-existant JSON path. Skipping.
Traceback (most recent call last):
  File "D:\MyStuff\Programming\Python\AI\animate-anything\train_svd.py", line 841, in <module>
    main(**args_dict)
  File "D:\MyStuff\Programming\Python\AI\animate-anything\train_svd.py", line 585, in main
    train_dataloader = torch.utils.data.DataLoader(
                       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "d:\MyStuff\Programming\Python\AI\animate-anything\animate_anything\Lib\site-packages\torch\utils\data\dataloader.py", line 351, in __init__
    sampler = RandomSampler(dataset, generator=generator)  # type: ignore[arg-type]
              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "d:\MyStuff\Programming\Python\AI\animate-anything\animate_anything\Lib\site-packages\torch\utils\data\sampler.py", line 107, in __init__
    raise ValueError("num_samples should be a positive integer "
ValueError: num_samples should be a positive integer value, but got num_samples=0

Currently the training requires captions. You can generate captions following instructions in readme.


:sparkles: Improve grapher upserts performance

Improve performance of uploads to R2 by only uploading if checksums of R2 file and uploaded file differ. This is done by setting metadata of R2 object.
This could be further improved by calculating checksum of a dataframe using hash_pandas_object rather than converting to dictionary, then JSON and then checksum from the string.
UPDATE: It's not making such difference, typically reduces grapher upsert speed by 30% or so.

The speedup is really minimal. Uploading files is not a bottleneck, but comparing checksums is.


Add unit tests for TypingDNA connector

Is your suggestion related to an experience? Please describe.

Need to add unit tests for the TypingDNA repo.

Effort BE::3

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


settings: Refactor CSS selectors for tables to be more specific.

First commit is to remove unnecessary div.
Second commit is to refactor the CSS selector for file name.
Third commit is to refactor CSS for users and bots table.

Follow-ups of #30832.

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Merged, thanks @sahil839!


Styles not getting picked up

Somewhat new to React, so this might be a dumb fix...
Below code is me trying to create a simple Vertical Timeline using this package. When I deploy it, the styles don't get picked up?
Full component code here
Projects.js
import React from 'react';
// import '../resume.scss'
import {
  VerticalTimeline,
  VerticalTimelineElement,
} from 'react-vertical-timeline-component';
import 'react-vertical-timeline-component/style.min.css';
import { FaSuitcase, FaSchool, FaStar } from 'react-icons/fa';

const Projects = () => {
  return (
    //
test

    <VerticalTimeline>
      <VerticalTimelineElement
        className="vertical-timeline-element--work"
        contentStyle={{ background: 'rgb(33, 150, 243)', color: '#fff' }}
        contentArrowStyle={{ borderRight: '7px solid  rgb(33, 150, 243)' }}
        date="2011 - present"
        iconStyle={{ background: 'rgb(33, 150, 243)', color: '#fff' }}
        icon={<FaSuitcase />}
      >

Creative Director

Miami, FL

          Creative Direction, User Experience, Visual Design, Project
          Management, Team Leading

      </VerticalTimelineElement>
      <VerticalTimelineElement
        iconStyle={{ background: 'rgb(16, 204, 82)', color: '#fff' }}
        icon={<FaStar />}
      />
    </VerticalTimeline>
  )
}

export default Projects

Browser tools

I found the problem. You use gatsby-plugin-purgecss in your project, so it removes unused styles in your project. You can configure it in your gatsby-config.js. Maybe add an ignore rule.

Great catch @stephane-monnot , thanks! I have it working well now.


Update README.md

Just a trivial test to make sure that https://github.com/ProjectPythiaCookbooks/cookbook-actions/pull/25 didn't break the deployment to GitHub Pages.

Looks good. Closing this.


[GOBBLIN-1333] Bring label size under Apache requirements

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-1333

Description

[x] Here are some details about my PR, including screenshots (if applicable):
Bring label size under Apache requirements.

After last commit, Apache Infra sent the following auto-email:
An error occurred while running github feature in .asf.yaml!:
.asf.yaml: Invalid GitHub label 'data integration' - must be lowercase alphanumerical and <= 35 characters!
So trying to fix it
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
NA

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

@abti there is a merge conflict. Can you check?

Codecov Report

Merging #3171 (3ed3bef) into master (d7fc475) will decrease coverage by 37.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3171       +/-   ##
============================================
- Coverage     46.19%   9.10%   -37.09%
+ Complexity     9752    1723     -8029
============================================
  Files          2014    2014
  Lines         77099   77099
  Branches       8559    8559
============================================
- Hits          35615    7020    -28595
- Misses        38176   69394    +31218
+ Partials       3308     685     -2623

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/apache/gobblin/util/FileUtils.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-4.00%)

...n/java/org/apache/gobblin/fork/CopyableSchema.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...java/org/apache/gobblin/stream/ControlMessage.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/dataset/DatasetResolver.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...va/org/apache/gobblin/converter/EmptyIterable.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...org/apache/gobblin/ack/BasicAckableForTesting.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...n/java/org/apache/gobblin/salesforce/SfConfig.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

.../org/apache/gobblin/yarn/HelixMessageSubTypes.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/cluster/SingleHelixTask.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-5.00%)

.../apache/gobblin/records/ControlMessageHandler.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

... and 1059 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #3171 (3ed3bef) into master (d7fc475) will decrease coverage by 37.08%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #3171       +/-   ##
============================================
- Coverage     46.19%   9.10%   -37.09%
+ Complexity     9752    1723     -8029
============================================
  Files          2014    2014
  Lines         77099   77099
  Branches       8559    8559
============================================
- Hits          35615    7020    -28595
- Misses        38176   69394    +31218
+ Partials       3308     685     -2623

Impacted Files
Coverage Δ
Complexity Δ

...c/main/java/org/apache/gobblin/util/FileUtils.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-4.00%)

...n/java/org/apache/gobblin/fork/CopyableSchema.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-2.00%)

...java/org/apache/gobblin/stream/ControlMessage.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/dataset/DatasetResolver.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...va/org/apache/gobblin/converter/EmptyIterable.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...org/apache/gobblin/ack/BasicAckableForTesting.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-3.00%)

...n/java/org/apache/gobblin/salesforce/SfConfig.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

.../org/apache/gobblin/yarn/HelixMessageSubTypes.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

...va/org/apache/gobblin/cluster/SingleHelixTask.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-5.00%)

.../apache/gobblin/records/ControlMessageHandler.java
0.00% <0.00%> (-100.00%)
0.00% <0.00%> (-1.00%)

... and 1059 more

Continue to review full report at Codecov.

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


Pyscript fails to load script file

It's my first time using Pyscript so maybe I'm missing something obvious.
Error Message

2023-08-16 20:37:35.726 ERROR (MainThread) [custom_components.pyscript.file.solis_control] Exception in </config/pyscript/solis_control.py> line 1:
from homeassistant.helpers.aiohttp_client import async_get_clientsession
^
ModuleNotFoundError: import from homeassistant.helpers.aiohttp_client not allowed

configuration.yml

pyscript: !include pyscript/config.yaml`

pyscript/config.yaml

allow_all_imports: true
hass_is_global: true

Versions

Home Assistant 2023.8.2
Supervisor 2023.08.1
Operating System 10.2
Frontend 20230802.0 - latest

Seems I was missing the allow_all_imports: true config. Even though I was setting this in pyscript/config.yaml it was being ignored as I had previously touched the config from the integration UI...

All working great now!


Adding support for ArgoCD bootstrap apps.

Issue #, if available:
Description of changes:
This pull request adds support for bootstrapping an EKS cluster with multiple workloads via an ArgoCD App of Apps repo.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

LGTM


2-radix sort

Binary bucket sort / 2-Radix sort
Time: O(NLog2N)
Space: O(N)

Hi Dear @NMNDV
Great code
Thanks for your contribution......
Show your support on my youtube channel web Pixar(The institute of coding)
like , share , subscribe
youtube - https://youtu.be/5qnQjgdpFPg
telegram-channel - https://t.me/WebPixar
join our telegram channel for more goodies , swag packs


feat(avoidance): add another same grad filter

Description

Related links

Tests performed

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

NOTE: These two commits will be removed.

https://github.com/autowarefoundation/autoware.universe/pull/3160/commits/55e5e75cccf4c7ef202b7f523a4e8d29bbce0b07
https://github.com/autowarefoundation/autoware.universe/pull/3160/commits/e15a833b2f296e9f2d40b3dd3383f926379d06d2


[BUG]: Subscriptable objects doesn't seem to work

What happened?
Using the example from the documentation:
import enterpriseattack
attack = enterpriseattack.Attack(subscriptable=True)
wizard_spider = attack.groups.get('Wizard Spider')

results in the error
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'list' object has no attribute 'get'

Version
0.1.6
Relevant log output
No response

Hi @bontchev, thanks for reporting. Looks like my build and publish for 0.1.6 was missing the wheel.
Before:

After:

Which means pip would've grabbed 0.1.5 instead, and that feature was not available....
Please try the following:
pip install --no-cache enterpriseattack==0.1.6

Confirm you're using 0.1.6:
>>> import enterpriseattack
>>> enterpriseattack.__version__
'0.1.6'

Then retry:
>>> attack = enterpriseattack.Attack(subscriptable=True)
>>> wizard_spider = attack.groups.get('Wizard Spider')

You are right, pip had actually installed version 0.1.5. I have updated to version 0.1.7 and the problem has disappeared. Closing the issue now.


Added CryptopiaExchangeAPI

Check notes in comments.

I'll be doing a single pull request per new Exchange. This will make it easier to verify.

What is ExchangeTemplateAPI?

That's not supposed to be in there. I use it as a Template for adding new Exchanges. I told git to ignore on the push, but it didn't.

Did you need me to create a new pull request without the .git additions? All that is needed is the ExchangeCryptopiaAPI.cs file and the update to ExchangeNames.
I have other exchanges I can add after this is merged.

Yea let's just do a new one without the .git and template api.


If I use bundler, the project will fail to build.

I run the scikitjs (version: 1.24.0) in my project. and package it by parcel-bundler (version: 1.12.5). When I start the project I get the following error:

if I import the bundle by script tag (CDN), works fine.
The error location: /node_modules/scikitjs/dist/es5/index.js (line: 20). And, Comment out this line of code and it works fine.

Same issue in Webpack

@1093919186  I'm happy to take a look at this if it is still an issue.


When I create ibmdashdb/preview, error

server error 500 when  excute step 6)

Restart Kitematic.

In the Kitematic GUI, click on "My Repos" in the FILTER BY field.

Find the dashDB preview tile and click on the CREATE button.

500 usually means that the container failed to create because of a specific issue with what the container needs to start - Since this is a private repo, I can't see the build, but it's not specific to Kitematic, rather what the container needs to start properly.
Try to run it via the command line and see what the difference is.


CRYPO_128_wrap with 8-byte key

The CRYPTO_128_wrap and CRYPTO_128_unwrap functions are not reversible with an 8 byte key.
CRYPTO_128_wrap will correctly wrap this, however, the unwrap function will not accept the 16-byte wrapped key, as the length is decremented to 8 before testing the length of (inlen < 16).
I also wonder whether the test for (inlen > CRYPTO128_WRAP_MAX) in CRYPTO_128_wrap might better be CRYPTO128_WRAP_MAX-8, so the wrapped length remains less that (1UL << 31), but the logic is still reversible between the wrap and unwrap functions.

I don't quite see your point.  CRYPTO_128_wrap produces inlen+8 result,
the shortest input is 16 bytes, and produces 24 bytes out.
therefore the CRYPTO_128_unwrap reverses the 24 bytes output to produce the 16 byte original
input.

The shortest may correctly be 16 bytes, however CRYPTO_128_WRAP allows for 8 bytes
if ((inlen & 0x7) || (inlen < 8) || (inlen > CRYPTO128_WRAP_MAX))
    return 0;

Should this not be "< 16" in that case

at master the check at CRYPTO_128_wrap looks differently:
https://github.com/openssl/openssl/blob/7f02a0932c8c182bc2f6320748d81e86d650c1a6/crypto/modes/wrap128.c#L55-L56
not sure if that needs to be fixed in 1.0.2

I am using FIPS module, so cannot currently upgrade, so it would be good to have this changed.
Thank you


Fix func count for dtype O with numpy and numba

This fixes #137.
Changes

Generalized the datetime reduce test so it tests all engines.
When func='count' and engine != 'flox', requires_numeric becomes True, which avoid type errors in numba and numpy.

Performance
The "flox" engine is not included because I saw a loss of performance in my (small) tests.
import xarray as xr
from flox.xarray import xarray_reduce

t = xr.DataArray(xr.cftime_range('1900-01-01', periods=100000, freq='H'), dims=('time',))
xarray_reduce(t, t.dt.month, func='count', engine=ENGINE)

With engine 'flox', this code was around 10% slower if I added the numeric conversion.

Yeah I can see it being slower; flox runs isnull to convert to bool directly; the rest will convert to int and then bool.
This looks great, Thanks for catching it so early!


[Suggestion] Move to pipx

Hello, I want to suggest that you look into pipx before spending more time to create and maintain Shell code installer scripts to set up virtual environments and integration into the target platform.
With regards to Systemd I would prefer a User Service instead of putting the service unit in /etc (it should be /usr/local if you want it system wide, I recently stumbled upon work of a former colleague at work and wondered why a certain unit could not be masked, it was because of using /etc). Someone also wrote a tutorial to better integrate Python project  with Systemd: https://github.com/torfsen/python-systemd-tutorial My guess is that something similar should exist for Windows and Mac.
Using older versions of Python with pipx should be less of a hassle from my point of view. I just tried pipx today with Puddletag, which worked quite well even though they say it is intended for CLI apps.

Thanks for the suggestion @lwbt , I will keep it on my todo list.


Port test_datasets_video_utils.py to pytest

Port test_datasets_video_utils.py to pytest. See #4033 for more details.

Seems like the CI failed due to timeout there.

Yeah it's unrelated. Thanks @AnirudhDagar !


Support TypeScript > v4.5

Starting in<EMAIL_ADDRESS>we receive type errors in the following files:

src/AvatarPair.tsx
src/deprecated/ChoiceInputField.tsx
src/deprecated/InputField/InputField.tsx
src/FormControl/FormControl.tsx

Error message
src/AvatarPair.tsx:20:34 - error TS2769: No overload matches this call.
  The last overload gave the following error.
    Argument of type '{ size: number; }' is not assignable to parameter of type 'Partial<unknown> & Attributes'.
      Object literal may only specify known properties, and 'size' does not exist in type 'Partial<unknown> & Attributes'.

20       React.cloneElement(child, {size: 40})
                                    ~~~~~~~~

  node_modules/@types/react/index.d.ts:329:14
    329     function cloneElement<P>(
                     ~~~~~~~~~~~~
    The last overload is declared here.

src/deprecated/ChoiceInputField.tsx:90:19 - error TS2769: No overload matches this call.
  The last overload gave the following error.
    Argument of type '{ id: string; disabled: boolean | undefined; "aria-describedby": string | undefined; }' is not assignable to parameter of type 'Partial<unknown> & Attributes'.
      Object literal may only specify known properties, and 'id' does not exist in type 'Partial<unknown> & Attributes'.

90                   id,
                     ~~

  node_modules/@types/react/index.d.ts:329:14
    329     function cloneElement<P>(
                     ~~~~~~~~~~~~
    The last overload is declared here.

src/deprecated/InputField/InputField.tsx:138:17 - error TS2769: No overload matches this call.
  The last overload gave the following error.
    Argument of type '{ id: string; required: boolean | undefined; disabled: boolean | undefined; "aria-describedby": string; }' is not assignable to parameter of type 'Partial<unknown> & Attributes'.
      Object literal may only specify known properties, and 'id' does not exist in type 'Partial<unknown> & Attributes'.

138                 id,
                    ~~

  node_modules/@types/react/index.d.ts:329:14
    329     function cloneElement<P>(
                     ~~~~~~~~~~~~
    The last overload is declared here.

src/FormControl/FormControl.tsx:145:21 - error TS2769: No overload matches this call.
  The last overload gave the following error.
    Argument of type '{ id: string; disabled: boolean | undefined; "aria-describedby": string | 0 | undefined; }' is not assignable to parameter of type 'Partial<unknown> & Attributes'.
      Object literal may only specify known properties, and 'id' does not exist in type 'Partial<unknown> & Attributes'.

145                     id,
                        ~~

  node_modules/@types/react/index.d.ts:329:14
    329     function cloneElement<P>(
                     ~~~~~~~~~~~~
    The last overload is declared here.

src/FormControl/FormControl.tsx:170:55 - error TS2571: Object is of type 'unknown'.

170               {(React.isValidElement(slots.Label) && !slots.Label.props.visuallyHidden) || slots.Caption ? (
                                                          ~~~~~~~~~~~~~~~~~

src/FormControl/FormControl.tsx:198:19 - error TS2698: Spread types may only be created from object types.

198                   ...InputComponent.props
                      ~~~~~~~~~~~~~~~~~~~~~~~

Found 6 errors in 4 files.

Errors  Files
     1  src/AvatarPair.tsx:20
     1  src/deprecated/ChoiceInputField.tsx:90
     1  src/deprecated/InputField/InputField.tsx:138
     3  src/FormControl/FormControl.tsx:145

The goal of this PR is to support TypeScripts versions >4.5, I believe addressing this will result in support for 4.6, 4.7, and 4.8. Addressing 4.8 would also enable us to unblock the consumer-test range over in: #2290

Closed by https://github.com/primer/react/pull/2360


Upgrade outdated packages to make image more secure

What about add this block?..
RUN set -ex; \
    apk upgrade --no-cache

before
RUN apk add --no-cache openssh-client git

?..

I don't think this is needed.


NewResourceName: AdcsAiaExtension

Description
My team has been working on adding a new DSC resource module to the ActiveDirectoryCSDsc module. This new resource module allows us to add/remove AIA URI extensions on a Certificate Authority and to restart the CertSvc service if required.
Proposed properties
[Required, Description("Specifies the URI location where issuer of certificate is located")] String AiaUriPath[];
[Write, ValueMap{"Present","Absent"}, Values{"Present","Absent"}] String Ensure;
[Key, Description("Specifies the resource is a single instance, the value must be 'Yes'."), ValueMap{"Yes"}, Values{"Yes"}] String IsSingleInstance;
[Write, Description("Specifies if the service should be restarted.")] Boolean RestartService;
    AdcsAIAExtension AddUriPath
    {
        AiaUriPath       = @(
            'http://setAIAPathTest1/Certs/<CATruncatedName>.cer'
            'http://setAIAPathTest2/Certs/<CATruncatedName>.cer'
            'http://setAIAPathTest3/Certs/<CATruncatedName>.cer'
        )
        Ensure           = 'Present'
        IsSingleInstance = 'Yes'
        Restartservice   = $true
    }

Special considerations or limitations
N/A

@PlagueHO  Daniel, this is another resource our team would like to contribute.  Thanks.


train on my  own dataset

If I want to train LaneATT on my own dataset , Do I need to get the coordinate points of the lane lines for labeling?

Yes.

Thank you,Do you have any suggestions on how to get the coordinates

You can annotate the dataset with a tool such as Scalabel.

OK, thank you very much. I'll have a try

Since my dataset is all straight lines, can I label only two or three coordinate points for each lane line?

I think so.


Compile error:  libGRMustache7-iOS.a(GRMustache.o) does not contain bitcode

I have the static library included in my project along with the necessary. My project has Enable Bitcode turned On. Every time i build i get an error:
"You must rebuild it with bitcode enabled (Xcode setting ENABLE_BITCODE), obtain an updated library from the vendor, or disable bitcode for this target. for architecture arm64".
After opening the iOS target settings i do see the "Enable Bitcode" set to Yes. Any thoughts on how to make this work?

Yes. The static library has been built well before bitcode has been introduced. Today, you don't have many options: drop bitcode, or use the manual source embedding technique documented under the Instalation guide.


Cannot Run Camera, Init Error[OTHER]

I am trying to set up the this new camera, but I cannot get passed this init error that it keep giving me the error
RuntimeError: Camera init sequence did not complete.
picamera2.py, line 254
I have the lastest version of Bullseye and have version
VERSION_ID="11"
VERSION="11 (bullseye)"
VERSION_CODENAME=bullseye

python3-picamera2 is already the newest version (0.3.12-2)
Is there a way around this error?

Forgot to leave code
from picamera2 import Picamera2, Preview
import time
picam2 = Picamera2()
camera_config = picam2.create_preview_configuration()
picam2.configure(camera_config)
picam2.start_preview(Preview.QTGL)
picam2.start()
time.sleep(2)
picam2.capture_file("test.jpg")

Solved


MongoDB Support

I came across this project and plan on going through the tutorials for the SQL Server portation using .NET Core 3.1. I have a side project that I work on.
Is there any support for executing stored JavaScript in MongoDB?

I suspect what you're looking for is well outside of the scope of this integration, if you're looking to pull JavaScript from MongoDB documents and then execute it server-side.
Overall, you might find the CosmosDB input/output bindings useful (https://learn.microsoft.com/azure/azure-functions/functions-bindings-cosmosdb-v2) as well as the more general Azure Functions repository (https://github.com/Azure/Azure-Functions/).


improve the title

We have to format the date:

I think 2023-03-01 15:05:02 will be better, right?

@malikovmm let's improve the title.
change the order:
Trivy Report - Alpine:3.14.1 (alpine 3.14.1) - 2023-03-01 15:05:02

#10 fixed it


Module not found: Error: Part "part:@sanity/base/reset-icon" not implemented by any plugins

When try run development I recieve:
C:\squizzy\studio>npm run dev

<EMAIL_ADDRESS>dev C:\squizzy\studio
sanity start

√ Checking configuration files...
| Compiling...webpack built a29f16388662b612a47a in 13436ms
√ Compiling...
Failed to compile.
 I'm seeing this issue now.

<EMAIL_ADDRESS>(part:@sanity/base/sanity-root)

has nobody found the solution???? I am also trying to solve this😭


Support Intl.ListFormat

Introduce types for Intl.ListFormat. The code below shall not generate errors:
new Intl.ListFormat('bg', {type: 'conjunction'}).format(['a','b','c'])

To expedite the triage process, we need everyone to follow the issue template and instructions.
When you clicked "Create New Issue", the issue form was pre-populated with a template and some instructions. We need you to read those instructions completely, follow them, and then fill in all the fields in that template.
We are not able to assist with issues that don't follow the template instructions as they represent a significantly larger amount of work compared to issues which are correctly specified. Thank you for understanding.


Whether to consider supporting HashedModuleIdsPlugin by default

Hello，thank you for the rekit framework！
Would you consider adding HashedModuleIdsPlugin to the default webpack configuration of the rekit framework?
plugins: [
    ...
    new webpack.HashedModuleIdsPlugin(),
].filter(Boolean),

HashedModuleIdsPlugin : This plugin will cause hashes to be based on the relative path of the module, generating a four character string as the module id. Suggested for use in production.
It can stabilize webpack's hash, ensure that the hash value of unchanged files is the same, and make full use of the browser's caching mechanism.

If we pack is creating a bundle, does it really matter? Unless you're saying the chunking would be different... I applaud better use of caching at every level so long as it can be expired at will.


French model

I tested the docker image, it works well, thanks !. Do you have specific instructions to rebuild the image with a French model ?

If you're looking for something pre-packaged, I have a Docker image available with a French model: https://github.com/rhasspy/fr_larynx-siwis
$ docker run -it -p 5002:5002 \
    --device /dev/snd:/dev/snd \
    rhasspy/larynx:fr-siwis-1

This should be a drop-in replacement for the MozillaTTS Docker image :smile:
If you happen to use Home Assistant, I also have a Hass.io addon available: https://github.com/rhasspy/hassio-addons


feat(Meetings): publishing and getting outgoing talk schedules

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


qpdfview: fix build pin poppler

Description of changes
maybe fixes https://github.com/NixOS/nixpkgs/issues/243015
I haven't see pinning of older versions much in nixpkgs, so curious to see what feedback i get
cc @jtojnar @gador
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Introducing another versions of a software is typically only done as a last resort.
In this case, https://github.com/NixOS/nixpkgs/pull/243037 will probably be a better solution.

closing in favor of https://github.com/NixOS/nixpkgs/pull/243037


Merge the v7.5.0-preview.3 release branch to GitHub master branch

PR Summary
The v7.5.0-preview.3 release completed. Merge the release branch to GitHub master branch.

Some commits need to be backported to servicing branches as they are related to fixing the build/packaging/release pipelines.


'utf8' codec can't decode

With the current version "Tvheadend 4.1-459 ~ g987dc11" I get back the error "JSON Request failed: 'utf8' codec can not decode byte". And the API version I had to change again to 18.
Can someone help me?

Has done first. New update of Tvheadend has eliminated the problem


Question: Comparing Freshmvvm features

Sorry if these are dummy question. As a freshmvvm user, I decided to use this library for my new project. Concept looks like pretty exciting but I am having difficulties to understand some things and I would like to ask a few questions. I think that there are many people like me switching from freshmvvm to xamvvm. So it can be also useful

Freshmvvm has viewmodel function to override ViewIsappearing and disappearing, I heavily used them. I couldnt see something similar. Should we just use standard XF ones?
Freshmvvm has 2 functions. Init and ReverseInit. Similar exists? ReverseInit is useful if a page is popped with data and retrieve in the previous page in the stack.

Hi @EmilAlipiev,

Freshmvvm has viewmodel function to override ViewIsappearing and disappearing, I heavily used them. I couldnt see something similar. Should we just use standard XF ones?

Yes, but you need to use prerelease nuget (It'll be stable soon). Just implement IPageVisibilityChange or override methods which are already implemented in BasePageModel class. https://github.com/xamvvm/xamvvm/blob/master/source/Xamvvm.Core/Interfaces/IPageVisibilityChange.cs

Freshmvvm has 2 functions. Init and ReverseInit. Similar exists? ReverseInit is useful if a page is popped with data and retrieve in the previous page in the stack.

I prefer strongly typed Frameworks (which Xamvvm is). It allows you to create your own strongly typed methods for such cases. Simple example here:
public class YourPageModel : BasePageModel
{
    Action _onSuccessAction;

    public YourPageModel()
    {
        SuccessCommand = new BaseCommand()(obj) => _onSuccessAction?.Invoke());
    }

    public void Init(SomeStrongTypeNeeded obj, Action onSuccessAction)
    {
        Model = obj;
        _onSuccessAction = onSuccessAction;
    }

    public SomeStrongTypeNeeded Model { get; private set; }

    public IBaseCommand SuccessCommand { get; set; }
}

//Then use it like that:

this.GetPageFromCache<YourPageModel>(v => v.Init(someStrongTypeNeededInstance), () => { //TODO do something } );


HOLD FOR 2024 - Changes to Playoff Schedule

https://www.firstinspires.org/robotics/frc/blog/2023-double-elimination-2023-data-and-updates#:~:text=Removes the first two breaks entirely

Actually not holding because MIFl2 is using the new schedule


Allow css props with no values to be shown

Without this PR, when setting a single CSS Prop with no values, it doesn't show in the CSS Custom Properties panel.
Replication steps:

Add the following code in parameters of your story:

cssprops: {
  "css-custom-property-4": {
    // Allows us to set document a CSS Custom Property without applying anything,
    description: "The value can be left intentionally blank",
  }
}

🌟 Expected result:

🥵 Actual result:

Thanks for the contrib, apologies on the delay 🙏


Snow albedo change model

Hi everyone,
Dan Riley and Chris Pike developed this model to calculate albedo change from snow depths and temperatures.
Here is the reference to their conference proceeding with a full paper currently in submission.
10.1109/PVSC48317.2022.9938897
The model is already making it's way into pvlib-matlab. I'm currently working on a python version to be included soon.

I can't access the paper, could you type up the equation and post it here?

I can't access the paper, could you type up the equation and post it here?

Or post a preprint somewhere?

I'll see what I can do about getting a preprint available.

I can't access the paper, could you type up the equation and post it here?

Or post a preprint somewhere?

I'll see what I can do about getting a preprint available.

Our paper was published in MDPI Solar. Here's the citation.
Pike_ C, Riley D, Toal H, Burnham L. Presenting a Model to Predict Changing Snow Albedo for Improving Photovoltaic Performance Simulation. Solar. 2024; 4(3):422-439. https://doi.org/10.3390/solar4030019


The catchment/waterbody/COMID mapping and relationships need cleanup

https://github.com/NOAA-OWP/ngen/blob/874110b041036118a0811b270de25a5dbcb17d6c/src/NGen.cpp#L152
Here is an example where a waterbody ID is being treated as a catchment id in order to associate a COMID with a catchment to read its GIUH data.  This is technically the wrong semantics, and should not propagate further.
One potential solution is to add the catchment id to the crosswalk.json file.  This has been raised on hygeo as a potential fix.
https://github.com/dblodgett-usgs/hygeo/issues/15

Note that there is a "realized_catchment" attribute on the waterbody_data.geojson features that should provide this information.

@hellkite500 We need to step back from this a bit. I'm realizing I missed a really important nuance regarding waterbodies and flowpaths that I need to correct. I summarized the issue here. https://github.com/dblodgett-usgs/hygeo/issues/21
Working to clean this up now. At the end of the day, I think this means that we will no longer mix waterbody ids into catchment stuff at all. For the case where we have hydrologic routing, we shouldn't be thinking of the conveyance as a waterbody at all -- it is the flowpath realization of the catchment.
I think this may mean that the flowpaths may actually get the catchment id and as we go forward and need distinct waterbody features we will add a new identifier scheme that will relate to catchments at hydrologic locations along flowpaths.

I heavily rewrote the README here: https://github.com/dblodgett-usgs/hygeo and added this uml:

Additionally, I pretty heavily refactored the naming conventions in hygeo to focus on flowpath rather than waterbody. We haven't tackled waterbodies yet -- having only done flowpaths.
The instance you point out above, @hellkite500 -- will now be cat-88 when we get things updated over here.
Let's talk more about what testing data we need to update and what we should do otherwise to make sure these changes get updated in this repo.


Update workspace-data-exfiltration-protection.md

The original sentence is a bit incorrect and misleading. Replace it with a more clear description.

@WilliamDAssafMSFT
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @MaxMeng1985, thanks for taking the time to create this PR. While this is in review internally, please accept the commit I've suggested. Will update.

Hi @MaxMeng1985, thanks for taking the time to create this PR. While this is in review internally, please accept the commit I've suggested. Will update.

Thanks for checking, I accepted the suggested commit.

Thanks @MaxMeng1985 for taking the time to submit this PR! Your commit will be merged and this article update is in flight. After the updated article is published, GitHub will list you as an article contributor.
Thanks again for this feedback, your suggestions help improve our documentation.
#sign-off


Extract all hook-related files when running subkit/param hooks

This should fix the issue seen in https://github.com/genesis-community/cf-genesis-kit/issues/2


Fix AttributeError on closing of django runserver

When closing Django runserver, valkey client consistently throws the AttributeError in the function del
This patch fixes that problem
Pull Request check-list

[ ] Do tests and lints pass with this change?
[ ] Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
[ ] Is there an example added to the examples folder (if applicable)?

Description of change

Hey! Thanks for your contribution.
The code unfortunately doesn't look like a fix but rather like a workaround for an existing problem.
Could you please create an issue with the detailed description of the problem? What exactly do you do to get this exception thrown and what is the stack trace?
Thank you in advance!

Issue #158

Thank you! I'll have to close this PR in order to fix the root cause of the issue.


Add support for query parameters in the RTSP path within the configuration file

Describe the feature
Which version are you using?
v1.9.3
Feature Request
At present, mediamtx supports query parameters for RTSP paths only when they are defined outside of the configuration file (mediamtx.yml). Attempting to add an RTSP path with a query parameter in the configuration file results in an error:
ERR: invalid path name 'Streaming/Channels/102?transportmode=unicast&profile=Profile_3': can contain only alphanumeric characters, underscore, dot, tilde, minus or slash.
Could support for query parameters be added to the path field within the configuration file as well?

Hello, if what you want is specifying both path name and query parameters in this way:
paths:
  'mypath/Streaming/Channels/102?transportmode=unicast&profile=Profile_3':
    source: mysource

This is technically possible to implement but i don't think it's a good idea to do that - the entire server is built on the concept of using path names as keys, furthermore query parameters are often used to propagate dynamic and sensitive data (credentials) through reverse proxies and identity servers, and binding them to specific values would cause troubles to the vast majority of users.
However, it seems to me that your aim is simulating a HikVision camera, therefore my advice is to use gortsplib rather than MediaMTX and build a custom RTSP server that checks for path name and query parameters and act accordingly.
A server example is here:
https://github.com/bluenviron/gortsplib/blob/main/examples/server/main.go
A proxy example is here:
https://github.com/bluenviron/gortsplib/blob/main/examples/proxy/main.go


Improving blind_nosql

Hello,
Here's a proposal for improvement !

I removed the data_size variable, instead the loop stops when no more characters are found.
Benefits : It can find the full value without knowing its length + Avoid infinite loops.

I also added the possibility to choose a custom charset. If left blank its set to abcdefghijklmnopqrstuvwxyz1234567890.

Best Regards

Made mistake. Closed.


The custom field type is missing the "Change" event

Hello.
The custom field type in my form, there is no "Change" event. Therefore, the form when sending does not see the value in this field and tells me "Required field".
https://jsfiddle.net/antitim/yL47eope/
Thank you.

I recompiled JS, download latest from /dist folder and it will be working. Thanks.

Hello @vitmalina . Is there a way to require valid field before submitting the form and prevent the form submission with an invalid field?

Yes. (1) attach to onValidate event and the form will send you event.errors array with all errors found by the form. (2) If you add your error object to this array, even if it was empty at first, those will be considered errors. (3) form's save fill not proceed if there is an error (array not empty)
You can see the structure of the error object if you catch one that form sends. Simply mark one field as required and see what is in error array when event comes.

@vitmalina Thank you.


adding DeepMind x UCL Reinforcement Learning cource

What does this PR do?
Add Resource(s)
For resources
Description
DeepMind x University College London's - Introduction to Reinforcement Learning Course.
The website links to YouTube videos of the course lectures, course lecture notes as PDFs and contains the examinations and assessments from the course.
Why is this valuable (or not)?
The course is very detailed and applicable. It teaches the algorithmic basics of RL without requiring tools or programming knowledge.
How do we know it's really free?
I've taken the course! It's free. You don't need to enroll or sign up. All resources are available free. 😄
For book lists, is it a book? For course lists, is it a course? etc.
It's a course 👍
Checklist:

[x] Not a duplicate
[x] Included author(s) if appropriate
[x] Lists are in alphabetical order
[x] Needed indications added (PDF, access notes, under construction)

Thanks, and welcome to free-programming-books!


Update baseline-zone-redundant-content.md - Increase readability, #4095

Update baseline-zone-redundant-content.md - Increase readability, #4095

@madsd
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

LGTM #sign-off

@ShannonLeavitt it does not look like I can sign it off.


fix(Makefile): fixed script to push the image

fix script which assigns the image name, which will be pushed to docker hub and quay
Signed-off-by: Akhil Mohan<EMAIL_ADDRESS>

Codecov Report

Merging #320 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #320   +/-   ##
=======================================
  Coverage   52.09%   52.09%
=======================================
  Files          51       51
  Lines        2405     2405
=======================================
  Hits         1253     1253
  Misses       1052     1052
  Partials      100      100

Continue to review full report at Codecov.

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


skip integration tests during unit test build

Impact

[ ] Bug fix (non-breaking change which fixes expected existing functionality)
[ ] Enhancement/New feature (adds functionality without impacting existing logic)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Things to be aware of

Things to worry about

Additional Context

https://github.com/liquibase/liquibase/actions/runs/7616105116


PR: Add an "Fit plots to window" option to the plots pane

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added unit test(s) covering the changes (if testable)
[x] Included a screenshot (if affecting the UI)

While using the plotting plugin, I found it annoying to have to zoom in and out the image frequently when testing different sizes of image. Feels like an Auto fit plotting option is necessary.
So I tried adding one, like (default to False, aka, unchecked):

An example of the original behavior when dealing with different sizes of images (with the auto_fit_plotting option set to false):

Compared to the new behavior when set to true:

Example code:
from matplotlib.pyplot import figure, text
def test_plot(w, h):
    figure(figsize=(w, h))
    text(.5, .5, f'figsize={w}, {h}', ha='center', fontsize=2.5*w)

test_plot(4, 3)
test_plot(15, 7)
test_plot(8, 12)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: gepcel

@gepcel thanks for working on this. Looks pretty useful :-)
@jnsebgosselin could you chime in with your thoughts?

@goanpeca Sure, I'll take a look at it as soon as I can.

@goanpeca I'll let @CAM-Gerlach handle this, thanks.

@CAM-Gerlach I've updated some commits. Now:

[x] Affirmed the DCO.
[x] Added a test. I do need guidance on this, added some code for a start.
[x] The option is default now.
[ ]   Shouldn't we make it a persistent config setting so we don't have to set it every time
[x] Changes based on code comments.

As to the "persistent config setting" suggestion, I'm not sure I understand you correctly, now:

After setting Fit plots to windows  Checked/Unchecked, the state will be remembered after closing and reopenning spyder, until the next change or until start spyder with --safe-mode.
The Fit plots to windows state change is not one plot specific, it applies to all existing and all after comming plots in the

After setting Fit plots to windows Checked/Unchecked, the state will be remembered after closing and reopenning spyder, until the next change or until start spyder with --safe-mode.

This is what I meant, thanks. I thought it wasn't but it was probably because I was switching in and out of --safe-mode, Windows and Linux and lost track.

Other than your call-out about proper plot window width that perhaps @jnsebgosselin can provide insight on, looks like you addressed all my suggestions, thanks!

Hello @gepcel! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file spyder/plugins/plots/widgets/tests/test_plots_widgets.py:

Line 382:1: W293 blank line contains whitespace

I think I have figured out the accurate plot size when scaling image.

I'll try to take a look at this PR a review it before the end of the week if it is possible to wait a little before merging this.

@gepcel, tests are failing for Python 2.

@ccordoba12 good to go?

We're waiting for @jnsebgosselin's review.

@jnsebgosselin :-p ?

@goanpeca I've tested it locally and I agree with the way this feature is implementer as it is. I've also reviewed the code and everything LGTM. I apologize for taking so long to review this and thanks for waiting, this is much appreciated.
@gepcel Thanks for the awesome work on this, this is going to be very handy!

Thanks @jnsebgosselin for the review! Merging then.


Not working on iOS with mov files

Current behavior
Its not working on IOS with .mov file format at all.
Platform

[X] iOS

React Native Version
react-native: "0.64.2"
React Native Compressor Version
react-native-compressor: "0.5.17"

@Bayramito did you find any solution to this?
it is quite sad indeed :/

@Bayramito Yes indeed

@Bayramito Yes indeed
I used a converter in a promisr and used mp4.

but it might work on the newest version i think.

Any update it's still not working.
& also application crashes whenever we try to compress mov file.

🙏 sorry for a long ago I was not maintaining due to the huge office workload, hopefully, I am going to start fixing bugs on a weekend basis,
I will fix at least 2 issues on the weekend, for priority bugs you can join the discord channel https://discord.gg/dtYzk8sp where we can discuss which bug is most important

We need an urgent fix for this, the app crashes every single time

@vincentzierigen it is working on my side with mov file, can you send me your mov file so i can reproduce this issue on my side and fix it

@vincentzierigen did you check latest version?

@vincentzierigen did you check latest version?

I just did and now it works! Thank you so much!


Black rectangle appears on screen on project run

Godot version:
58f5c2b

OS/device including version:
macOS Catalina 10.15.5

Issue description:
Every time I run a project in the editor after making a change, a black rectangle that acts like a window appears in place of the "saving scene" popup. The rectangle doesn't disappear until the project is run again.

Steps to reproduce:
1: modify a scene
2: run the project
Minimal reproduction project:
Happens to me on any project, including newly created ones.

Duplicate of #39620.


Peer Review 2

Summary
The paper investigates the role of various financial knowledge and skill measures in determining financial well-being. It uses multiple predictors, including the Financial Well-Being Score (FWBscore), the Financial Skill Score (FSscore), and two financial knowledge scales: Lusardi and Mitchell's Financial Knowledge Scale (LMscore) and Knoll and Houts' Financial Knowledge Scale (KHscore). The analysis demonstrates the importance of financial skills and knowledge in shaping overall financial well-being, with particular emphasis on how different financial literacy scales contribute to a comprehensive understanding of financial outcomes. The findings highlight the significant role of financial knowledge in improving individual financial well-being, offering valuable insights for future financial education programs.
Strong Positive Points

The visualizations, particularly the plots, are intuitive and significantly enhance the understanding of the analysis.
The linear regression model is written properly.

Critical Improvements Needed

The Introduction, Data, and Model sections still need to be completed.
Complete the sketches and ensure the LLM is updated in the README.
The model file and related scripts are currently missing.

Suggestions for Improvement

You've completed most of the graphs and plots, just remember to include tables in the model section for data validation.
Try to add a plot in the Results section.
No needs to view the dataset in your paper unless you specify the full name of each column in measurement section
Reference section can start a new page

Evaluation
R/Python Cited: 1/1 cited
Data Cited: 0/1 missing
Class Paper: 1/1
LLM Documentation: 0/1
Title: 1/2. could be more informative y adding subtitle
Author, Date, and Repo: 2/2. present
Abstract: 0/4 missing
Introduction: 0/4 missing
Estimand: 0/1 missing
Data: 5/10. lack of explanation
Measurement: 0/4. missing
Model: 4/10. lack of justification, validation and checking
Results: 2/10 lack of text associated
Discussion: 0/10. missing
Prose: 0/6. lack of text
Cross-references: 1/1. present
Captions: 2/2. present
Graphs/Tables/etc.: 4/4. graphs and tables are informative
Surveys, Sampling, and Observational Data Appendix: 0/10. missing
Referencing: 4/4 present
Commits: 2/2
Sketches: 0/2. missing
Simulation: 4/4. great
Tests: 4/4
Parquet: 1/1. saved
Reproducible Workflow: 2/4. text explanation is needed
Enhancements: 0/4. missing
Miscellaneous: 1/3
Final Grade
41 of 112 ( LLM should be documented properly, or it's gonna be 0)
Other Comments
The graphs look good! keep working on the literal part

Thank you for your comment!


Tree Details Page [execution list]: add hover in status column in the table header

Add a (?) question mark right to the Status label in the execution list for Boots and Tests tabs.
Message for the Builds List table:
Success - valid builds
Failed - invalid builds
Inconclusive - builds with unknown status

Message for the Boots List table:
Success - boots with PASS status
Failed - boots with FAIL status
Inconclusive - groups boots with ERROR, MISS, SKIP and DONE statuses

Inconclusive boot statuses are relevant for continuous integration system maintainers.

Message for the Tests List table:
Success - tests with PASS status
Failed - tests with FAIL status
Inconclusive - groups tests with ERROR, MISS, SKIP and DONE statuses

Inconclusive tests contains status relevant for continuous integration system maintainers.

FYI Tests might also have unknown (null) status

For builds:
Success - builds completed successfully
Failed - builds failed
Inconclusive - builds with unknown status, including ongoing builds that didn't finish yet.

For boots
Success - boots with PASS status
Failed - boots with FAIL status
Inconclusive - boot test concluded with inconclusive results such as infrastructure errors.

Inconclusive groups boots with ERROR, MISS, SKIP and DONE statuses defined by KCIDB.

For tests I'd add do the same as I did for boots.


Update wording of ethnicity questions

Context
This updates the wording of the Ethnicity questions to match that in the Equality information pattern in the GOV.UK Design System, which is in turn based on the Ethnicity harmonised standard from the Government Analysis Function.
The changes are mostly ordering changes, ie the ordering of the groups in the options lists, and the ordering within the wording (eg "Irish Traveller or Gypsy" becomes "Gypsy or Irish Traveller").
"Another" has also changed to "Any other" and "Another Mixed background" has changed to "Any other mixed or multiple ethnic background".
Things to check

[ ] Does this beak anything to do with the existing stored values in the database?

@carlosmartinez @tomas-stefano this isn't critical to get in for the start of the cycle. Will these wording changes cause any issues with existing data?

@tomas-stefano @carlosmartinez this isn't urgent and can get changed after the new cycle has opened. I'm a bit nervous that it'll break things with existing data, as don’t we store ethnicity as strings in the database?

@carlosmartinez @tomas-stefano this isn't critical to get in for the start of the cycle. Will these wording changes cause any issues with existing data?

thanks @frankieroberto. suggest we label this as do-not-merge for now - worth spending a bit of time investigating possible impact.

@frankieroberto @carlosmartinez It has been 24 days since last discussion, do we have anything further planned with this PR?

I think best thing is to close this PR and link it in a card in the backlog, as it requires a bit of non-trivial dev work.

Yeah I'm happy to close this if we can add a backlog card for it. Presumably the dev work would be to update either write a data migration to update all the old values, or figure out how to denormalise all this somehow?

@frankieroberto have made a trello card for the dev work. ok to close this PR for the moment?

@frankieroberto @carlosmartinez Are we keeping this branch around?

@elliotcm it can probably be deleted as I think GitHub still keeps a record?
Ideally we'd still do this at some point but it was more complicated than it seemed and isn't a priority.

Alright I'll leave it around for the mo.


adding placeholder on input search

now you can add a placeholder for search input
Example:
<select class="selectpicker" data-live-search="true" data-live-search-title="Search"></select>

very good, you are pro

Good property. Add documentaion this property


Adds trace support for Postgresql jdbc driver 9.4.1207+

Also fixes some issues with the plugin for other postgresql jdbc driver versions
resolves #2239

Codecov Report

Merging #3159 into master will increase coverage by 0.02%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3159      +/-   ##
==========================================
+ Coverage   38.72%   38.75%   +0.02%
==========================================
  Files        2014     2011       -3
  Lines       66469    66447      -22
  Branches     9295     9271      -24
==========================================
+ Hits        25743    25753      +10
+ Misses      38131    38099      -32
  Partials     2595     2595

Impacted Files
Coverage Δ

...point/plugin/jdbc/postgresql/PostgreSqlPlugin.java
0% <0%> (ø)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
75.37% <0%> (-0.76%)
:arrow_down:

...m/navercorp/pinpoint/test/OrderedSpanRecorder.java
74.03% <0%> (+1.92%)
:arrow_up:

...om/navercorp/pinpoint/rpc/codec/PacketDecoder.java
75.9% <0%> (+7.22%)
:arrow_up:

.../com/navercorp/pinpoint/rpc/packet/PongPacket.java
64.28% <0%> (+14.28%)
:arrow_up:

...npoint/rpc/server/SimpleServerMessageListener.java
100% <0%> (+20%)
:arrow_up:

Continue to review full report at Codecov.

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

Hi,
I need this fix. Are there master nightly build  ?
Or I need to build it by myself ?
Thanks
Olivier

@ofauchon
There isn't a separate build for the master branch, so you'll have to clone and build it.
Feel free to open up a new issue if anything doesn't work if you decide to try it out.
Thanks!

Can I use a fresh build for the agent, and mainstream 1.6.2 server build ?
Or I need to upgrade both server & agents ?

@ofauchon With the current master code, it should work, but I can't guarantee it.
Connecting a lower version agent to a higher version collector works, but the other way around is not always guaranteed.


[SPARK-45969][DOCS] Document configuration change of executor failure tracker

What changes were proposed in this pull request?

It's a follow-up of SPARK-41210 (use a new JIRA ticket because it was released in 3.5.0), this PR updates docs/migration guide about configuration change of executor failure tracker
Why are the changes needed?

Docs update is missing in previous changes, also as requested https://github.com/apache/spark/commit/40872e9a094f8459b0b6f626937ced48a8d98efb#r132516892 by @tgravescs
Does this PR introduce any user-facing change?

Yes, docs changed
How was this patch tested?

Review
Was this patch authored or co-authored using generative AI tooling?

No

What is the default value - I would have expected to see one about yarn and one about k8s?

The default value of spark.executor.maxNumFailures is dynamically calculated in the runtime.
  // Default to twice the number of executors (twice the maximum number of executors if dynamic
  // allocation is enabled), with a minimum of 3.
  def maxNumExecutorFailures(sparkConf: SparkConf): Int = {
    val effectiveNumExecutors =
      if (Utils.isStreamingDynamicAllocationEnabled(sparkConf)) {
        sparkConf.get(STREAMING_DYN_ALLOCATION_MAX_EXECUTORS)
      } else if (Utils.isDynamicAllocationEnabled(sparkConf)) {
        sparkConf.get(DYN_ALLOCATION_MAX_EXECUTORS)
      } else {
        sparkConf.get(EXECUTOR_INSTANCES).getOrElse(0)
      }
    // By default, effectiveNumExecutors is Int.MaxValue if dynamic allocation is enabled. We need
    // avoid the integer overflow here.
    val defaultMaxNumExecutorFailures = math.max(3,
      if (effectiveNumExecutors > Int.MaxValue / 2) Int.MaxValue else 2 * effectiveNumExecutors)

    sparkConf.get(MAX_EXECUTOR_FAILURES).getOrElse(defaultMaxNumExecutorFailures)
  }

The default value of spark.executor.failuresValidityInterval is -1
  private val executorFailuresValidityInterval =
    sparkConf.get(config.EXECUTOR_ATTEMPT_FAILURE_VALIDITY_INTERVAL_MS).getOrElse(-1L)

The code uses createOptional to match the previous behavior, but I'm open to change it if the reviewer thinks necessary.
  private[spark] val MAX_EXECUTOR_FAILURES = ConfigBuilder("spark.yarn.max.executor.failures")
    .version("1.0.0")
    .intConf
    .createOptional

cc @tgravescs since this is required by him

@tgravescs thanks for your advice, updated

@tgravescs thanks, suggestion applied, also update the ConfigEntry definition for consistence


Ivan/pol 2365 all modifications to the envoy core

Conn manager part: https://github.com/polyscale/connectionmanager/pull/244


Test github codacy, don't merge.

test codacy

Codecov Report

Merging #400 (bacfef2) into master (8634481) will increase coverage by 67.75%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master     #400       +/-   ##
=============================================
+ Coverage      8.70%   76.46%   +67.75%
- Complexity       47      576      +529
=============================================
  Files            82       82
  Lines          2596     2596
  Branches        188      188
=============================================
+ Hits            226     1985     +1759
+ Misses         2350      489     -1861
- Partials         20      122      +102

Impacted Files
Coverage Δ

...lsar/handlers/mqtt/messages/MqttPropertyUtils.java
91.66% <ø> (+91.66%)
:arrow_up:

...qtt/exception/handler/MqttPubExceptionHandler.java
6.66% <0.00%> (+6.66%)
:arrow_up:

...qtt/exception/handler/MqttSubExceptionHandler.java
7.14% <0.00%> (+7.14%)
:arrow_up:

...e/pulsar/handlers/mqtt/support/MessageBuilder.java
20.83% <0.00%> (+20.83%)
:arrow_up:

.../pulsar/handlers/mqtt/MQTTServerConfiguration.java
100.00% <0.00%> (+25.00%)
:arrow_up:

...t/exception/handler/MqttUnsubExceptionHandler.java
27.77% <0.00%> (+27.77%)
:arrow_up:

...tt/exception/handler/MqttConnExceptionHandler.java
33.33% <0.00%> (+33.33%)
:arrow_up:

...exception/handler/MqttDisConnExceptionHandler.java
33.33% <0.00%> (+33.33%)
:arrow_up:

...e/pulsar/handlers/mqtt/utils/PulsarTopicUtils.java
74.54% <0.00%> (+37.27%)
:arrow_up:

...ar/handlers/mqtt/support/MQTTMetricsCollector.java
99.01% <0.00%> (+44.11%)
:arrow_up:

... and 64 more

Continue to review full report at Codecov.

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

https://github.community/t/expected-waiting-for-status-to-be-reported/18001
This article may can help with this issue.


Feature Request: rework SetConsoleTextAttribute() to SetConsoleMode() + ANSI-Sequences

Regarding SetConsoleTextAttribute(GetStdHandle(STD_OUTPUT_HANDLE), winColor) ...
This kind of Windows Console Color Codes are subject to stopped support in some (midterm) future.
Please take a look into the outdating warming
Looks like MicroSoft goes a step towards Linux/Unix behaviour (or just reactivates old MS-DOS 4.01 ANSI-sequences via a new API), read here: how to rework
Ready-to-adopt sample code is on the page, just search for "wprintf(L"\x1b[31mThis text has a red foreground using SGR.31.\r\n");"
Anyway, there is no urgency in doing this right now.

OK I know that. This should be changed.
But currently I cannot reach my Windows PC for some time (school's started), so please provide a PR if you like. I won't add those codes unless I have really tested them on Windows.

Hi there! This is fixed in the latest version of v1.5.3.
Issue closed.

Hi,
thank you for the fix and for the notification.
Recently I am involved in many hours of extra work at my daytime job, so I hope to continue my project in October or so.
Yours,
Mo

Am 28. Juni 2023, 04:53, um 04:53, Zheheng Zhang @.***> schrieb:

Hi there! This is fixed in the latest version of v1.5.3.
Issue closed.
--
Reply to this email directly or view it on GitHub:
https://github.com/zhangzheheng12345/Lightest/issues/13#issuecomment-1610595011
You are receiving this because you authored the thread.
Message ID:
@.***>


cloud-init does not complete installs

First, awesome repo @matt-FFFFFF ... Found this extremely helpful when learning how to deploy VMSS ala Bicep.
What I've come to realize is the ado pipeline will pick-up an instance/agent before cloud-init has completed the post-deployment installations. I thought about incorporating cloud-init status --wait to ensure the pipeline does not progress until cloud-init completes the post-deployment tasks, but that's pretty messy. Every pipeline would need this noise... And the next stage could pick up a totally different instance/agent (let's say one that has not completed the cloud-init).
While not directly related to this repo (everything works beautifully), have you run into this issue before? Curious what your solution was to this dilemma. Thanks for your contributions!

Hey @jeremycolson
Glad you found it useful 😄
It's been a while since I used this but I seem to remember a race condition where the cloud init installation took longer than the registration with ADO.
This was a one time issue though IIRC? Once it has provisioned then the VM sits there ready to go for next time.
Let me know if this chimes with you or if I've missed something.


help: Want to understand how i can give custom names to the multiple sitemap generated from dynamic data feteched with defaultSitemapsChunkSize value set to 500 urls in each sitemap

📚 What are you trying to do?
I am not sure how to set custom names to multiple sitemap xml files generated
🔍 What have you tried?
No response
ℹ Additional context
No response

same problem...

This is not currently supported, please track here https://github.com/nuxt-modules/sitemap/issues/265.


Undoing delete of a container selection fails

Reproduce:

Select multiple paragraphs
hit backspace
hit ctrl+z

I stumbled across the same bug and the reason seems to be that while inverting the changes for pushing on undo stack, the before and after selections get detached from the document. Once restored, this works fine.
If this fix looks acceptable, I can send a PR.

This seems to be fixed.


Multiple query parameters with same key aren't handled properly

Specify this route in your code:
Router.route('/testroute', function() {
  console.log(this.params.query)
}, {
  where: 'server'
});

Now load: localhost:3000/testroute?param=1&param=2
Expected result in console:
{
   "param": ["1", "2"]
}

Actual result in console:
{
   "param": "2"
}

Hey @bradvogel this is expected behavior, you need to pass the query params like param[]=1&param[]=2 for them to be treated as an array

Hey @bradvogel this is expected behavior, you need to pass the query params like param[]=1&param[]=2 for them to be treated as an array

That isn't how the browser works though. The following form:
    <form action="/" method="get">
        <input name="param">
        <input name="param">
        <input type="submit">
    </form>

will submit to the url: /?param=one&param=two

That isn't how the browser works though. The following form:
    <form action="/" method="get">
        <input name="param">
        <input name="param">
        <input type="submit">
    </form>

will submit to the url: /?param=one&param=two

just rename the field names to param[]

just rename the field names to param[]

Yes, I see that's a workaround, but this is still a bug. The router should support arbitrarily name multiple parameters, as the spec allows.

Yes, I see that's a workaround, but this is still a bug. The router should support arbitrarily name multiple parameters, as the spec allows.

I wouldn't say it's a bug. The router/whatever can choose to handle them however they wish and the convention of automatically coercing field names with square brackets into arrays is quite common and it makes handling things much less complicated. I know that Rails and Django (and pretty sure PHP) do the same thing.

I wouldn't say it's a bug. The router/whatever can choose to handle them however they wish and the convention of automatically coercing field names with square brackets into arrays is quite common and it makes handling things much less complicated. I know that Rails and Django (and pretty sure PHP) do the same thing.

Here's the issue @tmeasday opened with some reasoning behind the choice https://github.com/EventedMind/iron-url/issues/1

Here's the issue @tmeasday opened with some reasoning behind the choice https://github.com/EventedMind/iron-url/issues/1

Ok, thanks for the help. I filed https://github.com/EventedMind/iron-url/issues/8 as a feature request. Closing this. Thanks!

Ok, thanks for the help. I filed https://github.com/EventedMind/iron-url/issues/8 as a feature request. Closing this. Thanks!


Site is not crawler friendly

No description when I search for "try dot net" in Bing. Which will put some folks off trying it.

Google the same

I assume this is because of https://try.dot.net/robots.txt blanket disallow:
User-agent: *
Disallow: /

Perhaps there's a way to "Allow" some file that the search engines would use for a description: or possibly you could just stop disallowing them?

The site is actively being worked on and I will fix this.  Thanks for brining it to my attention.

Thanks @danmosemsft! The TDN home page is not yet super user friendly, so we're not actively driving traffic to it yet. Most of our users today access TDN through partner sites like docs.microsoft.com. We'll be improving discoverability as the home page's purpose becomes more obvious over coming sprints :)

Still not fixed -- is it worth trying a few cheap experiments with edits to the robots.txt? I appreciate your point that you are still working on the homepage, but it seems a pity to put off folks searching.

We have a new homepage https://www.microsoft.com/net/platform/try-dotnet  hopefully this will solve this soon.

The current landing page wasn't entirely intended to be the landing page for search results. Along the with new homepage, the fix for this is on its way to production: https://github.com/dotnet/try/issues/147.


Allow form data modifying before submission with enhance

Describe the problem
Based on this line, I just found out that if I change form values in the submit() function to enhance then the old values are submitted. e.g.
<script>
  function submit({form}) {
    form.querySelector('input').value = "bar"
  }
</script>
<form use:enhance={submit}>
  <input name="test" value="foo" />
</form>

I would expect "bar" to be submitted, not "foo". What do y'all think?
Describe the proposed solution
Just construct the form data for submission after the caller's submit function runs.
Alternatives considered
I imagine there are cases where this might cause confusion but can't think of any off the top of my head.
Importance
nice to have
Additional Information
No response

Oh derp, just realized enhance gives you a FormData object to do this with. Can probably close this.

Yes, the FormData object is the source of truth at that point, Dom modifications don't have an effect - closing.


Is there any plan for support enableES5?

🙋♂ Question
I have many customers using morden browsers, but also some Internet Explorer. Old browser customers are offline in computer labs only with local network. Some techs are also limited by Microsoft, just like office 2019 plugin (Volume-licensed) which must use IE 11.
If I upgrade the immer, I will lost old browser customers. If not, I will not get the new features of immer. It is very pain for me to this.
So is there any plan to let enableES5 come back? 😭
Link to repro
Please provide a CodeSandbox demo to clarify your question if possible. Typically, questions without minimal codesandbox demo won't be answered.
Environment
We only accept questions against the latest Immer version.

Immer version:
10.0.0

There is no active development going on introducing new features, so I recommend to just stay at the previous major version instead.

@mweststrate
Got it. So is next version of immer focus on just performance improving? That sounds good. I will keep the immer version on 9.0.21, thanks! 👍


Throwing [Error: CERT_HAS_EXPIRED]

pb talk Hi

on the command line is spitting out "[Error: CERT_HAS_EXPIRED]". Any ideas?

Here's what's happening using curl:
$ curl -v -X GET "htt/bot/XXX?user_key=XXX"

* Hostname was NOT found in DNS cache

*   Trying <IP_ADDRESS>...

* Connected to aiaas.pandorabots.com (<IP_ADDRESS>) port 443 (#0)

* SSL certificate problem: Invalid certificate chain

* Closing connection 0

curl: (60) SSL certificate problem: Invalid certificate chain

More details here: http://curl.haxx.se/docs/sslcerts.html

curl performs SSL certificate verification by default, using a "bundle"

 of Certificate Authority (CA) public keys (CA certs). If the default

 bundle file isn't adequate, you can specify an alternate file

 using the --cacert option.

If this HTTPS server uses a certificate signed by a CA represented in

 the bundle, the certificate verification probably failed due to a

 problem with the certificate (it might be expired, or the name might

 not match the domain name in the URL).

If you'd like to turn off curl's verification of the certificate, use

 the -k (or --insecure) option.

@djfdev All fixed?


OCM-7794 | fix: handle nil ptr when rosa version is not retrievable from mirror

Should be handled via - https://github.com/openshift/rosa/pull/1960