dry-run in provision --how=connect leads to error

This is dry mode so not an urgent thing but I assume because its not a real run the remote tmt directory structure is not created and so rsync fails.
tmt run --id remote-001 --all provision --how connect --guest ampere-mtsnow-altra-08.khw4.lab.eng.bos.redhat.com --dry
/var/tmp/tmt/remote-001
/plans/kselftests
discover
how: fmf
directory: /home/sbertram/gitstuff/kernel
path: selftests/wrapper
tests: rhivos
summary: 1 test selected
provision
how: connect
guest: ampere-mtsnow-altra-08.khw4.lab.eng.bos.redhat.com
user: root
key: not provided
summary: 1 guest provisioned
prepare
how: shell
name: Enable Repos
overview: 1 script found
how: install
name: Packages
package: tar, yum, autoconf and 15 more
how: shell
name: sendip rpm
overview: 1 script found
how: install
summary: Install recommended packages
name: recommends
package: netcat, traceroute, man and 10 more
finish
summary: 0 tasks completed
stderr (3/3 lines)
rsync: [sender] link_stat "/var/tmp/tmt/remote-001/plans/kselftests/data" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1663) [Receiver=3.1.3]
rsync: [Receiver] write error: Broken pipe (32)

Failed to run command 'rsync -Rrz --links --safe-links --protect-args -e ssh -oForwardX11=no -oStrictHostKeyChecking=no -oUserKnownHostsFile=/dev/null -S /run/user/4202847/tmt/tmp31__du98 root@ampere-mtsnow-altra-08.khw4.lab.eng.bos.redhat.com:/var/tmp/tmt/remote-001/plans/kselftests/data /'. Reason: Command returned '23'.

Yes, during dry run all guest.push() and guest.pull() actions should be ignored. Should be very easy to fix. Maybe some of the new contributors would be interested?

@psss, I'd like to fix it to get familiar with TMT.

Just FYI: simpler reproducer (uses virtual provisioner to get local VM, any VM/remote machine is enough)
0. fresh directory

tmt init -t base
tmt run --id ownVM plans --default provision -vvv -h virtual --connection system
Note IP from the step above
tmt run -a provision -h connect -g  --dry

To stop and remove machine created in 2. step tmt run --id ownVM plans --default finish

@sbertramrh, would you please show me the detailed steps about how to reproduce this issue? I failed to reproduce it according to @lukaszachy 's suggestion.
$ export GUEST=hpe-dl380pgen8-02-vm-7.hpe2.lab.eng.bos.redhat.com
$ tmt run --id remote-001 --all provision -vvv --how connect --guest $GUEST --dry
/var/tmp/tmt/remote-001

/plans/example
    discover
        status: done
        summary: 1 test selected
    provision
        status: done
        summary: 1 guest provisioned
    prepare
        status: done
        summary: 0 preparations applied
    execute
        status: done
        summary: 1 test executed
    report
        how: display
        summary: 1 error
    finish
        status: done
        summary: 0 tasks completed

total: 1 error

@idorax The only other details I can provide would be the actual test. I have sent you a separate email with that test case. Also, looking at your test run output, I think you are running against the same output directory and not cleaning it after your test. This will in most cases cause an error and not run properly. --id forces tmt to run the test from the same location So either clean it after each run or point --id to another directory.
Also can you do a clean run of the test you are running and lets see the output, this is without --dry. I want to make sure the test is running successfully. Thanks.

@idorax What tmt version do you have? Either 1.11 or current github main branch should fail
$ tmt run --id remote-001 --all provision -vvv --how connect --guest <IP_ADDRESS> --dry
/var/tmp/tmt/remote-001

/plans/example
    discover
        how: fmf
        directory: /tmp/r_152128_mAc
        summary: 1 test selected
    provision
        how: connect
        order: 50
        guest: <IP_ADDRESS>
        user: root
        key: not provided
        summary: 1 guest provisioned
    prepare
        how: install
        summary: Install required packages
        name: requires
        package: beakerlib

        fail: Failed to pull workdir from the guest. This usually means that login as 'root' to the guest does not work.
    finish
        summary: 0 tasks completed

stderr (3/3 lines)
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
rsync: link_stat "/var/tmp/tmt/remote-001/plans/example/data" failed: No such file or directory (2)
rsync error: some files/attrs were not transferred (see previous errors) (code 23) at main.c(1816) [Receiver=3.2.3]
rsync: [Receiver] write error: Broken pipe (32)

Failed to run command 'rsync -Rrz --links --safe-links --protect-args -e ssh -oForwardX11=no -oStrictHostKeyChecking=no -oUserKnownHostsFile=/dev/null -S /run/user/17062/tmt/tmpn25550cz root@<IP_ADDRESS>:/var/tmp/tmt/remote-001/plans/example/data /'. Reason: Command returned '23'.

Well, my tmt looks pretty old, its version is 1.8.0.  I'll try 1.11 later.
# tmt --version
tmt version: 1.8.0 (0577fe6)

If  --dry is specified, I found that self.opt('dry') is True in guest.push() but it is None in guest.pull().
So currently I added a property self.dry to __init__() in class class Guest(tmt.utils.Common):

https://github.com/teemtee/tmt/blob/main/tmt/steps/provision/__init__.py#L224

@@ -252,6 +252,8 @@ class Guest(tmt.utils.Common):
         # Initialize role, it will be overridden by load() if specified
         self.role = None
         self.load(data)
+        # Save 'dry' flag in case it's gone
+        self.dry = self.opt('dry')

Then in both guest.push() and guest.pull(),  such code is added to fix this defect.
+        # Skip to pull in dry mode
+        if self.dry:
+            return

@lukaszachy and @sbertramrh, please help to review, thanks!

ups ....

If --dry is specified, I found that self.opt('dry') is True in guest.push() but it is None in guest.pull().

this stinks, it will need to be investigated

Ouch. Reading code and command line properly - the --dry is parameter of 'provision' not of the whole run.
For (see that --dry moved from end of line just after run) tmt run --dry --id remote-001 --all provision --how connect --guest <IP_ADDRESS>
True
(Pdb) p guest_copy.opt('dry')
True

