HHL.solve() doesn't use specified quantum_instance.

Environment

Qiskit Terra version: 0.21.2
Python version: all
Operating system: Linux

What is happening?
HHL.solve() in qiskit/algorithms/linear_solvers/hhl.py constructs a quantum circuit qc, stores it in solution.state and calls HHL._calculate_norm(self, qc).
This function performs an evaluation of the quantum circuit "qc" using qiskit's built-in StateVector simulator and does not use the quantum_instance provided to the constructor of hhl:
https://github.com/Qiskit/qiskit-terra/blob/070b9e214718b413f3d2f1142ffdc07d1d750c91/qiskit/algorithms/linear_solvers/hhl.py#L238
Is this intentional?
The tutorial https://qiskit.org/textbook/ch-applications/hhl_tutorial.html shows the following example at the end of section 4A
    from qiskit import Aer

    backend = Aer.get_backend('aer_simulator')
    hhl = HHL(1e-3, quantum_instance=backend)

    accurate_solution = hhl.solve(matrix, vector)
    classical_solution = NumPyLinearSolver().solve(matrix, vector / np.linalg.norm(vector))

    print(accurate_solution.euclidean_norm)
    print(classical_solution.euclidean_norm)

suggesting that quantum_instance allows the use of qiskit-aer in HHL.solve().
Additional information:
hhl._calculate_norm(self, qc) boils down to an evaluation via
https://github.com/Qiskit/qiskit-terra/blob/main/qiskit/opflow/state_fns/circuit_state_fn.py#L258
in CircuitStateFn.to_matrix():
[3]   /usr/local/lib/python3.8/dist-packages/qiskit/algorithms/linear_solvers/hhl.py(525)solve()
-> solution.euclidean_norm = self._calculate_norm(solution.state)
[4]   /usr/local/lib/python3.8/dist-packages/qiskit/algorithms/linear_solvers/hhl.py(220)_calculate_norm()
-> norm_2 = (~StateFn(observable) @ StateFn(qc)).eval()
[5]   /usr/local/lib/python3.8/dist-packages/qiskit/opflow/list_ops/composed_op.py(141)eval()
-> return reduce(tree_recursive_eval, reversed(eval_list))
[6]   /usr/local/lib/python3.8/dist-packages/qiskit/opflow/list_ops/composed_op.py(131)tree_recursive_eval()
-> return l_arg.eval(r)
[7]   /usr/local/lib/python3.8/dist-packages/qiskit/opflow/state_fns/operator_state_fn.py(249)eval()
-> front = front.eval()
[8]   /usr/local/lib/python3.8/dist-packages/qiskit/opflow/state_fns/circuit_state_fn.py(308)eval()
-> vector_state_fn = self.to_matrix_op().eval()
[9]   /usr/local/lib/python3.8/dist-packages/qiskit/opflow/state_fns/state_fn.py(429)to_matrix_op()
-> return VectorStateFn(self.to_matrix(massive=massive), is_measurement=self.is_measurement)
[10] > /usr/local/lib/python3.8/dist-packages/qiskit/opflow/state_fns/circuit_state_fn.py(251)to_matrix()
-> def to_matrix(self, massive: bool = False) -> np.ndarray:

How can we reproduce the issue?
Script based on the tutorial https://qiskit.org/textbook/ch-applications/hhl_tutorial.html :
import numpy as np
from qiskit.algorithms.linear_solvers.hhl import HHL
from qiskit.providers.aer.backends import AerSimulator
from qiskit.algorithms.linear_solvers.matrices.tridiagonal_toeplitz import TridiagonalToeplitz

matrix = np.array([[1, -1/3], [-1/3, 1]])
vector = np.array([1, 0])

backend = AerSimulator()
hhl = HHL(1e-3, quantum_instance=backend)

accurate_solution = hhl.solve(matrix, vector)
print(accurate_solution.euclidean_norm)

What should happen?
HHL should use the backend (e.g., qiskit-aer simulator) passed to the HHL constructor for the circuit evaluation when HHL.solve() is called.
Any suggestions?
No response

@anedumla Any comment on this one.
@ahehn-nv I will note that as from the upcoming release of Terra, in early October, HHL will be deprecated and later removed from Terra https://github.com/Qiskit/qiskit-terra/blob/bcec9a314c68d9412a89904701d45491a68a12c0/qiskit/algorithms/linear_solvers/hhl.py#L112-L114 The tutorial will have to be amended accordingly too.

@woodsp-ibm it must be a leftover from an older version of HHL and the part with the quantum instance was not updated - my bad.
I think one of the reasons I did not notice is that we are not really using it on real hardware, which is the reason it is being deprecated. At the moment I am working on moving it to the research repo - is there something else that needs to be amended in the tutorial other than changing the imports from qiskit-terra to qisit-research?

I did not know that the algorithm would be moved over - as you see from the deprecation notice we are referring to the textbook. But that uses this algo as part of it so maybe that will be updated to use what you are putting on qiskit-research and the deprecation  message can stay referring to the textbook via which people will see it on qiskit-research?
Are you also updating the HHL to use primitives when you relocate it? QuantumInstance is planned for deprecation/removal - the other algorithms remaining here are all being refactored over to use primitives for this upcoming release (0.22). The original algorithm using QuantumInstance are planned to be deprecated in 0.23 (at present they are marked pending deprecation and will work alongside the primitive based ones for one release before being deprecated)

Actually turns out that there is some pendant discussion as to which repo it should be moved, so will have to wait for that decision before updating the textbook.
Regarding the primitives - I was not aware QuantumInstace was going to be deprecated too, thanks for the notice. Will update that too once we decide where to put it.

@anedumla As HHL is deprecated here and you have the code in another repository linked from the textbook  what to do with the issues here. I don't think there is any intent of altering the code here to potentially address any of these since its now deprecated. Do you want to take note of these and maybe create issues in the other repository. At some point these will be closed as not planned. So with this issue in the list too (first one), the ones relevant to HHL I can find are
#8735
#8332
#8102
#8069
#7926


Merging Develop into Master

Codecov Report

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

@@          Coverage Diff           @@
##             master    #6   +/-   ##
======================================
  Coverage          ?   98%
======================================
  Files             ?     8
  Lines             ?   200
  Branches          ?    43
======================================
  Hits              ?   196
  Misses            ?     0
  Partials          ?     4

Impacted Files
Coverage Δ

dork/gameitem.py
100% <100%> (ø)

dork/__init__.py
100% <100%> (ø)

dork/yamlreader.py
100% <100%> (ø)

dork/types.py
100% <100%> (ø)

dork/room_printing.py
100% <100%> (ø)

dork/cli.py
100% <100%> (ø)

dork/yamlloader.py
100% <100%> (ø)

dork/mapvalidation.py
92.45% <92.45%> (ø)

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 fbc6f2d...528bc68. Read the comment docs.


Hot reloading not working for files in demo folder

If I make changes to a file a in the demo folder, the Mesop app breaks and I need to restart the CLI again.
I think the issue is because the demo folder isn't really covered in execute_module.py since it's not in the mesop directory. Haven't looked at it too much more yet.
It seems like it's not able to load any pages: User Error: Accessed path: /embed/llm_playground not registered

Yeah, the way I got the demo imports working for the Bazel CLI was quite hacky: https://github.com/google/mesop/blob/d97cd21c97bd7172d4a0d7831aac7ae65b98eebe/demo/main.py#L12
Using the mesop pip CLI for the demo directory works w/ hot reload though.

Hmm, yeah that sounds like a reasonable workaround for now.


Resolve-NetBiosName - Return ComputerNamePhysicalNetBIOS if clustered

Type of Change

