Add pop_base_yr to default_parameters.json

This PR adds a parameter names pop_base_year to OG-Core default_parameters.json. I am trying to implement this parameter in a PR to the OG-UK repository (PR #36). This parameter is valuable for the OG-UK country calibration repository because the UK demographics from Eurostat come from a base year. We then have to age the current population to the correct start_year.
The current settings in this PR of min and max and default values are based on restrictions for OG-UK. However, we can adjust those settings to be more general to include OG-USA and other country calibration repositories. I think the principle should be that this parameter's bounds in OG-Core should have max and min values that cover the maximum range of values available in country calibrations. Then each country calibration repository can place more restrictive settings within its respective demographics.py module and corresponding functions. But I think this parameter is general enough to be warranted in the OG-Core repo.
For example, we should probably update OG-USA's demographics.py module to also use this parameter. I think the current OG-USA module hard codes a pretty old year, and treats that at the demographic values of the start_year.
cc: @jdebacker

Codecov Report

Merging #802 (dedd6dc) into master (cdb9b7f) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #802      +/-   ##
==========================================
- Coverage   84.10%   84.09%   -0.01%
==========================================
  Files          19       19
  Lines        3434     3414      -20
==========================================
- Hits         2888     2871      -17
+ Misses        546      543       -3

Flag
Coverage Δ

unittests
84.09% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ogcore/firm.py
90.19% <0.00%> (-0.19%)
:arrow_down:

ogcore/SS.py
86.81% <0.00%> (-0.17%)
:arrow_down:

ogcore/txfunc.py
39.28% <0.00%> (-0.13%)
:arrow_down:

ogcore/TPI.py
93.13% <0.00%> (-0.09%)
:arrow_down:

ogcore/tax.py
99.35% <0.00%> (-0.01%)
:arrow_down:

ogcore/fiscal.py
100.00% <0.00%> (ø)

ogcore/household.py
100.00% <0.00%> (ø)

ogcore/aggregates.py
100.00% <0.00%> (ø)

ogcore/elliptical_u_est.py
100.00% <0.00%> (ø)

ogcore/execute.py
17.14% <0.00%> (+1.35%)
:arrow_up:

@jdebacker. Yes, I can definitely put this parameter in the oguk.calibration.Calibration class object. But it won't have the automatic ParamTools validation that comes with the parameters in OG-Core's default_parameters.json. But I can build validation into the parameter in oguk.calibration.Calibration. I just starts looking like something that should be run through ParamTools.

Closing. Will put this in oguk.calibration.Calibration class object.


About  TCP rotating XYZ

Hello!
When the X axis is rotated, set the X value to 1.55, the angle of the A3 is 86, and when the X value continues to increase to 1.65, the A3 value is -89, then the machine will swirl quickly. How to fix this problem?

A3 will only operate in the range of +-90 degree. The idea is that you can reach all poses with this range. (Not without a reconfiguration of the wrist in some positions though)
The goal is to implement proper robot configurations that circumvent this issue. Currently one configuration is hardcoded.
If you need a smooth motion over these boundaries you may need to hack this yourself. Look for the kinematics.cpp. At the end you can check if the angle is <=0 and add something like angle[3] = 90Deg2Rad+90Deg2Rad+angle[3] (basically changing the range of motion from  -90/90 to 0/180. not tested. ) You also have to flip the axis 4 then.

Am 28.04.2018 um 11:09 schrieb natdon<EMAIL_ADDRESS>Hello!
When the X axis is rotated, set the X value to 1.55, the angle of the A3 is 86, and when the X value continues to increase to 1.65, the A3 value is -89, then the machine will swirl quickly. How to fix this problem?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Clarify impact of running Update targets

[Enter feedback here]
The document says, "You can force the agents on the target servers to be upgraded to the latest version without needing to redeploy them by selecting Update targets from your deployment groups page." However, it does not state whether this process is interruptive to runs that are currently executing.

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

ID: ee31adc7-dad4-2df7-639c-eabdf3e1ae86
Version Independent ID: 50eda1c6-7339-f636-b243-1fae4529bab0
Content: Use deployment groups in a release pipeline - Azure Pipelines
Content Source: docs/pipelines/release/deployment-groups/index.md
Service: azure-devops-pipelines
Sub-service: azure-devops-pipelines-release
GitHub Login: @RoopeshNair
Microsoft Alias: ronai

@jamesbr-ashn -- thank you for your feedback.  Please consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

@RoopeshNair -- please look into this issue.


xtask: utility to launch zombienet

Current dependencies on/for this PR:

#248  👈
main

This stack of pull requests is managed by Graphite.


Introduction Screen

Design Introduction screen.
Introduction screen should display concept of our application.
Add animations on Introduction screen.

Design Introduction screen - Done.
Introduction screen should display concept of our application - Done.
Add animations on Introduction screen - Done.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #302 (aa9bff8) into main (2974dfe) will increase coverage by 12.10%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##             main     #302       +/-   ##
===========================================
+ Coverage   83.33%   95.44%   +12.10%
===========================================
  Files           2        1        -1
  Lines         126      395      +269
  Branches       31       46       +15
===========================================
+ Hits          105      377      +272
+ Misses         21       18        -3

Impacted Files
Coverage Δ

src/index.ts
95.44% <100.00%> (+13.44%)
:arrow_up:

example/src/main.ts

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 2d49f79...aa9bff8. Read the comment docs.


Malfunctioning of the Check button when h5p-highlight-the-words inserted in interactive book.

Malfunctioning of the Check button when h5p-highlight-the-words inserted in interactive book.
Steps to reproduce
1.- Use h5p-highlight-the-words latest version 0.1.5 on a dev site
2.- h5p-column create a new branch, e.g. testing_highlight and, in semantics, add "H5P.HighlightTheWords 0.1", and in h5p-column.js var isTasks add "H5P.HighlightTheWords"
3.- Create an interactive book
4.- Create page 1 with only a Text content in it
5.- Create page 2 with a Multiple Choice question in it
6.- Create page 3 with an HighlightTheWords content in it. Options enable Show solutions.
7.- Save and run the H5P Interactive Book
8.- Page 1 : read the text
9.- Navigate to page 2 (with the MCQ question) : The Check button below the question appears very briefly then disappears!
10. Navigate to page 3 (with the HighlightTheWords content) Colour some words, then click the Check button: the score is not displayed, but the Show solution and Retry buttons are displayed.
11. Click the Show solution button: nothing happens.
12. Press the F5 key on your keyboard to refresh the page: colour some words, click the Check button, now things work as expected, the Score is displayed, and the Show solution button works, etc.
13.- Navigate back to page 2 (with the MCQ content) and now you see that the Check button is displayed.

@rezeau Please feel free to fix that yourself. I do not intend to work on this content type anymore.

OK, I'll have a look.


Implementation of parallel upload/download functions

Is your feature request related to a problem? Please describe.
We would like to upload blocks in parallel in a blob storage and I'm frustrated to not having the CommitBlockListAsync function implemented while it is already available in azure-sdk-for-net, for example.
We also need a function to download in parallel.
Describe the solution you'd like
The implementation for parallel upload/download in a blob storage.
Describe alternatives you've considered
We don't see another alternative.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Description Added
[x] Expected solution specified

@youennL-cs
We have concurrent upload and download in blob SDK.
for example, with BlobClinet::DownloadTo, you can download a blob to a file or memory buffer with multiple threads, the concurrency is configurable with DownloadBlobToOptions.

I saw this function but can I consider Async as DownloadToFileParallelAsync available for .NET ?
And I can't see anything for upload/commit.
The current CommitBlockList (which is not Async) as no concurrency parameter in the option (CommitBlockListOptions) and so it doesn't look like having a way to parallelise the commit.

C++ SDK doesn't support async functions. You can call sync methods from multiple threads to accomplish parallelism.


"json.decoder.JSONDecodeError: Invalid \escape: line 6 column 21 (char 128)"

I cannot for the life of me figure out how to fix this.
this is the full error
Traceback (most recent call last):   File "<frozen runpy>", line 198, in _run_module_as_main   File "<frozen runpy>", line 88, in _run_code   File "c:\users\miche\.local\bin\zotify.exe\__main__.py", line 7, in <module>   File "C:\Users\miche\AppData\Local\pipx\pipx\venvs\zotify\Lib\site-packages\zotify\__main__.py", line 64, in main     args.func(args)   File "C:\Users\miche\AppData\Local\pipx\pipx\venvs\zotify\Lib\site-packages\zotify\app.py", line 21, in client     Zotify(args)   File "C:\Users\miche\AppData\Local\pipx\pipx\venvs\zotify\Lib\site-packages\zotify\zotify.py", line 20, in __init__     Zotify.CONFIG.load(args)   File "C:\Users\miche\AppData\Local\pipx\pipx\venvs\zotify\Lib\site-packages\zotify\config.py", line 106, in load     jsonvalues = json.load(config_file)                  ^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\miche\scoop\apps\python\3.12.2\Lib\json\__init__.py", line 293, in load     return loads(fp.read(),            ^^^^^^^^^^^^^^^^   File "C:\Users\miche\scoop\apps\python\3.12.2\Lib\json\__init__.py", line 346, in loads     return _default_decoder.decode(s)            ^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\miche\scoop\apps\python\3.12.2\Lib\json\decoder.py", line 337, in decode     obj, end = self.raw_decode(s, idx=_w(s, 0).end())                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\miche\scoop\apps\python\3.12.2\Lib\json\decoder.py", line 353, in raw_decode     obj, end = self.scan_once(s, idx)                ^^^^^^^^^^^^^^^^^^^^^^ json.decoder.JSONDecodeError: Invalid \escape: line 6 column 21 (char 128)
please help me

I also have this error, word for word, except the very last line reads as:
json.decoder.JSONDecodeError: Invalid \escape: line 3 column 32 (char 65)
I am just a little noob who doesn't know code.

I also have the same error. Has anyone found a solution?

never had that error so no idea ...
just one hint though :  in my case that decode.py file is located in C:\Users********\scoop\apps\python\3.12.4\Lib\json
vs. yours in ..........python\3.12.2\lib.................
=> I would suggest : try to update Python (or just that json library maybe) to a more recent or latest version

Nah found the problem, in the config.json file I had written my download path with only one \ (C:\Example\example1, which caused an error, fixed by simply replacing with \\ (C:\\Example\\example1


IOS: Improve error reporting

We should validate and improve how errors are reported from PowerAuthSDK class:

We should be checking the output of JSON parsing for errors in custom activation method, so that we can differentiate JSON parsing related errors better.
We should be checking for HTTP status code - while the server should technically return only status codes handled by SDK, we are unable to assure this in case of the custom activation, since the back-end code is implemented by the bank API provider (not part of our SDK).
It's unfortunate that SDK is hiding some error states reported from low level code or reinterprets them to more vague meaning. We should just extend "core" error codes with values needed in high level code.
Related to previous point: PA2Session should produce NSErrors instead of keeping last error code returned from C++
We should add textual description to reported NSErrors. That might improve debugging a lot.

Related to #44 #46

PA2Client should process authentication failure response properly and return appropriate NSError object with PA2ErrorCodeAuthenticationFailed. Right now, the error code is defined, but never referenced in the SDK.
It is actually very difficult to determine whether the AUTH error occurred. The SDK usually returns misleading PA2ErrorCodeNetworkError and the programmer have to dig more into additional objects, attached with the error, like model and internal enums (will be internal in future releases).
Due to this, I'm switching from enhancement to bug. The current implementation makes no sense at all and has to be fixed asap.

We should add two more error codes:

PA2ErrorCodeActivationBlocked - will be returned when AUTH fails and the activation is now blocked (if we can distinguish this state from regular AUTH failure)
PA2ErrorCodeActivationRemoved - will be returned when the activation is no longer valid

Removed from 2018.06 project. I think we can switch this to "sleeping", because it looks like the issue is not very important after all

Closing because it's not relevant after all. The proposed changes may significantly break existing contracts, so it's better to take this ticket as an inspiration for the future SDK versions. For example, once we decide rewrite high level codebase to swift.


Entry point and some general adjustments

Added entry point for Ontolearn: #321

added script main.py
added model_adapter.execute() method

Added option to include implicit individuals: #324

Specified by the boolean argument include_implicit_individuals of class KnowledgeBase

Fixes

ERROR: InvocationError for command /usr/bin/conda env create -p /home/runner/work/Ontolearn/Ontolearn/.tox/docs --file /home/runner/work/Ontolearn/Ontolearn/environment.yml (exited with code 1)

I guess we should start working on removing conda from ontolearn due to its excidingly large memory need


Visibility

This started as a simple PR to implement visibility: hidden on the gallery to avoid the massive icons flashing every so often on (re-)load (a corresponding front-end side changes are required, BTW) but I kept going...
I added code to pass a flag to the front end to let it know whether an album has subalbums or not; that way we can limit the recursion in get_albums(), hopefully further speeding up Album::get (see #316).
I also noticed a stray return from the last round of patches that's no longer needed now that we return through an array passed by reference.

Codecov Report

Merging #326 into master will increase coverage by 0.07%.
The diff coverage is 11.11%.


LNK2005

Do anybody know how to solve that problem DriverEntry already defined

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

ID: 81568a12-c39e-a18f-8f33-c54545f14bac
Version Independent ID: 73217b22-1795-b5d0-65cb-1984f38dc661
Content: Write a Hello World Windows Driver (KMDF) - Windows drivers
Content Source: windows-driver-docs-pr/gettingstarted/writing-a-very-small-kmdf--driver.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @EliotSeattle
Microsoft Alias: eliotgra

Hi @judah-b2
Thanks for reaching out to our team. This particular channel is monitored by the men and women who write the  driver documentation posted on docs.microsoft.com/drivers. Though we'd like to, we don’t have the resources to help you with your code.

Your best bet is to use one or both of the two really good forums where other driver developers and Microsoft engineers can be found. The first is Microsoft Q&A: https://docs.microsoft.com/answers/products/.
The other is the NTDEV forum from OSR Online: https://community.osr.com/.

You can post your question in those locations, and the community of developers usually give feedback fairly quickly.

Again, thanks for reaching out. Best of luck with this issue.
Eliot


Render Box Log requested bug report bc nto enough info given by flutter in log.

RenderBox was not laid out: RenderRepaintBoundary#81ec0 relayoutBoundary=up21 NEEDS-PAINT
'package:flutter/src/rendering/box.dart':
Failed assertion: line 1979 pos 12: 'hasSize'
Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=2_bug.md
The relevant error-causing widget was:
Column Column:file:///Users/neil.warner/dev/ri_driver/lib/main.dart:57:14
When the exception was thrown, this was the stack:
#2      RenderBox.size (package:flutter/src/rendering/box.dart:1979:12)
#3      ChildLayoutHelper.layoutChild (package:flutter/src/rendering/layout_helper.dart:57:18)
#4      RenderFlex._computeSizes (package:flutter/src/rendering/flex.dart:829:43)
#5      RenderFlex.performLayout (package:flutter/src/rendering/flex.dart:931:32)
#6      RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#7      RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#8      RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#9      RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#10     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#11     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#12     RenderCustomPaint.performLayout (package:flutter/src/rendering/custom_paint.dart:545:11)
#13     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#14     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#15     _RenderCustomClip.performLayout (package:flutter/src/rendering/proxy_box.dart:1376:11)
#16     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#17     RenderPadding.performLayout (package:flutter/src/rendering/shifted_box.dart:233:12)
#18     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#19     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#20     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#21     ChildLayoutHelper.layoutChild (package:flutter/src/rendering/layout_helper.dart:56:11)
#22     RenderFlex._computeSizes (package:flutter/src/rendering/flex.dart:829:43)
#23     RenderFlex.performLayout (package:flutter/src/rendering/flex.dart:931:32)
#24     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#25     ChildLayoutHelper.layoutChild (package:flutter/src/rendering/layout_helper.dart:56:11)
#26     RenderFlex._computeSizes (package:flutter/src/rendering/flex.dart:829:43)
#27     RenderFlex.performLayout (package:flutter/src/rendering/flex.dart:931:32)
#28     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#29     _RenderSingleChildViewport.performLayout (package:flutter/src/widgets/single_child_scroll_view.dart:513:14)
#30     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#31     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#32     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#33     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#34     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#35     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#36     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#37     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#38     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#39     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#40     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#41     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#42     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#43     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#44     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#45     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#46     RenderCustomPaint.performLayout (package:flutter/src/rendering/custom_paint.dart:545:11)
#47     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#48     RenderProxyBoxMixin.performLayout (package:flutter/src/rendering/proxy_box.dart:116:14)
#49     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#50     RenderPositionedBox.performLayout (package:flutter/src/rendering/shifted_box.dart:437:14)
#51     RenderObject.layout (package:flutter/src/rendering/object.dart:1915:7)
#52     MultiChildLayoutDelegate.layoutChild (package:flutter/src/rendering/custom_layout.dart:171:12)
#53     _ScaffoldLayout.performLayout (package:flutter/src/material/scaffold.dart:1003:7)
#54     MultiChildLayoutDelegate._callPerformLayout (package:flutter/src/rendering/custom_layout.dart:240:7)
#55     RenderCustomMultiChildLayoutBox.performLayout (package:flutter/src/rendering/custom_layout.dart:403:14)
#56     RenderObject._layoutWithoutResize (package:flutter/src/rendering/object.dart:1757:7)
#57     PipelineOwner.flushLayout (package:flutter/src/rendering/object.dart:887:18)
#58     RendererBinding.drawFrame (package:flutter/src/rendering/binding.dart:504:19)
#59     WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:892:13)
#60     RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:370:5)
#61     SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1146:15)
#62     SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1083:9)
#63     SchedulerBinding._handleDrawFrame (package:flutter/src/scheduler/binding.dart:997:5)
#67     _invoke (dart:ui/hooks.dart:151:10)
#68     PlatformDispatcher._drawFrame (dart:ui/platform_dispatcher.dart:308:5)
#69     _drawFrame (dart:ui/hooks.dart:115:31)

Hi @neiljaywarner, please provide a completed and minimal reproducible code sample so that we may verify this. Also, please provide the output of flutter doctor -v as well.
Thank you!


allow custom tags to be specified

@geuben Why was this closed?  I'm trying to do something like this and I can't figure how to do it. Do you?


Fix null parameter deprecation

When using PHP 8.1, it constantly throws the deprecation log on the Log display screen.

hey @arukompas, after updating log-viewer to v2 we have same issue. While waiting on the screen, the warning logs are looped.

@faytekin thanks for finding and fixing the bug! 👏
I found one more place where the array_filter was needed.
Tagged new releases for both 1.x and 2.x versions.
cc: @nozdogan


Add CUDA C++ Core Libraries: cub, libcudacxx, thrust

Stable URLs aren't available for these older versions which were in separate repos.
Latest releases are now in a monorepo and I have put in a FR for stable release urls: https://github.com/NVIDIA/cccl/issues/622

Hello! I'm doing some routine cleanup of stale PRs. If you're still working on this and planning to make progress soon, please let me know and we can re-open it.


Fix various issues in container-related code

👍

@karthiknadig , can you please take a look as well?


feat: Add new recognizer for IN_VOTER #1344

Change Description

Add new reconizer IN_VOTER with regex
Added test module for the same

Issue reference
This PR fixes issue #1344
Checklist

[x] I have reviewed the contribution guidelines
[x] I have signed the CLA (if required)
[x] My code includes unit tests and linting
[x] All unit tests and lint checks pass locally
[x] My PR contains documentation updates / additions if required

@microsoft-github-policy-service agree

/azp run

/azp run

@omri374 thank you for your time to reviewing this, i have updated the changes accordingly as per your suggestions

/azp run

/azp run

@kjdeveloper8 the CI is failing due to a test counting the number of predefined recognizers. Can you please update it?

updated

/azp run

/azp run

Thanks! Merged


upstream: trigger original dst clean only when hosts are there

Description: The log line Cleaning up stale original dst hosts is triggering quite often when upstream log is enabled even there are no hosts. My intention was to control this log line, but I think it does not harm to guard whole cleanup logic with this condition.
Risk Level: Low
Testing: Low
Docs Changes: N/A
Release Notes: N/A

@venilnoronha That was my original thought - but then looking at the log lines in the loop, if we drop this to trace and leave other two log lines in debug they seem to come out of context. But I agree we will loose the information that it is running - Are you OK, making all three log lines to trace? I am OK changing it like that.

@venilnoronha  I went ahead and changed all three log lines to trace. LMK, if you think there is a better alternative.

Can we keep your previous logic (and the 3 debug logs) and add an additional trace log outside the if condition?

I see. I can do that.

/retest

@snowp could you help on reviewing this?


decode_message MemoryError

Hello,
I'm quite new in python, and I'm trying to use cantools, mainly the CAN decode_message feature.
I'm using Python 3.9 and cantools 35.4.0
My issue is that, when I parse a blf file and decode each message, I can see the memory usage increase until I got an error MemoryError and the script stop.
My loop is this one :

 for file in fileList:
     # Read BLF
     print("Loading BLF file from '{}'.".format(inputFilePath + file))
     log = can.BLFReader(inputFilePath + file)

     logFull = list(log)

     # Keep only message in the filter
     logLight = [msg for msg in logFull if msg.arbitration_id in keepMsg]

     for msg in logLight:
         decodedMessage = db.decode_message(msg.arbitration_id, msg.data)

Maybe I don't use the function correctly ? I can get decoded data correctly, but there is this memory issue.
If I remove the line with decode_message, the memory is fine.
Maybe someone have a clue to solve this issue ?
Thanks

You are not using this project. Create an issue here instead: https://github.com/hardbyte/python-can

The project you indicate is for the BLFReader I think, I don't have any issue with this feature.
I got the memory issue when I use the decoding function from this project I think ?

Ohh, sorry, you are right. Could you provide the traceback?

You can find the traceback here :
Traceback (most recent call last):
  File "test.py", line 38, in <module>
  File "test.py", line 38, in <listcomp>
MemoryError
[9952] Failed to execute script test

line 38 is :
logLight = [msg for msg in logFull if msg.arbitration_id in keepMsg]
and memory is full

Well, that's before cantools is used, right? So the problem is that the log is too big and your machine runs out of memory?

Each log files are around 1.4 MB.
If I parse all files without decoding them, I have no issue with the memory

Could you provide an example I can use to reproduce the issue? Makes it easier for me to troubleshoot it.

I think I managed to reproduce it myself.

Update the cantools dependency bitstruct to version 8.11.1.  Something similar to pip install -U bitstruct. -U is important.

ok I will try and let you know
Thanks :)

After updating bitstruct dependency, memory issue disappear :)
Thanks for your quick help !


Fix errors in managed resources reconciliation

This PR fixes a few issues found when trying to reconcile the auxiliary managed resources.
Signed-off-by: Jose A. Rivera<EMAIL_ADDRESS>

I have manually verified that StorageClasses are updated appropriately now. The other resources also respond to toggling the reconcile strategy.

/override ci/prow/red-hat-storage-ocs-ci-e2e-aws

/cherry-pick release-4.6


using jquery.inputmask in laravel mix

Hello,
I want to use jquery.inputmask in laravel mix.
How can I do this job.

bump

@AGhasemzadeh , @isudakoff ,
I guess by using npm install inputmask
And in your js require or import inputmask.
Someone has experience with laravel mix?

@RobinHerbots it is failing when I require it in the bootstrap.js file.
Not sure why it isn't working 🤔
I am trying the standalone version ( no jquery ) and see if that works.

@alighasemzadeh This thread is a little old, but based on your github profile I assume your using bootstrap. If so, put the input-mask.js within the /resources/assets/js/ folder, then put require('./input-masks.js'); between the try { } within the bootstrap.js file (which is within the same /resources/assets/js/ folder).


bigtable: Implement consistent retry mechanisms

Is your feature request related to a problem? Please describe.
It is frustrating to utilize a provided retry mechanism for one API call, and being forced to implement our own retry mechanism for a very similar API call because retries aren't provided.
https://googleapis.github.io/google-cloud-python/latest/bigtable/table.html#google.cloud.bigtable.table.Table.read_row
https://googleapis.github.io/google-cloud-python/latest/bigtable/table.html#google.cloud.bigtable.table.Table.read_rows
Describe the solution you'd like
table.Table.read_rows accepts a Retry, but table.Table.read_row does not. It would be nice if table.Table.read_row accepted the same retry mechanism.
Describe alternatives you've considered
In the absence of a retry mechanism built into read_row, we have written our own.

Wrong project!  https://github.com/googleapis/google-cloud-python/issues/7955


Improve Kuzu Cypher generation prompt

Thank you for contributing to LangChain!

[x] PR title: "community: improve kuzu cypher generationprompt"

[x] PR message: Delete this entire checklist and replace with

Description: Improves the Kùzu Cypher generation prompt to be more robust to open source LLM outputs
Issue: N/A
Dependencies: N/A
Twitter handle: @kuzudb

[x] Add tests and docs: If you're adding a new integration, please include
No new tests (non-breaking. change)

[x] Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Hi @baskaryan, just wanted to say that I work at Kùzu and will be taking over maintenance of the Kùzu graph LangChain functionality from @mewim - feel free to tag me in any future PRs that address Kùzu.

Tested the example workflow for Kùzu with two models:
OpenAI GPT-4
> Entering new KuzuQAChain chain...
Generated Cypher:
MATCH (p:Person)-[:ActedIn]->(m:Movie) WHERE m.name = 'The Godfather: Part II' RETURN p.name
Full Context:
[{'p.name': 'Al Pacino'}, {'p.name': 'Robert De Niro'}]

> Finished chain.
Al Pacino and Robert De Niro played in The Godfather: Part II.

Mistral Instruct v0.2 7B, Q8_0 GGUF
> Entering new KuzuQAChain chain...
Generated Cypher:

MATCH (p:Person)-[:ActedIn]->(m:Movie)
WHERE m.name = "The Godfather: Part II"
RETURN p.name

Full Context:
[{'p.name': 'Al Pacino'}, {'p.name': 'Robert De Niro'}]

> Finished chain.
 Al Pacino and Robert De Niro both acted in The Godfather: Part II.


be smarter about adding nodes for IDE when symbolizing classes

Motivation

Stop doing work we don't need to do.  We could probably be even smarter and only add these when --lsp hasn't been passed, but I'm not sure that's worth it.
Also, this combined with #6274 speeds things up enough that adding #6265 into the mix becomes ~performance-neutral with respect to current master.
Test plan

See included automated tests.

We could probably be even smarter and only add these when --lsp hasn't been passed, but I'm not sure that's worth it.

I don't think we should have substantial parts of Sorbet's pipeline that are only active when LSP is active. We get a lot of mileage in terms of being able to debug things by having the batch and incremental pipelines not be different.


Field value expressions (aka formula fields) and custom validators

Value expressions implemented.
This means:

ability to dynamically calculate/generate values (a.k.a. formula fields)
sanitizers (how values are cleaned BEFORE validation)
validators (with errors)
formatters (how data is reformatted before it is stored into the db)

There is a high pos. that we'll remove formatters from UI before merging.

New previews of validation/expression settings for field:

Added help page for field expression.
Here's a screenshot, but it's best if you click through it yourself:

Almost there:

open this with window.open() and extra params for width & height (eg 650x900px)
we need a bigger button/link, maybe something that is part of the input group (if f is on the leftside, a ? could be on the right?
not sure if you noticed but there is a typo in parseISOTime example one has extra )
example could also be inside  tag, right?
can we make table a bit more narrow? do we really need the "name" col? can others be removed/merged/reordered?

TODO:

Add text to translations

Added link to expression help page

Modified validation tab

Looks good enough. Merge please. ty.

Merged into 2020.12.x


thead, tbody and tfoot in separate table tags

I'm trying to produce minimal working example using ember-light-table
Somehow thead, tbody and tfoot renders in separate table tags

here is the link for repo: https://github.com/chilicoder/ember-light-table-example
at the same time tables in document site renders in one table.
Can you please show where I'm wrong?
PS My Ember version in 2.11

@chilicoder that is intended behavior. What exactly is the problem you're having?

now i have 3 different table tags. And it should be one. Smth like
<table>
<thead>
</thead>
<tbody>
</tbody>
<tfoot>
</tfoot>
</table>

@chilicoder believe it shouldn't be one table ... just because you want fixed (sticky) header and\or footer ... with horizontal scrolling maybe and in the end you'll come to the solution with separate tables for header, body and footer (the only reliable solution nowadays ...)

I have run into this too. My only issue (this far) is that I cannot style using bootstrap. If set tableClassNames to e.g table table-hover, bootstrap treats every separate <table> element as a different table.
This means that if you set tableClassNames on the {{light-table}} component, you will have a weird spacing after the header (see attached image) and whatever else bootstrap uses on the last row of a table & after the table.
If you set tableClassNames on the {{table.body}} component, you need to style the header manually.
Not sure if this is an issue or expected behaviour, I was led here from a google search & I thought my experience might help

My only issue (this far) is that I cannot style using bootstrap.
Same for me

I don't need fixed columns and horizontal scrolling (I think it is possible to implement this without having 3 table tags, but I did zero research on it), but I need to style the table using bootstrap theme.
anyway, looks like it is not the bug, but a feature. Because I saw the same in examples.

If set tableClassNames to e.g table table-hover, bootstrap treats every separate table element as a different table. This means that if you set tableClassNames on the {{light-table}} component, you will have a weird spacing after the header

I think some basic custom CSS targeting .ember-light-table table will resolve your issue here.
As of right now, the best current solution to support all the requested features is unfortunately using the 3 table method.


freeModeSticky doesn't clear it's timeout function when swiper instance is destroyed

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.5.0

Platform/Target and Browser Versions: macOS, Windows, CORDOVA, IOS, ANDROID, CHROME, FireFox, Safari.

What you did
Running Swiper in an SPA and with freeMode: true, freeModeSticky: true. Swiping to scroll swiper sometimes triggers a page Back navigation which results in the Swiper instance being destroyed/cleaned up.
Expected Behavior
Swiper is destroyed and everything runs on as expected.
Actual Behavior
Swiper crashes due to freeModeSticky's setTimeout function that calls slideToClosest(). That function tries to access this.params.speed which doesn't exist anymore since it was destroyed.
Looks like this is another instance of https://github.com/nolimits4web/swiper/issues/1189 but for freeModeSticky.
handler is in components/mousewheel/mousewheel.js line 170

Would be good to see live example or JSFiddle with the issue


Add some missing docs

These docs complete the dartdocs for their respective libraries.

@chinmaygarde

lgtm

After this lands, I will get rid of the tick on the _Symbol.


data added via node.files.add in browser not available elsewhere

I am running your browser example adding some data node.files.add but the resulting hash is not available elsewhere (e.g. local node or gateway). This is what I tried:

I downloaded the html file locally and opened it in Chrome (latest Chrome on Ubuntu 16.04).
I opened the browser console via Ctrl+Shift+j and copy pasted the commands for adding a buffer (node.files.add(new node.types.Buffer('Hello world!'),.....
I get a resulting IPFS hash that I can retrieve again via ipfs files cat.
I take the hash and try getting it via the gateway but just get nothing (loading forever) and the same happens with my local go-ipfs client when attempting to ipfs cat that hash (daemon running in separate terminal window). Just FYI ipfs add and ipfs cat is working well from my bash terminal using go-ipfs.

What am I doing wrong?

Hey @SCBuergel just to make sure there is no misunderstanding, are you looking to use js-ipfs, the JavaScript Implementation of IPFS, or js-ipfs-api, the HTTP Client library to talk with IPFS daemons?

Hi again @SCBuergel. I've just updated both examples browserify and webpack to make them simpler to read through since we simplified our Init process a ton. I can confirm they are fully functional. Hope it helps :)

You got me! I posted this issue in the wrong repo - as my link to the example was suggesting, I was indeed using js-ipfs.
Thanks for updating the examples. The one that I was referring to (using script-tag and pulling ipfs from unpkg: https://github.com/ipfs/js-ipfs/blob/master/examples/browser-script-tag/index.html ) does not seem updated yet.
Am I even correct in assuming that this example is able to add content to IPFS (so that I can ipfs cat it, e.g. from another machine or gateway)? That would mean that my browser is connecting via some libp2p to another node and maybe event keeping data locally - which sounds crazy low-level to work in a browser...

Am I even correct in assuming that this example is able to add content to IPFS (so that I can ipfs cat it, e.g. from another machine or gateway)? That would mean that my browser is connecting via some libp2p to another node and maybe event keeping data locally - which sounds crazy low-level to work in a browser...

You are assuming it correctly, that is the whole goal and we are only missing a piece, Relay, which is now in Experimental/Testing phase, so soon :)
However, you can already work around it. Just do a ipfs.files.add in the browser and then make a request to one of the gateways so that they fetch the content themselves and then the other ipfs node you have locally with go-ipfs will be able to fetch it from the gateways. This won't be a solution in the long term as you should not need to rely in those points, but it can get a prototype working if you are working on an app :)


Clean up production after install

It's an enhancement but could be nice for clean installations.
After the installation is configured and working, run the following to prune dev/compile dependencies.
composer update --no-dev --no-scripts && rm -Rf node_modules bower_components npm-debug.log

Aside from removing a bunch of dev dependencies for disk space and security, this will also make a smaller composer autoload file so it's a little performance boost.
If you need to regenerate the frontend, like for an update from the git repository, you will need to install and later delete them again.
I'm not sure if you want to use this suggestion and how. Maybe it's for the wiki, maybe part of the install scripts... That's up to you.

Don't think this is needed actually.


Resolves #32 - Save model

Implements Save model on sequential

Just rebase this PR and I will merge it.


BUG Don't calculate levels if we aren't expanding

If a user has excluded a column because they know that column has too many levels, they shouldn't then get a warning about that column. Make sure we don't count the levels in columns which are going to be dropped anyway. This fix also makes the _cols_to_expand attribute make more sense -- it no longer contains columns which we aren't going to expand.

I noticed this but I had not had time to report the bug. Thanks for the fix!


Satoshi unit on telegram bot messages

What about showing the amount in satoshi instead of bitcoin?
Current
💰 Forwarded 0.00100002 NodeA → NodeB. Earned 0.00000007 0.01% (60)
⛓ (pending) Received 0.01069142
Suggested
💰 Forwarded 100,002 sats NodeA → NodeB. Earned 7.202 sats 0.01% (60)
⛓ (pending) Received 1,069,142 sats

It is shown as satoshis, just ignore the zeros to the left

I think it's something you can get used to if it's consistent

I actually agree with OP, for what it's worth, arguments being:

readability
less chars in TG messages
future proofing, as whole BTCs will be more and more rare (already are not a thing in LN anyway) while msats will be more and more worthy

Just flying opinions.

@asvdf I followed your advice and made a fork, for now I changed only the "forwarded" message.
Anyway I think that a better approach can be add a parameter to the telegram command (something like --use-satoshis) so anyone can choose is preference.
@alexbosworth it's fine for you if I try to make a pull request for that?

sure

Anyway I think that a better approach can be add a parameter to the telegram command (something like --use-satoshis) so anyone can choose is preference.

this is actually a smart approach! makes everyone happy

@alexbosworth thank's for having fast merged the PR!
Do you think that I can propose a new one to show even the millisats on the "forwarded" message when the --use-small-units option is set?
Some users on PLEBNET, even me, ask for that

sure ok

@alexbosworth thank's for the fast merge, as usual!
Do you forgot to bump up the ln-telegram/balanceofsatoshis version or just waiting for something else?

Ah I thought maybe you were going to put up a PR for that, I can look at it in a bit though


question about sub-collections in "node-red-contrib-cloud-firestore"

Hello, I'm a beginner in node-red and I'm using the "node-red-contrib-cloud-firestore" and I have a question that I can't find anywhere. In my firestore, I have a collection with two documents and within those documents I have another collection in both.

I would like to receive data from the entire collection, with the document and the sub-collections. I know that I can receive by defining the collection on each node as: "place / House / user", but I would like to receive all data from "House" and "Mom house".
Is there a way to do this?
Thank you!

Hi,
The limitation is on Firebase by design: you're not supposed to do that.
But you have 2 way to bypass this:

If you always want to get those data together, do not use a sub-collection but a array or a map as value type.
If you want those collections to be independent (as sub-collections), you'll need to loop. First, loop on the level 1 collection and for each key, get all the values in the linked collection.

I use both solutions, depending on constraints and usage. Choose wisely :wink:

The second way seems to solve my problems, thank you very much.
I'm really a beginner in node-red, can you give me any tips on how to do it?
Thank you again.


bug: LSP crash with starklings, prelude is not a core submodule.

Bug Report
Cairo version: 2.5.3
Current behavior:
LSP worker panic with Starklings-cairo1 exercices.
thread 'main' panicked at /cargo/registry/src/index.crates.io-6f17d22bba15001f/cairo-lang-semantic-2.5.3/src/corelib.rs:57:28:
`prelude` is not a core submodule.

Expected behavior:
Working like when I create a new project
Steps to reproduce:
Install vscode, scarb stable, add cairo 1.0 extension, clone starklings-cairo1 and open the project.
Other information:

it is a configuration issue - please open an issue at starklings - not here.


Continuous logging and unhandled exception

In your package, I tried to console log to check if the status code is equals to 200, and the result is that it's continuous log the print function.
In my use case, I have integrated your package to monitor changes in internet status, triggering UI components such as dialogs and snack bars through the whole app by using navigator context. But the problem is that the code won't stop after timeout. In that case, it would lead to an exception: [ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: 'package:flutter/src/widgets/navigator.dart': Failed assertion: line 4516 pos 12: '!_debugLocked': is not true.
Am I missing something while using your package?

Yo @dxgiangdev3, Can you elaborate on the issue and also share code snippets (Minimal Reproducible Sample) and also a video of the error.

any updates about the issue? it keeps throwing exceptions and showing that the network is unreachable , wouldn't it be better to handle it and just return false instead of throwing it.

@84rrry, are you using custom addresses to check network connectivity or the default ones?

@OutdatedGuy, both using custom and default addresses

it keeps throwing exceptions and showing that the network is unreachable

Can you provide stack trace and error logs, cuz I have already implemented exception handling when trying to connect to a address.

@OutdatedGuy  Hello, sorry for the late reply it's hard to identify the exact problem or to get the full stack trace but this should help you identify the problem

@dxgiangdev3 @84rrry on what Flutter version are you guys on?

Looks like the issue is with the latest Dart SDK that comes with Flutter 3.13: https://github.com/dart-lang/sdk/issues/53334

@dxgiangdev3 @84rrry on what Flutter version are you guys on?

@OutdatedGuy  i'm using
- Flutter 3.10.3
- Dart 3.0.3

Looks like the issue is with the latest Dart SDK that comes with Flutter 3.13: dart-lang/sdk#53334

Yep i think you're right , dart-lang/sdk#53331 is related too ,  i'll attempt to downgrade temporarily if possible hoping that this will be fixed soon

@OutdatedGuy  Hello , can you please reopen this as the issue still persist

This was the call stack:
_ConnectionTarget.connect.<anonymous closure>.<anonymous closure> (dart:_http/http_impl.dart:2533)
_RootZone.runUnary (dart:async/zone.dart:1661)
_FutureListener.handleError (dart:async/future_impl.dart:174)
Future._propagateToListeners.handleError (dart:async/future_impl.dart:852)
Future._propagateToListeners (dart:async/future_impl.dart:873)
Future._completeError (dart:async/future_impl.dart:649)
Future._asyncCompleteError.<anonymous closure> (dart:async/future_impl.dart:739)
_microtaskLoop (dart:async/schedule_microtask.dart:40)
_startMicrotaskLoop (dart:async/schedule_microtask.dart:49)

@84rrry can you provide output of flutter doctor -v and flutter run -v

Hey @OutdatedGuy  , sorry i got a lil busy and lost track of the notification . here's the flutter doctor -v output :
`[√] Flutter (Channel stable, 3.13.9, on Microsoft Windows [Version 10.0.22621.2506], locale en-US)
• Flutter version 3.13.9 on channel stable at C:\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision d211f42860 (8 days ago), 2023-10-25 13:42:25 -0700
• Engine revision 0545f8705d
• Dart version 3.1.5
• DevTools version 2.25.0
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
• Android SDK at C:\Android
• Platform android-33, build-tools 33.0.2
• ANDROID_HOME = C:\Android
• Java binary at: C:\Program Files\Java\jdk-<IP_ADDRESS>\bin\java
• Java version Java(TM) SE Runtime Environment (build <IP_ADDRESS>+1-1)
• All Android licenses accepted.
[!] Android Studio (not installed)
• Android Studio not found; download from https://developer.android.com/studio/index.html
(or visit https://flutter.dev/docs/get-started/install/windows#android-setup for detailed instructions).
[√] VS Code (version 1.84.0)
• VS Code at C:\Users\Barry\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.76.0
[√] Connected device (1 available)
• Pixel 6a (mobile) • 29061JEGR11086 • android-arm64 • Android 14 (API 34)
[√] Network resources
• All expected network resources are available.
! Doctor found issues in 1 category.`

Weird issue, will check up on it

@OutdatedGuy  It's pretty annoying, I had to implement my own alternative .  However i'll happily provide you with further information/help

Can you provide the crash logs while running the command flutter run -v.

@OutdatedGuy , I wanna inform you that the issue stopped happening after updating flutter to the latest version here's  my flutter doctor -v output:
[√] Flutter (Channel stable, 3.16.0, on Microsoft Windows [Version 10.0.22621.2715], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.2)
[!] Android Studio (not installed)
[√] VS Code (version 1.84.2)
[√] Connected device (1 available)
[√] Network resources
! Doctor found issues in 1 category.
I think you it's safe to go ahead and close the issue , as i can't reproduce it anymore at least on my end .


Update puppeteer.ts

Added flag in browser to bypass hcaptcha detection

Sure, I'll try it 🙂

Turns out this is already included in the stealth plugin, so this didn't change anything really: https://github.com/berstend/puppeteer-extra/blob/master/packages/puppeteer-extra-plugin-stealth/evasions/navigator.webdriver/index.js#L41

when I tested it without this argument, it fails to solve the captcha.  it was detecting that there was an automation in the browser


Azure FrondDoor: Spec incorrect or Service API misbehaving

Hi,
I'm using the Go SDK for Azure here https://github.com/lawrencegripper/azurefrontdoor-ingress to talk to Azure Frontdoor.
I see errors when using any of the individual resource clients. An InvalidResourceType response is returned when attempting to update/get or list the following:

/Microsoft.Network/Frontdoors/lgtestfrontdoor/LoadBalancingSettings
/Microsoft.Network/Frontdoors/lgtestfrontdoor/backendPools
/Microsoft.Network/Frontdoors/lgtestfrontdoor/healthProbeSettings

Here is a quick example, the behavior isn't limited to the Go SDK as attempting to browse these resources using resources.microsoft.com also returns the same errors.
curl 'https://resources.azure.com/api/operations' -H 'User-Agent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:62.0) Gecko/20100101 Firefox/62.0' -H 'Accept: application/json, text/plain, */*' -H 'Accept-Language: en-GB,en;q=0.5' --compressed -H 'Referer: https://resources.azure.com/subscriptions/your-sub-id-here/resourceGroups/temp-frontdoortest/providers/Microsoft.Network/frontDoors/lgtestfrontdoor/frontendEndpoints' -H 'Content-Type: application/json;charset=utf-8' -H 'Cookie: [object Object]' -H 'DNT: 1' -H 'Connection: keep-alive' -H 'Pragma: no-cache' -H 'Cache-Control: no-cache' --data '{"Url":"https://management.azure.com/subscriptions/your-sub-id-here/resourceGroups/temp-frontdoortest/providers/Microsoft.Network/frontDoors/lgtestfrontdoor/routingRules","HttpMethod":"GET","ApiVersion":"2018-08-01"}'

Response
HTTP/1.1 400 Bad Request
Cache-Control: private
Content-Length: 107
Content-Type: application/json; charset=utf-8
Content-Language: en-US
Server: Microsoft-IIS/8.5
Strict-Transport-Security: max-age=31536000; includeSubDomains
x-ms-ratelimit-remaining-subscription-reads: 11998
x-ms-request-id: 2ae5e8a7-b232-474a-bb52-3720abbb570b
x-ms-correlation-request-id: 2ae5e8a7-b232-474a-bb52-3720abbb570b
x-ms-routing-request-id: WESTUS:20181011T212358Z:2ae5e8a7-b232-474a-bb52-3720abbb570b
X-Content-Type-Options: nosniff
X-AspNet-Version: 4.0.30319
X-MS-Ellapsed: 392ms
Request-Context: appId=cid-v1:a7a0aa45-52ad-43f4-a8c4-8896f1392829
instance-name: explorer-bay
X-Powered-By: ASP.NET, ASP.NET
Date: Thu, 11 Oct 2018 21:23:58 GMT

{
  "error": {
    "code": "InvalidResourceType",
    "message": "The resource type is invalid."
  }
}

Sample from GOSDK with HTTP Traces on

INFO[0001] PUT /subscriptions/your-sub-here/resourceGroups/temp-frontdoortest/providers/Microsoft.Network/Frontdoors/lgtestfrontdoor/LoadBalancingSettings/loadBalancingSettings-1539281898722?api-version=2018-08-01 HTTP/1.1
Host: management.azure.com
User-Agent: Go/go1.10.1 (amd64-linux) go-autorest/v10.15.5 Azure-SDK-For-Go/v21.1.0 frontdoor/2018-08-01-preview
Content-Length: 95
Authorization: Bearer *removed*
Content-Type: application/json; charset=utf-8
Accept-Encoding: gzip

{"properties":{"sampleSize":4,"successfulSamplesRequired":2,"additionalLatencyMilliseconds":0}}

INFO[0001] HTTP/1.1 400 Bad Request
Content-Length: 107
Cache-Control: private
Content-Language: en-US
Content-Type: application/json; charset=utf-8
Date: Thu, 11 Oct 2018 20:28:41 GMT
Server: Microsoft-IIS/8.5
Strict-Transport-Security: max-age=31536000; includeSubDomains
X-Aspnet-Version: 4.0.30319
X-Content-Type-Options: nosniff
X-Ms-Correlation-Request-Id: 054cedff-75ed-4d27-a954-76f58ed1743b
X-Ms-Ratelimit-Remaining-Subscription-Writes: 1199
X-Ms-Request-Id: 054cedff-75ed-4d27-a954-76f58ed1743b
X-Ms-Routing-Request-Id: WESTUS:20181011T202842Z:054cedff-75ed-4d27-a954-76f58ed1743b
X-Powered-By: ASP.NET

{
  "error": {
    "code": "InvalidResourceType",
    "message": "The resource type is invalid."
  }
}

According to https://github.com/Azure/azure-rest-api-specs/blob/5682ce2525b7fa4b3115938a948af3bc46d6387f/specification/frontdoor/resource-manager/Microsoft.Network/preview/2018-08-01-preview/frontdoor.json#L947 it should work

Thanks @sabbour glad to know its not just me missing something. @marstr Are they're anymore details you need on this one or is it case of waiting for a response from the frontdoor team now?

Waiting on the frontdoor team. They should get looped in automatically with the labels I just added.

Removing my assignment because this issue is triaged

@sabbour I found a workaround but it doesn't seem to behave quite right for my scenario. However, it may be useful to you, in this PR I move to using the FrontDoorsClient not FrontDoorClient this allows you to GET and PUT the entire Frontdoor state in one call rather than talking to the subresources. I talk about the issue I see with stale reads in the PR details.

Hi, Any update on this issue?

@marstr has this been seen by the frontdoor team?

I'm not sure of the current status. Looks like the current assignees are Network team members. I've been transitioned onto a different team, so I'm not in the day-to-day loop here.

Ah sorry missed the assignees, @gmainar / @jmelvinwork do you have any update on this Azure Front Door API issue?

I have moved to another team. Please follow up with @johncrane .

Is the Front Door team not using Github? Could we please get some attention to this issue, it's blocking a welcomed implementation of an Ingress controller for Kubernetes.

Hello, this issue has been recently brought to the attention of the Front Door team.
Front Door recently corrected their SDK and documentation to remove CRUD on these sub-resources. Updates to Front Door must be invoked on the entire Front Door object.
Unfortunately, the old SDK generated with these API will continue to have these sub-fields, causing confusion among customers.

How can this API support writing from multiple user? Say you have two machines which are updating the same frontdoor instance, as they have to put the whole object it's likely they'll overwrite each other's changes.
Do you support an etag or updateif or patch?

You are correct. If you have two machines updating the same front door instance, the update operations will happen in series and the last writer will win.
We currently do not support etag or updateif or patch.

I had a bit of a play and this is likely to effect users who make updates in the portal too.
When I do an update in the portal is PUTs the whole frontdoor resource. If I've edited routes and another admin edits backend rules we'd trample on each others changes rather than see an error. Feels like that is going to hurt someone.

Are there any plans to bring back CRUD functionality to these subresources? We currently have several microsites each owning their own IAC. In the current scenario they cannot independently add their own backend pools/routes/front ends so instead we are forced to have a central IAC repo for front door that must be kept up to date with knowledge of all products which is quite counter productive.

This is a huge blocker for my team when thinking of using this product.  Is there any plan to have machines register independently to frontdoor?


Collect note offset for programmatic scrolling

This is to enable library user to collect note view offset so that they can programmatically scroll to a note.
I tried vanilla ScrollView and ScrollViewReader, however it failed to scroll to the correct note view with anchor id. Therefore I switched to scroll with offset.
Demo code (not included in the PR as I'm using a 3rd party library SolidScroll in order to scroll to certain offset):
// Copyright AudioKit. All Rights Reserved. Revision History at http://github.com/AudioKit/AudioKitUI/

import PianoRoll
import SwiftUI
import SolidScroll // add https://github.com/edudnyk/SolidScroll package into demo project for this demo to work

public struct PianoRollDemoView: View {
    @State var model: PianoRollModel
    @State var scrollViewProxy: SolidScrollViewProxy?

    let notes = [
        PianoRollNote(start: 0, length: 2, pitch: 100),
        PianoRollNote(start: 4, length: 1, pitch: 120),
    ]

    @State var offsetInfos: [NoteOffsetInfo] = []

    public init() {
        self._model = .init(initialValue: .init(notes: notes, length: 128, height: 128))
    }

    public var body: some View {
        VStack {
            Button("Scroll") {
                guard let scrollViewProxy = scrollViewProxy, let offset = offsetInfos.first?.offset else { return }
                let contentOffset = CGPoint(x: offset.width, y: offset.height)
                scrollViewProxy.setContentOffset(contentOffset, animated: true, completion: { completed in
                  print("Scrolled via proxy to \(contentOffset)! Completed: \(completed)")
                })
            }
            .padding(.top, 10)
            SolidScrollView([.horizontal, .vertical], showsIndicators: true) {
                PianoRoll(model: $model, noteColor: .cyan)
            }
            .onPreferenceChange(ContainedScrollViewKey.self) {
                scrollViewProxy = $0
            }
            .onPreferenceChange(NoteOffsetsKey.self) {
                offsetInfos = $0
            }
        }

    }
}

struct DemoView_Previews: PreviewProvider {
    static var previews: some View {
        PianoRollDemoView()
    }
}

https://user-images.githubusercontent.com/4270232/197404948-63b2db0f-5f54-416c-91ff-b25ca51bd321.mp4

@aure conflicts resolved, please check.

Your release is 1.0.4. If we end up doing any of things differently, you may want to follow along or pin the package reference to this version.

Your release is 1.0.4. If we end up doing any of things differently, you may want to follow along or pin the package reference to this version.

Hi @aure, I notice that the tag is named 1.04 not 1.0.4. Would you please fix this? Thanks.


pressing up arrow only does the expected thing once

Input:
» echo "hi"
hi
» echo "bye"
bye

type in "echo"
press up arrow once: expected result is echo "hi".  Pass
press up arrow again.  expected result is echo "bye".  Actual result: something random.  :(
Press up arrow a few more times: a random walk through history.

In fish, the result would be an orderly series of results which start with "echo"

This is a known bug, currently the repo is strongly in need of someone to jump in and do some serious coding - want the job?

Point taken!  I don't know if I'm the one, though.  :-/

The wait continues.

Just a hint: I’m using autosuggestions together with zsh-history-substring-search plugin and it works correctly here.


Concurrency control max limit has changed from 100 in v1 to 50 in v3

Describe the Bug with repro steps

What type of Logic App Is this happening in?
Consumption (Portal)
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
No response
Browser
Chrome
Additional context
No response

Did some investigation and this looks like we're just not adjusting the limits when an operation is a trigger.
We have the proper limits defined for both action and trigger but we are only using the action limits.


fix: log forwarder server upgrade order

This will make sdpctl upgrade LogServers/LogForwarders after upgrading all controllers and before the rest of the appliances.
Fixes:

SA-19393

E2E: https://got-testrail.agi.appgate.com/index.php?/plans/view/1025689


createDisplayMode = 'row' not working with enableRowVirtualization = true

material-react-table version
v2.0.6
react & react-dom versions
v18.2.0
Describe the bug and the steps to reproduce it
The table enable rowVirtualization = true and createDisplayMode = 'row'. This row not showing on table.
Minimal, Reproducible Example - (Optional, but Recommended)
const table = useMaterialReactTable({
    columns,
    data: fetchedUsers,
    createDisplayMode: 'row',
    editDisplayMode: 'row',
    enableEditing: true,
    enableRowVirtualization: true,
    getRowId: (row) => row.id
)}

Screenshots or Videos (Optional)

Do you intend to try to help solve this bug with your own PR?
None
Terms

[X] I understand that if my bug cannot be reliably reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

Yeah this is a current incompatibility. Open to suggestions or fixes

@KevinVandy , I think we can split that component "creatingRow" into a TableBody. Do you think that's okay?
  {creatingRow && createDisplayMode === 'row' && (
        <TableBody
          {...tableBodyProps}
          sx={(theme) => ({
            display: layoutMode?.startsWith('grid') ? 'grid' : undefined,
            position: 'sticky',
            top: tableHeadHeight - 1,
            zIndex: 1,
            ...(parseFromValuesOrFunc(tableBodyProps?.sx, theme) as any),
          })}
        >
          <MRT_TableBodyRow row={creatingRow} rowIndex={-1} table={table} />
        </TableBody>
  )}


reviews showing pagenumbers when setting firstshort

I encountered another unexpected output for @review entries when I set the option firstshort=true. When you cite them it shows the page number of the article. In subsequent citations, this is gone.

Actually this also seems to happen to articles when using \footcite with optional fields. So \footcite[See, e.g.][]{source} gives

See, e.g. Source, 1--20

Hello again!
Sorry for the delay. Things are a bit hectic here.
I may need to see an example, but it seems you've encountered expected output in those cases. If the postnote field of an @article or @review entry is blank, say, if you haven't cited a particular page, the style always prints the full page range on the first citation. Here's the macro responsible (see windycity.cbx):
\newbibmacro*{cite:pages}{%
  \iffieldundef{pages}
    {}
    {\ifboolexpr{ test {\ifentrytype{article}}
                  or test {\ifentrytype{review}}}
       {\iffieldundef{postnote}
          {\printtext[bibhyperref]{\printfield{pages}}}
          {}}
       {}}}

I put a note about this issue right before it:

If the postnote is empty, print the page ranges of journal articles. See CMOS, 17th ed., 14.153, 14.167, and 14.174. As for
incollection and inbook entries, 14.106 seems to suggest the same, but the example in 14.30 suggests otherwise. The
examples in 14.108 only muddy the waters further. For magazines, CMOS is clear that the range can be left out. To do so,
simply omit it from your bibliography database.

I should really edit those comments to make them clearer. Ugh!
But in short, it seems plausible to me to print pages for some works in some contexts. It might help to make this an optional feature. As a workaround, though, you could edit the macro.
I'll look at your other open issue soon. Again, sorry for the delay!

Thanks for the reply, I never noticed that this was the default behaviour in CMOS. Thanks for pointing out where to look in the code, I'll see if I can edit the macro to get the behaviour I want.

If you never want pages to print on the first citation, it's best to remove cite:pages from postenote:first (also in windycity.cbx). This is what you'd need:
\newbibmacro*{postnote:first}{%
  \ifboolexpr{ test {\iffieldundef{postnote}}
               and test {\iffieldundef{pages}}
               and test {\iffieldundef{url}}}
    {\usebibmacro{doi+url+etc}}%
    {\usebibmacro{postnote}%
     \usebibmacro{doi+url+etc}}}


Summonerd: Figure out how to reuse Aleo's phase1 output

Regardless of how we actually want to run the two phases, having them as a starting point for phase1 provides a good amount of additional security and trust, and so reusing them would be nice.
This involves a bit of code-assisted reverse engineering / spelunking, but maybe a bit of code shimming on our end, so I feel like tracking this here is good, even if the work may not result in a PR per se.
As a concrete goal, we should be able to massage their transcript output file into our own phase1 input file. Note that we'll have to copy the thing 7 times: once for each circuit.

Draft code to load the Aleo ceremony output and convert: https://github.com/AleoHQ/aleo-setup/compare/master...penumbra-zone:aleo-setup:master

So, after writing property tests to figure out the origins of our failure to convert the output of their phase1 to the structure our own, we realized that their phase1 actually used a different generator point for their Bls12-377 implementation:
https://github.com/AleoHQ/snarkVM/commit/21c7859a4835e9965534e5740401a96bc76661e4
The original Zexe paper didn't specify a generator, although many libraries such as GNark, Arkworks, etc. have standardized on a particular generator we use now. However, this is different than the one used by Aleo back when they ran the ceremony.
Math you can ignore
The problem is that they have a generator H with an unknown relation to our generator G, i.e. H = w * G for some unknown w. (Presumably, we can't find this w either).
This means that something we expect to be of the form x * G, x^2 * G, ... is actually w * x * G, w * x^2 * G, ..., which doesn't match our expectations, and thus will fail validation as a valid phase output, and ultimately as a proving key when we try and construct one from the output.
A theoretical way to get around this
We could get around this by modifying our implementation of proof creation and verification to use a different generator.
However, this makes interoperability with Penumbra more complicated, because all consumers of transactions have to be aware of this different verification method for proofs.
Proposal
Running our own phase1 is a one time pain and annoyance, whereas using a different generate point than the rest of the ecosystem is an enduring pain and annoyance. Because of that, I propose that we switch gears and plan on running our own phase1.


CIS 22H2 list availability

Hi there,
Just working on a Windows 11 22H2 machine today. Any chance you have this CIS list for 22H2? Unless you think 21H2 would suffice....
finding_list_cis_microsoft_windows_11_enterprise_21h2_machine.csv
Thanks

Hi @Cyb3rC3lt
To build/update a CIS list is a piece of work (comparing the PDF with my lists) but I did it now for all current Windows releases and the new versions of CIS. The lists are now in the development repo, here: https://github.com/0x6d69636b/windows_hardening/commit/09858da476d339dc0957f7c79ebffd42161edea1

Thanks very much Michael, that is very useful. Much appreciated.


Integrate cab controls with Loco Lights Mod

Interested to learn what would be involved here, and if I can help. Let me know! 🙂

With the updates to the cab input handling, it's unclear if direct integration with loco lights would be necessary, since that mod deals with existing locos, and custom locos could be equipped with working lights via CCL scripts. Perhaps a new type of indicator component that uses a light as its output, this can then be hooked to cab controls using the existing CCL event system.

Shouldn't be any compatibility issues going forward. CCL now has native headlight components & controls.


[KYUUBI #2104] Kill yarn job using yarn client API when kyuubi engine …

…initialization times out and yarn application status is accepted

Why are the changes needed?

How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[x] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

thanks @942011334  for the contibution, also cc @wForget @SteNicholas @pan3793

good idea .
------------------ 原始邮件 ------------------
发件人: "Kent @.>;
发送时间: 2022年3月12日(星期六) 下午4:34
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [apache/incubator-kyuubi] [Kyuubi#2104] Kill yarn job using yarn client API when kyuubi engine … (PR #2110)
@yaooqinn commented on this pull request.
In kyuubi-server/pom.xml:
> @@ -385,6 +385,13 @@              <scope>test</scope>          </dependency>   +        <dependency> +            <groupId>org.mockito</groupId> +            <artifactId>mockito-core</artifactId> +            <scope>test</scope> +            <version>3.4.6</version>
can we add it to dependencyManagement then inherit it here？
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Please help to update the title to [Kyuubi #2104] Kill yarn job using yarn client API when kyuubi engine ….

Codecov Report

Merging #2110 (28665f3) into master (7d66e9a) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

:exclamation: Current head 28665f3 differs from pull request most recent head 2db9c45. Consider uploading reports for the commit 2db9c45 to get more accurate results

@@             Coverage Diff              @@
##             master    #2110      +/-   ##
============================================
- Coverage     60.70%   60.68%   -0.03%
  Complexity      108      108
============================================
  Files           320      320
  Lines         15468    15465       -3
  Branches       1989     1987       -2
============================================
- Hits           9390     9385       -5
- Misses         5273     5276       +3
+ Partials        805      804       -1

Impacted Files
Coverage Δ

...ache/kyuubi/engine/spark/SparkProcessBuilder.scala
88.23% <100.00%> (+1.56%)
:arrow_up:

...ache/kyuubi/operation/KyuubiOperationManager.scala
92.50% <0.00%> (-2.50%)
:arrow_down:

...rg/apache/kyuubi/engine/trino/TrinoStatement.scala
67.46% <0.00%> (-2.41%)
:arrow_down:

.../org/apache/kyuubi/operation/KyuubiOperation.scala
61.33% <0.00%> (-1.34%)
: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 7d66e9a...2db9c45. Read the comment docs.

cc @cxzl25

hello , when use scalatestplus-mockito instead ,"build/dependency.sh" test failed, help me to see the problem , tanks!
------------------ 原始邮件 ------------------
发件人: "Kent @.>;
发送时间: 2022年3月12日(星期六) 晚上9:14
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [apache/incubator-kyuubi] [KYUUBI #2104] Kill yarn job using yarn client API when kyuubi engine … (PR #2110)
@pan3793 commented on this pull request.
In kyuubi-server/pom.xml:
> @@ -385,6 +385,13 @@              <scope>test</scope>          </dependency>   +        <dependency> +            <groupId>org.mockito</groupId> +            <artifactId>mockito-core</artifactId> +            <scope>test</scope> +            <version>3.4.6</version>
Please use scalatestplus-mockito instead.
<properties>     <scalatestplus.version><IP_ADDRESS></scalatestplus.version> </properties> <dependency>     <groupId>org.scalatestplus</groupId>     <artifactId>mockito-3-4_${scala.binary.version}</artifactId>     <version>${scalatestplus.version}</version> </dependency>
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

hello , when use scalatestplus-mockito instead ,"build/dependency.sh" test failed, help me to see the problem , tanks!
…
------------------ 原始邮件 ------------------ 发件人: "Kent @.>; 发送时间: 2022年3月12日(星期六) 晚上9:14 收件人: @.>; 抄送: @.>; @.>; 主题: Re: [apache/incubator-kyuubi] [KYUUBI #2104] Kill yarn job using yarn client API when kyuubi engine … (PR #2110) @pan3793 commented on this pull request. In kyuubi-server/pom.xml: > @@ -385,6 +385,13 @@ <scope>test</scope> </dependency> + <dependency> + <groupId>org.mockito</groupId> + <artifactId>mockito-core</artifactId> + <scope>test</scope> + <version>3.4.6</version> Please use scalatestplus-mockito instead. <properties> <scalatestplus.version><IP_ADDRESS></scalatestplus.version> </properties> <dependency> <groupId>org.scalatestplus</groupId> <artifactId>mockito-3-4_${scala.binary.version}</artifactId> <version>${scalatestplus.version}</version> </dependency> — Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android. You are receiving this because you were mentioned.Message ID: @.***>

You pinned <artifactId>mockito-3-4_${scala.binary.version}</artifactId> in compile scope, please take a look at my comments

yes, the purpose is to override this method for the test class .
------------------ 原始邮件 ------------------
发件人: "Kent @.>;
发送时间: 2022年3月13日(星期天) 中午1:20
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [apache/incubator-kyuubi] [KYUUBI #2104] Kill yarn job using yarn client API when kyuubi engine … (PR #2110)
@wForget commented on this pull request.
In kyuubi-server/src/main/scala/org/apache/kyuubi/engine/spark/SparkProcessBuilder.scala:
> @@ -44,6 +45,10 @@ class SparkProcessBuilder(      import SparkProcessBuilder._   +  val yarnClient = getYarnClient + +  def getYarnClient: YarnClient = YarnClient.createYarnClient
The name of getYarnClient method seems to be a little confused. It looks like the getter method of yarnClient variable.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were assigned.Message ID: @.***>

Thanks to every expert's guidance, support and help.
------------------ 原始邮件 ------------------
发件人: "Kent @.>;
发送时间: 2022年3月13日(星期天) 下午2:12
收件人: @.>;
抄送: @.>; @.>;
主题: Re: [apache/incubator-kyuubi] [KYUUBI #2104] Kill yarn job using yarn client API when kyuubi engine … (PR #2110)
@wForget commented on this pull request.
In kyuubi-server/src/main/scala/org/apache/kyuubi/engine/spark/SparkProcessBuilder.scala:
> @@ -44,6 +45,10 @@ class SparkProcessBuilder(      import SparkProcessBuilder._   +  val yarnClient = getYarnClient + +  def getYarnClient: YarnClient = YarnClient.createYarnClient
what about createYarnClient?
LGTM
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you were mentioned.Message ID: @.***>

thanks @942011334 for the first contribution, merged to master


Add CLA/Contributing document

Per Intel guidelines we don't need CLA, but we require use of the Developer Certificate of Origin (DCO)/“signed-off-by” convention -- we need CONTRIBUTING and CODE_OF_CONDUCT files that explain our contribution and community behavior requirements --- see here for an example: https://github.com/intel/amber-docs


Pattern advice: How to use config file with required field without setting it?

Hey hey,
I just read a few issues in this repo but so far I haven't been able to figure out if what I want to do is possible.
To me, one of the main advantages of reading configuration from a file is you can define exactly which fields are required and which are optional. This lets you shift a great amount of validation into the parsing layer.
Assume we have this function:
pub fn build_config(fname: PathBuf): Result<Config> {
    let file = File::open(fname)?;
    let reader = BufReader::new(file);
    let config: Self = serde_yaml::from_reader(reader)?;
    Ok(config)
}

This first code snippet here is much better than the second:
#[derive(Deserialize, Serialize)]
pub struct Config {
    postgres_connection_string: String,
}

fn main() -> Result<()> {
    let config = build_config("config.yaml")?;
    use_database(config.postgres_connection_string)?;
    Ok(())
}

#[derive(Deserialize, Serialize)]
pub struct Config {
    postgres_connection_string: Option<String>,
}

fn main() -> Result<()> {
    let config = build_config("config.yaml")?;
    use_database(config.postgres_connection_string.expect("postgres_connection_string was not set!"))?;
    Ok(())
}

Problem is, sometimes I want to declare a field in my config as not an Option, but I also don't want to read it from the config file sometimes (e.g. in some environments it's easier to read such fields as env vars). My first thought was Figment would be that magic solution. I have this code:
let config = Figment::new()
    .merge(Yaml::file(args.config_path))
    .merge(Env::prefixed("APP_"))
    .extract()
    .context("Failed to load config")?;

My assumption was this would happen:

Read config file.
Read env vars.
If I set a value in the env vars it would override what's in the config file, even if that value was missing.

Unfortunately this last part doesn't seem to be the case:

even if that value was missing

This code doesn't work:
APP_POSTGRES_CONNECTION_STRING='testing' cargo run --config-path config.yaml

I get this error:
Error: Failed to load config

Caused by:
    missing field `postgres_connection_string`

If the value is missing from the file then it doesn't matter that I set an env var for it. I thought perhaps there would be some kind of config "staging area" where Figment builds the final config, but it seems like maybe it just uses the underlying tool at each stage, so each config stage needs to be complete independently.
I have tried using figment_file_provider_adapter but that doesn't seem to help either (e.g. by using the _FILE env var suffix).
Is what I want to do possible with Figment?

Oh well well I actually just made a minimal example and it works how I want!
src/main.rs
use figment::providers::Env;
use figment::providers::Format;
use figment::providers::Yaml;
use figment::Figment;
use serde::Deserialize;
use serde::Serialize;

#[derive(Debug, Deserialize, Serialize)]
pub struct Config {
    postgres_connection_string: String,
}

fn main() {
    let config: Config = Figment::new()
        .merge(Yaml::file("config.yaml"))
        .merge(Env::prefixed("APP_"))
        .extract()
        .unwrap();
    dbg!(config);
}

config.yaml

$ APP_POSTGRES_CONNECTION_STRING=yo cargo run
[src/main.rs:19] config = Config {
    postgres_connection_string: "yo",
}

In reality my config is much more complicated and nested. I'll see if I can find which part of this causes it not to work.

Ha! I see, the issue was using an env var for a nested value.
 I found that using _ didn't work as I'd hope because it doesn't really know where to split stuff if the fields themselves have underscores in them.

I see some kind of curly brace syntax is supported but I can't quite seem to get it to work. Nonetheless, just using .split("___") is probably sufficient for me. Thanks!

I see some kind of curly brace syntax is supported but I can't quite seem to get it to work. Nonetheless, just using .split("___") is probably sufficient for me. Thanks!

The syntax is here: https://docs.rs/figment/latest/figment/providers/struct.Env.html.
APP_STORAGE_CONFIG={postgres_connection_string="value"}

Also, using __ (two instead of three) would suffice.


[1.20] p. 49, item 6:yntax description does not match playbook

In the description it says:
you use the syntax {var1: value, var2: value} to define each element

In the playbook the syntax is
- src: httpd.conf
  dest: /etc/httpd/conf/httpd.conf

While obviously both is valid to describe a dict, it is confusing to use the two different syntax styles here – at least without explanation.

You're correct; I think I did that to avoid writing a code sample in a double-nested list (translation: being lazy and avoiding complex markdown formatting testing). So I've corrected that and this is how it will look in the next version:

Thanks for raising the issue, it will be fixed on all platforms soon!


TypeError when I try use CORS

Sails version:0.12.3
Node version:v4.2.6
NPM version:3.5.2
Operating system: Ubuntu 16.04 LTS(xenial)

I'm using Sails 0.12.3 and I'm getting a "TypeError" when i tryed uses swagger-ui to read the generated json API by sails-swagger.
Apparently, this error is about CORS but I have had setted appropriately my CORS configuration.
Below is the error:

error: Sending 500 ("Server Error") response:
TypeError: Cannot read property 'split' of undefined
at _sendHeaders (/usr/local/lib/node_modules/sails/lib/hooks/cors/to-prepare-send-headers.js:54:75)
at routeTargetFnWrapper (/usr/local/lib/node_modules/sails/lib/router/bind.js:176:5)
at callbacks (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:164:37)
at param (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:138:11)
at pass (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:145:5)
at nextRoute (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:100:7)
at callbacks (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:167:11)
at sails.router.bind._middlewareType (/usr/local/lib/node_modules/sails/lib/hooks/csrf/index.js:87:20)
at routeTargetFnWrapper (/usr/local/lib/node_modules/sails/lib/router/bind.js:176:5)
at callbacks (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:164:37)
at param (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:138:11)
at pass (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:145:5)
at nextRoute (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:100:7)
at callbacks (/usr/local/lib/node_modules/sails/node_modules/express/lib/router/index.js:167:11)
at _addResViewMethod (/usr/local/lib/node_modules/sails/lib/hooks/views/res.view.js:372:3)
at routeTargetFnWrapper (/usr/local/lib/node_modules/sails/lib/router/bind.js:176:5) [TypeError: Cannot read property 'split' of undefined]

My cors.js configuration is:
    allRoutes: true,

   origin: '*',

   credentials: true,

   methods: 'GET, POST, PUT, DELETE, OPTIONS, HEAD',

   headers: 'content-type'

To reproduce the issue, create a new Sails application. Create some models and generate some api code with sails-swagger. Install swagger-ui. Then, acces swagger-ui on broswer on try generate the documentation inserting "localhost:1337/swagger/doc".

ok, fixed!

ok, fixed!

Updating: I've need to add some polities to can enable my app CORS:
in my "/api/policies" i add a file called "cors.js" with the code:
module.exports = function cors(req, res, next){
    res.set('X-Powered-By', 'Ubby');
    res.header("Access-Control-Allow-Origin", "*");
    res.header("Access-Control-Allow-Headers", "Origin, X-Requested-With, Content-Type, Accept");
    res.header("Access-Control-Allow-Methods", "POST, GET, DELETE, PUT, OPTIONS");
    res.header("Content-Type", 'application/json');
    next();
}


Fill groupstate table at each state progress of the facilitator

To be able to trace progress in the game and report on efficiency, the groupstate table is added, where timestamped data is collected about the exact time of reaching a state by each table (group). The groupstate table has a n:1 relation with groupround and contains a copy of the group_state string.
Each time when the facilitator allows the group to progress to a next state, a new record should be inserted in the groupstate table, with the state, the timestamp, a pointer to the groupround table, and potentially some content.

GroupStates are filled as intended.

Implemented and working.


add a getter function which gets a value, if it's not in the cache

It would be handy, if you could provide a getter function for the cache to retrieve actually retrieve the content, if the cache doesn't contain the requested value like in googles LoadingCache.

Closing this since it won't be implemented. I guess wrapping the cache in a new struct that does loading of missing values is a solution. I decide to do this since the complexity of the cache is quite high and therefore i'd like to keep it focused on just that.


[CONTROLLER] fix db migrate bug

This PR is for:

Server

Fixes db migrate bug
Steps to reproduce the bug

start server in slave controller

Changes to fix the bug
Affected branches

main

Checklist

[ ] Added unit test to verify the fix.

 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.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Added modals

Don't know if this fully works, but from my initial testing it does.

Looking at the code, it seems reasonable.
But do you have some kind of demo that shows how to use it?
You don't have to put it in index.html, just a little example would be nice.

added the example usage in the code;

One of the problems I've had with modals in the past is when it closes it doesn't create / have its own scope.  Since the DOM element resides on the page and is not created and destroyed each time the controller and the $scope stay around.
A problematic use case would be:

Admin clicks the "Add new user" button to open a modal.
Admin fills out some values.
Admin submits the modal and the user is added.
Admin clicks "Add new user" to add another user.
The previous values are still there.

A possible work around is to have an ng-click on the button as well that calls some initialization code but it is an inconvenience.
The pattern used by the angular bootstrap ui (I forget the exact name of the library), is that you pass in a template and a scope.  The template is compiled and injected into the DOM and destroyed when the modal closes.  That way the controller inside the template starts fresh each time.

Just merged it, and added an example in index.html

@brennancheung
I've created a plunker that you may find helpful (inspired in ui.bootstrap's modal):
http://plnkr.co/edit/rZxtyBSbJua2IcgFWCHA?p=preview
It's a service and a directive, and lets you open modals programatically.
It accepts the following parameters:

templateUrl (required): the template containing the modal's content (the template is not longer defined with an ID in the view).
controller (optional): a controller to be initialized with an isolated scope.
controllerAs (optional): name of the controller.

When the modal is closed, the scope is automatically destroyed.
Surely it can be improved, so I leave it for revision.

@federicojasson :+1:


Manually debug log each request and response

What type of PR is this?

/kind feature
What this PR does / why we need it:
The manual way without using the interceptor brings us the possiblity to
log the actual fields of the request and responses.
Which issue(s) this PR fixes:

Closes #67
Special notes for your reviewer:
Ref: https://github.com/hyperium/tonic/issues/333
Does this PR introduce a user-facing change?

Added request and response logging in debug log level.

The first is there seems to be a lot of duplicated code wrapping each call with two log messages, I explain it in more detail below, but it would be cool if we could make some middlewere to handle this or something.

The main reason to manually intercept the messages has been discussed there: https://github.com/hyperium/tonic/issues/333
Unfortunately tonic does not support an interceptor or middle layer where I can get back the types of the messages in a comfortable way. I can intercept at h2 level, but this would mean that I have to wait for the whole body to be available, collect it in a vector and then find a way to get my message types back.
The maintainers of tonic are thinking about a feature to intercept on a higher level but right now the recommendation is to go the way proposed in this PR.

The second question I wanted to ask is I did not see a way to turn this on/off via the config file? maybe i'm missing that though?

There is no need for any configuration here because debug!() logs only if the corresponding --log-level is set. :)


CL Adapter - Will NFL Market Creation work correctly with unpacked parameters?

@pgebheim is this still relevant and is this related to (or blocked by) any other ticket?

We should double check the adapter code but I'm 98% sure this was changed for NFL code as is.

Confirmed closed.


Ambient upgrade automated testing

[P0] Minimal downtime upgrade testing, L4 only first.

Hi @kfaseela this could be a small one to pick up if you have bandwidth.

I think we'll start with in place upgrade for ztunnel first, potentially using node cordon.  Basically update it as a k8s daemonset. No existing docs afaik.
Istiod update should be same as today, either in place or revision based.  I recall there are some existing upgrade tests already: for example helm_upgrade_test.go

@linsun : @howardjohn was indicating that the upgrade strategy is not completely finalized. Would it be too early to pick up this automation task?

@kfaseela I've started an ambient Helm upgrade guide for in-place upgrade here: https://github.com/istio/istio.io/pull/14491
While we may do something fancier/more automated later, this is a baseline that should always work, and if you want to write the tests we can at least start there.

@kfaseela I've started an ambient Helm upgrade guide for in-place upgrade here: istio/istio.io#14491
While we may do something fancier/more automated later, this is a baseline that should always work, and if you want to grab this ticket and write the tests we can at least start there.

Thanks! Will take a look!

Sounds a good plan!  Thanks @bleggett and @kfaseela!

@linsun not sure if we want to close this with just helm tests. Feel free to reopen if you think other upgrade paths also need to be tested


Alias doesn't work on main branch

from daft import DataFrame
df = DataFrame.from_pydict({"A": [1, 2], "B": [3, 4]})

df.select(df["A"].alias("A2")).collect()

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/orbit/workspace/daft/daft/viz/dataframe_display.py", line 49, in __repr__
    return vpartition_repr(
  File "/home/orbit/workspace/daft/daft/viz/repr.py", line 159, in vpartition_repr
    data_stringified = _stringify_vpartition(
  File "/home/orbit/workspace/daft/daft/viz/repr.py", line 67, in _stringify_vpartition
    assert all(
AssertionError: Data does not contain columns: {'A2'}

Looks like a regression from a recent commit - I'm taking a look!

Regression introduced here: https://github.com/Eventual-Inc/Daft/pull/299/files#diff-8ce570b3f2f114483c776e449047082aaebf681b7f965f8ec989d12337a421bdR210-R214
Working on a test and fix now


Error: If apache runs on loglevel INFO and write 404 into error.log, go-ftw can't find marker.

If /status/200 is not existing and if apache is set to loglevel info, we get the following error message(s) in the log:
[2022-11-12 23:08:18.012572] [-:error] <IP_ADDRESS>:36126 Y3AZUo3Gja4gB-tPE9uasgAAAA4 [client <IP_ADDRESS>] ModSecurity: Warning. Unconditional match in SecAction. [file "/apache/conf/httpd.conf_pod_2022-11-12_22:23"] [line "265"] [id "999999"] [msg "x-crs-test 6c7427f7-6638-46fd-8ad5-aa66961d3ffe"] [hostname "localhost"] [uri "/status/200"] [unique_id "Y3AZUo3Gja4gB-tPE9uasgAAAA4"]
[2022-11-12 23:08:18.013007] [core:info] <IP_ADDRESS>:36126 Y3AZUo3Gja4gB-tPE9uasgAAAA4 AH00128: File does not exist: /apache/htdocs/status/200

The 404 message prevents go-ftw from identifying and parsing the marker line correctly.

Ideally, this is caught correctly, or the README has to explain this.

I can't recreate the issue. I've written a test that uses the lines you supplied but parsing works. What is the error you get when it doesn't work?

Here is the output:
ftw run --dir /home/dune73/crs/tests/regression/tests --include 920410-1
9:56AM INF 🛠  Starting tests!

🚀 Running go-ftw!
        skipping 920360-1
        skipping 920370-1
        skipping 920380-1
        skipping 920390-1
👉 executing tests in file 920410.yaml
        running 920410-1: 9:56AM FTL ../runner/work/go-ftw/go-ftw/runner/run.go:126 > Failed to find start marker error="can't find log marker. Am I reading the correct log? Log file: /apache/logs/error.log"

Config attached.
conf.zip


Fix typo

,
 ->

@changeworld : Thanks for your contribution! The author, @PatAltimore, has been notified to review your proposed change.

@changeworld - we've moved your suggestions into the private repo for staging and validation. Thank you!


Can't pickle _thread.RLock objects

Been seeing the following error on my flow.
Context
Essentially I had stored dataframes to self.x, self.y variables within a parallelized function triggered through foreach, hoping each parallel call will maintain its own copy that I can access later. These variables are not initialized/referenced/used anywhere outside that step.
Hypothesis
I do store a bunch of large dataframes, not sure if that is the reason for causing it.

Hi @Viveckh ! This happens when you assign a complex object (maybe your dataframe) to self. that doesn't support pickling. A workaround is to use the object's native serialization function to convert the object to a bytes object and store the bytes in self. instead. On the reading side, you'll need to deserialize the bytes.
You can have a set of helper functions like self.df = save_dataframe(my_dataframe_object) and df = load_dataframe(self.df) to make this easier.
We have plans to handle serialization of large dataframes better automatically but this approach will work for any complex objects (e.g. Keras models, which have hit a similar issue).

Thank you @tuulos for the prompt and thorough response, that was very helpful.
An unrelated question - Is it possible to override namespace while kicking off a flow. Or a way to access the tag of current flow from within while it is running?

For anyone who encounters this, in my case, it was not the pandas dataframes but instead instances of my custom python classes that I was assigning to self. variables. And I guess they couldn't be serialized cause those class instances were storing db connections and such.

I am facing this similar issue while using Lime with Metaflow.
self.explainer = lime.lime_tabular.LimeTabularExplainer(X.values, feature_names=X.columns.tolist())
Wondering how could this be resolved. Any help would be appreciated.
Thanks


Conversion issues found for topic "Async Programming in C#/VB"

Ref: http://dotnet.readthedocs.org/en/latest/async/async-csharp-vb.html

Code blocks are not rendering properly: some are not rendering as code and some are inside a block but not the entire sample, just parts of it
Use of blockquote for two of the bulleted lists seems wrong
The table Use this / Instead of this / When wishing to do this is unreadable in MD

Putting some issues here:

Title falls off the edge of the margin on index
Table at the bottom is missing
The "#" in "F#" is missing in all headers

Also found conversion issue on this one http://dotnet.github.io/docs/porting/supporting-core.html
The images aren't displaying.

Yup, looks like images are broken for all articles.

Feel free to make a PR fixing it. :)

I can get the first and last (rename title to be shorter and re-add the "#" that went missing in the conversion), but the table might be a bigger issue.  It renders as a valid markup table on Github and vscode, but doesn't show any borders on the site (thus making it hard to tell that it's a table).  I'm not familiar enough with the doc system to make a change to how tables are rendered.

Ah, young padawan. :) I jest. I will take that. Please file issues for each of them so we can refer the properly. I will be fixing the CSS overall anyway.

@blackdwarf and I are following up with Ken on the images issues. I don't know if they were ported to the new site.
For the table, it does look better on the site than it looked on my machine, which was all garbled. And the code samples also look ok here. Not sure why it was displaying differently on my machine when I was validating it locally.

I think this one can also be closed. I think it refers to this topic https://docs.microsoft.com/en-us/dotnet/articles/csharp/async but the issues I've listed don't repro anymore