Bug: Images list filename not being used for generated #include statement

Description:

It looks like the filename for the Images list is not being used for the #include generated in any C++ header files that use functions/data from the Images list file. This is currently reproduciblle in wxUiTesting/quick/dlg_test.wxui which uses my_images as the filename, but the generated test_dlg.h uses #include "images.h".

I just tried to reproduce this and it worked fine. I have made some changes to image list handling since the bug was created, so I'm going to assume it got fixed.


Falco under configuration management

Idea: it would be nice to have a Puppet module (or other type of configuration management software "recipe") to deploy and manage falco.

Sounds like a good suggestion. I'm not that familiar with Puppet so I'll have to do some research first.

Just submit a RFC for an ansible role
https://github.com/draios/falco/issues/128
https://github.com/juju4/ansible-falco/


ServiceBuilder dependency error

1. Which version of kreait/firebase-php are you using?
Version 4.6.0
2. Which version of PHP are you using?
PHP 7.2.3
3. What's the issue?
Hello,
One of my dependencies (superbalist/flysystem-google-cloud-storage) requires usage of google/cloud-storage <1.4, not allowing a recent google/cloud to be installed.
I can install firebase-php 4.6.0 but get this error :
Class 'Google\Cloud\Core\ServiceBuilder' not found
Should composer.json require a recent version of google/cloud ?
How can I can make flysystem-google-cloud-storage and firebase-php work ?

The SDK requires the same library, but in version ^7.0 - if another dependency in your project requires version <1.4, this is a difference that cannot be bridged. If possible, try to update your other dependencies as much as possible or replace them with equivalents. Other than that there is not much we can do in the scope of the SDK. I could test it using older releases, but from 7.x down to 1.x is not likely to work.

I'm closing this issue now because there is not much I can do in scope of this library, but feel free to add further comments if you'd like.


Documentation Should Be Updated To Specify That Docker Desktop Is No Longer Free For Commercial Users

As of August 31st, Docker Desktop requires a paid license for commercial users, so I recommend the documentation be revised to include this detail when instructing Windows 10 users on downloading the product and creating an account, since the free Docker account is now only good for personal, educational, or small business use, according to their definition of small.
[Enter feedback here]

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

ID: e09e1b22-1344-d7b9-9462-dc4c2e160e86
Version Independent ID: f6411e97-4419-1209-56ad-9140ceb37329
Content: Prep Windows operating system containers
Content Source: virtualization/windowscontainers/quick-start/set-up-environment.md
Product: windows-server
Technology: windows-containers
GitHub Login: @v-susbo
Microsoft Alias: viniap

Thanks for your input on this topic. I'll look into the new requirement for a Docker Desktop paid license, and update it accordingly.

Similar issue as #1643 .

Great thanks

I'd like to announce Stevedore project. It aims to provide a free and convenient way to use Docker on Windows (both client and server). Currently, it only supports Windows containers. Stevedore is distributed as an MSI installer. Chocolatey.org package is planned but not there yet.
Installation

Download MSI installer
Double-click on it
Next-next-next
Reboot your computer
Profit! docker command is now available on your PATH and dockerd daemon is already running.


Export as logicterms / s-expressions

See #20

cc @gregr
(node_prop 'UBERON:0000949' iri 'http://purl.obolibrary.org/obo/UBERON_0000949')
(node_prop 'UBERON:0000949' 'http://www.geneontology.org/formats/oboInOwl#hasDbXref' 'TAO:0001158')
(node_prop 'UBERON:0000949' 'http://purl.obolibrary.org/obo/chebi#homology_notes' 'Multicellular organisms have complex endocrine systems, allowing responses to environmental stimuli, regulation of development, reproduction, and homeostasis. Nuclear receptors (NRs), a metazoan-specific family of ligand-activated transcription factors, play central roles in endocrine responses, as intermediates between signaling molecules and target genes. The NR family includes ligand-bound and orphan receptors, that is, receptors with no known ligand or for which there is no ligand Pocket. Understanding NR evolution has been further improved by comparison of several completed genomes, particularly those of deuterostomes and ecdysozoans. In contrast, evolution of NR ligands is still much debated. One hypothesis proposes that several independent gains and losses of ligand-binding ability in NRs occurred in protostomes and deuterostomes. A second hypothesis, pertaining to the NR3 subfamily (vertebrate steroid hormone receptors and estrogen related receptor), proposes that before the divergence of protostomes and deuterostomes, there was an ancestral steroid receptor (AncSR) that was ligand-activated and that orphan receptors secondarily lost the ability to bind a ligand. (...) Our analysis reveals that steroidogenesis has been independently elaborated in the 3 main bilaterian lineages (...).[well established][VHOG]')
(node_prop 'UBERON:0000949' 'http://www.geneontology.org/formats/oboInOwl#hasExactSynonym' 'endocrine system')
(node_prop 'UBERON:0000949' 'http://www.geneontology.org/formats/oboInOwl#hasOBONamespace' uberon)
(node_prop 'UBERON:0000949' 'http://www.geneontology.org/formats/oboInOwl#id' 'UBERON:0000949')
(node_prop 'UBERON:0000949' 'http://purl.obolibrary.org/obo/chebi#external_definition' 'Anatomical system containing glands which regulates bodily functions though the secretion of hormones.[AAO]')
(node_prop 'UBERON:0000949' name 'endocrine system')

...

(edge None 'UBERON:0000949' 'MONDO:0005151')
(edge_prop None 'UBERON:0000949' 'MONDO:0005151' predicate 'http://purl.obolibrary.org/obo/RO_0004026')
(edge_prop None 'UBERON:0000949' 'MONDO:0005151' provided_by None)
(edge None 'UBERON:0000066' 'UBERON:0000949')
(edge_prop None 'UBERON:0000066' 'UBERON:0000949' predicate 'http://purl.obolibrary.org/obo/RO_0002492')
(edge_prop None 'UBERON:0000066' 'UBERON:0000949' provided_by None)

the Nones are an upstream error, will fix...


Updated the App Authentication Extension section

Added the following
(The App Authentication Extension is available as a seperate extension for visual studio 2017 Update 5 and from update 6 and above the exension is bundled within the product . You may require to install the same by selecting Azure Development tools from within the Visual Studio  installer.) in the section Authenticating with Visual Studio , detailing the version specific requirements for the extension within Visual studio 2017 .
Closes #25684

@shashishailaj : Thanks for your contribution! The author, @bryanla, has been notified to review your proposed change.

Thanks @shashishailaj. Agree that this change makes sense, and Issue #25684 should be closed.
Adding @msmbaldwin as he owns Key Vault content now and can sign this one off.

@msmbaldwin - please take a look.

Hi, @shashishailaj. I was already editing the "Service-to-service" article when this came in, so I included the updates with my other changes.  The page now reads:

Authenticating with Visual Studio has the following prerequisites:

Visual Studio 2017 v15.5 or later.

The App Authentication extension for Visual Studio, available as a separate extension for Visual Studio 2017 Update 5 and bundled with the product in Update 6 and later. With Update 6 or later, you can verify the installation of the App Authentication extension by selecting Azure Development tools from within the Visual Studio installer.

Sign in to Visual Studio and use Tools > Options > Azure Service Authentication to select an account for local development.

@msmbaldwin - Would you add a comment to show what action to take on this PR:
Merge the PR: #sign-off
Close the PR: #please-close
Wait on the PR: #hold-off
Thanks!

#please-close


Dark mode colours

Small issue with using a pure white #FFFFFF hex value for general text on NRDB in dark mode, should be decreased slightly so it's easier on the eyes at night, see discussion from GLC for screenshots: https://discord.com/channels/425105101451952128/822475344669835315/1195483726398894143

I'm pretty sure this got fixed

Yep, the <body> color is set to #c8c2bc via --nrdb-color--text CSS variable.
Closing.


figure11 some addition

In this PR, 4 subplots were added to figure 11 to show the average phase lengths for different states versus the concentrations, with the average phase lengths being shape \times scale. I thought this might be a good visualization.
@aarmey I remember you mentioned yesterday that we can ultimately have cell numbers versus concentrations, something like what we have in the ODE model. I may have heard wrong and you said something else, but if I heard correctly do you mean that we have a specific time point and calculate cell number in different concentrations for that specific time point and plot versus the concentration?

No, I was only talking about phase lifetime (scale * shape) like it sounds you have here.

Codecov Report

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

@@            Coverage Diff             @@
##           master     #611      +/-   ##
==========================================
+ Coverage   87.29%   87.47%   +0.18%
==========================================
  Files          21       21
  Lines        1613     1613
==========================================
+ Hits         1408     1411       +3
+ Misses        205      202       -3

Flag
Coverage Δ

