[Bug] 包含%导入报错

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
1.2.1
What's Wrong?
varchar类型字段的值中，包含%，stream load导入报错，如下：
{"timestamp":"2023-03-01T02:44:08.106+00:00","status":500,"error":"Internal Server Error","path":"/api/example_db/table20/_stream_load"}

但如果将数据放入本地文本文件，通过curl的方式stream load，没有报错，可正常写入
What You Expected?
stream load导入数据成功
How to Reproduce?
建表语句：
CREATE TABLE `table20` (
  `id` int(11) NULL,
  `name` varchar(32) NULL DEFAULT ""
) ENGINE=OLAP
UNIQUE KEY(`id`)
COMMENT 'OLAP'
DISTRIBUTED BY HASH(`id`) BUCKETS 1;

导入：
header = {
'Content-Type': 'application/x-www-form-urlencoded;charset=UTF-8',
'Accept': 'application/json;charset=UTF-8',
'Expect': '100-continue',
"label": label,
"format": "json",
"strip_outer_array": "true"
}

data = [{"id":1,"name":'1'},{"id":2,"name":'a%'}]
rsp = requests.put(url=url, headers=header, auth=('root','123'), json=data)
be_url = rsp.url
rsp = requests.put(url=be_url, headers=header, auth=('root','123'), json=data)
print(rsp.text)

或
curl -XPUT  --location-trusted -u root:123 -H "label:test3" -H "timeout:10000" -H "format: csv" -H "columns: id,name" -H "column_separator:," -H "Expect: 100-continue" http://xxx:8031/api/example_db/table20/_stream_load -d '1,a
2,b
3,c
4,%'

Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I test OK
curl --location-trusted -u root:pass -H "label:test" -H "column_separator:," -H "format:csv" -H "columns: id,name"  -T test.csv http://ip:8030/api/test/table20/_stream_load

not this way has problem.
@feng-kui said ，the following not work
curl -XPUT  --location-trusted -u root:123 -H "label:test3" -H "timeout:10000" -H "format: csv" -H "columns: id,name" -H "column_separator:," -H "Expect: 100-continue" http://xxx:8031/api/example_db/table20/_stream_load -d '1,a
2,b
3,c
4,%'

I test OK version 1.2 lts curl --location-trusted -u root:pass -H "label:test" -H "column_separator:," -H "format:csv" -H "columns: id,name" -T test.csv http://ip:8030/api/test/table20/_stream_load

curl -XPUT  --location-trusted -u root:123 -H "label:test3" -H "timeout:10000" -H "format: csv" -H "columns: id,name" -H "column_separator:," -H "Expect: 100-continue" http://xxx:8031/api/example_db/table20/_stream_load -d '1,a
2,b
3,c
4,%'
not this way has problem.
--------------------has proble below----------------------------------
curl -XPUT  --location-trusted -u root:123 -H "label:test3" -H "timeout:10000" -H "format: csv" -H "columns: id,name" -H "column_separator:," -H "Expect: 100-continue" http://xxx:8031/api/example_db/table20/_stream_load -d '1,a
2,b
3,c
4,%'

I test OK version 1.2 lts curl --location-trusted -u root:pass -H "label:test" -H "column_separator:," -H "format:csv" -H "columns: id,name" -T test.csv http://ip:8030/api/test/table20/_stream_load

not this way has problem
has problem below
curl -XPUT  --location-trusted -u root:123 -H "label:test3" -H "timeout:10000" -H "format: csv" -H "columns: id,name" -H "column_separator:," -H "Expect: 100-continue" http://xxx:8031/api/example_db/table20/_stream_load -d '1,a
2,b
3,c
4,%'


Remove bagel.yml from root directory.

It does not need to be there. Must've snuck in by accident.

Closes #6700


error in linux

Hello,
actually, i test QuikcLogger in Ubuntu 18.04 with QuickLib. (latest version 2.0 and 1.43)
just include Quick.Logger, Quick.Logger.Provider.Files.
(few months ago, i reported this error. but it's not fixed)
i try execute in linux machine after than result is show this error.
realloc(): invalid next size
중지됨 (core dumped)
i patch source code in Quick.Commons.pas like this.
(in GetComputerName procedure .. line 948)
{$IFDEF DELPHILINUX}
var
phost : PAnsiChar;
begin
try
phost := AllocMem(256);
if gethostname(phost,_SC_HOST_NAME_MAX) = 0 then
begin
{$IFDEF DEBUG}
Result := Copy(Trim(phost),1,Length(Trim(phost)));
{$ELSE}
Result := Copy(phost,1,Length(phost));
{$ENDIF}
end
else Result := 'N/A.';
FreeMem(phost);
except
Result := 'N/A';
end;
end;
gethostname need allocated buffer. but phost not allocation error.
i hope it helps.

Thanks. I've made your changes.


Accessing adjacent properties of different types in Document object

The situation I am at right now is that I want to locate a table which the content should be instructed by a paragraph right above it. In another word I should be given a string that appears in a paragraph and is expected to get a reference of the table right below it. In the docs both tables and paragraphs are block items but Document objects don't have a "blocks" property, and we don't have a "BlockItem" interface. Is there any way to work around?

I think I am in the same page and don't know how to work it out. My objective is to be able to read "line by line" and interact with it, it being a paragraph or a table. This is so because they are directly related and being able to read both, one after each other is key.
I can iterate through the Doc with say, for loops, on tables or on paragraphs but they are incompatible with each other meaning the for loop will ignore the counterpart.
Anyone has any idea on that?
W

Search this issues list on iter_block_items.
Especially this one: #40.


[Feature]: support trace api

Hi, i have added three RPC interfaces for trace api(starknet_traceTransaction、starknet_simulateTransactions、starknet_traceBlockTransactions), plus an associated PR https://github.com/xJonathanLEI/starknet-jsonrpc-codegen/pull/2

Any plans to proceed with this? Would also close https://github.com/xJonathanLEI/starknet-rs/issues/437

Superseded by #481.


TypeScript setup

I'm leaving some notes here for when we eventually take a stab at adding back TypeScript.
Issues we encountered
We had originally tried getting @rollup/plugin-typescript working, but encountered a couple issues:

https://github.com/rollup/plugins/issues/287 - this issue has the most votes of any Rollup bug in either the core repo or plugin repo
https://github.com/rollup/plugins/issues/247 - this issue is closed because no one had agreed to work on it, but is still present

We instead setup rollup-plugin-typescript2, which was really easy to get setup. However, we eventually did hit one issue:

https://github.com/ezolenko/rollup-plugin-typescript2/issues/254 - errors are swallowed and you need to run tsc manually to get the real error message. I suspect this has something to do with our usage of pnpm and corresponding directory structure (e.g. some libraries are present in ../../node_modules if you're in a subpackage).

@rollup/plugin-typescript is super finicky and really hard to get working. rollup-plugin-typescript2 was really easy to setup though it seems not to support pnpm yet and swallows error messages as a result. Normally I would think the best place to focus my efforts would be to fix the one issue in rollup-plugin-typescript2 and then eventually re-enabled it. However, it turns out that rollup-plugin-typescript2 does not have any tests! So in the long-run perhaps it's better to put more work into @rollup/plugin-typescript even though it might be further from working really well
The other issue we were hitting was that the project wasn't being compiled on the CI, so we had a few PRs that broke things. That seems easily avoidable and basically fixed at this point
Suggested solution
I wonder if a lot of our issues may be avoided by just calling tsc directly and removing Rollup from our build process given that the Rollup TypeScript plugins are so finicky. It certainly seems like it would avoid an extra layer of potential complication. TypeScript doesn't yet support npm's new exports map, which means we'll have to continue outputting files to the subpackage root directory instead of using something like a dist/ directory (https://github.com/microsoft/TypeScript/issues/33079)

Out of curiosity, I just tried @rollup/plugin-typescript and it works fine.
No configuration necessary. See this branch and particularly the last commit.
This commits only sets up typescript for the kit package.


Make keepalived.tmpl customizable

Issue:
If something in keepalived.tmpl does not work for any reason on your cluster, then your out of luck as the docker image has crystallized it.
Suggestion:
Optionally allow for the addition of a customized keepalived.tmpl, possibly through a configmap - Not sure how go code works in terms of compilation, etc but would that work - not sure.

see #67


Cant add A child Domain to an existing Forest, W srv 2019 essentials.

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

ID: 6116630e-8753-9f35-cd7c-1bac97ace1cd
Version Independent ID: 03d21e87-8b89-ff82-3977-bfbb8b7b4bb9
Content: AD DS Installation and Removal Wizard Page Descriptions
Content Source: WindowsServerDocs/identity/ad-ds/deploy/AD-DS-Installation-and-Removal-Wizard-Page-Descriptions.md
Product: windows-server
Technology: ad-ds
GitHub Login: @iainfoulds
Microsoft Alias: daveba

@mkmma Would you please add more details to the issues with the documentation? The title is worded in such a way that we cannot tell if you are having problems adding a child domain to an existing forest or that the document states something that is not possible. Thanks!

Hello @mkmma,
Thanks for contacting Microsoft. We value your feedback.
As we understand it, you've described an issue with the Window Server product or service, rather than the documentation itself.  Note that per https://learn.microsoft.com/windows-server/identity/ad-ds/deploy/upgrade-domain-controllers, "Active Directory Domain Services isn't supported on the following Windows operating systems: Windows MultiPoint Server, Windows Server Essentials." Therefore the AD DS configuration procedure described in this article can't be done with Windows Server Essentials.
As you're probably aware, we offer many channels for engaging with us, whether you have a support question, product feedback, or other feedback.  GitHub Issues in the Windows Server repo are monitored for content issues we can address, such as broken links, grammar or wording, typos, updates, or incorrect code examples.
For support cases or support tickets, extended troubleshooting, technical support, or product support, use one or more of the following methods:
- From Windows, use Windows Feedback Hub or press Windows + F. You can also provide feedback with Feedback.
- Open a ticket with Microsoft Support.
- Search or engage with Microsoft TechNet.
- Search or engage with the Microsoft Developer Network.
- Search or engage with the Microsoft Support Community.

Bug submittal or feature requests

Because this issue is a non-documentation issue, we’re closing this thread.
Thank you.
#please-close


WIP: Tag system + Version System

This PR attempts to add tag system + versioning system to PerfChecker.

Codecov Report
Attention: Patch coverage is 33.33333% with 34 lines in your changes are missing coverage. Please review.

Project coverage is 58.33%. Comparing base (eb188fd) to head (07447d1).

Files
Patch %
Lines

src/versions.jl
0.00%
33 Missing :warning:

src/PerfChecker.jl
94.44%
1 Missing :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##              dev      #25       +/-   ##
===========================================
- Coverage   88.88%   58.33%   -30.56%
===========================================
  Files           5        6        +1
  Lines          45       96       +51
===========================================
+ Hits           40       56       +16
- Misses          5       40       +35

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Todo:

Add an example which deals with testing multiple versions of the same package.
Deal with head. The only reason I haven't done it is because I feel like users can just easily provide it inside the extra_pkgs key in the config dictionary.

On the next episode PR:

Convert the entire "Config Result" into a (TOML?) file and the ability to parse results from a (TOML?) file.

Nightly issue is related to some Pkg changes. We probably should not try to fix it before any v11-alpha/beta is out anyway.


Add new project and new ontology wizards

The new project wizard configures the project to have OML and Xtext natures and creates a src directory.
The new ontology wizard creates a vocabulary, bundle, or description ontology, and allows the user to specify a namespace IRI, separator, and prefix.
By default, description ontologies are saved as OML XMI and vocabulary and bundle ontologies are saved as OML text.
New Project Wizard

New Ontology Wizard

You may want to adopt OML 0.7 in this PR. In that release, Bundle got renamed to VocabularyBundle and a new DescriptionBundle got added. So, you may want to accomodate those changes in this PR.

Updated to 0.7.1 (and made some other fixes)


Clean package after boot

Thanks. I don't see any big advantages for this change, so going to leave thins as they are for now.


Add media

This PR will:

Add the next-tinacms-cloudinary as a dependency and bump tinacms to support latest media API updates.
Update schema so that image fields are of type image vs text
Add additional setup instructions to readme reflecting necessary pieces for cloudinary.

I imagine there will be some tweaking to the readme, I went with very basic instructions.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jbevis:x: tina-cloud-app[bot]You have signed the CLA already but the status is still pending? Let us recheck it.

I'm currently in the process of adding this work into @spbyrne updated design PR https://github.com/tinacms/tina-cloud-starter/pull/99
Assuming that goes well, I will close this one out in favor of that.

Closing as this PR has become irrelevant.


Feature request: Configurable file size limit

Hi,
I am having an issue with bracket pair colorizer 2 when opening large C++ files, it causes the remote development connection to vscode server to be dropped since it makes it run too slowly. Would it be possible to have a configurable file size limit above which the extension ignores a file? For instance, if file is above 10,000 lines, then don't bother colorizing the brackets? That would allow me to keep using the extension (most of my files are smaller than 10,000), but for these really large files that make it choke not to cause any issues. Thanks!

Hello, I have a similar issue with long JSON files (a few MB). It causes all other extensions to hang and I can't even use IntelliSense until VS decides to cancel the operation (and that's long). I believe that a file size limit would solve the issue.


