"az aks update" ignores --load-balancer-outbound-ports 0

Describe the bug
az aks update ignores the --load-balancer-outbound-ports (load balancer outbound allocated ports) parameter if the intended new value is 0 (which is valid).
We can clearly see in debug that the PUT request to Microsoft.ContainerService/managedClusters sends the currently configured value instead of the desired value of 0.
Related command
az aks update
Errors
No error, the 0 parameter is silently ignored
Issue script & Debug output
Let's set load-balancer-outbound-ports to a non-zero value (8000) first:
$ az aks update -n foo -g maxhorstmann-test --load-balancer-managed-outbound-ip-count 1 --load-balancer-outbound-ports 8000 --debug

...
cli.azure.cli.core.sdk.policies: Request URL: 'https://management.azure.com/subscriptions/137f0351-8235-42a6-ac7a-6b46be2d21c7/resourceGroups/maxhorstmann-test/providers/Microsoft.ContainerService/managedClusters/foo?api-version=2023-02-01'
cli.azure.cli.core.sdk.policies: Request method: 'PUT'
...
cli.azure.cli.core.sdk.policies: Request body:
cli.azure.cli.core.sdk.policies: {..."allocatedOutboundPorts": 8000}...
...

^^ This is correct.
Now let's try setting load-balancer-outbound-ports to 0:
$ az aks update -n foo -g maxhorstmann-test --load-balancer-managed-outbound-ip-count 1 --load-balancer-outbound-ports 0 --debug
...
cli.azure.cli.core.sdk.policies: Request URL: 'https://management.azure.com/subscriptions/137f0351-8235-42a6-ac7a-6b46be2d21c7/resourceGroups/maxhorstmann-test/providers/Microsoft.ContainerService/managedClusters/foo?api-version=2023-02-01'
cli.azure.cli.core.sdk.policies: Request method: 'PUT'
...
cli.azure.cli.core.sdk.policies: Request body:
cli.azure.cli.core.sdk.policies: {..."allocatedOutboundPorts": 8000}...
...

^^ We can see that the PUT request sent the current value (8000) instead of the desired new value (0).
Expected behavior
The PUT request should send {..."allocatedOutboundPorts": 0}...
Environment Summary
azure-cli                         2.50.0 *
core                              2.50.0 *
telemetry                          1.0.8 *
Extensions:
containerapp                      0.3.28
Dependencies:
msal                              1.22.0
azure-mgmt-resource             23.1.0b2
Python location '/opt/homebrew/Cellar/azure-cli/2.50.0_1/libexec/bin/python'
Extensions directory '/Users/maxhorstmann/.azure/cliextensions'
Python (Darwin) 3.10.12 (main, Jul 28 2023, 18:34:01) [Clang 14.0.3 (clang-14<IP_ADDRESS>.1)]
Additional context
IcM 411113950

Thank you for opening this issue, we will look into it.

Non customer reported issue. Adding Service team to look into this.

ack, taking a look

The issue has been fixed in PR #27050 and released with 2.52.0

The issue has been fixed in PR #27050 and released with 2.52.0

Hi Fuming Zhang
Just tested this on my side and it seems issue still present
I am using 2.56.0 CLI version
From my test
az aks show --resource-group newResourceGroup --name testenew123 | grep allocatedOutboundPorts
WARNING: The behavior of this command has been altered by the following extension: aks-preview
"allocatedOutboundPorts": 2048,
az aks update -g newResourceGroup -n testenew123 --load-balancer-outbound-ports 0 --debug
but after this last command I can see that 2048 remains, when running show command
Can you help please?

WARNING: The behavior of this command has been altered by the following extension: aks-preview

@rabpoi, could you please uninstall aks-preview and try again? Also can you share which version aks-preview are you using (az version)?
@MartinForReal, the previous fix #27050 was applied in azure-cli 2.52.0, but aks-preview relies on 2.49.0, so maybe there is still this problem in aks-preview?

WARNING: The behavior of this command has been altered by the following extension: aks-preview

@rabpoi, could you please uninstall aks-preview and try again? Also can you share which version aks-preview are you using (az version)?
@MartinForReal, the previous fix #27050 was applied in azure-cli 2.52.0, but aks-preview relies on 2.49.0, so maybe there is still this problem in aks-preview?

@FumingZhang my aks-preview version is 1.0.0b2
az version
{
"azure-cli": "2.56.0",
"azure-cli-core": "2.56.0",
"azure-cli-telemetry": "1.1.0",
"extensions": {
"aks-preview": "1.0.0b2",
"amg": "1.2.9"
}
}

@FumingZhang
Already uninstall aks-preview and issue still the same
Now new version installed is "aks-preview": "1.0.0b3"

@rabpoi
The issue has been fixed in azure-cli since 2.58.0 (#28273) and aks-preview since 2.0.0b7 (#7230).


DMOD-227 drop unused vertx-auth-jwt dependency

This avoids a security audit of vertx-auth-jwt.

Can one of the admins verify this patch?

Thanks anyway, Julian. Closing this PR, as Kurt did this as part of commit b9fc1c319a45ff99da031c1e1519e08e9be416af


Accordion component page can follow GOV.UK style better for numbers

In the 'Accordions, tabs and details' section of the accordion component page.
"better if you only have one or 2 sections" can be improved to "better if you only have 1 or 2 sections"
The GOV.UK style guide says it’s OK to write 1 numerically if it’s directly next to another number and would look strange written out.
Screenshot of issue

Step by step
1. Claim this issue
Comment below to let other people know that you're working on this issue.
2. Edit the markdown file
You may be asked if you want to fork this repository.
You'll need select 'fork this repository and propose changes'.

You can find the markdown file for this change by going to:
https://github.com/alphagov/govuk-design-system/edit/master/src/components/accordion/index.md.njk#L52
In the markdown file make the following changes:

Tabs may work better for users who need to switch quickly between 2 sections. Accordions push other sections down the page when they open, but tabs do not move which makes it easier to switch.

- Consider using an accordion instead of the [details](/components/details/) component if there are multiple related sections of content. The details component might be better if you only have one or 2 sections of content. The details component is less visually prominent than an accordion, so tends to work better for content which is not as important to users.
+ Consider using an accordion instead of the [details](/components/details/) component if there are multiple related sections of content. The details component might be better if you only have 1 or 2 sections of content. The details component is less visually prominent than an accordion, so tends to work better for content which is not as important to users.

## How it works

3. Propose a change
Once you’re happy, scroll to the bottom of the page to ‘propose file change’.
Add a brief description of what you have changed in the first field. This information will be added to the file’s changelog. Try to be as clear as possible, so that future users can understand what you have done.
If you need to provide information about why you made the change, you can add more detail in the larger field below.
When you are happy with your description, select ‘propose file change’. You will have a chance to review and confirm your changes on the next page.

4. Confirm your changes
You will be shown a confirmation page where you can review the changes you’ve made.  If you spot a mistake, you can go back to the previous page and correct it.  If you are happy, select ‘Create pull request’. You will have one more chance to review your change on the next page before you submit it to the GOV.UK Design System team to review.

5. Create a pull request
A pull request is a request sent to the team who maintains the project you’re making changes to – in this case, the GOV.UK Design System team.
It tells them what you’ve changed and gives them the option to ‘pull’ your changes into the project where it can then be published.
At this point, you can write additional comments about the work you’ve done if you want to.
These will be shared with the GOV.UK Design System team when you submit your proposed change.
It will be pre-filled with the information you entered when you proposed the file change in step 3. Please be aware that updating the information here will not change the original description you provided, but will send it in addition.

Once you’re happy, select ‘Create pull request’.
6. Done, wait for a review from the team
The GOV.UK Design System team will be notified of your proposal and will review it as soon as possible.
Questions
If you’ve got a question, you can comment in this issue or email the Design System team on<EMAIL_ADDRESS>

The 'Numbers' section of the style guide states:

Write all other numbers in numerals (including 2 to 9) except where it’s part of a common expression like ‘one or two of them’ where numerals would look strange.

So, the update should probably be to "one or two," instead of "1 or 2."


Harmonize pretty print settings

In the same principle as #773, different code uses different styles for keyword/identifier upper/lower case.
Propose standardising on the most commonly used format of keyword upper / identifier lower case to minimise large diffs due to pretty print.
See also Style Guides on the topic.

also see https://github.com/sapmentors/abap2xlsx/issues/658

I did search for an issue about this, but only open ones.

So, if I understand it correctly, this should be once-in-project-lifetime occurrence: once the "keywords in uppercase / identifiers in lowercase" consensus reached in #658 is implemented, we shouldn't have to deal with this issue any more.
In this case, I'll move this one to my todolist and review it quickly.

Correct, and with abapLint we can verify that all PRs conform in future.

Guess I'll now have to learn how to deploy the linter in my setup, now :)


Move to tower

This is an attempt to move the request client to tower.
In the future this will help us with

~improved retry handling~ #981
middleware support
better caching
...

Thanks to @ddprrt for the support to fix the compilation issues.

There are two things that I found out:

You introduced new generics like T and the error messages are related to that. Throughout your flow T is supposed to be generic but it becomes a concrete type at some point (Request). Rust can't substitute within a function. Changing it back to Request makes those type errors go away, but I wonder if you had anything particular in mind when introducing generic type parameters?

When I change T back to Request, this part throws some nasty errors:

    tokio::spawn(async move {
        futures::StreamExt::for_each_concurrent(
            ReceiverStream::new(recv_req),
            max_concurrency,
            |request: Result<T>| async {
                let request = request.expect("cannot read request");
                let response = handle(&mut client, cache.clone(), request).await;

                send_resp
                    .send(response)
                    .await
                    .expect("cannot send response to queue");
            },
        )
        .await;
    });

First it can't pass a mutable reference to client within a spawned future and async block, and if I introduce an Arc or similar it turns out that BoxService is not Sync... and I honestly don't know how to get rid of that right now.
But before going into that rabbit hole, I thought I'd ask about the generic type parameters and what your intention was :) Maybe that makes a few things more clear to me

Thanks a lot for your comment. I really appreciate the support.
The idea behind the generic type was that I didn't want to be dependent on any single request implementation (that of reqwest or any other). However in hindsight that is unnecessary because we fully control the internals of the library and could change it if we wanted to, which is - quite frankly - unlikely.
With that in mind, let's change it back to the concrete type if it makes things easier.
As for BoxService, my guess is that the canonical way might be to use SyncWrapper as described in https://github.com/tower-rs/tower/issues/697, which also includes a link to a pull request which does just that. What do you think?
Maybe if we use a concrete type and SyncWrapper we might get closer to a working version.

Closing this as I don't see it getting merged anytime soon. I want to revisit this idea at some point once I'm a bit clearer of what I want to achieve by moving to Tower. We got better retry-handling now and the middleware part is still a bit far in the future, so no need to merge a rushed design. Thanks, @ddprrt, for the feedback and support, though!


Example for Swin Transformer

Added a directory name usage_example
Moved vit_example.py to usage_example
changed model.blocks[-2].norm1 to model.blocks[-1].norm1 in usage_example/vit_example.py
Added a new swinT_example.py to try out gradient mapping with Swin Tranformers
Added a new section Swin Transformer in Readme.md with generated examples and method to use Swin Transformer with the library.
Changed references formating

Hi, thanks!
Some comments:

The dog category examples for the swin transformer don't look convincing
I think we should improve them first.
Is this with the smoothing? If yes, maybe try without?
Try an earlier block?
Lets please keep the previous reference formatting

I'll try it on with other blocks
Please tell me what is the ImageNet class index of that dog and the cat as shown in images, this can make it easier for me
Is it possible to use other images for generating such examples?

Category
GradCAM
AblationCAM
ScoreCAM

Dog

Cat

I'm sorry, I checked and found that the previous maps were generated using ImagNet 22k with untrained Classifier head.
These are the changes now
I'll undo the references formatting

Thanks!
Another comment -
Looks like one of the Swin transformer dog images is from ViT
./examples/vit_cat_scorecam_cam.jpg

Changed that.

Super cool.
The Swin Transformer images are pretty convincing.
Thanks for your contribution!


Add Workflow to Build, Test, and Publish Image

This PR sets up a workflow to build and test the project's container on every push to master and every PR.
There is also a workflow to publish the built container when needed.

Oh wow, this is absolutely fantastic! Thank you so much for doing this! :)


