ASUS ROG Strix G15 Advantage Edition (G513QY-HQ008T), crash with device 'gfx902:xnack-'

    Hi
    I'm doing some basic OpenCL programming and things are working fine
    with my 'main PC' with an 'AMD Radeon PRO WX 7100 8GB PCIe 3.0'.

    clGetDeviceIDs() reports one device of CL_DEVICE_TYPE_GPU for
    the platform and clGetDeviceInfo() reports 'gfx803' as CL_DEVICE_NAME.

    A few days ago I bought the mentioned notebook but things did not
    work as expected. This notebook has an AMD Ryzen 9 5900HX CPU and an
    AMD Radeon RX 6800M GPU.

    Now clGetDeviceIDs() reports two devices of CL_DEVICE_TYPE_GPU for
    the platform and clGetDeviceInfo() shows 'gfx1031' and 'gfx902:xnack-'
    for CL_DEVICE_NAME.

    With my OpenCL programm I can use device 'gfx1031' but when trying
    to use device 'gfx902:xnack-' I get 'random errors'.

    Comparable problems also occur with clinfo. The symptoms are:

    When run the first time after power up of the PC the programs
    run up to some point and then stop with this error:

    Memory access fault by GPU node-2 (Agent handle: 0x5652e8860350) on address (nil). Reason: Unknown.

    In this case dmesg is showing things like these:

[  199.015019] amdgpu 0000:07:00.0: amdgpu: [gfxhub0] no-retry page fault (src_id:0 ring:173 vmid:8 pasid:32772, for process clinfo pid 1382 thread clinfo pid 1382)
[  199.015026] amdgpu 0000:07:00.0: amdgpu:   in page starting at address 0x0000000000000000 from IH client 0x1b (UTCL2)
[  199.015035] amdgpu 0000:07:00.0: amdgpu: VM_L2_PROTECTION_FAULT_STATUS:0x00840A51
[  199.015036] amdgpu 0000:07:00.0: amdgpu:      Faulty UTCL2 client ID: CPC (0x5)
[  199.015037] amdgpu 0000:07:00.0: amdgpu:      MORE_FAULTS: 0x1
[  199.015038] amdgpu 0000:07:00.0: amdgpu:      WALKER_ERROR: 0x0
[  199.015038] amdgpu 0000:07:00.0: amdgpu:      PERMISSION_FAULTS: 0x5
[  199.015039] amdgpu 0000:07:00.0: amdgpu:      MAPPING_ERROR: 0x0
[  199.015039] amdgpu 0000:07:00.0: amdgpu:      RW: 0x1
    When rerun the programs do NOT stop but seem to be in a busy loop
    with top showing about 100% CPU for the process. I do not see dmesg
    messages in this case.

    In my OpenCL program the last activity before the crash is just
    before I call clCreateContext().
    For clinfo the last messages printed on the screen before the crash
    are these:

...
Device Name                                     gfx902:xnack-
Device Vendor                                   Advanced Micro Devices, Inc.
...
...
Preferred work group size (AMD)                 256
Max work group size (AMD)                       1024
*** crash***
    BTW, I first noticed this problem with my Arch Linux and ROCm 4.3
    (and also 4.2) which I installed from the AUR.

    In order to make sure that the problem was not introduced by some
    error in the Arch installation I set up a fresh UBUNTU 20.04.02
    and installed the latest ROCm according to instructions from here:

    https://rocmdocs.amd.com/en/latest/Installation_Guide/Installation-Guide.html#supported-operating-systems

    After some minor correction for LD_LIBRARY_PATH I could/can run
    clinfo and my program also on UBUNTU 20.04.02 but the crash I see
    still is there. So the problem should not be with the Arch installation.

    Would anybody have a clue what might be the problem here or what
    could we do to narrow down the problem?

    BTW rocminfo does run without problems in any case.

Thanks @rudolfgh
Can you please share the output of /opt/rocm/bin/rocminfo and /opt/rocm/opencl/bin/clinfo
Thank you.

And also please share the output of lspci -nn | grep AMD/ATI for better understanding of the problem.
Thank you.

    Thanks for the info. Indeed I was not aware that

    'The integrated GPUs of Ryzen are not officially supported targets for ROCm'

    as can be read at the link you mentioned. This might well be the
    problem I ran into now. I just read somewhere that the integrated
    graphics for my CPU are based on GCN and GCN should be supported by
    ROCm. But perhaps because it is 'integrated' into the CPU it will
    nevertheless not work?

    Anyway I've generated the requested logs. Interestingly on my freshly
    installed UBUNTU 20.04.02 I can find two clinfo binaries and these
    produce (slightly) different output but both do crash at the end:

    -rwxr-xr-x 1 root root  80568 Jul 25 19:12 /opt/rocm/opencl/bin/clinfo*
    -rwxr-xr-x 1 root root 110136 Dez 19  2018 /usr/bin/clinfo*

    Log files included as uploaded files...

rog_lspci.txt
rog_opt_rocm_bin_rocminfo.txt
rog_opt_rocm_opencl_bin_clinfo.txt
rog_usr_bin_clinfo.txt

Hi @rudolfgh
Thanks for sharing the information.
ROCm supports GCN based cards but not all. gfx902 is not a ROCm supported card and as I told integrated GPUs are not supported. Only discrete cards are supported as of now.
Please check for supported hardwares @ https://github.com/RadeonOpenCompute/ROCm#hardware-and-software-support.
And also you have Navi10 XL card which is also unsupported with ROCm.
Thank you.

Hi,
thanks again for the 'refresh'. So I guess chances are low to see my
integrated GPU running (soon) with ROCm:-(
But one last question: you mention a 'Navi10 XL card' which is not
supported. If I'm right, isn't the dedicated GPU in my Notebook
(AMD Radeon RX 6800M) based on RDNA2 / Navi 22 and not Navi10?
Interestingly this GPU IS running since ROCm 4.2. So obviously there IS
some sort of support in ROCm for this GPU, even if not officially.
And yes, you're right, 'intergrated' GPUs are not supported, as I had to
learn. But even then I'd not expect programs to crash if an attempt is
made to use such an unsupported card when using officially documented APIs.
Instead a decent message like:
    GPU not supported

would be more appropriate than a 'Memory access fault'. The latter rather
seems to me like a programming error and does not necessarily refer to
supported or unsupported hardware. So perhaps your programmers might have
a chance to avoid this 'Memory access fault'?
BR
R. Schubert
--
Rudolf Schubert
Kirchstr. 18a                    \  @.***
D-82054 Sauerlach                /  http://www.dose.muc.de
Tel. 08104/908311               /
On Tue, 10 Aug 2021, ROCmSupport wrote:

Hi @rudolfgh
Thanks for sharing the information.
ROCm supports GCN based cards but not all. gfx902 is not a ROCm supported card and as I told integrated GPUs
are not supported. Only discrete cards are supported as of now.
Please check for supported hardwares @
https://github.com/RadeonOpenCompute/ROCm#hardware-and-software-support.
And also you have Navi10 XL card which is also unsupported with ROCm.
Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS orAndroid.[AVDS43EPVF45IWSDB2PH5QTT4EOCXA5CNFSM5BWFHSUKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOOR
PWSZGOGVT6D2A.gif]

Thanks @rudolfgh for the suggestions on showing a decent message like: GPU not supported for non-supported cards.
Let me work on this and I am creating an internal issue to move it further.
Thank you.

I created an internal ticket on this.
Thank you.


[Refactoring] Move more logic to view models to bookmarks screens #5543

Closes #5543

Force-pushed after rebase master


Docs link returns a 404

This link which is on the README.md links to a GitHub 404 page. The correct link should be this. Can we have this corrected ?

You can create a PR for that. But it would be faster if the core devs make a quick fix.
@Thecarisma @Youngestdev

fixed

I like your new docs by the way. Very informative. 👍

All the thanks to @Youngestdev

Nice! Since this is fixed i'll close


SamplePlayer no sound if child of a RigidBody2D

Operating system or device - Godot version:
OSX, Godot 2.1.2
Issue description:
I created a miniscene RigidBody2D, and add a SamplePlayer child for it.
(Add sample library, and wav file too)
This miniscene is instanced in a main game scene (control2D).
If i want to play sound from script, no sound comes.
But if i remove the SamplePlayer from RigidBody2D, and add directly to the game scene, there works.
Steps to reproduce:
See issue

can't reproduce the issue, same version of the engine.
most likely the OP did something wrong

I think you've used a SamplePlayer2D and it's out of the camera range...


3037 support privacy declaration resource type for custom fields

partially closes #3037
there will need to be some follow up changes on the fidesplus API side to make this fully work, but these are the necessary updates in fides
Code Changes

update sqlalchemy and DB resource_type enum for CustomFieldDefinition and CustomField tables/models to allow privacy_declaration as a value

include a migration for this change. migration downgrade path has to remove any records (CustomFields and CustomFieldDefinitions) that have the privacy_declaration as their resource_type

Steps to Confirm

The only real testing we can do on this PR specifically would be around the migrations.
Testing the actual application functionality will require some updates to the API, which lives in fidesplus

Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

@SteveDMurphy since you've touched some custom fields work, thought you may be good to review this?
also @TheAndrewJackson you're working on FE, not sure how familiar you are with the custom fields BE, but you can take a look too!
a real small change in fides, some corresponding changes in fidesplus coming shortly thereafter


add version

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Closes #623
Sorry for a pull request via a new branch, I was facing some issues.

Just added the Version in the navbar.
Thank you.

@FidalMathew Is this instead of the pull request #646 or as well as it? If it's instead of (totally understand if you feel the other got a bit unwieldly) then either you can close that one yourself or let us know and one of us will do it.

@FidalMathew Is this instead of the pull request #646 or as well as it? If it's instead of (totally understand if you feel the other got a bit unwieldy) then either you can close that one yourself or let us know and one of us will do it.
Yup, It got a bit unwieldy. I have now closed the other pull request, thank you.

@emmalearnscode could you check if it's fine now?

LGTM 🚀 Thank you for your contribution! 😄👍

Thank you! Had fun collaborating :)


my DispatchSemaphore  work error

I've read, understood, and done my best to follow the *CONTRIBUTING guidelines.
What did you do?
 let semaphore = DispatchSemaphore(value: 1)

...
    for j in 0..<assetsFetchResults.count {
            self.semaphore.wait()
             uploadfile(asset: assetsFetchResults[j])
      }
