Simplify observability configuration for customers

This PR:

changes how we configure the observability bundle to align with other bundles in the company towards https://github.com/giantswarm/roadmap/issues/2879

This PR will have a lot of known breaking changes that have to be done before releasing described below:
### Changes to do before releasing
- [x] logging-operator promtail enablement https://github.com/giantswarm/logging-operator/pull/60
- [x] renaming prometheus-operator-app to kube-prometheus-stack, waiting for the other 2 to be released first https://github.com/giantswarm/prometheus-operator-app/pull/337 Did not change the repo name yet
- [x] disabling psp disablement on MCs in giantswarm/config https://github.com/giantswarm/config/pull/2001

Checklist

[x] Update changelog in CHANGELOG.md.
[x] Make sure values.yaml and values.schema.json are valid.

@giantswarm/team-atlas I tested this on CAPI and Vintage but I'd like another pair of eyes especially around cluster upgrade. I'll do an upgrade test tomorrow as well

Upgrade tested and it's working fine but I'd love to have some other eyes cc @giantswarm/team-atlas

✅  Tested on gaia, more details here https://github.com/giantswarm/roadmap/issues/2879#issuecomment-1804102392


added equals and hashcode to summary objects

Pull Request Test Coverage Report for Build 3490

0 of 44 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 69.98%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/src/main/java/com/netflix/conductor/common/run/WorkflowSummary.java
0
20
0.0%

common/src/main/java/com/netflix/conductor/common/run/TaskSummary.java
0
24
0.0%

Totals

Change from base Build 3480:
-0.2%

Covered Lines:
11159

Relevant Lines:
15946

💛  - Coveralls


feat(prover): initL1Current from genesisHeight

when LatestVerifiedId 0, set l1Current to stateVars.GenesisHeight instead start over

Merged, thanks for the contribution!


how to fix this ?

You need to use the AUTHORIZATION environment variable from .env as the Authorization header.
This app is only meant to be used as an API, there is no UI and you can't use it in a browser.

Can you show an example
Using this api I tried discord bot.
Bot getting error when try to connect domain


Python wrappers: return proto Empty type

Previously returned None.
Fixes #760.

Sorry, I realize now that we do not need to return google.protobuf.Empty, only other empty messages. I need to revise this PR.

Current coverage is 83.28% (diff: 100%)

Merging #796 into master will decrease coverage by 0.01%

@@             master       #796   diff @@
==========================================
  Files           314        314
  Lines         11910      11907     -3
  Methods           0          0
  Messages          0          0
  Branches       1564       1562     -2
==========================================
- Hits           9921       9917     -4
  Misses         1561       1561
- Partials        428        429     +1

Powered by Codecov. Last update f23a41b...c683a94

@jmuk @swcloud Since this is beta blocking for Python and @bjwatson is OOO, it would be great if one of you has time to do this review.

@jmuk Thanks!

Actually I think it's better use Empty.getDescriptor().getFullName() instead of using the hard-coded string. Also I think you should call MethodConfig.isReturnEmptyMessageMethod.

@swcloud OK, I will make that change on a follow-up PR.

@geigerj Which PR or issue is this? Does this follow-up PR need to be done for beta?

No, it is a style change. There is no impact on the surface. It was done in #797.

Got it. Thanks!


[Ray] Fix ray executor progress test

What do these changes do?
Ray executor progress test does not run due to lack of await. This pr fixed it.
Related issue number

Fixes #xxxx
Check code requirements

[x] tests added / passed (if needed)
[x] Ensure all linting tests pass, see here for how to run them

@fyrestone Could you please review this PR to see if there is any further issue?


Typescript - Could not find a declaration file for module 'web-vitals'

What is happening
With the new version of<EMAIL_ADDRESS>typescript cannot find a declaration file for the module.
How to reproduce
https://github.com/magicaltrip/webvitals-issue
Thanks

Sorry! This should be fixed in v1.0.1. Let me know if you're still seeing issues.


Added some initial temp Dir directory handling

I think that only works on linux. I'll try using the library that chiapos uses.


Coloring JFXTabPane

Hello!
First of all thank you for your work!
Is there a way to set color the background ( tab-header-background ) and the active tab's line ( tab-selected-line ) without using css?
If there's no way can you send me an example of changing the color of "tab-selected-line"? ( I have tried soo many times and I couldn't change this )

this issue is related to #63


Implement Communication Level API (L2)

The uP-L2 API needs to be implemented based on the uP-L1 API

closed via #147


Add support for CycloneDX 1.5

CycloneDX 1.5 was released with changes to the schema. Gatecheck should check the version of the cyclonedx sbom to see if it supports the current file and fail before.

Nvm must have been an old version of grype/syft. Closing


[Do Not Merge] chore: Shadow PR to external contribution #36231

Description

Shadow PR to run tests on external contribution https://github.com/appsmithorg/appsmith/pull/36231

Fixes https://github.com/appsmithorg/appsmith/issues/36073
Automation
/ok-to-test<EMAIL_ADDRESS>:mag: Cypress test results

[!TIP]
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10899078369
Commit: 383128d9f4be1fde88bf84ffb41d05ebf8938aaf
Cypress dashboard.
Tags: @tag.Sanity
Spec:
Tue, 17 Sep 2024 08:31:09 UTC

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[ ] No

Summary by CodeRabbit
Release Notes

New Features

Enhanced logging capabilities across multiple plugins by replacing System.out.println with structured logging using log.debug and log.error.

Bug Fixes

Removed outdated logging method stopAndLogTimeInMillisWithSysOut from the Stopwatch class, improving logging consistency.

Chores

Updated dependencies in the project configuration to better support the new logging framework.

These changes improve maintainability, performance, and debugging capabilities across the application.

Closing this as the original PR https://github.com/appsmithorg/appsmith/pull/36231 has been merged


Permission denied when using persistent volumes

chart version = 0.20.1
when using
persistence:
  enabled: true
  type: dynamic

deploying this for the first time getting this error
Error: EACCES: permission denied, open '/home/node/.n8n/config'
    Code: EACCES

Running on EKS

The pvc is mounted to /home/node/.n8n/ (here)
and n8n assumes the dir config exists, but It is not there.
You have two options as a workaround:

Create that dir config somehow, manually.
set ENV var N8N_CONFIG_FILES= /n8n-config/config.json,/n8n-secret/secret.json,/home/node/.n8n/

I will find a solution to that problem. The only options that jumps into my mind right now is init-container.
Happy to accept PRs
Btw, why do you need a volume?

Btw, why do you need a volume?

Im using a community node for 'AWS Secret Manager', and installing it via the UI writes it to the home directory of N8N, so i wanted to have the persistent.
I will try the workaround
Thanks for the quick response

ahh makes sense, thanks

I tried the second workaround and it didnt work..
I dont think there is a way to do the first one..

having the same error, "Create that dir config somehow, manually" will not work in k8s, since i cant acces the pod when it fails.

quick and dirty fix:
add this to your values.yaml:
  securityContext:
    runAsUser: 0
    runAsGroup: 0

this will run the container as root!
this then propagates the root user security context in the container (see here and makes the config accessible again.
this is not a good thing, i actually don't want to run my containers in k8s as root. but its the only way i to get persistency running rn.

@Vad1mo any news regarding a proper fix for this?

fixed security context  in 0.22, not sure if that fixes your issue. We might need to have another fix
https://artifacthub.io/packages/helm/open-8gears/n8n?modal=changelog

do you see still the issue with 0.22 @aep and @omri-shilton ?

Yes. Copying the config (not a dir!) manually works, although it's super annoying.

I don't have time this week to implement a fix. But I am happy to accept any PRs coming that ways .
What you basically need to do is create an init container. I would use the same n8n container and add a command that will overwrite the entrypoint.
the command should be mkdir -p /home/node/.n8n/


161

closes #161

PR-Codex overview
This PR focuses on updating the environment variables and configuration files for the QA Admin application.
Detailed summary

Updated .env file in the client directory to include VITE_ADMIN_API_URL and VITE_ENABLE_LOGGING variables.
Modified application.yaml file in the qa-rest and qa-admin directories to update the database URL and Hibernate configuration.
Updated docker.env file to include new values for POSTGRES_HOST, POSTGRES_PORT, QA_ADMIN_PORT, QA_ADMIN_CLIENT_PORT, QA_REST_PORT, SMTP_HOST, and SMTP_PORT.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Codecov Report
Patch coverage has no change and project coverage change: +0.53 :tada:

Comparison is base (347c88d) 61.20% compared to head (83b1ed2) 61.74%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #223      +/-   ##
============================================
+ Coverage     61.20%   61.74%   +0.53%
  Complexity      141      141
============================================
  Files            92       92
  Lines          1678     1678
  Branches        119      120       +1
============================================
+ Hits           1027     1036       +9
+ Misses          542      533       -9
  Partials        109      109

see 2 files with indirect coverage changes

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


[CHALLENGE SUBMISSION]

Please provide the following information for your submission:
What is your name? (First, Last)
Robert Steele
Where is your GitHub Repo ?(Link to your challenge solution)
challenge one
What Challenge is this for? (Challenge number between 1 and 7)
1
(Optional) Anything else we should know? e.g., language used, location, blog post, tell us how you solved this solution, including any challenges you faced solving it.

hi, can you give me details how to run/test this solution? thanks!

His solution is under another directory of the repository: https://github.com/antempus/SeasonsOfServerlessSolutions/blob/primary/lib/turkey.ts
If you want to launch it, I believe that opening the folder given in the issue on VS Code then pressing F5 should work (haven’t tried) :)

From all of us at Developer Relations at Microsoft, thank you for participating in #SeasonsOfServerless and submitting a solution!

Feel free to share your solution on Twitter with the tag #SeasonsOfServerless.

Thanks for your awesome submission!
🎉 We've now added you and your solution to the Hall of Fame. 🎉

Thanks @Christophe-Ch for responding, that should've been the way to run/validate.
@jansche Thanks! 🥳


registrations#new view is not translated

Is there any reason that this file is not translated?
https://github.com/mcasimir/devise-i18n-views/blob/master/app/views/devise/registrations/new.html.erb

Password and Remeber me also should be translated.
https://github.com/maximalink/devise3-i18n-views/blob/master/app/views/devise/sessions/new.html.erb

What makes you think they're not translated?

Password confirmation label
https://github.com/mcasimir/devise-i18n-views/blob/master/app/views/devise/registrations/new.html.erb#L12
and
Password label
https://github.com/mcasimir/devise-i18n-views/blob/master/app/views/devise/registrations/new.html.erb#L9

Read about localization at http://api.rubyonrails.org/classes/ActionView/Helpers/FormHelper.html#method-i-label.
This is translatable but perhaps not yet provided in the language you care about?

Thanks. I think inline translation would be better instead of ActiveRecord translation in this case. Devise support any name for resource. If somebody generate member, client, staff, or other this current translation method will not work.
Another reason is DRY. These attributes are not model name specific. For example the password is the same for client, staff, member, user, now we need to translate for all generated model. Why we need to translate more than once?

