Floatbar

Windows client: Suppress minimize and restore buttons in float bar when toggle-fullscreen is false, added window title to float bar, improved overall appearance with gradients, shadow and transparent icons.

@freerdp-bot test

@dcatonR1 what is the source resp. license of the .bmp images?

@freerdp-bot test

Multiple coding style issues (mainly spaces vs. tabs) see https://github.com/FreeRDP/FreeRDP/wiki/Coding-Guidelines

I've just commented a few but there are several API calls with missing error checking

@freerdp-bot test

@dcatonR1 quite a while passed. Still interested in getting this upstream then please rebase your code to the current master and fix/address the comments in the pull request.

I understand you want formatting consistent with whatever your standards are, but I'm up to my eyeballs at work and cannot spend time on formatting issues right now.  Hopefully the code I submitted is useful; please do as you wish with it.  When I have time to come up for air I'll check back and see if it's still open.

@dcatonR1 thanks for your feedback. The formatting is one thing. But the more blocking are some of the comments like from @nfedera regarding the license/copyright/source of the bmp files.

I created the bmp files myself and don't claim any ownership or copyright issues on them, or anything else I've submitted.  Is that sufficient or is there something else I need to explicitly agree to?

@dcatonR1 thanks for the feedback.

NP.  I'll check back when things lighten up at work, probably not for a few weeks at least though.

@dcatonR1 still time/interest to get this in?


chore: Remove Segment Resync

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Removal of feature leftovers

What is the current behavior? (You can also link to an open issue here)
Segment Resync feature is available in the UI to be triggered by the user, and the method is also triggered automatically after leaving an orphaned (before: unsynced) segment. There is no longer a reason to perform segment resync, because of the data model changes (https://github.com/nrkno/tv-automation-server-core/pull/417, https://github.com/nrkno/tv-automation-server-core/pull/445).

What is the new behavior (if this is a feature change)?
All the remaining client and server fragments regarding segment resync are removed.

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

Merging #557 (1439966) into release37 (8dd70a6) will increase coverage by 15.04%.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           release37     #557       +/-   ##
==============================================
+ Coverage      67.30%   82.34%   +15.04%
==============================================
  Files            259       12      -247
  Lines          17780     1008    -16772
  Branches        4026      213     -3813
==============================================
- Hits           11966      830    -11136
+ Misses          5775      177     -5598
+ Partials          39        1       -38

Impacted Files
Coverage Δ

meteor/server/api/playout/api.ts

meteor/server/api/ingest/commit.ts

meteor/server/api/blueprints/__tests__/lib.ts

meteor/server/migration/0_18_0.ts

...eor/lib/collections/ExpectedPackageWorkStatuses.ts

meteor/lib/lib.ts

meteor/server/api/integration/expectedPackages.ts

meteor/lib/api/blueprint.ts

meteor/lib/api/rundownNotifications.ts

meteor/server/api/playout/lib.ts

... and 232 more

Continue to review full report at Codecov.

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


consumer: Add scripts to access service

Signed-off-by: Romain Arnaud<EMAIL_ADDRESS>

/retest

/retest

CI was green earlier. The shellcheck error is fixed and "Pipelines as Code CI / pipeline-service-static-code-analysis" is green. Merging the PR.


Test Endpoint support Define.xml files

Acceptance Criteria
The endpoint receives the rules and the datasets so we need another property for Define-XML.
A define_xml property needs to be added. The value will be the xml content. The endpoint will pass the Define-XML content to the rules engine.

The acceptance criteria requires fulfilment of following
1.	A define_xml property needs to be added
The define_xml property is successfully added to the endpoint TestRule which can also be seen in below screenshot (line 30).

But it is recommended to add a check after line 30 to see if ‘json_data.get(‘define_xml’)’ returned none. This will be helpful in cases when define_xml actually have a content but due to fetch error none is returned.
2.	The endpoint will pass the Define-XML content to the rules engine.
The line 31 in above screenshot confirms the implementation of this. The endpoint passes te define_xml to the Rules Engine. Changes in Rules Engine are also proper to receive the define_xml value.
Testing: The test code for testing the inclusion of define_xml is also proper and enough for testing
Conclusion: The acceptance criteria for this issue if successfully fulfilled. The Issue can be closed successfully.

Validation Summary
Validation Methodology:
The validation process incorporated rigorous testing methodologies to ensure the accuracy and reliability of the system. This included:
Unit Testing:
Unit Tests were updated based on the changes made to the test endpoint in core.py file in the Rules Engine repository. The unit tests covered edge cases and checked all . It covered the cases such as validation test_rule with a valid definexml
Code Review:
The code changes in the code files were reviewed and were found to be in accordance with the Acceptance criteria. Also, the code was reviewed for following:

Code Quality Evaluation
Logic and Functionality Review
Error Handling and Exception Handling Assessment
Security Considerations

Regression Testing:
Regression tests were run to validate that the changes did not affect any functionality of the overall system.
Test Cases:

Test Case Name
Description
Status

test_rule_with_errors
Validates the rule execution with the presence of errors.
Passed

test_rule_without_errors
Validates the rule execution without any errors.
Passed

test_rule_skipped
Validates the rule execution when the rule is skipped due to domain exclusion.
Passed

test_rule_with_define_xml
Validates the rule execution with Define.xml file and checks for errors in variable validation.
Passed

Test Execution:

The Tests were run by executing the unit tests using pytest framework. Following command was run for execution of Unit tests:
python -m pytest tests/unit/test_rule_tester/test_rule_tester.py
The regression tests were executed using the following command:
Python -m pytest test_cli.py

Results:
After successful running of unit and regression tests and careful validation, the issue was found to be good to close as the Acceptance criteria was fulfilled and all validations passed.
Deviations and Non-conformances:
No deviations or non-conformances were observed.
Summary of Findings:
Test Results: All tests conducted during the validation process passed successfully, indicating that the system functions as intended and meets the specified requirements. The tests covered various scenarios and edge cases, providing confidence in the system's reliability and correctness.
Unit Regression: The unit regression tests were executed, and no regressions or issues were identified. This indicates that the changes made during the development or modification of the system did not introduce any unintended side effects or break existing functionality.
Code Reviews: The code reviews conducted by both development and QA teams were completed, and no major issues or concerns were identified. The code adheres to coding standards and best practices, demonstrating good code quality, readability, and maintainability.


Line weight and color regression introduced in latest commit

The latest commit 1e0793dbbb03100128c24f983477b5b83beda8ef introduced some bugs:

The weight setting for lines do not work anymore.
Random colors seems to all be yellow with random transparency

Below is the result of running example.ts (I am hiding the markers to see the polygons better).
With previous commit (71ef08d):

With current commit (1e0793dbbb03100128c24f983477b5b83beda8ef ):

Ty for catching this!


NdArrayTensor should be public

Feature description
Currently NdArrayTensor is private which doesn't allow to create a Tensor from existing Array.
Feature motivation
It seems the only way for now is to convert Array into Data and then into Tensor which requires data copying because of different ndarray layouts. Also WgpuTensor and TchTensor are public which makes it a bit inconsistent.
Probably the same should apply to ADTensor.

Also it's now possible to get NdArrayTensor and access its public field via into_primitive method, which means it doesn't make much sense for it to be private.


Changing set to with functions

Checklist

[ ] I have linked to any relevant issues.
[x] I have updated the documentation.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] I have added necessary labels.
[x] I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
[x] I have requested a review from the relevant team or maintainers.

Mismatch function here
https://docs.fuel.network/docs/fuels-rs/running-scripts/#calling-contracts-from-scripts

IMO we just need to update the documentation as we use with now and not set

Closed in favor of https://github.com/FuelLabs/fuels-rs/pull/1146


