Add Shopping Cart Button to the Header

📘 Description
A shopping cart button needs to be added to the header for quick access to the cart and easier product review before checkout.
🔢 Steps

Add the shopping cart button to the header, ensuring it aligns with the other elements.
Ensure the button is responsive and looks good.

✅ Acceptance Criteria

The button is correctly aligned with the other header elements, and its responsive design.
Use the icons from Lucide React.

🌎 References

Lucide React.

📜 Additional Notes

Blocked by: #54

⚠ Do not apply until the ODHack begins.
⚠ Read our guidelines before applying.

I am Nze Shalom, a full stack developer with strong foundation in modern web development frameworks and technologies. My expertise includes: Nuxt.js, Next.js and Tailwind CSS with a bit of knowledge in Cairo. I am Experienced in crafting responsive, visually appealing designs using Tailwind CSS, ensuring consistency and efficiency in the development process. I would love to contribute to this project.
My approach towards this issue after forking the repo and creating a new branch would be first to locate the header and analyse the code. After that I would create a button with the destination to the cart. I would then style the button (aligning it to the centre and changing the colour to match the other elements, its size more). I would then save it and make Sure its responsive across all screens before creating for a PR. ETA is 12hrs

hello! my name is Gladchux.
I have extensively used JavaScript and typescript to deliver quality and responsive interactive projects.
i have worked on Ecommerce sites, you can check out my GitHub: https://github.com/Gladchux14.
My plan to resolve this issue:
create a branch after cloning the repository.
Study the codebase to understand it.
locate the header component.
use icon from lucid react.
implement styling from the design.
ensure responsiveness for all screen sizes and alignment with other header elements .
raise an explicit PR and notify maintainers.
my ETA for this would be 20hrs.

Hello @danielcdz , I’d love to work on this issue as I have strong experience with front-end development.
My Approach:

I will add the shopping cart button to the header, ensuring it aligns with the other elements and follows the responsive design principles.
I'll use icons from Lucide React to keep the visual style consistent.

Please assign this to me, and I’ll deliver an efficient and secure PR within 12-24 hours.

Hi I am David, I am a frontend web developer that specializes in Typescript, Javascript and tailwind-css. I plan to approach this issue by adding the "Shopping Cart Button" to the header and making sure it is aligned and responsive. ETA 1 day


How to view qlog ?

Hi,
How can I view qlog files ? When downloading they are not human readable.
Thanks

javascript: https://github.com/commaai/log_reader_js
python: https://github.com/commaai/openpilot/blob/master/tools/lib/logreader.py

You can also just use Cabana


istioctl kube-inject does not find $KUBECONFIG: automated tests failure

This affects automated tests:
https://circleci.com/gh/istio/istio/26735?utm_campaign=vcs-integration-link&utm_medium=referral&utm_source=github-build-link
2018-02-06T04:55:28.759610Z	info	Running command /go/out/linux_amd64/release/istioctl kube-inject -f /tmp/istio.e2e.017813612/servicesToBeInjected.yaml115107966.yaml -o /tmp/istio.e2e.017813612/KubeInject358003653.yaml --hub istio --tag a71687a627661063e2f785bee258f33cb2c1a18a -n istio-system -i istio-system --meshConfigMapName=istio
2018-02-06T04:55:29.322488Z	info	Command output:
Error: kubernetes configuration file "$KUBECONFIG else $HOME/.kube/config" does not exist
, err: exit status 255
2018-02-06T04:55:29.322785Z	error	istioctl [/tmp/istio.e2e.017813612/servicesToBeInjected.yaml115107966.yaml /tmp/istio.e2e.017813612/KubeInject358003653.yaml istio a71687a627661063e2f785bee258f33cb2c1a18a istio-system istio-system] failed
2018-02-06T04:55:29.322933Z	error	KubeInject failed command failed: "Error: kubernetes configuration file "$KUBECONFIG else $HOME/.kube/config" does not exist\n" exit status 255

This should have been fixed by https://github.com/istio/istio/pull/3126. Are there any reported instances of this problem with PR that have been rebased on the latest from master?

This should be fixed by #3126. Closing for now. We can (re)open a new issue if the problem persists after rebase.


cmd/cue: cmd hangs waiting for input

What version of CUE are you using (cue version)?

$ cue version
cue version v0.0.0-20220729051708-0a46a1624353

       -compiler gc
     CGO_ENABLED 1
          GOARCH arm64
            GOOS linux
             vcs git
    vcs.revision 0a46a1624353da7327d356040a5612251e01659f
        vcs.time 2022-07-29T05:17:08Z
    vcs.modified false

Does this issue reproduce with the latest release?
Not @latest, but with @master.
What did you do?
Note: this example must be run interactively in a terminal, and not via cmd/testscript or similar:
-- x_tool.cue --
package p

import (
	"tool/exec"
)

command: printer: {
	first: exec.Run & {
		cmd: ["echo", "test"]
	}
}

Run:
cue cmd printer

What did you expect to see?
Output as follows, and cue cmd exiting with a zero exit code.
test

What did you see instead?
The following output is printed to the console:
test

but cue cmd never exits.
Bisected to 4432aa66244b1c704143327709741edfc803cc82.
cc @mvdan

Thanks for the report; I can reproduce. It seems like os/exec hangs on copying from stdin. Interestingly enough, entering a single newline is enough to unblock the command.

I narrowed it down to https://github.com/golang/go/issues/7990, a known limitation of os/exec. In short, when os/exec.Cmd.Stdin was a *os.File before - os.Stdin - the file handle was passed directly to the child process. Now that os/exec.Cmd.Stdin is a *bufio.Reader, we don't have a file descriptor, so an io.Copy happens in Go, via a separate goroutine.
https://cs.opensource.google/go/go/+/refs/tags/go1.18.4:src/os/exec/exec.go;l=254-274
This is usually fine, except that os/exec.Cmd.Wait now has to wait for the io.Copy goroutine to finish when the child process exits, and our copy goroutine is blocked on an os.Stdin.Read call - since the copying happens in Go code now. And the only way to unblock that read call is by letting the read syscall finish with some result: by either inserting a newline, or entering ^D as an EOF on stdin.
I honestly forgot that this is a limitation of os/exec when Stdin is not an *os.File. I think going with a *bufio.Reader was a mistake, because there's no way to avoid the regression that you've spotted in a nice way.
I think the only reasonable solution is to drop the *bufio.Reader entirely, and instead have cli.Ask read its input one byte at a time, just like fmt.Scan used to do before my previous commit. We would also drop the sync.Mutex on all read operations, because we would have to pass os.Stdin around directly to exec.Run. So we would revert to all the logic races that existed before, but at least we would avoid this interactive CLI regression. And that seems like the right approach, because the example you posted isn't wrong in any obvious way.
Does that sound good to you, Paul?

I also looked at two libc implementations (musl and uclibc) and they both implement readline and readdelim (equivalent to our usage of bufio.Reader.ReadString) by reading one byte at a time. So I think us doing the same is consistent with how most other C-like programs will read one line of input interactively.

Sounds good to me. Thanks for the analysis


Better unit promotion indication

The unit promotion UI is a little lackluster right now and easy to miss. I'd like to add some kind of animation to the unit panel that comes up whenever a unit is selected that's hard to miss. Maybe something like having the XP bar cycle through rainbow colors when ready to promote?

This makes more sense as a notification, will see about implementing it in #514


Add optimized_reduced_symmetric_tensor_product().

Implements an optimized version of the fully symmetric case of reduced_tensor_product().

The problem we have now I think is due to jmp not being updated https://github.com/deepmind/jmp/issues/34


Rich presence

The rich presence should say something

Will work on this

The is already linked to a pull request. I didn't assign Napkins to this because you haven't added them as a collaborator yet so I can't

@NapkinsXYZ can you make another branch and pull request for this? It doesn't look like the change in #19 stuck, not sure why

Alrighty, will do so later today

Scratch that, I see now it is merged into my my branch. This should be fine then. I'll link it when I open my pull request


XD-3430

LGTM, merging.
TODO:// to specify count property as constance in SPI (possibly in ModuleDeployer interface or a common DeploymentProperties class).

Merged fd440a765707e917d00f80d6fbde682fa9b6e267 along with the change to support wildcard prefix for deployment properties: 5779a094485f5e48ed2e158595909bc33e33e80d


Fix of hash_payload for Ledger signData calls.

It would be nice to get this merged, so dapps can update their libs and code to be able to handle Ledger's hashed payloads.

Hi @MarcelKlammer ! Could you give a link to get more context ?

There is CIP8:
https://github.com/cardano-foundation/CIPs/blob/master/CIP-0008/README.md
And Ledger's implementation, which hashes payloads with length > 99.
This small PR fixes the message signing lib, to properly set hashed: true, if the payload is indeed hashed.


Live plots do not automatically update on Codespaces

Looks like something weird is happening with our watchers on Codedspaces. Local dev version runs fine for me.
How to reproduce
I've tried both - GPU and non-GPU setup. Two repositories: Get Started and Hackathon.
I'm getting only the first dot for the first epoch and then only at the very end it updates everything:
https://user-images.githubusercontent.com/3659196/218223477-c201c726-d819-4e23-bef5-13e7486cbdb5.mov
https://user-images.githubusercontent.com/3659196/218223521-e099b00e-6e8f-4b2a-a043-61f5f35c2e20.mov
Environment
Extension version: 0.6.4
DVC Version:
DVC version: 2.44.0 (deb)
-------------------------
Platform: Python 3.10.8 on Linux-5.4.0-1101-azure-x86_64-with-glibc2.31
Subprojects:

Supports:
        azure (adlfs = 2023.1.0, knack = 0.10.1, azure-identity = 1.12.0),
        gdrive (pydrive2 = 1.15.0),
        gs (gcsfs = 2023.1.0),
        hdfs (fsspec = 2023.1.0, pyarrow = 11.0.0),
        http (aiohttp = 3.8.3, aiohttp-retry = 2.8.3),
        https (aiohttp = 3.8.3, aiohttp-retry = 2.8.3),
        oss (ossfs = 2021.8.0),
        s3 (s3fs = 2023.1.0, boto3 = 1.24.59),
        ssh (sshfs = 2023.1.0),
        webdav (webdav4 = 0.9.8),
        webdavs (webdav4 = 0.9.8),
        webhdfs (fsspec = 2023.1.0)
Cache types: hardlink, symlink
Cache directory: ext4 on /dev/loop0
Caches: local
Remotes: https
Workspace directory: ext4 on /dev/loop0
Repo: dvc, git

dvclive==2.0.2

Making it a p1 priority since it affects us demoing this to customers. @alex000kim @daavoo do you also see this behavior?

I can recreate the behaviour in example-get-started but not magnetic-tiles-defects.
Looks like there is at least one issue related to images:

I think it might be a timing issue between plots diff writing the file to the tmp directory and the message being sent to the webview/the webview trying to load the file. Will come back to this once I've got to a breakpoint in the Studio integration work.

