Modelica_FundamentalWave to be included in the MSL

Reported by christiankral on 24 Nov 2009 09:57 UTC
The FundamentalWave library is made for modeling fundamental wave effects in electric machines. It is intended to include this library in
  Modelica.Magnetic.FundamentalWave

since FundamentalWave is the extension of the 1D FluxTubes library towards rotational 2D fundamental wave effects.
The library is currently located on the Modelica subversion server:
https://svn.modelica.org/projects/Modelica_ElectricalSystems/Modelica_FundamentalWave/trunk

Migrated-From: https://trac.modelica.org/Modelica/ticket/243

Comment by dietmarw on 24 Nov 2009 10:21 UTC
I have the following objections against including it already in the next release:

the current version is still using Modelica 2.2.2
some of the components still don't follow the naming convention of Modelica Standard Library
it should be available on http://modelica.org/libraries for a certain amount of time before it's integrated
the components in MoveToModelica needs to be actually integrated into MSL first
All this let's me think that [milestone:MSL3.2] is more realistic.

Comment by christiankral on 24 Nov 2009 10:24 UTC
The FundamentalWave library currently relies on the auxiliary package Complex.mo (which is also located on the subversion server) because complex numbers are currently not satisfactory supported in Modelica. Once complex number including units, operator overloading and function overloading are properly working, the dependency on the Complex library will be removed.

Comment by christiankral on 24 Nov 2009 10:41 UTC
Replying to [comment:1 dietmarw]:

I have the following objections against including it already in the next release:
* the current version is still using Modelica 2.2.2

For compatibility reasons the library was developed based on MSL 2.2.2. We did it the same way with the Modelica_FluxTubes library. This way also users which still use older tools can check out and test the library. Yet it is planned to include it in Modelica 3.X only.

* some of the components still don't follow the naming convention of Modelica Standard Library

I assume this refers (only) to the underscores in the class names of the machine models. The very only reason for having underscores in these class names, is that we used the same class names as in
Modelica.Electrical.Machines

I am in favor of keeping the name as they are for compatibility reasons. Once we decide to have conversion scripts we can convert the Electrical.Machines and the Magnetic.FundamentalWave class names.

* it should be available on http://modelica.org/libraries for a certain amount of time before it's integrated

This will be done after discussion on the Modelica Design Meeting.

* the components in MoveToModelica needs to be actually integrated into MSL first

This is clear since the library will only be provided for MSL 3.X

All this let's me think that [milestone:MSL3.2] is more realistic.

Comment by nr on 11 Jan 2010 14:18 UTC
There´s another issue to be mentioned: SaliencyCageWinding should not be modelled with "normal" plugs and m=2 but with SpacePhasors taken from the MSL.
Maybe the SpacePhasor library from the MSL should be extended to all the standard items that are included in other electrical libraries (Elecrical.Analog, QuasiStationary, MultiPhase) like resistor, capacitor, inductor, switches, sources,...

Comment by christiankral on 12 Jan 2010 13:19 UTC
As long as we do not have a sound implementation of a SpacePhasor Library I do not want to have a coupling element between FundamentalWave and SpacePhasors. The currently used two phase plug represents an equivalent two phase winding -- so this is OK on my opinion. Space phasors are basically equivalent to the used two phase winding. The only difference is the scaling of the magnitudes of the two axis components.

Comment by christiankral on 12 Jan 2010 13:22 UTC
The development of a SpacePhasor library is an entirely different issue, however. With such library the SaliencyCageWinding could be re-designed.

Comment by nr on 13 Jan 2010 06:18 UTC
The point is that in SaliencyCageWinding the values of the resistor and inductor are set to R={R.d,R.q} and L={Lsigma.d,Lsigma.q} where R and Lsigma are taken from the FundamentalWave.Math package. When using the letters d and q it´s exactly the terminology that is known from the space phasor theory. That´s astonishing me. By introducing a space phasor resistor and a space phasor inductor IMHO the FundamentalWave.Math package is no longer needed and the compatibility to the MSL is enhanced.

Comment by dietmarw on 12 Feb 2010 08:43 UTC
I just added the [raw-attachment:Modelica.Magnetic.FundamentalWave_StyleCheckLog.html StyleCheckLog.html] as a orientation what needs to be improved/fixed documentationwise before the release.

Comment by christiankral on 15 Feb 2010 07:36 UTC
Update of documentation has been performed. The class names of the electric machine models and the examples still contain an underscore (_) for compatibility reasons with the package Electrical.Machines.

Comment by christiankral on 22 Apr 2010 07:11 UTC
Library is fully migrated to trunk of MSL


Optional field for rbac resources

It seems that the ClusterRoleBinding to the admin ClusterRole is necessary for Portainer to function fully in Kubernetes, particularly for deployments beyond Edge compute features. This enables the option to disable RBAC when full Kubernetes cluster management is not needed.
The current Portainer Helm chart does not provide an option to disable the RBAC-related templates, including ClusterRoleBinding creation, via the values file. This lack of configurability prevents us from aligning with our security requirements.
A proposed solution would be to include a flag in the values file like: rbac: true/false and create the RBAC template accordingly

@emagiz you should modify the following files for this to work. charts/portainer/templates/serviceaccount.yaml charts/portainer/templates/rbac.yaml charts/portainer/templates/deployment.yaml //need to add condition to not include service account when values.rbac=false not the file under deploy

Are you sure you want to disable the SA when disabling RBAC? This would disable the option to create an SA without having the RBAC. One possible usecase for having a SA but not a CR would be to use AWS IAM roles for by assigning them to SA's. (see https://docs.aws.amazon.com/eks/latest/userguide/associate-service-account-role.html). I understand the view that portainer does not need the SA, but you would block people from having a flexible chart.

@emagiz you should modify the following files for this to work. charts/portainer/templates/serviceaccount.yaml charts/portainer/templates/rbac.yaml charts/portainer/templates/deployment.yaml //need to add condition to not include service account when values.rbac=false not the file under deploy

Are you sure you want to disable the SA when disabling RBAC? This would disable the option to create an SA without having the RBAC. One possible usecase for having a SA but not a CRB would be to use AWS IAM roles for by assigning them to SA's. (see https://docs.aws.amazon.com/eks/latest/userguide/associate-service-account-role.html). I understand the view that portainer does not need the SA, but you would block people from having a flexible chart.

I don't see why there is a need to create the Service Account when it's not being used. Service account with no roles is not going to be useful anyway and there is no point to have AWS IAM role for it.

@stevensbkang
Could you please review this and comment? Thanks


Newer method(s) missing from TrackedArray type

The types for TrackedArray don't include the newer (but fully supported) method findLast()
I think that means we just need to add it here, but not sure?
It may be worth doing a general review of all the built-ins to see what other methods have achieved full browser support since the initial implementation and ensuring those methods are properly accounted for since the built-ins are intended to have 1:1 functionality with their native counterparts.

@elwayman02 do you happen to have time to put into this? There are various aspects to TrackedArray that could use some love based on looking at issues ...

do you have a reproduction? TrackedArray uses the same types as Array -- so this smells like a tsconfig configuration issue more than anything

https://www.typescriptlang.org/play/?target=99#code/JYWwDg9gTgLgBAbzgFSgQwMYGsCmATAQSnQE84BfOAMyghDgHIZ1t8BaAIwFdgAbGNsAB2AZwYBuAFCTeOeGmJoyAXjhCcAdxQtchRSQAUAbQCMAGgBMZgMwBdAJRTJC0gDoqwvABk0ImAaEuemUAPjUguABSOAt7IA
yea, it works -- check your tsconfig.
Closing!


HWKBTM-346 : Improve UI for Ignored BTxns

Add "No ignored BTxns" slate panel;
Fix Loader;
Add "No matching filter" slate panel;
Match filter input style with Managed;
Other internal fixes & alignments with Managed tab

Known error in building on travis. PR has been locally tested and works fine.
Thanks Alexandre!


please create a option for HTTP or HTTPS url base

it is more flexible for user to choose HTTP or HTTPS
HTTP will give more flexibility to developers
thanks

While offering the flexibility to choose between HTTP and HTTPS might seem advantageous in terms of developer convenience, it's important to consider the broader implications, particularly around security.
Using HTTP in a production environment exposes users to significant risks, including data breaches and man-in-the-middle attacks. Since .NET MAUI applications can run on various devices and operating systems, ensuring the highest security standards, like enforcing HTTPS, is crucial to protect all users.
Given the critical nature of security in today’s digital environment, and the support for HTTPS in .NET MAUI, I recommend sticking with HTTPS to safeguard both user data and the integrity of applications built with .NET MAUI. This approach also aligns with modern security practices and standards.
just my two cents...

we both know the advantages of HTTPS.
you can default setting "HTTPS" base,
but offer an option to developer, when they are developing.
thanks


[ci skip] Updates to Readme.md

[ADDED] Note describing that streaming server subscriptions do not support wildcards
@mcqueary @kozlovic @ColinSullivan1

LGTM


Disable auto rebase for cargo dependabot updates

With bors enabled on this repo now, this PR disables the auto rebase strategy for Cargo update PRs. When bors is trying a batch it will cancel it when one of the branches in that PR is pushed (e.g. when dependabot force pushes a rebase) e.g. https://github.com/substrait-io/substrait-rs/pull/19. The other updates are less likely to trigger a rebase (for Cargo it's typically the lock file).

@mbrobbel Sorry, what is "bors"?


feat(cli): add lacework query preview-source

