Virtual serial console somewhat working.

Somewhat working virtual console. To test (neither works completely, but shows that communication is working), create 2 virtual serial consoles:
-device virtio-serial-pci,max_ports=1 \
-device virtconsole,chardev=vc02 \
-chardev socket,id=vc02,path=/tmp/vc02 \
-device virtio-serial-pci,max_ports=1 \
-device virtconsole,chardev=vc00 \
-chardev socket,id=vc00,path=/tmp/vc00,server,nowait \

Note that virtio-serial-pci is specified twice. I decided not to complicate the logic with the multiport feature support, but this means that for several consoles we need several buses (which is provided by virtio-serial-pci). Only port 0 is supported, which means that only "virtconsole" can be used (not sure if there is any difference between virtconsole and virtserialport).
Run ufs on one of them (before starting QEMU so ufs listens already):
$HARVEY/util/ufs -ntype=unix -addr=/tmp/vc02 -root=/tmp

Connect terminal input to another:
socat stdio unix-connect:/tmp/vc00

The virtual consoles will be visible as:
--rw-rw-rw- C 0 harvey harvey 0 Feb  7  2006 /dev/virtcon/vcons0
--rw-rw-rw- C 0 harvey harvey 0 Feb  7  2006 /dev/virtcon/vcons1

Try to mount vcons0 via 9p. The resulting message will be:
bad fversion conversion on reply

This looks the same as in the prior experiments with virtio-pci: mount asks for a buffer size 8192 bytes, and gets this number of bytes as the result of read, but expects a much shorter message. We can discuss this later, this just shows that data go back and forth over the virtqueue.
Try to communicate with the attached terminal on the host:
echo 123 >/dev/virtcon/vcons1

will show 123 in the terminal running socat. The reverse however does not work very well:
cat </dev/virtcon/vcons1

again, it requests an input buffer large enough. If 123 is typed in the host terminal, the whole buffer is returned, showing 123 in the beginning, and the rest is garbage.
So, we have raw communication working primarily. In order to make some protocols work over it, we may need additional filter program which for the case of 9p will try to determine the length of the message received by decoding its header, and pass only the necessary number of bytes. For terminal input, assuming it is in line-mode, such filter may look for the line end symbols and discard the rest.
This is of course not sufficient level of testing; I suggest that after this is merged, more people try to play with this.
Thanks.

Sorry I didn't get to review this quickly; FYI it seems to break harvey. I'm getting write faults trying to initialize the virtio console. If I disable the initialization routine, I can boot:
CPU Freq. 2266MHz
schedinit...
Scan fffffffff0000000
Scan fffffffff00f0000
get_coreboot_info: ret -1
virtio-serial-pci initializing
write fault fail, no segment, pid 1 addr 0xfffffffff4000000 pc 0xfffffffff020af24
panic: cpu0: fault: 0xfffffffff4000000


Better handle cases where govuk-text-colour is set to a non-colour value

Was: Reinstate support for 'inherit' as a text colour
The use of the rgba function, which was added to work around a bug in
Safari, prevents users from setting $rgba-text-colour to inherit.
This is useful if you wish to control the text colour by cascading a
style down from, say, a body tag, which in turn is useful in order to
apply this control at run-time rather than build-time.
This fixes the problem by first testing for the value inherit. When
found we simply evaluate to inherit, which is probably the best we
can do in the circumstance. When it is not found we apply the rgba
trick.
Addresses: #2231

I couldn't find any tests for this mixin so I've tried to add some very simple ones to demonstrate the cases that this PR caters to.
When I ran the tests locally, I would get sporadic failures, due to time outs, in random tests. I'm assuming that is a known issue and is nothing to do with these changes.

@36degrees: How's that?

@vanitabarrett: Any chance of a (patch?) release for this?

@daniel-ac-martin the Design System team are working on a couple of things at the moment which might be ready for release soon, so we're going to see how they progress. It may make sense for us to wait and include them in a feature release in the next couple of weeks.


What is the best way to determine whether an order is executed and against which price?

I'm able to create and confirm an order based on your documentation.
When trading on the DeGiro website, once an order is executed, a popup is shown briefly.
Is there a way to subscribe to this event via the connector?
Or should we poll actively to see the changes.
For example to retrieve pending orders, and when the id is not in, assume it's executed.
Or via OrdersHistory or TransactionsHistory?

I think you are referring at this feature from the website :

after : creating an Order
is display : a notification pop-in

Notification will let you now that an Order as been created.
As magic this notification looks like it seems to be done by :

retrieving pending Orders
every 2 seconds or so

I was not referring to the confirmation popup that the order is created.
I'm referring tot the 2nd popup, that informs that the order is executed against price X.

Do you have a capture of the location of this "2nd popup" ?

Not for now, but the flow on the DeGiro website is quite self-explanatory:

When creating an order, it will be shown in the left table Outstanding, while the confirmation popup "new order created" is shown briefly:

When the target price is met and the order is executed, a confirmation popup "order executed against pice X" is shown briefly, while the order is moved to the right column, where it shows the execution price (since that can be different than the target price in the original order)

Ok got it.
Oustanding : orders
Lastest Transactions : transactions
Just checked how the website is doing.
It still seems to use this endpoint :

trading/secure/v5/update

That you can consume with the method update.

Awesome, I'll try this out.
Just for my understanding, is the get_update() used as a single call with a single answer, or is it more similar to fetch_data() as a a data stream, which will send updates once they occur?

Not exactly : fetch_data uses long polling.
Here this is just refreshing : the website seems to reload the data every 2 secondes.

I have tried this, and I see the recent transactions.
What is the meaning of the  value for parameter last_updated?
I.e. what if set to a value other dan 0?

I think this is how this endpoint avoid sending you the same data twice.

This is how we've implemented it after placing an order:
# IMPORTANT
# The response doesn't show any details whether an order was executed, only that the order is accepted.
# To check whether the order is executed and for what price, 2 events should be started:
#
# 1) Request pending orders via `get_update()` with a list that contains `Update.Option.ORDERS`
#       This returns an array with pending orders, including the order ID. Keep polling while the order ID is present in the ID.
#       Once the order is not present (this can already be at the first request when the order was executed immediately),
#       any of these 2 events have occured:
#           a) The order has been deleted
#           b) The order has been executed
#       To determine whether it's deleted or executed, the Transaction history should be requested:
#
# 2) Request the Transaction History for today via `get_transactions_history()`
#       Unfortunately this array doesn't contain the order ID.
#       So we need to filter on the ticker (`productId`), amount and action for transactions that were created after the confirmation date of the order (as returned by `confirm_order()`).

We even take into account that the order might be executed by multiple transactions.

This is how we've implemented it after placing an order:
# IMPORTANT
# The response doesn't show any details whether an order was executed, only that the order is accepted.
# To check whether the order is executed and for what price, 2 events should be started:
#
# 1) Request pending orders via `get_update()` with a list that contains `Update.Option.ORDERS`
#       This returns an array with pending orders, including the order ID. Keep polling while the order ID is present in the ID.
#       Once the order is not present (this can already be at the first request when the order was executed immediately),
#       any of these 2 events have occured:
#           a) The order has been deleted
#           b) The order has been executed
#       To determine whether it's deleted or executed, the Transaction history should be requested:
#
# 2) Request the Transaction History for today via `get_transactions_history()`
#       Unfortunately this array doesn't contain the order ID.
#       So we need to filter on the ticker (`productId`), amount and action for transactions that were created after the confirmation date of the order (as returned by `confirm_order()`).

We even take into account that the order might be executed by multiple transactions.

Who is « we » ?
I think this can be more straightforward.
If I remember right « get_update » can let you know that the order was executed.
But you need to provide the « last_update ».

@Chavithra Well, "we" is a group of friends who are using your awesome connector into a project. I recently helped you with PR https://github.com/Chavithra/degiro-connector/pull/42 .
There were some issues processing trading_api.get_update(), hence this is what we came up with.
It works for now, just wanted to inform other users about this, as it provides at least an answer to the issue (maybe not the best)

@funnel20 I think I know how to integrate this "order state detection feature".
But I don't have the time for this yet.
If you are up for the challenge we can brainstorm a specification for this together.

@Chavithra I've finally found a way to link the order id and transaction id.
Once I've finished my prototype, I'll post my findings here.

@funnel20 I'm very curious about the way you link order_id and transaction_id. Is there any followup?

@funnel20 I'm very curious about the way you link order_id and transaction_id. Is there any followup?

Unfortunately not, since we have put our project on halt.


Wish a android building

Wish a android building

I am planning to build an abstraction layer to support the sqlite plugin on multiple frameworks such as Cordova and React Native. Unfortunately I cannot promise when.

