Adding plugin to project fails to install NPM dependency

Problem
pnpm tauri add process --verbose
> tauri "add" "process" "--verbose"
    Info [tauri_cli::helpers::cargo] Installing Cargo dependency "tauri-plugin-process"...
    Info [tauri_cli::helpers::npm] Installing NPM dependency "<EMAIL_ADDRESS>    Error [tauri_cli_node] Failed to install NPM dependency
 ELIFECYCLE  Command failed with exit code 1.

Reproduce
The issue is reproducable on a dummy project:
pnpm create tauri-app --rc
cd tauri-app
npm install
npm run tauri dev

Hypothesis
The script attempts to install<EMAIL_ADDRESS>Version 2.0.0-rc doesn't exist, but 2.0.0-rc.0 does (npm versions).

pnpm tauri add<EMAIL_ADDRESS>works.

This should be fixed in the next release, see https://github.com/tauri-apps/tauri/pull/10795


Duplicated entities

Sometimes, during the day, the sensors became unavailable.  I have to restart HA to get them working again. When the HA come back online, I have several duplicate entities, like the screenshot below

Any idea why this happens?  I usually remove the missing entities, but I have to reconfigure the Energy panel :-/

This is not directly related to the integration, try the HA forum. I found the following which might work: Delete all unavailable (orphan) items and restart Homeassistant. Now all is working as before. At least you don't have to reconfigure the Energy panel if this works. Another suggestion is to restore a backup.
Offcourse it is unwanted behaviour of HA and there is a lot of talk about this subject in the community.

If the sensors become unavailable it might be due to the ECU loosing WiFi connectivity, are both led's on the ECU green?

After configuring the ECU using WiFi, I keep it connected using only the ethernet cable. Does it need to be connected on WiFi, even if there is ethernet connection?

I found the following which might work: Delete all unavailable (orphan) items and restart Homeassistant. Now all is working as before

That is what I am doing... Removing orphan entities and restarting HA.

One of the possible reasons is MQTT auto discovery

How MQTT is related to this integration?

This component only works if the ECU-R or ECU-B is attached to your network by Wifi. But with the latest firmware you never now what feature is next due to missing release notes. This integration is based on communication between the ECUAPP and the temporary AP of the ECU when setting it up. Previously we found out that port 8899 is only available on WiFi not ethernet but that might have been changed. I'll experiment with that today.
Best practice is not to experiment with the integration or ECU when inverters are down. It does indeed create orphan entities.
MQTT is not related to this integration, it is mentioned in the forum as one of the possible causes of creating orphan entities. The message broker somehow autodiscovers entities it cannot map on excisting entities thus creating new entities. Sounds plausible to me.

Nope, ECU-R firmware ECU_R_1.2.19009 does not support the use of port 8899 on the ethernet port, it does with WiFi. Unfortunately, the integration still relies on a WiFi connected ECU.

This component only works if the ECU-R or ECU-B is attached to your network by Wifi. But with the latest firmware you never now what feature is next due to missing release notes.