Summary
As a user I would like to see a sample event from a specified datasource
How did you test this change?
Automated integration testing
Issue
https://lacework.atlassian.net/browse/ALLY-879
Example
❯ lacework-dev query preview-source LW_HE_FILES
{
  "BATCH_END_TIME":<PHONE_NUMBER>058,
  "BATCH_START_TIME":<PHONE_NUMBER>058,
  "BLOCK_COUNT": "1760",
  "BLOCK_SIZE": "4096",
  "FILEDATA_HASH": "4f8a3fa663e9756da6b43ae130f29fa8bfb2f8ecf8318f3e6acf897811ed5835",
  "FILE_ACCESSED_TIME":<PHONE_NUMBER>000,
  "FILE_CREATED_TIME":<PHONE_NUMBER>378,
  "FILE_MODIFIED_TIME":<PHONE_NUMBER>000,
  "FILE_NAME": "ldconfig",
  "FILE_PERMISSIONS": "33261",
  "FILE_TYPE": "application/x-executable",
  "HARD_LINK_COUNT": "2",
  "INODE": "8848",
  "IS_LINK": "0",
  "LINK_ABS_DEST_PATH": "",
  "LINK_DEST_PATH": "",
  "METADATA_HASH": "a8958947e742f45b699d8f8c2a9c1c2c9626bc68cd8becfc1a43891a3224c1a7",
  "MID": "4620",
  "OWNER_GID": "0",
  "OWNER_UID": "0",
  "OWNER_USERNAME": "root",
  "PATH": "/usr/sbin/ldconfig",
  "RECORD_CREATED_TIME":<PHONE_NUMBER>258,
  "SIZE": "897784"
}

@hazedav you have a CI failure:
=== Failed
=== FAIL: integration TestQueryPreviewNoSuchSource (0.48s)
    lql_preview_test.go:51:
                Error Trace:    lql_preview_test.go:51
                Error:          "Usage:\n  lacework query preview-source <datasource_id> [flags]\n\nFlags:\n  -h, --help   help for preview-source\n\nGlobal Flags:\n  -a, --account string      account subdomain of URL (i.e. <
ACCOUNT>.lacework.net)\n  -k, --api_key string      access key id\n  -s, --api_secret string   secret access key\n      --api_token string    access token (replaces the use of api_key and api_secret)\n      --debug
    turn on debug logging\n      --json                switch commands output from human-readable to json format\n      --nocache             turn off caching\n      --nocolor             turn off colors\n      --noninteracti
ve      turn off interactive mode (disable spinners, prompts, etc.)\n      --organization        access organization level data sets (org admins only)\n  -p, --profile string      switch between profiles configured at ~/.lace
work.toml\n      --subaccount string   sub-account name inside your organization (org admins only)\n\nERROR unable to retrieve data source: \n  [GET] https://customerdemo.lacework.net/api/v2/Datasources/NoSuchSource\n  [404]
Datasource NoSuchSource not found\n" does not contain "unable to retrieve datasource"
                Test:           TestQueryPreviewNoSuchSource

DONE 195 tests, 1 failure in 395.363s
make: *** [Makefile:57: integration-only] Error 1

A funny one! 😂


menu item for adjust network menu item window

We need at minimum a way to set port number via menu item.


Support for PSQL attached partition by hash ?

Found Behavior
I've been running harbourbridge on some of my PSQL schema to test how hard a migration to Google Spanner would be.
One of the main key of our project is that some of our table has the PostgreSQL 12 partition strategy.
Running a first pass on a config schema went perfectly. Doing a test on a more complex schema end-up doing a failed to parse the data file.
I'm doing a simple pgdump in schema only mode:
pg_dump -h myhost -d mydb -U myusername -s -n myschemaname| $GOPATH/bin/harbourbridge -driver=pg_dump -dbname=my-destination-db
which result in:
Using driver (source DB): pg_dump target-db: spanner
Generating schema: 100%
Failed to parse the data file: Error parsing last 6404 line(s) of inputpanic: failed to parse the data file

goroutine 1 [running]:
main.main()
        /app/go/pkg/mod/github.com/cloudspannerecosystem/harbourbridge@v0.0.0-20210811071152-f98dc5b0fd75/main.go:157 +0xecc
root@2462e825148a:/app#

So I export my pg_dump to a file to see what line 6404 was all about. Which is the exact spot where the first partitioned table get created in the script order.
sed -n '6403,6405'p pgdump.txt ALTER INDEX affiliate.affiliate_referral_code_pk ATTACH PARTITION affiliate.referral_code_0_pkey;
Expected Behavior
If a statement isn't supported I would expect to get it back on the report and have harbourbridge go ahead with the dump migration completion.
Steps to Reproduce the Problem

Get a simple table that has a partition by hash strategy on a PG12 instance.
Pass a pg_dump of that table to harbourbridge.

Specifications

Version:
Platform:

Does anyone as experience such behavior when migrating a table that has partition support ?

Hey @mikesimard, thanks for reaching out. We ran some sample dump files and discovered harbourbridge can successfully parse and convert the CREATE TABLE statements with partition support. The issue you are getting seems to be particularly for
ALTER INDEX affiliate.affiliate_referral_code_pk ATTACH PARTITION affiliate.referral_code_0_pkey;
when we try to attach partitions to an index because the current parser is unable to process this statement.
The fix right now is to manually remove these ALTER INDEX index_name ATTACH PARTITION partition_name; from the dump and try to re-run harbourbridge. Please circle back if it still causes an error.
On a side note, automatically skipping lines that failed to parse is a known limitation in harbourbridge that we are still working on.

@mikesimard Is this still an issue?


diag: sklearn API upgraded classes plus skbase test suite

This PR combines https://github.com/astrogilda/tsbootstrap/pull/64 and https://github.com/astrogilda/tsbootstrap/pull/26 for diagnostics.
It is used to bring main into a state where #26 tests pass.

obsolete, as #66 is now merged, and includes #64


[Improvement-12700][Project]Improve the sql of listAuthorizedProjects

Purpose of the pull request

close: #12700

Brief change log
Currently, projectIds is not used in the SQL of listAuthorizedProjects() as below
https://github.com/apache/dolphinscheduler/blob/7d0e2cbbb9fcd2e1543f5166f0518eda03afae89/dolphinscheduler-dao/src/main/java/org/apache/dolphinscheduler/dao/mapper/ProjectMapper.java#L139
https://github.com/apache/dolphinscheduler/blob/7d0e2cbbb9fcd2e1543f5166f0518eda03afae89/dolphinscheduler-dao/src/main/resources/org/apache/dolphinscheduler/dao/mapper/ProjectMapper.xml#L173-L182
So this PR restrict the SQL query scope to projectIds, just like in

Verify this pull request
manually tested

Hi, @SbloodyS , thanks for the comment. I've rebased this PR and CI has passed.

Hi, @SbloodyS @Radeity could you please help review this?

Hi, @Radeity thanks for your check, but I think projectsIds is used as below:
https://github.com/apache/dolphinscheduler/blob/cff53feaeb44e38584fe7856e3612dc620ce20e7/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ProjectServiceImpl.java#L553-L555

Hi, @Radeity thanks for your check, but I think projectsIds is used as below:
https://github.com/apache/dolphinscheduler/blob/cff53feaeb44e38584fe7856e3612dc620ce20e7/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ProjectServiceImpl.java#L553-L555

Hi, @rickchengx , I know, and it's called by the following api, but this api is never called in front-end, seems already deprecated.
https://github.com/apache/dolphinscheduler/blob/107643dba8b33fe2e14bddd5e3d666cfc82844dc/dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/controller/ProjectController.java#L278-L289

Hi, @Radeity thanks for your double check. But I think this backend-api could also be called directly by user (not front-end). And these apis are in the controller, we should correct the sql of them. WDYT?


Dashboard UI/UX Realignment

Is your feature request related to a problem? Please describe.
The current UI and UX of the dashboard needs improvements to make it more professional looking and better user experience.
Describe the solution you'd like
I think it needs more realignment in terms of elements position to clearly set hierarchy and make it simple and a lot cleaner. Here's my (small) realignment for now

Describe alternatives you've considered
It just needs improvements to look a lot better.
Additional context
In the above image
Wallet address
0x95a54A149f1437B4e7579E3b5dDF2bB62400f563

Thank you for your input on this feature request. After reviewing the current state of this repository, we’ve decided that it is no longer actively maintained and can be archived.
This issue and others will now be closed as part of this archival process. If you have further feedback or need assistance, please reach out through our active channels or repositories.
Thank you for your contributions!


chore: release main

:robot: I have created a release *beep* *boop*
@set-app/cli: 2.7.0
Miscellaneous Chores

@set-app/cli: release as 2.7.0 version (bb7dc6d)
release 2.7.0 (3fb8e72)

This PR was generated with Release Please. See documentation.

:robot: Release for @set-app/cli is at https://github.com/Frankeo/set-app/releases/tag/cli-v2.7.0 :sunflower:


Azure Container Apps CORS issue

I have a big problem with my web app. Starting with some info... the backend is in ASP.NET Core 6, and the frontend is in React. I deploy this site using Azure Container Apps inside the same Resource Group end in the same Container Environment.
All works fine until I decided to change backend visibility from External to Internal. From this moment, when the frontend calls backend I have CORS error and the backend is never called.
Inside the Program.cs of the application is managed yet the CORS policy that accepting ALL. But it still don't work.
NOTE: I can't expose my backend public.

@giannicaringi did you try with the latest Azd bits and templates. There was a recent fix we made #1899 @hemarina @vhvb1989

@giannicaringi , is your template public?  If not, can you share:

How you are setting cors for .cs ?
How you are hiding the endpoint from the server?

@vhvb1989 @hemarina can we close this? this should be handled as part of the fixes you made recently?

@rajeshkamal5050 , we don't have any template for .Net in the backend hosted in aca
For .Net, we host on web-apps only (including funct).
Also, our .Net templates don't have CORS implemented at application level. Since we use web-apps, we rely on it for setting and controlling CORS.
When it comes to Azure Container Apps, there's is not CORS supported from the service (not that I am aware of). What we do is to use CORS at application level in that case, setting the frontend url as env-var for the backend container. Then the backend application gets the url and make it an allowed url.
But for what @giannicaringi is saying, the backend application is not setting CORS and it should be allowing all requests.
This is indeed not an azd bug, but it would be interesting to see if @giannicaringi has found the issue and a solution here for what configuration he needs

Well, cors is now supported by aca
https://learn.microsoft.com/en-us/azure/container-apps/cors?pivots=azure-portal&tabs=arm
It is on preview.
@giannicaringi check if this setting is enabled on your backend container and if so, add the frontend url to the allowed origins


Support for scikit-image

Tested on Linux, macOS and Windows (probable regression from #2103) compilation_report.txt:
Traceback (most recent call last):
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/YouAndEyePhoto.py", line 1, in <module>
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/you_and_eye.py", line 26, in <module>
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/sub_window.py", line 18, in <module>
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/image_processing.py", line 27, in <module>
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/skimage/feature/__init__.py", line 2, in <module>
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "_cascade.pyx", line 18, in init skimage.feature._cascade
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/skimage/transform/__init__.py", line 35, in <module>
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/skimage/transform/hough_transform.py", line 4, in <module>
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "_hough_transform.pyx", line 13, in init skimage.transform._hough_transform
  File "<frozen importlib._bootstrap>", line 1178, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1149, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 690, in _load_unlocked
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/skimage/draw/__init__.py", line 3, in <module>
  File "/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/lazy_loader/__init__.py", line 245, in attach_stub
ValueError: Cannot load imports from non-existent stub '/home/bgallois/Codes/Codes/You-And-Eye-Photo/YouAndEyePhoto.dist/skimage/draw/__init__.pyi'

Nuitka version:
 1.7.3
 Commercial: None
 Python: 3.11.3 (main, Jun  5 2023, 09:32:32) [GCC 13.1.1 20230429]
 Flavor: Unknown
 Executable: /home/bgallois/Codes/Codes/You-And-Eye-Photo/venv/bin/python
 OS: Linux
 Arch: x86_64
 Distribution: Arcolinux (based on arch) None
 Version C compiler: /usr/bin/gcc (gcc).

How did you install Nuitka and Python: pip installation in a virtual environment.

The specific PyPI names and versions:
 autopep8==2.0.2
 certifi==2023.5.7
 chardet==5.1.0
 charset-normalizer==3.1.0
 checksumdir==1.2.0
 idna==3.4
 imageio==2.31.1
 lazy_loader==0.2
 networkx==3.1
 Nuitka==1.7.3
 numpy==1.25.0
 opencv-python-headless==<IP_ADDRESS>
 ordered-set==4.1.0
 packaging==23.1
 Pillow==9.5.0
 py-machineid==0.3.0
 pycodestyle==2.10.0
 PySide6==<IP_ADDRESS>
 PySide6-Addons==<IP_ADDRESS>
 PySide6-Essentials==<IP_ADDRESS>
 PyWavelets==1.4.1
 QDarkStyle==3.1
 QtPy==2.3.1
 rawpy==0.18.1
 requests==2.31.0
 scikit-image==0.21.0
 scipy==1.11.1
 sentry-sdk==1.26.0
 shiboken6==<IP_ADDRESS>
 tifffile==2023.4.12
 toml==0.10.2
 urllib3==2.0.3
 winregistry==1.1.1
 zstandard==0.21.0

* Note if this is a regression: was working with scikit-image==0.20.0 and Nuitka==1.7.3, probably related to #2103

Yes, this one probably needs to be in Nuitka-Watch, every .pyi file added can cause it, and it seems new releases spread it. You didn't provide a reproducer, so I kind of have to guess here, but it's still a good report. I feel like I would have to make a kind of generator that imports all modules of this package in one test case, such that we are sure of coverage anyway.

I made the fix based on your traceback, and it's on factory, however I didn't yet test it myself, but it may already be enough for you.
Thanks for your report, this is fixed on the factory branch, which is a development version under rapid development. You can try it out by going here: https://nuitka.net/doc/factory.html
Feedback if this is working is very welcome, just please do not share plans of doing it, but rather confirmations or denials of it working.

This is not working, included a reproducer
test.py.txt

Interesting, I really have to make that import all test for skimage there, actually --include-package ought to do it. I was going over my sklearn tests, I got 4 of them, one that fork bombed due to no joblib support recently, and the first one was working very nicely, and this is just because of using only a subset of the package I guess.

So, my examples all work with what is on factory though, I added your imports to one of mine, but I actually do a kind of import them all, then walk them all with --include-package=sklearn and I also want to make it bloat free and keep it like that at the same time.
# nuitka-project: --standalone
# nuitka-project: --noinclude-default-mode=error
# nuitka-project: --include-package=sklearn
# nuitka-project: --nofollow-import-to=sklearn.*.tests
# nuitka-project: --nofollow-import-to=sklearn.tests

In an ideal world, where I will have the data center ready to do it, we would then be triggered by PyPI updates and immediately know of any breakage through a package update.
I imaging that anti-bloat is going to cause a few iterations needed here, but I really want to take the fundamental approach here to cover a package like this well for once.
This code is then doing the walk:
# For coverage right now.
import sklearn

def walkPackage(package_name, package):
    print("Walk", package)

    for item in pkgutil.iter_modules(package.__path__):
        if item.name == "tests":
            continue

        full_name = "%s.%s" % (package_name, item.name)
        print("Containing", item, full_name)

        importlib.import_module(full_name)
        imported = sys.modules[full_name]
        print(imported)

        if item.ispkg:
            walkPackage(full_name, imported)

walkPackage("sklearn", sklearn)

I would expect it to crash on the .pyi files that it is lacking. The alternative would be to hardcode this in a plugin, that just does them all, but I like this more.

So this is now solved on factory. With the all including skimage test, the .pyi files are all covered. Of course there was only one more to add, but with Nuitka-Watch, I can now try and monitor this state automatically at some point.

This is part of the hotfix release 1.7.5 that I just made.


Mixer should make adapter responses available as attributes

When an adapter rejects a request, Mixer sends back this information to the Proxy.
However this information is not available as an attribute.
This information should be available as an attribute so that it can be used in reports or logs.
This attribute can be a stringmap or something else that is structured.

@liminw @douglas-reid

There are two attributes populated by the proxy on error response (https://github.com/istio/proxy/blob/4cfa354bbeb42ae67c3c634e19ad01a54339cb3a/src/istio/control/http/attributes_builder.cc#L225):

check.error_code is the gRPC error code
check.error_message is the error string from Mixer that includes the failing rule

Can you clarify what other information we need?

@robertpanzer

It would be nice though to also see at least a flag in the access log of the proxy that initiated the istio mixer check that the request was rejected because of a failed mixer check.

@munrodg can we keep this open, but with a different milestone?  we still don't have attributes that carry information on policy adapter response per policy adapter.
@mandarjog @kyessenov: comments?


Create Funny Quotes

This is not something that belongs in the library, especially not at the root level.
If you're testing out pull requests, please use your own repository to do so.


Andriod whatsapp can't send/receive media via proxy,text is ok

I have set up a WhatsApp proxy on Ubuntu 20 .WhatsApp proxy works very well on macOS/iOS app, but on Android only can send text, can't send/receive pictures videos and documents. When will it be possible to update the version of WhatsApp to resolve this issue?

@Nicepaul, can you please submit a bug report about this? Does the proxy you set up support media?

ubuntu 20 server

I would like to explain my situation. I am in China, and when I use Chinese home broadband or Chinese mobile data to access the internet, my Android phone can send messages through WhatsApp Proxy but cannot send media. However, when I switch to a U.S. SIM card for internet access, WhatsApp Proxy can send both messages and media. In fact, with a U.S. SIM card, there is no need for a proxy at all because the Chinese firewall does not block traffic from U.S. SIM cards.
It appears that Google/Android might be more susceptible to DNS hijacking. On iOS, regardless of the internet connection method, the proxy can send media without any issues. Additionally, I can send media without any trouble on Signal.
I hope WhatsApp can address and resolve this issue.

I have set up a WhatsApp proxy on Ubuntu 20 .WhatsApp proxy works very well on macOS/iOS app, but on Android only can send text, can't send/receive pictures videos and documents. When will it be possible to update the version of WhatsApp to resolve this issue?


CheckoutSession::create ?

I can see the CheckoutSession struct but it doesn't appear to have the ability to create one.  Am I missing it somewhere or does it just not exist?  Thx.

Hi! The way this library works is that we generate rust types for as many endpoints as possible, then implement the actual logic in the ext file. So you're right, we're missing the impl for CheckoutSession, but I am very open to PRs given that the original project seems unmaintained. I've marked this as a feature request.

I also added a short paragraph explaining the general 'setup' to contributing.md. I think it should be fairly easy to implement, but if you'd like some pointers I'm happy to help!

@rrrodzilla some more context: https://github.com/wyyerd/stripe-rs/issues/85

@rrrodzilla I handled this while working towards #32, please target master and give it a spin

Hey Alexander,
Sorry about the long delay, I had to work on other things.  This seems to be working great.  The struct names are pretty verbose but nothing some aliases during usage.  Either way it works just fine and produces a valid checkout session.  Thanks for that!  If I use it for implementation I'll clean up some example code to include in the docs.
-Roland


New maintainer wanted

This project has been started by Slavek Kabrda, but he does not have time for it anymore. Copr team took this project as we use it. But this is at the bottom of our priorities.
If you like this project and you want to maintain it, then please drop a comment here.

I'm not personally interested in this project but can offer my help in case you need to fix something related to new Pythons etc. The question I have is whether you need to keep Python 2 support here.

Our team does not need python2 at all. And because even EPEL7 supports py3 I think we can kill py2 with a fanfare. :)

Hi, I would like to help to maintain this project. See more background here: https://github.com/greyli/flask-extension-status
Please grant me the repository maintenance and PyPI permission (ID: greyli). I will help to release a new version to fix the installation issue (https://github.com/fedora-copr/flask-whooshee/pull/85) in the 0.9.0 version.

I added you. Check your email for an invitation.

Thank you, @greyli

0.9.1 version released:

https://github.com/fedora-copr/flask-whooshee/releases/tag/v0.9.1
https://pypi.org/project/flask-whooshee/


Room upgrades do not state which state events should be migrated

The current spec states "migrating as much data as possible over to the new room" which is rubbish and doesn't explain if non-state events should be migrated at all (Synapse does not). It also doesn't explain whether all state should be migrated or just a subset (Synapse whitelists some state)
Furthermore, https://matrix.org/docs/spec/client_server/unstable.html#post-matrix-client-r0-rooms-roomid-upgrade references it's parent module for more details which is a bit weird.
EDIT:
I found some information below, but it was linked incorrectly by the above.

Replicates the power levels, privacy, topic, and other transferable state events to the new room. This generally excludes membership events but may include client-specified events and other presentation details.

At minimum we should be stating these as their types rather than the human readable names. "other transferable state events" is still too vague and needs expanding on what makes them transferable, or a blacklist of nontransferable events.

err, non-state events aren't intended to be transferred at all.

it was deliberately left as an implementation-specific "do the best you can here" thing

Further with regard to the last paragraph of the module (https://github.com/matrix-org/matrix-doc/blob/1f05c44a0bcdedb36b1c7cf28a5ca03bba199464/specification/modules/room_upgrades.rst):

When a user joins the new room, the server may wish to automatically transfer/replicate some of the user's personalized settings such as notifications, tags, etc.

That doesn't read like a specification but a proposal. For one thing, "the server" should say "the user's server" to clarify the responsibility. Secondly, if this is a "MAY" thing, it should simply say "May" rather than "may wish to". It's better both stylistically and for clarity of someone implementing a client or server.
And third, with regard to this paragraph as well as the state-event migration verbiage brought up by Half-Shot, why is this something we wish to leave implementation-specific? Shouldn't we want clients to be able to rely on consistent behavior between homeservers for room-version upgrades? Apologies if this was discussed in the proposal somewhere already.


empty output

Hi,
I am trying to run armatus on a sparse matrix, but I seems to get no domains at all, trying with different matrix... I believe there is a problem with my run.
Could i send you my matrix so you can test it, and could I gt an example of a sparse matrix on which you have tested armatus to see if the problem come from my installation?
Thanks,
Samuel

Hi Samuel,
Please send me your matrix (nsauerwald@cmu.edu), and I will try to figure out the issue.
In the meantime, any of the sparse Hi-C matrices from the 2012 Dixon publication (available here: https://www.ncbi.nlm.nih.gov/geo/query/acc.cgi?acc=GSE35156), or from the 2014 Rao publication (available here: https://www.ncbi.nlm.nih.gov/geo/query/acc.cgi?acc=GSE63525) should run correctly with Armatus (make sure to use the -R flag for the Rao dataset, as the file format is different).
Best,
Natalie

Hey,
I've got the same problem. Just to be clear, is the input for Rao's dataset the sparse matrix or domain list? I would think the sparse matrix (3 columns) make sense to me, however, the introduction of Armatus seems to suggest to use domain list with -R?

The input should be the sparse matrix, as you said. If you use the -R flag, the software will automatically look for both the ".RAWobserved" and the ".KRnorm" files, and normalize the Hi-C data before finding TADs. If you just want to run Armatus on the raw Rao data, use the -N flag to skip the normalization step. For example, if you wanted to find TADs on the normalized 5kb Hi-C matrix, on chromosome 1 of the GM12878 data, use "-R -i GM12878_combined/5kb_resolution_intrachromosomal/chr1/MAPQGE30/chr1_5kb".

Thanks, Natalia.
Here is another basic question for you, if my data is in sparse matrix, does the matrix is supposed to be normalized or not?
Thanks.

Armatus will run on either normalized or unnormalized data, so the choice of whether to normalize (and which normalization method to use) is up to you based on your application.

Hi,
I seem to be having a similar problem. The sizes of all my output files are zero. I am using an iced matrix produced by HiC-Pro and split by chromosomes using it's utility script split_sparse.py.
My first guess would be that there is some problem with formatting of the input.
Any chance you could help?
All the best,
Agnieszka
This is the command:
armatus -m -r 5000 -N -S -c ${i} -i $INPUT -g 1.0 -s 0.05 -o armatus.domains/chr${i}
And that's the head of the input file:
head NP1_5000_iced_Chr1.matrix
1       1       88.000000
1       2       84.000000
1       3       55.000000
1       4       28.000000
1       5       30.000000
1       6       22.000000
1       7       19.000000
1       8       7.000000
1       9       12.000000
1       10      4.000000
Some lines from std.out
Reading input from NP1_5000_iced_Chr10.matrix.
Building matrix for chromosome 10 at resolution 5000bp with 1 rows.
Initializing matrix to zero elements
10.9114%
21.8229%
32.7343%
43.6457%
54.5572%
65.4686%
76.38%
87.2915%
98.2029%
MatrixParser read matrix of size: 1 x 1
gamma=0
OPTIMAL SCORE: 0
begin computeTopK()
In topK()
The 0th-best solution had score 0
gamma=0.05
OPTIMAL SCORE: 0
begin computeTopK()
In topK()

I think the issue is that your data is already divided by the resolution (notice that your TADs start and end at values like 1 and 2 instead of 5000 and 10000), and when you put in that the resolution of your data is 5000, the algorithm tries to divide the TAD boundaries by the resolution, and gets values like 1/5000 for your boundaries. Try using the same command but without the "-r 5000" flag, as shown below.
armatus -m -N -S -c ${i} -i $INPUT -g 1.0 -s 0.05 -o armatus.domains/chr${i}


Fix double globe when lighting is enabled

In CesiumJS 1.63, try running:
var viewer = new Cesium.Viewer('cesiumContainer');
viewer.scene.globe.enableLighting = true

You'll get a double globe:

You can fix this by going back to the old resolution scale behavior we had in 1.61:
viewer.resolutionScale = 1.0 / window.devicePixelRatio;

This happens because in #8318 we added a multiplication by czm_pixelRatio in metersPerPixel.glsl. This caused a second globe to render at half the size when globe lighting was enabled. This fixes it by dividing back by that value. But I'm not sure if this is the right fix. @lilleyse @IanLilleyT what do you think?
@mramato is this worth a 1.63.1 patch release?
This was reported on the forum.

Thanks for the pull request @OmarShehata!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

:grey_question: Unit tests were not updated.

Make sure you've updated tests to reflect your changes, added tests for any new code, and ran the code coverage tool.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@mramato is this worth a 1.63.1 patch release?

The bar for a patch release is usually pretty high, since this only happens with lighting enabled, only on machines that have devicePixelRatio !== 1, and has an easy workaround, I don't think doing a release is warranted unless we find out it affects a large piece of the community.
I would love to move to more frequent CesiumJS releases in general, but that's a larger conversation.

@OmarShehata I have a related fix that also affects labels. So once that's up we can merge mine and close this one.

Here's the PR that fixes the globe lighting and labels: https://github.com/AnalyticalGraphicsInc/cesium/pull/8351

Superseded by https://github.com/AnalyticalGraphicsInc/cesium/pull/8351


Keep the last docker host when its last docker container is removed

When DockerInfrastructure.REMOVE_EMPTY_DOCKER_HOSTS is set to true and a container is removed a check will be performed to see if any containers are running on the host. If the host has no other containers it will also be removed. This will also remove the last host when the last container is removed. I would suggest that the last host should be kept even when it is empty.

@mattunderscorechampion looks good, I'll merge this - we should make the scale-down mechanics more configutable, but can do this later...


When can we expect a new release (current v0.9.2)?

I've been waiting to update my company's vendored version of rsa, which is currently still on the v0.8 series, until #352 is part of a release. But that was merged in July and there hasn't been a release since. Should I expect one soon, or should I stop waiting and vendor an unreleased Git revision instead? Thanks!

v0.9.3 is out now, so I'll close this. Thanks so much!


Fix spacing of comment

I was reading the primitives header file to see what v3 deems deprecated. Noticed this out of line comment. Just wanted to fix it

Sometimes clang-format doesn’t produce perfect results and this is unfortunately one of those instances. Thank you for looking out for docs fix ups but in this case we have to deal with the slightly janky formatting.


Help me

Hi, I need someone to help me with the settings. I have run this command to the terminal and correct, and I would like to know if the settings for the settings.ini file is filled with the correct data
[DEFAULT]
verbose = True
MQTT_listen = False
MQTT_update = False
num_retries = 3
update_interval = 3
[TV]
host = <IP_ADDRESS>
port = 1926
apiv =  6
user = pczyoRn3qriN9jsq
pass = f20d23004e7e72a2f5980eb47c091bb2adca56da5ead6d53197b3107d0df03a9
protocol =
[MQTT]
host =
port =
user =
pass =
TLS = False
cert_path =
topic_pylips =
topic_status =
——-——————-———-——
jbnoxs@MiniNoXs ~ % python3 /Users/jbnoxs/Documents/philips_android_tv-master/philips.py
usage: philips.py [-h] [--host HOST] [--user USER] [--pass PASSWORD] command
philips.py: error: the following arguments are required: command

What do I do wrong?

What do I do wrong?

You are trying to run "philips_android_tv-master/philips.py", which is a separate project that is not related to Pylips in any way. Check your paths and make sure you are actually calling Pylips.

You are trying to run "philips_android_tv-master/philips.py", which is a separate project that is not related to Pylips in any way. Check your paths and make sure you are actually calling Pylips.


Ignore xcshareddata/ folder for Xcode 9

@Silence-GitHub any links to StackOverflow discussions or other documentation about this change?


Add GoogleTest C++ examples?

Since I am a bit more familiar with GoogleTest
I used that as the C++ option for teaching testing in a seminar,
so I have codes and solutions for every task.
If this is of interest I'd be happy to add it.

This would be really cool! (sorry for the very late follow-up)


Warnings about incorrect README location when publishing crates

Describe the bug

There was a recent change to add a [workspace.package] section in the top level Cargo.toml, with the readme specified there. This does not appear to be working as intended, as it is trying to publish the top-level README in each crate.
warning: readme `../../README.md` appears to be a path outside of the package, but there is already a file named `README.md` in the root of the package. The archived crate will contain the copy in the root of the package. Update the readme to point to the path relative to the root of the package to remove this warning.

To Reproduce

Expected behavior

Additional context

Closed by https://github.com/apache/arrow-datafusion/pull/7964


Jump Function

Using the Jump Function returns an error saying that jump is not a function.

Can you show me your code? I don't think it's a bug.

const Discord = require("discord.js")
module.exports.run = async (bot, message, args, distube) => {
distube.jump(message, args.join(" "))
.catch(err => message.channel.send(new Discord.MessageEmbed() ect ));
}
module.exports.config = {
ect
}

Maybe you added distube.jump = etc in somewhere else. Or distube is not DisTube. I don't detect any problem with your comment.

You can join DisTube Support Server for asking about your code.
I'll close #9 cause it's not a package bug.


[INS-1776] adds ws dependency explicitly

closes INS-1776
I noticed in https://github.com/Kong/insomnia/pull/5050 that the websocket npm we use, ws, is unspecified by the insomnia package.  This means we do not control the version of this critical package.  It's a very very simple fix: specify the dependency in our package.json for the insomnia package, just as we did for insomnia-smoke-test.
To demonstrate this: we were in fact actually getting version 8.5.0, when we want the latest, 8.8.1.

ticket 1776 woop woop

ran
npm run hard-reset

and ran through the websocket flow and it works as expected


Create CONTRIBUTING.md

This content was missed in the migration. It is linked to https://github.com/devcontainers/images#why-do-dockerfiles-in-this-repo-use-run-statements-with-commands-separated-by-

Simple copy/paste of the old document for updating to the new repo

Dropping this as we won't be migrating this file for now


[wip] Switch to using kubelet config file for all supported flags

Proposed Changes
Switch to using kubelet config file for all supported flags
Types of Changes
enhancement / tech debt
Verification
See linked issue
Testing
Linked Issues

https://github.com/k3s-io/k3s/issues/2116

User-Facing Change

Further Comments
Currently using a hacky string replace when marshaling the config file to work around an upstream issue:

https://github.com/kubernetes/kubernetes/issues/79421

I'm going to hold on this until the August release cycle, as I'd like more lead time on testing it.

@dereknola I have confirmed this works on Windows, using RKE2: https://github.com/rancher/rke2/pull/6909
root@dev02:~# kubectl get node -o wide
NAME    STATUS   ROLES                       AGE     VERSION          INTERNAL-IP   EXTERNAL-IP   OS-IMAGE                                  KERNEL-VERSION     CONTAINER-RUNTIME
dev02   Ready    control-plane,etcd,master   3h42m   v1.31.1+rke2r1   <IP_ADDRESS>    <none>        Ubuntu 24.04 LTS                          6.8.0-45-generic   containerd://1.7.21-k3s2
win02   Ready    <none>                      3h31m   v1.31.1          <IP_ADDRESS>    <none>        Windows Server 2022 Standard Evaluation   10.0.20348.2113    containerd://1.7.21-k3s2

PS C:\Users\Administrator> c:\usr\local\bin\rke2.exe --version
rke2.exe version v1.31.1+dev.045680fc (045680fc54b2e82cbb5491c9a916655535cd8fb7)
go version go1.22.6

PS C:\Users\Administrator> dir C:\var\lib\rancher\rke2\agent\etc\kubelet.conf.d\

    Directory: C:\var\lib\rancher\rke2\agent\etc\kubelet.conf.d

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a----         10/5/2024  12:51 AM           1516 00-rke2-defaults.conf

PS C:\Users\Administrator> cat C:\var\lib\rancher\rke2\agent\etc\kubelet.conf.d\00-rke2-defaults.conf
address: <IP_ADDRESS>
apiVersion: kubelet.config.k8s.io/v1beta1
authentication:
  anonymous:
    enabled: false
  webhook:
    cacheTTL: 2m0s
    enabled: true
  x509:
    clientCAFile: C:\var\lib\rancher\rke2\agent\client-ca.crt
authorization:
  mode: Webhook
  webhook:
    cacheAuthorizedTTL: 5m0s
    cacheUnauthorizedTTL: 30s
cgroupDriver: cgroupfs
clusterDNS:
- <IP_ADDRESS>
clusterDomain: cluster.local
containerRuntimeEndpoint: npipe:////./pipe/containerd-containerd
cpuManagerReconcilePeriod: 10s
evictionHard:
  imagefs.available: 5%
  nodefs.available: 5%
evictionMinimumReclaim:
  imagefs.available: 10%
  nodefs.available: 10%
evictionPressureTransitionPeriod: 5m0s
failSwapOn: false
fileCheckFrequency: 20s
healthzBindAddress: <IP_ADDRESS>
httpCheckFrequency: 20s
imageMaximumGCAge: 0s
imageMinimumGCAge: 2m0s
kind: KubeletConfiguration
logging:
  flushFrequency: 5s
  format: text
  options:
    json:
      infoBufferSize: "0"
    text:
      infoBufferSize: "0"
  verbosity: 0
memorySwap: {}
nodeStatusReportFrequency: 5m0s
nodeStatusUpdateFrequency: 10s
resolverConfig: C:\var\lib\rancher\rke2\agent\etc\resolv.conf
runtimeRequestTimeout: 2m0s
serializeImagePulls: false
shutdownGracePeriod: 0s
shutdownGracePeriodCriticalPods: 0s
staticPodPath: C:\var\lib\rancher\rke2\agent\pod-manifests
streamingConnectionIdleTimeout: 4h0m0s
syncFrequency: 1m0s
tlsCertFile: C:\var\lib\rancher\rke2\agent\serving-kubelet.crt
tlsPrivateKeyFile: C:\var\lib\rancher\rke2\agent\serving-kubelet.key
volumeStatsAggPeriod: 1m0s

moving to WIP until closer to 1.32.

Hey @brandond, are we still planning to do this for 1.32?

I am watching what upstream does with this feature. It looks like it might be another minor release until it is on by default.


Maximum frequent character in the string

Enter your question -
Given a string str, we need to find maximum occurring character in the string.
Enter link to the question(if question belongs to any online platform) -
N/A
Tags for the question(eg - Array, Basic, Stack, etc.) -
Strings
Example:

As a part of Hacktoberfest, @SarthakKeshari please assign this issue to me

@aadityasinha-dotcom,
Kindly add your question to "String Handling" folder
Deadline - 02/10/2021
Gentle advice for futher Issues -
A bit more tough questions would be appreciated for Hacktoberfest tag.


[$500] [Wave Collect] [Ideal Nav] Make the header of the workspace list fixed

On the Workspace List page (URL: https://new.expensify.com/settings/workspaces), make the header of the list fixed.
The ideas was suggested in this internal Slack thread.
This is a potential pattern that we could adopt across the app wherever we have a list. As we will add another list on the Money page, that new list can also adopt this pettern.
Creating this issue for now so that we can track this in our backlog.
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b868d550d02a9a9a
Upwork Job ID:<PHONE_NUMBER>789016064
Last Price Increase: 2024-02-08
Automatic offers:

bernhardoj | Contributor | 0

Proposal
Please re-state the problem that we are trying to solve in this issue.
We want to make the workspace list header fixed.
What is the root cause of that problem?
A design change request.
What changes do you think we should make in order to solve the problem?

To make the header position fixed, we can add stickyHeaderIndices={[0]} to the list
https://github.com/Expensify/App/blob/b2a305a82efd39c9f32e46759c5a653ea68c575a/src/pages/workspace/WorkspacesListPage.tsx#L376-L380
Then, we will update the header list style to have a bg color and use padding instead of margin.
https://github.com/Expensify/App/blob/b2a305a82efd39c9f32e46759c5a653ea68c575a/src/pages/workspace/WorkspacesListPage.tsx#L210-L211
<View style={[styles.flexRow, styles.p5, styles.pl10, styles.appBG]}>

Can't access Slack Thread

@godofoutcasts94 you don't need access to the Slack thread to understand the requirements of this issue 👍

PR is ready
cc: @thesahindia

Merged yesterday

What's going on here? Does it need paying?

Is this issue open for proposals ? @trjExpensify

@trjExpensify yes, it needs paying

maybe we can added another step saying that the tester should shrink the screen vertically or ensure they have enough workspaces to scroll the list

Thanks @thesahindia and @hayata-suenaga , TestRail issue created
https://github.com/Expensify/Expensify/issues/377270

$500 approved for @thesahindia


[Enhancement] Double click on tab to close it

First thank you for your amazing work
Since NPAPI is no more used on browser, and the confinement of web-extensions it not possible anymore to make an extension to close tab on double click...
there for i am suggesting this feature
Thanks

I'm fine with this, as long as it lives behind a command-line flag (at most a setting behind chrome://flags). But, I can't guarentee that it will continue to be maintained in the event that the codebase changes significantly.
I'll see to adding this info into the README, since I think I've seen this come up often enough.

i made a modified version with an ugly patch to "chrome/browser/ui/views/tabs/tab.cc" around line 1000 next to middle click close tab function... if i can made my patch clean i will pull request...
i am just having trouble with libc version to make my compiled version work on my main machine which have older libc...

Here is my ugly patch... it works but it need some additional work
`--- tab.cc      2018-09-10 04:24:37.114899715 +0000
+++ chromnormal/ungoogled-chromium/build/src/chrome/browser/ui/views/tabs/tab.cc        2018-08-08 19:10:41.000000000 +0000
@@ -964,14 +964,10 @@
}
} else if (!IsSelected()) {
controller_->SelectTab(this);

   //intika

   dblClick = 0;
   base::RecordAction(UserMetricsAction("SwitchTab_Click"));
 }

} else if (!IsSelected()) {
controller_->SelectTab(this);

 //intika

 dblClick = 0;
 base::RecordAction(UserMetricsAction("SwitchTab_Click"));

}
ui::MouseEvent cloned_event(event_in_parent, parent(),
@@ -986,9 +982,6 @@
return true;
}

-//Intika Fucking dblclick
-int dblClick = 0;
void Tab::OnMouseReleased(const ui::MouseEvent& event) {
controller_->OnMouseEventInTab(this, event);
@@ -997,16 +990,13 @@
// In some cases, ending the drag will schedule the tab for destruction; if
// so, bail immediately, since our members are already dead and we shouldn't
// do anything else except drop the tab where it is.

if (controller_->EndDrag(END_DRAG_COMPLETE)){

   dblClick = 0;

  return;

}

if (controller_->EndDrag(END_DRAG_COMPLETE))
return;

// Close tab on µmiddle click, but only if the button is released over the tab

// Close tab on middle click, but only if the button is released over the tab
// (normal windows behavior is to discard presses of a UI element where the
// releases happen off the element).
if (event.IsMiddleMouseButton()) {

dblClick = 0;
if (HitTestPoint(event.location())) {
controller_->CloseTab(this, CLOSE_TAB_FROM_MOUSE);
} else if (closing_) {
@@ -1023,28 +1013,11 @@
// If the tab was already selected mouse pressed doesn't change the
// selection. Reset it now to handle the case where multiple tabs were
// selected.

//--------------------------------------------------------INTIKA
if (!IsSelected()) {

   dblClick = 0;

}

if (dblClick > 1) {

   dblClick = 0;

   if (HitTestPoint(event.location())) {

     controller_->CloseTab(this, CLOSE_TAB_FROM_MOUSE);

   } else if (closing_) {

     Tab* closest_tab = controller_->GetTabAt(this, event.location());

     if (closest_tab)

       controller_->CloseTab(closest_tab, CLOSE_TAB_FROM_MOUSE);

   }

} else { //------------------------------------------------INTIKA

   controller_->SelectTab(this);

      dblClick++;

   if (alert_indicator_button_ && alert_indicator_button_->visible() &&

       alert_indicator_button_->bounds().Contains(event.location())) {

     base::RecordAction(UserMetricsAction("TabAlertIndicator_Clicked"));

   }

controller_->SelectTab(this);

if (alert_indicator_button_ && alert_indicator_button_->visible() &&

   alert_indicator_button_->bounds().Contains(event.location())) {

 base::RecordAction(UserMetricsAction("TabAlertIndicator_Clicked"));

}
}
}
`

I made my custom version available here https://github.com/intika/UGChrome/

Do you still want a switch for this feature? If not, I'll close this.

just close it, it's a little bit out of the scope of the project, i will maintain this patch on https://github.com/intika/UGChrome/ tho

I don't have a use for this feature.
For anyone reading this, feel free to submit a PR if inclined.


YARN-10915. AbstractCSQueue: Simplify complex logic in methods: deriveCapacityFromAbsoluteConfigurations and updateEffectiveResources

…eCapacityFromAbsoluteConfigurations and updateEffectiveResources

Description of PR
How was this patch tested?
For code changes:

[ ] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m 39s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
31m 40s

trunk passed

+1 :green_heart:
compile
1m  1s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
1m  0s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
0m 49s

trunk passed

+1 :green_heart:
mvnsite
1m  2s

trunk passed

+1 :green_heart:
javadoc
0m 51s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 42s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
1m 58s

trunk passed

+1 :green_heart:
shadedclient
14m 56s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 54s

the patch passed

+1 :green_heart:
compile
1m  1s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
1m  1s

the patch passed

+1 :green_heart:
compile
0m 50s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 50s

the patch passed

+1 :green_heart:
blanks
0m  1s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 39s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 1 new + 57 unchanged - 0 fixed = 58 total (was 57)

+1 :green_heart:
mvnsite
0m 50s

the patch passed

+1 :green_heart:
javadoc
0m 37s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 35s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
2m  0s

the patch passed

+1 :green_heart:
shadedclient
14m 57s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
96m 45s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 31s

The patch does not generate ASF License warnings.

174m 34s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3418

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 3b70732bd41a 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 51856d3a84ad3b4484331a7af54b386b597a5c03

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/1/testReport/

Max. process+thread count
947 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 40s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
31m 45s

trunk passed

+1 :green_heart:
compile
1m  8s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
0m 57s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
0m 46s

trunk passed

+1 :green_heart:
mvnsite
1m  6s

trunk passed

+1 :green_heart:
javadoc
0m 47s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 42s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
1m 56s

trunk passed

+1 :green_heart:
shadedclient
15m  2s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 52s

the patch passed

+1 :green_heart:
compile
0m 58s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
0m 58s

the patch passed

+1 :green_heart:
compile
0m 48s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 48s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 38s
/results-checkstyle-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager: The patch generated 1 new + 57 unchanged - 0 fixed = 58 total (was 57)

+1 :green_heart:
mvnsite
0m 51s

the patch passed

+1 :green_heart:
javadoc
0m 37s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 35s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
1m 59s

the patch passed

+1 :green_heart:
shadedclient
14m 57s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
96m 33s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

173m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3418

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 329ea03e88ab 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 51856d3a84ad3b4484331a7af54b386b597a5c03

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/2/testReport/

Max. process+thread count
945 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 56s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
34m 19s

trunk passed

+1 :green_heart:
compile
1m  3s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
0m 53s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
0m 48s

trunk passed

+1 :green_heart:
mvnsite
1m  2s

trunk passed

+1 :green_heart:
javadoc
0m 47s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 44s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
1m 50s

trunk passed

+1 :green_heart:
shadedclient
15m  1s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 52s

the patch passed

+1 :green_heart:
compile
0m 56s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
0m 56s

the patch passed

+1 :green_heart:
compile
0m 48s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 48s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 38s

the patch passed

+1 :green_heart:
mvnsite
0m 51s

the patch passed

+1 :green_heart:
javadoc
0m 38s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 34s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
1m 59s

the patch passed

+1 :green_heart:
shadedclient
15m  4s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
95m 27s

hadoop-yarn-server-resourcemanager in the patch passed.

+1 :green_heart:
asflicense
0m 34s

The patch does not generate ASF License warnings.

175m 36s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3418

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux a3c8f266c0c9 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a61f59ba335114e4d96a9a331f5d0685b0f6991c

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/3/testReport/

Max. process+thread count
995 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Thanks @brumi1024 for working on this.
Latest state LGTM, committed to trunk.
Thanks @tomicooler for the review.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  4s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
32m 30s

trunk passed

+1 :green_heart:
compile
1m  4s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
compile
0m 55s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
checkstyle
0m 46s

trunk passed

+1 :green_heart:
mvnsite
1m  1s

trunk passed

+1 :green_heart:
javadoc
0m 47s

trunk passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 43s

trunk passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
1m 54s

trunk passed

+1 :green_heart:
shadedclient
15m  0s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 53s

the patch passed

+1 :green_heart:
compile
0m 55s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javac
0m 55s

the patch passed

+1 :green_heart:
compile
0m 47s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
javac
0m 47s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 38s

the patch passed

+1 :green_heart:
mvnsite
0m 51s

the patch passed

+1 :green_heart:
javadoc
0m 39s

the patch passed with JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04

+1 :green_heart:
javadoc
0m 35s

the patch passed with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

+1 :green_heart:
spotbugs
1m 53s

the patch passed

+1 :green_heart:
shadedclient
14m 52s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
50m 41s
/patch-unit-hadoop-yarn-project_hadoop-yarn_hadoop-yarn-server_hadoop-yarn-server-resourcemanager.txt
hadoop-yarn-server-resourcemanager in the patch passed.

-1 :x:
asflicense
0m 33s
/results-asflicense.txt
The patch generated 70 ASF License warnings.

128m 56s

Reason
Tests

Failed junit tests
hadoop.yarn.server.resourcemanager.security.TestDelegationTokenRenewer

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3418

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell

uname
Linux 832782ff4f3f 4.15.0-151-generic #157-Ubuntu SMP Fri Jul 9 23:07:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 93fca3abe45840a17bc05fa83ad405ad318e663c

Default Java
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/4/testReport/

Max. process+thread count
968 (vs. ulimit of 5500)

modules
C: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager U: hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3418/4/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.


Fix global options for spo list webhook get and set

We have missing global options for commands spo list webhook get and spo list webhook set in the docs. This is because an unnecessary backslash character was inserted in the global options path.
https://github.com/pnp/cli-microsoft365/blob/6d50b23db34ff34299763af8dddbad2f5179c4af/docs/docs/cmd/spo/list/list-webhook-get.md?plain=1#L28
https://github.com/pnp/cli-microsoft365/blob/6d50b23db34ff34299763af8dddbad2f5179c4af/docs/docs/cmd/spo/list/list-webhook-set.md?plain=1#L34

assign me please :-)

Awesome, thx @nicodecleyre!


update the width to match the parent.

IMPORTANT: All PRs must be linked to an issue (except for extremely trivial and straightforward changes).
Fixes #2057
Description
Update the imageview width to match the parent.
Alternative(s) considered
Have you considered any alternatives? And if so, why have you chosen the approach in this PR?
Type
Choose one: Bug fix
Screenshots (if applicable)
Checklist

[x] I have read and acknowledged the Code of conduct.
[x] I have read the Contributing page.
[x] I have signed the Google Individual CLA, or I am covered by my company's Corporate CLA.
[x] I have discussed my proposed solution with code owners in the linked issue(s) and we have agreed upon the general approach.
[x] I have run ./gradlew spotlessApply and ./gradlew spotlessCheck to check my code follows the style guide of this project.
[x] I have run ./gradlew check and ./gradlew connectedCheck to test my changes locally.
[x] I have built and run the demo app(s) to verify my change fixes the issue and/or does not break the demo app(s).

Before Fix
https://github.com/google/android-fhir/assets/86107848/f610aca3-006f-4992-8094-4f7796078fbf
After fix :
https://github.com/google/android-fhir/assets/86107848/38cde211-fd81-4052-accd-df8c3019a097


bom material index should more clearly point to bom

vw_bom_material_index should reference the bom to which an entity belongs.
I implemented a partial fix that works for elements in vw_bom_material but not vw_bom_material_composite.

This turned out to be pretty easy to fix with a sql CASE


Mention How to Use Static IPs in Other Resource Groups

This page could benefit from an explanation and example of how to use a static IP in a resource group that is not the node-managed one.
This can be done with the following additional override to the ingress-nginx Helm install:
--set controller.service.annotations."service\.beta\.kubernetes\.io/azure-load-balancer-resource-group"=$static_ips_resource_group  \

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 12009c1f-282d-2632-fac0-64b09c432833
Version Independent ID: 6052d3d4-d295-9340-b16d-4c200e34cd73
Content: Use ingress controller with static IP - Azure Kubernetes Service
Content Source: articles/aks/ingress-static-ip.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

Thanks for the feedback!  I have assigned the issue to the content author to investigate further and update the document as appropriate.

Thank you for the feedback.
We've added a task to our backlog and will prioritize accordingly.
#please-close


attempted to update to the last rustc version

!!! WARNING !!! This is not compilable. I've tried to update, but got stuck with unable to infer enough type information about closure error.

Your gl-rs fork is out of date from head by a few weeks. Can you rebase this change?

Sorry, forgot to update...
But it looks to be fixed now.


Extend .gdns class

I'm trying to extend the mycppnode written in c++ inheriting the godot::Node class using this syntax in the gdscript:
extend "res://bin/mycppnode.gdns"
but it gives the following parse error:
res://myscript.gd:1 - Parse Error: Couldn't load the base class: res://bin/mycppnode.gdns
 modules/gdscript/gdscript.cpp:576 - Method failed. Returning: ERR_PARSE_ERROR
 modules/gdscript/gdscript_tokenizer.cpp:1129 - Condition "tk_rb[ofs].type != TK_IDENTIFIER" is true. Returned: StringName()

When I simply extend a Node and assign the script using the gui it doesn't give any error.

Are you able to preload(res://bin/mycppnode.gdns and use it in a test scene?

Actually it looks like it's not supported, judging by this issue: https://github.com/GodotNativeTools/godot-cpp/issues/291

I ran into this too, it's an annoying limitation...


Remove properties which encapsulate reference types

the access protection properties offer in these case is misleading as many of the types they protect contain methods for modifying them. All the private set of properties accomplishes is preventing other classes from change what object the property reference, not the details of the property itself.

these are used much more sparingly now, there are still some but only where they make sense


Update Swift Documentation

Update Swift Documentation import.

👍

thanks for the contribution @quentinR! as silly as it seems, I do need you to sign our Individual Contributor License Agreement (it makes our lawyers happy). then we'll be good to merge!
-luna


[python O11Y] Refactor census propagation flow.

Refactored OpenCensus context propagation flow, now propagation happens for each call and context will be automatically propagated from gRPC server to gRPC client.
We're using execution_context in OpenCensus since the context is related to OpenCensus and it helps wrap contextVar for us.
Testing

Added a new Bazel test case for context propagation.

@gnossen PTALA


if ERR__QUEUE_FULL, use list instead of list_deals

LGTM

@haipome Can you please verify it?

Thanks, this is a mistake.


Improve basic Position from Server

Currently the position sync is jittery. It needs more smoothing and some client side prediction.

Gah, found a bug where I was applying movements only when the server sent a movement update. That was the main reason movement looked laggy

Fixed!


Validate form filename when uploading

If a form contains spaces or some special characters, it can make the openmapkit-server instance break, so we need to validate it when uploading.

Modified the frontend to show the error message returned by the backend.
Added a xls form filename validation for special characters. https://github.com/hotosm/OpenMapKitServer/pull/101/commits/726bbf9643dbdb7d0a64c6297db08a96b97d889a

https://github.com/hotosm/OpenMapKitServer/pull/101/


Advanced config reflects 'conf' changes

After the setting in the constructor advancedConfig.interface is not updated from the conf and always stay consumer-api.

Merged with gratitude. Thank you!

:tada: This PR is included in version 2.35.0 :tada:
Your semantic-release bot :package::rocket:


Develop

Merging develop and updating to release v0.7 per conversation with @pnrobinson and @vidarmehr

@pnrobinson and @vidarmehr -- Good with me merging and tagging new release as v0.7?

Looks great!

Excellent, thanks @pnrobinson -- merging now.


Problem with timestamp and timezone

Hello!
I tried this code:
const cal = new CalHeatmap()

cal.paint({
  data: {
    source :  [{t:<PHONE_NUMBER>000, value: 6}],
    x : 't',
    y : 'value',
    groupY : 'sum'
  },
  range: 2,
  date: { timezone: 'Europe/Paris', start: new Date('2023-10-4') },
  domain:{
    type: "day",
    label:{
      position: 'top'
    }
  },
  subDomain:{
    type: "hour",
    label: 'H'
  },
  scale: {
    color:{
      scheme: 'Cool',
      type: 'linear',
      domain: [0,30]
    }
  }
});

But even if I change the timezone, the date is always the same (in UTC).
I also tried a date like 2023-10-04T14:03:31.040000+00:00 or 2023-10-04T16:03:31.040000+02:00 but same problem.
Thank you!

Perhaps this is the problem?
https://github.com/wa0x6e/cal-heatmap/blob/6cb8ddf3b40e8b3f0c5522b89f1c89e4eb7d6981/src/helpers/DateHelper.ts#L161

I think it works by removing this line, thanks!

@ValentinS33 did you have any luck in getting this to appear in the proper local timezone?

No :( For the moment, I use only Europe/Paris timezone so I set .utcOffset(1) and and I will change to .utcOffset(2) when there is the time change and the 2 hour difference in France...

也许这就是问题所在？
https://github.com/wa0x6e/cal-heatmap/blob/6cb8ddf3b40e8b3f0c5522b89f1c89e4eb7d6981/src/helpers/DateHelper.ts#L161

👍🏻


fix: Add missing comma

Add missing comma

You are indeed right @webknjaz -- apologies for the noise.


Remove assert usage in comparison code

The comparison code was originally written to only be used from within tests, so used assertions to detect and report identified differences. It was converted to report errors and be usable as a CLI (outside of tests), but this instance of assert was missed in the conversion.
This is a minor fix to clean it up after noticing it when reviewing other code.
Testing:

[x] Ran tests and they passed OK
[x] Added new tests for the new feature(s)

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- improver_tests/utilities/test_compare.py  3

See the complete overview on Codacy

Looks good to me!


Adding the Evmos network information

You need to sign the CLA

Coverage decreased (-6.1%) to 85.219% when pulling aec126767fd98508f6a06d9e0b2960334e1d9fe9 on tharsis:feature/add_evmos_network into dd99f9ef304fc6734947b642096ddbe14ae7e7bf on gnosis:master.

I have read the CLA Document and I hereby sign the CLA


add back authz functionality

mainly affects the toProto methods in Device.

token validation should follow RFC 9068, relevant section copy/pasted here:
Resource servers receiving a JWT access token MUST validate it in the
following manner.

The resource server MUST verify that the "typ" header value is
"at+jwt" or "application/at+jwt" and reject tokens carrying any
other value.

If the JWT access token is encrypted, decrypt it using the keys
and algorithms that the resource server specified during
registration.  If encryption was negotiated with the authorization
server at registration time and the incoming JWT access token is
not encrypted, the resource server SHOULD reject it.

The issuer identifier for the authorization server (which is
typically obtained during discovery) MUST exactly match the value
of the "iss" claim.

The resource server MUST validate that the "aud" claim contains a
resource indicator value corresponding to an identifier the
resource server expects for itself.  The JWT access token MUST be
rejected if "aud" does not contain a resource indicator of the
current resource server as a valid audience.

The resource server MUST validate the signature of all incoming
JWT access tokens according to [RFC7515] using the algorithm
specified in the JWT "alg" Header Parameter.  The resource server
MUST reject any JWT in which the value of "alg" is "none".  The
resource server MUST use the keys provided by the authorization
server.

The current time MUST be before the time represented by the "exp"
claim.  Implementers MAY provide for some small leeway, usually no
more than a few minutes, to account for clock skew.

The resource server MUST handle errors as described in Section 3.1 of
[RFC6750].  In particular, in case of any failure in the validation
checks listed above, the authorization server response MUST include
the error code "invalid_token".  Please note that this requirement
does not prevent JWT access tokens from using authentication schemes
other than "Bearer".


Change text programmatically

If I set the text property to a string after initialization, it loads, but the text is truncated. Is this the right way to set text in Notepad?

Just tried both macOS and iOS sample codes with
DispatchQueue.main.asyncAfter(deadline: .now() + .seconds(3)) { editor.text = text }

To see what happens when the view is already loaded and you reset the text. Works fine. Can you provide more context information?

The problem was the string length calculation in Storage.replaceCharacters. I submitted a pull request with the fix. For certain strings, str.characters.count does not return the correct length, NSString.length does.

Of course it doesn't, good catch, thanks! So it's solved now?

Yes, I think so.
On Sun, Jul 30, 2017 at 9:11 PM, Christian Tietze<EMAIL_ADDRESS>wrote:

Of course it doesn't, good catch, thanks! So it's solved now?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ruddfawcett/Notepad/issues/25#issuecomment-318888770,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARoo1EZ8HGaCozhCz0Wo-zRfonMH3h1bks5sTEjWgaJpZM4OnLYY
.


FIX: Use circular-json to handle circular references (fix #5)

Ok, I found that the problem illustrated in #5 is not related to circular dependency in js module. The real reason is the use of the browserify.transform option field in a node package when used with watchify (circular reference with packageCache).
I made a small project to illustrate the issue: https://github.com/nopnop/fix-circular-flatcache
Using the suggested https://github.com/WebReflection/circular-json for both serializing and deserializing fix the issue nicely.
The test passes: yet I'm not sure how to handle this special case of Circular Reference in the test suite.
The stateful fileCache variable I've added is used to reproduce the caching mechanism of the removed read-json-sync package

Very nice!, Since this is also used by eslint I guess I will consider to publish this under a major version to avoid breaking it, until we have a chance to test it properly.
About how to add a test, you're right it might not be that simple, but I guess that the easier will be to create a circular structure and assigning it to a key, and then attempt to serialize it.
Once you add the test, I will be very happy to merge this. Thank you so much for your time and effort.

There is just a minor suggestion and I will merge. This is really awesome!

Thanks! I was on the go yesterday: I haven't took the time to explain my last commits..
I'll push a fix on the readJSON (see my answer above).

Hi @nopnop
I really appreciate the help, thank you for your PR it was merged already. Will publish this soon.

Hi @nopnop, it is published now as<EMAIL_ADDRESS>can you do a PR to persistify to update the dependency?
Thank you.

Hi @royriojas,
I've made a bump PR.
Many thanks!
(And sorry for the missed files field )


This extension change my website design

This extension change my website design.
Some padding and other css style was modified with this extension.

Yes, there are some global overrides in the builds. This is sadly something V1 is stuck with. V2 is 100% insular components that don't affect anything global so these occurrences won't happen.


stringsThatMean rework

Pull Request Description
What issue does this change request address? (Use "#" before the issue to link it, i.e., #42.)
Closes #1191
What are the significant changes in functionality due to this change request?
Abstracts string truth checking to a utils method instead of a utils-list-of-strings method.
Also limits valid user choices to a more maintainable set, keeping the legacy set for reference in the event they are needed.

For Change Control Board: Change Request Review
The following review must be completed by an authorized member of the Change Control Board.

[ ] 1. Review all computer code.
[ ] 2. If any changes occur to the input syntax, there must be an accompanying change to the user manual and xsd schema. If the input syntax change deprecates existing input files, a conversion script needs to be added (see Conversion Scripts).
[ ] 3. Make sure the Python code and commenting standards are respected (camelBack, etc.) - See on the wiki for details.
[ ] 4. Automated Tests should pass, including run_tests, pylint, manual building and xsd tests. If there are changes to Simulation.py or JobHandler.py the qsub tests must pass.
[ ] 5. If significant functionality is added, there must  be tests added to check this. Tests should cover all possible options.  Multiple short tests are preferred over one large test. If new development on the internal JobHandler parallel system is performed, a cluster test must be added setting, in  XML block, the node <internalParallel> to True.
[ ] 6. If the change modifies or adds a requirement or a requirement based test case, the Change Control Board's Chair or designee also needs to approve the change.  The requirements and the requirements test shall be in sync.
[ ] 7. The merge request must reference an issue.  If the issue is closed, the issue close checklist shall be done.
[ ] 8. If an analytic test is changed/added is the the analytic documentation updated/added?
[ ] 9. If any test used as a basis for documentation examples (currently found in raven/tests/framework/user_guide and raven/docs/workshop) have been changed, the associated documentation must be reviewed and assured the text matches the example.

the Database in  tests/framework/DataObjects/DataobjectsAttributes/DatabaseStorage/testGridRavenDatabase.h5

has been added again

I removed the database file; I feel like I've made that commit before. We'll see if it causes failed tests.

It appears that database is required for one of the tests:
https://civet.inl.gov/job/492012/
What would you like to do?

@alfoa @Jimmy-INL tests passed.

Interesting the database thing... I will investigate further O_o

PR approved... Checklist passed...


Read string pointers the right way

By getting the value from aws.StringValue(s *string) string.
Which returns an empty string "" if the pointer is nil. Prevents potential seg. faults.
Fixes #171

LGTM. Thanks @dtx


Fatal error:  Uncaught TypeError: Argument 3 passed

Fatal error:  Uncaught TypeError: Argument 3 passed to Hitrov\OciApi::checkExistingInstances() must be of the type string, bool given, called in /home/ubuntu/oci-arm-host-capacity/index.php on line 76 and defined in /home/ubuntu/oci-arm-host-capacity/src/OciApi.php:135
Stack trace:
#0 /home/ubuntu/oci-arm-host-capacity/index.php(76): Hitrov\OciApi->checkExistingInstances()
#1 {main}
thrown in /home/ubuntu/oci-arm-host-capacity/src/OciApi.php on line 135
After doing the tutorial got this error.  I am assuming this is related to how I declared the variables.
I have never worked with php, therefore I apologize if this is an easy fix.
PHP 7.4.33 (cli) (built: Feb 14 2023 18:31:54) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
with Zend OPcache v7.4.33, Copyright (c), by Zend Technologies
My installed php version.

Please revert index.php and check README for .env file usage

I am getting a "Fatal error: Uncaught TypeError: Argument 1 passed to Hitrov\OciConfig::__construct() must be of the type string, bool given, called in /www/wwwroot/..."

This could happen when env file wasn’t parsed (not found) - you didn’t rename .env.example to .env, for instance.

Thank you very much for the help. Actually, I did the it using CMD in my windows. It worked perfectly. Then I zipped all files and uploaded to my Apache web server and opened the index.php file in browser using domain name.


Modified dashboard page

Changes proposed in this pull request:

Modified dashboard page UI to make it similar to the current EvalAI UI

Link to live demo: http://pr-141-evalai.surge.sh

Codecov Report

Merging #141 into master will increase coverage by 0.75%.
The diff coverage is 67.22%.

@@            Coverage Diff            @@
##           master    #141      +/-   ##
=========================================
+ Coverage   53.35%   54.1%   +0.75%
=========================================
  Files          55      59       +4
  Lines        2356    2497     +141
  Branches      253     263      +10
=========================================
+ Hits         1257    1351      +94
- Misses       1026    1065      +39
- Partials       73      81       +8

Impacted Files
Coverage Δ

...onents/home/twitter-feed/twitter-feed.component.ts
100% <ø> (ø)
:arrow_up:

src/app/components/nav/footer/footer.component.ts
62.5% <100%> (+1.2%)
:arrow_up:

...omponents/nav/dash-footer/dash-footer.component.ts
100% <100%> (ø)

.../components/utility/side-bar/side-bar.component.ts
100% <100%> (ø)

...d/dashboard-content/dashboard-content.component.ts
51.21% <51.21%> (ø)

...nents/nav/header-static/header-static.component.ts
74.24% <57.14%> (+0.1%)
:arrow_up:

src/app/services/auth.service.ts
36.75% <57.14%> (+13.46%)
:arrow_up:

...nents/nav/simple-header/simple-header.component.ts
80% <80%> (ø)

...rc/app/components/dashboard/dashboard.component.ts
87.5% <80%> (+7.01%)
:arrow_up:

src/app/services/endpoints.service.ts
62.9% <0%> (-3.23%)
:arrow_down:

... and 4 more

Impacted Files
Coverage Δ

...onents/home/twitter-feed/twitter-feed.component.ts
100% <ø> (ø)
:arrow_up:

src/app/components/nav/footer/footer.component.ts
62.5% <100%> (+1.2%)
:arrow_up:

...omponents/nav/dash-footer/dash-footer.component.ts
100% <100%> (ø)

.../components/utility/side-bar/side-bar.component.ts
100% <100%> (ø)

...d/dashboard-content/dashboard-content.component.ts
51.21% <51.21%> (ø)

...nents/nav/header-static/header-static.component.ts
74.24% <57.14%> (+0.1%)
:arrow_up:

src/app/services/auth.service.ts
36.75% <57.14%> (+13.46%)
:arrow_up:

...nents/nav/simple-header/simple-header.component.ts
80% <80%> (ø)

...rc/app/components/dashboard/dashboard.component.ts
87.5% <80%> (+7.01%)
:arrow_up:

src/app/services/endpoints.service.ts
62.9% <0%> (-3.23%)
:arrow_down:

... and 4 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 a5c7cae...639df0f. Read the comment docs.

@lunayach this can be reviewed now

@sanketbansal resolve the conflicts.
Since, #182 and #158 have been merged, also clean this PR.

@sanketbansal resolve the conflicts.
Since, #182 and #158 have been merged, also clean this PR.

okay

@sanketbansal please verify the changes once before pushing. This big image comes when visiting any link in the application:

@lunayach it's working fine on my machine.

@lunayach it's working fine on my machine.

Yes, re-building it helped. But as you might see, there is something wrong in the Home-page UI. Would this get fixed in #140?

@sanketbansal when I click on the available options on the side-bar (i.e. All Challenges, Hosted Challenges, etc.), the corresponding side-bar having these options dis-appears. (unlike current version). Can you please check this?

@sanketbansal when I click on the available options on the side-bar (i.e. All Challenges, Hosted Challenges, etc.), the corresponding side-bar having these options dis-appears. (unlike current version). Can you please check this?

That's not a bug actually after clicking on the options it gets routed to other components which are
being modified in different PR. There side bar remains intact.

@sanketbansal  The stats for the GitHub are not displayed in one go and I have to do a refresh and only then they are displayed.


Pull docker image from Amazon ECR

For anonymous users of Docker Hub, the rate limit is set to 100 pulls per 6 hours per IP address. This is an issue with large runners or runners that have static IPs. AWS ECR does not have this constraint.
Could you please add the ability to use any given registry, like AWS ECR?
container:
      image: cypress/browsers: latest
      options: --user 1001

@h2oearth
According to GitHub documentation Defining the container image:

Use jobs.<job_id>.container.image to define the Docker image to use as the container to run the action. The value can be the Docker Hub image name or a registry name.

It sounds like what you are asking for is already available from GitHub Actions. Would you like to cross-check?

According to Cypress documentation Continuous Integration > AWS CodeBuild, Cypress Docker images are also available on the Amazon Elastic Container Registry (ECR) and can be found under https://gallery.ecr.aws/cypress-io.
These images can be used in Cypress GitHub Action workflows, for example:
    container:
      image: public.ecr.aws/cypress-io/cypress/browsers:latest
      options: --user 1001

Since the functionality being requested is already available, I will close this issue.