solenoid:options not showing in /status endpoint

I have a use-case where I need to pass some additional parameters from a selenium client to the container running the browser instance.
The approach I'm trying is to pass the parameters inside solenoid:options. The browser container (a fork of the solenoid Firefox container) will have an extra process that inspects http://<IP_ADDRESS>:4444/status to get this data.
However, the data doesn't show up here. This is despite the fact that it is present in the stdout logs produces when the browser is launched.
I realise this is an unusual use-case but I wonder is it working as intended?

@digiology works for me. Here is for example what I'm getting:
vania-pooh:~$ curl -s http://my-selenoid.example.com:4444/status
{"total":5,"used":1,"queued":0,"pending":0,"browsers":{"chrome":{"70.0":{"unknown":{"count":1,"sessions":[{"id":"638450b9155bac55439f88264308660c","container":"ca8f01a8199cf07bc948a523573d142e221c9027a62f621c97120e994588bc56","containerInfo":{"id":"ca8f01a8199cf07bc948a523573d142e221c9027a62f621c97120e994588bc56","ip":"<IP_ADDRESS>"},"vnc":true,"screen":"1920x1080x24","caps":{"browserName":"chrome","version":"70.0","platform":"ANY","screenResolution":"1920x1080x24","enableVNC":true,"videoScreenSize":"1920x1080","logName":"selenoid2708857f89d729037c2e92a5d2b9296f.log","selenoid:options":{"enableVNC":true}}}]}}}}}

Note the following key:
"selenoid:options":{"enableVNC":true}

Yes I've found that enableVNC gets passed but my custom keys do not.
I'll double-check and write up the steps to reproduce it.
On Wed, Dec 19, 2018 at 3:08 PM Ivan Krutov<EMAIL_ADDRESS>wrote:

@digiology https://github.com/digiology works for me. Here is for
example what I'm getting:
vania-pooh:~$ curl -s http://my-selenoid.example.com:4444/status
{"total":5,"used":1,"queued":0,"pending":0,"browsers":{"chrome":{"70.0":{"unknown":{"count":1,"sessions":[{"id":"638450b9155bac55439f88264308660c","container":"ca8f01a8199cf07bc948a523573d142e221c9027a62f621c97120e994588bc56","containerInfo":{"id":"ca8f01a8199cf07bc948a523573d142e221c9027a62f621c97120e994588bc56","ip":"<IP_ADDRESS>"},"vnc":true,"screen":"1920x1080x24","caps":{"browserName":"chrome","version":"70.0","platform":"ANY","screenResolution":"1920x1080x24","enableVNC":true,"videoScreenSize":"1920x1080","logName":"selenoid2708857f89d729037c2e92a5d2b9296f.log","selenoid:options":{"enableVNC":true}}}]}}}}}
Note the following key:
"selenoid:options":{"enableVNC":true}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/selenoid/issues/630#issuecomment-448627821,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADSyr2JV8BT03OOhY7KCmJtQ0aC9wBMwks5u6lZZgaJpZM4ZaY8F
.

@digiology only these capabilities should be passed: https://github.com/aerokube/selenoid/blob/master/session/session.go#L11

I see, thank you. I will try with env.
On Wed, Dec 19, 2018 at 4:37 PM Ivan Krutov<EMAIL_ADDRESS>wrote:

@digiology https://github.com/digiology only these capabilities should
be passed:
https://github.com/aerokube/selenoid/blob/master/session/session.go#L11
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/selenoid/issues/630#issuecomment-448660757,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADSyr-kmxfpxYcwz_0pk7SRiJRLavotYks5u6mtQgaJpZM4ZaY8F
.

Closing this because of inactivity. Feel free to reopen if still having questions.


Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/run Operating System: windows

Problem encountered on https://dotnet.microsoft.com/learn/dotnet/hello-world-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

Provide details about the problem you're experiencing. Include the exact error message, code sample, and anything else that is relevant.


[MERGEBACK][HOTIFX] Fix/pin scalecodec

3.5.1 hotfix merged back into to nobunaga
Adding upper bound for scalecodecs dependency

not applies anymore


Obtaining complete ciphertext in a printable form

Hello,
I am working on a client-server HE architecture that uses SEAL as the main client component and a hardware accelerator on the server side. I need SEAL to only take care of the key creation, batching, encryption and decryption using the BFV scheme. Computations like multiplications and additions of ciphertexts should be handled by the hardware side.
It is therefor necessary for me to obtain the full ciphertexts and relinearization keys in a form that is readable for my accelerator (like a vector of unsigned 64 bit integers). Following issue #537 and example 3 it seems there is no obvious way to obtain this data because of the way  SEAL handles modulus switching chains.
With a polynomial modulus degree of 8192 and coefficient modulus consisting of 6x 30-bit Primes I would expect to obtain 2 x 6 x 8192 values as one ciphertext in RNS format. But using cipher.dyn_array().size() on a ciphertext reveals 2 x 5 x 8192 values.
Is there any way in the current SEAL implementations to obtain all of these values?
Any help is greatly appreciated!

Hello again,
I tackled my problem by calculating a seventh prime number and using it as the "special prime" as described in example 3.
Is this the intended way to achieve a six prime RNS?
And is there an implementation reason, why modulus switching once is required when using the BFV scheme?
Thank you for any answers.

Working further with the code base has allowed me to understand that SEAL uses BEHZ RNS multiplications and the relinearisation keys are stored in NTT format. The question simply came from a lack of understanding the library.


test: cargo build validator in CI

Many hugr-building tests are showing up as s == skipped after #455. We need to cargo build the validator first.
Also "fix" (perhaps hack) run_int_fn to extract the first module Hugr from the Package coming back from compilation.

We should probably have an issue for this, nonetheless.

Feel free to create one 👍

I think I reasoned that anyone wanting run_int_fn to do anything else....would encounter the same problem, and have to deal with it then. run_int_fn already has no docs of what it expects/contracts to do...
However, the bigger issue that playing around here reveals - is that the type annotations here (i.e. in tests/) are not checked; we never run mypy. So the only way a type annotation results in an error is at execution (e.g. pytest) time, when an annotation using a type that doesn't exist, fails to load the type. Do we plan to enable them? I guess not, since a lot of test code is @guppy ?
(If so, we could provide a custom Protocol for the return value of the run_int_fn fixture that would have detected this problem, say.)


Fix the docs and Debug output of BufWriter.

The BufWriter docs inaccurately stated that it flushes on drop, which it does not do. This PR changes the docs, as well as the example, to highlight that the user must explicitly flush a bufwriter.
There were also two places where the BufWriter code referred to it as a BufReader: in the link to the std docs, and in the Debug output. Those have also been fixed.
Closes #586

Gladly :)


How do I copy some files from host to container?

docker cp Improved in release 1.8, now we can send files from host to container.
https://github.com/docker/docker/wiki/Engine-1.8.0
https://docs.docker.com/reference/api/docker_remote_api_v1.18/
This is a nice feature to docker-client  improvement too?

+1

+1

+1

This issue can be closed, right @mattnworb ?


Title changes and goes empty when a modal is opened

When a modal is opened on a click or a date picker is opened the title changes to an empty string

Please create a JSFidlle or a https://glitch.com/edit/ project to replicate this behaviour.


RTE Configuration file PLM - reconsider filenaming convention

While answering and discussing this issue https://github.com/Open-CMSIS-Pack/devtools/issues/313 the concerns got raised, that on UNIX operating systems files and directories starting with a "." are hidden by default but not on windows.
In order not to confuse users with multiple "similar" filenames, one suggestion was to store the "versioned" files in a different directory location. Which may make it harder for users to identify and localize the files required for a three-way merge.
a) project specific configuration file used by the build process
b) copy of the configuration file version used as the basis of the project specific file as provided by the component version at the time when the component was added or an updated version merged

unmodified
read-only
provides version information

c) copy of the configuration file as provided by the current component version / pack version when processed

unmodified
read-only
provides version information

Alternative using current and update
<configfile>.<ext>
<configfile>.<ext>.current@version
<configfile>.<ext>.update@version

Note:
a) we need to be able to identify which versioned file is the basis of the actual configuration file.
b) the tools shall remove any "update" configuration file that was not merged.
c) there is a maximum of one "update" file present or none in case the version of "update" would match "original".

Reflected in the next version of the documentation https://github.com/Open-CMSIS-Pack/devtools/blob/main/tools/projmgr/docs/Manual/Overview.md#plm-of-configuration-files (will be live tomorrow)

Is this completed?

This has not been implemented in csolution  (RTE-Model) yet.
During discussion @edriouk  and @brondani suggested to use base instead of current as this term is also used by git merge.
<configfile>.<ext>
<configfile>.<ext>.base@version
<configfile>.<ext>.update@version

implemented


Improve search in the references selector

Currently the reference selector only enables users to search for references using their title, searching by author name shoud and other attributes should also be added

Also how does the selector behave as the number of references scale?

The selector shows only up to 5 results. From a performance point of view, I think we can keep the filter function very basic for now, and rework it later if we realise that it does not scale when working with numerous references.


[saas-namespace-selector] SaaS File namespaceSelector