Thin the noise in a multi-workspace-root for debugging.

The debug configurations set for a workspace should only show for the workspace of the currently open file.  If I have 23 repos, 23 x number of configurations in .vscode/launch.json of the same thing in the drop down is daunting to sort through.
In addition, debug breakpoints should only show for the workspaces of currently open files.  With 23 sets worth of debug breakpoints the ones I am working with are quickly lost.  Ideally the workspace would be a line seperating groups of active breakpoints instead of a mixed alphabetically sorted list.
Also, there does not seem to be a way to make .vscode pick up the multi-workspace-root launch configurations and use those for all open workspaces.  Thuis causing identical configurations to need to be duplicated and maintained in each workspace.

Filtering down the list is a fair feature request thus I can keep this open.
As for .vscode picking up a launch configuration to use for all open workspaces please checkout the launch configurations in a workspace configuraiton file https://code.visualstudio.com/updates/v1_20#_improved-debugging-support-for-multi-root-workspaces


Add toContainAnyEntries matcher

What
I added a toContainAnyEntries matcher

Why
This matcher checks if object contains at least one of the provided entries

Notes
Housekeeping

[x] Unit tests
[x] Documentation is up to date
[x] No additional lint warnings
[x] Add yourself to contributors list (yarn contributor)

@vikneshwar I think you need to fix the snaphots for the test you can do this with yarn test --u

Codecov Report

Merging #79 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #79   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          70     72    +2
  Lines         318    330   +12
  Branches       49     51    +2
=====================================
+ Hits          318    330   +12

Impacted Files
Coverage Δ

src/matchers/toContainAnyEntries/index.js
100% <100%> (ø)

src/matchers/toContainAnyEntries/predicate.js
100% <100%> (ø)

Continue to review full report at Codecov.

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

Hi @mattphillips thanks for the help. Fixed it.

Thanks a lot @vikneshwar for your contribution! :smile:


no paste item when select a file

when we change a file , we ofter need to make a copy first.
but now before paste a file  we have to select a folder,  if too many files in a folder, it's easy to select to wrong folder.
if we select any file and paste, it's certain in the same folder.

Did you try any extensions? For example https://marketplace.visualstudio.com/items?itemName=sleistner.vscode-fileutils looks relevant.


Mamba alternative to conda

Pull Request Description
What issue does this change request address?
Closes #2106
What are the significant changes in functionality due to this change request?
Uses mamba instead of conda for installation

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.

Job Test qsubs sawtooth on 2664cba : invalidated by @joshua-cogliati-inl
Multiple strange failures

Job Test qsubs sawtooth on 2664cba : invalidated by @joshua-cogliati-inl
Failed tests/framework/InternalParallelTests/PostProcessor Timeout tests/framework/InternalParallelTests/ROMscikit Timeout tests/framework/pca_grid/gridPCAValue

Job Test qsubs sawtooth on 2664cba : invalidated by @wangcj05
Error detail: Resource temporarily unavailable

Job Test qsubs sawtooth on e69d985 : invalidated by @joshua-cogliati-inl
Diff tests/cluster_tests/test_mpi_forced Failed tests/cluster_tests/InternalParallel/test_parallel_ensemble_picard Timeout tests/framework/InternalParallelTests/PostProcessor

@joshua-cogliati-inl It seems it is the TensorFlow that doubled our installation time.

@mandd @PaulTalbot-INL What do you think about this PR?
The followings are based on my observation:

Include Tensorflow in conda install will significant impact the installation time
Use pip to install Tensorflow may cause the failure of qsub test, but significantly improve the installation process on Windows
mamba in general is much faster than conda

I think we can use mamba as default for our test machines, what do you think?

Just for reference, the current status of the changes is:

mamba install is off by default, --mamba is needed to use mamba
tensorflow is installed with pip on windows, but not on linux or mac

Job Test CentOS 7 on ae38d3f : invalidated by @joshua-cogliati-inl
computer rebooted

Job Test Ubuntu 18 PIP on e69d985 : invalidated by @joshua-cogliati-inl
computer rebooted

Job Mingw Test on ae38d3f : invalidated by @joshua-cogliati-inl
Failed tests/framework/pca_adaptive_sgc/test_adaptive_sgc_poly_pca_analytic  Failed tests/framework/pca_adaptive_sgc/test_adaptive_sgc_poly_correlation

Job Mingw Test on ae38d3f : invalidated by @wangcj05
fetch issue, resource busy

Job Mingw Test on ae38d3f : invalidated by @joshua-cogliati-inl
Timeout tests/framework/pca_adaptive_sgc/test_adaptive_sgc_poly_pca_analytic

Job Mingw Test on ae38d3f : invalidated by @wangcj05

Job Mingw Test on ae38d3f : invalidated by @wangcj05

Job Test Ubuntu 18 PIP on bf98951 : invalidated by @joshua-cogliati-inl
restarted civet

PR checklists are satisfied.


Wrong Icon Showing up

Here is the screenshot:

The split-screen icon appears to be different than what is in the .md file. Please look into this

Fixed via https://github.com/misolori/vscode-fluent-icons/commit/b71cd254d5eb4cb2f5d67388cabf83731cda5b50


Reduce displayed precision for runtimes to seconds

Network latency and other effects mean we do not have true microsecond
precision anyway...

Makes sense to me, merging.


German Translation: Removing "denglisch" word "updaten"

And replaced it with good old and correct word "aktualisieren"

@MarcoWilli In order for us to be able to take this change, you need to agree to the Brackets Contributor License Agreement.
This somehow wonky wording has made me wanna change it, too, but I've not yet gotten around to submitting a PR. This is a welcome change!

I can not submit the license agreement form because it says my email address is not valid! I use mail adress<EMAIL_ADDRESS>Please fixe this -.-

@ryanstewart It looks like the correct-mail-address check doesn't consider the newly added, generic top level domains (like .rocks, in this case).

@prksingh I guess we have to touch the ancient ColdFusion scrolls at the CLA server to make this work?

If that is too hard, maybe we could add a cla exception? https://github.com/adobe/brackets/blob/master/tasks/cla-exceptions.json

@TomMalbran, the CLA must be signed. cla-exceptions.json is not for these cases (you either have to be a bot or still sign a CLA). This is actually urgent enough to fix the mail check.

I have created an issue for this here

The CLA check will take a bit of time to resolve, so moving this PR from this release.
For the time being, we will not be able to accept generic TLD mail addresses.
@MarcoWilli We apologize for this inconvenience.

Tagging @nethip @ryanstewart for prioritizing this.

Hey guys. Do I need to do something or do we have to wait until the CLA is fixed?

Closing in favor of #11371, which includes these changes.


Node 13 request.connection deprecated

On file paperplane/lib/parseUrl.js
at line 19: req.connection.encrypted ? 'https' : 'http'
Node http request docs

Tested the fix on Node 13 and when using http.createServer the encrypted key on req.socket is undefined.
I did not try when the server is started with https.createServer.
Also I could not find documentation on socket.encrypted.

@JohanCodinha, first of all, thanks for making an issue for this, because I was unaware that was deprecated!
From what I can tell, req.socket.encrypted is undefined normally (because it is never set), but is true for TLS connections.  You would need to start the server with https.createServer to see that behavior.  Here's what I found in the docs for tlsSocket.encrypted:
https://devdocs.io/node/tls#tls_tlssocket_encrypted

Well thanks for explaining it, it make sense now.
I'm using paperplane and enjoying it so perfectly normal that I notified you.


Low F1 on the other subsets of SMAP

Hi Shreshth, when I changed the sub dataset of SMAP from P-1(which you set in the main.py) to others(such as A-1, A-2), the performance becomes very unfavorable. Could you help me out with some explanation of this? Do I need to set different hyperparameters for each subset of SMAP to achieve the the F1 on SMAP which is posted in the paper？
on SMAP A-1
{'FN': 0,
'FP': 8556,
'Hit@100%': 1.0,
'Hit@150%': 1.0,
'NDCG@100%': 0.9999999999999998,
'NDCG@150%': 0.9999999999999998,
'ROC/AUC': 0.5,
'TN': 0,
'TP': 84,
'f1': 0.019257030697754137,
'precision': 0.00972222221096965,
'recall': 0.9999998809523951,
'threshold': 0.19118465318625302}
on SMAP A-2
{'FN': 0,
'FP': 141,
'Hit@100%': 1.0,
'Hit@150%': 1.0,
'NDCG@100%': 0.9999999999999999,
'NDCG@150%': 0.9999999999999999,
'ROC/AUC': 0.9909661711942593,
'TN': 7663,
'TP': 110,
'f1': 0.6094140115854588,
'precision': 0.4382469944921516,
'recall': 0.9999999090909173,
'threshold': 0.16136778487480496}

Many thanks for your interest in our project. This code has a static set of hyperparameters for the models for ease of use. We are working to finalize the parameters used for generating the results and the grid-search strategy for hyperparameter tuning for trace-level optimization, which we shall upload as part of a separate branch as soon as it is ready.


Adding support for detecting nested contexts in function literals

Function literals are usually called multiple times during their lifetime (consider middlewares, mapping functions etc). If they are modifying a context variable that was defined out of their scope - maybe in the parent function- it can very quickly bloat the original context variable. This can cause memory leak. I have experienced the same while working with golan with at my day job.
I think this change will help catch many such bugs.
I have also added testcases for this and have improved the identifier scope checking logic.

Hi! Thank you for your contribution :)
I've updated master to always run all test suites + dropped support for go 1.21.
Could you please rebase? At first it looks like your changes aren't compatible with go >=1.22 but I'd like to confirm that.

Hi, i am currently away from my laptop. i'll update this soon.
On Sun, 25 Aug, 2024, 19:32 Gabriel Augendre, @.***>
wrote:

Hi! Thank you for your contribution :)
I've updated master to always run all test suites + dropped support for go
1.21.
Could you please rebase? At first it looks like your changes aren't
compatible with go >=1.22 but I'd like to confirm that.
—
Reply to this email directly, view it on GitHub
https://github.com/Crocmagnon/fatcontext/pull/18#issuecomment-2308861909,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJOI74G7SKFE46334YSEP2LZTHPYXAVCNFSM6AAAAABNB4S7R2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBYHA3DCOJQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

After rebasing, i found tests are failing only for go 1.22 (and not for older version). The reason being go 1.22 changed the way Scope of an a function parameter variable is computed.
For example,
func addOne(x int) int {
    return x + 1
}

In go 1.21, the scope of variable `x` started at `{` and ended at `}`
In go 1.22, the scope of variable now starts at `f` and ends at `}`

I have made some changes to handle this.

my new change also handles this case
// this is fine because the context is created in the loop
	for {
		if ctx := context.Background(); doSomething() != nil {
			ctx = wrapContext(ctx)
		}
	}

Thanks! Released in v0.5.0 :shipit:


Add Yarn install to documentation

Description
Currently, the documentation only shows an installation with NPM. It'd be a good idea to add:
yarn add @brazilian-utils/brazilian-utils

In my case, as I use the Yarn package manager, I had problems using the NPM Install directly. Some problems arose with nonexistent modules and types not found.
I had to delete node_modules and yarn.lock and then reinstall yarn to restore my project.

Nice idea! Can you help us adding this info to our docs?

For sure! I will do a PR! Ok?

Yes, go ahead!

How is the status of this request?
I think this can be considered to be added with #110 update.

Closed by #247


backdrop-filter blur

Hi,
First things first, thanks for your amazing work on PixiJS.
I'm struggling to create a backdrop-filter blur like this one done in CSS : https://codepen.io/gesteves/pen/PwRPZa
The main idea is to have a blurred Graphics above other "layers" (zIndex) of contents (sprites, graphics...)
Any clue achieving this ?
Thanks in advance.

Yes, its possible, i just didnt make that demo yet. Thanks for reminding me :)
I want to make a simple filter that can accept any other filter and make it backdrop. In theory, considering what I already have in pixi-picture - its easy. But i didnt have time to try it yet

Thanks for you fast reply.
Any clue achieving this by my own ?
thanks

Im debugging it at the moment

Done. npm 2.0.1, or dist in this repo.
https://pixijs.io/examples/#/plugin-picture/mask-blur.js

wait, something is wrong in easy version of window

OK, try again. If it doesnt work, ctrl+f5 this file and reopen examples: https://pixijs.io/examples/pixi-plugins/pixi-picture.js

I see one strange artifact in "easy" window, at the bottom of it.

OK, fixed again. Same procedure regarding browser cache.

thank you so much for your work ! This is exactly what I was lookin' for !


plugin/monochromatic

I think it would be really useful to have an ability to generate monochromatic colors (tints, shades, and tones). You mentioned usefulness of it in #63 and it was on my list of future proposals.
I have already working implementation, it looks like this:
color.tints(5); // generates 5 tint instances of original color;
color.shades(5); // generates 5 shade instances of original color;
color.tones(5); // generates 5 tone instances of original color;

During the prototyping, I have got some questions regarding the API:

Should it be one method with type of palette as the parameter like color.monochromatic("tones", 5) or to be a separate methods for each type? I think it would be more readable for this particular case to have different methods.
 It looks like it is a good idea to include them, but the problem arises with the steps parameter, it gets not so understandable. For example, user wants 3 shades and gets: original color, shade, black. Don't look good.

I would like to hear you ideas and suggestions.

Hi @EricRovell! Sorry for the delay: was busy at work. I like the idea!

I just checked the Wikipedia article and feels like this functionality can be a part of the existing mix plugin and use Colord's color mixing magic.
Example:
  // plugins/mix.ts

  ColordClass.prototype.tints = function (count = 5) {
    const tints = []
    for (let ratio = 0; ratio <= 1; ratio += 1 / count) tints.push(this.mix('#fff', ratio));
    return tints;
  };

No problem!
Right now I have implemented it for myself using saturation and lightness values from HSL model. I have read about your mix plugin and it says it has better results. I guess I will dive in and try extend your plugin then!

Hi @omgovich & @EricRovell,
I really like colord and wanted some experience creating NPM packages, so I decided to work on similar package called ColorMaster.
I implemented the above functionality and thought I would share my findings/results here. The code is open source, so you can check it to confirm.
Since tints and shades, are just lightness manipulations (adding white/black), we can use the HSLA color-space for this as it has a lightness channel. Likewise, for tones, we can use the saturation channel of the HSLA color-space:
...
case "monochromatic": {
  // tones uses saturation, tints/shades use lightness
  const valueToAdjust = effect === "tones" ? s : l;  // effect is a function argument - 'tints' | 'shades' | 'tones'

  // form array of n (amount) evenly spaced items from current saturation/lightness to min/max value
  let delta = (effect === "tints" ? 100 - valueToAdjust : valueToAdjust) / amount; // amount is a function argument
  delta = effect === "tints" ? delta : -1 * delta;

  const valArr: number[] = [valueToAdjust];
  for (let i = 0; i < amount; i++) {
    valArr.push(valArr[i] + delta);
  }

  return effect === "tones"
    ? valArr.map((sat) => new HSLColors(h, sat, l, a))
    : valArr.map((light) => new HSLColors(h, s, light, a));
}
...

This returns the original color, along with amount extra items representing the harmony.
You can see that the results match expectations as these tests pass:

I hope this can provide some insight and is helpful to you in some way.
Cheers,
Lior

Hi, @lbragile! Same situation here :) Working on similar package for self-education.
For my package I used the lightness values to generate tints and shades and I was thinking about one edge-case as I see it.
For example, in case of such a color that is too close to pure black/white and user wants too many variations, these colors would be too similar and it is not practical. I have decided to put a minimal step as 1% of lightness. This way it would generate less variations that are more distinguishable.
Do you think it is a good idea?

@EricRovell That is a great consideration.
Let's consider tints, assuming the user has a color with lightness of 95% and they decide to get an array of 10 - extra - tints (amount parameter has a clamped upper value of 10).
In this case the step size will be 0.5% (difference between neighboring tints). As this is a user decision, we would want to provide them with 10 different tints, regardless of how close they are in value, right?
If we suddenly make the minimal step 1%, the user will now only get 5 extra tints, rather than the 10 they asked for. Namely, they will get only 95%, 96%, 97%, 98%, 99%, 100%, ..., 100% where (96%, 97%, 98%, 99%, 100%) are "new" tints.
The same logic applied to shades and tones.
What I am hinting at is that the user made the decision to get 10 extra tints, so that decision should be respected. Otherwise, we are limiting their choices in a way that they cannot control (for such extreme cases).
This is what the approach I outlined in https://github.com/omgovich/colord/issues/65#issuecomment-890753941 aims to achieve.
Do you agree?

@lbragile I think it is quite important to respect the user's decision. I just got some sort of inner conflict, so to speak, about this edge-case with almost pure color and was thinking that generated palette would be to similar or even consist of same colors.
What do you think about limiting the number of tints/shades/tones the user asks? It would we definitely a bad idea to allow user to ask for 1000 tints.

In my opinion, we should return the number of colors the developer asked for — another behavior would be unpredictable and will cause errors in projects using the library.

I doubt someone will request 1000 tints, but if they did, we should return 1000 colors even if some of them are equal.

Alright, I got the idea. Before I begin to work on implementation, I would like to specify one detail. Should I use mix plugin to implement this, or should I use lightness and saturation increments to handle palette creation?

I think mix is the better option here, since it uses LAB color space which will produce a better perceptual difference between tones. Developers might find a lot of different libraries that produce tints and tones using HSL values, so it would be cool for Colord to offer another better alternative.

Alright, I will extend the functionality of mix plugin then.


It doesnt work with fragment

Right now you have mentioned that while inflating layout in onCreateView() use
getActivity().getLayoutInflator(),
But in this case I am unable to find my fragment in the layout using getChildFragmentManager(),
Which returns NullPointerException.
Let me know if you have solution for this also.
Thanks

No support for getChildFragmentManager()

Because it's really evil and a horrible implimentation on Google's behalf.
It has been fixed in the upcoming 2.x version. Which the latest snapshot can be tested.


Fix the https client/server failure

Purpose
$subject.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@18bab2f). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master   #16299   +/-   ##
=========================================
  Coverage          ?   11.88%
  Complexity        ?     2748
=========================================
  Files             ?     1496
  Lines             ?    72405
  Branches          ?     8693
=========================================
  Hits              ?     8606
  Misses            ?    62993
  Partials          ?      806

Impacted Files
Coverage Δ
Complexity Δ

...main/java/org/ballerinalang/net/http/HttpUtil.java
38.77% <0%> (ø)
98 <0> (?)

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 18bab2f...d6bfbe2. Read the comment docs.


Research Open Source models

Talking with jgb he has a presentation about this topic, specially focused in StableDiffusion:
https://jgbarah.github.io/presentations/stable-diffusion/slides-2022-12-01.pdf
The video for the talk: https://m.youtube.com/watch?v=NOGB2BrDG88

StableDiffusion
BLIP-2

An alternative for ChatGPT with the goal of having your own service:
https://github.com/nsarrazin/serge
Let's explore it a bit!

Transferring knowledge between models is easier than expected: https://www.reddit.com/r/singularity/comments/11tmkqi/alpacapromptchaining_considerations_on_the_latest/


Capture Soft Navigations

Why
Instead of loading individual web pages per user navigation, Single Page Applications (SPAs) use  soft-navigations when a user navigates the app.
Here, the content of a page is changed by dynamically altering the browser URL by leveraging the history API and by replacing a pages content through JavaScript.
To get insights into navigation performance and UX metrics within in Single Page Applications Faro should support tracking soft-navigations.
What
The Chrome team is currently experimenting with tracking soft navigations which are emitted as a dedicated PerformanceEntry form a PerformanceObserver hooking into the browsers performance timeline.
Once this is activated in chrome we can start using this api to get the data. Given the big market share of Chrome we can get a good amount of data even if other browser need some more time to implement the api.
In the meantime we need to find out what else we can do.

For example Google's Web-Vitals library provides an experimental mode to  report Web-Vials for soft navigations.
Faro already uses this library to get the measurements for Web-Vitals.
We can built a custom instrumentation which hooks into the history API to track navigations. This is not straight forward because it's complicated to find out what is a navigation and what is simply a URL update,

Increment 0: Highlight views in the UX set via setView API


upgrade rancher from 2.3.6 to 2.4.2

