It is possible to automate checking of PDF for certain words/phrases?

user query

Yes of course.

If the PDF is image-based (scanned images of text but not real text), it is harder but if the quality of image is high and the text is printed fonts, then OCR (optical character recognition) can be used to retrieve it.

If the PDF contains text (you can select, highlight, copy out etc), then even easier and more accurate.

From googling, there are some software that is designed just for extracting text from PDF, though most are paid commercial software -

https://docacquire.com/resources/blog/extract-text-from-pdf/
https://docparser.com/blog/extract-data-from-pdf/
https://www.pdftron.com/document-understanding/
https://www.ghostscript.com/faq.html (free open-source tool)

Otherwise TagUI for Python or TagUI can also do that. I'm not sure how the other software do it and how good they are. For this eg it takes 5 lines of instructions to do that. eg below will extract the text.
t.run('start file.pdf')
t.keyboard('[ctrl]a')
t.keyboard('[ctrl]c')
variable = clipboard()
t.click('close_pdf.png')

The tedious part is writing the 'business logic' what text you want to extract, and maybe what combinations of text etc etc. This may be very easy or very difficult depending on what is needed and how complicated the PDF contents is (eg giving false positives on what you are searching for due to occurrence at different part of the PDF for the wrong context etc).

reply from user

Wow! Amazing stuff! Thanks for the advice! One of the things I have to do in my job is a daily audit of medical notes to make sure certain documentation is done. So far have been reading through myself everyday and it’s getting quite a chore 😅

my reply

Welcome! let me know if you have more questions..
if using the open-source software that i created, i can advise more.. what you are saying should be able to be automated. some considerations to think of are -

whether IHIS or the IT policy in your hospital allows staff installing software on their laptops (need to try then know). the software i did no need installation but when I tried on SingHealth laptop, their policy block some components from running, so need to route through central IT for clearance.

if what you are doing can be specified strictly as rules, then it should be doable. however, in practice, depending on the level of difficulty, it can be very tedious to write out all the rules for the computers. though instinctively the maybe 100 rules to you as the expert comes to you in a blink of an eye.

there is a more modern branch of AI using machine learning and classification. that approach may work for some situations but there is always probability of being wrong and having a certain confidence level for each decision. because it is not based on specific rules given by users like 2, but base on the computer analysing a set of PDF files with annotations which one pass which one fail etc. then computer draw its own conclusions base on the statistics of the set of PDFs that is given.


dynamic profiles update

Both PowerShell and Command Prompt for Visual Studio dynamic profiles can be disabled.

Docs Build status updates of commit 2614382:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

TerminalDocs/customize-settings/startup.md
:white_check_mark:Succeeded
View

TerminalDocs/dynamic-profiles.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


W-008542: schedule seasonal addresses updates from install script.

@ceiroa Why did you need to do this?  does HEDAP no longer use our Schedulable object?  If not, how will you avoid push upgrade issues?

@davidhabib ~ Good point. We considered it unnecessary to use the Schedulable framework that we have in HEDA, given that the limits have increased to 100 jobs in the queue, and that we only have schedulable. But we should still instantiate the class dynamically to avoid locking it (or avoid having to perform two push upgrades, one to cancel the job and another to update the class and reschedule the job). I have created a work item for it: https://foundation--agf.na6.visual.force.com/apex/adm_userstorydetail?id=a2x800000001h9lAAA


KeyError:'model'

when i run demo.py ,i met a problem:
File "demo.py", line 194, in <module>
    fasterRCNN.load_state_dict(checkpoint['model'])
KeyError: 'model'

i wanna to know how to solve this question. Thanks

@buptwxl I think you are loading the wrong pertrained model. Please download the models linked in the first column of the table.


Delete “External Installation Planner Protocol” log

How to categorize this PR?

/kind bug
/area os
/os garden-linux
What this PR does / why we need it:
Create a clean image
Which issue(s) this PR fixes:
Fixes #626
Special notes for your reviewer:
Release note:

/lgtm

@zobelhelas Command /lgtm is not available to you but only to a Maintainer, Member.

/lgtm


Fix typo in type name

👀

Must be missing an optional file field with new networking stack because the files won’t load.
Sent with GitHawk

I don't get it. 😛

typeo 🤷♂
(sorry)

Talking to myself, I can’t load the files on GitHawk.

Sent with GitHawk

Ah I'll hunt.

oops i broke pods again

@BasThomas mind backing out the pods changes? Then I can merge this.

Yeah I’ll pick up on Tuesday!
Sent with GitHawk

Oops, I did something wrong. To be honest I don't get what I need to do here to fix this? 😕

You can just rebase and when it says there are conflicts run pod install again. We just want to update the local pod tho so it picks up the file name change.
Sent with GitHawk

Cleaning up & closing this out, low hanging fruit for someone that wants to start contributing


Update getting-started.md

lit-html and lit-element refer to an older version of Lit library.
What I did

I updated the reference to the base library in the last section of the text. The original document still mentions lit-html and lit-template, but they were upgraded to Lit  in the @open-wc generator.

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


Automated cherry pick of #6193: baremetal: find baremetal add scope system

Cherry pick of #6193 on release/3.2.
#6193: baremetal: find baremetal add scope system

Codecov Report

Merging #6208 into release/3.2 will increase coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           release/3.2    #6208   +/-   ##
============================================
  Coverage         7.04%    7.04%
============================================
  Files              646      646
  Lines           110882   110882
============================================
+ Hits              7813     7815    +2
+ Misses          102467   102466    -1
+ Partials           602      601    -1

Flag
Coverage Δ

#aFlag
7.04% <ø> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
70.58% <0.00%> (+2.94%)
:arrow_up:

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 af9bf35...79cecc5. Read the comment docs.


BSQL Q18 fails multi-node SF10K with UVM on

BSQL Q18 fails at SF10K on the 17-node cluster with UVM on ("managed" allocator for blazing)

This should have been closed by #82


Fixes #430: MD5 hashing should be done to the real body, not an escaped one

Resolves compatibility issues with Amazon PHP SDK for SQS.

Ah, great. Thanks!


chore: added the 'REFERENCE' word in the projects list so that it becomes easy to know the ID

What kind of change does this PR introduce?
In this PR, I added the 'REFERENCE' word in the projects list so that it becomes easy to know what is ID actually because when I was trying to delete the project, I was not sure what to enter in the ID.
The ID could be anything but it's good to mention that the reference ID is required.
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage remained the same at 58.587%

Files with Coverage Reduction
New Missed Lines
%

internal/gen/keys/keys.go
5
12.31%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5704

Relevant Lines:
9736

💛  - Coveralls

Thank you, @sweatybridge for merging this pull request!


new error: 'year' not found

https://github.com/drofphilosophe/USElecData/blob/6e5bed9ba5b337cd9fb5ac46be8df4f7b64a1a40/src/EPA-CEMS/loadCEMSData.R#L121
good news branch is working! bad news (not actually bad) when i run this code block i get "Error in group_by(Year, Quarter) (loadCEMSData.R#121): object 'Year' not found"
which is strange because i can see that the Year is an integer variable in input.file.log. so i guessed that it wasn't piping properly, and added a %>% at the end of line 120 and that seemed to fix it.

I've removed a line of diagnostic code that was causing the error. I'll go ahead and merge this into main.


Rough Implementation #RFC

Created a CPI source which keeps track of inflation
MarketOracle returns inflation-adjusted exchange rate and volume

For consistency and for being a better longer term solution I think we should have a whitelist similar to MarketSources  and we can use median.

Agreed 👍
Closing this!


v1.14 Backports 2024-10-07

[ ] #35033 (@WeeNews) :warning: resolved conflicts

:information_source: Hit minor conflict due to different surrounding context in service/id_test.go. Accepted combination.

[ ] #35109 (@jrajahalme) :warning: resolved conflicts

:information_source: Hit minor conflict due to different surrounding context, accepted combination. Additionally updated the TestMapState_AccumulateMapChangesOnVisibilityKeys test, which has been removed in the meanwhile on main, applying the changes originally part of https://github.com/cilium/cilium/commit/137abf7b4b6a (from https://github.com/cilium/cilium/pull/35109).

[x] #35262 (@aanm) :warning: resolved conflicts

:information_source: Hit minor conflict due to different surrounding context (i.e., missing Update Protobuf APIs step); adapted accordingly.

[x] #35264 (@aanm)
[x] #31538 (@ti-mo, @aanm)

Once this PR is merged, a GitHub action will update the labels of these PRs:
 35033 35109 35262 35264 31538

Fixed a unit test error caused by the backport of https://github.com/cilium/cilium/pull/35109, and additionally backported https://github.com/cilium/cilium/pull/31538 (first commit only).

I've double-checked the remaining commits, marking ready to merge.


StreamTransformer.bind dartdocs should be fleshed out

The documentation for StreamTransformer.bind says that the pause state should be propagated, but it doesn't say how to handle error events from the incoming stream. Should error events be propagated? Should they be ignored? Should they cancel the output? Are they a valid subject for transformation? What are the best practices for writing transformers?

It doesn't say anything, because there is nothing special to say - any function from Stream to Stream can be used. Even the comment about propagating pause is general stream behavior - if you are listening on a stream and don't want new events for a while, pause the subscription until you do.
All events are valid targets for transformation because you can do anything at all. What you do with them depends on what you want the transformer to do. The general rules for streams apply to the output stream you create (basically: don't forget to close it when you are done).
Best practice for writing transformers is to figure out what the transformation should do - what is the connection between input and output stream events - and then implement that.

Just pasting that comment verbatim into the docs would already be a huge improvement. :-)


No custom emojis in Placeholder for Select menues

Summary
Can't use custom emojis in placegolders
Reproduction Steps
Create a Select Menu
insert the custom emoji into Placeholder
Minimal Reproducible Code
emoji = client.get_emoji(911974529784479814)
        print(emoji)
        super().__init__(placeholder=f'Wähle ein Bundesland aus {emoji}', min_values=1, max_values=1, options=options)

