Add a 'Focus mode'

Is your feature request related to a problem? Please describe.
The right-side panel is kinda useless after you make the entries to the new terms.
Describe the solution you'd like
There should be a 'Focus mode' where the user can disable the right-side panel, have the text centered with some amount of padding on both sides, and the option to hide/show highlight.
Describe alternatives you've considered
Moving the side panel all the way to the right, but there's no option for padding now, so it makes text too wide and tedious to read.

I had thought: it would be nice if we could "dock" the right-side panel by moving the divider all the way to the side. It won't open unless you drag it back open (or maybe a little tab?). Doesn't need to load anything but the tooltips when it's closed over. Possible?

In progress, pretty much done in #262 .  @M-Biggles your suggestion is good but we've gone with a "toggle" button for now.  ... Let's see how this new design idea works out.

In progress, pretty much done in #262 . @M-Biggles your suggestion is good but we've gone with a "toggle" button for now. ... Let's see how this new design idea works out.

Sounds great! Looking forward to it :)

Launched in 3.2.1.


Add void return type to void methods

Right now this doesn't build with noImplicitAny set to true.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks for taking the time to review @bondz!

#618 already covers this BTW, this is redundant.

Thats true @bondz, this is solved by the previous PR #619.
@jedborovik, please add your vote to PR #619 to gain it priority for merging.
Thanks!


Authors, editors or creators search no results

Story
I can search through the UGent database to add affiliated authors, editors or creators – and if I get no results, I get guidance.
Epic: https://github.com/ugent-library/biblio-backend/issues/603
Success criteria

[ ] If the name I entered as a search query does not result into matches, I get the option to select an external person and a message I can refine my results.

Related: search results https://github.com/ugent-library/biblio-backend/issues/605
Related: blank state https://github.com/ugent-library/biblio-backend/issues/604
Design
No matches found

If the name does not result into matches, I get the option to select an external person and a message I can refine my results.

Variation A: add role in next screen

Variation B: directly add role

Implementation
Components
View components at https://github.com/ugent-library/biblio-backend/issues/603
Code example
Full layout within modal body:

Search results

      <button class="btn btn-secondary">
        Add as Lesly Joseph as external author

      </button>

No results found in the UGent database. Adapt your search query, or add this person as an external author.

Help message list item:

No UGent staff found.

Omit letters for a broader search (e.g. only enter the last name, or the last name and a latter of the first name), or add this person as an external author.

[Out of scope, for documentation purposes only]
Variation B: directly add role

Variation with External as more prominent option:
More explicit, but more cluttered too.


[bugfix] Fix persisted chromadb vectorstore

If a persist_directory param was set, chromadb would throw a warning that ""No embedding_function provided, using default embedding function: SentenceTransformerEmbeddingFunction". and would error with a Illegal instruction: 4 error.
This is on a MBP M1 13.2.1, python 3.9.
I'm not entirely sure why that error happened, but when using get_or_create_collection instead of list_collection on our end, the error and warning goes away and chroma works as expected.
Added bonus this is cleaner and likely more efficient.  list_collections builds a new Collection instance for each collect, then Chroma would just use the name field to tell if the collection existed.

@timothyasp theres been lots of dev on chroma side - is this error still happening with recent versions?

I'm seeing this right now with chromadb==0.3.11 and langchain==0.0.105
So yes, I think it's still happening. And I'm using the persist_directory parameter.

Yea, I believe so, I’ve been using this patched code now and been pulling
pretty much daily from both upstreams, and things have been working.
On Wed, Mar 8, 2023 at 8:17 PM Harrison Chase @.***>
wrote:

@timothyasp https://github.com/timothyasp theres been lots of dev on
chroma side - is this error still happening with recent versions?
—
Reply to this email directly, view it on GitHub
https://github.com/hwchase17/langchain/pull/1444#issuecomment-1461255453,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFMY4ZX62S45XEIVH6VZZTW3FKV5ANCNFSM6AAAAAAVP57MYA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
-Tim

@hwchase17 any chance we can get this merged into a patch release?

any update on this ? very confusing error. i'm not sure I understood how you fixed it

my bad didn't see the commit attached.
Hopefully it will be merged and released, that's an annoying issue

tried the fix
looks good to me
looking forward for a merge in a new release please, this issue is annoying

@hwchase17 can we get this merged?  It seems to be impacting a lot of people

In the meantime I forked the original repos and applied the patch (to be up to date with latest commit).
You can switch your langchain by langchain@git+https://github.com/lambda-science/langchain.git in a simple requirements.txt.
Not optimal at all for now, but it does the job

merging in! sorry for delay!

@hwchase17 I am still getting this error on version 0.0.146

@hwchase17  Can confirm this is still an issue in 0.0.161

Do we know if this error has been resolved I am getting an error message as follows: "You must provide embeddings or a function to compute them". How do I patch my current vectorstore chromadb file?


Provide builds using Operating System encoders

Using the os encoder decoders from ffmpeg also exposes h.264 from the os where the codec should be theoretically properly licensed.

