Automate invitation bootstrap in development

@ianco I believe you picked this one up and accomplished this in the issuer controller. Feel free to reopen if it still needs to be completed.


Qwen2.5-Coder-7B-Instruct 2024/11/06 update

https://huggingface.co/Qwen/Qwen2.5-Coder-7B-Instruct/commit/9092a8ae57da39f15b76b309b4f71ff11b6ef01a
They just updated the weights of Qwen2.5-Coder-7B-Instruct without any announcement, so I don't know what exactly what the update is, but it's an update

They were just waiting to make sure Kamala lost and now they can make Ameriqwen great again

I do not like this clandestine update,
These changes made by stealth as of late!
No changelog here, no notes to see,
What have they changed? It's mystery!
"Just trust the code," they say with glee,
But WHAT changed? They won't tell me!
Not a comment, not a line,
Just "updates" dropped at runtime!
Did they tweak the matrix math?
Or reroute the data path?
Did they change the learning rate?
No one thought to annotate!
Silent pushes in the night,
Leave me in an awful fright.
For how can I debug with care,
When documentation isn't there?
Oh, the frustration! Oh, the pain!
Of updates dropped without explain!

I'm not sure why the Qwen folks decided not to change the version number, but the new models are in the model library to use. You'll need to ollama pull qwen2.5-coder to get the latest version.
I'll go ahead and close the issue.


Fixing environment variable clustername for CLI

Bug fix: Incorrect environment variable for cluster name used in Azure CLI when deploying the cluster as well as when cleaning up.

@suhasaraos : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@rahulm23
Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@johnmarco - Can you review this old PR and determine whether it needs to be closed or merged?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#sign-off


rune inspect crashes on WSL