func uploadfile(asset:PHAsset){
....
   uploadimage(asset)
.....
}
  func uploadimage(asset:PHAsset){
            let option = PHImageRequestOptions()
                option.isSynchronous = true
            autoreleasepool {
             PHImageManager.default().requestImageDataAndOrientation(for: asset, options: option, resultHandler: { (imageData:Data?, dataURI:String?, orientation:CGImagePropertyOrientation, info:[AnyHashable : Any]?) in

                 AF.upload(multipartFormData: {
                    multipartFormData in multipartFormData.append(imageData!,
                                          withName:"file", fileName: dataURI, mimeType:"application/octet-stream")
                    }, to: "http://<IP_ADDRESS>:8080/file/upload",method: .post)
                    .responseJSON{responseJson in
                        debugPrint(responseJson)
                         self.semaphore.signal() //not work
                    }
                })
            }
        }

What did you expect to happen?
AF.upload is sync work
who can help me?
What happened instead?
delete code:
//  self.semaphore.wait()
// self.semaphore.signal()

debug:Terminated due to memory issue
add semaphore code  but self.semaphore.signal() not work
Alamofire Environment
**Alamofire version:5.2.2
**Xcode version:11.6
**Swift version:5.2.4
**Platform(s) running Alamofire:IOS
**macOS version running Xcode:11.15.6
Demo Project
N/A

Sorry, we use our GitHub project for bug reports and feature requests. In the future, you should open questions like this on Stack Overflow and tag alamofire.
Cheers. 🍻

From our Contribution Guidelines
Asking Questions
We don't use GitHub as a support forum. For any usage questions that are not specific to the project itself, please ask on Stack Overflow instead. By doing so, you'll be more likely to quickly solve your problem, and you'll allow anyone else with the same question to find the answer. This also allows maintainers to focus on improving the project for others.


Windows Support

It would seem that it is not possible to run pdf-extract from MS Windows as built? Just Googling around didn't yield any promising reasons why this is occurring? Any ideas?
System Configuration:

Microsoft Windows 8.1 x64
Ruby 2.2.3p173 (2015-08-18 revision 51636) [x64-mingw32]
DevKit-mingw64-64-4.7.2-20130224-1432-sfx
pdf-extract gem version 0.1.1

Error message:
C:\...\ > pdf-extract extract --references .\test2.mask.pdf
C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/commander-4.3.5/lib/commander/runner.rb:391:in `block in require_program': program version required (Commander::Runner::CommandError)
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/commander-4.3.5/lib/commander/runner.rb:390:in `each'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/commander-4.3.5/lib/commander/runner.rb:390:in `require_program'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/commander-4.3.5/lib/commander/runner.rb:52:in `run!'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/commander-4.3.5/lib/commander/delegates.rb:15:in `run!'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/commander-4.3.5/lib/commander/import.rb:5:in `block in <top (required)>'
C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require': 126: The specified module could not be found.   - C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/libsvm-ru
by-swig-0.4.0/ext/libsvm.so (LoadError)
        from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/libsvm-ruby-swig-0.4.0/lib/svm.rb:1:in `<top (required)>'
        from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from C:/Ruby22-x64/lib/ruby/2.2.0/rubygems/core_ext/kernel_require.rb:54:in `require'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/pdf-extract-0.1.1/lib/references/score.rb:1:in `<top (required)>'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/pdf-extract-0.1.1/lib/references/references.rb:3:in `require_relative'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/pdf-extract-0.1.1/lib/references/references.rb:3:in `<top (required)>'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/pdf-extract-0.1.1/lib/pdf-extract.rb:10:in `require_relative'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/pdf-extract-0.1.1/lib/pdf-extract.rb:10:in `<top (required)>'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/pdf-extract-0.1.1/bin/pdf-extract:5:in `require_relative'
        from C:/Ruby22-x64/lib/ruby/gems/2.2.0/gems/pdf-extract-0.1.1/bin/pdf-extract:5:in `<top (required)>'
        from C:/Ruby22-x64/bin/pdf-extract:23:in `load'
        from C:/Ruby22-x64/bin/pdf-extract:23:in `<main>'

Has this issue been resolved?

Not that I am aware of. I did not try to patch, and have not looked back
into it lately.
--
Jacob Hedrick
LinkedIn http://www.linkedin.com/in/jacobhedrick | Google+
https://plus.google.com/+JacobHedrick | Calendar
https://www.google.com/calendar/embed?src=jacob.hedrick%40gmail.com&src=stmnji0t28veqobcottlrcfh70<EMAIL_ADDRESS>Hedrick's Calendar
On Mon, Jun 13, 2016 at 7:03 AM, rishabh-joshi<EMAIL_ADDRESS>wrote:

Has this issue been resolved?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/CrossRef/pdfextract/issues/31#issuecomment-225551023,
or mute the thread
https://github.com/notifications/unsubscribe/AE3gspYJJZxpJngNwnvLuuK7Ly-ZlSL3ks5qLTjlgaJpZM4GvWHP
.


Adds the singleton-ops library to the community build

the repo readme seems to say that Typelevel Scala is required. is that not the case...?

perhaps we should only attempt to add this on the 2.13.x branch, where SIP-23 has been merged? but the repo doesn't have seem to have 2.13 support.
regardless, I'll do a test run and see what happens.

the repo readme seems to say that Typelevel Scala is required. is that not the case...?

No. singleton-ops works with LBS, as well. Its best use-case is SIP-23, but one can use it, just like people manage to use refined with shapeless Witness.

green in local test after fixups in e3b8257c0651d129f6fcb8a26935829e3d818fa2. thanks Oron!

I've merged this forward onto 2.13.x, where it will fail for a while because the latest 2.13.x nightly is weeks old because of recent Jenkins upheavals


In overview mode, prevent keyboard input into windows

Something like plugin:scroller:overview_ignore_input that prevents a focused window in the overview from receiving input from the keyboard. This is often the expected behavior of an overview like this.
Hyprland's follow_mouse distinguishes between mouse and keyboard focus, though I haven't checked whether this is exposed by the API.

The main differentiating factor between hyprscroller's overview and others, like you mention, is that you can interact with the windows. To me, that is a feature instead of something bad. You can move, re-adapt, navigate the overview, or even close/add windows. If I disable keyboard input, it would only be a nice, useless overview of your workspace. Remember, most people use the keyboard instead of the mouse in a tiling window manager. I am even working on more features in that sense (interactivity and scaling). There is also jump if you just want to see the windows.
Overview has to hack mouse positioning and which monitor the cursor belongs to. To Hyprland, each monitor covers a disjoint area of the frame buffer, and both workspaces in a two monitor setup share frame buffer coordinate space. Instead, hyprscroller uses a virtual coordinate space that can be "infinite", and that doesn't map to any facility in Hyprland. The truth is Hyprland's way of doing things is a bit strange to me in the sense that it separates workspaces, but then shares the framebuffer for visible workspaces but not for unseen ones, though.
Basically, what I am trying to say is overview and jump modes use a very fragile mechanism that hacks and modifies Hyprland's code from the plugin. This can break at any point. Any modification to Hyprland can, and at some point will break hyprscroller, and I will have to find another way to make it work. It was already quite complicated to make it work relatively well, so I am not going to add even more "hacking into Hyprland" complexity to it., because it is a recipe for an even earlier disaster.

I'm not asking for the keyboard to be disabled but rather for input not to be sent to the window from Hyprland. That's is, you could still resize or move focus, but not type something into the active window.
I mention follow_mouse because it suggests Hyprland makes this distinction already. Whether it's exposed by the API is another story.

Yes, but unfortunately I cannot control what gets sent to the active window. If the keyboard is on, it just works, so it will either translate a key binding to move/change focus, or send the input to the active window if there is no key binding associated to what the user typed.


Add option to only listen on one address

This fixes https://github.com/IBM-Swift/Kitura/issues/1450.
Description
Add address parameter to Kitura.addHTTPServer() and Kitura.addFastCGIServer() which are passed to Kitura-Net and Kitura-NIO for Server.listen().
Feedback is very welcome, especially if I accidentally missed some of your coding convention. I would pay special attention to the parameter order that I used for the methods.
Motivation and Context
This is necessary to fix https://github.com/IBM-Swift/Kitura/issues/1450.
How Has This Been Tested?
Manual testing by myself. I also modified the test harness to use the new code path.
Checklist:

[x] If applicable, I have updated the documentation accordingly.
[x] If applicable, I have added tests to cover my changes.

Does this look good?

This is now tagged in Kitura 2.8.0.


Feature chip imputed switch

Requires additional updates of the UI related to adding text helpers and table headers for characteristics filtering.

Looks otherwise fine... can you make the variant annotation table a little more table like...
Maybe bold the header and add horizontal divider between header and data rows.... center text


./gradlew appRun fails with yarn unable to start

./gradlew appRun fails with yarn unable to start. Blocked by #2

Fixed by porting to the official Gradle plugin: github.com/vaadin/vaadin-gradle-plugin/


重载函数 dismiss()

重载函数 dismiss()，便于用户手动关闭对话框时，在动画结束后处理后续业务逻辑。

I'm not sure what issue this solves. There is already a setter which consumer can set their dismiss listener via builder. setOnDismissListener. Closing this PR for now.


Prepare snapshot command

Added another command for creating snapshot content from mock already defined wallets. This will help app developers and tester to better prepare test scenario before this change, if they creates alias like "darek" in initials part of configuration, then in order to add new snapshot to mock there is a need to convert acquire public key in bytes of given wallet and add construct json out of it. It is still possible, but I've added another way to build snapshot by using /snapshot/create command. It can create snapshot from:

already defined wallets when resetting mock (with funds override, to create negative scenarios)
random entries
external entries - based on voting keys

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 96 (0.0%)  changed or added relevant lines in 12 files are covered.
6 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.03%) to 2.14%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

vitup/src/cli/generate/qr.rs
0
1
0.0%

vitup/src/builders/helpers/qr.rs
0
2
0.0%

vitup/src/builders/mod.rs
0
2
0.0%

vitup/src/cli/generate/snapshot.rs
0
3
0.0%

vitup/src/config/mod.rs
0
3
0.0%

vitup/src/cli/start/quick.rs
0
4
0.0%

vitup/src/config/initials/mod.rs
0
6
0.0%

vitup/src/config/builder.rs
0
7
0.0%

vitup/src/mode/mock/snapshot.rs
0
8
0.0%

vitup/src/mode/mock/mock_state.rs
0
10
0.0%

Files with Coverage Reduction
New Missed Lines
%

vitup/src/builders/mod.rs
1
0%

vitup/src/cli/start/advanced.rs
1
0%

vitup/src/config/builder.rs
1
0%

vitup/src/config/mod.rs
1
0%

vitup/src/mode/mock/mock_state.rs
1
0%

vitup/src/mode/mock/rest/mod.rs
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
94

Relevant Lines:
4392

💛  - Coveralls


Does not move to the next window in the stacking order

I have got two windows. The first window has a normal layer. The second window has a above layer. Also the second window is active.  When calls a command "nextstacked" does not move to the first window.

`diff -cr a/src/clientlist.c b/src/clientlist.c
*** a/src/clientlist.c  2016-09-14 03:57:29.000000000 +0300
--- b/src/clientlist.c  2016-09-25 17:52:46.000000000 +0300

