QT Issues- dry run.

•	The QT Sub dashboard do not display all candidates for example on activate/initialised on the dry run QT. We had 75 dry run candidates added you can only view around 40-50 candidates on the first page the next page is blank although we had 75 the 25 remaining were missing from display.
•	Summary/Review page (candidate view) - comes up to show which questions have been answered and those that have been skipped, there are no question numbers next to the question. This is a little off putting if you make a note, as a candidate, about a question you want to go back to- you either have to count the questions or try to work out which one it is- this adds to the candidate time. Nice to have but this has to be tested by UTG

Firefox issue (firefox/chrome/safari)  -  error message pop up “Function DocumentReference.update() called with invalid data. Unsupported field value: undefined (found in field client.deviceMemory in document qualifyingTestResponses/kqqja2JGLtRBkvg57Zyc)” -

Another version of the error code
Function DocumentReference.update() called with invalid data. Unsupported field value: undefined (found in field client.deviceMemory in document qualifyingTestResponses/XVKzoq0BlCwxMjt2cNOz)

@warrensearle These issues have to be fixed by the 13th of August please.

Hey @joy-ade as discussed the following items have been removed from this ticket and will be addressed separately. Browser compatibility is already captured in ticket apply#395:

Browser Compatibility: Ensuring all major browsers: Google Chrome, Mozilla Firefox and Safari work on the platform and the live QT. Microsoft Edge is the only browser which seems to work fine with no issues.
Device Compatibility: Ensuring the platform works effectively with Apple devices: Mac Book Pro and Ipads as well.
Password Reset: Ensuring the password reset function for candidates is very thorough as candidates still do not receive password reset emails in the first instance.


How to implement your firewall recommendations

Of your following list of recommendations, most of them do not exist in the default windows firewall list, furthermore some of these applications, such as wscript, can be located in obscure directories, placed by malware, as I have found.
That said, how do we successfully implement your firewall rules? It would be great if you would publicize a powershell command that allows one to automatically import them.
[-] ID 2300, HardeningKitty-Block-TCP-NetBIOS, Result=, Recommended=True, Severity=Low [-] ID 2301, HardeningKitty-Block-TCP-RDP, Result=, Recommended=True, Severity=Low [-] ID 2302, HardeningKitty-Block-TCP-RPC, Result=, Recommended=True, Severity=Low [-] ID 2303, HardeningKitty-Block-TCP-SMB, Result=, Recommended=True, Severity=Low [-] ID 2304, HardeningKitty-Block-TCP-WinRM, Result=, Recommended=True, Severity=Low [-] ID 2305, HardeningKitty-Block-UDP-NetBIOS, Result=, Recommended=True, Severity=Low [-] ID 2306, HardeningKitty-Block-UDP-RPC, Result=, Recommended=True, Severity=Low [-] ID 2307, HardeningKitty-Block-calc-x64, Result=, Recommended=True, Severity=Low [-] ID 2308, HardeningKitty-Block-calc-x86, Result=, Recommended=True, Severity=Low [-] ID 2309, HardeningKitty-Block-certutil-x64, Result=, Recommended=True, Severity=Low [-] ID 2310, HardeningKitty-Block-certutil-x86, Result=, Recommended=True, Severity=Low [-] ID 2311, HardeningKitty-Block-conhost-x64, Result=, Recommended=True, Severity=Low [-] ID 2312, HardeningKitty-Block-conhost-x86, Result=, Recommended=True, Severity=Low [-] ID 2313, HardeningKitty-Block-cscript-x64, Result=, Recommended=True, Severity=Low [-] ID 2314, HardeningKitty-Block-cscript-x86, Result=, Recommended=True, Severity=Low [-] ID 2315, HardeningKitty-Block-mshta-x64, Result=, Recommended=True, Severity=Low [-] ID 2316, HardeningKitty-Block-mshta-x86, Result=, Recommended=True, Severity=Low [-] ID 2317, HardeningKitty-Block-notepad-x64, Result=, Recommended=True, Severity=Low [-] ID 2318, HardeningKitty-Block-notepad-x86, Result=, Recommended=True, Severity=Low [-] ID 2319, HardeningKitty-Block-RunScriptHelper-x64, Result=, Recommended=True, Severity=Low [-] ID 2320, HardeningKitty-Block-RunScriptHelper-x86, Result=, Recommended=True, Severity=Low [-] ID 2321, HardeningKitty-Block-wscript-x64, Result=, Recommended=True, Severity=Low [-] ID 2322, HardeningKitty-Block-wscript-x86, Result=, Recommended=True, Severity=Low

I recommend creating a new finding list that only includes the firewall checks from the 0x6d69636b list and use HailMary mode
PS C:\> Invoke-HardeningKitty -Mode HailMary -FileFindingList .\name_of_your_firewall_list.csv

Or you can use the PowerShell cmdlet New-NetFirewallRule to create all rules you need.


condition logic semantics fix

(trying this again)
Short Description:
I know this was just fixed only minutes ago, but the current fix doesn't sit well with me as it doesn't "fix" the conditionals themselves, just short-circuits them. There's the (not-so-realistic) danger that if someone wants special processing later on "within" the conditionals when those values are 0, that logic will never be hit.
At any rate, if no one else cares or folks see this as pedantic, feel free to close! :)
Fixes:

conditional logic for release_config key checks

fixing a line alignment

Same as @douglascamata  pointed out, you are insane guys LOL


Certificates fail to update after failed run because force_enable was needed

I have multiple Scientific Linux 6 server where the ca_cert module is failing on Exec[enable_ca_trust] because of local modifications that necessitated doing a update-ca-trust force-enable. I manually ran update-ca-trust force-enable and then ran puppet agent -t. This time there was no error, but nothing happened. I verified that the certificate had not been deployed (by doing curl to an internal server using a certificate signed by my CA) even though the certificate file was installed in /etc/pki/ca-trust/source/anchors. I then removed the certificate from /etc/pki/ca-trust/source/anchors and ran puppet agent -t again. This time the certificate was successfully deployed.

Did puppet drop the certificate in /etc/pki/ca-trust/source/anchors or was that done some other way? The module updates the CA list only when it's making a change to the various CA folders, so if it landed there through some other means then it won't do anything.

Puppet dropped the file. I see four events when it dropped.
Exec[enable_ca_trust] - Failed - changed from notrun to ["0"]
Exec[enable_ca_trust] - Failed
File[my_certificate.crt] - Intentional change - changed from absent to file
Exec[ca_cert_update] - Skipped
For every run after that failure, I see just two events.
Exec[enable_ca_trust] - Failed - changed from notrun to ["0"]
Exec[ca_cert_update] - Skipped
Once I manually ran update-ca-trust force-enable, there were no change events or failures.
I see now that if I had used Puppet to do the force-enable instead of doing it manually, that would have worked, but I wanted to examine the situation first and then I figured I would just run it manually.
It seems to me that the files should not be dropped until the enable is successfully run.

One other observation. The documentation says the ca_cert class has a force_enable parameter, but it does not appear that it does based on what I see in the code.

It looks like the force_enable parameter was broken (I think I fixed it in ddbb046).
I'm a little confused about why a file would have been dropped by Puppet (nothing should be dependent on the commands running successfully unless the command is deleting the files).
Am I correct in assuming you used a ca_cert::ca resource to define the certificate itself?

No, I used the ca_certs parameter on the ca_cert class to define the certificate.

Here is the issue. You are including the ca_cert::update class at the top of the ca_cert class. That will attempt to run enable_ca_trust, but it does not run update_ca_trust unless notify event is sent to the resource, but that happens only when a certificate file is dropped. In my case, on the very first run, when the certificate file was dropped, the notify was sent to the ca_cert_update resource, but it was skipped because the enable_ca_trust had failed. On all later runs, there was no notify sent to update_ca_trust because the file did not need to be dropped. It was already there. There needs to be a dependency so that files are not dropped until enable_ca_trust is successful. I suppose I could just set the option to always run update_ca_trust, but I don't think I should have to.

I guess I should rephrase. A notify would still be sent to update_ca_trust, but with refreshonly enabled, it won't run unless a file is actually dropped, which wouldn't happen if the file is already there.


Upgrade to latest PBS.

Remove test pinning so that we're always testing latest and bump science
from Python 3.11.1 to 3.11.3.

I'll be damned. I'm going to sit on this result for a bit. My Windows machine has no such issues, but I have developer mode enabled. A quick browse of GH issues shows they had long paths enabled in the registry for Windows 2019 server and I'm using Windows 2022 here ...


Add command line tool check

This PR fix #80
A new protocol CommandLineExecuter has been introduced in order to reuse what has been done for SimCtl for other controllers that require IO from the command line.

Wonderful – thank you!


request - JSON and XML serialization for C++/CX

While I know there are various toolkits around the web, it would be great if we had a sample like this in pure winRT.

Yes, we're aware of this gap for C++ and have it in our backlog to add the sample.


feat: add jina to full requirment

Context
DocArray rely on Jina for the DocumentArray.post method.
Nevertheless there are no direct dependency in the requirement (only in test).
What this PR do:
This PR add jina to the full requirement

We don't want to explicitly introduce the a cyclic dependency with Jina which depend itself on docarray.
The need to install Jina is clearly explained in the documentation about the post method.


Updates for web extensions are not showing up in the add-ons manager

Firefox version tested on:
Firefox 43.0.4
test webextension used:
https://addons-dev.allizom.org/en-US/firefox/addon/shutterstock-tab/
steps to reproduce:

Install a web extension version from addons-dev 1
Make sure the web extension has an update which is approved and public
Tools -> Add-ons; check for updates

expected behavior:
Update for webextension shows up
actual behavior:
Updates for regular add-ons show up but updates for webextensions are not getting listed in the add-ons manager
about:support info
https://pastebin.mozilla.org/8856416
Console log:
https://pastebin.mozilla.org/8856415 is from the console. It clearly says  addons.update-checker      DEBUG Found an update entry for<EMAIL_ADDRESS>version 1.2.3

Problem is also reproducible on the latest nightly. See https://www.dropbox.com/s/hda9bj12o0ew8lg/Screenshot 2016-01-12 13.56.29.png?dl=0
Highlighted line shows that there is an update for the webextension i am looking at.

Because I think this is in Firefox, moving over to Bugzilla https://bugzilla.mozilla.org/show_bug.cgi?id=1239109.

See comments in the bug, if we set this add-on to 43.* it works because of strict compatibility checking.

I'm not sure what should be done here: I could make it so a WebExtension is set to 43-43.* if that's better, but if WebExtensions aren't default to compatible (that's in Firefox? We don't do anything specific regarding D2C for WebExtensions on AMO), then we'll have the same issue as soon as 44 is out, right?

So, I think we want to close this in favor of #1339, please reopen if I'm wrong.


gh-685: etcd locking for horizontal scaling

Effect: closes #685
Status: ready to be merged if Travis is green
Strategy: regular merge
Note: This will break the cloud-setup until the cloud setup also contains an etcd

@idcrosby please note the PR-remark.

@etiennedi can we configure the hostname on which weaviate will try to connect to etcd?

Yes, in the config file, similar to how we configure the other db backends:
https://github.com/creativesoftwarefdn/weaviate/blob/ff74a1a914f0b2fd08b9c8e3373f0d10703d4878/tools/dev/config.json#L11-L14


Automator: update proxy@master in istio/istio@master

Generated by Automator - 2022-09-15T01:52:17+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster_istio
a7d4a75c48dd652e3a831c180e3d2b66a7eb691d
link
true
/test integ-security-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-security-multicluster


Remove updating bounds for base rows in lazy vector

We found the loaded vector can have smaller size than expected, which caused invalid access to the base vector in our test. Through debugging, we found with baseRows.updateBounds(), the baseRows.end() can become less than lazyVector->size(). As a result, the size of vector loaded with rowSet = RowSet(rowNumbers) was smaller than lazyVector->size(). Details can also be seen in https://github.com/facebookincubator/velox/issues/2237.
This PR removed the bounds updating to ensure enough rows could be loaded. Not sure whether this is appropriate, but it can solve the failure in our test. Please help take a review, thanks.

Hi @FelixYBW, please help take a look on this PR, thanks.

Closing this PR because PR#2089 can solve this issue in our local test.


fetching:collection event returns empty collection

The 'fetching:collection' event is returning an empty collection but the fetchAll results contain expected models.
The below event returns a collection without any models. The actual fetchAll() call to this model does return the models I expect but I want to manipulate the data at the event stage.
Am I misunderstanding something about this event? How can I manipulate the data in each model (I specifically am decrypting some PII fields) on a fetchAll call via the event/model layer.
// Model
export class PublicEvent extends bookshelf.Model<PublicEvent> {
    pii: Array<string>;
    get tableName() { return 'public_events'; }
    get hasTimestamps() { return true; }

    initialize() {
        this.pii = [
            'first_name',
            'last_name',
            'email',
            'phone',
            'comments',
            'address'
        ];
        this.on('fetching:collection', (collection, attrs, options) => {
            console.log(collection.toJSON());
             // Returns '[]' but I expected collection models. I want to loop through each model
             // and decrypt some values.
        });
    }
}

That's just what I needed, thank you!

As far as I can tell, when doing allot of bookshelf hacking to get namedModifiers working (e.g. predefined fn's for .query() methods to be use on fetch and down the related chain) fetching:collection isn't necessary and actually inconsistent. It's better to just define a custom collection for said model and register fetching handler.


pipeline-input-step:2.8 Test cases run failed (Server Issue)

Jenkins and plugins versions report

Environment

Paste the output here

What Operating System are you using (both controller, and any agents involved in the problem)?
I want to run test cases for pipeline-input-step:2.8 . But after running mvn test, I am getting test failure.
and exception is "[28 10:25:55.417] | Running org.jenkinsci.plugins.workflow.support.steps.input.InputStepConfigTest
[28 10:26:11.601] | Tests run: 6, Failures: 0, Errors: 5, Skipped: 0, Time elapsed: 15.963 sec <<< FAILURE! - in org.jenkinsci.plugins.workflow.support.steps.input.InputStepConfigTest
[28 10:26:11.601] | configRoundTrip(org.jenkinsci.plugins.workflow.support.steps.input.InputStepConfigTest)  Time elapsed: 1.55 sec  <<< ERROR!
[28 10:26:11.601] | com.gargoylesoftware.htmlunit.FailingHttpStatusCodeException: 500 org/dom4j/DocumentException for http://localhost:46261/jenkins/job/test0/configure
[28 10:26:11.601] | at com.gargoylesoftware.htmlunit.WebClient.throwFailingHttpStatusCodeExceptionIfNecessary(WebClient.java:535)
[28 10:26:11.601] | at com.gargoylesoftware.htmlunit.WebClient.getPage(WebClient.java:360)"
I have already used JDK - 8 , 11 , 17 but got failed. Please help me with correct JDK, Maven version.
Reproduction steps
I want to run test cases for pipeline-input-step:2.8 . But after running mvn test, I am getting test failure.
and exception is "[28 10:25:55.417] | Running org.jenkinsci.plugins.workflow.support.steps.input.InputStepConfigTest
[28 10:26:11.601] | Tests run: 6, Failures: 0, Errors: 5, Skipped: 0, Time elapsed: 15.963 sec <<< FAILURE! - in org.jenkinsci.plugins.workflow.support.steps.input.InputStepConfigTest
[28 10:26:11.601] | configRoundTrip(org.jenkinsci.plugins.workflow.support.steps.input.InputStepConfigTest)  Time elapsed: 1.55 sec  <<< ERROR!
[28 10:26:11.601] | com.gargoylesoftware.htmlunit.FailingHttpStatusCodeException: 500 org/dom4j/DocumentException for http://localhost:46261/jenkins/job/test0/configure
[28 10:26:11.601] | at com.gargoylesoftware.htmlunit.WebClient.throwFailingHttpStatusCodeExceptionIfNecessary(WebClient.java:535)
[28 10:26:11.601] | at com.gargoylesoftware.htmlunit.WebClient.getPage(WebClient.java:360)"
Expected Results
Should be pass
Actual Results
failed
Anything else?
NA
Are you interested in contributing a fix?
No response

I'm closing this issue as there are no information to reproduce the problem, to analyze the issue or showing it is related to the official Docker image for Jenkins controllers.
You should open a discussion on the community forum to get help from the whole community at community.jenkins.io.
Also, note the issue templates asking for environment description is there for a reason: if you don't provide this kind of information, chances of getting a solution are close to zero: what is the Jenkins version? How is it installed? What are the installed plugins in which version?

Getting same above issue for mvn

I am also facing same thing.

@meetankit24 @sourabhmagdum45 as explained by @dduportal we need more details and context.

@meetankit24 @sourabhmagdum45 as explained by @dduportal we need more details and context.

Have used JDK 8 , 11 and 17 to run UT cases. Let me know what are required more?

@meetankit24 @sourabhmagdum45 as explained by @dduportal we need more details and context.

Have used JDK 8 , 11 and 17 to run UT cases. Let me know what are required more?

Please read my previous message.

Please find the attached log file.
build-pipeline-input-step-2.8.txt

Please find the attached log file. build-pipeline-input-step-2.8.txt

If you keep ignoring maintainers comment and instructions, you will never get help.
Last comment before I lock this issue: read my message at #1843 (comment) , and follow the instructions by opening a thread in the community forum with the requested information. If we don't know what jenkins version, plugins, pipeline, etc. you are doing, we cannot help.

We have already tried with below information.
Java - 8,11,17
Maven - 3.3.1 to 3.9.4
jenkins - 2.361. 1
Please let me know , what are correct version to be used.

I'm locking the conversation as the requester does not make the effort to read and follow my instructions. This is not the place for this is issue (community forum are) and no valid information for reproduction or analysis are provided as requested.


Wrong default pack directory under MacOS

I think that the default for the CMSIS_PACK_ROOT environment variable under MacOS in cpackget is wrong. I have tried to run the gen_pack.sh script on my new MacbookPro. I had not set the CMSIS_PACK_ROOT environment variable. And cpackget tried to search for packs in ~/.cache/arm/Packs while I had already an installation directory under ~/cache/arm/packs (last word with lower case letters at the beginning).
Tested on MacOS 13.4.

@KeilChris in the second path above there is no dot in front of cache. Is that a typo or another difference?
According to this document https://github.com/Open-CMSIS-Pack/cmsis-toolbox/blob/main/docs/installation.md#cmsis_pack_root
the correct default path should be all lower case and with a dot in front of cache.

Gents, I double checked and this was a misalignment between gen-pack library and cpackget. I adopted the path to ~/.cache/arm/packs on both, Mac and Linux, see Open-CMSIS-Pack/gen-pack#37.

After deleting and reinitializing, cpackget now uses ~/.cache/arm/packs.


Bazel doesn't use the optimization flag specified in configuration

Hi
I was compiling TensorFlow on a PowerPC machine and it kept failing with the error "unrecognized command line option -march=native" when compiling the file pcre_byte_order.c. This happened even though I specified the default flag -mcpu=native when running ./configure. I also used the --cxxopt=-mcpu=native flag to no avail, and the only way to solve the problem was to remove the --config=opt flag when invoking bazel.

@case540  is already working on the configuration scripts at the moment.
Could you look into this?

hmm, could you give more specs about your machine. Specifically, what
python -c "import platform; print platform.machine()"
outputs. If the above outputs 'ppc64le', then --config=opt won't add -march=native flag and will instead add -mcpu=native.

Tomorrow I'll run the command and post the output, but the architecture
should be ppc64le (I remember seeing it in the name of the whl file). The
configure script did specify -mcpu=native as the optimization flag for
--config=opt, but -march=native still popped up when compiling one of the
files (and consequently build failed). The problem disappeared when I built
without --config=opt.
Il mer 21 feb 2018, 00:10 Michael Case<EMAIL_ADDRESS>ha
scritto:

hmm, could you give more specs about your machine. Specifically, what
python -c "import platform; print platform.machine()"
outputs. If the above outputs 'ppc64le', then --config=opt won't add
-march=native flag and will instead add -mcpu=native.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/17154#issuecomment-367153066,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGzyMIxvfxL2MW1XRVN2Nn8i31revd6Fks5tW1DegaJpZM4SMRcs
.

We merged a potential fix. Could you test it out when you get the chance?

I tried to compile from master and everything went smooth (just a few warnings)

@eann Thank you very much for verifying!


Please review my resume

GitHub id : mohitrakhade20 :heart:
resume is attached
Mohit's Resume 2021.pdf
Remove all personal data, otherwise the resume will be deleted.
Items we can review on a live stream:

GitHub profile
Open source project
Portfolio
Resume

You can also join the discord community here
Feel free to check out other cool repositories of Eddie Jaoude Community here

I have a couple of tips already -

Schooling is mostly unnecessary, college will do
Display more metrics in each experience
Certifications are not recommended - use that space to show off something you did!
Two column resumes are not ATS friendly - maybe switch to a single column template? Here's a neat ATS parser tool - https://itsjafer.com/#/parser
Remove your photo & DOB, it's mostly unnecessary and sometimes even frowned upon

I have a couple of tips already -

Schooling is mostly unnecessary, college will do
Display more metrics in each experience
Certifications are not recommended - use that space to show off something you did!
Two column resumes are not ATS friendly - maybe switch to a single column template? Here's a neat ATS parser tool - https://itsjafer.com/#/parser
Remove your photo & DOB, it's mostly unnecessary and sometimes even frowned upon

Thank You @sreekaransrinath , what do you think @eddiejaoude on this same points

Thank You @sreekaransrinath , what do you think @eddiejaoude on this same points

Added some points :)

Added some points :)

This are really good ping points you analysied
Thank You @sreekaransrinath
I will work on it
would love connect with you

This are really good ping points you analysied
Thank You @sreekaransrinath
I will work on it
would love connect with you

My suggestion:

Host your resume online, you could do it in your portfolio

Because I don't think everyone wants to download resume specially when you want someone to review your resume.

My suggestion:

Host your resume online, you could do it in your portfolio

Because I don't think everyone wants to download resume specially when you want someone to review your resume.

This are really good ping points you analysied
Thank You @sreekaransrinath
I will work on it
would love connect with you

My social profiles are tagged on my github :)

This are really good ping points you analysied
Thank You @sreekaransrinath
I will work on it
would love connect with you

My social profiles are tagged on my github :)

My suggestion:

Host your resume online, you could do it in your portfolio

Because I don't think everyone wants to download resume specially when you want someone to review your resume.

This is great! You don't even need a website, all you need is an index.html page displaying your resume

My suggestion:

Host your resume online, you could do it in your portfolio

Because I don't think everyone wants to download resume specially when you want someone to review your resume.

This is great! You don't even need a website, all you need is an index.html page displaying your resume

Closing as reviewed on the stream :)

Thanks for submitting @mohitrakhade20 and thank you for logging the live stream link and closing 👍


Fix kubebench-config passing in operator

This fixes the missing config problem in kubebench operator.

This change is

/Assign @Akado2009

/retest

/retest

/lgtm
/approve


Thumbnails not refreshed when using useForCpThumbs due to browser (and Imgix) caching the image

Craft's default behaviour when generating CP thumbnails is to add the asset's last modified timestamp to the URL, so that when you replace an asset (or edit it with the image editor), the thumbnail refreshes, busting the browser's cache. But Imager doesn't do that, so the thumbnail refreshes but with the exact same URL, and the browser loads the outdated cached thumbnail.
That would be easily fixed when using the craft transformer; I think you would only have to replace line 224 of src/Imager.php from
$event->url = $transformedImage->url;

to
$event->url = $transformedImage->url . (strpos($transformedImage->url, '?') === false ? '?' : '&') . 'v=' . $event->asset->dateModified->getTimestamp();

But with the imgix transformer, it wouldn't work for two reasons:

Imgix caches images remotely, so even though they are purged on replace, it doesn't take effect quickly enough for the freshly updated image to be available when the thumbnail refreshes;
Not always the case, but when Imgix URLs are signed with a signKey, adding anything to the URL breaks the signature.

If Imgix accepted some kind of "version" parameter for cache busting (both server-side and client-side) this would be pretty easy to fix, but it doesn't seem to have anything like that, so I imagine it's tricky.

As we talked about on Discord, this is not really possible to fix, given how Imgix an Craft work. Only solution would be to rename image on save, which is something that could be done in a separate plugin, doesn't really fit into Imager.

@aelvan Thank you for your time discussing this. I understand it's not possible to fix it when using Imgix, but maybe when using the craft transformer it could add the asset's last modified timestamp to the URL like Craft does? I don't actually care since I won't be using useForCpThumbs without Imgix, but just a suggestion...


No tests found in your file

./node_modules/.bin/cypress open
in the cypress IDE, I got this error
cypress/integration/search.feature We could not detect any tests in the above file. Write some tests and re-run.

Cypress version:
Preprocessor version:
Node version:

node --version
v16.3.0
"cypress": "^5.6.0",
"cypress-cucumber-preprocessor": "^1.19.2"

here is the project repo
https://github.com/LayMui/shopee-automation-with-cypress/tree/cucumber

my step definition is fixed
When('she search for the item', (dataTable) => {
    const item = dataTable.hashes()[0].item
    console.log('item: ' + item)
    homePage.closePopUp();
    homePage.searchInput('mask')
});

Then('she should able to get the result on that item', () => {
    homePage.searchResultHeader().should('have.text', 'mask');
});

however I got the following error when run in the cypress IDE
I got this error
Error
Step implementation missing for: she search for the item
../../../../../usr/local/Cellar/node/16.3.0/lib/node_modules/cypress-cucumber-preprocessor/lib/resolveStepDefinition.js:199:1


Bollywood.py

A Bollywood game where you have to guess a movie name letter by letter

@sattwik2 are making the changes requested?

@sattwik21 nice, followed guidelines, Reviewed and merged!


Adding a Game

Fixes
#346

Description
A simple and interesting game which will help kids to learn English alphabets and numbers from 0-9.

Type of change

[x] New project (non-breaking change which adds functionality)

How Has This Been Tested?
This has been tested on local pc and local browser.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have included a README.md file
[x] My project follows this structure: Domain Folder/ Project Name Folder/ Project files and Readme.md files
[x] I have added demonstration in the form of GIF/video file
[x] I am a GSSoC'22 Participant

@Rakesh9100
Note : Screen Recorded video should include

code go-through
what are the files you have included
run your project in screen recording

make that these changes

Please Check I have changed the video.


make an intro

I do web development. I study front-end technologies. I am here to get new knowledge


[docs] Animate component's mounting and unmounting

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

Closes #24042
Deploy preview: https://5fde13885f4de40007494b4e--material-ui.netlify.app/components/transitions/#animate-components-mounting-and-unmounting

This is a PR for this issue:
https://github.com/mui-org/material-ui/issues/24042

@cjoecker Thanks for working on it!
I'm not sure why Netlify didn't report back, but here's the deploy preview: https://5fde13885f4de40007494b4e--material-ui.netlify.app/components/transitions/#animate-components-mounting-and-unmounting
Looks good so far. Could it perhaps be simplified a bit by adding items from a list, and removing the text field? On the flip side, I'm wondering if we should perhaps use a list and ListItems for a closer-to-real-world example?

@mbrookes the idea of the List and ListItem is good, will do it!
What do you mean by this?:

by adding items from a list, and removing the text field

@mbrookes Done :)

I have tried to polish the new section:

@cjoecker Thanks!


About the dataset

Could you please provide the "train.npz", "val.npz" and  "test.npz" in the DGCRN model. Thanks

Did you find it? Im also trying to run the code, but most dataset I find somewhere else wont run on this one.


Add distance param

-d, --distance=DIST
Set the distance of thread intervals in microseconds (default
is 500us).
When cyclictest is called with the -t option and more than one
thread is created, then this distance value is added to the
interval of the threads:
  Interval(thread N) = Interval(thread N-1) + DIST

distance=auto
distance = interval / threads

distance does not work w/ histogram
need to convert to int when auto

distance does not work w/ histogram

I'm very confused by this statement.  What exactly do you mean by this?
Histogram is a method of reporting the data collected while the distance is a variable that affects the way the test is run.  Why would one affect the other?  If it is, maybe this is a bug that needs fixing?

@rafaelfolco Do we still need this?  Or should we close this?

@rafaelfolco Do we still need this? Or should we close this?

Let's close/archive this one, I am not sure what was the motivation here.