How can I check the firmware version?  On my ECU, port 8899 is open on Ethernet
[dclobato@appserver] ~$ nc <IP_ADDRESS> 8899
APS1100160001END
APS12011700012162000068600▒ !0I(10015ECU_R_PRO_2.0.3017America/Sao_Paulo▒▒▒`Ũy▒00END
^C
[dclobato@appserver] ~$ arp <IP_ADDRESS>
Address                  HWtype  HWaddress           Flags Mask            Iface
<IP_ADDRESS>                ether   80:97:1b:02:a8:a4   C                     enp2s0
[dclobato@appserver] ~$

Best practice is not to experiment with the integration or ECU when inverters are down. It does indeed create orphan entities.

I'll try to remember this :)
Thanks

@dclobato So <IP_ADDRESS> is your ethernet connected IP-address of the ECU-R-Pro and 8899 is open on this connection? It still puzzles me reading your firmware version "ECU_R_PRO_2.0.3017" I can't find any Google results on ECU-R-Pro hardware/software besides our comments on this subject. But more on topic, duplicate entries are not really an issue which is caused by this integration, do you agree?

I didn't realize there might be errors in the log. Among other things, the length of the firmware string is 3 bytes longer. Something the integration does not take into account in de python code. What does home-assistant.log and home-assistant.log.1 say if the sensors are no longer available?

Have exact same issu here.
During the day or night sensors become unavailable and the only solution is to restart HA, restarting ECU doesn't have any effect.
After restarting i get duplicated entities.
Have same issue with wifi and wired connection

During the night, the inverters go offline. If you continue polling the integration has to rely on cached data from the last succesful pull, everything should just continue until the inverters come back online and the ECU can poll it again. Cached data is only present if there was at least one successful pull.
If this happens during the day, the integration was not successfull in pulling the first data so a cache could not be build. The error message can disappear by itself, so do not restart HA too quickly, the ECU is being polled once a minute by default.

In my case i can pull data from ECU in both cases when it connected via wifi and lan.
And in both cases after 4 to 6 hours sensors become unavailable. In first case i have waited for more than an hour with hope that connection will be resumed but with no succes. At the same time EMA app remain connected and also no connection drops from HA to ECU ip.
In first case i have restarted ECU to see if integration will pull data but it was not the solution, only way is to restart HA but then i get duplicated entities.
Also changed scan interval to 120 and 180 sec.
Al of the actions are done during the daylight to be sure that ECU have data from inverters

Ok, is it's ECU_R_PRO firmware? What are the results if you use sync io instead of async io?
The sync query closes the socket and can be used to try when you modify __init__.py at line 72
Replace data = await self.ecu.async_query_ecu() with data = self.ecu.query_ecu() and restart HA.

Where can i see firmware version?
I will try now to replace "data = await self.ecu.async_query_ecu() with data = self.ecu.query_ecu()" and report back
For now i have new error log from HA

In the Attributes section of your ECU Current Power Entity:

Have tried to replace "data = await self.ecu.async_query_ecu() with data = self.ecu.query_ecu()" and got this error after restarting HA

My firmware version is : ECU_R_PRO_2.0.3
Any ideas?

The ecu.query_ecu()function needs updates for the pro to work. The socket implementation is wrong. I know, because I have the same problem.

After each request the socket needs to be closed and an new socket opened.

It's hard to get a hold on the problem. @tv3 do you now use the query_ecu() function without problem? It's witten like:
`    def query_ecu(self):
    sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
    sock.connect((self.ipaddr,self.port))

    sock.sendall(self.ecu_info_query.encode('utf-8'))
    self.ecu_raw_data = sock.recv(self.recv_size)

    self.process_ecu_data()

    cmd = self.inverter_energy_query + self.ecu_id + self.cmd_suffix
    sock.sendall(cmd.encode('utf-8'))
    self.inverter_raw_data = sock.recv(self.recv_size)

    cmd = self.inverter_signal_query + self.ecu_id + self.cmd_suffix
    sock.sendall(cmd.encode('utf-8'))
    self.inverter_raw_signal = sock.recv(self.recv_size)

    sock.shutdown(socket.SHUT_RDWR)
    sock.close()

    data = self.process_inverter_data()
    data["ecu_id"] = self.ecu_id
    data["today_energy"] = self.today_energy
    data["lifetime_energy"] = self.lifetime_energy
    data["current_power"] = self.current_power

    return(data)`

You mean it would help if after each command the socket is being closed and again opened with the next command? Like so?
`
def query_ecu(self):
    sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
    sock.connect((self.ipaddr,self.port))
    sock.sendall(self.ecu_info_query.encode('utf-8'))
    self.ecu_raw_data = sock.recv(self.recv_size)
    sock.shutdown(socket.SHUT_RDWR)
    sock.close()
    self.process_ecu_data()

    cmd = self.inverter_energy_query + self.ecu_id + self.cmd_suffix
    sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
    sock.connect((self.ipaddr,self.port))
    sock.sendall(cmd.encode('utf-8'))
    self.inverter_raw_data = sock.recv(self.recv_size)
    sock.shutdown(socket.SHUT_RDWR)
    sock.close()

    cmd = self.inverter_signal_query + self.ecu_id + self.cmd_suffix
    sock = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
    sock.connect((self.ipaddr,self.port))
    sock.sendall(cmd.encode('utf-8'))
    self.inverter_raw_signal = sock.recv(self.recv_size)
    sock.shutdown(socket.SHUT_RDWR)
    sock.close()

    data = self.process_inverter_data()
    data["ecu_id"] = self.ecu_id
    data["today_energy"] = self.today_energy
    data["lifetime_energy"] = self.lifetime_energy
    data["current_power"] = self.current_power

    return(data)

`

@HAEdwin yes, correct

This also works on my part, I will run this for a while to see if the extra errors polling my printer will disappear also. @Emile86 are you using the latest version of the integration?

In the Hector's Go version, he also has a 'cooldown' (aka sleep) between requests. I haven't seen the need for that on my system yet, though.
ALso a polling of 5 minutes is fast enough. I think you won't get better sampling woth a higher frequency.

Correct, the ECU does no faster interval than 5 minutes so one minute polling is somewhat overkill.

Closing the socket could also be applied to the async io version. But let's see how this runs. (clean up code later).

Also, the ecu_r_pro returns more data from the first request. They added mac addresses, and some other unknown fields. Not really important for the integration, I think.

Yah. You get a null response

Yes, I left it out because on the ECU-R firmware the WiFi MAC is alway zero.

This also works on my part, I will run this for a while to see if the extra errors polling my printer will disappear also. @Emile86 are you using the latest version of the integration?

Yes I'm using the last version

I have mine working.

exec(open('./APSystemsECUR.py').read())
ecu=APSystemsECUR('<IP_ADDRESS>')
a=ecu.query_ecu()
print(a)
{'timestamp': '2022-01-15 00:07:04', 'inverter_qty': 3, 'inverters': {'502000026756': {'uid': '502000026756', 'online': True, 'unknown': '02', 'frequency': 0.0, 'temperature': -100, 'signal': 74, 'model': 'YC1000', 'channel_qty': 4, 'power': [0, 0, 0, 0], 'voltage': [0]}, '502000026705': {'uid': '502000026705', 'online': True, 'unknown': '02', 'frequency': 0.0, 'temperature': -100, 'signal': 73, 'model': 'YC1000', 'channel_qty': 4, 'power': [0, 0, 0, 0], 'voltage': [0]}, '502000025906': {'uid': '502000025906', 'online': True, 'unknown': '02', 'frequency': 0.0, 'temperature': -100, 'signal': 73, 'model': 'YC1000', 'channel_qty': 4, 'power': [0, 0, 0, 0], 'voltage': [0]}}, 'ecu_id': '216200006785', 'today_energy': 0.0, 'lifetime_energy': 536.4, 'current_power': 0}

You can make query_ecu asynchronous if HA likes that.
Just put async in front of definition. Then you have async (coroutine) function with plain vanilla socket code. No need for all the async stuff with sockets.
async def query_ecu(self):
Test in python

a=asyncio.run(ecu.query_ecu())
print(a)
{'timestamp': '2022-01-15 00:12:04', 'inverter_qty': 3, 'inverters': {'502000026756': {'uid': '502000026756', 'online': True, 'unknown': '02', 'frequency': 0.0, 'temperature': -100, 'signal': 74, 'model': 'YC1000', 'channel_qty': 4, 'power': [0, 0, 0, 0], 'voltage': [0]}, '502000026705': {'uid': '502000026705', 'online': True, 'unknown': '02', 'frequency': 0.0, 'temperature': -100, 'signal': 73, 'model': 'YC1000', 'channel_qty': 4, 'power': [0, 0, 0, 0], 'voltage': [0]}, '502000025906': {'uid': '502000025906', 'online': True, 'unknown': '02', 'frequency': 0.0, 'temperature': -100, 'signal': 73, 'model': 'YC1000', 'channel_qty': 4, 'power': [0, 0, 0, 0], 'voltage': [0]}}, 'ecu_id': '216200006785', 'today_energy': 0.0, 'lifetime_energy': 536.4, 'current_power': 0}

I'm trying to go through this thread and figure out what's going on, are both of you with issues with the ECU_R_PRO devices?
If so, the asyncio function already closes the socket for the ECU_R_PRO if you check line #112 in async_query_ecu function.  That's what self.writer.close() is doing.  After re-reading the python docs for the asyncio routines (https://docs.python.org/3/library/asyncio-stream.html#asyncio.StreamWriter.close) , it looks like you might want to add a await self.writer.wait_closed()
You might try that to see if it fixes your problem as well instead of reverting to the non async functions.

We can indeed best stick to one of the methods and drop the other. Is it really that easy to make the non-async query async by adding async in front of it ie: async def query_ecu(self):?

As for the asynchronous part.
The query code itself doesn't have to be async. It is code that needs to be run in sequence (procedural/batch), so there's no point in it being async.
What you still want is an async callable function for HA, so it doesn't block HA threads. And the async keyword is enough.
Other point is making sure exceptions are handled properly to prevent (socket) leaks etc.
I've changed things to handle exceptions in the socket interaction.

Great @tv3 because I tried to adapt things but I got more errors in the log over time so I went back to calling self.ecu.async_query_ecu() Looking foreward to your changes.

The async stuff is not for the benefit of this integration, it's for HA as a whole.  The point of the async socket code is to not block other co-routines running inside of HA.  While you are correct you can just put async in-front of the query_ecu function and await it, the standard socket code is still blocking.
See this example https://bbc.github.io/cloudfit-public-docs/asyncio/asyncio-part-5.html of why just running running synchronous code inside of an async co-routine isn't the best idea.  If the socket hangs or is slow to respond, it will cause all other co-routines HA is running in the same context to block.
And one of the error conditions I remember dealing with is the socket just hanging on the ECU. Since it's such a piece of junk hardware/software sometimes the recv function would block because the device wouldn't close the socket, nor would it provide any data.  Adding the timeout on the asyncio.wait_for to all the reads from the socket helps catch those conditions.
If you really want to move to using synchronous I/O please checkout the HA developers docs https://developers.home-assistant.io/docs/asyncio_working_with_async and the section about calling sync functions from within async.   You would need to change line 72 of init.py to something like this instead
data = await hass.async_add_executor_job(ecu.query_ecu)

I'm pretty hesitant to move away from asyncio as a whole because I've been running this for many months with no issues.  My ECU has been very stable, and has not locked up or needed a reset in a very long time, and my HA instance is working very reliably.

I'm pretty hesitant to move away from asyncio as a whole because I've been running this for many months with no issues. My ECU has been very stable, and has not locked up or needed a reset in a very long time, and my HA instance is working very reliably.

I totally agree! It has proven to be very stable in combination with the ECU-R. Though the ECU-R-PRO has issues and might move away to use the Sunspec Modbus solution if I'm not mistaken.

I'm pretty hesitant to move away from asyncio as a whole because I've been running this for many months with no issues. My ECU has been very stable, and has not locked up or needed a reset in a very long time, and my HA instance is working very reliably.
I totally agree! It has proven to be very stable in combination with the ECU-R. Though the ECU-R-PRO has issues running this integration.

@ksheumaker Thanks for the feedback. You built this module and have all the experience. I will go and read the link you sent. I'm still learning about the ECU stuff and appreciate all insights.
Btw: I did add a timeout to the socket code to prevent potential hangs.

I'm also having the same issue with an ECU-R-PRO. For what it's worth, mine is running 2.0.3016.
APS120116000121620001499501��  !02    10015ECU_R_PRO_2.0.3016Europe/Amsterdam��W`Ũz�00END

@renini I had tested it outside HA. Currently running tests with an HA setup. Don’t want to break your setup. When it looks good I’ll give you my changes to check.

@tv3 np, i think i have a working setup now, from your comments in this issue. Will see tomorrow whenever there is some sun. I changed the async to sync and the closing of the socket.

@tv3 np, i think i have a working setup now, from your comments in this issue. Will see tomorrow whenever there is some sun. I changed the async to sync and the closing of the socket.

And any luck?

Last update cleared my version. Just reapplied my changes again. As it's dark now, I'll have to wait. It seemed to be running ok. But haven't had a long run of a few days yet.
I also included a sock.settimeout() on the socket after open to prevent possible hangs.

I think I have figured out the cause of the original purpose of this issue, which was "Duplicated entries."
I think we need to move discussion about the ECU_R_PRO stability and async vs sync into a new issue.
Anyway, with the duplicated entries - it finally happened to me, and looking through the entity registry (/config/.storage/core.entity_registry) I found the unique_id on these sensors were None_current_power not <ECUID>_current_power.  I think this was caused by the caching I'm doing.

The integration tries to query the ECU, but if it doesn't get a response, it will pull from a cache stored in memory and return that instead.   The idea was to prevent blips where if it doesn't return correctly once, we don't go "unavailable" but try and return something useful (FYI the cache is limited to returning only 5 times before they finally will go unavailable).
There was a mistake in the logic, and that if the first query to the ECU failed it will still return the cache, which was empty.  So if after a restart of HA you didn't get a response from the ECU right away (for whatever reason) it would make a bunch of sensors with no data and a unique_id of None_current_power.  Once it starts communicating correctly with the ECU the sensors get a new unique id called <ECUID>_current_power hence the reason for duplicate sensors showing up.
I have a fix i'm testing now which will always check for "None" as the ECUID and raise an exception instead of create new sensors that are invalid.

Once I've had time to test this, I'll release an updated version.

@ksheumaker sounds promising! Many thanks for looking into it.

This fix has been added to the 1.1.2 release which is now available.  If people see duplicate sensors please re-open this, or open a new issue.


Remove zf != NULL check from ZIPPacker::start_file

This check was removed because it introduces a bug which prevents ZIPPacker from actually adding any files, since it must be opened before adding any files (and therefore shouldn't be NULL at the start of start_file).
Fixes #68130

Thanks!


[$500] New message does not appear at the top in the LHN

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.11-3
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @rafecolton
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1701899637321999
Action Performed:

Login to app as A
Receive a message from another account B
Look at the LHN of A

Expected Result:
New message bumped to top
Actual Result:
Appears at the bottom and had to scroll all the way down to see the new message in LHN
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/38435837/3a10fe54-4541-4b93-9015-1eefe892e434

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0102f48fc205c1acfd
Upwork Job ID:<PHONE_NUMBER>365197312
Last Price Increase: 2023-12-11

Can't reproduce
https://github.com/Expensify/App/assets/74202751/e7176cd3-92da-44e2-bf91-55d8f691cc7b

In this you can see the chat is below the deleted task. Here is the only one task which is deleted
https://drive.google.com/file/d/17bKCsJeg1TWsoIpLk2Iq79rk-T92ZMVQ/view?usp=sharing
Here is my another account chat where no deleted task so here chat appear in top
https://drive.google.com/file/d/1nSJYLM9qiKC1ZuyeleAxNVrgVmOHlrhS/view?usp=sharing
now in this account 3 task are deleted and chat appear after deleted task
https://drive.google.com/file/d/1eQWv1i-PdpoivOgclRXetyCZS2T3LCOi/view?usp=sharing
and in this account i have multiple task deleted task and chat appear after them here is the video
https://drive.google.com/file/d/1v4UcAoeNaLxWdeu2xdDEBNvTMLdVclxo/view?usp=sharing
so all in one conclusion , i can't reproduce the issue


mining level too high?

not sure exactly why, mines like obsidian except it gets destroyed with any pick

I found the issue, I forgot to set Omnihoppers to be mineable with a pickaxe :facepalm:
It should be fixed in the update I'll release in a few days. If you'd rather not wait that long you can grab a jar from the actions page, they should be stable.


[RFE] ManageIQ Workflows - Backend

Tracking issue for the backend implementation of the design: https://github.com/ManageIQ/manageiq/issues/22302

[x] Workflow JSON definition, based on Amazon States Language (https://docs.aws.amazon.com/step-functions/latest/dg/concepts-amazon-states-language.html)

[ ] Documentation

[x] Modeling / Schema https://github.com/ManageIQ/manageiq-schema/pull/679

[x] Workflow definition, jsonb column storing the serialized workflow, pull workflow from git source?
[x] Workflow Instance, jsonb context, userid, secrets, logs

[x] API - Reusing configuration_scripts / configuration_script_source

[x] ~Standard CRUD for Workflow / Workflow Instance~ - Requiring an external repository similar to Embedded Ansible
[x] ~APIs for workflow authors to access workflow-instance context, logs, given workflow / step scoped API token~ - Reusing configuration_scripts

[x] Workflow Provider / Runner

[x] ~Pull from registries before every run?  Only if N seconds/minutes out of date?  Leave up to user~ For now, we rely on docker pull / kubernetes pull defaults.
[x] ~Resolve references to resources (docker images/secrets)~  For now, we rely on docker pull / kubernetes pull defaults.
[ ] Validate workflow is valid (no missing references)
[x] Evaluate steps in the workflow, initiate execution of the resources by passing to orchestrator
[x] Use queue and generic workers or 1+ persistent services/runners

[x] Workflow Orchestration

[x] Appliance Podman/Dev Docker https://github.com/ManageIQ/manageiq-rpm_build/pull/401
[x] #22568
[x] Set the Floe runner based on platform https://github.com/ManageIQ/manageiq-providers-workflows/pull/28
[ ] How to handle images/registries on appliances https://github.com/ManageIQ/manageiq/issues/22640
[x] Passing secrets to containers

[x] #22435

[x] Dialogs https://github.com/ManageIQ/manageiq/pull/22441
[x] Provision/Reconfigure/Retire entrypoints

[x] Docker Images - Moving to a future Epic

[ ] Base images in various languages with utilities for interacting with API
[ ] Bash / Ruby initially
[ ] Some standard functions like sending an email, slack message, MIQ notification/alert, User approval

[x] Proof of Concept

[x] Workflow exhibiting all of the supported concepts

[x] States with Retry, Sleep, Choice, Failure
[x] Secrets
[x] Update context over API

Example API calls to set up a git repository which will sync workflows, and create credentials:
$ curl --user admin:smartvm -H "Content-Type: application/json" -X POST http://localhost:3000/api/authentications -d '{"name":"id_rsa","userid":"git","manager_resource":{"href":"http://localhost:3000/api/        providers/2"},"type":"ManageIQ::Providers::Workflows::AutomationManager::ScmCredential","auth_key":"","auth_key_password":""}' 2>/dev/null | jq
$ curl --user admin:smartvm -H "Content-Type: application/json" -X POST http://localhost:3000/api/configuration_script_sources -d '{"name":"github.com:agrare/provision-vm-service","manager_resource":{"href":     "http://localhost:3000/api/providers/2"},"scm_type":"git","scm_url":"git@github.com:agrare/provision-vm-service.git","scm_branch":"master","type":"ManageIQ::Providers::Workflows::AutomationManager::              ConfigurationScriptSource","authentication_id":2}' 2>/dev/null | jq
$ curl --user admin:smartvm -H "Content-Type: application/json" -X POST http://localhost:3000/api/authentications -d '{"name":"ManageIQ API Credential","ems_ref":"manageiq_api","userid":"admin","password":       "smartvm","manager_resource":{"href":"http://localhost:3000/api/providers/2"},"type":"ManageIQ::Providers::Workflows::AutomationManager::Credential"}' 2>/dev/null | jq
$ curl --user admin:smartvm -H "Content-Type: application/json" -X POST http://localhost:3000/api/authentications -d '{"name":"vCenter Credential","ems_ref":"vcenter_credential","userid":"root","password":       "vmware","manager_resource":{"href":"http://localhost:3000/api/providers/2"},"type":"ManageIQ::Providers::Workflows::AutomationManager::Credential"}' 2>/dev/null | jq
TODO $ curl --user admin:smartvm -H "Content-Type: application/json" -X PATCH http://localhost:3000/api/configuration_script_sources/3/configuration_script_payloads/20 -d '{"credentials":{"api_user":"$.          manageiq_api.userid", "api_password":"$.manageiq_api.password"}}'
TODO $ curl --user admin:smartvm -H "Content-Type: application/json" -X PATCH http://localhost:3000/api/configuration_script_sources/3/configuration_script_payloads/21 -d '{"credentials":{"api_user":"$.          manageiq_api.userid", "api_password":"$.manageiq_api.password"}}'
TODO $ curl --user admin:smartvm -H "Content-Type: application/json" -X PATCH http://localhost:3000/api/configuration_script_sources/3/configuration_script_payloads/23 -d '{"credentials":{"api_user":"$.          manageiq_api.userid", "api_password":"$.manageiq_api.password","vcenter_user":"$.vcenter_credential.userid","vcenter_password":"$.vcenter_credential.password"}}'

Marking this as completed.  All other things listed in the OP will be done as follow ups or future features.


Homework 3

drift video : https://video.drift.com/v/abIT97tVVQYIO0WrsfX25GRlwxXmABOCvGlsLUt1Qa6g/
screenshot :

Metric
Value

alert_status
:white_check_mark: OK

bugs
0

code_smells
14

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
0

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Bug: invoke, onDone type incompatibility when exactOptionalPropertyTypes: true

XState version
XState version 5
Description
Reproduction on typescritplang.org.
typescript: v5.6.3
xstate: 5.18.2
Code is the same as the #invoke-and-typescript example with a few tweaks.
Navigate to TS Config and set exactOptionalPropertyTypes.

The resulting error on onDone will be:

[!CAUTION]
          The types of '_out_TActor.src' are incompatible between these types.
            Type 'string' is not assignable to type '"fetchUser"'. [2322]

[!Note]
This only happens when I'm trying to use assign, otherwise no error.

(Probably anything in actions will set it off, but I didn't test that).

Expected result
No typescript error.
Actual result
   typescript: Type '{ actions: ActionFunction<{ userId: number; user?: User; }, DoneActorEvent<User, string>, AnyEventObject, undefined, ProvidedActor, never, never, never, never>; }' is not assignable to type 'SingleOrArray<TransitionConfigOrTarget<{ userId: number; user?: User; }, DoneActorEvent<User, string>, AnyEventObject, { src: "fetchUser"; logic: PromiseActorLogic<User, { userId: number; }, EventObject>; id: string | undefined; }, ... 4 more ..., MetaObject>>'.
     Types of property 'actions' are incompatible.
       Type 'ActionFunction<{ userId: number; user?: User; }, DoneActorEvent<User, string>, AnyEventObject, undefined, ProvidedActor, never, never, never, never>' is not assignable to type 'Actions<{ userId: number; user?: User; }, DoneActorEvent<User, string>, AnyEventObject, undefined, { src: "fetchUser"; logic: PromiseActorLogic<User, { ...; }, EventObject>; id: string | undefined; }, never, never, never, EventObject> | undefined'.
         Type 'ActionFunction<{ userId: number; user?: User; }, DoneActorEvent<User, string>, AnyEventObject, undefined, ProvidedActor, never, never, never, never>' is not assignable to type 'ActionFunction<{ userId: number; user?: User; }, DoneActorEvent<User, string>, AnyEventObject, undefined, { src: "fetchUser"; logic: PromiseActorLogic<User, { ...; }, EventObject>; id: string | undefined; }, never, never, never, EventObject>' with 'exactOptionalPropertyTypes: true'. Consider adding 'undefined' to the types of the target's properties.
           The types of '_out_TActor.src' are incompatible between these types.
             Type 'string' is not assignable to type '"fetchUser"'. [2322]

Reproduction
https://www.typescriptlang.org/play/?exactOptionalPropertyTypes=true&ts=5.6.3#code/JYWwDg9gTgLgBAbzgQwM6uAcwHYBo4BmUEIACsSMKgKb40wCuYcAvoRXAOQAeqMyMapwDccOAHpxcGAE8w1VAC44AVgB0ARgAcagEwAoQ8GyCoBZAGNqcAKo0oifWOAATZdgYgARtSjCncNjIINTKfFDGmP4shtTckLBwFhDYfHAglgAWxtYAvHD0TAAUCAGy8kqOYmLJJnEwyghsaFXVYgz2AJJugZ4+fgFtHb4A-Mp2vv5tLLgBMwGWMNCVpW0E1DAWmRNQykQk5CRU1AA8O-hIw1Dd7n2+rAB8RWgy2BZwJXDGYAzwLACUcFyD1abVqaSgCkgqTyKAA7shgPBDpQaGpIagIAAbABu1BKgzaXx631+aiu3VmROqQRCyk4AAkGJgIJwqUSAVMiZDGFBsHAMdCaFyxAD2fMAWoLJCBNQALJZHIEsSSOBqdUBWqCbgNUFwCk9AAsunF7OMSOAyCx9NcWKE7L4spWhNtoT1YhSjUJbWoLiROwAIhtEViVtJkFBMBt6ViIMg-dhMDtOKx2dM06nCbH45Edl7qcYcRAANZu1bUgpQCz09abbb2Nne6qk3WfLX1ViA4EfcsV-VdHrtnXkgf4VUAFTk1mo2AI0CsLmkEDgPgKMAiiabov+GbaKQDKTLW+qi2AKUqaAwOGVfeqV2Un2oeJMnaBIKfM5gaggvx+MDHUgAMrrikmBYjI0hTouLQ7Me267tU8zUkhiGmvonJAA
Additional context
No response

Short term remediation:
// @ts-expect-error exactOptionalPropertyTypes

Second remediation:
        onDone: {
          actions: ({ event }) => assign({
            user: event.output, // Strongly typed as User
          }),
        },

Making actions an anonymous function, either returning a single action or an array of actions works fine.

I have the same issue. I tried using enqueueActions to avoid using assign but it gives the exact same error. Seems like an issue with the built-in actions.
Function assigners gives the same error as well.
@pauldesmondparker I'm fairly sure your second remediation won't do anything (at least I can't make it do anything). I don't think you can provide a function that returns an action object.
// @ts-expect-error exactOptionalPropertyTypes is the only thing I've found that works.

Looks like this might be a duplicate of https://github.com/statelyai/xstate/issues/4613

You're right. The second remediation was completely useless. For arrays it removed the error, but was non-functional (i.e. despite satisfying typescript, it just doesn't work).


Oculus VR DK2 Support

Updated to work with 0.5.x SDK
Uses Oculus Rendering rather than PostFX
Stereo rendering refactored so more rendering info is grabbed from the DisplayDevice
Implements an Offscreen Canvas for in-game gui with oculus
Message dialogs and metrics display can now go to the OffScreen Canvas (if oculus demo is setup correctly)

Resolves issue #1296

Having my friend double-test this as he isn't half-blind in one eye without glasses. Should have his confirmation that it works as well as I thought it did tomorrow. If so, I'll merge this in.

:grinning:

'Aight, my friend tested it for me.
He said, like we saw, rendering seems to be fine, but there's that little bit of an interpolation jitter that needs to get sorted out going forward.
He also said that the starting prompt when a level loads looks fine, but if you hit escape to get the exit out ui pop-up, that's overshot. "Left eye: box too far right, Right eye: box too far left". So the respective eye's offsets for the UIs are overshot. So that'll need a tweak.
But otherwise, the root implementation seems to be solid. So if there are no immediate complaints, we can merge this in, and then patch those remaining holes in follow-up PRs.
Sound reasonable?

@Areloch were you running in debug or release? Release tends to be much smoother.
The jitter itself I think is caused by the different values returned from the sensor data which gets sent to the move manager and the data which is sent by the per-frame rendering callbacks.
You can force it to only use the data from the move manager by setting "$TSControl::useLatestDisplayTransform" which will give more consistent movement, but this may cause problems with the time warp distortion especially if the FPS is too low.
From what I can tell most examples tend to either use the rendering transform as-is OR they interpolate it with the transform of the control object. The additional problem with torque is in order to properly update the view frustum the movement needs to go through the MoveManager. On top of that you have additional movement caused by using the keyboard or mouse to move the player. Also to be honest the Oculus SDK isn't entirely clear on how these problems should be resolved in a real situation (though I did hear they changed the API yet again in 0.6 so I'll have to check if thats improved).
Rendering-wise this adds some crucial features for properly rendering stereo viewports with asymmetric  projections which would be useful for other VR devices.

Thinking on it, I may have passed him the debug build. So if it's even less of a problem in release, I won't fret on it majorly for now.
And I agree that this introduces stuff not only for oculus, but other VR systems, so it's definitely good to have.
So we can merge this in now, and just continue to polish the remaining oddities going forward.

:grinning:


Twice as tall as images

Hello,
Great plugin, but I'm having a problem.
When I include at least two images in the slide, the slider/carousel gets twice as tall as the images in it:

The yellow box is the height of the carousel which is twice the height of the images in it. The red box is is the "ghost" image causing the problem.
Any image I put there, no matter the height, makes the height of the carousel double the height is supposed to be.
I already tried building everything from scratch 3 times considering that maybe I've missed something, edited the CSS, etc., but the results are always the same.
Any idea what's causing this problem?
Thanks.

Please add more details (show the code). Are you reseting or normalizing css? If no, I've recommending https://necolas.github.io/normalize.css/.
For quick debug you can try adding temporary fix and see if I'm right:
* {
    margin: 0;
    padding: 0;
    border: 0;
}

Your troubleshooting rule was very helpful. Seems the padding was somehow causing the issue, but TBH I have no idea what element is exactly the offender.
So I did a global selector and removed the padding from all elements (I know, not entirely ideal but oh well):
.glide {
	position: relative;
	width: 100%;
	height: 100%;
	* { padding: 0; }
}

And yes, I am using normalize.css.
Thanks for your fast reply :)


Adding ai and ml categories to integrations

What does this PR do?
Adds new AI/Machine Learning category classifier to relevant integrations in this repo
Motivation
New AI & ML Categories were added
Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Changelog entries must be created for modifications to shipped code
[ ] Add the qa/skip-qa label if the PR doesn't need to be tested during QA.

PR Labels / labeler / apply (pull_request) is failing but isn't required to merge.

Yes, fix from AITOOLS-243 should take care of it.


docs: add one-liner tips

Eventually raen may wrap near. Until then, it’s useful to know how to build & deploy in one command.

Looks good.


In page links are not working

Does this package works for links that stays in the page?
e.g. a button with link target with only #bottom

@cbou
Yes, it should work
Here's our test for this case.
However, if your app for some reason swallows the click event, or the URL change is triggered programmatically, you will have to configure scroll-to-fragment accordingly.
Typically this means you have to pass your History integration in the history option.

FYI currently, scroll-to-fragment doesn't work with history version 5. We'll prepare an update.

Thanks 👍
You are right it works now. Except if I load the Scrivito editor then the link does not work...

Please note that while scroll-to-fragment works with history version 5, scrivito depends on history version 4.
See https://npm.anvaka.com/#/view/2d/scrivito for the dependency graph.
Please make sure to use history 4.x when passing it to Scrivito.useHistory().
Regarding HTML links in in-place editing (edit mode) - yes, you cannot click them to navigate. Editing has priority.


When will be added support for VS 2015

For now it is impossible to compile sources using VS 2015.
When you enable support for that version?

Could you give more details about "impossible"? I just tested and it is compiling fine here.
Have you installed the Windows 10 SDK 10,0,10240.0 as described on https://github.com/sharpdx/SharpDX#build ?

Yes, I have SDK version installed, but when I start copiling, it fails with error that version 2015 (14.0) is not supported)

I can't reproduce this... Which solution are you trying to compile? Do you compile from VS or msbuild/cmdline?

I compile from VS main solution - SharpDX. With VS 2013 I have no such issues

Hi xoofx,
I can confirm that on a system with VS 2015 only installed the compilation fails at early stages of preparing to parse include files with the error message that QuantumDeveloper mentions.
In order to make it compile I had to modify Source/Mappings.xml and GccXml.cs files.
You probably have both VS 2013 and 2015 installed on your machine and that combination works even when compilation is done via VS 2015 because header files and compile tools of 2013 exist.
B.

Currently, SharpDX doesn't compile with VS 2015 only. You need to have C/C++  runtime headers shipped with VS2013. No ETA when I will have time to look at this...

@bdachev maybe you could provide necessary fixes for this issue?

just for information. Yesterday compilation worked for me. I've installed VS 2015, not VS 2013 (but I had previously, then I've unistalled to install VS15).
Yesterday I've installed githhub extension... and now stop compiling. Can be usefull this information? What I must install to compile? Visual C++ 2013 or other components are enough?