*** 157,163 ****
/* Skip this window if it no longer exists or is currently in
* a state that doesn't allow focus.
*/
!          if(np == NULL || !ShouldFocus(np, 1)) {
continue;
}
--- 157,163 ----
/* Skip this window if it no longer exists or is currently in
* a state that doesn't allow focus.
*/
!          if(np == NULL || !ShouldFocus(np, 1) || np->state.status & STAT_ACTIVE) {
continue;
}
`

I'm not sure, but i think for solution of the problem need apply a patch:
diff -cr a/src/clientlist.c b/src/clientlist.c
*** a/src/clientlist.c  2016-09-14 03:57:29.000000000 +0300
--- b/src/clientlist.c  2016-09-25 17:52:46.000000000 +0300
***************
*** 157,163 ****
           /* Skip this window if it no longer exists or is currently in
            * a state that doesn't allow focus.
            */
!          if(np == NULL || !ShouldFocus(np, 1)) {
              continue;
           }

--- 157,163 ----
           /* Skip this window if it no longer exists or is currently in
            * a state that doesn't allow focus.
            */
!          if(np == NULL || !ShouldFocus(np, 1) || np->state.status & STAT_ACTIVE) {
              continue;
           }

Your patch makes sense and seems to improve the situation.  Thanks!


Context menu sometimes doesn't scale correctly on high DPI displays

This seems to happen fairly arbitrarily. Sometimes it's correctly sized, sometimes it isn't. A possible solution might be to stop using the Windows Forms API, and use something more modern instead.

Appears to be fixed as of 6e995a97


Allow Environment Shims

Hi,
There are command line tools that have commands to prepare the shell environment before executing commands. I'm not sure if there's a name for this, but some examples are things like pipenv run, op run,  and asdf env.
The main one I'm interested in is 1Password's op run because I'd like to be able to remove the sensitive environment variables I have from my .env files in my projects.
Here's how you might run a pipenv Python project outside of VSCode at the moment using op run
# .env
PYTHONPATH=.
SECRET=op://development/my-service/password

# main.py
import os

print("SECRET", os.environ["SECRET"])

$ pipenv run op run -- python main.py
Loading .env environment variables...
SECRET <concealed by 1Password>

Here pipenv run loads variables found in the projects .env and shims the virtual environment python into the PATH. op run -- then replaces any environment variables with the op:// paths with secrets from the password manager (authenticating their access). python main.py is then executed in this shell environment.
I currently can't see a way of integrating this into my vscode workflow. I've tried:

@brettcannon's suggestion of trying to trick vscode-python by putting {"python": "op", "pythonArgs": ["run", "--", "${command:python.interpreterPath}"}
writing an extension to try and leverage VSCode's command variables. Though doesn't work for a few different reasons, either. I can go though them if curious.

Would love to have a way to work around this, either with a different method from above, or from dimming down the sanity checks vscode-python does on the "python" argument. Or even better first class support for these kind of environment shims, something complimentary to "python.env" and "python.envFile":
// settings.json
{
    "python.envCommand": ["op", "run", "--"],
}

Could maybe accept a list of commands, if you want to stack environments:
// settings.json
{
    "python.envCommand": [
        ["pipenv", "run"],
        ["op", "run", "--"],
        ["asdf", "run"],
    ],
}

Anyway, quite a niche feature request, I know. But would love to be able to have the ability to do this, and happy to put in the grunt work of implementation, if I can be pointed in the right direction.

Another example is preparing a Node virtual environment as well as a python virtual environment on the command line with pypi:NodeEnv. At present I hack the venv/bin/activate to activate the node environment as I cannot access the python.envCommand, in web development of a Vite client to a Python server.

I cross-posted this onto the 1Password community forums, to try and farm some more eyes 👀.
They use vscode-go which currently does have a way of achieving this. Thought might be useful as a alternative implementation if there's any goals of making the settings interfaces between languages as homogeneous as they can be, for those who are bilingual.
Here's how you can achieve it in vscode-go:
# ~/.op/runners/go
if [ $# -ge 1 ] && ([ $1 = 'run' ] || [ $1 = 'test' ]); then
    op run -- go $@
else
    go $@
fi

// .vscode/settings.json
"go.alternateTools": {
    "go":"~/.op/runners/go"
}

So much like @brettcannon suggestion, but a more sanctioned way.

Thank you to everyone who upvoted this issue! Since the community showed interest in this feature request we will leave this issue open as something to consider implementing at some point in the future.
We do encourage people to continue 👍 the first/opening comment as it helps us prioritize our work based on what the community seems to want the most.

I'm glad this was brought up as a request: @jamesstidard
Would like to chime in and add what I thought would be viable but didn't pan out. Just adding the relevant bits here re: use of
# /src/set.sh
#!/bin/zsh
export GITHUB_ID=$(op read op://dev/auth-local/github/id)
export GITHUB_KEY=$(op read op://dev/auth-local/github/secret)
echo "hello ${GITHUB_ID} ${GITHUB_KEY}" // sanity check

{ // tasks.json
  "tasks": [
    {
      "label": "generateSecretReferences",
      "type": "shell",
      "command": "${workspaceFolder}/src/set.sh",
      "problemMatcher": []
    }
  ]
}

{ // launch.json
  "version": "0.2.0",
  "configurations": [
    {
      "name": "Python: 1Password Django",
      "cwd": "${workspaceFolder}/src",
      "preLaunchTask": "generateSecretReferences",
      "env": {
        "GITHUB_ID": "${env:GITHUB_ID}", // var substitution attempt
        "GITHUB_KEY": "${env:GITHUB_KEY}",
      }
    },
  ]
}

Through this route, the preLaunchTask successfully echoes the exported environment GITHUB_ID and GITHUB_KEY from 1Password. I tried vscode's variable substitution syntax to pipe the exported values back to the launch.json where Django was running and added a print function to see if the latter environment would show the values:
# src/manage.py
def main():
    """Run administrative tasks."""
    ...
    print(f"{os.environ.get('GITHUB_ID')=} {os.environ.get('GITHUB_KEY')=}")
    execute_from_command_line(sys.argv)

if __name__ == "__main__":
    main()

No dice. Result:
 *  Executing task: /src/set.sh

hello id-from-sanity-check key-from-sanity-check

# then it clears the window
# debug session starts

./.vscode/extensions/ms-python.python-2023.6.1/pythonFiles/lib/python/debugpy/adapter/../../debugpy/launcher 50892 --
....
os.environ.get('GITHUB_ID')='' os.environ.get('GITHUB_KEY')=''
Performing system checks...
...

So I'm guessing the preLaunchTask's environment where 1Password values were echoed, is a totally separate environment from the debug session proper where empty values were printed out when Django runserver was invoked.
Adding this tidbit in case it's helpful.


World Screen UI improvements

The world screen (which shows the map, resource sidebar, and tile projects sidebar) is the main screen for the game's UI. Better HTML+CSS design would make it prettier and more readable. For example, giving the sidebars their own scrollbars would make it so that players don't have to scroll the whole webpage to reach tile project options.

Working on this


[Enhancement] Refactor and deprecate Invoice repository class

The actual Sylius\InvoicingPlugin\Repository\DoctrineInvoiceRepository has been deprecated and a new class has been defined in the src/Doctrine/ORM folder with a name following the entity name.
The corresponding interface has been also deprecated and the new one is owning the only required method.
I propose this PR because the actual repository was not correctly used : it was an override of sylius_invoicing_plugin.custom_repository.invoice instead of a default value in the plugin configuration.

@pamil @GSadee I fix the ECS error, but this fix should maybe part of another PR I guess :
[ERROR] Class "Symplify\PackageBuilder\Console\HelpfulApplicationTrait" not found while loading
         "Symplify\EasyCodingStandard\Console\EasyCodingStandardConsoleApplication".

Thanks, Francis! :tada:


How to consistently load a file?

I am having issues with getting the script to load a file or files for analysis by chatgpt. Can you provide information on a way on how to do this better? My only known method is to add it to the code directory and tell it to look for the file there.

What are you trying to achieve exactly? That is in fact how the system works. You give it files and it will do with them what you ask. You can also use the --dir flag to set a working directory other than the code/ directory. Note that very large files won't fit in ChatGPT's context window, so you can't read them directly. I will answer more on this in #20

Okay. My goal was for it to read a file and explain what it is doing (a long yaml file, I will try again with a smaller one later). It would ignore or disregard all of my requests in the code directory despite multiple attempts.


[FEATURE] sql agent

Hello good morning
It would be excellent to have this functionality in the version of java that currently exists in python and js
https://python.langchain.com/v0.2/docs/integrations/toolkits/sql_database/#use-sqldatabasetoolkit-within-an-agent
https://js.langchain.com/v0.1/docs/use_cases/sql/agents/
allows chatting with an agent to a relational database

Hi, this is somewhat supported, please check https://github.com/langchain4j/langchain4j-examples/blob/main/rag-examples/src/main/java/_3_advanced/_10_Advanced_RAG_SQL_Database_Retreiver_Example.java


fix style of limit-req & limit-req-cn

Summary
fix style with markdownlint
Full changelog

fix style for limit-req & limit-req-cn

Issues resolved
Fix #1273

@gxthrj many thx, merged already


Add JinaAIEmbedding class

Description
This PR adds JinaAIEmbedding class to allow users to obtain embeddings from Jina AI Embeddings API (https://jina.ai/embeddings/).
I am hesitating now how to add Documentation about it and I would like to hear suggestions from the team. I see there is a notebook using Jina Embeddings from HuggingFace but the usage is very different, should I edit that one or create a new one?
Fixes # (issue)
Type of Change
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)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[ ] I stared at the code and made sure it makes sense

Suggested Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

@JoanFM Thanks for the contribution!
Yea I would add a new notebook using this embeddings class, and then link to the notebook in docs/core_modules/model_modules/embeddings/modules.md

Hey @ravi03071991 ,
Thanks for the review, I put this as draft as I am still working on the notebook.
I will open it again as Ready for Review once it is finished. Thanks for the heads up though.

Hey @ravi03071991 ,
Thanks for the review, I put this as draft as I am still working on the notebook.
I will open it again as Ready for Review once it is finished. Thanks for the heads up though.

Now it should be ready

Thanks JoanFM! Could you resolve the merge conflict and the lint? Will put it to automerge after it's fixed!

Thanks JoanFM! Could you resolve the merge conflict and the lint? Will put it to automerge after it's fixed!

Done @Disiok , thanks


Outrank

Thank you for your contribution to the IC Developer Portal.
Before submitting your Pull Request, please make sure that:

[ ] You have updated the .github/CODEOWNERS file according to your changes.
[ ] You have registered new documents to /sidebars.js if any.

Dear @TobyMeller,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation

Dear @TobyMeller,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation

@TobyMeller Can you please following through these CLA steps?

I have added this as a comment, I believe it is now correct?
On Sun, Jan 14, 2024 at 10:56 PM Jennifer K. Tran @.***>
wrote:

Dear @TobyMeller https://github.com/TobyMeller,
In order to potentially merge your code in this open-source repository and
therefore proceed with your contribution, we need to have your approval on
DFINITY's CLA https://github.com/dfinity/cla/blob/main/CLA.md.
If you decide to agree with it, please visit this issue
https://github.com/dfinity/cla/issues/427 and read the instructions
there. Once you have signed it, re-trigger the workflow on this PR to see
if your code can be merged.
— The DFINITY Foundation
@TobyMeller https://github.com/TobyMeller Can you please following
through these CLA steps?
—
Reply to this email directly, view it on GitHub
https://github.com/dfinity/portal/pull/2343#issuecomment-1891101019, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/A4T43XNBGTBKDZIRZTXNI4LYORPDRAVCNFSM6AAAAABBZIHTKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJRGEYDCMBRHE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Does not work for elements without an ID attribute

Hello @medabida, Savy does not work for elements without an ID attribute.
I've patched the code locally and this is what I have come up with a workaround for elements that do NOT have an "id" specified.
savy.js line # 6
add this
		  if(!this.id) this.id = $(this).attr("name");
after
   if (order == "load") {       $(this).each(function() {
so final code is
    if (order == "load") {       $(this).each(function() { 		  if(!this.id) this.id = $(this).attr("name");         if ($(this).is(":radio")) {

Hope this helps
~Rajesh

Hi Rajesh, the latest version of Savy should handle this correctly.


Provide more detailed context information on the spiderfied event

When listening to the spiderfied event it's pretty hard to get the spiderfied context. The only provided context is target which is a reference of whole cluster object.
But only one marker with ist child markers got spiderfied, so it's helpful to provide this information when fire the event.

Manually merged this finally, thanks :)


Update WellNode.toml

Description
Links to previous PRs
(Insert here)
Rules
All previous genesis validators should name their PRs "Update {validator_alias}.toml" and provide links to previous PRs merged.
All new validators should name their PRs "Create {validator_alias}.toml" and provide links to their previously closed PRs, if any.
Checklist before merging

[x] Only one toml is added in this PR
[x] The file being added has no special characters in its name (no '.' or ' ')
[x] The file being added is indeed a .toml file
[x] The toml being added is to the correct folder, and only to the correct folder
[x] The eth_hot_key and eth_cold_key are present
[x] The email is present

This accidentally had 2 validator_accounts in it. I deleted the second one.


Add amp-dynamic-css-classes sample

Resolves #353

Blocked on https://github.com/ampproject/amphtml/issues/4953

Sebstian and I chatted and I'm going to push the dynamic reordering of social-share buttons into a separate PR and create a simple boolean 'show-if-referred-by' example for this PR.

Update - PTAL


Lazy loading: modal from lazy loading page which loaded from split pane failed to load script in path location strategy

Short description of the problem:
modal from lazy loading page which loaded from split pane failed to load script in path location strategy.
What behavior are you expecting?
It should work as hash location strategy.
Steps to reproduce:

Clone repo
Start server, navigate to List page
Click any item in list, wait about 10 seconds and error handling screen will be displayed.

A sample application via Github
https://github.com/longgt/ionic-sidemenu-demo
Which @ionic/app-scripts version are you using?
cli packages:

    @ionic/cli-utils  : 1.9.2
    ionic (Ionic CLI) : 3.9.2

local packages:

    @ionic/app-scripts : 3.0.0
    Ionic Framework    : ionic-angular 3.7.1

System:

    Node : v8.5.0
    npm  : 5.4.2
    OS   : Windows 7

Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

Hello! Thank you for opening an issue with us!
This seems like it is a bug in the Ionic Framework and not a bug in the Ionic App Scripts. I will move the issue to the proper repo for that.
Thank you for using Ionic

This issue was moved to ionic-team/ionic#13156


update work pool health indicator

Updating these indicators in preparation for the new Dashboard view. The previous icons used for healthy and unhealthy appeared interactive/clickable (similar to a dismiss button or radio). This makes them more generic. The green dot is intended to be more passive than an illustrative icon. Distinction is maintained between each for accessibility (e.g. a red dot may not be visibly distinguishable from a green dot to certain users).
New healthy dark:

New healthy light:

New unhealthy dark:

New unhealthy light:

New paused dark:

New paused light:

On the dashboard we're going to have multiple of these side by side. Thoughts on making the healthy and others all the same size?

On the dashboard we're going to have multiple of these side by side. Thoughts on making the healthy and others all the same size?

Good call out. That was somewhat by design:

The unhealthy and paused states will be larger, but that will also allow them to stand out a bit more in the set and draw attention. Can always adjust if things get fugly.

@brandonreid the latest commit makes me wonder if we should just include the work pool queue name in the tooltip by default. We already have the name here and it would simplify this a decent bit and enforce some consistency.

@brandonreid the latest commit makes me wonder if we should just include the work pool queue name in the tooltip by default. We already have the name here and it would simplify this a decent bit and enforce some consistency.

@pleek91 that makes sense to me:


Closes #111 - Add landing page design

Here is the design update for landing page. It includes additional interactivity.
Please see the comments in the Figma project which also explain how the design technically behaves:
https://www.figma.com/file/FRqpUl9lXzAZqG02BHk626/Home_page_base?node-id=0%3A1
I'm yet too much noob for how to use Figma properly so I used comments and no components. I was extremely slow or inefficiently working with it. Every change costs so much time, I practically worked 4 working days on this.

I don't know why the small screen version was exported so large:

I also considered of all your last comments, @NickZecamp . Just the footer-banner, I thought, would be better to leave with the colour of the background because its position is fixed.

looks good

@NickZecamp I changed that loudspeaker to an eye. Is there some more things for this current design?

Everything looks good to me ok_hand
Just discovered that you are using unicode characters in your icons. The problem is that they can be displayed in different ways and do not always follow the same style. What do you think if we replace them with images?

Yes, that's okay. I didn't know about this problem. Could you help me with that? Maybe I should replace some of my cards and the input field with your field?

Everything looks good to me ok_hand
Just discovered that you are using unicode characters in your icons. The problem is that they can be displayed in different ways and do not always follow the same style. What do you think if we replace them with images?

I added a Figma commment there which explains to use images. It's too much for me now to change it, so I hope this sufffices.


Could not load file or assembly 'Swashbuckle.AspNetCore.Swagger'

Describe the bug
Using version 6.6.1, I have an exception when I run the following command in an Azure pipeline.
Command:
dotnet swagger tofile --output $(System.DefaultWorkingDirectory)/apim/artifacts/apis/dms-api/specification_v1.json bin/Release/net6.0/MyAPI.dll v1

Expected behavior
Should run fine like in version 6.5.0
Actual behavior
Throw exception
Steps to reproduce
Run command
dotnet swagger tofile --output <path> MyAPI.dll v1

Exception(s) (if any)
Unhandled exception. System.IO.FileLoadException: Could not load file or assembly 'Swashbuckle.AspNetCore.Swagger, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=62657d7474907593'. The located assembly's manifest definition does not match the assembly reference. (0x80131040)
File name: 'Swashbuckle.AspNetCore.Swagger, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=62657d7474907593'
   at Swashbuckle.AspNetCore.Cli.Program.<>c.<Main>b__0_4(IDictionary`2 namedArgs)
   at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable`1 args) in /_/src/Swashbuckle.AspNetCore.Cli/CommandRunner.cs:line 68
   at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable`1 args) in /_/src/Swashbuckle.AspNetCore.Cli/CommandRunner.cs:line 59
   at Swashbuckle.AspNetCore.Cli.Program.Main(String[] args) in /_/src/Swashbuckle.AspNetCore.Cli/Program.cs:line 141