Details here: https://discord.com/channels/843552361212805180/872328451728748544/888625888806187118
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace```

This is the actual error message
thread 'main' panicked at 'The project uses version control', /home/stephx/.cargo/registry/src/github.com-1ecc6299db9ec823/hotg-rune-cli-0.7.0/src/inspect.rs:56:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

It looks like we assume hotg-rune-cli was installed with a git repository, but crates.io doesn't keep the .git folder around.


NavigationControllerIOS set new root component

I ran into this use case while working towards converting my project. Looks like this isn't currently supported, but it shouldn't be too hard to implement.
Thoughts?

hmm.. Not sure I understand
It is currently possible to make NavigationControllerIOS as the only root of your application. Just remove all the others in the example project. This will leave you with a single screen app (no tabs, no side menu) that will have a navigation stack (you will be able to push and pop). If we were to change the example index.ios.js to this, this would be the result: https://gist.github.com/talkol/d71f176bf49bfbf25d0e
It is also possible to change application roots in runtime. Meaning start with one root and then change to a different root. We currently have this use-case in our production app where we switch between login state and logged-in state. They both have different roots. The app starts with the login root, and after you login, it changes root to the tab bar root (logged in). We do this by simply calling ControllerRegistry.setRootController('NewRoot');. In version 1.1.0 we've even added animations to make this transition. See the 2nd parameter for setRootController.
Or did you mean something else entirely and I've misunderstood :) If this is the case, please give me an example use-case.

Sorry, I should have been more specific. My use case involves using the drawer to switch sections of the app. Since I would like to keep the existing Drawer / NavigationController structure intact, I only need to switch the controller at the root of the NavigationController - not the root of the entire app.
With a UINavigationController, this can be accomplished with:
[navigationController setViewControllers:@[yourRootViewController] animated:NO];
I submitted a working pull request for you to take a look at. Let me know what you think

Resolved with #12

also added this feature to https://github.com/wix/react-native-navigation
and added examples to both projects
thanks!


feat: allow custom group.id and application.id

Addresses #9 and cleans up documentation.

Codecov Report

Merging #25 into master will increase coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master      #25      +/-   ##
============================================
+ Coverage     79.15%   79.22%   +0.06%
- Complexity      230      233       +3
============================================
  Files            50       50
  Lines           974      977       +3
  Branches         56       57       +1
============================================
+ Hits            771      774       +3
  Misses          174      174
  Partials         29       29

Impacted Files
Coverage Δ
Complexity Δ

...itops/domain/state/service/ApplicationService.java
100.00% <100.00%> (ø)
6.00 <0.00> (+1.00)

...tops/domain/state/service/KafkaConnectService.java
100.00% <100.00%> (ø)
5.00 <2.00> (+1.00)

...tops/domain/state/service/KafkaStreamsService.java
100.00% <100.00%> (ø)
6.00 <2.00> (+1.00)

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 04d35ff...4c4b401. Read the comment docs.


templates: use yq's indentation style

Fix #1389 so that we can eventually automate updating yamls (#1347) using yq.
Use git show --ignore-all-space --ignore-blank-lines to review this commit.
diff --git a/.yamllint b/.yamllint
index a7c8c3f..20183f4 100644
--- a/.yamllint
+++ b/.yamllint
@@ -3,8 +3,6 @@
 extends: default

 rules:
-  indentation:
-    indent-sequences: false
   truthy:
     allowed-values: ['true', 'false', 'on', 'off']
   comments-indentation: disable

Since @jandubois feels strongly about the indentation, I would rather patch go-yaml.
There really should be a preference in yq, to allow both type of sequence indent...


Shortcut guide and Remapping key don't work on 106 key keyboard

Environment
Windows build number: [run "ver" at a command prompt] Microsoft Windows [Version 10.0.18363.836]
![106 key2020-05-29 150010](https://user-images.githubusercontent.com/66047423/83226520-24696800-a1bd-11ea-9c80-b193961be428.png)

PowerToys version: 0.18.1

PowerToy module for which you are reporting the bug (if applicable):
Keyboard Manager

Steps to reproduce

I used 106 key keyboard because I am living in Asia. When I press the hanja or Hanyoung Key(They are on the both side of the space bar in 106 key keyboard, they are language change buttons for non English users ), Shortcut guide(long window key) and Remapping Key didn't work as if it's completely shut down. Fancy zone and the other stuff is alive in this case.
If I quit the Powertoys and re-open, or I lock the computer and unlock, it works again.
FYI I have attached 106 key keyboard.
Expected behavior

Actual behavior

Screenshots

I used 106 key keyboard because I am living in Asia. When I press the hanja or Hanyoung Key(They are on the both side of the space bar in 106 key keyboard, they are language change button for non English users ), Shortcut guide(long window key) and Remapping Key didn't work as if it's completely shut down. Fancy zone and the other stuff is alive in this case.
If I quit the Powertoys and re-open, or I lock the computer and unlock, it works again.
FYI I have attached 106 key keyboard.

Possibly related to #3397

Can't reproduce with 106-key hangul keyboard. Have you tried running in Administrator mode? Might be fixed in 0.18.2.
Video

@manofaciton0
is this still a problem with 0.27.1?

@manofaciton0
is this still a problem with 0.27.1?

No It seems to be solved!😀😀

@manofaciton0
thanks for the follow up, I'll close the issue.


RetailUpdateDatabase

Hi, is above article still valid? For version 10 of on-prem retail.
I tried to find RetailUpdateDatabase.ps1 script - but it does not exist neither in binary package, nor anywhere on AOS machine.
Also D365 is able to find retail packages without selfservicepackages folder creation and any steps described in 1-6 steps. Looks like it uses standard folder C:\RetailSelfService\Packages
Thank you!

Dokumentinformasjon
⚠ Ikke rediger denne delen. Den kreves for koblingen docs.microsoft.com ➟ GitHub-problem.

ID: ecc51933-08be-36c7-8f82-d9df6f220bfa
Version Independent ID: 29846b09-d0e5-9076-d451-0f86429a778a
Content: Installation steps for Retail channel components in an on-premises environment - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/deployment/deploy-retail-onprem.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @jashanno
Microsoft Alias: jashanno

@OksanaKovaliova, thanks for posting this!
@jashanno, please review and update the content as needed.

Bug 371167 has been logged to track this issue.


NR-304039: absorb title from AVPlayer metadata

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@avinash-newrelic please also update the version. Now is 2.0.2, with this change it should be 2.1.0 now.

@asllop  I have updated the version from 2.0.2 to 2.1.0, can you please review and approve it.


[WIP?] ispc: use libcxxStdenv

Motivation for this change
fixing ispc build.
Related to #28643
Things done

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

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

This is broken still, work in progress.

Edited the caption — that way you can change it back your self and it will be visible in the summary issue.


WIP: Augmented Bond Curves

ABCs are cool! This will try and implement much of what is in this excellent blog post.
Also, using Token Factory!

doing G-d's work, Ser! 👏

Codecov Report
Patch coverage: 55.02% and project coverage change: -1.77 :warning:

Comparison is base (9261bb3) 93.82% compared to head (1ac45a7) 92.05%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #697      +/-   ##
==========================================
- Coverage   93.82%   92.05%   -1.77%
==========================================
  Files          60       66       +6
  Lines        5149     5415     +266
==========================================
+ Hits         4831     4985     +154
- Misses        318      430     +112

Impacted Files
Coverage Δ

contracts/external/cw-abc/src/commands.rs
0.00% <0.00%> (ø)

contracts/external/cw-abc/src/contract.rs
46.00% <46.00%> (ø)

contracts/external/cw-abc/src/abc.rs
54.38% <54.38%> (ø)

contracts/external/cw-abc/src/queries.rs
78.57% <78.57%> (ø)

contracts/external/cw-abc/src/curves.rs
100.00% <100.00%> (ø)

contracts/external/cw-abc/src/state.rs
100.00% <100.00%> (ø)

... and 9 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

For the cw-curves package it may be worth looking and the Wynd DAO implementation: https://github.com/wynddao/wynddao/blob/main/packages/utils/src/curve.rs
Probably some useful stuff in there. Would be nice to have a solid curves package.


Access to postgresql servers

Request from @vvcb:
I would like to be able to access LSCSDE postgresql servers from the LTH network segments.
DNS Conditional Forwards should account for the way that private link works, so when performing a lookup internally for postgres server it should provide the local address rather than the remote one:
So:

Instead of:

Firewall needs to allow access to port 5432 from the LTH network.
Current postgresql server are:

lscsdesandbox-psql <IP_ADDRESS>
lscsdeprod-psql <IP_ADDRESS>
lscsdedev-psql <IP_ADDRESS>

However these addresses may be subject to change in future

I will pass through our cyber team for review and will get back to you Shaun

@alex-maddocks Please could I get an update on this one?

Apologies for the delay, i have been on annual leave. Conditional Forwarders and AZ Firewall updated. Please test and let me know.

@PaulWoodhouseLTH , does nslookup resolve the correct addresses for the private endpoints for those servers now?
I will test access through the firewall when I am back next week but it will be good to get the routing correct first.

Yes it resolves correctly to the local address.

Linking to lsc-sde/lsc-sde#38

@vvcb just sending through a reminder that this needs to be tested


MFA enforcement should remain as opt-in feature in IAM groups, as such enable_mfa_enforcement should default to false.

Description
#313 introduces undocumented change to force MFA for all console and API user. A similar issue #332 was created, and a supposed fix #333 was released in version 5.14.3, introduces new enable_mfa_enforcement variable which defaults to true, which causes IAM groups with attach_iam_self_management_policy set to true still inadvertently opts in to MFA enforcement due to this line:
https://github.com/terraform-aws-modules/terraform-aws-iam/blob/89fe17a6549728f1dc7e7a8f7b707486dfb45d89/modules/iam-group-with-policies/policies.tf#L157
In my opinion, MFA enforcement should be an opt in feature. Users migrating from older version (example version 5.3.1) to current version may find that IAM groups users with attach_iam_self_management_policy set to true suddenly lose access to their consoles, which can cause more harm than good.
If the maintainers agree, I am willing to submit a PR along with clear documentation to ensure that the implications of enabling MFA enforcement are made clear.
Versions

Module version [Required]: 5.43.0

Terraform version: 1.3+

Provider version(s): 5.x.x

Reproduction Code [Required]
module "iam_group_test" {
  source  = "terraform-aws-modules/iam/aws//modules/iam-group-with-policies"
  version = "5.43.0"

  name                              = "test-group"
  attach_iam_self_management_policy = true

  group_users = [
    module.iam_testuser.iam_user_name,
  ]

  custom_group_policy_arns = [
    "arn:aws:iam::aws:policy/AdministratorAccess",
  ]
}

module "iam_testuser" {
  source  = "terraform-aws-modules/iam/aws//modules/iam-user"
  version = "5.43.0"

  name                          = "testuser"
  create_iam_user_login_profile = true
}

Steps to reproduce the behavior:

Create iam-group-with-policies, with attach_iam_self_management_policy set to true, without specifying enable_mfa_enforcement (left as default)
Attach user to the group.
Login as user via console and try access any services.

Result:

AccessDenied error due to MFA being enforced

Expected behavior
enable_mfa_enforcement should be default to false
Actual behavior
enable_mfa_enforcement is currrently defaults to true, as shown in this line https://github.com/terraform-aws-modules/terraform-aws-iam/blob/89fe17a6549728f1dc7e7a8f7b707486dfb45d89/modules/iam-group-with-policies/variables.tf#L40

/keepalive


Fix for incorrect PArrived results

When the number of recv-side partitions is different from the number of actual partitions used (i.e. the number of send and receive requests underneath), MPIX_Parrived would incorrectly calculate which underlying requests it needed to check, often just checking the 0'th index. MPIX_Wait is unaffected by this bug since it always just uses the number of requests.
This PR fixes this bug by adding a new function to properly do the conversions from "user partition ID" to "network partition ID(s)". I also changed the send side to make use of this function. Lastly, I added a new test that can be used to verify MPIX_Parrived is working by using it to make progress; this test will never complete if MPIX_Parrived does not return true for all partitions.
Indepth summary changes:

examples/Basic/CMakeLists.txt - Added new test
examples/Basic/mpipcltest7.c - The new test described above. Run instructions are included in the file. Requires 2 processes.
include/mpipcl.h - Removed unused functions; Removed the MPI_Info variable from a helper function (it was not used)
src/pt2pt/mpipcl.c - Removed unused MPI_Info object from prep function (can always be added back if needed)
src/pt2pt/send.c - Bulk of changes; Added new helper function map_local_partition which maps user partition ID to network partition ID. Updated general_send to no longer just call another function (moved that function into general_send). Updated general_send and map_recv_buff to use new map_local_partition function. Removed unused "send mapping" functions.
src/pt2pt/setup.c - Removed unused MPI_Info object from prep function (can always be added back if needed)
src/pt2pt/sync.c - Added extra debug

@tonyskjellum Tagging you so you see this too.


Format on Save?

This might be a question for how to set up Vscode more so than this extension, but I've had an annoying thing where the sass formatter interrupts my workflow while I'm typing. I'd love to only have it run on save instead of keyup. How to do this?

Please provide me following details:

Operating system version
VS Code version
User and workspace settings (see below)

And I'll try to reproduce issue you described above.
To copy/paste your user and workspace settings use the menu command File > Preferences > Settings (Code > Preferences > Settings on Mac).

I'll close the issue due to inactivity, but feel free to re-open if needed.


Added missing using to NewsArticle

File uses JsonPropertyAttribute but doesn't include Newtonsoft.Json. This adds it in.

As a side note, this file isn't updated on Nuget either. Should I make an issue for it?


helm lint fails when global values are used in subcharts

When using global chart values, helm lint complains about the global key not being present in the subchart, even though it's supposed to be reserved and available to subcharts. Actually building the chart works, it's just the linter that complains. The error is like this:
[ERROR] templates/: template: <chart>/templates/<filename>.yaml:<line>:<char>: executing "<chart>/templates/<filename>.yaml" at <.Values.global.foo>: map has no entry for key "global"

Output of helm version:
version.BuildInfo{Version:"v3.2.4", GitCommit:"0ad800ef43d3b826f31a5ad8dfbb4fe05d143688", GitTreeState:"clean", GoVersion:"go1.13.12"}

Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"13", GitVersion:"v1.13.12", GitCommit:"a8b52209ee172232b6db7a6e0ce2adc77458829f", GitTreeState:"clean", BuildDate:"2019-10-15T12:12:15Z", GoVersion:"go1.11.13", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"12", GitVersion:"v1.12.7", GitCommit:"6f482974b76db3f1e0f5d24605a9d1d38fad9a2b", GitTreeState:"clean", BuildDate:"2019-03-25T02:41:57Z", GoVersion:"go1.10.8", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.): n/a

Adding empty values in the subchart keeps helm lint happy:
global:
  foo: ""

Can you try this with v3.3.0-rc.1? I tried to replicate the problem there and was unable to.

Can you try this with v3.3.0-rc.1?

helm version
version.BuildInfo{Version:"v3.3.0-rc.1", GitCommit:"5c2dfaad847df2ac8f289d278186d048f446c70c", GitTreeState:"dirty", GoVersion:"go1.14.4"}

helm lint still fails if I use --with-subcharts, or lint the sub-chart directly.
I can reproduce this with v3.1.3, v3.2.4 and now v3.3.0-rc.1.
My specific use-case is that the parent chart has this in values.yaml:
global:
  hostname: foobar.com

The sub-chart then uses global.hostname in a template, which works due to the special behaviour of the global key, as documented. However, helm lint complains about this, as per my original report, unless I add this to the sub-chart's values.yaml:
global:
  hostname: ""

I reproduced this issue, as @allanlewis said, only need to specify --with-subcharts when helm lint is required to recreate it. I can try to follow up on this issue and propose a pr solution
/assign
the ref https://github.com/helm/helm/pull/7288

it still fails on helm 3.3.4

helm version
version.BuildInfo{Version:"v3.4.2", GitCommit:"23dd3af5e19a02d4f4baa5b2f242645a1a3af629", GitTreeState:"dirty", GoVersion:"go1.15.5"}

I have a similar issue when using dependency charts.
I experience a similar error ("Additional property global is not allowed"), unless I modify the values.schema.json of the dependency chart. I set "additionalProperties": true for the root object of values.schema.json of the dependency chart. This is not a proper solution as it is not applicable when using dependency charts managed by others.

helm version
version.BuildInfo{Version:"v3.4.2", GitCommit:"23dd3af5e19a02d4f4baa5b2f242645a1a3af629", GitTreeState:"dirty", GoVersion:"go1.15.5"}

I have a similar issue when using dependency charts.
I experience a similar error ("Additional property global is not allowed"), unless I modify the values.schema.json of the dependency chart. I set "additionalProperties": true for the root object of values.schema.json of the dependency chart. This is not a proper solution as it is not applicable when using dependency charts managed by others.

Still facing this issue with latest Helm version. Can this issue be reopened?

I add "global": {} to the properties of the root object of values.schema.json of the dependency chart, and then it works.

BTW this only works if your subcharts are not trying to access nested objects within global. If you have a subchart that's trying to access something like global.image.tag, you also have to set:
global:
  image: {}

Which can be very boring depending on the chart. The PR below fixes the issue:

https://github.com/helm/helm/pull/8514

And it was very simple. Too bad it never got merged.


minionTimeout should be global in Builds.java

the current logic for local client with wait for completion in Builds.java will caused :
when there are some minions already done,
and some are not, and they will run for much longer.
in this siuation, We will change the "minionTimeout" setting in jenkins global configuration, which we thought this is a global timeout to a larger value.
and we thought plugin will continue poll job status with the interval of poll time which we defined in job configuration.
and then job will failed due to minionTimeout reached.
but actually plugin will directly wait for minionTimeout when not all minions had be done.
so it will wait for long time to job complete, even it will not take so much time for every time.

Greetings! I'm not completely clear on what you are requesting.
There are two timeouts:

The first, "poll time" controls how often jenkins will check in with the saltapi during a long running job. This can be set to something short (like 5 seconds) if you have a job that completes somewhat quickly. If you have a job that you know will run for an hour, checking every 5 seconds is a bit to frequent. In this case, you might want to have the saltapi plugin poll every minute or 5 minutes. The polltime is set at a global level in the Jenkins configuration screen. This value can be overwritten within the individual local job. That way if you have lots of short lived jobs, you can set the global poll time to something short. But for a single long running job, you are able to set a more sane setting on that single job.

The second timeout, "Timeout", is set from the global Jenkins configuration screen. This value is new with the 2.0 release. If you have a job that should be run on multiple minions and only some minions return, this will set how long you will wait for the remaining minions to come back before assuming them offline and marking the build as a failure. This is helpful as without the timeout, the jenkins salt plugin will wait forever for the possibly non-existent minions to return. The default setting for the salt cli is 30 seconds. There was a bug in the 2.0 release where this timeout was always running. This activity was corrected with the 2.0.1 release.

It sounds like you may have been running into the minionTimeout bug from the 2.0 release. Does updating to 2.0.1 correct the behavior?

Hi, mchugh19
Thanks for your explanation.
For first type timeout, It's very clear. What I confused it is the second type timeout.
With your current timeout design, for example.
I have a salt job which run on three minions. we notice that two minions will return fast, but one minion will run much longer maybe due to need for system provision for once.
So you can not always assumed this minion will run for such long time.
Based on the example, we may defined the timeout as bottom.
First type timeout = 10s
Second type timeout = 30mins (for once long run minions.)
As the result, we will wait for 30mins with high probability, or always wait for 30mins if average execution time is above the first type timeout.

How about if the plugin allows you to specify the minion timeout for the individual job as well? That way, for jobs where you expect that some minions might not return quickly, you can allow that in the job.

Version 2.1.0 was just released. The minionTimeout attribute is now able to be set per job. If you have any problems or questions feel free to re-open.


[Client] Fix memory leak when message payload processor is configured

Motivation
When MessagePayloadProcessor is configured, there will be a memory leak caused by unused MessagePayloadContextImpl#recyclerHandle.
Modifications
Call recycle() to recycle the MessagePayloadContextImpl object in recycle() method.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.

@codelipenghui I think the behavior depends on the implementation of Netty recycler.
BTW, I have sent an email to discuss about if there is any way to detect unused variables. If we can detect the unused private fields in CI, this bug could be exposed in time.

BTW, I have sent an email to discuss about if there is any way to detect unused variables. If we can detect the unused private fields in CI, this bug could be exposed in time.

@BewareMyPower Good point!


to_categorical num_values issue

Why does the below code return an error (below trace):
cols = [8, 4, 6, 3, 5, 9, 7, 2]
arr = to_categorical(cols, num_classes=len(cols))

The point here is that it does not accept any value below 10. Then if I add numbers to the list, for example so that there are 11 values, now it only accept 16.
It seems useful to be able to actually decide the number of num_classes to be exactly the number of unique values in the array.

---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-217-223c25687e9e> in <module>()
      1 cols = [8, 4, 6, 3, 5, 9, 7, 2]
----> 2 arr = to_categorical(cols, num_classes=len(cols))

~/dev/a_test/lib/python3.6/site-packages/keras/utils/np_utils.py in to_categorical(y, num_classes)
     29     n = y.shape[0]
     30     categorical = np.zeros((n, num_classes), dtype=np.float32)
---> 31     categorical[np.arange(n), y] = 1
     32     output_shape = input_shape + (num_classes,)
     33     categorical = np.reshape(categorical, output_shape)

IndexError: index 8 is out of bounds for axis 1 with size 8

ah ok I understood it already, it's done based on the max value so that it can be reversed back with argmax() ... my bad. Closing.

Can you explain what is the problem and how to fix it I am having the same error?


updates new vpc acceptance var to a string value

saasherder in app interface does not like bools, this updates the ENABLE_VPC_ACCEPTANCE var to be a string value, so that A-I can use strings as well. This matches similar method used in AAO as well. The code does not need to be changed to use a string as AAO still uses a bool in the config as well

/lgtm
/approve


[WIP] Put contributing guide(s) on kubernetes.io

This is a...

[X] Feature Request
[ ] Bug Report

Problem:
All codebase and community contributor guides, documents, and relevant materials are scattered in the community github repo. A recent survey of 110 participants concluded a pattern that contributors (new and others) need a central point where other k8s documents are located for a smoother flow and easier to locate. Focus groups are also ongoing with current contributors about their experience.
Proposed Solution:
Put all updated and relevant contributor information that is located in the community repo in multiple files into a single displayed source on kubernetes.io. This would fall under "Contributing to the Kubernetes Docs" with the title "Contributing to the Kubernetes Codebase & Community"
Page to Update:
https://kubernetes.io/docs/home/
will also 'live' on the user journeys/personas piece for 'code contributor'
Additional Info

this will have a max of 8 sections (per @chenopis advice)
a Developers Guide will be one of those sections; this is being worked on by another team (RyanJ at RH is point)
@castrojo  and I will run this through SIG-ContribX; join our bi-weekly discussions/mailing list
need docs for final edits
This is our raw document that we are working from -link. We will be trimming duplicate info, condensing, and adding information so that it's not 65 pages ( :) )

Initial PR for starting point:
https://github.com/kubernetes/community/pull/1409

Umbrella issue to flesh out the README and surrounding folder in PR above: https://github.com/kubernetes/community/issues/1413

/remove-lifecycle stale

/lifecycle frozen

I believe this is fixed by #9510

/reopen
/cc @mrbobbytables
This issue is about the overall k8s contributor's guide, not the documentation contribution guide. We're close to finishing this, @mrbobbytables can you update this issue when we go live?

@castrojo sure thing! Shouldn't take too long once the cname record is in place.

The cname record is put in place, netlify configured and the site is now live! :)

@mrbobbytables what is the url for site you mentioned above ?

Sorry for the delay in response, the project is over in https://github.com/kubernetes-sigs/contributor-site. As all work is going on over there. I think it's safe to close this issue :) Can be reopened if needed.
/close


Add ability to map  and  separately like XTerm or iTerm2 have done

Is your feature request about something that is currently impossible or hard to do? Please describe the problem.
Currently we can't remap for example <TAB> and <C-I> separately, they both always do the same thing.
Describe the solution you'd like
Would be nice if these, and other control keys (<CR> and <C-M>, <BS> and <C-H> etc.), could be mapped separately.
Additional context
<TAB> and <C-I> can already be mapped separately in a Vim 8.2 running XTerm using modifyOtherKeys https://invisible-island.net/xterm/modified-keys.html or in iTerm2 when 'Control sequences can enable modifyOtherKeys mode' has been enabled from Preferences -> Profiles -> Keys.
One can easily test if the keys get registered separately with:
nnoremap <C-I> :echom "Ctrl-I was pressed"<CR>
nnoremap <TAB> :echom "Tab was pressed"<CR>

Yeah this would be a good idea. I can look this up but do you know if gVim also supports this on Windows/Linux?

Yeah this would be a good idea. I can look this up but do you know if gVim also supports this on Windows/Linux?

Just checked on Windows and at least the out-of-the-box gVim 8.2 does not differentiate those keys.

I have verified that gVim 9.1 on Linux/Windows does differentiate between  and  mappings.


upgrade the svelte static adapter

fixes #206
It looks like main doesn't build, and this is probably because of an outdated adapter version.

This seems to actually work fine? Must have gotten my wires crossed.


Function failing for "Oops! An unexpected error occurred: java.lang.NullPointerException"

Issue description:
We have spring cloud function deployed to azure function app. The project was built with Gradle and released from DevOps.
The function is failing for error "Oops! An unexpected error occurred: java.lang.NullPointerException". Another deployment might fix this issue without any code modification.
Info:
We have the same project deployed to 2 function apps via the exact deployment pipeline and created a working and another non-working scenarios.
I would like to share the function urls, but urls contain function keys so not sure if it is a good idea to show here. I can also share the sample project that could reproduce this issue. So if needed, please let me know.
Since this issue could be mitigated without changing the source code, we suspect that it is related to the Spring Cloud functions framework or how it was build by Gradle.
We also found that this issue might be related to the OS environment. For example, the working function app works on Linux but not on Windows. The non-working function app does not work on Linux but work on Windows.
The main code part that act as function is
class GraphQlAzureFunction extends FunctionInvoker[Message[String], SystemGraphQlResponse] {

  /**
   * Return HttpResponseMessage with body containing JsonString.
   */
  @FunctionName(GraphQlFunction.FUNCTION_NAME)
  def execute(
               @HttpTrigger(
                 name = "graphQlRequest",
                 methods = Array(HttpMethod.POST),
                 authLevel = AuthorizationLevel.FUNCTION
               )
               httpRequest: HttpRequestMessage[String],
               context: ExecutionContext
             ): HttpResponseMessage = {
    val msg = MessageBuilder.withPayload(httpRequest.getBody).copyHeaders(httpRequest.getHeaders).build()

    try {
      val res = handleRequest(msg, context)
      buildHttpResponse(httpRequest, res)
    } catch {
      case ex: Throwable =>
        buildHttpResponse(httpRequest, SystemGraphQlResponse.ofException(ex))
    }
  }

Ask:
Need Spring team to help understand this issue

Where is the stack-trace???

Hi @olegz , thank you for checking! The stack trace:
{
  "errors": [
    {
      "message": "Oops! An unexpected error occurred: java.lang.NullPointerException",
      "locations": [],
      "extensions": {
        "messageKey": "CommonUnexpectedError",
        "args": [],
        "message": "Oops! An unexpected error occurred: java.lang.NullPointerException",
        "invalidFields": [],
        "origin": {
          "causedBy": {
            "class": "java.lang.NullPointerException"
          },
          "topThrownBy": {
            "class": "org.springframework.cloud.function.adapter.azure.FunctionInvoker",
            "method": "handleRequest",
            "file": "FunctionInvoker.java",
            "line": 129
          },
          "bottomThrownBy": {
            "class": "com.lpg.util_213.spring.cloud.function.graphql.azure.function.GraphQlAzureFunction",
            "method": "execute",
            "file": "GraphQlAzureFunction.scala",
            "line": 31
          }
        },
        "classification": "ExecutionAborted"
      }
    }
  ]
}

If anything else needed, please let me know.

I see that the issue is tagged as AWS. It is actually Azure.

Hi team, is there any thing else needed for next action? Please feel free to let me know.

@Yangsh-w based on your stack trace the error originates in com.lpg.util_213.spring.cloud.function.graphql.azure.function.GraphQlAzureFunction which is not a framework class, so not sure what to say. In other words the exception happens in your class.
If you can share a sample project that reproduces the issue that would help

Hi @olegz , Thank you so much for the information shared! Sorry for this late update, I was not able to log in in the past days due to an unforeseen outage.
Yes, we have a sample project that can reproduce this issue. But this project is not owned by me, I need some time to ask for permission to share it here. Thanks a lot!

Hi @olegz, Is there any way we could share the sample project privately? For example to you only?

No you can not.
Instead, you can create a new sample project which contains only enough to reproduce the issue. So there should not be any sensitive information

Thank you @olegz , I will confirm with the project owner see if we could make a public version. I will keep our thread updated.

Hi @olegz , the owner of this project is not available to create and provide a public sample project at the moment. Do you mind we temporarily close this issue thread and we will provide a response once the project is available? Thanks!


Look into redirecting cigetcert output so that we don't overwrite kx509-gotten proxy

@hschellman has brought up that a number of users are getting confused with how jobsub_lite obtains VOMS proxies.  In jobsub_client, we would run kx509/cigetcert, and then overwrite the VOMS proxy on top of that, in /tmp/x509up_u$(id -u).  Running voms-proxy-init without specifying the -file flag would write the proxy there as well.  A number of users rely on this functionality.
jobsub_lite runs cigetcert and deposits an X509 proxy there, but then does not overwrite the file with a VOMS proxy.  As a result, /tmp/x509up_u$(id -u) is NOT a VOMS proxy, and the working VOMS proxy is elsewhere.
This means that if a user runs voms-proxy-init, and then later on, a jobsub_lite command, /tmp/x509up_u$(id -u) will go from being a VOMS proxy, to not being one.  We might want to send our output of cigetcert to where we eventually write the output file, at /tmp/x509up_<group>_<role>_$(id -u).

I’ve added a voms-proxy-info to my job script to see when and where this happens.  We might want to discuss how to provide users with a prescript debug option that dumps the most relevant info for debugging jobs.
On Feb 3, 2023, at 20:25, Shreyas Bhat @.***> wrote:

[This email originated from outside of OSU. Use caution with links and attachments.]
@hschellmanhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fhschellman&data=05|01|heidi.schellman%40oregonstate.edu|0863da788aa046de69e908db0667d61c|ce6d05e13c5e4d6287a84c4a2713c113|0|0|638110815267627034|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=mc%2Bj0QzvCLetT4WPAFiQaO0tW%2BUpbYFBq7Ukb%2FNDi40%3D&reserved=0 has brought up that a number of users are getting confused with how jobsub_lite obtains VOMS proxies. In jobsub_client, we would run kx509/cigetcert, and then overwrite the VOMS proxy on top of that, in /tmp/x509up_u$(id -u). Running voms-proxy-init without specifying the -file flag would write the proxy there as well. A number of users rely on this functionality.
jobsub_lite runs cigetcert and deposits an X509 proxy there, but then does not overwrite the file with a VOMS proxy. As a result, /tmp/x509up_u$(id -u) is NOT a VOMS proxy, and the working VOMS proxy is elsewhere.
This means that if a user runs voms-proxy-init, and then later on, a jobsub_lite command, /tmp/x509up_u$(id -u) will go from being a VOMS proxy, to not being one. We might want to send our output of cigetcert to where we eventually write the output file, at /tmp/x509up_$(id -u).
—
Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmarcmengel%2Fjobsub_lite%2Fissues%2F265&data=05|01|heidi.schellman%40oregonstate.edu|0863da788aa046de69e908db0667d61c|ce6d05e13c5e4d6287a84c4a2713c113|0|0|638110815267627034|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=L5CEUa82BKvFOEIjJcQHOH10%2FOjMWQ%2FN7SoUiis6WCY%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAIA37DMRYPSZDZ74FBQ5B4LWVXK3JANCNFSM6AAAAAAUQ6E2GQ&data=05|01|heidi.schellman%40oregonstate.edu|0863da788aa046de69e908db0667d61c|ce6d05e13c5e4d6287a84c4a2713c113|0|0|638110815267627034|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=1QZr7WvZQUKCryxeBMq9X1E0h73DzVzHNaPxeeegang%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


[Searx] Changing default settings

Hello,
After cleaning the cookies of the browser the settings of searx are gone.
I've tweaked your Dockerfile to make it possible to add a volume where default settings can be store.
Let me know if it could be helpful.

I will implement something like this in the future, but I thought about a different way.
Thanks anyway!


Add folder has different behavior for ASP.NET projects

Hey guys,
It may be an upstream bug and thus won't fixable, but, putting it here to track it.
We cannot create a folder in the root directory of the project, only ASP folders:

It is however possible to create one within a folder:

Arghhh sorry my fault, I was confused: I didn't activated the option in my VS 2015 :)))


Examples in README allows invalid parameter in json POST request, it there a way to validate or you need to manual validation ?

{"Success":"User ss Pass: "}montana@localhost ~ $ curl -X POST -d '{"Useraaaaaaaaaaaaname":"ss"}' http://<IP_ADDRESS>:8080/auth/login
{"Success":"User  Pass: "}user@localhost ~ $ curl -X POST -d '{"Usaaaaaeraaaaaaaaaaaaname":"ss"}' http://<IP_ADDRESS>:8080/auth/login
{"Success":"User  Pass: "}user@localhost ~ $ curl -X POST -d '{"Usaaaaaeraaaaaaaaatttttttttttttttttttttttttttaaaname":"ss"}' http://<IP_ADDRESS>:8080/auth/login
{"Success":"User  Pass: "}user@localhost ~ $

This library is only mapping from json to struct, you can use something like http://godoc.org/gopkg.in/validator.v2for validation.

On 25 Jun 2015, at 18:55, jitendra kumar<EMAIL_ADDRESS>wrote:
{"Success":"User ss Pass: "}montana@localhost ~ $ curl -X POST -d '{"Useraaaaaaaaaaaaname":"ss"}' http://<IP_ADDRESS>:8080/auth/login
{"Success":"User Pass: "}user@localhost ~ $ curl -X POST -d '{"Usaaaaaeraaaaaaaaaaaaname":"ss"}' http://<IP_ADDRESS>:8080/auth/login
{"Success":"User Pass: "}user@localhost ~ $ curl -X POST -d '{"Usaaaaaeraaaaaaaaatttttttttttttttttttttttttttaaaname":"ss"}' http://<IP_ADDRESS>:8080/auth/login
{"Success":"User Pass: "}user@localhost ~ $
—
Reply to this email directly or view it on GitHub.


django_api: Rename send_event to send_event_rollback_unsafe.

Fixes:

Screenshots and screen captures:

Self-review checklist

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

Communicate decisions, questions, and potential concerns.

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

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

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

Completed manual review and testing of the following:

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

Is this expected to still be marked as a draft?

Is this expected to still be marked as a draft?

Yeah, I need to update the use of send_event in docs.

$ git grep "capture_send_event_calls" | wc -l
119

I didn't rename capture_send_event_calls to capture_send_event_rollback_unsafe_calls because I think it's easy to understand that "send_event_calls" refers to send_event_rollback_unsafe or send_event_on_commit calls. Also, the comments inside capture_send_event_calls explains it well. So, I think renaming the 119 occurrences is not worth the effort ?

I ended up just adding a bonus commit with some wording/documentation tweaks. Merged, thanks @prakhar1144!


[Task]: Extract Scala Beam symbols for Playground

What needs to happen?
Parse symbols so they can be suggested by the editor autocompletion.
See https://github.com/apache/beam/issues/23304 as a model.
Issue Priority
Priority: 3
Issue Component
Component: beam-playground

.take-issue


Broken output in symbols mode

This happens when i use symbols mode:
<symbol id="BG" viewBox="0 0 595.279 841.89">[object Object] [object Object] </symbol>

This is my task:
gulp.task('svg', function () {
	return gulp.src('images/svg/*.svg')
		.pipe(svgSprite({
			mode: "symbols"
		}))
		.pipe(gulp.dest('images/svg/test/'));
});

and i tried so
gulp.task('svg', function () {
	return gulp.src('images/svg/*.svg')
		.pipe(svgSprite({
			mode: "symbols"
			svg: {
				symbols: 'symbol_sprite.html'
			}
		}))
		.pipe(gulp.dest('images/svg/test/'));
});

window 10
  <symbol id="calendar" viewBox="0 0 14 12">

      [

      o

      b

      j

      e

      c

      t

      O

      b

      j

      e

      c

      t

      ]

      [

      o

      b

      j

      e

      c

      t

      O

      b

      j

      e

      c

      t

      ]

  </symbol>

The same is happening with me on Mac

The same problem!!!!

+1 same problem here on MacOS

windows 10 x64
The same problem!!!!

windows 10 x64
The same problem

@soenkekluth any solution to this problem?

I solved this problem by using gulp-svg-sprite
sorry @shakyShane

Bug happens upstream: https://github.com/shakyShane/svg-sprite-data/issues/6
Though in gulp-svg-sprite, the default template would have to be fixed (there’s no need to iterate on a string).

Have you found solution?

@xahon If you just need symbols I wrote https://github.com/mllrsohn/gulp-svg-symbol-sprite today which generates something like this:
<svg height="0" style="position: absolute" width="0" xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">
    <symbol id="arrow" viewbox="0 0 25 45">
        <path d="..."/>
    </symbol>
    <symbol id="cart" viewbox="0 0 21.5 17">
        <path d="..."/>
        <circle cx="4.793" cy="15.206" r="1.794"/>
        <circle cx="13.208" cy="15.206" r="1.794"/>
    </symbol>
</svg>

It works for my projects - so no tests so far. The code is relative simple. Have a look if it fits your needs.

Until it's fixed you can save the changed symbols template in your project and override the path:
.pipe(plugins.svgSprites({
        mode: 'symbols',
        preview: {
          sprite: 'index.html'
        },
        templates: {
          symbols: require('fs').readFileSync('./dev/symbols.svg')
        }
      }))```

