Run Dart tests with pub run test

Thanks for the cool plugin :) I wanted to update it a bit to work for regular old dart packages, not just those fancy Flutter apps.

Plain Old Dart Packages run with the default pub run test.
Flutter apps that make use of the flutter_test package run with flutter test.

Make sure these boxes are checked before submitting your pull request:

[X] Add fixtures and spec when implementing or updating a test runner
[X] Update the README accordingly
[X] Update the Vim documentation in doc/test.txt

@brianegan the standalone pub command is deprecated see here, because of that this change breaks for anyone not using the global pub command which will be the case for anyone who has dart installed as part of installing flutter. Note that dart test works and I think is now the recommended way of running dart tests not just for flutter (also discussed here)

Shall we revert this change or would someone like to some logic to check for the presence of pub before invoking it?

Would be good to get @brianegan's input. Since I'm not really familiar with what changes will be needed and if just dropping dart test in everywhere that pub run test is used will be enough. Seems from the docs like it would be but I use dart entirely in the context of flutter so I could easily be missing something.
Maybe we can leave this for a day or 2 to see if Brian or I get to trying to raise a PR to use dart test instead and if neither of us do then revert this. I think the number of users who are probably affected by this change might be greater than the people who were aware of this addition and are using the global pub run test

Hey, sorry about that -- I almost always use the standalone version of Dart for normal Dart projects, and the Flutter version of Dart only for Flutter projects, so I hadn't run into this issue.
I've opened up the following PR as a fix: https://github.com/vim-test/vim-test/pull/552

Hm, Can't edit my comment. Correct new PR is: https://github.com/vim-test/vim-test/pull/556


Add Joins

Abstract
This proposal is mostly based on #297
Proposal: JOINs
For now, Genji have no ways to select several tables in one query.
JOINs is the most commonly used way in relation databases.
Changes

Add SQL JOIN operation parser.

Add a Join operation to the planner and nodes which implementing it.
Joining algorithms implemented by #297:

Nested loops join
Basic algorithm with O(n*m) complexity in worst case.
Implementation: https://github.com/tdakkota/genji/blob/feature/inner-join/sql/planner/join.go#L65
Hash join
Algorithm with O(n+m) complexity in worst case, but requires a Equi-join predicat in join clause.
Implementation: https://github.com/tdakkota/genji/blob/feature/inner-join/sql/planner/join.go#L140

Joining algorithms which not implemented by #297:

Sort-merge join
Algorithm with O(n+m) complexity in worst case, but requires sorted document stream,
so with sorting complexity becomes quasilinear O(nlogn+mlogm)
Best in-use with index.

A good note about implementing joins from CrateDB blog: https://crate.io/a/joins-faster-part-one/

Add a optimizator rule to determine which algorithm should be used
For now, #297 adds a optimizer rule which replaces a Nested loops join to Hash join if clause is a equi-join.

Add a special joined document type which allow to access fields from all joined tables.
And there a problem, Genji is not a relational database, document fields can be undeclared in table definition. So any relative field path in query can be ambigous.
Some ideas to solve:

Force to use absoulte path to field when query contains JOIN clause, i.e.table_name.path_to_field
Disallow use fields which are not defined in table  when query contains JOIN

I like force to use absolute path because it’s explicit and clear.
In introducing relationships between tables there may be contradictions introduced with other aspects.
One thing that I would like to see a inner join to provide architecturally are material used views. These are a demormaliaed view from a aggregation of many tables or an inner join
It’s easy to build a hand make material view based on listening for changes to tables and then modifying the materialized view but it would be better to formalize the pattern
Anyway maybe other consider that inner joins opens the door to materialized views going forward and are keen to discuss a CQRS pattern for genji. It would be very powerful because badger already supports subscribe to any changes and sour easily lends itself to doing it. Quite exciting.

This probably wont work for all uses, but in my situation, I was coming from
SQLite and was using NATURAL JOIN with Junction tables. Using Genji I was able
to ditch that paradigm using a model like this:
type Artist struct {
   ID int
   Name string
   Album []int
}

type Album struct {
   ID int
   Name string
   Song []int
}

type Song struct {
   ID int
   Name string
}

So as can be seen, Genji has the ability to do Slice values. It some cases, this
can obviates the need for junction tables, and in turn JOIN as well. Again this
might not be the case for all users, but it was for me.

This probably wont work for all uses, but in my situation, I was coming from
SQLite and was using NATURAL JOIN with Junction tables. Using Genji I was able
to ditch that paradigm using a model like this:
type Artist struct {
   ID int
   Name string
   Album []int
}

type Album struct {
   ID int
   Name string
   Song []int
}

type Song struct {
   ID int
   Name string
}

So as can be seen, Genji has the ability to do Slice values. It some cases, this
can obviates the need for junction tables, and in turn JOIN as well. Again this
might not be the case for all users, but it was for me.

@89z Great work
I would like to extend your example to add graphQL: https://github.com/UHN/ggql/tree/master/examples/reflection#define-the-api. This is a new graphql lib that is much faster than others.
With Genji backing it , it will be highly efficient.
Maybe we can make it a genji example too that people can use as a reference.

@89z Great work
I would like to extend your example to add graphQL: https://github.com/UHN/ggql/tree/master/examples/reflection#define-the-api. This is a new graphql lib that is much faster than others.
With Genji backing it , it will be highly efficient.
Maybe we can make it a genji example too that people can use as a reference.

Instead of joins, graph model can be implemented, its faster.
Take a look at https://surrealdb.com/docs/surrealql/statements/relate
@tdakkota @clarkmcc


Move velero/v1alpha1 to veleroinstall/v1alpha2

/lgtm
Pretty straight-forward.
Take this as a one-of-two approval, so @luis-falcon still has a chance to see it.

/lgtm

@mbarnes @luis-falcon I've rebased this on top of your changes. Please review this again on Monday. Thanks!

/hold
for review

/lgtm

/hold cancel


Error Frames sending direction

When transmitting data with this Library I get randomly errorframes RX (so the device holding the library sending to other nodes). It occurs randomly and go onb for several minutes. The device still reads on CAN, just sending seems to be distorted.
I can´t identify hardware problems.
Error frames are the following:
ECC 10 10 1000 CRC Sequence
ECC 10 10 10 10 Data
ECC 01 11 1000 CRC Delimiter
ECC 1111 00 11 Tolerate Dominant Bits
Thankfull for any hints

I had the same problem. I fixed this by changing the oscillator from 8MHz to 16MHz in hardware. Then: mcp2515.setBitrate(CAN_500KBPS, MCP_16MHZ);


DID pkg test coverage

Increasing test coverage. Minor refactors.

Codecov Report

Merging #192 (356c848) into main (3ced400) will increase coverage by 0.70%.
The diff coverage is 40.00%.

@@            Coverage Diff             @@
##             main     #192      +/-   ##
==========================================
+ Coverage   54.82%   55.53%   +0.70%
==========================================
  Files          31       31
  Lines        3732     3710      -22
==========================================
+ Hits         2046     2060      +14
+ Misses       1310     1282      -28
+ Partials      376      368       -8

Impacted Files
Coverage Δ

did/interface.go
50.00% <ø> (ø)

did/key.go
66.67% <0.00%> (+18.30%)
:arrow_up:

did/model.go
40.00% <ø> (-15.88%)
:arrow_down:

did/pkh.go
82.22% <0.00%> (+7.78%)
:arrow_up:

did/util.go
36.21% <0.00%> (-1.29%)
:arrow_down:

did/web.go
59.80% <ø> (ø)

did/peer.go
54.51% <87.50%> (-1.38%)
:arrow_down:

cryptosuite/jsonwebkey2020.go
56.94% <100.00%> (ø)

actually found a few issues!


Make Integer-valued attributes work in constraints

Mesos 0.22.1 attributes that have integer values don't work if you try to use them as constraints in Marathon 0.8.2. Jobs never get scheduled.
Not quite sure where things go wrong.

Hey @alberts,
are you sure the attribute is defined on the slave?

If the specified attribute is not defined on the slave, then most operators will refuse to run tasks on this slave. In fact, only the UNLIKE operator will (and always will) accept this offer for now, while other operators will always refuse it.

See :
https://github.com/mesosphere/marathon/blob/master/docs/docs/constraints.md#attribute-field
Could you post the relevant parts of your JSON definition for the app you are trying to deploy?
Thanks.

