client.Get on missing network mapping returns StatusCode 400 instead of 404

Bug Report

import path of package in question azure-sdk-for-go/services/recoveryservices/mgmt/2018-01-10/siterecovery
SDK version v41.2.0
output of go version go version go1.13.7 darwin/amd64

What happened?
Invoking client.Get(ctx, fabricName, networkName, networkMappingName) where for a valid fabric and network but where no mapping has yet been created will return a response where the StatusCode is 400

What did you expect or want to happen?
Expected respinse code is 404. The other APIs return 404 if a resource is missing

Anything we should know about your environment.
This used to work before, not sure when it stopped working, but it is causing issues for terraform users.

See bug reported on terraform: https://github.com/terraform-providers/terraform-provider-azurerm/issues/6026

Thank you for reporting this issue to us. Marking issues for service team to respond. Here is the link to the operation in REST API spec.

This is a bug. We are issuing a fix in the next release (ETA 1st week of June 2020). Bug: https://msazure.visualstudio.com/One/_workitems/edit/6244666/

We are still reviewing and will respond with an update shortly.

We are planning to release a fix for this in our future updates.

Please feel free to reopen for any additional queries.

This bug has been fixed an released. Please feel free to reopen in case you still face any issues.
#please-close

#please-close

#please-close

@allenjzhang - could you please close this issue? I'm unable to close it.

Sure. Thank you for fixing the issue.


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2024-12-04)
Features

add link to GitHub emulator repository (#4) (4fb6054)

Continuous Integration

add content write permission to artifact upload (#12) (3697b08)
add release please action (#1) (a78ec8e)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.1.0
:sunflower:


Make script execute in a function

If the script is only partially downloaded this can result in a pipefail
when doing:
curl -fsSL script_url | bash

This makes it so that the main function isn't actually executed until
the very end of the download.
This is the same type of pattern that we follow in docker/docker-install
Change was made without changes with indentation so that the diff won't be huge.
Signed-off-by: Eli Uriegas<EMAIL_ADDRESS>

Closing this


Verify connectivity in chapter "Add connectivity between spokes" fails...

... since Azure Firewall blocks ping <IP_ADDRESS>

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

ID: 104eba32-c77e-b013-e161-ff616d16a0f7
Version Independent ID: 3bc27d0a-ef32-f5f6-4c85-fe8e2e45155e
Content: Implement a hub-spoke network topology in Azure - Azure Reference Architectures
Content Source: docs/reference-architectures/hybrid-networking/hub-spoke.md
Service: architecture-center
Sub-service: reference-architecture
GitHub Login: @telmosampaio
Microsoft Alias: pnp

@fawohlsc Thanks for the feedback! We will review and update accordingly.

@fawohlsc You can add ICMP as a protocol in the firewall rules to get this to work. For reference, the template would need to look something like in this commit.
We are assigning this issue to the content author for further review.

@fawohlsc The doc has been updated to use nc (netcat) instead of ping removing the need to allow ICMP and simply utilize the SSH port itself to check for connectivity.

THanks @PramodValavala-MSFT  :-)


Question of  training shuffle

Thank you for sharing your great work. I notice the parameter of training shuffle is True. I think it will break the order of time series. I don't understand

Shuffling is for the batches and not for the actual timesteps. For example you have two batches of your time series and let's say the input of the model has to be of size [n_batches, timesteps](No feature dimension for simplicity). What shuffling will do is shuffle the order that the model receives the batches and not the actual timesteps. Lets say we have two batches with 5 timesteps [[1,2,3,4,5],[0,-1,-2,-3,-4]] (n_batches=2,timesteps=5). We can shuffle the order of the batches and have [[0,-1,-2,-3,-4],[1,2,3,4,5]], but we are not shuffling the actual timesteps [[2,5,3,1,4],[-3,0,-2,-1,-4]] because that would hurt the model.

Thank you for answer this, I understand it!

Thanks for your valuable explanation! @ChuckTG

well done~


Linking boards needs to reset state on the second board

Currently, if the state on both boards is the same (ish) it's probably ok. However, you can link boards with different state and then a move will run moveSafely for both, and only work for one, which is annoying. We need a setter that is public for QuantumChessBoardEntity that takes care of the cloning process, call it quantumImprint and pass it a QuantumChessBoardEntity and then we don't have to worry about what else we need to clone down the road for other boards :D

Will have to ask for more instrctions for this issue. So far I created a skeleton for the quantumImprint() but if I understand well, we will need to deep cope the state before linking. I will push now what I added. Also I was reading and taking notes (especially NBT) on your new addition to the code and I added some comments to help me understand better.


YQL-17250: Add operation name in fallback message

@rvu1024, all comments are met, please, check again.

@rvu1024, please check again.


Getting Could not load file or assembly Microsoft.AspNetCore.StaticFiles, Version=<IP_ADDRESS>

Hi,
I am trying to use Swashbuckle 1.1.0 with AspDotnetCore 2.0 but I am getting the below error.
'Could not load file or assembly 'Microsoft.AspNetCore.StaticFiles, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. The system cannot find the file specified.'
Why does it search for <IP_ADDRESS>?
In the NuGet packages, I see that the SwaggerUI which depends on Microsoft.AspNetCore.StaticFiles uses version 2.0.0, but the error says it is looking for 1.0.4?
Could someone shed some light on what am I missing?

Hi,
We have the same problem in our project. We use Microsoft.AspNetCore.StaticFiles in version 2.0
Best Regards
Krzysztof

@itanev - can I close this or do you still feel there's a change that can me made to SB here?

Close it. :)


[map] a few small fixes

added missing replot on prop update
fix domain / color for wet bulb temp
remove borders for districts in india

also updated tooltips / hovers:

reduce opacity of non-selected regions, add shadow behind tooltips and update highlight border

small update to scatter & map tooltips too (content-only update):


Distributed setup bug

I got this error while executing the distributed setup described in README:
"./distributed.py", line 95, in allreduce_hook
AttributeError: 'Parameter' object has no attribute '_execution_engine'

With a bit of searching it seems to be related to: https://github.com/pytorch/pytorch/commit/6b3a4637d6e87a543b3df6edb825c07d7dabe611#diff-28369e7c07a82d014949812f10d9a188
(And it also seems the same problem is/was in https://github.com/NVIDIA/sentiment-discovery/blob/master/model/distributed.py)

@IwraStudios thanks for sharing.
@raulpuric any thoughts on this given that we had a similar problem in our sentiment-discovery repo?

@rafaelvalle Found the issue from sentiment-discovery on the topic https://github.com/NVIDIA/sentiment-discovery/issues/16

@IwraStudios can you share the full error trace?

Ah right sorry about that
Traceback (most recent call last):
  File "train.py", line 291, in <module>
    args.warm_start, args.n_gpus, args.rank, args.group_name, hparams)
  File "train.py", line 222, in train
    optimizer.backward(loss)
  File "/home/null/nv-wavenet/pytorch/tacotron2/fp16_optimizer.py", line 362, in backward
    self.loss_scaler.backward(loss.float())
  File "/home/null/nv-wavenet/pytorch/tacotron2/loss_scaler.py", line 80, in backward
    scaled_loss.backward()
  File "/home/null/anaconda3/lib/python3.6/site-packages/torch/tensor.py", line 93, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph)
  File "/home/null/anaconda3/lib/python3.6/site-packages/torch/autograd/__init__.py", line 89, in backward
    allow_unreachable=True)  # allow_unreachable flag
  File "/home/null/nv-wavenet/pytorch/tacotron2/distributed.py", line 88, in allreduce_params
    coalesced = _flatten_dense_tensors(grads)
  File "/home/null/nv-wavenet/pytorch/tacotron2/distributed.py", line 19, in _flatten_dense_tensors
    flat = torch.cat([t.contiguous().view(-1) for t in tensors], dim=0)
RuntimeError: Expected a Tensor of type torch.cuda.HalfTensor but found a type torch.cuda.FloatTensor for sequence element 3  in sequence argument at position #1 'tensors'

@IwraStudios thank you. we currently do not have the cycles to work on fixing this bug.
If you're ok with using DistributedDataParallel without FP16, just disable FP16.

Alright, I'll disable it for now.
Shall I close the issue or do you want to keep it open until fp16 gets fixed?

Let's keep it open as a record that the issue has to be solved!

@IwraStudios
One solution for   distributed FP16 training for Tacotron2 would be use NGC:18.03-py3  and simple patch.
I already build docker images you would just use docker pull hryu/pytorch:t2-2 it is base on NGC 18.03 which based on  pytorch 0.3 and APEX patch. see detail Dockerbuild file. after that, you need to implement simple patch for source codengc-patch for pytorch 0.3 this patch don't support pytorch 0.4. so it is just temporal solution.
detail instruction are in readme file