Setting globalStyles for vue to true causes TypeError: The "path" argument must be of type string.

Laravel Mix Version: 6.0.11
Node Version: v14.15.4
NPM Version: 6.14.10
OS: macOs 10.15.7

Description:
After turning on globalStyles …
.vue({
	version: 3,
	extractStyles: true,
    	globalStyles: true,
})

… I’ll get the error:
[webpack-cli] TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type boolean (true)
If I set globalStyles: false everything works.

glboalStyles is an array or object of arrays listing the files you want included globally — and false or null to disable them. true is not a valid value for globalStyles. It appears that our types are incorrect. I'll put this on my todo list fo fix soon.
Example usage:
mix.vue({
    version: 3,
    extractStyles: true,
    globalStyles: {
        // Any one of these works
        // You can provide multiple if you're using multiple preprocessors
        css: [`example.css`],
        sass: [`example.sass`],
        scss: [`example.scss`],
        less: [`example.less`],
        stylus: [`example.styl`],
    }
})


Use mermaid instead of pikchr for diagrams

See conversation about this that starts here:
https://github.com/gren-lang/book/pull/15#issuecomment-2126282446
Tested with https://github.com/blaix/fake-gren-package
Renders fine on the package overview and module docs pages, but the rss feed looks empty and I'm not sure why... should I have to do anything besides import a package for it to show up in the feed?

The rss feed doesn't look empty here (looking at the raw xml output). The package site returns the first version for use in the feed (the feed is only for brand new packages), so I had to tweak the SQL statement to return the latest version for use in the feed.
It shows the raw mermaid code, but I don't think you can import modules in rss so that's the best we can do.

Hah, I think we were doing the same thing at the same time: https://github.com/gren-lang/package.gren-lang.org/pull/37

Is this ready to merge?

Is this ready to merge?

It is now


Added the animation to cards

I am going to merge this right now. It looks cool. Thanks.
But there are few things which I dont understand; like what does transform cubic-bezier do? why those specific numbers there? and why -webkit-transform?
Please note I am not a css expert and asking you to get to know what they acually do. If you could provide simple explaination or provide links where I could get to know what it does, it would be helpful.

@ravikamath123 Any resources/links for the info/docs I requested!?


⚠ Science Web Services - QState has degraded performance

In af3aed9, Science Web Services - QState (https://sws.labbricks.com/qstate/) experienced degraded performance:

HTTP code: 523
Response time: 3157 ms

Resolved: Science Web Services - QState performance has improved in 4c1437a after 48 minutes.


Fixes #8: Messages always encoded with 8 bytes.

Fixed by:

Reading length attribute in kcd format.
Shorting the returned data to specified length with python slice operator.

I see tests failing. Let me fix that...

Coverage decreased (-0.06%) to 93.188% when pulling 87b3e2f534dffc402c15447bbda2f5293afe564d on AxelVoitier:master into 7b49494c375a966f4776b9f53bcc384fb6581a97 on eerimoq:master.

Coverage increased (+0.06%) to 93.308% when pulling 361be5edcb0ed3439f817e4ff86accb2773ca973 on AxelVoitier:master into 7b49494c375a966f4776b9f53bcc384fb6581a97 on eerimoq:master.

OK, check this out.

Looks good, thanks for the contribution! I'll make a new release and upload it to PyPi soon (probably tonight).

Neat! I should make other PR in the coming days concerning other issues with Kayak files. I let you see if you want to make new release that often or not ;).


SNS s3 notifications

to: @airbnb/streamalert-maintainers
cc: @austinbyers
size: small
resolves ?
Background
Streamalert currently allows adding an S3 bucket as a data source. This creates an S3 event notification for the S3 bucket. As S3 currently only supports one event notification per bucket, this also means that the source bucket would only be configured to notify Streamalert.
A common use case would be to have Streamalert subscribe to an existing S3 event notification channel such as an SNS topic.
In this PR, Streamalert checks if an SNS payload is an S3 event notification and if so, extracts the S3 object for parsing.
Changes

Additional test case test_pre_parse_sns_s3_notification.
Additional test helper method make_sns_s3_notification_raw_record .
Modification of the pre_parse method of the SnsPayload payload class to detect and parse S3 event notifications.

Testing

Additional test case test_pre_parse_sns_s3_notification was added.
A full test was also conducted with the following steps:

create a new cloudtrail with a target s3 bucket.
configure the target s3 bucket with an SNS s3 event notification.
spin up streamalert with the configuration templates below.
perform an action that triggers one of the community rules (create an IAM user using an IAM user profile that does not have MFA configured).

prod.json:
{
  "id": "prod",
  "modules": {
    "stream_alert": {
      "rule_processor": {
        "current_version": "$LATEST",
        "memory": 256,
        "timeout": 10,
        "inputs": {
          "aws-sns": [
            "arn:aws:sns:${AWS::Region}:${AWS::AccountId}:${sns_topic_name}"
          ]
        }
      },
      "alert_processor": {
        "current_version": "$LATEST",
        "memory": 128,
        "timeout": 25
      }
    }
  },
  "region": "ap-southeast-1"
}

