xc7-litex failing

failing in travis.ci
pre-req is fixissue 65 to get tests running
progress so far fixing is that the litex stuff is being installed outside of the symbiflow-examples directory so a cd needs to happen ... see line 5

i would prefer if the install instructions gave the user a measure control on where all the Litex stuff ends up ... needs a $LITEX_INSTALL_DIR
with the cd fix, the test is now failing in litex/litex/build/xilinx/symbiflow.py", line 103, in     ####_run_make
ROM usage: 20.96KiB 	(65.50%)
RAM usage: 1.63KiB 	(20.41%)
make: Leaving directory '/home/travis/build/scted/litex/litex/boards/targets/build/arty/software/bios'
Traceback (most recent call last):
  File "./arty.py", line 162, in <module>
    main()
  File "./arty.py", line 155, in main
    builder.build(**builder_kwargs, run=args.build)
  File "/home/travis/build/scted/litex/litex/soc/integration/builder.py", line 217, in build
    vns = self.soc.build(build_dir=self.gateware_dir, **kwargs)
  File "/home/travis/build/scted/litex/litex/soc/integration/soc.py", line 1054, in build
    return self.platform.build(self, *args, **kwargs)
  File "/home/travis/build/scted/litex/litex/build/xilinx/platform.py", line 53, in build
    return self.toolchain.build(self, *args, **kwargs)
  File "/home/travis/build/scted/litex/litex/build/xilinx/symbiflow.py", line 263, in build
    _run_make()
  File "/home/travis/build/scted/litex/litex/build/xilinx/symbiflow.py", line 103, in _run_make
    if tools.subprocess_call_filtered(shell + [script], common.colors) != 0:
