Can't Store balance into a Wallet

I am trying to add dash to a wallet, but I keep getting a message Not enough balance (0) to cover burn amount of 1000000
Expected Behavior
I Expect the address that I've added dash / money to it, to be able to create Identities.
Current Behavior
error Not enough balance (0) to cover burn amount of 1000000 when trying to create Identity
Possible Solution

Steps to Reproduce (for bugs)

import { Client } from 'dash';

const clientOpts = {
  network: 'local',
  wallet: {
    mnemoic: null,
    // mnemoic: 'later van settle digital bone zone decrease year robust waste seat banner'
  },
};

const client = new Client(clientOpts);

  const account = await client.getWalletAccount();
  const mnemnoic = client.wallet.exportWallet();
  const address = account.getUnusedAddress() // abc

dashmate wallet:mint 10 --address=abc --config=local_seed
4.
const identity = client.platform.identities.register();

error: Not enough balance (0) to cover burn amount of 1000000
Context

Your Environment

Version used: "@dashevo/dapi-client": "*",   "@dashevo/dpp": "*", "dash": "^3.23.2",
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop, server, or mobile): ( Windows 10 )
Link to your project:

Hey there :) Glad to see you here
Seems like you just have a typo in your client configuration:
mnemoic: null,

The field for the seed phrase should be mnemonic. And since its undefined in your case, you just ending up having fresh, new keychain (and new pool of addresses) every run. So its working, you just minted up Dash for the addresses you don't have access for.
I double checked your commands, and with the correct field name and mnemonic, I was able to get money in my pocket.

that was really it 😅 . thank you !


CA-160 - Update EMR module to prevent installing aws provider 4.0

Updates version file to prevent the major upgrade to the AWS provider version 4.0.
Updates examples to use the newest version of the s3-module.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Set time on Linux server

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
WIP
Self-service

[X] I'd be willing to address this documentation request myself.

https://docs.datenschmutz.dev/docs/linux-time-date/

Done


removing token containers and comin soon docs

Description

Remove token container docs now that we will have a concept page
Removing coming soon note

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

:tada: This PR is included in version 1.63.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Update to Apache Lucene 9.9.2

Description
Update to Apache Lucene 9.9.2
Related Issues
See please<EMAIL_ADDRESS>
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
[X] Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@msfroh @mch2 @andrross please take a look folks, thank you


Add wranglerVersion options

There is an issue with the cloudflare/next-on-pages library that causes a "Script startup exceeded CPU time limit" error after using the publish command. This issue has been fixed in https://github.com/cloudflare/next-on-pages/issues/54. However, currently only the publish command from<EMAIL_ADDRESS>version can run properly, so I need the wranglerVersion feature.

This action uses the latest Wrangler version, which right now is 2.20. Are you trying to go back to 2.16 or use the beta branch?

I want to use version 2.16, so I would like to be able to specify the version of Wrangler.


inappropriate images on /en-US/docs/Web/Security/Insecure_passwords

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

URL https://developer.mozilla.org/en-US/docs/Web/Security/Insecure_passwords
Firefox password security indicators
paragraph one lock icon https://support.cdn.mozilla.net/media/uploads/gallery/images/2017-04-21-23-52-53-ba340d.png
paragraph two warning https://support.cdn.mozilla.net/media/uploads/gallery/images/2015-11-17-12-13-18-2faa61.png

I've deleted the images altogether. Thanks for reporting.


Crash on startup

[ ] I have read the README.md file and visited the Documentation to see if the information there helps.

Describe the bug
When starting with the config below, there is an error that prevents the mqtt-io from working
Expected behavior
A working code
Error messages and traceback
ERROR:
python3 -m mqtt_io config.yml 2024-03-22 19:07:15 mqtt_io.server [INFO] Digital output 'rele1' current value is False (raw: False) 2024-03-22 19:07:15 mqtt_io.server [INFO] Digital output 'rele2' current value is False (raw: False) 2024-03-22 19:07:16 mqtt_io.server [INFO] Digital output 'rele3' current value is False (raw: False) 2024-03-22 19:07:17 mqtt_io.server [INFO] Digital output 'rele4' current value is False (raw: False) 2024-03-22 19:07:17 mqtt_io.server [INFO] Digital output 'rele5' current value is False (raw: False) 2024-03-22 19:07:18 mqtt_io.server [INFO] Digital output 'rele6' current value is False (raw: False) 2024-03-22 19:07:19 mqtt_io.server [INFO] Digital output 'rele7' current value is False (raw: False) 2024-03-22 19:07:20 mqtt_io.server [INFO] Digital output 'rele8' current value is False (raw: False) Exception ignored in: <function Client.__del__ at 0x75d3b198> Traceback (most recent call last):   File "/home/pi/.local/lib/python3.7/site-packages/paho/mqtt/client.py", line 874, in __del__     self._reset_sockets()   File "/home/pi/.local/lib/python3.7/site-packages/paho/mqtt/client.py", line 1133, in _reset_sockets     self._sock_close()   File "/home/pi/.local/lib/python3.7/site-packages/paho/mqtt/client.py", line 1119, in _sock_close     if not self._sock: AttributeError: 'Client' object has no attribute '_sock' 2024-03-22 19:07:20 mqtt_io.server [ERROR] Exception in critical task: Traceback (most recent call last):   File "/home/pi/.local/lib/python3.7/site-packages/mqtt_io/server.py", line 1380, in _main_loop     await self._connect_mqtt()   File "/home/pi/.local/lib/python3.7/site-packages/mqtt_io/server.py", line 703, in _connect_mqtt     self.mqtt_client_options   File "/home/pi/.local/lib/python3.7/site-packages/mqtt_io/mqtt/asyncio_mqtt.py", line 101, in __init__     clean_session=options.clean_session,   File "/home/pi/.local/lib/python3.7/site-packages/asyncio_mqtt/client.py", line 34, in __init__     self._client = mqtt.Client(client_id=client_id, protocol=protocol, clean_session=clean_session, transport=transport) TypeError: __init__() missing 1 required positional argument: 'callback_api_version' 2024-03-22 19:07:22 mqtt_io.server [INFO] Reconnecting to MQTT broker (infinite retries remaining)
Config
mqtt:
  host: 192.168.x.x
  port: 1883
  topic_prefix: home
  ha_discovery:
    enabled: yes

# GPIO
gpio_modules:
  # Use the Raspberry Pi built-in GPIO
  - name: rpi
    module: raspberrypi

digital_outputs:
  - name: rele1
    module: rpi
    pin: 18

  - name: rele2
    module: rpi
    pin: 23

Hardware
Raspberry pi 3b+  and the mqtt server is on another host
System:

Raspbian GNU/Linux 10,
Python version:  Python 3.7.3
User you're running as: pi
Using a virtualenv?: no

Hi have the same issue, same system and the following configuration:
mqtt:
  host: 192.168.xx
  port: 1883
  topic_prefix: mqtt_io
  ha_discovery:
    enabled: yes

sensor_modules:
  - name: dht22_sensor
    module: dht22
    type: DHT22
    pin: 2

sensor_inputs:
  - name: temperature
    module: dht22_sensor
    interval: 10
    digits: 4
    type: temperature

  - name: humidity
    module: dht22_sensor
    interval: 10
    digits: 4
    type: humidity

This looks like #348 - downgrading paho-mqtt to the latest 1.x version should solve it.

how do you do that?

how do you do that?

If you installed it using pip, then you can downgrade with pip install 'paho-mqtt<2', which will get you the latest 1.x version.

Downgrading paho-mqtt did the trick for me. But I had to upgrade my ancient OS to Raspbian 11 first.

will be fixed with PR: #364


Add latest tag

Hi,
To keep aligned with a common practice and trivy-db, it would be good to have a latest tag pointing to the latest oci image.
Thanks,
N.

We don't think the latest is needed. A different version means breaking change. The latest tag should not be used.

I cannot see how this is a breaking change as the tag 1 stays as is and only the latest tag is added, f.e. see: https://github.com/aquasecurity/trivy-db/pkgs/container/trivy-db
I will not argue with you, close the PR if you stick with won't do.


[incubator-kie-issues-1131] test migration from V7 to code generation-13

In addition to previous changes associated,
Migrated tests within EscalationEventTest.Java
The test can be identified by referring to EscalationEventTest.java:
https://github.com/apache/incubator-kie-kogito-runtimes/blob/main/jbpm/jbpm-tests/src/test/java/org/jbpm/bpmn2/EscalationEventTest.java
Closes https://github.com/apache/incubator-kie-issues/issues/1131.

Merge when green.


Create an example dashboard

This would serve two purposes:

show-case how easy it is to build out a "Robot Operations Center" using Transitive components
some customers don't yet have their own web application could use this as a starting-point.

Stack:

vite-express
shadcn (incl. tailwind css)

Front-end

sidebar navigation: one tab each for

Devices
Video (webrtc-video)
Teleoperation (remote-teleop)
Terminal (terminal)
Health (health-monitoring)
Configuration (configuration-management)
(more later, incl. ros-tool and file-sync)

body: the selected tab

Devices tab: list of active robots (similar to Portal)
all other tabs:

on top: dropdown to select the robot
then show the respective capability for that robot below

Back-end

authentication: username + password

can reuse code from the Portal for this

an API endpoint to sign JWTs on-demand for authenticated users
uses .env file to store (from transitiverobotics.com), filled in by user of the dashboard:

transitiverobotics.com account name,
robot token,
JWT secret.

README
The README should suggest to users to fork the repo, fill in the values in the .env file, and how to start it (ideally just npm start).

Milestone 1

[ ] @chfritz to create separate repo
[ ] @jlurgo create account on transitiverobotics.com

[ ] @chfritz to set it to "free"

[ ] @jlurgo to add test device to his account (e.g., using provided Docker container)

[ ] install capabilities
[ ] add capabilities to dashboard

Milestone 2

[ ] use SDK to get a list of active devices


24H2 Support

I know public release was only recent but is 24H2 support on the way?

I updated the apply driver package script with the following:
Line 279
	[ValidateSet("24H2", "23H2","22H2", "21H2", "21H1", "20H2", "2004", "1909", "1903", "1809", "1803", "1709", "1703", "1607")]
