add DataType::Error

I don't know if every errors are represented as a string.

Thanks!
I am wondering if we shouldn't have an enum instead of a String for the excel errors? Given that we know all kind of possible errors.
What do you think?

I agree that an enum would be better.
I updated the PR.
I found a list of possible errors but  no explanation so I translated the error messages from Excel.
I used the unimplemented!() macro in case we get an error we don't know, maybe we should use unreachable!() if we think we know every errors type. Maybe we shouldn't use either if they aren't in rust stable yet (I didn't test since I think office doesn't run with rust stable).
I wrote a function to parse the error string. It may be better since we can write a test for it like I did. Not sure if it's better to test a xlsx file with the errors or just the function or both. Maybe the function could be inline too.
Not sure if I should name the things Error or CellError. I don't want to confuse people with errors.rs.
summary

is it OK to use a function (parse_error)
test only a xlsx file, only the function or both
is unimplemented ok
CellError(CellErrorType) or Error(ErrorType)

There is a conflict because of #10 merge. Could you rebase? (I you don't have time I'll do it, no worries).
All your decisions look good to me, this is what I had in mind!

I'll rebase quickly.

Done.

:+1:


UWP C++

Is there a version which would support C++/WinRT UWP Apps as well?

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

ID: 06ece9ab-9965-2ddf-34b6-44f0f61fe599
Version Independent ID: 16261b98-a85e-060f-ccf4-3a24f5f14c4c
Content: Get Started with UWP - Visual Studio App Center
Content Source: docs/sdk/getting-started/uwp.md
Service: vs-appcenter
GitHub Login: @elamalani
Microsoft Alias: emalani

@deepaknarula I'll have the PM take a look at your question. Don't forget as well that App Center support is provided directly from within the App Center portal. Any time you need help, just click the blue chat button in the lower-right corner of any page and our dedicated support team will respond. For additional information, see the App Center Help Center.

@deepaknarula We only support UWP apps built with C#. We don't have a windows SDK for C++ or WinRT platform.

Are there any plans for Windows C++ support in SDK?

Hi @markghayden - C++ for UWP apps isn't on our immediate roadmap.
There are a few feature requests around crash reporting support for C++ apps:
https://github.com/microsoft/appcenter/issues/436
https://github.com/microsoft/appcenter/issues/374
Feel free to give these issues a +1 or create a seperate feature request on the App Center repo to help our team prioritize these features.
Thanks!


Update gflags.md

made Gflags to  bold,
removed  duplicate link under see also
Tools included in Debugging Tools for Windows
article link
https://learn.microsoft.com/en-us/windows-hardware/drivers/debugger/gflags

@DOMARS , Below two links, are added in this article two times
https://learn.microsoft.com/en-us/windows-hardware/drivers/debugger/gflags-examples
https://learn.microsoft.com/en-us/windows-hardware/drivers/debugger/global-flag-reference
under This section includes and under see also.
I think, above links must be removed from   under see also,

@RAJU2529 - Thanks for the edits, they look fine. It is OK to have links in the articles repeated in the see also. Some users use search to locate the topic, and give it a quick scan to see if it is what they want. They may skip to the bottom of the article or use the table of contents, to find something else, if their current article is not what they were after. So it would be good to restore the link in the see also in this article.

@RAJU2529 - Thanks for the PR and the interest in the docs. It is OK to duplicate links in the body of the article in the see also. Some people may scan all of the way to the bottom, after reading just the first sentence or so.


User creation fails with privilege 'REPLICATION CLIENT'

Maybe move to the community collection for mysql

Maybe move to the community collection for mysql


Winston.add2 Error

I am not sure what I am doing wrong here.
I am trying to make a basic plot using two axis in a FramedPlot.
x = 0:0.1:10
y1 = 0.05*x.^2
y2 = -y1
p = FramedPlot()
setattr(p.x, "range", (0,10))
setattr(p.y1, "range", (0,5))
setattr(p.y2, "range", (0,-5))
add(p, Curve(x,y1))
Winston.add2(p, Curve(x,y2))

When using Winston.add2 I get:
Error showing value of type FramedPlot:
ERROR: type: non-boolean (Nothing) used in boolean context

I am unsure if this is a usage error on my part or a bug in Winston.
Thanks

I think it is a bug. By default the log attribute of HalfAxis is nothing, which is not a boolean. For the first axes this is fixed by applying the settings from the FramedPlot block in Winston.ini.
My attempt at a fix is here: 202a03be74a1274cab641a39749cf74e6938189e.
I don't have time to test if this breaks anything right now, so I'm not yet creating a pull request, maybe tomorrow.

Well, making that change does allow me to plot thanks!
I ran the tests but I have absolutely no idea if they passed or not (nothing said fail or crashed at least).

Well I think I made a mistake in my original fix. It worked if you actually had two axes, but if you made a single plot with for example semilogy in which the primary axis was logarithmic, the secondary axis would remain non-logarithmic.
The PR is #232, but I'd be happy if somebody could take a look at it and test it a bit.


Re org debian docker

Hey all,
This is new layout.
Most of the changes are moves and changing targets.
The new layout is now like this:

One directory per base image at top level

Each base image directory has a reproducible dir which contains all the scripts and definitions for building reproducible images.

All tests are now under tests/<base_image>/
I moved a bunch of structure tests there.
Here is where you can browse through the dirs.
https://github.com/tejal29/base-images-docker/tree/re-org_debian_docer

@dlorenc , @sharifelgamal , @aaron-prindle PTAL

I thought about that, but ubuntu is not really reproducible. The Dockerfile is outside the ubuntu/reproducible.
I was thinking of removing them when switch to download_pkgs and install_pkgs.
I kept the node/reproducible for the same reason since it depends on ubuntu.
Can get rid of them in another PR.

I kept the node/reproducible for the same reason since it depends on ubuntu.
Can get rid of them in another PR.

SGTM.


OnStopstream using rtmpCamera2.stopStream(); app is crashing.

@pedroSG94 bro!! i'm facing this consistently, when ever i try to stop my live stream on any device it crashes after a second or to delay...
I've gone in underlying classes to identify point of crash but no luck.
It throws an  .IllegalStateException: Recording currently in progress - missing #endRecording() call?
evenThough I'm not using/enabling startRecording option nowhere in my basicActivity.(the UI controller activity)
I'm using latest version code i.e 1.2.7. not as dependency but as module
Here is my Start and StopStream Code.. don't know what am I doing wrong
here in code **webViewRenderer** is Object of **AndroidViewFilterRender**
    private void startLiveStreamAction() {
        if (!rtmpCamera1.isStreaming()) {

            if (rtmpCamera1.prepareAudio() && rtmpCamera1.prepareVideo()) {
                button.setText(R.string.stop_button);
                attachViewsToStream();
                //lockControls();
                rtmpCamera1.startStream(rtmpLink);
            } else {
                Toast.makeText(this, "Error preparing stream, This device cant do it",
                        Toast.LENGTH_SHORT).show();
            }
        } else {
            stopStream();
        }
    }
 private void attachViewsToStream() {
        if (glInterFace != null) {
            if (glInterFace.filtersCount() > 0)
                glInterFace.clearFilters();

            webViewRenderer.setView(clStreamContent);
            glInterFace.addFilter(webViewRenderer);
        }
}
    private void stopStream() {
        if (rtmpCamera1.isStreaming()) {

            button.setText(R.string.live_now);

            rtmpCamera1.stopStream();

        }
    }