@alex000kim @daavoo do you also see this behavior?

Yes, I saw this today in https://github.com/iterative/example-get-started-experiments
My workaround when demoing is to keep pressing the "Refresh" button.

0.6.8 didn't fix the issue. I'll keep investigating.

@alex000kim @shcheklein do either of you know the magic incantation to run example-get-started-experiments on M1? I am running into this error:
loc("mps_scatter_along_axis"("(mpsFileLoc): /AppleInternal/Library/BuildRoots/9e200cfa-7d96-11ed-886f-a23c4f261b56/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShadersGraph/mpsgraph/MetalPerformanceShadersGraph/Core/Files/MPSGraphUtilities.mm":350:0)): error: invalid input tensor shapes, indices shape and updates shape must be equal
/AppleInternal/Library/BuildRoots/9e200cfa-7d96-11ed-886f-a23c4f261b56/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShadersGraph/mpsgraph/MetalPerformanceShadersGraph/Core/Files/MPSGraphExecutable.mm:1641: failed assertion `Error: MLIR pass manager failed'
ERROR: failed to reproduce 'train': failed to run: python src/train.py, exited with -6

Have spent some time trying to find an answer but have been unable to make it work.

There seems to be an issue with watching heavily-nested folder structures on Codespaces. Below is a demo of the depth where events stop getting generated:
https://user-images.githubusercontent.com/37993418/219987038-f5ab8b90-5bac-410c-8957-fa7b70271257.mov
Looking at options now.

Not a viable option but partly confirms the above theory. In this demo I have patched the DVCLiveCallback to touch the main dvc.yaml in the after_epoch function:
https://user-images.githubusercontent.com/37993418/219994251-d76b6925-30d3-43ec-96fe-40ad8b8f4756.mov

Not a viable option but partly confirms the above theory. In this demo I have patched the DVCLiveCallback to touch the main dvc.yaml in the after_epoch function:

dvclive/metrics.json always gets updated on next_step

Thanks @daavoo as a patch we can rely on metrics.json being updated 🙏🏻.
I can think of a couple of cases when this won't work:

metrics.json is still too deep/events are still dropped. E.g DVCLive dir is set to model/results/stage/train.
Users not working with DVCLive.

With patch in place:
https://user-images.githubusercontent.com/37993418/220210459-3a50329e-fdec-4de0-beec-d36686b2fcb5.mov
https://user-images.githubusercontent.com/37993418/220210582-8075a619-3901-4900-a367-6ddb25ee84c6.mov

metrics.json is named something else.
Users not working with DVCLive.

These 2 are the same. DVCLive has metrics.json hardcoded as name, users can't change it

The second patch (#3321) watches all metrics files for updates (i.e all metrics files from the exp show workspace record). I think this is good enough to close this for now. We can reopen if (probably when) the issue reappears.

There seems to be an issue with watching heavily-nested folder structures on Codespaces. Below is a demo of the depth where events stop getting generated:

FWIW I have checked to see if this was a Linux issue by creating the devcontainer from example-get-started-experiments with locally with Docker. Training performance was TERRIBLE but I could not recreate the issue. All events were created as expect.

Thanks @mattseddon for the update, it makes sense.
I'm curious what is so special about Codespaces then ... I guess we would need to ask them. It's fine to this in background as we discussed.

Raised https://github.com/microsoft/vscode-discussions/discussions/486, will see if it goes anywhere.

Raised https://github.com/community/community/discussions/48658 as well.

For the record here. Got a response in community/community:

Now raised https://github.com/microsoft/vscode/issues/176327


Workload watcher continues to fetch k8s labels for pod that was removed

I'm seeing these extraneous log messages in a CI run which only changes datapath pieces:
$ grep -e 798125b1fb -e testclient-vf80h pod-kube-system-cilium-lxqb9-cilium-agent.log | grep -v "level=debug" | tail -n 5
2018-08-09T09:21:58.565627555Z level=info msg="Removed endpoint" containerID=798125b1fb endpointID=53278 ipv4=<IP_ADDRESS> ipv6="f00d::a0a:100:0:d01e" k8sPodName=default/testclient-vf80h policyRevision=6
2018-08-09T09:22:17.44870582Z level=warning msg="Error while getting Kubernetes labels" error="pods \"testclient-vf80h\" not found" subsys=workload-watcher
2018-08-09T09:22:30.460108078Z level=warning msg="Error while getting Kubernetes labels" error="pods \"testclient-vf80h\" not found" subsys=workload-watcher
2018-08-09T09:22:44.47031701Z level=warning msg="Error while getting Kubernetes labels" error="pods \"testclient-vf80h\" not found" subsys=workload-watcher
2018-08-09T09:22:59.492500854Z level=warning msg="Error while getting Kubernetes labels" error="pods \"testclient-vf80h\" not found" subsys=workload-watcher

Seems like something should be signalling to the workload-watcher that it no longer needs to fetch labels for the endpoint which was removed.
I believe that it stopped attempting after the log messages above, so it could just be that there's a short window where the workload-watcher isn't yet notified where it will print out warning messages that are harmless.
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-Validated/5522/testReport/junit/k8s-1/7/K8sDemosTest_Tests_Star_Wars_Demo/
57949187_K8sDemosTest_Tests_Star_Wars_Demo.zip

This was fixed by #5116


generator: add no_omitemptyoption

This PR adds no_omitempty option for protoc-gen-go.
I would suggest that when I implement HTTP API using gRPC backend, and respond to a client as JSON, we'd keep Go's Zero Value field without omiting.
Particularly when a client uses TypeScript, might be omitted field have to define as nullable type. so I'd like to choose to keep that field is omitted or not by applying no_omitempty option in protoc-gen-go.
// with 'omitempty' as default, so this PR doesn't break current behavior
protoc .. --go_out=plugins=grpc:.

// without 'omitempty' tag
protoc ... --go_out=plugins,no_omitempty:.

But I think there's room for considering argument name, Thanks

@googlebot I signed it!


LSP missing tokens: if let enums

We are unable to get span information from Result and Err in the following example.
if let Result::Err(a) = result_a {
  6
} else {
  42
}

This seems to already be in the ParseProgram. Just a matter of bringing it forward one more step through type checking.


How to add Managed Identities into assigned users and roles?

Azure App Service Authentication is a great feature, and now with Managed Identities it makes it possible to Authenticate API Management service to call the App protected by the Azure Active Directory.
But, there seems to be no way to limit access using the Enterprise Application management panel to the App without blocking access from the Managed Identity aswell. You cannot add a Managed Identity from the Users and Roles panel. How can we do this?

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

ID: 163a1f88-c268-18aa-a1e6-89b4a11bfb01
Version Independent ID: 1136a613-48f8-fb67-39d3-d9abdd351a0f
Content: Assign a user or group to an enterprise app in Azure AD
Content Source: articles/active-directory/manage-apps/assign-user-or-group-access-portal.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @msmimart
Microsoft Alias: mimart

@Ozzian Thanks for your feedback! We will investigate and update as appropriate.

Found and excellent how-to guide for this in here: https://joonasw.net/view/calling-your-apis-with-aad-msi-using-app-permissions
While there are no UI allowing you to add MSIs there, you can use Azure Powershell for it (no, couldn't find az cli command for this either).
New-AzureADServiceAppRoleAssignment -ObjectId {MANAGED-IDENTITY-ID} -Id {YOUR-APPROLE-ID} -PrincipalId {MANAGED-IDENTITY-ID} -ResourceId {ENTERPRISE-APP-OBJECT-ID}

@Ozzian Good to hear that you are unblocked and thanks for sharing your solution for any future readers. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Chapter 8 - Multithreading

Great! 1 small remark. I would have maybe created a new class for this Clock thread. That way, all the list of timezones (or rather, buttons) could be placed there and thus making the ZooFrame class a bit cleaner.
Nevertheless, great job. You're the first one to finish all the assignments. Kudos. I hope you enjoyed learning Java and, well, hopefully you won't get bored next semester.

Thank you!
Since I am enjoying learning Java, definitely I will not get bored next semester :)


submitting new package: O'Rly, Typst?

I am submitting

[x] a new package
[ ] an update for a package

Description: A package for creating O'Rly-/O'Reilly-like cover pages.
I have read and followed the submission guidelines and, in particular, I

[x] selected a name that isn't the most obvious or canonical name for what the package does
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] tested my package locally on my system and it worked
[x] excluded PDFs or README images, if any, but not the LICENSE

NOTE: I goofed and accidentally changed the LICENSE for the repository rather than just my submission to it.  I changed it back, so hopefully that doesn't cause any issues.

Forgot to create a sub-folder with the version number.


Autocomplete not working

Given below a code snippet
class Singleton(type):
    """Single metaclass to make any class singleton in nature"""
    _instances = {}

    def __call__(cls, *args, **kwargs):
        singleton = kwargs.pop('singleton', True)
        if singleton:
            if cls not in cls._instances:
                cls._instances[cls] = super(Singleton, cls).__call__(*args, **kwargs)
            return cls._instances[cls]
        else:
            return super(Singleton, cls).__call__(*args, **kwargs)

class A(metaclass=Singleton):
   def test(self):
      print("test method")

a = A()
a.test() # Autocompletion not working here

This is because pyright (the static type checker upon which pylance is built) is unable to infer the return type of the metaclass __call__ method. You would need to add appropriate type annotations to the __call__ method if you want completion suggestions to work in this case. I can't tell from your minimal code sample what the appropriate type annotation would be, but here's an educated guess.
_T = TypeVar("_T")

class Singleton(type):
    ...
    def __call__(cls: type[_T], *args, **kwargs) -> _T: ...

Hi Eric class Singleton(type): is the actual snippet all other class just use this as a metaclass

Yes it's working thank you Eric and heejachang :)


to get back to home page there is NO option

Current Behavior
https://github.com/OSCode-Community/OSCodeCommunitySite/assets/114687502/ca22d304-cb4b-4a61-bdf8-bcd57067d438
Desired Behavior

i would like add the link so that we get back to home page
Screenshots / Mockups

Can you assign this issue to me?

hey @Vaishnavi-Patil2211 this can't be solved from my side as there is direct link of the website so i could not get the source and i have checked no source code is present for blog section.
here i attached a ss please refer this . Sry my side i can't resolve until i get source code.

@Vaishnavi-Patil2211


Expand package entrypoints utility - want to reuse

Hey! I am looking for some advice/guidance with regards to creating import maps from package entrypoints. I have tried @jspm/generator package. It works, generally speaking, but instead of choosing a single subpath, I want to build an import map that installs all possible subpaths of a package. I could loop over my exports entrypoints and do so, however, I have entrypoints with wildstar subpaths, so I'd have to expand those first.
I can write a util that does this, but I prefer using something that already exists and is more battle-tested.
I noticed that on JSPM CDN, my package's package.json "exports" map is expanded --> https://ga.jspm.io/npm<EMAIL_ADDRESS>versus another CDN like unpkg where it only has the original exports with wildstars in them (not expanded) --> https://unpkg.com/browse/<EMAIL_ADDRESS>I was wondering if someone could point me in the right direction, I'd love to reuse the logic that's used for JSPM CDN to expand package entrypoints (resolving those * chars in paths).
Basically going from:
{
  "exports": {
    "./*": {
      "types": "./dist-types/exports/*",
      "default": "./exports/*"
    },
    "./calendar-translations/*": "./components/calendar/translations/*"
  }
}

to:
{
  "exports": {
    "./accordion.d.ts.map": {
      "types": "./dist-types/exports/accordion.d.ts.map"
    },
    "./button.d.ts.map": {
      "types": "./dist-types/exports/button.d.ts.map"
    },
    "./calendar-test-helpers.d.ts.map": {
      "types": "./dist-types/exports/calendar-test-helpers.d.ts.map"
    },
    "./accordion.js": {
      "default": "./exports/accordion.js"
    },
    "./button.js": {
      "default": "./exports/button.js"
    },
    "./calendar-test-helpers.js": {
      "default": "./exports/calendar-test-helpers.js"
    },
    "./calendar-translations/en-GB.js": "./components/calendar/translations/en-GB.js",
    "./calendar-translations/nl-NL.js": "./components/calendar/translations/nl-NL.js"
  }
}

This expansion is done by the CDN itself not the import map generation tooling. We did have an option to generate the "full map" in earlier versions of the generator but this is not currently exposed functionality since installing all subpaths quickly generated a very large map. Instead we treat the map as a specific traced subset always and encourage tracing the app each time through traceInstall. This is especially important since the import map is blocking on the main critical load path of the page.
We could certainly reenable full map generation though.


Importer: state param ignored when running param given

Trying to switch from state params to running param as suggested in https://github.com/Sp2000/colplus-frontend/issues/280
But this currently breaks filtering on states:
https://api-dev.col.plus/importer?limit=25&offset=0&running=false&state=canceled

yes, running is just a shortcut for the entire set of (not=false) running states which gets unified with the states given individually. If you need to filter states you need to avoid the running parameter


Jobs Overhaul:  Capture stdout/stderr, standard logging, and/or structlog entries

As ...
Patti - Platform Admin
I want ...
The ability to view all of the logs for a lob, not just the ones that are shown in the logging facility (e.g.  self.log_success() )
So that ...
so that I can see full stacktraces and similar issues that I may not otherwise have easy access to.
I know this is done when...
There is a UI and API that provides feedback that is the same as I would see on the console of a worker in debug mode.
Optional - Feature groups this request pertains to.

[ ] Automation
[ ] Circuits
[ ] DCIM
[ ] IPAM
[ ] Misc (including Data Sources)
[ ] Organization
[ ] Plugins (and other Extensibility)
[ ] Security (Secrets, etc)
[ ] Image Management
[X] UI/UX
[ ] Documentation
[ ] Other (not directly a platform feature)

Database Changes
unknown
External Dependencies
No response

I'd love to get rid of the bespoke logging API for Jobs and just let them use stdlib logging in all cases.

Seems logical. My only request is a convention to still have the same thing. E.g. If you log your system like:
LOGGER.debug("NAUTOBOT_OBJ: <uuid>  NAUTOBOT_DEBUG: Serial number data validation failed on the given object.")

You will get the same view that you currently have.

In the interest of clarity, I am all for using structlog or similar to provide the same functionality, and it becomes an "opt-in" type thing.

Need to consider how this intersects with https://github.com/nautobot/nautobot/issues/765#issuecomment-1272388605 - the request there as I understand it would be to also have an option for jobs to not capture logs (or at least, not to store them to the database).

Don't know if it's already covered in some of the related issues, but it would be great to offer, off the shelf, a logging level option, matching the log level messages.
This way, if as a user, I want to get only > WARNING message, I could select the WARNING level.
Today, every Job reinvents the wheel for DEBUG messages, with a boolean input and multiple if statements.
Supporting the level filtering by default would simplify logging management.

Worth considering: will #1622 be able to address this with standard available tools (see: https://docs.celeryq.dev/en/stable/userguide/tasks.html#task-logging)
Let's discuss a spike after more of #1622 is completed.

Instead of bespoke logging levels, let's align around the standard logging levels (see: https://docs.python.org/3/library/logging.html#levels, we don't currently support critical and success does not exist)
Using get_task_logger instead of self.log_success etc to accessing the logging utility

Needs interception to go to database: https://docs.celeryq.dev/en/stable/userguide/signals.html?highlight=logging#after-setup-task-logger

stdout/stderr are redirected by default: https://docs.celeryq.dev/en/stable/userguide/configuration.html#std-setting-worker_redirect_stdouts so may "just work" but shouldn't be over prioritized
Support structlog

Possibly use Django-Structlog
Possibly move to log entries message in DB from being strings to a JSON Serialize

https://github.com/nautobot/nautobot/blob/develop/nautobot/extras/models/jobs.py#L500-L508 becomes just a single payload (except created)

Don't know if it's already covered in some of the related issues, but it would be great to offer, off the shelf, a logging level option, matching the log level messages.
This way, if as a user, I want to get only > WARNING message, I could select the WARNING level.
Today, every Job reinvents the wheel for DEBUG messages, with a boolean input and multiple if statements.
Supporting the level filtering by default would simplify logging management.

This will be addressed in: https://github.com/nautobot/nautobot/issues/2071

If we don't achieve the stretch goal do we need to make the JobLogEntry.log_object field a JSONField for job authors that need to log debug messaging with more complex data structures?

Yes, or more generally we need to solve the problem of the existing non-message fields on JobLogEntry and figure out how to accommodate those in the updated logging API we use.

The active_test job property should go away as part of this issue as well and potentially use the JobResult.status if that's something that still needs to be tracked in the job's logs.

Closed with #3799

Have spoke with Nautobot core since this was initially closed, but the core of this issue has not addressed as of yet, which is:

There is a UI and API that provides feedback that is the same as I would see on the console of a worker in debug mode.

I understand there has been adjustments made to the logging facility, but the challenge is always the same, we do not know own the logging for non-Nautobot packages, so if Python requests has an error, we do not control that. The only constant I have found is what has been mentioned in the request of the issue, which is the  console of a worker logs (at the time it said debug mode as that was a thing in Nautobot 1.x).


Added the flaky pylint plugin, so that we can mark a couple of tests …

…to be re-run if they fail

this is neat. do you know how many times it'll retry, or something to that effect?

That's configurable.  By default it'll retry once, which is enough for the few flaky tests that we have that sometimes fail because of numerical instability.


fix(client/cli): default to warn for env conflicts

Removes the error path for conflicting envs and defaults to warn.
Closes: https://github.com/prisma/prisma/issues/10958

In the issue the user mentions this

It seems like the opposite behaviour is desirable - an error should be shown when vars are different but duplicate names should be permitted if the values are the same.

In this PR it looks like the change is to remove the thrown error and only warn, which is different from what the used expected. Is that what we want to do?

Good observation, I wasn't sure if "error should be shown" was the same as "error should be thrown".  This PR does indeed remove the throwing of the error but keeps the showing of it. . I believe that the warning is sufficient, however, realise this could be seen as breaking...

In the issue the user mentions this

It seems like the opposite behaviour is desirable - an error should be shown when vars are different but duplicate names should be permitted if the values are the same.

In this PR it looks like the change is to remove the thrown error and only warn, which is different from what the used expected.

That was my understanding too, just flip the check, these changes look far outside the scope of that issue.

My bad I completely miss read this one, here I have flipped the check.

After flipping the switch, and waiting for the suite to run, it looks like we need to differentiate between the conflicts and duplicates, here I've done that!

Ping


Bug: Duplication. .. Linter | Spacings | Tabsize already defined in Obsidian | Settings | Options | Editor | tab size

[I love the plugin. Please don't take this as criticism.]

[X] I have verified that I am on the latest version of the Linter

Describe the Bug
Obsidian collects tabs-to-spaces figure which shouldn't be incongruous to the linters spaces-to-tabs figure. Linter provides no default, which should be collected from the system tabs-to-spaces figure.
How to Reproduce
Steps to reproduce the behaviour:

Go to Obsidian | Settings | Options | Editor | tab size ("Number of spaces a tab is equivalent to")
Observe slider indicating figure.
Go to  Third-party plugin | Linter | Spacings | Tabsize ("Number of spaces that will be converted to a tab")
Observe no default.
Observe that this figure is unneeded and is already provided in the host application.  It doesn't make sense that the tabs-to-spaces and spaces-to-tabs would be different.

Hey @businessBoris . Thank you for the feedback. I am not 100% sure how to best handle this scenario. I could default the value to be whatever is present in the Obsidian setting, but my biggest concern is that once it is set it would not reflect the Obsidian setting anymore if it changes. Would that be the expected behavior? Or do you think there is a better way to handle that?

Hi @pjkaufman!
Can you think of a situation where the tabs-to-spaces conversion is x and the spaces-to-tabs conversion should be y, where (x != y)?
May I suggest that you:

Remove the box completely and refer the user to the main application conversion setting, in some form:  eg."Conversion back to tabs uses the same ratio as the editor tabs-to-spaces.".
Use the main application conversion setting when you convert back to tabs, being careful to act according to the switch Use tabs switch in Obsidian's editor settings.

To be clear, converting multiple spaces back to tabs if the Editor | Behaviour | Use tabs switch is off is the right linting thing to do. I don't know what you'd do if the Use tabs switch is on, and this linter switch is on too. Presumably the user will forever be wondering why their carefully inserted spaces always become tabs.
I think you can probably see the fun to be had!


ERROR: {"code":"UNABLE_TO_GET_ISSUER_CERT_LOCALLY"} even after cert import locally done

Index.js content:
var ActiveDirectory = require('activedirectory');
var config = { url: 'ldaps://example.com:636',
               baseDN: 'OU=Service Accounts,OU=XXX,OU=XXX,DC=xXX,DC=XXXX,DC=XXX',
               username: 'abcdefff',
               password: 'BBBBYYY'

                }
var ad = new ActiveDirectory(config);
var username =<EMAIL_ADDRESS>
var password = 'secretpass';

ad.authenticate(username, password, function(err, auth) {
  if (err) {
    console.log('ERROR: '+JSON.stringify(err));
    return;
  }

  if (auth) {
    console.log('Authenticated!');
  }
  else {
    console.log('Authentication failed!');
  }
});

i have tried it locally using ldapsearch CLI and it works fine.

Are you running on a Linux, Mac or Windows host? If running on Windows, see the following related blog post I created:  http://blog.heeresonline.com/2014/05/nodejs-cert_untrusted-error/
The fact that ldapsearch works isn't really relevant. The ldapsearch cli can be configured via the ldap.conf file to ignore certificate problems / errors.

I am running on Mac . I did check this blog prior to opening this issue and followed the steps to add the cert and even after cert addition ran into same issues.
here is a copy of my ldap.conf file

LDAP Defaults

See ldap.conf(5) for details
This file should be world readable but not world writable.
#BASE	dc=example,dc=com
#URI	ldap://ldap.example.com ldap://ldap-master.example.com:666
#SIZELIMIT	12
#TIMELIMIT	15
#DEREF		never
TLS_REQCERT	demand
is there a way to define or provide cert file within my defined script scope to eliminate system/OS dependencies especially around certs.

here are steps followed to add cert:
openssl s_client -showcerts -host  example.com -port 636 > certs.crt
sudo security add-trusted-cert -d -r trustRoot -k /Library/Keychains/System.keychain certs.crt
i have ran the script both as local user as well as root to address any user profile issues

The activedirectory module is just a wrapper around the ldapjs library to support Microsoft specific extensions. So you can pass custom configuration and settings for ldapjs through the constructor / config object.
When creating your ad object, try the following:
var config = {
  url: 'ldaps://example.com:636',
  baseDN: 'OU=Service Accounts,OU=XXX,OU=XXX,DC=xXX,DC=XXXX,DC=XXX',
  username: 'abcdefff',
  password: 'BBBBYYY',
  tlsOptions: {
    rejectUnauthorized: false
  }
};

For more information about the tlsOptions, see the node.js TLS documentation. In the node.js documentation, they provide an example on providing a certificate authority.
Obviously, you won't want to leave that enabled and you'll want to specify a certificate chain, but you should be able to verify if things are working.

this works ..with only CN in username but not email in username field...this might be a challenge as most ppl may not even know their CN ..i tried finduser with email in principal ( might be wrong )..with following snippet and obviously it errored out.
ad.findUser(userPrincipalName, function(err, user) {
if (err) {
console.log('ERROR: ' +JSON.stringify(err));
return;
}
if (! user) console.log('User: ' + userPrincipalName + ' not found.');
else console.log(JSON.stringify(user));
});

The findUser method internally will build the query string as follows:
/**
 * Gets the ActiveDirectory LDAP query string for a user search.
 *
 * @private
 * @param {String} username The samAccountName or userPrincipalName (email) of the user.
 * @returns {String}
 */
function getUserQueryFilter(username) {
  log.trace('getUserQueryFilter(%s)', username);
  var self = this;

  if (! username) return('(objectCategory=User)');
  if (isDistinguishedName.call(self, username)) {
    return('(&(objectCategory=User)(distinguishedName='+parseDistinguishedName(username)+'))');
  }

  return('(&(objectCategory=User)(|(sAMAccountName='+username+')(userPrincipalName='+username+')))');
}

It will take either a DN, userPrincipalName or sAMAccount Name.

When you make the call, what is the value of userPrincipalName that you are attempting to query? If you are attempting to build an LDAP search filter (i.e. userPrincipalName=joe@domain.com), then it won't work.

If you prefer to do that manually, you can use the find() method which expects an ldap search filter to be specified.

seems to be a disconnect in our viewpoints..let me try to put it differently..
username in ur authenticate(username, password, callback) - as per my understanding is CN and it worked that way for me
UPN(userprincipalname which usually is email with domain in it like<EMAIL_ADDRESS>which in my case is my enterprise mail id (..i might be wrong) but i am just referring to http://www.morgantechspace.com/2014/04/samaccountname-vs-userprincipalname.html
sAMAccountName is just loginid or networklogin ... I am not a windows expert and apologize for confusion..but just trying to understand if there is a way to connect CN which works for authenticate with my enterprisemail id or loginid as i have tried various combinations of
find & finduser and i am getting undefined error.
find works with ldapsearchfilter  for "abc" or wild-search but if i provide my email id in search string it errors out.
all I am trying to do is to use my email ID instead of CN/first&lastname to authenticate users .

was able to resolve it thanks

I am facing the same eror: {"code":"UNABLE_TO_GET_ISSUER_CERT_LOCALLY"} after importing all certificates. Please help me! thanks in advance


Ignore expired ca tls cert

Adding this line guarantees that expired CA certificates wont prevent orderer initialization as happened in https://github.com/hyperledger/fabric/issues/3238
Type of change
Bug fix
Description
Adding this line guarantees that expired CA certificates wont prevent orderer initialization as happened in https://github.com/hyperledger/fabric/issues/3238
Related issues
https://github.com/hyperledger/fabric/issues/3238

Closing to fix issue with extra commits


PWA support

https://github.com/vite-pwa/vitepress

https://chodocs.cn/program/vitepress-plugin/#文档支持-pwa


Memory

After review - looks like I should rewrite CopySBinary as well - further simplify things

Fixes #184


Change jQuery URL from "http://" to "https://" to avoid broken SSL connections.

Thanks - committed this patch to trunk.


Stackoverflow documentation needs to be updated to reflect that it's been retired.

Stackoverflow documentation shutdown
The page Stack Overflow Documentation for Microsoft Developers needs to be updated to reflect that Stackoverflow documentation has been retired. Maybe a link to the unmaintained archive would be ok, but it's only available as a dump that you have to setup on a server yourself so it's hardly useful to a leisurely browser.

@dend can you update the team blog to talk about that SO docs were shut down?

Issue moved to https://github.com/MicrosoftDocs/feedback/issues/218


[BUG]: References don't work with planetscale

What version of drizzle-orm are you using?
0.26.5
What version of drizzle-kit are you using?
0.18.1
Describe the Bug
Any schema with references fails with foreign key constraints are not allowed
For example, using the example mysql schema in the docs
import { int, mysqlEnum, mysqlTable, serial, uniqueIndex, varchar } from 'drizzle-orm/mysql-core';

// declaring enum in database
export const countries = mysqlTable('countries', {
  id: serial('id').primaryKey(),
  name: varchar('name', { length: 256 }),
}, (countries) => ({
  nameIndex: uniqueIndex('name_idx').on(countries.name),
}));

export const cities = mysqlTable('cities', {
  id: serial('id').primaryKey(),
  name: varchar('name', { length: 256 }),
  countryId: int('country_id').references(() => countries.id),
  popularity: mysqlEnum('popularity', ['unknown', 'known', 'popular']),
});

Expected behavior
Either this should work or a workaround should be detailed in the planetscale docs. Prisma references with planetscale work when relationMode = "prisma"
Environment & setup
No response

@rawsh I run into the same issue. The trick is to define your schema without relations in on file (schema.ts) and the relations in another file (relations.ts).
This way you can tell drizzle-kit in you drizzle-config to push your schema using only the schema.ts file so it wont try to add foreign key constraints.
import type { Config } from "drizzle-kit";
import * as dotenv from "dotenv";
dotenv.config();

export default {
  schema: "./src/database/schema/schema.ts",
  out: "./src/database/migrations",
  connectionString: process.env.DATABASE_URL,
  breakpoints: true
} satisfies Config;

Still you will be able to you use relational queries if you initialize your drizzle client with both the schema and the relations.
import { drizzle } from "drizzle-orm/planetscale-serverless";
import { connect } from "@planetscale/database";
import * as schema from "./schema/schema";
import * as relations from "./schema/relations";

export const db = drizzle(
  connect({
    host: process.env["DATABASE_HOST"],
    username: process.env["DATABASE_USERNAME"],
    password: process.env["DATABASE_PASSWORD"]
  }),
  {
    schema: {
      ...schema,
      ...relations
    }
  }
);

PlanetScale doesn't support FOREIGN KEY constraints.
https://planetscale.com/docs/learn/operating-without-foreign-key-constraints

@rawsh I ran into the same issue. The trick is to define your schema without relations in on file (schema.ts) and the relations in another file (relations.ts).
This way you can tell drizzle-kit in you drizzle-config to push your schema using only the schema.ts file so it wont try to add foreign key constraints.
import type { Config } from "drizzle-kit";
import * as dotenv from "dotenv";
dotenv.config();

export default {
  schema: "./src/database/schema/schema.ts",
  out: "./src/database/migrations",
  connectionString: process.env.DATABASE_URL,
  breakpoints: true
} satisfies Config;

Still you will be able to you use relational queries if you initialize your drizzle client with both the schema and the relations.
import { drizzle } from "drizzle-orm/planetscale-serverless";
import { connect } from "@planetscale/database";
import * as schema from "./schema/schema";
import * as relations from "./schema/relations";

export const db = drizzle(
  connect({
    host: process.env["DATABASE_HOST"],
    username: process.env["DATABASE_USERNAME"],
    password: process.env["DATABASE_PASSWORD"]
  }),
  {
    schema: {
      ...schema,
      ...relations
    }
  }
);

Is this solution usable for mid-sized projects or will I run into problems?

Can we please get an example of you schema and relations file.


[New Snap] walletchat.fun

A Snap needs to satisfy all items in the list below, with 1 approval from someone on the MetaMask Snaps team.

[x] Is the Snap repository publicly accessible? https://github.com/Wallet-Chat/walletchat-metamask-snap
[x] Is the Snap distributed on npm? https://www.npmjs.com/package/walletchat-metamask-snap
[ ] Has an approved third-party audit been performed and the original audit report attached to this issue?
[ ] Has a complete list of discovered vulnerabilities from the audit report presented in this issue?
[ ] For vulnerabilities that have been deemed necessary to be addressed, are the links to the fixes attached to this issue?
[ ] For vulnerabilities that have been deemed not necessary to be addresses, is there reason to each of them documented in this issue?

Audit performed by Cure53
WalletChat-WAC-01-report.pdf


Current project status

Hi, what is the current status of this project? Is it still actively maintained?

I personally think typscriptlang.org should mention this still for older projects but it shouldn't need to be actively maintained anymore (obviously isn't touched often).
#253 should be front and center on the docs. Also would be nice if the website was maintained through GitHub so we can make direct PRs unless it is and I'm missing it?


Cannot read property 'current' of undefined

In the view, "Cannot find imported React library"
In the console:
Uncaught TypeError: Cannot read property 'current' of undefined
    at traverse16 (installHook.js:202)
    at window.addEventListener (installHook.js:188)

Hi @timscott, I think this is fixed in the newest build and should be on the chrome store within a few hours


Fix Python 3 issues

I propose a few changes to make f3 compatible with Python 3:

change xrange into range;
use relative imports inside the package (import files => from . import files);
ensure indices are integer.

Finally, I propose adding kplr to install_requires in setup.py, to help users hit the ground running; setup.py will automatically install kplr using pip.
(I was also going to remove the pyfits dependency but speedy @mirca beat me to it!)

I like it.
Only issue with kplr is I believe the version you need hasn't been uploaded to pypi. We should verify/prod Dan if this is still true.

I removed kplr from install_requires in setup.py!


github: send docker images to artifactory

Due security reasons, we had to stop using the dispatch token and
start using the GitHub App in order to trigger the deployment in
argo.  Because argo is a private repository, we can't trigger from
a public one (juno), so then we start to change the approach to first
push the docker images to jFrog Artifactory, then argo will be notified
that a new image was pushed, then it will trigger the deployment
Right now the argo trigger is not working, but will be worked on next

Test done in https://github.com/NethermindEth/juno/actions/runs/9665457154/job/26662572310

Latest commit test is being executed in https://github.com/NethermindEth/juno/actions/runs/9680031290

This PR is now ready for review

@kirugan why did you push the commits?

@derrix060 I rebased the branch on top of main


Must take newer Dungeon screenshots

Take new screenshots from Dungeon.

This will be done now. I am on a Windows machine, which seems to make Swing apps neater.


PLDM: Fix code update Rename

This commit is to fix the bootside rename
defect:  SW544814 in the out of band code
update path.
Signed-off-by: Sagar Srinivas<EMAIL_ADDRESS>

Still need @sampmisr and @manojkiraneda to take a look but in the interest of time, merge now so we can get through CI.


update Next.js to v11

v11 of Next.js has been released introducing amongst which some performance improvements
See: https://nextjs.org/blog/next-11
See: https://nextjs.org/docs/upgrading
Our current version of TypeScript is below the recommended minimum for v11 of Next.js
Updating that has some complications. See: #377

https://github.com/vercel/next.js/issues/26348


feat(bridge): Remove tag input field for creating a sequence

This PR

Removes input tag
Change validation
Adapt unit test
Adapt integration test
Add form util function and unit test for it

Related Issues
Resolves #7704
How to test
Ensure that the environment variable KEPTN_INSTALLATION_TYPE contains CONTINUOUS_DELIVERY
Steps:

Trigger sequence
Choose delivery
Image fields is now full size and tag is removed

Thank you @heinzburgstaller for your contribution 🙌 Could you please attach a screenshot how the UI looks like to the PR description?

Thank you @heinzburgstaller for your contribution raised_hands Could you please attach a screenshot how the UI looks like to the PR description?

Done


Ask the bare metal how to use ACL

Hello, I hope to use ACL to accelerate the model on cortex-a7 running rtos. I have seen some examples in the code repository. Can these examples run in a bare metal environment? Or which examples are recommended to run as tests?

Hi @Derekduke
You can build ACL for bare metal alognside with all the examples. A sample build command is the following:
scons  arch=arm64-v8a os=bare_metal build_dir=bm -j52 Werror=1 opencl=0 neon=1 debug=0 asserts=1 standalone=1 embed_kernels=1 benchmark_tests=0 validation_tests=1 examples=1 openmp=0 cppthreads=0 logging=0
The above will build the validation tests and examples. If you need some special bootcode to be included in your final binary, create a folder called bootcode/, put your asm files there and scons will automatically pick them up.

In a bare metal environment, now I have a tflite model file or model file of other frameworks, how can I use ACL to complete the operation on cortex-a?

Hi @Derekduke
You need an OS support for running tflite models, these require many system calls to parse the model.
I'd advice you try running the model with the ExecuteNetwork tool from https://github.com/ARM-software/armnn/tree/branches/armnn_21_02/tests/ExecuteNetwork
Hope this helps.


docs: update modulepreload docs with latest browser support

caniuse says it's available on Safari 17 with "Technology Preview", is that on by default?

caniuse says it's available on Safari 17 with "Technology Preview", is that on by default?

It's on by default, but Safari 17 is not released yet and is in "technology preview" (i.e. beta)


Add 'spo list view remove' command capability

Summary
Add a support for spo list view remove command in order to :

[ ] Remove specific SharePoint view based on his ID or Title (ID or Title not both)
[ ] Here the list of necessary parameters :

[ ] --listId / --listTitle (ID or Title not both)
[ ] --viewId / --viewTitle  (ID or Title not both)
[ ] -u, --webUrl Web URL
[ ] --confirm (to suppress prompt)

[ ] Support at least the following additional parameters :

[ ]  -o, --output
[ ]  --verbose
[ ]  --debug

To Do

[ ] Add command to solution
[ ] Write documentation of usage (MkDocs)

Thanks for the suggestion @Laul0. In order to remove a list view you would also need the URL of the site where the list is located and the list ID/Name. See https://github.com/pnp/office365-cli/issues/269 for a comparable spec.

Yes, it was implicit for me... The web URL cannot be omit of course, you right. I'll precise

The spec looks solid. You want to take it or want to give someone else a shot at it?

I'll take it, unless someone is faster than me ^^

Perfect! All yours!


class active on Todos filter not showing up

The view todos-container.component.html is giving an error on
[ngClass]="{ active: todos.filter === 'ACTIVE' }
Indeed todos.filter does not exist instead add  in todos-container.component.ts
filter$: Observable<TodosFilter>; ....   ngOnInit() {     this.todos$ = this.store.pipe(select(selectTodos));     this.filter$ = this.store.pipe(select(selectTodosFilter));     this.removeDoneDisabled$ = this.store.pipe(       select(selectRemoveDoneTodosDisabled)     );   }
And in the view
<button mat-menu-item (click)="onFilterTodos('ALL')" [class.active]="(filter$ | async) === 'ALL'" rtl>
The ngClass directive is removed because in the official documentation it's written

The NgClass directive can be used as an alternative to direct [class] bindings. However, using the above class binding syntax without NgClass is preferred because due to improvements in class binding in Angular, NgClass no longer provides significant value, and might eventually be removed in the future.

Hi @bhahn57570 !
Sounds good, would you be willing to submit a pull request to fix this?
Cheers!

Hi Sorry for the late response, I am quite busy these days.
Yes, I would like to submit a pull request, I will do it next week. Is it
ok for you?
I am just not familiar with GitHub, I just have to push on a specific
branch and after asking you for a pull request that it?
Thanks a lot for the project it helped me a lot to start with angular.
Cheers!
Le dim. 10 mai 2020 à 13:58, Tomas Trajan<EMAIL_ADDRESS>a
écrit :

Hi @bhahn57570 https://github.com/bhahn57570 !
Sounds good, would you be willing to submit a pull request to fix this?
Cheers!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tomastrajan/angular-ngrx-material-starter/issues/524#issuecomment-626317013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJKGDWIXBCOVM5TL6YYPHKLRQ2JIBANCNFSM4M3WRIXA
.

Hi @bhahn57570 !
No worries, there is enough time!
The way to do it on github is first to fork this repository into your account. Then you clone it, make a branch and commit a fix to your repository (project fork)
Once you do it, Github will allow you to open pull request based on your fork, you should see that button just by visiting this repository.
Hope that helps!
Cheers,
Tomas


add is_diagonal utility and Transform property

Description
This small PR was split off from #4303 as requested by @andy-sweet for ease of review. It adds an is_diagonal property to Transforms that makes Affine transforms more efficient in the case when they are known to be diagonal.
I chose to ignore any entries below a certain tolerance when determining if a matrix is diagonal. I think in practice it is desirable as things like applying a rotation followed by an opposite rotation on a diagonal Affine, should ideally return the original Affine again, but in practice will have tiny off diagonal elements (e.g. ~1e-15) due to finite floating point precision. This scenario is covered in one of the test cases here.
Type of change

[x] New feature (non-breaking change which adds functionality)
maybe more of a "performance optimization" than new feature as end users would typically not need to interact with this at all.

References
split off from #4303 as it is not strictly related to permutations. The caching used only for the is_diagonal property here also is used for a couple of other properties in that PR.
How has this been tested?

[x] existing tests pass
[x] new tests were added

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[ ] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

Comments have been addressed aside from deciding on a potential different name like _mixes_dimensions or similar.

Some of my comments are being rendered strangely / duplicated (at least on my end) - sorry about that...

I'll merge this in ~24h if no one weighs in before then. Thanks @grlee77 and @andy-sweet!


Update library to Contributors library standards

This PR updates purescript-matryoshka to adhere to the Contributors library guidelines.
This is part of the https://github.com/purescript-contrib/governance/issues/13 effort.

Verified that the build works and the "tests" run:
λ spago build
[info] Installation complete.
[info] Build succeeded.

λ spago test
[info] Installation complete.
[info] Build succeeded.
expr example
24
list example
24
4 : 3 : 2 : 1 : Nil
720
[info] Tests succeeded.

Once this PR gets merged I'll add issues for the following outstanding items:

Adding a "quick start" guide
Adding real tests (currently they're just runnable examples)
Adding module documentation

Once this PR gets merged I'll add issues for the following outstanding items:

Adding a "quick start" guide
Adding real tests (currently they're just runnable examples)
Adding module documentation

I went ahead and created these issues now, since I forgot to do it once https://github.com/purescript-contrib/purescript-arraybuffer-types/pull/16 was merged :man_facepalming:

Couple of minor comments, but otherwise 👍


Error while trying to create a cluster with AWS Asia Pacific (Hong Kong) Region 'ap-east-1'

What kind of request is this (question/bug/enhancement/feature request):
feature request
Steps to reproduce (least amount of steps as possible):
I am provisioning rke-cluster in ap-east-1 (the new aws-hk-region open in 25-Apr-2019)
resource "rancher2_cloud_credential" "foo" {
  name = "foo"
  description = "foo test"
  amazonec2_credential_config {
    access_key = "<AWS_ACCESS_KEY>"
    secret_key = "<AWS_SECRET_KEY>"
  }
}
resource "rancher2_node_template" "foo" {
  name = "foo"
  description = "foo test"
  cloud_credential_id = "${rancher2_cloud_credential.foo.id}"
  amazonec2_config {
    ami =  "<AMI_ID>"
    region = "ap-east-1"
    security_group = ["<AWS_SECURITY_GROUP>"]
    subnet_id = "<SUBNET_ID>"
    vpc_id = "<VPC_ID>"
    zone = "<ZONE>"
  }
}

Result:
Error setting machine configuration from flags provided: Invalid region specified; Timeout waiting for ssh key

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI):
Rancher 2.2.3
Installation option (single install/HA):
HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):
Machine type (cloud/VM/metal) and specifications (CPU/memory):
Kubernetes version (use kubectl version):

(paste the output here)

Docker version (use docker version):

(paste the output here)

@alphawelab This issue will be fixed and released in the next release 2.2.4.
Please note the current default instance type for AWS node template is t2.medium
But t2.medium is not available in HK region. Please select t3.medium manually when you create the template.
Thanks

Tested with rancher/rancher:v2.2.4-rc19 from May 30
Steps:

Create an AWS node template
Region is ap-east-1
Instance type is t3.medium
Create EC2 cluster for region ap-east-1
Modify the count of cluster nodes
Delete cluster

Result:
Node template create successfully.
Cluster create/modify/delete successfully.
Verified fixed.


Checkbox

In this project,

I added some features to my application to make it interactive.
The user will also be able to mark the completion of a task by checking the corresponding box (or cancel it by unchecking the box).
The updated task list will be stored in the local storage.

I deploy it on netlify because my github forces me to create another pull request that is not allowed in this project. but the god part of the project I will deploy it on github because it will contain a new pull request


Disable or remove specific code action/fixes/refactoring

Search Terms
Suggestion
https://github.com/microsoft/vscode/issues/92305
Because when some code action is available, there will be a icon shows on the code editing area.
If some fixes I always not want, it's annoying that the clickable icon will show every time my cursor move on the code or select some code or some other actions. Especially when the current line left is not white spaces, the icon will show at previous line and cover the privious line code.
For example, I always want use require in js file not import.
And maybe some visual effect like the dash underline, the show fixes button in the popup tip, the dimmed color text should be removed, when the relate code action is disabled. Because these hint there are problems in my code but actually not.
Use Cases
Examples
Checklist
My suggestion meets these guidelines:

[x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
[x] This wouldn't change the runtime behavior of existing JavaScript code
[x] This could be implemented without emitting different JS based on the types of the expressions
[x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
[x] This feature would agree with the rest of TypeScript's Design Goals.

@mjbvz is there any editor-side support for this?

We don't have a UI for managing suggestions/actions. For now, this would likely be provided using a setting that VS Code could pass to TS
I believe there are two parts to this request:

Disabling specific suggestion diagnostics #28825 This seems like the main point of this issue

Disabling specific fixes / refactorings. Specifically refactorings that do not require any selection. We should get more user feedback before investigating this

I agree with @heroboy. I want the lightbulb to be displayed, but it is always suggesting I change modifiers to final where possible.
I'm using the Language Support for Java(TM) by Red Hat extension.

like this
"editor.codeActionsOnSave": [
    "source.organizeImports",
    "source.fixAll"
]

what about:
"editor.codeActionsNotShow": [
    "source.organizeImports",
    "source.fixAll"
]

I would like an option to remove this:

Is there any progress with this feature? I would like to remove SOME but not all VSCode suggestions.

This would help avoid the auto deletion of unreachable code like occurs with Prettier + ESLint in VSCode, if could just disable that single code action, while still allowing other Prettier formatting to occur, like mentioned in https://github.com/microsoft/vscode/issues/109530.


401 when attempting to unfold hidden code in diffs

Gitea version (or commit ref): 1.12.4 built with GNU Make 4.2.1, go1.14.8 : bindata, sqlite, sqlite_unlock_notify
Git version: 2.24.3
Operating system: Ubuntu 18.04.5 / Docker 19.03.12, build 48a66213fe
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
Whenever we click the code folding icon, i.e. <svg class="svg octicon-fold" width="16" height="16" aria-hidden="true"><use xlink:href="#octicon-fold"></use></svg>, a standard HTTP auth login modal dialog appears.
When we cancel that dialog, the log shows 401 errors on resources like these: http://gitea.internal/org/repo/pulls/794/undefined?undefined&anchor=undefined
Screenshots
I would if I could, but I believe my description is very precise.

Outdated, the code has changed a lot.


AI Allocations

Not sure if this has been something that's been featured in other quests up until now, but I noticed it being an oversight with the latest Ordeal Call quests.
Basically: category of AI that is stored in a new location(?) in JSONs that applies itself to a specific individuality (so it'll work whether the servant in question is the NPC support or the player's own).
For example, in Section 12, Phase 6 of OC2, this snippet of code is hiding in the JSON:
        "aiAllocations": [
          {
            "aiIds": [
              400700
            ],
            "applySvtType": 5,
            "individuality": 1101500
          }

This allocates the Servant AI 400700 to the entity with the ID 1101500 -- which in this case is the Nitocris Alter in the player's party, whether it's the support NPC or not.
Currently the AADB has no method of supporting this type of AI connection, meaning that it doesn't show up unless viewing the JSON data of the original quest and that no related quests show up for the AI on its page.

Fixed with https://github.com/atlasacademy/apps/commit/10354198250b26d9650a7c0375698de660585ed3 and https://github.com/atlasacademy/fgo-game-data-api/commit/c259da9b39e0c3392a0fbfeb98ad31b93ee49eb5


Documentation updates for 4.2

Description
Javadoc and markdown documentation updates for the 4.2 release, along with two new tutorials: onnx export and reproducibility.
Motivation
New features need docs.

Congratulations 🎉🎉🎉🎉
On Sat, Dec 18, 2021, 5:50 AM Philip Ogren @.***> wrote:

@.**** commented on this pull request.
In tutorials/external-models-tribuo-v4.ipynb
https://github.com/oracle/tribuo/pull/205#discussion_r771777979:

@@ -133,10 +133,7 @@
}
],
"source": [

"var lrTrainer = new LinearSGDTrainer(new LogMulticlass(),new AdaGrad(0.1),3,30000,Trainer.DEFAULT_SEED);\n",
"var lrModel = lrTrainer.train(mnistTrain);\n",
"var lrEvaluation = labelFactory.getEvaluator().evaluate(lrModel,mnistTest);\n",
"lrEvaluation.toString();"

this looks like a fat finger. "b" is not code per se....
—
Reply to this email directly, view it on GitHub
https://github.com/oracle/tribuo/pull/205#pullrequestreview-835740493,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWGLSQ3AAIHSNXL7X26OQ43URQAJZANCNFSM5KJRYP2Q
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Use custom yahoo finance for async compatability

Trending: https://query1.finance.yahoo.com/v1/finance/trending/US?count=50
OHLCV: https://query1.finance.yahoo.com/v7/finance/download/{ticker}
All info: https://query1.finance.yahoo.com/v8/finance/chart/AAPL?region=US&lang=en-US&includePrePost=false&interval=2m&useYfid=true&range=1d&corsDomain=finance.yahoo.com&.tsrc=finance
Shorter all info: https://query1.finance.yahoo.com/v8/finance/chart/aapl

In case we need cookies:
https://stackoverflow.com/questions/76065035/yahoo-finance-v7-api-now-requiring-cookies-python

Maybe we can remove yfinance and yahoo-fin completely

yfinance is used in:

earnings.py
stock.py
confirm_stock.py

yahoo_fin:

gainers
losers
trending
db

Gainers: https://query2.finance.yahoo.com/v1/finance/screener/predefined/saved?formatted=false&lang=en-US&region=US&scrIds=day_gainers&count=5&corsDomain=finance.yahoo.com
Losers: https://query2.finance.yahoo.com/v1/finance/screener/predefined/saved?formatted=false&lang=en-US&region=US&scrIds=day_losers&count=5&corsDomain=finance.yahoo.com
Trending: https://query1.finance.yahoo.com/v1/finance/trending/US?count=50
Nasdaq tickers: ?
Earning dates for given stock: ?

No API endpoint for upcoming earning date, leaving it as it is for now


Update editor configuration only after extensions are registered

Editor configurations are getting updated very aggressively (atleast 20 times) while the extensions are getting registered. This is consuming atleast 200ms.

Insiders (0cc0904c56)
joh/pausedEvents 7907361
PR c4bd3f8

builtin extensions
185.6
71         (-114.6ms)
94.5 (-90.2ms)

builtin extensions and 30 more
448.8
148.2  (-300.6ms)
163.8 (-285ms)

Thanks to @jrieken for comparing this change with the current insiders and also with his generic solution.
Please see https://github.com/microsoft/vscode/pull/138259 for more information and perf numbers.
This changes editor configuration to update only after extensions are registered, therefore configuration is being updated only once by this.

Why not just registerWorkbenchContribution on the LifecyclePhase.Restored phase rather than waiting for extensions explicitly?

The Restored phase is reached before scanning extensions. The issue is that while registering extension contributions too many configuration changes are fired, esp. from custom editors and notebooks. This PR makes sure that editor-land triggers the event only once

Main intention of this change is to reduce number of times this editor configuration updates configuration. I do not think LifecyclePhase.Restored is what when after extensions are registered. This editor auto lock configuration is listening to editor registrations and updating config, so the right phase is when extensions are registered. I think we can introduce new LifecyclePhase for extensions registered but is not this what IExtensionService.extensionService.whenInstalledExtensionsRegistered is and the new phase is going to be duplicate right.

I think we can introduce new LifecyclePhase for extensions registered

👍

but is not this what IExtensionService.extensionService.whenInstalledExtensionsRegistered

Yeah it is the same, but it would not duplicate anything, rather make it super easy for workbench contributions to register to this phase. That was the main intent of having lifecycle phases actually. Of course everyone could await promises but the phases avoid a lot of boilerplate code.


PFfile - filekey - heroku+mlab

I have migrated on heroku, i have set my filekey into parse server, but form app is impossible retrieve pffile(jpg)... there is a bug?

My Faultt i use FILE_KEY insteas PARSE_FILE_KEY -_-


feat(prometheus): added dashboards for prometheus-remote-write

Summary
TODO fill me out please

Pre merge checklist

[ ] Did you check you NRQL syntax? - Does it work?
[ ] Did you check your dashboard image quality? -  Do they look good?
[ ] Did you check that your alerts actually work?
[ ] Did you include an InstallPlan and Documentation reference?
[ ] Did you check your descriptive content for voice, tone, spelling and grammar errors?
[ ] Did you attach images of your dashboards to the PR so we can see them working?

Screenshots
Attach images of any visual changes, such as a dashboard here.

 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.

@tclayton-newr Thanks for the contribution, could you sign the CLA? You just need to do this once to contribute to our New Relic OSS repositories.

@jpvajda thank you for the feedback! I've updated the branch a bit and addressed any comments.
I also ran into a small issue with the sanitizer that wasn't removing the linkedEntityGuids as I was expecting it should. I made a quick PR to address future dashboard.json like mine (there were some line returns).
https://github.com/newrelic/newrelic-quickstarts/pull/861


Delete .dockerignore

Checklist:

[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 main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hey @Ultimate8788
Thank you for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.

If you have any questions, feel free to ask questions on the 'Contributors' category on our forum or the contributors chat room.


operator precedence of AND/OR in where

I see in #20 that operator precedence has been improved/fixed. Nevertheless I still observe an issue with AND and OR.
The following two expressions result in the same AST, but shouldn't.
select x from foo where (a or b) and c
select x from foo where a or b and c

Tested with v1.0.0 and also the online demo

You're right, AND should compose before OR in the second statement. There is an initial fix in master but I will need some more time to test it out before I publish a new release on npm. Thanks!

Thank you. Fix works for me and doesn't break any of my existing tests. I will add a few more to see if there are any side effects.

Released in v1.0.1, thanks!


Align with Fractal 1.3

Examples 01 and 02 are now working with the latest Fractal versions

fractal-server 1.3.0a2
fractal 1.3.0a0

Example 01 ran correctly with fractal-tasks-core 0.10.0a4:
2023-06-15 15:37:18,455 - WF1_job1 - INFO - Start execution of workflow "Workflow cardiac-test-3"; more logs at /home/tommaso/Fractal/fractal-demos/examples/server/artifacts/proj_0000001_wf_0000001_job_0000001/workflow.log
2023-06-15 15:37:18,455 - WF1_job1 - DEBUG - fractal_server.__VERSION__: 1.3.0a5
2023-06-15 15:37:18,455 - WF1_job1 - DEBUG - FRACTAL_RUNNER_BACKEND: local
2023-06-15 15:37:18,455 - WF1_job1 - DEBUG - slurm_user: None
2023-06-15 15:37:18,455 - WF1_job1 - DEBUG - worker_init: None
2023-06-15 15:37:18,456 - WF1_job1 - DEBUG - input metadata: {}
2023-06-15 15:37:18,456 - WF1_job1 - DEBUG - input_paths: ['/home/tommaso/Fractal/fractal-demos/examples/01_cardio_tiny_dataset/../images/10.5281_zenodo.7059515/']
2023-06-15 15:37:18,456 - WF1_job1 - DEBUG - output_path: /home/tommaso/Fractal/fractal-demos/examples/01_cardio_tiny_dataset/output_cardiac-test-3
2023-06-15 15:37:18,458 - WF1_job1 - DEBUG - job.id: 1
2023-06-15 15:37:18,458 - WF1_job1 - DEBUG - job.working_dir: /home/tommaso/Fractal/fractal-demos/examples/server/artifacts/proj_0000001_wf_0000001_job_0000001
2023-06-15 15:37:18,458 - WF1_job1 - DEBUG - job.workflow_dir_user: /home/tommaso/Fractal/fractal-demos/examples/server/artifacts/proj_0000001_wf_0000001_job_0000001
2023-06-15 15:37:18,458 - WF1_job1 - DEBUG - START workflow "Workflow cardiac-test-3"
2023-06-15 15:37:18,460 - WF1_job1 - DEBUG - SUBMIT 0-th task (name="Create OME-Zarr structure")
2023-06-15 15:37:20,147 - WF1_job1 - DEBUG - END    0-th task (name="Create OME-Zarr structure")
2023-06-15 15:37:20,147 - WF1_job1 - DEBUG - SUBMIT 1-th task (name="Convert Yokogawa to OME-Zarr")
2023-06-15 15:37:22,244 - WF1_job1 - DEBUG - END    1-th task (name="Convert Yokogawa to OME-Zarr")
2023-06-15 15:37:22,244 - WF1_job1 - DEBUG - SUBMIT 2-th task (name="Copy OME-Zarr structure")
2023-06-15 15:37:23,664 - WF1_job1 - DEBUG - END    2-th task (name="Copy OME-Zarr structure")
2023-06-15 15:37:23,665 - WF1_job1 - DEBUG - SUBMIT 3-th task (name="Maximum Intensity Projection")
2023-06-15 15:37:25,229 - WF1_job1 - DEBUG - END    3-th task (name="Maximum Intensity Projection")
2023-06-15 15:37:25,229 - WF1_job1 - DEBUG - SUBMIT 4-th task (name="Cellpose Segmentation")
2023-06-15 15:37:35,533 - WF1_job1 - DEBUG - END    4-th task (name="Cellpose Segmentation")
2023-06-15 15:37:35,533 - WF1_job1 - DEBUG - SUBMIT 5-th task (name="Napari workflows wrapper")
2023-06-15 15:37:38,081 - WF1_job1 - DEBUG - END    5-th task (name="Napari workflows wrapper")
2023-06-15 15:37:38,083 - WF1_job1 - INFO - End execution of workflow "Workflow cardiac-test-3"; more logs at /home/tommaso/Fractal/fractal-demos/examples/server/artifacts/proj_0000001_wf_0000001_job_0000001/workflow.log
2023-06-15 15:37:38,083 - WF1_job1 - DEBUG - END workflow "Workflow cardiac-test-3"

Merging this one, so that we can move on with independent PRs.


Add CI check for forc compiler warnings

Type of change

Improvement (refactoring, restructuring repository, cleaning tech debt, ...)

Changes
The following changes have been made:

Added CI check that errors on forc complier warnings when building projects

Related Issues

Closes #18
Checklist

[x] I have linked to any relevant issues.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have updated the documentation where relevant.
[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.
[x] I have updated the changelog to reflect the changes on this PR.

To be merged after #17


feature: support postgres 13.9

https://aws.amazon.com/about-aws/whats-new/2023/05/amazon-aurora-serverless-v1-postgresql-13

I've confirmed this works in my project. Haven't measured anything, but based on the eye test, there's a decent perf improvement as well.

Thanks!


(libreoffice-fresh) v24.2 out for ~1.5 months, v7.6.x is now Still

Checklist

[X] I have checked the moderation queue that no new version has been submitted (only visible when logged in), or the submitted package version has failed the automated checks.

Please navigate to the following link to view the moderation queue.

[X] I have verified that the new version is a stable release.
[X] I have looked to see if there are any Bug reports that prevents a new version being submitted and that no existing Outdated Reports have been created.
[X] I have verified the new version has a Windows binary (EXE or MSI installer, Zip or other archive, etc).
[X] I have verified that this is the correct repository, and the package is maintained by the chocolatey-community user.

New Software Version
24.2.1
Download location
https://download.documentfoundation.org/libreoffice/stable/24.2.1/win/x86_64/LibreOffice_24.2.1_Win_x86-64.msi.mirrorlist
Package Page
https://community.chocolatey.org/packages/libreoffice-fresh

Libreoffice switched to a date-based versioning scheme (ish) and launched v24.2 sometime in late-January/early-February 2024. The first bug fix release, v24.2.1, was released on 29 February 2024.
It appears that v7.6 - which is currently the "Fresh" line on chocolatey - is now considered Stable/Still.

See the note in the package description:

The libreoffice-still package has been updated to the v6.7.x minor version, as the update service has made it available to those on the still channel/stream. However, I just checked my Libreoffice Fresh installation, and it looks like they have not made the v24.x.y major version available via the update service yet. As it is not available yet via the update service, the package is not out of date, so this issue is not valid for the moment.
This issue could be reopened if the status changes in the future and the package does not get automatically updated.

@TheCakeIsNaOH I did notice the disclaimer about the update delay but I figured 1.5 months seemed long enough for the new version to hit the updater.... apparently not :)

@TheCakeIsNaOH Libreoffice Fresh and Still are now the same version. I unfortunately have no insight into the LibreOffice updater but is it possible they're not offering a direct upgrade path from v7 -> v24?

I unfortunately have no insight into the LibreOffice updater but is it possible they're not offering a direct upgrade path from v7 -> v24?

That is something you would have to ask The Document Foundation about (developers of LibreOffice).
To my understanding, the current package update mechanism was created at their request, as they desired the Chocolatey package not be updated until the version was available on the update api.


better AbstractFloat implimentation

at least for base 2 floating point numbers, you can just get the exponent.

Base.exponent has been available for this since Julia 1.0.
exponent also works for Int since Julia 1.7, but it converts the argument to a float.


V3 breaks "go install" and "go run"

Up to V2, it was very handy to be able to run, e.g., the schemagen tool with go run (go run github.com/scylladb/gocqlx/v2/cmd/schemagen@latest) or to install it with go install.
Since V3, this is no longer possible, as go would complain:
go: github.com/scylladb/gocqlx/v3/cmd/schemagen@latest (in github.com/scylladb/gocqlx/v3@v3.0.0):
        The go.mod file for the module providing named packages contains one or
        more replace directives. It must not contain directives that would cause
        it to be interpreted differently than if it were the main module.

This is specified in https://github.com/golang/go/issues/40276:

If that module has a go.mod file, it must not contain directives that
would cause it to be interpreted differently if the module were the main
module. In particular, it must not contain replace or exclude
directives.

(A longer discussion about this can be found in this issue.)
Thus, the reason for this to occur in the case of gocqlx is the replace directive that was added in v3:
replace github.com/gocql/gocql => github.com/scylladb/gocql v1.14.0

I want to raise awareness of this, as the ability to run schemagen with go run (or install it with go install) has been very useful to me.
Unfortunately, I cannot propose an immediate mitigation of this issue (other than replacing all imports to use the scylladb/gocql module directly).

Unfortunately for now there is no way to use scylladb/gocql in the imports, see the discussion here: https://github.com/scylladb/gocql/issues/161. Changing module name for scylladb/gocql is a breaking change so it would require major release in scylladb/gocql repo, but this would not happen soon, here is the issue for changes that require new version release https://github.com/scylladb/gocql/issues/232.
Unfortunately I do not have any quick workaround to make using go install possible @dkropachev any thoughts?

@alfa-alex , Thanks for bringing this to our attention, unfortunately, it won't go away until https://github.com/scylladb/gocql/issues/232 is merged and we need at least a year to make it happen.
Luckily there is a workaround:
git clone git@github.com:scylladb/gocqlx.git`
cd gocqlx/cmd/schemagen/
go install .