I will think of something for Android...

android support +1

There is already a possible solution to use the sqlite storage plugin with React Native for Android: react-native-cordova-plugin already supports it for Android once I fix litehelpers/Cordova-sqlite-storage#382.

Android support also added to react-native-sqlite-storage v 2.1.0

Thanks for your efforts to support this!


Question: is it possible to use astroturf without react?

I was under impression that css api simply produces a bunch of class names that can be used with any (or none) js framework. But looks like there is a hard dependency on React. Why is that? Is there a tool similar to astroturf that is not exclusive to react?
Thanks!

When you use css it need only Babel plugin.
Where it depends on React?

Sorry, that was my fault - I forgot to add webpack setup and so I was getting an error "can't find module React"


Added: support for passing builder options and sourceWidths as props

This change allows passing builderOptions and sourceWidths though to the image builder. This means you can modify the image using a simple object.
I updated the storybook as well to show the demo of the changes

If this is accepted, I can update the docs with additional examples

@james2doyle Thanks, this looks great. Really appreciate your contribution. I'll give it a second look in the next few days and make a new release once it's merged.

@BigglesZX Ok I'll wait till you've had a chance to review before I update the docs.
Might be worth tagging a 0.2.0 release given the changes. The additional props are all the same so it will still work with existing code. But the quality is now set to 90 by default which it wasn't set at all before

@james2doyle Looks good to me – thanks again for putting this together. If you're happy to add some additional examples to the docs, I can merge and create a 0.2.0 release after that.

@BigglesZX Made the updates to the source, updated the stories, and added to the docs

@james2doyle Thanks for the extra commits! Spotted those typo fixes in the README as well 😅. I've split up the stories a little more in my local copy to make the demonstration of features a bit clearer/more isolated. I'll merge this now, commit those changes then create a new release.
Thanks again for your contribution – really appreciate it.

@james2doyle<EMAIL_ADDRESS>:)

@BigglesZX Awesome! Thanks for providing this at all. We will be using this on a client project soon. I will share it out when it goes live.
If there are any other tweaks and changes, I can open more PRs


[WIP] IPAMBFAdapter

ref https://github.com/4am-robotics/orga/issues/4970
follow-up of https://github.com/4am-robotics/orga/issues/5368

not needed for the solution drafted in https://github.com/4am-robotics/orga/issues/4970#issuecomment-2093052842


changed if condition

if (participantCount <= 1) {         return {             totalBalance,             myBalance: 0,         }  }

Hi @Sumit-Mundhe!
thanks for taking part in this project!
Could you please place this if block after transactions.forEach? For now, if you are the only active participant, you'll get your balance = 0 and total balance = 0, while total balance should still be the sum of all transactions.
And one more thing: please increment appVersion prop in [SettingsManager](https://github.com/johnSamilin/rosplata/tree/master/src/core)/SettingsManager class?

Sure, I will place it after.
And about the appVersion, do i just need to change it to #appVersion = '0.0.8' right?

i just need to change it to #appVersion = '0.0.8' right?
Absolutely


⚠ Bofrost.lu has degraded performance

In e7f2810, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1238 ms

Resolved: Bofrost.lu performance has improved in 4f3545f.


jump to index

How can I jump to a specific index? I want to keep a reference to the carousel, and when a button is pressed, it should jump to a specific index of the carousel with/without animation.
ex: open page and it starts carousel from specific index (not zero)

You could use the scrollToIndex method, ref.current.scrollToIndex({ index: 1 }),. but I could also add snapToNext and snapToPrev method too


add mongoose and migrations [closes #11]

create add_user table via migrations

hey @jrob8577  - looking for feedback. wrote instructions in migrations.md

@jrob8577 we addressed your changes! thanks.


[Profiler] Fix makefile target

Simple rename.
Test Plan: make

Thanks! :)
@facebook-github-bot import

@tadeuzagallo btw do you have a trick to visualize the data like loading it into Web Inspector?

We are working on the tool for that internally, and making the profiling workflow smoother. I'll make it available here as soon as everything is working nice together.

Cool, that will be helpful for speeding up app code =)


Add ProxyConfig to list of non-stable resources

Please provide a description of this PR:
Adds ProxyConfig to the validating admission policy so that the ProxyConfig is unsupported when using the stable admission policy.
#50734

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@jaellio: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
7ffde7afb3a2a8ea45c634ded94d9b52837e93c8
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

In response to a cherrypick label: new pull request created: #50742


Configure workflow for auto releasing

Describe your feature request
Need 2 workflows:

For pull requests on main branch - build, test and upload artifacts
For push on main branch - build, test and push to nuget

Example
How important is this to you?
Important
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

I'll take care of this as soon as I'm available.

I think it is better to manually publish nuget because it can't be deleted after published. So it should be very carefully. It's better to publish to GitHub nuget.

I think it is better to manually publish nuget because it can't be deleted after published. So it should be very carefully. It's better to publish to GitHub nuget or just upload artifact.

Ok

    flowchart LR
        WindowsTest(build-and-test-windows-latest) --> IsTestSuccess{Is success?}
        UbuntuTest(build-and-test-ubuntu-latest) --> IsTestSuccess{Is success?}
        MacOSTest(build-and-test-macos-latest) --> IsTestSuccess{Is success?}
        IsTestSuccess -->|True| PublishNuget(build-and-publish)


fix: elastic search delayed response

Will likely remove trade_remedies_caseworker/templates/static/javascripts/vendor/ui/jquery-ui.min.css form PR when we understand why UAT does not work


Dropdowns not selectable with mouse

