Move flanneld + calico setup  to use Kubernetes storage

Flanneld + calico, a.k.a. canal setup can use Kubernetes as storage instead of Etcd. This removes need to keep Etcd certificate on all nodes, making whole setup more secure, as access to etcd grants unlimited powers to malicious user.

I just enabled Calico on my cluster and a bunch of reading led me to believe that Kubernetes storage for calico does not support EGRESS.  Only INGRESS.  To get EGRESS support you must use the etcd backend.

@jeremyd , Kubernetes network policy support ingress rules only and KDD is sufficient enough to implement it.
Egress rules is a calico-specific feature which has very little to do with Kubernetes.

@redbaron I realize that, just pointing out that users (such as myself) might want EGRESS and are only using calico for that specific feature.  Otherwise we don't need calico..

@jeremyd Hi, thanks for sharing the hard won info!
I think I understand your use-case.
Probably we need a switch for choosing a calico backend then?
Anyway, would you mind adding links to the related readings here so that everyone has chances to decide which backend one should use?

@jeremyd , I was under the impression that Calico would let you run user controlled calico+node daemonset + k8s policy controller and etcd for them (probably managed by etcd-operator) alongside KDD setup managed by kube-aws, but it seems that endpoints i etcd are created by CNI plugin,not by policy-controller.
Thanks for heads up, we'll make datastore type switch to keep both options available.

Egress rules in NetworkPolicy are part of 1.8  release https://github.com/kubernetes/kubernetes/pull/51351
Calico support is expected to arrive too.

Talking with @davidmccormick about implementing this.
See https://kubernetes.slack.com/archives/C5GP8LPEC/p1519213974000090 for more details, contexts.

The Canal project has this 3 container setup which incorporates calico/flannel/cni-install into a DS deployed to each node.  Link here https://github.com/projectcalico/canal/blob/master/k8s-install/1.7/canal.yaml.
I used this model at Zopa for our home rolled clusters and it worked well.

@davidmccormick Thanks for the valuable info!
I believe that we're talking about the 3rd option of https://docs.projectcalico.org/v3.0/getting-started/kubernetes/installation/hosted/kubernetes-datastore/#option-3-calico-policy-only-with-flannel-networking. Please correct me if I'm wrong.

I'm wondering what limitations exist after reading old calico doc vs the new one.
For example, does ipam issue really exist even when we used canal?

This Canal setup is getting more important as it would solve ip addr exhaustion issue https://github.com/coreos/flannel/issues/954 in shorter term

Note: calicoctl doesn't work for Canal+K8S API backend setup. Thx to @davidmccormick for the info

calicoctl works with k8s datastore starting from version 3.0.1

Thx for the info. Such a nice change for a patch version :)


Gradients w/ Hamiltonians fail to work if the coefficients are not vanilla NumPy float arrays

Consider the following circuit:
dev = qml.device("lightning.qubit", wires=2)

coeffs = np.array([0.1, 0.2])
terms = [qml.PauliX(0) @ qml.PauliZ(1), qml.PauliZ(0)]
H = qml.Hamiltonian(coeffs, terms)

@qjit
@qml.qnode(dev)
def circuit(x):
    qml.RX(x[0], wires=0)
    qml.RY(x[1], wires=0)
    qml.CNOT(wires=[0, 1])
    return qml.expval(H)

params = jnp.array([0.3, 0.4])

The gradient works fine as long as coeffs is a vanilla NumPy float array:
>>> jax.grad(circuit)(params)
Array([-0.05443844, -0.07440512], dtype=float64)

However, if coeffs is a PL NumPy or JAX NumPy array, this will fail:
>>> coeffs = np.array([0.1, 0.2])
...
>>> jax.grad(circuit)(params)
TypeError: get(): incompatible function arguments. The following argument types are supported:
    1. (array: buffer, signless: bool = True, type: Optional[jaxlib.mlir._mlir_libs._mlir.ir.Type] = None, shape: Optional[List[int]] = None, context: mlir.ir.Context = None) -> jaxlib.mlir._mlir_libs._mlir.ir.DenseElementsAttr

Invoked with: Array([0.1, 0.2], dtype=float64)

Similarly, execution will fail if coeffs is an integer array.

Although it is nice to see the following work:
dev = qml.device("lightning.qubit", wires=2)

@qjit
@qml.qnode(dev)
def circuit(x, coeffs):
    qml.RX(x[0], wires=0)
    qml.RY(x[1], wires=0)
    qml.CNOT(wires=[0, 1])

    terms = [qml.PauliX(0) @ qml.PauliZ(1), qml.PauliZ(0)]
    H = qml.Hamiltonian(coeffs, terms)

    return qml.expval(H)

>>> params = jnp.array([0.3, 0.4])
>>> coeffs = jnp.array([0.1, 0.2])
>>> jax.grad(circuit, argnums=[0, 1])(params, coeffs)
(Array([-0.05443844, -0.07440512], dtype=float64),
 Array([0.        , 0.87992318], dtype=float64))

That is, the ability to differentiate with respect to Hamiltonian coefficients.


update simplicity

I believe it is important to define ReactJS as a JavaScript library as stated on their website.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

LGTM  and congratulations on your first PR 😄🎉
Thanks for contributing!

thank you for accepting the PR.


Feature/cto 134 reservoir project

This feature isn't done but I've done some moves and refactoring and its getting harder to merge and track changes.


Docker on SLES12?

ref our docker ansible role for SLES12 https://github.com/adoptium/infrastructure/blob/master/ansible/playbooks/AdoptOpenJDK_Unix_Playbook/roles/Docker/tasks/sles.yml
We add the docker repos for SLES12 on any architecture, but we install docker only on x64.
@sxa Do we require docker on SLES12 s390x?

Can it be installed and used for the tests that require it? I'd say it's not critical that we have it, but ultimately the decision should come down to whether it can be installed on those systems and is usable.

Can't seem to install container-selinux which is a dependency for docker-ce
test-marist-sles15-s390x-2:~ # zypper install docker-ce
Refreshing service 'SMT-http_lxslsmt'.
Loading repository data...
Reading installed packages...
Resolving package dependencies...
Problem: nothing provides 'container-selinux' needed by the to be installed docker-ce-3:23.0.3-1.s390x
 Solution 1: do not install docker-ce-3:23.0.3-1.s390x
 Solution 2: break docker-ce-3:23.0.3-1.s390x by ignoring some of its dependencies

It can install on sles15 by adding the https://download.opensuse.org/repositories/security:SELinux/SLE_15_SP3/security:SELinux.repo repo

I can't find anything by googling, but I would assume installing docker on the sles12 box using container-selinux from a sles15 repo would cause issues?

I would expect so, so let's not do it on 12

I'll put the changes into the playbook


Service Timeout: Due to service inactivity, the client buffer exceeded maximum size. Resetting the buffer.

Describe the bug
When using continuous speech translation there is an intermittent cancellation of the recognizer, stopping translation for that client. This is an intermittent occurrence that seems to happen rarely.
We are passing a real-time stream (using AudioInputStream) to the recognizer, so we're not sure how the buffer is exceeding the maximum size as the rate shouldn't exceed any possible window allocated by the recognizer.
To Reproduce
Unknown, no behavior discovered yet that indicates what is required to reproduce consistently. It might be if the translator has been running continuous speech translation for too long, but I have seen instances longer running than the ones that it occurred in.
Expected behavior
The client sdk should continue to recognize speech when passing in a real-time audio stream for any length of time.
Version of the Cognitive Services Speech SDK
Java 1.28.0
Platform, Operating System, and Programming Language
OS: Linux (Debian 11 docker container) x64
Client SDK: Java 1.28.0
Additional context
Region: eastus
Session id: 80e2d12dc1c14edf92a87da4fba84707
(Please pm for specific azure project id if needed by azure team)
We unfortunately have not caught this error with full logging enabled, but will try to in the future to update here.
Stacktrace:
java.lang.Exception: Speech translation canceled.
  at com.microsoft.cognitiveservices.speech.util.EventHandlerImpl.fireEvent(Unknown Source)
  at com.microsoft.cognitiveservices.speech.translation.TranslationRecognizer.canceledEventCallback(Unknown Source)
Unexpected error message: [Speech translation canceled.]
Unexpected error details: [Due to service inactivity, the client buffer exceeded maximum size. Resetting the buffer. SessionId: 80e2d12dc1c14edf92a87da4fba84707]

I've seen mentions of ways to manually change the buffer size in the config for the recognizer. What would be a recommended size to try to mitigate this to happen at a lower rate, even if it doesn't 100% fix it?

Thanks for providing the session ID, it was helpful in starting an investigation.
From a client perspective, the buffer will hold up to about a minute of audio before ending recognition, and filling the buffer (typically) means the service responses are running about a minute behind real-time.
I'm talking with our service team about this session, the telemetry looks atypical but I'm not sure why yet.

We think we've identified a root cause and are working with the service teams to find the right fix.
In the meantime, there may be a workaround we can try. If you email me @microsoft.com I can share details.

I am using azure-cognitiveservices-speech==1.29.0 python3.10 SDK , initially it was working fine but now even with small files like 12MB size and length of 1 minute it stucks in this error
Note I also added pronunciation config to my speech_config but it should be deal
here am attaching my log file
my_logs_speechsdk.txt
I created a support ticket on the Azure portal, however no updates at all
@pankopon Thanks for considering

@Tryptophan the service fix should be rolling out now (It actually started earlier this week) and should greatly reduce the possibility of hitting this problem.
@bobir01, your timeout is a different root problem where the service isn't segmenting the phrase as expected.

@rhurey  thank you for your attention where should I report my problem, could you refer please

@bobir01, we've got the cause identified and are discussing the right long term solution.
If you could send me an email @microsoft.com (same alias) I may have a temporary solution you can try.
The problem you're running into is a convergence of a very long phrase, pronunciation assessment, and single phrase recognition.

@rhurey I got it,
so did you mean sending the email to<EMAIL_ADDRESS>in pm ? Did I understand it right?

@rhurey I got it, so did you mean sending the email to<EMAIL_ADDRESS>in pm ? Did I understand it right?

yes

I am not sure if i have a similar problem
buffer filled, pausing addition of binaries until space is made

This is the error that is coming up. This issue comes up on two cases

When an audio is generated using text to speech
When an audio is recognized using speech to text.

I am not able to debug this. Is there a way i can add a fail safe mechanism to reset this myself.
is there a way to retry this?
is there an event handler which i can listen to when this issue happens? right now the error phase does not retrieve this data

@albseb511 could you open a new issue for that? That error string is coming from the speaker output in the JavaScript SDK and it's going to be a different investigation, and we already have two separate issues comingled here.

Hey, give me sometime I'll do that.

https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2046
Done.

@albseb511 could you open a new issue for that? That error string is coming from the speaker output in the JavaScript SDK and it's going to be a different investigation, and we already have two separate issues comingled here.

Done.
https://github.com/Azure-Samples/cognitive-services-speech-sdk/issues/2046

