Morning Session Exercise Day 5

Hey Sanam, great design in your project!! The only thing I can say it is that you can add to your project responsiveness property...as you know is very imnportant today to all the codes, even it is not a project requirement...it is just suggestion...Have a nice weekend.

Hi. @sanamrahimi, congratulation. Your project look amazing.  👍 💯

Hey there @sanamrahimi

You did a great job 💯
Amazing design and implementation👍 🥇

You could add padding to the top of the nav bar. I think that would look more identical to the template.

Good Luck 👍


Option to not show main window/Notifications on startup

The main window always shows on startup and even though I tick 'start automatically at login'
it doesn't remain ticked when the window appears on startup. I'd also like to have an option to avoid having a Windows notification appear after closing the main window also.

Any chance you can get this done soon? Can't imagine it'd take much time and having to deal with every login is somewhat annoying.


Fixes typo errors

Fixed typo errors in lines 31 & 72.
Resolves #658

@MSFTTracyP - PR is ready for final review based on issue #658.  This is just a typo issue.
Thank you.
cc: @kenwith


extension missing on gene page

GO:1904776	regulation of protein localization to cell cortex IDA	Hersch M et al. (2015)
has extension
pom1 which is not displayed. However, it must have been in this run
because it is on the publication page
http://pombase2.aska.gen.nz/reference/PMID%3A26150232

It's there now I think.

yes, I think I was misreading the publication page, it wasn't there either ;)
all good now...


ParseError: KaTeX parse error: Expected 'EOF', got '\unsupported' at position 1: \̲u̲n̲s̲u̲p̲p̲o̲r̲t̲e̲d̲

ParseError: KaTeX parse error: Expected 'EOF', got '\unsupported' at position 1: \̲u̲n̲s̲u̲p̲p̲o̲r̲t̲e̲d̲
Version: 0.9.0

Could you clarify what your input is that generates this error?  It would appear to be \unsupported, but I'm guessing it was something else?

well it was just me there have made a typo.
A little question how to intergrade KaTeX into Marked ?

@BenjaminHoegh I still don't understand what you're doing that is resulting in that error. Could you provide, for example, an example file, or an input to the demo site that causes the problem?
Regarding Marked, a quick google search gives https://www.npmjs.com/package/marked-katex which might be useful.

It sounds like you resolved the issue, but if you're still having trouble, feel free to ask with more details.


OnInstallAppPerCompany

Hi
As in the below code when we run the app using F5 or Ctrl + F5 the trigger OnInstallAppPerCompany is not triggering.

Please Suggest
Thanks & Regards
Varun Reddy

Just a hunge...
This probably happens because you synchronize (default behaviour), therefore don't install, but upgrade.  Add this to your launch.

It will remove the previous version first, then actually perform an "install", which will trigger the code.
Note that your data will be wiped every time you deploy from VSCode!  So, only necessary when testing your install-codeunit.

I tested this.  I have the opposite behavior :-/.
It is always going to run that trigger, no matter what the schemaUpdateMode setting is.  #puzzled
Do notice that debugging does not seem to be an option.. .  don't know if that is what you are trying to do...

HI Waldo,
As you suggested i have put recreate in launch.Json but it does not seems to work in that way also the trigger OnInstallAppPerComany is not triggering.
Also is debug not possible for this trigger.
Please suggest?

In order to debug, you need code from the web client that triggers it for the moment. We have a backlog item to add more to how debugging works.
The why OnInstallAppPerCompany is not triggering needs investigating

It does trigger on my end though...

@kalberes  i am using March preview update which is 0.15.18771

I have submitted a request for the owning team. As soon as there is a response it will be logged here.

@bachii1010 I'm afraid I have not been able to reproduce this yet on the March preview you mentioned. Can you provide some sample version of the code you are using in the trigger? Or in the codeunit?

@HunterFeltman-MSFT  i am using the same code as in this document in the below link i additionally kept a message to check if the trigger is running or not.
https://docs.microsoft.com/en-us/dynamics-nav/developer/devenv-extending-application-areas

Please suggest

I'm not familiar with this particular example code, however, I can tell you that the OnInstallAppPerCompany trigger is going to be run in a UI-less 'background' session and as such will never display messages to users, so unfortunately you would never see things in the client.
If you are sure that the 'IsExampleApplicaitonAreaEnabled' function should be responding back with 'false' then you could check the DB to see if 'EnabledExampleExtension' code has actually executed (though it also seems that there are more 'quit early' checks inside that)
If you want to be really sure, I suggest creating a simple dummy table & page and then populate said table in this code.
for example:
table 50000 TempTable
{
    fields
    {
        field(1;PK; Integer) { }
        field(2;Value; Text[250]) { }
    }

    keys
    {
        key(PK; PK) { Clustered = true; }
    }
}

page 50000 TempPage
{
    PageType = Card;
    SourceTable = TempTable;

    layout
    {
        area(content)
        {
            group(GroupName)
            {
                field(Value;Value) { ApplicationArea = All; }
            }
        }
    }
}

codeunit 50000 TempCodeunit
{
    Subtype = Install;

    trigger OnInstallAppPerDatabase()
    var
        tempTable : Record TempTable;
    begin
        tempTable.PK := 1;
        tempTable.Value := 'Hello, World!';
        tempTable.Insert();
    end;
}

Note: Use the 'recreate' schemaUpdateMode to assert that data is cleared from this table before you F5 multiple times with this code.
You might have to adjust object ids, but with this you could then navigate directly to your page and see if data was inserted into the table (indicating that the trigger did, in fact, run)
To directly navigate to the page, use a ?page=50000 query string parameter to the webclient URL in your browser

@HunterFeltman-MSFT  thanks for the update yes i tried with your example it is executing i will go into further details what is going wrong with ours.
Thanks once agian!!!!!

No problem, thanks for letting us know about the potential issue. Hope you figure out the problem in the ApplicationArea example.


Use umi_count instead of duplicate_count in compliance with AIRR v1.4

Remove index_chains_legacy because the numba implemenation seems to work fine
Use 'umi_count' instead of 'duplicate_count'

Closes #478

[x] CHANGELOG.md updated
[x] Tests added (For bug fixes or new features)
[ ] Tutorial updated (if necessary)

@zktuong FYI in this PR I update scirpy to use umi_count instead of duplicate_count which is recommended since AIRR v1.4 (see release notes). Maybe you want to do the same thing in Dandelion?


SW dimming: conflict with flux even with "avoid gamma manipulation"

Before opening the issue, have you...?

[X] Searched for existing issues
[X] Looked through the wiki
[x] Updated MonitorControl to the latest version (if applicable)

Describe the bug
(I'm using a lenovo p27-h, but since this issue only occurs when the software dimming option is enabled I'm not sure if it's a monitor specific issue.). Basically, even though I have "ticked the avoid gamma table manipulation" option, I'm still seeing an issue where whenever flux changes the white-point (or it's changed manually via flux), the software dimming resets for a few seconds.
Note that this is different from the behavior that would occur if the "avoid gamma table manipulation" is not enabled. If gamma table manipulation is used, then the software dimming will always reset immediately after being applied. But when shades is used, for some reason software dimming only momentarily resets whenever the white-point is changed by flux.
Steps to reproduce

Tick "avoid gamma table manipulation"
Have flux enabled
Dim the monitor low enough so the SW dimming kicks in
Manually change the white point in flux (or wait for dusk for it to adjust automatically)
Every time it adjusts, software dimming turns off for a few seconds before turning back on.

Expected behavior
Display brightness remains constant even when flux manipulates white point.
Anything else?
Not sure if there's any log files I can attach. Let me know and I'd be happy to do so.
Environment Information (please complete the following information)
- macOS version: 11.6.1
- Mac model: MacBook Pro (16-inch, 2019)
- MonitorControl version:  4.0.1
- Monitor(s): Lenovo p27-h
- Apple Silicon/M1 (yes or no): no

Hey @krackers,
can you post a video (made via your phone) about the issue so I might better understand what is happening? Thank you for your help!

Hi, thank you for the quick response. I'm a bit paranoid to take a video since this is a company-issued laptop that has a watermark. However, I was able to narrow down the issue to ICC profile changes.
In particular, I can reliably replicate the issue on my end (even without flux) by just changing the icc profile (via system preferences > displays), which causes the software dimming to reset for a few seconds before turning back on. Flux is known to explicitly modify the ICC profile every time it does the white-point change (see https://justgetflux.com/news/2014/10/28/profile.html), which is probably what caused the issue.
As a workaround for me, I was able to use the linked command to disable flux's ICC profile modification. Feel free to close this issue if you are not able to replicate the ICC profile change induced software dimming issue.

Uhh. A watermark? On the screen? That's draconian. I'll try the ICC profile thing. MonitorControl does alter the profile (which actually defines the gammatable) but that should not happen when "avoid gamma table manipulation" is not engaged.

Hi @krackers - I am unable to reproduce this issue. I did the following:

enable Avoid gamma table manipulation.
Disable Hardware DDC control (so software control kicks in).
Dim the display somewhat.
Changed the whitepoint by moving the topmost slider in f.lux preferences.

It seems to work as expected. I am running f.lux 41.5. I tried the same with combined HW/SW control as well.
When the color profile profile is changed in System Preferences, it initiates a full display reconfiguration - this removes the shade briefly - but this is normal and expected.

Is there maybe some other trick to reproduce this issue? I know that f.lux had issues with some Macs on Monterey and there were some workarounds for it, maybe they do a full color profile reset somehow and that's what triggers a display reconfiguration.
You could check the console logs and see if there is a reconfiguration every time the issue happens: https://github.com/MonitorControl/MonitorControl/discussions/851

Thanks for the investigation. Those repro steps seem correct  to me. I do indeed see the following logged
monitor control job died because of sleep or reconfiguration

whenever the dimming gets disrupted due to f.lux's whitepoint changes. Given that disabling f.lux's icc modification fixes this issue (albeit at the cost of color temperature disruptions when switching between internal & external monitor).
We can close and move to the discussions section if you want, since it seems whatever particular combination that causes this is rare enough, it seems to be an issue on f.lux end (since you stated shade disruption on display reconfiguration is expected behavior), and there exists a workaround.

Yes, it seems f.lux does something on your setup which is not happening on mine and it causes a full display reconfiguration which triggers the destruction and recreation of the shade overlays (the alternate for gamma dimming). I think the problem here is with f.lux.
Thanks for the confirmtion, I'll close this issue now.

Or even better, I'll convert this to a discussion so others with this issue might be able to figure out what is happening.


Bump version to 0.4.9

Reverting https://github.com/astral-sh/uv/pull/5927 because it changed a bunch of versions and CI is failing.

https://github.com/astral-sh/uv/pull/7276 reverted the lock changes so I've rebased onto main and dropped the revert of https://github.com/astral-sh/uv/pull/5927


[FEATURE] - Bandwith limiting/reservation for different components/services (network QoS)

Is your feature request related to a problem? Please describe.
In some use cases, bare-metal servers only have 2 network adapters in LACP Mode (OVH for example).  So your WAN, harvester management and vlans all go trough the same 2 physical adapters.  This setup works just fine, but when dealing with network-intensive processes (big vm images, longhorn replicas rebuild) - we end up with packet loss on VM networks causing all kind of unexpected behavior.
While I agree that this setup is not the recommended one, IMHO, it will happen more often that not.
Describe the solution you'd like
It would be nice to be able to easily configure bandwidth limit/reservation based on the speed of the underlying bond/network interface (my case 50G LACP) for the following components:

VM Networks
VM Migrations
Longhorn
Images-related operations (this is the most problematic one for our use case - when a new vm created and attached to 2-3 different images, this is when the problems happens most often)

One thing that might help with the usage right now is the #1055 feature in v1.1.x.
Though I don't really expect that kind of issue happens on 50G bonded nic. #2541 should be the main issue for the network QoS.
Bump the priority of #2541 in this case. And close this as the duplicate issue for #2541


How to use in an environment variable in values?

sample
In .gitlab-ci.yml  set
GITLAB_NAMESPACE: $CI_PROJECT_PATH_SLUG-$CI_COMMIT_REF_SLUG

In values.yaml use
postgresql_volume_config:
  driver: local-persist
  driver_opts:
    mountpoint: /data/{{ gitlab_namespace }}-postgresql/

Hi @radist,
Thanks for the idea. Funnily enough we are currently discussing a similar usecase internally in our company.
Originally the idea behind nothelm.py was that it was used together with a git repository where you would have one checked-in yml file per environment you want to deploy to. But as far as I can tell from your usage example, you have a slightly different usecase.
At the moment, we dont have environment variable interpolation in the values.yml. The code for loading is pretty straightforward:
https://github.com/neuroforgede/nothelm.py/blob/8c23722f049e1bfafba51180f95231fb42199585/nothelm/lib/template.py#L10
We would probably want to expose the env variables with a prefix. Maybe Something like {{ nothelm.env.gitlab_namespace }}.

@radist would this be enough for your usecase? Do you need env var expansion in values.yaml or do you want it in the template?

Yes, it's enough in values.yml. May also be useful in the template

We would probably want to expose the env variables with a prefix. Maybe Something like {{ nothelm.env.gitlab_namespace }}.

I think the env prefix is enough?

I added a new version that allows templating of string variables in values files if their name ends with ".j2"


Keep consistent behaviors between pasting one empty folder and more than one empty folder against file shares

Storage Explorer Version: 1.22.0-dev
Build Number: 20210924.14
Branch: hotfix/1.21.2
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Big Sur 11.4
Architecture: ia32\x64
How Found: Ad-hoc testing
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> File Shares.
Create a file share-> Create three new empty folders (f1, test1, test2).
Copy the folder test1 then paste it into the folder f1.
Observer the activity log.
Back to the root -> Select the two folders test1 and test2.
Copy the two folders then paste them into the folder f1.
Observe the activity log.
Check whether the behaviors are consistent.

Expected Experience
The behaviors are consistent.
Actual Experience
The behaviors are inconsistent.

Additional Context

For ADLS Gen2 blob containers, always display '0 items transferred' regardless of how many empty folders are pasted.
Related issue 3193.

Let's understand why the numbers are different and open a bug against AzCopy if needed.

For single folder copy, the AzCopy command looks like
cp root/src/* somewhere/src/
And since root/src/ is empty, AzCopy thinks there is nothing to copy and exits.
For multiple folder copy, the AzCopy command looks like
cp root/* somwhere/
with list of files as:
src1/
src2/
AzCopy lists under root and founds the two folders and thus creates the two folders.
I think for us, this is a low priority bug. If the folder to copy is empty, the user can always create the folder at the destination instead.

This work needs a new AzCopy flag planned for their next release.

For file share, the activity now includes the directory in the job summary (e.g. 1 item if copying an empty directory, etc.). For blob/gen2 the behavior will remain the same (e.g. 0 item if copying an empty directory, 2 items if copying 2 empty directories) because the AzCopy flag doesn't work for blob.

@JasonYeMSFT
Verified this issue on the build main 20220926.1. The current behavior as below

For file share, display the correct number when copying empty folders for file share.
(1 item if copying an empty directory, 2 items if copying 2 empty directories)
For ADLS Gen2 blob container.
(0 item if copying an empty directory, 0 items if copying 2 empty directories)

BTW, Is there a typo in (e.g. 0 item if copying an empty directory, 2 items if copying 2 empty directories)

For file share, the activity now includes the directory in the job summary (e.g. 1 item if copying an empty directory, etc.). For blob/gen2 the behavior will remain the same (e.g. 0 item if copying an empty directory, 2 items if copying 2 empty directories) because the AzCopy flag doesn't work for blob.

Oh, sorry there is a typo. For gen2 and blob, empty directories will be ignored.

@JasonYeMSFT
Thank you for the response, The behaviors are the same as the comments. Will close this issue.


Inlay hint for @InjectMock annotated fields to show injected mocks

Tasks

Implement an inlay hint for @InjectMocks fields with the list of @Mock and @Spy fields that can be/are injected.
The inlay should be a block type one.
This would involve bringing in Kotlin support as well into the plugin.

To be clarified

Look into the logic of how mocks are injected and based on what criteria.
"Constructor injection; the biggest constructor is chosen"

What happens if there are multiple constructors with overlapping or completely different parameter list, with the same number of parameters?

com.intellij.codeInsight.daemon.ImplicitUsageProvider might be useful to disable highlighting @Mock and @Spy fields as unused if they are used only as injected mocks in @InjectMocks fields.

Resources

InjectMocks javadoc

For now, this is not straight-forward how to properly account for all possible cases from where mocks can come from. Closing this issue for the time being.


Moved CI code into continuous_integration module

Codecov Report

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

@@           Coverage Diff           @@
##           master      #80   +/-   ##
=======================================
  Coverage   92.88%   92.88%
=======================================
  Files           8        8
  Lines         492      492
=======================================
  Hits          457      457
  Misses         35       35

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 a01e4bd...3020bd7. Read the comment docs.


Unique Device Identifier and Device key are different or not?

yes, they should be different but can be same for now


embassy-boot: Copy paste error for SingleFlashConfig

It seems that here  the impl for SingleFlashConfig is suffering from a copy-paste mistake. The active, dfu and state functions seem to return the wrong types.
-    fn active(&mut self) -> &mut Self::STATE {
+    fn active(&mut self) -> &mut Self::ACTIVE {
        self.flash
    }
-    fn dfu(&mut self) -> &mut Self::ACTIVE {
+    fn dfu(&mut self) -> &mut Self::DFU {
        self.flash
    }
-    fn state(&mut self) -> &mut Self::DFU {
+    fn state(&mut self) -> &mut Self::STATE {
        self.flash
    }

It just so happens that functionally there's no difference, as all 3 types are the same. This means that this is merely a refactoring issue.

Seems like this functionality was reworked around commit 94046f30ffefc96a7b110ed1d9bb60d64cb4780f and can be closed now.
cc: @Dirbaio


Address bootstrap payload changes

This change is in sync with changes happening in the RSTUF API.
Reasons for this change and API pr you could see here:
https://github.com/repository-service-tuf/repository-service-tuf-api/pull/551
Signed-off-by: Martin Vrachev<EMAIL_ADDRESS>

It is expected the errors as this change has a breaking compatibility.


Eliminate render blocking?

Is there a way to stop this from render blocking ?

It's not something that is related to this library, but how you load your stylesheets. Read this article, it might help.
If the issue is solved, please close it! ;D

Are we allowed to concatenate the css file into one with others ?

Of course, animate.css is just plain old CSS, you can do anything you do with CSS.

Just wanted to confirm I wasn't breaking the license terms

Just keep the header with license info in the file and you're good, as animate.css is MIT licensed.

Closing this as I think it's solved. Fell free to re-open it if you need.


Unpin OTel version

Description
As per our discussion on Wednesday, I am unpinning OTel SDK/API. We will instead keep an eye on releases and adjust the exporter and distro accordingly.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.

Is there a way to make sure https://github.com/Azure/azure-sdk-for-python/issues/31164 does not happen again? Maybe we should create a checklist of things to manually smoke test before releasing?

Is there a way to make sure #31164 does not happen again? Maybe we should create a checklist of things to manually smoke test before releasing?

That issues was that the distro did not support the new version yet. Unpinning solves that, but to prevent breaking changes from unpinning, I've added a checklist to the OneNote. Will send on Teams


Fix bug when running non-local annotator instance.

Describe changes
zenml stack up attempts to start a local Label Studio instance even though the annotator is configured to use a non-local instance. I fixed it by checking if the instance URL is the same as DEFAULT_LOCAL_INSTANCE_URL before starting/stopping local annotator daemon.
Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[ ] If my change requires a change to docs, I have updated the documentation accordingly.
[ ] If I have added an integration, I have updated the integrations table and the corresponding website section.
[ ] I have added tests to cover my changes.

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

This looks good to me! Thanks for making this PR, @sheikhomar! I'll get it merged in.


Latest commit breaks debugging

Operating system or device - Godot version:
macOS 10.12 (Sierra) - Godot-master c67e3a485dedae96b82c3356d5f45ab0509d7759
Issue description (what happened, and what was expected):
This commit somehow broke debugging: when playtesting, nothing gets printed in the console and nothing gets shown in the debug window.
I had to revert to the commit before this one in order to get debugging working again.
Steps to reproduce:
Just play test any scene.

CC @punto- and @Faless.

Note that debugging runs over network (even in the same machine). That's likely the "somehow.

@vnen Gah, silly me.
How do I make that work, then?

@Keyaku He meant that for potential developers that would fix the bug, not as a hint for you that you might be doing something wrong ;) Your best bet is to waiting for the bug to get fixed by whoever knows how to do it :)

Can confirm :smile:

Ok, finally found the issue.. the editor listens on IPv4, while the game polls on IPv6... :open_mouth:
It mainly comes from here: drivers/unix/tcp_server_posix.h:43 or here: tools/editor/script_editor_debugger.cpp:1103
@punto- any reason listen has ipv4 as default? (or @Faless, if you know)

Can confirm as well on Windows 10, 914015f3b63dd956e72ea937d46ea4b2db005ada

Repro'd on OSX Sierra, 8321e48a .. also looks like print calls aren't printing to the Output window in the editor (they still appear in the terminal output when godot is ran from the CLI, tho)


What is the work around of "ERROR: Unable to download webpage: HTTP Error 429: Too Many Requests (caused by HTTPError());"

Checklist

[ ] I'm asking a question
[ ] I've looked through the README and FAQ for similar questions
[ ] I've searched the bugtracker for similar questions including closed ones

Question

WRITE QUESTION HERE

please help me!!! I have spent more than 2 months to make my music app.After two data is not working.

@dstftw

@dstftw what is the workaround for this?


put JSON lists together

i don't think this is an unreasonable change, or if you want @sindresorhus, I could change it to be underneath the javascript, since JSON is a subset of javascript data

like so?

like so?

GeoJSON could be indented too.


Improve Editor reference handling

This PR is based on the discussion in #110. It does not make any functional changes, but rather suggests an improvement of the internal API.
The part I am trying to improve is the following method from the Editor struct, which gets both a reference to itself (&mut self), but also an additional external reference to itself (self_ref: &Arc<Mutex<Editor>>).
fn do_rpc_with_self_ref(&mut self, cmd: EditCommand, self_ref: &Arc<Mutex<Editor>>) -> Option<Value>

This PR highly borrows from theRpcPeer implementation. It makes Editor a pub struct Editor(Arc<Mutex<EditorState>>);. The previous Editor is renamed to EditorState.
The question would now be, if this change makes the code easier to understand, or not? If not, I would say we simply keep the current implementation! If yes, before merging it, we may need to consider, for the sake of consistency, which methods go into Editor and which in EditorState (currently only the ones needed to make the code to compile are in Editor).

@rkusa Thanks for sending this PR, and sorry for taking so long to review it.
I honestly can't decide whether this is an improvement or a sideways move. The signature of the downgrade method is not very appealing to me - people will have to deal with both Editor and EditorState and it's not 100% obvious why one or the other is being used.
Given that the goal of this patch is clarity, what do other people think? I'll happily merge it if there's consensus it's an improvement.

No problem! When reflecting on the PR now, I also think that a sideways move probably describes this PR well. That being said and since it is not an obvious improvement, I think it is maybe not worth the time to invest further in it. Which is why I am also totally fine with just closing this PR to move on with focusing on actual plugin functionality instead. I still appreciate the feedback, so thanks for having a look anyway! 🙂


styles={{border: "none"}}

Is not respected.

I was able to go over it by adding a className with border: none
like this:
<Iframe url="some url"
        width="100%"
        height="10780px"
        id="myId"
        loading="Loading...."
        className="myClassname"
        display="initial"
        position="relative"
        allowFullScreen
        className="myClassname"/>

and then in my css
.myClassname{
border:none
}

Duplicate of #51  I think


Ship InSpec 2

Manhattan fixed with the Gemfile update:
http://manhattan.cd.chef.co/job/chef-trigger-ad_hoc/91/downstreambuildview/

Thanks @tas50. I think this is good to go.


Add Terraria module

Adds a standalone Terraria module with all its dependencies. Will be replaced by https://github.com/CubeCoders/AMPTemplates/pull/45 if its feature request ends up being implemented

Console doesn't function on windows. To look into further.

Make sure to manually grab the configtemplate when teesting, or to change the update stage to use my fork, since it will otherwise fail to template merge. WIndows console still does not work, so only autosaves and process-killing are available.


BACKEND_ENDPOINT_OVERRIDE parameter has an incorrect description

The description of BACKEND_ENDPOINT_OVERRIDE states that this environment variable is Useful when deploying outside OpenShift deployed AMP. Currently the value is already set to the route of backend-listener so the environment variable is no longer required when deploying APIcast outside the 3scale project. This has also been reported in THREESCALE-2462.
However, when deploying APIcast in the another project but in the same OpenShift cluster that contains 3scale API Manager, BACKEND_ENDPOINT_OVERRIDE can be used to override the public route for backend-listener and set it to the OpsneShift backend-listener service address.

It used to be the case. But then we switched the logic entirely and now it is useful when deploying inside the same cluster, so it can be used to use internal service over external Route.

Fixed in #1063


cps: add support for function pointer calls

isCpsCall assumes that the callee is always a nnkSym, which is not the
case when the callee is a function pointer stored inside an object.
Blocked by nim-lang/Nim#17836, but confirmed working with @saem
workaround.

Nicely done. :+1:


problem in scrapping algorithms section

Scraping link no: 244 Link:  http://www.geeksforgeeks.org/count-number-of-bits-to-be-flipped-to-convert-a-to-b/
Traceback (most recent call last):
File "g4g.py", line 106, in
scrape_category(categoryUrl)
File "g4g.py", line 82, in scrape_category
link_soup = BeautifulSoup(requests.get(link).text)
File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 65, in get
return request('get', url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests/api.py", line 49, in request
response = session.request(method=method, url=url, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 461, in request
resp = self.send(prep, **send_kwargs)
File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 567, in send
adapter = self.get_adapter(url=request.url)
File "/usr/local/lib/python2.7/dist-packages/requests/sessions.py", line 646, in get_adapter
raise InvalidSchema("No connection adapters were found for '%s'" % url)
requests.exceptions.InvalidSchema: No connection adapters were found for ' http://www.geeksforgeeks.org/count-number-of-bits-to-be-flipped-to-convert-a-to-b/'

@eldiago Resolved.

thanx mate...

:+1:


az dt is not in the az command group

I updated the azure cli to the newest version and install/update the azure-iot extension as proposed in the article.
Testing the installation I received the following error:
az: 'dt' is not in the 'az' command group. See 'az --help'. If the command is from an extension, please make sure the corresponding extension is installed. To learn more about extensions, please visit https://docs.microsoft.com/en-us/cli/azure/azure-cli-extensions-overview
Any help will be appreciated.

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

ID: 28bf2361-65e2-81e1-ebbc-8e4fc2ca11f3
Version Independent ID: 4aaf2947-de74-2678-dc42-a9f1d7a4e8d6
Content: Create an Azure Digital Twins instance - Azure Digital Twins
Content Source: articles/digital-twins/how-to-set-up-instance.md
Service: digital-twins
GitHub Login: @baanders
Microsoft Alias: baanders

Hi,
I already solved the issue. I had to remove the deprecated azure-cli-iot-ext as follows:
az extension remove --name azure-cli-iot-ext
Now is working as expected.

@pauldj54 Glad to know that you have self resolved and hence closed the issue. Thanks for the update.

@pauldj54  Thank you for the description of and resolution for this issue! I have updated the doc to add a check for the legacy extension and instructions to remove.


Capture build number

For some reason I thought we did this already, and we could just display them in xticks. Apparently not. So let's do it.

capture_build_numbers@550 aka 20200503.1 vs master ewma over 34 builds from 445 to 548

Keep in mind that build number is customizable, so this should probably be optional whether to add it as tick labels.

On the other hand, it is the thing that is displayed in DevOps (build id is only in the URL afaik), so I would actually use the build number exclusively in texts<EMAIL_ADDRESS>vs master ewma over 34 builds from 20200403.1 to 20200502.1

Yes, I think that’s the way to go, it’s the user-visible name of the build. The build id is still convenient for sorting and of course to link to actual build pages.


Add AsyncAPI instances

https://github.com/IBA-Group-IT/IoT-data-simulator/blob/fe82911139dccd881dc3aa2c841d473832d2dc38/services/src/main/resources/asyncapi.yaml
https://github.com/zijpn/npg-server/blob/master/schema/api/npgapi.json
https://github.com/Sinapse-Energia/IoT-Device-Simulator/blob/master/APIs/SCINAP/json/scinap_api_asyncapi.json - and MVP

Added.


Error go format upstream field

Summary
We moved from 0.2.2 to 0.3.0 today , and we are using kong v1.0.0-alpine moved from 0.14.1
routes services and everything is there and the migration went fine, but we are getting errors in kong-ingress-controller
.
Kong Ingress controller version
0.3.0
Kong or Kong Enterprise version
1.0.0-alpine
Kubernetes version
1.10.11
Environment

Cloud provider or hardware configuration: AWS
OS (e.g. from /etc/os-release): debian 9

What happened
We moved from 0.2.2 to 0.3.0 today , and we are using kong v1.0.0-alpine moved from 0.14.1
routes services and everything is there and the migration went fine, but we are getting errors in kong-ingress-controller . after migration we cant login we receive Kong Error An invalid response was received from the upstream server.
from log kong-ingress pod i see:
I0110 10:49:28.510073       6 controller.go:128] syncing Ingress configuration... I0110 10:49:28.528584       6 kong.go:1075] cert: 0xc000047a90 E0110 10:49:28.532580       6 kong.go:209] err fetching upstream mock-http-server.8080 from Kong: json: cannot unmarshal number<PHONE_NUMBER>.15 into Go struct field Upstream.created_at of type int64 E0110 10:49:28.532655       6 controller.go:131] unexpected failure updating Kong configuration:  syncing targets: json: cannot unmarshal number<PHONE_NUMBER>.15 into Go struct field Upstream.created_at of type int64
Expected behavior
Everything should work as before
Steps To Reproduce
1.Migrate from versions mentioned

The original issue in this post is fixed with Kong 1.0.1.
Thank you for reporting the issue.


Part 1 Fix beforeEachModified conditional checking

Refactor to fix issues where beforeEachModified conditional checking wasn't working as expected, as in some test cases, it was getting reset prematurely.
Updated behavior such that if the plugin needs to create a new beforeEach, it inserts it either after any test setup calls or at the top of the test module.
As part of the updated behavior, the refactor also includes checking for any custom "hooks" name passed into the test module's function.
Testing done on:

this stand-alone repo
a simple ember app
a test file in a larger, private ember app

Are there unit tests for this project. That might help catch these kinds of cases.

Are there unit tests for this project. That might help catch these kinds of cases.

There are tests, but they are snapshot-based and not as strict unit tests. I'm using https://github.com/babel-utils/babel-plugin-tester as the test infra. As cases come up, I add a test input and verify the snapshot output. Offline and outside of this project I also test on a simple ember app.

Just a few small changes, but overall it looks GTG! Thanks, @glnster!


I need to add Rakugo Project Settings window option to use project_setting_overwrite.

It must be done to make easy to work with other example projects begin just a branches from #211

partialy done in #213
I need to add :

project name
start scene

done in #219


Evaluate aggregation robustness

Nan/None values were contagious when aggregating metrics in evaluate function. For example, both [1,2,3, np.nan] or [True, False, None] would aggregatre to np.nan instead of 2 and 0.5 respectively.
I changed the averaging math used in out specific aggregation functions to avoid this - Note that the general aggregation is still vulnerable to this, but we need this fix now to hopefully address https://msdata.visualstudio.com/Vienna/_workitems/edit/3600498

What's the reason we can't fix this for all evaluators right now? And which evaluator aggregation functions are still vulnerable?

I changed the averaging math used in out specific aggregation functions to avoid this - Note that the general aggregation is still vulnerable to this, but we need this fix now to hopefully address

Discussed offline.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-evaluation


Cherry pick nil pointer fix for ISMPolicy issue

Description
Cherry pick nil pointer fix from upstream and apply to temporary branch

 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.


Strange Uplink ?!?

Hey,
i´m using the "send-class-A-OTAA.ino" example and i get strange messages in Serial Monitor after sending the counter.
Something like this:
Sending: Counter-34
⸮⸮⸮!⸮⸮⸮⸮⸮_⸮Dħ~=d]s⸮O⸮"*⸮⸮F⸮⸮^⸮⸮2:
I$\⸮Ӭb⸮⸮⸮y⸮⸮7m⸮⸮N⸮lV⸮⸮ez⸮⸮x%.⸮⸮⸮⸮⸮tK⸮⸮⸮p>⸮fH⸮a5W⸮⸮⸮⸮⸮⸮⸮iَ⸮⸮⸮⸮⸮U(ߌ⸮⸮
⸮⸮BhA⸮-⸮T⸮
Do i have to connect all DIO pins? i only used DIO0 and DIO1.

I´m using a Atmega328p running with 3.3V at 8MHz

Sometimes these ugly messages include partially my appKey, devEUI or other things from Serial.println


don't use Port forwarding in v1.0.0

I'm following this manual with slight changes

Install 1.0.0 version of docker

Remove all previous files

$ rm .boot2docker/boot2docker.iso
$ boot2docker delete

Create new vm image

$ boot2docker init
2014/06/12 15:39:17 Creating VM boot2docker-vm...
2014/06/12 15:39:17 Apply interim patch to VM boot2docker-vm (https://www.virtualbox.org/ticket/12748)
2014/06/12 15:39:18 Setting NIC #1 to use NAT network...
2014/06/12 15:39:18 Port forwarding [ssh] tcp://<IP_ADDRESS>:2022 --> :22
2014/06/12 15:39:18 Port forwarding [docker] tcp://<IP_ADDRESS>:2375 --> :2375
2014/06/12 15:39:18 Setting NIC #2 to use host-only network "vboxnet0"...
2014/06/12 15:39:18 Setting VM storage...
2014/06/12 15:39:27 Done. Type `boot2docker up` to start the VM.

Run it

$ boot2docker up
2014/06/12 14:50:39 Waiting for VM to be started...
...........
2014/06/12 14:51:01 Started.
2014/06/12 14:51:01 Auto detection of the VM's IP address.

Run docker image

$ docker run -d -P training/webapp python app.py

Find the port

$ docker port c6073a9fc281 5000
<IP_ADDRESS>:49153

Check if it works

curl localhost:49153
... hangs!!!

Check that in VirtualBox it works

$ boot2docker ssh "curl localhost:49153" 2>/dev/null
Hello world!

And by the way, manual port forwarding doesn't help either!

$ i=49153;
$ VBoxManage modifyvm "boot2docker-vm" --natpf1 "tcp-port$i,tcp,,$i,,$i";
$ VBoxManage modifyvm "boot2docker-vm" --natpf1 "udp-port$i,udp,,$i,,$i";

What should I do to make it work?
ps:
MacOS 10.9.3

@SvenDowideit, a note on use cases for the port forwarding. If you're working with TLS or any x.509 based PKI and STARTTLS and you've generated self-signed certs with a CN of localhost for development you do need to forward ports to localhost because https://{boot2docker ip}:{port}/ will not be localhost and the certificate will be rejected. Admittedly most people should generate properly named certificates for production, but boot2docker is a development tool and localhost is a pretty common CN when doing development. of course the work around is to just manually forward the ports in VirtualBox...


allow Serilog v2 on nuget package

The nuget package has a constraint on v1 of serilog.
It would be handy to have a serilog v2 capable nuget package.

#8

Closed by #8.


Add ability to disable CupertinoSegmentedControl

Summary
Add the ability to configure enabled or disabled segments in CupertinoSegmentedControl. The idea is to pass a segmentStates map, where the user can set the state according to segment key. User can also set background and text colors when the segment is disabled.
Demo
https://github.com/user-attachments/assets/4a02da02-a0fb-4ded-a271-033a8dc79ac3
Related issue
Fixes https://github.com/flutter/flutter/issues/52105
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Thank you for the suggestion! It looks better with Set type. I pushed the update.

@QuncCccccc Thank you so much for your thorough review and sorry for the delayed response, I've just come back from travel :). I've updated the code based on most of your suggestions, except for those concerning example and disabledColor, I'm eager to hear back from you again. Here is the updated demo video for the example:
https://github.com/user-attachments/assets/a4cab3b2-a11d-407b-aea8-ebe04aca73b0


Add retrieval of template id for Qualys WAS

These changes add a new functionality to qualysapi which allows to get the template id for the vulnerability data from Qualys WAS. It has been already in production in https://github.com/HASecuritySolutions/VulnWhisperer for a long time but it was maintained as a qualysapi fork until now.

@paragbaxi I believe this error has to do with the configuration of pytest itself?
pluggy.PluginValidationError: Plugin 'pytest_cov' could not be loaded: (pytest 3.3.0 (/home/travis/virtualenv/python2.7.14/lib/python2.7/site-packages), Requirement.parse('pytest>=3.6'))! make: *** [test] Error 1
It breaks when checking with python 2.7 which would make sense if requirement is >=3.6, but it shouldn't fail on 3.6 then. The error is the same in both.

agreed. have any advice on how to fix?

I have no clue... 😥

As the fail is not related to the PR code (the other tests are passing) but to the pytest config file used in the Travis CI instance, could the PR be checked for a merge? :)

@paragbaxi whenever you are able, not in a hurry, could you update the release in PyPi? Thanks a lot for merging the PR :)

SyntaxError: encoding problem: future_fstrings
On Thu, Mar 7, 2019 at 3:50 AM Quim Montal<EMAIL_ADDRESS>wrote:

@paragbaxi https://github.com/paragbaxi whenever you are able, not in a
hurry, could you update the release in PyPi? Thanks a lot for merging the
PR :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/paragbaxi/qualysapi/pull/52#issuecomment-470438751,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA6R8asLs6fAVXIMWGaiFQ3qnnxWj-5_ks5vUNLLgaJpZM4bQm7A
.

done

Thanks a lot!


Handle array values for JSON columns

Solves Issue #23684
It allows you to pass an array as a parameter when bulk updating models' JSON column.

Not changing this on a point release without much more description in the PR.


[error] Error loading plugin 'fastlane-plugin-firebase_app_distribution': Unable to activate google-apis-firebaseappdistribution_v1-0.3.0, because google-apis-core-0.3.0 conflicts with google-apis-core (>= 0.11.0, < 2.a)

Issue Description
When using it on Unity Cloud Build I have this issue :

[error] Error loading plugin 'fastlane-plugin-firebase_app_distribution': Unable to activate google-apis-firebaseappdistribution_v1-0.3.0, because google-apis-core-0.3.0 conflicts with google-apis-core (>= 0.11.0, < 2.a)

Configuration
  lane :ios_dev_app_distribution do #lane Firebase App Distribution
    desc "Pushing iOS build to Firebase App Distribution"
    firebase_app_distribution(
        app: ENV['FIREBASE_APP'],
        service_credentials_file: ENV['GOOGLE_APPLICATION_CREDENTIALS'],
        ipa_path: ENV['UNITY_PLAYER'],
        release_notes_file: ENV['RELEASE_NOTES_PATH'],
        groups: ENV['FIREBASE_GROUPS'],
        debug: ENV['FIREBASE_DEBUG_OUTPUT']
    )
  end

Complete verbose output
You can find it attached
Environment
Look at my gemfile.lock

Library
Version

fastlane
[2.223.1]

fastlane-plugin-firebase_app_distribution
[0.9.1]

error.log
Gemfile.zip

Looking at the error.log and Gemfile.lock I see that google-apis-core was eventually successfully updated to 0.11.3. The  firebase_app_distribution plugin the worked fine (see the end of the attached error.log):
---------------------------------------
--- Step: firebase_app_distribution ---
---------------------------------------
📡 Uploading the IPA.
🕵 Validating upload…
✅ Uploaded IPA successfully and created release 00.07.15 (000715).
⏩ No release notes passed in. Skipping this step.
📦 Distributing release.
🎉 App Distribution upload finished successfully. Setting Actions.lane_context[SharedValues::FIREBASE_APP_DISTRO_RELEASE] to the uploaded release.
+------------------------------------------------+
|                fastlane summary                |
+------+---------------------------+-------------+
| Step | Action                    | Time (in s) |
+------+---------------------------+-------------+
| 1    | firebase_app_distribution | 16          |
+------+---------------------------+-------------+
fastlane.tools finished successfully 🎉


Fix arguments to checkState in BatchViewOverrides

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[x] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

https://ci-beam.apache.org/job/beam_PreCommit_Java_Phrase/5710/ looks like local gradle cache lock contention

run java precommit


Use PHPUnit hooks for the Statistic tool

With the 4.3 phpunit setup, the statistic tool should now rely on hooks.

PHPUnit 10 drops support for hooks and it all has to be ported to the new Event System.
See https://github.com/sebastianbergmann/phpunit/issues/4595


Coverall fix v2

Pull Request Test Coverage Report for Build 310

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-32.4%) to 67.619%

Totals

Change from base Build 302:
-32.4%

Covered Lines:
497

Relevant Lines:
735

💛  - Coveralls


generated C++  DATA class

Is it supposed to represent received and to-send SBE data?

I don't understand what this question means. SBE codecs are an implementation of the flyweight pattern and not a data class.

leave it first and thanks.


clang-tidy: Fix maglev errors

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

related to #24485 some recently added/updated code is failing clang-tidy
i have added a fix for this specifically as i had spotted this failing in recent ci
not sure of the fixes, but its the ones suggested by clang-tidy


nan for q values in lineage driver test

Description
When running
g = GPCCA(kernel)
# ...
g.compute_lineage_drivers(return_drivers=False)

all q values are nan. Is there a reasonable explanation for this, @michalk8?

Just checked: This is also the case in the real time kernel tutorial.

I've also observed this before.

Just checked: This is also the case in the real time kernel tutorial.

This can be because some gene/lineage has a standard deviation of zero, which results in division by zero, causing the correlation to be NaN, which are then later using for pval/qval calculation. Are all p-values NaN as well?

This can be because some gene/lineage has a standard deviation of zero, which results in division by zero, causing the correlation to be NaN, which are then later using for pval/qval calculation. Are all p-values NaN as well?

No, the p values are not nan.

Strange, this should only happen when the p-values are NaN; running adata.varm['terminal_lineage_drivers']['Neural_1_pval'].isnull().sum() in the WOT tutorial,
I get 36 NaN values. Looking at those genes, they all have 0 expression.

Since the multiple tests correction can't handle NaN p-values, maybe we just subset non-NaN p-values for the q-value computation.

Ok, and what about the case in which the p-values are not NaN? In any case, we should issue a warning for zero-expressed genes (they should have been filtered out before by the user with any standard scanpy filtering anyways).

Ok, and what about the case in which the p-values are not NaN? In any case, we should issue a warning for zero-expressed genes (they should have been filtered out before by the user with any standard scanpy filtering anyways).

As I've suggested, for q-value calculation, let's subset only non-NaN p-values, which occurr when the gene expression for the gene is constant. We can also add the warning that some genes have no variability.

@michalk8, the q-values are also NaN when the p-values are not NaN.

@michalk8, the q-values are also NaN when the p-values are not NaN.

If any p-value is NaN, then all q-values are NaN (that's what statsmodels does).

I agree, let's go for your suggestion then @michalk8

closed via #835


Verify concurrency and make relevant changes to improve multithreaded or multiprocessing capabilities

While creating documentation and benchmarks for #176 I noticed I was unable to achieve multi-file writes for any data extractions to file where there should be many occurring at once or near the same time. Creating this issue to help verify what's happening and make any relevant changes.
References Parsl/parsl#3358

Relates to work in #176


Create visualization tools for views

A blockchain view might be a difficult concept to comprehend. Create visualization tools to view the different parts of the blockchain view, including segmentation by chain and gateway consortium. Some inspiration:
https://layerzeroscan.com/
https://tx.town/v/eth-btc
https://axelarscan.io/
This issue aims to create a simple extension to the CBDC example UI that allows to see the views. The UI essentially has to call the BUNGEE endpoints that SATP exposes.

@RafaelAPB please assign me this one

Depends on #3110


Improvements to the Usage Statistics page and graphs

The current Usage Statistics page layout is not intuitive and it is not obvious that the filter is not connected to the graphs below.
We propose that the filter lets the admin define settings which are then visualized in graphs, with possibility to click through to get more information / interrogate the underlying data.

This involves:

linking the filter and graphics below
Remove the "Topic" from the filter. Users will then only be able to select a date span.
Investigate JS library through which charts are created to see if we can offer a click through experience to investigate the underlying data

This issue builds on https://github.com/DMPRoadmap/roadmap/issues/1678.

Agreed actions on 31/03 call

remove filter as stats are gathered monthly
add a filter to the user / plans page tables
try to give a click though from graph to user / plans page (investigate javascript library)

We will break out filter to the user / plans page tables & click through to separate tickets

Filter removed which was main work left in this ticket so closing out


Is check_comparison needed at all?

The check_comparison() function seems to be reluctant. It works the same way as check_if(), but seems to be less clear. For instance,
check_if(x >= y)

is clearer, and more natural, than
from easycheck import gt

check_comparison(x, gt, y)

Should we deprecate check_comparison()?

Okay. This function, even if not used, does not constitute the problem, maybe a minor one: it makes the packages API more complicated than it can be. Let this issue be open for some time, and we will return to this in some time. Maybe we will change our decision to keep the function.


Network traffic occurs before Bluetooth NET L2 (IPSP) link setup complete

Describe the bug
This issue is about BT network l2 implementation. When creating a L2CAP channel, after receiving the request from initiator, BT RX thread on the responder will handle the packet, and the call stack would be:
(beginning of call stack omitted)

l2cap_recv(),
which calls le_conn_req();
which calls l2cap_chan_accept().

In this function, it call server's "accept" callback first, and then channel ops' "connected" callback if "accept" callback returns true. In current implementation, the "connected" callabck will execute "net_if_up(conn->iface)" operation. And along this function call, it

calls net_if_up(conn->iface),
which calls iface_ipv6_start(iface) as the condition !(l2_flags_get(iface) & NET_L2_POINT_TO_POINT) is true,
which calls net_if_start_dad(iface) as the condition IS_ENABLED(CONFIG_NET_IPV6_DAD) is true,
which calls net_if_ipv6_start_dad(iface, &ipv6->unicast[i]),
which calls net_ipv6_start_dad(iface, ifaddr),
which calls net_ipv6_send_ns(iface, NULL, NULL, NULL, &ifaddr->address.in6_addr, true),
which calls net_send_data(pkt).

And this will eventually pass pkt data to BT TX thread.
After calls l2cap_chan_accept() returns, inside le_conn_req(), l2cap_send() is called to send l2cap layer response to initiator. This indeed passes some data to BT TX thread as well. At this moment, the responder will consider this l2cap channel is setup already, and the bt net_if is up. When the responder switches to BT TX thread, it will send these data to the initiator.
The problem occurs on the initiator side. After sending the l2cap channel setup request, it is waiting for the response. However, as the responder will send network traffic data before the l2cap response. This means some unexpected data arrives before the channel is setup on initiator's side. Then it will discard the net traffic data, and consider the attempt of creating l2cap channel fails. Even if the l2cap response arrives, as the channel is discarded, the reponse is not handled. And the bt net_if is considered remaining down.
To Reproduce
Steps to reproduce the behavior:

use echo server and echo client samples, with CONFIG_NET_L2_BT_MGMT=y and CONFIG_NET_L2_BT_SHELL=y. On client sample, CONFIG_NET_CONFIG_BT_NODE=y, and on server sample, CONFIG_NET_CONFIG_BT_NODE=n.
platform: nrf5340dk_nrf5340_cpuapp
zephyr revision: I am using nrf connect sdk, which has cunstomed zephyr of v3.1.99-ncs1. But I think this problem also exists in original zephyr.
On server sample side, use shell to send "net_bt connect <peer's bt mac address> random", and the bt connection is setup. Send the command again to setup l2cap channel. The server/initiator side will show message like

[00:00:22.607,604]  bt_l2cap: No credits to receive packet
[00:00:22.706,817]  bt_l2cap: Ignoring data received while disconnecting
[00:00:22.756,835]  bt_l2cap: Ignoring data received while disconnecting
[00:00:22.756,958]  bt_l2cap: Ignoring data received while disconnecting
[00:00:22.806,884]  bt_l2cap: Cannot find channel for ident 1
[00:00:22.856,994]  bt_l2cap: Ignoring data received while disconnecting
[00:00:22.906,951]  bt_l2cap: Ignoring data received while disconnecting
[00:00:23.606,292]  bt_l2cap: Ignoring data for unknown channel ID 0x0040
[00:00:24.606,323]  bt_l2cap: Ignoring data for unknown channel ID 0x0040

and if you run "net iface", the interface is down.
5. On client/responder side, it shows

[00:00:25.797,729]  net_config: Interface 1 (0x20001160) coming up

and if you run "net iface", the interface is up.
Expected behavior
The l3 activity should start only after the l2 link is fully setup.
Impact
bt-based link layer connection cannot be setup. And discrepancy of the link state is observed.
Additional context
To fix it, currently I postpone the net_if_up(conn->iface) call by warpping it as a delayable work and just submit it to system queue with some delay (e.g. 10ms) in the connected callback. This can ensure the l2cap response is sent before any network traffic. But I think the problem should be solved in bluetooth stack. It should call "connected" callback after sending the l2cap_connection_rsp message. And this needs bluetooth expert to verify if it meets bluetooth specification, and if it breaks existing BT stack, as ATT and SM also use l2cap channel.

@wentong-li Can you see if PR #50476 fixes this issue?

Seems not. IMO, the receiver of L2CAP channel connection req does things in wrong order. Currently, it first call "accept" callback, then "connected" callback, and finally send L2CAP channel connection rsp. And I am sending data over this l2cap channel in "connected" callback. It is fine for the receiver, but the sender of L2CAP channel connection req is waiting for L2CAP channel connection rsp but gets other data before the response. Maybe we can reorder the sequence as call "accept" callback -> send L2CAP channel connection rsp -> call "connected" callabck. It is more intuitive that the "connected" callback is called after all setup is done, including sending L2CAP channel connection rsp.


how to make the carousel auto-scroll programmatically

I'm trying to programmatically cause the carousel to auto-scroll. I have the carousel operating vertically so someone can fill out a form and as they fill out the form, it auto-scrolls. I basically do this by setting a state variable called isAutoPlay.  Below is the sample code. If isAutoPlay is true it keeps auto-scrolling the carousel indefinitely until I set it to false. I currently have a javascript timer to turn it off after a second but the timing isn't perfect so sometimes it may scroll past a few questions before it stops. I am wondering if there is a better way to trigger the autoscroll so perhaps it just scrolls to the next slide instead of indefinitely.
<Carousel                             width={deviceWidth}                             height={200}                             style={{height:'100%'}}                             data={this.allQuestions}                             vertical={true}                             loop={true}                             autoPlay={this.state.isAutoPlay}                             renderItem={({ item }) => (                                 <View>{item}</View>                             )}                           />

Do you mean that you want to stop animation immediately after changing 'autoPlay'? Even if the animation is not finished? The current practice is that when the 'autoPlay' changes, the replacement will stop the next time the item is about to be replaced.

Thanks for your help. I don't want to stop the animation but just move it to the next slide and stop at the next slide, sort of like a powerpoint slideshow. In other words, I would like to be able to push a button and move to the next slide if that is possible.

This?

You can use these methods to control the slides.

Thanks that is very helpful. I appreciate your responsiveness. Forgive me for the naive question but how do you call these methods? My screen is classed based (not procedure style) and I'm trying to assign a ref to the Carousel so I can call next() and it's not working.
I use this import statement
import Carousel, { ICarouselInstance }  from 'react-native-reanimated-carousel';
Followed by (right before the class definition)
const ref = React.useRef<ICarouselInstance>(null);
My carousel in the render() function is like this:
<Carousel
                            width={deviceWidth}
                            height={200}
                            style={{height:'100%'}}
                            data={this.allQuestions}
                            vertical={true}
                            loop={false}
                            ref={ref}
                            renderItem={({ item }) => (
                                <View >{item}</View>
                            )}
                          />

To test the ref variable I just do getCurrentIndex() but it's undefined
        ToggleAutoplay(){

            console.log(ref.current?.getCurrentIndex());
}

My button runs but I just see undefined though.

I created a dummy function based version of my screen and I got the methods to work next(). Thanks again for pointing out the documentation.
I just used const ref = React.useRef(null);
Then I was able to call ref.current?.getCurrentIndex() with no issue.
However when I go back to my class component and tried to do the same thing it doesn't work. I get undefined. I use
this.ref = createRef();
And refer to the ref each with this.ref.
I would like not to have to refactor my code to be function based ( I will work towards that) but just wondering if the ref doesn't support class components here.

I think I figured out my issue. It appears that the ref=createRef() does work with the class (tried with a simplified test class) but the root cause of my issue is that I'm loading my carousel with dynamic data from asyncstorage. The data load correctly and the carousel works great but I cannot use the ref at all.
It appears that when I do this.ref=createRef from anywhere (constructor, render(), etc) it keeps showing undefined when I do ref.current?.getCurrentIndex(). I wonder if there is a way to have ref=createRef() give the updated ref somehow after the promise for asyncstorage is done. Otherwise, it appears I have to load all the data before I enter the screen with the carousel to avoid having the promise method (getting  data from asyncstorage) mess up my ref.

Does this library support having data dynamically generated for the Carousel?

Thanks for your details. It looks like a bug, Sorry for that. The carousel methods of ref need to update when the data changed but it didn't. And the ref props haven't called again, So you can try this workaround.
<Carousel key={data.length} data={data} />

I'll check it as soon.

Thank you very much ! You rock!! That fix worked :)    We will try it out and let you know after testing how it's working.
But from what I can see when I just put
I do createRef() in the constructor and after my async method runs to populate the data for my carousel the this.ref.current?.next() works now.


fix: add space to watch video button

Please describe the changes this PR makes and why it should be merged:
Before:

After:

Status

[X] Code changes have been tested against prettier, or there are no code changes
[X] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

[ ] This PR changes the codebase

[ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
[ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[X] This PR only includes non-code changes, like changes to documentation, README, etc.

LGTM


Tupek/trust region subspace solver

/style

/style

/style

/style

im not too familiar with petsc/ mfem, but could a lot of the petsc wrapper stuff use mfem's petsc wrapper classes? https://github.com/mfem/mfem/blob/master/linalg/petsc.hpp ?

hpp

The mfem wrappers are intended for 'parallel' across processor things.  The new wrapper is for local, small, dense things.  In general, many coding experts recommend creating internal abstractions to external library wrappers anyways, but in this case, the mfem one doesn't even really fix the purpose.

/style

/style


[bug]: lncli is unresponsive for commands like lncli connect, getinfo, getnetworkinfo, listpeers, ...

I am using version 0.17.0-beta.rc3 commit=v0.17.0-beta.rc3-19-g9f4a8836d, and for some reason there are some times during the day that it seems that my LND gets clogged and unresponsive.
The logs show this kind of error repeatedly, I think this is what is clogging my LND.

Some times it works with getnetworkinfo, it takes ~30s to return a result:

But since this afternoon, specially listpeers and connect stopped working at all, it doesnt return any error, just hangs.

I just noticed this after I restarted LND:

Might be the cause, or not, dunno. lncli connect never returns an error.


[Event Request] Table 9650 - Custom Report Layout - OnBeforeExportSchema

Dear ALAppExtensions team,
Through this issue we would like to request event publisher OnBeforeExportSchema to be added to Table 9650 Custom Report Layout with the following signature:
    [IntegrationEvent(false, false)]
    local procedure OnBeforeExportReportLayout(CustomReportLayout: Record "Custom Report Layout"; var DefaultFileName: Text; ShowFileDialog: Boolean; var IsHandled: Boolean)
    begin
    end;

with var parameters DefaultFileName and IsHandled.
which would be called as follows from ExportSchema:
    procedure ExportSchema(DefaultFileName: Text; ShowFileDialog: Boolean): Text
    var
        TempBlob: Codeunit "Temp Blob";
        FileMgt: Codeunit "File Management";
        IsHandled: Boolean; // NEW VARIABLE
    begin
        // NEW EVENT >>
        IsHandled := false;
        OnBeforeExportReportLayout(Rec, DefaultFileName, ShowFileDialog, IsHandled);
        if IsHandled then
            exit;
        // NEW EVENT <<

        TestField(Type, Type::Word);

        if DefaultFileName = '' then
            DefaultFileName := '*.xml';

        GetWordXML(TempBlob);
        if TempBlob.HasValue then
            exit(FileMgt.BLOBExport(TempBlob, DefaultFileName, ShowFileDialog));
    end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Feature/nwps 399 400 420

Hi Selva,
I've enable Homepage, Standard Content Page and Landing Page in this PR.
I've also created a components and migration folder, and have put old content here as we work on creating a new IA for development.
Lets discuss.
Thanks,
Raq

Hi @Raq-HEE - Thanks for the PR and organising pages, components, etc 👍
Could you take a look at the following:

Implementing manage content<EMAIL_ADDRESS>hippobean=${document} />) link for Home, Landing and Standard content experience pages (based on extras/xpage-work-for-reference branch).
Moving all content yaml files under repository-data/development so that it won't get imported on all brCloud env.

Observed that <title> meta tag is not rendering (title) properly on the Home page (http://localhost:8080/site/pages/page-types/home-page). But, I think it will get fixed as part of Home page refactoring being done in https://github.com/Manifesto-Digital/hee-cms-platform/pull/138 PR (which essentially introduces title field in the Home page (hee:HomePage) document) ?
Thanks!


Is there a way to override the tab feature?

We need our form to be ADA compliant and having the tab locked in this control is not ideal. It would be great if it was a flag on the directive.

I could look into that but please give me a couple of days as I have plenty of other work


Update the link from blockchain.info to blockchain.com

I've updated this because the Hyperlink to blockchain doesn't work
(Fixed Link)

@lukas-dallhammer Thanks for the contribution. Can you please solve the conflict?

Although they have redirected the old link to the present one.
Thanks for updating the link :)

@lukas-dallhammer  Please push to the master branch not to the alpha-site

Sorry, it's my first contribution now everything is done, right?

@lukas-dallhammer
Please make a new PR and set the head to master branch of the public-apis


Continuous requests sometimes cause a 'Reel::StateError: not in chunked body mode' error

We have an app that, among other things, runs Reel/Sinatra. The main application is a voice app primarily based on Adhearsion, and it uses a plugin of sorts called Virginia to run Reel and Sinatra, providing a web-interface for us.
We have one machine in our development environment that gets hit with a 'health check' over HTTP ever 15 seconds.
Reel::StateError: not in chunked body mode
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/request.rb:94:in `write'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:47:in `route_request'
org/jruby/RubyArray.java:1617:in `each'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/rack-1.4.5/lib/rack/body_proxy.rb:31:in `each'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:47:in `route_request'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:29:in `on_connection'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/connection.rb:73:in `each_request'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:25:in `on_connection'
org/jruby/RubyMethod.java:134:in `call'
org/jruby/RubyProc.java:255:in `call'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/server.rb:56:in `handle_connection'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/server.rb:55:in `handle_connection'
org/jruby/RubyBasicObject.java:1730:in `__send__'
org/jruby/RubyKernel.java:1932:in `public_send'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/calls.rb:25:in `dispatch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/calls.rb:122:in `dispatch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/actor.rb:322:in `handle_message'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/actor.rb:416:in `task'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks.rb:55:in `initialize'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks.rb:47:in `initialize'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks/task_fiber.rb:13:in `create'
[2014-11-05 17:48:08] ERROR Adhearsion::Initializer: <Reel::StateError> not in chunked body mode
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/request.rb:94:in `write'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:47:in `route_request'
org/jruby/RubyArray.java:1617:in `each'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/rack-1.4.5/lib/rack/body_proxy.rb:31:in `each'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:47:in `route_request'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:29:in `on_connection'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/connection.rb:73:in `each_request'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-rack-0.2.0/lib/reel/rack/server.rb:25:in `on_connection'
org/jruby/RubyMethod.java:134:in `call'
org/jruby/RubyProc.java:255:in `call'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/server.rb:56:in `handle_connection'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/reel-0.5.0/lib/reel/server.rb:55:in `handle_connection'
org/jruby/RubyBasicObject.java:1730:in `__send__'
org/jruby/RubyKernel.java:1932:in `public_send'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/calls.rb:25:in `dispatch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/calls.rb:122:in `dispatch'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/actor.rb:322:in `handle_message'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/actor.rb:416:in `task'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks.rb:55:in `initialize'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks.rb:47:in `initialize'
/srv/phone/apptastic/shared/bundle/jruby/1.9/gems/celluloid-0.15.2/lib/celluloid/tasks/task_fiber.rb:13:in `create'
[2014-11-05 17:48:08] WARN  Celluloid: Terminating task: type=:call, meta={:method_name=>:run}, status=:iowait
[2014-11-05 17:48:08] INFO  Celluloid: A Reel good HTTP server! (Codename "Bette")
[2014-11-05 17:48:08] INFO  Celluloid: Listening on http://<IP_ADDRESS>:5005

As you can see, the first lines are the regular 15s intervals. The timestamp at the crash is 30 seconds later, hence the one missing log line. Reel restarted right away and continue to serve requests.
We thought this was a freak occurrence due to some issue with our load balancer that issues that HTTP health-check, but we have since seen this happen multiple times.
Given the location of the error in Reel, I assume there is a bug with either how the state machine transfers to the :chunked_body state, or with how the load-balancer issues/handles the HTTP health-check. While we currently do not hit any web-based endpoints during our call processing, we might want to in the future, and having this occur would cause the call to crash, which would be pretty undesirable for us.
Any thoughts on how we might avoid this issue, or avenues to diagnose better? As an aside, we are currently pinned to celluloid 0.15.2 due to an issue with a Punchblock and cannot upgrade to 0.16.x yet.

I'm removing this from 0.6.0 because I need a repro. Please post a repro and I'll get it in 0.6.5


Karma DEPRECATED warning

Dear All,
I installed latest clean version of the boilerplate. When i run the test, i encounter warning message from karma (please see the last line in code below). Would you please guide me what is causing the DEPRECATED message in karma?
Thanks and regards.
express@micro-ubuntu:~/web-ci$ npm run test
><EMAIL_ADDRESS>pretest /home/express/web-ci
> npm run test:clean && npm run lint
><EMAIL_ADDRESS>test:clean /home/express/web-ci
> rimraf ./coverage
><EMAIL_ADDRESS>lint /home/express/web-ci
> npm run lint:js && npm run lint:css
><EMAIL_ADDRESS>lint:js /home/express/web-ci
> npm run lint:eslint -- .
><EMAIL_ADDRESS>lint:eslint /home/express/web-ci
> eslint --ignore-path .gitignore --ignore-pattern internals/scripts "."
><EMAIL_ADDRESS>lint:css /home/express/web-ci
> stylelint ./app/**/*.css
><EMAIL_ADDRESS>test /home/express/web-ci
> cross-env NODE_ENV=test karma start internals/testing/karma.conf.js --single-run
DEPRECATED: use your own version of lodash, this will go away in<EMAIL_ADDRESS>START:

It's just Karma announcing that as of 2.0, Karma will no longer export lodash '_'. It's just a warning, not a problem, until Karma 2.0 is released.
Karma source


Windows registration command does not appear when creating custom cluster through rancher

on master-head (21ab02c):
Issue:

registration command is not showing up for windows on custom cluster registration page

Expected Result:
Based on the original ask from this ticket and the layout of the registration page (after creating a custom cluster through rancher) a separate command should be shown to the user
from the original ticket:
* Registration command for Windows worker nodes
* Only allow worker role for Windows nodes to be registered (grey out etcd and controlplane for windows)

This might still be dependent upon the backend work here: https://github.com/rancher/rancher/issues/33625

The command goes where the message that says "The cluster must be up and running with Linux etcd and Control Plane nodes before you can add Windows workers" is shown.  It's not shown because you don't have that yet.
The checkboxes apply only to the Linux command.

on master-head (94bca99):

after registering etcd/CP linux node, windows registration command shows up -- pass


Add metrics

Add more metrics to goatcounter

@paloeka I've added the following events:

New user a new user is created (either a host or participant)
New chain a new loop is created (from a new or existing user)
Approve user a pending user is approved by a host
Deny user a pending user is denied by a host


Handle Linux build dependencies

From https://blog.axo.dev/2023/02/cargo-dist:

So minidump-debugger is based on egui, and for whatever reason egui builds on linux require you to install a ton of dev packages (on Windows and macOS it Just Works so I always forget this). [...] Welp, automating and understanding those kinds of build deps are another feature for me to add to cargo-dist one day, but for now I'll rely on the fact that the scripts are checked in so I can make bespoke hand-edits to the scripts if need be (being able to do this is an intentional feature, although obviously suboptimal because now we're getting back some dependency on the CI's precise behaviour).

I don't think there's already a tracking issue for this, so I made one. It would be a great boon to humanity to solve this problem. Please do.

I agree, someone beat you to the punch filing #102 :)

This could be handled together with generating binary packages (#76). The same list of dependencies could be used to specify the dependencies of generated packages and install those dependencies before building.


Gulp equivalent of grunt-shopify

Grunt-shopify works perfectly fine. But can you please provide a gulp equivalent for the same as configuring with gulp is quite simpler compared to grunt?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27024690-gulp-equivalent-of-grunt-shopify?utm_campaign=plugin&utm_content=tracker%2F432437&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F432437&utm_medium=issues&utm_source=github).

See https://www.npmjs.com/package/gulp-shopify-upload

Thank you!!


Add x_minmax and y_minmax feature, closes #120

For #120.
Utility features

x_minmax(zoom), and
y_minmax(zoom)

to get a tuple of (min, max) integer tile coordinates on a specific zoom level.
The main use case is to hide and abstract over the number of tiles and the max tile calculation from the user.
Note: I've cherry-picked some commits from https://github.com/mapbox/mercantile/pull/123 in here to develop locally.

Coverage decreased (-0.6%) to 99.02% when pulling 633939f3f5e98450f2d2f9745a0aadde88bc7ea5 on daniel-j-h:add-minmax into d00e29330663d2ca8dc1f4056f92194286250b38 on mapbox:master.


Add dotnet test cases

This will add test cases for .NET Core 2.1 and 3.1 images based on rhel7 and rhel8.
Each test uses oc new-app (so s2i) to deploy the container using our example app:
https://github.com/redhat-developer/s2i-dotnetcore-ex
It also runs dotnet --info inside the container and does a basic version number check (i.e. that the 2.1 container is running 2.1).
To get this to work I had to do a couple of things:

By default the current framework assumes that every test wants to clone a repo from sclorg/ and will error if it can't. I made that optional instead of required. If you define an scl_url item in your test var file, then it will try to clone the repo and error if it can't. Otherwise, it just ignores all of that.

Despite what the example template said, there was no way to run commands inside the container without writing a custom script for each thing you wanted to run (as is done for the DB check scripts). So I made a generic script that just lets you supply the command that you want to 'oc exec' inside the container and then looks for a substring in the output. Just put your command as a string when you declare test_exec_command. In theory, you can also pass params to the oc exec.

@RheaAyase @aslicerh Thank you for PR. My question is if we want to test .NET Core images together with RHSCL images or alone?
If alone, I would prefer to use them as EXT_TEST="NET_CORE" and slightly changes deploy-and-test.yml playbook.
What do you think?
Also please rebase this PR on the top of master. Thank you.

@phracek
What you said makes total sense. I've since merged master and moved the dotnet containers into their on ext_test task.
To differentiate it from the java ext_test task I had to change it a little bit. As far as I can tell everything works though.
Let me know if this is going to be an issue.


Move metrics opt-in to the SPLASH view.

Tested in Chrome/FF in dist and dev.
For FF this also fixes the extra wide buttons we were seeing on the metrics opt in page.
@dborkan I'm removing the "Anonymous metrics enabled" bubble in this PR too (it made things simpler) so I can wait until you have your changes in before having this reviewed!

Small comment, then :+1:

Review status: 0 of 4 files reviewed at latest revision, 1 unresolved discussion.

src/generic_ui/polymer/splash.ts, line 111 [r1] (raw file):
I think it might be good to save globalSettings here, same for disableStats below

Comments from the review on Reviewable.io


Investigate Semaphore for test and deployment

Same as review apps from heroku.  Provides the ability to spin up test environments for features to be reviewed before being released to live.
Aid continuous deployment in OS world


Cache memory data in instance

This patch moves the memory caching to instance. Since instance is always in a register, the jit can access is easily.

After some trying, the least painful location for the cache is between the memories and globals.

I have updated the patch. Could you check it? Thank you.


bad result for a new source image?

Dear Aliaksandr Siarohin,
When I use the checkpoint fashion.pth.tar you offerd，a test fashion video  and my own image  to run demo.py, I got the bad result. Does this indicate poor generalization performance for a new image?

Please post your source image and your driving video.

Please post your source image and your driving video.

Source image:

The driving vedio is selected from the fashion-png/test, which is named A1+O5s8cUuS.mp4 and resized to 256x256 .

The result looks like this. As can be seen from the face and trousers，it‘s not so satisfying...

So in this case yes, it is a generalization problem. This fashion dataset contains only the videos of womans in dresses. So it tries to draw some sort of dreess and some long hairs.
So it need to be trained also with men videos. If you don't have men videos, you can try to create synthetic training pairs.

So in this case yes, it is a generalization problem. This fashion dataset contains only the videos of womans in dresses. So it tries to draw some sort of dreess and some long hairs.
So it need to be trained also with men videos. If you don't have men videos, you can try to create synthetic training pairs.

I've got it. Thank you!

Could someone elaborate on the synthetic training pairs? Did @howardgriffin  manage to get this done?


Add separation for source and configuration folders.

Breaking change

This will not break as default setting.  Users of docker container and dynamic compile will have to add this setting though. Users of add-on will automatically use same setting as before if it is missing as an transistion time.
Proposed change

This PR makes it possible to configure where ND will look for source files when dynamic compile option is used and where the location of the config files are. This makes configurations more explicit. This will also allow for better default settings in the future app templates.
The deployment will differ following way:
ApplicationConfigurationFolder: Points to folder where the configuration are. If yam config is used both deployment models will use this
ApplicationSourceFolder: Points to folder where the source will be loaded. Will used if source deployment is used
ApplicationAssembly: Points to the assembly (dll) of custom built runtime and apps. Will be used when users compile own assembly. Only used in context of docker container or add-on
Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (which adds functionality to an existing integration)
[ ] Breaking change (fix/feature causing existing functionality to break)
[ ] Code quality improvements to existing code or addition of tests

Additional information

This PR fixes or closes issue: fixes #
This PR is related to issue:
Link to documentation pull request:

Checklist

[ ] The code change is tested and works locally.
[ ] Local tests pass. Your PR cannot be merged unless tests pass
[ ] There is no commented out code in this PR.
[ ] I have followed the [development checklist][dev-checklist]
[ ] The code compiles without warnings (code quality chek)
[ ] Tests have been added to verify that the new code works.

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated for www.home-assistant.io

Codecov Report

Merging #633 (8f66253) into dev (ea9e6ed) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##              dev     #633      +/-   ##
==========================================
- Coverage   80.12%   80.09%   -0.03%
==========================================
  Files         111      112       +1
  Lines        2903     2904       +1
  Branches      308      308
==========================================
  Hits         2326     2326
- Misses        450      451       +1
  Partials      127      127

Flag
Coverage Δ

unittests
80.09% <100.00%> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

...Common/Settings/AppConfigurationLocationSetting.cs
0.00% <ø> (ø)

...pModel/Common/Settings/AppSourceLocationSetting.cs
100.00% <100.00%> (ø)

...n.AppModel/Internal/Compiler/SyntaxTreeResolver.cs
100.00% <100.00%> (ø)

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 ea9e6ed...8f66253. Read the comment docs.

I changed my mind. This PR would allow controlling the structure better. Not really needed now how we talked about templates should work


kpack-image-builder must only add finalizers to its BuildWorkloads

Is there a related GitHub Issue?
See #2598
What is this change about?
Add a new SetPolicy for the BuildWorkload finalizer in kpack-image-builder which checks the BuildName on the spec, just like its reconciler does. This way multiple builders can co-exist in korifi without interfering.
This requires the signature of SetPolicy to be changed to give access to the Spec of the object. We now use an unstructured object which can be easily converted to an actual resource type if required.
Does this PR introduce a breaking change?
No
Acceptance Steps
Use kpack-image-builder in parallel with another builder. Check BuildWorkloads (and kpack.Builds) associated with the other builder do not get kpack-image-builder finalizers added.
Tag your pair, your PM, and/or team
@matt-royal

I just confirmed that this fixes the issue. Thank you, @kieron-dev!


Possible null pointer in ssl_decrypt_encrypted_pms

https://github.com/ARMmbed/mbedtls/blob/5f69cbe2ccf9477bb484c6f27d8a9a5719d6ad1c/library/ssl_srv.c#L3632
The code looks unsafe because NULL could be returned from mbedtls_ssl_own_cert( ssl ).

This function is only called from an SSL connection at the step MBEDTLS_SSL_CLIENT_KEY_EXCHANGE, for an RSA or RSA-PSK ciphersuite. To reach this step, the server has to first go through all previous steps including MBEDTLS_SSL_SERVER_CERTIFICATE which calls mbedtls_ssl_write_certificate. In a ciphersuite that requires a certificate, if mbedtls_ssl_own_cert( ssl ) is null, this function errors out.
So I don't see how mbedtls_ssl_own_cert( ssl ) could return NULL at this point.

Should be good enough then.
Thanks for checking the issue.


Fix crossfader

The current crossfader adds both signals when the fader is in the middle and the overall sound is louder.
I didn't test the code but I'm assuming that:

data2 is a value in the interval [0,127]
a and b should be a value in the interval [0,127] where the sum is always 127

Let me know if you need a tester (I will have to check how to do it).
Best regards.
Henrique Rocha

Sorry for spamming you,
I just tested your controller script for APC40 Mk2 and it's not working, apparently your controller works only for APC40.
I tough this was the official Mk2 control script.
I will decline this PR.
Best regards.
Henrique Rocha

Yeah, this controller script is for Mk I.


CV improved LB not

Hi guys,
i have extracted  some more features  from bureau file, it improved CV
from .7950 to  0.7974 ( std 0.0024)  but  LB drop from .804 to .802.   has anyone  experienced this before ?
i dont use any TARGET related  features, and dont think it is overfitting.

Hi there @davutpolat. Good job on improving the score so much.
The diff between CV and LB with this particular public LB (that is super tiny) could definitely be due to luck alone.
We will submit best local CV and best LB for our two submissions and I don't know if you can do more about it unfortunatelly.
Best and good luck!


Cherry pick 314 316

This PR cherry-picks the following commit(s) to release/2.4:

667f40e6b89b
4101a29ce26f

bors merge


Various updates

Hi @Rayraegah, sorry for lumping these all in to one PR.
We've been iterating quickly and fell behind pushing these back up, we'll try and be better going forward!
There's nothing major in here, just:

Adding some tags so it's easier to identify resources in the AWS console
Adding some outputs for the network resources (we needed these to set up a security group for a VPC endpoint we use to secure our lambda).
Renaming the subnets to align with the convention AWS uses.
Finally we used fmt to clean everything up.

Additionally, as an FYI, we're now exclusively targeting 0.12, which was released 9 days ago. All our PRs will this be targeting the terraform-0.12 branch.

We're migrating to v0.12 as well and I'll move this release to the master branch once I update the readme and create a support branch for v0.11.14

Awesome, and thanks for the working getting this module up and going!

Oh @Rayraegah one thing I should have mentioned:
Because we renamed the subnet resources (from hasura_rds/hasura_ecs to hasura_private/hasura_public) you'll have to terraform import the existing ones in to the new ones if you don't want them to be recreated (although, I don't think import vs recreate would actually make any difference).


Use header instead of its hash in create_inherent_data_providers

I needed parent block number in create_inherent_data_providers and while header is already available in all cases, I had to make a call into a client. Maybe the whole header can be exposed right away?

I don't see any problem of passing only the hash. I mean I get that you need the block number and need to call into the client, but I don't see this as justification for passing the full header. We pass the block hash to give a context. There could probably be multiple apis where passing a header or whatever would make it easier directly. I get that this is mainly a "taste" question, but we also should not change apis if not really required.

This isn't a critical feature in any case, just wanted to see what you folks think about it.

I think I thought about this when introducing this code, but I can not remember why I opted in for the block hash only. However, as I already explained my current thinking, I don't think we should change it now.

Better question: What are you actually trying to do?

Better question: What are you actually trying to do?

He needs to access the block number, as he has written :P

Better question: What are you actually trying to do?

There is an inherent extrinsic in our protocol and block number is required to create InherentDataProvider for it: https://github.com/subspace/subspace/blob/cff3a0f0abbaed126594945fa697f9598065b91f/crates/subspace-service/src/lib.rs#L409-L414
Closing since it seems unlikely to be merged. Thanks for the feedback!


Added disable instructions regarding "?_k" bit

I think it will be better to also include the code there, it's just 3 lines