That sounds pretty uncommon - a more reasonable scenario would be to have a user model and either attributes defining their role or submodels.
You can always generate the views and locales and customize away.


[GIE] Error parsing layer type Deconvolution index 379

Can everyone help me how to solve this following error when i want to use deconvolution layer?
i see the following website, it shows that GIE supports deconvolution layer, but i try to add deconvolution layer to my prototxt and build the new caffe model whose format is not like goolgenet, alexnet and detectnet. I get this error.
https://devblogs.nvidia.com/parallelforall/production-deep-learning-nvidia-gpu-inference-engine/
[GIE]  loading pvaNetClassifier.prototxt pvanet_frcnn_iter_100000.caffemodel
Caffe Parser: groups are not supported for deconvolutions
error parsing layer type Deconvolution index 379
[GIE]  failed to parse caffe network
Thank you very much.

Thank everyone. i find the developer document about GIE. This document is located in your deployment device (e.g. jetson TX1). The path is /usr/share/doc/gie.
Solution: The version one of GIE don't support the groups parameter of deconvolution layer.

Hi @Jia-HongHenryLee , could you tell me what is the groups parameter of deconvo layer?

Hi @lbin , you can visit the following website. it can solve your question.
http://stackoverflow.com/questions/40872914/caffe-what-does-the-group-param-mean

Thanks for the reference, @Jia-HongHenryLee !
This group param confused me. My segmentation model has deconv layer, and  I find the default value of the group is 1, and the model can be load by GIE. Unfortunately,  my result is totally wrong.
By the way, this model in caffe could get right results.

@lbin  yes, because TensorRT(GIE) don't support the group parameter of deconvolution layer. so said. i think that the second version of Tensor RT maybe will support it.

you can try to modify segnet which is in jetson-inference to let it have the same accuracy with your version

@Jia-HongHenryLee Thanks for the kindly reply. Now I am doing my work based on the Caffe.

@lbin  You're welcome.


Bitbucket remote repo failed

After input of password to  my account from command line I received exception:
Traceback (most recent call last):
File "/tmp/tmpr61y4z6r.py", line 203, in
main()
File "/tmp/tmpr61y4z6r.py", line 198, in main
setup_git_repo()
File "/tmp/tmpr61y4z6r.py", line 129, in setup_git_repo
auth_base = base64.b64encode('{}:{}'.format(*auth_info))
File "/home/oshcherbyna/PycharmProjects/test_venv/lib/python3.6/base64.py", line 58, in b64encode
encoded = binascii.b2a_base64(s, newline=False)
TypeError: a bytes-like object is required, not 'str'
Stopping generation because post_gen_project hook script didn't exit successfully