Hi @rhurey  we are seeing this issue again on the Java client, see the error log here:
2024-03-27T02:03:19.328178340Z CANCELED: Reason=Error
2024-03-27T02:03:19.328789335Z CANCELED: ErrorCode=ServiceTimeout
2024-03-27T02:03:19.329073137Z CANCELED: ErrorDetails=Due to service inactivity, the client buffer exceeded maximum size. Resetting the buffer. SessionId: cfb5a08c5b0440e98f8e3ca0e1db9688
2024-03-27T02:03:19.329686404Z java.lang.Exception: Speech translation canceled.
2024-03-27T02:03:19.329900580Z 	at lambda$buildRecognizer$4(Translator.java:362)
2024-03-27T02:03:19.330070095Z 	at com.microsoft.cognitiveservices.speech.util.EventHandlerImpl.fireEvent(Unknown Source)
2024-03-27T02:03:19.330248685Z 	at com.microsoft.cognitiveservices.speech.SpeechRecognizer.canceledEventCallback(Unknown Source)
2024-03-27T02:03:19.331123668Z Unexpected error message: [Speech translation canceled.]
2024-03-27T02:03:19.332238660Z Unexpected error details: [Due to service inactivity, the client buffer exceeded maximum size. Resetting the buffer. SessionId: cfb5a08c5b0440e98f8e3ca0e1db9688]

We seem to only be able to reproduce it when setting the following:
speechConfig.setProperty(PropertyId.Speech_SegmentationSilenceTimeoutMs, "2000");

This seems to be reproduced more on machines with lower memory availability if that's helpful.
We are on version 1.36.0 (latest) on the Java client sdk. This issue probably needs to be re-opened or I can open a new issue as well.
Let me know if you need other details, thank you!

Hi @rhurey @glharper any information on my last comment?


wont deploy

Describe the bug
wont deploy
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-m9fqez
Steps to reproduce

i press deploy
it wont deploy

Expected behavior
to deploy
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 130
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 45629899,
  "usedJSHeapSize": 41437999,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = waldo8888
Chat ID = adefbaf150cd

Additional context
Error:
Failed building the project. Make sure the build command is correct and try again.

Appreciate the feedback! We are aware of this issue and are tracking it here: #2019


Client-sdk components & interactions design

Effort of @zrlthk, @elli-andoulaki, @adecaro, in collaboration with @smithbk, @binhn .

@elli-androulaki I still can't assign this item to you because you havenot joined the fabric-contributors team. Could you check your email.

I think you should be able to do so now...

Is this issue for native golang client SDK?

Hi @GrapeBaBa, this is supposed to be generic enough to accomodate also other languages.

Hi @GrapeBaBa, Expanding on @adecaro, if you think there should be an additional language and there isn't a current issue open, would be worth creating a new issue for specific language support.

@ibmmark I already created a issue #1206 , from the maillist it seems someone already contribute python and php SDK, I'd like to contribute a java client SDK, I will upload to maven center once my sonatype account is available.

@elli-androulaki @smithbk can this be closed?

@christo4ferris Yes


[FEATURE] Alternative to scrapbook to execute notebooks programmatically for tests

Description

Scrapbook is not being developed anymore, and it doesn't support Python 3.10 (See https://github.com/recommenders-team/recommenders/pull/1988#issuecomment-1712425248)
Expected behavior with the suggested feature

Other Comments

WIP: https://github.com/miguelgfierro/pybase/pull/73

@miguelgfierro I don't find any alternatives to scrapbook after googling.  The solutions, I think, may be

Use nbformat as you do in https://github.com/miguelgfierro/pybase/pull/73
Borrow and modify the functions like sb.glue() as @anargyri said in the meeting and make them part of Recommenders.

For the 1st option, I saw you written the functions to do that, what else should we do?  I can try tomorrow.

Actually, option 2 is kind of what I was doing in 1. If you have time it would be great if you could try 1.

I think I have finished the functions, see https://github.com/miguelgfierro/pybase/pull/73
Man, they say the most difficult language in the world is Chinese, but I think it is regex. Thank goodness that we have ChatGPT.
@SimonYansenZhao  I'll start a PR soon to replace all the scrapbook

Fixed!


Create environment variable for TreeComm max fan out

Potentially undocumented environment variable for configuring the max fan out.  Target usage will be aimed at running the integration tests in multilevel configuration without needing large clusters.

Related to #100


raft: do not reset vote if term is not changed

raft MUST keep the voting information for the same term. reset
should not reset vote if term is not changed.
/cc @bdarnell

lgtm

LGTM


容器管理 - 在集群中部署第二调度器文档中helm应用安装开始应该是应用这个调度器而不属于安装这一块，建议标题进行区分

Describe the bug
容器管理 - 在集群中部署第二调度器文档中helm应用安装开始应该是应用这个调度器而不属于安装这一块，建议标题进行区分

A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

done.


Number of repositories is limited to 100

I am trying to scrape an organisation with more than 100 repos. I am getting only first 100.

Hey @mfornasa
Super open to pull requests to implement pagination support in the Github Exporter. Alternatively, you could provide the container with a list of repositories as an array (via REPOS environment variable) and that will allow you to return data for >100 repos.

@flock3 thank you for the workaround.

@mfornasa have you had any issues when running with more than a handle of repos? If I give an ORG that has 100 repos, trying to access /metrics just hangs forever

I also have this issue.. :-1:


Contract tests

An experiment to see if contract tests can replace (partially) the integration tests.

Codecov Report

Merging #189 (ee8d22d) into main (8cab4f5) will increase coverage by 0.28%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #189      +/-   ##
==========================================
+ Coverage   88.55%   88.84%   +0.28%
==========================================
  Files          19       19
  Lines         690      690
  Branches       78       78
==========================================
+ Hits          611      613       +2
+ Misses         64       60       -4
- Partials       15       17       +2

Impacted Files
Coverage Δ

forwarder/update_handlers/tdct_serialiser.py
82.92% <0.00%> (-4.88%)
:arrow_down:

forwarder/update_handlers/pva_update_handler.py
87.87% <0.00%> (+12.12%)
:arrow_up:


Drop support for Elasticsearch 1.x

With this commit we remove support in Rally to benchmark Elasticsearch
1.x.
Closes #715

Thanks for the review! :)


Pre-trained model performance issues?

Hi
I'm loading the last snapshot of my trained network on hopper environment using this line for ppo2:
model.load('[...]/baselines/logs/Hopper-v1/checkpoints/00048')
But for some reason the eprewmean starts from zero as of not-trained networks and then becomes better after 50K timesteps. Shouldn't it start from the last reward mean of the saved model?

Update:
In case the question is not clear, here's a screenshot that shows episode reward mean plot for (1) learning from scratch (blue curve) and (2) loading the blue curve network and continuing learning (red curve)

Can anyone tell me how to fix it? I think the red curve should start from where the blue curve has left off (around 600).

Hi @donamin . I'm pretty sure you should save and load observation scaling. Start from this line:     env = VecNormalize(env)


feat(windows): print error code string description in case of errors

Change-Type: minor
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Fixed!