Introduce dynamic selection of namespace (resourceTemplates.targets.namespaceSelector) in saas files.
Design doc: https://gitlab.cee.redhat.com/service/app-interface/-/merge_requests/59689
Ticket: APPSRE-7124

would you mind adding some schema level validation for "either namespace or namespace selector"?

How can that be done? I was looking for an example, but I need help finding one.

you can start with oneOf?

you can start with oneOf?

TIL oneOf.required. Thx

@maorfr a final LGTM would be appreciated 🙇


⚠ Bofrost.nl has degraded performance

In 5031a78, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 3923 ms

Resolved: Bofrost.nl performance has improved in 4941aae after 6 minutes.


Update MD file on new branch 2017-27-03_03-12-15

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

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


Constant JVM crashes.

Type
Other
Operating System
Windows
OS Version
Windows 11
Bug description
I keep getting sporadic JVM crashes. There doesn't seem to be a pattern.
Things I've tried:

Wiped runelite and all settings from PC and re-installed. Now running with minimal plugins.
Ran Memtest and came back with no errors so not a RAM issue.
Tried investigating the individual processes behind the PID's but they're all random .dll and java files so unfortunately exceeds my knowledge in this area.

I've also noticed graphical bugs within TOA whereby some mobs go completely invisible and I'm unable to click on them. Tried enabling and disabling entity hider with no luck.
Any help is appreciated, thankyou.
Log files attached.
Screenshots or videos
No response
RuneLite version
RL Version: 1.9.2
Launcher Version: 2.5.0
Logs
jvm_crash_pid_19912.log
client.log
jvm_crash_pid_18472.log

Getting the same errors as well. Just switched from Windows 10 to Windows 11, so maybe there is a problem with that.

i've been getting constant Exception Access Violations or the last few months on an AMD processor too, 3900x here.
Are you running in GPU mode? If so what is your gpu and what driver version? Also are you on windows 10 and what version?


Fix bug when parsing incomplete Mapnik metrics

Related to https://github.com/CartoDB/support/issues/1374
When partial metrics were returned from Mapnik (due to not rendering anything for example) an error appeared as it tried to read from an undefined object

@dgaubert Can this be merged now?


war main exhibit painting

A painting depicting a certain victory in a battle. It’s monochrome
Rose-Colored Glasses claims it "clearly shows the victory of the Redoinks."
Blue-Colored Glasses claims it "clearly shows the victory of the Blouououous."
it's grayscale, so it's not clear what side is winning, or if the sides are even color-themed to begin with.

Paintings with variations!
Credit to: SillySussy
Original: https://drive.google.com/file/d/1pbTSqJ0BdkwZDsrPpm_-xD7wE6o83dyn/view?usp=drive_link

I did the variations myself, so if they look bad or over-saturated, let me know and i'll fix them

Blue: https://drive.google.com/file/d/1HF2pi4fB9nzw7-gowd95q-VFB655v_tO/view?usp=drive_link

Red: https://drive.google.com/file/d/1RlNpOrDBXBHirhCmAeevJ5kPFC53XGUw/view?usp=drive_link

Green: https://drive.google.com/file/d/17nfVSC6lXjcVnL52bsl59T_QDDVXLglU/view?usp=drive_link

Inverted: https://drive.google.com/file/d/1xf_IJY199JdYDlqpIssWY6UFdusAUC5h/view?usp=drive_link


ConfigurationDbContext is Disposed

ConfigurationDbContext is disposed before the token is generated. When using IdentityServerTools.
I've tried this both with manually adding the ConfigurationDbContext and with relying on the configuration from the Configuration store
        //I've also tried without this line
        services.AddDbContext<ConfigurationDbContext>(options
            => options.UseSqlServer(connectionString));

                var migrationsAssembly = typeof(Startup).GetTypeInfo().Assembly.GetName().Name;
        services.AddIdentityServer()
            .AddDeveloperSigningCredential()
            .AddAspNetIdentity<ApplicationUser>()
            .AddOperationalStore(options =>
            {
                options.ConfigureDbContext = builder =>
                    builder.UseSqlServer(connectionString,
                    s => s.MigrationsAssembly(migrationsAssembly));

                // this enables automatic token cleanup. this is optional.
                options.EnableTokenCleanup = true;
                options.TokenCleanupInterval = 30;
            })
            .AddConfigurationStore(options =>
            {
                options.ConfigureDbContext = builder =>
                    builder.UseSqlServer(connectionString,
                    s => s.MigrationsAssembly(migrationsAssembly));

            });

I'm trying to generate a token to consume identityserver as a client from my api.  Sometimes the ConfigurationDbContext is already disposed, sometimes it throws on the first line sometimes it throws on the last
    private async Task<string> CreatePaymentsTokenAsync()
    {
        // Get client for JWT
        var idpClient = await this._configurationDbContext.Clients.Include(x => x.AllowedScopes)
            .FirstOrDefaultAsync(c => c.ClientId == Config.APIServerClientId);
        // Get scopes to set in JWT
        var scopes = idpClient.AllowedScopes.Select(s => s.Scope).ToArray();
        // Use in-built Identity Server tools to issue JWT
        var token = await _identityServerTools.IssueClientJwtAsync(idpClient.ClientId,
            idpClient.AccessTokenLifetime, scopes, new[] { "MyApi" });
        return token;
    }

I think the error was slightly different when it occurred on the last line,
something about accessing the provider.
Do you know of a way I can access the clients without manually injecting
the database, I feel like there’s something built into identity server but
I could be wrong
On Thu, Dec 6, 2018 at 3:26 AM Dominick Baier<EMAIL_ADDRESS>wrote:

hm - I can't see how that would be related.
As you can see:
https://github.com/IdentityServer/IdentityServer4/blob/master/src/IdentityServerTools.cs#L47-L63
Is not doing any DB related things.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IdentityServer/IdentityServer4/issues/2871#issuecomment-444788673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMsMnaoR2ThJKF16gyDkWwxFYK9FadpLks5u2NSugaJpZM4ZFuJn
.

As a side note - you don't need to use a "real client" - this helper will simply create a token based on the values you pass in.

Thanks, that resolves the issue


pull request for e2e test at 01/24/2017 10:31:24

:white_check_mark: Validation status: passed
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.


New Routes

Added new Routes

Get All Expertise
Get All Learnings

Lel

Lel

Fixing your Code in my PR ? :P


feat: fetch status initially, update preview, emit events

Fix #15
Fix #18
Fix #23

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


State transfer always fails to verify existing blockchain

It has been observed in PBFT testing that a peer with an existing blockchain will always fail to validate its chain, causing a loop where it attempts to retrieve corrupt blocks which are apparently not corrupt.
This is not apparent in mock testing, so may be a bug in the underlying ledger verification utilities or a difference in the mock vs. real API.
@kchristidis Could you tag this as a bug, for state transfer, and assign to me?

all yours @jyellick
ah, the power of being a committer :)


About importing pre-trained weights for the LLaMA model

In the code for action recognition, I'm a bit confused about the part where pre-trained weights for the LLaMA model are imported. Can you please point out the section of code that handles the import of pre-trained weights for the LLaMA model?
Can you provide the pre-trained weights for the LLaMA model you are using?

@Changwei-Ouyang Thanks for the question!

This line loads the weights of LLaMA. I wrote a function in llama.py that supports loading one layer of transformer at here.
Sorry about it that I am not sure if I am allowed to distribute the weights of LLaMA publicly. But I just made some updates to the doc of action recognition explaining the preparation of LLaMA. Hope this helps!

Thank you for your response, very interesting work! Now I understand.


add func find proc by ordinal

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!

@googlebot I signed it!

@googlebot I fixed it.


sha2_hmac was removed

sha2_hmac was removed - can't find any indications as to why

commit d2681d82e226d28d033390b61e94af04f0248dff states "SHA2 renamed to SHA256, SHA4 renamed to SHA512 and functions accordingly"

commit 7da726bb53d83f1b2b66750fd215b7781f11e3be indicates sha2_hmac usage was replaced with md_hmac


Fix long lines in all but one case

The following line in lexical_structure.md (and then grammar.md) is 81 characters long. It looks okay in the Word doc and would be hard to fix without a line break that reduces readability:
    | '.' Decimal_Digit Decorated_Decimal_Digit* Exponent_Part? Real_Type_Suffix?

I'm tempted to leave that and possibly hard-code warning suppression for that line.
(We could up the line limit from 80 to 81, but I'd prefer not to do that.)

Thanks @RexJaeschke - in this particular case I'd like @BillWagner to have a look too before I merge.


UI - On running scans on a Non RKE cluster, Unknown Action error is seen on the UI

What kind of request is this (question/bug/enhancement/feature request): bug
Steps to reproduce (least amount of steps as possible):

For a GKE cluster, the runSecurityScans action is disabled in the API>
But when tried from the UI,  Unknown Action error is seen

Expected Result:
The action is disabled from the API, which is expected.
The Run Scan option can be disabled from the UI. As the user will NOT be allowed to run scans on a non rke cluster
Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): master-head - commit id: df31802ac
Installation option (single install/HA): single

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported):GKE cluster
Kubernetes version (use kubectl version):