I updated to 1.1.1 (zfcampus/zf-apigility-admin 1.1.2 and zfcampus/zf-apigility-admin-ui 1.1.4 and the dropdowns in the UI appear to not work when the mouse is used to select an option. I think I can choose using the keyboard to type and hit enter to select but if the mouse is used to select something the keyboard doesn't work and the mouse doesn't appear to be able to actually make the selection and close the select box. It seems to be happening in every select box: content negotiation, filter and validator options, hydrator selection.
The only selection box that doesn't appear to be affected is the source code viewing one.

In the screenshot above, I can move the mouse over any of the options and it highlights, but clicking does nothing. Clicking outside the drop down will close the drop down but the selection remains whatever it was when I opened it.
I can use the selectors with the keyboard (arrow keys and enter or typing the selection or both).

@dstockto I cannot reproduce this issue, it's working fine now. I tested with Chrome, Firefox and Safari.

It does seem to be working correctly in the latest version. Thank you.


new additions : plugin interfaces, plugin service, urlfilter, regex url filter, config files....

Browse the commit log for the full details.
@karanjeets please review and merge. Let me know if you have questions!

please merge this it's been sitting for 12 days.

@thammegowda - Looks good to me. Merging it to the Master.

thanks @karanjeets @thammegowda


When user is not logged in

When user isn't logged in, it shows edit button on all users.

Hm i dont get button when im not logged :dancer:

I forgot I fixed this issue already.


Mi envoltura rdp funciona "totalmente compatible", pero no me permite hacer una conexión simultánea

My rdp Wrapper is working "fully supported", but it won't let me make simultaneous connection

s
Originally posted by @scrcrivera in https://github.com/stascorp/rdpwrap/issues/1823#issuecomment-1125720556

Me too,  Windows 11 OS of version 10.0.22621.1

Dont allow access Two o more user in the same Time.

Applicable to https://github.com/stascorp/rdpwrap/issues/1883

I have the same problem.  Will not allow more than one user to log in at a time.  Scrcrivera & just4id & lusinqui, were you able to get it working with anything in #1883 ?  I wasn't.  If so , can you explain exactly what fixed it?  If not then this should also be reopened along with mine.

should also be reopened

Just one issue for the same problem can be opened.
By rule - the first one. So i will search the first one, and will close all other with link to the first one


Deploy static tileserver to s3

GitHub CI is capable of deploying the static tileserver files to OSMUS's s3 bucket.  We can follow the approach that's already used in Americana.

Is there an advantage to this instead of using GitHub pages for static content?

If we can configure a domain name and there's no CORS constraints on GitHub pages, that should work fine.  The only thing I'm not sure how to do outside of s3 is the RSS feed which currently pushes to the bucket.

Okay, I'm fine using s3 instead of reinventing this.

Also -- preview.ourmap.us (Americana PR preview functionality) is also based on the use of the s3 bucket.


Improve tutorial for testing

Some things that come to mind might be evident for most of you, but not for 100% of validators:

specify if there are any hardware requirements.
specify if mev-boost must be installed on the consensus machine or the execution machine, both, or maybe whether geth+lighthouse must be on the same machine
specify for which consensus client it works at the beginning of the tutorial... I spent considerate amount of time to find in the end it doesn't currently support lighthouse ;)
specify that you'd need 32 test ETH, meaning that on the faucet links you provide it will take 24 days to gather them for users starting from zero...  (hopefully the merge already happened by then)

Hello @adv0r. Thanks for reporting this.

specify if there are any hardware requirements.

They are in the validator checklist. And then every development team should have the requirements on their docs. We can start measuring the specifics of mev-boost to also document the requirements. #201

specify if mev-boost must be installed on the consensus machine or the execution machine, both, or maybe whether geth+lighthouse must be on the same machine

It can be installed anywhere, as long as is reachable by the consensus client. We can mention this in our docs, somewhere :thinking: #202

specify for which consensus client it works at the beginning of the tutorial... I spent considerate amount of time to find in the end it doesn't currently support lighthouse ;)

done

specify that you'd need 32 test ETH, meaning that on the faucet links you provide it will take 24 days to gather them for users starting from zero... (hopefully the merge already happened by then)

I've linked the guide to #199.


add add/remove commands to the api

?add and ?remove, these are to be acompinied with ?key which means you can only add or remove them if you have the correct key.

Or just convert whatever you have now to ReSTful service?
http://en.wikipedia.org/wiki/Representational_state_transfer
http://www.restapitutorial.com/lessons/whatisrest.html

I think I'm already using it. I'm using get, post. Wasn't aware of put or delete. But if we use that you will have to find out how to send a put / delete request


[tests] Enable BookKeeper NativeIO feature in order to make the tests pass on JDK17

Motivation
Running a test that require a bookie to be started fails with JDK17, due to the BookKeeper Native IO logic. For example you can run the PulsarSourceE2ETest test to see it crash.
https://github.com/apache/bookkeeper/blob/4debbbf84db09f80ffeb4154e8e6feed134be0c0/bookkeeper-server/src/main/java/org/apache/bookkeeper/util/NativeIO.java#L59-L88
BookKeeper expects to be able to access to FileDescriptor and it will throw an assertion error if it something goes wrong. To make this possible w/ JDK17 we need to pass the Java runtime option to open the java.io package to be accessed.
Surefire add the -ea flag by default but the production process is not affected since we don't set the flag on Pulsar scripts.
An alternative could be to skip Java assertions error in the tests but I prefer to open the package as I see it as a more conservative choice.
Modifications

Added --add-opens java.base/java.io=ALL-UNNAMED to surefire options (only for build w/ jdk11+)

[x] no-need-doc

/pulsarbot rerun-failure-checks


MF-1403 - Change vernemq building source revision

Signed-off-by: Ivan Milosevic<EMAIL_ADDRESS>Since building vernemq docker image is broken on latest vernemq release, this fixes it by switching to revision where this issue is addressed.

@blokovi Can you please open an issue referring to this  building vernemq docker image is broken on latest vernemq release,
This PR is just a temporary fix, we need to investigate why it breaks and fix the build toolchain.

@blokovi Can you please open an issue referring to this  building vernemq docker image is broken on latest vernemq release,
This PR is just a temporary fix which we will merge ASAP, but We need to investigate why it breaks and fix the build toolchain.

@nmarcetic there is an issue:
https://github.com/vernemq/vernemq/issues/1767

@blokovi Can you please open an issue referring to this  building vernemq docker image is broken on latest vernemq release,
This PR is just a temporary fix which we will merge ASAP, but We need to investigate why it breaks and fix the build toolchain.

@nmarcetic there is an issue:
vernemq/vernemq#1767

@blokovi Great, I was thinking on our repo :) As a reminder and follow-up the reference. When you close this PR we will not be able to track this issue with vernemq.

Sorry

@blokovi Can you please open an issue referring to this  building vernemq docker image is broken on latest vernemq release,
This PR is just a temporary fix which we will merge ASAP, but We need to investigate why it breaks and fix the build toolchain.

@nmarcetic there is an issue:
vernemq/vernemq#1767

@blokovi Great, I was thinking on our repo :) As a reminder and follow-up the reference. When you close this PR we will not be able to track this issue with vernemq.

Sorry @nmarcetic, I didn't understand at first. Issue #1403 that should NOT be resolved with this PR.


Bug on custom dataset

Describe the bug
A clear and concise description of what the bug is.
I construct a dataset with size 50000*6(feat_dim)*48(context length), finetune meeting the following error:
To Reproduce
Please provide a code snippet of a minimal reproducible example for the error.
/root/miniconda3/envs/llm_ts/bin/python3 -m cli.train -cp conf/finetune run_name=demo_run model=moirai_1.0_R_small data=feed_train val_data=feed_eval

Expected behavior
A clear and concise description of what you expected to happen.
Error message or code output
Paste the complete error message, including stack trace, or the undesired output that the above snippet produces.
AssertionError: no valid patch size candidates for time series shape: (6, 48), freq: H, patch_sizes: [8, 16, 32, 64, 128], constraints: range(32, 33), min_time_patches: 2, patch_size_ceil: 24

Environment

Operating system:
Python version:
PyTorch version:
uni2ts version (have you pulled the latest version?)

The length of each training sample is required to be longer than min_time_patches (2). Here the patch size range is (32, 33), which indicates the training sample should be longer than 64. Your data (48 length) does not satisfy this constraints.
One quick fix is to modify the DEFAULT_RANGES in patch.py to "H": (1, 64),

Describe the bug A clear and concise description of what the bug is. I construct a dataset with size 50000*6(feat_dim)*48(context length), finetune meeting the following error: To Reproduce Please provide a code snippet of a minimal reproducible example for the error.
/root/miniconda3/envs/llm_ts/bin/python3 -m cli.train -cp conf/finetune run_name=demo_run model=moirai_1.0_R_small data=feed_train val_data=feed_eval

Expected behavior A clear and concise description of what you expected to happen.
Error message or code output Paste the complete error message, including stack trace, or the undesired output that the above snippet produces.
AssertionError: no valid patch size candidates for time series shape: (6, 48), freq: H, patch_sizes: [8, 16, 32, 64, 128], constraints: range(32, 33), min_time_patches: 2, patch_size_ceil: 24

Environment

Operating system:
Python version:
PyTorch version:
uni2ts version (have you pulled the latest version?)

patch.py to "H": (1, 64),

Have you solved this problem? I modified "H": (1, 64), "T": (1, 128) in patch.py, and the same error message appears.

Hi @qingyuanxingsi do you have any further questions? If not, i will close this issue


Load a donor's cases from a specific endpoint

The /donors/:id doesn't contain any child donor cases, making it impossible to create warnings for them.
/persons/:id/cases contains all of the parent and children's cases.
Fixes VEGA-1926 #patch

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.05 :tada:

Comparison is base (713c0c8) 72.94% compared to head (181e89f) 73.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #353      +/-   ##
==========================================
+ Coverage   72.94%   73.00%   +0.05%
==========================================
  Files          94       95       +1
  Lines        3711     3719       +8
==========================================
+ Hits         2707     2715       +8
  Misses        719      719
  Partials      285      285

Impacted Files
Coverage Δ

internal/server/warning.go
82.00% <100.00%> (ø)

internal/sirius/cases_by_donor.go
100.00% <100.00%> (ø)

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


Implement write batch size

Hello,
We would like to add tests to this new functionality, but we aren't sure how to to do so (as we haven't found unit tests in the relevant modules). Where should we do so? Thanks in advance.

Hello @chrisekelley ,
We have applied some of your changes into this PR. We added a test as well.
We have some questions:

Would you like to be credited as contributor of the commits? After all it's your code
Please note that we have renamed the property countIndexedDocs to indexed_docs in order to follow PouchDB's convention. This means that we need to apply changes to Tangerine, as we have found that is used here.

Thank you

@AlbaHerrerias Thanks a lot - that would be great if you'd credit me.
Also thanks for the head's up about the indexed_docs change. I'll keep that in-mind when we migrate to that version.

Hi again @chrisekelley,
We've created another PR with some of these changes here. We have added you as contributor in the last PR as well as this one (will keep doing it so).
Please note that we have renamed some variables:

startNextBatch to start_next_batch
pendingBatch  to pending_batch
They are used in Tangerine at least in this file.

Thank you


Add the ability to do Integration tests in projects easily

I left a comment on the pull request I left open, but I suggest "behave" framework. I wrote a very basic test, and I'm planning on adding more. You can check out the branch "hack-hack" on my fork of your code to see this. Once I'm happy with the amount of test coverage, I'll merge "hack-hack" back into "hack", and then pull request "hack" onto your branch "dev"

Ah, so this one was a fairly vague enhancement, that means more that projects that utilise simqle in them can create integration tests really easily. It definitely needs more canvassing - I had a few ideas but nothing down on paper yet