@hedss @lekkas I don't know what's going on, but VersionBot is not working on this repo anymore :(

@jviotti The release tag for 5.0.25 isn't valid, it should be v5.0.25 so versionist can't find the reference. I guess it was added manually.

@hedss I see, I always seem to get the v prefix wrong :P Could we make VersionBot print a more descriptive error message, or maybe even a warning, but still make it cope with the lack of the v?

I think if we used VersionBot 100%, and never merged manually, then we'd never get invalid tags?

True. I went for manual merges given that we needed to keep moving, and I didn't know how to fix it.

It's a fair comment. Ideally versionist needs to be libraried so we can can use it and get better error messages out. Unfortunately I've had zero time to do this. :-(


Edit for more readability

Hi
Edit for more readability.
Good luck

[x] Have you followed the contributing guidelines?
[x] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Please help me for This error "Process completed with exit code 1."

Please help me for This error "Process completed with exit code 1."

👋 Hi, @ciminbinazadeh! I see this annotation on the files changed tab. I think addressing it could get the tests passing.

I see this annotation on the files changed tab. I think addressing it could get the tests passing.
Hi @nholden
thank you 🙏🌸

Thanks to nick and his amazing magic work ,, glad u have it done


buld(deps): bump palette and boost hash

This PR resolves []
Description

PR Checklist

[ ] I have tested my changes on iOS and Android.
[ ] I hid my changes behind a feature flag, or they don't need one.
[ ] I have included screenshots or videos, or I have not changed the UI.
[ ] I have added tests, or my changes don't require any.
[ ] I added an app state migration, or my changes do not require one.
[ ] I have documented any follow-up work that this PR will require, or it does not require any.
[ ] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

Warnings

:warning:

❌ An error occurred while validating your changelog, please make sure you provided a valid changelog.

  Generated by :no_entry_sign: dangerJS against e3b6b6032560d778b3c9587c585c4b64786f8edd

closing this since we tested and it was successful ~ will open another PR for the actual dep update once palette-mobile pr is merged


feat: adding podLabels option

Adding podLabels support

Signed-off-by: Frédéric Gaudet<EMAIL_ADDRESS>

 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.


Add CBPinEntryView

Project URL
https://github.com/Fawxy/CBPinEntryView
Description

Added CBPinEntryView, a drop in view which allows easy numerical code entry with backspacing, continuation and view customisation.
Why it should be included to awesome-ios (optional)
Nothing similar in the list, other views use several text fields and add hidden characters to enable backspacing, this uses a single hidden text field which also allows the code to be grabbed as a single String or Int without having to concatenate a bunch of strings from several text fields.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

Hey @Fawxy Nic lib, thanks for contributing to awesome-ios🎉🎉🎉


Added HTML5 native Color picker

Fixes #38 .
PS : not sure about compatibility, we can switch to a polyfill library if this does not work.

There are compatibility issues. Polyfill also wont fix the problem.


NextJS  13 TS/Zod  dependency issue

First off thank you for contributing towards building this. I think this is super useful in building streaming applications with function calling.
I am trying to use this in React with NextJS 13. I am also using the Vercel AI SDK's useCompletion hook on the client side that then calls an API route in the api directiory to stream the responses. I get this error with this setup:
./node_modules/typescript/lib/typescript.js
Critical dependency: the request of a dependency is an expression

Import trace for requested module:
./node_modules/typescript/lib/typescript.js
./node_modules/zod-to-ts/dist/index.js
./node_modules/openai-partial-stream/dist/chunk-66SDEIDM.mjs
./node_modules/openai-partial-stream/dist/index.mjs
./src/app/api/completion/route.ts
./node_modules/next/dist/build/webpack/loaders/next-app-loader.js?name=app%2Fapi%2Fcompletion%2Froute&page=%2Fapi%2Fcompletion%2Froute&appPaths=&pagePath=private-next-app-dir%2Fapi%2Fcompletion%2Froute.ts&appDir=%2FUsers%2Fakashdeepdeb%2FDesktop%2Ftopshelf%2Fsrc%2Fapp&pageExtensions=tsx&pageExtensions=ts&pageExtensions=jsx&pageExtensions=js&rootDir=%2FUsers%2Fakashdeepdeb%2FDesktop%2Ftopshelf&isDev=true&tsconfigPath=tsconfig.json&basePath=&assetPrefix=&nextConfigOutput=&preferredRegion=&middlewareConfig=e30%3D!./src/app/api/completion/route.ts?__next_edge_ssr_entry__
<w> [webpack.cache.Pack

Any ideas if this is related to a zod to ts version issue?

Hi @akdeb, thank you for reporting the issue. I will investigate.

The zod-to-ts have replaced by zod-to-json-schema, on the latest version 0.3.7


Clarify the example that uses certificate config

https://docs.microsoft.com/en-us/azure/service-fabric/service-fabric-service-manifest-resources#example-specifying-an-https-endpoint-for-your-service
This example needs to highlight some very key points the absence of which makes developer life very hard.

The SF cluster only supports certificates which have CAPI1 as the key provider. See https://docs.microsoft.com/en-us/azure/service-fabric/cluster-security-certificates#troubleshooting-and-frequently-asked-questions where it talks about NTE_BAD_PROV_TYPE error.
Upon activating a service that specifies an https endpoint, Service Fabric will set the access control entry for the port, bind the specified server certificate to the port, and also grant the identity that the service is running as permissions to the certificate's private key. The activation flow is invoked every time Service Fabric starts, or when the certificate declaration of the application is changed via an upgrade. The endpoint certificate will also be monitored for changes/renewals, and permissions will be periodically reapplied as necessary. (This is verbatim from Endpoints section but adding a link to that here would help).
FindByThumbprint is generally not a great idea since when the cert is renewed, the config needs to be changed and pretty much removes the major benefit mentioned in ##2 above.

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

ID: 94aa1985-f344-537c-ba38-d5a9a1626a4d
Version Independent ID: 5d685dd3-dc6b-879c-202c-c20d1fe21066
Content: Specifying Service Fabric service endpoints - Azure Service Fabric
Content Source: articles/service-fabric/service-fabric-service-manifest-resources.md
Service: service-fabric
GitHub Login: @erikadoyle
Microsoft Alias: edoyle

Thanks for the feedback, I've assigned this issue to the doc author to take a look at it.

Thanks for your dedication to our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response. Though the scope of our feedback channel here on GitHub covers specific documentation fixes, we can help redirect you to the right support channel to get an answer to your question: [reference to alternative product/service or community endpoint]. Because this issue does not relate to documentation, we are closing this issue. #please-close


pyOCD/tools support for non-mbed boards

So I noticed that the MbedBoard class is hardwired into the tools so that there is no way to run the tools using a non-mbed board without changing the core code. This seems kind of silly since you can basically change two variables - the mbed_vid and mbed_pid - and everything else works as expected. Would it be possible to add something so that the user can specify vid/pid instead of hardwiring it?
Follow up question: Is theMbedBoard class really necessary? Or rather is it possible to merge that code into the Board class? It just seems like the only thing in that class that is really mbed specific in the class is the vid and pid. I think even the BoardInfo table could be included in the Board class without issue.
Perhaps the MbedBoard class could be dropped and the pid and vid could be arguments to the Board class with default values equal to the mbed device value which could be changed in the tools using command line arguments.
BTW, I'd be happy to implement this, but I'm just trying to get an idea if y'all would be supportive of taking the code in this direction before I actually code anything.

Hi @tgarc
Thanks for the interest in the project.  There are some changes planned in just this area.  The goal is to make pyOCD not tied to the VID/PID, but to still detect the attached CMSIS-DAP enabled debuggers.  If the board was an mbed then the target info could be easily extracted, otherwise the target would need to be detected or specified by the user.
Also, out of curiosity, what use case is giving you trouble?  Are you working on a custom CMSIS-DAP device with its own VID/PID?

@c1728p9 Thanks for the response. I'm glad to hear this is already in the works! You're right, I am using a CoLinkEx with the CMSIS-DAP firmware for debugging. I do have to provide the target to the command line tools, but that makes sense because I'm not using a pyOCD "supported" board. Works just fine other than that PID/VID restriction.
Is there anything planned along the lines of my questions about the MbedBoard class? The getAllConnectedBoards, listConnectedBoards and chooseBoard methods in particular seem like they could easily be moved to the Board superclass and would be generally useful to anyone wanting to write their own Board subclass.

The future of the MbedBoard class is a little uncertain at this point.  The logic might very well end up in the Board class.  The code which lists connected boards will definitely be changed, with most of those functions being moved down into the transport layer.

I see. Then I guess it will be just to keep my personal projects forked for now. Feel free to close the issue.

Just a heads up, the PR #229 removes the use of VID/PID.  Boards supporting CMSIS-DAP are now found based on the product string.


Add sponsor button

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (0c10df3) compared to base (f316fbd).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #258   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            7         7
  Lines          687       687
=========================================
  Hits           687       687

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


open edX: update learning MFE to support PublicPath webpack setting

As a @mitodl/devops, I'd like to be able to route the learning MFE (frontend-app-learning) from a path on the lms server, instead of having to run it on a separate server.
For other MFE's this was implemented by updating the version of frontend-platform and frontend-build used by the MFE, and making the route configurable in index.jsx. For example, see https://github.com/edx/frontend-app-gradebook/pull/145
It may also help to read https://discuss.openedx.org/t/deploying-mfes-in-the-community/
Acceptance Criteria:

[x] fork https://github.com/edx/frontend-app-learning/ into the mitodl organization
[ ] create a branch with the necessary changes
[ ] test them on devstack by setting PublicPath and rebuilding frontend-app-learning with webpack
[ ] alert @mitodl/devops so we can test on our QA deployment.
[ ] open a PR on edx/frontend-app-learning -- I'm reasonably optimistic we can get this merged quickly

To put this in a devstack context, if you run devstack master currently and access a course, it will be displayed through the Learning MFE (also known as frontend-app-learning) on port 2000.
Instead of accessing the Learning MFE at / on port 2000, you should be able to configure it to run on the lms port (18000) at /learn/ -- the same way you can configure the Gradebook MFE to run from https://localhost:18000/gradebook.

To post an update:
While following the instructions mentioned in the PR and a few other documents e.g. Technical Discovery Proposal I was able to run the gradebook at localhost:1994/gradebook/<course_id> but definitely not on something like localhost:18000/gradebook/<course_id> . What I did in the process mostly is:

Setting up PUBLIC_PATH='/gradebook/' in .env.development
Rebuilding the platform assets/webpack e.g, make dev.static

However when you mention

the same way you can configure the Gradebook MFE to run from https://localhost:18000/gradebook

Have we done it somewhere on any server e.g. https://lms-qa.mitxonline.mit.edu/ ? Are there any additional steps to above to run gradebook like localhost:18000/gradebook or maybe?
Or maybe devops can put more light on running something localhost:1994/gradebook/<course_id> is fine and they can configure it to run on 18000 with the /gradebook path setup on, in which case I can configure the similar changes in frontend-app-learning making it run on a path something like localhost:2000/learn/courses/<course_id> which currently only runs on localhost:2000/courses/<course_id>.

Additionally, I'm going to try and update frontend-build to 7.0.0 or above which supports public path setup using PUBLIC_PATH env and see if it makes us able to run the MFE directly on 18000.
More updates are to be posted soon.
CC: @pdpinch

It's probably sufficient to get the MFE to deploy to localhost:2000/learn/courses/<course_id>, assuming you can configure devstack sufficiently to test.
@shaidar can tell you more about how we are deploying the MFEs on QA and production.
It might help to bring up the issue of how to develop and test path-based deployments in devstack (and/or Tutor) on discuss.edx.org. There's a very long thread about the initial creation of this feature, but I find it too complex to follow.

Correct, we don't need for the MFE to be accessible on the same port as the LMS, we just need to be able to specify a path where it is located. In our Fastly configuration we will be able to conditionally serve the assets from S3 at those paths but with the same domain as the LMS.

Thanks for adding the details, Based on above, it makes sense to configure the frontend-app-learing to be accessible at localhost:2000/learn/course/<course_id> which currently runs localhost:2000/course/<course_id>.
I will make the required changes for the path to be configurable based on PUBLIC_PATH env e.g. something like https://github.com/edx/frontend-app-gradebook/pull/145 so that frontend-app-learning could run on localhost:2000/learn/course/<course_id>.

@pdpinch,
Here are some additional details:
I looked more into the index.jsx implementation of frontend-app-learning which uses the AppProvider component from frontend-platform and it turns out that the public path is already supported in it and it won’t need similar changes like this gradebook PR(These changes in grade book index.jsx were actually a workaround because it is using IntlProvider from react-intl instead of AppProvider component from frontend-platform unlike other MFEs -- here is the link for this workaround in gradebook)
For some of the other MFEs which were using AppProvider, they only needed frontend-build and frontend-platform version bumps for them to support path-based routing e.g. frontend-app-profile, frontend-app-account. The required  frontend-platform>=1.6.1 and  frontend-build>=5.3.2 are already done in lilac & master of frontend-app-learning.
Based on the above I think we might not need any specific change in frontend-app-learning. I was also able to run the MFE directly without the changes we thought would be required for supporting path-based routing e.g. https://github.com/mitodl/frontend-app-learning/pull/1 (This PR now becomes invalid).
Local changes I did to run frontend-app-learning at localhost:2000/learn/course/<course_id>

Set PUBLIC_PATH variable to ‘/learn’ in .env.development — This will be set in .env in the production
Rebuild the MFE webpack (you could simply run make dev.static.lms)
Restart the frontend-app-learning container to reload the above change e.g. — docker-compose restart frontend-app-learning

Other changes that might be needed for redirect support are with idx-platform:

Setting https://github.com/edx/edx-platform/blob/master/lms/envs/common.py#L4639 to the URL of the MFE e.g. http://localhost:2000/learn in my case after setting PUBLIC_PATH to ‘/learn/’ (This will make sure that upon clicking on a course, the details are opened in the learning MFE )
There are other settings as well for e.g. setting WRITABLE_GRADEBOOK_URL  to open the gradebook from instructor page value for example.

Testing:
I tested it with the current master of edX and Lilac and frontend-app-learning with path-based routing is working. Attaching a screenshot of it working on /learn:

Wow, thanks @arslanashraf7. That's unexpected.
Do you know when this change was made? I just want to make sure that we have the right version in our deployment.

@pdpinch The specifically required or above version upgrades were done in the below PRs:

For frontend-build it was upgraded in This PR
For frontend-platform it was upgraded in This PR
And as mentioned in the gradebook PR we were referring to, the frontend-app-learning is already using AppProvider right from the start.

As a matter of fact, all of the above changes are part of the lilac release of frontend-app-learning already and I was able to test the routing on lilac as well where after setting the PUBLIC_PATH='/learn/' enabled me to load the learning MFE on localhost:2000/learn/course/<course_id>.
I'm not sure if we tried the lilac release of this MFE or there is something else to this that I might be missing?

Also just to add more, there is this PR specific to edx/configurations that added the support for subdirectories support.

This has been confirmed working for RC and production. Closing.


Made the zoom controls more accessible

As mentioned in issue #7211 i made the controls more accessible by adding the aria-disabled attribute when it is disabled

i am not allowed to review code, but i read through the changes and thought about the implications.
this commits:

add DomUtil.setAttribute(element, attribute, value) in src/dom/DomUtil.js
seem to correctly set 'aria-disabled' to the Zoom control in src/control/Control.js

i think it's clear that this code needs to add a DomUtil.setAttribute function,
which makes this a lot more impactful than obvious at first glance.
this means that

a DomUtil.removeAttribute(element, attribute) function should be added too
the documentation has to be updated to include the DomUtil methods
if other controls need aria attributes, those should be added as well.

@Augani you read into this a bit, are there more controls that need aria attributes?

Already fixed in #7280


ENH: Add default atlases

Closes https://github.com/miykael/atlasreader/issues/72.
As discussed in https://github.com/miykael/atlasreader/issues/72, this PR adds default atlases. I'm not sure if the name default is the best solution or if we should handle it differently.

Codecov Report

Merging #73 into master will increase coverage by 0.15%.
The diff coverage is 85.71%.

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
+ Coverage   91.89%   92.05%   +0.15%
==========================================
  Files           6        6
  Lines         358      365       +7
==========================================
+ Hits          329      336       +7
  Misses         29       29

Impacted Files
Coverage Δ

atlasreader/cli.py
0% <0%> (ø)
:arrow_up:

atlasreader/atlasreader.py
97.93% <100%> (+0.04%)
:arrow_up:

atlasreader/tests/test_atlasreader.py
100% <100%> (ø)
:arrow_up:

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 673069e...b87ffa1. Read the comment docs.

I think this is ready to be merged. Only open question is if you're happy with the default flag or if we should use something else.


Parameter of empty list results in TypeError

When calling register function of https://explorer.stacks.co/txid/0x89cff4241836c31c6a47bc268225b379f076a04709a27ec4665b7747d191d9c2?chain=mainnet
on https://stacks-pool-registry.pages.dev/pools the call results in an half-empty screen and the console log shows:
Uncaught (in promise) TypeError: c is undefined
Screenshot:

@markmhx can we confirm this is still an issue?

@friedger are you still seeing this? You're the only one to report this so far iirc.

Seems fixed.

This is still a problem with empty integer lists, as seen in Boom dev
@dantrevino

Stacks wallet version 2.24.1
console output:
TypeError: u is undefined
    f clarityValue.js:111
    f clarityValue.js:133
    h function-argument-item.tsx:21
    React 7
    unstable_runWithPriority scheduler.production.min.js:18
    React 5
    fe index.js:509
    xe index.js:418
    xe index.js:418
    xe index.js:408
    We index.js:303
    write index.js:633
    X index.js:281
    We index.js:301
    C wallet.hooks.ts:123
react-dom.production.min.js:216:199

Looks like the current transaction js version is does not include the fix https://github.com/blockstack/stacks.js/blob/master/CHANGELOG.md#201-beta1-2021-07-26

Yes, confirmed I'm seeing this as well.

This is fixed in our dev branch, can you try this build ?
https://github.com/hirosystems/stacks-wallet-web/releases/tag/v2.24.0-beta.11

@beguene do you know why this version number is behind main? We released v2.24.1 yesterday ...do I need to do something after the revert PR?

Yes I was surprised too. As such it should auto-increment correctly after merge into main, but the beta version is still wrong. If you don't find out or don't have time, I will have a look at the github actions tomorrow.


Layout not inflating properly on when StepperAdapter in Fragment

I did not see anything related to the Stepper being used in Fragment (F1) instead of Activity, so I gave it a try. At first, it works, but has a bug where the Stepper is not inflated correctly when F1 is remove and then added again.
Steps to reproduce:

Implement an Activity with a FrameLayout to contain the Fragments and the Fragment that will start the Fragment with the Stepper, which I called CallerFragment;
Implement the Stepper in a Fragment, which I called MainFragment;
Implement 3 Fragments that extends Step, which I called FirstStepFragment, SecondStepFragment and ThirdStepFragment;
On a click event, replace the CallerFragment for the MainFragment;
It will inflate everything properly
Finish the MainFragment and repeat step 4;
The Steps will not be loaded properly, as I need to go to the third step and come back to the first to be able to see the content

Finally, is it a problem with using the Stepper with Activity? Or am I doing something wrong?
Replacing CallerFragment for MainFragment

final Fragment fragment = new MainFragment();
final FragmentTransaction transaction = getFragmentManager().beginTransaction();
transaction.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_OPEN);
transaction.replace(R.id.main_container, fragment).addToBackStack("callerFragment").commit();

Destroying the MainFragment, which contains the Stepper, and popping the backstack

FragmentManager fragmentManager = getActivity().getSupportFragmentManager();
FragmentTransaction transaction = fragmentManager.beginTransaction();
transaction.remove(this).commit();
fragmentManager.popBackStack("callerFragment", FragmentManager.POP_BACK_STACK_INCLUSIVE);

Hi @lucasharada,
I have never tried putting the StepperLayout in a Fragment before so I'm not really sure how it would behave.
When using constructing AbstractFragmentStepAdapter does the FragmentManager come from the Activity or are you using MainFragment#getChildFragmentManager()?

@zawadz88 I'm currently using MainFragment#getFragmentManager() (not sure if the same as Activity#getFragmentManager()).

I think that for MainFragment this will be the same as Activity#getFragmentManager().
Please try with MainFragment#getChildFragmentManager() and let me know if it helps.

@zawadz88 That worked! Thanks a lot!

I think that for MainFragment this will be the same as Activity#getFragmentManager().
Please try with MainFragment#getChildFragmentManager() and let me know if it helps.

I suggest you to insert this information in the main page.
Thank you


library not found

Error in library(nrBrand) : there is no package called ‘nrBrand’

{nrBrand} is a private R package so cannot be installed from a public repository. It loads font files and provides helper functions for captions with icons as described in this blog post: https://nrennie.rbind.io/blog/adding-social-media-icons-ggplot2/.


Support defining functions in code instead of function.json

This work was unblocked in the host by https://github.com/Azure/azure-functions-host/pull/7606. For now, it requires users to include EnableWorkerIndexing in the AzureWebJobsFeatureFlags app setting.
This specific issue is to track basic support in the core worker api for defining functions in code. I imagine this essentially just means users can copy/paste their current "function.json" content into a js/ts file. It should be a simple change to their codebase that will let them completely redefine their file structure. Anything fancier will be tracked by the new programming model package in https://github.com/Azure/azure-functions-nodejs-worker/issues/480
Prereqs:

Provide api for the worker: https://github.com/Azure/azure-functions-nodejs-worker/issues/567
Add entrypoint for app-level code: https://github.com/Azure/azure-functions-nodejs-worker/issues/537

I did a rough first commit on this branch that only works with "OptionA-Simple" in the prototype repo here. Keep in mind you have to add this to the local.settings.json in the app you're testing:
"AzureWebJobsFeatureFlags": "EnableWorkerIndexing"
Plan as discussed today was to have @hossam-nasr take it from here, which includes:

Get the other prototype options to "work" - for the purposes of a user study and to validate the core api is on the right track
Make the rough commit an actual PR for merging


Adding a performance boost with an index on visitable_id, visitable_t…

…ype, and date.

I have about 8 million+ records of visits and this helped boost the speed of the queries retrieving the data.

I have about 8 million+ records of visits and this helped boost the speed of the queries retrieving the data.

Thanks!! Merged @smayzes

Thanks!! Merged @smayzes


Make SpirvValue contain an enum for its value

This is a prerequisite for all sorts of work (panics, printf extension, builtin functions, nested constants, all sorts of fun stuff) - SpirvValueKind is eventually going to have a decent number of cases. However, just for a proof-of-concept, I've added one "extra" case: ConstantPointer. Not only does this work towards my goal of removing the horrible zombie_even_in_user_code hack, it significantly improves error messages, since we're able to report violations of using the constant pointer directly in the compiler, not the linker, and so we have span information.
Before:
  error: dynamic use of address of constant
    |
    = note: Stack:
            main_fs
            Unnamed function ID %343

After:
  error: Cannot use this pointer directly, it must be dereferenced first
     --> examples/shaders/sky-shader/src/lib.rs:184:5
      |
  184 |     f(&2);
      |     ^^^^^

Because this is such a huge change, it's going to merge conflict with pretty much any compiler change, so I'm going to just merge this now.


feat: add support for GKO native API

see: https://gravitee.atlassian.net/browse/GKO-702

📚  View the storybook of this branch here

Hey @kamiiiel, I'm wondering if we should merge this PR first: https://github.com/gravitee-io/gravitee-api-management/pull/10003.


Upgrade to Angular 7

Need that full Shadow DOM V1 support
https://github.com/angular/angular/blob/master/CHANGELOG.md#700-2018-10-18

Hi @seanforyou23, do you say this is a bug fix so we can squeeze in to 7.2, or post 7.2 thing? I'm gonna release today for 7.2 RC build and wonder if we want this in.

Given this is a major upgrade, maybe better to have it post 7.2

@igarashitm totally something that can wait. Not a bug, just an enhancement :)

merged to master.


refactor: remove useless field in header

What this PR does / why we need it:
This PR remove some useless field in block header structure.
Which issue(s) this PR fixes:

Fixes #
Which docs this PR relation:
Ref #
Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:
NIL
CI Description

CI Name
Description

Chaos CI
Test the liveness and robustness of Axon under terrible network condition

Cargo Clippy
Run cargo clippy --all --all-targets --all-features

Coverage Test
Get the unit test coverage report

E2E Test
Run end-to-end test to check interfaces

Code Format
Run cargo +nightly fmt --all -- --check and cargo sort -gwc

Web3 Compatible Test
Test the Web3 compatibility of Axon

v3 Core Test
Run the compatibility tests provided by Uniswap V3

OCT 1-5 | 6-10 | 11 | 12-15 | 16-19
Run the compatibility tests provided by OpenZeppelin

CI Usage

Check the CI you want to run below, and then comment /run-ci.

CI Switch

[ ] Chaos CI
[x] Cargo Clippy
[ ] Coverage Test
[x] E2E Tests
[x] Code Format
[x] Unit Tests
[x] Web3 Compatible Tests
[ ] OCT 1-5 And 12-15
[ ] OCT 6-10
[ ] OCT 11
[ ] OCT 16-19
[ ] v3 Core Tests

/run-ci

/run-ci

/run-ci

/run-ci


ItemList.get_select_index

Offer small improvements (improvements) ItemList
In continuation: A&Q
It would be very convenient if ItemList were method:
ItemList.get_select_index
which returns the index of the selected list item.

That's a good idea.
The main issue however is that the ItemList can be used in single-select or multi-select mode, so the method would have to return either an index or an array of indexes depending on the mode.. That may be why up to now it was not implemented, letting users relying on signals instead.

In fact, you can always return an array:

the length of the array = 0 - no selected items
the array length is >0 - there are selected items

If I chose single-select, I can do without the extra checks call:
ItemList.get_select_index[0]
I can't speak for everyone, but for me personally this decision is fully satisfied.

ItemList.get_select_index[0]

You couldn't do that, as it would be a function, not a member variable. So it would be get_select_index()[0].
For clarity, I guess the function should be named get_selected_items(), what do you think?

You couldn't do that, as it would be a function, not a member variable. So it would be >get_select_index()[0]

Yes, you are right. I was not thinking of the listing rules.

For clarity, I guess the function should be named get_selected_items(), what do you think?

Sorry, I not speak English... so most likely you are right.

Closed by #5468.


Add detail view page

Love this library and where it's headed. 🙏 Thanks!
Is your feature request related to a problem? Please describe.
Often a table can't display clearly all information about an entity, Would prefer not having to horizontally scroll.
Describe the solution you'd like

I would like to be able to view an entity with perhaps many fields as a vertical scroll rather than in table form (horizontal) on a separate page from "List". Call it "View"
I would suggest losing the hamburger menu and replacing it with a "View" button
And from this "View" page give the option to "Edit"
And move the delete option to the "Edit" page or have "Delete" next to "Edit" on the "View" page, either way

Describe alternatives you've considered
Using the edit page as the detail page. Not ideal.
Would you accept a PR for this?

Hey, thanks for kicking off the discussion on this. I've been putting off adding a "detail view" since as you noted, the edit page can perform that function even if it is not ideal. However, I think now that the architecture is a bit more stable I think it's a reasonable time to do it. If you want to open a PR I'd be happy to review it.
Some current thoughts:
The table buttons/links are one thing that's changed a lot. Beside the edit and delete controls, which used to each be separate buttons, the hamburger also lists any "actions" defined on the resource. Since there can potentially be a few of those, I wanted to avoid rendering buttons for each, and I don't want to force the user to enter a detail view to run them. Another option there would be to introduce a row selection UX (e.g. checkboxes) which could be used to run any of those supplementary actions on one or multiple rows using a single dropdown, in which case we could replace the row hamburger with a view button. But that seems like a discussion that should happen in a separate thread dedicated to that feature.
As for the detail view itself, one thing I've been going back and forth on is whether it should be a separate route, or a modal that is shown when a given url param is present on the list view. I am a bit inclined to the latter because my guess the UX will feel a bit snappier to enter/exit detail views quickly, but I'd be interested in your thoughts there.
cheers

Awesome to hear this is aligned with your vision.
Re hamburger button: How about we just leave is as is and add the "view" action for now? Or is your preference for word "detail"?
Re View/Detail as modal. I see how how a page redirect is unnecessary because the data is already there. The modal is like a "zoom in", more ephemeral experience. But our needs for the library differ quite a bit from that. We want a "View" page because each individual entity will have various actions and extra contextual data that doesn't fit nicely into a modal or the table. The "View" page is where we would end up having custom templates for the most part, not just custom actions (that could go in the hamburger menu).
Maybe the table-first approach makes more sense for a library though -- with actions embedded there. Somehow our admin pages "List" pages are more like listViews (with filters and sorting only) and actions on the entity instance level. Not sure if that's the case for other people.

One other quick point on this. We noticed there isn't a link for association fields. Which makes sense: If an article has a authorId column, clicking the cell couldn't possibly have a destination link without the "View" as a page. We have a certain need for this in our admin.

How about we just leave is as is and add the "view" action for now? Or is your preference for word "detail"?

Yes I think for let's just add a link to the new component alongside the other actions in the hamburger. I think "View" or "Show" make the most sense. I am planning on gettext integration at some point hopefully so users will be able to customize this kind of copy easily.

We want a "View" page because each individual entity will have various actions and extra contextual data that doesn't fit nicely into a modal or the table.

This makes sense to me. A separate route is the most customizable so let's go with that. It can work similar to most of the existing routes that load a component inside the Container depending on route.

We noticed there isn't a link for association fields.

I think association cells in the table did actually link to the edit page at one point. I removed that when I added the ability to override the render logic for each field. The logic there has become a bit complex and might take some refactoring to get it to work, so although I agree we should figure out a way to get it to link to this new view, we can tackle that in a separate PR after the show itself has been added.


Audio Controller not getting using kernel driver vfio-pci

i have 2 gpus and virtualization enabled, so it should work
this is the output of
lspci -k | grep -E "vfio|NVIDIA"
07:00.0 VGA compatible controller: NVIDIA Corporation GA106 [GeForce RTX 3060 Lite Hash Rate] (rev a1)
        Kernel driver in use: vfio-pci
07:00.1 Audio device: NVIDIA Corporation GA106 High Definition Audio Controller (rev a1)

also, i checked dmesg and it showed that both devices are being added:
[    6.910118] vfio_pci: add [10de:2504[ffffffff:ffffffff]] class 0x000000/00000000
[    6.910137] vfio_pci: add [10de:228e[ffffffff:ffffffff]] class 0x000000/00000000

vfio_pci.ids=10de:2504,10de:228e

(they are both correct, i triple checked)
and also in /etc/modprobe.d/vfio.conf i also added options vfio-pci ids=10de:2504,10de:228e but it didnt work either

the output of utils/ls-iommu -grk is:
IOMMU Group  12: 0000:01:00.0 USB controller [0c03] Advanced Micro Devices, Inc. [AMD] unknown [1022:43d0] (rev 01)         Subsystem: ASMedia Technology Inc. unknown [1b21:1142]         Kernel driver in use: xhci_hcd IOMMU Group  12: 0000:01:00.1 SATA controller [0106] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset SATA Controller [1022:43c8] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset SATA Controller [1b21:1062]         Kernel driver in use: ahci IOMMU Group  12: 0000:01:00.2 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Bridge [1022:43c6] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Bridge [1b21:0201]         Kernel driver in use: pcieport IOMMU Group  12: 0000:02:00.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]         Kernel driver in use: pcieport IOMMU Group  12: 0000:02:01.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]         Kernel driver in use: pcieport IOMMU Group  12: 0000:02:02.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]         Kernel driver in use: pcieport IOMMU Group  12: 0000:02:03.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]         Kernel driver in use: pcieport IOMMU Group  12: 0000:02:04.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]         Kernel driver in use: pcieport IOMMU Group  12: 0000:02:09.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)         Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]         Kernel driver in use: pcieport IOMMU Group  12: 0000:03:00.0 Ethernet controller [0200] Realtek Semiconductor Co., Ltd. RTL8111/8168/8211/8411 PCI Express Gigabit Ethernet Controller [10ec:8168] (rev 15)         Subsystem: ASUSTeK Computer Inc. PRIME B450M-A Motherboard [1043:8677]         Kernel driver in use: r8168 IOMMU Group  12: 0000:07:00.0 VGA compatible controller [0300] NVIDIA Corporation GA106 [GeForce RTX 3060 Lite Hash Rate] [10de:2504] (rev a1)         Subsystem: Micro-Star International Co., Ltd. [MSI] GA106 [GeForce RTX 3060 Lite Hash Rate] [1462:397d]         Kernel driver in use: vfio-pci IOMMU Group  12: 0000:07:00.1 Audio device [0403] NVIDIA Corporation GA106 High Definition Audio Controller [10de:228e] (rev a1)         Subsystem: Micro-Star International Co., Ltd. [MSI] GA106 High Definition Audio Controller [1462:397d]         Kernel driver in use: snd_hda_intel IOMMU Group  12: 0000:08:00.0 Non-Volatile memory controller [0108] Kingston Technology Company, Inc. A2000 NVMe SSD SM2263EN [2646:2263] (rev 03)         Subsystem: Kingston Technology Company, Inc. A2000 NVMe SSD SM2263EN [2646:2263]         Kernel driver in use: nvme IOMMU Group  13: 0000:09:00.0 VGA compatible controller [0300] Advanced Micro Devices, Inc. [AMD/ATI] Baffin [Radeon RX 550 640SP / RX 560/560X] [1002:67ff] (rev cf)         Subsystem: Sapphire Technology Limited Radeon RX 560 [1da2:e348]         Kernel driver in use: amdgpu IOMMU Group  13: 0000:09:00.1 Audio device [0403] Advanced Micro Devices, Inc. [AMD/ATI] Baffin HDMI/DP Audio [Radeon RX 550 640SP / RX 560/560X] [1002:aae0]         Subsystem: Sapphire Technology Limited Baffin HDMI/DP Audio [Radeon RX 550 640SP / RX 560/560X] [1da2:aae0]         Kernel driver in use: snd_hda_intel
the debug.log is:
`2024/03/24 16:59:31
/etc/modprobe.d found on the system
Creating config/etc/modprobe.d
2024/03/24 16:59:31
/etc/default found on the system
Creating config/etc/default
2024/03/24 16:59:31
/etc/dracut.conf.d found on the system
Creating config/etc/dracut.conf.d
2024/03/24 16:59:31
/etc/default/grub found on the system
Creating config/etc/default/grub
2024/03/24 16:59:31
Executing: utils/ls-iommu -g -F vendor:,prod_name,optional_revision:,device_id
2024/03/24 16:59:31
ls-iommu query returned
IOMMU Group  12: NVIDIA Corporation: GA106 [GeForce RTX 3060 Lite Hash Rate] (rev a1): [10de:2504]
IOMMU Group  13: Advanced Micro Devices, Inc. [AMD/ATI]: Baffin [Radeon RX 550 640SP / RX 560/560X] (rev cf): [1002:67ff]
2024/03/24 16:59:34
Executing: utils/ls-iommu -g -r -i 12 -F vendor:,prod_name,optional_revision:,device_id
2024/03/24 16:59:36
ls-iommu query returned
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Bridge (rev 01): [1022:43c6]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Port (rev 01): [1022:43c7]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset SATA Controller (rev 01): [1022:43c8]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: unknown (rev 01): [1022:43d0]
IOMMU Group  12: Kingston Technology Company, Inc.: A2000 NVMe SSD SM2263EN (rev 03): [2646:2263]
IOMMU Group  12: NVIDIA Corporation: GA106 High Definition Audio Controller (rev a1): [10de:228e]
IOMMU Group  12: NVIDIA Corporation: GA106 [GeForce RTX 3060 Lite Hash Rate] (rev a1): [10de:2504]
IOMMU Group  12: Realtek Semiconductor Co., Ltd.: RTL8111/8168/8211/8411 PCI Express Gigabit Ethernet Controller (rev 15): [10ec:8168]
2024/03/24 16:59:45
Executing: utils/ls-iommu -g -rr -i 12 -F vendor:,prod_name,optional_revision:,device_id
2024/03/24 16:59:48
ls-iommu query returned
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Bridge (rev 01): [1022:43c6]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Port (rev 01): [1022:43c7]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset SATA Controller (rev 01): [1022:43c8]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: unknown (rev 01): [1022:43d0]
IOMMU Group  12: Kingston Technology Company, Inc.: A2000 NVMe SSD SM2263EN (rev 03): [2646:2263]
IOMMU Group  12: NVIDIA Corporation: GA106 High Definition Audio Controller (rev a1): [10de:228e]
IOMMU Group  12: NVIDIA Corporation: GA106 [GeForce RTX 3060 Lite Hash Rate] (rev a1): [10de:2504]
IOMMU Group  12: Realtek Semiconductor Co., Ltd.: RTL8111/8168/8211/8411 PCI Express Gigabit Ethernet Controller (rev 15): [10ec:8168]
2024/03/24 17:00:12
Executing: utils/ls-iommu -g -i 12 --rom
2024/03/24 17:00:12
ls-iommu query returned
/sys/devices/pci0000:00/0000:00:01.3/0000:01:00.2/0000:02:04.0/0000:07:00.0/rom
2024/03/24 17:00:12
Writing utils/dump_vbios.sh
2024/03/24 17:00:21
Adding vfio_pci.disable_vga=1 to config/kernel_args
2024/03/24 17:00:21
User has disabled vfio video output on host, adding disable_vga=1 to the optional hardcoded vfio_pci options
2024/03/24 17:00:21
Writing config/etc/modprobe.d/vfio.conf:
This is an autogenerated file that stubs your graphic card for use with vfio
This file should be placed inside /etc/modprobe.d/
Uncomment the line below to "hardcode" your graphic card to be bound to the vfio-pci driver.
In most cases this should not be neccessary, it will also prevent you from turning off vfio in the bootloader.
#options vfio_pci ids=10de:228e,10de:2504 disable_vga=1
Make sure vfio_pci is loaded before these modules: nvidia, nouveau, amdgpu and radeon
softdep nvidia pre: vfio vfio_pci
softdep nouveau pre: vfio vfio_pci
softdep amdgpu pre: vfio vfio_pci
softdep radeon pre: vfio vfio_pci
2024/03/24 17:00:21
Writing to config/etc/dracut.conf.d/vfio.conf:
add_drivers+=" vfio_pci vfio vfio_iommu_type1 "
2024/03/24 17:00:21
Configuring grub2 manually
2024/03/24 17:00:21
Read /etc/default/grub:
GRUB boot loader configuration
GRUB_DEFAULT='0'
GRUB_TIMEOUT='5'
GRUB_DISTRIBUTOR='EndeavourOS'
GRUB_CMDLINE_LINUX_DEFAULT='nowatchdog nvme_load=YES resume=UUID=2b98b356-0833-443f-8ce8-d6800e9cb6ac loglevel=3'
GRUB_CMDLINE_LINUX="iommu=pt amd_iommu=on vfio_pci.ids=10de:2504,10de:228e vfio_pci.disable_vga=1 rd.driver.pre=vfio_pci"
Preload both GPT and MBR modules so that they are not missed
GRUB_PRELOAD_MODULES="part_gpt part_msdos"
Uncomment to enable booting from LUKS encrypted devices
#GRUB_ENABLE_CRYPTODISK=y
Set to 'countdown' or 'hidden' to change timeout behavior,
press ESC key to display menu.
GRUB_TIMEOUT_STYLE=menu
Uncomment to use basic console
GRUB_TERMINAL_INPUT=console
Uncomment to disable graphical terminal
#GRUB_TERMINAL_OUTPUT=console
The resolution used on graphical terminal
note that you can use only modes which your graphic card supports via VBE
you can see them in real GRUB with the command `videoinfo'
GRUB_GFXMODE=auto
Uncomment to allow the kernel use the same resolution used by grub
GRUB_GFXPAYLOAD_LINUX=keep
Uncomment if you want GRUB to pass to the Linux kernel the old parameter
format "root=/dev/xxx" instead of "root=/dev/disk/by-uuid/xxx"
#GRUB_DISABLE_LINUX_UUID=true
Uncomment to disable generation of recovery mode menu entries
GRUB_DISABLE_RECOVERY='true'
Uncomment and set to the desired menu colors.  Used by normal and wallpaper
modes only.  Entries specified as foreground/background.
#GRUB_COLOR_NORMAL="light-blue/black"
#GRUB_COLOR_HIGHLIGHT="light-cyan/blue"
Uncomment one of them for the gfx desired, a image background or a gfxtheme
GRUB_BACKGROUND='/usr/share/endeavouros/splash.png'
#GRUB_THEME="/path/to/gfxtheme"
Uncomment to get a beep at GRUB start
#GRUB_INIT_TUNE="480 440 1"
Uncomment to make GRUB remember the last selection. This requires
setting 'GRUB_DEFAULT=saved' above.
#GRUB_SAVEDEFAULT=true
Uncomment to disable submenus in boot menu
GRUB_DISABLE_SUBMENU='false'
Probing for other operating systems is disabled for security reasons. Read
documentation on GRUB_DISABLE_OS_PROBER, if still want to enable this
functionality install os-prober and uncomment to detect and include other
operating systems.
GRUB_DISABLE_OS_PROBER=false
2024/03/24 17:00:21
Replacing line in config/etc/default/grub:
GRUB_CMDLINE_LINUX="iommu=pt amd_iommu=on vfio_pci.ids=10de:2504,10de:228e vfio_pci.disable_vga=1 rd.driver.pre=vfio_pci"
With:
GRUB_CMDLINE_LINUX="iommu=pt amd_iommu=on vfio_pci.ids=10de:228e,10de:2504 vfio_pci.disable_vga=1 rd.driver.pre=vfio_pci"
2024/03/24 17:00:27
Applying grub2 changes
2024/03/24 17:00:27
Executing command:
sudo cp -v "config/etc/default/grub" /etc/default/grub
2024/03/24 17:00:27
'config/etc/default/grub' -> '/etc/default/grub'
2024/03/24 17:00:29
Generating grub configuration file ...
Found background: /usr/share/endeavouros/splash.png
Found linux image: /boot/vmlinuz-linux-lts
Found initrd image: /boot/amd-ucode.img /boot/initramfs-linux-lts.img
Found fallback initrd image(s) in /boot:  amd-ucode.img initramfs-linux-lts-fallback.img
Found linux image: /boot/vmlinuz-linux
Found initrd image: /boot/amd-ucode.img /boot/initramfs-linux.img
Found fallback initrd image(s) in /boot:  amd-ucode.img initramfs-linux-fallback.img
Warning: os-prober will be executed to detect other bootable partitions.
Its output will be used to detect bootable binaries on them and create new boot entries.
Adding boot menu entry for UEFI Firmware Settings ...
done
2024/03/24 17:00:29
'config/etc/modprobe.d/vfio.conf' -> '/etc/modprobe.d/vfio.conf'
2024/03/24 17:00:29
'config/etc/dracut.conf.d/vfio.conf' -> '/etc/dracut.conf.d/vfio'
2024/03/24 17:00:29
Executing: sudo dracut -f -v --kver 6.6.22-1-lts
2024/03/24 17:00:29
dracut: Executing: /usr/bin/dracut -f -v --kver 6.6.22-1-lts
dracut: Can't write to /boot/efi/843f973e33514afdb46423d177c1bd29/6.6.22-1-lts: Directory /boot/efi/843f973e33514afdb46423d177c1bd29/6.6.22-1-lts does not exist or is not accessible.`
it seems that the problem might have something to do with dracut but im not an expert.