Hey @OksanaShc :wave:
I appreciate the feedback! I need to integrate continuous integration testing (see #8 and #9), but I'll manually test Bitbucket and get that working today. I only use GitHub and GitLab myself, so apologies for the slight overlook. :sweat_smile:
Thanks for your interest in cookiecutter-git! :slightly_smiling_face:

Hey @nathanurwin !
I guess that it will be a problem with Python versions. I used 3.6.4. Is it supported?

I thought it was only a Bitbucket thing, but you were right it was a Python 3 error! Manually testing takes some time and is not all comprehensive, but both Bitbucket and Python 3 should be working now (I'm going to push the release here in a minute). Thank you for your time! :+1:

thank you too for your work and quick fix!


Allow retry on logon

When trying to logon to windows without otp if the password is entered incorrectly the user is taken back to the logon screen to enter credentials again. With the OTP credential provider installed trhis does not happen. This is especially problematic when using RDP as it closes the entire session which the user must start again. Ideally if a user enters incorrect details they would be taken back to the logon screen with the option to enter details again, is this possible?

Which windows version are you using?

Server 2012 R2

Unfortunately that architecture is unknown for me. But if this is similar to Win 8 then I might be able to help you but if this is more Win 7 style - then you have to ask other participants.

Thanks 2012r2 is effectively the same base O/S as win 8.1 yes. I will test this on win8 too to see if it is the same behavior.

In that case read my post in #16 and #9

Sorry which part? Are you suggesting using the RDP only dll will fix this issue?

Yes, if you know how to manually configure it via registry (it is described in issue 9). In case of RDP and custom credential provider the first credential you enter is for security layer of remote connection, the second is simple logon process and in my dll it is a part of normal process to enter incorrect credentials - it does not disconnect the client (not the first time anyway and not on my Win 8.0).

Thanks, I have tried setting up you DLL, I can see it being accessed in procmon but it doesnt seem to actually prompt for OTP. I tried using the deval dll and it doesnt seem to create the log either? Im sure I am missing something. Thanks

By default register.reg has option forbidding OTP for local user (so you can install it and not block your computer if the multiotp doesn't work). First try to connect via remote desktop if it works and OTP will correctly log you into the system then switch MultiOTPRDPOnly in the registry settings so it will ask you for the OTP every time you login or unlock the station.

Thanks, I have tried with 1 and 0, i dont get otp for rdp logon or standard logon on any reg setting.

Check the dll ntfs rights (add full grants for system). log will be created when the dll is invoked by the LogonUI.

Thanks, NTFS permissions look fine. Just to explain the setup I have done:
On a clean server, installed multiotp and configured, testing using the debug command line and everything is working.
Copied the DLL to system 32 (currently the logging DLL but tried with both) and checked NTFS permissions.
Imported the reg files (with the correct multiotp path set) and rebooted.
When trying to logon with RDP or a console session the multiotp is not shown, I have also changed the rdponly key to 0 and rebooted and the multiotp is still not shown for RDP or console logon.
Thanks

I have just uploaded updated dll - it should work ok (I've tested it on my win 8.0) Do you have appropriate reg entries, does ProcMon show the dll being loaded?

I tried the new DLL and still nothing. This is procmon (filtered) and it looks like it is reading registry and DLL fine

Are you sure you have 64bit architecture - the dll does not try to access registry or log file - that is probably because it is loaded but not initialized by the LogonUI.

Yes this is a 64bit machine. If there are things missing from that screenshot it may be because they are filtered out, where else are you expecting to be accessed?

First I'm trying to create log C:\multiotp.txt (you could try and create this empty file for all users R/W) then if it's an rdp connection I check the rdp port, if local connection HKEY_CLASSES_ROOT\CLSID{FCEFDFAB-B0A1-4C4D-8B2B-4FF4E0A3D978}\MultiOTPRDPOnly I do not see this in your screen.

Thanks, created C:\multiotp.txt and everyone, system, users, administrators all have full control. Nothing being added to that file. I cant see anytihng in procmon trying to read that multiotprdponly key.

I have done some more tests today and found out that Win 8.1 is not providing me with a user list... That's completely unacceptable behavior according to CPV2 samples and tutorials I've found on msdn. Could you try and add a local user account and try again to confirm that this is a problem? You can remove registry key that is named [HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Authentication\Credential Provider Filters{FCEFDFAB-B0A1-4C4D-8B2B-4FF4E0A3D978}] and Set the MultiOTPRDPOnly=0 to speed up the checking process (you won't have to connect via rdp and still be able to logon using other credential providers if my CP fails).

Sorry still not getting any OTP prompts with the latest DLL. Nothing using RDP or console logon with local or domain account.

I will have to migrate to win 8.1 (and that's not an easy task for me)... I'll come back to you as soon as possible.

I have just uploaded new version and it should work now (my Win 8.1 test machine works...)


Netron connects to Google Analytics without any warning

Netron connects to Google Analytics, but doesn't offer any warning or consent option. This is not permitted under the GDPR.

Please provide feedback if this works as expected. The rules are complicated and keep changing. Since this is a hobby it might be easier to deprecate support for some countries unless there is interest by someone local to debug and provide fixes.

This doesn't seem to be working, I'm using Netron from a country that is listed in the countries that are supposed to be given consent, however I never got that consent confirmation. I checked the return from https://ipinfo.io and it is correct.

@perone can you debug the issue and provide more context. Are you using the browser or desktop app?


MUWM-2414

Coverage decreased (-0.02%) when pulling 87b6e64cd817f0cbbe465346b5256d27e5bad19b on bug/muwm-2414 into 937f1f7f506edea594f68d5b41f2ca00ecae39da on develop.


Slackin invite not working!

The slackin invite is not working. It gives the following error. Found no solutions online

Here is the issue open in the Slackin Github

It has been fixed


todo: de-deuplicate code between user page and home page

There's code that's been duplicated on both the user page and the home page. Such code should be cleaned up and centralised. Tests would also be welcomed.

Could you point it out to me the file names where this duplicate code is present ? I would like to take a stab at this.

Home page: /routes/index.tsx
User page: /routes/user/[username].tsx

To clarify, what I had in mind was to move the duplicated code to /utils or into /components, depending on which code it is.

Fixed in #237.


Missing mobile view

Thanks for making this great site but do you think it's possible to add a mobile friendly view for it to show one tree at a time at full screen?

That's planned, it might take a little bit though since I have a couple busy weeks this month.


Why have a default max_seq_length of 256?

I noticed that both the data prep / tokenization script (https://github.com/Lightning-AI/lit-parrot/blob/main/scripts/prepare_alpaca.py#L26) and the fine-tuning scripts (https://github.com/Lightning-AI/lit-parrot/blob/main/finetune/adapter.py#L41, https://github.com/Lightning-AI/lit-parrot/blob/main/finetune/adapter_v2.py#L46) have max_seq_length=256.
While this does seem to speed up tokenization it has the unfortunate property of truncating fine-tuning inputs and also requiring someone to change both scripts to use the full context length of a language model. I'm curious why this parameter got added and whether it might be possible to switch to a default of None or 4096?

That is a good observation. I agree that we should remove the max_seq_length value passed to the model forward and separate it from the max_length used to split the dataset.
I don't know why this was chosen originally. Perhaps a mixup.
Would you be interested in opening a PR?

Sure, let me take a look tomorrow
On Wed, Jun 7, 2023, 6:24 PM Carlos Mocholí @.***>
wrote:

That is a good observation. I agree that we should remove the
max_seq_length value passed to the model forward and separate it from the
max_length used to split the dataset.
I don't know why this was chosen originally. Perhaps a mixup.
Would you be interested in opening a PR?
—
Reply to this email directly, view it on GitHub
https://github.com/Lightning-AI/lit-parrot/issues/122#issuecomment-1581592668,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAL2OJU5FP76FEMEEM5HJDXKD5P7ANCNFSM6AAAAAAY6N2HZA
.
You are receiving this because you authored the thread.Message ID:
@.***>

Yes, definitely! Great idea, thanks @iskandr

Started a PR (https://github.com/Lightning-AI/lit-parrot/pull/127/) but still have to actually test that it doesn't change or break anything.


Spike: lazy union reader

This is a spike to try to reduce some of Syft's memory allocations; in particular we have had user reports of OOM when running the elf-binary-package-cataloger. (report, report).
Currently, Syft uses interfaces to hide the underlying file system from the catalogers, because the filesystem might be a tar, or a representation of an OCI image, or a directory, or whatever, and the catalogers shouldn't need to know about those differences. In order to achieve this, Syft uses a type called UnionReader: https://github.com/anchore/syft/blob/5326efcef13db1df2f03e77ce580feef9857569d/syft/internal/unionreader/union_reader.go#L14-L19
Currently, instantiating a UnionReader, if the underlying reader doesn't implement ReadAt or Seek, results in calling io.ReadAll on the underlying reader, and using the resulting bytes to back the Seek and ReadAt calls: https://github.com/anchore/syft/blob/5326efcef13db1df2f03e77ce580feef9857569d/syft/internal/unionreader/union_reader.go#L47
The new elf-binary-package-cataloger needs to call ReadAt and Seek on a potentially large number of files, resulting in a lot of io.ReadAll calls if this cataloger is enabled.
This branch spikes out making a new struct called lazyUnionReader, which wraps an io.ReadCloser and presents a UnionReader. Basically, instead of calling io.ReadAll when it is created, it copies all bytes read (or Seeked past) into a backing byte slice. In other words, it has similar behavior as the current UnionReader, except that the allocation is lazy; if we never read past byte 102 in a given file, the backing array will only be (about) 102 bytes long, instead of the whole length of the file.
I thought maybe another package in the go ecosystem would provide a lazy read seeker, but I haven't found one, so this branch implements one with some unit tests.
I'll post comments with some profiling.

Here are 3 profiles:
Main, all default image catalogers:

this branch, all default image catalogers:

main, elf-binary-package-cataloger specifically disabled:

The profiles were generated by adding the following code (+ necessary imports) to cmd/syft/main.go:
	defer func() {
		f, err := os.Create("mem.prof")
		defer f.Close()
		if err != nil {
			panic(err)
		}
		runtime.GC() // get up to data mem info
		if err = pprof.WriteHeapProfile(f); err != nil {
			panic(err)
		}
	}()

And then rendered by running go tool pprof -http=:8080 mem.prof, selecting "Flame graph" from the view menu, and selecting "alloc_space" from the sample menu.

@kzantow asks a good question: How much do we save of maximum heap size, rather than total allocations over the whole execution?
I'll try to find the answer, and profile some more images.

Superseded by #2814


Worker launch command contains references to non-existant & invalid directories

Commit I am working on:
https://github.com/mesos/storm/commit/a49e8ce1af34969daf0479b09ce7a741e181d4ef
Actions:

vagrant up
start nimbus
submit a topology

Problem:
Following is the ps output of the running working. The directoris /mnt/mesos/sandbox/logs, /opt/storm/lib/ etc do not exist.
root     12914  5.8  7.0 2963108 285176 ?      Sl   17:36  15:19 /usr/lib/jvm/java-7-openjdk-amd64/bin/java -server -Xmx384m -XX:+PrintGCDetails -Xloggc:artifacts/gc.log -XX:+PrintGCDateStamps -XX:+PrintGCTimeStamps -XX:+UseGCLogFileRotation -XX:NumberOfGCLogFiles=10 -XX:GCLogFileSize=1M -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=artifacts/heapdump -Djava.library.path=storm-local/supervisor/stormdist/SampleTopology-1-1458074947/resources/Linux-amd64:storm-local/supervisor/stormdist/SampleTopology-1-1458074947/resources:/usr/local/lib:/opt/local/lib:/usr/lib -Dlogfile.name=worker-31000.log -Dstorm.home=/opt/storm -Dstorm.conf.file= -Dstorm.options=storm.log.dir%3D%2Fmnt%2Fmesos%2Fsandbox%2Flogs -Dstorm.log.dir=/mnt/mesos/sandbox/logs -Dlogback.configurationFile=/opt/storm/logback/cluster.xml -Dstorm.id=SampleTopology-1-1458074947 -Dworker.id=869a17ff-9121-432b-bd43-ae4dfec81ef0 -Dworker.port=31000 -cp /opt/storm/lib/commons-io-2.4.jar:/opt/storm/lib/commons-codec-1.6.jar:/opt/storm/lib/commons-exec-1.1.jar:/opt/storm/lib/reflectasm-1.07-shaded.jar:/opt/storm/lib/storm-core-0.9.6.jar:/opt/storm/lib/disruptor-2.10.4.jar:/opt/storm/lib/commons-fileupload-1.2.1.jar:/opt/storm/lib/ring-core-1.1.5.jar:/opt/storm/lib/json-simple-1.1.jar:/opt/storm/lib/clj-stacktrace-0.2.2.jar:/opt/storm/lib/tools.macro-0.1.0.jar:/opt/storm/lib/math.numeric-tower-0.0.1.jar:/opt/storm/lib/jetty-6.1.26.jar:/opt/storm/lib/tools.logging-0.2.3.jar:/opt/storm/lib/asm-4.0.jar:/opt/storm/lib/commons-lang-2.5.jar:/opt/storm/lib/ring-jetty-adapter-0.3.11.jar:/opt/storm/lib/clout-1.0.1.jar:/opt/storm/lib/core.incubator-0.1.0.jar:/opt/storm/lib/carbonite-1.4.0.jar:/opt/storm/lib/logback-classic-1.0.13.jar:/opt/storm/lib/log4j-over-slf4j-1.6.6.jar:/opt/storm/lib/jgrapht-core-0.9.0.jar:/opt/storm/lib/tools.cli-0.2.4.jar:/opt/storm/lib/ring-servlet-0.3.11.jar:/opt/storm/lib/chill-java-0.3.5.jar:/opt/storm/lib/hiccup-0.3.6.jar:/opt/storm/lib/objenesis-1.2.jar:/opt/storm/lib/clojure-1.5.1.jar:/opt/storm/lib/snakeyaml-1.11.jar:/opt/storm/lib/kryo-2.21.jar:/opt/storm/lib/storm-mesos-0.1.0-SNAPSHOT-storm0.9.6-mesos0.27.0.jar:/opt/storm/lib/joda-time-2.0.jar:/opt/storm/lib/ring-devel-0.3.11.jar:/opt/storm/lib/commons-logging-1.1.3.jar:/opt/storm/lib/minlog-1.2.jar:/opt/storm/lib/servlet-api-2.5.jar:/opt/storm/lib/compojure-1.1.3.jar:/opt/storm/lib/slf4j-api-1.7.5.jar:/opt/storm/lib/logback-core-1.0.13.jar:/opt/storm/lib/jetty-util-6.1.26.jar:/opt/storm/lib/clj-time-0.4.1.jar:/opt/storm/lib/jline-2.11.jar:/opt/storm/conf:storm-local/supervisor/stormdist/SampleTopology-1-1458074947/stormjar.jar backtype.storm.daemon.worker SampleTopology-1-1458074947 master 31000 869a17ff-9121-432b-bd43-ae4dfec81ef0

I believe this is specific to using the Docker image option, since the /opt/storm path only is referenced by the Dockerfile in this repo:
% git grep /opt
Dockerfile:  mkdir -p /opt/storm && \
Dockerfile:  tar xf storm-mesos-*.tgz -C /opt/storm --strip=1 && \
Dockerfile:WORKDIR /opt/storm

@dsKarthick :  I'm still unsure what the issue is here.
The docker images I've looked at do have /opt/storm/lib.
Also, regarding storm.log.dir, it just uses the MESOS_SANDBOX environment variable that is set explicitly by mesos's docker containerizer code:
(mesos) [tags/0.22.1] % git grep MESOS_SANDBOX
docs/docker-containerizer.md:3, Run the docker image with the configured DockerInfo options, and map the sandbox directory into the Docker container and set the directory mapping to the MESOS_SANDBOX environment variable.
src/docker/docker.cpp:  argv.push_back("MESOS_SANDBOX=" + mappedDirectory);

So I'm closing this for now.  Please reopen if you have more info.


Include private repo example

It would be really helpful to include an example that shows how to configure a private Jitpack-hosted repository (e.g. how to set the credentials properly)

The example which you are asking for is available here: https://jitpack.io/private#auth


Modernise license format in Cargo.toml

https://doc.rust-lang.org/cargo/reference/manifest.html#slash

Previously multiple licenses could be separated with a /, but that usage is deprecated.

removes ambiguity about whether AND or OR is intended, I guess.

Thanks @dimbleby.


Added new Loveland Webcams apps

Description
Added new Loveland Ski Area Webcams app

Copilot

🤖 Generated by Copilot at 8061275
Summary
📷🎿🎲

This pull request adds a new applet called Loveland Webcams, which displays a random webcam image from Loveland Ski Area on the Tidbyt. The pull request includes the applet code in loveland_webcams.star and the manifest file in manifest.yaml.
Walkthrough

Create and register a new applet for displaying random webcam images from Loveland Ski Area (link, link)
Import tidbyt, random, and image modules to access Tidbyt display, random selection, and image manipulation functions (link)
Define a list of webcam image URLs from the Loveland website (link)
Select a random URL from the list and fetch the image using image.fetch (link)
Cache the image for 10 minutes using tidbyt.cache to avoid excessive requests (link)
Resize and crop the image to fit the Tidbyt display using image.resize and image.crop (link)
Render the image as a marquee on the Tidbyt display using tidbyt.pixlet and tidbyt.marquee (link)
Add some comments and debug messages to explain and monitor the applet behavior (link)
Specify the applet metadata in the manifest.yaml file, including the ID, name, summary, description, author, file name, and package name (link)

need to fix my first commit


Fixed clang-format validation (close #99)

Formatting is not required as the files were moved to the build tree.


test cache

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4b24c98). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage        ?   60.88%
=======================================
  Files           ?       10
  Lines           ?      951
  Branches        ?        0
=======================================
  Hits            ?      579
  Misses          ?      372
  Partials        ?        0

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Redisson Pro initializing without key for unit tests

We are intending to use the Redisson Pro license for running sharded elastic cache cluster. For unit tests and other dev environment we prefer avoiding the pro version redissonclient instance which requires a key. Is there a way to construct a redisson client with the pro license for this purpose?

Please contact<EMAIL_ADDRESS>


File libgcc_s_seh-1.dll is not included

I built the ROM with the branch port-pdplus-solomulti-gex but for some reason, an important extension was excluded. Even if I download one from an external source, the game does not start up!

Resolved on N64 Vault discord server.
This issue is closed.


add config for konflux-workspaces in staging

This adds the necessary configuration for konflux-workspaces to be deployed into staging environments.  Production environments support will come in a separate PR once this has been verified to be working and stable.

/retest

@dperaza4dustbit Before I add comments about minor stuff to change, I want to know why not adding the config to development overlay first?

@dperaza4dustbit Before I add comments about minor stuff to change, I want to know why not adding the config to development overlay first?

It looks like we delete kubesaw operators and dependencies from development overlay @hugares https://github.com/redhat-appstudio/infra-deployments/blob/120f1dbfcb526f39fb41b8596b754a5b6780fb22/argo-cd-apps/overlays/development/delete-applications.yaml#L62-L78 so that is why I suspect @sadlerap did not considered that. Workspace will depend on KubeSaw

@dperaza4dustbit Before I add comments about minor stuff to change, I want to know why not adding the config to development overlay first?

It looks like we delete kubesaw operators and dependencies from development overlay @hugares
https://github.com/redhat-appstudio/infra-deployments/blob/120f1dbfcb526f39fb41b8596b754a5b6780fb22/argo-cd-apps/overlays/development/delete-applications.yaml#L62-L78
so that is why I suspect @sadlerap did not considered that. Workspace will depend on KubeSaw

Yeah, kubesaw is deployed in dev using passing an option to the bootstrap script (--toolchain). I was suspecting that dependency so this means that you will need to provide another option to the build script or deploy workspaces when toolchain is deployed.
Where I am heading with this is we need somehow to be able to deploy workspaces in dev mode otherwise you will not be able to have any test for workspaces in e2e tests. I would also allow to test this change by deploying this in dev first.
That said, I will leave it up to you to decide if you want to deploy workspaces in dev overlay. I will review this PR and add my comments for issues I see in staging overlay

Since I don't have access to @sadlerap infra-deployment fork I will push this branch to my fork and open a new PR to addressing @hugares comments

Since I don't have access to @sadlerap infra-deployment fork I will push this branch to my fork and open a new PR to addressing @hugares comments

Did you try, I can update branches in the forked branches, I do this often. Let me know and I can update this PR if you want

Created new PR under my fork with @sadlerap commit and my changes with @hugares recommendation: https://github.com/redhat-appstudio/infra-deployments/pull/4040
Closing this on in favor of more updated 4040


Update codecov.yml

token not required for Travis
fix ignore location

Thank you! 👍

Thank you for your help!


A guide for releases

@nvasilakis if you have comments/edit suggestions, just go ahead and commit them and then merge this when you are happy with it :)


Trying to allocate about 5000 GiB

Hello!
I'm interested in your great work and trying to run your code.
Although, I have a little trouble solving following error that says I'm trying to allocate about 5000 GiB.
I think the number is too large.
Do you have any idea about this　error regarding your data or model size, etc?
Environment:

Docker image: pytorch/pytorch:1.7.0-cuda11.0-cudnn8-devel
GPU: GeForce RTX 3090, 24GB

root@oucyz:/workspace# scripts/train_small.sh
not detected /blob directory, execute locally
Utilize 1 gpus
/root/.local/lib/python3.8/site-packages/MinkowskiEngine/__init__.py:36: UserWarning: The environment variable `OMP_NUM_THREADS` not set. MinkowskiEngine will automatically set `OMP_NUM_THREADS=16`. If you want to set `OMP_NUM_THREADS` manually, please export it on the command line before running a python script. e.g. `export OMP_NUM_THREADS=12; python your_program.py`. It is recommended to set it below 24.
  warnings.warn(
using instance norm
2022-11-07 09:07:10.839856: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-11-07 09:07:11.057767: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2022-11-07 09:07:11.771248: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2022-11-07 09:07:11.771496: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2022-11-07 09:07:11.771546: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
not detected /blob directory, execute locally
Utilize 1 gpus
using instance norm
2022-11-07 09:07:13.650011: I tensorflow/core/platform/cpu_feature_guard.cc:193] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN) to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2022-11-07 09:07:13.897804: E tensorflow/stream_executor/cuda/cuda_blas.cc:2981] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2022-11-07 09:07:14.678678: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer.so.7'; dlerror: libnvinfer.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2022-11-07 09:07:14.678788: W tensorflow/stream_executor/platform/default/dso_loader.cc:64] Could not load dynamic library 'libnvinfer_plugin.so.7'; dlerror: libnvinfer_plugin.so.7: cannot open shared object file: No such file or directory; LD_LIBRARY_PATH: /usr/local/nvidia/lib:/usr/local/nvidia/lib64
2022-11-07 09:07:14.678800: W tensorflow/compiler/tf2tensorrt/utils/py_utils.cc:38] TF-TRT Warning: Cannot dlopen some TensorRT libraries. If you would like to use Nvidia GPU with TensorRT, please make sure the missing libraries mentioned above are installed properly.
load data from data/train_small
packed pkl folder detected, will load from packed pkl file
Successfully Loaded from 19 files:19
max number of corners in single sample: 32
2 curves at least
448 valid curves total
250 valid corners total
225 patches total
min and max points in single patch: 512 512
0 open shapes
squared curve length statistics: 448 3.682233176771585e-05 9.269843007646973 0.3276165163696903
patch area statistics: 225 0.000722008498996729 1.3858339398102544 0.1676870428241
normal is included in input signal
load data from data/train_small
packed pkl folder detected, will load from packed pkl file
Successfully Loaded from 19 files:19
max number of corners in single sample: 32
2 curves at least
448 valid curves total
250 valid corners total
225 patches total
min and max points in single patch: 512 512
0 open shapes
squared curve length statistics: 448 3.682233176771585e-05 9.269843007646973 0.3276165163696903
patch area statistics: 225 0.000722008498996729 1.3858339398102544 0.1676870428241
normal is included in input signal
number of params: 22057152 87052323
Try to restore from checkpoint
  0%|                                                                                | 0/5 [00:00<?, ?it/s]Start Training
train data size 19
/workspace/data_loader_abc.py:248: NumbaWarning:
Compilation is falling back to object mode WITH looplifting enabled because Function "points2sparse_voxel" failed type inference due to: No implementation of function Function(<function norm at 0x7fb7228699d0>) found for signature:

 >>> norm(array(float32, 2d, A), axis=Literal[int](1), keepdims=Literal[bool](True))

There are 2 candidate implementations:
  - Of which 2 did not match due to:
  Overload in function 'norm_impl': File: numba/np/linalg.py: Line 2351.
    With argument(s): '(array(float32, 2d, A), axis=int64, keepdims=bool)':
   Rejected as the implementation raised a specific error:
     TypingError: got an unexpected keyword argument 'axis'
  raised from /root/.local/lib/python3.8/site-packages/numba/core/typing/templates.py:791

During: resolving callee type: Function(<function norm at 0x7fb7228699d0>)
During: typing of call at /workspace/data_loader_abc.py (255)

File "data_loader_abc.py", line 255:
def points2sparse_voxel(points_with_normal, voxel_dim, feature_type, with_normal, pad1s):
    <source elided>
    voxel_coord = np.clip(np.floor(points / voxel_length).astype(np.int32), 0, voxel_dim-1)
    points_normal_norm = linalg.norm(points_with_normal[:,3:], axis=1, keepdims=True)
    ^

  @numba.jit()
/root/.local/lib/python3.8/site-packages/numba/core/object_mode_passes.py:151: NumbaWarning: Function "points2sparse_voxel" was compiled in object mode without forceobj=True.

File "data_loader_abc.py", line 249:
@numba.jit()
def points2sparse_voxel(points_with_normal, voxel_dim, feature_type, with_normal, pad1s):
^

  warnings.warn(errors.NumbaWarning(warn_msg,
/root/.local/lib/python3.8/site-packages/numba/core/object_mode_passes.py:161: NumbaDeprecationWarning:
Fall-back from the nopython compilation path to the object mode compilation path has been detected, this is deprecated behaviour.

For more information visit https://numba.readthedocs.io/en/stable/reference/deprecation.html#deprecation-of-object-mode-fall-back-behaviour-when-using-jit

File "data_loader_abc.py", line 249:
@numba.jit()
def points2sparse_voxel(points_with_normal, voxel_dim, feature_type, with_normal, pad1s):
^

  warnings.warn(errors.NumbaDeprecationWarning(msg,
  0%|                                                                                | 0/5 [00:05<?, ?it/s]
Traceback (most recent call last):
  File "Minkowski_backbone.py", line 4582, in <module>
    mp.spawn(pipeline_abc,
  File "/opt/conda/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 199, in spawn
    return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
  File "/opt/conda/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 157, in start_processes
    while not context.join():
  File "/opt/conda/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 118, in join
    raise Exception(msg)
Exception:

-- Process 0 terminated with the following error:
Traceback (most recent call last):
  File "/opt/conda/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 19, in _wrap
    fn(i, *args)
  File "/workspace/Minkowski_backbone.py", line 4370, in pipeline_abc
    patch_loss_dict, patch_matching_indices = patch_loss_criterion(patch_predictions, target_patches_list)
  File "/opt/conda/lib/python3.8/site-packages/torch/nn/modules/module.py", line 727, in _call_impl
    result = self.forward(*input, **kwargs)
  File "/workspace/Minkowski_backbone.py", line 2532, in forward
    losses.update(self.get_loss(loss, outputs, targets, indices, num_corners))
  File "/workspace/Minkowski_backbone.py", line 2495, in get_loss
    return loss_map[loss](outputs, targets, indices, num_patches, **kwargs)
  File "/workspace/Minkowski_backbone.py", line 2220, in loss_geometry
    loss_geom[uclose_id] = emd_by_id(target_patch_points_batch[uclose_id], src_patch_points[uclose_id], self.emd_idlist_u, points_per_patch_dim)
RuntimeError: CUDA out of memory. Tried to allocate 4966.70 GiB (GPU 0; 23.69 GiB total capacity; 9.58 GiB already allocated; 12.36 GiB free; 9.62 GiB reserved in total by PyTorch)

oucyz

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

hi, oucyz. I've not encounter this issue before. Have you tried to forward the checkpoint directly by running: ./scripts/test_default.sh?


Update static.md

It doesn't work with prefixed slash.

I think this should work with latest echo (v4).
Needs confirmation.

@zdebra Did you had the chance to check if it works on v4?


Jedis socket settings causing issues under load ?

While under load, we have noticed that some jedis connections raise unexpected exceptions such as 'classcastexception' and 'numberformatexception' while running redis operations. These error can occur if multiple threads acces the same Jedis instance, but this is not the case for us.
The scenario we have seen is that some connections (we use a pool of connections) raise a timeout to an operation (eg zcore). As a result we close that connection and remove it from the pool.
To handle new load, a new connection is created (new jedis instance, probably using the same local port), and that new instance seems to receive some of the responses that where meant for the 'old' connection.
This is normally not possible with TCP, but is is possible with the sockets created by Jedis:
socket.setReuseAddress(true);
socket.setKeepAlive(true); // Will monitor the TCP connection is
// valid
socket.setTcpNoDelay(true); // Socket buffer Whetherclosed, to
// ensure timely delivery of data
socket.setSoLinger(true, 0); // Control calls close () method,
I have no strong evidence that the socket settings are the root cause of the problems we occasionally see but I wonder why client connections have all these fancy settings. Most literature i find on the internet does not recommend messing with these, especially the reuseAddress & linger settings.
Redis / Jedis Configuration
Jedis version:
1.8.1
Redis version:
3.0.7
Java version:

@janssk1
Are you implementing your own pool or using Jedis pre-built ones ?
By connection timeout you mean some form of jedis exception  or socket timeout ?
How are you closing the connection ? How are you re-instantiating a new one ?
Can you provide means of reproducing ?
I myself have  been using Jedis object pools in a highly concurrent environment ( doznes of servers , each sending 1000 of requests per second )  and encountered no such issues , so this scenario intrigues me

Closing due to inactivity.


Implement transformation to record synonymous constants.

#3132
This pull request only includes the code related to TransformationRecordSynonymousConstants and related tests (after feedback for #3486 , with changes based on the feedback from code review).

@stefanomil The CI has failed due to formatting.  Please search for "Code should be formatted with clang-format." in README.md for instructions on how to get that set up.

I have implemented the suggested changes. In the test file, since the transformation is applied many times, I wrote a function to apply the transformation and check the results. Let me know if that is ok or if there are any downsides in terms of readability or other.

@Vasniktel please see the transformation added here.  We will need to adapt this so that - when we have IdIsIrrelevant facts - we do not allow facts that make constants synonymous where either one is irrelevant.

Thanks, I'll take a look.


Districts in West Africa with reported Ebola outbreaks needs Access and Use Constraints

Districts in West Africa with reported Ebola outbreaks id=b8966bd015e64a6d9576a57573c9610e needs Access and Use Constraints
@karlterrey can help with demo content wording; however, we still need to cite data

@nfeuerstein @jrweakland @karlterrey @sschwoppe @lfunkhouser @elinz
Based on the review from legal I am going to delete all Ebola content at 1100 today.

I deleted this item.


 I think it should actually be merged but not recognizing :( :( :(

Okay here's my launch list. I haven't slept in like 30 hours so its a mess. I'll circle back tonight, but at least its in writing so we can conquer.
I have no idea whats going on with the branch. I know that I pulled in your branch - or at least I thought I did. it's definitely got the new random stuff now. 😭
https://gist.github.com/tayvano/b66fd9c327a9498573249c071f041d46

closing, migrated to #732 for version 3.10


blob triggered azure functions are supported in KEDA according to documents it should by tried it out its not working

Azure documentation issue guidance
Thanks for opening an issue in the Azure technical documentation repository.
We use GitHub issues as the primary channel for customer and community feedback about the Azure documentation.
Creating an issue
We prefer that you create documentation feedback issues using the Feedback link on the published article - the feedback control on the doc page creates an issue that contains all the article details so you can focus on the feedback part.
You can also create a feedback issue here in the repo. If you do this, please make sure your issue lists:

[ ] The relevant Azure service or technology.
[ ] A link to the published documentation article that you have feedback about.
[ ] Clear, specific feedback that the author can act on.

Pull requests and article contributions
If you know the change that is needed in an article, we encourage you to submit the changes directly using a pull request. If the change is large, or if you want to contribute an entire article, follow these guidelines:

[ ] Don't surprise us with a big pull request or a pull request with a new article! Submit an issue that describes the details of the proposed large change or new article.
[ ] Include the service or technology area.

We'll route the issue to the appropriate content team for review and discussion.
Tech support and product feedback
If you would like to contact Microsoft about other things, such as product feedback or tech support, please review these guidelines:

If you need technical support using Azure, the paid and free support options are described here: https://azure.microsoft.com/support/options/.

Each article in the Azure technical documentation contains a product feedback button - it's best to submit product feedback directly from a relevant article. Otherwise, you can submit product feedback for most Azure products in the following product feedback forum: https://feedback.azure.com/forums/34192--general-feedback.

Thank you for your feedback! Since this channel is for driving improvements towards MS Docs, could you provide the URL of the specific documentation that your feedback is applicable to? That way, we are able to make sure your feedback is addressed by the right team.

Added the details followed the steps for deploying in keda
Un able to see the hpa pod or event it's not triggered also

Duplicate of #54352


Alpha Encode

HI
Anyone please help me to encode the video in alpha mode using FFmpeg params
Regards
Senthuran

ffmpeg  -c:v qtrle -r 30 "out.mov"

thank you so much
On Sun, Apr 4, 2021 at 4:48 PM crisdeb @.***> wrote:

ffmpeg -c:v qtrle -r 30 "out.mov"
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/inlife/nexrender/issues/560#issuecomment-813015741,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBYW2MBC2OC2T5XGM7QZU3THBDIHANCNFSM4XJHTU7Q
.


No module named 'modelsnet'

Hi, @diegovalsesia ,
Thanks for releasing the package. When I run the command python main_dgcnn_hycore.py --epoch=250 --learning_rate 0.1 --msg=Offv_dgcnn_hycore_var --workers=8, I got the following error:
(hycore) root@fa0e99c9a2ce:/data/code13/HyCoRe/classification_ScanObject#  python main_dgcnn_hycore.py --epoch=250 --learning_rate 0.1 --msg=Offv_dgcnn_hycore_var --workers=8
Traceback (most recent call last):
  File "main_dgcnn_hycore.py", line 21, in <module>
    import models as models
  File "/data/code13/HyCoRe/classification_ScanObject/models/__init__.py", line 3, in <module>
    from .pointmlp import pointMLP, pointMLPElite, Hype_pointMLP
  File "/data/code13/HyCoRe/classification_ScanObject/models/pointmlp.py", line 7, in <module>
    from modelsnet.ops.manifold_layers import GeodesicLayer, MobiusLayer, LogZero, ExpZero, GyroplaneConvLayer
ModuleNotFoundError: No module named 'modelsnet'
(hycore) root@fa0e99c9a2ce:/data/code13/HyCoRe/classification_ScanObject#

Any hints to fix this issue?
Thanks~

It was a leftover typo from a change in the directory name. I fixed it now.


Fix some typos in the prose

This corrects some minor spelling errors I found in the prose.

Thanks, Ralienpp!


How to get alert hits in enhancement ?

hi all,
I have create a rule that will alert only if get 10 failures have occurred for a given user.
   filter:
      - term:
             _type: log
       - term:
             outcome: failure
   type: frequency
   num_events: 10
   timeframe:
   hours: 1
   query_key: username

but how can I get these 10 failure hit documents  in enhancement ?

Did you read http://elastalert.readthedocs.io/en/latest/recipes/adding_enhancements.html#enhancements?

Qmando,
I have see that , but  it's will get the last record that exceed the threshold.
but I want to get all records.
for example ,
Record _1 match the filter
Record_2 match the filter
..........
Rdcord_10 match the filter
-------it's will alert
Record_11 match the filter
but in enhancement only can get Record_11 by match array
but I want Record_1 to Record_10 all Records.
how can I get these 10 failure hit documents in enhancement ?

attach_related: true


Add a semver statement to the README.

Addresses #21.

Current coverage is 77.96% (diff: 100%)

Merging #23 into master will not change coverage

@@             master        #23   diff @@
==========================================
  Files            24         24
  Lines           785        785
  Methods           0          0
  Messages          0          0
  Branches        102        102
==========================================
  Hits            612        612
  Misses          140        140
  Partials         33         33

Powered by Codecov. Last update 067480f...269d6a2

Looks good to me, @criccomini any thoughts?

This looks good to me, and @criccomini is on vacation till next week, so I'm going to merge it.


[Executors] Remote distributed actors get "crash on enqueue" default executor

This follows up on SE review feedback that we should use an executor that crashes on enqueue, rather than an optional property for the unownedExecutor of a distributed actor

There was good discussion of what should happen when code asks a distributed actor instance what its executor is. The question doesn't really make sense to ask of a remote actor instance, since the only code that locally executes on its behalf is the nonisolated remote call stub, and it doesn't make sense to try to enqueue additional local work on a remote actor's executor. One possibility would be to have the unownedExecutor property of an actor be Optional, and have distributed actor instances return nil, but the Language Workgroup agrees with the proposal's choice not to take this route, since doing so would add friction to code that only works with local actors. Instead of returning a "default" executor, though, the language workgroup would like to see if it's possible to use an executor implementation that raises a fatalError if any code tries to enqueue additional jobs on it. Although it may make sense to ask an arbitrary actor for its executor to log the value as a debugging aid, it is likely a programming mistake to attempt to use that executor reference to enqueue work, so it may be preferable to trap at the attempt rather than to fall back to some default executor, to avoid expectations that the work will actually occur exclusively with work being done on the remote actor instance.

Associated SE update: https://github.com/apple/swift-evolution/pull/2000

@swift-ci please smoke test

@swift-ci please smoke test

I think i messed up rebasing somehow, will fix in the morning

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

This should also calm down this flaky test actually: rdar://107631491


Expand JsonToGrpcGatewayFilterFactory.resourceLoader to load all proto files under classpath

Can ResourceLoader be supported to load all *. proto files under the classpath * path and cache them

I don't think that is a good idea. What about a list of them?

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

Closing due to lack of requested feedback. If you would like us to look at this issue, please provide the requested information and we will re-open the issue.


API Gateway deployment with CodePipeline

I would like to create the following two stages for the tile delivery API,

development

always calls the latest Lambda functions

production

calls the latest approved Lambda functions

And I would like to deploy an API instance in a CodePipeline pipeline. A simple static CloudFormation template does not work for deployment. Because a stage rolls back to the initial deployment every time a template is updated unless the resource name of a deployment is changed.
Maybe I have to introduce a Lambda function that deploys an API instance.

I can use a stage variable to specify a Lambda function version.
NOTE: If I add a version qualifier in a Lambda invocation URI, a Lambda permission specific to that version has to be granted.

The AutoPublishAlias property of AWS::Serverless::Function may be utilized.


Gifts falling animation

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes:  #694

Type of change

[x ] New feature (non-breaking change which adds functionality)

Checklist:

[ x] I have made this from my own
[ x] My code follows the style guidelines of this project
[x ] I have performed a self-review of my own code
[ x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Please merge, #713 , instead of this PR, I created this by mistake, hence I am closing it.


[Feature Request] Remote repository URL for specific line(s) of code

GitHub recently provided a feature in their web application, wherein we could drag and select lines of code and copy the exact URL.
(previously we could only copy the URL from the search bar, however, now we can get the URL that always points to that piece of code, even if there are a lot of changes being made to that file)
This is done simply by using the commit hash. For example:
https://github.com/eamodio/vscode-gitlens/blob/54b2c06c1b8e4f573ed4495f42029c9df146efb0/LICENSE#L1 => https://github.com/eamodio/vscode-gitlens/blob/54b2c06c1b8e4f573ed4495f42029c9df146efb0/LICENSE#L1
(there’s also https://github.com/eamodio/vscode-gitlens/blob/main/LICENSE#L1)
When we’re debugging code on the web client, it’s all fine, but if we’re in a VSCode environment, currently there’s (I think) no way we can select a piece of code and get the remote repository URL.
(it sounds pretty straightforward (which I'm sure it's not): local git connection + getting the remote URL + using the current commit hash)
This isn’t a useless request either, since, on GitHub web client these URLs render as code snippets (which is the end goal).
^see rendered snippet for example
Being a useful utility as it is, I don’t think this is discussed anywhere (I might be searching wrong keywords), and I’m quite surprised this isn’t already baked in by default.
If this sounds like a feasible request, I could try and pitch in! :D

@aitikgupta GitLens already supports this via the Copy Remote Commit Url. Just select the code you want and either run the command from the command palette or right-click or configure a keybinding for the command (for example, I use Ctrl+Shift+C).

Hey @eamodio, thanks for reaching out.
However, what you're pointing out isn't quite what I want/suggest. Sure, we can get the remote 'commit' URL using the method you suggested, but we still can't get those exact lines as a remote URL.
I'm not sure how I can explain it better, the end goal is that rendered snippet (in my original comment) from VSCode.
Remote 'commit' URL just points to the actual commit, whereas the suggestion is to add something which points to the actual blob (and the specific line(s)).

Ugh, sorry I got mixed up -- Use Copy Remote File Url -- basically everything I said applies, I just said the wrong command

Ugh, sorry I got mixed up -- Use Copy Remote File Url -- basically everything I said applies, I just said the wrong command

I should've figured it out, my bad. Thanks again! :D


Error exporting certificate in staging - Can not find issuer

When trying to export the certificate i get the following error when using staging.
var certPem = cert.ToPem();

Certes.AcmeException: Can not find issuer 'C=US,O=(STAGING) Internet Security Research Group,CN=(STAGING) Doctored Durian Root CA X3' for certificate 'C=US,O=(STAGING) Internet Security Research Group,CN=(STAGING) Pretend Pear X1'.
   at Certes.Pkcs.CertificateStore.GetIssuers(Byte[] der)
   at Certes.CertificateChainExtensions.ToPem(CertificateChain certificateChain, IKey certKey)

I found this which explains why.
https://community.certifytheweb.com/t/error-building-certificate-as-pfx-cannot-find-issuer/1253

Duplicate


Feature Request: SystemVerilog Plugin

It would be awesome to have a better integration of SystemVerilog in openlane2 through the SystemVerilog Plugin for Yosys: https://github.com/chipsalliance/yosys-f4pga-plugins/tree/main/systemverilog-plugin
This could be enabled/disabled via a configuration option.

I think this can be closed now that #227 has been merged. Thanks again!


Do not place commas in 88 continue'ed text

hi,
I was noticing random, unexpected commas in transaction text fields parsed by bai_parser.
ex:
WIRE REFERENCE:<PHONE_NUMBER>3,0000041275XXXXX FOOBAR CORPORA,TION MAIN ACCOUNT 123 SOME ST, 4TH FL, SAN DIEGO CA 95,108 EC1502353021118 ORG=Foobar Funding ...

turns out this sub! highlighted in the diff was replacing the line delimiter / of the continue'ed record (the preious line) by a comma. As far as I can tell this is incorrect, and is introducing a comma that was not in the original text.
I'm using this doc as reference in which i fail to notice such a requirement. https://www.bai.org/libraries/site-general-downloads/cash_management_2005.sflb.ashx
the above line is now produced as I expected it, without the unexpected commas:
WIRE REFERENCE:<PHONE_NUMBER>30000041275XXXXX FOOBAR CORPORATION MAIN ACCOUNT 123 SOME ST, 4TH FL SAN DIEGO CA 95108 EC1502353021118 ORG=Foobar Funding ...

Hi
Thanks for the pull request!  I'm looking at the reference doc you linked and reading about the continuation record on page 20:
"Delimiters are the same as for the preceding record. If the preceding physical record does not end within a text field, that record should end with a delimiter slash “/”, even if the following Continuation (88) includes or begins with text. If the preceding physical record ends within text that is to be continued, no delimiter should be placed at the end of that record."
So in your case it sounds like the preceding record ends within a text field that is to be continued but sounds like it incorrectly ends with a delimiter, '/'?

you are right, my partner bank does it wrong. I've opened their supporting documentation and their examples show that they always add a trailing delimiter, contrary to what the bai.org spec specifies (and that you read better than I).
So i guess this issue can be closed. I can fork the repo to handle my bank's case. Sorry for the noise.
Unless you think it would be acceptable for bai_parser to support the following case: if you you are parsing the final field of record, and that field is a TEXT field, then ignore any / delimiters.
Also another option would be to support a spec deviation, possibly in the form of an option to the parse method, that would allow such delimiters.
I understand that branching off the official spec is a decision that should not be taken lightly, but note that this irregular spec behavior is exhibited by a top 5 US bank (as listed here, so this is not such a narrow use case. I'm not sure I am allowed to share the bank's name or institution (sorry), but I will get confirmation as soon as I get to speak with my colleagues next week, and will hopefully share their spec documentation, should you be interested in exploring this any further.

I have to be honest I'm having a hard time being sure about anything, this is my first day looking into the BAI file format. I'm going to keep verifying my understanding of all this over the next few days.


Shopware 5.2.8 causes error on Bootstrap.php

Shopware Version 5.2.8
Clean Shop, without any other Plugins.
Are you able to figure out, what's the problem here? We really appreciate the new Lightweight functionality - but can't use it at the moment.
Thanks in Advance!
Controller "ExampleModulePlainHtml" can't load failure in engine/Shopware/Components/Plugin/Bootstrap.php on line 578

Stack trace:

#0 engine/Library/Enlight/Event/Handler/Plugin.php(149): Shopware_Components_Plugin_Bootstrap->getDefaultControllerPath(Object(Enlight_Event_EventArgs))
#1 engine/Library/Enlight/Event/EventManager.php(251): Enlight_Event_Handler_Plugin->execute(Object(Enlight_Event_EventArgs))
#2 engine/Library/Enlight/Controller/Dispatcher/Default.php(368): Enlight_Event_EventManager->notifyUntil('Enlight_Control...', Array)
#3 engine/Library/Enlight/Controller/Dispatcher/Default.php(447): Enlight_Controller_Dispatcher_Default->getControllerPath(Object(Enlight_Controller_Request_RequestHttp))
#4 engine/Library/Enlight/Controller/Dispatcher/Default.php(485): Enlight_Controller_Dispatcher_Default->isDispatchable(Object(Enlight_Controller_Request_RequestHttp))
#5 engine/Library/Enlight/Controller/Front.php(223): Enlight_Controller_Dispatcher_Default->dispatch(Object(Enlight_Controller_Request_RequestHttp), Object(Enlight_Controller_Response_ResponseHttp))
#6 engine/Shopware/Kernel.php(177): Enlight_Controller_Front->dispatch()
#7 vendor/symfony/http-kernel/HttpCache/HttpCache.php(487): Shopware\Kernel->handle(Object(Symfony\Component\HttpFoundation\Request), 1, true)
#8 engine/Shopware/Components/HttpCache/AppCache.php(255): Symfony\Component\HttpKernel\HttpCache\HttpCache->forward(Object(Symfony\Component\HttpFoundation\Request), true, NULL)
#9 vendor/symfony/http-kernel/HttpCache/HttpCache.php(258): Shopware\Components\HttpCache\AppCache->forward(Object(Symfony\Component\HttpFoundation\Request), true)
#10 engine/Shopware/Components/HttpCache/AppCache.php(103): Symfony\Component\HttpKernel\HttpCache\HttpCache->pass(Object(Symfony\Component\HttpFoundation\Request), true)
#11 shopware.php(113): Shopware\Components\HttpCache\AppCache->handle(Object(Symfony\Component\HttpFoundation\Request))
#12 {main}

I can reproduce the error, hope you can fix it soon!
Thanks in Advance

Thanks for reporting, this issue should be fixed with commit https://github.com/shopwareLabs/SwagLightweightModule/commit/f57bbdcc964d5c28039716f49d22e1aee868ab6c.

This error was caused by commit https://github.com/shopwareLabs/SwagLightweightModule/commit/70ded8562b06c98184ce6aba8c1cfdfddbb33177.


Create example.dart

The pub site can show example code for a package.
https://www.dartlang.org/tools/pub/package-layout#examples
related issue : dart-lang/site-www#413

Closing this as outdated. Happy to see a follow-up!


Finish up version 1.0.0

There will possible be more minor versions before this, but most the functionality is in place for version 1.0.0.
Outstanding work includes:

Add SqliteContext and use instead of QueryHandler.
Add documentation.
Add tests (if possible).

QueryHandler has now been marked as obsolete. Going forwards SqliteContext should be used. QueryHandler will be removed before version 1.0.0 so version 0.5.0 will be needed (still with QueryHandler for projects relying on it.

More for cleaning up API:

Remove unneeded select methods.

For example: _.SelectSingle<'T> (marked as obselete)

Rename _.Select<'T>  to _.Select<'T>.
Make different types overrides(?)

Added build pipeline via forge. Also updated code generation to be more standardized (/opinionated).


Typing, phase 3: Python 3.11, TypeVarTuple, recursive types

[x] Wait for Python 3.11 to be integrated into both Pyodide and JupyterLite.
[ ] Start using TypeVarTuple.
[x] Should generator counts be covariant or invariant? Should they be restricted to int?

Covariant, we can punt on int restriction since we need variable type to represent tuples of varying size anyways.

[x] Recursive types

Seems to work well for the return type of #128. Not sure if it will make Die / Deck constructors type-checkable though.
I ended up removing the recursive interpretation of arguments anyways.

[x] Put all TypeVars in the common file?
[x] Export typing?

Done.

[x] Add explicit @override.

PEP 698 is accepted, but won't be implemented until 3.12. We'll delay this to another phase.

[ ] Typing for tests?

This might catch some typing bugs. Unfortunately, type inference seems to have a hard time.

[ ] map overloads

[ ] Invariant Population?

These should really be covariant but the handling of this is kind of jank, and I don't think the covariance is actually used all that often.

I think this is about as far as I'm willing to wrangle it in 3.11.


test/func: assertion failure when SSA-built

test/func.go currently seg faults when SSA-built and then executed on a sparc64 system.
It is cross-compiled and cross-linked for sparc64 from an amd64 system using:
SSATEST=1 GOSSAHASH="" GOOS=solaris GOARCH=sparc64 go tool compile func.go
GOOS=solaris GOARCH=sparc64 go tool link func.o

When executed it fails as follows:
$ ./func
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x8 pc=0x8]

goroutine 1 [running]:
panic(0xcdca0, 0x110dc0)
        /builds/srwalker/golang/go-sparc/src/runtime/panic.go:507 +0x5f8

This was due to a bad FSTOI/FDTOI or FSTOX load/store related to the problems found in resolving #23.
Fixed.


Logo for coding books

Create a high quality SVG or PNG logo for coding books
Compressed the logo
make sure you reach out to the maintainer and share the first look of that logo.

Go ahead @Hussain6

Logo has been added.

Thanks for contributing @Hussain6. Your pull request is merged to the master branch, but we want our coding books logo UI similar to books and all...

I want to do this. @mohitjaisal Please assign

Go ahead @prakhart111 👍


3.0.0 head root CA cert generation fails to set AKID to "keyid".

$ openssl x509 -req -sha256 -out rootcert.pem -extfile exts.cnf \
    -signkey rootkey.pem -set_serial 1 -days 30 -in req.pem

fails with:
Error adding extensions from section default
007E7E0D01000000:error:100C0102:BIO routines:BIO_puts:passed a null parameter:../openssl/crypto/bio/bio_lib.c:406:
007E7E0D01000000:error:100C0102:BIO routines:BIO_puts:passed a null parameter:../openssl/crypto/bio/bio_lib.c:406:
007E7E0D01000000:error:1100007B:X509 V3 routines:v2i_AUTHORITY_KEYID:unable to get issuer keyid:../openssl/crypto/x509/v3_akid.c:145:
007E7E0D01000000:error:11000080:X509 V3 routines:X509V3_EXT_nconf_int:error in extension:../openssl/crypto/x509/v3_conf.c:48:section=default, name=authorityKeyIdentifier, value=keyid

given a req.pem containing:
-----BEGIN CERTIFICATE REQUEST-----
MIHMMHQCAQAwEjEQMA4GA1UEAwwHUm9vdCBDQTBZMBMGByqGSM49AgEGCCqGSM49
AwEHA0IABBtsOGoMvJutXATlHANTkIsfZpoMP56JDV2oS6CscZ9RQB3tWNi5IE1T
kJBTNeD0z3VVk6F6RFpTfrmtWqQwwW+gADAKBggqhkjOPQQDAgNIADBFAiEAjmaK
oYGl5R88NH/UIWGVUmspkJYzD1SkBMSUhhhkwqwCIB6SloEb07S8/mOqFpdqjwQb
YwUJiCExG+aLzl+qctRn
-----END CERTIFICATE REQUEST-----

an exts.cnf containing:

authorityKeyIdentifier  = keyid
basicConstraints = CA:true

and a rootkey.pem containing:
-----BEGIN PRIVATE KEY-----
MIGHAgEAMBMGByqGSM49AgEGCCqGSM49AwEHBG0wawIBAQQgIO+WeBhKWkyZQnyB
iw7EBtAUWUXH47aItOp7m8Ve1SOhRANCAAQbbDhqDLybrVwE5RwDU5CLH2aaDD+e
iQ1dqEugrHGfUUAd7VjYuSBNU5CQUzXg9M91VZOhekRaU365rVqkMMFv
-----END PRIVATE KEY-----

The same invocation works fine with e.g. 1.1.1 and 1.0.2

Note that changing exts.cnf to add an SKID, fixes the problem for master:

subjectKeyIdentifier = hash
authorityKeyIdentifier  = keyid
basicConstraints = CA:true

So it looks like the attempt to compute the issuer keyid fails when the self-signed cert's SKID is not requested, but "keyid" is documented as optional when not marked with ":always":
    If keyid is present, an attempt is made to compute the hash of the
    public key corresponding to the signing key in case the certificate is
    self-signed, or else to copy the subject key identifier (SKID) from the
    issuer certificate. If this fails and the option always is present, an
    error is returned.

That said, in 1.0.2 and 1.1.1 the generated certificate has:
        X509v3 extensions:
            X509v3 Authority Key Identifier:
                0.
            X509v3 Basic Constraints:
                CA:TRUE

Which is not exactly ideal. So for a self-signed root having an AKID of "keyid" without an SKID of "hash" (or explicit) is perhaps not a good idea.  But it used to "work" for 1.0.2 and 1.1.1.
IMHO the correct behaviour is to suppress the AKID when there's not going to be a SKID and "keyid" does not have a ":always" suffix.

Why not just compute the hash? This is a self-signed certificate generation, isn't it? @DDvO any opinon on this one?

Why not just compute the hash? This is a self-signed certificate generation, isn't it? @DDvO any opinon on this one?

On reflection, I think the correct behaviour when just keyid (and not keyid:always) is specified for a self-signed certificate may be to not create an AKID if the subjectKeyIdentiifier is not also set (since keyid in AKID is conditional on the issuer having a SKID).
That said, while the above is pedantically correct, the user may be expecting an automatic subjectKeyIdentifier = hash to happen when the keyid AKID is requested.  So that choice would also be fine.
Both generating an AKID of 0 or blowing up seem suboptimal. :-(

Why not just compute the hash? This is a self-signed certificate generation, isn't it? @DDvO any opinon on this one?

On reflection, I think the correct behaviour when just keyid (and not keyid:always) is specified for a self-signed certificate may be to not create an AKID if the subjectKeyIdentifier is not also set (since keyid in AKID is conditional on the issuer having a SKID).

I see this as the most clean solution.

On reflection, I think the correct behaviour when just keyid (and not keyid:always) is specified for a self-signed certificate may be to not create an AKID if the subjectKeyIdentifier is not also set (since keyid in AKID is conditional on the issuer having a SKID).

I see this as the most clean solution.

I agree, are you willing to have a go at the requisite PR?

I meant, in the sense that a more to-the-point error message should be thrown. Will do.
And how to handle keyid:always ?

So on keyid no error will be thrown and no AKID produced, and on keyid:always a more to-the-point error message will be thrown.

So on keyid no error will be thrown and no AKID produced, and on keyid:always a more to-the-point error message will be thrown.

Correct, when just keyid (without subjectKeyIdentifier = ...) is requested, and the cert is self-signed, the keyid should be ignored.
With keyid:always an error should be given indicating the missing SKID.  A likely complication is that the AKID and SKID might be requested in either order, which means that AKID processing needs to be sometimes delayed (if not already).

So on keyid no error will be thrown and no AKID produced, and on keyid:always a more to-the-point error message will be thrown.

Correct, when just keyid (without subjectKeyIdentifier = ...) is requested, and the cert is self-signed, the keyid should be ignored.

Yes (and essentially the same should hold for not self-signed certs with issuers not having an SKID.

With keyid:always an error should be given indicating the missing SKID. A likely complication is that the AKID and SKID might be requested in either order, which means that AKID processing needs to be sometimes delayed (if not already).

Good point, will take care of that.

BTW, the spurious errors
007E7E0D01000000:error:100C0102:BIO routines:BIO_puts:passed a null parameter:../openssl/crypto/bio/bio_lib.c:406:
007E7E0D01000000:error:100C0102:BIO routines:BIO_puts:passed a null parameter:../openssl/crypto/bio/bio_lib.c:406:

mentioned in the OP are due to an unfortunate design decision on BIO_puts(NULL, ...) and will be handled in a separate PR.

I also found that the configuration parsing for AKIDs is incomplete; this will also be handled in a separate PR.


Error appears when attempting to fetch browser history from specific browser using fetch [BUG]

Describe the bug
Method for fetching specific browser history doesn't work
Error:
TypeError: fetch() missing 1 required positional argument: 'self'

To Reproduce
Steps to reproduce the behavior:
Followed the same steps as in README.
from browser_history.browsers import Chrome
f = Chrome.fetch()

Expected behavior
No errors
Desktop (please complete the following information):

OS: Ubuntu
Browser Firefox
browser-history Version v0.2.0,
Python version:  3.7.2

Oops! That's a mistake in the documentation.
Browsers must be initialized before using them (instead of using the classes directly).
from browser_history.browsers import Chrome

c = Chrome()

his = c.fetch().get()

Fixed in #29


Unsafe pointer create exception

Hi,
i tried to use the newest version of sqlite3 and get this error:

my code looks like this:
import { Database } from<EMAIL_ADDRESS>
const db = new Database("test.db");

db.close();

my runtime args looks like this:
deno run -A --unstable test.ts
Did i miss something ?

That's weird.. are you on latest deno version?

Version is up to date.

Seems to be a problem with vscode launch.json.
{
    "version": "0.2.0",
    "configurations": [

        {
            "request": "launch",
            "name": "Deno",
            "type": "node",
            "program": "${workspaceFolder}/test.ts",
            "cwd": "${workspaceFolder}",
            "runtimeExecutable": "C:\\Users\\6rube\\.deno\\bin\\deno.EXE",
            "runtimeArgs": [
                "run",
                "-A",
                "--unstable"
            ],
            "attachSimplePort": 9229
        }
    ]
}

I suspect the --unstable argument is not getting passed correctly, as this error is only reproducible when --unstable is not passed. Either way, seems like it's not a problem in sqlite3 module.


add user balances in deposit

Add user balances from: https://github.com/weweswap/app-page/issues/103
We can still do some adaptions on the decimals etc. if you want

Summary by CodeRabbit

Refactor

Improved code organization and readability in the PoolDeposit component.
Enhanced formatting for props, hooks, and rendering logic.
Streamlined token balance fetching and restructured deposit/withdraw sections for clarity.

Pls check conflicts here @masterjacy01


oauth2: got http status 403 on google get user info

Caddyfile:
localhost:3000 {
  proxy / localhost:3040
  jwt {
    path /
    allow domain readdle.com
  }
  login {
    google "client_id=***,client_secret=***,scope=https://www.googleapis.com/auth/userinfo.profile"
  }
}

Upon login it shows "Internal error" and an error in log: oauth2: got http status 403 on google get user info
Also tried these scopes: https://www.googleapis.com/auth/plus.me, https://www.googleapis.com/auth/plus.login with same effect.
Client ID and Client Secret are created for a fresh sample project. Same Client ID and secret work  for bitly/oauth2_proxy, but I'd prefer to use caddy+loginsrv in our setup.

Also having this issue, tried with the following permissions (both with and without custom domain):
email
profile
openid
https://www.googleapis.com/auth/userinfo.email
https://www.googleapis.com/auth/userinfo.profile

Custom domain has API's enabled, and the OAuth app in Google Cloud has no additional settings that would change any behaviour.

I've managed to fix this, @domudall
I've enabled Google Plus API and Google People API. It worked after a minute or so. Works great for me now.

@dobegor Good to here that!!
Maybe you could file a PR for the README, so that others do not have to figure that out by there self?

@smancke sure, will do that. Just need to check if both APIs are required to work. Google's OAuth 2 Scopes doc has absolutely zero info on this, unfortunately.

Amazing, thanks all. Managed to get it working with just the Google+ API (tried just People first, then disabled). Here's a direct link to enable for anyone who finds this issue before the README PR: https://console.cloud.google.com/apis/library/people.googleapis.com/

People from future,
Please enable Google's Plus API, NOT people API. This is true when you need only the email of the user.
As the scope use https://www.googleapis.com/auth/userinfo.email on the line
google "client_id=......,client_secret=.....,scope=https://www.googleapis.com/auth/userinfo.email   inside the login { } block on Caddyfile.
You're welcome.

What is the replacement API now that Google+ has been deprecated? It's very unclear in Google's own documentation.

Hi @rochdev
as mentioned in the README:

** Attention: Update to v1.3.0 for Google Login Update !!!! **
Google will stop support for the Google+ APIs. So we changed loginsrv to use the standard oauth endpoints for Google login. Please update loginsrv to v1.3.0 if you are using google login.


1.3.x uses too much RAM to work on SAMD21

A demo by Mirko Pacioni used to compile fine on SAMD21 until 1.2.0.
I have noticed you considerably upped the buffers on 1.3.0
1.2.0
#define LZW_BUF_SIZE (6*MAX_CHUNK_SIZE)
#define LZW_HIGHWATER (4*MAX_CHUNK_SIZE)
#define MAX_WIDTH 320
#define FILE_BUF_SIZE 4096

1.3.0
#define LZW_BUF_SIZE (12*MAX_CHUNK_SIZE)
#define LZW_HIGHWATER (10*MAX_CHUNK_SIZE)
#define MAX_WIDTH 320
#define FILE_BUF_SIZE 8192

It would be nice to handle these things gracefully if you want to still support the SAMD21 :)
Thank you, mate :)

Thanks for catching that - it was an experiment to see how it affected the performance on "bigger" systems. I didn't mean to permanently change it. I'll fix it.


Summing up Time and Showing values in time format

Hi,
Great plugin!
I'd like to use it to display mysql-php output in json format. My table has data in time format (HH:MM:SS). I'd like to show these values and also sum up in the row Totals and column totals.
If I try Sum or Count or Min or Max, it returns nothing. However, if I try "List Unique Values", it shows the values correctly. But the Row Totals and Column does not work. Here is a screenshot. Can you help me guide in the right direction to get the totals as well.

As you see in the image above, the time is not summing up. This is the code I am trying:

$.getJSON("http://salesapp.matenek.com/salesmatenek/navtests/getInputForPivot.php", function(mps) {
        $("#output").pivotUI(mps,
        {
            cols: ["report_date"],
            rows: ["rid"],
            vals: ["time_spent"]
            })
    })

The current behaviour is explained by the fact that a string in the format "hh:mm:ss" is not a number, so is not summable, averagable, etc. "list unique values" will concatenate all unique values of the field with a comma as a divider, which is why you're seeing the output you're seeing.
In order to get the behaviour you're looking for you will either have to convert your input to numbers, and use the built-in aggregator templates with a custom number formatter, or you can roll your own aggregators which will parse "hh:mm:ss" to a number first if you really need to leave your input as-is.

Thanks for your ideas on how to approach this problem. From your suggestion, I choose the first option of converting my input to numbers and then once pivot is generated I created a script to manipulate the format (instead of using built-in aggregator templates and custom number formatter).
I had to take this route because of my limited abilities to understand how aggregator templates work. I would be grateful if you can post an example script.
Meanwhile, here is my workaround - which I guess is not a professional work.
Step 1: I wrote php to generate a json where time_spent is in Time-to-Seconds format (in other words, numbers).
Step 2: html (note that i am hiding output initially so your script runs in background and shows only changed format after i click a button.
        <button class="convert">Convert Seconds to Time</button>

Step 3: script where I am calling my script on button click because I couldn't hook my functions after your function is executed. I tried a lot to do that. As I failed, this is the work around I implemented.
    // This example is the most basic usage of pivot()
            var tpl = $.pivotUtilities.aggregatorTemplates;

        $.getJSON("http://salesapp.matenek.com/salesmatenek/navtests/getInputForPivot.php", function(mps) {
            $("#output").pivotUI(mps,
            {
                cols: ["report_date"],
                rows: ["rid"],
                vals: ["time_spent"],
                aggregators: {
                    "Time Spent":      function() { return tpl.sum()(["time_spent"]) },
                    "Number of Visits":      function() { return tpl.count()(["time_spent"]) }

                },

                })
            //console.log(mps);

        })

    $('button.convert').click(function(){

        function secondsTimeSpanToHMS(s) {
                var h = Math.floor(s/3600); //Get whole hours
                s -= h*3600;
                var m = Math.floor(s/60); //Get remaining minutes
                s -= m*60;
                return h+":"+(m < 10 ? '0'+m : m)+":"+(s < 10 ? '0'+s : s); //zero padding on minutes and seconds
            }

        function manipulateClass(y) {
            var divs = document.getElementsByClassName( y );
            [].slice.call( divs ).forEach(function ( div ) {
                var x = div.innerHTML; //fetch the text in the class pvtVal
                x=x.replace(/\,/g,''); // 1125, but a string, so convert it to number
                //x=parseInt(x,10);
                div.innerHTML = secondsTimeSpanToHMS(x); //send to function and get seconds converted to time
            	});
            }

        manipulateClass('pvtVal')
        manipulateClass('pvtTotal')
        manipulateClass('pvtGrandTotal')

        $('#output').show();
    });

I am sure my solution is lengthy and may not be the correct one. But it is giving the desired solution although not neat. I would be grateful if you can show an example of changing format in built-in aggregator.

Here is the documentation for localization, which is basically what you want to do with number formatting: https://github.com/nicolaskruchten/pivottable/wiki/Localization ... it contains a link to the aggregator/templates documentation: https://github.com/nicolaskruchten/pivottable/wiki/Aggregators
You can avoid using a button by using the onRefresh callback, which is called after every refresh.

Thank you again. onRefresh callback was very useful.


SDK: Merge main branch into feature/xplat_sdk

@shishirb-MSFT Going to merge main branch here into feature/do_xplat, do the namespace/c++14 changes in feature/do_xplat, then merge the public into the private.


3D ?

Hi,
When is 3D support added ?
Ty!

I'm afraid the only answer I have for you is "eventually." I have a list of things that need to be done, and whenever I finish one of them I grab something else. I'm just one person. 😄

I copied monogame stuff, if you want to have same design as XNA i can clean it and send a pull request it is ok for you ? :D

But one problem there is no lightning yet and no animation yet

I'm not sure I want to just copy MonoGame; one of the reasons I've been putting off 3D for so long is that I want to take the time to step back and do it in a way that makes sense for Ultraviolet.
Maybe you could put the work you've done into a separate NuGet package for people who need 3D in the meantime?

I never used nuget i don't know how to do that i think i'm just make git repo

do you think you can add 3d into ultraviolet in next update if i can pay ?
im strugling with 3D animation, this is so hard for me :(

Congrats on new job!
Ahhhh next is 3D ty!! i was trying for this last month but it's so hard i mean i follow different opengl tutorial but it is different and hard to translate to ultraviolet, and they are on c++ wich make it even harder for me


DBBACKUP_S3_SERVER_SIDE_ENCRYPTION option

Ability to request server side encryption for S3 objects.
https://docs.aws.amazon.com/AmazonS3/latest/dev/UsingServerSideEncryption.html

Wow, that was simple :)


Updated topview.py

Corrected Spelling mistake in topview.py, Please merge @ReimarBauer .

👋 hi @ReimarBauer,
I have done the necessary changes, if there is something else to update please provide info I will update it again and if no further changes are required please merge the repo.

👋 hi @ReimarBauer,
I have done the necessary changes, if there is something else to update please provide info I will update it again and if no further changes are required please merge the repo.


issue #53 add appender

Close #53

Codecov Report

Merging #54 (57748f9) into main (7446ae7) will increase coverage by 0.08%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   99.62%   99.71%   +0.08%
==========================================
  Files           7       10       +3
  Lines         796     1041     +245
==========================================
+ Hits          793     1038     +245
  Misses          3        3

Flag
Coverage Δ

unittests
99.71% <100.00%> (+0.08%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

source/karasux/appender.d
100.00% <100.00%> (ø)

source/karasux/parser/core/primitive.d
100.00% <100.00%> (ø)

source/karasux/parser/core/source.d
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7446ae7...57748f9. Read the comment docs.


Allow document start for yaml

Description

Resources are separated by --- when there are multiple resources in one yaml file, which is needed by PR #52. If we set document-start to false, yamllint won't allow the existence of --- in yaml file. Can we set document-start to true to allow PR #52 passing style check?
Are you willing to submit PR?
yes.

@Hooollin  can we just suppress the checking in your yaml file?

@Hooollin can we just suppress the checking in your yaml file?

It's feasible. I'll do that in #52.


[OTHER] Disable click on profile button if user is not logged in

What would you like to share?
I want to make the profile button non-clickable if the user is not logged in.
Currently, it shows "Please create an account" which is not good for user experience.
Additional information

👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

I would like to work on this. Assign it to me.

It's actually good user experience, thanks for raising not an accepted issue


NetWnv module is not available in W2022 datacenter

PS C:\Users\administrator.corp> Import-Module netWnv
Import-Module : The specified module 'netWnv' was not loaded because no valid module file was found in any module
directory.
At line:1 char:1

Import-Module netWnv

  + CategoryInfo          : ResourceUnavailable: (netWnv:String) [Import-Module], FileNotFoundException
  + FullyQualifiedErrorId : Modules_ModuleNotFound,Microsoft.PowerShell.Commands.ImportModuleCommand

If configured through SCVMM:
Error (50362)
Logical networks that use network virtualization and are not managed by Network Controller are not supported on the selected host ('sdnhost1.sdn.lab') OS version.
Recommended Action
Logical Networks that use Network Virtualization without Network Controller are not supported on Windows Server 2019 and above. Please manage the logical network using Network Controller for Network Virtualization.
PS C:\Users\administrator.corp> systeminfo
Host Name:                 SDNHOST1
OS Name:                   Microsoft Windows Server 2022 Datacenter
OS Version:                10.0.20348 N/A Build 20348
OS Manufacturer:           Microsoft Corporation

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

ID: 35ba03f9-84d4-a1a3-b771-27c38f39e94a
Version Independent ID: 18405455-dc92-a2ca-d24f-25eba3b4a7f4
Content: NetWNV Module
Content Source: docset/winserver2022-ps/netwnv/NetWNV.md
Product: w10
Technology: windows
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

This repository is for PowerShell core documentation. You seem to be looking for support, which we can't provide here.
I suggest you try posting your issue with context in one of the available community support forums.


[Request] Reload retroarch after exiting Dolphin

If you could add an option to relaunch RetroArch after exiting Dolphin instead of just quitting that would be awesome.

I don't think it quits RetroArch afterwards. How are you quitting Dolphin? Maybe the Escape key repeats and quits both? Maybe the core is crashing?
It actually just runs Dolphin on top of RetroArch and doesn't quit while running.

Wasn't there a setting in RA to quit the gui while a core is running to save CPU? Might be imagining it.