Yes, that looks like it. So only provision was run in dry mode (so nothing was provisioned) but the other steps tried to use the provisioned guest.
The guest_copy thing was introduced so that guest uses options of the currently running step. Previously, it was strictly tied to provision so that when tmt run prepare -vvv was used, no verbose output would be seen from the guest which was rather confusing. But as a side-effect, when only provision has --dry this means that after creating a copy and changing the parent to prepare, guest no longer sees the dry mode and hence tries to do something. This may be tricky to solve (at least I currently don't see a straight forward solution)

Indeed - guest.push sees 'dry' is bug (strictly speaking). However - Is there a valid use case when one can run --dry for provision but not for later steps? Imho it doesn't make sense, no provision -> no SUT to run steps on.
IMO dry should be global option. Or is the expected usage to skip over one step (so one doesn't need to enumerate rest?)

Yes, --dry should not be used for skipping steps when not needed or when already done, for this --skip should be used. And I don't see any real life use case to run only provision --dry while attempting to perform other actions without a provisioned guest. A single --dry option for the whole tmt run as suggested by @lukaszachy makes probably the most sense to me.

IMO dry should be global option. Or is the expected usage to skip over one step

Thinking about it a bit more I (probably) found a relatively reasonable use cases for using --dry with a single step:

It could make sense to run the prepare step in dry mode (just quickly show me which required packages would be installed or which ansible playbooks would be applied)
For long test scripts I can imagine doing an execute --dry to quickly double check that the right command line is used to run the tests and the right set of environment variables is provided to it

@lukaszachy, @FrNecas, does it make sense? If so, then perhaps using the guest.ready() method which was discussed on today's hacking session could be a better solution.

It does.

@lukaszachy, currently class Guest doesn't have a method ready(), do we need to add one, right?

Exactly - And ideally have it set properly: let it be true when guest lives (start() or wake(), I'm not sure) and false once guest is not available (stop() maybe)

Yes, Guest.ready() should return True after start() and/or wake() methods are completed and return False after stop() is called or before the guest is fully initialized. Perhaps we could use an internal guest._ready attribute for storing the current status.

@psss, @lukaszachy and @FrNecas, if I run tmt run --dry --id remote-001 --all provision --how connect --guest GuestVM, the output looks like:
$ tmt run --dry --id remote-001 --all provision --how connect --guest GuestVM
/var/tmp/tmt/remote-001

/plans/example
    discover
        how: fmf
        directory: /home/huanli/dev/1064/foo
        summary: 0 tests selected
    provision
        how: connect
        guest: GuestVM
        user: root
        key: not provided
        summary: 1 guest provisioned
    prepare
        summary: 0 preparations applied
    execute
        how: tmt
        summary: 0 tests executed
    report
        how: display
        summary: no results found
    finish
        summary: 0 tasks completed

total: no results found

Is this expected output? If yes, looks patch https://github.com/teemtee/tmt/pull/1094/commits/6bad57278c4921a5898bc693e9277be408251b17 fixed this issue because with the patch, tmt run --id remote-001 --all provision --dry  --how connect --guest GuestVM has the same output to tmt run --dry  --id remote-001 --all provision --how connect --guest GuestVM.
Sorry I'm confused with the solution above about Guest.ready() because if we use tmt run --dry ..., --dry is a global option as a matter of fact.  Please explain more details if we should have the solution above, thanks!

https://github.com/teemtee/tmt/blob/main/tmt/cli.py#L255

@force_dry
def run(context, id_, **kwargs):
    """ Run test steps. """
    # Initialize
    run = tmt.Run(id_, context.obj.tree, context=context)
...<snip>...

Hi @psss and @lukaszachy, to fix this issue, do we need to change something in steps of prepare, execute, finish? Currently I only updated class Guest(), please help to take a look at the latest commit in https://github.com/teemtee/tmt/pull/1094, thanks!

Since PR#1094 is ready, let's give a brief summary right here. To fix this issue, the final solution  looks like:

Introduce new _options attribute to Common class
Propagate dry mode from provision to prepare, execute and finish in method go() in Run class
Add unit test to case tests/core/dry


update proxy of master to raftstore-proxy by proxy pr 262

What problem does this PR solve?
Issue Number: ref #4982
Problem Summary:
We update Proxy here to adapt bugfix of flashback.
update proxy to raftstore-proxy
Proxy PR:
https://github.com/pingcap/tidb-engine-ext/pull/262
Including:\nSubmodule contrib/tiflash-proxy 453b9df4bd..6444536dc4:

Merge pull request #262 from CalvinNeo/merge-tikv-release-6.6-2

What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

/run-integration-test

/merge


convert_dataset_json.py does not respect parameter --split

https://github.com/mosaicml/llm-foundry/blob/bd8127252c660e45ed01413645d29427f86c085a/scripts/data_prep/convert_dataset_json.py#L204C4-L204C4
out=os.path.join(args.out_root), should be out=os.path.join(args.out_root, folder_split), as in convert_dataset_hf.py.

I will work on fixing this now


Please

@supernova you are cute, lovely, and beautiful, we know your fav IDE is notepad of windows.


Fix typo for sigmas in mne_surf2bem.py

Tiny fix to take care of error "_surfaces_to_bem() got an unexpected keyword argument 'sigma'".

Codecov Report

Merging #4598 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #4598      +/-   ##
==========================================
+ Coverage   87.04%   87.04%   +<.01%
==========================================
  Files         349      349
  Lines       66030    66030
  Branches    10251    10251
==========================================
+ Hits        57476    57477       +1
+ Misses       5661     5660       -1
  Partials     2893     2893

Wow, I guess that's not used too often. Thanks @jhouck
At some point we should probably devise a test that actually runs those functions and gets an output, but we can do that separately


create nsswitch.conf file to resolve unknownhost exception issue

In absence of /etc/nsswitch.conf file it uses /etc/hosts (file) or /etc/resolv.conf (dns) to resolve hostname. Using /etc/nsswitch.conf we can define the order in which host name resolution should be done. More details on the following blog.
On looking further into this issue some folks have also reported similar issue in java+alpine combo and here is the suggested fix.

http://arthur-c.github.io/posts/alpine-linux-java-unknownhostexception/
http://stackoverflow.com/questions/32537977/java-net-unknownhostexception-on-docker?answertab=votes#tab-top
https://github.com/gliderlabs/docker-alpine/issues/11#issuecomment-91329401
https://github.com/gliderlabs/docker-alpine/issues/11#issuecomment-106233554

Basically we need to write /etc/nsswitch.conf file with following entry.
hosts: files mdns4_minimal [NOTFOUND=return] dns mdns4

To understand the exact meaning of the above line you can read here
Some more details on nsswitch.conf file is available here.

@bryanlatten : Please take a look at this PR.

👍

@ko-be @bossjones?

👍

👍


feat: Documenting Access Rules on applications and multiple token containers

Description

Adds docs for Access Rules
Updates token container docs to change attribute to an array

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

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

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


Port to futures-locks

src/util.rs stuff can be dropped once https://github.com/asomers/futures-locks/pull/6 gets merged.
requires https://github.com/asomers/futures-locks/issues/5 to be fixed in order to work properly

Coverage decreased (-10.4%) to 41.948% when pulling 04f541c50da291d657a86675189df8fe080eebd6 on locks into 0a78cbc70057182a828b79dfd2a28f6b24b95193 on master.

Now waiting for a new release of futures-locks and we can merge this!
(will probably happen once https://github.com/asomers/futures-locks/pull/9 gets merged)


Appstudio update test-custom-default-branch-qhqm

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-nwtb-on-pull-request-
creating pipelinerun test-simple-build-nwtb-on-pull-request- in namespace build-e2e-yqwm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-imul-on-pull-request-
creating pipelinerun test-simple-build-imul-on-pull-request- in namespace build-e2e-myic-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-annotations-mijr-on-pull-request-
creating pipelinerun test-annotations-mijr-on-pull-request- in namespace build-e2e-yqwm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-custom-default-branch-qhqm-on-pull-request-
creating pipelinerun test-custom-default-branch-qhqm-on-pull-request- in namespace build-e2e-myic-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-uqzm-on-pull-request-
creating pipelinerun test-simple-build-uqzm-on-pull-request- in namespace build-e2e-myic-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-uqzm-on-pull-request-
creating pipelinerun test-simple-build-uqzm-on-pull-request- in namespace build-e2e-yqwm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-simple-build-mcmm-on-pull-request-
creating pipelinerun test-simple-build-mcmm-on-pull-request- in namespace build-e2e-yqwm-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


fix(mpd): Check state before calling playback controls

Hello.
This should fix #500
I also  added checks before play(), pause() and stop() for consistency, if you have any concerns please let me know.

Looks good to me, thanks!


RuntimeError: No available devices on NVIDIA Jetson Xavier (Oak-D)

Hello,
I am trying to interface the Oak-D camera with the NVIDIA Jetson Xavier.
I follow this guide word by word:
https://docs.luxonis.com/projects/api/en/latest/install/#jetson
But in the end I am running into the following error when I run rbg_preview.py:

I cannot even see the name of the device in lsusb (although its ID is there).
However, when I do a dmsg -w, I can see the device.

Can someone help me out?
Sorry about the crappy quality photos, it was easier to take pictures than to transfer screenshots on that computer.

Do you use a USB3 capable cable to connect the OAK-D?
Do you think your Jetson device is providing enough power to the OAK-D?
Can you test and connect the OAK-D to an actively powered USB3-capable USB-Hub?
The device "idVendor=03e7, idProduct=2485" is the MyriadX device.
Are you sure you installed the UDEV-rule?

Hi, I am having the same issue. I have installed the udev rule and I even checked with a powered USB hub. However the device is still not recognised when I check with lsusb.


First commit for event index

For #37

Summary by CodeRabbit

New Features

Introduced event indexing with a new _EVENT_INDEX column in various aggregation and predicate functions.
Enhanced extraction and processing of subtree data with event indexing.

Improvements

Enhanced data consistency by including event index columns in summary outputs.

Tests

Updated end-to-end tests to include new _LAST_EVENT_INDEX fields.

I think more generally here, each aggregate function should return:

The timestamp at the start of the window. If the window does not exist, this should be null.
The timestamp at the end of the window. If the window does not exist, this should be null.
The index of the first event inside the window. If there are no events in the window, this should be null.
The index of the last event inside the window. If there are no events in the window, this should be null.

Note that the distinction between "the window existing" and "events existing" is as follows:

A window is an interval in the patient timeline where (a) the left endpoint is <= the right endpoint and (b) either left or right endpoint can be open or closed. A window does not exist if either of these constraints are not satisfied. We can also modify this to say that "windows don't exist if they are provably empty -- e.g., [12:01, 12:01) of (3:23, 3:23)" or that "windows with equal left and right endpoints where both are open don't exist" as desired.
Events are in the window if they occur with a timestamp within the interval defined by the window (respecting the openness or closedness of its bounds).

@justin13601, as we'll discuss more shortly, I'd actually suggest de-prioritizing this in favor of other things, and closing the PR for now as a result (with plans to re-visit later). Does that sound reasonable to you? If not, feel free to re-open.


Improve font stack with standard font families

Hi! This change extends the current font stack for both sans-serif and serif with new "standard font families". Please see: https://www.w3.org/TR/css-fonts-4/#ui-serif-def
For the default font and font-sans class/variants this won't have much effect for now, since system-ui nad -apple-system are already included. For font-serif it now includes -apple-system-ui-serif and ui-serif, for which support from Safari landed just last month with Safari 13.1.
To summarize the change:

The sans-serif font stack is prepended with ui-sans-serif, which makes it:

ui-sans-serif, system-ui, -apple-system, BlinkMacSystemFont, "Segoe UI", Roboto, "Helvetica Neue", Arial, "Noto Sans", sans-serif, "Apple Color Emoji", "Segoe UI Emoji", "Segoe UI Symbol", "Noto Color Emoji"

The serif font stack is prepended with ui-serif and -apple-system-ui-serif, which makes it:

ui-serif, -apple-system-ui-serif, Georgia, Cambria, "Times New Roman", Times, serif

Thanks and keep up the good work! Tailwind CSS is awesome 🚀

Maybe ui-mono too?

Maybe ui-mono too?

It's actually ui-monospace.
So I've now added ui-monospace to the mono font stack and also decided to drop -apple-system-ui-serif (and not add -apple-system-ui-monospace) since they were never a main part of Safari. So now this PR is just about adding ui-sans-serif, ui-serif and ui-monospace to the respective font-family classes. I'll edit the PR description accordingly.

Merged manually to resolve conflicts, not sure why GitHub doesn't pick it up :/ https://github.com/tailwindlabs/tailwindcss/commit/2c6f237685c2a114f9f1f071f0d9cc1d09be6912
Thanks!


How to get the thumbs path of uploaded image.

I have uploaded image successfully,and can access it.How can i get the path of the thumnail image ?
Do i need manually append "thumb" in file path to retrieve it ? or is any other way workaround for this ?
Like
File path : /uploads/shares/test/1410463802_fcity.png
Thumb : /uploads/shares/test/thumbs/1410463802_fcity.png

@asimshazad Yes, you should append "thumbs" to get thumbnail's path. We can only return one path, original image or thumbnail, at a time. Would you suggest how we can enhance this feature in the future?

Yup,i did it using creating a helper function,and appending a thumb in the image path.
This is what i did.
static function getThumb($path)
   {
       $str_to_insert = '/thumbs';
       $oldstr = $path;
       $pos = strrpos($path,"/");

        return substr_replace($oldstr, $str_to_insert, $pos, 0);
   }

Definitely you people can do this in more efficient way.

is it implemented in version 1.9 and could you provide an example on how to approach this ?

try
$thumb_path =  Str::replaceLast(basename('$image_path'),'','$image_path')
$thumb_path .= 'thumbs'. basename('$image_path')


R-graph method implemented

R-graph
paper:
https://openaccess.thecvf.com/content_cvpr_2017/papers/You_Provable_Self-Representation_Based_CVPR_2017_paper.pdf
All Submissions Basics:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your core changes, as applicable?
[x] Have you successfully ran tests with your changes locally?
[x] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor? (error in another part)
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[x] Have you created a .py in ~/pyod/models/?
[x] Have you created a _example.py in ~/examples/?
[x] Have you created a test_.py in ~/pyod/test/?
[x] Have you lint your code locally prior to submission?

Thanks for checking. If you do not mind, please still finish as complete as possible. We will then read the paper and try to review. Thanks~

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

188 of 240 (78.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 95.559%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/test/test_rgraph.py
80
81
98.77%

pyod/models/rgraph.py
108
159
67.92%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
7575

Relevant Lines:
7927

💛  - Coveralls

Thanks for checking. If you do not mind, please still finish as complete as possible. We will then read the paper and try to review. Thanks~

Everything looks fine now. I believe it ready to be pulled into the master. Thanks!

Dear @yzhao062,
This implementation is also ready to be merged into development. However, a weird error occurs now in the AppVeyor, which was previously not there. I think this error is on the AppVeyor side, and not the implementation (?).

Except for coverage/coveralls, everything seems to work!

Hi, I started reviewing and updating this PR. Just wonder if you could point out the paper for it? I will build that in the documentation :)

Thanks for checking. If you do not mind, please still finish as complete as possible. We will then read the paper and try to review. Thanks~

Good to hear! This is the paper https://openaccess.thecvf.com/content_cvpr_2017/papers/You_Provable_Self-Representation_Based_CVPR_2017_paper.pdf

Thanks for checking. If you do not mind, please still finish as complete as possible. We will then read the paper and try to review. Thanks~

Good to hear! This is the paper https://openaccess.thecvf.com/content_cvpr_2017/papers/You_Provable_Self-Representation_Based_CVPR_2017_paper.pdf

Perfect. I will do it this afternoon and release in 1.0.4 :) thanks a lot!


Update Readme file

Update readme file with information about environment setup and different kind of workflow development on floxx

PR #169


Spine bones being misplaced

Spine version 2.1.18
Pixi Version 2.2.7 and 2.2.8

I'm seeing a similar issue to #1468.  Bones appear to be misplaced, although unlike #1468 this doesn't just occur for animations with a draw order keyframe but for all animations.
You can see from this image that the head and tie are too low, the arms are too close in and the lower leg (below knee) is too high up.  I don't know anything about Spine (I was given the files from my animator), so excuse any incorrect terminology, but it appears that the offset for the bones are using a flipped axis.  I've attempted man.skeleton.flipX = true and the Y axis counterpart but neither of those solved this issue, but did have a noticeable effect.  Everything works correctly in Spine and their skeleton viewer for the exported files.
Code used to create the animation:
var man = new PIXI.Spine('anims/man.anim');
man.skeleton.setSkinByName('Skin Name');
man.skeleton.setSlotsToSetupPose();
man.position.x = window.innerWidth / 2;
man.position.y = window.innerHeight - 200;
man.scale.x = man.scale.y = window.innerWidth / 1200;
man.state.setAnimationByName(0, 'dancing', true);
this.stage.addChild(man);

Unfortunately I'm not in a position where I can share the .spine file or any of the assets, but I would be happy to provide screenshots of anything that may help.
/cc @mreinstein

I've taken a screenshot of the man from spine.  I've been asked to obscure the face, for some reason the PM is a little careful about putting anything on the internet which could expose the project.

instead of setSlotsToSetupPose() try setToSetupPose() and see if that fixes it for you.

I've found this document to be incredibly useful on some occasions: http://esotericsoftware.com/spine-using-runtimes

Thanks for the suggestion, but no luck there.  This exact code works fine for the sample spine animations that are found in examples (sans the set skin line which isn't needed for those), so there's an issue with my Spine project.  Either the way its exported or the way its actually created.  But I'm hoping that those animation don't have to be re-exported or even completely remade, and that a line or two of JS will solve the problem.
If there are any pointers about the specifics of where in the Pixi file I should investigate then I'll take a look at that.  I'm not fully sure whether this is the bones, slots, attachments or something else which is being incorrectly placed.

so there's an issue with my Spine project

yeah one thing I noticed about your bones (which I didn't mention because you said it animated fine in the spine editor) is that they dont seem to have a traditional hierarchy. Looking at your image. It seems that the upper leg and lower leg bones descend from the knee joint. this is atypical I think. Notice that in my example, the lower leg bone is a child of upper leg, and upper leg is a child of the hip.

Thanks, if you don't mind I'll quote that to the animator and see if I can get something done about it.
I'll keep this open as it may still be a bug with Pixi if it cannot manage this hierarchy?  I appreciate what I've got is probably incorrect, but an error message or some documentation on this would be sweet if it cannot be resolved.

it may still be a bug with Pixi if it cannot manage this hierarchy?

It's really hard to say what the problem is with the given information. without the assets or a video or more info I can't really say what the issue is. I suspect it is with your animation structure but can't be sure.

but an error message or some documentation on this would be sweet if it cannot be resolved

It's not clear to me what the problem is though. So I'm not sure what error message would be displayed. are you saying something like "detect when a non-humanoid skeleton is used and alert the user" ? Humanoids are one type of an infinite variety of animations that you could produce with spine, everything from dragons to turrets to powerups to more exotic things I can't even think of.

Humanoids are one type of an infinite variety of animations that you could produce with spine, everything from dragons to turrets to powerups to more exotic things I can't even think of.

That's true, however even if this was a non-humanoid skeleton, surely this is still an issue?  I would expect everything to still be in the correct placement if this were a turret or any other object.  Am I missing something?
I appreciate your help and I understand it's difficult when I'm not able to share my assets with you.

I've managed to learn enough about Spine myself to create a quick stickman test animation and that works with no issues.  I'll get my animator to view what I created and to amend his hierarchy to match.
Nonetheless, it's still odd that my original animation viewed fine in Spine and the Skeleton Viewer but not Pixi.  To me this seems like a bug, but I could easily be wrong.

To me this seems like a bug, but I could easily be wrong.

It may be, but it's not actionable in terms of a fix. I'd recommend closing the issue, and re-open if it turns out to be something specific. Just my 2 cents.
good luck!

Thanks for all your help!
  I couldn't see anything on your profile page about it.  If not, then I'll keep this open for a dev to take a peek at and they are more than welcome to close it off if this is not specific enough or is not an issue.

It's definitely not offensive. :) No, I'm not a contributor, I'm a new user to it like you. I have a desire to contribute though, and one way to do that is to help with easing the support burden (trying to answer github issues, etc.)

If not, then I'll keep this open for a dev to take a peek at

You're certainly welcome to do that. I don't mean to imply any authority in telling you my thoughts on the problem. I suspect that this particular issue as it is currently described is not very actionable, and hard to solve. but that's just my gut feeling. My hope was that I could help you with this issue myself and save them a few cycles since they are very busy releasing v3 stuff.

You're awesome.  People like you really help the community.
I'll close this off to keep the burden low on the devs.  With any luck they'll still see it and reopen if it is an actionable bug.  There is a workaround so it's a minor issue at best, especially as nobody else seems to have this issue
Thanks again, and I wish you the best with everything.


Fix row group dictionary filter handling of null values.

This is a fix for PARQUET-1510 ported to Iceberg's implementation of dictionary filtering.

Thanks for reviewing, @fbocse!


/miniplaceholders parse can be executed without permission

Permission check only exists in /miniplaceholders with no arguments, /miniplaceholders parse and /miniplaceholders help both can be used by all players.

On which platform are you testing it? If it is on Krypton, it currently has a problem with the Pointers system

Ok, I saw the problem, although I'm sure I fixed it a long time ago... Maybe it was in another branch and I didn't notice it

In effect, it was in another branch https://github.com/MiniPlaceholders/MiniPlaceholders/commit/39c7455b5385c2d49dacb461b633745f7fadc550 apologies for this bug, now I release a new release fixing it


Remove hard-coding of x and y dimensions from RunModule.f90

https://github.com/NOAA-OWP/noah-owp-modular/blob/8161eab9a3f8f99694ec062a92cd3b21da7e806d/src/RunModule.f90#L87
For testing, we hard-coded these values, but this will get superseded by NetCDF read functionalities in the gridded driver.

PR #85 removed hard-coding of gridded spatial dimensions. They are now read-into the model via the NetCDF input file.


Custom timestamp format

Hello!
I'm using your plugin with the Audio plugin:
https://github.com/noonesimg/obsidian-audio-player
And they complement each other amazingly! The only problem is that the transcription from AI model are using diffrent timestamp format than the audio-player plugin is using. It will be beneficial for all kind of other plugin if user can specify the timestamp format.
For example the obsidian-audio-player needs format:
[[my awesome audio file.mp3]]
00:00:44 --- chapter
00:01:50 --- chapter 2
00:02:40 --- chapter 3 in which nothing happened

Added in the latest dev build - also now works on any device, - contact me at<EMAIL_ADDRESS>or sign up at https://scribe.gambitengine.com/ for early access!


Outlike

👇

I'm afraid I don't know what you mean by "Outlike", so I'm closing this issue. Please feel free to reopen and add more details.

Какой code intra
ср, 2 сент. 2020 г., 17:58 Justin Henck<EMAIL_ADDRESS>
I'm afraid I don't know what you mean by "Outlike", so I'm closing this
issue. Please feel free to reopen and add more details.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Jigsaw-Code/Intra/issues/279#issuecomment-685718295,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQZPKIFENPM3AEF2HQGWAZLSDY6QPANCNFSM4QFWUV5Q
.


add k3s/k0s support for the Kubernetes recipe

Updated the install logic to include a check for k3s and k0s clusters.  Previously, this was failing for users even though Pixie supports both versions.

Closing in favor of https://github.com/newrelic/open-install-library/pull/741


Fix grammar errors, improve documentation

Sorry for the flood of pull requests. I resolved a couple of spelling errors in the more recent changes, fixed a couple errors I introduced in my earlier changes to the documentation, and improved the overall wording of the documentation files.

I appreciate the attention to detail and helping out with this. Other than my two comments this is looking better and better. What you think about my comments?

Can the npm run doc and npm run coverage be integrated into a travis build?
I had a really great reply all typed up, but I accidentally clicked off the page and it disappeared. :(
The gist of it was that I'd prefer we err on the side of giving discretion to contributors and reviewers. Rather than proscribe a precise two-commit pull request flow, offer general guidelines for what is expected and what is suggested. So we require test coverage and passing tests, and we suggest updates to the sample RAML and documentation. Additional guidelines or restrictions may arise naturally as we see how contributions work in practice.

Something like this is what I'm referring to with travis: https://github.com/steveklabnik/automatically_update_github_pages_with_travis_example

Travis, or something like it, is going to happen. I just haven't had time to dedicate to it.
What do you think about my comments to this PR?

I agree with them as far as they go, however, like I said in my 9:24 comment, I would prefer a different approach to what we expect from contributions.
This might be something to discuss internally, so we can establish guidelines for all the stuff QL open sources? Might make more sense than a patchwork of CONTRIBUTING.md files across our organization.

pinging @kalisjoshua -- updated per our hipchat discussions.

Coverage remained the same at 100.0% when pulling 7c32c12feb6f20edddff0e2580b0f1db391b7bb8 on jdpedrie:doc-updates into 6297211f2ea28ce2497377e2a53032d33f385e40 on QuickenLoans:master.

Coverage remained the same at 100.0% when pulling e4197a8212124b47825f4422d5d4c3a5f73033d5 on jdpedrie:doc-updates into 6297211f2ea28ce2497377e2a53032d33f385e40 on QuickenLoans:master.


disabling parallel

Before creating a pull request make sure that:

[ ] commit messages are meaningful and follow good commit message guidelines
[ ] README and other documentation has been updated / added (if needed)
[ ] tests have been updated / new tests has been added (if needed)

Please remove this line and everything above and fill the following sections:
JIRA link (if applicable)
Change description
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

Builds seem to be working now with increased memory, so disabling parallel tests not currently required.


Add aarch64 NEON 8-bit SATD assembly for 4x4-transformed block sizes

This PR only adds assembly for block sizes that use a 4x4 transform for now.

I only have access to my M1 for the time being, so my other PR for adding avx2 hbd satd assembly will be a bit delayed

Coverage increased (+0.2%) to 85.327% when pulling 8f41258f337431b2423d7cf5e16a7e5cfcf18bbb on redzic:aarch64-satd into 9c4b2972a3bbd4e44cc87b1d4ed2535567ad0e03 on xiph:master.

I'll try to do actual benchmarking later, but some initial testing suggests around a ~2-3% overall reduction in encoding time (tested on speed 2, 4 tiles, 4:2:0 1080p on an M1 pro). This only affects speed <= 2 in practice, I guess because faster presets don't search small block sizes.
Hopefully once I add assembly for 8x8 transform-based blocks in another PR, the speedup will be higher.

Codecov Report

Merging #2972 (345e520) into master (c402d7a) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2972   +/-   ##
=======================================
  Coverage   86.54%   86.55%
=======================================
  Files          87       87
  Lines       33886    33886
=======================================
+ Hits        29328    29329    +1
+ Misses       4558     4557    -1

Impacted Files
Coverage Δ

src/dist.rs
98.94% <ø> (ø)

src/asm/x86/lrf.rs
93.48% <0.00%> (-0.64%)
:arrow_down:

src/me.rs
95.10% <0.00%> (-0.49%)
:arrow_down:

v_frame/src/plane.rs
88.78% <0.00%> (-0.12%)
:arrow_down:

src/encoder.rs
87.41% <0.00%> (+0.03%)
:arrow_up:

src/deblock.rs
96.03% <0.00%> (+0.14%)
:arrow_up:

src/partition.rs
77.74% <0.00%> (+0.15%)
:arrow_up:

src/context/transform_unit.rs
90.49% <0.00%> (+0.71%)
:arrow_up:

src/context/mod.rs
92.63% <0.00%> (+1.05%)
:arrow_up:

src/tiling/tile_state.rs
89.85% <0.00%> (+1.44%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

The tests were failing because I used src_stride for dst in some places, and the tests actually tested when the 2 planes had different strides. Fixed now.
@barrbrain do you know what would be required to fix compilation with gcc? and is there a way to test gcc compilation on macos?

@barrbrain do you know what would be required to fix compilation with gcc? and is there a way to test gcc compilation on macos?

I found that I had to set CFLAGS=-m64 with GCC as the default compiler in my environment.


dataset

Can I only use imagenet to train the generator? The training set cannot be downloaded and used as it reaches 138G.

Hi @tanlingp , BIA is trained by ImageNet. What does "Can I only use imagenet to train the generator?" mean?

Can I use other datasets to train BIA?

Of course. You can use any dataset to train BIA. The case of modifying the source domain is also discussed in our paper Appendix A.5


web page not showing up

Environment: raspberry pi 4 model b, latest openwrt pull from git
Description:
Hi,
I followed the instructions, did a clean compile and it installed on my raspberry pi. But when I go to the url, it is a blank page. Nothing is showing.. Any help?
thanks

after some more investigating and keeping in mind that vuejs is new to me, I think the problem occurs when it cannot find the language (locale) setting...is the locale set in openwrt for OUI? Anyone paste an a sample of their config file where lang or locale is defined so get_lang function can work properly?
thanks

after some more investigating and keeping in mind that vuejs is new to me, I think the problem occurs when it cannot find the language (locale) setting...is the locale set in openwrt for OUI? Anyone paste an a sample of their config file where lang or locale is defined so get_lang function can work properly?
thanks

ok, I figured it out...I compiled and deployed as normal. I then went into /etc/config/oui file and change the option lang from auto to en....rebooted and it worked perfectly! Hopefully this will help others.

ok, I figured it out...I compiled and deployed as normal. I then went into /etc/config/oui file and change the option lang from auto to en....rebooted and it worked perfectly! Hopefully this will help others.

ok, I figured it out...I compiled and deployed as normal. I then went into /etc/config/oui file and change the option lang from auto to en....rebooted and it worked perfectly! Hopefully this will help others.

can you explain creating files and running the theme

ok, I figured it out...I compiled and deployed as normal. I then went into /etc/config/oui file and change the option lang from auto to en....rebooted and it worked perfectly! Hopefully this will help others.

can you explain creating files and running the theme


Provide a vision for Pathfinder's purpose and use

As a Pathfinder developer, I want to understand what Pathfinder's purpose and field use are in order to make better decisions about Pathfinder architecture, design, and implementation.

@texasbrok can this be closed

@noelo yes.  see https://github.com/redhat-cop/pathfinder/blob/master/legacy/pathfinder-vision.rst


Is there any way to blur the whole webpage, while user moves focus away ?

Is your feature request related to a problem? Please describe.
We're trying to make the chat webpage screenshot proof, It is hard but exploring options.
Describe the solution you'd like
We can use javascript, which blurs the page when user moves focus. and unblur when focused on page.

I can work on it!!

Could you explain approach first ?

How will you do it?
Which pages will be affected?
Will other components be affected negatively ?

so I plan to add js and use the visibility change event to add a class to the webpage (we can add this class directly to body or specifically to the main class where all chats are). I can Disable the Print Screen (Not Foolproof) functionality for screenshots, which will be done by adding some CSS only.
overall base.html will be edited and there will be no negative affects.

visibilitychange works, when another app, other than browser is opened?
I checked, it works while another tab is opened, but not sure about the another app.

there are other events such as "blur", and "focus". we can use if the chat screen is not in focus we can blur the whole chat screen and if it is in focus we will remove blur
implemented overview-
https://github.com/princekhunt/privateping/assets/60603199/8ac5ba22-2676-4874-b105-588b3738cbcb

That's exactly what's required.

and here I disable the print screen functionality.
https://github.com/princekhunt/privateping/assets/60603199/e0872cca-4891-4667-97a2-dcc357cf6588
if every thing is fine for you I can create pull request right now!

and here I disable the print screen functionality.
Recording.2024-06-15.000008.mp4
if every thing is fine for you I can create pull request right now!

at 0:09 in this video, toast is also shown and message in back as well, maybe blur event didn't collaborate properly.

let me check and fix it!

done by my side
https://github.com/princekhunt/privateping/assets/60603199/c9b50de4-929d-4555-88c9-f1c9a7368234
should I create a pull request?

Okay, create PR.
let me test from my end.

Done!

Will you merge it so that i will be open to create pull requests for other assigned issues.

I think you forgot to make PR.
you pushed changes to your repo.
kindly create the pr.


adds obvious explanation of concept tagging

What is Concept Tagging
By concept tagging, we mean you can supply text, for example:
Volcanic activity, or volcanism, has played a significant role in the geologic evolution of Mars.[2] Scientists have known since the Mariner 9 mission in 1972 that volcanic features cover large portions of the Martian surface. and get back predicted keywords, like volcanology, mars surface, and structural properties, as well as topics, like space sciences, geosciences, from a standardized list of several thousand NASA concepts with a probability score for each prediction.

yup, good idea


WIP: New protocol implementation

Rebased changes to merge conflicting travis file with master.
Added 4 commits, after the last review.
@jodh-intel @dlespiau @sboeuf  Can you review the last 4 commits, so that this can be merged?

@jodh-intel I have addresses your review comments. Please have a look.

@dlespiau @sboeuf @sameo @sameo Can we have this merged?

LGTM

LGTM


"wtr" attribute redundant

The wtr attribute in the sync-protection-spec grouping seems to be a duplicate of the wait-to-revert-time attribute of the control-parameters container the sync-protection-spec is attached to.

Proposal to the 5G-xhaul call on 7th of September 2022:
There is a waitToRevertTime attribute in the ControlParameters_Pac class of the Core IM.
The ControlParameters_Pac class gets augmented by the SyncProtectionSpec class of the Synchronization model.
The ControlParameters_Pac  contains a wtr attribute.
The wtr attribute is redundant to the already existing waitToRevertTime attribute.
The wtr attribute shall be deleted from the modelling.


Upgrade to Cascading 2.7

Cascading 2.7 was released along side with 3.0 some time ago.
It's worth investigating what it takes to upgrade (especially for 3.0).

With the decision to no longer support Cascading after 7.0, I'm going to go ahead and close this out.
Related #1222


Cannot find dataset

Hello, fristly ,Thank you very much for the code you provided.Now,i try to run your code ,but it  doesn't work , the problem is OSError: outputMultipleRouting_5.txt not found
 I really can't find this file.

Hi,
Please, try to specify this "fileData" parameter with a file from the
provided datasets.
Best regards,
Albert
On Fri, May 3, 2019 at 4:20 AM yzf123456<EMAIL_ADDRESS>wrote:

Hello, fristly ,Thank you very much for the code you provided.Now,i try to
run your code ,but it doesn't work , the problem is OSError:
outputMultipleRouting_5.txt not found
[image: image]
https://user-images.githubusercontent.com/43875437/57117208-f7197300-6d8c-11e9-809a-e045e0fd1c71.png
I really can't find this file.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/knowledgedefinednetworking/-knowledge-defined-networking/issues/1,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARBLLYJGULOT7RWVUXOK63PTOOO7ANCNFSM4HKPSG4Q
.

OK，thanks for your help!
Best wish!

Hello, I see there are 2 inputs:
FILE=dataLong_2.txt in run.sh
fileName = 'outputMultipleRouting_5.txt' in sysTest.py.
I couldn't find these 2 data in your provided dataset. And I randomly use datasets in your provided datasets, there is an error:
data = data[0:samples,0:456]
IndexError: too many indices for array
May you recommend datasets to use in these two input?


Fix empty object params bug

Usage related changes
Closes https://github.com/0xSpaceShard/starknet-devnet-rs/issues/345
Development related changes

Checklist:

[x] Applied formatting - ./scripts/format.sh
[x] No linter errors - ./scripts/clippy_check.sh
[x] No unused dependencies - ./scripts/check_unused_deps.sh
[x] Performed code self-review
[x] Rebased to the last commit of the target branch (or merged it into my branch)
[ ] Documented the changes
[x] Linked the issues which this PR resolves
[ ] Checked the TODO section in README.md if this PR resolves it
[x] Updated the tests
[x] All tests are passing - cargo test

@marioiordanov

circle ci will fail on ./scripts/format.sh

Fixed now


Tables won't match

I have the following table
Invoice Date:	Due Date:	Source:	Reference:
07/17/2019	08/16/2019	SO1972	INV/2019/1189/25

yaml file:
- start: Invoice Date:+\s+Due Date:+\s+Source:\s+Reference:
  end: Description
  body: (?P<invoice_date>\d\d\/\d\d\/\d\d\d\d)\s+(?P<due_date>\d\d\/\d\d\/\d\d\d\d)\s+(?P<source>[A-Z]{2}\d\d\d\d)\s+(?P<reference>INV\/\d\d\d\d\/\d\d\d\d\/\d\d)

The online regex tester matches all the data but I get the following error:
no table body found - start None, end <_sre.SRE_Match object at 0x7f9299e1c6b0>
Thanks for any suggestions!

Upon further inspection it seems that it sticks all the fields of each row together so I don't quite understand how to seperate out the different values then.

Solved, had to disable the remove_whitespace option.
Would be good to add that to the tutorial that this option needs to be disabled in order to be able to use the table plugin

This is not specific to the tables plugin, just the behavior of the remove_whitespace option. That option removes whitespace before running your regex, so your regex needs to expect that. You could have changed your body regex to remove the \s+ tokens to get the matches you want.

True, I guess it just confused me since I missed the option. Case closed.


878: Fixed issue with ThemeTokens typing

Simple fix to address the typing issue in #878; the offending type could be of types beyond string | number, so I added a type guard.
CC: @dlehmhus

hey, thanks! looks fine at a glance, @hadihallak can you double check? cheers!

@peduarte @hadihallak hi! any chance this could be released, please? Running into the above issue. Thanks!

@peduarte @hadihallak hi! any chance this could be released, please? Running into the above issue. Thanks!

@gmlnchv: until it's released, you should be able to use it in builds by putting it in your dependencies like so:
"@stitches/react": "https://gitpkg.now.sh/withneutron/stitches/packages/react?release/749_Temp-release",

That also includes a fix for #749, as I happen to be running into both of those, with my setup. That's working like a charm for me, at least locally (haven't tried deploying it yet).

@LucasUnplugged perfect, thanks! Was just trying to figure out how to install it from you branch, much appreciated!

Hey @LucasUnplugged
From a first glance, this looks good to me but i will pull this branch and do some tests early next week then merge it if all goes well because i'm always very careful when it comes to TS as types can cascade and have side-effects on each others in weird ways

Hey, we're running into this same issue as well.

Hey @LucasUnplugged
Hope you had wonderful holidays
I pulled your branch and tested locally and it worked great! left a note regarding a simpler fix for the bug and i would appreciate if you could give it a look.

@hadihallak I've pushed the update, and also rebased my other PR (#900), to pick up these changes as well. Thanks for your feedback!

@LucasUnplugged Thanks for the PR and addressing the feedback. will be merging this now.
Will take a look at the other PR ASAP.

Has this error been officially resolved?

@peduarte  @hadihallak oi! alguma chance disso ser liberado, por favor? Correndo para o problema acima. Obrigado!

@gmlnchv : até que seja lançado, você deve poder usá-lo em compilações, colocando-o no seu dependenciesgosto assim:
"@stitches/react": "https://gitpkg.now.sh/withneutron/stitches/packages/react?release/749_Temp-release",

Isso também inclui uma correção para #749 , já que estou correndo em ambos, com minha configuração. Isso está funcionando como um encanto para mim, pelo menos localmente (ainda não tentei implantá-lo).

This solved my problem

Thankss!

@vinimedeiros13 Thanks for the ping. yes this is fixed but not yet released. Will release next Monday.

Thankss


Spacy model reload issue

When fixed piaget issue, a new issue come out:

direct dependency in install_requirements is a relatively new feature. What version of python and pip are you using? I couldn't really find good documentation on this but

for pip, could you try python -m pip install --upgrade pip and see if that fixes the issue?
for python, I used python3.8 and python3.9. I think it works with both of them.

Ohhhh, I see what you mean. CI is failing because of this. I'll take a look. Do you have any issue installing via pip install . locally?

"Do you have any issue installing via pip install . locally?"
I haven't test it locally, I will once I have the time.

Hi, I actually can install it locally.


feat(IsolatedTestRunner): Handle promise rejections

Handle unhandled promise rejection warnings by handling the unhandledRejection and rejectionHandled events on process. This way, the following messages will not show up in your error log anymore:

UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: ${unhandledPromiseId}): ${reason}
PromiseRejectionHandledWarning: Promise rejection was handled asynchronously (rejection id: ${unhandledPromiseId})

Note: They will still show up when using logLevel: 'trace'
Fixes #350

Looks good!


[User Management] [RC based ] - Test

User self-registration |

User self-registration via MyAccount
User self-registration via REST APIs
User self-registration via business Application
User manage his own user account | Profile Management
Account Lock
Account Deletion via UI

Recover user account |

Recover username
Recover password


Red users have double available RAM (and slightly better CPU) than non-Red users

So, recently I discovered (via the help of @pochmann) that my own solution for Mxyzinjin's Stream times out pretty badly when run on non-Red users only, while Red users gets a consistent ~5s pass. Recalling that I've read about people trying to determine CPU model and total physical RAM of various OJ platforms not long ago, I tried to apply the same method on CW (runs on C++ language):
CPU model from cpuid, taken from https://zhuanlan.zhihu.com/p/28322626:
#include <stdint.h>
#include <iostream>
#include <cpuid.h>
static void cpuid(uint32_t func, uint32_t sub, uint32_t data[4]) {
    __cpuid_count(func, sub, data[0], data[1], data[2], data[3]);
}
void print_cpu_model() {
    uint32_t data[4];
    char str[48];
    for(int i = 0; i < 3; ++i) {
        cpuid(0x80000002 + i, 0, data);
        for(int j = 0; j < 4; ++j)
            reinterpret_cast<uint32_t*>(str)[i * 4 + j] = data[j];
    }
    std::cout << str;
}

Total physical RAM, code taken from http://nadeausoftware.com/articles/2012/09/c_c_tip_how_get_physical_memory_size_system:
/*
 * Author:  David Robert Nadeau
 * Site:    http://NadeauSoftware.com/
 * License: Creative Commons Attribution 3.0 Unported License
 *          http://creativecommons.org/licenses/by/3.0/deed.en_US
 */

#if defined(_WIN32)
#include <Windows.h>

#elif defined(__unix__) || defined(__unix) || defined(unix) || (defined(__APPLE__) && defined(__MACH__))
#include <unistd.h>
#include <sys/types.h>
#include <sys/param.h>
#if defined(BSD)
#include <sys/sysctl.h>
#endif

#else
#error "Unable to define getMemorySize( ) for an unknown OS."
#endif

/**
 * Returns the size of physical memory (RAM) in bytes.
 */
size_t getMemorySize( )
{
#if defined(_WIN32) && (defined(__CYGWIN__) || defined(__CYGWIN32__))
	/* Cygwin under Windows. ------------------------------------ */
	/* New 64-bit MEMORYSTATUSEX isn't available.  Use old 32.bit */
	MEMORYSTATUS status;
	status.dwLength = sizeof(status);
	GlobalMemoryStatus( &status );
	return (size_t)status.dwTotalPhys;

#elif defined(_WIN32)
	/* Windows. ------------------------------------------------- */
	/* Use new 64-bit MEMORYSTATUSEX, not old 32-bit MEMORYSTATUS */
	MEMORYSTATUSEX status;
	status.dwLength = sizeof(status);
	GlobalMemoryStatusEx( &status );
	return (size_t)status.ullTotalPhys;

#elif defined(__unix__) || defined(__unix) || defined(unix) || (defined(__APPLE__) && defined(__MACH__))
	/* UNIX variants. ------------------------------------------- */
	/* Prefer sysctl() over sysconf() except sysctl() HW_REALMEM and HW_PHYSMEM */

#if defined(CTL_HW) && (defined(HW_MEMSIZE) || defined(HW_PHYSMEM64))
	int mib[2];
	mib[0] = CTL_HW;
#if defined(HW_MEMSIZE)
	mib[1] = HW_MEMSIZE;            /* OSX. --------------------- */
#elif defined(HW_PHYSMEM64)
	mib[1] = HW_PHYSMEM64;          /* NetBSD, OpenBSD. --------- */
#endif
	int64_t size = 0;               /* 64-bit */
	size_t len = sizeof( size );
	if ( sysctl( mib, 2, &size, &len, NULL, 0 ) == 0 )
		return (size_t)size;
	return 0L;			/* Failed? */

#elif defined(_SC_AIX_REALMEM)
	/* AIX. ----------------------------------------------------- */
	return (size_t)sysconf( _SC_AIX_REALMEM ) * (size_t)1024L;

#elif defined(_SC_PHYS_PAGES) && defined(_SC_PAGESIZE)
	/* FreeBSD, Linux, OpenBSD, and Solaris. -------------------- */
	return (size_t)sysconf( _SC_PHYS_PAGES ) *
		(size_t)sysconf( _SC_PAGESIZE );

#elif defined(_SC_PHYS_PAGES) && defined(_SC_PAGE_SIZE)
	/* Legacy. -------------------------------------------------- */
	return (size_t)sysconf( _SC_PHYS_PAGES ) *
		(size_t)sysconf( _SC_PAGE_SIZE );

#elif defined(CTL_HW) && (defined(HW_PHYSMEM) || defined(HW_REALMEM))
	/* DragonFly BSD, FreeBSD, NetBSD, OpenBSD, and OSX. -------- */
	int mib[2];
	mib[0] = CTL_HW;
#if defined(HW_REALMEM)
	mib[1] = HW_REALMEM;		/* FreeBSD. ----------------- */
#elif defined(HW_PYSMEM)
	mib[1] = HW_PHYSMEM;		/* Others. ------------------ */
#endif
	unsigned int size = 0;		/* 32-bit */
	size_t len = sizeof( size );
	if ( sysctl( mib, 2, &size, &len, NULL, 0 ) == 0 )
		return (size_t)size;
	return 0L;			/* Failed? */
#endif /* sysctl and sysconf variants */

#else
	return 0L;			/* Unknown OS. */
#endif
}

#include <iostream>
void print_memory_size() {
    size_t memorySize = getMemorySize();
    std::cout << memorySize << '\n' << (memorySize/1048576) << "MB";
}

And what I got is pretty surprising:
CPU model:

Non-Red users: Intel(R) Xeon(R) CPU E5-2676 v3 @ 2.40GHz
Red users: Intel(R) Xeon(R) CPU E5-2676 v3 @ 2.40GHz or Intel(R) Xeon(R) CPU E5-2670 v2 @ 2.50GHz  (it seems that both appears 50% of the time)

Total physical RAM:

Non-Red users:<PHONE_NUMBER> or 1996MB
Red users:<PHONE_NUMBER> or 3948MB

I wouldn't say it'd be a noticable advantage for Red users to have a slighter better CPU (I assume it's so), but double availabe RAM? This is giving Red users a huge edge over non-Red users. And as the kata mentioned above has shown, once you're in certain very memory-heavy katas, this will matter a lot.

Red users have 2 vCPU and 4 GB Mem, non-Red users have 1 vCPU and 2 GB Mem.
(I think the subscription page lists this as one of the features.)
Note that these are the numbers of the host VM and not equivalent to the amount of resources allowed in containers.
By the way, if you just want general information, you can simply do
puts `cat /proc/cpuinfo`
puts `cat /proc/meminfo`

This is one of the reasons I refactored the entire service and rebuilt the infrastructure.
With previous design, there's no way to make much resources available to non-Red users because of the cost. Even Red users don't have dedicated CPU cores. They're shared and throttled depending on the usage history of the instances.
And those numbers alone doesn't mean much because there're many other containers running at the same time that shares resources. The amount of available memory varies significantly because of the way the system is set up.
I don't know the history since I joined this year, I'm guessing that as the number of supported language increased and more advanced Kata authored, this became a bigger problem.
The new system is much more cost effective and improves this situation. I can't guarantee the exact number at the moment because I'll be tuning it, but it currently runs on VMs with 4 dedicated cores (cat /proc/cpuinfo on preview should print 4 cores). The VM is dedicated to running submitted code.

Closing this because this is not an issue. Feel free to keep posting though.
With the new runner, both non-Red and Red users' code are executed on machines with same amount of resources. The CPU model can vary depending on the data center location, but there shouldn't be any significant performance differences.


Create README

README filr for repo

Sorry! but it doesn't match the requirements of the issue. You need to make a beautiful Readme including Stack, libraries , the structure of program, etc.


Replace reducer incorrect store state

Description
Case: replace reducer when specific props changes.
Case description:

Parent component pass props to child
Child attach reducer (with props from parent) into store in useLayoutEffect

What is the current behavior?
When child receives a new prop from parent a new reducer attached to store but store has previous state.
Steps to Reproduce
I've created example to demonstrate behaviour (https://codesandbox.io/s/priceless-rgb-c9hdw8).
What is the expected behavior?
In this case I expect to see a new value rendered on screen when user changes value in dropdown.

I'm not sure I'm following what the expected behavior is here.  Could you give me a step-by-step bullet point list of what should happen and what you're expecting to see?

Yep, of course. If speak precisely behaviour of this example is used in application builder. So there is a settings layer (in our case Wrapper) that provides some props to underlying components (result components of the page) and each of these underlying components are connected to redux store. All props from parent components are settings for underlying components and should be stored in redux store.
This case should work as follows:

Child components (connected to store) receives props from redux store;
Parent component has ability to pass some settings (props) to child
These passed props should update props in the store for child compoent (but to do so we need to replace existing reducer with props from parent component)
Child component re-renders when props from store changes.

Can't you return the value that you're receiving as prop instead of pass the useSelector? 🤔
You might not have your state up to date when you're trying to get that value, since it might be asynchronous. So, you're updating your state but you're returning the old state value because it didn't finish to update the state when you first try to return that value.

I'm poking at the CSB and adding some logging to the useSelector call.  I can see the selector running and eventually returning a meaningful value from state[name], so I'm not quite sure what the mismatch is (as in, I'm still not sure I see the actual problem being discussed).
If there is a problem here, it's most likely on the React-Redux side, and  if there's a problem it's probably a timing question.
I'm going to move this over to the React-Redux repo.  Can you expand the example to show more specifically where and how you are expecting the component to re-render?


SSR code is not importable in ESM

Version
3.2.16
Reproduction link
codesandbox.io
Steps to reproduce
This code is generating an import that does not work in ESM:
https://github.com/vuejs/vue-next/blob/87c86e4cc29ce3d09c11f27c7ba0eb13e7353b6c/packages/compiler-core/src/codegen.ts#L422-L428
The error is:
Error [ERR_UNSUPPORTED_DIR_IMPORT]: Directory import '/project/node_modules/vue/server-renderer' is not supported resolving ES modules imported from /project/dist/server/server.mjs
Did you mean to import vue/server-renderer/index.js?

What is expected?
The generated code should be:
import { ssrRenderComponent, ssrRenderSuspense, ssrRenderVNode, ssrRenderAttrs, ssrRenderAttr } from "vue/server-renderer/index.mjs"

What is actually happening?
The generated code is:
import { ssrRenderComponent, ssrRenderSuspense, ssrRenderVNode, ssrRenderAttrs, ssrRenderAttr } from "vue/server-renderer"

Ok... this actually has a bigger implication than it seems, because without adding an exports field to vue there is no way to retain the same vue/server-renderer import path for both CJS and ESM. And we need to do that because Vite currently converts the ESM-based SSR code to CJS in its runtime.

For now you can pass this option to change the import location

Fixed in 3.2.18


Problem: every block ingestion calls state.spawn_secondary_database()

This leads to an unbounded number of uuid droppings in the database directory.
It should be sufficient to inject one secondary database to the IPCActor instead on each block ingested.

Will we try_catchup_with_primary()? on the secondary when the IpcActor handles queries?

Will we try_catchup_with_primary()? on the secondary when the IpcActor handles queries?

Yes, since we want the latest value available in the store

@jhult Interested in this one?

This method has since been removed and by definition no longer an issue.


Compile source to the library other than 'work'

Hi,
If I have f.e. list of mixed VHDL/Verilog sources and I want to compile them in different libraries - how can I set this in Cocotb?
Now I work through internal simulators makefile changes but it's a quite ugly solution.

You can compile the libraries and link them by adding to simulator arguments.

Can you elaborate? Does it mean manually stating the vcom commands?
Thanks

For Questa one can add VSIM_ARG=-l lib_name into Makefile where lib_name is pre-compiled library.


Json formatter changes the datetime to local datetime

Description
"Date": "2012-04-21T18:25:43-05:00" will be "Date": "2012-04-22T01:25:43+02:00" here in Hungary.
I think, a formatter tool should not change any content.

Steps To Reproduce
No response
Expected behavior

Go to Json formatter

Enter to input:
{ "Date": "2012-04-21T18:25:43-05:00" }

The output in Hugary (UTC+2):
{ "Date": "2012-04-22T01:25:43+02:00" }

DevToys Version
<IP_ADDRESS> verzió | X64 | RELEASE | db9b428 | db9b428
Relevant Assets/Logs
No logs for this

Hello,
Thanks for the feedback. I fixed the issue and opened a pull request for it. :)


Fix broken error handling around Idempotent producer + Ensure strict ordering when Net.MaxOpenRequests = 1

There have been numerous issues filed lately that:

Strict ordering no longer works after request pipelining was introduced even when Net.MaxOpenRequests is set to 1.
Requests can still end up failing / mis-sequenced when Idempotent producer is enabled.

I was able to reproduce these issues both locally and in a staging environment and have fixed both of them. P.R has many comments explaining the changes.
Relevant issues:

#2619
#2803
#2860

Hmm... I think it may still be possible for concurrent requests to be issued sendResponse writes to a channel, and once the response is picked up off that channel there could be an inflight request from the next request + an inflight retry actually

any progress on this ？I think this bug is very urgent to fix

As we are seeing also a lot of kafka server: The broker received an out of order sequence number errors, we would be interessted in this fix.

I guess @richardartoul has not answered to @puellanivis review comments yet?
@richardartoul : Do you think you can continue with this PR, os is this PR abandoned?

We are waiting for this MR to be continued

Hey guys, I'm really sorry but I wasn't able to convince myself this P.R was correct or materially improved the situation. There is definitely a bug in the library, but I'm suspicious there is more than one issue and I timeboxed myself to two days on this and don't have more time to continue trying to fix it


Where do "intermediary" contributions and their earmarked metadata surface in the database?

We can see them visible in filings, like Charles Munger's major donor sheets, but can we find them all in the database http://cal-access.ss.ca.gov/PDFGen/pdfgen.prg?filingid=1980975&amendid=0

Looks like we can!
select *
from "TEXT_MEMO_CD"
where "TEXT4000" like '%MADE THROUGH INTERMEDIARY CALIFORNIA REPUBLICAN LEADERSHIP FUND (#1340317); 455 CAPITOL MALL, SUITE 600, SACRAMENTO,%'
and "FILING_ID" = 1980975;

Interesting thing. There's probably some way to ultimately surface all
these "pass throughs" in one place, though I don't think it should jump the
line of the higher priority models we've got on tap.
On Fri, Jan 8, 2016, 1:29 PM James Gordon<EMAIL_ADDRESS>wrote:

Looks like we can!
select *from "TEXT_MEMO_CD"where "TEXT4000" like '%MADE THROUGH INTERMEDIARY CALIFORNIA REPUBLICAN LEADERSHIP FUND (#1340317); 455 CAPITOL MALL, SUITE 600, SACRAMENTO,%'and "FILING_ID" = 1980975;
—
Reply to this email directly or view it on GitHub
https://github.com/california-civic-data-coalition/django-calaccess-raw-data/issues/1164#issuecomment-170131369
.


Search field is reader multi time on map when reuseMaps=true

I implemented like in your example. Search field working well. But if I switch to an other page or when local dev server is reloaded on files change, the search field is not destroyed and an other appears.
How to fix that?

Hi @matinfo, what version are you using? Are there any errors in the dev console when this happens?

@SamSamskies the version used are:
"mapbox-gl": "^1.3.0",
"react": "^16.9.0",
"react-dom": "^16.9.0",
"react-map-gl": "^5.0.10",
"react-map-gl-geocoder": "^2.0.11",
"react-router-dom": "^5.0.1",

Thanks for the info. I've never used the reuseMaps prop before. I'll need to look into that.

I wrap my ReactMapGl with Deck.gl, I use the suggestion to use reuseMap but still rendered minima three times,
and after the viewport changed, it become so many search field

Hi @mohridwanhdp, could you provide an example in Code Sandbox that demonstrates your issue?

Hi @mohridwanhdp, could you provide an example in Code Sandbox that demonstrates your issue?

I'm sorry it's my mistake here cause not use the right ref
Before, I use this in my ReactMapGL
<ReactMapGL               mapStyle={mapStyle}               mapboxApiAccessToken={MAPBOX_TOKEN}               preserveDrawingBuffer={true}               ref={(reactMap) => (this.reactMap = reactMap)}               reuseMaps={false}               onLoad={(e) => {                 this._onMapLoaded();               }}             >
Then i change the ref like this :
ref={this.reactMap}
And It's become normal


Accomodate module change for JL3 move

I have a mild preference for using a somewhat more succinct name for this module since it includes helpers that users may want to import and run directly. Let's talk about this at co-work.

Module name is now lsst.rsp.  Closing this one unmerged.


feat(cpo): Support disable profiling annotation

What this PR does / why we need it: Add annotation to support disabling profiling for control plane components.
Which issue(s) this PR fixes
Fixes #1438
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[x] This change includes docs.
[ ] This change includes unit tests.

If needed, I can implement unit tests after review.

/lgtm

/approve

/cherry-pick release-4.9


Radar chart is always filled

Regardless of the setting for "Area Filled" a radar chart is always drawn filled.

Released with version 4.01.01


Vizabi should not trigger changeHistory event on first load

Right now, when we load Vizabi, it already triggers a  changeHistory event, which is used by the external page to update the URL.
As far as I'm aware of, this happens because of splash screen.
Such event should only be triggered when there's an actual change after first load (apart from splash) and probably only after user interaction.

@jheeffer Maybe we can look into this together or you can try to find your way through

still an issue, as far as i can see
right after load there is one step back

current URL: http://dev-tools.gapminderdev.org:3001/tools/bubbles#_
previous URL: http://dev-tools.gapminderdev.org:3001/tools/bubbles

but it's definitely not an urgent thing, it's an optimisation of a tools page, not even vizabi itself

Is already a lot better, bút splash screen still triggers a url-change.

Fixed, but splashscreen is still hacky.


feat(defi): abstract withdraw percent click handlers

Description
Layers on top of https://github.com/shapeshift/web/pull/3171 and https://github.com/shapeshift/web/pull/3180, abstracting percent click handlers into a single reusable one
Note that this assumes opportunities with the same base logic - DeFi providers with a domain-specific one still pass onPercentClick down, only now renamed onPercentOptionClick for consistency
Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
N/A
Risk

Percent click handlers could be fully borked for Foxy and Yearn withdraws
Mac click handlers could be fully borked for Foxy and Yearn withdraws

Testing

Engineering

Operations

Screenshots (if applicable)

Current dependencies on/for this PR:

develop

PR #3192   👈

This comment was auto-generated by Graphite.

Blocked-ish by https://github.com/shapeshift/web/pull/3441 - once we base unit all the things, having abstracted percent click handlers will be a no-brainer, we will be able to have a single-implementation-fits-all

Will close it for now and revisit after we go full base units


Adding Java_Complete_Guide

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
Adding resource
For resources
Description
Why is this valuable (or not)?
Yes, it is valuable. This is a complete guide for JAVA.
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

The repo does not host books. Please read docs/CONTRIBUTING.md guidelines. If the book is free licensed, consider depositing in archive.org or unglue.it


Move PQC docs to this repo

Move this into docs in this repo; split into two docs, "about" and "required for this project."

They're now here
https://github.com/apache/mahout/blob/main/docs/p_q_c.md
https://github.com/apache/mahout/blob/main/docs/qumat_gap_analysis_for_pqc.md

Closing


Update links to point to new topic locations for Secrets and Replication Clusters

Closes #509

/ready-to-test

/ready-to-test

/ok-to-merge


"unreachable" error when using heightmap

I'm getting the following error:
Uncaught RuntimeError: unreachable
    at std::panicking::rust_panic_with_hook::h9d91fa0fae16500f (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[788]:0x108529)
    at std::panicking::begin_panic_handler::{{closure}}::h2d82b5b2db976f3f (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[897]:0x10d32a)
    at std::sys_common::backtrace::__rust_end_short_backtrace::h24539600b7e2452c (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[1245]:0x114d35)
    at rust_begin_unwind (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[1161]:0x113c87)
    at core::panicking::panic_fmt::h91d2023e5afe1929 (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[1247]:0x114d99)
    at core::panicking::panic::h5da66f104e83c46a (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[1111]:0x112e78)
    at rapier3d::geometry::broad_phase_multi_sap::sap_axis::SAPAxis::batch_insert::h22440a950023a6c1 (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[58]:0x6e464)
    at rapier3d::geometry::broad_phase_multi_sap::broad_phase::BroadPhase::update::hcb9a53f41ef84931 (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[46]:0x5b78c)
    at rapier3d::pipeline::physics_pipeline::PhysicsPipeline::detect_collisions::he6a99ba9c9b1190f (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[321]:0xdbb17)
    at rapier3d::pipeline::physics_pipeline::PhysicsPipeline::step::h0bcdada6b45b091e (http://localhost:3000/0c0ef481b8428110185b.module.wasm:wasm-function[45]:0x59992)

What is interesting is that if I fill the heightmap with all zeroes, I don't get this error. If I pass in real terrain data I do.
Even more strangely, if I multiply the data by a small constant (.01) the error goes away, but I multiply by (.99) it happens. So basically it is as if the magnitude of what is in the array matters. I've tried a bunch of other combinations, it's completely consistent but I can't figure out the pattern.
Note: it took me a few hours to even get this stack dump, as I had to build the library locally in order to turn on debug symbols, and I had to fuss with it to get it to build on my system.
The way I am building the height map isn't very interesting either:
/** Physics object representing a height field such as terrain. */
export class PhysicsHeightField {
  #body: RigidBody;
  #collider: Collider | null = null;

  constructor(private rapier3d: R, private world: World, position: Vector2) {
    // TODO: Add friction and such.
    const rbDesc = rapier3d.RigidBodyDesc.newDynamic().setTranslation(position.x, 0, position.y);
    this.#body = this.world.createRigidBody(rbDesc);
  }

  public dispose() {
    if (this.#collider) {
      this.world.removeCollider(this.#collider);
      this.#collider = null;
    }
    this.world.removeRigidBody(this.#body);
  }

  public setHeights(heights: Float32Array) {
    // Remove previous collider, if any.
    if (this.#collider) {
      this.world.removeCollider(this.#collider);
    }
    // Generate new height map from data.
    const clDesc = this.rapier3d.ColliderDesc.heightfield(
      PLOT_LENGTH,
      PLOT_LENGTH,
      heights,
      terrainScale
    );
    this.#collider = this.world.createCollider(clDesc, this.#body.handle);
  }
}

So changing it to static made the problem go away - still don't know why.
I'm finding it very difficult to debug problems, such as when I pass in a bad param - it just throws "unreachable" with no explanatory message as to what went wrong.

Thanks for following up with how you've solved this. I'm running into the same error with the Heightfield but making it Static hasn't worked yet. I'll follow up if I find a solution, for others who are running into this.


Add bch_cad and btg_cad books

Coverage remained the same at 100.0% when pulling 46dd335ef0339e8ed4c17548973b146d545b4f8f on julienld:dev into ba1591eead956923c500c92d10804088650c07b8 on joowani:dev.

The Python 3.6 Travis CI has failed, but i'm quite sure that it's not related to this commit.

Hi @julienld!
Thank you for the commit. I have actually added the two books in the new release I am currently working on. It should be ready in a few days. The new version, by the way, has been refactored with non-backward compatible changes. I will be closing this one.
Thanks,
Joohwan


fix link for hooks example

Signed-off-by: Steve Kriss<EMAIL_ADDRESS>Fixes #1648

I see version 0.11 and 0.10 have the same issue:

I don't see any reason to bother updating pre-1.0 docs going forward as a policy. Let me know if you disagree.

I don't see any reason to bother updating pre-1.0 docs going forward as a policy. Let me know if you disagree.

Yes, that's fine. I forgot about this guideline.

Per https://github.com/heptio/velero/pull/1649#discussion_r303144248, this should be good to merge if everyone's OK with https://github.com/heptio/velero/pull/1662

OK now that #1662 is done, we can merge this!


Missing documentation

The README does not document the following:

What is the effective range? -2^47 to 2^47?
What overflow behaviors are expected? Does it work with checked/unchecked blocks?
Are conversions to other numeric types explicit/implicit?

Due to the optimisation for speed, the muls and divs are significantly faster, but use the whole range of 32 bits to overflow into, meaning its only having a range of ^15.
Overflow is as described above.

Conversions to other numeric types are explicit: https://github.com/RomanZhu/FixedPoint-Sharp/blob/8bbcc9d3523277035f2cc3fc026264bdc4f9901b/FixedPointSharp/fixnum/fp.cs#L277
All basic operations also accept ints: https://github.com/RomanZhu/FixedPoint-Sharp/blob/8bbcc9d3523277035f2cc3fc026264bdc4f9901b/FixedPointSharp/fixnum/fp.cs#L96


Remove self link from Axes

Changes

Remove self link from Axes class

Checks
By submitting your pull request for review, you agree to the following:

[x] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[x] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[x] To the best of my knowledge, all proposed changes are accurate.

Thanks a bunch @WalkQuackBack!


Correct indentation, CamelCase function and content tag

Ok, one last thing. Do you mind squashing/renaming these commits? Otherwise lgtm

Sorry, I don´t understand you

Currently there are two commits, one with the title "Correct indentation, CamelCase function and content tag", and one with the title "correct indentation". In the previous pull request (https://github.com/dart-lang/polymer-dart/pull/657/commits) the first commit was titled "Create Polymer 1.0 elements automatically" which was more descriptive of the actual issue.
It would be nice if you could at least rename the first commit here to be the same.

oh whoops didn't see that this had been updated, merging now thanks!


Global Shadowing warning is incorrect or absent in some situations

Godot version
v4.1.stable.official [970459615]
System information
Godot v4.1.stable - Windows 10.0.19045 - Vulkan (Compatibility) - NVIDIA GeForce GTX 1080 (NVIDIA; <IP_ADDRESS>5) - Intel(R) Core(TM) i7-7700K CPU @ 4.20GHz (8 Threads)
Issue description
this was first discussed in #34161 as I thought this was the issue here, but This comment remarked that it was actually probably not the case.
Orientation is a global Enum, however when I attempt to access it by clicking on the values in new(), it points to my own class's Enum, and the Engine never warns me about it being already defined by the global scope.
This enum is also entirely inaccessible to me, unless I copy and paste one of its values such as Orientation.HORIZONTAL, the value itself will auto-complete, but the enum is not inspectable and doesn't link to the docs.

Steps to reproduce
As seen in the screenshot above:

Create an enum named Orientation in a script
Create a class which takes this enum for a parameter in its _init()
Attempt to call YourClass.new( ... ) and pass it your parameter

Minimal reproduction project
N/A

It appears that the names are somehow secretly registered, hard to tell


Check collision thickness to ignore pillars

Add collision_thickness property to operator.script.
It will cost additional 4 raycasts. Two to check the horizontal thickness and two for the vertical thickness.

Think about using a sphere collider on the camera object. Not totally understand how to use it but need to take a look at Untiy extensions.


Setup on Heroku

Hi,
I'm trying to deploy this on Heroku and don't have the database schema. Do you have a database schema script? Is there anything else we need to do to make this working?
Thanks,
Vlad

@vdanciu There is only one table needed, I just added the schema at the end of the readme file.


Use new theme?!

This is an attempt to try using the new sphinx-pythia-theme on the Foundations book.
I believe the critical missing issues for the Foundations book are dealing with the* top navigation bar*.  Currently, I have the links in the top navbar to be the same as those on the Portal, but I don't think we actually want that.  Perhaps we can discuss what they should be.

At first glance, it looks great! I agree that the top nav bar needs some thought.
Let's put a discussion about a "unified" nav bar across our sites on the agenda for an upcoming EWG.

Yes, and I think there is also room to discuss having 2 separate/different navbars but with similar-themed logos.  There are many options.

One thing I notice in the preview is that the Foundations footer is missing.
This is where we currently list the licenses and give links to the feedback form and Contributor's guide.

Yeah.  I think we need a place for that in the new theme.

@brian-rose: Can you take a look at this again?  I've made some updates to the theme and I think the issues you mentioned are now fixed.

Most notable changes are the unified navbar along the top and the new navigation menu in the footer. Also note the GitHub and Twitter icons at the top-right of the page.
But there is also an extra_navbar text saying where the theme comes from, and I moved the extra_footer content to there to make room for the NSF acknowledgement in the extra_footer.

OK, I see those improvements! Definitely heading in the right direction. I really like the unified nav bar between this and https://github.com/ProjectPythia/projectpythia.github.io/pull/197
A few critiques:

The left nav bar should have a link to the Foundations landing page at the top of the TOC. (Yes I know that there is a "Foundations" link in the top nav bar, but the TOC should be self-contained). I suggest there should just be a link labeled "Pythia Foundations" right above "Preamble".
We're still missing the "rocket ship" Binder launch button for notebook pages.
The feedback form is pretty hard to find now. It's in a small link at the bottom of the bottom footer. I feel like we need to draw more attention to this if we want to solicit good feedback.
Related: this feedback form is meant for Foundations only. But now it's integrated into the theme so appears on the bottom footer of all pages, including non-Foundations pages. So maybe we could move the Feedback form link to the extra_navbar area?

[ ] The home-page link in the left nav bar does need to be present.  I'll figure out how to fix that.
[ ] I'll look into the rocket ship button.  Not sure why that isn't displaying.
[ ] We could add the feedback form to the top navbar just for the Foundations.  I don't think that would be too jarring for people.  ???
[ ] The bottom navbar content (i.e., feedback form link) is not integrated with the theme.  The capability to display the nav menu on the bottom is a new feature of the theme, but the content can be changed in the _config.yml file (in the sphinx: config: html_theme_options: footer_menu: section).  This can be different for the Portal or Foundations, so I think that content needs to be discussed for each site.  So, we could drop the link to the feedback for on the Portal for now.

@brian-rose:  Ok.  I figured out why the rocket ship / Binder button wasn't being displayed.  It turns out this is because Jupyter Book sets Sphinx configuration options (specifically Sphinx html_theme_options) and then overrides them wholesale if you use a sphinx: config: ... section in the _config.yml.  Let me explain.
When you set the repository: url: ... option in _config.yml like so:
repository:
  url: https://path.to/my/repo

Jupyter Book translates this option to this:
html_theme_options = {
    'repository_url': 'https://path.to/my/repo',
}

in the Sphinx configuration.
If you then set any html_theme_options in the sphinx: config: ... section of the _config.yml, like so:
sphinx:
  config:
    html_theme_options:
      my_option: my_value

this will cause Jupyter Book to completely overwrite the Sphinx html_theme_options dictionary to be:
html_theme_options = {
    'my_option': 'my_value',
}

and the html_theme_options['repository_url'] value gets lost.
This is all because Jupyter Book overwrites the top-level Sphinx configuration options with anything in the sphinx: config: ... section of the _config.yml file.  It does not update the Sphinx configuration options recursively (i.e., add/overwrite values in any dictionaries or lists).
Sigh.
So, the solution is to put all of the Sphinx configuration options (even the ones that would appear in the launch_buttons: or html: or repository: sections of the _config.yml file) in the sphinx: config: ... section of the _config.yml file.
It's not pretty, but it works.  I've raised issue https://github.com/executablebooks/jupyter-book/issues/1588 to discuss a possible fix to this, but I'm not sure if it is problematic for other reasons.
That is, Jupyter Book pulls options from the launch_buttons: and html: and repository: sections of the _config.yml and uses them to set Sphinx configuration options.  And then if you supply a sphinx: config: ... section in the _config.yml file, these settings will completely override any Sphinx settings

@kmpaul that sounds like top-quality sleuthing on your part, congrats!
If the hack works, it works! I can see that we have our Launch button back now.  I would just suggest putting comments in the _config.yml file to explain why things are not in standard Jupyterbook format (possibly including a link to this discussion) so any future Pythians are less confused about it.

Great work @kmpaul ! I've successfully built the Foundations site with the new theme. Couple things:

sphinx-pythia-theme is available in conda-forge, but is listed as a pip install in environment.yml
For some reason, the footer-logo-nsf.png is not displaying.

Yes. It is available on conda-forge. I wasn't sure how long it would take to appear there, so I can not relied on pip for now. But we can make it a conda dependency now.

I'll look into the footer logo.

The footer logo displays on the preview. @ktyle, are you not seeing it in your local builds?

Yes, on my local build. Other PNG/SVG images substituted in the  tag work fine. I'm going to try a build on another system to see if it's isolated to the Apache httpd-served system on which I serve the jupyterbook.

@ktyle: You might try looking in the _static folder in your build directory (_build/html/_static/) to see if the image is there. If it is, then there is a problem with the link.
On what page were you seeing the image not found error?
-- Kevin

@kmpaul The issue looks to be with the leading "/" that precedes _static in _config.yml. Eliminating that makes the image appear.
Any reason to place the NSF logo in _static as opposed to images/logos , as with our other logos?

That's odd. If you leave the leading /, then it won't display on any pages that are in any subdirectories.

In other words, it needs to be an absolute path, not a relative one.

As for why I put it in the _static folder instead of images, it's because the NSF acknowledgement section is now in the extra_footer section of the PyData Sphinx Theme (which the Sphinx Book Theme uses, too).  However, because that extra_footer is just text (i.e., can accept raw HTML), it is not processed by any of the Sphinx extensions.  Hence, Sphinx doesn't know there is an image in that section to copy over to the _images folder in the build directory.  So, to get it to work, you need to put it in a place where everything will get copied over, and that is what the _static folder does.

@ktyle:  After removing the leading /, can you right click on the image in your browser and select the "display image in another tab/window" option?  Then tell me the URL that appears in your browser.

When I do that, I see something like this:  http://<IP_ADDRESS>:8080/_static/footer-logo-nsf.png.

Ok, figured it out. The issue with the leading "/" is only problematic for my use case, where I'm actually serving the _build directory via the Apache web server. The server uses "/" as the document root. If I place a link there to where my Foundations _static directory is, then it all works. I'm not sure if we're going to have any users who would deploy Foundations in this manner, so it is probably just worth noting in case anyone actually does this.

Hmm.  Yeah.  That's kind of what I was thinking it must have been.  I guess to extend you question, is there anyone (other than us) who is going to deploy the Foundations book at all?
Incidentally, I usually use the NodeJS http-server application for viewing Sphinx-built HTML sites on my laptop for development.  And I use it primarily because the results are usually the same as what GitHub Pages deploys.  Some people, though, just use the built-in HTTP server that comes with Python 3:  python -m http.server --directory /path/to/html/.  I like the npm module, and I just install it on my laptop with homebrew (like I do with pre-commit and some other "I always use this" stuff).

Hmm. Yeah. That's kind of what I was thinking it must have been. I guess to extend you question, is there anyone (other than us) who is going to deploy the Foundations book at all?

I would say that we are aiming to support users contributing to the Foundations book, but we are not directly supporting deploying the Foundations book elsewhere. I'm sure people will use it as an exemplar for their own JupyterBook projects, but we are not trying to document and support that ourselves.

Handy to add form data to google sheets
find google sheets id

@brian-rose @ktyle:  I think this PR is in a reviewable state.  I've fixed the top navbar links according to our discussion, and I've created a footer menu that primarily includes quick links to other places on the two sites.  Let me know if you like this format and I'll open it up for review.

@brian-rose @ktyle: I think this PR is in a reviewable state. I've fixed the top navbar links according to our discussion, and I've created a footer menu that primarily includes quick links to other places on the two sites. Let me know if you like this format and I'll open it up for review.

I think it looks fantastic @kmpaul. I have no further nitpicks at this time :-)

Looks good to me as well ... ready to open for review.

Note that sphinx-pythia-theme#42 does drastically change the appearance of the theme for Jupyter book usage (eg here), moving away from the intentionally columnar look of the pydata/sphinx book themes, not just on the "single page" sections of the site. I'm not sure if that was by design @kmpaul, but it should probably be noted for anyone reviewing this!

@dcamron How does it drastically change it?  Is it a problem?

@dcamron How does it drastically change it? Is it a problem?

Updated my previous comment!

@dcamron: It should only change the layout for wide high-res screens.  Are you  specifically saying that for normal "book pages" the narrower width pages should be preserved?  I think I can do that.  I just want to know specifically what you think looks best.

Ok.  So you are referring to the normal "book pages".  I can fix that.

Just FYI:  I'm waiting on sphinx-pythia-theme#35 before creating a new release of the theme and then I will merge this, barring any additional feedback from people that requires changes.

Looks like this is good to merge! The failed link will be fixed when #205 is merged.