the output of utils/ls-iommu -grk is:
IOMMU Group  12: 0000:01:00.0 USB controller [0c03] Advanced Micro Devices, Inc. [AMD] unknown [1022:43d0] (rev 01)
        Subsystem: ASMedia Technology Inc. unknown [1b21:1142]
        Kernel driver in use: xhci_hcd
IOMMU Group  12: 0000:01:00.1 SATA controller [0106] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset SATA Controller [1022:43c8] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset SATA Controller [1b21:1062]
        Kernel driver in use: ahci
IOMMU Group  12: 0000:01:00.2 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Bridge [1022:43c6] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Bridge [1b21:0201]
        Kernel driver in use: pcieport
IOMMU Group  12: 0000:02:00.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]
        Kernel driver in use: pcieport
IOMMU Group  12: 0000:02:01.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]
        Kernel driver in use: pcieport
IOMMU Group  12: 0000:02:02.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]
        Kernel driver in use: pcieport
IOMMU Group  12: 0000:02:03.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]
        Kernel driver in use: pcieport
IOMMU Group  12: 0000:02:04.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]
        Kernel driver in use: pcieport
IOMMU Group  12: 0000:02:09.0 PCI bridge [0604] Advanced Micro Devices, Inc. [AMD] 400 Series Chipset PCIe Port [1022:43c7] (rev 01)
        Subsystem: ASMedia Technology Inc. 400 Series Chipset PCIe Port [1b21:3306]
        Kernel driver in use: pcieport