This is how mesos slave reports in /state of mesos master api:
    {
      "active": true,
      "attributes": {
        "rack": 109,
        "type": "compute"
      },
      "hostname": "myhost",

This is how constraint is specified:
[["rack","CLUSTER","109"]]

Marathon fails to schedule tasks in this case.
Mesos 0.25.0, Marathon 0.11.1.

This is a known limitation that is documented here (https://mesosphere.github.io/marathon/docs/constraints.html):
"""
Note that currently Marathon only supports text values for attributes and will not be able to process any numberic values. For example the attribute pair foo:bar will be recognised, but marathon will not be able to match cpu:4.
"""
That doesn't make it pretty, though.
I would like to keep this issue as a reminder to extend the functionality.

@kolloch do you have any plans for using numeric constraints differently? I thought that attributes are opaque strings, looks like I could be wrong.

Attributes have the same types as resources but they are not consumed. Thus if the value has the "shape" of an integer, Mesos will use a different type for them. See attributes/resources in Mesos

Any chance to at least pretend that numeric attributes are strings before implementing full support for numbers and ranges?
This way you can have your rack:123 attribute now and group by it. Once numbers land, you'll be able to use ranges, if you wish. All of that without changing attributes to hacks like rack:r123 and wiping mesos slave state twice.

If I am not mistaken, treating all values as strings should be ridiculously easy. I would have to think about whether that's always what you want (for ranges it's really strange) -- but we simply could introduce separate operators for scalar types where needed.
Can you explain why this is important to you? Right now, the biggest problem in my view is that things don't work as expected but you can work around them once you got why.

Restarting slaves and losing their tasks is the problem I have. I don't wan't to do that. I already have rack:123 attributes in place and I want to use them.

@alberts 🦄🚀👾!

This should be resolved by #3865.


[Bug] File "/u02/liuys/sglang/python/sglang/srt/server.py", line 621, in _wait_and_warmup Killed

Checklist

[ ] 1. I have searched related issues but cannot get the expected help.
[ ] 2. The bug has not been fixed in the latest version.
[ ] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.
[ ] 4. If the issue you raised is not a bug but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[ ] 5. Please use English, otherwise it will be closed.

Describe the bug
执行 python -m sglang.launch_server  --model-path /u01/liuys/.cache/modelscope/hub/qwen/Qwen2___5-7B-Instruct  --tp-size 4   --host <IP_ADDRESS>  --port 8008 过一段时间就出现下面的错误
[2024-12-09 16:10:09] INFO:     Uvicorn running on http://<IP_ADDRESS>:8008 (Press CTRL+C to quit)
[2024-12-09 16:10:51] INFO:     <IP_ADDRESS>:46322 - "GET /v1/models HTTP/1.1" 200 OK
[2024-12-09 16:10:51] INFO:     <IP_ADDRESS>:46322 - "POST /v1/chat/completions HTTP/1.1" 200 OK
[2024-12-09 16:10:51 TP0] Prefill batch. #new-seq: 1, #new-token: 25, #cached-token: 0, cache hit rate: 0.00%, token usage: 0.00, #running-req: 0, #queue-req: 0
[2024-12-09 16:10:56 TP0] Decode batch. #running-req: 1, #token: 66, token usage: 0.00, gen throughput (token/s): 0.83, #queue-req: 0
[2024-12-09 16:10:57 TP0] Decode batch. #running-req: 1, #token: 106, token usage: 0.00, gen throughput (token/s): 122.39, #queue-req: 0
[2024-12-09 16:10:57 TP0] Decode batch. #running-req: 1, #token: 146, token usage: 0.00, gen throughput (token/s): 122.59, #queue-req: 0
[2024-12-09 16:10:57 TP0] Decode batch. #running-req: 1, #token: 186, token usage: 0.00, gen throughput (token/s): 121.76, #queue-req: 0
[2024-12-09 16:10:58 TP0] Decode batch. #running-req: 1, #token: 226, token usage: 0.00, gen throughput (token/s): 122.17, #queue-req: 0
[2024-12-09 16:10:58 TP0] Decode batch. #running-req: 1, #token: 266, token usage: 0.00, gen throughput (token/s): 123.13, #queue-req: 0
[2024-12-09 16:14:24] Initialization failed. warmup error: Traceback (most recent call last):
File "/u02/liuys/sglang/python/sglang/srt/server.py", line 621, in _wait_and_warmup
assert res.status_code == 200, f"{res=}, {res.text=}"
^^^^^^^^^^^^^^^^^^^^^^
AssertionError: res=<Response [503]>, res.text=''
Killed
Reproduction
无
Environment
无

一样的错误

#./XXXXXXXX/sglang/srt/server.py   line 617：把循环改大后不报错了
# Wait until the server is launched
success = False
for _ in range(12000):
time.sleep(1)
try:
res = requests.get(url + "/get_model_info", timeout=5, headers=headers)
assert res.status_code == 200, f"{res=}, {res.text=}"
success = True
break
except (AssertionError, requests.exceptions.RequestException):
last_traceback = get_exception_traceback()
pass

#./XXXXXXXX/sglang/srt/server.py line 617：把循环改大后不报错了
code： # Wait until the server is launched success = False for _ in range(12000): time.sleep(1) try: res = requests.get(url + "/get_model_info", timeout=5, headers=headers) assert res.status_code == 200, f"{res=}, {res.text=}" success = True break except (AssertionError, requests.exceptions.RequestException): last_traceback = get_exception_traceback() pass

unset http_proxy
unset https_proxy
ok!


Create a chatbot with coral and raspberry pi 4b

Description
I'm trying to create a chatbot with coral and raspberry pi. But I have used LLama2 models to create the text recognosation and I can make question to the model about the text and documents. It is posible to do that with coral o use it to proccess all the information?
Click to expand!
Issue Type
Support, Feature Request
Operating System
Linux
Coral Device
USB Accelerator
Other Devices
Rapsberry Pi 4
Programming Language
No response
Relevant Log Output
No response

no, coral has not enough memory


Support for other file formats

Would it be possible to import and/or export in other formats? KML and GeoJSON are pretty standard in spatial files, others are less important I think. For instance the Maps.me mobile app only takes KML files, and is widely used for hiking, thus it would help to be able to export from gpxstudio directly in KML. For GeoJSON it's more for import, as it's kind of the standard web format for geospatial files (in a general way).
You could use leaflet-omnivore.

Woops, should have better searched in the open issues first. Duplicate of #32


Context menu on the column headers to query Jira for issues in a state

This could also be done from the settings drawer to query all issues visible in the current filter

test


PgConnection: use more obvious query for ping()

closes #340

It seems a lot of postgres drivers made the same mistake we did. Diesel has this same latent bug so it likely doesn't trigger much in practice. From a quick survey (omitting those that did SELECT 1):

vert.x (Java) never pings
pgx (Go) pings with a query of ;
pq (Go) pings with an empty query
r2d2-postgres (Rust) pings with an empty query
DB::Pg (Perl) pings with /* DB::Ping v<IP_ADDRESS> */

I vote for switching to /* ... */ for the comment because it'll look better in the logs and just have it be /* ping */, it should be obvious it's coming from SQLx / PgConnection .

Agreed on the block comment, although I think the comment should still make it unambiguous what's triggering the ping as the user could be connecting with a different client. Maybe /* SQLx ping */ then if we want to keep it shorter?

I'm not especially against it so I'd be fine with SQLx ping.
I do think its a bit superfluous as we should add support for application name to answer "what client is connected" in postgres logs which would solve that question more generally.
Totally orthogonal to this PR but that would mean detecting ?application_name=NAME in the URL and defaulting to PGAPPNAME according to the docs. We would then pass that to postgres in the Startup message. It allows DBAs to prefix the name of the application in the log files.

Tested locally, merging.


compare stepped and leap accrual

(uses the cantina-review branch of morpho-blue)
Calling accrueInterests frequently seems to accumulate errors. I haven't looked into why yet but it seemed worth flagging.
The tests start from a market with 1e18 supplied and 0.9e18 borrowed.

forge test --mt CompareAccrual -vv: calling accrueInterests every 1000 seconds over a 9 week period increases borrow by 1627% compared to calling accrueInterests once after 9 weeks.

forge test --mt SteppedAccrual: calling every 1k seconds over a 13 week periods overflows adding interests to totalBorrowAssets and totalSupplyAssets, ie there:

// _accrueInterests in Morpho.sol
market[id].totalBorrowAssets += interest.toUint128();
market[id].totalSupplyAssets += interest.toUint128();

This happens on main as well, with the same starting params & update period the overflow occurs faster (under 9 weeks).

Closing (initial rate 200% + 9 weeks untouched market not realistic)


misoc.interconnect.stream -> DMAC | PRI

stream sink
ignored EOP
request burst transfer via DMAC | PRI

Refer to ARM DDI 0424D.

The Linux driver expects a single request, not a burst request as currently implemented:
root@zedboard-zynq7:~# devmem2 0xf8003100
/dev/mem opened.
Memory mapped at address 0xb6f68000.
Read at address  0xF8003100 (0xb6f68100): 0x00000007

Fixes the type: a065766c634fd20a7f6ae2fbecd71cf66056e7e0.

Tested on Zedboard using w/ fb60d1dc3e32f381c10119cdfc861cf4255c3333.


SharpNamespaceExtension = InfoBar/StatusBar possible?

Hello,
I'm currently trying to get the size of every directory shown in the "InfoBar" (I don't know the correct name - Windows 7 Explorer the bottom bar (if activated!))
https://msdn.microsoft.com/en-us/library/windows/desktop/cc144095(v=vs.85).aspx#unknown_65306
( I guess it's called "StatusBar" )
So I read a bit and thought it would be achievable by creating a NamespaceExtension (or rather a SharpNamespaceExtension).
Did you (or anyone else) try this before?
I saw the examples in the source but currently it's not working for me
(It can not open the "Virtual Folder" (MyDesktop))
It is (yet?) not documented so I wasn't sure if it is "supported" at all.
Thanks.
Countryen

Hey again,
I made progress:

So it really works, thats nice.
Now, I want the information shown at 1 at position 2 (in the "StatusBar")
Any Idea?
I will try the different VirtualFolders next.
Solution so far:
Added Attributes as following:
    [DisplayName("C0.Windows.Explorer.InfoBar.SizeofDirectoryInfo.Prototype4")]
    [ComVisible(true)]
    [NamespaceExtensionJunctionPoint(NamespaceExtensionAvailability.Everyone, VirtualFolder.MyComputer, "Prototype4")]

Then I had to manually add a RegistryKey:
HKEY_LOCAL_MACHINE\Software\Microsoft\Windows\CurrentVersion\Shell Extensions\Approved
-> This key was not there (not even "Shell Extensions")

Error: System.InvalidOperationException: Failed to open the Approved Extensions key.
After I added the key, it worked.

Info:
NamespaceExtensionAvailability.CurrentUser did not work ->
I only got System.InvalidOperationExceptions and System.SecurityExceptions
QUESTION:
Why does SharpShell.ServerRegistrationManager.IsExtensionApproved
not approve keys with RegistryHive.CurrentUser aswell?
Because at HKEY_CURRENT_USER\Software\Microsoft\Windows\CurrentVersion\Shell Extensions\Approved I had the key already.
I guess the same reason the NamespaceExtensionAvailability.CurrentUser doesn't work?
Thanks so far for this great Framework.
Hope anyone has got some information :)
Thanks.
Countryen

Namespace extensions have been updated recently, might be worth revisiting this at some stage. Until then, I'm closing this to focus on recent issues, please re-open if needed.


request: Ability to disable certain commands on the fly

Is your feature request related to a problem? Please describe.
I am the server admin for a server with almost 14,000 members. We've survived without a moderation-focused bot for a while, but we're feeling the strain. Skyra seemed a good choice, until I found out about all the extraneous commands that have the potential to create a massive moderation headache, like the Pokemon and Weeb commands, among many others. Ours is a semi-technical server, so it is very important that all channels be kept free of extraneous junk.
Describe the solution you'd like
I would like a way to disable certain commands from being used. Some commands I would prefer be kept out of the reach of the general public of the server, and other commands would be very helpful for them to use. Though I understand that this bot is best used in small-scale situations, I still feel that most people would like to have this ability.
Describe alternatives you've considered
I've looked into this issue, and there don't seem to be any pre-existing solutions in regards to Skyra. We will most likely remove the bot from the server and find a more moderation-focused bot, rather than a general-use bot, but I still think that it'd be a nice feature to have for anyone looking to use Skyra.
Additional context
None.

I hope the information above was of help, if your problem still persist, please don't hesitate to re-open or create a new issue :)


[oc_erchef] replace ejson calls with chef_json

In preparation for removing couchdb, remove all calls to ejson
which is only pulled in via the couchdb dependency.
ping @chef/lob