Expected Results
show thie custom emoji
Actual Results
shows the emoji string e.g:
<:Deutschland:911974529784479814>
Intents
all
System Information

Python v3.9.9-final
nextcord v2.0.0-alpha

nextcord pkg_resources: v2.0.0a3

aiohttp v3.6.3
system info: Windows 10 10.0.19042

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
Idk whether thats possible but normal emojis work so i guess custom ones should work as well

this is most likely a discord limitation as nc never renders the emoji string but discord does

this is most likely a discord limitation as nc never renders the emoji string but discord does

yeah probably


Formatter: don't add <?php open tag if not exists in code snippet

In Code Snippet:

keep <?php open tag  if exists
don't add <?php  if not exists

All checks have passed green 🎉 . Applied in both Markdown and HeredocNowdoc formatter. @TomasVotruba ready for review

I've added handle multi new line before/after code inside code snippet to be trimmed and correctly checked.

Thank you


Adding VPC endpoint support

This PR adds support for VPC endpoints as per the documentation here:
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-ec2-vpcendpoint.html

@markpeek Not sure how busy you are, but I sure would appreciate a new release of troposphere once this is merged. I need it for a consulting client next week. Thanks mate!

Actually, I'll close this PR and let's use #304 as @kalefranz got there first with the full implementation.

@pas256 Due to a vacation I'm a bit behind in consolidating the various pull requests and other changes. I'll try to get out a new release before the end of day tomorrow. Cheers.

You rock!


crashplan is broken

Issue description
Crashplan for home users has been shut down as of October 23, 2018.
Crashplan for small business current installer has been removed from download list, and needs to be updated.
Steps to reproduce
Enable services.crashplan or services.crashplansb.
Technical details

system: "x86_64-linux"
host os: Linux 4.14.65, NixOS, 18.03.133126.4df3426f5a5 (Impala)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.0.4
channels(root): ""
nixpkgs: /nix/store/s7wklxxa5xw0sp1rmmgrck9vag7811ph-nixexprs.tar.xz

So nixos/modules/services/backup/crashplan.nix and pkgs/applications/backup/crashplan/default.nix can be removed?

sounds like a plan.

Can you have a look at the download:
trying https://web-eam-msp.crashplanpro.com/client/installers/CrashPlanSmb_6.7.0_1512021600670_4503_Linux.tgz
    % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                   Dload  Upload   Total   Spent    Left  Speed
    0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  curl: (22) The requested URL returned error: 404 Not Found
  error: cannot download CrashPlanSmb_6.7.0_1512021600670_4503_Linux.tgz from any mirror
cannot build derivation '/nix/store/lnsaiyhw828k9raifspcasrrncv6wpa0-CrashPlanSmb_6.7.0_1512021600670_4503.drv': 1 dependencies couldn't be built

It probably just needs an upgrade.

Yep, they silently replace the installer with new versions. I'll fix and PR.

Ok. I marked it as broken for the moment: https://github.com/NixOS/nixpkgs/pull/51618


Format of acceptedTokens when calling register does not follow the specification

Describe the bug
As per the specification presented here, the format of the POST HTTP call body to the DBS on the /register endpoint should look like:
{
      "type": "filecoin",
      "description":  "File storage on FileCoin",
      "url": "http://microservice.url",
      "payment":[
        {
          "chainId": "1",
          "acceptedTokens": [
            {
              "OCEAN": "0xOCEAN_on_MAINNET"
            },
            {
              "DAI": "0xDAI_ON_MAINNET"
            }
          ]
        }
      ]
    }

Right now it looks like:
{
      "type": "filecoin",
      "description":  "File storage on FileCoin",
      "url": "http://microservice.url",
      "payment":[
        {
          "chainId": "1",
          "acceptedTokens": "0xOCEAN_on_POLYGON"
        }
      ]
}

Please adjust to the desired format :-)

Thanks for your Feedback. This was treated in PR #18


docs: 📝 added ai tools in Legal Assistant

Description

I have added ai tools in Legal Assistant.
I have removed an extra #art link from readme.md

Checklist
Before submitting this pull request, please ensure that you have done the following:

[x] Read and understood the Contributing Guidelines.
[X] Checked that your changes align with the repository's goals and content.
[X] The submission is not already present in the list

Related Issues
If this pull request resolves or is related to any issues, please mention them here using the following format: "Fixes #123" or "Relates to #456."
Additional Notes (if applicable)
Add any additional information or context that may be relevant to this pull request.

By submitting this pull request, you confirm that you have read and agree to abide by the Code of Conduct and the Contributing Guidelines of the AI Tools Collective project.

You did great, thanks for contributing, I hope you will stick around and continue to contribute to this project.


[CP] Fix Snackbar TalkBack regression

issue_link
b/261136371
Commit Hash
https://github.com/flutter/flutter/commit/22cbef305035573671297172c9204f01d210d380
Target
beta
pr_link
https://github.com/flutter/flutter/pull/116608
Impacted Users
All end-users using TalkBack
Impact Description
This PR fixes a regression in TalkBack functionality in the SnackBar.
Workaround
There is no workaround.
Risk
low
Test Coverage
yes
Validation Steps
Ensure TalkBack works with SnackBars.

Friendly bump

The CP PR was already merged, so I think this can be closed.


About evaluation

HI, could you please share the script for evaluation like FVD?

HI, could you please share the script for evaluation like FVD?

Please refer stylegan-v.