fix: [#3969] useLanguagePolicy setting turn state, instead of languagePolicy

Reverts #3970
Fixes #3969
Fix: commit
Description
This PR fixes an issue where the bot wasn't able to respond to messages after the Welcome message is sent.
When the AdaptiveDialog.beginDialog method is executed, the context.turnState for the second message (e.g.: help) was the same as the first message (WelcomeMessage), thus, having the activityProcessed property set to true, the bot didn't process the new Activity and ended the Dialog right away.
Specific Changes

Revert PR #3970
Updates the LanguageGeneratorExtensions.useLanguagePolicy method to set the correct LanguagePolicy like in DotNet (link) instead of setting the turnState object with the languagePolicy.

Change: dialogManager.initialTurnState.set(TurnPath.languagePolicy, policy);

Testing
The following images show what was causing the issue, the DialogContext using the same turnState as the message sent before, and the bot working after the fix.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

69 of 78 (88.46%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 84.593%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

libraries/botbuilder-dialogs-adaptive/src/functions/missingPropertiesFunction.ts
12
14
85.71%

libraries/botbuilder-dialogs-adaptive/src/generators/multiLanguageGeneratorBase.ts
40
47
85.11%

Totals

Change from base Build<PHONE_NUMBER>:
0.05%

Covered Lines:
19706

Relevant Lines:
22072

💛  - Coveralls


[Suggestion] DH/LoD clouds

something for the list much later down the line. having DH compatability to render (pseudo) clouds for DH's fake LoD chunks. i believe DH's API may have some basic info stored that may make it possible to approximate climate effects. these distant clouds would best be rendered as a 2d texture based off a slice of the noise map used.

Not a bad idea, would definitely allow for farther render distances, I'll consider it


Facing issue in android, loads infinite or restarts the app

I recently started facing an issue with the way payu package works. I am just trying to use the .openCheckoutScreen method which opens up the payment screen. When the app is freshly installed I am able to do 3-4 payments successfully however after that the payu payment gateway shows infinite loading or after successful payment it restarts my app. No callback is received in these cases in failure/cancel/success.
Here is my flutter doctor and a piece of code I am trying to implement.
`Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.10.3, on macOS 13.4.1 22F770820d darwin-arm64,
locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.80.1)
[✓] Connected device (3 available)
[✓] Network resources
• No issues found!`
Code implemented:
    try {       _checkoutPro.openCheckoutScreen(           payUPaymentParams: payUPaymentParams,           payUCheckoutProConfig: payUConfigParams);     } catch (e) {       print('exception_payment $e');     }

Using the stable v1.0.0 which does not cause this issue.

Here is a comprehensive guide on how to seamlessly integrate PayU's payment gateway into your Flutter app, which you can explore and benefit from: https://sudhanshupatil.medium.com/integrating-payu-payment-gateway-in-your-flutter-app-c3849433b32e

kindly use the latest version https://pub.dev/packages/payu_checkoutpro_flutter


Add feature to resize the IntelliSense window

Sometime the IntelliSense window is too small to read all information, so I am looking forward to resize the window using mouse, thx~

Duplicate of #29126.


Update expression using set should only specify provided attributes

When Updating entity with only few attributes, generated update transform overwrites existing attributes with null.

Expected: Set Update expression should only include provided fields

Upon further debugging, this seems to be how @netjs injections work, closing it now.

Upon further debugging, this seems to be how @netjs injections work, closing it now.


Fixes issue 915 - displaying invalid counts instead of irrelevant arguments

Why?
Fix for issue #915 - Confusing failure message when other arguments involved with received messages. Invalid counts more relevant in situations where multiple arguments in play.
What Changed?
Previously, an exception was raised immediately after a received message did not match one of items in the argument list. Now the standard invalid count message is returned unless there is no match in the entire list. Tests by @myronmarston copied from #841 added in.
What Else Changed?
(updated 04/03/2015 with code refactor, adding in suggestions from @myronmarston and @JonRowe)

Took out new method and rolled it into existing raise_unexpected_message_args_error method
DRY'd up some error methods by adding common code to message_error (name to generic?)
Rolled up received_arg_list into format_received_args and setup to handle single/multiple messages with single/multiple arguments.
Removed a few splat operators so a more standardized messages list can be sent for single and multiple messages.

(updated 04/05/2015)

Added fix for 2nd part of ticket. Display errors fixed for expect to receive messages.
A couple of different options on this one, so took the path of least resistance.  Added similar_messages to SimpleMessageExpectation should allow the removal of (defined?) check. Adding check for similar_messages.blank? is not necessary, but would reduce redundant calls to unadvise method. Open to suggestions if not matching existing patterns.

Thanks for working on this, I have some minor feedback which I'll put inline. Don't worry about the appveyor failure it's failed to start the build entirely so...

Good to know, the appveyor failure had me scratching my head a bit.  Thanks for the the feedback @JonRowe.  All seems like good input, so I'll touch this up and get an update in soon.

Updated with some of the suggested refactoring.

I switch form all? to none? didn't work as expected, but using any? did
Additional method removed in favor of inline return
Couldn't get fail_with to play nice with a_string_containing, but raise error specifying error type worked.

Seemed like some connection issues might have been happening during Travis CI process, so perhaps a rebuild would be needed. Some failures were because to position of end it seems.
 return if @messages_received.any? do |(method_name, args, _)|
                !expectation.matches_name_but_not_args(method_name, *args)
        end

Should I align it under the return in order to pass?

Should I align end under the return in order to pass? Seems that would be against what I would expect.

No it needs to align with @messages, end is expected to line up with the start of it's statement, which is what it's being passed to as a block so the start of @messages_received.any?

Looks like that did the trick @JonRowe , thanks.

In #841 there's a failing spec in once_counts_spec.rb that concerns this same behavior for expect(...).to receive(...) (rather than for have_received as you've dealt with here).  Any chance you'd want to tackle that in this PR as well since it's related?  If you don't want to (or don't have the time), that's fine; we'll just need to open another issue for that so we don't lose track of it.

Good point on the grouped arguments, didn't think of that scenario. I'll try a few different versions to see if one looks more appealing then the other.
If you won't be needing it right away, I don't mind taking a stab at the receive.  Just learning the project as I go.

I made a new method to raise the errors to since we were a little unsure of how exactly we want the formatting to be done. I also couldn't figure out how to distinguish between with/non-with errors, so things were getting messy trying to use the existing error formatting methods. New method seemed the like better option.
I took a look at the once_counts_spec test as well, but we might need to tackle that in a different PR if that's ok. Flow didn't seem as straight-forward on that one and I don't know when I'd be able to get back around to it.

I took a look at the once_counts_spec test as well, but we might need to tackle that in a different PR if that's ok. Flow didn't seem as straight-forward on that one and I don't know when I'd be able to get back around to it.

That's fine.

(this was updated @myronmarston)

Looks like adding in the 2nd part fix is causing CI to throw a complexity warning. Any suggestions, or should I remove the 2nd part of this fix?

Sorry, @johnceh, I didn't know you had pushed new commits since GitHub doesn't notify repo owners when PRs get new commits.  Next time please post a comment saying "Ready for another review" (or whatever) and that'll let me know.

Looks like adding in the 2nd part fix is causing CI to throw a complexity warning. Any suggestions, should I remove the 2nd part of this fix, or do we tackle a complexity refactor on the message_received method?

To repro that failure locally, run bundle exec rubocop lib.
Anyhow, that method looks like it should be refactored at some point, but I consider that out of scope for this PR.  So I recommend disabling rubocop there:
      # rubocop:disable all
      def message_received(message, *args, &block)
        # ...
      end
      # rubocop:enable all

I'll try to review the rest of the PR later today or tomorrow.

Rubocop change is in, and it's ready for another review @myronmarston
Thanks for the tip on bundle exec rubocop lib, I wasn't sure how to go about testing if I tried making changes.

OK, I need to head to bed (I'm exhausted...) but I haven't finished my review.  I'll try to wrap it up tomorrow or the next day.

Sorry @myronmarston, didn't mean to ruin your Easter weekend. No rush on my side of things. :)

Ready for another review @myronmarston
I think I got most of your suggestions in, and it's looking a lot better.

Thanks @johnceh -- you did great work here!


add style-loader to tutorial "Code Splitting - CSS"

fixes #1131

thanks for the feedback, implemented!

Awesome, great work!


Pretraining weights for the non-vision task?

Hi, based on the paper, it sounded like there would be a release of the pretraining weights for the text and layout pretraining tasks for UDOP. When will that be released? Thanks!

Hello, we appreciate your interest in our work. We will release the pretraining weight shortly. Will update here once done.

We will release it this/next week at latest.

Thank you very much for the effort @zinengtang


Add more environment variables for configuration

This PR adds some more environment variables to allow better configuration of M/Monit.

add multiple MMONIT_PROXY_ env vars to allow running M/Monit behind a reverse proxy as documented inside the official manual
better tuning of database connection pool. On smaller setups with a dozen hosts monitored only the minimum connection count of 5 wastes to much memory (on client and server side) as 2 minimum connection are often enough without parallel usage of the web ui.
allow setting session timeout

@jchonig any change these two PR gets merged?

Please refactor these into one PR with at most one commit instead of several commits that fix each other.
Thank you for pointing out the typo and switching to apt-get, I made those changes you'll need to drop them.
Thanks and sorry for the delay.
Jeff


Update troubleshooting.md

Correcting PowerShell Script
The original script didn't work. I have updated it so that it is correct.

@sdwheeler
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@pmatthews05 Thanks for the submission. Can you make the same change to the bash example above this?

@pmatthews05 Thanks for the submission. Can you make the same change to the bash example above this?

Sure, let me take a quick look and I'll update the PR.

@sdwheeler I have updated the Bash scripts and tested.

#sign-off


I want to get the image data after face detection. What should I do?

Can I output face detection image data after face detection, whether it can be similar to ImageCroppingCalculator operator, configured in face_detection_mobile_cpu(gpu).pbtxt

Are you trying to capture the output of a calculator? Also, are you trying to do this on Android/iOS or desktop?

Are you trying to capture the output of a calculator? Also, are you trying to do this on Android/iOS or desktop?

i want to get image data,in order to register or recongnize


RuntimeError: reciprocal is not implemented for type torch.cuda.LongTensor

Hi, Im a new hand, when I use python3 to run your code and jwyang/fpn.pytorch meeting the same problem, as follow:
Preparing training data...
done
before filtering, there are 10022 images...
after filtering, there are 10022 images...
10022 roidb entries
Loading pretrained weights from data/pretrained_model/resnet101_caffe.pth
/home/wangshaoju/.conda/envs/ycy/lib/python3.6/site-packages/torch/nn/functional.py:1749: UserWarning: Default upsampling behavior when mode=bilinear is changed to align_corners=False since 0.4.0. Please specify align_corners=True if the old behavior is desired. See the documentation of nn.Upsample for details.
"See the documentation of nn.Upsample for details.".format(mode))
/home/wangshaoju/jwyang/FPN_Pytorch/lib/model/rpn/rpn_fpn.py:79: UserWarning: Implicit dimension choice for softmax has been deprecated. Change the call to include dim=X as an argument.
rpn_cls_prob_reshape = F.softmax(rpn_cls_score_reshape)
Traceback (most recent call last):
File "trainval_net.py", line 366, in
roi_labels = FPN(im_data, im_info, gt_boxes, num_boxes)
File "/home/wangshaoju/.conda/envs/ycy/lib/python3.6/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/wangshaoju/jwyang/FPN_Pytorch/lib/model/fpn/fpn.py", line 191, in forward
rois, rpn_loss_cls, rpn_loss_bbox = self.RCNN_rpn(rpn_feature_maps, im_info, gt_boxes, num_boxes)
File "/home/wangshaoju/.conda/envs/ycy/lib/python3.6/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/wangshaoju/jwyang/FPN_Pytorch/lib/model/rpn/rpn_fpn.py", line 109, in forward
rpn_data = self.RPN_anchor_target((rpn_cls_score_alls.data, gt_boxes, im_info, num_boxes, rpn_shapes))
File "/home/wangshaoju/.conda/envs/ycy/lib/python3.6/site-packages/torch/nn/modules/module.py", line 491, in call
result = self.forward(*input, **kwargs)
File "/home/wangshaoju/jwyang/FPN_Pytorch/lib/model/rpn/anchor_target_layer_fpn.py", line 141, in forward
positive_weights = 1.0 / num_examples
File "/home/wangshaoju/.conda/envs/ycy/lib/python3.6/site-packages/torch/tensor.py", line 320, in rdiv
return self.reciprocal() * other
RuntimeError: reciprocal is not implemented for type torch.cuda.LongTensor
This virtual environment can run the jwyang/faster-rcnn.pytorch. I have no idea, can you help me?
Thanks very much!

this code might be based on pytorch 0.3.0

I met the same problem.Have you solved it?could you show your solution?thanks


Garbage collector collecting, in empty project only window open

I have no clue, if thats an issue or not. just very annoying  in visual studio profiler pops up very often.(the yellow arrow)
New Project, install via nuget command : Install-Package Silk.NET -Version 2.15.0
and pasted "hello window" code in.

I have no clue, if thats an issue or not. just very annoying in visual studio profiler pops up very often.(the yellow arrow)
New Project, install via nuget command : Install-Package Silk.NET -Version 2.15.0 and pasted "hello window" code in.

can you see what its collecting?

something called "signature" native event runtime source. sorry very not much knowlegde , about this stuff, what im looking for

issue found ,it was a me problem sorry :< my small laptop could not handle too much apps open at once. and triggered propably the gc.


Minor improvements to string handling. Preliminary for the work in PR#230

Something went wrong with git. Don't merge this.


Updated README and ruby/npm dependencies

Now, both the Rails and Webpack Dev servers start with
foreman start -f Procfile.dev

@mbreining Can you please review.

Looks good and works great. I tested the changes locally. Just a few minor comments re: README.


Hero section

Demo :
https://github.com/scaffold-eth/buidlguidl-v3/assets/80153681/42ed959f-6185-46dc-b206-3eb2b58b02e7
Please feel free to nitpick on code / font sizes or anything, you don't like !!

This is looking amazing Shiv!!! Left a tiny thing


Generating config yaml from DreamBooth model based on SD v2.1 (along with ckpt)

Hello,
I trained a DreamBooth model based on SD v2.1. To use it in AUTOMATIC 1111, need a configuration yaml file along with ckpt. Using Colab, I generated the ckpt but can't get yaml from the model output (weights). Am I missing something? Could you please help me on this?   Thanks!!

Same request here, how to get the ckpt or pth file trained after local training? Looks like they are all bin files, thanks


How to access form state in a custom form?

Is there any way to access the formState in a custom form implementation? We want to display some kind of loading spinner when the form is submitting.
When i'm in a field, i can easily use useTsController but this is only allowed within fields, not in forms.

Do you mean in a custom form component? IE you're wanting to do:
function FormContainer() {
  const formState = useFormState();
  // ...
}

const MyForm = createTsForm(mapping, {FormComponent: FormContainer});

Or do you mean accessing the form state in the parent component of the form?
Right now this should be possible via passing props to the form component. Something like:
function FormContainer({isLoading}:{isLoading: boolean}) {}
//...
const MyForm = createTsForm(mapping, {FormComponent:FormContainer});

return (
  <MyForm
    formProps={{isLoading: isLoading}}
  />
)

Gonna close this, but feel free to reopen if the above isn't able to do something you're looking for


[HW]

Name: Jasleen Singh Bhangoo
Email<EMAIL_ADDRESS>Linkedin Profile: <www.linkedin.com/in/jasleen-singh-81549b43
Stargate_Spring.docx

Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

Hey Jasleen Singh Bhangoo, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/NVObutMoTDK8XQPekYkziQ


visualisation -> visualization

misprint on main page https://github.com/UDST/vizicities

Fixed.

@vovanmozg thanks for pointing this out – I'm English so you'll have to excuse some of the differences in spelling. We're currently discussing how to approach using a single language / way of spelling across ViziCities so this'll be cleared up one way or the other in the near future.


TIMX 345 - Add CLI command init-job

Purpose and background context
This PR introduces the first real CLI command init-job that is used to initialize a new job:

require job-directory argument to know where to create new job
require "A" and "B" git commit SHAs of Transmogrifier to build images
run core.init_job function to create new job
run core.build_ab_images to create new images based on A/B commit SHAs

How can a reviewer manually see the effects of these changes?
pipenv run abdiff --verbose init-job \
-d output/hello-world \
-m "Hello World" \
-a 395e612f8887b279ccf35363694bafb06ef9a25a \
-b 1910426fbcc98e098de222ccdcce3cf3ac1337e7

This should:

create a new directory at output/hello-world
build two images of Transmog for the SHAs provided

can confirm from terminal with docker images | grep "transmogrifier-hello-world"

log the contents of job.json

Re-running the command without any modification should return:
Error: Job directory already exists: 'output/hello-world', cannot create new job.

Includes new or updated dependencies?
NO
Changes expectations for external applications?
NO
What are the relevant tickets?

https://mitlibraries.atlassian.net/browse/TIMX-345

Developer

[X] All new ENV is documented in README
[X] All new ENV has been added to staging and production environments
[X] All related Jira tickets are linked in commit message(s)
[X] Stakeholder approval has been confirmed (or is not needed)

Code Reviewer(s)

[ ] The commit message is clear and follows our guidelines (not just this PR message)
[ ] There are appropriate tests covering any new functionality
[ ] The provided documentation is sufficient for understanding any new functionality introduced
[ ] Any manual tests have been performed or provided examples verified
[ ] New dependencies are appropriate or there were no changes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

35 of 35 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.2%) to 97.647%

Totals

Change from base Build<PHONE_NUMBER>7:
1.2%

Covered Lines:
166

Relevant Lines:
170

💛  - Coveralls

Approval stands


events POC

Lacks functionality for:

no waitlist domain logic is implemented

New schema for attendance

Attendance is completely independant from event. it does not even have a FK to event. this makes it nice to model in the backend.

Dashboard functionality:

finding users by name and marking as attendance
creating new attendance pools (capacity, year restrictions)
manage basic stuff for attendasnce like attendance start, end i.e.

Basic design for event detail page with

Functioning attend event button
Showing groups the logged in user can attend + all groups

There are some issues in apps/web/src/pages/events/[id].tsx here ATM. Will be looking at it soon.

199 💀


Comments on G4DarkBremstrahlung.cxx

Should this be G4cout or something else which isn't a manditory output (or controlled by a #define for verbosity)?
https://github.com/tomeichlersmith/G4DarkBreM/blob/c8a9ccf2018f84bd0a8f824f352b8ef2383b0c44/src/G4DarkBreM/G4DarkBremsstrahlung.cxx#L51
Please search for all use of cout here and make sure you're doing what is reasonable.  For example, old debugging couts can go.
Should this magic number be defined in a header or somehow determined in a way that it doesn't break in the future without warning?
https://github.com/tomeichlersmith/G4DarkBreM/blob/c8a9ccf2018f84bd0a8f824f352b8ef2383b0c44/src/G4DarkBreM/G4DarkBremsstrahlung.cxx#L30
This method seems like it could use a little more documentation:
https://github.com/tomeichlersmith/G4DarkBreM/blob/c8a9ccf2018f84bd0a8f824f352b8ef2383b0c44/src/G4DarkBreM/G4DarkBremsstrahlung.cxx#L148

Resolved with 6b60cdfdf8fbaa5d872c07c642093bfa57cc0462, 7a1da3ef199e7ac314619ddccf126a8a804ab8b7, and b8752ab49853684018f34eda73180a919e86b494


Every datasette plugin on the ecosystem page should have a screenshot

https://github.com/simonw/datasette/blob/master/docs/ecosystem.rst

I need to find a good tool for taking screenshots on OSX that are reliably the same dimension across multiple sessions. Maybe in Firefox?

Chrome and Firefox both support headless screengrabs from command line, but I don't know how parameterised they can be?

Headless Chrome and Firefox via Selenium are a solid choice in my experience. You may be interested in how pretix and pretalx solve this problem: They use pytest to create those screenshots on release to make sure they are up to date. See this writeup and this repo.

I have a pattern for creating screenshots using Puppeteer running in a GitHub Action now, see https://simonwillison.net/2020/Sep/3/weeknotes-airtable-screenshots-dogsheep/#weeknotes-2020-09-03-social-media-cards-tils


OAuth2 failed to run with https

I tried to enable SSL on Microsoft Edx and then enable  an OAuth2 client as the below :

however, when I am trying to invoke it and it failed all the time and complains that a request is not https for some reason like the below :

My further investigation indicates that the request variable in /edx/app/edxapp/venvs/edxapp/local/lib/python2.7/site-packages/provider/views.py comes with http all the time , even though I have enable OAUTH2 with SECURE  in both of lms.env.json and cms.env.json, like below :
"ENABLE_OAUTH2_PROVIDER": true,
"OAUTH_ENFORCE_SECURE": true,
"OAUTH_ENFORCE_CLIENT_SECURE": true,

I am wondering why oauth goes to /edx/app/edxapp/venvs/edxapp/local/lib/python2.7/site-packages/provider ?   why not any one of the below :
/edx/app/edxapp/venvs/edxapp/lib/python2.7/site-packages/oauth2
/edx/app/edxapp/venvs/edxapp/lib/python2.7/site-packages/oauth2_provider/
/edx/app/edxapp/venvs/edxapp/lib/python2.7/site-packages/oauth_provider/
@sdolenc  Please advise it and I believe there must be some false config in oauth by default in this distribution.

Thanks for identifying this issue. I apologize for the delayed response. Our team will be scheduling an investigation soon

@sdolenc any update on this issue ?

Yes, SSL is enabled in LMS and CMS on my site at https://edx.dolphin-labs.com.  It's deployed in full stack.
For some reason, oauth is propertly set up and Please find the attached config file for my env.
cms.env.json.txt
lms.env.json.txt

you can always reach me at<EMAIL_ADDRESS>

Assuming this issue is settled... Please re-open if necessary.


Add explicit subdomain Host Header to service.civicpdx.org/budget

This is a test of the pattern in PR #22, since I had to do something extra to get PR #44 to work under a different scenario.

The URLs http://service.civicpdx.org/ and http://service.civicpdx.org/budget still work, so I'll take this as a sign that everything's fine.  Closing a fixed.


Update MD file on new branch 2017-33-09_13-12-38

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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


This GUI library is very cool, I hope you can continue to update

thank you very much for sharing your code

I close this thread to keep the list of issues clear!
Thank you very much for the support!


Problems with rendering

In my app I have tried the following using your usage example:
import React, { Component } from 'react';
import Timeline from 'react-calendar-timeline';
import moment from 'moment';

const groups = [
  {id: 1, title: 'group 1'},
  {id: 2, title: 'group 2'}
];

const items = [
  {id: 1, group: 1, title: 'item 1', start_time: moment(), end_time: moment().add(1, 'hour')},
  {id: 2, group: 2, title: 'item 2', start_time: moment().add(-0.5, 'hour'), end_time: moment().add(0.5, 'hour')},
  {id: 3, group: 1, title: 'item 3', start_time: moment().add(2, 'hour'), end_time: moment().add(3, 'hour')}
];

class TestComponent extends Component {
  //...
  render() {
    return (
      <div>
        <Timeline
          groups={groups}
          items={items}
          defaultTimeStart={moment().add(-12, 'hour')}
          defaultTimeEnd={moment().add(12, 'hour')}
        />
      </div>
    );
  }
}

export default TestComponent;

but the output is wrong. It looks like this:

whats your dev setup like? Are you using webpack? If so, how are you handling css files?

This issue is related to how you are handling styles in webpack:
https://github.com/namespace-ee/react-calendar-timeline/issues/72

@viestat did you get this working? I'm having the same issue


HTML packages produces urlencoded %2f

Hi there. We noticed that the output generated is producing a %2f encoded value instead of a /. That's not a problem for a browser but it can be a problem for any tool looking for validity of references. In our case, it can't validate the page because it thinks it misses the links.

Packages:

camel.apache.org/v1alpha1

camel.apache.org/v1

camel.apache.org/v1alpha1

An easy workaround we're using is to replace the encoded value, so, the check is not failing anymore.

Yeah, I think you can consider a minor issue as it does not affect directly the user experience. However I wonder why this %2f is only present in the Packages section and the rest of html document is using normal / in the hrefs.

argh my memory doesn't serve me as to why we added urlencode. technically you can have id with / and hrefs to that as urlencoded version of it. (see recent PRs/commits) the links should still work?


close the file after output so it doesn't get stuck

Fix the error
java.io.IOException: Unable to delete file: E:\proyectos\tr2\tr2-web\build\resources\main\changelog.md

Sorry, this is not correct. The file.close() is not Ok, closing

I was going to look into seeing if Groovy had a try with resources equivalent -- maybe that would help resolve the issue?

Sure. I didn't do that because I'm not used to groovy and build doesn't
work, but it is the same (better actually)
On Wed, Oct 25, 2017 at 5:44 PM, Alex Selesse<EMAIL_ADDRESS>wrote:

I was going to look into seeing if Groovy had a try with resources
https://docs.oracle.com/javase/tutorial/essential/exceptions/tryResourceClose.html
equivalent -- maybe that would help resolve the issue?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/selesse/gradle-git-changelog/pull/7#issuecomment-339374746,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC8_7E-ZjQssuW5t0rOgtON7rJdXZdYPks5sv1dOgaJpZM4QGMXg
.


feat: allow to specify read-only replica for SELECTs

There are some minor API changes. Most notable of them is the removal of GetConn() IConn method, because the internal query conn IConn must be initialized to nil so Bun can resolve to a random read-only connect.
Overwriting the connection using query.Conn(conn) still works.
The usage should be like this:
db := bun.NewDB(sqldb, dialect,
  bun.WithReadOnlyReplica(sqldb2),
  bun.WithReadOnlyReplica(sqldb3))

When/if we want to provide custom logic how to pick a replica (e.g. replica with the lowest latency), it can be done like this:
db := bun.NewDB(sqldb, dialect,
  bun.WithConnResolver(resolver))

Closes https://github.com/uptrace/bun/issues/1037

Awesome.
I think variadic function is a good way to allow multiple replicas to be set at once.
db := bun.NewDB(sqldb, dialect,
  bun.WithReadOnlyReplicas(sqldb2),
  bun.WithReadOnlyReplicas(sqldb3, sqldb4),
  bun.WithReadOnlyReplicas(replicas...))

When we want to use the master for a specific query to avoid master-slave replication delay, how should we proceed?

There is a Conn() method on each query that allow to specify the connection that will be used. So you can just:
db.NewSelect().Conn(db)


Incomplete type used in nested name specifier in exhange-traits.h

Howdy, this is a problem that's been stumping me for a while, and I have no idea if it's from my stupidity or if it's actually an issue. I'm running on Ubuntu 20.04 with gcc v 9.3.0. This is Soci 3.2v5 (provided by Ubuntu) and I'm trying to compile targeting x86_64 with the C++11 flag enabled.
On line 35 of exhange-traits.h, I'm getting this error:
incomplete type ‘soci::details::exchange_traits<soci::indicator>’ used in nested name specifier
Which led me to this snippet of code (I fixed the formatting on the x_type definition so I could make more sense of it):
template <typename T>
struct exchange_traits
{
     // this is used for tag-dispatch between implementations for basic types
     // and user-defined types
     typedef user_type_tag type_family;

     enum // anonymous
     {
         x_type = exchange_traits<typename type_conversion<T>::base_type>::x_type
     };
 };

This seems to be a strange interaction between soci::indicator and exchange_traits<>, but everything I've done doesn't stray too far from the 3.2 manual, so maybe it's an issue elsewhere?

Okay, so as it turns out the package libsoci-dev doesn't pull in libsoci-core3.2v5 or libsoci-sqlite3-3.2v5 as dependencies. Installing these and linking to -lsoci_core and -lsoci_sqlite3 resolved the problem.


Update samples to use version constructor

Automate changes shown in: https://github.com/GoogleCloudPlatform/google-cloud-ruby/pull/2146

@frankyn the bucket name is a configuration option (i.e. you need to update the yaml file). For speech that value is here: https://github.com/googleapis/googleapis/blob/7ccfa2cd4a2737dc828061809d60ee03b23e2b5d/google/cloud/speech/v1p1beta1/speech_gapic.yaml#L35
The version thing I will update shortly.

@jbolinger the line you mentioned seems to be apart of smoke tests. I think that should stay the same. The lines below for sample snippets have gs://bucket_name/file_name.flac. Unless there's a bug and it's using the smoke test value instead.

@frankyn yeah, you're correct. It is indeed a bug and it's using the wrong value. I'll open a new issue for it.


Datetimes

Basic datetime IO support.
Currently this converts everything to numpy datetime arrays at write time. I'm not preserving pandas array types since there are multiple seemingly overlapping ways to deal with datetimes in pandas. This implementation also does not support time zones but that would be easy to add.
It would be good to get someone working with time series data to try this out and see if it meets their needs.
(I thought this would solve #455, but now see that was for datetime scalars which this does not currently support)

It would be good to get someone working with time series data to try this out and see if it meets their needs.

we do technically. If we detect datetime we always just copy it to obs directly.

An example (or you giving this branch a shot) would be great.
Do you have a way of saving these AnnData's at the moment?

@imipenem can you help here?

Do you have a way of saving these AnnData's at the moment?

At ehrapy, it just worked out of the box when writing these AnnDatas to .h5ad files. But this might be due to the fact, that we save columns with datetime values in obs only (and pandas treats these datetimes kind of different in comparison to numpy from what I've read), neither in uns or X. So we do not have any np.datetime values stored in the AnnData object at any time, which (IMO) fits our needs here (for now). So this would not affect us currently or do I miss something @Zethson?

Thought so as well. We didn't run into any issues.

I'm a little confused here. If I put any sorts of dates into obs, that anndata will fail to write in 0.7.8.
Can you make an example of this? For me:

 Failing example
import anndata as ad, pandas as pd, numpy as np
from vega_datasets import data
print(ad.__version__)

0.7.8

cars = data.cars()

dt_array = cars["Year"]
np_dt_array = dt_array.to_numpy()

N = np_dt_array.shape[0]
adata = ad.AnnData(X=np.ones((N, N)), obs=pd.DataFrame({"dt": dt_array}))

adata.write_h5ad("test_dt.h5ad")

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
~/github/anndata/anndata/_io/utils.py in func_wrapper(elem, key, val, *args, **kwargs)
    208         try:
--> 209             return func(elem, key, val, *args, **kwargs)
    210         except Exception as e:

~/github/anndata/anndata/_io/h5ad.py in write_array(f, key, value, dataset_kwargs)
    184         value = _to_hdf5_vlen_strings(value)
--> 185     f.create_dataset(key, data=value, **dataset_kwargs)
    186

/usr/local/lib/python3.9/site-packages/h5py/_hl/group.py in create_dataset(self, name, shape, dtype, data, **kwds)
    148
--> 149             dsid = dataset.make_new_dset(group, shape, dtype, data, name, **kwds)
    150             dset = dataset.Dataset(dsid)

/usr/local/lib/python3.9/site-packages/h5py/_hl/dataset.py in make_new_dset(parent, shape, dtype, data, name, chunks, compression, shuffle, fletcher32, maxshape, compression_opts, fillvalue, scaleoffset, track_times, external, track_order, dcpl, allow_unknown_filter)
     90             dtype = numpy.dtype(dtype)
---> 91         tid = h5t.py_create(dtype, logical=1)
     92

h5py/h5t.pyx in h5py.h5t.py_create()

h5py/h5t.pyx in h5py.h5t.py_create()

h5py/h5t.pyx in h5py.h5t.py_create()

TypeError: No conversion path for dtype: dtype('<M8[ns]')

The above exception was the direct cause of the following exception:

TypeError                                 Traceback (most recent call last)
~/github/anndata/anndata/_io/utils.py in func_wrapper(elem, key, val, *args, **kwargs)
    208         try:
--> 209             return func(elem, key, val, *args, **kwargs)
    210         except Exception as e:

~/github/anndata/anndata/_io/h5ad.py in write_series(group, key, series, dataset_kwargs)
    288     else:
--> 289         write_array(group, key, series.values, dataset_kwargs=dataset_kwargs)
    290

~/github/anndata/anndata/_io/utils.py in func_wrapper(elem, key, val, *args, **kwargs)
    211             parent = _get_parent(elem)
--> 212             raise type(e)(
    213                 f"{e}\n\n"

TypeError: No conversion path for dtype: dtype('<M8[ns]')

Above error raised while writing key 'dt' of <class 'h5py._hl.group.Group'> from /.

The above exception was the direct cause of the following exception:

TypeError                                 Traceback (most recent call last)
~/github/anndata/anndata/_io/utils.py in func_wrapper(elem, key, val, *args, **kwargs)
    208         try:
--> 209             return func(elem, key, val, *args, **kwargs)
    210         except Exception as e:

~/github/anndata/anndata/_io/h5ad.py in write_dataframe(f, key, df, dataset_kwargs)
    262     for col_name, (_, series) in zip(col_names, df.items()):
--> 263         write_series(group, col_name, series, dataset_kwargs=dataset_kwargs)
    264

~/github/anndata/anndata/_io/utils.py in func_wrapper(elem, key, val, *args, **kwargs)
    211             parent = _get_parent(elem)
--> 212             raise type(e)(
    213                 f"{e}\n\n"

TypeError: No conversion path for dtype: dtype('<M8[ns]')

Above error raised while writing key 'dt' of <class 'h5py._hl.group.Group'> from /.

Above error raised while writing key 'dt' of <class 'h5py._hl.group.Group'> from /.

The above exception was the direct cause of the following exception:

TypeError                                 Traceback (most recent call last)
/var/folders/bd/43q20k0n6z15tdfzxvd22r7c0000gn/T/ipykernel_4792/2332825967.py in <module>
----> 1 adata.write_h5ad("test_dt.h5ad")

~/github/anndata/anndata/_core/anndata.py in write_h5ad(self, filename, compression, compression_opts, force_dense, as_dense)
   1910             filename = self.filename
   1911
-> 1912         _write_h5ad(
   1913             Path(filename),
   1914             self,

~/github/anndata/anndata/_io/h5ad.py in write_h5ad(filepath, adata, force_dense, as_dense, dataset_kwargs, **kwargs)
    109         else:
    110             write_attribute(f, "raw", adata.raw, dataset_kwargs=dataset_kwargs)
--> 111         write_attribute(f, "obs", adata.obs, dataset_kwargs=dataset_kwargs)
    112         write_attribute(f, "var", adata.var, dataset_kwargs=dataset_kwargs)
    113         write_attribute(f, "obsm", adata.obsm, dataset_kwargs=dataset_kwargs)

<EMAIL_ADDRESS>in wrapper(*args, **kw)
    875                             '1 positional argument')
    876
--> 877         return dispatch(args[0].__class__)(*args, **kw)
    878
    879     funcname = getattr(func, '__name__', 'singledispatch function')

~/github/anndata/anndata/_io/h5ad.py in write_attribute_h5ad(f, key, value, *args, **kwargs)
    128     if key in f:
    129         del f[key]
--> 130     _write_method(type(value))(f, key, value, *args, **kwargs)
    131
    132

~/github/anndata/anndata/_io/utils.py in func_wrapper(elem, key, val, *args, **kwargs)
    210         except Exception as e:
    211             parent = _get_parent(elem)
--> 212             raise type(e)(
    213                 f"{e}\n\n"
    214                 f"Above error raised while writing key {key!r} of {type(elem)}"

TypeError: No conversion path for dtype: dtype('<M8[ns]')

Above error raised while writing key 'dt' of <class 'h5py._hl.group.Group'> from /.

Above error raised while writing key 'dt' of <class 'h5py._hl.group.Group'> from /.

Above error raised while writing key 'obs' of <class 'h5py._hl.files.File'> from /.

Sure:
import ehrapy.api as ep
adatas = ep.dt.mimic_3_demo(encoded=False, mudata=False)
print(adatas["INPUTEVENTS_CV"].obs)
adata = adatas["INPUTEVENTS_CV"]
# This may take 5-20 minutes
ep.pp.knn_impute(adata)
adata_encoded = ep.pp.encode(adata, autodetect=True)
ep.io.write("test.h5ad", adata_encoded)

I would not be surprised if we store things differently than you somewhere, but feel free to play around with it. Feedback is always appreciated!

I have the suspicion that the datetimes are somewhere just read as strings and then mapped to categoricals.

That seems to be the case.
adata_encoded.obs["charttime"].cat.categories.dtype

dtype('O')

Would it be useful if these were actual datetimes? The you could do things like ask how far apart the times were.

I have the suspicion that the datetimes are somewhere just read as strings and then mapped to categoricals.

That seems to be the case.
adata_encoded.obs["charttime"].cat.categories.dtype

dtype('O')

Would it be useful if these were actual datetimes? The you could do things like ask how far apart the times were.

Not surprised. Our primary motivation was the coloring of plots and things like that.
Yeah, your suggested use-case is a good one. Although, in general I am trying to reduce the dependency on real time as much as possible with ehrapy and to work more with pseudotime :)

@ivirshup is this PR still one approach that you'd follow or did it change since Pandas 2.0 got released? Datetime support would still be great for ehrapy - especially for stuff like comparing them and more


Confusion in inference.py

Hi @leoxiaobin
I am confused about the following code:
px and py
    coords, maxvals = get_max_preds(batch_heatmaps)
    ....
    # post-processing
    if config.TEST.POST_PROCESS:
        for n in range(coords.shape[0]):
            for p in range(coords.shape[1]):
                hm = batch_heatmaps[n][p]
                px = int(math.floor(coords[n][p][0] + 0.5))
                py = int(math.floor(coords[n][p][1] + 0.5))
                if 1 < px < heatmap_width-1 and 1 < py < heatmap_height-1:
                    diff = np.array([hm[py][px+1] - hm[py][px-1],
                                     hm[py+1][px]-hm[py-1][px]])
                    coords[n][p] += np.sign(diff) * .25

coords[n][p][0] is an integer (get_max_preds returns the integer coordinates).
Here we plus 0.5 and then use a floor function.
This will make it equal to coords[n][p][0]
so +0.5 here is useless?
Am I right?
In addition, I find that the hourglass code is different.
hourglass
 -- Very simple post-processing step to improve performance at tight PCK thresholds
    for i = 1,p:size(1) do
        for j = 1,p:size(2) do
            local hm = tmpOutput[i][j]
            local pX,pY = p[i][j][1], p[i][j][2]
            scores[i][j] = hm[pY][pX]
            if pX > 1 and pX < opt.outputRes and pY > 1 and pY < opt.outputRes then
               local diff = torch.Tensor({hm[pY][pX+1]-hm[pY][pX-1], hm[pY+1][pX]-hm[pY-1][pX]})
               p[i][j]:add(diff:sign():mul(.25))
            end
        end
    end
    p:add(0.5)

They plus 0.5 after getting the sub pixel coordinates.
Would you please explain the difference?
Thanks!

yes, when coords is integer, +0.5 is useless, but when coords is a float, floor(coords+0.5) can get the nearest integer of coords.

yes, when coords is integer, +0.5 is useless, but when coords is a float, floor(coords+0.5) can get the nearest integer of coords.

when does coords become float?
I find get_max_preds function always returns the integer coordinates(although the data type is float, but the value is actually integer.)

It's a general function, which should support float and integer.


Convert fonts in logo to outlines

Converts fonts in logo to outlines and trim border in svg.
Closes #1662

LGTM


Show separate packages for Windows vs macOS in github status

Thanks for the pull request @kring!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium for Unreal Samples works.

Looks good, thanks @kring!


OrbitDB can't be opened - AggregateError: All promises were rejected

From time to time in my local Chrome browser (and also in an Opera instance) after deleting a record from a document type db, records can't be read anymore. Reloading the browser doesn't help, no records can't be read at all.
An excecption is thrown, after db.all()
"AggregateError: All promises were rejected "
@orbitdb/core 2.1.0
helia 4.1.0
Current Workaround:

drop db when no peers are connected
reload browser, so browser reconnects to other peers
other browser peers with our db are replicating our db back to use (restore)

This works 50% of the time, the other 50% OrbitDB Access Controller complains about he can't write into our db without access rights. At that time with an empty db, no access rights were given. A fresh db should have been created with only my own identity like so:
mydb = await _orbitdb.open("/mydb/"+myDBName, {
type: 'documents',
sync: true,
AccessController: OrbitDBAccessController({ write: [_orbitdb.identity.id]})
})
In fact not beeing able to write into this db would be the expected behaviour. (So the error message is correct)
But: The question is now why it works sometimes (which is the desired behaviour - so I can get my db back from the other peers)
Other observation: Sometimes I drop the db, add a new record to it and still other peers starting the replicate records back to the previously dropped one. Which is ok but, it sort of confuses my understanding of above stated configuration of the OrbitDBAccessController.
Deleting one of those replicated records results then again into a "broken" or seamingly broken db which doesn't return any records anymore with db.all()
This is in fact two different issues. Apologize.

At a glance, the aggregate error would suggest an issue from reading records from the underlying IPFS blockstore. My hunch is that, for whatever reason, the in-memory blockstore is being unloaded and reloaded resulting in the inability for OrbitDB to read the blocks. Deleting a record may trigger a re-read of the blocks (perhaps for re-indexing).
Can I replicate the issue from deContact? If not, I'll hopefully have some time to try and set up a test to isolate the problem.

I'll try to give you some more info on how to reproduce it. For now I got a better error message:
exception while opening (reading) dbMyAddressBook AggregateError: All promises were rejected
promise) Error: Database is not open
at BrowserLevel.iterator (2.Dj7UZdDY.js:201:45244)
at Object.iterator (2.Dj7UZdDY.js:201:56728)
at iterator.next ()
at Object.iterator (2.Dj7UZdDY.js:201:2863)
at async l (2.Dj7UZdDY.js:199:26301)
at async Object.c [as all] (2.Dj7UZdDY.js:199:26375)
at async T (2.Dj7UZdDY.js:199:28154)
at async Object.Z [as iterator] (2.Dj7UZdDY.js:201:1565)
at async q (2.Dj7UZdDY.js:201:59667)
at async Object.all (2.Dj7UZdDY.js:20

I can't reproduce it anymore. But I guess it is hiding somewhere around the corner. But until then I'll close the issue to reduce the whole noise it makes. ;)

Just made another test with two Brave browsers (one Mac - one Linux) with a collegue and could reproduce the problem one more time following scenario:

I added a record on my Brave (fresh identity with fresh and clean db)
Collegue added a record (fresh identity with fresh and clean db)
College replicated my db and wrote a record into my db
I replicated his db and wrote a record into his db
Replications ok - I deleted his record in my db -> db crash

I will try now to reproduce this with another person!

Another Test with two DuckDuckGo Browser (Safari Clone) which are distant to each other immediatly resulted in a corrupted db when the remote party deleted an entry of my db.

After testing with both @silkroadnomad and the libp2p devs, we are thinking the problem is related to the stream returned by libp2p.dial() prematurely breaking. According to the libp2p devs, the a connection should be re-established but this does not seem to be the case. The workaround suggestion is to provide some kind of keep-alive to ensure two peers are reconnected if the connection prematurely ends.

I have the issue with an app where clients can write an entry for themselves in a public DB. If the DB doesnt sync well it can reach this state, without any record delete.
I'm not sure how it works behind the hood but what I see is that the last DB write isnt propagated, but on restarting the process the client will query my persistent bootstrap server who will try to find a non existent CID and crash.
After that I have to remove the DB state or opening the DB will always crash looking for a non existing CID.
It looks like the DB gets corrupted if some data chunk is missing, it happens regularly as I'm in dev and reload client apps often (nextjs reloads)
On my node process:
        Error: fetching raw block for CID bafyreieb7qahxwpzcu6hj7424yapfsqd57mbtsmbhy5kyk2nkjz453n3au from gateway https://trustless-gateway.link/ was aborted

I have the issue with an app where clients can write an entry for themselves in a public DB. If the DB doesnt sync well it can reach this state, without any record delete. I'm not sure how it works behind the hood but what I see is that the last DB write isnt propagated, but on restarting the process the client will query my persistent bootstrap server who will try to find a non existent CID and crash.
After that I have to remove the DB state or opening the DB will always crash looking for a non existing CID. It looks like the DB gets corrupted if some data chunk is missing, it happens regularly as I'm in dev and reload client apps often (nextjs reloads)
On my node process:
        Error: fetching raw block for CID bafyreieb7qahxwpzcu6hj7424yapfsqd57mbtsmbhy5kyk2nkjz453n3au from gateway https://trustless-gateway.link/ was aborted

Have you found any way to avoid this error? Hopefully I'm doing smth wrong, bc if the DB can get corrupted and cant be used anymore it is quite a critical issue.

Are you using a persistent blockstore with IPFS? It sounds like restarting the OrbitDB service clears any blocks in the blockstore which would result in a failed fetch of the block.
See https://github.com/orbitdb/orbitdb/blob/main/docs/GETTING_STARTED.md#block-storage for configuring a persistent blockstore.
If the problem persists, are you able to provide some kind of basic script to replicate the problem?

I have same error "createDBAggregateError: All promises were rejected".
When i open db with any address in my chrome.

I have same error "createDBAggregateError: All promises were rejected". When i open db with any address in my chrome.

This usually means the db address you are trying to replicate cannot be reached.
Could you please provide more information as to how you are connecting your peer to other peers (libp2p configuration, helia configuration, etc)?

No further information has been provided so closing. Please feel free to re-open if more information about the problem is available.


Fix #11

Just for feedback here:

I'm looking for another way to fix this issue based on your implementation: I think a "lazy" load of prettyconf should work
Other alternative is not use prettyconf ".env/env.ini" capability and add an option like --conf to cli
Also, provide a better to instantiate a custom Settings object


Make DBus calls Async (2nd attempt)

Follow up on https://github.com/aleph-im/aleph-vm/pull/595/files

Problem: dbus call were not async
use mypy procotol method for typing
fix CI, specify bus_type
fix init in async, isort
mypy
dbus fast is not in debian 11

@olethanh should we drop support for Python 3.9 / Debian 11 for this PR ?

@olethanh should we drop support for Python 3.9 / Debian 11 for this PR ?

Didn't we already have this discussion ? I'm not against it but I have no idea of what the processus or the consequence for this is :D
Anyway I have fixed the problem

Pr is ready for review. Updated with main
Note that there is a chance of behavior: Now in is_running for persistent vms we only check the stop time like we do for other VMs, to stay consistent. The behaviour is load_persistent_executions has been properly adapted

Also, will be good to have some tests about these refactor to prevent implementation issues.

Also, will be good to have some tests about these refactor to prevent implementation issues.

on which part?

Also, will be good to have some tests about these refactor to prevent implementation issues.

on which part?

On the whole Systemd class and on the main methods of the application that calls the systemd start, enable, stop and disable methods.


Improve README

Questions
Answers

Description?
Improve readme

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?

How to test?

Thank you @matks


[WIP] Added tutorial and updated documentation for 'to email' node

for https://github.com/thingsboard/thingsboard/pull/4429


Add code version to consensus config

Adds the code version to consensus config for the purposes of validation (see #950)

@elsirion My thinking is that once we validate the consensus configs on startup it should prevent binary incompatibility, although a database migration may also need to be done.

This looks fine to me. In my opinion, for the time being it is probably easier to deal with consensus config/binary incompatibility and database migrations separately. Once we try to support online updates though we'll need to think about the two in tandem.


Documentação api

:metal:

@mrsantos probably meant to send this to another repo! :stuck_out_tongue:


mask_size change causes size mismatch error

Hi,
I'm trying to change mask_size parameter from 7 (which is default) to 14 or 28 hoping to get more precise mask results. My config file is as below:

_base_ = '../point_rend/point_rend_r50_caffe_fpn_mstrain_3x_coco_downloaded.py'
model = dict(
    roi_head=dict(
        bbox_roi_extractor=dict(
            roi_layer=dict(type='RoIAlign', output_size=14, sampling_ratio=0),),
        bbox_head=dict(
            roi_feat_size=14,
            num_classes=1,
        ),
        mask_head=dict(
            num_classes=1,
        ),
        point_head=dict(
            num_classes=1,
        ),
    ),
    train_cfg=dict(
        rcnn=dict(
            mask_size=14,
        ),
    ),
    test_cfg=dict(
        rpn=dict(
            nms_pre=1000,
            max_per_img=100,
            nms=dict(type='nms', iou_threshold=0.7),
            min_bbox_size=0),
        rcnn=dict(
            score_thr=0.05,
            nms=dict(type='nms', iou_threshold=0.5),
            max_per_img=10,
            mask_thr_binary=0.5,
            subdivision_steps=5,
            subdivision_num_points=784,
            scale_factor=2)),
)
dataset_type = 'COCODataset'
classes = ('card',)
log_config = dict(interval=4)
checkpoint_config = dict(interval=128)
runner = dict(type='EpochBasedRunner', max_epochs=128)

optimizer = dict(type='AdamW', lr=0.00001, weight_decay=0.000005)
optimizer_config = dict(grad_clip=None)

data = dict(
    samples_per_gpu=2,
    workers_per_gpu=2,
    train=dict(
        img_prefix='data/base_set/training/',
        classes=classes,
        ann_file='data/base_set/training/training_labels.json'),
    test=dict(
        img_prefix='data/base_set/testing/',
        classes=classes,
        ann_file='data/base_set/testing/testing_labels.json'),
    val=dict(
        img_prefix='data/base_set/testing/',
        classes=classes,
        ann_file='data/base_set/testing/testing_labels.json'),
    # val=dict(
    #     img_prefix='data/base_set/validation/',
    #     classes=classes,
    #     ann_file='data/base_set/validation/val_labels.json'),
)

load_from = 'checkpoints/point_rend_r50_caffe_fpn_mstrain_3x_coco-e0ebb6b7.pth'

When I run under this configuration I get the following error about size mismatch:
Traceback (most recent call last):
  File "/home/patroclos/PycharmProjects/mmdet/tools/train.py", line 209, in <module>
    main()
  File "/home/patroclos/PycharmProjects/mmdet/tools/train.py", line 198, in main
    train_detector(
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/apis/train.py", line 210, in train_detector
    runner.run(data_loaders, cfg.workflow)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmcv/runner/epoch_based_runner.py", line 127, in run
    epoch_runner(data_loaders[i], **kwargs)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmcv/runner/epoch_based_runner.py", line 50, in train
    self.run_iter(data_batch, train_mode=True, **kwargs)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmcv/runner/epoch_based_runner.py", line 29, in run_iter
    outputs = self.model.train_step(data_batch, self.optimizer,
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmcv/parallel/data_parallel.py", line 75, in train_step
    return self.module.train_step(*inputs[0], **kwargs[0])
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/detectors/base.py", line 248, in train_step
    losses = self(**data)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmcv/runner/fp16_utils.py", line 109, in new_func
    return old_func(*args, **kwargs)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/detectors/base.py", line 172, in forward
    return self.forward_train(img, img_metas, **kwargs)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/detectors/two_stage.py", line 147, in forward_train
    roi_losses = self.roi_head.forward_train(x, img_metas, proposal_list,
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/roi_heads/standard_roi_head.py", line 111, in forward_train
    mask_results = self._mask_forward_train(x, sampling_results,
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/roi_heads/point_rend_roi_head.py", line 34, in _mask_forward_train
    mask_results = super()._mask_forward_train(x, sampling_results,
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/roi_heads/standard_roi_head.py", line 175, in _mask_forward_train
    loss_mask = self.mask_head.loss(mask_results['mask_pred'],
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmcv/runner/fp16_utils.py", line 198, in new_func
    return old_func(*args, **kwargs)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/roi_heads/mask_heads/fcn_mask_head.py", line 177, in loss
    loss_mask = self.loss_mask(mask_pred, mask_targets, labels)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
    return forward_call(*input, **kwargs)
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/losses/cross_entropy_loss.py", line 246, in forward
    loss_cls = self.loss_weight * self.cls_criterion(
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/mmdet/models/losses/cross_entropy_loss.py", line 164, in mask_cross_entropy
    return F.binary_cross_entropy_with_logits(
  File "/home/patroclos/PycharmProjects/.venvs/mmdet/lib/python3.9/site-packages/torch/nn/functional.py", line 2980, in binary_cross_entropy_with_logits
    raise ValueError("Target size ({}) must be the same as input size ({})".format(target.size(), input.size()))
ValueError: Target size (torch.Size([25, 14, 14])) must be the same as input size (torch.Size([25, 7, 7]))

I have tried to find which parameter I need to change to avoid this size mismatch error but couldn't find any. Checked tutorial below as well:
https://github.com/open-mmlab/mmdetection/blob/master/docs/en/tutorials/config.md
but still no chance. Can anyone help me about how I can increase mask_size parameter? Thanks in advance.

Since you change the mask_size=14 in train_cfg, the shape of mask_target is 25X14X14 but the shape of prediction is still 25X7X7. You can try to increase the output size of mask_roi_extractor or reduce the downsample_factor in CoarseMaskHead.


Multiple word scheme is truncated ✂

Bitrise codesign wizard generates the following command:
bash -l -c "$(curl -sfL https://raw.githubusercontent.com/bitrise-tools/codesigndoc/master/_scripts/install_wrap-xcode.sh) --scheme=Multiple Word Scheme 2"

But the generated xcodebuild command will be
INFO[12:01:19] 🔦  Running an Xcode Archive, to get all the required code signing settings...
INFO[12:01:19] $ xcodebuild "-workspace" "/Users/heyitsme/path/The Workspace.xcworkspace" "-scheme" "Multiple" "clean" "archive"

Apparently the scheme name is truncated after the first space.

Thanks for reporting, we'll try to fix this ASAP.
The workaround right now is to wrap the scheme name in single quotes, unless the scheme includes a single quote of course, something like:
bash -l -c "$(curl -sfL https://raw.githubusercontent.com/bitrise-tools/codesigndoc/master/_scripts/install_wrap-xcode.sh) --scheme='Multiple Word Scheme 2'"

Or with escaped double quotes:
bash -l -c "$(curl -sfL https://raw.githubusercontent.com/bitrise-tools/codesigndoc/master/_scripts/install_wrap-xcode.sh) --scheme=\"Multiple Word Scheme 2\""

Closing this ticket as this is not an issue of codesigndoc, but the bitrise.io website / how the parameter is passed to codesigndoc - we have a separate, internal ticket for the bitrise.io website, to fix this issue.
Thanks for reporting @andrassomogyi !


Shared libraries

One more modification request that would make it easier for an OpenBSD port/packages: use shared versions of libiconv and libintl instead of statically linking them.  Thanks.
Reported by: jmprev
Original Ticket: "feature-requests2/14":https://sourceforge.net/p/feature-requests2/14

I think this may be resolved by the move to autotools (since 3.1.1). I don't have a FreeBSD system to play with, but if there's interest in a FreeBSD port can someone try with the latest master?

I have a FreeBSD system (and happen to be the misc/liflines port maintainer there).  Just haven't got to this yet :(

This doesn't affect Linux at all, where libiconv / libintl are part of glibc => always included via shared libraries.
This mostly affects the BSDs where libiconv / libintl are separate shared libraries since BSD libc doesn't include this functionality.

[ Tested on a FreeBSD 13 machine ]
With shared NLS libraries (now the default):
$ ldd llines
llines:
        libintl.so.8 => /usr/local/lib/libintl.so.8 (0x8002e4000)
        libiconv.so.2 => /usr/local/lib/libiconv.so.2 (0x8002f3000)
        libncursesw.so.8 => /lib/libncursesw.so.8 (0x8003f2000)
        libm.so.5 => /lib/libm.so.5 (0x800454000)
        libc.so.7 => /lib/libc.so.7 (0x800486000)

With static NLS libraries (previous default, now enabled via ./configure --with-static-nls):
$ ldd llines
llines:
        libncursesw.so.8 => /lib/libncursesw.so.8 (0x8003e4000)
        libm.so.5 => /lib/libm.so.5 (0x800446000)
        libc.so.7 => /lib/libc.so.7 (0x800478000)

Compare this to Linux, which is not affected by this option, as libiconv/libintl are part of glibc:
$ ldd llines
        linux-vdso.so.1 (0x00007fffc8361000)
        libncursesw.so.6 => /lib/x86_64-linux-gnu/libncursesw.so.6 (0x00007fdc21f70000)
        libtinfo.so.6 => /lib/x86_64-linux-gnu/libtinfo.so.6 (0x00007fdc21f40000)
        libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007fdc21df1000)
        libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007fdc21bf0000)
        libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007fdc21be0000)
        /lib64/ld-linux-x86-64.so.2 (0x00007fdc22083000)


Unable to run base project

Expected Behavior
npm run ios should build and run application on iOS simulator
Current Behavior

Possible Solution
idk
Steps to Reproduce

npx create-expo-stack@latest --nativewindui --blank
npx expo prebuild --clean
npm run ios

CES Config File
{
  "cesVersion": "2.11.17",
  "projectName": "my-expo-app",
  "packages": [
    {
      "name": "nativewindui",
      "type": "styling",
      "options": {
        "selectedComponents": ["text"]
      }
    },
    {
      "name": "expo-router",
      "type": "navigation",
      "options": {
        "type": "stack"
      }
    }
  ],
  "flags": {
    "noGit": false,
    "noInstall": false,
    "overwrite": false,
    "importAlias": true,
    "packageManager": "npm",
    "eas": false
  },
  "packageManager": {
    "type": "npm",
    "version": "10.2.3"
  },
  "os": {
    "type": "Darwin",
    "platform": "darwin",
    "arch": "arm64",
    "kernelVersion": "23.5.0"
  }
}

iOS 17.5 was not installed, solution was to open project in Xcode, install 17.5, and rerun from step 3

iOS 17.5 was not installed, solution was to open project in Xcode, install 17.5, and rerun from step 3


crashing on A11 tablet

Steps to reproduce
Updated to 3.2.5 from 3.2.4 and then the app failed to launch and keeps crashing after splash screen.
Expected behavior
App V3.2.5 should launch
Actual behavior
App doesn't launch whereas 3.2.4 launches as expected
Crash logs
No response
Tachiyomi version
3.2.5
Android version
Android 11
Device
realme pad
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] If this is an issue with an extension, I should be opening an issue in the extensions repository.
[X] I have tried the troubleshooting guide.
[X] I have updated the app to version 1.8.2.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

same here app got crashed

Will release a fix soon


fix: mark the topolvm storageclass as default

This commit marks the topolvm storageClass as the default
if there are no other storageClasses that are present or
is no other class is marked default.
Signed-off-by: N Balachandran<EMAIL_ADDRESS>

/cherry-pick release-4.11


Fail gracefully when Jupyter notebook requirements are missing

initial import at top, should be able to give beginners better error messages that explain the import errors and installation errors, or point them to a launch URL (econ-ark) or MyBinder.

jupyter is not a HARK requirement
https://github.com/econ-ark/HARK/blob/master/requirements/base.txt

But then the user wouldn't even begin to be able to run a notebook and
encounter missing requirements, right?
On Wed, Jul 3, 2024 at 11:15 AM Alan Lujan @.***> wrote:

jupyter is not a HARK requirement
https://github.com/econ-ark/HARK/blob/master/requirements/base.txt
—
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/HARK/issues/719#issuecomment-2206501466, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFJIICZOVNL7STPILMLZKQISLAVCNFSM6AAAAABKJZMNVGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMBWGUYDCNBWGY
.
You are receiving this because you commented.Message ID:
@.***>

Yes we shouldn't have jupyter as a requirement. It's a frontend requirement to run the examples, it's not a requirement for HARK to work. Adding jupyter as a core requirements will add that to the packaged pip and conda package which adds a ton of bloat when someone does pip/conda install econ-ark. This also pollutes the environment.
I think this issue may not be still relevant but we need docs or some command to make sure users can set it up locally.


Support for orthogonal configurations in PEPOLAR with TOPUP

I believe we currently only handle AP/PA correction, but LR/RL is possible as well.

We do handle the two pairs. What I think we don't support is using orthogonal (e.g., LR/PA) or orthogonal&colinear acquisitions (e.g., RL/LR/PA/AP).

I changed the title:

Additional axis -> it's orthogonal schemes what's missing.
correction -> TOPUP is estimation (it can output "corrected" results, but the fundamental output is the estimated fieldmap, not the corrected EPIs)

I'm going to close this one for the following reasons:

Our circle CI tests include one dataset with AP/PA blips and one with LR/RL.
I have looked into the code and I can't see where we are enforcing that the PE directions are colinear (or, another possibility, that all must have the same total readout time).
There's currently no dataset in which we can test this.

If we found  a dataset to test this and turns out we are not supporting the use case, please reopen.


Fix spec tests by specifying :lsbdistcodename

Debian tests expect that to be set, otherwise spec tests fail.
This should fix the travis-ci build error as well, I believe.

Hello,
Thank you for your work.
I fixed this in the develop branch and I will publish a new release soon.
Regards,


Fix handling of Nullable types

The UdtWrapper now initializes the UnderlyingType property with the underlying type of the udtType, if it is nullable.
It looks like this issue was hidden by 2 tests in YAXLibTests/SerializationTest.cs:

NullableSample2Test()
NullableSample2WithNullAttributeTest()

Both succeeded, but should have failed, because Seasons enum of the test class is decorated with YAXEnum attribute, which is the exact replicate of issue #159. That's why serialization and deserialization of the serialization result are now included in these tests.
Still, special care should be taken before accepting this PR because of the 2 false test successes, but also because of the central role of the UdtWrapper.

Codecov Report

Merging #160 (9481d77) into master (7b3c829) will increase coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #160   +/-   ##
=====================================
  Coverage      79%    80%
=====================================
  Files          60     60
  Lines        3764   3767    +3
  Branches      724    726    +2
=====================================
+ Hits         2988   2997    +9
+ Misses        562    559    -3
+ Partials      214    211    -3

Impacted Files
Coverage Δ

YAXLib/MemberWrapper.cs
81% <100%> (+2%)
:arrow_up:

YAXLib/UdtWrapper.cs
82% <100%> (-<1%)
:arrow_down:

Continue to review full report at Codecov.

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

Maybe also update the change log in this pr :)

@axunonb guess you master is behind.
PS I would recommend for next time to create a branch in your fork instead of adding commits to (your) master. After this PR is merged, you will have a hard time syncing your master to this master ;)


duino-coin  Release 3.0

Hi, are you planning to upgrade DuinoCoinI2C ?

upgrading to this release is required. Servers will stop accepting version 2.X miners in a few weeks.

I need to take a look at the new Release. 😭

Lets do it!

Hi, are you planning to upgrade DuinoCoinI2C ?

upgrading to this release is required. Servers will stop accepting version 2.X miners in a few weeks.

Maybe only changing the "Miner Version" in the I2C Software to 3.0 and thats it ;)

We can't mine using this project?
19:05:30.229 -> [ ]FreeRam: 39056 I2C [ 0. ]
19:05:30.839 -> [0]Job BAD,Outdated miner: Share #11 0:01:34
Thank you.

Thank you very much.

I don't know what changed... 😑

Yeah, I guess they flipped the switch last night. I was mining happily, then BAM nothing. I'm a little dissapointed since I just finished my rig a couple of days ago :(

@revoxhere change some checks on the server, now should work.
The problem basically was related with the rigIdentifier.


[Rust] Decoder for dictionary encoding

Support reading dictionary encoded values.

Verified over oxford pet dataset on S3.


DBZ-1830 new connector property datatype.propagate.source.type

This change provides support for the new connector property
datatype.propagate.source.type
to propagate the column data type as a parameter to the corresponding field schemas in the emitted change messages. The expected value is a comma-separated list of database-specific type names, like VARCHAR, INT, or TIMESTAMP. The listed type names are propagated.

@FabioCantarini Hi, thanks for the PR, it is definitely nice to see this working!
Before the final merge I'd like to discuss the design shortly.
I wonder, do we need a separate config option to configure this behavior? Would it it be possible to extend column.propagate.source.type so you could you matching based on name or type?
What I envision is a syntax like
name:<regex>, name:<regex>, type: <regex>, type:<regex> where if name: is omitted it will be handled as a column name but a warning would be generated.
The  in case of type would be .. so you could either map for all cloumns in a single database like db1\..+\.integer, or to all like .+\.integer or for single table like db1.table1.integer.
@gunnarmorling, @FabioCantarini  Your thoughts?

Hi Team,
nice to meet you on GitHub.
Thanks @jpechane for taking the time to review this PR.
In my understanding, the proposed review is aimed to implement a configuration capability like the following:
column.propagate.source.type : name:db1.table1.columnA,type:.*\.VARCHAR

instead of
column.propagate.source.type : db1.table1.columnA
dataype.propagate.source.type : VARCHAR

I have a few concerns about the alternate design:

it's partially compatible with the existing implementation, which does not foresee the "name" or "type" keys
backup compatibility is proposed using by default the "name" key if it was missing, and generating a warning log
usage is more complex, due to the syntax a bit more tricky
more changes required in the code

ColumnMappers.propagateSourceTypeToSchemaParameter()
  may split the value in a "name" list and a "type" list,
  then call map() and mapByDatatype() respectively

ColumnMappers.mapByDatatype()
  may support matching against a "fully qualified data type" like "db1.table1.DATATYPE"

On the other side, after this proposed review we would get a new feature, that is applying the type propagation to the matching database names and table names only.
With the current implementation we can only enable propagation for all the columns with a specific data type. I believe this was exactly the original requirement from DBZ-1830, but not clear to me if it could raise a performance issue. By the way, I wonder why such source type property is not always propagated to Kafka, thus avoiding the need of a config property to enable this info.
--
At the end, I have some preference for the current design because it's easier.
Still I'm in doubt whether "datatype" prefix is the best choice or not for the property name.
Of course, waiting for your decision about, and fine for me to review the implementation according to the final design.

Thanks a lot for this PR, @FabioCantarini!
In regards to @jpechane's suggestion, I think there's value in making this option a bit more flexible (indeed going beyond what has been described in DBZ-1830). Being able to control this on a table level seems a reasonable suggestion. That said, why should it be baked into the existing option, @jpechane? Why not having a new option as suggested here, but making this a bit more powerful by means of regexs applied to table names instead of only the type name as is? I.e. a combination of your two proposals essentially?

Therefore we get a more powerful config property, like this in example
datatype.propagate.source.type : db1\.table1\.VARCHAR,db2\..+\.INTEGER

Seems fine to me.
Agreed.

Yes, exactly. That'd seem like a reasonable middle-ground to me; in particular, we wouldn't have any compatability concerns.

@FabioCantarini @gunnarmorling :+1:

Ok, cool. Seems we got a plan then :)

Yep. Going to amend the PR.

Excellent, thanks!

As reported here above, both after the amend and a further rebase from upstream, there's some test failure in the CI job. I believe this failure is unrelated from the changes in this PR. May I miss something instead?

Latest CI build passed, thanks to re-triggering I guess.
Please let me know if further changes are needed.

Great work! Thanks a lot, @FabioCantarini!


[Admin cabinet.Pricing page] Not all regions can be deactivated when selected more than 1 item

Environment: Microsoft Windows 10 Pro, Сhrome 111.0.5563.65.
Reproducible: always.
Build found: 08/06/2023.
Preconditions:

The web page https://www.pick-up.city/#/ubs-admin/tariffs is opened.
The User is logged in as a Super Admin.
English localization is selected.

Steps to reproduce:

Go to the page 'Тарифи' in the UBS-Admin menu.
Tap on the “gear” pictogram in the upper right corner.
Tap on “Деактивувати”.
Choose point “Регіон”
Choose more than 1 region to deactivate

Actual result
Only the first selected region is deactivated
https://github.com/ita-social-projects/GreenCity/assets/101433204/d0ae71ce-6fe6-4d46-98a2-ccd2ce64bc20
Expected result
All selected regions are deactivated

This bug is fixed. Tnx


large scale GP project

@yorkerlin I'm tuning the GSoC 2016 project descriptions and removed the "note: this description is not yet complete" for the Large-scale GP project (looked pretty complete to me). If it's not, please complete it.
Thanks!

Thanks!
On Mon, Feb 22, 2016, 16:53 Lea Goetz<EMAIL_ADDRESS>wrote:

Closed #3001 https://github.com/shogun-toolbox/shogun/issues/3001.
—
Reply to this email directly or view it on GitHub
https://github.com/shogun-toolbox/shogun/issues/3001#event-560481326.


Translate quint select

Towards #2437, #2479
As discussed synchronously, add an in-code translation for select/SelectSeq.

[x] Tests added for any new code
[x] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] ~Documentation added for any new functionality~
[ ] ~Entries added to ./unreleased/ for any new functionality~

Codecov Report

Merging #2483 (36e6421) into main (a398cff) will increase coverage by 0.00%.
The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main    #2483   +/-   ##
=======================================
  Coverage   78.36%   78.36%
=======================================
  Files         441      441
  Lines       15435    15436    +1
  Branches     2505     2505
=======================================
+ Hits        12095    12096    +1
  Misses       3340     3340

Impacted Files
Coverage Δ

...ain/scala/at/forsyte/apalache/io/quint/Quint.scala
83.87% <50.00%> (+1.16%)
:arrow_up:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Added the image uploads in forms tutorial

The tutorial was sorted chronologically : image uploads in forms by @lb-
Closes #5284
Thanks for contributing to Wagtail! 🎉
Before submitting, please review the contributor guidelines http://docs.wagtail.io/en/latest/contributing/index.html and check the following:

Do the tests still pass? (http://docs.wagtail.io/en/latest/contributing/developing.html#testing)
Does the code comply with the style guide? (Run make lint from the Wagtail root)
For Python changes: Have you added tests to cover the new/fixed behaviour?
For front-end changes: Did you test on all of Wagtail’s supported browsers? Please list the exact versions you tested.
For new features: Has the documentation been updated accordingly?

@a-mere-peasant - what name did you want on the contributors list?

@a-mere-peasant  - this has been merged in, thanks for getting those fixes up
let me know if you want your name revised in the contributors list - just @ me on this PR
https://github.com/wagtail/wagtail/commit/1eae1e46141a252e79dd2eca78462ee9fe1ce789


Placement Attribute of the Modal Component doesn't work

I encountered an issue where the placement attribute of the Modal component is not functioning as expected. Here are the details:
I implemented the Modal component as follows:
<Modal
  show={false}
  position="top-right"
  onClose={onClose}
>
  {/* ...data... */}
</Modal>

However, regardless of the specified position attribute, the Modal continues to appear in the default position. I have also checked the official documentation page for the Modal component at flowbite-react.com/modal, but the mentioned feature does not work there either.
This issue needs attention and should be addressed to ensure the correct functioning of the placement attribute in the Modal component.

Yeah, this is consistent with the documentation: https://flowbite-react-phi.vercel.app/docs/components/modal
We have an open PR that will fix this #766 and I'm waiting for cooperation with the user who submitted that since we just totally rewrote most of the code in this repository. You'll see here once that's been merged.


TextNode::getWholeText does not preserve leading newline after  tag

If we parse the fragment
\n\tfoo \t\n
, and then call getWholeText() on the text node, the leading \n will be missing from the returned value. getWholeText() seems to work as documented for all tags (i.e. Get the (unencoded) text of this text node, including any newlines and spaces present in the original.), except
.
Here's a test program that demonstrates the issue:
import org.jsoup.nodes.*;
import org.jsoup.parser.*;
import org.jsoup.select.*;

public class Test2 {
    public static void main(String[] args) {
        String expectedText = "\n\tfoo \t\n";

        HtmlTreeBuilder treeBuilder = new HtmlTreeBuilder();
        Parser parser = new Parser(treeBuilder);
        Document document = parser.parseInput("
" + expectedText + "
", "");
        NodeTraversor.traverse((Node node, int depth) -> {
            if (node instanceof TextNode textNode) {
                String actualText = textNode.getWholeText();
                printText("Expected: ", expectedText);
                printText("  Actual: ", actualText);
            }
        }, document);
    }

    static void printText(String prefix, String text) {
        System.out.print(prefix);
        for (int i = 0; i < text.length(); i++) {
            char chr = text.charAt(i);
            switch (chr) {
                case '\n': System.out.print("[\\n] "); break;
                case '\t': System.out.print("[\\t] "); break;
                default: System.out.printf("[%c] ", chr);
            }
        }
        System.out.println();
    }
}

And the difference between the expected and the actual results:
$ java -cp ~/.m2/repository/org/jsoup/jsoup/1.15.4/jsoup-1.15.4.jar Test2.java
Expected: [\n] [\t] [f] [o] [o] [ ] [\t] [\n]
  Actual: [\t] [f] [o] [o] [ ] [\t] [\n] # Here the leading \n was not preserved by getWholeText()

Yes, this is deliberate. Per the HTML spec, if the first character of a pre tag is a newline, it gets dropped from the element's text node.
Could you clarify how this impacts you? Perhaps there is a workaround or another approach.


[feature] Support multiline regex in text filtering

It seems that Trigger/wait for text, Ignore lines containing, Block change-detection while text matches in Text filtering section do not support multiline regex.
Narrowed it down to:
https://github.com/dgtlmoon/changedetection.io/blob/5dea5e1def0fe54dd5d913cadfa74097e61b88e3/changedetectionio/html_tools.py#L365-L403
The function iterates over the content line by line and matches each regex to each line:
for line in content.splitlines(keepends=True):

The function could be reworked to use re.finditer/re.findall on the whole content instead.

it COULD be reworked, but then it would maybe break all existing filters, whats your thoughts on how to handle that?

Unless I'm missing something it would only break regex filters that have s or m flags set (currently those flags have no effect) or regex that captures \n in the middle of the pattern (currently such regex matches nothing). Everything else should behave the same.
Other option is to match on the whole content only when the s or m flag is set, otherwise use the current implementation.

aaah so if the regex ends in \m or \s then switch to multiline (regex the blob) instead of processing each line

Downsides are that you have to supports two versions of text filtering, and that filters that already set s or m flags could break.

There's a few tasks to achieve this
https://github.com/dgtlmoon/changedetection.io/blob/master/changedetectionio/html_tools.py#L365 supports this "line number" mode which is used to render the 'preview' diffs of the filter section
also lots of small references for checking if the content changed or not uses the line number mode and needs to be refactored

Yes, but it should not be hard to support line mode with multiline regex, you just count the number of new lines up to each match start/end. And since re.finditer returns matches in order you can optimize the new lines count by starting from the previous match instead of from the content start.