IOMMU Group  12: 0000:03:00.0 Ethernet controller [0200] Realtek Semiconductor Co., Ltd. RTL8111/8168/8211/8411 PCI Express Gigabit Ethernet Controller [10ec:8168] (rev 15)
        Subsystem: ASUSTeK Computer Inc. PRIME B450M-A Motherboard [1043:8677]
        Kernel driver in use: r8168
IOMMU Group  12: 0000:07:00.0 VGA compatible controller [0300] NVIDIA Corporation GA106 [GeForce RTX 3060 Lite Hash Rate] [10de:2504] (rev a1)
        Subsystem: Micro-Star International Co., Ltd. [MSI] GA106 [GeForce RTX 3060 Lite Hash Rate] [1462:397d]
        Kernel driver in use: vfio-pci
IOMMU Group  12: 0000:07:00.1 Audio device [0403] NVIDIA Corporation GA106 High Definition Audio Controller [10de:228e] (rev a1)
        Subsystem: Micro-Star International Co., Ltd. [MSI] GA106 High Definition Audio Controller [1462:397d]
        Kernel driver in use: snd_hda_intel
IOMMU Group  12: 0000:08:00.0 Non-Volatile memory controller [0108] Kingston Technology Company, Inc. A2000 NVMe SSD SM2263EN [2646:2263] (rev 03)
        Subsystem: Kingston Technology Company, Inc. A2000 NVMe SSD SM2263EN [2646:2263]
        Kernel driver in use: nvme