CrashLogs
E/FrameEvents: updateAcquireFence: Did not find frame.
E/FrameEvents: updateAcquireFence: Did not find frame.
I/chatty: uid=10321(com.myproj.live) RenderThread identical 7 lines
E/FrameEvents: updateAcquireFence: Did not find frame.
D/AndroidRuntime: Shutting down VM
E/AndroidRuntime: FATAL EXCEPTION: main
    Process: commyproj.live, PID: 18993
    ```
java.lang.IllegalStateException: Recording currently in progress - missing #endRecording() call?
        at android.graphics.RenderNode.beginRecording(RenderNode.java:372)
        at android.view.View.updateDisplayListIfDirty(View.java:21359)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ViewGroup.recreateChildDisplayList(ViewGroup.java:4524)
        at android.view.ViewGroup.dispatchGetDisplayList(ViewGroup.java:4497)
        at android.view.View.updateDisplayListIfDirty(View.java:21346)
        at android.view.ThreadedRenderer.updateViewTreeDisplayList(ThreadedRenderer.java:559)
        at android.view.ThreadedRenderer.updateRootDisplayList(ThreadedRenderer.java:565)
        at android.view.ThreadedRenderer.draw(ThreadedRenderer.java:642)
        at android.view.ViewRootImpl.draw(ViewRootImpl.java:4230)
        at android.view.ViewRootImpl.performDraw(ViewRootImpl.java:3944)
        at android.view.ViewRootImpl.performTraversals(ViewRootImpl.java:3215)
        at android.view.ViewRootImpl.doTraversal(ViewRootImpl.java:2057)
        at android.view.ViewRootImpl$TraversalRunnable.run(ViewRootImpl.java:8360)
        at android.view.Choreographer$CallbackRecord.run(Choreographer.java:1058)
        at android.view.Choreographer.doCallbacks(Choreographer.java:880)
        at android.view.Choreographer.doFrame(Choreographer.java:813)
        at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:1043)
        at android.os.Handler.handleCallback(Handler.java:938)
        at android.os.Handler.dispatchMessage(Handler.java:99)
        at android.os.Looper.loop(Looper.java:236)
        at android.app.ActivityThread.main(ActivityThread.java:7889)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:600)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:967)

I can't see a line related with my library so I supose it is related with android lifecycle.
This look a problem related with AndroidViewFilterRender that try to use your view that is already destroyed.
First of all try reproduce your error without that filter to discard that the filter is the problem.
If the filter is the problem make sure that you clear filters before your activity/fragment anything that you are using to show preview is paused or destroyed to avoid this problem.
Let me know the result of your test to know where I can start to test.

@pedroSG94 I'm clearing filter before stoping preview or stop stream... but this issue still persists...
What I notice is that if is top/start screen in quick successions it doesn't crash... but if I'm streaming for a while .. let's say 3-4 minutes .. it crashes..
Can't understand what I'm doing wrong with the lifecycle/stop streaming events.
Actually, the lifeCycle issue shouldn't be the issue as I am not doing any screen changes in screen configs or anything that relates to activity lifecycle (as I'm standing on same screen just triggering stopEvent).

Try to reproduce it without AndroidViewFilterRender. This seems an error related with canvas rendering:
https://android.googlesource.com/platform/frameworks/base/+/master/graphics/java/android/graphics/RenderNode.java#389
AndroidViewFilterRender is the only side where I'm using canvas.
I will try to reproduce it. I will need your AndroidViewFilterRender configuration (View used)

@pedroSG94 sorry for the late response.. actually I was away for a week!!
here are both my java and XML gist...
plz have a look.
Layout and java class gist url

@pedroSG94 bro please can i have your attention here...

Hi bro... I've added my code link .. plz have a look.
Thanks
On Tue, Mar 8, 2022, 2:23 PM Pedro Sánchez @.***> wrote:

Try to reproduce it without AndroidViewFilterRender. This seems an error
related with canvas rendering:
https://android.googlesource.com/platform/frameworks/base/+/master/graphics/java/android/graphics/RenderNode.java#389
AndroidViewFilterRender is the only side where I'm using canvas.
I will try to reproduce it. I will need your AndroidViewFilterRender
configuration (View used)
—
Reply to this email directly, view it on GitHub
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/issues/1068#issuecomment-1061570951,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG3Q7ANX632JHENNXQTN6U3U64L77ANCNFSM5P5OI77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I already saw that yesterday. I had no time to test it yesterday and today I'm still working.
Please, be patient. I only can work in this library in my free time.
I will notify you if I can reproduce your case or not with your code shared this night or tomorrow.

Sure bro!! actually, that was a panic call (deadLine).. didn't have the slightest of idea what was going wrong, tried to work from a different perspective but not a single one worked for me. Now thinking of re-assessing the filters assignment work again.
Would be waiting for your assessment, if mine still doesn't work.

Hello,
After few tests I can comfirm that the problem is related with AndroidViewFilterRender and stop render it.
You can fix it replacing your stopStream method to this:
    private void stopStream() {
        if (rtmpCamera2.isStreaming()) {
            webViewRenderer.setView(null); //stop render view
            clStreamContent.setVisibility(View.VISIBLE);
            button.setText(R.string.start_button);
            rtmpCamera2.stopStream();
        }
    }

With this line, the problem never happens. I'm looking the exact reason about the problem to avoid need stop render manually in this case.

@pedroSG94 thanks for the solution. Now I can have a sigh of relief!!
There is another issue popping, feel related to this.
when I put my app in the background and resume again.
the screen gets black. Previews stops and
I found that upon resuming **openGlView.getHolder().getSurface().isValid()** always returns null and I cant start preview again, even if I try to reinitialize my opengl using my initializeRTMPCameraInstance() method.
    @Override
    protected void onResume() {
        super.onResume();

       if (rtmpCamera2 != null && !rtmpCamera2.isOnPreview() && openGlView.getHolder().getSurface().isValid())
            rtmpCamera2.startPreview();
    }

Hello,
This is a code problem. You should read app examples.
In this case the problem is that you should start preview only on onSurfaceChanged callback and stop preivew onSurfaceDestroyed callback like in the examples.

Code example is doing the same thing bro.
On Fri, Mar 18, 2022, 4:42 PM Pedro Sánchez @.***>
wrote:

Hello,
This is a code problem. You should read app examples.
In this case the problem is that you should start preview only on
onSurfaceChanged callback and stop preivew onSurfaceDestroyed callback like
in the examples.
—
Reply to this email directly, view it on GitHub
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/issues/1068#issuecomment-1072331025,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG3Q7ANDDL6V65U6XL4HFX3VARTYTANCNFSM5P5OI77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


atomicWriteFileSync is misnamed

If there is an exception or other error the function does not return the .bkp file to the original name, leaving the systme in a broken state.

Fixed in 8056d7842fb3cf881ff2b40ec630bcefaa1cb695


Error: Unable to open file '~/Documents/CDDATiled/tiled_cdda_extension.json': No such file or directory

I got this error when using 'Import DDA Tileset':
Error: Unable to open file '~/Documents/CDDATiled/tiled_cdda_extension.json': No such file or directory
Stack traceback:
  initialize@file:///home/user/.config/tiled/extensions/CDDA_Map_extension.mjs:31
  @file:///home/user/.config/tiled/extensions/CDDA_Map_extension.mjs:924

Steps to reproduce:

Downloaded Tiles
Put the .mjs in ~/.config/tiled/extensions
Created a new project in ~/Documents/CDDATiled
use the  'Import DDA Tileset' option from the menu

My OS:

I reproduced this in ubuntu and found the issue (it probably happens in every OS).
It doesn't seem to resolve relative paths, so it doesn't like the "~". I'm not sure how to resolve relative paths in Tiled yet, so the workaround may have to be to use fully-resolved path names in the meantime while I figure it out.

The workaround worked! I used /home/user/Documents/GitHub/Cataclysm-DDA-1 for cdda path and /home/user/Documents/CDDATiled for the tiled project folder and it executed without error

😅 I'm so glad everything didn't immediately fail


WDAC Wizard crashes when creating a file path allow rule in a supplemental policy

WDAC Wizard crashes when creating a file path allow rule in a supplemental policy. Also enabling UMCI at the same time. Does not crash when creating a file path allow rule in a base policy, only in supplemental policy. Trying to create file path allow rule for "C:\Users\UB01609\AppData\Local\HP\ALM-Client" directory.
How to recreate:
Settings App (version 2.5.01):

Create a file path allow rule in a supplemental policy:

Make sure UMCI is enabled (not sure if this is 100% required to recreate.

Try and create supplemental policy. Wizard should crash:

Adding log files from "C:\Users\UB01609\AppData\Local\Temp" (.log file is empty; Unable to upload .db-wal file because of file type restrictions.)
mat-debug-3352.log

Fixed by changing ProductSigners field under ID_SIGNINGSCENARIO_WINDOWS from:

to

I believe this was caused by me manually removing hash allows incorrectly. I tested this with the WDAC Wizard and found that it works as expected.


Cannot install with node-webkit > 0.8.6 because of node-xpc-connection dependency

Hi,
xpc-connection dependency required on OSX does not seem to build correctly for node-webkit > 0.8.6.
Please see https://github.com/sandeepmistry/node-xpc-connection/issues/5
Any help would be highly appreciated.
Thanks,
Michael

Work done in https://github.com/sandeepmistry/node-xpc-connection/pull/6 by @francishart to use nan
@mhle please try latest noble

Thanks Sandeep, I worked with Francis to fork noble and xpc-connection any chance we can get these changes into next version?
Thanks,
Michael

On 6 Jan 2015, at 01:06, Sandeep Mistry<EMAIL_ADDRESS>wrote:
Work done in sandeepmistry/node-xpc-connection#6 by @francishart to use nan
@mhle please try latest noble
—
Reply to this email directly or view it on GitHub.


Experiment with a SigStore Rekor instance

Install Rekor server and cli: https://docs.sigstore.dev/logging/installation/
Sign and Upload to Rekor-server: https://docs.sigstore.dev/logging/sign-upload/#pkixx509
Verifying Rekor entry: https://docs.sigstore.dev/logging/verify-release/#verify-the-entry-is-in-the-transparency-log

Sign as x509 and Upload:
# Create keys
openssl ecparam -genkey -name prime256v1 > ec_private.pem
openssl ec -in ec_private.pem -pubout > ec_public.pem

# Sign
openssl dgst -sha256 -sign ec_private.pem -out TestArtifact.txt.sig TestArtifact.txt

# Upload to Rekor
rekor-cli upload --rekor_server http://<IP_ADDRESS>:3000 --artifact TestArtifact.txt --signature TestArtifact.txt.sig --pki-format=x509 --public-key=ec_public.pem

Some notes:

The artifact being "uploaded" to Rekor, is NOT uploaded to Rekor(!) Only it's hash, publicKey, and signature metadata are uploaded. Thus you cannot use it as a "store" of the metadata, so for example the CDXA bom itself would still be stored in the binaries repo


ci: adapt to protocol-9p.0.8.0

Signed-off-by: Thomas Gazagnaire<EMAIL_ADDRESS>

I think Docker's caching is too clever for you...


The synthesized video has no sound, what could be the reason?

It's alright because we haven't merge the audio into the generated video result, so the video is silent. You can do it via ffmpeg or other video editors. We may update this part later.

You can use the ffmpeg command line to process： ffmpeg -i video.mp4 -i audio.wav -c:v copy -c:a copy output.mp4

LOL,I have been trying to find the reason why there is no sound in the video generated by the final inference. Then,I searched issues using the word "voice" before, no report related to it were found. So,I thought maybe I had made a mistake in the configuration or other steps, but when I replaced with "sound", oh my god,that's the reason.

最后用这个
ffmpeg -i ngp_ep0028.mp4 -i aud.wav -c:v copy -c:a aac output.mp4


Remove deprecated termination codes (Mosek v9)

MSK_RES_TRM_MIO_NEAR_REL_GAP and MSK_RES_TRM_MIO_NEAR_ABS_GAP are deprecated in Mosek v9, and are not defined in Mosek#master.
This results in an error when calling JuMP.value(), since the termination status is checked.

We should probably wait for Mosek.jl to be released for Mosek v9.

I ran into this error the other day too... Since there are already some things that do not work with the tagged version of Mosek.jl e.g. https://github.com/JuliaOpt/MosekTools.jl/issues/4, would it make sense to merge this anyway?

Let's say MosekTools#master is for Mosek#master, from now on, we can create a separate branch for Mosek#v0.9

I added a clarification in the README and a mosekv8 branch

Sounds great!


Unable to Render Graph View when Working with Multiple Vaults

Describe the bug
If I try creating a site that hosts multiple vaults, then the Graph views do not work.
To Reproduce
Steps to reproduce the behavior:

Do not define a NOTES_PATH variable.
Have the Demo directory inside the perlite directory
Have multiple other vaults inside the perlite directory

Expected behavior
Graph views for each individual vault when navigating into it.
Desktop (please complete the following information):

OS: Ubuntu 22/04 LTS
Browser Chrome
Version 103

Additional context
I even tried moving the metadata.json file to the root perlite directory when setting up the Demo site. In this case, the Graph only shows the README file. The only time the Graph seems to show is if the Perlite instance is set up to serve only one vault.

Hi @athshean,
Yes, at the moment this won't work because only one metadata.json is supported.
The problem is, that:

The metadata.json needs to be located in the root of the NOTES_PATH or perlite folder (if none is defined).
Because of the relative paths in the metadata.json, the file needs to be in the root folder of the vault.

Example 1 (no vault defined):
-perlite
--metadata.json
--test.md
--test2.md
Example 2 (vault1 defined)
-perlite
--vault1
---metadata.json
---test.md
---test2.md
What should work is: copy the contents of all your vaults into the perlite folder and also merge all the metadata.json files into one that you provide in the perlite folder. Then Perlite will act like a "vault" and the graph should work.
If you want to keep your vault root folders for the structure and menu as well, you will need to adjust/expand the contents of each metadata.json vault to include the vault name before each relativePath key, you will also need to add another ../ before each displayText key. A simple Python script should automate this task.
To get the demo graph working without defining a NOTES_PATH, you can copy the contents of the demo folder to the perlite folder, provide the metadata.json in the Perlite folder as well, and the graph should work.
So this is not a bug, it's just not implemented yet... I will think about it, maybe i can implement it in the future.
BR,
Sec77


Define HAVE_TIMERFD on Linux system, for efficient MicroTimer.

Compile the C++ driver with -DHAVE_TIMERFD.
Motivation:
Performance profiling of a service using swift-cassandra-client showed that large amount of time was spent in managing timer list (maintained as a heap) deep in the Cassandra DataStax C++ driver. Some investigation later showed that the key blob of code responsible here:
void MicroTimer::on_timeout(Timer* timer) {
  uint64_t now = uv_hrtime();
  if (now >= timeout_ns_) {
    // The goal timeout was reached, trigger the callback.
    callback_(this);
  } else {
    // There's still a sub-millisecond part to wait for so spin the loop until
    // the timeout is reached.
    timer_.start(timer_.loop(), 0, bind_callback(&MicroTimer::on_timeout, this));
  }
}

This effectively degrades to a spin-loop, in case we fall through to the else part of the if. And we see that in profiling traces. I saw a case where that was responsible for ~50% of CPU (might be particularly bad case).
Fortunately, there is an alternative timing implementation. One that is guarded by define HAVE_TIMERFD.
Modifications:

define HAVE_TIMERFD when built on linux

Result:

CPU usage reduces significantly once HAVE_TIMERFD is applied

very nice 👍

@swift-server-bot test this please


How to Turn off update rate?

I think this is not an issue, I am unable to configure the following requirement.
As per document update rate by default is 15min. But I want to turn off the update rate and update index will execute only I placed new document or delete an old document.
On my web application when user uploads a document then only the index call should execute.

So you want to run manually the river? Is that what you are looking for?

Yes

The only workaround I see is:
Suspend or restart a file river
If you need to stop a river, you can call the `_stop' endpoint:
GET _river/mydocs/_stop
To restart the river from the previous point, just call _start end point:
GET _river/mydocs/_start

