Only allow valid types in Decal, Light3D projector and PointLight2D texture

If an invalid type is supplied (which can still be done from a script), a warning is printed (along with a workaround for ViewportTexture).
Testing project: test_decal_warning.zip

This closes https://github.com/godotengine/godot/issues/77564.
See also https://github.com/godotengine/godot/issues/54506,
https://github.com/godotengine/godot/issues/73400,
and https://github.com/godotengine/godot/issues/75868.

Preview

Before
After

WARNING: ViewportTexture is not supported as a PointLight2D texture (/root/Node3D/PointLight2D). As a workaround, use `get_image()` on the ViewportTexture to assign the returned value to the PointLight2D texture property, but keep in mind this is a slow operation.
     at: set_texture (./scene/2d/light_2d.cpp:405)
WARNING: AtlasTexture is not supported as a Light3D projector texture (/root/Node3D/OmniLight3D).
     at: set_projector (./scene/3d/light_3d.cpp:207)
WARNING: MeshTexture is not supported as a Decal texture (/root/Node3D/Decal2).
     at: set_texture (./scene/3d/decal.cpp:66)

There are these errors to handle for the GDExtension compatibility checker:
Validate extension JSON: Error: Field 'classes/Decal/properties/texture_albedo': type changed value in new API, from "Texture" to "CompressedTexture2D,PortableCompressedTexture2D,ImageTexture,CurveTexture,CurveXYZTexture,GradientTexture1D,GradientTexture2D,PlaceholderTexture2D,NoiseTexture2D".
Validate extension JSON: Error: Field 'classes/Decal/properties/texture_emission': type changed value in new API, from "Texture" to "CompressedTexture2D,PortableCompressedTexture2D,ImageTexture,CurveTexture,CurveXYZTexture,GradientTexture1D,GradientTexture2D,PlaceholderTexture2D,NoiseTexture2D".
Validate extension JSON: Error: Field 'classes/Decal/properties/texture_normal': type changed value in new API, from "Texture" to "CompressedTexture2D,PortableCompressedTexture2D,ImageTexture,CurveTexture,CurveXYZTexture,GradientTexture1D,GradientTexture2D,PlaceholderTexture2D,NoiseTexture2D".
Validate extension JSON: Error: Field 'classes/Decal/properties/texture_orm': type changed value in new API, from "Texture" to "CompressedTexture2D,PortableCompressedTexture2D,ImageTexture,CurveTexture,CurveXYZTexture,GradientTexture1D,GradientTexture2D,PlaceholderTexture2D,NoiseTexture2D".
Validate extension JSON: Error: Field 'classes/Light3D/properties/light_projector': type changed value in new API, from "Texture2D" to "CompressedTexture2D,PortableCompressedTexture2D,ImageTexture,CurveTexture,CurveXYZTexture,GradientTexture1D,GradientTexture2D,PlaceholderTexture2D,NoiseTexture2D".
Validate extension JSON: Error: Field 'classes/PointLight2D/properties/texture': type changed value in new API, from "Texture2D" to "CompressedTexture2D,PortableCompressedTexture2D,ImageTexture,CurveTexture,CurveXYZTexture,GradientTexture1D,GradientTexture2D,PlaceholderTexture2D,NoiseTexture2D".

I'm not sure how those hints are used in godot-cpp when there's multiple classes, cc @dsnopek

I'm not sure how those hints are used in godot-cpp when there's multiple classes

AFAIK godot-cpp doesn't use the properties from the JSON. Since C++ doesn't have properties I guess there's nothing to generate, so they only generate the getter/setter methods.
In C#, we use the getter/setter method signature to determine the property type not the hint, so it doesn't affect us either.

I have not a clue as to why the check is still failing but I wanted to mention it as a reminder.

If this is going to "break compatibility", it's better to do it properly. I was going to open a PR for that, but feel free to add this commit and use it to simplify the hints:
https://github.com/godotengine/godot/commit/ff8f797b54f64da2cc166971aa6e63d0dfa205c6
You can do e.g. "Texture2D,-ViewportTexture" to allow everything except ViewportTexture.

If this is going to "break compatibility", it's better to do it properly. I was going to open a PR for that, but feel free to add this commit and use it to simplify the hints: ff8f797 You can do e.g. "Texture2D,-ViewportTexture" to allow everything except ViewportTexture.