IOMMU Group  13: 0000:09:00.0 VGA compatible controller [0300] Advanced Micro Devices, Inc. [AMD/ATI] Baffin [Radeon RX 550 640SP / RX 560/560X] [1002:67ff] (rev cf)
        Subsystem: Sapphire Technology Limited Radeon RX 560 [1da2:e348]
        Kernel driver in use: amdgpu
IOMMU Group  13: 0000:09:00.1 Audio device [0403] Advanced Micro Devices, Inc. [AMD/ATI] Baffin HDMI/DP Audio [Radeon RX 550 640SP / RX 560/560X] [1002:aae0]
        Subsystem: Sapphire Technology Limited Baffin HDMI/DP Audio [Radeon RX 550 640SP / RX 560/560X] [1da2:aae0]
        Kernel driver in use: snd_hda_intel

the debug.log is:
2024/03/24 16:59:31
/etc/modprobe.d found on the system
Creating config/etc/modprobe.d

2024/03/24 16:59:31
/etc/default found on the system
Creating config/etc/default

2024/03/24 16:59:31
/etc/dracut.conf.d found on the system
Creating config/etc/dracut.conf.d

2024/03/24 16:59:31
/etc/default/grub found on the system
Creating config/etc/default/grub

2024/03/24 16:59:31
Executing: utils/ls-iommu -g -F vendor:,prod_name,optional_revision:,device_id

