Support activeDeadlineSeconds for Tekton pods 🦌

Changes
Kubernetes (and OpenShift) mark Pod as either Terminating — has a
"relatively" short life and will terminate at some point — and
NonTerminating — is supposed to run for ever. Kubernetes does the
difference between the two using the ActiveDeadlineSeconds field. A
Pod with activeDeadlineSeconds set will be considered as
Terminating. For example, Job's Pod have this field set and are
considered as Terminated.
Currently the pods created by tekton fall under the NonTerminating
quota limits of Kubernetes and OpenShift. This can create issues as
generally builds should fall under the separate terminating quota
limits.
This sets the activeDeadlineSeconds field or TaskRun's Pod so that
they are considered Terminating. It also sets the value of this field
to a higher value than the specified (or default) Timeout set on the
TaskRun so that Kubernetes won't try to terminate the Pod before
Pipeline does.
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>/kind feature
/cc @sbwsg @bobcatfish @mattmoor @pritidesai @jerop
Tentatively adding the 0.28 milestone 🙃
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Docs included if any changes are user facing
[x] Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been filled in or deleted (only if no user facing changes)

Release Notes
Set `activeDeadlineSeconds` on Tekton's Pod so that they are considered Terminating for Kubernetes.
This helps supporting ResourceQuota a bit better as now Tekton Pipeline's pod are considered terminating and thus can be using a specific scoped ResourceQuota for those.

Nice. This also has the effect that pods won't run forever if the controller happens to be unavailable or late to process timeouts.

Nice. This also has the effect that pods won't run forever if the controller happens to be unavailable or late to process timeouts.

Indeed 😛

/lgtm

/test pull-tekton-integration-tests

I imagine the active deadline seconds are counted from when the pod is scheduled?
It might be nice to add a note in the docs about this, but it's ok as a follow-up pr if you prefer.
/approve


Service discovery failure at beginning of service startup

Scenario:
when default consul config:  enableHttp2=false
In the service API, the service method need call multiple downstream service API.
At   beginning of service startup, some API Service discovery from consul by service Id will failed, and after for while, the become success with no error.
Code:
It looks like thread issue. in ConsulRegistry, for some serviceName:
  urls = serviceCache.get(serviceName);

The urls return empty list. It means the thread load the service with empty url value from consul discovery.
Changed the consul config to enableHttp2=true. The issue disappear.

It happened only when use https connect to consul. It works for http consul connection

I think this is the same issue. https://github.com/networknt/light-4j/issues/634

 Service discovery failure at beginning of service startup 1.6.x

 Service discovery failure at beginning of service startup master


Slow requests

The are two microservices, the maximum load was about 500 requests per second. This is normal? How can I improve the performance?
Example of code

Each request does the following:

Lookup service discovery for service
Select node from service list
Create new connection to node
Make request
Teardown connection

If you want more performance you can use the caching selector https://godoc.org/github.com/micro/go-micro/selector/cache and client side connection pooling https://godoc.org/github.com/micro/go-micro/client#PoolSize.
You can either do this in code or with flags like so:
--selector=cache # enables in memory caching of discovered nodes
--client_pool_size=10 # enables the client side connection pool

If you want to discuss further please join the slack.


Bug increase zk timeout

Purpose of the pull request
This pull request fixed zk registry timeout problem.
Someone meet the same issue #5792 and maybe #5968

Brief change log
According to Hadoop configuration, increase the default timeout, and added them to the config file

Verify this pull request

This pull request is code cleanup without any test coverage.

oops, mixed with some other commit, I will fix ASAP


Conditional scaffolder steps

This PR introduces conditional steps into the scaffolder. The new if property allows to use handlebar templates to evaluate parameters or step outputs.
Closes #5742
- id: fetch-base # A unique id for the step
  name: Fetch Base # A title displayed in the frontend
  if: '{{ parameters.name }}' # Optional condition, skip the step if not truthy
  action: fetch:cookiecutter # an action to call
  input: # input that is passed as arguments to the action handler
    url: ./template
    values:
      name: '{{ parameters.name }}'

Skipped steps are displayed as such in the UI:

