Build failure on linux-ia64

Hello,
I recently upgraded from OpenSSL 3.0.2 to OpenSSL 3.0.3 and, for some reason, the build now fails on linux-ia64 (it also fails on other OSes, but this is an unrelated issue, waiting for https://github.com/openssl/openssl/pull/18241)
This is the error I got on linux-ia64 :
make[1]: Entering directory `/path/to/openssl'
gcc -fPIC -pthread -Wa,--noexecstack -Wall -O3 -DOPENSSL_USE_NODELETE -DOPENSSL_PIC -DOPENSSL_BUILDING_OPENSSL -DNDEBUG -DOPENSSL_TLS_SECURITY_LEVEL=0 -c -o crypto/chacha/libcrypto-lib-chacha-ia64.o crypto/chacha/chacha-ia64.s
crypto/chacha/chacha-ia64.s: Assembler messages:
crypto/chacha/chacha-ia64.s:21: Error: Unknown opcode `addp r27=4,r35'
crypto/chacha/chacha-ia64.s:24: Error: Unknown opcode `addp r35=0,r35'
crypto/chacha/chacha-ia64.s:25: Error: Unknown opcode `addp r36=0,r36'
make[1]: *** [crypto/chacha/libcrypto-lib-chacha-ia64.o] Error 1
make[1]: Leaving directory `/path/to/openssl'
make: *** [build_sw] Error 2

When I look at the chacha-ia64.s file, it is exactly the same between 3.0.2 and 3.0.3. I didn't change my generation method, just the OpenSSL sources.
I saw that at the beginning of the .s file :
#if defined(_HPUX_SOURCE)
# if !defined(_LP64)
#  define ADDP  addp4
# else
#  define ADDP  add
# endif
#else
# define ADDP   add
#endif

If I just replace the following ADDP by add, it works fine. I really don't understand what happens here. It seems the #define are, for some reason, ignored.
If someone has the slightest idea of why it fails, I'll be happy to test it.
Regards,
Robin

After a little more testing, thanks to an issue I opened, I found the culprit : https://github.com/openssl/openssl/pull/18136
So, it's my fault. It seems .s or .S is important.
There is in fact a difference between 3.0.2 and 3.0.3. In 3.0.2, the file is "chacha-ia64.S". In 3.0.3, it's "chacha-ia64.s". And it seems gcc (4.9.2) handles it differently...
If I rename the file to ".S" instead of ".s", everything works fine.

After some tests :

on linux-ia64, using .S or .sx instead of .s works
on hpux-ia64, using .S does not work (issue opened : https://github.com/openssl/openssl/issues/17781, crappy compiler), neither does .sx

It seems the issue is more complicated than hoped, simply renaming them to .s does work for cc (HPUX compiler) but not for gcc (and, to be honest, gcc is working correctly while HPUX compiler is crappy)

@t8m I agree that the previous fix should be reverted. The original behavior (file extension ".S" instead of ".s") was correct. HP-UX compiler is the only one being tedious. I asked for help on HP forum, to know whether this is due to an old compiler (mine is from 2008...) or if this is still the case with more recent version.
Note that on HP-UX with gcc, .S files are perfectly handled, the HP-UX default compiler is the only one to be broken.

I asked HP-UX community, but didn't got an answer yet.
I don't know if there is an update on the HP cc compiler which would fix the issue.
Thus, I can only think about 2 fixes :

either add an option during build
for HP-UX with cc only (not with gcc), use .s files instead of .S

Sadly, I'm not familiar enough with the OpenSSL architecture to help on this. If you can give me some pointers, I'll happily do it.
Note that there is still the workaround of building with asm support.


uable to use MAV_CMD_DO_SET_SERVO

I am trying to use MAV_CMD_DO_SET_SERVO to send PWM commands to AUX1 (channel 9). But unable to find the right protocol structure to do it.
Aim: I want to integrate activating and Deactivating magnet into a FCU code that I have built (set Magnet at AUX1 normally operated through RC)
msg_hi = self.vehicle.message_factory.command_long_encode(
0, 0,   # target_system, target_command
mavutil.mavlink.MAV_CMD_DO_SET_SERVO, # command
0,
8,    # servo number
2006, # servo position
0, 0, 0, 0, 0)
Here, I know here in this example I have tried using similar dronekit function not knowing how I can do using MAVSDK or MAVROS,
Pleaseee guide me if any alternative ways how I can do it?
I don't see my drone responding even after sending self.vehicle.send_mavlink(msg_hi), checked the AUX1 pin with Oscilloscope, but I don't see any fluctuating signal..
I am using Px4 and Odroid as my onboard computer..
%%
#!/usr/bin/python
import rospy as rp
import threading
from dronekit import connect, VehicleMode
from pymavlink import mavutil
from std_msgs.msg import Int8
from sensor_msgs.msg import Joy
from geometry_msgs.msg import PoseStamped, TwistStamped
class magdroneControlNode():
def __init__(self):
    rp.init_node("magdrone_deploy")

    # Connect to the Vehicle
    rp.loginfo('Connecting to Vehicle')
    self.vehicle = connect('/dev/ttyUSB0', wait_ready=True, baud=921600)

    # Set up Subscribers
    self.joy_sub = rp.Subscriber("/joy", Joy, self.joy_callback, queue_size=1)

    # Create thread for publishers
    self.rate = 30
    t = threading.Thread(target=self.send_commands)
    t.start()

    rp.spin()

def joy_callback(self, data):
    # Button Controls
    if data.buttons[0] == 1.0:
        rp.loginfo("Mission set to 1 - Deploying Sensor")
        self.mission_id = 1

    if data.buttons[1] == 1.0:
        rp.loginfo("Mission set to 2 - Retrieving Sensor and Exiting")
        self.mission_id = 2

'''
    Magnet Control
'''

def engage_magnet(self):
    msg_hi = self.vehicle.message_factory.command_long_encode(
            0, 0,   # target_system, target_command
            mavutil.mavlink.MAV_CMD_DO_SET_SERVO, # command
            0,
            8,    # servo number
            2006, # servo position
            0, 0, 0, 0, 0)

    msg_neut = self.vehicle.message_factory.command_long_encode(
            0, 0,   # target_system, target_command
            mavutil.mavlink.MAV_CMD_DO_SET_SERVO, # command
            0,
            8,    # servo number
            1500, # servo position
            0, 0, 0, 0, 0)

    # Send command
    self.vehicle.send_mavlink(msg_hi)
    rp.loginfo("Magnet Engaged")
    time.sleep(5)
    self.vehicle.send_mavlink(msg_neut)
    rp.loginfo("Magnet in Neutral")
    self.docked = True

def disengage_magnet(self):
    msg_low = self.vehicle.message_factory.command_long_encode(
            0, 0,   # target_system, target_command
            mavutil.mavlink.MAV_CMD_DO_SET_SERVO, # command
            0,
            8,    # servo number
            982,  # servo position
            0, 0, 0, 0, 0)

    msg_neut = self.vehicle.message_factory.command_long_encode(
            0, 0,   # target_system, target_command
            mavutil.mavlink.MAV_CMD_DO_SET_SERVO, # command
            0,
            8,    # servo number
            1500, # servo position
            0, 0, 0, 0, 0)

    # Send command
    self.vehicle.send_mavlink(msg_low)
    rp.loginfo("Magnet Disengaged")
    time.sleep(5)
    self.vehicle.send_mavlink(msg_neut)
    rp.loginfo("Magnet in Neutral")
    self.docked = False

'''
    Main Loop
'''

def send_commands(self):
    rp.loginfo("Accepting Commands")
    r = rp.Rate(self.rate)
    while not rp.is_shutdown():

        if (self.mission_id == 1) & (not self.magnet_engaged):
            self.magnet_engaged = True
            t = threading.Thread(target=self.engage_magnet)
            t.start()

        if (self.mission_id == 2) & (self.magnet_engaged):
            self.magnet_engaged = False
            t = threading.Thread(target=self.disengage_magnet)
            t.start()
        r.sleep()

Start Node
magdrone = magdroneControlNode()

I am not familiar with MAV_CMD_DO_SET_SERVO. Is it similar to MAVSDK's SetActuator action?

It looks like this is a PX4 question and not a MAVSDK question. Also your example code is DroneKit or pymavlink and not MAVSDK, so not sure what to do with it.


Add GetTLFCanonicalID to implement KeybaseServiceBase

To support public chats, need this info from KBFS.
r? @songgao
cc @strib @maxtaco @mmaxim

Would it help to rename the function by adding a suffix (DoNotCacheID)?

It's for public chats where we don't need encryption key, just tlfid and canonical tlf name.

@patrickxb: right, but what do you do with the TLF ID after you get it?  Ideally you wouldn't store it in any persistent chat data structures, where it could later be used by some other client as the primary identifier of the TLF.

It is sent to Gregor. Same as private chats. Nothing in service stores it but chat server needs tlfid

Ok sounds good, thanks.  I think the name or the avdl documentation around the function should explain what's allowed to happen with the ID.

PTAL

LGTM!


Support POSIX file globbing, and STDIN.

Could someone explain to me the logic behind having to specify a path with --suffixes and --exclude options instead of being able to stream in files, use STDIN, or use file globs?
I'm trying to write a phpmd plugin for gulp.js and this is an extreme hindrance for me.

@kid-icarus feel free to create a PR which implements this.

lol I don't use php anymore. good luck!

STDIN has been implemented in the upstream.    Support stdin #262
Specify '-' or 'php://stdin' as filename to use STDIN.
But it was broken now. (By Apply the filter on files as well #231)
Maybe I will submit a PR to fix it later.

@withwind8 that would be cool 👍

About STDIN, you can already use it this way:
cat src/MyFile.php | ./src/bin/phpmd php://stdin text foo/rules.xml

But we could add the same short-cut - than in PDepend.


Group tasks

Sample (which sample are you talking about)
project-gantt-chart
Suggestion (the more details, the better)
Group tasks by a column. Group can be collapsed and expanded

Hi @GerryHGraham,
I don't think that would be really possible, since I don't think it is possible group lines or compare them with others.
On the other hand, you could make up a column view and then simple use the grouping option in normal views.
But, that would be a completely different approach.
So I do not see this as a possibility straight away.


feat(updater): add a more fine grained public api

Codecov Report

Merging #138 (909c17a) into main (66335ef) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
- Coverage   94.90%   94.88%   -0.02%
==========================================
  Files          16       16
  Lines         902      900       -2
==========================================
- Hits          856      854       -2
  Misses         46       46

Impacted Files
Coverage Δ

src/ops2deb/fetcher.py
99.18% <100.00%> (+0.72%)
:arrow_up:

src/ops2deb/generator.py
93.85% <100.00%> (-0.06%)
:arrow_down:

src/ops2deb/updater.py
89.44% <100.00%> (-0.76%)
:arrow_down:

src/ops2deb/utils.py
100.00% <100.00%> (+3.70%)
:arrow_up:

Continue to review full report at Codecov.

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


Support OTA for LPC54018 IoT module including bootloader

This code comes from @PavelChromyNXP.  Also present are changes I've made to integrate the new code.  This code passes OTA regression tests and OTA end-to-end tests.

/bot run checks


sdcard.py missing

https://inkplate.readthedocs.io/en/latest/get-started.html#micropython shows copying lib files to the device. However, sdcard.py missing in the repo.

Hi @jwefers ,
would you please point me to where this file would be required? The SD Card code is in the 'machine' module in the .bin of the micropython firmware. You don't need to copy sdcard.py to use the sdcard.
Please give one of the examples a try, if I'm missing something please let me know and I'll happily write a follow-up and test any example which is required.
-Rob

step 3:
python3 pyboard.py --device /dev/ttyUSB0 -f cp mcp23017.py sdcard.py inkplate6.py image.py gfx.py gfx_standard_font_01.py

Ah, I see, the documentation is different on readthedocs and in this repository. That's my mistake and I apologize, going to correct it right now. Please use the guide provided in this repository.
My best guess is that this file was required in an older version.


Add archival fallback

I think this is not an ideal solution, but I think it is necessary to provide for the possibility of obtaining archived data.
I would be glad to have any suggestions to make a better solution.
export NEAR_ENV=mainnet
near tx-status wasmgit.near:JBb2DDe3i1CtBwESisLuhxXkWVZpCKYL4J1AdYwAQPsQ

Referenses:
https://stackoverflow.com/q/66839103/3123272
https://stackoverflow.com/q/67125552/3123272

@ilyar now we are working on a new version of the CLI that will probably have this functionality. But thank you for the contribution!


Concurrency issue when setting custom variables

Seems to be very rare, happened only a few times out of a few 10k events.
Writing this down so I don't forget it :).
Maybe fix this and do #69 at the same time?
java.util.ConcurrentModificationException
        at java.util.HashMap$HashIterator.nextEntry(HashMap.java:806)
        at java.util.HashMap$EntryIterator.next(HashMap.java:843)
        at java.util.HashMap$EntryIterator.next(HashMap.java:841)
        at org.json.JSONObject.<init>(JSONObject.java:128)
        at org.piwik.sdk.CustomVariables.toString(CustomVariables.java:90)
        at org.a.a.e.a(Tracker.java:16684)
        at eu.thedarken.sdm.tools.e.a.a(PiwikHelper.java:13159)
        at eu.thedarken.sdm.tools.piwik.PiwikHelper$1.run(PiwikHelper.java:305)
        at java.lang.Thread.run(Thread.java:841)

I would like to take care of it, if You don't mind

Sure :+1:, happy hacking.

Fixed by @dotsbb in #90


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

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

Questions ?
Please refer to the Backport tool documentation


Add video upload support

This resolves #640 and is based on #655, #672, #929, and #1414, with contributions from @Choko256, @fitnr, @jamesandres, @jamiewinspear, and @Maradonna90.

@LuisMayo As per our discussion in #1414, I've added a wait_for_async_finalize keyword-only argument for API.chunked_upload that defaults to True. It can also be passed to API.media_upload when setting chunked to True or uploading videos, and it'll be passed along to API.chunked_upload when API.media_upload calls it.

This branch/pull request should be complete now.
Any feedback and/or review from @Choko256, @fitnr, @jamesandres, @jamiewinspear, @LuisMayo, @Maradonna90, or anyone else would be appreciated.

The code looks a lot clean to me. Worked perfectly in a quick manual test.


Set the size parameter to a large value, to generate UUIDs that don't collide

I ran into issues with the instance for UUID generating colliding UUIDs.
I expected UUIDs to be globally unique, and have a hunch that others might expect the same.
So I decided to upload this patch in case it's relevant.
Note: I chose the number 10 000 out of thin air.
Changelog entry:

Reduce the likelihood of producing colliding UUIDs by hard-coding the size parameter to a big value.

Ugh. I though Word64 etc generators were uniform. It's better to change the generator than to increase size.


Add page to display public request details

Breakout from #704
Add a simple page which displays a request's status and (if available) planned deployment date.
No private info (names or addresses etc) should be shown on this page.

@stevejgordon Can you be more specific about the requirement?

WIP

@pmarshallandrew anything we can do to help you as you work on this cc @stevejgordon

@tonysurma - I know @pmarshallandrew is on vacation until the NDC London code-a-thon event. I also know he'd made some progress with this. I'm happy to jump in and resolve this issue if you think it will be needed before NDC London? If not @pmarshallandrew is joining me there so could finish off at that point?

Picking this back up today after coming back off holiday.


MASM code emitter

I'm struggling to figure out how the emitter is supposed to fit in with the other stuff in the midenc directory and with miden-diagnostics. This draft PR is my current attempt, but I have no idea if it's anywhere close to what we need, and what the exact division of responsibilities is supposed to be between the various files.
For instance, the Emitter trait doesn't seem to use file names at all, so presumably the file buffer we need to write to is passed to the emitter from the outside?

@bitwalker: I think this is more like it.
The ASTs in the miden-vm crate implement the formatter we need, so in essence the emitter just calls that formatter. We need MASMAst to distinguish between programs and modules, though, but I'm not sure where to put it. For the time being I've put it in the emitter file, but it can be moved around.
I wasn't sure exactly how you want to do it wrt. the output file names, but I figure it's something along the lines of what is in compiler/mod.rs (the code is commented out because it needs the artifacts variable, which is also commented out).


registration

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/86785
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 8f08c161d7aa03307e1170bc43b610888c93921c
git push origin v0.1.0


API call and redirect to cf-form with previous chat history

Firstly @felixnielsen and team, Kudos on your effort and the overall product. 🙌👍
I am finding it hard to add an API redirect to that point in chat where we left off from the cf-form, based on a click event.
To give you an example we are trying to add Facebook login to cf-form , something like this example/manual-start.html and after the user logins, we would like to redirect him to the chat, where user left off.
Please can you help us with this? Is it feasible with current conversation-form. If not can you suggest any hack regarding the same?
Appreciate your help!

hi @abhinav118 thanks! :) I understand you correctly that the user opens a different window, for the facebook flow or do you redirect to user from cf, login and back to cf?

yeah.. we could do it may hacking and redirecting to new html with previous questions and answers..
Any easier method is much appreciated.

@abhinav118 jsbin and facebook API does not play nicely together, but this example should get you all the way in a none iframe solution. Let me know how it goes.

@abhinav118 if above code-example did not work for you, then you are also able to setup the facebook callback url to contain a querystring that you can read and exclude the field in the form when the user returns, all in all it will require some custom code (client or server).


Making docs file smaller by around 40% using multi-stage builds

Description

I am proposing to use multi-stage builds for building the python part of Docs.
This can be done by installing all required OS and python dependencies and then to just copy over those files to the run stage. This would cause a nearly 35-40% reduction in size!
Checklist:

[X] I've made sure that tests/test_cookiecutter_generation.py is updated accordingly (especially if adding or updating a template option)
[X] I've updated the documentation or confirm that my change doesn't require any updates

Rationale

Smaller images are highly desirable because they would greatly reduce the time of re-deployment and deployment of containers and would also save a lot of time and money especially bandwidth costs.

While this is a great improvement in #2815, especially for production image, I'm less convinced of the benefit of the docs image. This size improvement makes the Dockerfile a bit more complicated after all, but the docs image doesn't need to be deployed anywhere AFAIK, one can build it and just keep its output (which are a bunch of static files).

@browniebroke Agreed. I did this just to be consistent with the Django Dockerfile.

@browniebroke I forgot to merge upstream changes with my repo and that caused all of that. I have merged all changes and raised another PR. https://github.com/pydanny/cookiecutter-django/pull/2835
Closing this.


LND Channel backup. Bug fixed. Motivated by Issue #958

What
Based on the implemented backup option with Dropbox, I wanted to switch the backup to the new variants. At the same time I also wanted to make a pull request to fix issue #958. I found a few more small bugs and fixed them.
Fixes #958
Fixes #981
So I got the following error messages because the file channel.backup exists with a size of 0 bytes and root access rights:
admin@raspibolt:~ $ ls -al ~/.lnd/data/chain/bitcoin/mainnet/channel.backup
-rw-r--r-- 1 root root 0 Feb  1 09:58 /home/admin/.lnd/data/chain/bitcoin/mainnet/channel.backup
......
Apr 10 17:29:04 raspibolt lnd[114726]: 2022-04-10 17:29:04.547 [ERR] LTND: Shutting down because error in main method: unable to start server: unable to refresh backup file: unable to extract on disk encrypted SCB: payload size too small, must be at least 24 bytes
Apr 10 17:29:04 raspibolt lnd[114726]: 2022-04-10 17:29:04.548 [INF] RPCS: Stopping RPC Server
......
Apr 10 17:29:26 raspibolt systemd[1]: Stopped LND Lightning Network Daemon.
Apr 10 17:29:26 raspibolt systemd[1]: lnd.service: Consumed 14.296s CPU time.
Why
The imprecise path information is not easy to handle for users without Commandline knowledge.
How
I changed the text ;-)
Scope

[ ] significant change to core configuration
[ ] independent bonus guide
[x] simple bug fix

Fixes #958
Test & maintenance
I implemented the installation according to the corrected guide
Animated GIF (optional)
Add some snazz if you feel like it :)
A fulfilling job

I did some research on the 0Byte channel.backup file.
Based on my recording, I see that I tested the backup to Dropbox at the beginning of February (the 0 byte file was created on February 1st).
For the test, I changed the channel.backup file with "touch /data/lnd-backup/channel.backup" to trigger a backup manually.
It's possible that I accidentally created the ~/.lnd/data/chain/bitcoin/mainnet/channel.backup file as well.
So I think the problem was caused by myself.
I suggest I delete my addendum and make another pull request.
Is this procedure correct?
Sorry for the circumstances.

Hi @mktech-gh!
Rather than closing this PR and opening a new one without your addendum, you could simply remove it from your branch. The PR will then show the updated version.
Also, it seems you added a change for the charge-lnd bonus guide in this PR. It should belong to a separate new PR. The way to do it would be to create a new branch in your repository (e.g. a branch named "charge-lnd-bug-fix"), make your changes in that new branch and then create a PR from it.
Basically, rather than working from your master branch (mktech-gh/master), you could create a new branch for each PR you'd like to make. Let me know if that makes sense or not.

Hello @VajraOfIndra
Thank you for your help 🙏🏻. That's great for me. That way I can constantly improve.
Using the branch now makes sense to me.🤔and i will use it now
I have now deleted my changes regarding the 0 byte file in channel-backup.md.
How can I undo the changes in "charge-lnd bonus guide"? I would like to re-do these changes in a new branch?

How can I undo the changes in "charge-lnd bonus guide"? I would like to re-do these changes in a new branch?

There might a much better way (like cancelling commits), but the way I do it usually is either removing the changes I made if I remember exactly what they were or else, I delete the entire content of the file and paste the content of the file from the RaspiBolt master branch (the one being used for the official website), like that I'm sure the file is exactly as it should be and all my changes are gone.
But again this is hacky, I'm sure there is a more porfessional way to do it playing with commits :)

OMG: 3 attempted commits ;-). But your hack worked :-)
@VajraOfIndra: My thanks to you.


site wise penalty

Pull Request
Description
The following changes in the code makes sure that we are tracking site wise penalty calculations, as each site/state/generation type has different penalty bands
How Has This Been Tested?
The code has been ran locally to see if it works

[x] Yes

If your changes affect data processing, have you plotted any changes? i.e. have you done a quick sanity check?

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have checked my code and corrected any misspellings

a few comments, but im principle looks right

Looks like you need to update a test too


Fix TinyMCE-Related Console Warnings

Fixes #3424
Changes proposed in this PR:

Other than preventing specific console warnings from appearing, these changes should not affect the behaviour of the app.

Removed forced_root_block: '' from app/javascript/src/superAdmin/notifications/edit.js

The [forced_root_block](https://www.tiny.cloud/docs/tinymce/5/content-filtering/#forced_root_block) option will no-longer accept the false value or an empty string value in TinyMCE 6.0 https://www.tiny.cloud/docs/tinymce/5/6.0-upcoming-changes/
Removing this code resolves the console warning, and it doesn't seem to introduce any other changes (documented here: https://github.com/DMPRoadmap/roadmap/issues/3424#issuecomment-2166424160)

Removed "Resize any TinyMCE editors" code  from app/javascript/src/utils/conditionalFields.js

This code block was attempting to set the height using tinymceEditor.settings.autoresize_min_height. However, tinymceEditor.settings. was evaluating to undefined. In addition, TinyMCE replaced autoresize_min_height with min_height when it upgraded from v4 to v5 (https://www.tiny.cloud/blog/how-to-migrate-from-tinymce-4-to-tinymce-5/).
This code block didn't appear to be affecting anything. Rather, it was being set by the value of min_height:, within the the defaultOptions object of app/javascript/src/utils/tinymce.js (documented here: https://github.com/DMPRoadmap/roadmap/issues/3424#issuecomment-2166552337).

Closing this for the moment because it is preventing a Dependabot Security PR from being generated.


Moderation panel!

(this should have been added earlier...)
Also misc changes to stuff; see the commit log.

Me with MeowyMod rn:

MM with bridges:
On Tue, Mar 21, 2023, 4:48 PM MikeDev101 @.***> wrote:

Me with MeowyMod rn:
[image: image]
https://user-images.githubusercontent.com/12957745/226748829-29d03e50-7a41-42a4-bf13-e5133b3e4dfa.png
—
Reply to this email directly, view it on GitHub
https://github.com/meower-media-co/Meower-Svelte/pull/116#issuecomment-1478629453,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQHW4P5SBU3QIB76JEEHQRDW5IOZBANCNFSM6AAAAAAWC2IMUI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

resub to thread, clicked the wrong email button

That did not fix it... your not checking if they can get IP's

That did not fix it... your not checking if they can get IP's

i thought i did (i don't have a l1 account to test though)

works

My review still stands. Please make it look at least a bit better on mobile.

@CST1229 can this be merged?

@CST1229 can this be merged?

i guess


Send controller and other trackers position to the server

Need to send controller and other trackers position to the server to use them as refrence for other body parts (like elbows) or references instead of HMD.

per https://github.com/ValveSoftware/openvr/issues/1473 it seems like the API provided to drivers does not give role information.
Would it make more sense to:

Make SlimeVR-Server an openvr application/(dashboard) overlay so that it can acquire the information itself?
Make an additional "feeder" openvr application/(dashboard) overlay that is separate from SlimeVR-Server?

Former makes sense to me from an integration standpoint (i.e. editing slimevr settings from your steamvr dashboard), latter might be easier to quickly prototype.

 I don't have a plan right now to rework the whole slimevr to make it openvr app, both because this will be a hell to develop, and that SlimeVR is not planned as only open-vr.
There is another option: send data without role information, guestimate it by id, and let users rebind it if it's wrong.

True that. Given that you don't want to make it openvr only, it makes sense to me to make a feeder application, and maybe there could be additional communication between feeder and server in the future for changing settings.
Is there anything that might break if sending hmd position back isn't precisely synced with the driver?

@kitlith you don't need to send hmd back

@kitlith you don't need to send hmd back

sorry, i meant positions in general. i was kinda figuring in my head that we could send hmd in the same place as controllers and trackers, which would also be a good sanity check that  works well before needing to actually use the new controller/tracker data.

I've got a start on a feeder application here: https://github.com/kitlith/SlimeVR-Feeder-App
still working on it of course, but i think I've covered enough of the input system so that we can actually start pushing/pulling actual pose data.

after add tracker I found that we can‘t get the contoller data.
I did the following.

run the openvr opengl_example (It could run that I can use controller to disable the background)

close the opengle_example and open the slimevr-server

re run the opengl_example I found that the controller can't work.(I can't use controller to disable the background)

I found that when we load the driver we can't get the controller data.          T-T

Controller and other trackers data is sent via the feeder app: https://github.com/SlimeVR/SlimeVR-Feeder-App
This issue is no longer relevant, driver only sends HMD data.


Nested enum import - typescript

In order to describe an external enum mapping we need to mention:
The required enum (placed in server/proto/service_pb.d.ts):
export namespace EmailOnlyLoginRequest {
   export enum LoginOrigin {
      AGENT_REGISTRATION = 0,
    }
}

gql-schema:
enum LoginOrigin {
    AGENT_REGISTRATION
}

codegen.yml - there is no way to import nested enum in namespace
config:
  enumValues:
      LoginOrigin: server/proto/service_pb#EmailOnlyLoginRequest.LoginOrigin

will generate in graphql.ts:
import { EmailOnlyLoginRequest as LoginOrigin } from 'server/proto/service_pb';
which is obviously wrong, it refer to the namespace rather than the enum itself.

@kamilkisiela we can apply the same rules we have in mappers, right?

Hi @bennyKY
I've implemented it in this PR: https://github.com/dotansimha/graphql-code-generator/pull/3651
The changes are available in this alpha version: 1.13.1-alpha-612fffa4.26.
Use it the same way you did before, but it should give you now something like that:
// This is the import of the namespace
import { EmailOnlyLoginRequest } from 'server/proto/service_pb';
// This is internal aliasing, to point to the type from the namespace
import LoginOrigin = EmailOnlyLoginRequest.LoginOrigin;

// ... some more stuff ...

// Re-export the enum with the name that appears on the schema
export { LoginOrigin };

Available in v1.13.1 🎉

I have the following config:
enumValues:
  Session: "../services/db/interface#DBSessionEnum"

which now leads to the incorrect:
import { DBSessionEnum } from "../services/db/interface";
import Session = DBSessionEnum;

Earlier it was generating:
import { DBSessionEnum as Session } from "../services/db/interface"; which was working.
Can you kindly look into this?

We changed the behaviour to do import aliasing.
So importing DBSessionEnum as Session or importing DBSessionEnum and then aliasing it with import Session = DBSessionEnum is the same.
Why it doesn't work for you? @WaqasAliAbbasi

We changed the behaviour to do import aliasing.
So importing DBSessionEnum as Session or importing DBSessionEnum and then aliasing it with import Session = DBSessionEnum is the same.
Why it doesn't work for you? @WaqasAliAbbasi

Ah I just realised why. I am trying to map it to a type.
error TS2693: 'DBSessionEnum' only refers to a type, but is being used as a value here.
error TS2702: 'DBSessionEnum' only refers to a type, but is being used as a namespace here.

DBSessionEnum is actually a union type: type DBSessionEnum = "SUMMER" | "WINTER" | "MARCH";
Unfortunately I cannot control DBSessionEnum as it is automatically generated by Prisma.
I only wanted Session to be a union type so I used mapping as a workaround. enumsAsTypes: true generates all enums as union types which isn't ideal in my case. Any way I can fix this? Thanks so much for taking out the time to look at this.

Unfortunately, it looks like the import X = Y;syntax is not supported by @babel/plugin-transform-typescript, which leads to this producing broken code for us...

Unfortunately, it looks like the import X = Y;syntax is not supported by @babel/plugin-transform-typescript, which leads to this producing broken code for us...

You are right, @schmod . Can you please share more about the use-case? which plugins are you using and how? It will help me to find a better solution for developers who are using babel for TS.

You are right, @schmod . Can you please share more about the use-case? which plugins are you using and how? It will help me to find a better solution for developers who are using babel for TS.

Our use-case is fairly straightforward.  We build our app using babel instead of tsc because we've still got a bit of flowtype code in our repository, and it's generally easiest for us to just build everything with Babel.
The patch that I'm currently using to work around this issue looks like this:
diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index f3badce..7cc15e9 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -1152,11 +1152,11 @@ class BaseTypesVisitor extends BaseVisitor {
         return `import${useTypeImports ? ' type' : ''} { ${identifier} } from '${source}';`;
     }
     handleEnumValueMapper(typeIdentifier, importIdentifier, sourceIdentifier, sourceFile) {
-        const importStatement = this._buildTypeImport(importIdentifier || sourceIdentifier, sourceFile);
         if (importIdentifier !== sourceIdentifier || sourceIdentifier !== typeIdentifier) {
-            return [importStatement, `import ${typeIdentifier} = ${sourceIdentifier};`];
+            return [this._buildTypeImport(`${sourceIdentifier} as ${typeIdentifier}`, sourceFile)];
+        } else {
+            return [this._buildTypeImport(importIdentifier || sourceIdentifier, sourceFile)];
         }
-        return [importStatement];
     }
     getEnumsImports() {
         return flatMap(Object.keys(this.config.enumValues), enumName => {

My patch probably isn't a good general-purpose fix, but I think we could adapt it in a way that allows us to avoid the import X = Y syntax unless we actually need it (for namespaces and such).

Unfortunately, it looks like this is still an issue in 2.x
It'd be great if we could avoid these namespace imports wherever they're avoidable.

Also having this issue :/


AlarmManager Set to SetExact

If this issue is for a new feature, please skip the "Bug" section and fill out the "Feature Request" section at the bottom.
Bug
Plugin Version:
Platform (e.g., Android v21):
Device/Hardware:
Expected behavior
Should fire an alarm at given time.
Actual behavior
Device keeps shifting notification time.
Steps to reproduce the behavior

Feature Request
Feature description:

@jetro157 need to include more details, specifically steps to reproduce, including sample code you're using if possible. Thanks.


Fix linking to pages docs - prepend /content

This PR updates broken links to the pages documentation.

My misunderstanding of how docs get deployed out. We can close!


Enhancement: Better display for pagination

Right now every single button is displayed for pagination.  We should keep that to a manageable number of buttons.
https://intranet.fws.gov/region4/demo/news/

We'll need to override the internal template.

Closed via #38.


⚠ Bofrost.fr has degraded performance

In 817d13e, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1811 ms

Resolved: Bofrost.fr performance has improved in 615cd08.


idea: if you are spinning for a while, always end on a vertical alignment

when tapping quickly, you'd probably want to just rotate 90, but if you're holding it in, chances are you'd like to stop spinning on a vertical alignment
thoughts @secretrobotron

I've actually implemented this in https://github.com/secretrobotron/games/pull/118 so I'll close. I barely even notice the change, which I think is a good sign.


[SPARK-33478][SQL] Allow overwrite a path that is also being read under dynamic partition overwrite

What changes were proposed in this pull request?
Insert overwrite same table can overwrite a path that is also being read under dynamic partition overwrite
Why are the changes needed?
Currently, Insert overwrite cannot overwrite a path that is also being read under dynamic partition overwrite.
In the class DataSourceAnalysis, DDLUtils.verifyNotReadPath is called to determine whether it can be overwrite, but in the InsertIntoHadoopFsRelationCommand.dynamicPartitionOverwrite method, we need to consider the scenario where staticPartitions.size equals partitionColumns.length, which should also be allowed to overwrite path this is also being read.
Consider the following statement:
CREATE TABLE insertTable(i int, part1 int, part2 int) USING PARQUET PARTITIONED BY (part1, part2);
INSERT OVERWRITE TABLE insertTable PARTITION(part1=1, part2=1) SELECT i + 1 FROM insertTable;
Does this PR introduce any user-facing change?
no
How was this patch tested?
Add test in InsertSuite, start line:307

Can one of the admins verify this patch?

Can one of the admins verify this patch?

cc @viirya

We have the same issue with the following case:
INSERT OVERWRITE TABLE insertTable PARTITION(part=1) SELECT col FROM insertTable where part=2

Similar issue: https://issues.apache.org/jira/browse/SPARK-32838


'File not found' after repo list

This is OK with 0.3, but I am experiencing this issue with the last master release and previous ones. I am using lighttpd and gitlist to follow only a bare repo. Today, I discovered that the problem is that index.php has disappeared from the internal path that the web server is managing, so a workaround is to include manually this in the navigator address bar.
Some details. This is the behavior with 0.3 when all running right:
2013-05-17 20:33:54: (response.c.300) -- splitting Request-URI
2013-05-17 20:33:54: (response.c.301) Request-URI  :  /gitlist/tuns.git/
2013-05-17 20:33:54: (response.c.302) URI-scheme   :  http
2013-05-17 20:33:54: (response.c.303) URI-authority:  server
2013-05-17 20:33:54: (response.c.304) URI-path     :  /gitlist/tuns.git/
2013-05-17 20:33:54: (response.c.305) URI-query    :
2013-05-17 20:33:54: (response.c.300) -- splitting Request-URI
2013-05-17 20:33:54: (response.c.301) Request-URI  :  /gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.302) URI-scheme   :  http
2013-05-17 20:33:54: (response.c.303) URI-authority:  server
2013-05-17 20:33:54: (response.c.304) URI-path     :  /gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.305) URI-query    :
2013-05-17 20:33:54: (response.c.349) -- sanatising URI
2013-05-17 20:33:54: (response.c.350) URI-path     :  /gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (mod_access.c.135) -- mod_access_uri_handler called
2013-05-17 20:33:54: (response.c.470) -- before doc_root
2013-05-17 20:33:54: (response.c.471) Doc-Root     : /var/www/lighttpd
2013-05-17 20:33:54: (response.c.472) Rel-Path     : /gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.473) Path         :
2013-05-17 20:33:54: (response.c.521) -- after doc_root
2013-05-17 20:33:54: (response.c.522) Doc-Root     : /var/www/lighttpd
2013-05-17 20:33:54: (response.c.523) Rel-Path     : /gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.524) Path         : /var/www/lighttpd/gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.541) -- logical -> physical
2013-05-17 20:33:54: (response.c.542) Doc-Root     : /var/www/lighttpd
2013-05-17 20:33:54: (response.c.543) Rel-Path     : /gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.544) Path         : /var/www/lighttpd/gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.561) -- handling physical path
2013-05-17 20:33:54: (response.c.562) Path         : /var/www/lighttpd/gitlist/index.php/tuns.git/
2013-05-17 20:33:54: (response.c.711) -- after pathinfo check
2013-05-17 20:33:54: (response.c.712) Path         : /var/www/lighttpd/gitlist/index.php
2013-05-17 20:33:54: (response.c.713) URI          : /gitlist/index.php
2013-05-17 20:33:54: (response.c.714) Pathinfo     : /tuns.git/
2013-05-17 20:33:54: (response.c.719) -- handling subrequest
2013-05-17 20:33:54: (response.c.720) Path         : /var/www/lighttpd/gitlist/index.php
2013-05-17 20:33:54: (mod_access.c.135) -- mod_access_uri_handler called
2013-05-17 20:33:54: (mod_fastcgi.c.3676) handling it in mod_fastcgi

This is when failing with the last master release (and previous also):
2013-05-17 20:34:07: (response.c.300) -- splitting Request-URI
2013-05-17 20:34:07: (response.c.301) Request-URI  :  /gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.302) URI-scheme   :  http
2013-05-17 20:34:07: (response.c.303) URI-authority:  server
2013-05-17 20:34:07: (response.c.304) URI-path     :  /gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.305) URI-query    :
2013-05-17 20:34:07: (response.c.349) -- sanatising URI
2013-05-17 20:34:07: (response.c.350) URI-path     :  /gitlist_build/tuns.git/
2013-05-17 20:34:07: (mod_access.c.135) -- mod_access_uri_handler called
2013-05-17 20:34:07: (response.c.470) -- before doc_root
2013-05-17 20:34:07: (response.c.471) Doc-Root     : /var/www/lighttpd
2013-05-17 20:34:07: (response.c.472) Rel-Path     : /gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.473) Path         :
2013-05-17 20:34:07: (response.c.521) -- after doc_root
2013-05-17 20:34:07: (response.c.522) Doc-Root     : /var/www/lighttpd
2013-05-17 20:34:07: (response.c.523) Rel-Path     : /gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.524) Path         : /var/www/lighttpd/gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.541) -- logical -> physical
2013-05-17 20:34:07: (response.c.542) Doc-Root     : /var/www/lighttpd
2013-05-17 20:34:07: (response.c.543) Rel-Path     : /gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.544) Path         : /var/www/lighttpd/gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.561) -- handling physical path
2013-05-17 20:34:07: (response.c.562) Path         : /var/www/lighttpd/gitlist_build/tuns.git/
2013-05-17 20:34:07: (response.c.618) -- file not found
2013-05-17 20:34:07: (response.c.619) Path         : /var/www/lighttpd/gitlist_build/tuns.git/

As you can see, index.php has dropped from the path, so the 'manual' workaround is not to navigate to:
http://server/gitlist_build/tuns.git/
but to:
http://server/gitlist_build/index.php/tuns.git/

This configuration works fine for me: Put the following lines in /etc/apache2/sites-enabled/gitlist.conf
<Directory "/var/www/html/ementario">
    Options FollowSymlinks
    Options -Indexes
    AllowOverride All
    Require all granted
    RewriteEngine On
    RewriteCond %{REQUEST_FILENAME} !-f
    RewriteCond %{REQUEST_FILENAME} !-d
    RewriteRule ^(.*)$ index.php/$1 [L]
</Directory>

Then, with root:
sudo a2enmod rewrite
sudo service apache2 restart


AutoGPTQ not working with HuggingFace accelerate (multi GPU)

If I run the following command:
accelerate launch -m lm_eval --model hf --model_args "pretrained=TheBloke/Llama-2-7B-Chat-GPTQ,gptq=True,load_in_4bit=True" --tasks "arc_challenge" --num_fewshot 25 --batch_size auto

I get the following error:
ValueError: You can't train a model that has been loaded with `device_map='auto'` in any distributed mode. Please rerun your script specifying `--num_processes=1` or by launching with `python {{myscript.py}}`.

If I add device_map_option=balanced to --model_args, I get the same issue.
But if I try an unquantized model with multi GPU e.g. model_args="pretrained=meta-llama/Llama-2-7b-chat-hf", it works perfectly.
Is 4-bit AutoGPTQ compatible with multi GPU (accelerate)?
I installed AutoGPTQ from source, as follows
pip install<EMAIL_ADDRESS>
instead of pip install -e ".[gptq]", because I encountered errors with the latter.
GPTQ with single GPU works fine.

What are the contents of your accelerate config?

compute_environment: LOCAL_MACHINE
debug: false
distributed_type: MULTI_GPU
downcast_bf16: 'no'
gpu_ids: 0,1
machine_rank: 0
main_training_function: main
mixed_precision: 'no'
num_machines: 1
num_processes: 2
rdzv_backend: static
same_network: true
tpu_env: []
tpu_use_cluster: false
tpu_use_sudo: false
use_cpu: false

Note that we've changed the GPTQ argument name to autogptq=True to match the library, and that load_in_4bit=True should not be set if using GPTQ. Does making these changes solve your issue?
We will ideally be reexamining our quantization integrations as part of #1225 to make error messaging around these sorts of conflicts more intuitive.

I'm using v0.4.0, so I believe the argument is still gptq=True? Using autogptq=True threw an error anyways.
I tried removing load_in_4bit=True but the error still persists.

I see--I'd recommend trying with the current main branch installed from github and seeing if there is still an error. We'll put out a v0.4.1 on PyPI very soon.

@JeevanBhoot to narrow this down, could you try inserting device_map={"":torch.cuda.current_device()} into the from_pretrained() call?
It sounds as though, despite you not passing a device_map or using parallelize=True in the first snippet (this is our intended behavior + usecase), Huggingface or AutoGPTQ is inferring some device map nevertheless, which we don't want. (Again, load_in_4bit=True should also be removed here.)

To which from_pretrained() calls should I add the device_map argument? I added it to all from_pretrained() calls in huggingface.py but the error persisted.
I have removed load_in_4bit=True and I am now working from main.


Line chart is always going under arearange

I am adding several rangeseries and linechart with below code
for(var i = 0;i<=8;i++)
{

   niftyseriesup[i] = chart[1].addAreaRangeSeries().setHighStrokeStyle(greenline).setLowStrokeStyle(greenline).setHighFillStyle(greencolors[i]);

 niftyseriesdown[i] = chart[1].addAreaRangeSeries().setHighStrokeStyle(redline).setHighFillStyle(redcolors[i]);

 if(i == 8)
 {

 setTimeout(function(){
   niftylineY = chart[1].addAxisY(true);
   niftylinechart = chart[1].addLineSeries({ dataPattern: DataPatterns.horizontalProgressive ,yAxis: niftylineY}).setMaxPointCount(8000);
  }, 200);

 }

}

I am adding the line chart at last , Still Its appear under the rangeseries chart sometimes.
Please see image below.
https://i.stack.imgur.com/ncL1H.png

I'm unable to reproduce this issue. I would need a minimal code example that reproduces this issue.

https://xtremealgo.com/test.php ,  also as time goes on .. it goes under other range series too.
Btw are you snekw from stackoverflow ?

Thank you for that, I will try to troubleshoot it and see if there is something that will resolve the issue.
Yes, I'm the same Snekw from stackoverflow.

This seems to be an issue with the LightningChart JS library. I will continue to investigate the root cause for this.

Hi , any update on this..

I'm pretty sure I have an idea where the issue is coming from. I can't promise any timeline for when the fix is released but I hope it would be in 2.1 release.

Hi , any update on this ?

Sorry, I don't have any updates for this.

Hi , any update on this ? seems like 2.1 released , was the issue fixed ?

Sadly this issue hasn't been fixed in 2.1 release and I don't have any estimate when it will be fixed.

Hi , any update on this ? to set z-index , I have used the version 2.2.1 , still there is this bug
https://xtremealgo.com/test.php


Include e2e tests

The e2e tests should include invocations to the real radicle CLI binary and target a real seed node.
At least in Linux, this should be doable, by installing the radicle CLI with apt and starting a seed node using docker.

we should probably wait till after Heartwood to get this done

https://excalidraw.com/#room=d66b3446151822511b6c,nf0QDCatwsr-b2kroGvR9Q


Added information about Neovim’s built-in ls + nvim-lspconfig

Cool, thanks. I was hoping now that this was released things would work - any issues apparent?


fixing extensions to include a dot

getFileExtension already back to its old behavior. This fix is no longer needed


Update mockito-core to 3.12.0

Updates org.mockito:mockito-core from 3.10.0 to 3.12.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.mockito", artifactId = "mockito-core" } ]

labels: test-library-update, semver-minor

Superseded by #194.


[CRIMAPP-459] Remove NI number for pse applications

Description of change
NI number is currently showing as None for pse applications, even when a NI number was submitted for a initial application
NI number should not be displayed for pse applications and this PR removes it
In review, there is no reference to NI number so review does not require any updates
Link to relevant ticket
CRIMAPP-459
Notes for reviewer
Screenshots of changes (if applicable)
Before changes:

After changes:

How to manually test the feature
Note: requires a PSE application to have been submitted
Log into apply and navigate to the completed applications tab and select a PSE application
Verify that the national insurance row in the client details section is not displayed

Thanks for the comments @timpeat, i've updated the pr


You must provide an AppConfig within your application!

I get a warning in console:
"You must provide an AppConfig within your application!"
How to provide AppConfig?
Thanks!
Erki

Solution:

create empty configuration file "formio-config.ts" with content:
export const AppConfig = {

};

add import to "app.module.ts":
import {AppConfig} from '../formio-config';

add provider (providers section) to "app.module.ts":
{provide: FormioAppConfig, useValue: AppConfig}

import FormioAppConfig in  "app.module.ts":
import {FormioModule, FormioAppConfig} from 'angular-formio';

@krixerx This workaround breaks the form builder

I have the same problem. Does anyone have a solution? When I add the empty config, the form builder no longer works.

I have the same problem. Does anyone have a solution? When I add the empty config, the form builder no longer works.

Create a valid config

okay, but I don't use the formio backend service. Do you use the formio service? Or what valid data have you entered?

i also havethis issue using angular formio with my own custom backend

@Marc-Hoch
`
import { FormioAppConfig } from "@formio/angular"
export const AppConfig: FormioAppConfig = {
appUrl:'',
apiUrl: 'api',
}
`

`import { FormioAppConfig } from "@formio/angular"
export const AppConfig: FormioAppConfig = {
appUrl:'',
apiUrl: 'api',
}`
@Marc-Hoch


Migrates WeDeploy from Alpha to Beta

Are we adding wedeploy.json just to show it has one? it should work without that too.

Makes sense @zenorocha , thanks!


Use executor allocator in spin

Is there a reason for asking here for the RCL default allocators if we have set an allocator for the executor?
Thanks

however, I have to look why the CI jobs are not passing!

sorry for the delay, good catch - there was no reason for using again the default allocator.

@pablogs9 I signed signed-off your commit with (type as many '^' as there are commits in the PR)
git rebase --signoff HEAD^^
git push --force-with-lease origin fix/allocator_spin

@pablogs9 LGTM, you can merge with PR.


xxxhdpi not available for action icons

Xxxhdpi is currently available for launcher icons, but not for action icons (just xxhdpi). It would be an awesome feature.

+1

Just fixed it for myself. Try this: http://joerogers.github.io/AndroidAssetStudio/

According to this xxxhdpi assets are only needed for launcher icons, which are already provided by this tool.
#44 adds this functionality.

This started coming up when Android Studio 1.1 started flagging for not having XXXHDPI assets for items other than the launcher. The Nexus 6 along with several other devices are running above the XXHDPI density. This android blog talks about the need to consider adding them. I wish I could support 5.0+ only so that I could ship exclusively with vector drawables instead.

Yeah we were talking about the same in the #44 PR.
Everybody wishes we could support only the latest :smile:

BTW my change fixes all icons including action bar, generic, notification and support for 9 patches at xxxhdpi. #44 only did action bar. I needed a more robust patch.

Why don't you send a PR?

Done. I was letting a few friends "validate" everything seemed to be working the last two days.


fix: upgrade rseqc container

Changes
Upgrade the rseqc docker to use the newer base container which explicitly sets the PYTHONPATH.
Issues
depends on https://github.com/CCBR/Dockers/pull/40
fixes #171
PR Checklist
(~Strikethrough~ any points that are not applicable.)

[x] This comment contains a description of changes with justifications, with any relevant issues linked.
~[ ] Update docs if there are any API changes.~
[x] Update CHANGELOG.md with a short description of any user-facing changes and reference the PR number. Guidelines: https://keepachangelog.com/en/1.1.0/

@samarth8392 can you test that this fixes the error?

just ran a quick test using renee test dataset /gpfs/gsfs10/users/CCBR_Pipeliner/Pipelines/RENEE/.v2.6.1/.tests/ and all jobs finished successfully.
@kelly-sovacool can you please cut the new release v.2.6.2 and update it on biowulf? Thanks


Programm without libUSB?

Hey,
I have an important question.
Does my programm run without libUSB installed on the computer, or MUST it be installed.
Because I want to code an USB-Locker for any OS, and I don't want, that everyone has to install LibUSB to run my application.
Dechasa
PS: If LibUSB must be installed, is there any other lib I can use which don't use LibUSB?

You can define your own backend without libusb.

Closing old issues.


Incorrectly indents python inner classes

See this issue for details or I can also paste it there.
Basically, it looks like this plugin is reformatting python code blocks and doing it wrong at least for this case. The code has nested classes and it incorrectly unindented the inner class.
I have just mdformat and mdformat-mkdocs installed in a pipx environment. mdformat itself seems to be leaving the code block alone but the addition of mdformat-mkdocs is what made it reformat the block. Uninjecting the plugin doesn't stop it from happening. I had to uninstall and reinstall mdformat to stop it from formatting the code.
Is the reformatting behavior part of this plugin or is it picking up and using a formatter from somewhere?
I have blacken-docs in my pre-commit and it looks like black is fine with that code block. I'm just saying how other formatters behave. I'm using only commandline for testing this bug to eliminate variables. Involving pre-commit was what made me file the bug incorrectly in the mdformat project.

Thanks for posting such a detailed issue. You’ve caught me right before leave, so I’ll try to resolve before tomorrow!

Wow, that's a fast response!
I didn't think to look in the source for answers and now I'm realizing and remembering reading that this project uses markdown-it. I went to look at their docs and then I'm thinking maybe it's faster for you to figure out if the problem is there or here or just me.
I'm adding mkdocs to a project that has a few other devs and mdformat feels like a great tool to enforce a level of consistency.

Thanks for looking into this. I only managed to trace it to the calculate function :smile:

I think the fix will be to keep a reference to the code block indentation (last indent) when within a code block, but I ran out of time to finish the patch release.
In the interim, you could unindent the code blocks so that they aren’t part of the list and add a comment to revisit. I may not have a chance to finish the patch until next week

That's fine. I'll continue to other work for now. Thanks!
I guess it's just been reformatting the code like regular content before. Just to test, I indented the input code block to 2 spaces increments and it got indented to 4 spaces in the output.
The solution probably requires skipping processing of the code block content lines until the end sequence is found rather than keeping track of the internal indents. Or maybe apply the indent adjustment from the beginning sequence line to the whole block. Just some thoughts...

Should be fixed in v1.1.1 (https://pypi.org/project/mdformat_mkdocs/#history)!
Let me know if you find any edge cases or have any issues


Displaying chants/sequences without sources

From @jacobdgm

Also - we may want to check in with Debra about whether to display chants/sequences without sources. In OC, there was a boolean visible property (sources could definitely have it, and possibly chants/sequences could too), which we got rid of, rolling everything into our published. Since all our visibility-related logic is based on a chant/sequence's source, we may not want to default to displaying chants to everyone on the grounds that they don't belong to an unpublished source.

Originally posted by @jacobdgm in https://github.com/DDMAL/CantusDB/pull/990#discussion_r1296349020

@annamorphism - guessing you'll say we should check with Debra, but perhaps you already have a strong sense of what we should do here. Thoughts?
My gut instinct: if a chant/sequence doesn't have a source, it's not in publishable form, and we should not display it to unauthenticated users.

I don't fully understand the issue--do you have an example of a source-less chant?

Hang on, I found one--https://cantusdatabase.org/sequence/623263 leads to a 500.
Here it is on OldCantus: https://cantus.uwaterloo.ca/sequence/623263 .
It's kind of bizarre--it has a RISM ID, but there's not actually a source associated with that ID, so you can't tell what else is in there, and it doesn't show up in concordances (or even in the admin list of sources on OldCantus).  Weirder still, I can't figure out what source it's supposed to be--"SN-SBfm" should correspond to a country+library but I don't know which.
On the other hand https://cantus.uwaterloo.ca/sequence/631313 (also broken in NewCantus) has a siglum that belongs to something I can find (Bibliotheque de l'Arsenal 608, a 14th c. missal from Poissy.)
Anyway, so:
-We should not display chants/sequences without sources, because they are displeasing and broken
-Some sort of way to turn these up (for me and Debra et al) would be helpful, to figure out how many of these there are. I suspect most of them are sequences that got imported into the Bower database from some file with blanks in required fields, but maybe there are some /chant objects as well.
-Is it possible to "attach"errant chants and sequences onto a source? Normally you can't make chants without starting from a source, but clearly it has occasionally happened and we need a way to fix it when it has.

update: there are also a few chant objects that do this. e.g. https://cantus.uwaterloo.ca/chant/688442 (broken on NewCantus: https://cantusdatabase.org/chant/688442). I have no idea what this particular chant is supposed to be part of, or how it even got made....

Some sort of way to turn these up (for me and Debra et al) would be helpful, to figure out how many of these there are.

It should not be possible to create sourceless chants/sequences, so I expect no new ones to be added.  It must have been created in NewCantus. #996

just reiterating from our meeting:
-sourceless chants should not be viewable to non-logged-in users.
-Somebody will eventually clean these up by inputting the sources for the relevant chants, which can be done via the Admin interface (admittedly in a somewhat tedious fashion, since each individual chant has to be updated)

I have updated the code so that chants and sequences without a source are not displayed on the site other than in the admin area. We can close this issue or leave it open until we know all of the objects Jacob gathered have been dealt with or assigned a source.


command line arg doesn't override config file

Describe the bug
Command line arg doesn't override settings in config file
To Reproduce
Put root = "./" in config file, and try run static-web-server -d web/, it shows ./ instead of web/
Expected behavior
It shows web/ when I passed -d web/ to it.
Logs/Screenshots
If applicable, add logs or screenshots to help explain your problem.
Environment and Specs

static-web-server: 2.10.0
OS: Arch Linux x86_64, 5.15.60-1-lts
Docker: null
Client: 104.0.5112.79

Additional context

This functionality is intended by design.
You can have a have a look at https://sws.joseluisq.net/configuration/config-file/#precendence
So if you want to have CLI precedence then remove your option from the config file and just go with the CLI one.


Update model.py

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #3009
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Feature/Issue validation/testing:
Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

for custom predictor v1/v2 are both supported by default, you do not need to specifically set the version. The self.protocol field on model server is used for transformer to call out to predictor not by predictor itself


Fixed variant decoding Segmentation Fault

When encoding a variant there exists a check on the depth of recursion, but similar check is missing for decoding.
This can lead to Seg faults on synthetic data. Since RPC's also use this it leads to remote crashes.
Issue also exists in 3.4.2.

Thanks!

Cherry-picked for 3.5.

Cherry-picked for 3.4.3.

Cherry-picked for 3.3.5.


publish zeroconf style scripts to download and install gtoolkit

https://github.com/feenkcom/gtoolkit/tree/master/scripts/localbuild

Does publish also mean update the gtoolkit.com/install page?

on mac we have the following zeroconf style script:
curl https://raw.githubusercontent.com/feenkcom/gtoolkit/master/scripts/localbuild/mac.sh | bash


Add files via upload Grammar School, Tišnov

Institution/School Name
Grammar School, Tišnov
Instiution/School Website
{Website URL}
Email Users
Please place an x between the square brackets if yes

[x] Students
[ ] Academic/Teaching Staff
[ ] Other/Support Staff
[ ] Alumni

Education Levels
Please place an x between the square brackets if yes

[ ] Post-graduate research
[ ] Graduate School
[ ] College (University) or Undergraduate School or equivalent
[ ] Community College or equivalent
[x] High School or equivalent
[ ] Elementary School or equivalent

@adelavalach First stage of verification passed, official domain is gym-tisnov.cz as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.


A vegetation type is resulting in false positives

As below on specific vegetation type is resulting in false positives.
many of these are small than buildings and can be generalized out of existence.
It is however assumed this is occurring as data with this back ground class has not been supplied in the training image set. The training set should be extent to include imagery with this vegetation class to provide further context to the model

Hypothesis proven correct. The model was not aware of this type of vegetation. By passing images to training that included vegetation of this model all of the above false positives were removed


Broken Links in Related Resources

Description
In certain legacy courses, there are related resources linked from the video pages. The links are embedded in the videos' metadata and currently written as raw links. Links to other resources, rather than absolute URLs, are not working. For example, the link https://ocw.mit.edu/courses/res-18-005-highlights-of-calculus-spring-2010/resources/big-picture-derivatives/resources/mitres18_05s10_big_picture_derivatives/ should actually be https://ocw.mit.edu/courses/resources/mitres18_05s10_big_picture_derivatives/ (i.e., the URL is relative to the referring resource page but shouldn't be).
There are three components to this issue:

[ ] Update the video partial (https://github.com/mitodl/ocw-hugo-themes/blob/main/base-theme/layouts/partials/video.html) to handle resource link shortcodes.
[ ] Add a Playwright test to verify that related resource links are handled correctly, both for absolute URLs and resource link shortcodes.
[ ] Fix the broken links in the database by updating the relevant metadata with resource_link shortcodes to the relevant UUIDs and republishing the affected courses.

This plan makes sense to me, but we should prioritize a quick fix https://github.com/mitodl/hq/issues/1091 before addressing the widespread issue.


打开特殊文件没有反应

打开没有后缀名的文件(且没有绑定此文件打开方式的应用)时没有反应

I cannot repro your problem.
On my computer, Windows will prompt you selecting the open method for your file.

@xiaofengsoft First, minimize all applications using Windows + M keyboard shortcut and then open FIles UWP again and make it maximized. Then the dialog will be shown.
首先，使用Windows + M键盘快捷键最小化所有应用程序，然后再次打开FIles UWP并将其最大化。 然后将显示对话框。
首先，使用Windows + M鍵盤快捷方式最小化所有應用程序，然後再次打開FIles UWP並將其最大化。 然後將顯示對話框。

@Jaiganeshkumaran Is this still an issue? If so, can you describe it to me?

Thanks again for your feedback.
Since we didn't hear from you on this topic for 7+ days, we'll assume everything is worked out and close this feedback item. If you had more feedback to provide, please re-open this and help the owners by describing the current status and any steps taken to troubleshoot this.


When testing locally, you might get timeouts

During testing this week, I have noticed that sometimes running make helm-install will end up timing out while waiting on cert-manager to come up.
❯ make helm-install
kind delete cluster && sleep 10
Deleting cluster "kind" ...
kind create cluster --config ./cli/tests/kind-1.23.yaml
Creating cluster "kind" ...
 ✓ Ensuring node image (kindest/node:v1.23.6) 🖼
 ✓ Preparing nodes 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹
 ✓ Installing CNI 🔌
 ✓ Installing StorageClass 💾
Set kubectl context to "kind-kind"
You can now use your cluster with:

kubectl cluster-info --context kind-kind

Have a nice day! 👋
kubectl wait --for=condition=Ready pods --all --all-namespaces --timeout=300s
pod/etcd-kind-control-plane condition met
pod/kube-apiserver-kind-control-plane condition met
pod/kube-controller-manager-kind-control-plane condition met
pod/kube-scheduler-kind-control-plane condition met
kubectl apply -f https://github.com/cert-manager/cert-manager/releases/download/v1.6.1/cert-manager.yaml
customresourcedefinition.apiextensions.k8s.io/certificaterequests.cert-manager.io created
customresourcedefinition.apiextensions.k8s.io/certificates.cert-manager.io created
customresourcedefinition.apiextensions.k8s.io/challenges.acme.cert-manager.io created
customresourcedefinition.apiextensions.k8s.io/clusterissuers.cert-manager.io created
customresourcedefinition.apiextensions.k8s.io/issuers.cert-manager.io created
customresourcedefinition.apiextensions.k8s.io/orders.acme.cert-manager.io created
namespace/cert-manager created
serviceaccount/cert-manager-cainjector created
serviceaccount/cert-manager created
serviceaccount/cert-manager-webhook created
clusterrole.rbac.authorization.k8s.io/cert-manager-cainjector created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-issuers created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-clusterissuers created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-certificates created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-orders created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-challenges created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-ingress-shim created
clusterrole.rbac.authorization.k8s.io/cert-manager-view created
clusterrole.rbac.authorization.k8s.io/cert-manager-edit created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-approve:cert-manager-io created
clusterrole.rbac.authorization.k8s.io/cert-manager-controller-certificatesigningrequests created
clusterrole.rbac.authorization.k8s.io/cert-manager-webhook:subjectaccessreviews created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-cainjector created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-issuers created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-clusterissuers created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-certificates created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-orders created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-challenges created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-ingress-shim created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-approve:cert-manager-io created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-controller-certificatesigningrequests created
clusterrolebinding.rbac.authorization.k8s.io/cert-manager-webhook:subjectaccessreviews created
role.rbac.authorization.k8s.io/cert-manager-cainjector:leaderelection created
role.rbac.authorization.k8s.io/cert-manager:leaderelection created
role.rbac.authorization.k8s.io/cert-manager-webhook:dynamic-serving created
rolebinding.rbac.authorization.k8s.io/cert-manager-cainjector:leaderelection created
rolebinding.rbac.authorization.k8s.io/cert-manager:leaderelection created
rolebinding.rbac.authorization.k8s.io/cert-manager-webhook:dynamic-serving created
service/cert-manager created
service/cert-manager-webhook created
deployment.apps/cert-manager-cainjector created
deployment.apps/cert-manager created
deployment.apps/cert-manager-webhook created
mutatingwebhookconfiguration.admissionregistration.k8s.io/cert-manager-webhook created
validatingwebhookconfiguration.admissionregistration.k8s.io/cert-manager-webhook created
# Give enough time for a cluster to register new Pods
sleep 7
# Wait for pods to be up and running
kubectl wait --for=condition=ready pod -l app.kubernetes.io/instance=cert-manager -n cert-manager
pod/cert-manager-cainjector-8c7796555-g8g7j condition met
pod/cert-manager-webhook-55f45df998-q9jv7 condition met
error: timed out waiting for the condition on pods/cert-manager-cdc85d4c4-sn5bq
make: *** [Makefile:36: cert-manager] Error 1

We might want to look at increasing the sleep time.

I don't think it's the sleep command that is at fault here. The sleep command is only to ensure pods are registered but the issue here is that kubectl wait --for=condition=ready pod -l app.kubernetes.io/instance=cert-manager -n cert-manager is timing out. Which means the pod couldn't be seen in Ready state for more than 30s. We might want to specify and increase that particular timeout.
Basically instead of:
kubectl wait --for=condition=ready pod -l app.kubernetes.io/instance=cert-manager -n cert-manager

We could do sth like:
kubectl wait --timeout=120s --for=condition=ready pod -l app.kubernetes.io/instance=cert-manager -n cert-manager


class Attribute docs and object creation

Problem or new feature
https://docs.raku.org/type/Attribute has confusing parts.
Suggestions
:local appears 17 times as an argument to methods.
It seems undocumented. It seems superfuous.  Why no syntax error?  What is going on???
The below references the method required section:
The syntax example says --> Any:D  the next line says if required return 1 -- Why not True?!  But  okay.
Then continues if not-- return Mu -- not an Any and not a :D??
Then continues if  the attrib was `is required( 'Some Str' ) return the text.  Which I think is wrong on two levels:

Given the state of this section, this part of the docs is not  read because the method is expected to return a Bool.
I can write class Fu { has $.attr is required(''); }  breaking the bool-ishness of Attribute.required.

The last line of the section has the wrong method called.

In general, unused named arguments do not issue an error. But you are right, it needs to be very much improved. We'll try to do it when we possibly can. Thanks for the report.

Additionally, it does not include how to actually build an Attribute, or a complete example.

It would seem that the Introspection page should reference the Attribute Class page; and vice versa.
Additionally, the ^attributes  method should be documented, which would allow meaning to be attached to its args.  The provided caret-methods are generally difficult to look up.


Expose RENDERMODE_CONTINUOUSLY and RENDERMODE_WHEN_DIRTY

To save power, MapLibre only renders the map scene when it becomes dirty (an Android event causes the map to change). This behavior is desired in production. However, when testing performance and particularly trying to automatically detect stuttering, it is better to use continuous rendering.
GLSurfaceView already has the ability to switch between RENDERMODE_CONTINUOUSLY and RENDERMODE_WHEN_DIRTY. This PR simply exposes these options to MapLibre client apps.

@louwers CI is failing. Can you tell me what log I should look at to understand the failure?

Hmm the logs should show up if you click through but
Test spec output for run 'arn:aws:devicefarm:us-west-2:373521797162:run:20687d72-0e46-403e-8f03-0941850665bc/5146eb99-3855-474d-aa1c-8f132baf221e' not found

AWS Device Farm logs can also be found via this guide: https://github.com/maplibre/maplibre-native/wiki/AWS-Device-Farm-Logs
Did you try running the (Android) C++ Unit Tests locally? They are in test/android.

@louwers All tests are passing on my phone. The runner hangs at the end but this is true on main as well. It shows this log without returning:
2024-09-05 14:36:42.267 11218-11255 GTest                   org.maplibre.cpp_test_runner         I  Test program ended with 955 tests run.
2024-09-05 14:36:42.267 11218-11255 mbgl                    org.maplibre.cpp_test_runner         I  {Thread-139}[General]: TestRunner finished with status: '0'
2024-09-05 14:36:42.268 11218-11247 mbgl                    org.maplibre.cpp_test_runner         I  {Thread-2}[General]: TestRunner done

@louwers You're right that exposing it as-is to client apps without a proper enum is wrong. I fixed this. It should work in both Vulkan and GL. It won't work for TextureView which seems to be a deprecated feature.


LAB_09a-Implement_Web_Apps.md

Current Translation:
Buat slot penyebaran pentahapan
Tukar slot pentahapan
Suggested Translation:
Membuat slot penyebaran pentahapan
Menukar slot pentahapan

Hello contributor,
Thank you for your contribution! We are processing your suggestions and we will update you shortly.
Kind regards,
Microsoft Worldwide Learning Team

Run the bugfix tool and waiting for result. ICMS QUERY


Adaptive Common Neighbour Analysis

I tried to update our interface method to use pyscal rather than ovito and noticed the results for the same structure differ. An example structure is available in:
https://github.com/jan-janssen/ovito-vs-pyscal
and the comparison can be accessed via mybinder:
https://mybinder.org/v2/gh/jan-janssen/ovito-vs-pyscal/master?filepath=ovito-vs-pyscal.ipynb
I wrote a short wrapper around the pyscal code to return the output in the ovito format and these are the results of the comparison:
Ovito:
{'CommonNeighborAnalysis.counts.BCC': 10, 'CommonNeighborAnalysis.counts.FCC': 5387, 'CommonNeighborAnalysis.counts.HCP': 0, 'CommonNeighborAnalysis.counts.ICO': 0, 'CommonNeighborAnalysis.counts.OTHER': 2603}

pyscal:
{'CommonNeighborAnalysis.counts.OTHER': 3701,
 'CommonNeighborAnalysis.counts.FCC': 4296,
 'CommonNeighborAnalysis.counts.HCP': 0,
 'CommonNeighborAnalysis.counts.BCC': 3,
 'CommonNeighborAnalysis.counts.ICO': 0}

For the interface method it is important to identify the regions of solid structure, so in this case primarily FCC vs. OTHER.

Thanks for pointing it out @jan-janssen ! I will check the issue.

Still the results for the melting point calculation give more or less similar results (926K):
https://github.com/jan-janssen/melting-pyscal/blob/master/plot.ipynb
and (927K):
https://github.com/pyiron/pyiron_meltingpoint/blob/master/examples/fccAl/plot.nbconvert.ipynb

Thanks! Thats good to know. So far I have been unable to see why this difference occurs.

I noticed an other issue:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-5-2b18f17b0857> in <module>
----> 1 analyse_cna_adaptive(struct, mode="total")

<ipython-input-2-ed0435d41a51> in analyse_cna_adaptive(atoms, mode)
     20     sys = pc.System()
     21     sys.read_inputfile(atoms, format="ase")
---> 22     cna = sys.calculate_cna(cutoff=None)
     23
     24     if mode == "total":

/u/system/SLES12/soft/pyiron/dev/anaconda3/lib/python3.7/site-packages/pyscal/core.py in calculate_cna(self, cutoff, calculate_neighbors)
   1267         if cutoff is None:
   1268             if calculate_neighbors:
-> 1269                 self.find_neighbors(method="number", nmax=14, assign_neighbor=False)
   1270             cna = self.calculate_acna()
   1271             return cna

/u/system/SLES12/soft/pyiron/dev/anaconda3/lib/python3.7/site-packages/pyscal/core.py in find_neighbors(self, method, cutoff, threshold, filter, voroexp, padding, nlimit, cells, nmax, assign_neighbor)
    558             finished = self.get_all_neighbors_bynumber(threshold, nmax, assign_neighbor)
    559             if not finished:
--> 560                 raise RuntimeError("Could not find enough neighbors - try increasing threshold")
    561
    562         elif method == 'voronoi':

RuntimeError: Could not find enough neighbors - try increasing threshold

I uploaded the POSCAR file to the repository above https://github.com/jan-janssen/ovito-vs-pyscal/blob/master/POSCAR2

This one is easier! It is one of the configurations with 'holes'. Pyscal is just complaining that there are not enough 'reasonable' neighbors for some atoms, which is quite true. This can be overcome by raising the threshold values. I will do it.

That would be great, it is really just a single calculation where the calculation failed, so I am quite optimistic that we can use pyscal, we just have to figure out what causes the difference in counts then I am happy to update the official version.

Yeah, I am taking a look at the difference in counts again tomorrow. Hopefully, this will be sorted out soon.

I noticed an other issue:
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
<ipython-input-5-2b18f17b0857> in <module>
----> 1 analyse_cna_adaptive(struct, mode="total")

<ipython-input-2-ed0435d41a51> in analyse_cna_adaptive(atoms, mode)
     20     sys = pc.System()
     21     sys.read_inputfile(atoms, format="ase")
---> 22     cna = sys.calculate_cna(cutoff=None)
     23
     24     if mode == "total":

/u/system/SLES12/soft/pyiron/dev/anaconda3/lib/python3.7/site-packages/pyscal/core.py in calculate_cna(self, cutoff, calculate_neighbors)
   1267         if cutoff is None:
   1268             if calculate_neighbors:
-> 1269                 self.find_neighbors(method="number", nmax=14, assign_neighbor=False)
   1270             cna = self.calculate_acna()
   1271             return cna

/u/system/SLES12/soft/pyiron/dev/anaconda3/lib/python3.7/site-packages/pyscal/core.py in find_neighbors(self, method, cutoff, threshold, filter, voroexp, padding, nlimit, cells, nmax, assign_neighbor)
    558             finished = self.get_all_neighbors_bynumber(threshold, nmax, assign_neighbor)
    559             if not finished:
--> 560                 raise RuntimeError("Could not find enough neighbors - try increasing threshold")
    561
    562         elif method == 'voronoi':

RuntimeError: Could not find enough neighbors - try increasing threshold

I uploaded the POSCAR file to the repository above https://github.com/jan-janssen/ovito-vs-pyscal/blob/master/POSCAR2

This is now fixed

pyscal cna results are now slightly better:
{'other': 3011, 'fcc': 4984, 'hcp': 0, 'bcc': 5, 'ico': 0}

New results-
{'others': 2606, 'fcc': 5384, 'hcp': 0, 'bcc': 10, 'ico': 0}


Fides Experience form fixes

Closes PROD-2008
Description Of Changes
Backend defaults weren't reflected in the Frontend. These fields weren't being passed at all if not set, which caused the backend to apply its defaults. Since backend defaults are set to "true" on both of these fields, it was a mismatched experience.
Setting these to default to "true" on the Frontend now to match BE defaults. User can turn them off without issue during initial save.
Code Changes

Set defaults to true for the 2 fields causing problems

Steps to Confirm

log in to fides plus
click on the experiences
click on create new experience
complete the following fields

name
experience type (banner and modal)
add a privacy notice (data sales and sharing)
note that "allow users to dismiss" and "auto detect language" are now on by default.

click save

user is now at the main experience screen showing all experiences

click the newly created experience to edit it

observe that the "allow users to dismiss" and "auto detect language" are in the correct state. Repeating the steps above and opting to turn off those 2 should reflect them as off here as well.
Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
[x] Issue Requirements are Met
[x] Update CHANGELOG.md

UAT ✅
https://github.com/user-attachments/assets/43bf001c-6e2a-4178-8c02-ba2374282382


Improving the landing Page Carousel

The Carousel doesn't work with the arrow keys and the hover keys are almost non-existent and very basic.

@MinavKaria
Do Assign me this Issue, as I want to work on both components and modify them for standards.
Also tag the appropriate tags of level and events.

@MinavKaria I have worked on this, Can I perform my PR?


Simplify replacement calculations

Instead of individually declaring colour codes, why not just replace the & symbol with the section sign (§) in general? More readable and saves a little bit of time.

I had the same idea originally. But I than realize that players could use the & symbol to declare things other than colour codes. Example 'M&M server'

I hadn’t thought of that, never mind :)