This code works, but it causes the items to be reordered with this hint (it's always the same order in every editor session):
"Texture2D,-AnimatedTexture,-AtlasTexture,-CameraTexture,-CanvasTexture,-MeshTexture,-Texture2DRD,-ViewportTexture"

I'd like the original order to be preserved.

Seems like HashSet does not preserve order when elements are erased 🤔
I updated the branch to fix it. It's a bit more complex and slower, but it should be fine.

Seems like HashSet does not preserve order when elements are erased 🤔 I updated the branch to fix it. It's a bit more complex and slower, but it should be fine.

Can you link to the branch in question? I can't find it on your fork.

Right, the force push invalidated previous commit. Here's the branch: KoBeWi/godot@%F0%9F%9B%91No_Resource_allowed

It works, thanks a lot! I've added you as a co-author :slightly_smiling_face:

CC @dsnopek Can you confirm that godot-cpp ignores these property hints? (see https://github.com/godotengine/godot/pull/88349#issuecomment-1967456455)

Rebased and tested again following https://github.com/godotengine/godot/pull/84443, it works as expected.

We still have the GDExtension compat issue to figure out, which is blocking this.

Sorry for missing all the pings on this one!
I can definitely say this won't have any effect on godot-cpp: we don't use properties, because there isn't really an equivalent in C++ - we just use the getters and setters.
However, this may have an effect on bindings for languages that do have properties. That said, I would expect that in most cases, they'd use the return value type of the getter, rather than the type hint in order to determine which type should be used for the property. There's plenty of properties which list multiple types, which would potentially make using that tricky.
If any bindings actually do use the property type, then the addition of the - types would probably break things. But I personally feel like this sort of change is acceptable in a new minor release - we've made other small breaking changes to the format of extension_api.json in previous minor releases. But I don't think this is something that should be cherry-picked to Godot 4.2 or earlier.

Rebased and tested again, it works as expected.
This now integrates https://github.com/godotengine/godot/pull/99959 by hiding non-manifold meshes in the CSGMesh3D mesh property.

Reposting my comment from the old thread above for visibility, as IMO that's still a blocker for doing these otherwise relatively straightforward changes:
Honestly I'm not sure I like this syntax to exclude unsupported types. We're excluding as many times as we'd have to explicitly include using the usual syntax.
And I think future proofing the addition of new types is not a good idea, as we don't know whether those new types would be supported. So it's better to state explicitly what we know we support. This would also make it easier possibly in the future to actually parse this hint and provide this information automatically in the class reference for example.

I amended the PR to finally fix the extension validation issue (same issue as in #90993, previous changes done in another stable version of Godot conflict with new changes to the same API, and the workaround is to list both in the current 4.3.stable.expected file).
I still have concerns about the new syntax in cases like Texture2D where we're excluding 7 types explicitly to keep 10, where we might want to just list the 10 types we know we support.
But let's not block it further, for other cases like CSGMesh it makes some sense (excluding 3 to keep 12).

Also added PointMesh to the list of primitives not supported by CSGMesh3D, as points are (I suppose?) non manifold.

Thanks!


Issue with trim_excess in dask.array.coarsen

I am trying to use dask.array.coarsen on 3-d arrays and am finding that the result depends on the chunking of the data, which doesn't seem right. The following example illustrates the issue:
In [1]: import numpy as np

In [2]: import dask.array as da

In [3]: array1 = da.random.random((623, 768, 768)).rechunk((322, 256, 256))

In [4]: array2 = da.random.random((623, 768, 768)).rechunk((623, 768, 768))

In [5]: import numpy as np

In [6]: da.coarsen(np.nanmean, array1, {1: 3}, trim_excess=True).shape
Out[6]: (623, 255, 768)

In [7]: da.coarsen(np.nanmean, array2, {1: 3}, trim_excess=True).shape
Out[7]: (623, 256, 768)

Since 768 is divisible by 3, I would expect the final array to have shape (623, 256, 768) as for array2. The result for array1 seems incorrect, so I think this is a bug?

I think the issue is that the coarsen is happening on each chunk so you are getting a different result than expected because 256 is not divisible by 3. That is why you get an error with array1 if you take out trim_excess=True. So I am not entirely sure if this is worth changing or not.
The relevant code is here:
https://github.com/dask/dask/blob/366c7998398bc778c4aa5f4b6bb22c25b584fbc1/dask/array/routines.py#L1303:L1305

I feel like a top-level function such as this should probably return the same result independent of how the data is chunked (unlike a function like say map_blocks which explicitly operates on chunks)

da.coarsen doesn't apply any overlaps between neighboring blocks, so the output is always dependent on the chunking of the input. There may be a bug here in terms of the output shape; I'm not familiar enough with coarsen to know, perhaps @mrocklin remembers - this method is one of the first things added to dask in 2015 and hasn't been touched much since. However, the output values will still differ depending on chunking, since no windowing is used.
For windowed computations that don't depend on how the data is chunked I recommend looking at da.map_overlap.

The result for array1 seems incorrect, so I think this is a bug?

Looks like it to me

I feel like a top-level function such as this should probably return the same result independent of how the data is chunked

+1
I think that the solution here is to verify that all chunks of the coarsened dimensions align with the coarsening, and if not, to call a minimally invasive rechunk automatically within this function.
This shouldn't add too much extra cost, and should make sure that we get correct and consistent results.

If someone wanted to solve this I would recommend the following approach:
Make a function that takes in the chunks for a single dimension and an integer coarsenting value and produces a new chunking that is aligned with that value
def aligned_coarse_chunks(chunks: List[int], multiple: int) -> List[int]:
    """ Returns a new chunking aligned with the coarsening multiple"""

>>> aligned_coarse_chunks((20, 10, 15, 23, 24), 10)
(20, 10, 20, 20, 20, 2)

Note that the sum of the two lists should be the same, all numbers should be multiples of the multiple, and any excess should be at the end.
Then I would apply that function to all of the dimensions that are being chunked with the right multiple.  If the resulting chunking is different from the inputs .chunks value then I would call rechunk with that combined tuple.

Here's my attempt at this function


AttributeError: '_Scholarly' object has no attribute 'use_proxy'

Can't understand why there is no attribute for use_proxy, im in the latest version. Is this because i'm windows user?
File "", line 1, in
scholarly.use_proxy(http=proxy, https=proxy)
AttributeError: '_Scholarly' object has no attribute 'use_proxy'

This fixes it - use_proxy is on nav.
scholarly.nav.use_proxy(**proxies)

On this issue, there is a bug where pip installs a different version of scholarly than what is in this repo. If you do not use the scholarly in this repo, you will run into this error, or an AttributeError module scholarly.scholarly has no attribute nav. I believe that is what this issue is really getting at.

Thank you very much for your reply, and indeed the scholarly.nav.use_proxy executed the command, however its seems like its stuck when iterating the search query (which doesn’t happen without proxy), you think you know why its stuck?
Thanks a lot for the support, appreciate it,
Itay
From: Emile Petrone<EMAIL_ADDRESS>Sent: Wednesday, May 20, 2020 12:51 AM
To: scholarly-python-package/scholarly<EMAIL_ADDRESS>Cc: gambastyle<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [scholarly-python-package/scholarly] AttributeError: '_Scholarly' object has no attribute 'use_proxy' (#107)
On this issue, there is a bug where pip installs a different version of scholarly than what is in this repo. If you do not use the scholarly in this repo, you will run into this error, or an AttributeError module scholarly.scholarly has no attribute nav. I believe that is what this issue is really getting at.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/scholarly-python-package/scholarly/issues/107#issuecomment-631102515 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AOYLJUEBWRGKEJOUOPCJMZ3RSL5KTANCNFSM4NFDGXXA .  https://github.com/notifications/beacon/AOYLJUCRJONU7MTR7HY6AF3RSL5KTA5CNFSM4NFDGXXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEWO5YMY.gif

To improve the pattern, I have included a use_proxy method in Scholarly now.

This is the error I get:
search_query = scholarly.search_author("name")
df_search = pd.DataFrame(map(vars, search_query))
Traceback (most recent call last):
File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\connectionpool.py", line 380, in _make_request
httplib_response = conn.getresponse(buffering=True)

TypeError: getresponse() got an unexpected keyword argument 'buffering'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\contrib\pyopenssl.py", line 280, in recv_into
return self.connection.recv_into(*args, **kwargs)

File "D:\Dropbox\Anaconda\lib\site-packages\OpenSSL\SSL.py", line 1840, in recv_into
self._raise_ssl_error(self._ssl, result)

File "D:\Dropbox\Anaconda\lib\site-packages\OpenSSL\SSL.py", line 1646, in _raise_ssl_error
raise WantReadError()

WantReadError
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "", line 1, in
df_search = pd.DataFrame(map(vars, search_query))

File "C:\Users\itaya\AppData\Roaming\Python\Python37\site-packages\pandas\core\frame.py", line 469, in init
data = list(data)

File "D:\Dropbox\Anaconda\lib\site-packages\scholarly_navigator.py", line 172, in search_authors
soup = self._get_soup(url)

File "D:\Dropbox\Anaconda\lib\site-packages\scholarly_navigator.py", line 161, in _get_soup
html = self._get_page(_HOST.format(url))

File "D:\Dropbox\Anaconda\lib\site-packages\scholarly_navigator.py", line 86, in _get_page
timeout=_TIMEOUT)

File "D:\Dropbox\Anaconda\lib\site-packages\requests\sessions.py", line 521, in get
return self.request('GET', url, **kwargs)

File "D:\Dropbox\Anaconda\lib\site-packages\requests\sessions.py", line 508, in request
resp = self.send(prep, **send_kwargs)

File "D:\Dropbox\Anaconda\lib\site-packages\requests\sessions.py", line 640, in send
history = [resp for resp in gen] if allow_redirects else []

File "D:\Dropbox\Anaconda\lib\site-packages\requests\sessions.py", line 640, in
history = [resp for resp in gen] if allow_redirects else []

File "D:\Dropbox\Anaconda\lib\site-packages\requests\sessions.py", line 218, in resolve_redirects
**adapter_kwargs

File "D:\Dropbox\Anaconda\lib\site-packages\requests\sessions.py", line 618, in send
r = adapter.send(request, **kwargs)

File "D:\Dropbox\Anaconda\lib\site-packages\requests\adapters.py", line 440, in send
timeout=timeout

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\connectionpool.py", line 601, in urlopen
chunked=chunked)

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\connectionpool.py", line 383, in _make_request
httplib_response = conn.getresponse()

File "D:\Dropbox\Anaconda\lib\http\client.py", line 1336, in getresponse
response.begin()

File "D:\Dropbox\Anaconda\lib\http\client.py", line 306, in begin
version, status, reason = self._read_status()

File "D:\Dropbox\Anaconda\lib\http\client.py", line 267, in _read_status
line = str(self.fp.readline(_MAXLINE + 1), "iso-8859-1")

File "D:\Dropbox\Anaconda\lib\socket.py", line 589, in readinto
return self._sock.recv_into(b)

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\contrib\pyopenssl.py", line 292, in recv_into
rd = util.wait_for_read(self.socket, self.socket.gettimeout())

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\util\wait.py", line 33, in wait_for_read
return _wait_for_io_events(socks, EVENT_READ, timeout)

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\util\wait.py", line 26, in _wait_for_io_events
selector.select(timeout) if key[1] & events]

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\util\selectors.py", line 320, in select
self._writers, timeout)

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\util\selectors.py", line 64, in _syscall_wrapper
return func(*args, **kwargs)

File "D:\Dropbox\Anaconda\lib\site-packages\urllib3\util\selectors.py", line 310, in _select
return select.select(r, w, [], timeout)