[13pt] Validation of ABRFC 2015 and 2019 events - review RFC feedback (Vlab #86655)

Review feedback from Eric Jones at ABRFC and check if behavior is consistent in FIM3. https://vlab.ncep.noaa.gov/redmine/issues/86655

Review feedback from Eric Jones at ABRFC and check if behavior is consistent in FIM3. https://vlab.ncep.noaa.gov/redmine/issues/86655

Review feedback from Eric Jones at ABRFC and check if behavior is consistent in FIM3. https://vlab.ncep.noaa.gov/redmine/issues/86655

Web map produced by ABRFC: https://noaa.maps.arcgis.com/apps/View/index.html?webmap=6996fafb78ed430dae9c6324db609b77

Web map produced by ABRFC: https://noaa.maps.arcgis.com/apps/View/index.html?webmap=6996fafb78ed430dae9c6324db609b77

This Vlab ticket is not specific to the HS FIM project. Closing.


Building freestyle IoT application

Hi,
The end of the tutorial does not refer to the next tutorial in the program: https://www.sap.com/developer/tutorials/iotae-api-postman.html

Yes, the postman part is optional.
We will deprecate this tutorial once our free trial is available - hence i will not change this right now.

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


Move Actions-related interfaces into their own packages for clarity

This is a proposal for how we could better separate the actions of the view/model from their implementations' code. In other words, rather than having a GenericStyledArea that is 1000 lines of code long, most of which is filled with properties, javadoc, and other implementation details, making it difficult for new users to figure out how to use the code immediately, we can move that to their own corresponding interface so that it's easier to find all the relevant methods one can call on an area.

StyleableSequence was added so that a future GenericRichArea (non-styled area) could become the new base area more easily by simply removing that interface from the base classes and adding it to subclasses.
The view (GenericStyledArea) could have two interfaces: 1) ViewActions for other view-related actions (e.g. scrolling, showing a specific line, etc.) and 2) ViewNavigation for other navigation that can only be done in the view, not model (e.g. lineStart/End, not paragraphStart/End in #419).
Additionally, many methods currently implemented in StyledTextAreaBehavior (e.g. selectWord, deletePrevWord, deleteNextWord, etc.) could be added to these interfaces so that developers who are implementing their own custom behavior can create them much faster.

Can I get some feedback on this approach? A few questions I have:

Virtualized defines its own estimatedScrollX/YProperty and totalWidth/HeightProperty. I would like these to be in ViewActions for two reasons: first, the getter/setters are not so easily hidden in the midst of the other properties getters/setters because the property stands out, and second, the javadoc for that property will be found in ViewActions, not the area itself, which continues to approach of defining all the view-related javadoc in one interface. If I add those methods in ViewActions, however, then I'm redefining the same method signature. So, I'm not sure whether this is desirable or not. If I don't, then I can only define the javadoc in the actual base area.
I'm not sure what other methods should be in the ViewActions interface, and if any of that interface's methods should actually be in a different interface. On one hand, it would be better if all the view methods were found in the same interface. On another hand, splitting them up by category as the model's actions package does makes it easier to find things.
Which methods in StyledTextAreaBehavior should be added to ViewActions as default methods?

This was accomplished to a lesser degree of detailedness via #461.


Please add steps to disable default profiling for container app service on Azure portal if possible

[Enter feedback here]

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

ID: 31d8edaa-caac-7361-56fb-eb412e2dcc7c
Version Independent ID: f341239f-f871-c37a-412c-ede76279692a
Content: Profile Azure Containers with Application Insights Profiler - Azure Monitor
Content Source: articles/azure-monitor/profiler/profiler-containers.md
Service: azure-monitor
GitHub Login: @hhunter-ms
Microsoft Alias: hannahhunter

@ahmed-shah0906 Thanks for your feedback.
Assigning to content author @hhunter-ms  for review.

Hello @ahmed-shah0906 - You can disable default profiling by turning off the sampling trigger. I hope this helps!

Closing this issue as it's been resolved. Don't hesitate to reach out with a new issue if you have any further questions! #please-close


Can't consume AA.js from nom

Hey, excellent project! I'm trying to integrate in a personal project built on node.js.
I can't do that out of the box because only the src/ folder is distributed via npm (node doesn't support es modules at this time). So I have to build it using rollup:
$ rollup node_modules/aa.js/src/index.js --file aa.js/index.js --format cjs

Then I can import it doing:
const { moon } = require("./aa.js");

It's obviously not ideal. I can see a webpack configuration, but the resulting files should be added to npm and a new version of the package published.

Hi @gmarty I just published a new version (0.10) using a completely new (and a lot simpler) webpack config. A dist minified file is now included in the package. Please, git it a try, and let me know!

That looks great, I didn't have time to work on a pull request. I assume the work was done in the develop branch.
Now, you need to republish a new version of the package using something like npm publish. The latest version you published was 0.3.6 (See your package entry on npm website). Let me know if you need any help!

The development process of that package is a bit messy, I am sorry. The branch develop is just brand new because I wanted to throw some new imported stuff that will come later. Since I wanted to fix that import thing rapidly, I worked directly on master.
Anyway, one of the consequences of that messy and not-really well made process, the package is actually published under another name since quite a while! Here it is:
https://www.npmjs.com/package/astronomical-algorithms
I'll update the Readme and the npm website, sorry!

The README is updated, and the package 'aa.js' is deprecated in the npm registry.


[CARBONDATA-3931]Fix Secondary index with index column as DateType giving wrong results

Why is this PR needed?
On data load to SI with date type, dictionary values is loaded from factToIndexDictColumnMapping instead of getting from main table query results. factToIndexDictColumnMapping will have value as 0, hence only first row data will be loaded
What changes were proposed in this PR?
Get dictionary Keys from wrapper and convert it to Int and load to SI. Removed factToIndexDictColumnMapping.
Does this PR introduce any user interface change?

No

Is any new testcase added?

Yes

Build Failed  with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1806/

Build Failed  with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3546/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1816/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3556/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3675/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1936/

Build Success with Spark 2.3.4, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbonPRBuilder2.3/3695/

Build Success with Spark 2.4.5, Please check CI http://<IP_ADDRESS>:12545/job/ApacheCarbon_PR_Builder_2.4.5/1956/

LGTM


fixes z-index issue

This adds a default z-index of 1 which can be set to any other value by passing a zIndex property.
This can be useful if you are using e.g. the jQuery colorbox plugin or anything that makes the overlay invisible.