I am closing it, but feel free to ask more questions or even reopen it.


Make allow_data_races_* public and use it during EnvVars::cleanup

Fixes https://github.com/rust-lang/miri/issues/2020
I've tried for hours now to come up with a test case for this ICE with no luck. I suspect there's something about the way the data race detection works under these conditions that I just don't understand :weary:.
But I tried this change out on a handful of crates and I don't see any more ICEs of this form. For whatever reason it seems like bastion==0.4.5 is a good way to run into this, with the flags
MIRIFLAGS="-Zmiri-tag-raw-pointers -Zmiri-panic-on-unsupported -Zmiri-disable-isolation" cargo +miri miri test --no-fail-fast --doc

I think all the cases I've run into with this involve both -Zmiri-panic-on-unsupported and -Zmiri-tag-raw-pointers, so it could be that the combination of an unexpected panic and a machine halt is required.

@bors r+

:pushpin: Commit 3cfce6ffb23d8ea83593b4956212954a0459d372 has been approved by oli-obk

:hourglass: Testing commit 3cfce6ffb23d8ea83593b4956212954a0459d372 with merge d58abab2e74004f54ebbd778a9602b28314b9a39...

:sunny: Test successful - checks-actions
Approved by: oli-obk
Pushing d58abab2e74004f54ebbd778a9602b28314b9a39 to master...