Thank you very much. I have another question about what is difference between vae and sd-vae-ft (https://huggingface.co/maxin-cn/Latte/tree/main/sd-vae-ft-ema)? Which version of vae you used in Latte?

Thank you very much. I have another question about what is difference between vae and sd-vae-ft (https://huggingface.co/maxin-cn/Latte/tree/main/sd-vae-ft-ema)? Which version of vae you used in Latte?

Both vae can be used in Latte.


Contact Us - Link to smartsheets issue intake form

Problem statement
As an help desk analyst, I need the onboarded sender/receivers to be able to create their own support tickets by going to a link on reportstream.gov site to access the existing smart sheet form.
The purpose of this work is to avoid manual work being undertaken currently by heldesk analysts in monitoring the shared reportstream.gov mailbox and creating tickets in smartsheet as well as getting all required information upfront from the user without additional emails back and forth.
What you need to know
Link to the smartsheet form
https://app.smartsheetgov.com/b/form/ff33efa457be461c9893301d4c0ec12d

@drew-usds is checking to see if the form can be embedded on the reportstream.gov website.
If the form cannot be embedded, the link will send the user directly to the smartsheet.gov site.

Acceptance criteria

A user is able to click on the Contact Us link and access the smartsheet support site either via embedded content or being sent directly to https://app.smartsheetgov.com/b/form/ff33efa457be461c9893301d4c0ec12d

Additional test - this is not part of A/C for this ticket but should be tested for feature completion

Submitting the support ticket with all required fields creates a support ticket in smartsheet

To do

cc @drew-usds @bgantick @a-chappelear @HeatherRienks

Duplicate of #6588


Publish views, controllers and models

Is there a way to publish the views?
It would be better to have full control over views, controllers and models instead of keeping it inside the package folder.
Thanks

Yes please

This have been fixed in the latest version.
Now you can run php artisan vendor:publish.

/close please


Wish list: Add the rowStats attributes of the columns of the sys.tasks table

Specifically processed, processedWithError, thrownAway, unparseable
This is currently available from the coordinator task report API but it would be amazing to be able to surface it via DruidSQL
"rowStats":{"buildSegments":{"processed":3630,"processedWithError":0,"thrownAway":0,"unparseable":0}}
Then we could use all the cool SQL magic on it like filtering.
In particular we could surface it in the new Druid console Tasks view as a column:

This will prevent the common confusion of "my task succeed but my where is my data (0 rows were ingested)

To get rowStats attributes of a task, the overlord api is documented here. In order to get rowStats for each task, we'll have to make a call to this api per task as part of the query. This could potentially slow down retrieval of tasks substantially because it would introduce a network hop for each task. We can speed up the queries by adding a cache, but not sure if it's worth the memory cost, the cache might add.


[RDBMS] Add support for advanced threat protection setting for a PostgreSQL flexible server

Related command
az postgres flexible-server advanced-threat-protection-setting update --resource-group testGroup --server-name testserver --state Enabled/Disabled' to enable/disable state for advanced threat protection setting. az postgres flexible-server advanced-threat-protection-setting show --resource-group testGroup --server-name testserver` to show the advanced server threat protection setting
Description
Added support for Advanced Server Threat Protection Setting for PostgreSQL flexible server. It supports commands to update and show the advanced threat protection setting
Testing Guide
Added automated tests
History Notes

[RDBMS] az postgres flexible-server advanced-threat-protection-setting show: Show advanced threat protection setting
[RDBMS] az postgres flexible-server advanced-threat-protection-setting update: Update advanced threat protection setting using --state as Enabled/Disabled

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

RDBMS

Kind reminder that our code complete date is 11/28/2023 10:00 UTC. Please address comments and CI failures in time if you want to catch November 2023 (2023-12-05)

Kind reminder that our code complete date is 11/28/2023 10:00 UTC. Please address comments and CI failures in time if you want to catch November 2023 (2023-12-05)

I have addressed comments and CI failures. PR is ready for merge


Rename this project

i'm the only r2 in this place

I prefer request2 instead of r2.
It also direct refer to request library, which this one suppose to replace.

r2 is a well-known LTR CSS converter https://github.com/ded/R2, fwiw

I have rz and sr npm packages available. For this or any project who requires the name.

maybe rqst?

fetchquest

How about req2

How about potato

@qaisjp unfortunately potato is taken on npm... :(
https://www.npmjs.com/package/potato

refetch

What about pota2o
On Tue, 5 Sep 2017, 1:09 pm Alejandro Garcia Anglada <
<EMAIL_ADDRESS>wrote:

refetch
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mikeal/r2/issues/7#issuecomment-327144535, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA4WamtgYMKTZI6movNAM46s6BMZyHiaks5sfSvbgaJpZM4PKKmh
.

r2q2 ¿?

refetch

Already taken. 😞

How about req2

Already taken.

I prefer request2 instead of r2.

And request2` is already taken as well.

maybe rqst?

Already taken.

So, of all the suggestions here, the only ones currently available on NPM are:

fetchquest
r2q2

I'll also throw in a few other available ones:

reqfetch
fetchreq

what about rq2? it's available on NPM

radquest
(rad is always rad, but also loosely based on http://www.dictionary.com/browse/red-)

Also: restquest

or maybe roquest
ro seems to mean two in Javanese, which is kind of cool I think, since it's a JavaScript libary

or maybe roquest
ro seems to mean two in Javanese, which is kind of cool I think, since it's a JavaScript libary

Keep the name the same, r2 is fine.

I prefer request2.
My ideas:
mikerequest
richrequest
requester
request-tool

The problem is that it isn’t available on NPM. @mikeal contacted the owner of the current(not active) r2 package on NPM asking for a transfer.
Another problem with the name is that r2 is a well known LTR CSS converter. Look at the discussion above.
On 12 Sep 2017, 20:54 +0300, Alex Crocker<EMAIL_ADDRESS>wrote:

Keep the name the same, r2 is fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

or r-2??

for the record, this is the best thread going on right now in github 😆

It seems like many people prefer "request2" (but as @JamesMGreene pointed out, that is taken)
(I also think it would be a great name, as it is descriptive and seems to refer to the original package.)
So what about "request-2"? It has all the pros of "request2" but is available on npm...

what about scoped packages? this way you can choose whatever name you want and have the prestige of your own organization :-)

How about request3 request4 ...

Reply to this email directly, view it on GitHub
https://github.com/mikeal/r2/issues/7#issuecomment-328973368, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AATMYKy7MGFOZFnw1TsV7pw2ShEitZYCks5shugVgaJpZM4PKKmh
.

How about re-request ? :neutral_face:
Thanks for your amazing job, But please specify the package name in readme of repo. I lost a lot of time finding package name, I'm sure other people will do same.

request-2 seems the best. Also, how about 2quest or toquest?

What about: potatothisthingisnotbeingrenamedpleasestoppleasepleaseplease
On Tue, 12 Sep 2017, 2:38 pm Yoav Maman<EMAIL_ADDRESS>wrote:

what about rq2? it's available on NPM
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mikeal/r2/issues/7#issuecomment-328852509, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA4WatIY9H3DuILstigwkDk8OeaWeHyqks5shoZkgaJpZM4PKKmh
.

rquest

This may be an instance of bikeshedding.
@mikeal If you have no interest in renaming this please say so and save everyone the energy.
Personally I think request-2 > r2 as it has some semantic significance, but really it's not of utmost importance

@mattbrunetti The author of this repository has not commented here, so it's not like he's deliberately trying to mess with all of you.
 I guess you could take that to mean "i don't care"

Published as r2 https://www.npmjs.com/package/r2


[Docs] Fix - Nullable reference types not always shown in API documentation

Pull Request
📖 Description
API documentation did not always show nullable type for parameters or method return.
🎫 Issues

Only the underlying type shown even though it was declared nullable. It does not match the source code.

Before

After

👩💻 Reviewer Notes
Changes were made to the ToTypeNameString extensions instead of APIDocumentation. Only concern I have is that the original extensions were sourced from https://github.com/loxsmoke/DocXml.
📑 Test Plan

I manually checked many of the components.

✅ Checklist
General

[ ] I have added tests for my changes.
[X] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[X] I have read the CONTRIBUTING documentation and followed the standards for this project.

⏭ Next Steps

Changes were made to the ToTypeNameString extensions instead of APIDocumentation. Only concern I have is that the original extensions were sourced from https://github.com/loxsmoke/DocXml.

Hi Gary (@oneolddev). Why does that concern you? We used that as the base back then but have made significant changes along the way. We are not going back anytime to copying the original code again, I think.


CSS variable --color-icon-tertiary and more are gone 😩

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.

Description
Particularly noticeable in dark mode since our defaults are quite bright sometimes.
For example the author border and conflict marker should not be so bright here:

Also related, contains a similar fix:

https://github.com/sindresorhus/refined-github/pull/4842

Screenshot
No response
Console errors
No response
Example URL
https://github.com/fregante/github-sandbox/pulls?q=is%3Apr+is%3Aopen+sort%3Aupdated-desc+conflict
Browser(s) used
safari

Several screenshots with the expected colors can be found in:

https://github.com/sindresorhus/refined-github/pull/3806


Missing validation error message for numeric inpouts

Describe the Bug
When validating numeric fields, there's no error message in the configuration form.
Steps to Reproduce the Bug or Issue
1
Screenshots or Videos

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 0.30.0]

Thanks @d4n1b for catching this

No worries! I created another PR to add a small test coverage


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_ad7vnmo9fv/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_ad7vnmo9fv/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_ad7vnmo9fv/clone/recipe/meta.yaml:54: missing_license_family: The recipe is missing the about/license_family key.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_03vxaneb6c/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_03vxaneb6c/clone/recipe/meta.yaml:32: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_03vxaneb6c/clone/recipe/meta.yaml:54: missing_license_family: The recipe is missing the about/license_family key.
Errors were found


Experiment request: DPO with different betas

TLDR: How much does beta impact a DPO model accuracy by controlling KL distance?

Closing. No longer logging experiments here.


Add parameter into DataBaseRenderer

In org.mapsforge:mapsforge-map:0.8.0, new fuctionality "HillShading"
was introduced. DataBaseRenderer needs seventh
parameter "HillsRenderConfig".

Thanks for the PR but compile failed...
:osmdroid-mapsforge:compileReleaseJavaWithJavac - is not incremental (e.g. outputs have changed, no previous execution, etc.).
/home/travis/build/osmdroid/osmdroid/osmdroid-mapsforge/src/main/java/org/osmdroid/mapsforge/MapsForgeTileSource.java:26: error: cannot find symbol
import org.mapsforge.map.rendertheme.InternalRenderTheme;
^
symbol:   class InternalRenderTheme
location: package org.mapsforge.map.rendertheme
/home/travis/build/osmdroid/osmdroid/osmdroid-mapsforge/src/main/java/org/osmdroid/mapsforge/MapsForgeTileSource.java:97: error: cannot find symbol
xmlRenderTheme = InternalRenderTheme.OSMARENDER;
^
symbol:   variable InternalRenderTheme
location: class MapsForgeTileSource
/home/travis/build/osmdroid/osmdroid/osmdroid-mapsforge/src/main/java/org/osmdroid/mapsforge/MapsForgeTileSource.java:133: error: package InternalRenderTheme does not exist
return new MapsForgeTileSource(InternalRenderTheme.OSMARENDER.name(), minZoomLevel, maxZoomLevel, tileSizePixels, file, InternalRenderTheme.OSMARENDER, MultiMapDataStore.DataPolicy.RETURN_ALL, null);
^
/home/travis/build/osmdroid/osmdroid/osmdroid-mapsforge/src/main/java/org/osmdroid/mapsforge/MapsForgeTileSource.java:133: error: cannot find symbol
return new MapsForgeTileSource(InternalRenderTheme.OSMARENDER.name(), minZoomLevel, maxZoomLevel, tileSizePixels, file, InternalRenderTheme.OSMARENDER, MultiMapDataStore.DataPolicy.RETURN_ALL, null);
^
symbol:   variable InternalRenderTheme
location: class MapsForgeTileSource
Note: /home/travis/build/osmdroid/osmdroid/osmdroid-mapsforge/src/main/java/org/osmdroid/mapsforge/MapsForgeTileSource.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
4 errors
:osmdroid-mapsforge:compileReleaseJavaWithJavac FAILED

again, love to merge this but there are a number of compile errors

Thanks for your reply! Didn't know the compile test. I will take time to improve it.

https://github.com/mapsforge/mapsforge/issues/980 opened a ticket over a forge to find out what happened with the class that was removed

just committed the fix for this on branch feature/PR#597 and will reopen a PR for that branch. Closing this one. Thanks for the submission


Server took too long to respond: DeadlineExceededError: The overall deadline for responding to the HTTP request was exceeded.

Status = Error
Progress = 4695
Server took too long to respond: DeadlineExceededError: The overall deadline for responding to the HTTP request was exceeded.
Please report this error at github.com/Google-Tasks-Backup/tasks-backup/issues

This error indicates that GTB was unable to retrieve all the user's tasks within the maximum 10 minutes allowed.
GTB can usually retrieve up to ~20,000 tasks from the Google Tasks server in the allowed 10 minutes, but it appears that in this case, the Google Tasks server was running very slowly, and only returned 4,695 tasks during that time.
I can only suggest trying again later, hoping that Google fixes the problem with their server.

I have increased the App Engine Instance Class and enabled Basic Scaling, so that the process that retrieves tasks will now run for more than 10 minutes.
This should allow GTB should to retrieve all tasks, even if the Google Tasks Server is running slower than normal, or if a user has a very large number of tasks.

I have increased the App Engine Instance Class and enabled Basic Scaling, so that the process that retrieves tasks will now run for more than 10 minutes.
This should allow GTB should to retrieve all tasks, even if the Google Tasks Server is running slower than normal, or if a user has a very large number of tasks.


feat: Add option to set closing tag virtual text priority

This PR adds an option to set the priority for the closing tags virtual text.
This would help keep the order of virtual text items configurable, in cases when there are multiple virtual text items, eg.: git blame text from the gitsigns plugin or with LSP diagnostics text.
Example:
Current behavior (not preferable, especially in cases when commit messages are longer):

When the priority is set (to 0, in this case) (preferable, as the tag is always kept next to the actual code):

I've kept the default priority to be 0, as I think logically one would want this text to be closest to the code. Open to discuss if there are any other thoughts on this.
Do let me know your thoughts on this. If this looks like a good change to have, I can add this in the docs (vimdoc and README) as well.
Thanks!

@devansh08, thanks for the PR! The change looks good. Yes, documentation is needed, and it should be fine to merge.

Docs updated!


[various] Enable avoid_dynamic_calls

Enables the avoid_dynamic_calls lint, to bring us into better alignment with flutter/flutter lints.
There are some miscellaneous changes, but the large majority of these are cases where platform channel code was using dynamic values directly (usually for method call arguments). Common changes:

Converting dynamic to a Map for method call arguments.
Marking force-unwraps that were not previously flagged because someDynamicVarible as NonNullableType compiles, while someObject?Variable as NonNullableType doesn't and needs an explicit force-unwrap (due to another lint we already have enabled).

Since these changes aren't mechanical, and can cause runtime exceptions, they should be reviewed carefully (as we may well not have complete test coverage of all touched lines). In particular, please check that any as List<...> or as Map<..., ...> isn't using any types other than dynamic or Object?, as anything else is likely to be an exception-causing bug. (The correct construction for better typing is something like (... as Map<Object?, Object?>).cast<AType, AnotherType>().)
In general I preferred using Object? for updated code, per modern Dart style, but in some cases where there was already significant explicit use of dynamic I kept that style to minimize disruption. We may want to separately clean up areas of code still using dynamic at another time. Although ideally, we would replace most of the code here with Pigeon and avoid the need for manual, error-prone casting at all.
Part of https://github.com/flutter/flutter/issues/76229
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

I'll reconcile the webview parts of this on my side once webview v4 lands, but I wanted to get this completed and out for review since it touches a bunch of things.

Oops, forgetting to check the repo tooling locally strikes again...

Though there are still some analyzer errors.
   info - lib/src/common/git_version_finder.dart:91:35 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - lib/src/common/pub_version_finder.dart:48:10 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - lib/src/create_all_packages_app_command.dart:284:38 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - lib/src/dependabot_check_command.dart:61:35 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - lib/src/pubspec_check_command.dart:248:11 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - lib/src/pubspec_check_command.dart:268:36 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - lib/src/pubspec_check_command.dart:280:11 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - lib/src/readme_check_command.dart:237:36 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - test/common/git_version_finder_test.dart:27:11 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - test/common/git_version_finder_test.dart:30:18 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls
   info - test/common/package_looping_command_test.dart:114:11 - Avoid method calls or property accesses on a "dynamic" target. - avoid_dynamic_calls

I held off on landing this so as not to disrupt webview v4; I need to update it to include those changes and then it'll be ready to land.


Always show the copilot header buttons

Is there a way to make these buttons always visible?
It's especially bad because the git controls have a similar plus symbol.
I've confused them before and staged a file instead of opening a new copilot conversation.

No, they show on hover, that's just how these buttons work in vscode


The npm package has the .git directory included

When trying to install other packages, npm raises an error and refuses to continue installation:
npm ERR! path ......./node_modules/value-object
npm ERR! code EISGIT
npm ERR! git ....../node_modules/value-object: Appears to be a git repo or submodule.
npm ERR! git Refusing to remove it. Update manually,
npm ERR! git or move it out of the way first.
A temporary workaround is to delete the .git repository existing into the node-modules/value-object repository.
Tested with:
npm version 6.4.1
value-object version: 0.4.1

Thanks @AlvaroMaceda!
This is fixed in 0.4.3


Hand-write CustomComparison and CustomEquality on Timestamp

I hand-wrote the code. Should we generate it instead? But comparison and equality might just use space unnecessarily on other message types.
I implemented GetHashCode naively. Please suggest revisions freely.
Resolves #30.

Thanks to Isaac Abraham at Compositional IT.


Update/Add [Xcode 12.5]

Tool information

Tool name: Xcode 12.5
Tool license: n/a
Add or update?: update
Desired version: 12.5
Approximate size:

Area for Triage:
Apple
Question, Bug, or Feature?:
Feature
Virtual environments affected

[ ] Ubuntu 16.04
[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] macOS 10.15
[x] macOS 11
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Can this tool be installed during the build?
no, it would take far too long
Tool installation time in runtime
20-30 min
Are you willing to submit a PR?
no
Other:
here is a screenshot from a recent build using macOS latest. Xcode 12.5 is missing:

Hi @BrentMifsud,
macos-latest tag still points to macOS 10.15. MacOS 11 contains Xcpde 12.5 but this OS is still in private preview, although we start opening it. You can track progress here https://github.com/actions/virtual-environments/issues/2486.
Closing this issue, please let me know if you have any concerns.
Thanks!

thanks for the heads up. I didnt realize macOS-latest was still pointing to 10.15.


Infinite recursion on moderator report

Opening the moderator report tab results in an infinite recursion.

I can't reproduce this.

I never got the full log but this is a piece he posted
java.util.concurrent.CompletionException: org.springframework.web.reactive.function.client.WebClientRequestException: java.lang.IllegalArgumentException: Infinite recursion (StackOverflow  .............................


Test Prow cherrypicker plugin

/lgtm

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0

/cherrypick release-1.0


[CI] Go back to tarpaulin for coverage

grcov is not improved.
I gave up waiting it improved.

Merging #249 (50a4c68) into main (6f11577) will increase coverage by 13.97%.

🤔


New sign-in link doesn't look right on mobile

Love the new sign in link, however on mobile it looks to be right justified. I'm pretty sure it should either be centered on one line.

Hey @Unreal1david, thanks for the feedback. It's now fixed in 5.4.4.


Live

Docs Build status updates of commit 461d82e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

windows-iotcore/Resources.md
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/HardwareCompatList.md
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/PinMappings/PinMappingsMBM.md
:white_check_mark:Succeeded

windows-iotcore/learn-about-hardware/SoCsAndCustomBoards.md
:white_check_mark:Succeeded

windows-iotcore/mDNS.md
:white_check_mark:Succeeded

windows-iotcore/tutorials/quickstarter/PrototypeBoards.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.


Conform NSError to Debuggable

Fixing annoying non saying errors like this:
[ ERROR ] The operation could not be completed (ErrorMiddleware.swift:26)
[ DEBUG ] Conform `NSError` to `Debuggable` for better debug info. (ErrorMiddleware.swift:26)

Wasn't sure how "unique" the identifier should be, so included a random integer, for the reason just included the debugDescription

What if we set possibleCauses to self.localizedFailureReason and suggestedFixes  to self.localizedRecoverySuggestion? I'm not sure if they will ever be helpful, but it might be.

Hey @MrLotU, you just merged a pull request, have a coin!
You now have 73 coins.

This is causing issue #193


wl-mirror: 0.12.2 -> 0.13.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/Ferdi265/wl-mirror/releases.
meta.description for wl-mirror is: Mirrors an output onto a Wayland surface.
meta.homepage for wl-mirror is: https://github.com/Ferdi265/wl-mirror
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.13.0 with grep in /nix/store/3hlasq9608dd7c1djbc7xsi2hzxx20r2-wl-mirror-0.13.0
found 0.13.0 in filename of file in /nix/store/3hlasq9608dd7c1djbc7xsi2hzxx20r2-wl-mirror-0.13.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
wl-mirror

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/3hlasq9608dd7c1djbc7xsi2hzxx20r2-wl-mirror-0.13.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A wl-mirror https://github.com/r-ryantm/nixpkgs/archive/a697e7965ac31c382e1ded8d631b411c1d6e2c35.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/3hlasq9608dd7c1djbc7xsi2hzxx20r2-wl-mirror-0.13.0
ls -la /nix/store/3hlasq9608dd7c1djbc7xsi2hzxx20r2-wl-mirror-0.13.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

wl-mirror

Maintainer pings
cc @Synthetica9 @twitchyliquid64 for testing.

Result of nixpkgs-review pr 219517 run on aarch64-linux 1

1 package built:

wl-mirror


Add a button in the tracking list for users to manage their own episodes.

I dont want to add extra button to tracking list.
Instead, I used a long-press gesture to implement this feature.
Users might have difficult to discover this feature, so I want to add toturial dialog for it #223 .
https://github.com/user-attachments/assets/9e2780b1-5c5c-4456-a81e-7f56a93848b3

Ohh i think it's okay


Add ddb_item helper macro.

This is a follow-up to #783.

Now that #785 is merged, you can rebase this branch on top of master and it should fix the CI errors.

@adimarco It's done!

Thanks for the PR!
I see the new functionality is in Rusoto helpers, which currently isn't maintainted.  That means we should find a better place to put this.  Perhaps putting it into the DynamoDB crate itself is a good spot: we may be able to put it in rusoto/services/dynamodb/src/custom/mod.rs.
Also, looks like the botocore version was changed in this PR.  We prefer to keep those in separate PRs.
My recommendation is to have a PR with the helper macro in rusoto/services/dynamodb/src/custom/mod.rs.  This will make the PR much smaller and usable without having to drag the rusoto_helpers crate up to date. 😄

Thank you for taking time to have a look at this.  I agree with your recommendations.
I'm super busy right now, but will probably have time to make a new PR in a few weeks.


Add region definitions and mappings for EMPIRE v1.0

fyi @stianbacke

Permissions assigned for EMPIRE modelers to submit scenarios for „EMPIRE v1.0“ to the openentrance-internal Scenario Explorer


I need help.

I'm new to flutter and I'm having trouble

you should post it to stackoverflow

okey, thank you


(feat) (improv) Improves data-conversion feature and groups more functionalities

This PR covers a number of features because they are interrelated we couldn't separate them.
The features included are listed below:

Data Conversion
Splitting RemoteArrays (which is BastionLab's internal Intermediate Representation for conversion).
Updates RemoteDataset architecture.

Data Conversion
In this feature, we convert RemoteDataFrames (DFs) into RemoteTensors.
But before we convert DFs into Tensors, we use an intermediate representation (IR) called RemoteArray
class RemoteArray:
      identifier: str

      def to_tensor() -> RemoteTensor

The conversion logic is currently uni-directional RemoteDataFrames -> RemoteArrays -> RemoteTensors
Splitting RemoteArrays
This features is very similar to sklearn's train_test_split. It split's RemoteArrays into train and test sub arrays
Updates RemoteDataset
The old RemoteDataset in BastionAI used to represent a Dataset remotely but in this new representation, we use RemoteDataset only as a tiny collection of RemoteTensors.

Important Things to Note
For RemoteDataset, we have to implement sending PyTorch Dataset to the server.
Currently, that hasn't been implemented because I haven't found the best way to implement it.
Although, there's an elementary implementation in https://github.com/mithril-security/bastionlab/blob/1a7858c5cffa68484a664fa42f2a9e69a5e5d780/client/src/bastionlab/torch/remote_torch.py#L117.
In that case, send_tensor which RemoteDataset.from_dataset will use it hasn't been implemented
   @staticmethod
    def send_tensor(tensor: torch.Tensor) -> "RemoteTensor":
        raise Exception(
            "Sending tensors to the BastionLab Torch service is not yet implemented"
        )

(was split from #140)

Important Things to Note
For RemoteDataset, we have to implement sending PyTorch Dataset to the server.
Currently, that hasn't been implemented because I haven't found the best way to implement it. Although, there's an elementary implementation in
https://github.com/mithril-security/bastionlab/blob/1a7858c5cffa68484a664fa42f2a9e69a5e5d780/client/src/bastionlab/torch/remote_torch.py#L117
.
In that case, send_tensor which RemoteDataset.from_dataset will use it hasn't been implemented
   @staticmethod
    def send_tensor(tensor: torch.Tensor) -> "RemoteTensor":
        raise Exception(
            "Sending tensors to the BastionLab Torch service is not yet implemented"
        )

is this something we can ship later?

Important Things to Note
For RemoteDataset, we have to implement sending PyTorch Dataset to the server.
Currently, that hasn't been implemented because I haven't found the best way to implement it. Although, there's an elementary implementation in
https://github.com/mithril-security/bastionlab/blob/1a7858c5cffa68484a664fa42f2a9e69a5e5d780/client/src/bastionlab/torch/remote_torch.py#L117
.
In that case, send_tensor which RemoteDataset.from_dataset will use it hasn't been implemented
   @staticmethod
    def send_tensor(tensor: torch.Tensor) -> "RemoteTensor":
        raise Exception(
            "Sending tensors to the BastionLab Torch service is not yet implemented"
        )

is this something we can ship later?

Not really, because this feature would break how we used to send datasets in the old torch codebase.
The implementation has to done, just that I haven't figured out the best way to send the tensors yet.
If you wish, you can implement it.

Check #198 to see why I needed to make the PRs this way.

Now, this has been solved by re-using the client.send_dataset method from the existing code torch client code and then changing how we handle the BastionLab.Dataset on the server.
We use this helper method https://github.com/mithril-security/bastionlab/blob/cleaned-conversion/server/bastionlab_torch/src/lib.rs#L136
And return a new type in the proto
message RemoteDatasetReference {
    repeated bastionlab.Reference inputs= 1;
    bastionlab.Reference labels = 2;

}

TODO: Add more validation checks on train_test_split and verify computation after floor.

Add the remainder to test_set.


Builtins Documentation

Hi @jimenezz22 sure! Notice this issue belongs to a much bigger task #115, so checkout its description to get a better idea of what needs to be done
In the case of builtins is not required for you in this task to write about each one of them. Only a general section describing how they work would suffice.
We have implemented builtins in issue #126. This might be helpful for this task


Draft

PR of old stuff I have on my computer, in redaction

Hello @DaluS! Thanks for opening this PR.

In the file pygemmes/_models/_def_fields.py:

Line 571:21: E241 multiple spaces after ':'
Line 576:21: E241 multiple spaces after ':'

In the file pygemmes/_models/_model_3Layerclimate.py:

Line 3:101: E501 line too long (127 > 100 characters)
Line 3:128: W291 trailing whitespace
Line 5:101: E501 line too long (1202 > 100 characters)
Line 35:9: E266 too many leading '#' for block comment
Line 35:27: W291 trailing whitespace
Line 37:101: E501 line too long (129 > 100 characters)
Line 41:101: E501 line too long (159 > 100 characters)
Line 45:101: E501 line too long (106 > 100 characters)
Line 49:101: E501 line too long (138 > 100 characters)
Line 53:101: E501 line too long (101 > 100 characters)
Line 61:1: W293 blank line contains whitespace
Line 62:9: E266 too many leading '#' for block comment
Line 62:25: W291 trailing whitespace
Line 63:1: W293 blank line contains whitespace
Line 64:1: W293 blank line contains whitespace
Line 65:1: W293 blank line contains whitespace
Line 73:101: E501 line too long (109 > 100 characters)

In the file pygemmes/_models/_model_CopingWithCollapse.py:

Line 3:101: E501 line too long (127 > 100 characters)
Line 5:101: E501 line too long (1202 > 100 characters)
Line 33:101: E501 line too long (129 > 100 characters)
Line 37:101: E501 line too long (159 > 100 characters)
Line 41:101: E501 line too long (106 > 100 characters)
Line 45:101: E501 line too long (138 > 100 characters)
Line 49:101: E501 line too long (101 > 100 characters)
Line 91:101: E501 line too long (111 > 100 characters)
Line 135:101: E501 line too long (109 > 100 characters)
Line 178:101: E501 line too long (102 > 100 characters)
Line 182:101: E501 line too long (103 > 100 characters)
Line 208:101: E501 line too long (164 > 100 characters)
Line 212:101: E501 line too long (124 > 100 characters)
Line 254:15: E241 multiple spaces after ':'
Line 255:15: E241 multiple spaces after ':'
Line 256:17: E241 multiple spaces after ':'

Do see the Hitchhiker's guide to code style


@Logged vs @NotLogged

I'd like to suggest an additional NotLogged annotation that, instead, makes the assumption that all operations are authenticated operations, unless the NotLogged annotation has been added.
As it is now, security is set such that, any missing Logged annotation, could expose the API.  I think this is dangerous, and would much prefer the option of taking the alternate approach to security.

To further expand on this.  A simple BC solution would be to add a new method to the AuthenticationServiceInterface.  Something like function getAuthenticationPolicy(): AuthenticationPolicy.  The AuthenticationPolicy class can define defaults for the auth policy.  This can also support future settings.

FWIW we've put our auth layer higher in the stack and forgone the need for this within GraphQLite.  I still think that it's a better assumption that operations require authentication by default.  I realize that'd be a BC break.  I guess a setting could be added :/

I do understand your idea on this.
I'm not quite completely sure how to handle this.
In particular, in the future, there might be third party packages providing GraphQL controllers. I'm not sure whether those third party controllers should be part or not of those default settings.
Maybe we could have a AuthenticationPolicy by namespace?
An alternative might be to be able to put @Logged annotations in the class docblock (rather than in the methods docblock) as proposed in #138.
While not as safe as your proposal, it could be a step forward to increased security.
Anyway, this requires a bit of work. I'll focus on releasing v4.0 first, and I'll come back to this topic when 4.0 is released.

@moufmouf yea, as I was saying, we just put a layer in front of GraphQL to handle this, it can definitely wait.  Unfortunately, the solution requires us to parse the graphql request body prior to passing the request to GraphQLite, which is redundant and less performant, but necessary in our case.
Ultimately, the issue is that annotations are terribly easy to overlook and miss.  And when it comes to something like security, that's problematic.  I'd be all for any way to define a policy.  I just find the current method to be insufficient.

Following back on this.  I think the @Logged annotation should be deprecated and removed from this lib entirely.  I just don't see why we need auth being handled in this lib.  If it's a symfony thing, I think it should be added to the symfony bundle instead.


feat(processors): add a ProcessorPipe

Add sghi.etl.commons.processors.ProcessorPipe, a Processor that pipes the raw data applied to it through a series of other Processor instances, passing the output of one Processor as the input to the next.

:tada: This PR is included in version 1.0.0-rc.1 :tada:
The release is available on:

v1.0.0-rc.1
GitHub release

Your semantic-release bot :package::rocket:

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

v1.0.0
GitHub release

Your semantic-release bot :package::rocket:


Explorer can't display data due to CORS errors

Hello.
Latest explorer throws the following error in the console
Cross-Origin Request Blocked: The Same Origin Policy disallows reading the remote resource at https://symbol.services/nodes?filter=suggested&limit=30&ssl=true. (Reason: CORS request did not succeed).
I have tested this in firefox  94.0 64-bit for ubuntu and the
Chromium web browser version 96.0. also on ubuntu.
BTW: add blockers were disabled.
The same problem happens in the latest desktop universal wallet.
I'm attaching full console output from both browsers in the zip file.
errors.zip
The outputs were generated while visiting https://symbol.fyi page.

Hi there, thanks for the feedback.
are you able visit symbol.services/nodes?filter=suggested&limit=30&ssl=true this url on your machine?

Hello. No i can't access this site from browser.
I have also tried in console with wget and the following message is given:
user@machine:~$ --2021-11-22 21:26:51--  https://symbol.services/nodes?filter=suggested Resolving symbol.services (symbol.services)... failed: Temporary failure in name resolution. wget: unable to resolve host address ‘symbol.services’
It seems a problem with local router firewall or ISP.
I tried to access the URL from another network and it works.

It seems a problem with local router firewall or ISP.

I'm not sure why the firewall is blocking the URL.
maybe it needs to whitelist the URL on your router?

Hello. It seems is not a router I haven't checked it but something
strange happened that makes me doubt that is router problem.
I wanted to check the site
http://forkwatch.symbol.tools/nem/summary
and I' ve got the same problem. I know 2 days ago I was able to access the site normally and
today not any more.
I tried to ping the domain but was getting
ping: unknown host forkwatch.symbol.tools
Then I went here
https://domaintoipconverter.com/index.php
to find out the IP address of the site.
Then I have put the IP  address and the domain into the /etc/hosts file and retried.
The site loaded. I have then make the same for the https://symbol.services and the
explorer worked.
I think is something with the DNS that the ISP provider uses.

This is an interesting issue, but great you had resolved it.
nice work mate.

seem the problem already solved, I will close this issue now.


Format custom editor view types

This commit formats custom editor view types with prefixes:
one.editor.* (for editors)
one.viewer.* (for viewers, without editing functionality)
ONE-vscode-DCO-1.0-Signed-off-by: dayo09<EMAIL_ADDRESS>
For #1165

Targets

current name
suggestion
type

onevscode.mondrianViewer
one.viewer.mondrian
viewType

onevscode.part-editor
one.editor.part
viewType

onevscode.circleViewer
one.viewer.circle
viewType

cfg.editor
one.editor.cfg
viewType

JsonTracer
one.viewer.jsontrace
viewType

one.viewer.jsonTracer
one.viewer.jsontrace.open
comnmand (not a viewtype, open command for the panel)

@ai-moiseev
As our release deadline is coming, and as this PR is quite simple, I would merge this PR without your approval :)
If you have any further opinion, please feel free to leave it!


fix: Import Item Completion

Part fix for #945

Fix: for import "lib.typ": tes|, the parent syntax is import item path, then grandparent is import items

So, is this an all-editors fix or does import items completion currently work in some editors?

Looks like a patch that can be ported to the upstream, here

It is not completely fixed, if we start to really use the ImportPath syntax, as the code here doesn't consider the newly introduced syntax as well.

Looks like a patch that can be ported to the upstream, here

Yes, I tested in the webapp and there is such issue there. Let me do it, I haven't contributed to Typst yet :)

In fact, there are 6 related bugs to fix at the same time.

Why there is no LF at the end of the Typst files? This (1) triggers me and (2) adds unnecessary visual noise into diffs so they become harder to read.

Why there is no LF at the end of the Typst files? This (1) triggers me and (2) adds unnecessary visual noise into diffs so they become harder to read.

@Andrew15-5 The only reason might be the formatter doesn't add LF at the end for me and I'm not used to adding them by myself. @Enter-tainer

Yeah, I think in Neovim it automatically adds this + it hides the last empty line. But in GUI editors I don't know if there is a built-in feature for this. In the web app I just have to add it manually, like I did this before I switched to Neovim.


new Subnautica setting

adds the new setting "free samples" to the yaml, with a 50% chance of being true

@Berserker66 you good with these weights for the free samples setting?

seems fine


fix(rules): Fixed an issue where errors would be printed in the log w…

…hen handling a show without a valid tvdb id

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


BUG: Text Encoding issues (backgroundImage path, tabTitle)

Environment
Windows build number:
Microsoft Windows [Version 10.0.18362.267]

Windows Terminal version (if applicable):
Windows Terminal (Preview)
Version: 0.3.2142.0

Any other software?
VS 2019, latest build

Steps to reproduce
These profile settings cause the specified background image to display.  Each of these images is viewable via File Explorer and in use as desktop background images.
  "backgroundImage": "C:\\Users\\kfarm\\OneDrive\\Backgrounds\\Characters (Landscape)\\Oddr [Vizsla] - desktop.png",
  "backgroundImageAlignment": "bottomRight",
  "backgroundImageOpacity": 0.5,
  "backgroundImagesStretchMode": "fill"

These backgroundImage paths fail to render:
  "backgroundImage": "C:\\Users\\kfarm\\OneDrive\\Backgrounds\\Characters (Landscape)\\Þiálfi [Vizsla] - desktop.png",

  "backgroundImage": "C:\\Users\\kfarm\\OneDrive\\Backgrounds\\Characters (Landscape)\\Fálki [Vizsla] - desktop.png",

Additionally, adding these characters to tab name causes them to display the bad-encoding diamond.
Expected behavior
Background image renders for any valid NTFS path or URI
Tab title renders without encoding issues
Actual behavior
No background image shown, bad characters, great sadness.
... Given the rash of other character issues, I think it's a failure to interpret UTF-8, perhaps?
... Further investigation showed that switching the encoding of the file in VS would fix the tab issue, but Terminal would rewrite the background image paths with \uxxxx codes which I'd have to re-type.

Yea I'd concur that this is a utf-8 roundtripping problem. This could be on either end of serializing or deserializing, impossible to know four sure without debugging further.

I can't repro this with a JSON file saved in codepage 65001.


Support auro-menu optional checkmark

Is your feature request related to a problem? Please describe.
Auro-menu now supports an optional checkmark. Per design the, default to no checkmark but keep it optional in auro-select.
Describe the solution you'd like
Default to no checkmark, add a checkmark attribute that returns it.

After further discussion, select will default to checkmark, but be reversed when nocheckmark is present in the auro-menu.


apigateway-yunionauth-domain-setting

这个 PR 实现什么功能/修复什么问题:

yunionauth-domain-setting

是否需要 backport 到之前的 release 分支:

release/3.2

/area apigateway
/cc @swordqiu @zexi

Codecov Report

Merging #6190 into master will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6190   +/-   ##
=======================================
  Coverage    7.04%    7.04%
=======================================
  Files         647      647
  Lines      110903   110903
=======================================
+ Hits         7813     7815    +2
+ Misses     102488   102487    -1
+ Partials      602      601    -1

Flag
Coverage Δ

#aFlag
7.04% <ø> (+<0.01%)
:arrow_up:

Impacted Files
Coverage Δ

pkg/appsrv/response.go
70.58% <0.00%> (+2.94%)
:arrow_up:

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 4e325d0...4e9264c. Read the comment docs.

/lgtm

/lgtm


NavigatorTransaction

To make it easier to avoid pushing twice in one frame, provide a
transaction mechanism for the navigator.
Fixes #556.

I have an e-mail written for flutter-dev ready to go when this is merged.

LGTM

You should email flutter-dev about the API change.


Use class names helper instead of string interpolation

Addresses https://github.com/joemasilotti/railsdevs.com/issues/587
Pull request checklist

[ ] My code contains tests covering the code I modified
[x] I linted and tested the project with bin/check
[ ] I added significant changes and product updates to the changelog

This looks great! Replying to your question in the issue: no, I don't think we need to add tests. Anything that already exist should be good enough.
Here are some more files that have CSS classes in the component's Ruby file:

app/components/collapse_control_component.rb:11
app/components/avatar_component.rb:24

These two files deal with CSS classes around iconography:

app/components/developers/availability_component.rb:16
app/components/developers/search_status_component.rb:36

I'm also now inclined to remove yass now that I know about class_names.

app/components/toast_message_component.rb:16
app/components/toast_message_component/toast_message_component.html.erb:1
app/helpers/form_helper.rb:4

Can you take a look at the first two? If you have any ideas for the icon-related ones I'd love to hear it, too. But no worries if not. I'll take care of yass in a new issue/PR.

So, I was able to use the class_names helper for collapse_control_component.rb:11
For avatar_component.rb:24, the classes used are either passed as an argument to the component or we use a static set of classes. I am not sure how we can avoid interpolation in this case.
For the two css files that deal with iconography (availability_component.rb:16 and search_status_component.rb:36), I saw that we are using inline_svg_tag which comes from the inline_svg gem. I tried using the new style of passing class names for this tag but it didn't work as expected and I also couldn't find a way to do it in the gem's documentation. So, I'm not sure how we'd do it. I'd love to hear your thoughts.

Ah, good catch!
I moved the CSS classes to the HTML files at the expense of a few more if/else statements. This feels like a solid tradeoff for now as I prefer the styling elements in HTML, not Ruby files.
Thanks for your contribution! I'm going to merge this in once CI passes.


Speed up website build in CI?

The current bottleneck in building the website in CI is executing the vignette, which takes ~10m.

If we add additional vignettes we may wish to pre-execute the vignettes before building the website, which would enable us to parallelize vignette execution
We may also wish to explore possible ways to avoid re-running the vignette every time (but NB potential setup and/or robustness costs).

Tagging @kaitejohnson, @cbernalz, and @seabbs for discussion.

My thoughts right now are that this is currently almost serving as an end-to-end test (which I know isn't great, but also a proper end-to-end would likely take the same amount of time to run in CI?). In particular, as we have been modifying pre and post-processing it has been a nice way to catch when there are errors in for example changing the names of arguments or elements in a list.
I think that after we put out our first release and we are somewhat committed to stabilizing the format of the outputs from the main fitting wrapper function wwinference() , we should consider doing something akin to what @seabbs set up in epinowcast which triggers rebuilding of the vignette if anything in the vignette changes +  scheduled updates on some cadence.
I think as we are developing pretty actively, something like weekly or biweekly seems reasonable. Also worth considering how much speed up we could get by 1.) reducing the complexity of the vignette (e.g. fewer sites) and 2.) modifying the fitting options to reduce warm-up iterations, acceptance probability, etc. I by default used the MCMC specifications we were using in production, but this is likely overkill for a vignette.


Update colors to material

Update themed drawer background colors to material values.
Update divisor color to be more neutral on both light and dark backgrounds.

Can you add a screenshot of the result? :)

good work!

@neokree It seemed to work, but I'm not sure that I changed the divisor color everywhere that it is referenced. Could you make sure?

certanly


Supporting node 0.11 using Nan

I have started working on a music app recently and wanted to use node 0.11 (I'm using Koa). So I've ported the bindings to use Nan.
Tests pass on node 0.10 and 0.11, on Ubuntu. I haven't tried on OSX or Windows.
I'm not sure if this would have been better based off another branch? Anyway, let me know and I can make any changes necessary.

+1 any updates on this?

thanks for the porting. until this gets merged, I'm using your version.
tests are passing with:
OSX Yosemite 10.10.2
latest taglib from git
node v0.12.2


microdontia

Add:
microdontia (child of abnormal tooth morphology MP:0002100)
DEF: abnormal reduction in the size of one or more teeth
SYN: decreased tooth size
SYN: reduced tooth size
SYN: decreased tooth width
SYN: reduced tooth width
SYN: small teeth
SYN: small tooth
SYN: microdontism
SYN: microdont
Anna

microdontia MP:0030088

Anna, I changed the tooth width synonyms to narrow synonyms since you could have short teet that would also be considered to be 'small teeth' even though they have normal width. Same thing for macrodontia and increased tooth width synonym.


Allow local paths in -lib command-line option

In my case I have a set of projects with a references to my own libraries (with dependencies). Local haxelib repo looks like a not good solution:

haxelib don't search for repo in parent folders (so I must copy .haxelib to every project);
links to my sources in .dev (inside .haxelib) are absolute and
must be finely maintained.

So, I think, haxe -lib path_to_the_library is a good solution...

This should be a relatively small change in path. It's important to handle paths first and dependencies and named libs after them, otherwise if you have -lib ./liba -lib ./libb and liba depends on libb you're going to get a conflict between the libb from the haxelib repo and the one from the path.

Now I think, what -lib path_to_lib feature is not intuitive and brings the unnecessary complexity to haxe. If nobody think else, I close this issue.


Separate the KV pairs from the data part and index part into different engines

Feature Request
Is your feature request related to a problem? Please describe:

The SQL dump of a table created by mydumper contains the data rows sorted by its primary index. The table can also contain secondary indices, which themselves won’t appear in their natural order. When a table is separated into multiple batches, the KV pairs from the secondary indices will be out of order. Therefore, the index parts will overlap and slows down import exponentially.
Describe the feature you'd like:

Separate the KV pairs from the data part and index part into different engines, to reduce the chance of overlapping engines, and increase ingestion speed.
Describe alternatives you've considered:

Lightning, after encoding, will classify the KV pair into “data” and “index” pairs. The original engines per batch (see RFC 3) will only store the data pairs. The index pairs are placed into a separate engine, and will not be batched.
Teachability, Documentation, Adoption, Optimization:

Use the following pseudo code representation
parallel for each table {
    for each index engine of table { // for now, only 1 engine per table
        open index engine
    }
    parallel for each data engine of table {
        open data engine
        parallel for each data file of data engine {
            parse SQL
            encode SQL to KV
            write data part of KV to data engine
            write index part of KV to index engine #-1
        }
        close data engine
        sequentially import data engine
    }
    for each index engine of table {
        close index engine
        sequentially import index engine
    }
    alter table auto_increment
    checksum
    analyze
}

Index engine UUID
The engine UUID of the engines will be deterministically computed using the Version 5 (SHA-1) algorithm. The data engine UUID is computed using $TableName:$EngineID. The index engine UUID is computed using $TableName:$IndexEngineID, where the index engine ID is counted as -1, -2, -3, …

#132


Remove diff "non flat mode" ... so diff going forward only supports flat mode

So flat mode was the default and now it is the only option.
Removing from ServerConfig the method:
  /**
   * Set to true if diff should return flat properties with dot notation rather than
   * embedded beans or reference beans (when the associated bean id is different).
   */
  public void setDiffFlatMode(boolean diffFlatMode) {
    this.diffFlatMode = diffFlatMode;
  }

... and removing the associated non flat mode diff.  Diff always in flat mode now.

I know I'm bringing up an old item here, but the Play Framework has updated to 2.5.x which brought in this change from Ebean (via a 3-major version jump) and has severely changed how we have to handle diffs now.  Given that we were relying on the non-flat version of the diff, can you provide a migration path to allow for that to still occur?  Or if not, would you mind letting us know why this change was necessary so we can at least understand it?
Thanks in advance.

It's kind-of a bad workaround (it relies on "id" to be the name of the @Id property), but I was able to get around this by the following (so please if you happen to know of a better way, I'd love to know, but given how we relied on the unflatted diff, this seems the best course of action without having to refactor many classes and/or lines of code):
package util;

import com.avaje.ebean.Model;
import com.avaje.ebean.ValuePair;

import java.util.HashMap;
import java.util.Map;

public class EbeanDiffUtil {
    public static Map<String, ValuePair> unFlattenDiff(Class rootClass, Map<String, ValuePair> diff) throws Exception {
        if (rootClass == null || diff == null) return null;
        if (diff.size() == 0) return diff;

        Map<String, ValuePair> returnMap = new HashMap<>();
        for (String s : diff.keySet()) {
            String[] props = s.split("\\.");
            // This requires the rootClass to have a property called id which is the @Id of the entity bean.
            //   - This is a poor convention, but a workaround for https://github.com/ebean-orm/avaje-ebeanorm/issues/406
            if (props.length > 1 && props[1].toLowerCase().equals("id")) {
                ValuePair valuePair = new ValuePair();

                Class beanType = getClassOfProperty(rootClass, props[0]);
                valuePair.setOldValue(getBeanById(beanType, diff.get(s).getOldValue()));
                valuePair.setNewValue(getBeanById(beanType, diff.get(s).getNewValue()));

                returnMap.put(props[0], valuePair);
            } else {
                returnMap.put(s, diff.get(s));
            }
        }

        return returnMap;
    }

    private static Class getClassOfProperty(Class rootType, String property) throws NoSuchFieldException {
        return rootType.getField(property).getType();
    }

    @SuppressWarnings("unchecked")
    private static Object getBeanById(Class beanType, Object id) {
        Model.Finder find = new Model.Finder(beanType);
        return find.byId(id);
    }
}

That getBeanById() method should change to return a reference bean.
    @SuppressWarnings("unchecked")
    private static Object getBeanById(Class beanType, Object id) {
        // This find.byId() would hit the database every time ...
        //Model.Finder find = new Model.Finder(beanType);
        //return find.byId(id);

        // ... use a reference bean instead
        return Ebean.getReference(beanType, id);
    }

why this change was necessary so we can at least understand it?

The flattened diff was needed to support both @History  and @Changelog features.  The flattened diff I believe is much better at handling embedded beans and converting to JSON. At that point we had 2 modes for doing 'diff' and later on we get to the point of questioning why we would keep around the older diff mechanism.
I did ask the google group about changing the diff mode in July 2015 but there was not much reaction at that time. https://groups.google.com/forum/#!searchin/ebean/flat$20mode/ebean/n_1adkZdflk/0-mpRi1ueAgJ
I probably didn't communicate well when looking to drop the original diff mode (this ticket).  Given my previous conversations on the google group it seemed that the old diff mode could be dropped.
In general we want Ebean over time to deprecate and then remove deprecated features in order to keep the internals as simple and clean as possible.  I realise that didn't work so well for you wrt this feature being dropped.

Thanks for the explanation!  I'll test your code change suggestion tomorrow and make sure that it works well with how we're using it.
I think my main issue is that Play is upgrading very fast and we, as Play and Ebean users, are not getting informed by Lightbend of every fallout of this kind of upgrade.   The three major version upgrade of Ebean by Play is an enormous jump for a single dot-version of Play.  Couple that with the Play documentation lacking in a few places just allows these issues to occur for those who wish to perform the upgrades.
As for the google group, I'm subscribed but wasn't getting emailed (though I will do so asap) so I didn't notice that topic.  I suspected you had good reason for this, only curious what the reason was (which you kindly explained).  Unless it was a burden to keep a utility to allow an unflattened diff, I definitely would have suggested keeping an API that allowed the diff as such (something like Ebean.unflatDiff() or some other better name).  Sadly I am too late to the game :)  The good news is that this workaround should work for now until we stop using Ebean.diff for what was originally intended in our code (some developers previous to me were using it as a way to do field level validation to determine if a field was even allowed to be updated based on certain criteria, and I'd prefer the validation to be much more explicit than they had originally coded).
Thank you very much for your responsiveness on this.  I will keep a keen eye on the google group for Play and for Ebean.

No problem.

as Play and Ebean users, are not getting informed by Lightbend of every fallout of this kind of upgrade

A while back I suggested to Ligthbend (was Play at the time obviously) that we could offer a combined support option but that idea was not supported (at least by James Roper at the time). The 2 projects operate independently with basically no cooperation  / communication and I don't think that will change.
Things are a lot better with the newer PlayEbean plugin to allow people to upgrade Ebean versions more freely. That they locked developers into a old version of Ebean for so long ... yes, painful for many. Maybe we won't have this type of issue again.
Don't expect the Lightbend folks to know much about what is going on with Ebean. There is no regular  communication / connection there, they have never asked about where Ebean is going or and don't contribute to the Ebean project as such.

I will keep a keen eye on the google group for Play and for Ebean.

Great. With Ebean we have pushed out a lot of versions over the last 9 months or so (with the help of sponsorship). That will slow down shortly but it's good to keep an eye on the changes and feedback as necessary etc.
Cheers, Rob.


[BUG]: null value in column "XXX" of relation "YYY" violates not-null constraint

What version of drizzle-orm are you using?
0.31.2
What version of drizzle-kit are you using?
0.21.6
Describe the Bug
I'm trying to create a userId column, fk to users.id but optional (so I don't set the notNull property):
userId: char("user_id", {
  length: 12,
}).references(() => users.id),

But when I try to insert a value with NULL user_id, I get the error:

null value in column "user_id" of relation "notifications" violates not-null constraint

I've read that we're supposed to be able set null as fk if notNull is not present.
I also tried with every combination of onDelete/onUpdate (set null, cascade etc.) but nothing works.
Expected behavior
I should be able to set a NULL value
Environment & setup
No response

I checked in drizzle-kit, the generated schema has a notNull() while I have clearly not set any notNull in my table declaration.
This is a major bug.
In drizzle-kit schema explorer, I have :
userId: char("to_user_id", { length: 12 }).notNull(),

It seems after creating a new project, it is not adding notNull() when not specified, so may be due to a node_modules cache issue. Will reopen if happens again.

I have a simmilar issue.
Query: insert into "chat_conversation_file" ("id", "title", "language", "file_extension", "text", "created_at", "updated_at", "chat_user_id", "chat_conversation_id", "chat_conversation_message_id") values (default, $1, $2, $3, $4, default, default, $5, $6, $7) returning "id", "title", "language", "file_extension", "text", "created_at", "updated_at", "chat_user_id", "chat_conversation_id", "chat_conversation_message_id" -- params: ["", "python", "py", "print(\"Hello World!\")", 2, 202, 379]

How is this causing the following error?
ERROR  Failed to create chat conversation files in database null value in column "extension" of relation "chat_conversation_file" violates not-null constraint


License excel template file download!

License Excel Template File Download! Request URL： http:// Request method：GET

ok, we are working on that.


分库分表支持批量sql吗

例如insert foreach操作 现在报错 需要配置什么吗

do you mean use mybatis?
Sharding-sphere faced on SQL, JDBC and MySQL protocol , any ORM can support

If you have a look at release notes, maybe you can find which version can support which type of SQL


Add API token-based authentication

See http://docs.uchiwa.io/en/latest/features/federated-api/ for basic documentation, more to come!

Coverage increased (+0.5%) to 26.741% when pulling 195962a98a6eaa6d63ca8b970675fde326f21259 on palourde:api-authentication into f28990a3ac7b77ecf3f656860a2e3eb90585e514 on sensu:master.


Error in preprocessing CO3D dataset

Hello author:
Thank you for this amazing work! I ran the CO3D data preprocessing script and got this error

It seems like the frame.crop_bbox_xywh attribute is None

Hi! That's unexpected, I tried reproducing this error but was not able to. On my side frame does have the crop_bbox_xywh attribute. Could you check if this is an issue with this attribute in particular or data in general by checking if your frame has some other attributes (e.g. image_rgb or camera)?

I printed all attributes in the frame object, here's the screen shot:

Seems like other attributes are here?

same problem from my side: no crop_bbox_xywh

Hi @mengxuyiGit @FantasticOven2 thanks for highlighting this. I double checked my data and it looks the same except crop_bbox_xywh is not None: see attached screenshot. I'll try to dig in why this might be an issue.

I also have the same problem from my side: no crop_bbox_xywh for all sequences. The dataset I am using is CO3Dv2. I also tried CO3D you recommand but no 'set_list' folder in the dataset while pytorch3d.implicitron.dataset.json_index_dataset_map_provider_v2 requires this folder.

very great work, thanks!
I have the same issue when processing the data, would it be possible to share your annotation file with us?

Hi @apchenstu , @mengxuyiGit  @jwubz123 @FantasticOven2  - I updated the download instructions in Readme. I also tested them myself just now. As far as I can see, crop_bbox_xywh is available in all frames. Could you re-download the dataset with instructions from the Readme and check this works on your side too?

Hi! I am having the same issue where crop_bbox_xywh=None, I downloaded and preprocessed the CO3D dataset with the updated README.

Hi @SJTUwxz OK, thanks for letting me know. Since it runs on my side I will need some information to reproduce your issue. Can you tell me the exact steps you ran, and can you run the following snippet:
from pytorch3d.implicitron.dataset.json_index_dataset_map_provider_v2 import JsonIndexDatasetMapProviderV2
from pytorch3d.implicitron.tools.config import expand_args_fields
from omegaconf import DictConfig

subset_name = "fewview_dev"

expand_args_fields(JsonIndexDatasetMapProviderV2)
dataset_map = JsonIndexDatasetMapProviderV2(
      category="hydrant",
      subset_name=subset_name,
      test_on_train=False,
      only_test_set=False,
      load_eval_batches=True,
      dataset_root=DOWNLOAD_FOLDER,
      dataset_JsonIndexDataset_args=DictConfig(
          {"remove_empty_masks": False, "load_point_clouds": True}
      ),
  ).get_dataset_map()["train"]

And check that files in dataset_map, for example dataset_map[0] has crop_bbox_xywh?

Thanks for the update, I still have the same issue after using the new script. dataset_map[0]['crop_bbox_xywh'] is None on my side.

Hi @SJTUwxz OK, thanks for letting me know. Since it runs on my side I will need some information to reproduce your issue. Can you tell me the exact steps you ran, and can you run the following snippet:
from pytorch3d.implicitron.dataset.json_index_dataset_map_provider_v2 import JsonIndexDatasetMapProviderV2
from pytorch3d.implicitron.tools.config import expand_args_fields
from omegaconf import DictConfig

subset_name = "fewview_dev"

expand_args_fields(JsonIndexDatasetMapProviderV2)
dataset_map = JsonIndexDatasetMapProviderV2(
      category="hydrant",
      subset_name=subset_name,
      test_on_train=False,
      only_test_set=False,
      load_eval_batches=True,
      dataset_root=DOWNLOAD_FOLDER,
      dataset_JsonIndexDataset_args=DictConfig(
          {"remove_empty_masks": False, "load_point_clouds": True}
      ),
  ).get_dataset_map()["train"]

And check that files in dataset_map, for example dataset_map[0] has crop_bbox_xywh?

Thanks for the update, I still have the same issue after using the new script. dataset_map[0]['crop_bbox_xywh'] is None on my side.

Thanks for the quick response! @szymanowiczs I also have the same issue with the new script, and the output dataset_map["train"][0] has crop_bbox_xywh=None:
As instructed by Readme, I ran the following commands to download the co3d dataset:
git clone https://github.com/facebookresearch/co3d.git
cd co3d
mkdir DOWNLOAD_FOLDER
python ./co3d/download_dataset.py --download_folder DOWNLOAD_FOLDER --download_categories hydrant,teddybear
and I created a proprocess folder under DOWNLOAD_FOLDER to store the preprocessed data, and set that as CO3D_DATASET_ROOT in proprocess_co3d.py.

What are your pytorch3d versions? I'm using pytorch3d 0.7.2

Mine is 0.7.5. I installed the pytorch3d library from a local clone of their repo.

Hello! Sorry for late reply!
I tried the second solution first, and add the following lines to L106 of the preprocess_co3d.py:
bbox_xyxy = get_clamp_bbox(
frame.bbox_xywh,
image_path=frame.image_path,    # pyre-ignore
box_crop_context=0.3,)
clamp_bbox_xyxy = clamp_box_to_image_bounds_and_round(bbox_xyxy, image_size_hw=tuple(frame.effective_image_size_hw), )
crop_bbox_xywh = bbox_xyxy_to_xywh(clamp_bbox_xyxy)
x0, y0, box_w, box_h = crop_bbox_xywh

The generated crop_bbox_xywh of the same frame (frame_timestamp=0.0893) as you mentioned earlier is tensor([189, 240, 353, 560]), where the height is different from the one you got (560 vs 592).
Preprocessing also stopped later after processing several frames with this error:

You are right that the second one might generate crop_bbox_xywh that is inconsistent with yours. I will try the first solution instead which will save some time to debug :) I am working on replicating the results on the SRN-cars/chairs dataset now but I will get back later to update if solution 1 works!
Thank you!

I get the same results by directly pulling and pulling pytorch3d. It also shows the same error mentioned by @SJTUwxz
Then I wget the version <IP_ADDRESS> and build locally.
wget https://github.com/facebookresearch/pytorch3d/archive/refs/tags/v0.7.2.zip
unzip v0.7.2.zip
cd pytorch3d-0.7.2
pip3 install .

then it seems to work fine this step.

Thanks for reporting that wget 0.7.2 and building locally works. Readme contents are now updated to indicate that this is the required pytorch3d version, so I'm closing this issue. Feel free to reopen if you have further issues.

This was a big gotcha for me! Maybe we could add this as a warning to the README?

Hi, thanks for the discussion!
I am struggling to install pytorch3d 0.7.2 on my computer. Thus, I configure a docker image here in case someone needs it:
docker pull uerrob/pytorch3d:0.7.2


distribute workers port ranges, closes #708

This patch distributes the configured ports range among all the created workers.

We should backport this fix..


Support Arbitrary IMU Orientation

I had to mount my naze for my autogyro upside down.  How do I fly?

Wondering what the status of this issue is, because our team has the naze upside down on the foamy... Can I help get this working?

I looked into the branch for this fix. It looks like you were planning on supporting any arbitrary angle for the orientation. Is that necessary or could we just multiply the raw imu input by -1 based on an orientation parameter in the x y and z axis?

Sort of. You have to multiply both the z and y axes by -1, (otherwise you'll end up with a left-handed coordinate frame).  Since there are so many configurations to consider, it just made sense to support arbitrary orientations.  There is actuallly a pull request active on this one, which adds 3 parameters.  If you merged master into that branch, it would probably work.

I had to make a small tweak, but I have it working now. The parameters say degrees, but everything is treated like radians. I figured degrees made more sense in this context, so I added a scalar. I've also made a few organization changes to match what I've been seeing elsewhere in the code. If you would allow it, how may I submit my changes?

Why don't you submit a pull request and put me as a reviewer?

Finished!


Add support for transient connections

Similar to go-libp2p  we can add a flag to the ConnectionStat interface as well as an option to opt-in for opening streams on connections that will indicate whether that connection is transient and thus may be closed soon.

Closing based on conversation in triage


Change binary names for consistency

Changes to Binary Names and Default Behavior
Please consider this PR. I am happy to re-add node-gui to the defaults if this creates any issues. The goal of this PR is to simplify the build process. Most users are not interested in building the GUI, and it comes with many unwanted dependencies, such as GTK. Additionally, we aim to ensure consistency in binary names, as we are beginning to develop scripts and documentation that reference them.
1. Updated Binary Names
The binary names have been updated as follows:

node-gui is now mintlayer-gui
node-daemon is now mintlayerd
wallet-cli is now mintlayer-cli
wallet-address-generator is now mintlayer-address-generator
web-server is now mintlayer-api

To run the binaries, use the new names as in the example below:
cargo run --bin mintlayerd

2. Changes to node-gui
node-gui has been removed from the default build and test targets. To run it, specify the package and binary explicitly:
cargo run -p node-gui --bin mintlayer-gui

As discussed, this isn't necessary now.