From: Victor Silva<EMAIL_ADDRESS>Sent: Wednesday, May 20, 2020 1:13 AM
To: scholarly-python-package/scholarly<EMAIL_ADDRESS>Cc: gambastyle<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [scholarly-python-package/scholarly] AttributeError: '_Scholarly' object has no attribute 'use_proxy' (#107)
To improve the pattern, I have included a use_proxy method in Scholarly now.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/scholarly-python-package/scholarly/issues/107#issuecomment-631111527 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AOYLJUBAXMQYCHY4ZTTODGDRSL77JANCNFSM4NFDGXXA .  https://github.com/notifications/beacon/AOYLJUELRQKA36Z2P7XHXYDRSL77JA5CNFSM4NFDGXXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEWO76ZY.gif


Unbale to found any helping material Live Camera with Pose Detection API using flutter dart.

I am facing an issue not found any example where Live Camera implemented with Pose Detection API using flutter dart.
Please let me know anyone using it or found any helping material related this i am just found Pose Detection using static image but not using Live Camera..
Thanks

Hi @marslannasr7koncepts thanks for opening an issue.
I had written the example for live detection on new-apis branch but I'm not getting proper results while using the camera stream. So for time being I have commmented out that part. If I have any further developments I will update you. For time being you may refer to this example which covers live pose detection.
Thanks.

Thanks @bharat-biradar for response I have a Question did it work when Camera is streaming no need to click on button just show Pose Detection using Live Camera not depend on any Button Click etc.
And what type of issue your facing and how much time it take to fix?
Thanks

@marslannasr7koncepts the problem is occurring with live stream. I have tried all the methods possible but still when it comes to live stream detection no poses are being detected from the camera stream. It has been nearly a week still I haven't found a solution, so I cant say for sure for now when issue will be resolved.
Sorry for the inconvenience caused.

@bharat-biradar do you have the same issue when you are trying to perform live text recognition using the phone camera?

@malithkuruppu I haven't tried live text detection yet, I will get back to you once I have tried it.

@bharat-biradar is there any solution to the above problem?

@malithkuruppu Sorry but I have not found any solution, I was thinking of filing an issue on the official google ml kit api. Maybe there I can find why this problem is happening.

@bharat-biradar appreciate if you can open an issue! Please leave the link to the issue in this thread.

@malithkuruppu  link for the issue opened in Issue Tracker.
Also I asked a question on stackoverflow if this helps

@bharat-biradar will you be updating the code in this repo for live camera text and barcode recognition?

@bharat-biradar also what is the issue that needs to be addressed at this moment for the live camera text and barcode recognition to work?

@malithkuruppu The issue is related to rotation degree that we have to pass as a parameter while creating the input image. May be due to wrong calculation of rotation degree neither text nor poses are being detected.

@bharat-biradar  the issue I am having is that I am unable to use live camera preview for text and barcode detection using ML kit in a Flutter app. Do you have a solution for this problem?

@malithkuruppu Yes I have solution to display live preview but don't you want to use the camera stream data to be processed.

@bharat-biradar I am not quite sure how about to use live camera preview. As you said, will need camera stream data for processing with ML kit.
So the current issue in the solution you have is that when the device is tilted, the app stops working?
Does it work if the rotation of the screen is locked?

@malithkuruppu I'll share you a gist for using live camera preview. But it's not a problem with the plugin itself.

@malithkuruppu  Currently yes, this plugin is not officially from google and they mentioned in this issue that they are not thinking of creating a flutter plugin currently.

@bharat-biradar do you have a branch that you are working on specifically for this issue?

@malithkuruppu Except for master on both the branches I'm trying trial and error. You can see them for reference. Currently, I'm adding support for a new API and not focussing on this problem as I am not having any developments.

@malithkuruppu , @marslannasr7koncepts : pull the latest code from master and run the example app, we added support for the camera plugin


feat: security ci to check vulnerability in superset added for superset

SUMMARY

Security ci contains the following modules which detect the vulnerabilities
-> Bandit
-> PIP Audit
-> safety
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

@dpgaspar  I have created a workflow that provide information about the vulnerability. Mostly uses open source projects and  database.
Mentioning the issue here https://github.com/apache/superset/issues/23621

@villebro have you guys checked ?

It seems this PR slipped off our collective radar. It's still an interesting idea, but seems to have failed CI because of the added test itself. Maybe we can set some stuff to warn instead of error, or clean up the errors to get it through? In any case, I'm closing and reopening to reboot CI, since the logs have vanished.

Hi, @rusackas  thanks for looking at it.
@mistercrunch That is a great idea for sending alerts,
for the second one: it must run on every pr as there can be vulnerabilities that are found later not when the dependency gets upgraded.

@mistercrunch agreed with it, a scheduled workflow is fine.
So to summarize
-> Must run on PRs that are doing some changes in dependencies
-> A Scheduled action that checks on a daily or weekly basis for vulnerabilities and send mail or alerts.

Yes exactly!
Here's an example of using paths: in GitHub actions https://github.com/apache/superset/blob/master/.github/workflows/check_db_migration_confict.yml#L4-L5
Given bandit is python-only, I think the path you want to add is requirements/**


ikos: 2.1 → 3.0

Motivation for this change
Fixes & improvements: https://github.com/nasa-sw-vnv/ikos/blob/master/RELEASE_NOTES.md#user-content-ikos-version-30-release-notes
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

cc/ @AtnNn maintainer

Merged into 20.09 as 07e5844fdf6fe99f41229d7392ce81cfe191bcfc


[Feature Request] Collision with Create contraptions

I'm not sure how much of this needs to happen on Create's end vs. on your end, but it would be awesome if Automobility cars could collide properly with Create contraptions. It would give players a lot more options to make ramps, elevators, gates, and so on--especially when playing in Survival.

Create support has been planned!
From the CurseForge page:


Allow custom endpoint url configuration. Closes #1050

Closes #1050.
Integration test logs

Logs
Add passing integration test logs here

Example query results

Results
Add example SQL query results here (please include the input queries as well)

@srgg Thanks again for the contribution, we're excited for our plugin to now be compatible with local AWS services, LocalStack, and similar tools!


[Build] Allow exceptions and runtime type information

Build issue(s)
Greetings,
Some SDKs work with exceptions and require rtti. In order for such SDKs to be integrated, flags like -fexception and -frtti have to be set. As far as I can see, there are no methods for setting -fexceptions and -frtti flags other than changing the compiler/BUILD.gn file.
Additionally, rtti flag can be set by providing an additional argument to gn:
enable_rtti = true
which overrides these few lines in compiler/BUILD.gn:
declare_args() {   /# Enable Runtime Type Information (RTTI)   enable_rtti = current_os == "mac" || current_os == "ios" }
This method does not sit well with me because of overriding.
However, such mechanism doesn't exist for exceptions, so the only way to set the -fexceptions flag is to change this line in compiler/BUILD.gn:
config("exceptions_default") {   configs = [ ":no_exceptions" ] }
to:
config("exceptions_default") {   configs = [ ":exceptions" ] }
Platform: Linux ubuntu 20.04
If there is a way to do this adequately, please correct me.
Thanks.
Platform
No response
Anything else?
No response

This method does not sit well with me because of overriding.

This is not "overriding".  This is a build config option (declare_args means "this is an argument consumers can set"), which has some default value (enabled on macOS/iOS, disabled elsewhere) but you can set it to whatever you want to get the behavior you want.
@xvortep It seems like the right thing to do here is to add a similar enable_exceptions build config option, right?

This method does not sit well with me because of overriding.

This is not "overriding". This is a build config option (declare_args means "this is an argument consumers can set"), which has some default value (enabled on macOS/iOS, disabled elsewhere) but you can set it to whatever you want to get the behavior you want.
@xvortep It seems like the right thing to do here is to add a similar enable_exceptions build config option, right?

Sounds right. Thank you.


Arrange food, drink, and venue access for OKC# event | 09/04/2017 11:30 am

What: TBD
When: 09/04/2017 11:30 am
Where: 1141 W Sheridan Ave, Oklahoma City
Check meetup for RSVP count: https://www.meetup.com/OKC-Sharp/events/swlthmywmbgb/

Moved to https://github.com/techlahoma/user-groups/issues/69


[Accordion]feat: 2/3

Description
Finished Accordion API and a11y
Related Issue
#131
Motivation and Context
There is a major change. I removed all the AccordionImpl components. I don't think we need them.
The main current problem now is that as far as I know we cannot express the following logic into Vue props:
{
  type: AccordionType.Single;
  modelValue?: string;
} | {
  type: AccordionType.Single | AccordionType.Multiple;
  modelValue?: string[];
}

Basically change the type of modelValue depending on the type. Since we cannot do that, I just declare modelValue, defaultValue and onValueChange as a combination of: string, undefined and string[].
Please verify that the exports are ok.
Let me know if you see any bugs or room to improvments!
Thank you!

Alot of things to unpack here.

When typing props, you need to extend the existing props if the child is another component. E.g. if the child is CollapsilbleTrigger, import the CollapsibleTriggerProps and extend them, that way you dont need to declare asChild default to false because it is already done in the bottom component(the primitives). This is what @zernonia done here but he probably missed some in accordion.
We dont do onValueChange, every value or open we bind with v-model and in components we implement useVModel. You can refer to other components on how these are implemented.
Thanks for the refactors, we had that in planned! Actually there is a very relevant example on how to implement single and multiple props. They are toggle group and select. I think you should refer those and see how they are implemented.
This is just enhancer, I think we should remove the dependancies on Collapsible components and use the Primitives right away, although these are almost the same features but removing dependancies would make it much more maintainable and cleaner.

Alot to take in, overall pretty good and very nearly ready. Let me know if you need some help!

Great feedback. Working on it

I think everything is ready. I will recheck everything tomorrow in case I miss something I don't see now.
I can squash the commits to clean the tree before merging

@khairulhaaziq you mention to remove Collapsible. I feel like I have to copy and paste a lot of the logic.
See Radix implementation:

@khairulhaaziq you mention to remove Collapsible. I feel like I have to copy and paste a lot of the logic.
See Radix implementation:

I dont recommend remove Collapsible. What's the issue that leads to the idea of removing ya?

not working for me:
https://github.com/radix-vue/radix-vue/assets/101852870/fddfcef3-d7d9-4436-acdf-1d903729e2df

this has been a slow progress do to my absence, sorry. @zernonia you can take it over

@zernonia I pushed some changes.
This comment is the missing thing now

@onmax since our reviews halt the development abit, do consider working on another component meanwhile. @zernonia i think can merge?

Yeah, I forgot to create a new branch for the accordion. So now I was waiting for the merge, to start working in a new component.
Nonetheless, today I will be travelling, so I can wait 😄

@khairulhaaziq  @onmax  this is working well! And loves the validation messages. Clear and easily actionable!


Upgrade ~0.9 -> 1 breaks app settings unless loaded via external file in F#

Values being read from App settings in F# are coming through as null.
Repro steps

Add an app setting to a functions app
In an F# function's run.fsx attempt to read the app setting using something like

let mySetting = System.Environment.GetEnvironmentVariable("MY_APP_SETTING")

Run the function app and observe (via log.Info for example) that mySetting is null.

Expected behavior
mySetting has the value that was configured in the function app settings.
Actual behavior
mySetting is null.
Known workarounds
It seems that putting this loading code in an external, shared fsx file (e.g. ../shared/settings.fsx) then using #load "../shared/settings.fsx", then open Settings causes the app settings values to come through. I'm not sure if it's a timing issue or something else.

We cannot repro. I just ran this:
#r "System.Net.Http"
#r "Newtonsoft.Json"

open System.Net
open System.Net.Http
open Newtonsoft.Json

type Named = {
    name: string
}

let Run(req: HttpRequestMessage, log: TraceWriter) =
    async {
        log.Info(sprintf
            "F# HTTP trigger function processed a request.")

        let mySetting = System.Environment.GetEnvironmentVariable("FUNCTIONS_EXTENSION_VERSION");
        log.Info(mySetting);

        // Set name to query string
        let name =
            req.GetQueryNameValuePairs()
            |> Seq.tryFind (fun q -> q.Key = "name")

        match name with
        | Some x ->
            return req.CreateResponse(HttpStatusCode.OK, "Hello " + x.Value);
        | None ->
            let! data = req.Content.ReadAsStringAsync() |> Async.AwaitTask

            if not (String.IsNullOrEmpty(data)) then
                let named = JsonConvert.DeserializeObject<Named>(data)
                return req.CreateResponse(HttpStatusCode.OK, "Hello " + named.name);
            else
                return req.CreateResponse(HttpStatusCode.BadRequest, "Specify a Name value");
    } |> Async.RunSynchronously

Got the value logged properly:
2017-01-09T18:37:46.481 Function started (Id=3e5827b6-f9df-4bdf-a611-97351f100217)
2017-01-09T18:37:46.481 F# HTTP trigger function processed a request.
2017-01-09T18:37:46.481 ~1
2017-01-09T18:37:47.269 Function completed (Success, Id=3e5827b6-f9df-4bdf-a611-97351f100217)

Closing no repro for now

Thanks for trying that out!
One difference is that I was doing something along the lines of
open System.Net
open System.Net.Http
open Newtonsoft.Json

let mySetting = System.Environment.GetEnvironmentVariable("FUNCTIONS_EXTENSION_VERSION");

type Named = {
    name: string
}

let Run(req: HttpRequestMessage, log: TraceWriter) =
  log.Info (sprintf "%A" mySetting)

Specifically, I was setting mySetting inside of the #else of a !COMPILED precompiler directive, in order to differentiate the deployed Azure Functions environment and my local environment.
Do you think that could be making the difference? Is putting such lets not allowed outside of Run, or inside of !COMPILED?


Don't smart-match when using gopass search

I'm often using something like for secret in (gopass search evernote); echo $secret; gopass $secret; echo ''; end to check out several secrets at once, but if there is only one match it breaks the whole command, as it immediately shows the content like this:
gopass search evernote
Found exact match in 'evernote.com/username'
<content>
<more content>

I would rather want it to only display the name. gopass evernote already provides the smart search functionality ...

Yes, this sounds reasonable. I'll see if I can come up with any arguments why we wouldn't do this, if I can't find any I'll adjust the current behaviour.

Hi @dominikschulz, this behavior seems to be reintroduced! Is this intended, or is it a regression bug?

It's definitely not intended. I'll try to reproduce it.

@ad-si Sorry, but I'm unable to reproduce with a current master build.

@dominikschulz This is stil an issue, and it's quite annoying 😅.
Both gopass search and gopass find directly show the match if there is only one.
Can we please change it to only list the name? 😭

This is my workaround script:
#! /usr/bin/env bash

for secret in $(gopass list --flat | rg "$@")
do
  echo -e "\n===== $secret ====="
  gopass "$secret"
  echo -e "\n"
done

I'd really like to be able to use gopass programmatically without magic search. Could this be reopened please?

Sure thing, but I still can't reproduce it :/

If I've understood the OP's request, and @raphink's comment: This gets my vote for a change. I want to use gopass programmatically; so an option such as gopass show --password --strict path/name would be very useful, rather than the fuzzy search. A return code != 0 could then indicate that an exact match was not found.


Add a Gitter chat badge to README.md

cujojs/most now has a Chat Room on Gitter
@briancavalier has just created a chat room. You can visit it here: https://gitter.im/cujojs/most.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

We should merge this : )

Thanks for the nudge :)  Merging


Adds :ukzip to built in validator.

UK postal codes are weird.

@markrickert  can you add this validator to the docs on RMQ's website?

Done - http://rubymotionquery.com/document/validation-with-rmq/


NullPointerException when publishing events to EventHub

Describe the bug
When publishing to EventHub using the latest SDK, we get an error LinkedList.node(int) is null [ Full Error attached below ]
Exception or Stack Trace
Exception : NullPointerException
Stack Trace:

To Reproduce

Create a JAVA project with EventHub SDK. Use the producer snippet below and start publishing events.

Code Snippet
public class EventPublisher {

    private static final EventHubProducerClient producer;
    private static EventDataBatch batch;

    static {
        producer = new EventHubClientBuilder()
                .connectionString("<CONNECTION_STRING>", "<EVENT_HUB_NAME>")
                .buildProducerClient();

        batch = producer.createBatch();
    }

    public static void publish(Event event) {

        EventData eventData = new EventData(event.getEventData().encode());
        eventData.getProperties().put("eventName", event.getEventName());
        eventData.getProperties().put("isValid", Boolean.toString(event.isValid()));

        if (event.isValid() && !batch.tryAdd(eventData)) {
            //batch is full, send the batch and create a new batch
            System.out.println("Pushing to hub");
            producer.send(batch);

            batch = producer.createBatch();

            // since we couldn't add the event earlier, we add it in the batch now
            batch.tryAdd(eventData);
        }

    }

    public static void flushEvents() {
        System.out.println("Flushing Events");
        if (batch.getCount() > 0 ) {
            producer.send(batch);
        }

        producer.close();
    }
}

Expected behavior
Events are published to the event hub without errors
Screenshots
If applicable, add screenshots to help explain your problem.
Setup (please complete the following information):

OS: MacOS
IDE: ntelliJ
Library/Libraries:  com.azure:azure-messaging-eventhubs:5.10.4 [ Tried 5.10.3 too ]
Java version: JDK 16
App Server/Environment: VertX inbuilt server
Frameworks: VertX

Additional context
There have been rare cases where events are published, but I can't find a pattern. In most cases, an error is thrown but there have been instances where events were published. There was absolutely no change in the code.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x ] Bug Description Added
[ x] Repro Steps Added
[ x] Setup information Added

Thanks for reporting this @sriram-adpusuhup.
@conniey @anuchandy could you please take a look at this NullPointerExcception leaking from Event Hubs?

The line of code throwing a NullPointerException is this: https://github.com/Azure/azure-sdk-for-java/blob/main/sdk/eventhubs/azure-messaging-eventhubs/src/main/java/com/azure/messaging/eventhubs/EventHubProducerAsyncClient.java#L540. However, access to its indexes are via a for-loop based on its count.
I noticed that your code uses static for reads and writes to the EventDataBatch. Is your code running in a single-threaded environment? If not, I can see a race condition where multiple threads are in the same static void publish(EventData) method between sending and re-assigning the new EventDataBatch.


debugcc: package Bjorn's debugcc tool

The debugcc tool can be useful on the target platforms. Add a recipe to
build corresponding package.
Signed-off-by: Dmitry Baryshkov<EMAIL_ADDRESS>

Marking this as a draft until https://github.com/andersson/debugcc/pull/4 is merged


'ResolveInfo' object has no attribute 'path' in middleware

Whenever the server receives a request with the HTTP header 'Authorization':'JWT <token>' the response contains the error

'ResolveInfo' object has no attribute 'path'

Playing around with the debugger I noticed that the exception is raised from the 'resolve' method in the 'JSONWebTokenMiddleware' class of the middleware.py file.
The exception is caused by this piece of code:
field = getattr(                 info.schema,                 'get_{}_type'.format(info.operation.operation),             )().fields.get(info.path[0])
The exception says that 'info' does not have the attribute 'path'.
Playing with the debugger I noticed that the 'path' attribute exists inside the 'context' object. So I tried to change the code from info.path[0] to info.context.path[0] and everything started working correctly.
Since I'm fairly new to both Django and GraphQL I ask you: am I missing something here? Did I do something wrong in my code (i.e. url patterns, GraphQL types definitions, etc.) or in my settings/configurations that caused the info object to not have the expected structure? Is there something I can do to fix the problem? Obviously manually changing the library code is not the answer.
Thanks.

Hi @darktal91 ,
What version of graphql-core do you have installed?
pip freeze|grep graphql-core

Hey @mongkok,
Not trying to hijack, but just ran into the same issue after starting to use graphql_jwt.middleware.JSONWebTokenMiddleware
graphql-core version is 2.0
Other configs (in case helpful):
MIDDLEWARE = [
'django.middleware.security.SecurityMiddleware',
'django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.common.CommonMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware',
]
AUTHENTICATION_BACKENDS = [
'graphql_jwt.backends.JSONWebTokenBackend',
'django.contrib.auth.backends.ModelBackend',
]
GRAPHQL_JWT = {
'JWT_VERIFY_EXPIRATION': True,
'JWT_EXPIRATION_DELTA': timedelta(minutes=5),
'JWT_REFRESH_EXPIRATION_DELTA': timedelta(days=30),
}
Happy to answer any questions!

Update:
Issue was resolved by upgrading to graphql-core 2.1

Thanks @Benjaminryejones !,
I've added graphql-core to the install_requires.


[[Power-Fx]] Localization hand-back [CTAS - RunID=20240130-181356-qt82as5705]

Translation handback check-in (PR Created by automation).

✅ No public API change.


Can't Install model

I tried this on the 22nd and was able to install models but not get it to serve (it complains model not found).
With the latest version it doesn't appear to be installing models anymore.
catai install Vicuna-13B
$ cd /usr/lib/node_modules/catai

When I run install I just see a cd command echo'd out to the terminal and nothing else. Same thing if I try to run it from that directory.

I tried this in Windows as well and got the same result.

Thank you for reporting, I see the same issue - I am on that :)

The problem is the URL of the model change, for the meantime, you can install the model directory from a link:
catai install https://huggingface.co/eachadea/ggml-vicuna-13b-1.1/resolve/main/ggml-vic13b-q4_0.bin

I tried that install line, and got this one when serving

I know there is a bug with installing by version.
I will look into that

Thank you. I really appreciate the work you're putting in here. This sort of library helps simplify the crazy world of open source LLMs.

The problem is a new GGML format that is not yet supported by llama.cpp, for now, you can try the old format:
https://huggingface.co/eachadea/ggml-vicuna-13b-1.1/resolve/819b698c54b906ca7f3e1774cf748c2018932b07/ggml-vic13b-uncensored-q4_0.bin

The model links are now updated, so this issue should be fixed until llama.cpp can handle ggml v3.

Works for me! Thank you sir.


fix: handling of multipleInstrumentationNodeModuleDefinition with the same name

When there are multiple InstrumentationNodeModuleDefinitions the  urrent implementation chooses one of theInstrumentationNodeModuleDefinitions in bunle-time and then in runtime selects it by name. This breaks when there are multiple InstrumentationNodeModuleDefinitions with the same name, as the wrong one may be selected.
This is the case with:
mongodb - https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/plugins/node/opentelemetry-instrumentation-mongodb/src/instrumentation.ts#L96
winston - https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/plugins/node/opentelemetry-instrumentation-winston/src/instrumentation.ts#L48
socket.io - https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/plugins/node/instrumentation-socket.io/src/socket.io.ts#L218
generic-pool - https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/plugins/node/opentelemetry-instrumentation-generic-pool/src/instrumentation.ts#L43
This PR approaches the problem analogically to how instrumentation.files are handled. By iterating over the instrumentation in runtime and applying the ones with matching version.

Don't we still need to check instrumentation.name for cases like the aws sdk which has InstrumentationNodeModuleDefinition's with many names?

https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/plugins/node/opentelemetry-instrumentation-aws-sdk/src/aws-sdk.ts#L101
https://github.com/open-telemetry/opentelemetry-js-contrib/blob/main/plugins/node/opentelemetry-instrumentation-aws-sdk/src/aws-sdk.ts#L129

Indeed we do. I converted this PR to a draft exactly because I realized aws-sdk stopped working after these changes.

@RafalSumislawski i'll merge if that works for you?

Sure 👍

thanks again for the fix!


[WFCORE-6978] Upgrading JBoss Marshalling to 2.2.1.Final

https://issues.redhat.com/browse/WFCORE-6978

Thanks @ropalka


When creating bare repos, clone with --mirror instead of --bare

I keep a directory of bare repositories, which I created by running sync-ebooks -v -b and regularly update with the same command.
Behind the scenes, sync-ebooks runs git fetch -v in each repository (whether bare or not). In a normal repository, this doesn't update the checked‐out master branch, but does update remote/origin/master and remote/origin/HEAD. In a bare repository, though, there are no remote branches, only the one branch master, which remains unchanged. I assume git fetch is bringing in newer commits, but without any references they are effectively inaccessible as far as I can tell. (My not realizing this led to merge conflicts in a recent pull request.)
We could change sync-ebooks to run git fetch origin master:master. In a bare repo, that does what I consider intuitive: master will be updated to point to the fetched commits. But checked‐out repos don’t play well with this command, so we would have to split the command paths for bare and non‐bare repos. (Besides making the code more complicated, a reason I consider this undesirable is that it would eliminate an undocumented but useful characteristic of sync-ebooks, that it will update both bare and checked‐out repositories present in the same directory.)
However, we can get this behavior in bare repos automatically by cloning with with --mirror. Doing so adds this line to the new repo’s Git config:
fetch = +refs/*:refs/*

With this sync-ebooks’ behavior for checked‐out repos is unchanged, while bare (now mirror) repos update their only branch, master, to the latest master commit in the origin, which I think is a more natural and useful behavior.

Good catch, thanks!


Add support for functions in an interval expression

Added support for functions in an interval expression. This far, only columns and JDBC parameters were supported.
A sample query that is now supported:
SELECT DATE_ADD(start_date, INTERVAL COALESCE(duration, 21) MINUTE) AS end_datetime FROM appointment

Coverage remained the same at 88.783% when pulling 6986d38812c4752106d6d23e3f425c6c8e3ecdd1 on tomershay:add_support_for_function_in_interval_expression into 5706fb4e1133884a8ffedb3a22eb3c460fb4e1d1 on JSQLParser:master.

Coverage remained the same at 88.783% when pulling 6986d38812c4752106d6d23e3f425c6c8e3ecdd1 on tomershay:add_support_for_function_in_interval_expression into 5706fb4e1133884a8ffedb3a22eb3c460fb4e1d1 on JSQLParser:master.

Hi @wumpz, did you get a chance to review this please? Can be great to have it merged.

Sorry. Somehow overlooked it. I am on it.


Add scraper that supports Android and IOS.

Right now the only scraper available seems to require passing a referrer to the player, such a thing is not supported on Android and iOS making otaku-watcher not supported for Android.
I am working on a new scraper here but you can leave that one to me.

I didn't test it yet

it works but there's still more work I wanted to do, hence me stating leave that one to me


Setting service_ensure to 'stopped' causes failure

If the firewalld service is running and I call this module with service_ensure => 'stopped', resource chaining causes a reload which fails. Here is the debug output from one such Puppet run.

Debug: Executing: '/bin/systemctl is-active firewalld'
Debug: Executing: '/bin/systemctl is-enabled firewalld'
Debug: Executing: '/bin/systemctl stop firewalld'
Debug: Executing: '/bin/systemctl disable firewalld'
Notice: /Stage[main]/Firewalld/Service[firewalld]/ensure: ensure changed 'running' to 'stopped'
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Info: /Stage[main]/Firewalld/Service[firewalld]: Scheduling refresh of Exec[firewalld::reload]
Debug: /Stage[main]/Firewalld/Service[firewalld]: The container Class[Firewalld] will propagate my refresh event
Debug: Execfirewalld::reload: Executing 'firewall-cmd --reload'
Debug: Executing: 'firewall-cmd --reload'
Notice: /Stage[main]/Firewalld/Exec[firewalld::reload]/returns: FirewallD is not running
Error: /Stage[main]/Firewalld/Exec[firewalld::reload]: Failed to call refresh: 'firewall-cmd --reload' returned 252 instead of one of [0]
Error: /Stage[main]/Firewalld/Exec[firewalld::reload]: 'firewall-cmd --reload' returned 252 instead of one of [0]
Notice: /Stage[main]/Firewalld/Exec[firewalld::complete-reload]: Dependency Exec[firewalld::reload] has failures: false
Warning: /Stage[main]/Firewalld/Exec[firewalld::complete-reload]: Skipping because of failed dependencies
Debug: /Stage[main]/Firewalld/Exec[firewalld::complete-reload]: Resource is being skipped, unscheduling all events
Debug: Class[Firewalld]: Resource is being skipped, unscheduling all events
Info: Class[Firewalld]: Unscheduling all events on Class[Firewalld]

I am having the same issue. if I set the service to stopped, the puppet apply will fail. Any solution for this? I guess if the service is ensured stopped, no firewall-cmd --reload should be run.

#182 seems related. In both cases firewalld::reload fails to verify if service is running.


Release 2.3 create release notes for vision, audio and text

Create Draft Release notes to be published during release day:
ETA: M5: External-Facing Content Finalized (4/19/24) <- Draft need to be ready
M6: Release Day (4/24/24) <- Release day

draft release notes are available:

https://github.com/pytorch/vision/releases
https://github.com/pytorch/audio/releases
https://github.com/pytorch/text/releases


Fix typo

Github -> GitHub

@changeworld : Thanks for your contribution! The author, @sanjeev3, has been notified to review your proposed change.

Thanks @changeworld! #sign-off

Merging typo fix


how to change the language

Option to change text for language: https://wordpress.org/support/topic/couple-of-questions-previousnext-functionality-language-changefeatured-image/
We can build user friendly options in backend.

This was added in the latest version


Refactor discogs_tagger.py

This module has slightly outgrown itself. I believe we should do a refactoring, to make future enhancements easier.
There are a couple of opportunities here:
Make the whole config handling easier -> handle this in its own module and use the parameters from an object, this will also make the setting of default values easier, in that those are not set in the taggerutils.py but in the one configure handling module.
Refactor the track-loop, so that this can be adopted easier then right now. Probably we should define an own object "Track"/"TrackContainer" and define certain properties on this one.
Refactor the source directory determination to allow for batch calls (e.g. my source dir is ~/Music and in there are several sub-directories which contain audio files). Up until now the source dir has to contain (except multi-disc releases) audio files.

closing this old ticket, as work to separate and refactor the configuration data was completed in v3.0.0


Rustup

@bors r+

:pushpin: Commit 94d37d4a347eb86e64a11f0d1d6ae6cec4a9211b has been approved by RalfJung
It is now in the queue for this repository.

:hourglass: Testing commit 94d37d4a347eb86e64a11f0d1d6ae6cec4a9211b with merge 4681b6ea23bee1c6fc5c30afaaf99ab91881333d...

:sunny: Test successful - checks-actions
Approved by: RalfJung
Pushing 4681b6ea23bee1c6fc5c30afaaf99ab91881333d to master...


Test/vrt update

DO NOT MERGE.
Testing updates to VRT action

Closing as test changes have been merged in the visual regression testing action


Immediate executor

Note: get_associated_immediate_executor works with composed ops.

Specific tests are still needed. boost.async supports it, so that might be a useful first test.

@ashtum Can you review this?

The changes in the code appear to be consistent and well-structured.
Updates made to the function comments, which generate documentation, are appropriate and coherent.
There don't seem to be significant changes to the API, with the exception of async_base::get_immediate_executor(). Given this minor adjustment, the tests appear to remain effective and relevant.
Codecov has identified several newly added lines of code that aren’t covered by tests. While I’m not entirely certain, these seem to be false positives. It would be beneficial to review and confirm these findings, either by adding the necessary tests or silencing irrelevant warnings. For example:
Codecov
/ codecov/patch
include/boost/beast/http/impl/write.hpp#L224-L225
Added lines #L224 - L225 were not covered by tests

The PR lacks a description. I recommend adding a brief description to assist both current reviewers and anyone looking at this PR in the future. This description should include the rationale behind the proposed changes (even if it seems obvious), and whether the changes resolve or address any specific issues or problems.
In summary, this PR appears to be mostly well-structured and presented. The most crucial step at this point would be to address the areas identified by Codecov and to enhance the PR description to provide clear context and complete understanding to reviewers and future observers.
Good job 🚀


Create test Google service account

From: https://github.com/kubernetes-sigs/k8s-container-image-promoter/pull/62
@listx
We should give<EMAIL_ADDRESS>"Storage Admin" and "Storage Object Admin" perms for the test-only GCRs:
- gcr.io/k8s-staging-cip-test
- gcr.io/k8s-cip-test-prod

/assign @listx @thockin

Also related: https://github.com/kubernetes/k8s.io/blob/master/infra/gcp/lib.sh
@dims suggests we should update this script: https://github.com/kubernetes/k8s.io/blob/master/infra/gcp/lib.sh

And for the record, we should choose a new test-only SA, and not reuse<EMAIL_ADDRESS>for this.

In progress: Working on updating the lib.sh script (and learning some bash)

Send me the PR when ready :)

/close
From clarification in https://github.com/kubernetes/k8s.io/pull/302
These service accounts exist.


Support typical system text-editing shortcuts

Is your feature request related to a problem? Please describe.
I keep trying to paste into an <input> with Command-V, but the only thing that works is "Right-click, select Paste".
Similar with Command-A to select all; presumably also Command-C to copy but I haven't tried it yet.
I did try Control-A for select-all, but ^A/^E seem to work like the rest of the OS (same as bash/readline).
Describe the solution you'd like
Make common Edit-menu shortcuts work in Tauri on MacOS

Describe alternatives you've considered
I could keep using the right-click workaround forever, but that probably won't fly with arbitrary end-users.
Sorry if this falls under "Create native application menus and context menus" in #544

I believe this has been fixed here https://github.com/tauri-apps/tauri/pull/1560
will be in the next release

Indeed this falls under #544, and @lemarier is working on that.


Pin elasticsearch version

see #2203 and #2204
Alternatively if #2194 is merged in soon this PR can be ignored, as I presume #2194 would fix it.

Can you resolve conflicts please :)

Can you resolve conflicts please :)

Actually I think this was fixed already by https://github.com/Yelp/elastalert/pull/2194


haste-task-logger logs 'start'/'finish' messages only on initial run

Codecov Report

Merging #90 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #90   +/-   ##
=======================================
  Coverage   96.25%   96.25%
=======================================
  Files          18       18
  Lines         187      187
  Branches       16       16
=======================================
  Hits          180      180
  Misses          7        7

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 8001163...73e8cdc. Read the comment docs.


DBZ-498 Unknown datatypes handling

https://issues.jboss.org/browse/DBZ-498

Good, thanks for the improvements!


Define rule DDF00010: If Procedure is Conditional then procedureIsConditionalReason must be present

The description is "If Procedure is Conditional then procedureIsConditionalReason must be present"
In UML speak, I think it would be more accurate to say that for a Procedure, if procedureIsCondition = true, then procedureIsConditionalReason must be populated. Or maybe "must be present" or "may not be null".  How "technically" should the rule description be written?

Closing as not planned as this rule was implemented as DDF00009


What Does This Do Exactly?

At very least it appears to disable the Registry Key in HKLM\SOFTWARE\Microsoft\Windows\Windows Error Reporting by setting Disabled = 1 however there are several other options here, for us we also expect to have DontShowUI Set, but this tooling obviously does not do it perhaps it would be good to link the underlying documentation and describe what is done by this task https://docs.microsoft.com/en-us/windows/desktop/wer/wer-settings

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

ID: cbf6dc60-7aff-ff5b-f426-09dc77a55f1f
Version Independent ID: 69d5a644-2750-6d63-3ac9-cabb3eff1c02
Content: Disable-WindowsErrorReporting (windowserrorreporting)
Content Source: docset/windows/windowserrorreporting/disable-windowserrorreporting.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Hello @aolszowka , please check https://github.com/MicrosoftDocs/windows-powershell-docs/pull/865 , is it what are you talking about? or you mean anything else?
Thank you

@kenwith PR #865 available for the issue.

@MaratMussabekov Not quite; we want to understand which registry keys are being disabled by this specific task.
Based on the Windows Documentation there are plenty of things that could be happening; but because the documentation for this task is vague its unclear. Is there a link to the implementation of this cmdlet?

Dear @aolszowka , this cmdlet set registry key "HKEY_LOCAL_MACHINE\Software\Microsoft\Windows\Windows Error Reporting\Disabled" to the value 1.
However, cmdlet descriptions in Microsoft docs do not usually specify this kind of information. I mean, it is usually "cmdlet disables Windows Error Reporting on the server...", not "this cmdlet changes the value of the registry key..."
What do you think?
Thank you

@MaratMussabekov Sounds good to me; the reason I think this originally came about is we thought that you needed to set DontShowUI in addition to setting the disable flag. I think we can close it out as resolved with the additional documentation change to link the Windows Docs as you did in your PR.


Block spots while cancelling block place event.

        globalEventHandler.addListener(PlayerBlockPlaceEvent.class, event -> {
            event.setCancelled(true);
        });

whenever i place a block sometimes it just leaves a black spot behind? Video explains it better.

Can't reproduce this


Support importing code snippets from files

Using +exec for code blocks is not always suitable since the amount of code in my presentation varies from 10 lines to sometimes 50 lines. And hiding the code (e.g. using e.g. # . for Rust) is too much manual work.
But I still want to verify that the code in my presentation builds fine so I created a separate folder called code which contains all the code used along with unit tests and so on.
I would like to request a feature that works very similar to remark-code-import so that I can simply import snippets from a file into the presentation. We can use anchors and so on to name/designate the start/end of sections. See ratatui-website repository to see how we use this.
What do you think? 🐻

Not on computer now but can you check if https://github.com/mfontanini/presenterm/pull/328 does what you need? Another unreleased feature :)

Works like a charm! Thanks 💖

One thing that I realized is that the presentation is not updated when I update the file so I need to restart the app. I think it needs a file watcher for auto reloading 🤔

Sorry, been quite busy. This is now fixed in https://github.com/mfontanini/presenterm/pull/372

Can confirm that it works, thanks a lot!


Error response from daemon: ENV must have two arguments

○ docker build -f Dockerfile .
Sending build context to Docker daemon 2.048 kB
Error response from daemon: ENV must have two arguments

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
You do NOT have to include this information if this is a FEATURE REQUEST
-->
Output of docker version:
Client:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 21:15:28 2016
OS/Arch:      darwin/amd64
Server:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 23:54:00 2016
OS/Arch:      linux/amd64

**Output of `docker info`:**

Client:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 21:15:28 2016
OS/Arch:      darwin/amd64
Server:
Version:      1.12.0
API version:  1.24
Go version:   go1.6.3
Git commit:   8eab29e
Built:        Thu Jul 28 23:54:00 2016
OS/Arch:      linux/amd64
jay at jai in ~/Movies/AWS/DOCKER/Docker for Web Developers By Dan Wahlin/ExpressSite
○ docker info
Containers: 1
Running: 1
Paused: 0
Stopped: 0
Images: 1
Server Version: 1.12.0
Storage Driver: aufs
Root Dir: /mnt/sda1/var/lib/docker/aufs
Backing Filesystem: extfs
Dirs: 16
Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
Volume: local
Network: null bridge host overlay
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.16-boot2docker
Operating System: Boot2Docker 1.12.0 (TCL 7.2); HEAD : e030bab - Fri Jul 29 00:29:14 UTC 2016
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.955 GiB
Name: default
ID: HUKM:NHAP:KW6B:4YIA:72UJ:EYF4:6QTD:M6LV:XVLV:27WV:KG7V:R2QR
Docker Root Dir: /mnt/sda1/var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
File Descriptors: 19
Goroutines: 35
System Time: 2016-08-10T18:41:11.087962587Z
EventsListeners: 0
Registry: https://index.docker.io/v1/
Labels:
provider=virtualbox
Insecure Registries:
<IP_ADDRESS>/8

**Additional environment details (AWS, VirtualBox, physical, etc.):**

**Steps to reproduce the issue:**
1.open the terminal
2.write the command =   docker build -f Dockerfile .
3.

**Describe the results you received:**

**Describe the results you expected:**
I was expecting it to run the Docfile that contains this :

FROM node:latest
MAINTAINER Jay P
ENV NODE_ENV=production
ENV PORT=3000
COPY                    ./var/www
WORKDIR         /var/www
VOLUME        [“/var/www”, “/logs"]
RUN                      npm install
EXPOSE              $PORT
ENTRYPOINT    [“node”,“start"]
ENV                NODE_ENV\

**Additional information you deem important (e.g. issue happens only occasionally):**
Everytime

it is complaining about the last line ENV                NODE_ENV\ - what did you mean here? The NODE_ENV will already be set in the runtime environment.

As far as I can see there's no bug here, so I'll close this issue, but feel free to continue the conversation.
I also spotted that your Dockerfile contains several "curly" quotes (“); this may be due to copy/pasting here, but if not, you should replace those with regular quotes ", otherwise your Dockerfile may not work


Cloud IoT Core Python mqtt example waits 60 seconds between messages, inconsistent with the Readme

Environment details

OS type and version: Ubuntu 20.10
Python version: 3.10
pip version: 22.1.2
google-cloud-iot version: 2.5.1

Steps to reproduce

Follow steps in the Readme, create and register a device in IoT Core, set up authentication
Run python cloudiot_mqtt_example.py --algorithm="RS256" --cloud_region="us-central1" --device_id="my-device" --private_key_file rsa_private.pem --project_id="my-project-id" --registry_id="my-registry" device_demo python cloudiot_mqtt_example.py --algorithm="RS256" --cloud_region="us-central1" --device_id="hexagon0" --private_key_file rsa_private.pem --project_id="imposing-medium-358202" --registry_id="hexagon-registry" device_demo (but with the actual project, device, and registry names)
The messages publish correctly, but 60 seconds apart instead of one second apart as described in the readme and a comment.

Code example
It looks like this is caused by the nested for loop starting in line 566 of cloudiot_mqtt_example.py . I tried running it with that loop removed and time.sleep(1) in the main loop, and it works as described (approximately one message per second).
Or, if once every 60 seconds is the expected behavior, then the readme and comment in line 565 are wrong.
Stack trace
N/A

Closing as obsolete as the service is being retired on August 16, 2023.
Google Cloud IoT Core is being retired on August 16, 2023. Contact your Google Cloud account team for more information.

https://cloud.google.com/iot-core
https://cloud.google.com/iot/docs


Update Post “jupyter/index”

Automatically generated by Netlify CMS

👷 Deploy Preview for fascinating-douhua-8d1544 processing.

Name
Link

🔨 Latest commit
27c56095861520866c844f3807f4cb0c0a749cf3

🔍 Latest deploy log
https://app.netlify.com/sites/fascinating-douhua-8d1544/deploys/64c3f042ef32f200088a13f9


Remove defunct website from README

slimcoin.info is not the defunct site we've done some editing on it lately, the defunct one is slimco.in

I think we need to implement this change ASAP,  so I'm merging this pull request.
If there is any other thing we should discuss, please let me know.


quickstart doc for how to use the TF Grid

[x] register 3bot
[x] create wallet
[ ] add money
[ ] script to deploy an ubuntu (alpine would even be better)
[ ] connect the container to web over https
[ ] use wireguard to connect to it
[ ] make remote ssh connection to deploy something

https://github.com/threefoldtech/js-sdk/blob/9b238ef3cff4f2cf79acb038887d9c010cdaade5/docs/wiki/quick_start.md

add money: https://github.com/threefoldtech/js-sdk/blob/development/docs/wiki/tutorials/add_funds_to_wallet.md which is linked in the quick_start doc

verified


Simulated data from LAT plugins.

Feature Request
Is your feature request related to a problem? Please describe.
LAT Plug-ins need simulated data
Describe the solution you'd like
To have the methods added

This is still active


Port to Python 3

I've had most of this done for a few days now. I was waiting for a response to my question about whether or not dropping Python 2 support was acceptable in #5. I really hate merging things though, so I'm pulling now so I don't have to merge commits again ^_^
This is not compatible with Python 2. It was much easier to do it this way, as you can probably imagine. I did not remove any of the puli code, but that can probably be done if this is merged. Also, you should test whatever uses /contrib/metadata/freesound.py. I've gone over that code briefly, and nothing glaringly obvious has changed, but some of the libraries may have slightly different behavior in Python 3.

Merged. Thank you!


Add an option to pipline spec for handling datum file name conflicts

What is the goal / desired outcome?
Provide an option in the pipeline spec that explicitly describes how to handle when multiple datums write to the same file path in their /pfs/out. Currently the default seems to be to concatenate the contents of both files together in random order.
If there is a way to accomplish this today via workaround, what does that require?
The only workaround is to completely avoid datums writing to the same path.
(Optional) What is your proposal for a feature to solve this?
Add a fileConfict key to the pachyderm pipeline spec that allows specifying one of the following options:
fail - Fail the datum(s) if conflicting files are found. This should IMO be the default behavior.
overwrite - Overwrite the file, so that the last datum to write to the path gets priority.
overwriteFirst - Overwrite the file, so that the first datum to write to the path gets priority.
directory - Create a directory with the same name as the conflicting file. Then put the output of each datum to that path as a separate file inside the created directory, with the datum hash as the file name.
concatenate - The current behavior. Concatenate the conflicting files together in random order.

I think the current behavior is very surprising. I accidentally had a script that was saving the same path in multiple datums, and ended up concatenating a binary file together with itself, corrupting the file! I think at the very least the default behavior should be changed to simply fail the datum(s).

The current behavior is a core part of how Pachyderm pipelines work that's designed to be analogous to how MapReduce works, wherein identical keys from mappers get passed to the same reducer. I can definitely see how these semantics would come as a surprise though. However I think gating this behavior behind a flag is going to make it more confusing without adding much value, and we certainly can't change the default, for one thing it would be a huge compatibility issue for our existing users, but it also doesn't make sense for a feature that's required to implement a lot of workloads to error unless you pass the right flag.
In general we try to avoid adding things to the pipeline spec that  implement things that can be implemented in user space, because it litters pipeline specs with a lot of arcane flags and we feel makes understanding pipelines more confusing. We're particularly resistant to doing it in cases where the behavior can be implemented in user space, which all of these behaviors can with the exceptions of fail.  I can see how having that flag could help in debugging, but since it can't be the default it's unlikely to save anyone from initial confusion and once you understand the concatenating behavior (which you'll have to if you're passing fail) I don't think it adds much.

@jdoliner at one point i believe we had talked about adding a "pass-through/map-only" mode, that could simplify processing by skipping the merge step. not sure how much that still makes sense in the context of the new storage layer stuff, but broadly the idea of saying "datums can be fully processed independently, and no shuffling behavior is expected" does seem like a pretty common use-case, and useful both semantically as well as as a performance optimization.


CI: add tidyverse

I tried adding tidyverse as a full package, but got alarmed when GH took 11m36s to install the dependencies (lots of C++17 compilation).
Thinking about it some more, we definitely don't need ggplot2 (a big plotting package), and probably don't need tidyr or forcats (mainly about data cleaning). Adding "dplyr", "stringr", "readr", "purrr", "tibble" brought installation time down to 6m20s. I assume GH will cache the results somewhere?

For anyone wondering, this is prompted by PR #365 and problems Erik hit in that case.

I assume GH will cache the results somewhere?

Not automatically, no. But let's see how bad it is. Maybe the R action does add some caching. If not, we can add it ourselves.


Cherrypick last fixes from gh-pages to master

This is the cherrypick required in #11 and #12.
Fix 404 on null (3bab92e - a317ea8)
The qualifyURL script was setting the img.src to null, causing a 404 error.
A better approach is to use the window.location.href to get the base url and concat it with the relative one.
Show a full-width graph in visualize (4d3652f - 9333b75)
A large application topology causes the nodes that represent some of the components to end up outside the image canvas. This edit shows a full-width graph in the Visualize tab. This will put the table after the graph instead of aside, allowing to display a larger and more relaxed topology to improve the readability.
Note that this feature doesn't fix the bug: a very large topology will still causes the nodes to end up outside the canvas, even at full with. A possible fix could be to take the size passed to the createGraph function as max width and drow nodes inside that space. Then, by setting the svg width equal to the width used by the elements that it contains, you can obtain a smaller canvas with the table aside when possible, and a full-width canvas otherwise.

Thanks @daquinoaldo


Group study README.md commit

[ ] Markdown code is correct well-formatted
[ ] Spelling and grammar are ok
[ ] The Norms make sense to you

This is a test PR with our README from Norms, and I see it is doing tests! I did't find where to disable them when creating a Pull Request

Anyone with access can merge the pull request

Do the merge


ssearch36 multithreaded output not consistent

Hi,
would like to report that when i use more than 2 threads and option '-m B' or '-m BB' with SSEARCH36 (36.3.8), i am getting different number of hits every time. Sometimes i get for example 524 hits, sometimes 523.
I am using the following command line:
'ssearch36 -T 8 -s BL62 -E 10 -m B -O OUTPUT_FILE IN.fa LIBRARY.fa'
After some testing i think the problem is coming from option '-m'.
Is it possible to avoid this behavior? Or maybe already resolved on newer versions?
Best regards
Dimitar

Hi again,
i think i figured it out. It was my library.fa which was producing the strange variation of outputs. After clustering the library and then perform the search with this library, output is consistent.
Best regards
Dimitar

This I’d expected behavior from the FASTA programs, including SSEARCH.  Basically, most of the statistical estimation options sample a subset (60,000) of all the similarity scores calculated during the search, and estimate the parameters used to calculate the statistical significance and bit score. This has nothing to do with the n7mber of threads or output format.
This can be confusing when the results from one run to 5he nex5 vary slightly, but it is the expected behavior and reinforces the fact 5hat the statistical estimates are just that, estimates, and may produce slightly different values.
Bill Pearson
On Sep 4, 2021, at 11:07 PM, kenietz @.***> wrote:

Hi,
would like to report that when i use more than 2 threads and option '-m B' or '-m BB' with SSEARCH36 (36.3.8), i am getting different number of hits every time. Sometimes i get for example 524 hits, sometimes 523.
I am using the following command line:
'ssearch36 -T 8 -s BL62 -E 10 -m B -O OUTPUT_FILE IN.fa LIBRARY.fa'
After some testing i think the problem is coming from option '-m'.
Is it possible to avoid this behavior? Or maybe already resolved on newer versions?
Best regards
Dimitar
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/wrpearson/fasta36/issues/36, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABQYNPYEM3H3LE2GR7LTR5DUAL3KPANCNFSM5DOE4SYQ.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Thank you for your reply.
yes, i kinda figured out it might be a sampling effect as well. So after clustering my LIBRARY.fa was below 60k sequences and the search result was stable between runs.

This I’d expected behavior from the FASTA programs, including SSEARCH. Basically, most of the statistical estimation options sample a subset (60,000) of all the similarity scores calculated during the search, and estimate the parameters used to calculate the statistical significance and bit score. This has nothing to do with the n7mber of threads or output format. This can be confusing when the results from one run to 5he nex5 vary slightly, but it is the expected behavior and reinforces the fact 5hat the statistical estimates are just that, estimates, and may produce slightly different values. Bill Pearson On Sep 4, 2021, at 11:07 PM, kenietz @.***> wrote:  Hi, would like to report that when i use more than 2 threads and option '-m B' or '-m BB' with SSEARCH36 (36.3.8), i am getting different number of hits every time. Sometimes i get for example 524 hits, sometimes 523. I am using the following command line: 'ssearch36 -T 8 -s BL62 -E 10 -m B -O OUTPUT_FILE IN.fa LIBRARY.fa' After some testing i think the problem is coming from option '-m'. Is it possible to avoid this behavior? Or maybe already resolved on newer versions? Best regards Dimitar — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#36>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABQYNPYEM3H3LE2GR7LTR5DUAL3KPANCNFSM5DOE4SYQ. Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[<IP_ADDRESS>] Updated Sequence Diagram

Issue Details
Description
This Task is to finalize our 2 Sequence Diagrams based on our Updated Use Case Descriptions.
Goals
To have 2 finalized Sequence Diagrams, 1 for Use Case 7 and 1 for Use Case 9.
Success or Completion Criteria

[x] [<IP_ADDRESS>.1] UC-7 SD Final
[x] [<IP_ADDRESS>.2] UC-9 SD Final

Allocated Time
5 Days
Start Date: 17/10/23, 0900
End Date: 23/10/23, 2359

Issue Comments
Description
All Sub-tasks have been completed.
Changes to success or completion criteria? y/n (if y, describe)
No Changes, Criteria Completed!
Start Date: 17/10/23, 0900
End Date: 23/10/23, 2359


Add be_hidden variant for options

I'd like my application to be able to have an option set, without it being visible to regular users. A be_hidden variant as already exists for commands would be helpful here.

Fixed by #44.


Include prometheus rules to the chart for alerting

Hope these are helpful: https://github.com/dragonflydb/dragonfly-operator/issues/242#issuecomment-2329811171

Those rules look good, and we are happy to review and get them into repo if someone wants to take it up.


Add documentation for the -Zdoctest-xcompile feature

@alexcrichton This adds the documentation for the doctest-xcompile feature that (woops) didn't make it into the first pr

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 4b5f104f4f4df0925eb6b494c86da2c26d666398 has been approved by alexcrichton

:hourglass: Testing commit 4b5f104f4f4df0925eb6b494c86da2c26d666398 with merge 9f7a68f5f4f55d735f611fc11883faf25a885acc...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing 9f7a68f5f4f55d735f611fc11883faf25a885acc to master...


How Can This Be Used with a Windows Service

Can this Nuget Package be used with a Windows Service?
If so, what is the proper use?
The windows service does a polling pass once every hour.  Each polling pass I want to create a nested container and then dispose of everything that was created within that polling pass.
Thank you!

The usage would be the same as shown in the readme. You'd just have to create the IServiceCollection yourself. You can do that by installing Microsoft.Extensions.DependencyInjection and new'in up a ServiceCollection.

Great! Thank you! That is what I am doing I just wanted to make sure that was the correct way to do it. Thanks!


[SPARK-32508][SQL] Disallow empty part col values in partition spec before static partition writing

What changes were proposed in this pull request?
Write to static partition, check in advance that the partition field is empty.
Why are the changes needed?
When writing to the current static partition, the partition field is empty, and an error will be reported when all tasks are completed.
Does this PR introduce any user-facing change?
No
How was this patch tested?
add ut

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@gatorsmile
In this pr(SPARK-19129 https://github.com/apache/spark/pull/16583), SessionCatalog blocked the empty value of the partition, but it did not block the InsertIntoHiveTable, which caused the task to be executed first and failed in loadPartition.
Maybe we can check this behavior in advance to avoid task execution?

@cloud-fan If you have time, help review this pull request. Thank you.

ok to test

Test build #128405 has started for PR 29316 at commit 221564b.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33029/
Test PASSed.

Test build #128405 has finished for PR 29316 at commit 221564b.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128405/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33039/
Test PASSed.

Test build #128415 has started for PR 29316 at commit e012e7a.

Test build #128416 has started for PR 29316 at commit ab4e04c.

Test build #128416 has finished for PR 29316 at commit ab4e04c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128416/
Test PASSed.

Test build #128415 has finished for PR 29316 at commit e012e7a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128415/
Test PASSed.

Test build #128442 has started for PR 29316 at commit 232a835.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33066/
Test PASSed.

Test build #128442 has finished for PR 29316 at commit 232a835.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33155/
Test PASSed.

Test build #128531 has started for PR 29316 at commit a7b9a17.

Test build #128549 has started for PR 29316 at commit 65f781a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33174/
Test PASSed.

Test build #128549 has finished for PR 29316 at commit 65f781a.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128549/
Test FAILed.

Test build #128560 has started for PR 29316 at commit 965ed5a.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33185/
Test PASSed.

Test build #128560 has finished for PR 29316 at commit 965ed5a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128560/
Test PASSed.

Test build #128720 has started for PR 29316 at commit b969c73.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33344/
Test PASSed.

Test build #128720 has finished for PR 29316 at commit b969c73.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128720/
Test FAILed.

Test build #128735 has started for PR 29316 at commit c8ac369.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33359/
Test PASSed.

Test build #128735 has finished for PR 29316 at commit c8ac369.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128735/
Test FAILed.

Test build #128755 has started for PR 29316 at commit 76acc07.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/33378/
Test PASSed.

Test build #128755 has finished for PR 29316 at commit 76acc07.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/128755/
Test PASSed.

thanks, merging to master!


Update dpi-and-device-independent-pixels.md

Both of  the value GetDeviceCaps returns and USER_DEFAULT_SCREEN_DPI are int, integer division result is truncated.
If at 144 DPI, the division result will be truncated to 1.0f, but expect 1.5f.

Any progress? @QuinnRadich @GrantMeStrength @Karl-Bridge-Microsoft


mac cmake error

Issue report
issue, bug, question
Environment (Hardware)

Hardware: mac
Software: macOS 11.5.2, cmake version 3.18.4
(Please include version information)

Project Name
pj_tflite_face_blazeface
Error Log
CMake Error at /Users/gqs/Downloads/play_with_tflite/InferenceHelper/third_party/cmakes/tflite.cmake:35 (file):
  file COPY cannot find
  "/Users/gqs/Downloads/play_with_tflite/InferenceHelper/third_party/cmakes/../tflite_prebuilt/ubuntu/libtensorflowlite.so":
  No such file or directory.
Call Stack (most recent call first):
  /Users/gqs/Downloads/play_with_tflite/InferenceHelper/inference_helper/CMakeLists.txt:101 (include)

download error


Update sveltekit-deployment.md

Minor fix for link to example repository. It was pointing to the wrong location.

Thank you! 😄

No, thank you for your hard work!!


read_block should include terminator

We chop off the end-of-block delimiter when doing read_block, but should not.

Why or why not?  I have a vague recollection that this was causing trouble for someone at some point?

It fits with things like readlines to keep it. We changed it in s3fs after discussion, but not here. Seeing this now, because I'm adapting these tests for client.

Alright.  Makes sense to me, especially if other tools agree and usage dictates.


Save layouts to local file and restore them

Hi,
is it possible to save layout to local file ( .txt? maybe different) and then 'load' it from this file and restore it?

Hi @martaszmit! Yes, this is exactly what JupyterLab's layout restoration logic does. The life cycle is described here: https://github.com/jupyterlab/jupyterlab/blob/master/packages/application/src/layoutrestorer.ts#L67-L115
The save() method in the connector returns a promise, so it could be writing to disk or speaking to an external API.


Modal no longer supports default props via theme

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯
Default props provided via the theme.props.MuiModal are no longer respected in v4. It appears that the removal of withStyles() from v3 caused the default props to no longer be respected from what I can tell. Perhaps withTheme() should have the same behavior of injecting default props? Not sure if that makes sense or not.
Expected Behavior 🤔
Allow overriding default props of Modal via the theme.props object.
Context 🔦
In v3, we were able to override the default props of Modal using the theme.props object. This provided a very convenient way to override the behavior of Modal across its usage everywhere in the app, including for components that use Modal internally, such as Menu, or Popover of which we can't control the Modal props.
Specifically in our case we were providing a default manager prop so that we had knowledge in our application when modals were open to prevent certain behavior such as keyboard shortcuts for accessible users.
Users may also want to change other props such as to disable the scroll lock by default for example by setting disableScrollLock: true in their theme in one place.
Your Environment 🌎

Tech
Version

Material-UI
v4.x

@ianschmitz This is related to #16442. We could easily restore the functionality. e.g.
https://github.com/mui-org/material-ui/blob/93e9ab602fb57eb340738065725e0b799a82142b/packages/material-ui/src/useMediaQuery/useMediaQuery.js#L10-L14

Sounds right. It wasn't clear in the migration docs that the removal of the classes customization API also removes the props API as well. Found out after digging through the MUI code that it's actually withStyles() that provides that functionality. 😄

I just spotted #16605 which appears to be the same issue. Didn't see it previously, whoops!


website format issue

some format error

/assign

@windsonsea  there is lots of similar issue on other pages, so reopen this

updating the mkdoc.yml configuration file should fix the code formatting issues

markdown_extensions:
  - meta
  - toc
  - tables
  - fenced_code
  - tables
  - attr_list
  - pymdownx.superfences:
      custom_fences:
      - name: mermaid
        class: mermaid
        format: !!python/name:pymdownx.superfences.fence_code_format
  - pymdownx.arithmatex:
      generic: true
  - pymdownx.betterem:
      smart_enable: all

Document layout has changed