fixed by https://github.com/shakyShane/gulp-svg-sprites/pull/90


no license

(i am not a lawyer)
as far as i can tell, this project has no license. i don't think i or other people can legally use or contribute to it without a license.
also, the readme says it's based on cargo-3ds, which i'm assuming means it uses the code, since the code looks extremely similar. this means this project should have the apache or mit license from that project somewhere.

bumping this because although i could try to recreate it myself without checking the source code it would be a lot nicer if this just had a license and i could use it. i've had this blocking a project i want to work on for a long time now.


Set model ids

Since we want to use it in petab/pypesto, it would be good if every model id (in the model sbml file) is set to a clear id, e.g. the same name as that of the folder).

done in #56


Keyboard behavior in select_for_key_down elides over pauses, combining keys.

The design of 'select_for_key_down()' in behaviors.py allows for multiple key strokes to be returned.  For example, running examples/widgets/compound_selection.py, typing "13" first sees an input of '1' and then aggregates it for a second input of '13'.   However pausing will randomly reset the typing button, so waiting a few seconds and typing '13' again returns aggregates of '131' and '1313'.   It should have reset and returned '1' and '13' as before.
The issue is calling time.clock() instead of time.time().  A fix can be found in https://github.com/merriam/kivy/commit/d735622ad5802d6d42a2a739ab83a7521a34e6d4, an
unmerged PR.

