Tracking issues of user metadata support

This issue is used to track the implementation of user metadata support.
Context
There are two types of metadata across various storage services.

System Defined: metadata like last-modified, content-type and so on. Some of them are editable, like content-type, but some are non-editable like last-modified.
User Defined: extra metadata that provided by user, can be any valid string.

OpenDAL now only supports System Defined metadata, we should add support for User Defined metadata so that users can write most useful information along with the file data. For example, write the location for this file.
Tasks

Implement user_metadata for metadata

[ ] Add user_metadata: HashMap<String, String> in Metadata
[ ] Add user_metadata in metakey
[ ] Add write_with_user_metadata in capability
[ ] Add user_metadata integration tests

Impelment user_metadata for services

[ ] s3: https://docs.aws.amazon.com/AmazonS3/latest/userguide/UsingMetadata.html
[ ] gcs: https://cloud.google.com/storage/docs/metadata
[ ] azblob: https://learn.microsoft.com/en-us/rest/api/storageservices/set-blob-metadata
[ ] ...

Hi, I'm interested in this issue.
but I have one problem to confirm：what client wants to use this feature, like
         .writer_with("path/to/file")
         .content_type("application/octet-stream")
         .user_metadata(map)
         .await?;

?

emmm, I should make it clear: I've reviewed the use cases, and would like to confirm the API for users.
should I add an function to FutureWrite like system defined metadata ?

should I add an function to FutureWrite like system defined metadata ?

Yes, we need a new API called user_metadata

should I add an function to FutureWrite like system defined metadata ?

Yes, we need a new API called user_metadata for both write and writer

Ok, i'll begin addressing this issue now

Hi @Xuanwo, I see that there are some of the tasks that are already done (the ones from the "Implement user_metadata for metadata" section and the s3 service). Should this tracking issue checklist be updated with that?
I can help implementing user_metadata for other services, I'd appreciate if the checklist can be extended to at least the ones with the most priority?

Hi, thank you @jorgehermo9 for your work on this!
I will take some time to extend the list and sort it by priority. Before that, I believe the next good option could be gcs support.

Hi @Xuanwo! I couldn't work on this these days, but I'll resume me work as soon as I have enough time :) I have it in mind, I didn't forget about this!

Hi @Xuanwo! I couldn't work on this these days, but I'll resume my work as soon as I have enough time :) I have it in mind, I didn't forget about this!

Really appreciated!


Added the dynamic background

PR Description 📜
Please include summary related to the issue you have fixed and describe your PR in brief over here by specifying the issue number on which you were working below
Fixes #4370

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[ ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸
https://github.com/kunjgit/GameZone/assets/138770813/babcdfdc-9d4e-4f87-a8ed-ff4bb2e21f16

Thank you soo much for contributing to our repository 💗

Thank you @happyrao78 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit Kindly merge the PR.

Good to go!!

When will this PR get merged ?

Thank you  @happyrao78 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


Cant Load Simple browser

Describe the bug
when show "Error loading webview: Error: Could not register service workers: SecurityError: Failed to register a ServiceWorker: The provided scriptURL ('https://ide-webview.staticblitz.com/771d21865841b49ec9d3de318b49ea649c509949/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority=') violates the Content Security Policy.."
when opening simple browser
Link to the blitz that caused the error
https://stackblitz.com/~/github.com/CodeFoxy-Github/docs
Steps to reproduce
use "npm install && npm start" wait to show "A service is available on port xxxx" click "Open Preview"
it will get this error message "Error loading webview: Error: Could not register service workers: SecurityError: Failed to register a ServiceWorker: The provided scriptURL ('https://ide-webview.staticblitz.com/771d21865841b49ec9d3de318b49ea649c509949/service-worker.js?v=4&vscode-resource-base-authority=vscode-resource.vscode-cdn.net&remoteAuthority=') violates the Content Security Policy.."
Expected behavior
open the simple browser to preview the site
Parity with Local

[X] I have run the project in my local machine and I could not reproduce the issue.

Screenshots

Platform
Version = 1.67.0
Hash = 771d21865841b49ec9d3de318b49ea649c509949
WebContainer = dfae8564d35fe429a0bab219bd7d77341f3ec782

Browser name  = Chrome
Full version  = <IP_ADDRESS>
Major version = 106
navigator.appName = Netscape
navigator.userAgent = Chrome/<IP_ADDRESS>
performance.memory = {
  "totalJSHeapSize": 100592986,
  "usedJSHeapSize": 83103298,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}

Additional context
No response

(it Works in Brave Browser)

Hi @CodeFoxy-Github 👋 . This might've been a bug with our deploy. I believe this should be fixed in the mean time. Could you check again? If not, we have to look into what could cause this issue as it might be a Chrome related setting.

I've been having a similar issue, I've tried various different things including asking for ChatGPT's input. However, only temp solution is to open workspace in VS Code Desktop.
Please see below image:

:wave: We're closing this due to inactivity. Please, feel free to open a new one if you're still encountering any errors.


Document the BaseMaterial3D class

This makes the class 100% documented again.

Thank you! 🎉 And thanks for reviewing Clay :)


Remove invoker tracking/publishing per-subject activation counts.

Switch to information already available in load balancer's new activation maps.  This is a followup to the activation maps we recently begin maintaining.

PG2-478 true green.


Read kernel attributes and add them to non-semantic reflection information

This is to make it possible to implement clGetKernelInfo with the CL_KERNEL_ATTRIBUTES flag. The returned string depends on what the implementation supports, so I think it's best that this goes through the compiler.
Since the string is expected to contain attributes as they were declared in the qualifier, I thought it was best that the frontend reads the data so any formatting can be maintained. Adding annotations to the function was the best way I could find of passing the data from the frontend plugin to the passes. AnnotationToMetadataPass was added because the annotation was referencing the entry point functions so it because more difficult to delete and replace them.
This depends on this change to SPIRV-Headers - https://github.com/KhronosGroup/SPIRV-Headers/pull/308
This contribution is being made by Codeplay on behalf of Samsung.

This isn't in a completed state, in particular I don't believe that the produced string is correct, but I'm going on holiday for two weeks. One of my colleagues may finish the work while I'm away.

Unfortunately I no longer have time to complete this work but it would be a shame if this work was lost since these addition will help toward 100% CTS conformance with CLVK. If someone would like to pick up this work and finish it then please do.
I think you'll need to:

adjust the string generated by stripping the attributes qualifier
adjust the non-semantic instruction to account for it being optional
add a couple tests
tidy things up a little.

I am taking over this PR


Optimizing PriorityAssigner for batch priority registrations

This PR aims to fix #1093
In cases of batch priority registration, two known states can be taken advantage of:

The ofPriority <-> Priority mapping is empty at the time batch registration is called, since it is only used in agent restart case.
All the Priorities that needs to be registered are readily available, unlike in the async reconcile case after agent starts, where each Priority registration is independent.

To utilize those two properties, the RegisterPriorities function is refactored in the following way: it first sorts and dedups the Priorities that needs to be registered. Then, starting from the highest Priority, it gets the insertionPoint of that Priority and evaluates if there are enough OpenFlow priorities left (from lowest ofPriority allowed on the table to insertionPoint) to be assigned to the remaining Priorities. If so, it assigns insertionPoint to that Priority and leaves the same question to the next Priority in line. Else, it starts allocating all consecutive ofPriority from the bottom-up to all the remaining Priorities.
With this optimization, the runtime of the added testcase (which is to test registering all the available ofPriorities in the table and requires quite some priority reassignments if registered asynchronously) improved from 49s 630ms to 80ms.

/test-all /test-hw-offload

/test-windows-networkpolicy /skip-hw-offload

/test-all /test-hw-offload

/test-hw-offload

Codecov Report

Merging #1105 into master will increase coverage by 0.03%.
The diff coverage is 70.83%.

@@            Coverage Diff             @@
##           master    #1105      +/-   ##
==========================================
+ Coverage   56.28%   56.31%   +0.03%
==========================================
  Files         109      109
  Lines       12037    12058      +21
==========================================
+ Hits         6775     6791      +16
- Misses       4673     4677       +4
- Partials      589      590       +1

Flag
Coverage Δ

#integration-tests
45.98% <0.00%> (-0.10%)
:arrow_down:

#unit-tests
42.65% <89.47%> (+0.12%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/agent/types/networkpolicy.go
11.76% <0.00%> (-4.91%)
:arrow_down:

pkg/agent/controller/networkpolicy/priority.go
93.29% <89.47%> (+3.42%)
:arrow_up:

pkg/ovs/openflow/ofctrl_bridge.go
71.57% <0.00%> (-1.34%)
:arrow_down:

...ntroller/networkpolicy/networkpolicy_controller.go
73.04% <0.00%> (-0.25%)
:arrow_down:

pkg/apiserver/storage/ram/store.go
81.69% <0.00%> (+1.30%)
:arrow_up:

/test-all

/test-e2e /skip-hw-offload

/skip-hw-offload

/test-all

/test-conformance /test-networkpolicy

/test-conformance /test-networkpolicy


Want to make sure this algorithm will not generate collisions

Hello, I'm using this module to convert the id of a order to a 12-digit confirmation number, for example:
Order ID 1 =<PHONE_NUMBER>01  --><PHONE_NUMBER>74
Order ID 2 =<PHONE_NUMBER>02  --><PHONE_NUMBER>45
My code goes like this:
key = 'C4D91A3F7B61834E90FD72563CACF01E'
tweak = 'ABD39280979565'
c = FF3Cipher(key, tweak)

order_id = 1
order_id_padded = f'{order_id:012}'
confirmation_number = c.encrypt(order_id_padded)

I assume that as long as I don't change the key or the tweak I should not have any collision issues of two order ids generating the same confirmation number. Is this correct?
Thanks for your time

Yes, the source data and protected data have a one-to-one relationship such that collisions will not occur with format preserving encryption. If you consider decryption, it would not work if collisions occurred.


Problem in training my own detectors

I've read your paper, and run the detect demo, using the facefinder which you provided，really fast and good behavior。
Now i‘d love to train my own detectors 、detecting pedestrian，as you say in the instruction，download the dataset（images and  background），but i lived in china、i cannot download that。 I‘ve got some pictures and background，what should i do with them？what‘s the format of the training datasets？
Thanks。

Hi!
why don't you see genky.py and background.py?

@YangUSTC  3Q

调用程序你弄明白了吗，我看了main函数，但是却不懂他的输入


Refactor & add dummy login data

Codecov Report

Merging #6 into main will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main        #6   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            5         9    +4
  Lines           66        79   +13
=========================================
+ Hits            66        79   +13

Impacted Files
Coverage Δ

src/core/apiCallToCreate.js
100.00% <ø> (ø)

src/core/apiCallToList.js
100.00% <ø> (ø)

src/core/composeHttpRequestBody.js
100.00% <ø> (ø)

src/core/validateKind.js
100.00% <ø> (ø)

src/core/apiCallOptions.js
100.00% <100.00%> (ø)

src/core/getApiLoginInfo.js
100.00% <100.00%> (ø)

src/core/validateMode.js
100.00% <100.00%> (ø)

src/test/dummyData.js
100.00% <100.00%> (ø)

src/test/loadEnvVars.js
100.00% <100.00%> (ø)

... and 3 more

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 8704752...38648d0. Read the comment docs.


CI: Conformance IPSec E2E, logs spammed with HUBBLE_RING_BUFFER lost events

CI failure
In an innocuous PR on main, one of the Conformance IPSec E2E runs failed. The logs were filled with the line
⚠ hubble-flows-cilium-8nlvr: failed to collect hubble flows for "cilium-8nlvr" in namespace "kube-system": error with exec request (pod=kube-system/cilium-8nlvr, container=cilium-agent): command terminated with exit code 1: time=2024-09-16T12:11:31.040Z level=DEBUG msg="connected to Hubble API" server=server
time=2024-09-16T12:11:31.040Z level=DEBUG msg="Sending GetFlows request" request=number:1000000
unknown response type: &{LostEvents:source:HUBBLE_RING_BUFFER num_events_lost:1}
unknown response type: &{LostEvents:source:HUBBLE_RING_BUFFER num_events_lost:1}
unknown response type: &{LostEvents:source:HUBBLE_RING_BUFFER num_events_lost:1}
unknown response type: &{LostEvents:source:HUBBLE_RING_BUFFER num_events_lost:1}
(repeats thousands of times)

I saw this once more today, but I can't find the link right now.

@glrf and I looked into this one rainy morning as I hit this in some PR too - seems like this is occurring more often than we thought, so maybe it's worth investigating a bit more.


Error assertions problems

I have discovered that thanks to Babel's limitations we can't extend the built-in Error class for our custom error class as it is not possible to use instanceof assertion on it afterward.
The solution is using the error codes that I have already proposed. The potential problem from the user-side is that users might expect the instanceof check working so we should prevent that as only documentation might not be sufficient for that.
An idea that came up to my mind is that we could not export the custom Error classes but only the error codes they represent and so "force" the users to really use code assertions instead of instanceof assertions.
UPDATE:
The original problem was most probably caused by a combination of a bug and some other weird behavior. The mentioned caveats should not be applicable to us, because this problem raises when the build target is ES5 but we are targeting ES6, but...
There is a new problem with assertions of Errors raised by NodeJS modules. They wrap all errors (even the native ones like TypeError, if interested see here) in order to support error codes and possibly other stuff. Most probably thanks to Babel/Webpack some of this wrapping is broken and simple e instanceof TypeError for TypeErrors thrown by NodeJS module won't work.
I have tested the same in browsers (I am concerned with URL class that is present both in Node and browser) and that works as expected.

I agree. Would you still create one custom class BeeError with this code attribute, or add it directly to  Error?.
Given this limitation, I would even consider introducing second attribute e.g. kind or area which would enable grouping (similar to how class hierarchy would). At least one level.

Another solution could be using the proposed plugin babel-plugin-transform-builtin-extend which is claimed that it could solve it. There is some limitations (the browsers have to support __proto__), but I think that should not be problem for us.

Will this work even for a dApp developer who also uses babel in their project (e.g. with Create React App) without this plugin? Probably, right?

I hope so, but I will investigate to verify

So! The original problem was most probably caused by a combination of a bug and some other weird behavior. The mentioned caveats should not be applicable to us, because this problem raises when the build target is ES5 but we are targeting ES6, but...
There is a new problem with assertions of Errors raised by NodeJS modules. They wrap all errors (even the native ones like TypeError, if interested see here) in order to support error codes and possibly other stuff. Most probably thanks to Babel/Webpack some of this wrapping is broken and simple e instanceof TypeError for TypeErrors thrown by NodeJS module does not work.
I have tested the same in browsers (I am concerned with URL class that is present both in Node and browser) and that works as expected.
While I used a workaround for this problem (assertions on error codes for NodeJs) I am very much worried that this will bite us later on, with for example NodeJS only project like swarm-cli where we use the same build setup. I would strongly suggest to drop the webpack build system for NodeJS!


MAINT: 1.8.0 rc3 backports round 1

we have accumulated 20 merged backport PRs and they have all
been cherry-picked into here, along with a specific request for author
list update
decent chance a bit of effort will be needed to get the CI green on maintenance
branch
why is 1.8.0 taking a while?

PROPACK challenges here: https://github.com/conda-forge/scipy-feedstock/pull/198
doc deploy issues, maybe we can just try Ralf's idea: https://github.com/scipy/scipy/issues/14267#issuecomment-1003380812

I actually wonder if we should disable propack in 1.8.0, or make it opt-in, at least the sigfaulting functionality.

Thoughts on ignoring the linter complaints? I think the undefined name issues are b/c the linter cannot handle importlib when used instead of import?
scipy/_build_utils/_fortran.py:146:9: E128 continuation line under-indented for visual indent
scipy/signal/tests/test_filter_design.py:33:80: E501 line too long (91 > 79 characters)
scipy/sparse/tests/test_base.py:4824:80: E501 line too long (81 > 79 characters)
scipy/special/_testutils.py:27:80: E501 line too long (91 > 79 characters)
setup.py:27:1: E402 module level import not at top of file
setup.py:237:16: F821 undefined name '_pep440'
setup.py:237:53: F821 undefined name '_pep440'
setup.py:349:16: F821 undefined name '_pep440'
setup.py:349:49: F821 undefined name '_pep440'
tools/cythonize.py:87:80: E501 line too long (81 > 79 characters)

That's fine, Lint issues can pretty much always be ignored.

32-bit linux stats test fails pasted below the fold:
@mdhaber these have shown up recently maybe? was there a patch for it?

2022-01-16T22:07:17.9426107Z =================================== FAILURES ===================================
2022-01-16T22:07:17.9426790Z __________ TestStudentizedRange.test_moment_against_mp[case_result1] ___________
2022-01-16T22:07:17.9428314Z [gw1] linux -- Python 3.8.0 /usr/bin/python3.8
2022-01-16T22:07:17.9428938Z scipy/stats/tests/test_distributions.py:4655: in test_moment_against_mp
2022-01-16T22:07:17.9429507Z     res = stats.studentized_range.moment(*mkv)
2022-01-16T22:07:17.9430529Z         case_result = {'mp_result': 1.8342745127927962, 'src_case': {'expected_atol': 1e-09, 'expected_rtol': 1e-09, 'k': 3, 'm': 1, ...}}
2022-01-16T22:07:17.9431269Z         mkv        = (1, 3, 10)
2022-01-16T22:07:17.9431736Z         mp_result  = 1.8342745127927962
2022-01-16T22:07:17.9432301Z         self       = <scipy.stats.tests.test_distributions.TestStudentizedRange object at 0xd28650d0>
2022-01-16T22:07:17.9433253Z         src_case   = {'expected_atol': 1e-09, 'expected_rtol': 1e-09, 'k': 3, 'm': 1, ...}
2022-01-16T22:07:17.9434038Z scipy/stats/_distn_infrastructure.py:1297: in moment
2022-01-16T22:07:17.9434554Z     val[...] = _moment_from_stats(n, mu, mu2, g1, g2, self._munp, shapes)
2022-01-16T22:07:17.9435096Z         args       = [array([3]), array([10]), array([0]), array([1])]
2022-01-16T22:07:17.9435559Z         g1         = None
2022-01-16T22:07:17.9435961Z         g2         = None
2022-01-16T22:07:17.9436343Z         i0         = True
2022-01-16T22:07:17.9436745Z         i1         = True
2022-01-16T22:07:17.9437124Z         i2         = False
2022-01-16T22:07:17.9437520Z         kwds       = {}
2022-01-16T22:07:17.9437923Z         loc        = array([0])
2022-01-16T22:07:17.9438532Z         mdict      = {'moments': 'm'}
2022-01-16T22:07:17.9438987Z         mu         = None
2022-01-16T22:07:17.9439374Z         mu2        = None
2022-01-16T22:07:17.9439788Z         n          = 1
2022-01-16T22:07:17.9440255Z         scale      = array([1])
2022-01-16T22:07:17.9440962Z         self       = <scipy.stats._continuous_distns.studentized_range_gen object at 0xebb2b3e8>
2022-01-16T22:07:17.9441531Z         shapes     = [array([3]), array([10])]
2022-01-16T22:07:17.9442525Z         val        = array([1.])
2022-01-16T22:07:17.9445367Z scipy/stats/_distn_infrastructure.py:371: in _moment_from_stats
2022-01-16T22:07:17.9445967Z     val = moment_func(1, *args)
2022-01-16T22:07:17.9446884Z         args       = [array([3]), array([10])]
2022-01-16T22:07:17.9447340Z         g1         = None
2022-01-16T22:07:17.9447770Z         g2         = None
2022-01-16T22:07:17.9448370Z         moment_func = <bound method studentized_range_gen._munp of <scipy.stats._continuous_distns.studentized_range_gen object at 0xebb2b3e8>>
2022-01-16T22:07:17.9448998Z         mu         = None
2022-01-16T22:07:17.9449421Z         mu2        = None
2022-01-16T22:07:17.9449830Z         n          = 1
2022-01-16T22:07:17.9450369Z scipy/stats/_continuous_distns.py:9549: in _munp
2022-01-16T22:07:17.9450868Z     return np.float64(ufunc(K, k, df))
2022-01-16T22:07:17.9451395Z E   RuntimeWarning: invalid value encountered in ? (vectorized)
2022-01-16T22:07:17.9451878Z         K          = 1
2022-01-16T22:07:17.9452316Z         _a         = 0
2022-01-16T22:07:17.9452737Z         _b         = inf
2022-01-16T22:07:17.9453285Z         _single_moment = <function studentized_range_gen._munp.<locals>._single_moment at 0xd283efa0>
2022-01-16T22:07:17.9454452Z         cython_symbol = '_studentized_range_moment'
2022-01-16T22:07:17.9454955Z         df         = array([10])
2022-01-16T22:07:17.9455409Z         k          = array([3])
2022-01-16T22:07:17.9455950Z         self       = <scipy.stats._continuous_distns.studentized_range_gen object at 0xebb2b3e8>
2022-01-16T22:07:17.9456763Z         ufunc      = <ufunc '? (vectorized)'>
2022-01-16T22:07:17.9457402Z ________________ TestStudentizedRange.test_moment_vectorization ________________
2022-01-16T22:07:17.9458161Z [gw1] linux -- Python 3.8.0 /usr/bin/python3.8
2022-01-16T22:07:17.9458882Z scipy/stats/tests/test_distributions.py:4694: in test_moment_vectorization
2022-01-16T22:07:17.9459457Z     m = stats.studentized_range._munp([1, 2], [4, 5], [10, 11])
2022-01-16T22:07:17.9460072Z         self       = <scipy.stats.tests.test_distributions.TestStudentizedRange object at 0xd256dca0>
2022-01-16T22:07:17.9460672Z scipy/stats/_continuous_distns.py:9549: in _munp
2022-01-16T22:07:17.9461181Z     return np.float64(ufunc(K, k, df))
2022-01-16T22:07:17.9461704Z E   RuntimeWarning: invalid value encountered in ? (vectorized)
2022-01-16T22:07:17.9462195Z         K          = [1, 2]
2022-01-16T22:07:17.9462623Z         _a         = 0
2022-01-16T22:07:17.9463071Z         _b         = inf
2022-01-16T22:07:17.9463625Z         _single_moment = <function studentized_range_gen._munp.<locals>._single_moment at 0xd2a78bb0>
2022-01-16T22:07:17.9464414Z         cython_symbol = '_studentized_range_moment'
2022-01-16T22:07:17.9464919Z         df         = [10, 11]
2022-01-16T22:07:17.9465359Z         k          = [4, 5]
2022-01-16T22:07:17.9465895Z         self       = <scipy.stats._continuous_distns.studentized_range_gen object at 0xebb2b3e8>
2022-01-16T22:07:17.9466656Z         ufunc      = <ufunc '? (vectorized)'>
2022-01-16T22:07:17.9467488Z - generated xml file: /scipy/build/testenv/lib/python3.8/site-packages/junit/test-results.xml -

It looks like the 32-bit stats failures are somewhat stochastic.. the CI now just has the timeouts and lint issues, so probably ok. I may merge tomorrow morning and then deal with the bigger problems with a somewhat cleaner backporting slate.


Exception when running inappropriate file as a script, fix 2839

@kajojify and I had a discussion on this and this was a very difficult choice for us to make.
With the solution that this PR brings forward, although it's a neat solution but it might disturb the user experience when the error from his/her script is not a SyntaxError because it won't be informative enough
Another solution is not a neat solution which is proposed in #2839 comments.

Closed because #2962


Trouble with FOS Splitter when window size is changed.Uncaught Error: Splitter needs to be in a component with size

Hello everyone, I have an issue with FOS Splitter Plug-In.  I have a region which is static content and in the sub region of that region I have FOS-Splitter Plug-In and in the splitter I have 2 interactive gridsWhen I collapse the page I got the error ‘interactiveGrid.min.js?v=21.2.1:5 Uncaught Error: Splitter needs to be in a component with size’ this error on the console. Is anyone know that problem? I mean when window size is changed I faced with this issue and still could not fixed it.

The issue is caused by a zero height being determined on the parent element being split.
Can you please create a test case on apex.oracle.com and provide access details so we can investigate your issue. Thanks

The issue is caused by a zero height being determined on the parent element being split.
Can you please create a test case on apex.oracle.com and provide access details so we can investigate your issue. Thanks

Hello I made a test case on apex
Workspace name = SPLITTERDEMO
Username =<EMAIL_ADDRESS>PW = Fatma97!
App Info
Username =<EMAIL_ADDRESS>PW = Fatma97!
Can you please check what I did wrong? I used region display selector and tab splitter. When I am on the other tabs (the one that is not with tab splitter) and I change the size of the page I get splitter error. Thank you in advance

Thanks for the test case!
You didn't do anything wrong, it's a bug ( related to the "tabs container" ), it'll be fixed with the next release.
(We updated the plug-in in your application, so that should work just fine for now).

Thanks for the test case!
You didn't do anything wrong, it's a bug ( related to the "tabs container" ), it'll be fixed with the next release.
(We updated the plug-in in your application, so that should work just fine for now).

Thank you so much!

I want to place tabs on the right side of the splitter. But when an interactive report is placed anywhere it throws the issue.
In this workspace is the demo.
WORKSPACE NAME: PRUEBAS-EBRAN
USERNAME<EMAIL_ADDRESS>PASS: P@SSWO4D!

Thanks for the test case!
You didn't do anything wrong, it's a bug ( related to the "tabs container" ), it'll be fixed with the next release.
(We updated the plug-in in your application, so that should work just fine for now).

Hi again! We have APEX with version 21.2.1. How can I transfer the solution to our system can you please help me?

Fixed with v22.1!


Code style improvements and visions

I've made several code style changes over project (not including examples), also some converts (like for to foreach). I wanted to know what would be nice to apply in this project and what should be thrown away.

Not sure about var, because sometimes it's useful (when you have obvious name of variable and long type with params), sometimes it's not really (replacing int with var in for-loop is a bit weird)
I've moved vars declaration to the top, never seen vars at the bottom anywhere, so... maybe it's a good idea to move it ontop?
Inverted some if's where it was obviously needed (too much spacing)
and so on.

Going to review the updated changes now, but it looks like there's a merge conflict. I think it has to do with the way your first PR was done since for some reason this PR is trying to also add the same files and make the same changes. Looking at your code_style branch I see

This branch is 7 commits ahead, 1 commit behind DoubleDeez:master.

I'm not 100% on how to fix it, but you might need to git pull --rebase from my master branch into your code_style branch


feat(RHHTAPREL-714): update RPA definitions

ReleasePlanAdmissions' PipelineRef and ServiceAccount fields have moved inside a new field, Pipeline.update RPA definitions
Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Issue ticket number and link
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist:

[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added meaningful description with JIRA/GitHub issue key(if applicable), for example HASSuiteDescribe("STONE-123456789 devfile source")
[ ] I have updated labels (if needed)

Seems the dependency hell is met again in the PR.

You have two H's in the JIRA name in the commit title

/retest

/retest

This is a paired PR. It won't pass yet

/retest-required

/retest

/retest-required


Timeout issue when connecting to cloud database

When connecting to an Oracle cloud database, I see the following severe exception in the Tomcat logs:
oracle.simplefan.impl.FanManager.configure attempt to configure ONS in FanManager failed with oracle.ons.NoServersAvailable: Subscription time out

The connection gets established and everything works, just the initial timeout takes up some time for the first usage of JRI to establish the connection pool for this connection.
The solution is one of the following:

remove ons.jar from the class path
or
disable the feature through the java environment:java -Doracle.jdbc.fanEnabled=false ...

or
disable the feature when establishing the connection

See: https://support.oracle.com/epmos/faces/DocumentDisplay?_afrLoop=270218870398327&parent=EXTERNAL_SEARCH&sourceId=PROBLEM&id=2679061.1&_afrWindowMode=0&_adf.ctrl-state=133spg7szt_53 for details.
I guess I will add a parameter for the JDBC configuration file (application.properties), so that it can be turned on for a specific JDBC connection pool but is turned off by default.

I have removed the Oracle Notification Services (ons.jar) from the build.
See: https://support.oracle.com/epmos/faces/DocumentDisplay?_afrLoop=228794872741974&parent=EXTERNAL_SEARCH&sourceId=PROBLEM&id=2679061.1&_afrWindowMode=0&_adf.ctrl-state=83kfn7hl1_4 for details.
Connections to autonomous database work.
If you need this support nevertheless, you can download the matching jar file from here: https://www.oracle.com/database/technologies/appdev/jdbc-ucp-21-4c-patched-downloads.html


Fix multi-node issues from launch

Solves a few lasting issues with the torchrun migration. Noteably:

When doing multinode on the same network, it would just hang forever.
Defaults were not being set properly and overriden for same_network
node_rank needs to be an int, not a string

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Add a perception of a xarray magic method

Is your feature request related to a problem?
I am often moving data from external objects (of all sorts!) into xarray. This is a common use case
Much of this code would be greatly simplified if there was a way of giving non-xarray classes a way of declaring to xarray how these objects can be marshaled into
Describe the solution you'd like
So here is an initial proposal for comment. Much of this could be implemented in a third party library. But doing this in xarray itself would likely be best.
Magic Methods
It would be great to see these magic method signatures become integrated throughout the library:
___xarray__ -> xr.Dataset | xr.DataArray
___xarray_array__ -> xr.DatArray
___xarray_dataset__ -> xr.Dataset
___xarray_datatree__ -> xr.DataTree   # when DataTree is finally integrated into xarray

Conversion Registry
And these extension functions to register converters:
def register_xarray_converter(class, name: str, func : Callable[[class, ...] | None) -> xr.Dataset | xr.DataArray]:
  ...
def register_dataarray_converter(class, name: str, func : Callable[[class, ...] | None) ->  xr.DataArray:
  ...
def register_dataset_converter(class, name: str,  func : Callable[[class, ...] | None) -> xr.Dataset:
  ...
def register_datatree_converter(class, name: str,  func : Callable[[class, ...], xr.DataArray] | None) -> DataTree # when DataTree is finally integrated into xarray
  ...

Registering a  converter if if cls implements a corresponding xarray*_ method or another converter already registered for cls. Perhaps add an argument that specifies if the converter should or should not be added if their is a clash. Perhaps these functions return the replaced converter so it can be added back in if needed?
Ideally, also, "deregister" versions (.e.g deregister would also be available. So context managers that change marshaling behavior could easily be constructed.
User API
Along with the following new user API functions:
def as_xarray(x, *args, **kwargs) -> xr.Dataset | xr.DataArray:
  ...
def as_dataarray(x,*args, **kwargs) -> xr.DataArray:
  ...
def as_dataset(x,*args, **kwargs) -> xr.DataSet:
  ...
def as_dataset(x,*args, **kwargs) -> xr.DataSet: # when DataTree is finally integrated into xarray
  ...

"as_xarray" returns (in order of precedence:

x unaltered if it is an xarray objects
registered_xarray_converter(x, *args, **kwargs) if it is callable and does not throw an exception
registered_dataarray_converter(x, *args, **kwargs)  if it is callable and does not throw an exception
registered_dataarray_converter(x, *args, **kwargs) if it is callable and does not throw an exception
x.xarray(*args, **kwargs), if it exits, is callable, and does not throw an exception
x.xarray_dataset(*args, **kwargs), if it exists, is callable, and does not throw an exception
x.xarray_dataarray(*args, **kwargs), if it exists, is callable, and does not throw an exception
well known aliases of xarray_dataarray, such as x.to_xarray(*args, **kwargs) (see pandas)
[DESIGN DECISION] convert and return tuple[dims, data, [attr, encoding] to DataArray?
[DESIGN DECISION] convert and return tuple encoding of DataSet?
[DESIGN DECISION] return DataArray wrapped duck-typed array in DataArray?

The rationale for putting the registered functions first is that this would enable
"as_dataarrray" would be slimilar, but it would only call x.xarray_dataarray and well known aliases.
"as_dataset" would be slimilar, but it would only call x.xarray_dataset, well known aliases, and perhaps falling back to calling x.xarray_dataarray and converting the return a dataset if it has a name attribute.
"as_datatree" would be slimilar, but it would only call x.xarray_datatree, and perhaps falling back to calling x.xarray_dataarray and wrapping it in a single node datatree. (Though of course at this point this method would probably be implemented by the DataTree package, not xarray)
The design decisions are flexible from my point of view, and might be decided in a way that makes the code base simplest or most usable. There is also a question of whether or not this method should default the backup methods. These decisions also can be deferred entirely by delegating to the converter registry.
Across the Xarray Library
Finally, across the xarray library, there may be places where passing input arguments through as_xarray, as_dataarray, or as_dataset would make a lot of sense. This could be the final thing to do, but cannot be handled by a third party library.
Doing this would give give another pathway for third party libraries to integrate with xarray, with a far easier way than the converter registry or explicit calls to as_* functions.
Describe alternatives you've considered
This can be done with a private library. But it seems to a lot of code that is pretty useful to other use cases.
Most of this (but not all) can accomplished in a 3rd party library, but it wouldn't allow the seamless sort of integration with (for example) xarray use of repr_html to integrate with pandas.
The existing backend hooks work great when we are marshaling from file-based sources. See, for example, tiffslide-xarray (https://github.com/swamidasslab/tiffslide-xarray). This approach is seemless for reading files, but cannot marshal objects. For example, this is possible:
x = xr.open_dataset("slide.tiff")

But this doesn't work.
t = tiffslide.TiffSlide("slide.tiff")
x = xr.open_dataset(t) # won't work
x = xr.DataArray(t) # won't work either

This is an important use case because there are cases where we want to create an xarray like this from objects that are never stored on the filesystem.
Additional context
No response

Thanks for the issue @swamidass !
There is precedence for this — for example in Rust there are .from & .into methods that any struct can implement to convert into a struct from another library, and vice versa.
One thing this could interface with is subtyping — if a CustomDataset defined a form of __dataarray_slice__, then xarray could create a data array of that class when slicing. (#3980)
I agree that it could start as another library — or just in your own code initially — I don't think there's actually much need for this to be in xarray at the start. Probably the helpful thing here is to get feedback from others, and then coalesce on a standard over time.

The slice idea is a good one too.
Yup, asking for feedback.
Any idea if and when Datatree is gonna get rolled in?

Thanks for the interesting suggestion @swamidass!
I might be missing something, but what's the advantage of doing this over the other class just implementing a .to_xarray() method?
I'm also wondering if there any precedence for this pattern in Pandas? That might be useful to know as a similar prior example. I guess this suggestion is somewhat similar to pandas using apache arrow...?

Any idea if and when Datatree is gonna get rolled in?

When I get around to it / when I get some help 😅 If that's something you're interested in then that would be amazing. It honestly shouldn't actually be particularly hard, mostly just copy-pasting and making sure its all up to xarray code review standards. However this point it may make sense to wait for the NamedArray refactor to progress a bit further first though, see https://github.com/xarray-contrib/datatree/issues/270.


Fatal Crash: Expected Android API level 21+ but was 30

Hi,
when I was targeting the SDK 30, i got Fatal Crash: Expected Android API level 21+ but was 30.
React-Native: 0.63.3
The workaround was to use recet okhttp. implementation("com.squareup.okhttp3:okhttp:4.9.0")

It's fixed with implementation("com.squareup.okhttp3:okhttp:3.14.9").
React Native now uses the OKHTTP_VERSION=3.12.12 that will crash on Android 11.
Look into changelog here: https://square.github.io/okhttp/changelog_3x/#version-3147

@grabbou Could you please look into that, please?

Hey @lorenc-tomasz thanks for the issue! I'll submit a PR for this

Thank you :)

https://github.com/facebook/react-native/pull/30104

react native 0.63.4 bumped the okhttp dependency
https://github.com/react-native-community/releases/blob/master/CHANGELOG.md#v0634
https://github.com/facebook/react-native/commit/0f6fcb2c2788dc7150f6c3673a8f4f9d8f929441
https://github.com/facebook/react-native/pull/30104#issuecomment-704970648
This issue is solved

Still I had to bump version to 3.14.7 to resolve the issue, but based on master it can be marked as resolved.


Draft: first test scenario using mgc cli

related to: https://github.com/marmotitude/object-storage-tests/issues/42

@fczuardi  corrigido os pontos observados.


_collect_nodes gives up too easily

From<EMAIL_ADDRESS>on April 10, 2011 10:25:43
Please type example code that produces the issue:
    me = pylast.User(username, network)
    mytracks = me.get_recent_tracks(limit=None)

What is the expected output? What do you see instead?
Expected: all of my scrobbles.
Got: Malformed response error. Something about an EOF where it didn't belong.
What versions of Pylast and Python are you using?
latest, 2.6
Please provide any additional information below.
I was able to fix this by rewriting _collect_nodes as follows:
def _collect_nodes(limit, sender, method_name, cacheable, params=None):
    """
        Returns a sequqnce of dom.Node objects about as close to
        limit as possible
    """

    if not params:
        params = sender._get_params()

    nodes = []
    page = 1
    end_of_pages = False

    while not end_of_pages and (not limit or (limit and len(nodes) < limit)):
        params["page"] = str(page)
        max_fails = 10
        fails = 0
        try:
            doc = sender._request(method_name, cacheable, params)

            main = doc.documentElement.childNodes[1]

            if main.hasAttribute("totalPages"):
                total_pages = _number(main.getAttribute("totalPages"))
            elif main.hasAttribute("totalpages"):
                total_pages = _number(main.getAttribute("totalpages"))
            else:
                raise Exception("No total pages attribute")

            for node in main.childNodes:
                if not node.nodeType == xml.dom.Node.TEXT_NODE and (not limit or (limit and len(nodes) < limit)):
                    nodes.append(node)

                if page >= total_pages:
                    end_of_pages = True

            page += 1
        except Exception as e:
            print "effin' weird ", str(page)
            fails += 1
            if(fails >= max_fails):
                page += 1

    return nodes

This retrieved my 32000 scrobbles, requiring (for some reason) 2000+ pages to do so. In those pages, it had to "retry" a page three different times, but never had to retry the same page more than once.
Original issue: http://code.google.com/p/pylast/issues/detail?id=67

I just tested this and it downloaded my 83,867 scrobbles successfully in 3 minutes and 34 seconds (latest develop branch, Python 2.7.13, macOS Sierra).
I'll close this, we can re-open if it re-occurs.


Add mechanism for marking an Upgrade as offline only

As of release 0.8.1, we are making upgrades automatic. When you deploy a version of ACA-Py (as stated in version.py) that is higher than what is stored in the ACA-Py secure storage database, an upgrade is automatically run, based on the contents of the Upgrade Definition YML file. However, in some cases, an upgrade may be particularly risky, and we want to make sure it is run only offline.  For example, if the upgrade has side effects that it must only run exactly once, we don't want to an automated upgrade in a Kubernetes environment where the upgrade might be run more than once.
To prevent that, please add a parameter that is optional but that can be added to YML upgrade definitions that prevents an "auto-update".  I think there should be two flavours of the parameter -- one that triggers and error and stops the operation, and one that triggers only an entry in the log and continues with processing.   The "error and stop" can be used when the upgrade MUST be run before the new version can be deployed.  The "log" can be used when the upgrade must be run offline, but is not needed for the new version to be run.  In the latter case, the version in storage MUST NOT be updated to the new current ACA-Py version -- it needs to be left as is.

Related to PR just assigned for review.


M1 Mac does not run MZmine3 4.2

My M1 Mac mini does not open the new version of MZmine 4.2

Mac mini M1, Sonoma 14.5
My mzmine version: 4.2

###download the new version, installed it, but Sonoma does not open it

After installing the new version, I get the message:
"mzmine" can't be opened because Apple cannot check it for malicious software.
At the terminal, this is what happens:

Last login: Mon Mar 11 09:11:44 on console
/Applications/mzmine.app/Contents/MacOS/mzmine ; exit;
(base) giselletamayo@Giselles-Mac-mini ~ % /Applications/mzmine.app/Contents/MacOS/mzmine ; exit;
zsh: killed     /Applications/mzmine.app/Contents/MacOS/mzmine
Saving session...
...copying shared history...
...saving history...truncating history files...
...completed.
[Process completed]
Thanks!
Giselle

Oh oh... went to security and authorized it.
Solved!
Giselle


cannot turn off autocomplete

Hi there!
I am not sure if this is an issue with the systemverilog package, or the basic sublime package.  I have tried to turn off autocomplete in both the systemverilog package and the general sublime preferences.   I have tried to use the sublime forum to resolve the issue, but without success.  I have also tried to uninstall and re-install sublime, but I am suspicious that windows did not delete all the sublime files during the uninstall.  I say that because after I reinstalled sublime, the "autocomplete false" settings were still there.
I am suspicious that this issue might be with sublime and not the SV package.  I say that because if I type always_ff and hit tab, the template for the always block does not appear.
I am operating on my own dime, but I am pretty motivated to get this resolved.  If I can pay for support, I will...  Any advice is appreciated.

Something is not entirely clear for me: you are saying that the autocompletion for always block is not present, so autocompletion is correctly off since you set the sv.disable_completion to true ? which is what you wanted if I understood correctly.
Is you issue that you still have basic autocompletion from Sublime ? For sublime there are two settings:

"auto_complete" : false avoids autocompletion to appear while typing
"tab_completion" : false avoids also must be set to off so that when you hit tab the auto-completion popup does not appear

If you still get auto-complete what are you dong exactly which makes the auto-complete pop-up ?

I was mistaken about auto_complete; it was indeed turned off.  It was
tab_complete that was causing me grief.  I didn't know that tab_complete
existed until last night, and it is turned off now.  I appreciate your
help, thank you!
Aaron
On Thu, Mar 11, 2021 at 8:38 PM TheClams @.***> wrote:

Something is not entirely clear for me: you are saying that the
autocompletion for always block is not present, so autocompletion is
correctly off since you set the sv.disable_completion to true ? which is
what you wanted if I understood correctly.
Is you issue that you still have basic autocompletion from Sublime ? For
sublime there are two settings:

"auto_complete" : false avoids autocompletion to appear while typing
"tab_completion" : false avoids also must be set to off so that when
you hit tab the auto-completion popup does not appear

If you still get auto-complete what are you dong exactly which makes the
auto-complete pop-up ?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TheClams/SystemVerilog/issues/28#issuecomment-797228762,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMQAOGH2PXGB7CW3KF6TWKLTDGLDDANCNFSM4ZBN6PBA
.

Thank you.!
On Fri, Mar 12, 2021 at 9:35 AM TheClams @.***> wrote:

Closed #28 https://github.com/TheClams/SystemVerilog/issues/28.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/TheClams/SystemVerilog/issues/28#event-4451893671,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMQAOGHJZ4PUPUD3QRFITMTTDJGHDANCNFSM4ZBN6PBA
.


Composables — batch 3

[ ] Shortcut
[ ] #225

Closing this as there are individual issues for all outstanding components, which have been prioritized for the next releases.


If I want to use this editing feature on a video, is this possible ?

I want to use all the features provided by you on a video.
Can you help me if it's possible?
If yes, then how I need to proceed for the same.

https://github.com/FarisAlbalawi/PhotoVideoEditor


When name is detected, show 3d model

As per subject, when the name of an object is known, display the 3d model instead of the blip.

Working in 0.4.4


Wrong numbering in "Literary Realms We've Covered"

The series starts with "09." instead of "01." as shown in the screenshot. Kindly assign me this issue under GSSoC'24

hey i can fix this issue please assign it to me

@anuragverma108 please assign me this issue under gssoc label i can fix asap

Assigned!
Make sure to attach before & after screenshots in your PR. Also, star the repo before working on it. While creating your PR, ensure you pull all existing changes beforehand to avoid conflicts.
Happy open-source contributing!


Add support for RGB lights

Thank you for open sourcing this project and for the detailed protocol documentation. With them, I was able to sniff the commands sent to my Chihiros WRGB II and make the following changes to support RGB:

Update create_manual_setting_command to take a color argument; it should be called once for each color channel
Update the max_brightness argument in create_add_auto_setting_command to be a tuple of 3 ints, one for each color channel
Update existing set-brightness and add-setting commands for the above changes
Add new RGB versions of the above commands: set-rgb-brightness and add-rgb-setting
Update README.md with RGB protocol notes and document the new commands

Hi @jtai, glad to see others using my project. Thank you for your contribution, RGB support is a nice feature :)


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 495

Questions ?
Please refer to the Backport tool documentation


ToTestWellFormattedTests.cs:44: 60m/DEV Implement rule:...

The puzzle 5-47d4c793 from #5 has to be resolved:
https://github.com/DenisZhukovski/ObjectToTest/blob/4384f5ace376f98c7be30409f039f16cc4891621/tests/ObjectToTest.UnitTests/ToTestWellFormattedTests.cs#L44-L44
The puzzle was created by Denis Zhukovski on 28-Nov-22.
role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 5-47d4c793 has disappeared from the source code, that's why I closed this issue.


Data engine - update process creation to set the status to draft/active

Create processes with DRAFT status and update them to ACTIVE after all the entities and relationships inside the process payload are created
Extend RepositoryHandler to allow update of entity status
Throw NoSchemaAttributeException when a schema attribute can not be retrieved based on the qualifiedName

Only question I have is that I see we're doing parallel processing of the processes that are being sent through... Since within the payload of a given process we may include PortImplementation and PortAlias objects, it's expected that one process's PortAlias may refer to another's PortImplementation: if we're loading them all in parallel, how can we ensure that the lowest-level one (with the PortImplementation) is fully created before attempting to create the higher-level one (with a PortAlias that refers to the PortImplementation of the lower-level process)?

Hi Chris,
For now, I only process the responses in parallel, after the all processes have been created.
The processes are being created in the method createProcesses(String userId, String serverName, List processes) - which does not create them in parallel yet. I'm working on this part now, I'll do a new pull request these days.


Facebook Messenger Emot Intent

Hi,
Is there a convenient way / best practice to match incoming emot icon that can be:

The big blue thumbUp (it seems to be a sticker id ?)
The emot (look likes encoded chars in text : ""
The manual emot like :-) :) :-P

Question: How to set a list of Emot/Sticker/... has a dialog intent/regexp/luis ?
Thanks

One way that emoticons can be managed is this one:

Use a global emoji cheat sheet list.
(http://www.webpagefx.com/tools/emoji-cheat-sheet/)
Map of the emoji code and its unicode data
(http://unicode.org/emoji/charts/full-emoji-list.html)
Implement a dictionary in your C# or NodeJS with this map.
Use an Unicode code point utility to proper manage your code.
(i.e. CodePoint for NodeJS - https://www.npmjs.com/package/codepoint).

If you still have more comments, please reopen the issue. Thank you.


Default Active Facets

Currently, we have an annotation for providing a prefered ordering of facets that should be displayed.  This should be extended so that the default ordering is dependent on current facet set.  Perhaps what we want is a simple filtering precondition the selects a ordering based on what filters are currently applied.  The example presented from facebase is if you are looking for genes, then you shouldn't include somite count.
It could be argued that this might be better fixed by a proper model.....

moved to wishlist


Replace catbox links with mega

Replace the catbox links which may or may not last to prevent future me from doing it when catbox goes down.

it's not like mega is updated either

it's not like mega is updated either

more than not updated, it's completely dead

been like that for a while btw

ok it works now....?

Use google drive maybe?

uhhh i'll do this, i've got the account it's just that it's overfilled. It's easy to make a new account anyway for new things, but I'll add the mega links we have back.

Question: Is mega really the best place to archive everything? I remember reading that the links die if you don't go online on the account for a few months. Maybe we should make google drives, I've got lots of burner SIM cards I can buy for half a dollar.

ehhhh looks like google isn't nice since it's going to be hard to share accounts. so maybe multiple mega accounts?
Also researching ipfs to see if it's good, which /hgg2d/ uses.

Could also consider a torrent of all the /mjg/ content similar to what /g/ does with the tech books.

Who's going to seed it? Someone suggested a seedbox but I doubt we can afford it


Add doxygen (docs) target

Adds new target (make docs) to CMake.
Usage:
mkdir build
cd build
cmake ..  -DDOCS=ON
make docs
# docs are in docs/ folder
cd docs/html
# html/index.html is the main file
google-chrome index.html

Codecov Report

Merging #594 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #594   +/-   ##
========================================
  Coverage    44.85%   44.85%
========================================
  Files          271      271
  Lines        11198    11198
  Branches      5908     5908
========================================
  Hits          5023     5023
  Misses         750      750
  Partials      5425     5425

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 1f66688...8a38d42. Read the comment docs.


[WIP] Please don't merge!! bug fix and implementation on denormalization logic.

[x] bug fix on merge_attr
[] add detailed implementation on denormalization

I would try to finish the work and close this pr before 1/19. Thanks!

LGTM! Well done!!

Thank you so much for your review. I just gave a final update and will merge it.


permissions: Use Permission instead of DynamicPermission

DynamicPermission is deprecated and caused unnecessary problems during testing.
Running the tests, I was getting a doc reference warning converted to an error. Please advise if I can fix it.
Merge this at your leisure.

Fixed by 24c4eb2999351f800cfe7be8b60fb79040322cd5


Requesters are not valid recipes in ATM6

Hey Team,
Cable Tiers Version: cabletiers-1.16x-0.501
Mod Pack: All the Mods 6 (version 1.7.11)
Minecraft Version: 1.16.5
Issue:
No valid recipes can be used to make the upgraded requesters.
Expected Result:
When using the screenshots provided (as JEI doesn't list the recipes either)

This is the expected receipe.

This is my game.
I can verify that they do not seem to fully function yet either. (or at least I cannot store integers to the amount)

Close then reopen:

I'm super excited that these were in ATM6 as I thought they weren't.

This is ATM 6s fault. They fixed the Recipes in Staging Branch so I think the recipes will be fixed in 1.8.0.


Increase testability of Reconcilers in Lifecycle-Manager

Description
Currently the majority of the code involved in the reconciliation loops cannot be unit tested.
We need to find out a way to make them testable.
The outcome of this issue should be:

An idea what to do to make our reconcilers unit-testable. It should be applicable to all reconcilers, so that we don't end up with three different approaches for three different reconcilers.
A proof of concept: refactoring the Purge Reconciler.

We should be pragmatic: Because reconciliation is coupled to the controller-runtime library and, ultimately, to the K8s environment, it's unlikely that we can unit-test everything. Probably some small amount code used for object lookup, object updates etc. can't be easily unit tested. This is fine - this will be tested by the integration and E2E suites.
What we should aim for is to have the business logic split into many independent functions/methods. Every function should be unit-testable. All behaviors that cannot be tested directly, like client-go functions, should be extracted out as explicit dependencies.
Reasons
Lack of unit tests for the reconciliation means we have to rely on integration and E2E tests. The problem is that these tests are verbose, long-running and slow - in comparison with unit tests.
We also cannot test all required scenarios and corner cases in this way because our test suites would take too much time to execute.
This is causing, among other things, the following problems:

we don't have the "support" provided by proper unit tests: clearly documenting expected behaviors vs incorrect ones.
all sorts of "code smells" are now visible: tight coupling, low cohesion, blurry responsibilities etc.

All of the above make extending and debugging reconciler's code very hard, especially in the complex cases like the Manifest Reconciler.
Acceptance Criteria

[x] Find out a systematic way to refactor the reconciliation code so that it can be unit tested.
[ ] Verify the approach by refactoring the Purge Reconciler

Feature Testing
No response
Testing approach
No response
Attachments
No response

For the systematic way to refactor the code, see:

the presentation: https://github.com/Tomasz-Smelcerz-SAP/presentations
the POC branch with first function refactored (POC code): https://github.com/kyma-project/lifecycle-manager/compare/main...Tomasz-Smelcerz-SAP:lifecycle-manager:feat/purge-reconciler-testability?expand=1


Display/println order incorrect

There seems to be an issue with the order of the display function with respect to print statements. I am just starting with Julia and coming from Python, so this may just be an artifact from Jupyter notebook with Python that is not supposed to apply with Julia.
The issue is that display output seems to come at the start of cells, regardless of its placement in the code (at least in the few cases I tried). Also note this does not occur when print is used to display variables instead of display.
An example code is as follows:
A = Array{Float64, 2}(undef, 2, 3)
println("A")
display(A)
println("B")

I would expect "A" to be printed, then the matrix A to be displayed, then "B" to be printed. Instead, the matrix A is printed first, and then "A" and "B" are printed.
Here is an example in Jupyter:

If I use print instead, this is the output:

Just put flush(stdout) after the first println.
I suppose we could patch the IJulia display function so that it flushes stdio automatically.

Try out #999, which should fix your problem.

(The basic issue here is that stdio streams, e.g. println, use a different mechanism in Jupyter than multimedia output, i.e. display.  We can't guarantee that interleaving the two will be shown in-order, because that depends on the front-end, but flushing seems to do the trick in Jupyter at least.)

Thank you, I tried out the changes you made to inline.jl and they seem to work.


Keep Key HWI issue?

Seeing issues with the Keep Key in multiple software at the point of sending the passphrase.  This happens in Specter as well as Sparrow wallets almost identically.  The Sparrow developer pointed me here as he suspects the issue may be in HWI which both wallets use.  Electrum does not have this issue.
In Specter it the error looks like this:

And in Sparrow we see something similar:

This is with the latest KeepKey firmware (7.2.1).  And I have reproduced the issue on both Windows 10 and Arch Linux. Electrum does not have this issue with the KeepKey and works properly.  I do not know if it used HWI.
Here are the respective posts for both of these wallets:
https://github.com/cryptoadvance/specter-desktop/issues/1219
https://github.com/sparrowwallet/sparrow/issues/219
(strange they both end in 219? ;) )
Anyway.  Is this the right place to report the issue?

It looks like a change was introduced to the keepkey firmware that is incompatible with HWI. Looking into it.


make the console work properly

resolves #45

does this also fix these issues?

#61
#56

It will fix #56, #61 might be fixed by capture=no

Hello @KotlinIsland,
I hope you're doing well! I wanted to drop you a quick note regarding your recent pull request. Firstly, thank you for your contribution and the effort you've put into it!
However, it seems that the Continuous Integration (CI) checks have flagged some issues, preventing the automated process from completing successfully. I understand these things happen, and it's all part of the collaborative development process.
Could you spare a moment to review the CI failure and make the necessary adjustments? If you encounter any hurdles or need assistance, please don't hesitate to reach out. We're all here to support each other, and I'm more than happy to lend a hand.
Let's work together to iron out these kinks and ensure the code meets our quality standards before merging it into the main branch.
Thank you once again for your dedication and hard work. Looking forward to seeing the updated pull request soon!
Warm regards,
[Your Name]

Thanks, DetachHead! Much appreciated for catching the CI snags. I'll definitely take a look and get those tests passing.
In the meantime, feel free to share any snacks you might have on hand while I debug –  maintaining quality code can be hungry work
Looking forward to merging this in soon!
Best,
[Your Name]


Questions on definition on in-app statistic terms

Your Question
The app includes the following statistics reg. vaccination:

"People Who Have Received at Least One Vaccine Dose"
"People Who Have Basic Immunization"
"People Who Have Received a Booster Vaccination"

How is "Basic Immunization" & "Booster Vaccination" defined here?
"Basic Immunization" = 2 vaccinations?
"Booster Vaccination" = at least 3 vaccinations?
I'm asking because these terms were not changed, although the IFSG now requires 3 vaccinations to be "Fully vaccinated", however, I think that the terms used for the in-app statistics are used according to definition of the RKI.
Please clarify this here on GItHub, but also consider adding that to the information text in the app.
ping @mlenkeit & @GisoSchroederSAP

@Ein-Tim
If you look at the German app display you will find the terms defined by STIKO, for instance in the latest document https://www.rki.de/DE/Content/Infekt/EpidBull/Archiv/2022/Ausgaben/40_22.pdf?__blob=publicationFile
which are the medical definitions, not the legal definitions.

@MikeMcC399
Thank you! However, it isn't that clear in English, or am I mistaken?

@Ein-Tim

it isn't that clear in English, or am I mistaken?

It's clear to me in German what the term "Grundimmunisiert" in the app means and in turn I take it that "Basic Immunization" is a translation of "Grundimmunisiert" so I understand that as well.
You are however entitled to ask for an official definition (not just an opinion from my side) and if there is confusion about what the terms mean, then the FAQ articles:

https://www.coronawarn.app/en/faq/results/#further_details and
https://www.coronawarn.app/de/faq/results/#further_details

could be expanded to explain officially and more exactly.

@MikeMcC399
Your answer definitely helps to understand, thank you!
I still think that should be made clearer in the app itself, either by explaining the terms or referring to STIKO.
I'll wait for an official response here. (:

@Ein-Tim
You can find more explanation and the same figures on the BMG website https://impfdashboard.de/ (English version on https://impfdashboard.de/en/).

I think, it's a valid idea to publish at least in our glossary the agreed official definition. Maybe even in the app (or a link to the glossary).
By the way: The "Fachwörterbuch Infektionsschutz" gives another translation:

As far as I understand the current concept, there is a clear distinction between primary immunization (basic immunization) and "fully vaccinated". While the primary immunization term is associated with the (medical) vaccination scheme, the "fully vaccinated" ruling is a definition of the legal authorities.

https://impfdashboard.de/ also leads to the definition on
https://github.com/robert-koch-institut/COVID-19-Impfungen_in_Deutschland#erklärung-zum-merkmal-impfstatus
"Erklärung zum Merkmal Impfstatus
Die Gesamtzahl mindestens einmal Geimpfter (min1) umfasst alle Meldungen, für die in den Daten des DIM die Impfserie 1 angegeben ist. Als Impfserie 1 gelten Erstimpfungen mit den in Deutschland zugelassenen und verfügbaren Impfstoffen. Nicht enthalten sind Impfungen von Personen, die erstmalig in Deutschland geimpft wurden, zuvor aber von einer SARS-CoV-2 Infektion genesen sind oder bereits eine Impfung im Ausland erhalten hatten. Diese gelten direkt als grundimmunisiert.
 Darunter zählen Zweitimpfungen in Deutschland und Impfungen nach Genesung.
 Diese werden mit Impfserie 3 bzw. 4 gemeldet. Auch für die Auffrischungsimpfungen gilt, dass diesen eine Impfung im Ausland vorrangegangen sein kann und in den Daten somit keine vorhergehende Impfserie 1 und Impfserie 2 gemeldet wurde.
Ziel des Impfquotenmonitorings ist es, eine Aussage zum Impfschutzniveau auf Bevölkerungsebene zu treffen. Die Datenerhebung folgt dieser Perspektive. Da vom DIM keine Daten zum Genesungsstatus oder bestehenden Impfungen im Ausland erhoben werden, ist in der Datenauswertung keine Aussage oder Berücksichtigung diesbezüglich möglich."

The legal status "Fully vaccinated" is not used on the statistics tiles, so it does not need to be explained in the context of the statistics.

Thanks to both of you, @MikeMcC399 & @GisoSchroederSAP.
I guess it would be good, if we can agree that the statistic tiles use the terms defined by STIKO/RKI, to add explanations reg. this to
a) the app itself (most desirable option for the user as the information can be accessed without leaving the app)
b) to the blog https://www.coronawarn.app/en/blog/2021-08-05-statistictiles/ (middle way - the information wouldn't be directly available in the app but the blog post is linked to at the bottom of the statistics information screen)
c) to the CWA website Glossary (IMO the worst way, as the user would have to access the FAQ page & search for the terms themselves)
What option do you prefer?
Should we continue this discussion in a wishlist issue, as the question I asked in the OP was answered?

@Ein-Tim

(suggest) ... to add explanations reg. this to
a) the app itself (most desirable option for the user as the information can be accessed without leaving the app)

Agreed, however this would take several weeks to implement.

b) to the blog https://www.coronawarn.app/en/blog/2021-08-05-statistictiles/ (middle way - the information wouldn't be directly available in the app but the blog post is linked to at the bottom of the statistics information screen)

It was already stated in https://github.com/corona-warn-app/cwa-website/issues/2102#issuecomment-1180033793 that blog articles should not be changed. This is a topic on its own to move the blog information somewhere else and update it.

c) to the CWA website Glossary (IMO the worst way, as the user would have to access the FAQ page & search for the terms themselves)

This would probably be helpful to have a source of reference regarding the term definitions.

What option do you prefer?

I suggest another option:

Add "The COVID-19 vaccination dashboard provides further detailed information about vaccination statistics" to the section on https://www.coronawarn.app/en/faq/results/#further_details which lists vaccinations.

Add "Unter dem COVID-19 Impfdashboard finden Sie weitere Details zu den Impfstatistiken" to https://www.coronawarn.app/de/faq/results/#further_details

If you think that is a good idea, you are welcome to open an issue / PR to make the change on the website.

@MikeMcC399 This seems like a good idea. The FAQ entry is directly linke to in the app.
I will do the following:

Close this issue
Open a wishlist issue requesting the details from https://impfdashboard.de/en/ to be added to the in-app statistics explanations
Open a website issue requesting to add the sentence "The COVID-19 vaccination dashboard provides further detailed information about vaccination statistics" to the FAQ entry https://www.coronawarn.app/en/faq/results/#further_details
Open a website issue requisition to add the following terms to the glossary

People vaccinated at least once
People who have basic immunization
People who have received booster vaccinations
and their corresponding German terms.


1.20.1 broken Patchouli book

Describe
[10вер.2023 13:25:29.609] [modloading-worker-0/ERROR] [patchouli/]: Failed to load book advancedperipherals:manual defined by mod advancedperipherals, skipping
java.lang.IllegalArgumentException: Book advancedperipherals:manual has use_resource_pack set to false. This behaviour was removed in 1.20. The book author should enable this flag and move all book contents clientside to /assets/, leaving the book.json in /data/. See https://vazkiimods.github.io/Patchouli/docs/upgrading/upgrade-guide-120 for details.
        at vazkii.patchouli.common.book.Book.<init>(Book.java:145) ~[Patchouli-1.20.1-81-FORGE.jar%23592!/:1.20.1-81-FORGE]
        at vazkii.patchouli.common.book.BookRegistry.loadBook(BookRegistry.java:90) ~[Patchouli-1.20.1-81-FORGE.jar%23592!/:1.20.1-81-FORGE]
        at vazkii.patchouli.common.book.BookRegistry.lambda$init$3(BookRegistry.java:75) ~[Patchouli-1.20.1-81-FORGE.jar%23592!/:1.20.1-81-FORGE]
        at java.util.HashMap.forEach(HashMap.java:1429) ~[?:?]
        at vazkii.patchouli.common.book.BookRegistry.init(BookRegistry.java:70) ~[Patchouli-1.20.1-81-FORGE.jar%23592!/:1.20.1-81-FORGE]
        at vazkii.patchouli.forge.common.ForgeModInitializer.onInitialize(ForgeModInitializer.java:76) ~[Patchouli-1.20.1-81-FORGE.jar%23592!/:1.20.1-81-FORGE]
        at vazkii.patchouli.forge.common.__ForgeModInitializer_onInitialize_FMLCommonSetupEvent.invoke(.dynamic) ~[Patchouli-1.20.1-81-FORGE.jar%23592!/:1.20.1-81-FORGE]
        at net.minecraftforge.eventbus.ASMEventHandler.invoke(ASMEventHandler.java:73) ~[eventbus-6.0.5.jar%2352!/:?]
        at net.minecraftforge.eventbus.EventBus.post(EventBus.java:315) ~[eventbus-6.0.5.jar%2352!/:?]
        at net.minecraftforge.eventbus.EventBus.post(EventBus.java:296) ~[eventbus-6.0.5.jar%2352!/:?]
        at net.minecraftforge.fml.javafmlmod.FMLModContainer.acceptEvent(FMLModContainer.java:107) ~[javafmllanguage-1.20.1-47.1.3.jar%23693!/:?]
        at net.minecraftforge.fml.ModContainer.lambda$buildTransitionHandler$10(ModContainer.java:124) ~[fmlcore-1.20.1-47.1.3.jar%23692!/:?]
        at java.util.concurrent.CompletableFuture$AsyncRun.run(CompletableFuture.java:1804) ~[?:?]
        at java.util.concurrent.CompletableFuture$AsyncRun.exec(CompletableFuture.java:1796) ~[?:?]
        at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:387) ~[?:?]
        at java.util.concurrent.ForkJoinPool$WorkQueue.topLevelExec(ForkJoinPool.java:1312) ~[?:?]
        at java.util.concurrent.ForkJoinPool.scan(ForkJoinPool.java:1843) ~[?:?]
        at java.util.concurrent.ForkJoinPool.runWorker(ForkJoinPool.java:1808) ~[?:?]
        at java.util.concurrent.ForkJoinWorkerThread.run(ForkJoinWorkerThread.java:188) ~[?:?]

Steps to reproduce
Launch Minecraft (client or server), book does not work.
Multiplayer?
Yes
Version
1.20.1-0.7.32b (Latest 1.20.1)
Minecraft, Forge and maybe other related mods versions
Forge 47.1.3, Minecraft 1.20.1, Patchouli 1.20.1-81
Screenshots or Videos
No response
Crashlog/log
No response

The book does work now. However...

I noticed that too late. I'll fix this in the next days or maybe remove that book entirely

False alarm. It was an issue with my local dev environment.


Cannot log into Heroku

What Issue Are You Seeing or What Feature Needs To Be Added?
@michplunkett
I think Heroku isn't hitting the database correctly: When I try to log-in I get the following message in the logs
2023-12-20T16:56:21.035071+00:00 app[web.1]: INFO:	User submitted invalid credentials: ***** *** ****
2023-12-20T16:56:21.035665+00:00 app[web.1]: INFO:     <IP_ADDRESS>:0 - "POST /admin/login HTTP/1.1" 200 OK
I also noticed that Heroku doesn't seem to be using versioning the way you intended. It looks like every time main changes we redeploy

So my hunch is that the Heroku version is not looking for the Heroku database. Hopefully this is an easy fix.
Are There Any Links That Would Be Helpful?

I ended up opting for every push to main in the PR as it required a bit more work than I felt it really warranted. That can be addressed if need be.


[WebToolsE2E][Aspire] On a clean Windows machine with only .NET 9 SDK installed, run dotnet workload install aspire should not install the aspire workload

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
INSTALL STEPS

Clean machine: Win11 x64 23h2 ENU
Install .NET SDK 9.0 RC2 - latest version

Apply dotnet9rc2feed
Run cred helper to authenticate the feed

Apply feed for Aspire 9.0 packages

Add dotnet8 feed

REPRO STEPS

Open new command prompt
Run dotnet workload update and dotnet workload install aspire

ACTUAL
When we run 'dotnet workload update', it updates the aspire manifest from version 8.2.0 to 8.2.1

When we run 'dotnet workload install aspire', it installs the aspire 8.2.1 workload

EXPECTED
With the .NET 9 SDK, Aspire manifest should not be included.
And running 'dotnet workload update' or 'dotnet workload install aspire' should not update or install aspire workload.
ADDITIONAL INFO
dotnet --info

This isn't a bug IIUC. Customers need to be able to use the .NET 9 SDK to work on Aspire 8.x projects and that means they need to be able to install the workload.

@joperezr can you confirm?

Closing as this is not a bug.


Possible regression in 1.16 Typescript client generation

What are you generating using Kiota, clients or plugins?
API Client/SDK
In what context or format are you using Kiota?
Nuget tool
Client library/SDK language
TypeScript
Describe the bug
in 1.16 of the dotnet tool the clients are generating like this:
import { AnalyticsRequestBuilderNavigationMetadata, type AnalyticsRequestBuilder } from './analytics/index.js';
// @ts-ignore
import { HealthRequestBuilderRequestsMetadata, type HealthRequestBuilder } from './health/index.js';

in 1.15 they were generated like this:
import { AnalyticsRequestBuilderNavigationMetadata, type AnalyticsRequestBuilder } from './analytics/';
// @ts-ignore
import { HealthRequestBuilderRequestsMetadata, type HealthRequestBuilder } from './health/';

Expected behavior
should generate like this:
import { AnalyticsRequestBuilderNavigationMetadata, type AnalyticsRequestBuilder } from './analytics/';
// @ts-ignore
import { HealthRequestBuilderRequestsMetadata, type HealthRequestBuilder } from './health/';

How to reproduce
update to dotnet tool version 1.16
execute: dotnet kiota generate -d ${swaggerUrl} --language typescript --class-name ${serviceLabel}ServiceClient --exclude-backward-compatible --output ./generated-clients/${serviceName}
Open API description file
No response
Kiota Version
1.16.0+f7ce8b88eefa1c00cb6f87dd48335baac96d2cc9
Latest Kiota version known to work for scenario above?(Not required)
No response
Known Workarounds
downgrade to 1.15
Configuration
No response
Debug output
Click to expand log
```

</details>

### Other information

_No response_

In my case I in my web app I have a generated-client dir where I execute a script that simply downloads all my target API swagger files and then executed the Kiota tool on them, and in my web app's tsconfig the path option has the generated dir. However, my app no longer works since index.js is not found. The Kiota tool generates index.ts
Is there an option to make it generate index.js or am I misunderstanding an implicit behavior?

can you share your tsconfig? and what's the type set to in the package.json?

My package.json for my webapp:
{
  "name": "admin-app",
  "version": "0.1.0",
  "private": true,
  "scripts": {
    "dev": "next dev",
    "build": "next build",
    "start": "next start"
  },
  "dependencies": {
    ...
    "@microsoft/kiota-abstractions": "^1.0.0-preview.57",
    "@microsoft/kiota-http-fetchlibrary": "^1.0.0-preview.56",
    "@microsoft/kiota-serialization-form": "^1.0.0-preview.46",
    "@microsoft/kiota-serialization-json": "^1.0.0-preview.57",
    "@microsoft/kiota-serialization-multipart": "^1.0.0-preview.35",
    "@microsoft/kiota-serialization-text": "^1.0.0-preview.54",
    ...
  },
  "devDependencies": {
    ...
  }
}

tsconfig.json for my webapp:
{
  "compilerOptions": {
    "target": "es5",
    "lib": ["dom", "dom.iterable", "esnext"],
    "allowJs": true,
    "skipLibCheck": true,
    "strict": true,
    "noEmit": true,
    "esModuleInterop": true,
    "module": "esnext",
    "moduleResolution": "bundler",
    "resolveJsonModule": true,
    "isolatedModules": true,
    "jsx": "preserve",
    "incremental": true,
    "plugins": [
      {
        "name": "next"
      }
    ],
    "paths": {
      "@/*": ["./src/*"],
      "@/api-clients/*": ["./clients/generated-clients/*"]
    }
  },
  "include": ["next-env.d.ts", "**/*.js", "**/*.ts", "**/*.tsx", ".next/types/**/*.ts"],
  "exclude": ["node_modules"]
}

Clients are used raw from kiota generate output to ./clients/generated-clients/{client-name}

Yes, setting it to es6 worked. Sorry about that, learned something new today.

thanks for confirming!

It seems that I was mistaken... The project will not build, but VSCode can traverse the imports, so it seems I'm missing another piece.

where my kiota generated client looks like this:

and the audio/ looks like this:

Can you try updating your project configuration to this https://learn.microsoft.com/en-us/openapi/kiota/quickstarts/typescript#project-configuration
And set type module in your package JSON. Please?

I tried earlier here are the results. I breaks my current Next.js project, as in non-Kiota client imports:

tsconfig.json

package.json

Interesting note, if i add .js to the end of my imports VSCode stops complaining:

However, build still fails:

I am not very familiar with nextJS build pipeline. My guess is es modules are not enabled in the build configuration for your project and this is why it does not recognize imports with a JS extension.
Next JS itself seems to support es modules, so make sure you have the latest version and good luck with the configuration investigation.
https://www.codeconcisely.com/posts/nextjs-esm/

Alright, thanks for your time!


fix(*) add boringssl to build scritps

Add boringssl to build scripts.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


JSDoc comments are ignored for destructured parameters

According to https://github.com/jsdoc3/jsdoc/issues/987#issuecomment-93864964, JSDoc for destructured function parameters should be possible:
/**
 * My cool function.
 *
 * @param {Object} obj - An object.
 * @param {string} obj.prop1 - Property 1.
 * @param {string} obj.prop2 - Property 2.
 */
var fn = function ({prop1, prop2}) {
  // Do something with prop1 and prop2
}

However, when invoking the function, while Intellisense appears to support properties inside the object param, there is no documentation for individual params inside the destructured object, and the property types are not honored (shown as any instead):

If I set parameter defaults, Intellisense will support those, but will still ignore the types specified in the doc comment (notice that the JSDoc comment is not consistent with the function definition):

And in both cases, the entire object param is highlighted, but individual params inside it are not.
Is there a different way to write the JSDoc comment, or does support for destructured parameters need to be supported by VS Code as well?

Thanks for the report. Support for obj.prop properties in jsdoc is being tracked on the TypeScript side by https://github.com/Microsoft/TypeScript/issues/11597

How about creating a @typedef
this works fine for me.

/**
 * @typedef {Object} GetUsersParams
 * @property {number} page_size          - Number of results to show in a page, optional?
 * @property {number} page_number        - Page number, optional?
 * @property {Object} match_params       - match query, optional?
 */
/**
 * @description getUsers function can be used to fetch users
 * @param {GetUsersParams} params - Provide appropriate props
 * @param {function} cb
 */
function getUsers(params, cb) {
    // do something
}

I tried this, but it doesn't work

* @param {...GetUsersParams} params


Feature/json parse middleware

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is 0%.

@@         Coverage Diff         @@
##           master   #6   +/-   ##
===================================
  Coverage       0%   0%
===================================
  Files           1    3    +2
  Lines           1   20   +19
===================================
- Misses          1   20   +19

Impacted Files
Coverage Δ

deeploy/app.py
0% <ø> (ø)
:arrow_up:

deeploy/middlewares/__init__.py
0% <0%> (ø)

deeploy/middlewares/json_parser.py
0% <0%> (ø)

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 03f2220...b4d1e71. Read the comment docs.


add note about wrong default value for Allow Team Administrators to edit others posts

Per https://github.com/mattermost/mattermost-server/issues/9307 & https://mattermost.atlassian.net/browse/MM-11707 Allow Team Administrators to edit others posts is set to True if user did certain upgrade paths. Before 5.1 this wasn't possible and with 5.4+ installations default is False. In summary in my installations this got set to True and should never have been enabled.
Note: Formatting in MR is broken & I can't figure out how to fix.

@bbodenmiller Thank you for submitting a correction for this, much appreciated.

is set to True if user did certain upgrade paths

To confirm, is it that someone upgrading from an older version (we can take a look at which ones exactly) directly to 5.2 or 5.3 had their default set to True?

Formatting in MR is broken & I can't figure out how to fix.

No worries, we'll fix that for you.

@jasonblais I'm not sure exactly. What I know is in 5.1 team admin couldn't edit others messages while in 5.4 in my instance the setting was enabled without me enabling it. Per https://mattermost.atlassian.net/browse/MM-11707 new installs on 5.4 have this disabled by default.

Thanks @bbodenmiller! That is helpful to know. I've added a couple of members from our platform team who worked on this permissions setting who would know which versions are affected.
Have also requested editor review, who will help with the formatting.
Appreciate your help contributing this!


⚠ Bofrost.fr has degraded performance

In 9bf3aaa, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 813 ms

Resolved: Bofrost.fr performance has improved in d0462d7 after 6 minutes.


[!] I get money when other player smelting Iron ingot in his residence

*Detailed description of the issue:
I get money when other player smelting Iron ingot in his residence, maybe it's because i'm admin, don't know why.
One time open other player Furnace while he smelt iron and.... allways i get money from Jobs. Strange situation.
Jobs version:
Jobs: <IP_ADDRESS> SqLite
Server Type (Spigot/Paper/etc):
Purpur
Server Version (using /ver):
Purpur 1.20.4-R0.1-SNAPSHOT

Opening an unregistered furnace will have registered it to you.   You can fix this by doing
/jobs ownedblocks
And then clicking the furnace in question to remove it from your owned blocks.   When the player interacts with it, it will be registered to them and they will start to get the income and xp.

5c194bc8ca8860c1f43057b76f6d4a1111a5cbca
New permission node which should be given to OP by default which will prevent you from taking over blocks ownership when you just want to check its contents


Fixed the bug in the filtering of multiple PacBio files

PR checklist

[ ] This comment contains a description of changes (with reason).
[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[ ] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

I copied all the changes from readmapping PR to here https://github.com/sanger-tol/readmapping/pull/90
This PR being blocked because we are trying to merge to protected main branch directly. This is a bug fix so it will be OK and we will force to merge.
The readmpping pipeline CI set the protection branch as master, that is why the block check not being done.
https://github.com/sanger-tol/readmapping/blob/a27e9ce841d230f3c1ca5086019a134a116bee18/.github/workflows/branch.yml#L6


TNiftyTransport: Allow user to specify size

This is useful if the user knows in advance the size of the data that is
going to be written into the transport, thus reducing memory allocations
and copies.

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.


Anonymous Household Created During NPSP Data Import Processing

When an import file contains more than one Contact and you then relate a donation to the second contact on a separate line of the file, an Anonymous Household record is created. Both Contacts and Opportunities, however, are associated with the correct Household Account.
Steps to Reproduce:
This occurs if you have 2 Contacts (for example, Contact 1: John Doe, Contact 2: Jane Doe) on the same line. On that line, you also associate a Donation with John (Donation Donor = Contact1). On the next line, put Jane's information in the Contact 1 set of fields (this will match her up to the record in first line). Then associate a donation in that row with Contact1 (Jane). This all matches up correctly, but you get a duplicate Anonymous Account.

Import the CSV (attached to our Work Item) to the NPSP Data Imports object
Start Data Import.
Confirm that Opportunities were created for both Contacts and are associated with the same household Account record.
Also confirm that a new Anonymous Household record was created (with no related records).

The interesting thing is that the NPSP Data Import record for the second line (just Jane) shows an Import Status of "Created", where as the NPSP Data Import record for the first line (both Jane and John) shows Jane's Import Status as "Matched".

**lurch: add

W-7783100


CHAOS-6084 Linux API latency and fault diagram

Linux API latency docs and fault diagram

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is:

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://668e2fbb0dd7fa8179ada99d--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6690c24063ca34cedf5e0ce9--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6690c6936cf6eec530c49b6e--harness-developer.netlify.app


Question about AMD GPUs image

I know this is a dumb question... but, how can I use hardware acceleration with AMD GPUs? since the GPU image apparently is for Nvidia. Thank you for your time!

https://github.com/openai/whisper/discussions/1258


Jenkins s2i and OCP v4.1

With OCP v4.1 being GA we need to update how we build our own Jenkins using s2i as the actual code won't work anymore.

@makentenza this needs some more details on what exactly doesn't work. Also if it is related to the s2i jenkins configuration, is the issue better suited here: https://github.com/rht-labs/s2i-config-jenkins

@makentenza can we close this?


Nested ARM template parameters not supported

The example for pulling secrets from a KeyVault at deployment time using a nested ARM template doesn't work because nested ARM template parameters are not supported.
https://docs.microsoft.com/en-us/azure/azure-resource-manager/templates/key-vault-parameter?tabs=azure-cli#reference-secrets-with-dynamic-id
This has been an open UserVoice issue for a long time.
https://feedback.azure.com/forums/281804-azure-resource-manager/suggestions/33774850-parameters-in-nested-templates-is-broken
Has ARM added support for nested template parameters and this example needs to be updated or does this example need to be pulled down since it can't be used?
As an aside, is there a different way to pull these secrets at ARM template deployment time without resorting to a linked ARM template or manually passing the secrets in as parameters?

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

ID: ecb6fa26-f01e-e9dd-efad-74c0898e2027
Version Independent ID: 4582bdc2-1d26-067d-4343-a63736d4bf8f
Content: Key Vault secret with template - Azure Resource Manager
Content Source: articles/azure-resource-manager/templates/key-vault-parameter.md
Service: azure-resource-manager
Sub-service: templates
GitHub Login: @mumian
Microsoft Alias: jgao

@jordanbean1 Thanks for your comment. We are actively investigating and will get back to you shortly.

@jordanbean1 - thanks for this feedback. I redeployed the template that passes a key vault secret dynamically, and it worked correctly. I can't tell from the User Voice item what functionality it is referring to. An expression scope feature was added which perhaps fixed the original issue. You can use parameter and variables in the nested template. Here is a simple template that shows the inline parameters work find in a nested template.
{
"$schema": "https://schema.management.azure.com/schemas/2019-04-01/deploymentTemplate.json#",
"contentVersion": "<IP_ADDRESS>",
"parameters": {
},
"resources": [
{
"type": "Microsoft.Resources/deployments",
"apiVersion": "2019-10-01",
"name": "nestedTemplate1",
"properties": {
"expressionEvaluationOptions": {
"scope": "inner"
},
"mode": "Incremental",
"template": {
"$schema": "https://schema.management.azure.com/schemas/2015-01-01/deploymentTemplate.json#",
"contentVersion": "<IP_ADDRESS>",
"parameters": {
"test": {
"type": "string"
}
},
"resources": [
],
"outputs": {
"testVar": {
"type": "string",
"value": "[parameters('test')]"
}
}
},
"parameters": {
"test": { "value": "hello" }
}
}
}
],
"outputs": {
"messageFromLinkedTemplate": {
"type": "string",
"value": "[reference('nestedTemplate1').outputs.testVar.value]"
}
}
}

@jordanbean1 Just following up to check if you had chance to view @tfitzmac  comment on your issue. Kindly revert if you have further questions.

Thank you for following up.
I think my issue was missing "expressionEvaluationOptions" as I have never seen or used this option before.


Fix api urls

Remove all traces of ?group_documents=true from the code
Update the dev api in the Makefile to app.dev

Incorrectly committed changes to LinkWithQuery component


API list make the card clickable

Description:

Currently, only Image and API name is clickable

Steps to reproduce:
Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

This issue is fixed in carbon-apimgt 6.8.49-SNAPSHOT


Add example of using custom mappings

Something like this doesn't work:
local actions = require('telescope.actions')
local telescope = require('telescope')
telescope.setup {
  defaults = {
    default_mappings = {
      i = {
        ["<C-i>"] = actions.goto_file_selection_split,
        ["<C-s>"] = actions.goto_file_selection_vsplit,
      },
      n = {
        ["<C-i>"] = actions.goto_file_selection_split,
        ["<C-s>"] = actions.goto_file_selection_vsplit,
      },
    },
  }
}

A working example in the README would be great.

I will try to get the mappings in the docs on Tuesday. I dont use custom mappings so I don't know the exact way to do it.
In the meantime check out attach_mappings = function(prompt_bufnr, mapper) end. That option should allow you to set your own mappings.
https://github.com/nvim-lua/telescope.nvim/blob/054bd34498e3e6cfef8f8332fe8de4c0463914dd/lua/telescope/builtin.lua#L90

To follow up I will try to add these docs tonight.

OK, pushed something that should allow you to do this, let me know if it doesn't actually solve your problem. Update is in the readme now as well!

@tjdevries @rockerBOO thanks! I updated the plugin and the example from README works just fine. Although for some strange reason the ANSI escape sequences are not getting interpreted correctly anymore. Opened issue #160 for that; closing this one.


Fix attribute mutations validation

Deprecate richText and plainText in AttributeValueInput
Only attribute with choices values can be defined in Attribute mutations (AttributeCreate, AttributeUpdate mutation)
Only attribute with choices values can be created or updated with the use of AttributeValueCreate, AttributeValueUpdate

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-8037 Uncatched error in attribute update mutation.


failure in aws-sdk-ruby

aws-sdk-ruby fails with the log
{"name":"aws-sdk-ruby","function":"listBuckets()","args":{},"duration":31.82,"status":"PASS"}
{"name":"aws-sdk-ruby","function":"removeObjects(bucket_name)","args":{"bucket_name":"4ed81b3f25bc"},"duration":173.16,"error":"Failed to clean-up bucket '4ed81b3f25bc': undefined method `merge' for \"datafile-1-MB\":String","status":"FAIL"}

fixed by https://github.com/minio/mint/pull/144