feat: allow to run a subset of pr_tester

- currently run default 8 11 17 18 19 takes a lot resources
- allow to use "run tests quick" to only run build on 19
- this still generates all jobs as a basic check

test https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/openjdk-build-pr-tester/1719/console

run tests quick

@zdtsw wondering if we could add the version to run on maybe? eg.
run tests quick 18

run tests quick 11

run tests quick 9,19

Setting status of 8491ead90366bff610aeda3216d17de6104624d0 to PENDING with url https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/openjdk-build-pr-tester/1720/ and message: 'Building & Testing PR...'
Using context: pipeline-build-check
Obtained pipelines/build/prTester/kick_off_tester.groovy from git https://github.com/andrew-m-leonard/ci-jenkins-pipelines.git
[Pipeline] Start of Pipeline
[Pipeline] node
Running on [C3jenkins](https://ci.adoptopenjdk.net/computer/C3jenkins/) in /home/jenkins/.jenkins/workspace/build-scripts-pr-tester/openjdk-build-pr-tester
not sure why it tries to git clone ci-jenkins-pipeline.git from your repo? @andrew-m-leonard

GitHub pull request #365 of commit 8491ead90366bff610aeda3216d17de6104624d0, no merge conflicts.
Setting status of 8491ead90366bff610aeda3216d17de6104624d0 to PENDING with url https://ci.adoptopenjdk.net/job/build-scripts-pr-tester/job/openjdk-build-pr-tester/1720/ and message: 'Building & Testing PR...'
Using context: pipeline-build-check
Obtained pipelines/build/prTester/kick_off_tester.groovy from git https://github.com/andrew-m-leonard/ci-jenkins-pipelines.git
[Pipeline] Start of Pipeline
[Pipeline] node
Running on [C3jenkins](https://ci.adoptopenjdk.net/computer/C3jenkins/) in /home/jenkins/.jenkins/workspace/build-scripts-pr-tester/openjdk-build-pr-tester

not sure why it tried to git clone from your repo andrew-m-leonard/ci-jenkins-pipelines.git ? @andrew-m-leonard

run tests quick 11

run tests quick

run tests

run tests quick

run tests

run tests quick 8,11,17,18,19

@andrew-m-leonard could you another look at this PR?


update cli-v2-install with instructions for jammy

The packages required for jammy are slightly different. The old instructions are confusing people.

Thanks! @metron2


Listen Interstitial Ad close event

Need Help
I have a scenario in my application, where i used a timer, i want to pause timer while showing full screen Ad and resume timer when user close the full screen Ad, is there is any way to achieve this, i can pause the timer when calling prepareInterstitial() , but how can i resume my timer ?, any way to listen close button event of full screen Ad.

document.addEventListener('onAdDismiss', function(data){
 });

 can solve my need. sorry for asking this help.


Transparency issues

sometimes whitespace and certain text backgrounds don't properly become transparent
working correctly:

not correctly:

(im using my pager because the init.vim for it is very small, my main nvim config doesnt have vim-solarized8 installed so there's no chance its a plugin conflict, im just using it to illustrate how it should look wrt transparency)
thanks!

nnoremap <f10> :call <SID>SynStack()<CR>
function! <SID>SynStack()
	if !exists("*synstack")
		return
	endif
	echo map(synstack(line('.'), col('.')), 'synIDattr(v:val, "name")')
endfunc

I am unable to replicate this. Can you use the above snippet? just take the cursor over the area which isn't highlighted properly and press f10. It will show you the highlight groups. Also do you have let g:solarized_termtrans=1 in your config?

over a space it shows ['vimSet'] and over one of the words (like =dark) it shows ['vimSet', 'vimSetEqual']
i added let g:solarized_termtrans=1 to the config and it didn't change anything. also i am manually typing hi Normal guibg=NONE into the editor to have the transparency work in the first place.

can you check the output for the command for hi vimSet and hi vimSetEqual? It should have guibg=none in it.

editor to have the transparency work in the first place in the pager for this test. I don't understand this part can you elaborate?

:hi vimSet
vimSet         xxx ctermfg=134 ctermbg=7 guifg=#839496 guibg=#002b36 guisp=nil```

```vim
:hi vimSetEqual
vimSetEqual    xxx ctermfg=134 ctermbg=7 guifg=#839496 guibg=#002b36 guisp=nil```

so doesnt look like they're being set. as for your second comment there what i mean is even with `let g:solarized_termtrans=1` being set, it's not actually turning transparent unless i manually enter `hi Normal guibg=NONE`


Missed one internal call that the NetworkManager uses

This is called in the NetworkManager and in doing the work to make a fully custom NetworkManager - as i talked about in vis2k#2039 I found i needed to call this on the NetworkClient.RegisterHandler callback - as you can see here https://github.com/vis2k/Mirror/blob/master/Assets/Mirror/Runtime/NetworkManager.cs#L1265
Honestly though walking the code for this is a very weird situation where NetworkServer.SetClientNotReady() which sends a NotReadyMessage message to the client which it then eventually in NetworkManager calls ClientScene.ready = false feels very weird to me because the other side of things has a dedicated function ClientScene.Ready() - so it feels like there should be a ClientScene.NotReady() and the property should be left as is.
Likely not something to fix here and now - but i need this setter to not be internal so i can continue making a fully custom NetworkManager without inheriting.

I found it hard to know what ClientScene.ready is intended to do, It is used it multiple places to do multiple things: set when the player connects, set when the scene is loaded/ready and set when the player marks themselves as ready via the hud.


Add db UPDATE feature to frontend

in a few scenarios (e.g. connection issues or GitHub api problems), webhook events can be missed which causes an inconsistent state in the database.
Example:

job1 is queued, INSERT into database
job1 is in progress, not inserted into the database because of connection issues
job1 is completed, INSERT into database

job1 will now be in status "queued" forever and be shown in the dashboard, although it it is already completed.
The current logic will only show jobs in the /queue endpoint if the same job_id is never inserted into the database as "in_progress".
Instead of coding logic for each combination of possibly missing jobs, the frontend should check the status of a given job (by using job_id) through the GitHub API and update the job's status accordingly (e.g. for jobs that are in status "queued" for more than 1 hour).

A button was added that uses the GitHub api to get the current status of jobs and updates them accordingly in the database.
Theoretically webhooks could be re-triggered from the webhook UI to achieve the same outcome but this is not user friendly.


Ignoring /release/output.json

Android Studio 3 Canary 1 generates a <module>/release/output.json file.
Reasons for making this change:
Android Studio 3 Canary 1 generates a <module>/release/output.json file when selecting from menu: Build -> Generate Signed APK..., which looks like:
[{
  "outputType": {
    "type": "APK"
  },
  "apkInfo": {
    "type": "MAIN",
    "splits": [],
    "versionCode": 32
  },
  "outputFile": {
    "path": "/path/to/the/generated/release/filename.apk"
  },
  "properties": {
    "packageId": "com.example.android",
    "split": ""
  }
}]

Links to documentation supporting these rule changes:
I failed to find any documentation on it.
If this is a new template:

Link to application or project’s homepage: N/A

@leesah any links to StackOverflow discussions or threads about this change?

I didn't specifically search for that in StackOverflow. Maybe I should ask about it if not found, at least to prove this is a change in Android Studio instead of some local configuration in my laptop. Is that what you mean?

Closing this out in favour of #2684 which seems similar but also doesn't really have any additional context.


[FLINK-26168][runtime] The judgment of chainable ignores StreamExchangeMode when partitioner is ForwardForConsecutiveHashPartitioner

What is the purpose of the change
The judgment of chainable should ignore StreamExchangeMode when partitioner is ForwardForConsecutiveHashPartitioner
Verifying this change
ForwardForConsecutiveHashPartitionerTest
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@flinkbot run azure

Thanks for review @zhuzhurk. I've addressed the comments. Please help to review again.

@flinkbot run azure

@flinkbot run azure

The CI always fails due to FLINK-26199

The failed case is a known issue and unrelated. Merging.


lombok: add bin output with lombok wrapper script

Motivation for this change
To provide a convenient way to run delombok.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Sorry about the delay. No, it needs to be jdk since the lombok tool uses tools.jar, which is only distributed in jdk.

Sorry about the delay. No, it needs to be jdk since the lombok tool uses tools.jar, which is only distributed in jdk.


CUDA backend

This PR implements the feature proposed in #7 and adds a CUDA backend for faster host to device transfers. Backwards compatibility is maintained other than the need to replace !$acc present statements with !$acc deviceptr. The CUDA backend is enabled by passing -DENABLE_CUDA=ON at build time.

Do you have some reproducible benchmarks using field_api that show the performance benefits of using the pinning method?
I would like to try them on our clusters.

Oh, sorry, I think there was a misunderstanding.
I am not asking you to add benchmarks to the repository.
I just want to know how much improvements you got on your clusters, and how I can reproduce it on our clusters.
You can add the files directly in this conversation thread.

Ah right, sorry about that. This is a very simple test that copies data back and forth. You can add it to the field_api tests and run it with CUDA turned ON/OFF. I saw the total run-time drop from 6.5 to 3.5 when I ran with CUDA enabled.
cpu_to_gpu_pingpong.txt

FYI @dareg. I have also prepared a real test-case that shows the effect of pinning by creating a variant of CLOUDSC that uses the FIELD_API CUDA backend: https://github.com/ecmwf-ifs/dwarf-p-cloudsc/tree/naan-update-field_api
Once you create the bundle (instructions in the README) and build with the --with-gpu and --with-cuda options. You can run two variants of CLOUDSC to see the performance impact of the CUDA backend.
With CUDA backend:
naan-> NV_ACC_CUDA_HEAPSIZE=12G bin/dwarf-cloudsc-gpu-scc-field 1 262144 128
 Executing CLOUDSC-GPU, "SCC" variant with FIELD API, PACKED STORAGE  F
     NUMPROC=1, NUMOMP=1, NGPTOTG=262144, NPROMA=128, NGPBLKS=2048
 Reference MFLOP count for 100 columns :  12.48232900
     NUMOMP    NGPTOT  #GP-cols     #BLKS    NPROMA tid# : Time(msec)  MFlops/s     col/s
          1    262144    262144         1       128    0 :        171    191142   1531304 @ rank#0:core#195
          1    262144    262144      2048       128   -1 :       1192     27449    219906 : TOTAL @ rank#0
    1 x   1    262144    262144      2048       128   -1 :       1192     27449    219906 : TOTAL

Without CUDA backend:
naan-> NV_ACC_CUDA_HEAPSIZE=12G bin/dwarf-cloudsc-gpu-scc 1 262144 128
    Executing CLOUDSC-GPU, "SCC" variant...
     NUMPROC=1, NUMOMP=1, NGPTOTG=262144, NPROMA=128, NGPBLKS=2048
 Reference MFLOP count for 100 columns :  12.48232900
     NUMOMP    NGPTOT  #GP-cols     #BLKS    NPROMA tid# : Time(msec)  MFlops/s     col/s
          1    262144    262144         1       128    0 :        171    190907   1529421 @ rank#0:core#67
          1    262144    262144      2048       128   -1 :       2727     11997     96113 : TOTAL @ rank#0
    1 x   1    262144    262144      2048       128   -1 :       2727     11997     96113 : TOTAL

For both runs, the kernel performance is around 190 GFs, but with the CUDA backend, the overall performance (including data offload) jumps from 12 GFs to 27 GFs.

Hello Ahmad,
I was wondering whether you could try the following things:

Hello Ahmad,
I was wondering whether you could try the following things:

Remove the PINNED attribute and apply the cudaHostRegister/cudaHostUnregister directly on %PTR. It is supposed to have the same effect and we avoid the PINNED attribute in the definition of the types

Remove the DEVICE attribute in the definition of the type, and use the "use_device" clause before calling cuda functions; Lukas is doing that here : https://github.com/lukasm91/field_api/blob/68726f6f55e72056509114f31f1db2c4c273a6e3/field_RANKSUFF_module.fypp#L395C35-L395C39 ; again this would have the effect of minimizing the impact on the implementation but also on the user code where we could keep the present clause

Thanks a lot for reviewing the code and your suggestions @pmarguinaud! Your suggestions have really improved the implementation, and also made it much easier to switch to a HIP backend in the future for AMD GPUs.
Allocations in page-locked memory are no longer achieved using CUDA-Fortran pinned arrays but via cudaHostRegister as you suggested. An interesting feature I discovered whilst doing this is that memory allocated via Fortran has an upper limit for pinning for single allocations: very large arrays allocated via ALLOCATE(SELF%PTR) cannot be registered in page-locked memory. cudaHostAlloc suffers from the same limitation. If however memory is allocated via C_MALLOC, then the only upper limit is the OS limit on page-locked memory allocations. This is also why I don't register wrapped fields in page-locked memory, as these will most probably be Fortran allocations.
I also no longer use CUDA-Fortran device arrays and allocate device memory directly on SELF%DEVPTR. Again, I have used the CUDA-C API rather than the CUDA-Fortran API so that we can use TYPE(C_PTR) rather than TYPE(C_DEVPTR) to store the allocation.
The decision to use the !$acc deviceptr clause rather than !$acc present clause was actually very intentional and not driven by the CUDA API. Currently in FIELD_API, we allocate SELF%DEVPTR on the host and use openacc to map this to a device allocation. We need to do this because the openacc map can only register contiguous arrays, and so we cannot register SELF%PTR directly in the openacc map because there is no guarantee SELF%PTR is contiguous. However if we make SELF%DEVPTR a true device pointer, we don't need to register it in the openacc map and we also don't need to do a redundant host allocation for SELF%DEVPTR. We can  therefore roughly half the memory footprint on host if we avoid the openacc map and use !$acc deviceptr instead of !$acc present.

Hello Ahmad,
I am very happy you succeeded in making the implementation simple. Regarding the large memory footprint on the host, could you provide a small test case to illustrate this ? We observed something similar, and solved it with dev_alloc.c

Hi Phillipe,
Please correct me if I am mistaken, but my understanding of dev_alloc.c is that we use a buddy allocator to allocate SELF%DEVPTR on host first, and then we use !$acc enter data create to map this host allocation to a device allocation. So currently in FIELD_API, SELF%DEVPTR is allocated both on host and on device.
This extra allocation of SELF%DEVPTR on host seems a bit wasteful to me. With cudaMalloc, we can allocate SELF%DEVPTR directly on device, without needing to allocate anything extra on the host. In this sense, no matter how efficient dev_alloc.c is, it will always require some unnecessary allocations on the host, whereas cudaMalloc does not.
I also confirmed this on a real application. The standalone ecWAM dwarf uses ec_meminfo, which provides a very convenient mechanism to track the total heap size. I ran an O320 grid (~ equivalent to Tco399) both with and without the FIELD_API CUDA backend, and I set DEV_ALLOC_SIZE to 2GB. The total heap usage without the CUDA backend was 24433 MB, dropping to 21368 MB with the CUDA backend. So a reduction of more than 10%.

Hello Ahmad,
I am interested in understanding what is happening here. First some explanations on dev_alloc.c: here we allocate a large chunk of virtual memory on the host, which will never be backed by physical memory. This chunk of memory is just allocated to have a mapping between host virtual memory and device memory. So it will never use any physical memory, unless somebody writes into DEVPTR on the host. We added this mechanism, because each time we allocated/deallocated/reallocated/... memory, some physical memory was leaking into DEVPTR members (we saw the physical memory increasing at each time step).
The total heap size is not the figure to look at; there is also stack memory, and what really matters anyway is physical memory (RSS).
Now, I would be happy to remove the dev_alloc.c; I do believe you that what you did solves the issue, but we also need to make sure that something similar exists on other architectures. What do you know about this ?

Thanks a lot Phillipe for the explanation on dev_alloc.c, it makes a lot more sense to me now.
For AMD GPUs, we can use the analogous hipMalloc:
https://docs.amd.com/projects/HIP/en/latest/.doxygen/docBin/html/group___memory.html
For Intel GPUs, we can use the OpenMP extension omp_target_alloc_device:
https://www.intel.com/content/www/us/en/docs/oneapi/optimization-guide-gpu/2023-1/memory-allocation.html
(or even the underlying SYCL malloc_device)
My suggestion would be to (for the short-term) keep dev_alloc.c as a fallback for when we are running with the CUDA backend disabled. When we later add a HIP (and/or OpenMP) backend for AMD GPUs, then we could possibly remove dev_alloc.c.
@mlange05 your thoughts on this?

Hello Ahmad,
Some other comments:

memory pinning should not be a compile time option, but should be enabled at run time (eg an option for the field constructor)
the deviceptr/present could also be a run time option (again a field constructor option)
I like the deviceptr, it is very neat, when compared to host mapped pointers, and it avoids the need of the complicated dev_alloc.c; but we may have to use some other software where host mapped pointers are expected (eg spectral transforms), so unfortunately, we have to keep both options for now
please check that dev_alloc.c solves your memory problem on ecwam (look at the rss value only)

Hi Phillipe,
Thank you for the suggestions. As requested, I have now made pinning of host allocations and host-mapped device pointers run-time options. Pinning is enabled by default if CUDA is enabled. Host-mapped device pointers are the default even if CUDA is enabled. If we want true device pointers the argument MAP_DEVPTR=.FALSE. has to be passed to the field constructor.
I have also checked the RSS size for ecWAM with dev_alloc.c and with cudaMalloc. In both cases it is essentially identical, so dev_alloc.c indeed does solve the memory problem.

Hi Phillipe,
I've implemented your suggestions, the highlight being that CUDA is enabled by default if CMAKE_CUDA_COMPILER is found and OpenACC is also enabled. The changes also highlighted some bugs in the testing, which I've also fixed.

Hi @pmarguinaud,
Happy new year and I hope you enjoyed your holidays 😄  Are there any more changes you would like me to make to this PR? If not, could you please approve it?


Add version property to HotTable component.

Description
Add version property to HotTable component. We have this for Handsontable and Vue/React wrappers. Helps in troubleshooting / demos.

Released in v5.0.0


[Feedback]: A repo linter and pull request generator tool

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Description
We have a bunch of very detailed policies about what MUST be done and MUST NOT be done, but it'll be impossible for humans to follow them and check all policies manually.
I found a handy tool called SecureRepo which could scan our github repo and submit a corresponding pull request to harden the CI's security.
I also found RepoLinter which could let us define the linting rules that check a github repo, and give suggestions on how to fix the issues it finds.
We might have hundreds of repos in the future, and our policies might keep changing and involving, it'll be impossible for us to keep all repos update-to-date manually.
Maybe we should consider a tool that combines them two, we can lint whether a repo meets our requirements, and if it's not, this tool could open a remediation pull request.

Hey @lonegunmanb,
Completely agree, as we discussed earlier in the week and we will follow up in our syncs to get this resolved.

@jtracey93  just circling back on this one to check if there are any actions that could be taken at this point in time to address the feedback and close the issue ?

This is being worked on by @lonegunmanb in a tool called grept

I'm closing this issue since we have grept now.


iOS Image Format

I have a sample app successfully running on both Android as well as iOS .. once the images are selected i use FileTransfer to upload them to my web server .. both os's do upload files; but I can not open the JPG images from the iOS device with any application other than Photoshop CS6.
I suspect the file is not a true JPG or contains some bits and pieces that make most apps including PREVIEW on OSX report the file is corrupt. Any assistance or guidance would be much appreciated.
I have attached two sample files.

have resolved the issue .. it is to do with the FileTransfer plugin and the server side handling of the multipart post


Add cake-contrib user on NuGet.org

First of all, I wanted to thank you for adding to the Cake community by adding this addin.
I was just wondering if you had seen this blog post:
http://cakebuild.net/blog/2016/08/cake-contribution-organization
We are currently going through a process of increasing the visibility of addins, and also trying to ensure their long term maintainability.
To that end, we are asking addin creators to add the cake-contrib user on NuGet as a co-owner (this can be done through the NuGet website by clicking on Manage Owners on the package page).
Would you be interested in doing this for Cake.AssemblyInfoReflector? If you have any questions about this, please let me know. There was some initial concern that the Cake Team were trying to "take over" packages, and that couldn't be further from the truth, and if you have this concern, or others, I would like to address them.
Thanks!

@wexman what were your thoughts on this one?

Just did it!

@wexman said...
Just did it!

Thank you very much for doing this, appreciate it!


Moving fsx (4) resources to v1beta1 version

Description of your changes
Moving 4 resources in the fsx group:

[x] aws_fsx_backup
[x] aws_fsx_data_repository_association
[x] aws_fsx_lustre_file_system
[x] aws_fsx_windows_file_system

Fixes https://github.com/upbound/provider-aws/issues/297
I have:

[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested
Manually

/test-examples="examples/fsx/windowsfilesystem.yaml"

/test-examples="examples/fsx/datarepositoryassociation.yaml"

/test-examples="examples/fsx/lustrefilesystem.yaml"

/test-examples="examples/fsx/backup.yaml"

/test-examples="examples/fsx/datarepositoryassociation.yaml"

/test-examples="examples/fsx/datarepositoryassociation.yaml"


Fixed the typo in blog file and added redirect page for it

Could you add the issue no to the commit message ("#572 ..."), please? Also I think if you add "Closes #572" as a comment here, merging this PR should auto-resolve the issue.

Fixes #572


Watch for DRPolicy spec changes and act accordingly

Watch for DRPolicy changes.  Whenever the Spec changes, all DRPC instances for that DRPolicy
will execute and reflect the change accordingly. If a new s3Profile is added or ScheduleInterval
is changed, then, each active VRG for that DRPC instance is updated.
Note: Updating the SchduleInterval is desirable only if the user is correcting a mismatch between
the ScheduleInterval values of the DRPolicy and VolumeReplicationClass.

Closing older PRs that are not applicable at present (or stale). Reopen if required.
Also, I think we should add webhooks to prevent updates to DRPolicy, and if an application requires a change in policy, it should disable DR and renable it with the newer policy.
The caveat is change to S3 stores as that is from the ConfigMap, if that can be prevented as well using webhooks it would be useful, or we may need to handle that one case.
Track this at https://github.com/RamenDR/ramen/issues/268


Task from the scheduler isn't run

I have created the logon task via the button from the settings GUI some time ago and for some reason it never opens FalconX, as it should....
I have tried deleting and re-adding it but it didn't change things at all
After that i have opened the Task Scheduler to see if it actually exists and if it is configured correctly and these are the results:

The task exists
The action of opening the app is set correctly
The trigger also seems set correctly
In the 'History' tab there is absolutely nothing! ("Number of events:0")
So i guess the trigger is messed up?
N.B.: I have never modified the task!
So yeah that's my issue....

I have been messing with the task and I believe the delay might be the problem....When I set it to 30 secs it works like a charm, but when i set it to 3 or 5 secs it doesn't even start the process...Will still look into it more tho.

Thanks for testing it out much appreciated 😁👍.
I didn't get the time to get into it yet 😓.
What if you set the delay to just 5 sec and make it restart every 5 seconds if it fails for 10 times or so?

The option where it reruns the task if failed has a minimum limit of 1 minute and that's not cool.
I think the best option is to set the delay to 10 seconds (just like i've set it now) so that it can work flawlessly on every slow hard disk cries


Update codecov action to v3

/ok-to-test

@toothstone this change is also present in #3 but it was more or less a test for the ok-to-test command


WebAPI: sometimes it does not do validation in ActionFilter

I'm using Version 7.1.1 for my solution Dotnet 4.6.2 using WebAPI template. I have a scenario as below:
Action Method
UpdateItem(Guid ItemId, UpdateItem command)
var validate = _validator.GetValidator<UpdateItem>().Validate(command);
            if (!validate.IsValid)
            {
                return Content(HttpStatusCode.BadRequest, "message");
            }

I also have an action filter as below
public override void OnActionExecuting(HttpActionContext actionContext)
        {
            if(!actionContext.ModelState.IsValid)
            {
                actionContext.Response =  actionContext.Request.CreateResponse(HttpStatusCode.Conflict, "message");
            }

            base.OnActionExecuting(actionContext);
        }

When I debug, sometimes action filter does not catch the violations, the _validator catch the violations instead. Someone know the reason?

Hi @JeremySkinner , thank for your support. Currently We accept this minor bug. It may be from our project, Not related to the FluentValidation.


Update remaining reference to OpenStackDataPlane

Signed-off-by: James Slagle<EMAIL_ADDRESS>As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

Thanks you @slagle - I've imported that on in #528 in a distinct patch, so we should be OK. Both are needed anyway to unlock the situation.
Closing this one, since it should be in the mentioned PR.


Add MicroShift Prometheus Exporter

Which issue(s) this PR addresses:

Closes #

@ggiguash should I, do we need a ticket for this?


[Bug] restrict-wildcard-verbs policy parsing error

Kyverno Version
1.6.x
Kubernetes Version
1.20.x
Kubernetes Platform
EKS
Description
Hi,
I was trying to add the restrict-wildcard-verbs policy to my cluster. I copy pasted the exact policy that's listed here - https://kyverno.io/policies/other/res/restrict-wildcard-verbs/restrict-wildcard-verbs/
But I get the following error:
Error: UPGRADE FAILED: parse error at (kyverno-policies-custom/templates/other/restrict-wildcard-verbs.yaml:43): unexpected "(" in operand

Steps to reproduce

Download the following helm charts:

kyverno: https://github.com/kyverno/kyverno/tree/main/charts/kyverno
kyverno-policies: https://github.com/kyverno/kyverno/tree/main/charts/kyverno-policies

Add the policy (https://kyverno.io/policies/other/res/restrict-wildcard-verbs/restrict-wildcard-verbs/) inside the kyverno-policies/other/restrict-wildcard-verbs.yaml

Add the following lines to the includeOtherPolicies inside the values file of the kyverno-policies chart:

includeOtherPolicies:
 - restrict-binding-clusteradmin
 - restrict-wildcard-verbs

Apply both of the helm charts to the cluster

Expected behavior
I expect the restrict-wildcard-verbs policy to be applied successfully without any parsing errors.
Screenshots
Error:

Kyverno logs
No response
Slack discussion
No response
Troubleshooting

[X] I have read and followed the documentation AND the troubleshooting guide.
[X] I have searched other issues in this repository and mine is not recorded.

You will need to modify the policy, if you want to deploy it with Helm, to escape Kyverno's variable syntax. See the docs here.

Thank you!


[Feature Request] - Configure IDE path on "use" command

Problem:
Right now the user needs to manually configure their IDE path for VSCode in order to have all IDE tooling running properly. This causes some support issues to be opened, need to view the documentation, and also a less ideal Dev experience.
Solution:
When the developer does fvm use <version> fvm should automatically detect the IDE and configure the version if it has not been configured before.
VS Code:

Look for the .vscode folder in the project
Look for settings settings.json
Look for dart.flutterSdkPath or dart.flutterSdkPaths
If exists it should be ignored, if not it should set the path to <version>
Should also configure all remaining settings as described here if they do not exist -> vscode config

Android Studio:

Studio also creates a .idea folder however some research needs to be done if this can be created the same way as vscode or needs to be configured within the IDE itself.

I was actually coming to suggest this, but I just noticed, in the logs for the VSCode flutter extension, the extension looks for a flutter SDK at the root of the workspace in a folder called .flutter. I ran ln -s .fvm/flutter_sdk .flutter and upon re-opening my project, the editor found the sdk automatically! Maybe we could get an option in fvm use to link the SDK there instead of .fvm/flutter_sdk.

That is a great suggestion. Do you want take a stab at it with a PR, it would be important to maintain both directories, to offer backwards compatibility, and provide a deprecation notice.

Any strong preference on it being a flag vs just the default behavior with a notice? I don't mind to make a PR, but it'll likely be this weekend if it's more than a simple change.

Yes I believe that makes sense... I suggest keeping more specific like flutterProjectSdkPath since we have the fvm path, and some others which are the cache version.

This has changed considerably for version 3.0, I believe on Vscode. this already is behaving correctly, but I am not sure what is the best approach to change the PATH on android studio.

@leoafarias I'm trying FVM 3.0 beta 5 and I've noticed, that the structure within the .fvm has been changed.
In the docs of FVM 2.x there is:

Copy the absolute path of fvm symbolic link in your root project directory. Example: /absolute-project-path/.fvm/flutter_sdk

As per FVM 3.0, there is no flutter_sdk folder anymore, the structure looks like:
.fvm
  - version
  - release
  - versions
    - 3.16.0

To make it work in Android Studio, I've to point it to an exact version, like /absolute-project-path/.fvm/versions/3.16.0.
While FVM still makes it easier to handle multiple Flutter versions, within a project each team member will have to change their paths.
It looks like that in Android Studio the paths are saved under .idea/libraries/Dart_SDK.xml and .idea/libraries/Dart_Packages.xml.
So, there are two possible solutions:

When fvm use is invoked, change the paths in those files.
Re-introduce a flutter_sdk folder that points to whatever version in .fvmrc.

Did you check the .fvm/version folder?

Did you check the .fvm/version folder?

What exactly do you mean?
version is a file.
ls -al
total 16
drwxr-xr-x   5 manuelrauber  staff   160 Nov 16 09:39 .
drwxr-xr-x  38 manuelrauber  staff  1216 Nov 16 11:57 ..
-rw-r--r--   1 manuelrauber  staff     6 Nov 16 09:39 release
-rw-r--r--   1 manuelrauber  staff     6 Nov 16 09:39 version
drwxr-xr-x   3 manuelrauber  staff    96 Nov 16 09:39 versions

Ah ok, thought this might be the new linked version folder, nvm.

@ManuelRauber I think adding the symlink to flutter_sdk might make sense, however I had the symlink resolve every time in my android studio setup. Also I am really interessted in changing the android studio path similar to what we are doing with vscode, is that something you can provide more insight, or maybe even do a rough PR that I can take over?

@leoafarias I can try to make a rough sketch if changing the paths in the .idea folder actually works.

@ManuelRauber looks good, could you make a PR with that change, so I can bring it over before releasing 1.0, also if its not too much to ask, I want to create a page about Android Studio integration, and maybe explain that behavior, so people are aware of these details.

@ManuelRauber looks good, could you make a PR with that change, so I can bring it over before releasing 1.0, also if its not too much to ask, I want to create a page about Android Studio integration, and maybe explain that behavior, so people are aware of these details.

@leoafarias yes, I can do a PR that includes:

the SymLink
Docs
Maybe a little check in fvm doctor or so that simply outputs if PROJECT_DIR is found in idea settings


Allow the use of fqdn in neutron_service_check.py

This change allows for use of either hostname or fqdns when running
the neutron_service_check.py plugin. It did not find the host when
fqdn was used.
Connects #1603

https://github.com/rcbops/rpc-maas/pull/233

Closing in favor of https://github.com/rcbops/rpc-maas/pull/233


Expire blocks outside retention period

cc @robskillington
We should expire blocks outside the retention period and return them to the pool so they don't live in memory forever. Also fixing a bug when determining the min block time in RemoveBlockAt and adding tests.

Coverage increased (+1.09%) to 75.45% when pulling cb9670e57aa8436f910d9294e3ec637d9cacc02d on xichen-expire-blocks into bd4e562e7b359b7c25e4a96f7518c6f3416afce0 on master.

Other than minor comments LGTM, feel free to address as necessary and mergehoy :shipit:

Coverage increased (+1.05%) to 75.413% when pulling 93d15538fc90794e6f5beb60ed52445f943b8f90 on xichen-expire-blocks into bd4e562e7b359b7c25e4a96f7518c6f3416afce0 on master.


Crashes under go1.14 runtime

This is the runtime issue we encountered the hard way, through crashes of tile38:
https://github.com/golang/go/issues/37688
The issue manifests with tile38 if the query contains TIMEOUT and WHEREVAL and the timeout gets triggered (query does not finish within the allotted timeout). After such queries, the next GC cycle crashes the server, exactly like the issue above.
Rebuilding with go1.14.3 fixed the issue for us. We can reliably crash 1.14 runtime and not crash with 1.14.3 runtime on the same code.

There's nothing to fix in tile38 for this, but all the released binaries should not be built with go 1.14.

Yikes. That's good to know.
I just made sure that the official Docker releases will be 1.14.3+


Failed to compose memo response after upgraded to 0.13.1 (from 0.13.0)

Describe the bug
Upgraded the docker image from 0.13.0 to 0.13.1, the old notes cannot be open anymore.
There is error in the docker logs.
{"time":"2023-05-31T16:37:08Z","method":"GET","uri":"/api/memo?rowStatus=NORMAL&limit=20","status":500,"error":"code=500, message=Failed to compose memo response, internal=not found"}

Steps to reproduce

stop current container
remove container memos
pull the latest image 0.13.1
start memos with the latest image
open browser, there is error message on the upper right conner "Failed to compose memo response"

Screenshots or additional context
This is not a browser cache issue.  I also tried using incognito mode and with different browser.

Could you please try to vacuum database with this button? Maybe it will works.

Could you please try to vacuum database with this button? Maybe it will works.

What does that mean? "vacuum" looks scary for me.

Could you please try to vacuum database with this button? Maybe it will works.

What does that mean? "vacuum" looks scary for me.

https://www.sqlite.org/lang_vacuum.html

The VACUUM command rebuilds the database file, repacking it into a minimal amount of disk space.

Could you please try to vacuum database with this button? Maybe it will works.

What does that mean? "vacuum" looks scary for me.

https://www.sqlite.org/lang_vacuum.html

The VACUUM command rebuilds the database file, repacking it into a minimal amount of disk space.

VACUUM works. Thanks @boojack @Zeng1998


support set retry count through RpcContext,  cover the settings of default invoke times

support set retry count through RpcContext,  cover the settings of default invoke times.
支持通过RpcContext设置重试次数，如果设置会覆盖默认重试设置。

Codecov Report

Merging #7777 (c42e40c) into master (c40af2a) will decrease coverage by 0.27%.
The diff coverage is 62.50%.

@@             Coverage Diff              @@
##             master    #7777      +/-   ##
============================================
- Coverage     59.47%   59.19%   -0.28%
+ Complexity      556      554       -2
============================================
  Files          1083     1083
  Lines         43748    43753       +5
  Branches       6380     6363      -17
============================================
- Hits          26017    25899     -118
- Misses        14870    14986     +116
- Partials       2861     2868       +7

Impacted Files
Coverage Δ
Complexity Δ

...bo/rpc/cluster/support/FailoverClusterInvoker.java
88.00% <62.50%> (-0.89%)
0.00 <0.00> (ø)

...ng/transport/dispatcher/all/AllChannelHandler.java
62.06% <0.00%> (-27.59%)
0.00% <0.00%> (ø%)

...ntext/event/AwaitingNonWebApplicationListener.java
48.48% <0.00%> (-24.25%)
0.00% <0.00%> (ø%)

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0.00%> (-15.39%)
0.00% <0.00%> (ø%)

...rg/apache/dubbo/remoting/utils/PayloadDropper.java
76.92% <0.00%> (-15.39%)
0.00% <0.00%> (ø%)

...ng/transport/dispatcher/WrappedChannelHandler.java
47.82% <0.00%> (-15.22%)
0.00% <0.00%> (ø%)

...bo/config/event/listener/LoggingEventListener.java
50.00% <0.00%> (-12.50%)
0.00% <0.00%> (ø%)

.../apache/dubbo/rpc/protocol/AsyncToSyncInvoker.java
62.06% <0.00%> (-10.35%)
0.00% <0.00%> (ø%)

.../remoting/transport/netty4/NettyClientHandler.java
76.27% <0.00%> (-10.17%)
0.00% <0.00%> (ø%)

...o/remoting/transport/ChannelHandlerDispatcher.java
60.00% <0.00%> (-10.00%)
0.00% <0.00%> (ø%)

... and 29 more

Continue to review full report at Codecov.

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


msgSeqNum in the Header

what is msgSeqNum?
msgSeqNum is a field in the FIX protocol message header that is used to assign a unique sequence number to each message. This field contains a numeric value that is incremented by one for each message sent. The sequence numbers are used by the receiving party to ensure that messages are received in the correct order and to detect missing or duplicate messages.
For example, if a sender sends three messages with sequence numbers 1, 2, and 3, the receiver will expect to receive those messages in that order. If the receiver receives message 3 before message 2, it will know that a message is missing and can request a retransmission. Similarly, if the receiver receives two messages with the same sequence number, it will know that there is a duplicate message and can discard one of them.
The msgSeqNum field is an important part of the FIX protocol's reliability and consistency mechanisms, and helps to ensure that messages are properly processed and recorded by both the sender and receiver.

Is msgSeqNum automatically generated?
Yes, msgSeqNum is automatically generated by the FIX protocol implementation and assigned by the sending party. The sender increments the value of msgSeqNum by one for each message sent and includes it in the message header. The value of msgSeqNum is unique for each message and helps to ensure that messages are received in the correct order and that duplicate or missing messages can be detected.
The msgSeqNum field is essential to maintaining the integrity and consistency of the FIX protocol message stream. It provides a reliable and consistent means of identifying each message and establishing its position within the message stream. As such, it is important that each implementation of the FIX protocol maintains accurate and synchronized msgSeqNum values to prevent errors and ensure that messages are properly processed and recorded

can the user input the field manually?
While it is technically possible to manually input the msgSeqNum field in a FIX protocol message, it is not recommended to do so.
The msgSeqNum is a critical part of the FIX protocol's reliability and consistency mechanisms, as it is used to ensure that messages are received in the correct order and to detect missing or duplicate messages. If a user manually inputs the msgSeqNum value, it can lead to errors, such as duplicate or missing messages, that can cause problems in the transaction processing.
Moreover, FIX protocol implementations are expected to generate msgSeqNum values automatically and maintain their accuracy and synchronization. If the msgSeqNum is manually inputted, it may cause issues with compliance or regulatory requirements that require an audit trail of message transmission.
In general, it is best to rely on the automatic generation of msgSeqNum by the FIX protocol implementation rather than manually inputting the value

should it always be the same in the same session
No, msgSeqNum should not always be the same in the same session. The msgSeqNum field in the FIX protocol message header is designed to be incremented by one for each message sent within a given session. Each new message should have a msgSeqNum value that is one greater than the previous message sent in that session.
This ensures that messages are received in the correct order and that any missing or duplicate messages can be identified and handled appropriately. If the msgSeqNum remains the same throughout the session, it could lead to errors and inconsistencies in the transaction processing.
It is important to note that the msgSeqNum field is reset to 1 at the start of each new session. Therefore, the sequence of msgSeqNum values is unique for each session and is not meant to be continuous across different sessions.
In summary, the msgSeqNum field should be unique and incremented for each message sent within a session, and reset to 1 at the start of each new session


enable Terraform apply on CI merges

This PR will also be used to verify the TF Plan once we add the ci-check/terraform tag
Description

Why is this needed

Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

It looks like the state from TF Enterprise provides a terraform.tfvars which still has the token variable

This isa done 👍  #20


[Go Live][Oct 27]

Push to live

Learn Build status updates of commit d5be068:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/adaptive-cards/channel-teams_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/Addascope_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/addclientsecret_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/add-conversation-bot_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/add-desktop-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/addscope.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/appidURI.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/apply_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/Azurebotcreate.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/Azurebotresource_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/botserviceconnection.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/certificate.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/expose-an-api_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/exposeAPI_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/ExposeAPI_2.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/filepath_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/gotoresource_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/manage-apps_3.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/manage-your-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/manifest_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/messagingURL_1.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/messagingURL_2.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/newclientsecret_2.PNG
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/open-app.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/bots/redirectURIadded.png
:white_check_mark:Succeeded
View

This comment lists only the first 25 files in the pull request.
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Discourse Misconfigured Cert - Firefox

Is this a feature request or a bug?
bug
Expected behavior:
being able to connect to https://discourse.entropic.dev/t/looking-for-help-setting-up-fluent-rs/203
Actual behavior:
security warning from firefox

Steps to replicate:

have firefox 67
go to https://discourse.entropic.dev/t/looking-for-help-setting-up-fluent-rs/203

Environment info (where relevant)
firefox on ubuntu
also happens with firefox on android

This doesn't happen for me on browsers that have been to the Discourse instance before, but does on a clean install of Firefox Nightly on Android. The issuer there is Gandi so I wonder if FF doesn't recognize them as a CA by default, or if something was revoked?

Some research implies that it may be missing intermediate certificates, as if you've visited a different site previously that provided them, it'll work:
http://wiki.gandi.net/en/ssl/intermediate


#1864 - Add Guzzle version 7

#1864 Added Guzzle version 7 for new packages.

There is more to it than that. Closing this in favor of #1860


Default column type for currency in MySql

The version of liquibase: 3.8.0
When a column type is set to 'currency', liquibase for MySql uses by default 'decimal' type. At the same time for oracle, liquibase uses 'number(15,2)', which I assume is more reasonable, cause money do have cents.
To allow to store money as fractional numbers by default and to have a consistency  between different databases, I'd offer 'instead of 'decimal', use 'decimal(15,2)' as 'currency'.
It seems both for MySql and Postgresql the same 'decimal' type is used now. If you accept this offer, can you please adopt both database types.
┆Issue is synchronized with this Jira Bug by Unito

Hi @AlexandrSokolov Thanks for writing up this feature request for MySQL default for currency.  We will add it to the list of requests for changes to PostgreSQL and MySQL.

Thank you, @molivasdat


Update vep endpoint docs

Description
One more VEP plugin will be available through the VEP endpoint. This documentation is needed for the plugin.
Use case
Plugin functionality available through REST.
Benefits
The endpoint needs documentation to be displayed.
Possible Drawbacks
None, the default behavior is the same.
Testing
Have you added/modified unit tests to test the changes?
If so, do the tests pass/fail?
The tests are failing because of branches.
Have you run the entire test suite and no regression was detected?
No regression detected.
Changelog
Are you changing the functionality of an endpoint? If so, please give a one line summary for the public facing changelog.
[/vep/:species/hgvs/] new plugin option added: SpliceAI
[/vep/:species/id/] new plugin option added: SpliceAI
[/vep/:species/region/] new plugin option added: SpliceAI

Please wait to merge until a parallel PR for release/100 is opened. Then, merge both.

Merging as tests passed on travis.


No validationError raised on multiple "SET" operators in UpdateExpression

Hi,
I have encountered the following discrepancy between production dynamodb and the moto mock:
Running a transact_write_items request with two "SET" operators under UpdateExpression causes ValidationException on dynamodb, while my mock db accepts it and my tests pass.
Example query:
    self._client.transact_write_items(
        TransactItems=[
            {
                "Update": {
                    "TableName": self._table_name,
                    "Key": {
                        "part_id": {"N": str(condition_state.part_id)},
                    },
                    "ConditionExpression": "version = :version AND is_final = :false",
                    "UpdateExpression": "SET expire_at_utc = :expire_at_utc SET is_final = :true ADD version :one",
                    "ExpressionAttributeValues": {
                        ":version": {"N": str(condition_state.version)},
                        ":true": {"BOOL": True},
                        ":false": {"BOOL": False},
                        ":one": {"N": "1"},
                        ":expire_at_utc": {"N": str(int(state_item.expire_at_utc.timestamp()))},
                    },
                }
            },
            {"Put": {"TableName": self._table_name, "Item": dynamo_item, "ConditionExpression": "attribute_not_exists(version)"}},
        ]
    )

Dynamodb raises the following exception:
File "/opt/python/botocore/client.py", line 1021, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.exceptions.ClientError: An error occurred (ValidationException) when calling the TransactWriteItems operation: Invalid UpdateExpression: The "SET" section can only be used once in an update expression;

While moto doesn't.

Hi @nirmachlev, thanks for raising this! I've opened a PR with a fix for this.

Hi @jasonrdunne, can you share an example test that fails but shouldn't?

Thanks @jasonrdunne - looks like it's mostly the last example that is failing? That should now be fixed as of moto >= 5.0.11.dev17 - I'll try to get a proper release with this fix out later today.


Unable to view events and days off in calendar

Hi,
I am unable to view events and days off in calendar even though my access rights are of stakeholder.
Please help me.
Thanks,
Sumit

Can you press F12 in your browser, refresh the page, and then look at / send any errors that appear in the console or network tabs?

I have the same problem with our installation. It seems to have happened after one of the latest TFS updates. We have Update 3 of 2017.
The console shows the following errors:
Current window: jvsoutalm01/tfs/JeevesERP/JeevesPlatform/Kernel%20Feature%20Team/_apps/hub/ms-devlabs.team-calendar.calendar
Error getting event data.
Event source: Event
Reason: Error: Could not load event source Event. Request timed out.
Calendar.js (241,21)
Error getting event data.
Event source: Days off
Reason: Error: Could not load event source Days off. Request timed out.
Calendar.js (241,21)
The Calendar add-in is performing extremely slow after latest updates.

Hi, we are also seeing this error ,

Failed to set referrer policy: A policy specified by a meta element must contain only one token. The referrer policy has been left unchanged.
main.js:20961
       Error getting event data.
Event source: Days off
Reason: Could not load event source Days off. Request timed out.
(anonymous) @ main.js:20961
Can you please let us know if you have found a solution for this issue ?

We're closing old issues so we can get back to actively maintaining the extension. Feel free to reopen if this is still an issue for you.


Manage Database not working

Issue Type: Bug
When Right-Click'ing on database in connections or server dashboard and selecting "manage database", the database dashboard does not open
Azure Data Studio version: azuredatastudio 1.10.0 (5fdb967ffcea41df0335ec17cfa396a694bb174e, 2019-08-14T18:28:10.311Z)
OS version: Windows_NT x64 10.0.18362

System Info

Item
Value

CPUs
Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz (16 x 3600)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
31.94GB (14.58GB free)

Process Argv
--folder-uri file:///c%3A/Source/SqlScripts

Screen Reader
no

VM
0%

Extensions (4)

Extension
Author (truncated)
Version

admin-tool-ext-win
Mic
0.0.1

import
Mic
0.9.0

profiler
Mic
0.9.0

schema-compare
Mic
0.5.0

I'm unable to repro this. Let's keep an eye out and see if we or someone else experiences this. Could've just been the tooling crashed.

Hi
I have updated the repro steps
Can reproduce everytime i try this


if an admin clicks EMAIL REQUESTER but doesn't follow through, the account state still changes

Steps to reproduce:

Request an account, say chaos11
As an admin, at [https://shib-idp1.gpolab.bbn.com/manage/display_requests.php], select "EMAIL REQUESTER" and click "SUBMIT"
From [https://shib-idp1.gpolab.bbn.com/manage/request_actions.php], click "Return to main page" (without sending the email)
[https://shib-idp1.gpolab.bbn.com/manage/display_requests.php] now shows the chaos11 account in the "Account Requests Waiting for Requester Response" section, with no "Action Performer" or "Email Sent" entries filled in in the database

Imported from trac ticket #41,  created by chaos on 09-17-2013 at 15:39, last modified: 10-03-2013 at 16:50

Since the EMAILED_REQUESTER state doesn't correspond to anything in LDAP, this isn't a blocker from the point of view of LDAP and the database being consistent with each other (it's just about the DB not being consistent with itself).
If it's something y'all can fix easily, IMO the right fix is to avoid moving the request into a new state until the e-mail is actually sent, so if the admin wanders off before sending the e-mail, it's a noop.  But, again, i think it's not a blocker.
Trac comment by chaos on 09-17-2013 at 15:44

Verified that this is fixed.
Trac comment by chaos on 09-26-2013 at 16:52


Create a CHANGELOG.md to track changes

As we do the versioning this should update

Addressed in PR #183


SergeyPirogov Driver manager

Proposed changes
The second attempt to integrate automatic driver manager with Selenide.
Checklist

[x] Checkstyle and unit tests pass locally with my changes by running gradle check chrome htmlunit command
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Coverage decreased (-0.5%) to 64.021% when pulling a16ec9c282a79affd128822e28275cf232f34911 on wd_manager into f24d0f5d4c15837a882b7ace13a8067fe891f051 on master.

Codecov Report

Merging #610 into master will decrease coverage by 0.55%.
The diff coverage is 4.16%.

@@             Coverage Diff              @@
##             master     #610      +/-   ##
============================================
- Coverage     60.93%   60.38%   -0.56%
  Complexity      759      759
============================================
  Files           147      148       +1
  Lines          2698     2721      +23
  Branches        257      263       +6
============================================
- Hits           1644     1643       -1
- Misses          956      979      +23
- Partials         98       99       +1

Impacted Files
Coverage Δ
Complexity Δ

...codeborne/selenide/webdriver/WebDriverFactory.java
83.33% <0%> (-3.21%)
13 <0> (ø)

...ain/java/com/codeborne/selenide/Configuration.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...rne/selenide/webdriver/WebDriverBinaryManager.java
0% <0%> (ø)
0 <0> (?)

...orne/selenide/webdriver/AbstractDriverFactory.java
69.04% <100%> (ø)
10 <1> (ø)
:arrow_down:

...e/selenide/impl/WebDriverThreadLocalContainer.java
77.24% <0%> (-0.69%)
30% <0%> (ø)

Continue to review full report at Codecov.

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

Coverage decreased (-0.6%) to 63.984% when pulling a16ec9c282a79affd128822e28275cf232f34911 on wd_manager into f24d0f5d4c15837a882b7ace13a8067fe891f051 on master.


Release 3.0.0rc1

Not sure where to ask this, but will this be released as jinja3 (allowing jinja2 and jinja3 to coexist)? Or stay jinja2 version 3?

The import name remains jinja2.


CTS fix copy paste bug on max fanout

This PR fixes a copy paste error on PR https://github.com/The-OpenROAD-Project/OpenROAD/pull/5516.

We currently don't have any regression tests for max fanout limit on CTS. I can look into adding one.

We currently don't have any regression tests for max fanout limit on CTS. I can look into adding one.

That would be good

We currently don't have any regression tests for max fanout limit on CTS. I can look into adding one.

That would be good

Ok, I will add a regression test for this.


[WIP] Clarify pKa submission instructions, add additional microstates; update logP instructions to allow optional logD

Thanks to consultation with Stefan Kast, Marilyn Gunner, Mehtap Isik, Nicolas Tielker, and Bogdan Iorga, we are making a number of fixes/clarifications:

[x] Update README files/instructions for pKa challenge to clarify exactly what free energies we are after/what we mean by state free energies. Mention proton free energy contribution to help QM community make sense of what we are asking for. Clarify direction of free energy difference that we're looking for.
[x] Update pKa instructions to request macro pKa submission in methods section of prediction files
[x] Update pKa prediction template and example to show prediction of macro pKa
[x] Update README files for pKa challenge to indicate what happens if states are omitted from predictions (populations assumed negligible)
[x] Update mol2, sdf and csv files to add additional microstates from Bogdan Iorga
[x] Update README for logP to request optional logD values for consistency check
[x] Update logP prediction template and example to show prediction of logD
[x] Add folder that contains slides of the transition networks of the challenge molecules

The following molecules have had additional microstates added:

SM25: two additional states added (+1 and 0)
SM26: two additional states added (+1 and 0)
SM28: one additional state added (-1)
SM31: one additional state added (+1)
SM34: one additional state added (+1)
SM42: one additional state added (0 )
SM43: two additional states added (+1 and 0)
SM46: one additional state added (+1)

I checked the new states added and they look correct, except for SM31 and SM34 where we are still missing one each. These will be pushed to complete the updates here.

Thanks, @bergazin !


Error when loading the model on HuggingFace

Hello!
I pushed my models on HuggingFace and everything worked well until today. I'm getting these error and I saw that there are the same thing for the SpeechBrain models on HuggingFace. Is it related to "facebook/..." models, so I have nothing to do in my side, do I?

Hum, indeed HF problem I believe.

Yeah, it definitely comes from HF. So I'm waiting until they fix it!


integral A^2

Hello, Dear Experts' Libcint
Is it possible implement

this integral, with Libcint?
Have nice day!

int1e_cg_a11_part_sph (int1e_cg_a11_part_cart) computes the integral for r_K/r_K^3 * r_O. You will receive 9 components in the output buffer. They are orderd as K_xO_x, K_xO_y, K_xO_z, ..., K_zO_y, K_zO_z. Before calling the integral function, you may need to set the common gauge origin env[PTR_COMMON_ORIG:PTR_COMMON_ORIG+3] and 1/r origin env[PTR_RINV_ORIG:PTR_RINV_ORIG+3] . Definition of PTR_COMMON_ORIG and PTR_RINV_ORIG can be found  https://github.com/sunqm/libcint/blob/master/include/cint.h.in#L29


ARM platform builds of the CLI

ARM64 (and other) compatible versions of the cli would be very helpful. It would be great to be able to use the cli from a Raspberry pi or a M1 mac.
Scope (please mark with X where applicable)

New Functionality          [ ]
Install                    [ ]
SMI Traffic Access Policy  [ ]
SMI Traffic Specs Policy   [ ]
SMI Traffic Split Policy   [ ]
Permissive Traffic Policy  [ ]
Ingress                    [ ]
Egress                     [ ]
Envoy Control Plane        [ ]
CLI Tool                   [x]
Metrics                    [ ]
Certificate Management     [ ]
Sidecar Injection          [ ]
Logging                    [ ]
Debugging                  [ ]
Tests                      [ ]
CI System                  [ ]
Demo                       [ ]
Project Release            [ ]

Fixed in #4553


Adonis not Receiving Ajax Post Parameters

Hello,
I have been making ajax post requests with parameters from javascript to adonis successfully, (meaning that adonis receives the post parameters correctly) using jQuery. However a similar request being made using a JS software called Stimulsoft Report.JS is being sent, but it's post parameters are totally ignored by Adonis. I have attached pictures of the console logs (console.log(request.all())) and the requests that were made to adonis.
Unsuccessful request:

Successful request (using jquery ajax):

Do you know what the issue might be?

Hi @raen79!
As you can see it's not the same request.  It should be handled differently.

Hi!
I assumed it was because stimulsoft was sending a payload, but adonis doesn't seem to have a specific way to handle it (at least I can't find it in the documentation).

Ideally Request Payload is used by browser when you send raw data. I can see the Content-type of the request is set to text/plain which needs to be application/json to make Adonis parse the JSON body as a object.

And is there no way of receiving a request made with text/plain in adonis and converting it to json?

@raen79 Why would you do that?

When I wrote this I thought I wouldn't be able to change Stimulsoft's ajax request's content-type but I was able to so I solved the issue.
Thank you very much!

Closing the issue


To add a calculator to find Area of Quadrilateral if a,b,c,d vectors are given

Is your feature request related to a problem? Please describe.
To add a calculator to find Area of Quadrilateral if a,b,c,d vectors are given

Hi, I want to work on this issue I will start working on it as soon as I get assigned!! I am a part of GSSoC'21.
Kindly assign this issue to me!
Discord Username : Abhijeet Sinha(P)
Discord Tag: #4018


After deleting files in Lakehouse the screen struck with old files details

Describe the bug
After deleting files in Lakehouse the screen struck with old files details
What happened?

i needed to refresh the page to remove them
What did you expect to happen?
Removal and show what are not deleted
How would you reproduce the bug?
Delete files in Lakehouse
Did this happen via Desktop and/or Smartphone?
Desktop
Operating System
No response
What browsers are you seeing the problem on?
Chrome
Version
No response
Device
No response
What is your current role?
Student Ambassador
What technical topic is this bug related to?
AI/Data Science/Machine Learning
Relevant log output
No response

@vivsridh4 When i Refreshed the page after deleting then only the deleted files info was removed or else no i tried multiple times.


Packages incompatibility, - impossible to upgrade "dart pub get / - null safety  "

Hello are you still maintaining this project ?

邮件已经收到，我会尽快查收！

I won't maintain it anymore because I don't have the time now and it won't bring me any profit.
If there is a business cooperation, I can consider it.

Yes a business collaboration is on the table if you open to that . How can I contact you ?

@momojdy
Yes, you can send me your requirements by email<EMAIL_ADDRESS>

@GanZhiXiong please do let me know if you have received my email , I also attached my wechat for easier communication.


unnecessarily complicated ifdef handling and namespace polution

https://github.com/openssl/openssl/blob/a227ff336fc934b4a52c4659ae423dfa10efd5f3/providers/fips/self_test.c#L71-L157
That is wrong to have declared a public init and cleanup function - they should not be visible as such. They should be static. That pollutes the namespace with a function name that is highly likely to clash with user function names and it also allows direct user invocation of the functions and if we planned to allow for that they should be ossl_ prefixed.
And the ifdef soup here is not the right approach IMHO - these functions are required.
#if defined(DEP_INIT_ATTRIBUTE) && defined(DEP_FINI_ATTRIBUTE)
The whole mess here is more complicated than it needs to be.
We should have a default define of those two items that does nothing (which is what the majority of platforms use) and then just the GNUC attribute tagging. Or define a default if one is not set.
Repeating the same "junk" all over the place just clutters up the code with noise.
And on top of that the whole DEP_DECLARE() stuff is just wrong-thinking. It isn't at all required on any of the platforms if the code is simply slightly reorganised to have the GNUC attribute tagging done first then it can be directly used.

Noting that there isn't any namespace pollution.  These symbols are linked into a shared object which doesn't export them.  The shared object's exports are not merged into the callers namespace.
That doesn't mean we shouldn't make them static.

AFAIK at least in some cases the DEP entry points cannot be static.


PnP brief doxygen is not particularly brief

From @ahsonkhan feedback on #1825, az_iot_hub_client_properties.h has functions where the doxygen brief is anything but.
az_iot_hub_client_properties_writer_begin_component is just one example but there are others.

@jspaith do you have a milestone/timeline in mind for this issue?

@ahsonkhan  - hopefully next week.  Before I do though a question to you.
For best practices here, I'll copy/paste in one of examples below.  My thoughts here would be that the @brief should just be that first sentence and then a @remarks could come starting with the This API should be used in response....
The @remarks is how I've done this before & how az_core level stuff does this (though they're not as verbose as this section) so it naively seems OK but pre-PR I'll ask.
Thanks
/**
 * @brief Begin a property response to a writable property request from the service.
 *
 * This API should be used in response to an incoming writable properties. More details can be
 * found here:
 *
 * https://docs.microsoft.com/azure/iot-pnp/concepts-convention#writable-properties
 *
 * The payload will be of the form:
 *
 * **Without component**
 * @code
 * {
 *   "<property_name>":{
 *     "ac": <status_code>,
 *     "av": <version>,
 *     "ad": "<description>",
 *     "value": <user_value>
 *   }
 * }
 * @endcode
 *
 * **With component**
 * @code
 * {
 *   "<component_name>": {
 *     "__t": "c",
 *     "<property_name>": {
 *       "ac": <status_code>,
 *       "av": <version>,
 *       "ad": "<description>",
 *       "value": <user_value>
 *     }
 *   }
 * }
 * @endcode
 *
 * To send a status for properties belonging to a component, first call the
 * az_iot_hub_client_properties_writer_begin_component() API to prefix the payload with the
 * necessary identification. The API call flow would look like the following with the listed JSON
 * payload being generated.
 *
 * @code
 * az_iot_hub_client_properties_writer_begin_component()
 * az_iot_hub_client_properties_writer_begin_response_status()
 * // Append user value here (<user_value>) using ref_json_writer directly.
 * az_iot_hub_client_properties_writer_end_response_status()
 * az_iot_hub_client_properties_writer_end_component()
 * @endcode
 *
 * @note This API only writes the metadata for the properties response.  The
 * application itself must specify the payload contents between calls
 * to this API and az_iot_hub_client_properties_writer_end_response_status() using
 * \p ref_json_writer to specify the JSON payload.
 *
 * @param[in] client The #az_iot_hub_client to use for this call.
 * @param[in,out] ref_json_writer The initialized #az_json_writer to append data to.
 * @param[in] property_name The name of the property to write a response payload for.
 * @param[in] status_code The HTTP-like status code to respond with. See #az_iot_status for
 * possible supported values.
 * @param[in] version The version of the property the application is acknowledging.
 * This can be retrieved from the service request by
 * calling az_iot_hub_client_properties_get_properties_version.
 * @param[in] description An optional description detailing the context or any details about
 * the acknowledgement. This can be #AZ_SPAN_EMPTY.
 *
 * @pre \p client must not be `NULL`.
 * @pre \p ref_json_writer must not be `NULL`.
 * @pre \p property_name must be a valid, non-empty #az_span.
 *
 * @return An #az_result value indicating the result of the operation.
 * @retval #AZ_OK The JSON payload was prefixed successfully.
 */


feat: adapt code to use KeptnWorkloadVersion instead of KeptnWorkloadInstance

Fixes: #66

should we make a refactoring issue for this guy? https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&pullRequest=2255&id=keptn_lifecycle-toolkit&open=AYsU8dz3bEml1DOqvng7

should we make a refactoring issue for this guy? https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&pullRequest=2255&id=keptn_lifecycle-toolkit&open=AYsU8dz3bEml1DOqvng7

yep I guess we should

should we make a refactoring issue for this guy? https://sonarcloud.io/project/issues?resolved=false&types=CODE_SMELL&pullRequest=2255&id=keptn_lifecycle-toolkit&open=AYsU8dz3bEml1DOqvng7

yep I guess we should

https://github.com/keptn/lifecycle-toolkit/issues/2261


Missing shock calculation on Inpulsa

I just realize that shock damage wont affect the final calculation on Inpulsa's Broken Hearth Sadist Garb item.. Thanks GL n HF ! :)

Actualy shock effectiveness doesn't seem to be implemented right now

Hm, the current shock mechanic still isn't implemented as of now.  Was there any word on why this calculation has been delayed?


sds: remove Google CA clients

Change-Id: I0e62b071c1f459b7321eccd91804ed4f7a8d0a19
Google maintains a separate build for Istio that will host these custom CA providers.
CC @sergii-ssh

@kyessenov: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
4aab2588d259317145b7750c9eb7082552d75f60
link
true
/test release-notes

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-sigs/prow repository. I understand the commands that are listed here.

Fixes https://github.com/istio/istio/issues/50112

/retest

@kyessenov: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
4aab2588d259317145b7750c9eb7082552d75f60
link
true
/test gencheck

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-sigs/prow repository. I understand the commands that are listed here.

To be clear - these CAs aren't  deprecated on Google builds, only OSS builds. It's better for both - Google can continue maintenance and OSS can focus on the core.


Disagg: TiFlash may hangs when exiting

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)

2. What did you expect to see? (Required)
3. What did you see instead (Required)
When deployed under disagg arch, the non-owner TiFlash wn may hangs forever
4. What is your TiFlash version? (Required)
master

It is because that when EtcdOwnerManager::cancelImpl is called, client->campaign is still stuck on the RPC call and th_camaign.join() may never get return.


Added missing = to ||= assignment operator.

Just noticed that the rspec and rails method used the ||= operator and the ruby method didn't.

Thanks!


Revert back the tenantUUID change in the CSI driver

Description
Since #3124 the CSI driver provisioner already uses the new tennantUUID from the status, which was planned to be changed later. This leads to issues like #3533 and breaks CSI driver mounts.
This PR reverts the change.
Bug Ticket
How can this be tested?

Run cloudnativefs
Create some pods that are injected
Pods should start up again and be monitored

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 57.52%. Comparing base (6e5f9ba) to head (8fb8056).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3553   +/-   ##
=======================================
  Coverage   57.52%   57.52%
=======================================
  Files         384      384
  Lines       20776    20776
=======================================
  Hits        11952    11952
  Misses       7561     7561
  Partials     1263     1263

Flag
Coverage Δ

unittests
57.52% <100.00%> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Feature: Add provider functions (send_transaction_async, tx_status, chunk, block, validators)

This looks good looking at the json rpc client implementation and examples.
I will be merging this for now. However, I think it will be a good practice in the future to submit tests as well.


Update themes on re-provision

Not sure if this is desired, but seemed like a reasonable adjustment to make.

Well, I guess this would be nice for people who want to reference what's being done in current themes?

Or just to have an up-to-date version of Twenty-something available to compare code snippets with ;-)


Initial work

This is a PoC showing the backend's output displayed in a simple table. This table has back links to Umple Online and the Umple file generated. I need to open an issue against the back end to keep the original files in the output directory as well so this can be displayed.

I really recommend just looking at the home.html.php since that's all I changed. I realized I need to add the file output for where the original content is saved (I forgot to add it as output in the backend..). The umpleonline links will work with a non-localhost server. :)

I don't see the "public/*.php" files, they are coming?

"Sorry, we could not display the entire diff because it was too big."
At least github said sorry.  The files are there, just not in the summary

I might suggest creating a separate PR with just simpletest to reduce the changeset on this one

I rebased all of the changes, now the PR is much more manageable to read. master now has the original code before I started working against it. This PR just has my modified changes.

Opened umple-ucosp/umple.sample.downloader#5 to address the lack of "file to import" being present (i.e. the original ECore in this simple example).


Speech Separation

Hi,
Did you try to use this network for separating two speakers sources from a mixture of them?

Hello,
I have not tried yet but I guess it works since Deep Clustering does so.


jshint 2.7.0+ breaks Komodo Edit, lodash context

I'm using Komodo Edit 9.2.1 for Javascript projects, it ships with a very old jshint but has an option whereby you can point it to a newer version. All works fine upto jshint 2.6.3 but if I go to 2.7.0 or above then errors are thrown and jshint no longer works. A bit of investigation seems to show that as of 2.7.0 there was a change from underscore to lodash so I replaced lodash in jshint 2.8.0 with underscore 1.8.3 and all starts working again normally. It seems that there is some problem with lodash detecting the context in which it is executing in the Komodo environment. Can this be fixed in jshint? Perhaps move back to underscore? Does underscore need a fix? Perhaps Komodo is at fault? I have informed Komodo off the problem, but I'm not sure that the fault is theirs (https://github.com/Komodo/KomodoEdit/issues/603).

After speaking with jdalton at lodash, the issue seems to have been with the UMD loader in lodash that now appear to have been fixed in the trunk. So, waiing on a new release of lodash and jshint.

Nearly a year since I posted this. Still unable to use JSHint newer than v2.6.3 with Komodoedit. Package config shows lodash as version "3.7.x", current is "4.16.x"
Issue was supposedly fixed in lodash v4
https://github.com/lodash/lodash/issues/1493
I changed the version of lodash, replaced all _contains with _.includes, ran build and all tests passed. JSHint now runs in Komodoedit.
Any reasons for not updating? Thanks.


Implement RFC0038: SBOM

To implement Paketo RFC0038, this buildpack (and the implementation buildpacks inside) will need to move from storing SBOM information in layer metadata to storing it in files that the CNB lifecycle can manipulate during the build. The RFC outlines what these files are and what they should contain.
This issue serves as a meta-issue for work required to complete this work for the Web Servers language family. This will require (link Github issues as they are created):

[x] https://github.com/paketo-buildpacks/httpd/issues/311
[x] https://github.com/paketo-buildpacks/nginx/issues/369

I believe that this track of work is over. Is there anything else that needs to be done?

Closing this as all of the criteria has been fulfilled.


Feature Request: Add vnet_id as an output

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description
Is your feature request related to an issue?
No

We would like to use the vnet_id of any vnets that are generated from this vending module without having to use a data source.
Describe the solution you'd like

outputs.tf to include the vnet_id of any vnets it creates. At present it only outputs the subscription_id's. It looks like vnet_id is in the output.tf from the virtualnetwork submodule

Additional context
This will allow us to use this vending module to create a 'connectivity' subscription and then pass the vnet_id into any other subscriptions for vnet peering them back to the connectivity.

Hi @ChrisTav424
Thanks for the request - shouldn't be a problem for the v2 release

When is the V2 Due? There is no date on the milestone

Hi,
Now TF 1.3 is released I'll work on it in October.
It's quite a big change given the network complex object.
I will update the milestone


Add IMPLOT_AUTO*

These values are useful as they allow to only change specific elements and use defaults for others.

This cannot be fully implemented because of C# constants rules.
Added them to ImPlot class so they can still be used.
db5746a0a1872587de0d42c07668674ceb96ed9a


Fix paradox generated documentation by replacing target/mdoc with src/main/paradox

Fixes #240

Codecov Report

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

@@           Coverage Diff           @@
##           master     #241   +/-   ##
=======================================
  Coverage   83.85%   83.85%
=======================================
  Files         107      107
  Lines        1251     1251
  Branches       22       22
=======================================
  Hits         1049     1049
  Misses        202      202

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 4e5426f...c0198ef. Read the comment docs.


Build Service API for requesting a new build

This would allow you to e.g. make a workflow where you push to git, after which it runs steroids deploy immediately and then requests a new Ad Hoc build.

+1 on an API or any way to automate this, we're developers, we prefer automation over manual labour. :)

+1 on an API like PhoneGap Build api to intregate it on my build circle.

I guess we could do a mini version the API (=you could just request a build, but the configuration etc would still need to be done with browser) what other requirements would there be for a mini API?

At a minimum:

queuing up a build for a given cloud.json and a target runtime/platform
a curl-able url to the build file. The current one requires cookies from the browser login, that's not very handy for pulling and distributing builds.

If the API would also accept a callback url which can be invoked once a build succeeds/fails, that would be swell.  To get around this, I setup some Zapier hooks to parse the notification emails and trigger build hooks.  It's kludgy but it works. See my previous comment in this thread.

Hi i've made a small list of requests and requirements here.
http://daanluttik.nl/steroids/buildapi/

@matti Could we get an estimation of when this feature might be implemented.

@Luttik thanks! @pre please check this out.

Having an API would ease up my internal test builds also .. I currently request them manually one-by-one so I can feel your pain :dancers:
I'll move this up on the "this could be done next if the kids don't cry for icecream" list.

+1
@Luttik thanks for putting together the list (minor typos: patform ->platform; responce -> response)
My $.02 from guesswork:
From the way the Build Service currently works, I would think that updating version numbers would be independent of queuing a build, so two separate calls would match the underlying mechanic better.
An immediate build download link might be tricky as the url may only be defined on complete and upload, but a polling endpoint to wait for the build to complete that returns a build number when complete would be great.
A possible (proposed) conversation is then:

update version number
deploy (already implemented)
queue build
poll, poll, poll
download binary
do magic (...)

Of these, the highest value for me is from step 3, then step 1.
@pre looking forward to it... let's hope the kids don't cry for icecream :)

You're totally right @campey.
Splitting version change and build request would be the logical way to implement.

+1 for kicking off a deploy and a build. It would be great if one could do:
steroids deploy
steroids build scanner
steroids build adhoc

If the build command would kick off a build and sleep, poll every minute to see if it's done, and download it when done, that would be awesome.
@pre I would not mind doing the steroids pieces if someone at appgyver did the build side pieces.

That would be a convenient option however for us the primary goal of the build  API is to allow remote build automation from systems that do not/cannot have the  steroids cli installed.
Typical triggers would be changes to native plug-ins, commit to a release branches for ad hoc or App Store builds.  For the latter, it would require solving auto incrementing version & build numbers at the api level.
But as you say, with the api in place, the cli would be another client.
Sent from my mobile phone.

On Feb 19, 2015, at 9:14 AM, hir3npatel<EMAIL_ADDRESS>wrote:
+1 for kicking off a deploy and a build. It would be great if one could do:
steroids deploy
steroids build scanner
steroids build adhoc
If the build command would kick off a build and sleep, poll every minute to see if it's done, and download it when done, that would be awesome.
@pre I would not mind doing the steroids pieces if someone at appgyver did the build side pieces.
—
Reply to this email directly or view it on GitHub.

+1
Not having command line/API level control over build is a major turn off when you're trying to iterate quickly e.g. updating versions, plugins or even icons.
I'm guessing whoever is seriously using the build service has already scripted this through some page scraping, reverse engineering and email hooks and just not sharing :)
Also, consider adding ability to configure multiple build configs of the same type e.g. two ad-hoc builds with different types of web views. This should include having separate plugin configs e.g. we don't want TestFairy iOS plugin in our app store builds.
I can certainly work around this by deploying them as separate apps at the expense of having to update a lot more configurations.

@pre  @matti, Can we have an update on this issue? Possibly an estimation for the releasedate of the API.

sorry, no estimates at this point. it's still at the very top of our backlog though.

+1 as this feature will make for nice integration with our companies internal build server.

starting work on this right now!

@matti That is awesome! This is a big one for my app at my company to be able to automate our build process. Kudos.

Thats great. I do really hope you'll add version changin into the api aswell.

Any luck on this? Thanks.

Still on the backlog, unfortunately. We'll of course make the API sensible to include version bumps etc.

+1

any news on this yet ???

Just a suggestion, perhaps this tweak can make life a little better for automation.
I'm trying out the IFTTT (now IF) to automate the download file to dropbox when Build service is completed successfully.
However the issue is the link provided requires user to login before download. Is it possible to include a hash token to the download url to download without logging in?

What happened to this?!? Did everyone just give up? In this day and age, you assume you can automate everything thru proper channels. This feature is a huge time-saver and needs to be addressed... Please attend!


fix 'model' variable added to global namespace

A quick fix to stop the 'model' variable from being added to the global namespace.

Thanks for the pull request, your changes is in v0.8.2 and published to npm


mismatch in example snippet and arguments section

One of the arguments in the "Yaml snippet" example code is "azurePowerShellVersion"
In the arguments section, its listed as "TargetAzurePs" which causes confusion when trying to review the required arguments
Please can this be reviewed and corrected - Thanks! :)

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

ID: dcda4373-62ae-e1ae-c15d-63f441179b47
Version Independent ID: 305f78c0-2e89-78f2-d2d1-89471602610c
Content: Azure PowerShell task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-powershell.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

They are both correct due to aliases. This issue is fixed in our new task reference, which is generated using the task source code and includes all task inputs, including any aliases here:
Please check it out and let us know what you think. Thank you.

https://learn.microsoft.com/en-gb/azure/devops/pipelines/tasks/reference/azure-powershell-v5?view=azure-pipelines


Add Saint-Adjutor school

Institution/School Name
Lycée Saint-Adjutor
Instiution/School Website
https://www.stadjutor.com/
Email Users
<EMAIL_ADDRESS>Please place an x between the square brackets if yes

[X] Students
[X] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[X] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[ ] High School or equivalent
[ ] Elementary School or equivalent

@ImadBilS First stage of verification passed, official domain is stadjutor.com as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@ImadBilS Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Getting black image!

First of all thank you very much for this.
When I try to run your example code with these two images, I always get a black image!

This is the whole code which is given in the first page , I just replaced the random numpy array statement with these two images!:
import imgaug as ia
from imgaug import augmenters as iaa
import numpy as np

im = caffe.io.load_image('buckskin_s_000331.png')
im2 = caffe.io.load_image('buckskin_s_000005.png')
images = np.zeros([2,32,32,3])
images[0] = im
images[1] = im2
# Sometimes(0.5, ...) applies the given augmenter in 50% of all cases,
# e.g. Sometimes(0.5, GaussianBlur(0.3)) would blur roughly every second image.
st = lambda aug: iaa.Sometimes(0.3, aug)

# Define our sequence of augmentation steps that will be applied to every image
# All augmenters with per_channel=0.5 will sample one value _per image_
# in 50% of all cases. In all other cases they will sample new values
# _per channel_.
seq = iaa.Sequential([
        iaa.Fliplr(0.5), # horizontally flip 50% of all images
        iaa.Flipud(0.5), # vertically flip 50% of all images
        st(iaa.Superpixels(p_replace=(0, 1.0), n_segments=(20, 200))), # convert images into their superpixel representation
        st(iaa.Crop(percent=(0, 0.1))), # crop images by 0-10% of their height/width
        st(iaa.GaussianBlur((0, 3.0))), # blur images with a sigma between 0 and 3.0
        st(iaa.Sharpen(alpha=(0, 1.0), strength=(0.75, 1.5))), # sharpen images
        st(iaa.Emboss(alpha=(0, 1.0), strength=(0, 2.0))), # emboss images
        # search either for all edges or for directed edges
        st(iaa.Sometimes(0.5,
            iaa.EdgeDetect(alpha=(0, 0.7)),
            iaa.DirectedEdgeDetect(alpha=(0, 0.7), direction=(0.0, 1.0)),
        )),
        st(iaa.AdditiveGaussianNoise(loc=0, scale=(0.0, 0.2), per_channel=0.5)), # add gaussian noise to images
        st(iaa.Dropout((0.0, 0.1), per_channel=0.5)), # randomly remove up to 10% of the pixels
        st(iaa.Invert(0.25, per_channel=True)), # invert color channels
        st(iaa.Add((-10, 10), per_channel=0.5)), # change brightness of images (by -10 to 10 of original value)
        st(iaa.Multiply((0.5, 1.5), per_channel=0.5)), # change brightness of images (50-150% of original value)
        st(iaa.ContrastNormalization((0.5, 2.0), per_channel=0.5)), # improve or worsen the contrast
        st(iaa.Affine(
            scale={"x": (0.8, 1.2), "y": (0.8, 1.2)}, # scale images to 80-120% of their size, individually per axis
            translate_px={"x": (-16, 16), "y": (-16, 16)}, # translate by -16 to +16 pixels (per axis)
            rotate=(-45, 45), # rotate by -45 to +45 degrees
            shear=(-16, 16), # shear by -16 to +16 degrees
            order=ia.ALL, # use any of scikit-image's interpolation methods
            cval=(0, 255), # if mode is constant, use a cval between 0 and 255
            mode=ia.ALL # use any of scikit-image's warping modes (see 2nd image from the top for examples)
        )),
        st(iaa.ElasticTransformation(alpha=(0.5, 3.5), sigma=0.25)) # apply elastic transformations with random strengths
    ],
    random_order=True # do all of the above in random order
)

images_aug = seq.augment_images(images)
plt.imshow(images_aug[0])
plt.show()

what is wrong here?

OK, I ran some more tests individually on each of those options, it seems onlyiaa.Fliplr(0.5)and  iaa.Flipud(0.5) work. the rest result in a black image which I posted above!
What am I missing here?!

caffe's load_image seems to return floats in the range 0-1.0, while the library usually expects uint8 in the range 0-255. So you would have to do something like (img*255).astype(np.uint8) after loading the images.
Considering your image size of 32x32, the number of augmenters is also rather large. You should probably decrease each augmenters probability by changing st = lambda aug: iaa.Sometimes(0.3, aug) to st = lambda aug: iaa.Sometimes(0.1, aug).
Further, you should use something like seq.show_grid(images[0], rows=8, cols=8) to show your augmenters results instead of plt.imshow(images_aug[0]), as the latter one will result in the same augmentation at every script run.

@aleju : Thank you very much. The black image issue is gone.
and speaking of seg.show_grid(), it fails for me with this error RuntimeError: Could not execute image viewer.
I could'nt find a way  to solve it, I have no idea whats causing this! so thats why I used the plt.imshow instead

The threads that I can find sound like the command see must be available for misc.imshow to work (or an image viewer must be set in the environment variable SCIPY_PIL_IMAGE_VIEWER). I guess that command isn't available in Windows 10. Maybe I'll switch to OpenCV's imshow equivalent.

I installed see through pip yet I still get the same error.

I meant a command line program see, not a pypi package with that name, i.e. you must be able to type see in the command line and not get an error (that's how I understood it from the help threads - the program seems to be available on Ubuntu by default).

how to install the see program over mac os