Hi Guys, I committed in my fork quick and dirty fix to build SharpDX with VS2015 only. It is in vs2015 branch. That configuration will break pure VS2013 build so not good idea to merge in master.

@bdachev thanks for that.

thx

Further to my previous message I was able to compile master branch using VS2015 only. What I was missing before and installed in order to make it work was tools for Window 8.1 development.

@bdachev thanks for info. Will test your solution.

Have you seen my previous comment? Do you have Tools for Window 8.1 installed in VS 2015 installer configuration?

Sorry I wasn't sure if that was the only required fix since there were no other replies. Do you know what it needs from the Tools for Windows 8.1?

Visual Studio 2013
Windows 10 SDK 10.0.10586.0

It seems there is still confusion as @bdachev appears to have it working with VS2015 and just the "Tools for Windows 8.1" option.

Yes, it work in VS2015, but must installed VS2013.
In build using absolute path
https://github.com/sharpdx/SharpDX/blob/master/Source/Mapping.xml#L28

I can confirm that VS2013 is NOT needed. Installing VS2015 with "Tools for Windows 8.1" option enabled actually installs C++ compilers and tools that come with VS2013. I just did some tests on a Win 10 + VS2015 only installed and I am able to compile both SharpDX solutions. Make sure 10.0.10586.0 SDK is installed as well.