Line 974
			"Windows 11" { 				switch (([System.Version]$InputObject).Build) {     				"26100" { 						$OSVersion = '24H2' 					}
I haven't personally tested this yet.

I updated the apply driver package script with the following:
Line 279
	[ValidateSet("24H2", "23H2","22H2", "21H2", "21H1", "20H2", "2004", "1909", "1903", "1809", "1803", "1709", "1703", "1607")]
Line 975
			"Windows 11" { 				switch (([System.Version]$InputObject).Build) {     				"26100" { 						$OSVersion = '24H2' 					}
I haven't personally tested this yet.

I did the same thing.  Updated the package script by adding the 24H2 build to it and made some changes to the TargetOSVersion parameter to "24H2".  Imaging a laptop now and it worked perfectly.

It worked like a charm, Thank you!!

Hey, dumb question, how do you build it ?
i changed the ps1 file for 24h2 but it always starts with the old one


Error while moving order from Incoming to Ready for pickup tab in Ship to Store

Current behavior
'Something went wrong' error is popping up while moving the order from Incoming to Ready for pickup tab in Ship to Store.
After that order is moved to the Ready for Pickup tab.
Expected behavior

Steps to reproduce the issue

Can you handle fixing this bug by yourself?

[ ] YES
[x] NO

Environment details

Browser:
OS:
Code Version:

Additional information

Hi @Dhiraj1405 Sir, I would like to contribute to this issue, can you please assign it to me.

Hi @Dhiraj1405 Sir, I would like to contribute to this issue, can you please assign it to me.

Not tagged with Hacktoberfest, please pick another issue

Closing issue due to lack of information and old creation date


YakYak.desktop example on wiki is not valid

I have successfully installed YakYak and I am now using it as my primary Hangouts client. Kudos on a great, simple, standalone app.
However I ran into trouble with the example YakYak.desktop file on the wiki at https://github.com/yakyak/yakyak/wiki

The "Add" line is not valid .desktop syntax and until I removed that the file was not valid. I think this was an attempt to say "add these lines below" but it's all part of the same block of code. I can fix that myself, but wanted to clarify intent here.
The binary files do not have icons in the same place as the source distribution, so I had to hunt around a bit. I ended up configuring mine to point at YakYak/resources/app/icons/yakyak-logo.svg.

My complete YakYak.desktop is provided below. Note I have not tried to build rpm/deb files, and it's possible they do this correctly.
#!/usr/bin/env xdg-open

[Desktop Entry]
Name=YakYak
GenericName=YakYak
Comment=Electron App from git
Path=/usr/share/applications/YakYak
Exec=/usr/share/applications/YakYak/yakyak
Terminal=false
Type=Application
X-GNOME-Autostart-enabled=true
Icon=/usr/share/applications/YakYak/resources/app/icons/yakyak-logo.svg
Name[en_US]=YakYak

Again, kudos on a great app. I'm glad someone finally did this.

Oh, other issue: the exec line in the example is wrong and needs to be a full path to the yakyak executable.

Revised YakYak.desktop with a useful Comment:
#!/usr/bin/env xdg-open

[Desktop Entry]
Name=YakYak
GenericName=YakYak
Comment=Chat client for Google Hangouts
Path=/usr/share/applications/YakYak
Exec=/usr/share/applications/YakYak/yakyak
Terminal=false
Type=Application
X-GNOME-Autostart-enabled=true
Icon=/usr/share/applications/YakYak/resources/app/icons/yakyak-logo.svg
Name[en_US]=YakYak

i updated the wiki trusting you 100%

yakyak does not install to any specific directory in linux if I'm not mistaken, shouldn't it be something like:
?
what are you using to install?
Path=/home/my-username/some/folder/path/yakyak/
Exec=/home/my-username/some/folder/path/yakyak/yakyak

@averissimo I did not see any canonical way to install yakyak until after I had just grabbed a prebuilt Linux tarball. I used the wiki to get it manually installed as a GNOME app and ran into this issue.
I installed yakyak under my applications dir just because I didn't know a better place. If there's a canonical place, that should be used in the .desktop file for packages, of course.

@davibe @averissimo I generified the paths in the wiki a bit.

yeah i understand your dilema, many places to place it /opt/yakyak or /usr/local/yakyak or /usr/local/lib/yakyak
Your changes seems great!
If we are able to switch our build framework to output deb, rpm, etc.. then this would become much better. There's a ticket open with that, hope someone picks it up!

Just wanted to add to this:
If you want to have YakYak appear in your menu, in the YakYak.desktop file, add this line under Type=Application;
Categories=Network;WebBrowser;
Then the app will appear under the Internet options in your menu.

@mxyzplx thanks, updated the wiki :)


[Task]: Connectors Comparison

Connector Name
other (None Listed Above)
Task Description
Comparison our connectors with the industry level connectors.

Duplicate of https://github.com/ballerina-platform/ballerina-extended-library/issues/136


Fix the random color function in the README

Math.round(Math.random() * 2) might produce the following sequence: 2, 2, 0. The find callback is called with 0, 1, 2, so in that case, it would return undefined.
Also, as vanderZwan pointed out on HN, you should use Math.floor instead of Math.round, because Math.round skews the probability (red: 25%, yellow: 25%, blue: 50%).
Also, if null should be a valid result, then I think it's better to be explicit:
['red', 'blue', 'yellow', null][Math.floor(Math.random() * 4)]

(P.S. Sorry, I was bored, and this is a cool library!)

Be careful:
var counts = {"red": 0, "blue": 0, "yellow":0};
for(var i = 0; i < 100000; i++) {
  counts[['red', 'blue', 'yellow'][Math.floor(Math.random() * 2)]]++;
}
console.log(counts);

Object {red: 50054, blue: 49946, yellow: 0}
I think you want:
var counts = {"red": 0, "blue": 0, "yellow":0};
for(var i = 0; i < 100000; i++) {
  counts[['red', 'blue', 'yellow'][Math.floor(Math.random() * 3)]]++;
}
console.log(counts);

Object {red: 33349, blue: 33325, yellow: 33326}

Haha whoops, you're right! I changed it from Math.round to Math.floor and forgot to update it.

  Would you be opposed to me just changing it to _.sample(['red', 'blue', 'yellow']) ?  This feels like it would extremely improve the readability of the example.

Haha even better!
 Would you be opposed to me just changing it to _.sample(['red', 'blue', 'yellow']) ? This feels like it would extremely improve the readability of the example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub (https://github.com/tannerlinsley/react-move/pull/1#issuecomment-295276423), or mute the thread (https://github.com/notifications/unsubscribe-auth/AAIhEMczyvoE1a3eDXbBqCJQ4MGlqSRWks5rxhClgaJpZM4NBP9v).

Thank you for caring guys.  I had no idea this library would get so much attention!


Configurator-O365

Hi there.
I'm getting a noification to link my account after every re-start. Other than that everything is working.
Hav I missed someting?
Thanks for the add-on!
Regards

Have a look in the logs and see if anything is being reported there.

2022-10-24 20:21:34.560 ERROR (MainThread) [homeassistant.components.sensor] o365: Error on device update! Traceback (most recent call last):   File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 493, in _async_add_entity     await entity.async_device_update(warning=False)   File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 703, in async_device_update     await task   File "/config/custom_components/o365/sensor.py", line 271, in async_update     data = await self.hass.async_add_executor_job(self._teams.get_my_presence)   File "/usr/local/lib/python3.10/concurrent/futures/thread.py", line 58, in run     result = self.fn(*self.args, **self.kwargs)   File "/usr/local/lib/python3.10/site-packages/O365/teams.py", line 720, in get_my_presence     response = self.con.get(url)   File "/usr/local/lib/python3.10/site-packages/O365/connection.py", line 847, in get     return self.oauth_request(url, 'get', params=params, **kwargs)   File "/usr/local/lib/python3.10/site-packages/O365/connection.py", line 836, in oauth_request     return self._internal_request(self.session, url, method, **kwargs)   File "/usr/local/lib/python3.10/site-packages/O365/connection.py", line 798, in _internal_request     raise HTTPError('{} | Error Message: {}'.format(e.args[0], error_message), response=response) from None

Please post your config redacting any secrets.

Also when posting it please use the smaller than greater than in the GitHub editor bar to format it correctly. If you can do that for the log entry as well, it would make it clearer.

o365:   client_id: "x"   client_secret: "x"   alt_auth_method: False   enable_update: False   email_sensor:     - name: inbox       max_items: 2       is_unread: True       download_attachments: False   query_sensors:     - name: "Example"       folder: "Inbox/Test_Inbox" #Default is Inbox       from<EMAIL_ADDRESS>     subject_contains: "Example subject"       has_attachment: True       max_items: 2       is_unread: True   status_sensors: # Cannot be used for personal accounts     - name: "User Teams Status"   chat_sensors: # Cannot be used for personal accounts     - name: "User Chat"

o365:
client_id: "x"
client_secret: "x"
alt_auth_method: False
enable_update: False
email_sensor:
- name: inbox
max_items: 2
is_unread: True
download_attachments: False
query_sensors:
- name: "Example"
folder: "Inbox/Test_Inbox" #Default is Inbox
from<EMAIL_ADDRESS>subject_contains: "Example subject"
has_attachment: True
max_items: 2
is_unread: True
status_sensors: # Cannot be used for personal accounts
- name: "User Teams Status"
chat_sensors: # Cannot be used for personal accounts
- name: "User Chat"

Please repost formatted correctly. Should show like this. Without this it isn't possible to see if it is formatted correctly.
o365:
  accounts:
    - account_name: roger
      client_secret: !secret o365_secret
      client_id: !secret o365_id
      alt_auth_method: False

cal_id: X
entities:

device_id: o365 Essies Calendar
end_offset: 24
name: Calendar
start_offset: 0
track: true

Please, please just post using the GitHub formatting icon in the toolbar. Please post log, config.yaml no o365_configuration.yaml.

To be honest, I would cut the config back to the basic, as I've shown above. Prove that works before you add any other sensors.

Changed the config to basic and not get a request to reconnect.

Ok, so build up from there. It could be you don't have permission for everything you are asking for. Note the comments in the example config about personal accounts. If you try to setup something which your O365 won't allow, you will get a security prompt every time.

Cool - Thank you for your help.

a Bit off topic but do you maybe have a card for the frontend to create a calander entry?

Sorry, I don’t have one but I use Atomic Calendar Revive available via HACS


Add files via upload

Your checklist for this pull request
Author name
Author: Nabia

About your game
What is your game about? Its Snake game , snake chasing pears.

**How do you play your game?
To play game Press WASD to move.  **

Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Closing this PR due to inactivity. If you want it reopened, ping me here.


Add Year token

Hey, thanks for the contribution! Just as a heads up, the app is currently undergoing a rewrite (you can see this in the v3 branch), and as a result this code will no longer be usable in the future. I'm going to go ahead and approve & merge this, and if a build comes out before the rewrite is released this will be included in it.


Try to enroll new device in response to 401 invalidToken error from s…

…erver
Problem Analysis (Technical)
Server returns 401 Invalid Token for enrollment request with device information from stale device object(old phone).
Some user can't reenroll on new phone within the app and had to reinstall the app in order to enroll new account
Solution (Technical)

Do the best effort to restore device information on client/server side
Retry enrollment request with new device registration in case of 401 Invalid Token

LGTM. Is there any case where enrollmentToUpdate could be included where this issue could occur? I guess the Re-enroll flow should not be called in that case.

We add enrollmentToUpdate into transaction context when we want to update(PUT) existing enrollment. I would like to omit retry logic for that case and only retry for new enrollment flows


operator merge

This merges the operator branch into "devel", at this point without addressing any of the other opens like updating documentation or moving deployment files.

https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/view/change-requests/job/PR-639/2/consoleFull got stuck during the direct-testing stress test. worker-8 did not terminate, and the last I could find about it is this attempt to run rmdir:
02:13:15.211  STEP: worker-8-volume-16: done, in 2m31.493899209s
02:13:15.211  May 12 22:08:03.547: INFO: ExecWithOptions {Command:[rmdir /var/lib/kubelet/plugins/kubernetes.io/csi/pv/pmem-sanity-staging.GeMjkK/worker-8] Namespace:default PodName:pmem-csi-node-testing-l78vz ContainerName:socat Stdin:<nil> CaptureStdout:true CaptureStderr:true PreserveWhitespace:false}

Usually it's quickly followed by Output: , but not here.

Tests runs are so slow, let me add a commit for issue #624 to this branch.

got stuck during the direct-testing stress test

Can't reproduce that locally.

Tests runs are so slow, let me add a commit for issue #624 to this branch.

In the CI, "make test" time was reduced by 16 minutes.

Tests runs are so slow, let me add a commit for issue #624 to this branch.
In the CI, "make test" time was reduced by 16 minutes.

Wav, This was a great improvement.

3647cbb75861c0994336faa072a4e70ac4cb88a3 has fixed testing on Kubernetes 1.18 (https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/view/change-requests/job/PR-639/16 and https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/view/change-requests/job/PR-639/17/ for the rebased commits). It's a workaround, but something that we need until Kubernetes does garbage collection as quickly and reliably as previous Kubernetes releases.
6c11ea7c77f86fcb01ce5193c1069d2b685796e2 helped a bit on the operator branch (compare https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/job/operator/ against https://cloudnative-k8sci.southcentralus.cloudapp.azure.com/view/pmem-csi/job/pmem-csi/job/operator2/) but there were still issues. This is mute now, though, because we are moving towards operator development on the "devel" branch.

And another clean test run... @avalluri please review and merge.

And another clean test run... @avalluri please review and merge.

Though I have not done an in-depth review, gone through the changes, and couldn't find any issues.
My feeling is that merging this PR is a high priority, Lets deal in devel even if there are issues discovered after the merge.
Saying that, Looking good to me.


MapService: add calcArea() and calcLength()

Give the MapService two high-level methods to calculate the length and area.
The MapService detects which kind of calculation is needed and does all the necessary transformation of the coordinates.
Then it calls the low-level methods of the OlCoordinateService. For getLength its implementation could be:
/**
	 * Calculates the length of an array of coordinates.
	 *
	 * The kind of calculations depends on the  `geodesic` parameter:
	 * When `geodesic` is set to `true`, the coordinates must be either in 4326 or 3857 and calculation is done in a geodesic manner.
	 * When `geodesic` is set to `false`, the coordinates must transformed in a projected SRID before and calculation is done on a projected plane.
	 *
	 *
	 * @param {Array<module:domain/coordinateTypeDef~Coordinate>} coordinates input coordinates
	 * @param {boolean} [geodesic=true] `true` if calculation should be done in geodesic manner
	 * @returns {Number} the length
	 */
	getLength(coordinates, geodesic = true) {
		const getGeodesicLength = (coordinates) => {
			const wgs84 = Geodesic.WGS84;
			return coordinates.reduce((sum, current, index, coordinates) => {
				if (index === coordinates.length - 1) {
					return sum;
				}
				const next = coordinates[index + 1];
				const r = wgs84.Inverse(current[1], current[0], next[1], next[0]);

				return sum + r.s12;
			}, 0);
		};
		return geodesic ? getGeodesicLength(coordinates) : new LineString(coordinates).getLength();
	}

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

15 of 15 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.001%) to 99.57%

Totals

Change from base Build<PHONE_NUMBER>:
0.001%

Covered Lines:
12041

Relevant Lines:
12046

💛  - Coveralls


How to quantize

How would I go about quantizing the pretrained model for execution on an accelerator? I'm guessing I would quantize the 2D model, then the 3D model, and then finally the fusion layer.

Could you give more explanation about what is quantization and accelerator? I am not very familiar with these terms.

My aim with CLOCs is to get it running on a small, indoor robot. The focus of my research is to be able to run CLOCs on a lightweight device, for which I will be using an edge TPU, a very tiny computer which is designed to run inference on models very quickly. PyTorch and Tensorflow support model quantization, which is a method of taking floating point weights and converting them to integer representations to save on model size and inference time without losing too much in accuracy. https://pytorch.org/docs/stable/quantization.html explains it pretty well.

Ok, I see, then I think you are right. First quantize the 2D and 3D model, then the CLOCs fusion layer.


Bug 1807128: Don't block on ctx.Done() if startup fails.

Operator startup errors aren't recoverable without restarting the
process, so it's critical that the operator process is allowed to
terminate in these cases.

/hold

/hold cancel

/cherry-pick release-4.4

/cherry-pick release-4.3


Need configuration to add files to ignore

I am working with mobx and it generates .g.dart files. I think it would be usefull to have a posibility to ignore them from barrel file.

Good idea @vytautas-pranskunas-! I will try to add it as soon as possible.
If you feel like contribuiting, I will look forward to merging some PR! 💯

What other generated filed do you think should be excluded?
I've thought of .g.dart and .freezed.dart

I think if you can add some ignore files array which can be modified then we do not have to think about all cases :)

I wish i have time for contributing :( But This feature would be super helpful ;)

I've implemented test, where .g.dart and .freezed.dart are excluded by default (it can be changed in settings).
I look forward to adding the possibility of inserting an array of file extensions that should be excluded when generating the barrel file. What I fear is that, for instance, if the folder has many files and the user configuration array has many, many file extensions to ignore, it may take a bit of time to run the command.
However, it would be a very extreme case.
I'll update the extension. I'll be glad if you could give me some feedback 😄

I will
On Mon, Feb 8, 2021, 3:27 PM Miquel de Domingo i Giralt <
<EMAIL_ADDRESS>wrote:

I've implemented test, where .g.dart and .freezed.dart are excluded by
default (it can be changed in settings).
I look forward to adding the possibility of inserting an array of file
extensions that should be excluded when generating the barrel file. What I
fear is that, for instance, if the folder has many files and the user
configuration array has many, many file extensions to ignore, it may take a
bit of time to run the command.
However, it would be a very extreme case.
I'll update the extension. I'll be glad if you could give me some feedback
😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mikededo/dartBarrelFileGenerator/issues/4#issuecomment-775187027,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA54ZF2VGYQMLGOUHZYXSX3S57YEBANCNFSM4XHJC6NA
.


Any plans to support new version 0.29?

Hi,
Are there any plans to support the new version of 0.29?

We're likely going to be upgrading our Sensu deployment to Sensu 1.x sometime in 2018 Q2, and likely will update this library's compatibility prior to that.  If you have specific needs prior to that, we're open to pull requests.

Glad to hear that. I cloned the repo. I will start working on upgrading the library. I want to use add results api endpoint functionality.
Also, I realized the gradle and other open source libraries versions needs to upgrade

Both updates to the version of gradle and versions of the open source libraries used would be changes I'm open to.  If you pursue a pull request, please constrain it to just one type of change at a time (for example, just upgrading gradle version, or just upgrading versions of dependencies, or just updating API compatibility) to facilitate easier review.

Agree. Will try to work in that way in coming days

This project has been deprecated, and will see no further updates.  Please feel free to fork if it's still useful to you.


[FS-200]: Delete Family Api Integration

Changes

Create DeleteFamilyMutation.
Integrate the mutation hook with family buttons.
Add failure and success feedback.

@ASafaeirad @AmirabbasJ
Guys, we have duplication in the FamilyDetail and FamilyTableRow files. what do u think? what's ur suggestions?

@ASafaeirad @AmirabbasJ Guys, we have duplication in the FamilyDetail and FamilyTableRow files. what do u think? what's ur suggestions?

What part is duplicated?


Importer: remove ImportAsMember inference support

This functionality is not actively in use and the last usage of this has
been removed.  Remove the infrastructure that is no longer in need.

CC: @gribozavr @hlopko @scentini

@swift-ci please test

Unfortunately we cannot take this as is. I will reopen Dmitri’s PR when we can.

Converting to draft to prevent accidental merge

@CodaFi - I don't think that opening @gribozavr's PR would be as helpful, the tree has diverged, so this change is likely more useful at this point.  Would you mind just converting this change to an open PR from draft when we can actually merge this change?

@compnerd Please rebase this, and let's get it merged. I want to commend you on your patience here - I know this was blocking good work you were doing elsewhere and I'm sorry it took so long to get the stars to align to make this happen.

@swift-ci please test

@swift-ci please test Windows platform

Build failed
Swift Test Linux Platform
Git Sha - 714eaefc7879567d902f9a071ad6b21599302656

@swift-ci please test Linux platform

@swift-ci please test Windows platform

@swift-ci please test Windows platform

@swift-ci please test Windows platform

⛵


add support for arm64 for docker images

Please add support for arm64 for docker images

I've added support in my branch if you want to take a look.
https://hub.docker.com/r/patrickdk/s3-proxy

I've added support in my branch if you want to take a look.
https://hub.docker.com/r/patrickdk/s3-proxy


Fix some generic casting issues.

Remove jcenter from subproject build scripts. This is redundant since it
is already declared in the root.

Thx @runningcode for this PR. I just wanna check the impact on the Rx chains (make sure there is no additional warning down the chain) before merging and releasing.


Push Permission Request JavaScript Question

@khmyznikov Firstly, thanks for creating ios-pwa-wrap much appreciated!
I have a newbie question if you have time. I have my PWA building and running on my iPhone, and I am trying to work out how/where I put this JS https://github.com/khmyznikov/ios-pwa-wrap#push-permission-request in my PWA.
Forgive me, but this is all very new for me.
My PWA is written in Vaadin, so I have a TypeScript client-side component with a simple button for the user to click to request push permissions. I have copied-paste the JS from that link into there, but this doesn't work. Would you happen to have any pointers on where this JS should go?

I realised I was adding JS into my polymer JS module, not a plain JS file. Upon doing this, I can confirm I get a native popup for push permissions!!


kubectl diff failed when creating the namespace and resources at the same time

What happened?
kubectl diff failed when creating the namespace and resources in the new namespace at one YAML file with multiple documents.
kubectl shows the below error.
Error from server (NotFound): namespaces "new-namespace" not found

What did you expect to happen?
Show message to create new namespace and resources.
How can we reproduce it (as minimally and precisely as possible)?
Exec to kubectl diff for the below YAML file.
$ kubectl diff -f new-pod.yaml

# new-pod.yaml
apiVersion: v1
kind: Namespace
metadata:
  name: new-namespace
---
apiVersion: v1
kind: Pod
metadata:
  labels:
    run: example
  name: example
  namespace: new-namespace
spec:
  containers:
  - image: nginx:latest
    name: example
  restartPolicy: Never

Anything else we need to know?
I tried to fix this issue, but I didn't decide how to fix it. I write my thinking to the below list.

handle namespace not found error at this line on diff.go.

It needs kubectl saves create namespace history on the series of diff commands to memory.
I think this way is hard to check the effect to create real resources because I think any effects are happening on the k8s clusters like admission webhooks.

actual create namespaces for kube-apiserver during exec diff command and delete those namespaces after kubectl diff finishing.

I think this way affects the cluster and maybe happen a few problems, in the case of kubectl exec on parallel or sigkill happening while exec kubectl diff.

Kubernetes version

$ k version --short
Client Version: v1.25.4                                                                                Kustomize Version: v4.5.7
Server Version: v1.23.5

Cloud provider

baremetal

OS version

# On Linux:
$ cat /etc/os-release
# paste output here
$ uname -a
# paste output here

# On Windows:
C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture
# paste output here

Install tools

Container runtime (CRI) and version (if applicable)

Related plugins (CNI, CSI, ...) and versions (if applicable)

Thanks for the issue @koba1t
This is the same behavior of kubectl apply --dry-run=server. kubectl sends to api server one resource at a time and api server can not know that user has an intention to create namespace prior to this resource. That's why, it returns an error.

handle namespace not found error at this line on diff.go.
It needs kubectl saves create namespace history on the series of diff commands to memory.
I think this way is hard to check the effect to create real resources because I think any effects are happening on the k8s clusters like admission webhooks.

Saving namespace history in kubectl does not fix the problem. Because everything is happening at api server. How can we ensure that diff output is correct after discarding the response of api server?

actual create namespaces for kube-apiserver during exec diff command and delete those namespaces after kubectl diff finishing.
I think this way affects the cluster and maybe happen a few problems, in the case of kubectl exec on parallel or sigkill happening while exec kubectl diff.

Diff is not allowed to create anything in cluster. Because creating something has an impact on cluster. What about the edge cases like you are trying to create namespace already existed and diff decides to delete it after executing difference?. Wouldn't that bring a lot of complexity.
I think, supporting this kind of complexity breaks the simplicity of flow which we don't want and brings about several edge cases. Asking user to create dependent resources before diff or apply --dry-run=server looks to me simpler.
I'd prefer closing this issue.

@apelisse @seans3 what is your opinion about this problem?

Yeah, this is hard, I have no clue how we can fix this properly, not without very complicated machinery.

That's working as intentded, sorry.
/close

C:\GitHub\01-hello-world-rest-api>kubectl diff -f deployment.yaml
The Service "kubernetes" is invalid:
* metadata.uid: Invalid value: "d77215bb-86c7-40b8-b505-d039a6365897": field is immutable
* spec.clusterIPs[0]: Invalid value: []string{"<IP_ADDRESS>"}: may not change once set

Am I also seeing the same issue? I keep getting this error when I dump the deployment.yaml file, change the image tag and issue a diff command.

Well, you seem to have a service in that deployment.yaml that you're trying to update in a way that makes the apiserver unhappy.

Two options here:
if ! kubectl get namespace newrelic &> /dev/null; then
    echo "Namespace 'newrelic' not found. Creating for first run..."
    kubectl create namespace newrelic
fi
(
    kubectl diff -f manifest.yaml || result=$?; result="${result:-$?}"
    [[ $result -eq 1 ]] || && echo "diff exited with $result; skipping" || exit "$result"
)

Or what I think is worse because it will hide other exit 2 errors.
(
    kubectl diff -f manifest.yaml || result=$?; result="${result:-$?}"
    [[ $result -eq 1 ]] || [[ $result -eq 2 ]] && echo "diff exited with $result; skipping" || exit "$result"
)

These assume you're using set -e in your script but will work regardless. The subshell is so the result isn't carried further into your script for the next time you use it.  || exit "$result" is reached for exit 0 or any other value that isn't skipped.


SUP-5165 - Nested supporting modules example

Results in the outputs that could be used to create a VM:
aws-iam-policies = {
  "emr_creator_policy_arn" = "arn:aws:iam::825693534638:policy/NestedExampleEmrCreatorPolicy"
  "s3_policy_arns" = [
    "arn:aws:iam::825693534638:policy/test-hbase-root-directory-bucket-read-write",
    "arn:aws:iam::825693534638:policy/test-hbase-logs-bucket-read-write",
    "arn:aws:iam::825693534638:policy/test-spark-logs-bucket-read-write",
  ]
}
aws-iam-role = {
  "tamr_instance_profile_id" = "nested-example-tamr-ec2-instance-profile"
  "tamr_instance_role_name" = "nested-example-tamr-ec2-role"
}
aws-security-groups = {
  "tamr_security_group_id" = "sg-072482e817b89a330"
}

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket

This PR doesn't appear to be linked to a DevOps/SRE jira ticket


[4.9] stateGuard state for choose action creator

Hey, I've just started playing with the new choose action creator. Amazing job @Andarist and @davidkpiano. I just saw the tests and was amazed to see even nested choose actions work. 🔥
Description
It would be very nice to pass the whole guardMeta object, including the state to the conditional expressions of the choose action creator.
entry: choose([
  {
    // state is undefined :(
    cond: (_, __, { state }) => state.matches('bar'),
    actions: assign<Ctx>({ answer: 42 })
  }
])

Expected Result
Passing the current state on top of context and event.
Actual Result
The third argument of the condition expression callback already exists.
When I log the stateGuard, this is what I get:
cond: {
  type: "xstate.guard",
  name: "isPasswordResetMode",
  predicate: ƒ (context, event, stateGuard), // <-- how do I get to this?
}

However, state is undefined.

Considerations
I'm not sure if this is SCXML compatible but since any choose action can be translated to a regular state transition with normal guards, this shouldn't be a problem right?
Additional context
XState 4.9.1

Great that you're playing around with it! By the way, for your specific example, there might be a better way to model it. You shouldn't use state.matches() in a guard function - that function should be part of the matched state naturally.

@davidkpiano I very rarely have the need to use state.matches in a guard. When I do, it's typically to check the machine state of orthogonal nodes. Is there a better way to do this?

Closing this issue. stateGuard.state is not available during the initialization of the machine. It should work right after. See the discussion here.

Is there a better way to do this?

Yes, using in: https://xstate.js.org/docs/guides/guards.html#in-state-guards
Remodeling might also help.

Is there a better way to do this?

Yes, using in: https://xstate.js.org/docs/guides/guards.html#in-state-guards
Remodeling might also help.

Is it possible to us the in predicate in choose action? If not, would it be possible to add it as a feature?

@FadySalama Can you ask this in a discussion? github.com/statelyai/xstate/discussions/new


add cluster related and rollback release open api

#2590 #2581

Codecov Report

Merging #2631 into master will increase coverage by 0.09%.
The diff coverage is 56.6%.

@@             Coverage Diff              @@
##             master    #2631      +/-   ##
============================================
+ Coverage     50.06%   50.15%   +0.09%
- Complexity     2046     2055       +9
============================================
  Files           412      413       +1
  Lines         12594    12642      +48
  Branches       1292     1293       +1
============================================
+ Hits           6305     6341      +36
- Misses         5841     5852      +11
- Partials        448      449       +1

Impacted Files
Coverage Δ
Complexity Δ

...p/framework/apollo/openapi/dto/OpenClusterDTO.java
87.5% <0%> (+87.5%)
5 <0> (+5)
:arrow_up:

...framework/apollo/openapi/dto/OpenNamespaceDTO.java
4.34% <0%> (-0.2%)
1 <0> (ø)

...ework/apollo/openapi/dto/OpenNamespaceLockDTO.java
9.09% <0%> (-0.91%)
1 <0> (ø)

...ramework/apollo/openapi/dto/OpenEnvClusterDTO.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../framework/apollo/common/utils/InputValidator.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...pollo/openapi/v1/controller/ClusterController.java
20% <0%> (ø)
1 <0> (ø)
:arrow_down:

...trip/framework/apollo/openapi/dto/OpenItemDTO.java
54.54% <0%> (-5.46%)
4 <0> (ø)

...mework/apollo/openapi/dto/OpenAppNamespaceDTO.java
52.38% <0%> (-2.62%)
7 <0> (ø)

...p/framework/apollo/openapi/dto/OpenReleaseDTO.java
4.34% <0%> (-0.92%)
1 <0> (ø)

...pollo/openapi/v1/controller/ReleaseController.java
10.52% <0%> (ø)
1 <0> (ø)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35e1302...4665fd8. Read the comment docs.

Coverage increased (+0.09%) to 53.71% when pulling 4665fd89d30c3df9a820e3d756cd6d51ef63bfa5 on nobodyiam:open-api-client into 35e130207d3944c856a62a20e8eef7f5a7171bb0 on ctripcorp:master.


Remove dev layer

Renames dev_layer to layer in semiannuals cfg
Removes obsolete old layer and zoom style

The diff preview looks weird let me check it

Ok the diff preview still does look weird on the PR but it looks ok in my Sandbox. I should have just deleted "layer" and renamed "dev_layer" to "layer" like you did in the annual cfg Alex.


S3 Proxy with bucket expiration

We are using bazel-remote with a proxy to an s3 bucket that has a 30 day expiration. I believe we've started to encounter a situation where the CI machine's bazel-remote cache retains an artifact that is dropped from s3. Because the artifact exists in the bazel-remote cache, it is never checked whether it exists in the s3 bucket, therefore it is never refreshed after it expires from s3. This means that developer builds (which are read-only on the s3 bucket) cannot find the artifacts and the CI never refreshes them.
Is there a way to configure bazel-remote for this sort of expiration? If not, is there an API that could be used to purge the cache periodically, or some other way to ensure the presence of artifacts in the backing store is regularly queried.

Hi, I don't think bazel-remote currently has a good solution for this. I suspect that most users of bazel-remote's s3 backend simply have deep pockets and never delete s3 objects, or maybe they simultaneously clear bazel-remote's disk cache- but these are just guesses.
There has been talk of possibly implementing a proxy-only mode (ie no local disk cache, proxy all requests to s3), but I have some performance concerns, and I don't have an AWS account to test with.
Another option would be to implement asynchronous uploads for every local disk cache hit (checking if they exist on s3 before uploading).

Would some kind of periodic bucket scan solve this?

I don't think S3 keeps track of the last access time for a blob, so I'm not sure what we would scan for.

You can with cloudwatch but I mearly meant to scan for deleted objects. I want to expire S3 objects and re-populate if needed.

Re-reading the original issue description, @djmarcin has CI talking to bazel-remote which proxies blobs to S3 (which expires blobs 30 days after they're created), and developers using direct read-only access to S3. And the problem is that blobs that stay in bazel-remote's disk cache are never repopulated in S3 after they're deleted there.
One potential solution (which https://github.com/znly/bazel-cache/ uses with GCS) is to make bazel-remote update the metadata for the S3 blob every time a blob is accessed from its disk cache, for S3 I guess this would mean copying the blob to itself, and then re-uploading it if it no longer exists. I think that copy can be done without uploading the data again.
Another, simpler solution might be to run a separate bazel-remote instance for developers which they have read-write access to, which has readonly access to the CI bazel-remote instance (using the http proxy backend). However this might also require modifications to bazel-remote if you're using compressed storage, which I haven't gotten around to yet.

I have been considering this problem for S3 too, as we want a more sophisticated solution which allows us to keep build graphs referenced while using --remote_download_toplevel or --remote_download_minimal.
The problem with those options is bazel will avoid downloading object files from 'cas', unless you need to relink that part of the build. If you have a cache expiration policy which deletes on last access time file irrespective of type, then in the rare case when you need to relink it will not be able to pull the required files from cas.
I wondered about adapting remote-cache such that when it validates the ac records, it will record all references to cas items in a linked redis instance.  We can then write another simple container that queries all items in S3, and expires records which  are no-longer referenced.
Our current solution parses http server access logs, which is error prone and slow due to the sheer quantity of data in the access logs.
If the data stored in redis is a simple hashmap of (cas/ac)/checksum date last accessed, then you don't need much information to exchange with redis as items only change at most once per day.  S3 sweep could be as simple as query all items in S3, lookup last access date in redis, schedule for removal if not accessed.
It works with reproducible builds where many ac records may reference one cas record, because it only takes one ac access to keep its cas records alive.


Distributed MXNet Import Error

Environment info
Operating System: Ubuntu 16.04.1 LTS
Python version and distribution: python 2.7
MXNet version: 0.9.3 Official Release Version
Downloaded from https://github.com/dmlc/mxnet/releases/tag/v0.9.3
But nnvm, dmlc-core, ps-lite and mshadow are not in the release version. So I clone the newest version of these four packages to their directories.
Error Message:
The compilation procedure of distributed MXNet (USE_DIST_KVSTORE = 1) is passed, but after the compilation and sudo python setup.py install, the error occurred when import mxnet.

import mxnet
[libprotobuf FATAL google/protobuf/stubs/common.cc:61] This program requires version 2.6.0 of the Protocol Buffer runtime library, but the installed version is 2.5.0.  Please update your library.  If you compiled the program yourself, make sure that your headers are from the same version of Protocol Buffers as your link-time library.  (Version verification failed in "/build/mir-pkdHET/mir-0.21.0+16.04.20160330/obj-x86_64-linux-gnu/src/protobuf/mir_protobuf.pb.cc".)
terminate called after throwing an instance of 'google::protobuf::FatalException'
what():  This program requires version 2.6.0 of the Protocol Buffer runtime library, but the installed version is 2.5.0.  Please update your library.  If you compiled the program yourself, make sure that your headers are from the same version of Protocol Buffers as your link-time library.  (Version verification failed in "/build/mir-pkdHET/mir-0.21.0+16.04.20160330/obj-x86_64-linux-gnu/src/protobuf/mir_protobuf.pb.cc".)
Aborted (core dumped)

The MXNet version on Single machine (USE_DIST_KVSTORE = 0)  works well.
I've found that the ps-lite package has used the v2.5.0 of protobuf, but the program needs v2.6.0. I don't know how to solve this.
Thanks.

@pluskid @mli

This issue is closed due to lack of activity in the last 90 days. Feel free to reopen if this is still an active issue. Thanks!


Reuse your Tiktok logged in browser instead of logging in to a new browser every time you post a video

Can I reuse my logged in browser to post videos? I don't want a new login every time I run the code

Yes. Your cookies are how your login state is kept (ie: the keep me signed in button), so you should have your cookies be valid as long as you would normally be signed into your browser

So, when driving a antidetect browser instance which has my tiktok account logined, do I still have to copy the cookies to use this uploader?


Could not find i18n-0.8.6 in any of the sources

Command:
docker run --name "blog" -v "$PWD":/usr/src/app -p "4000:4000" starefossen/github-pages

Logs:
Unable to find image 'starefossen/github-pages:latest' locally
latest: Pulling from starefossen/github-pages
ff3a5c916c92: Pull complete
8e2da6035957: Pull complete
42e99ed6de92: Pull complete
70c638bbd0d9: Pull complete
347588c2c13c: Pull complete
3ee934533da5: Pull complete
b0b3e87bd53d: Pull complete
Digest: sha256:22cdc5603886daea887da7890ef930551d04b470f527e875019167db835eec4b
Status: Downloaded newer image for starefossen/github-pages:latest
/usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/spec_set.rb:88:in `block in materialize': Could not find i18n-0.8.6 in any of the sources (Bundler::GemNotFound)
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/spec_set.rb:82:in `map!'
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/spec_set.rb:82:in `materialize'
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/definition.rb:170:in `specs'
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/definition.rb:237:in `specs_for'
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/definition.rb:226:in `requested_specs'
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/runtime.rb:108:in `block in definition_method'
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler/runtime.rb:20:in `setup'
	from /usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.1/lib/bundler.rb:107:in `setup'
	from /usr/local/bundle/gems/jekyll-3.6.2/lib/jekyll/plugin_manager.rb:50:in `require_from_bundler'
	from /usr/local/bundle/gems/jekyll-3.6.2/exe/jekyll:11:in `<top (required)>'
	from /usr/local/bundle/bin/jekyll:23:in `load'
	from /usr/local/bundle/bin/jekyll:23:in `<main>'

+1, I'm having the same issue.

Just ran into the same issue, but related to finding commonmarker-0.17.9 instead:
/usr/local/lib/ruby/gems/2.5.0/gems/bundler-1.16.3/lib/bundler/spec_set.rb:91:in `block in materialize': Could not find commonmarker-0.17.9 in any of the sources (Bundler::GemNotFound)

Removing Gemfile.lock helped for me


[Bug] Steam Deck low/unstable fps WebGL2

Checklist

[X] I will only use English in my report.
[X] I have used the search function for open and closed issues to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

Questions

[X] xCloud officially supports my country/region.
[ ] The bug doesn't happen when I disable Better xCloud script.
[ ] The bug didn't happen in previous Better xCloud version (name which one).

Device
Other
Device
Steam Deck
Operating System
SteamOS / Linux
Android app/Browser Version
both Chrome and Edge, linux
Better xCloud Version
3.5.1
Game list
No response
Reproduction Steps
If you change to WebGL2, the fps constantly jumps between 30fps/50fps, super unstable and pretty much unplayable. It does look better than the default renderer, but runs bad. In the Steam Deck performance overlay it kinda looks like the SD even uses less power than with the default renderer. (lower cpu/gpu usage, similar battery life ~7w). Lowering or increasing the gpu mhz does not change anything. (lower = more cpu power if needed / higher = more gpu)
The default renderer does not have any issues, stable 60fps. But does not look as good and the sharpening even on setting 1 is to much in my opinion.
Happens in Edge and Chrome (linux).
Screenshot/video
No response

Does it happen when sharpness value is Off?

Does it happen when sharpness value is Off?

Yes, and using sharpening does not decrease performance. It runs the same with or without sharpening. (webgl2)

so this is a problem with WebGL2, not the shader.
This issue is a duplicate of #433, so let's move to there.

chrome://settings/system
Did you have this setting enabled?

chrome://settings/system
Did you have this setting enabled?

Yes, its enabled.
I now also tried Firefox, runs way better BUT it locks at 90fps instead of 60fps (OLED 90hz) and the Steam Deck gets quite hot/loud + uses >12w instead of 7w. The CPU runs at max speed (3.5GHz).


LogEvent with parameters

I'm using your plugin in my app and thank you very much for creating it.
I'm currently using the LogEvent feature but it can only log the event name, so can you make the LogEvent method pass additional parameters?
Thank you!

Hello, I have made the LogEvent theme with parameters, I would like to upload it but while I see that theme, if you want I can share it with you.
Email<EMAIL_ADDRESS>you can write me with pleasure I can help you.

I also need solution for LogEvent Parameters.


Kong 2.0.3 start error

start kong error for openssl
openssl

kong

openresty

os

error log

Hi, your version of OpenSSL is too old. Kong 2.0.3 requires OpenSSL 1.1.1f.

Version dependencies can be groked from https://github.com/Kong/kong/blob/master/.requirements


Cannot pass filenames containing spaces through path

Probably true for a bunch of other weird characters too, like \n.

Through which path?

Of course! The last line of this snippet:
      - name: Run CodeSpell
        if: steps.changed-files.outputs.any-changed == 'true'
        uses: codespell-project/actions-codespell@v2
        with:
          check_filenames: true
          ignore_words_file: ./codespell-whitelist
          path: ${{ steps.changed-files.outputs.changed-files }}

Understood, thank you for the example. I must confess I don't know much about GitHub actions. I do see parameter path documented, but am not sure how it is processed. Could this be a YAML question more than a question on actions-codespell?

I've done some very surface level digging, and it looks like the problem occurs here:
https://github.com/codespell-project/actions-codespell/blob/406322ec52dd7b488e48c1c4b82e2a8b3a1bf630/entrypoint.sh#L49
INPUT_PATH is the value from the path input in the action, and it is passed into codespell as one-or-more arguments. Unfortunately, that means a file here is a file.md would get parsed by bash as four separate arguments: here, is, a, and file.md.
The safer way to pass an environment variable as an argument would be to enclose it in quotes:
res=`{ { codespell --count ${command_args} "${INPUT_PATH}"; echo $? 1>&4; } 1>&5; } 4>&1`

But I don't know if that would be an acceptable solution, since using a space-delimited list has been recommended a few times.

INPUT_PATH is the value from the path input in the action, and it is passed into codespell as one-or-more arguments. Unfortunately, that means a file here is a file.md would get parsed by bash as four separate arguments: here, is, a, and file.md.
The safer way to pass an environment variable as an argument would be to enclose it in quotes:
res=`{ { codespell --count ${command_args} "${INPUT_PATH}"; echo $? 1>&4; } 1>&5; } 4>&1`

But I don't know if that would be an acceptable solution, since using a space-delimited list has been recommended a few times.

Yes, we shouldn't quote it within the command @SamWilsn as that would preclude all those options. As well as if people wanted to use single quotes or backslash escapes or...
To quote (if you'll pardon the pun) yourself:

The safer way to pass an environment variable as an argument would be to enclose it in quotes

So if you quote your argument(s) into actions-codespell, it should all work as intended.
Your "changed-files" step implies it could output multiple files already, so that would also break if it was quoted.
Can you not get your existing previous step to quote the arguments, or use xargs?

I did actually attempt to quote my arguments, but ran into a weirdness with POSIX shells. You can see this effect with:
$ export CHANGED_FILES="'first file' 'second file'"
$ cat $CHANGED_FILES
cat: "'first": No such file or directory
cat: "file'": No such file or directory
cat: "'second": No such file or directory
cat: "file'": No such file or directory

Yeah I'm not sure sorry.
I'd agree that either of these work:
cat "first file" "second file"
cat: 'first file': No such file or directory
cat: 'second file': No such file or directory

cat first\ file second\ file
cat: 'first file': No such file or directory
cat: 'second file': No such file or directory

But yes, from a quick try I can't sort a way to do it in one environment variable. I suspect it really wants to be an array of some sort.

With other actions, I've been getting by with a hack of splitting on newlines (my "changed files" action will error if it finds a filename with one), but that isn't a general solution.
What would you think about re-implementing your entrypoint in python, and taking JSON as input?
GitHub Actions has great support for JSON (toJSON / fromJSON), you could implement backwards compatible support (keep path as is, put the new functionality in a new input), and requires no new dependencies since you already use a Python base image.


Get the parent node when dragging:finish

I'm trying to get the "new parent" of a node after drop it in that new parent node. I'm using the event node:dragging:finish, like this:
<tree
    :data="treeData"
    :options="treeOptions"
    ....
    ....
    @node:dragging:finish="dragFinished"
    ref="tree"
>

And in the handler
dragFinished(node) {
    console.log(node.parent);
}

However, the console shows a null value.  Is this an expected behavior or a bug?
It fact, if I want to get the new parent of the node I have been forced to do something like this:
dragFinished(node) {
    var theNode = this.$refs.tree.find({id: node.id})[0]
    console.log(theNode.parent);
}

And then with these little ugly code I can get the parent.

Looking for the same functionality. @amsik have you had a chance to look?


Publish workflow updates

This PR makes several changes to the publishing workflow to solve issues observed during the November release.

Breaks each library's publish into a separate job

Enables manually triggering a publish of any combination of libraries (provided the versions have already been bumped)

Adds a script that checks npm and waits for the recently published version to be available before publishing dependent libraries

Uploads CDN markdown updates as an artifact so that they can be recovered if a later step fails

Includes a bootstrap step to symlink local dependencies in order to prevent failures when npm run build:all is run in the prepack step

Codecov Report

Merging #4244 (6109f0b) into dev (d2cd0cd) will not change coverage.
The diff coverage is n/a.

Flag
Coverage Δ

*Carryforward flag

msal-angular
96.39% <ø> (ø)

Carriedforward from d2cd0cd

msal-browser
87.77% <ø> (ø)

Carriedforward from d2cd0cd

msal-common
85.41% <ø> (ø)

msal-core
77.07% <ø> (ø)

Carriedforward from d2cd0cd

msal-node
81.93% <ø> (ø)

Carriedforward from d2cd0cd

msal-react
94.19% <ø> (ø)

Carriedforward from d2cd0cd

*This pull request uses carry forward flags. Click here to find out more.


is there an API for "Refresh Content" ?

I do not see any API that is available to "Refresh Content" as part of QnAMaker. I believe, even if there is an API to "Refresh Content", it may not work as we do not have api for "Train" ( #106 ). Is my understanding correct?
If that is the case, is it possible to let me know if "Refresh Content" api is in your roadmap or is there any other workaround ?

@dhinag, can you elaborate on what you mean by "refresh content"? Do you mean refresh a kb, the contents of one, or something else?

@dhinag, I'm looking into this and hopefully will have a response for you before too long.

@dhinag, you can achieve this by making an update call against the knowledge base using the Update Knowledge Base API. You can read documentation here on how to do so. This link is for building in C#, but there are instructions for Node, Java, and Python, as well.
When you make the call, your JSON body should be: {"add":{},"delete":{},"update":{"urls":["your URL"]}}
Hope of help!

Thank you, Steven. I'll give this a try.


Update user info

Seeing individual roles in the channel

Hey, thanks for the suggestion. I added user roles to the user info command in the latest commit.


Double entries / filter by installed

Hi!
This is a pretty cool TUI for steam, I gotta be honest.
Two things that appeared to me though:
I get multiple entries for the same game, if they are installed:

Is this an issue of your implementation?
Also, would it be possible to add a filter to only show installed games? Would make things easier for ppl with larger libraries.
I know some rust, so If you point me to the right places, I might be able to modify that myself.
Cheers!

Does this solve your issue? https://github.com/dmadisetti/steam-tui#hiding-games
If the entries have different ida, then it's a steam thing. Let me know if they have the same id

I saw your branch, does this seem to work?
https://github.com/Sir-Photch/steam-tui/commit/d27c561edc90fe0d18aa6764d0a74e81b8f8c197
If you create a PR I'll merge it

There are no duplicates anymore, but launching games does not work properly on my machine. Maybe you have a try and see if it still works for you? I am not sure whether this is a breaking change or it's just an issue with my system.


use gradle jvmToolchain for setting jvm compilation internally rather than external installation

@F43nd1r in version 1.5.0, jackson 2.16.1 was added,

jackson are using multi-version jars, 2.16.1 was added java 21 for the first time

this causes a gradle issue when applying this plugin, throwing an error Unsupported class file major version 65
the only fix I could find is upgrading to gradle 8.5 onwards in order to fully support java 21

okay, this is the first I hear of multiversion jars. Is there any fix I can apply on the plugin side?


Share zenodo files for Scriberia image (Book Dash June 2024)

Publish scriberia images from BookDash June 2024 @aleesteele

Great! We'll do this on Monday, 30 September 😄


Rewrite New-TemporaryFile in C# #2732

Literal translation of New-TemporaryFile to C# cmdlet.

Hi @kittholland, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

And the command to the UNIX version of the psd1 to fix the build error.

@kittholland Thanks for your contribution!
My question about the copyright header was not addressed to you. Please don't rush to remove it.
With this in mind, the PR is LGTM. Please wait review from PG.

Are the tests in test/powershell/Modules/Microsoft.PowerShell.Utility/NewTemporaryFile.Tests.ps1 sufficient, or should there be more?

It would be nice to test i/o on the file.

I am happy to add an additional test regarding IO but I am uncertain if testing IO after the new file was generated is within the scope of this command.
I could validate additional attributes of the resulting file, for example that the extension is tmp, that the length is 0, and that it was created during the span of testing, but I'm not certain there are any conditions that would ever cause those tests to fail.

I believe that there is no need to examine the underlying level. We need to check only the cmdlet.
$tmp=New-TemporaryFile
Get-Content -Path $tmp.FullName | Should Be $null
$teststring ="any test string"
$teststring | Out-File -Path $tmp.FullName
Get-Content -Path $tmp.FullName | Should Be $teststring
Remove-Item $tmp.FullName -Force

We can merge as soon as the conflicts are resolved.

Conflicts have been resolved, thanks.

@kittholland - can you rebase? The PR now has a bunch of commits it shouldn't.

@kittholland Thanks for your contribution!


Does array_to_bounding_box_table work?

At some point while working on #194 (at the level of this commit: 3f2ba78562fb9e8a9eb286fd9a0e8586e833ab96), I observed that the bounding boxes for a certain set of (organoid) labels are wrong, namely they are slightly shifted from the organoids -- see below.
I don't know, yet, if the error is with the bounding boxes or with the way I am using them in the prototype/example.

First guess: the origin position (that is used to shift ROIs when computing indices, in convert_ROI_table_to_indices) is currently obtained by minimization of some relevant columns in the ROI table, but this is probably not consistent - and it may become especially visible when the table only includes a few very sparse ROIs.

First guess: the origin position (that is used to shift ROIs when computing indices, in convert_ROI_table_to_indices) is currently obtained by minimization of some relevant columns in the ROI table, but this is probably not consistent - and it may become especially visible when the table only includes a few very sparse ROIs.

Just as an observation, the current workaround that fixes bounding boxes is to provide an additional (optional) origin_xyz argument to convert_ROI_table_to_indices. In some cases (which ones? for sure the ones where ROIs are sparse objects that do not cover the whole well) we may want to set this by hand, instead of looking for the minimum values.
In the current case, it turns out that during secondary labeling the "correct" value to set is (0,0,0), but it is not immediately clear why. What is clear is that it produces the correct boxes, see below:

Recap:

In convert_ROI_table_to_indices we are (currently) always resetting the origin when computing indices from physical-units ROIs.
This is correct as long as ROIs cover the entire well, since the "inferred" origin is always the same for all ROI tables.
This is not correct when ROIs define only some sparse regions (e.g. organoids), because the "inferred" origin would be an arbitrary point (the minimal coordinates across all organoids).
We fix this issue here via #305, by exposing a reset_origin argument in convert_ROI_table_to_indices. The default is True, so that nothing changes in the current behavior. We defer to other issues (like #195) the actual choice on when to set reset_origin=False.

Thanks for digging into this @tcompa !
Maybe good to briefly touch on on Wednesday (or at some other call). Not fully sure I understand the finer parts of how we're using these origins and when we'd want to be resetting them.

Maybe good to briefly touch on on Wednesday (or at some other call). Not fully sure I understand the finer parts of how we're using these origins and when we'd want to be resetting them.

Sure thing.
I'm moving on with this PR, since for the moment it simply enables a possible change of behavior - without ever activating it in the current tasks. Let's re-discuss it in detail.

Briefly, my current understanding is:

We do reset the origin when building the well&FOV ROIs, at the beginning of a workflow
We should never reset the origin later, e.g. when building label ROIs. We are currently doing it, but probably we shouldn't. And the reason why we never had any problem is that we always used ROIs that cover the entire well.


Fixed non-monitorable areas triggering overlap

Non-monitorable areas are never removed from the monitor query of other areas. This makes areas that turn non-monitorable while overlapping with other areas get stuck in the other areas' monitor query and trigger false overlaps.
This is a  fix for issue #9148.

Can you please rebase and discard all other commits?

 I'm still trying to get a grasp on git. Sorry for the mess.
What I did is that I realised I made the PR from master and it seemed more logical to do it from a separate branch. So I made a separate branch with the commit fixing the issue and on a master branch I reverted that commit and rebased with godot/master hoping to get my master in sync with godot/master.
I can rebase the branch I created and open a new PR from it I guess? Any advice how to simply sync my master with godot/master without all the rubbish? Or is it betetr just to create a new master branch and abandon the old one?

You can "reset" it to origin master and do this change again.

Thanks, I'll give it a try!


Add a flag to the Check goal to include direct plugin artifacts

Fixes #61
Add functionality to check the age of plugins with an extra configuration option.

Add a new parameter includePlugins to CheckMojo.java to include plugins in the age check.
Update execute method in CheckMojo.java to include plugins in the query if includePlugins is true.
Add logic to filter and check the age of plugins similar to dependencies in CheckMojo.java.
Update DependencyResponse to include a p parameter representing the packaging type and add a method to check if the dependency is a plugin.
Update README.md to document the new includePlugins configuration option and update the command line example.
Add tests in CheckMojoTest.java to verify the functionality of checking plugin ages and the includePlugins configuration option.
Add methods in TestFakes.java to create Artifacts instead of Dependencies and mock client responses for plugins.
Update exampleResponse.json to include a plugin example.
Update QueryClientTest.java to validate the maven-help-plugin is present and mapped correctly with the packaging type.

For more details, open the Copilot Workspace session.

PR awaits fixed version of jackson-jr for record deserialisation with instance methods. Should be released with 2.18.2 (available in 2.18.2-SNAPSHOT)


feat(Terra): Terra Smart Contract Script

Task linked: CW-5719 [SDK] Terra Integration - smart contract


Update SDK used to compile calculator

Increase Minimum version to 19041 and SDK to build with to 22621

/azp run


Cannot load XML with embedded binary data

Hi all,
I have an XML file exported from SQL Server that embeds a field of binary data.
I use spark-xml to load this XML file, but the result is just an empty DataFrame.
Any suggestion how to deal with such file?
Thanks.

Would you mind share your codes?

This is the code I wrote to load the xml file:
val fn = "test.xml"
val xml_data = spark.read
    .format("com.databricks.spark.xml")
    .option("rowTag", "Record")
    .load(fn)

and the content of test.xml file
<Record><Id>1</Id><Data>H4sIAAAAAAAAC7PxCPH1sbPxcHV0sbMJ8QzxcbULzSvJLMlJTbHRh/BtgkMigZSBjT6EYaMPUe3k7xJpx8tlE2AXkpGqkJZfoZCYl6JQAmSn5+enKKQXJRakFuvZ6AdAFAGNCfL3c7dzB0rqgcwC84iQttEH2wS0FuRUAMzGgnKxAAAA</Data></Record>

I just run the code again and the result is no longer an empty DataFrame.
This is quite strange. What happened here was that I copy the content of the xml file exported from SQL Server and paste to another file, then everything seems to work.
My guess is that the binary data is encoded in to string while copying-pasting into the SubLime Text editor.

Hm.. I tried few times but I could not reproduce. Will probably take another look soon when I have some time.

I resolved this problem. What I need to do was setting the option 'charset' to 'UTF-16BE' or 'UTF-16LE'. Then, I can load the xml file correctly to spark.
val fn = "test.xml"
val xml_data = spark.read
    .format("com.databricks.spark.xml")
    .option("rowTag", "Record")
    .option("charset", "UTF-16BE")
    .load(fn)

The other encoding (UTF-8 or UTF-16) didn't work.


SafeDatabase class that checks the DB existence on the filesystem

It has the same API as SQLite3::Database but #new and #open mean different things now:

#new causes the database to be created, skipping the existence check
#open causes the database path to be checked, raising a SQLite3::DatabaseNotFound exception if it doesn't exist

Before the following methods pass to super the check is done:

.execute
.execute2
.execute_batch
.query

Should fix the phantom disk usage bug identified in #173

The problem you identified in #173 would also occur with just a regular file handle.  Why is SQLite3 special?

Well, it's not special... except that we are treating a file handle like a database, so we should take care to ensure that the database is always there, and if it's not raise an error to tell the calling program that it's database disappeared to:

ensure data integrity (by not writing to a deleted database)
ensure file system integrity (but not writing to a phantom file)

I think this is the responsible thing to do, but obviously it will be slower than the vanilla implementation and some people might not care, so they shouldn't be nannied by the library they are using (especially if they would rather do the check elsewhere).  I think it would be handy for people to have a class they can use that will do these checks for them, and that's why I did this PR.

I don't think it is the responsibility of the gem to look for deleted files. That is for the implementation using the gem. What I think would be more useful is more access to the mode flags passed to sqlite3_open_v2. For example, today, you can only test for the existance of the file on SQLite::Database.new when you include { readonly: true }, because if you don't use :readonly in the options, the C code uses sqlite_open or sqlite_open16, both of which default to SQLITE_OPEN_CREATE.
I have to work around this in order to open a database for writing, and fail if the file does not exist. In C, I would just sqlite_open_v2 with mode SQLITE_OPEN_READWRITE (i.e. leave off SQLITE_OPEN_CREATE). I think this is the appropriate level of responsibility of the gem.

Is it not a good idea to include this class as a kind of util class to help users of the library?  Or at least include it in an example somewhere in case users of the library don't know about this potentially dangerous pitfall in SQLite3.
It's all about making the library easier to use and reason about as right now the principle of least surprise is not in effect.  IMHO this library should abstract away the inefficiencies and pitfalls of SQLite (of which there are many, LOL) to make it easier for everyone to use.
Does that make sense?

If you decide not to merge this PR then you should at least include .exist? and .path methods on the main database class so that users can protect themselves.

@penguinpowernz well, I agree and disagree. I don't think that it should check for the existence of the file on every command, IMHO. But it would be nice if it checks for the existence of the file at open, using the mode flags already available from SQLite. If you had access to the full set of mode flags of sqlite_open_v2, then you could ask the open to fail if a read/write database file did not exist.
In any case, it's the maintainers of the gem who decide, not me. ;-)

I don't really want to maintain this class, but I'm not against adding code to enable someone else to maintain it.

If you decide not to merge this PR then you should at least include .exist? and .path methods on the main database class so that users can protect themselves.

Would providing just path be sufficient?  (Though I'll probably call it filename)  The other problem is that a SQLite3 connection can actually be composed of multiple files.  See ATTACH DATABASE.  This means that you need to provide the database name that you care about (though we could just default that to "main" for the filename method).

If you had access to the full set of mode flags of sqlite_open_v2, then you could ask the open to fail if a read/write database file did not exist.

I agree with this too.  I think it's an API oversight that you can't provide the mode flags.  We should make it take those flags.

@tenderlove I would like that mode access to sqlite_open_v2 myself, so I'll take a look at the code again in database.c and propose a solution.

I would like that mode access to sqlite_open_v2 myself, so I'll take a look at the code again in database.c and propose a solution.

Thanks.  I think we can either add a third option to initialize on the database, or use the options hash better.  Right now you can kind of toggle the mode, but maybe just taking a bitmask integer would be less maintenance work (in case sqlite3 adds more options, we don't need to update hash keys, etc).

But it would be nice if it checks for the existence of the file at open, using the mode flags already available from SQLite.

Hey yea, could have an open! method maybe, which raises an exception if the file doesn't exist?  Most people should be used to that convention from rails .find! right?

I don't think that it should check for the existence of the file on every command, IMHO

Yea that would be slow, but it would offer protection against POSIX systems which let open files be deleted.  But then there's the portability problem....  If a library uses the SafeDatabase class and it's running on a mac they'll get no benefit, just a speed penalty :(

The other problem is that a SQLite3 connection can actually be composed of multiple files

Shit, I totally forgot about that.  Could possibly make .filename into .filenames and return an array? IMHO .exist? should be added just as a convenience method (db.exist? looks nice and is idiomatic) and make it check all the files in that array.  Although that's not very useful when it comes to figuring out which file is non-existent, but that possibly could be solved with a .exist! method which raises an error containing the filename in the message (spitballing here).

I think the easiest, and least disruptive, is to add an option called 'mode' to the options hash, along with Ruby constants for the mode bits used now by SQLite3.
If "mode" is not in the options hash, then you get the behavior exactly as it is today:
int mode = SQLITE_OPEN_READWRITE | SQLITE_OPEN_CREATE;
or, if options[:readonly]:
mode = SQLITE_OPEN_READONLY;
This code would not change. Existing applications see the same behavior.
If options[:mode], then this sets the mode, and again, options[:readonly] would add SQLITE_OPEN_READONLY to this (because otherwise you might be surprised). So, options[:mode] is an integer, and replaces this line:
int mode = SQLITE_OPEN_READWRITE | SQLITE_OPEN_CREATE;
Adding the Ruby constants would be a nice syntax sweetener. I will code up a pull request.

Hmmm... that solution with the the bitmask seems ambigous and not ruby like (looks more like something you'd see in C code)... but thats probably just be me. Wouldn't opening it read only mean that you couldn't write anything into the database? Seems weird.

@penguinpowernz In the end, I kept it simple and Ruby-like, in this pull request.

Closing this due to the agreed consensus, maybe we could add this class to the wiki for POSIX users who more worried about data integrity than performance?


initial implementation of rw_array based on const generics

Here's another idea I had.  There are the existing functions on cell owners rw, rw2, and rw3, which can borrow multiple cells at a time, but that method is limited by how many arity variants this crate wants to implement.  With const generics, we can provide a version of the rw function which supports many more simultaneous borrows.
This is limited to only an array of cells with the same generic type, although in practice that is a very common use case.
I added this behind a const_generics feature flag, so that it could be added without bumping MSRV.  By putting the methods in a separate module, an older compiler can still compile it as long as that feature flag disabled.  It has the unfortunate side effect of requiring the inner UnsafeCell in the cell types to become pub(crate) though.
The main limiting factor in this implementation is the function which checks for duplicates (for safety).  I benchmarked a couple implementations, and settled on two, depending on the size of the array, the comments in the code should explain them.  I'm attaching the benchmark file I used and the output graph from my machine.  I'm using the sealed trait pattern to limit it to 255 simultaneous cells, as that limit provides the easiest ways to implement the uniqueness check.  You get a compile-time error if you try to use it with a longer array, which should look something like
the trait `LessThan256` is not implemented for `ValueOf<300_usize>`

which seems like a decent self-explanatory error to me.
Also submitting this one as a draft PR, without a test suite.
Benchmark results:

main.rs.txt

Just because we can do something doesn't mean we should. Can I back up a bit and ask about motivation?
Do you have a motivating example of where this is needed? When I looked at rw2 and rw3 I looked around my code and tried to think of examples where I'd need to access to more than one item. Actually I had cases where I needed to access 2 at the same time, e.g. swapping data around efficiently. I could just about imagine needing 3, but really I couldn't imagine needing 4.
The other thing is that I think you're creating ~256 types. Is this going to blow up time or space resource requirements for the crate?

Just because we can do something doesn't mean we should.

Totally fair, probably it's a good idea to table this one for now until someone actually asks for a >3 arity method.  Although, if someone does come along, I think we should consider this general solution over adding a rw4+.

The other thing is that I think you're creating ~256 types. Is this going to blow up time or space resource requirements for the crate?

The types are empty and only used for a where clause, I don't believe they will have any impact at runtime (even for debug).  It could have an impact on compile time (my testing says it's about a 100ms bump on my machine); but then, only if the user enabled the feature flag.

Okay, thanks. I think I'd need to see an example even if someone claimed they needed 4 or more, because there may be another way that is more efficient. For example if not all combinations of that set of values are needed, then just doing a series of rw2 operations might work out as less comparisons and faster. But certainly it would be worth coming back to this patch if there is a genuine requirement. I've written up issue #27 to see if anyone passes by who has such a requirement.

I've thought about that, and actually do have a use case where I could possibly use such a thing, however there are additional complexities.  If we're just talking about a &[QCell<T>],  it's pretty simple (different slots obviously are different cells), however at least in my use case you're rarely looking at that (since, in the cases you might encounter that, it would be better to use &QCell<[T]>).  Instead, I have something like &[Rc<QCell<T>>] and while I "know" they're unique, there's no way to guarantee that at compile time.
To provide a safe interface there, qcell would have to do some kind of uniqueness tracking (e.g. keeping a temporary HashSet around while iterating to make sure we never yield the same pointer twice).  HashSets aren't free (from runtime performance costs) and have the caveat that they're presently only available in std (no_std can depend directly on hashbrown, but then that's an additional dependency).

In my use case, I use HashMap with TypeId keys and I want to get multiple mutable references on several values by unique keys. I don't know if this is possible using &QCell<[T]> (without collecting values into a vector), but it could be with &[QCell<T>].

Right, and that requires the same type of runtime uniqueness checking.  It does seem like multiple unique borrows from a hashmap is a bit of an API hole that qcell might be positioned to solve, although I'm curious what precise benefit this hypothetical qcell API has over HashMap::iter_mut.

For uniqueness checking:

Obvious implementation is O(n^2), i.e. check each address against all following ones
Maybe could be implemented as a sort, then check for dups, i.e. O(N.logN), which could be in-place

There is no way to avoid this overhead safely unless some static guarantees can be made, which in general can't.  Regarding the issue, I think breaking things down, i.e. putting the cells at a finer level, and doing cell-access at a finer level may get around most cases. But really if there is some case that definitely needs code to handle 3+ simultaneous borrow, I'm still willing to be persuaded.


Load CMaps for non-english-fonts

Sorry to bother you. Thanks for your help.
There is another problem. https://huggingface.co/datasets/freddyaboulton/bucket/resolve/main/cmaps/ This URL can not be accessed in some places in China.
Can you change huggingface.co to something else?
: )

<EMAIL_ADDRESS>This URL meets same problem.
Maybe can change this URL to<EMAIL_ADDRESS>

Yikes


Add var.private_key for setting a specific SSH key for provisioners

Typhoon's default behavior is to rely on ssh-agent (via Terraform) to provide an appropriate key for SSH provisioner connections. This is ideal for an existing human user's keys but does not allow the caller to pass in a specific key. We want to generate a short-lived key for initial provisioning and therefore to pass in a specific key.
While we could probably find a usable hack to add the generated private key to the ssh-agent, passing the key string along is more straightforward.

Closing in favor of #15


Don't add PY3 in data path

Drop data.py support for the added PY3 folder in the old pickle packages.
This is related to issue #3305, which would not happen anymore anyway, because support is already dropped in compat.py,

Sorted the last wordnet tests, so that CI succeeds.

Thanks @ekaf great catch


Flutter JNI in Flutter background

W/FlutterJNI(20048): Tried to send a platform message to Flutter, but FlutterJNI was detached from native C++. Could not send. Channel: step_count. Response ID: 202
im using this plugin with flutter_foreground_task to gather the user step in background when app is terminated
but there is an error when app open again

Hi @mhmd435 I have not tested it in the background. Not sure it works. I'll look into it this week to see if I can make it work.
If you find a way and can contribute, it would be more than welcomed.
Are you trying in Android, IOS or both?


feat: set priority in monadic class instances

I added the high priority to these instances with monad transformers

MonadReaderOf ρ (ReaderT ρ m)
MonadWithReaderOf ρ (ReaderT ρ m)
MonadStateOf σ (StateT σ m)
MonadStateOf σ (EStateT ε σ m)
MonadExceptOf ε (ExceptT ε m)
MonadExceptOf ε (EStateT ε σ m)
MonadExceptOf ε (ExceptCpsT ε m)
MonadCallStackOf κ (CallStackT κ m)
MonadCycleOf κ (CycleT κ m)
MonadStore1Of k (β k) m

I added the low priority to this instance with a monad transformer

MonadExceptOf Unit (OptionT m).

I added the low priority to MonadExcept.instOrElse, because it needs to have lower priority than instOrElseOfAlternative (this change doesn't affect the synthesis order, because the order of the files in which the instances were declared was already 'correct').
In Prelude.lean, instead of setting the priority to high or low, I had to set it to 10000 or 100, which is the same.
The increased priority of MonadExceptOf ε (ExceptT ε m) affected a few files that had to be fixed.
Closes #4212

The test that broke had the number from the name generator offset by 8.
The line local infix:65 (priority := high) "*" => mul previously used to increment the name generator by 1560, and now this became 1568. I don't understand where these 8 extra names come from, but I equally don't understand why a simple infix declaration needs to use so ridiculously many names.

!bench

I'm concerned that there's an unexplained performance regression revealed by !bench. Can you determine if that is real or not, and if so try to localize the exact change that causes it?

Can you benchmark it again? There seems to be nothing in the rbmap.lean benchmarking test related to the changes in this PR.


Funnel on Item Vault with comparator output crashes server

Describe the Bug
When an Item Vault has a Comparator output and a Brass Funnel (have not tested Andesite Funnel) output, the server crashes on attempting to output new stack after current stack is retrieved by a player.
Reproduction Steps

Setup an item Vault and store some items in it.
Attach an output Brass Funnel on the Item Vault
Add a comparator output to the Item Vault
Pick up the item stack dropped by the funnel.

Expected Result
The funnel should drop another stack from the stored items in the Item Vault without crashing the server / game.
Screenshots and Videos

Crash Report or Log
https://pastebin.com/XBGAfgZv
Operating System
Linux (Endeavour OS)
Mod Version
0.4.1
Minecraft Version
1.18.2
Other Mods
appleskin-fabric-mc1.18.1-2.3.0.jar
architectury-4.6.77-fabric.jar
BetterF3-1.2.4-Fabric-1.18.2.jar
cammies-minecart-tweaks-1.4.jar
cloth-config-6.2.57-fabric.jar
continuity-1.1.0+1.18.2.jar
create-mc1.18.2_v0.4.1+637.jar
fabric-api-0.58.0+1.18.2.jar
fabric-language-kotlin-1.8.2+kotlin.1.7.10.jar
indium-1.0.2+mc1.18.2.jar
iris-mc1.18.2-1.2.2-build.32.jar
malilib-fabric-1.18.2-0.12.1.jar
mcw-lights-1.0.3fabric-mc1.18.2.jar
modmenu-3.1.0.jar
mousewheelie-1.10.2+mc1.18.2.jar
RoughlyEnoughItems-8.3.510.jar
sodium-extra-0.4.6+mc1.18.2-build.46.jar
sodium-fabric-mc1.18.2-0.4.1+replaymod.rev.23e83a1.jar
starlight-1.0.2+fabric.89b8d9f.jar
tweakeroo-fabric-1.18.2-0.13.1.jar
Xaeros_Minimap_22.12.0_Fabric_1.18.2.jar
Zoomify-1.6.0.jar
Additional Context
This behavior was tested on an Aternos server as well as a local world.

looks smiliar #269

looks smiliar #269

Does the fix address this specific bug as well?

fixed in dev


Automated Registration of Querydsl Repositories

The Spring GraphQL Querydsl integration makes it easy to create a DataFetcher from a QuerydslPredicateExecutor but it's still necessary to bind the resulting DataFetcher to the schema manually via RuntimeWiring.Builder.
We can automate this by matching fields in the schema that correspond to the generic type of the QuerydslPredicateExecutor. However, it should probbaly be an explicit mechanism, to avoid unintended side effects. Perhaps an @GraphQlRepository stereotype.

This is now in the main branch. If a repository is annotated with @GraphQlRepository it is automatically registered for top-level queries whose return type matches the repository domain type. We can now extend this further in #99 with similar support for Spring Data Repositories, including queries and mutations.


Update README

Need to import annotations from __future__ for using new shorthand | type checking
from __future__ import annotations

Details: https://stackoverflow.com/questions/72644693/new-union-shorthand-giving-unsupported-operand-types-for-str-and-type