:+1: chef_json is a wrapper around jiffy which also tends to be faster than ejson (although it appears only one of the changes is in an actual module, so I doubt we'll see much impact :).

The only components left will get wrapped up when we remove the actual couchbeam dependency which needs  ejson, in order to avoid breaking dialyzer and build:

chef_db/Makefile
reltool.config
rebar.config
rebar.config.lock


TensorRT tutorial

Dear MONAI Team,
Thanks for the great tutorial on acceleration the training process.
https://github.com/Project-MONAI/tutorials/blob/main/acceleration/README.md
Would it be possible to provide some tutorials on using TensorRT to speed up the inference process (e.g., 3D segmentation)?

Hi @JunMa11 ,
I think PyTorch TensorRT lib can optimize a TorchScript model, and still use it as a regular TorchScript model in a PyTorch program, you can find more details: https://github.com/pytorch/TensorRT
Thanks.

Thanks:)


Override default log level colors

Added an option to override the default log level colors.

:tada: This PR is included in version 1.10.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Adjust screen size and connectivity issue

I have two problems.

x11docker --home xorilog/telegram
it open too large screen then i tried to add x11docker --home --size 800x600 xorilog/telegram but doesn't effect, seems command incorrect.

x11docker -- jess/chromium --no-sandbox
even above telegram container connect properly but chromium container has network connectivity issue(cannot connect).

Double post #284, closing.


Adds function to set multiple properties in the same api call

Adds a new set_combined function that allows both the colour and brightness to be set during the same api call regardless of the colour mode (hs / xy / hex / temp / etc.) used. This was previously only possible with the set_hsb function and not any of the other modes.
This allows for smoothly transitioning colour and brightness simultaneously. Example use cases would be sunrise / sunset simulation, switching from bright cool white task lighting to dim warm white relaxation lighting etc.
Important caveat: In my testing this doesn't work with 1st party (IKEA) bulbs, whilst the gateway behaves correctly a physical change to the bulb only occurs for the numerically first parameter, other changed parameters are ignored. It does seem to work for Philips hue bulbs paired with the gateway. For this reason a can_set_combined feature flag is included, which is currently only set for Philips bulbs, further testing will be needed to determine exactly which bulbs can and cannot transition different settings simultaneously.
Previous functionality should be unchanged (non-breaking)

Coverage increased (+0.3%) to 69.223% when pulling 17819d3e554dda7b62bf11cb4f1806e8ac692972 on Hate-Usernames:set_combined into 8eb1129a758295b188855a14d9a26bc5878b15a1 on ggravlingen:master.

Coverage increased (+0.3%) to 69.223% when pulling 17819d3e554dda7b62bf11cb4f1806e8ac692972 on Hate-Usernames:set_combined into 8eb1129a758295b188855a14d9a26bc5878b15a1 on ggravlingen:master.

Coverage increased (+0.3%) to 69.223% when pulling 17819d3e554dda7b62bf11cb4f1806e8ac692972 on Hate-Usernames:set_combined into 8eb1129a758295b188855a14d9a26bc5878b15a1 on ggravlingen:master.

I'm wondering if we can extend our existing functionality to include
something that allows us to combine calls? So we can essentially have a
little builder for a command.
I'm all for exposing features of the gateway (even if the bulbs don't
support it), but we need to be careful not to create misleading
functionality for unsupported hardware.
On Thu, 10 May 2018, 11:08 pm Coveralls<EMAIL_ADDRESS>wrote:

[image: Coverage Status] https://coveralls.io/builds/16930145
Coverage increased (+0.3%) to 69.223% when pulling 17819d3
https://github.com/ggravlingen/pytradfri/commit/17819d3e554dda7b62bf11cb4f1806e8ac692972
on Hate-Usernames:set_combined into 8eb1129
https://github.com/ggravlingen/pytradfri/commit/8eb1129a758295b188855a14d9a26bc5878b15a1
on ggravlingen:master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ggravlingen/pytradfri/pull/164#issuecomment-388200934,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA1TO0Cz2TKYZrLrxasSxgbsv1p3ksq7ks5txLpfgaJpZM4T6n-N
.

I did consider something like that, but realistically this is only use case I could see (improving transitions when colour and brightness are changed together). A method to join commands seemed less discoverable and we would still need to have a flag which indicates whether the bulb supports this.
I haven't looked too deeply into the Command class, if the data parameter is always a dict, we could introduce an __add__ method which merges the data of two commands so that we could then run something like api(command1 + command2) But I haven't tested this, so I don't know how it would handle e.g. combining commands sent to different bulbs etc. It's also in my opinion less obvious that this wouldn't be supported by all bulbs.
Other options would be to add brightness to all the colour methods (as in set_hsb), but again this is misleading. Or to raise an error if we try to set too many parameters at once on an unsupported bulb, but this isn't currently done anywhere else (e.g. setting the colour on a non-colour bulb).
Happy to take advice as to how to proceed. Admittedly my main goal is to use this in homeassistant, so it doesn't really matter to me how the functionality is exposed :smile:

I assume this can be closed now?

Yes, this use case can now be achieved with #167.


completions: delete brew-cask check

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?
[x] Have you successfully run brew man locally and committed any changes?

delete the check whether the command is brew cask or not  in completions.rb

Thanks again @hyuraku!


osx: scroll delta should take hidpi factor into account

Fixes https://github.com/servo/servo/issues/9502

These scrolling issues are a bit over my head. I'm going to trust you and merge blindly.


The proposal to set up the Drupal Code Sniffer

https://www.drupal.org/project/coder
https://www.drupal.org/node/1419988
I believe it will increase the worth of this product for developers.
Regards, Artu.

Since Composer is included, I think it's best to leave CS setup to individual projects—they can include coder sniffer in their composer.json, configure the sniffs per-project (every project is usually different), and then run the sniffs as part of the project's own build process.
See Acquia's BLT for an example: https://github.com/acquia/blt
Also, a different CI/CD project I maintain (minimally) also has an example of separately installing Coder Sniffer and integrating with Jenkins and SonarQube: https://github.com/geerlingguy/drupalci-sonar-jenkins

For those interested, setup is pretty easy. In your config.yml add the following to composer_global_packages::
  - { name: drupal/coder, release: '*' }
  - { name: dealerdirect/phpcodesniffer-composer-installer, release: '*' }

I like relying on the VM to provide this rather than my local machine (the host). Makes switching workstations more seamless.


Crons: POC implementation of celery beat integration

TODO: Add more documentation here. This will require https://github.com/getsentry/sentry/issues/45295

May we track this in https://github.com/getsentry/sentry-python/issues/1932?

Yes I will merge.


[BUG] CheckmarxAST Task Fails with “Object Reference Not Set” Error and Duplicate Project Creation Error on Re-run

Bug
When running the Checkmarx AST task in our Azure DevOps pipeline, the scan completes, but the results export fails with an “Object reference not set” error. Additionally, on a second attempt to run the task, we encounter a “Failed to create a project” error due to the project already existing.
Logs:
First Run:
Scan Finished with status: Completed
Failed listing results: export error: Object reference not set to an instance of an object.
Exit code received from AST-CLI: 1

Second Run:
Failed creating a project: CODE: 208, Failed to create a project, project name already exists
Exit code received from AST-CLI: 1

Expected behaviour

The scan results should be exported successfully without any errors.
On re-running the task, it should recognise that the project already exists and proceed without attempting to recreate it.

Actual behaviour

The scan completes, but fails during the results export phase with an "Object reference not set" error.
On the second run, the task fails due to an attempt to recreate an existing project, causing a "Failed to create a project" error.

Steps to reproduce

Run the pipeline for the build branch on a project.
On the first run, observe that the scan finishes, but results export fails with an “Object reference not set” error.
Re-run the pipeline. On the second run, observe the “Failed to create a project” error due to the project already existing.

Environment

AZURE PLUGIN version 2.0.39
Platform RHEL 8

It was permissions issue, now resolved..


Error on Bun run dev

I cd to the ui folder and bun run dev but I receive the following error message:
$ vite
file:///mnt/Essam/AI/devika/ui/node_modules/vite/bin/vite.js:7
await import('source-map-support').then((r) => r.default.install())
^^^^^
SyntaxError: Unexpected reserved word
at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18)
at async link (internal/modules/esm/module_job.js:42:21)
error: script "dev" exited with code 1

you should first install bun if u dont have it and then run bun i or bun install.. its just like npm

I'm also having problem with running the UI.
> cd ui
> bun --version
1.0.35

> bun install
bun install v1.0.35 (940448d6)

Checked 510 installs across 526 packages (no changes) [12.00ms]

> bun run dev
$ vite
(node:80977) UnhandledPromiseRejectionWarning: SyntaxError: Unexpected token '??='
    at Loader.moduleStrategy (internal/modules/esm/translators.js:145:18)