Awesome. Thanks for the help!


All executor servlets to use POST

ExecutorApiGateway was changed to use POST for all requests, but I missed that azkaban-web also makes calls to /serverStatistics, /jmx & /stats via the gateway, not just /executor.
The problem wasn't seen in manual tests with AzkabanSingleServer because it doesn't use multi-executor mode.
Bug was reported by @chengren311 at https://github.com/azkaban/azkaban/pull/1659#issuecomment-376768014.

@juhoautio thanks for the fix. Would you mind adding the test methodology in the description?

verified the fix in our cluster. thanks!


internationalization

[ ] implementing I18n
[ ] offering the choice to between English and French as a secondary language

Ive trieed the webapp, looks good machallah,
i have one suggestion, with quote of day, مقولة اليوم ، it's good if you use human like quotes, but it's preferable to write آية اليوم، i douno if the data you work with have explanation of the Aya if it doens't it could be inspiring
thank you for all efforts

great catch @Zakarialabib thanks! I'll take a look at the options to implement your suggestion for the verses' explanation, as for the head title of the quote it's doable to change it if it's a verse, might do it in coming days.


Does the Tornado adapter support AsyncApp?

Hello, I am trying to set up a slack app using the Tornado adapter. I am trying to use AsyncApp instead of App, as that is the purpose of Tornado as an Asyncio server. however, it looks like the Tornado adapter tries to call self.app.dispatch and doesn't attempt to call await self.app.async_dispatch which is what the AsyncApp class uses. CC @seratch (Saw that you had worked on the Tornado code)
I am successfully able to get it working with App as seen below:
slack_app = App(
    token=config.get("_global_.secrets.slack.bot_token"),
    signing_secret=config.get("_global_.secrets.slack.signing_secret")
)
.... (routes):
("/api/v3/slack/events", SlackEventsHandler, dict(app=slack_app)),
("/api/v3/slack/install", SlackOAuthHandler, dict(app=slack_app)),
("/api/v3/slack/oauth_redirect", SlackOAuthHandler, dict(app=slack_app)),

but I am unable to get this working with an AsyncApp. The app starts correctly but when I receive events, I get an error like this:
HTTPServerRequest(protocol='http', host='REDACTED', method='POST', uri='/api/v3/slack/events', version='HTTP/1.1', remote_ip='<IP_ADDRESS>')
Traceback (most recent call last):
  File "env/lib/python3.10/site-packages/tornado/web.py", line 1711, in _execute
    result = method(*self.path_args, **self.path_kwargs)
  File "env/lib/python3.10/site-packages/slack_bolt/adapter/tornado/handler.py", line 17, in post
    bolt_resp: BoltResponse = self.app.dispatch(to_bolt_request(self.request))
AttributeError: 'AsyncApp' object has no attribute 'dispatch'
2022-11-01:16:26:54,418 ERROR    [web.py:2271] 500 POST /api/v3/slack/events (<IP_ADDRESS>) 34.36ms
2022-11-01:16:26:55,176 ERROR    [web.py:1798] Uncaught exception POST /api/v3/slack/events (<IP_ADDRESS>)

Reproducible in:
slack-bolt==1.15.2
slack-sdk==3.19.2
Python 3.10.8
OS info
#58-Ubuntu SMP Thu Oct 13 08:03:55 UTC 2022
(Ubuntu 22.04)

Hi @castrapel, thanks for asking the question.
Since tornado.web.RequestHandler does not support async functions for get/post etc., the built-in Tornado adapter does not support AsyncApp use cases. See also: https://www.tornadoweb.org/en/stable/web.html
Checking the following resources may be helpful too. However, as far as I know, the async related decorator does not work with bolt-python in the way that you expect.

https://www.tornadoweb.org/en/stable/guide/async.html
https://stackoverflow.com/questions/49199638/tornado-web-asynchronous

If you would like to make your listeners/middleware async for some reason, I'd suggest going with other modern asyncio-based frameworks such as AIOHTTP, FastAPI, Falcon, and Sanic. You can find examples here.
If you have a certain reason to use Tornado over others, I highly recommend going with App and synchronous code. With that being said, if you are keen to explore some ways to adjust the adapter, you can fork it and try whatever may better work for you. Here is the adapter code: https://github.com/slackapi/bolt-python/tree/v1.15.2/slack_bolt/adapter/tornado As you can see, there are only two files and they are quite simple.
I hope this was helpful to you.

Thank you for the response. tornado.web.RequestHandler does indeed support async def post, async def get, but I am not sure why they do not document it well.  Here is a way to test:
mkdir -p /tmp/tornadotest
cd /tmp/tornadotest
python3 -m venv env
pip install tornado
vim server.py # Paste in the contents of server.py below and save
python server.py
# Visit http://localhost to confirm that you receive a text response: `GET request`

Contents of server.py:
import tornado.web
import tornado.ioloop
import tornado.options
import tornado.httpserver

import tornado.web

class TestHandler(tornado.web.RequestHandler):
    async def get(self):
        self.write("GET request")

    async def post(self):
        self.write("POST request")

routes = [
    (r"/", TestHandler),

]

def make_app(debug: bool):
    return tornado.web.Application(routes, debug=debug)

def run_service():
    debug_mode = True
    port = "8000"
    app = make_app(debug=debug_mode)
    app.listen(port)
    tornado.ioloop.IOLoop.current().start()

if __name__ == "__main__":
    run_service()

@castrapel Thanks for sharing this and I'm glad to hear that async functions are actually supported!
I am taking day offs until Friday. I will look into the enhancement after coming back. If you are in a hurry, please consider copying the adapter code and editing it to work with async functions for now.