Add Rails > 6.0 compatibility

Also see https://github.com/globalize/globalize/pull/755.

I've released an alpha gem
Once Rails 6.1 is out and it works we can revisit the gem release. 😄


Mark aether language as world necessity

Putting this change up for code review as it touches the LevelLoader. Will be merged into webpack branch next.
This registers the Aether languages necessary (as determined by the session(s)) as world necessities, because Aether needs those before it can be initialized (in Spell.coffee#190).
I suspect that before (on brunch) this was sort of a race condition dependent on load order of various files, which no longer happens to work out with Webpack.
Please use your domain knowledge to see if there's any race conditions or the like that might be created by adding this to worldNecessities.

Coverage remained the same at 61.969% when pulling 0c6f41ececc88cd3dbc88276494b1f0f6620fb52 on load-aether-lang into dd9d675eedd651118b21221e06cc08bb9ade2099 on webpack.

Coverage remained the same at 62.036% when pulling 589a4c8aa693146d913da898db669d54c3a59d96 on load-aether-lang into 66bf22129a950ea800bd5686dad8f2eb0ef16da4 on webpack.

Seems like this would work fine. If we want to be extra cautious, we can test across verifier, ladder simulator, game dev, web dev, level editor, home, and classroom, with Python and also with Lua where possible, with and without resources already loaded.

Coverage remained the same at 62.036% when pulling 98bf6f7c56c82c1a824e95a2acf2028e8b4b0c00 on load-aether-lang into a236fa2cd680871dee1864ecf4daa71c731c1091 on webpack.


timestamp column

Hi, can you tell me how did you get the timestamp column in the sample dataset

The tool used for collect CSI is ESP32-CSI-Tool.
The serial_append_time.py python utility enables you to append the timestamp column.


Notebooks: cells do not have aria labels set

Turn on your screen reader
Navigate between cells
Notice how when you navigate on a non-code cell Screen Reader does not read anything

Inspecting the DOM it seems like the aria-lable is not set on non-code cells.
List supports this out of the box via accessibility providers so I think this should be easy to fix
reported by @webczat

actually sometimes it does read things. for example in case of markdown cells sometimes it probably reads the whole output, sometimes it says that it is markdown editor... hmm not sure if I can trigger different behaviors in some specific way or not

What should it read, the markdown source as text?

currently unsure, not much experience with notebooks in any form. Not sure what identifies the cell for a sighted person? is that only the ordering and type of cell?

Yes, that's it. There's no label or anything useful like that. Checking other notebook systems, Jupyter doesn't read anything when the cell is selected. When focus goes into a cell editor, it starts reading the full document content (starting from the top instead of the selected cell). Google Colab reads the names/keyboard shortcuts of all commands in the cell toolbar when a cell is selected, followed by the cell contents. So I don't think there's a good precedent and we'll have to make it up as we go.

Well, I did have an idea, I mean, sometimes vscode would read just the type of editor like markdown, I don't remember how it reads that exactly, probably something like "markdown editor", and it seems more appropriate than something like reading it top to bottom when I focus a closed markdown cell.
As for jupyter, it is not really accessible in general, unless it changed.

I like the idea of reading the type. For example, "focus on a markdown cell", "focus on a code cell" and then users would at least know what would happen when pressing "Enter" (editing markdown, or focus into the code cell editor) or "Ctrl+Enter" (run the code if it's a code cell).

it could even go and read the beginning of the contents, but it is the least important piece of info there

We now have aria support for markup cells.


Only publish the .ts files

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

Project coverage is 89.19%. Comparing base (fcbe39b) to head (ea9f36b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   89.19%   89.19%
=======================================
  Files           8        8
  Lines        2175     2175
  Branches      380      380
=======================================
  Hits         1940     1940
  Misses        234      234
  Partials        1        1

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


[CELEBORN-1446] Enable chunk prefetch when initialize CelebornInputStream

What changes were proposed in this pull request?
https://github.com/apache/celeborn/pull/2348 avoids fetching first chunk in the constructor
of CelebornInputStreamImpl, but in some cases, i.e. coalescing 3000 partitions into one in Spark,
it can be beneficial to do so for performance. This PR adds back prefetching with knobs default to false.
Why are the changes needed?
Does this PR introduce any user-facing change?
Yes, two configs are added.
How was this patch tested?
Extended MemorySkewJoinSuite and ReusedExchangeSuite, and manual test.

@RexXiong comments addressed, please take a look again

Merge to main(v0.6.0) and branch-0.5(v0.5.1)


Audit for landscape mode compatibility

We need to go through and make sure everything looks good in landscape. Let's add things to this issue that need fixing as we go.

[ ] Properly offset for safe areas in SingleSubredditViewController in landscape

Hijacking this issue, iPad bottom drawer is wrong size in split mode

I fixed my issue, did your PR close the rest of this?

Sure did.


native_entry.cc: 236: error: Failed to resolve native function 'ParagraphBuilder_constructor' in 'ParagraphBuilder._constructor@17065589'

Steps to Reproduce
With the latest Dart SDK and Flutter engine,
flutter clean && flutter run -v --release --no-preview-dart-2 --local-engine=android_release

Logs
[   +1 ms] To repeat this help message, press "h". To quit, press "q".
[ +247 ms] I/FlutterActivityDelegate(18279): onResume setting current activity to this
[ +201 ms] E/DartVM  (18279): ../../third_party/dart/runtime/vm/native_entry.cc: 236: error: Failed to resolve native function 'ParagraphBuilder_constructor' in 'ParagraphBuilder._constructor@17065589'
[        ] E/DartVM  (18279):
[+189076 ms] /usr/local/google/home/alexmarkov/Android/Sdk/platform-tools/adb -s ZY2245H6M5 shell am force-stop io.flutter.demo.gallery
[+1038 ms] Application finished.

Flutter gallery runs fine in Dart 2 mode (default). Also, it runs fine with currently rolled Dart SDK and Flutter engine.

Bisected to the commit https://github.com/flutter/engine/commit/d42b5b73b5057ef6a5ad5c61ccc6a137ffdf0fbf in the Flutter engine.
/cc @jason-simmons @rmacnak-google


NetworkTransform with clientAuthority set true cannot be moved by server when no client has authority

When you have a game object with a NetworkTransform with clientAuthority set to true, it can only be moved by clients with authority. When no client has authority (i.e. the server has authority), it cannot be moved by the server.
Is this intentional behaviour?
I would expect the server to be able to move the object when no client has authority. This was the behaviour in previous versions of Mirror.
I believe the issue is in NetworkTransformBase where the serverBuffer is used when clientAuthority is set and current client has no authority on this line.
Demonstration Project
I've set up a simple project with the smallest amount of code. There's only empty player objects and a cube that moves vertically when the server has authority.
Start up a build as the client and the unity editor as host. The cube is being moved automatically by the server (when connection to client is null).
When clientAuthority on the Network Transform is checked then the cube stops because the server is not able to move the cube (that's the bug).
Uncheck clientAuthority and the cube moves.
https://user-images.githubusercontent.com/1922279/172066366-6160c7be-ac2b-482f-afac-6551ccd1ad4c.mp4
Desktop (please complete the following information):

OS: Windows
Build target: Windows
Unity version: 2021.1.25f
Mirror branch: release version 66

Many thanks in advance!

FYI I don't need tis answering anymore but leaving for posterity.

Hello
I updated Mirror in a project and having that problem now too. I also figured out that moving objects around on the multiple clients has problems. It seams that they have different last states that replace the object when the authority is lost. That leads to the problem that objects are not on the same position on the different devices.
@brainwipe  May i ask how you solved the problem?
Would it make sense to turn on and off the clientAuthority on all clients+server at runtime when needed?
Mirror Version 66.0.9
Unity 2021.3.1f1
Tested devices: Mac Book, Windows 10, Oculus Quest 2


Quick Accent languages selection

Description of the new feature / enhancement
Change the language setting to use checkboxes (toggle button) for each character set/language so that we can pick up multiple languages without having to select "All available".
Also, for convenience, include a preview of the characters included for each languages
Scenario when this would be used?
This would be useful for people who frequently work with multiple targeted languages without having to list the complete set of characters
Supporting information
PowerToys settings -> Quick Accent -> Characters

/dup https://github.com/microsoft/PowerToys/issues/21261


Move basic Python configuration to the root of the repo.

After way too much trying to figure stuff out I realized: why not just put the pyproject.toml right at the root of the repo? This eliminates all the #subdirectory=... when doing pip installs and also makes setuptools-scm work right again. As a bonus, lots of tools now work without any additional configuration and we can just run them as-is.
This moves the bare minimum files needed to the repo root:

pyproject.toml
MANIFEST.in

and updates everything else to make that work right.

haha whoops

we’re back


503 - Backend Error

What steps will reproduce the problem?
1. GET
https://www.googleapis.com/freebase/v1/mqlread?query={%22id%22:[],%22*%22:[{}],%
22name~=%22:%22sampleQuery%22,%22type%22:[],%22limit%22:10}
2.
3.

What is the expected output? What do you see instead?
some array of types that have name like 'sampleQuery'...

i got, json valid..

{
 "error": {
  "errors": [
   {
    "domain": "global",
    "reason": "backendError",
    "message": "Backend Error"
   }
  ],
  "code": 503,
  "message": "Backend Error"
 }
}

What version of the product are you using? On what operating system?
api v1, on Chrome latest

Please provide any additional information below.
just trying to execute query in browser...  -.-

regards, Tino

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 18 Jun 2013 at 5:37

+1


should not override DATASTORE related environment variables if already defined

motivation
I want to use custom emulator instance in Docker environment. (e.g. running datastore emulator under Docker container).
Original implement cannot do so even run container with given correct environment variables such as DATASTORE_EMULATOR_HOST=db:8432.
resolution
does not override if target environment variable already defined.

@bmclean could you review this PR?

thanks for your quick response!


Create unit/integration tests

This will help us avoid breaking things in the future

The manager is still changing too much for this to be feasible currently


Patch historical klines to handle case where results exactly equal limit

Hi Sam,
It was reported that historical klines will crash when the number of data points returned matches the limit exactly.
Kline loop was only exiting when the data return was < limit so this makes sense.
It caused the loop to run once more, retrieve an empty kline dataset, then fall over when trying to access the [-1] index of the empty dataset.
I added a test that will also break the loop if an empty kline dataset is returned (after the symbol_existed).
Pretty sure this would have taken you about as long to write yourself than handle this PR but I'm new to github and fancied the chance to try this out.
Thanks for the library, it helped me out a lot getting off the ground trading,
Michael

Coverage decreased (-0.1%) to 36.304% when pulling a3cac52e6d140945cf11f315b46c4719aa898f80 on michja:klineLimitPatch into d35b2bb0e1c4cd4d8b9042fc130d82c041fe3c3c on sammchardy:master.

Coverage decreased (-0.1%) to 36.304% when pulling a3cac52e6d140945cf11f315b46c4719aa898f80 on michja:klineLimitPatch into d35b2bb0e1c4cd4d8b9042fc130d82c041fe3c3c on sammchardy:master.

Coverage decreased (-0.1%) to 36.304% when pulling a3cac52e6d140945cf11f315b46c4719aa898f80 on michja:klineLimitPatch into d35b2bb0e1c4cd4d8b9042fc130d82c041fe3c3c on sammchardy:master.

Coverage decreased (-0.1%) to 36.304% when pulling a3cac52e6d140945cf11f315b46c4719aa898f80 on michja:klineLimitPatch into d35b2bb0e1c4cd4d8b9042fc130d82c041fe3c3c on sammchardy:master.

Coverage decreased (-0.1%) to 36.304% when pulling a3cac52e6d140945cf11f315b46c4719aa898f80 on michja:klineLimitPatch into d35b2bb0e1c4cd4d8b9042fc130d82c041fe3c3c on sammchardy:master.

Thanks @michja