Project references are not converted to Nuget Dependencies

Description
Project PackageB with a dependency on the project PackageA where both projects have a paket.template and the packages use paket pack with the --include-referenced-projects flag should have a Nuget dependency on PackageA but does not.

Paket automatically replaces inter-project dependencies with NuGet dependencies if the dependency has it's own paket.template. In addition to this the parameter --include-referenced-projects instructs Paket to add project output to the package for inter-project dependencies that don't have a paket.template file.

Repro steps
Here is the raw paket command:
C:\Projects\demo\.paket\paket.exe pack output "../../artifacts"  --build-config  Release --template  C:\Projects\demo\src\PaketDemo.PackageB\paket.template --symbols --minimum-from-lock-file --verbose --include-referenced-projects

Here is a project which reproduces the issue:
https://github.com/Lavinski/paket-demo
Expected behavior
ProjectB should have a dependency on ProjectA
Actual behavior
Project references are not reflected in the resulting package.
Known workarounds
None

I was able to figure out why I was encountering this issue.
In my case, the working directory I used when calling paket pack was the project directory. Changing the working directory to the parent solution directory resulted in the expected behaviour.
I'm not sure if the working directory is meant to impact this behaviour. However, if it is it would be worth showing a warning or error in this case. In the meantime, I've created a pull request for the paket documentation to help clarify it's behaviour.
https://github.com/fsprojects/Paket/pull/3102


Jbaker/UI fixes

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Resolves: #239, #242
Summary:
Fix two UI issues:

The price slot content is now the right color on a selected date cell when not hovering over it
The divider between months when viewing a range date picker in desktop is now correctly centered

Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Change dataflow Job log from info to debug

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @chamikaramj
CC: @aaltay

LGTM

Retest this please

cc: @jasonkuster Do you know why Jenkins didn't trigger here ?

Coverage decreased (-0.2%) to 70.033% when pulling e232ddecb2bc7f8a5afa129fcd8d9763ed671b59 on vikkyrk:py_log into bebee2a7284ca3c463ce69dee217e4dfa998be57 on apache:master.

Coverage decreased (-0.2%) to 70.033% when pulling e232ddecb2bc7f8a5afa129fcd8d9763ed671b59 on vikkyrk:py_log into bebee2a7284ca3c463ce69dee217e4dfa998be57 on apache:master.

Jenkins had a hiccup over the weekend as I understand it; watching new PRs to see how they behave.


Rework Mikrotik device tracker to use new hub

Dependent on RP #25576 (New Mikrotik Hub)
Breaking Change:
Requires configuration.yaml change. Move from device_tracker into mikrotik hub component.
Description:
Reworked Mikrotik device tracker component to use the new Mikrotik hub.
Example entry for configuration.yaml (if applicable):
mikrotik:
  - host: 192.168.881.
    username: !secret router_user
    password: !secret router_pass
    ssl: true
    sensors:
      - sysinfo
      - cpu
      - memory
      - disk

  - host: <IP_ADDRESS>
    username: !secret capsman_user
    password: !secret capsman_pass
    ssl: true
    track_devices: false
    sensors:
      - sysinfo
      - cpu
      - memory
      - disk
      - upload_speed
      - download_speed
    binary_sensors:
      - internet
      - netwatch

Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

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

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

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

[x] The manifest file has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[x] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[x] Untested files have been added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Closing this PR and merging device tracker into RP #25576 (New Mikrotik Hub)


template7Pages in index.html

This is a (multiple allowed):

[ ] feature-discussion (RFC)

Framework7 Version: 1.6.0

What you did
template7pages feature not work in index.html. in this issue was told that ".router.refreshPage" is a way to slove this problem. but there is a problem. this way cuase Repeat somde code. is there another way for slove this problem? or is possible run main page as dynamic page?

@nolimits4web
I have the same problem here. I hope you can fix that

Refer to #1449

@nolimits4web
i test this soulation already. but when i run some code in pageinit event for example:
myApp.alert("message");
the code run again. It would be great if template7pages behavior like other pages.


[Bug]: Image style in exported PDF differ from dev mode

Describe the bug
Markdown:
# 毕设进度

支持附加 Windows 进程

![attach process](/assets/bedbg-index-process.png)

---

Start in dev mode:

Export as PDF:

To Reproduce
Steps to reproduce the behavior:

Package.json

{
  "private": true,
  "scripts": {
    "build": "slidev build",
    "dev": "slidev --open",
    "export": "slidev export"
  },
  "dependencies": {
    "@slidev/cli": "^0.29.2",
    "@slidev/theme-default": "*",
    "@slidev/theme-seriph": "*",
    "slidev-theme-vuetiful": "^0.1.0"
  },
  "devDependencies": {
    "playwright-chromium": "^1.20.1"
  }
}

Install using pnpm
2. Start slides using pnpm dev
3. Export slides usint pnpm export
Desktop (please complete the following information):

OS: Windows 10 21H1
Browser: Microsoft Edge 100.0.1185.36
Slidev version:  @slidev/cli: 0.29.2

Same as https://github.com/slidevjs/slidev/issues/333
See my response in the post.
Solutions are:

make sure that the image does not overflow the slide
or use a absolutely positionned image
or add a parent element with display that is flex or grid


Illegal reflective access by com.google.inject.internal.cglib.core.$ReflectUtils$2

On starting the extension the following warning is issued
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.google.inject.internal.cglib.core.$ReflectUtils$2 (file:/home/user/.vscode-server/extensions/typefox.yang-vscode-2.0.5/server/lib/guice-3.0.jar) to method java.lang.ClassLoader.defineClass(java.lang.String,byte[],int,int,java.security.ProtectionDomain)
WARNING: Please consider reporting this to the maintainers of com.google.inject.internal.cglib.core.$ReflectUtils$2
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Version: 1.51.1
Commit: e5a624b788d92b8d34d1392e4c4d9789406efe8f
Date: 2020-11-11T01:11:34.018Z (2 wks ago)
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 19.6.0