Thanks David, I will try this out.


I slightly changed the text color from an off-black to a black

What the title says

Great work!
But let's talk about how you would typically want to issue pull requests: they should have a clear, singular focus. This one would have met that requirement before you included the switch to the Numbering via CSS.
I'm going to close this pull request, but would like you to issue two new ones:

that covers the 'smaller' formatting changes
that just does the numbering via CSS

For each of them, give the pull requests clear names and you can add comments (like this) on GitHub to clarify what you are doing in the pull request.


How to lower the total needed shared memory?

Hi Chou,
I am encountering the following problem when trying to run convolutional neural network.
terminate called after throwing an instance of 'std::runtime_error'
  what():  [Error] In function "getSuitableShmConfig" (at src/cnn-utility.cu:418): Exceeds maximum shared memory available. (49152 bytes)
kernel = (100, 92), grids = (8, 8, 40), threads = (4, 4, 1)  => 75940 bytes of shared memory needed.

What is the standard way to fix this problem if I really don't have that much memory? (ex. lowering thread count)
Thanks in advance.

Actually the kernel size 23*14 is able to be run on my computer using theano. I am still wondering about what Caffe have done to get the 7% error rate. I can only achieve around 15% error rate with Lenet implemented by theano.
BTW, I found out the the nn-init would crash without any error message when the format of input training data is wrong. (I accidentally start my array from zero)

Actually the kernel size 23*14 is able to be run on my computer using theano. I am still wondering about what Caffe have done to get the 7% error rate. I can only achieve around 15% error rate with Lenet implemented by theano.
BTW, I found out the the nn-init would crash without any error message when the format of input training data is wrong. (I accidentally start my array from zero)

Can you please tell me the kernel size you choose in your LeNet ?
Okay, I'll add a guard clause at line 145 in src/data-io.cpp:
size_t j = stof(token.substr(0, pos)) - 1;

Thanks !!

Can you please tell me the kernel size you choose in your LeNet ?
Okay, I'll add a guard clause at line 145 in src/data-io.cpp:
size_t j = stof(token.substr(0, pos)) - 1;

Thanks !!

The filter size is 23x14 and 14x12
number of kernel is 40 and 100
I ran it on GTX780, the implementation is pretty much like the code in the following link. http://deeplearning.net/tutorial/lenet.html
I think the kernel size in libdnn is the same as LeNetConvPoolLayer.filter_shape in theano, right?

The filter size is 23x14 and 14x12
number of kernel is 40 and 100
I ran it on GTX780, the implementation is pretty much like the code in the following link. http://deeplearning.net/tutorial/lenet.html
I think the kernel size in libdnn is the same as LeNetConvPoolLayer.filter_shape in theano, right?

Yes, kernel is filter.
But the error message you mentioned before (kernel = (100, 92), grids = (8, 8, 40), threads = (4, 4, 1))  is the the way you think. XD
You see at line 772 in file src/feature-transform.cpp
Z[i] += convn_2(rot180(iImgs[i][k]), Y[k], this->get_output_img_size());

and line 442 in file src/cnn-utility.cu
mat convn_2(const mat& data, const mat& kernels, SIZE k) {
...
size_t SHM_SIZE = getSuitableShmConfig(grids, threads, k.m, k.n);
...
return output;

You can see that the third argument in convn_2 is this->get_output_img_size(), which is later passed to getSuitableShmConfig as variable k. And the size of output image in the first ConvolutionalLayer is [100, 92] ( = [122, 105] - [23, 14] + [1,1] )
Well, that doesn't means I won't fix this issue.
I'm currently refactoring src/cnn-utility.cu and try to make it more friendly, readable, and faster. (it's 50% already 50% faster now). I'll change the runtime_error message to a warning message in the next release.
But before that, I still suggest you use a smaller image by scaling down to 64 x 64, and try a smaller kernel.
I already heard that it's possible to achieve 95% in-sample accuracy and 92% out-of-sample accuracy using 3 layers of ConvolutionalLayer and SubsamplingLayer with 20% dropout. : )


Updated defaultDistinctId for iOS 10 changes to limiting ad tracking.

Starting with iOS 10 a user having "Limit Ad Tracking" enabled results in the request for the users IFA to respond with "00000000-0000-0000-0000-000000000000".  Since the current IFA check in the defaultDistinctId checking for nil multiple users could be provided with the same DistinctId and therefore combining their data into one mixpanel user.
In order to resolve this I simply added a check to the check if there is a need to fall back to the users IFV instead of their IFA for the 0'd out IFA.
See: https://developer.apple.com/reference/adsupport/asidentifiermanager

👍

We already check this here:
https://github.com/mixpanel/mixpanel-iphone/blob/master/Mixpanel/Mixpanel.m#L770
If it is limited, it won't pass the conditional statement

Did you ever receive an all 0s identifier, and if so, what SDK version are you using? Thanks!

We were receiving all 0's but it would appear that it turned out we were using an older version of the SDK in our app than I thought my apologies!  Closing the pull request thank you for the rapid response!


add davidfw1960/bom-weather-card

The list is case sensitive, please use the correct casing.
As described here
I see that this is a fork, is the original developer aware that you wish to publish this?
https://github.com/davidfw1960/bom-weather-card/graphs/contributors

I don't understand what you are asking me to change? The list is case-sensitive? The link for the decscription points to a page that says the list is case-sensitive? There is no list I can see. The original author has not responded to any PR's or interacted with anyone since April. I made some changes to this card and people asked me to add it to HACS.


Fix env variable use in integration tests library

See if we can do a quick fix to allow env variables use in our integration tests. I know it's not ideal right now since we haven't yet mirrored the project, set up the pipeline, etc. But this fix is necessary to run our tests so let's do it manually this time.

I'll make a fix based off @LaurenceDupont's branch and release on pypi on a temporary token.

This was fixed and pushed to rasa-integration-testing 0.1.3


Supporting selecting multiple cells and move them all at once

I am using DeepNote. Horrible to use. I can only move one cell each time. This is killing me.
We shall allow selecting multiple cells and move them together.
In addition, can we support grouping cells/pods? Then we move them as a whole. This is like grouping in PPT.

In fact, Reactflow have already multi-selection (see https://github.com/wbkd/react-flow/issues/609), in our current version CodePod, a user can try it in two ways:

Press Shift,  use the mouse to draw a rectangle area covering those pods you want to select, like what we often do to choose multiple files on a Windows desktop, then you can drag them together.

Select one pod, press the meta key on your keyboard (for my Windows OS, that is the Win key, I don't know the corresponding key on MacOS, @lihebi can you do me a favor to confirm that?) and click on another pod at the same time, then you can drag the both two pods together.

I can change the shortcut if you feel uncomfortable with those keys (yes, I also hate win key, which toggles the windows menu), or we can discuss how to group pods in a more flexible way, for example, assigning them with customized tags and then dragging on the tag?

Option 1 works, but you have to drag over the entire node to select it. Not the best experience.
Option 2 sounds better to me. It is the⌘ Command key on mac. But this only works on React Flow nodes (on their website), not our Code/Scope nodes. I guess we need to set some "selected" property.

@forrestbao If you try option 1, you won't like it. You have to drag over the entire node to select it.
I propose to disable option 1, and re-use the shift key for option 2. Because

The ctrl key does not work on mac, and the meta key does not work on windows. The shift key seems to be the best choice.
But the shift key can only be used in either option 1 or option 2, not both (https://github.com/wbkd/react-flow/discussions/2208#discussioncomment-2908766).

@lihebi I can check the OS first and then bind the hotkey if you need. Shift method may be hard to implement when considering the single-parent rule because almost all nodes are selected at the same time, I can't determine which node to drop if the rule is violated.


Endpoint for applications list in UHS (not YK) returns 'null' when UHS db shows many apps

When UHS collects applications from YK, the UHS database does have those entries, however the /api/v1/partition/:partition/queue/:queue/applications just returns null.
To reproduce, go to a copy of the UHS repo:
$ make kind-all
$ make run

Then submit the attached jobs file:
$ kubectl apply -f denis-jobs.yml

(This is just a regular decent-sized list of jobs to submit to Yunikorn - you can use most any other job submission file).
Verify that the Yunikorn web UI shows the applications are queued and/or running, by visiting http://localhost:30001/#/dashboard in your browser.z
Optionally, you can also check the UHS database, by running a postgresql psql client in the cluster, e.g.
$ kubectl run postgresql-client --rm --tty -i --restart='Never' --namespace yunikorn --image docker.io/bitnami/postgresql:17.2.0-debian-12-r1 --env="PGPASSWORD=psw" --command -- psql --host postgresql -U postgres -d uhs -p 5432
uhs=# select count(*) from applications;
81
\q
$

The actual bug is in the applications endpoint on UHS - query it by doing:
$curl  http://localhost:8989/api/v1/partition/default/queue/root/applications
null
$

This should instead return a large JSON list of the  applications that UHS detected from YK.

Thank you for providing detailed information on the ticket!
After further investigation, I identified the root cause of the issue:

The URL used to fetch applications is slightly incorrect. The correct URL should include partition_id and queue_id instead of partition_name and queue_name.
Example:
curl http://localhost:8989/api/v1/partition/01JE4M3K3KK3ZB2JPVH6A28B0F/queue/01JE4M3K3K5FF5WY2J4YWKTMXA/applications

The queue_id is not included in the YuniKorn-Core response, so it is always stored as null in the queue_id field of the application table. Consequently, even if the correct ID parameters are provided, the query will still return null.
Reference: YuniKorn-Core Code (Lines 335–339)

Solution: Added queueID in yunikorn core. After this PR is merged we should get desired result in the application endpoint.
PR: https://github.com/G-Research/yunikorn-core/pull/11


[BUG] MySQL Exception: Specified key was too long

Details

Translate Version: [ 3.1.0 ]
Install type: [ Extension Repository ]
Bolt Version: [ 3.0.11 ]
PHP version: [ 5.5.38 ]
MySQL version: [ 5.6.20 MySQL Community Server (GPL) ]
Used webserver: [ Apache ]

Reproduction
It seems that when I try to update my DB after the extension installation, MySQL throws and exception related to the INDEX CREATION.
This is the exception - SQLSTATE[42000]: Syntax error or access violation: 1071 Specified key was too long; max key length is 767 bytes
Here's an screenshot

@EzeRangel Could you try changing this: https://github.com/AnimalDesign/bolt-translate/blob/master/src/Storage/ContentTypeTable.php#L35 to 255 and see if that helps?
Seems like the MySQL length for a utf8 char is 3 bytes, so 767 / 3 =<PHONE_NUMBER>67, so 255 should be okay, and then if it works I can patch that into the next release :)

That's it
Everything is right :)


Add EOL policy

Solves #91 by adding some information regarding how to deal with EOL in VCS.

Pinging here @ninad-kamat, who opened the issue. Also, pinging @greschd since this same problem arose some time ago in the ansys-templates repo, see https://github.com/ansys/ansys-templates/issues/51

Pinging here @ninad-kamat, who opened the issue. Also, pinging @greschd since this same problem arose some time ago in the ansys-templates repo, see ansys/ansys-templates#51

@jorgepiloto The proposed changes look good. Thanks!


Fix spelling of "enrollment" in code

Currently the concept of enrolment is misspelled with only one "l". Please find and replace these instances so that the correct spelling is used: enrollment.

@amurphy-cl This issue is resolved and PR is merged. If verified can you please close this issue?


make url in abstract clickable

@mjstealey there is essentially only one line change to "urlize" the abstract text so that urls included in the abstract are clickable. This is at request of @rayi113 who wants this feature quick (the earlier, the better), and I have tested it out locally and it worked well with this one line change. Can you test it out and if it works, get it merged and deployed? @rayi113 if you want this one line change to be a hotfix on www, @mjstealey can apply it on www easily. Just let us know.

+1, also manually added to www

@mjstealey Great! I have tested it out on www and url does appear to be clickable now in abstract. @rayi113 Can you test it out on www and see whether it meets your need now?

Yes, this works very nicely.  It appears that all URLs in HydroShare Resource Abstracts including those there prior to this update are now clickable.  Thanks!


Fix embarassing bug, check the counter each iteration

not just after searching
This was noted by Andrew Dalke.  It can be a large optimization or a minor one depending on the number of false positives in the bit vector.

Reference Issue

What does this implement/fix? Explain your changes.
Any other comments?

This looks fine to me. There's really no way to add a test for it, is there?

Not unless we add some monitoring.  However, Andrew has a test case with chembl and searching for "O=O" :)


refactor: remove legacy code

Main points:

[x] Drop Internet Explorer support into a pit of flames! 🔥 IE 🔥 🥳
[x] Remove hacks when inheriting from native error classes. This makes no sense if ES2015+ is used.
[ ] Remove support for ES5.
[ ] Switch compilation to ES2022.

If you know any other places that are legacy and need to be deleted/refactor, welcome to the comments.
BREAKING CHANGE: EmptyError cannot be used if target is below ES2015
BREAKING CHANGE: NotFoundError cannot be used if target is below ES2015
BREAKING CHANGE: SequenceError cannot be used if target is below ES2015
BREAKING CHANGE: TimeoutError cannot be used if target is below ES2015
BREAKING CHANGE: ObjectUnsubscribedError cannot be used if target is below ES2015
BREAKING CHANGE: ArgumentOutOfRangeError cannot be used if target is below ES2015
BREAKING CHANGE: AjaxError cannot be used if target is below ES2015
BREAKING CHANGE: AjaxTimeoutError cannot be used if target is below ES2015
BREAKING CHANGE: UnsubscriptionError cannot be used if target is below ES2015
BREAKING CHANGE: ajax no longer supports IE
Related issue (if exists): #6367

Hi, can you tell my why you closed this PR? I stumbled across the createErrorClass function recently and it has more problems besides being 'hacky':

It does not create 'native' errors and thus can not be identified with Node.js's util.types.isNativeError() function.
It does instantiate the stack trace eagerly. This means that error creation is about 10 times slower compared to a normal error (in my micro benchmark).
All stack traces contain createErrorClass, which is not pretty


[fix] fix schedule lib calendar.js.erb eventRender

Coverage decreased (-2.5%) to 82.38% when pulling fa4d7b3067309837fa2503b2245aef73abd92f0a on fix-gws-schedule-calendar into 7df59d20d661b6ad25b5f12addb8fca9c0176d91 on master.


Improve handling of new files created by CodeActions

When using a CodeAction such as "Move type to SomeNewClass.cs", there are various problems that should be smoothed out:

The new file is created but is empty. The generated code is written to the file buffer but not written (:e! will show that the file is actually empty). Should we automatically write the buffer to disk?
Users with set nohidden receive errrors, because we switch to the new file buffer, write the content, then switch back. This should work for users with set nohidden - either by temporarily setting set hidden or by i.e. opening the buffer in a new window/tab.
The new file is not automatically picked up by OmniSharp-roslyn. I am not sure if it should be, but it ought to be simple enough to trigger an /updatebuffer.
When the Code Action moves code from the bottom of the current buffer, it causes an ERROR: cursor position outside buffer. This happens while restoring the window position when the cursor position no longer exists in the buffer. We should handle this correctly, and move the cursor to the last line of the file.

For 1.
Just tried that with VS and it does exactly that. It writes both buffers/files after applying the change.

For 2.
We could have that as an option to set, which behaviour you prefer. As both have their ups and downs i think. and i think both are quite simple to implement. even though i couldnt do it :-)

@Confuset I actually think that for 2. it will be very simple to save the current value of &hidden, then set hidden, implement the changes and then restore the user's &hidden preference.
Adding options for doing this in splits and tabs makes it more confusing. If we can word the documentation clearly enough though, then I suppose we could do this - opening the new buffer in a split and leaving it visible might be a useful option.

i think i am fine with your first proposal regarding hidden. it has some sort of fire and Forget while cleaning up your code after writing it. and it is way easier to implement ;-) now that i think about it

@Confuset I think I covered all 4 points pretty well, please have a try and tell me what you think.

@nickspoons nice. will look into this hopefully this evening...currently Fighting c++/win32/mfc…

looks good and does the trick. thank you :-)
while testing it i recognized that the code action: "rename file to typename" is not working. but this is unrelated to this issue.
created a new one: https://github.com/OmniSharp/omnisharp-vim/issues/454


BOOTSTRAP_URL is changed and last support will be until end of Feb 2021

We noticed that during bootstrap=True in lookup_rdap method, the queries returned slower than in the case without bootstrap. Perhaps it's the fact that the bootstrap_url has changed according to Arin webpage.
Quote from web:
The URL for Registration Data Access Protocol (RDAP) bootstrap queries has changed to https://rdap-bootstrap.arin.net/bootstrap. If you have queries that use our RDAP bootstrap server, please update your links. Note that the current URL, https://rdap.arin.net/bootstrap, will be supported for 90 days, and that support will end on 18 February, 2021. Please use this 90-day period to migrate to the new URL. After 90 days, the old URL (https://rdap.arin.net/bootstrap) will no longer be supported.

Updated in dev branch. Thanks.


how to use or improve js_ua_run like the following?

// javascript code will be interpreted by mujs

var  ua = new user_agent("https://api.example");
us.add_header("Content-Type", "application/json");
us.add_header("Authorization", "Bear xxxx");

var resp = ua.run("GET", "/user");
if (resp.status == 200) {
   // resp.body
}

resp = ua.run("POST", "/user", {...});

if (resp.status == 400) {
   // report error
}

resp = ua.run("DELETE", "/user/abc");
...

Any update on this issue?

I haven't got around to it yet, but I will need to wrap some user_agent.c functions anyway for the slack bot

This wrapper has to be done ASAP.


[Play] - Leaderboard teams regenerate on each resize of screen.

Issue:
Solves the issue of the team avatar values being regenerated and randomized every time the screen is resized. Github: https://github.com/rightoneducation/righton-app/issues/652
Resolution:
Originally the team avatars were being set at a random value which would regenerate each time the component was re-rendering, including when the screen would be resized. I took it outside of the component function and used a useMemo hook so that it is randomized only once and then stored and returned. It is only regenerated when the teams or teamId also changes.

Leaderboard.tsx - this is where the team avatars values are being generated

Preview:
https://github.com/rightoneducation/righton-app/assets/79948102/8665d395-7224-41bb-9621-5b675fa51cc1
Video of the problem^^
https://github.com/rightoneducation/righton-app/assets/79948102/4978e09c-9690-4c4b-a24f-679925ed6c0a
Video of the change^^
@drewjhart

Closed via: https://github.com/rightoneducation/righton-app/pull/683


Unable to use template-tester

I trying to validate a binding template and when I tried to execute the following command,
template-tester  <NameSpace> bind.yaml
I get the following error,
secrets is forbidden: User "system:anonymous" cannot list resource "secrets" in API group "" in the namespace
It seems like  ~/.kube/config file is used to grab credentials for K8s client in template-tester. Although I can use kubectl with the same ~/.kube/config to get secrets in the particular namespace.
Your help is much appreciated.

Interesting. Yes, it uses that same context. It looks like an RBAC issue, but, if the same config works with kubectl I don't have a theory offhand. What kind of k8s cluster is this? Is the token you're using for a user bound to cluster-admin?

The K8s cluster is created using PKS CLI. Yes, the user has the cluster-admin role.

@cholick Do you any idea what might cause this issue or something that I could try and see.

We've been thinking about this and, no, don't have a theory. The only other thing I can think to validate is that you're using a token for a service account on the cluster, rather than some other configured authentication system (like an authenticating proxy).


Kurl proxy min tls version is 1.2

Also prefer server cipher suites

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