[x] Bug fix (non-breaking change, fixes #6053  )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[x] Breaking change (will change functionality of all commands using Resolve-NetBiosName against clustered instances)
[ ] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Unit test is included
[ ] Documentation
[ ] Build system

Purpose

Resolve-NetBiosName now does what the name says: It resolves the (physical) NetBiosName of the computer, so in clustered instances the ComputerName of the cluster node the instance currently runs on.
So no change for all non-clustered instances. Only risk: This would fail if the computer would only accept connections to the cluster name and not the node name. But I think the more offen case is like in the linked issue, that the computer only accepts connection to the node name.

This pull request can be closed if we work on #5988 and remove Resolve-NetBiosName and replace it by Resolve-DbaNetworkName in all commands, as the later returns information about the physical node instead of the virtual cluster name.

I dig it, thank you!


projector: add notebook renderer

This change introduces Python API to render the new projector in Juypter
notebook. It currently provides work in progress API, embedding which
visualizes the embedding data and label rendered in the visualization.
In order to support both Colab and Jupyter, I have decided to introduce
an abstraction, Renderer which knows how to render an output cell and
how to communicate with it. Do note that Jupyter and Colab are very
different; Jupyter does not encapsulate JavaScript context for each
output cell so a symbol exposed on globalThis bleeds over to other
output cells while Colab does not do that.

I think were just missing some type annotations. Looks like the static typing tests are failing for tensorflow_similarity/visualization/projector_v2/notebook.py:71

Thanks for taking a look at the CI. I am doing the work on the side and will get to it when ready. Also, will mark the PR as "Ready for review" when it is truly ready.

Sounds good, thanks!


Bugs: toc layout is still exist when I set toc: false in frontmatter

The text has been disappeared correctly, but the layout is still here.

use toc to control Table of Contents
use aside to control right sidebar (maybe in dev)


Xuhumsft securityinsights microsoft.security insights 2023 11 01

ARM (Control Plane) API Specification Update Pull Request

Tip: overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

 Click here to see the details of Step 1
Breaking changes review (Diagram Step 1)
If the automation determines you have breaking changes, i.e. Step 1 from the diagram applies to you,
you must follow the breaking changes process.
IMPORTANT This applies even if:

The tool fails while it shouldn't, e.g. due to runtime exception, or incorrect detection of breaking changes.
You believe there is no need for you to request breaking change approval, for any reason.
Such claims must be reviewed, and the process is the same.

 Click here to see the details of Step 2
ARM API changes review (Diagram Step 2)

If this PR is in purview of ARM review then automation will add the ARMReview label.
If you want to force ARM review, add the label yourself.
Proceed according to the diagram at the top of this comment.

 Click here to see the diagram footnotes
Diagram footnotes
[1] ARM review queue (for merge queues, see [2])
The PRs are processed by time opened, ascending. Your PR may show up on 2nd or later page.
If you addressed Step 1 from the diagram and your PR is not showing up in the queue, ensure the label ARMChangesRequested
is removed from your PR. This should cause the label WaitForARMFeedback to be added.
[2] public repo merge queue, private repo merge queue (for ARM review queue, [1])
If you need further help with anything, see Getting help section below.

Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[ ] New API version. (If API spec is not defined in TypeSpec, the PR should have been generated using OpenAPI Hub).
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[ ] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the Diagram Step 2, "ARM API changes Review", for this PR.

Additional information

 Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

This branch is created to merge as the protected branch has conflicts
https://github.com/Azure/azure-rest-api-specs/pull/27169
All the change is already reviewed and approved in the following PR.
https://github.com/Azure/azure-rest-api-specs/pull/26873

Breaking changes previously reviewed and approved in https://github.com/Azure/azure-rest-api-specs/pull/23121

Hi @xuhumsft, please fix duplicate schema: MetadataDependencies
for refer: https://github.com/Alancere/azure-rest-api-specs/commit/8fd20b29691e4c89d38b5782523e63cccff3bb79

Please request to merge again once the above comment is addressed

updated according to the comments.

Also check the failures in https://github.com/Azure/azure-rest-api-specs/pull/27173/checks?check_run_id=20012857275
Add follow up using this guide if needed link


BUG: plot_joint should pick channels

@jona-sassenhagen
Currently, plot joint takes all chans to compute the GFP and find the ERF/P peaks. IMO, it should only get the peaks from the the channels it plots:
import numpy as np
from mne import EvokedArray, create_info
from mne.viz.utils import _find_peaks

# setup info
n_time, n_chan = 10, 3
chan_names = ['ch_%02i' % chan for chan in range(n_chan)]
chan_types = ['mag', 'mag', 'misc']
info = create_info(chan_names, 1, chan_types)

# setup data
data = np.zeros((n_chan, n_time))
data[0, 0] = 10  # add ERP on first two channels
data[1, 0] = -10
data[2, 5] = 1e3  # add misc response somewhere else
evoked = EvokedArray(data, info, 0)

_find_peaks(evoked, 1)  # return 5, the peak of the misc response

Good catch. Also pinging @jaeilepp if I don't get here in time.

If you do a PR , could you externalize the spatial color function, so that
we can use it for home made plotting (with error bar etc) ? Else I can just
copy paste the code.
On Tuesday, 8 March 2016, jona-sassenhagen<EMAIL_ADDRESS>wrote:

Good catch. Also pinging @jaeilepp https://github.com/jaeilepp if I
don't get here in time.
—
Reply to this email directly or view it on GitHub
https://github.com/mne-tools/mne-python/issues/2996#issuecomment-193966389
.

externalize the spatial color function

Sorry, I don't understand what you mean.

Sorry, I don't understand what you mean.

@jona-sassenhagen the spatial colors code would be nice to have in many other context, maybe we can pull it out and expose it somehow as a util in viz. Clearer?

Ah okay. I'll see if I can get it out.

@kingjr is this really about plot_joint?
evoked.data[0, 10] = 1000
evoked.plot_joint(picks=range(1, 32));
evoked.plot_joint(picks=range(0, 32));

Compare.
(plot_joint takes all channels, but first it creates a surrogate object with all non-picked channels dropped.)
Or am I missing something?

From what I had seen, plot_joint actually calls plot, which itself deals with the _find_peak. So no, it's not really about plot_joint, it's lower.

Well but plot_joint will run _find_peaks on only the picked channels, and _find_peaks is a private function that doesn't have a picks arg. So I'm still not sure what specifically the problem is?
(To be precise, plot_joint calls _find_peaks by calling plot_topomap.)

Do you manage to reproduce the error ?

You mean that _find_peaks also considers misc channels?

I would expect the lines above to do as you say, but the idea is that
plot_joint doesn't end up doing an equivalent operation under the hood
(since it does subselect).
Do you get this error with plot_joint, plot, or what? Can you produce a
minimal example that uses the public plotting functions incorrectly? That
would help isolate the problem.

Sure
import numpy as np
from mne import Evoked, pick_types
from mne.datasets import sample
fname = sample.data_path() + '/MEG/sample/sample_audvis-ave.fif'
evoked = Evoked(fname, 'Left Auditory')
evoked.pick_types(eeg=True, meg=False, eog=False, stim=True)
# reref
evoked.data -= np.tile(evoked.data.mean(1), [len(evoked.times), 1]).T

# clean stim channels
stim = pick_types(evoked.info, stim=True)
evoked.data[stim, :] = 0
evoked.plot_joint(show=False)

# add crazy value on irrelevant channel
time = np.where(evoked.times >= -.100)[0][0]
evoked.data[stim, time] = 1e20
# in principle this crazy value shouldn't change the plot, but the _find_peak
# seems to use all channels
evoked.plot_joint()

That does seem to run counter to what Jona said, thanks. Right
@jona-sassenhagen?
BTW instead of de-meaning the data that way, read_evokeds has a baseline
argument you can use.

(And even without that, instead of np.tile you could use np.newaxis to
exploit broadcasting, like using bsxfun instead of repeat in MATLAB)

Thanks, I knew these tricks existed, but I never remember them on the fly.

Ah, I see. This is most likely because if picks is empty, _find_peaks gets forwarded the stim channel by evoked.plot_joint.
The same thing goes for evoked.plot_topomap btw (try it).
I'm not sure how to best approach this. Perhaps the easiest option is for _find_peaks to default to only data channels, or, alternatively, for evoked.plot_topomap to default to only data channels if picks is empty. Opinions?

IMO the peaks should be set from and only from the plotted channels, whether they are manually or automatically defined.

I have an idea ... I'll try.

Okay, I got it. I'll make a PR when #2975 is done, is that okay @kingjr ?
And I'd like to handle factoring out the spatial color code in another PR.

Ok, no rush on my side.

@kingjr do you still think the spatial_colors cmap should be externalized btw?

I liked that idea.
On Thu, Jul 7, 2016 at 11:07 PM jona-sassenhagen<EMAIL_ADDRESS>wrote:

@kingjr https://github.com/kingjr do you still think the spatial_colors
cmap should be externalized btw?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/2996#issuecomment-231222158,
or mute the thread
https://github.com/notifications/unsubscribe/AB0firm5NS0v2_Hwpwnfyhkz9CJpei_8ks5qTXi4gaJpZM4Hr6fi
.

no strong opinion.


[ISSUE #349]Nacos config module supports extended parameters

Please do not create a Pull Request without creating an issue first.
What is the purpose of the change
Nacos config module supports extended parameters
Config 模块现在可以支持以nacos.config开头的额外扩展参数
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.

 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.

LGTM


MoltenVK deprecation warning - add vkCreateMetalSurfaceEXT

When running Ash on Mac through MoltenVK we get this warning:
[mvk-info] vkCreateMacOSSurfaceMVK(): This function is obsolete. Consider using the vkCreateMetalSurfaceEXT() function from the VK_EXT_metal_surface extension instead

Talked to @MaikKlein and think vkCreateMetalSurfaceEXT is not exposed yet in Ash though so would have to be added to solve this properly

Updated the generator in #222. Now we just need to expose the extension.


Files not copied if folder doesnt exist on local

Hi,
When I try to download all folders/files, only those files are downloaded for which parent folder exists in local.
The script does create folder for bucket name automatically but not for any of the internal folder.

Hi Vivek,
I haven't used or maintained this script for 4 years. If you need a solution for downloading an entire S3 bucket locally I suggest the s3 cli: https://docs.aws.amazon.com/cli/latest/userguide/cli-services-s3-commands.html

Thanks for such quick reply @woganmay . Appreciate it.
Regarding cli, I can do that one as well.. Its just that using this, I dint have to install aws-cli and related dependencies.. So I found this easier. I did change the script to locally to make it work for internal folders as well. But I understand your point that it is not currently maintained and hence can close this issue.


Firebase authentication works in dev mode but not build

I was able to use this plugin as outlined in tauri-firebase-login to get Firebase Google redirect authentication working in dev mode, but not in build mode. My investigations found that, in build mode, the localhost redirect server was properly started and the auth token was correctly passed to the app, but when I called the Firebase SDK's signInWithCredential() function, it seemed to not do anything, and also didn't throw any errors that could be caught with .catch().
As was mentioned previously on this thread, this is likely related to authorized domains, and the fact that the domains used by the built Tauri app can't be authorized in the Firebase console.

Hi, thanks for reaching out. I am out of office at least until Sunday.
Until then, which OS are you using?
And do you think it's possible to create a minimal reproduction repo? I don't have much (or any) firebase experience so that would help a whole lot🙏

I'm using Ubuntu 22.04.3 LTS.
I didn't make any tweaks to the relevant functionality from the aforementioned repo so it will do for replication.
Specifically, when I run the app in build mode, logging in causes the app to update with my user information. When in build mode, logging in doesn't seem to update anything, though the login window does show up in my browser, and my investigations in my own project revealed that the token is getting to the app properly.

Okay, thank you :) I will look into it once I'm back home then.


Fix typo in Concepts.md doc

In the Concepts.md file, JWT has been written JTW. This MR fixes that word.

 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.

thanks!


Add management_project_id support to Group and Project Cluster

Update Project and Group Cluster to include management_project_id structure on update and get of cluster.
Closes https://github.com/xanzy/go-gitlab/issues/809

Ready for review.

Thanks @svanharmelen , I made the updates you suggested. This feature was added in MR 18429 , however there is a bug that prevents setting the management_cluster_id on AddCluster, it takes it but silently ignores it. If you UpdateCluster it works.


/bin/sh not bash

Shebang should be corrected to
#!/usr/bin/env bash
for portability as /bin/sh might not be bash.

Fixed in v0.1.8


new django application url not reflecting

My actions before raising this issue

Analyzed the opencv and dataset_repo app functionality before creating my own application

i have created a new django application under cvat.apps folder, added the app name in cvat.settings.base.py  and also updated url in cvat.urls.py file
But, unable to access the urls of my application. Tried adding the URL in existing dataset_repo and opencv applications but to no avail.
please help me if i have missed any change or let me know how to create new djanog app inside cvat folder

Hi @pavandayala
I do not think there is some specific for adding apps in CVAT.
Probably better to ask in django community with providing some examples.


fix: Fix duplicate symbol definition build error (ObjC < import)

The < import (module maps) fixes duplicate symbol errors on RN 0.67.

@mkuczera could we have this merged?

sorry, missed. Released in 1.13.1


Comments should support edit and delete

The editor comment widget should have a way for extensions to provide edit and delete actions on comments

@RMacfarlane You beat me to this! 😄 The VS Live Share team would also like to see this functionality, as part of our exploration to allow adding comments within a Live Share session.

@RMacfarlane  was just coming to open a issue on this 😛 Would be helpful, I just had to open Github to delete a comment

I think this would be a really useful addition to make the workflow from vscode even better


Project not always load js files from cli.json

I bought a HTML template and I am trying to adaptive on my Angular 2 project.
CSS files are OK, no problem, but JS files are really problem.
For example this is my angular.cli.json
  "styles": [
    "styles.scss",
    "theme/dist/css/pages/login-register-lock.css",
    "theme/dist/css/style.css",
    "theme/dist/css/pages/register3.css",
    "theme/libraries/register-steps/steps.css"
  ],
  "scripts": [
    "../node_modules/jquery/dist/jquery.js",
    "../node_modules/popper.js/dist/umd/popper.min.js",
    "../node_modules/bootstrap/dist/js/bootstrap.min.js",
    "theme/libraries/register-steps/jquery.easing.min.js",
    "theme/libraries/register-steps/register-init.js",
    "theme/index.js"
  ],

CSS files always work, JavaScript files under node_modules always work, but under theme JavaScript files only work when I start the project or ctrl + f5
I am using these JavaScript files in register component
    "theme/libraries/register-steps/jquery.easing.min.js",
    "theme/libraries/register-steps/register-init.js",

Start the app on register component its OK working but when I change route with button then come back again these are not working also there are no error on console.
please someone help me to solve this issue

Hello, we reviewed this issue and determined that it doesn't fall into the bug report or feature request category. This issue tracker is not suitable for support requests, please repost your issue on StackOverflow using tag angular-cli.
If you are wondering why we don't resolve support issues via the issue tracker, please check out this explanation.
I would like to say that on a Single Page Application, JS files in index.html are only loaded once. That is how single page applications work generally.


[FEA] Make blazingsql debug builds work, add BlazingContext.sql implementation

Fixes issues blocking blazingsql debug builds
Adds an initial single-node BlazingContext.sql implementation

I forced pushed a wrong branch or something... pushed this branch since it seems to have been what I wanted to do https://github.com/rapidsai/node-rapids/pull/271


Fix GetCurrentMonitor

GetCurrentMonitor now finds the monitor that has the largest area of intersection with the game window

@evgenydrainov I have my doubts on this improvement, it increases code complexity to take care of a very specific use case.
In what real-use-case situation a user would need that functionality? I mean, user usually open the window in one monitor or another. Even in the case the window was wrongly opened in the middle of two monitors, why does the application need to know that? I can think about scaling but the application can just chust the first one, it would be up to the user to move the window to another monitor...

I'm not merging this improvement for now, it seems to solve a very specific use case situation in exchange of a code complexity increment.


Talkback speaking device output text twice in VolumeScreen

Issue doesn't seem to be surfaced in the a11y screenshot tests:

https://github.com/google/horologist/blob/bccd49197af93ae1c863b2ef1888dc1b334aaaf7/media/audio-ui/src/test/snapshots/images/com.google.android.horologist.audio.ui_VolumeScreenA11yScreenshotTest_volumeScreenAtMinimums.png
https://github.com/google/horologist/blob/main/media/audio-ui/src/test/snapshots/images/com.google.android.horologist.audio.ui_VolumeScreenA11yScreenshotTest_volumeScreenNotConnected.png


Error slack.rtm.client client.py:_dispatch_event

@DandyDev  I got the error in 0.19.x version
[ERROR] slack.rtm.client client.py:_dispatch_event:445 | When calling '#handle_message()' in the 'machine.dispatch' module the following error was raised: 'DU6G9SQBC'
Would you please help me check this? It worked fine on 0.18.2 version

Hi same for me when i try to @respond_to in a private channel. If i try to execute same function not in a private channel everything went ok

I'm getting the same error :/

This is fixed in 1db9ec84f14830bfbe226294b75abff1bf7bb018

You can upgrade to v0.20 to get the fixes


style(route): indentation correction

Added a new line before (function(){ and indented the template blocks within the <% if %><% else %> statements.

Sorry I made  a minor layout mistake. Will fix this


Update clm_data artifact

Purpose
Update Artifacts.toml to use updated clm_data.
Change calls that used mechanism_map.nc to use vegetation_properties_map.nc because
mechanism_map.nc is deleted in the new clm_data artifact
Content

Update artifacts.toml
Update artifact usage in benchmark/land.jl, longruns/land.jl, and landa_regional.jl

[x] I have read and checked the items on the review checklist.

Maybe check also longruns and benchmark and crosslink the ClimaArtifacts PR?


Add config parameters to enable livolo integration

Add the possibility of setting ext_pan_id and experimental (livolo) to allow users of the edge (dev) version of zigbee2mqtt to use livolo switches.  https://github.com/Koenkk/zigbee2mqtt/issues/592

Experimental livolo has been removed in zigbee2mqtt 1.7, it works out of the box now. Only ext pan id is needed.

I saw there is already a pr for this, so closing this one #250


feat(folder-management): made folder related methods public

Issue
https://github.com/sasjs/cli/issues/73
Intent
Make methods related to folder management publicly available.
Implementation
Updated 'SASjs' class and fixed logic in 'SASViyaApiClient'.
Checks

[x] Code is formatted correctly (npm run lint:fix).
[x] All unit tests are passing (npm test).

[x] All sasjs-tests are passing (instructions available here).

The documentation on this one looks great


Added support for user colors

Added nine configurable colors for the user. I'm going to experiment with more colors so I had need of this myself.

Seems ok to me with minor comments. What's left to do here?

Nothing really. I wasn't sure that this was the most elegant solution. But it is working already and I can't really see any immediate downsides with it except for it being kind of la sledgehammer approach with a lot of copy paste.

You know what, I like this. Running with an extra colors seems to work fine. Feel free to merge this.

I don't think it's that important so in that case I will just remove it.


feat: tmp disable tiflow ci

Temporarily disable tiflow ci on ksyun. revert this after ksyun adds machine resources

/lgtm
/approve


Does this support multiple  elements inside the DOM?

Pretty much the title. If not, is it at all possible? I am trying to embed it multiple times in the same page.
This is what I am thinking:

The red "JSON DATA" is where I will make json-viewer appear.
But right now it doesn't seem to be triggering it. Here is my code:
function JsonParseORCatch(str) {
  try {
    return JSON.parse(str);
  } catch (e) {
    return false;
  }
}

var outerMostDiv = document.querySelector("#details > div > div.col-md-8 > div");

var script = document.createElement("script");
script.type = "text/javascript";
script.src = "https://pfau-software.de/json-viewer/dist/iife/index.js";
outerMostDiv.appendChild(script);

var allRows = outerMostDiv.querySelectorAll(".row");
allRows.forEach((row) => {
  var allDivs = row.querySelectorAll("div");
  var secondDiv = allDivs[1];
  const parsed = JsonParseORCatch(secondDiv.innerHTML);
  if (parsed) {
    secondDiv.innerHTML = "Parsing JSON";
    const jsonViewer = document.createElement("andypf-json-viewer");
    jsonViewer.id = "json";
    jsonViewer.expanded = 2;
    jsonViewer.indent = 2;
    jsonViewer.showDataTypes = true;
    jsonViewer.theme = "monokai";
    jsonViewer.showToolbar = true;
    jsonViewer.showSize = true;
    jsonViewer.showCopy = true;
    jsonViewer.expandIconType = "square";
    jsonViewer.data = parsed;
    secondDiv.append(jsonViewer);
  }
});

I get this instead:

Here is the DOM tree:

The issue is that when you load the script from the source https://pfau-software.de/json-viewer/dist/iife/index.js as shown in the example, it loads in parallel with the rest of the page. This means that all subsequent instructions are ignored until the script finishes loading.
You can fix this by either loading the script in a separate

solution provided

Thanks for the reply and taking the time to explain. I will experiment and update here. Cheers!


Function adds for #82

Get-NSCurrentTime: Gets the current time from the Netscaler
Get-NSlicenseExpiration: Reads license files found on the Netscaler and returns data around expiration
Update-NSAppliance: Updates a Netscaler and uses firmware from a remote location

Description
Returns the current NS time as a date object.
Get-NSCurrentTime -Session $session

Thursday, October 19, 2017 4:14:52 AM

Grabs license information from Netscaler
Get-NSlicenseExpiration -Session $session

LicenseFile                      DaysLeft Expires   Feature
-----------                      -------- -------   -------
FID_4a7dd8_15ddcd4166e__7084.lic      330 9/15/2018 CNS_V3000P_SSERVER
FID_4a7dd8_15ddcd4166e__7084.lic      330 9/15/2018 CNS_V3000_SERVER
FID_4a7dd8_15ddcd4166e__7084.lic      330 9/15/2018 CNS_SPE_SERVER

Updates NS firmware from HTTP(s)
Update-NSAppliance -Session $Session -url "https://mywebserver/build-11.1-47.14_nc.tgz"
Related Issue
#82
How Has This Been Tested?
tested with version NS 11.1 and 12
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Thanks for the new functions @ryancbutler!
I made a few small changes to fix the Pester tests as well as some minor code style changes. Hope you don't mind.

no problem at all


Removal of logging limit exceeded test

#83
Local testing results from this PR using services from edgexfoundry/edgex-go#617

Signed-off-by: Trevor Conn<EMAIL_ADDRESS>

@tsconn23 may we still keep this test and modify the content to test the scenario you mention in the issue?  The test result become 200 OK and confirm we can retrieve 100 records although the users request 101.  That't the test for the new change(edgexfoundry/edgex-go#617).  What do you think?
@weichou1229 could modify the test if you agree

@cloudxxx8 Sure, that sounds fine. If you're going to make the change, then you can close this PR.

@tsconn23 Thanks, we will do that
@weichou1229 Please help follow up


Quantum Oracle for Solving (Bounded) Knapsack Problem

Hello,
I would like to contribute by creating a Kata that involves a quantum oracle that can be used to solve the (Bounded) Knapsack Problem using Grover's Algorithm. I have already written most of the code for this oracle, and I have tested that it works. I will need some time to convert my code into a Kata format, though.
Could my proposed contribution be added to the roadmap? What are some first steps I need to do to transform my code into a Kata? Are there any specific guidelines on the formatting?
Please let me know if there are any questions. Thank you.

I think the SolveSATWithGrover Kata could provide you with a decent template/guideline on the layout of writing this Kata.
It is probably the closest Kata to what you are proposing since it also aims at solving a classical problem (3SAT) with Grover's Algorithm.
You will have to write tests, reference implementations and other scaffolding to Kata. I guess, they could be done in a similar vein as to SolveSATWithGrover Kata.

Hi @emax-wenjun!
Sounds exciting! (Well, I might be biased, since I like Grover's algorithm and its applications a lot, but still exciting :-)) I added it to the Roadmap and linked it to this issue.
As @jainvasu631 mentioned, the good examples of the katas to base this one upon are SolveSATWithGrover and GraphColoring.
The biggest challenge is usually splitting the big task into a sequence of smaller ones that build up to it - making the steps too large makes the kata very hard to approach. It might be a good idea to discuss the sequence of steps here before you spend a week or more implementing them. I haven't thought about how I would approach building this kata yet, but I should be able to follow your sequence and suggest possible additional steps or places in which the complexity increase is too steep.
Thank you! I'm looking forward to solving your kata! (I don't get nearly enough katas to enjoy solving :-))

Thank you, @jainvasu631 for the reference. I will probably use GraphColoring, since I'm more familiar with it, having solved it before.
@tcNickolas, for the splitting of the task into smaller parts, I currently have split the oracle into many operations and layers, but I will need to further decompose them to an extent that works for a kata. I have, however, calculated an estimation for the complexity, and from this estimate, the increase is not too steep, if the oracle is designed correctly.
As of current, my oracle's sequence of steps goes as follows (each step is an individual operation):

Less-than-or-equal-to comparator (To compare a Qubit[] representing an integer, with an Int. I don't believe there exists an operation for these particular datatypes in the API, but if there is, please let me know).
Greater-than comparator (could skip, since it's the same as the above comparator + inverter)
Bounds Checking module (uses comparators to evaluate if qubits satisfy bounds requirement in Knapsack Problem)
Addition operation (Increases a Qubit[] representing an integer by an Int value. Again, if there's an existing operation in the API that does this, please let me know).
Multiplication operation (repeated additions, to increase the value of a Qubit[] by the product of an Int and a different Qubit[] representing an integer )
An operation to calculate a summation, that will be used for the total profit and total weight in the Knapsack Problem.
Weight Checking: returns whether the total weight satisfies the constraint
Profit Checking: returns whether the total profit is greater than a threshold value
KnapsackValidationOracle: performs Step 3 (Bounds), Step 7 (Weight), and Step 8 (Profit), and performs a Toffoli gate to return whether all three conditions are satisfied.

Please let me know if anyone has comments, suggestions, or questions about my approach. In addition, I can also explain how specifically the Knapsack Validation Oracle will be used with Grover's Algorithm to solve the Knapsack Problem.

@emax-wenjun I have a suggestion for an alternative implementation of the oracle, which potentially makes it a bit simpler to create the Kata.
Imagine we have a qubit register for the oracle input (in the length of the number of available items) and a target register for the oracle. Then we have a register of size W, where W is the sum over all item weights. Think of this register being partitioned for all items, each in the length of the item's weight. Then we can apply X operations to a partition, controlled by the qubit from the input register for the corresponding item. Then the sum of all active qubits in W is the total weight and can be compared with the Less-than-or-equal comparator you are describing.
This avoid constant addition and constant multiplication.

@msoeken Thank you for your proposal, which would be of great value if it can avoid constant addition and multiplication. First, I would like to make sure we are on the same page regarding the problem. The version of the Knapsack Problem that I'm aiming to solve is the Bounded Knapsack Problem, in which there are n types of items, and each type can have multiple instances. Specifically, the ith type can have up to bi instances (e.g. Object type 5 can have up to 20 instances).
Thus, the oracle input would not be just n qubits, but rather consist of log2 bi qubits for each of n types of items.
In light of the item types having multiple instances, partitioning a register of size W, as you describe, would require biwi qubits for the ith object type (wi is the item's weight). Note that currently the QDK can handle up to 30 qubits at a time, including input, output, and ancilla qubits. It may be difficult to implement the described register within this range and be able to run reasonably large values of bi and wi at the same time. However, I will definitely keep in mind your idea of using a weight register to avoid constant addition and multiplication, and consider if it can be effectively implemented to achieve that goal.
Please let me know if I have misunderstood your suggested implementation, or if you have any other questions or comments.

7/31/2020 Updates on the Kata:

I discovered that the IncrementByInteger operation, which may have been added recently, performs the same task as the custom adder that I had previously described, except more efficiently. Therefore, the kata will no longer need a custom adder, instead using this operation.

I have essentially finished writing the ReferenceImplementation.qs for the kata, although I'm still trying to implement the quantum counting algorithm for calculating M for Grover Iterations. If anyone already knows how to implementation the quantum counting algorithm in Q#, that would be very helpful for this task. Otherwise, I will need to do some light research to write a Q# implementation.

There is a PR covering quantum counting at https://github.com/microsoft/QuantumKatas/pull/168, though I still haven't gotten to reviewing it :-(
Speaking of the difference in approaches described by you vs @msoeken, it seems that they are solving different problems, and the main difference is the number of instances of each item type that can be used. If this is so, would it make sense to cover the single-instance problem in the first part of the kata, as the easier one, and the multi-instance in the second part? This looks like it could be a nice build up in complexity

@tcNickolas That is a fantastic idea! I had been worrying that my kata would build up in complexity too quickly, and doing the single-instance problem first easily resolves this issue. Thank you very much for your suggestion. I will look into the quantum counting, as well.

8/9/2020 Updates:
I have finished writing all of the tasks, tests, and reference implementation for the kata, with the exception of quantum counting. Although I am able to implement quantum counting if necessary, I feel that its complexity requires its own kata to be covered efficiently. I would be willing to help create that kata, but I would like to finalize this kata first, since it is nearly finished. Thus, I will likely put a "placeholder" for quantum counting in this kata, such as the iteration method used in GraphColoring. If anyone else has a suggestion, or a different way to "place-hold" for quantum counting, please let me know.
Finally, once I am finished, how would I incorporate the BoundedKnapsack kata into the repository? This is my first time working on Github so I'm not as familiar with its mechanics yet.

@tcNickolas, @msoeken, @jainvasu631 I have finished writing the code for the Knapsack Problem kata, which can be found in the pull request I created. I would appreciate any feedback and any suggestions for improvement so that it can be incorporated into the repository.


When I am trying to SignUp. I am facing an error. With no error message at the top right.

Thanks for the update. I was able to do yarn install and yarn start.
When I am trying to SignUp. I am facing the below issue. We do not have backend code to test this issue?

And you can see there is no proper error message at the top right as well.
Originally posted by @mahesha1150 in https://github.com/rohitagr0310/Bookflow-app/issues/283#issuecomment-1755430695

As part of this repo, Main contributions are related to UI?

@mahesha1150
you can access the inner routes by adding /admin and /student to the base URL.
if you wish to contribute, can you help within the student panel or admin panel as they are still in development

Sure Rohit. I was trying to setup the application and trying to signup. If contributions are needed on UI with /admin or /student. Please let me know where I can contribute.

@mahesha1150
As the panels are still in development phase, the mobile responsiveness and overall ui is still lacking.
If you can improve the ui in any way while following the home page style, it would be very helpful

@mahesha1150 please create a new issue with the task you wish to work upon
thank you


Wrong docs link in vscode error message

Hey @Mnigos , thanks for filing the issue! In the meantime, you can check the docs for each rule here


[iOS] DatePicker DatePromptConfig.SelectedDate is not taken in consideration

Please check all of the platforms you are having the issue on
[x] iOS
[ ] Android
[ ] UWP
[ ] PCL

Please state the issue to the best of your ability below
On iOS the DatePromptConfig.SelectedDate parameter is ignored when showing the dialog.
The dialog shows the current date without taking in consideration the input parameter.

Please refrain from these defects for now since the code isn't actually enabled right now.  This code was removed from iOS for a reason.  Some of these are duplicates of my internal list.

released with 5.2


tikv-conf: update default pessimistic-txn config (#1101)

Signed-off-by: youjiali1995<EMAIL_ADDRESS>

Already cherry-pick and merged in #1115


4.0.0--beta.18

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Verify that the external gems change the version

External gems all have the same version (0.1.0). It should be checked that each version of the gem is higher than the previous one, so it can be compared.

? The gems you see with 0.1.0 are because we don't yet actually release cut gems for these dependencies, they are included via Github directly. There's nothing to verify, it's simply something we need to start doing in each gem (versioning and releasing).

After learning that bin/setup does not update gems, this would be a way to know that the gems you are using in development are out of date.

That doesn't change what I said about how we don't version our git based gems yet :)
I have a solution to your issue, however.

😄

@sergio-ocon closing since it looks like bin/update should do what you want... please let me know if this is still an issue


fix: fix gtm not working across pages

Codecov Report

Merging #292 (e38349f) into main (d641e9c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #292   +/-   ##
=======================================
  Coverage   81.98%   81.98%
=======================================
  Files          28       28
  Lines         372      372
  Branches       55       55
=======================================
  Hits          305      305
  Misses         61       61
  Partials        6        6

Impacted Files
Coverage Δ

src/pages/index.tsx
78.57% <ø> (-1.43%)
:arrow_down:

src/components/seo.tsx
100.00% <100.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 d641e9c...e38349f. Read the comment docs.


Kernel updates for 2023-05-25

Description of changes
https://cdn.kernel.org/pub/linux/kernel/v6.x/ChangeLog-6.3.4
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Building 6.3.4

6.3.4 is what I've tested, builds fine.

Hunk #1 succeeded at 117 (offset 3 lines).
applying patch /nix/store/57gcvvf12m4lnkqnbp7flnsdkdwhkakd-make-maple-state-reusable-after-mas_empty_area.patch
patching file lib/maple_tree.c
Reversed (or previously applied) patch detected!  Assume -R? [n]
Apply anyway? [n]
Skipping patch.
1 out of 1 hunk ignored -- saving rejects to file lib/maple_tree.c.rej
error: builder for '/nix/store/5hfl1wyjx49kq3flgnq5652i0wl0linj-linux-config-6.3.4.drv' failed with exit code 1

Fixed with https://github.com/NixOS/nixpkgs/pull/233927/commits/d64a44465772dfad7bce53cab03872609d4fb456, forgot to commit before pushing initially :(

Oh, right, well upstreamed patch. So good news that has been upstreamed. :3

commit a0306268276e2354f84ccd1296e9f229e86302ed
Author: Peng Zhang<EMAIL_ADDRESS>Date:   Fri May 5 22:58:29 2023 +0800

    maple_tree: make maple state reusable after mas_empty_area()

    commit 0257d9908d38c0b1669af4bb1bc4dbca1f273fe6 upstream.

    Make mas->min and mas->max point to a node range instead of a leaf entry
    range.  This allows mas to still be usable after mas_empty_area() returns.
    Users would get unexpected results from other operations on the maple
    state after calling the affected function.

    For example, x86 MAP_32BIT mmap() acts as if there is no suitable gap when
    there should be one.

    Link<EMAIL_ADDRESS>    Fixes: 54a611b60590 ("Maple Tree: add new data structure")
    Signed-off-by: Peng Zhang<EMAIL_ADDRESS>    Reported-by: "Edgecombe, Rick P"<EMAIL_ADDRESS>    Reported-by: Tad<EMAIL_ADDRESS>    Reported-by: Michael Keyes<EMAIL_ADDRESS>      Link<EMAIL_ADDRESS>      Link<EMAIL_ADDRESS>    Reviewed-by: Liam R. Howlett<EMAIL_ADDRESS>    Tested-by: Rick Edgecombe<EMAIL_ADDRESS>    Cc<EMAIL_ADDRESS>    Signed-off-by: Andrew Morton<EMAIL_ADDRESS>    Signed-off-by: Greg Kroah-Hartman<EMAIL_ADDRESS>    ```

    Rebuilding with new commit.


Make GetCommittedTip public

[!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

Current dependencies on/for this PR:

#1073  👈
#1072
feature/pos-networking-and-syncing

This stack of pull requests is managed by Graphite.

Merge activity

Feb 29, 9:32 AM EST: @lazynina started a stack merge that includes this pull request via Graphite.


[TBDGen] Track unavailable declarations against the active platform for api generation

This resolves the bug where api json files recorded declarations as unavailable purely because it was unavailable on unrelated platforms.
Resolves: rdar://113552185

@swift-ci please test

I don't think they need to be fixed in this PR, but I'll file some radars

rdar://127126725

Thanks Cyndy! I think this is the first time I'm reading this code in detail, and now that my attention has been drawn to it I see a number of issues because there's some complex logic implemented in Sema for availability checking that doesn't appear to be taken into account here. I don't think they need to be fixed in this PR, but I'll file some radars to track the following:

Availability declared in the enclosing scope, but not directly on the specific declaration, is not considered here.
For some platforms, availability is inherited implicitly from the attributes for other platforms when there is no explicit availability for the target platform (e.g. iOS availability is used for macCatalyst frameworks).
There may be some nuances with application extension availability attributes that aren't handled here.

Sounds good! Thanks for flagging this. My preferred approach here is to be able to query given the active target triple, if this declaration is unavailable and the serialization code does not handle any platform-specific details like implied availability. Maybe it's useful to distinguish whether it's deployment version-specific. Not sure if sdkdb generation happens strictly after Sema but it likely should so that concrete availability has already been figured out.

@swift-ci please smoke test


"Code is Unreachable" when using numpy.cross() in a function

Language Server version: 2022.8.51
OS and version: Windows 11
Python version: 3.10.6
numpy version: 1.23.2

I have found a bug, when i use the  numpy cross() function inside a function, then all the code that follow get the message * code is unreachable * and the code color fade.This only happens in a function. An example which give me this error is this:
import numpy as np

def fun(a, b):
    c = np.cross(a, b)
    return c

then the line return c is not rachable

This is a bug in numpy. See https://github.com/microsoft/pylance-release/issues/3277#issuecomment-1237782014.

duplicate of #3277


"CCNG Catalog Manager:(v2) Error decoding reply from gateway" when having a lot of service instances

When trying to start a service gateway which is connected to a Cloud Foundry with a lot of service instances we end up in a restart loop.
This happens because during the multiple page fetch sometimes an http call fails and as a result the whole gateway gets restarted.

Hello Ruben,
As vcap-services-base is deprecated, we're not adding features to it. Is it an option for you to update your brokers to the v2 broker API?

@shalako I know this repository has been deprecated. But this repository is still a dependency of cf-services-contrib-release. Which is still being used by the community. Would it be an option to move vcap-services-base into the cloudfoundry-community organisation as well?

Hi Ruben, nothing would please me more than to move vcap-services-base into cloudfoundry-community. I'll have to check if we need to go through the foundation to get that approved.


Unhelpful error message on bad image file

I ran memery on a large directory of images, and after some time it failed:
rob@tortuga ~/Dropbox/Camera Uploads$ memery . 'music' --n 5
Loaded 0 encodings
Encoding 25632 new images
...
31/201 [0 15%|██████████▎                                                        | 31/201 [06:00<32:56, 11.63s/it]
Traceback (most recent call last):
  File "/home/rob/.local/bin/memery", line 8, in <module>
    sys.exit(__main__())
  File "/home/rob/.local/lib/python3.9/site-packages/memery/cli.py", line 23, in __main__
    app()
  File "/usr/lib/python3.9/site-packages/typer/main.py", line 214, in __call__
    return get_command(self)(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/typer/main.py", line 497, in wrapper
    return callback(**use_params)  # type: ignore
  File "/home/rob/.local/lib/python3.9/site-packages/memery/cli.py", line 16, in search_folder
    ranked = queryFlow(path, query=query)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/core.py", line 54, in queryFlow
    dbpath, treepath = indexFlow(root)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/core.py", line 30, in indexFlow
    new_embeddings = image_encoder(crafted_files, device)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/encoder.py", line 17, in image_encoder
    for images, labels in tqdm(img_loader):
  File "/home/rob/.local/lib/python3.9/site-packages/tqdm/std.py", line 1133, in __iter__
    for obj in iterable:
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 435, in __next__
    data = self._next_data()
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1065, in _next_data
    return self._process_data(data)
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1111, in _process_data
    data.reraise()
  File "/home/rob/.local/lib/python3.9/site-packages/torch/_utils.py", line 428, in reraise
    raise self.exc_type(msg)
OSError: Caught OSError in DataLoader worker process 3.
Original Traceback (most recent call last):
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/_utils/worker.py", line 198, in _worker_loop
    data = fetcher.fetch(index)
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/rob/.local/lib/python3.9/site-packages/memery/crafter.py", line 43, in __getitem__
    sample = self.loader(path)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/crafter.py", line 27, in pil_loader
    return img.convert('RGB')
  File "/usr/lib/python3.9/site-packages/PIL/Image.py", line 904, in convert
    self.load()
  File "/usr/lib/python3.9/site-packages/PIL/ImageFile.py", line 249, in load
    raise OSError(
OSError: image file is truncated (11 bytes not processed)

This stack trace is very extensive, but doesn't tell me the one piece of information I actually want, which is the name of the file which has broken memery.
As a test I modified pil_loader in crafter.py to look like this:
def pil_loader(path: str) -> Image.Image:
    # open path as file to avoid ResourceWarning (https://github.com/python-pillow/Pillow/issues/835)
    with open(path, 'rb') as f:
        img = Image.open(f)
        try:
            return img.convert('RGB')
        except OSError as e:
            print("Failed to convert file '%s'" % path)
            raise e

and that worked to tell me the problematic file, which I could delete. But the crafter.py file says not to to edit it, so this isn't a usable patch.
Also it should probably skip over the bad file and keep going rather than crashing out, especially since this process takes a long time, and isn't able to pick up where it left off if you have to restart it.

Relatedly, if a file has an image file extension but is actually empty, you get this traceback:
Traceback (most recent call last):
  File "/home/rob/.local/bin/memery", line 8, in <module>
    sys.exit(__main__())
  File "/home/rob/.local/lib/python3.9/site-packages/memery/cli.py", line 23, in __main__
    app()
  File "/usr/lib/python3.9/site-packages/typer/main.py", line 214, in __call__
    return get_command(self)(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.9/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.9/site-packages/typer/main.py", line 497, in wrapper
    return callback(**use_params)  # type: ignore
  File "/home/rob/.local/lib/python3.9/site-packages/memery/cli.py", line 16, in search_folder
    ranked = queryFlow(path, query=query)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/core.py", line 54, in queryFlow
    dbpath, treepath = indexFlow(root)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/core.py", line 30, in indexFlow
    new_embeddings = image_encoder(crafted_files, device)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/encoder.py", line 17, in image_encoder
    for images, labels in tqdm(img_loader):
  File "/home/rob/.local/lib/python3.9/site-packages/tqdm/std.py", line 1133, in __iter__
    for obj in iterable:
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 435, in __next__
    data = self._next_data()
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1065, in _next_data
    return self._process_data(data)
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/dataloader.py", line 1111, in _process_data
    data.reraise()
  File "/home/rob/.local/lib/python3.9/site-packages/torch/_utils.py", line 428, in reraise
    raise self.exc_type(msg)
PIL.UnidentifiedImageError: Caught UnidentifiedImageError in DataLoader worker process 2.
Original Traceback (most recent call last):
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/_utils/worker.py", line 198, in _worker_loop
    data = fetcher.fetch(index)
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/_utils/fetch.py", line 44, in fetch
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/rob/.local/lib/python3.9/site-packages/torch/utils/data/_utils/fetch.py", line 44, in <listcomp>
    data = [self.dataset[idx] for idx in possibly_batched_index]
  File "/home/rob/.local/lib/python3.9/site-packages/memery/crafter.py", line 48, in __getitem__
    sample = self.loader(path)
  File "/home/rob/.local/lib/python3.9/site-packages/memery/crafter.py", line 26, in pil_loader
    img = Image.open(f)
  File "/usr/lib/python3.9/site-packages/PIL/Image.py", line 2967, in open
    raise UnidentifiedImageError(
PIL.UnidentifiedImageError: cannot identify image file <_io.BufferedReader name='dual/8c6598d6ef98f5eaeba38dbaf5d381b0.jpg'>

Which is good in that it gives the file path, but bad in that it also crashes out the whole process instead of just skipping the empty file

thank you! You're right, it needs better error handling especially in the indexFlow which can take a long time to run. did you get it to work on your data set after removing the offending files?
As for not editing the Python files directly, that's because this is developed with nbdev. So that patch will work fine, just needs to be added in the Jupyter notebook that is the source of truth for the code, rather than in the finished .py files. I guess I should put some stuff in the read me about how to help develop on the project...

I haven't been able to figure out a clean way to implement skipping over broken files, so it's a slow process of running it until it fails, deleting the problem file, and trying again.
The latest issue is actually the Decompression Bomb error from issue #10 , with an image which is very large but isn't actually corrupted or invalid so I don't want to just delete it

Yes, these are all connected due to Pillow preferring to error out rather than silently drop images. I think it shouldn't matter to the index if we drop images, since this happens before the index is written. So it's just a matter of ignoring PIL errors and passing on to the next image.
Maybe there's a way to check the image file for corruption in the loader? I'm work on this today some

Okay @robertskmiles I think i have fixed this in 0.0.7.
There's not an elegant way to verify an image inside of a list comprehension, it seems. So I just made a try/except loop like yours above -- except I put it in the loader instead of the crafter, so that corrupted files never even make it in the door. See 5ed4f4856c46786a68ad4fa9fd07a4f2954bae2c near the bottom
Let me know if this runs against your datasets and if it does we can close the issue! thanks

Well no that made it slow also. The compromise was to put it in the archive_loader as a boolean check on the new_files coming in. It's not as fast as it once was for encoding, but much faster than checking all the images before crafting them> And the pil_loader can't be allowed to return None because it's used in the DataLoader which wants batches of already cleaned data.
Anyway, I'll close this once I've tested on a couple other machines btu it think it's as good as can get right now.

Regarding list comprehensions, might it be possible to use a pattern like: [x for x in list_of_xs if x]
So then list_of_xs can have None values and they'll just be skipped over?
Or [x for x in list_of_xs if is_valid_image(x)] ?

Yes :D that's essentially what i did, with this line in loader:
    new_files = [(str(path), slug) for path, slug in filepaths if slug not in archive_slugs and verify_image(path)]

It is kind of annoying that it prints the errors every time now. And if there's an errored file it re-builds the treemap because it thinks it needs to re-index some new images, even though 0 images will actually be passed. Hmm...

this has been ~solved so i'm closing, we can open new issues for similar issues


[HierarchicalIterative] Fix bug in right hand side setter.

This pull request fixes issue #71. Unfortunately, it does not improve the poor performance of benchmark romeo-placard.

Thanks. I will have a look at it ASAP.


Auto Clone Bundles in Companion Objects

Allows autoclonetype to work on Bundle classes nested in companion objects, by detecting class symbols that are static and not trying to assign an outer instance in those cases, because one isn't necessary.

Related issue: #785

Type of change: other enhancement

Impact: API addition (no impact on existing code)

Development Phase:  implementation
Release Notes

retest this please

@ucbjrl tests appear to be failing for reasons unrelated to this PR?

Ugh, I merged the annotations refactor to Firrtl so the tests are probably pulling from Firrtl master and the slight API change will cause it to fail.

I could merge https://github.com/freechipsproject/chisel3/pull/767 then we can merge master and tests should pass?

I was kind of hoping to merge this first (to make rocket-chip bumping easier) but I forgot that merging the firrtl PR would make this fail 🤷♀

lol, and #767 also just failed tests...

And I can't seem to log in to a17 to look at jenkins remotely...

🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥

#767 https://github.com/freechipsproject/chisel3/pull/767 is failing sbt
test with:
...
[info] Compiling 84 Scala sources to
/vm/home/jenkins/sharedspace/chisel3_3/target/scala-2.11/test-classes
...
[error] /vm/home/jenkins/sharedspace/chisel3_3/src/test/scala/chiselTests/AnnotatingDiamondSpec.scala:10:8:
object RunTransformAnnotation is not a member of package
firrtl.annotations
[error] import firrtl.annotations.{
[error]        ^
[error] one error found
[error] (Test / compileIncremental) Compilation failed
[error] Total time: 36 s, completed Feb 27, 2018 7:06:45 PM
Makefile:39: recipe for target 'test' failed
...
On Tue, Feb 27, 2018 at 7:10 PM, Richard Lin<EMAIL_ADDRESS>wrote:

🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freechipsproject/chisel3/pull/788#issuecomment-369106840,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGh1wHdW_zpq6pvkKUUFOrbnXlUDdc7hks5tZMOVgaJpZM4SV7WD
.

I forgot to push a commit that fixed that test 🙈


[Test] Un-XFAIL ParseableInterface/verify_all_overlays.py.

The test is passing now--un-XFAIL it.
rdar://problem/50648519

@swift-ci please test and merge


Document and/or simplify how to create compressed tarball

How do I create a .tar.gz?   Is this the way?
https://github.com/JuliaIO/Tar.jl/blob/095fb5afdc2f0f41220fc1d8fc50bad24a649026/test/perf/perf.jl#L62-L64
It'd be nice to document this or possibly even simplify it a bit.  I really just want to Tar.create("dir", "dir.tar.gz") and have it do what I want, but I get the dependency issues that incurs.

That's one way to do it. The simplest if you have a gzip program is Tar.create("dir", pipeline(`gzip -9`, "dir.tar.gz")).


Support ibazel coverage

Hey team, we're looking at integrating coverage into our repository with bazel and we're after continuous updates to our coverage.
Currently we need to develop our tests with ibazel run :target then separately we run bazel coverage :target, however ideally we'd just run ibazel coverage :target to get fast updates to our code coverage

Oh how I wish there was a team behind this 🤣. Sorry, it's just me.
Alright, here is how I would go about doing this:

Add a new Test method into ibazel.go called Coverage

https://github.com/bazelbuild/bazel-watcher/blob/35ab9a9994dea4bc2822039c3617da28317094f0/ibazel/ibazel.go#L264-L267

Add a new case statement to handle coverage being passed in on the command line:

https://github.com/bazelbuild/bazel-watcher/blob/35ab9a9994dea4bc2822039c3617da28317094f0/ibazel/main.go#L198-L199

Add to the e2e helper a new method Coverage

https://github.com/bazelbuild/bazel-watcher/blob/35ab9a9994dea4bc2822039c3617da28317094f0/e2e/ibazel.go#L58-L61

And the associated internal coverage function based on

https://github.com/bazelbuild/bazel-watcher/blob/35ab9a9994dea4bc2822039c3617da28317094f0/e2e/ibazel.go#L266-L283

Add a new test case called TestSimpleCoverage shaped like this:

https://github.com/bazelbuild/bazel-watcher/blob/35ab9a9994dea4bc2822039c3617da28317094f0/e2e/simple/simple_test.go#L65-L71
Please note that you'll probably need to add an sh_test to the test fixture. The fixtures are formatted as txtar.
https://github.com/bazelbuild/bazel-watcher/blob/35ab9a9994dea4bc2822039c3617da28317094f0/e2e/simple/simple_test.go#L15-L57
It looks like I don't have any test coverage of ibazel test in there. If you were feeling like a total boss it would be amazing if you did the e2e stuff also for a new test.


Rule arguments

It is not very clear from the doc (neither readme, nor wiki) what the arguments for Rule function are.
E.g.
Rule("$$","$$","tex") -- not very clear what `$$` are. 3rd argument is file extension?

-- disable for .vim files, but it work for another filetypes
Rule("a","a","-vim") -- what is "a", "a"? What rules does it disable for vim files

Would appreciate some help.

1st and second argument is a pairs and 3rd is a filetype.  You can use string or a table(array).
If you add minus "-" before filetype mean that rule is not working on that filetype.
maybe it is not clear but when you understand it is make easy to add new rule .


跑services/app.py报错

跑services/app.py报错，具体报错如下：
from datasets import load_dataset

ImportError: cannot import name 'load_dataset' from 'datasets' (/home/xt/txiang/LMFlow/src/lmflow/datasets/init.py)
检查了一下datasets的函数dataset.py里面确实没有load_dataset这个模块

Hi,
这里的datasets是huggingface提供的一个第三方库datasets. 您可以通过pip install datasets来进行安装。
谢谢

This issue has been marked as stale because it has not had recent activity. If you think this still needs to be addressed please feel free to reopen this issue. Thanks


Fix issues caused by line breaks.

Add file '.gitattributes', fix problems caused by newline conversion in different operating systems.

Close this pr. Commit to the 4.0.0-beta3 branch.


[Debugger] Showing values of complex types

Currently, the "<Complex_Value>" is returned for complex types as the value. Need to show actual values in arrays, maps, etc.

added with pull https://github.com/ballerinalang/ballerina/pull/2883


GHOST and NEXT coin not showing in add currency

check your configs

check your configs

can you point me out where sir?
i mean exact configs lcoation
thank you

hm, just noticed this not in the readme. now added https://github.com/swaponline/MultiCurrencyWallet/blob/master/docs/README.md#enabledisbale-blockchains-on-domain

Thank you now GHOST and NEXT coin is showed now.   :)


GUI:  Filter-own-keys block needs a checkbox:  when unchecked, no filter applies

Blocks #90.

Fixed!


Clarify point serialization

The ciphersuite's octets_to_point is ambiguous, as it is unclear which point type should be returned (from G1 or G2). This can be inferred by cross-referencing other parts of the spec, but implementers would benefit from a clearer description; e.g., either adding a _g1 of _g2 suffix to the method or a type input. The BLS spec support two modes (where G1 and G2 are swapped), so the ambiguity is by design; BBS fixed the group, so we don't need to keep the BLS notation. point_to_octects is not as problematic, since presumably you can infer the input type, but we should harmonize both function names.
Another clarification question: both point_to_octets and octets_to_point refer to the CRFG pairing-friendly curves spec's Appendix C, but doesn't mention which mode to use (compressed or not).  As an example, the BLS spec uses compressed serialization (see its Appendix A)

I may be missing something but isn’t it possible to determine if the point should be in G1 or G2 by the encoded to octets value of the point itself, based on the length of that value (called s_string in the pairing-friendly-curves spec). The same holds for compression vs non-compression case that can be determined by the C_bit defined here.
If I’m not missing something and the proposal is to add the _g1 and _g2 suffixes to increase readability but have the operations essentially be the same in both cases I’m all for it.

Yes, I think adding _g1 and _g2 would be the simplest solution. Maybe there are ways to figure out what needs to be done, but for devs calling a 3rd party BLS library (like me), it's not immediately clear which function to call (having to consciously recall which group if for pub keys and generators). Moreover, future ciphersuites might have different rules, so an explicit target group removes potential ambiguity.


Update installation docs

Today in the exercism/support gitter there was an error message reported about Test/More.pm not found.
He used a minimal and plain centos:7 inside a Docker-container. He did not install anything further, because documentation of the track states that everything should be fine on linux.
Further research on this topic discovered, that in fact, perl itself is preinstalled on CentOS, because its needed internally. Perhaps even because it were needed if one wants to rebuild the Kernel?
Anyway, Test::More is not bundled with perl in Fedora and CentOS, one needs to yum install perl-Test-Simple there. On other RPM based distributions Test::More seems to be included in perl, see rpmfind: https://www.rpmfind.net/linux/rpm2html/search.php?query=perl(Test%3A%3AMore)
Nevertheless, as we have found at least 1 major distributions where the current docs of "no need to install anything" do not hold, I think they should be re-evaluated and the majors should be written out explicitely, ala:

CentOS and Fedora: install perl-Test-Simple additionally
Ubuntu and Debian: no need to install anything further (This is an unverified assumption to make explicitness of the table clear)

Addition: Even if perl is probably available and a core dependency of most linux distributions, I think we should make sure to list the actual packages anyway, and if it is just to mark them as manually installed instead of beeing installed as a dependency (which by its nature may vanish at any time when other packages are uninstalled).

I may edit the docs to suggest using plenv so users can have a version of Perl installed and set locally to their exercism directory.

I do not like to require any kind of version manager unless technically
necessary. Packages are available and should be therefore preferred. Adding
another layer of package management often brings more trouble than
necessary. I think though they can be suggested in a kind of further read
or if you know what you are doing.
Daniel Mita<EMAIL_ADDRESS>schrieb am Do., 15. Juni 2017, 10:53:

I may edit the docs to suggest using plenv
https://github.com/tokuhirom/plenv so users can have a version of Perl
installed and set locally to their exercism directory.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/exercism/xperl5/issues/147#issuecomment-308670852,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADmR7IeiwNzoDwBuXVCJbHc7X_cc6UNks5sEPEfgaJpZM4N4QPB
.

Ah I see now. I don't know much about Fedora-based distros, but it turns out that they intentionally don't include all of the core modules.
https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/7/html/Developer_Guide/lib.details.html#libraries.perl
Perhaps add an instruction for Fedora/Red Hat/CentOS that they should also install perl-core would be the best solution.

Updates in https://github.com/exercism/xperl5/pull/152

Closing this as https://github.com/exercism/perl5/pull/152 has now been merged.


Ability to override default authentication scheme for HttpContext.SignInAsync(...)

Currently it just gets the value from DefaultAuthenticationScheme
https://github.com/IdentityServer/IdentityServer4/blob/413daee4b7d0b07728ab77b08c3945e44ea97c96/src/IdentityServer4/Extensions/HttpContextAuthenticationExtensions.cs#L264-L267
https://github.com/IdentityServer/IdentityServer4/blob/413daee4b7d0b07728ab77b08c3945e44ea97c96/src/IdentityServer4/Extensions/HttpContextAuthenticationExtensions.cs#L286-L295
I'd would guess there's good reason why it's like this?
Not really sure if my use case is appropriate for such a change, but basically I was modifying an existing project to have idsrv built in, and someone had set
services.AddAuthentication(options =>
{
    options.DefaultAuthenticateScheme = "Bearer";
    options.DefaultChallengeScheme = "Bearer";
}).AddJwtBearer("Bearer",  opts => { /* ... */});

Which caused an exception when calling HttpContext.SignInAsync(...) because JwtBearer's handler can't be used for sign in...
One idea I had for this is to create IdSrvCookieAuthentication from CookieAuthentication something like and create associated handler. Then for idsrv sign-in we can get the scheme where handler type is IdSrvCookieHandler and use it.
Then for overwriting the cookie handling instead of
services.AddAuthentication("MyCookie")
    .AddCookie("MyCookie", options =>
    {
        options.ExpireTimeSpan = ...;
    });

can do
services.Configure<IdSrvCookieAuthenticationOptions>(options => {
        // configure....
    });

Not even sure this is necessary, but I can sketch something together if you like... 🤷
Or in fact, why not have methods on idsrv builder to configure cookie auth settings, then you could always use idsrvcookieauth scheme.

You can always call SignIn and pass the explicit scheme. But our design is that the DefaultSignInScheme and DefaultAuthenticationScheme are useed to know what cookie scheme you're using for your IdentityServer users.

Which, of course, would be a valid observation!
Is using the default auth scheme not assuming too much? I can create a PR with my idea if you like.
Are you able to elaborate more on https://github.com/IdentityServer/IdentityServer4/issues/2489 because it sounds like my proposal may help?


Mixed content breaks web accounts for login

Log in
Go to your profile
Go to Login Options
Click Add Another Account

You get:

and an error in the JS console:

Mixed Content: The page at 'https://ep2015.europython.eu/accounts/profile/' was loaded over HTTPS, but requested an insecure resource 'http://ep2015.europython.eu/en/openid/embed?token_url=https%3A%2F%2Fep2015.europython.eu%2Faccounts%2Fjanrain%2Ftoken%2F&language_preference=en'. This request has been blocked; the content must be served over HTTPS.

My browser is Chromium 41.

Thanks for reporting this.
The OpenID functionality appears to be broken - even with https:// the URL does not exist and we also don't have Janrain account configured for the website.

This is no longer relevant – we removed janrain completely and set up updated integration using only Google.


Layer detection should not require DFP to fullfil mounted device

The Problem To Be Solved
In CMSIS-Toolbox 2.6.1 it was not required to specify the DFP as the mounted device was not checked during layer detection.
cbuild v2.6.1-25-g3f083c7 requires now the DFP, otherwise it reports:
error csolution: board mounted device STM32F756ZGTx not found
error csolution: no compatible software layer found. Review required connections of the project

To duplicate
Use https://github.com/Arm-Examples/AWS_MQTT_Demo and modify Demo.csolution.yml as shown here:
  packs:
    - pack<EMAIL_ADDRESS>    - pack<EMAIL_ADDRESS># Add BSP
#    - pack<EMAIL_ADDRESS>    - pack<EMAIL_ADDRESS># Add CMSIS-Driver for WiFi Shields

  target-types:

    - type: MyBoard
      board: NUCLEO-F756ZG

Related https://github.com/Open-CMSIS-Pack/devtools/issues/1889

I thought it was clear since I asked for confirmation. This is a side effect of PR https://github.com/Open-CMSIS-Pack/devtools/pull/1875 addressing issues https://github.com/Open-CMSIS-Pack/devtools/issues/1841 and https://github.com/Open-CMSIS-Pack/devtools/issues/1850.
The expectation is that all required packs (including DFPs) are listed in the solution itself, not in candidate layers.

Nightly build of CMSIS-Toolbox: manifest_2.6.1-38-gc654765.yml
Even after removing the line - pack<EMAIL_ADDRESS>specifying the DFP the layer detection successfully determines the 5 possible configurations relying on the device specified by the selected board: NUCLEO-F756ZG as mountedDevice.


How to add this to existing hooks?

Can I run this as a script on top of my own git hooks? I'm using https://github.com/typicode/husky and it manages my git hooks for me. Basically I can configure husky to run any terminal command as a git hook.

Yes, you should be able to deduce the required configuration for husky from:
https://github.com/awslabs/git-secrets/blob/1.3.0/git-secrets#L195-L215
e.g. "pre-commit": "git secrets --prepare_commit_msg_hook [...]"


Feat/spec issue5

Would close #5

maybe I'm reading it wrong but you used capitalization in property names that do not reflect the DB schema

I'll fix that

and you seem to have duplicated topic.skills in resource.skills - in Resources the skills is a string[] not an object[].

From Topic
                "skills": {
                        "description": "The list of skills related to this topic",
                        "bsonType": "array",
                        "items": {
                                "bsonType": "objectId"
                        }
                },

https://github.com/agile-learning-institute/mentorHub-topic-api/blob/6975f7e94bc11503eac638b9be9bb83ae4a27219/docs/openapi.yaml#L191-L194
From Resources
                "skills": {
                        "description": "The list of skills this resource contributes to",
                        "bsonType": "array",
                        "items": {
                                "bsonType": "objectId"
                        }
                },

https://github.com/agile-learning-institute/mentorHub-topic-api/blob/6975f7e94bc11503eac638b9be9bb83ae4a27219/docs/openapi.yaml#L231-L234
I'm not sure where the disconnect is?
Sample output from the swagger editor
{
  "Id": "string",
  "Name": "string",
  "Status": "Active",
  "Description": "string",
  "Category": "Foundations of ITSM",
  "Skills": [
    {
      "Name": "string",
      "Status": "Active",
      "Description": "string",
      "Mastery": "Candidate",
      "Speciality": "Generalist"
    }
  ],
  "Resources": [
    {
      "Name": "string",
      "Status": "Active",
      "Description": "string",
      "Duration": 2880,
      "Link": "https://5SKOODwl7ThyO303HnDFFxFhZxTKPsh3RgUazcF35y5NuieF7pkl6sshW.-LmWoRnWh7V3VSRJIBwdsfA.kTkZTW6BM2DsaT0wt2bexydpWj2U1Hwxbk-pWUAmkWIHStn-DmdyCcKu.61JBfQ3F5TFWxDbzl0zzidRTSZAI1cF9jO6D.K9.b7NMP8Pifq1AL7lSG8qn7CjoCeFptQIl6aC7ULOPpuuEU3w-yVB.3tqwUcF9WLS-84HEy",
      "Skills": [
        {
          "Name": "string",
          "Status": "Active",
          "Description": "string",
          "Mastery": "Candidate",
          "Speciality": "Generalist"
        }
      ],
      "Tags": [
        "$"
      ]
    }
  ]
}

we need a GET /api/path/ endpoint to get a list of paths

Just names and IDs like with GET /api/topic or full path objects?

I think we need POST/GET/PATCH/DELETE endpoints for /api/topic/skills/

I made this a separate issue


localize substitutes at most one given placeholder

Due to a bug in argument passing between the default localize function and util.format, only the first placeholder of a given localization string is replaced correctly.
While util.format expects the substitutes as single arguments, localize passes only the first argument which is given to it on to util.format.
Even using the way that is shown in the examples by passing in an array as values argument doesn't work with util.format as it doesn't know what to do with that array.

Thanks for the PR here, great catch! It looks like 5.1.1 was published, is this resolved now for you?

Yes, everything works now.


Refactoring work done

Hi @privateOmega,
I was delighted stumbling over your lib and took some steps further to improve it a little. My main concern was the bundle size and the usage in a browser. I started to work on it and had some success with it. If you are interested in it please have a look here: https://github.com/Cruaier/html-to-docx.
I replaced virtual-dom with htmlparser2, removed all (not anymore) needed dependencies and switched to UintArray and Blob instead of Buffer.
Bundling is done with vite instead of rollup (it was working better for me).
I tested my changes with your prepared tests and the result document is pretty much the same. The main benefit is the bundle size though as the artifacts are now less than half the size as before.
Feel free to take my changes (or the parts you like) and put them into your lib.
BR

Can you publish this version into npm I am interested to compare with existing one ?
Look a cool work, thank you !

@traycho here you go: https://www.npmjs.com/package/html-to-docx-lite
I'll add to it when I have more time

@traycho here you go: https://www.npmjs.com/package/html-to-docx-lite I'll add to it when I have more time

Thank you for ur work, however, I tried to open the exported .docx or .doc via Microsoft® Word 适用于 Microsoft 365MSO ( 2403 Build 16.0.17425.20176) 64 , but I failed.


extra space at the end of generated Versions object

.. as one could see here.
Thank you for the coolest gradle plugin I've seen maybe in the entire 2018!

Thank you for the coolest gradle plugin I've seen maybe in the entire 2018!
@alexvas Thanks for your kind words!

extra space at the end of generated Versions object #24

I can fix that :)
In the meantime, the IMHO other coolest plugin from 2018 allows you to reformat your entire codebase so that you don't have to talk about tabs versus space in your pull requests.
https://github.com/shyiko/ktlint
https://github.com/JLLeitschuh/ktlint-gradle


🚀 | Add resume option

Feature Description
What functionality would you like to see in your phoniebox?
Also in future3 the resume functionality should be provided.
Eg some folders (audiobook) should be set to always resume from the previous position.
How do you envision the feature to work from a users perspective?
Set in the configuration of a card to resume it and the next time switching from another card to this card, the playback should resume from the old position.

With a month or two of real-life-with-kids experience after building, I've been thinking about this issue and related functionality 'tweaks' that stem from the dual use cases of 1) jukebox (music) and 2) audiobook player. Resume functionality is pretty critical for a manageable audiobook solution, and I've addressed that by installing votti:future3/feature/resume.
As @votti mentioned when opening this issue, it would be great if audiobook folders could be set to automatically enable this function. It would also be great if other existing functions could have their behavior tweaked by the same logic (e.g. perhaps mapping the 'next track' and 'previous track' buttons to seek forwards/backwards a preset (but configurable) number of seconds.
Novice question - does the way the application current reads metadata about the contents of each folder help it to 'know' the difference between music and an audiobook? How difficult/trivial would it be to implement something that made that distinction simpler?
While the use of m4b files could help (similar to the podcast/URL stream implementation), many audiobooks have historically come as tracks of MP3 files, similar to an album of music. Perhaps in those cases it's just a matter of the user setting a flag somewhere to inform the jukebox app that the currently-playing content should be treated as an audiobook.

@Seeking backwards and forward: Actually that is also a behaviour I wanted:
For the seek forward I found this already existed in the form of a seek function.
For seek backwards I implemented something to work both nicely for large and small audios:
https://github.com/votti/RPi-Jukebox-RFID/commit/c746000ec9d20172f93b96c867c4a3316f3c0592
Seeing that I am not the only one that likes this kind of behavior, I am opening a new issue.

Very nice, thanks for sharing that commit.
Reading through your issue, I see that file length might have as much to do with the desired behavior than just media type.
How did you go about mapping your custom RPC call to a physical action? Did you use a custom control card, a physical switch, or something else?

I mapped the custom RPC to a physical buttons (forward/backwards buttons on the box for the kids to smash).


introduce stalebot

We do have quite a lot of support-type issues that are answered but went stale. This PR introduces stalebot with almost the same config (different labels) as we have in prometheus-operator to solve this problem.
/cc @coreos/team-monitoring

Actually, with GitHub Discussions enabled I'd like to first see how that goes.
Most of the issues then should really only be technical and not as many so that we might even need the stale bot?
At least that's my hope.


Error not bubble up

While leaping :) from v0.21.0 to 0.60.0, ran into this.
GraphiQL started throwing this error:

Even updated to 0.17.5 with same message.
Decided to remove the nuget package and reference the project directly. Then VS threw this message.

Added the custom scalar type and the Failed to fetch error when away.
schema.AddCustomScalarType(typeof(DateTime), "DateTime");
schema.AddCustomScalarType(typeof(DateTime?), "DateTime");

Not sure why it didn't bubble up into the QueryResult.

Hmm, I can't seem to reproduce, can you reproduce in the demo project by changing it?

Attempted in the demo project, but much trouble has I am using Core 3.1 and not 2.2.
I thinking it would be the same issue if you commented out your custom scalar types. Then tried running it.
 public class GraphQLSchema
    {
        public static SchemaProvider<DemoContext> MakeSchema()
        {
            // build our schema directly from the DB Context
            var demoSchema = SchemaBuilder.FromObject<DemoContext>();

            //demoSchema.AddCustomScalarType(typeof(DateTime), "Date");
            //demoSchema.AddCustomScalarType(typeof(DateTime?), "Date");

If that doesn't throw the error, I will attempt again tomorrow.

That is what I tried. It worked find. I shall have dot net 3.1. It happily runs the 2.2 project. Although I will upgrade the demo project when I next get the chance

From: Jeremy R. Travis<EMAIL_ADDRESS>Sent: Wednesday, May 13, 2020 6:45:33 AM
To: lukemurray/EntityGraphQL<EMAIL_ADDRESS>Cc: Luke Murray<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [lukemurray/EntityGraphQL] Error not bubble up (#43)
Attempted in the demo project, but much trouble has I am using Core 3.1 and not 2.2.
I thinking it would be the same issue if you commented out your custom scalar types. Then tried running it.
public class GraphQLSchema
{
public static SchemaProvider MakeSchema()
{
// build our schema directly from the DB Context
var demoSchema = SchemaBuilder.FromObject();
        //demoSchema.AddCustomScalarType(typeof(DateTime), "Date");
        //demoSchema.AddCustomScalarType(typeof(DateTime?), "Date");

If that doesn't throw the error, I will attempt again tomorrow.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/lukemurray/EntityGraphQL/issues/43#issuecomment-627584025, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AABGHUIQCAHSKLNOP2ISMPDRRGYO3ANCNFSM4MZA5V5A.

Another thought. When using GraphiQL, which schema file are you loading; SDL or Introspection?
I notice your demo wanting to create a SDL.
The error generated from the Introspection query.

Yep, I just ran the demo project which launches Grahiql and that runs the
introspection. I didn't get any error even when commenting out the dates
On Fri, May 15, 2020 at 12:43 AM Jeremy R. Travis<EMAIL_ADDRESS>wrote:

Another thought. When using GraphiQL, which schema file are you loading;
SDL or Introspection?
I notice your demo wanting to create a SDL.
The error generated from the Introspection query.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lukemurray/EntityGraphQL/issues/43#issuecomment-628681640,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABGHUJKNPOZTAQDZTSSENTRRP7QVANCNFSM4MZA5V5A
.

Couldn't get the demo to work with 3.1.1, so I installed v 2.2.207 to test out the demo. Commented out the two custom scalar methods, then run.
VS did break with error "No mapped entry found...", however the error was different in GraphiQL.
Confirmed and using the same version 0.17.5.

Even when referencing the library via nuget. The same error... without the VS break.
Probably not a huge deal, usually I see a introspection error instead.


Indefinite Load of Sign In to Admin after Verifying Email<EMAIL_ADDRESS>

When I inspect element --> console, it's blank, so might not be a bug but something with the loading path blocked? if that makes any sense.
Reply to this email directly, view it on GitHub
https://github.com/keepid/keepid_client/issues/227#issuecomment-765551803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APUVKXP26GG7LNXSBO6J2CDS3GU6PANCNFSM4WEMAAOA
.

Yep, thanks
On Fri, Jan 22, 2021 at 11:57 AM Connor Chong<EMAIL_ADDRESS>wrote:

this is still a a bug that we are investigating, but we have turned off
2fa for this account for now
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/keepid/keepid_client/issues/227#issuecomment-765551803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APUVKXP26GG7LNXSBO6J2CDS3GU6PANCNFSM4WEMAAOA
.