2024/03/24 16:59:31
ls-iommu query returned
IOMMU Group  12: NVIDIA Corporation: GA106 [GeForce RTX 3060 Lite Hash Rate] (rev a1): [10de:2504]
IOMMU Group  13: Advanced Micro Devices, Inc. [AMD/ATI]: Baffin [Radeon RX 550 640SP / RX 560/560X] (rev cf): [1002:67ff]

2024/03/24 16:59:34
Executing: utils/ls-iommu -g -r -i 12 -F vendor:,prod_name,optional_revision:,device_id

2024/03/24 16:59:36
ls-iommu query returned
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Bridge (rev 01): [1022:43c6]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Port (rev 01): [1022:43c7]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset SATA Controller (rev 01): [1022:43c8]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: unknown (rev 01): [1022:43d0]
IOMMU Group  12: Kingston Technology Company, Inc.: A2000 NVMe SSD SM2263EN (rev 03): [2646:2263]
IOMMU Group  12: NVIDIA Corporation: GA106 High Definition Audio Controller (rev a1): [10de:228e]
IOMMU Group  12: NVIDIA Corporation: GA106 [GeForce RTX 3060 Lite Hash Rate] (rev a1): [10de:2504]
IOMMU Group  12: Realtek Semiconductor Co., Ltd.: RTL8111/8168/8211/8411 PCI Express Gigabit Ethernet Controller (rev 15): [10ec:8168]

2024/03/24 16:59:45
Executing: utils/ls-iommu -g -rr -i 12 -F vendor:,prod_name,optional_revision:,device_id

2024/03/24 16:59:48
ls-iommu query returned
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Bridge (rev 01): [1022:43c6]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset PCIe Port (rev 01): [1022:43c7]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: 400 Series Chipset SATA Controller (rev 01): [1022:43c8]
IOMMU Group  12: Advanced Micro Devices, Inc. [AMD]: unknown (rev 01): [1022:43d0]
IOMMU Group  12: Kingston Technology Company, Inc.: A2000 NVMe SSD SM2263EN (rev 03): [2646:2263]
IOMMU Group  12: NVIDIA Corporation: GA106 High Definition Audio Controller (rev a1): [10de:228e]
IOMMU Group  12: NVIDIA Corporation: GA106 [GeForce RTX 3060 Lite Hash Rate] (rev a1): [10de:2504]
IOMMU Group  12: Realtek Semiconductor Co., Ltd.: RTL8111/8168/8211/8411 PCI Express Gigabit Ethernet Controller (rev 15): [10ec:8168]

2024/03/24 17:00:12
Executing: utils/ls-iommu -g -i 12 --rom

2024/03/24 17:00:12
ls-iommu query returned
/sys/devices/pci0000:00/0000:00:01.3/0000:01:00.2/0000:02:04.0/0000:07:00.0/rom

2024/03/24 17:00:12
Writing utils/dump_vbios.sh

2024/03/24 17:00:21
Adding vfio_pci.disable_vga=1 to config/kernel_args

2024/03/24 17:00:21
User has disabled vfio video output on host, adding disable_vga=1 to the optional hardcoded vfio_pci options

2024/03/24 17:00:21
Writing config/etc/modprobe.d/vfio.conf:
## This is an autogenerated file that stubs your graphic card for use with vfio
## This file should be placed inside /etc/modprobe.d/
# Uncomment the line below to "hardcode" your graphic card to be bound to the vfio-pci driver.
# In most cases this should not be neccessary, it will also prevent you from turning off vfio in the bootloader.
#options vfio_pci ids=10de:228e,10de:2504 disable_vga=1

# Make sure vfio_pci is loaded before these modules: nvidia, nouveau, amdgpu and radeon
softdep nvidia pre: vfio vfio_pci
softdep nouveau pre: vfio vfio_pci
softdep amdgpu pre: vfio vfio_pci
softdep radeon pre: vfio vfio_pci

2024/03/24 17:00:21
Writing to config/etc/dracut.conf.d/vfio.conf:
add_drivers+=" vfio_pci vfio vfio_iommu_type1 "

2024/03/24 17:00:21
Configuring grub2 manually

2024/03/24 17:00:21
Read /etc/default/grub:
# GRUB boot loader configuration

GRUB_DEFAULT='0'
GRUB_TIMEOUT='5'
GRUB_DISTRIBUTOR='EndeavourOS'
GRUB_CMDLINE_LINUX_DEFAULT='nowatchdog nvme_load=YES resume=UUID=2b98b356-0833-443f-8ce8-d6800e9cb6ac loglevel=3'
GRUB_CMDLINE_LINUX="iommu=pt amd_iommu=on vfio_pci.ids=10de:2504,10de:228e vfio_pci.disable_vga=1 rd.driver.pre=vfio_pci"

# Preload both GPT and MBR modules so that they are not missed
GRUB_PRELOAD_MODULES="part_gpt part_msdos"

# Uncomment to enable booting from LUKS encrypted devices
#GRUB_ENABLE_CRYPTODISK=y

# Set to 'countdown' or 'hidden' to change timeout behavior,
# press ESC key to display menu.
GRUB_TIMEOUT_STYLE=menu

# Uncomment to use basic console
GRUB_TERMINAL_INPUT=console

# Uncomment to disable graphical terminal
#GRUB_TERMINAL_OUTPUT=console

# The resolution used on graphical terminal
# note that you can use only modes which your graphic card supports via VBE
# you can see them in real GRUB with the command `videoinfo'
GRUB_GFXMODE=auto

# Uncomment to allow the kernel use the same resolution used by grub
GRUB_GFXPAYLOAD_LINUX=keep

# Uncomment if you want GRUB to pass to the Linux kernel the old parameter
# format "root=/dev/xxx" instead of "root=/dev/disk/by-uuid/xxx"
#GRUB_DISABLE_LINUX_UUID=true

# Uncomment to disable generation of recovery mode menu entries
GRUB_DISABLE_RECOVERY='true'

# Uncomment and set to the desired menu colors.  Used by normal and wallpaper
# modes only.  Entries specified as foreground/background.
#GRUB_COLOR_NORMAL="light-blue/black"
#GRUB_COLOR_HIGHLIGHT="light-cyan/blue"

# Uncomment one of them for the gfx desired, a image background or a gfxtheme
GRUB_BACKGROUND='/usr/share/endeavouros/splash.png'
#GRUB_THEME="/path/to/gfxtheme"

# Uncomment to get a beep at GRUB start
#GRUB_INIT_TUNE="480 440 1"

# Uncomment to make GRUB remember the last selection. This requires
# setting 'GRUB_DEFAULT=saved' above.
#GRUB_SAVEDEFAULT=true

# Uncomment to disable submenus in boot menu
GRUB_DISABLE_SUBMENU='false'

# Probing for other operating systems is disabled for security reasons. Read
# documentation on GRUB_DISABLE_OS_PROBER, if still want to enable this
# functionality install os-prober and uncomment to detect and include other
# operating systems.
GRUB_DISABLE_OS_PROBER=false

2024/03/24 17:00:21
Replacing line in config/etc/default/grub:
GRUB_CMDLINE_LINUX="iommu=pt amd_iommu=on vfio_pci.ids=10de:2504,10de:228e vfio_pci.disable_vga=1 rd.driver.pre=vfio_pci"
With:
GRUB_CMDLINE_LINUX="iommu=pt amd_iommu=on vfio_pci.ids=10de:228e,10de:2504 vfio_pci.disable_vga=1 rd.driver.pre=vfio_pci"

2024/03/24 17:00:27
Applying grub2 changes

2024/03/24 17:00:27
Executing command:
sudo cp -v "config/etc/default/grub" /etc/default/grub

2024/03/24 17:00:27
'config/etc/default/grub' -> '/etc/default/grub'

2024/03/24 17:00:29
Generating grub configuration file ...
Found background: /usr/share/endeavouros/splash.png
Found linux image: /boot/vmlinuz-linux-lts
Found initrd image: /boot/amd-ucode.img /boot/initramfs-linux-lts.img
Found fallback initrd image(s) in /boot:  amd-ucode.img initramfs-linux-lts-fallback.img
Found linux image: /boot/vmlinuz-linux
Found initrd image: /boot/amd-ucode.img /boot/initramfs-linux.img
Found fallback initrd image(s) in /boot:  amd-ucode.img initramfs-linux-fallback.img
Warning: os-prober will be executed to detect other bootable partitions.
Its output will be used to detect bootable binaries on them and create new boot entries.
Adding boot menu entry for UEFI Firmware Settings ...
done

2024/03/24 17:00:29
'config/etc/modprobe.d/vfio.conf' -> '/etc/modprobe.d/vfio.conf'

2024/03/24 17:00:29
'config/etc/dracut.conf.d/vfio.conf' -> '/etc/dracut.conf.d/vfio'

2024/03/24 17:00:29
Executing: sudo dracut -f -v --kver 6.6.22-1-lts

2024/03/24 17:00:29
dracut: Executing: /usr/bin/dracut -f -v --kver 6.6.22-1-lts
dracut: Can't write to /boot/efi/843f973e33514afdb46423d177c1bd29/6.6.22-1-lts: Directory /boot/efi/843f973e33514afdb46423d177c1bd29/6.6.22-1-lts does not exist or is not accessible.

it seems that the problem might have something to do with dracut but im not an expert.

Well this is easy, the card you have selected will never work in the PCIe slot it is in, because it is in IOMMU group 12 which is shared with a bunch of other stuff.
The dialog where you select the gpu clearly states that the list should only contain the GPU and anything associated with it.

i mean i thought that if i went over to advanced, then manual select, it would somehow isolate those 2 from iommu group. that was my mistake. im going to look for a way to set it to another group, thanks for all the help!

i mean i thought that if i went over to advanced, then manual select, it would somehow isolate those 2 from iommu group. that was my mistake. im going to look for a way to set it to another group, thanks for all the help!

no the advanced one is for situations where the other stuff in the group is fine to be passed through, but you shouldnt bind it to vfio-pci or in situations where you have an intel ARC card which will share the vendor ID with intel chipsets and the intel cpu and anything else from intel (which causes the relative checks to be unreliable, so you have to supply it manually then) 😄


Just working on git, please ignore

This issue is created for the learning purpose, please ignore.

I am learning more about git and github and hence created #78


Streaming via PI WebAPI websocket

Does the plugin currently have the ability to stream data via WebAPI Websocket?  It would be very valuable to have a streaming connection to get real-time updates from PI server, and could be more timely, efficient, and easier on the server than polling.  This would fall into the feature request category if it is not currently supported.
Relevant documentation from Grafana:
https://grafana.com/docs/grafana/latest/developers/plugins/build-a-streaming-data-source-plugin/
Documentation on channels can be found in the PI WebAPI built-in documenation at /piwebapi/help/topics/channels.