(Use `node --trace-warnings ...` to show where the warning was created)
(node:80977) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag `--unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:80977) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

you should first install bun if u dont have it and then run bun i or bun install.. its just like npm

bun is installed my OS is ubuntu
> bun --version
1.0.35

delete the node_modules folder and reinstall the dependencies, you need to execute "bun install" or "npm install" if bun doesn't work

I cd to the ui folder and bun run dev but I receive the following error message:
$ vite file:///mnt/Essam/AI/devika/ui/node_modules/vite/bin/vite.js:7 await import('source-map-support').then((r) => r.default.install()) ^^^^^
SyntaxError: Unexpected reserved word at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18) at async link (internal/modules/esm/module_job.js:42:21) error: script "dev" exited with code 1

try reinstalling the Vite package globally (npm install -g vite) or locally (npm install vite) to ensure you have the latest version installed.

I cd to the ui folder and bun run dev but I receive the following error message:
$ vite file:///mnt/Essam/AI/devika/ui/node_modules/vite/bin/vite.js:7 await import('source-map-support').then((r) => r.default.install()) ^^^^^
SyntaxError: Unexpected reserved word at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18) at async link (internal/modules/esm/module_job.js:42:21) error: script "dev" exited with code 1

try reinstalling the Vite package globally (npm install -g vite) or locally (npm install vite) to ensure you have the latest version installed.

same issue

Running below
> bun install
Got the error below
`
bun install v1.0.35 (940448d6)
⚙  @sveltejs/kit [1/2] file:///mnt/d/dev/github/devika/ui/node_modules/@sveltejs/kit/postinstall.js:7
const cwd = process.env.INIT_CWD ?? process.cwd();
^
SyntaxError: Unexpected token '?'
at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18)
at async link (internal/modules/esm/module_job.js:42:21)
`
Any ideas?

Running below on WSL > bun install Got the error below ` bun install v1.0.35 (940448d6) ⚙ @sveltejs/kit [1/2] file:///mnt/d/dev/github/devika/ui/node_modules/@sveltejs/kit/postinstall.js:7 const cwd = process.env.INIT_CWD ?? process.cwd(); ^
SyntaxError: Unexpected token '?' at Loader.moduleStrategy (internal/modules/esm/translators.js:133:18) at async link (internal/modules/esm/module_job.js:42:21) ` Any ideas?

> npm install does not have the issue. Then I ran with npm run dev instead of bun run dev on WSL. It works but no reponse.

Try running nvm install 21, verify everything is still installed with bun install and then bun run dev should work

Try upgrading your version of npm / node. I had Fixed it for me on WSL.
sudo npm install -g n
sudo n lts
bun run dev

Before installing latest node:
(devika) voxie3d:~/repos/devika/ui$ bun run dev
$ vite dev
/home/voxie3d/repos/devika/ui/node_modules/vite/bin/vite.js:2
import { performance } from 'node:perf_hooks'
       ^

SyntaxError: Unexpected token {
    at Module._compile (internal/modules/cjs/loader.js:723:23)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10)
    at Module.load (internal/modules/cjs/loader.js:653:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:593:12)
    at Function.Module._load (internal/modules/cjs/loader.js:585:3)
    at Function.Module.runMain (internal/modules/cjs/loader.js:831:12)
    at startup (internal/bootstrap/node.js:283:19)
    at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)
error: script "dev" exited with code 1

(devika) voxie3d:~/repos/devika/ui$ npm install
npm ERR! code ENOTSUP
npm ERR! notsup Unsupported engine for<EMAIL_ADDRESS>wanted: {"node":">= 18"} (current: {"node":"10.19.0","npm":"6.14.4"})
npm ERR! notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Not compatible with your version of node/npm<EMAIL_ADDRESS>npm ERR! notsup Required: {"node":">= 18"}
npm ERR! notsup Actual:   {"npm":"6.14.4","node":"10.19.0"}

failed to load config from C:\Users\Headstart\devika\ui\vite.config.js
error when starting dev server:
Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@sveltejs/kit' imported from C:\Users\Headstart\devika\ui\vite.config.js.timestamp-1711603273083-4bf5db431ce54.mjs
at new NodeError (node:internal/errors:405:5)
at packageResolve (node:internal/modules/esm/resolve:887:9)
at moduleResolve (node:internal/modules/esm/resolve:936:20)
at defaultResolve (node:internal/modules/esm/resolve:1129:11)
at nextResolve (node:internal/modules/esm/loader:163:28)
at ESMLoader.resolve (node:internal/modules/esm/loader:835:30)
at ESMLoader.getModuleJob (node:internal/modules/esm/loader:424:18)
at ModuleWrap. (node:internal/modules/esm/module_job:77:40)
at link (node:internal/modules/esm/module_job:76:36)
error: script "dev" exited with code 1

Here : https://github.com/stitionai/devika/issues/36#issuecomment-2015688337


the '^' doesn't work

I want to find file name starts whit "utils",so i use "ag -g '^utils'  ",but it finds nothing,the '^' doesn't work

oh I get it,I need to add './'


Move app dependencies to requirements.txt

Many developers expect a requirements.txt file to be present in a project in order to get a quick overview of the application's dependencies and to be able to do a light-weight install of an application for development via pip install -r requirements.txt.
Moving the dependencies out of setup.py also increases separation of concerns in the project and it's very simple to parse requirements.txt in setup.py, so really there's little downside to moving the dependencies into its own file.

Thanks @c-w


Boson bath construction

This adds a general interface for the construction and use of bosonic baths within embedding calculations, along with support for these new options within EWF. EDMET isn't yet rewritten to make use of this new interface, but will be in future and the approach used there remains supported. Examples and tests for the new functionality are included.

Saw that there was a comment from Ollie about multiple ways to get the eris when needed. If it is not appropriate to use a particular interface due to it's lack of support for some functionality, can you please leave a comment in the code at the appropriate place with a TODO to harmonise the interfaces in the future with the required functionality. This should make it easier to understand the limitations for future developers.

This is a fundamental constraint on the pyscf cderi routines for pbc vs molecular systems, rather than an interface limitation on our side to be fixed/harmonised later. The pbc routines have to return sign information that the molecular ones don't- we could wrap this and generalise, but there aren't major gains to code reuse or similar to be had here.


Show cookie banner text in French

Hi, many thanks for this gem.
Just a quick question: is there a readme file that explains what configuration should be made to have the text shown in French?
Many thanks.

Hello,
Set the default locale in your application.rb to French.
config.i18n.default_locale = :fr


Navigation tabs

We now have a sidebar, with tabs for the lists.

Well this was a big challenge. Finally LGTM!


Handle the parsing for different commands (v1.2)

Parsing for:

[x] Add Command
[x] List Command
[x] Delete Command
[x] Find Command
[x] Edit Command
[x] View Command

Should be all done for v1.2!


arch option of cli does support multi values on windows.

Pre-flight checklist

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project uses.
[X] I have searched the issue tracker for a bug that matches the one I want to file, without success.

Electron Forge version
^6.0.4
Electron version
22.0.0
Operating system
Microsoft Windows [Version 10.0.22621.1485]
Last known working Electron Forge version
No response
Expected behavior
Execute yarn make --arch x64,ia32 successfully. As the doc saied.

Target architecture to make for. Defaults to the arch that you're running on (the "host" arch). Allowed values are: "ia32", "x64", "armv7l", "arm64", "universal", or "mips64el". Multiple values should be comma-separated.

Actual behavior
When I tried to package both 32 and 64 bit app on windows using yarn make --arch x64,ia32. It failed with the following errors.
[DATA] "D:\code\node\cursor\node_modules\node-pty\build\binding.sln" (default target) (1) ->
[DATA] "D:\code\node\cursor\node_modules\node-pty\build\conpty_console_list.vcxproj" (default target) (3) ->
[DATA]   C:\Users\Liu.D.H\.electron-gyp\22.0.0\include\node\v8-internal.h(128,27): error C2338: static_assert failed: 'Pointer compression can be enabled only for 64-bit architectures' [D:\code\node\cursor\node_modules\node-pty\build\conpty_console_list.vcxproj]
[DATA]   C:\Users\Liu.D.H\.electron-gyp\22.0.0\include\node\v8-internal.h(222,46): error C2338: static_assert failed: 'The minimum reservation size for a sandbox must be larger than the pointer compression cage contained within it.' [D:\code\node\cursor\node_modules\node-pty\build\conpty_console_list.vcxproj]
[DATA]   C:\Users\Liu.D.H\.electron-gyp\22.0.0\include\node\v8-internal.h(231,44): error C2338: static_assert failed: 'The maximum allowed buffer size must not be larger than the sandbox's guard regions' [D:\code\node\cursor\node_modules\node-pty\build\conpty_console_list.vcxproj]
[DATA]   C:\Users\Liu.D.H\.electron-gyp\22.0.0\include\node\v8-internal.h(236,48): error C2338: static_assert failed: 'The maximum size of a BoundedSize must be synchronized with the kMaxSafeBufferSizeForSandbox' [D:\code\node\cursor\node_modules\node-pty\build\conpty_console_list.vcxproj]
[DATA]   C:\Users\Liu.D.H\.electron-gyp\22.0.0\include\node\v8-internal.h(260,56): error C2338: static_assert failed: 'kExternalPointerTableReservationSize and kExternalPointerIndexShift don't match' [D:\code\node\cursor\node_modules\node-pty\build\conpty_console_list.vcxproj]
[DATA]
[DATA]
[DATA] "D:\code\node\cursor\node_modules\node-pty\build\binding.sln" (default target) (1) ->
[DATA] "D:\code\node\cursor\node_modules\node-pty\build\deps\winpty\src\winpty-agent.vcxproj" (default target) (4) ->
[DATA] (Link target) ->
[DATA]   LINK : fatal error LNK1104: cannot open file 'C:\\Users\\Liu.D.H\\.electron-gyp\\22.0.0\\x64,ia32\\node.lib' [D:\code\node\cursor\node_modules\node-pty\build\deps\winpty\src\winpty-agent.vcxproj]
[DATA]
[DATA]
[DATA]
[DATA] "D:\code\node\cursor\node_modules\node-pty\build\binding.sln" (default target) (1) ->
[DATA] "D:\code\node\cursor\node_modules\node-pty\build\deps\winpty\src\winpty.vcxproj" (default target) (5) ->
[DATA]   LINK : fatal error LNK1104: cannot open file 'C:\\Users\\Liu.D.H\\.electron-gyp\\22.0.0\\x64,ia32\\node.lib' [D:\code\node\cursor\node_modules\node-pty\build\deps\winpty\src\winpty.vcxproj]
[DATA]
[DATA]     20 Warning(s)
[DATA]     12 Error(s)
[DATA]
[DATA] Time Elapsed 00:00:42.90
[DATA]
[FAILED] node-gyp failed to rebuild 'D:\code\node\cursor\node_modules\node-pty'.
[FAILED] For more information, rerun with the DEBUG environment variable set to "electron-rebuild".
[FAILED]
[FAILED] Error: `C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\MSBuild.exe` failed with exit code: 1
[FAILED]
[FAILED]
[FAILED] node-gyp failed to rebuild 'D:\code\node\cursor\node_modules\node-pty'.
[FAILED] For more information, rerun with the DEBUG environment variable set to "electron-rebuild".
[FAILED]
[FAILED] Error: `C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\MSBuild.exe` failed with exit code: 1
[FAILED]
[FAILED]
[FAILED] node-gyp failed to rebuild 'D:\code\node\cursor\node_modules\node-pty'.
[FAILED] For more information, rerun with the DEBUG environment variable set to "electron-rebuild".
[FAILED]
[FAILED] Error: `C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\MSBuild.exe` failed with exit code: 1
[FAILED]
[FAILED]
[FAILED] node-gyp failed to rebuild 'D:\code\node\cursor\node_modules\node-pty'.
[FAILED] For more information, rerun with the DEBUG environment variable set to "electron-rebuild".
[FAILED]
[FAILED] Error: `C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\MSBuild.exe` failed with exit code: 1
[FAILED]
[FAILED]

An unhandled rejection has occurred inside Forge:
Error: node-gyp failed to rebuild 'D:\code\node\cursor\node_modules\node-pty'.
For more information, rerun with the DEBUG environment variable set to "electron-rebuild".

Error: `C:\Program Files\Microsoft Visual Studio\2022\Community\MSBuild\Current\Bin\MSBuild.exe` failed with exit code: 1
at NodeGyp.rebuildModule (D:\code\node\cursor\node_modules\@electron\rebuild\lib\src\module-type\node-gyp.js:120:19)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async ModuleRebuilder.rebuildNodeGypModule (D:\code\node\cursor\node_modules\@electron\rebuild\lib\src\module-rebuilder.js:98:9)
    at async ModuleRebuilder.rebuild (D:\code\node\cursor\node_modules\@electron\rebuild\lib\src\module-rebuilder.js:128:14)
    at async Rebuilder.rebuildModuleAt (D:\code\node\cursor\node_modules\@electron\rebuild\lib\src\rebuild.js:149:13)
    at async Rebuilder.rebuild (D:\code\node\cursor\node_modules\@electron\rebuild\lib\src\rebuild.js:112:17)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Steps to reproduce

set DEBUG=electron-rebuild
yarn make --arch x64,ia32

Additional information
No response

Now I have to execute both yarn make --arch x64 and yarn make --arch ia32 separately to package the two targets.

While debugging the forge "make" task, I discovered that commander does not receive commas (,) in arguments on Windows. This is because, in the Windows command line, the comma (,) is a special character. Therefore, when passing arguments that contain commas, they need to be enclosed in quotes.
So, on Windows, if you want to pass multiple values, they should be enclosed in quotes.
For example: yarn make --arch "x64,ia32"
Lastly, I believe this usage, where multiple values on Windows need to be enclosed in quotes, should be documented.
@erickzhao

https://github.com/electron-forge/electron-forge-docs/pull/137


[BUG] getInlineAnchorList - when 2 elements are displayed, the first one looses its

What happened?
I am adding a small button bar on top of a gmail compose view using this line (full code in the end):
export const getInlineAnchorList = () => document.querySelectorAll(".bAs");
I use getInlineAnchorList because the compose view can appear in 2 more variations: a modal and a bottom drawer, and I want all of them to have this button bar injected.
Here is how it looks like in a reply to email view, notice the blue bar on the left, and the

Please see this example for mounting emotion style onto multiple csui: https://github.com/PlasmoHQ/examples/blob/main/with-mantine/contents/multiple.tsx
(it uses Mantine but the principle applies - if you need it to be available in the plain emotion example, let us know (or feel free to send us a PR :D )

Thank you


[NTS Radio] Live stream keeps dropping

Extension
https://www.raycast.com/arjanski/nts-radio
Description
This extension is great and I would use it daily. However, when streaming live, QuickTime terminates unexpectedly. It happens each time, crashing a few minutes after launch. I haven't been able to pinpoint a regular pattern or anything else that could be causing it.
Steps To Reproduce
[Raycast Version 1.52.1 on MacOS 12.5.1] Run the 'Live 1' command.
Current Behaviour
QuickTime terminates unexpectedly.
Expected Behaviour
QuickTime should run until I close the stream.

Hello @martindisley , great to hear that you like the extension and that you want to use it. I've not had the issue with QT myself, so I'll have to see what I can do about.
Generally, one could also e.g. use the Apple Music app to play the stream. I chose QT for start up performance, but it wouldn't be hard to let users pick in the extension settings. If I cannot reproduce/fix the QT issue you are describing, would that be an interesting workaround for you?
I'll also have a deeper look into the whole streaming topic (something I haven't really wrapped my head around before).

Hi @arjanski, thanks investigating this so quickly! I think that would be a suitable workaround. Personally speaking, that might even improve the functionality for me as I regularly also use the Apple Music app.


Revert "CAV-328: Non mobile numbers cannot be verified via UI"

Reverts hmrc/cip-phone-number-frontend#16

I guess this PR can be closed?


How can I perform inference using a model I trained

I am trying to use my trained model to infer object detections in images.
By looking at the code (mainly the one using to produce image summaries in tensorboard) I managed to gather all functions needed for this task. Nevertheless early results seem a bit peculiar and I would like to know that I didn't do any mistake.
Basically I am reimplementing the model, load the weights, read an image and finally use model.predict() to get the object detection results.
Image is loaded via:
    img_path = 'path/to/an/image'
    img = image.load_img(img_path, target_size=(height, width))
    x = image.img_to_array(img)
    x = np.expand_dims(x, axis=0)

Something like:
    y_pred = model.predict(x)
    all_filtered_boxes, all_filtered_classes, all_filtered_scores = filter_batch(y_pred, cfg)

which produce a y_pred of shape (16848,12) so I pass it through filter_batch to acquire  usable results, i.e. bboxes, classes and scores.
Is this the valid method to do it? Did I miss something in the process?

Any luck on this? I am trying to run inference but the results are bad. It predicts multiple bounding boxes despite choosing a weight file with negligible validation loss. Tried to use this method as well as opencv imread and keras preprocessing load_img function but there seems to be incorrect detections.


wrong u-boot filename for banana-pi if installed from ports

I tried to create a banana-pi image with crochet and installed the u-boot for banana-pi from the ports tree. The setup.sh looks for sunxi-spl.bin, but there is only a u-boot-sunxi-with-spl.bin file.

Can you please submit a Pull Request with the fix?  Thanks.

I believe this is fixed by #199.

wouldn't it be easier to understand/debug if you would specify the u-boot file name somewhere depending on what was found, log that and then use dd once? As far as i can see, in the merged pull request #199 you use dd twice, what could case the boot loader to be written twice, if for example the package and the port are installed.

...  if you would specify the u-boot file name somewhere depending on what was found, log that and then use...

Could you send us a pull request showing exactly how you think that could be implemented?

... you use dd twice ...

I'm not sure I understand.  Are you talking about these two lines?
dd if=${UBOOT_PATH}/u-boot-sunxi-with-spl.bin conv=notrunc,sync of=/dev/${DISK_MD} bs=1024 seek=8
dd if=${UBOOT_PATH}/u-boot.img conv=notrunc,sync of=/dev/${DISK_MD} bs=1024 seek=40

Those are installing different things to different places in the image.


Rename bigcode large-model to starcoder. Update entries in the cache.

https://huggingface.co/bigcode

Will you have time to do the update or would you like help?

Will you have time to do the update or would you like help?

I can make the code change. Do you mind just updating the Mongo cache after? We should have starcoder results for most scenarios except maybe just a few.


Do not render reserved title whitespace in vega_card if the title is empty

Even though the title is empty, in the vega_card there is still a whitespace rendered, which is just increasing unused space. This may be true for other cards as well.
Describe the solution you'd like
Do not render reserved title whitespace if the title is empty.

Hi @bkos , I am interested to resolve the issue. can I start working on this?

Hi @FahadYameen, as far as I'm concerned - sure, go for it 👌

Hi @FahadYameen , Are you still working on this? I would like to give this ago - thanks

Hey I have worked on around 7-8 company based project, but never countribute to open source this is my first time can you assign me this task.


feat: database initialization in local env

To ease the development, there should be a way to populate the local database with questions and answers.

@drodil Were you thinking about a pre-populated database for development? Or some route/button ta generates dummy data?

Yes exactly something like that. It always takes time to get to work with this plugin especially with memory only database, so this kind of feature would be very helpful.

There's now testdata.sql in the root but it must be manually run and doesn't contain that much data yet. But it's a start. Maybe it would be nice to have a script to run it in from the command line..


[UIMA 6273] isBeforeV3 variable not reset in BinaryCasSerDes

Suggested fix for issue UIMA 6273. This is critical to applications that have mixed loading of v3 and before v3 CAS'es in runtime environments such as the CPE where the CAS objects are recycled with reset.

@mjunsilo Is it feasible to have a unit test for this?

I am still thinking about how a unit test could be done, but I am short on time at the moment. A proper integrated test requires a solution where a v2 binary is loaded from test resources followed by a CAS reset, load of CAS v3 and final verification of the CAS. Problem is that I have to reproduce the error first with a small enough example before applying the fix, so there can be a fair amount of work to get to the point. The slightly simpler version would be just to check that the isBeforeV3 variable is probably set before and after v3 load, v2 load and reset instead of verifying the whole contents. I also had a quick look at the existing tests like "org.apache.uima.cas.impl.SerDesTest6", but I need to spend more time understanding how they work, before I could say that I could reuse some of that.

@mjunsilo In general, the fix looks good to me. It would be nice if we had a clearer idea under which circumstances not clearing this flag actually causes a problem so we could prep a corresponding CAS with UIMAv2 and bundle it up with a unit test loading this CAS in UIMAv3. I didn't yet look at what the flag actually does control. Do you know what it does?

I actually don't have detailed understanding of this flag, except from what I have picked up from the comments. Marshall introduced it in 2016 with the following original comment "If true, adjust type codes in serialized forms to allow v2 type codes to be correctly read by v3 impls, because v3 impls have a greater number of built-in types". One year later this comment was just reduced to "used to calculate total heap size". The variable is only used in BinaryCasSerDes6.deserializeAfterVersion where it appears to be used in some heap calculations, but my understandings of the low level workings of the CAS are too limited to understand what exactly happens.  Obviously there seems to be a difference between v3 and v2 in heap allocation that the deserialization must take account.
As already pointed out, it is currently not vital to clear this flag with the changes I made, and adding unit tests that captures the issue would help. My problem is that I don't know how the use of this flag changes in the future, so I currently feel more safe with the clearing as an extra insurance.
I am not sure I can reproduce the exact same problems as those we had with a small test example. We experienced two different behaviours for v2 and v3 CAS when this flag was incorrectly set. Deserialization would fail with an EOF exception in the case of v2, but it would succeed with v3, except that most annotations were missing.
Currently these problems only occur when you are running an application in the CPE, or similar environment where the CAS is recycled using reset, and both v2 and v3 CAS versions will be loaded during the lifespan of the application. In our case this is an elaborate integration test, and I initially thought the problem was maybe more general until I found the root cause and a fix.

@mjunsilo I have generated a few serialized CASes using the UIMAv2 CasIOUtilsTest and create a unit test trying to load these classes in UIMAv3. Without your changes, none of the form-6 files can be loaded. With your change, they load and the number of annotations in the loaded CAS matches the expected number.

Awesome Richard! I wasn't sure how to approach this exactly myself, and I couldn't find the extra time to mess around with it, but I will note to myself for another time that CasIOUtilsTest and the CasCreationUtils that it uses are very handy helpers for generating CAS test cases.


KSECURITY-792: Upgrade from Scala 2.13.8 to 2.13.10

JIRA: https://confluentinc.atlassian.net/browse/KSECURITY-792
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Can we fix the compilation issues in :streams:streams-scala here and other PRs


Updated nl.json

As in response of #585
As for the future, you should probably add a version counter to the translations file. To notice what changed. I now had to copy from the previous file into the new file which had to be done manually. Do you know a way to see the changes easily?
Thanks in advance @ludeeus

Do you know a way to see the changes easily?

Not yet, but I'm looking into options.
I'll probably have my bot create a new issue when new keys are added to en.json


The MSTest runner fails to run when path contains spaces

Create a test project in the folder with a space, for example, New Folder. Configure it to use the new MSTest runner

<PropertyGroup>
  <OutputType>Exe</OutputType>
  <EnableMSTestRunner>true</EnableMSTestRunner>
  <TestingPlatformDotnetTestSupport>true</TestingPlatformDotnetTestSupport>
</PropertyGroup>

Execute dotnet test TestProject1.csproj while in the project folder.
Observe the error

C:\Users\xxx\.nuget\packages\microsoft.testing.platform.msbuild\1.0.2\buildMultiTargeting\Microsoft.Testing.Platform.MSBuild.targets(228,5): error : Tests failed: The application to execute does not exist: 'C:\New' [C:\N
ew folder\TestProject1\TestProject1.csproj]

Thanks for reporting this I confirm the issue cc: @Evangelink


Need way to signal an entire pipeline is done

See if an extra, optional sync.WaitGroup can be tied into the pipeline via the config.

This was done in #23.


ERROR: No matching distribution found for ansible==5.1.0

For some reason, my requires.io watch wasn't active anymore. I enabled it again, received #53 and pipeline fails with following error:
ERROR: No matching distribution found for ansible==5.1.0

followed by
error: failed to solve: process "/bin/sh -c apk add --update --no-cache     --virtual .build-deps     make     gcc     python3-dev     && pip install --no-cache-dir --upgrade pip     && pip install --no-cache-dir -r requirements.txt     && apk del .build-deps" did not complete successfully: exit code: 1

https://github.com/GabLeRoux/docker-ansible/runs/4780013696?check_suite_focus=true
:shrug: I'll investigate this a bit.

Also I just learned that my automerge action doesn't care if the pipeline succeeded or not when adding the dependencies label :eyes:. Reverting previous merge.


Problemo

This takes section needs a fix. Would be great
Thanks for your work.

Not possible without breaking upgrade planner.

But we cannot see the item stats, this is in itself broken.

i dont care.


Show assert result details under Asserts tab in Response section

Below the summary of assert success/failure, show expression, comparator, actual and expected value. Refer screenshot.

Could not find screenshot for reference.Please attach

Issue has already been assigned to @rka12

Attached the design.

Fixed in PR #325 .


Update CosmosDBAccountProvider.ts

Hello, the next release for the azure account extension has sovereign support, and to work with those changes this extension needs to include the base url when creating the cosmosdbManagementClient.
I have gone through testing; please let me know what needs to be done to get these changes merged in.
Thank you!

Hello, the next release for the azure account extension has  sovereign support, and to work with those changes this extension needs  to include the base url when creating the cosmosDBManagementClient. I have gone through testing; please let me know what needs to be done to get these changes merged in. Thank you!

Looks good to me. I'll wait for @StephenWeatherford's thoughts before merging.

There seems to be a build error.
It looks like you're passing the wrong number of arguments into getCosmosDBManagementClient in one of the calls.

@PrashanthCorp  any other changes needed?


[Polish] Keep unified lowercase format

Keep unified lowercase format

Coverage decreased (-0.009%) to 87.302% when pulling 9d07e92efb0cc598d66363d65ee3e8f400308996 on Oliverwqcwrw:master-lowercase-alias-19 into a599fdc30d73bf9fd3c925b1b66c2838c34aef17 on mybatis:master.

it seems right but maybe the change would be unfriendly to the old project used MyBatis.

See what other people think, if everyone feels unfriendly, I will close PR

PTAL


[Bug]: Streaming response drops first token when identical to prompt

What happened?
I've noticed an odd behavior with the response after updating to the latest commit(326069d). When AI's response starts with the same token as the user's prompt, that first token is not displayed while streaming.
Steps to Reproduce

Enter the prompt "Hello" (without quotes)
If the AI's response starts with "Hello", the streaming output will skip this first word

For example, if the full response is "Hello! How can I assist you today?", the streaming output shows:
!
! How
! How can
! How can I assist you today?
Hello! How can I assist you today?

What browsers are you seeing the problem on?
Chrome
Relevant log output
No response
Screenshots
No response
Code of Conduct

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

Fixed, thanks!


suggestion-page columns not consistent

The columns in the rows are not consistent in their rows.

fixed this by adding before and after appending of dom elements in structure.


Relaxed some query ordering assertions in expressions tests.

It accounts for differences seen on CockroachDB.

Thanks Tim :+1:


NoMethodError when running RSpec (ActionController::TestCase)

Re-opening @caulfield's #7 for investigation
All my controller(and acceptance) specs was broken with error
> undefined method `as' for nil:NilClass