source.json:
{
  "sns": {
    "${sns_topic_name}": {
      "logs": [
        "cloudtrail"
      ]
    }
  }
}

Notes
You will have to configure the source bucket policy to allow access to the Streamalert AWS lambda function role for this to work:
{
    "Version": "2012-10-17",
    "Statement": [
        ...
        {
            "Effect": "Allow",
            "Principal": {
                "AWS": "arn:aws:iam::${AWS::AccountId}:role/${stream_alert_prefix}_prod_streamalert_rule_processor_role"
            },
            "Action": [
                "s3:Get*",
                "s3:List*"
            ],
            "Resource": [
                "arn:aws:s3:::${s3_bucket_name}",
                "arn:aws:s3:::${s3_bucket_name}/*"
            ]
        }
    ]
}

Coverage increased (+0.005%) to 96.857% when pulling 0def992418c47fcdc204fb15bd1949bfd63dddbf on ts33:feature/sns-s3-notifications into 435f90fc1f5c13dba08c9894155fe03e87beb2b3 on airbnb:master.

+1

hey @ts33 thank you for this contribution!! I apologize for not acknowledging this a bit sooner, but will be getting to look at it soon. please standby :)

this support has been added in a different way, but this is now supported in 2.1.0


Fix array tangent gen functionality

I believe the code that I already have in this PR is enough to fix the problem. I now need to double check that this is the case in all situations that I'm aware of, and write an isolated regression test. This should then be good to go.

This is basically good to go now. It seems to have fixed some problems that I was having with DiffEq integration, and fixes some functionality which was quite plainly incorrect.
It does, however, highlight he need to finish off the overhaul that we started a while ago on making tangent generation functionality work properly in the presence of aliasing and circular referencing, but that's not something to tackle right now.
If CI passes, I will merge + tag.


feat(build): uplift Storybook to v8

feat(build): uplift Storybook to v8
SUMMARY

Uplift Storybook used in superset-frontend to latest version as of now (v8). I ran npm run storybook build-storybook plugins:build-storybook plugins:storybook and check most stories are OK except for a few stories seemingly to fail before the Storybook uplift

legacy-chart-plugins-legacy-preset-chart-nvd3-pie--basic

legacy-chart-plugins-legacy-plugin-chart-paired-t-test--basic

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

All stories executed by npm run storybook and npm run plugins:storybook must be successful.
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

The lockfile is synced now :D
Hopefully no new pkg change incoming 😝

Yikes, looks like my attempt at reconcilating the lockfile didn't work. 1 moment please 🎶

PR is merge-able now :D

I think this is good to move forward, but we'll need a code owner review (probably due to the Metadata Bar component) from @michael-s-molina @kgabryje or @geido to merge this.

hi Superset folks, anything I can do to have this merged?

Pinging those three folks again on Slack... hopefully we can get a code owner stamp soon.


Chinese  User

什么时候出汉化版
When will it be available in Chinese？

Closing as duplicate of https://github.com/microsoft/PowerToys/issues/199


Payment gateway with extra fees

it would be very nice if payment module comes with payment fees.

How do you think if we note the fee like the image below

In the Payment table,  we will add another columns to store this fee.
On the payment provider configuration, add a field for admin to enter the fee

yes look nice . where you will calculate % in papal component before sending request ?

I’m up and running, thank you Thien! I guess were waiting to be able to delete product options, etc!
Great job
Sent from my iPhone

On Mar 4, 2018, at 7:39 AM, Thien Nguyen Le Quynh<EMAIL_ADDRESS>wrote:
Closed #413.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


two minor tweaks that are useful (to me)

handling when someone recursively deletes a folder in git and actually removing it from p4 too.
also creating a link to the commit in git in the p4 checkin (i happen to use an internal github thus the parsing and not just straight github.com replacement... although if you don't end with .com it'll need to be changed)

just realized this isn't very platform agnostic of a change... find doesn't normally exist on windows :-/ ehh... still useful :)


Prevent dublicating hosted services registered with AddHostedService

Hi!
This issue related with https://github.com/aspnet/Extensions/pull/552
Is your feature request related to a problem? Please describe.
If one hosted service is registered in two or more parts of an application (with AddHostedService), this leads to the creation of several instances of this service.
// services.AddFirstComponentServices()
services.AddHostedService<MyHostedService>();

...

// services.AddSecondComponentServices()
services.AddHostedService<MyHostedService>();

Two instances of MyHostedService will be created and started.
I think this is not the expected behavior.
Describe the solution you'd like
Change AddHostedService to using TryAddEnumerable instead of AddSingleton or AddTransient.
Or add TryAddHostedService extension method.
public static class ServiceCollectionHostedServiceExtensions
{
    public static IServiceCollection AddHostedService<THostedService>(
        this IServiceCollection services, ServiceLifetime serviceLifetime = ServiceLifetime.Transient)
        where THostedService : class, IHostedService
    {
        services.TryAddEnumerable(
            new ServiceDescriptor(typeof(IHostedService), typeof(THostedService), serviceLifetime));

        return services;
    }
}

Describe alternatives you've considered
Manual hosted service registration without AddHostedService method.

Yes we should do this. @Chakrygin do you want to send a pull request?


Update index.md

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

ErikjeMSDocset0419171241/index.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.


transition-group appear class not disappear inside Suspense

Version
3.0.0-rc.5
Reproduction link
https://github.com/xcuiz/vue-next-test.git
Steps to reproduce
npm i
npm run dev
Start dev server and navigate to localhost:8080/index
What is expected?

vue-next 0

vue-next 1

vue-next 2

vue-next 3

vue-next 4

What is actually happening?

vue-next 0

vue-next 1

vue-next 2

vue-next 3

vue-next 4

e.g:
in parent vue component:
<Suspense>

            <template #default>

                <router-view></router-view>

            </template>

            <template #fallback>

                加载中...

            </template>

 </Suspense>

in child component:
<transition-group

                appear

                appear-to-class="appear-to"

                appear-active-class="appear-active">

                    {{item.userName}}

 </transition-group>

I removed the router from the demo I submitted a PR: https://github.com/xcuiz/vue-next-test/pull/1


Fix a few spellos

The majority of these are in comments.

Noticed the spello in MANY_WRITES_EXPETED and did a brief sweep of nearby files.

I thought about setting up https://github.com/codespell-project/codespell to avoid spelling errors from getting into the code base in the first place, but this certainly helps too

I thought about setting up codespell-project/codespell to avoid spelling errors from getting into the code base in the first place, but this certainly helps too

Yes, no objections to that. I have https://marketplace.visualstudio.com/items?itemName=streetsidesoftware.code-spell-checker install locally in vscode, but mostly just use it for new code.

Thanks for the review, especially at the weekend.


Refactor AddressBook to FindMyIntern

#102

Codecov Report

Merging #103 (bfec907) into master (ff15589) will not change coverage.
The diff coverage is 68.49%.

@@            Coverage Diff            @@
##             master     #103   +/-   ##
=========================================
  Coverage     69.14%   69.14%
  Complexity      473      473
=========================================
  Files            79       79
  Lines          1598     1598
  Branches        169      169
=========================================
  Hits           1105     1105
  Misses          437      437
  Partials         56       56

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/Model.java
100.00% <ø> (ø)

...du/address/model/internship/InterviewDateTime.java
64.28% <ø> (ø)

...a/seedu/address/model/internship/SortCriteria.java
75.00% <ø> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <0.00%> (ø)

...rc/main/java/seedu/address/logic/LogicManager.java
75.00% <60.00%> (ø)

...ain/java/seedu/address/storage/StorageManager.java
77.77% <71.42%> (ø)

...seedu/address/storage/JsonFindMyInternStorage.java
80.95% <80.95%> (ø)

...ava/seedu/address/logic/commands/ClearCommand.java
100.00% <100.00%> (ø)

...seedu/address/logic/parser/FindMyInternParser.java
75.00% <100.00%> (ø)

... and 3 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Resolve C4819 warnings for building on Windows using Visual Studio

Hello,
There are some C4819 warnings when building on Windows using Visual Studio. The long list of warnings is annoying when users try to find the build outputs for their codes in the build log.
The file contains a character that cannot be represented in the current code page (number). Save the file in Unicode format to prevent data loss.
Most of them are in the comments, for example https://github.com/mfem/mfem/blob/master/fem/estimators.cpp#L249, so resolving them won't affect the codes. I am glad to open a PR to address some of them in the sequential mfem code files, by using the latex equations. However, before doing that I need to ask the code owners about their choices.
Best,
Adam

We don't have policy against unicode characters in comments, and generally have found those to be useful in several places.
Is this really not allowed in typical Windows systems?
I  don't see it in our Windows CI builds. (I do see C4244, C4250, C4267, C4805 and C4996).