NameError: name 'shell' is not defined`

Indeed this seems to fail on ubuntu:trusty and old centos but not on other OSes? (other OSes have travis timeouts but that is much later in the run)

i just tested it on Ubuntu:focal

still not passing ubuntu:focal fwiw
erroring out for the directory issue:

i assume it will fail later in symbiflow.py line 103 (see above for Traceback) which is
if tools.subprocess_call_filtered(shell + [script], common.colors) != 0:
i can't find reference to 'shell', 'script' or 'common' anywhere in symbiflow.py

assume it will fail later in symbiflow.py line 103 (see above for Traceback) which is

if tools.subprocess_call_filtered(shell + [script], common.colors) != 0:
i can't find reference to 'shell', 'script' or 'common' anywhere in symbiflow.py
Hi @scted, this issue has been solved in litex upstream, and the error should not appear anymore if you get the latest litex version (I think you can do ./litex_setup.py update install to get the latest)

@acomodi
i still can't get it to complete in Travis (see bottom for Traceback) ... now failing line 84 ... apologies if it is usererror on my part

tools.subprocess.py not returning 0 ... way beyond my capability to troubleshoot

still had to modify the xc7/README.rst to cd up one level to get this to get this far

@scted I think that we resolved all the problems here (according to https://github.com/SymbiFlow/symbiflow-examples/pull/73#issuecomment-734935737) and we can close this issue. Is that true?

thanks

@rw1nkler FYI, there is a discrepancy between the Litex arty platform and the new migen arty_a7 platform ... was using that because i am using migen to build my own stuff ... the migen stuff almost works but it says the device is 'xc7a35tic324-1L' ... symbiflow was failing at step 8 of synth ... i changed the device to 'xc7a35tic324-1' and everything worked ... not sure where to report that ... lot of new migen stuff

@scted Thank you for reporting this! This is a good place to report any problem with the SymbiFlow Toolchain.
I would suggest opening a separate issue for that, to track the problem. (This issue is closed)
In fact, I probably know what is causing the problem. Can you please open a separate issue with the information that you gave above? Can you also provide more detailed information on how to reproduce your problem?


[Feature-8215][Alert] Support WebexTeams Plugin

Purpose of the pull request
Support WebexTeams Plugin

Brief change log
Add dolphinscheduler-alert-webexteams to root pom.xml
Add dolphinscheduler-alert-webexteams to dolphinscheduler-alert/dolphinscheduler-alert-plugins/pom.xml
Add dolphinscheduler-alert/dolphinscheduler-alert-plugins/dolphinscheduler-alert-webexteams folder

Verify this pull request
Added WebexTeamsAlertChannelFactoryTest & WebexTeamsSenderTest to verify the change.
close #8215

Codecov Report

Merging #8217 (5fbac56) into dev (6f74241) will increase coverage by 0.22%.
The diff coverage is 80.74%.

@@             Coverage Diff              @@
##                dev    #8217      +/-   ##
============================================
+ Coverage     41.40%   41.62%   +0.22%
- Complexity     3801     3833      +32
============================================
  Files           658      664       +6
  Lines         27657    27818     +161
  Branches       3132     3143      +11
============================================
+ Hits          11451    11580     +129
- Misses        15106    15129      +23
- Partials       1100     1109       +9

Impacted Files
Coverage Δ

...in/alert/webexteams/WebexTeamsParamsConstants.java
0.00% <0.00%> (ø)

...lugin/alert/webexteams/WebexTeamsAlertChannel.java
16.66% <16.66%> (ø)

...uler/plugin/alert/webexteams/WebexTeamsSender.java
77.46% <77.46%> (ø)

...cheduler/plugin/alert/webexteams/WebexMessage.java
78.94% <78.94%> (ø)

...plugin/alert/webexteams/WebexTeamsDestination.java
80.00% <80.00%> (ø)

...lert/webexteams/WebexTeamsAlertChannelFactory.java
97.91% <97.91%> (ø)

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...inscheduler/common/task/sqoop/SqoopParameters.java
74.00% <0.00%> (-2.00%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
53.52% <0.00%> (+0.70%)
:arrow_up:

Continue to review full report at Codecov.

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

Can you add a picture of the UI interface and the final format result of the alert?

Hi @liqingwang , please also need to add document about how to use this alert in the same level in https://dolphinscheduler.apache.org/en-us/docs/dev/user_doc/guide/alert/enterprise-wechat.html

Hi @zhongjiajie , the document is here #dolphinscheduler-website/pull/662, please to be review.


Annotation output

I'm looking to render items that will appear on top of a CandleStick chart at various positions along the x axis. It looks as though I may be able to accomplish this via Annotations, but I could also see a Scatter working, with a custom Marker. Annotation feels like a more natural fit, but it the drawing seems limited to paths, at least with SvgPathAnnotation. What might be the simplest way for me to render my own SVG output for an Annotation?

ScatterSeries with custom marker is simpler, but if you go with the canvas version, there cannot be click events attached (atleast not yet). By using Annotate, and a custom path as an annotation you can draw any svg path you like and it could also have events attached to it.

Very helpful, thanks! Have decided to go the Annotate route.


Can't train a good "place" skill.

🐛 Bug
I used the provided code to train the "place" skill with the command below. I can successfully run training, while the "ovmm_place_success" and "ovmm_placement_stability" metrics during training stay 0 all the time. I have attached the training logs of the 10-th update and the 5000-th update below.
#/bin/bash

export MAGNUM_LOG=quiet
export HABITAT_SIM_LOG=quiet

set -x
python -u -m habitat_baselines.run \
        benchmark/ovmm=place \
        habitat_baselines.checkpoint_folder=data/new_checkpoints/ovmm_ori_lab/place \
        --config-name ovmm/rl_skill \

update: 10	fps: 302.235
Num updates: 10	Num frames 25600
Average window size: 10  articulated_agent_force.accum: 696.418  articulated_agent_force.instant: 378.308  ee_to_rest_distance: 0.646  force_terminate: 0.011  num_steps: 24.232  obj_anywhere_on_goal.0: 0.045  object_at_rest: 0.121  ovmm_ee_to_place_goal_distance.0: 0.971  ovmm_object_to_place_goal_distance.0: 1.355  ovmm_place_reward: -6.169  ovmm_place_success: 0.000  ovmm_placement_stability: 0.000  reward: -13.333  robot_collisions.obj_scene_colls: 0.129  robot_collisions.robot_obj_colls: 0.171  robot_collisions.robot_scene_colls: 0.751  robot_collisions.total_collisions: 1.051  robot_collisions_terminate: 0.992
	Perf Stats: trainer.rollout_collect: 8.000 trainer.sample_action: 0.002 trainer.obs_insert: 0.000 trainer.step_env: 0.056 trainer.update_stats: 0.004 trainer.update_agent: 0.463

update: 5000	fps: 441.130
Num updates: 5000	Num frames 12800000
Average window size: 50  articulated_agent_force.accum: 1250.525  articulated_agent_force.instant: 661.172  ee_to_rest_distance: 0.787  force_terminate: 0.038  num_steps: 23.874  obj_anywhere_on_goal.0: 0.121  object_at_rest: 0.057  ovmm_ee_to_place_goal_distance.0: 0.949  ovmm_object_to_place_goal_distance.0: 0.859  ovmm_place_reward: -5.815  ovmm_place_success: 0.000  ovmm_placement_stability: 0.000  reward: -3.097  robot_collisions.obj_scene_colls: 0.059  robot_collisions.robot_obj_colls: 0.138  robot_collisions.robot_scene_colls: 0.838  robot_collisions.total_collisions: 1.035  robot_collisions_terminate: 0.967
	Perf Stats: trainer.rollout_collect: 5.359 trainer.sample_action: 0.002 trainer.obs_insert: 0.000 trainer.step_env: 0.036 trainer.update_stats: 0.004 trainer.update_agent: 0.439

Steps to Reproduce
Steps to reproduce the behavior:
Here is the specific commit of habitat-lab I'm using.
After setting up the environment, I simply ran the command above without any modifications to the code.
Expected behavior
As shown the training logs attached above, the "ovmm_place_success" and "ovmm_placement_stability" metrics during training stay 0 all the time.
Additional context

Thanks for the issue. We will take a look at this, but would be helpful if you could answer these:

How many environments and gpus are you using?
Are you able to train other skills under your setup?

Hi, thanks for your quick reply.

I'm using 20 environments on a single RTX 4090 GPU. (BTW, will the number of envs and gpus make a huge difference?)
Yes, I tried to train a "gaze" skill before with the same rl_skill.yml, the pick_success metric can gradually rise from ~0 to ~0.82 during the total of 100M updates. I wonder if all the skills you trained use the same config with the only difference on the action_distribution_type.

Yes, they do make a difference. The policies we trained were trained with 16 envs per gpu on 2 nodes having 8 gpus each. The place skill does take a while to pick up (10M) but perfect zero success is odd. I will confirm and let you know if the issue is with the number of environments.
Yes, that is right.

Any updates on this? Were you able to train a place skill?
@yvsriram should we update instructions on the github repo?

Oh, I didn't try training it few nodes yet. Will try and get back on this.


$(document).ready() not called after open page in content.

The problem is that when I want to open another page by using the clickable container widget from Mendix app store, sometimes $(document).ready() function is not being called. In my case I want to find the document contains a certain class, which does, and the do some action with it. I also tried with simple alert messages.

Why are you using the ready function on document? You don't have to do this since the DOM is ready when the snippet is going to run.


[External Request] - Codeunit CRMSetupDefaults - AddEntityTableMapping

Could you make this function global?

Availability update: We will publish a fix for this issue in the next update for release 18.


valgrind investigations

Be sure to compile with WIDE_INTEGER_HAS_COVERAGE which (even though intended for coverage runs) drastically reduces the number and scope of test cases. This makes a slow valgrind run feasible on GHA pipelines.
The first local valgrind run was very satisfying. See report below.
==4353==
==4353== HEAP SUMMARY:
==4353==     in use at exit: 0 bytes in 0 blocks
==4353==   total heap usage: 15,124,969 allocs, 15,124,969 frees, 7,127,874,355 bytes allocated
==4353==
==4353== All heap blocks were freed -- no leaks are possible
==4353==
==4353== For lists of detected and suppressed errors, rerun with: -s
==4353== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

TBD: Add valgrind run to CI and check (via, for instance, grep) for:
ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

Hi Chris. Curious: have you identified errors caught by valgrind which are
not caught by ASan? I've found ASan generally a lot easier to work with.
On Sat 22 Jul 2023, 12:44 Christopher Kormanyos, @.***>
wrote:

TBD: Add valgrind run to CI and check (via, for instance, grep) for:
ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
—
Reply to this email directly, view it on GitHub
https://github.com/ckormanyos/wide-integer/issues/368#issuecomment-1646564562,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFTGN366YQSAG5OWCKPL5LXRO4LBANCNFSM6AAAAAA2TY53NM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi John (@johnmcfarlane)
Locally, I get 0 bytes missing in any way on valgrind. And I also see no problems whatsoever with ASAN (i.e., address sanitize runs). So these are consistent.
However, uggghhhh. I am struggling at the moment as well on the runners. I did get one successful valgrind run on the pipeline but now I get a very curious core dump.
So I'm not sure yet if valgrind actually adds any value.

Hi John (@johnmcfarlane) I was curious about this valgrind. I think remember it came out after I had finished heavy numerics in grad school on FORTRAn in the 80s. Then when I was long out of numerics, I heard some chatter about it. Now it looks like it's supported so I thought I'd investigate it.
I'll keep you posted. I also ran across a POSIX sync checker calld helgrind. I do not know if that adds anything beyod TSAN?

found ASan generally a lot easier to work with

Yes John (@johnmcfarlane) I just empirically stumpled across a problem running valgrind on GHA Ubuntu runners. It seems like valgrind consistently core dumps on certain code sequences compiled with -march=native and/or -mtune= native. So I worked around this with the more vanilla tuning -m64.
I think I will, in fact, retain the valgrind run. Although it does not seem to really find anything that ASAN didn't find, I think I'll keep it going for a while on the nightly, but I'll freeze in on something like GCC-12.
But I'm watchful of it. If valgrind ever causes big problems, I'll eliminate it's use on the pipelines.


fix: truncate long tx metadata

Summary
Adds margin right to key text and truncate to value text in KeyValueBox
Testing
Platforms

Desktop

[ ] MacOS
[x] Linux
[ ] Windows

Mobile

[ ] iOS
[ ] Android

Checklist

Please tick all of the following boxes that are relevant to your changes.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or modified tests that prove my changes work as intended
[x] I have verified that new and existing unit tests pass locally with my changes
[x] I have verified that my latest changes pass CI workflows for testing and linting
[ ] I have made corresponding changes to the documentation

I think there is two issues still:

The little triangle of the tooltip is not showing, likely something to do with the scroll bar?
When hovering over the tooltip, then going back over the text the flag doesn't toggle correctly and then you are left in an inverted state.

Suggestion: We should have the tooltip positioned on top and not right.

@nicole-obrien  I pushed some changed to fix these suggestions


Using FoldableLayout with TabLayout

The problem is that I want the details view to unfold from the RecyclerView and either be below the Toolbar and above the TabLayout, or above both the Toolbar and the AppBarLayout.
I figured out 2 possible solutions. Firstly, programatically  change layout params of the ViewPager which contains the RecyclerView that houses the CoverView, so that when the RecyclerView item is clicked, the ViewPager positions itself above the TabLayout allowing the details view to draw above the TabLayout.
Or secondly, to have the ViewPager always above the TabLayout and setting the XML topMargin on the RecyclerView to make it look like its below the TabLayout, but as the ViewPager is above the TabLayout this allows the details view to unfold above the TabLayout.
This approach works really well, except when the RecyclerView item is scrolled behind the TabLayout, when this happens, the unfoldable layout ** animates through **the TabLayout and whole item row becomes visible through the TabLayout, maybe this is the result of the bitmap you create ?
Problem in a nutshell: How would you suggest I use the FoldableLayout with a TabLayout if I want the details view to appear above the TabLayout and/or Toolbar ??
Thanks in advance,
I can provide video of the problem if it would help
Edit: Actually, when the RecyclerView scroll so any item is behind the TabLayout, the item behind the tabLayout will appear drawn over the TabLayout even if it is not the item that has been clicked.

It looks like you have a problem with layouts arrangement, not sure if it is an issue of the library.
But can you please provide some video, it's pretty hard to understand the issue just from description.

Didn't receive a video, but after re-rereading it seems that the problem you have is that in unfolded mode you want an item to be above tab bar, while in folded mode (inside the list) items should be behind tab bar, but at same time you want both unfolded view and list view to be in same page in view pager. Your second solution should mostly work fine, and the issue you experience with animation is expected but hard to fix (can you imagine how animation should look like in this case?).
My suggestion would be to try to put everything under tab bar and then animate tab bar alpha during unfolding animation. In this case animation will start and finish in correct positions.


Release v0.6.1

Here are some items that didn't make it into v0.6.0:

[ ] Honor netrc #171
[ ] Fix issue with ssh shorthands #278
[ ] Dynamic size cutoff for blob scanning #524
[ ] git-lfs staged but unstaged changes with 0B tracked file #557
[ ] fix usage to show 'git lfs' and missing pull flags #661
[ ] fetch/pull don't respect tracking graph #660
[ ] pushing to a new remote fails if local storage isn't full #662

Investigate

[ ] Detect when credential cacher is not setup #523

I'm half way through purge but on polish terms I'd like to get git-lfs <command> --help / git lfs help <command> actually showing the proper man pages, it's becoming confusing for people to find the docs especially on Windows.

My first attempt at that, much prettying that can be done: https://github.com/github/git-lfs/pull/665


Change conversion from inch to metrical system

According to issue #960 the conversion is 1 / 2.54

When most of PRs are merged, I have to recreate all test files.... uff.


Update example_baseline_keras2.py

A study contains experiments.
An experiment has runs.
Runs compare models on the same data.

@rajeeja any last comments?

Looks fine, merge it


Recover Account: Improve error messages

Description
The error messages in the recover account page should be improved. Currently the error messages are not very user friendly and has keywords like exception and Invalid arguments before the message itself. also there's an error message that mentiones that the mnemonic is invalid while mnemonic is correct it should be Seed since the field itself is called Seed Phrase and there's no mention of mnemonic anywhere else in the app.

Environment

Android Version: 13
Threefold Connect version: 3.7.3 - 173

Work completed:

Refactor validateNameSeed
Remove 'Exception'

Verified on Android 14,
Threefold-Connect: 4.0.0-184
APK:12-12-2024
Currently the error messages in the "Recover Account" process are clear.

Related Links:

https://app.testlodge.com/a/26076/projects/40893/suites/206114?expand_section=407470#case_4050848
https://app.testlodge.com/a/26076/projects/40893/suites/206114?expand_section=407470#case_4050846
https://app.testlodge.com/a/26076/projects/40893/suites/206114?expand_section=407470#case_4050845

The "Exception" was not removed as mentioned above.
The empty Seed field case doesn't follow the approach.


VS Code: New "Shot Diff" is marking the code snippet incorrectly and causing confusion.

Version
v1.24.2
Describe the bug
Platform: macOS
Steps to reproduce:

Have Cody plugin installed in your VS Code editor
Login to Cody with Free/Pro user
Open a source code file and right click
Choose "Cody > Edit Code" and ask generate document and hit enter
Wait for Cody to change the code and check the diff status

Observed behavior:
The diff is marked incorrectly and causing confusion.
Expected behavior
The diff should correctly highlight the changed area.
Additional context
Video link:
https://drive.google.com/file/d/1zXNQonrS-9A2jI1I3eDllX1fMos2akwz/view?usp=drive_link

Thanks for raising @jay-fibi. It looks like the diff is technically correct, but the LLM is wrong as it tries to replace the whole function with a new class.
Could you try with Claude 3.5 Sonnet, we have noticed it is much better with this kind of tasks, or possibly with improved instructions.
I'm updating the default to 3.5 in this PR, and will look at any prompt improvements we can make on our side separately

Sure @umpox I will re-test once the PR gets merged.


[Question] Which queries can be cached by a PWA?

Summary
First of all, thanks for your platform, is amazingly easy to use :)
Anyways, we have a PWA in our company and we are using Stripe on it.
I'd like to optimize a bit the queries that Stripe does back to its servers, but of course, I do not want to break or somehow make insecure my transactions.
So: which queries can be cached by the Service Worker? Or, maybe, should we just set the default caches as given by Stripe servers?
Mostly I can see that there arre requests to js.stripe.com and m.stripe.com.
Are there any guidelines on this?
Thanks a lot!

Other information

Hi @Sturgelose, as far as I'm aware, I don't think your PWA or service worker can operate on network requests made by Stripe iframes, as they are on a different origin. Rest assured that we do our best to keep requests secure and performant, and you should be able to use this library out of the box.


feat: monic_descPochhammer

[ ] depends on: #6918

see #6906 for the ascPochhammer version (before the rename...)


Node 10 requires a callback to rmdir

Makes it work with Node 10

It would be nice if this could be merged so Node 10 is supported.

+1

+1 please merge!

Not ideal, but I solved this by just updating the source (as in PR) locally. https://github.com/phonegap/node-phonegap-build/issues/68

Wow still not merged? I've got the same problem today and this fixed it.

I'm not so awesome at all this node/phonegap config stuff, and my remote build is failing as well. Can someone explain for us dummies how we'd go about using this patched gust42 version of phonegap build?

I'm not so awesome at all this node/phonegap config stuff, and my remote build is failing as well. Can someone explain for us dummies how we'd go about using this patched gust42 version of phonegap build?

you can find the file locally on your computer and make the changes, lib/phonegap-build/create/zip.js

Need this also. Is the project dead/inactive?


fix(bandersnatch): GLV bounds + test

Description
When we revisited GLV scalar decomposition https://github.com/Consensys/gnark-crypto/pull/213, we increased conditionally the loop bounds by 1 for all mulGLV methods for all curves but we forgot to do so for Bandersnatch twisted Edwards curve. This PR fixes the issue.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

How has this been tested?
added scalarMulWindowed which does a windowed double-and-add and compares the result with scalarMulGLV which uses the endomorphism. The method ScalarMultiplication picks scalarMulGLV whenever an efficient endomorphism is available (currently only for Bandersnatch).
How has this been benchmarked?
benchmark                          old ns/op     new ns/op     delta
BenchmarkScalarMulExtended-8       30378         30271         -0.35%
BenchmarkScalarMulProjective-8     33844         33941         +0.29%

No major diff for random scalars.
Checklist:

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] I did not modify files generated from templates
[x] golangci-lint does not output errors locally
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@ThomasPiellard this is just consolidating GLV in twisted Edwards with Weiestrass.


chore(master): release 1.1.35

:robot: I have created a release beep boop
1.1.35 (2023-04-26)
Miscellaneous

deps: update dependency tailwindcss to v3.3.2 (9bbf630)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/keywind/releases/tag/v1.1.35 :sunflower:


NormalFormulaRepository: bug in impliedVolatility

NormalFormulaRepository
There is a bug in impliedVolatility. It does not work well with negative forwards.
Issue is in lines 282 and 283.
Line 282: If forward is negative, the sigma can become also negative. The default starting value should be reviewed.
Line 283: Same issue with maxChange.
In a normal setting, it is very difficult to have an idea of the problem scale (that is what the two lines above are attempting to do). A perfect solution is not trivial, but at least the code should be able to deal with the negative case.

Corrected in PR https://github.com/OpenGamma/Strata/pull/2372


style: polish UI of provision dialog

Description
polish provision UI:

[x] polish popup hint in provision dialog
[x] polish title, label and description in provision dialog
[x] polish the detail list in review page, like don't omit the description in review page.

Task Item
close #5392
Screenshots

Coverage remained the same at 54.801% when pulling aecba37e7392420797814c1261ddbd51dbd8cdca on wenyluo/polishUI into 67990cc1b7c18180b7516bfd2d131ddbe731ad89 on main.

Coverage remained the same at 54.801% when pulling aecba37e7392420797814c1261ddbd51dbd8cdca on wenyluo/polishUI into 67990cc1b7c18180b7516bfd2d131ddbe731ad89 on main.

Hi @JaskeeratBedi-cmk, already updated that:

Hi @JaskeeratBedi-cmk, already updated that:

Looks good! We can close this ticket.

Looks good! We can close this ticket.

@JaskeeratBedi-cmk this isn't a ticket. this was a pull-request with associated code changes.

@JaskeeratBedi-cmk this isn't a ticket. this was a pull-request with associated code changes.


How to check if a utf8 string is already inscribed?

For example I want to inscribe a .txt file that contains the name "Bernd" it would be inscribe the txt value Bernd, correct?
How can I search or check if the utf8 string "Bernd" is already inscribed?

related: https://github.com/casey/ord/issues/1762


Testing | Add framework for E2E testing

Looks for a framework (and any required library) that will allow to perform E2E tests over the extension (preferability Cypress)

Hints:

https://docs.cypress.io/guides/tooling/plugins-guide#Browser-Launching
https://walrus.ai/blog/2021/02/simple-steps-for-testing-a-chrome-extension-in-puppeteer/

Requirements:

[x] Search for a framework (and any additional library) that will allow to perform E2E tests in the extension.
[x] Implement a simple E2E test for select a work on a page, open the context menu, and search it using the "Google" option.

Related to #57

I have chosen to use Puppeteer & Jest frameworks for e2e testing, as Cypress doesn't support protocols other than http and https (see this thread for more information).
Also note that the requirement Implement a simple E2E test for select a work on a page, open the context menu, and search it using the "Google" option. can't be implemented as the context menu opened in chrome is not part of the actual DOM, hence it can't be accessed. So, instead, I added a test for checking if an option name in options pages is changed when typing another value


Set up a "Hello world" project as one app with Webpack

In this milestone, I;

Implement a connection between a Ruby on Rails back-end and React front-end.

Created an App component with react-router and render it in your static view.

Created a Greeting component that will display a greeting. and set it up as a route in your App component.

Created a store, an action, and a reducer that will connect to my API endpoint to get the random greeting.

Displayed the random greeting in my Greeting component.

@addod19 im using a Windows OS so that's not my starting command, I've been trying for the past hours but couldn't get a solution to it. I use rails c  to start and it works fine at my end.

@addod19 can i meet you? i sent you a message on slack

@addod19 I left you a message on Slack after your reply. but I see your notifications are turned off

@addod19 here is how it looks like when I serve it with rails s


Permit numerical setting of baud rate

It would be welcome addition if all baud rates could be set with an integer via the BaudOther() type. Presently one must use Baud115200 instead of BaudOther(115200) but must use BaudOther(230400) instead of Baud230400. Is there a reason for this pattern?

See previous commentary on #10.

@tkornack The BaudRate enum exposes which baud rates defined by termios and the Windows API (B9600, B115200, etc) are cross-platform. There is a constructor to create an instance of the enum from arbitrary integers if that's what you prefer.


union and pagination creates wrong sql query making it unusable

I have a category table, and products table. The products can belong to up to two different categories, this is represented by having a category1_id and category2_id in the products table. I have a relationship in the category where I fetch all the products using eloquent. When using it, it results in an error that the select statements have a different number of columns because the sql creates one select count(*) as aggregate.. where..., and unions a select * where...

Why not create a middle table product_categories. In this table have the category_id, and product_id.
Then define the relationship as hasMany. This should solve all your issues. You won't need to have columns with two category_id and you can scale in the future much easier.
I would recommend readying the relationships of eloquent.
https://laravel.com/docs/5.2/eloquent-relationships

Using a middle table would require a join. Having a cat1 / cat2 id specified in the product makes sense - this is for ebay which only allows two categories.

@ChangePlaces The relationship structure you have defined is incorrect which is resulting in issues you are facing. Its not a framework issue, its simply an issue with your table relationship structure. I agree with @johnpaulmedina as It simply is a many-to-many relationship between categories & products.  You should have a pivot table called 'product_category' containing product_id & category_id. You should simply define relations in your eloquent models like this:

App\Category.php

public function products()
{
    return $this->belongsToMany(App\Product::class, 'product_category');
}

App\Product.php

public function categories()
{
    return $this->belongsToMany(App\Category::class, 'product_category');
}

The real root of the issue is I have a table that has two relationships with the same model with Eloquent can't deal with I assume. A perfectly reasonable table set up. There's no need for a middle table despite what normalization chapters will tell you. My schema is for efficiency, not an 'A' grade at college.

What are you calling that is results in the SQL error?

@ChangePlaces So if you have two relations with the same model. In case of product & category, here is what i think you are assuming:

A product has one primary category.
A product has one secondary category.
A primary category has many products.
A secondary category has many products.

To me, this structure simply doesn't makes sense. But if i have to set it up using eloquent, i may do something like this:

App\Category.php

public function primary_products()
{
    return $this->hasMany(App\Product::class, 'category1_id');
}

public function secondary_products()
{
    return $this->hasMany(App\Product::class, 'category2_id');
}

App\Product.php

public function primary_category()
{
    return $this->belongsTo(App\Category::class, 'category1_id');
}

public function secondary_category()
{
    return $this->hasMany(App\Category::class, 'category2_id');
}

Despite the table structure i guess we should at least consider this an issue because you could set up a task list, following the linked list oo pattern with two pointers:
//task table
id
name
next_id
previous_id

Both next_id and previous_id would relate to another task record and AFAIK this would also throw the error that @ChangePlaces is facing.

@fernandobandeira If we follow the link list oop pattern, then the relationship among tasks can be like:

App\Task.php

public function nextTask()
{
    return $this->hasOne(App\Task::class, 'next_id');
}

public function previousTask()
{
    return $this->hasOne(App\Task::class, 'previous_id');
}

I don't think the above would create a problem. I still think @ChangePlaces has not defined the relations properly in his models, thats why he is facing this issues. I don't think its a framework related error.

+1
In my case, I have 2 sale types inside products: BUY NOW or AUCTION.
When trying to do some filtering, I need to merge auction products and buy now products all together, and for this I require the union so I can do some where clauses to the auction products table.
I need to use union all since left join will bring me all products even with the where clauses inside the left join.
What happens is that paginate creates the wrong sql query and doesn't work with union all, so this needs to be addressed.

Updated with a solution.

@d0minationPT if you have a solution can you open a Pull Request?

Hello yes, I will create one.

Why was this closed?

This issue still exists, just ran into it when trying union'd pagination with GraphQL / Relay.
@gtapps this is how I added your (IMHO perfectly valid) solution in the latest 5.6 branch:
I don't mind creating a new PR for this, but since this is a verbatim copy of your solution, I think you should do this ;-)  Credit where credit is due..
@taylorotwell @GrahamCampbell can you elaborate why this never made it in? Are we overseeing something here?  The various proposed solutions (maka a custom Paginator or create a new model linking both models oa.) is just wrong IMHO since unions should work as documented and there's no reference in the docs that pagination with unions doesn't work out of the box
--- illuminate/database/Query/Builder.php	2018-05-24 12:08:01.241627889 +0200
+++ illuminate/database/Query/Builder.php	2018-05-24 12:12:33.305623092 +0200
@@ -1988,10 +1988,20 @@
      */
     protected function runPaginationCountQuery($columns = ['*'])
     {
-        return $this->cloneWithout(['columns', 'orders', 'limit', 'offset'])
-                    ->cloneWithoutBindings(['select', 'order'])
-                    ->setAggregate('count', $this->withoutSelectAliases($columns))
-                    ->get()->all();
+
+        if(count($this->unions) < 1) {
+            return $this->cloneWithout(['columns', 'orders', 'limit', 'offset'])
+                        ->cloneWithoutBindings(['select', 'order'])
+                        ->setAggregate('count', $this->withoutSelectAliases($columns))
+                        ->get()->all();
+        } else {
+            return $this->newQuery()
+                        ->selectRaw("COUNT(*) as aggregate FROM (".$this->toSql().") as aggregate_query")
+                        ->mergeBindings($this)
+                        ->get()->all();
+        }
+
+
     }

     /**

Did you do a PR?


Add mention about achieving zero-downtime rolling updates

This PR adds mention achieving zero-downtime rolling updates to Nginx Ingress Controller docs.
Based on investigation made in #322.

This change is

Coverage remained the same at 44.239% when pulling e16efc0a21107e5d172f92a9ed97e6a392fb269f on tyranron:nginx-rolling-restart-docs into 1468fcb1aaac9ad3c7d21e135a9e117a5d850f87 on kubernetes:master.

Coverage remained the same at 44.239% when pulling e16efc0a21107e5d172f92a9ed97e6a392fb269f on tyranron:nginx-rolling-restart-docs into 1468fcb1aaac9ad3c7d21e135a9e117a5d850f87 on kubernetes:master.

@tyranron I am not sure this is a good advice. If you use liveness and readiness probes + a strategy in the deployment, you don't need to use the hook
spec:
  replicas: 3
  strategy:
    rollingUpdate:
      maxSurge: 1
      maxUnavailable: 0

As I understand it, the concern is how long it takes nginx-ingress to notice when a pod becomes unready? how does kube-proxy handle it? Cause a pod can go unready for reasons other then deletion.

@kfox1111 good question. Please check this comment https://github.com/kubernetes/contrib/issues/1140#issuecomment-228934295 (and the next ones from thockin)

@aledbf that just does not work in most cases. It often works for simple single-container Pods, but for multi-container Pods it works quite rare. The reasons why this happens are discussed exactly in a issue you've referred above.
To refresh: Pod can still receiving new traffic after receiving SIGTERM, because in Kubernetes SIGTEM means not just "terminate gracefully", but "wait a sec, and terminate gracefuly". But most applications do not do that behavior out-of-the-box.
I understand that this solution is not ideal and looks quite strange. But this simple hack, proposed by @foxylion in #322  just does work in almost all cases. Even @thockin agreed a bit with that.

Yet another investigation/confirmation of situation in relative thread.

@aledbf We use the same strategy mentioned here, as there are some (ugly) java processes that are pretty damn slow to stop.
So, using a preStop command worked fine for us, as the POD is marked as 'stopping' and is removed from the Service previously to fully stop.
IMHO it's a good advice :)


UHF-X Helfi test content update

What was done

Fixed ckeditor buttons on test content
Added test content for all branding colours

How to install

Build a new test-site

composer create-project City-of-Helsinki/drupal-helfi-platform:dev-main test --no-interaction --repository https://repository.drupal.hel.ninja/

Update the Helfi Platform config

composer require drupal/helfi_platform_config:dev-UHF-X_helfi_test_content_update

Get the site up and running

make new

Run make shell

In shell run drush en -y helfi_test_content

How to test

[ ] Check that there is a new page for DC: Colors
[ ] Check that code follows our standards

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

Project coverage is 14.96%. Comparing base (b57b267) to head (cdd54b7).
Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main     #750   +/-   ##
=========================================
  Coverage     14.96%   14.96%
  Complexity      258      258
=========================================
  Files            31       31
  Lines           982      982
=========================================
  Hits            147      147
  Misses          835      835

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


Bug 1076 conversion error flattening 2

This is an alternative approach to PR https://github.com/stellar/rs-soroban-sdk/pull/1216 where we plumb the env::Error code all the way up to the surface of the SDK's UI.
I don't feel strongly about doing this or the other one, they both seem to work, it's a question of UI complexity. Neither approach suffers the problem that motivated this change at the env level, where we went from Error to ConversionError and then back to Error and thereby produced misleading Error codes.

decided against this one for now.


Create it.js

Mille grazie. Usate la pagna peer.school?

I updated the project to use your translation. Thanks!

Mille grazie. Usate la pagina peer.school?

yes, we using. I saw that the project is suspended, is it true?
it's a beautiful project and should grow with some new whiteboard features like adding shapes, objects, brushes, etc.

Ciao @albanobattistella Yes indeed I'm currently not putting much effort into the project any more, because I had the impression it was not used that much. Instead I continued video conferencing efforts in https://brie.fi/ng Of course some stuff could again be ported back to peer.school if there was some more traction on the project.
Is it used in a a school environment or something else? Did you also consider using https://bigbluebutton.org/ ? I would love to know more about how it is used.

yeah finally ich hab dich gefunden, Warst du nicht ebenfalls beim #WirVsVirus Hackaton dabei? 👍
Ich bin auch interessiert daran den code zu nutzen - ebenfalls für Bildungs-Zwecke. Allerdings will ich das ganze in andere Systeme einbetten um zeitgleiche sowohl zeitunabhängige kollaboration zu ermöglichen. Würd gern mal mit dir quatschen. mfg Christian --- Email<EMAIL_ADDRESS>

Hi @DigitEgal I'll write you via email.


Git Authentication not works in Portainer 1.19.1

Bug description
Git Authentication not works in Portainer 1.19.1
Expected behavior
In a swarm environment the stack creation of a yml file hosted in a git repository doesn't work. The GUI shows a error in upper right corner e in the logs it's showed:
2018/07/17 11:00:27 Starting Portainer 1.19.1 on :9000
2018/07/17 11:01:28 http error: Unable to clone git repository (err=repository already exists) (code=500)
The username and password are correct.
Briefly describe what you were expecting.
Steps to reproduce the issue:
Briefly describe what you were expecting.
Steps to reproduce the issue:
Steps to reproduce the behavior:

Go to stacks/Add Stack/git Repository
Enter Repository URL, Compose Path, Username ad Password.
SClick Deploy the Stack
See error

Technical details:

Portainer version: 1.19.1
Docker version (managed by Portainer): 18.03.0-ce, build 0520e24
Platform (windows/linux):linux
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer):

docker stack deploy --prune --compose-file /gfs/docker-compose/portainer.yml portainer --with-registry-auth
where /gfs/dc/portainer.yml is:
version: '3'
services:
portainer:
image: portainer/portainer:latest
ports:

"9000:9000"
volumes:
/etc/hosts:/etc/hosts
/gfs/portainer:/data

Browser:

Additional context
Add any other context about the problem here.

@stefano-gristina I believe that is caused by the previous issue that was fixed in 1.19.1. You need to remove the folder with the higher value in the /data/compose directory of Portainer to fix this and you should never encounter this issue again.

I deleted inside /data/compose the directory test_p6jf28m105ht3bs34lo3i6tdq that was the only with .git directory. Now I have this error:
2018/07/30 14:08:17 http error: Unable to clone git repository (err=object not found) (code=500)
This command works:
#git clone http://<IP_ADDRESS>:8999/repos/test.git
#cd test
#ls
stack-haproxy-01.yml
By Portainer, I put:
Repository URL: http://<IP_ADDRESS>:8999/repos/test.git
Compose path: stack-haproxy-01.yml
Username and password.
Best Regards
Stefano

@stefano-gristina you should remove a folder that is named with a integer value (such as 1, 2...)

@deviantony The folder with the highest number is 43:
[root@docker-04 compose]# ls -tlr
totale 84
drwx------. 2 root root 4096 17 ott  2017 3
drwx------. 2 root root 4096 17 ott  2017 4
drwx------. 2 root root 4096 17 ott  2017 6
drwx------. 2 root root 4096 28 feb 19.19 5
drwx------. 2 root root 4096 28 feb 19.19 2
drwx------. 2 root root 4096 11 apr 18.05 10
drwx------. 2 root root 4096 13 apr 18.59 12
drwx------. 2 root root 4096 20 apr 12.31 8
drwx------. 2 root root 4096 20 apr 15.56 9
drwx------. 2 root root 4096 25 apr 23.12 1
drwx------. 2 root root 4096 18 mag 14.33 11
drwx------. 2 root root 4096 28 mag 11.35 14
drwx------. 2 root root 4096 12 giu 12.26 16
drwx------. 2 root root 4096 19 giu 14.48 17
drwx------. 2 root root 4096 26 giu 09.55 18
drwx------. 2 root root 4096 29 giu 10.02 20
drwx------. 2 root root 4096 27 lug 10.48 32
drwx------. 2 root root 4096 27 lug 12.46 34
drwx------. 2 root root 4096 27 lug 13.12 35
drwx------. 2 root root 4096 27 lug 16.53 36
drwx------. 2 root root 4096 30 lug 17.48 43
[root@docker-04 compose]# ls -ltr 43
totale 11
-rw-------. 1 root root 10828 30 lug 17.57 docker-compose.yml
But docker-compose.yml contains the last stack added in swarm. If I delete this 43, I will lost it. Right it?

I'll need to push a specific version for you to help you debug this situation, what I think is happening is the following:

You're trying to create a new stack, Portainer will retrieve a new identifier for the stack from the database, let's say 43
It will then try to clone the git repository containing your compose file inside the /data/compose/43 folder
It will fail because this folder already exists

I'll push a specific version of Portainer for you to test tomorrow that will help us diagnose which identifier Portainer is trying to use.

Great thanks ! So, looks like this is not related to Portainer but rather to the go-git library (the error object not found is raised by the lib). Just wondering, is the master branch available in the git repo that you're trying to clone?

@deviantony .Yes.

Just wondering if this is related to your env, can you try to deploy a stack from a public git such as https://github.com/portainer/portainer-compose (use docker-stack.yml)?

It works..........

Ok, we'll try to reproduce with a private git repo.

@stefano-gristina Could you give us more details about the repository that you're trying to clone?

I can't reproduce. I did the following:

create a private repo out of https://github.com/portainer/portainer-compose
create a stack from this repo.
after this worked tried to create another stack from the same repo - succeed

This issue has been marked as stale as it has not had recent activity, it will be closed if no further activity occurs in the next 7 days. If you are able to reproduce this issue on the latest version of Portainer, feel free to leave a comment and we can pick up this issue again.

Since no further activity has appeared on this issue I will close it for now. If you are able to reproduce this issue on the latest version of Portainer, feel free to leave a comment and we can pick up this issue again.

The extra step might have been "have two factor authentication enabled". I have it enabled and when I try to create a stack, I get "Deployment error, authentication required" even though I specified the correct user and password.

The extra step in order to reproduce this might have been "have two factor authentication enabled" on the private repo. I have it enabled and when I try to create a stack, I get "Deployment error, authentication required" even though I specified the correct user and password.

I was able to clone a private repo with two factor enabled by generating a personal access token and using it instead of my password.

https://github.com/settings/tokens
Generate New Token button

@awickham10 I will add a note about this to our FAQ and may possibly introduce a tool-tip in the UI about using a PAT :+1:


Missing Quotes in _ConfigurationFile leads to failure

This lack of quotes in the _ConfigurationFile template will confuse the SQL installer when the SSAS feature is installed and lead to problems.
https://github.com/sous-chefs/sql_server/blob/61404aca55267913c4735470d2c44881880a5bb3/templates/_ConfigurationFile.ini.erb#L240
Should be:
ASSVCACCOUNT="<%= @as_svc_account %>"

Just as in the code in the above examples. Like here:
https://github.com/sous-chefs/sql_server/blob/61404aca55267913c4735470d2c44881880a5bb3/templates/_ConfigurationFile.ini.erb#L210

apparently, this is already reported


Support MLSDK 0.18.0

Fixes https://github.com/webmixedreality/exokit/issues/598.
Not many changes were needed.

This build system works.


Warnings on clustermesh startup

On clustermesh startup, we're getting a bunch of warnings as follows. Those probably shouldn't be warnings (or should be happening) given they're just signaling transitory startup issues while everything settles down.
msg="Unable to get remote cluster configuration" clusterName=c1 error="failed to retrieve cluster configuration: not found" hint="If KVStoreMesh is enabled, check whether it is connected to the target cluster. Additionally, ensure that the cluster name is correct."
msg="Unable to list keys before starting watcher" config=/var/lib/cilium/clustermesh/c1 endpoints="[https://clustermesh-apiserver.kube-system.svc:2379/]" error="context canceled" prefix=cilium/.heartbeat
msg="Error observed on etcd connection, reconnecting etcd" clusterName=cluster2-with-long-name-01234567 error="etcd cluster ID has changed: expected<PHONE_NUMBER>2166824766, got<PHONE_NUMBER>2270824672"
msg="Unable to list keys before starting watcher" config=/var/lib/cilium/clustermesh/cluster2-with-long-name-01234567 endpoints="[https://cluster2-with-long-name-01234567.mesh.cilium.io:32380/]" error="etcdserver: permission denied" prefix=cilium/.heartbeat

From https://github.com/cilium/cilium/actions/runs/11661644313.
cc @giorio94

I've opened https://github.com/cilium/cilium/pull/35867 to silence most spurious logs.
I'd honestly prefer to leave Error observed on etcd connection, reconnecting etcd at warning level, and ignore it in the CLI though. Although it is expected that it occurs if the remote clustermesh-apiserver is restarted, IMO it is still better to keep it visible given that it triggers expensive operations, and a high number of occurrences is typically a symptom of other problems.

What's a high number of occurrences? You know what I'm going to suggest next 😅
More seriously, I don't mind either way.

What's a high number of occurrences? You know what I'm going to suggest next 😅

Yes, I thought about having some sort of counter, or making it based on the time since the last occurrence, but it can get quickly pretty messy (especially because in the upgrade/downgrade tests it would fail a few times in a short time window) for relatively little value. I'd keep things simple and just ignore it.

Another one:
time="2024-11-08T11:32:23.000648396Z" level=warning msg="Error getting CEP" ciliumEndpointName=kube-system/clustermesh-apiserver-c8d8885c4-8chqs containerID=c6a50cc764 containerInterface=eth0 controller="sync-to-k8s-ciliumendpoint (1336)" datapathPolicyRevision=0 desiredPolicyRevision=0 endpointID=1336 error="Get \"[https://<IP_ADDRESS>:443/apis/cilium.io/v2/namespaces/kube-system/ciliumendpoints/clustermesh-apiserver-c8d8885c4-8chqs?resourceVersion=0\](https://<IP_ADDRESS>/apis/cilium.io/v2/namespaces/kube-system/ciliumendpoints/clustermesh-apiserver-c8d8885c4-8chqs?resourceVersion=0\)": context canceled" identity=77535 ipv4=<IP_ADDRESS> ipv6="fd00:10:242:1::b043" k8sPodName=kube-system/clustermesh-apiserver-c8d8885c4-8chqs subsys=endpointsynchronizer (1 occurrences)

Hmm, this is not clustermesh related actually. The fix looks pretty easy, likely just skipping the warning log if the error is context canceled here. However, I guess there will be quite a few more of this type that may come up from time to time.

Hitting this on a test doing downgrade to v1.14: https://github.com/cilium/cilium/actions/runs/12058380700/job/33624971131
Do we need to backport this to 1.14/1.15?

I see the fix is marked for backport, reopening until then 🙂


comments

[ ] https://github.com/kipoi/examples/blob/master/1-predict/load-visualize.ipynb I don't think a line plot makes sense, should be dots or bars in Average prediction per interval_set per model.
[ ] https://github.com/kipoi/examples/blob/master/2-score_variants/1-load-visualize.ipynb same here, I am not sure that the line plot is the best to compare models, since the order on the x-axis has no meaning
[ ] https://github.com/kipoi/examples/blob/master/5-appendix/choosing-right-models.ipynb in Let's load this model something is missing, at least some description

ValueError: Conda environment kipoi-shared\envs\kipoi-py3-keras2 doesn't exist
Show this error in my anaconda command prompt/

@iamjalees can you please provide more information? Specifically: what commands did you execute, platform, which Kipoi version are you using.

thank you for your reply: Yes I was trying to implement basenji model command to run the dataset file then I face that command error.


How to solve the problems?

Dear professor:
I have already install the ParallelWaveGAN,however,when I run the egs/vctk/voc1 run.sh, it comes with the error:
Stage 0: Data preparation
Successfully split data directory.
ERROR: num_first + num_second must be the same # utts in src. (20 vs 10)
I wonder if you can help me solve this problem,thank you very much.

Did you modify any parameters from default?

No, I did not modify the parameters.


Docs: add {array} import to example, add note about not using dependencies with form values, dont shadow form in example, update immer docs link

Released in 3.2.4


chore (workflows): update deploy all demos to run on version changes

What is the purpose of this pull request?

What problem is this solving?

Types of changes

[ ] Feat: (new functionality)
[ ] Bug fix: ([#ref number] non-breaking change which fixes an issue)
[ ] Chore: (improvements that will not reflect in production behaviour)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Requires change to documentation, which has been updated accordingly.

#135  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @keeganpotgieter and the rest of your teammates on  Graphite


TextField cannot be controlled input

Problem description
TextField updates the input field without onChange when used like the controlled example on the docs site.
Steps to reproduce
Same as the docs example, but make onChange not update value as below.
As a side note, I was not able to figure out how to load material-ui on jsbin or jsfiddle, but that might be worth looking into to make these issues easier for contributors to verify
import React from 'react';
import TextField from 'material-ui/TextField';

export default class TextFieldExampleControlled extends React.Component {

  constructor(props) {
    super(props);

    this.state = {
      value: 'Property Value',
    };
  }

  handleChange = (event) => {
    console.log(event)
  };

  render() {
    return (
      <div>
        <TextField
          id="text-field-controlled"
          value={this.state.value}
          onChange={this.handleChange}
        />
      </div>
    );
  }
}

Versions

Material-UI: 0.15.2
React: 15.2.1
Browser: latest chrome (51.0.2704.103)

I'm not sure to understand the issue. What's the expected result?

As a side note, I was not able to figure out how to load material-ui on jsbin or jsfiddle

Sadly, I'm not aware of any jsbin / jsfiddle working with Material-MI.

It is the documentation that is not correct.
I suppose the error message is "Missing class properties transform."
Change the documentation example to
constructor() {
this.handleChange = this.handleChange.bind(this);
}
handleChange() {
}
This way it works for me.

I just had a similar issue, except I was binding defaultValue={this.state.value} which wouldn't update when this.state.value updated. Make sure you're binding to value

Sorry for the delay. I expected the input value to not change, but I tried this again in a fresh project and it works fine. I'll check on the other issue I opened as well.


AttributeError: module 'spconv' has no attribute 'version'

Hi Yukang,
I was trying to run the evaluation script and am getting the following error.
--cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
+ NGPUS=2
+ PY_ARGS='--cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth'
+ python -m torch.distributed.launch --nproc_per_node=2 test.py --launcher pytorch --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
Traceback (most recent call last):
  File "test.py", line 14, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 1, in <module>
    from .detector3d_template import Detector3DTemplate
  File "../pcdet/models/detectors/detector3d_template.py", line 7, in <module>
    from ...utils.spconv_utils import find_all_spconv_keys
  File "../pcdet/utils/spconv_utils.py", line 4, in <module>
    if float(spconv.__version__[2:]) >= 2.2:
AttributeError: module 'spconv' has no attribute '__version__'
Traceback (most recent call last):
  File "test.py", line 14, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 1, in <module>
    from .detector3d_template import Detector3DTemplate
  File "../pcdet/models/detectors/detector3d_template.py", line 7, in <module>
    from ...utils.spconv_utils import find_all_spconv_keys
  File "../pcdet/utils/spconv_utils.py", line 4, in <module>
    if float(spconv.__version__[2:]) >= 2.2:
AttributeError: module 'spconv' has no attribute '__version__'
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 261, in <module>
    main()
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 257, in main
    cmd=cmd)
subprocess.CalledProcessError: Command '['/opt/conda/bin/python', '-u', 'test.py', '--local_rank=1', '--launcher', 'pytorch', '--cfg_file', '/mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml', '--ckpt', '/mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth']' returned non-zero exit status 1.

I am using the docker image that you provide. docker pull djiajun1206/pcdet:pytorch1.6
When I run nvcc --version in the docker image, I get CUDA version 10.1.
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2019 NVIDIA Corporation
Built on Sun_Jul_28_19:07:16_PDT_2019
Cuda compilation tools, release 10.1, V10.1.243

So, with CUDA 10.1 and pytorch 1.6, I am unsure which version of spconv to install

Your help will be much appreciated.
Thank you
Naren

Hi Naren,
You can try to directly remove these two sentences in the pcdet/utils/spconv_utils.py
if float(spconv.__version__[2:]) >= 2.2:
    spconv.constants.SPCONV_USE_DIRECT_TABLE = False

Regards,
Yukang Chen

Thanks Yukang. I tried that. It went past that error into another one.
root@369701605e02:/mnt/VoxelNeXt/tools# bash scripts/dist_test.sh 1 --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
+ NGPUS=1
+ PY_ARGS='--cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth'
+ python -m torch.distributed.launch --nproc_per_node=1 test.py --launcher pytorch --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
Traceback (most recent call last):
  File "test.py", line 14, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 1, in <module>
    from .detector3d_template import Detector3DTemplate
  File "../pcdet/models/detectors/detector3d_template.py", line 8, in <module>
    from .. import backbones_2d, backbones_3d, dense_heads, roi_heads
  File "../pcdet/models/backbones_3d/__init__.py", line 7, in <module>
    from .spconv_backbone_voxelnext_sps import VoxelResBackBone8xVoxelNeXtSPS
  File "../pcdet/models/backbones_3d/spconv_backbone_voxelnext_sps.py", line 4, in <module>
    from spconv.core import ConvAlgo
ModuleNotFoundError: No module named 'spconv.core'
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 261, in <module>
    main()
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 257, in main
    cmd=cmd)
subprocess.CalledProcessError: Command '['/opt/conda/bin/python', '-u', 'test.py', '--local_rank=0', '--launcher', 'pytorch', '--cfg_file', '/mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml', '--ckpt', '/mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth']' returned non-zero exit status 1.

It is missing spconv.core module. Are the dependencies not getting installed properly. I ran the python setup.py develop as well.

Hi,
What is your version of spconv?
Regards,
Yukang Chen

1.2.1 according to pip show spconv
Name: spconv
Version: 1.2.1
Summary: spatial sparse convolution for pytorch
Home-page: UNKNOWN
Author: Yan Yan
Author-email<EMAIL_ADDRESS>License: UNKNOWN
Location: /opt/conda/lib/python3.7/site-packages
Requires:
Required-by:

Please try a new spconv version. For example, pip install spconv==2.1.25

Hi Yukang,
I got a different error after pip install spconv==2.1.25.
bash scripts/dist_test.sh 2 --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
+ NGPUS=2
+ PY_ARGS='--cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth'
+ python -m torch.distributed.launch --nproc_per_node=2 test.py --launcher pytorch --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
Traceback (most recent call last):
  File "test.py", line 14, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 1, in <module>
    from .detector3d_template import Detector3DTemplate
  File "../pcdet/models/detectors/detector3d_template.py", line 8, in <module>
    from .. import backbones_2d, backbones_3d, dense_heads, roi_heads
  File "../pcdet/models/backbones_3d/__init__.py", line 2, in <module>
    from .spconv_backbone import VoxelBackBone8x, VoxelResBackBone8x
  File "../pcdet/models/backbones_3d/spconv_backbone.py", line 30, in <module>
Traceback (most recent call last):
  File "test.py", line 14, in <module>
    class SparseBasicBlock(spconv.SparseModule):
AttributeError: module 'spconv' has no attribute 'SparseModule'
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 1, in <module>
    from .detector3d_template import Detector3DTemplate
  File "../pcdet/models/detectors/detector3d_template.py", line 8, in <module>
    from .. import backbones_2d, backbones_3d, dense_heads, roi_heads
  File "../pcdet/models/backbones_3d/__init__.py", line 2, in <module>
    from .spconv_backbone import VoxelBackBone8x, VoxelResBackBone8x
  File "../pcdet/models/backbones_3d/spconv_backbone.py", line 30, in <module>
    class SparseBasicBlock(spconv.SparseModule):
AttributeError: module 'spconv' has no attribute 'SparseModule'
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 261, in <module>
    main()
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 257, in main
    cmd=cmd)
subprocess.CalledProcessError: Command '['/opt/conda/bin/python', '-u', 'test.py', '--local_rank=1', '--launcher', 'pytorch', '--cfg_file', '/mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml', '--ckpt', '/mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth']' returned non-zero exit status 1.

I tried other versions of spconv (CUDA variants) available in https://github.com/traveller59/spconv

I am getting a different runtime error with them.
bash scripts/dist_test.sh 2 --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
+ NGPUS=2
+ PY_ARGS='--cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth'
+ python -m torch.distributed.launch --nproc_per_node=2 test.py --launcher pytorch --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
Traceback (most recent call last):
  File "test.py", line 14, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 12, in <module>
    from .mppnet import MPPNet
  File "../pcdet/models/detectors/mppnet.py", line 9, in <module>
    from pcdet.datasets.augmentor import augmentor_utils, database_sampler
  File "../pcdet/datasets/__init__.py", line 15, in <module>
    from .argo2.argo2_dataset import Argo2Dataset
  File "../pcdet/datasets/argo2/argo2_dataset.py", line 11, in <module>
    from .argo2_utils.so3 import yaw_to_quat
  File "../pcdet/datasets/argo2/argo2_utils/so3.py", line 10, in <module>
    def quat_to_mat(quat_wxyz: Tensor) -> Tensor:
  File "/opt/conda/lib/python3.7/site-packages/torch/jit/__init__.py", line 1550, in script
    fn = torch._C._jit_script_compile(qualified_name, ast, _rcb, get_default_args(obj))
  File "/opt/conda/lib/python3.7/site-packages/torch/jit/_recursive.py", line 583, in try_compile_fn
    return torch.jit.script(fn, _rcb=rcb)
  File "/opt/conda/lib/python3.7/site-packages/torch/jit/__init__.py", line 1550, in script
    fn = torch._C._jit_script_compile(qualified_name, ast, _rcb, get_default_args(obj))
RuntimeError:
python value of type 'QuaternionCoeffOrder' cannot be used as a value:
  File "/opt/conda/lib/python3.7/site-packages/kornia/geometry/conversions.py", line 492
            raise ValueError(f"order must be one of {QuaternionCoeffOrder.__members__.keys()}")

    if order == QuaternionCoeffOrder.XYZW:
                ~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
        warnings.warn(
            "`XYZW` quaternion coefficient order is deprecated and"
'quaternion_to_rotation_matrix' is being compiled since it was called from 'quat_to_mat'
  File "../pcdet/datasets/argo2/argo2_utils/so3.py", line 19
        (...,3,3) 3D rotation matrices.
    """
    return C.quaternion_to_rotation_matrix(
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~...  <--- HERE
        quat_wxyz, order=C.QuaternionCoeffOrder.WXYZ
    )

Traceback (most recent call last):
  File "test.py", line 14, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 12, in <module>
    from .mppnet import MPPNet
  File "../pcdet/models/detectors/mppnet.py", line 9, in <module>
    from pcdet.datasets.augmentor import augmentor_utils, database_sampler
  File "../pcdet/datasets/__init__.py", line 15, in <module>
    from .argo2.argo2_dataset import Argo2Dataset
  File "../pcdet/datasets/argo2/argo2_dataset.py", line 11, in <module>
    from .argo2_utils.so3 import yaw_to_quat
  File "../pcdet/datasets/argo2/argo2_utils/so3.py", line 10, in <module>
    def quat_to_mat(quat_wxyz: Tensor) -> Tensor:
  File "/opt/conda/lib/python3.7/site-packages/torch/jit/__init__.py", line 1550, in script
    fn = torch._C._jit_script_compile(qualified_name, ast, _rcb, get_default_args(obj))
  File "/opt/conda/lib/python3.7/site-packages/torch/jit/_recursive.py", line 583, in try_compile_fn
    return torch.jit.script(fn, _rcb=rcb)
  File "/opt/conda/lib/python3.7/site-packages/torch/jit/__init__.py", line 1550, in script
    fn = torch._C._jit_script_compile(qualified_name, ast, _rcb, get_default_args(obj))
RuntimeError:
python value of type 'QuaternionCoeffOrder' cannot be used as a value:
  File "/opt/conda/lib/python3.7/site-packages/kornia/geometry/conversions.py", line 492
            raise ValueError(f"order must be one of {QuaternionCoeffOrder.__members__.keys()}")

    if order == QuaternionCoeffOrder.XYZW:
                ~~~~~~~~~~~~~~~~~~~~~~~~~ <--- HERE
        warnings.warn(
            "`XYZW` quaternion coefficient order is deprecated and"
'quaternion_to_rotation_matrix' is being compiled since it was called from 'quat_to_mat'
  File "../pcdet/datasets/argo2/argo2_utils/so3.py", line 19
        (...,3,3) 3D rotation matrices.
    """
    return C.quaternion_to_rotation_matrix(
           ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~...  <--- HERE
        quat_wxyz, order=C.QuaternionCoeffOrder.WXYZ
    )

Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 261, in <module>
    main()
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 257, in main
    cmd=cmd)
subprocess.CalledProcessError: Command '['/opt/conda/bin/python', '-u', 'test.py', '--local_rank=1', '--launcher', 'pytorch', '--cfg_file', '/mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml', '--ckpt', '/mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth']' returned non-zero exit status 1.

Hi,
I think this is the problem of kornia. Would you please check its version?
Regards,
Yukang Chen

Hi,
It was 0.5.7.
I uninstalled it and installed the latest one (0.6.11) using pip install kornia and reran the evaluation command. I got the following error (test.py failed).
 bash scripts/dist_test.sh 2 --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
+ NGPUS=2
+ PY_ARGS='--cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth'
+ python -m torch.distributed.launch --nproc_per_node=2 test.py --launcher pytorch --cfg_file /mnt/tools/cfgs/nuscenes_models/cbgs_voxel0075_voxelnext.yaml --ckpt /mnt/VoxelNeXt/voxelnext_nuscenes_kernel1.pth
/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py:188: FutureWarning: The module torch.distributed.launch is deprecated
and will be removed in future. Use torchrun.
Note that --use_env is set by default in torchrun.
If your script expects `--local_rank` argument to be set, please
change it to read from `os.environ['LOCAL_RANK']` instead. See
https://pytorch.org/docs/stable/distributed.html#launch-utility for
further instructions

  FutureWarning,
WARNING:torch.distributed.run:
*****************************************
Setting OMP_NUM_THREADS environment variable for each process to be 1 in default, to avoid your system being overloaded, please further tune the variable for optimal performance in your application as needed.
*****************************************
Traceback (most recent call last):
  File "test.py", line 14, in <module>
Traceback (most recent call last):
  File "test.py", line 14, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from pcdet.models import load_data_to_gpu
  File "../pcdet/models/__init__.py", line 6, in <module>
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 1, in <module>
    from eval_utils import eval_utils
  File "/mnt/VoxelNeXt/tools/eval_utils/eval_utils.py", line 8, in <module>
    from .detector3d_template import Detector3DTemplate
  File "../pcdet/models/detectors/detector3d_template.py", line 6, in <module>
    from ...ops.iou3d_nms import iou3d_nms_utils
from pcdet.models import load_data_to_gpu  File "../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 9, in <module>

  File "../pcdet/models/__init__.py", line 6, in <module>
    from . import iou3d_nms_cuda
ImportError: ../pcdet/ops/iou3d_nms/iou3d_nms_cuda.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe28TypeMeta21_typeMetaDataInstanceIdEEPKNS_6detail12TypeMetaDataEv
    from .detectors import build_detector
  File "../pcdet/models/detectors/__init__.py", line 1, in <module>
    from .detector3d_template import Detector3DTemplate
  File "../pcdet/models/detectors/detector3d_template.py", line 6, in <module>
    from ...ops.iou3d_nms import iou3d_nms_utils
  File "../pcdet/ops/iou3d_nms/iou3d_nms_utils.py", line 9, in <module>
    from . import iou3d_nms_cuda
ImportError: ../pcdet/ops/iou3d_nms/iou3d_nms_cuda.cpython-37m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe28TypeMeta21_typeMetaDataInstanceIdEEPKNS_6detail12TypeMetaDataEv
ERROR:torch.distributed.elastic.multiprocessing.api:failed (exitcode: 1) local_rank: 0 (pid: 114) of binary: /opt/conda/bin/python
Traceback (most recent call last):
  File "/opt/conda/lib/python3.7/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/opt/conda/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 195, in <module>
    main()
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 191, in main
    launch(args)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launch.py", line 176, in launch
    run(args)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/run.py", line 756, in run
    )(*cmd_args)
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launcher/api.py", line 132, in __call__
    return launch_agent(self._config, self._entrypoint, list(args))
  File "/opt/conda/lib/python3.7/site-packages/torch/distributed/launcher/api.py", line 248, in launch_agent
    failures=result.failures,
torch.distributed.elastic.multiprocessing.errors.ChildFailedError:
============================================================
test.py FAILED
------------------------------------------------------------
Failures:
[1]:
  time      : 2023-04-18_19:27:29
  host      : 97863fda069f
  rank      : 1 (local_rank: 1)
  exitcode  : 1 (pid: 115)
  error_file: <N/A>
  traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
------------------------------------------------------------
Root Cause (first observed failure):
[0]:
  time      : 2023-04-18_19:27:29
  host      : 97863fda069f
  rank      : 0 (local_rank: 0)
  exitcode  : 1 (pid: 114)
  error_file: <N/A>
  traceback : To enable traceback see: https://pytorch.org/docs/stable/elastic/errors.html
============================================================

Hi,
Have you ever installed OpenPCDet?
python3 setup.py develop
Sorry. I did not use docker.
Regards,
Yukang Chen

Yes, I did python3 setup.py develop
I was using the docker image you specified at https://github.com/dvlab-research/VoxelNeXt/tree/master/docker
docker pull djiajun1206/pcdet:pytorch1.6 . I guess that has problems then.

Hi, what is the output from python setup.py develop?
In addition, it seems that the problem is still in the installation of openpcdet.


link to assets from css

Please how do I link to assets from css
I have this in the css of a component

  background: url('../assets/bg.png') 0 0 no-repeat;

but it is not found

These docs should help you out...
https://github.com/vuejs-templates/webpack/blob/develop/docs/static.md#asset-resolving-rules


[harfbuzz] Build error on x64-linux

Package<EMAIL_ADDRESS>Host Environment

Host: x64-linux
Compiler: GNU 11.4.0
vcpkg-tool version: 2024-02-07-8a83681f921b10d86ae626fd833c253f4f8c355b
vcpkg-scripts version: 7f9f0e44d 2024-03-13 (13 hours ago)

To Reproduce
vcpkg install
Failure logs
-- Using cached harfbuzz-harfbuzz-8.2.0.tar.gz.
-- Cleaning sources at /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/src/8.2.0-52a1211f23.clean. Use --editable to skip cleaning for the packages you specify.
-- Extracting source /home/henrik/.vcpkg-clion/vcpkg2/downloads/harfbuzz-harfbuzz-8.2.0.tar.gz
-- Using source at /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/src/8.2.0-52a1211f23.clean
-- Getting CMake variables for x64-linux-dbg
-- Getting CMake variables for x64-linux-rel
-- Configuring x64-linux-dbg
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:112 (message):
    Command failed: /usr/bin/python3 -I /home/henrik/projects/project_name/cmake-build-relwithdebinfo/vcpkg_installed/x64-linux/tools/meson/meson.py -Dicu=disabled -Dgraphite=disabled -Dcoretext=disabled -Dglib=disabled -Dgobject=disabled -Dfreetype=enabled -Dcairo=disabled -Ddocs=disabled -Dtests=disabled -Dbenchmark=disabled -Dintrospection=disabled --backend ninja --wrap-mode nodownload -Dbuildtype=plain -Doptimization=plain --libdir lib --native /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/meson-x64-linux-dbg.log -Ddebug=true --prefix /home/henrik/.vcpkg-clion/vcpkg2/packages/harfbuzz_x64-linux/debug --includedir ../include -Dcmake_prefix_path=['/home/henrik/projects/project_name/cmake-build-relwithdebinfo/vcpkg_installed/x64-linux/debug','/home/henrik/projects/project_name/cmake-build-relwithdebinfo/vcpkg_installed/x64-linux'] /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/src/8.2.0-52a1211f23.clean
    Working Directory: /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/x64-linux-dbg
    Error code: 1
    See logs for more information:
      /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/config-x64-linux-dbg-meson-log.txt.log
      /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/config-x64-linux-dbg-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_meson.cmake:436 (vcpkg_execute_required_process)
  buildtrees/versioning_/versions/harfbuzz/d804324af44593c0877fdf035cda178123d9c87f/portfile.cmake:53 (vcpkg_configure_meson)
  scripts/ports.cmake:175 (include)

/home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/config-x64-linux-dbg-out.log
WARNING: Recommend using either -Dbuildtype or -Doptimization + -Ddebug. Using both is redundant since they override each other. See: https://mesonbuild.com/Builtin-options.html#build-type-options
The Meson build system
Version: 0.62.1
Source dir: /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/src/8.2.0-52a1211f23.clean
Build dir: /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/x64-linux-dbg
Build type: native build

../src/8.2.0-52a1211f23.clean/meson.build:1:0: ERROR: Value "plain" (of type "string") for combo option "Optimization level" is not one of the choices. Possible choices are (as string): "0", "g", "1", "2", "3", "s".

A full log can be found at /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/x64-linux-dbg/meson-logs/meson-log.txt

/home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/config-x64-linux-dbg-meson-log.txt.log
Build started at 2024-03-14T14:45:43.121765
Main binary: /usr/bin/python3
Build Options: -Dicu=disabled -Dgraphite=disabled -Dcoretext=disabled -Dglib=disabled -Dgobject=disabled -Dfreetype=enabled -Dcairo=disabled -Ddocs=disabled -Dtests=disabled -Dbenchmark=disabled -Dintrospection=disabled -Dbuildtype=plain -Doptimization=plain -Ddebug=true '-Dcmake_prefix_path=['"'"'/home/henrik/projects/project_name/cmake-build-relwithdebinfo/vcpkg_installed/x64-linux/debug'"'"','"'"'/home/henrik/projects/project_name/cmake-build-relwithdebinfo/vcpkg_installed/x64-linux'"'"']' -Dprefix=/home/henrik/.vcpkg-clion/vcpkg2/packages/harfbuzz_x64-linux/debug -Dincludedir=../include -Dlibdir=lib -Dbackend=ninja -Dwrap_mode=nodownload '--native-file /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/meson-x64-linux-dbg.log'
Python system: Linux
The Meson build system
Version: 0.62.1
Source dir: /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/src/8.2.0-52a1211f23.clean
Build dir: /home/henrik/.vcpkg-clion/vcpkg2/buildtrees/harfbuzz/x64-linux-dbg
Build type: native build

../src/8.2.0-52a1211f23.clean/meson.build:1:0: ERROR: Value "plain" (of type "string") for combo option "Optimization level" is not one of the choices. Possible choices are (as string): "0", "g", "1", "2", "3", "s".

Additional context
vcpkg.json
{
  "name": "project-name",
  "version-string": "1.0.0",
  "builtin-baseline": "2c401863dd54a640aeb26ed736c55489c079323b",
  "dependencies": [
    {
      "name": "eigen3",
      "version>=": "3.4.0#2"
    },
    {
      "name": "geographiclib",
      "version>=": "2.3#1"
    },
    {
      "name": "gtest",
      "version>=": "1.14.0"
    },
    {
      "$comment": "    # this is heuristically generated, and may not be correct\n\n    find_package(manif CONFIG REQUIRED)\n\n    target_link_libraries(main PRIVATE MANIF::manif)\n",
      "name": "manif",
      "version>=": "2023-07-17"
    },
    {
      "$comment": "    # this is heuristically generated, and may not be correct\n\n    find_package(OpenCV CONFIG REQUIRED)\n\n    # note: 15 additional targets are not displayed.\n\n    target_link_libraries(main PRIVATE opencv_ml opencv_dnn opencv_core opencv_flann)\n",
      "name": "opencv",
      "default-features": false,
      "features": [
        "ffmpeg"
      ],
      "version>=": "4.8.0"
    }
  ]
}

Will be fixed in #37396

Nice! Do you have a guess for when it will be merged in? I will have to find a workaround if it takes a while.

Already merged.


proposal: html/template: Better handling of HTML comments

Background
The html/template package completely strips HTML comments from the rendered output. This is apparently intentional (#14256) but is not documented (#28628). The best summation of the rationale I could find comes from the mailing list:

That would not be safe. html/template needs to know the proper context at each point of the template evaluation, but these types of comments cause their contents to be in a browser-dependent context, so the autoescaping can not be done reliably.

To wit, "these types of comments" seems to refer to old-school conditional comments like . I don't know if these comments even do anything anymore on modern browsers, and a quick Google search wasn't very revealing.
The canonical solution to this problem seems to be to use the template.HTML type to wrap "trusted" strings. For example, there's this StackOverflow answer recommending this approach. However, this means that only way to use comments literally in the template is via function-call indirection, e.g.:
{{ safe "" }}

This also imposes the limitation that comments cannot have injected values within them.
Rationale for Change
I contend that based upon my own and a number of others' experiences, this behavior is unexpected. First of all, there are uses for comments besides affecting the browser's behavior (e.g. including debugging info useful to the developer). Second, many will have never seen or used IE "conditional comments" in no small part because IE is (almost) dead. Third, if the rationale for this behavior is to address the specific concern of conditional context confounding the auto-escaping mechanism, then I would think that only those comments which raise that issue would be stripped, not all comments. Fourth, of course, is the total lack of documentation of this behavior in the package itself.
There is another proposed "solution" already which is to fall back to using text/template but that is obviously a dangerous regression as then one gives up on all the benefits of html/template.
Though server-side manipulation of HTML is falling by the wayside in favor of script-heavy web apps doing extensive DOM manipulation client-side, the html/template package is not deprecated, and all browsers still (mostly) support "Web 1.0" sites that have minimal or no scripts.
Finally, the package documentation does say the following:

The security model used by this package assumes that template authors are trusted, while Execute's data parameter is not.

Whereas the policy of stripping comments seems to contradict this maxim.
Thus I believe this unintuitive behavior is ripe for change, albeit ideally in a backwards-compatible way.
Possible Paths

Add a new built-in function to the template language: Basically take the SO answer and bless it as canonical, eliminating the need for the user to define a function on their own. This is probably the most conservative approach, especially if the user is allowed to override the built-in function with one of their own (since otherwise there could be a naming conflict from old code), however I also consider this the least beneficial solution, since you still can't safely inject values into comments.

An improved approach would be to add a comment function that takes a string argument, escapes it properly for the body of a comment, surrounds it with comment markup, and finally wraps the result in template.HTML, this way you could at least inject values safely but indirectly with printf

Just stop stripping comments altogether. If there's a literal comment in the source, pass it through, injecting properly escaped values if there are any. Put a tombstone on IE while you're at it. This is probably the most aggressive approach and could have thorny repercussions if turned on by default.
Fail to compile any template containing HTML comments. The author probably put the comments there on purpose, and may not even be aware that they're being stripped. The template language has its own comment syntax which can be used for local-only comments. This replaces one unexpected outcome (comments just disappear) with another (some templates that used to compile now don't) but IMO reduces the amount of surprise involved.
Narrow the scope of the comment stripping to only those comments known to be problematic. I don't know if there are other kinds of problematic comments besides the aforementioned conditional comments from IE.
Introduce a new block to the template language inside of which comments are allowed. This improves on idea 1 by allowing safely escaped value injection.

Any of the above suggestions could also be gated, e.g. with template.Option, to improve backwards compatibility.

Well, I just discovered an interesting wrinkle that could undermine some of the suggestions: there doesn't seem to be a good rule for escape the things you put inside of comments

@kbolino I'm not clear: are you retracting the proposal?

No, at least not without some kind of resolution (even if it is merely to finally document this behavior). I thought a bit on this last night, and putting some of the other ideas along with that unfortunate discovery together, I'd propose a middle-ground solution:
Define a template option htmlcomments which takes one of the following values:

remove (= default) which corresponds to the current behavior
error which causes an error to be returned (ideally by the Parse* functions, i.e. as early as possible) if an HTML comment is present in the template
allow which enables HTML comments to pass through to rendered output, but Execute will still fail with an error if a value substitution inserts -- or > into a comment block

I have no idea how feasible this solution is, but could do some code research into whether it would be possible.

We have interest in resolution of this issue - can we help with resolving this? Currently we stumbled upon this issue because we are using html/template for mail templates where  comments is being stripped out of my HTML that I generate, causing broken emails for outlook users :roll_eyes: Is there much likelihood that this proposal going to be followed up?

Here's an adaptation of the suggestion from the StackOverflow answer, which seems to work fine in the playground:
const src = `