This Issue has turned into a whole new project... https://github.com/Harlekuin/sqema, so I'll close it here


Port adjacent difference into CUB

The PR contains:

a port of thrust::adjacent_difference algorithm;
deprecation of FlagHeads and FlagTails methods from BlockAdjacentDifference structure;
fixed API for BlockAdjacentDifference along with documentation and tests.

Note that the PR is based on some of the features introduced in MergeSort porting.

gpuCI: NVIDIA/thrust/pull/1577
DVS: 30766806


paloalto platform

hopefully will rip this out into a unified python and go community repo but for now... here it is!


Simplify Change.raw_str()

what's the reason for this change?

Less code to do the same thing? name does automatically what was done manually.

🤦 sorry, I read the PR the wrong way around. Makes complete sense.

thanks so much.


[REQUEST] View + Controllers Separation

Is your feature request related to a problem? Please describe.
Next.js is useful for using both server and client components, however, most of our components are client components. We are not taking full advantage of the server components. Furthermore, we fetch a lot of data in client components that may be better performed in server components and passed to child, client components.
Describe the solution you'd like
I believe that we can separate our components into multiple layers (controller/logic/view), allowing us to take advantage of the benefits of server vs. client components.
The most obvious spot to look is our forms, which we have lots of.
The earlier we catch (and fix) this the better, as we can standardize it and remove tech debt later on.
Describe alternatives you've considered
Not separating these layers and operating more like a React app. Why use Next.js if we aren't going to use server components like we should?
Additional context
N/A

Labeled as 'configuration' and 'ui'. Although it doesn't directly affect the UI, this is definitely not a major nor minor feature, so we'll call it just a UI change.


PIA

Describe the issue
A clear and concise description of what you want to happen.
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context
Add any other context, attachments or screenshots
Definition of done
Identify what will need to happen/be delivered for this to be completely done

This seems like an epic - should be broken down into smaller tasks

this can be deleted - broken into smaller chunks


[stable/logdna-agent] Support the newest 2.1.9 image + improve chart

This commit updates and adds yamls to support the latest image of the logdna-agent. This commit also includes some changes to make this helm chart more configurable and to better follow the standards set forth by the Helm community.
Could I get the existing maintainer, @leeliu, to take a look as well.
Signed-off-by: Jacob Hull<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Continued support of the latest agent image.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @mattfarina

/ok-to-test

/test pull-charts-e2e

/cc @mattfarina @viglesiasce @prydonius

@leeliu can you take a look at these changes?
@jakedipity have you considered having the logdna chart being hosted and developed out of a logdna GitHub repo? If you do that you can get it listed in the Helm Hub and the Artifact Hub for discoverability. You can host it all on GitHub and we even have GitHub actions to help.
If y'all host it you don't need to wait on us for approvals.

@mattfarina Thanks for the suggestion, we'll look into that. If we hosted our own repository would it make sense to also publish this chart in the stable repository here? We want to make sure we have maximum exposure.
Also, if we did go that route, would it be okay to use a git submodule to avoid duplication?

@jakedipity The stable repository is going to be deprecated and turned off later this year. If you keep it in both places it would be a short term thing. Helm 3 does not ship with the stable repository configured in it but it does enable searching the Helm Hub.
We don't accept submodules but we are ok with bots that apply the changes from an upstream repo to the charts repo. Bitnami has long done that.
Once you do move to a new location you can follow the deprecation process and point people to the new location. The wordpress chart in the stable repository is an example of that.

/lgtm


Update glide deps

This PR explicitly tags the influx client to match the server version in our image (0.13), and updates the remaining dependencies to the latest versions.

@subfuzion influxdb 1.0 is not far (planned for summer), we can already start to work on the RC. I'll create a branch but I'll leave master (and latest) on 0.13.

@ndegory Yes, I was actually testing with v1.0.0-beta3. The main thing we have to ensure is to update the entire TICK stack at the same time to maintain version alignment, and then we have to update our glide files (for the client package dependencies).

LGTM
for verification:
make install-deps
make test

:+1:


Synchronize with nearest camera position

Hey, love this viewer! One thing that I think would definitely help is to add a wdiget that gives a side-by-side view of the nearest camera image as you zoom and rotate around the splat, and the ability to jump the viewer into that exact camera position as well for a direct comparison.

I agree, this would be awesome. I will keep this in mind when developing new features. But feel free to add this in a PR, if you have the time


[Feature Request] Add View Info to Lora Loaders from other Custom Nodes.

This might not be possible without collaboration, but it'd be swell if the View Info feature could be added to other Lora Loaders.
Alternatively, perhaps allow an input that takes any input, allow selection of a widget that node and use that for viewing info.
Currently, I simply use a detached default Lora Loader for a Lora Checker, and Efficiency Node's Lora Stacker.

I'm currently using rgthree's Power Prompt node for this, but that means I'm still missing the view info button (which could work exactly the same as for the embeddings)

You can use this
LoraLoader.lora_name,LoraLoader|pysssss,LoraLoaderModelOnly.lora_name,LoRA Stacker.lora_name.*
In the setting called 🐍 Model Info - Lora Nodes/Widgets
This should add support for the stacker, and you can add support for other nodes in the same manner.


teleport issue

I am running into an issue that when I use the /tp feature, I am being teleported to right area, however I am under the map stuck in water unable to move.

I stand corrected, it has been fixed!


How to get Tello video feed from camera?

Sorry for stupid question, but it will be great if you can help me.
Right now I have been successfully Tilt and Land the Tello drone with your library. All controls (W: Up, -> right etc.) works but video stream is off.
How can i turn it on?

@comoc I looked in scripts components (Tello Controller, Tello Video Texture) but didn't find anything that could help turn on Video. Maybe I'm missing something?

Hi,
I haven't touched this repo recently, but I think it's a Firewall issue that's preventing me from receiving video.
Please see here.
https://github.com/comoc/TelloForUnity#windows-specific-settings

@Wiserax Did you find a solution?I also met the same problem

@Wiserax Did you find a solution?I also met the same problem

I have the same problem, do you find a solution or a workaround?
Greetings

I changed wi-fi routers and now video stream is off. I would appreciate if you could help me with a solution.


More consistent way for release process

Hey, I am working with this lib everyday. Idea behind it is awesome and I appreciate all of work especially in few last weeks. But I am missing more frequent release notes to catch the moment when and what kind of bugsfixes/changes was made. Can I ask for update on this? I promise to donate few bucks for coffee :)

I'm gonna update the changelog markdown file and draft a release.

I've got to agree, I noticed there hasn't been many release notes the past version, and when I went to upgrade I noticed the entire styling for my sliders broke. At a minimum, tagging releases would be greatly appreciated!


[5.1] Safeguard User password is encrypted before sending to database

In several places Laravel assumes that a password is either encrypted or unencrypted (e.g. ResetPasswords, Guard, ...). This works great if one uses only the built-in user management tools Laravel provides (such as authentication, password reset, user creation).
In case a developer wants to code tailored versions of a user registration or update routine, the developer currently has to hash the received password explicitly using bcrypt() before sending it to the User model. Otherwise plain passwords are being stored in the database and some functions will start to fail.
This tiny PR overcomes this behavior by introducing a mutator for the password attribute. It sort of abuses Hash::needsRehash to determine if the passed value is a plain string and then hashes the value if it is not a hash.
This way it is assured that any password value that is being passed to the User model gets inserted into the database as a hash.
So both
User::create(['password' => 'myValue'])
User::create(['password' => bcrypt('myValue')])
will yield the same hash being inserted into the database.

All commit comments are fixed.

CS fixed

We don't want any magic behavior around passwords.


Unexpected token when trying to use phaser 3 plugin

I am getting the following error when importing the plugin:
ERROR in ./node_modules/phaser-navmesh/dist/phaser-navmesh-plugin.js 1:4581
Module parse failed: Unexpected token (1:4581)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders

Using typescript and webpack. Hope you can help!

Hi @craigharman - hmm, could you share a link to your project (or a stripped down version)?

Thanks for your response mike (and your great plugin). I started building a Phaser game from scratch to provide you with an example and it worked, so I upgraded node to latest 14.x and it seems to work now.

Woohoo! Glad you got it working


Windows File Follower: service wont' exit it it cannot connect to ingester

What are the steps to reproduce this issue?

Deploy windows file follower
Ensure the config is valid but cannot talk to indexer
try to restart or stop the service

What happens?
it hangs
What were you expecting to happen?
not hang
Any logs, error output, etc?
nope
Any other comments?
i hate windows
What versions of software are you using?
5.1.2-alpha

validated


Add RemoteLoggingLevel to MeshConfig