#unittests
87.47% <ø> (+0.18%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lineage/Analyze.py
73.04% <0.00%> (+2.60%)
: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 a2012c8...519ded9. Read the comment docs.

Please let me know if anything needs to change.

Can you add a plot/plots for the Bernoulli parameters?

Sure.

There's some tweaking you can do with the axes, etc, but the basic output is working well.

I will ask about them at the next meeting. Thanks.


Add descriptive doc comments to withMemoryRebound(to:capacity:_:) and…

… bindMemory(to:capacity:).
SR-2480: Improve documentation for withMemoryRebound(to:capacity:_:)
rdar://problem/28440528

@swift-ci smoke test.

@dabrahams Please review for Swift 3.
It's clear from user feedback that these APIs needed at least slightly more descriptive comments. It's difficult, because they only way to fully explain is to point people to the migration guide or SE-0107.

Regardless of my comments, this is an improvement and has my blessing for CCC

@swift-ci smoke test.

@swift-ci Please smoke test linux

@swift-ci smoke test and merge.

@atrick Sorry, we are currently updating Xcode. You will need to re-trigger the build once CI is back online.


Resolves absent function errors encountered on service details state

Euwe Backport details:
$ git log
commit 09396ece96e82f9169586b70af020366498a6eb0
Author: Chris Kacerguis<EMAIL_ADDRESS>Date:   Thu Sep 22 10:14:59 2016 -0500

    Merge pull request #172 from AllenBW/hotfix/restore-service-details-state

    Resolves absent function errors encountered on service details state
    (cherry picked from commit b146645e2603a6b530aa3942690eaa5065b6275d)


Make status fields non-nullable and fixup any existing invalid nulls

Closes: #DNE
What's Changed
All objects with status fields had an inconsistency in that the field was blank=False (requiring the UI or API to always provide a value for these fields) but null=True (meaning that the ORM could, and by default would, create objects with NULL status fields). The intent was always for these fields to be required/non-nullable; this PR fixes the code and DB to actually match that intent, including data migrations to create a NULL Status and use it if any null statuses are found in the database.
I've left the StatusModel mixin in place, and kept its default behavior of blank=False, null=True because I know we have various apps using that mixin; however I've changed all of our core models to simply include a StatusField directly rather than using an unnecessary mixin class.
I've removed the RoleModelMixin and RoleRequiredRoleModelMixin classes as well (those were next-only and so have no adoption outside of core to worry about) and replaced their usage with direct inclusion of RoleField. This did result in a migration of the Device.role field to be (correctly) non-nullable; I haven't bothered with a data migration here because its nullability was only a temporary bug introduced in next and shouldn't be encountered in the wild.
~I haven't run tests yet; it's quite possible that some number of tests were creating bad data and may now fail as a result; if so I'll fix those as well.~ Tests should all be passing now. Had to make so many fixes to bad test data...
TODO

[x] Explanation of Change(s)
[x] Added change log fragment(s) (for more information see the documentation)
n/a Attached Screenshots, Payload Example
[x] Unit, Integration Tests
[x] Documentation Updates (when adding/changing features)
n/a Example Plugin Updates (when adding/changing features)
[x] Outline Remaining Work, Constraints from Design

I see the benefit of making a model "Status-able" and "Role-able" with just a mixin. Are you leaving it open so people can let them be nullable in their models instead of us enforcing it?

I see the benefit of making a model "Status-able" and "Role-able" with just a mixin.

It's one line of code to add a status or role field to a class, there's no other functionality attached to the mixins. Having two different RoleModelMixin classes just to distinguish between role = RoleField(blank=True, null=True) and role = RoleField(blank=False, null=False) seemed like overkill to me.

Are you leaving it open so people can let them be nullable in their models instead of us enforcing it?

Partly that, but also because it looks like overriding or setting defaults for blank or null in the __init__() of these fields is oddly problematic (seems to have been why Device.role was nullable despite using the RoleRequiredRoleModelMixin?)

I see the benefit of making a model "Status-able" and "Role-able" with just a mixin.

It's one line of code to add a status or role field to a class, there's no other functionality attached to the mixins. Having two different RoleModelMixin classes just to distinguish between role = RoleField(blank=True, null=True) and role = RoleField(blank=False, null=False) seemed like overkill to me.

Are you leaving it open so people can let them be nullable in their models instead of us enforcing it?

Partly that, but also because it looks like overriding or setting defaults for blank or null in the __init__() of these fields is oddly problematic (seems to have been why Device.role was nullable despite using the RoleRequiredRoleModelMixin?)

Any concern about developers not setting the StatusField to the status property on a model? It would make it harder on the UI to magically put the status value in the header if a model has it:

burrito = StatusField(blank=False, null=False)

I see the benefit of making a model "Status-able" and "Role-able" with just a mixin.

It's one line of code to add a status or role field to a class, there's no other functionality attached to the mixins. Having two different RoleModelMixin classes just to distinguish between role = RoleField(blank=True, null=True) and role = RoleField(blank=False, null=False) seemed like overkill to me.

Are you leaving it open so people can let them be nullable in their models instead of us enforcing it?

Partly that, but also because it looks like overriding or setting defaults for blank or null in the __init__() of these fields is oddly problematic (seems to have been why Device.role was nullable despite using the RoleRequiredRoleModelMixin?)

Any concern about developers not setting the StatusField to the status property on a model? It would make it harder on the UI to magically put the status value in the header if a model has it:

burrito = StatusField(blank=False, null=False)

If an app has a use case for a StatusField that isn't semantically equivalent to status, I don't think that's necessarily something we need to worry about preventing or automatically handling. (I'd be fine with a pylint warning since those can always be disabled by someone if it's a false positive.) I think we can safely assert in general that "if you have a status field, it must be a StatusField" (though JobResult already violates that, IIRC) but not "if you are using StatusField, you must name it status."


Syntax Error

Q
A

Bug report?
yes

Feature request?
no

RFC?
no

How used?
3party

Swiftmailer version
6.0.3

PHP version
5.6.32

Observed behaviour
Parse error: syntax error, unexpected '?' in ../swiftmailer/lib/classes/Swift/Transport/EsmtpTransport.php on line 211
Expected behaviour

Example

No Bug.

Swift Mailer is a component based mailing solution for PHP 7.

https://github.com/swiftmailer/swiftmailer/blob/master/README


background/foreground user position update

Hi Liodali,
is there an option to update the user location when the app is in background/foreground?
Thank You so much

@frxk you can use the tracking option:
 osmOption: const OSMOption(
          userTrackingOption: UserTrackingOption(
            enableTracking: true,
            unFollowUser: false,
          )
)


clevis init uses open-zeppelin head

Use the latest release version

Thanks for the PR, just saw it!


PromptCompressor error - OpenAIGPTLMHeadModel.forward() got an unexpected keyword argument 'past_key_values'

I am trying to use OpenAI GPT-2 model for prompt compression. However, getting error "OpenAIGPTLMHeadModel.forward() got an unexpected keyword argument 'past_key_values'". Has anyone faced/facing similar issue and how can this be fixed. Thanks.
`!pip install llmlingua
from llmlingua import PromptCompressor
llm_lingua = PromptCompressor(
model_name = "openai-gpt",
device_map="cpu",
)
prompt = """
The following is a conversation with an AI assistant. The assistant is helpful, creative, clever, and very friendly.
Human: Hello, who are you?
AI: I am an AI created by OpenAI. How can I help you today?
Human: I'd like to cancel my subscription.
AI: I'm sorry to hear that. What is your subscription number?
Human: 123456
AI: Thank you. Your subscription has been cancelled.
Human: Thank you. Goodbye!
AI: Goodbye!
"""
compressed_prompt = llm_lingua.compress_prompt(prompt, instruction="", question="", target_token=200)
`

Hi @manojsharmadcx,
Thank you for your support. The issue arises because the OpenAIGPTLMHeadModel (link to code) does not support the input of a KV cache. You might consider using "gpt2" or "lgaalves/gpt2-dolly" instead.

Hi @iofu728,
Thanks for your response. Currently the gpt-2 model seem to be downloaded from HuggingFace on application file system where it need to be used, which means the application(eg: Azure App Service) needs extra resources to run prompt compression. Is there a way to use hosted Azure Open AI model for compression instead?
Thanks,
Manoj.

Hi @manojsharmadcx, yes, currently a local deployment of the corresponding small model is required to use this method. If the API model supports obtaining the log probabilities of the prompt section, then it's possible to implement LLMLingua through API calls.


fix: stringify parameter value arrays

What it solves
Resolves #3502
How this PR fixes it
An array parameter value is always stringified when processing a transaction.
When receiving decoded data from the back-end while creating a transaction, decoded data parameters are of type
DecodedDataBasicParameter which types the parameter value as a string.
How to test it

Propose a transaction that contains a nested array as a parameter (e.g. claiming a merkle airdrop, such as vCOW)
Check that nested array is displayed

Screenshots

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 34.54%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/components/DecodeTxs/index.tsx
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3318

Relevant Lines:
8622

💛  - Coveralls

Tried this with a dummy contract that can use nested arrays, but it works fine in dev as well, so probably testing it wrong. We need another way to test this.
Also idk what kind of app is this "(e.g. claiming a merkle airdrop, such as vCOW)" is not in rinkeby or mainnet Eth.

@francovenica you can use the claimMany method from https://etherscan.io/address/0xD057B63f5E69CF1B929b356b579Cba08D7688048 to test it. Hit me up if you are struggling with it.

@francovenica you can use the claimMany method from https://etherscan.io/address/0xD057B63f5E69CF1B929b356b579Cba08D7688048 to test it. Hit me up if you are struggling with it.

Ok, I was able to use that contract ("0xD057B63f5E69CF1B929b356b579Cba08D7688048") ABI and use the method "claim many" to see the data in the modal. I was able to see it failing in dev and fixed here. It seems that the issue wasn't just nested array by the type as well.
Dev:

This PR:


Protobuf libs

You probably saw these
https://github.com/jvdsandt/protobuf-smalltalk
https://github.com/zecke/pharo-protobuf
http://smalltalkhub.com/#!/~gokr/Protobuf
but I wanted to be sure :)

Yes, none of them are usable yet. The closest one is zecke's, which I have
partially tested and which need just some more work to be usable. I think
he is very busy though and/or have shifted his priorities. To everyone's
defense, it's quite complex, and the approach by zecke to leverage a
pre-built .dll/so for the target environment is probably the best way
forward.
On Thu, Aug 6, 2020 at 8:35 PM Paul DeBruicker<EMAIL_ADDRESS>wrote:

You probably saw these
https://github.com/jvdsandt/protobuf-smalltalk
https://github.com/zecke/pharo-protobuf
http://smalltalkhub.com/#!/~gokr/Protobuf
but I wanted to be sure :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/psvensson/firebase-st/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAJVASHU37ZNZTXFHD4YB3R7LZYTANCNFSM4PW3SOMQ
.

--
Peter Svensson
http://se.linkedin.com/in/petersvensson (LinkedIn)
http://twitter.com/psvensson (@psvensson)

But thanks anyway :) The main problem is that gRPC is 10x as complex as it needs to be, IMO


Make LoomProvider respect caller chain ID override for static calls

Otherwise static calls to EVM contracts may end up with the wrong caller chain ID & address.

LGTM, also looks like a test failed

Yeah, one of these jobs tends to fail every so often. Something flaky in there.


Streamlining page layout with a global navbar and footer

Issue Overview:
The current state of the project requires the addition of a navbar and footer on every page, which can be time-consuming and error-prone. I propose a significant improvement to the project by implementing a more efficient and consistent approach to handling the layout across all pages.
Proposed Solution:
To address this issue, my proposal involves making alterations to the layout.js file. By doing so, we can establish a global layout that automatically includes the navbar and footer on every page. This enhancement will simplify the development process, enhance code maintainability, and ensure a unified user experience throughout the entire website.
Expected Changes:
Refactor the layout.js file to include the navbar and footer components, so they are consistently rendered across all pages without manual additions.
I would like to work on this issue please assign it to me

Hello BhavinRaichura, thanks for opening a issue, your contribution is valuable to us. The maintainers will review this issue and provide feedback as soon as possible.

hey @shreya024  is this issue has resolved?
if not, Please Assign me the task under GSSOC'23. I am eager to work on it.

@BhavinRaichura Any updates?

@BhavinRaichura Any updates?
Sorry for the delay i have finished my work


⚠ SceneTime has degraded performance

In 5e03bfe, SceneTime ($ST) experienced degraded performance:

HTTP code: 503
Response time: 26596 ms

Resolved: SceneTime performance has improved in 3e13d3d after 8 minutes.


Black screen bios still exists

I still get the black screen when entering bios.
Is there any way to fix it

Same here. I wanted to remove the CMOS battery but it turned out to be behind the motherboard and I don't have the time to go this deep.
Does anyone know of another way to reset the BIOS without removing the battery? Does flashing a new BIOS from Windows help?

Hi . I will recovered video for it ( next week) . It doesn't need to remove battery ! Now clover v 11 is ok but clover v10 made bios black screen .

Hi . I will recovered video for it ( next week) . It doesn't need to remove battery ! Now clover v 11 is ok but clover v10 made bios black screen .

Thank you very much!

Yeah, i wonder the same

U guys can use the AFUWIN to reflash the NVRAM of BIOS,
first download and open the AFUWIN, then download the bios file of gl553vd in asus driver website
Then open the bios file and only tick nvram in flash
after flash u can restart and the bios working again

@shivaluma thank you for the quick response.
I'm trying to download it from a trusted source though. Do you know which one it is on this page? https://ami.com/en/products/firmware-tools-and-utilities/bios-uefi-utilities/

you can download it from here
https://github.com/MohammadtaghiFarkhondekar/macOS-Mojave-For-Asus-ROG-GL553VD/issues/24
Then open AMIBIOS - AFUWIN - 64 from Windows

@shivaluma thank you. I know about that link, but I can't trust the website, and I only download from trusted source on Windows. Do you know what kind of BIOS we have in the GL553VD?

https://www.asus.com/vn/Laptops/ROG-GL553VD/HelpDesk_BIOS/
download the latest version (308)

@shivaluma thank you so much

Hi, I just pseudofix it.  With the program recommended from shivaluma, my Asus got blue screen when I opened it, so I had to use the program from "Aptio\afuwin\64\AfuWin64\AFUWINGUI.exe" (recommended from valinor61 from another issue). I clicked in open, show all files, select the bios file, and the second tab, the program should to be selected the flash rom, something like that, untick it and then tick the option that say "NVRAM" and next, flash. I finally could get access to bios and fix some things (I could boot my pc without a pendrive, very very weird).
Now when I boot to the hackintosh, again I can't boot to bios, so i have to do the previous process again...any ideas?

change your clover to V11 of mohammad and then reflash the nvram again

Next week I will send V12 + Guide .

does v12 ready sir

yes . updating ...


Move skuba PR test to shared lib

Why is this PR needed?
Does it fix an issue? addresses a business case?
add a description and link to the issue if one exists.
Fixes #
Reminder: Add the "fixes bsc#XXXX" to the title of the commit so that it will
appear in the changelog.
What does this PR do?
please include a brief "management" technical overview (details are in the code)
Anything else a reviewer needs to know?
Special test cases, manual steps, links to resources or anything else that could be helpful to the reviewer.
Info for QA
This is info for QA so that they can validate this. This is mandatory if this PR fixes a bug.
If this is a new feature, a good description in "What does this PR do" may be enough.
Related info
Info that can be relevant for QA:

link to other PRs that should be merged together
link to packages that should be released together
upstream issues

Status BEFORE applying the patch
How can we reproduce the issue? How can we see this issue? Please provide the steps and the prove
this issue is not fixed.
Status AFTER applying the patch
How can we validate this issue is fixed? Please provide the steps and the prove this issue is fixed.
Docs
If docs need to be updated, please add a link to a PR to https://github.com/SUSE/doc-caasp.
At the time of creating the issue, this PR can be work in progress (set its title to [WIP]),
but the documentation needs to be finalized before the PR can be merged.
Merge restrictions
(Please do not edit this)
We are in v4-maintenance phase, so we will restrict what can be merged to prevent unexpected surprises:
What can be merged (merge criteria):
    2 approvals:
        1 developer: code is fine
        1 QA: QA is fine
    there is a PR for updating documentation (or a statement that this is not needed)

tested and working at https://ci.suse.de/view/CaaSP/view/CaaSP PRs/job/caasp-jobs/job/pr/job/test/view/change-requests/job/PR-1452/45/
Requires https://github.com/SUSE/caasp-automation/pull/809


chore: Add OWNERS file

Signed-off-by: Radim Hopp<EMAIL_ADDRESS>Description
This adds OWNERS file to the root of the repo. Since recently we enabled Tide for our repo, we should also specify approvers in this OWNERS file for tide to properly work

/lgtm


How to pass id with findOne when offline?

I have a question with findOne url which does not contain “id”, I already can custom the url to get remote data. However, when offline, if I want to call findOne again, I will need to provide an id (to find that object in local storage). Is that correct?
(I tried to put a fixed id when offline but can not retrieve the data from local storage)

Correct. You can hit a remote endpoint such as /me and that payload should return an ID. You should use that ID to retrieve it from local storage. All local storage resources are only addressable by a type and an ID.

I tried to add ID when loading local data, but No data returned from local.

I can't do much with the information you provide. What operations did you do before this? What was the payload?
A useful thing you could share with me is graph.toMap() in those lines you're debugging.

Hi @frank06 ,
In my graph.toMap(), I can see the object key and id there (screenshot below), does that mean the object (from findOne) is saved to local storage successfully?

However, when we use localAdapter.findOne to get the model, it returns null.

I also tried to save the model (when remote = true) manually but no luck.
_myProfileRepo.save(myProfiles, remote: false);

This is strange because we have a test covering this exact case.
Could you please send me the exact method you're invoking prior to this (findOne with remote=true, is it overridden? How?) and the exact server response payload?
Also when debugging, what is the output of:
(localAdapter as HiveLocalAdapter<ProfileMes>).box!.keys

Hi @frank06
Model :

Adapter :

findOne :

Hi @frank06  ,
Please help me when you have free time. I'm still waiting for you, thanks!

@nhatvu95 I need you to understand that if you don't send me the exact response from your API I cannot help you.
Also, what is ProfileMes? Is that a type? Show me the definition for that model. What does it stand for? It's confusing to follow.
If your model is PersonDetail then I would like to see (localAdapter as HiveLocalAdapter<PersonDetail>).box!.keys and please the full response payload from your API.

@frank06  my model is PersonDetail
Api response:

(localAdapter as HiveLocalAdapter<PersonDetail>).box!.keys

Interesting. I'm not sure what it is, as we have tests that should reproduce this:
https://github.com/flutterdata/flutter_data/blob/master/test/repository/repository_test.dart#L81-L112
My suggestions would be: if you are debugging try stepping into super.deserialize and see what happens in these lines:
https://github.com/flutterdata/flutter_data/blob/master/lib/src/repository/remote_adapter_serialization.dart#L89-L93 , more specifically line 91.
Or otherwise could you share your code and/or API with me so I can debug it myself?

Were you able to solve the issue?

Hi @frank06 ,
Sorry for the delay, I have solved my issue.

@nhatvu95 would you mind sharing how you fixed it?


Formalizing Goals, Features for OpenMined POC

Closes #11
Closes #12

I think given the progress in the last few days "Proof of Concept"  here needs to be replaced with a specific milestone name. (Milestone 3). @samsontmr thoughts?

@iamtrask agreed! However, not sure if we need these anymore if we record the goals under milestones. Thoughts?

Yeah I think you're right. #processKeepsEvolving


Add switch for checkOwn (#197)

(cherry picked from commit 57d8fdfea9137b0ac8e0ee9ad6b279561934915a) (cherry picked from commit 162e9e304715e5cea33893d7843c2dbf814f3378)

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

May i ask could we keep only one configuration file, such as just yml? because now every service has a same single common.properties.

May i ask could we keep only one configuration file, such as just yml? because now every service has a same single common.properties.

This need to refactor a lot of code, since some utils is not spring bean

May i ask could we keep only one configuration file, such as just yml? because now every service has a same single common.properties.

This need to refactor a lot of code, since some utils is not spring bean

ohh, right. get it. thx


Update Composer dependencies (2020-12-21-00-12)

Visual regression test passed!
View the visual regression test report


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83844

Triggering TagBot for merged registry pull request
This extra notification is being sent because I expected a tag to exist by now, but it doesn't.
You may want to check your TagBot configuration to ensure that it's running, and if it is, check the logs to make sure that there are no errors.


fix: #12812

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?

when alwaysShowContent is true change or rotation window size and restore will hide tooltip
Fixed issues
Close #12812

Details
Before: What was the problem?

1- display echart tooltip exemple: https://echarts.apache.org/examples/en/editor.html?c=line-tooltip-touch
2- add option 'alwaysShowContent: true' for tooltip
3- click on the chart for display tooltip
4- change window size or rotation
the tooltip move, not corresponding to the selected point position
After: How is it fixed in this PR?

when alwaysShowContent is true change or rotation window size and restore will hide tooltip
Usage
Are there any API changes?

[ ] The API has been changed.

Related test cases or examples to use the new APIs
NA.
Others
Merging options

[ ] Please squash the commits into a single one when merge.

Other information

Why hide tooltip rather than relocate it to the proper coordinate?
And I think doing it here is a bit inappropriate.

Why hide tooltip rather than relocate it to the proper coordinate?
And I think doing it here is a bit inappropriate.

Let me have a try.

Why hide tooltip rather than relocate it to the proper coordinate?
And I think doing it here is a bit inappropriate.

@plainheart The effect is as follows，Please help review, thanks!

@plainheart All the problems are solved, Please help review, thanks!

Finally, the same changed should be applied to TooltipRichContent, which will be used when renderMode is richText.

Finally, the same changes should be applied to TooltipRichContent, which will be used when renderMode is richText.

@plainheart All the problems are solved, Please help review, thanks!

Finally, the same changes should be applied to TooltipRichContent, which will be used when renderMode is richText.

@plainheart Thank you for your guidance.

@liulinboyi Thank you, great work! All looks good to me now, please @pissang have a look.

@liulinboyi Would you mind providing a test case for this?

@liulinboyi Would you mind providing a test case for this?

Let me have a try.

@liulinboyi Would you mind providing a test case for this?

@plainheart Test case is done, Please help review, thanks!

Looks good to me. Thanks for the wonderful work! @liulinboyi @plainheart

Perhaps the _moveTooltip can be renamed to a more specific name like _moveTooltipIfResized
Others all look good to me. Thanks for the wonderful work! @liulinboyi @plainheart

Ok, no problem.

Perhaps the _moveTooltip can be renamed to a more specific name like _moveTooltipIfResized
Others all look good to me. Thanks for the wonderful work! @liulinboyi @plainheart

Thank you for your guidance. @pissang I have fix some spell error and the _moveTooltip  was renamed, Please help review, thanks!

Seems there are still two request changes from @plainheart @susiwen8  need to be resolved before this PR can ben merged

Seems there are still two request changes from @plainheart @susiwen8 need to be resolved before this PR can ben merged

@pissang  All request changes are done. Can i fix other files spell error?

Seems there are still two request changes from @plainheart @susiwen8 need to be resolved before this PR can ben merged

@pissang All request changes are done. Can i fix other files spell error?

Merged. Thanks for your contribution!
And for the request to fix spell error, I think you are welcome and free to do it.


[launcher] Allow classic link to associate VPC security groups by name

Add a new security_groups field to classic_link, which should contain an array of VPC security groups to associate with a classic instance. If specified, the security group names are resolved to their ids in the given VPC.

LGTM


STT 考虑对中文的支持

考虑对中文的支持。
Originally posted by @cubxxw in https://github.com/telepace/voiceflow/issues/7#issuecomment-2519633063

done


Enhancement: support resource deltas for existing resources

topics may/may not support changeset (partition, retention) and fail for incompatible change - u.e. cleanup from DELETE -> COMPACT
ACL might be to alter an existing one
Schema needs to confirm it is compatible

Closing this for now - will raise a new issue/enhancement about the deletion of resources. ATM everything is additive/updated. deletion is destructive and currently a manual requirement.


Fix the empty key error in keyless sign-blob.

I think this should fix #142
I need to stop procrastinating and figure out how to get testing for the oauth flow setup ASAP. #88
Does anyone know a a good way to use secrets in github actions during PR reviews?
Signed-off-by: Dan Lorenc<EMAIL_ADDRESS>

Why not write a mock oidc flow for it?

Why not write a mock oidc flow for it?

Yeah, that's basically the only option I can think of - mock out the browser and everything. I wish we could do a headless webdriver with a test account :(


update test datasets

need a small dataset for CI
large dataset is https://quansight.nyc3.digitaloceanspaces.com/datasets/RC2_w18.tar.gz

done.


HTTP only not working

I try to use AWS ELB SSL termination and therefore run gitlab http only. The following vars do not work:
    gitlab_external_url: "http://gitlab.example.com"
    gitlab_redirect_http_to_https: "false"
    gitlab_nginx_listen_port: 80
    gitlab_nginx_listen_https: "false"

nginx throws a 400 with "The plain HTTP request was sent to HTTPS port".

After several hours of running playbooks back and forth I just discovered that using gitlab_external_url:"http://gitlab.example.com" does the trick -_-
Thanks for sharing this great role. Please close this issue. Sorry for the inconvenience.

No problem, thanks for posting the solution for the benefit of Google searches in the future!


No new features planned?

My understanding is that the CWA project is in ramp-down mode and that there will be no new features in the next app version 3.1.
If that is definitely the case, then the README etc. should be updated to avoid disappointment, since it still suggests that this repository is open for suggestions and, by implication, that there is some likelihood that a community suggestion could be implemented in a future app version.

@MikeMcC399 , even with the likelihood of another disappointment: At this current point of time we will proceed with usual community work. This also includes the open repository for suggestions - everything here still remains a valuable resource, at least for retrospective project activities.

I withdraw this request, since there was never any general promise in this repository that wishlist items would be implemented. It is entirely up to the product owners what they decide.
I note however that in practice all the enhancement requests from the corona-warn-app/cwa-app-android repository have now been closed. See enhancement list search showing 0 Open and 446 Closed.

See issue https://github.com/corona-warn-app/cwa-wishlist/issues/877


Plex Debrid not working with torrentio

Plex Debrid hooking "0 releases" for every single file I add to wish lists. When scrapping with torrentio always getting 0 releases for any file.
This need to be solved ASAP, thanks.

Wrong repo. Solved ASAP.

Wrong repo. Solved ASAP.

Excuse me, what do you mean by "Wrong repo"?

This is the arrogance of developers. They close the question without resolving it and with arrogance. Never forget that we are the users who test your tools. If you're not going to answer me, don't let it be so noticeable.

which repo is this suppossed to be? At least answer me


Updating the Datafactory SDK from swagger repo, preparing release

@hvermis there is one change(sent out for more than one week now) not checked in which needs to be in the SDK by end of this month.

@wenbof  Let's do that in a separate PR to minimize the risk of not making it.

@hvermis Hopefully we can make that by end of this month. it is a hard date. Thanks.

@hvermis , could you please share swagger spec PR link related to this change? I need to understand what's change in the spec PR and whether it is merged?

@erich-wang  I generated the .Net code from the public repo url, ADF readme file. So this code corresponds exactly to whatever is merged in swagger repo. Here are the PRs in swagger repo:
https://github.com/Azure/azure-rest-api-specs/pull/6631
https://github.com/Azure/azure-rest-api-specs/pull/6548
https://github.com/Azure/azure-rest-api-specs/pull/6607
https://github.com/Azure/azure-rest-api-specs/pull/6568


typing(interactions): improve type annotations for disnake.Interaction and subclasses

Summary

Closes GH-1036

Checklist

[x] If code changes were made, then they have been tested

[ ] I have updated the documentation to reflect the changes
[x] I have formatted the code properly by running pdm lint
[x] I have type-checked the code by running pdm pyright

[ ] This PR fixes an issue
[x] This PR adds something new (e.g. new method or parameters)
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

I've removed all the unnecessary type ignores


Update thrift

Primary motivation for this update was to include the "last" details for a pending activity. This allow you to see this information in the UI rather than having to hit the describe workflow endpoint via the CLI.
lastFailureReason
lastWorkerIdentity
lastFailureDetails

Running locally seems to show everything working as expected.
@just-at-uber anyone else I should tag for a review?

 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.

Hi @jredl-va , Thank you for posting a pull request. I had a look at the code and it looks like all the thrift file updates are taken from the cadence-idl repository. I have setup a new branch to pull in the cadence-idl files using NPM which should make updating to the latest version of these files significantly easier (just update the commit hash - see package.json under napa). You can have a look at the branch here: https://github.com/uber/cadence-web/tree/feature/server-thrift-submodule . Let me know if this addresses your main issue, or if I need to add anything extra to get lastFailureReason, lastWorkerIdentity,
lastFailureDetails working. Thanks!

@just-at-uber very nice! Yes, I had copied these manually over as that had looked like how this was done up until your branch. Note, I checked it out and it looks as though you have everything needed. Here is a screen grab from the same workflow I was testing with:

Will close this pr! Thanks!

See PR here: https://github.com/uber/cadence-web/pull/90


CPU Mode for This Batch and stop

Thanks for your wonderful work! I train on my custom datasets (about 70000 images) and after 5 epochs the logger shows that '------------CPU Mode for This Batch-------------' and the running stops without doing anything! I use fewer images (3000 images) and everything is fine. Can you please help me fix this? Is there anything not normal in the label assignment strategy? Thank you very much!
The log is here:
2021-08-08 16:04:19 | INFO     | yolox.core.trainer:248 - epoch: 6/100, iter: 320/1142, mem: 26793Mb, iter_time: 0.562s, data_time: 0.002s, total_loss: 3.399, iou_loss: 1.920, l1_loss: 0.000, conf_loss: 0.966, cls_loss: 0.513, lr: 2.000e-03, size: 736, ETA: 19:56:26
2021-08-08 16:04:32 | INFO     | yolox.core.trainer:248 - epoch: 6/100, iter: 340/1142, mem: 26793Mb, iter_time: 0.647s, data_time: 0.003s, total_loss: 3.410, iou_loss: 1.907, l1_loss: 0.000, conf_loss: 0.993, cls_loss: 0.510, lr: 2.000e-03, size: 608, ETA: 19:56:06
------------CPU Mode for This Batch-------------

I also tried to reduce image size and batch size, but this issue still happened after 1-5 epochs. After printing "------------CPU Mode for This Batch-------------", it does not do anything.

This CPU mode only occurred when your GPU mem is not enough. So you may have to continue to reduce your batchsize and img size.

Can reducing the 'max_labels' parameter help? There are more objects in my dataset than COCO, about 50 - 100 objects in one image.
dataset = MosaicDetection(
dataset,
mosaic=not no_aug,
img_size=self.input_size,
preproc=TrainTransform(
rgb_means=(0.485, 0.456, 0.406),
std=(0.229, 0.224, 0.225),
max_labels=200,
),
degrees=self.degrees,
translate=self.translate,
scale=self.scale,
shear=self.shear,
perspective=self.perspective,
enable_mixup=self.enable_mixup,
)

Yes, reducing max_label is helpful for reducing GPU mem, but it will also decrease your performance. Your case may be complicated for handle ~100 objects in one image. I suggest you remove mosaic aug (maybe preserve the mixup?) for better performance.

Thank you very much!

This OOM issue is probably caused by a few images which contains 500+ objects (CrowdHuman dataset). I solve it by reducing 'max_labels' to 150 in MosaicDetection or filter these very crowded images. Thank you again for your answers!

In the earliest version, I set 'max_labels' to 600, this error was not reported. It works very well.


"Apply" regression and foobar2000 freezes

Thanks for the original "Apply" fix, which all seemed to work in 1.1.1.

However, there seems to have been a regression in 1.1.2-beta. At least on my machine the following don't load images when press "Apply" in configure or panel properties. They do work if press "OK".
gdi.LoadImageAsyncV2
utils.GetAlbumArtAsyncV2
(I didn't test any other async operations)
I like the extra flexibility that the V2 versions offer, & so they are an excellent addition. So I hope this can be fixed to make testing easier.

Deadlocks (foobar2000 not responding:  I have to exit the program).

I see you fixed some freezes. However, I still see some (1.1.1 & 1.1.2 beta). These seem to be due to a bad interaction with other applications. Specifically they occur after pasting into an existing panel property. I noticed it as I have title format snippetts in Word (Word files weren't in a foobar2000 folder). Occasionally the deadlock is immediate after OKing panel properties. More likely it eventually occurs after using the script menu to change the layout, type of image view or theme. It may require quite a number of changes before the deadlock occurs and perhaps a layout where a lot of image handling & rearranging is involved, but I'm not sure (it occurs with both old async & new V2 async methods). It doesn't matter what text is pasted in. I've seen the issue after pasting from Word, another word processor, internet explorer & gmail running in former.
Importantly, the issue doesn't occur in the absence of a paste event or if the same text is pasted from notepad or notepad++, which, along with console traces, indicates it's a not a problem with the scripts. It also doesn't occur if text is typed in.
Anyway seeing as I spotted the issue I thought I would draw your attention to it in case you can see a fix. Now I know what's going on, I can avoid the problematical scenarios, and so it's not a big problem for me. But others could encounter it...

Thanks for the bug report!
I'm currently on vacation (without a proper PC), but I'll take a look once I'm back =)

At least on my machine the following don't load images when press "Apply" in configure or panel properties

Regretfully, this is by design: ECMAScript requires that all jobs (which includes Promises as well) are executed only once script executing stack is empty. In other words, Promise can't be executed while the there is an active JavaScript scope.
Configure dialog is a blocking operation, thus it prevents the clearing of JavaScript stack and therefore the execution of Promises.
This limitation brings some advantages as well though: your Promise will never execute while there is some other JavaScript code being executed. I.e. Promise won't ever interrupt any function in mid-execution and change variables that could result in a broken state.
fb.DoDragDrop, MenuObject.TrackPopupMenu, window.ShowProperties and a few others methods also block and prevent Promises execution.
But, IMHO, potential pros of Promises greatly outweighs this con =)

PS: The corresponding part of ECMAScript, that describes jobs execution policy - https://www.ecma-international.org/ecma-262/6.0/#sec-jobs-and-job-queues

Deadlocks

I'll take a look at the code, but most likely I will need your help in diagnosing this issue (once I'm near a proper PC), since I've never encountered it myself.

Regretfully, I can't reproduce the issue, so I need your help.
I need a memory dump which is captured during the freeze/deadlock. Sorry in advance for the probably unnecessary lengthy instruction, since I don't know, if you have any experience working with Visual Studio.
Preparation:

Download Visual Studio 2017 Community Edition (free at https://visualstudio.microsoft.com/en-us/downloads/).
Install it with Development of Classic C++ Applications option enabled.
Download and install nightly SMP build: https://ci.appveyor.com/api/projects/theqwertiest/foo-spider-monkey-panel/artifacts/_result%2FWin32_Release%2Ffoo_spider_monkey_panel.fb2k-component?branch=master&job=Configuration%3A Release
Download SMP PDB symbols archive and unpack them into SMP component folder (e.g. foobar2000\user-components\foo_spider_monkey_panel): https://ci.appveyor.com/api/projects/theqwertiest/foo-spider-monkey-panel/artifacts/_result%2FWin32_Release%2Ffoo_spider_monkey_panel_pdb.zip?branch=master&job=Configuration%3A Release

Memory dump capture (once fb2k freezes/deadlocks):

Open Visual Studio.
Debug>Attach to Process... (or press CTRL-ALT-P).
Select foobar2000 process (make sure that Attach to: field is set to Native Code only).
Debug>Break All (or press CTRL-ALT-Break).
Click on Debug>Save Dump As... and save it to the location of your choice.
Post the resulted dump on the sharing service of your choice. Or, if you don't have one that can be used with big files, I can PM you the link to my private one.

In the mean time I will continue trying to find it manually by looking through the code...

Updated post with instructions with an easier method to capture memory dump without Visual Studio.

That was fast!
Sent you a msg via hydrogenaud.io PM

I've uploaded 3 dumps obtained with latest 1.1.2 beta, each from a separate deadlock. One prepared with Task Manager & two with procdump.
HTH

Thanks!!! Hopefully it will be enough =)

Could you test the latest nightly build?
Added a dirty(?) workaround, which (in theory) should fix the deadlock.

Thanks for the test version & trying to sort the problem.
Unfortunately the issue isn't fully resolved (tested in a multi-tab DUI layout).
Freezes*** are still occurring but are now restricted to the affected SMP panel. SMP panels in other tabs and foobar2000 seem OK.
*** i.e. script menu & nothing works in affected panel + if go to another tab can't open the tab containing the affected panel.

Darn... Could you post memory dumps with the newest version, plz? (upload dump login/pass is still the same)

Do you have a sample script perhaps, that reproduces this problem?

Nvm, I think I've found the problem, which causes the panel freeze. Please check the latest nightly build once it's ready.

I've run the test scenario many times & I'm pleased to say that so far I haven't had any freezes.
Thanks for the fix (& also the typo issue fix).

That's awesome! Thanks for all the testing =)


invalid HTTP message at byte position 6: HTTP/2<-- HERE --> 200

https://data.commoncrawl.org/crawl-data/CC-NEWS/2020/09/CC-NEWS-20200921024254-00130.warc.gz  invalid HTTP message at byte position 6: HTTP/2<-- HERE --> 200 \r\nserver: Apache\r\nx-gen-mode: full\r...
multiple errors from files this year/month

See commoncrawl/news-crawl#42 - http/2 was enabled by a security upgrade of JDK and the HTTP headers were written as they were "stringified" by the protocol layers.


Improvement in the testimonials section

The testimonial page can be improved by adding animation and also adding a hover effect on them.
please @mksharma1 @Aniketbhanderi @yash982000 @M2kishore @pragati-kumar  assign this issue to me.

The testimonial page can be improved by adding animation and also adding a hover effect on them.
please @mksharma1 @Aniketbhanderi @yash982000 @M2kishore @pragati-kumar assign this issue to me.

@Khushi-agarwal The hover part is done. The animation you may feel free to do.
Related to this, I have some similar tasks too if you can do. You have to make a box or table kind of thing where one can see the requests.
These requests will be asked to the Alumni from the users/junior college students about asking them for a profile/resume/LinkedIn review. It should look like a connection request sent on Linkedin or a friend request on facebook.
The only difference is it will directly contain the link of the Linkedin or resume of that user.
Something like below:-

Here, you can see the requests section, where a table of pending requests along with their profile or resume link should be seen.
I have assigned you. Let m know if you can do it or not.

The link of the linkedin or resume of the user will also be in the form of table in the request section?

Yes @yash982000 I can do it.

Yes @yash982000 I can do it.

Great!!
Please do it and make a PR soon as you complete.

I have created a pr #42 . please @yash982000 review it.

I have created a pr #42 . please @yash982000 review it.

@Khushi-agarwal Yes, I am checking there and closing this issue.

@yash982000 Can you please add labels to this.


Progress badge on radicle icon in Activity Bar while loading

Resources:

https://code.visualstudio.com/api/ux-guidelines/views#views-with-progress
https://code.visualstudio.com/api/extension-capabilities/common-capabilities#progress-api
https://github.com/microsoft/vscode-extension-samples/tree/main/progress-sample

Relates to https://github.com/cytechmobile/radicle-vscode-extension/issues/5

FYI this is already implemented in the work done for the new Patches view https://github.com/cytechmobile/radicle-vscode-extension/issues/43#issuecomment-1724566221

Here's how it actually works behind the scenes:

reload button needs to be bound to a View and a Command https://github.com/cytechmobile/radicle-vscode-extension/blob/5b22e1c34b7cc63d005c97e4334e8674f1176fb7/package.json#L123
clicking the button triggers the command
which fires an eventEmitter
that triggers the Patches TreeViewProvider to run its getChildren(undefined) (according to the docs, when called with undefined it's trying to build the children of the treeview root, i.e. all list items)
apparently getChildren() automatically uses the Progress API to show that the view is busy
and because the View is bound to the Activity Bar (a.k.a. sidebar with Radicle logo) the Progress (a.k.a. busy) indicator is shown there automatically too

Now this ticket started out more as "let's use this feature" and, as of writing this, has no explicit spec to implement.
I suggest we either move AC No. (3) from https://github.com/cytechmobile/radicle-vscode-extension/issues/5

into this issue description (and delete the "relates to" comments in both tickets). This would also better narrow the scope of #5.
Other alternatives are to just close this ticket as done and create a new one with the moved out AC of #5 or close and not change anything else. (or something else?)
Marking this as needs-refinement and leaving it to @gsaslis for follow-up.

Closing this as done. #5 is not in the current refinement scope, so we will edit accordingly when we come round to picking up that work (and shouldn't really beforehand). ;)


Could not resolve host: mirrors.163.com; Unknown error

node2: http://mirrors.163.com/centos/7/os/x86_64/Packages/audit-libs-python-2.7.6-3.el7.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/extras/x86_64/Packages/container-storage-setup-0.8.0-3.git1d27ecf.el7.noarch.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/extras/x86_64/Packages/container-selinux-2.42-1.gitad8f0f7.el7.noarch.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/os/x86_64/Packages/checkpolicy-2.5-4.el7.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/extras/x86_64/Packages/docker-1.13.1-53.git774336d.el7.centos.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/extras/x86_64/Packages/docker-client-1.13.1-53.git774336d.el7.centos.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/extras/x86_64/Packages/docker-common-1.13.1-53.git774336d.el7.centos.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/os/x86_64/Packages/libcgroup-0.41-13.el7.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/os/x86_64/Packages/libsemanage-python-2.5-8.el7.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/extras/x86_64/Packages/oci-register-machine-0-6.git2b44233.el7.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.
node2: http://mirrors.163.com/centos/7/extras/x86_64/Packages/oci-systemd-hook-0.1.15-2.gitc04483d.el7.x86_64.rpm: [Errno 14] curl#6 - "Could not resolve host: mirrors.163.com; Unknown error"
node2: Trying other mirror.

This error is not related to this repo and the author must have mixed repos.

change your Host's DNS server to <IP_ADDRESS>


Using requests.get with unverified certificates

Hello, I am trying to use requests module to get & post our intranet websevice that has unverified certificates. We had no issues until the new version of python that actually does verify certificates by default. What is the correct way (or is it even supported) of doing so?
What I am currently doing is that I get the certificate and public key this way:
openssl s_client -connect <IP_ADDRESS>:4443 | openssl x509 -pubkey > ca.crt
The content of ca.crt then looks as follows:
 -----BEGIN PUBLIC KEY-----
...
-----END PUBLIC KEY-----
-----BEGIN CERTIFICATE-----
...
-----END CERTIFICATE-----

And the request I send looks like:
requests.get('https://<IP_ADDRESS>:4443', verify=True, cert='ca.crt')

which gives me error:
Traceback (most recent call last):
  File "req.py", line 2, in <module>
    requests.get('https://<IP_ADDRESS>:4443', verify=False, cert='testicek')
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 69, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 465, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 573, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/adapters.py", line 431, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: [Errno 336265225] _ssl.c:351: error:140B0009:SSL routines:SSL_CTX_use_PrivateKey_file:PEM lib

Can you please guide me what I am doing wrong?
Thank you for all your support!

Thanks for raising this issue!
The correct form of the request is:
requests.get('https://<IP_ADDRESS>:4443', verify='ca.crt')

The cert keyword argument is for client certificates.

Thank you, I just tested it, the code now looks as follows:
import requests
requests.get('https://<IP_ADDRESS>:4443', verify='ca.crt')

However when I attempt to run it, I get error:
Traceback (most recent call last):
  File "req.py", line 2, in <module>
    requests.get('https://<IP_ADDRESS>:4443', verify='testicek')
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 69, in get
    return request('get', url, params=params, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 50, in request
    response = session.request(method=method, url=url, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 465, in request
    resp = self.send(prep, **send_kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 573, in send
    r = adapter.send(request, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/requests/adapters.py", line 431, in send
    raise SSLError(e, request=request)
requests.exceptions.SSLError: [Errno 1] _ssl.c:504: error:14090086:SSL routines:SSL3_GET_SERVER_CERTIFICATE:certificate verify failed

Is it possible that the ca.crt that was generated with the previous mentioned command is incorrect?
Thank you!

Yeah, that command is wrong. You don't want the public keys, you want the certificates themselves. It's kinda tricky to write the command, but if you run openssl s_client -showcerts -connect <IP_ADDRESS>:4443, and then grab each of the BEGIN CERTIFICATE/END CERTIFICATE blocks and put them together in a pem file. Give that a shot.

Thank you!


ValueError,缺少配置文件。

Running on local URL:  http://<IP_ADDRESS>:7860
Running on public URL: https://3b124d98a8be961bc5.gradio.live
This share link expires in 72 hours. For free permanent hosting and GPU upgrades, run gradio deploy from Terminal to deploy to Spaces (https://huggingface.co/spaces)
text_config_dict is provided which will be used to initialize CLIPTextConfig. The value text_config["id2label"] will be overriden.
D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\transformers\models\clip\feature_extraction_clip.py:28: FutureWarning: The class CLIPFeatureExtractor is deprecated and will be removed in version 5 of Transformers. Please use CLIPImageProcessor instead.
warnings.warn(
D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\diffusers\pipelines\pipeline_utils.py:761: FutureWarning: torch_dtype is deprecated and will be removed in version 0.25.0.
deprecate("torch_dtype", "0.25.0", "")
D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\torch_utils.py:776: UserWarning: TypedStorage is deprecated. It will be removed in the future and UntypedStorage will be the only storage class. This should only matter to you if you are using storages directly.  To access UntypedStorage directly, use tensor.untyped_storage() instead of tensor.storage()
return self.fget.get(instance, owner)()
Traceback (most recent call last):
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\gradio\routes.py", line 488, in run_predict
output = await app.get_blocks().process_api(
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\gradio\blocks.py", line 1431, in process_api
result = await self.call_function(
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\gradio\blocks.py", line 1103, in call_function
prediction = await anyio.to_thread.run_sync(
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\anyio\to_thread.py", line 56, in run_sync
return await get_async_backend().run_sync_in_worker_thread(
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\anyio_backends_asyncio.py", line 2134, in run_sync_in_worker_thread
return await future
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\anyio_backends_asyncio.py", line 851, in run
result = context.run(func, *args)
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\gradio\utils.py", line 707, in wrapper
response = f(*args, **kwargs)
File "D:\Program\Moore-AnimateAnyone-for-windows\app.py", line 69, in animate
self.denoising_unet = UNet3DConditionModel.from_pretrained_2d(
File "D:\Program\Moore-AnimateAnyone-for-windows\src\models\unet_3d.py", line 668, in from_pretrained_2d
motion_state_dict = torch.load(
File "D:\Program\Moore-AnimateAnyone-for-windows\venv\lib\site-packages\torch\serialization.py", line 808, in load
raise pickle.UnpicklingError(UNSAFE_MESSAGE + str(e)) from None
_pickle.UnpicklingError: Weights only load failed. Re-running torch.load with weights_only set to False will likely succeed, but it can result in arbitrary code execution.Do it only if you get the file from a trusted source. WeightsUnpickler error: PytorchStreamReader failed reading file data/17: invalid header or archive is corrupted
已将所有权重文件放置“pretrained_weights”目录，还是提示缺少配置文件

应该是某个文件损坏了没下完，看起来是denoising那个。
另外使用自动下载就可以，现在不读取外部VAE，使用sd1-5的


Added dupulicated travis-ci launch avoiding code.

Codecov Report

Merging #274 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master    #274   +/-   ##
======================================
  Coverage    81.8%   81.8%
======================================
  Files          33      33
  Lines        4737    4737
======================================
  Hits         3875    3875
  Misses        862     862


Support global resource aliases

For some things, the type and id are dynamic (e.g., flow relationships), we should let auto complete work for any type, some challenge will be that the alias might be ambiguous so we might need to just filter out duplicates.

Duplicate of #311 (hopefully) :)


History Tab Is messed up

Your description here
History tab is messed up in the new version
Expected outcome
What should happen?
Last used API should show up on the top
Actual outcome
What happened?
The API order is messed up
Versions
App: <IP_ADDRESS>9-stable
Chrome: 54.0.2840.71
Platform: Mac OS X (10_10_5)
Steps to reproduce

Hi,
Thank you for the issue report.
This is a duplicate of #883 so I'm closing this one.


Issue 41675: allow updating the LSID of an existing exp.data

Rationale
EvaluationContextTest is failing due to insert or update on table "data" violates foreign key constraint "fk_data_lsid" when importing the eval data folder.  The crux of the issue is that an existing exp.data is having it's LSID updated when it is imported into an assay.
In the EvaluationContentTest failure: First, we assign an LSID to the file (something like urn:lsid:labkey.com:UploadedFile.Folder-142:<...guid...>).  Later when the file is imported into an assay, it gets a new LSID (something like urn:lsid:labkey.com:AssayRunTSVData.Folder-142:<...guid...>).  This happens in DefaultAssayRunCreator.createData() near the "Reset its LSID so that it's the correct" comment. It looks like we also could change the ExpData LSID in ThawListResolverType.configureRun.
Related Pull Requests

https://github.com/LabKey/platform/pull/1741

Changes

update LSID for existing exp.data by deleting and creating new exp.object

EvaluationContentTest passed on TeamCity:
https://teamcity.labkey.org/viewLog.html?buildId=1190242&


Add a way to lazily autoload argument completers

Summary of the new feature / enhancement
Currently, argument completers for native commands must be explicitly registered using Register-ArgumentCompleter. However, this means that the module which registers the completer must be eagerly loaded in $PROFILE during startup, otherwise the completions aren't loaded. If many different argument completers are registered, this may take a considerable amount of time on each startup, even when most of the completions are never used in the session.
As an example, importing https://github.com/MatejKafka/PowerShell-ArgumentCompleters takes about 175 ms on my computer. Currently, I'm loading them in an OnIdle event callback, which helps, but it's still not ideal and doesn't scale well to a larger number of completers.
It would be nice to add an autoloading support for argument completers. Completions for native commands (together with inline description) are the main reason why I prefer fish shell autocompletion over the one in PowerShell.

As an additional food for thought (which is definitely out-of-scope for this issue), currently there isn't any official repository for argument completers. If that was added, it could be possibly to interactively offer the user to download completions for the given command (if available) when he first triggers autocomplete on it, greatly improving the experience of using native commands in PowerShell. It already kinda works this way on Linux, where many packages also install autocompleters for commonly used shells.
Proposed technical implementation details (optional)
Possible implementation: Add an entry to the module manifest, listing the commands for which it provides argument completers. When completions for the command are requested, the module is autoloaded, which registers the argument completers.
This implementation has an issue where if the module is large (slow to load), there may be a considerable delay before the completions appear. This could be further improved by also specifying which file contains the argument completer for the given command, so that only that file could be loaded, without loading the whole module. This would allow for one big module, which registers many different completers, each in a separate file, without having to load the whole module.

Auto loading a module at the time of user pressing Tab is dangerous. It's likely will block the user input for a very noticeable period of time and the user simply cannot cancel whatever PowerShell is doing at that point. So, it seems PowerShell freezes, and the user has to wait till it becomes responsive.

The current alternative is eagerly loading the autocompletion module on startup, so the user has to pay this cost on each startup, not just on the first completion invocation.

Your described behavior is exactly what already happens for PowerShell commands – if I invoke completion on a command from a not-yet-loaded module, there's a delay where PowerShell is loading the module to provide completion. Thus, allowing autoload for external completers does not make this situation any worse.

The WG discussed this. We feel that there are a number of threads that need pulling here:

@daxian-dbw raised an excellent point about performance (think the huge az module)
We did agree that the module should likely have a way to specify completions
We absolutely agree about having a good way to acquire/share completions
We can't ignore completions for non-Modules (scripts/native apps)
fish/bash/zsh all do completion a little differently, what can we learn from them (and what can we avoid)?

We think a comprehensive approach is going to provide better results long term, rather than just trying to solve this problem for modules. We're suggesting that an RFC is really needed here.

fish/bash/zsh all do completion a little differently, what can we learn from them (and what can we avoid)?

They use scripts for completion. This is a way for code injection. I highly doubt that MSFT will agree to run someone else's extra code! Mandatory signing of such files is not convenient for users.
Now the .Net Command Line project uses the approach of starting target application with every tab. It is obvious how problematic this approach is.
I would suggest joining forces with this project and developing a standard for declarative description of meta information about parameters and arguments.
PowerShell uses this now through attributes. If there were a standard, it would be able to read external information (XML/Json) from any application and use it for completion.

I wonder if it would make sense to add a new parameter to Register-ArgumentCompleter called -Persist along with an optional -Scope (default to CurrentUser, but can accept AllUsers) which modifies powershell.config.json as the way to persistent auto re-registration of completers on startup?  Of course, we'd need a corresponding Unregister-ArgumentCompleter cmdlet which doesn't exist today and decide separately if it works beyond persisted ones.

A bit tangential, but modifying existing files during an install - whether MSI on Windows, brew, apt on debian flavors, etc. - is very problematic. Those "resources" can't really be tracked and removed safely with the packages or upgraded correctly, so most installers - even with custom solutions - don't even try. For years this was a problem with .NET Framework's machine.config, which is why we eventually came up with other solutions for first- and third-party solutions.
A common solution on most linux distros is a directory for drop-in customizations, like /etc/apt/sources.list.d (".d" for a directory is a common naming convention). Files (which, on linux, everything is a file) in whole are best relegated to installers.
That said, PowerShell already has a directory-based solution: modules. Could Crescendo fill this void?

Regarding @SteveL-MSFT's proposal in https://github.com/PowerShell/PowerShell/issues/17283#issuecomment-1374125422, it is still eager loading, meaning the long delay in startup reported here will remain an issue. Also, to register completion, it requires an installer to start the pwsh process to run Register-ArgumentCompleter during the installation, which is error prone in my opinion.
I think we should have a "completion path", similar to $PSModulePath, which contains directories that powershell will probe for completion meta files (this aligns with how it works in other shells). The completion meta file can be named as <command-name>+<.pwsh>, and it has to be a powershell data file just like a .psd1 file. Initially, the metadata contained in the file should just be a 2-element hash table:
@{
    module = "module-name"
    command = "command-name"
}

When tab completion on a native command is triggered, the engine completion code will:

first look into context.NativeArgumentCompleters to find if a completer is already registered for the native command.

if so, run the registered completer and done.

If not, then the completion code searches "completion path", and stop when finding the first matching completion meta file.

do nothing if the search fails

retrieve the metadata, then call the "module-name/command-name" assuming that the module has been deployed when installing the native command.

do nothing if the command call failed

again, look into context.NativeArgumentCompleters to find if a completer is now available

do nothing if still not available, OR
run the newly registered completer and done.

With this approach, an installer just needs to deploy a completion module and a completion meta file to enable completion for the native command in powershell, which is way less error prone, and also allow lazy-loading of the completer automatically when it's needed.
One may argue that the lazy-loading upon tab completion happens may cause unresponsive interactive experience at least for the first tab for a particular native command. That's true, but I don't think that's a bad thing -- it forces the completion module author to be more performance oriented in designing the module and argument completer.

@daxian-dbw so with a file path based approach would we have both shared and user directories?  would modules also use this mechanism?

My suggestion was #13428 - create generic support for a native command wrapper.
To do something clever with native commands PowerShell needs meta information. PowerShell already has the means to create such meta information. With pseudo/native cmdlets we could add any meta information in the usual way. (Apart from parameter descriptions it could be (optionally) transcoding, output parsing, etc.)
I don't understand why we have to invent something else if we already have everything.
A more fundamental approach would be to synchronize with the .Net CommandLine project team so that PowerShell can take meta information directly from the native command dll.

so with a file path based approach would we have both shared and user directories?

@SteveL-MSFT It's similar to the module path concept we have in PowerShell. We can have a "completer" folder under $PSHome just like the module folder, in case we want to ship completer for native commands along with PowerShell. And we can have another completer folder under MyDocument (or a different place where we want to move our current MyDocmuent module folder to) for custom/third-party completers.

would modules also use this mechanism?

It would be great for modules to use the same mechanism. We can have a new field in the PrivateData section of the module manifest for completer metadata, and then Install-Module can take care of creating the completer file for the module in the completer path.
In fact, if all modules that contain completers for native command have that metadata included in the PrivateData section, then, in theory, we don't really need the completer path, but can look through all modules to find the one that contains completer for a particular native command. However, this discovery will be way slower than having the completer path approach, because you don't know which module to look for until you find it. Even the module cache won't help us because when a native command doesn't exist in the cache, we will have to do a full search because the completer may just be added but not yet included in the cache. I don't think this will work well for tab completion given the search overhead.
So, I think the completer path approach is better practically.

Thanks @DHowett for providing the context about AppX and MSIX packages.
It sounds there is no way for a MSIX package to ship and deploy a PowerShell module on install then (completer/feedback provider all depend on a module to be placed under the module path).

would modules also use this mechanism?

It would be great for modules to use the same mechanism. We can have a new field in the PrivateData section of the module manifest for completer metadata, and then Install-Module can take care of creating the completer file for the module in the completer path.

Have we really still not made module manifest parsing ignore unknown fields yet, so we can add things without crashing previous versions of PowerShell?
Absolutely what I would like to see is module that exports native argument completers, so you could put in a module manifest:
CommandsToComplete = @("kubectl")
And then PowerShell would autoload the module when it needed to complete for the kubectl native command.
But why do you need a new path variable and these new metadata files? Why isn't the metadata in the module enough on it's own?
P.S. Agree with daxian about MSIX -- "installer" packages that can't append to (PSMODULE)PATH (at least for the user scope) are not a solution for shipping command-line tools, or anything else for PowerShell.

Bump

Could we engineer a method to initiate the preloading of parameter autocomplete suggestions for a cmdlet immediately after the user enters the cmdlet name as the initial word in the prompt and hits space following the cmdlet's name?

Still relevant. Having many completion scripts slows down startup, even if they end up unused in the session.

@Jaykul

What if there are two of them?

Fair point, but the same issue occurs when multiple modules provide the same exported cmdlet, so there are existing rules that we could follow.


Add issue templates to PGE repo

opera-sds-pge repository is still in need of issue templates for pull requests, bug reports, feature requests etc...
@riverma has kindly provided some samples here as a starting point
Feature requests:
https://raw.githubusercontent.com/aria-jpl/coseismic_enumerator/main/.github/ISSUE_TEMPLATE/feature_request.md
Bug reports:
https://raw.githubusercontent.com/aria-jpl/coseismic_enumerator/main/.github/ISSUE_TEMPLATE/bug_report.md
Issues:
https://aria-jpl.github.io/#/?id=issues
Note links may require JPL vpn to access.

@collinss-jpl - thanks for setting this up.
My vote for templates is to start out with the aforementioned Feature Request and Bug Report templates and to set up issue forms for them. One benefit is the templates are based on GitHub standard issue templates, so many devs will feel familiar with filing them. The "Issue" template is probably unnecessary at this point since it overlaps with the earlier two.
@collinss-jpl @hhlee445 what do you think about starting with those templates? If you're happy with them, I'm happy to post a quick PR to set it up for this repo and /opera-sds-pcm

@riverma Templates look fine to me, was going to try and knock this ticket out this week but if you'd like to submit a PR I won't stand in your way

@riverma Templates look good. I agree with Scott.


⚠ BCR Bot has degraded performance

In 6af5fde, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 847 ms

Resolved: BCR Bot performance has improved in e92ed27.


Pod dependencies

How would you go about creating a pod with dependencies? Currently there is no use of CocoaPods in either the module project or the example app project.

In the example app project you can add a podfile and run pod install as usual.
That approach is used in this project https://github.com/CocoaPods/pod-template/pull/187
Some use that approach to add UI testing to the example app.

Funnily enough, I saw your pull request there pointing to this repo, thus the question. Won't doing this break the existing (generated) workspace?
I don't ask it for UI testing reasons, more because I'm creating pods that depend on other pods.

Hmmm, I think it should work. Because the only special thing that workspace is doing is depending on the module.
I haven't thought about this issue yet. Mainly because my only goal so far is to steal what AlamoFire is doing so well and distilling it into a reusable template. The problem is that I don't know any projects that are organized as beautifully as AlamoFire and also have CocoaPods dependencies.

Allright, I did the following steps:

Deleted existing workspace
Removed "link with" old framework
Added Podfile in example directory
pod install

Seems to work. Only thing for now, using your sample project, is that the image doesn't load. The bundle it refers to is this:

.../iOS Example.app/Frameworks/TestLib.framework

Whereas the image's path is actually:

.../iOS Example.app/Frameworks/TestLib.framework/TestLib.bundle/wk.png

Podfile:
platform :ios, '9.0'
inhibit_all_warnings!
use_frameworks!
source 'https://github.com/CocoaPods/Specs.git'

target 'iOS Example' do
	pod 'TestLib', :path => '../'
end

Thank you, great help. This is very good progress.

The thing is, I don't know if there should be a step in the configure script asking if you want to have pod dependencies, or just document what steps are needed for people that want it.

Replacing
let bundle = Bundle(for: type(of: self))
let image = UIImage(named: "wk", in: resources, compatibleWith: nil)

with
let bundle = Bundle(for: type(of: self))
let resources = Bundle(path: bundle.path(forResource: "AppwiseCore", ofType: "bundle")!)
let image = UIImage(named: "wk", in: resources, compatibleWith: nil)

fixes the image issue.
I imagine this is something that's needed when the module is installed via cocoapods. No idea about SPM or Carthage, never used those.

Thank you. In general I would like to provide extra instruction for people that would want this.
Personally I think CocoaPods as a dependency manager will go out of style immediately when SPM takes off. I think of CocoaPods as a publishing outlet but I prefer not to build it in as a dependency. That's why it will be "extra work" if people want to use it for their dependency manager. People might want to use Carthage or SPM and this should be compatible with that.
Sorry if all this sounds theoretical. I plan on replacing all my projects to be based off of this template so I am hoping to get it right :-)

Fair enough.
When do you think SPM will take off? Last I heard was that it's still in early stages.

Apple states is schedule for release as part of Swift 3.
Swift 4 is scheduled for Fall 2017, so I expect SPM before then.

Closing this.

Thanks for all the discussion here. I have added a note the the README.md referencing here.


Installation failures (using Yarn)

Having an issue installing that I think has to do with the usage of NPM (I am using Yarn). This could be a strawman though.
yarn add iam-policy-generator
error <my_package>/node_modules/iam-policy-generator: Command failed.
Exit code: 127
Command: npm run compile && node ./lib/generator/PolicyGenerator.js; tsc ./lib/generated/*.ts
Arguments:
Directory:<my_package>/node_modules/iam-policy-generator
Output:
><EMAIL_ADDRESS>compile <my_package>/node_modules/iam-policy-generator
> tsc

info: Fetching IAM policy metadata from https://awspolicygen.s3.amazonaws.com/js/policies.js
info: Saving policy file.
info: Generating TS file containing Supported IAM Services enum.
info: Generating TS file containing AWS Service Policies enums.
info: Generating TS file containing ServiceArn
info: library data built. Please import package and have fun!

Actually, it installs everything into node_modules and I can use it, but the error causes it to not persist into my package.json and would cause issues with my CI/CD.

It is a bug we need to fix

Looks like it's related to Typescript not being available on your machine (it's not your fault, since it is not a requirement). Installing tsc with npm I -g typescript should solve the problem. However, version 2.0 is going to include #19 that removes tsc dependency.

Is going to be solved by #19


Move #1609 from 3.3.0 to unreleased

This was accidentally added to the changelog for 3.3.0 after the release had been made – we've yet to release it, so it should be included in the unreleased section.

My mistake not noticing this, thanks Ollie 👍

Easily done :)


Changed default options to use InjectionToken instead.

Hi there and thanks for an excellent component!
This is my first ever pull request, so please have patiens with me ;)
Not sure what to change regarding versions and documentation or other files.
Please tell me if there is additional steps or other info needed.
Cheers!
Breaking changes:
This makes the options properties type safe by using an interface instead of strings.
The demo site is also updated with and commented out section in the app.module provides array.
Could not test since the version of angular and npm is old, sorry

Thanks for submitting this! Sorry I haven't merged this yet. I've been busy. I'll look at it soon :)

Thanks for submitting this.
This has been merged.
I've tested it on the demo & it seems to work fine.
I modified the syntax slightly & also updated the readme/etc.
Please let me know if you have any concerns with the changes I made.
Thanks!

Thanks @speige 👌


PIF build errors when using language mode v5

Is it reproducible with SwiftPM command-line tools: swift build, swift test, swift package etc?

[X] Confirmed reproduction steps with SwiftPM CLI. The description text must include reproduction steps with either of command-line SwiftPM commands, swift build, swift test, swift package etc.

Description
In grpc-swift we produce a fat binary of one of our products using swift build --arch x86_64 --arch arm64. The product uses the .v5 Swift language mode but this fails to compile:
$ swift build --arch x86_64 --arch arm64
error: 'somepackage': Some of the Swift language versions used in target 'SomePackage' settings are supported. (given: [5], supported: [4.0, 4.2, 5.0, 6.0])

SwiftPM appears to treat .v5 as "5" internally but is expecting "5.0" for PIF builds.
Changing .swiftLanguageMode(.v5) to .swiftLanguageMode(.version("5.0")) works for PIF builds but fails for a regular swift build.
Relatedly, there's a bug in the error message too:

Some of the Swift language versions used in target 'SomePackage' settings are supported.

should read

Some of the Swift language versions used in target 'SomePackage' settings are unsupported.

Expected behavior
Build should succeed.
Actual behavior
Build fails with an error.
Steps to reproduce

Download attached repro project
Run swift build --arch x86_64 --arch arm64

SomePackage.zip
Swift Package Manager version/commit hash
No response
Swift & OS version (output of swift --version ; uname -a)
Xcode 16 rc 1
swift-driver version: 1.115 Apple Swift version 6.0 (swiftlang-<IP_ADDRESS>.10 clang-16<IP_ADDRESS>)
Target: arm64-apple-macosx15.0

Same issue
Xcode 16.0


(PUP-6241) Only expect incorrect File exception on JRuby 1.7

Previously, our tests for an environment bug on JRuby 1.7 would execute
on any Puppet Server instance.
Since we have a regular JRuby 9k version of our Puppet Server jobs this
means this test, which expects a bug that only exists in JRuby 1.7
fails.
This patch updates the acceptance test to skip if Puppet Server is using
JRuby 9k.

CLA signed by all contributors.

Updated and retested both sides of the change locally.


SetLabelColor() does not work, SetLabel("[color]...") does.

What's wrong with the code below?
I am trying to set the label color using InputField.SetChangedFunc(), but the label color remains yellow. It works if I replace the SetLabelColor lines with i.e. SetLabel("[red]Name"). According to the tview wiki it is not necessary to call any Draw() function to redraw the label.
I'm executing the code on Linux, via PuTTY, so I suppose only the 16 ANSI colors are available to me.
package main

import (
        "github.com/gdamore/tcell"
        "github.com/rivo/tview"
)

func main() {
        app := tview.NewApplication()

        nameField := tview.NewInputField().SetLabel("Name")
        nameField.SetChangedFunc(func(name string) {
                if len(name) == 0 {
                        nameField.SetLabelColor(tcell.ColorRed)
                } else {
                        nameField.SetLabelColor(tcell.ColorGreen)
                }
        })

        form := tview.NewForm().
                AddFormItem(nameField).
                AddButton("Ok", func() { app.Stop() })

        if err := app.SetRoot(form, true).Run(); err != nil {
                panic(err)
        }
}

Sorry, I wanted to post this as a tview issue, not tcell.


Color Change using custom input feild

Please give an option to setColor using a custom input feild of my choice

Can you describe the problem in more detail?

I have a requirement that requires a custom input feild which can take the hex value and change in the color picker. but I can't see any way to set the hex color directly to the setColor. for now it requires some custom calculations from my side to find out the rgb, hsv values and create the color object and set it back. So it is good to have a method to do the same just like the hook useColor is doing

also it will be good to have some custom style acceptance as well

I have a requirement that requires a custom input feild which can take the hex value and change in the color picker. but I can't see any way to set the hex color directly to the setColor. for now it requires some custom calculations from my side to find out the rgb, hsv values and create the color object and set it back. So it is good to have a method to do the same just like the hook useColor is doing

ColorService is exported from the library, you can use it for custom color manipulation.
In your situation, you can use ColorService.convert() method, which takes 2 arguments: model and color (in the selected model), the result will return IColor object, which you can then pass to setColor().


Getting Various Warnings while running time series prediction

I'm trying to fit  GreyKite Model to my time series data.

I have attached the csv file for reference.

Since some of my target values are zero it tells me that MAPE is undefined.

Also, since I'm only forecasting one step into the future, it gives me an UndefinedMetricWarning : R^2 score is not well-defined with less than two samples.'

I have attached a few images displaying the warnings.

Any help to get rid of these warnings would be appreciated!

This is the code I'm using to fit the data:

`class GreyKiteModel(AnomalyModel):
def __init__(self, *args,model_kwargs = {}, **kwargs) -> None:
    super().__init__(*args, **kwargs)
    self.model_kwargs = model_kwargs

def predict(self, df: pd.DataFrame, ) -> pd.DataFrame:
    """Takes in pd.DataFrame with 2 columns, dt and y, and returns a
    pd.DataFrame with 3 columns, dt, y, and yhat_lower, yhat_upper.

    :param df: Input Dataframe with dt, y columns
    :type df: pd.DataFrame
    :return: Output Dataframe with dt, y, yhat_lower, yhat_upper
    columns
    :rtype: pd.DataFrame
    """
    df = df.rename(columns = {"dt":"ds", "y":"y"})
    metadata = MetadataParam(time_col="ds", # ----> name of the time column
                             value_col="y", # ----> name of the value column
                             freq="D"       # ----> H" for hourly, "D" for daily, "W" for weekly, etc.
                            )
    forecaster = Forecaster()  # Creates forecasts and stores the result
    result = forecaster.run_forecast_config(df=df, # result is also stored as forecaster.forecast_result
                                            config=ForecastConfig(model_template=ModelTemplateEnum.SILVERKITE.name,
                                                                  forecast_horizon=1,  # forecasts 1 step
                                                                  coverage=0.95,
                                                                  metadata_param=metadata
                                                                  )
                                            )
    forecast_df = result.forecast.df
    forecast_df = forecast_df.drop(columns=['actual'])
    forecast_df.rename(columns={'ds':'dt',
                                'forecast':'y',
                                'forecast_lower':'yhat_lower',
                                'forecast_upper':'yhat_upper' },
                       inplace=True)
    return forecast_df`

df.csv

Hi @Amatullah Could you try
import warnings
warnings.filterwarnings('ignore')

I tried that and the warning do get suppressed. I'm looking for a way to not get these warning altogether.
In my program I always need to forecast one step in to the future only, does that mean that the warning will always remain?

Hi @Amatullah the library raises these warning whenever the condition meets, but actually you could specify message in filterwarnings for example
warnings.filterwarnings("ignore", message="y_true is constant")

You could suppress all specific warning you want to suppress at the beginning of your code.

I had the same problem with warnings about holidays, I was able to suppress those warnings by changing events=None to explicitly declare the dict:

Like you, my response has zero values so I am getting the same y_pred and MAPE-related warnings in the output. I have opened issue #51 on this because what we want is to be able to control the amount of metrics being computed rather than forcibly suppress useful warnings which are there to show us what may be going wrong.