when i upgrade rancher server from 2.3.6 to 2.4.2
it failure！
the rancher log was
2020/04/04 20:01:45 [INFO] Refreshing driverMetadata in 1440 minutes
2020/04/04 20:01:45 [INFO] update kontainerdriver azurekubernetesservice
2020/04/04 20:01:45 [INFO] update kontainerdriver googlekubernetesengine
2020/04/04 20:01:45 [INFO] update kontainerdriver tencentkubernetesengine
2020/04/04 20:01:45 [INFO] update kontainerdriver aliyunkubernetescontainerservice
2020/04/04 20:01:45 [INFO] update kontainerdriver amazonelasticcontainerservice
2020/04/04 20:01:45 [INFO] update kontainerdriver baiducloudcontainerengine
2020/04/04 20:01:45 [INFO] update kontainerdriver rancherkubernetesengine
2020/04/04 20:01:45 [INFO] update kontainerdriver huaweicontainercloudengine
2020/04/04 20:01:45 [INFO] driverMetadata: refreshing data from upstream https://releases.rancher.com/kontainer-driver-metadata/release-v2.4/data.json
2020/04/04 20:01:50 [INFO] kontainerdriver azurekubernetesservice listening on address <IP_ADDRESS>:42649
2020/04/04 20:01:50 [INFO] kontainerdriver googlekubernetesengine listening on address <IP_ADDRESS>:44155
2020/04/04 20:01:50 [INFO] kontainerdriver amazonelasticcontainerservice listening on address <IP_ADDRESS>:44554
2020/04/04 20:01:50 [INFO] kontainerdriver googlekubernetesengine stopped
2020/04/04 20:01:50 [INFO] dynamic schema for kontainerdriver googlekubernetesengine updating
2020/04/04 20:01:50 [INFO] kontainerdriver azurekubernetesservice stopped
2020/04/04 20:01:50 [INFO] dynamic schema for kontainerdriver azurekubernetesservice updating
2020/04/04 20:01:50 [INFO] kontainerdriver amazonelasticcontainerservice stopped
2020/04/04 20:01:50 [INFO] dynamic schema for kontainerdriver amazonelasticcontainerservice updating
time="2020-04-04 20:01:58" level=info msg="Collecting anonymous data from https://localhost:443/v3"
2020/04/04 20:02:29 [ERROR] Unknown error: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: waiting for cluster [c-8stcj] agent to connect
time="2020-04-04 20:02:29" level=error msg="Failed to get Namespaces err=Bad response statusCode [500]. Status [500 Internal Server Error]. Body: [message=Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: waiting for cluster [c-8stcj] agent to connect, baseType=error, code=ServerError] from [https://localhost:443/v3/cluster/c-8stcj/namespaces]"
2020/04/04 20:02:30 [ERROR] ClusterController c-8stcj [cluster-deploy] failed with : Get https://<IP_ADDRESS>:6443/apis/apps/v1/namespaces/cattle-system/daemonsets/cattle-node-agent?timeout=30s: context deadline exceeded
2020/04/04 20:02:51 [INFO] Stopping cluster agent for c-8stcj
2020/04/04 20:02:57 [ERROR] Unknown error: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: waiting for cluster [c-8stcj] agent to connect
time="2020-04-04 20:02:57" level=error msg="Failed to get Namespaces for project c-8stcj:p-47cdm err=Bad response statusCode [500]. Status [500 Internal Server Error]. Body: [baseType=error, code=ServerError, message=Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: waiting for cluster [c-8stcj] agent to connect] from [https://localhost:443/v3/cluster/c-8stcj/namespaces?limit=-1&projectId=c-8stcj%3Ap-47cdm&state_ne=removed]"
2020/04/04 20:03:27 [INFO] Error on LIST namespaces: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: context deadline exceeded (Client.Timeout exceeded while awaiting headers). Attempt: 1. Retrying
2020/04/04 20:03:57 [ERROR] Unknown error: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: waiting for cluster [c-8stcj] agent to connect
time="2020-04-04 20:03:57" level=error msg="Failed to get Namespaces for project c-8stcj:p-68ltv err=Bad response statusCode [500]. Status [500 Internal Server Error]. Body: [baseType=error, code=ServerError, message=Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: waiting for cluster [c-8stcj] agent to connect] from [https://localhost:443/v3/cluster/c-8stcj/namespaces?limit=-1&projectId=c-8stcj%3Ap-68ltv&state_ne=removed]"
2020/04/04 20:04:27 [INFO] Error on LIST namespaces: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: context deadline exceeded (Client.Timeout exceeded while awaiting headers). Attempt: 1. Retrying
2020/04/04 20:04:52 [INFO] Stopping cluster agent for c-8stcj
time="2020-04-04 20:04:57" level=error msg="Failed to get Namespaces for project c-8stcj:p-74rh7 err=Get https://localhost:443/v3/cluster/c-8stcj/namespaces?limit=-1&projectId=c-8stcj%3Ap-74rh7&state_ne=removed: net/http: request canceled (Client.Timeout exceeded while awaiting headers)"
2020/04/04 20:04:57 [INFO] Error on LIST namespaces: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers). Attempt: 2. Retrying
2020/04/04 20:05:27 [ERROR] Unknown error: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)
2020/04/04 20:05:27 [INFO] Error on LIST namespaces: Get https://<IP_ADDRESS>:6443/api/v1/namespaces?timeout=30s: context deadline exceeded (Client.Timeout exceeded while awaiting headers). Attempt: 1. Retrying
what the problem?

Hi had the same issues with imported k8s cluster, mainly with oke ones Oracle kubernetes engine. Had to revert back to 2.3.5.


Fix skill -> work_process association

closes #63
Updates

Remove skill belongs_to work_process association.
Add belongs_to association on occupation_standard_skill to occupation_standard_work_process
Update pdf to nest work_process -> skills
Update occupation_standard admin dashboard to nest work_process -> skills
Update pdf filename to use a timestamp for now so you can tell when it's been updated

QA

[x] On Data Import admin page, import test file.
[x] View newly created occupation standard. Should see table with work processes that list skills.
[x] Click Generate PDF button.
[x] PDF file should display skills nested under work processes

Looks good to me!


Use awesome font icon instead of bootstrap glyphicon

I do prefer font awesome instead of bootstrap glyphicon.
I used :

http://fontawesome.io/icon/check/
http://fontawesome.io/icon/times/

Cheers 🍺