Description:
Add RemoteLoggingLevel to MeshConfig

Testing done:
Done

Affected area:

Functional Area

New Functionality
[ ]

CI System
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Demo
[ ]

Documentation
[ ]

Egress
[ ]

Ingress
[ ]

Install
[ ]

Networking
[ ]

Observability
[ ]

Performance
[ ]

SMI Policy
[ ]

Security
[ ]

Sidecar Injection
[ ]

Tests
[ ]

Upgrade
[ ]

Other
[X]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project? no

Is this a breaking change? no

Has documentation corresponding to this change been updated in the fsm-docs repo (if applicable)? no

Codecov Report
Patch coverage has no change and project coverage change: -0.06% :warning:

Comparison is base (b8d1dce) 52.67% compared to head (3356eef) 52.61%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the GitHub App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #49      +/-   ##
==========================================
- Coverage   52.67%   52.61%   -0.06%
==========================================
  Files         175      175
  Lines       12819    12829      +10
==========================================
- Hits         6752     6750       -2
- Misses       6036     6048      +12
  Partials       31       31

Flag
Coverage Δ

unittests
52.61% <0.00%> (-0.06%)
:arrow_down:

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

Files Changed
Coverage Δ

pkg/configurator/methods.go
24.21% <0.00%> (-0.13%)
:arrow_down:

pkg/configurator/mock_client_generated.go
0.00% <0.00%> (ø)

... and 1 file with indirect coverage changes

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


Inclusivity & Accessibility - Section 508 compliance validation

In the US, there is a fairly extensive list of technical accommodations for extending inclusiveness in Section 508 - https://www.access-board.gov/guidelines-and-standards/communications-and-it/about-the-section-508-standards/section-508-standards
In context, it may be worth reviewing how the front-ends of CHAOSS products (Dashboards) comply with these standards so we're walking the talking:)

:D that's a great idea.
Tagging @sduenas and @sgoggins as respective leads for GrimoireLab and Augur.
Maybe create a separate issue for each software project?

Agree that this is a 'bucket of work' and should have multiple issues opened underneath it.   @mpgjon are you interested in in organizing this work into actionable pieces?

Sure can:)
On Fri, Jul 12, 2019 at 12:28 PM Georg J.P. Link<EMAIL_ADDRESS>wrote:

:D that's a great idea.
Tagging @sduenas https://github.com/sduenas and @sgoggins
https://github.com/sgoggins as respective leads for GrimoireLab and
Augur.
Maybe create a separate issue for each software project?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chaoss/wg-diversity-inclusion/issues/192?email_source=notifications&email_token=AETQEH3GHRG5CLTQM7Z4PZLP7DLNRA5CNFSM4ICNF3EKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZ2VI6Q#issuecomment-511005818,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AETQEH2Z526V6DHUISKQF4TP7DLNRANCNFSM4ICNF3EA
.

I submitted two issues to start. I was thinking about submitting a third for supporting blind users, but at this time, I don't think there's any technical way to do ALT descriptions on dynamic graphs (dashboards).  Would love to hear about anything to the contrary, but I think that one may be a much longer term goal.

@mpgjon : I think the website for CHAOSS is addressing its own self internally.  There is also a released metric right now that includes document inclusivity. I think this an e closed.

Closing this; can re-open if needed.


Blazor Server memory analyzer tool

Check mid-'23 per https://github.com/dotnet/AspNetCore.Docs/issues/27768#issuecomment-1329554214 ...

In addition to that, for future reference, there is an experimental tool that I built. dotnet blazor-server-memory-analyzer that analyzes the memory of a running Blazor Server app or memory dump and display useful report information, like the number of circuits in memory, the number of components on each circuit, and for each circuit, an approximate of the memory the circuits use.

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

ID: 9b20b470-161a-8723-b66d-c0c1804874f0
Version Independent ID: 2712f7a2-90e7-0012-f7d9-f3a7e7e345d7
Content: Host and deploy ASP.NET Core Blazor Server
Content Source: aspnetcore/blazor/host-and-deploy/server.md
Product: aspnet-core
Technology: aspnetcore-hostdeploy
GitHub Login: @guardrex
Microsoft Alias: riande

Closing as not supported (experimental) at this time.


Add list of contributors to changelog/release notes