I guess the ideal is that they are actually available for h264 on all platforms because it's super popular? Also would need a way to deal with the possibility that it's not (probably not going to be there on Linux(

Cisco has made efforts to provide a fully licensed decoder.

As long as it's in your distro :) just need a way gracefully keep running if it's not


Fixed panic with leafnode and gateway when no interest registered

Say there are 2 clusters, A and B. A client connects to A and
publishes messages on an account that B has no interest in.
Then a leaf node server connects to B (using same account than
the no-interest is for). Cluster B will ask cluster
A to switch to interest mode only for leaf node account. This
would cause a panic.
Signed-off-by: Ivan Kozlovic<EMAIL_ADDRESS>

Coverage decreased (-0.02%) to 90.757% when pulling dce9d672c1492361f8a4bd8e95de3aefb213de88 on fix_panic_with_leafnodes_and_gws into f04fb3775dc696e9b609316fb8494b804195736d on master.


Save raw pcap data

Currently, we are collecting pcap data, but we are not saving it anywhere. It may need to be saved to a separate bucket, similar to file writes

This will be great to have.


perspective version of function panoEdgeDetection

Is the perspective image version of function panoEdgeDetection available somewhere in the code base? I can only find the pano one.
Thank you!

Sorry, I realized that the edge map is not necessary for the perspective images. Closing this.


Update README.md

Coverage remained the same at 100.0% when pulling bf1fb4b1f430478d5735b4b76b17f4195e892535 on wilsongp-patch-1 into<PHONE_NUMBER>6569de557b5cabd3bf05756f6ffd6b on development.


Bucket Replication Support

Thanks for the very handy module!
I've recently needed to add S3 replication to a bucket managed by this module. Unfortunately, the replication configuration is a part of the s3_bucket resource and can't be modified independent of that resource in Terraform. That means this module would have to support optional replication configuration.
My goal is to get support for configuring a buckets created by this module to be replication sources. Specifically looking at cross-account, same-region, replication support. Support for creating the replication service roles would be a nice to have as well. :smile:

@davidski it looks like S3 replication support was added in #56 but I guess you would have solved it by now! :D


Printing tweaks 2

This is just #563 , but the CI is happy now for reasons I do not understand

Deeply confused at CI.


User Edit Button Functionality

The edit button for the user modal is set up and needs to connect with the back end once that is completed.

Looks good!


vertex features and custom vertex ids

This is on the current titan09 HEAD (a0ba586cf56ac191020e113b0661e05aad2f06cd).
@dkuppitz pointed out this behavior:
gremlin> graph.features().vertex().supportsCustomIds()
==>true
gremlin> graph.addVertex(id, 1)
Vertex does not support user supplied identifiers
Display stack trace? [yN]

Titan's graph.set-vertex-id still exists, but graph.addVertex does not appear to respect it, despite some sentences in its javadoc comment that talk about validity checking on user-supplied vertex IDs.  I think none of Titan's vertex features check set-vertex-id.  Some of Titan's other vertex feature methods also seem to have incorrect behavior, such as:
gremlin> g.features().vertex().supportsStringIds()
==>true

I got confused when I tried to figure out how all of the various vertex features interact.  For example, how precisely does supportsCustomIds() differ from supportsUserSuppliedIds()?  Only the latter appears in TP3's own unit tests -- maybe the former is something vestigial?  Neither appears in the user documentation.
Some TP3 references:

Javadoc comment above Graph.vertices declaration describing how a subset of vertex features interacts with its method parameters (this is just the read side): https://github.com/apache/incubator-tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java#L149-L197
ElementFeature method declarations (+ their method javadoc): https://github.com/apache/incubator-tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java#L571-L619

At this point, I'm pretty confident that at least some of Titan's vertex feature methods are incorrect, but I'm not confident that I understand the semantics of TP3 vertex features well enough to completely correct it.  I would appreciate guidance from @spmallette or @okram.

@dkuppitz on HipChat says:

https://github.com/apache/incubator-tinkerpop/blob/master/gremlin-core/src/main/java/org/apache/tinkerpop/gremlin/structure/Graph.java#L726-L742
anyIds => just eat it
customIds => eventually allow any id, but validate it first

I guess the javadoc wasn't as clear as I thought.  supportsUserSuppliedIds means that a user can specify the identifier - so g.addV(id,123) should work.  The other related features:

supportsNumericIds
supportsStringIds
supportsUuidIds
supportsCustomIds
supportsAnyIds

refer to how the id is internally represented in the graph.  So, if you store it as a long, then you should set supportsNumericIds to true.  If the graph uses a specific custom object for an id, like OrientDBs RID (RecordID), then it would set supportsCustomIds to true.  You only set supportsAnyIds to true if the graph can take any Java object as an identifier - like TinkerGraph.
I have a tough time writing tests for supportsCustomIds because we don't have a graph internally that supports them.
Does that help?  I'll try to tack in some more comments to the javadoc.

Thanks for explaining.  From your comment and a little bit of back and forth in chat, it now sounds like we should do the following in Titan:
Vertex features:

supportsNumericIds=true
supportsUserSuppliedIds=[value of graph.set-vertex-id in the underlying config]
all other vertex feature support methods, including supportsCustomIds=false

Edge features:

supportsCustomIds=true (because edges are identified by a specific non-numeric, non-UUID, non-String relation ID type)
all other edge feature support methods=false


GPM didn't load

Issue with loading GPM.
Ubuntu 19.10 amd64
4.6.1 (from Ubuntu store)
GMP issue
For my case just need to open GPMDP

Please see the pinned issues.


[Bug]: minDynamicHeight shows error even if it is less than maxDynamicHeight

Is there an existing issue for this?

[X] I have searched the existing issues

Description
minDynamicHeight shows error even if it is less than maxDynamicHeight.
Steps To Reproduce

Drag a Checkbox group widget
Enable Auto Height with limits
Set Max height to 20
Set Min Height to 25
You will see a valid error that min height should be less than equal to max height.
Now set Max height to 30.
It still shows that error in point 5.

Public Sample App
No response
Version
Cloud

@riodeuno

@ankurrsinghal : when minHeight throws error, signifier minHeight stay as it is. Check screenshot

@ankurrsinghal : let's keep it open till #17159  is fixed

@rohitagarwal88 shall we close this ? As we have right now removed he limits from property pane , This can be dealt later


Switch away from my local ip, test failure mode

r @raynos

I think requiring leaked-handles might create a leak...

Tests will probably pass if you remove leaked-handles.

lgtm.


Add Caltech Pedestrian Detection dataset

Add Caltech Pedestrian Detection dataset to app.activeloop.ai with the correct version.
For more information regarding creating and uploading a dataset please see the Hub documentation.

Hey @kristinagrig06 want to work on this.

@sparkingdark , you can find some examples of dataset loading and uploading here: https://github.com/activeloopai/Hub/tree/master/examples

Thanks Will do it soon and check the example dataset is there any discord or slack channel regarding communication @kristinagrig06

Hi @sparkingdark, we actually got into the beta access of Github Discussions (https://github.com/activeloopai/Hub/discussion). Would love to get your feedback, otherwise, if this doesn't work, we will likely revert to Slack.

I got error 404

@sparkingdark sorry about that, now the link should be fixed.

Yeah Now I can access and @davidbuniat can add new datasets as well with opening a issue?

Sure @sparkingdark if you feel that some dataset would be a good addition to the platform, feel free to create a new issue for it!

Hi, @sparkingdark ! Hope this finds you well. Dropping a note to check in on you an ask if you need a hand with uploading the dataset. Feel free to ask us in the GitHub Discussions (we have beta access!) or our dedicated Slack channel.
We've also added a couple of other datasets that we think will be a good addition to the platform - let us know if you'd like to take on some of them, too!
Thanks a mil!

That's awesome actually I am quite busy with some other work that's why can't focus on this work for some days.But after finishing the work I will totally starting working on this and love to get some other dataset as well.
For now if I get some guide on uploading a dataset on activepool/hub like how to start that will be great.

Hey @sparkingdark - thanks so much for letting us know. Did you check this? Also, some basic info is available in our readme. Let us know if that information is insufficient, this is valuable feedback for us! Good luck with all the work this week and hit us up in case you have any questions!

Hey @mikayelh send me the slack invite link I can not access the channel.

Heya @sparkingdark - the link seems to be working fine for other users - here it is again. Do you have a Slack workspace?

If not, can you send me your email and I'll gladly send you an invite to the channel!

<EMAIL_ADDRESS>

Sent an invite, let me know if it works for you!
On Wed, Oct 21, 2020 at 9:00 AM Debojyoti Chakraborty <
<EMAIL_ADDRESS>wrote:

<EMAIL_ADDRESS>—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/activeloopai/Hub/issues/58#issuecomment-713354654,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALM4YO4RUMWIB6RP5F5MCA3SL2BHVANCNFSM4SKO4BPA
.

Joined and thanks for inviting me.

Closing this due to inactivity, will reopen if theres widespread interest.


allow for blank ip addresses

I found that in some CI systems they'll often do things like --ip=$SOME_IP in the case that $SOME_IP is blank or a blank string it would be helpful if we didn't fail out right away and allow for defaulting, this is somewhat similar to https://github.com/spf13/pflag/pull/84 and related to https://github.com/kubernetes/kubernetes/issues/58686

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


loading-lazy-attr enabled by default in Chrome 76+

According to Chrome platform status (https://www.chromestatus.com/feature/5645767347798016) this feature is enabled by default in version 76.

loading-lazy-attr is not enabled by default in Chrome 76 (which is currently in Beta), and it's not in Chrome 77 (Canary) either.
You can test yourself at: https://mathiasbynens.be/demo/img-loading-lazy
From what I gather, there are still some bugs and issues to work out before it's on by default:
REF: https://groups.google.com/a/chromium.org/forum/m/#!topic/blink-dev/jxiJvQc-gVg

Sorry! Thank you for the information.


Why does the FormPlot user control no longer forward mouse events?

Due to the refactoring of the WinForm control by adding a picturebox in the FormsPlot user control the mouse events like OnMouseDown are no more forwarded like in the laste stable release.
A fix would be something like this
private void PictureBox1_MouseDown(object sender, MouseEventArgs e)
{
    Backend.MouseDown(GetInputState(e));
    base.OnMouseDown(e);
}

Or is it by intension to remove that?

Thanks for mentioning this @grabul!
OnMouseMove is forwarded, but it would be good to forward all mouse events too:

OnMouseDown

OnMouseUp

OnDoubleClick

onMouseWheel

https://github.com/ScottPlot/ScottPlot/blob/591c80560618787b844e7430c9426c70c83900db/src/controls/ScottPlot.WinForms/FormsPlot.cs#L115-L119


Added processPbrMetallicRoughness

@lilleyse, this is the file that has been tested within Cesium.

@lilleyse, all the models from the sample repo have been tested and load correctly into Cesium when passed through the pipeline.

The latest code changes look good.
There are some jsHint errors and test failures.
(Yes, this branch and 2.0 are pre-eslint).

Thanks @moneimne. Will merge this into the 2.0 branch and then leave that parked for a little while.


CHORE: updates to readme and docs

Ticket URL: paste URL here
About these changes
I set up a local build this morning, and these are a few minor tweaks to the README and docs that I think could be useful based on that experience.
How to check these changes
Review the revised instructions and check that they make sense.
Where possible, provide guidance to help your reviewer
Before assigning to reviewer, please make sure you have

[x] Checked things thoroughly before handing over to reviewer.
[x] Checked PR title starts with ticket number as per project conventions to help us keep track of changes.
[x] Ensured that PR includes only commits relevant to the ticket.
[x] Waited for all CI jobs to pass before requesting a review.
[x] Added/updated tests and documentation where relevant.

Merging PR guidance
Follow docs\developer-guide\contributing.md
Deployment guidance
Follow docs\infra\environments.md

@ahosgood I've assigned this to Jane to review but would be great if you also have time to check it over.


Fixed hard dependencies on pandas

There were two hard dependencies on pandas, which are resolved by this PR. The first involved import the PandasInterface, which is not available unless pandas was installed. The other was an issue in the bokeh backend outlined in #589.

Ready to merge.


Strange Adafruit TFT FeatherWing - 3.5" 480x320 issue

Hi
I'm using the Adafruit 3.5"  TFT Featherwing with an Adafruit Huzzah32 board and having an issue.
The display appears to be working fine using this example code in that it it shows the write/list/chart example screens. However the touch operation is not working, and every 3s roughly the screen switches to the next of the three tabs in the example cycling around them. I am using the configuration for the TFT featherwing in menuconfig but this appears to also select settings for the touch controller without a way to change the CS so I'm not sure if it is this.
I'm building on Windows with ESP-idf 4.0.1 and everything compiles and runs without error but not what I'd expect on screen.

However the touch operation is not working, and every 3s roughly the screen switches to the next of the three tabs in the example cycling around them.

The example is configured to automatically switch tabs.
We're about to merge #110 into master, could you test it again later today? I't updates to LVGL v7 an several bug fixes.

@petestewardson-schneider https://github.com/lvgl/lv_port_esp32/blob/c1f43bfb090df293059ab52baabbdbdd8df00712/components/lv_examples/lv_ex_conf.h#L45

OK so I've updated to the latest code after the merge  and disabled the slideshow as above. I'm still getting no touch input in the example app.
I'm new to using esp-idf but is there anyway I can help to diagnose with the HW I have?

Hi,
I've attached the config, but there is clearly something not correct when using the monitor command ( I didn't know about that one so thanks )
`--- idf_monitor on COM8 115200 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
ets Jun  8 2016 00:22:57
rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0030,len:4
load:0x3fff0034,len:7064
load:0x40078000,len:13072
ho 0 tail 12 room 4
load:0x40080400,len:3896
0x40080400: _init at ??:?
entry 0x40080688
I (30) boot: ESP-IDF v4.1-beta2 2nd stage bootloader
I (30) boot: compile time 10:52:01
I (30) boot: chip revision: 1
I (33) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (40) boot.esp32: SPI Speed      : 40MHz
I (45) boot.esp32: SPI Mode       : DIO
I (50) boot.esp32: SPI Flash Size : 2MB
I (54) boot: Enabling RNG early entropy source...
I (60) boot: Partition Table:
I (63) boot: ## Label            Usage          Type ST Offset   Length
I (70) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (78) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (85) boot:  2 factory          factory app      00 00 00010000 00100000
I (93) boot: End of partition table
I (97) boot_comm: chip revision: 1, min. application chip revision: 0
I (104) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x0c54c ( 50508) map
I (132) esp_image: segment 1: paddr=0x0001c574 vaddr=0x3ffb0000 size=0x0225c (  8796) load
I (136) esp_image: segment 2: paddr=0x0001e7d8 vaddr=0x40080000 size=0x00404 (  1028) load
0x40080000: _WindowOverflow4 at C:/Users/SESA406760/Desktop/esp-idf_4_1/esp-idf/components/freertos/xtensa_vectors.S:1778
I (140) esp_image: segment 3: paddr=0x0001ebe4 vaddr=0x40080404 size=0x01434 (  5172) load
I (150) esp_image: segment 4: paddr=0x00020020 vaddr=0x400d0020 size=0x43e90 (278160) map
0x400d0020: _stext at ??:?
I (263) esp_image: segment 5: paddr=0x00063eb8 vaddr=0x40081838 size=0x09950 ( 39248) load
0x40081838: esp_intr_enable at C:/Users/SESA406760/Desktop/esp-idf_4_1/esp-idf/components/esp32/intr_alloc.c:826
I (286) boot: Loaded app from partition at offset 0x10000
I (286) boot: Disabling RNG early entropy source...
I (287) cpu_start: Pro cpu up.
I (290) cpu_start: Application information:
I (295) cpu_start: Project name:     lvgl-demo
I (300) cpu_start: App version:      d01c8d3
I (305) cpu_start: Compile time:     Jun 11 2020 11:09:07
I (311) cpu_start: ELF file SHA256:  18f1ef3fde52e208...
I (317) cpu_start: ESP-IDF:          v4.1-beta2
I (323) cpu_start: Starting app cpu, entry point is 0x400811fc
0x400811fc: call_start_cpu1 at C:/Users/SESA406760/Desktop/esp-idf_4_1/esp-idf/components/esp32/cpu_start.c:275
I (0) cpu_start: App cpu up.
I (333) heap_init: Initializing. RAM available for dynamic allocation:
I (340) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (346) heap_init: At 3FFCE010 len 00011FF0 (71 KiB): DRAM
I (352) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (359) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (365) heap_init: At 4008B188 len 00014E78 (83 KiB): IRAM
I (371) cpu_start: Pro cpu start user code
I (389) spi_flash: detected chip: gd
I (390) spi_flash: flash io: dio
W (390) spi_flash: Detected size(4096k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (400) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (20) lvgl_helpers: Display hor size: 480, ver size: 320
I (20) lvgl_helpers: Display buffer size: 19200
I (20) lvgl_helpers: Initializing shared SPI master
I (30) lvgl_helpers: Configuring SPI host VSPI_HOST (2)
I (30) lvgl_helpers: MISO pin: 19, MOSI pin: 18, SCLK pin: 5
I (40) lvgl_helpers: Max transfer size: 38400 (bytes)
I (50) lvgl_helpers: Initializing SPI bus...
I (50) disp_spi: Adding SPI device
I (50) disp_spi: Clock speed: 26000000Hz, mode: 0, CS pin: 15
I (190) HX8357: Initialization.
I (990) STMPE610: Initialization.
I (1060) STMPE610: SPI_CFG = 0x8
E (1060) STMPE610: Incorrect version: 0x0`
sdkconfig.txt

I'm just trying to get this working on the FeatherWing as well. Display is working but touch not working. I've tested the hardware with the Arduino touchpaint_featherwing example application and both display and touch works.
Is there anything I can try?
sdkconfig.txt
Same error in idf.py monitor
--- idf_monitor on COM6 115200 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
ets Jun  8 2016 00:22:57

rst:0x1 (POWERON_RESET),boot:0x13 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0018,len:4
load:0x3fff001c,len:6924
load:0x40078000,len:14076
ho 0 tail 12 room 4
load:0x40080400,len:4304
entry 0x400806e8
I (71) boot: Chip Revision: 1
I (71) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (41) boot: ESP-IDF v4.0.1-dirty 2nd stage bootloader
I (41) boot: compile time 11:58:53
I (41) boot: Enabling RNG early entropy source...
I (46) boot: SPI Speed      : 40MHz
I (50) boot: SPI Mode       : DIO
I (54) boot: SPI Flash Size : 2MB
I (58) boot: Partition Table:
I (62) boot: ## Label            Usage          Type ST Offset   Length
I (69) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (77) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (84) boot:  2 factory          factory app      00 00 00010000 00100000
I (92) boot: End of partition table
I (96) boot_comm: chip revision: 1, min. application chip revision: 0
I (103) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x0c618 ( 50712) map
I (130) esp_image: segment 1: paddr=0x0001c640 vaddr=0x3ffb0000 size=0x021b4 (  8628) load
I (134) esp_image: segment 2: paddr=0x0001e7fc vaddr=0x40080000 size=0x00400 (  1024) load
0x40080000: _WindowOverflow4 at C:/Users/richa/Desktop/esp-idf-2/components/freertos/xtensa_vectors.S:1778

I (137) esp_image: segment 3: paddr=0x0001ec04 vaddr=0x40080400 size=0x0140c (  5132) load
I (148) esp_image: segment 4: paddr=0x00020018 vaddr=0x400d0018 size=0x43b90 (277392) map
0x400d0018: _stext at ??:?

I (254) esp_image: segment 5: paddr=0x00063bb0 vaddr=0x4008180c size=0x0990c ( 39180) load
0x4008180c: esp_intr_enable at C:/Users/richa/Desktop/esp-idf-2/components/esp32/intr_alloc.c:810

I (278) boot: Loaded app from partition at offset 0x10000
I (278) boot: Disabling RNG early entropy source...
I (279) cpu_start: Pro cpu up.
I (282) cpu_start: Application information:
I (287) cpu_start: Project name:     lvgl-demo
I (292) cpu_start: App version:      d144046
I (297) cpu_start: Compile time:     Jul  2 2020 12:42:05
I (303) cpu_start: ELF file SHA256:  9f229b833fdb4bfd...
I (309) cpu_start: ESP-IDF:          v4.0.1-dirty
I (314) cpu_start: Starting app cpu, entry point is 0x40081210
0x40081210: call_start_cpu1 at C:/Users/richa/Desktop/esp-idf-2/components/esp32/cpu_start.c:271

I (0) cpu_start: App cpu up.
I (325) heap_init: Initializing. RAM available for dynamic allocation:
I (331) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (337) heap_init: At 3FFCE760 len 000118A0 (70 KiB): DRAM
I (344) heap_init: At 3FFE0440 len 00003AE0 (14 KiB): D/IRAM
I (350) heap_init: At 3FFE4350 len 0001BCB0 (111 KiB): D/IRAM
I (356) heap_init: At 4008B118 len 00014EE8 (83 KiB): IRAM
I (363) cpu_start: Pro cpu start user code
I (381) spi_flash: detected chip: generic
I (381) spi_flash: flash io: dio
W (381) spi_flash: Detected size(4096k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (392) cpu_start: Starting scheduler on PRO CPU.
I (0) cpu_start: Starting scheduler on APP CPU.
I (20) lvgl_helpers: Display hor size: 480, ver size: 320
I (20) lvgl_helpers: Display buffer size: 19200
I (20) lvgl_helpers: Initializing shared SPI master
I (30) lvgl_helpers: Configuring SPI host VSPI_HOST (2)
I (30) lvgl_helpers: MISO pin: 19, MOSI pin: 18, SCLK pin: 5
I (40) lvgl_helpers: Max transfer size: 38400 (bytes)
I (50) lvgl_helpers: Initializing SPI bus...
I (50) disp_spi: Adding SPI device
I (50) disp_spi: Clock speed: 26000000Hz, mode: 0, CS pin: 15
I (190) HX8357: Initialization.
I (990) STMPE610: Initialization.
I (1070) STMPE610: SPI_CFG = 0x8
E (1070) STMPE610: Incorrect version: 0x0

I've not been able to address this issue so far.

Is there any help anyone can give on this? It seems it's confirmed not to be working but I don't know where to start to debug.

Is there any help anyone can give on this? It seems it's confirmed not to be working but I don't know where to start to debug.


Update New-SPBusinessDataCatalogServiceApplication.md

Filled in placeholder text.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/New-SPBusinessDataCatalogServiceApplication.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/New-SPBusinessDataCatalogServiceApplication.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Bug report] create catalog error with properties shows key already exist

Version
main branch
Describe what's wrong
Properties key error when creating catalog multiple times.
Error message and/or stacktrace
N/A
How to reproduce

create a catalog
create the same properties catalog

Additional context
No response

After multiple tests, I couldn't reproduce this bug. If it happens again, I will reopen it.


Add state tag configuration setting

The tag-key used to determine the state of an AMI can now be set with the AMIQUERY_STATE_TAG environment variable. The default value if unspecified is "state".
The aliased query parameter "status" still exists and "state" has been added, as well.
In addition, the tag query parameter now properly accepts tag values with colons, e.g. tag=foo:bar:baz translates to foo=bar:baz.
/cc @moofish32 @thomaswhitcomb @ellery44 @jk-intuit

@jmassara Looks good to me. 👍


Don't grab GIL from C-Core during shutdown (retry with fix)

This is a retry of #37762 with a fix. We no longer hold the C mutex and GIL at the same to avoid a deadlock.
MetadataPluginCallCredentials passes a function pointer to C-core to be called during destruction. This function grabs GIL which may race between GIL destruction during process shutdown. Since GIL destruction happens after Python's exit handlers, we cana mark that Python is shutting down from an exit handler and don't grab GIL in this function afterwards using a C mutex.

FYI, #37762 broke +100 tests in google3 and all of them now pass with +100 runs.


feat: add forceskip and voting system to skip

Closes #44.
These are the 3 situations that can happen with the /skip command, which I exemplified below:

There are 3 users listening to the bot. One requests a skip, but it does not constitute a majority.
One of the users disconnected, therefore there were only two users listening to the bot, and a single vote constitutes a "majority vote" insert PS joke here.
Nothing is playing.

Quality PR, I like the choices that were made here, even if I originally envisioned interaction-based voting. This approach, while less flashy, ends up being more functional and less error-prone.

Thanks! 🚀  In addition to being more functional and less error-prone, something that I didn't like about the interaction based approach is that it would make it too easy to skip something, and, in my opinion, it would be bad UX for this functionality.
I'll look at the other comments now!


[ALLUXIO-3168] Web UI and CLI master configuration broken

https://alluxio.atlassian.net/browse/ALLUXIO-3168

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19402/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19408/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.client.rest.AlluxioMasterRestApiTest.getConfiguration
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19409/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19415/Failed Tests: 1org.alluxio:alluxio-tests: 1alluxio.server.ft.journal.JournalShutdownIntegrationTest.multiMasterMountUnmountJournal
Test FAILed.

@AmplabJenkins test this please

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19417/
Test PASSed.

Merged build finished. Test PASSed.

@apc999 I previously describe the issue and reason in the Jira ticket, now I copy it to the description section of this PR to make it clear. Thanks for reminding me.
@aaudiber @apc999  PTAL, thanks!

According to the discussion with @aaudiber  and @apc999, we decide to put all the configurations inside Configuration.java instead of putting default properties inside PropertyKey.java. This is reasonable since we have the assumption that we do not change configurations at runtime and PROPERTIES map inside Configuration.java should be served as the single source of truth.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19427/Failed Tests: 1org.alluxio:alluxio-core-client-hdfs: 1alluxio.hadoop.HadoopConfigurationUtilsTest.mergeHadoopConfiguration
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19434/
Test PASSed.

@apc999 @aaudiber PTAL, maintain the toMap() in Validate and fix the Configuration.init() bug.
Configuration.init() should not only clear PROPERTIES, but also clear SOURCES
 static void init() {
    PROPERTIES.clear();
    SOURCES.clear();
}

Thanks!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19452/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/19454/
Test PASSed.

@aaudiber @apc999  Adding the jira link
[ALLUXIO-3183] Support properties with null values in web ui and cli
https://alluxio.atlassian.net/browse/ALLUXIO-3183


Ctrl+A does not work consistently on Windows 10 in ETW

ctrl + A conflicts with ctrl + 0, and when there are that many bottom bar buttons, the bottom bar action takes priority


Trail edit/creation screen missing user selection option

Looks like we lost the ability to add users to a trail.  Previously created trails show users associated with them in the Trails table, but newly created trails are missing the user checkbox options and come up with empty users column entry.

Mike explained this is as intended, and that user data gets added to a trail automatically as the user contributes to a trail.


Numbers higher than $3FFFFFFF don't work correctly

Try following
n=$3FFFFFFF
print n

n=n+1
print n

I've reproduced this on the emulator. The docs say integers are about +/- 2 billion, i.e. 7fff ffff, so this should be well within range.
Interestingly this is a VERY similar result to the one I got in #50:


Waypoint: Build an Adventure Game

The link in Waypoint: Build an Adventure Game takes you to, what Codeacademy says, is an unsupported curriculum.  http://www.codecademy.com/courses/getting-started-v2/0/1

It may be a small issue of completeness of the address in the link, but this link appears to function as expected, though the only difference is the end of the address.  http://www.codecademy.com/courses/getting-started-v2/0/1?curriculum_id=506324b3a7dffd00020bf661

Just wanted to pass it along in case someone else runs into this or in case the link should be updated (the courses in each link appear to be exactly the same).

I think this is still an issue and should be fixed in the waypoint.

Closing as all codecademy courses will be removed in next iteration of curriculum.
Happy Coding!


Wrong input filed name will not be updated.

JIRA: PDC-1099

Looks good.

Prefer to @lubomir's by extending StrictSerializerMixin, since we already have such common handler.

:+1:


[Bug] Light Value changes wont load / apply until debug mode has been entered once every load

If the light values have been changed via /config command and saved they will reset to default on map loading and only change back if the player enters debug mode at least once so the player would need to do it every time the he loads the map.

Hopefully fixed with https://github.com/Adventurecraft-Awakening/AC-Legacy-Mod/commit/5e7d2e528c0bd8dc7e5cc84669beec9c3e6ebe1f and https://github.com/Adventurecraft-Awakening/AC-Legacy-Mod/commit/51e133495521a47c1d8e48fc16d9053cc8dd7f36


Upgrade deps and peg versions

Pegging a peer depedency like phantomjs is a bad idea.
The whole point of semver is that you can move up in patch and minor versions without compatibility issues.
Are you seeing any particular issues?

No issues, just upgrade-mania I guess and a way to say "perhaps it's time to upgrade deps".
While I like the semver way of versioning and try to follow it myself, I do think updates still should be manually handled because you don't know for sure if a dependency has maintained compatibility regardless of intentions. Also using non-pegged versioning can result in non-identical builds from the same source which I also think can be a potential issue. But opinions vary and that's ok with me.
To conclude, feel free to ignore this PR. :)

Pegging a peerDepedency version is a really bad idea. if someone globally installs this mocha-phantomjs and has 1.9.17 installed it will uninstall that version and go to 1.9.16.
And in general, I disagree with you. the community is good at keeping to semver - we occasionally see issues, but we see them earlier rather than later this way.

Agreed too Nathan. Thanks to you both too.


Investigate authorization/access management for Referral Handler Service to write to Social Care Hub

How will the Social Care Hub authenticate the Referral Handler Service?
How will the Social Care Hub allow Referral Handler Service to access it to save data?

Using Azure AD for authentication token.
Using the patch to write the data.


example/configs: remove "type"

This further cleans up filter types. I left "type" in the old v1 integration
test configs so we still have coverage there that the config doesn't blow up.
Signed-off-by: Matt Klein<EMAIL_ADDRESS>

@envoyproxy/maintainers


Can't define same route for differents methods

Hi, I started to test crow, works fine but when I try to do the following  :

    CROW_ROUTE(app,"/db/<string>")
    .methods("GET"_method)
    ([](string dbName){
       return crow::response(200);
    });

    CROW_ROUTE(app,"/db/<string>")
    .methods("PUT"_method)
    ([](string dbName){
       return crow::response(200);
    });

    CROW_ROUTE(app,"/db/<string>")
    .methods("DELETE"_method)
    ([](string dbName){
       return crow::response(200);
    });

it throw an exception "handler already exists for /db/"
Why I can't use the same route but for differents methods ? Is this normal ?
Thank's

No, it's a bug. I will fix this soon.
Current implementation only allows one handler for a route. If you want a quick workaround, you can put all handlers for different method in one route.
CROW_ROUTE(app,"/db/<string>")
.methods("GET"_method | "PUT"method | "DELETE"_method)
([](const request& req, string dbName) {
  if (req.method == "GET"_method)
    ...
});

seems to not working
In file included from main.cpp:1:
./server.hpp:53:27: error: invalid operands to binary expression ('crow::HTTPMethod' and 'crow::HTTPMethod')
.methods("GET"_method | "PUT"_method | "DELETE"_method)
~~~~~~~~~~~~ ^ ~~~~~~~~~~~~
/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/../include/c++/v1/bitset:1045:1: note: candidate template ignored:
could not match 'bitset<_Size>' against 'crow::HTTPMethod'
operator|(const bitset<_Size>& __x, const bitset<_Size>& __y) _NOEXCEPT

Sorry again, I'm late.
You should write like this:
.methods("GET"_method, "PUT"_method, "DELETE"_method)

Works great thank's !
Just to say, I use crow for a school project, and after long time searching, crow apear to be the best and simplier lib to do a simple web service. Great work !


class Tune

🚀 Feature Request
Implement Tune class for tuning pipelines with Optuna.
Proposal

class Tune(AutoBase):
    pipeline: Pipeline
    target_metric: Metric
    horizon: int
    metric_aggregation: MetricAggregationStatistics = "mean"
    backtest_params: Optional[dict] = None
    experiment_folder: Optional[str] = None
    runner: Optional[AbstractRunner] = None
    storage: Optional[BaseStorage] = None
    metrics: Optional[List[Metric]] = None
    sampler: Optional[BaseSampler] = None
    params_to_tune: Optional[dict] = None  # custom params to tune

    def _init_optuna():
        # init optuna study with TPE sampler by default
        # if sampler is not None, use it
        # if params_to_tune is not None, use it
        # if params_to_tune is None, use Pipeline.params_to_tune

    def objective(trial):
        # objective function for optuna with sampling logic
        # return metric value to minimize
        # use pipeline.set_params to set sampled params
        #  params = {
        #     "model.n_iterations": optuna.distributions.CategoricalDistribution((10, 100, 200)),
        #     "transforms.0.mode": optuna.distributions.CategoricalDistribution(("per-segment", "macro")),
        #   }
        # We could sample params via Trial._suggest(params.key, params.value)

Test cases

add tests cases like for Auto class
check if it works with other samplers
check if custom params_to_tune works

Additional context
Is blocked by #1026 #1023 #1025

I'd like to take this issue.

I have stumbled upon a problem.
In optuna, to optimize a hyperparameter, its range of values must be specified (using .suggest_float(..) and its likes). These ranges are usually set manually and a big challenge in implementing Tune class is to develop a way to propose these ranges automatically.
Is there an inbuilt way to get hyperparameters value range in Etna (maybe there are some descriptions of features are passed which I don't know about)?
If there is not, do you have any algorithm for hyperparameter value range setting on your mind that you think I should implement? I could go browse the web for possible implementation ideas, but I think I should ask for your vision beforehand.
The only algorithm I can think of as of now is a brute-force one proposing ranges for hyperparameters based on their names.

@GooseIt, are you doing this task? Do you have any news, problems do discuss?

@GooseIt, are you doing this task? Do you have any news, problems to discuss?

Yes, I've done my research, now all I have left is to write the code.
I'm quite busy with my studying assignments, though.
I'll do it by March 28th, 23:59 with my current pace. Is this alright, or  do I have to prioritize the task more?

It is fine, I just wanted to make sure that task is in progress.

May be it wasn't mentioned. Definition of default params_to_tune are currently present in automl-2.0 branch.

Is it true that params_to_tune in automl-2.0 currently provide no functional of getting said params for a basic model or a transform beside default {}?
I've seen the Pipeline method params_to_tune, it aggregates results for basis models and transforms well, the question is what these results are.
If what I asked is true, should I implement class Tune with assumption that params_to_tune method for a base model or a transform returns all tunable params, as it is supposed to do?

Why aren't changes in automl-2.0 (namely params_to_tune definitions) brought into master?

Yes, params_to_tune currently are just giving some interface to work with. Probably, some objects will have this empty result {} for a long time and in Tune we should be able to work with it.
You can examine tests for params_to_tune in tests/test_pipeline/test_mixins.py. It should give some idea how the result looks like.
We thought that this track (automl) should be made in a separate branch until it is complete. We forgot about it during merging your pull requests. Probably, we should continue doing this track in a automl-2.0 branch. You can continue doing this task as you do. We will discuss this details during PR.
In Tune we have external params_to_tune parameter. If it is empty, pipeline.params_to_tune() is used. If it is empty too let's check what happens to optimization if we do nothing. After that we will decide are we going to raise exception manually or not.

I've encountered the following problem.
params_to_tune returns params_dict such that params_dict[param_name] = param_distr, where param_distr is an optuna.distribution object.
Current optuna API provides the following options to set up optimization process:

Pick up sampler from optuna.samplers, then for each parameters call trial.suggest_* specifying params ranges from params_to_tune in objective.
The downside of this approach is that code will have to use multiple if clauses to set trial params. As example, for a given param_name with param_distribution we will have to go through all the possible distributions of said param:
if type(param_distribution) == optuna.distributions.UniformDistribution:
    params[param_name] = trial.suggest_uniform(...)
elif type(param_distribution) == optuna.distributions.CategoricalDistribution:
    params[param_name] = trial.suggest_categorical(...)
...

Create custom optuna sampler that encapsulates neccessary logic of sampling from optuna.distribution in it (etna.auto.optuna.ConfigSampler is an example of encapsulating logic of sampling that consists of picking pipelines from a predetermined pool)
The downside of this approach is inability to use many samplers from Optuna (possibility of having a hyperparameter responsible for which sampler from Optuna to use is very attractive).

Which of the approaches do you think is the best one? Or is there a better alternative implementation you can propose?

Ok, we will investigate this problem and suggest some solution.

Hello. I think that first solution is better. Some details for the better implementation:

We can probably define some dict that specifies all cases for this if/else logic. For example, key is distribution class, value is suggest_xxx function with tuple of parameters to extract from distribution object to pass inside suggest_xxx function.
It is better to use isinstance instead of type.

There is another issue.
As stated in poetry.lock file, an outdated verison of optuna is used:
name = "optuna"
version = "2.10.1"

Version 3.0.0 was out for more than half a year, and it introduced changes that impact optuna.distributions logic.
In version 2.10.1 there are many distribution classes: UniformDistribution, LogUniformDistribution, DiscreteUniformDistribution, IntUniformDistribution, IntLogUniformDistribution, CategoricalDistribution.
Version 3.0.0 implemented unified interface for these classes - new optuna.distributions IntDistribution, FloatDistribution, CategoricalDistribution and made the 2.10.1 classes deprecated due to removal in 6.0.0 (though documentation states removal schedule may change).
The question is:
Should I implement Tune in 2.10.1 code (creating a dict of all deprecated optuna.distributions like LogIntUniformDistribution, etc) or should we prioritize moving Etna to a newer optuna version?
I think sticking with outdated version of optuna will cause numerous problems integrating the new version into Etna later (and we want Etna to support newly-added features of optuna, don't we?)
In my perspective, moving Etna to optuna 3.x.y will make implementation of first solution easier, it can be done in two steps:

cast a (maybe deprecated) distribution to 3.0.0 distribution class (IntDistribution, FloatDistribution, CategoricalDistribution)
dict as proposed by you in the previous comment with only 3.0.0 distributions as keys

Let's try to update optuna up to version 3.

Update pyproject.toml: find optuna and set: version="^3.0"
Update poetry.lock: poetry lock.
Check that tests tests/test_auto are working.

If there are some difficult to fix problems with this transfer, let's postpone it and don't do this in this task.

poetry lock gives the following output:
Because no versions of pytorch-forecasting match >0.9.0,<0.9.1 || >0.9.1,<0.9.2 || >0.9.2,<0.10.0
 and pytorch-forecasting (0.9.0) depends on optuna (>=2.3.0,<3.0.0), pytorch-forecasting (>=0.9.0,<0.9.1 || >0.9.1,<0.9.2 || >0.9.2,<0.10.0) requires optuna (>=2.3.0,<3.0.0).
And because pytorch-forecasting (0.9.1) depends on optuna (>=2.3.0,<3.0.0)
 and pytorch-forecasting (0.9.2) depends on optuna (>=2.3.0,<3.0.0), pytorch-forecasting (>=0.9.0,<0.10.0) requires optuna (>=2.3.0,<3.0.0).
So, because etna depends on both pytorch-forecasting (^0.9.0) and optuna (^3.0.0), version solving failed.

It seems to me to be caused by fact that pytorch-forecasting library uses optuna 2.x.y (namely, 2.10.1 in its latest version).
Which course of further action do you advise?

There isn't easy fix for this. Let's for now stick with optuna 2. We will think what to do with upgrading later.

Let's change the signature of def objective:

Remove it from AbstractAuto
Add new signature into Tune

Add parameter pipeline after parameter ts

I think it should solve our problem.

Closed by #1200.


Is it possible to use a transparent back color?

Is it possible to use a transparent back color?

I don't think it's possible atm, but there might exist the possibility of allowing for a transparent background. I'm not sure how DX OpenGL interop would handle that but it might be possible.

Fixed with #71


Small tweaks in the quickstart guide

One thought regarding the chapter dependencies.
Currently, the subchapter "Exporting to PDF" is dependent on the subchapter "Microsoft Word Documents" with the sentence 🛠 In 01-paper.md: replace export: docx with a list: I think it would be nice to rephrase this sentence in a way that does not require the previous knowledge from the Microsoft Word Documents chapter, as people might skip that Microsoft Word chapter.

Export Documents 📜

Microsoft Word Documents
Exporting to PDF

So cool, thank you for these edits!!
What about:
🛠 In 01-paper.md: create an exports list with `docx` and `pdf` formats

Do you want to make that change or similar if you have it open?

Thanks @kolibril13, appreciate the fixes!! Absolutely would love the help on other docs pages as you are going through them, and on calling out with issues any places you get stuck as well!
🚀

I am happy that this is helpful! :)
One more thing:
here the abstract background is grey, but for me, it's not:
https://myst-tools.org/docs/mystjs/quickstart-myst-documents#use-myst-markdown

Yes -- that is a known bug, I took the screenshot on a branch of @myst-theme we are aiming to get in over the next week or so. The very experimental article theme is pushing the way here.
I will see if I can merge these changes in today, but if not I should probably fix the screenshots.

I opened up a bug to track that! #216

aand one more:
The emoji does not render in latex.
Maybe this can be noted somewhere in the quick start that imagemagick is needed, like a nice rephrasing of the error message
"To convert these images, you must install imagemagick or inkscape"

% myst build 01-paper.md

📬 Performing exports:
   01-paper.md -> _build/exports/paper.docx
   01-paper.md -> _build/exports/paper.pdf
📖 Built 01-paper.md in 41 ms.
📖 Built 02-notebook.ipynb in 41 ms.
📖 Built 01-paper.md in 36 ms.
📖 Built 02-notebook.ipynb in 36 ms.
⛔ README.md Cannot convert SVG images, they may not correctly render.
   To convert these images, you must install imagemagick or inkscape
📖 Built README.md in 199 ms.
📚 Built 3 pages for project in 227 ms.
📑 Exported TeX in 2.52 ms, copying to _build/temp/mystlV4LyM/paper.tex
'parts' missing required key: abstract (at 01-paper.md)
'frontmatter' missing required key: short_title (at 01-paper.md)
🖨  Rendering PDF  (volcanica) to _build/temp/mystgBLlMl/paper.pdf
⛔ README.md Cannot convert SVG images, they may not correctly render.
   To convert these images, you must install imagemagick or inkscape
📖 Built README.md in 285 ms.
📚 Built 3 pages for project in 306 ms.
📄 Exported DOCX in 88 ms, copying to _build/exports/paper.docx
📄 Exported PDF  (volcanica) in 6.8 s, copying to _build/exports/paper.pdf

Fixed up in #218.


fix(swiper): autoplay fails  when userScrolling

背景描述
如果是userScrolling(用户只是想上下滑动屏幕,不是左右滑动滑块), autoplay意外被中断
复现: https://didi.github.io/mand-mobile/examples/#/swiper 第一个swiper, 鼠标按住想上滑一下, 松手, 即可复现
主要改动
滑动结束, 恢复自动播放

Codecov Report

Merging #636 into dev will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##              dev     #636      +/-   ##
==========================================
- Coverage   83.48%   83.47%   -0.01%
==========================================
  Files          98       98
  Lines        2773     2772       -1
  Branches      528      527       -1
==========================================
- Hits         2315     2314       -1
  Misses        314      314
  Partials      144      144

Impacted Files
Coverage Δ

components/swiper/index.vue
83.9% <100%> (ø)
:arrow_up:

components/picker/picker-column.vue
84.27% <0%> (-0.1%)
:arrow_down:

Continue to review full report at Codecov.

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


[Request] Add a mute toggle in the encode options

As the title says. I thought I didn't look carefully in the options but it looks like there is no toggle to encode without audio (with the -an flag)

Muting the video doesn't work?

What do you mean? Clicking on the audio icon on the mpv gui and trying encoding after that? Because I see no mute option

Yeah.


Remove existing options of thelounge start and add a generic --config override

Resolves https://github.com/thelounge/lounge/issues/419.
Removes the --public / --private / --host / --bind / --port options for thelounge start in favor of a new slightly-more-intricate-but-much-more-powerful --config / -c option.
Example on how to use this (note that I have successfully tested all examples given in this description):
thelounge start -c public=true \
                -c lockNetwork=true \
                -c defaults.port=6667 \
                -c defaults.tls=false

(Multiline for GitHub readability but it obviously works on a single line)
There are a bunch of different cases I handle, and tests for all situations I could think of.
Some more example of what is possible:
-c https.enable=true    # Supports booleans
-c port=9001    # See below for caveats on integers
-c debug.raw=true    # Supports dot-notation for objects (also comes with array-notation out-of-the-box, but not very useful in our case
-c transports=[websocket,polling]    # Supports arrays
-c "transports=[websocket, polling]"    # Supports spaces assuming option is wrapped in quotes
-c logs.format="DD MMMM YYYY HH:mm:ss"   # Can also quote just the value
-c ldap.searchDN.rootDN="cn=thelounge,ou=system-users"   # Supports values containing `=`
-c ldap.searchDN.filter='(objectClass=person)(memberOf=ou=accounts,dc=example,dc=com)'   # Complex value

I expect we will discover weird edge cases later, but I think this covers maybe 95% of the needs if not more.
One thing this does not support is passing JavaScript code, like functions, for example for webirc. This would require using eval() which is a no-no for me considering this can affect a whole instance. I will argue that if you need to pass complex JavaScript code through the CLI, you should definitely define your own config file! (which is also valid for some complex cases, like LDAP, but it's easier and less risky to support)
Integers are not parsed as such. I have checked and tested almost all usages of integer options in our configuration files (maybe expect identd), and they all work fine. In some cases, we parseInt, but also note that strings are coerced into numbers in almost all cases in JS. For example, "5" * 3 is 15 (as an integer) and "5.1" * 3 is 15.3 (as a float) so all </> comparisons work fine.
If we ever discover an issue with that, I can think of at least 3 ways of handling this, from easy to rather complex (the simplest would be to automatically parseInt if value is integer-looking, though I'm sure there are other pitfalls), but I think we should do this simply right now and fix as we go.
After all, this Just Works™ :)

About the first commit: Define a raw logger to avoid using console.log, use it in extra help for environment variables, and add a test for this
As said in commit message, this has a few benefits:

Respects the "Do not mock what you do not own" principle, instead we mock log.raw when necessary
Lets us not re-assign console.log, which breaks as Mocha uses console.log as well
Save and restore initial log.raw in test hooks (before/after), otherwise this would break Mocha/Chai

This allowed me to write extra tests :)

Note: This PR currently fails on Node v4 but will pass once #1727 gets merged.

Should we ship this in 2.x (minus the removal of existing arguments) to test it and give deprecation notice?

Should we ship this in 2.x (minus the removal of existing arguments) to test it and give deprecation notice?

Good call, I'll update my PR to do so.

I'll make my PR work with Node v4 then.

OMG I am spending my entire weekend on this PR...
@xPaw, managed to almost make the -c option solve the problem you described in #1807...
thelounge -h with a config file saying public: true:

thelounge -h -c public=false with a config file saying public: true:

Buuuuuuuut I have one more issue.
As you can see, running thelounge -h -c public=false says:
2017-12-10 21:07:58 [WARN] Configuration key public was already specified, igoring...

-c public=false gets parsed twice. Not just that, but the second time it's parsed, the memo object contains previous values.
I tried sooo many things and I am not sure what's going on here. The easy solution could be to remove that warning, and just silently do nothing when giving -c public=true -c public=false, but I'd rather not do that. I'll spend 10 more minutes on that but then I give up, it's driving me crazy. If you have any idea of what's going on (I've been looking at commander's code for far too long), I'd love to hear it lol.

Wooooot I finally fixed it.
We were running .parseOptions(process.argv), parsing options a first time, then parse(process.argv), also parsing options. It turns out parseOptions returns argv without the options already parsed, but unfortunately for me it splits the result in 2, arguments and options so I had to re-concatenate them...
With the problematic example above (thelounge -h -c public=false, this is how things look like:
# process.argv
[ '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/node',
  '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/thelounge',
  '-h',
  '-c',
  'public=false' ]

# argvWithoutOptions.args.concat(argvWithoutOptions.unknown)
[ '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/node',
  '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/thelounge',
  '-h' ]

With a more complex example  (thelounge -c public=false -h --home /tmp --foo bar start -P 1234):
# process.argv
[ '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/node',
  '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/thelounge',
  '-c',
  'public=false',
  '-h',
  '--home',
  '/tmp',
  '--foo',
  'bar',
  'start',
  '-P',
  '1234' ]

# argvWithoutOptions.args.concat(argvWithoutOptions.unknown)
[ '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/node',
  '/Users/jeremie/.nvm/versions/node/v6.11.4/bin/thelounge',
  'start',
  '-h',
  '--foo',
  'bar',
  '-P',
  '1234' ]

so we should be fine.
I have wasted soooo much time on this PR. I need to not be in front of a computer now :)
@xPaw, I hope you like how things look now! ❤

Great catch, will fix, thanks!
Jérémie Astori
https://jeremie.astori.fr

FYI, if anyone is interested, I have written the docs around --config in the new docs I'm working on. It's WIP, but can be seen at https://github.com/thelounge/thelounge.github.io/blob/680e9aec389215b7120491f9f38189e07b4ca4f6/_docs/usage.md#configuring-the-lounge (no website styling but GitHub does not support Pages on PRs/branches, and I still haven't configured an alternative way to do so).


Implements database data comparator

Use Story
As an user I want to compare table and database data.
Today the system provides a database schema comparator, but is extremely useful for database managers and software testers compare data.

Example
Database 1 - Table 1

First Field
Second Field

1
2

3
4

Database 2 - Table 1

First Field
Second Field

1
2

5
4

Diff between Database 1 and Database 2

First Field
Second Field

5
4

The feature can be used like the schema comparator.

Use cases

Compare deployment and production database
Check algorithm's results in database
Help software CRUD tests
Help software regression tests
Generate data mutation history (log)
ETC

Sorry about my english (I am from Brazil ><).
Thanks a lot.

That feature would be awesome.
As a software tester, in legacy system I can't mock or test software routines without database, and that's useful for check the result data.
I searched for a good software to do this, but I can't find nothing good and price free, and free software, impossible! :smile:
I think it will be a great feature!
My suggestion for implementation is export data to a file format like JSON and call the Eclipse file comparator. Today I use something similar, exporting the data to JSON files and using WinMerge to see the diff.

+1 for this feature!

Good tables data comparer is implemented in the JetBrains DataGrip:


Worker thread

Spawns a thread on driver init that updates the operating stats of the load balancers, pools, and members.

check

-1, feedback inline.

I don't see any unit tests.


[prometheus-nut-exporter] metrics consistency

This is just adding some env vars and bringing some consistency with metrics port names.
Checklist

[X] Chart version bumped in Chart.yaml according to semver.
[X] Title of the PR starts with chart name (e.g. [home-assistant])
[X] Variables are documented in the README.md (this can be done with using our helm-docs wrapper ./hack/gen-helm-docs.sh stable <chart>)

I'll let @bjw-s review this since he uses it.


Free up space in test data

Changelog
Free up space in test data
Issue ticket number and link
~Closes #xxxx (Replace xxxx with the GitHub issue number)~
Checklist before requesting a review

[x] I have performed a self-review of my code

[x] Conforms to the Contributor's guide

Check the correct box. Does this PR contain:

[ ] Breaking changes
[ ] New functionality
[ ] Major changes
[x] Minor changes
[ ] Documentation
[ ] Bug fixes

[ ] Proposed changes are described in the CHANGELOG.md

[x] CI tests succeed!

We might not need HLCA as well. I used it for tests in labels transfer component but in the new annotation components some PBMC dataset is used

We might not need HLCA as well. I used it for tests in labels transfer component but in the new annotation components some PBMC dataset is used

I did a search, and I think the scarches tests uses HCLA, no?

I did a search, and I think the scarches tests uses HCLA, no?

@VladimirShitov I am going to merge this one when ready because we currently need this fix. But I will keep in mind your suggestions when I look at reducing the size of the test resources in the comming weeks :)


enhance(capture) add support to skip processing a render, full abort.

Adds an abort option to the stop functions to override the default behavior of processing partial renders. When abort is true, the captured frames are fully discarded as if the render never started.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 6 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 25.9%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

modules/core/src/adapters/deck-adapter.js
0
1
0.0%

modules/core/src/capture/video-capture.js
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

modules/core/src/capture/video-capture.js
1
0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
231

Relevant Lines:
776

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 6 (0.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.1%) to 25.9%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

modules/core/src/adapters/deck-adapter.js
0
1
0.0%

modules/core/src/capture/video-capture.js
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

modules/core/src/capture/video-capture.js
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.1%

Covered Lines:
231

Relevant Lines:
776

💛  - Coveralls


SQL errors

I still haven't figured out what's wrong with our hub by going through the logs .... this shows up regularly:
2017-04-29 19:39:39
select uid from pconfig where cat = 'perm_limits' and k = 'view_stream' and ( v & 1 ) > 0 ;
SQLSTATE[42883]: Undefined function: 7 ERROR:  operator does not exist: text & integer
LINE 1: ...cat = 'perm_limits' and k = 'view_stream' and ( v & 1 ) > 0 ...
                                                             ^
HINT:  No operator matches the given name and argument type(s). You might need to add explicit type casts.

This one showed up only once when somebody (unsuccessfully) tried to log in for the first time - we have ldap. He might have used the wrong username, but still shouldn't have caused an error.
2017-04-29 13:35:47
alter table channel add channel_password varchar(255) not null default '' ;
SQLSTATE[42701]: Duplicate column: 7 ERROR:  column "channel_password" of relation "channel" already exists
2017-04-29 13:35:47
alter table channel add channel_salt varchar(255) not null default '' ;
SQLSTATE[42701]: Duplicate column: 7 ERROR:  column "channel_salt" of relation "channel" already exists

The first issue seems to have been fixed with https://github.com/redmatrix/hubzilla/commit/d3fcef43f8b87ba1a78f2bcf7526a6a8c0e5c819. I'll pull and test later.
The second issue is connected to update_r1188 which seems to have failed on our database.


Added from edge swipe functionality

Tested on Chrome ion Windows 7 using Mobile Mode
Tested on Chrome on Android Nougat

Thanks for the PR.
The debugging feature feels like a definite no-no, and the fact that the events (e.g. swiperight) are replace by others (e.g. swipefromleft) is completely out of the question, in particular because it introduces major incompatibilities.
I think that maybe adding the start & end positions (maybe both absolute and relative) and letting the user decide what to do with it is a better idea. In particular, it would be easy to write a small library that relies on the current events to post the swipefrom... events, if the startX/Y warrant it.

Fair enough.
I never used the debug in the end as you can't see the console easily on
the phone.
Swipe from edge hit the use case of a user swiping in from the edge of the
screen, but we found on iOS that if you do that in Safari it switches tabs
anyway, so wasn't much use. On Android our implementation is great for
swiping in a menu with swipefromleft, then a swipe left set on the menu
element let it close nicely.
If it had a start and/or end then the implementor would have to do their
own calculation as to whether it is near the edge, which whilst achievable,
isn't obvious.
Entirely up to you though, it's your repo and we have enjoyed using it :)
On Mon, 20 Feb 2017, 19:01 Marc-André Lafortune<EMAIL_ADDRESS>wrote:

Thanks for the PR.
The debugging feature feels like a definite no-no, and the fact that the
events (e.g. swiperight) are replace by others (e.g. swipefromleft) is
completely out of the question, in particular because it introduces major
incompatibilities.
I think that maybe adding the start & end positions (maybe both absolute
and relative) and letting the user decide what to do with it is a better
idea. In particular, it would be easy to write a small library that relies
on the current events to post the swipefrom... events, if the startX/Y
warrant it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/marcandre/detect_swipe/pull/13#issuecomment-281158445,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATeGMRjjVJgg1jVKig3-etS5QCH-2lAKks5reeMXgaJpZM4MELv8
.


Fix for tensor_db get_aggregated_tensor returns tuple in corner case 1087

Describe the bug
Calling get_aggregated_tensor on a tensor that has already been aggregated returns a tuple of (tensor, {}).
For more information please refer https://github.com/securefederatedai/openfl/issues/1087
Reproduced by  calling get_aggregated_tensor twice from aggregator while running experiment with 2 collaborators.
Before:

After:

My impression was that this scenario: aggregator calling get_aggregated_tensor on a tensor that has already been aggregated should not occur in normal conditions. For e.g. this may happen when there is an undesired behavior due to which aggregator is performing the aggregation twice. In this case I found that the existing functionality of returning a tuple + subsequent exception actually helps catch undesired behavior in the system
Can you please clarify if there are use cases where this is an intended behavior?

@tanwarsh Could you provide a MWE for this bug?

Closing this PR accordingly following https://github.com/securefederatedai/openfl/issues/1087#issuecomment-2478561800.


README: Fix the Sponsors badge

This PR repairs the OpenCollective Sponsor counter.
Update the link on the Backers badge, to the new URI structure, too.

@TomWFox I think sponsor badges this is your métier - what should we do with this PR?

🎉 This change has been released in version 1.2.0


Add a better structure to the application

Description
The application must receive a split structure into the following layers:
- src
   |- index.js
   |- routes
   |- controllers
   |- middlewares
   |- lib
       |- formatters.js

Acceptance Criteria

[ ] The structure follows the necessity

All you want is a new folder structure? Could I do that?

@iagorm , Hello I created a MR addressing this issue (#4).

@iagorm , Hello I created a MR addressing this issue (#4).

@jhidalgoesp I will review this ASAP.


Chromedriver for Chrome 116 issue

Hi,
I upgraded to Chrome 116 and now I am getting this error:
selenium.common.exceptions.SessionNotCreatedException: Message: session not created: This version of ChromeDriver only supports Chrome version 114
Current browser version is 116.0.5845.97 with binary path C:\Program Files\Google\Chrome\Application\chrome.exe
I downloaded the new chromedriver from this url https://googlechromelabs.github.io/chrome-for-testing/ but I still get the same error, has anyone else experienced this?
Thank you,
Paul

I'm getting the same error since the last few days.

I'm getting the same error but found a temporary solution.

Download chromedriver 116. You can verify the version with chromedriver --version which should output ChromeDriver 116.0.5845.96
In your mint constructor,  set use_chromedriver_on_path=True. If this is not set, then your chromedriver will be overwritten with chromedriver 114

I'm getting the same issue.  I've tried setting "use_chromedriver_on_path=True" on both the Browser.py and Signin.py files.  I'm downloading chromedriver.exe (ver 116) to my working directory, but it is still pulling the 114 version down and overwriting the 116 version.

@jraymond4321 , can you share which version of mintapi you're using? Its strange that its still overwriting with use_chromedriver_on_path=True since mintapi should be looking for the chromedrive instead of downloading a new one.
      mint = Mint(
          email=email,
          password=password,
          mfa_method="soft-token",
          mfa_token=mfa_token,
          use_rest_client=True,
          use_chromedriver_on_path=True,
          headless=True,
          wait_for_sync=False,
      )

@CalvinChanCan  I'm running 2.14 also (see below).  I'm not a python expert, so I'm not sure what you mean by constructor.
C:\Python311\Lib\site-packages\mintapi>pip install mintapi
Requirement already satisfied: mintapi in c:\python311\lib\site-packages (2.14)
Thanks.

@jraymond4321, thanks for confirming its mint 2.14. Sorry for the confusion. By constructor, I meant the arguments we're using to instantiate the Mint object. In this case, I wanted to show the example that use_chromedriver_on_path is being set to true when using Mint(...). Can you share your code on how're you're setting use_chromedriver_on_path=True on both the Browser.py and Signin.py files?
mint = Mint(
email=email,
password=password,
mfa_method="soft-token",
mfa_token=mfa_token,
use_rest_client=True,
use_chromedriver_on_path=True, <---
headless=True,
wait_for_sync=False,
)

Hi @CalvinChanCan , thanks for your help.  I made a mistake, it was working.  I had not saved the batch file with the passing the parameter to use the Chromedriver in path.  Once I saved it, it ran fine.


Show the meridian line

Feature request from the CN forum:
"One thing that would be a nice addition would be a unique line showing the Meridian.  The grid works but a unique color line would be nice."

Added in 21.4


Sqs messages deserialization to LocalDateTime does not work

Type: Bug
Component:
"SQS"
Describe the bug
Using spring cloud (spring-cloud-aws-starter-sqs) with version 3.0.2 for my async Sqs listener (annotated with @SqsListener) I face an issue with deserialization of LocalDateTime and setting up proper ObjectMapper for the process.

[sdk-async-response-0-3] ERROR i.a.c.s.l.s.AbstractPollingMessageSource - Error processing message
org.springframework.messaging.converter.MessageConversionException: Could not read JSON: Java 8 date/time type java.time.LocalDateTime not supported by default: add Module "com.fasterxml.jackson.datatype:jackson-datatype-jsr310" to enable handling

How can I configure custom (or even default spring) objectMapper for deserialization?
Sample
My listener method:
@Component
class Example{
@SqsListener(value = "${queue}")
    public CompletableFuture<Void> populateNewsActivity(List<Msg> messages) {
    ....
   }
}

I tried by setting up configuration:
@Bean
SqsListenerConfigurer configurer(ObjectMapper objectMapper) {
    return registrar -> registrar.setObjectMapper(objectMapper);
}

And by providing custom MappingJackson2MessageConverter like:
@Bean
    MappingJackson2MessageConverter mappingJackson2MessageConverter(ObjectMapper objectMapper) {
        MappingJackson2MessageConverter converter = new MappingJackson2MessageConverter();
        converter.setObjectMapper(objectMapper);
        converter.setPrettyPrint(true);
        return converter;
 }

In both cases I got the same error (my configuration was omitted).

Hi @atakume, this is a known issue which is in the process of being addressed. See #721.  In the meantime that ticket contains a couple of suggested workarounds (read from the top), including one I contributed here.
Regards, Neil.

Hi @neiljbrown, thanks I'll try this way.
I missed this one while searching 🙈

I'm closing this one as it's duplicate of known issue


Are Client Id and Application Id the same value or different

In one place you reference Cleint ID but you never say what this is.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 6e349050-7bbf-e076-2b4d-80be1b19611c
Version Independent ID: e19f0325-71bb-9cca-c6df-56675ebff3b0
Content: Integrate Key Vault with SQL Server on Windows VMs in Azure (Resource Manager) - Azure SQL Server VM
Content Source: articles/virtual-machines/windows/sql/virtual-machines-windows-ps-sql-keyvault.md
Service: virtual-machines-sql
GitHub Login: @MashaMSFT
Microsoft Alias: mathoma

Hi @mrdenny , thanks for contacting Microsoft! You are correct, ApplicationID and ClientID are interchangeable:

I will update the article for more clarity.
Thanks for bringing this to our attention and improving our documentation!
Masha from the SQL Docs team

Thanks @MashaMSFT for the confirmation and for the doc update task.
@mrdenny Thanks for bringing this to our attention. We will now close this issue. If there are further questions regarding this matter, please tag me in a comment. I will reopen it and we will gladly continue the discussion.


MINOR: Improve Log layer segment iteration logic and few other areas

In Log.collectAbortedTransactions() I've restored a previously used logic, such that it would handle the case where the starting segment could be null. This was the case previously, but the PR https://github.com/apache/kafka/pull/10401 accidentally changed the behavior causing the code to assume that the starting segment won't be null.
In Log.rebuildProducerState() I've removed usage of the allSegments local variable. The logic looks a bit simpler after I removed it.
I've introduced a new LogSegments.higherSegments() API. This is now used to make the logic a bit more readable in Log. collectAbortedTransactions() and Log.deletableSegments() APIs.
I've removed the unnecessary use of java.lang.Long in LogSegments class' segments map definition.
I've converted a few LogSegments API from public to private, as they need not be public.

Tests:
Relying on existing unit tests.

cc @junrao

LGTM
Thanks @kowshik

Thanks for the review @junrao! I have addressed your comments in 4b836034415c3d5f6b84384ef9be1e75b66edc4b.

@junrao Thanks for the review and for catching this issue! I've fixed the build now. The issue was that the code in Log.scala used the Iterator.nextOption()API that was introduced only in scala v2.13, so the build failed for scala v2.12.
I have now also added unit tests for the newly introduced LogSegments.higherSegments() API.
The PR is ready for review again.


Change location of the Stratis Breeze Wallet Files

Hi,
On the C:\ of my server i have a shortage of storage (only 60GB). Is it possible to give the user the option to select an other drive, so I won't risk flooding the drive?
I'm using Windows 10 x64 - Creaters Update Fall.
With kind regards,
Daan Damhuis

What do you mean by this? I downloaded the win64bit zip file. I didn't have to install the wallet. Just move the files to your other drive and start the breeze-ui.exe again solved that part for me. Switched to 3 drives after that without any problems.

I think you mean storage of the node data.
It's possible but not so easy, you need to go to the execution file and add a data location.

@dangershony Yes, that is correct. It's not something that is important for me now, but when it hits the mainnet, I would be a nice option to have.

@Daandamhuis yeah I agree, this should be possible in the future and was already on our future feature list.


Add commandhandler command syntax change

Now the syntax is

dolittle add commandhandler {new_commandhandler_name}

which leads to a selction of which command to handle
I feel it should be

dolittle add commanhandler {existing_command}

which might ask what to call the command, I think it should just call it {existing_command}_handler
Even better would be to ommit "add" and add a new "handle" command:

dolittle handle command {existing_command}

The add is for consistency throughout.
I'm not sure I follow the first part. Are you talking about one command handler file per command? If so, that's not what we're seeing in all projects. One tends to group handlers for processes - or even aggregate roots.


feat: Add converter to support outbound calls to OpenAI servers that want tool not function

Please describe the purpose of this pull request.
The function naming (function / function_call / function role) is deprecated in OpenAI API and partially (?) deprecated in Azure OpenAI. OpenAI and Azure both still support the old naming methodology, but newer "OpenAI API-compatible" services that also allow function calling (eg Anyscale) don't support the old naming scheme and want tool only.
This PR adds a converter function that goes from the old format to the new one. This is done on-the-fly, ie we still store messages in the functions style but we modify it on the way out to the OpenAI API server.
@sarahwooders eventually we probably want to migrate the internal representations to the tool_call format, but I'm not sure how important it is for now, since unless we want to have parallel function calling support nothing really changes and these light patches work fine (AFAIK).
How to test
Try using MemGPT with a non-OpenAI OpenAI-compatible endpoint that has function calling, but only the tool version of function calling (eg Anyscale).
Have you tested this PR?
Yes, seems to work OK with Anyscale.

Force push rebased on 0.2.11

Deprecated by 0.3.0


Have the option to set the scanning speed of the artifact same as scan material

Maybe is just me but I notice sometime when you or the scanner select things too fast in game, the game will ignore the input thus cause scanner skip to record the artifact. Even though we have the 100ms option but sometime the game still fail to register the input. However material scanning doesn't face this problem due to slower select speed. Which is why I propose to add an option to have the same scanning speed as the material scanning

Thanks for your input. I'll look into adding options for longer delays.

The 'slow' delay in the newest release should be long enough for most computers that are not as powerful as other computers. I have a pretty strong one myself.


Uncaught TypeError: Cannot set property 'value' of null

I get this error when I click on an element.
Uncaught TypeError: Cannot set property 'value' of null
    at ft (vue-orgchart.min.js?f5c3:1)
    at HTMLDivElement.eval (vue-orgchart.min.js?f5c3:1)

Seems the problem is that
document.getElementById("selected-node");

Always returns null

you should add this in your

    选中树节点
    <input type="text" id="selected-node" class="selected-node-group">

@spiritree I'm still not able to update the org-chart. When I change chart-data it is not updated, how can I trigger the update?

Please describe more or give a demo.

<template>

        <vo-edit
        :data="chartData"
        :direction="direction"
        :chartClass="chartClass"
        :nodeContent="nodeContent"
        >
      </vo-edit>

        <input type="text" id="selected-node" class="selected-node-group">

  </template>

When I update chartData the chart doesn't reload with new data provided

@robertsLando
#10 Do you mean that?

@spiritree Like you can see in my example the only way to trigger an update is to hide/show the org chart. Yes my problem is like the one of #10 but I was looking for a better solution, I think the component itself shoul watch for data changes and update itself

I have fixed the problem in #10
19cc16bfe96f4ab29c5d28ac9c478b7f24dc6f2e


rascsi-web crashes when a device is attached to ID 7

Info

Which version of Pi are you using: any
Which github revision of software: develop 62f6c8
Which board version: any
Which computer is the RaSCSI connected to: any

Describe the issue
If you use rascsi or rasctl to attach a device to ID 7, rascsi-web will crash with "RASCSIWEB[5326]: ValueError: list.remove(x): x not in list" when loading index.html

Fixing in https://github.com/akuker/RASCSI/pull/218


Kivy UI freezes on spend attempt

Describe the bug
Attempting to spend TBTC from the send screen results in the UI freezing up.
[Theory]:  Kivy doesn't seem to be playing well with asyncio, even with the async_gui lib.
To Reproduce
Steps to reproduce the behavior:

Go to the send screen
Enter a valid TBTC address and spend amount
Tap the "SEND TX" button
UI freezes up

Expected behavior
Should sign/broadcast TX, then show "Transaction sent!" message dialog with fee and TXID.
Desktop (please complete the following information):

OS: Ubuntu 18.04 bionic
Interface: Kivy UI
Version: current tip of master branch 743158175f8656c42e926bfd324f41d137f4b4e9

Additional context
Python 3.6.5

Fixed with new async approach: https://github.com/metamarcdw/nowallet/commit/c35f33a32be247c52962afb817dfc03f1be7d2ab


Clean up old proto client and regen new proto go clients

Description
Clean up old proto client and regen new proto go clients

Clean up old proto client since we moved pkg/proto to dapr/proto
Regen proto with new dapr.proto.* package

Note: service invocation e2e test can be failed because of the wrong proto client. but it will work once this pr is merged.

Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #[issue number]
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

Found the available test cluster - dapr-aks-e2e-01. Please wait until test is done.

FYI - service invocation e2e test will be failed :)


Dependabot, all in one

#169, #168, #164, #161, #153, #152, #151, #142

Also #174

All tested locally


crypto: Add runtime checks for SHA hardware

@edtubbs , this is marked WIP, let me know (if github doesn't) on when it's no longer WIP And you want me to look at this; since I'd like to follow it and see when it's done.

@edtubbs , this is marked WIP, let me know (if github doesn't) on when it's no longer WIP And you want me to look at this; since I'd like to follow it and see when it's done.

@michilumin this one's ready too, and merge after https://github.com/dogecoin/dogecoin/pull/3085


@WIP refactor java image to work with concreate

Example for new concreate tool.

A minor update in that I defined a xpaas.java.feature module, which aggregates all the modules required for images requiring a Java runtime.  This allows other images not based on the openjdk18-openshift image to include the functionality without having to specify the individual component modules (i.e. they need to simply include xpaas.java.feature).


Support multiple dataframe input to CE construct

Thank you for your contribution to the Graph Data Science Client project.

Before submitting this PR, please read Contributing to the Neo4j Ecosystem.
Make sure:

[x] You signed the Neo4j CLA (Contributor License Agreement) so that we are allowed to ship your code in our library
[x] Your contribution is covered by tests

The test failure is an unrelated test that is also failing on main, which @DarthMax is working to resolve.

Rebased to get the fix for the failing test


RC (Release Candidate) Generation process by Decoupling Release Notes

Is your feature request related to a problem? Please describe
For today's release, having component release notes is an entry criteria, meaning the release notes must be merged into the release branch before generating the release candidate (RC). In past releases, this requirement was not strictly followed, as missing release notes could be a blocker for RC generation, preventing teams to test the RC. As a result, RC builds have been started even when some components were missing the release notes. This adds extra work for the release manager, who must follow up on missing release notes, and puts in situation where the locked RC commit ends up missing these notes.
Automations, such as the release notes check workflow, may fail because the input manifest includes a component commit that lacks release notes, leading to manual interventions like this PR which has the consolidated release notes.
Finalizing the RC before all entry criteria are met is not a good practice, but blocking the RC build until all plugin teams provide release notes is also inefficient, as it delays plugins that are ready for release.
Describe the solution you'd like

Plugins should develop the habit of following to the entry criteria and adding the release notes.

Maintain a new branch called release-notes where all plugins must add their release notes. This commit will not, and need not, be part of the RC, as it is neither a feature nor a bug fix. Moving forward, automations can use the release-notes branch to check version-specific release notes. The RC can be generated and finalized only after meeting the entry criteria of having the release notes, and having a separate release-notes branch ensures that the RC doesn't need to be rebuilt, since the branches are now independent.

For example, if we start the build and later declare it as the RC, which is often done for patch releases, the RC need not be rebuilt just because the release notes were initially missing (adding release notes creates a new commit). With the release notes now added to the release-notes branch, the RC can be finalized, as the entry criteria of having release notes is met, without introducing a new commit to the release branch, avoiding unnecessary rebuilds.

Describe alternatives you've considered
No response
Additional context
No response

Adding @dblock @getsaurabh02 @peterzhuamazon @gaiksaya @rishabh6788 to provide some inputs.

Adding @dblock @getsaurabh02 @peterzhuamazon @gaiksaya @rishabh6788 to provide some inputs.

The issue I see is if user use the bundle manifest to get the commit, those commit will not contain the release notes.
I dont think we need to create a new branch to do that which will add more confusion.

Adding @dblock @getsaurabh02 @peterzhuamazon @gaiksaya @rishabh6788 to provide some inputs.

The issue I see is if user use the bundle manifest to get the commit, those commit will not contain the release notes. I dont think we need to create a new branch to do that which will add more confusion.

Thanks.

Consider using major.minor branch for consolidating the release notes rather than a fix commit used in manifest file.
Open to more suggestions.


Switch to studio.datachain.ai

check this PR https://github.com/iterative/vscode-dvc/pull/5626/files please - seems one place is missing?


PPO Finetuning Example

Description
This is an example of TRL PPO, a kind of RLHF, where uses dataset HuggingFaceH4/helpful_instructions that on our whitelist.
1. Why the change?

Enable PPO on Intel GPU with IPEX LLM.
Add class AutoModelForCausalLMWithValueHead from trl to IPEX-LLM
fix readme of lisa finetuning

3. Summary of the change
PPO Finetuning Example
4. How to test?

[ ] N/A
[ ] Unit test
[x] Application test
[x] Document test
[ ] ...

5. New dependencies

[x] New Python dependencies
- trl

@Jasonzzt From the log, it is found that PPO also applies PEFT LoRA.
Therefore, like QLoRA, rather than from_pretrained a peft model with lora config, we should first load the base model, and then use get_peft_model, prepare_model_for_kbit_training etc. methods  in qlora.py to create a peft model. Such a model is built on top of layers with supported operators like here.


Updated value to use for the dex storage

Updated the README file as per the information provided in the comment of this issue: https://github.com/madfish-solutions/quipuswap-sdk/issues/10

Thanks!


test: request trial from

Proposed changes (including videos or screenshots)

Implement testing in RequestTrialForm

Issue(s)

Further comments

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: PedroRorato:x: ujorgeleiteYou have signed the CLA already but the status is still pending? Let us recheck it.


Version available via iobroker.admin seriously outdated

Hi!
First off, thanks a lot for this adapter!
The version I get via iobroker admin sadly is still 0.3.5, which in some cases never reconnects for me (for example if the instance was started while the receiver was not connected to the network).
In the repository here i find a much newer version, and the commits indicate a lot of work has been done mostly on reconnecting etc.
Could an update be pushed, please?
Thanks! :-)

Hi,
I have published version 0.3.12.  I think this Version will be available in ioBroker over night.
Greetings.

Hi!
Thanks, however, it seems the iobroker server did something strange...
Looking at:
http://download.iobroker.net/sources-dist.json
I find:
  "yamaha": {
    "meta": "https://raw.githubusercontent.com/soef/ioBroker.yamaha/master/io-package.json",
    "icon": "yamaha.png",
    "version": "0.3.5",
    "name": "yamaha",
    "news": {
      "0.3.12": {
        "en": "Version incr. for npm",
        "de": "Version fr npm erhÃ¶ht"
      },

i.e. the "news" is updated, but the version is still at "0.3.5"....
Greetings,
Oliver

Hi, I'm using the Onlinesource
https://raw.githubusercontent.com/ioBroker/ioBroker.js-controller/master/conf/sources-dist.json
Otherwise you can manually update the Adapter by this commands:
Change to your ioBroker root Directory and
npm install iobroker.yamaha
iobroker upload yamaha
Now you have to restart the yamaha adapter.

Hi! Thanks, with the online-source it works fine!
Still a bit strange that the default configuration (and thus the majority of users) will not see the new version.

Do you want to update stable repo?!
Stable Repo still has v 0.3.5. When still the most stable version, everything great, if not please update in stable-repo.
(see https://github.com/ioBroker/ioBroker.repositories)
Thanx for checking
Ingo F

Ok, i will send a pullrequest


Still working?

Hey is this code alive?

This code is working afaik, but I've migrated my own bots to very different infrastructure (C# and using Discord.Net).  Since this was an example for Echo Llama (https://www.cattail.nu/midjourney/echo-llama/index.html) I've left this version up as-is.


Update Storage to azure-core preview 3

Closing in favor of https://github.com/Azure/azure-sdk-for-java/pull/4489

We discussed this in our storage sync today and this can be merged now with follow up issues opened for the two follow ups:

Figure out something for the URL encoding in the files client
File quotas is in bytes so we need to measure with int64s instead of int32s