Thanks @seratch, I'd appreciate your help here! Not in a big hurry on my end.


Add byteorder::U24

There are binary file formats which utilize 24-bit unsigned integers, so this will make it easier to work with those files.

Codecov Report
Attention: Patch coverage is 97.84946% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 87.88%. Comparing base (76915d0) to head (cec8c16).

Files
Patch %
Lines

src/byteorder.rs
97.84%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1352      +/-   ##
==========================================
+ Coverage   87.71%   87.88%   +0.16%
==========================================
  Files          15       15
  Lines        5138     5226      +88
==========================================
+ Hits         4507     4593      +86
- Misses        631      633       +2

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

Thanks for the contribution! We've currently got a pretty big backlog, so I expect it'll be a few weeks until we'll get a chance to look at this - likely late June. Apologies for the delay, but I promise we haven't forgotten about it 🙂

Sorry for letting this sit for so long!
Can you say a bit about your use case for this feature?

Sorry for letting this sit for so long!
Can you say a bit more about your use case for this feature? What specific file formats are you hoping to parse, and in what context? Do you have other use cases besides that?

No worries! I ended up going with a different library, https://github.com/jam1garner/binrw. The specific file format I was processing was BYML (https://zeldamods.org/wiki/BYML) which uses a U24 for number of entries in a few node types.

Sounds good!


Let's break down the config file into smaller objects

The problem faced currently?
One huge config file can be overwhelming to edit and work with. People end up using mission-control exclusively because of this. This will make devOps practisely close to impossible with Space Cloud
How can we solve it?
Lets break down the config file into smaller objects like what kubernetes does. Then we can have a space-cli apply -f  /path/to/file to apply all configuration to space-cloud.

If you want this feature to be implemented, give it a thumbs up reaction, so that we can determine which features are important to you.
👍

fixed in #884


Mavenized and modified to support modified ice4j

This PR requires https://github.com/jitsi/ice4j/pull/38 to be merged first.

I can't make out the differences between the various changes. Please create a one pull request at a time, with the Mavenization being either the first or the last. Also IDE files (Eclipse) should not end up in the repo (anymore).
Some files were apparently not detected as a move or contain line-endings changes. If the repo is unclean (line-ending wise) then please also create a separate pull request for that.
What is that modified ice4j about?

The majority of the change here is that the files are moved to the maven directory structure from an ant style setup; I'll take a look tomorrow and resubmit.

I'm closing this so the work can be resubmitted in smaller more correct parts.


Depthmaps format

HI,
I have a depthmaps that looks like this: click
I've tried using them in your code in the form as show on the picture, and after making them grayscale, but generated point-cloud seems to be empty with those.
Any advices - how I should prepare pictures to make them work with your code ?
Regards.

Depth maps should be saved in format: 16-bit PNG, depth in millimeters.

Ok, thanks for quick answer - what about the resolution - does it have to be 640x480. or I can just change it in code - and it should work ?

Changing it should work. I haven't tested it though. Keep in mind that your camera intrinsics matrix cam_k will likely also have to change.

Is it possible to apply different camera intrinsics to different frames ?
My frames are from iPhone, and it changes the camera intrinsics matrix every shoot.

It's possible, but you'll have to modify demo.cu so that it changes cam_K as it's looping through depth frames.

Perfect, thanks again !

hello, @adamroszyk
how do you convert image 32-bit to 16-bit as with as depth in millimeters?


Implement PEP-487: customisation of class creation

class ABC:
    def __init_subclass__(cls):
        print(cls)

class SubClass(ABC):
    pass

If you run this with python directly, it gives:
<class '__main__.SubClass'>

But after I compile it with cython, it gives:
TypeError                                 Traceback (most recent call last)
<ipython-input-1-a0954f22e2a4> in <module>
----> 1 import test

test.pyx in init test()

TypeError: __init_subclass__() takes exactly one argument (0 given)

How to solve it?

The relevant bits of PEP-487 need to get implemented. Specifically, __init_subclass__() is an implicit class method and must be treated as such, and class creation should call the __init_subclass__() method in older Python versions (before 3.6).
I assume you were using Py3.6 or later for your tests, right?

PR welcome, seems relatively easy. If you want to give it a try, see the HackerGuide for hints how to get started, and ask on the cython-dev mailing list whenever you feel like needing help.

Python 3.9 error still isn't fixed

Test:
# subclass.py
class A:
    def __init_subclass__(cls):
        print(cls)
        return cls

class B(A):
    pass