This needs to become configurable (see #290)


minTime and maxTime not working on elementor form field

I'm trying to limit the time field on an elementor pro form's flatpickr time field, but I can't figure out how to solve this. Tried these methods:

flatpickr('#form-field-idopont', {
disableMobile: 'true',
enableTime: true,
noCalendar: true,
dateFormat: 'H:i',
time_24hr: true,
onOpen: function(){
$('.flatpickr-hour').attr('min', '07');
$('.flatpickr-hour').attr('max', '22');
$('.flatpickr-hour').attr('step', '01');
}
});

jQuery(document).ready(function( $ ){
$('.flatpickr-hour').attr({
'min' : '7',
'max' : '22'
});
});

flatpickr('#form-field-idopont', {
disableMobile: 'true',
enableTime: true,
noCalendar: true,
dateFormat: 'H:i',
time_24hr: true,
minTime: "16:00",
maxTime: "22:30",
}
None of them working. What am I doing wrong? The page where the form located: https://stiflerbars.hu/foglalas/.

up

Hi Sotusatti
I noticed on your website https://stiflerbars.hu/foglalas/ that you have a date selector that has disabled all dates before todays date. I assume as per your previous comment that you used Elementor Pro for this? Could I ask how you managed to get it to work? I have a date selector on my site and I need to disable all previous dates including todays. end user will only be able to book a table from tomorrows date. Any help much appreciated. :)

Thanks for that! Can you tell me where you put your code pls? I’ve been
trying to add it to a page using Elementor custom code. Also do I need to
add the

That's exactly the place I put the code, Elementor -> Custom code. You should add the script tags as well. Here is the full copy-paste code without unneceserry parts than:


Ensure we dual-write latency stats.

This will give us visibility on a per-worker basis
for which hyperbahn instance has the highest latency
r: @jcorbin @rf

lgtm, and yay more tests.


First day of week incorrect for pt_BR locale

The Flutter Material Date Pickers depend on the DateFormat.dateSymbols.FIRSTDAYOFWEEK to determine the start day for the week shown in the date grid.  However, for the pt_BR locale this is returning Sunday. According to https://github.com/flutter/flutter/issues/57050 it should be Monday.
Specifically:
DateFormat.yMMMMEEEEd('pt-BZ').dateSymbols.FIRSTDAYOFWEEK

returns 6 which maps to Sunday, when it should be 0 for Monday.

The CLDR data, which is the source of truth for this repository, specifies
<firstDay day="sun"  territories="
	AG AS
	BD BR BS BT BW BZ
	CA CO
	DM DO
	ET
	GT GU
	HK HN
	ID IL IN
	JM JP
	KE KH KR
	LA
	MH MM MO MT MX MZ
	NI NP
	PA PE PH PK PR PT PY
	SA SG SV
	TH TT TW
	UM US
	VE VI
	WS
	YE
	ZA ZW"
/>


[BUG] Module deployment failure: jupyter-hub

Describe the bug
addf-demo-ide-jupyter-hub deployment failure, due to no longer supported runtime.
To Reproduce
deploy jupyter-hub module
Expected behavior
jupyter-hub deployed without issues
Screenshots
na
Additional context
...
Failed resources:
addf-demo-ide-jupyter-hub | 10:17:07 AM | CREATE_FAILED        | AWS::Lambda::Function                 | AWSCDKCfnUtilsProviderCustomResourceProvider/Handler
handler returned message: "The runtime parameter of nodejs12.x is no longer supported for creating or updating AWS Lambda functions. We recommend you use a supported runtime while creating or updating functions. (Service: Lambda, Status Code: 400,
❌  addf-demo-ide-jupyter-hub failed: Error: The stack named addf-demo-ide-jupyter-hub failed creation, it may need to be manually deleted from the AWS console: ROLLBACK_COMPLETE
...

@serge-dolgavin-dxc
You can try this before merge, I need to test from the groundup so this may take awhile before its merged.
File: ide-modules.yaml
name: jupyter-hub
path: git::https://github.com/awslabs/autonomous-driving-data-framework.git//modules/demo-only/jupyter-hub?ref=chore/583&depth=1

@malachi-constant ,
unfortunately,
name: jupyter-hub
path: git::https://github.com/awslabs/autonomous-driving-data-framework.git//modules/demo-only/jupyter-hub?ref=chore/583&depth=1

doesn't work for me:
$ seedfarmer apply ./manifests/demo/deployment.yaml --dry-run
...
[2024-09-05 07:10:17,386 | INFO | _deployment_commands.py:636 | MainThread ]  Verifying all modules in ide for deploy
Traceback (most recent call last):
...
  cmdline: git pull -v -- origin chore/583
  stderr: 'fatal: couldn't find remote ref chore/583'

During handling of the above exception, another exception occurred:
...
.../autonomous-driving-data-framework/.venv/lib/python3.8/site-packages/seedfarmer/mgmt/git_support.py", line 79, in clone_module_repo
    raise InvalidConfigurationError(f"\n Cannot Clone Repo: {ge} {messages.git_error_support()}")
seedfarmer.errors.seedfarmer_errors.InvalidConfigurationError:
 Cannot Clone Repo: Cmd('git') failed due to: exit code(1)
  cmdline: git pull -v -- origin chore/583
  stderr: 'fatal: couldn't find remote ref chore/583'
    1. Make sure your path to the repo is correct and valid (check your module manifests!)
    2. The credentials used to call SeedFarmer have access to the repo
    3. The credentials used to call SeedFarmer have not expired

@malachi-constant ,
with
name: jupyter-hub
path: modules/demo-only/jupyter-hub/

I got the following error:
...

addf-demo-ide-jupyter-hub |  4/11 | 7:17:32 AM | CREATE_IN_PROGRESS   | Custom::AWSCDK-EKS-KubernetesResource | addf-demo-ide-jupyter-hub-eks-cluster/manifest-jupyter-hub-namespace/Resource/Default (addfdemoidejupyterhubeksclustermanifestjupyterhubnamespaceXXXXXXXXXXXX) Resource creation Initiated

1321

...

@malachi-constant ,
please find the attached the codebuild log  for jupyter-hub module: jupyter-hub_CodeBuild.log

@serge-dolgavin-dxc Can you try this module from main that branch was deleted after merge

@malachi-constant ,
Sorry that me messages are not clear and for confusion.
I have recognized that the branch was deleted and I am already using main for the last 5 days.
The yesterday's codebuild log for  jupyter-hub module is based of the recent main branch.

Gotcha missed that, taking a look...

@serge-dolgavin-dxc Are you able to provide the trust policy for arn:aws:iam::XXXXXXXXXXXXX:role/addf-demo-core-eks-clusterCreationRoleXXXXXXXXXXX\ with account values sanitized as well so I compare to what I have tested? I am not able to replicate.

@malachi-constant , please find the attached policy details along with the latest codebuild log.
jupyter-hub.zip

Ok so the trust is not being added for some reason, can you also tell me which version of the eks module is deployed?

I am using the latest main branch (default demo / example-dev manifests).
name: eks
path: git::https://github.com/awslabs/idf-modules.git//modules/compute/eks?ref=release/1.11.0
dataFiles:
  - filePath: git::https://github.com/awslabs/idf-modules.git//data/eks_dockerimage-replication/versions/1.29.yaml?ref=release/1.11.0
  - filePath: git::https://github.com/awslabs/idf-modules.git//data/eks_dockerimage-replication/versions/default.yaml?ref=release/1.11.0

Ok thanks, was able to replicate, working on it...

@serge-dolgavin-dxc
See manifest in PR
This error is resolved by updating ide-modules.yaml
name: jupyter-hub
path: modules/demo-only/jupyter-hub/
parameters:
 - name: eks-cluster-admin-role-arn
   valueFrom:
     moduleMetadata:
       group: core
       name: eks
       key: EksClusterMasterRoleArn

@serge-dolgavin-dxc
I think your credentials for kubectl are pointing to the wrong cluster (do you have multiple clusters defined in .kube?)...this command:
kubectl get ing jupyterhub -n jupyter-hub -o jsonpath="{.status.loadBalancer.ingress[0].hostname}"
Should be executed against the proper cluster...
REF: https://kubernetes.io/docs/tasks/access-application-cluster/configure-access-multiple-clusters/

@dgraeber , thanks a lot for your hint!
addf-demo-core-eks-cluster configuration was missing.
The first issue was solved, but the second still remain. Is it an issue with access rights?


Mention dynamic-import-node

I needed that one for jest, and I spent some time PRing the webpack one before finding it existed in a separate project...

@SimenB thanks!

Woah, quick! Do you have some way of getting notifications on new PRs? 😀

@SimenB haha, just serendipitously clicked on my notifications at the right time :)


Error on pagination query

Describe the bug
"options.removeUndefinedValues=true" error when using pagination on querying by GSI when PK does not use a SORT key.
I specify a table that only uses a hash key but no sort key for the primary key. The GSI uses hash and sort key.
When I query on the table using pagination the followup request using the next param fails.
Error Output (from adjusted test suite):
  ● Find with next offset

    Pass options.removeUndefinedValues=true to remove undefined values from map/array/set.

      1054 |                 }
      1055 |             } else {
    > 1056 |                 item = client.marshall(item, options)
           |                               ^
      1057 |             }
      1058 |         } else {
      1059 |             if (Array.isArray(item)) {

      at convertToAttr (node_modules/@aws-sdk/util-dynamodb/dist-cjs/convertToAttr.js:7:15)
      at node_modules/@aws-sdk/util-dynamodb/dist-cjs/convertToAttr.js:118:54
      at convertToMapAttrFromEnumerableProps (node_modules/@aws-sdk/util-dynamodb/dist-cjs/convertToAttr.js:122:7)
      at convertToAttr (node_modules/@aws-sdk/util-dynamodb/dist-cjs/convertToAttr.js:23:16)
      at Dynamo.marshall (node_modules/@aws-sdk/util-dynamodb/dist-cjs/marshall.js:6:62)
      at Table.marshall (src/Table.js:1056:31)
      at Expression.command (src/Expression.js:616:57)
      at Model.run (src/Model.js:289:30)
      at Model.queryItems (src/Model.js:1005:27)
      at Model.find (src/Model.js:630:27)
      at Object.<anonymous> (test/pagination.ts:54:28)

To Reproduce
I adjusted pagination.ts test case as follows (commented out and replaced one line each file):
Adjusted schema (to use a PK with only a hash key):
export default {
    format: 'onetable:1.1.0',
    version: '0.0.1',
    indexes: {
        //primary: {hash: 'pk', sort: 'sk'},
        primary: {hash: 'sk'},
        gs1: {hash: 'gs1pk', sort: 'gs1sk'},
    },
    models: {
        User: {
            pk: {type: String, value: '${_type}#'},
            sk: {type: String, value: '${_type}#${name}'},
            id: {type: String, generate: 'ulid'},

            name: {type: String, required: true},
            email: {type: String, required: true},

            gs1pk: {type: String, value: '${_type}#'},
            gs1sk: {type: String, value: '${_type}#${email}'},
        },
    },
}

Adjusted test case (to use the index on the most general pagination test case):
test('Find with next offset', async () => {
    let pages = 0,
        total = 0,
        next
    let items: any
    do {
        //items = await User.find({}, {limit: PerPage, next})
        items = await User.find({}, {limit: PerPage, next, index: 'gs1'})
        if (items.length) {
            expect(items[0].name).toBe(`user-${zpad(total, 6)}`)
            expect(items[0].email).toBe(`email-${zpad(total<EMAIL_ADDRESS>            total += items.length
            pages++
        }
        next = items.next
    } while (next)

    expect(total).toBe(MaxUsers)
    expect(pages).toBe(MaxUsers / PerPage)
})

(I am a newbie in regard to DynamoDb, so maybe I am doing something wrong. Regarding the general setup, I intend to use the primary key only for direct lookups, not for scans/queries, not sure yet if that will be useful in the end...)

Thank you for submitting. Fix committed.

Getting this issue still in 2.7.1, I believe the problem still occurs when querying an index that has no sort key

Can you please submit a debug.ts test case that demonstrates this. The test case above works.


pull request for e2e test at 12/18/2016 23:15:19

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Split coveralls into coverage-collection & coverage-upload?

Currently coverage is tightly coupled to coveralls.io, which is fine for opensource projects as it is free, but for other projects maybe not an option.
Collecting just the kcov output might me desirable nonetheless and invoking it from travis-cargo and possibly handling the upload somewhere else would be nice.
This would also make it possible to execute it locally for a quick overview.

Good idea. I've implemented travis-cargo coverage which will collect without the upload.
I've not quite split it, since the common case is collect-and-upload and having that entirely performed by a single travis-cargo coveralls is nice.

Thanks! Exactly what I want. :)


[HOPS-1123], [HOPS-1124], [HOPS-1125], [HOPS-1126], [HOPS-1127], [HOPS-1128], [HOPS-1129], [HOPS-1130], [HOPS-1131], [HOPS-1132], [HOPS-1134], [HOPS-1135], [HOPS-1136], [HOPS-1137], [HOPS-1138], [HOPS-1139], [HOPS-1140], [HOPS-1141], [HOPS-1142], [HOPS-1143], [HOPS-1144], [HOPS-1146], [HOPS-1147], [HOPS-1148], [HOPS-1158] and [HOPS-1159]

Make sure there is no duplicate PR for this issue

Please check if the PR fulfills these requirements

[ ] Tests for the changes have been added and passed (for bug fixes / features)
[x] HOPS JIRA issue has been opened for this PR
[x] All commits have been squashed down to a single commit
[x] The commit message has the following format: [HOPS-XXX] message

Post a link to the associated JIRA issue
https://hopshadoop.atlassian.net/browse/HOPS-1123
https://hopshadoop.atlassian.net/browse/HOPS-1124
https://hopshadoop.atlassian.net/browse/HOPS-1125
https://hopshadoop.atlassian.net/browse/HOPS-1126
https://hopshadoop.atlassian.net/browse/HOPS-1127
https://hopshadoop.atlassian.net/browse/HOPS-1128
https://hopshadoop.atlassian.net/browse/HOPS-1129
https://hopshadoop.atlassian.net/browse/HOPS-1130
https://hopshadoop.atlassian.net/browse/HOPS-1131
https://hopshadoop.atlassian.net/browse/HOPS-1132
https://hopshadoop.atlassian.net/browse/HOPS-1134
https://hopshadoop.atlassian.net/browse/HOPS-1135
https://hopshadoop.atlassian.net/browse/HOPS-1136
https://hopshadoop.atlassian.net/browse/HOPS-1137
https://hopshadoop.atlassian.net/browse/HOPS-1138
https://hopshadoop.atlassian.net/browse/HOPS-1139
https://hopshadoop.atlassian.net/browse/HOPS-1140
https://hopshadoop.atlassian.net/browse/HOPS-1141
https://hopshadoop.atlassian.net/browse/HOPS-1142
https://hopshadoop.atlassian.net/browse/HOPS-1143
https://hopshadoop.atlassian.net/browse/HOPS-1144
https://hopshadoop.atlassian.net/browse/HOPS-1146
https://hopshadoop.atlassian.net/browse/HOPS-1147
https://hopshadoop.atlassian.net/browse/HOPS-1148
https://hopshadoop.atlassian.net/browse/HOPS-1158
https://hopshadoop.atlassian.net/browse/HOPS-1159

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the new behavior (if this is a feature change)?

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Other information:
depends on hopshadoop/hops-metadata-dal#154 and hopshadoop/hops-metadata-dal-impl-ndb#190

tests: https://bbc1.sics.se/jenkins/view/pull-manual/job/hops-pull-manual/63/


Support random_access_iterator requirements in MapView::Iterator

Summary:
As the title suggests, this diff makes it so that MapView::Iterator meets the requirements
of a random_access_iterator.  This lets std algorithms on instances of the iterator like
distance know that they can use constant time implementations rather than linear time.
The new functions include:
In the IndexBasedIterator:
pre-decrement
post-decrement
+=
-=
In the MapView::Iterator
iterator + int
int + iterator
iterator - int
iterator - iterator
[n] to access the value at iterator + n
In addition I added calls to the validateBounds() function in IndexBasedIterator when
dereferencing the iterator to ensure we don't accidentally dereference values off the end
of the current collection.
I also added a static check to the Iterator to ensure it's properly declaring its
iterator_category (Iterators don't inherit this from the parent, which it seems like the code
was assuming before).
Differential Revision: D34911295

Done in https://github.com/facebookincubator/velox/pull/1216


use the libinit to compute the integral

the libint seems a great library to compute the multipole integrals

The libint2 library to compute the integrals is not portable. Either build a conda recipe or abandon the idea.

See branch libint

There is now a conda recipe for libint

solved by #126


Blog fixes

Fixes:

There appears to be some artefact on the triangle (please find attached example) (iPad)
There appears to be tiles over and under lapping each-other

'half pixel artefact'! :palm_tree:

Retina screens, woo :iphone:


--create_source_map throws exception on windows when using input source maps and chunks

Closure throws an exception when I try to build source maps for chunks:
java.io.FileNotFoundException: .\dist\closure\common.js.map (The system cannot find the path specified)
It seems like there's a race condition issue if the source map of a depending module is built before the dependency. It seems to work properly if I run the build without --create_source_map, and then run it again with --create_source_map.
Reproducer is available here: https://github.com/DerGernTod/ts-esbuild-closure-reproducer/blob/main/index.js
The used flags:
--chunk common:2
--chunk_wrapper common:(function(){%s})()
--source_map_input dist/esbuild/chunk-7THFJKJQ.js|dist/esbuild/chunk-7THFJKJQ.js.map
--js dist/esbuild/chunk-7THFJKJQ.js
--source_map_input dist/esbuild/filecommon.js|dist/esbuild/filecommon.js.map
--js dist/esbuild/filecommon.js
--chunk filea:1:common
--chunk_wrapper filea:(function(){%s})()
--source_map_input dist/esbuild/filea.js|dist/esbuild/filea.js.map
--js dist/esbuild/filea.js
--chunk fileb:1:common
--chunk_wrapper fileb:(function(){%s})()
--source_map_input dist/esbuild/fileb.js|dist/esbuild/fileb.js.map
--js dist/esbuild/fileb.js
--language_in ECMASCRIPT_2017
--language_out ECMASCRIPT_2017
--chunk_output_path_prefix ./dist/closure/
--rename_prefix_namespace myNamespace
--compilation_level ADVANCED_OPTIMIZATIONS
--create_source_map %outname%.map
I also tried --source_map_include_content, but that doesn't seem to work either (even if you build it once without source maps to prevent the error mentioned above, and then build it with source maps). The source is not embedded at all.

Assigning to myself to investigate further

I'm having trouble reproducing the race condition. Here's what I've tried:
$ git clone https://github.com/DerGernTod/ts-esbuild-closure-reproducer.git
$ cd ts-esbuild-closure-reproducer/
$ node --version
v16.4.1
$ npm install
$ npm start

><EMAIL_ADDRESS>start
> tsc && node index.js

done 0
$ ls dist/closure
common.js  common.js.map  filea.js  filea.js.map  fileb.js  fileb.js.map

@DerGernTod could you confirm that following the above steps triggers an exception for you?

I have tried it out and can confirm that I can reproduce the issue.
One thing to note:
I am using node "v14.16.0" and it throws the FileNotFoundException. Just to be sure I tried to update to node v16.4.0 and the issue is still there. Unfortunately I can not update to v16.4.1, because there is an issue that prevents the update: https://github.com/nodejs/node/issues/39224
Also doesn't seem to happen randomly, I tried it 10 times for each version and the Exception happened every time

Thanks for adding more information @kamtschatka.
Looks like you're on Windows since you're affected by https://github.com/nodejs/node/issues/39224, so I wonder if the bug is specific to Window. I was trying to reproduce on Linux.
@concavelenz could I pass this to you to investigate now that you've taken over triage duty?

I just tested with openjdk 11 and 16 on wsl and it worked fine, so it seems really to be a problem that only appears on windows.

i just ran a few more test with windows/linux and there's definitely differences in the produced source maps. on windows, it seems that --source_map_include_content is completely ignored, while on linux, --source_map_location_mapping doesn't do anything. can be reproduced with the reproducer repo above. the source maps created on windows and linux respectively look completely different (while the linux one seems to work properly though, at least with included content as far as i can tell)

Some times back, I have implemented exactly what you are doing, that is, using input source maps and chunks on Windows. It was done as a part of https://github.com/theseanl/tscc.
A difference of my case is that, I am feeding JS files to the compiler via stdin, via --json_streams=BOTH flag. In the JSON format that the compiler accepts, there is a path field you have to specify, so that closure compiler can know what content to refer to when it is given with a file path.
While I am implementing it, I have also observed that Closure Compiler cannot resolve paths in input sourcemaps when the above path fields are Windows-style paths. Then I changed any Windows-style paths in path value to a unix-style path, which is a lossless conversion I guess. And it seemed that it enables closure compiler to correctly resolve files in sourcemaps.
From compiler flags in the original post, I guess there is no more place to fiddle with paths, maybe json_streams mode processes some paths differently and may help getting correct sourcemaps.

Since the Google team that forms the core support for closure-compiler do not use Windows at all, reproducing this problem in order to debug it is not something we can reasonably spend time on.
Would someone from the Open Source community be willing to debug this?

@brad4d do you have a hint of where one could start fixing this? in the meantime i worked with WSL, which was fine so far, but now we're switching to pnpm and for some reasons it causes issues in combination with WSL on my machine (works elsewhere). so i'm back at having to build source maps on windows with closure compiler :D

re: the directory not existing
I would not expect closure-compiler to create a directory in which to put an output file.
Throwing an exception is a very poor way to handle the directory not being there,
but it's still a case of passing bad command line options if the compiler is being told to write a file into a non-existent directory.
It seems as if you can resolve that problem by just creating the directory before executing the compiler.
Re: missing "sourcesContent"
Have you tried using the --apply_input_source_maps to closure-compiler?
I think you probably need that.
Can you confirm that the source maps generated by tsc contain sourcesContent?
When using input source maps, closure-compiler copies the sourcesContent from them rather than using source code it is compiling. This is generally what you want, since you'd like to point back to the original TS, rather than the intermediate JS code it produced for compilation.
You might also try asking tsc to generate inline sourcemaps inside the JS file it outputs instead of separate files.
Then ask closure-compiler to read those with --parse_inline_source_maps.
That's what we typically do.

alright, so i tried a few more things you suggested today.
i inlined all source maps (so no lookup for .map files needed anymore), got rid of the "--source_map_input" for each file and added "--apply_input_source_maps" and "--parse_inline_source_maps" (i found no documentation about that... missing? https://github.com/google/closure-compiler-js/blob/master/README.md and https://github.com/google/closure-compiler/wiki/Flags-and-Options). unfortunately with no avail. the "sourcesContent" is still missing on windows.
i can confirm that it's available after both tsc and esbuild. example esbuild source map (base64 decoded from the inline map):
{
  "version": 3,
  "sources": ["../../source/filecommon.ts"],
  "sourcesContent": ["function fileCommonFunction() {\r\n    exportedFunction();\r\n}\r\n\r\nexport function exportedFunction() {\r\n    console.log(\"i'm exported\");\r\n}\r\n\r\nfileCommonFunction();"],
  "mappings": ";AAAA,8BAA2B;AACvB;;AAGE,4BAA0B;AAC5B,UAAQ,IAAI;;AAGhB;",
  "names": []
}

so the question remains: why does closure compiler not include the "sourcesContent" on windows? now it doesn't even make sense that it's a path problem since the sources are inlined

@brad4d i spent quite some time yesterday and today checking out the closure compiler repo and building it myself. didn't manage to get it running within an ide but system out debugging helped, i found the issue. not sure why sourcesContent are related to that, but i found why the exception happens on windows and not on linux. within AbstractCommandLineRunner.java, in maybeCreateDirsForPath, the given directory from --chunk_output_path_prefix ./dist/closure/ is not created on windows, because File.separatorChar is actually \. so if the user passes / on windows, the exception happens. if the path is manually generated beforehands, the sourcesContent property is missing. i didn't dig into why this is related, but if i resolve the maybeCreateDirsForPath issue using \ instead of / when passing --chunk_output_path_prefix, the sourcesContent is there.
so in the end i'd say it's a user error. but due to the fact that you can interchange / and \ on powershell on windows, and for all other flags (such as --js), i think it's very hard to diagnose for a user to find the problem why their source maps don't work on windows.

I'm submitting a change to unhide the two command line flags I mentioned.
It turns out they defaulted to "on" anyway, but there's really no reason to hide them.

Well done finding the problem and submitting the PR.
I've done an initial review over there.


Questions about traffic laws in the U.S

https://github.com/instructlab/taxonomy/issues/811

Closing per combined PR here: https://github.com/instructlab/taxonomy/pull/1151


Make input/output parameters specifiable on a per-node basis

Same deal as most recent PR.
One thing that's making me nervous as I work on this - tool driver authors need to be careful to tack on the step/index for these keypaths when calling get(), since if they forget there might be a silent error if a user tries to supply a step/index-specific setting and it doesn't go through.
It might be worth going back and making step/index required when doing get/find_files for "optional" per-node parameters, any thoughts on that?

@gadfort I updated this PR to add a check to get() that complains if you don't supply step/index for "optional"-pernode params.
I'd say it's medium invasive. I had to fix some things in the core, but they were probably important to tweak anyways. A big problem is that we have a lot of places where we loop over all keypaths and do something, and this change makes these loops even more complicated since we're introducing more irregularity into how different types of parameters are implemented.
I also realized that these sorts of loops require some way to explicitly get the "global" value. I decided to implement this by allowing the string "default" to be used for the step/index keywords:
>>> chip.set('asic', 'logiclib', ['mylib'])
>>> chip.set('asic', 'logiclib', ['synlib'], step='syn', index=0)
>>> chip.get('asic', 'logiclib')
# raises exception
>>> chip.get('asic', logiclib, step='default', index='default')
['mylib']
>>> chip.get('asic', 'logiclib', step='syn', index=0)
['synlib']

I'm not 100% happy about this, but I think it works out okay - it requires the author of the code to be explicit about bypassing the step/index override values, it should be invisible to users, and it's safe since "default" isn't a legal step value (would conflict with the schema placeholders).

I decided to move the check logic to #1296 to make it easier to see the impact of those changes. Will mark this as draft and rebase on top of it once that one is merged, to make sure that I had good coverage adding step/index.

Looking at this, I am starting to see the problem with the per node value, mostly around the user having to remember the priority and value of the pernode value vs global. Having the get below raise an exception feels like a step backwards, and adding the 'default' step does not help. Without a step/index, wouldn't we expect that to return the global value? Sorry, I am getting lost in the new logic. This may be my own fault (or not..), user
>>>chip.get('asic', 'logiclib')

@aolofsson in speaking with @nmoroze there doesn't appear to be any cases where the getter needs to get the global value (they would still get it if the value is not set for that step/index). By generating an error if the parameter has a step/index that is optional it prevents users from accidentally getting the global when they were intending to get the step/index value.

Yeah, the thinking is that this mostly affects tool driver writers - a designer-user probably doesn't have to call get() much unless they're trying to do some sort of design introspection. Adding the exception revealed a lot of places where I had forgotten to add step/index (which would have resulted in buggy drivers), so it seems value in that regard.
I think the best alternative to the verbosity would be to do the "magic" getter that determines step/index from [arg, step] and [arg, index], but that has its own pitfalls as we've discussed.


Getting strange types after generating

Bug description
After running prisma generate and restarting the TypeScript server, I am not getting types which match my models
How to reproduce
I'm honestly not sure how to reproduce.
I have tried:

Removing node_modules and pnpm-lock.yaml, reinstalling, and running pnpm dlx prisma generate
Restarting the typescript server

Expected behavior
No response
Prisma information

model Character {
    id                  String   @id @unique @default(cuid())
    name                String
    race                String?
    class               String?
    campaign            String?
    image_url           String?
    character_sheet_url String?
    user                User     @relation(fields: [userId], references: [id])
    userId              String
    logs                Log[]
    created_at          DateTime @default(now())

    @@index([userId])
}

const character = await prisma.character.findFirst({
	// ^ any
	// Should be Character | undefined
	include: {
		user: true
	},
	where: { id: characterId }
});

export async function getCharacters(userId: string) {
	//                ^ Promise<$Public.PrismaPromise<T>>
	//                Should be Character[]
	return await prisma.character.findMany({
		where: { userId: userId }
	});
}

Environment & setup

OS: macOS
Database: PlanetScale
Node.js version: 21.5.0

Prisma Version
5.9.0

Reverting back to Prisma Version 4.15.0 seems to have fixed it.

I also ran into this issue where the prisma types stopping working after upgrading.
I was able to confirm that everything works in 5.8.0, but breaks in 5.9.0
I also tried deleting node_modules, npm installing, npx prisma generate, restarting TS Server in VSCode, etc, and nothing fixed it.

Spent like the whole day trying to figure this out as this is the first time I'm trying to use prisma. I' m also getting all my prisma data typed as $Public.PrismaPromise~
Using these versions:
"@next-auth/prisma-adapter": "^1.0.7",
"@prisma/client": "^5.9.0",
"prisma": "^5.8.1",

Hey @sillvva @jessethomson @mnemosx, thanks for the report.
Yes, we have been working on a fix as soon as we have seen the regression. We have a good amount of reports that say our fix is working, but I'd like to ask you if you could take the time to try out pour integration version for the fix?
If you want, please try it out by updating both prisma and @prisma/client to this integration version: 5.10.0-integration-fix-client-exports-map.14 (once, installed don't forget to run prisma generate).
I'd appreaciate if you can let me know how it went, thanks!
We are aiming for a patch release with these fixes.

rollback to 4.8.1 worked correctly

@Lebraz98 if you have the time, could you please give this a try? Thanks

Hi @millsp, I was experiencing the same issue with 3.9.0 and with the 5.10.0-integration-fix-client-exports-map.14 works totally fine.

@millsp  im sorry tomorrow i will check it

5.9.0 breaks，and 5.10.0-integration-fix-client-exports-map.14 works fine.

Comments look promising. I'll give it a try once it's in stable

Similar to @mnemosx, I am new to Prisma and was starting to believe that it wasn't worth the hype because all the properties of my models were implicit any with no auto-completion. However, 5.10.0-integration-fix-client-exports-map.14 does fix this bug. It might be worth it to display this somewhere more prominent and/or update the docs to be clear that the model itself is available as a type with all of its properties (not just the Select/Include/Query Specific Types/export declare type Model that isn't actually what's generated), so that it would be more clear that this was a bug and not just how Prisma works.

I have the same issue; ever since I switched to version 5.9.0, my types no longer work and are all any

@thomaslenaour @kinseyky1 Can you follow this and see if this resolves your problem? If so, this will be fixed with 5.9.1 that will come out soon.

@thomaslenaour @kinseyky1 Can you follow this and see if this resolves your problem? If so, this will be fixed with 5.9.1 that will come out soon.

Hey @janpio @millsp, I just tried https://github.com/prisma/prisma/issues/22903#issuecomment-1919930108 and it seems to be working fine on my end.

5.10.0-integration-fix-client-exports-map.14 works fine.
I also checked in two repositories.

https://github.com/prisma/prisma/releases/tag/5.9.1 is now available with the fix ✨
Please create a new issue if you find a bug.


[GALQFM] Quest for Magic - Crashes upon entering character name

Module Information

Identifier: GALQFM
Module Name: Quest for Magic
Version: unknown

Describe the bug
Crashes upon entering character name when you first enter module
To Reproduce
Steps to reproduce the behavior:

Enter Module
Enter your name

Expected behavior
You enter the game world
Screenshots

MBBSEmu Log:
Unhandled exception. System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'Unable to locate FFFF:00A9')
   at MBBSEmu.Memory.MemoryCore.GetWord(UInt16 segment, UInt16 offset)
   at MBBSEmu.CPU.CpuCore.GetOperandValueUInt16(OpKind opKind, EnumOperandType operandType)
   at MBBSEmu.CPU.CpuCore.Tick()
   at MBBSEmu.HostProcess.ExecutionUnits.ExecutionUnit.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.Module.MbbsModule.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassSetState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.HostProcess.MbbsHost.WorkerThread()
   at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ThreadHelper.ThreadStart()

Software Information:

OS: Windows 2019 Server Standard
Telnet Client: NetRunner

API REPORT:
{"UniqueIdentifier":"GALQFM","File":"GALQFM.DLL","Imports":{"DOSCALLS":[89],"PHAPI":[16],"MAJORBBS":[520,403,476,11,169,401,468,578,604,579,629,463,628,444,624,487,474,534,543,68,437,455,441,566,456,492,589,331,574,544,713,459,726,351,485,117,162,654,621,317,659,340,335,603,573,522,402,559,511,550,516,113,571,420,167],"GALGSBL":[40,58,59]}}

Access to memory location 0xFFFF:0x00A9 would be MAJORBBS.169 which is int digalw;
Specifies if Digits are allowed to be in User-ID's. We'll go ahead and just hard-code this to 1


[GHSA-4m48-j3xj-px27] MyBatis plus v3.4.3 was discovered to contain a SQL...

Updates

Affected products
Summary

Comments
https://www.cvedetails.com/cve/CVE-2022-25517 provides the affected library
and its maven url is https://mvnrepository.com/artifact/com.baomidou/mybatis-plus

Hi @q5438722, the maintainers believe this is not a valid vulnerability. Have you found anything that leads you to believe CVE-2022-25517 is valid?
Maintainers have disputed the validity of the vulnerability at these links:
https://github.com/baomidou/mybatis-plus/issues/5591
https://gitee.com/baomidou/mybatis-plus/issues/I5XPVJ
https://gitee.com/baomidou/mybatis-plus/issues/I56ZDZ
The original bug report is no longer online, but an archived copy is available. https://web.archive.org/web/20220323002637/https://github.com/HaHarden/mybatis-plus-sql-Injection

Actually, this vulnerability has been argued whether it belongs to a vulnerability and they have not achieved a consensus now (shown in the following url).
https://github.com/baomidou/mybatis-plus/issues/4407
Their main difference is that whether the passed field 'column' needs to be validated by the framework MyBatis.
Now that this vulnerability has been included in the following vulnerability database, I think it is valuable to complement the affected packages (and versions).
https://www.opencve.io/cve/CVE-2022-25517
https://nvd.nist.gov/vuln/detail/CVE-2022-25517
https://github.com/advisories/GHSA-4m48-j3xj-px27

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.

👋 This pull request has been marked as stale because it has been open with no activity.  If you do nothing this pull request will be closed eventually by the stale bot. Please see CONTRIBUTING.md for more policy details.


Render client config into HTML instead of embedding in bundle

Why?
At the moment, we are embedding the client config inside the JS bundle. This means that we cannot distribute rctf Docker images, and that people have to build their own.
How?
We can move all the client config into a JSON blob rendered into the HTML by the server, allowing us to remove all config from the Docker images. The JSON blob can be derived from the server config.

We can also load the blob via a script that runs just before server start and modifies the html file, so that we can use the same script when deploying the frontend to an entirely static server like netlify or nginx.

Fixed in #251


Addresses issue #204

Added test for adding a project through the API (/api/projects/projects.spec.js).

This is good for now. I need to add authorization checks, Thanks! I hope you could see that from the Travis CI output

This has already gotten added, so this PR is no longer orphaned!


[Documentation/Bug] Ensure authorizable documentation error

The Ensure Authorizable github.io page has an invalid example snippet:
Currently, it is
<?xml version="1.0" encoding="UTF-8"?>
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:cq="http://www.day.com/jcr/cq/1.0"
    xmlns:jcr="http://www.jcp.org/jcr/1.0" xmlns:nt="http://www.jcp.org/jcr/nt/1.0"
    jcr:primaryType="sling:OsgiConfig"
    principalName="my-service-user"
    type="add"
    ensure-immediately="{Boolean}true"
    aces="[type=allow;privileges=jcr:read\,rep:write;path=/content/foo;rep:glob=/jcr:content/*]"/>

It should have operation instead of type (per both the screen shot and the code)
<?xml version="1.0" encoding="UTF-8"?>
<jcr:root xmlns:sling="http://sling.apache.org/jcr/sling/1.0" xmlns:cq="http://www.day.com/jcr/cq/1.0"
    xmlns:jcr="http://www.jcp.org/jcr/1.0" xmlns:nt="http://www.jcp.org/jcr/nt/1.0"
    jcr:primaryType="sling:OsgiConfig"
    principalName="my-service-user"
    operation="add"
    ensure-immediately="{Boolean}true"
    aces="[type=allow;privileges=jcr:read\,rep:write;path=/content/foo;rep:glob=/jcr:content/*]"/>

It took me a little bit to figure out why changing from add to remove wasn't working :slightly_smiling_face:

@paul-bjorkstrand Please open this issue (in the best case with a PR) at https://github.com/Adobe-Consulting-Services/adobe-consulting-services.github.io/issues.

Done: https://github.com/Adobe-Consulting-Services/adobe-consulting-services.github.io/pull/197
I thought that the site was a private repo or something, but really my googling skills just failed me (even more so since I already had a fork for that repo). That was an utter fail on my part.


fsverity_test.go: fix nil pointer derefence, fix test fail, fix minor/major device numbers resolving

Fix nil pointer dereference:

Nil pointer deference happens when IsEnabled returns false, nil and err.Error() is called on nil value.

Fix test fail

Currently test fails when IsEnabled returns false, nil, i don't think it's fine (plus it only happens on one architecture, s390x), so i changed it to only accept false, err, or this test should also report false, nil?

Fix minor/major device numbers resolving:

Taken from https://tip.golang.org/src/cmd/vendor/golang.org/x/sys/unix/dev_linux.go
Current formulas for resolving major/minor device numbers is incorrect, which leads to warnings like this:
=== RUN   TestEnable
    fsverity_test.go:51: invalid device: device mount not found for device id 3:0
--- SKIP: TestEnable (0.04s)
=== RUN   TestIsEnabled
--- PASS: TestIsEnabled (0.05s)
PASS

I also renamed variables a bit to fix conflict with built-in identifier min.
Tested on x86_64 and s390x (virtual machine) architectures.
Should fix #10971 too.

Your commits are missing the Signed-off-by trailer.  Can you sign off on these commits so CI can run?

  Running [/home/runner/golangci-lint-1.60.1-linux-arm64/golangci-lint run  --timeout=8m] in [/home/runner/work/containerd/containerd] ...
  internal/fsverity/fsverity_test.go:146: File is not `gofmt`-ed with `-s` (gofmt)
  	major := uint32((stat.Dev & 0x00000000000fff00) >> 8) | uint32((stat.Dev & 0xfffff00000000000) >> 32)
  	minor := uint32((stat.Dev & 0x00000000000000ff) >> 0) | uint32((stat.Dev & 0x00000ffffff00000) >> 12)

/ok-to-test

thank you!


Additional functionality

Sorry about the last pull request. Checked in messed up dev-example, with my credentials. Reverted and corrected.
Add:
Get Sections
Update Task
Delete Task
Get resources
Get Task status
additional error handling

thanks for contributing to this project!! in fact i'll be using this package again so you might see new things too ;)

thanks for contributing to this project!! in fact i'll be using this package again so you might see new things too ;)

My pleasure! In case you're interested, I'm using it here: https://github.com/thesamim/TickTickSync
Looking forward to further developments.


ROX-19121: Add conditional RBAC for global search

Description
Analysis

GET /v1/search request requires several resources

Inferred from https://github.com/stackrox/stackrox/blob/master/central/search/service/service_impl.go#L66-L82
Confirmed from category values in counts property of response

Solution

Add resources to resourceAccessRequirements in routePaths.ts file.
Conditionally render Search button/link only if search route is enabled.
Add useIsRouteEnabled hook.

Residue

Conditional render links in search results as Not available if user role does not have (additional) resource requirements for route.

For example, risk require DeploymentExtension in addition to Deployment resource.

Replace route paths with route description keys from string enumeration type for better developer experience in a future follow up after necessary pull requests have been merged.

Checklist

[ ] Investigated and inspected CI test results
[x] ~Unit test and regression tests added~

Testing Performed

yarn lint in ui
yarn build in ui and then compute branch - master for the following

wc build/static/js/*.js
main.css: 338 = 3760119 - 3759781
total: 338 = 9949512 - 9949174
ls -al build/static/js/*.js | wc -l
0 = 81 - 81 files

yarn start in ui

Manual testing

Visit /main/search and search for Namespace: stackrox

Before conditional rending, without resources, see 403 Forbidden status: not authorized

After conditional rendering, without resources, see absence of Search button and not found for route

With minimum resources, see search results including links to routes that are not enabled (see Residue item 1)

/test ocp-4-13-ui-e2e-tests


write script for toggeling bluetooth controller in status bar

should be possible with sed and bluetoothctl show

works actually with just bluetoothctl show | rg 'Powered: on'


Terraform Output coming as a Json

I used to run  something like this to fetch terraform outputs and run some comparison on top of it,
	// Checking EFS file system is created.
	efsArn := terraform.Output(t, terraformOptions, "cluster_efs_arn")

It used to provide a output like this ,
TestTerraformAwsEcs 2020-11-25T13:16:05Z retry.go:91: terraform [output -no-color cluster_efs_arn]
2280TestTerraformAwsEcs 2020-11-25T13:16:05Z logger.go:66: Running command terraform with args [output -no-color cluster_efs_arn]
2281TestTerraformAwsEcs 2020-11-25T13:16:06Z logger.go:66: arn:aws:elasticfilesystem:us-east-1:21354357:file-system/fs-7ec38b

now all of a sudden i see a change, i.e it started to provide output as json for no reason.
TestTerraformAwsEcs 2020-12-13T16:00:06Z logger.go:66: Running command terraform with args [output -no-color -json cluster_efs_arn]
TestTerraformAwsEcs 2020-12-13T16:00:11Z logger.go:66: {
TestTerraformAwsEcs 2020-12-13T16:00:11Z logger.go:66:     "sensitive": false,
TestTerraformAwsEcs 2020-12-13T16:00:11Z logger.go:66:     "type": "string",
TestTerraformAwsEcs 2020-12-13T16:00:11Z logger.go:66:     "value": "arn:aws:elasticfilesystem:us-east-1:213543574:file-system/fs-f10d7904"
TestTerraformAwsEcs 2020-12-13T16:00:11Z logger.go:66: }

i have not made any change either in terraform or in terratest,, how can i get rid of this json for permanent basis

What version of Terratest are you using? We intentionally changed how terraform.Output works under the hood to use -json due to a backwards incompatible change in Terraform itself. See https://github.com/gruntwork-io/terratest/pull/719 for details. This was released in https://github.com/gruntwork-io/terratest/releases/tag/v0.31.0 and then patched in https://github.com/gruntwork-io/terratest/releases/tag/v0.31.1. That said, while what's happening under the hood is different, the external behavior of the function should be unchanged: i.e., terraform.Output should return the same value as before. Are you not seeing that?

so i was using the lastest one (0.31.1), i.e i had not locked its version before,,so this was the one that broker it some 5 to 6 days back. Now i had to downgrade it  manually to below version to unblock my production realease.
And Nope, now the default behaviour is json output as shared above, which fails my test,,,  is there any way that we can use the latest version and still get json less output
[[override]]
  name = "github.com/gruntwork-io/terratest"
  version = "v0.30.8"

so i was using the lastest one (0.31.1), i.e i had not locked its version before,,so this was the one that broker it some 5 to 6 days back. Now i had to downgrade it manually to below version to unblock my production realease.

Yea, we always recommend pinning a version to avoid accidentally pulling in backwards incompatible changes.s

And Nope, now the default behaviour is json output as shared above, which fails my test,,, is there any way that we can use the latest version and still get json less output

Can you share more details on what you're seeing? That is, your Terraform code with the output and your Terratest code that calls terraform.Output?

so i do some thing like this to set some variables after terraform succeeds.
	// Checking ALB & Other services are UP and RUNNING.
	albDNS := terraform.Output(t, terraformOptions, "alb_url")
	flowerDNS := terraform.Output(t, terraformOptions, "flower_dns_name")

And then do a simple endpoint test of these services.
	http_helper.HttpGetWithRetryWithCustomValidation(t, fmt.Sprintf("http://%s", albDNS), &tlsConfig, retries, SleepBetweenRetries, func(statusCode int, body string) bool {
		if statusCode == albCode {
			logger.Logf(t, "Got expected status code %d from URL http://%s", albCode, albDNS)
			return true
		}
		return false
	})

And what happens when you run that code? The JSON output in the logs is expected... That's the output terraform output returns when you pass the -json flag. The terraform.Output method should be taking that JSON output and parsing it into a string for you, so the value returned for albDNS and flowerDNS should be the exact output strings you expect.

That's where the problem lies,, I don't need the json output,, and I have not made the change to get the json output i.e -json not set by me,,, it's automatically picking up -json with the new update,,,
Now for me to use it as is, I would need to add more logic later to extract the value,, so that I can run the http testing,,
So the main question can we or do we have something which can eliminate the json output,, like passing flag or something

Can you paste the EXACT output you're getting from terraform.Output? Not what you see in the logs, but the value of the value returned. E.g., Print out albDNS and flowerDNS in your example above.

Hey, I'm experiencing a similar issue to this.  I think the terraform.output function is returning the command rather than the value of the output variable.
I've raised a question in StackOverflow.  Would it be helpful for me to raise a separate issue in GitHub or do you think it's similar enough to track here?

Ignore my comment, the solution proposed in this issue solved it for me

Since we haven't seen any recent activity here, I'll assume the issue has been solved and close this ticket. However @ma-kadri please feel free to reopen if you still need help with this!

Since we haven't seen any recent activity here, I'll assume the issue has been solved and close this ticket. However @ma-kadri please feel free to reopen if you still need help with this!

I'm coming across this same issue. The output function isn't parsing the JSON and just returning the string as is.

Hi @howdoicomputer , can you open a new issue, with the following details?

version of terratest
version of terraform
terratest go code snippet for the relevant call to Output
relevant log snippets that you can share


what is the''please wait while GUI connect to COM3D2''

u mother fk:)

Greetings!
Please provide information about the game version and Maid Fiddler version you're using.
Since you've posted this, both game and Maid Fiddler were updated.
Generally, the message means that Maid Fiddler is waiting for your game to launch so it connect to it.
If you have any issues, please create a new issue and follow the provided template.


cumulative  design  problem

because both histogram and summary are cumulative,  when i want to write prometheus metirc to other db like elasticsearch,i can not get the right result  using other db's aggregation query.
and other problems happen when one instance of my service died,because the count and sum become 0 after restart!
can prometheus provide metrics  this way? for example ,when one api is requested,the following metric is generated:    request_test(uri="/api/test",value=1,lantancy="2ms")
this time,because value is always 1,caculating the max count is very easy, lantancy aggregations like max ,min and avg   are alse easy😄

It makes more sense to ask questions like this on the prometheus-users mailing list rather than in a GitHub issue. On the mailing list, more people are available to potentially respond to your question, and the whole community can benefit from the answers provided.


Code commit for RT-3.1

Code commit for RT-3.1

/gcbrun to see if it fixes featureprofiles-pr CI check

PR exist for same https://github.com/openconfig/featureprofiles/pull/213

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
66 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-1.7%) to 60.202%

Files with Coverage Reduction
New Missed Lines
%

topologies/binding/binding.go
66
37.35%

Totals

Change from base Build<PHONE_NUMBER>:
-1.7%

Covered Lines:
596

Relevant Lines:
990

💛  - Coveralls

/gcbrun to see if it fixes featureprofiles-pr CI check

/gcbrun to see if it fixes featureprofiles-pr CI check

/gcbrun

Hi @smulky please rebase or merge this PR with the latest commit from main to ensure you have all dependencies up to date, including the featureprofiles paths (that are unrelated to your changes, but possibly out of date )

@bstoll do you think you could finish this PR on behalf of @smulky ?

Due to some merge issue, i have raised a new PR: https://github.com/openconfig/featureprofiles/pull/521 Please see.

Should we close this and continue the review in #521?


Fixing updates failures using the Fields new_fields_structure method

retest this please

retest this please

Trying to close the PR and resend