Swashbuckle.AspNetCore version
6.6.1
.NET Version
.Net 6
Anything else?
No response

Any chance you could provide a minimal reproducible example as a GitHub repository that demonstrates the issue? It would make it easier/faster for us to investigate and fix.

Also, does this help? https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/2651#issuecomment-1882968072

I can't see any obvious differences in the package contents between 6.5.0 and 6.6.1.
Does your project consistently use v6.6.1 everywhere, so for example, isn't picking up references to a different version in your application's output directory?

Does your project consistently use v6.6.1 everywhere, so for example, isn't picking up references to a different version in your application's output directory?

I had this problem, and a dependency elsewhere on 6.5.0 was the cause. Adding the 6.6.1 package directly to my project resolved the issue.

Thanks for the hint !
Indeed, pipeline was using latest cli version but code still had dependency over older swashbuckle versions.

This is causing 6.7.3 to fail. Have to use 6.6.1 instead.
Unhandled exception. System.IO.FileNotFoundException: Could not load file or assembly 'Swashbuckle.AspNetCore.Swagger, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=62657d7474907593'. The system cannot find the file specified.
File name: 'Swashbuckle.AspNetCore.Swagger, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=62657d7474907593'
   at Swashbuckle.AspNetCore.Cli.Program.<>c.<Main>b__0_4(IDictionary`2 namedArgs)
   at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable`1 args) in /_/src/Swashbuckle.AspNetCore.Cli/CommandRunner.cs:line 68
   at Swashbuckle.AspNetCore.Cli.CommandRunner.Run(IEnumerable`1 args) in /_/src/Swashbuckle.AspNetCore.Cli/CommandRunner.cs:line 59
   at Swashbuckle.AspNetCore.Cli.Program.Main(String[] args) in /_/src/Swashbuckle.AspNetCore.Cli/Program.cs:line 159

Ensure that all your references, including transient ones, are using the same version of Swashbuckle.


fix: save dora_scale in locon.py

there is no save dora_scale code in the locon.py

I guess loha/lokr may have similar issues
Can you also fix them?

loha, lokr is include dora_scale when save model. but I have an issue when I inference image with trained dora file in the a1111
merged_scale1 / merged_scale1(dim=self.dora_mean_dim, keepdim=True) * self.dora_scale
TypeError: 'Tensor' object is not callable

loha, lokr is include dora_scale when save model. only locon not include dora_scalebut.
I have another issue when I inference image with trained dora file in the a1111
merged_scale1 / merged_scale1(dim=self.dora_mean_dim, keepdim=True) * self.dora_scale
TypeError: 'Tensor' object is not callable

This is something weird things that should not happend
but can be easily fixed
will open PR later


Test snapshots

Test a few different scenario's for the daily created snapshots: https://github.com/threefoldtech/grid_deployment/tree/development/snapshots#public-rsync-provided-by-threefold

Extracting using multiple cores compared to one core (default): possible ram issue
Run the snapshots on low spec'd machines
Time sync difference with hdd(?) / ssd / nvme
..

devnet snapshots working again since https://github.com/threefoldtech/tf_operations/issues/1996


Smart Max Stack, Max Locals, and Frames Computation

In this PR I fixed the problem with automatic computation of max stack, max locals, and frames per method basis.
The solution is quite suboptimal since Java ASM library doesn't provide this functionality.
You can read more about this problem here.
In order to solve this problem I used reflection which has some risks for the future maintainability of CustomClassWriter:

it might work incorrectly in a concurrent environment
it's rather slow since constantly changes ClassWriter class definition in runtime (reflection)
It's better to find the proper solution for this problem in the future.

Closes: #528.
History:

feat(#528): add compute frames flag to all ClassWriter implementations
feat(#528): suggest ugly solution
feat(#528): remove the puzzle for #528 issue
feat(#528): remove redundant code
feat(#528): move all writers under CustomClassWriter class
feat(#528): fix all linter suggestions
feat(#528): add one more puzzle

PR-Codex overview
This PR refactors bytecode generation in the project. It introduces a CustomClassWriter and improves method writing logic.
Detailed summary

Introduces CustomClassWriter for bytecode generation
Updates method writing logic in BytecodeMethodProperties
Replaces ClassVisitor with CustomClassWriter in BytecodeMethod
Improves bytecode generation in BytecodeClass

The following files were skipped due to too many changes: src/main/java/org/eolang/jeo/representation/bytecode/CustomClassWriter.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 16min)

@rultor release, tag is 0.4.0, title is 0.4.0.

@rultor release, tag is 0.4.0, title is 0.4.0.

@volodya-lombrozo OK, I will release it now. Please check the progress here

@rultor release, tag is 0.4.0, title is 0.4.0.

@volodya-lombrozo Done! FYI, the full log is here (took me 17min)


Fixed undefined error log on sendMessage

I was getting random undefined printed on the console. This fix the issue.

Thank you for the PR will review soon

Seems simple enough to me. Thank you for the PR


docs: update release documentation

As we discussed in the #275 I've added documentation for the release process

Lovely - very clear!

Cool - do we do the beta -> master PR now?

@johnnyreilly I think you can safely pull the trigger in #275 :)

Bad news - when you try you get this message:

This branch can't be rebased

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

How did you get it to work? Can you just not rebase on a phone?


Forearm in rigged hand

Good morning,
I'm trying to use a full-rigged model of a hand with the LeapMotion Orion. Particularly, I need to update the RiggedPepperFullHand (v2.3) to v4.4. Everything works well with models without the forearm (RiggedPepperCutHand), however with the global model the arm doesn't' work properly.
I thought that it would have been sufficient to select the forearm bone and drag&drop it into the RiggedHand. Forearm field, but it doesn't work. I've seen that the bones' hierarchy in the prefab (v2.3) is different from the one in the model (always v2.3). Maybe it is the issue, but I don't know how to modify it. Can you help me, please?
Thank you very much in advance!

I recently managed to get both the old "Salt" and "Pepper" hands (with forearms) to work with Orion. It's a bit of a struggle as the bones don't appear to align with the default Orion hand structure.
I have created a Unity Package which you can download here. Let me know if you have any issues.

Thank you very much @brandonjmatthews
I've tried your unitypakage and it works!
Only a side note, I've noticed that the bones of the "SaltBaseFullRightHand" were not defined in the example scene, but I've just followed the "PepperBaseFullRightHand" setup and all of them works great now! Thanks


Unprocessable Entity error in Pytest despite conventional curl request works

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from enum import Enum
from typing import Optional

from pydantic import BaseModel, Field, root_validator

class OrderBy(str, Enum):
    ascending = 'ascending'
    descending = 'descending'

class BaseGet(BaseModel):
    order: Optional[OrderBy]
    order_name: Optional[str]
    search_string: Optional[str]

    class Config:
        use_enum_values = True
        orm_mode = True

    @root_validator
    def validate_date(cls, values):
        if not values["order"]:
            values["order_name"] = None
        return values

class GetWithSkipLimit(BaseGet):
    skip: int = 0
    limit: int = Field(default=10, le=100)

@router.get("/groups", status_code=200, response_model=List[Group])
async def get_groups_by_user(payload: GetWithSkipLimit):
    pass

Description
I have setup some Pydantic classes
from enum import Enum
from typing import Optional

from pydantic import BaseModel, Field, root_validator

class OrderBy(str, Enum):
    ascending = 'ascending'
    descending = 'descending'

class BaseGet(BaseModel):
    order: Optional[OrderBy]
    order_name: Optional[str]
    search_string: Optional[str]

    class Config:
        use_enum_values = True
        orm_mode = True

    @root_validator
    def validate_date(cls, values):
        if not values["order"]:
            values["order_name"] = None
        return values

class GetWithSkipLimit(BaseGet):
    skip: int = 0
    limit: int = Field(default=10, le=100)

and an API
@router.get("/groups", status_code=200, response_model=List[Group])
async def get_groups_by_user(payload: GetWithSkipLimit):
    pass

When I run the pytest
response = await client.get('/user/groups', headers=headers, params={
        'skip': 0,
        'limit': 100,
        'order': 'ascending',
        'order_name': 'name',
        'search_string': '',
    })

I receive the 422 Unprocessable Entity error  {'detail': [{'loc': ['body'], 'msg': 'field required', 'type': 'value_error.missing'}]} as answer, so apparently the test did not work out.
However, when I run
curl -X 'GET' \
  'http://localhost:8000/user/groups' \
  -H 'accept: application/json' \
  -H 'Content-Type: application/json' \
  -d '{
  "skip": 0,
  "limit": 100,
  "order": "ascending",
  "order_name": "name",
  "search_string": ""
}'

I get the desired response.
How could I make it work in that test?
Operating System
macOS
Operating System Details
No response
FastAPI Version
0.85.0
Python Version
3.10
Additional Context
No response

Using curl, you are sending your parameters as a JSON body. When using the test client, you are using params =, as using a GET request cannot have a body posted to it (requests makes the params as query parameters). However, since they are pydantic models, FastAPI does expect it to be passed as body.

@JarroVGIT Many thanks for your input. Do you have a tiny example what I have to di exactly? I am unfortunately not very experienced here.

@JarroVGIT Many thanks for your input. I solved it now by changing the get function to
async def get_page_groups_by_user(page: int, size: int,
                                  order: Optional[OrderBy],
                                  order_name: Optional[str], search_string: Optional[str]):
    pass

However, I am wondering whether there is an elegant way to condense it to fewer parameters. Nevertheless, I am already happy that it works, so a big thank you 😀

Amazing answer and explanation @JarroVGIT ! 🙇
@junoriosity currently there's no better way. In a future version it will be possible to declare multiple query parameters in a single Pydantic model, but not yet. 🤓


Train fails on each and every command

Hi
I ran:
python main.py --mode distill_basic --dataset Cifar10 --arch AlexCifarNet     --distill_lr 0.001 --train_nets_type known_init --n_nets 1     --test_nets_type same_as_trai
and got:
File "cifar_100_dataset_distillation/dataset-distillation/train_distilled_image.py", line 176, in accumulate_grad
d.grad.add_(g)
AttributeError: 'NoneType' object has no attribute 'add_'
Please advise.
Gonen

We haven't tested this code on later pytorch versions. If you see problems, please try using pytorch 1.0.0.


Dynamic notification titles

Is your feature request related to a problem? Please describe.
I want to have a list of messages to display to the user at a interval, for example:

The user has a list of ["Hello", "World", "This", "Is", "Me"]

I then would like to schedule daily notifications that would iterate over that list and send a different element every day:

On day 1 it would send "Hello"
On day 2 it would send "World"
etc....

Describe the solution you'd like
What I've described above
Describe alternatives you've considered
The only alternative that I could think of, is every time the user logs into the app, I would clear the scheduled notification and replace it with a new one pointing to a different element in the list. But this is messy and requires the user to open the app everyday.
Additional context
None.

Is this completed?

Sorry, i had seen this post. This is not possible to implement. You can create scheduled notifications, each one with different body and different title, but you can't code notifications like this. To update this values you can create a recurrent background task to make this updates and update an entire week ahead each time.


Update for OpenFF-Units v0.1.7

There is an API change with the latest openff-units so this PR updates paprika/restraints/openmm.py. I'm also pinning openff-units to version 0.1.7.

older versions

from openff.units.simtk import to_simtk

new version

from openff.units.openmm import to_openmm

I'll fix the pytests in another PR.

Some tests still failing?


new solver system

tests fail because UnfoldSim.jl does not yet have the solver-rng update released

woop woop, new solver system seems up and running! let's merge


[BUG] every installation will show fatal: [voiptcel]: FAILED! => {"msg": "Failed to connect to the host via ssh: ssh: connect to host X.X.X.X port 22: Connection timed out"}

Hi
I have follow the installation process and wait so long time during process installation
but will show error
2022-07-20 01:46:47,808 p=6612 u=vassupport n=ansible | fatal: [voiptcel]: FAILED! => {"msg": "Failed to connect to the host via ssh: ssh: connect to host X.X.X.X port 22: Connection timed out"}
2022-07-20 01:46:47,808 p=6612 u=vassupport n=ansible | RUNNING HANDLER [nginx : restart nginx] ****************************************
2022-07-20 01:46:47,809 p=6612 u=vassupport n=ansible | NO MORE HOSTS LEFT *************************************************************
2022-07-20 01:46:47,810 p=6612 u=vassupport n=ansible | PLAY RECAP *********************************************************************
then I try again and again it still shows same things
what I have to do, to solve it
please help me
Thank you so much

Look like your ansible unable to access host due to mis-configuration of firewall.

Okay it's running now
i change config on file inventories/production/hosts.yml
set ansible_host to <IP_ADDRESS>
but now show another issue :
TASK [captagent : Copy captagent role source to target host] ***************************************************************************************************************************************************************************
fatal: [voiptcel]: FAILED! => {"changed": false, "cmd": "/usr/bin/rsync --delay-updates -F --compress --archive --rsync-path='sudo -u root rsync' --out-format='<>%i %n%L' /home/user/libresbc/third-party/captagent/ /opt/libresbc/0.5.9/third-party/captagent/", "msg": "rsync: change_dir#1 "/opt/libresbc/0.5.9/third-party/captagent/" failed: Permission denied (13)\nrsync error: errors selecting input/output files, dirs (code 3) at main.c(639) [Receiver=3.1.3]\n", "rc": 3}
PLAY RECAP *****************************************************************************************************************************************************************************************************************************
voiptcel                   : ok=9    changed=4    unreachable=0    failed=1    skipped=0    rescued=0    ignored=0
why can it happen?
in another issue you said :
"or the rsync issue, seems like the permission issue that ssh user was not able become root user. unable to reproduce on my side."
but how to solve it, I can't get it
please help me
Thank you so much


chore(trading): 4134 see if a closed market was succeeded

Related issues 🔗
Closes #4134
Description ℹ
Add cell renderer for successor market
Demo 📺
Gif/video/images of new/corrected functionality if applicable
Technical 👨🔧
Details of technical implementation that reviewers may need to be aware of, if applicable.

wrongly rebased, branch to remove


Controlling  Joints (J1,J2,J3,J4) requested

Using move_to() method, now we can easily move x,y,z,r but we can't control individually. So can we control Joints (J1,J2,J3,J4)?
Thanks.

@wiroon just use one of these modes in the move_to function:
PTPMode.MOVJ_ANGLE
PTPMode.MOVL_ANGLE
PTPMode.JUMP_ANGLE


fix for deprecated packages (identity, tumbler)

Codecov Report

Merging #44 (a84d859) into master (75c874f) will not change coverage.
The diff coverage is 41.66%.

@@           Coverage Diff           @@
##           master      #44   +/-   ##
=======================================
  Coverage   90.53%   90.53%
=======================================
  Files          23       23
  Lines        1669     1669
=======================================
  Hits         1511     1511
  Misses         91       91
  Partials       67       67

Impacted Files
Coverage Δ

pkg/server/environment.go
69.23% <33.33%> (ø)

cmd/iso8583/main.go
76.87% <50.00%> (ø)

pkg/server/server.go
85.71% <50.00%> (ø)

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 75c874f...a84d859. Read the comment docs.


View Summary on the Overview Page

We've redesigned the website completely and this issue does not apply anymore.


Integrate get block with transaction id api

Description
Following from issue #1946 this PR implements GET /api/v1/blocks?transaction_id={transaction_id}&status={UNDECIDED|VALID|INVALID} api. In order to maintain compatiability with 1.x there is no change to the query parameters i.e. the client still specifies transaction_id and status but in the current scenrio specifiying status other than VALID will always return empty list in response.

Codecov Report

Merging #2021 into tendermint will increase coverage by 0.39%.
The diff coverage is 87.5%.

@@              Coverage Diff               @@
##           tendermint    #2021      +/-   ##
==============================================
+ Coverage       51.66%   52.06%   +0.39%
==============================================
  Files              71       71
  Lines            3509     3517       +8
==============================================
+ Hits             1813     1831      +18
+ Misses           1696     1686      -10

@kansi I am having a hard time understanding what all this pull request is doing. I guess it's some code related to the
GET /api/v1/blocks?transaction_id={transaction_id}&status={UNDECIDED|VALID|INVALID}
endpoint, but there doesn't seem to be any change to the Flask code, nor any changes noted in the HTTP API documentation. Maybe you could add some description in the opening comment of the PR.

@ttmc I updated the issue description, updating the docs.

For me, we can drop the status parameter.

@vrde @ttmc The status query paramter is removed. A note has been added in the documentation to reflect the same.


Change the way cutoff for rewards is calculated

Description
In this version, we had added a nextDistribution field to Firebase. For some reason, it was under the content item, which made no sense and caused the app to crash in older versions because it was expecting a string before.
This nextDistribution was the date when the next CELO rewards distribution would be made. This is always the following Monday, since that's the cutoff date, so instead of having that value on Firebase now we are just calculating it in the client directly. This will make sure it's always up to date and we don't forget to update it as well.
Tested
Manually.
Related issues

Fixes #64

Backwards compatibility
Yes

Hi @gnardini We have verified this task using  latest Android Internal Build V1.12.0(1004294340) & Test flight build V1.12.0(49) and found mentioned issue is fixed.
Fixes #64
Can you please let us know if we need to test anything else in this task for now.


Unblock @blackgirlbytes - receiving a 500 and 400 error when running this project

This function call fails for me:     const employeeStatus = await checkForEmployee(user);
Oddly enough, it works for bdougie even when he hardcodes my username in, but doesn't work for me at all. It's weird because I am an employee and have all the same permissions.
I tried hitting the endpoint via CURL and i received:

I ran the actual app and received this error:

Message from bdougie:
Feel free to reach out to engineering to see if you can get unblocked on the #ecosystem-api. I would first open an issue in the automation repo first though. Also gr2m is a good resource.

Looks like the github token in my .env file is not being pulled..and that's the reason this issue is happening


testnet is not sync

I've looked at some of the questions I've been asking。but still error。
server_info.   "complete_ledgers" : "empty"
{
"result" : {
"state" : {
"build_version" : "1.1.1",
"closed_ledger" : {
"base_fee" : 10,
"close_time" : 595941480,
"hash" : "CA94B8C605A6479460AF158BC95E4143468E7A5CEBF30D42105A2FF8A88AD308",
"reserve_base" : 200000000,
"reserve_inc" : 50000000,
"seq" : 7
},
"complete_ledgers" : "empty",
"io_latency_ms" : 1,
"jq_trans_overflow" : "0",
"last_close" : {
"converge_time" : 2000,
"proposers" : 0
},
"load" : {
"job_types" : [
{
"job_type" : "untrustedValidation",
"per_second" : 5
},
{
"job_type" : "untrustedProposal",
"per_second" : 5
},
{
"job_type" : "ledgerData",
"peak_time" : 5
},
{
"in_progress" : 1,
"job_type" : "clientCommand"
},
{
"job_type" : "writeObjects",
"per_second" : 3
},
{
"avg_time" : 5,
"job_type" : "sweep",
"peak_time" : 19
},
{
"job_type" : "peerCommand",
"per_second" : 26
},
{
"job_type" : "SyncReadNode",
"per_second" : 7
},
{
"job_type" : "AsyncReadNode",
"per_second" : 1
},
{
"job_type" : "WriteNode",
"per_second" : 6
}
],
"threads" : 6
},
"load_base" : 256,
"load_factor" : 256,
"network_ledger" : "waiting",
"peer_disconnects" : "0",
"peer_disconnects_resources" : "0",
"peers" : 2,
"pubkey_node" : "n9Lz8wd1phnTaixzDvnEp1zHdGSqPUeSN9H1ajQ6gFXosPmtSWT9",
"pubkey_validator" : "none",
"published_ledger" : "none",
"server_state" : "connected",
"state_accounting" : {
"connected" : {
"duration_us" : "311645645",
"transitions" : 1
},
"disconnected" : {
"duration_us" : "1254422",
"transitions" : 1
},
"full" : {
"duration_us" : "0",
"transitions" : 0
},
"syncing" : {
"duration_us" : "0",
"transitions" : 0
},
"tracking" : {
"duration_us" : "0",
"transitions" : 0
}
},
"time" : "2018-Nov-19 11:18:09.192339",
"uptime" : 314,
"validation_quorum" :<PHONE_NUMBER>,
"validator_list_expires" : 0
},
"status" : "success"
}
}
Here is my cfg

Here is log file:

node_size I had set: tiny

Your server encountered difficulties retrieving the UNL. I just checked locally and could retrieve the file just fine.
Can you change the line that says:
https://vl.altnet.rippletest.net

To this:
http://vl.altnet.rippletest.net

and try again please?We’ve had instances where this solved the problems that server operators had.

Closing this stale issue. Feel free to reopen if you have new details to add.


[Question] Is there an embeddings model in MLC format?

❓ General Questions
Hi, Is there an embeddings model in MLC format that I can use with the MLC?
My reasoning is to use the same API as OpenAI and to do so, I need to be able to use endpoints such as
/v1/chat/completions/embeddings

consolidating to #2324


Models not available

Hi, I am attempting to use peakonly. An error occurs.  it looks like the models can not be available remotely. Please check. Thanks.

Hi!
You can download model weights manually by the following links on Google drive:
Classifier https://drive.google.com/file/d/1ryr6KAQ7QPN488D0BH4OI5f_I9KZ4oU7/view?usp=drive_link
Segmentator https://drive.google.com/file/d/12m-7b-MlkNyyTAqmgXehz4NEAU1uS_UE/view?usp=drive_link
RecurrentCNN https://drive.google.com/file/d/1TLKv5UZLYOb4r0IwFqukCeHmbFXs71jl/view?usp=drive_link
and put them into peakonly folder: \Peakonly\data\weights\

Thanks so much.
--
Yunying Xie, Professor
Institute of Medicinal Biotechnology,
Chinese Academy of Medical Sciences & Peaking Union Medical college
Tiantan No.1, Beijing, China
At 2023-12-21 18:33:41, "Vadim Yanshole" @.***> wrote:
Hi!
You can download model weights manually by the following links on Google drive:
Classifier https://drive.google.com/file/d/1ryr6KAQ7QPN488D0BH4OI5f_I9KZ4oU7/view?usp=drive_link
Segmentator https://drive.google.com/file/d/12m-7b-MlkNyyTAqmgXehz4NEAU1uS_UE/view?usp=drive_link
RecurrentCNN https://drive.google.com/file/d/1TLKv5UZLYOb4r0IwFqukCeHmbFXs71jl/view?usp=drive_link
and put them into peakonly folder: \Peakonly\data\weights\
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/429 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit 1c986f1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


PREVIEW: [Jetsurvey] Changed to use Navigation Compose

Depends on #1058 landing first, so only opening as a draft.
This eliminates all fragments in favor of navigating at the composable
level. I also updated the screen-level composables to take in explicit
lambdas rather than event handlers to follow best practices.
One notable change here is that there is no longer a fragment from which
to launch the camera. Instead, the PhotoQuestion uses
rememberLauncherForActivityResult. This also uses TakePicturePreview,
which doesn't require a pre-registered URI, so the file permission is no
longer necessary (the contract returns a Bitmap directly).

This depends on #1058 landing first but fixes #1038 when landed.


Set reason for condition IAMConfigured to NA

Signed-off-by: zhujian<EMAIL_ADDRESS>Address code review comments: https://github.com/stolostron/hypershift-deployment-controller/pull/76

/cc @jnpacker

/lgtm


Question about building with NCCL

The doc say:
https://xgboost.readthedocs.io/en/stable/build.html

Enabling distributed GPU training
By default, distributed GPU training is disabled and only a single GPU will be used. To enable distributed GPU training, set the option USE_NCCL=ON. Distributed GPU training depends on NCCL2, available at this link. Since NCCL2 is only available for Linux machines, distributed GPU training is available only for Linux.
mkdir build
cd build
cmake .. -DUSE_CUDA=ON -DUSE_NCCL=ON -DNCCL_ROOT=/path/to/nccl2
make -j4

But I does not add -DUSE_NCCL=ON and build the package it can still run with multiple GPU cores (testing by xgboost spark).
Do I miss something ?

CC @trivialfis Thank you!

@rongou Recently enabled GPU training without nccl federated training. But that might cause significant performance lost on GPU clusters.

@rongou Recently enabled GPU training without nccl federated training. But that might cause significant performance lost on GPU clusters.

Understood.
Btw, for the official xgboost package release on pypi, does it build with NCCL ? @trivialfis

Yes. It's built with NCCL.


Increase grpc max message size

Functions Host is updated to send command line argument: --grpcMaxMessageLength
Python worker needs to be updated to set this value for grpc.max_send_message_length and grpc.max_receive_message_length on the channel

@1st1 / @elprans  - We are rolling out functions runtime soon.  It would be good to have the change on the worker sooner to support working with large blobs

@pragnagopa Can you point me to the Host artifact that contains this please?

Just to confirm, does 11709 also contain the enable_content_negotiation (#101) bit?

Yes. You can use these for proto changes as well!


Ordinal Provenance

@cmdruid appreciate all the work and examples in this library!
I had a question regarding ordinal provenance as documented here.
I have an inscription flow in place which generally follows the example you provided for inscriptions. While I believe i have followed the ordinal provenance documentation correctly as far as the script construction and input values, I am having issues signing the transaction.
Code snippets below, any thoughts/suggestions on how to get this working is appreciated! I am currently getting  RPC error: {"code":-26,"message":"non-mandatory-script-verify-flag (Invalid Schnorr signature)"}. The ordinal I am trying to use as the parent is here: https://mempool.space/testnet/tx/e13f106578fe0955b375cfb66e4a212fd794b7168a7da5873fada2b78c47e035

The problem is that for sig2, you are using the tapkey to sign, however the tapkey is a tweaked public key from Tap.getPubKey, and you need the tweaked private key in order to sign.
There should also be a Tap.getSecKey which you can use to get the tweaked private key, and you should be able to use that to produce a proper sig2.
Let me know if that change fixes the issue.
The API isn't very clear in this regard, so I apologize. I'm hoping to make V2 of the library even simpler to use.

@cmdruid thanks for the quick response! I just tried that but still facing the same issue. does the whole process make sense in context of the ordinal provenance docs? I think the relevant instructions are in the screenshot below. i believe i have correctly added the script tag 3, but think I am having issues with the spending the parent in input of the child.

So far I have

created an inscription with this same code/approach without the parent code i recently added. Inscription here: https://testnet.ordinals.com/inscription/e13f106578fe0955b375cfb66e4a212fd794b7168a7da5873fada2b78c47e035i0
setup a testnet wallet on the address where the parent inscription is held and using the secret key hardcoded in the child inscriber
added the 3 script tag to the script with the txid of the parent serialized according to the ordinals docs (i believe)
added the parent txid as an input to the new child inscription
added a second signature to try and sign the transaction

I'm not clear how the child tx is supposed to be formatted in order to spend the parent.
Do you have a link to a pair of example transactions that perform a successful child inscription? That would help clarify how this should be formatted.

here is a child

txn: https://mempool.space/tx/080eb48f5d1d4afeeaff6a1ca4d7dbb049578b4fcd6cdb63961d8a1ed7b85bfb
ordinal: https://ordinals.com/inscription/080eb48f5d1d4afeeaff6a1ca4d7dbb049578b4fcd6cdb63961d8a1ed7b85bfbi0

Looking at the transactions that you linked, the example inscriptions do not seem to be following the documentation.

I see the '03' being pushed, along with the 32-byte txid.
I do not see a 4 byte index being pushed.
There's also a '05' being pushed, with an additional blob of data. I'm not sure what that is supposed to be.
The parent also has these attributes in the script. So is the example parent also a child to another inscription?

In any case, the second input doesn't seem to be anything important to the spec, it looks like just an input being used for fees and change. So that input should just be a simple P2TR key-spend of the selected utxo.

thanks @cmdruid

I will check on the 4 byte index
the 05 tag is passing the metadata, see screenshot from the docs below. however this is not mandatory for parent/child
yes in the example the parent is also a child. here is the topmost child in that tree (https://ordinals.com/inscription/6390ee5c262da84fc217c8a66423750f102d35fb929eab236700f66608a877dfi0, https://mempool.space/tx/6390ee5c262da84fc217c8a66423750f102d35fb929eab236700f66608a877df)\
Is the second input not the Spend the parent P in one of the inputs of T. part of the spec? Can you elaborate on So that input should just be a simple P2TR key-spend of the selected utxo being used for fees

Is the second input not the Spend the parent P in one of the inputs of T. part of the spec? Can you elaborate on So that input should just be a simple P2TR key-spend of the selected utxo being used for fees

From my understanding, only one input is handing the inscriptions stuff, so it is both spending the parent and minting the child. All the inscription related stuff should be in that tx input.
Any additional inputs are merely adding funds to the tx, and are not relevant to the inscription protocol. You can use any utxo for that, it doesn't even need to be P2TR actually.

@skyrover7
Did you find a way inscribing the child inscription using Parent?
Even I was trying do the same thing , seeing your code
I think I have a suggestion
the var=endian you are using happens to be a wrong value (i guess)
//Pass the inscription id to the below function
`function convertHexString(hexString) {
// Reverse the hexadecimal string
const reversedHexString = hexString
.match(/.{2}/g) // Split string into pairs
.reverse() // Reverse the order
.join("");
// Convert reversed hex string to a byte array
const byteArray = [];
for (let i = 0; i < reversedHexString.length; i += 2) {
byteArray.push(parseInt(reversedHexString.substr(i, 2), 16));
}
// Convert byte array to hexadecimal string with '0x' prefix
const formattedHex = "0x" + Buffer.from(byteArray).toString("hex");
return formattedHex;
}
`
Try using the above helper function to convert
and let me know , if it worked
Coz I need your help after this !!

@rohitkeshav27 nope, still no luck
i believe the endian value is correct. I am using the below function and am able to make an inscription with the delegate ID. the delegate ID and parent ID need to be serialized the same way. I have confirmed that the below code is creating the correct endian value and is recognised as a valid delegate ID. Here is an inscription with valid delegate ID using the below code for ID conversion https://ordinals.com/inscription/20bd89f9f9b29314f7995286c18a80386d245efc1f03aa76e07feb81bf1748a8i0
my trouble is still on signing the transaction, hoping @cmdruid can shine some light 🙏🏾
export function convertInscriptionId(inscriptionId) {
    // Extract the TXID from the inscription ID
    const txid = inscriptionId.slice(0, 64);

    // Extract the INDEX value following the 'i' after the TXID
    const indexPart = inscriptionId.slice(65); // Assuming the index always follows 'i' after the TXID
    const index = parseInt(indexPart, 10);

    // Reverse the bytes of the TXID
    const reversedTxid = txid.match(/.{2}/g).reverse().join('');

    // Convert the INDEX to a 4-byte little-endian format
    const littleEndianIndexBuffer = Buffer.alloc(4);
    littleEndianIndexBuffer.writeUInt32LE(index);

    // Concatenate and return the result
    let result = reversedTxid + littleEndianIndexBuffer.toString('hex');

    // Omit trailing zeros
    result = result.replace(/0+$/, '');

    return result;
}

Do you have updated code for signing the transaction?
Your previous example was using the tapkey to sign the second input, which was incorrect. It should be the secret key that signs for that second utxo.

@skyrover7
I had a doubt in your code
you are spending the parent inscription in input
But you are not creating an appropriate output , does this mean your parent inscription is spent ( lost)
and
2nd input is a taproot address , so the witness passed also shd be like the first input
There is something missing !!
rohitkeshav
the above is my discord username - we could connect and work this together ( coz even I have an urgent need to complete this )

@rohitkeshav27 sent you a fr on discord

I am closing this for now. Feel free to reopen this issue if you still need help.


Restreaming

I din't get it, i mean if i subcribe paid iptv with My isp but that only give 1 device, can i using this metode to restream To another device(More than 5 device) Without pay more to my isp ? specialy Not using Vlc but like another Modifcation App ?

Yes, but only if your ISP utilizes source-specific multicast (UDP) for receiving the stream. Then IPTV-ReStream is able to receive it once from the ISP and re-broadcast it to as many devices as you want.
Other apps are theoretically supported, but practically only VLC can play this properly.

Actuly I know some IPTV Restreaming seler But i can't get any data Like channel list or M3U files because it's already in the form of an application , so I need pay and Get some username and password to acses Channel in application for one device ,Can you have solution like this?? So i can only pay for one device And Restream To another device?? Or maybe i still need some Source Link/list Data Like M3U to Restream using this method?

You definitely need some kind of stream source. You can probably extract it from the app. Nevertheless IPTV-ReStream cannot handle any kind of source authentication to access the stream, which is probably needed in your case. You'd need to heavily adjust IPTV-ReStream to your demands.


problem in setup metro env

Thanks for your nice work! When I run setup the metro conda env, this following code:
cp ../../code/body_models/* metro/modeling/data/
It seems that there is not existing this folder?

these are the MANO pkls. The download code should have downloaded a mano zip file and inside are the files that you need.


Need help move image

This library is very good and helpful.
With the first time, I can not move image. In scaled mode, I only can move with dx <= 0, dy <= 0.
I want to custom to can move to any position on screen.
Please help me. Thanks so much.

why moving the image instead of moving the whole view?!


Path to sqlite db?

To confirm, do we instructions to fill out .env to path to sqlite file or is sqlite:// sufficient?

Actually this setting isn't even required since the path is hardcoded.
https://github.com/TwilioDevEd/airtng-flask/pull/61/files#diff-fa59bd3b08fbbc037c01717e802afddcL17

Cool, can we remove it from .env?


The Application is not working after send 100000 messages

Hello, i need your help because, we have a company communicator and send a 100K messages (all my users), the app broke for a two weeks and not sen any messages to the users, in the application the appear the warning that said
Our Services aren't available rigth now
We're working to restore all services as soon as posible. Please check back soon
Attache the screenshots of the error

Hi @reinaldo27,
Thanks for raising the query, we're looking into your query, will get back to as early as possible with response.

Hi @reinaldo27,
Can you please try the below steps and let us know if it resolves the issue,
Login to Azure portal and navigate to resource group,
-> Click on Azure function which name is ending with Prep function
-> Open Prep function URL (https://.prep-function.azurewebsites.net
-> Add. SCM in URL like https://.prep-function.scm.azurewebsites.net
-> Click on Debug console
-> Click on CMD
-> Navigate to Site -> wwwroot
-> Open host.json file by clicking on edit icon
-> Add "maxOrchestrationActions": 150000 and save.
After the above changes are completed, go back to the azure prep function, and restart the service.

Hi @reinaldo27,
Please let us know if the solution provided above resolved the issue and if we can close this issue

Closing the issue, if still not resolved please create new issue


Adding a shape check for set_module_tensor_to_device.

Fixes #1729

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Write message changes to a logging channel

When Apollo logs changes to messages (edits/deletion), write it to a #logging channel
https://github.com/UWCS/apollo/blob/0db0711a924b8cd1485c88ccd89a5a52e0ace452/cogs/database.py#L84-L126

Superseded by #40.


Fix incorrect access to track fields on disc getters

Tagging @pinkforest and @Serial-ATA -- am I missing something here or is this a bug?
Also any thoughts on #27 and #28?
Thanks, and sorry to bother!

You're not missing anything, I just don't have the ability to merge this. :smile:

Sorry I thought I had watch on.

Looks like we need to fix clippy - this will be a breaking change for 0.5 - @Serial-ATA any reason why we should not merge ?

Yeah, this is good to merge. The clippy fix is in #28

Thanks! Appreciate you both looking over the changes.


Refactor the Philips Hue support to split bridge support from the lights platform

Description:
This is in preparation to supporting sensors.
Related issue (if applicable): fixes #10609
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#<home-assistant.github.io PR number goes here>
Example entry for configuration.yaml (if applicable):
hue:

Checklist:
If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Hi @andreacampi,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

There are a few things I still have to clean up, but I wanted to get the ball rolling and get early feedback.
To do:

support multiple switches;
support the older config schema and provide upgrade instructions;
add back support for discovery.

Re config, this changes from:
light:
  - platform: hue
    host: DEVICE_IP_ADDRESS

to:
hue:
  - host: DEVICE_IP_ADDRESS

I find this makes more sense when the component can support more than just lights, yes?

cc @robmarkcole

Hi @andreacampi,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Hi @andreacampi,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

It seems like you have some commits in this branch that are not yours, please clean base your work off a fresh copy of the dev branch.

I haven't looked at the code at all, but yes, migrating hue from just a platform to be a component for auth and a platform for lights = 👍

Im not sure I follow this refactor, appears to be a lot of light specific code in components/hue.py  which is the hub/auth..?

@robmarkcole yeah that's fair, not sure why I decided to go that way. I'll clean that up and/or explain.

I'll rebase and check the failed tests; just not today as I've had a long day on ~4 hours of sleep.

@andreacampi looks good and new config looks suitable. Cheers!

Thanks for the great feedback. I'm pretty happy that most of it is existing code I was too lazy to fix in the refactoring, but happy to do it now :D I'll get on it.

OK, latest push addressed all concerns from @balloob. Ready for one more round of feedback :)

I noticed some of the tests were red on 3.5, fixed that. Not sure why coveralls is complaining give the coverage increased; and also it's mostly concerned with a file (homeassistant/components/discovery.py) I barely touched. I'd ignore that.

Is it possible to write the tests to work with a mock of the library  instead of importing it?
cc @balloob for his opinion on this.

So, question: should I squash all commits together? I like working with small commits but this is getting a bit messy :)

Please don't squash until code review is finished. Squashing makes it hard to follow changes. We always squash when we merge, so the contributor is never required to squash.

Hey folks, how do we move forward with this?

This is an unrelated test that failed. I can try rebasing I guess.

🎉  🐬  Awesome, great job !

These changes seem to be allowing HASS to discover it's own emulated hue bridge as a bridge device and throws an error because it can't find the config file.

I'm also having some problems since this commit after re-working my configuration. Once the hubs are added with emulated hue disabled, I see the error below for each light. Should I open up a new issue?
Update for light.garage__right fails
Traceback (most recent call last):
  File "/srv/homeassistant/homeassistant_venv/lib/python3.5/site-packages/homeassistant/helpers/entity.py", line 199, in async_update_ha_state
    yield from self.async_device_update()
  File "/srv/homeassistant/homeassistant_venv/lib/python3.5/site-packages/homeassistant/helpers/entity.py", line 306, in async_device_update
    yield from self.hass.async_add_job(self.update)
  File "/usr/lib/python3.5/asyncio/futures.py", line 380, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.5/asyncio/tasks.py", line 304, in _wakeup
    future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 293, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.5/site-packages/homeassistant/components/light/hue.py", line 385, in update
    self.update_lights(no_throttle=True)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.5/site-packages/homeassistant/components/light/hue.py", line 149, in <lambda>
    lambda **kw: update_lights(hass, bridge, add_devices, **kw))
  File "/srv/homeassistant/homeassistant_venv/lib/python3.5/site-packages/homeassistant/util/__init__.py", line 306, in wrapper
    result = method(*args, **kwargs)
  File "/srv/homeassistant/homeassistant_venv/lib/python3.5/site-packages/homeassistant/util/__init__.py", line 306, in wrapper
    result = method(*args, **kwargs)
TypeError: update_lights() got an unexpected keyword argument 'no_throttle'

Both of these are pretty bad :( I'll try to report and open tasks as needed, but @mezz64 @m2k2 if you want to do that'd be helpful.
@m2k2 unless I broke something last second while iterating on unit tests, I don't know that I have enough to reproduce it. I'll ping you if I have trouble.

@balloob thoughts on me removing the @Throttle annotation? that would remove the issue @m2k2 is seeing.

See https://github.com/home-assistant/home-assistant/pull/11126

Another pull request: https://github.com/home-assistant/home-assistant/pull/11128

How would one set the scan_interval now on the new hue component? Before I was able to set it per the docs here: https://home-assistant.io/docs/configuration/platform_options/
Leaving this in place with the new Hue component seemed to break the configuration. Once I know the new way to set a scan_interval, I'll go ahead and submit a PR to update the docs.
Thanks!

Please open an issue if you suspect a bug. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. Thanks!

This work appears to change the filename of the default phue.conf to phue-<bridge serial number>.conf for discovered bridges. This causes HomeAssistant to boot with:
2017-12-28 10:15:26 INFO (MainThread) [homeassistant.setup] Setting up hue
2017-12-28 10:15:26 INFO (MainThread) [homeassistant.setup] Setup of domain hue took 0.0 seconds.
2017-12-28 10:15:26 INFO (MainThread) [homeassistant.core] Bus:Handling <Event component_loaded[L]: component=hue>
2017-12-28 10:15:26 INFO (MainThread) [homeassistant.core] Bus:Handling <Event platform_discovered[L]: service=philips_hue, discovered=host=<IP_ADDRESS>, port=80, ssdp_description=http://<IP_ADDRESS>:80/description.xml, name=Fhue (<IP_ADDRESS>), model_name=Philips hue bridge 2015, model_number=BSB002, serial=00178848ac26>
2017-12-28 10:15:26 INFO (SyncWorker_6) [phue] Attempting to connect to the bridge...
2017-12-28 10:15:26 INFO (SyncWorker_6) [phue] Error opening config file, will attempt bridge registration
2017-12-28 10:15:26 WARNING (SyncWorker_6) [homeassistant.components.hue] Connected to Hue at <IP_ADDRESS> but not registered.
2017-12-28 10:15:27 INFO (SyncWorker_6) [homeassistant.loader] Loaded configurator from homeassistant.components.configurator
2017-12-28 10:15:27 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_registered[L]: domain=configurator, service=configure>
2017-12-28 10:15:27 INFO (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=configurator.philips_hue, old_state=None, new_state=<state configurator.philips_hue=configure; configure_id=139857224447368-1, fields=[], friendly_name=Philips Hue, entity_picture=/static/images/logo_philips_hue.png, description=
Press the button on the bridge to register Philips Hue with Home Assistant.

![Location of button on bridge](/static/images/config_philips_hue.jpg)
, submit_caption=I have pressed the button @ 2017-12-28T10:15:27.009680+00:00>>

And to fail to connect to the bridge.
My hue bridge is in the roof and pressing the button was quite a pain.
It would be handy to either, upgrade (i.e. rename) the existing phue.conf file automatically, or, to have a log message showing the filename the component was trying to read, so I could manually rename it.

Please open an issue in the appropriate reoo if you suspect a bug or there's a problem with documentation. If you need help please use our help channels:
https://home-assistant.io/help/#communication-channels
Merged PRs should not be used for support or bug reports. You can link to the PR that you want to tag from the issue. Thanks!