v1.14.8-gke.17

Verified on master-head - commit id: 5f917419b

For a GKE cluster, Navigate to CLuster --> Tools --> CIS Scans
CIS Scans are only available to RKE Clusters is seen on the UI


[HOLD E/E#294647] Update PersonalDetails migration of reports to remove email-based data

Details
Now that the back-end is no longer sending email-based data to reports, we update the migration to remove any email-based data that is found. We aren't removing entire reports from onyx, just the email-based data that is found.
Fixed Issues
$ https://github.com/Expensify/App/issues/21454
Tests
The automated tests should mostly cover this since it's an Onyx migration. However you can also test the various cases manually by:

In the JS console do Onyx.set('report_123', {ownerEmail: 'hi', ownerAccountID: 1000});
Check report_123 in application storage and make sure you see what you set
Refresh the page
Make sure that report_123 is now just {ownerAccountID: 1000}
In the JS console do Onyx.set('report_123', {ownerEmail: 'hi'});
Check report_123 in application storage and make sure you see what you set
Refresh the page
Make sure that report_123 is now an empty object
Do a similar test with other keys that are being updated

[ ] Verify that no errors appear in the JS console

Offline tests
Same tests work online or offline.
QA Steps
There is nothing in particular to QA for this PR as there are no user-facing changes. Regular regression testing that the app is working as normal should be enough.

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

Will this be off hold soon? 😄 Also just a heads up there's conflicts!

Soonish hopefully, still working through migrating these emails 😅 thanks for the heads up about conflicts!

I guess we can close this since some pieces have already been merged, and managerEmail / ownerEmail pieces are being handled in https://github.com/Expensify/App/pull/31664


youtube view controll can't load in android

Was working yesterday night, and this morning same problem.  Only the preview image load.
I reproduced on a virgin project.

It seems to be only on emulator, not on the phone

Use a height hack on #131 solved my problem with view controls

solved in c9a0c38c1799f7cfb7d3ad9435ad834c52ba56d8


Incompatible version error

I'm facing following error while deploying application on heroku, it seems that shuffle-array version is not compatible with Node. Can you please guide what is the solution for it?
remote:        npm ERR! Linux 3.13.0-79-generic
remote:        npm ERR! argv
remote:        npm ERR! node v5.9.1
remote:        npm ERR! npm  v3.7.3
remote:
remote:        npm ERR! No compatible version found<EMAIL_ADDRESS>remote:        npm ERR! Valid install targets:
remote:        npm ERR! 2.0.0, 0.0.1-security
remote:        npm ERR!
remote:        npm ERR!
remote:        npm ERR! If you need help, you may report this error at:
remote:        npm ERR!     https://github.com/npm/npm/issues
remote:
remote:        npm ERR! Please include the following file with any support request:
remote:        npm ERR!     /tmp/build_697d8b8fec943eeebab6cbbf6178b3dc/npm-debug.log
remote:
remote: -----> Build failed
remote:
remote:        We're sorry this build is failing! You can troubleshoot common issues here:
remote:        https://devcenter.heroku.com/articles/troubleshooting-node-deploys
remote:
remote:        Some possible problems:
remote:
remote:        - Node version not specified in package.json
remote:        https://devcenter.heroku.com/articles/nodejs-support#specifying-a-node-js-version
remote:
remote:        Love,
remote:        Heroku
remote:
remote:
remote:  !     Push rejected, failed to compile Node.js app
remote:
remote: Verifying deploy...
remote:
remote: !       Push rejected to .
remote:

https://github.com/pazguille/shuffle-array/issues/5#issuecomment-207865446


Fix a couple of regressions

@nitin-dhevar Some more tests were failing and I noticed a couple of other regressions. We're not going to be able to remove the intermediate state from the PullSecretCredentialsForm until some other refactoring happens :/

@nitin-dhevar Your PR is blocked by this. I would like to keep your original PR intact so that you retain authorship. I'll give this another few days, but I will need to unblock the story by other means if this doesn't merge soon.

@TheRealJon I apologize for the delay, I somehow missed this. It’s merged now. Please let me know if there’s anything more I can do to help move things along, and I’ll address it right away.


Store Peer ID

Minor change: Instead of always calling publicKey.SerializeCompressed(), store peerID permanently in memory.

No longer needed


Fixed windows installation

I didn't test the windows installation. Which was a mistake (sry).
Now i fixed it...

thank you for merging! (you are fast 👍 )


os/mac/xcode: require Xcode 14.1 on Ventura

As is pretty much standard now, 14.0 RC/final removed the Ventura SDK, but the 14.1 betas do have it. As such, we need to now require Xcode 14.1 on Ventura.
(Side note: I've not updated latest_stable on Monterey from 13.4 yet. This is intentional due to the regular reports of GCC breakages, which is fixed in 14.1, as well as our own issues such as #13863, which all means we really shouldn't promote it as the recommended version yet - but hopefully we can for Xcode 14.1 by the time it releases in a couple weeks)

Review period will end on 2022-10-10 at 18:31:34 UTC.

Review period skipped due to critical label.

Unfortunately, this was a bad assumption. Ventura shipped with 14.0.1.

Xcode 14.0 does not ship with the macOS 13 SDK so there's nothing we can do about this unexpected last minute delay.
Xcode 14.1 RC 2 is available to download for free from the Apple developer website if you need something. Hopefully it'll be on the App Store within the next day or two.

Apple basically shipped a final macOS release without any corresponding (non-beta) Xcode release? I was already worrying that I'm doing something wrong because of the brew doctor warnings, but seems like it's not on me then. But also not on Homebrew. Just Apple being unfathomably sloppy with their releases.

This the only other time this has ever happened, at least in terms of major macOS releases, was when Xcode 6.1 had its App Store release delayed by 4 days in 2014 (but was still available as a final release on developer.apple.com). So it has surprised us as well.


Errors on Windows 11 / MSVC 2022 / Vulkan SDK <IP_ADDRESS> / NV 3060 latest drivers

I get about 250 errors in vk_video_samples:
VkVideoCore/VkVideoCoreProfile.h(171,47): error C2065: 'VK_STRUCTURE
_TYPE_VIDEO_DECODE_H265_PROFILE_INFO_KHR': undeclared identifier (compiling source file vk_video_samples\vk_video_encoder\NvEncodeApp.cpp) [vk_video_samples\build\vk_video_encoder.vcxproj]
error C2065: 'm_h265DecodeProfile': undeclared identifier (compiling source file vk_video_samples\vk_video_encoder\VkCodecUtils\N
vPictureBuffer.cpp)
this is after a build_all clone, and clone_all.bat.
I need to get an encoder example working is all.
I'm unclear why vulkan headers exist in this example when you request VulkanSDK be installed?  Anyway its not looking at the headers installed on the system -- its looking at nvidia's headers local to project: "vulkan/vulkan.h"

Are there cmake build args I need to pass for vk_video_encoder?

I actually think this is because I didnt cd ../nvpro_core/nvvk and run the generator script there.  I just went to linux and did so and it built.


error when trying create db  make_db.py in database2021 subdirectory

hello,
I interesting in your project and want to recreate the meta database
so I want to recreate the database by downloading json from GCP:
https://console.cloud.google.com/storage/browser/danbooru_public/data?project=danbooru1
but I get the following error when I try to replace all /mnt/D2/metadata/{filename.json}
here is the error:
json_line["file_ext"],json_line["file_size"],json_line["md5"],int(json_line["has_children"]),
KeyError: 'md5'

from:
import_tags("/mnt/D2/metadata/tags000000000000.json", conn)
import_artists("/mnt/D2/metadata/artists000000000000.json", conn)
import_notes("/mnt/D2/metadata/notes000000000000.json", conn)
import_pools("/mnt/D2/metadata/pools000000000000.json", conn)

replace:
import_tags("~/danbooru_public/tags.json", conn)
import_notes("~/danbooru_public/notes.json", conn)
import_pools("~/danbooru_public/pools.json", conn)

since I don have your previous data I can't check the error.
hope you can help me out thanks.

Thank you for your interest!
The last time I downloaded data from BigQuery, the posts json no longer included the md5 value. So any reference to that field needs to be removed. Rather than change the database schema, I used a constant value.
So, for example, in this section in make_db.py, the following lines:
    c.execute("INSERT OR IGNORE INTO images VALUES (?,?,?,?, ?,?,?,?, ?,?,?,?, ?,?,?,?, ?,?,?)",
              (pId,json_line["rating"],json_line["source"],pixiv_id,
              json_line["image_width"],json_line["image_height"],json_line["created_at"],json_line["updated_at"],
              json_line["uploader_id"],int(json_line["is_banned"]),int(json_line["is_deleted"]),int(json_line["is_flagged"]),
              json_line["file_ext"],json_line["file_size"],json_line["md5"],int(json_line["has_children"]),
              int(json_line["has_visible_children"]),int(json_line["has_active_children"]),parent_id
              ))

are changed to:
    c.execute("INSERT OR IGNORE INTO images VALUES (?,?,?,?, ?,?,?,?, ?,?,?,?, ?,?,?,?, ?,?,?)",
              (pId,json_line["rating"],json_line["source"],pixiv_id,
              json_line["image_width"],json_line["image_height"],json_line["created_at"],json_line["updated_at"],
              json_line["uploader_id"],int(json_line["is_banned"]),int(json_line["is_deleted"]),int(json_line["is_flagged"]),
              json_line["file_ext"],json_line["file_size"],"0",int(json_line["has_children"]),
              int(json_line["has_visible_children"]),int(json_line["has_active_children"]),parent_id
              ))

Note how on the 5th line, json_line["md5"] is replaced by "0", i.e. all rows in the database will have their md5 column value set to zero.
That is the only place where a reference to md5 needs to be changed. I can't find any similar issues for other tables (e.g. tags, artists, etc) but the same process would apply if necessary.

 That is unfortunate ...

thanks your fixed really help me out.
can you tell me how you check md5 not working anymore?
the GCP post.json file and danbooru API which I found the string of md5 so I don know when they suddenly stop using it.
if you using bigquery to collect the danbooru API i interested in learning it.

OK, I'm starting to remember the details now.  I did not download the post.json file from GCP as you linked; I manually downloaded a partial query, and inadvertently dropped the md5 column. As a result, I got the KeyError: 'md5' message, and assumed a similar scenario was the cause for your situation.
So I've not seen any of the metadata from GCP. You could help me get on the "same page" if you could upload a single json record from posts.json, as I did from the 2021 metadata.
A python KeyError means the specified field doesn't exist. In my case, there was no md5 value. Note the field is case sensitive: if the md5 field exists in the GCP data but has a different name, then the KeyError would still occur.

It occurs to me that a record in the json might be incomplete, which would cause a KeyError to occur on that record. To skip records which are incomplete, add exception handling as so:
    try:
       # existing code, idented
       c.execute("INSERT OR IGNORE INTO images VALUES (?,?,?,?, ?,?,?,?, ?,?,?,?, ?,?,?,?, ?,?,?)",
           ...
       buildImageTags( ... ) # existing 5 calls to buildImageTags
    except:
        pass    # consider printing some info for debugging

If the KeyError is general, you'll need to fix it, otherwise no records will be loaded.

are /Danbooru2021/database/makedb.py still useable? even thought I only have posts.json and tags.json in metadata folder
I keep getting keyerror

Just in case of a communication disconnect ... The version of makedb.py in github is for the 2021 metadata, and I'm not making any changes to that version. So continuing to use that version will continue to cause a KeyError.
To address your issue with the GCP latest metadata, you need to modify and execute your local copy of makedb.py as necessary.


window 执行”flutter pub get”报错

报错如下：
Error reading LocalFile: 'D:\reposite\github\gsy_github_app_flutter_gitee\android\app\src\main\AndroidManifest.xml' even though it exists. Please ensure that you have read permission to this file and try again.

不是很明显么，说读不到


Change API for property-based script testing

Motivation
Currently tasty-plutus API for property-based testing have not enough expressiveness.
It can take some time to make special tools to cover most common script testing use cases, but even before that happen, API still must be useful, and allow users to express tools they need by themselves using it.
To achieve that makes sense to provide more general API than existing one, but having more expressive power (in cost of specialization).
Changes
Currently basis of property-based tests API is  a couple of datatypes and scriptProperty function
scriptProperty :: String -> Generator p -> (TestData p -> ContextBuilder p) -> WithScript p ()

Which 'ForSpending variant can (with some simplification) actually be represented as:
scriptProperty ::
  ( ToData datum
  , ToData redeemer
  , FromData datum
  , FromData redeemer
  , Show datum
  , Show redeemer) =>
  String ->
  (datum -> redeemer -> Value -> Example) ->
  Methodology datum ->
  Methodology redeemer ->
  Methodology Value ->
  (datum -> redeemer -> Value -> ContextBuilder  'ForSpending)
  -> WithScript 'ForSpending ()

Here three separated generators disallows to:

generate coherent, bounded datum, redeemer, Value
Also, concrete function signatures and lack of free generator disallows to:
use some generated data to control exactly ContextBuilder without touching datum, redeemer and Value
use some generated data to control exactly case result prediction, without touching datum, redeemer and Value

Seems that such possibilities can be achieved by using one generator and more general functions for building test case from generated data.
So, the proposal is to change it into something like:
withProperty ::
  ( ToData datum
  , ToData redeemer
  , FromData datum
  , FromData redeemer
  , Show datum
  , Show redeemer
  ) =>
  String -> -- property name
  -- generated data can be any complex, so it's possible to keep there some metadata
  -- to use in case result prediction and context building
  Methodology a -> -- data generator
  -- all stuff including prediction about case result is just a function from generated data
  (a -> (datum, redeemer, Value, ContextBuilder  'ForSpending, Maybe Example)) ->
  WithScript  'ForSpending ()

Details
That was the way of representing things as one single function.
Now in terms of actual API. The only things this proposal offers to change is Generator and scriptProperty.
Instead of generating validator inputs separately we just require one generator and a way to extract them out of generated data.
That allows us generate coherent arbitrarty validator inputs, and provide any additional data for usage in case building and result prediction.
data Generator (a :: Type) (p :: Purpose) where
  GenForSpending ::
    -- Gen can be any 'a'
    Methodology a ->
    -- But we must know how to make TestData and ContextBuilder from it
    (a -> (TestData 'ForSpending, ContextBuilder 'ForSpending)) ->
    Generator a 'ForSpending
  GenForMinting ::
    Methodology a ->
    (a -> (TestData 'ForMinting, ContextBuilder 'ForMinting)) ->
    Generator a 'ForMinting

The property itself (a -> Maybe Example) takes a as an argument, not TestData p or any more concrete type.
So it is more general and allows to build more restricted test builders based on this one.
That's important, so even if user can't find some concrete test builders satisfy his needs, will have possibility to make it by himself.
scriptProperty ::
  forall (a :: Type) (p :: Purpose).
  String -> -- property name
  Generator a p -> -- data generator
  (a -> Maybe Example) -> -- must it fail or succeed, or thrown away
  WithScript p ()

Usage example
-- positive test
scriptProperty
  "From any pkh, CS, inital data and value its always possible to make valid context and transaction"
  (GenForSpending
    (Methodology (QC.arbitrary,QC.arbitrary,QC.arbitrary,QC.arbitrary) genericShrink)
    (\pkh cs initalData val ->
        ( SpendingTest
            MyDatum {owner = pkh, tokenCS = cs}
            (MyRedeemer $ mkMyRedeemer pkh initalData)
            (mkValidValue val cs)
        , buildValidContext pkh cs ))
  )
(const $ Just Good)

Maybe add a function that's just "any value produced by this generator should validate"? Because, if I'm not mistaken, the scriptProperty test will fail if there aren't roughly an equal number of failing and passing cases.

5.0
Added

passIf combinator for creating Outcome from a condition.
TestItems (p :: Purpose) type represens the data set for script checking
scriptPropertyFail to test the conditions under which a script should always succeed
scriptPropertyPass to test the conditions under which a script should always fail
Property based testing example (test/Properties/Main.hs)

Changed

Rename Example to Outcome and its two constructors Good and Bad to
Pass and Fail respectively, unifying with the internal type used in unit
tests.
Changed API for property based testing:

Changed type and implementation for scriptProperty

Generator (p :: Purpose) replaced with Generator (a :: Type) (p :: Purpose),
changed types for data constructors GenForSpending and GenForMinting
Property based testing coverage check changed from 0.5% to 45%

Love it!


Implement numpy autowrapping in remote context

What do these changes do?
This replaces numpy module whenever a user code executes import numpy with a special class that returns different things in local and remote contexts.
It also registers special reducers (support for pickling), as, when trying to pickle an object from "local" numpy when "remote" is activated leads to TypeError: different class without special reducer.
Note that access to numpy is always local for modin.* and pandas.* packages, as NumPy is used internally there and is not needed to be proxied to the remote.

 Let's wait CI result.


[de] sync content/de/docs/concepts/_index.md from upstream

/approve

please review @sftim

That's an informal LGTM. I'm not a reviewer for the German localization.

Those contents are not "removed". It's rewritten to each page, please refer to PR #21997.

#21997

/hold

Hey @JarHMJ,
It's been a while since last update, would you like to sync the rewritten article so that we can move forward?

Hey @JarHMJ,
It's been a while since last update, would you like to sync the rewritten article so that we can move forward?

Sorry. I can't translate German, maybe someone else can.


rename class

Q
A

Bug fix?
yes

New feature?
no

Deprecations?
no

Issues
Fix #50

License
MIT

classes marked as deprecated


Fancy layout styling

Winter CMS Build
dev-develop
PHP Version
8.0
Database engine
MySQL/MariaDB
Plugins installed
Winter.Test
Issue description
PR #763 introduce degenerative styles when using nested forms under repeater fields.
winter/wn-backend-module 1.2.1

winter/wn-backend-module dev-develop 8972c89

Lost padding:
Tab panes have lost their inner padding.

Extras chevron:

The extra right chevron seems to come from:
https://github.com/wintercms/winter/blob/develop/modules/system/assets/ui/storm.css#L1226
.control-tabs.scroll-after::after {
  display: block;
}

Note:
On my screenshots, the repeated section fields have a dedicated small-panels class added which allows me to remove the unnecessary padding.
.small-panels .field-repeater li.field-repeater-item {
  padding: 0 !important;
}

Steps to replicate

Install Winter
Install this fork/commit of Winter.Test plugin to have the complex field for page model https://github.com/damsfx/wn-test-plugin/commit/d4b773f064f87363f5364194fa65f25d8565b873
Go to backend to edit the complex page

Workaround
No response

@damsfx are you able to add a test case for this in the Test plugin under the Record model's fields.yaml? We're using it as a kitchen-sink testing area for every widget & field and complex cases like these that could cause issues.


Update .eslintrc.js to allow chai chainable getters

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

While adding some chai expect assertions using their chainable getters (like expect(foo).to.be.true) I ran into @typescrpt-eslint/no-unused-expressions warnings. This small change to the .eslintrc.js config allows those assertions in test files. I assume this is something everyone will want to use with expect, since that's a good deal of this assertion syntax's value, in my opinion.

use .to.equal(true) instead

use .to.equal(true) instead

You can use .to.equal(true) instead.

Sorry to belabor this a little more, but I'm just curious about the thinking on foregoing parts of the assertion API. Is the config change too risky for some reason, or do these just not seem useful in the context of how MUI tends to write tests?
Is the plan to never make use of the fairly robust (and arguably more readable) BDD chainables then?
I suppose I should have asked before I bothered with the MR (sorry about that), but I just assumed we'd want the full power of the assertion. Stuff like:

expect(42, 'nooo why fail??').to.be.NaN;
expect(undefined).to.be.undefined;
expect(foo).to.not.be.undefined;
expect([]).to.be.empty;
expect({}).to.be.empty;
expect({ thing: 1 }).to.not.be.empty;
expect(frozenObject).to.be.frozen;

Thanks for any additional color here.

You can use .to.equal(true) instead.

Sorry to belabor this a little more, but I'm just curious about the thinking on foregoing parts of the assertion API. Is the config change too risky for some reason, or do these just not seem useful in the context of how MUI tends to write tests?
Is the plan to never make use of the fairly robust (and arguably more readable) BDD chainables then?
I suppose I should have asked before I bothered with the MR (sorry about that), but I just assumed we'd want the full power of the assertion. Stuff like:

expect(42, 'nooo why fail??').to.be.NaN;
expect(undefined).to.be.undefined;
expect(foo).to.not.be.undefined;
expect([]).to.be.empty;
expect({}).to.be.empty;
expect({ thing: 1 }).to.not.be.empty;
expect(frozenObject).to.be.frozen;

Thanks for any additional color here.

Is the plan to never make use of the fairly robust (and arguably more readable) BDD chainables then?

We have been ruling out this option in the past. We encourage the usage of .to.equal( anytime the improvement of the error message when it failed with an alternative is negligeable. We shouldn't have to disable these eslint rule, nor increase the API surface to write tests. It's already hard enough to write new tests when you are not familiar with all its nuances.

Is the plan to never make use of the fairly robust (and arguably more readable) BDD chainables then?

We have been ruling out this option in the past. We encourage the usage of .to.equal( anytime the improvement of the error message when it failed with an alternative is negligeable. We shouldn't have to disable these eslint rule, nor increase the API surface to write tests. It's already hard enough to write new tests when you are not familiar with all its nuances.

Cool. Thanks for taking the time to explain the thought behind this a little more.

Cool. Thanks for taking the time to explain the thought behind this a little more.


Refactor Encyption for Accounts

We need to hash users passwords in the frontend before sending them to the server.

We are gonna use HTTPS for the website and don't need this. Here's why


[bug] WalletConnect + MetaMask does not reconnect on page refresh

Is there an existing issue for this?

[X] I have searched the existing issues

Package Version
0.2.9
Current Behavior
When using the autoConnect prop on the Provider, if a user connects via Wallet Connect using MetaMask (I did not test other Wallet Connect clients) and refreshes the page, the user appears connected from the perspective of wagmi (useAccount) returns an address, but actions involving the user's wallet will fail. Specifically, prompting a signature with useSignMessage seems to send a message across the Wallet Connect bridge and displays within the user's wallet, but never resolves in wagmi. Prompting a transaction with useTransaction calcuates gas, but does not display the transaction within the user's wallet.
Expected Behavior
After connecting via Wallet Connect and refreshing the page, the user remains connected and is able to complete actions with their wallet.
Steps To Reproduce

Connect using Wallet Connect
Refresh the page, have autoConnect set on wagmi provider
Attempt to use useTransaction or useSignMessage

Link to Minimal Reproducible Example (CodeSandbox, StackBlitz, etc.)
No response
Anything else?
No response

Was able to confirm this issue using the demo in the Sign Message guide. This doesn't happen with the MM or WalletLink connectors.
Seems related to this https://github.com/WalletConnect/walletconnect-monorepo/issues/718

Agree that the issue on walletconnect looks to be the cause, thanks for flagging that. Not sure if you want to close this in favor of the upstream issue or leave this open to track.


data problems

we need to add moods to every song with some python codes and we need so much more songs data.

completed


Add shorthand for watch plugins and runners

Summary
This PR fixes #7156
Test plan
Tests were added.

Flow errors :(

Is this ready to land @rogeliog? I was holding off on merging due to https://github.com/facebook/jest/pull/7213#discussion_r226869812

@SimenB it was addressed in https://github.com/facebook/jest/pull/7213/commits/3d7207503d11ab98efc78b45b1a649f5282bab8d


Change the border colour of the container

I have tried looking through the information available on this GitHub repo but can't seem to find this, if it already exists please point me in the right direction.
Put simply I need to change the colour of the container in the unused, error and being used state, let me know if this is a possibility or where I could look into it myself

Hi @HowardRHard 👋 You can do this by setting the relevant *BorderColor parameters when calling TextFieldDefaults.outlinedTextFieldColors(...) and passing that to the picker's colors parameter


Response details in ValidatableResponse thrown AssertionException

When asserting a standard API call's response in a test
given().get("https://foo.com/bar").then().statusCode(200);
This throws an exception whenever the status code received doesn't match what's expected.
However many testing frameworks delegate their exceptions thrown in the tests (e.g. Junit5 TestExecutionExceptionHandler)
There, the functions only get access to an AssertionError with message
Expected status code <200> but was <500>.
And nothing else.
If it would be possible to have the response object logged in the Assertion then it'll make it much easier to identify the Exception thrown for same type of statuses and to log it in test reporting tools better
It would be helpful to have a similar setup like Filters to delegate the assertions thrown.
So that any exception thrown can also have access to the Response body and can format the error as needed
given()
    .then()
        .whenError(new ResponseValidatorHandler())
    .request
    .get("https://foo.com/bar")
    .then()
    .statusCode(200);

For now, I've chosen to utilize delegation to achieve the same effect by wrapping actual ValidatableResponse thrown exceptions
SO question


doc_id always overwritten by default_id_func

Bug Description
Hi, according to the docs here it should be possible to assign a custom doc_id to documents. But when creating a list of documents with custom ids, those always get overwritten during the node creation phase.
Version
0.9.34
Steps to Reproduce
from llama_index import Document, ServiceContext, StorageContext, VectorStoreIndex
from llama_index.embeddings import HuggingFaceEmbedding
from llama_index.retrievers import VectorIndexRetriever
from llama_index.vector_stores import ChromaVectorStore

import chromadb

db = chromadb.PersistentClient(path="./db")
chroma_collection = db.get_or_create_collection("mediabiasfactcheck.com")
embed_model = HuggingFaceEmbedding(model_name="BAAI/bge-base-en-v1.5")
service_context = ServiceContext.from_defaults(embed_model=embed_model)
vector_store = ChromaVectorStore(chroma_collection=chroma_collection)
storage_context = StorageContext.from_defaults(vector_store=vector_store)

data = ["bla", "dibla"]
documents = []
for i, text in enumerate(data):
    document = Document(doc_id=i, text=text)
    documents.append(document)
index = VectorStoreIndex.from_documents(
    documents, storage_context=storage_context, service_context=service_context
)
retriever = VectorIndexRetriever(index=index)
response = retriever.retrieve("bla")
print(response)

Relevant Logs/Tracbacks
No response

@Morriz the original doc id should be available on node.ref_doc_id right? Each document is chunked into nodes, and those nodes contain references to the original parent doc

@logan-markewich no, those are all overwritten with the uuid.
But you were right, we should not rely on the custom id anyway as it all gets chunkified.
I just added the external id to the metadata and got it from there...solved!


cmd/compile: inline of f(m T) from package containing type m causes stack overflow

What version of Go are you using (go version)?
Development tip as of bug report (2018-06-20)
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/drchase/Library/Caches/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/drchase/GoogleDrive/work/gocode"
GOPROXY=""
GORACE=""
GOROOT="/Users/drchase/GoogleDrive/work/go"
GOTMPDIR=""
GOTOOLDIR="/Users/drchase/GoogleDrive/work/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/gr/vvb66dqx6jl6lh8wckfd5p9w0095tn/T/go-build491052553=/tmp/go-build -gno-record-gcc-switches -fno-common"
VGOMODROOT=""

What did you do?
mkdir testfoo/src/pkg
cd testfoo
cat >> main.go <<EOF
package main

import (
	"pkg"
	"runtime"
)

//go:noinline
func countThreadCreate() int {
	n, _ := pkg.ThreadCreateProfile(nil)
	return n
}

//go:noinline
func main() {
	println("ctc=", countThreadCreate())
	n, _ := runtime.ThreadCreateProfile(nil)
	println("tcp=", n)
}
EOF
cat >> src/pkg/pkg.go <<EOF
package pkg

type m struct {
	alllink     *m // Comment out this line for a different error.
}

var allm *m

type StackRecord struct {
	Stack0 [32]uintptr
}

func ThreadCreateProfile(m []StackRecord) (n int) {
	if mp := allm; mp != nil {
		n++
	}
	return
}
EOF
GOPATH=`pwd` go build main.go

What did you expect to see?
Successful compilation.
What did you see instead?
# command-line-arguments
runtime: goroutine stack exceeds<PHONE_NUMBER>-byte limit
fatal error: stack overflow
(etc)

1.10 seems to be okay.  This is not caused by the recent CL for importing FOR loops, unless I am utterly failing at "git checkout".  I'm still looking for the commit where it goes wrong.
Bad commit is between these two:
Previous HEAD position was 58c231f244 runtime: FreeBSD fast clock_gettime HPET timecounter support
HEAD is now at d2c7dec183 net: implement (*syscall.RawConn).Read/Write on Windows

The good commit is dated "Sat Apr 14 05:36:36 2018 +0000"

I'm bisecting, back from evening events.

I pruned some unnecessary code from the test case, and managed to replicate a different error that is the one I was originally looking for.  If you remove
	alllink     *m // Comment out this line for a different error.

the error is instead (at least at tip)
# command-line-arguments
./main.go:10:30: pkg.m redeclared during import "pkg"
	previous declaration at ./main.go:10:30

Ugh, this is yet another scoping issue due to resolving symbols within import bodies.
The problem is export.go:importsym uses s.Def. But when called to import a package-scoped symbol that's shadowed within an inline body (e.g., the recursive m type declaration shadowed by the m parameter to ThreadCreateProfile), s.Def will point to the local parameter instead of the package-scoped definition.
It needs to use s.PkgDef() instead, and maybe a corresponding s.SetPkgDef method.


Pull functionality missing for standard abapgit

Yesterday I started working on it, I have my own proprietary abapgit interface I created at work, which does pull but mix it with lots of unrelated proprietary stuff. So rather than open sourcing that I started working on making this work with this.
The code to do it is commented, so I started an uncomment and fix exercise, but stopped almost immediately: method zcl_abapgit_res_repo_pull->validate_request_data has several issues:

tries to validate package and url sent by the request, but the eclipse plugin sends none of that (I even debugged it to make it sure, there simply isn't any code to do that)
validates that a valid transport is sent, even when the package is local
validates that the package and url don't belong to an existing project, which they must for a pull to make sense

Looks like it was designed to create an abapgit project and then partially changed.
I have no problems fixing that (except I'll be on holidays for the next 10 days) but the change will be quite invasive.
Perhaps was a copy of zcl_abapgit_res_repos->import_repository, which does currently work.
I understand this is more or less a fork of what is actually running on the cloud platform, and would like it to diverge as little as possible. Above all, I don't want to make and maintain my own fork.
Would be great if you guys could sync this with the cloud version as much as possible, or give me some guidance on how things are supposed to work.

Please consider this repository as a highly experimental.
The basis of this project is indeed from the version in cloud platform, but that shouldn't be a reason to keep everything as it is. Also, please feel free to make invasive changes. The idea behind this repository is to make it work with the zabapgit report.
You just need to take care of the request versions in the handlers otherwise this could cause problems with the ADT Plugin. So it's my recommendation to edit everything in the HTTP methods (post, get ...) inside the _res_repo handlers.
Long story short, just go ahead and feel free to break it :)


[FEATURE] Load markdown files from online sources

We will need the posibility to load Markdown files from online sources.
The files should be cached so that if we lose the internet connection we can still show something.
Edit: removed spelling mistakes

This is required for #82

This is required for #82


Is it necessary to manage our own observedGeneration field to trigger the reconcile loop?

When reading the object's metadata, I can not find the observedGeneration field in the status of the object. how is the reconcile loop triggered? Can we exclude triggering on status change?
Another example would be not to trigger a reconcile when it is just the metadata that changes.

I don't believe it is possible to exclude triggering for only certain types of changes. The Kubernetes server sends all changes that occur to the operator that is watching the resource type, which is what triggers the reconcile loop.

@gpanagiotidis

Another example would be not to trigger a reconcile when it is just the metadata that changes.

You can filter out events by using Predicates https://github.com/operator-framework/operator-sdk/blob/1b1ebc09edf6d967c96da6ea6318c8e555209846/doc/user/event-filtering.md

Just commenting to say that I am using operator-utils which manages the triggering of the reconcile loop.


Issue with FetchContent when not using master tag

If I try to build with any tag other than master I get the following error:
  Target "testExe" links to:

    CppUTest::CppUTest

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

This is the CMakeLists.txt file:
cmake_minimum_required(VERSION 3.27)

project(TestingCppUTest
        VERSION 0.1.0
        LANGUAGES C CXX
)

set(EXECUTABLE_NAME appExe)
set(TEST_EXECUTABLE_NAME testExe)

set(CMAKE_EXPORT_COMPILE_COMMANDS ON)

include(FetchContent)

FetchContent_Declare(
    CppUTest
    GIT_REPOSITORY https://github.com/cpputest/cpputest.git
    GIT_TAG        v4.0
)
# Set this to ON if you want to have the CppUTests in your project as well.
set(TESTS OFF CACHE BOOL "Switch off CppUTest Test build")
FetchContent_MakeAvailable(CppUTest)

set(EXECUTABLE_SOURCES
    "${CMAKE_CURRENT_SOURCE_DIR}/main.c"
    "${CMAKE_CURRENT_SOURCE_DIR}/main_code.c"
)
set(TEST_SOURCES
    "${CMAKE_CURRENT_SOURCE_DIR}/test.cc"
    "${CMAKE_CURRENT_SOURCE_DIR}/allTests.cc"
)

add_executable(${EXECUTABLE_NAME} ${EXECUTABLE_SOURCES})

add_executable(${TEST_EXECUTABLE_NAME} ${TEST_SOURCES})
target_link_libraries(${TEST_EXECUTABLE_NAME} PRIVATE
    CppUTest::CppUTest
    CppUTest::CppUTestExt
)

If I change the GIT_TAG to master in the CMakeLists.txt file it all works. So like this:
cmake_minimum_required(VERSION 3.27)

project(TestingCppUTest
        VERSION 0.1.0
        LANGUAGES C CXX
)

set(EXECUTABLE_NAME appExe)
set(TEST_EXECUTABLE_NAME testExe)

set(CMAKE_EXPORT_COMPILE_COMMANDS ON)

include(FetchContent)

FetchContent_Declare(
    CppUTest
    GIT_REPOSITORY https://github.com/cpputest/cpputest.git
    GIT_TAG        master
)
# Set this to ON if you want to have the CppUTests in your project as well.
set(TESTS OFF CACHE BOOL "Switch off CppUTest Test build")
FetchContent_MakeAvailable(CppUTest)

set(EXECUTABLE_SOURCES
    "${CMAKE_CURRENT_SOURCE_DIR}/main.c"
    "${CMAKE_CURRENT_SOURCE_DIR}/main_code.c"
)
set(TEST_SOURCES
    "${CMAKE_CURRENT_SOURCE_DIR}/test.cc"
    "${CMAKE_CURRENT_SOURCE_DIR}/allTests.cc"
)

add_executable(${EXECUTABLE_NAME} ${EXECUTABLE_SOURCES})

add_executable(${TEST_EXECUTABLE_NAME} ${TEST_SOURCES})
target_link_libraries(${TEST_EXECUTABLE_NAME} PRIVATE
    CppUTest::CppUTest
    CppUTest::CppUTestExt
)

All other tags fail as well. This is on a new install of Fedora 39.

CppUTest::CppUTest and CppUTest::CppUTestExt were added after the latest release. Use CppUTest and CppUTestExt. If you're using a tag, I suggest referencing the README from that tag, e.g. https://github.com/cpputest/cpputest/tree/v4.0.

I see, I used the README from the main project and not the tag. Thanks.


Authentication Fails

Hi,
I cannot for the life of me get swifter to work. These are the steps I took:

run the swifter demo as is, it works fine
copy and paste my Consumer Key, and Consumer Key Secret instead of yours

i.e Swifter(consumerKey: "...", consumerSecret: "...")

re-run the application

it fails

when I set useACAccount = true I get:

HTTP Status 403: Forbidden,
Response: {"errors":[{
"message":
"Your credentials do not allow access to this resource",
"code":220
}]}

Am I missing something?
P.S I'm using IOS 8, XCode 7.1

Oh sorry I was missing the callback URL in twitter. I'm new to the twitter API. May be worth mentioning in the readme. Thanks.


What is the difference between pyrsmi and rocm_smi.py

Wondering what is the difference between pyrsmi and rocm_smi.py .

@jcarranz97  Thank you for your query.
Assuming you're asking about python_smi_tools/rocm_smi.py, which is part of rocm_smi_lib, pyrsmi is based on the former but is more to be used in a Python project and imported as a package, while rom_smi_lib and rocm_smi.py is a CLI application and can be used as a standalone application. So, for applications in Python projects, CLI application will incur overhead of spawning a separate process, while pyrsmi function calls within Python will have much smaller overhead.
Hope this clarifies.


First Prototype Implementation for the Entire Website

This pull request marks the introduction of the first prototype for our entire website. This foundational step includes the initial setup, core pages, and a responsive design framework using Next.js and Tailwind CSS. The primary focus at this stage has been on establishing a scalable architecture and a user-friendly interface, starting with the Contact Us page as our initial content offering.
Key Features:

Next.js Framework: Utilizes Next.js for its server-side rendering capabilities, ensuring fast load times and a seamless user experience.
Tailwind CSS Integration: Implements Tailwind CSS for styling, facilitating a responsive and customizable design approach.
Initial Page Setup: Includes the Contact Us page，Future Plan page, About Us sections, and Affiliation page  as the first content page.
Responsive Design: Ensures that the website is fully responsive, providing an optimal viewing experience across all devices.

Issues to solved in the future

Font and Typography: The current prototype does not fully address font and typography styling issues. Future work will focus on refining font choices, ensuring consistent typography across the site, and improving readability.
SEO Optimization: The initial setup does not include comprehensive SEO optimization. Future updates will implement SEO best practices, including meta tags, alt attributes for images, and improved content structure to enhance search engine visibility.

PR submission guidelines are needed in the future.


[wxwidgets] Add 'WXWIDGETS_USE_STD_CONTAINERS' option

What does your PR fix?
It adds a new option WXWIDGETS_USE_STD_CONTAINERS to set wxUSE_STD_CONTAINERS macro, which isn't controlled by wxUSE_STL that is set by WXWIDGETS_USE_STL option.

Does your PR follow the maintainer guide?
Yes, I think.

Yes.

Thanks for your PR. vcpkg_fail_port_install has been deprecated, VCPKG use supports in vcpkg.json instead of it. Could you please remove it in portfile.cmake?

Hmm based on the documentation it appears that this causes breaking changes when turned on: https://docs.wxwidgets.org/trunk/overview_container.html

The standard container build is mostly, but not quite, compatible with the default one
[...]

I think that means it doesn't follow "do not use features to implement alternatives" https://github.com/microsoft/vcpkg/blob/master/docs/maintainers/maintainer-guide.md#do-not-use-features-to-implement-alternatives
and as a result I think if you want this setting you really need to use an overlay port. Unless I have misunderstood the situation?

That's not a feature though, just an option for a custom triplet that is off by default.

Hmm based on the documentation it appears that this causes breaking changes when turned on:

It's the same as wxUSE_STL actually. I added similar option for wxUSE_STD_CONTAINERS . There's a discussion about it on https://github.com/microsoft/vcpkg/pull/19274, I don't really know what else I could add to this.

That's not a feature though, just an option for a custom triplet that is off by default.

Hmm based on the documentation it appears that this causes breaking changes when turned on:

It's the same as wxUSE_STL actually. I added similar option for wxUSE_STD_CONTAINERS . There's a discussion about it on #19274, I don't really know what else I could add to this.

That does it as a triplet option, not a feature.

Oh wait, I am blind. I could have swore I saw something in this PR that talked about using features to control this rather than a triplet option, let me review again.

Thanks again!


Change transform css shrink

I made a change in the style of the labels from the TextFields to a font-size of 14px. I would like to make a change in the shrink transform, because when it is applied to the default theme, the font gets too small. But I could not find a way to do this.
Is there a way to change the value of the shrink transform's css in a TextField?
Current style:
transform: translate(0, 1.5px) scale(0.75);
Material-UI: v1.0.0-beta.11
React: 15.6.1

@alex-olivera Yes, there is many way to change such style. I would suggest you to have a look at the overrides section of the documentation. If this is not enough, maybe you could ask on StackOverflow. I won't answer this type of question every time. The patterns are here and working. Unless you have got any idea around how to make it more intuitive, I'm closing.

Sorry, been looking for an hour but can't find a solution. How do I modify an input label that is dense and outlined in theme overrides?
.MuiInputLabel-outlined.MuiInputLabel-marginDense

Never mind. Figured it out:
        MuiInputLabel: {
            outlined: {
                '&$marginDense': {
                    transform: 'translate(14px, 17px) scale(1)'
                }
            }
        },

@alex-olivera Yes, there is many way to change such style. I would suggest you to have a look at the overrides section of the documentation. If this is not enough, maybe you could ask on StackOverflow. I won't answer this type of question every time. The patterns are here and working. Unless you have got any idea around how to make it more intuitive, I'm closing.

Link not working


acevalidator: fix stderr reference

4b128a485bc00687dc8d6f3bcb1adbe6e2c255dd tried to standardise on stderr() but
unfortunately updated the validator to use the function when it's in a
different package; this wasn't caught since we did not have travis confiugred.
For now just introduce the same function to this package.

This does not quite fix the build, there is still some fallout from #82, but I will clear that up in subsequent commit.


pdfstudio: fix hash mismatch

Fixes #360176
Close #287313
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

There seems to be no stable download link for pdfstudioviewer

So you have to use an archived download url from e.g. webarchive.


Input select-accross is not in action form

This input is not inside the changelist-form so when you click on select all items in the list and then select an action, the action is only applying to the 10 items displayed on the page.
<input type="hidden" name="select_across" value="1" class="select-across">

Hi, any workaround for this?

I did something kind of hacky to fix it.
I overrode admin/base_site.html and in the footer section I added my own javascript to fix it.
`
$(document).on('click', '.question', function () {
$("#changelist-form").append('');
let clearButton = document.querySelectorAll('.card-content span.clear')[0];
clearButton.style = "display: inline;"
})
$(document).on('click', '.card-content span.clear a', function () {
let clearButton = document.querySelectorAll('.card-content span.clear')[0];
clearButton.style = "display: none;"
})
`

You can also just edit/override the template actions.html and add "display: none;" to the proper span tags. No JS required.
Page should initially load with display set to none anyway but seems it was forgotten.
Also for input select_across. you can also just add the form attribute and give it the value "changelist-form"


wrote comments for some of HTML & JS codes

🛠 Fixes Issue

Related Issue/Addition to code

Issue goes here

👨💻 Changes proposed

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

✔ 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 of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Original
Updated

** original screenshot
** updated screenshot **

Merge the branch when feasible so then I can close the PR

Merge the branch when feasible so then I can close the PR

What do you mean? You said there is more to add to this PR above?

will complete the rest

Oh sorry! Yeah yeah. I'm sorry for the inconvenience.

I've almost finished the issue. Please review the code, and please add me to the project's list of contributors. For comments, I'll also look through additional files.

@sharmavivek223 . I forgot about the credits thing. Many people may be missing. Let me add them too. Thanks for reminding.

Don't edit credits/index.html. it may cause some issues.

Okay got it.

@KendallDoesCoding can you confirm if I have covered all the project directories. If there's any left do tell. Thank you!

@KendallDoesCoding can you confirm if I have covered all the project directories. If there's any left do tell. Thank you!

I don't think you've done index.html and css files.

@KendallDoesCoding can you confirm if I have covered all the project directories. If there's any left do tell. Thank you!

I don't think you've done index.html and css files.

Okay got it. I'm on it.

@KendallDoesCoding can you confirm if I have covered all the project directories. If there's any left do tell. Thank you!

I don't think you've done index.html and css files.

And apart from index.html and css files all the rest is great right?

@KendallDoesCoding can you confirm if I have covered all the project directories. If there's any left do tell. Thank you!

I don't think you've done index.html and css files.

And apart from index.html and css files all the rest is great right?

Yup.

Thanks for your contribution @harisdev-netizen

No problem @KendallDoesCoding, I am really enjoying contributing to open-source.

I hope you have added me to contributors list.

@add-contributors pls add @harisdev-netizen to the doc

@harisdev-netizen - adding ur pfp now, will add ur contribution (pr) later.

@all-contributors please add @harisdev-netizen to the doc.

Kindly review the code and merge.

already merged bro


[README] Document YellowPages::Category

We added YellowPages::Category in https://github.com/hackclub/yellow_pages/pull/26
We do have generated documentation located at https://yellowpages.bank.engineering/YellowPages/Category.html
However, it would be nice to mention the existence and maybe usage of YellowPages::Category in the README.

Another edit to make:

the gem's "branding" doesn't need to resolve around Stripe since the network ids are VISA specific, and categories should be consistent across networks (it's an ISO standard).
Also, I think the link we have on Stripe MID is incorrect since that doesn't refer to network ids.

i think we gotta mention stripe, i'm pretty sure nobody but them uses the same keys for categories

Yeah, we should still mention Stripe


Fixed credentials authentification

DESCRIPTION
When a user tries to sign in with a valid credentials, he succeed but as soon as he is forwarded to his account space a client side error is raised:
this raise this errorObjects are not valid as a React child (found: [object Error]). If you meant to render a collection of children, use an array instead.
it works locally just fine, but when deploying to Vercel then the error gets generated.
SOLUTION
It turns out that vercel only provide a function excution timeout of 10s which its seems the total duration of the sigin in, verifying user and redirection exceed that timeout given which was raising the following error:
Vercel Runtime Timeout Error: Task timed out after 10 seconds
So, to solve this, i had to reduce the task duration by reducing supabase database calls, and I discovered that the SignIn callback of the AuthJS gets excuted even after the success of credentials authorize() function which is meant to authenticate the user, and since the both functions gets executed then there was multiple uncessary supabase calls for the both functions, so I conditionally disabled signin() when a user is trying to sign in through credentials  provided, and left signin() handle authentication requests through other OAuth providers such as Google and Facebook in my case.

        async signIn({ account, user }) {
  // When credentials are valid, there is no need to go through SignIn callback because
  // all the needed validation have been handled in the authorize() of the credentials provider
  if (account.provider === "credentials") return true;

  // This is for facebook provider, since some accounts might not be bound with an email address
  if (!user.email) return false;
 ......

}

Also i found some other delaying simulations I previously created to test <Suspense> for async tasks and it was also adding to the excution time which helped to exceed the 10s given by vecel on free tiers.

fixed credentials authentication error.


MI dashboard and CLI copy edits

 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.


Keyboard shortcuts on Receive Coins not working

Looks like the only one that works is Copy Address <ctrl+c>
<ctrl+q> for Show QR Code quits the application
None of the others seem to work.

I've only tested this on linux so let me know if they work on anything else please.

It would appear that none of the keyboard shortcuts defined by use of Ampersand in the ui file actually have any keyboard shortcuts set to them.
Basically whomever set them was just being silly as they're non-existent. The only one that is is the Ctrl+Q function which is set to quit so this cannot be used for the QR code stuff.
(we also have Alt+1-6 to change tabs.

It works for me on Windows. Although from testing I have discovered the Ctrl+S shortcut for Sign Message clashes with the Settings shortcut in the top toolbar.

Have you been testing this on your arch box or Ubuntu? I know we ran into this issue on your arch box once before but Ubuntu was fine.

Specific to my machine config, closing issue.


how to reload PageControlViewController after I added new view controller ?

Hi, you can call the function updateData()
I hope this helps

Thanks man !!! :)
Sent from my iPhone

On Mar 12, 2018, at 18:44, Rodrigo Martins<EMAIL_ADDRESS>wrote:
Hi, you can call the function updateData()
I hope this helps
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Regression: reintroduce [source] link

In nbdev1, when generating the docs it created a [source] link that opened the source code for the specific function or class in GitHub, but I see that's not there new.

I'd be happy to work on bringing it back if you think it's a good idea, and someone can point me in the right direction!
Thanks,
Kristian.

We're already working on adding it back, don't worry! But I see we don't have an issue for it, so we'll track it here.

great, let me know if there's anything I can help with :)


Source locations

Resolves #17

Shit @favonia I should have had you start from this branch LOL!

@jonsterling Done (?).

@Favonia Thank god!! LOL

Can you approve https://github.com/jonsterling/cooltt/pull/71?

this looks great!

@ivoysey 👍 Thanks!!


Add test scripts and inrease code coverage

Codecov Report

Merging #129 (5aec369) into master (1799751) will increase coverage by 0.25%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage   77.55%   77.81%   +0.25%
==========================================
  Files         117      117
  Lines       33440    33441       +1
==========================================
+ Hits        25935    26022      +87
+ Misses       7505     7419      -86

Impacted Files
Coverage Δ

...t/dynadjust/dnaadjustwrapper/dnaadjustprogress.cpp
71.27% <0.00%> (-0.56%)
:arrow_down:

dynadjust/dynadjust/dnaadjust/dnaadjust.cpp
87.18% <0.00%> (-0.02%)
:arrow_down:

dynadjust/dynadjust/dnaplot/dnaplot.cpp
78.59% <0.00%> (+3.98%)
: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 1799751...5aec369. Read the comment docs.


Replaced fixed 'bolder' font weight value in reboot with variable.

The default font weight value for <strong> <br> in reboot was set by a fixed value we now use the variable $font-weight-bolder to let users override this value.
Fixes #27774

@MartijnCuppens 🤘


Make Version::license optional

The license field in the API response can be null.
The hyper crate is "not found" in online mode because of this.

I see, thanks

I published a new version, v0.9.7 which incorporates this fix along with bumping the dependencies.
I really should spend some time maintaining this utility, the new crate (feature) resolver is now stabilized and will be the default for rust 2021.  I'm pretty sure its behavior is a bit different from the assumptions of this tool.


Input Value Substitution Not Working in Action.Http Body

Platform
What platform is your issue or question related to? (Delete other platforms).

[ ] .NET HTML
[ ] .NET WPF
[ ] Android
[ ] iOS
[ ] JavaScript
[ ] UWP

Author or host
Author: Sending cards to myself as a test using Microsoft Flow.
Version of SDK
V1.0
Details
Input value substitution is not working properly. The below adaptive card renders correctly in Outlook and functions correctly. However when receiving the HTTP Reponse, I am getting {{newVendorNames.value}} instead of whatever I typed into newVendorNames. Same with {{newVendorDocs.value}} and {{existingVendorAction.value}}.
{
  "type": "AdaptiveCard",
  "originator": "ORIGINATOR ID",
  "body": [
    {
      "type": "TextBlock",
      "size": "medium",
      "weight": "bolder",
      "text": "Title Block"
    },
    {
      "type": "TextBlock",
      "text": "BLAH BLAH.",
      "wrap": true
    }
  ],
  "actions": [
    {
      "type": "Action.ShowCard",
      "title": "New Vendor",
      "card": {
        "type": "AdaptiveCard",
        "body": [
          {
            "type": "TextBlock",
            "text": "FIRST RESPONSE",
            "wrap": true,
            "color": "attention",
            "size": "medium"
          },
          {
            "type": "Input.Text",
            "id": "newVendorNames",
            "placeholder": "newVendorNames",
            "isMultiline": true
          },
          {
            "type": "Input.Toggle",
            "title": "newVendorDocs",
            "id": "newVendorDocs",
            "valueOn": "Yes",
            "valueOff": "No"
          }
        ]
      }
    },
    {
      "type": "Action.ShowCard",
      "title": "Existing Vendors",
      "card": {
        "type": "AdaptiveCard",
        "body": [
          {
            "type": "TextBlock",
            "text": "SECOND QUESTION",
            "wrap": true,
            "color": "attention",
            "size": "medium"
          },
          {
            "type": "TextBlock",
            "text": "SECOND QUESTION",
            "wrap": true
          },
          {
            "type": "Input.ChoiceSet",
            "choices": [
              {
                "title": "Answer 1",
                "value": "Answer 1"
              },
              {
                "title": "Answer 2",
                "value": "Answer 2"
              },
              {
                "title": "Answer 3",
                "value": "Answer 3"
              },
              {
                "title": "Answer 4",
                "value": "Answer 4"
              }
            ],
            "id": "existingVendorAction",
            "style": "expanded"
          }
        ]
      }
    },
    {
      "type": "Action.Http",
      "title": "Submit Response",
      "method": "POST",
      "headers": [
        {
          "name": "Authorization",
          "value": ""
        },
        {
          "name": "Content-Type",
          "value": "application/json"
        }
      ],
      "url": "HTTP RECEIVE URL",
      "isPrimary": true,
      "body": "{\"newVendorNames\": \"{{newVendorNames.value}}\",\"newVendorDocs\": \"{{newVendorDocs.value}}\",\"existingVendorAction\": \"{{existingVendorAction.value}}\"}"
  }
  ],
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.0"
}

I had a look at the issue.
We do capture input values for an action for following scenarios:
Action button in Show card:
Inputs defined in show card and parent card will be captured.
Action button in main card:
Inputs defined in parent card will only be captured.
Here, the action button is part of main card and we are expecting to capture inputs defined in the different show cards, which is not supported as per design. It seems a bit confusing to the users as well. The suggestion from our side is to move the Submit action inside the show cards.
Especially in mobile, we open show card in a new window, and if I follow the current card design, I will have to fill input value in the new pop-up window, close it and then click on the submit button. IMO, this is a bit counter intuitive. It would be more natural if user fills up the input and click on an action button which is inside the show card itself.

Thanks @pankajpalOlk. I'll see if I can redesign this. Wasn't aware of any scope limitations with the cards.

Confirming that was the issue! Thanks!
Suggesting that the Input Value Substitution documentation be updated to be a little clearer.