I'm using rspec-rails 2.13.2 and rails 4.0.0.

I found out that ActionController::TestCase (rspec uses it) instance fire `#params=` twice. nil first and expected params then.

I am not sure it's acrance bug, but I think we should handle nil and Arcane::Parametrs'able variables

Is there an ETA for this? My whole test suite is broken...

Ping


Something went wrong

I followed the whole deployment process and had no issues. When I want to install the app in Teams however, I get the error message "Something went wrong"
Which is not helpful at all

The package manifest run through without any errors, so I guess the issues can't be in the manifest, right?
I have no idea where to look

Thank you for submitting this issue, as we work to ensure consistent responses with items logged in our issues list, we are working to address these items that did not get responses. Please let us know if you are still facing this issue and require assistance and we will be happy to discuss. If no further action is needed at moment and no response is received, we will close out the issue in 3 business days. You may also always open a new issue or respond here if something does come back up. Thank you!


Can we set custom play and pause buttons?

At the moment it is not possible. I will implement this feature in the next version of the library.

Starting from v2.2.0 you can hide the play/pause button in the player.


RuntimeError: Unable to connect to Redis.

error:
Traceback (most recent call last):
File "J:/PyCharm项目/package_test_/modin测试/modin测试.py", line 7, in
from modin.pandas import pandas as pd
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\modin\pandas_init_.py", line 280, in
execution_engine.subscribe(update_engine)
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\modin_init.py", line 93, in subscribe
callback(self)
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\modin\pandas_init_.py", line 253, in update_engine
initialize_ray()
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\modin\pandas_init.py", line 199, in initialize_ray
lru_evict=True,
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\ray\worker.py", line 723, in init
ray_params=ray_params)
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\ray\node.py", line 180, in init
self.start_head_processes()
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\ray\node.py", line 681, in start_head_processes
self.start_redis()
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\ray\node.py", line 502, in start_redis
fate_share=self.kernel_fate_share)
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\ray\services.py", line 763, in start_redis
fate_share=fate_share)
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\ray\services.py", line 957, in _start_redis_instance
wait_for_redis_to_start("<IP_ADDRESS>", port, password=password)
File "D:\Users\Administrator\AppData\Local\Programs\Python\Python37\lib\site-packages\ray\services.py", line 562, in wait_for_redis_to_start
raise RuntimeError("Unable to connect to Redis. If the Redis instance "
RuntimeError: Unable to connect to Redis. If the Redis instance is on a different machine, check that your firewall is configured properly.

Hi @reporter-law, thanks for the report!
This is an issue with initializing Ray, there is a similar issue on the Ray repo found here: https://github.com/ray-project/ray/issues/4931.

I am currently running into the same issue. I'd like to switch to modin with dask for the moment, how can I make the switch?

I am also running into the same issue. I'm running on Windows 10, and specifying that modin should be running dask as the MODIN_ENGINE. I would expect this issue to be avoided by using dask.
(Python 3.8.2)
I have included several examples of what I've tried here:
Example 1:

import os
os.environ["MODIN_ENGINE"] = "dask"  # Modin will use Dask
import modin.pandas as pd
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin\pandas_init_.py", line 182, in
execution_engine.subscribe(update_engine)
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin_init.py", line 93, in subscribe
callback(self)
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin\pandas_init_.py", line 118, in _update_engine
initialize_ray()
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin\engines\ray\utils.py", line 139, in initialize_ray
ray.init(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\worker.py", line 696, in init
_global_node = ray.node.Node(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\node.py", line 198, in init
self.start_head_processes()
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\node.py", line 800, in start_head_processes
self.start_redis()
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\node.py", line 568, in start_redis
process_infos) = ray.services.start_redis(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\services.py", line 739, in start_redis
port, p = _start_redis_instance(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\services.py", line 937, in _start_redis_instance
wait_for_redis_to_start("<IP_ADDRESS>", port, password=password)
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\services.py", line 552, in wait_for_redis_to_start
raise RuntimeError("Unable to connect to Redis. If the Redis instance "
RuntimeError: Unable to connect to Redis. If the Redis instance is on a different machine, check that your firewall is configured properly.

Example 2:

from distributed import Client
client=Client(n_workers=8)
import modin.pandas as pd
Traceback (most recent call last):
File "", line 1, in
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin\pandas_init_.py", line 182, in
execution_engine.subscribe(update_engine)
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin_init.py", line 93, in subscribe
callback(self)
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin\pandas_init_.py", line 118, in _update_engine
initialize_ray()
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\modin\engines\ray\utils.py", line 139, in initialize_ray
ray.init(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\worker.py", line 696, in init
_global_node = ray.node.Node(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\node.py", line 198, in init
self.start_head_processes()
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\node.py", line 800, in start_head_processes
self.start_redis()
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\node.py", line 568, in start_redis
process_infos) = ray.services.start_redis(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\services.py", line 739, in start_redis
port, p = _start_redis_instance(
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\services.py", line 937, in _start_redis_instance
wait_for_redis_to_start("<IP_ADDRESS>", port, password=password)
File "C:\Users\Ian.conda\envs\pytrade\lib\site-packages\ray\services.py", line 552, in wait_for_redis_to_start
raise RuntimeError("Unable to connect to Redis. If the Redis instance "
RuntimeError: Unable to connect to Redis. If the Redis instance is on a different machine, check that your firewall is configured properly.

Sorry this got dropped, looking over past issues I found this.
We have some updated approach of handling the engine updates in #2580. It should not still cause this problem. Are you able to check? You will need to install from github master: https://modin.readthedocs.io/en/latest/installation.html#installing-from-the-github-master-branch

Sorry this got dropped, looking over past issues I found this.
We have some updated approach of handling the engine updates in #2580. It should not still cause this problem. Are you able to check? You will need to install from github master: https://modin.readthedocs.io/en/latest/installation.html#installing-from-the-github-master-branch

@devin-petersohn I too ran into the same issue using stable version installed with pip install modin[ray]. Later as suggested in above comment I installed modin from the master branch and followed by installation of ray. But still run into the same issue.

I am using modin inside a python script, not notebook

Installed packages:

python 3.7.1
modin  0.8.3+31.g77d40ce
ray 1.1.0
redis 3.5.3

Please find below the complete error stack trace

Error Stacktrace

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "D:\ProgramData\Anaconda3\envs\denali\lib\concurrent\futures\process.py", line 232, in _process_worker
    r = call_item.fn(*call_item.args, **call_item.kwargs)
  File "D:\ProgramData\Anaconda3\envs\denali\lib\concurrent\futures\process.py", line 191, in _process_chunk
    return [fn(*args) for args in chunk]
  File "D:\ProgramData\Anaconda3\envs\denali\lib\concurrent\futures\process.py", line 191, in <listcomp>
    return [fn(*args) for args in chunk]
  File "F:\DENALI\denali\scripts\market_analyzer.py", line 605, in _helper
    prev_output_dict=analyzer_input[8])
  File "F:\DENALI\denali\scripts\market_analyzer.py", line 1493, in analyze
    return mpd.DataFrame()
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\modin\pandas\dataframe.py", line 90, in __init__
    Engine.subscribe(_update_engine)
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\modin\config\pubsub.py", line 107, in subscribe
    callback(cls)
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\modin\pandas\__init__.py", line 114, in _update_engine
    initialize_ray()
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\modin\engines\ray\utils.py", line 159, in initialize_ray
    _lru_evict=True,
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\ray\worker.py", line 722, in init
    ray_params=ray_params)
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\ray\node.py", line 216, in __init__
    self.start_head_processes()
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\ray\node.py", line 767, in start_head_processes
    self.start_redis()
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\ray\node.py", line 597, in start_redis
    fate_share=self.kernel_fate_share)
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\ray\_private\services.py", line 843, in start_redis
    fate_share=fate_share)
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\ray\_private\services.py", line 989, in _start_redis_instance
    wait_for_redis_to_start("<IP_ADDRESS>", port, password=password)
  File "D:\ProgramData\Anaconda3\envs\denali\lib\site-packages\ray\_private\services.py", line 651, in wait_for_redis_to_start
    redis_ip_address, redis_port)) from authEx
RuntimeError: Unable to connect to Redis at <IP_ADDRESS>:6379.
"""

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File ".\scripts\market_analyzer.py", line 1607, in start
    current_prices_dict=current_prices_json
  File ".\scripts\market_analyzer.py", line 620, in execute
    run_df_list = list(pool.map(self._helper, params))
  File "D:\ProgramData\Anaconda3\envs\denali\lib\concurrent\futures\process.py", line 476, in _chain_from_iterable_of_lists
    for element in iterable:
  File "D:\ProgramData\Anaconda3\envs\denali\lib\concurrent\futures\_base.py", line 586, in result_iterator
    yield fs.pop().result()
  File "D:\ProgramData\Anaconda3\envs\denali\lib\concurrent\futures\_base.py", line 432, in result
    return self.__get_result()
  File "D:\ProgramData\Anaconda3\envs\denali\lib\concurrent\futures\_base.py", line 384, in __get_result
    raise self._exception
RuntimeError: Unable to connect to Redis at <IP_ADDRESS>:6379.

Additional Details :

OS : Windows 10

I tried using dask engine for windows as recommended but run into a different issue as reported here, so I tried to use ray engine

Also I would like to know is modin suitable for use in standalone scripts or python modules ?

@bhavyalatha26 Thanks for the update!
Can you tell me if this works for you on windows?:
import ray
ray.init()

Windows support for Ray is still fairly new, so any help we can give the Ray team will be great!

Also I would like to know is modin suitable for use in standalone scripts or python modules ?

Yes, it is. This comment in the issue you show is how to do that with Dask: https://github.com/modin-project/modin/issues/843#issuecomment-761625592
Unfortunately, this is a property of Dask that cannot be worked around because of how it spawns new workers. Ray does not use the same mechanism for spawning new processes, so it does not have the same requirements.

There is another Ray on Windows bug with similar symptoms https://github.com/ray-project/ray/issues/13213

@gshimansky Can you please recheck quickly?

I checked windows version on current master branch (commit f9c99c3817d35f7bf0d91df9aa9644545ea27517) with Ray 1.4.0 now installed from pip. Statement from modin import pandas as pd no longer triggers Ray initialization but I made sure that Ray is initialized by creating a modin Series object and Ray initialization works fine.
I think we can close this issue.


onPaste does not fill all selected rows

Currently, I am working on a table with a company that expects almost identical Excel behavior. One missing thing is copying and pasting one row into multiple selected rows. Is this a feature that you intend to add or is this something you might delegate to devs using your library to implement themselves due to some technical complexity/edge cases?
Also if I am not mistaken, onPaste is currently working with onCellEdited, is this something that can be moved to onCellsEdited? That would greatly improve DX

In my quick test onCellsEdited gets called for both onPaste and fillHandle

In my quick test onCellsEdited gets called for both onPaste and fillHandle

that is good to know. thanks @pzcfg !

So sounds like this is not an issue?

Implementation proposal:
interface DataEditorProps {
  ...otherCrap,
  onPasteBehavior: undfined | {
    demuxRows: boolean,
  }
}

I dont understand why you think that would be better.

Now we have to maintain both onCellsEdited and onCellEdited. Sure, we can prevent onPaste from triggering onCellEdited but IMO onCellEdited is kinda redundant anyway, since we have to maintain basically two functions to handle data change. I believe that it would be best if we could decide if we want to use one or the other, for example I would rather handle everything in onCellsEdited

onCellsEdited was updated in 5.0.1 to always trigger, you can not implement onCellEdited at all if you like. onCellEdited will always trigger if onCellsEdited triggered.

Ah well, in that case, everything is solved basically. Great stuff!

This is as good as its going to get for a bit in 6.0.0. It now handles ranges much better. We might do more improvements in the future with pattern filling and handlng demuxing.


Enable snapshots

Enable snapshots for testnet to enable validators to more quickly deploy nodes/run experiments

Overview

Checklist

[ ] New and updated code has appropriate documentation
[ ] New and updated code has new and/or updated testing
[ ] Required CI checks are passing
[ ] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

Try using a newer version of "go".

Okey, cool

I corrected it, check.

I think we need to work on fixing the "lint".

Yeah just did that now

Okey, cool.


Add Arcade.Body#updateFromGameObject

This PR

Adds a new feature

This extracts the relevant code from Body#preUpdate() as a new public method. It also lets refreshBody() work for dynamic bodies (previously it would error).
Rationale
After changing the body offset or sprite transform, you might want to read the body's new position and center immediately, before the next physics step.

👍


Do not set value of IMMER_{HAMTS,RBTS}_TAGGED_NODE if it is already set

Currently the data layout of the immer data types depends on the value
of the NDEBUG macro. This is rather unfortunate because if a library is
built with debugging disabled, and then linked into an application built
with debugging enabled, the code will crash with an assertion error.
Thus, I provide a way to explicitly set the layout intended for the
application so that it does not get overridden by the value of NDEBUG,
which allows the user to have a consistent data layout across multiple
code units with different values of debugging.

This is proving more complicated than I expected to get working, so I am going to close this PR and recreate it when I get it working.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #98   +/-   ##
=======================================
  Coverage   93.96%   93.96%
=======================================
  Files          89       89
  Lines        7837     7837
=======================================
  Hits         7364     7364
  Misses        473      473

Impacted Files
Coverage Δ

immer/detail/hamts/node.hpp
96.97% <ø> (ø)
:arrow_up:

immer/detail/rbts/node.hpp
95.39% <ø> (ø)
:arrow_up:

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 559ab37...d024460. Read the comment docs.

Cool. Thanks for your other PR!


removed duplicate in csv and updated readme

resolved issue #78 using https://en.wikipedia.org/wiki/The_Grand_Illusion as reference.

also resolves issue #70

Thanks!


Disable depot path for long runs

This PR disables the julia depot path for the long runs, since it's arguably less important for turnaround time.

bors r+


Update index.js

webview is not supported for latest react native.

Hi @ziggybaba1 , thanks for contributing but you altered the entire structure of the project and did not follow the rules for contributing.
Please check out the contribution guideline and make appropriate changes and I'll be happy to merge your PR. Once again, Thanks!


PSExec is not downloaded and executed from Windows host

Hello,
I am have the following test case :

Windows 10 build is 19041.76 in a domain.
For the MITM portion, I configured via gpedit the Wsus to be my pywsus server (running on Debian 10).
My pywsus  server receives a SyncUpdate and a GetExtendedUpdateInfo is triggered.
The windows host does not download my PsExec at http://<WSUS_IP>:8530/8c1dee0a-8dfa-48c9-b21f-9993ec978214/PsExec64.exe

[edit : I managed to have the GetExtendedUpdateInfo, once I joined a domain as you mentioned in another issue. Do you know, what are the registry keys involved ?]
Thanks for your time,

To be sure: In verbose mode, did you see a GET request for “/PsExec64.exe” or did the targeted host did not requested PsExec64.exe.
Thank you

The targeted host did not request PsExec64.exe and Windows update shows "device is up to date".
In gpedit.msc, "Configure Automatic Update" is set to " 3 = (Default setting) Download the updates automatically and notify when they are ready to be installed", so my understanding is that at least the download should occur.
Thank you,

Hi @GeeBay
Did you restart Pywsus to have new UUIDs ? If it is still not working, PsExec64 is probably cached by the client. I would suggest you try another signed Microsoft signed binaries and see if you receive the GET request.
I will try to reproduce your problem at soon I have some time.
Thank you.

Hi @GeeBay,
I have the same problem as you. I will continue to work on it.
Thank you.


Add ifilonenko & yuzliu as reviewer

What this PR does / why we need it:
ilan and yuzhui have made various contributions to KFServing and currently working on multi model serving they originally proposed.

@ifilonenko can you add yourself to the kubeflow org?

Welcome guys!
/lgtm
/approve

waiting on https://github.com/kubeflow/internal-acls/pull/305 so this could be merged

/verify-owners

@jlewi this is odd... i thought it merged on the kubeflow/internal-acls?

/verify-owners


How to install the development version of Poetry

Comment:
Actually this is documentation, not a real issue. But since the procedure is not obvious during this awkward period where the most useful Poetry versions are in pre-release, I figure that this is a quick and easy way to put instructions where people will find them. (Feel free to close this if you have a better idea.)
As described in https://github.com/conda-forge/poetry-feedstock/issues/57#issuecomment-1022805023, if you want to create a Conda environment called pdev which contains Poetry v1.2.0b1 together with the latest poetry-core, run
mamba create --yes --name pdev -c conda-forge/label/poetry_dev -c conda-forge/label/cleo_dev -c conda-forge "poetry=1.2.0b1"

where conda also works in place of mamba.
(Duplicate of https://github.com/conda-forge/poetry-feedstock/issues/64)

We finally got some buildable releases, so this is no longer necessary.


Text extraction

What does this Pull Request do?
Adds text extraction module to playbook.
What's new?
With text extraction in place images with OCR will have that text extracted and put into an editable media.
Any Original File with an Original File tag will also have text extracted into an editable media
How should this be tested?
After playbook is spun up create a node tagged as both Image and Digital document.
Add image media (containing test) tagged as Original File.
Extracted Text media should be created and attached to node.
Create another node and attach a media tagged with Original File and with a media type of application/pdf.
Extracted Text media should be created and attached to node.
Interested parties
@Islandora-Devops/committers

Box provisions ok and I can confirm the listeners are deploying as is the islandora_text_extraction module, but when uploading a PDF, I'm getting a white screen with this in the logs
Notice: Undefined offset: 0 in Drupal\islandora_text_extraction\Plugin\Condition\GeneratePdfTextCondition->evaluateEntity() (line 46 of /var/www/html/drupal/web/modules/contrib/islandora_text_extraction/src/Plugin/Condition/GeneratePdfTextCondition.php)

I was looking to test this as well.  Please modify the instructions/playbook to include the text-extraction play.  I vagrant up, then realized that I did not have the text-extraction.  I will hold on until Danny's comments are addressed.
Currently, only one model can be chosen.  Thus, not sure how to tag a node "as both Image and Digital document."
Questions

I assuming tagging something as "Image and Digital Document" will trigger text extraction.  Which is ok.  But, if we want to optionally not to do text extraction for a given individual image/document, how can we do that?

It would be better to enable the user to select if they want to do OCR, then select which service to use.  We can provide tesseract-ocr as the default, but if we can provide option to add other end points, that would be useful as tesseract-ocr does not support many languages well.

How does this info get indexed in Solr.  i.e how if we search for a word found in the extracted doc, would it bring back the node?

Thank you.

@Natkeeran - Good questions!
You have to change the field definitions for the repository item content type to allow more than one tag.  We had to do this because some images will be ocr'ed and some will not so double tagging allows the behaviours we want to apply to both images and digital documents.
(Ultimately I'd like to see a list of microservices as a taxonomy and on any given ingest you'd select the ones you want, but for now we're relying on media use tags)
If we come with other text extraction methods or microservices we'll just have to create actions.contexts to trigger those instead of these.
Both OCR and PDF text extraction create a media with an 'editable_text' field.  Index that field like any other in the solr configuration and it becomes solr searchable.

Linking to https://github.com/Islandora-CLAW/CLAW/issues/932

Testing and review of this PR to be done as part of our paged content sprint!

@ajstanley, @dannylamb  :  I'm testing the PR and running into some issues.  I tried to run the test, but after modifying the "Model" field to allow the user to check both Image and Digital Document,  Drupal said that it could not make the change to the database.  I tried to walk through the steps anyway, but the ocr file was not generated.  So I'm guessing Drupal was being serious about not being to update the database.
A few requests/questions:

Can you update the PR so that it includes the necessary changes to the Content model?
If for some reason that is not possible,  can you detail the steps for bringing the database into alignment to enable this to work?
Is it necessary for me to modify any of the playbooks or vars to get this to work?  If so, shouldn't the PR be updated first before trying to test it?

@dbernstein - You should be able to navigate to
http://localhost:8000/admin/structure/types/manage/islandora_object/fields/node.islandora_object.field_model/storage
and change allowed number of values to unlimited.

Getting the following error:
The website encountered an unexpected error. Please try again later.

Also, wondering if it is passing any language parameters to tesseract.

Superseded by https://github.com/Islandora-Devops/claw-playbook/pull/140


[DF] Add support for CREATE TABLE | VIEW AS statements

Adds support for CREATE TABLE AS and CREATE VIEW AS statements. Some follow up stuff to look into:

parsing the schema_name through DataFusion instead of hard-coding
parsing whether or not to persist created tables instead of hard-coding
consolidating the shared logic between CreateTableAsPlugin.convert and Context.sql in one utility function

Codecov Report

:exclamation: No coverage uploaded for pull request base (datafusion-sql-planner@f948429). Click here to learn what that means.
The diff coverage is n/a.

@@                    Coverage Diff                    @@
##             datafusion-sql-planner     #656   +/-   ##
=========================================================
  Coverage                          ?   66.03%
=========================================================
  Files                             ?       73
  Lines                             ?     3645
  Branches                          ?      759
=========================================================
  Hits                              ?     2407
  Misses                            ?     1087
  Partials                          ?      151

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[err] This profile was last used with a newer version of this application. Please create a new profile.

Unexpected playwright error after second reinstall
chatgpt version 0.4.1
jhyeo@jhyeoui-MacBookPro ~ % chatgpt
Traceback (most recent call last):
  File "/Users/jhyeo/Library/Python/3.9/lib/python/site-packages/chatgpt_wrapper/chatgpt.py", line 56, in create
    self.browser = await playbrowser.launch_persistent_context(
  File "/Users/jhyeo/Library/Python/3.9/lib/python/site-packages/playwright/async_api/_generated.py", line 14251, in launch_persistent_context
    await self._impl_obj.launch_persistent_context(
  File "/Users/jhyeo/Library/Python/3.9/lib/python/site-packages/playwright/_impl/_browser_type.py", line 155, in launch_persistent_context
    from_channel(await self._channel.send("launchPersistentContext", params)),
  File "/Users/jhyeo/Library/Python/3.9/lib/python/site-packages/playwright/_impl/_connection.py", line 44, in send
    return await self._connection.wrap_api_call(
  File "/Users/jhyeo/Library/Python/3.9/lib/python/site-packages/playwright/_impl/_connection.py", line 419, in wrap_api_call
    return await cb()
  File "/Users/jhyeo/Library/Python/3.9/lib/python/site-packages/playwright/_impl/_connection.py", line 79, in inner_send
    result = next(iter(done)).result()
  File "/Library/Developer/CommandLineTools/Library/Frameworks/Python3.framework/Versions/3.9/lib/python3.9/asyncio/futures.py", line 201, in result
    raise self._exception
playwright._impl._api_types.Error: Browser.enable): Browser closed.
==================== Browser output: ====================
<launching> /Users/jhyeo/Library/Caches/ms-playwright/firefox-1369/firefox/Nightly.app/Contents/MacOS/firefox -no-remote -headless -profile /tmp/playwright -juggler-pipe about:blank
<launched> pid=2324
[pid=2324][err] *** You are running in headless mode.
[pid=2324][err] This profile was last used with a newer version of this application. Please create a new profile.
[pid=2324] <process did exit: exitCode=0, signal=null>
[pid=2324] starting temporary directories cleanup

try rm -r /tmp/playwright then reinstalling

Closing due to inactivity.

try rm -r /tmp/playwright then reinstalling

This worked for me

There is a chatgpt reinstall command that wipes your existing profile, probably a little easier and cleaner.

great


curl: remove ? null

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

cc maintainer @lovek323

Not sure about removing the assertions. Might wanna keep them to prevent someone trying to enablr features during bootstrap?
I think the packages are null when undefined? Or what are they when undefined? Will it maybe cause eval errors?

If you don't have ? null in the inputs nix will complain if the packages are null.

What I thought: No rebuilds so this should be safe to do ™

Can you fix the conflicts?


The previous prompt gets partially deleted after using helix

Platform Windows 10
Terminal software Windows Terminal 1.15.3465.0, nu 0.74
When using helix and the current line is close to the bottom of the terminal window, the line invoking helix disappears when starting to type.
I haven't tested with other tui software.
Steps to reproduce

fill up the terminal window
open and exit helix
start typing

Screenshots/Screencaptures
out.webm

I think this could be related.
In the nu_scripts/benchmarks folder there is a benchmark called turtle.nu. If you run it, you get this:
benchmark { source turtle.nu }
Turtle 1 = 167
Turtle 2 = 464
Turtle 3 = 897
Turtle 4 = 1466
Turtle 5 = 1ms 184µs 583ns

Note that the line for Turtle 5 does not contain the timing for turtle 5, it contains the output of the benchmark command. This is what the output should look like.
benchmark { source turtle.nu }
Turtle 1 = 167
Turtle 2 = 464
Turtle 3 = 897
Turtle 4 = 1466
Turtle 5 = 2172
1ms 184µs 583ns


override ToString() in DTOs

it can be very useful for debugging to override ToString() in DTOs to make it show summary information
for example, ApiTradeHistoryDTO can show info like this:
Currency: "CAD", Direction: "buy", MarketId: 99524, Quantity	: 50
etc

Have a look at https://github.com/cityindex/CIAPI.CS/blob/master/src/CIAPI/PrettyPrinterExtensions.cs
The functionality you're after can be achieved with:
  MyDTOInstance.ToStringWithValues()


docs: convert index to md format

Part of #2691

coverage: 87.545%. remained the same when pulling 3aa0942fff1587eba7b746512ae0509b89629ffa on awxiaoxian2020:convert-index into 5d51f64b9cf81f73c235b79fcd2ca17e089728df on mybatis:master.

Thank you, @awxiaoxian2020 !


Always use windows server 2022 for dockerless workflow

LLDB is not working on the 2019 images for the vscode-swift tests

Issue: #1263

@shahmishal confirmed that's the issue, works downstream https://github.com/swiftlang/vscode-swift/pull/1291


[Bug] No artwork or previous artwork for songs with no artwork

If a song (I’m playing ones from Files, I don’t have any in Apple music so I cannot test that ¯\_(ツ)_/¯) has no artwork, it displays no artwork instead of the song icon, or if a song with an artwork was played before it, it retains that artwork.

Do you have a video or a screenshot of your problem?

I’ll take a video just now!

https://imgur.com/a/MvGh9L1

It's actually a problem with the file that you're playing (missing artwork, etc), and is not a problem of the tweak itself.

Yeah, but why doesn’t it set it to a blank artwork when there’s no artwork? It would be fine, other than this problem where it keeps the previous track’s artwork.


update travis

Changes Unknown when pulling c2d09003b6a3645ff40b03b41674a5761d7e6173 on travis into ** on master**.

Changes Unknown when pulling c674fa9cc8214edf7affef4cf253d2068f357137 on travis into ** on master**.

Changes Unknown when pulling dc29623c736b4d35863b1c7fa6b38f2542e8d16d on travis into ** on master**.