What version of MSVC are you using? This warning does not show up in our Appveyor CI either (e.g. here https://ci.appveyor.com/project/mfem/mfem/builds/43337614) which uses Visual Studio 15 2017.
When I open that file with VS Code, it does say "UTF-8" encoding, so I'm not sure what the compiler is complaining about. Maybe it expects the file to have BOM (https://en.wikipedia.org/wiki/Byte_order_mark)? Unfortunately, other compilers do not like BOM to the extend that they just crash if they encounter one. (some details can be found here: https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-1-c4819)
I suspect that there is a flag you can add to suppress this particular warning -- see https://docs.microsoft.com/en-us/cpp/build/reference/compiler-option-warning-level. Maybe we can add this flag by default in our CMake file(s) when building with MSVC.

Hi @v-dobrev , I am using Visual Studio 2022.

According to the log, the Windows CI that @tzanio linked above, https://github.com/mfem/mfem/issues/2976#issuecomment-1114350140, used the following:

-- Building for: Visual Studio 17 2022,
-- The CXX compiler identification is MSVC 19.31.31107.0,
-- Check for working CXX compiler: C:/Program Files/Microsoft Visual Studio/2022/Enterprise/VC/Tools/MSVC/14.31.31103/bin/Hostx64/x64/cl.exe - skipped

I'm not sure what to make of all these different version numbers. I assume, in your setup, something is different?
In any case, does adding the flag /wd4819 (to suppress this specific warning) work for you?

Hi @v-dobrev , I think the flag /wd4819 is the simplest way to suppress the warnings.

Hi @v-dobrev , I think the flag /wd4819 is the simplest way to suppress the warnings.

Okay. @tzanio what do you think about adding the flag /wd4819 to CMake when using MSVC? I'm fine with such addition.
@adam-sim-dev, if @tzanio is also fine with adding this to CMake, do you want to create a small PR for that?

@v-dobrev , should I add the following scripts to mfem/CMakeLists.txt?
if (WIN32)
  set_target_properties(${PROJECT_NAME} PROPERTIES COMPILE_FLAGS "/wd4819")
endif()

Which exact line do you expect it to appear in the CMakeLists.txt? Then I can open a PR.

what do you think about adding the flag /wd4819 to CMake when using MSVC? I'm fine with such addition.

sounds good to me

Hi @adam-sim-dev,
It is better to use target_compile_options (https://cmake.org/cmake/help/v3.8/command/target_compile_options.html) instead of set_target_properties and also check if (MSVC) instead of if (WIN32).
A good place to add this is after these lines: https://github.com/mfem/mfem/blob/343480ff18492f2d328986699f023bd75724ac2b/CMakeLists.txt#L546-L548
Thanks for taking care of this addition!

Discussion continued in #3004


stuck infinite loading loop with vitesse-like setup starting from v0.6.x

I upgraded to v0.6.0/0.6.1 and now when loading a page in development, the app is constantly reloading. I see this this in console:

and the counter (x185) continues to go up. Previous versions (v0.5.11) are fine.

some problem

I can't reproduce it on Vitesse.

Same issue here for sveltekit.
I could also not reproduce, until I added postcss/tailwind to the project.
Example repo with the infinite reload: https://github.com/RickMeijer/unplugin-autoimport-error
The behaviour goes away when you remove the css in index.svelte

It doesn't happen when I downgrade the autoimport to 0.5.x, which lead me to think it had to be a change here that caused the issue. No matter the cause: in my case it's fixed by making the content part of my tailwind more specific.
from
content: ['./src/**/*.{js,jsx,ts,tsx,svelte}'],

to
content: [
  './src/routes/**/*.{js,jsx,ts,tsx,svelte}',
  './src/app.html'
],

for example.
It entirely depends on where you write your *.d.ts file if you use one. Thank you for the active involvement @antfu , appreciate the work.

@antfu yes tailwindcss problem
me fixed

I think in the TW config you need to fine a way to exclude .d.ts file

@antfu problem still exist when I using Windi css, with auto import.

`

{
"name": "",
"version": "0.0.0",
"author": "",
"license": "",
"scripts": {
"start": "vite",
"build": "vue-tsc --noEmit --skipLibCheck && vite build",
"lint": "eslint src",
"preview": "vite preview"
},
"dependencies": {
"@intlify/unplugin-vue-i18n": "^0.9.2",
"@vueuse/core": "^9.13.0",
"axios": "^1.3.4",
"date-fns": "^2.29.3",
"naive-ui": "^2.34.3",
"nprogress": "^0.2.0",
"pinia": "^2.0.33",
"prism-theme-vars": "^0.2.4",
"qs": "^6.11.1",
"vue": "^3.2.47",
"vue-i18n": "^9.2.2",
"vue-router": "^4.1.6"
},
"devDependencies": {
"@iconify/json": "^2.2.35",
"@types/node": "^18.15.3",
"@types/nprogress": "^0.2.0",
"@types/qs": "^6.9.7",
"@types/svgo": "^2.6.4",
"@typescript-eslint/eslint-plugin": "^5.55.0",
"@typescript-eslint/parser": "^5.55.0",
"@vicons/carbon": "^0.12.0",
"@vicons/utils": "^0.1.4",
"@vitejs/plugin-legacy": "^4.0.1",
"@vitejs/plugin-vue": "^4.0.0",
"@vitejs/plugin-vue-jsx": "3.0.0",
"@windicss/plugin-scrollbar": "^1.2.3",
"eslint": "^8.36.0",
"eslint-config-airbnb-base": "^15.0.0",
"eslint-config-prettier": "^8.7.0",
"eslint-plugin-import": "^2.27.5",
"eslint-plugin-prettier": "^4.2.1",
"eslint-plugin-vue": "^9.9.0",
"lint-staged": "^13.2.0",
"markdown-it-link-attributes": "^4.0.1",
"markdown-it-prism": "^2.3.0",
"prettier": "^2.8.4",
"sass": "^1.59.3",
"typescript": "^4.9.5",
"unplugin-auto-import": "^0.15.1",
"unplugin-icons": "^0.15.3",
"unplugin-vue-components": "^0.24.1",
"vite": "^4.1.4",
"vite-plugin-fonts": "^0.7.0",
"vite-plugin-md": "^0.21.5",
"vite-plugin-windicss": "^1.8.10",
"vite-svg-loader": "^4.0.0",
"vue-tsc": "^1.2.0"
},
"type": "module",
"lint-staged": {
"*.{ts,js,vue}": [
"eslint"
]
}
}

`


[FEATURE] a separate quantification algorithm or ability to input a single x-y coordinate as the position of a cell (instead of a mask)

Is your feature request related to a problem? Please describe.
I am trying to use some existing data to create my own model (using human-in-the-loop feature), but the data we have from our lab only has the location of cells marked with an X.
Our existing data does not have each cell carved out as its own region with a given mask/ROI like the current cellpose training feature requires. Combing through 4-5 years worth of lab data again to create a training set is impractical.
Describe the solution you'd like
Ability to input a single set of x,y coordinates as the location of a cell and be able to use this to generate a cell count for an image or maybe use this data to generate masks of cells in the same image.
Describe alternatives you've considered
I have tried importing the locations of x-y coordinates that we have for various images and trying to generate masks by creating a circle centered around the point that has been marked. However, given that cells having varying morphologies with protrusions and occasionally overlap, this has not worked so well.
Additional context
Add any other context or screenshots about the feature request here.

example of image where the number of cells is quantified with a red 'X' instead of with masks
Another image where I use cyto3 to try to use segment cells but it is not very accurate (used red channel to segment)

This might be outside cellpose's scope.
Have you considered SAM (Segment Anything Model) and its derivatives for medical images? They work with annotations like yours to do a good job segmenting a 'thing' around it.

This might be outside cellpose's scope. Have you considered SAM (Segment Anything Model) and its derivatives for medical images? They work with annotations like yours to do a good job segmenting a 'thing' around it.

Apologies if my question is unclear. I am not trying to generate the yellow lines (big segmentation). I am trying to generate small masks for each individual cell. (number of masks I expect to generate in this image is around the number of red 'X's that are in the labelled image)
I would presume this is within the purview of CellPose since I am indeed trying to segment an image of cells; I just don't have the training data in the right format and was wondering if there was a better way to convert my training data into usable input data.

@tanpancodes cellpose requires pixel-wise instance labeled cell masks to train a model. There is no way to determine which pixels belong to a cell from just a single centroid label. Unfortunately, your request is outside of the scope of cellpose because cellpose is a segmentation algorithm. In line with @ashishUthama suggestion, you can look for object detection algorithms for your purpose. Qupath and napari may have some solutions/plugins available
However, using the cellpose GUI with HITL training approach may be faster than you suspect and get the job done once you have an adequately finetuned model


Export to image?

Is it possible to export h264 frame data to image?
Would this be difficult to add?
If possible and you can't work on it please put pointers the best way you think to do it.

That would be a useful feature.
And not in a decode-reencode way, but to just dump the slice and any
metadata like SPS/PPS/etc blocks that are in effect for it.
On Friday, December 28, 2018, Edin Mujagic<EMAIL_ADDRESS>wrote:

Is it possible to export h264 frame data to image?
Would this be difficult to add?
If possible and you can't work on it please put pointers the best way you
think to do it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/shi-yan/H264Naked/issues/4, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQedq2tZNnzOMallI4SOmnNzvl3yR4gks5u9oM7gaJpZM4ZkNRt
.

should be doable. if you don't need to see the images, it should be very straight forward.
if you need to see the image, you might use avcodec for decoding.
This is just a quick weekend hack, I haven't maintained it for quite long time.


[Test] fix the unit tests

For some reason, the tests are now failing. Fix them so that CI will fully work.
Version
@nuxtjs/cloudinary
nuxt:
Reproduction Link

Steps to reproduce
What is Expected?
What is actually happening?

Fixed with https://github.com/nuxt-modules/cloudinary/commit/99be07c4383069d06a2af178d210583189a7c974


feat: benchmark document construction

Please don't forget to add python dependencies to requirements.txt file.

Please don't forget to add python dependencies to requirements.txt file.

Which dependencies do u see?

Which dependencies do u see?

Does numpy come with jina itself? Sorry, I am not sure.


Test Coverage Error

The first issue that I ran into, trying to copy the Booking example in order to use go test for full coverage. When I was trying to use the Get method for the testing suite, it would instantly give me a
revel  panic.go:522: Recovered error in startup                error="runtime error: invalid memory address or nil pointer dereference"
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
        panic: Fatal error in startup
[signal 0xc0000005 code=0x0 addr=0x8 pc=0x631651]

After a few hours of debugging, I finally realized that this was actually the correct response for the test, because it was actually the Get method was appending the wrong URI, causing a panic into a test failure.
What I had to do was actually use GetCustom, and used this instead.
import(
	"testing"
	"github.com/revel/modules/server-engine/gohttptest/testsuite"
)
func TestForGETRequests(t *testing.T) {
	suite := testsuite.NewTestSuite(t)

	url := "http://" + suite.Host() + "/somePath"
	suite.GetCustom(url)
}

However, the Get method works normally using the normal test format from the main page tutorial, but I think it's better to use this way to get test coverage.
I was wondering why this was an issue, and if anyone ran into this issue too, I hope it helps.

At a closer look, GetCustom looked like it worked, but it actually does nothing. We don't get a response to decode, because there was nothing that decoded the response. When adding .Send() method at the end of it, it panics again.
Here is the stack trace,
goroutine 21 [running]:
github.com/revel/revel.Run.func1()
        C:/Users/Spots/go/src/github.com/revel/revel/server.go:70 +0x158
panic(0xaeac60, 0x11f6140)
        C:/Programming Languages/Go/src/runtime/panic.go:522 +0x1c3
net/url.(*URL).String(0x0, 0x85e578, 0xc000144448)
        C:/Programming Languages/Go/src/net/url/url.go:777 +0x51
ffxiv-profit-api/app.glob..func1(0xc00029a820, 0xc000144490, 0xd, 0xe)
        C:/Users/Spots/go/src/ffxiv-profit-api/app/init.go:65 +0xac
github.com/revel/revel.glob..func33(0xc00029a820, 0xc000144488, 0xe, 0xf)
        C:/Users/Spots/go/src/github.com/revel/revel/watcher.go:298 +0x84
github.com/revel/revel.handleInternal(0xd42bc0, 0xc000293060)
        C:/Users/Spots/go/src/github.com/revel/revel/server-engine.go:193 +0x3b9
github.com/revel/modules/server-engine/gohttptest/testsuite.(*GoHttpServer).Handle(0xc000278e40, 0xd4b800, 0xc0001a6e00, 0xc0003f7900)
        C:/Users/Spots/go/src/github.com/revel/modules/server-engine/gohttptest/testsuite/engine_adapter_go_httptest.go:108 +0x381
github.com/revel/modules/server-engine/gohttptest/testsuite.(*GoHttpServer).Start(0xc000278e40)
        C:/Users/Spots/go/src/github.com/revel/modules/server-engine/gohttptest/testsuite/engine_adapter_go_httptest.go:73 +0x98
github.com/revel/revel.Run(0xffffffffffffffff)
        C:/Users/Spots/go/src/github.com/revel/revel/server.go:88 +0x228
ffxiv-profit-api/app/tmp/run.Run(0xffffffffffffffff)
        C:/Users/Spots/go/src/test-application/app/tmp/run/run.go:28 +0x37
github.com/revel/modules/server-engine/gohttptest/testsuite.RevelTestHelper.func2(0xbcb950)
        C:/Users/Spots/go/src/github.com/revel/modules/server-engine/gohttptest/testsuite/test_helper.go:53 +0x36
created by github.com/revel/modules/server-engine/gohttptest/testsuite.RevelTestHelper
        C:/Users/Spots/go/src/github.com/revel/modules/server-engine/gohttptest/testsuite/test_helper.go:52 +0x283

This also doesn't work for any of the other HTTP methods.

@icywaffle did you find a solution? If not, can you reproduce the issue with a new revel app (revel new testapp)?


1.19.2??

To which version?
can you please make a forge 1.19.2 version?
thanks in advance

Duplicate, will be added in NTL 0.2 along with Fabric support

ok!

Hello, what with 1.19.2 port?


[2402] Add blazer

Context
We want to provide a DB interface for our BAs to be able to analyse register data. Blazer will give us that and has been successfully used in Apply.
Changes proposed in this pull request

Install blazer
Add a route constraint to make it system_admin only

Guidance to review
Using persona login locally, review apps don't have the env yet:

Login as an admin, navigate to /system-admin/blazer and marvel at the DB interface
Login as a non-admin, navigate to /system-admin/blazer and you should be returned to sign-in
Logout, navigate to /system-admin/blazer and you should be returned to sign-in

I've not linked to this page as yet. The BAs can go directly via URL. It's not something that most people would be concerned with.

 Add blazer 🧥


Translate pagination buttons

The labels of the "previous" and "next" buttons are translatable with these changes.

Thanks for the translations. I suggest to change the Dutch labels to the following values:
 - id: paginationPrevious
   translation: "Vorige"
 - id: paginationNext
   translation: "Volgende"

I suggest to change the Dutch labels to the following values:

Sure!
Sidenote: I used Google Translate and was unsure about the "De". Actually I'm learning a bit Dutch at the moment and take this as a TIL. Hope it was okay to do it so.
Nevertheless, changes coming up in a minute. ☺

Sidenote: I used Google Translate and was unsure about the "De". Actually I'm learning a bit Dutch at the moment and take this as a TIL. Hope it was okay to do it so.

i appreciate the effort! You were pretty close… :-)
I suggest to translate the various aria-label attributes too, to improve accessibility. Agree?


feat: Update Arkworks' dependencies on acir_field

Related issue(s)
Resolves #68
Description
Summary of changes
Update Arkworks' dependencies on acir_field's Cargo.toml.
Dependency additions / changes

Update ark-bn254 from 0.3.0 to 0.4.0.
Update ark-bls12-381 from 0.3.0 to 0.4.0.
Update ark-ff from 0.3.0 to 0.4.0.
Add ark-serialize 0.4.0.
Remove ark-bn254 from dev-dependencies.

Test additions / changes

Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[x] I have linked this PR to the issue(s) that it resolves.
[x] I have reviewed the changes on GitHub, line by line.
[x] I have ensured all changes are covered in the description.

Additional context
The additional context was provided in #68

This looks good, thanks! Just to make sure it doesn't break anything, I'll run tests against the noir compiler before merging

This looks good, thanks! Just to make sure it doesn't break anything, I'll run tests against the noir compiler before merging

Nice! Let me know if there is something more for me to do in the case that the tests don't pass


No master changes

There seems to be a lot of commits in master that are not related to latest 3.1.x release, but master changelog is empty. Is it possible to update it periodically?
https://github.com/spyder-ide/spyder/compare/3.1.x...master

You can take a look at the milestone that tracks master right now: https://github.com/spyder-ide/spyder/milestone/23
Or you can use our own Loghub to get a more clear view of what's been added or changed. We created Loghub and use milestones heavily to avoid updating our changelog a lot.

That's crazy, but awesome! :D


Create tags on push

This PR will compile and push protos to a PR when that PR is approved. Then, once the PR is merged, it will tag the release and push it to buf.

OK @edaniels I've tested this locally using act and on my own fork. Things seems to be working

Should we enable auto merge as well once the check passes?


Add OP_CHECKSIGADD and OP_SUCCESSxxx

Adds support for CHECKSIGADD and OP_SUCCESSxxx. BIP 342 link for reference https://github.com/bitcoin/bips/blob/master/bip-0342.mediawiki#specification .
Also makes a minor change to add INVALIDOPCODE (was previously OP_RETURN)

If I understand this correctly this change assumes everything is Tapscript. That doesn't seem right as old script will continue to exist. Perhaps put opcodes into different submodules?

That doesn't seem right as old script will continue to exist. Perhaps put opcodes into different submodules?

Yes, this is correct. I am in favor of doing this, but this seemed too much code diff for opcodes that will never be used (and are not meant to be used). And I was not too concerned about this because this is handled correctly with ScriptContext in rust-miniscript.
If you feel like the diff(a new opcodes submodule along with APIs for it, and updating all usage of current APIs) is worth it, I don't mind updating the PR.

never be used (and are not meant to be used)

Ah, true, but I think so are OP_SUCCESSxxx until there is a new soft fork making them useful?

Here is another possible alternative. We keep the opcodes as they are, only add CHECKSIGADD and classify API as shown below
/// Classification context for the opcode. Some opcodes like `OP_RESERVED`
/// abort the script in [`Legacy`] context, but will act as OP_SUCCESS in tapscript
#[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]
pub enum ClassifyContext {
    /// Opcode used in tapscript context
    TapScript,
    /// Opcode used in legacy context
    Legacy,
}

impl All {
    /// Classifies an Opcode into a broad class
    #[inline]
    pub fn classify(self, ctx: ClassifyContext) -> Class;

I like this alternative ... we already have issues with opcodes having multiple names (e.g. OP_TRUE vs OP_1, or OP_0 vs OP_FALSE vs OP_PUSHBYTES_0, or OP_NOPx vs OP_CSV, etc etc). So I think it's okay to rely on the classifier to figure out what an opcode "really" is.

Oops, I think I misclicked close

I will do a separate commit + PR to your branch to refactor classificator match as suggested by
Kixunil to make it more readable and address this issue

Thanks :)

@sanket1729 sorry it happened not to be a PR but just commit - I didn't expect that I can push to your branch directly :( Should I force-push without my commit and to a PR from outside instead?

sanket1729 sorry it happened not to be a PR but just commit - I didn't expect that I can push to your branch directly :( Should I force-push without my commit and to a PR from outside instead?

No worries. Looks like I explicitly gave all maintainers the permission to edit it.


test server using postgresql

Test that the SlipStream server behaves correctly when pointed to a postgresql database.  Ensure that the correct database configuration is available for both the java and clojure sides.

During the server startup, test for the existence of the user super fails in the User.load() with
org.postgresql.util.PSQLException: ERROR: column user0_.resourceuri does not exist
Checking in the DB, the User table is not there, while in the debugging mode it's possible to see that hibernate executes the corresponding CREATE statement:
Hibernate:
    create table User (
        resourceuri varchar(255) not null,
        category int4,
        creation timestamp,
        deleted boolean not null,
        description varchar(1024),
        jpaVersion int4 not null,
        lastModified timestamp,
        activeSince timestamp,
        email varchar(255),
        firstName varchar(255),
        isSuperUser boolean not null,
        lastExecute timestamp,
        lastName varchar(255),
        lastOnline timestamp,
        name varchar(255),
        organization varchar(255),
        password varchar(255),
        state varchar(255),
        primary key (resourceuri)
    )

It turns out that in the PostgreSQL USER is a reserved key word http://www.postgresql.org/docs/9.4/static/sql-keywords-appendix.html.  However, hibernate doesn't detect the table creation failure, which can be clearly seen when creating manually:
postgres=# CREATE DATABASE slipstream;
CREATE DATABASE
postgres=# \c slipstream
You are now connected to database "slipstream" as user "postgres".
slipstream=# CREATE TABLE User (resourceuri varchar(255) not null);
ERROR:  syntax error at or near "User"
LINE 1: CREATE TABLE User (resourceuri varchar(255) not null);
                     ^
slipstream=#

The failure to detect the failure during the table creation looks to be a bug in Hibernate.
http://stackoverflow.com/questions/22256124/cannot-create-a-database-table-named-user-in-postgresql
Setting the name to the User entity as 'User_' and updating the named queries resolves the issue.

Closing.  Continues in https://github.com/SixSq/tasklist/issues/367


Add comment to show how test package can be excluded from installation

After signficant development effort a python package / system / app may be deployed to users without the tests directory, as end users would not be expected to run the tests.
The full range of tests would be available for use by developers checking out the source code from a source archive
The setup.py file can be amended
packages=find_packages(exclude=["tests*"]),

Done


How to download co_dino_5scale_lsj_vit_large_o365.pth?

Good works! Could you tell me how to download co_dino_5scale_lsj_vit_large_o365.pth? Many thanks

Please email me to obtain the weights.


Menu entry swaps for clan chats

Having a rank in clan chats and the ability to kick can be problematic due to left click option being kick user. This could cause accidental kicks from missclicking on people in the clan chat.
Perhaps a menu entry swap for clan chats can be made to move the kick user option to the bottom of the right click menu and prioritizing hop-to option instead

The right-click menu is opened by default on left click, and that behavior is included in the vanilla client, so this shouldn't be an issue.


lightning app

Hello @rasbt! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file app/app.py:

Line 12:80: E501 line too long (118 > 79 characters)
Line 17:80: E501 line too long (83 > 79 characters)
Line 19:80: E501 line too long (102 > 79 characters)
Line 27:33: E261 at least two spaces before inline comment
Line 36:12: E222 multiple spaces after operator
Line 39:80: E501 line too long (83 > 79 characters)
Line 40:80: E501 line too long (84 > 79 characters)
Line 41:80: E501 line too long (85 > 79 characters)
Line 42:80: E501 line too long (100 > 79 characters)
Line 99:77: W291 trailing whitespace
Line 100:80: E501 line too long (86 > 79 characters)
Line 101:80: E501 line too long (88 > 79 characters)
Line 104:1: W293 blank line contains whitespace
Line 105:68: W291 trailing whitespace
Line 106:80: E501 line too long (105 > 79 characters)
Line 108:80: E501 line too long (143 > 79 characters)
Line 109:80: E501 line too long (151 > 79 characters)
Line 110:80: E501 line too long (176 > 79 characters)
Line 111:1: W293 blank line contains whitespace
Line 115:80: E501 line too long (144 > 79 characters)
Line 119:80: E501 line too long (99 > 79 characters)
Line 150:33: W292 no newline at end of file

In the file app/components.py:

Line 11:1: E303 too many blank lines (3)
Line 56:9: E265 block comment should start with '# '
Line 57:80: E501 line too long (105 > 79 characters)
Line 61:10: W292 no newline at end of file

In the file app/helper.py:

Line 3:1: E302 expected 2 blank lines, found 1
Line 36:28: W292 no newline at end of file


mac-28 lost external connection phone device.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-28 lost external connection phone device.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

Rebooting machine to restore connection.


experiment: does this trigger from PR allows us to access secrets env?

Do not merge

Closing it since it doesn't allow to access env secrets


Try to use leveldown@5 as a backend

To see if there can fix the segment fault problem.
See: https://github.com/Chatie/wechaty/issues/1355

See #45


[DOCS] Use newer GitHub Docs link

Description
In the contributing guidelines file, there's a link to a GitHub Docs article, however, it's using the old format (help.github.com, which is now docs.github.com). It still redirects you, but it is best to use the latest links, IMO.
Steps to contribute

Fork the repository here.
Create a new branch named update_github_link.
Head over to the CONTRIBUTING.md file.
In line 94, update the help link to: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request.
Commit and push your changes.
Create a pull request to compare your changes.
That's it!

If you need any help with this, let us know about it. 🙂
Screenshots
No response
Additional information
This is a good first issue. Please leave it open to first-time contributors and beginners.

hey, can you provide some additional information? Like what exactly I've to work on

hey, can you provide some additional information? Like what exactly I've to work on

Hey there. There's a link that needs updating in the contributing guidelines file (see all the steps for more information). 🙂
If you're still confused or need any help, let us know about i!

Hey, thanks for that; I received it!
While I notice that you have included all of the stages in the file, I propose that you edit it further. Because most of the steps provided here aren't really abundantly clear. If you need it changed, I can do so, and if you like it, you can include the modifications. First, I attempt to resolve the issue raised in this post, namely, updating the link in Docs.

Sure, that'd be great, thanks! Feel free to create separate PRs for those. :)

Sure! I'll be working on it.

Hey, I updated the CONTRIBUTING.md file by rewriting the guidelines and providing information for contributors in a more clear and more minimal approach. Please provide comments so that I can create any additional add-ons or modifications.

Done.

Hey There! This issue can be closed now.

Hey There! This issue can be closed now.

Hmm, it appears you created the PR in your fork and not the original repository. 😅

Hii, I am looking  for project to contribute .Can I work on this please assign me

this issue is closed?

this issue is closed?

co-asks 👀


[BUG] Searches 29/30 and 30/30 routinely fail, but actually work....

Before submitting a bug report...

[X] This bug wasn't already reported.
(I have checked every bug report on GitHub)

[X] I've cleared the sessions folder.

Title

[X] The title is no longer "[BUG] Title" and I edited it with the right error name.

Branch
master
Commit
0.1.0
Describe the bug
Just about every run for me, desktop searches 29/30 and 30/30 fail and it does the full set of retries. The system already counted the points, causing the failures. Could a few people check their logs for past few days to see if just me? Not a major issue, just adds 30 minutes of run time.
Copy and paste your error
2024-08-09 04:36:08,897 [DEBUG] rootTerm=nikki hiltz
2024-08-09 04:36:09,017 [DEBUG] term=nikki hiltz
2024-08-09 04:36:10,367 [DEBUG] [BING] Search attempt failed 1/7, sleeping 14 seconds...
2024-08-09 04:36:24,495 [DEBUG] term=nikki hiltz born girl or boy
2024-08-09 04:36:25,817 [DEBUG] [BING] Search attempt failed 2/7, sleeping 28 seconds...
2024-08-09 04:36:53,990 [DEBUG] term=nikki hiltz runner
2024-08-09 04:36:55,386 [DEBUG] [BING] Search attempt failed 3/7, sleeping 56 seconds...
2024-08-09 04:37:51,524 [DEBUG] term=nikki hiltz gender
2024-08-09 04:37:53,058 [DEBUG] [BING] Search attempt failed 4/7, sleeping 112 seconds...
2024-08-09 04:39:45,363 [DEBUG] term=nikki hiltz born male
2024-08-09 04:39:46,718 [DEBUG] [BING] Search attempt failed 5/7, sleeping 224 seconds...
2024-08-09 04:43:30,867 [DEBUG] term=nikki hiltz wikipedia
2024-08-09 04:43:32,134 [DEBUG] [BING] Search attempt failed 6/7, sleeping 448 seconds...
2024-08-09 04:51:00,321 [DEBUG] term=nikki hiltz olympics
2024-08-09 04:51:01,757 [DEBUG] [BING] Search attempt failed 7/7, sleeping 896 seconds...
2024-08-09 05:05:57,908 [DEBUG] term=nikki hiltz gender at birth
2024-08-09 05:05:59,290 [ERROR] [BING] Reached max search attempt retries
2024-08-09 05:05:59,294 [DEBUG] Moving passedInTerm to end of list
2024-08-09 05:06:09,297 [INFO] [BING] 30/30
2024-08-09 05:06:10,036 [DEBUG] terms=['usa vs serbia basketball score', 'usa vs serbia basketball', 'usa vs serbia basketball 2024', 'usa vs serbia basketball box score', 'usa vs serbia basketball 2014', 'usa vs serbia basketball olympics 2024', 'usa vs serbia basketball live', 'usa vs serbia basketball abu dhabi', 'usa vs serbia basketball stats', 'usa vs serbia basketball online', 'usa vs serbia basketball highlights', 'usa vs serbia basketball abu dhabi tickets', 'usa vs serbia basketball live score', 'usa vs serbia basketball 2024 score', 'usa vs serbia basketball 2024 box score', 'serbia vs usa basketball final score', 'team usa basketball vs serbia', 'women basketball usa vs serbia', "usa men's basketball vs serbia"]
2024-08-09 05:06:10,036 [DEBUG] rootTerm=usa vs serbia basketball
2024-08-09 05:06:10,343 [DEBUG] term=usa vs serbia basketball score
2024-08-09 05:06:11,841 [DEBUG] [BING] Search attempt failed 1/7, sleeping 14 seconds...
2024-08-09 05:06:25,980 [DEBUG] term=usa vs serbia basketball
2024-08-09 05:06:27,338 [DEBUG] [BING] Search attempt failed 2/7, sleeping 28 seconds...
2024-08-09 05:06:55,557 [DEBUG] term=usa vs serbia basketball 2024
2024-08-09 05:06:56,939 [DEBUG] [BING] Search attempt failed 3/7, sleeping 56 seconds...
2024-08-09 05:07:53,164 [DEBUG] term=usa vs serbia basketball box score
2024-08-09 05:07:54,590 [DEBUG] [BING] Search attempt failed 4/7, sleeping 112 seconds...
2024-08-09 05:09:46,721 [DEBUG] term=usa vs serbia basketball 2014
2024-08-09 05:09:48,264 [DEBUG] [BING] Search attempt failed 5/7, sleeping 224 seconds...
2024-08-09 05:13:32,440 [DEBUG] term=usa vs serbia basketball olympics 2024
2024-08-09 05:13:33,768 [DEBUG] [BING] Search attempt failed 6/7, sleeping 448 seconds...
2024-08-09 05:21:01,947 [DEBUG] term=usa vs serbia basketball live
2024-08-09 05:21:03,321 [DEBUG] [BING] Search attempt failed 7/7, sleeping 896 seconds...
2024-08-09 05:35:59,667 [DEBUG] term=usa vs serbia basketball abu dhabi
2024-08-09 05:36:01,111 [ERROR] [BING] Reached max search attempt retries
2024-08-09 05:36:01,111 [DEBUG] Moving passedInTerm to end of list
2024-08-09 05:36:16,116 [INFO] [BING] Finished Desktop Edge Bing searches !
Screenshots
Can't do, as in docker
Value of dashboard variable
having trouble withthis

Yeah this has consistently happened with the refactor in #137. The underlying cause appears to be a lag between the searches being done on the bot end, and the searches counting on the Microsoft server-side.
My initial thought was it had something to do with the way we get the current account points, but when I tried an alternate way it was the same result which makes me think it's server-side and out of our control.
Since the searches generally complete for me (even if logging indicates otherwise), and apprise.summary=ON_ERROR notifies me via email if not all are done, I haven't really bothered to dig into it further.

Actually I'm facing a similar issue, but not just on searches 29 and 30,also on middle ones, and many time I'm seeing the bot is still waiting maybe for search 25 while looking to the ms page it already have finished all the searches. Actually I'm trying this code, do you see any possible problem with it? It seems to work....
       remainingSearches = self.browser.getRemainingSearches()         searchCount = 1         while searchCount < remainingSearches:         #for searchCount in range(1, remainingSearches + 1):             # todo Disable cooldown for first 3 searches (Earning starts with your third search)             logging.info(f"[BING] {searchCount}/{remainingSearches}")             self.bingSearch()             time.sleep(random.randint(10, 15))             if searchCount % 3:                 searchCount = remaining Searches - self.browser.getRemainingSearches() - 1             searchCount += 1

I suggested something like that a while back, but it was rejected due to possibly going for ever if getRemainingSearches() did not update . Perhaps if add
while searchCount < remainingSearches and searchCount < 30 : to ensure it will never go on forever

Good suggestions, but getRemainingSearches relies on src.utils.Utils.getDashboardData which for me had the same disconnect that src.utils.Utils.getAccountPoints had.
That being said, I could be conflating what I saw earlier when I tried using dashboard data versus bing info for figuring how many remaining points.
I'll give it a try at some point to see if it's more reliable.

As a follow up to https://github.com/klept0/MS-Rewards-Farmer/issues/172#issuecomment-2283966228, what I think's happening is that the re-attempts during the disconnected phase register as further searches, which then leads to the 1 search per call to bingSearch not holding true.

Should be fixed in #184


Fixes #8329: Added support of options for message console command

Replaces https://github.com/yiisoft/yii2/pull/8739

@yiisoft/core-developers need review. Especially:

framework/console/Controller.php.
framework/console/controllers/MessageController.php.

Looks great.
I'm not sure whether we need to add @since everywhere :)


Update dependencies

Hi, @seybi87. Can you please merge it?

@fsanaulla  thanks a lot for your contribution.
Before merging the updates to the master can you confirm that the Livy Server 0.6.0 and the Spark Version 2.4.4 are still compatible with the currently used versions of the Livy Server and Spark?

I'll provide few updates tomorrow, and check it

Let's maybe build it as a separate branch? Not sure is it compatible with the previous version


feat(expansion): Allow expansion indicator positioning

#6726
Allows expansion indicator to be positioned at start, end and hidden.

@jelbourn When you have a moment can you take a look at this PR?  I don't remember where we landed on this one.

@josephperrott as discussed we're going to keep hideToggle and make togglePosition separate to avoid the breaking API change.

Closing in favor of #8199


feat: simple testing library

Alternative to #32
This takes more of a lightweight approach where we provide a bare minimum api for testing rather than controlling the whole testing pipeline. This should allow for better flexibility as we're no longer tied to any particular testing framework.
Notes:

Had to rename the directory of the vitest addon as it was somehow interfering with vitest or tinypool during runtime, causing shit to randomly break
Storybook has been pinned to v8.3.x as v8.4.x flat out doesn't work. I haven't had a moment to fully investigate the issue yet.

TODO:

Snapshots?
look into using vitest workspaces
Should we continue to use degit to clone the vite templates? If so, should we pin it to a specific commit hash (see here)? Or should this be something we manage fully (perhaps as part of create)?

Ohhh and there is another thing I just noticed: How are the directory names inside .test-output/_tests generated? Here are my directories inside drizzle
-1930824653_0
-1930824653_1
-1930824653_2
-1930824653_3
-1930824653_4
-1930824653_5
-1930824653_6
-1930824653_7

They are semi-useful at least. In #32 I generated them based on the options passed for each test, so that if a test failed you could identify it by directory name and run it by hand.

How are the directory names inside .test-output/_tests generated? Here are my directories inside drizzle

they're based on the task id provided by vitest. Each test gets their own meta.json (similar to how we had it in svelte-add) that documents the options that were passed in.

They are semi-useful at least. In #32 I generated them based on the options passed for each test, so that if a test failed you could identify it by directory name and run it by hand.

not sure if that's a scalable solution. imagine having multiple test cases that share the same addon options but they evaluate different things


Testing: run mocha tests in ci

We should add the mocha tests to run in CI

Closing as not planned


Update cloud-cost-management.md

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

 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.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6647509833bf64c2e3c7832b--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://664754ed84ebc7c704d9f27e--harness-developer.netlify.app


feat(legacy-plugin-chart-country-map): added partial finland map

🏆 Enhancements - added finland map (partial)

Thanks @Anushka-Hapukotuwa for this PR 🇫🇮! Would you be open to adding the rest of the missing regions? https://en.wikipedia.org/wiki/ISO_3166-2:FI

@villebro This pull request was created for a testing purpose. If this is successful, can add the complete geojson file with all the regions.

Complete finland geojson with ISO codes added


Added the game

#4850

Close #4850

Checklist:

[X] I have mentioned the issue number in my Pull Request.
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have created a helpful and easy to understand README.md

@MohitGupta121 @TusharKesarwani added the game please review it

@MohitGupta121 @TusharKesarwani please review it

@MohitGupta121  @TusharKesarwani  pleae review it asap


fix: Changed "login anonymously" text color #566

Fixes #566
**My PR closes #566 **
👨💻 Changes proposed
Changed "Login Anonlymously" text color to white
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title and description of the PR is clear and explains the approach.

Note to reviewers

📷 Screenshots

PR Compliance Checks
Thank you for your Pull Request! We have run several checks on this pull request in order to make sure it's suitable for merging into this project. The results are listed in the following section.
Conventional Commit PR Title
In order to be considered for merging, the pull request title must match the specification in conventional commits. You can edit the title in order for this check to pass. Most often, our PR titles are something like one of these:

docs: correct typo in README
feat: implement dark mode"
fix: correct remove button behavior

Linting Errors

Found type "null", must be one of "feat","fix","docs","style","refactor","perf","test","build","ci","chore","revert"
No subject found

@Raj100679
Please fix this error, if you don't know how, try this:

Read the error carefully or read the contributing file
Checkout past merged Pull requests to see how they were done for it to be merged

Note: you can edit your PR message, this PR won't be merged if the error isn't gone and you have a week max to fix it before the PR gets closed without being merged

@Dun-sin Can you check this PR


Server Performance Advisor download link does not work

Server Performance Advisor download link does not work
https://docs.microsoft.com/en-us/windows-server/administration/server-performance-advisor/microsoft-server-performance-advisor#bkmk-downloadspa

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

ID: 697b9e0a-e082-cadd-3c08-ffb54f640e45
Version Independent ID: 45250bd4-d7e9-6e79-a6d4-80d169cb5ac7
Content: Microsoft Server Performance Advisor
Content Source: WindowsServerDocs/administration/server-performance-advisor/microsoft-server-performance-advisor.md
Product: windows-server
Technology: administration
GitHub Login: @eross-msft
Microsoft Alias: lizross

Confirmed. The download links are as follows:

Microsoft Server Performance Advisor 3.1 (32-bit): https://go.microsoft.com/fwlink/p/?linkid=327751
(points to http://download.microsoft.com/download/0/3/D/03D07D11-18D4-4160-B4AC-915061B85669/SPAPlus_x86.cab)
Microsoft Server Performance Advisor 3.1 (64-bit): https://go.microsoft.com/fwlink/p/?linkid=327752
(points to http://download.microsoft.com/download/0/3/D/03D07D11-18D4-4160-B4AC-915061B85669/SPAPlus_amd64.cab)

Error message:

By the way, I also tested changing the http:// link to https:// - but still the same error page.

@xcabur1 @illfated  I also see the 404 error. I'm looking into this to see if I can find the article owner to provide updated info. I'll let you know if I find out anything.

Thanks. I don't know if the target files have been removed because of potentially unsafe software due to old and outdated versions, but it is a possible scenario.
Best case scenario would be if there are newer versions of the same files to link to the referred page in question.
Second best option would be if new scripts or methods have replaced the Server Performance Advisor software.

@illfated  Current thoughts are that it was taken down for the same reason dozens of other downloads were taken down in the past year: it used SHA-1 signature, and no owners were around to update it with a modern signature.
But I'm hoping we can find someone to help us do something about this...

Thank you for that very useful reminder. I had completely forgotten about it, since my daytime job challenges have occupied most of my thoughts lately.
Anyway, I will be looking forward to more feedback later regarding this content. Thank you for keeping us informed. 👍

( ref. #5551 )

@eross-msft : Do you know if anyone new will be assigned to this article & issue ticket?

#assign:xelu86

The microsoft-server-performance-advisor.md article has either been deprecated or absorbed into another article. Closing threads #4905 & #5551 as updates cannot be made.
#please-close


How to make TotalThroughputSampler in terms of events per second

We have traces that vary in in length substantially, and want to use TotalThroughputSampler to help us stay within rate limits, but having it sample in terms of traces per second results in too much variability in the output events per second.
Calls to the underlying dynsampler are done one per trace, which to my understanding increments the internal counter once per trace:
https://github.com/honeycombio/refinery/blob/v1.4.0/sample/totalthroughput.go#L58
To make this in terms of events per second, I see the only way to do this would be to call that line above in a loop, where the iterations == the number of spans in the trace, but that would be a pretty inefficient way of making this work. What approach would you recommend?

Hey @william-tran - thanks for creating the issue.
We agree incrementing the counter in a loop for each span within a given trace would be inefficient and wouldn't be compatible with current usage of the TotalThroughputSampler.
We'd need sometime to think about this, we'll put it on our backlog for prioritisation and may get back in touch to ask more questions. Thank you.

This was done in Refinery 2.0!


[docs] Add banner in solidarity of Ukraine

[x] I have followed (at least) the PR section of the contributing guide.

Let me know any improvement to the writing or anything else.
Preview: https://deploy-preview-31275--material-ui.netlify.app/getting-started/installation/

There is a visual bug. Also, is the yellow expected?

@oliviertassinari

There is a visual bug.

It was solved in the most recent commit but for some reason, Netlify deploy preview is taking too much time to update the Preview link...

Also, is the yellow expected?

Well, I guess you didn't notice 😅 But it is the creative solution I mentioned above: I've created the Ukraine flag within the TOC banner to avoid using the emoji (only available on macOS, as you've pointed out).

IMO a flag image would work better.
But aside from that, I think it's important to publish it soon. So let's merge and publish it as soon as possible and then improve if needed.

a flag image would work better.

Yup, followed your suggestion! I think we're ready to go now.

If we get feedback like https://github.com/facebook/create-react-app/pull/12113#issuecomment-1057848416, 👍 to only scope it to browsers that have Russian as an accepted language. It should approximate https://inews.co.uk/news/un-vote-ukraine-russia-countries-abstained-general-assembly-result-resolution-explained-1495346 well.


Fold ask implementations for ListAnswer and OrderedListAnswer together

Actually the mechanics of answering the two different kinds of questions are different enough that it would be easier to keep them separate.


add a delete announcement handler/endpoint

As an admin, I want to be able to delete an announcement, so that I can remove incorrect/outdated info
Definition of Done

[ ] add handler and endpoint

Tasks

[ ] create delete handler and endpoint for announcements

created the handler and dao function to support deleting specific announcements, will start testing it now to ensure it works correctly.
[t:25]

Tested it, made a few slight changes and it works! Ready to be pushed into main...
[t:15]


Enhancement: Column definition feature

Description
The user should be able to define from their dataset what columns correspond to what, so the app could do the assignation for them. In other words, instead of the hardcoding of the columns as it is done now, users should be able to pick with different inptus which column corresponds to:
When implementing take into consideration that these variables are used by PKNCA::PKNCAconc, PKNCA::PKNCAdose and PKNCAdata.
Basic Variables

AVAL: Actual Value of Analyte Concentration
ANALYTE: Analyte measured
ADOSE: Actual Value of Dose Concentration
DRUG: Treatment or Drug administrated

Time variables

ARRLT: Actual Relative Time from Reference/Previous Dose
AFRLT: Actual Relative Time from First Dose

Unit measures

DOSEU: Unit Value of ADOSE
AVALU: Unit Value of AVAL
RRLTU: Unit for all time variables (e.g, "hours")

Grouping Concentration Variables (used in formulas of PKNCA functions)

STUDYID: Identification for the Study
DOSNO: Dose/Cycle number
ROUTE: Route of administration ("Intravascular" or "Extravascular")
PCSPEC: Specimen/matrix sample (blood, urine, plasma...)
OTHER_GROUPING_COLUMNS: A multiple option should be given to the user so they can keep adding

Optional Variables

ADOSEDUR: Actual Dose Duration
NRRLT: Nominal Relative Time from Reference/Previous Dose

Definition of Done
The requirements for the feature to be complete:

[ ] The user can easily specify in "Data" section how its columns are defined
[ ] The NCA runs well
[ ] Hardcoding parts of the code have been substituted for dynamic coding

Suggestion
This is a very demanding issue so help should be considered at least to eliminate all hardcoding

Related to #3 and #125
The use of either the ANALYTE or PCTEST column can replace the need for #3.
I'd like to have a general column identification method for standard data formats (SDTM and ADaM) in PKNCA. That way, it can help others who are doing scripted analysis of NCA, too. While building this out, can it be considered as a general module that may eventually move into the main PKNCA package?

We could think about it, but not sure how adaptable will be! Our current idea is to allow the user introduce this by selection inputs, although we do want a default automatic filling of those inputs for simplification when ADNCA format is used as the file input