Possibly related to https://github.com/google/guice/issues/1133


Should add API for take_group_member/2?

Hi,
I using pooler to manage my db clusters(include MySQL cluster and Mongo cluster ...) via group feature.
I noticed that, take_group_member/1 API's doc:
%% @doc Take a member from a randomly selected member of the group
%% `GroupName'. Returns `MemberPid' or `error_no_members'.  If no
%% members are available in the randomly chosen pool, all other pools
%% in the group are tried in order.

That is, if no members are available in all pools,  it will return error_no_members.
So, should wait for a member to become available from chosen pool?
Thanks.

How would you wait for a member to become available in any of the gen_servers available in the pool?
I've been thinking of this as well but I haven't found a reasonable approach other than having a retry on top of the call with current setup.
take_member/2 essentially waits until the gen_server responds back with a worker, given the timeout. But in the case of take_group_member/1 you're not communicating directly with a gen_server rather asking pg2 about what gen_servers are there, then asking each one for worker with timeout 0. If you would be passing in a customized timeout for each of those calls, it wouldn't make sense to have a pool of gen_servers.
Unless there is more information about the gen_servers which is combined with pg2 information, there isn't really a good way of making a call to wait.  What sort of information could one leverage of the gen_server without breaking the isolation of the process state? Doesn't really need to be super accurate but something that gives an order to which gen_server to poke first who has the best chance of having available members. a simple LRU perhaps, that would be possible to maintain without poking the gen_server states? (i'm just throwing random ideas here)
If one addresses this I guess the walking of gen_servers in
            {PoolPid, Rest} = extract_nth(Idx, Pools),
            take_first_pool([PoolPid | Rest])

would be improved as well. I don't have a urgent use of this but I would definitely be interested in discussing a solution and implementing it.


modeless: {} not a valid mode config

Get the following error when trying to use the modeless approach:
events.js:141
      throw er; // Unhandled 'error' event
            ^
ArgumentError: SVGSpriter.compile: "{}" is not a valid mode configuration

Theoretically should work as listed in https://github.com/jkphl/svg-sprite#d-no-sprite-at-all

Thanks for pointing this out, @cshold, and sorry for not responding earlier. This doesn't seem to be a problem specific to the Gulp wrapper, so I filed a new issue over at svg-sprite.


Multi import of “slowly absorbed meal”

slowly absorbed meal carbohydrates somehow sometimes got imported multiple times. I could not figure out so far when and why it happens.
See pictures:
See pictures:

oh wow ... that shouldn't happen. Which version of the bridge are you running on? There was a bug in versions 0.7.1 to 0.7.4 where it could happen that events were imported multiple times. Never seen anything like this before though. Seems like in each iteration, it imports all of the past events again.
Usually, the loop is designed to only load events between the last known event (+1s) and the current time which should prevent processing events multiple times.

Did the duplication of events stop after the 0.6 correction bolus? Because that would probably mean that the slowly absorbed meal events were for some reason not considered as standard boli and therefore the algorithm always started the search for events after the last "normal" bolus or carb correction.

Looks like it’s a problem with the webspace / Script / something else. all carbs got multiplied

I am using a forg of your latest master CGM-Repo. Problem still occurs, this morning only for the second Bolus:

nightscout acts unresponsive.
I have switched the hosting service from heroku to railway. but for the first 2 days there has been no issue… will investigate

in the database there are multible entries like:
_id
6358ccd89ce7b1cef78fea6f
eventType
"Meal Bolus"
insulin
0.6
carbs
10
app
"diasend"
date
1666761660000
device
"KidsAP Pump (TCM00092FB_2022410536)"
created_at
"2022-10-26T05:59:52.309Z"
utcOffset
120

_id
6358ce439ce7b1cef79065d2
eventType
"Meal Bolus"
insulin
0.6
carbs
10
app
"diasend"
date
1666761660000
device
"KidsAP Pump (TCM00092FB_2022410536)"
created_at
"2022-10-26T06:05:55.806Z"
utcOffset
120

This morning I cleared all treatments in mongo db, since nightscout website was not able to reload. Afterwards I redeployed and everything was running smooth. A meal bolus worked fine, but then another "slowly absorbed meal" Bolus was added and it started again.
Eyery 4 minutes 2-3 treatment entries are pushed ito the database. So the multible entries are growing eveytime the diasend script ist checking for new values and treatments. As already pointed out before, this behavior started since I moved to the railway stack.

I've also seen this issue yesterday with a "normal" bolus - not really sure what's going on there. Will take a deeper look in the upcoming days. But for me, this happened in heroku so I don't think it's an exclusive Railway thing. But it definitely appears to only happen under certain conditions. Sorry for the troubles there. For now, you might also be able to downgrade to an older version, like e.g. 0.7.0. In that version, some events may go missing because the loops for CGM and other events like boli or carbs are not separate yet but likely this also prevents them from being imported multiple times.

My two cents (but I would not call myself a developer at all):
If I compare the datasets from diasend api for type: carb ("slowly absorbed meal) it has a created at timestamp that is converted into an epoch ts as attribute in nightscout / mongo "date" field. But there is also the "created at" field in nightscout / mongodb which gets a slighty different value every time (I guess) the importer sript runs again.
Dublicated dataset:
_id
6363b3f09ce7b1cef73fee26
eventType
"Carb Correction"
carbs
3
app
"diasend"
date
1667478400000
device
"KidsAP Pump (TCM00092FB_2022410536)"
created_at
"2022-11-03T12:28:32.236Z"
utcOffset
60
_id
6363b4e39ce7b1cef7403f56

ObjectId
eventType
Carb Correction

String
carbs
3

Int32
app
diasend

String
date
1667478400000

Double
device
KidsAP Pump (TCM00092FB_2022410536)

String
created_at
2022-11-03T12:32:35.260Z

String
utcOffset
60

Wouldnt it be better to use the same TS from diasend for both "created at" and "date" in mongo db? So duplikates would be overriden automatically ?

seems like this is happening in particular for snacks and slowly absorbed meal entries.

Wouldnt it be better to use the same TS from diasend for both "created at" and "date" in mongo db? So duplikates would be overriden automatically ?

As far as I understood nightscout, it never overrides entries but always creates new ones. If you want to override an entry, you need to manually delete and recreate it.

The multiple import of events should be solved with version 0.7.7.

Ran this for a few days now for testing purposes and couldn't reproduce the multi-import issue anymore. Closing this issue. Feel free to reopen if you still encounter issues.


M05-LAB13: Instructions deviate from UI

Contact Details
No response
What happened?
Exercise : 01
Task: 02
Step: 04
Description of issue:
Repro steps:

When getting to this step the "Add Access Policy" is now '+ Create'
The select none option is not available, user must select permissions
The next section then allows user to finish as you would expect need the instructions prior to the 'reviiew and Create' then 'create' steps.

Thanks
Lab
Lab 13 Integrate Azure Key Vault with Azure DevOps
Relevant screenshots
paste here 😉
Do you want to help us? 👏

[ ] Create a Pull Request by following this project's Contribution Guide

this PR will renew lab #303

3 weeks after, the bug is still present

Lot of labs will be renewed/published in December. If you cannot wait take a look at the opened PRs , #303 for this one.


Selection of visible segments

[x] multiple segments (not necessarily preceding)

Already implemtented in the initial graph editor


feat(semantic-release): setting up semantic release

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[🐛] Error when generating hints shows incorrect config option (showChallengeHints instead of showHints)

:bug: Bug report
Description

When attempting to generate a zip file with hints based on a juice shop instance that doesn't have hints enabled (for example one running in CTF mode) the error message displayed shows the wrong configuration value that needs updated.
See the screenshot below for further information:

The message that showChallengeHints: true in its config should be showHints: true as this value was updated in release 10.0.0. to be challenges.showHints.
Steps to reproduce

Download the latest juice-shop docker container
Run the following command to run the container in ctf mode:

docker run -d -e "NODE_ENV=ctf" -p 3000:3000 bkimminich/juice-shop

Run the juice-shop-ctf CLI
For the Juice Shop URL add the following:

http://localhost:3000

Note the error message displayed

It looks like there are only two references that need updated:
Here
And here
I'm happy to fire up a pull request if you'd like!

Thank you! I've fixed this.


jdk_svc_sanity and jdk_jdi has 32 duplicate testcases, which both include jdk_jdi_sanity

Describe the bug
jdk_svc_sanity and jdk_jdi has 32 duplicate testcases, which both include jdk_jdi_sanity
jdk_svc_sanity:

jdk_jdi:

jdk_svc_sanity include jdk_jdi_sanity:

jdk_jdi_sanity:

jdk_jdi:

duplicate testcases:
jdk_jdi_sanity.txt
Expected behavior
The sanity.openjdk and extended.openjdk test target has no any duplicate testcases.

Mine solution: delete jdk_jdi_sanity in jdk_svc_sanity:
diff --git a/openjdk/playlist.xml b/openjdk/playlist.xml
index 9dd53626..5e71eac5 100644
--- a/openjdk/playlist.xml
+++ b/openjdk/playlist.xml
@@ -1442,7 +1442,11 @@
        -exclude:$(Q)$(TEST_RESROOT)$(D)$(PROBLEM_LIST_FILE)$(Q) \
        ${FEATURE_PROBLEM_LIST_FILE} \
        ${VENDOR_PROBLEM_LIST_FILE} \
-       $(Q)$(JTREG_JDK_TEST_DIR):jdk_svc_sanity$(Q); \
+       $(Q)$(JTREG_JDK_TEST_DIR):jdk_management_sanity$(Q) \
+       $(Q)$(JTREG_JDK_TEST_DIR):jdk_instrument_sanity(Q) \
+       $(Q)$(JTREG_JDK_TEST_DIR):jdk_jmx_sanity(Q) \
+       $(Q)$(JTREG_JDK_TEST_DIR):jdk_jfr_sanity(Q) \
+       $(Q)$(JTREG_JDK_TEST_DIR):svc_tools_sanity(Q); \
        $(TEST_STATUS)</command>
                <versions>
                        <version>11+</version>

This approach seems reasonable.

jdk_jfr_sanity( also a subset of jdk_svc_sanity) and jdk_jfr have the exactly same issue. We may need to open an issue for that too.
@sendaoYan Is there a way to do a minus jdk_jdi_sanity from jdk_jdi instead of minus jdk_jdi_sanity from jdk_svc_sanity? jdk_svc_sanity is part of sanity level. The solution of https://github.com/adoptium/aqa-tests/issues/3993#issuecomment-1259419584 will move jdk_jdi_sanity from sanity to extended. If the solution can keep the tests in same level that would be great.

jdk_jfr_sanity( also a subset of jdk_svc_sanity) and jdk_jfr have the exactly same issue. We may need to open an issue for that too.
@sendaoYan Is there a way to do a minus jdk_jdi_sanity from jdk_jdi instead of minus jdk_jdi_sanity from jdk_svc_sanity? jdk_svc_sanity is part of sanity level. The solution of #3993 (comment) will move jdk_jdi_sanity from sanity to extended. If the solution can keep the tests in same level that would be great.

I have no idea how to minus jdk_jdi_sanity from jdk_jdi.
The solution of #3993 (comment) is delete jdk_jdi_sanity from sanity. I think extended.openjdk may include some of the sanity test targets(jdk_jdi include jdk_jdi_sanity).


fix: prefix strategy with i18n.pages

Description
Following the migration to v4.x, the locale prefix with the i18n.pages + prefix strategy has ceased to function. This PR addresses the issue by incorporating the locale in cases where the "prefix" strategy is utilized with i18n.pages. Additionally, it ensures compatibility with alternate links.
Linked Issues
https://github.com/harlan-zw/nuxt-simple-sitemap/issues/175
Additional context
Built upon this PR
This PR specifically modifies the behavior of the 'prefix' strategy in relation to i18n.pages (i18nPagesSources).
Screenshots
Before:

After:

Okay, looks like I have misunderstood the i18n docs, this fix you're providing is the right direction, thanks again.
The only thing I think you'd need to cover is the prefix_except_default behavior, unfortunately, there isn't a util for this, you'll need to just write the code. (I plan to make some at some point).
Also please bump the snapshots vitest -u

I only need to ensure the results for the 'prefix_and_default' case. I am currently creating the tests (Please don't check the PR yet).
Thank you for your efforts!

Oops didn't see your comments, ignore me :)

Ready to be checked 👍

Looks good to me, thanks for your work @PabloGBarcelo!


Fix ValueError by image.transform in eager mode

This fix tries to address the issue raised in #23654 where in eager mode tf.contrib.image.transform will throw out
ValueError: The truth value of an array with more than one
    element is ambiguous. Use a.any() or a.all()

This fix addresses the issue. This fix fixes #23654.
Signed-off-by: Yong Tang<EMAIL_ADDRESS>

Ping @ringw, any chance to take a look at the PR?


changes for swift4.0 and fix to add reason for error messages

Coverage remained the same at 83.039% when pulling 4fd869403a97359b933737a4bb770b2ae7865fd4 on development into 0948cdcea59254054a1e0cf4fe37c907737681ad on master.


key (ctrl+v) to paste in input Box with custom editor doesn’t work

Issue Type: Bug
Issue
I am trying to show an inputBox in one of our custom editors. But the key binding(ctrl+c / ctrl+v) doesn't work.
Everything else works as expected, i.e. you can paste using right click and choose option paste. Other key input works as well. I was able to replicate the same in the Sample custom editor code.
link to the repository where the issue can be seen - https://github.com/ap891843/vscode-extension-samples/blob/bugReplication/custom-editor-sample/src/pawDrawEditor.ts#L410
Please find the below key-binding debug screenshots:

Please find below keybinding mapping on my system

NOTE: It seems ctrl+shift+v, works on windows. But not on mac (of course, I am talking about the equivalent of ctrl+shift+v in mac)
keybinding debug output for ctrl+shift+v

Expected behavior:
ctrl+v should be able to paste to the inputBox in the custom editor.
 If not, I suggest you wait a week, by which time 1.59 should be released.

Thanks, @gjsjohnmurray. Just checked with insiders and it's working as expected. Waiting for the 1.59 release.

👍 Please close this issue


Huge amount of log data

I recently enabled auditing to a Log Analytics instance. This resulted in over 160GB of data in under 30 days. The BATCH_COMPLETED_GROUP essentially logs every action that occurs on the database. The documentation should be more explicit about the cost implications of auditing to log analytics.

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

ID: 0c27fe13-a824-84a5-e314-892a27c1dec5
Version Independent ID: 510b5427-c927-84df-563f-31dfe13fc771
Content: Get started with Azure SQL database auditing
Content Source: articles/sql-database/sql-database-auditing.md
Service: sql-database
Sub-service: security
GitHub Login: @vainolo
Microsoft Alias: arib

Thank you for the feedback. This is being assigned to the content author to evaluate and update as appropriate.

Hi @jachin84 and thanks for the feedback.
The size of the audit is very dependent on how much activity happens in the database.
If you want to limit what is audited you can change the audit groups or use a 'WHERE' clause.

This is not cool. Security Center suggests to enable auditing on the server level (recommendation "Enable auditing on SQL server"). Only a small sentence at the end of this documentation page states about the fact that all queries are logged (seriously). If auditing means simply logging every request made against the database, this should not be encouraged at a server level.
Perhaps this feature was not intended to be used with Log Analytics (where cost is payed based on ingestion rate, without any daily cap by default), but you should really warn users about that.
How could you recommend without any warning such a feature ? Our database costs nearly doubled because of this "feature".
Sorry for the rant, but as a customer I am really scared about the extend of how our bill could be skyrocketed when simply enabling this feature.

@tbolon - I understand what you say... I'm taking your feedback seriously (as I do all feedback :-)). We'll have to think of some way to better warn the users about this.
@KalyanChanumolu-MSFT  - please close this item. I've added it to our internal system.

@tbolon - note also that you can store the audit logs in an azure storage account instead of Log Analytics and the cost will be much much lower

+1 Another user running into the same issue

+1 Got a new complain on the same issue , looking forward to a resolution


How can i start the embedded Kafka server to unit test

Please, take a look into the https://github.com/spring-projects/spring-integration-kafka/blob/master/src/test/java/org/springframework/integration/kafka/rule/KafkaEmbedded.java
and here use can find something independent of Spring Integration Kafka test source coded:
https://github.com/spring-projects/spring-integration-java-dsl/blob/master/src/test/java/org/springframework/integration/dsl/test/kafka/KafkaTests.java#L167.

Is there any example how to configure in the XML file to use the embedded kafka and zookeeper

Hi,
Take a look at https://github.com/spring-projects/spring-integration-kafka/blob/master/src/test/java/org/springframework/integration/kafka/listener/ChannelAdapterWithXmlConfigurationTests-context.xml

The Spring-Kafka project has been just released in its 1.0.0.M1 release and Spring Integration Kafka 2.0 has been moved to that code based already.
That project provides KafkaEmbedded JUnit Rule which can be used from the test config together with the KafkaTestUtils to populate producer or consumer properties for the DefaultKafkaConsumerFactory/DefaultKafkaProducerFactory beans for the target KafkaTemplea and/or MessageListenerContainer beans.
Consider the issue as fixed.
Feel free to reopen or open a new one in the appropriate project.


Add a feature ClickHouse database support

ClickHouse can be tested along with all other DBses.
ClickHouse implementation is based on TimescaleDB.

Codecov Report

Merging #26 into master will decrease coverage by 10.77%.
The diff coverage is 22.12%.

@@             Coverage Diff             @@
##           master      #26       +/-   ##
===========================================
- Coverage   62.02%   51.24%   -10.78%
===========================================
  Files          60       67        +7
  Lines        2591     3329      +738
===========================================
+ Hits         1607     1706       +99
- Misses        976     1611      +635
- Partials        8       12        +4

Impacted Files
Coverage Δ

query/clickhouse.go
0% <0%> (ø)

...erate_queries/uses/devops/groupby_orderby_limit.go
0% <0%> (ø)
:arrow_up:

...s_generate_queries/databases/timescaledb/devops.go
9.16% <0%> (-7.76%)
:arrow_down:

cmd/tsbs_load_timescaledb/main.go
39.06% <0%> (-2.61%)
:arrow_down:

cmd/tsbs_load_clickhouse/profile.go
0% <0%> (ø)

cmd/tsbs_generate_queries/uses/devops/lastpoint.go
0% <0%> (ø)
:arrow_up:

load/duplex_channel.go
100% <100%> (ø)
:arrow_up:

query/scanner.go
90.47% <100%> (+0.47%)
:arrow_up:

load/scan.go
100% <100%> (ø)
:arrow_up:

cmd/tsbs_load_clickhouse/creator.go
19.13% <19.13%> (ø)

... and 18 more

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 cd0377f...3d467fb. Read the comment docs.

Implementation of this issue

Thanks for this! May be a few days before we get to this, but nice to see!
One note, for the commit msgs what I meant was "Add [a feature]" where [a feature] is replaced with what you are implementing. In this case "Add support for Clickhouse benchmarking"

... a few days before we get to this...
Yes, sure, no problem at all

... "Add [a feature]" where [a feature] is replaced with what you are implementing. In this case "Add support for Clickhouse benchmarking"
Understood, sorry, missed this idea. Hope to be more precise next time

Fixed commit message, looks better now

There are several refactorings that are not related to the Clickhouse code
...
some of them seem to change Golang-style short names (1-2 letters even) for longer, descriptive names.
...
to revert some of the more subjective changes...

Thanks for the review, and yes, I agree - there are some refactorings not directly related to the ClickHouse functionality. These refactorings originate from the code reversing process - when I've started with implementation, I had very vague understanding of the project, therefore each tool has to be reversed and understood. That's where small code-related refactorings are very handy. As Martin Fowler said: "Whenever I have to think to understand what the code is doing, I ask myself if I can refactor the code to make that understanding more immediately apparent." So through those small refactorings comes code understanding. As to extracting those small refactoring into separated PRs - yes, of course this can be done, but those refactorings are not the goal by themselves and as Martin says: "You refactor because you want to do something else, and refactoring helps you do that other thing."
As a next step, I'll review all the changes, especially subjective ones and revert those that does not makes code better or are not liked by formatter.
I'll notify as soon as update would be ready.

Great, thank you! With regards to refactorings, some of them are definitely improvements -- spelling fixes, adding code comments where they are sparse before, and some of the name changes (e.g., scaleVar  -> scale). However so that the review process does not get too unruly or drawn out, I'd like to focus the PR on Clickhouse stuff.
In general I agree with the gist of what you're saying and definitely want to make the code approachable for others to contribute so we can definitely look at those changes in isolation and see which ones make sense. Just want to keep the PRs more narrowly focused.

PR updated, some files are completely excluded, some other are partially reverted back.

Looking into code modifications and it seems to me, that there are some more smaller PRs, that can be extracted from this mega-PR (as we did with random functions. This may be the way to go.

@sunsingerus You might be right. I think reviewing some things piece meal will go more smoothly.

PR updated, one more smaller-PR extracted as a separate PR

Continue to extract separated items into smaller PRs. One more with code cleaning just added

One more smaller PR extract from this mega-PR. Code refactoring and naming clarification PR

Continue to extract small PRs from this mega-PR.
shell script PR
code refactoring

Can you elaborate on whether that is the case, or if you were mostly trying to re-use TimescaleDB scaffolding here?
... what the benefits or drawbacks of using those are? ...
... Since it seems to support a variety of connection methods (HTTP, JDBC, etc), did you see if there were performance differences?

ClickHouse exposes two API endpoints:

native binary API endpoint
HTTP endpoint

Native binary API is less (should actually be read as poorly) documented and thus very few driver implementations are based on it. Go adapter used is one of those native-API implementations. Native protocol is considered to be more effective and faster than HTTP. HTTP - on the other hand - is simpler, well documented and generally more welcomed by application developers. JDBC driver implementation, for example, is based over HTTP protocol under the hood. So, general idea is that native protocol is expected to be faster, however, there are no any numbers available (at my knowledge). This may be one of the possible future test-cases - compare speed of native vs HTTP protocols on the same tasks.
So my thought were like:

Seems to be faster
Close to existing code - easier to adopt.

Thanks for the explanations of why you chose this.
My pleasure!

I see this code was used in a blog post (I believe, unless some one else did it as well?),
CTO of Altinity Alexander Zaytsev wrote blogpost with detailed comparison of databases available in the test

so presumably the performance was satisfactory.
Each of databases has an area where it shines! Timescale and ClickHouse are in general faster than Influx, however Influx simply surprised with disk usage - it seems not to need any disk space at all!

Thank you for your efforts - really appreciate and enjoying the process!
Two small PRs more are ready - Influx shell script and docs update. And that would be all for now, have no more ready updates at the moment

As discussed here doc modifications are merged into this PR.
PR updated.

Thanks again, merged!


OpenVPN doesn't get an IP when connecting with layer 2 (tap)

Prerequisites

[x] Can you reproduce?
[x] Are you running the latest version of SoftEtherVPN?

SoftEther version: Build from master https://github.com/SoftEtherVPN/SoftEtherVPN/commit/3842e813d6a74a8c1779c3be33e1a6d70cb242ad
Component: Server
Operating system: Ubuntu 18.04 server
Architecture: 64 bit
Description
I'm using SoftEtherVPN server as an Openvpn server. Connection with an layer 3 (tun) config works fine, but when I connect with an layer 2 (tap) config I won't get an ip address.
The server has dhcp enabled as made clear by the ip that gets leased by connection to the server with an tun config.
Expected behavior:
Get an ip lease with a tap config.
Actual behavior:
I don't get an ip.
Some relevant logs
When connecting with tun I do get an ifconfig push, but not with tap.
Log from tun config:
Wed Nov 28 12:11:08 2018 PUSH: Received control message: 'PUSH_REPLY,ping 3,ping-restart 10,topology subnet,ifconfig <IP_ADDRESS> <IP_ADDRESS>,dhcp-option DNS <IP_ADDRESS>,route-gateway <IP_ADDRESS>,redirect-gateway def1'
Log from tap config:
Wed Nov 28 11:43:56 2018 PUSH: Received control message: 'PUSH_REPLY,ping 3,ping-restart 10'

It's the intended behavior, because the operating system is supposed to be sending a DHCP request through the virtual network interface, which is what happens on Windows.
On Linux you have to do it manually, before the client times out. You could set up a script for OpenVPN to do it automatically:
dhcp-manager.sh
#!/bin/bash

case $script_type in

up)
	echo "Requesting IP address for ${dev} from DHCP server..."
	dhclient -v "${dev}"
	;;
down)
	echo "Releasing ${dev}'s DHCP lease..."
	dhclient -r "${dev}"
	;;
esac

In OpenVPN's configuration:
up dhcp-manager.sh
down dhcp-manager.sh

Alright, after some more testing I did notice that Windows did indeed get an ip and it's only a problem on Linux.
Though when using the standard OpenVPN server (which I plan to replace with a SoftEther one), the server does send an ip to Linux and Windows without the manual dhclient call.
Maybe this could be a new feature for SoftEther? Perhaps a setting that forces the SoftEther OpenVPN server to send out dhcp leases.

The OpenVPN server acts as a DHCP server when you set the network in its configuration, otherwise it doesn't send the IP address because it isn't a DHCP client.
We can implement the feature in SoftEther VPN, with a client-side environment variable such as UV_NO_DHCP to disable it.

The client timing out turned out to be an OpenVPN bug, we implemented a workaround in #828.
#827 implements the desired feature, however in Layer 2 mode the DHCP lease is supposed to be requested by the client and not by an intermediary entity (the VPN server).
Do you still want the feature, now that the timeout issue is not present anymore?

I've tested #827 and #828 and both seems to be working fine. If its up to me this feature can be merged. Again, thank you for the great work!

You're welcome!

#827 implements the desired feature, however in Layer 2 mode the DHCP lease is supposed to be requested by the client and not by an intermediary entity (the VPN server).

We're probably not going to merge #827 for this reason, sorry.

Alright, that totally understandable. I'll make a patch from your implementation for my own projects.

No problem.


ARM template deployment with AKS broken

An arm template that was working last week is suddenly broken throwing this error:
Exception Details:
Error Code: InvalidParameter
Message: Provisioning of resource(s) for container service <AKSresourceName> in resource group <resourceGroupName> failed. Message: {
"code": "InvalidParameter",
"message": "The value of parameter orchestratorProfile.OrchestratorVersion is invalid.",
"target": "orchestratorProfile.OrchestratorVersion"
}. Details:
There is nothing in the arm template that references an "OrchestratorVersion" parameter. I can't find anything on the AKS arm template reference that mentions OrchestratorVersion or OrchestratorProfile.

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

ID: 60491b3e-5e7a-6eab-cbfd-d87247e59c7d
Version Independent ID: 623c51cc-3eda-a4c3-2719-d354ca3befbe
Content: Deploy resources with PowerShell and template
Content Source: articles/azure-resource-manager/resource-group-template-deploy.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

So in my ARM template's AKS properties, I had
"kubernetesVersion": "1.11.3"

When I changed it to:
"kubernetesVersion": "1.11.5"

The deployment worked.


OCPBUGS-10423: Add validation for taint.value in nodePool

What this PR does / why we need it:
Fail earlier for invalid input.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

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

/cherry-pick release-4.13


Need revamping

[Enter feedback here]
Refresh the docs

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

ID: 020e0c95-525b-eebd-d7f6-eb2214e5d6e9
Version Independent ID: 8ce00223-24fb-809c-b97f-9cd7996aa25c
Content: Add an advanced option and multiple actions - Power Automate
Content Source: articles/multi-step-logic-flow.md
Service: flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

test


Use plus### and minus### shorthands when possible for dates/times

Uses e.g. LocalDate#plusDays instead of more contrived alternatives.
Quite a big PR as sadly none of the involved classes share a common interface for the shorthand methods.

Added two commits.

Added two commits.


Change output format to pandas dataframe

Hi,
thanks for creating this package. In my opinion, it would be more convenient to have the outputs in pandas dataframe format (e.g. similarly as in the yfinance package).
Have you thought about that, is there an argument to stick with the current implementation with dictionary outputs?
If not, I would be happy to contribute to this project and change outputs to pandas dataframes

I love this idea!

As much as I love using Pandas DataFrames, I don't think this is a good idea, for multiple reasons.
Some people would want to use the raw data, without DataFrames.
As for me, I sometimes like to rearrange / edit data before getting them to a DataFrame, which is easier on raw data.
Plus, getting raw data allows people to choose what type they want in the DataFrame, which is important, especially regarding floats or decimals : some will want to use floats (with the inevitable base 2 imprecision), some will prefer getting accurate decimal type.
If you commit the data directly in the framework, it will most probably be as floats, and you'll lose the ability to keep accurate decimal data. Or the developer will have to implement both, letting the user choose, which leads me to...
...As a final note, I don't think that such a framework is the right place to do any transform on data.
This belongs to the logic of the program using the data, not to the framework used to retrieve them.

I get your point.
What about introducing a boolean parameter and the service returns raw data when given the one value and a dataframe when given the other value?
It is your decision, I am just thinking that for some people it is not that easy to work with the raw data. If you have a dataframe, you can export it to Excel with a one-liner, etc. This might increase adoption of the package.

Hi!
Since this package is a wrapper around the API, in my opinion should return raw responses with the data. Lists and dictionaries formats are in agreement with the returns of the API. No extra processing of the responses is implemented in the wrapper.
You can check how CoinGecko returns data for its endpoints in the API documentation (https://www.coingecko.com/en/api).
I think the best solution is to always keep the data in raw format, as returned from the API. This allows developers to choose how to handle better the data, based on their needs. Also, keeps debugging simpler, both for the package and the programs that use the package.
Doing extra processing, using DataFrames, would add extra - unneeded, in my opinion - requirements for the package, and would make it more demanding to maintain and use (e.g. if the API changes the responses for some endpoints, the wrapper might be needed to be updated and users will also need to update it). Since there are a lot of endpoints, the package will be required to create proper DataFrames for each one, making sure that it handles properly the additional (optional) parameters.
Also, documentation would be more demanding (to include the DataFrames, etc). As it now, users can work with the CoinGecko API documentation.
This kind of data handling seems to me as a developers' decision. It should better be decided by the developers of a program that uses the wrapper and not by the package itself, in order to be as modular as possible.
Also, converting a raw response from the API to a pandas DataFrame seems simple for most endpoint response using functions that the pandas package provides itself (e.g. using pandas.DataFrame.from_dict).
So, i think working with raw data, fits best for the wrapper.
In any case, thank you for your suggestions!


The duration based filters in the redaction tool do not take accuracy into account enough

The dataset I am using is me in my house for 2.5 hours with GPS / bluetooth switched on.  Most of the data points correctly show in pink as I am in the same location.  However, when I use the "Hide < 1hr" it hides all non-travel.  I would expect that these locations would be identified as similar enough to be the same location, and therefore that I had been there over an hour, and would not be hidden.
Steps to reproduce:

Unzip and upload the attached
Select Hide < 1hr
1587216857693.zip

Looking at this issue right now:  can reproduce the bug

Update on this bug:
The reason for bug: The data is for 2.5 hrs, in between:  you moved to a different location.
When we add a new marker to exposure Set we check the first location point of the last exposure set.
So if you are home: you stayed at home for 35 and 42 mins each, the gap was enough to create a new exposure set. SO total wasn't calculated properly we didn't put the two locations together.
The issue I am facing to fix this bug is:

How many exposure sets do I compare before I plug values  in the existing exposure set?
how much time duration before we can say the location is actually new?


feat: adding dataset start time

Description
This PR add the field startTime to raw datasets to match the field endTime.
Motivation
We were using scientific metadata to store start and end time of a data acquisition, without using the endTime field.
We wanted to enforce consistency and have both information in the high level fields of the datasets
Fixes:

datasets dto and schemas

Tests included

[ ] Included for each change/fix?
[ ] Passing? (Merge will not be approved unless this is checked)

Documentation

[x] swagger documentation updated [required]
[ ] official documentation updated [nice-to-have]

official documentation info
If you have updated the official documentation, please provide PR # and URL of the pages where the updates are included

Error: ception during run: n [Error]: Max -730985476 should be greater than min 0.
[0]     at y.int (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/@faker-js/faker/dist/cjs/modules/number/index.js:1:859)
[0]     at T.between (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/@faker-js/faker/dist/cjs/modules/date/index.js:1:2136)
[0]     at faker.helpers.multiple.count (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/@faker-js/faker/dist/cjs/modules/date/index.js:1:2482)
[0]     at Function.from (<anonymous>)
[0]     at ee.multiple (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/@faker-js/faker/dist/cjs/modules/helpers/index.js:1:8233)
[0]     at T.betweens (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/@faker-js/faker/dist/cjs/modules/date/index.js:1:2464)
[0]     at Object.<anonymous> (/home/runner/work/scicat-backend-next/scicat-backend-next/test/TestData.js:9:33)
[0]     at Module._compile (node:internal/modules/cjs/loader:1358:14)
[0]     at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
[0]     at Module.load (node:internal/modules/cjs/loader:1208:32)
[0]     at Module._load (node:internal/modules/cjs/loader:1024:12)
[0]     at Module.require (node:internal/modules/cjs/loader:1233:19)
[0]     at require (node:internal/modules/helpers:179:18)
[0]     at Object.<anonymous> (/home/runner/work/scicat-backend-next/scicat-backend-next/test/Auth.js:2:22)
[0]     at Module._compile (node:internal/modules/cjs/loader:1358:14)
[0]     at Module._extensions..js (node:internal/modules/cjs/loader:1416:10)
[0]     at Module.load (node:internal/modules/cjs/loader:1208:32)
[0]     at Module._load (node:internal/modules/cjs/loader:1024:12)
[0]     at cjsLoader (node:internal/modules/esm/translators:348:17)
[0]     at ModuleWrap.<anonymous> (node:internal/modules/esm/translators:297:7)
[0]     at ModuleJob.run (node:internal/modules/esm/module_job:222:25)
[0]     at async ModuleLoader.import (node:internal/modules/esm/loader:316:24)
[0]     at async formattedImport (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/mocha/lib/nodejs/esm-utils.js:9:14)
[0]     at async exports.requireOrImport (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/mocha/lib/nodejs/esm-utils.js:42:28)
[0]     at async exports.loadFilesAsync (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/mocha/lib/nodejs/esm-utils.js:100:20)
[0]     at async singleRun (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/mocha/lib/cli/run-helpers.js:162:3)
[0]     at async exports.handler (/home/runner/work/scicat-backend-next/scicat-backend-next/node_modules/mocha/lib/cli/run.js:375:5)
[0] wait-on http://localhost:3000/explorer/ && npm run test:api:mocha exited with code 1
--> Sending SIGTERM to other processes..
[1] npm run start exited with code SIGTERM
Error: Process completed with exit code 1.

This errors occured too many times. We need to check what is the cause before merge the PR.


client allows ignoring spurious ModifyResponse.SessionParamsResult

This is controlled by a global variable default to false (the original
behavior).  When set to true, it will allow the gRIBI server to send
SessionParamsResult unconditionally which is a non-compliant behavior.
This is necessary to workaround some buggy implementations so it does
not block testing.

Withdrawing PR. Users of gribigo should not expect to be dealing with a server implementation that exhibits non-compliant behavior. Tests that wish to recover from non-compliant behavior in order to keep going should be scoped to test this at the gRPC level and not using the client library.


OT-2 Refresh: Configurable currents depending on hardware revision

Add support for saving, configuring, and changing gantry and Z-stage currents based on the CRB revision.

All currents found in the robot settings should be configured dependent on the stepper motor driver hardware revision.
The exact changes we need for now, these should apply to all stepper driver boards rev A and higher. (12.5 and lower get the old settings)

Holding current - we want the X/Y motor holding current to be 0.7A.
I believe this is the "low_current" in the robot settings.

Z/A general movement current set to 0.5A. The goal here is to have all general movements of the Z/A (so basically everything except tip pick up (which is set separately) to be 0.5A.
I believe this means the "high_current" and "default_current" should be 0.5A for Z/A but someone should check that that actually gives the desired function.


Remove has_new_dashboard

~We have overriden render in organization_users_controller.rb~
We should be able to get rid of it and the rest of has_new_dashboard calls once the dashboard migration has finished

Can be closed in favour of this one


avg_memory_usage_percent may not be supported on SQL Managed Instance

avg_memory_usage_percent may not be supported on SQL Managed Instance. I think the document should note this limitation.
SELECT end_time,avg_memory_usage_percent,avg_instance_memory_percent FROM sys.dm_db_resource_stats;

https://learn.microsoft.com/en-us/sql/relational-databases/system-dynamic-management-views/sys-dm-db-resource-stats-azure-sql-database?view=azuresqldb-current

@akikawa1123 -- Akikawa, thank you for your feedback.
@rwestMSFT -- Randolph, please look into this issue.

We're investigating, thanks.

We will be removing MI as a supported option from the "Applies to" section at the top. Thanks for raising this issue, @akikawa1123. When the internal PR is merged, this issue will automatically close.


impulse response is missing an input?

julia> ACE_Traeger_replication.Impulse_response_combined()
ERROR: File "/Users/floswald/git/Comp2024/ACE_Traeger_replication.jl/impulse_timestep_5_logi_001.mat" does not exist and create was not specified
Stacktrace:
 [1] error(s::String)
   @ Base ./error.jl:35
 [2] matopen(filename::String, rd::Bool, wr::Bool, cr::Bool, tr::Bool, ff::Bool, compress::Bool)
   @ MAT ~/.julia/packages/MAT/K2D15/src/MAT.jl:45
 [3] matopen(fname::String, mode::String; compress::Bool)
   @ MAT ~/.julia/packages/MAT/K2D15/src/MAT.jl:88
 [4] matopen
   @ ~/.julia/packages/MAT/K2D15/src/MAT.jl:87 [inlined]
 [5] matopen
   @ ~/.julia/packages/MAT/K2D15/src/MAT.jl:97 [inlined]
 [6] matread(filename::String)
   @ MAT ~/.julia/packages/MAT/K2D15/src/

Please make sure that you have all of the following .mat datasets downloaded from the replication package of the paper, and that you saved them in your "datapath" folder of choice:
impulse_timestep_5_logi_001.mat
impulse_timestep_1_logi_001.mat
Venmans_CS3_lam_1p06.mat
Impulse_Response_Bern2p5_Joos.mat
Impulse_Response_Bern2p5_PD_Joos.mat
Venmans_CS3p1.mat

Please do let us know if you still run into issues once this has been checked or if you have further questions!

ok but then in your package installation guide you cannot have point 4 "run the module". you have to get the data before that point.

The installation guide includes instructions on downloading the data:

sure. but you arrive to step 4 here and you fail. you need to put the data downloading before step 4.

By using include("ACE_Traeger_replication.jl") you don't actually run the functions, so it will still work without the data. Please do try, and let us know if it fails. Happy to change the structure of instructions otherwise.