The following divagation is just to explain the motivations behind my choices and why I had deliberately chosen not to include that option before.
There has been a similar request (#3) some time ago and honestly I must say I don't really fancy z-index at all. I think it can be avoided in most cases, leading to a more well-structured code.
It's all about DOM manipulation: one shouldn't rely on z-index when the real problem is the hierarchy in the DOM.
That request (#3)  ended up with a manually-set z-index in the CSS, because I thought it was the appropriate place to set it: in the CSS, close to another element the z-index was relating to.
All this made clear, I understand that in some cases, like the one you mentioned (another plugin appendig its own DOM elements and styles), we cannot have full control over the DOM, thus an explicit zIndex property could come in handy.
I will merge your changes now, and in a few days I will release a new version with the minified file and maybe some other minor adjustments myself .
Thanks.

I see. Maybe it would be the better approach to only set the z-index if it was passed as an option? undefined by default. Not sure.
This was just a quick fix that worked out fine for me. I thought it may be helpful for others.
Thanks for merging.

Yep, I will leave it unset if no option is passed, so a value set in an external stylesheet using the .loadingoverlay class won't be overwritten by the default option.
Thanks for using my plugin and taking time to review it, cheers.


Support subscription to registry changes

Closes #8
Motivation and Context
As demonstrated in the dog mode logic application, applications can use the Inspect intent on the registry to adapt their behaviour based on the presence or absence of certain intents for a namespace.
This is more useful if an application gets notifications from the registry that the available intents for a certain namespace changed. If the application receives a notification that one of the relevant namespaces has a different set of intents, the application can re-inspect the registry and adapt its behaviour based on the updated content of the registry.
Description
To support the feature above, the registry passes more detailed information to the observer in form of a sequence of Change<'a>. By introducing another observer, which connects the observed changes to the event sub-system (ESS), subscribers are notified on relevant changes to the registry.
Relevant changes encompass changes that either add or remove intents for a namespace, but not ones that only modify service registrations for an already provided intent.
As per acceptance criteria:

The applications can subscribe to system.registry with keys of the form namespaces[<namespace>] to be notified of changes to a namespace.
The subscription is allowed whether or not the namespace is currently registered in the registry

Different from the acceptance criteria:

The event payload on registry changes is empty. We could list all intents that changed for a namespace together with whether they were added or removed, but the applications are likely going to re-inspect the registry and as such this could be wasted additional complexity.

General comment on the description:

The event payload on registry changes is empty. We could list all intents that changed for a namespace together with whether they were added or removed, but the applications are likely going to re-inspect the registry and as such this could be wasted additional complexity.

I agree not listing the intents makes sense. I assume the source namespace is still communicated (as we register for multiple keys, only one namespace is triggering the change).

General comment on the description:

The event payload on registry changes is empty. We could list all intents that changed for a namespace together with whether they were added or removed, but the applications are likely going to re-inspect the registry and as such this could be wasted additional complexity.

I agree not listing the intents makes sense. I assume the source namespace is still communicated (as we register for multiple keys, only one namespace is triggering the change).

Thanks for your input. This is indeed the case: the namespace is delivered via the source key of the event payload.

General comment on the description:

The event payload on registry changes is empty. We could list all intents that changed for a namespace together with whether they were added or removed, but the applications are likely going to re-inspect the registry and as such this could be wasted additional complexity.

I agree not listing the intents makes sense. I assume the source namespace is still communicated (as we register for multiple keys, only one namespace is triggering the change).

Thanks for your input. This is indeed the case: the namespace is delivered via the source key of the event payload, and will be something like namespaces[sdv.vdt].

We should re-consider the formatting here. I understand that we want to have a way to express the subscription domain (namespaces) and inside the domain the target. Generally, we should try to come up with a concept to handle multiple target domains for subscriptions with a stronger typed fashion. We could add a domain to the subscription interface indicating a sub area for the subscription to target. It would be an optional indicator and the applications would implement a default if it were left out. The event then would indicate the area the event was fired on.


Middle class?

Is the semantic-ui middle class supposed to work?  Because it's not for me.

Hi there, need a bit more context before I can answer this one - which middle class do you mean? Apologies for the slow reply on this.


Governance. Lock. Countdown

Description
There is no countdown for governance lock into the window. In order to update lock timeout need to open that window again.
Steps to Reproduce

For the Lock countdown less than 1 hour open Lock window

Expected Behavior
Into the Lock window is shown countdown with updating data
Actual Behavior
Snap shot of time when open the window
https://github.com/user-attachments/assets/13abe008-e6df-4bc9-bc40-17361b85410e
Environment
https://github.com/novasamatech/nova-spektr/commit/769e2c447f837eea9cdc8d946a1bd416cacb1fc2
Additional info:
When Time for lock is run out it is shown -1 day -1 hour and unlock state have not change

Status verification: ✅


"Titanium Root Certificate Authority" Not trusted in MAC.

Hi,
I am using .net core 2.0 on mac 10.13.6  with the latest beta build of Titanium-Web-Proxy.
Using the same code I have able to add your certificate in Mac keychain it installed but not trusted.
so it is possible to trust that certificate using code.
As of now used below code:-
 var proxyServer = new ProxyServer();
 //locally trust root certificate used by this proxy
 proxyServer.CertificateManager.TrustRootCertificate = true;

For more details I have attached screenshot:-

Please help me for the same...

I've done a little research on this myself. As far as I know, there's only two ways to programmatically trust a certificate with macOS.

Via commandline - see here

security add-trusted-cert -d -r trustRoot -k /Library/Keychains/System.keychain <certificate>

That seems to be the easiest way. If you don't want a password prompt coming up for that, you'll have to make sure that your app is running with elevated privileges (i.e. sudo)

This is a lot more complicated and requires P/Invoke + a custom objective-c library. I personally have not figured out this library. Apple Security

I tried using your command,
This command requires  .cer file only and we have rootCert.pfx file.
So we converted .pfx file to .cer file by using below command:
openssl pkcs12 -in rootCert.pfx  -nokeys -out  rootCert.cer
After using the below command to install and trust same certificate:
security add-trusted-cert -d -r trustRoot -k /Library/Keychains/System.keychain rootCert.cer.
Certificate added and trusted successfully but still problem with titanium after proxy start it says that 'Certificate Invalid on browser page'.
So please help me add certificate at the time of proxy start using .netcore .

Could you please try to xprot the cer file from titanium with the following code and try that on MAC?
 File.WriteAllBytes("rootCert.cer", proxyServer.CertificateManager.RootCertificate.Export(X509ContentType.Cert));

I close this isse, feel free to reopen if needed.


makes this project available at jupyter.sns.gov

https://jupyter.sns.gov/user/{UID}/notebooks/notebooks/CG1DImaging/VENUS-chopper-design/Time_Distance_Diagram.ipynb


Add a way to delete an entry from the index and archive

Occasionally I want to remove a URL from my archive. Currently this is a manual process of finding the entry in index.json, pulling out the timestamp, deleting the relevant lines, doing the same for index.html, and finally rm -r output/archive/$timestamp.
It would be nice if there was some slightly more automated way of doing this. Ideally I think this would be done with a final step after archiving, where the script would try to match each directory name in output/ with a timestamp in index.json. If a match isn't found, the user is prompted with something like:
1536723384 not found in bookmark index. Delete output directory? (y/n)

This may be a behavior that is only enabled by an optional config option.

:+1: Good idea

I cant promise I'll get around to this soon since I'm super busy with my day-job, but it's on the TODO list!

I've recently imported my complete Pinboard archive, there where a lot of bookmarks with dead links in it:
[√] [2018-10-17 23:28:37] Update of 4249 links complete (133.12 min)
    - 15219 entries skipped
    - 714 entries updated
    - 1063 errors

(The script crashed a few times with "Too many open files" errors, so I had to rerun it a couple of times)
My idea is to run this script once a day with a fresh dump from my pinboard export (I've wrote a little go program which dumps the whole list from pinboard). But with that 1063 links with errors, it will take hours (even with small timeouts) and is totally useless to retry that links.
Because that 1063 dead links will always be in that exported list, the archiver will always retry to download it. It would be nice if there where a flag or environment variable to skip that links which where previously failed to download. A "cleanup" flag would be even better, but skipping that links would be sufficient for my usecase.

How about an --only-new flag which makes it only add new links to the index, without attempting to update/retry older archived links?

Sounds good to me!

I've opened a pull request for that.

While I like the ONLY_NEW addition, I don't think it closes this issue. Eventually I'd still like to see some cleaner method for removing bookmarks, even if their content has already been downloaded.
One of my use cases is similar to the one @f0086 mentioned: I've imported a Pinboard json file which includes old URLs that now 404. It's annoying to have bookmark-archiver waste time on every run attempting to archive them. However, I don't want to enable  ONLY_NEW. If I do that, and I add a good URL, but my machine goes offline during the next run (or the machine serving the URL happens to be offline, or if they happen to be restarting their web server at that second) the archive will fail and it will not try again next time, despite there being a good chance the next attempt would succeed. This use case would be better solved by tracking the number of attempts per URL and having some sort of MAX_URL_ATTEMPTS configuration option. If this was set to 1, it would basically act exactly the same as ONLY_NEW: try once and ignore forever on failure. To address my concerns about temporary connectivity issues, I could set it to a value like 5.
My other use case is that I have successfully imported a URL and archived it, but it no longer provides any value to me and I want to recover the disk space. In this case neither ONLY_NEW or MAX_URL_ATTEMPTS helps me. This may also be useful if you are serving your archive publicly and receive a take-down request that you wish to comply with.
I do think this is a low priority issue, since bookmark-archiver's focus is sort of the opposite of deleting things.

Agreed, I didn't mean to close this issue by merging that PR, Github did it automatically.
I like the idea for MAX_URL_ATTEMPTS, that wouldn't be too hard to track in our index.json files.

@pigmonkey I've also thought about the fact that a page could potentially went offline at the time I scrape it. But for that case, you can simply run the archiver with ONLY_NEW=False once a day or so. Most of the time, a page went offline either forever or for a short time (hours/days). So, If I set MAX_URL_ATTEMPTS to 10 and run that script every hour, I have the same problem as with ONLY_NEW for a page which is offline for a day or two. I set up two cronjobs: Once a week with ONLY_NEW=False and once a hour with ONLY_NEW=True.

The new django version has both the ability to remove snapshots from the archive, and a separate archivebox update command independent from archivebox add so that you can control when to retry previously failed links.
git checkout django
git pull
docker build . -t archivebox
docker run -v $PWD/output:/data archivebox init
docker run -v $PWD/output:/data archivebox add 'https://example.com'
docker run -v $PWD/output:/data archivebox remove --delete 'https://example.com'
docker run -v $PWD/output:/data archivebox update

Adding a MAX_URL_ATTEMPTS option will be tracked in this separate issue: https://github.com/pirate/ArchiveBox/issues/109


Vickel: Watch dognpet.com

Vickel requests the watch of the watch_keyword dognpet\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

dognpet\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


fix: bump lavaplayer

There don't seem to be any breaking change for Lavalink in the versions in between, however there are some important fixes. In particular in 1.3.43+1.3.44 Soundcloud fixes, 1.3.45 and 1.3.47 YouTube live streams, 1.3.46 Youtube mixes, and in 1.3.49 Twitch playback.

This is already on dev. Use that branch instead.

Oh woops I didn't check that branch. Closing then.

I will draft a release of the dev version whenever I get around to it.


Add ponyfill export

This PR introduces a new exported path: import 'urlpattern-polyfill/pony' that will not pollute the global scope and will always provide the polyfill implementation.
Notes:

Importing urlpattern-polyfill will pollute the global scope both in runtime and in TypeScript
Importing urlpattern-polyfill/pony will not pollute the global scope at all

Related:

This supersedes #91

Not sure about the name though, would be best if we had urlpattern/ponyfill and urlpattern/polyfill 😅

Yeah, I am not a fan of the pony name :) Wouldn't it be nicer we we could just import from "urlpattern-polyfill/urlpattern" as that is the name of the object

Generally this looks fine to me, but I would like Sander to have a look as well

I do like it!


Import spreadsheet of TA applicants

As an NUS professor, I can import a spreadsheet of TA applicants to my module, so that I don't have to add them individually.

Similar to #102.


[API Docs] Fails to generate a table with proper indentation

Description:

$Title on the website.

The Stdlib module page, which shows module.md and other functions altogether do not seem to indent properly (learn/api-docs/ballerina/java/index.html)

The page which directs when you click on the function shows it properly (learn/api-docs/ballerina/java/functions.html)

Tabel in the function description
https://github.com/ballerina-platform/ballerina-lang/pull/22556/files#diff-98c8c70e07faba09acc1dd804da4b3f1R62
Affected Versions:
v1.2.0

Closing as this is a duplicate of https://github.com/ballerina-platform/ballerina-lang/issues/22789.


If the date has an initial answer, then opening the widget selects todays date as default instead of the initial date

Describe the bug
If the date has an initial answer, then opening the widget selects today's date as default instead of the initial date
Component
SDC library
To Reproduce
Steps to reproduce the behavior:

Use sample questionnaire with a date type item with an initial value
Press the date widget
Selected date will be "today" instead of the initial value.

Expected behavior
If an answer already exists then that date should be the default value selected in the calendar.
Screenshots
video
Smartphone (please complete the following information):

Device: One plus 7t pro
Android version: 11
Build number: Running Lineage OS 18.1

ETA 19th August 2021


Standardised Images

Added new images and standardised images so that the console in all example (minus the .gif) have the same colour scheme.

@gabru-md I can change the urls to have yogdaan in instead of yourself if you like? I think the current urls in the README have your name though?


Licensing

What is the license of the OpenAPI Description files in this repository?
Please could you license them under a free license?

Hi @jcassette, project is kind of abandoned, but sure, I will add license file.


Add endpoint to update owners of a request

At the moment it appears to require the deletion of a Request to change its owners. Since a running Request might change ownership over time, without needing a redeploy, it seems untoward to require downtime for such a change.
What I'd like to propose (in advance of submitting a PR for) is the addition of
PUT /api/requests/request/{requestId}/owners to replace the existing owner list for a Request.

@nyarly You can POST new request data to the /api/requests endpoint (e.g. same data except for updated owners). This does not initiate a redeploy and will update the request data. You can also go to the /requests/edit/{requestId} path in the ui (or get there via the edit link on the request page) and change the owners of the request there.

Okay, thanks.

Quick aside, then: why are there scale and skipHealthchecks endpoints then?

Those were meant as convenience endpoints for easier use by the UI (i.e. not needing the whole request json), but also ended up taking on some other functionality when we added expiring actions for skipping heathchecks and scaling


Add "Java Date and Time" in JavaHackerrank Section

Add a perfect solution to the given challenge.
Refer:-https://www.hackerrank.com/domains/java

Ill work on this!


Entries are now sorted by default

@sir-argo You need to include previous versions in your script.json for api scripts so that versioning and upgrading works correctly.  I've made this change for you here, but something to keep in mind in the future.

Yep.  I was looking over it and saw that.  I apologize, this is my first api, so I’m learning.  I really did try…
From: Jeffrey Lamb<EMAIL_ADDRESS>Sent: Tuesday, May 1, 2018 7:06 AM
To: Roll20/roll20-api-scripts
Cc: Darren; Mention
Subject: Re: [Roll20/roll20-api-scripts] Entries are now sorted by default (#619)
@sir-argohttps://github.com/sir-argo You need to include previous versions in your script.json for api scripts so that versioning and upgrading works correctly. I've made this change for you here, but something to keep in mind in the future.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Roll20/roll20-api-scripts/pull/619#issuecomment-385678734, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AkjcfpWwfm1MttwT3g687Ci7g1BC8lvHks5tuGvVgaJpZM4TtPnw.


Issue with version 0.11.0-beta-3

How to reproduce

Deploy Spring application to Zalando Platform without token credentials.
Observe error

ERROR 11 --- [           main] o.s.boot.SpringApplication               : Application startup failed

org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'secrets' defined in class path resource [org/zalando/stups/tokens/config/SecretsAutoConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.zalando.stups.tokens.Secrets]: Factory method 'secrets' threw exception; nested exception is java.lang.ClassCastException: org.zalando.stups.tokens.AccessTokenRefresher cannot be cast to org.zalando.stups.tokens.Secrets

Explanation
The problem comes from dependency tokens-0.11.0-beta-2. It starts here: org/zalando/stups/tokens/AccessTokensBean.java:171 when accessTokensDelegate is created
accessTokensDelegate = builder.start();

The builder returns implementation of org.zalando.stups.tokens.AccessTokens interface which is fine. Internally it looks for a file with -token-secret suffix (check org.zalando.stups.tokens.AccessTokensBuilder#isFilesystemSecretsLayout) and if such file was not found, it creates an instance of org.zalando.stups.tokens.AccessTokenRefresher. If the file was found, it creates instance of org.zalando.stups.tokens.fs.FilesystemSecretRefresher.
The problem happens when org.zalando.stups.tokens.config.SecretsAutoConfiguration creates a bean with either AccessTokenRefresher or FilesystemSecretRefresher. It happens because  AccessTokenRefresher DOES NOT implement Secrets interface, so the type cast fails in method
    @Bean
    public Secrets secrets(AccessTokensBean accessTokensBean) {
        return new SecretsAdapter((Secrets) accessTokensBean.getDelegate());
    }

Possible solution
Here are possible solutions we should discuss:

Make org.zalando.stups.tokens.AccessTokenRefresher implement Secrets interface
If a file with -token-secret suffix is mandatory, then throw an exception with meaningful message in case if it's not found.

Hi, thanks for reporting. I'm using this version myself and did not struggle with this issue.
But I will have a look into this over the next few days and try to reproduce it.


security fixes

Addresses various security issues found out by automated scans.

Thanks; I’ll run another integration test on the build server and do a minor release… hopefully tonight.


[BUG] Type references are not assignable with primitive types in typescript

Is there an existing issue for this?

[X] I have searched the existing issues

Nature of Your Project
TypeScript
Current Behavior
When using a type reference https://cap.cloud.sap/docs/cds/cdl#typereferences to re-use the type of a element of a entity, e.g. String, the generated property in TypeScript can not be assigned the respective primitive, e.g. string.

Expected Behavior
No response
Steps To Reproduce

    entity Foo {
        key name    : String(10);
        key counter : Integer;
    }

    event FooCreated {
        // re-use types
        fooName    : Foo:name;
        fooCounter : Foo:counter;
    }

    const eventParams: BooksServiceTypes.FooCreated = {
      // error
      fooName: "",
      // error
      fooCounter: 1,
    };

Environment
- **cds-typer**: 0.30.0
- **cds**: 8.5.0

Repository Containing a Minimal Reproducible Example
No response
Anything else?
No response

Hi Andreas,
thank you for your report!
This only happens when the referred property in question is a key. When you remove key from name and regenerate the model, the problem goes away.
Do you actually want to manually assign values to a field that is marked as a key?
Best,
Daniel

Hi Daniel,
Yes, I do want to re-use the type of an key. Consider the following example:
entity Orders : managed {
          @assert.format: '^\d+$'
      key ID              : String(10);
      // ...
}

event OrderCanceled {
    orderID  : Orders:ID;
   // ...
}

// more events

// returns created orderIDs
action createOrder(partialOrder : PartialOrder)                                      returns array of Orders:ID;

Admittedly, I could also define the ID type explicitly, like
type OrderID : String(10);

and re-use it. However, the type reference approach does not require an extra type.

Hi Andreas,
just to be clear: reusing the type already works as of today, as long as you only read them:
this.on('READ', FooCreated, req => {
  console.log(req.data.fooName)
})

it's the assignment of a new value to this key property that throws the type system off. Manually assigning something to a key, which I generally assume to be a primary key, therefore runs at risk of violating uniqueness. So therefore my question: are you sure you need to have a key field you can write to manually? Do you expect fooName to be a key string, or just a string?
Best,
Daniel

Ah, I see the problem. There are actually two use cases/problems.

are you sure you need to have a key field you can write to manually?

Yes, consider the example above. The ID of Orders is not auto-generated, as we want an incrementing ID and we have to use a string instead of int due to @cap-js/change-tracking only working with strings. Thus, when creating an order, the ID field needs to be assigned manually:
const order : OrdersService.Order = {
  // error
  ID : "0",
  // ...
}
await INSERT.into(Orders).entries(order);

For non-entity types, e.g. events, actions and functions, there are no key fields. In those cases, it's not relevant that the referenced type is a key, is it? Thus, there the primitive type is relevant. Consider the event example:
action cancelOrder(ID : Orders:ID);

event OrderCanceled {
    orderID  : Orders:ID;
   // ...
}

this.on("cancelOrder", req => {
  const order = await SELECT.one.from(Orders).where({ID : req.data.ID});
  // cancel order
  const eventParams : OrdersService.OrderCanceled = {
   // error
    orderID : order.ID
  }
  await this.emit("OrderCanceled", eventParams);
});

Hi Andreas,
thanks for elaborating! We can probably make this work, but I'd like to check back with the team first:
@David-Kunz could you please let me know whether in the following case OrderCanceled.orderID is supposed to be treated as key & String(10), or just as String(10)? I.e.: is key a strict part of the type and therefore propagated via type references, or is it just a local annotation?
entity Orders : managed {
  key ID: String(10);
}

event OrderCanceled {
  orderID: Orders:ID;
}

Best,
Daniel

Hi Daniel,
I think we can fix the DeepRequired type so the behavior is as expected.
type RemoveKeyType<T> = T extends Key<infer U> ? U : T;

export type DeepRequired<T> = {
    [K in keyof T]: DeepRequired<RemoveKeyType<T[K]>>
} & Exclude<Required<T>, null>;

export declare class BookCreated {
  declare bookId: __.DeepRequired<Book>['ID'] | null;
}

the key part is still included, but it works. I have not tested if this could break something different where DeepRequired is used, but it looks promising.
Regards,
Ludwig

Hi Ludwig,
thanks for weighing in! That might be a possible approach. At the very least this can be remedied where the type reference is actually used:
type Key<T> = T & { key: true }
type Unkey<T> = T extends Key<infer U> ? U : T

class Foo {
    declare prop: Key<string>
}

class Bar {
    declare prop: Unkey<Foo['prop']>
}

new Bar().prop = '42'

which would not interfere with the key type elsewhere.
Bottom line is: we can certainly handle this. Just making sure this is actually supposed to be this way first. 😃
Best,
Daniel

Key properties are not inherited:
entity Orders : cuid {}
entity Foo {  bar: Orders:ID; }

console.log(cds.model.definitions['Orders'].elements.ID.key) // true
console.log(cds.model.definitions['Foo'].elements.bar.key)   // false

Thank you, David!
@aschmidt93 I have prepared a fix to address this.
Feel free to try it out before I merge and release it.
Best,
Daniel

Hi Daniel,
wrapping Unkey around DeepRequired works fine for non key fields, but not for key field references

Regards,
Ludwig

Hi Ludwig,
thanks for trying out the fix! The reason I'd prefer to "unkey" the properties there is that DeepRequired is just a verbatim copy of how it is defined in cds-types (as you are likely aware).
At some point(TM) instead of duplicating that type (and others), we will just directly import it from cds-types. I have not investigated what the implications on cds-types would be, when we remove the key inside that deep-requiring mechanism. I'd have to check that out first.
Best,
Daniel

Change included, can be tried out now

Looks good 👌

Hi Daniel,
the fix seems to do the trick, thank you :)

Hi Andreas,
thanks for confirming, and also thanks Ludwig, for your suggestion and testing out the fix as well. 👍
Best,
Daniel


Convert to arrow function

Converting a few traditional functions to ES6 arrow syntax. It's a pure refactoring, does not affect behavior.
Feel free to squash the commits.

hm, I'll have a look at this

yep, it doesn't like the syntax. not sure it's worth integrating the change, but thanks for responding quickly to this.

there's one failure left under "Allowed Failures". it doesn't seem related to these changes.

the change is tiny now. feel free to merge with squash if it's in any way useful.


[flash_ctrl] oob error for info partitions

@a-will discovered that the flash out-of-bound error only applies to the data partition.
It really should apply to whatever partition we are currently accessing.
This needs to be fixed.
@nikola-miladinovic FYI

We then kept reading and found that the info-partition-specific OOB handling shows up as a transaction error in flash_mp.sv. The big block-level wrap is covered by a separate OOB check in flash_ctrl.sv.
With the code in flash_mp.sv, both types of OOB are covered. @tjaychen may wish to remove the block-level OOB check and truncate FLASH_CTRL.ADDR, though, to harmonize the address handling for the two interfaces (host and flash controller).


Drop down list of commands

added a drop down list that the user can select from
auto-completion not yet added

Coverage decreased (-0.05%) to 92.235% when pulling 99768343e4586d2fedd02e44ae609f26568d27a7 on mingruiii:auto-complete into 6e90c5734180e7ade4ce208f2fb39024e759baee on CS2103AUG2017-W11-B2:master.

Try to make a test that contains an error to trigger /src/main/java/seedu/address/ui/CommandBox.java line 155. Otherwise the coveralls test coverage won't pass.

Can we close this PR?


[Bug]: Documentation Error

Contact Details
<EMAIL_ADDRESS>What happened?
On the Notes page in the Documentation at https://rafaelgb.github.io/obsidian-db-folder/features/Notes/ for the section Deleting and Renaming Notes, it accidentally says to select the number on the right to rename or delete a note.  This should be on the left according the video.  I highlighted it with bold.
"Deleting & Renaming Notes
You can right-click on the numbers to the right of every row, to bring up a menu where you can delete or rename the note associated with the row"
What platform were you using?
Desktop
Version of the plugin
Version irrelevant, documentation.
Relevant log output
No response
Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

yeah, thanks for the report. This event was changed with 3.0.0 but the docu was not updated


Windows 10 MakeCode BETA - not canceling download

This is using the Windows 10 MakeCode BETA app

Create a project
Do not give it a name,, leave it as untitled
Select the save icon next to the project name
In the window "Rename your project" select Cancel

Result: The Rename your project window closes as expected.
BUG: The Save As window appears
Expected result: I expect when I press Cancel that saving the project will cancel.

@Lorrainbow please open micro:bit related bugs in https://github.com/Microsoft/pxt-microbit . Thanks for reporting this issue!

This has been reported on pxt-microbit so can be closed on pxt https://github.com/Microsoft/pxt-microbit/issues/625

Closing


[BUG] Set-PnPPageTextPart and Set-PnPPageWebPart cmdlets are appending extra banner section along with existing banner in the updated page after updating the new text in the Text OOB component as well as in other Components

Reporting an Issue
While setting new text value to the Text Out of the box webpart(using Set-PnPPageTextPart cmdlet) and also while updating the webpart properties back to other oob/custom webpart (using Set-PnPPageWebPart cmdlet) programmatically through powershell, Additional banner is getting appended along with the existing banner in the updated page.
Expected behavior
Set-PnPPageTextPart and Set-PnPPageWebPart cmdlets should set the new data only on the text out of the box webpart and custom/oob webpart respectively. It should not append any new banner in the updated page after setting the text/webpart properties.
Actual behavior
Set-PnPPageTextPart and Set-PnPPageWebPart cmdlets are appending new banner after setting the new text or webpart properties for text oob/custom webparts which is weird and strange.
Ideally It should not have duplicate banner in the updated page after setting new text content in webpart.
It should update only the new text content in the oob/custom webpart.
Steps to reproduce behavior
#Note: Please create a new Modern Page and add text OOB component and add some content in it and publish the page.
Then utilize the same page name below. Please use other parameters like credentials if you are unable to connect to Sharepoint Online.
Code Snippet
$siteUrl = ""
$appId = ""
$appSecret = ""
Connect to SharePoint Online
$global:pnpConnection = Connect-PnPOnline -Url "$($siteUrl)" -ClientId $appId -ClientSecret $appSecret -WarningAction Ignore ReturnConnection
$pageName = "<Any new page where a simple Text OOB webpart is added in a new section after the default banner section , Eg : Page-Text-Webpart.aspx>"
try {
#Main Function
$page = Get-PnPPage -Identity "$($pageName)" -Connection $global:pnpConnection

foreach ($control in $page.Controls) {

    try {

        #region Set Webpart Properties according to the language

        $updatedText = "Iam fine"
        #$updatedText = "How are you?"
        #$updatedText = "Hope your family is well"
        # ConvertTo-Html -InputObject $control | Out-Null
        # Write-Host $($control.ControlType)
        if($control.Type.ToString().IndexOf("PageText") -ge 0){
            $control.ToHtml($control.ControlType) | Out-Null
            Set-PnPPageTextPart -Page $pageName -InstanceId $control.InstanceId -Text $updatedText -Connection $global:pnpConnection
        }
        elseif ($control.WebPartId -eq "<AnyGuid>"){
            $control.ToHtml($control.ControlType) | Out-Null
            Set-PnPPageWebPart -Page $pageName -Identity $control.InstanceId -PropertiesJson $control.JsonWebPartData -Connection $global:pnpConnection
        }

    }
    catch {
        Write-Host $_
        $pageConversionLogCount.Status = "Error"
    }
}

}
catch {
Write-Host "[ERROR]: $_ at $($.InvocationInfo.ScriptLineNumber) LineNumber In Main Function (Triggering Point)"
# log -content "[ERROR]: $ at $($_.InvocationInfo.ScriptLineNumber) LineNumber In Main Function (Triggering Point)" -path $logParentFilePath
$parentLogCount.Status = "Error"
}
$global:pnpConnection = $null
Page before updating new text in Text OOB webpart :

Page after updating new text in Text OOB webpart :

What is the version of the Cmdlet module you are running?
ModuleType Version  Pre-release            Name                               PSEdition
Manifest        2.1.1                                 PnP.PowerShell                      Desk
Which operating system/environment are you running PnP PowerShell on?

[ ] Windows

##Powershell Version - 7.4.5
##node version - v16.17.1
##npm version - 8.15.0
Please suggest a resolution for this issue.
Note : I'm able to reproduce the same issue by using latest PnP Version - 2.12.0, Kindly check and update.
Thank you,
Sreekrishna Sai

I am facing the same issue. For me, it happens when the Page has a Vertical Section. If I remove this Vertical section, it does not add the duplicate Page Title.

Hi Aakash,
Thanks for the acknowledgement, Even i'm also able to reproduce this issue only in the vertical section.
But after creating a new copy page of this existing page for translating language and try to set newly translated text back to webpart, I'm getting below error :
"You cannot host text controls inside a one column full width section, only an image web part or hero web part are allowed".
As part of latest developments, Microsoft is allowing us to delete the default full width section with banner and also it is allowing us to add banner webpart inside Full width section.
Please Correct me if i'm wrong.
Also It is giving above error for all webparts including OOB ones in the page when we use cmdlets with set verb. Thus i'm unable to update newly translated text in the newly created copy pages.
This is the API i'm using to create page : $url = "$($siteUrl)/_api/sitepages/pages($pageId)/translations/create"
$response = Invoke-PnPSPRestMethod -Url $url -Method Post -Content $body -ContentType "application/json;odata=verbose" -Raw -Connection $global:pnpConnection | ConvertFrom-Json
Can anyone from PnP Team check this issue and resolve asap.

I am facing the same issue. For me, it happens when the Page has a Vertical Section. If I remove this Vertical section, it does not add the duplicate Page Title.

Hi Aakash, Could you please check once again as issue is again reproducing, Thanks

new version of the PowerShell
2.12.0 has fixed the issue.

We already tried V2.12.0 earlier but still the same. Following are some screenshots of the same,

Hello, can you please try again with latest nightly builds and let us know ? Looks like it has been fixed .

I tried with 2.12.19-nightly but still the same.

I tried even with 2.12.20-nightly but still facing the issue.

@gautamdsheth It seems to me like the issue is connected with the recent changes by Microsoft related to Page content handling. Could it be that the CSOM library does not support the latest M365 version yet? Is there any solution or workaround that you can suggest?

I'm seeing this issue with pages with vertical sections when I just update/publish them, without adding or modifying the page itself. The page will have it's banner-section "reset" and an extra banner web part is added.
Repro with 2.12.0:
$CurrentItem = Get-PnPListItem -List SitePages -Id 69
Set-PnPListItem -List SitePages -Identity $CurrentItem.Id -Values @{"Modified" = $NowValue} -UpdateType UpdateOverwriteVersion
Set-PnPPage -Identity $CurrentItem.FieldValues.FileLeafRef -Publish

This is how it looks after:

I ran into this when creating automatic updating of page with a script based of PNP Powershell. I don't think the bug is in PNP Powershell, but when working with the page, like replacing a webpart, an additional header is inserted into "LayoutWebpartsContent" field. It looks like it is related to a vertical section on the page.
You can reset the "LayoutWebpartsContent" with the following script:
Set-PnPListItem -List "Site Pages" -Identity [insert here id of page item] -Values @{ LayoutWebpartsContent = "

" } -UpdateType SystemUpdate

You can reset the "LayoutWebpartsContent" with the following script:
Set-PnPListItem -List "Site Pages" -Identity [insert here id of page item] -Values @{ LayoutWebpartsContent = "

" } -UpdateType SystemUpdate

This works great, thank you!

Duplicate banner issue on pages with a vertical section (with PnP 2.12.0)
Bug Description
This issue occurs when creating a page with a banner (full-width section) and a vertical section. After saving the page using Get-PnPPage, the banner is duplicated. Subsequent attempts to fix the issue temporarily (e.g., using Set-PnPListItem) result in a loss of the full-width format for the banner, and the issue reoccurs after each update.
Steps to Reproduce

Create a page with the following structure:

A banner in a full-width section.
A vertical section.

Save any update on this page using the Get-PnPPage command.
Navigate to the page in a browser and observe that the banner is duplicated.
Apply the workaround:

Use the command Set-PnPListItem -Identity <PageId> -Values @{"LayoutWebpartsContent"="

"}.

Navigate to the page again:

The duplicate banner is resolved, but the full-width format for the banner is lost.

Save the page again using Get-PnPPage.
Navigate to the page and observe that the duplicate banner issue reoccurs.

Expected Behavior

The page should maintain the full-width section layout for the banner.
The banner should not be duplicated after saving or updating the page.

Actual Behavior

The banner is duplicated after the first save using Get-PnPPage.
Applying the workaround removes the duplicate banner but breaks the full-width layout.
The issue reoccurs after each save or update.

Screenshots

Additional Context
This issue makes it difficult to maintain the desired layout when pages include both a full-width section and a vertical section.


Add verified for STOS

Description

Checklist

Upgrading Asset to Verified

If upgrading an Asset to Verified, please see the requirements specified at LISTING, and ensure the following:

[x] Asset is defined thoroughly at the Cosmos Chain Registry.

[x] A meaningful description (and extended_description, unless: meme, variant, or non-primary chain token).
[x] Associated socials, including website and twitter (unless: meme, variant, or non-primary chain token).
[x] Logo Image has a square Aspect Ratio, < 250 KB file size, and appropriate visual contrast with Osmosis Zone colors.

[x]  Liquidity: This pool contains contains at least $1k USD-worth of liquidity of the asset (Provide Pool ID): ______ (Skip if alloy constituent)

'Verified' Status Validation Checklist (to be completed by Osmosis Zone maintainers):

[ ] Verify appearance and metadata
[x] Accurate Price
[x] Trading and routing functionality
[ ] Withdraw and Deposit

[ ] Deposit Transaction URL (if in-app)

https://app.osmosis.zone/pool/2216
https://app.osmosis.zone/pool/2196
~28k$

Hi! I have a question. We could not find a token in the "Trade" section, however we could find it into a pool. When I go to https://app.osmosis.zone/pool/2216 and click on "Trade Pair" I see a following errors (in screenshots).
denom is not a valid chain denom (ibc/ABD49F44559C…57CC458459CB6A67CEBD66E23C7674A0B2B445230BDA1F6C)
Should we register it somewhere else or is it something that asset not verified?

The error arises because it is a preview asset (which are not routed), although we are working on changing the behaviour so that even preview assets are tradable.

Hi! I have a question. We could not find a token in the "Trade" section, however we could find it into a pool. When I go to https://app.osmosis.zone/pool/2216 and click on "Trade Pair" I see a following errors (in screenshots).
denom is not a valid chain denom (ibc/ABD49F44559C…57CC458459CB6A67CEBD66E23C7674A0B2B445230BDA1F6C)
Should we register it somewhere else or is it something that asset not verified?

The error arises because it is a preview asset (which are not routed), although we are working on changing the behaviour so that even preview assets are tradable.

I do not see an extended_description for the asset at the Chain Registry.

@JeremyParish69 PR for socials, description + extended_description https://github.com/cosmos/chain-registry/pull/5459

Hmm, Deposits seem to fail

pubkey doesn't match signer address...

@JeremyParish69 which wallet are you using? We have integration with keplr for signing (pub key type):
https://github.com/chainapsis/keplr-wallet/pull/1105/files

@JeremyParish69 I just did a quick test, if I make the ibc transfer through https://app.osmosis.zone/portfolio?transferDirection=deposit&transferAsset=STOS, it fails. but if I directly make a ibc transfer on keplr wallet, it works fine. the signing msg the keplr wallet receives is also mostly identical.
`
// from osmosis app
{
"account_number": "4062",
"chain_id": "stratos-1",
"fee": {
"gas": "135297",
"amount": [
{
"amount": "216475200000000",
"denom": "wei"
}
]
},
"memo": "FE",
"msgs": [
{
"type": "cosmos-sdk/MsgTransfer",
"value": {
"receiver": "osmo1whnzllzrsqkkn658czztpd8n6j4grl47ymgkqq",
"sender": "st1tfqhz86fuhfuh9psdqsetl86evr90rqy6f739x",
"source_channel": "channel-1",
"source_port": "transfer",
"timeout_height": {
"revision_height": "22465310",
"revision_number": "1"
},
"token": {
"amount": "1545792644216376092",
"denom": "wei"
}
}
}
],
"sequence": "2",
"timeout_height": "5571823"
}
// from keplr directly
{
"account_number": "4062",
"chain_id": "stratos-1",
"fee": {
"gas": "140248",
"amount": [
{
"amount": "168297600000000",
"denom": "wei"
}
]
},
"memo": "",
"msgs": [
{
"type": "cosmos-sdk/MsgTransfer",
"value": {
"receiver": "osmo1whnzllzrsqkkn658czztpd8n6j4grl47ymgkqq",
"sender": "st1tfqhz86fuhfuh9psdqsetl86evr90rqy6f739x",
"source_channel": "channel-1",
"source_port": "transfer",
"timeout_height": {
"revision_height": "22465418",
"revision_number": "1"
},
"token": {
"amount": "600000000000000000",
"denom": "wei"
}
}
}
],
"sequence": "2"
}
`

@JeremyParish69 PR here to resolve pub key issue:
https://github.com/osmosis-labs/osmosis-frontend/pull/3896

@JeremyParish69 PR here to resolve pub key issue: osmosis-labs/osmosis-frontend#3896

Thanks for this. I think that will help provide context for getting the issue resolved.

Should work right now @JeremyParish69


GoTestFunc got error

run GoTestFunc in my test function and got an error
E488: 多余的尾部字符: :Gmake -run ..

Is there some dependence I missed ?

My bad.
There were some missing pieces.
I fixed that and also made some improvements. Thanks

@ray-x when I run :GoTestFunc I got an error Unable to find any nodes. Is your query correct?.
In my understanding, I need to point the cursor to the test function and run the above command. Any idea?

You should be able to test current function if the cursor is inside the test function scope. (anywhere from function name to end of the function


Component details disappear after save of new component

Steps to recreate:

Go to the components screen
Click create new component
Enter a name (and anything else you need)
click save
Application saves and the details disappear.
Go back to search screen and notice the details of the component are still there (saved)
Go into the component details and notice that the component details are still missing

This makes it really hard to modify components, it is easier to delete and recreate which will be a problem as amount of data in the system grows
Browser: Chrome 89.0.4389
Ortelius running in the latest image downloaded from Quay
Before save:

After Save:

Search Screen after save:

Not able to recreate this.  We will need to do a web meeting to debug.


Consider using semver 2.0 for versioning

this is a floow up from the issue #527
I use the assembly in a msi installer. The msi installer is doing major updates and for this it checks the version of the dlls to decide if an update is required. the msi installer is using the first three numbers of the version. see https://docs.microsoft.com/en-us/windows/win32/msi/productversion for reference.
So it would be great if you switch to the MAJOR.MINOR.PATCH versioning.

We are using Semver 2 versioning. I am not sure what you want us to change?

Yes, you use semver for the nuget package version, but not for the assembly version / product version.
Because this can cause some issues, lets say you publish <IP_ADDRESS> and later <IP_ADDRESS>, there is no way to update this with msi.

Yeah, exactly. The nuget version is set automatically on build, but the Assembly/FileVersion were manually added, but I will make sure it's set in the CI for the next release.

Thanks. You can close this issue. 🙂

fyi - you can adapt REINSTALLMODE like so https://github.com/icsharpcode/SharpZipLib/issues/527#issuecomment-709870841 to force an overwrite regardless of the version installed.

fyi - you can adapt REINSTALLMODE like so #527 (comment) to force an overwrite regardless of the version installed.

Thanks for the tip. 👍
However we have a large installer and when only a few files changed the reduced install time is a real benefit.
I created a new fixed package for me that works so far. Just have to replace it in the near future with an official one. 😃