I also added a publish:log action for debugging, to make it easier to see the content of the workspace without publishing to Git and to log contents of parameters and outputs (I know about publish:file, but this one could also be active in production). You could even use the action to write a summary at the end of the process (see also #5848 for that).
The example template is extended to cover an example of a conditional step.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Fox32 could you kindly rebase to fix the issues on master 👍

@Fox32 any idea if it’s possible to hide a step if the condition was not met vs marking it as “Skipped” ?

@Fox32 any idea if it’s possible to hide a step if the condition was not met vs marking it as “Skipped” ?

No, right now they are always marked as skipped. Hiding is not implemented.

@Fox32 any idea if it’s possible to hide a step if the condition was not met vs marking it as “Skipped” ?

No, right now they are always marked as skipped. Hiding is not implemented.

Gotcha, Thanks!


Links  - Padding  - BRCA underline effect

https://www.dannyguo.com/blog/animated-multiline-link-underlines-with-css/
Found this cool solution to add underline effects to multi line spanning links!
a:hover, a:focus{     text-decoration: none;     background-image: linear-gradient(currentColor, currentColor);     background-position: 0% 100%;     background-repeat: no-repeat;     background-size: 0% 2px;     transition: background-size .3s;     }

resolved in #51


feat: typeahead/autocomplete/dropdown suggestions for the 'Pull a model from Ollama.com' input box

Is your feature request related to a problem? Please describe.
This would be a nice-to-have feature for improved UX.  Potentially saves the user time from going to https://ollama.com/models, copy pasting the name of a model and going back to OpenWebUI to pull the model.
Describe the solution you'd like
A typeahead, autocomplete, and/or a suggestion select menu dropdown implemented, which would effectively turn the input box into a search bar, using a hardcoded list of models on the frontend.
Describe alternatives you've considered
Non hard-coded solution, however Ollama.com doesn't have an API that returns a list of the available models to pull, thus we would need to web scrape ollama.com/models to get a list of the models each time
Additional context
I have a suggestion select menu dropdown using a hardcoded list of models implemented locally, can make a draft PR but would like to see if anyone has any input or comment on this.

Heh.. just so happens I know a thing or two about scraping websites to create the API their owners neglected to provide.. let me see what kind of BeautifulSoup I can scrape together 🤣

It looks like your pull request encompasses "official" models only but it's noteworthy that user submitted models have recently become searchable. If these models are ever supported I'd hope to see a warning of some sort!


Implement np.lcm

Building on #4780, this PR implements np.lcm as a ufunc. It's very similar to that PR, and uses much of the same boiler-plate.

@esc, I think this is ready for review – though that might be easier once the gcd branch merges. The only relevant commits here are the most recent two. Two notes:

This doesn't have overflow guards, but neither does numpy.
I've only tested this like the other gufuncs are tested, but this method also doesn't exist in math, so there are fewer tests running on the underlying implementation.

Also, here is a quick benchmark:
import numpy as np
from numba import njit

def lcm(a, b):
    np.lcm(a, b)

nlcm = njit(lcm)
plcm = njit(parallel=True)(lcm)

a = np.random.randint(0, 10000000, 10000)
b = np.random.randint(0, 10000000, 10000)

# warmups
nlcm(a, b)
plcm(a, b)

%timeit lcm(a, b)
1.45 ms ± 22.4 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

%timeit nlcm(a, b)
591 µs ± 6.99 µs per loop (mean ± std. dev. of 7 runs, 1000 loops each)

%timeit plcm(a, b)
133 µs ± 14.4 µs per loop (mean ± std. dev. of 7 runs, 10000 loops each)

@ivirshup thanks for updating this, I have changed the label of the PR accordingly. The benchmarks look impressive from a first glance. 👍

/AzurePipelines run

@ivirshup https://github.com/numba/numba/pull/4780 is merged if you want to rebase/merge in?

@stuartarchibald, done rebasing


python main.py --api takes a long time

I have been running the code for around 10 minutes, and nothing has appeared. I saw some other people had the issue, but I am curious as to how long it should take. Here is my output:
[INFO] Listening to sound from Microphone: #9 - Microphone (High Definition Audio Device)
[INFO] Listening to sound from Speaker: #8 - Speakers (High Definition Audio Device) [Loopback]
[INFO] Adjusting for ambient noise from Default Speaker. Please play sound from Default Speaker...
It is stuck there, there is no error.

@jackokeefe11
playing a random sound such as a youtube video or spotify should solve it.

I figured out the issue, I assumed it would take any input for sound, but because I am using a Windows VM from Mac, it will not pick up sound if you play it outside of the VM. I played sound within it, and it worked right away.


Solve Bee1017

https://judge.beecrowd.com/pt/problems/view/1017

Testes Ok

https://judge.beecrowd.com/pt/runs/code/42346445


bug: Incorrect colors on .jsx  files.

Did you check docs and existing issues?

[X] I have read all the solarized-osaka.nvim docs
[X] I have searched the existing issues of solarized-osaka.nvim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
0.10
Operating system/version
Arch Linux
Describe the bug
The colors are not properly working for the html tags which you can see from the image below:

On a .tsx files this issue goes away.
Steps To Reproduce

Create a .jsx file
Open

Expected Behavior
The colors should be consistent as they are when using .tsx format but instead are not working properly.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "craftzdog/solarized-osaka.nvim",
  "craftzdog/solarized-osaka.nvim",
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("solarized-osaka")
-- add anything else here

I don't see the same issue in <> colors in JSX 🤔

Nvim v0.9.5

I did test this on v0.9.5 and and currentl on nightly build v0.10.0 and for me it shows like this.
I do not have another system on which i could test this.

Can you share what the Inspect output looks like?
Mine is:

In my case, @tag.delimiter.javascript is applied to the angle brackets.
This theme has Special = { fg = c.orange500 }, so they are orange.
Can you check javascript is installed properly? Run TSInstallInfo.
As you know, tokyonight doesn't have @tag.delimiter.javascript. So, I don't understand why it is necessary on this theme.

For me it also outputs the same but the colors are evident as they are.
This occurs only in jsx and html files but in tsx it correctly uses the right colors.
I have tested on a clean build of LazyVim too without any of my configuration and the results stay the same.
Explicitly specifying it might make it a better solution so there are not any edge cases.

These screenshots on this PR also show the same issue occurs in Vue as well and is not something that is occurring on only my setup.


When I launch tensorboard, it is completely empty. The figure is shown as follows:

When I launch tensorboard, it is completely empty. The figure is shown as follows:

The log file is complete. However, the problem is surprising, and I hope to get solution from relevant experts

me too

Same here. No suspicious logs were found.

Optional solution: https://github.com/HFAiLab/jupyterlab_tensorboard_pro
Also you might be able to fix this by installing tornado==6.2


No module named 'gfootball.eval_server'

When I follow the steps below:
sudo apt-get install git cmake build-essential libgl1-mesa-dev libsdl2-dev
libsdl2-image-dev libsdl2-ttf-dev libsdl2-gfx-dev libboost-all-dev
libdirectfb-dev libst-dev mesa-utils xvfb x11vnc libsdl-sge-dev python3-pip
pip install gfootball
pip3 install grpcio google-cloud-storage
The following problems occurred:
~$python -c "import gfootball.eval_server"
Traceback (most recent call last):
File "", line 1, in
ModuleNotFoundError: No module named 'gfootball.eval_server'
But "import gfootball" is good.
What's wrong with my steps?
thanks a lot!

Most likely, the reason is that you're mixing python2 and python3. Many systems still use python2 when you type python in the Terminal. The same goes for pip and pip3. So the correct way is pip3 install gfootball instead of pip install gfootball.
And then you can run python3 -c "import gfootball.eval_server" without errors.
I'll send a pull request to correct the Readme.


Java API for PQS (Phoenix thin client)

This document outlines C# and .Net SDK to access PQS. One of the key thing here is using hbase cluster
URL.
Do you have similar way to access PQS via sqlline-thin or java API. Currently I specify details of one of the PQS (generally hosted on Region servers of HBase cluster) like below
/usr/hdp/<IP_ADDRESS>-1/phoenix/bin$ ./sqlline-thin.py http://[One of the region server IP]:8765
Is there anyway through HDInsight wherein I can utilize single load balancer or edge node to access PQS

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

ID: 2e3f3527-a43e-f1b4-2353-58291655e1ed
Version Independent ID: 9cfa27a2-1449-7493-ac95-6be77ae4abf9
Content: Phoenix Query Server REST SDK - Azure HDInsight
Content Source: articles/hdinsight/hbase/apache-hbase-using-phoenix-query-server-rest-sdk.md
Service: hdinsight
GitHub Login: @ashishthaps
Microsoft Alias: ashishth

@dhananjaypatkar Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@dhananjaypatkar thanks for the question. Sorry we are slow to respond!
I checked with our developer, and we believe to access sqlline-thin.py in general we cannot use an HDInsight hbase cluster URL; this is due to the way the sqlline-thin.py script is written. It expects the format :8765. You could try to modify the script to not avoid this requirement and then the same principle should work.
Try pointing to https://.azurehdinsight.net/hbasephoenix or https://.azurehdinsight.net/hbasephoenix0
Thanks, Jason

#please-close


fix(config): improving SetConfig function

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[PackageModel] Make package dependencies immutable

We also get some performance boost when creating packages noticeable using this test https://github.com/apple/swift-package-manager/pull/858

@swift-ci please test

@swift-ci please test

@swift-ci please test


Identifying the table in testeff output file

Improving the method to identify the table in the testeff output in the camera_efficiency module.
See: https://github.com/gammasim/gammasim-tools/pull/33#discussion_r444902659

Potentially resolved by #36.

Solved by #36, closing.


setup_dxvk.sh: install with reflinks if supported

Some filesystems, such as XFS and BTRFS, may perform a lightweight copy with CoW. This is most useful for rarely changing content, and the usecase of installing DXVK is one of such usecases, as library files are usually not modified (might be removed, but it's not modification).
So make use of it whenever supported.

Isn't this the default behaviour of cp already? Not that I'm opposed to pulling this in but this should only affect people who use BTRFS on old distros.

Isn't this the default behaviour of cp already? Not that I'm opposed to pulling this in but this should only affect people who use BTRFS on old distros.

Well, cp only recently started defaulting to --relfink=always. I didn't find offhand the exact version, but I found that it should be after 8.32, and the 8.32 release is the one used in latest Ubuntu 22.04. So I would say, this affects a lot of people as Ubuntu and its derivatives like Mint are pretty popular.
I should also note that reflinks are supported not only on BTRFS but on XFS as well.

ping


Fuzzing Irmin.Types

The goal is to fuzz Irmin.Types to ensure serialization and deserialization are mutually inverse functions.
Fixes https://github.com/mirage/irmin/issues/670.

There is a recipe in cstruct for running these fuzz tests automatically from Drone CI (which has very nice bare metal boxes) https://github.com/mirage/ocaml-cstruct/blob/master/.drone.yml

@CraigFe Here is the rationale for introducing the rewriter:

Essentially, we use a code rewriter (see fuzz/rewriter/rewriter.ml) to replace the [%impl_record 5] and [%impl_variant 5] extension points with code to check dynamically-generated records (resp. variants) with up to 5 fields (resp. cases). This number can be adjusted at will, but the number of lines generated by impl_variant is exponential in the number of cases, so 5 seemed like a reasonable compromise.
As for the reason why we need to use a code rewriter in the first place, the issue is essentially that the public API for records and variants in Irmin.Type expects us to build functions whose signature (and, in particular, whose number of arguments) depends on the fields of the record or enum that we are building.
Here is an example from the documentation:
type menu = { restaurant : string; items : (string * int32) list }

let t =
  record "t" (fun restaurant items -> { restaurant; items })
  |+ field "restaurant" string (fun t -> t.restaurant)
  |+ field "items" (list (pair string int32)) (fun t -> t.items)
  |> sealr

As you can see, we need to pass a "constructor function" to record (here fun restaurant items -> { restaurant; items }) whose signature depends on the number and type of fields in the record. But, since the fuzzer is building record types dynamically, and it is not possible–to the best of my knowledge–to build functions whose signature depends on runtime information, we have a bit of a problem.
My compromise was to use code generation as a way to define "constructor functions" for all possible signatures (or, to be exact, for all possible number of arguments, since the type of each argument is made generic by hiding it inside a GADT) at build time. This does mean, however, that we can't test dynamic records, variants or enums whose number of fields exceeds the number of generated "constructor functions", hence the limit of 5 here.

@CraigFe Here is the rationale for introducing the rewriter:

...
As you can see, we need to pass a "constructor function" to record (here fun restaurant items -> { restaurant; items }) whose signature depends on the number and type of fields in the record. But, since the fuzzer is building record types dynamically, and it is not possible–to the best of my knowledge–to build functions whose signature depends on runtime information, we have a bit of a problem.
...

For the record, this is possible using extensible types such as those in Irmin's Witness module:
module W = Irmin_type.Witness

type witnessed = E : ('a W.t * 'a) -> witnessed

let () =
  Random.self_init ();
  let w0 : int W.t = W.make ()
  and w1 : (unit -> int) W.t = W.make ()
  and w2 : (unit -> unit -> int) W.t = W.make () in
  (* Generate a list of 10 functions of differing -arities *)
  List.init 10 (fun _ ->
      match Random.int 3 with
      | 0 -> (0, E (w0, 10))
      | 1 -> (1, E (w1, fun () -> 10))
      | 2 -> (2, E (w2, fun () () -> 10))
      | _ -> assert false)
  (* Use witnesses to pass the appropriate number of units *)
  |> List.map (fun (arg_count, E (wit_actual, f)) ->
         match arg_count with
         | 0 -> W.cast wit_actual w0 f |> Option.get
         | 1 -> W.cast wit_actual w1 f |> Option.get |> fun f -> f ()
         | 2 -> W.cast wit_actual w2 f |> Option.get |> fun f -> f () ()
         | _ -> assert false)
  (* Sum and print: 100 *)
  |> List.fold_left ( + ) 0
  |> Printf.printf "%d"

so it's not clear to me that the rewriter is strictly necessary. Regardless, the rewriter may be the clearest way to express this logic, I don't know :woman_shrugging:

@CraigFe Thanks a lot for the snippet, I'll look into it.
From what I understand, you still have to create a witness for every function signature that you want to support, right? And, assuming that I had defined all the witnesses that I needed by hand, I would still need to augment my open_record with as many field declarations as needed, which would still require code generation I believe.

You need to create a witness wherever you want to recover type information at run-time. In this case, that probably means any_type → witnessed_type and any_case_type → witnessed_case_type. Augmenting the open record with the corresponding fields can perhaps be done by pairing it with a heterogeneous list of witnesses of field types that it expects, then folding that against the list of witnessed fields.
I'm not sure it's worth looking into; I only mentioned it as a suggestion that it's probably possible :slightly_smiling_face: FWIW, I think the current rewriter implementation is fine, and perhaps more maintainable than another full dynamic types library running alongside Irmin.Type.

Is this ready to merge? Can you run dune fmt && dune promote and rebase?

This is not ready to merge.

[ ] Outstanding review comments re. the use of enum and commented-out integer fuzzing (which is perhaps unblocked now thanks to @liautaud's work on floating-point values)
[ ] Needs an opam file to state necessary dependencies / pins (I think we still need unreleased crowbar)
[ ] We should probably add a dune rule to have CI ensure that this actually compiles (as done here)

I have a rebased form of this PR at CraigFe/types-fuzz that deals with the above issues.
It also refactors the rewriter to use Ppxlib rather than the existing ad-hoc string processing. This has the advantage of keeping location information intact in fuzz/fuzz_types.ml. At some point, we can pull out the common components between this rewriter and ppx_irmin, of which there are a few, but that will require some Opam package restructuring.
@pascutto: Let me know what you think :slightly_smiling_face:

Done. Let's see what the CI makes of it :slightly_smiling_face:
We should set up a fuzz job for this somewhere, otherwise it'll probably just stagnate.

The CI is happy and so am I. Merge when ready :slightly_smiling_face:

Merge when ready slightly_smiling_face

Ready.


improve about us section

i would like to improve the about us section by properly aligning the heading with the proper font size.

would like to help in this with more enhancement or styling there

Could you please assign the issue to me ? I' am eager to drive in and make a meaningful impact .Thankyou for considering my request!"


Prepare package for release on PyPi

What does this PR do?
prepare this package for release on PyPi
Why was this PR needed?
fix a typo and refine the setup.py
What are the relevant issue numbers?
No issue.
Does this PR meet the acceptance criteria?

[x] Passes flake8 (refer to .travis.yml)
[x] Docs are buildable
[x] Branch has no merge conflicts with master
[x] Is covered by a test

Coverage remained the same at 90.988% when pulling fae52bc42f4cbd58ffb0f0f7a86c5b287d900113 on b3nab:master into 118fc6c004942fe4d270a2097cce065bd29540c7 on ping:master.


Add two constants for span.kind

@objectiser Besides your pr, I thought we should add these two constants, as we had client and server constants.

Mention producer/consumer are only in messaging scenario.
Add a link to Modelling special circumstances.
Fix an invalid link in Tags class comment.

@wu-sheng :+1:

Wait 24 hours to merge.


Further shrink jupyter/minimal and jupyter/demo

A separate PR for shrinking jupyter/demo and jupyter/minimal.
As requested by @rgbkrk  https://github.com/jupyter/docker-demo-images/pull/29#issuecomment-99899375

Thank you @sumitsahrawat!


Allow margins to get set before Show() is called.

This should solve the app crashing when you set the margin parameters.

Thanks a lot! Could you however use C# 6 null check? Like so: leftMarginConstraint?.value = value

Oh that is a neat trick, I will make that change.

Hmm doesn't seem like you can use it like that.
I get a "the left-hand side of an assignment must be a variable property or indexer" compiler error.
Mentioned here as well
http://stackoverflow.com/questions/35887106/using-the-null-conditional-operator-on-the-left-hand-side-of-an-assignment

You are right, i'm sorry! Will merge this. Thanks!


Remove faulty qubits and gates sections from transpile()

Summary
This commit removes the "faulty qubits" support from the transpile(). Previously the transpile() function would attempt to filter out any faulty qubits or gates reported in a BackendV1's BackendProperties object and remap the qubits based on that. However, this code was never actually functioning as the first BackendV1 based backend to attempt to set these flags is causing an internal panic in rustworkx trying to created the remapped coupling map. This code path was mildly controversial at the time of introduction (and was already reverted once in the past) and since it provably isn't valid anymore (and I wasn't clear it ever functioned as expected) as the only backend to leverage the feature crashes transpile() this commit just opts to remove it as a potential breaking change.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.02%) to 85.397%

Files with Coverage Reduction
New Missed Lines
%

qiskit/compiler/transpiler.py
1
88.89%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
1
95.14%

qiskit/circuit/quantumcircuit.py
3
93.96%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
67366

Relevant Lines:
78886

💛  - Coveralls

I think after #9911 and #9840 merge I think we should update the release note to point people to using BackendV2Converter with the filter flag as an alternative path.

confirming. Let's remove it.

The second two references need to stay: backendproperties is the model object that defines the payload format and the backend_compat file supports filtering a target to not include them as an alternative to this code. The dense layout reference should be removed in the patch series ending in #9927. The vf2_utils comment I think we can either leave in place or update in a followup. It's just explaining why there is an if statement guard for a condition which should be impossible.

I went ahead and enqueued this for merging because of those remaining references only 1 arguably needs to be update, but it's just a code comment


Nested schenes no longer working?

Do others also fail to use nested schenes?
Today I updated to 3.2.4 but it seems I'm can no longer navigate to any of the nested ones.
Version

react-native-router-flux v3.2.4
react-native v0.22.2

Please give example (by modification of this component Example)

Ok I close the issue until I create an example of this.

Could it be duplicate of #370?

I'll check it and let you know!


"Internal error" when I try to create account

I want to leave a comment on a Django Snippet on djangosnippets.org, which requires logging in. I don't currently have an account, but as I don't see a button for creating an account, I tried the log in buttons. Here's what happened with the various log in methods:

BitBucket: "Social Network Login Failure. An error occurred while attempting to login via your social network account."
GitHub: "There's been an internal error on the server; it probably wasn't anything you did, so don't worry too much about it. The site administrators have been emailed with the details, and they'll probably get it fixed real soon."
Reset password with my email address which currently doesn't have an account: "There's been an internal error on the server; it probably wasn't anything you did, so don't worry too much about it. The site administrators have been emailed with the details, and they'll probably get it fixed real soon."
Logging in with my email (which hasn't been registered) and a random keyboard mash password returns me to the login page without even a message like "log in failed"
I haven't tried Twitter yet.

I have not received any emails from Django Snippets after these registration attempts.
If it is intentional that you can't create an account, it would probably be good to clarify this and make the error messages more informative.
The FAQ says "I've done my best to make signing up as easy as possible", linking to https://djangosnippets.org/accounts/register/, but that is a dead link.

Thank you for your email Michael I will look into this and get back to you
Kind regards
Chris
On Fri, 11 Sep 2020 at 05:43, Michael Chen<EMAIL_ADDRESS>wrote:

I want to leave a comment on a Django Snippet, which requires logging in.
I don't currently have an account, but as I don't see a button for creating
an account, I tried the log in buttons. Here's what happened with the
various log in methods:

BitBucket: "Social Network Login Failure. An error occurred while
attempting to login via your social network account."

GitHub: "There's been an internal error on the server; it probably
wasn't anything you did, so don't worry too much about it. The site
administrators have been emailed with the details, and they'll probably get
it fixed real soon."

Reset password with my email address which currently doesn't have an
account: "There's been an internal error on the server; it probably wasn't
anything you did, so don't worry too much about it. The site administrators
have been emailed with the details, and they'll probably get it fixed real
soon."

If it is intentional that you can't create an account, it would probably
be good to clarify this and make the error messages more informative.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/django/djangosnippets.org/issues/170, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJW5M7YFYCLTBMXJ7G7LQLSFGTH7ANCNFSM4RGSVPGA
.

I have re-created the issue attempting to create a new account and I also get "Error
There's been an internal error on the server; it probably wasn't anything you did, so don't worry too much about it. The site administrators have been emailed with the details, and they'll probably get it fixed real soon.".
There is nothing showing up in  sentry so I will need to look a little longer to find the issue.

@ML-Chen  I haven't forgotten about this. I've been dealing with a severe technical issue at work but will look at this when things have settled. I did have a quick look and it looks like the sentry configuration may be a bit dated.

Don't worry, take your time

So I have tried twitter and github and It is working for me.
Bitbucket is definitely broken and it looks like it may be issues with the way in which the app has been setup. Will need to dig deeper on this one.
I cant seem to re-create the internal server error.
@ML-Chen can you retry with twitter and/or github?
We have sorted out our sentry configuration so we will be able to capture the error and diagnose.

Awesome, I was able to create an account with GitHub.

Thats great news 2/3 is not bad! I'll leave this open till we sort out bitbucket issues

I tried creating an account both via Twitter and via GitHub, and got server errors in both cases. It does seem to associate my email with an account, though. If I try to use the password reset form, I also get a server error.

Thanks Omen I’ll have a look 👍
On Tue, 23 Feb 2021 at 22:23, Omen Apps<EMAIL_ADDRESS>wrote:

I tried creating an account both via Twitter and via GitHub, and got
server errors in both cases. It does seem to associate my email with an
account, though. If I try to use the password reset form, I also get a
server error.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/django/djangosnippets.org/issues/170#issuecomment-784556057,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJW5M2KQM6ZCDE44QANYC3TAQTFHANCNFSM4RGSVPGA
.

@OmenApps Ive re-created the issue on the live site and will need to figure out what has changed as we haven't deployed for a while.
Will  let you know when its fixed 👍

The issue seems to lie with sendgrid. I'm going to wait till these issues are resolved before doing further investigations

The error is coming out of Python smtp lib when trying to send an email. Going to try recreate this locally with python3.7

@OmenApps  This has been resolved. Please try log in and see if you can get in. You twitter and github accounts were successfully created.
Im not sure if Sendgrid has changed but we were not following this documentation:
https://sendgrid.com/docs/for-developers/sending-email/django/


Support for LMStudio

While I won't have time to experiment and patch privy/lib/extension/src/ai/AIClient.ts, here's some info about a hypothetical case of provider.startsWith("openai") or provider.startsWith("lm-studio"):
2024-11-29 15:01:52  [INFO] [LM STUDIO SERVER] Supported endpoints:
2024-11-29 15:01:52  [INFO] [LM STUDIO SERVER] ->	GET  http://<IP_ADDRESS>:5001/v1/models
2024-11-29 15:01:52  [INFO] [LM STUDIO SERVER] ->	POST http://<IP_ADDRESS>:5001/v1/chat/completions
2024-11-29 15:01:52  [INFO] [LM STUDIO SERVER] ->	POST http://<IP_ADDRESS>:5001/v1/completions
2024-11-29 15:01:52  [INFO] [LM STUDIO SERVER] ->	POST http://<IP_ADDRESS>:5001/v1/embeddings

Docs are here: https://lmstudio.ai/docs/api/openai-api
OpenAI's client should be usable with LM Studio, per LM Studio's documentation:
import OpenAI from 'openai';

const client = new OpenAI({
+  baseUrl: "http://localhost:1234/v1"
});

They provide a Python example, presumably a Typescript/Javascript one is similar:
# Example: reuse your existing OpenAI setup
from openai import OpenAI

# Point to the local server
client = OpenAI(base_url="http://localhost:1234/v1", api_key="lm-studio")

completion = client.chat.completions.create(
  model="model-identifier",
  messages=[
    {"role": "system", "content": "Always answer in rhymes."},
    {"role": "user", "content": "Introduce yourself."}
  ],
  temperature=0.7,
)

print(completion.choices[0].message)

However, I have used /v1/chat/completions directly with fetch() to avoid a dependency for something that's reasonably simple to do without extra dependencies:
fetch(`http://${host}/v1/chat/completions`, {
    method: 'POST',
    headers: {
        'Content-Type': 'application/json'
    },
    body: JSON.stringify(request)
}).then(response => {
    if (response.ok) {
        return response.json();
    }
    throw new Error('Failed to get response: ' + response.statusText);
}).then(data => {
    console.log('Response:', data);
    let raw_json = data.choices[0].message.content;
    // we could decode the response, but we could also just write it out:
    console.log(raw_json);
}).catch(error => {
    console.error('Error:', error);
});

I've built the request like so:
let request = {
    messages: [
        { role: 'system', content: 'You are a helpful assistant and will return a response in JSON.' }, // note: this is because I was supplying a JSON schema.
        { role: 'assistant', content: prompt }
    ],
    temperature: 0.7,
    max_tokens: -1,
    stream: false,
};
if (model) {
    request.model = model; // model can be omitted if exactly one model is loaded in LM Studio GUI
};
if (schema) { // we want to force the model to generate JSON according to a JSON schema
    request.response_format = {
        type: 'json_schema',
        json_schema: {
            name: 'some.name.for.schema.json',
            strict: true,
            schema: schema // deserialized JSON Schema object
        },
    }
}

Given at least some endpoints are supported by OpenAI, this might be a common enough endpoint that it might unlock usability of more runtimes.
LM Studio is rather trivial to experiment with on Windows and Mac; I've had luck on AMD GPU on Windows and on Apple Silicon. I have not tried LM Studio on Linux at this time.

Note that depending on what you want to do, this might even simplify the codebase given Ollama says they have OpenAI API compatibility nowadays:
https://ollama.com/blog/openai-compatibility


[DataFactory]Package store support for  Azure-SSIS integration runtime

https://github.com/Azure/azure-rest-api-specs/pull/8689

Txt uploaded.
No changes in tag. Release notes included in changelog.
No intension to update version.


teams app uninstall with increased test coverage

Solving #843

Coverage remained the same at 100.0% when pulling 81a2b6a247dc14b61dbe34208fc8f27d77b07054 on vman:teamsappuninstall2 into c32043955bbeb553703afce0f1d3cb2e1126d15b on pnp:dev.

Awesome @vman! I'll review it shortly!

Thanks for the update! I'll review it shortly 👍

Merged manually. Thank you! 👍


Added Lmax option and fixed broken legend

You can now use "--Lmax=100000" for example as an option to set Lmax

This now also depends on my request at LSSTDESC/SACC#13, so that should likely be approved before merging this pull request.


Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/CORPUSERS/xxx/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/CORPUSERS/xxx/projects/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=

What did you do?

Trying to run go get on a repository hosted on code commit:
go get -d -u git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo
What did you expect to see?
The repository cloned and dependencies installed.
What did you see instead?
package git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo: unrecognized import path "git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo" (parse https://git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo?go-get=1: no go-import meta tags ())
My machine is properly setup with SSH keys so git clone ssh://git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo works fine. Also take note that my end goal is to the the build pipeline on AWS Code build run the dockerfile in the end but as of now it gives the same result as running it locally.

Try go get -d -u git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo.git
See https://golang.org/cmd/go/#hdr-Remote_import_paths

Try go get -d -u git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo.git
See https://golang.org/cmd/go/#hdr-Remote_import_paths

Adding .git to the end gave me another problem, so at least some progress. :)
package git-codecommit.eu-west-1.amazonaws.com/v1/repos/battery-historian.git: cannot download, git://git-codecommit.eu-west-1.amazonaws.com/v1/repos/myRepo uses insecure protocol
Isn't Go get using HTTPS by default? Is that unsercure?

Perhaps not directly related, but #25792 is older AWS CodeCommit vgo bug, and includes final comment:

It's fixed with one small detail.
Package imports as git-codecommit.us-east-1.amazonaws.com/v1/repos/return2.git require that packages go.mod contains module git-codecommit.us-east-1.amazonaws.com/v1/repos/return2.git.

@Baxtex Probably worth at least skimming that bug.

@Baxtex, the go command probes .git paths using all known protocol schemes for the VCS that meet security requirements (see here).
Can you git clone or git ls-remote the https URL for the path?

After some more research, I discovered that I probably don't need to do go get/git clone at all. As I'm running this on AWS code build, it already has access to the repository. So I realized I could use the COPY command instead. Here is my updated dockerfile:
#Use latest golang version
FROM golang:latest

MAINTAINER Me

#Install Java
RUN apt-get -y update && apt-get install -y openjdk-8-jre-headless

#Get the battery historian code
WORKDIR /go/src/codecommit/battery-historian
COPY . /go/src/codecommit/battery-historian

RUN go get ./...
#RUN go get -d -u
RUN go run setup.go

#Open default port
EXPOSE 8888

#Run the start file for battery historian in a container
CMD go run cmd/battery-historian/battery-historian.go --port 8888 --name my-battery-historian

As you can see, I use copy to copy in my own battery historian repository instead of using go get. Do I have to do something more? Because when I study my logs, it looks like it cannot find all my dependencies:

Step 7/9 : RUN go run setup.go
 ---> Running in f1735a0eb865

Generating JS runfiles...
Couldn't generate runfile: failed to run command "python /go/src/codecommit/battery-historian/third_party/closure-library/closure/bin/build/depswriter.py --root=/go/src/codecommit/battery-historian/third_party/closure-library/closure/goog --root_with_prefix=js ../../../../js":
  exit status 2
  python: can't open file '/go/src/codecommit/battery-historian/third_party/closure-library/closure/bin/build/depswriter.py': [Errno 2] No such file or directory

For reference, the repo I got on Code commit is a clone of googles battery historian: https://github.com/google/battery-historian

@Baxtex - Have you manually checked that third_party/closure-library/closure/bin/build/depswriter.py exists ?
Could you use rebuild=true and run docker build again ?
And in any case, this does not seem to be a Go issue anymore as you are just copying files.

I don't know if you are still having issues, but i've managed to solve this by running this command
git config --global<EMAIL_ADDRESS>"ssh://git-codecommit.${AWS_REGION}.amazonaws.com"
Wich added those lines in the global .gitconfig
[url<EMAIL_ADDRESS>insteadOf = ssh://git-codecommit.${AWS_REGION}.amazonaws.com
And so now i'm able to get Go packages from AWS CodeCommit using this command
go get git-codecommit.${AWS_REGION}.amazonaws.com/v1/repos/${REPOSITORY_NAME}.git
The ${AWS_REGION} placeholder has to be replaced with the actual region the repository is, for example in my case is EUWest1, so in the go get command i replaced it with "eu-west-1".
The ${REPOSITORY_NAME} placeholder is, well you guessed it, the name of the repository your GO package is in. For example, if your package is in a repository called "TestPackage" you must replace that placeholder with "TestPackage" as well (i don't know if it works having all lowercase too).
TL;DR:
-Run this command
git config --global<EMAIL_ADDRESS>"ssh://git-codecommit.${AWS_REGION}.amazonaws.com"
replacing ${AWS_REGION} with the Region the CodeCommit repository is in (for example eu-west-1)

Get the package via go get using this command
go get git-codecommit.${AWS_REGION}.amazonaws.com/v1/repos/${REPOSITORY_NAME}.git
where the ${AWS_REGION} placeholder is the same as before and the ${REPOSITORY_NAME} is the name of the repository (in your case should be myRepo). Don't forget to add the .git at the end, otherwise it won't work


Support for XvT/BoP

These are patches that allow it to work quite a bit in these older games.
However I would very much like you to review it and possibly help out with a few things.
Biggest issue: I removed the memset(buffer, 0x80, ...) that I later realized is necessary for some colorkey support. I didn't find a way to make it work without that, and I didn't find what removing that breaks.
Other issues, in order of priority, in case you have some hints/ideas:

There is a crash on leaving 3D mode. I haven't been able to get a useful backtrace on that. Possibly wild function pointers?
The image in the CMD/HUD is offset, the 3D models appear in the lower half (though at full scale) instead of centered.
The movies play wrong,  looks like16 bit rendered onto a 32 bit surface.

Any comments/help welcome.

Updated version.
I haven't squashed bugfix commits with the earlier changes they relate to, please tell me if you'd like me to that kind of cleanup.
Everything except the movies now seems to work reasonably.

This is no longer useful I think. Thanks for starting this project.


Trailing candidate needs text display in breakdown improved

Describe the bug
Right now the trailing candidate's text is anchored go the start of their bar, so it disappears if they get a low proportion of the votes.
Expected behavior
Text needs to be right-justified
Screenshots

Smartphone (please complete the following information):

Device: Galaxy S10+
OS: Android 10
Browser Firefox
Version Beta

Thanks! We're working on this.

It's not just the trailing candidate apparently. If the current leader gets few votes in a batch, their percentage gets cut off.

Yep, we have a change in motion to address that as well.

For now we've reverted in #334


restrict to versions of numpy that still have numpy.distutils

Fixes danielreardon/scintools#21
numpy.distutils is removed from numpy as detailed in the numpy documentation
These changes will prevent scintools from using the newer versions of numpy until it can be transitioned away from using this module.
requirements.txt has also been updated to match install_requires in setup.py.

Thanks John, I have merged this change with some other updates to the requirements - removing a specific skimage version and removing corner


$identityName and  clarification in the documentation

$identityName is referred to here in step 3:
https://docs.microsoft.com/en-us/azure/key-vault/general/key-vault-integrate-kubernetes#assign-managed-identity
<idname> is referred to here in step 2:
https://azure.github.io/secrets-store-csi-driver-provider-azure/configurations/identity-access-modes/pod-identity-mode/
I'm more than likely just dense, but it isn't clear to me from the context and the rest of the documentation what this is supposed to be. Can someone clarify?

Actually, I think I figured it out. It is just an arbitrary name. Wasn't clear if this was a value that needed to be inherited from one of the previous steps.


Add 'getting help' section to point customers to Azure CLI Repo for help

Add 'getting help' section to point customers to Azure CLI Repo for help with CLI issues in their inline scripts.

Thanks @jamesc for the contribution.


Is there an ETA for AppBar and BoundingBox example?

I was under the (quite possibly unwarranted) impression that these capabilities were coming soon after the MRDL merge began, but it seems like the process has stalled. Are there any updates in terms of timeline for these and/or any other features coming from MRDL?

We've been working on multi-pointer support and other input prereqs for bounding box manipulation. Once #1486 is merged into the dev branch we'll create a new feature branch for the bounding box. In the meantime any input or suggestions you have for #1486 are always welcome. (C&V from #1497)

Duplicate of #1497

@jimstack We opened a pull request for the Bounding Box normal mode with Two hand Gesture.
#1727
Adjust mode Bounding Box with handles will be added through a different pull request.


codeigniter4/codeigniter4 package not exists in packagist

For example, I want to get the latest develop branch of the framework to my current app-starter app. The only way is by clone the project and copy into vendor directory or manually register this repository into composer.json.
How about add codeigniter4/codeigniter4 to packagist ?

codeigniter4/codeigniter4 is not intended to be directly installed and make it available as a composer package would encourage that use and create many more support headaches.
You can easily add it to your composer.json file as a repository. See this example file: https://github.com/lonnieezell/myth-auth/blob/develop/composer.json#L19

Actually you probably want to use the builds script that toggles between framework and codeigniter4: admin/starter/builds. If you're using a post-release version of the AppStarter this should be in your project root. Simply run ./builds and it will switch you to using the latest develop branch of codeigniter4/codeigniter4 so when you run composer update you'll always be fresh.

ok, thank you ;)

That sounds like a much better way of doing it, @MGatner Thanks for the reminder. :)

@MGatner I created PR #2756 to handle ./builds development try to get all dev version of dependencies with prefer-stable" => true, which should only get dev version when required in composer.json


StructureBoard-position does not update if 'right' would become 0

Description
The StructureBoard receives a right: ${scrollBarWidth } when scrollBarWidth !== 0. This makes sense, because the StructureBoard would be mis-positioned otherwise. If the scrollBarWidth changes after page load in such a way that scrollBarWidth === 0, right-property is never updated.

## Steps to reproduce

Add the following (inline) style to body: <body style="display: none">
Add a script-tag: <script>setTimemout(() => $('body').css({ 'display': 'initial' }), 2000</script>
Open StructureBoard once the page appears

What happens:
Step 1 there is no body. Therefore the width of the CMS Toolbar will be zero. The CMS is initiating the structureboard. It will run the following lines of code:
https://github.com/divio/django-cms/blob/75c39582ec5a58bee70fc5d67a72f90c47fc4b98/cms/static/cms/js/modules/cms.structureboard.js#L359-L360
The innerWidth of the window is the actual width of the page (eg. 1240px). The width of the toolbar is 0. Therefor the scrollBarWidth is a negative number (eg. -1240).
The next lines of code will apply this negative scrollBarWidth:
https://github.com/divio/django-cms/blob/75c39582ec5a58bee70fc5d67a72f90c47fc4b98/cms/static/cms/js/modules/cms.structureboard.js#L366-L368
Step 2 will reveal the body and CMS Toolbar. Now the width of the toolbar is the same as the width of the window.
Step 3 will show the structureboard and run the same code as in step 1.
https://github.com/divio/django-cms/blob/75c39582ec5a58bee70fc5d67a72f90c47fc4b98/cms/static/cms/js/modules/cms.structureboard.js#L359-L360
This time the result is scrollBarWidth = 0 which is correct. The right property should be changed to right = 0. But!
https://github.com/divio/django-cms/blob/75c39582ec5a58bee70fc5d67a72f90c47fc4b98/cms/static/cms/js/modules/cms.structureboard.js#L366-L368
The if(scrollBarWidth) (ie. if (0)) prevents this line from being executed.

Expected behaviour
this.ui.toolbar.css('right', scrollBarWidth); should be 0.
Actual behaviour
this.ui.toolbar.css('right', scrollBarWidth); remains negative.
Additional information
The easy fix would be to always execute this.ui.toolbar.css('right', scrollBarWidth); (no check if it is zero). I do not know why this line is added, so I'm not sure it can just be removed.

Next steps for this issue is to first validate / check that the issue exists and then create a solution.


Fixes in preparation to homework 2

Pull Request Test Coverage Report for Build 870

25 of 27 (92.59%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+4.0e**-05%**) to 79.933%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/typecheck.js
4
6
66.67%

Totals

Change from base Build 869:
4.0e-05%

Covered Lines:
8375

Relevant Lines:
10084

💛  - Coveralls

Whoa LGTM caught an actual bug! It's a miracle!


[FIX] At end of Everest fix Agent ID

https://github.com/valory-xyz/trader-quickstart/blob/e828867f0309cbe1f3aa21276a0219dd6f469764/run_service.sh#L315

Addressed in https://github.com/valory-xyz/trader-quickstart/pull/196


move validation to parity-wasm

I'm a user of parity-wasm but not of wasmi. When I load wasm from an untrusted source, or I manipulate a wasm AST, I'd like to ensure it is valid. There is a validator in https://github.com/paritytech/wasmi/blob/master/src/validation/mod.rs#L168 that I would like to use without adding a dependency on wasmi. Are you open to moving that code to the parity-wasm crate, along with the ModuleContext machinery required to make it work? I'm sorry if this questions a design decision you already made - its OK if you don't feel its appropriate to move this code.

The validator is used to be inside parity-wasm. It was moved simply because it was essential for wasmi and at that time it was already obvious that it would be more of loading wasm module rather than just validating it. And recently we started to use this tight coupling for implementing compilation of wasm into a format which is more convenient for interpretation. So it might be a little complicated to move it as is.
That said, I still think it may be useful to have validation in parity-wasm. It was even considered to just have a copy of the validation code if there is a great need.

Those sound like good reasons to me, thanks for explaining!


Update Pagination

Q
A

Bug fix?
yes

New feature?
no

BC breaks?
no

Deprecations?
no

Tickets
n/a

License
MIT

Doc PR
n/a

Update pagination to ensure there are really multiple pages.
If you add Search parameters to the api call you may get a response with a hydra:view even if there are not enough rows to create pagination

Thanks @electricBonfire


Add file selection UI to vpk creation

Have a button to add file/folder, when adding a folder, add files in them recursively
Then have a list of all the current files, and either a checkbox to unselect individual files, or remove them.
One thing that may be problematic is when adding individual files, it needs to have a correct path that users would expect (does that require having a field to edit the path?)
This can be further expanded to pack resources such as maps. When adding a _c file, parse external references out of it and add them too.

The UI stuff was implemented.
Just need to implement reading compiled files and adding its dependencies (code to detect _c files is already there). Ref: https://github.com/xen-000/CS2-MapPacker


HTTP 2 support

Would be nice if Express had http 2 support on the roadmap, now that all major browsers support it in their current versions. So I am tossing it in here.

Open this issue - https://gigaom.com/2015/02/09/google-ditches-its-own-protocol-for-http2-and-a-faster-web/

@alejonext I'm not sure I understand your comment--Express has supported HTTP2/SPDY for a long time.

Ha! ok @dougwilson

Upon reading http2 docs im under impression that you need to write code for server push both on client side and server... is this true? And isnt server push the greatest benefit, to be able send alresources required together and avoid separate requests.

This is absolutely true, though I think you can use a module like https://github.com/jshttp/spdy-push ? I'm not that familiar, but Express does not have any knowledge of the underlying protocol version (or even the socket), since at it's core, all Express is is a function you pass as the handler to .createServer.
You can always extend your Express like
var express = require('express')
var app = express()

express.response.push = /* define some method that will become res.push() in your app */

I agree with what @dougwilson says above and where this conversation ended up. http2/spdy should be implemented at a lower level then express and be pretty much transparent to express itself.
@techsin

Does this code still work?
var express = require('express')
var app = express()

app.get('/', function (req, res) {
  res.send('hello, http2!')
})

var options = {
  key: fs.readFileSync('./example/localhost.key'),
  cert: fs.readFileSync('./example/localhost.crt')
};

require('http2').createServer(options, app).listen(8080);

I tried the same thing with newest versions and got an error.
Which library's issue is this?
_stream_readable.js:505
    dest.end();
         ^
TypeError: undefined is not a function
    at Stream.onend (_stream_readable.js:505:10)
    at Stream.g (events.js:199:16)
    at Stream.emit (events.js:129:20)
    at _stream_readable.js:908:16
    at process._tickCallback (node.js:355:11)

Node: v0.12.0
node-http2: v3.2.0
express: v4.11.2

File an issue at https://github.com/molnarg/node-http2

For people still dropping by here, use https://github.com/indutny/node-spdy instead of https://github.com/molnarg/node-http2
The spdy module does in fact support full HTTP/2 despite it's name and works out-of-the-box with Express.js


193 winners

changed the schedule to winners along with the link to their hackerearth submission and a short description of each project

Can you merge in develop?

As in merge develop into 193_winners.

Took care of the merge

lgtm


Keyboard manager ignores keyboard setup (US international no deadkey) and switches to US only

Microsoft PowerToys version
0.43.0
Running as admin

[X] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
I would like to use the keyboard manager to re-map keys, (left ctrl - alt) ,
but I use a US international keyboard layout, since in need/want english keyboard but need letters like äåßöü just available (without switching language and having a different layout
Unfortunately, when activating the keyboard manager, may AltGr mappings, like AltGr + q = ä do not work anymore
✔ Expected Behavior
with my keyboard layout, pressing
AltGr + q = ä
AltGr + w = å
AltGr + s = ß
and all the other international keys just work, and my US keyboard is international with no dead keys
❌ Actual Behavior
when the keyboard manager is activated, all my AltGR + whatever do not work anymore, and I have just an US keyboard. no international :-(
Other Software
No response

Hi @a4z,
I believe the main reason that is causing this is that on the US International Keyboard mapping, "Alt Gr" is seen as "Ctrl(Left)+Alt(Right)" by Windows, for historical reasons, and so the "Ctrl(Left)" would be mapped to "Alt(Left)" after your remapping and Windows will see "Alt(left)+Alt(right)" after your remapping.
And the actual shortcut for the special letters is:
Ctrl + Alt + q = ä
Ctrl + Alt + w = å
Ctrl + Alt + s = ß
I don't think there's a way around this, since there's no actual "Alt Gr" key after Windows applies the US International Keyboard mapping.
One workaround I can suggest is trying to set shortcuts in order to use the same characters with another key, for example the Right Windows key which is usually next to Alt Gr.

Would this work for you?

Thanks a lot for the super fast reply!
I can verify that the reason for this behavior you explain is as you mention,
as soon as I remap the  left Ctrl key, AltGr + what ever stops working :-(
I use to work on Thinkpad keyboards, (even on my desk, the usb version) so I have no right Windows key,
and PrtSc key + something does not work :-(
But now I know the reason for this problem, I might figure out some other work around.
Thanks again for your explanation!

Thank you for the feedback.
Indeed, using Ctrl(Right)+Alt(Left) will also work for this.


PowerToys Workspaces / Discord app

Microsoft PowerToys version
0.84.0
Installation method
GitHub, PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Workspaces
Steps to reproduce
Install and launch Discord
Create a workspace with the open window of discord
Discord don't appaer in the workspace and cannot be launch

✔ Expected Behavior
I was hoping that the Discord window would be integrated into the application after capturing the workspace and would open even if the application is already running in the background
❌ Actual Behavior
Discord window does not integrate and cannot be launch
Other Software
No response

/dup #34558


Fixed minor spelling error

Pull Request Test Coverage Report for Build 341

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.06%) to 94.74%

Totals

Change from base Build 340:
0.06%

Covered Lines:
1495

Relevant Lines:
1578

💛  - Coveralls


About  Apiparams  a little suggest  of swagger

Sometime,When I define an interface parameter.I found that some fields of the entity class are not required.like this ↓
@RequestMapping(value = "/update", produces = {"application/json;charset=UTF-8"})    public void update(@RequestBody Book param) {}
But I only need a few fields in the entity,How do I hide the other fields.Do I have to define a new entity? That's a hassle.

I believe you could be using springfox. If this is the case please open ticket there and close this one.


Workround to fix trunk hot-reload issue on Windows

to address issue mentioned at https://discord.com/channels/1031524867910148188/1031524868883218474/1162459389949653103

Thank you!


Graph link are not create when using template

Hi Facette team,
When a graph is created from a template and added to a collection, the link to this graph is not added to the collection on the left part of the interface.
You could see my test on your demo, I have created a "collection test" with only one graph from a template and the link part is empty while the graph is shown.
I also have tested to add the graph "test" to the existing collection "host1.example.net" and the problem is the same.
Regards.
F.

Hi @fcauvet
This is a known problem (#204), fix is already shipped in the master branch – version 0.3.0rc2 should be out very soon (hint hint @vbatoufflet :smirk:).


structlog issue on el7 release

Describe the bug
Reconfigure fails likely because of a wrong version of structlog is installed on el7. See the stacktrace below.
To Reproduce
Just reconfigure the factory on el7
Screenshots
[mmascher@vocms0205 ~]$ sudo gwms-factory reconfig
Using default factory config file: /etc/gwms-factory/glideinWMS.xml
Reconfiguring the factory...
Traceback (most recent call last):
  File "/sbin/reconfig_glidein", line 24, in <module>
    from glideinwms.factory import glideFactoryConfig, glideFactoryLib, glideFactoryMonitorAggregator
  File "/usr/lib/python3.6/site-packages/glideinwms/factory/glideFactoryLib.py", line 33, in <module>
    from glideinwms.lib import (
  File "/usr/lib/python3.6/site-packages/glideinwms/lib/condorExe.py", line 10, in <module>
    from . import logSupport, subprocessSupport
  File "/usr/lib/python3.6/site-packages/glideinwms/lib/logSupport.py", line 418, in <module>
    structlog.processors.CallsiteParameterAdder(
AttributeError: module 'structlog.processors' has no attribute 'CallsiteParameterAdder'
Factory reconfiguration...                                 [FAILED]
[mmascher@vocms0205 ~]$ sudo vim /usr/lib/python3.6/site-packages/glideinwms/lib/logSupport.py
[mmascher@vocms0205 ~]$ sudo vim /usr/lib/python3.6/site-packages/glideinwms/lib/logSupport.py
[mmascher@vocms0205 ~]$ python3
Python 3.6.8 (default, Jun 20 2023, 11:53:23)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-44)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import structlog
>>> structlog.__version__
'17.2.0'
>>>

Info (please complete the following information):

GlideinWMS version: 3.10.3.rc2
OS version: el7
Priority: critical
Stakeholders: all
Components: factory

This also affects the frontend
Using default frontend config file: /etc/gwms-frontend/frontend.xml
Traceback (most recent call last):
  File "/sbin/reconfig_frontend", line 20, in <module>
    from glideinwms.creation.lib import (
  File "/usr/lib/python3.6/site-packages/glideinwms/creation/lib/cvWCreate.py", line 21, in <module>
    from glideinwms.lib import condorExe, condorSecurity
  File "/usr/lib/python3.6/site-packages/glideinwms/lib/condorExe.py", line 10, in <module>
    from . import logSupport, subprocessSupport
  File "/usr/lib/python3.6/site-packages/glideinwms/lib/logSupport.py", line 417, in <module>
    structlog.processors.CallsiteParameterAdder(
AttributeError: module 'structlog.processors' has no attribute 'CallsiteParameterAdder'
Upgrading the frontend[FAILED]


[BD-46] feat: add ability to render Breadcrumbs with custom elements

Description
Add ability render Breadcrumbs with custom element, e.g. using react-router's Link instead of simple a tag.
related issue: https://github.com/openedx/paragon/issues/1836
Deploy Preview
https://deploy-preview-1867--paragon-openedx.netlify.app/components/breadcrumb/#with-custom-link-element
Merge Checklist

[x] If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
[x] Does your change adhere to the documented style conventions?
[x] Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
[x] Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
[x] Were your changes tested in the example app?
[x] Is there adequate test coverage for your changes?
[x] Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

[ ] Verify your changes were released to NPM at the expected version.
[ ] If you'd like, share your contribution in #show-and-tell.
[ ] 🎉 🙌 Celebrate! Thanks for your contribution.

Thanks for the pull request, @viktorrusakov!
When this pull request is ready, tag your edX technical lead.

@viktorrusakov 🎉 Your pull request was merged!  Please take a moment  to answer a two question survey so we can improve your experience in the future.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 21.0.0-alpha.12 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[BUG] Link to https://www.textualize.io/rich/gallery in Readme.md is no longer valid.

[x] I've checked docs and closed issues for possible solutions.
[x] I can't find my issue in the FAQ.

Describe the bug
Link to https://www.textualize.io/rich/gallery in Readme.md is no longer valid.

Still broken

It is also broken https://www.textualize.io/gallery-instructions
Should https://www.textualize.io/rich/gallery  be replaced by https://www.textualize.io/projects/ ?

This is still a problem. Taking a look at the wayback machine link of the page it seems a similar purpose is now served by https://www.textualize.io/projects/. However they are two different libraries and it seems like the projects on that page are more curated than the large list from before. Honestly the end section of the "Projects Using Rich" section should probably just be removed.

With all due respect, but graph from https://github.com/Textualize/rich/graphs/contributors does tell a sad story, @willmcgugan no longer have time to maintain the library and apparently nobody else does it. That trivial bug report is just the icing on the cake, there are lots of other ones with real impact.
Last commit was 2 months ago, and for a project with >90 open PRs, the message is clear.


Contribute info

Add info on how contribution is intended, and a developers' guide. Both include in docs.

#138


Adding an ISODateTime and ID property to all data objects.

A few changes here, all related to the DataObjectMixin base class.

All data objects now have an ID property that is a UUID automatically generated on instantiation of the derived class.
All data objects now have an ISODateTime property that displays the date in standard ISO format. It is updated automatically every time the date is changed.
Fixed various existing bugs related to ID and dates.


udevadm trigger -y nitro_enclaves does not appear to apply udev rules properly when run from userdata during node provisioning

I am adding Nitro enclave nodes to my EKS cluster and overall it is working well, but I do have an issue that I have had to work around.
I am spinning up an EKS cluster using:

AMI: amazon-eks-node-1.18-v20210125

from: https://github.com/awslabs/amazon-eks-ami/releases

Instance Type: i3en.xlarge
Region: us-west2

I have one node that is Enclave Enabled and in the userdata for that node I have the following steps (among others):
amazon-linux-extras install aws-nitro-enclaves-cli -y
yum install aws-nitro-enclaves-cli-devel -y
usermod -aG ne ec2-user
usermod -aG docker ec2-user
systemctl start nitro-enclaves-allocator.service && systemctl enable nitro-enclaves-allocator.service
systemctl start docker  && systemctl enable docker

When the system is done provisioning, if I login as ec2-user, I am unable to run an enclave, unless I reboot the node first.
After some help from @andraprs, we determined that the issue appears to be related to:
https://github.com/aws/aws-nitro-enclaves-cli/blob/main/SPECS/aws-nitro-enclaves-cli.spec#L114
From what I can tell this line doesn't do anything in this scenario. Maybe because services are still starting?
The initial fix I discovered was to simply add chgrp ne /dev/nitro_enclaves to the bottom of my userdata script.
Running udevadm trigger -y nitro_enclaves from the RPM with an additional readiness and success check might help here.
@andraprs and I have more detailed logs, etc, but I think that this should cover the important details.

https://github.com/openebs/openebs/issues/2814#issuecomment-603709862
"The udev rules should be configured at the boot time before the worker nodes join the cluster. This can be achieved by adding a user-data script that is executed at the boot time."
May be an option to have several commands run during boot time e.g. user setup, package install.
https://docs.aws.amazon.com/eks/latest/userguide/launch-templates.html#launch-template-user-data
https://cloudinit.readthedocs.io/en/latest/topics/examples.html

Until a proper fix is found, the spec file could in theory also use the workaround I mentioned above  (chgrp ne /dev/nitro_enclaves) before trying to reload the udev rules, assuming that this is the expected result in all cases. That would at least ensure that the group is correct even if the udev trigger doesn't work before a full system reboot.


Stop data-diff when maximum time or # different records is exceeded

Is your feature request related to a problem? Please describe.
We run data-diff for many tables. Sometimes there are a lot of differences between the diffed tables. If so, the data diff for this tablepair might take a very long time (multiple hours). I prefer to skip this diff at a certain point, e.g., when a maximum diff time or # different records is exceeded. For such a diff, I do not care which records differ precisely, I am ok with knowing that this table is very off.
Describe the solution you'd like
Define a:

maximum diff time
OR, a maximum # different records
OR, a maximum % different records

If this threshold is exceeded, the diff is aborted, with a WARNING or ERROR message, and maybe an Exception.
Describe alternatives you've considered
I run data-diff programmatically and built this feature myself in the Python script that calls data-diff. This did not work as I hoped because data-diff uses a ThreadPool that continued with the diff after I broke out of the diff_tables iterable.
Additional context

I am still interested in this issue

Agreed this would be very useful.
We just added some automation here yesterday to help us wrangle the open issues list, thanks for bearing with us

☝

This is very useful as as it is data-diff would otherwise just stay stuck for many hours in our automation script if there are a big number of changes.

@JCZuurmond @dungkhuc ,
I'm sorry for the delay in following up on this. Thank you for taking the time to raise this issue!
We made a hard decision to sunset the data-diff package and won't provide further development or support.
If that's of interest, over the past few months, we have rewritten the diffing engine in Datafold Cloud and solved many issues that existed in this package. In particular, we implemented sampling, per-column diff limits, and real-time result updates to help with the problem you are describing.
-Gleb


feature: add duration in glossary

Description

This PR adds a glossary entry for Duration to clarify its usage in Kubernetes APIs.
The entry defines a duration as a time interval specified in Go's time.Duration format, highlights the requirement for non-negativity, and provides examples of valid expressions.
This addition aims to enhance user understanding of duration fields within the Kubernetes API.

Issue

Closes: #47909

/assign

@sftim Thank you for reviewing the PR I've added the changes according to the style guide and modified the spelling error.

/label tide/merge-method-squash


Starter fails to build due to misconfiguration of sitemap plugin

Trying to build the starter fails with this error:
error Plugin gatsby-plugin-sitemap returned an error

  Error: SiteMetaData 'siteUrl' property is required. Check out the documentation to see a working example: https://www.
  gatsbyjs.org/packages/gatsby-plugin-sitemap/#how-to-use

  - internals.js:43
    [blog-gatsby]/[gatsby-plugin-sitemap]/internals.js:43:13

It is probably better to delete the custom options for both the sitemap and feed plugins and rely on the defaults, which will require adding both siteUrl and description fields to the site metadata.

Please change url to siteurl at line #6 in gatsby-config.js
module.exports = {
siteMetadata: {
siteurl: 'https://lumen.netlify.com',
title: 'Blog by John Doe',


unable to build from source without existing cargo

I have built and installed Rust from source. I am building Cargo using instructions from README.md (case "Otherwise, if you have rustc installed and not Cargo")
git clone --recursive https://github.com/rust-lang/cargo
cd cargo
./configure

It fails with the following error message:
configure: error: needed, but unable to find any of: CFG_CARGO cargo

looks like a kind-of dup of https://github.com/rust-lang/cargo/issues/3437

Yes, it is a dup.


[next]  and  improvements

use observe helper and default binding
match docs some more

closing this cuz it'll mostly be overlap with #1163


Create the ability for additional UI customizations

Thanks for the pull request! I always wanted to have this added.
I replaced 100/0 and 0/100 options with better handling of hide options. Top section didn't expand to 100% when bottom section was chosen to be hidden.  It's all looking nice now.


[Feature Request] Arrow Key Page Navigation

Is your feature request related to a problem? Please describe.
It would be appreciated to be able to navigate between pages by the arrow keys on a keyboard, rather than having to click every time.
Describe the solution you'd like
Perhaps a bit of javascript listening for event.key/code "ArrowRight"
Describe a small use-case for this feature request
Not having to use the mouse, and the RSI associated, to navigate pages.
Additional context
Example implementation is the lanraragi reader (https://github.com/Difegue/LANraragi)

Great idea! We can also allow using the vim keybindings (J and K) to navigate between pages. I will work on this soon.

It's now added in v0.16.0. Thanks for the feature request!