Protobuf v5 support

Is your feature request related to a problem? (你需要的功能是否与某个问题有关?)
The brpc vcpkg port depends on protobuf. Since vcpkg maintains a single-version scheme, updating protobuf requires brpc to support it.
Describe the solution you'd like (描述你期望的解决方法)
Protobuf v5 support. V4 support can be broken in the process.
Describe alternatives you've considered (描述你想到的折衷方案)
Patching brpc, see https://github.com/microsoft/vcpkg/blob/052b06a95da4cee184bab4422e02e7f316055a99/ports/brpc/protobuf.patch
Additional context/screenshots (更多上下文/截图)

cc @wasphin

Just missed this, will try it this weekend, thanks.


WFCORE-3237 Seam integration jar name mismatch

Please send up 3.x and wildfly-core-eap/3.0.x variants. Thanks!

@bstansberry sent


Implement mutexes and semaphores without redis

Once issue #1 is completed, it may be possible to get rid of redis as a dependency by replacing it with the posix_ipc package.
Cf. this branch for a proof of concept.

or use https://docs.python.org/3.5/library/multiprocessing.html#multiprocessing.Semaphore?

I guess this is fixed by commit 343e9d7705e87909b2873b74e3840252e51db200

I guess so :-)


Disabling navigation does not behave as expected.