@IwraStudios  close this bug.
I check FP16 training work well in NGC 18.06-py3 with below method.
I check this is solved with below method.
replace current distributed.py with new apex distributed.py
(https://raw.githubusercontent.com/NVIDIA/apex/master/apex/parallel/distributed.py)

@yhgon
Do you know (estimate) how long would take to train nv-wavenet with LJSpeech dataset and
1 Tesla V100 GPU?

@IwraStudios @yhgon  I think you can just " flat = torch.cat([t.half().contiguous().view(-1) for t in tensors], dim=0)" to fix this.


captureMessage can attach stack trace via stacktrace: true option

See also #581

not by default, must specify new option: { stacktrace: true }
this can be passed directly to individual captureMessage calls, or set globally during config
overrides grouping/fingerprint so that stack trace doesn't change existing behavior

good idea? makes sense when user intentionally calls captureMessage, but maybe not if captureMessage was the result of a failed captureException call (e.g. bad error object)

fixes #544
fixes #503 (since attached stack trace will include URLs that filters can/will reject)

cc @mattrobenolt @mitsuhiko @dcramer @bradvogel

This change is

Let's use the sentry.interface.Stacktrace interface explicitly and avoid sending along an Exception interface for the synthetic exception.

Let's use the sentry.interface.Stacktrace interface explicitly and avoid sending along an Exception interface for the synthetic exception.

I mean, this is ideal, but to do so means a colossal refactor of Raven.js :(

Example of Raven.captureMessage('lol', { stacktrace: true }) in the UI (triggered from console so no source maps etc):

This is no longer WIP. My plan is to test this live on app.getsentry.com for a short while, then we'll roll it into 3.4.0.

@benvinegar Any updates on this? This would be a huge help for me :grin:

@simon-weber yep trying to push this through. Big changes like this make me nervous so I just need to QA ourselves first.

Okay, after some testing I've decided that attempting figure out the "tail" Raven frames has more cases than I've considered, so I've reverted that part of the code.
This now strictly just does synthetic stack traces via Raven.captureMessage (or as a fallback if captureException is given a string or non-error).
cc @mattrobenolt @macqueen – can you look at this again?

Reviewed 2 of 3 files at r6, 3 of 3 files at r7.
Review status: all files reviewed at latest revision, 2 unresolved discussions, some commit checks failed.

Comments from Reviewable

Reviewed 1 of 1 files at r8.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Cool! What version is this targeting? We're past 3.4.0 now.

3.6.0.


fix bugs for the newest version of pytorch_lightning

sync for pytorch-lightning 1.5.10:

trainer.hparams -> save_hyperparameters
trainer.reload_dataloaders_every_epoch -> trainer.reload_dataloaders_every_n_epochs
trainer.running_sanity_check -> trainer.sanity_checking
periods -> every_n_epochs

@PhilippThoelke do you see any other issues?

@lsnty5190 would you mind to update the version (https://github.com/torchmd/torchmd-net/blob/main/environment.yml), so the CI can pass?

The branch where I started moving to a more recent pytorch-lightning version is still there (I did this 8 months ago though, stuff has changed in torchmd-net and pl since then but maybe it still helps): https://github.com/torchmd/torchmd-net/tree/new_pl
Have a look at this for a list of changes that I made: https://github.com/torchmd/torchmd-net/compare/new_pl

I made a new attempt at upgrading the code base to pytorch-lightning 1.6.3 (most recent version currently) and some of the problems we had earlier seem to be gone now. Closing this now as it is superseded by #97.


support VALUES as a table we can join on

SELECT a.name, v.address FROM (VALUES('101 Main St City USA', 'foo', 'bar'), ('24 Sussex Drive Ottawa Canada', 'foo2', 'bar2')) vals(address, foo, bar) CROSS JOIN A
with any kinds of joins being supported as usual. So (VALUES) looks like a subquery (SELECT ...).

I think this is already supported via the literal read, but we can add a test too.


add mypy to tox.ini

Description
integrate mypy into tox environment -- fixes issue #41
Implemented changes

Insert a description of the changes implemented in the pull request.

[x] mypy.ini (exclude missing imports, pymovements/plot.py)
[x] tox.ini -- call mypy tests
[x] fix mypy error in transform.py

Type of change
Please delete options that are not relevant.

[x] new python code check

How Has This Been Tested?

[x] it's a mypy -- a test

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

LGTM thanks for your work!


Any plans for react component library?

Hey,
great job with Open props. Make much more sense to me than tailwind.
Is there a plan to release a component library for react on top of this?
It would be the obvious way to monetize Open props while providing a huge value for developers.
Thank you

it's crossed my mind, but I'm leaving the door open for someone else to build right now


[multistage] support the rest of the agg functions currently implementable

Codecov Report

Merging #11208 (bb54a29) into master (35328c0) will increase coverage by 0.00%.
Report is 2 commits behind head on master.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master   #11208     +/-   ##
=========================================
  Coverage    0.11%    0.11%
=========================================
  Files        2225     2171     -54
  Lines      119268   116811   -2457
  Branches    18036    17747    -289
=========================================
  Hits          137      137
+ Misses     119111   116654   -2457
  Partials       20       20

Flag
Coverage Δ

integration1temurin11
?

integration1temurin17
?

integration1temurin20
?

unittests1temurin17
?

unittests1temurin20
?

unittests2temurin11
?

unittests2temurin17
0.11% <0.00%> (-0.01%)
:arrow_down:

unittests2temurin20
?

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

Files Changed
Coverage Δ

...che/pinot/segment/spi/AggregationFunctionType.java
0.00% <0.00%> (ø)

... and 69 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Concerned with Data Usage

First of all, fantastic idea!  Great instructions: I got it running in no time.
Now for the potentially bad news...  I've let it run for 3-4 days: the RPi is pulling down an almost constant 3.16Mbps (fluctuating to 1.59Mbps every now & then) & has used 10GB of data over the last 24 hours.  Is this normal expected behavior?  Still digging into why, but here is why this bothers me...  (hopefully it's just a config problem as I have seen others mention)
Note: I haven't started using pi-hole yet - it's configured, but none of my machines are using it.  I wanted to see if the rest of the "playbook" would suit my needs before I start using pi-hole.
My Situation: I live in a rural area with P2MP wireless internet (still on the Starlink waiting list - 1.5 years later).  All the WISPs in the area are small "mom & pop" shops with limited bandwidth & throughput options (max 50 down x 4 up) for high dollar costs...  So, I went looking for something to help me track uptime, outages & current speed.  Basically, I was looking for a way to document outages (we have many), a dashboard to quickly see the "health" of my internet connection, & another data source to potential automate from.
Unfortunately, if the constant +3Mbps down is normal, I might need to look elsewhere (it could be pi-hole, but I would have expected it to do almost nothing until I started using it).  I would have expected the speedtest CLI to use bursts of data when run then sit idle.  Am I missing something?  I do see pinhole working, but it's just processing the localhost work.
Any thoughts on what could be wrong or is it normal?  I unfortunately can't tie up 3Mbps for monitoring.

3 Mbps shouldn't be constant, I wonder what kind of data is being pulled (like maybe a Docker image download in a perpetual cycle or something...).
The main data consumption would be periodic speedtest runs, which do pull a bit of data per run, so you can configure that frequency with monitoring_speedtest_interval — to conserve data, consider a value like 6h or 12h so it's only a couple times a day.
But I would install iftop and do a sudo iftop eth0 to monitor what kind of connection is using so much constant data. It should at least tell you what host or IP address is being called out to, and may give a hint.

@geerlingguy - Thanks for the quick reply!    I was only seeing it previously at the aggregate level.  The little RPi is busy though...  I have a few other things (local monitoring) running...  Here's a screenshot.
Note: I waited till the speedtest was done.

Much more traffic than I expected from google (1e100.net) since I'm not configured to use them on pi-hole.  Also the blipped 1-2mb traffic is from GitHub (once every 2 seconds).  Which isn't expected either.  Thoughts?
Looks like it's not as bad as I thought, but still hmmmm.

@geerlingguy - additional detail.  I installed Portianer to see if I could glean any detail from there.  All looks healthy & I don't see anything wacky going on from a container perspective.

What I do see is the is the container: internet-monitoring_ping_1 has an extreme amount of network data usage (where the other containers don't)

Thanks again in advance for any help or advice.  I think I may start over (same machine - new image build) to see if something went bonkers during the playbook install.

That's a lot! The ping container does reach out and perform I believe an HTTP HEAD request on all the configured URLs in https://github.com/geerlingguy/internet-pi/blob/master/example.config.yml#L16-L19, but that shouldn't be a lot of traffic (in terms of bytes).

Agreed - I will rebuild clean to see if it's this is specific to the execution of my build.  Any other thoughts before I do?

@geerlingguy - a bit too soon to tell, but it looks like it is running better on the clean rebuild.  I went ahead & changed the ping interval to 30 seconds, but I don't think 5 seconds was the problem, as it seemed the first build container was confused & doing things it should not have been doing.  I'll report back if I see it misbehaving after a few days of data collection.

@geerlingguy - not sure what the cause was (I stopped looking), but the new clean build doesn't exhibit the same network chatty behavior. This can be closed.
Might not be the best place for an enhancement asks, but have you considered the following:

Adding a Cloudflared container to your playbook? (I will be testing & adding manually soon)
Changing the use of SpeedTest CLI for Cloudflare or giving the option at least?
Exposing Jitter as a meter in grafana (there is space to the right of the ping meter).  Looks like the data is in Prometheus, but needs to be massaged to be used.  (I've tried but failed on a few brief attempts)

Note: I can post separately if needed as an enhancement request.

Closed - issue resolved with rebuild


Model associations?

Are there any plans/ideas about having associations between models? Maybe like has_many, belongs_to, etc?
I can start working on this with some guidance on how do we want it to look like.

My initial sketch is for a family of generic classes that manage the db, such as HasManyHasManyAssociation, BelongsToAssociation, that you add as private properties to your Model implementation and corresponding public properties exposing the models returned by the associations
class Post: Model {
    // Model implementation - id, init, etc

    private let authorAssociation = BelongsToAssociation(User, via: "author_id")
    var author: User {
        get {
            return authorAssociation.model
        }
        set {
            authorAssociation.model = newValue
        }
    }
}

Yes we definitely want to include those types of relations at some point. Though I think that's farther down on the roadmap than we currently are. I'd like to have working SQL and MongoDB support for the basic features (creating, reading, destroying db models, etc) before we move into more advanced features.

This is on the roadmap now.


application design patterns

Is there a paper that documents the design patterns (e.g., C2G, C2GI) for some of the newer benchmarks (e.g., ep)?

@apanqasem
Here are two papers we published recently.
https://www.researchgate.net/publication/309206460_Hetero-Mark_A_Benchmark_Suite_for_CPU-GPU_Collaborative_Computing
https://www.researchgate.net/publication/323745962_Evaluating_Performance_Tradeoffs_on_the_Radeon_Open_Compute_Platform
Note that the second paper is not presented at the conference yet. There are some benchmarks and features mentioned in the second paper but are not included in the current Github repo. We are still organizing it and we will publish the code soon.

got it, thanks.


Ahead of Time (AoT) Issue when running ngc

When running Ngc command I get this error:
Unexpected value 'CarouselModule' imported by the module 'SharedModule'
Hopefully you can make it work with AoT compiler.
Tried importing each of the components and directives, but no luck

It is related to metadata.json issue

I'm simply getting:
Error: Unexpected value 'Ng2BootstrapModule' imported by the module 'AppModule'

Related to that issue as well?


Unable to Remotely Debug APIM Policies with VS Code and APIM Extension

I followed the instructions at the link below to debug my APIM policies with VS Code 1.53.2 and APIM Extension 0.1.8 on Windows x64 10.0.19042, but the the APIM Extension did not automatically open a new editor window after establishing the debugging session via the "Start Policy Debugging" context menu.  So I manually opened a new editor wiindow and sent an HTTP request to my API.  Although a successful response was received, the APIM Extension did not open a debug  window showing that it had entered my policy, nor did it stop at any breakpoints.
https://docs.microsoft.com/en-us/azure/api-management/api-management-debug-policies
Previous to installing VS Code 1.53.2, this APIM policy debugging feature was working for me.   My launch.json file looks like this:
{     "version": "0.2.0",     "configurations": [             {             "type": "apim-policy",             "request": "launch",             "name": "Attach to APIM",             "stopOnEntry": true         }     ] }

@marcami Can I know the skuType of the instance?

@RupengLiu Developer

while you are writing the HTTP Request file, did you add the Ocp-Apim-Debug header like the example below:

I added the Ocp-Apim-Debug header, and I get the Ocp-Apim-Trace-Location in my response.  However, I am not breaking on entry into my APIM policy.

Week ago, there was an regression from the backend that breaks the debugging feature. I fixed it and reupload the vscode extension (same version) to the marketplace, can you try uninstall and install again

I uninstalled v0.1.8 of the APIM extension and re-installed it, but I still get the same behavior.


Add mahval to Contributors list

@mahval, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


perf(material/checkbox): do not run change detection when the change event is dispatched on the input

We talked about this within the team and we decided that all the extra code isn't enough to justify an extra change detection so far down in the component tree.


Added additional infos to 503 error on cache warmup

1. Why is this change necessary?
Save error seek time
2. What does this change do, exactly?
adds information to 503 error
3. Describe each step to reproduce the issue or behaviour.
set maintenace mode to true and execute a cache warmup
4. Please link to the relevant issues (if any).
no issues
5. Which documentation changes (if any) need to be made because of this PR?
no changes
6. Checklist

[x ] I have written tests and verified that they fail without my change
[x ] I have squashed any insignificant commits
[x ] This change has comments for package types, values, functions, and non-obvious lines of code
[x ] I have read the contribution requirements and fulfil them.

Other ideas:

You could check for header "Retry-After" to detect maintenance mode
adding a new header

https://github.com/shopware/shopware/blob/6308ab80aab842f3050959d0efec3ad5d4e57e0b/engine/Shopware/Controllers/Frontend/Error.php#L197-L201

this could be added in a other branch 😬  at the moment I dont really know how to realize this 😄

Hey @Dwza ,
Thanks for your contribution 💙
Could you please squash your commits?

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://developers.shopware.com/contributing/contribution-guideline/.


Requisition lines disappear after Supplier Invoice is synced back to mobile store

Build Number: current master (last commit 17/08/17 34a449a27baaab071b4817085c9bedb0282e3a1b) and mobile inter-store-requisition-cleanup branch.
Description: When a customer invoice is synced out from Store B and a Supplier Invoice is created in Store A following a request from Store A, the requisition lines on the original Supplier Requisition (in Store A) and the Customer Requisition (on Store B)  disappears on mobile.
Reproducible: Always (as long as SI is being created on mobile)
Reproduction Steps: With two mobiles running, create Supplier Requisition on Store A. Sync both stores to create Customer Requisition and Customer Invoice on Store B. Sync both stores again. If a supplier invoice is created in Store A, the requisition lines on the Supplier Requisition and Customer Requisition disappears, but the requisitions themselves remain (as in screenshots below.)

Comments:
The Requisition and the Requisition lines are still there on the mSupply server, even after they have disappeared on mobile.

@edmofro off the hook, @andreievg takes full responsibility (and totally not me for letting it past review). Fixed in #481


URL for Proxy Settings

In order to configure a proxy to allow traffic for this task, the URL needs to be supplied. There is no URL in the logs showing what the task is calling. Please provide the URL the task is using, either in the logs or in the documentation.

@StephenRSkinner Thank you for the feedback.
Please include the task name or article referenced and I can assign an author.


[BUG] Navbar in Darkmode stays white and light

Describe the bug
Clicking the dark mode toggle switch every content to dark, except the navbar. Did I miss something ?
Steps To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Screenshots

Environment

Item
Version

Laravel
8

Project
v3.7.0

OS
OSX

Additional context

Hi @invaders-xx , It works like on the underlying AdminLTE package, you can test it here, open the right control sidebar and check the dark mode option.
Feel free to open an issue on the AdminLTE package, since this style concept is related to that package.

My apologies! You're right and I'am wrong. I thought that the navbar should also change. Many thanks for your quick feedback.


two-mode-banner + svg-logo

@afred : please review. Two distinct things:

The navbar needs a homepage mode, and a non-home mode. This does that.
The SVG is a tiny bit gratuitous, but it will make it so much easier if we need to change size, or wording, or color down the line.

@mccalluc don't we need the SVG to include the "Stock Sales & Licensing" moniker? Or was your plan to overly text?

@mccalluc also.. what's the best way to view an SVG? When i view the file in github, it looks like this:

and when I copy the source to my local machine, and view it in a browser, it looks like this...

and I'm not sure of what other app to view it in... Preview doesn't work.

We need to make sure that however you handle the Stock Sales logo, the proportions of the WGBH logo to the text is maintained for branding purposes. That's why it might be better to use the png and scale it down for smaller browser widths (?) I'm noticing now that it getting squished horizontally in mobile view.

(fixes #28)

@mccalluc the cropped version of the logo I was seeing in chrome is apparently due to the height="70". I increased that and was able to see the text. Does that need to happen in order to correctly display on the website?

@afred review. Not sure why SVG preview here is weird, but looks good in context.


Mocking library which solves the method chaining mocking diffculties

I'd like to nominate a PHP Mocking Framework inspired by Java Mockito library (support from PHP 5.4+). which solves the Method Chaining difficulties when mocking chained methods.
and also contains partial mocks
https://github.com/danrevah/ShortifyPunit
install able by Composer and PSR-4 compilant
100% Code coverage, high code quality/standards and is actively developed

Thanks for the suggestion.


how to deploy to xbox?

Hi, I'm using win10 insider preview and visual studio 2015 RC to develop the uwp app. How can I deploy my apps to my xbox one? Is there any information about this? Thanks a lot!

Thank you for your interest in the Windows 10 platform.  Unfortunately, this is no the correct forum for this discussion.  I recommend you follow the xbox site for more information - http://www.xbox.com/en-US/developers


How to handle brace expression restrictions?

Various places do not allow struct expressions. All of the following fail to parse in libsyntax, but pass in the current lyg grammar:
if S{a:1} {}
if let x=S{a:1} {}
while S{a:1} {}
while let x=S{a:1} {}
for pat in S{a:1} {}
match S{a:1} {}
This restriction applies recursively (I'm not sure how to express this), that is the following is also rejected:
if x==S{a:1} {}
if S{a:1}.foo() {}
I included a field just because rustc will give a helpful suggestion, whereas a fieldless expression gets confused by the second {} which rustc just treats an independent block, as this hilarious example shows:
#[derive(PartialEq)]
struct S;
let x = S;
if x != S{} {
    // This block is executed because it is not part of the `if` expression,
    // even though it appears to be.
    println!("hm, x!=S{} should be false");
}

Should this kind of ambiguity rejection be part of the syntax?
See https://github.com/rust-lang/rust/pull/59981 for some examples, and where this was recently changed. See also https://github.com/rust-lang-nursery/reference/issues/569.

For the if x != S{} { /* block */ } version, this has to parse the way it currently does, to allow the perfectly legal and normal code of comparing two variables in a condition. Here we just have to lean on rustfmt formatting, I guess.

Oh, I'm not implying it is wrong for libsyntax to interpret it the way it does. Although a warning might be nice, this is a contrived example that I doubt anyone would ever write.
I'm more trying to understand how the grammar can be written to express this behavior.  libsyntax uses "restrictions", and in https://github.com/rust-lang-nursery/wg-grammar/pull/13#issuecomment-436125874 @eddyb mentioned the open question of how to handle this.  #39 is another example where restrictions need to be expressed.
Is it on the table for the GLL parser to be able to express these directly?  If not, then it seems you have to bifurcate the definitions (ExprNoStruct would have all expressions except Struct, but that would require recursively copying all the expressions, which I think would be terrible).
What are the options for handling these kinds of restrictions?

I think https://github.com/rust-lang/gll/issues/27 could be a way to handle it with "flags".
I did something similar in the specification of or-patterns, https://github.com/rust-lang/rfcs/blob/master/text/2535-or-patterns.md#grammar.


:recycle: Chore: move INITIAL_ACCOUNTS to it's own file

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1056
#1055  👈
#1054
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


Enable gpu addon in an offline method

Hi,
We are already using offline install method to deploy microk8s using --> https://microk8s.io/docs/install-alternatives#heading--offline
Now we would like to enable gpu addon using an offline method. While checking few issues on github I found this issue
-->  https://github.com/canonical/microk8s/pull/1295/files.
If I understood it correctly we need to perform following steps:

Add a hook --> /snap/microk8s/current/hooks/connect-plug-images

#!/usr/bin/env bash
set -eux

ARCH="$($SNAP/bin/uname -m)"
export LD_LIBRARY_PATH="$SNAP/lib:$SNAP/usr/lib:$SNAP/lib/$ARCH-linux-gnu:$SNAP/usr/lib/$ARCH-linux-gnu"
export PATH="$SNAP/usr/sbin:$SNAP/usr/bin:$SNAP/sbin:$SNAP/bin:$PATH:/usr/bin:/usr/local/bin"

for image in $(ls $SNAP_COMMON/images/*.tar); do
    $SNAP/microk8s-ctr.wrapper image import $SNAP_COMMON/images/$image
done

Update /snap/microk8s/current/snapcraft.yaml file with following changes:

plugs:
  images:
    interface: content
    content: images
    target: $SNAP_COMMON/images

slots:
  microk8s:
    interface: content
    content: microk8s
    source:
      read: [$SNAP/.microk8s-info/microk8s]

save images under $SNAP_COMMON/images location for microk8s to look there first.

In this way will be be able to achieve offline installation of all such addons where container images are downloaded from internet?
Kindly suggest.

looks like it.  In order to change and add the hook, you will most likely have to build the snap itself.
Instructions on how to build MicroK8s here

Any other workaround like using ctr to load images locally & then using that as the first source instead of building snap altogether?

Yes using ctr to manually load the images is one way.  Like its mentioned in the docs.  That you don't need to rebuild microk8s.
For the gpu, microk8s is using the helm chart to install the gpu-operator.
You can find it here how its done. https://github.com/canonical/microk8s-core-addons/blob/main/addons/gpu/enable


Docs: change quotes for better reading

In the scripts, it works if using single quotes as well. This should improve the readability.

Coverage remained the same at 99.206% when pulling e151a6804f45e7b3f2ddf2c74b74369b4cfd0298 on JPeer264:patch-1 into 9842e3edf4792d57581685db34870e2cf9862091 on styled-components:master.


[DOC] Model Health Dashboard

What do you want to do?

[ ] Request a change to existing documentation
[ x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
This is a request to provide documentation for the ML model health dashboard.
What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
(github issue pending) https://github.com/opensearch-project/.github/commit/b426500f4a07bbebb79c4037329c21b6a23904c9

Is this a duplicate of https://github.com/opensearch-project/documentation-website/issues/2240?

Yes. Closing with comment


v2 has two permissions resources

permissions.js and user-permissions.js appear to be the same with user-permissions adding a session bust parameter.

core/api/permissions.js should be updated with new sdk api


No Show Button

Allow interviewers to hit a button for no-show

Fixed in https://github.com/BUUPE/Inquisitor/tree/improvements/3.0


envio: 0.5.0 -> 0.5.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/envio/versions.
meta.description for envio is: Modern and secure CLI tool for managing environment variables
meta.homepage for envio is: https://envio-cli.github.io/home
meta.changelog for envio is: https://github.com/envio-cli/envio/blob/0.5.1/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.1 with grep in /nix/store/4fhx0sb6fa0ljpbpivr0wplf0892n6rq-envio-0.5.1
found 0.5.1 in filename of file in /nix/store/4fhx0sb6fa0ljpbpivr0wplf0892n6rq-envio-0.5.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
envio

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4fhx0sb6fa0ljpbpivr0wplf0892n6rq-envio-0.5.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A envio https://github.com/r-ryantm/nixpkgs/archive/5764e7e457ba2b7bae730006e6e05c2bdd9a6151.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5764e7e457ba2b7bae730006e6e05c2bdd9a6151#envio

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4fhx0sb6fa0ljpbpivr0wplf0892n6rq-envio-0.5.1
ls -la /nix/store/4fhx0sb6fa0ljpbpivr0wplf0892n6rq-envio-0.5.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

envio

Maintainer pings
cc @afh for testing.

Result of nixpkgs-review pr 289463 run on aarch64-darwin 1

1 package built:

envio

Thanks for merging, @sikmir, much appreciated :rocket:


concentration parameter for document and topics distributions in ml_lda

I am not quite sure what is the default value of beta and alpha as I can not find it here: https://spark.apache.org/docs/latest/api/scala/index.html#org.apache.spark.ml.clustering.LocalLDAModel

One way to get Spark to spit out the default values for parameters (+ their documentation) is to call the explainParams method on the object; e.g.
lda <- invoke_new(sc, "org.apache.spark.ml.clustering.LDA")
params <- invoke(lda, "explainParams")
cat(params, sep = "\n")

For that, I see e.g.
> cat(params, sep = "\n")
checkpointInterval: set checkpoint interval (>= 1) or disable checkpoint (-1). E.g. 10 means that the cache will get checkpointed every 10 iterations (default: 10)
docConcentration: Concentration parameter (commonly named "alpha") for the prior placed on documents' distributions over topics ("theta"). (undefined)
featuresCol: features column name (default: features)
k: The number of topics (clusters) to infer. Must be > 1. (default: 10)
keepLastCheckpoint: (For EM optimizer) If using checkpointing, this indicates whether to keep the last checkpoint. If false, then the checkpoint will be deleted. Deleting the checkpoint can cause failures if a data partition is lost, so set this bit with care. (default: true)
learningDecay: (For online optimizer) Learning rate, set as an exponential decay rate. This should be between (0.5, 1.0] to guarantee asymptotic convergence. (default: 0.51)
learningOffset: (For online optimizer) A (positive) learning parameter that downweights early iterations. Larger values make early iterations count less. (default: 1024.0)
maxIter: maximum number of iterations (>= 0) (default: 20)
optimizeDocConcentration: (For online optimizer only, currently) Indicates whether the docConcentration (Dirichlet parameter for document-topic distribution) will be optimized during training. (default: true)
optimizer: Optimizer or inference algorithm used to estimate the LDA model. Supported: online, em (default: online)
seed: random seed (default:<PHONE_NUMBER>)
subsamplingRate: (For online optimizer) Fraction of the corpus to be sampled and used in each iteration of mini-batch gradient descent, in range (0, 1]. (default: 0.05)
topicConcentration: Concentration parameter (commonly named "beta" or "eta") for the prior placed on topic' distributions over terms. (undefined)
topicDistributionCol: Output column with estimates of the topic mixture distribution for each document (often called "theta" in the literature).  Returns a vector of zeros for an empty document. (default: topicDistribution)

...although apparently, in this case, the default values are undefined, so that doesn't really help! :)

Ok I've found the default valus for default EM optimization

topicConcentration: DoubleParam

Concentration parameter (commonly named "beta" or "eta") for the prior placed on topics' distributions over terms.
This is the parameter to a symmetric Dirichlet distribution.
Note: The topics' distributions over terms are called "beta" in the original LDA paper by Blei et al., but are called "phi" in many later papers such as Asuncion et al., 2009.
If not set by the user, then topicConcentration is set automatically. (default = automatic)
Optimizer-specific parameter settings:
EM
    Value should be > 1.0
    default = 0.1 + 1, where 0.1 gives a small amount of smoothing and +1 follows Asuncion et al. (2009), who recommend a +1 adjustment for EM.
Online
    Value should be >= 0
    default = (1.0 / k), following the implementation from https://github.com/Blei-Lab/onlineldavb.

Definition Classes
LDAParams
Annotations
@Since( "1.6.0" )

@kevinushey I've updated default values of beta and alpha to be the same as in Spark ML.
I recommend to merge #179 before this PR can be merged

reopened again in https://github.com/rstudio/sparklyr/pull/186 due to too many conflicts


Database connection

Status
READY
Documentation
[X] I have fully documented features in this PR's code. Please put an X in the box to confirm.
[ ] This PR does NOT require documentation.
Description
A few sentences describing the overall goals of the pull request's commits.
This PR allows for users to connect to the database through their environment variable.
Resolves Issue Number
#8
Deploy Notes
Notes regarding deployment the contained body of work.  These should note any
db migrations, etc.
git pull --prune
git checkout database-connection
dotnet restore
dotnet run

Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.

Make sure to recompile code, as well as restore the JSON file as dependencies are changed.
You must also make sure that your environment variable is set prior to compiling and running this PR.

Impacted Files in Application
List general components of the application that this PR will affect:

data/bangazonconnection.cs

LGTM 🐑 💨

LGTM


[SQL] A standalone python script that helps test SQL parsing for OpenLineage SQL library

Signed-off-by: howardyoo<EMAIL_ADDRESS>
Problem
A usual way to develop and test customer's custom extractor is by running those extractors on top of airflow and troubleshooting it. However, this introduces many inefficiencies if customers are simply trying to test how the sql parser package is parsing their SQL statements - and whether the parsing is working correctly.
Closes: #997
Solution
In order to better troubleshoot parsing of SQL statements, there is a need to have a utility that can
Parse the SQL statement only and derive the SQL metadata
check whether there hasn't been any errors during the parsing scheme, as well as check the generated metadata output to ensure proper data are extracted and present.
Therefore, I would like to propose a 'sql test' utility which can:
input any arbitrary SQL statement in string format.
generate sql metadata out of it.
print the content of the metadata to console.
This will greatly help reduce the development and test cycle of users who are either working with any SQL related extractors for openlineage.

Note: All schema changes require discussion. Please link the issue for context.

[ ] Your change modifies the core OpenLineage model
[ ] Your change modifies one or more OpenLineage facets

If you're contributing a new integration, please specify the scope of the integration and how/where it has been tested (e.g., Apache Spark integration supports S3 and GCS filesystem operations, tested with AWS EMR).
Checklist

[x] You've signed-off your work
[x] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[x] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] You've updated the CHANGELOG.md with details about your change under the "Unreleased" section (if relevant, depending on the change, this may not be necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[x] You've added a header to source files (if relevant)

Codecov Report

Merging #998 (424d75e) into main (abd3938) will increase coverage by 0.51%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #998      +/-   ##
============================================
+ Coverage     84.27%   84.79%   +0.51%
============================================
  Files            78       56      -22
  Lines          3517     3110     -407
  Branches         20        0      -20
============================================
- Hits           2964     2637     -327
+ Misses          525      473      -52
+ Partials         28        0      -28

Impacted Files
Coverage Δ

.../openlineage/client/transports/KafkaTransport.java

...openlineage/client/OpenLineageClientException.java

integration/dagster/openlineage/dagster/sensor.py

...ntegration/dagster/openlineage/dagster/__init__.py

...va/io/openlineage/client/transports/Transport.java

.../io/openlineage/client/transports/KafkaConfig.java

...o/openlineage/client/transports/HttpTransport.java

...in/java/io/openlineage/client/OpenLineageYaml.java

...penlineage/client/transports/TransportFactory.java

...a/src/main/java/io/openlineage/client/Clients.java

... and 12 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


AI Studio terraform templates

@grayzu @stemaMSFT @lonegunmanb can someone take a look at this pr?
cc: @deeikele

@grayzu changes have been made.

Looks good to me.

@grayzu when can this be merged?

@andyaviles121 please fix the issues with the PR. When it passes e2e tests we can merge.

@lonegunmanb looks like this doc is working on create and destroy and the retry is where it fails. Can you take a look?

@lonegunmanb have you had a chance to look over this pr? I'm not sure how to fix issues with the e2e check failure

@lonegunmanb is the issue due to the reused resource names? Seems like the key vault is having issues being created.
cc: @deeikele

@stemaMSFT can you try to run the e2e test again? I added in a section to the azurerm provider to hopefully help with the key vault issues.

@stemaMSFT Can you please help unblock this E2E build? This seems not triggerable on our side. We have several customers looking for these templates to be merged? Thanks!

Sorry for letting this slip for a few weeks. Just kicked off those E2E tests.

@andyaviles121 @deeikele @lonegunmanb we're still running into E2E test issues.

@andyaviles121 @lonegunmanb the e2e tests have failed again, could you take a look?

@stemaMSFT seemed like the connection had an incorrect field. I removed it. Can you try the e2e test again?

cc: @grayzu, @stemaMSFT. @lonegunmanb
@andyaviles121 This sample is failing in part because it doesn't adhere to the Contributor Guide requirements for Terraform samples. https://review.docs.microsoft.com/en-us/help/contribute/global-terraform-template?branch=main

@TomArcherMsft I rewrote the files to match the contributor guide. Can you review and run the e2e tests?

@andyaviles121 This looks good. Please change "output.tf" to "outputs.tf" and I'll run the e2e tests. Thanks.

@TomArcherMsft done :)

CC: @andyaviles121
@stemaMSFT @lonegunmanb I ran the e2e tests it looks as though everything passes now:
https://github.com/Azure/terraform/actions/runs/9616809065/job/26527170852?pr=322
Can this PR be merged now?


Issue: Handling killing properly with parallel's stdout buffering

On PBS, it seems to be polite and let parallel dump its buffered STDOUT if the job is killed.
In comparison, if the job fails (due to memory or walltime) then SGE isn't polite enough to let parallel print out its buffer, so we get no job output.
parallel has several options to disable or provide progressive buffering, we should investigate which is appropriate.

Second thought: parallel's buffering also effectively prevents anyone from following the progress of their jobs. This is bad news (tm)

--line-buffer
--lb
Buffer output on line basis. --group will keep the output together for a whole job. --ungroup allows output to mixup with half a line coming from one job and half a line coming from another job. --line-buffer fits between these two: GNU parallel will print a full line, but will allow for mixing lines of different jobs.
--line-buffer takes more CPU power than than both --group and --ungroup, but can be faster than --group if the CPU is not the limiting factor.
See also: --group --ungroup

See also

--ungroup
-u
Ungroup output. Output is printed as soon as possible and by passes GNU parallel internal processing. This may cause output from different commands to be mixed thus should only be used if you do not care about the output.

Fixed via configuring SGE to send a SIGTERM 60 seconds before SIGKILL to allow time for processes to "finish up"

Turns out this is a problem on scinet as well. Having empty log files is just unacceptable. It makes it impossible to debug anything.

On the one hand, I want to say that unbuffered output is going to be a dog's breakfast of madness and that users should handle logging themselves in each of their commands if they really want it done right... (i.e. redirection at the very least)
And on the other hand, a dog's breakfast is better than nothing. How about --ungroup then?

I'm okay with --line-buffer to keep lines separated

Okay, looks like the combination of --tag and --line-buffer works pretty well, example:
$ cat joblist
./mb_register.sh brain1_2.mnc brain1_2.mnc output
./mb_register.sh brain1_2.mnc brain2_2.mnc output
./mb_register.sh brain1_2.mnc brain3_2.mnc output
./mb_register.sh brain1_2.mnc brain4_2.mnc output
./mb_register.sh brain1_2.mnc brain5_2.mnc output
./mb_register.sh brain2_2.mnc brain1_2.mnc output
./mb_register.sh brain2_2.mnc brain2_2.mnc output
./mb_register.sh brain2_2.mnc brain3_2.mnc output
./mb_register.sh brain2_2.mnc brain4_2.mnc output
./mb_register.sh brain2_2.mnc brain5_2.mnc output
./mb_register.sh brain3_2.mnc brain1_2.mnc output
./mb_register.sh brain3_2.mnc brain2_2.mnc output
./mb_register.sh brain3_2.mnc brain3_2.mnc output
./mb_register.sh brain3_2.mnc brain4_2.mnc output
./mb_register.sh brain3_2.mnc brain5_2.mnc output
./mb_register.sh brain4_2.mnc brain1_2.mnc output
./mb_register.sh brain4_2.mnc brain2_2.mnc output
./mb_register.sh brain4_2.mnc brain3_2.mnc output
./mb_register.sh brain4_2.mnc brain4_2.mnc output
./mb_register.sh brain4_2.mnc brain5_2.mnc output
./mb_register.sh brain5_2.mnc brain1_2.mnc output
./mb_register.sh brain5_2.mnc brain2_2.mnc output
./mb_register.sh brain5_2.mnc brain3_2.mnc output
./mb_register.sh brain5_2.mnc brain4_2.mnc output
./mb_register.sh brain5_2.mnc brain5_2.mnc output

$ parallel --line-buffer < joblist
All_Command_lines_OK
Using double precision for computations.
All_Command_lines_OK
Using double precision for computations.
All_Command_lines_OK
Using double precision for computations.
All_Command_lines_OK
Using double precision for computations.
All_Command_lines_OK
Using double precision for computations.
All_Command_lines_OK
Using double precision for computations.
All_Command_lines_OK
Using double precision for computations.
All_Command_lines_OK
Using double precision for computations.
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain1_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
=============================================================================
  number of levels = 7
  fixed image: brain1_2.mnc
  moving image: brain1_2.mnc
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain3_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
=============================================================================
  number of levels = 7
  fixed image: brain3_2.mnc
  moving image: brain1_2.mnc
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain4_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
=============================================================================
  number of levels = 7
  fixed image: brain4_2.mnc
  moving image: brain1_2.mnc
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain2_2.mnc and moving image: brain2_2.mnc (type = Euler3DTransform)
=============================================================================
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain2_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
=============================================================================
  number of levels = 7
  fixed image: brain2_2.mnc
  moving image: brain1_2.mnc
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain5_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
=============================================================================
  number of levels = 7
  fixed image: brain5_2.mnc
  moving image: brain1_2.mnc
  number of levels = 7
  fixed image: brain2_2.mnc
  moving image: brain2_2.mnc
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain1_2.mnc and moving image: brain2_2.mnc (type = Euler3DTransform)
=============================================================================
  number of levels = 7
  fixed image: brain1_2.mnc
  moving image: brain2_2.mnc
=============================================================================
The composite transform comprises the following transforms (in order):
  1. Center of mass alignment using fixed image: brain3_2.mnc and moving image: brain2_2.mnc (type = Euler3DTransform)
=============================================================================
-------------CLIP-------------------

parallel --line-buffer --tag < joblist
./mb_register.sh brain1_2.mnc brain4_2.mnc output	All_Command_lines_OK
./mb_register.sh brain1_2.mnc brain4_2.mnc output	Using double precision for computations.
./mb_register.sh brain1_2.mnc brain1_2.mnc output	All_Command_lines_OK
./mb_register.sh brain1_2.mnc brain1_2.mnc output	Using double precision for computations.
./mb_register.sh brain1_2.mnc brain3_2.mnc output	All_Command_lines_OK
./mb_register.sh brain1_2.mnc brain3_2.mnc output	Using double precision for computations.
./mb_register.sh brain1_2.mnc brain2_2.mnc output	All_Command_lines_OK
./mb_register.sh brain1_2.mnc brain2_2.mnc output	Using double precision for computations.
./mb_register.sh brain1_2.mnc brain5_2.mnc output	All_Command_lines_OK
./mb_register.sh brain1_2.mnc brain5_2.mnc output	Using double precision for computations.
./mb_register.sh brain2_2.mnc brain1_2.mnc output	All_Command_lines_OK
./mb_register.sh brain2_2.mnc brain1_2.mnc output	Using double precision for computations.
./mb_register.sh brain2_2.mnc brain3_2.mnc output	All_Command_lines_OK
./mb_register.sh brain2_2.mnc brain3_2.mnc output	Using double precision for computations.
./mb_register.sh brain2_2.mnc brain2_2.mnc output	All_Command_lines_OK
./mb_register.sh brain2_2.mnc brain2_2.mnc output	Using double precision for computations.
./mb_register.sh brain2_2.mnc brain1_2.mnc output	=============================================================================
./mb_register.sh brain2_2.mnc brain1_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain2_2.mnc brain1_2.mnc output	  1. Center of mass alignment using fixed image: brain1_2.mnc and moving image: brain2_2.mnc (type = Euler3DTransform)
./mb_register.sh brain2_2.mnc brain1_2.mnc output	=============================================================================
./mb_register.sh brain2_2.mnc brain1_2.mnc output	  number of levels = 7
./mb_register.sh brain2_2.mnc brain1_2.mnc output	  fixed image: brain1_2.mnc
./mb_register.sh brain2_2.mnc brain1_2.mnc output	  moving image: brain2_2.mnc
./mb_register.sh brain1_2.mnc brain2_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain2_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain1_2.mnc brain2_2.mnc output	  1. Center of mass alignment using fixed image: brain2_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
./mb_register.sh brain1_2.mnc brain2_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain2_2.mnc output	  number of levels = 7
./mb_register.sh brain1_2.mnc brain2_2.mnc output	  fixed image: brain2_2.mnc
./mb_register.sh brain1_2.mnc brain2_2.mnc output	  moving image: brain1_2.mnc
./mb_register.sh brain2_2.mnc brain3_2.mnc output	=============================================================================
./mb_register.sh brain2_2.mnc brain3_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain2_2.mnc brain3_2.mnc output	  1. Center of mass alignment using fixed image: brain3_2.mnc and moving image: brain2_2.mnc (type = Euler3DTransform)
./mb_register.sh brain2_2.mnc brain3_2.mnc output	=============================================================================
./mb_register.sh brain2_2.mnc brain3_2.mnc output	  number of levels = 7
./mb_register.sh brain2_2.mnc brain3_2.mnc output	  fixed image: brain3_2.mnc
./mb_register.sh brain2_2.mnc brain3_2.mnc output	  moving image: brain2_2.mnc
./mb_register.sh brain2_2.mnc brain2_2.mnc output	=============================================================================
./mb_register.sh brain2_2.mnc brain2_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain2_2.mnc brain2_2.mnc output	  1. Center of mass alignment using fixed image: brain2_2.mnc and moving image: brain2_2.mnc (type = Euler3DTransform)
./mb_register.sh brain2_2.mnc brain2_2.mnc output	=============================================================================
./mb_register.sh brain2_2.mnc brain2_2.mnc output	  number of levels = 7
./mb_register.sh brain2_2.mnc brain2_2.mnc output	  fixed image: brain2_2.mnc
./mb_register.sh brain2_2.mnc brain2_2.mnc output	  moving image: brain2_2.mnc
./mb_register.sh brain1_2.mnc brain5_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain5_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain1_2.mnc brain5_2.mnc output	  1. Center of mass alignment using fixed image: brain5_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
./mb_register.sh brain1_2.mnc brain5_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain5_2.mnc output	  number of levels = 7
./mb_register.sh brain1_2.mnc brain5_2.mnc output	  fixed image: brain5_2.mnc
./mb_register.sh brain1_2.mnc brain5_2.mnc output	  moving image: brain1_2.mnc
./mb_register.sh brain1_2.mnc brain1_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain1_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain1_2.mnc brain1_2.mnc output	  1. Center of mass alignment using fixed image: brain1_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
./mb_register.sh brain1_2.mnc brain1_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain1_2.mnc output	  number of levels = 7
./mb_register.sh brain1_2.mnc brain1_2.mnc output	  fixed image: brain1_2.mnc
./mb_register.sh brain1_2.mnc brain1_2.mnc output	  moving image: brain1_2.mnc
./mb_register.sh brain1_2.mnc brain4_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain4_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain1_2.mnc brain4_2.mnc output	  1. Center of mass alignment using fixed image: brain4_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
./mb_register.sh brain1_2.mnc brain4_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain4_2.mnc output	  number of levels = 7
./mb_register.sh brain1_2.mnc brain4_2.mnc output	  fixed image: brain4_2.mnc
./mb_register.sh brain1_2.mnc brain4_2.mnc output	  moving image: brain1_2.mnc
./mb_register.sh brain1_2.mnc brain3_2.mnc output	=============================================================================
./mb_register.sh brain1_2.mnc brain3_2.mnc output	The composite transform comprises the following transforms (in order):
./mb_register.sh brain1_2.mnc brain3_2.mnc output	  1. Center of mass alignment using fixed image: brain3_2.mnc and moving image: brain1_2.mnc (type = Euler3DTransform)
./mb_register.sh brain1_2.mnc brain3_2.mnc output	=============================================================================
-----CLIP------

Solved


Add appdynamics-org to the list of default protected orgs

This prevents confusion when appd stops working every so often. If the org is deleted, no attached services instances can be deleted either.

Thanks @matthewfischer


Extenison package.json badges: Incorrect type. Expected "object".

Versions:
Version: 1.73.0-insider (Universal)
Commit: 3544aabd5a2b0da2cd3a6f3fa3f00759aa9d9709
Date: 2022-10-27T21:41:11.954Z
Electron: 19.0.17
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 22.1.0
Sandboxed: Yes

Looks like badges is now also defined in the standard schema for package.json, but with an object as type:
https://github.com/SchemaStore/schemastore/blob/cfe02821ecef951790bf387a726c3c7183daf485/src/schemas/json/package.json#L767
https://github.com/SchemaStore/schemastore/blob/master/src/schemas/json/npm-badges.json
conflicting with the badges we support for VS Code packages (array)
We were first, but I guess that doesn't help ;-)

Hopefully https://github.com/SchemaStore/schemastore/issues/2582 will be accepted.

https://github.com/SchemaStore/schemastore/issues/2582 was accepted


QC flag variables - warn/error/refuse if trying to add invalid flag data values

Add an option (or default) to error if a user tries to add (for example) a flag value of 5 when only 0-4 are defined.

Addressed in commit https://github.com/joshua-hampton/ncas_amof_netcdf_template/commit/510ecbb974381e5907a59df0cb967820d66c46f2


Update flavor to depend on cloud controller

Can you help me understand what this PR does? While testing it:

I could delete the cloud while a flavor was in place
the creation of a flavor when its cloud doesn't exist triggered the usual error in the controller


Update Marathon to 0.9.0-RC3

Attention: Wait for the docker build to actually finish and test this. I haven't tested this yet!

I changed it to a new directory.
It is still untested and I have not added any of the new command line arguments.

Ok, updated it again. Added package.json and config.json. They are both copies of the files in the 0 directory. I just changed the version number in package.json.

Now I also updated the index.json via running ./scripts/build.sh

Connor: "Ship that shit"


QG checks (Release 24.08)

QG checks
Please open and fill in this issue in your product repository to document the compliance with our Tractus-X Release Guideline (TRGs)
Show compliance with TRGs by referencing to a tagged link in the respective repository where possible, example: TRG 1.01 (see github.com/eclipse-tractusx/example-repo/tree/1.0.0/README.md)
Close this issue once the compliance with the TRGs has been documented
Committer(s):
Helm Chart Version:  1.5.9
App Version:  1.5.6
Release Management Reference Issue:  https://github.com/eclipse-tractusx/sig-release/issues/743
Check of Tractus-X Release Guidelines

Currently implemented automatic checks can be found under your product on our Release Guidelines Checks Board
This QG Check is depending on the mandatory information from our current Release Guidelines

TRG 1 Documentation

[x] TRG 1.01 appropriate README.md
[x] TRG 1.02 appropriate install instructions either INSTALL.md or in README.md
[x] TRG 1.03 appropriate CHANGELOG.md
[x] TRG 1.04 editable static files
[x] TRG 1.05 architecture docs
[x] TRG 1.06 administrator guide
[x] TRG 1.07 user manual
[x] TRG 1.08 open api docs

TRG 2 Git

[x] TRG 2.01 default branch is named main
[x] TRG 2.03 repository structure
[x] TRG 2.04 leading product repository
[x] TRG 2.05 .tractusx metafile in a proper format
[x] TRG 2.06 Dependabot

TRG 3 Kubernetes

[x] TRG 3.02 persistent volume and persistent volume claim or database dependency (subchart) are in place when needed

TRG 4 Container

[x] TRG 4.01 semantic versioning and tagging
[x] TRG 4.02 base image is agreed
[x] TRG 4.03 image has USER command and Non Root Container
[x] TRG 4.05 released image must be placed in DockerHub, remove GHCR references
[x] TRG 4.06 separate notice file for DockerHub has all necessary information
[x] TRG 4.07 root file system is set to read access by default, but can be overwritten by the user

TRG 5 Helm

[x] TRG 5.01 Helm chart requirements
[x] TRG 5.02 Helm chart location in /charts directory and correct structure
[x] TRG 5.03 proper version strategy
[x] TRG 5.04 CPU / MEM resource requests and limits and are properly set
[x] TRG 5.06 Application must be configurable through the Helm chart
[x] TRG 5.07 Dependencies are present and properly configured in the Chart.yaml
[x] TRG 5.08 Product has a single deployable helm chart that contains all components
[x] TRG 5.09 Helm Test running properly
[x] TRG 5.10 Products need to support 3 versions at a time
[x] TRG 5.11 Upgradeability

TRG 6 Released Helm Chart

[x] TRG 6.01 Released Helm Chart

TRG 7 Open Source Governance

[x] TRG 7.01 Legal Documentation
[x] TRG 7.02 License and copyright header
[x] TRG 7.03 IP checks for project content
[x] TRG 7.04 IP checks for 3rd party content
[x] TRG 7.05 Legal information for distributions
[x] TRG 7.06 Legal information for end user content
[x] TRG 7.07 Legal notice for documentation (non-code)
[x] TRG 7.08 Legal notice for KIT documentation

TRG 8 Security

[x] TRG 8.01 Mitigate high and above findings in CodeQL
[x] TRG 8.02 Mitigate high and above findings in KICS
[x] TRG 8.04 Mitigate high and above findings in Trivy
[x] TRG 8.03 No secret findings by GitGuardian or TruffleHog

TRG 9 UX/UI Styleguide

[ ] TRG 9.01 UI consistency/styleguide for UI

Hints
Information Sharing

I approve the QG with the following version numbers:
Helm Chart Version: 1.5.9
App Version: 1.5.6


Hello I'm installed your app and I can connect but service show anything only stay on continue watching all other is black. I like to crunchy roll on tizen any idea what's going on. After an relog I can see the last series I'm watching on m, phone but I can't starting.  I'll pick it back up and complete it.


Use role_arn in S3 backend

Hi,
I'm using cross-account roles with clients.
I can assume a role with terraform s3 backend config but terragrunt seems to ignore the role_arn  parameter.

Ah, you're right. Looks like Terragrunt supports these settings: https://github.com/gruntwork-io/terragrunt/blob/master/remote/remote_state_s3.go#L17-L24. But the S3 backend now supports quite a few more: https://www.terraform.io/docs/backends/types/s3.html#configuration-variables. PRs very welcome!

BTW, as a workaround, you can assume an IAM role using env vars plus the AWS CLI. You call assume-role and set the returned AccessKeyId, SecretAccessKey, and SessionToken as the env vars AWS_ACCESS_KEY_ID, AWS_SECRET_ACCESS_KEY, and AWS_SESSION_TOKEN, respectively. Both Terraform and Terragrunt would pick those up automatically.

That's what I'm doing actually but it's a bit overwhelming.
I'm not a go developer (yet) so I hope someone will find the time to do it :)

@qrevel I'm having a look at this as well, need it for cross-account buckets using assume roles. Will do a PR if I get it working.


Current games: switch between mini and user games API when the number of user ids is larger than a threshold

This might optimize the speed of the pages as well as use less server resources.
Always use mini when 429 errors start coming.

The API doesn't work as it should, see https://github.com/lichess-org/lila/issues/14280

This is already implemented.


BUG: Enable filtering group for Windows Python CI

WIP attempt to address #3057

PR Checklist

[ ] No API changes were made (or the changes have been approved)
[ ] No major design changes were made (or the changes have been approved)
[ ] Added test (or behavior not changed)
[ ] Updated API documentation (or API not changed)
[ ] Added license to new files (if any)
[ ] Added Python wrapping to new files (if any) as described in ITK Software Guide Section 9.5
[ ] Added ITK examples for all new major features (if any)

Refer to the ITK Software Guide for
further development details if necessary.

@dzenanz I just noticed that Windows Python is not in the list of CI -- should we be running that for each push?

We recently removed it from the list, because it was going over the Azure's time limit (6h). This test failure was a rare instance where that was a problem.

We recently removed it from the list, because it was going over the Azure's time limit (6h). This test failure was a rare instance where that was a problem.

It looks like it is still running for updates to master though. Was it only running over the time limit when run against unmerged PRs?
Would it be reasonable to try re-enable it here to assist in testing a possible fix, then disable it again before marking the PR as ready? I'm having trouble finding the list of pipelines to run, is that set somewhere in .github or Testing?

I'm still working on rebuilding on my side, having issues with ITKPyBasePython not being found for import despite its .pyd binary being located at the correct path. I think this is an issue with a CMake setting on my side and not directly related to the FFT problem tracked here.
>>> import itk
>>> itk.force_load()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\repos\ITK-build-unstable\Wrapping\Generators\Python\itk\support\init_helpers.py", line 116, in force_load
    getattr(itk, k)
  File "C:\repos\ITK-build-unstable\Wrapping\Generators\Python\itk\support\lazy.py", line 137, in __getattribute__
    base.itk_load_swig_module(module, namespace)
  File "C:\repos\ITK-build-unstable\Wrapping\Generators\Python\itk\support\base.py", line 102, in itk_load_swig_module
    itk_load_swig_module(dep, namespace)
  File "C:\repos\ITK-build-unstable\Wrapping\Generators\Python\itk\support\base.py", line 110, in itk_load_swig_module
    l_module = loader.load(swig_module_name)
  File "C:\repos\ITK-build-unstable\Wrapping\Generators\Python\itk\support\base.py", line 259, in load
    l_spec.loader.exec_module(l_module)  # pytype: disable=attribute-error
  File "<frozen importlib._bootstrap_external>", line 783, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "C:\repos\ITK-build-unstable\Wrapping\Generators\Python\itk\support\..\ITKPyBasePython.py", line 13, in <module>
    from . import _ITKPyBasePython
ImportError: DLL load failed while importing _ITKPyBasePython: The specified module could not be found.

It could be that it was exceeding the time limit only occasionally, but that was enough trouble to disable it. I think it needs to be enabled somewhere in Azure configuration, not by a change in this repository.

The tests for merge to master might be doing an incremental build, so that's why they were not running over the time limit. I think it might be easier to merge this PR to see whether it resolves the problem, then to search through Azure configurations and fiddle with that.

Even if the change here is small, testing by merging to master seems like it could get really messy, especially if the problem is not resolved and we need to try more small changes.
Let me try a little longer to build locally with the test's CMake configuration, if I haven't made progress we can revisit tomorrow?

Reading back through a failed run log, it actually looks like ITKFFT is already enabled as a dependency, so this change will have no effect.
2022-01-10T15:57:22.5006457Z -- Enabled ITKFFT, needed by [ITKTestKernel].

Back to the drawing board...

I believe I've identified the issue as a cascading dependency failure, where ITKImageIntensity was implicitly required by the Python FFT test but not built accordingly. I am still resolving an issue with building ITK Python modules on my local Windows machine and cannot yet verify that this resolves the Windows CI issue. However, if all other CI passes then these minor changes are not expected to break anything and the best path forward may be to go ahead and merge, then check incremental Windows CI from master.

Your analysis in the issue description instills confidence in this solution.


Extension issue

Issue Type: Bug
Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.18363
VSCode version: 1.52.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:

https://github.com/ritwickdey/vscode-live-server/issues/new/?body=- Issue Type: Bug

Extension Name: LiveServer
Extension Version: 5.6.1
OS Version: Windows_NT x64 10.0.18363
VSCode version: 1.52.1

:warning: We have written the needed data into your clipboard. Please paste! :warning:&title=Extension issue


Wrong API user is defined - 13-kube-apiserver-to-kubelet

Hello,
I see the user to which we are binding the ClusterRole 'system:kube-apiserver-to-kubelet' is 'system:kube-apiserver'. But the user defined in the '--kubelet-client-certificate' of API server is different i.e 'kube-apiserver'. This is the user for which the certificate was generated in the earlier lectures.
This causes the API server authorization to kubelet fail while running 'kubectl exec' against any pods.
vagrant@master-1:~$ kubectl exec -ti busybox -- nslookup kubernetes
error: unable to upgrade connection: Forbidden (user=kube-apiserver, verb=create, resource=nodes, subresource=proxy)
vagrant@master-1:~$
Please update the document to update the correct user in the clusterrolebindings.

Hello,
I see the user to which we are binding the ClusterRole 'system:kube-apiserver-to-kubelet' is 'system:kube-apiserver'. But the user defined in the '--kubelet-client-certificate' of API server is different i.e 'kube-apiserver'. This is the user for which the certificate was generated in the earlier lectures.
This causes the API server authorization to kubelet fail while running 'kubectl exec' against any pods.
vagrant@master-1:~$ kubectl exec -ti busybox -- nslookup kubernetes error: unable to upgrade connection: Forbidden (user=kube-apiserver, verb=create, resource=nodes, subresource=proxy) vagrant@master-1:~$
Please update the document to update the correct user in the clusterrolebindings.

Thanks, but If I want to keep the name of user in "13-kube-apiserver-to-kubelet.md" as it is. Which part should I change in earlier lectures so it will be similar with the name in -13-kube-apiserver-to-kubelet.md- if I want to start over from beginning?

Hi @kareotoko
We have now merged a major change to bring this to v1.24.
This has been addressed as part of the upgrade. Please try it now and feel free to raise further issues if you find them.


[Feature Request] Update PettingZoo version

Eventually, feature requests will be treated as a scrum board for open-source contributors. At the current scale, you should come chat with us on the Discord #development channel before writing one of these.
Hi, would it be possible to update this repo to use the most recent version of PettingZoo? This project is listed in PettingZoo's third-party-environments, but we only want to include environments which work with the current version.
If you need any help working out issues due to different versions feel free to ask, there were some breaking changes in version 1.2, so it requires a bit of code changes to adapt. The previous API returned done in the step() function, whereas the new one returns truncated and terminated (matching gymnasium). There is a migration guide for gymnasium explaining the changes further, the steps should be basically the same (we're working on making resources for updating old PettingZoo repositories as well): https://gymnasium.farama.org/content/migration-guide/
I am trying to: Use NeuralMMO with current PettingZoo/Gymnasium versions
**It is hard/impossible because:" There have been breaking changes since the version used (v1.19.0 of PettingZoo)
The solution should look like: The internal logic needs to switch to using terminated and truncated instead of done.

Hi,
Working on this. It is difficult. There are tons of breaking changes between 0.19 and gymnasium. Supporting Gymnasium requires me to first update PufferLib not only to support the latest version, but also all of the intermediate ones too. Once that is done, I can update NMMO as well.

Great to hear, if you need any help with things related specifically to PettingZoo and/or Gymnasium let me know--would love to help however possible. Gymnasium is nearing its 1.0.0 release and will have an alpha release prior to that FYI, mainly improvements and new features but may be worth double checking to see that there isn't anything that would make things difficult to update.
We've been working on adding training tutorials and more resources for beginners for Gymnasium and PettingZoo, would love to promote any sort of updated starting scripts/tutorials for NeuralMMO to new users asking in our discord and such

Hi Elliot, Neural MMO 2.1 has moved to pettingzoo==1.24.1 and gymnasium==0.29.1, we are currently testing it.
The 2.1 passes pettingzoo's parallel_api_test --
https://github.com/NeuralMMO/environment/blob/2.1/tests/test_pettingzoo.py
Thanks!


consolidate AssetBundle::entries and AssetBundle::entryKinds into a new type, AssetBundleEntry

Part of work on https://github.com/flutter/flutter/pull/141194
The AssetBundle class contains two members, entries and entryKinds. entries contains asset data indexed by asset key. entryKinds contains the "kinds" of these assets, again indexed by asset key.
Change. Rather than have two separate maps, this PR proposes combining these maps into one by wrapping the asset data and kind into a single data type AssetBundleEntry.
Purpose. In https://github.com/flutter/flutter/pull/141194, I am considering associating more information with an asset. In particular, what transformers are meant to be applied to it when copying it to the build output. Rather than adding another map member onto AssetBundle (e.g. entryTransformers), I decided to make things neater by introducing the AssetBundleEntry type.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Also, this is a great refactor!


'AGE' column for istioctl

Replaces https://github.com/istio/istio/pull/6013 which I couldn't convince to pass circle tests.
Created using
git checkout master
git checkout -b istioctl_age_v3
git cherry-pick ce78830517b47e961bcdb23a9a8ddd41678ff52f
rm -rf ~/go/pkg/dep/
dep ensure --add k8s.io/apimachinery/pkg/util/duration
git add Gopkg.lock Gopkg.toml vendor
git commit -m "Add dependency to k8s.io/apimachinery/pkg/util/duration"
git push --set-upstream origin istioctl_age_v3

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

Needs approval from an approver in each of these files:

OWNERS

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

@esnible: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
18692a051a265839e2af3cf5cc5e8580bc63df5d
link
/test istio-unit-tests

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

@esnible: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
18692a051a265839e2af3cf5cc5e8580bc63df5d
link
/test istio-unit-tests

prow/istio-presubmit.sh
18692a051a265839e2af3cf5cc5e8580bc63df5d
link
/test istio-presubmit

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

Giving up on this variant.


Minestom Support

It's time for it... Minestom LuckPerms. It's made by LeoDog896 (He allowed for me to create pr), i just updated it to latest minestom. This use minestom permission system and doesn't override hasPermission method, because minestom currently doesn't allow to do this (it's possible to add PlayerProvider, but most of servers will use custom PlayerProvider). Sorry for my bad English btw.

I haven't worked on Minestom in a while, but I am absolutely willing to help smoothen the process on adding Minestom to LuckPerms. I think I talked about this with lucko quite a while ago, and since then the Minestom API is much more stable.

I completed this, i used some classes from dev-hydrogen's luckperms. Currently to update permissions for player - player has to rejoin. If you tell me how to listen to update permissions event, i can open new pr (if you merge this).

Ignore higher comment, i fixed this, permissions have worked, but i forgot to call player.refreshCommands().

I dont exactly appreciate that my commits are being used in this PR in another persons name. This pull request should just include the commits directly from the project-cepi (as well as my) repositories.
That also ignores the issue that the way this works is at best untested, and at worst unstable. How i made my fork work, is that it (tries) to sync up permissions between luckperms and minestom when the player join, and then listens to the luckperms api to sync any changes that happen after that. This approach doesnt allow for any nice-to-haves like wildcard support, as that would mean needing to override hasPermission, which currently is not possible on minestom.
Theres a reason i never made a pull request from my fork, I don't trust it enough to be used anywhere other than on my own servers.

I dont exactly appreciate the fact that my commits are being used in this PR in another persons name, as if this was somehow merged, i'd like the small piece of credit. This pull request should just include the commits directly from the project-cepi (as well as my) repositories.
That also ignores the issue that the way this works is at best untested, and at worst unstable. How i made my fork work, is that it (tries) to sync up permissions between luckperms and minestom when the player join, and then listens to the luckperms api to sync any changes that happen after that. This approach doesnt allow for any nice-to-haves like wildcard support, as that would mean needing to override hasPermission, which currently is not possible on minestom.
Theres a reason i never made a pull request from my fork, I don't trust it enough to be used anywhere other than on my own servers.

I agree somewhat, but I tested today and had no issues after all the commits.

Then if you merge it - credit all Contributors (Me (I have did small changes too), LeoDog, dev-hydrogen)

You should redo the PR with a proper git history then -- commits from me and hydrogen instead of just you.

Because hydrogen wanna credit - #3521


Tests don't like the last line to be a comment

I had commented out the last of my tests and got "There was an error evaluating the test script. Unexpected end of input" and no tests ran. Removing/moving the commented test to the top of the list got rid of this.

This has been fixed in the latest release.


Convert interface nil fix

@catplant Thanks for the report and PR https://github.com/mattn/anko/pull/283

Codecov Report

Merging #284 into master will decrease coverage by 0.17%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##           master     #284      +/-   ##
==========================================
- Coverage   83.54%   83.37%   -0.18%
==========================================
  Files           8        8
  Lines        1939     1937       -2
==========================================
- Hits         1620     1615       -5
- Misses        240      241       +1
- Partials       79       81       +2

Impacted Files
Coverage Δ

vm/vmConvertToX.go
85.98% <100%> (-1.64%)
:arrow_down:

vm/vmLetExpr.go
84.18% <60%> (-0.91%)
:arrow_down:

Continue to review full report at Codecov.

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

@mattn Good to go

Thanks


v1.7.0 is broken on Python 2

When v1.7.0 got released today, my CI builds started to fail with:
Traceback (most recent call last):
  File "/usr/local/bin/mbed", line 7, in <module>
    from mbed.mbed import main
  File "/usr/local/lib/python2.7/dist-packages/mbed/mbed.py", line 22, in <module>
    from future.builtins.iterators import zip
ImportError: No module named future.builtins.iterators

Dockerfile to replicate the issue:
FROM debian:stretch-slim
RUN apt update && \
    apt install -y lbzip2 ca-certificates wget python-pip git mercurial --no-install-suggests && \
    rm -rf /var/lib/apt/lists/*
RUN pip install mbed-cli
RUN mbed config -G GCC_ARM_PATH "/usr/bin"

Strangely, Python 3 also doesn't seem to work:
Traceback (most recent call last):
  File "/usr/local/bin/mbed", line 7, in <module>
    from mbed.mbed import main
  File "/usr/local/lib/python3.5/dist-packages/mbed/mbed.py", line 22, in <module>
    from future.builtins.iterators import zip
ImportError: No module named 'future'

FROM debian:stretch-slim
RUN apt update && \
    apt install -y lbzip2 ca-certificates wget python3-pip git mercurial --no-install-suggests && \
    rm -rf /var/lib/apt/lists/*
RUN pip3 install mbed-cli
RUN mbed config -G GCC_ARM_PATH "/usr/bin"

@amq It looks like we're depending on the future package in pip, but it was not added to the dependencies in setup.py.

As a workaround you should be able to pip install future.

Installing future fixes the problem.

There is a different problem when building:
Traceback (most recent call last):
  File "/home/amq/project/mbed-os/tools/make.py", line 41, in <module>
    from tools.tests import TESTS, Test, TEST_MAP
  File "/home/amq/project/mbed-os/tools/tests.py", line 18, in <module>
    from tools.data.support import DEFAULT_SUPPORT, CORTEX_ARM_SUPPORT
  File "/home/amq/project/mbed-os/tools/data/support.py", line 17, in <module>
    from tools.targets import TARGETS
  File "/home/amq/project/mbed-os/tools/targets/__init__.py", line 555, in <module>
    update_target_data()
  File "/home/amq/project/mbed-os/tools/targets/__init__.py", line 544, in update_target_data
    in Target.get_json_target_data().items()
  File "/home/amq/project/mbed-os/tools/targets/__init__.py", line 70, in wrapper
    CACHES[(func.__name__, args)] = func(*args, **kwargs)
  File "/home/amq/project/mbed-os/tools/targets/__init__.py", line 142, in get_json_target_data
    Target.__targets_json_location_default)
  File "/home/amq/project/mbed-os/tools/utils.py", line 351, in json_file_to_dict
    object_pairs_hook=OrderedDict)
  File "/usr/lib/python3.5/json/__init__.py", line 312, in loads
    s.__class__.__name__))
TypeError: the JSON object must be str, not 'bytes'
[mbed] ERROR: "/usr/bin/python3" returned error code 1.
[mbed] ERROR: Command "/usr/bin/python3 -u /home/amq/project/mbed-os/tools/make.py -t GCC_ARM -m REDACTED --source ./bootloader --source ./mbed-os --source ./lib/flashiap-driver --build ./BUILD/REDACTED/GCC_ARM -c" in "/home/amq/project"

cc @cmonr

@amq Thanks for the bug report. I've immediately reproduced the issue by removing future. I've also reissued a hotfix - #678 and published the Mbed CLI 1.7.1 release. Could you please update your Mbed CLI to 1.7.1 and let me know whether Python 2 is still broken for you?

Python 2 is fine now.
Python 3 problem remains.

@amq Interesting. Looks like there are still some tools in mbed-os that I missed.
I'll fix the issue and make a commit in the mbed-os repo, and ask if you can verify the fix.

Closing this here, and moving this to https://github.com/ARMmbed/mbed-os/issues/7026 since the issue is now focused around an mbed-os tool.


[BUG] Compilation error when using real decorators

Describe the bug
When using a decorator either on fields, methods, accessors or classes, the compilation fails with TypeError: Cannot read properties of undefined (reading 'kind') here.
Of course, this only happens when using ts-macros (with ttypescript).
Code to reproduce
The following snippet is enough to reproduce the bug on the playground (you don't have to use ts-macro at all):
declare function decorator(...args: any[]): any

class Foo {
    @decorator // <--- comment out to make it work
    get bar() {
        return 0;
    }
}

Expected behavior
It should compile!
Additional context
I believe this is ts-macro  specific-issue - I already use ttypescript with another transformer in a repo of mine and it works fine.

Thanks for the bug report! The next version of ts-macros will be released as soon as ttypescript gets updated to support typescript 5.0!


Support Dart-Define

Support Dart-Define to dynamically set port or part of path(token、key...).
flutter run --dart-define=port=1234

@RestApi(baseUrl: 'https://xxxxxx')
abstract class ApiClient {
  @POST(":${String.fromEnvironment('port')}/api/v1/login")
  Future<LoginRes> login(@Body() LoginReq req);
}

I don't want to use the injectable package because it depended on the get_it state management library.

How about using the baseUrl as a parameter?
You can pass any kind of URLs then.
abstract class RestClient {
  factory RestClient(Dio dio, {String baseUrl}) = _RestClient;
}

Thanks for the response,
baseUrl is fixed in my project,
but the port or part of the path(token、key...) is variable,
Is there a way to do this dynamically?

use your baseUrl + port.  as new baseUrl, then pass it to RestClient
For other scenarios, please use dio interceptors.

Ok, thanks for replying.


Create subscribe fragment

Is this a BUG REPORT or FEATURE REQUEST?:
feature
What happened:
No subscribe fragment.
What you expected to happen:
Subscribe fragment.
How to reproduce it (as minimally and precisely as possible):
Look for subscribe fragment.
Anything else we need to know?:
Use convertkit as default.
Environment:

Syna Theme version:  v0.7.0
Hugo version:  v0.30.2

Reusing the embed fragment via iframes. Not perfect, but works best considering CORS issues etc.


Cognitive - Form Recognizer for JavaScript (January-June 2021)

Azure Form Recognizer is a document recognition service that enables customers to extract data from images of forms (such as receipts, invoices, etc.). It supports several pre-built models for common document types as well as custom document models that the customer trains themselves. This enables the processing of documents and entry of their data into databases at scale.
The @azure/ai-form-recognizer package provides the SDK for JavaScript & TypeScript developers to write applications that use Azure Form Recognizer. Version 3.x of this package adds support for features released in the service API version 2.0 and later, with version 3.1.0 bringing support for new custom model features and prebuilt models in service API version 2.1.
This issue tracks the work we have planned for this package for the duration of January - June 2021
New Features

Add support for new prebuilt document models:

Identity Documents

Support new custom model features

Recognition and extraction of object structures (aka. structured tables)

Quality Improvements

Implement performance testing to defend against performance regressions
Refactors to testing and sample code to utilize new autorest-layer features (known-value enums)
Improvements to documentation rendering

Between Jan - June of 2021, we shipped multiple betas for version 3.1.0 of the JS SDK for Form Recognizer.
As of June 2021, version 3.1.0 of @azure/ai-form-recognizer package has moved out from beta to stable status


react 16 TypeError: _React$PropTypes is undefined

hello
i am getting the bellow error when i import Form.
TypeError: _React$PropTypes is undefined

@hannanstd hi, you solved that?

https://reactjs.org/docs/typechecking-with-proptypes.html

Note:
React.PropTypes has moved into a different package since React v15.5. Please use the prop-types > library instead.


Centering dialog option is scaling with zoom

The center dialog option won't work properly if zoom is set to something different than 100%. If set to 150%, it will go towards the lower right, and lower than 100% will go towards higher left.
Also, I feel the center setting should be by default, cursor is basically impractical all the time for me.

0.6.3 (2020-03-15)

fix dialog spawns not centered if zoomed (#315)