NoMethodError: undefined method `int' for RubyWrapperBeanAdapter:Class

I got this problem while trying to run a jrubyfx-generated Dummy.fxml...
Code here: https://gist.github.com/jomendoz/5a1ef9dd0fb11519afe9
And error here: https://gist.github.com/jomendoz/c06268416be82148c8e9
Any clue?

Specs:
JRuby 1.7.18
JRubyFX: github 892af575c876cf7c2152ee365736c2a724dc4445 > 1.1.1
OpenJDK: jdk8-openjdk 8.u25-2
SceneBuilder: 2.0.1

I think I caught the nail in the rye. It seems everything is ok but TableView's property, "Column Resize Policy". Its default goes for "unconstrained-resize" but if I use "constrained-resize", theoretically, all columns resize evenly to match parent's width. But when I use this .fxml in JRubyFX, the error that I previously shared appears again.
Diffing the two .fxml shows  the following:
7a8

66a68,70

     <columnResizePolicy>
        <TableView fx:constant="CONSTRAINED_RESIZE_POLICY" />
     </columnResizePolicy>

So my guess is that JRubyFX::Controller::fxml doesn't deal well with that constant. But no time for further research.

fwiw I've had luck setting this in ruby land (I have no idea if the long constant can be shortened):
@some_table.column_resize_policy = javafx::scene::control::TableView::CONSTRAINED_RESIZE_POLICY

Fix dependent upon https://github.com/jruby/jruby/issues/3366

JRubyFX 2.0 has been released. Your issue may have been fixed in 2.0. If your issue still applies to JRubyFX 2.0, please reopen this issue.


fix: updated Kibana app endpoint to private

Description
Update Kibana app endpoint to private.
Release required?

[ ] No release
[x] Patch release (x.x.X)
[ ] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Kibana app will have private endpoint instead of public for receiving requests.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

What impact does this have on upgrade? Is it going to tear down anything? Re-deploy kibana? Lets ensure we are deploying kibana as part of DA upgrade test for a start

It is only going to update the endpont for Kibana app from local_public to local_private. This means any existing users will no longer be able to access Kibana app from the public Internet and is only accessible via the IBM Cloud private network.

Also, due to the schematics limitations, we cannot run the upgrade test.

What impact does this have on upgrade? Is it going to tear down anything? Re-deploy kibana? Lets ensure we are deploying kibana as part of DA upgrade test for a start

It is only going to update the endpoint for the Kibana app from local_public to local_private. This means any existing users will no longer be able to access the Kibana app from the public internet and is only accessible via the IBM Cloud private network.

@ocofaigh , does this come under breaking change?

@Ak-sky No It depends on what actions occur. Will the kibana app be redeployed to to the config error (meaning dashbaord will go down). It seems the upgrade test is not deploying kibana. I think you need to change it so it does so we test that use case

@Ak-sky If it was using public before, then how did the dashboard ever work since our DA only deploys private only ES instance?

So @Ak-sky how is somebody supposed to hit the kibana url private endpoint in their browser? We will wan't to document that? Is it even possible to set up a VPN that can access it?

So @Ak-sky how is somebody supposed to hit the kibana url private endpoint in their browser? We will wan't to document that? Is it even possible to set up a VPN that can access it?

@ocofaigh, we need to discuss about this, I will add this topic for deep dive.

/run pipeline

Wait for https://github.com/terraform-ibm-modules/terraform-ibm-icd-elasticsearch/pull/363 to be merged fist

/run pipeline


Installer with choice of which utility to install

Description of the new feature / enhancement
I would like that when I install and/or update Powertoys I can choose which utility to install, because there are utilities that I don't use
Scenario when this would be used?
I consider this important for hard disk space and also for utilities that may not be considered useful for certain users
Supporting information
No response

/dup #3170


Bump microscope version used in CI

This change is

test-me-please

@nebril probably needs backport?

test-me-please


Project#index Last Seen metric

Add the Last Seen metric to the Project tables on the Index action.

Fixed by commit 6d616bed0d4709a9eab5ed9f0f080eb5d40f2953


fix sgx cert validator

Signed-off-by: Xiao, Ziyang<EMAIL_ADDRESS>
Commit Message: fix sgx build failed
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] https://github.com/intel/envoy/issues/35
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

Thanks!


Misleading error on docker pull

Pulling an image larger than the available storage space results in a misleading error message claiming the image was not found. The process typically halts during the download phase once 100% storage space is used, followed by the image not found error message. This has bitten us at least twice at the company I work for. I have seen this with different docker versions, graph drivers, filesystems and kernels.
One of our machine's setup:
Ubuntu 14.04 running Linux 3.19.0-30-generic x86_64, Docker 1.8.3 using aufs on xfs
Steps to Reproduce:

Find/create a large image (e.g. baseimages)
Make sure Docker's data directory resides in a filesystem with insufficient space for the image of your choosing
docker pull largeimage
An image not found error should appear before the download finishes
df -h should report 100% usage on the mount point where Docker's data directory resides

Actual Results:

Image not found error

Expected Results:

Insufficient space error
Even better: "Insufficient storage space, cannot pull image" prior to starting the download (does not fill the filesystem 100%)

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

Looks similar to https://github.com/docker/docker/issues/6484 (and the related PR https://github.com/docker/docker/pull/6503)
@LK4D4 you looked into the other issue, any idea if we can make improvements here? I think the problem is that upfront, we don't know how large the image is before pulling IIRC

I was looking into the code (~docker/graph/pull_v2.go->download() and pullV2Tag()), but was unable to find any checks being performed for determining the disk space during a pull request. I would like to work on this issue (require some guidance), if no one is working. #dibs :)
Thanks to @cpuguy83 for giving some heads up on the issue, some highlights or approaches to resolve this:

the checks for every pull request maybe be a overkill on the server, but its is expected to return a more appropriate error code as returned from the syscall. This may not be limited to aufs/xfs or specific to a platform, require more investigation where the errors are propagating from.
refer to #16450 (could be linked) - checking before pulling the actual layers, to be able to essentially do a dry-run pull which would let the client inspect the size/etc before doing the real thing.

@cpuguy83 Here are the steps to reproduce the issue on ubuntu 14.04 using aufs / default drivers.
$ docker version
Client:
 Version:      1.8.2
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   0a8c2e3
 Built:        Thu Sep 10 19:19:00 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.8.2
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   0a8c2e3
 Built:        Thu Sep 10 19:19:00 UTC 2015
 OS/Arch:      linux/amd64

On the remote host the root partiton has about ~100MiB space file systems reserved blocks.
$ df -lh
Filesystem                   Size  Used Avail Use% Mounted on
/dev/mapper/ubuntu--vg-root   75G   75G   31M 100% /

Docker daemon run with -D gives the output (***), when a pull request of a large enough image, we get error "no space left on device" (pkg/chrootarchive/archive_unix.go:invokeUnpack) but the docker client does not show anything relevant. We are getting error message printed (in two parts "output: write /usr/lib/klibc/bin/mkdir: no space left on device"), one returned while uncompromising the image, which may be propagating to the caller functions (pullV2Repository()) which prints "Error trying v2 registry".
...
....

Please let me know your thoughts on narrowing the issue further, any suggestions are welcome. Thanks.

The main issue here is the way ErrNoSupport or discardNoSupportErrors is being handled in graph/pull.go:Pull(). The docker client first sends a v2 pull request which fails with "no disk space error", this error is stored in lastErr var, on failure the error message is not returned to the client, instead v1 pull is attempted on the docker daemon which results in an error. Therefore the var lastErr gets overwritten with a subsequent error returned on v1 (as the first attempt with v2 failed). Therefore overwriting and returning "error: no disk image found"  with lastErr causing the issue.
To solve this issue, would it be better to return v2 error (more valuable) instead of the v1 error or is there some way to tell if the request was for v1/v2. I am not sure if we need to return both error code/messages to the docker client (concatenated as a single string). Any better ways to handling this?

When the docker daemon is started with "--disable-legacy-registry=true", the correct error message is sent to the client. Not sure if could be recommend as possible work around till the issue gets fixed.
$ docker pull <IP_ADDRESS>:5000/askb/devbox
Using default tag: latest
latest: Pulling from askb/devbox
972087ac0357: Extracting [=====>                                             ] 20.61 MB/191.4 MB
6b248174e4db: Download complete
........
11ee04239602: Download complete
Untar re-exec error: exit status 1: output: write /usr/bin/python2.7: no space left on device

This is caused in the puller.Pull() function, which returns lists of endpoints (v2,v1,v0) pointing to the same registry. Therefore, when legacy registry is enabled, daemon loops into looking of requested image in the older registry version. This looping causes subsequent errors to overwrite the error returned from (graph/pull.go:puller.Pull() function) the first time ("no disk space left on device"), with the later error messages returned, therefore returning "image not found" to the docker client.
Sorry, I am not very familiar with discardErrNoSupportError and how its handled, any suggestions fixing this would be appreciated. included @dmcgowan @thaJeztah

ping @RichardScothern ^^

@askb : --disable-legacy-registry- effectively hides the problem because it prevents the daemon from falling back to a V1 registry and producing the NotFound error.  However, if the daemon were configured with two v2 registries, it may produce multiple disk space errors (one for each fallback).  Ideally there should be a class of errors which cannot be retried, of which out of disk space is one.

@RichardScothern, I agree, daemon should not proceed on "no disk space" error, while checking if the error belonged to a class and avoid retries should be implemented to address this issue. Can this be achieved with adding a case for "no disk space errors" to registry/registry.go:ContinueOnError(), which checks if a fall back is required on the type of error ?


SCR5-3.1 - Tables

Collections (Tables) got finalized with respect to covering the functionalities.


Cant get out of the test.

Challenge Manipulating Complex Objects has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:53.0) Gecko/20100101 Firefox/53.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
  // Add record here
  var myMusic = [
  {
    "artist": "Milind Sharma",
    "title": "My Parents",
    "release_year": 2017,
    "formats": [
    "CD",
    "BlueRay"]
  }

];

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!


Setting cross-compile flag in CI file

fixing #96

Can you reenable building http on cross targets?

Mmh, can you try rebasing again?  Let me know if I can help at all.


added Alarm controller

I added a new type support Alarm for control it, i test in my own home and works fine.
With this new code i can fix this issue:
Oct 15 20:16:49 raspi homebridge[751]: [2018-10-15 20:16:49] [Loxone] Platform - The widget 'Alarma Intrusión in Central' of type Alarm is an item not handled. that i have on my home.

very nice, thx!
but a kind of return channel does not exist isnt it? means, apple home is not able to check the status of the alarm controller.

Could you please also add outlets?

Hey @cuni0716 ,
Ever had time to complete this? Would be nice to have the alarm available in Apple Home!
Greets,
Roy

@royteeuwen I haven’t time now, and I already purchased a MyHome license that works better than this.

Hey Ramon,
Could you explain what MyHome does? If I google it, it states that it works with HomeKit devices, but Loxone does not support HomeKit?
Thanks,
Roy

Van: Ramon Cuni<EMAIL_ADDRESS>Verzonden: dinsdag, december 31, 2019 12:53 PM
Aan: Sroose/homebridge-loxone-ws
CC: Roy Teeuwen; Mention
Onderwerp: Re: [Sroose/homebridge-loxone-ws] added Alarm controller (#38)
@royteeuwenhttps://github.com/royteeuwen I haven’t time now, and I already purchased a MyHome license that works better than this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Sroose/homebridge-loxone-ws/pull/38?email_source=notifications&email_token=ABF42TQ6K74AOFY5YPZW523Q3MXDNA5CNFSM4F3UH5N2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEH4D44Y#issuecomment-569917043, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABF42TSAYMACHJOSDT325GTQ3MXDNANCNFSM4F3UH5NQ.

very nice, thx!
but a kind of return channel does not exist isnt it? means, apple home is not able to check the status of the alarm controller.

I just tested the PR, and if I turn off the alarm in Loxone, it immediatly shows up in  Apple Home as well, so what do you mean by this exactly?

Could you please also add outlets?

What is an outlet?


TrueNAS, Unpackerr in Jail: rarReader.Next: rardecode: invalid file block

deleted logfile, restarted service, new logfile; https://atomicrhino.net/upload/ZeroBin/?68e7014646d2b7d2#Fx0l1aaKR/t3BefhsW//vMVsA5m6Y+IuYpsGIIsr4Gc=
conf file; https://atomicrhino.net/upload/ZeroBin/?a8a10bfeec15c2fc#YOuSa9pAAz921JKgw7bKLLTgmsINWymduNUADZjYfyQ=
problem seems to be: 2021/12/30 13:36:29.732674 handlers.go:165: Extraction Error: Family.Guy.S20E04.1080p.WEB.H264-CAKES: rarReader.Next: rardecode: invalid file block, but can't find anything on it.
Extracts the files fine to a new _unpackerred dir, then on completion deletes both and nothing in the log but that.

Discussing on Discord. This seems to be a corrupted file.

This happened to 19 out of 20 downloads, and appears to be something specific to the TrueNAS configuration. Trying to find more info.

Were you able to get this working? You kinda fell off the Discord.


[FLINK-32056][Python][Connector/Pulsar] Upgrade flink-connector-pulsar in flink-python to v4.0.0

What is the purpose of the change

Upgrade used flink-connector-pulsar in flink-python to 4.0.0, since that's compatible with Flink 1.17+. Without this upgrade, we can't upgrade flink-shaded since the older versions of flink-connector-pulsar directly use flink-shaded and there's a breaking change in flink-shaded.

Brief change log

Upgrade to flink-connector-pulsar to 4.0.0
Remove and refactoring all occurrences of PulsarSerializationSchema

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? not applicable

@MartijnVisser Thanks a lot for this work! Besides the sink, I guess we should also update PulsarSourceBuilder.set_deserialization_schema

I noticed that the documentation of 1.17 has already been updated to 4.0.0-1.17, see more details on https://nightlies.apache.org/flink/flink-docs-release-1.17/docs/connectors/datastream/pulsar/
I guess we need also port this to release-1.17?

I guess we need also port this to release-1.17?

Yes indeed, because the existing used connector in release-1.17 actually doesn't work with 1.17


feat: add SITE.title in Post Title

Yeah, what you see is true. In the previous versions, the SITE.title was NOT added to the Post Title.
I fixed this issue in this PR.

Thanks for your PR.


fix(pg): Do not add SQLCommenter comments to prepared statements

Which problem is this PR solving?
Currently the pg instrumentation will add SQLCommenter comments to any query it can, even prepared ones. In the case of prepared statements this usually results in the trace context within the comment being incorrect for all but the first instance of the query being run.
Short description of the changes
This adds a simple check to skip adding the comment for prepared statements. See https://node-postgres.com/features/queries#prepared-statements.

@pichlermarc is this consider a breaking change? Just want to make sure everything is properly flagged before giving any approvals

Wouldn't this be a bugfix rather than a breaking change given this behaviour is always incorrect ?


Precompiling issues

Precompiling issues after updating local repository. The issue is resolved when checking out older commits. It might be a local conflict.
Possible reason: Different Julia versions?
Error:
[ Info: Precompiling TMI [582500f6-28c8-4d8f-aabe-b197735ec1d4]
ERROR: LoadError: ArgumentError: Package LogExpFunctions does not have ChainRulesCore in its dependencies:

You may have a partially installed environment. Try Pkg.instantiate()
to ensure all packages in the environment are installed.
Or, if you have LogExpFunctions checked out for development and have
added ChainRulesCore as a dependency but haven't updated your primary
environment's manifest file, try Pkg.resolve().
Otherwise you may need to report an issue with LogExpFunctions
Stacktrace:
[1] macro expansion
@ ./loading.jl:1167 [inlined]
[2] macro expansion
@ ./lock.jl:223 [inlined]
[3] require(into::Module, mod::Symbol)
@ Base ./loading.jl:1144
[4] include(mod::Module, _path::String)
@ Base ./Base.jl:419
[5] include(x::String)
@ LogExpFunctions ~/.julia/packages/LogExpFunctions/Vfuhq/src/LogExpFunctions.jl:1
[6] top-level scope
@ ~/.julia/packages/LogExpFunctions/Vfuhq/src/LogExpFunctions.jl:17
[7] include
@ ./Base.jl:419 [inlined]
[8] include_package_for_output(pkg::Base.PkgId, input::String, depot_path::Vector{String}, dl_load_path::Vector{String}, load_path::Vector{String}, concrete_deps::Vector{Pair{Base.PkgId, UInt64}}, source::String)
@ Base ./loading.jl:1554
[9] top-level scope
@ stdin:1
in expression starting at /home/xda021/.julia/packages/LogExpFunctions/Vfuhq/ext/LogExpFunctionsChainRulesCoreExt.jl:1
in expression starting at /home/xda021/.julia/packages/LogExpFunctions/Vfuhq/src/LogExpFunctions.jl:1
in expression starting at stdin:1
ERROR: LoadError: Failed to precompile LogExpFunctions [2ab3a3ac-af41-5b50-aa03-7779005ae688] to /home/xda021/.julia/compiled/v1.8/LogExpFunctions/jl_mTjAX4.

Which version of Julia are you running? Julia 1.8.5 passes all tests on GitHub Actions https://github.com/ggebbie/TMI.jl/actions/runs/4847037311/jobs/8637366543 .
Here are 2 things to try.

TMI should not really ship with a Manifest.toml. It has to (I think) because it relies on some GitHub repos. You could try import Pkg; Pkg.resolve(). It probably won't help too much.

You could try Julia 1.9.0-rc3. It is not released yet but it is fast and seems to be working out for me. Here are some quick steps to use juliaup to upgrade.

curl -fsSL https://install.julialang.org | sh
juliaup add 1.9.0-rc3
juliaup default 1.9.0-rc3

import Pkg; Pkg.resolve() did indeed solve the issues. Thanks!


Feature/log caching

Description
Re-adding the log-store caching feature to reduce the usage on db-connections

we may need to make the buffer size configurable so we don't have to change the value in the source-code


incomplete contour lines for curvilinear data

Reported by Remik Ziemlinski on 20 Oct 2004 19:30 UTC
use rjs.nc
shade/lev=(-.4,.4,.05) dif, geolon, geolat
contour/ov/lev=(-.4,.4,.05) dif, geolon, geolat
! the zero contour is not complete, but the shade is okay
! this applies to
! FERRET v5.70 (gcc3.2)
! Linux(g77) 2.4.18 - 10/08/04
! 20-Oct-04 15:18
! and
! FERRET v5.53 (rev .2)
! IRIX 6.5 - 12/12/03
! 20-Oct-04 15:18
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1027

Comment by<EMAIL_ADDRESS>on 20 Oct 2004 19:46 UTC
couldn't attach sample netcdf data. please obtain from
ftp://ftp.gfdl.noaa.gov/pub/rsz/rjs.nc

Attachment from<EMAIL_ADDRESS>on 20 Oct 2004 19:41 UTC
sample image
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1027_GIT_472/rjs.png

Attachment from<EMAIL_ADDRESS>on 20 Oct 2004 21:59 UTC
SHADE fills zero as if it's > 0; FILL fills as if it's < 0.
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1027_GIT_472/fill_shade_compare.gif


Set up Github issue templates for bugs and features like the Foundry repo has

The Foundry repo (foundry-rs/foundry) has great drop-down bug and feature templates.
This makes it a lot easier for them to review, reproduce, and fix bugs.
Our currently issue template:

issue-template.md

Foundry issue templates:

BUG-FORM.yml
FEATURE-FORM.yml

Could be done along with:

#29


Font Variant outputs invalid property value (typography field)

Issue description:
The font variant is outputting invalid property value . See screenshot: prntscr . Italic values are not working ( prntscr ) but others are fine. The regular property also returning invalid property value. But if I replace normal with the regular property it works fine.
Version used:
(Did you try using the develop branch from github? There's a chance your issue has already been adressed there)
2.3.5
Using theme_mods or options?
theme_mods
Code to reproduce the issue (config + field(s))
    Kirki::add_field( 'tcsupport_config', array(
        'type'        => 'typography',
        'settings'    => 'tcsupport_title_area_text_font_option',
        'label'       => __( 'Select title font', 'tcsupport' ),
        'section'     => 'tcsupport_blog_archive_section',
        'default'     => array(
            'font-family'    => 'Open Sans',
            'variant'        => '700',
            'font-size'      => '55px',
            'subsets'        => array( 'latin-ext' ),
            'text-transform' => 'none',
            'color'          => '#ffffff',
            'line-height'    => '1.2',
        ),
        'priority'    => 10,
        'output'      => array(
            array(
                'element' => '#tc-subhead .tc-subhead-area .tc-subhead-inner h1',
            ),
        ),
        'active_callback'    => array(
            array(
                'setting'  => 'tcsupport_title_area_option',
                'operator' => '==',
                'value'    => '1',
            ),
        ),
    ) );

How can we separate these two properties (font-weight & font-style) from the variant value. As you can see in this screenshot both values are coming from the variant property together.
Used Code in frontend:
if( isset( $title_area_font['variant'] ) ){
	$title_area_font_variant 		= 'font-weight:' . esc_html( $title_area_font['variant'] );
 }

Thanks for you help. highly appreciated.

Hey there!
This shouldn't be happening in the frontend...
If you're using the output argument of the field everything should be fine unless you override the automatically-generated CSS with your own custom functions...
The part that takes care of variants in Kirki is here: https://github.com/aristath/kirki/blob/2.3.5/includes/output/field/class-kirki-output-field-typography.php#L44-L56

I didn't wrote any custom functions. Just registered the typography field ( like above code) & injected the output in wp_head action using the code below.
if( isset( $title_area_font['variant'] ) ){
    $title_area_font_variant  = 'font-weight:' . esc_html( $title_area_font['variant'] );
 }

As you can see Im printing the value in font-weight property. Can't find how to separate these two property from the variant. Note that: in the customizer preview the variant shows font-weight & font-style separately which is actually I am trying to achieve for the frontend. Do you have any idea how I can get this? Thank You.

Doing some cleaning-up in the issues queue and I saw I dropped the ball on this one..
Did you manage to figure this one out?

Version 3.0 works fine so I'm closing this one.


Fix crash on NaN offset in path_follower 2d and 3d

Setting the offset of PathFollower2D or PathFollower3D to NaN caused a crash.
I found this out when accidentally setting the offset to sqrt(some_negative_number)
(Today I learnt that casting a NaN float to a double actually results in a number apparently)

Makes sense.
There's probably way more places in the code base where passing a NAN would cause a crash though (I haven't seen too much such checks).

Yeah I thought the same thing, but figured it's still worth preventing an errorless crash

Thanks!


Join room error handling

Description
Provided a specific error message when joining a room fails due to an expired auth service passcode.
Breakdown

Altered dialog error message based on the type of exception that gets thrown.
Updated target SDK version to 29.
Updated kotlin plugin to 1.3.71.

Validation

Manually validated
Passed CI

Additional Notes
N/A
Submission Checklist

[x] The source has been evaluated for semantic versioning changes and are reflected in versionName under app/build.gradle
[x] The CHANGELOG.md reflects any feature, bug fixes, or known issues made in the source code

@aaalaniz Alright. Reverted back to target SDK version 28. Looks like it cleared up the lint errors that were appearing from upgrading to version 29.


Move opam-repo-ci to ocaml.org

Closes https://github.com/tarides/infrastructure/issues/388
With this change, I think we now have all of the indicated services grouped
correctly (i.e. all "Ecosystem services" under the ocaml.org deployer), with the
exception of https://github.com/ocurrent/current-bench. The latter has been
indicated as belonging in this group, but it is not currently handled by the
deployer.

While the move here is trivial, I suggest that you break this into two PRs, one for each service and that check is updated first to test the workflow.
Obviously, the PR needs to be pushed to both affected live branches.  However, more importantly, the GitHub Apps need to be reconfigured to account for these changes.  If you aren't familiar with this process, then we could do this together on the call on Thursday.

I factored out the two changes into different PRs. This one depends on #231

Merged and deployed! Thanks again for the tutorial :)


This is empty-commit to retrigger-build

Motivation
[Describe why this change is needed]
Modifications
[Describe the code changes]
Result
[Describe how the changes affects existing behavior and how to test it]
Related Issues
[Resolves #123]

/ok-to-test

/lgtm
/approve


Fix typo

php version should match with the formula associated on readme

name: 🐞 Bug Fix, ⚙ Improvement or 🎉 New Feature
about: You found a bug, want to improve something or add a new feature
labels: bug or enhancement

A Pull Request should be associated with an Issue.

If you're fixing a bug, adding a new feature or improving something please provide the details in Issues,
so that the development can be pointed in the intended direction.

Related issue:

Further notes in Contribution Guidelines
Thank you for your contribution.

Description
This PR [briefly explain what it does]

@davidpella Thanks for fixing this.


Note right 60% / left 40% window modes in README

I only noticed this when glancing at the code; here I've been creating this layout by hand like an animal.

@danseely: Thank you! 🍻


License Finder returns wrong license for oj_minic_json

Ran license finder on my project.  It returned:
oj_mimic_json, 1.0.1, "GPL-3.0, MIT"

If you go to the project page on github https://github.com/ohler55/oj_mimic_json, the license says  BSD-3-Clause license.
Getting even one wrong undermines trust in the program.  If it can't figure it with a great deal of certainty, the program should should mark the license as unknown.

@brunzefb This is an issue with your source code. You will want to update your gemspec to accurately report the license.
See https://github.com/ohler55/oj_mimic_json/blob/a8a53ffc77a1efe7f6dfcdac34040c4a01059ad0/oj_mimic_json.gemspec#L10

Thanks for responding @nertzy ! This does seem to be the case!


(fix) Field args that has defaultValue should not be mandatory.

Per GraphQL spec(418), non-null object fields can be omitted if they supply a default value:
https://github.com/graphql/graphql-spec/pull/418

Added tests


Revised version of the protocol - team Grasshoppers

Hi @open-sci/grasshoppers,
As anticipated in yesterday's lecture (practical part), you have to revise your protocol published on Protocols.io according to the reviews received by the members of the other team. Once your revision of the workflow is finalised, please publish it using the Protocols.io interface.
In addition, you must also write and publish a response letter to the reviewers, where you explain either how you have addressed their comments or the justification for not having addressed them (please remember that if the latter applies, then your justifications must be very robust). Once finalised, the response letter must be published in Zenodo, and it should explicitly link and mention (even in the metadata) the reviews received, the old version of the protocol and the revised one.
Then, close this issue by attaching in the comment the clickable DOI URLs (https://doi.org/...) of the revision of the protocol and of the response letter. The deadline to provide all this information is the 28th of April.
For further doubts, do not hesitate to contact me in the Signal group.

This is the revised protocol: https://dx.doi.org/10.17504/protocols.io.bufwntpe
This is the response letter to the protocol reviews: https://zenodo.org/record/4724007#.YIhwhX0zb5Y

There's a newer version of the protocol available:
Ricarda Boente, Deniz Tural, Cristian Santini, Arcangelo Massari 2021. Protocol: Investigating DOIs classes of errors V.4. protocols.io https://dx.doi.org/10.17504/protocols.io.bunnnvde


feat(minor): change repository

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


PLANNER-620 Workbench: CH: Support difficulty comparator definition

Preview, further changes may come.
https://issues.jboss.org/browse/PLANNER-620 defines a requirement to support creating comparator objects for existing data objects (ones annotated with @PlanningEntity). The comparator is created in OptaPlanner WB and added as a nested static class of the associated data object. This PR provides a way to define nested classes & methods for data object.

OptaPlanner WB PR: https://github.com/droolsjbpm/optaplanner-wb/pull/90

@Rikkola would you mind taking a look at this one as well. You're probably more familiar with this than me?

As I talked to @mcimbora in IRC. Went through this with a fine comb. guvnor and kie-wb-common are shared modules, so we should be extra careful and do some clean up every time we run into anything that smells bad.

@Rikkola I updated the PR. I added JavaFileIndexerExtension to allow client apps to define the index as well. This is due to the fact the Comparator class is defined in OptaPlanner WB. I also added new methods to DomainHandler interfaces (one to process events on client side - when the planner data editor is not constructed yet, we'd miss the updates if we relied on observer method, other one to process data object when model is loaded - it initializes Comparator object of a data object).

Ok. Thank you.
+1

Hello @mcimbora , I wasn't included in this thread, not a problem, also I believe the PR was merged during my holidays. But I could see some coded added here during the review of  https://github.com/droolsjbpm/kie-wb-common/pull/467, and I found something that adds some problems, and potential regressions.
It's ok that you add an "auto-generated" inner class if needed. But, when the java source is being upated, all existing inner classes (the generated by you, but eventually any other existing inner class) are being deleted. But only the generated by you is regenerated. Since  we must maintain the round-trip between "user added stuff" and "WB added stuff", we can only remove/overwrite the code we know that is generated by us.
see:
https://github.com/droolsjbpm/kie-wb-common/blob/master/kie-wb-common-services/kie-wb-common-data-modeller-core/src/main/java/org/kie/workbench/common/services/datamodeller/driver/impl/JavaRoasterModelDriver.java#L707
I think that for this stuff we should apply the same strategy as the one we are working on for the planer generated methods.


Search for nvcc in /usr/local/cuda, if not found in PATH

Thanks!

  Searching a bit.  Seems like maybe something like:
#!/bin/bash

set -e
set -x

if [ -z ${foo+yes} ]; then {
  echo not defined
} else {
  echo defined
} fi

coudl be the correct way (reference http://stackoverflow.com/questions/3601515/how-to-check-if-a-variable-is-set-in-bash/13864829#13864829 and also http://unix.stackexchange.com/questions/56837/how-to-test-if-a-variable-is-defined-at-all-in-bash-prior-to-version-4-2-with-th/56846#56846 )

Interestignly, it's complicated slightly, because zsh doesnt return nothing for which, if not found:
% which nvcc
nvcc not found

Seems like maybe simply testing $? is or isnt 0 could be sufficient?  in bash:
ubuntu@peach:~/git/distro$ a=$(which nvcc)
ubuntu@peach:~/git/distro$ echo $?
1
ubuntu@peach:~/git/distro$ a=$(which gcc)
ubuntu@peach:~/git/distro$ echo $?
0

In zsh:
ubuntu@peach ~/git/distro % a=$(which nvcc)
ubuntu@peach ~/git/distro % echo $?
1
ubuntu@peach ~/git/distro % a=$(which gcc)
ubuntu@peach ~/git/distro % echo $?
0


Embedded Picture Cover Type...

Embedding an image with "tone --meta-cover-file=image.jpg audio.mp3", works nicely, but no Type is set (not is there a command line for setting it). For mp3 this defaults to "Other" and not "Cover (Front)" like the default in just about any tool I've tried before.
You can check with mediainfo, exiftool, ... and id3-images.exe extracts the file to "other.jpg" and not front_cover.jpg.
Using tone release 0.1.5 on Windows.

Also atldotnet maintainer.

Is there any update on this? I tried using tone to add metadata to some files, and the album art is correctly shown for Apple Music, but it isn't shown for Finder and Spotify, and I assume it's related.

I try to fix this in the next release.

Well, investigating the code, I found something I completely forgot. If you name the cover cover.front.jpg it will be used as front cover.
While this is not fixing your issue, it may be interesting as a workaround.
For the next release I'm not going to implement a new parameter style but just using the first cover as FRONT, the second as BACK and the third as CD, as long as the filename does not contain keywords.
Using GENERIC as default is not helpful at all.


fix for plugin version update

Proposed changes
fix for plugin version update
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce to agents-aea?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the develop branch (left side). Also you should start your branch off our develop.
[ ] Lint and unit tests pass locally with my changes and CI passes too
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked that code coverage does not decrease.
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...
DELETE INCLUSIVE THIS AND BELOW FOR STANDARD PR
Release summary
Version number: [e.g. 1.0.1]
Release details
Describe in short the main changes with the new release.
Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the main branch (left side), from develop
[ ] Lint and unit tests pass locally and in CI
[ ] I have checked the fingerprint hashes are correct by running (aea packages lock --check)
[ ] I have regenerated the latest API docs
[ ] I built the documentation and updated it with the latest changes
[ ] I have added an item in HISTORY.md for this release
[ ] I bumped the version number in the aea/__version__.py file.
[ ] I bumped the version number in every Docker image of the repo and published it. Also, I built and published them with tag latest
(check the READMEs of aea-develop
and aea-user)
[ ] I have pushed the latest packages to the registry.
[ ] I have uploaded the latest aea to PyPI.
[ ] I have uploaded the latest plugins to PyPI.

Further comments
Write here any other comment about the release, if any.

@angrybayblade
experimental one. just to discuss. feel free to close

Codecov Report
Base: 93.28% // Head: 93.36% // Increases project coverage by +0.08% :tada:

Coverage data is based on head (bd66986) compared to base (6616082).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #571      +/-   ##
==========================================
+ Coverage   93.28%   93.36%   +0.08%
==========================================
  Files         353      353
  Lines       28531    28409     -122
==========================================
- Hits        26615    26525      -90
+ Misses       1916     1884      -32

Flag
Coverage Δ

unittests
93.36% <100.00%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

aea/helpers/base.py
100.00% <100.00%> (ø)

packages/fetchai/contracts/erc1155/contract.py
56.83% <0.00%> (-7.02%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[site support request] yjc.ir

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dl version 2019.09.01
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Description
YJC or Young Journalists Club is a news service.
Some of the posts on their website may contain a video or audio file.
They simply provide a download link themselves.
Copyright
They allow you to use their content as long as you refer them as the intellectual owner.(as noted by a warning at the website's footer section)
Example URLs
Here's a sample post containing a video
A sample download link: https://cdn.yjc.ir/files/fa/news/1398/6/20/10541513_741.mp4
details: https://cdn.yjc.ir/files/fa/news/[year]/[month]/[day]/[video_id].mp4

Only the Persian section of the website contains video and audio files.(as it seems)
They provide only a single-quality permanant non-expirable downlaod link.

Have you even tried before posting request?

I have tried it on three different videos and it is working on all of them

I suppose you are thinking about downloading from the links provided but I made the extractor for the parent website containing the link in the issues. Sample website is "https://www.yjc.ir/fa/news/7310480/وقتی-لباس-شخصی‌های-هند-برای-متفرق-کردن-مردم-وارد-عمل-می‌شوند-فیلم" which contains this link "https://cdn.yjc.ir/files/fa/news/1399/1/20/11683420_225.mp4" as the downloadable link.
My extractor works fine. It is more user friendly as the user can just post the parent website and download videos.


add generate_profile_shell_scripts.py

Motivation and Context
Reference python script for profiling DDPPO PointNav on the FAIR internal cluster used by the Habitat team. This script can be modified locally to suit your needs, for example, to profile a different program or to profile with different settings.
For an overview of profiling and optimization in Habitat, see:
https://colab.research.google.com/gist/eundersander/b62bb497519b44cf4ceb10e2079525dc/faster-rl-training-profiling-and-optimization.ipynb
How Has This Been Tested
Ad hoc testing on my Fedora desktop and FAIR cluster.
Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@eundersander Can you make sure the generated scripts pass shellcheck? We use it in Habitat-Sim as a pre-commit hook, maybe it's time to add it this repo as well if we continue to add shell scripts.

Can you make sure the generated scripts pass shellcheck?
Done. I fixed a few things which are included in the "polish" commit.

Mind dropping in the shellcheck pre-commit hooks, because why not.

Mind dropping in the shellcheck pre-commit hooks, because why not.
https://github.com/facebookresearch/habitat-lab/pull/509


Use typechain-polkadot in a script when it becomes possible instead of spawning a child process

Currently this is spawned as a child process due to limitations in typechain-polkadot

Not 100% sure what this means, moving to needs refinement


Created a kubernetes cluster and tried to access the logs for the pods via machinectl.
Currently, below pods are running:
rkt list --full
UUID					APP		IMAGE NAME					IMAGE ID		STATE	CREATED				STARTED				NETWORKS
1b7fa8c9-e3c5-4f42-83e6-2c866ef9e1c5	flannel		asdrepo.isus.emc.com_8091/flannel:0.5.5		sha512-4cd14fc8b649	exited	2016-11-14 18:03:24 +0000 UTC	2016-11-14 18:03:24 +0000 UTC
6b541ce7-6386-4ab6-af2e-0ec5bce43dc0	hyperkube	asdrepo.isus.emc.com_8091/hyperkube:v1.3.4	sha512-02c35ea1df4b	running	2016-11-14 18:03:25 +0000 UTC	2016-11-14 18:03:25 +0000 UTC
71d006f0-8f8a-4a72-9a26-3af1bf20ecde	flannel		asdrepo.isus.emc.com_8091/flannel:0.5.5		sha512-4cd14fc8b649	running	2016-11-14 18:03:23 +0000 UTC	2016-11-14 18:03:23 +0000 UTC

What did you expect to see?
Machinectl should list the above machines
What did you see instead?
machined is not registering the pods listed by rkt.
machinectl list
MACHINE CLASS SERVICE

0 machines listed.

Note that the above k8s cluster was started using Openstack Magnum, which uses the scripts here https://github.com/openstack/magnum/tree/master/magnum/drivers/k8s_coreos_v1/templates/fragments

I believe the above pods are all using stage1-fly. stage1-fly does not support machined registration as far as I know.

Thanks @euank . What's the best way to look at logs of individual apps in this case?

stage1-fly just dumps on stdout/stderr as any other process would. That means that if it's run as a systemd unit, its logs will be picked up by journald and you can use the normal journalctl -u kubelet.service or journalctl -u flanneld.service commands you'd use for any other systemd daemon.

Thank you @euank I do see the logs using journalctl.


Updating everything to use es6 imports, setting sideEffects false

This commit is to standardize all imports in nosqlprovider to es6, and also set sideEffects as false.
The sideEffects false helps with tree-shaking in webpack and other libraries.
I've also fixed up all the TSLint errors that were present in some of the test files.

LGTM!