Package
filament/filament
Package Version
v3.0.99
Laravel Version
10.32.1
Livewire Version
v3.2.1
PHP Version
8.2.11
Problem description
According to this I should be able to disable the navigation.
https://filamentphp.com/docs/3.x/panels/navigation#disabling-navigation
use Filament\Panel;

public function panel(Panel $panel): Panel
{
    return $panel
        // ...
        ->navigation(false);
}

Which stops rendering the navigation but it still takes up space for the sidebar.
As a workaround I used the top navigation
https://filamentphp.com/docs/3.x/panels/navigation#using-top-navigation
use Filament\Panel;

public function panel(Panel $panel): Panel
{
    return $panel
        // ...
        ->topNavigation();
}

Setting both settings kinda results in the expected behavior. Except on mobile the sidebar still gets rendered.
It still shows the menu button.

And It still opens and displays an empty menu

Expected behavior
By adding following code

use Filament\Panel;

public function panel(Panel $panel): Panel
{
    return $panel
        // ...
        ->navigation(false);
}

I would expect that the navigation including sidebar and mobile menu to be removed.
If this is expected behavior I would expect there to be a  >sidebar(false); or something.
Steps to reproduce
Just deactivate the navigation
use Filament\Panel;

public function panel(Panel $panel): Panel
{
    return $panel
        // ...
        ->navigation(false);
}

