Downward API should expose pod IP

The pod IP is an incredibly useful value for containers to have access to via an env var - it removes very ugly hacks from simple code that must provide an IP to others (the "simple" bash solution is ifconfig + awk + grep which really inspires confidence).  We should make it available via the downward API.

errm, hostname -i?  ioctl(SIOCGIFADDR)?  ip addr list?
I don't have a problem exposing more stuff, but this is a problem that has
existed time immemorial and is not at all related to kube or containers :)
On Tue, Jul 14, 2015 at 9:57 PM, Clayton Coleman<EMAIL_ADDRESS>wrote:

The pod IP is an incredibly useful value for containers to have access to
via an env var - it removes very ugly hacks from simple code that must
provide an IP to others (the "simple" bash solution is ifconfig + awk +
grep which really inspires confidence). We should make it available via the
downward API.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288.

Not every container has 'ip'.  hostname -i returns <IP_ADDRESS> on at least
half my systems.  podIP is always correct (the IP other pods should use
to contact this pod).

I don't have a problem exposing more stuff, but this is a problem that has
existed time immemorial and is not at all related to kube or containers :)

The fact that the problem has existed from time immemorial and that there
is no way to easily figure it out is a crime. :)
On Wed, Jul 15, 2015 at 1:01 AM, Tim Hockin<EMAIL_ADDRESS>wrote:

errm, hostname -i? ioctl(SIOCGIFADDR)? ip addr list?
I don't have a problem exposing more stuff, but this is a problem that has
existed time immemorial and is not at all related to kube or containers :)
On Tue, Jul 14, 2015 at 9:57 PM, Clayton Coleman<EMAIL_ADDRESS>
wrote:

The pod IP is an incredibly useful value for containers to have access to
via an env var - it removes very ugly hacks from simple code that must
provide an IP to others (the "simple" bash solution is ifconfig + awk +
grep which really inspires confidence). We should make it available via
the
downward API.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288#issuecomment-121488274
.

--
Clayton Coleman | Lead Engineer, OpenShift

Is it possible to expose more like spec.nodeName as well?
On Wed, Jul 15, 2015 at 11:42 PM, Clayton Coleman<EMAIL_ADDRESS>wrote:

Not every container has 'ip'. hostname -i returns <IP_ADDRESS> on at least
half my systems. podIP is always correct (the IP other pods should use
to contact this pod).

I don't have a problem exposing more stuff, but this is a problem that
has
existed time immemorial and is not at all related to kube or containers :)

The fact that the problem has existed from time immemorial and that there
is no way to easily figure it out is a crime. :)
On Wed, Jul 15, 2015 at 1:01 AM, Tim Hockin<EMAIL_ADDRESS>wrote:

errm, hostname -i? ioctl(SIOCGIFADDR)? ip addr list?
I don't have a problem exposing more stuff, but this is a problem that
has
existed time immemorial and is not at all related to kube or containers
:)
On Tue, Jul 14, 2015 at 9:57 PM, Clayton Coleman <
<EMAIL_ADDRESS>
wrote:

The pod IP is an incredibly useful value for containers to have access
to
via an env var - it removes very ugly hacks from simple code that must
provide an IP to others (the "simple" bash solution is ifconfig + awk +
grep which really inspires confidence). We should make it available via
the
downward API.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288.

—
Reply to this email directly or view it on GitHub
<
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288#issuecomment-121488274
.

--
Clayton Coleman | Lead Engineer, OpenShift
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288#issuecomment-121655753
.

NodeName crosses a different philospohical line and would require more
debate :)
On Wed, Jul 15, 2015 at 10:00 AM, Lv Lv<EMAIL_ADDRESS>wrote:

Is it possible to expose more like spec.nodeName as well?
On Wed, Jul 15, 2015 at 11:42 PM, Clayton Coleman <
<EMAIL_ADDRESS>wrote:

Not every container has 'ip'. hostname -i returns <IP_ADDRESS> on at least
half my systems. podIP is always correct (the IP other pods should use
to contact this pod).

I don't have a problem exposing more stuff, but this is a problem that
has
existed time immemorial and is not at all related to kube or containers
:)

The fact that the problem has existed from time immemorial and that there
is no way to easily figure it out is a crime. :)
On Wed, Jul 15, 2015 at 1:01 AM, Tim Hockin<EMAIL_ADDRESS>wrote:

errm, hostname -i? ioctl(SIOCGIFADDR)? ip addr list?
I don't have a problem exposing more stuff, but this is a problem that
has
existed time immemorial and is not at all related to kube or containers
:)
On Tue, Jul 14, 2015 at 9:57 PM, Clayton Coleman <
<EMAIL_ADDRESS>
wrote:

The pod IP is an incredibly useful value for containers to have
access
to
via an env var - it removes very ugly hacks from simple code that
must
provide an IP to others (the "simple" bash solution is ifconfig +
awk +
grep which really inspires confidence). We should make it available
via
the
downward API.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288.

—
Reply to this email directly or view it on GitHub
<

https://github.com/GoogleCloudPlatform/kubernetes/issues/11288#issuecomment-121488274

.

--
Clayton Coleman | Lead Engineer, OpenShift
—
Reply to this email directly or view it on GitHub
<
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288#issuecomment-121655753
.

—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/11288#issuecomment-121679831
.


Todo web app made with react

A step-by-step summary

create react app with vite
create structure of the webpage
add functionalities of the web app
add styling
create the readme file

STATUS: APPROVED! 🍾 🎆 🥇
Hi @matee-safi
I'm Ezra.
You did a great job working on this milestone, creating a todo-app using react tutorial.
Your project is complete! 🚀
There is nothing else to say other than... it's time to merge it 🤖
Congratulations! 🎉

Highlights
✅ Great use of Git flow. 💯
✅ Your project is documented professionally. 📜
✅ Great UI design. 💻
✅ Descriptive PR title, summary and commit messages. 💬
Optional suggestions

[x] Just Keep doing the great work :wink:

Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding! :clap:  :clap: :clap:
Feel free to leave any questions or comments in the PR thread if something is not 100% clear. Please, remember to tag me @ezraermy in your question so I can receive the notification.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[Feature] Add bitmap/hll type for lead/lag

What type of PR is this：

[ ] BugFix
[x] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Problem Summary(Required) ：

Add bitmap/hll type for lead/lag,
now we can use case like following:
select bitmap_to_string(user_id), bitmap_to_string(lead(user_id, 1, bitmap_empty()) over (partition by dt order by page)) as next_id_float from pv_bitmap order by page;
+---------------------------+---------------+
| bitmap_to_string(user_id) | next_id_float |
+---------------------------+---------------+
| 1                         | 2             |
| 2                         | 3             |
| 3                         | 4             |
| 4                         | 4222          |
| 4222                      |               |
+---------------------------+---------------+

mysql> select bitmap_to_string(user_id), bitmap_to_string(lead(user_id, 1, null) over (partition by dt order by page)) as next_id_float from pv_bitmap order by page;
+---------------------------+---------------+
| bitmap_to_string(user_id) | next_id_float |
+---------------------------+---------------+
| 1                         | 2             |
| 2                         | 3             |
| 3                         | 4             |
| 4                         | 4222          |
| 4222                      | NULL          |
+---------------------------+---------------+

