HomePage fixed and always showing

Hi, I`m trying to create a homepage tab that stays in place even if there is a lot of tabs opened (outside the ScrollViewer). I had 2 ideas but both failed.
-First tried to change DragablzItemsControl so it displays a DragablzItem bound to the first Item, but the clicks events were not working.
-Next try was to use the HeaderPrefix to add a fake tab with the same style, but then again I didn't know how to bind the action and the IsSelected.

You can use the FixedHeaderCount, but yes, this is would scroll out of view.
So yeah, the header prefix is your best bet.  You could put a ToggleButton in there, we showed a Grid which is laid over the main content of the tab control.

But i still don't know what event I bind to the click and how do i know if the homepage is open because there is many ways besides clicking, for example closing a tab or dragging then out.

I found the solution i used the SelectedIndex and used a trigger to set the style if the SelectedIndex is 0.

:+1:

Hey i got another problem. The new windows that is created when dragging out a Tab must have the home page, but the problem is that when a window have 2 or more tabs when you try to drag a tab it creates a new window and this new windows have the dragged out tab and the home page. This way is impossible to drag the tab to a existing window, it always create a new window. How do I solve this?

dont quite understand

In the ItemTabClient I've add the home page in every new Window, so every new window have 2 tabs, instead of the default that is one. And by default when you got 2 tabs in a window and try to drag the tab it creates a new window.
The only way to merge a window it to drag a window with only one tab ta a existing window.
But when i try to drag a tab it creates a new window with 2 tabs, and this window is impossible to merge merge with a existing because it have 2 tabs.

One way to fix this is my home page not being really a tab.
Is there a way to unselect a tab?

I fixed this by making my home page not a Tab but a control that is display above the tabs control when selected and below if not selected.


As a user, I want to have the Kerth winners listed on the website, not just on the message boards

Presently, the Kerth winners for 2018 - 2023 are only held on the lcficmbs.com message boards. They should be copied to the Kerth site.

[ ] Copy the Kerth winners for 2018 to the Kerth site
[ ] Copy the Kerth winners for 2019 to the Kerth site
[ ] Copy the Kerth winners for 2020 to the Kerth site
[ ] Copy the Kerth winners for 2021 to the Kerth site
[ ] Copy the Kerth winners for 2022 to the Kerth site
[ ] Copy the Kerth winners for 2023 to the Kerth site

relates to #16

followed by #17


[Fonts] Icon request: currency_rupee

Hi @Ali-codesweb, Thanks for filing the issue, the above icon is already present as of lateststable 2.10.3 .

Closing as the icon is present. In case you disagree feel free to write in the comments and we will. reopen it.


org.springframework.social.vkontakte.api.IWallOperations does not expose filter to filter on type of posts to return

Per http://vk.com/dev/wall.get the Wall API has a filter field which allows to get posts by owner, others, all (the default) and postponed.  The org.springframework.social.vkontakte.api.IWallOperations does not expose this.

The filter also has a suggests value

I just tested using the current Spring Social API and trying filtering out only owner posts and I got 93 out of a 100 messages yet when I use the VK API straight with the owner filter I got the last 100 messages from the owner. It would be great to be able to get the desired type of post since one can get more in one call (as seen from above test).

Since library depends on VK API SDK closing this. Refer to SDK.


Add quotes to cluster name in kurberos

In our environment clusters have number names. 10, 110 etc.
If i put it to kuberos configuration it will be converted to name: 10 , without quotes. Such yaml will conver to json with 10 as number, not a string.
When kuberos will try to load this configuration, json decoding will fail because it expects string there.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

In our environment clusters have number names. 10, 110 etc.
If i put it to kuberos configuration it will be converted to name: 10 , without quotes. Such yaml will conver to json with 10 as number, not a string.
When kuberos will try to load this configuration, json decoding will fail because it expects string there.
Signed-off-by: Alexander Shemshurenko<EMAIL_ADDRESS>


add function 'getLastCommit' in GHRepository.java

What a useful and interesting project, I hope to contribute some power.
Just added a method

@MagicLu550  I'm going to close this for now.  Please feel free to request reopen when ready.


Getting issue in Order, Customer and Catalog Datagrid, unable to see the details in some specific locales.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
Master
Issue Description
When Checking the Admin Panel and there  in  the Sales Order section, Customers customer Section and Catalog Product Section there it was found that the datagrid is having issue. Data grid details are not appearing.
Issue Pages
Catalog - Products
Customers - Customer
Sales - Orders
Locales -> Chinese - zh_CN, Hindi - hi_IN, Portuguese - pt_BR
Preconditions

While Installation all locales must be added.

Catalog - Products, Customers - Customer, Sales - Orders
See error, we are unable to see the data grid details.

Expected Result
We should be able to see the details.
Actual Result
Please check the video and screenshot
Video
https://github.com/bagisto/bagisto/assets/92721837/fa61f04c-6118-45e1-83e8-d9a0a97707b6
Screenshot

Screenshots
No response
Additional context
No response

Checked and Found that the issue has been fixed and working fine, now for all locales we are able to see the pages clearly. Please check the video for reference.
Video
https://github.com/bagisto/bagisto/assets/92721837/bf5526f4-1756-48de-b72c-9fc7e5fca8cb


Using monospace in the values of the variable viewer

This PR changes the font family of the cells in the table of the variable viewer to use monospace. Looks as follows:

You can compare that screenshot with the screenshot available in the related issue.
Fixes #10223

Codecov Report

Merging #10724 (fc7bf97) into main (d701ada) will increase coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #10724   +/-   ##
======================================
  Coverage     62%      62%
======================================
  Files        476      476
  Lines      32985    32985
  Branches    5356     5356
======================================
+ Hits       20732    20746   +14
+ Misses     10261    10256    -5
+ Partials    1992     1983    -9

Impacted Files
Coverage Δ

src/notebooks/outputs/tracebackFormatter.ts
93% <0%> (-7%)
:arrow_down:

.../standalone/survey/dataScienceSurveyBanner.node.ts
66% <0%> (-6%)
:arrow_down:

...rc/kernels/debugger/debugLocationTrackerFactory.ts
96% <0%> (-4%)
:arrow_down:

src/platform/common/process/baseDaemon.node.ts
50% <0%> (-2%)
:arrow_down:

src/platform/msrCrypto/msrCrypto.js
16% <0%> (-1%)
:arrow_down:

src/kernels/raw/launcher/kernelProcess.node.ts
70% <0%> (+<1%)
:arrow_up:

...lPythonAndRelatedNonPythonKernelSpecFinder.node.ts
71% <0%> (+<1%)
:arrow_up:

...ctive-window/editor-integration/codeLensFactory.ts
89% <0%> (+<1%)
:arrow_up:

src/platform/common/utils/async.ts
51% <0%> (+<1%)
:arrow_up:

src/kernels/kernelDependencyService.node.ts
86% <0%> (+<1%)
:arrow_up:

... and 6 more

--debug-console-font-family makes it look a bit too times-new-romany. Not sure what to do next.

So we only have access to some variables in a webview I don't believe the --debug-console-font-family is one of them. I was just referring to the VS Code setting, but I'm not 100% sure that's the correct setting. Looking at VS Code it's using 'SF Mono' via --monaco-monospace-font.

So we only have access to some variables in a webview I don't believe the --debug-console-font-family is one of them. I was just referring to the VS Code setting, but I'm not 100% sure that's the correct setting. Looking at VS Code it's using 'SF Mono' via --monaco-monospace-font.

Bleh, yeah, looking at the setting for debug.console.fontFamily it just pulls default so we can't load that setting from the extension side and send it to the webview.

@IanMatthewHuff thank you, Ian!

that we want debug.console.fontFamily per my other comment

id suggest filling a feature request and or subjugating a pr upstream.
i know @rebornix exposed some css variables for notebook output renderers too work as well. we could do the same here. else using settings feels more work

@DonJayamanne , @IanMatthewHuff I'll close this PR and add a comment on the issue to add a feature request @ upstream.


Elevate user privs in UI or otherwise

Hi Brad,
We just upgraded to the latest drone today.   Looking great!  I notice the Settings page is blank.  I'm trying to figure out how to give other users Drone admin privs in the UI (so they can see settings and user administration).  Is there a command-line method?

finished. added to UI and rest API in 0.4 branch

Where is the 0.4 branch? I see only master, bolt and revert-751-pr1
I also don't see the relevant commit in https://github.com/drone/drone/commits/master


Canceling ChangeBlockEvent.Place Stack Issues

@ImMorpheus
When canceling ChangeBlockEvent.Place and you used a carrot (basically planting the carrot), it does stop the carrot from being placed but eats one of the carrot's stack in the inventory.

@ImMorpheus Status?

Tested that a bit more with clean SpongeForge 3728 with just GP 689 and Luckperms. The issue seems only apply to the mentioned ChangeBlockEvent.Place. For example, Blocks are not being eaten when placing them into itemframes even if its canceled


(maint) Fix spelling errors and typos in lib/puppet

Fix low-priority errors in comments, strings, and error messages.

CLA signed by all contributors.

@gguillotte unfortunately there are conflicts. Could you rebase and we'll merge?

@joshcooper Done!

The appveyor failure is due to PUP-6473, I'll close this and reopen, to pick up the change.


chore: upgrade node to LTS

Close #795

About the comment on #767
Not sure if it was just to move node-fetch dependency to devDependencies on the providers workspace or just to remove its import on providers test file


sst-env.d.ts invalid for bound workers

I am on version 0.0.481
With this config:
{
  const bucket = new sst.cloudflare.Bucket("MyBucket");

  const workerB = new sst.cloudflare.Worker("WorkerB", {
    handler: "./src/worker-b.ts",
    link: [bucket],
    url: true,
  });

  const workerA = new sst.cloudflare.Worker("WorkerA", {
    handler: "./src/worker-a.ts",
    link: [workerB, bucket],
    url: true,
  });

  return {
    url: workerA.url,
  };
}

The generated types clash, with a TS error

Please let me know if any other information would be helpful 👍

If helpful: still exists on sst 3.0.54


drivers: gpio: xlnx: Add interrupt support

Added support for interrupt-driven GPIO for this driver if an interrupt line is defined. Note that interrupt support is optional in the GPIO core so it may or may not be present.
The core provides a single state change notification whenever any line changes state on a given bank, so we need to check the states of the lines ourselves to determine what has actually changed.
Also, the driver treats the first and second GPIO banks as mostly separate devices, but a single interrupt is shared by both banks, so the ISR ends up being registered by the first bank which also triggers handling on the second bank if required.

Pushed a change to the DT schema to fix a compliance error.

Please make all these additions compile-time #ifdef'ed based on whether any instance has an interrupt line configured. You can use https://github.com/zephyrproject-rtos/zephyr/blob/main/drivers/can/transceiver/can_transceiver_gpio.c#L16 as inspiration.

I've added this in to make the interrupt support conditional. I also noticed another issue where some of the DT properties being read for the second channel were incorrect, I've added in a fix for that in a separate commit.

Rebased onto the updated main branch.

@henrikbrixandersen I think your requested changes have been addressed now.


unable to craft Misty wood planks from timbers

When I try to craft planks from Misty timbers, all I get is the crafting table. The recipes are conflicting.

As a workaround, you can craft the slabs first, and then the planks from the slabs.


Label scaling

Firstly thank you so much for this. I've been in need of a script like this for a long time.
Would there be a way to increase the size the labels? Or even take into consideration of working at different scales? I typically work in a fairly large format so typically things will be a 10th of their actual size (so something that is 24", will read in AI as 2.4").
Thank you!

There are a couple places where you can modify the code in the Specify.jsx file to accomplish what you're wanting:
To increase the font size, change the value of line 284, to whatever size works best for your typical use case:
t.textRange.characterAttributes.size = 8; // Default is 8

You can also change the default spacing for measurement lines, as well as the number of decimals in the measurement, and pixels per inch on lines 55 - 64:
// gap between measurement lines and object
var gap = 2;

// size of perpendicular measurement lines.
var size = 6;

// number of decimal places in measurement
var decimals = 3;

// pixels per inch
var dpi = 300;


throw error in build time when target type has not mapped all properties.

for example;
source type => CarDto {
int Id;
Color Color;
}
target type => Car{
int Id;
Color Color;
decimal Price;
}
And i want to throw a error in build time because Price was not mapped. I can only do it now via property on Mapper Attribute (ThrowOnPropertyMappingNullMismatch = true) but sadly it's in runtime.

Probably what you are looking for are strict mappings.

thanks, solved. Im blind :D


Get rid of the core repository and use media repositories as well.

As soon as possible I'd like to remove the media repository from core and pull it via the media repository feature introduced by @aschaeffer

Do we not care about old Inexor versions not working anymore or do we somehow make sure to checkout some specific version of the media repositorys?
Note: This speeds up nightly packaging and uploading and saves space.

We could add a release tag for this as well. I wouldn't add "stability" as a priority for alpha releases though

already done in 4a09301bd2ab77b77cbf101927aa43752e77b2dd


Add option to replace empty path

I have an rtsp client which pulls the stream from the empty path (rtsp://<ip>/). I am not happy about this, but I can't change it at this time, so I need rtsp-simple-server to allow this.
To support this without changing a lot of code, I implemented a new config value emptyPathReplace, which, when set, the empty path is replaced with. For example when set to test, a client requesting / sees the stream published to /test without knowing anything about the replacement. This works for my use case.
The empty path seems to be nothing completely unknown, VLC for example does support this as a client too.
I know this is a quite special pull request and I am uncertain if this feature is implemented in a good way or even if this should be merged at all. I just wanted to commit this idea in case there are others with this problem. For me, this is currently solved in my fork, so I would be fine with all options:

You can reject this PR at all if you do not want to support this in rtsp-simple-server, I would not be surprised if there a good reasons against this feature as I don't really like it myself
You can give me suggestions for a better implementation of this feature (I am not a go developer)
You can do any changes to my implementation

Thank you!

Codecov Report

Merging #1476 (2347742) into main (f1455a6) will increase coverage by 0.17%.
The diff coverage is 25.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main    #1476      +/-   ##
==========================================
+ Coverage   65.34%   65.52%   +0.17%
==========================================
  Files         118      118
  Lines       12977    12985       +8
==========================================
+ Hits         8480     8508      +28
+ Misses       3827     3803      -24
- Partials      670      674       +4

Impacted Files
Coverage Δ

internal/conf/conf.go
73.70% <ø> (ø)

internal/core/path_manager.go
89.61% <14.28%> (-1.76%)
:arrow_down:

internal/core/core.go
84.28% <100.00%> (+0.02%)
:arrow_up:

internal/core/hls_server.go
76.63% <0.00%> (-0.41%)
:arrow_down:

internal/core/rtsp_source.go
49.69% <0.00%> (ø)

internal/core/webrtc_conn.go
55.22% <0.00%> (+0.33%)
:arrow_up:

internal/core/webrtc_server.go
53.70% <0.00%> (+0.92%)
:arrow_up:

internal/core/rtmp_source.go
67.12% <0.00%> (+2.05%)
:arrow_up:

internal/core/path.go
72.05% <0.00%> (+2.42%)
:arrow_up:

internal/core/source_static.go
75.00% <0.00%> (+3.33%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Hello,
In order to allow a RTSP client that use the empty path to interact with a RTSP server, you can build and use the server example of gortsplib that is available here:
https://github.com/aler9/gortsplib/blob/main/examples/server/main.go
gortsplib was recently updated in order to support empty paths.
I don't like empty paths too, since paths are the unique ID that allow to identify a stream between different protocols (RTSP, RTMP, HLS, WebRTC, etc...). That's why i prefer not to include this feature into the server, for the moment, but if there's more support for it in the future i'll reconsider my position.


python3Packages.{mocket,samsunsgtvws,segments,csvw: update

Motivation for this change

https://github.com/xchwarze/samsung-tv-ws-api/releases/tag/v1.6.0
https://github.com/cldf/csvw/blob/fbeb899e2af7d10221fe1230d56aba6e33fcda12/CHANGES#L14-L17
https://github.com/mindflayer/python-mocket/releases/tag/3.9.36
https://github.com/mindflayer/python-mocket/releases/tag/3.9.37
https://github.com/mindflayer/python-mocket/releases/tag/3.9.38
https://github.com/mindflayer/python-mocket/releases/tag/3.9.39
https://github.com/cldf/segments/blob/master/CHANGES.md#220---2021-01-11

Things done

[x ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 109515 run on x86_64-linux 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

1 package failed to build:

tts

22 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.geoip2
python37Packages.mocket
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python37Packages.sopel
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

Result of nixpkgs-review pr 109515 run on x86_64-linux 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

1 package failed to build:

tts

22 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.geoip2
python37Packages.mocket
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python37Packages.sopel
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

Trying to get the mocket tests running, bear with me.

Trying to get the mocket tests running, bear with me.

Result of nixpkgs-review pr 109515 run on x86_64-linux 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

22 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.geoip2
python37Packages.mocket
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python37Packages.sopel
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

Result of nixpkgs-review pr 109515 run on x86_64-linux 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

22 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.geoip2
python37Packages.mocket
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python37Packages.sopel
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109515 run on x86_64-darwin 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

22 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.geoip2
python37Packages.mocket
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python37Packages.sopel
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.mocket:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'setup_requires=["pipenv"]' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python37Packages.samsungtvws:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'websocket-client==' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.mocket:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'setup_requires=["pipenv"]' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.samsungtvws:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'websocket-client==' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.mocket:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'setup_requires=["pipenv"]' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.samsungtvws:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'websocket-client==' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109515 run on x86_64-darwin 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

22 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.geoip2
python37Packages.mocket
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python37Packages.sopel
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

The following issues got detected with the above build packages.
Please fix at least the ones listed with your changed packages:
python37Packages.mocket:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'setup_requires=["pipenv"]' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python37Packages.samsungtvws:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'websocket-client==' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.mocket:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'setup_requires=["pipenv"]' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python38Packages.samsungtvws:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'websocket-client==' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.mocket:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'setup_requires=["pipenv"]' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

python39Packages.samsungtvws:

A substituteInPlace with an unused --replace got detected:
substituteStream(): WARNING: pattern 'websocket-client==' doesn't match anything in file 'setup.py'

Please check the offending substituteInPlace for typos or changes in source.

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109515 run on x86_64-linux 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

3 packages failed to build and are new build failures:

python37Packages.geoip2: log was empty
python37Packages.mocket: log https://termbin.com/u8imo
python37Packages.sopel: log was empty

19 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

This is a semi-automatic executed nixpkgs-review which is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
If you have any questions or problems please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109515 run on x86_64-linux 1

2 packages marked as broken and skipped:

python39Packages.geoip2
python39Packages.sopel

3 packages failed to build and are new build failures:

python37Packages.geoip2: log was empty
python37Packages.mocket: log https://termbin.com/u8imo
python37Packages.sopel: log was empty

19 packages built:

python37Packages.clldutils
python37Packages.csvw
python37Packages.phonemizer
python37Packages.samsungtvws
python37Packages.segments
python38Packages.clldutils
python38Packages.csvw
python38Packages.geoip2
python38Packages.mocket
python38Packages.phonemizer
python38Packages.samsungtvws
python38Packages.segments
python38Packages.sopel
python39Packages.clldutils
python39Packages.csvw
python39Packages.mocket
python39Packages.phonemizer
python39Packages.samsungtvws
python39Packages.segments

class unittest.IsolatedAsyncioTestCase(methodName='runTest')
This class provides an API similar to TestCase and also accepts coroutines as test functions.
New in version 3.8.

Aha, fixed by ignoring tests38 on pythonOlder 3.8.

class unittest.IsolatedAsyncioTestCase(methodName='runTest')
This class provides an API similar to TestCase and also accepts coroutines as test functions.
New in version 3.8.

Aha, fixed by ignoring tests38 on pythonOlder 3.8.


ROI polygon and path don't draw

I can't draw ROI polygons or paths when the shapes layer is created by napari annotation project plugin.
If I create the shapes manually there's no problem and they draw.
This video shows what's going on (and the messages in the terminal) better than I can explain.
Any hints?

Hi @joaomamede,
this problem comes from the fact that I turn shape coordinates to integers (to be able to make a crop later on) on mouse release when drawing a square. Now when you draw a polygon the mouse is released after defining the first point and as the shape is not finished yet the conversion creates a problem.
As the plugin is meant to be used to extract rectangular crops from images (e.g. for ML training) I'm a bit reluctant to fix this and give the impression that it's fine to use any shape. Maybe if you explain your specific use case of using non-rectangular shapes, I can find a solution.
Guillaume

I believe your approach is the correct one indeed and you shouldn't implement my request.
I understood the plugin wrong at the beginning. I was trying to do ROIs to then do the labels from the ROIs polygon. I understood now that ROI are just meant to crop the image that will then contain the labels from the "annotations".

Glad this has been clarified. I'll try to explain this better in the docs.
I'm curious though: are you creating labels from the ROIs instead of painting them? There's a way to transform shape layers into labels, so if that's how you annotate, I could try to add an option in the plugin.

My initial idea was indeed to go from shape to layers, hence my confusion with what the ROI were for. I thought we could do shape polygons or labeled paints.
That would certainly a feature I would use for sure.
The other feature we would use would be 'click' in the center and make a bounding box or label of n x n pixels (basically to annotate diffraction limited viruses). But this is very specific to what we do and probably wouldn't apply to a lot of people.
Thanks for the plugin again.


OCPBUGS-16793: openshift/Dockerfile: exclude unpack binary from FIPS build overrides

/jira refresh

/jira refresh

Thanks @joelanford.
/lgtm

/hold
Just want to make sure the exception pattern is correct.

/hold cancel


async function instanceof Promise === false

Version
2.5.16
Reproduction link
 https://yadi.sk/i/kEgLN0O-3XFsi6
Steps to reproduce
https://yadi.sk/i/kEgLN0O-3XFsi6
What is expected?
What for?
I use vue-cli to install the initial version of the project
What is actually happening?

That's not a valid reproduction.

Open project with vue and run code in console. Global promise polyfill not work with async await

Sorry, its babel.
How to use

vue create testPromise
use new Promise in helloWorld file
open in chrome 66

https://yadi.sk/i/QsTMDQhT3XGvD7

https://bugs.chromium.org/p/chromium/issues/detail?id=830565
problem solved but Babel use custom Promise


Implement celery format for hooks

It would be super nice if someone will implement hook format similar to celery task.

Added in #74.


Error creating new cert :: CSR contains more than 100 DNS names

Hi,
I have just tried using release 0.9.6 (thanks for the update btw).
However after running for approx 20 minutes it continually gets the attached error.
If you'd like the full log (approx 20MB), let me know and I can DM it to you.
ssl_renew_log_truncated.txt

It's not clear from your description - are you actually trying to issue a cert with more than 100 DNS names? If so, that's a Let's Encrypt limit. You can work around it by issuing multiple certs: https://github.com/ohadschn/letsencrypt-webapp-renewer#multiple-certificates-for-a-single-site.
If you aren't trying to issue a cert with more than 100 DNS names and you believe this to be a bug with the WebJob, the full log would be good, maybe share via OneDrive.

Hi Ohad,
Yes it was just a single domain name for the cert.
I was able to downgrade to v<IP_ADDRESS> and it worked successfully.
I will send you a DM with the full log.
Thanks!

(Replied via e-mail)

Updating our offline discussion here -
Looking it at the first 10 lines of the log, it looks you have indeed configured more than 100 domains for a single cert. The new version deals with host name renewals more intelligently, and the reason the old version worked was that it missed one of the domains you recently added (renewing all the domains except the new one). The new version however detected this and tried try to create a new cert containing all of them, which failed as there were more than 100.
I recommend you use the groups feature to separate these to several certificates (see link in the GitHub issue) - even one per domain if you're so inclined. Simply have all the settings shared except hosts which you can separate as you'd like. If you don't want to do this manually, modifying the configuration script to do it for you should be straightforward.

A couple of points:
Yes, we have over 100 hostnames configured on the web app, however they all have individual certificates which has worked well with this code up until release 0.9.6.
The issues we see are:

We only had a small number of certs that require renewing, however the new logic seems to want to create a single certificate for all hostnames whether the need renewing or not. This is a breaking change in behavior
This breaks the renewXNumberOfDaysBeforeExpiration setting, as all certificates are being renewed as soon as a new hostname is added to the list.
Creates a breach of privacy as it will allow anybody to see all our customer names via the SSL certificate.

The current logic works well when you have a small number of your own hostnames that you want to manage, but for those that want to manage their customer SSL renewals, this solution no longer works.

Thanks Ohad,
We appreciate the time you put into this code!
Yes, we have over 100 hostnames configured on the web app, however they all have individual certificates which has worked well with this code up until release 0.9.6.
The issues we see are:

We only had a small number of certs that require renewing, however the new logic seems to want to create a single certificate for all hostnames whether the need renewing or not. This is an unexpected change in behavior. (Previously we had to use the standard WebApp extension to initially create the cert. This created a single cert per host and the renewals via your code were therefore always 1 per hostname)
This breaks the renewXNumberOfDaysBeforeExpiration setting, as all certificates are being renewed as soon as a new hostname is added to the list.
Creates a breach of privacy as it will allow anybody to see all our customer names via the SSL certificate.

The new logic works well when you have a small number of your own hostnames that you want to manage, but for those that want to manage their customer SSL renewals, this solution no longer works.
I will look in to the group feature, but this seems like it would need an overwhelming number of settings to give us a single cert per hostname.
A new setting to renew a single certificate per hostname would solve the issue (e.,g. CreateCertificatePerHost). Is this something you could consider?

I will look in to the group feature, but this seems like it would need an overwhelming number of settings to give us a single cert per hostname.

Which is why I suggested you modify the configuration script to automate it. To reiterate: I recommend you use the groups feature to separate these to several certificates (see link in the GitHub issue) - even one per domain if you're so inclined. Simply have all the settings shared except hosts which you can separate as you'd like. If you don't want to do this manually, modifying the configuration script to do it for you should be straightforward.
So per host name you would only have a single setting that looks like webAppName[foo]-hosts = foo.com and you would append  webAppName[foo] to letsencrypt:webApps.


Cannot use window.setMenu()

I didn't find a way to use windows.setMenu() (I want to remove all the menu items)
Is it possible to expose the window object in desktop.js ?

Yes, the window object is received through an event.
https://github.com/wojtkowiak/meteor-desktop/blob/master/scaffold/desktop.js#L44

Did you manage to do that or do you need more help?

No thank you, it's a perfect solution !!

So closing 👍


feat: Add Craft CMS 5

Description
Adds SetList::CRAFT_CMS_5 constants for Craft CMS 5 support
Yes, it's just an empty config that inherits from Craft CMS 4's config... but it feels right... and the addition in the docs gives people the confirmation that yes, this is updated and you can and should use it with Craft CMS 5.
Related issues

@brandonkelly, can you please accept this pull request?


eval_batch_size=auto

Adds eval_batch_size= auto, expanding grad_accum=auto.
Interestingly, eval_batch_split is often far smaller than grad_accum (with resnet batch size 8k, I was seeing 2 vs. 8), so making them two different variables was the right call.

Oops meant to open draft PR -- not ready for review yet!

I think this works -- tested on resnet with large batch size, but I'd love to get some help testing this on more workloads.

Is this ready for review?

Is this ready for review?

Yes! :)

Do we clear the already computed gradients if we run out of memory while backward is partially executed? PyTorch will do += for such already computed gradients on retry.


Locality details

I added three new tabs.

on Ecometrics tabs there is one field, Mean hypsodonty, which will be computed based on species tooth crown heights in the Locality. Similar fields can be found on the Species-Teeth Tab: Developmental Crown Type and Functional Crown Type
These kinds of fields should be calculated and displayed but not edited in the edit mode

@karilint

multiple fields can be destructured from the context hook at the same time, so the hook has to be called only once:
  const { textField, dropdown } = useDetailContext<LocalityDetails>()
In taphonomy tab, the assemblage formation options were unused and in assemblage formation, the options were instead timeRepresentedOptions which was also used in another field, so I assume assemblageFormationOptions are meant for the assemblage formation field, so I changed it to that.
I will merge this to development


Crash when connecting to the world

Hello.
When using atmosfera 1.3.0 in minecraft 1.16.2, it crashes while connecting to the world.
The crash log is shown below.
single play
https://paste.ee/p/aWBJZ
multiplay
https://paste.ee/p/sc8A5

Fixed in version 1.3.1.


chapman-kolmogorov test Error

Hi, pyemma developers,
I am following the pyemma tutorials for the pentapeptide and I encountered errors at the CK test step, with the following error message:
"m = self._D.shape[0]  # this will raise and exception if self._D doesn't exist yet.
AttributeError: 'BayesianMSM' object has no attribute '_D'"
So, how to solve this problem.
Thanks
Delin

cross-posted markovmodel/pyemma#1416, fixed by markovmodel/pyemma#1418

Hello pyEMMA Team,
I am getting this Error after running below code.
"nstates = 5
cktest = msm.cktest(nstates, mlags=6)
pyemma.plots.plot_cktest(cktest, dt=0.1, units='ns');"
Please tell me how can i fix it?
AttributeError                            Traceback (most recent call last)
Cell In[34], line 2
1 nstates = 5
----> 2 cktest = msm.cktest(nstates, mlags=6)
3 pyemma.plots.plot_cktest(cktest, dt=0.1, units='ns');
AttributeError: module 'pyemma.msm' has no attribute 'cktest'


Added STLocationRequest

STLocationRequest is a UIViewController Extension to show a beautiful 3D Flyover location request screen to get access to the user location at the very first time in style. (written in Swift)
https://github.com/SvenTiigi/STLocationRequest
*Now added ad the end of the permission section :+1:

Thanks


Allow to set the width and height as percentages

Yakuake from Linux allows you to set the width and height as a percentage of screen width and height. This is more user friendly as it's easier to comprehend, plus a default value of 80% or 100% makes the program "just work" on any screen, without a need to configure anything beforehand. Also, if this is implemented, I suggest making the "centered" alignment the default choice.

Good idea, thanks. I'll aim to include this in the next release. 👍


[DDW-651]: UI - Delegation Center

This PR implements delegation center screen on staking module.
YouTrack Ticket
Todos

[x] Header section with DonutRing component
[x] Body section with wallets data with dropdown menu
[x] Inject Japanese translations

Screenshots

Testing Checklist

Slack QA thread
[x] Test delegation center screen on storybook by visiting Staking => Delegation Center story
[x] Test all other UI components if they are not broken.

Review Checklist
Basics

[ ] PR has been assigned and has appropriate labels (feature/bug/chore, release-x.x.x)
[ ] PR is updated to the most recent version of the target branch (and there are no conflicts)
[ ] PR has a good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added to the top of the appropriate section (Features, Fixes, Chores) and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance and unit tests are passing (yarn test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (yarn dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (yarn package / CI builds)
[ ] There are no flow errors or warnings (yarn flow:test)
[ ] There are no lint errors or warnings (yarn lint)
[ ] There are no prettier errors or warnings (yarn prettier:check)
[ ] There are no missing translations (running yarn manage:translations produces no changes)
[ ] Text changes are proofread and approved (Jane Wild / Amy Reeve)
[ ] Japanese text changes are proofread and approved (Junko Oda)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (yarn storybook)
[ ] In case of dependency changes yarn.lock file is updated

Code Quality

[ ] Important parts of the code are properly commented and documented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-renderings
[ ] Any code that only works in main process is neatly separated from components

Testing

[ ] New feature/change is covered by acceptance tests
[ ] New feature/change is manually tested and approved by QA team
[ ] All existing acceptance tests are still up-to-date
[ ] New feature/change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review

[ ] Merge the PR
[ ] Delete the source branch
[ ] Move the ticket to done column on the YouTrack board
[ ] Update Slack QA thread by marking it with a green checkmark

@nikolaglumac @daniloprates @MarcusHurney Please review this.
Thanks.

icon color is from other theme

@nikolaglumac @darko-mijic mb  we should hide this circle when 100%?

I like this circle. Let's discuss this on our next design call on Tuesday together with @daniloprates.

my initial thought was to show this options only on "cog" click (not on hover)

my initial thought was to show this options only on "cog" click (not on hover)

I prefer click as well 👍

same bug as @DeeJayElly is fixing

same bug as @DeeJayElly is fixing

@yakovkaravelov please ignore this one. It will be fixed by @DeeJayElly in the scope of this PR: https://github.com/input-output-hk/daedalus/pull/1437

in the design there are colored indicators of current pool rank. Do we ignore this feature? @nikolaglumac

All green text should have "cursor: pointer".
It can be implemented later when we cave all endpoints.
p.s. just mentioning to bring some attention

in the design there are colored indicators of current pool rank. Do we ignore this feature?

@yakovkaravelov did you implement it as such?

All green(link_color) text should have "cursor: pointer".
It can be implemented later when we cave all endpoints.

@yakovkaravelov please make it like that.

All green(link_color) text should have "cursor: pointer".
It can be implemented later when we cave all endpoints.

@yakovkaravelov please make it like that.

@nikolaglumac not sure it should be done right now, as other screens (e.g. "rewards" and "epochs") use these "in-app actions" but not using proper cursor.

OK then let's skip it for now but to it in the end at once.

Drop-down is not fully shown for the last wallet in the list when minimized mode is selected

Also not sure if it is important that when we click on Learn More link we are redirected to the first page of Storybook

Dark theme bugs:
cog color

separator color

bg circle doesn't use opacity

I've revisited dark theme again to make an example of this screen...
https://zpl.io/aX50nGM
Changed small drop menu a big to have a better definition

@a-rukin Please review again. Thanks.

Drop-down is not fully shown for the last wallet in the list when minimized mode is selected
I have added video on the thread

@IuliaDolishniak
This is due to Select component of react-polymorph.
All pages work same way if you try them in small view port.
Need to update that global component to fix this issue.
I think it should be rather separate ticket, not this PR.

Also not sure if it is important that when we click on Read article link we are redirected to the first page of Storybook

@nikolaglumac Please give me decision about this.

don't know if it's an issue, but if the pools is the same color should be same too

cog should be opaque when dropdown is opened
and dropdown should be aligned to right (like in designs)

this line should have 30% opacity

@nikolaglumac what elements should be under this cog? 🤔

Also not sure if it is important that when we click on Read article link we are redirected to the first page of Storybook

@nikolaglumac Please give me decision about this.

This is fine @yakovkaravelov ...

image don't know if it's an issue, but if the pools is the same color should be same too

@a-rukin you are right but I am not sure what kind of dummy stake pool data @yakovkaravelov is using. Anyways, @yakovkaravelov if the pool is the same please use the same color 🙏

@nikolaglumac what elements should be under this cog? 🤔
I mean that we can't delete delegation here and change could be done via link below

@yakovkaravelov @a-rukin in this case the menu should have only one item "Delegate"

@IuliaDolishniak
This is due to Select component of react-polymorph.
All pages work same way if you try them in small view port.
Need to update that global component to fix this issue.
I think it should be rather separate ticket, not this PR.

@yakovkaravelov @IuliaDolishniak we definitely need to see how to fix this

@yakovkaravelov @a-rukin in this case the menu should have only one item "Delegate"

as an option we can remove this icon as it makes not to much sense

@IuliaDolishniak
This is due to Select component of react-polymorph.
All pages work same way if you try them in small view port.
Need to update that global component to fix this issue.
I think it should be rather separate ticket, not this PR.

@yakovkaravelov @IuliaDolishniak we definitely need to see how to fix this

@nikolaglumac in this PR or another?

Another PR @IuliaDolishniak - please create a card for this on YT 🙏

@nikolaglumac @IuliaDolishniak @a-rukin Please review again.
Thanks.

If I remember correctly it should be "ADA" here

bug with colored line here

on hover it should change opacity too

as an option we can remove this icon as it makes not to much sense

@nikolaglumac should we?

as an option we can remove this icon as it makes not to much sense

@a-rukin I would not do that but OK... Let's discuss this on the design call today.

@yakovkaravelov I have moved this PR into the WIP stage (together with the card on YT) until the issues @a-rukin reported are resolved.

@IuliaDolishniak - @DominikGuzei has fixed the issue which you reported (https://iohk.myjetbrains.com/youtrack/issue/DDW-723) in the scope of this PR. Can you please re-test?

The issue with the dropdown appears to be fixed. But now I cannot see the icon that we need to click for the dropown to be shown @yakovkaravelov can you please have a look?

@IuliaDolishniak @nikolaglumac @a-rukin Would you please review again? Thanks.


[Feature Request]

Please add support for a lora stack (to be more specific, the one from Efficiency Nodes), as many people tend to use that, instead.

The lora_stack type is specific to the efficiency nodes repo, so I don't want to depend on it directly (in general efficiency is quite a special node set that does things it's own way). But I submitted a node there that would convert it to a simple string. If it is accepted, you could then use a string concatenate node of your choice to combine the prompt and the LoRAs string together and feed that to the Image Saver node.

The converter node has been merged to Efficiency Nodes.
If you update Efficiency and these nodes, you can now do something like this to combine lora infos with the prompt:

I use string combine from WLSH here, but I'm sure there are many other nodes like it.


Ignore some errors in demo sentry

I want to merge this change because in saleor-demo sentry we want to ignore some of the exceptions that may occur in demo env which are: ReadOnlyException and Invalid HTTP_HOST header

Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[ ] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-4843 Demo report ReadOnlyException and Invalid HTTP_HOST header to Sentry


Kim file permissions

Closes #598.
The current key info manager files are not having any permissions set on them. This MR adds suitable permissions to directories and files of the KIM and also adds unit tests to verify if the permissions set are correct.
Signed-off-by: Gowtham Suresh Kumar<EMAIL_ADDRESS>

The discussion about the KIM file permissions for parsec user and group is still ongoing. This MR can be merged once we agree on a particular approach.
https://github.com/parallaxsecond/parsec/issues/598


Implements the ability to add items to layout/tag by name (search).

I've implemented the ability to add an item to the layout via search. #5
https://www.youtube.com/watch?v=E01dj89zxzg
Have a look if you think this implementation is sufficient.
Could use a code review as it's my first contribution to your project, but it seems to be working great!
Definitely something I was looking for on my group iron.
When adding a bunch of items a keybind for this action would be nice too.
Maybe it'd be nice to add the item on the first visible index, when adding items to a long tab you have to scroll up and drag it over to where you where trying to add something

I think having the add item to layout option on right-clicks in the bank over empty spaces might be better, what do you think?

Item is pre-positioned, no need to drag it to the spot you want it at.
downsides:
it's not going to be obvious to people because they don't right-click empty space very often.

A checkbox for adding an item's variants might be neat, for stuff like barrows armor.
Also, looking at the code (haven't run it yet), it looks like adding an item to a non-open tab will NPE.

add item to layout option on right-clicks in the bank over empty spaces
Yep, that'd be better

A checkbox for adding an item's variants
Maybe, I wouldn't use it personally, but I can see how people would like separate placeholders per variant

adding an item to a non-open tab will NPE
Oops! Yeah I wouldn't be surprised if it did, first point should take care of it

I've also gone and implemented the ability to open the add item search dialog  via hotkey which is really really nice when creating a bunch of them. It's customizable in the settings tab. The default is Ctrl+A (for add). Ctrl+N could be an alternative for the default binding.
I can push that later.

separate placeholders per variant
The variants thing I mentioned is not for separate placeholders but for adding variants of an item to the tag only, not the layout, so that if you add barrows armor and it degrades it will still show up in the tag.

ctrl-A
I don't like the idea of a default keybind because people will have to go to the config panel to even know that it exists, and if they do that they may as well set it themselves. This avoids people asking "why am I no longer able to press ctrl-a" in runelite discord (assuming this can actually happen, it depends on how you implement the hotkey I guess).

Wynadorn force-pushed the add-item branch from 1a139f9 to efad26a 7 minutes ago

Cleaned this branch up a little


2nd (and/or 3rd) color for Multi-color effects

I understand it is not possible to set the 2nd or 3rd color for multi-color effects from the WS2812FX library?
For example: "Comet" can use 2 colors: foreground (Comet itself) and a background color.
--EDIT-- Bad example... Turns out "Comet" effect only uses one color.   ;-)
If this is the case, I'm looking into adding this functionality.

@AvanOsch Yes, this is indeed currently the case. Would be great to have a solution here.

I got this working.
A couple of things though:

I'm no coding guru. I can imagine a couple of changes need to be made.
I'm totally new to Vue.js. Might need some adjusting as well.
I only use the web interface at the moment, so only tested with that (WebSocket)
I don't use MQTT or Home Assistant (but I think it should work)
I'm not sure if we need RGB values in Arduino code... We could save some code by just using HEX, and converting from RGB client-side.

Usage:
WebSocket:
#<hexrgb> --> color1
##<hexrgb> --> color2
###<hexrgb> --> color3
$ --> {"mode":0, "delay_ms":50, "brightness":255, "color":[255, 20, 147, 255, 20, 147, 255, 20, 147]}

HTTP REST:
/status --> {"mode": 5, "delay_ms": 50, "brightness": 128, "color": [255, 20, 147, 255, 20, 147, 255, 20, 147]}
/set_mode --> rgb or r, g, b & rgb2 or r2, g2, b2 & rgb3, or r3, g3, b3

As I said, I use the web interface, so I also adjusted the McLightingUI (v2) for use with 3 colors.
My code is here: https://github.com/AvanOsch/McLighting
And here: https://github.com/AvanOsch/McLightingUI

Have been reading through the WS2818FX user manual, and figured out why "Comet" (among other effects) don't use the second color as background.
It needs an extra setting (FADE) when initializing the strip segment.
Updated the code on my fork.

I totally re-wrote McLightingUI in plain Javascript: https://github.com/AvanOsch/McLightingHTML
Apart from not knowing Vue.js, this made the file size about 10x smaller. I can also imagine some people (like myself) are more comfortable with plain Javascript HTML and CSS.
I also added very basic language support and made interface colors changeable.

This is great! I hope it will be in the main soon. I would also appreciate an auto command (to start autoplay) and more than 99 leds support in the range command.

WebUI is all the hard work of @toblum. I dont do web. I spent the whole thanksgiving holidays to learn javascript & css and I'm amazed at what it can do. I have created such a flavor as well for fun, but I think you can create a branch/fork for https://github.com/toblum/McLightingUI . This way we can have a pure js flavor for people to try.

@bojanul The autoplay command is supported as well (the interface is almost the same as McLightingUI v2, and has all it's features)
I don't know what you mean by "more than 99 leds support in the range command", but I'm testing everything on a 240 LEDs strip, and it works fine.
@debsahu I first forked McLightingUI (see 3rd post above), but i found Vue.js more complicated than just using pure Javascript. Also, the resulting file was almost 1MB in size, and saving values on the nodeMCU stopped working (memory full).
Although there are a lot of functions I used from toblum's version, the two versions are very different. That's why I put it in a separate repo...

Actually, the autoplay and range remarks/wishes ware directed towards the author of this great peace of software: as far as I understand, there is no websocket/MQTT command for autoplay, and the range command has only 2 digits for the number of leds. Of course, MQTT can be used to emulate autoplay, but the command would be also useful.

@bojanul Ah, I see.
Well, autoplay seems to be supported (send "play" or "stop" over websocket)
I've not (yet) added 3 color support for saved autoplay patterns though.
I have no idea what the "range" command would do. Have not even run into it while adjusting everything for 3 colors...

Hi @AvanOsch,
great to see another UI for McLighting. It was built that way to enable multiple interfaces and it's always good to have many variants to choose from. I built  McLightingUI because I wanted to try out vue.js (which is great but overkill for that purpose) and see if I could implement some missing features faster. But yes, it's quite heavy.
I will try out your version soon. Maybe we can link to it (and other frontends) from the wiki.
Regards
Tobias

@toblum Thanks! Please also check out my modifications on McLighting for the 3 color support. Or should I do a pull request or something?

@toblum If you come around to testing my 3 color implementation, I'd like to get some feedback on my "convertColors()" function:

Somehow, I needed to add a delay. It was working fine, until I removed a debugging line. After re-tracing my steps I found out a short delay after converting colors fixed the problem. Do you have any idea why?
I'm making an "Interactive PoolTable", which animates a McLighting-strip whenever a ball drops in a pocket (by setting separate segments for each pocket).
After adding sensors, I get random crashes (especially when 2 are triggered at the same time). Tracking the problem, I found out this also happens when triggering the "convertColors()" function.

More debugging needed here, because I think it could also have to do with the NodeMCU pins I'm using...
During normal operation (without my PoolTable changes) everything works fine, but as some weird stuff is going on around that "convertColors()" function, I'm kind of hoping you can find something wrong with it.

You have a pointer to a local variable declared in the function that's gets destroyed after execution of function. Make it global or memory manage the array manually.

@debsahu Thanks for your reply, but could you clarify a bit?
I didn't make a global variable because after the function returns the color array, I thought it wasn't needed anymore...
Thinking about it, I could make a global variable so the function only needs to be called whenever a color changes.

See section 2.5 here http://www.ntu.edu.sg/home/ehchua/programming/cpp/cp4_pointerreference.html

uint32_t color is declared inside the function which is destroyed after the execution of function. The pointer now points to this destroyed location, which is the cause of your issues.

Hi @AvanOsch,
I found time to try out your mod. Flashing the software via pio worked flawlessly, but unfortunately the UI didn't work. It starts up until 100 % but the modal doesn't close.

I like how you solved to color channel switch and also supplied a translated version.
Regards
Tobias

Thanks @toblum
I didn't account for a missing uistate.json...
Fixed a couple of other small bugs too, and added "Hide Autoplay" setting.

Will these additional effects be added to mclighting as default?

I‘m currently porting this to man RGBW branch (3colors)
I‘m also porting your WebUI to be used with RGBW.
Looks promissing.

Ready to test
https://github.com/FabLab-Luenen/McLighting/tree/3colors
GUI:
https://github.com/FabLab-Luenen/McLightingHTML

@AvanOsch Great work, tracking this. @bpohvoodoo thanks, now I know the changes required.


I2p: fix service

These are the changes I had to make to get i2p running as a proper systemd service.
cc @joelmo

Thanks. You removed jdk, does it still build, I don't think jre have the compiler? No hardcoded openjre was used :smile_cat:

Fixed. Thanks! It built on my machine because oraclejdk == oraclejre.

I'm unsure if hydra builds unfree software, does jdk refer to oraclejdk? If this is the case its nice to keep openjdk because building may take around 30-60min on my machine.

jdk and jre refer to icedtea, openjdk or oraclejdk in order of availability. They are also the canonical point for packageOverrides, thus packages should respect them, except if they depend hard on a specific vm.

https://github.com/NixOS/nixpkgs/blob/master/pkgs/top-level/all-packages.nix#L3837

Thanks this is nice, I would squash the last three commits, from f1b86e8 and up as they all improve the build.

there you go

Great guys!


Subscripts not showing up correctly

The html file when run shows the subscripts not as subscripts.  They look like superscripts.  See attached screen shot.  It is the equation after "guess" and "root".
Please help!  What am I doing wrong for such a simple case.
I am using Chrome.

This is my html file and  the content is given below as well<EMAIL_ADDRESS>

This is my .js file and  the content is given below as well
<EMAIL_ADDRESS>
var x = []
x.push((Math.floor(Math.random()*(3-2)*10)/10)+2);
var equation = katex.renderToString("x^2 = 22");
var initial = katex.renderToString("x_{0}  = "+x+"");
var second = katex.renderToString("x_{2}");
document.write("Given the equation "+equation+" and an initial guess "+initial+", the second iterative value of its root, "+second+" by Newton-Raphson method is");
for (i = 1; i <=2;i++){
nrm2 = x -(Math.pow(x,2)-22)/(2*Math.pow(x,1));
x=nrm2;
}
//document.write(nrm2);
var simModel5 = new simcapi.CapiAdapter.CapiModel({
nrm2: nrm2,
simEnabled: true
});
simcapi.CapiAdapter.expose('nrm2',simModel5);
simcapi.CapiAdapter.expose('simEnabled', simModel5,{writeonly:true})
simcapi.Transporter.notifyOnReady();

You need to add a doctype to your HTML file to make KaTeX render correctly. (there is a warning in the console telling you this!) See e.g. #332

Thank you.  It worked.
Autar Kaw
http://autarkaw.com

On Feb 17, 2017, at 12:15 PM, Emily Eisenberg<EMAIL_ADDRESS>wrote:
You need to add a doctype to your HTML file to make KaTeX render correctly. (there is a warning in the console telling you this!) See e.g. #332
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Use the trusty image on TravisCI

Coverage remained the same at 85.904% when pulling b2ba219d8115d4348187e51b0875b0cbcc9e533a on travis-trusty into 491b7c8d6fe2c7fea615ed81aa90d96044b88a1d on master.


TextDocumentContentProvider + d3

TextDocumentContentProvider works. I used this as a starting point https://github.com/Microsoft/vscode-extension-samples/tree/master/textdocumentprovider-sample
This works if you save it as an html file and open it in a browser, but not if I return it in provideTextDocumentContent:
https://gist.github.com/dfinke/f51f8cba643b1b153ef8
Is there a way to make it work or debug?

I didn't implement this, but I think we use an <iframe> to render the preview and we disable all <scripts> because they would run in the UI thread.
fyi @jrieken

@alexandrudima correct, we do not disable JS execution but I don't think we strip scripts (this was the state when we last discussed it). The code is here:
vscode\src\vs\workbench\parts\html\browser\htmlPreviewPart.ts
@dfinke to debug this look https://github.com/Microsoft/vscode/wiki/How-to-Contribute#development-workflow in the Debugging section. The htmlPreview code runs into the renderer process.
I'll look into this later.

@dfinke @alexandrudima
In the markdown preview the JavaScript from the above gist is executed as shown below. I was assuming that the html preview mostly mimics the mark down preview, but this is not the case and I did not succeed to execute scripts in the html preview.

Very cool. So a few tweaks to the html preview and vs code can be an interactive d3 pad.
I was looking at the htmlPreviewPart.ts to see if I could swap the iframe out, but I don't know if that would be enough. Maybe cloning the markdown preview is the place to start.

Maybe cloning the markdown preview is the place to start.

well this is where it started :smile:, need to understand the differences with regard to JS handling in the two implementations.
As a workaround you could already do the d3 things using the markdown preview as is done in the twitter extension, but we should really get this to work.

The short Tweet on this drove questions in my direction. The snippet does not work in the 0.10.8 release. I know the Build conference is coming. Hopefully this can make it into the editor no too long after. It'd be a great feature.

I'll take a look - things are rendered in an iframe and script aren't blocked in anyway.

@jrieken Maybe we should use a <webview>

The web view is past GA and tracked in #2414

:+1:


Update documents

Mention the Python binding and Go binding in README.md and
embed_wamr.md.
Fix typo of esp-idf link in build_wamr.md.

LGTM


Possibility to remove pending transactions

Is your feature request related to a problem? Please describe.
Some of the transactions I have in my wallet were sent multiple times while only one have been confirmed. The rest stays at the top of the tx history pending forever. I would like to remove them
Describe the solution you'd like
I would like to be able to remove pending transactions
Describe alternatives you've considered
None

SW can pretend that "transaction remove" exist on the protocol level by adding a button that would send 0 ae from the account to itself with the correct nonce and a high fee to replace a pending transaction.

ok, but this would take about 9 minutes to remove transaction by itself

AE coin related

Potential pending transaction cases and what wallet can do (to be implemented):
Nonce is higher than needed
This type of pending is not removable (instant removing),
but user is capable to push another transaction with this nonce and with higher fee in order to override it.
Send flow is not frozen.
Spend transaction: transaction amount > balance > fee

User can wait till balance will be bigger than amount, send flow will be frozen
all other transaction would wait until stuck transaction is mined
User can override stuck transaction with another transaction
balance > amount + fee
User can override stuck transaction with spend transaction with amount 0

Spend transaction: balance > fee + amount (Node is heavily loaded; stuck > 10-15 sec)
User may have time to override stuck transaction, by sending transaction with a higher fee
All transaction types: fee > balance
User can wait till balance will be bigger than fee
All transaction types (spend excluded): balance > fee

User can override stuck transaction with another valid transaction
User can override stuck transaction with spend transaction with amount 0

Nonce is higher than needed

it should be within current nonce + 1 and current nonce + 5 because otherwise, the node won't accept such tx.

This type of pending is not removable (instant removing)

Probably the wallet can make N (< 5) spend TXs with 0 amount.

Node is heavily loaded

Meaning there is no empty space in currently mining blocks for the gas price specified in the tx.

Based on the above I tried to simplify the flow for the end users of Superhero Wallet defining 2 basic cases.

DESIGN:
1. Nonce is higher then needed:

2. Transaction is pending for more than expected because of network congestion:

Sub-case: the stuck transaction has been processed by the network before the "Remove from tx queue" transaction is broadcasted (so the stuck transaction can not be removed from queue and Superhero Wallet displays a failed transaction modal).

FIGMA REFERENCE:
https://www.figma.com/design/3oGLWzSH0oJljo4RETZtur/Superhero-Wallet-UI-(✔Updated)?node-id=34345-240130&node-type=frame&t=xKcUtmuhcvofmWU5-0

All the cases that are marked with red are possible, just not in our wallet. User can import aeternity account from another wallet into ours.

All the cases that are marked with red are possible, just not in our wallet. User can import aeternity account from another wallet into ours.

In such cases:

And wording update here (adding insufficient account balance as a possible cause of stuck transaction):


CLS Token

Hi all, thanks for the great work. I've been looking at the example codes and modules and I noticed you have an arg for choosing the class strategy as projection or cls_token or average. I guess it doesn't apply to iTransformer since it only works with projection. But I was wondering whether you had feedback about using a learnable class token as a final representation for tasks like timeseries classification. Have you tried it already?

We tested itransformer on the classification task with tslib, but it seems alternative models can be a good choice.
We implement it by averaging the variate tokens. However, introducing learnable class tokens like ViT has not been trialed. Did you get some results?


Fix cmdline cf-client-utils

Surround serviceInstance.create parameters with quotes.
Signed-off-by: Martin Aleksandrov<EMAIL_ADDRESS>Signed-off-by: Denica Madzhirova<EMAIL_ADDRESS>

Coverage remained the same at 92.055% when pulling dffe6b1dcb4862a5a896d518482d4d4a366a1f81 on SAP:fix-cmdline into 5181a7a6f1092fc2b0c7865c6b717764987d8014 on cloudfoundry-incubator:master.


git clone command

git clone git@github.com:psq/stacks-dump.git
did not work for me, what worked:
git clone https://github.com/psq/stacks-dump

this would fail if git is not setup for ssh.  Having people using git assumes a certain level of knowledge.


Revert "Update CloudflareBypasser.py"

Reverts sarperavci/CloudflareBypassForScraping#24

i need userscript for send to server


Move storage format over to binary, this change allows the addition of:

Checksums, File format versioning and feature flags i.e: encryption, this change also makes the db files way more compact.
Other changes:
tighten except clauses in Util class, change some names to snake case
This is a more involved change so take the time to review it and consider it's impacts and ask away any doubts you may have about it
closes #10

PS: the assertion that if data is JSON serializable it can be stored to the file format remains valid

@DarthUdp My understanding is Elara can now store objects in the files (since you're moving it out of json) along with the checksum feature.
Can you possibly add some more comments and a few basic tests for this util function before I merge it in?

You can add a new test file into the test folder. Merging these commits will essentially be a breaking change.

No it cannot store arbitrary objects, such serialization is essentially not achievable without the use of the very unsafe pickle module, and even then it cannot store any and all objects, what an implementer of a class can do is implement the Serialize interface. But then they are just describing a way to encode their class state into primitive types that can then be serialized.
refs:

https://docs.python.org/3.3/library/json.html#encoders-and-decoders
https://docs.python.org/3/library/pickle.html

No it cannot store arbitrary objects, such serialization is essentially not achievable without the use of the very unsafe pickle module, and even then it cannot store any and all objects, what an implementer of a class can do is implement the Serialize interface. But then they are just describing a way to encode their class state into primitive types that can then be serialized.
refs:

https://docs.python.org/3.3/library/json.html#encoders-and-decoders
https://docs.python.org/3/library/pickle.html

Yup, got it 👍🏽  thanks

Do you think the code could use more comments?

Do you think the code could use more comments?

To me the code is pretty self explanatory but since you will have to work on it eventually :)

Do you think the code could use more comments?

To me the code is pretty self explanatory but since you will have to work on it eventually :)

Yup I think the code is fine. I'll format it later if needed. But please do add some basic tests.

I added one test of a full cycle data store and recovery.

I added one test of a full cycle data store and recovery.

Thanks, I'll merge the PR. :)


snappy project had moved

It had moved to: https://github.com/google/snappy
get an error:

package code.google.com/p/snappy-go/snappy: unable to detect version control system for code.google.com/ path

fixed in commit https://github.com/mreiferson/go-snappystream/commit/028eae7ab5c4c9e2d1cb4c4ca1e53259bbe7e504


Update generic nodes to use static icon api

Currently generic nodes still use encoded strings of the LabIcon svgs.
This PR adds copies of those svgs to the backend static API and
updates the pipeline editor to use those for the persisted values,
like we already do with kfp and airflow nodes.
Fixes #2907
Developer's Certificate of Origin 1.1
   By making a contribution to this project, I certify that:

   (a) The contribution was created in whole or in part by me and I
       have the right to submit it under the Apache License 2.0; or

   (b) The contribution is based upon previous work that, to the best
       of my knowledge, is covered under an appropriate open source
       license and I have the right under that license to submit that
       work with modifications, whether created in whole or in part
       by me, under the same open source license (unless I am
       permitted to submit under a different license), as indicated
       in the file; or

   (c) The contribution was provided directly to me by some other
       person who certified (a), (b) or (c) and I have not modified
       it.

   (d) I understand and agree that this project and the contribution
       are public and that a record of the contribution (including all
       personal information I submit with it, including my sign-off) is
       maintained indefinitely and may be redistributed consistent with
       this project or the open source license(s) involved.

running front end integration tests to regenerate snapshots are failing, might need to look into it as substask


[Feature]: Output logps of given output

🚀 The feature, motivation and pitch
vllm is a great project. and i have meet the problem that how could i acquire the logps of a given input and output?
for example, when the question is "who are you?", i want acquire the logps of answer "i am GPT model". i could not find any functions to realize this issue.
Alternatives
No response
Additional context
No response
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

You can pass sampling_params=SamplingParams(prompt_logprobs=...).

it seems like not fit this problem. when i ask the LLM that "who are you?", i want it output the logps of the answer "i am GPT model". and the "i am GPT model" what i expected.
what is expected may be a function like:
def get_logps(question, answer):
return logps
or may you explain more about your methods?

You can also pass logprobs argument. See this page.


Cross-gen more assemblies to improve startup

When running a very simple scenario like powershell -noprofile -command "echo 1", PowerShell on .Net Core is much slower than Windows PowerShell.
On my machine, in Windows PowerShell, this scenarios is 330ms, on .Net Core, it's 1380ms.
After analyzing the performance, cross-gen will help quite a bit, here are all assemblies with >5ms of JIT time at startup:

Assembly
JitTime msec
Num Methods
IL Size
Native Size

System.Linq.Expressions.dll
236.0
592
40,518
117,732

System.Management.Automation.dll
70.5
59
14,596
42,074

Microsoft.CSharp.dll
64.3
88
14,500
51,248

System.Runtime.Extensions.dll
63.1
181
9,059
41,329

System.Linq.dll
55.6
96
5,873
22,785

System.Private.CoreLib.ni.dll
52.0
139
9,756
21,120

System.Collections.Concurrent.dll
49.0
82
7,752
21,467

System.Collections.dll
43.5
106
6,329
13,983

Newtonsoft.Json.dll
38.2
67
7,552
18,844

System.IO.FileSystem.dll
26.5
75
4,356
12,125

System.Diagnostics.Process.dll
24.5
30
3,144
7,746

System.Threading.Tasks.Parallel.dll
20.2
30
3,466
9,483

System.Security.AccessControl.dll
20.1
45
4,215
10,367

System.Text.Encoding.CodePages.dll
17.5
32
4,034
10,010

System.Private.Uri.dll
12.4
26
4,229
8,563

System.Threading.dll
12.2
9
1,966
5,419

System.Security.Principal.Windows.dll
11.5
26
2,227
6,602

System.Console.dll
9.5
34
1,213
3,840

Microsoft.Win32.Registry.dll
8.2
18
2,045
4,814

System.IO.Pipes.dll
6.0
13
883
2,369

System.Diagnostics.FileVersionInfo.dll
5.3
10
1,011
2,523

System.Collections.Specialized.dll
5.2
13
507
2,263

Note that some of these methods can't be compiled ahead of time, e.g. dynamic methods or some generic methods in System.Management.Automation.dll. Still - if we cross-gen'd most of these, startup would be more similar to Windows PowerShell.

@joeyaiello do we have package size constraints?

@lzybkr You are using cross-gen in a way that is unfamiliar to me. It that what ngen is called in .net core?

@lzybkr thanks so much for doing this. So I'm understanding correctly, the JIT msec is what happens at initial startup time?
@SteveL-MSFT I'm not aware of any right now, but I can certainly check with Nano folks.
Given the number of asks to improve performance, I think we should move ahead with crossgen-ing as much as possible and thin it out if size becomes an issue.

@powercode https://github.com/dotnet/coreclr/blob/master/Documentation/building/crossgen.md

Note that cross-gen might not be stable enough for us yet, e.g. see  https://github.com/dotnet/coreclr/issues/9745

I'll do a few experiments to see if we should do this now

Using @lzybkr's list above, crossgen'd everything I went from a startup time of 1998 ms (103 MB) to 1067 ms (133 MB).  So about half the startup time and an increase of 30% in size.
Seems like a good trade off.


question to get value

Hi. Thank you for your work with flaui, I am starting to use it and looks better alternative.
But I can't get the actual value of a control like a textbox or label, how can I achieve it?
Maybe some more samples will help us.
That you.

Have a look at the UI test project (https://github.com/Roemer/FlaUI/tree/master/src/FlaUI.Core.UITests) there you can find a lot of samples.
In general you convert your element once found into the element you expect it to be (for example AsTextBox() and then you have the most important methods / properties directly available.

@poloab an alternative is asking on StackOverflow, questions and answers there will help many devs.

Or you can also ask on gitter

Thank you. I'll try it. The UI test project that I didn't see is helpful. Thanks for your help.


Build process

The master branch build should run the steps outlined at https://github.com/cityindex/log4net.Appenders.Contrib#development,
Specifially:

Update the VERSION.txt
Run build.ps1
If successful, publish to NuGeta

There should also be a Pull Request build that is triggered by all any commit to a pull request that just runs build.ps1

Send email to build requests mailing list

I've setup a build process on the master branch on CityIndex's (internal) Team City Server - http://teamcity.cityindex.co.uk/viewType.html?buildTypeId=Infrastructure_Log4netAppendersContrib
@fandrei - I don't think there is a way you can get access to this; however, all it is doing is running
build.ps1 and then build.ps1 -nuget_api_key XXXXXX -nuget_api_url https://www.nuget.org/api/v2/package NugetPublish; so you should be able to replicate any build errors locally.
@ewilde - one remaining issue is that running the build twice with the same VERSION.txt number will fail silently with:
-------------------------[NugetPublish]-------------------------
[12:42:24]Error: 21/07/2015 12:42:23:
[12:42:24]At D:\Agent2\work\1a64ee37c9cd750e\build\modules\nugettasks.psm1:21 char:5
[12:42:24] +     Invoke-CraneNugetPublishAllProjects $global:context.solution_context -NugetO ...
[12:42:24] + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ [<<==>>] Exception: Error executing nuget push for project log4net.Appenders.Contrib.
[12:42:24]Command: D:\Agent2\work\1a64ee37c9cd750e\build\NuGet.exe push "D:\Agent2\work\1a64ee37c9cd750e\build-output\nuget\log4net.Appenders.Contrib.<IP_ADDRESS>.nupkg" -Source https://www.nuget.org/api/v2/package -ApiKey 76b040c7-68dd-4a6f-b2da-ffbd6dfce615, StandardOutput: Pushing log4net.Appenders.Contrib <IP_ADDRESS> to 'https://www.nuget.org/api/v2/package'...
[12:42:24], ErrorOutput: Failed to process request. 'A package with id 'log4net.Appenders.Contrib' and version '2.0.4' already exists and cannot be modified.'.
[12:42:24]The remote server returned an error: (409) Conflict..
[12:42:24], ExitCode: 1

I'm not sure if this is a bug or a feature - preventing publishing unless you explicitly update the VERSION.txt is a simplistic way to prevent excessive NuGet publishing of "minor" builds.
How do you think we should proceed?

TeamCity build number is now appended to the build version; so every successful build published to NuGet


Testing compilers fix bug

Hi @shahzebsiddiqui, please let me know if more changes are needed.

Hi @shahzebsiddiqui, please review again and advise. I have addressed all your changes.

Hi @shahzebsiddiqui, I have addressed all changes needed. Please review again!

Hey @shahzebsiddiqui, we are testing the content inside the load now!!

LGTM


GDPR : why avatar image is public and not protected ?

Hello, attached avatar image is stored in public storage.
To be GDPR compliant, could it be stored in protected directory ?
When storing a blob image with :
$user->avatar = Input::file('avatar');
I can't change attribute is_public. Thanks

The RainLab.User plugin is for frontend users, and protected files are locked to being viewed by backend users only. Thus this request makes no sense, and saying that it would need to be protected files "for GDPR compliance" also makes no sense.

Luke I missed that protected files can only being viewed by backend users.
But when I connect to /storage/app/uploads/public/ I can browse all avatars

@Incremental92 that's the fault of your server configuration then, you shouldn't be allowing automatic directory listings.

You're right. I'm developing on local and the Nginx default conf was set to autoindex on;
Thanks, you can delete or close this post.


Installing rCharts package problems

Hi everybody,
I am wondering whether you guys are also having issues with installing the 'rCharts' package for R.
When using install.packages("rCharts") I get the following error message:
Warning in install.packages :
  package ‘rCharts’ is not available (for R version 3.1.1)

When trying to use the code require(devtools) and install_github('rCharts', 'ramnathv') from the website http://ramnathv.github.io/rCharts/ I get:
Downloading github repo ramnathv/rCharts@master
Installing rCharts
"C:/PROGRA~1/R/R-31~1.1/bin/i386/R" --vanilla CMD INSTALL  \
  "C:\Users\a6p\AppData\Local\Temp\Rtmp8qMYZJ\devtools127413d7605b\ramnathv-rCharts-929875d"  \
  --library="C:/Users/a6p/Documents/R/win-library/3.1" --install-tests

* installing *source* package 'rCharts' ...
 Please use ramnathv/rCharts

Have you been able to install the rCharts package? Can you help out?
Thanks a lot.

To whom it may concern:
I tried to follow the instruction on this website: http://ramnathv.github.io/rCharts/
Here is my R console information:

install.packages('Rcpp')
install_github('rCharts', 'ramnathv')

Downloading github repo ramnathv/rCharts@master
Installing rCharts
"C:/PROGRA~1/R/R-31~1.1/bin/i386/R" --vanilla CMD INSTALL "C:/Users/Administrator/AppData/Local/Temp/RtmpAxG5xa/devtools23d4d8e3233/ramnathv-rCharts-389e214" --library="C:/Program
Files/R/R-3.1.1/library" --install-tests

installing source package 'rCharts' ...

** R
** demo
** inst
** preparing package for lazy loading
** help
*** installing help indices
** building package indices
** testing if installed package can be loaded

DONE (rCharts)
Warning message：
Username parameter is deprecated. Please use ramnathv/rCharts

names(iris) = gsub("\.", "", names(iris))
rPlot(SepalLength ~ SepalWidth | Species, data = iris, color = 'Species', type = 'point')
ERROR: No"rPlot" function

My question: Why is there No"rPlot" function?
Thanks!

In the code above it doesn't look like you've loaded the package (library(rCharts)). I would give that a shot first.

Now, it worked. Thanks!


Removing boundary artifacts

Hi @marcomusy,
I have two sub-meshes, bottom and upper parts scanned with a 3d scanner, of a complete mesh which I then register in order to get this complete mesh:

Though the extracted transformation seems to work quite good, if you notice I am getting this kind of artifacts in the boundaries where it seems that the overlapping between the two sub-meshes is not perfect. I've tried to apply a smoothing filter but it didn't really help much thus I wanted to ask you if you have anything in mind that might help to get rid of or decrease the effect of these artifacts.
My initial idea was to trim the boundaries before the registration as discussed here but I still get some of them around the edges:

frag_3a.ply.zip

sorry for the late reply - if i guess correctly the 2 parts of the mesh have some large overlapping... you can remove it by cutting both with a horizontal plane (.cutWithPlane()).
If you aim at a watertight final mesh, what I would try is to create a Ribbon() strip from the 2 boundary lines (you can reorder each of the boundary vertices with .join(reset=True) and finally merge the 3 parts.
If a watertight mesh is not needed you may just warp() one boundary line to the other while keeping fixed most of the mesh points (use for example .decimate() or clean(tol=0.1) to pick points that you want to remain fixed).

sorry for the late reply

No worries.

if i guess correctly the 2 parts of the mesh have some large overlapping... you can remove it by cutting both with a horizontal plane (.cutWithPlane()).

Indeed this is the case, this is due to the fact that the two scanned outputs are not perfectly precise. So if you see here:

import vedo as vd

m1 = vd.Mesh("../data/box_8/frag_3_final.ply").c("green").alpha(0.8)
m2 = vd.Mesh("../data/box_8/frag_3__final.ply").c("red").alpha(0.8)

T = np.array([[-0.9969830354, -0.0742881466, -0.0224966296,<PHONE_NUMBER>662],
                       [0.0534353633, -0.4466713800, -0.8931009687,<PHONE_NUMBER>664],
                       [0.0562982151, -0.8916086303, 0.4492934024,<PHONE_NUMBER>738],
                       [0.0000000000, 0.0000000000, 0.0000000000, 1.0000000000]])

m3 = m2.clone().applyTransform(T, reset=True)

vd.show(m1, m2, "Before", at=0, N=2, axes=1)
vd.show(m1, m3, "After", at=1, interactive=True).close()

There is always some kind of overlapping either from one or the other (notice that sometimes the green is above red and vice versa), this also causes some kind of self intersecting faces problem where when the to surfaces are not perfect aligned I get some kind of artificial "bubbles" from the two overlapped surfaces. Thus, I am not sure that just cutting with a horizontal plane would work. In principle I would get the same result as with trimming the boundaries. An idea to address the "bubble" issue would be to solidify the mesh by filling it in so that I do not have any hollow inner parts anymore but then still on the boundaries I would have these parts extending out of the main core of the mesh. These parts I would like to trim out.

If you aim at a watertight final mesh, what I would try is to create a Ribbon() strip from the 2 boundary lines (you can reorder each of the boundary vertices with .join(reset=True) and finally merge the 3 parts. If a watertight mesh is not needed you may just warp() one boundary line to the other while keeping fixed most of the mesh points (use for example .decimate() or clean(tol=0.1) to pick points that you want to remain fixed).

Can you elaborate a bit more on these, in practice I would like to get a watertight mesh (I am planning to 3D print the pieces) but more important is to manage to get a final mesh with as much as possible less artifacts on the alignment. From my experience transforming a non-watertight mesh to a watertight one is quite easy once you have a well reconstructed mesh.
frags.zip

I had a try with it:
import vedo as vd
import numpy as np

m1 = vd.Mesh("frag_3_final_dec.ply").c("green4").alpha(1)
m2 = vd.Mesh("frag_3__final_dec.ply").c("red4").alpha(1)

# m1.decimate(0.1).write('frag_3_final_dec.ply')
# m2.decimate(0.1).write('frag_3__final_dec.ply')

T = np.array([[-0.9969830354, -0.0742881466, -0.0224966296,<PHONE_NUMBER>662],
               [0.0534353633, -0.4466713800, -0.8931009687,<PHONE_NUMBER>664],
               [0.0562982151, -0.8916086303, 0.4492934024,<PHONE_NUMBER>738],
               [0.0000000000, 0.0000000000, 0.0000000000, 1.0000000000]])

m3 = m1.clone()
m4 = m2.clone().applyTransform(T, reset=True)

a=60
m3.RotateX(a)
m4.RotateX(a)

m3.cutWithPlane((0,0,205),'-z')
m4.cutWithPlane((0,0,206),'z')

b3 = m3.boundaries().extractLargestRegion().c('green6')
b4 = m4.boundaries().extractLargestRegion().c('purple6')

pts3 = m3.points()
pts4 = m4.points()
pcl3 = vd.Points(pts3)
pcl4 = vd.Points(pts4)

lines=[]
pb = vd.ProgressBar(0, b3.NPoints())
for p in b3.points():
	pb.print()
	idx3 = pcl3.closestPoint(p, returnPointId=True)
	idx4 = pcl4.closestPoint(p, returnPointId=True)
	lines.append([pts3[idx3], pts4[idx4]])
	pts3[idx3] = pts4[idx4]

m3.points(pts3) # update mesh vertices to seal the gap
m = vd.merge(m3,m4).clean().c('indigo5')

lns = vd.Lines(lines, lw=2)
vd.show(m3, m4, b3, b4, lns, axes=1)
vd.show(m, b3, b4, axes=1).close()

you'll need to uncomment the two decimate() lines first time you run to generate the downsampled eample.

Thanks Marco,
I had a look on your solution I also played a bit with different options but I didn't like the output that much, it seems that I am still getting some artifacts where the connection between the two parts are. However, while I was trying to understand your code I got another idea for which I would need your opinion and possibly your help.
If I load the initial merged and extract the boundaries as you can see it gives me the points that are extending from the main body of the mesh:

Now since I have this points I can iterate though and delete them, however as you can see at some point some of the points go in the inner side thus what I wanted to do is that once a point is in the inner part of the mesh to exclude it from deletion. However, I am not sure how to define whether a boundary point is in the outer or inner side of the mesh.
import vedo as vd

m = vd.Mesh("../data/box_8/frag_3a.ply").c("gray")
b = m.boundaries().c('red')
vd.show(m, b, axes=1).close()

while b.points().any():
    idxs = []
    for p in b.points():
        if p is outer: # <--- here I should somehow filter inner and outer boundary points
            idx = m.closestPoint(p, returnPointId=True)
            idxs.append(idx)

    if not idxs:
        break

    m.deletePoints(idxs, renamePoints=0)
    b = m.boundaries().c('red')
    vd.show(m, b, axes=1).close()

Any idea how to define inner and outer boundary points?

There is a method called isInside() to check for that - but I really think the method would not work because you will easily lose track of the face-vertex connectivity!
Another idea could be to still cut with the plane to remove each side of the 2 meshes but then just forget about all faces and recreate the surface with recoSurface(), check:
examples/advanced/recosurface.py

There is a method called isInside() to check for that - but I really think the method would not work because you will easily lose track of the face-vertex connectivity!

Indeed, this didn't work plus it was quite slow.

Another idea could be to still cut with the plane to remove each side of the 2 meshes but then just forget about all faces and recreate the surface with recoSurface(), check: examples/advanced/recosurface.py

Finally I found another solution, which seems to work quite nicely, by using the poisson reconstruction filter from pymeshlab:

as you can see it seems to do quite some nice job.
And here is the code snippet for the reference:
import vedo as vd
import pymeshlab

vd_mesh = vd.Mesh("../data/box_8/frag_3a.ply").c("gray")

m = pymeshlab.Mesh(vertex_matrix=vd_mesh.points(), face_matrix=vd_mesh.faces())
ms = pymeshlab.MeshSet()
ms.add_mesh(m)
ms.surface_reconstruction_screened_poisson(depth=8, pointweight=1, preclean=True)

mlab_mesh = ms.current_mesh()

reco_mesh = vd.Mesh(mlab_mesh)

vd.show(vd_mesh, "Before", at=0, N=2, axes=1)
vd.show(reco_mesh.c("gray"), "After", at=1, interactive=True).close()

As far as I understood, by checking on the manual, as well on some videos the result should be manifold (also checking on some of the output files they seem to be).
For the overlapping regions the parameters that I used seem to do the trick for most of my models. Thus, I would stick with that and actually in the cases that it fails is due the bad retrieved sub-meshes. Yes, I noticed that the result from the meshlab lib is (or looks at least) quite good.


unicode normalization

Added Normalizer to normalize unicode strings for comparison.
Supports two levels of normalization:

regular: NFC
aggressive: NFKC, casefold, diacritic removal, some extra character replacements

I think that the Normalizer may be useful beyond Odinson, so we may want to move it to common in the future, but I think it should live here for a while first.


When cloning reports to another workspace, we get "bad request"

Hi,
In ProvisionSample, When cloning reports to another workspace, we get "bad request".  The report clone indeed works in the same workspace.
Is this a bug or do I miss anything?

I have a report in workspace X
Want to clone the report from workspace X to workspace Y
I'm not specifying the dataset id, as it is an optional value and I assume will be cloned as well

@wbreza @t-yoigra @bertrandBot

I had same issues here, I clone the report to another workspace and different dataset.

Anyone please help, huhuhu

Has this been resolved?? I am using Microsoft.PowerBI.Api v 2.0.2

I am having the same issue here. We get "bad request", when the targetModelId(datasetId) is not specified. However, this should be optional, right?

I am having the same issue, but I suspect that the documentation is wrong and that the targetModelId parameter is only optional when cloning within the same workspace.


Unnecessary allocation in pointer!() macro?

As it currently stands, pointer!() macro allocates a Vec (or a boxed slice, rather). Why is it necessary where a simply array would do? An array also implements IntoIterator, so you could just have [pointer_node1, pointer_node_2, pointer_node3] without extra allocations.
It's understandable that it's nice to be able to .push() into it, but having an allocation by default would really hurt the most common case where you just use pointer!() as is, and is a bit un-rust-like. Perhaps the vec-like pointer! could be a separate macro? Or, you can always do an pointer!(...).into_vec() if you need to mutate it.
Suggestion:
#[macro_export]
macro_rules! pointer {
    () => (
        ([] as [$crate::PointerNode; 0])
    );
    ($($x:expr),+ $(,)?) => (
        [$($crate::PointerNode::from($x)),+]
    );
}
``

Yes, an array can be used here to reduce an allocation.
This is a break change and will be published in sonic-rs 0.4.