Reproduction repository
https://github.com/filamentphp/filament
Relevant log output
No response

Its a fresh install
https://github.com/filamentphp/filament
Just add one line of code I don't understand why it needs a repo!


Add ability to see when an image has last been used by a container

With this pull request now the images graph updates its last used time when one or more of these actions occur:

Image

build
pull
load
tag

Container

create
start
stop

Now images config has a new field named LastUsed, uses the same type as Created, (time.Time)
So whenever the actions listeed before occurs, it walks all over the image grahp updating the LastUsed field.
I set the update LastUsed in the actions described on the opened issue and the ones that I think that have more sense but If there are actions that you think that will be good to add or remove, would be an easy change, so lets discuss :)
Closes #4237

I need to review the tests, this is tricky, Is a bad Idea to think that all the CI systems will be fast enough all the time to be a second or less between actions.

I changed the tests to base on "before & after" and not on concrete time intervals (1 second & less than a second), although I don't understand the windows CI server timing results.

Collective review with @crosbymichael @jfrazelle @calavera
Thanks for the patch @slok! Unfortunately, we don't think this is something we absolutely need right now, and unfortunately this is something we're going to have to maintain until the end of time in an already cluttered API :-(
We're looking into improving the API, and hopefully this will help with its long term maintainability.
Feel free to ping us on IRC if you are looking for areas to help us contribute. We will also be filing more PR in the future with a 'roadmap' label for the community to participate in more important efforts. Thanks again!


Smooth chat streaming output

This won't build because types are missing.
src/hooks/useSubmit.ts:103:11 - error TS7034: Variable 'messageQueue' implicitly has type 'any[]' in some locations where its type cannot be determined.

103       let messageQueue = [];
              ~~~~~~~~~~~~

src/hooks/useSubmit.ts:108:34 - error TS7005: Variable 'messageQueue' implicitly has an 'any[]' type.

108             let messageContent = messageQueue.shift();
                                     ~~~~~~~~~~~~

src/hooks/useSubmit.ts:130:33 - error TS7006: Parameter 'stream' implicitly has an 'any' type.

130       async function readStream(stream) {
                                    ~~~~~~

Found 3 errors in the same file, starting at: src/hooks/useSubmit.ts:103

Great idea! It's definitely smoother, but there is still some room for improvement to match the fluidity of ChatGPT.


Running python file on MacOS gives errors about modules

Hi. When I try to run python file on my Mac (python3 installed, pip3 install -r requirements.txt python3 OxygenX-0.8.py) It gives me errors about modules not found.
exerhythm@Matthews-iMac OxygenX-master % python3 /Users/exerhythm/Downloads/OxygenX-master/OxygenX-0.8.py
Traceback (most recent call last):
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/utils.py", line 29, in
import tkinter as tk  # python3
File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.7/lib/python3.7/tkinter/init.py", line 36, in
import _tkinter # If this fails your Python may not be configured for Tk
ModuleNotFoundError: No module named '_tkinter'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/utils.py", line 36, in
import Tkinter as tk  # python2
ModuleNotFoundError: No module named 'Tkinter'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/button_box.py", line 15, in
from . import utils as ut
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/utils.py", line 43, in
raise ImportError("Unable to find tkinter package.")
ImportError: Unable to find tkinter package.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/exerhythm/Downloads/OxygenX-master/OxygenX-0.8.py", line 14, in
from easygui import fileopenbox
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/init.py", line 34, in
from .boxes.button_box import buttonbox
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/button_box.py", line 18, in
import global_state
ModuleNotFoundError: No module named 'global_state'
Any fix? Thanks

It aint made for mac

I installed Windows 10 Virtual Machine. How can I download the .exe file?
Thanks
чт, 27 авг. 2020 г. в 11:34, Yeetmachien<EMAIL_ADDRESS>
It aint made for mac
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ShadowOxygen/OxygenX/issues/45#issuecomment-681786844,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMQHNDXVWOKQEO3NNZWCZV3SCYLAZANCNFSM4QMU55SA
.

Hi. When I try to run python file on my Mac (python3 installed, pip3 install -r requirements.txt python3 OxygenX-0.8.py) It gives me errors about modules not found.
exerhythm@Matthews-iMac OxygenX-master % python3 /Users/exerhythm/Downloads/OxygenX-master/OxygenX-0.8.py
Traceback (most recent call last):
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/utils.py", line 29, in
import tkinter as tk # python3
File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.7/lib/python3.7/tkinter/init.py", line 36, in
import _tkinter # If this fails your Python may not be configured for Tk
ModuleNotFoundError: No module named '_tkinter'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/utils.py", line 36, in
import Tkinter as tk # python2
ModuleNotFoundError: No module named 'Tkinter'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/button_box.py", line 15, in
from . import utils as ut
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/utils.py", line 43, in
raise ImportError("Unable to find tkinter package.")
ImportError: Unable to find tkinter package.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/exerhythm/Downloads/OxygenX-master/OxygenX-0.8.py", line 14, in
from easygui import fileopenbox
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/init.py", line 34, in
from .boxes.button_box import buttonbox
File "/Users/exerhythm/Library/Python/3.7/lib/python/site-packages/easygui/boxes/button_box.py", line 18, in
import global_state
ModuleNotFoundError: No module named 'global_state'
Any fix? Thanks

The problem is the module easygui since its not made for MacOS
The exe is in the release section
https://github.com/ShadowOxygen/OxygenX/releases


decide whether we should use a special prompt for LUMI shell commands

episodes 1-2 so far just use $

Do we have to use any prompt? Personally, I think it's easier to copy and paste commands if there's no symbol for the prompt.

the reason for having a special prompt is to distinguish a local shell from a remote shell. When just using $ in a console block one can copy-paste the command and the $ is automatically ignored! Not sure if that's the case if one uses username@uan01:~$ or similar

Ah, the more you know! I'm fine with any solution for now, should be easy to find and replace with sed or similar if we change our mind later?


rosbag_rviz_panel: 0.1.9-2 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository rosbag_rviz_panel to 0.1.9-2:

upstream repository: https://github.com/fada-catec/rosbag_rviz_panel.git
release repository: https://github.com/fada-catec/rosbag_rviz_panel-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: null

rosbag_rviz_panel
* Added CHANGELOG.rst
* Prepare package for release

Holding for Noetic sync


ReportbackItemDetailView -(CGFloat)heightForWidth:width

Closes #697 - renders Reportback Items with dynamic height in the Campaign Detail view controller.
Instead of using iOS self sizing cells, we manually calculate the height based on xib values in a new -(CGFloat)heightForWidth:width method.
I've been fighting with getting a sizing cell to return the appropriate height (#687, #688, #694), but it keeps returning sizes that are too large (and the results vary upon each load, making it even trickier).
Reviewed with @pixelrevision -- using good ol' math (similar to what we're already doing in the LDTCampaignDetailViewController sizeForCellAtIndexPath: iimplementation) seems just way more understandable and maintable at this point, vs fumbling around in the dark trying to get autolayout to magically calculate the numbers via layoutSubviews, layoutIfNeeded, etc.

Would make sense to remove the displayShareButton functionality and related hide/collapse to keep things tidy, as we're just manually determining whether or not to show it based on the height we return in -(CGFloat)heightForWidth:width

Going to keep the hide/collapse functionality in for now, as it could come in handy for kudos/props (#621)


How to stop pg cluster (scale 0)?

Hi. How do you stop a pg cluster? The CR doesn't allow instances: 0 since the minimum is 1. And the operator doesn't use statefulsets to scale the ss to 0. So how can I stop/pause a pg cluster that I've created?

You can hibernate the cluster, see: https://cloudnative-pg.io/documentation/1.20/declarative_hibernation/


Deduplicate schema constants

Codecov Report
Base: 31.69% // Head: 35.27% // Increases project coverage by +3.57% :tada:

Coverage data is based on head (9b1cf80) compared to base (8b75d32).
Patch coverage: 43.11% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #674      +/-   ##
==========================================
+ Coverage   31.69%   35.27%   +3.57%
==========================================
  Files          95      105      +10
  Lines       11588    12219     +631
==========================================
+ Hits         3673     4310     +637
+ Misses       7586     7521      -65
- Partials      329      388      +59

Impacted Files
Coverage Δ

api/server/v1/error.go
0.00% <0.00%> (ø)

api/server/v1/marshaler.go
9.96% <0.00%> (-0.27%)
:arrow_down:

api/server/v1/validator.go
0.00% <0.00%> (ø)

internal/data.go
34.17% <0.00%> (ø)

query/search/search.go
44.44% <ø> (-1.34%)
:arrow_down:

server/metadata/database.go
0.00% <0.00%> (ø)

server/metrics/quota.go
31.37% <0.00%> (-7.66%)
:arrow_down:

server/middleware/forwarder.go
0.00% <ø> (ø)

server/middleware/measure.go
0.00% <0.00%> (ø)

server/middleware/metadata_extractor.go
0.00% <0.00%> (ø)

... and 54 more

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.