[FE PR Coverage Check]
:disappointed: fail : 6 / 12 (50.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyticAnalyzer.java
0
6
00.00%
[54, 55, 56, 59, 60, 61]

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
2
2
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/transformer/WindowTransformer.java
4
4
100.00%
[]

[FE PR Coverage Check]
coverage check fail. please retry. :fearful:
Please let me know when error again.
java.io.IOException: Server returned HTTP response code: 500 for URL: https://api.github.com/repos/StarRocks/starrocks/issues/comments/1276957703

[FE PR Coverage Check]
:heart_eyes: pass : 13 / 13 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
3
3
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/transformer/WindowTransformer.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyticAnalyzer.java
6
6
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 13 / 13 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
3
3
100.00%
[]

:large_blue_circle:
com/starrocks/sql/optimizer/transformer/WindowTransformer.java
4
4
100.00%
[]

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyticAnalyzer.java
6
6
100.00%
[]

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 13 / 16 (81.25%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
3
4
75.00%
[546]

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyticAnalyzer.java
6
8
75.00%
[57, 62]

:large_blue_circle:
com/starrocks/analysis/FunctionCallExpr.java
4
4
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 13 / 16 (81.25%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
3
4
75.00%
[546]

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyticAnalyzer.java
6
8
75.00%
[57, 62]

:large_blue_circle:
com/starrocks/analysis/FunctionCallExpr.java
4
4
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 13 / 16 (81.25%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
3
4
75.00%
[546]

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyticAnalyzer.java
6
8
75.00%
[57, 62]

:large_blue_circle:
com/starrocks/analysis/FunctionCallExpr.java
4
4
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 13 / 16 (81.25%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/catalog/FunctionSet.java
3
4
75.00%
[546]

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyticAnalyzer.java
6
8
75.00%
[57, 62]

:large_blue_circle:
com/starrocks/analysis/FunctionCallExpr.java
4
4
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test


Using ${{ secrets.GITHUB_TOKEN }} doesn't work in reusable workflow

In order to use secrets you need to define them similar to inputs and pass them from the calling workflow.
At least that's what I believe when looking at this error and this documentation.

The issue should be solved?


Add requirements to setup.py

Thanks but pypdf and pil are only optional dependencies so they shouldn't go into the setup.py! I'd be happy to include matplotlib.

I disagree, without installed dependencies the bevhaviour of savefig is all over the place:
Without Pillow
plt.savefig("out.png")

works, but warns about Pillow missing and fails to inject metadata.
Without PyPDF2
plt.savefig("out.pdf")

works fine and metadata gets injected in to the PDF.
But then, when trying to read the metadata, without Pillow
$ python -m savefig out.png

results in a totally cryptic error message that does not convey what went wrong:
Traceback (most recent call last):
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/Users/nwerner/Arbeit/mdct/lib/python2.7/site-packages/savefig.py", line 255, in <module>
    info = get_file_info(args.filename)
  File "/Users/nwerner/Arbeit/mdct/lib/python2.7/site-packages/savefig.py", line 147, in get_file_info
    img = Image.open(fn)
AttributeError: 'NoneType' object has no attribute 'open'

and similarily without PyPDF2
$ python -m savefig out.pdf

You get yet another error message that isn't helpful
Traceback (most recent call last):
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/local/Cellar/python/2.7.11/Frameworks/Python.framework/Versions/2.7/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/Users/nwerner/Arbeit/mdct/lib/python2.7/site-packages/savefig.py", line 255, in <module>
    info = get_file_info(args.filename)
  File "/Users/nwerner/Arbeit/mdct/lib/python2.7/site-packages/savefig.py", line 151, in get_file_info
    pdf = PdfFileReader(f)
TypeError: 'NoneType' object is not callable

I can see that you want to have optional dependencies for when you are only saving metadata however I find the resulting behaviour not very reassuring.
Firstly, if you want to only install the minimum set of dependencies for writing, you should at least define all dependencies that are required to fully support writing for all available file types (declare pillow as a dependency).
Secondly, you demand of the user to find out the dependencies manually. This is especially demanding as the reading mode gives no useful help at all, there is no mention of dependencies in the README.md and one needs to go through the code to find the right one. So additionally you should at least clean up the error messages and add all dependencies to a reading extra, so that optional dependencies can be precisely defined and quickly installed.
Thirdly, the library isn't that big and doesn't have that many different usecases that having distinct classes of dependencies is really required. Think IPython, where separating out dependencies for notebooks really make sense as there are many non-trivial dependencies for a usecase not everybody needs. You don't need all of that here, really.
So I would just go for the principle of least surprise: You do
pip install savefig

and get a fully functioning library.

OK fair enough!  I'm always hesitant to add dependencies to setup.py because I prefer to use conda for package management but you've convinced me :-)
Thanks!

Glad I could help :-) I think if you installed all deps using conda, pip should just say "requirement already satisfied" and only install savefig.

Definitely but it's a pain when you install something using pip and then realize that you should have installed all the other deps with conda first. The Python packaging system is so broken!


[BUG] Out of order PUBACKs are ignored

Describe the bug
Out  of order PUBACKs are being ignored.
For QOS 2 publishes that were sent in loop I got packet ids in order 3, 4, 5, 6, 7,
but from MQTT server I got out of order PUBACKs: 3, 6, 5, 7, 4
Resulting onPublish was only called for packet id 3 and 4.

Observed on ESP8266 using Non TLS version. Arduino IDE with latest esp8266 board library.
Expected behaviour
onPublish should be called for every PUBACK regardles of order.

Simple commenting out this break made it work for me:
https://github.com/bertmelis/espMqttClient/blob/835bcfb16853cd19637c2f7001df24320425cfb2/src/MqttClient.cpp#L486
I see that QOS 2 behavior is differs from async-mqtt-client.
This library sends QOS 2 publish right away, while async-mqtt-client waits for PUBACK before sending anything else.

espMqttClient

async-mqtt-client

PUBACKS should be sent in the order in which the PUBs are sent. That is a MUST: http://docs.oasis-open.org/mqtt/mqtt/v3.1.1/os/mqtt-v3.1.1-os.html#_Toc398718105
That is why the callbacks are not called.
I didn't implement retry yet because I've never seen this behaviour. I'll put it on the top of the TODO list.
And indeed, this lib doesn't wait for PUB packets to be acknowledged before sending new ones. It's to speed up things.
Do you also have a wireshark when the PUB packets were sent by the client (or received by the broker)?

Hmmm, I might be mistaken here. The spec talks about MUSTs for clients. The broker has more freedom.

I'm using ThingsBoard as mqtt server.
I've checked traffic on the server side and indeed it's mqtt server which sends out of order PUBACKs.

wireshark on mqtt server side

I created a branch/PR to remove the message ordering check. I reread the specification and concluded the ordering is not mandatory for the broker. So, the client has to accept the out of order ACKs. I assume you agree?

Now, it's more complicated than this. If in the middle of this unordered acking, the connection interrupts, the client still has to resend the original messages in the same order. the PR is therefore not final because this requirement is not met anymore.

I found that this issue only happens with ThingsBoard's "Gateway API" topics (v1/gateway/*).
With default topics (v1/devices/me/*) all PUBACKs are in order, even if I publish more than 150 messages at once.
I'll implement queing for gateway topics myself then.
Thank you for pointing out that it might be a server problem.

In this case the client in too strict. I'll solve it in the next release. IT wiill take some time to implement "retransmission".

Closing this in favour of #46.


slides over each other when screen resolution is 1920*1080

dear mrs.
when view the slider in 24" monitor in sizes such as 1920*1080 the slides show over each other and slider not work correctly such as this sample with two different types of use. anybody can help me?
https://www.demo2.eshop24.ir/

Hi,
Are you still experiencing the issue?


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

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

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

Questions ?
Please refer to the Backport tool documentation


Reminder: Emojis for every planet in Solar System

Proposal from 2017
It's 2021, Starship will go to Mars soon, but people can't communicate about it with Emoji... only "Ringed Planet"🪐, really?
You have Emoji that nobody uses like 📇 , yet for Mars you don't have. But it's better to add Emojis for all planets in one update.
If it's already planned, please tell me when it will be released?

Could this be achieved as a zero width joiner (U+200D) sequence of the planet’s symbol (e.g. ♂ U+2642 for Mars, or ♆ U+2646 for Neptune) and the ringed planet (🪐 U+1FA90; or some more generic heavenly body symbol like a comet [☄ U+2604] or a star [⭐ U+2B50])?

originally was here: #327


Please add support for plural to return "sticazzi" as response

I don't think that this feature should be implemented. 'Sticazzi' means 'I couldn't care less'.

@stefanofrigerio I think it as well. Maybe I should close this but I wanna wait some argumentations :)

Great! I agree with your explanation too, so I close the issue.


Support IAgileObject and IMarshal when exporting to WinRT

IAgileObject supported as of https://github.com/tristanlabelle/swift-winrt/commit/8f4b8167e478521d07e8008c0a907113c65252b8


Select2 4.0.0 Beta 2 not working in modal popup

Hi,
I have been using select2 3.5.2 and recently updated to the latest version 4.0.0 Beta 2. It has been noted for me that the latest version is not supporting in modal popup which comes. Can anyone please suggest if there is any solution for this. Temporally I have moved back to the previous version itself.

I faced with the same bug, using Jquery 11.1 and Jquery-UI 10.4.
Select's element did not opens up a list of options (however select2 works with manual method .select2("open") in a modal window)

Can someone create a jsbin/plunkr that demonstrates the issue(s)? It's difficult to test this for me as we don't use modals anywhere.

http://plnkr.co/edit/GPQTa1aaRJNn1cuz5kos?p=preview
Select2 4.0 works in Firefox, but it seems that selection menu is hidden behind modal-window.

It appears to be a z-index issue (which is unfortunate).

Bumping the z-index of the dropdown to 101 solved it here:

I need to find out what the Bootstrap z-index for the modals is, and then I'll push a fix.

.modal-dialog {
...
z-index: 1050;
}

Changing the z-index helped to show select2 element in front of modal window, but still there is a issue with select2 element's search function in modal window.

but still there is a issue with select2 element's search function in modal window.

I have a feeling this is related to the modal taking the focus back. I'll have to do some more testing to figure out what combination of event blocking and modal patching fixes it.
The issue sounds exactly like https://github.com/select2/select2/issues/1246, so at least we have a starting place.

I'm quickly read the thread, and i want to notice that everything works with select2 ver. 3.5.1 (if it helps)

The solution still appears to work, though it needed a slight modification. I've posted an updated snippet in https://github.com/select2/select2/issues/1246#issuecomment-71710835 for future visitors to use.


[Icons] Add a way to access icons via PHP code

It would be helpful to have a way to access icons from PHP code (for example, to send SVG data in a payload to be loaded by JS).
This could possibly be by loading the icon registry as a service and then fetching an icon (using the same identifier as I would use in the Twig ux-icon('icon-name') function.

I'd like to stabilize the API a bit before adding this but for sure, this should be possible. I'm thinking to make IconRegistryInterface auto-wireable (removing the @internal flag from it and Icon).

Agree with @kbond ! Yes to the idea, but let’s take some time before exposing anything, as the next week could require some changes in the implementation.

Hi @gubler .. it's been a while, but if you missed it, you now can inject / autowire the Renderer where you need to use it :)
--> Symfony\UX\Icons\IconRendererInterface

Thanks! I really appreciate it :)


[Feature] The about us, upcoming events and our team sections can be made better than it is now

Contact Details
<EMAIL_ADDRESS>What's on your mind?
The about us, upcoming events and our team sections of the website can be made good by applying some styles to the file.
Contribution Guidelines

[X] I have gone through the contribution guidelines

Code of Conduct

[X] I agree to follow this project's Code of Conduct

i want to work on this section

@ShakthiW This issue is addressing #4 #5 #6 together
As those are independent issues
We have to close this Issue
Thank you for reaching out to us.


Feature: Proposal View

add proposal-station list view ( comment, status, ... ) but only for proposal owner
redesign overview

closed by 2c84b78337474e0a7d7fb4322aa42e3e2839cbb7


Language Server API for AI

Update the language server API to support the conversational style of interaction with AI
TODO
The currently supported ai/complete method makes a single request to AI, expecting a code snippet in response
The new method should support the case when AI may ask for auxiliary information by requesting to evaluate an expression.
The client should receive notifications to see the progress of the conversation with the AI happening under the hood.

I'd like to point out the potential similarity of this task with

#5629

Also in the Table.join case the computation of dynamic drop downs wants to access additional values. In the current PoC

#9410

case such an access is achieved by a builtin that exposes runtime cache content and allows the Enso code computing dynamic drop down content to find value of any Table.join argument via sharing its UUIDs.
I am convinced there shall be no special AI support in the language server protocol. Whatever IDE needs shall be achieved by IDE invoking an Enso library method, just like visualizations do and widgets do. If we need additional support (to report pending progress for example), we should design a generic language protocol mechanism.
I that historically the AI support involves special modifications to language server. However that shall be eliminated - please keep this in mind when designing a new solution. Make sure new features are moving us closer to such "IDE invokes an Enso method" goal and not further.


Fix xcopy invocation for spaces in path

The post-build xcopy invocation failed because /EXCLUDE cannot deal with spaces, not even quoted. Fixed by adding a cd to $(SolutionDir) and using a relative path in the argument.
I have not written a test for this fix.

Done, thanks for the fix, @sjmulder!

Cheers, thanks for merging!


Add support for iOS 13

iPhone X (10,3)
iOS 13.2.2
Checkra1n 0.9.5

While I wasn't initially intending to add support for iOS 13 to Ultrasound, due to Apple's new volume HUD (which I really like!), I've gotten a lot of requests for it since checkra1n dropped, so I've decided to reconsider. This is now a priority for the next release.

I've pushed a commit on the support/ios13 branch that adds preliminary support for iOS 13. Most things work. The main thing that doesn't is OLED mode, which will take some significant effort to fix.

Thanks, now its working fine in iOS 13 iPX, I don't really care about OLED mode so its okay
have a nice day


Use lodash.cloneDeep to ensure we never emit any references to the ra…

…w document stored in a LocalCollection

yay thanks


FC-16 Write Multiple Holding Registers fails

As per the ReadMe.md neither writing a single register or multiple registers are supported.
There is a statement:
"Please note all other function codes (including vendor-specific extensions) are supported on a "best-effort" basis and most likely will fail."
I would like to extend this to support writing multiple register requests, where do you feel the shortcomings are specifically?
Interestingly the problem I experience is writing a single register (Function Code 6) works but an app that I am trying to use the gateway with is writing a single register using Write Multiple Holding Registers (Function Code 16) and this fails.

I have same problem. I use Eastron SDM120-Modbus. It uses 32bit float.
Reading (03 - Read holding registers) is OK.
But writing of the same registers (16 - Preset multiple registers) fails.
mbusd_err16.txt

@sgel83 hmm, actually mbusd supports multiple registers reading/writing (function codes 03 and 16) and it should work, but for specific devices your mileage may vary. I've checked your logs and see many lines with "tty: dropped  bytes" message. That's quite unusual, are you sure mbusd is single master on the Modbus RTU network?
@zmata it looks like your device doesn't respond to the request. Could you try to perform the same request using some other master RTU software directly connected to your device?

Hi Victor,
Thanks for taking a look at this.  Here is a new log without the secondary master on the RS-485 network.  The behavior of the app is still the same where it fails to write.  I actually think it is similiar to zmata, as you will see in the wireshark trace it shows the gw is not responding.
This shouldnt be the case though because if I stop mbusd and use pymodbus I can change the value using FC-16 without an issue.
single_master.zip

Hello @sgel83, thanks for the log files, I've checked .pcapng file and it looks like fc16 request is malformed:

It seem the root cause of a problem is invalid Modbus/TCP length field, while actual encapsulated Modbus frame size is 9 bytes, the length field indicates it's only 6 bytes long. That looks quite odd. What kind of Modbus TCP master software you're using?

Its an android app for a centralized ventilation system:
https://play.google.com/store/apps/details?id=com.Amalva.c4_app
According to the reviews it works but maybe thats how they keep the customer from using other gateways, they just look at the data size in the packet instead of modbus/tcp length header.
Good catch.

I use Simply Modbus TCP Master. It's free, and super useful. They also have Simply Modbus RTU, to test RTU devices.

My log is from test with Simply Modbus TCP Master.
If I use Simply Modbus RTU for direct writing registers - succes.
If I use Loxone Smart Home miniserver to send through MBUSD, it fails (same as Simply Modbus TCP Master)

@zmata, I think we have different issues though.  My issues is the client app is misbehaving by not calculating the mbdata length correctly for the mbtcp length header.  If I use pymodbus as a TCP client to write the same register message (but with the correct size in the length header) it is working..
@3cky Are you open to the idea of adding graceful handling of this type of situation where it reads the mbus data and calculates the length?  Or do you think its too much of a performance hit?

@sgel83 still thinking about the best way to handle this type of situation gracefully. There are some options, will look into them.

@3cky Great, in the meantime I also emailed the manufacturer, and requested them to fix it.  No feedback yet.

Regarding my problem, that is definitelly another then one by @sgel83
Everything is ok using Simply Modbus TCP Master. I can write 16 - Preset multiple registers. It was problem in my configuration of the Simply Modbus TCP master before - I'm sorry.
But it fails using Loxone Smart home Miniserver.
I tried to sent same command from Loxone and Simply Modbus TCP Master through MBUSD, receiving it by Simply Modbus TCP Slave. I sent value 2 (32bit float) to address 40029. There is result:
From Loxone:
2018/04/12 01:02:26  >>> 19 DF FF C7 FF FB F7 FF FF 7F FF 3B 61 FE
2018/04/12 01:02:26  >>> 19 DF FF C7 FF FB F7 FF FF 7F FF 3B 61 FE
2018/04/12 01:02:27  >>> 19 DF FF C7 FF FB F7 FF FF 7F FF 3B 61 FE
2018/04/12 01:02:27  >>> 46 F7 C7 FF FB F7 FF FF 7F FF 3B 61 FE
From Simply Modbus TCP Master:
2018/04/12 01:12:44  >>> 46 F7 C7 FF FB F7 7F FF FF FF 73 61 FE
2018/04/12 01:12:44  >>> 19 DF FF C7 FF FB F7 7F FF FF FF 73 61 FE
2018/04/12 01:12:45  >>> 19 DF FF C7 FF FB F7 7F FF FF FF 73 61 FE
2018/04/12 01:12:45  >>> 19 DF FF C7 FF FB F7 7F FF FF FF 73 61 FE
Can you help me analyze the problem? Thanks.

@zmata It seems that its possible to change the behavior of Simply Modbus TCP Slave, have you tried this?
http://www.simplymodbus.ca/RTUSlaveManual8.htm
Section "The Simply Modbus Slave Window"
You have 2 check boxes:
"High byte first"
"High word first"
http://www.simplymodbus.ca/FAQ.htm
It outlines 4 possible configurations:
                                                                                also know as
 AE41 5652       high byte first      high word first    "big endian"
 5652 AE41       high byte first      low word first
 41AE 5256       low byte first      high word first
 5256 41AE       low byte first      low word first     "little endian"

I would reccomend trying all 4 combinations in the Slave program

Hi, there is no problem with receiving in slave app, but in slave Energy Metter, and it's not possible to switch this setting in it.

@3cky
I made a fix, and tested it.  It seems to be working but I am not a modbus expert so I think you should review it critcially.
I made the following assumptions.

modbus length header only applies to TCP connections.
If the header length matches the actual length of what was received it leaves it alone. If not it changes the value to the calculated length before any further processing.

@sgel83 I've pushed some fixes into master, please update and test.

My problem is another one, so I open new issue 33

Closing issue, the code change corrected this.  Thanks again


NameError: name 'cc' is not defined

Hey,
running command like:
python coverageMaster.py sample.cov sample.flagstat.txt -r total_ref_m_std -o out -b my.bed
results in error:
Thu Sep 15 09:29:26 2022 - --------------------------------------------------------------------------------
CoverageMaster is warming up
Thu Sep 15 09:29:27 2022 - Query index created
Thu Sep 15 09:30:30 2022 - Reference index created
Thu Sep 15 09:30:30 2022 - Processing Gene: XIST - Region chrX:73040485-73072588
Traceback (most recent call last):
  File "/mnt/data/production/CNV/samDepths/refsout2/../../coverageMaster/coverageMaster.py", line 360, in <module>
    for n,c in enumerate(cc):
NameError: name 'cc' is not defined

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/mnt/data/production/CNV/samDepths/refsout2/../../coverageMaster/coverageMaster.py", line 398, in <module>
    raise Exception("Error in __main__")
Exception: Error in __main__

indeed in coverageMaster.py cc is defined only in optional cases:
    try:
        signalBuffer = Manager().dict()
        if options.control:
            cc = open(options.control).read().strip().split("\n")
        if options.single_control:
            cc = [options.single_control]
        for n,c in enumerate(cc):

options.control and options.single_control suppose to be optional
parser.add_option("-c",'--control',  dest="control", help = "<optional> txt file with a control file per line (.cov with .report.txt in same folder)")
parser.add_option("-s",'--single-control',  dest="single_control", help = "<optional> single control file (.cov with .report.txt in same folder)")

ok, it's just a matter of documentation...


Profiles missing after resetting config

Windows Terminal version (or Windows build number)
1.10.2383.0
Other Software
No response
Steps to reproduce

have terminal ver 1.9 with default profiles including powershell/WSL/Azure shell
upgrade to v1.10
open settings.json
delete contents of file and save
reload terminal

as per notes at top of config file, i wanted to ensure that i had any new defaults now that v1.10 has been released
"..It should still be usable in newer versions, but newer versions might have additional
settings, help text, or changes that you will not see unless you clear this file
and let us generate a new one for you."
Expected Behavior
expected that all default profiles get regenerated
Actual Behavior
only "windows powershell" and "Command prompt" appear.
Also note that terminal did infact launch "powershell" (v7.1.4) and had an empty dropdown in the "default profile" box in settings as what was launched did not have any profile.
I had to reset the app to restore all my profiles

HMMMMMM This is a good point. In 1.11 (and backported to 1.10), we added the ability to just delete dynamic profiles from the settings, and have them not auto-regenerate. But if you totally blow away the settings file, that's kinda a different case now.
@DHowett @lhecker for discussion: if settings.json is totally empty, or missing, should we clear out state.json too? Or is that just a bad heuristic? Is there a reason we would want to keep state.json even if they blow away their settings?

@DHowett @lhecker for discussion: if settings.json is totally empty, or missing, should we clear out state.json too?

Sounds good to me actually.


Use ReadOnlyMemoryStream for BinaryData.ToStream

ReadOnlyMemoryStream copied from https://github.com/dotnet/runtime/blob/6072e4d3a7a2a1493f514cdf4be75a3d56580e84/src/libraries/Common/src/System/IO/ReadOnlyMemoryStream.cs
Removed overrides that are not part of .NET Standard 2.0:

public override int Read(Span<byte> buffer)
public override ValueTask<int> ReadAsync(Memory<byte> buffer, CancellationToken cancellationToken = default(CancellationToken))

Also not including as it would require copying additional files:

 public override IAsyncResult BeginRead(byte[] buffer, int offset, int count, AsyncCallback? callback, object? state)
public override int EndRead(IAsyncResult asyncResult)

Also not overriding CopyTo/CopyToAsync as the implementation in .NET Core relies on .NET Core Stream WriteAsync method that takes a Span (which is not included in Standard 2.0).

We need more tests for the new stream type. Changing position seeking etc. etc.


Adds Lacework integration tile

What does this PR do?
Adds Lacework integration tile
Motivation
We released an integration with lacework and a tile in app allows it be searchable

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Discussed this should be on extras repo


ARROW-18073: [CI][Conda][macOS] Require macOS 10.13 or later

See "[Discuss][C++] macOS minimum requirements":
https://lists.apache.org/thread/pjgjrl716gvqzql586cnnoxb38nb0j5w

@github-actions crossbow submit conda-osx-*

@github-actions crossbow submit conda-osx-clang-*

This doesn't solve anything...


Add preference to open new outlines in a new window with the sidebar collapsed

This will now happen automatically when the main window has been closed.


Bugfix/TS 342 Stat Con Contact

What's included?
User Raised Issue BR_ADMIN_DE_000239 #342
Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
Example exercise
When Stat Con is waived, check if you can complete the contacts section without specifying a Stat Con contact.
Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
Demo:
video-30141148-1532855a73cc4873e598363eb3cdd513.webm
Related permissions
Have permissions been considered for this functionality?

No permission changes required
Permissions have been added / updated. Details:

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

why is it blank when I click No?

@HalcyonJAC why is it blank when I click No? see the 2 screen shots below

@NormaJAC Thank you for spotting that. It has been fixed. When you click No, it should display No.


afl-clang-fast: remove useless codes related to maybe_linking

Now that the maybe_linking check is removed in afl-clang-fast.c, maybe it is better to remove useless codes caused by it since maybe_linking is now defined but not used.
I think in the old code, "maybe_linking" should not be set to zero on seeing the "-shared" argument because dynamic libraries should also statically link the AFL runtime into themselves. But now that the maybe_linking variable is removed, it is no more a problem.
Besides, the code 'if (argc == 1 && !strcmp(argv[1], "-v")) maybe_linking = 0;' in line 146 itself is useless regardless of the maybe_linking variable because in main() we have check "if argc < 2, then exit".

@googlebot I signed it!

Just linking the PR that removed the usage of maybe_linking for reference: https://github.com/google/AFL/pull/80


Remove Testing folder.

https://github.com/econ-ark/HARK/issues/303

So, I've labeled this "in progress" since it seems like it is awaiting your revisions -- @pkofod, when you post your update let me and Matt know so we can merge it in.

I'm not sure that there's much else to do. We can discuss in #303 if we want to re-use the code deleted here in another setting or form.

I lost track of this.  I know I meant to merge before, but what's the relationship with the other identically named PR from Shauna?

Same question from me, @pkofod and @shaunagm

I didn't see this PR - @pkofod and I had a discussion about this on another issue and I didn't realize he had (or was going?) to submit a PR himself.  I'll go close my PR.

Should be ready to merge, and any further discussion should happen at #303

Happening today.
On Sun, Jun 16, 2019 at 6:06 PM Patrick Kofod Mogensen <
<EMAIL_ADDRESS>wrote:

Should be ready to merge, and any further discussion should happen at #303
https://github.com/econ-ark/HARK/issues/303
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/304?email_source=notifications&email_token=ADKRAFKETGGMBTO576SHO5TP222P5A5CNFSM4HX5O26KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXZWFLA#issuecomment-502489772,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADKRAFKL2MGIXHY5KL7X6WLP222P5ANCNFSM4HX5O26A
.

Today? :)

Today is a very long day.

Today is a very long day.

That can be good or bad, but thanks for the merge!

today and today and today creeps in this petty pace from day to day?
On Tue, Jun 18, 2019 at 10:09 AM Matthew N. White<EMAIL_ADDRESS>wrote:

Today is a very long day.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/pull/304?email_source=notifications&email_token=AAKCK7YL4BYDVHAZWEYZLLTP3DUAHA5CNFSM4HX5O26KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX6X5LY#issuecomment-503152303,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKCK744ZAGS7RSJIN4IATTP3DUAHANCNFSM4HX5O26A
.

--

Chris Carroll

Release note: Consolidated tests and set them up for automatic running


[MINOR][INFRA] Add -Pdocker-integration-tests to GitHub Action Scala 2.13 build job

What changes were proposed in this pull request?
This aims to add -Pdocker-integration-tests at GitHub Action job for Scala 2.13 compilation.
Why are the changes needed?
We fixed Scala 2.13 compilation of this module at https://github.com/apache/spark/pull/30660 . This PR will prevent accidental regression at that module.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Pass GitHub Action Scala 2.13 job.

Test build #132404 has started for PR 30661 at commit 36c5ebf.

@dongjoon-hyun Thanks for the work! I planed to do this work after #30660 merged and verifying all docker-integration-tests pass.

Oh, I see. Thanks, @sarutak .

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37004/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/37004/

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

Scala 2.13 build pass on GitHub Actions. Merged to master and branch-3.1.
Thanks @dongjoon-hyun !

Thanks!

Test build #132404 has finished for PR 30661 at commit 36c5ebf.

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

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

Nice, LGTM!


The AdGuard CA certificate from the system store is not recognized on Android

So, we have a lot of errors when the AdGuard app shows a notification "Some app cannot recognize our certificate ..." for applications that don't have SSL-pinning.
This bug can catch on Android 9 and 10, too. Fix it, please.

Sorry, guys, my fault.


Update README.md

Add some callout boxes for ✨ visibility ✨
Remove a couple of module references because they're now years old and no one still even thinking about using them should be allowed access to dnd5e v3.0.x
Miscellaneous touch-ups because I'm a perfectionist

on that note; MidiSRD: still relevant?
the second-hand word-on-the-street I hear wafting in my 22nd-floor office window is "no," but I don't play this game

MidiSRD: still relevant?

... Maybe?
There's some things that Midi SRD automates that CPR doesn't. However, it is suffering from development lag as the maintainer appears to be having Life Things™ happening. Last I heard they were getting ready for a dnd5e v3.0.x version but (as you can see in the issue) there have clearly been delays.
At the very least, I don't see a reason to get rid of the integration just yet.


Add Authentication Provider Support

Add the necessary interfaces, provider factories, and configuration file support to enable external Authentication Providers to be implemented and referenced
Update ORAS store to support using new Authentication Provider for credentials

Note: Implementation of specific external Authentication Providers will be tracked in separate issues
Design document
Related to overall Authentication model discussed in #30

@etrexel @sajayantony @mnltejaswini Here is the design document regarding the interfaces and scaffolding required to be added for AuthProvider support. Appreciate a review when you have the time.


Watch has problem detecting changed files on macos Big Sur

After having upgraded to macos Big Sur 11.6, it seems chokidar has problems detecting file changes when watching.
I am posting here because I have found a temporary fix with chokidar-cli but all my watch tasks on all my projects seem to be affected (browser-sync, onchange, other cli tools with a watch task, etc)
Steps to reproduce

Install homebrew and Xcode command line tools
Install Node and NPM via homebrew or NVM (I've tried both)
Run brew doctor: no errors or warnings
Run npm i no errors or warning
Create a simple watch task using chokidar-cli
Run the watch task

The watch task will run but will not pickup any modified filles.
Reduced test case
"scripts": {
    "scripts:dev": "esbuild \"./src/js/app.js\" --bundle --outfile=\"./dist/js/app.bundle.js\"",
    "watch:scripts": "chokidar \"./src/js/**/*\" -c \"npm run scripts:dev\"",
 },

npm run watch:scripts runs fine but does not pickup any file changes
adding the --polling flag seems to "fix" the issue

Modified scripts
"scripts": {
    "scripts:dev": "esbuild \"./src/js/app.js\" --bundle --outfile=\"./dist/js/app.bundle.js\"",
    "watch:scripts": "chokidar \"./src/js/**/*\" -c \"npm run scripts:dev\" --polling",
 },

I cannot manually modify every tool and watch file of my toolchain. Is there any way I can avoid needing that --polling flag and have macos / chokidar notice my changed files ?

Waas a user permission issue apparently. Nothing to do with Chokidar.

created another admin user
watch worked
nuked the other account

All is fine in my world again


Wasmtime: avoid table lazy-init runtime checks on nullable funcref tables with null default

From this comment: it would be great to have an optimization such that

if a table's default funcref is null, don't do lazy-init, and make the 0 bit-pattern mean null rather than non-initialized-pointer-to-anyfunc.

This comes from a circumstance where we know we'll initialize a table slot manually before calling it (not doing so is a program error that should lead to a trap), possibly because we already have a branch for the case where it is null at the program-logic level (e.g. fallback IC). We still must trap on null somehow but #8159 can do that implicitly by detecting a segfault in a load from a NULL anyfunc pointer. The above optimization then means that we do not need the lengthy lazy-init-check sequence on loads from such tables, and we can do so without requiring any initialization at instantiation time either.
cc @alexcrichton @jameysharp

I like this idea! Some implementation points I can think of off the top of my head:

This would only be applicable when there are no elem segments with non-null entries for a table.
We'd need to store runtime state for each table to know whether it contains tagged functions or non-tagged functions.
In the Module's type information we'd need to store this in a TablePlan as well so the table access codegen could take it into account as well as runtime instantiation

See also https://github.com/bytecodealliance/wasmtime/issues/8002#issuecomment-2008139279

An additional requirement, I think, is that we can only do this for tables which are not exported. If another module imports a table, it has no way of knowing at compile time whether this optimization applies.
I think this technique is still broadly applicable in spite of that though.
Nick pointed out that we could build a static page full of pointers which are null except the low bit is set, and CoW-map that page repeatedly as the initial table contents, if we know that the table is initially all null. That would be compatible with codegen that can't optimize away the lazy-init check, which would make it safe to export such tables.
He also pointed out that we could use the same trick to swap the meaning of our sentinel values: a (void*)1 would then mean "uninitialized", but we could still instantiate quickly without writing a bunch of bytes into a bunch of new pages. Then we'd be able to avoid setting and clearing the lazy-init flag on every table access. When we can prove the initial values are all null, we'd map zero-filled pages instead.
We discussed many more ideas, but I'll leave it at that for now.

We talked some more in the Cranelift meeting this morning again about this and I wanted to summarize our thinking here as well.
The main realization we had was that we can address the concern of "now there's two kinds of null pointers" by, well, not having two kinds of null pointers. For example initialization of of an all-null table where the null pointer is (void*) 1 would be a bulk-initialization using that value (a memset-but-with-usize-value-things). Either that or an mmap'd memfd or page or something to use VM tricks. More-or-less we realized that we could implement the optimization in this issue, fast initialization of all-null tables, and then in codegen we can omit all lazy init checks since everything is statically known to be initialized.

As a quick data-point to drop here: in a very hacky (work-in-progress) dev branch of SpiderMonkey + weval with traditional funcptr ICs with signature checks simply removed (unsafe!), I measured the impact of the lazy-init table scheme on runtime directly and came up with ~8% -- fairly significant and makes the optimizations discussed here worthwhile, IMHO. (I'm going to "speculatively live in the glorious future" with this branch of wasmtime for all my experiments, which also removes table bounds-checks and stack limit checks, and hope that we can productionize all of this at some point!)


Consider adding annotations to the pod template

Hi! We've been using kbld for over two years now with huge success. Thanks! There is just a tiny thing that is bugging us:
Describe the problem/challenge you have
As it is usually done, we scrape our pods to collect metrics from them. When using kbld, the deployed image-tag cannot be retrieved just by looking at the pods. Unfortunately, kbld adds its annotations to the owning object (Deployment, StatefulSet, ...) only, but not to its pod template.
Concrete example: We have this manifest:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: opa
spec:
  template:
    spec:
      containers:
      - image: docker.io/openpolicyagent/opa:0.55.0
        name: opa

When feeding it to kbld, it resolves the tags and adds its annotations:
apiVersion: apps/v1
kind: Deployment
metadata:
  annotations:
    kbld.k14s.io/images: |
      - origins:
        - resolved:
            tag: 0.55.0
            url: docker.io/openpolicyagent/opa:0.55.0
        url: docker.io/openpolicyagent/opa@sha256:b40919fc5eaf22b70437e81e6575841ad1b034f46b1150873e55fa446cd9f0ca
  name: opa
spec:
  template:
    spec:
      containers:
      - image: docker.io/openpolicyagent/opa@sha256:b40919fc5eaf22b70437e81e6575841ad1b034f46b1150873e55fa446cd9f0ca
        name: opa

Instead, I would like to see the annotations also (or exclusively, maybe guarded by an option) added to the pod template:
apiVersion: apps/v1
kind: Deployment
metadata:
  annotations:
    kbld.k14s.io/images: |
      - origins:
        - resolved:
            tag: 0.55.0
            url: docker.io/openpolicyagent/opa:0.55.0
        url: docker.io/openpolicyagent/opa@sha256:b40919fc5eaf22b70437e81e6575841ad1b034f46b1150873e55fa446cd9f0ca
  name: opa
spec:
  template:
    metadata:
      annotations:
        kbld.k14s.io/images: |
          - origins:
            - resolved:
                tag: 0.55.0
                url: docker.io/openpolicyagent/opa:0.55.0
            url: docker.io/openpolicyagent/opa@sha256:b40919fc5eaf22b70437e81e6575841ad1b034f46b1150873e55fa446cd9f0ca
    spec:
      containers:
      - image: docker.io/openpolicyagent/opa@sha256:b40919fc5eaf22b70437e81e6575841ad1b034f46b1150873e55fa446cd9f0ca
        name: opa

I don't really care about the annotations on the outer Manifest. Maybe it should be an option to add the annotations to the pod-template only.
Describe the solution you'd like
I don't really care if this proposed behaviour gets enabled by default or if this would be opt-in using some kind of option. Maybe this could be configured using a new annotation on the outer manifest, like so:
apiVersion: apps/v1
kind: Deployment
metadata:
  annotations:
    kbld.k14s.io/add-pod-annotations: 'true'

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

It just occurred to me that you probably decided to add the annotations to the outer manifest because metadata.annotations is a valid attribute for all kinds of kubernetes api objects, while spec.template.metadata.annotations is only valid for a specific set of api resources.

Hey
kbld has no understanding of Kubernetes at all. What kbld does is add to your yaml a field metadata.annotations with the build information. The majority of use cases we saw using kbld were in a scenario where k8s manifests are provided. I'm not sure how much we want to teach kbld about processing k8s resources.
@cppforlife do you have any reservations about this?

@joaopapereira Thanks. Your comment pretty much confirmed my assumption from my previous comment.
I think that's reasonable. If I absolutely need the annotations on the podSpec, I could probably write my own tooling to just move/copy the annotations around after kbld added them.


issues/295: Use testing.Testing() where appropriate

Fixes: https://github.com/komuw/ong/issues/295

Codecov Report
Patch coverage: 57.14% and project coverage change: -0.10% :warning:

Comparison is base (7f64d26) 74.95% compared to head (9aa99f0) 74.86%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #351      +/-   ##
==========================================
- Coverage   74.95%   74.86%   -0.10%
==========================================
  Files          43       43
  Lines        4963     4969       +6
==========================================
  Hits         3720     3720
- Misses        994      997       +3
- Partials      249      252       +3

Files Changed
Coverage Δ

automax/automax.go
50.00% <0.00%> (-50.00%)
:arrow_down:

internal/acme/acme.go
74.48% <33.33%> (-0.60%)
:arrow_down:

internal/acme/helpers.go
65.42% <100.00%> (+0.12%)
:arrow_up:

internal/acme/http_client.go
69.37% <100.00%> (+0.19%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Navbar design

This is a new PR for the Navbar Component 🙏🏼
Also, the changes are:

Navbar Design
Navbar added to every page.
Also, img tag changes to Image Component.

Issue Ref.  #107 & #114  😄

Prem screenshots please

Prem screenshots please

Prem screenshots please

Prem screenshots please

You are try loom for video too, check it out

You are try loom for video too, check it out

Homepage Desktop View:

Homepage Mobile View:

Mobile View of shop page:

@Harshi7016 Next time for sure i'll upload a video using loom.

Homepage Desktop View:

Homepage Mobile View:

Mobile View of shop page:

@Harshi7016 Next time for sure i'll upload a video using loom.


Analyzer does not report when a late final is initialized in both an initializer and constructor body

This code:
class C {
  late final int i;
  C(this.i) {
    i = 7;
  }
}

is guaranteed to fail, but analyzer does not complain. This is similar to https://github.com/dart-lang/sdk/issues/43265. But that issue requires flow analysis. I think for this case, we can have logic like, "if a field is set in an initializing formal parameter, it cannot be set in the constructor body." No flow analysis necessary.

And same for initializer: C(): i = 6 { i = 7; }

@scheglov


In Demo, change completion for GitHub.userRepositories to use dematerialize()

This creates a much cleaner completion block by using the Result method dematerialize() in the same do-try-catch block as the response mapping. Adds a mapNSArray() extension to Response in GitHubAPI.swift to support this and as an example. Also an example of how to create a error message from an error with default.
See #950 for more discussion.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #960   +/-   ##
=======================================
  Coverage   73.72%   73.72%
=======================================
  Files          22       22
  Lines         723      723
=======================================
  Hits          533      533
  Misses        190      190

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 e0924cd...c22653c. Read the comment docs.

Thanks a lot for contributing to Moya! I've invited you to join the Moya GitHub organization – no pressure to accept! If you'd like more information on what that means, check out our contributor guidelines and feel free to reach out with any questions.


docs: fix typo in "ranging"

📝 Description
Fixes a small typo in the docs: "rangining" should read "ranging"

@alcontributors please add @nathanhleung for docs.

@allcontributors please add @nathanhleung for docs.


Ported last unit tests

These are quite literally the last unit tests there are. Every one of them has been ported.

All the remaining tests that we could possibly want have now been implemented.
However, it won't build. The tests have highlighted some flaws in the porting of MediapipeUnityPlugin, and I intend to fix that in a future PR.

Yayyy we gucci

Yeah, we'll have to figure out this Windows integration problem.


Easy way to sync your solved questions with github repo (without affecting submission rate)

Just simply create a new account of leetcode.
Open your original account in incognito window and new account in simple chrome window. Because leethub extension does not work in incognito mode.
Now, in your real leetcode account go to problems section then sort them by solved(https://leetcode.com/problemset/all/?status=AC&page=1)
now open 1st code then go to submission section and copy your last submitted code from there and then paste it into new account question and submit your code.

Are you able to save submissions ?


fine tune

Hellio,
Is there a way to do fine tuning? I want to start from your model and continue training the network for my task..
Thanks

Hi @omriba - please try to post questions to the mailing list in the future since the GitHub issue tracker is for development discussions and bug tracking. We had a short discussion on fine tuning on the mailing list at https://groups.google.com/forum/#!topic/cmu-openface/niJuhF5jvM0
-Brandon.

@bamos OK, thanks for the link.


[FEATURE] Support for wait_for_completion flag in force merge API

Is your feature request related to a problem?
Currently the force merge api in the client waits for force_merge to get completed. But OpenSearch provides a way where if we pass wait_for_completion = true, it gives a task id and poll the task id to see if the force merge is completed or not.
What solution would you like?
Add the support for wait_for_completion flag in force merge api.
POST /hotels-index/_forcemerge?max_num_segments=1&wait_for_completion=true

What alternatives have you considered?
The only other way is to send Raw HTTP request, but the domain is backed with security creating raw HTTP request is very difficult and not feasible.
Do you have any additional context?
NA

@saimedhi What's the right way to bring this in with the new generated code?

@dblock To add 'wait_for_completion' parameter to the 'forcemerge' API in Python, one approach is through the Python Generator. For a more comprehensive solution, it's best to integrate this parameter into the OpenSearch OpenAPI spec, making it accessible across all language client libraries.

@saimedhi yes exactly, is there a walkthrough / dev guide of what to add where?

Yes we have a developer_guide on the api repo: https://github.com/opensearch-project/opensearch-api-specification/blob/main/DEVELOPER_GUIDE.md

@dblock, @nhtruong please transfer this issue to opensearch-api-specification. I don't have access to do it.

@saimedhi once this issue is moved there how long it will take for the attribute to be available in python client?

Hello @navneet1v,

Once a PR is merged into the spec, we can generate and update the Python client whenever needed.
If we directly add changes to the Python client, they may get overwritten when the generator runs. So, it's best to make the initial change in the spec.

If you're familiar with Smithy, consider contributing. This way, we can ensure our PR gets merged quickly.


Trying to retrieve data from nested objects

Hi, I'm trying to map data from a Json file to individual objects in order to sort them and perform other operations on them. However, the data structure required is quite complex and makes it hard to navigate.
Here is a snippet of the Json file:
{
  "items":
  {
    "item":
    [
      {
        "id": "0001",
        "type": "donut",
        "name": "Cake",
        "ppu": 0.55,
        "batters":
        {
          "batter":
          [
            { "id": "1001", "type": "Regular" },
            { "id": "1002", "type": "Chocolate" },
            { "id": "1003", "type": "Blueberry" },
            { "id": "1004", "type": "Devil's Food" }
          ]
        },
        "topping":
        [
          { "id": "5001", "type": "None" },
          { "id": "5002", "type": "Glazed" },
          { "id": "5005", "type": "Sugar" },
          { "id": "5007", "type": "Powdered Sugar" },
          { "id": "5006", "type": "Chocolate with Sprinkles" },
          { "id": "5003", "type": "Chocolate" },
          { "id": "5004", "type": "Maple" }
        ]
      },
      [...] Other items could be there
    ]
  }
}

I cannot change the way it is organised. However, I cannot figure out how to properly get every element and extract it.
My current code:
ifstream input("sample.json");

    json json1;

    input >> json1;

    json json2 = json1["items"]["item"];

    for ( auto& item : json2.get<json::object_t>() ){
        Item *tempItem = new Item();

        tempItem->setId(item.at("id"));
        tempItem->setName(item.at("name"));
        tempItem->setType(item.at("type"));
        tempItem->setId(item.at("ppu"));
        //Trying to go in the array batters
        object_t temp_obj = item.at("batters");

        items.push_back(new Item());
    }

Note: Even then I get an error because json2.get<json::object_t>()  returns a pair<>.
Thanks in advance!

This does answer my initial question, but now I get a parsing error.
    ifstream input;
    input.open(R"(C:\Users\Etienne\Documents\GitHub\Collaboration\assingment 5\sample.json)");

    json json1;

    input >> json1; //Error

    cout<<"after parse";

    json json2 = json1["items"]["item"];

For an unknown reason it doesn't accept the parsing.

What is the error?

It just stops, no error or anything. It tried printing after each statement and it happens at the JSON parsing.

Can you check whether you can print the file content? Can you run the code in a debugger?

Thanks for checking back!

How can one push batter["id"] into  another json object or vector.
I'm getting the following error for above query:
error:'batter' was not declared in this scope

Hey, thanks for your example code.
I have one question regarding it.
The following returns 0
    const auto& output = jFile["properties"]["rules"]["output"];
    std::cout << (output.contains("somekey")) << std::endl;

while this returns 1
    for (const auto& output1 : jFile["properties"]["rules"]["output"])
    {
        std::cout << (output1.contains("somekey")) << std::endl;
    }

my debugger is unable to unpack the type so im just wondering, why is the first 0 and the second 1?

Can you share the complete code example and JSON file?

thanks for you fast response.
Here is the original json
{
    "title": "Data Transform Configuration",
    "type": "generic",
    "properties": {

        "operations":[
            {
                "unique_id": "left_shift_1",
                "type": "operation_shift",
                "direction": "left",
                "bit_start": 0,
                "bit_length": 32,
                "bit_shift": 4
            }
        ],

        "rules": {

            "input": {},
            "output": [
                {
                    "lengthNumberType": "byte",
                    "dataLength":12,
                    "descritpion": "format either via references or direct in place, example shows in-place",
                    "format": [
                        {
                            "node_type": "text",
                            "unique_id": "1",
                            "content": "urn:epc:tag:grai-96:"
                        },
                        {
                            "node_type": "data",
                            "unique_id": "2",
                            "byte": 1,
                            "bit_start": 0,
                            "bit_length": 3,
                            "format": "decimal_ascii"
                        }
                    ]
                }
            ],

            "notify": {

            }
        }
    }
}

the example code did not contain that much more.
Just a read in of the above file
    std::ifstream i(configFilePath);
    i >> jFile;
    std::cout << jFile.dump(4) << std::endl;

where the following went with ok
    for (const auto &output : jFile["properties"]["rules"]["output"])
    {

        if (!(output.contains(Constants::KEY_LENGTH_NUMBER_TYPE)) ||
            !(output.contains(Constants::KEY_RULE_DATA_LENGTH)) ||
            !(output.contains(Constants::KEY_FORMAT)))
        {
            std::cout << "error while parsing. incomplete rule definition" << std::endl;
            return;
        }

    ...
    }

but when i used
const auto &output : jFile["properties"]["rules"]["output"]
instead of the for-loop it just gave 0 for the contains method

jFile["properties"]["rules"]["output"] seems to be an array. If you call contains directly on it, it will always return false, see https://json.nlohmann.me/api/basic_json/contains/.
If you iterate it in the for loop, output is a reference to a JSON object. Here, contains does what you expect.

Thank you very much for the clarification!


Array functions UNION_DISTINCT and INTERSECT - invalid argument count

My Environment

ArangoDB Version:       3.7.3
Storage Engine:            RocksDB
Deployment Mode:       Single Server
Deployment Strategy:   Manual Start
Infrastructure:               AWS
Operating System:        Ubuntu 20.04
Total RAM in your machine:
Disks in use:                  SSD
Used Package:               Ubuntu .deb

Component, Query & Data
Affected feature:
AQL query using web interface and arangosh
AQL query (if applicable):
LET outerside = (FOR j IN 1..65537
        LET mini = (FOR year IN (1900+RAND() * 100)..(1900+RAND() * 100)
            RETURN year )
        RETURN mini )

RETURN APPLY( "UNION_DISTINCT", outerside

AQL explain and/or profile (if applicable):
EXPLAIN in 3.7.3:
Query String (202 chars, cacheable: false):
 LET outerside = (FOR j IN 1..65537
         LET mini = (FOR year IN (1900+RAND() * 100)..(1900+RAND() * 100)
             RETURN year )
         RETURN mini )
 RETURN APPLY( "UNION_DISTINCT", outerside )

Execution plan:
 Id   NodeType               Est.   Comment
  1   SingletonNode             1   * ROOT
 14   SubqueryStartNode         1     - LET outerside = ( /* subquery begin */
  3   CalculationNode           1       - LET #8 = 1 .. 65537   /* range */   /* simple expression */
  4   EnumerateListNode     65537       - FOR j IN #8   /* list iteration */
 16   SubqueryStartNode     65537         - LET mini = ( /* subquery begin */
  6   CalculationNode       65537           - LET #10 = (1900 + (RAND() * 100)) .. (1900 + (RAND() * 100))   /* range */   /* simple expression */
  7   EnumerateListNode   6553700           - FOR year IN #10   /* list iteration */
 17   SubqueryEndNode     6553700             - RETURN  year ) /* subquery end */
 15   SubqueryEndNode     6553700         - RETURN  mini ) /* subquery end */
 12   CalculationNode     6553700     - LET #12 = APPLY("UNION_DISTINCT", outerside)   /* simple expression */
 13   ReturnNode          6553700     - RETURN #12

Indexes used:
 none

Functions used:
 Name    Deterministic   Cacheable   Uses V8
 RAND    false           false       false
 APPLY   false           false       false

Optimization rules applied:
 Id   RuleName
  1   splice-subqueries

EXPLAIN in 3.3.12
Query string:
 LET outerside = (FOR j IN 1..65537
         LET mini = (FOR year IN (1900+RAND() * 100)..(1900+RAND() * 100)
             RETURN year )
         RETURN mini )
 RETURN APPLY( "UNION_DISTINCT", outerside )

Execution plan:
 Id   NodeType             Est.   Comment
  1   SingletonNode           1   * ROOT
 11   SubqueryNode            1     - LET outerside = ...   /* subquery */
  2   SingletonNode           1       * ROOT
  3   CalculationNode         1         - LET #8 = 1 .. 65537   /* range */   /* simple expression */
  4   EnumerateListNode   65537         - FOR j IN #8   /* list iteration */
  9   SubqueryNode        65537           - LET mini = ...   /* subquery */
  5   SingletonNode           1             * ROOT
  6   CalculationNode         1               - LET #10 = (1900 + (RAND() * 100)) .. (1900 + (RAND() * 100))   /* range */   /* simple expression */
  7   EnumerateListNode     100               - FOR year IN #10   /* list iteration */
  8   ReturnNode            100                 - RETURN year
 10   ReturnNode          65537           - RETURN mini
 12   CalculationNode         1     - LET #12 = APPLY("UNION_DISTINCT", outerside)   /* v8 expression */
 13   ReturnNode              1     - RETURN #12

Indexes used:
 none

Optimization rules applied:
 none

Problem:
We have a database configuration script that is run to import data, establish relationships, and to precompute +8000 complex queries that we can regard as static to eliminate runtime delays. Script runs fine in 3.3.12 and fails in 3.7.3.  Both the old and new systems are running RocksDB.
The problem has been traced to array functions like UNION_DISTINCT and INTERSECTION failing due to argument count exceeding 65536. The AQL query above creates 65537 arrays - it functions on 3.3.12 and fails on 3.7.3.  Version 3.3.12 seems to have no argument count limit but can fail spectacularly (server rebooted) after running out of memory at 100k+.
The real query is iterating collections in the outer loops and LIMIT is not useful to break this into chunks.
Expected result:
Update documentation to indicate argument upper limit change.  If this value is configurable, provide instructions.

Hi @davidmw,
thanks, this should probably be added to our documentation.
If it helps, you can rewrite your query like this:
RETURN (
  FOR j IN 1..65537
    FOR year IN (1900+RAND() * 100)..(1900+RAND() * 100)
      RETURN DISTINCT year
)

This produces the same results, is more efficient, and doesn't have the limit.
It'd be a little bit better even to use
FOR j IN 1..65537
  FOR year IN (1900+RAND() * 100)..(1900+RAND() * 100)
    RETURN DISTINCT year

, but that changes the form of the result slightly, from e.g. [ [ 1915, 1916, ... ] ] to [ 1915, 1916, ... ].
If you need to be more flexible, you can also use COLLECT instead of RETURN DISTINCT.
HTH
Tobias

Thanks much Tobias. You got me looking for solutions not so dependant on UNION_DISTINCT. I've employed FLATTEN and SORTED_UNIQUE with good result.
LET arrayArray = [ [2, 3], [3, 1, 3, 5], [4, 8, 0, 3] ]

//RETURN APPLY( "UNION_DISTINCT", arrayArray )
RETURN SORTED_UNIQUE( FLATTEN( arrayArray ) )

Glad I could help!
Note that usually the high-level operations of AQL (like COLLECT) are better suited to process large numbers of documents than AQL functions, so I'd generally still recommend COLLECT (or RETURN DISTINCT) over SORTED_UNIQUE. But if it works well for you, it is, of course, fine. :)


If resource has a route show a Visit URL button

What would you like to see added?
If you have assigned a route to a resource, it would be great to have a button on the edit entry page to visit the URL, like in standard Statamic.
Is there a workaround you can use in the meantime?
Just remember the route.
Priority wise - where's this feature for you?
This is just a nice to have. I can deal without.

Actually, there should already be a 'Visit URL' button there so this is probably more of a bug than anything else. See here

Interesting, I will take a look at that.
Here's my code for future reference.
        \App\Models\MyModel::class => [
            'name' => 'My Model',

            'hidden' => true,
            // 'icon' => '',

            'blueprint' => [
                'sections' => [
                    'main' => [
                        'fields' => [
                            [
                                'handle' => 'title',
                                'field' => [
                                    'type' => 'text',
                                    'validate' => 'required',
                                ],
                            ],
                            [
                                'handle' => 'body',
                                'field' => [
                                    'type' => 'markdown',
                                    'validate' => '',
                                ],
                            ],
                            [
                                'handle' => 'images',
                                'field' => [
                                    'type' => 'assets',
                                    'container' => 'assets',
                                    'validate' => '',
                                ],
                            ],
                            [
                                'handle' => 'bard_content',
                                'field' => [
                                    'type' => 'bard',
                                    'display' => 'My bard content'
                                ],
                            ],
                        ],
                    ],
                    'sidebar' => [
                        'fields' => [
                            [
                                'handle' => 'slug',
                                'field' => [
                                    'type' => 'slug',
                                    'validate' => 'required',
                                ],
                            ],
                        ],
                    ]
                ],
            ],

            'listing' => [
                'columns' => [
                    'title',
                ],

                'sort' => [
                    'column' => 'title',
                    'direction' => 'asc',
                ],
            ],

            'route' => '/models/{{ slug }}',
            'template' => 'models.show',

        ],

<?php

namespace App\Models;

use DoubleThreeDigital\Runway\Routing\Traits\RunwayRoutes;
use Illuminate\Contracts\Support\Responsable;
use Illuminate\Database\Eloquent\Factories\HasFactory;
use Illuminate\Database\Eloquent\Model;

class MyModel extends Model implements Responsable
{
    use RunwayRoutes;

    // protected $guarded = [];

    protected $casts = [
        'bard_content' => 'json',
        // 'published_at' => 'datetime',
    ];
}

Out of interest, if you run php please runway:rebuild-uris and go back to that page, does the button show?

I can confirm the php please runway:rebuild-uris fixes it.


⚠ Bofrost.ch has degraded performance

In b7d61b4, Bofrost.ch (https://www.bofrost.ch) experienced degraded performance:

HTTP code: 200
Response time: 1694 ms

Resolved: Bofrost.ch performance has improved in 41470c5 after 1 day, 16 hours, 14 minutes.


Mend event

https://hub.spigotmc.org/javadocs/spigot/org/bukkit/event/player/PlayerItemMendEvent.html

Added in latest dev builds thanks to https://github.com/mcmonkey4eva/Denizen/pull/162


added gen2 cumulative object counting example

Cumulative object counting for both RGB camera or external videos.
Based on:

Tensorflow 2 Object Counting repository
OpenCV People Counter
tensorflow_object_counting_api

@Erol444 Didn't know there was a Tracker built into the depthai library. I just rewrote the code to use it instead of the CentroidTracker.

Sweet thanks @TannerGilbert !

Just tried it out, works great! Only thing I would change is to have the model set by default (to the /models/mobilenet-ssd.blob) and possibly adding the video from the readme (if it's not too big) to the demo folder. Thanks!

@Erol444 Done

@VanDavv Moved it into main.py


limit error message

truncate error to 256 chars (would it be possible to change it @ludovicc ? I was hoping to save tracebacks with error messages too)
improve round_dict function

I will remove this limitation of 256 chars for errors. Merging for now however.


support for fsspec.generic writes

@martindurant we recently addressed reading from s3, so you can now use _cp_file to copy files from s3 to the local fs (or to another fs that supports writes), but we can't write to s3 using this function.
I believe the issue is that S3AsyncStreamedFile doesn't implement the write function, so it defaults to the parent class write. The parent class write then fails because self.closed is not handled properly.
Do you know why write isn't implemented? Can we just use _pipe_file something like this:
    async def write(self, data):
        if self.mode not in {"wb"}:
            raise ValueError("File not in write mode")
        await self.fs._pipe_file(self.path, data)
        if self.size is None:
            self.size = len(data)
        else:
            self.size += len(data)
        self.loc += len(data)
        return len(data)

Maybe check that put_object returns a 200, add and manage a chunksize setting, buffer the data and do a single _pipe_file or turn everything into a multipart_upload? What do you think? I can put a PR together if it would help. I'm interested in getting the fsspec.generic cp and rsync functionality fully supported for s3.

When starting this, I had hoped that actual streaming uploads were possible. However, aiohttp and derivatives use a pull model on upload rather than push: the caller must supply a file-like which supports async reading (as opposed to an async write method we can call). It may be possible to work with this model, but it would be painful.
The other option is essentially to replicate the logic in the blocking code, and expose the async calls there (in initiare_upload, fetch_range and upload_chunk) as - which would limit the sizes of the individual calls as allowed by S3 rules.
pipe_file would not be useful on its own, since it is a call to send a whole file in one go. It may be async, but there is no way to pause mid-way to read more data nor a way to repeatedly call it for the same file (it would overwrite each time).


Mention passing files from stdin as potential use case

This has been added in https://github.com/dlang/dmd/pull/6880, but never got much attention and thus is probably not well-known.

thanks Seb, didn't know that.


stdlib: enable RuntimeUnittest on Windows

No longer limit this to POSIX platforms.  The threading has been
generalised to support all targets.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test and merge


Note: (XLM-)RoBERTa-based SpanMarker models require text preprocessing

Hello!
This is a heads up that (XLM-)RoBERTa-based SpanMarker models require text to be preprocessed to separate punctuation from words:
# ✅
model.predict("He plays J. Robert Oppenheimer , an American theoretical physicist .")
# ❌
model.predict("He plays J. Robert Oppenheimer, an American theoretical physicist.")

# You can also supply a list of words directly: ✅
model.predict(["He", "plays", "J.", "Robert", "Oppenheimer", ",", "an", "American", "theoretical", "physicist", "."])

This is a consequence of the RoBERTa tokenizer distinguishing  , and , as different tokens, and the SpanMarker model is only familiar with the  , variant.
Another alternative is to use the spaCy integration, which preprocesses the text into words for you!
The (m)BERT-based SpanMarker models do not require this preprocessing.

Tom Aarsen

Hey @tomaarsen ,
I have may a possible alternative solution: In Flair we also can construction and predict from sentences that are given by user. For this tokenization problem we use the v1 version of segtok. You could split the input into sentences, but for just tokenizing the word_tokenizer can be used:
https://github.com/fnl/segtok/blob/master/segtok/tokenizer.py#L210
I think this could easily be added in the Model Hub Inference logic:
https://github.com/huggingface/api-inference-community/blob/main/docker_images/span_marker/app/pipelines/token_classification.py#L35
So inputs could first be tokenized by word_tokenizer. I think that segtok would be a great alternative compared and more lightweight compared to spaCy.
Another alternative: not just implementing it on the Model Hub side: maybe it can be implemented in model.predict directly 🤔

I'll certainly consider this approach, whether with segtok, spaCy or NLTK. The spaCy version is already implemented.
By default, perhaps I can apply the tokenization only if Hello, there. tokenizes differently than Hello , there .?

I've discovered that the issue only persisted for XLM-RoBERTa, and I've been able to tackle it in f2edd06072aac2110b63aa9a7f1c52e45d6c6710!


Restore SonarCloud configuration

Description
This PR will restore SonarCloud workflow in the build pipeline. The reason for this is that we need to be ready for a security audit in near future.
At a later stage this will be moved to be a part of the generic build flow in geh-core.

I think we had sonar cloud on python part too, is that also meant to be restored as well as the .NET part?

I think we had sonar cloud on python part too, is that also meant to be restored as well as the .NET part?

The config for sonarcloud python is still present in the repo - no modifications needed for this part


Shorten arguments for ncclimo

Shorten arguments for ncclimo calls related to diurnal cycle. Resolves #96.

@golaz I'm not sure I did the fix correctly. I'm getting the following errors:
In climo_atm_monthly_diurnal output:
ncks: ERROR file time,1 not found. It does not exist on the local filesystem, nor does it match remote filename patterns (e.g.\
, http://foo or foo.bar.edu:file).
ncks: HINT file-not-found errors usually arise from filename typos, incorrect paths, missing files, or capricious gods. Please\
 verify spelling and location of requested file. If the file resides on a High Performance Storage System (HPSS) accessible vi\
a the 'hsi' command, then add the --hpss option and re-try command.
ncclimo: ERROR Inferred value of tpd= is not a positive integer. User requested high-frequency climatology or splitter operati\
ons without specifying the number of timesteps per day (tpd) in the input files, so ncclimo tried to infer tpd from the first \
input file. This operation resulted in a nonsense value. This is not surprising since a number of assumptions must hold true f\
or the inferral to work correctly.
HINT: Explicitly specify the number of timesteps-per-day in every input file with the --tpd option.

In all ts_* output:
ncclimo: ERROR Specify input file(s) with -i $drc_in or with positional argument(s) or with stdin
ncclimo: HINT Pipe input file list to stdin with, e.g., 'ls *.nc | ncclimo'

Looks like it's not working. Maybe rerun with the debug option and check the content of input.txt.

@golaz I added a second commit that seems to work when I run zppy with it. Let me know if this change looks good and I'll merge.

@forsyth2 : interesting how the change from < input.txt to cat input.txt | solves your problem. Some OS subtleties that I don't understand (and don't need to).
You may want to make additional minor changes: remove the trailing \ on lines preceding the deleted last line. It probably works as is, but relies on the presence of white lines which could break easily in the future.

Note: this might be difficult to do for ts.bash due to the if statements. Maybe just leave it as is.


Issue while fast scrolling vertically

First thanks alot for that wonderful efforts
.My issue is when I fast scrolling vertically its scrolling not smooth as compared to scroll slowly vertically.
Please suggest something.

Hi! Sorry for the late reply. My guess is some other view might be stealing touch events (like SwipeRefreshLayout, for example)


add index file

We don't

TST
On Sun, Jul 26, 2015 at 4:28 PM, WhoopDaBase<EMAIL_ADDRESS>wrote:

We don't
—
Reply to this email directly or view it on GitHub
https://github.com/Demango/JS-Osu-/pull/1#issuecomment-124986526.

--
Edgars Polis

@WhoopDaBase

We don't

No need to include a reference to a not-yet-existing file


Problems with MDFileManager

Description of the Bug
The file manager returns wrong path in non-preview mode. It's add to folder path full abs path of file instead just it`s name.
For example, i have Win10PE_v5_1_1_RU.txt file in D:\Zona Downloads\Windows 10 PE.
And I expect to get D:\Zona Downloads\Windows 10 PE\Win10PE_v5_1_1_RU.txt as result of choosing file.
But instead i got /Zona Downloads/Windows 10 PE//Zona Downloads/Windows 10 PE\readme — Win10PE_v5_1_1_RU.txt
Tha part of path is duplicated because files are shown not as  (for example) Win10PE_v5_1_1_RU.txt but /Zona Downloads/Windows 10 PE\readme — Win10PE_v5_1_1_RU.txt
So of cource i cant open file because of invalid path.
The fun part - no problems in preview mode. But i dont need images.
No code here, cause it the same even on example code from documentation.

Fix https://github.com/HeaTTheatR/KivyMD/commit/2407f3c4b2b23f390c29a7233235f8765f55186b


Adding 7.4 as it's still being supported PHP version

Adding 7.4 as it's still supported till the end of the year.
Supported Versions
closes (#132)

Can you resolve the conflicts?

@BrainMaestro I'll sort the conflicts but, PHP 7.4 is no longer actively supported do you think this is still useful?

PHP 7.4 is longer support the PR is redundant.


Adding Profile card to card components.

Is your feature request related to a problem? Please describe.
We need to share our details many times.
Describe the solution you'd like
I made a profile card so that one can share their details with anyone.
Describe alternatives you've considered
I made a profile card using HTML and CSS.
Additional context
https://user-images.githubusercontent.com/98695632/179393328-505722ee-f35d-4e58-8ab7-c33c059765de.mp4

Assigning it to you🥳🥳


SyntaxError: Unexpected token 'export' when importing slate-table

Description:
When trying to import the slate-table package in my editor in my Next.js application, I'm encountering the following error:
SyntaxError: Unexpected token 'export'
    at internalCompileFunction (node:internal/vm:128:18)
    at wrapSafe (node:internal/modules/cjs/loader:1280:20)
    at Module._compile (node:internal/modules/cjs/loader:1332:27)
    at Module._extensions..js (node:internal/modules/cjs/loader:1427:10)
    at Module.load (node:internal/modules/cjs/loader:1206:32)
    at Module._load (node:internal/modules/cjs/loader:1022:12)
    at Module.require (node:internal/modules/cjs/loader:1231:19)
    at mod.require (/Users/mikhail/Desktop/BeSavvy_Code_Base/savvy-front/node_modules/next/dist/server/require-hook.js:65:28)
    at require (node:internal/modules/helpers:179:18)
    at slate-table (/Users/mikhail/Desktop/BeSavvy_Code_Base/savvy-front/.next/server/pages/blog.js:492:18)
    at __webpack_require__ (/Users/mikhail/Desktop/BeSavvy_Code_Base/savvy-front/.next/server/webpack-runtime.js:33:43)
    at eval (webpack-internal:///./components/editor/Editor.js:31:70)
    at ./components/editor/Editor.js (/Users/mikhail/Desktop/BeSavvy_Code_Base/savvy-front/.next/server/components_editor_Editor_js.js:20:1)
    at Function.__webpack_require__ (/Users/mikhail/Desktop/BeSavvy_Code_Base/savvy-front/.next/server/webpack-runtime.js:33:43)
/Users/mikhail/Desktop/BeSavvy_Code_Base/savvy-front/node_modules/slate-table/dist/index.js:1
export * from "./table-cursor";
^^^^^^

Environment:

Operating System: macOS (version 14.4.1 )
Node.js version: v18.19.1 (also tried on LTS – v20.12.2)
Next.js version: 14.1.4
slate-table version: 0.1.0

Expected Behavior:
The Next.js application should start without any errors, and the slate-table package should be imported and used correctly.
Actual Behavior:
The application starts and works, but the error SyntaxError: Unexpected token 'export' is thrown every time when attempting to import the slate-table package.
Additional Information:

I've tried transpiling the slate-table package using Babel, but the issue persists.
The issue looks like the ES6 module syntax used by the slate-table package, which is not natively supported by my current Node.js version (v18.19.1). But I don't think this is the case and I can't just see the real problem.

Please let me know if you need any additional information or if there are any potential solutions or workarounds for this issue.
PS: It would be nice to resolve this issue because the package and documentation are great! I have been waiting for something like this for quite a while. And would like to see it develop!

Hello @MikhailKochkin, thank you for creating a detailed issue. I have now bundled the package for both CJS and ESM and pushed a new version (v0.1.1) to NPM.
Could you please update slate-table to version 0.1.1 and let me know if this resolves the issue?

Hello @nlulic,
Yes, I don't see any error messages any more. So I think the issue is resolved.
Thank you!

Great! If you encounter any more problems feel free to raise another issue.
And thanks a lot for the encouragement, I hope to find some time soon to improve the library. :)


Routing Flags not working on python

In the example of cvrptw https://github.com/google/or-tools/blob/master/examples/python/cvrptw_plot.py#L641 you suggest we can change the lsp flag like this parameters.no_lns = False but I get the error 'RoutingSearchParameters' object has no attribute 'no_lns'
This happens with every flag. ej. no_tsp

I think the flags and parameters were changed in bb6e091027eee3c88b855fbf67775be10f00c16f.   I can't really find any documentation on how you are supposed to set parameters, but the edit below seems to work for me.   I am even less sure about what is the default behaviour, and the rational behind setting various local search operators.  I will open a pull request if I hear this is a good enough example on how the parameters should be set, otherwise I will suggest a pull request removing the offending comments.
    parameters = routing.DefaultSearchParameters()

    # Setting first solution heuristic (cheapest addition).
    parameters.first_solution_strategy = (
        routing_enums_pb2.FirstSolutionStrategy.PATH_CHEAPEST_ARC)
    # Disabling Large Neighborhood Search, ( I think this is the default behaviour)
    parameters.local_search_operators.use_path_lns = False
    parameters.local_search_operators.use_inactive_lns = False
    # Routing: forbids use of TSPOpt neighborhood,
    parameters.local_search_operators.use_tsp_opt = False

    parameters.time_limit_ms = 10 * 1000  # 10 seconds
    parameters.use_light_propagation = False
    # To log the search uncomment the following.
    # parameters.log_search = True

Hello,
Since or-tools is using protobuf, you can find the parameters in .proto files. for example, routing parameters are defined in routing_parameters.proto and constraint solver parameters in solver_parameters.proto
As @mjfwest mentioned, you can set use_path_lns parameter like so : parameters.local_search_operators.use_path_lns = False
Please note that local_search_operators is instantiated here.
I agree that the use of parameters should be documented, and we will work on this as soon as possible, so I will close this issue, and open another one about documentation.
Regards,
Driss

What exactly is the benefit of disabling lns?


Configuration options for Kafka/RabbitMQ reconnect

The reconnect to Kafka works fine, but it is attempted with 'full speed' (about 10 per second on my machine). This behaviour should be configurable (for all binders). At least there should be a parameter for the intervall between two attempts and maybe some sort of backoff makes sense, too.
Furthermore, in the case of connection loss, an exception is logged. This should be replaced by dedicated logging.

Thanks Mark,
Both these issues would need to be fixed via Spring Integration Kafka, and we will address them as part of upgrading to SI Kafka 2.0 (which should be fairly soon).

@mark--
The reconnect frequency of the Kafka consumers to the brokers is set to 5 seconds, but I believe that the noise that you are currently seeing is caused by the Zookeeper client trying to reconnect. Are you stopping your Zookeeper ensemble as well ?
In the case of Zookeeper, the wait times on reconnect are random (between 0 and 1000ms) and I think that's what you see in the logs. Since that's just indicating that the client is trying to reconnect, I would suggest raising the logging levels to ERROR instead of WARN. We will set that by default in the client to avoid the noise.

Closed via #556 in https://github.com/spring-cloud/spring-cloud-stream/commit/6c4a271078bd8e32ed3e88ebad2c08755a39bbbe and cherry-picked to 1.0.x branch.


coverage: fix wrong tags parameter

run skydive-coverage

run skydive-scale-tests

Codecov Report

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

@@            Coverage Diff            @@
##             master     #797   +/-   ##
=========================================
  Coverage          ?   34.89%
=========================================
  Files             ?       60
  Lines             ?     8423
  Branches          ?        0
=========================================
  Hits              ?     2939
  Misses            ?     5171
  Partials          ?      313

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 ec9de93...24f7502.


Static methods

Very cool!
Is there an easy way to use static methods without building for Java 8 (Android N+)?  Unfortunately, interfaces don't support static methods prior to Java 8.
Android studio will give you an error in the IDE when you try to define a static method inside an interface:

Extension Methods are not supported at this language level

I see, there's a way to call static methods by creating a null instance of the mirrored object.
    @Test
    public void checkThatCallingAStaticMethodIsWorking() {
        dummy.setInstance(null);

        String result = dummy.getStaticString('b');

        assertThat(result).isEqualTo("b");
    }

Not super pretty though.  Would be nice if there was another way.


It's possible to change time duration of .ts file ?

Hi,
I can see the HLS system split movie into many .ts with 8 seconds time duration for each file.
That is possible to change it to 1 seconds ?
Best regards.
Thanks in advance

Yes, for example here in app.js
hlsFlags: '[hls_time=10:hls_list_size=6:hls_flags=delete_segments]',

I have see that in documentation but nothing change for me

when i change that, the ts duration changes and also the list... after restarting the app

What is your OS ? I'm on Win10 and only hls_list_size do something, hls_time changement not impact the duration of .ts (movie file)


Add missing 'raise' keyword

This triggers a misleading failure mode when there is an error while
retrieving pod annotations.
Signed-off-by: Salvatore Orlando<EMAIL_ADDRESS>

+1


feat: added page not found

In this PR:

Added router to catch unknown URL, will redirect to 404 page.

lgtm, could sit this in common ui probably, when have time

true, ok let me take a look.