Some HTML content

{{ ifmso }}

Microsoft Outlook Only

{{ endif }}
`

func main() {
	t := template.Must(template.New("").Funcs(template.FuncMap{
		"ifmso": func() template.HTML { return template.HTML(`<!--[if mso|IE]>`) },
		"endif": func() template.HTML { return template.HTML(`<![endif]-->`) },
	}).Parse(src))
	t.Execute(os.Stdout, nil)
}

Caveat emptor: The docs for template.HTML specifically say not to use it for unterminated comments, so YMMV.
The right solution here of course is to bully Microsoft as much as possible into dropping its antiquated email rendering engine, and/or getting clients off of Microsoft Outlook. But that is beyond the scope of what Go can address.

If Go could get users off of Outlook it would truly be a marvel 😅
The trouble I had is that I'm using MJML to generate my email html templates and it isn't simple (nor very nice for the next dev) to have to go through and replace the comments it applies with the something like you've suggested. But I appreciate that for a lot of situations that is a decent workaround.
Thanks!


Revert monitoring value back to 8 minutes.

We changed the value in ad9dd9 to cause less alerts
to be raised for a self correcting problem. This change didn't
prevent issues so we're reverting it back to the older, more tested,
value.

:th


Pass through lowerCaseAttributeNames to htmlparser2

Coverage remained the same at 100.0% when pulling da010a89e0721ff667baaabed27c210fb9029f8f on dcollien:patch-1 into f28597ee0d010d4142d0077c911868de69535efd on wrakky:master.

Coverage remained the same at 100.0% when pulling da010a89e0721ff667baaabed27c210fb9029f8f on dcollien:patch-1 into f28597ee0d010d4142d0077c911868de69535efd on wrakky:master.


add new plugin hexo-generator-issues

The Hexo plugin publish your posts to GitHub issues.

Thanks for the PR. 😄


remove torch and numpy dependencies of limbus

Changes

Fixes # (issue)
Type of change

[ ] 📚  Documentation Update
[ ] 🧪 Tests Cases
[ ] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🔬 New feature (non-breaking change which adds functionality)
[ ] 🚨 Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] 📝 This change requires a documentation update

Checklist

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Did you update CHANGELOG in case of a major change?

Shouldn’t be the setup.py also updated to remove dependencies?

it is done. Only typeguard is a dependency now if you use the base Component.