It would be really nice to see the list of contributors for a new version, or at least the contributor of each change in the list (like GitHub automated release notes does).
We could wait for release-please to implement this natively, or we try adding our own action for thet (see e.g. https://github.com/googleapis/release-please/issues/292).

+1


DS-2750 REST API ignores expand=all on communities

https://jira.duraspace.org/browse/DS-2750

+1 Looks good to me. (Haven't run though).

:+1:  Looks like an obvious fix to me. Probably should just be merged

If you REALLY wanted to propogate the expand, you would have to make it so that we don't construct a sub-collection, or subcommunity with a null expand.
https://github.com/DSpace/DSpace/blob/master/dspace-rest/src/main/java/org/dspace/rest/common/Community.java#L87
https://github.com/DSpace/DSpace/blob/master/dspace-rest/src/main/java/org/dspace/rest/common/Community.java#L102
..and others
from
subcommunities.add(new Community(subCommunity, null, context));
to
subcommunities.add(new Community(subCommunity, expand, context));
You would have to ensure you don't expand parents, otherwise that will definitely cause infinite loops.
So. I'm -1 on this PR as is. and about plus-or-minus-1 on the concept of sudo-expand


[ISSUE-3887]linkis-daemon.sh can dynamically specify the debug port and dynamically specify the jvm memory

What is the purpose of the change
The linkis script supports sbin/linkis support to specify the debug port at startup linkis-daemon.sh restart/start [service-name] [debug-port] , and each microservice can specify the JVM memory separately
Related issues/PRs
Related issues: #3887
Related pr:#3891
Brief change log
linkis-dist
Checklist

[x] I have read the Contributing Guidelines on pull requests.
[ ] I have explained the need for this PR and the problem it solves
[ ] I have explained the changes or the new features added to this PR
[ ] I have added tests corresponding to this change
[ ] I have updated the documentation to reflect this change
[ ] I have verified that this change is backward compatible (If not, please discuss on the Linkis mailing list first)
[ ] If this is a code change: I have written unit tests to fully verify the new behavior.

@aiceflower
@jacktao007  help review


release 5.6.13-23 was not tagged or released

There is no release git tag for v5.6.13-23.  There is one for the previous 5.6 release v5.6.12-23.  There is also no v5.6.13-23 at https://github.com/floragunncom/search-guard-ssl/releases
You appear to have pushed a release v5.6.13-23 to search.maven.org without tagging/releasing at https://github.com/floragunncom/search-guard-ssl/releases

tags pushed


Return 408 http code for gunicorn timeouts

I was mistaken, I was reading documentation and it looks like 408 should only be returned when the client has problems transmitting data, so we shouldn't do this: https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/408


chore: cleanup

Clean up indicator code since direct require is supported now

Might want to move indicator.js out of the indicator folder or rename it to something like _indicator.js so it is obviously not a part of the demo

Thanks for this but the extra indicator.js/_indicator.js file would make this demo differ from the JVM-based and Python demos. As I add new features I'll be updating all the demos at the same time so I want to make sure they differ as little as possible. I'm basically just copy-pasting and fixing syntax whenever I update them.


Remove trailing white space

I was getting a bunch of noise with git and in my editor with trailing white space, so I just stripped it out from the source code.
This is just a suggestion, so don't feel obligated to merge. But thought it was a nice clean-up task. If you want to keep the white space just say so. I can figure out how to get my editor to play nice.

Merge conflicts with this and #4, I am gonna close this and follow up with a more robust change.


Add a Kotlin Standalone Generator

It looks like the difference between spaces vs tabs - try enable invisible characters

Whats the preferred for Swift? Spaces or tabs?

For now, I would recommend tabs for this project. Please feel free to reconsider. The reason is that we are likely to use string literal small test lexicons and if you IDE is converting tabs to spaces, it's not so much fun... However, we could also develop a dsl so that we don't need string literals. Of course we can (and often should) load test lexicon files, but this is not helping readability of unit tests so much...

am just hitting Ctrl+I and thats how it formats it. Can I swap XCode to use tabs?

@ghosalmartin yes, you can make the whole of Xcode use tabs by going to:

Xcode > Preferences > Text Editing > Indentation

or if you want to just do it for this project, you can use the File Inspector on the right  under "Text Settings"

All tabbed up, thanks Gents.


Adjust required permissions for system schema

This PR implements the following change described in #7563:

The permissions needed for system table access could be adjusted to not require additional permissions beyond datasource-specific permissions for non-server related info (segments, tasks) and to require STATE-read permissions for server-related tables (servers, server_segments) to be consistent with the non-SQL APIs.

A STATE.READ permission check is added to server_segments sys table handling (servers already had one, and #7567 removed the unnecessary sys datasource check).
This PR also adds integration tests for sys schema + auth. A new segment and task for auth_test datasource are inserted directly into metadata during IT setup, and this datasource is used for testing the auth checks.

will need to adjust tests to handle when the full IT test suite is run (since cluster state is not isolated across tests, things like "total stored segment size on historicals" change)

I updated the SQL docs with info on necessary permissions, also added tests for a user without access to the auth_test datasource

LGTM other than the comment about doc wording.

Fixed tests:

Updated to handle changes from is_overshadowed patch
Changed the created year for the inserted task to 2030 so it doesn't fall out of the completed tasks window


Fix/scrolling

scroll fixed to keep chat at the bottom only if user at the bottom of the chat.
Some css added.

Great!


jsdoc-loader issues

Hi !
Thanks for this great boilerplate :)
I'm trying to adapt it to mine and am getting the following error on compiling :
Syntax Error: Error: SyntaxError: /Users/marine/Documents/MAKESENSE/makesense_design-system/src/system/components/typography/Text/Text.vue: Expecting Unicode escape sequence \uXXXX (24:9)

  22 |  */
  23 | export default {
> 24 |   name: \"DsText\",
    |          ^
  25 |   provide() {
  26 |     return {
  27 |       $parentText: this

It seems to be related to the jsdoc-loader file located in src/loader.
Im using version "2.6.12" of the vue-docgen-api, and the package.json says ^2.3.13`
Do you know where that could come from?

It seems that adding .replace(/\\"/g, '\"') to the loader file does the trick. 🤷♀


Could you please provide some of the source videos from the project page?

We would like to test the source videos featured on the project page, but they are not included in the package folder. Could you provide them or share a link? Thank you! @yangchris11

@wf-hahaha The demo video from the movie 1917 is from: https://www.youtube.com/watch?v=cwUzUzpG8aM&t=4s. We also plan to update our project page shortly with all the reference and source.
All rights are reserved to the copyright owners (TM & © Universal (2019)). This clip is not intended for commercial use and is solely for academic demonstration in a research paper.


community-operators [R] reportportal-operator (0.0.5 0.0.6 0.0.7 0.0.8)

Update reportportal operator to 0.0.8

Signed-off-by: Wayne Sun<EMAIL_ADDRESS>Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[x] Are you familiar with our contribution guidelines?
[x] Have you packaged and deployed your Operator for Operator Framework?

[x] Is your submission signed?
[x] Is operator icon set?

Updates to existing Operators

[ ] Did you create a ci.yaml file according to the update instructions?
[ ] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[x] Modify more than one operator
[x] Modify an Operator you don't own
[x] Rename an operator - please remove and add with a different name instead
[x] Submit operators to both upstream-community-operators and community-operators at once
[x] Modify any files outside the above mentioned folders
[x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x] Description about the managed Application and where to find more information
[x] Features and capabilities of your Operator and how to use it
[x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[x] Human readable name and 1-liner description about your Operator
[x] Valid category name1
[x] One of the pre-defined capability levels2
[x] Links to the maintainer, source code and documentation
[x] Example templates for all Custom Resource Definitions intended to be used
[x] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

/merge possible

@waynesun09 maybe you can consider following and next time you will add new version it might be merged automatically

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

@waynesun09 maybe you can consider following and next time you will add new version it might be merged automatically

Current PR can be merged automatically, but there is missing authorized-changes label. One can find out more info here.

Sure, I'll add that.

/merge possible

/test all

/merge possible

/merge possible


The getter 'colorTag' was called on null (🔴)

The app wont start anymore due to a null error.
The following NoSuchMethodError was thrown building Consumer<ThemeProvider>(dirty, dependencies: [_InheritedProviderScope<ThemeProvider>]):
The getter 'colorTag' was called on null.
Receiver: null
Tried calling: colorTag

The relevant error-causing widget was:
  Consumer<ThemeProvider> file:///D:/111Programming/Flutter/Projects/smooth_app/master/smooth-app/packages/smooth_app/lib/main.dart:87:20
When the exception was thrown, this was the stack:
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:54:5)
#1      _MyAppState.build.<anonymous closure>.<anonymous closure> (package:smooth_app/main.dart:99:36)
#2      Consumer.buildWithChild (package:provider/src/consumer.dart:180:19)
#3      SingleChildStatelessWidget.build (package:nested/nested.dart:259:41)
#4      StatelessElement.build (package:flutter/src/widgets/framework.dart:4569:28)
...

It has to do with the multiple color schemes, and was probably caused by the upgrade from provider. @monsieurtanuki could you have a look on it?

Working on it...


Can't get to work with ollama

I am getting the follwoing when hitting tab in insert mode:
Error executing vim.schedule lua callback: ...cal/share/nvim/lazy/llm.nvim/lua/llm/language_server.lua:154: attempt to index local 'completion_result' (a nil
value)
This is my config
  {
    'huggingface/llm.nvim',
    config = function()
      require('llm').setup({
        model = "starcoder2:7b",
        backend = "ollama",
        url = "http://localhost:11434/api/generate",
        -- cf https://github.com/ollama/ollama/blob/main/docs/api.md#parameters
        request_body = {
          -- Modelfile options for the model you use
          options = {
            temperature = 0.2,
            top_p = 0.95,
          }
        }
      })
    end
  },

Additionally, the command LLMSuggestion does nothing and auto suggestion doesnt seem to do anythingeither . ollama is installed, including the specified model.
Other models dont work either. Using the same model via ollama run does work. It also seems to be spawning the respective ollama serve processes.
Any idea what could be going on?

@Bios-Marcel you need to specify the lsp.
If you're using mason:
{
  -- ...
  lsp = {
    bin_path = vim.api.nvim_call_function("stdpath", { "data" }) .. "/mason/bin/llm-ls",
  },
  -- ...
}

While it solves the error. I still don't get completions and the LLMSuggestion command does nothing. Am i missing something?

I can't seem to get starcoder2 to work either.
Try starcoder to verify your config.
return {
	{
		"huggingface/llm.nvim",
		opts = {
			enable_suggestions_on_files = {
				"*.*",
			},
			backend = "ollama",
			model = "starcoder",
			url = "http://<IP_ADDRESS>:11434/api/generate",
			lsp = {
				bin_path = vim.api.nvim_call_function("stdpath", { "data" }) .. "/mason/bin/llm-ls",
			},
			tokenizer = {
				repository = "bigcode/starcoder",
			},
		},
	},
}

The following config works well for me:
  {
    "huggingface/llm.nvim",
    opts = {
      backend = "ollama",
      model = "codellama:7b",
      accept_keymap = "<S-CR>",
      dismiss_keymap = "<CR>",
      url = "http://localhost:11434/api/generate",
      request_body = {
        options = {
          temperature = 0.2,
          top_p = 0.95,
        },
      },
      enable_suggestions_on_startup = false,
      lsp = {
        bin_path = vim.api.nvim_call_function("stdpath", { "data" }) .. "/mason/bin/llm-ls",
      },
    },
  },

While it solves the error. I still don't get completions and the LLMSuggestion command does nothing. Am i missing something?

@Bios-Marcel I think I know the problem, you should just wait longer. Try manually running ollama server in a separate terminal: ollama serve. When you trigger LLMSuggestion ollama should output something. Make sure to be in insert mode on next line after comment/code. I suspect you just didn't wait enough for it to generate and suggest something.
Tbh I personally don't find such an LSP useful unless you have good enough hardware (which I don't), because it takes too long to generate a response, and smaller code models (which are faster) may produce worse results or hallucinate a lot.
If you are also limited by hardware like me I recommend you take a look at https://github.com/David-Kunz/gen.nvim . It has a pretty similar development workflow, but instead of LSP like experience, you can select code that you want to ask about which reduces load time. You can also see the response load in real time.

@life00 appears to be correct in that running models locally for autocompletion requires a beefy setup, generating completions can be slow.


Build Provider Skeleton Service

Description
As a developer, I want to implement the build system against a build provider mock service.
Functional Acceptance Criteria

There is a build provider implementation as a service available.
The build provider service implements the Build Provider API.
The build provider service returns mock responses for requests.

Non-functional Acceptance Criteria

The build provider service should be usable as a development target for the build system implementation as well as providing a baseline skeleton for implementing actual build providers.
As the resulting build provider skeleton is used for 3rd party implementations, it should be properly documented on code level.

Note: this story is intended to lay the groundwork for the build provider implementation as well as to provide an easy to use mock or reference implementation for implementing the build system.
Related

Userstory #517

@tnozicka can you add links to the SPI api definition?

It's not merged yet; temporary link is: https://github.com/tnozicka/almighty-devdoc/blob/add-build-spi-definition/design/api/build-spi/swagger.yaml

FAC - sgtm
NFAC:
@michaelkleinhenz should we make this story OSBP specific? I guess the goal should be to have an initial skeleton of OSBP; (which will become OSBP). I don't think we want to commit to maintaining a separate skeleton (mocked-only). At least for now I think that having a reference implementation in the form of OSBP should be enough.
We might add a skeleton as a starting point for build provider implementation later when things are more stable. But honestly there is probably not gonna be that many build providers (+ in the same language) to justify maintaining it and reference implementation should be enough.

@michaelkleinhenz What I have in mind for this story is to accommodate tasks like:

integrate http framework, logging
write Makefile and Dockerfile
add Build SPI methods endpoints, but do not implement them yet ("mocked")
setup initial (and simplified) CI/DC
...

@michaelkleinhenz, the above proposed by @tnozicka sounds good as writing a separate skeleton and maintaining it will be a pain, instead starting with OSBP will be a better decision. And for other providers there will be a reference of OSBP, which will be always up-to date.

new link: https://github.com/almighty/almighty-devdoc/blob/master/design/api/build-spi/swagger.yaml

Obsolete.


Add elm-review

Hello :wave:
I'm adding elm-review to the list.

Very nice. Thank you very much. 🍔


[FEATURE] Ability to process stereo pipelines

Is your feature request related to a problem? If so, Please describe.
Allow KWIVER pipelines which take in multiple (1, 2, or 3) input streams to be run
Describe the solution you'd like
I'd imagine there should be some configurable keywords, e.g. LEFT RIGHT which is the only difference in the filenames to indicate the respective streams.

@mattdawkins please add sample pipelines and dataset examples

Example - https://github.com/VIAME/VIAME/tree/master/examples/measurement_using_stereo also [install]/examples/measurement_using_stereo

Current VIAME does not support an output format we are able to process.  First pass will involve running pipeline without processing results.

First pass will be on Desktop, because Matt says this is more important.

Not sure how users are going to provide their own camera configuration.

I'm sure you noticed, but there's already a kwiver runner option for doing this, and you can use command line argument overrides to change the image input files, so I don't think a whole new pipeline type is necessary.
I'm only sharing this because when I looked at it, I spent the first 30 minutes thinking a whole new binary (and supporting functions, argument types, ipc calls etc) were necessary.

@mattdawkins - for Questions, @subdavis - for structure and if I'm missing anything for first pass.
Writing this for my own understanding of the ultimate goal here and to formulate some questions comments:

We allow users to upload an image sequence of both left/right frames where we use a keyword like ('left'/'right' or 'L'/'R') in the system.  Our current viewer doesn't support this so it will just swap through left/right frames if they view it.
In the upload we also need to support a .npz file which is of the format specified in the readMe (going to assume it is proper format).
On execution of the stereo pipeline it does some key checks first:

looks through the dataset folder trying to find the keyword to delineate between left/right images
Looks to see that an .npz camera calibration file is located alongside the images
Errors out if it can't find both

Continuing on if it finds all necessary files it will compose the cam1_images.txt file and the cam2_images.txt file so they can be fed into the system through cam1_imread:video_filename and cam2_imread:video_filename.  Ignoring Video support now.
The loaded
QUESTION:  There is a cam1_imread:frametime input that seems to be at 0.0333333 does this need to be changed based on anything?
The camtrawl_measure:output_fpath_cal_fpath - should be set to the calibration .npz file that is included with the data.
If all goes well the pipeline runs and returns out the output_raw_measurements.csv which the user can get to by going to show in file manager in the jobs tab to see the folder with the output file?

I think this is the flow we want for the first pass before we support visualization of stereo stuff.  We can also swap how it handles input for stereo items in the future as well.
Future Possible Updates (OTHER PEOPLE CAN IGNORE):

Checking if dataset meets requirements on load and only specifying measurement pipelines for datasets that have left/right and npz file.
Stereoscopic import, where he user can choose the left/right folders and we do some management of the files on import through metadata instead of specifying a strict naming structure.

This isn't quite what I had in mind, but it doesn't mean I'm right.

I was not thinking this would affect upload at all.  You upload 2 datasets independently and name them whatever you want.  You select them as left and right from some new pipeline run dialog.

Matt never mentioned this as a requirement.  Let's not plan for it unless he says otherwise.

I don't think any introspection is necessary.  Just having 2 dropdowns before you run the pipeline with "Pick the left one" and "Pick the right one" (where obviously you only need to set one and the other one sets itself.  This pipeline should only be available if you have exactly 2 datasets selected. (ii. see above)

I believe Matt is changing the pipeline to output the measurements in VIAME csv as attributess on detections.  This work may be blocked partially on that change.  Meanwhile, yeah, they could just open it in file manager.

Just to show some progress.  The code isn't the best right now and I'll have to update and iterate as well as determine if I want a separate interface or utilize some of the same stuff for n-camera displays.
https://user-images.githubusercontent.com/61746913/112917550-de2cb000-90d0-11eb-9069-345f57bcc404.mp4

Example EO/IR pipeline
https://github.com/VIAME/VIAME/blob/master/configs/pipelines/detector_arctic_seal_fusion_yolo.pipe


Color Coding New Content

Describe the Issue
Hello,
To whom it may concern I thought I might offer a suggestion: When introducing a new topic to the student (in this example the 1st time styling has been broached) I find that color coding can be helpful in the explanation.  I have included two images, one with color coded content and the other as the content currently stands.
Thank You,
John Bush
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-basic-css-by-building-a-cafe-menu/step-11
Steps to Reproduce
Nothing, this was an issue opened as a request from Quincy Larson @ossia on X (formerly Twitter).
Expected behavior
Nothing, this was an issue opened as a request from Quincy Larson @ossia on X (formerly Twitter).
Screenshots

System
Nothing, this was an issue opened as a request from Quincy Larson @ossia on X (formerly Twitter).
Additional context
Hope you all have a great day!

Hello there,
Thank you for taking the time to open this.
I like this idea, and it could be done by using the existing classes given by prismajs:

Center your h1 element by setting its text-align property to the value center.

@jobu777 thank you for taking the time to create this issue. I love this idea of color-coding new terms and their corresponding parts in the code examples.

i am interested to work on this issue. i would be happy if anyone tell me what should i do now..

I would like to work on this issue

It is worth noting that it is an accessibility failure to rely only on colour to communicate information.

It is worth noting that it is an accessibility failure to rely only on colour to communicate information.
@naomi-lgbt could you please explain this a bit. and also tell me can i work on this issue or not..

Thanks for your interest in contributing. This issue is still under discussion and not open to contribution yet.

could you please explain this a bit.

When colour is used to apply visual distinction to informative content, that content needs to be visually distinct without colour as well - so that users who may be visually impaired (e.g. Deuteranopia) still see the same emphasis.

thanks for the quick response. i will wait until it is open for contribute and also understood your explaination

It is worth noting that it is an accessibility failure to rely only on colour to communicate information.

In this case, I think we are clearly communicating the content already. This would just be an "added bonus" - kind of like how syntax highlighting already is.

i am not able to find the files , so can pls help me to find the files ?

Color Coding New Content #51554 @naomi-lgbt assign me this issue i am interested

@prachidoddi this issue is not open for contribution. Check out issues with the 'help wanted' label.


docs: example cluster-wide health endpoint

Signed-off-by: Shikugawa<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #14871
This PR adds some descriptions and examples to configure cluster-wide health endpoint.

Also, sorry to ask, but did you use your real name in the Signed-off-by: tag? Please note that this cannot be a pseudonym.

Ah, sorry. I will fix it.

I don't think we need to wait for the @cilium/policy review as I tested this. Marking as ready to merge.


Overlayke Issues Thread: Bugs, Misrecognition, it can't hecking count, etc.

Post any bugs or issues related to Overlayke here. Screenshots with miscalculations would be appreciated.

When I try to scan the tablet it says No calibration data. I tried to calibrate it but it said "Calibrating..." and nothing else.

Did you screen-cap a single water tile or something larger (with the snipping tool)? There shouldn't be anything to get stuck on, but it may take very long if you screen-cap an image that's too large pixel-wise. You could check task manager to see if AutoHotkey has high CPU usage. If not, the script could indeed be stuck on something.

I'll be offline soon (sleep), so there won't be any response for a few hours. If it's still not working for you, here are some steps that may help:

restart the script, open the overlayke section in the settings menu, and click reset
calibrate again but this time with a very small screen-cap (just as a test because it should finish quickly)
if it still gets stuck, you could try updating AutoHotkey
when restarting the script, does the it load the hotkey you set correctly? If not, the script may not have write permissions to save the config for some reason

The calibration code is quite simple and I can't think of anything that would make it behave as if stuck, so something outside the script has to be the issue.


Data changes are not applied after pressing persist data changes when we close DBeaver

System information:

Operating system (distribution) and version Mac
DBeaver version 21.0.1

Steps to reproduce, if exist:

Open a table in the RersultSet
Change a cell value (or add/delete a row)
Press the cross to close DBeaver
Press Yes in persist data changes pop up to apply changes
Open DBeaver and open the table to check changes

Expected: the changes are applied
Actual result: the changes are not applied
Note: The issue can be reproduced in manual and auto-commit mode

Changes weren't saved during shutdown process because all monitors were in state "canceled"
Transaction commit was forced BEFORE data save

Now fixed.

verified


error while attempting to start service

Windows version professional 7 sp3
reboot had the same result on restart.  installation used the right click option of runas administrator
Thanks
Robert

I had forgotten about this issue and should've included info about it in the README. Unpatched Windows 7 systems require the hotfix at Microsoft Security Advisory 3033929 in order to verify SHA-2 (SHA256) digital signatures. Current driver signing certificates can only sign using SHA256 so we unfortunately can't sign with SHA-1 to be backwards compatible. More details about Microsoft's deprecation of SHA-1 can be found at Windows Enforcement of Authenticode Code Signing and Timestamping.
If this fix works for you let me know so I can move to close the issue by updating the project README and making that 577 dialog box clearer. Thank you for the bug report, we appreciate it!

I'm having a problem finding the file to download from MS. The  site link
doesn't download the specific OS file. Feel free to close the issue until I
can get the  correct file from MS.
Thanks
Robert
On Sat, Jul 30, 2016 at 3:30 PM, Emerson R. Wiley<EMAIL_ADDRESS>wrote:

I had forgotten about this issue and should've included info about it in
the README. Unpatched Windows 7 systems require the hotfix at Microsoft
Security Advisory 3033929
https://technet.microsoft.com/en-us/library/security/3033929 in order
to verify SHA-2 (SHA256) digital signatures. Current driver signing
certificates can only sign using SHA256 so we unfortunately can't sign with
SHA-1 to be backwards compatible. More details about Microsoft's
deprecation of SHA-1 can be found at Windows Enforcement of Authenticode
Code Signing and Timestamping
http://social.technet.microsoft.com/wiki/contents/articles/32288.windows-enforcement-of-authenticode-code-signing-and-timestamping.aspx
.
If this fix works for you let me know so I can move to close the issue by
updating the project README and making that 577 dialog box clearer. Thank
you for the bug report, we appreciate it!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godaddy/procfilter/issues/1#issuecomment-236385312,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AD9CutWqIuYH-18s8xrARPjSeCzNNu0gks5qa6ZbgaJpZM4JY069
.

The links to the patches should be at the above security advisory link under the "Affected Software" section. If you're able to apply the patch and still have problems, don't hesitate to open another issue and we'll get it figured out. Thanks Robert!

The links to the patches should be at the above security advisory link under the "Affected Software" section. If you're able to apply the patch and still have problems, don't hesitate to open another issue and we'll get it figured out. Thanks Robert!

Just a quick update The patches fixed the issues.
Regards
Robert
On Mon, Aug 1, 2016 at 12:26 AM, Emerson R. Wiley<EMAIL_ADDRESS>wrote:

Closed #1 https://github.com/godaddy/procfilter/issues/1.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godaddy/procfilter/issues/1#event-740504004, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AD9CuhNxJ8m-X9NL-eKLC63UfvG1oVdNks5qbXWLgaJpZM4JY069
.

Great! Thanks for the update!


glyph.moveBy((x, y)) upgrades coordinates to floats

from fontParts.world import OpenFont

p = 'path/to/font.ufo'
f = OpenFont(p)
g = f["one"]
print(g[0].points[0])
g.moveBy((10, 0))
print(g[0].points[0])

<RPoint move (508, 1021) at<PHONE_NUMBER>>
<RPoint move (518.0, 1021.0) at<PHONE_NUMBER>>

it happens here https://github.com/robotools/fontParts/blob/master/Lib/fontParts/base/normalizers.py#L988-L1008
when the matrix is normalised
this was an issue on py2.7 not on py3 anymore...


GDScript 2.0: Setget variable does not enter the set block when a value is assigned in the declaration.

Godot version
v4.0.dev.calinou.795cd2eb3
System information
Windows 10
Issue description
A variable that uses setget and that assign a value already in it's declaration does not enter in the set block.
In the MRP, the test variable is only to get it's value from inside the set block and not from the assignment from outside.
Steps to reproduce
Create a setget variable and assign a value right in it's declaration. Note that it will get the value from the assignment without entering in the set block.
Minimal reproduction project
setget_test_godot_4.zip

This is intended. Setters aren't called for default values. Not assigning value is the same as assigning 0.

This is intended. Setters aren't called for default values. Not assigning value is the same as assigning 0.

Interesting. The behavior changed from Godot 3.x. So in the situation below it will still be necessary to call the _ready function even using the @onready for the test variable.
1)
@onready var_1: int = 200
@onready var test : int = 100:
	get:
		return test
	set(value):
		var_1 = var + value
		test = value/10

var_1 == 200
test == 100
2)
@onready var_1: int = 200
@onready var test : int = 100:
	get:
		return test
	set(value):
		var_1 = var + value
		test = value/10

func _ready():
	test = 100

var_1 == 300
test == 10

Closing since this is intended behavior.


useStateWithHistory: initialState of function type are not handled correctly

What is the current behavior?
With the hook useStateWithHistory, when using a function for the initialState, the function is correctly called and its output returned as the first state value
But when using the back handler, the state returned is the function and not its output.
Steps to reproduce it and if possible a minimal demo of the problem. Your bug will get fixed much faster if we can run your code and it doesn't have extra dependencies other than react-use. Paste the link to your JSFiddle or CodeSandbox example below:
https://codesandbox.io/p/sandbox/withered-sunset-gwjw8r
What is the expected behavior?
In case the initialState is a function, we should return its output, always, and not even store the function, as per the React Dev documentation stating this function should be triggered only once:

If you pass a function as initialState, it will be treated as an initializer function. It should be pure, should take no arguments, and should return a value of any type. React will call your initializer function when initializing the component, and store its return value as the initial state. See an example below.

A little about versions:

OS: N/A
Browser (vendor and version): N/A
React: N/A
react-use: latest
Did this worked in the previous package version?: No

The issue seems to be that history saves the function itself rather than the result.
I created a fix and posted a PR.


[staging] cpan2nix: remove perl libraries which are not newer than built-ins

Motivation for this change
remove perl libraries which are not newer than built-ins in the oldest supported perl-5.22
@rycee @orivej

there is one merge conflict as it appears.

fixed


validate_number does not work with decimals

@josevalim tried looking into this and validate_number/3 works fine with decimals. Looks like my earlier report was wrong. Sorry for the trouble!

Yay, even better, no bug to fix! Thanks for pinging back!

Anytime! :+1:


How to add actix plugin to API operations implemented with trait?

I am trying to add paperclip to Lemmy, but that isnt working because paperclip seems to expect that API handlers are implemented as simple methods. But our API is handlers are implementations of a Perform trait, like this:
### Generic trait for all API calls
#[async_trait::async_trait(?Send)]
pub trait Perform {
  type Response: Serialize + Send;

  async fn perform(
    &self,
    context: &Data<LemmyContext>,
    websocket_id: Option<ConnectionId>,
  ) -> Result<Self::Response, LemmyError>;
}

### Concrete implementation for a specific API call (ListCategories in this case)
#[async_trait::async_trait(?Send)]
impl Perform for ListCategories {
  type Response = ListCategoriesResponse;

  async fn perform(
    &self,
    context: &Data<LemmyContext>,
    _websocket_id: Option<ConnectionId>,
  ) -> Result<ListCategoriesResponse, LemmyError> {
    ...
  }
}

I tried by adding #[api_v2_operation] to perform() in various different places, but no matter what I do its always throwing lots of errors. Is there any way to make this work?

As of now, paperclip uses unit structs to implement a trait (Apiv2Operation) which contains the auto-generated schema information. You're right that it's designed for function-like handlers, but we can make it work for method-like handlers too. I think it'll only require some minor fixes in the macro implementation, so that we can specify the #[api_v2_operation] on handler struct methods directly, which will then implement that trait for those methods.
Thanks for taking the time and trying it out! I'll keep this in mind. :smile: