Add ability to regenerate KSK keys in TP

What does this PR do?
Adds a button on CDN/DNSSEC page to regenerate KSK keys.
Fixes #(issue_number)
Required 2908 to be merged in order to work.
Which TC components are affected by this PR?

[ ] Documentation
[ ] Grove
[ ] Traffic Analytics
[ ] Traffic Monitor
[ ] Traffic Ops
[ ] Traffic Ops ORT
[x] Traffic Portal
[ ] Traffic Router
[ ] Traffic Stats
[ ] Traffic Vault
[ ] Other _________

What is the best way to verify this PR?
In order to generate the actual key, it will require 2908. Otherwise, run locally, click on CDNs, select a CDN with DNSSEC active, click the 'More' dropdown and select Manage DNSSEC. If KSK keys have already been generated, you will see the Regenerate KSK Keys. Click it to start the process.
Check all that apply

[ ] This PR includes tests
[ ] This PR includes documentation updates
[ ] This PR includes an update to CHANGELOG.md
[ ] This PR includes all required license headers
[ ] This PR includes a database migration (ensure that migration sequence is correct)
[ ] This PR fixes a serious security flaw. Read more: www.apache.org/security

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2593/
Test FAILed.

Branched from https://github.com/apache/trafficcontrol/pull/2908 - please merge that first.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2673/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/2737/
Test FAILed.


SideViewController content gets "pushed down" when first displayed

The first time either a left or right view controller is displayed, its content appears to be pushed down and into place once the VC is finished animating in.  This looks like an issue with MMDrawerController as well.  The comment here seems to fix it.  The short and sweet is this:
Add the following line to the viewWillAppear:animated method of your SideViewController:
self.navigationController?.view.layoutSubviews()

So it looks something like this:
override func viewWillAppear(animated: Bool) {
    super.viewWillAppear(animated)

    // Workaround for DrawerController issue with iOS 8
    self.navigationController?.view.layoutSubviews()
}

Thanks! layoutSubviews() should not be called directly though, which is why I'm using setNeedsLayout()


Multiple templates

Description
It would be awesome to have different templates for either different files (for example I'd like to use different doxygen comments in source and header files respectively). This could be implemented by multiple trigger sequences like /** [enter] for auto guessing and \*c [enter] for c files and so on.
Is that a big effort? Unfortunately, my TS skills are very limited.

The information is available easily for the plugin.
Problem with the user experience is that there may be confusion of which setting to use and they can easily get out of sync which may cause unexpected behavior.
I think a primary and a secondary setting may be manageable. An arbitrary number of config styles is not possible (due to how VS Code handles settings).
One could the apply the primary or secondary style based on some regex or w/e.
What would you suggest?

The only idea I have for unlimited templates is an additional json file where as many configs can be created as templates required. But I don't need that so far.
For me, a template for a header and a template for a sources file would be sufficient.
What do you mean with w/e? I think a different trigger is sufficient, i.e. /**h for a header and /**c for source file. You could make the triggers configurable as well :)


CompoundBarcodeView.pause() causes the semi transparent frame to disappear

Hi,
I wan't to achieve a continuous scan that stands still for 2 seconds if barcode was found. Therefore I use the BarcodeCallback and call compoundBarcodeView.pause() then wait 2 seconds and call compoundBarcodeView.resume() because I don't know any other option.
What happens is the following

continous scan
sill image full screen (with no semi transparent frame)
still image with frame
continous scan

I'd like to have it like this

continuous scan
still image with frame
continuous scan

This is the code I use in the callback
private BarcodeCallback callback = new BarcodeCallback() {
private long lastTimestamp = 0;
    @Override
    public void barcodeResult(BarcodeResult result) {
        String scannedCode = null;

        if (result != null) {
            scannedCode = result.getText();

            if (scannedCode != null) {
                mBarcodeScannerView.pause();

                if(System.currentTimeMillis() - lastTimestamp < DELAY) {
                    // Too soon after the last barcode - ignore.
                    return;
                }
                lastTimestamp = System.currentTimeMillis();
                mBarcodeScannerView.resume();
            }
        }
        handleScan(scannedCode);
    }

    @Override
    public void possibleResultPoints(List<ResultPoint> resultPoints) { /* do nothing */ }
};

Will work on this for the next release.

As a workaround I took the ViewfinderView and made it invisible (gone). With this way I also got rid of the laser which i wanted anyway. Then I created a View by myself. This view never gets distracted.
CompoundBarcodeView.getViewFinder().setVisibility(View.GONE);
I leave this ticket open so you can close it @rkistner if you have fixed the ViewFinderView.


fix js syntax error

got error under node(v7.9.0)

Whaaaaa but it's a promise? Did the promise syntax change?

I'm running the same version and have no problems. @mrzzcn Can you please provide steps to reproduce the issue?

sorry, missed this.

so unprofessional !!!

The steps image isn't loading, and I can't repro an error, so closing :(
Please re-open if there's actual repro steps I can look at, or an explanation as to what you think may be causing this


new tests and (FORTRAN/F2C) support methods

Once more with feeling ...

Coverage increased (+0.01%) to 98.472% when pulling 7d7398fe3321f3e97daa5259a464b8e439f9835d on drbitboy:master into e9d37d90eb248b1a59a2d3eb6d92378181bfbea3 on AndrewAnnex:master.

hah!  meant to use vars(spicey.spiceypy.libspice).keys()

Coverage increased (+0.01%) to 98.473% when pulling 76b9848adfdd931c0de6f71bbff9c2ea38d48c03 on drbitboy:master into e9d37d90eb248b1a59a2d3eb6d92378181bfbea3 on AndrewAnnex:master.


nc_test fail in 4.5.0 with CMake

 70/177 Test  #70: nc_test ...............................***Failed   26.31 sec

It works without -DCMAKE_BUILD_TYPE=Release (i.e. DEBUG build), but not with it.

Sorry for entering too fast, edited. Likely related to the warning in https://github.com/Unidata/netcdf-c/issues/501.

Interesting; what platform are you on? I'm unable to recreate this but will keep trying.

Specifically, which distribution of 64-bit linux, sorry :)

ArchLinux as usual, sorry to have not mentioned it. I’ll add the log requested by @DennisHeimbigner shortly.

https://paste.xinu.at/xrLi/
Relevant section starts at https://paste.xinu.at/xrLi/#n2211

This also fails for me:
# Features
--------
NetCDF-2 API:		yes
HDF4 Support:		no
NetCDF-4 API:		yes
NC-4 Parallel Support:	no
PNetCDF Support:	no
DAP2 Support:		no
DAP4 Support:		no
Diskless Support:	yes
MMap Support:		no
JNA Support:		no
CDF5 Support:		no

With gcc 7.2, zlib 1.2.11, szip 2.1.1, hdf5 1.8.18. Debian (latest).

I can confirm that the test passes if the netcdf-c library is build with debugging flags (all dependent libraries compiled with high optimization).

zerothi: So you’re having the same failure with autotools? That’s very interesting, because I don’t (I’ve reverted to autotools since #244 was fixed meaning every test pass this way while this fails with CMake).
Maybe something with GCC 7.2 then. @zerothi What are your compile flags?

-m64 -fPIC -O3 -ftree-vectorize -fexpensive-optimizations -funroll-loops -fprefetch-loop-arrays -march=native
I agree, they are pretty aggressive, but hdf5 etc. completes tests without problems.

CFLAGS:			-march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -O3 -DNDEBUG
CPPFLAGS:
LDFLAGS:		-Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now

The -O2 is system default, the -O3 is a RELEASE addition apparently. So I expect -O3 (+ gcc 7.2?) to be the culprit.

I can confirm that a debug build with -O3 fails this test. And likely with autotools too, but I’ve just realized that building with autotools does not provide DEBUG or RELEASE builds, and no flags are added, so it’s only O2.

I can confirm that if I only change -O3 to -O2 in my above flags, then tests does not fail.

You might be able to get some additional information as follows:

For CMake, look in the build directory for the file: Testing/Temporary/LastTest.log
Search (I think) for the string "Test Fail"

For Automake, look for a file named nc_test.log

Here it is:
*** testing nc_put_var_uchar ... FAIL nc_test (exit status: 139)

@DennisHeimbigner Mine is attached here: https://github.com/Unidata/netcdf-c/issues/500#issuecomment-338374512

@ArchangeGabriel when I try that link I get an error message that there is no such file.

Hum indeed. Maybe the file expired, though that’s not normal. Re-uploaded here: https://gist.github.com/ArchangeGabriel/a5d0abb6363b31f71ce4ad44736c60da

Looking at the log, it appears the problem is in nc_put_att and nc_rename att.

With some of the recent memory issues being addressed via PR's, does this issue persist? I'm unable to recreate it on my end.

@ArchangeGabriel I have looked at your logs.
@WardF was there some change merged recently relating to the unsigned/sign char issues somewhere?
Also, @ArchangeGabriel, can you try:
cd nc_test4
make check

The challenge with nc_test is that it's a heck of a program to debug. If we can find a simpler test that is failing, that could make it a lot easier to find the problem.
Also have you tried this with the autotools build?

Still failing with CMake and O3. Confirmed failing on
Unrelated, but I’ve noticed a compilation warning:
/build/netcdf/src/netcdf-c/libdispatch/dutil.c: In function ‘NC_mktmp’:
/build/netcdf/src/netcdf-c/libdispatch/dutil.c:249:15: warning: return makes pointer from integer without a cast [-Wint-conversion]
        return (NC_EPERM);
               ^

One more in ncgen3:
gcc -DHAVE_CONFIG_H -I. -I..  -I../include -I../oc2  -D_FORTIFY_SOURCE=2  -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -O3 -MT main.o -MD -MP -MF $depbase.Tpo -c -o main.o main.c &&\
mv -f $depbase.Tpo $depbase.Po
main.c: In function ‘main’:
main.c:482:10: warning: ignoring return value of ‘fread’, declared with attribute warn_unused_result [-Wunused-result]
          (void)fread(bom,1,1,fp);
          ^~~~~~~~~~~~~~~~~~~~~~~

One in a test:
gcc -DHAVE_CONFIG_H -I. -I..  -I../include -I../oc2  -I../libsrc -DTOPSRCDIR=/build/netcdf/src/netcdf-c -DTOPBINDIR= -I../liblib -I../include -I../libsrc -D_FORTIFY_SOURCE=2  -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -O3 -MT tst_utf8_validate.o -MD -MP -MF $depbase.Tpo -c -o tst_utf8_validate.o tst_utf8_validate.c &&\
mv -f $depbase.Tpo $depbase.Po
tst_utf8_validate.c:98:44: warning: null character(s) preserved in literal
 {0,"2.1.1", "1 byte  (U-00000000)",        " "},
                                            ^

Now running the check in nc_test4.

One warning in this folder:
gcc -DHAVE_CONFIG_H -I. -I..  -I../include -I../oc2  -D_FORTIFY_SOURCE=2  -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -O3 -MT tst_camrun.o -MD -MP -MF $depbase.Tpo -c -o tst_camrun.o tst_camrun.c &&\
mv -f $depbase.Tpo $depbase.Po
tst_camrun.c: In function ‘get_mem_used2’:
tst_camrun.c:685:7: warning: ignoring return value of ‘fscanf’, declared with attribute warn_unused_result [-Wunused-result]
       fscanf(pf, "%u %u %u %u %u %u", &size, &resident, &share,
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
       &text, &lib, &data);
       ~~~~~~~~~~~~~~~~~~~

make check worked in nc_test4.

Thanks for confirming the issue is still there. I’ll take a look shortly or we will see if Ed has an idea/contribution (given the rate of recent contributions!)

Howdy @ArchangeGabriel! Thanks for the detailed info! ;-)
No need to tell us about warnings. There are lots of warnings in the code that are going to go away shortly (when outstanding PRs get merged). But we still won't be warning-free, yet.
Did you try the autotools build? Did it work or not work?

OK, I am reproducing your problems. There are some new warnings, as well as a lot of warnings that are old-frenimies. So I will wait until @WardF merges my outstanding PRs, which clear up a lot of warnings.

Yes, we already have determined above that -O3 was the culprit. You can try with CMake or autotools, as long as you’re just -O2 you’re fine. But with -O3 both fail.
The reason why CMake revealed it is that its Release mode includes -O3, while autotools has no such mode and never adds -O3. But if you specify it yourself, it then fails.

Try add -fno-inline when using -O3, i.e. CFLAGS="-O3 -fno-inline".

A bit late then, but I can confirm that it works with -O3 -fno-inline. :)
I can retry with just -O3 and your patch if you want. ;)


Awesome books website with navigation

There are three pages on this website which you can switch to by clicking the navigation elements List. Add New. Contact

The project looks neat and clean. Nice work done harry. 🎉🙌


How to use ":not(:disabled)" selector?

With a spec:
subject { render }
it { should have_selector "select:not(:disabled)" }

I get:
RuntimeError:
       xmlXPathCompOpEval: function disabled not found

It would be good if this would be supported since :disabled pseudo-class is part of w3.org.
Current workaround is to assert against a specific css class added to the element or wrapper.

Capybara uses Nokogiri to convert CSS selectors to XPath. I think
Nokogiri only support CSS2 selectors and some CSS3 selectors, not
the full spec. Just FYI, :not isn't part of any CSS spec as far as I
can see, I think that's something the jQuery guys just made up.
/Jonas
On Wed, Feb 15, 2012 at 7:42 AM, Dmytrii Nagirniak
<EMAIL_ADDRESS>wrote:

With a spec:
subject { render }
it { should have_selector "select:not(:disabled)" }

I get:
RuntimeError:
      xmlXPathCompOpEval: function disabled not found

It would be good if this would be supported since :disabled pseudo-class is part of w3.org.
Current workaround is to assert against a specific css class added to the element or wrapper.

Reply to this email directly or view it on GitHub:
https://github.com/jnicklas/capybara/issues/640


Added directory column in songs table to track location of fingerprinted songs

Nothing major changed, just added an extra element to _fingerprint_worker and a new column to the songs table.

Can you help me to build this system for me?


Add the seed parameter

https://platform.openai.com/docs/api-reference/chat/create#chat-create-seed
Default should be nil
Hide the json key with omitempty in case it is nil
Test the reflection logic to see if pointers to strings work as expected

Note: wait until the seed parameter is no longer in Beta.

Fixed here: https://github.com/kardolus/chatgpt-cli/commit/65dc6101e84410bd08fcb5799b4938993d49eca0


Update: Return selected item on wheel start

You can return the selected item when the wheel starts, with this data we could prevent some items are selected multiple times or any other strategy using wheel.value.reset() before the wheel start and running again wheel.value.launchWheel () for necessary times after the wheel unselect the option that we wan't

I saw that you're working on an option to define the selected item, this could be a temporary fix for the package

If my understanding is right, the wheel-result-index property can already force the wanted result which will be transfered across both events wheel-start and wheel-end


Compatibility certification request for EI of JXWS 4.0.0

[x] Organization Name ("Organization") and, if applicable, URL:
Eclipse Foundation
[x] Product Name, Version and download URL (if applicable):
Eclipse Implementation of Jakarta XML Web Services 4.0.0
[x] Specification Name, Version and download URL:
Jakarta XML Web Services 4.0
[x] TCK Version, digital SHA-256 fingerprint and download URL:
Jakarta XML Web Services TCK 4.0.0, SHA-256: fceaf20e7da6cc898ce46941b5f324968cef53f621c2e36ca5dea1ba616ddcb7
[x] Public URL of TCK Results Summary:
TCK results summary
[x] Any Additional Specification Certification Requirements:
None
[x] Java runtime used to run the implementation:
OpenJDK 11.0.2+9
[x] Summary of the information for the certification environment, operating system, cloud, ...:
Linux
[x] By checking this box I acknowledge that the Organization I represent accepts the terms of the EFTL.
[x] By checking this box I attest that all TCK requirements have been met, including any compatibility rules.

accepted by lazy consensus


issue

it randomly came up with a log in screen, and whenever i click it, it does the authentication process and then says this:

absolute lifesaver


Update azure and dns discovery dependencies

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@eed8729). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #6852   +/-   ##
=========================================
  Coverage          ?   83.06%
  Complexity        ?    13277
=========================================
  Files             ?     1526
  Lines             ?    49377
  Branches          ?     5937
=========================================
  Hits              ?    41014
  Misses            ?     5569
  Partials          ?     2794

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 eed8729...82d9d20. Read the comment docs.


ShouldSerializeAProperty() method not called if marked as internal

Hi,
following your guide about ignoring properties at runtime, I faced a problem with the internal access modifier. If the method ShouldSerializeManager() is marked as internal, then the library won't invoke it even if I put the following assembly's attribute on my namespace:
[assembly: InternalsVisibleTo("Newtonsoft.Json")]

This works:
public class Employee
{
    public string Name { get; set; }
    public Employee Manager { get; set; }

    // Called!
    public bool ShouldSerializeManager()
    {
        // don't serialize the Manager property if an employee is their own manager
        return (Manager != this);
    }
}

This one not:
[assembly: InternalsVisibleTo("Newtonsoft.Json")]
namespace SomeNamespace
{
    public class Employee
    {
        public string Name { get; set; }
        public Employee Manager { get; set; }

        // Never called!
        internal bool ShouldSerializeManager()
        {
            // don't serialize the Manager property if an employee is their own manager
            return (Manager != this);
        }
    }
}

I think this is due to reflection queries filters. You should also filter by internal fields, not only public ones.
Thanks for your efforts.
Best regards.

I think the issue is here.
Calling GetMethod() without specifying any BindingFlags, will result in a query for all and only public fields. You should specify something like BindingFlags.Instance.

Duplicate of #1324

Closing as a duplicate


Recent installation issue

When trying to install with python setup.py develop or while running tox -e build we have this issue:
error in idpconfgen setup command: 'python_requires' must be a string containing valid version specifiers; Invalid specifier: '>=3.8.*'
Has been solved by fixing a line in the setup.py file:
@@ -83,7 +83,7 @@ setup(
     keywords=[
         # eg: 'keyword1', 'keyword2', 'keyword3',
         ],
-    python_requires='>=3.8.*,<4',
+    python_requires='>=3.8,<4',
     install_requires=[
         ],
     extras_require={

Soon the CI would need to be a litle bit updated. Thanks for spotting this. See also: https://github.com/joaomcteixeira/python-project-skeleton/pull/33 and recently closed PRs.
Cheers!


Health bar

I could not find suitable texture from resources for health bar, so I decided to only implement text for now (remainingHP / totalHP).

Is it okay to add separate texture to res folder? I'd add an actual bar too :slightly_smiling_face:

Wasn't sure whether TankHealthBar class should contain logic (i.e. take_hit that is in Tank class). For now it is only for displaying the health status
take_hit calculates remaining health and returns bool depending if health is bigger than 0. Not sure if I should move the mHealth <= 0 ? true : false condition to another function like void is_destroyed
Maybe KeyboardPlayer and DummyPlayer should have a common base class? They seem the same except for the update function

I will merge this PR after adding some UTs to the helper function :) Great work!


RegEx -> XPath

Changed RegEx searches to XPath searches (supported by selenium)

I should've did it this way when i created it.
Completely rewrote the code now without any regex.


Create CI/CD deployment flow for eCR Viewer

Background
The home stretch! It's time to get the DIBBs components in the cloud. eCR Viewer is half of our equation.
Action requested
Work with the Streamline eCR team to get the eCR viewer deployed in the shared CDC cluster, along with updates to the team's release workflow to automatically deploy out to us.
Acceptance Criteria

eCR viewer workload exists in our cluster
Workload behaves as expected

Additional context
The CI/CD integration may be achieved through GH actions OR through Octopus Deploy. If we go with the former, we may want to create another ticket for the latter to keep track of that work.

Superseded by "DIBBs Central Hosting Redux" epic. Architecturally obsolete. Closing as "Won't Do".


 LB stuck in initializing/reinitializing was caused by the bug when ip address was changed for the instance that was restarting

Version - master 11/06
Verified fixed


Get attribute names for replacement fields

Didn't add an issue for this since it's a pretty minor fix.
All it does is calls this._getAttributeName for all custom attribute replacements that has parameters referencing other fields.

@hillerstorm can you add tests for your changes?

@skaterdav85 done :) also added the same rule that was asked for in #138

thanks! released on 3.13.6


Update Fields setting pre defined parameter as null to fix compatibility with Nova 4

Hello,
I'm with some problems with laravel nova excel package after update my laravel to version 10.
This version use Nova 4 and  seems not be supported with laravel nova excel.
To fixe it we made a change in vendor folder, in file ExportToExcel, in method "fields", setting NovaRequest $request parameter as pre defined as null.
But make changes in vendor folder it's not a nice thing to do, so i made a custom solution here!

Cannot reproduce. Looking at the latest version of Nova (4.28.0) the method signature hasn't change. It's still public function fields(NovaRequest $request)


await asynccp.delay(seconds=0.1) seems to delay for a full second?

We're using https://github.com/WarriorOfWire/CircuitPython_async for async and for some reason it doesn't want to make delays any shorter than 1 second.

https://learn.adafruit.com/cooperative-multitasking-in-circuitpython-with-asyncio
Adafruit has its own asyncio library. That might be a little more put-together.

Fixed in #28


Enhancement: Add support for Indian languages

Hi
As far as I can see, sacrebleu does not have support for Indian languages. This should be a matter of adding a an option to call the tokenizer in https://github.com/anoopkunchukuttan/indic_nlp_library . I see the tokenizer is getting a refactoring at the moment. I'm happy add indicnlp - just wondering what the preferred route would be, and plans for the tokenizer?
cheers - Barry

Actually Indic tokenisation is generally straightforward so this may be a non-issue, if the international option handles it.

I would say that tokenization in BLEU does not need to be perfect from the linguistic and language-specific point of view, it should be just approximately adequate (separating punctuation, not keeping the whole sentence as a single token in Chinese/Japanese/Thai...), standardized (easy to replicate outside of SacreBLEU) and stable (so that it does not change with each version of SacreBLEU/IndicNLP/MeCab).
I am no expert on the languages of India, but I think that only ancient forms of the languages (vedic Sanskrit,  Kannada) use script continuo (no spaces between words), otherwise the current -tok intl should be OK. However, IndicNLP does not seem to handle scripto continuo anyway, I see just a list of punctuation, which should be a subset of all Unicode punctuation symbols, so I don't see any strong reason for adding IndicNLP into SacreBLEU.

Makes sense.


Alternative to VcXsrv/MobaXterm

Is your feature request related to a problem:
Nope
Describe the solution you'd like:
Is there any way to use this as alternative to VcXsrv/MobaXterm? What I can think of, if i have a Linux running somewhere and accessible by ssh, I can port map the DISPLAY to localhost so any GUI apps can be presented by WSLg.
Describe alternatives you've considered:
Nope, just wondering if I have WSLg installed why do I need other Xserver.
Additional context:
No response

WSLG doesn't seem to be accessible to other Windows applications, but if you launch your SSH session from within WSL, it will have access to launch GUI apps within WSL as separate windows on your desktop. It's worth knowing that while this works, the hosted windows don't have any of the Windows window-management options like snapping, and are often temperamental with maximising, etc. - see #22.

MobaXterm v23.0 build 5042 here...
Disable or shutdown the X server then reset your DISPLAY variable:
export DISPLAY=:0
Works for "native connector", "localhost connection" and "wsl command".
+1:
It would be nice if MobaXterm could do this automagically so that we don't have to tweak our DISPLAY manually...
(under WSL2 only of course)...
It would be nice also because MobaXterm already tries to tweak the DISPLAY and with the above manual tweak then announce from the banner message makes the DISPLAY inconsistent with what we actually define manually.
BTW under WSL2 MobaXterms says it uses "MYEXTERNALIP:0.0" in the banner but echo $DISPLAY shows "<IP_ADDRESS>:0.0" ... strange ...


Create release tarball without preserving current user

As of now the tarball we produce from the CI to distribute,
have user/group info associated with current jenkins user, with
this commit we remove this user:group info.
$ tar -tvf crc-linux-amd64.tar.xz
drwxrwxr-x prkumar/prkumar   0 2020-06-01 03:54 crc-linux-1.11.0-amd64/
-rw-rw-r-- prkumar/prkumar 11325 2020-06-01 03:54 crc-linux-1.11.0-amd64/LICENSE
-rw-r--r-- prkumar/prkumar<PHONE_NUMBER>-06-01 03:54 crc-linux-1.11.0-amd64/doc.pdf
-rwxrwxr-x prkumar/prkumar<PHONE_NUMBER> 2020-06-01 03:55 crc-linux-1.11.0-amd64/crc

https://www.gnu.org/software/tar/manual/html_section/tar_33.html#SEC69

I am not seeing the point... as when you change this to root you actually
want people to perform a chown?
it would make more sense to use 1000 ?
On Mon, Jun 8, 2020 at 2:59 PM Christophe Fergeau<EMAIL_ADDRESS>wrote:

@cfergeau approved this pull request.
For what it's worth, even GNU tar tarballs are not changing this ;)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/code-ready/crc/pull/1295#pullrequestreview-425957959,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAOZXN6ACJIPHTKW3RZRDRVSD6ZANCNFSM4NX7JTGQ
.

--
Gerard Braad | http://gbraad.nl
STEM is the source, but BUILD is how you develop!
[ Better Understanding Involves Learning and Doing ]

I am not seeing the point... as when you change this to root you actually
want people to perform a chown?

@gbraad it is not the user:group ownership when extract the bits but when you just list the content for the tarball.

I am not seeing the point... as when you change this to root you actually want people to perform a chown? it would make more sense to use 1000 ?

In my testing, tar tvf showed the files as owned by root:root, but after extracting them, they are owned by my current user.


update readme

update readme to be current

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.

Your review app has been deployed and is available at 'http://<IP_ADDRESS>:80'.Learn More about how to test and provide feedback for the app.


Elkhart Lake Unsupported

Hello Intel,
the tool looks really nice. But when i try to run it on my Intel Atom Elkhart Lake machine i receive following output:
=====  Processor information  =====
Linux arch_perfmon flag  : yes
Hybrid processor         : no
IBRS and IBPB supported  : yes
STIBP supported          : yes
Spec arch caps supported : yes
Max CPUID level          : 27
CPU model number         : 150
Error: unsupported processor.  CPU model number: 150 Brand: "Intel Atom(R) x6414RE Processor @ 1.50GHz"
Access to Intel(r) Performance Counter Monitor has denied (no MSR or PCI CFG space access).

Is the Elkhart Lake unsupported by this tool?
Thanks for your help.
BR,
Andreas

Hi Andreas, unfortunately we do not support Elkhart Lake in pcm yet.

Hi @rdementi,
thanks for the clarification.
Do you have a raw guess what is needed to add support to this tool? Is there an guide available how to add the a new processor? In general is the processor appending driven by Intel or the community?
Thanks for your help.
BR,
Andreas

Hi Andreas,
I see that Elkhart Lake has only core events: https://github.com/intel/perfmon/tree/main/EHL/events therefore the tools/feature support will be limited in PCM, similar to Apollo Lake.
Usually the new processor support is added by Intel employees. This case is more difficult because I don't have access to Elkhart Lake. I prepared a debug change re-using some of the Snowridge code because Elkhart Lake has the same Tremont core:
https://github.com/intel/pcm/tree/EHL-debug
Could you please try it and let me know if the pcm utility works?

Hi @rdementi,
the branch https://github.com/intel/pcm/tree/EHL-debug seems to work:
$ sudo ./pcm -i=1

 Intel(r) Performance Counter Monitor ($Format:%ci ID=%h$)

=====  Processor information  =====
Linux arch_perfmon flag  : yes
Hybrid processor         : no
IBRS and IBPB supported  : yes
STIBP supported          : yes
Spec arch caps supported : yes
Max CPUID level          : 27
CPU model number         : 150
Number of physical cores: 4
Number of logical cores: 4
Number of online logical cores: 4
Threads (logical cores) per physical core: 1
Num sockets: 1
Physical cores per socket: 4
Last level cache slices per socket: 1
Core PMU (perfmon) version: 5
Number of core PMU generic (programmable) counters: 4
Width of generic (programmable) counters: 48 bits
Number of core PMU fixed counters: 3
Width of fixed counters: 48 bits
Nominal core frequency:<PHONE_NUMBER> Hz
IBRS enabled in the kernel   : yes
STIBP enabled in the kernel  : no
The processor is not susceptible to Rogue Data Cache Load: yes
The processor supports enhanced IBRS                     : yes
Package thermal spec power: 9 Watt; Package minimum power: 0 Watt; Package maximum power: 0 Watt;

PCM warning: found 0 uboxes. Expected 1
INFO: Linux perf interface to program uncore PMUs is NOT present
Socket 0: 0 PCU units detected. 0 IIO units detected. 0 IRP units detected. 0 CHA/CBO units detected. 0 MDF units detected. 0 UBOX units detected. 0 CXL units detected. 0 PCIE_GEN5x16 units detected. 0 PCIE_GEN5x8 units detected.

 Disabling NMI watchdog since it consumes one hw-PMU counter. To keep NMI watchdog set environment variable PCM_KEEP_NMI_WATCHDOG=1 (this reduces the core metrics set)
 Closed perf event handles
Trying to use Linux perf events...
Successfully programmed on-core PMU using Linux perf

Detected Intel Atom(R) x6414RE Processor @ 1.50GHz "Intel(r) microarchitecture codename Snowridge" stepping 1 microcode level 0x1a

 UTIL  : utlization (same as core C0 state active state residency, the value is in 0..1)
 IPC   : instructions per CPU cycle
 CFREQ : core frequency in Ghz
 L3MISS: L3 (read) cache misses
 L2MISS: L2 (read) cache misses (including other core's L2 cache *hits*)
 L3HIT : L3 (read) cache hit ratio (0.00-1.00)
 L2HIT : L2 cache hit ratio (0.00-1.00)
 L3MPI : number of L3 (read) cache misses per instruction
 L2MPI : number of L2 (read) cache misses per instruction
 READ  : bytes read from main memory controller (in GBytes)
 WRITE : bytes written to main memory controller (in GBytes)
 TEMP  : Temperature reading in 1 degree Celsius relative to the TjMax temperature (thermal headroom): 0 corresponds to the max temperature
 energy: Energy in Joules

 Core (SKT) | UTIL | IPC  | CFREQ | L3MISS | L2MISS | L3HIT | L2HIT | L3MPI | L2MPI |  TEMP

   0    0     0.00   0.17    1.01    1505     1819      0.89    0.49  0.0048  0.0058     56
   1    0     0.00   0.18    1.08     561      447      0.72    0.75  0.0044  0.0035     56
   2    0     0.01   0.05    0.96    1373     4748      0.97    0.32  0.0031  0.0106     56
   3    0     0.00   0.37    1.03    4142     5445      0.83    0.34  0.0034  0.0045     56
---------------------------------------------------------------------------------------------------------------
 SKT    0     0.00   0.14    0.99    7581       12 K    0.91    0.39  0.0036  0.0059     48
---------------------------------------------------------------------------------------------------------------
 TOTAL  *     0.00   0.14    0.99    7581       12 K    0.91    0.39  0.0036  0.0059     N/A

 Instructions retired: 2094 K ; Active cycles:   15 M ; Time (TSC): 1499 Mticks ; C0 (active,non-halted) core residency: 0.39 %

 C1 core residency: 0.00 %; C3 core residency: 0.00 %; C6 core residency: 99.24 %; C7 core residency: 99.15 %;
 C0 package residency: 100.00 %; C2 package residency: 0.00 %; C4 package residency: 0.00 %; C6 package residency: 0.00 %;
                             ┌──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┐
 Core    C-state distribution│66666666666666666666666666666666666666666666666666666666666666666666666666666667777777777777777777777777777777777777777777777777777777777777777777777777777777│
                             └──────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┘
                             ┌────────────────────────────────────────────────────────────────────────────────┐
 Package C-state distribution│00000000000000000000000000000000000000000000000000000000000000000000000000000000│
                             └────────────────────────────────────────────────────────────────────────────────┘
---------------------------------------------------------------------------------------------------------------

MEM (GB)->|  READ |  WRITE | CPU energy |
---------------------------------------------------------------------------------------------------------------
 SKT   0     0.00     0.00       3.14
---------------------------------------------------------------------------------------------------------------

Cleaning up
 Closed perf event handles
 Zeroed uncore PMU registers
 Re-enabling NMI watchdog.

Thanks for adding support.
BR,
Andreas

Hi Andreas, only the main pcm utility is expected to work on your CPU. Other pcm utilities require special hardware performance monitoring units which are not present in your CPU.

addressed in https://github.com/intel/pcm/pull/836/commits/53aab34d11b5ee26f8c9b231779f58ee93239a7a

Hi Andreas, only the main pcm utility is expected to work on your CPU. Other pcm utilities require special hardware performance monitoring units which are not present in your CPU.

Thanks for clarification and support.


Tests refactoring and typo fixes

Minor typos fixes in comments and refactoring of tests to use beforeEach when possible.
Refs #259

Coverage remained the same at 83.508% when pulling 6904f120328986284dc5090376234f49287b7fad on rudygodoy:master into c3a30e9be30bd5a08e83f34534de482585df2ee6 on OpenZeppelin:master.

Coverage remained the same at 83.508% when pulling 6735a3ccd61c6f2921bec2cd3ca91b1964f86769 on rudygodoy:master into c3a30e9be30bd5a08e83f34534de482585df2ee6 on OpenZeppelin:master.

Thanks @rudygodoy!


Error HH604: Error running JSON-RPC server: ERROR processing

I try to install hardhat-ethernal, but when I put the import hardhat-ethernal in the hardhat config file, I then got the error on my deploy file

Error HH604: Error running JSON-RPC server: ERROR processing /home/nathan/ethereum/game/packages/hardhat/deploy/120_deploy_packages.ts:
Error: invalid signer (argument="signer", value="0x70997970C51812dc3A010C7d01b50e0d17dc79C8", code=INVALID_ARGUMENT, version=contracts/5.1.1)

The line that raise the error in my deploy file:
  const diamondContract = ((await ethers.getContractAt(
      "Diamond",
      diamondAddress,
      admin,
  )) as unknown) as Diamond;

I'm using hardhat-deploy and hardhat-deploy-ethers. If I remove the import hardhat-ethernal  it's working again. Any idea how to fix this please?

got a solution. I think there is somewhere a link that allows to retrieve the Signer object from a string address, and hardhat-ethernal seems to not use this link so it cant retrieve the Signer. So I try to pass directly the Signer object instead of the address string and it's working. Here is the solution:
        let adminSigner = await ethers.getSigner(admin);
        const diamondContract = ((await ethers.getContractAt(
            "Diamond",
            diamondAddress,
            adminSigner ,
        )) as unknown) as Diamond;

Also I put here the full error log (my bad I should have put it at the beginning) maybe it could help you to add this feature
Error HH604: Error running JSON-RPC server: ERROR processing /home/nathan/test/game/packages/hardhat/deploy/120_deploy_packages.ts:
Error: invalid signer (argument="signer", value="0x70997970C51812dc3A010C7d01b50e0d17dc79C8", code=INVALID_ARGUMENT, version=contracts/5.1.1)
    at Logger.makeError (/home/nathan/test/game/node_modules/@ethersproject/logger/src.ts/index.ts:205:28)
    at Logger.throwError (/home/nathan/test/game/node_modules/@ethersproject/logger/src.ts/index.ts:217:20)
    at Logger.throwArgumentError (/home/nathan/test/game/node_modules/@ethersproject/logger/src.ts/index.ts:221:21)
    at new ContractFactory (/home/nathan/test/game/node_modules/@ethersproject/contracts/src.ts/index.ts:1130:20)
    at getContractFactoryByAbiAndBytecode (/home/nathan/test/game/node_modules/hardhat-ethernal/node_modules/@nomiclabs/hardhat-ethers/src/internal/helpers.ts:259:10)
    at getContractAt (/home/nathan/test/game/node_modules/hardhat-ethernal/node_modules/@nomiclabs/hardhat-ethers/src/internal/helpers.ts:272:27)
    at Object.func (/home/nathan/test/game/packages/hardhat/deploy/120_deploy_packages.ts:112:35)
    at DeploymentsManager.executeDeployScripts (/home/nathan/test/game/node_modules/hardhat-deploy/src/DeploymentsManager.ts:1006:22)
    at DeploymentsManager.runDeploy (/home/nathan/test/game/node_modules/hardhat-deploy/src/DeploymentsManager.ts:844:5)
    at Environment._runTaskDefinition (/home/nathan/test/game/node_modules/hardhat/src/internal/core/runtime-environment.ts:217:14)

Ah nice! I was suspecting it had something to do with signers as well. But I'm not sure why Ethernal makes this fail, there isn't request sent or signed through the package...
However, I've seen that there might be some conflicts between packages using hardhat-ethers/hardhat-ethers-deploy, maybe it comes from there. I will talk to the Hardhat team about this, maybe they've an idea about what's going on here.
In any case, I'm glad you were able to fix this!!

Yes I also think it must be a conflict between packages. Anyway thanks for your help, and ça fait plaisir de voir des projets crypto français ;) Bonne continuation !

Ahah merci :)


Ops Man export fails if someone else is logged in

Just need to know if anyone has considered DELETE-ing all current sessions before starting the om export installation.
My Ops Manager export installations fail when someone else is logged into Ops Manager.
A simple "om curl DELETE /api/v0/sessions" should help avoid failed installation exports.
I maybe able to submit a PR for this if interested.

Hey @findneville, we will check with the Ops Manager team on whether this is expected behaviour, and come back to you soon.

Hi @findneville,
We've tried this on PCF 2.2, triggering our pipeline while logged in to Ops Manager, and were unable to reproduce the issue. Can you provide more details?

2.2 is cutting edge. You should be able to reproduce this in Pre 2.2 environments.
Not sure if it is worth checking the opsman version and then applying some logic for backward compatibility.
Thanks!
NEVILLE GEORGE

Hi @findneville
Please could you tell us which version of PCF you see this issue with? And what is the error message in the export-om-installation job?
Thanks
Josh and @aclevername

We are running it on both 2.1 and 2.2 PCF environments.
In the 2.1 environments, the pipeline fails with the following error when someone is logged in
======
Using Client ID credentials
exporting installation
265 B / ? [--------------=----------------------------------------]   2.65% 0s
could not execute "export-installation": failed to export installation: request failed: unexpected response
Running the om_cmd command manually will show you the error that "someone is already logged in. Run the API command to delete existing sessions"

Sorry had to do a lot of edits to the previous comment to figure out the whole  structure within the comment :(

Thank you, we have reproduced the behaviour.

Interestingly.. I thought it would be nice to clean out all sessions before I ran the other commands and it looks like that failed too. My clientuser has opsman.admin scope and authority. It still complains that I have no rights.
myuser } (~/) om -k -c clientuser -s clientpwd -t https://opsman/uaa curl -p https://opsman/api/v0/sessions -x DELETE
Status: 403 Forbidden
Cache-Control: no-cache, no-store
Connection: keep-alive
Content-Type: application/json; charset=utf-8
Date: Wed, 22 Aug 2018 22:24:10 GMT
Expires: Fri, 01 Jan 1990 00:00:00 GMT
Pragma: no-cache
Server: nginx/1.4.6 (Ubuntu)
Strict-Transport-Security: max-age=15552
X-Content-Type-Options: nosniff
X-Frame-Options: SAMEORIGIN
X-Request-Id: ab3ea649-ca92-4e2d-a6de-ad28005a0a09
X-Runtime: 0.069540
X-Xss-Protection: 1; mode=block
{
  "errors": {
    "base": [
      "opsman.admin or opsman.user or opsman.full_control or opsman.restricted_control scope is required to perform this operation"
    ]
  }
}
could not execute "curl": server responded with an error

I went back and checked the scope of the client I was using and it was not set to opsman.admin . I updated the scope of the user and I was able to logout sessions successfully.
We use client credentials because Ops Manager access in our environments are SAML authenticated. Pretty sure that you cannot use user credentials when SAML authentication is enabled.

I think even for the 2.2 versions, I ran into problems with backing up when someone is already logged in.
Either way as a stop gap, I wrote a task that will delete logged in sessions before and after the bbr backups and it works for me. Would be happy to contribute if you feel that this needs to be integrated with the bbr backups

Hi @findneville,
thanks for opening this issue. The team has looked into this and decided that deleting logged-in sessions before exporting OM settings is not something we would like to recommend for our customers. We will let the opsmanger team aware of this request, and close this issue for now.

@pivotal-cf/pcf-ops-manager what's the reason for allowing one logged-in session when using client_id and client_secret? Is there any recommended workaround for this?

Cheers,
Chunyi && @MirahImage


Independent (non-fork) repo

@liggitt How did we get the go-ldap repo marked as an independent (non-fork) repo?
I'd like to initiate that process with this repo too if possible, and if that seems right.

You'd need to contact github support and ask them to detach this repo from the zerklabs/asn1-ber fork network (but leave downstream forks intact)

Thanks @nickcannariato


Android iscroll 5 problem v5

An error occurs when running on android directly v5 not working.
Anyone know how to solve that problem?

maybe try to precise the exact issue ?

Also please set up a reduced test case so others can confirm it:
https://css-tricks.com/reduced-test-cases/


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/169 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit b11a2f4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/index.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/sql-database-engine.png
:white_check_mark:Succeeded

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Utility formula: we need to be more specific about the model and review the math notation (which is currently incorrect).

[ ] Utility formula: we need to be more specific about the model and review the math notation (which is currently incorrect).

for details, see page 376 of Athey et al (2021)
the model assumes unit demand for each category, independent choices across categories, and error term distributed according to Guembel distribution (logit)
there needs to be a discussion on how the outside option is modelled. How does the model choose that the user biuys nothing from a given category? Can we change the value of the outside option in each category or is normalized to 0 for each category?
Regarding notation: (i) need to index the variables by _{uis} and (ii) decompose the utility into a deterministic part and the error term:  $\mathcal{U}{uis} = U{uis} + \varepsilon_{uis}$ .
Then $P(i|u,s)$ is a function of $U_{uis}$ instead of $\mathcal{U}_{uis}$
I suggest we write the utility function that the package can accommodate in its most general form (ie. sum all the terms that can be included) and then discuss each term one by one

I have added a more complete and self-contain description on the BEMB model here, I have fixed the math notation as the following: the utility is \mathcal{U}_{uis} = U_{uis} + \varepsilon_{uis} and the model is modelling the determinsitc part U_{uis}.
Could you please review it to check if there's any mistake in it? @kanodiaayush @charlespebereau


Don't Merge! PR real time test for target branch full build

Docs Build status updates of commit b58af89:
:clock10: Full build: the average full build time is 7 min(s) 22 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit b58af89:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Split outputs

Fixes #5

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@dopplershift @ocefpaf (mainly because you both helped me a lot) or anyone in @conda-forge/core, could you please quickly double check on splitting this recipe to address issue #5? Thanks!

@conda-forge-admin, please rerender

I tested it locally and everything is fine. However, the tests don't run for coreutils if I split it this way for some reason... and I am not sure how to fix that.

I tested it locally and everything is fine. However, the tests don't run for coreutils if I split it this way for some reason... and I am not sure how to fix that.

Possibly a bug in conda build?

@isuruf
Multiple meta files found. The meta.yaml file in the base directory (/tmp/tmpt9bnz8fb/info/recipe) will be used.
WARNING:conda_build.utils:Multiple meta files found. The meta.yaml file in the base directory (/tmp/tmpt9bnz8fb/info/recipe) will be used.
WARNING conda_build.utils:find_recipe(1320): Multiple meta files found. The meta.yaml file in the base directory (/tmp/tmpt9bnz8fb/info/recipe) will be used.

I tested it locally and everything is fine. However, the tests don't run for coreutils if I split it this way for some reason... and I am not sure how to fix that.

Possibly a bug in conda build?

fixed in 5785ff9 by isuruf

@asmeurer could you check if everything is working correctly on your end when you get a chance? We now have the two outputs and soon we will have them in all arches (arm, ppc, aarch) except windows. Let me know if anything could be fixed --- also, happy to add you as maintainer here if you wish

Seems to work for me (Intel Mac).


Display a spinner when listing photos are loading

When a listing photo is loading, it displays as an empty white card, and it's not clear if it's loading or doesn't have a photo

A very simple way to do this would be to set the default background to an animated gif of a spinner.
Then when the listing background is loaded it would cover it.
Currently the default background is blank.

A very simple way to do this would be to set the default background to an animated gif of a spinner.

That sounds easy enough, but wouldn't it be just as easy to have our standard svg spinner embedded in there (stacked below the photo)? That would keep the spinner consistent and it's also theme-able. Without theme-ability, there's a chance someone would choose a background color that would obscure the spinner if it's similarly colored.

Why do the listing photos take so long to load? Shouldn't they be downloaded from the search provider?

Why do the listing photos take so long to load? Shouldn't they be downloaded from the search provider?

No, they are downloaded for each node. We're wouldn't want to centralize the images and give the search provider the responsibility of hosting and serving them.

In order to implement this in the way we've proposed above, we'd need to be quite sure we don't want to support transparent images or if we allow them in, we give them a solid background... which sounds reasonable. @morebrownies @jjeffryes What do you think?

I would think we would add a solid white background over the spinner after the photo loads @rmisio
Is that possible without major scope changes?

I would think we would add a solid white background over the spinner after the photo loads

That's not necessary because we could just specify a background color for the listing image, which would be easier than having to bind to image load events.
Sounds like we're not seeing a need to support image transparency, which means the semi-easy proposed solutions are a go.

Yeah that sounds good 👍

Yeah, detecting when an image has finished loading is a lot more complicated than it sounds. We tried that and decided it wasn't worth it in v1. Solid backgrounds makes it easy.
@rmisio you don't think the spinner would keep repainting if it's covered up, do you? I don't think it would, but if it did for some reason that could be a performance issue.

@rmisio you don't think the spinner would keep repainting if it's covered up, do you? I don't think it would, but if it did for some reason that could be a performance issue.

hmmm.... I don't know. I would assume it did unless I had clear evidence it didn't'. And if it does, it's a problem.
I think we should probably have a chat on this one and see how important it is, since it could be a rabbit hole to do it the right way.

I think we should probably have a chat on this one and see how important it is, since it could be a rabbit hole to do it the right way.

The current experience needs to be improved. Let's figure out level of effort required and we can build it into our backlog.

ok, getting some performance thoughts from the community:
https://stackoverflow.com/questions/46333377/will-hidden-obscured-svg-animations-still-cause-browser-repaints-and-or-perfor


Wrong power factor reading

Some readings of the power factor are around 6000%. Using the latest available release of the integration.

Are you still running your inverter with active firewall? Cannot reproduce the issue at the moment.

No. Did nothing I am aware of that could be causing the issue

Can you run the get-real-data-new command of the hoymiles-wifi library to see what the DTU returns?
https://github.com/suaveolent/hoymiles-wifi


Update kramdown to latest 2.3.0

This pull request no longer applies, as Gemfile.lock has been deleted.


Added cmake variables to make submoduling the project easier

That explains it

Coverage remained the same at 100.0% when pulling ce44286e96bcbba4c3b49f8f93d5a6f35bd65aa8 on abelmul:master into f0cd9e35864b173ef519f9fdba91a731ad94a28d on SRombauts:master.

Hello, thanks for this proposal.
I think that what you should do here is use the provided Find SQLiteCpp cmake file instead, no ?
Cheers

To quote the readme file in order to submodule and add this library you have to do
add_subdirectory(${CMAKE_CURRENT_LIST_DIR}/thirdparty/SQLiteCpp)

include_directories(
  ${CMAKE_CURRENT_LIST_DIR}/thirdparty/SQLiteCpp/include
)

add_executable(main src/main.cpp)
target_link_libraries(main
  SQLiteCpp
  sqlite3
  pthread
  dl
  )

With this pull request you can do
add_subdirectory(${CMAKE_CURRENT_LIST_DIR}/thirdparty/SQLiteCpp)

include_directories(${SQLITECPP_INCLUDE_DIRS})
add_executable(main src/main.cpp)
target_link_libraries(main ${SQLITECPP_LIBRARIES)

I think it keeps application code clean.

If linking multiple libraries is required then that hints at the fact that the SQLiteCpp target is missing some PUBLIC dependencies. That's the modern CMake solution and is preferable to the old school approach of having LIBNAME_INCLUDE_DIRS and LIBNAME_LIBRARIES variables.


Update cloudevents description

Right now it says: A specification for describing event data in a common.
It's missing "way" at the end.
Can we get that updated?
Also, we're not part of "App Definition" can we get that fixed too? We're part of the "Serverless WG".

Sure, we'll update the description, no problem 👍
Regarding the category, we got it from the CNCF landscape. We can update that too, just wanted to double check.

hmmm the icon only appears under Streaming and Messaging. I guess if we can have others listed (like App Def) can we add "Serverless" too?

I think Streaming and Messaging is the subcategory.

At the moment we can have only one, up to you 🙂

Ah ok - I missed that sidebar thingy... let's just let it remain as it then.

Well, except for the missing "way" :-)


cannot read property children of undefined

with
import React, {
  AppRegistry,
  Component,
  Image,
  StyleSheet,
  View,
  ListView,
  Navigator
} from 'react-native';

import Svg,{
    Circle,
    Ellipse,
    G,
    LinearGradient,
    RadialGradient,
    Line,
    Path,
    Polygon,
    Polyline,
    Rect,
    Symbol,
    Text,
    Use,
    Defs,
    Stop
} from 'react-native-art-svg';

var styles = StyleSheet.create({
  container: {
    flex: 1

  }
});

export default class MyApp extends Component {
  render() {

    // Just imagine we're getting loggedInUserID from some other part of the app.
    return(
      <View style={styles.container}>
        <Svg
                height="100"
                width="100"
            >
                <Circle
                    cx="50"
                    cy="50"
                    r="45"
                    stroke="blue"
                    strokeWidth="2.5"
                    fill="green"
                />
                <Rect
                    x="15"
                    y="15"
                    width="70"
                    height="70"
                    stroke="red"
                    strokeWidth="2"
                    fill="yellow"
                />
            </Svg>
          </View>);

  }
}

I get cannot read property children of undefined Svg.js @66:0

It doesn't like the arrow function in Svg.js :
if I replace
getChildren = ()=> {
        return Children.map(this.props.children, child => {
            return cloneElement(child, {
                svgId: this.id
            });
        });
    };

with:
getChildren = function(){
        return Children.map(this.props.children, child => {
            return cloneElement(child, {
                svgId: this.id
            });
        });
    };

it works!

I made a PR #15


Rename causal style files to confirm with *.styles.ts

Description
Rename causal style files to confirm with *.styles.ts
Areas changed

npm packages changed:

[x] responsibleai/causality
[ ] responsibleai/core-ui
[ ] responsibleai/counterfactuals
[ ] responsibleai/dataset-explorer
[ ] responsibleai/fairness
[ ] responsibleai/interpret
[ ] responsibleai/localization
[ ] responsibleai/mlchartlib
[ ] responsibleai/model-assessment

Python packages changed:

[ ] erroranalysis
[ ] raiutils
[ ] raiwidgets
[ ] rai_core_flask
[ ] responsibleai

Tests

[x] No new tests required.
[ ] New tests for the added feature are part of this PR.
[ ] I validated the changes manually.

Screenshots (if appropriate):
Documentation:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Codecov Report

Merging #1342 (61eebf5) into main (9c30636) will decrease coverage by 5.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1342      +/-   ##
==========================================
- Coverage   86.28%   80.43%   -5.86%
==========================================
  Files         100       18      -82
  Lines        4675      322    -4353
==========================================
- Hits         4034      259    -3775
+ Misses        641       63     -578

Flag
Coverage Δ

unittests
80.43% <ø> (-5.86%)
:arrow_down:

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

Impacted Files
Coverage Δ

...is/erroranalysis/_internal/surrogate_error_tree.py

responsibleai/responsibleai/utils.py

responsibleai/responsibleai/_internal/__init__.py

erroranalysis/erroranalysis/_internal/utils.py

responsibleai/responsibleai/_tools/__init__.py

...idgets/raiwidgets/responsibleai_dashboard_input.py

erroranalysis/erroranalysis/_internal/metrics.py

erroranalysis/erroranalysis/version.py

raiwidgets/raiwidgets/interfaces.py

raiwidgets/raiwidgets/model_analysis_dashboard.py

... and 72 more

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 9c30636...61eebf5. Read the comment docs.


Main Window switch Page during Secondary Window Consolidation

Repro steps
If I break on source code while a secondary window is open, it navigates the main window to a blank page. I've traced the issue to this piece of code:
    private void OnViewConsolidated(ApplicationView sender, ApplicationViewConsolidatedEventArgs args)
    {
        ViewModel.Unsubscribe();
        ViewModel = null;
        Bindings.StopTracking();
        frame.Navigate(typeof(Page));
        var appView = ApplicationView.GetForCurrentView();
        appView.Consolidated -= OnViewConsolidated;
        ServiceLocator.DisposeCurrent();
    }

As a workaround I've commented the frame.Navigate line.
Actual Behavior
Following is a video on how to reproduce the issue:

System
Visual Studio 2017
Windows 10 Build 1803

Related to this issue there's one more (let me know if I should create an additional issue).
The secondary Window is kept on the Live Tree and it's not disposed. I've added to the above method a call to Window.Current.Close(); in order to fix it but I don't know if it's the right solution. Please advise.

@codingdna2 Solution you suggest seems to be working correctly,  you can raise a PR with the fix.


Hudi reads failing on upgrade to Hudi 0.12.2  from Hudi 0.11.1

Describe the problem you faced
We have a spark streaming job which does only hudi upsert to load data to the partitions. This is a spark streaming app which only calls hudi upsert and is a single writer MOR table. We have 1000’s of collections/partions where data is upserted at high frequency interval. Lets call it as the main application and it was running on Hudi 0.11.1[emr-6.7.0].
There are nearly 100+ downstream application which reads the data written by this main app based on daily/weekly scheduled run.
These downstream apps uses hudi api to read the source data from main app as well. They range from 0.11 to 0.9 hudi version.
All these downstream apps were able to read the Hudi 0.11.1 (V4) data. But when we changed our main app version to Hudi 0.12.2[emr-6.10.0] they all started failing with attached stacktrace. Basically an emr job running on older V4 hudi version is not able to read data created by Hudi 0.12.2 (v5) version. It is not backward compatible. This seems to be a basic requirement but is not provided from hudi.
We do want to upgrade to latest hudi 0.12.2 version to be able to fix some of long running issues like duplicate primary key issue. But to be able to do this we cannot ask to change all our downstream apps to upgrade as well.
There are  logistics challenges at different levels to do that and doesn’t seem feasible.
To Reproduce
Steps to reproduce the behavior:

Start emr cluster with 6.10 for hudi .12.2  write to a table using the upsert and config properties attached[Hudiconfigs_and_upsertMethod.txt]. Using the hudi config and Upsert method (both in the same attached file) write data to an S3 bucket hudi partition s3://<bucket>/<table>/ . Run this on EMR as a spark job.
Spin up a Jupiter notebook emr to replicate Downstream at 6.7 EMR hudi 0.11 which reads the created table.
Then call from the notebook:
val basePath=s3://<bucket>/<table>/
val ViewDF = spark.read.format("org.apache.hudi").load(basePath)
ViewDF.createOrReplaceTempView("hudi_doc_table_branch_dest")
Reads fail with version 5 failure[ attached stacktrace ]

Expected behavior
Downstream job running on hudi 0.11 should be able to read the data generated from main app running on hudi 0.12.2
Environment Description

Hudi version : 0.12.2

Spark version :  3.3.1

Hive version : Hive not install on EMR Cluster.

Hadoop version : 3.3.3

Storage (HDFS/S3/GCS..) : s3 parquet

Running on Docker? (yes/no) :no

Will 0.13 version of hudi would have backward compatibility to be able to read 0.11 version hudi data? If so what could be the estimated release date for it?
Additional context
Aws ticket on the same : Case ID<PHONE_NUMBER>1
Stacktrace

2023-04-07T09:15:37.718+0000 [INFO] [eb423741-4f44-4fea-b76f-912a6b3f0177] [com.amazon.ws.emr.hadoop.fs.s3n.S3NativeFileSystem] [S3NativeFileSystem]: Opening 's3://a206760-novusdoc-s3-qa-use1/novusdoc/.hoodie/hoodie.properties' for reading
2023-04-07T09:15:37.896+0000 [ERROR] [eb423741-4f44-4fea-b76f-912a6b3f0177] [org.apache.spark.deploy.yarn.ApplicationMaster] [ApplicationMaster]: User class threw exception: org.apache.hudi.exception.HoodieException: Unknown versionCode:5
org.apache.hudi.exception.HoodieException: Unknown versionCode:5
	at org.apache.hudi.common.table.HoodieTableVersion.lambda$versionFromCode$1(HoodieTableVersion.java:58)
	at java.util.Optional.orElseThrow(Optional.java:290)
	at org.apache.hudi.common.table.HoodieTableVersion.versionFromCode(HoodieTableVersion.java:58)
	at org.apache.hudi.common.table.HoodieTableConfig.getTableVersion(HoodieTableConfig.java:472)
	at org.apache.hudi.common.table.HoodieTableConfig.fetchConfigs(HoodieTableConfig.java:305)
	at org.apache.hudi.common.table.HoodieTableConfig.<init>(HoodieTableConfig.java:244)
	at org.apache.hudi.common.table.HoodieTableMetaClient.<init>(HoodieTableMetaClient.java:125)
	at org.apache.hudi.common.table.HoodieTableMetaClient.<init>(HoodieTableMetaClient.java:78)
	at org.apache.hudi.common.table.HoodieTableMetaClient$Builder.build(HoodieTableMetaClient.java:668)
	at org.apache.hudi.HoodieDataSourceHelpers.allCompletedCommitsCompactions(HoodieDataSourceHelpers.java:77)
	at org.apache.hudi.HoodieDataSourceHelpers.latestCommit(HoodieDataSourceHelpers.java:67)
	at com.<pkg>.spark.parser.BaseStreamingParser.batchParser(BaseStreamingParser.scala:210)
	at com.<pkg>.spark.parser.BaseStreamingParser.batchParser$(BaseStreamingParser.scala:146)
	at com.<pkg>.spark.parser.<pkg>$.batchParser(PkgParser.scala:11)
	at com.<pkg>.spark.parser.BaseStreamingParser.main(BaseStreamingParser.scala:129)
	at com.<pkg>.spark.parser.BaseStreamingParser.main$(BaseStreamingParser.scala:96)
	at com.<pkg>.spark.parser.<pkg>$.main(PkgParser.scala:11)
	at com.<pkg>.spark.parser.<pkg>.main(PkgParser.scala)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.apache.spark.deploy.yarn.ApplicationMaster$$anon$2.run(ApplicationMaster.scala:740)
2023-04-07T09:15:37.899+0000 [INFO] [eb423741-4f44-4fea-b76f-912a6b3f0177] [org.apache.spark.deploy.yarn.ApplicationMaster] [ApplicationMaster]: Final app status: FAILED, exitCode: 15, (reason: User class threw exception: org.apache.hudi.exception.HoodieException: Unknown versionCode:5
	at org.apache.hudi.common.table.HoodieTableVersion.lambda$versionFromCode$1(HoodieTableVersion.java:58)
	at java.util.Optional.orElseThrow(Optional.java:290)
	at org.apache.hudi.common.table.HoodieTableVersion.versionFromCode(HoodieTableVersion.java:58)
	at org.apache.hudi.common.table.HoodieTableConfig.getTableVersion(HoodieTableConfig.java:472)
	at org.apache.hudi.common.table.HoodieTableConfig.fetchConfigs(HoodieTableConfig.java:305)
	at org.apache.hudi.common.table.HoodieTableConfig.<init>(HoodieTableConfig.java:244)
	at org.apache.hudi.common.table.HoodieTableMetaClient.<init>(HoodieTableMetaClient.java:125)
	at org.apache.hudi.common.table.HoodieTableMetaClient.<init>(HoodieTableMetaClient.java:78)
	at org.apache.hudi.common.table.HoodieTableMetaClient$Builder.build(HoodieTableMetaClient.java:668)
	at org.apache.hudi.HoodieDataSourceHelpers.allCompletedCommitsCompactions(HoodieDataSourceHelpers.java:77)
	at org.apache.hudi.HoodieDataSourceHelpers.latestCommit(HoodieDataSourceHelpers.java:67)
	at com.<pkg>.spark.parser.BaseStreamingParser.batchParser(BaseStreamingParser.scala:210)
	at com.<pkg>.spark.parser.BaseStreamingParser.batchParser$(BaseStreamingParser.scala:146)
	at com.<pkg>.spark.parser.<pkg>$.batchParser(PkgParser.scala:11)
	at com.<pkg>.spark.parser.BaseStreamingParser.main(BaseStreamingParser.scala:129)
	at com.<pkg>.spark.parser.BaseStreamingParser.main$(BaseStreamingParser.scala:96)
	at com.<pkg>.spark.parser.<pkg>$.main(PkgParser.scala:11)
	at com.<pkg>.spark.parser.<pkg>.main(PkgParser.scala)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at org.apache.spark.deploy.yarn.ApplicationMaster$$anon$2.run(ApplicationMaster.scala:740)
)
2023-04-07T09:15:37.917+0000 [INFO] [eb423741-4f44-4fea-b76f-912a6b3f0177] [org.apache.spark.SparkContext] [SparkContext]: Invoking stop() from shutdown hook
2023-04-07T09:15:37.925+0000 [INFO] [eb423741-4f44-4fea-b76f-912a6b3f0177] [org.sparkproject.jetty.server.AbstractConnector] [AbstractConnector]: Stopped<EMAIL_ADDRESS>(http/1.1)}{<IP_ADDRESS>:0}
2023-04-07T09:15:37.927+0000 [INFO] [eb423741-4f44-4fea-b76f-912a6b3f0177] [org.apache.spark.ui.SparkUI] [SparkUI]: Stopped Spark web UI at http://ip-100-66-72-129.3175.aws-int.thomsonreuters.com:36067
2023-04-07T09:15:37.973+0000 [INFO] [eb423741-4f44-4fea-b76f-912a6b3f0177] [org.apache.spark.MapOutputTrackerMasterEndpoint] [MapOutputTrackerMasterEndpoint]: MapOutputTrackerMasterEndpoint stopped!
2023-04-07T09:15:38.044+0000 [INFO] [eb423741-4f44-4fea-b76f-912a6b3f0177] [org.apache.spark.SparkContext] [SparkContext]: Successfully stopped SparkContext

Hudiconfigs_and_upsertMethod.txt

org.apache.hudi.exception.HoodieException: Unknown versionCode:5

It seems the old hudi bundle jar is still in use.

As Siva commented on duplicate ticket-
" yes. unfortunately, the readers has to be upgraded before you can upgrade the writers. this a known limitation."
You able to upgrade?

Yes we are planning to upgrade all downstream readers first before the writer in upcoming months.
We can close this ticket. Could you please point us to the documentation where this is mentioned.
Thank you @nsivabalan  for the quick response!

@SabyasachiDasTR this is a forward compatibility issue unfortunately, which is not always guaranteed just like most other open source applications. Backwards compatibility is guaranteed. This is called out: https://hudi.apache.org/docs/0.11.0/deployment/#upgrading


refactor: migrate to node-lnd-grpc

Description:
Leverage node-lnd-grpc for low level grpc handling.
Motivation and Context:

Extract low level code into external library that is more easily testable.
Add support for new gRPC sub services

How Has This Been Tested?
Manually
Types of changes:
Refactor
Checklist:

[ ] My code follows the code style of this project.
[ ] I have reviewed and updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[ ] My commits have been squashed into a concise set of changes.

Coverage increased (+0.02%) to 20.114% when pulling 1a759b34f4fd7a48823ca98080b79466b989c1e6 on mrfelton:refactor/node-lnd-grpc into ceca56f5778227359902227c3f8e9d9729ae3c38 on LN-Zap:next.

Coverage increased (+0.02%) to 20.114% when pulling 1a759b34f4fd7a48823ca98080b79466b989c1e6 on mrfelton:refactor/node-lnd-grpc into ceca56f5778227359902227c3f8e9d9729ae3c38 on LN-Zap:next.


Move default_migration_table to a separate database

We have the issue that for test and some development, we sync the data between production (a MySQL slave) and the local database. Obviously, this is a problem because the default_migration_table will be overwritten. What I would like to see is the option to specify a default_migration_database, with a schema of the table name being the schema being synced. For example:
environments:
default_migration_database: phinx
default_database: development
development:
adapter: mysql
host: mysql.pridedallas.com
name: db1
user: xxxxxxxxxxxxx
pass: xxxxxxxxxxx
port: 3306
charset: utf8
So that if default_migration_database is used, instead of default_migration_table, then that the phinx schema would have a table for each schema that migrations are performed on (in this case a table named db1).

Structure would be something like this:

schema: db1
schema: db2
schema: phinxlog
** table: db1 (phinxlog table for db1)
** table: db2 (phinxlog table for db2)

Maybe this helps with https://github.com/robmorgan/phinx/issues/180 as well?

Have you just tried specifying the table name for phinxlog with the dot syntax to set a different database?
eg. default_migration_table = db2.phinxlog
?

Yes, that does work - though I was hoping to separate migrations, but I think we can work with this, thanks!

If you follow the process I described on ticket #180 then this can be done using the same config split.

Actually, that does not work. It worked the first time, but not on subsequent runs:
[InvalidArgumentException]
There was a problem creating the schema table: SQLSTATE[42S01]: Base table
or view already exists: 1050 Table 'phinxlog' already exists

Seems like the migration table creation works, but then at some point when it checks to see if the migration table exists, that comes back negative and then attempts to create it anyway.

The issue is the hasTable() method does not work with database.tablename:
public function hasTable($tableName)
{
    $options = $this->getOptions();

    $exists = $this->fetchRow(sprintf(
        "SELECT TABLE_NAME
        FROM INFORMATION_SCHEMA.TABLES
        WHERE TABLE_SCHEMA = '%s' AND TABLE_NAME = '%s'",
        $options['name'], $tableName
    ));

    return !empty($exists);
}

I'm looking at ways to modify the project to allow for this, but the problem is the database name is set as a single option: "name". Everywhere that that is used, there is no way to say "If this is the schema, use this database, otherwise use this database", without heavy modifications.

I mean, I could do something hacky like create a new option called "schemadb" and then do this:
    public function hasSchemaTable()
    {
        $options = $this->getOptions();
        $name = $options["name"];

        $this->setOptions(["name" => $options["schemadb"]]);

        $return = $this->hasTable($this->getSchemaTableName());

        $this->setOptions(["name" => $name]);

        return $return;
    }

Or going with previous recommended solution of phinxlog.phinxlog (db.tablename), this is a little less hacky of a solution:
    public function hasTable($tableName)
    {
        $options = $this->getOptions();

        if(strstr($tableName, ".")){
            $dbtable = explode(".", $tableName);
            $options['name'] = $dbtable[0];
            $tableName = $dbtable[1];
        }

        $exists = $this->fetchRow(sprintf(
            "SELECT TABLE_NAME
            FROM INFORMATION_SCHEMA.TABLES
            WHERE TABLE_SCHEMA = '%s' AND TABLE_NAME = '%s'",
            $options['name'], $tableName
        ));

        return !empty($exists);
    }

I would like to see that universal change to the project.

So why does Postgres use an option called schema?

If you clone a DB that is migrated to a different point than the phinxlog says, how would you get them in sync? Having phinxlog in the db makes sense.

If you clone a DB that is migrated to a different point than the phinxlog says, how would you get them > in sync?

I'm not even sure what that means.
Can you give an example?
We rely on the migrations themselves to ensure the database is correct. We write our migrations so that if the change is already done, for example, they are skipped. For example: If a column has already been added, we don't add the column again.
So if I clone a DB today and my phinxlog table does not match the cloned DB, then all migrations will be run and bring the DB up to date.

Phinx will only run migrations that don't exist in the phinxlog. So they are run-once. If you now copy a DB that hasn't had all the migrations, you cannot re-run them without dropping phinxlog and then relying on your migrations to NOT repeat themselves in some way. Handling views/stored procedures/triggers/events/etc. all get a bit messy. As do migrations that change data, not just schema.

Yes, I know and that is exactly how we write these. Because we also sync data and schema manually at times between the various copies of this DB (1 production, 4 dev, 1 test), we have to write all of the migrations so that they can only be run once, even if phinx attempts to run the more than once.

It's not even hard to right the migrations with these kind of "checks" in place:
	public function up(){

		if($this->hasTable("premier_creditadj"))
			$this->dropTable("premier_creditadj");

		if($this->hasTable("premier_debitadj"))
			$this->dropTable("premier_debitadj");

	}

Closing due to lack of activity or out of scope. If there is new interest, please open a PR.


vim hanging with some large files

vim-perl is freezing up on a large file i'm editing
only in certain parts
~9k line file, i'll be scrolling through certain areas and then suddenly whole thing freezes up
rolling back to 7.3 tag everything works fine
it’s in a reasonably large sub and seems happen when i start to scroll to the upper or lower bounds of curly braced statements
Will add additional details as I am able to get the time and narrow it down

@fendent Could you try this from within a recent checkout of vim-perl:
vim -c 'setf perl' <(git show p5-corpus-ng:corpus/Regexp-Debugger-0.001016/lib/Regexp/Debugger.pm) +977
...and see if the behavior you're describing shows up there?  When I try to remove lines on that line, Vim is pretty sluggish.

Related to #170, perhaps?


Visualise data in bar charts

User Story : As a data analyst I want to display a selected dataset in the form of bars, so that I can easily see the categories and their weight in the dataset. The data should be sorted in decreasing order.
Acceptance Criteria:
Given a two column dataset, with one column being the categories and the other it’s values, the system should show bars for each of the categorical values set of data in a sorted decreasing order.

Y axis has been implemented in our project repository. There are two different ways to display the bar charts in splite-bars and stacked, we have set the Stacked bar by default and Y-axis data in numeric type.


Analyzer crash on extension method doc reference

The area of code at this offset is below; the offset refers to [extensionNumber].
extension SimpleStringExtension on String {
  /// Print this and [another].
  /// Refer to [indexOf], from [String].
  /// Also refer to [extensionNumber].
  void doStuff(String another) {
    print(this + another);
  }

  int get extensionNumber => 3;
}

--snip--
Analyzer Feedback from IntelliJ
Version information

IDEA IC-192.6817.14
2.6.0-dev.7.0
IC-192.6817.14, JRE 11.0.4+10-b304.69x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.14.6, screens 1440x900  Retina

Exception
Dart analysis server, SDK version 2.6.0-dev.7.0, server version 1.27.3, FATAL error: Failed to handle request: {id: 3783, method: edit.getFixes, params: {file: /Users/jcollins/dart/dartdoc/testing/test_package/lib/example.dart, offset: 14213}, clientRequestTime:<PHONE_NUMBER>519}
NoSuchMethodError: The getter 'length' was called on null.

This is from the testing/test_package subdirectory inside Dartdoc.  The pubspec.yaml:
name: test_package
homepage: http://github.com/dart-lang
description: Best package ever.
version: 0.0.1
dependencies:
  meta: ^1.0.0
  args: ^1.5.0
  test_package_imported:
    path: "../test_package_imported"


Refactor OtherUses Component to New agGrid Setup

Description
Refactor the OtherUses component to use our new agGrid setup, similar to the recently refactored FuelSupply component. This will improve consistency across our application and potentially enhance performance and maintainability.
Purpose and Benefit to User

Improve consistency in user interface and behavior across different parts of the application
Potentially enhance performance and responsiveness of the OtherUses grid
Simplify future maintenance and feature additions by aligning with our new standard setup

Acceptance Criteria

The OtherUses component uses the new agGrid setup, matching the structure and patterns used in the FuelSupply component.
All existing functionality is preserved, including:

Displaying and editing other uses data
Adding new rows
Duplicating rows
Deleting rows
Validation of data

The refactored component correctly interacts with the backend API, including saving changes and retrieving data.
The user interface remains consistent with the current design, despite the underlying changes.
Performance is maintained or improved compared to the current implementation.
All existing unit tests pass, and new tests are added or updated as necessary.

Development Checklist

[ ] Review the current OtherUses component implementation and the new FuelSupply component
[ ] Refactor the OtherUsesColDefs to match the new structure
[ ] Update the AddRowsButton component to work with the new setup
[ ] Refactor the OtherUsesActions component to align with the FuelSupplyActions implementation
[ ] Update the main OtherUses component to use the new grid setup
[ ] Ensure all API interactions (useSaveOtherUses, etc.) are correctly implemented
[ ] Update or add unit tests to cover the refactored code
[ ] Perform manual testing to ensure all functionality works as expected
[ ] Update any relevant documentation

Additional Notes

Pay special attention to any OtherUses-specific logic that may need to be preserved during the refactor
Consider any performance optimizations that can be applied during the refactor
Coordinate with the team to ensure this refactor aligns with any ongoing work or planned changes to the OtherUses feature

Getting the following error when the row should save. Save button shouldn't be there either.


The browser does not load/respond when I type in the script. Can't move on or learn.

Challenge Learn how Script Tags and Document Ready Work has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

jQuery Playground

#left-well

        <button class="btn btn-default target" id="target1">#target1</button>
        <button class="btn btn-default target" id="target2">#target2</button>
        <button class="btn btn-default target" id="target3">#target3</button>

#right-well

        <button class="btn btn-default target" id="target4">#target4</button>
        <button class="btn btn-default target" id="target5">#target5</button>
        <button class="btn btn-default target" id="target6">#target6</button>

The document ready function should be inside your script tags

Much appreciated William! I typed my code over and over, correctly and
incorrectly so MANY times before realizing (and surprisingly enough) the
problem actually happened to be my adbocker that was messing with the
chrome browser.
On Tue, Jun 6, 2017 at 4:19 AM, williamsb<EMAIL_ADDRESS>wrote:

The document ready function should be inside your script tags

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/15229#issuecomment-306429040,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATCBkw4kUphQu0pk5Gr8yM9kNjkIHMyDks5sBRmEgaJpZM4NxBsO
.

--
Miguel J. Angel
The University of Texas at Austin
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>Telephone:<PHONE_NUMBER>

Hi,
Thanks for reporting this issue.
Here are some next steps that you should follow before we can help you with your query:

Check these troubleshooting steps first.
Please use the Chat Rooms or try looking through our forum for help with a specific challenge and any coding assistance.

IMPORTANT:
Please NOTE that this tracker is for reporting bugs and enhancements to freeCodeCamp's code base only.
Happy Coding!

Yes!


[WIP] pkg/cluster: use kubelet.conf instead of cmdline parameters

WIP
Since K8s v1.11, kubelet options should be defined in kubelet.conf instead of cmdline parameters. Still cmdline parameters are not deprecated yet, so it's not an urgent issue for now. Though we need to test this PR for possible scenarios.
Fixes https://github.com/kinvolk/kube-spawn/issues/286

Actually that transition has not happened upstream. https://github.com/kubernetes/kubernetes/pull/65191
Closing.


Fails to build on 0.8.0

Receiving this error when trying to build:
/[...]/node_modules/react-native-tooltip/ToolTipMenu/RCTToolTipText.m:27:23: No visible @interface for 'RCTEventDispatcher' declares the selector 'sendTextEventWithType:reactTag:text:eventCount:'

Any ideas?

v0.8.0 is not supported anymore in the latest version of tooltip. If you still want to use it download v0.1.5 of react-native-tooltip

Thanks! Managed to find time to downgrade. Hitting this error using 0.1.5 on 0.8.0:
[error][tid:main][ToolTipMenu.m:25] Cannot find view with tag #(null)
Any ideas?

Hi ifran,
Like i said, you should upgrade to the latest version. RN is 0.12 now also.
As for the specific error, it seem the view is not passed on correctly to native side. Are you using React.findNodeHandle(this.refs.input)?
Make sure this.refs.input is not null.

Thanks @chirag04. I bit the bullet and managed to upgrade to 0.12. Now to see if I can get tooltip working smoothly! Thanks for getting back to me.

:+1: awesome.
It would be interesting to know how you are using tooltip in your project. just out of curiosity. ;)

this error also occurs on 0.14.0(react-native) when I want to build
no visible @interface for 'RCTEventDispatcher' declares the selector 'sendTextEventWithType:reactTag:text:key:eventCount:' ?

finally, I found _eventDispatcher have changed , so I need modify this to adapt 0.14.0
** [_eventDispatcher sendTextEventWithType:RCTTextEventTypeChange  reactTag:self.reactTag text:text eventCount:_nativeEventCount];**

@passol1988 for reactnative 0.14 use tooltip version 2.3.0.
3.x for this module is supporting react native 15+


Some audio stream don't work

I'm submitting a ...

[x] bug report
[ ] feature request
[ ] support request

General information
Addon version used
Current HEAD
da8ede27ded08a95
I reproduced it also with beta 23 and an older release.
Description
Some audio stream don't work.
Steps to Reproduce

Open "Chilling Adventures of Sabrina", first episode.
English audio stream doesn't work

In some episodes the english stream doesn't work, but it's not always the same.
Episode 1: english, italian, spanish not working
Episode 6: spanish and italian not working
These seem to change over time.
Context (Environment)
Installation

[ ] I'm using this Netflix Repo
[ ] I'm using other Netflix Repo (Please specify which)
[x] I'm using a different source: Github clone

Operating System

[x] Linux (x86/x64)
[ ] OSX (x86/x64)
[ ] Windows (x86/x64)
[x] Linux (ARM)
[ ] Android

I reproduced this with a Raspberry and with Archlinux x64.
Debug log
https://pastebin.com/EGLNWhk6
Other information
This started happening Saturday (the same episode was working the day before). I think something might have changed on the Netflix side...

i tried the episodes mentioned and other episodes, but to me everything language works as it should,
the errors in the logs seem to describe problems in audio streaming,
maybe they've already solved

hey i have the same issue although different for something different; Travelers season 2, episode 6,7,8, default audio stream is silent changing the audio stream to for example "audio description" works OK. I haven't seen this problem on any other programs. running 18.1, Netflix 14,23

For other reasons I'm switching account. On the new one the issue is not present anymore.
Old account is a "basic plan", the new one is a "premium plan". I don't know if the issue depends on the plan or some other account-setting.

I'm on the premium (4 devices / 4K)  not sure it's "plan" specific

I also have a premium account, so i've never had a problem like this before,
in any case from the netflix addon i can not do anything, it concerns instead inputstream addon and/or kodi

Makes sense, I've just played the "silent" episodes ok on libreelec (RPI2), maybe its platform specific (Android?)

@ASHJG
please put a log with debug infos
and the device model name

I'm collecting information, i thought the problema was limited to android 7 of certain devices,
but this on linux it changes the things, with the latest archlinux updates is the problem still there?

I got this problem on Rpi2.
The default language (in my case german) is silent in some episodes whereas English works perfect.

@cpetry can you provide debug logs? Can you also post 'guisettings.xml' ?
The only problem I ever had here on RPi3 is only because passthrough was enabled and the receiver was not able to decode it. I'd like to see your full config.
Thanks.

Here my guisettings:
guisettings.zip
Do I need to enable debug logging somewhere? Didn't find them in userdata

@cpetry yes please debug log it's important. You can enable it here in Kodi: Settings->System->Logging->Enable debug logging, and then try to play the movie/episode with the 'good' audio Trask then switch to the buggy audio track.

now you should no longer have problems, i removed the streams in question in favor of one with more quality, which will probably only be available for premium accounts
you can download the daily build

@cpetry when you have time please make a test and give a feedback thanks

I checked at home ... and did not even know that I had this problem too ...

@CastagnaIT I had no audio on Jessica Jones S03E01 (Linux x86, both 720p and 1080p). Now that I've installed the the daily build it works just fine. Thanks!

Maybe I'll explain - still at 0.13 I had a problem with the Polish soundtrack in many movies. The problem appeared when the soundtrack was 2.0 and 5.1 - sometimes 2.0 did not want to work, 5.1 was OK. But there was always a problem with the volume level, and I will not play cartoons through AMP. So I set Dolby by definition and forgot about it. And since the last day, the daily works beautifully 2.0 :) I did not even know that I had this problem ...

excellent then i close this issue


Use the C++ override keyword

Signed-off-by: Darby Johnston<EMAIL_ADDRESS>Fixes #1542
In C++11 the "override" keyword was added which checks at compile time whether a virtual function has been properly overridden. This can catch bugs from typos in derived classes and ensures that base class destructors are marked virtual. Some interesting discussion here, especially on whether destructors should use "override":
https://github.com/isocpp/CppCoreGuidelines/issues/721
Some compilers (e.g., the latest Xcode) now emit warnings for this:
/Users/darby/Dev/otio/OpenTimelineIO/src/opentimelineio/item.h:36:18: warning: 'visible' overrides a member function but is not marked 'override' [-Winconsistent-missing-override]
    virtual bool visible() const;

This PR changes:

Use the "override" keyword on all functions that override a virtual function in a base class
Make the virtual function SerializableObject::_schema_name_for_reference() protected instead of private
Make the functions UnknownSchema::~ UnknownSchema() and UnknownSchema::_schema_name_for_reference() protected instead of private

To clarify my comment, the outcome of the discussion on the iso issue you linked was the guidance in the document that I linked, and it seems, to my reading of the document, that they concluded destructors should be virtual.

I think that is just for base classes though, once a function has been marked virtual doesn't that make the function virtual in any derived classes? I thought the last comment in the thread described it well "memory leaks (and other nasty problems), when deleting an object through a superclass pointer, because of forgetting to mark the superclass's destructor virtual". However I do think having "override" on the destructors looks a bit odd.

Effective C++ says "If a class has any virtual function, it should have a virtual destructor, and that classes not designed to be base classes or not designed to be used polymorphically should not declare virtual destructors." Although it's true that if a destructor B is not virtual and it's super is A, that deleting a pointer to B will call the destructor on B then A, if you have a non-virtual C pointer, then only C's destructor will run.
So the only case where override without virtual would be logical in our use, is if the class is also final, and the superclass is virtual. Since OTIO leverages polymorphism a lot, I think it is easier to reason about the code if we follow the "Effective" rule.
Effective C++ is older than the override keyword so if Meyers were to revise it he might publish a different opinion, but I think I am convincing myself that override is not going to be as maintainable and predictable as virtual, and that we know for sure we want the "virtual chained destructor" behavior in all cases. I hope this argument convinces you as well ;)

Codecov Report

Merging #1543 (da5646a) into main (2d61921) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1543   +/-   ##
=======================================
  Coverage   80.11%   80.11%
=======================================
  Files         197      197
  Lines       21699    21699
  Branches     4299     4299
=======================================
  Hits        17384    17384
  Misses       2170     2170
  Partials     2145     2145

Flag
Coverage Δ

py-unittests
80.11% <ø> (ø)

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

Impacted Files
Coverage Δ

src/opentimelineio/composable.h
100.00% <ø> (ø)

src/opentimelineio/composition.h
41.37% <ø> (ø)

src/opentimelineio/effect.h
100.00% <ø> (ø)

src/opentimelineio/externalReference.h
100.00% <ø> (ø)

src/opentimelineio/generatorReference.h
100.00% <ø> (ø)

src/opentimelineio/imageSequenceReference.h
48.27% <ø> (ø)

src/opentimelineio/item.h
100.00% <ø> (ø)

src/opentimelineio/linearTimeWarp.h
100.00% <ø> (ø)

src/opentimelineio/marker.h
100.00% <ø> (ø)

src/opentimelineio/mediaReference.h
100.00% <ø> (ø)

... and 7 more

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 2d61921...da5646a. Read the comment docs.

I still think the comment makes a good case for catching non-virtual destructors in base classes, but seeing "override" on destructors is confusing enough maybe it's not worth it. Especially with OTIO since there is only a single base class? I updated the PR replacing override with virtual on the destructors.
Speaking of the single base class, SerializableObject, is that supposed to be instantiable? If not maybe it should have a pure virtual destructor?

Ah, never mind, it looks like at least the type registry system needs to be able to create SerializableObjects.

Thanks for the change! The semantics around override and virtual and destructor are slightly puzzling, that's for sure :)

Do you have further work you want to do on this? If no, I say we are ready to land it :)

I thinks it is good to go, thanks!


Error opening input: No such file or directory - crash on start

python server.py (on Windows 10) gives:
[in#0 @ 0000024b20115740] Error opening input: No such file or directory
Error opening input file ./data/samples/vision_qa_audio.wav.
Error opening input files: No such file or directory

vision_qa_audio.wav should be formed by myself ?

let me check, please wait for moment.

sorry that they are lost when merging changes. we add them in /data/samples, and you can try it after add it.

i put fake vision_qa_audio.wav and vision_qa_image.jpg but still cant start even if i set flask to debug mode=True:

404 error

@zoldaten it seems you use the GET method to do the request, please try post mothed? The sever seems working, do the streamlit work?

ok. seems it`s working! didnt catch that i should start another terminal for app.

addition:
have to install pip install streamlit-webrtc
and there are gaps in online speech after it was made by system.
When i start playing a fragment on steamlit page there is no gaps and audio is clear.

great, I update the requirements.
This issue is related to the computational platform. It appears that your testing platform can only generate 5 tokens per second. For real-time, uninterrupted responses on Streamlit, a theoretical minimum of 13 tokens per second is required. Of course, model inference can be optimized; currently, our model is using the original fp32 format. We hope that in the future, members of the community will be able to assist with further optimizations.

now it`s great enough!
and one more addition misleading in audio-video mode:
even if camera selected and access granted to it (and page reloaded) there is a wrong message on the web page:

it is not interfere in inference process just frustrate.

@zoldaten you need to click START to activate the video stream, we may a demo video later to better illustrate the process.