Run:
cythonize -3 -q -i subclass.py
Python 3.9.4 (tags/v3.9.4:1f2e308, Apr  6 2021, 13:22:44) [MSC v.1928 32 bit (Intel)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import subclass
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "subclass.py", line 6, in init subclass
    class B(A):
TypeError: __init_subclass__() takes exactly one argument (0 given)

$ ls
__pycache__  sub.c  sub.cp39-win32.def  sub.cp39-win32.pyd  sub.o  sub.py

Python 3.9 error still isn't fixed

No. Your example works fine on the current Cython 3 alpha version. You're probably using an older version where this feature isn't supported.

I use AppVeyor for building wheels and it uses the latest Cython version (using pip install cython). Is the fix in a release that I need to specify manually?

It's in the 3.0 alpha release (since 3.0a2 I think). It isn't (and won't be) in the 0.29.x branch. Since the 3.0 branch is still an alpha branch you probably do need to specify it manually. The alpha branch is pretty usable at this stage and is where most new features are going, so it may be worth your while trying it.

How do I install it with pip?

What about just choosing the latest prerelease?

Never mind, pip install --pre cython works!

I don't know what is going on now, but the error is still there. Here is the build: https://ci.appveyor.com/project/rayzchen/pyunity/build/job/i011xbqufc0tj03o
Lines 39-533 are the main focus. Line 39 installs cython 3.0a8, line 57 creates the cython files (using cythonize -3 -q file.py) and the rest is just creating the bdist. However, when I install the wheel provided (tested on both python 3.9 win32 and win_amd64) and run an example, the error happens when subclassing from pyunity.Behaviour, which defines __init_subclass__.

I don't know what is going on now, but the error is still there. Here is the build: https://ci.appveyor.com/project/rayzchen/pyunity/build/job/i011xbqufc0tj03o
Lines 39-533 are the main focus. Line 39 installs cython 3.0a8, line 57 creates the cython files (using cythonize -3 -q file.py) and the rest is just creating the bdist. However, when I install the wheel provided (tested on both python 3.9 win32 and win_amd64) and run an example, the error happens when subclassing from pyunity.Behaviour, which defines __init_subclass__. But for some reason, it works randomly on another computer that I have open. I really am too confused right now.

If that was too confusing, here is the summary:
https://ci.appveyor.com/api/buildjobs/i011xbqufc0tj03o/artifacts/dist%2Fpyunity-0.7.0-cp39-cp39-win32.whl - win32 python 3.9 build
https://ci.appveyor.com/api/buildjobs/i011xbqufc0tj03o/artifacts/dist%2Fpyunity-0.7.0-cp39-cp39-win_amd64.whl - win_amd64 python 3.9 build
I have 3 computers: 2 win32 computers, 1 win_amd64 computer (this is python version)
Install correct wheels on each computer
All compiled using cython 3.0a8 (see above build)
2 computers fail on a line which is just class Rotator(Behaviour):
The error is TypeError: __init_subclass__() takes exactly one argument (0 given)
The Behaviour class defines __init_subclass__ with two arguments, self and cls
python 3.9 win32 1                        python 3.9 win32 2                    python 3.9 win_amd64
Fail                                                    Pass                                          Fail
There is no difference on the computers, I've even tested using a venv.

I don't really know how I fixed it but its fixed now...


Configuring default value with fluent API doesn't support enum value without explicitly casting to int

Suppose we have model class with property of enum type that should be mapped to INT column in database. The class looks as follows
public class Employee
{
    public int Id { get; set; }

    public OccupationType OccupationType { get; set; }
}

public enum OccupationType
{
    Fulltime,
    Parttime,
    Freelance
}

Also we want to configure default value for that column with fluent API as follows:
builder.Entity<Employee>()
    .Property(e => e.OccupationType)
    .HasDefaultValue(OccupationType.Fulltime);

Everything looks fine. Let's generate migration:
dnx ef migrations add AddEmployees

Here's what we get:
System.NotSupportedException: Annotations of type 'OccupationType' are not supported. Only common simple .NET types are currently supported.
   at Microsoft.Data.Entity.Infrastructure.TypedAnnotation.CheckType(Type type)
   at Microsoft.Data.Entity.Infrastructure.TypedAnnotation..ctor(Object value)
   at Microsoft.Data.Entity.Metadata.RelationalPropertyAnnotations.SetDefaultValue(Object value)
   at Microsoft.Data.Entity.Metadata.RelationalPropertyAnnotations.set_DefaultValue(Object value)
   at Microsoft.Data.Entity.RelationalPropertyBuilderExtensions.HasDefaultValue(PropertyBuilder propertyBuilder, Object value)
   at Microsoft.Data.Entity.RelationalPropertyBuilderExtensions.HasDefaultValue[TProperty](PropertyBuilder`1 propertyBuilder, Object value)
   at SomeProject.Models.ApplicationDbContext.OnModelCreating(ModelBuilder builder) in C:\Users\Alexander\Work\Github\EvAlex\SomeProject\src\SomeProject.Website\Models\ApplicationDbContext.cs:line 24
   at Microsoft.Data.Entity.Infrastructure.ModelSource.CreateModel(DbContext context, IConventionSetBuilder conventionSetBuilder, IModelValidator validator)
   at Microsoft.Data.Entity.Infrastructure.ModelSource.<>c__DisplayClass8_0.<GetModel>b__0(Type k)
   at System.Collections.Concurrent.ConcurrentDictionary`2.GetOrAdd(TKey key, Func`2 valueFactory)
   at Microsoft.Data.Entity.Infrastructure.ModelSource.GetModel(DbContext context, IConventionSetBuilder conventionSetBuilder, IModelValidator validator)
   at Microsoft.Data.Entity.Internal.DbContextServices.CreateModel()
   at Microsoft.Data.Entity.Internal.LazyRef`1.get_Value()
   at Microsoft.Data.Entity.Internal.DbContextServices.get_Model()
   at Microsoft.Extensions.DependencyInjection.EntityFrameworkServiceCollectionExtensions.<>c.<AddEntityFramework>b__0_5(IServiceProvider p)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.FactoryService.Invoke(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.ScopedCallSite.Invoke(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.<>c__DisplayClass12_0.<RealizeService>b__0(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderExtensions.GetService[T](IServiceProvider provider)
   at Microsoft.Data.Entity.Design.Internal.DesignTimeServicesBuilder.<>c__DisplayClass7_0.<ConfigureContextServices>b__8(IServiceProvider _)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.FactoryService.Invoke(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.TransientCallSite.Invoke(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceLookup.ConstructorCallSite.Invoke(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.TransientCallSite.Invoke(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.<>c__DisplayClass12_0.<RealizeService>b__0(ServiceProvider provider)
   at Microsoft.Extensions.DependencyInjection.ServiceProvider.GetService(Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderExtensions.GetRequiredService(IServiceProvider provider, Type serviceType)
   at Microsoft.Extensions.DependencyInjection.ServiceProviderExtensions.GetRequiredService[T](IServiceProvider provider)
   at Microsoft.Data.Entity.Design.MigrationsOperations.AddMigration(String name, String outputDir, String contextType)
   at Microsoft.Data.Entity.Commands.Program.Executor.<>c__DisplayClass6_0.<AddMigration>b__0()
   at Microsoft.Data.Entity.Commands.Program.Executor.Execute(Action action)

It turns out that the problem is default value type. The error is in default value configuration and the correct syntax is:
builder.Entity<Employee>()
    .Property(e => e.OccupationType)
    .HasDefaultValue((int)OccupationType.Fulltime);
//                   ^^^^^ here's the difference

The exception message is absolutely not clear.

Related to https://github.com/aspnet/EntityFramework/issues/4061 (though that issue says we scaffold the migration, where as this issue fails generating the migration).

@smitpatel Upon closer inspection, it looks the stack trace is purely in model building (getting DbContextServices.Model). Would you mind taking a look at this one?

The issue lies in TypedAnnotation. It supports a fixed set of types only. So having any other default value type would throw similar exception.
TypedAnnotation is being used by DefaultValue API only. It was added when Annotation did not support object type value. Is it still needed?
It was added by @ajcvickers  in #861


[Moon] Trying to merge desk without permission ends in infinite loop

What:
If one tries to merge without permission between two moons, the source moon will enter infinite loop, even when the merge is canceled, even if the permission is turned on.
There is no way to prevent %ames from spinning and utilizing 100%, except CTRL-C, but it will relapse back to this state after few seconds.
Reproduction:

Have two moons
Sync a desk from one moon to the other without setting public first.
What ensues

I haven't tested it between random ships, but I guess it will end up with the same situation.

Short ctrl-c
[%swim-call-vane %a %want ~]
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/zuse
  <[2.700 7].[2.701 46]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/zuse
  <[2.701 7].[2.701 46]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/zuse
  <[2.701 12].[2.701 45]>

Repeated:
  <[1.086 7].[1.093 65]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[1.088 7].[1.093 65]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[1.088 17].[1.088 66]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[1.507 7].[1.511 32]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[1.508 7].[1.511 32]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[475 7].[485 15]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[478 13].[485 9]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[478 17].[478 40]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[478 21].[478 40]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[1.510 7].[1.511 32]>
  /~nolpem-sanrul-ponmep-litsem/home/~2018.3.24..20.31.43..e17f/sys/vane/ford
  <[1.510 23].[1.510 55]>

I have hit the exact same bug, this time with moon having public desk enabled.

Ames loop can stopped if one shuts down merging moon.
Once it goes online again, the source moon starts spinning again.

The merge which hanged with proper permissions, was %fine one, not %init.
Probably worth checking out, it might turn out that for moons that one is broken regardless of permission status.

Possible suspect has been identified, issue https://github.com/urbit/urbit/issues/936

@mikolajpp, these symptoms also sound like #740 (and, I suspect, #594).

Closing as it's very likely this one has been resolved by #756.


Migrate to stratos-ui

Create new branch in stratos-ui for v2 master (master-v2?)
migrate, with history, stratos v2 master
migrate issues along with labels/milestones/etc
will need to update

Travis
code climate (will this work against suse org??)

This issue was moved to cloudfoundry-incubator/stratos#1496


20211104 intros

temp spot to copy pasta

Last seen
video / audio

10/17/2021
Mike Abbot Jacket sword v gun / The Soft Boys - I Wanna Destroy You

10/17/2021
Kamen Riders and Hanuman vs Silver Bull / The Go Team - Bust Out Brigade

10/17/2021
Kaiju Konga feat Hanuman Evisceration / Meiko Nakahara - Fantasy

10/17/2021
Mary Can You Hear Me / King Pari feat Velvet Negroni - Mary

Last seen
video / audio

7/13/2021
Little Red Cynthia Hood vs Pit Fighter / Edwards Generation - That's How Much I Love Her

New...
Original Audio
Ruins... spool... tommy guns... hurt guts... grenade

10 YEARS LATER

Last seen
video / audio

10/17/2021
Gordon's Hill Meditations / Original audio, Kate Bush - Running Up That Hill

Last seen
video / audio

5/27/2021
Island Rydeen / Senor Coconut - Rydeen

Last seen
video / audio

10/17/2021
Sitar Suit Seaside Scramble / (the affair, then) J Dilla - Coming Back

New...
Mike (Rex) gets out of Datsun to Clan of Xymox whisper track, then overlaid Tornado Siren Tik Tok song, and Mark Watson (Bill) wants nothing of it

Last seen
video / audio

11/02/2021
Studded headband crossbow lady vs G etc / Bob James - Shamboozie

New...
(more Platoon the Warriors)
Fisheye lens on Rex vs Bill till Rexplosion The END / Original Audio? first track is breakbeat, second track is castlevania

Update...
Intro Vortex / Kavo
New footage instead of Rydeen stuff...

20230801
Shamboozie intro
World of Tom Clark (gnomes)
Cat TV (fake birbs)
Bob Mann’s Unauthorized History of Golf
Norma outro

20230804
Mary intro
Don’t mind a little danger
fought against all evils
Norma outro


[Docs] Document Genesis Setup (Below installation)

Is your feature request related to a problem? Please describe.
Create or reuse existing documentation for Genesis Setup to own section under installation on the wiki

@jeffcampbellmakesgames I would like to work on this :)

@jeffcampbellmakesgames I would like to work on this :)

@jeffcampbellmakesgames Please review this
https://github.com/jeffcampbellmakesgames/Entitas-Redux/wiki/Genesis-Setup
I also added the page to the wiki sidebar
https://github.com/jeffcampbellmakesgames/Entitas-Redux/wiki/_Sidebar/_compare/e48c39f1dd2bd6a95c78514adc8e6082a7a05d95

@jeffcampbellmakesgames Please review this
https://github.com/jeffcampbellmakesgames/Entitas-Redux/wiki/Genesis-Setup
I also added the page to the wiki sidebar
https://github.com/jeffcampbellmakesgames/Entitas-Redux/wiki/_Sidebar/_compare/e48c39f1dd2bd6a95c78514adc8e6082a7a05d95

Looks good! I made some minor changes, added some screenshots, and created a Getting Started section so it was clearer in that part of the sidebar that installation and genesis setup were related. If you're happy with how this looks and reads I would mark this as complete.

Looks good! I made some minor changes, added some screenshots, and created a Getting Started section so it was clearer in that part of the sidebar that installation and genesis setup were related. If you're happy with how this looks and reads I would mark this as complete.

Yes, this is fine :)
You can close this

Yes, this is fine :)
You can close this


The app does not work on macOS 10.15.7

Is it possible to rebuild the app to support old mac os?
In my case I got the following error:
You can’t use this version of the application “SwiftPlantUMLApp.app” with this version of macOS.
You have macOS 10.15.7. The application requires macOS 11.0 or later.

Hi @YuriiLokhmatovGitHub , I did necessary code changes in branch macOS1015 so that app can be compiled for deployment target 10.15.
Please try to build from branch macOS1015 and install the Xcode Extension.

Thank you very much Marco.
I checked it out and build. On 1st run I got the following error:
No Info.plist file in application bundle or no NSPrincipalClass in the Info.plist file, exiting
Then I set NSPrincipalClass = NSApplication in the plist.info file.
After that the app could launch, there was its window.
Though the issue is that there is no Xcode Source Editor in the settings. I could see such an option on my newer mac running Big Sur, but it looks I am out of luck on my old mac. Or maybe my Xcode 12.3 does not have such an option.
Any way thank you for your time.
Sincerely,
yurii.

Successfully built and run app from 1015 branch.
But same as @YuriiLokhmatovGitHub there is no label as Xcode Source Editor in the Extensions.
MacOS 10.15.7
XCode Version 12.4

Hi @YuriiLokhmatovGitHub and @emretufekci,
the above version app version below should work for macOS 10.5.7

Unpack SwiftPlantUMLApp.zip
Copy SwiftPlantUML to your Applications folder.
Launch SwiftPlantUML once. You can close it immediately afterwards (i.e. Quit from Dock).
Go to System Preferences > Extensions > Xcode Source Editor > select SwiftPlantUML

It works on my machines (macOS 10.5.7 and macOS 11) but if one of you could cross-check on your machine that would be awesome. Once I have confirmation I will push my changes create a GitHub release.
Cheers,
Marco


Fix Multithreaded Scene Loading

Hi!
I hope you don't mind, but I noticed that when loading scenes from threads that contain ControllerIcon textures I would get crashes. (https://docs.godotengine.org/en/stable/tutorials/io/background_loading.html)
This PR fixes the crash by ensuring the texture load happens on the main thread.
I also updated the import file for steamdeck dpad_right to match the other import files.

Thank you so much for your contribution! :heart:


Add link on top right of demo, leading user to specification repo

Features added

App.tsx: Add link to specification repo, wrapped in div for styling
App.csx: Add CSS selector 'nav' to style the div

Author
kev-ngo<EMAIL_ADDRESS>Linked issues
Closes #219

Moved this over to the demos repo


Implement TextEditor fixes/changes

TextEditor

Line indicator has weird positioning
Whitespace issues / lack of clarity in spacing

It appears that some of the quirks of the text editor may be bugs in the 3rd party text editor. An issue to investigate this issue has been posted at https://github.com/sstur/react-rte/issues/133.

Done:

Line indicator has weird positioning
Whitespace issues / lack of clarity in spacing


Install-Module -Name Az -AllowClobber failed

Install-Module -Name Az -AllowClobber
WARNING: Unable to resolve package source 'https://isrmsafetnuget.azurewebsites.net/nuget/'.

@williamwyz Thank you for the feedback!  We are investigating this and will get back to you shortly.

It appears that you may be experiencing an issue with your connection.  I would troubleshoot your connection to make sure that you have access to all required resources.  If that's not an option, it is possible to complete installation offline:
https://docs.microsoft.com/en-us/powershell/azure/install-az-ps?view=azps-3.6.1#install-offline
For verification, the command does work as I was able to successfully execute this on a fresh machine:

@williamwyz We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


The unit of 3d pose

Hello. Thank you for your work.
I wonder that what's the unit of 3d pose? I found it completely different from the annotations in 2d which are the pixel value. Instead, they are the value at around [-5,5] like the screenshot.
What's the actual meaning of these values and how to measure them in the real world?

It seems like the ground truth of 3DPW dataset.
The unit is meter.

But why the meter has negative value? Is it zero-centered, and the center point is pelvis?

No. the 'center point' you mentioned is the origin of the 3D coordinates.
About the details of 3DPW datasets, please ask the author of the paper.


[calendar] align to template

Purpose
What is the context of this pull request? Why is it being done?
align calendar skill to skill template
Changes
Are there any changes that need to be called out as significant or particularly difficult to grasp? (Include illustrative screenshots for context if applicable.)
remove bing spell check for luisRecognizerV3 do not support this option currently
Tests
Is this covered by existing tests or new ones? If no, why not?
Feature Plan
Are there any remaining steps or dependencies before this issue can be fully resolved? If so, describe and link to any relevant pull requests or issues.
Checklist
General

[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated the appropriate tests
[ ] I have updated related documentation

forget MockLuisRecognizer. Use LuisRecognizerOptionsV3

updated


Adding reference to script to simplify localization of existing project

Adding link to script trying to simplify the localization of an existing project without any previous localization.
The main pain is to have to perform multiple actions for each localization (copy the existing string, replace it by a tr(...) syntax, paste it in the Localizable.strings's file along with the correct key, replacing the variable by string type argument...)

@AliSoftware updated... I will cleanup commits at the end if required... cheers!

Nah, we just squash & merge 😄

👍 ⚠ the script is pretty raw but I believe it can be helpful ^^

@djbe Is it ok to merge, re: The Great Split? I believe this only affects the README and CHANGELOG, which won't move away from this repo, so even if we're in the middle of the Great Split this won't impact the big refactoring?

Jup, perfectly fine!


[EventRequest] -  codunit 423 "Change Log Management": OnAfterLogInsertion, OnAfterLogModification, OnAfterLogDeletion

Hi BC-Team,
We need the following events in codunit 423 "Change Log Management":

OnAfterLogInsertion
OnAfterLogModification
OnAfterLogDeletion

Reason:
we would like to summarise multiple change logs and monitor which change logs belong to a given database transaction.
The events would help us by indicating that all change logs entries have been inserted.
 procedure LogInsertion(var RecRef: RecordRef)
    var
        FldRef: FieldRef;
        i: Integer;
    begin
        OnBeforeLogInsertion(RecRef);
        if RecRef.IsTemporary then
            exit;

        if not IsLogActive(RecRef.Number, 0, 0) then
            exit;
        for i := 1 to RecRef.FieldCount do begin
            FldRef := RecRef.FieldIndex(i);
            if HasValue(FldRef) then
                if IsNormalField(FldRef) then
                    if IsLogActive(RecRef.Number, FldRef.Number, 0) then
                        InsertLogEntry(FldRef, FldRef, RecRef, 0, true);
        end;
        //new event begin
        OnAfterLogInsertion(RecRef);
        //new event end
    end;

procedure LogModification(var RecRef: RecordRef)
    var
        xRecRef: RecordRef;
        FldRef: FieldRef;
        xFldRef: FieldRef;
        i: Integer;
        IsReadable: Boolean;
    begin
        OnBeforeLogModification(RecRef);
        if RecRef.IsTemporary then
            exit;

        if not IsLogActive(RecRef.Number, 0, 1) then
            exit;

        xRecRef.Open(RecRef.Number, false, RecRef.CurrentCompany());
        xRecRef."SecurityFiltering" := SECURITYFILTER::Filtered;
        if xRecRef.ReadPermission then begin
            IsReadable := true;
            if not xRecRef.Get(RecRef.RecordId) then
                exit;
        end;

        OnLogModificationOnBeforeRecRefLoopStart(RecRef, xRecRef);
        for i := 1 to RecRef.FieldCount do begin
            FldRef := RecRef.FieldIndex(i);
            xFldRef := xRecRef.FieldIndex(i);
            if IsNormalField(FldRef) then
                if Format(FldRef.Value) <> Format(xFldRef.Value) then
                    if IsLogActive(RecRef.Number, FldRef.Number, 1) then
                        InsertLogEntry(FldRef, xFldRef, RecRef, 1, IsReadable);
        end;
        //new event begin
        OnAfterLogModification(RecRef);
        //new event end
    end;

procedure LogDeletion(var RecRef: RecordRef)
    var
        FldRef: FieldRef;
        i: Integer;
    begin
        OnBeforeLogDeletion(RecRef);
        if RecRef.IsTemporary then
            exit;

        if not IsLogActive(RecRef.Number, 0, 2) then
            exit;
        for i := 1 to RecRef.FieldCount do begin
            FldRef := RecRef.FieldIndex(i);
            if HasValue(FldRef) then
                if IsNormalField(FldRef) then
                    if IsLogActive(RecRef.Number, FldRef.Number, 2) then
                        InsertLogEntry(FldRef, FldRef, RecRef, 2, true);
        end;

        //new event begin
        OnAfterLogDeletion(RecRef);
        //new event end
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


[Doc] adjust TOC for administration and agg function.md

What type of PR is this：

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

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test


Rm expiration

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 50.82%. Comparing base (99e4b40) to head (7730e48).
Report is 1 commits behind head on main.

:exclamation:  There is a different number of reports uploaded between BASE (99e4b40) and HEAD (7730e48). Click for more details.
HEAD has 3 uploads less than BASE

Flag
BASE (99e4b40)
HEAD (7730e48)

oats-test
1
0

k8s-integration-test
1
0

integration-test
1
0

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1049       +/-   ##
===========================================
- Coverage   81.46%   50.82%   -30.64%
===========================================
  Files         139      139
  Lines       11108    11110        +2
===========================================
- Hits         9049     5647     -3402
- Misses       1544     5071     +3527
+ Partials      515      392      -123

Flag
Coverage Δ

integration-test
?

k8s-integration-test
?

oats-test
?

unittests
50.82% <100.00%> (-0.19%)
:arrow_down:

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

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


Nested template literals with nullish coalescing break syntax highlighting

JavaScript code:
const stringHighlighting = `${"any value" ?? `${"any value"}``;

function someOtherFunction() {
}

Is rendered as the following in CodeMirror:

Instead, the highlighting should look like the highlighting that is shown in the rendering of this GitHub issue. E.g. correct highlighting of the function keywords at all after the template string.
This was reported as upstream DevTools issue at https://crbug.com/1154241

You're missing a closing } between those two backticks on the first line, making this highlighting correct. Could it be that the person reporting the original issue was using an older version of CodeMirror?

You are absolutely correct. The markdown syntax highlighting in this GitHub issue is therefore invalid, which threw me off. I can confirm that upgrading to CodeMirror 5.58.3 in DevTools fixes this issue, so closing this issue as invalid. Thanks for the quick reply!


feat: Update to Rollup v4

Fixes plugin type incompatibility warnings when using with Rollup v4 or Vite v5
Changes

Updated all deps and dev deps
Improved "exports" field for correct ESM/CJS typing
Added separate named export and default export
Add Rollup v4 to peerDeps

Thank you so much for this! ❤ Looks good to me, I did some light testing and seems to work well. I'll release this as 0.3.0 later tonight.


Light baking - no good quality options (suggestion)

It's hard ( I would say impossible ) to hack to make the scene look great (sharp) with FULL baked light. It mostly overexposed, or/and with lower light (energy) or/and Energy Mult. levels it blurs out the scene:

Even after a long bake It's especially obvious in corners.
I played with all kinds of settings for hours with no success and found that only good results are produced by combing dyn. Light with Indrect+shadows bake.

Light baking is obsoleted in 3.0.


education

[Enter feedback here]

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

ID: 8da07311-409a-3de1-d501-93f6a798f6ff
Version Independent ID: 1878ec1a-a173-8234-65db-e7835f1c4438
Content: Frequently asked questions about Azure for Students and Azure Dev Tools for Teaching
Content Source: articles/education-hub/azure-dev-tools-teaching/program-faq.yml
Service: azure-education
GitHub Login: @rymend
Microsoft Alias: rymend

education

@Mahmoudsemsem
Could you please elaborate the query for better understanding. So, we can assist you further.


New line between CDX chunks

I know it doesn't matter when using with secondary index (part-xxxxx). But
curl -s https://aws-publicdatasets.s3.amazonaws.com/common-crawl/cc-index/collections/CC-MAIN-2015-40/indexes/cdx-00000.gz | gzip -d | head -n 3000 | tail -1
will get two CDX in one line:
197,49,203,173)/professionalassemblers<PHONE_NUMBER>3042 {"url": "http://<IP_ADDRESS>/ProfessionalAssemblers/", "mime": "message/rfc822", "status": "200", "digest": "YS2M252TZOZUT7DXH7ACPEODEMY6VMEQ", "length": "169526", "offset": "420509", "filename": "common-crawl/crawl-data/CC-MAIN-2015-40/segments/1443737913039.71/warc/CC-MAIN-20151001221833-00211-ip-10-137-6-227.ec2.internal.warc.gz"}197,52,186,146)/viz<PHONE_NUMBER>0328 {"url": "http://<IP_ADDRESS>/viz/", "mime": "text/html", "status": "200", "digest": "57MXUYXIIKQ7VDOKWGYOXYXCQNFBIBL6", "length": "2381", "offset": "255201", "filename": "common-crawl/crawl-data/CC-MAIN-2015-40/segments/1443736673439.5/warc/CC-MAIN-20151001215753-00158-ip-10-137-6-227.ec2.internal.warc.gz"}

It would be great if adding a new line between chunks around this code:
https://github.com/ikreymer/webarchive-indexing/blob/0005813e9007640a23c69e89af60f7fcf6e4206c/zipnumclusterjob.py#L157

Yes, definitely makes sense to have the newline, thanks!

Will be there for future indexes going forward.

Yay! I was just about to resort to Python to work around this when everything else I needed to do could have been done using Unix utilities.  I see it's already fixed in CC-MAIN-2015-48.
I don't suppose there's any chance of getting the fix applied to the historical indexes?


[KOGITO-4699] BDD tests for Jobs Service with PostgreSQL persistence

Signed-off-by: Karel Suta<EMAIL_ADDRESS>https://issues.redhat.com/browse/KOGITO-4699
Many thanks for submitting your Pull Request :heart:!
Please make sure your PR meets the following requirements:

[x] You have read the contributors' guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains a link to the JIRA issue
[x] Pull Request contains a description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[x] You've ran make before-pr and everything is working accordingly
[x] You've tested the new feature/bug fix in an actual OpenShift cluster
[ ] You've added a RELEASE_NOTES.md entry regarding this change

/jenkins test


Missing markdown output with YAML header

I am trying to create some slides using Weave.jl and Remark.jl. When I have a YAML header, I am missing slides in the markdown output. For example, weaving the file
---
title: "myTitle"
author: "myAuthor"
---

# myTitle

## mySubtitle

myAuthor

myDate

---

## Hello World

```julia
println("Hello")
```

with
Weave.weave("test.jmd", out_path = "build", doctype = "github")

produces the output
## Hello World

````julia
println("Hello")
````

````
Hello
````

When removing the YAML header,

# myTitle

## mySubtitle

myAuthor

myDate

---

## Hello World

```julia
println("Hello")
```

the output contains both slides

# myTitle

## mySubtitle

myAuthor

myDate

---

## Hello World

````julia
println("Hello")
````

````
Hello
````

I am using Julia v1.4.0 and Weave v0.9.4.

dup of https://github.com/JunoLab/Weave.jl/issues/197

This is not a dup of #197. The problem is not that the header is stripped, but that the piece between the header and ## Hello World is stripped.

ah I see. I just found the existing header striping logic is kinda awful and now trying to fix this, but lets's keep this open until it's solved.

It seems adding an empty julia block to the first slide works around this issue, i.e.,
---
title: "myTitle"
author: "myAuthor"
---

```julia
```

# myTitle

## mySubtitle

myAuthor

myDate

---

## Hello World

```julia
println("Hello")
```

produces correct results.


Deepspeed

I don't know much about these stuff but, Fun works in cp311 cu118 on Windows 11 with pre-compilied whl. At least that is what I could do by myself. Does your repo use deepspeed? Cause it really makes a difference.
I mention it as I see things being changed right now in the repo.

I don't know what wheel you're referring to? I've never seen deepspeed being used for inference, just training...had to check if that's possible and looks like they have inference mode. I have no idea if it would work here or if it would make a difference.
From what I've tried, using Torch compile with the latest torch version is the fastest method at least on 4090.

this one deepspeed-0.14.0+cu118-cp311-cp311-win_amd64.whl
and there is cu121 one.
But they are only for cp311 and from my tests work perfect with about 8 steps at 720px with a 4090.


[BUG] docs/readthedocs.org:sktime — Read the Docs build failed!

Describe the bug
the readthedocs tests are failing on multiple, unrelated PRs with the file

To Reproduce
to reproduce, create a PR into main. Dee, for example
https://github.com/alan-turing-institute/sktime/pull/1584
https://github.com/alan-turing-institute/sktime/pull/1583
Expected behavior
the test should pass since nothing has changed in the PRs that could feasibly and independently generate it

I have no clue about this CI stuff. Any ideas @freddyaboulton or @mloning? This may be relevant? Need setuptools 5.5.3?
https://gitmemory.cn/repo/pyodide/pyodide/issues/1930
https://github.com/pypa/setuptools/issues/2849

@TonyBagnall Can we try re-running the RTD build on those PRs? Looks like setuptools pushed a new release since the last commit on those PRs that should fix the problem.

yep fixed by the external package, thanks @freddyaboulton


Add Learn.md file

Add Learn.md file according to Gssoc guidelines

Please assign me the issue @4darsh-Dev

@debangi29  assigned to you.

Thank you.. I am working on it


Rust project refactoring

This moves all PyO3 modules into a single binary, reducing the overall memory footprint of Rust modules, allowing multiple modules to share .text sections.
Modules are exported via a single .so living in edb/server/_rust_native.
As the module loads, it places the submodules in sys.modules directly, allowing consumers to directly import from edb.server._rust_native._MODULE.
Logging
As part of this work, the custom per-project logging has been replaced with one single logger integration. This logger avoids asyncio and creates a blocking i/o thread using a unix dgram socket to send messages internally.
Logging may happen around Python shutdown which PyO3 doesn't handle gracefully (unhappily failing to acquire the GIL in some cases), so a blocking thread avoids any complexity around that, as well as any async I/O shutdown issues.
Log messages that are not enabled (ie: more fine-grained than info! in Rust) are filtered out in the Rust log subscriber, and are effectively free.

Looks good. I've not read the logger impl fully, but I concur on the approach you've outlined in the comment.
Should this refactoring also include the parser? It is used by the compiler process, so it would not benefit from merging binaries.

I think we can move the parser, but it doesn't make sense to include it in the mega-binary I believe at this time. It might be useful to share pages between the compiler and server but I don't think it's a pressing need right now.


Add Go To Implementation Api

Fixes #10806
Adds a new API for supporting  go to implementation type commands in languages. Implements an example for TS
The current approach added a new public interface called ImplementationProvider (a suggestions for a better name would be appreciated). This resulted in some duplicate code that I will try to clean up further. An alternate design would be to extend the DefintionProvider with an optional provideImplemtnation method. This would allow reusing our existing internal interfaces without as much duplication, but we may want to keep the two concepts separate. Please let me know if you have any thoughts one way or the other.

Removing myself from reviewer, unless there is a very specific reason why my feedback is needed in this area. Please let me know.

Good stuff - added some nit comments, ignored the actual implementation in the TypeScript extensions.
I would definitely go for a separate provider despite them being quite similar. My name proposal is TypeDefinitionProvider.

+1 for the separate provider. Name wise I would go with TypeOfDefintionProvider :-)

Thanks for the review. I've updated the interface names and am looking into the keybindings

@dbaeumer The language service API should probably pick up support for this feature, but I wanted to check with you to see that should block this PR. I think we can either:

Ship this new API as part of VSCode 1.9 and update the language server API to support it later
Hold off on shipping this API in VSCode until we can update the language service API to also support it

What are your thoughts on this?

@mjbvz we always ship new API in VS Code and then update the LSP later. Can you please file n issue here https://github.com/Microsoft/language-server-protocol so I don't forget to do it.

Done: https://github.com/Microsoft/language-server-protocol/issues/156


Create Bucket sort file folder

Create Bucket sort file folder and file

@s1042992 already implemented. Thank you for your contribution nonetheless.


Do you really support Node 10.X?

I originally setup my development environment a year or two back.  Recently I was having issues, so I uninstalled Node and reinstalled the latest 10.X version following your instructions.  I also uninstalled/installed GULP, YO, and
@microsoft/generator-sharepoint
When I attempted to use GULP, I received the following error:
"Your dev environment is running NodeJS version v10.21.0 which does not meet the requirements for running this tool. This tool requires a version of NodeJS that matches >=8.9.4 <9.0.0".
I found this reference where a user has the same issue and the response explains how to install multiple versions of Node: https://sharepoint.stackexchange.com/questions/255951/nodejs-not-compatible
https://github.com/coreybutler/nvm-windows/releases
After switching to version 8.9.4, I am able to use GULP and build my projects.  Is this happening because my project is built on an older version of the SharePoint Framework?

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

ID: 5bee09a8-022e-1b5a-4aab-d6e7c6216808
Version Independent ID: 49edce1e-f2be-e989-917f-c6b389363e2f
Content: Set up your SharePoint Framework development environment
Content Source: docs/spfx/set-up-your-development-environment.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Hi mkeysae2,
I am using V10.21.0 for SPFx web part development. It works well here.

Based on the doc, v8.x.x is for SharePoint Server 2016 and v10.x.x is for SPO.
I attach the test demo here, you may have a look.
Baker Kong
Microsoft SharePoint Community Support

Hi, yes this is because you are using an older version  of SPFx. My recommendation would be to use the latest version of the SharePoint Framework. Also, if these are scenarios where you are working on different versions of SPFx, my recommendation would be to look into NVM so you can have different versions of node running on your machine.

Thank you for the comments.  It makes sense now that this issue is because we are using an older version of SPFx.  I am running NVM so I can switch between versions.


Relative links in Atom  don’t work; turn into file: URLs (possible security issue?)

Specifics
In an Atom feed, when an entry’s link to the article’s web page is a site-relative URL (starts with /, like <link href=“/some/path/“/>), NNW misinterprets this as a filesystem path and in its GUI makes the article title link to the corresponding file: URL. As a result nothing happens when the title is clicked. (See below for an example.)
Security
I’ve learned to be very nervous about anything where untrusted content can produce file URLs. I could create a feed that would make clicking an article link launch an arbitrary app or document on the user’s machine — for example, using <link href=“/Applications/TextEdit.app/“/> would probably launch TextEdit. (I haven’t tried it.)
I don’t know if that’s dangerous on its own, but I’ve read about several real-world attacks that first download a dangerous file to a known location and then exploit a separate bug to open it. (Yeah, there’s GateKeeper, but there have been several vulnerabilities found in that lately too…)
Details
The feed in question is one generated by some code I’m currently working on. It’s passed the W3C feed verifier, so I tried subscribing in NNW and got the behavior above.
I don’t want to attach the feed itself because it links to some stuff I’m not ready to make public yet, but an expurgated summary is like:
<?xml version="1.0" encoding="utf-8"?>
<id>XXXXX</id>
<feed xmlns="http://www.w3.org/2005/Atom">
  <link href=“/path/to/site/"/>

  <entry>
    <id>XXXXX</id>
    <link href=“/path/to/site/path/to/page/"/>

Assuming the feed was downloaded from https://my.host/path/to/site/atom.xml, The expectation is that in NNW the article title would link to https://my.host/path/to/site/path/to/page/.
Note that the links are site-relative; I did this bc generating the feed is independent of uploading it to a specific host and I didn’t want to hardcode the hostname at this point.
It’s been a long time since I worked on a feed reader 👴🏻, but my recollection was that site-relative URLs are OK and are interpreted relative to the feed’s URL. Apparently not! I can fix my feed pretty easily; the only reason I’m filing a bug report is because the way NNW turns this into a file: URL is kinda scary.
Config
NNW 6.1 (6106)
macOS 13.2.1
MBP 16” 2021 (Apple M1 Pro CPU, 16GB RAM)

Thanks, Jens — we’ll look into this asap.

I was able to reproduce the file URL bug pretty easily with Atom. Haven’t checked with other formats yet.
Some preliminary good news: when I tried /Applications/TextEdit.app and similar, nothing appeared to happen on clicking the links, and the following appeared in the console, which suggests that WebKit is dealing with these to a certain extent.
2023-02-23 22:47:25.821361-0800 NetNewsWire[4975:144997] [Loading] 0x11a0704f0 - [PID=4983] WebProcessProxy::checkURLReceivedFromWebProcess: Received an unexpected URL from the web process
2023-02-23 22:47:25.821433-0800 NetNewsWire[4975:144997] [Process] 0x155907018 - [pageProxyID=7, webPageID=8, PID=4983] WebPageProxy::Ignoring request to load this main resource because it is outside the sandbox

App sandbox FTW!
When I went to update my code, I remembered about the xml:base attribute, which establishes a base URL for all <link> elements in & inside the element it’s attached to. So I added one to the feed element:
<feed xmlns="http://www.w3.org/2005/Atom" xml:base="https://MY.SITE/">

This didn’t change NWW’s behavior, though. It’d be nice to fix that.

Seems to be related to #3662 as well. I would love to see this fixed because I use Hugo without a baseURL (so that that the same site works on different hosts) and the RSS feeds it generates are therefore all broken with NetNewsWire.