I took a rough look at this (ee7dbda18fd4608ef36d51c6e5fee616b91929bd), and I think the improvements are are not significant issues. The biggest challenge is that it only makes sense with the Access parameter set to "Browser" at the moment. The "Server" option has no way to proxy the websocket connection using your grafana server, and the websocket needs to be opened directly from your browser.
Without trying to speak for the developer team, I think this function would make more sense if the plugin was updated with a backend server component.

Actually, I was wrong. You can pass the websocket traffic through the proxy like this: 5c275723d63689d95a62a467b188c43724103cf6.

Thank you so much for looking into this so quickly.  Sounds like maybe there's a way forward on this?
If I can help in any way, please let me know.  I can check out your commit and see if I can find any issues.

One additional thing we would want to do besides connecting the websocket with the "includeIntitialValues" flag is to do a simple query first to fill the dashboard date range (say, for instance that the dashboard showed the last few hours to now).  Then, any additional data would come in on the websocket if the data coming in on the websocket falls into the dashboard range.


Issue-Added svg logos

Automated Review Comments
Issues in package-lock.json:

(3, 'Hardcoded string found')
(4, 'Hardcoded string found')
(6, 'Hardcoded string found')
(14, 'Hardcoded string found')
(15, 'Line exceeds 80 characters')
(15, 'Hardcoded string found')
(16, 'Line exceeds 80 characters')
(16, 'Hardcoded string found')
(17, 'Hardcoded string found')
(18, 'Hardcoded string found')
(23, 'Hardcoded string found')
(24, 'Hardcoded string found')
Issues in package.json:
(3, 'Hardcoded string found')
(4, 'Hardcoded string found')
(6, 'Hardcoded string found')
Issues in public/logo.svg:
(2, 'Hardcoded string found')
(6, 'Line exceeds 80 characters')
(7, 'Hardcoded string found')
(10, 'Hardcoded string found')
Issues in public/utim.svg:
(2, 'Hardcoded string found')
(6, 'Line exceeds 80 characters')
(7, 'Hardcoded string found')
(10, 'Hardcoded string found')
Issues in src/components/Activity.js:
(5, 'Line exceeds 80 characters')
(8, 'Line exceeds 80 characters')
(8, 'Hardcoded string found')
(12, 'Line exceeds 80 characters')
(12, 'Hardcoded string found')
(14, 'Line exceeds 80 characters')
(17, 'Line exceeds 80 characters')
(17, 'Hardcoded string found')
(21, 'Line exceeds 80 characters')
(21, 'Hardcoded string found')
(24, 'Line exceeds 80 characters')
(24, 'Hardcoded string found')
Issues in src/components/layout.js:
(13, 'Hardcoded string found')
Issues in src/pages/_app.js:
(16, 'Hardcoded string found')
Issues in src/pages/_document.js:
(3, 'Hardcoded string found')
(9, 'Line exceeds 80 characters')
Issues in src/pages/index.js:
(4, 'Line exceeds 80 characters')
(5, 'Hardcoded string found')
(21, 'Line exceeds 80 characters')
(21, 'Hardcoded string found')
Issues in tailwind.config.js:
(11, 'Hardcoded string found')

I got it.


[BUG]: "Database already exists" and websockets errors when reloading the page with your sample code

Version:
Current from CDN
Platform:
Browser
Describe the bug
I am getting the following errors when I reload the browser page running your sample browser code from: https://simpleaswater.com/intro-to-aviondb/
Numerous instances of:
WebSocket connection to 'ws://<IP_ADDRESS>:8081/p2p/QmSoLPppuBtQSGwKDZT2M73ULpjvfd3aZ6ha4oFGL1KrGM' failed: Error in connection establishment: net::ERR_CONNECTION_REFUSED
t.exports @ index.min.js:2
_connect @ index.min.js:2
dial @ index.min.js:2
dial @ index.min.js:2
dialAction @ index.min.js:2
(anonymous) @ index.min.js:2

WebSocket connection to 'ws://<IP_ADDRESS>:8081/p2p/QmSoLueR4xBeUbY9WZ9xGUUxunbKWcrNFTDAadQJmocnWm' failed: WebSocket is closed before the connection is established.
(anonymous) @ index.min.js:2
c.close @ index.min.js:2
o @ index.min.js:2
r @ index.min.js:2
(anonymous) @ index.min.js:2
run @ index.min.js:2
async function (async)
run @ index.min.js:2
_createPendingDial @ index.min.js:2
connectToPeer @ index.min.js:2
_maybeConnect @ index.min.js:2
_onDidStart @ index.min.js:2
start @ index.min.js:2
async function (async)
start @ index.min.js:2
(anonymous) @ index.min.js:2
async function (async)
(anonymous) @ index.min.js:2
(anonymous) @ index.min.js:2
create @ index.min.js:2
async function (async)
create @ index.min.js:2
runExample @ aviondb.html:10
(anonymous) @ aviondb.html:67

Followed by:
Uncaught (in promise) Error: Database '/orbitdb/zdpuAn2qsM6P9E3DKz1hL4pUEaJwGqUh7LEX4eCnzgwABoeZV/employees' already exists!
    at k.create (aviondb.min.js:25)
create @ aviondb.min.js:25
async function (async)
runExample @ aviondb.html:10
(anonymous) @ aviondb.html:67

It would seem to me that by using aviondb.initCollection("employees"); it should simply open the existing collection, not try to create a new one.
Also, it would seem to me that by using await aviondb.drop(); the collection would be removed along with the database.
What else is notable here, is if I delete the databases in IndexedDB upon page reload, the issue does not happen. But that causes me to wonder if it's because my ipfs node ID is changing in that scenario and therefore I no longer can access the previously created database and collection.

@vasa-develop Have you been able to confirm this as a bug and is there any status on it? If I'm misunderstanding something, please let me know.


Vertical time series plot is corrupted where same as horizontal works fine

Reported by steven c hankin on 10 Mar 2015 23:31 UTC
Something is screwy in the scaling of the horizontal axis when transposed.  Probably in PLOT_SET_UP, and probably not too hard to find.
define axis/units=days/t0=1-jan-2001/t=0:500:4 t4day
let myvar = IF L[GT 50 AND L[gt=t4day](gt=t4day]) LT 56 THEN 500
plot myvar          ! just fine
plot/trans myvar    ! corrupted

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/2243

Comment by ansley.b.manke on 11 Mar 2015 23:41 UTC
This is something that's crept in just recently. It works fine in the last official public release, v6.93.

Comment by steven.c.hankin on 20 Mar 2015 00:27 UTC
If the above script is changed to avoid missing values, the corruptions disappears.
let myvar = IF L[GT 50 AND L[gt=t4day](gt=t4day]) LT 56 THEN 500  ELSE 0

This makes it appear likely that the source of the problem may be a failure to check for missing values in the data when determining axis formatting.

Comment by ansley.b.manke on 20 Mar 2015 20:44 UTC
This is fixed in setax.F. There were extra lines of code left from working on changes to have two calendar-formatted plot axes on a plot. This corrupted the settings that then get used to plot the time axis.


C4GNV kickoff deck

10-minute max presentation on the mission of C4GNV and the people involved.
Specs

10-minute maximum
Mission of C4GNV
Code of Conduct
How to communicate with the core team
Upcoming events

Tasks

[x] create deck
[x] show to core team
[ ] revise deck
[x] upload to drive and link from this ticket

bit.do/NDoCH-deck

bit.do/NDoCH-deck

bit.do/gnv-NDoCH-deck

bit.do/gnv-NDoCH-deck

bit.do/gnv-NDoCH-deck

@ambercabrera Awesome stuff! I added some comments inline. I'm thinking we can work through a couple items together Tuesday morning, and then get the rest of the info from the core team during standup.

Sounds good!

@ambercabrera We made some small changes. Consider deck done!


Elevator blocks disappeared when logging on server [Bug]

Describe the bug
I got on my server today and noticed all my elevator blocks were just gone, I checked a backup in my world in singleplayer and they are still there
A clear and concise description of what the bug is.
To Reproduce
not sure
Steps to reproduce the behavior:

First step ...
Second step ...

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment

Minecraft: [e.g. 1.16.5]
Fabric Loader: [e.g. 0.11.1]
Fabric API: [e.g. 0.29.4+1.16]

Logs
Pastebin the log [e.g. logs/latest.log] and link it below:

Additional context
Add any other context about the problem here.

If necessary, you can re-open this.


cmd/.../genutil: fix typo in comment

Description of the change:
Fix typo in the generateWithHeaderFile comment.
Motivation for the change:
"arguemnt" isn't a valid English word.

/ok-to-test

Not sure why the Travis job failed. Maybe it's unstable?

@johananl Yeah, it's flaky. We can babysit it until it passes.

@johananl If you can fix the merge conflict, I can LGTM again and see if we can get the tests to pass this time around. Sorry about the flakes on a change that obviously won't break anything!

@joelanford done. Let's hope CI passes this time.

/retest


fix small typo in introduction

Just a small typo, so I didn't go through the process. If you'd still like me to, I can go through it.
Side question: would you prefer an issue instead for really small things like this? I figured I could fix it real quick.
======
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): I'm not sure.
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Thanks.  Issues are easier.  We're insisting on clear
copyright ownership declarations now.
There's always an issue for the next manual.  The updates
are in for 2.10, but the next one's here:
https://github.com/stan-dev/stan/issues/1868
Just add a comment.
We may still take another pass before 2.10 depending
on how long it takes.

Bob

On Apr 16, 2016, at 6:34 PM, Chris Anderson<EMAIL_ADDRESS>wrote:
Just a small typo, so I didn't go through the process. If you'd still like me to, I can go through it.
Side question: would you prefer an issue instead for really small things like this? I figured I could fix it real quick.
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): I'm not sure.
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:
• Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
• Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)
You can view, comment on, or merge this pull request online at:
https://github.com/stan-dev/stan/pull/1869
Commit Summary
• fix small typo in introduction
File Changes
• M src/docs/stan-reference/introduction.tex (2)
Patch Links:
• https://github.com/stan-dev/stan/pull/1869.patch
• https://github.com/stan-dev/stan/pull/1869.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

I'm closing this pull request.


Fix selection toolbar not showing on drag end

Fixes #119314
The regression was caused initially by this commit: f014c1e6dcfdfbebaf379757273c4033a5092fbc
Before
Before, ending the drag didn't show the toolbar, making it impossible to copy the text after adjusting the selection handles.
https://user-images.githubusercontent.com/1659532/220207752-dae81cf2-5ccb-4f13-b3c4-45c0e900f494.mov
After
Now ending the drag will show the toolbar again:
https://user-images.githubusercontent.com/1659532/220207635-92275b58-7e14-4b57-83da-b12f7e9ed9af.mov
Pre-launch Checklist

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

The test failure is something unrelated that I've seen on another PR as well.  Can you push a merge commit?

Would be good cherry pick candidate IMO

A new stable release will be cut soon with this fix in it, so I think that will work out and we won't need to cherry pick.

Looks like this was tagged in 3.9.0-18.0.pre.