Hi @merriam,
I saw you answered to a numerous issues referencing your closed PR. Just to be sure, we never intented to reject your work, but submitting thousands of files with unrelated changes is not going to work. It is making the exchange harder, as we (as maintainer) can delay the merge until all the fixes are "ok", which can delay all the fixes that were ok as well.
Tiny atomic commit are the key for a single bug fix, or a bunch of commit on the same new features / refactoring, as long as it's the exact same subject.
On this specific issue, thanks again for finding it and providing a fix, you can resubmit a PR by doing this step:

Fetch all the latest ref in master: git fetch -a
Create a new branch: git checkout --track -b fix-3120 origin/master
Pick only this commit and merge it into the new branch: git cherry-pick d735622
Publish your branch on your repo: git push origin fix-3120
Then, go back to github, and it should show you the possibility to create a PR from your newly fresh branch: fix-3120.

Thanks you :)

Hi Mathieu,
Thank you for your kind note.   It does appear from previous exchanges that
the intent is indeed to reject any work I directly submit.  The larger
block of changes was submitted because every smaller block was nit-picked,
ignored, or randomly closed.   The nit-picks, usually about the number of
spaces after a period in comments, do not appear to be about the quality of
the submission, any errors, or any substantive issue.   In fact, each PR I
submit usually fixes these nits in other's accepted PRs.
I provide tiny, atomic commits.   I tried side-stepping the PR nit problem
by putting many commits in one PR.   As each uncommitted pull request is a
separate branch, I need to provide a reasonable limit to the number of open
and disputed Pull Requests.   I set that at about five or six.
As I enjoy coding, building, and fixing, I intend to continue fixing bugs
and cleaning up the /examples directory.   As I have already played the
sequence of "please resubmit a fix; and again; and again; and now it will
rejected just because", such as with examples/audio, I do not intend to
rebundle or resubmit these.   I'll reference them, and others may choose to
do so.  If I can be persuaded that the games would stop, I could put one or
two commits per PR as is my normal habit.
Thank you for your understanding,
Charles
On Mon, Mar 9, 2015 at 12:52 PM, Mathieu Virbel<EMAIL_ADDRESS>wrote:

Hi @merriam https://github.com/merriam,
I saw you answered to a numerous issues referencing your closed PR. Just
to be sure, we never intented to reject your work, but submitting thousands
of files with unrelated changes is not going to work. It is making the
exchange harder, as we (as maintainer) can delay the merge until all the
fixes are "ok", which can delay all the fixes that were ok as well.
Tiny atomic commit are the key for a single bug fix, or a bunch of commit
on the same new features / refactoring, as long as it's the exact same
subject.
On this specific issue, thanks again for finding it and providing a fix,
you can resubmit a PR by doing this step:

Fetch all the latest ref in master: git fetch -a
Create a new branch: git checkout --track -b fix-3120 origin/master
Pick only this commit and merge it into the new branch: git
cherry-pick d735622
Publish your branch on your repo: git push origin fix-3120
Then, go back to github, and it should show you the possibility to
create a PR from your newly fresh branch: fix-3120.

Thanks you :)
—
Reply to this email directly or view it on GitHub
https://github.com/kivy/kivy/issues/3120#issuecomment-77928527.

I don't really understand your issue, i see that others already explained why the few refused PR have been closed, mostly due to an invalid git history. This can be sorted out, i don't think what you were trying to resolve was the issue.
Maybe we can help you to find which part in your git process is polluting the PR, and make us stop to merge it; Except the audio example pr which has been refused, two others have been refused due to a mess into your git branch (core_stbtext, i guess at some point you tried it and merge it into your master?), and all the rest have been merged :)
In my point of view, you're doing it good, just a mess with git, which can be solve out ^^

@merriam 39 of your 53 PRs were merged (and it looks like quite a few of the unmerged ones were closed by yourself). I don't see how that qualifies as intending to reject any work you submit. But if you submit a PR which has something like double spaces, which we will want to fix, then you are just creating more work for other people.
Also, many things you seem to think are "nitpicking" are not at all - like rejecting PRs where you add and remove big files. You do realize that everyone still has to download the big file when you do that, even if you remove it in a later commit? And that file will always be in the repo? Unless that commit is removed in a rebase and the master branch is force pushed (generally a very bad idea), the file will never go away. That's the way git works.
It's also not ok to ask us to merge a PR when you know there is an issue with the PR and just saying "I'll submit another PR to fix that." You could easily push a commit with the fix to the original PR, and then ask us to merge it.
It really feels like you're just trying to cause problems at this point.


Remove javax.annotation package & sub-packages from gradle-api-.jar

Remove javax.annotation package & sub-packages from gradle-api-<version>.jar.
They cause duplicate package problems with the module path.
Maybe some additional packages should also be removed from that jar.

A Gradle plug-in with which I used the module path. I’m switching all my code over to module path. Currently, I just excluded the FindBugs JSR 305 jar from my transitive dependencies fit guava.
Dependencies would be more flexible if you decompose the gradle-api jar so people can choose what to exclude, or so the transitive dependencies of gradle-api will just be the same artifacts that others depend on, so nothing will need to be excluded.

As long as Gradle doesn't use the module path at runtime, I don't see much benefit in using it during development of a plugin. Am I missing something?

I don’t know all the benefits of the module path in all circumstances, but I’m trying to keep all my code consistent, and I’m switching all my code to module path.
Even without the module path, it’s generally nicer to have transitive dependencies instead of monolithic jars since it allows more flexibility in terms of upgrading / modifying individual artifacts to fix bugs, reduces duplicate classes on the class path (which could prevent version bugs), provides more flexibility for excludes, lets devs better understand the dependencies of their code when looking at dependency reports, etc.

Agreed.


App-Export - AppQuota contains API-ID instead of API-Name

API-Mgr. CLI version
1.3..5
API-Manager and Service-Pack Version
7.7-September
Actual behavior
When exporting applications that have Application-Quotas configured the exported JSON-Structure looks like this:
  "appQuota" : {
    "restrictions" : [ {
      "api" : "88fb1882-f145-4afe-939e-b27737398da8",
      "method" : "11477300-e944-462e-aa73-210bc3b6b6a2",
      "type" : "throttle",
      "config" : {
        "messages" : "50",
        "period" : "hour",
        "per" : "1"
      }
    }, {
      "api" : "ad5c51d1-1c14-48dd-af14-32fd3fc6995d",
      "method" : "*",
      "type" : "throttle",
      "config" : {
        "messages" : "500",
        "period" : "hour",
        "per" : "1"
      }
    } ]
  },

This makes it impossible to promote the application, including configured quotas, to higher stages as the API-ID will be different.
Expected behavior
The exported App-Quota must look like this:
  "appQuota" : {
    "restrictions" : [ {
      "api" : "My Great API-Name",
      "method" : "My great API-Method-Name",
      "type" : "throttle",
      "config" : {
        "messages" : "50",
        "period" : "hour",
        "per" : "1"
      }
    }, {
      "api" : "My Great API-Name",
      "method" : "*",
      "type" : "throttle",
      "config" : {
        "messages" : "500",
        "period" : "hour",
        "per" : "1"
      }
    } ]
  },

sorry for the late response, your solution sounds good for me.


Should we create something akin to the MODULE macros ptracer.h

It sounds like that part is repetitive boilerplate. If it's always the same, maybe we can create those macros ourselves?

I think not.
Because it is very broad and hard to say how will the modules adopt pallene-tracer. Some modules may use the Dispatch system like Pallene (dispatching the Lua C call to an actual C function), or some module may use the non-dispatching method. Some modules may also use static line numbers and module/filenames just like we are doing in Pallene.
Also on top of that, we need pt_cont_t * and the finalizer object to be passed around the function willing to be in the traceback. Now, different modules may handle it differently as well, there is no way of knowing how they would.
So finding out and including a generalized predefined macros in Pallene and keeping all the module happy would be a really hard task. I would suggest we do the ground work and let the module hackers take it from here.
But after the adoption of Pallene Tracer in broader modules if any repetitive pattern of the macros emerges, we may include it in the ptracer.h.
We can close the issue for now.

But after the adoption of Pallene Tracer in broader modules if any repetitive pattern of the macros emerges, we may include it in the ptracer.h.

Indeed, that's the main idea.
But it's likely that there are some low-hanging fruit that we can already pick. For example, we could have a version of SETLINE that uses  __LINE__ macro.


sleeptimeinactive time suggest for pythonanywhere 2

pythonanywhere free have 100s cpu
if i use 300s sleeptimeinactive in a day i have 156 fire.
Also if it keep cpu for 1s i overflow 100s or i wrong?

You can still comment in a closed issue please do not make another one.


unable to find where to open the editor

as of updating to blt 2.0 and updateing both beardlib and the editor i can not the the button in the options to open the editor where is the new location as i can not find out

what version do you have for both editor and beardlib?

(for beardlib and beardlib mods you should use the beardlib mods manager I'm just too lazy to update the version for that menu also everytime)
But yeah I need your log file, something is probably preventing it from being created.

im sorry what logs am i sending you?
the mini dump crash logs?

2017_09_06_log.txt
i think this is what you wanted

02:51:18 PM Lua: [ERROR][BeardLibEditor] BeardLib is not installed!
Try reinstalling BeardLib

yea i have reinstalled multiple times and the same error is comming up in the log im going to try to redownload the whole game and reinstall all mods


status: add option for support pipes

Added new option (POWERLEVEL9K_STATUS_SHOW_PIPESTATUS) that shows
the status for all piped commands, not just the right-most.
Closes #514

You'll need to close this PR and submit a new one, and Github doesn't allow you to change the branch you have filed against (I think).

Son of a Guinea Pig!  I just discovered that you can after manually closing and re-opening all the other PRs!
You just click "Edit" at the top (next to the title for the PR) and you can change it there.  As an admin, you should have also been able to do that.

Son of a Guinea Pig! After manually closing and re-opening all the other PRs, I discovered that you can change the base branch!
You just click "Edit" at the top (next to the title for the PR) and you can change it there. As an admin, you should have also been able to do that.

So, this would have worked, but wouldn't have been ideal. I could have changed the target of your merge from master to next, but because you based your original branch on master instead of next, if I had merged it into next it would have carried all of the other master commits that were parents of your branch, going back to the first common ancestor of master and next.
Realistically, this wouldn't have done any damage to the working status, as I try to make sure changes on master gets merged into next regularly so that bug fixes propagate. The issue is that if we ever needed to revert this merge, or simply wanted to look at the diff created by it, it would have been polluted by the changes on master. In short, it would have worked, but would have made for a messy git history that probably would have been fine but I which I like to avoid =)
Thanks so much for re-creating your PRs. I really appreciate it! This is a really great contribution - thank you 💯

True, you couldn't have changed it, but I can if I rebase my patch off the next branch.  I'll try to remember next time.


⚠ Bofrost.es has degraded performance

In 139472f, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1236 ms

Resolved: Bofrost.es performance has improved in adbfefb after 6 minutes.


Failed to configure system DNS

/usr/local/sbin/vpn: failed to configure system DNS: failed to backup DNS config file '/etc/resolv.conf.head' to '/etc/resolv.head.outlinecli.backup': rename /etc/resolv.conf.head /etc/resolv.head.outlinecli.backup: no such file or directory

How to fix this error on ubuntu 22.04.4 LTS?

Actually, I'll leave this one open.
The problem is not really distro-dependent, it's more about your networking stack. I had the same issue on a Fedora-driven device recently. In short, the upstream CLI wrongfully expects resolv.conf.head to always be present on the system, which, as you can see, is not the case. It doesn't even need this file to be present at all, it just backs it up. So, this clearly is not a place to throw an error.
I'll make a PR upstream and/or add a patch for that to this repo.

Should be fixed by Jigsaw-Code/outline-sdk#203.


[App] Bug: Parallel plot maintains old selections after being added to reports

Describe the bug
Adding parallel plots from sweeps to reports shows the following bug: the plot keeps showing the runs selected to be visualized at the time the plot was added to the report.
In the report, even when no run in selected to be visualized, the plot still shows runs (the ones visualized when the plot was added to the report).
I created a simple report showing this issue. The following three screenshots illustrate the bug:
Case 1: Plot added when NO runs are visualized -> it works as expected (I'm able to show/hide some runs)

Case 2: Plot added when SOME runs were visualized -> the selected runs are visualized even when no run is select

Case 3: Plot added when ALL runs were visualized -> ALL runs are visualized even when no run is select

To Reproduce
Steps to reproduce the behavior:

Create a sweep.
Visualize all runs in the parallel plot.
Add plot to a report.
Now visualize no runs in the parallel plot.
Add the new plot to the same report.
Now visualize some runs in the parallel plot.
Add the new plot to the same report.

Expected behavior
I would expect to manipulate the plots and make them identical if wanted. I expect no run to be selected in the plot when I deselect all runs (this happens only in case 1 above).
Operating System

Browser: Microsoft edge 96, safari (latest)
wandb version: 0.12.9

Hey Andrei, thanks for reporting the issue. I was able to reproduce it. I am filing a ticket so that the team can fix this asap. I'll notify you once that's done.


Run process with user different than postgres

Really nice work!
I've submitted this PR because I had to run postgres_exporter pointing to remote DB. I prefer to not use postgres as user account when there is no postgresql database running.
This PR is retro-compatible so when the role is already applied the postgres user is not modified.

@joshmoore sorry but I didn't understand why now is in error.

Tagged as 0.4.1


Added Yuri as Translator for Chapter 1

added Yuri as translator for chapter one.

I have made the PR to master by mistake. Will close this so I can reopen the PR for ja branch.


GitHubSource: Creation of GitHubSource instances fail

Describe the bug
I´ve applied a fix for #1048 locally such that the webhooks can be successfully registered. After this, the creation of new GitHubSources fail with
failed calling webhook "defaulting.webhook.github.sources.knative.dev": Post https://github-webhook.knative-eventing.svc:443/defaulting?timeout=30s: service "github-webhook" not found

Looks like a service definition is missing that so far was not noticed as the webhooks were not registered due tu #1048 ?
To Reproduce

install github source
add rbac permissions for webhook registration
try to create a new GitHubSource

Knative release version
v0.13.3
Thanks,
Fabian

I think this is fixed by #1059 .

Fixed in #1059
/close


API : Implement Validation for QA/Cert Data Maintenance PUT Endpoint

As the ECMPS system, I need to have validation for the QA/Cert Data Maintenance PUT endpoint so that all required parameters are chosen:
ENDPOINT: PUT /camd-services/admin/qa-maintenance
parameters:
certEventId: string
testExtensionExemptionId: string
testSumId: string
Error Messages:

6/7 Per planning this ticket is no longer needed


Support tablename prefixes

If speedment is generated against a database which has 2 tables. Let's say Device and User. Make it possible that on instantiation (at runtime) speedment accepts a prefix for those tables.
new SpeedmentApplication().withPrefix("test_").build() which would then use the the tables "test_Device" and "test_User", instead of "Device" and "User".

I don't know your exact use case here, but I assume it is for some sort of automatic testing. If so, isn't it better to use a separate database with the same structure? Then you can use existing functionality, such as new SpeedmentApplication().withXxx("yyy").withSchema("TestDatabase");.

Sure for Testing you would use a separate database.
I was more thinking about legacy code, or databases that were created by other software. Example
Owncloud (i know it is outdated, but just as ONE example) OwncloudUsingTablesPrefixes

You can choose to point an manager to a different table by overriding the getTable()-method in the manager implementation class. In that way you can have legacy tables in the database that is not visible in the generated code.
For an example, I have a database with two tables, physician and test_physician. I don't want to generate duplicate entities so I disable the test_physician table in the Speedment UI. If I want to point the generated PhysicianManager to use the test table instead, I can simply open the generated PhysicianManagerImpl-class and add the following:
public final class PhysicianManagerImpl extends GeneratedPhysicianManagerImpl implements PhysicianManager {

    public PhysicianManagerImpl(Speedment speedment) {
        super(speedment);
    }

    // Add the following:
    @Override
    public Table getTable() {
        return speedment().get(ProjectComponent.class)
            .getProject()
            .findTableByName("db0.physicians.test_physician"); // <-- The table is different
    }
}

Speedment will now load entities from the test table instead of the real one. Use this with caution though, if the tables are structurally different you might get unspecified behaviour.


Commands picker: search value no longer trimmed

Steps to Reproduce:

open the command picker
type " debug" with leading space

Stable:

Insider:

Thanks to @eamodio for hitting this in his demo yesterday 👍

Reviewed in the Stand up and decided not critical enough. Removing candidate.


Tonejs Transport error in pausing while songs are playing

So I'm trying to make a simple Digital audio workstation(D.A.W) kind of thing in react. To solve this problem I'm using Tonejs Transport to play and sync multiple songs on Transport timeline and is working fine but problem arises when i try to pause it. Eventually it pauses when i click to pause but also it crashes the application saying Error: buffer is either not set or not loaded
I tried to do something like this but this didn't solve the problem.
Tone.Buffer.on('load', () => {
if (this.props.play)
Tone.Transport.start()
if (!this.props.play)
Tone.Transport.stop()
})
This is where I'm setting the player
Player = (src, startTime,volume) => {
Tone.Transport.bpm.value = 108;
Tone.Transport.loop = false;
let buff = new Tone.Buffer({
  "url": `/MP3s/${src}`,

});
let play = new Tone.Player(buff).toMaster().sync().start(startTime)

Tone.Buffer.on('load', () => {
  if (this.props.play)
  Tone.Transport.start()
  if (!this.props.play)
    Tone.Transport.stop()

})
}
It's showing error on the let play line in code and i just couldn't figure out on my own about the reason of problem and how to resolve it. Also I'm playing these files from the public folder of my react app.

i'm unable to verify this error. is it possible to create a minimal test case or send a link of live code which includes the error?

Hi tambien, i,m sharing my repository link. You can check this out.
https://github.com/ParamjeetDahiya/music-app.

@ParamjeetDahiya, Unable to verify this issues since there's still quite a bit of code to get up and running and parse through. It'd make it much easier to debug if you could point me at a minimal example of this running in jsfiddle or similar live environment. that way we can debug just the Tone.js code.

Hello @tambien , I can't send you the minimal version because minimal version is working fine. It's the issue with react. Whenever the state change in react it doesn't affect tonejs part, that's why it's showing buffer is empty. I don't know why buffer is empty after re-rendering. Can you tell me how to work on tonejs with react.


How to use the script coco_eval.py?

Hi, I'm really confused about the usage of script coco_eval.py. There is a command line parameter named result with description "result file path". So I believe it should be a str in Python. But when calling the function coco_eval here with types=['bbox'], the parameter result should be a dict type as required here.
If the needed result should be a dict, then what are the keys and values?

You may refer to results2json. The keys are result types and the values are filenames.

@liliangqi have you find the right way to use coco_eval.py, i run the command: python tools/coco.eval.py result.pkl --ann ./data/coco/annotations/instances_val2017.json, but occured the error
typeerror: string indices must bu integers.

python tools/coco_eval.py  eval/faster_rcnn_101.pkl --ann data/coco/annotations/instances_val2017.json --types bbox

python tools/coco_eval.py ${RESULT_JSON_FILE} --ann ${ANNOTATION_FILE} --types bbox


Border Control Still Unreliable

The border style section in the inspector was recently fixed, and works correctly on some projects but not in others. cc @liady since you were the one working on this, feel free to tag someone else

yes, since the border actually has a color different than black, so we don't recognize it as a "non-existing" border.

the question is - do we want to treat any 0px border as a "non existent"? This can cause a situation where a user will change the border color (but will leave it at 0px width) - and after reselecting the element they will not see the controller again (since we'll treat it as "no border")


Support spawning cmdlets under sudo

Please support temporary privileged mode for powerhsell. Bash allows temporary privileged mode by prefixing a command with sudo. In powershell, this entails spawning the whole shell under sudo which means that all commands run privileged which an anti-pattern for shell use in the UNIX world. Usage of sudo should be intercepted so that sudo starts a powershell process that runs the command/cmdlet then exits immediately instead of letting sudo start just a bash shell which would not support cmdlets.
Steps to reproduce
sudo Update-Help
Expected behavior
The Update-Help cmdlet runs in admin privilege in a temporary powershell instance
Actual behavior
The sudo command ends up spawning a privileged bash instance which obviously knows nothing about PWSH cmdlets
Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      6.0.0-rc.2
PSEdition                      Core
GitCommitId                    v6.0.0-rc.2
OS                             Linux 4.14.10-041410-generic #201712291810 SM...
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Hi @borgdylan! Thanks for reporting the issue. however, this appears to be a duplicate of  #3232. Please go to that issue and comment/vote.


lvm: provider not defined

When installing via bare metal (not terraform), the provider variable is not defined  for lvm tasks, causing an ansible error.
https://github.com/CiscoCloud/microservices-infrastructure/blob/master/roles/lvm/defaults/main.yml#L3
provider needs to be defined somewhere or we need to check if provider is defined in the template:
lvm_physical_device: "{% if provider == 'gce' %}/dev/disk/by-id/google-lvm{% elif provider == 'aws' %}/dev/xvdh{% elif provider == 'openstack' %}/dev/vdb{% endif %}"

There is workaround for this?

@azimut you can try setting  provider=baremetal in the ansible var file.


User stories: some thoughts about common use cases

I try to translate my knowledge about Jails into the world of OCI, so here are some (not too technical) thoughts.
Are container engines implementing the runtime-spec like Podman indented to serve as an alternative to Jail managers such as BastilleBSD? Like e.g. As a user, I want to use container engines such as Podman for general Jail management.
Some Jail managers implement the concept of thin and thick Jails, where a thin Jail, contrary to a thick Jail, shares a FreeBSD base image with other Jails. Is this something the runtime-spec should reflect in any form? Like e.g. As a user, I want to be able to share one FreeBSD base image with multiple containers.
Not sure if this belongs to the runtime-spec: As a user, I want to create reusable containers and publish/download them to/from a container registry. (e.g. to install certain applications like NGINX or PostgreSQL or create my own)

Are container engines implementing the runtime-spec like Podman indented to serve as an alternative to Jail managers such as BastilleBSD?

I'm not very familiar with BastilleBSD, but from a quick look at the website I think that's a somewhat higher-level tool.  The runtime spec is more concerned with a single container (jail) at a time; I'd consider it more as an alternative to jail(8).

Some Jail managers implement the concept of thin and thick Jails, where a thin Jail, contrary to a thick Jail, shares a FreeBSD base image with other Jails. Is this something the runtime-spec should reflect in any form?

The runtime spec doesn't really know about images or whether anything is shared.  At a high level, a compliant OCI runtime consumes a "bundle" that consists of a JSON file describing how the container should be configured and a root filesystem (generally presented as a directory).  Higher-level tools built on top of OCI runtimes (such as containerd, Docker, etc) will manage images and determine whether sharing occurs; containerd and Docker typically use some form of a copy-on-write backend to implement this (overlayfs being the most common on Linux, but on FreeBSD zfs can be used).
The image spec (which is separate from the runtime spec) describes how images are formatted.

Not sure if this belongs to the runtime-spec: As a user, I want to create reusable containers and publish/download them to/from a container registry. (e.g. to install certain applications like NGINX or PostgreSQL or create my own)

Publishing and downloading images (aka pushing and pulling) is standardized as part of the distribution spec.
An OCI runtime does not (typically) implement anything other than the runtime spec.  Higher-level tools will use an OCI runtime and implement the image spec and/or distribution spec themselves.
Hopefully that helps!

This is mostly covered elsewhere in the image and distribution specification as @samuelkarp mentions. How the image is stored locally is somewhat dependent on the container engine's implementation. For instance podman on FreeBSD supports two storage drivers, ZFS which uses cloned datasets to make something like what I think you mean by 'thin jails' and VFS which just copies directory trees, which its sort of like a 'thick jail'.
I would also like to note that it is possible to avoid using images altogether although I think this is probably rarely used. With podman, you can uses the --rootfs argument to podman create and podman run to reference an existing directory tree:
# d=$(mktemp -d)
# tar -C / -cf - rescue | tar -C $d -xf -
# podman run -ti --rm --rootfs $d /rescue/sh
Cannot read termcap database;
using dumb terminal settings.
# /rescue/ls
dev	etc	rescue	var
# exit
# rm -rf $d

I'm not very familiar with BastilleBSD, but from a quick look at the website I think that's a somewhat higher-level tool. The runtime spec is more concerned with a single container (jail) at a time; I'd consider it more as an alternative to jail(8).

Thanks for the clarification. Yes, correct, BastilleBSD is a higher-level jail manager. My general thought was if, when we think about the future, e.g. podman in combination with an OCI-compliant runtime can serve as a tool for general purpose jail management (like e.g. BastilleBSD). From what I understood now, this can be assumed "yes". However, the runtime-spec itself, which this working group is about, will in that sense "only" form the foundation for high-level tooling (together with other specifications like the image and distribution spec). Hope this makes sense.

For instance podman on FreeBSD supports two storage drivers, ZFS which uses cloned datasets to make something like what I think you mean by 'thin jails' and VFS which just copies directory trees, which its sort of like a 'thick jail'.

Thin jails are basically read-only nullfs mounts, whereas thick jails independently maintain there own directory trees. For example:

/usr/local/bastille/releases/14.1-RELEASE /usr/local/bastille/jails/testjail/root/.bastille nullfs ro 0 0

The major advantage of thin jails is that upgrades of the base release across multiple jails become extremely simple (and fast).

The way I think about this is that the runtime specificiation, including whatever we agree on for a FreeBSD extension defines the interface between higher level tooling (e.g. Podman, containerd, CRI-O) to the low-level tooling (e.g., runj, ocijail) and clarifies the features offered by those low-level tools. This abstraction layer gives a lot of flexibility on how to implement the container and on Linux there is a wide variety of runtimes ranging from lightweight kernel-level virtualisation (FreeBSD equivalent is jail) to full virtual machines. This is similar to what you wrote above so I think we are on the same page.

Exactly, that sums it up perfectly.
I think this holds great potential for the acceptance and adoption of FreeBSD and its capabilities.


ODL 7.12.3 changelog

Docs Build status updates of commit f0bc01d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/changelog/odatalib-7x.md
:white_check_mark:Succeeded
View

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit acd8706:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/changelog/odatalib-7x.md
:white_check_mark:Succeeded
View

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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


GodrayFilter does not work with default config

Hello
GodrayFilter does not work.
Javascript warning:

Uncaught TypeError: Cannot set property 'dimensions' of undefined

Playground link: https://www.pixiplayground.com/#/edit/iuojVuHNaVBJa8BrrBNOt
Tested on Windows 10
Google chrome version 78

Unable to reproduce error:
https://jsfiddle.net/bigtimebuddy/kv2gc76a/


Add pagination where required

We need to add the pagination to pages where it is required. Like we need to show only 10 comments per page.

#15


Change diagnostic-id spelling to comply with other languages

.NET (track1, 2), Java, and Python ServiceBus and EventHub SDKs spell Diagnostic-Id, and java uses diagnostic-id  which causes context propagation between different languages to break. https://github.com/Azure/azure-sdk-for-net/issues/22775
Since tracing support is still in preview (both with the plugin and through application insights agent) and we presumably don't have a lot of customers using it, making behavior-breaking change in Java before GA is more preffereable than making all other languages support diagnostic-id spelling.

Do any of the other names need to be changed as well?

nope, just this one, other names are not used beyond Java SDKs

This may run into RevApi issues as a public property is changing value.

Is there a way/review process to bypass it? Even though it's defined in azure-core, it's never used without tracing enabled
Yeah, this is what happened.

@lmolkova, this is achievable with a RevApi suppression, but before that let's see if @JonathanGiles and @srnagar are okay with this change.

Make the change!

Given that this context is only used when tracing is enabled and the core-tracing library is still in preview, I am fine with making the change to this constant which fixes actually fixes a bug.


[bug]  slotted title pattern padding not aligned

Describe the bug
Link
In the "Slotted Title" pattern for card, the padding for the Header and the Body are not aligned.  The heading has a padding-inline of 24px and the body has a padding-inline of 32px.  Put more simply, the heading is using --rh-space-xl and the body is using  --rh-space-2xl.
Which Component?
rh-card
Steps to reproduce

Go to https://deploy-preview-1200--red-hat-design-system.netlify.app/patterns/card/#sample---slotted-title
Inspect the Header for the "Slotted Title" pattern
Confirm the padding-inline is 24px
Inspect the Body of the card
Confirm the padding-inline is 32px.

Expected behaviour
The heading and the body should be inline the same amount
Screenshots
No response
Element HTML
No response
Operating System (OS)
macOS
Browser
No response
More browser information
No response
Additional Info
No response

@brianferry, the padding in the title bar (16px) looks correct. I was wondering if we wanted to allow the header text to look like this:

or lock it down to look like this always:

cc: @coreyvickery

Fixed by #1380


my bad

typos in previous pr are causing panics
Signed-off-by: Brent Baude<EMAIL_ADDRESS>

LGTM

LGTM
On Tue, Apr 21, 2020, 12:37 OpenShift CI Robot<EMAIL_ADDRESS>wrote:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: baude
https://github.com/containers/libpod/pull/5923#
The full list of commands accepted by this bot can be found here
https://go.k8s.io/bot-commands?repo=containers%2Flibpod.
The pull request process is described here
https://git.k8s.io/community/contributors/guide/owners.md#the-code-review-process
Needs approval from an approver in each of these files:

OWNERS https://github.com/containers/libpod/blob/master/OWNERS
[baude]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/containers/libpod/pull/5923#issuecomment-617270211,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB3AOCBAK5EO5N3BF57OOY3RNXDSXANCNFSM4MNNJADQ
.

/lgtm
/hold

/hold cancel


travis: speed up (maint-2.0)

This is the maint-2.0-version of #3371

@crepererum please rebase. Thx


Add more SQLite selection parameters

After reading the SQLite docs I came to the conclusion that Inquiry already supports all selection parameters. If I'm not right just open a new issue here.


Fix file move failed. Fixex #1785

Description
File move failed when HTTP file upload was not.
Test code:
$file = new CodeIgniter\Files\File('/Users/haoliang/Desktop/WX20190311-114654@2x.png');
$file->move(WRITEPATH . 'uploads');

Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

This broke several tests. Please fix.

This broke several tests. Please fix.

My fault, test failure:
vfsStream don't support SplFileInfo::getRealPath().

Something similar to this has been suggested to someone when trying to upload files in CodeIgniger vr 4:
title: "CodeIgniter\HTTP\Exceptions\HTTPException", type: "CodeIgniter\HTTP\Exceptions\HTTPException",…}
code: 500 file: "midominio/system/HTTP/Files/UploadedFile.php"
line: 158 message : "Could not move file "phpXUmYkQ" to "/home/host/../writable/uploads/../folder/". Reason: Module 'sodium' already loaded"
title: "CodeIgniter\HTTP\Exceptions\HTTPException"


NTR - metabolism: [dihydroxyacetone kinase l]

Request,
dihydroxyacetone kinase
Currently related for glycerone kinase activity
but they seem to be  different activities?
https://pubmed.ncbi.nlm.nih.gov/16535475/ (1996)
Although both of the enzymes had some affinity for glycerol and dl-glyceraldehyde in addition to dihydroxyacetone and glyceraldehyde, V(infmax) values for dihydroxyacetone were much higher than those for glycerol and dl-glyceraldehyde.
Until now, three different types of kinases which are active on DHA have been found. The first is DHAK, which is highly specific for DHA (8, 9). However, several enzymes capable of phosphorylating both DHA and glyceraldehyde but with much higher affinity for DHA than for glyceraldehyde are also classified as DHAKs (2, 4, 6, 14). The second is triokinase (EC <IP_ADDRESS>), which is capable of catalyzing the phosphorylation of DHA and glyceraldehyde with similar rates for both substrates but cannot catalyze the phosphorylation of glycerol at all. The third is glycerol kinase (EC <IP_ADDRESS>). This enzyme preferentially phosphorylates glycerol but is also active on DHA and glyceraldehyde. The Km values of glycerol kinase purified from
Escherichia coli were reported to be 0.0013, 0.5, and 0.5 mM for glycerol, DHA, and DL-glyceraldehyde, respectively (9).
Although two kinases from S. pombe IFO 0354 have activity toward glycerol, phosphorylation of glycerol is a very minor
activity of DHAK I and DHAK II, with Vmax values for glycerol that were only 3.3 and 5.5%, respectively, of the velocities for
DHA as the substrate (Table 3). We concluded that these kinases should be classified as DHAKs on the basis of higher
affinities and higher Vmax values for DHA than for the other
substrates.
What do you think? Does this need its own term?
(sorry this is a really old paper but it's all I have)

cc @deustp01

according to the ChEBI entry for dihydroxyacetone, "glycerone" is a synonym for dihydroxyacetone and kinases can catalyze its reaction with ATP to yield ADP plus dihydroxyacetone phosphate (RHEA:15773).
Here it looks like different gene products can enable this reaction with different efficiencies and with different abilities to also act on other substrates. If I understand GO criteria correctly, a difference in the efficiency with which a reaction is enabled does not warrant creating a new GO term, nor would action by the same mechanism on related substrates (hence, a single term "alcohol dehydrogenase activity" appears to be appropriate for methanol, ethanol, ... .
But perhaps here the different enzymes use different mechanisms, something analogous to the distinction in GO between NAD- and NADP-dependent alcohol dehydrogenases. No clue from here as to whether that is the case but if it is (and the sanity test will be persuading our RHEA colleagues that the difference is legitimate), then a new GO term seems right.

It's OK, i'm happy close this and just stick with glycerone kinase activity.....

Ok closing.


[BUG] Double opt-in for registration, newsletter and password recovery

Shopware offers double opt-in functionalities for user and newsletter-registrations as well as a password reset route which works the same way.
In order to support these flows, the application has to "listen" on the following paths and pass the parameters to the respective API-Endpoints mentioned below AND fetch the corresponding configuration (are double-opt ins enabled?) from the API during build time:
Newsletter Confirmation
PWA-Route: /newsletter-subscribe?em=<email-hash>&hash=<subscribe-hash>
API-Route: POST /newsletter/confirm => hash and em body parameters

Parameter
Description

hash
Subscribe Hash

em
Email Hash

Registration Confirmation
PWA-Route: /registration/confirm?em=<email-hash>&hash=<register-hash>
API-Route: POST /account/register-confirm

Parameter
Description

hash
Subscribe Hash

em
Email Hash

Password Recovery
PWA-Route: /account/recover/password?hash=<password-hash>
API-Route: POST /account/recovery-password-confirm

Parameter
Description

hash
Subscribe Hash

newPassword
New Password

newPasswordConfirm
New Password (equality check)

Regarding passwort-recovery: it should be pwaHost + current domain ending here:
https://github.com/vuestorefront/shopware-pwa/blob/master/packages/shopware-6-client/src/services/customerService.ts#L299 the current default is simply wrong :)


[Bug] Backlog Storage ain't reduced after delete inactive subscription

Search before asking

[X] I searched in the issues and found nothing similar.

Version
Apache Pulsar: 2.9.3
AKS - K8s version: 1.22.11
Node OS: Ubuntu 18.04
Minimal reproduce step

Create a topic with 2 subscriptions (sub1, sub2)
Policy configuration: Retention: disabled (0, 0), backlog quota: size 500GB - time -1, messageTTL: 2days
Produce messages to that topic
Create consumer on sub1 and consume
Wait until backlog size large enough -> Delete subscription sub2

What did you expect to see?
Backlog Storage Size should be reduced
What did you see instead?
Backlog Storage size isn't reduced, and keep increasing.
On Local Backlog chart, blue line corresponds to sub1, yellow line is sub2

Anything else?
I get into this issue sometimes while this scenario works normally before, I've no ide. Can someone please point me out what should I do? Is there anyway to delete those storage
It's a big problem for me if data ain't released after cleanup inactive subscriptions.
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Backlog has been reducing since yesterday, I'm not quite sure what was happened

Backlog has been reducing since yesterday, I'm not quite sure what was happened


Add testing to qhub

qhub can test deployment on the 3 supported cloud providers

qhub can spin up dask workers and output the time to spin up as well as any errors

[x] create a qhub destroy command

qhub destroy will automatically retry the destroy

if it fails destroys anything with a unique tag

have warning messages and require user input if it will destroy data

@tylerpotts Is it okay if we make this into two different tickets?
One for testing, which will link Amit's PR for the testing rep,o and another one for the creation of the qhub detroy command.
Also, is the qhub destroy command linked to issue #227 and #99?

@tylerpotts Is it okay if we make this into two different tickets?
One for testing, which will link Amit's PR for the testing rep,o and another one for the creation of the qhub detroy command.
Also, is the qhub destroy command linked to issue #227 and #99?

@laisbsc QHub destroy was actually added in a community PR in https://github.com/Quansight/qhub/pull/219.
I would say it's partially related, but those two issues are different.
#227 is about saving money by not having a running k8s cluster when the compute isn't being used which would involve using the qhub destroy command.
#99 Is about having a backup strategy for QHub for data

@laisbsc QHub destroy was actually added in a community PR in https://github.com/Quansight/qhub/pull/219.
I would say it's partially related, but those two issues are different.
#227 is about saving money by not having a running k8s cluster when the compute isn't being used which would involve using the qhub destroy command.
#99 Is about having a backup strategy for QHub for data

Closing as too broad. @tylerpotts is creating an epic


Add artifact for Apple Silicon

The current mac-os artifact is only for x86_64, and it would be nice to have the arm64 version

Definitely a good idea.
https://github.com/love2d/love/issues/2008#issuecomment-1902277707:

According to https://store.steampowered.com/hwsurvey/Steam-Hardware-Software-Survey-Welcome-to-Steam?platform=mac 70% of mac steam users have an arm64 system, so you might not want to leave that out - and your x64 library will fail to work on those systems anyway unless you strip out the arm64 part of love.app to make it use Rosetta x64 emulation.

I want to do this, but I'd like it to be a fat binary containing both arm64 and x64 (just like love and other native apps and libraries), which probably needs a bit more fiddling with the CI setup and maybe the CMake project than otherwise.

Done in 221bb84


How do I uninstall?

I've been trying to work on existing projects but themekit is blocking. I've set up everything as per docs but still can't manage to upload/download any files. It keeps on giving this error code:
2015/08/11 13:29:54 json: cannot unmarshal string into Go value of type []themekit.Asset

So how do I uninstall?

I am still stuck with this issue and just need help to uninstall. Please advise.

Found it. Just do source theme
and delete the repo. Thanks!

source theme just returns a lot of parse errors, is there a new way to uninstall?

ive got the same issue as @markmakes
Is there a better way to uninstall?

If you installed using our install script you can delete the folder ~/.themekit or you can run rm $(which theme)


fix: add validation for resource group in a case existing SM CRN is not set

Description

Release required?

[ ] No release
[ ] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

"Result": {
│         "errors": [
│             {
│                 "code": "access_forbidden",
│                 "message": "API key validation failed. Error: Could not create IAM service ID using the provided configuration. Verify that the provided API key has the correct permissions. [secrets-manager.03121E]",
│                 "more_info": "https://cloud.ibm.com/apidocs/secrets-manager"
│             }
│         ],
│         "status_code": 403,
│         "trace": "a6a0f442-3235-138b-19c5-193ee382a950"

:tada: This PR is included in version 1.12.4 :tada:
The release is available on:

GitHub release
v1.12.4

Your semantic-release bot :package::rocket:


feat: support all UNIX OSs with sync I/O worker threads

on UNIX-based OSs that are not linux, support I/O by spawning a bunch of worker threads that use synchronous syscalls

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

#286  👈
#285
#284
master

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

Join @rphmeier and the rest of your teammates on  Graphite

Merge activity

Aug 19, 12:52 PM EDT: @pepyakin started a stack merge that includes this pull request via Graphite.


Any plan on Vue@3

Great work, wonder if have some plan on vue@3?

@justforuse thank you.
I actually did not have plans to migrate because I thought that was not relevant anymore (considering how Vue CLI tackles this better).
I Will think about it and maybe revamp it using a CLI template approach


Can't build directly since settings.gradle is not in the repo

settings.gradle has the url to the fabric maven for the loom plugin, and it can't be built without it.
It's not in the repo because it's in .gitignore.
This also affects Better Glow Squids, that's why the build on github actions failed.

Oh, I'm a bit new to the github repos, all done!