[SPARK-29007][STREAMING][MLLIB] Enforce not leaking SparkContext in tests which creates new StreamingContext with new SparkContext

What changes were proposed in this pull request?
This patch enforces tests to prevent leaking newly created SparkContext while is created via initializing StreamingContext. Leaking SparkContext in test would make most of following tests being failed as well, so this patch applies defensive programming, trying its best to ensure SparkContext is cleaned up.
Why are the changes needed?
We got some case in CI build where SparkContext is being leaked and other tests are affected by leaked SparkContext. Ideally we should isolate the environment among tests if possible.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Modified UTs.

Test build #110231 has started for PR 25709 at commit 8a66b15.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #110233 has started for PR 25709 at commit 7fe3b09.

Test build #110231 has finished for PR 25709 at commit 8a66b15.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110231/
Test PASSed.

Test build #110233 has finished for PR 25709 at commit 7fe3b09.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110233/
Test PASSed.

I agree with the direction, tests should be as independent as possible.
Maybe such functionality can be wrapped in a trait like SharedSparkContext. The actual implementation requires a lot of static function calls which can be forgotten easily on new tests.

Well, it's hard to enforce that tests do the right thing (either call the static methods or extend from a common class that does the cleanup); but I'd prefer to follow existing patterns that creating yet another one.
e.g. have a LocalStreamingContext trait like there is LocalSparkContext
The withStreamingContext approach is also good, but to use that everywhere it would mean a lot of changes (mostly related to indentation, though).

Thanks both of you for the valuable feedbacks! I agree trait would be better to not forget and blindly apply to test suites. I'll try to reflect the change.
Personally I'd in favor of try-with-resource-like patterns (withXXXContext methods are similar) so if I need to make a change I'd hope to apply this, but if the test suite already has a field for (Streaming/Spark)Context and cleanup with this, I won't change the approach.

Test build #110316 has started for PR 25709 at commit 0f3a4f4.

Added LocalStreamingContext and applied to test suites which clean up StreamingContext in afterXXX. To be consistent (and by intention to isolate SparkContext between test), I just changed the test suites which initialize SparkContext per suite to initialize SparkContext per each test.

Test build #110317 has started for PR 25709 at commit fda1c0e.

Test build #110316 has finished for PR 25709 at commit 0f3a4f4.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class StreamingKMeansSuite extends SparkFunSuite with LocalStreamingContext with TestSuiteBase
class DStreamClosureSuite extends SparkFunSuite with LocalStreamingContext with BeforeAndAfterAll
class StreamingListenerSuite extends TestSuiteBase with LocalStreamingContext with Matchers
class InputInfoTrackerSuite extends SparkFunSuite with LocalStreamingContext with BeforeAndAfter
trait LocalStreamingContext extends BeforeAndAfterEach with BeforeAndAfterAll

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110316/
Test FAILed.

Test build #110317 has finished for PR 25709 at commit fda1c0e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110317/
Test FAILed.

Test build #110323 has started for PR 25709 at commit 087e1cc.

Test build #110323 has finished for PR 25709 at commit 087e1cc.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait LocalStreamingContext extends BeforeAndAfterEach with BeforeAndAfterAll

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110323/
Test PASSed.

I think the question is the granularity of defensiveness. In the Structured Streaming area there were huge execution times because each and every test has created its own SparkSession. That's the main reason why SharedSparkSession was introduced.
In some cases here we're switching to per test StreamingContext creation. Do we have a rough number what does that mean in terms of execution time?

Unfortunately no, I don't have the number, and actually I expect these numbers would fluctuate unless we have some way to correctly measure it.
Both LocalJavaStreamingContext and LocalSparkContext deal with cleanup per test - I agree it would be generally slower then before if test suites deal with cleanup per suite, but that's also obvious that it would be safer. So that sounds me as preference or policy. Let's hear about further opinions - I'm open to roll back and just apply withStreamingContext or even call static function for these suites.

I'd just compare the runtimes for some key large test suites where this change causes the context to be re-created per test. You can see them in the PR builder and compare to a recent master build. Yes that's also my concern, whether that side effect causes the test time to increase significantly. I also agree it could be worth it for isolation. If we're adding a minute or so? hm, OK. If it's 10 minutes, we probably need to go back and restore the previous behavior in some cases where it's still correct to share the context across tests.

As @srowen said there are couple of tests involved and can be checked in PR builder. I don't expect really huge jumps (and don't think rollback is needed) but would be good to speak about measurable numbers.

Yeah I was careful how to run them in my dev env. and make the numbers being consistent, but looking PR builder would be a good idea. Let me see some elapsed time for test suites being touched in this patch.

#110323 is the build number this patch is activated. Let me find the others and compare with.

Here's the comparison between other builds (emphasized #110323  via rectangle):

(Showing N mins are only valid - some tests are disabled if flag is turned off.)

Looks like it might be able to contribute couple of seconds (or even more) but it seems to be small compared to the fluctuation among builds.

I've had a look and the time consumption numbers looks good, no significant increase so this part is fine.

Test build #110366 has started for PR 25709 at commit 1bd2b54.

Test build #110366 has finished for PR 25709 at commit 1bd2b54.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110366/
Test PASSed.

Addressed modified suites in this patch to use BeforeAndAfterEach. I didn't check other suites as the scope of PR is getting bigger.

Test build #110388 has started for PR 25709 at commit d2ed050.

Test build #110388 has finished for PR 25709 at commit d2ed050.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait LocalStreamingContext extends BeforeAndAfterEach
trait TestSuiteBase extends SparkFunSuite with BeforeAndAfterEach with Logging
class InputInfoTrackerSuite extends SparkFunSuite with LocalStreamingContext

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110388/
Test PASSed.

Test build #110450 has started for PR 25709 at commit d12b28e.

Test build #110450 has finished for PR 25709 at commit d12b28e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110450/
Test FAILed.

Failed tests:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/110450/testReport/org.apache.spark.sql.hive.execution/ObjectHashAggregateSuite/randomized_aggregation_test____with_partial___unsafe__with_distinct____with_grouping_keys___with_empty_input/
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/110450/testReport/org.apache.spark.sql.hive.execution/ObjectHashAggregateSuite/randomized_aggregation_test____with_partial___unsafe__with_distinct____with_grouping_keys___with_non_empty_input/
Both are not relevant.

retest this, please

Test build #110460 has started for PR 25709 at commit d12b28e.

Test build #110460 has finished for PR 25709 at commit d12b28e.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110460/
Test FAILed.

Failed test:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/110460/testReport/org.apache.spark.streaming.kinesis/WithoutAggregationKinesisStreamSuite/failure_recovery/
and...
[info] HiveSQLViewSuite:
Exception in thread "Thread-195" java.io.EOFException
	at java.io.ObjectInputStream$BlockDataInputStream.peekByte(ObjectInputStream.java:2960)
	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1540)
	at java.io.ObjectInputStream.readObject(ObjectInputStream.java:431)
	at org.scalatest.tools.Framework$ScalaTestRunner$Skeleton$1$React.react(Framework.scala:818)
	at org.scalatest.tools.Framework$ScalaTestRunner$Skeleton$1.run(Framework.scala:807)
	at java.lang.Thread.run(Thread.java:748)
Attempting to post to Github...
[error] running /home/jenkins/workspace/SparkPullRequestBuilder/build/sbt -Phadoop-2.7 -Pkubernetes -Phadoop-cloud -Phive -Phive-thriftserver -Pyarn -Pspark-ganglia-lgpl -Pkinesis-asl -Pmesos -Dtest.exclude.tags=org.apache.spark.tags.ExtendedHiveTest,org.apache.spark.tags.ExtendedYarnTest test ; process was terminated by signal 9

not relevant.

retest this, please

Test build #110464 has started for PR 25709 at commit d12b28e.

Test build #110464 has finished for PR 25709 at commit d12b28e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/110464/
Test PASSed.

(Well, I forgot to actually commit the import statement cleanup before the merge. Not a biggie though.)

Thanks all for reviewing and merging!


⚠ Bofrost.at has degraded performance

In 8dc6dac, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 877 ms

Resolved: Bofrost.at performance has improved in 981ec57.


bump heapster addon version to use inClusterConfig

Fixes #10356

status on this?

waiting on GoogleCloudPlatform/heapster#371 to finalize, and @vishh can push new images. Then e2e will pass (no images there yet).

I think @vishh is going to do this

@k8s-bot ok to test

Since @vishh is OOO, @rjnagal @vmarmol could you please take a look?

@rjnagal is also OOO, this LGTM

LGTM. Thanks for the PR @mikedanese

Two leads LGTM'd. Adding ok-to-merge for visibility


Implementation of fairy chess pieces

I was able to successfully add a fairy chess piece to my clone of this repo, and now was wondering if fairy chess pieces could also be added to shaack/cm-chessboard.
See:

https://github.com/chesslablab/cmblab/issues/1

Hi, thank you for the creation, but the cm-chessboard core should be reduced to the minimum.


Chromeheadless not captured inside a windows container

So i have a windows docker container that works as a build agent for AzureDevOps, formally VSTS.
When running Karma Jasmine tests, either via ADO or manually via the command line, Karma launches chrome but never connects. I know the process has launched as using Get_Process from powershell shows the process running and also when Karma tries the second time the Process IDs all change
Tried running with the no sandbox flag and also fails. Any ideas?
Versions:
"karma": "1.7.0",
"karma-chrome-launcher": "2.1.1",
"karma-cli": "1.0.1",
"karma-coverage-istanbul-reporter": "1.2.1",
"karma-jasmine": "1.1.0",
"karma-jasmine-html-reporter": "0.2.2",
"karma-trx-reporter": "0.2.9",
Karma Conf
module.exports = function (config) {
  config.set({
    basePath: '',
    frameworks: ['jasmine', '@angular-devkit/build-angular'],
    plugins: [
      require('karma-jasmine'),
      require('karma-chrome-launcher'),
      require('karma-jasmine-html-reporter'),
      require('karma-coverage-istanbul-reporter'),
      require('@angular-devkit/build-angular/plugins/karma'),
      require('karma-trx-reporter')
    ],
    files: [
      'test-mocks/mock.js',
      'test-mocks/mock.css'
    ],
    proxies: {
      "/test-mocks/": "/base/test-mocks/"
    },
    client: {
      clearContext: false // leave Jasmine Spec Runner output visible in browser
    },
    coverageIstanbulReporter: {
      dir: require('path').join(__dirname, 'coverage'), reports: ['html', 'cobertura'],
      fixWebpackSourcePaths: true
    },
    angularCli: {
      environment: 'dev'
    },
    reporters: ['progress', 'kjhtml', 'trx'],
    trxReporter: {
      outputFile: 'test-results.trx',
      shortTestName: false
    },
    port: 9876,
    colors: true,
    logLevel: config.LOG_DEBUG,
    autoWatch: true,
    browsers: ['ChromeHeadlessNoSandbox'],
    customLaunchers: {
      ChromeHeadlessNoSandbox: {
        base: 'ChromeHeadless',
        flags: [
                '--no-sandbox'
        ]
      }
    },
    browserDisconnectTolerance: 2,
    browserDisconnectTimeout: 180000,
    browserNoActivityTimeout: 180000,
    singleRun: true
  });
};

Debug Output
[plugin]: Loading inlined plugin (defining framework:jasmine).
[plugin]: Loading inlined plugin (defining launcher:Chrome, launcher:ChromeHeadless, launcher:ChromeCanary, launcher:ChromeCanaryHeadless, launcher:Chromium, launcher:Dartium, test).
[plugin]: Loading inlined plugin (defining reporter:kjhtml).
[plugin]: Loading inlined plugin (defining reporter:coverage-istanbul).
[plugin]: Loading inlined plugin (defining framework:@angular-devkit/build-angular, reporter:@angular-devkit/build-angular--sourcemap-reporter, reporter:@angular-devkit/build-angular--event-reporter, middleware:@angular-devkit/build-angular--blocker, middleware:@angular-devkit/build-angular--fallback).
[plugin]: Loading inlined plugin (defining reporter:trx).
[plugin]: Loading inlined plugin (defining launcher:ChromeHeadlessNoSandbox).
10% building modules 1/1 modules 0 active15 02 2019 02:01:31.210:DEBUG [web-server]: Instantiating middleware
[reporter]: Trying to load reporter: @angular-devkit/build-angular--sourcemap-reporter
[reporter]: Trying to load color-version of reporter: @angular-devkit/build-angular--sourcemap-reporter (@angular-devkit/build-angular--sourcemap-reporter_color)
[reporter]: Couldn't load color-version.
[reporter]: Trying to load reporter: @angular-devkit/build-angular--event-reporter
[reporter]: Trying to load color-version of reporter: @angular-devkit/build-angular--event-reporter (@angular-devkit/build-angular--event-reporter_color)
[reporter]: Couldn't load color-version.
[reporter]: Trying to load reporter: kjhtml
[reporter]: Trying to load color-version of reporter: kjhtml (kjhtml_color)
[reporter]: Couldn't load color-version.
[reporter]: Trying to load reporter: trx
[reporter]: Trying to load color-version of reporter: trx (trx_color)
[reporter]: Couldn't load color-version.
[karma]: Karma v1.7.0 server started at http://<IP_ADDRESS>:9876/
[launcher]: Launching browser ChromeHeadlessNoSandbox with unlimited concurrency
[launcher]: Starting browser ChromeHeadless
[temp-dir]: Creating temp dir at C:\Windows\ServiceProfiles\NetworkService\AppData\Local\Temp\karma-82484357
[launcher]: C:\Program Files (x86)\Google\Chrome\Application\chrome.exe --user-data-dir=C:\Windows\ServiceProfiles\NetworkService\AppData\Local\Temp\karma-82484357 --no-default-browser-check --no-first-run --disable-default-apps --disable-popup-blocking --disable-translate --disable-background-timer-throttling --disable-renderer-backgrounding --disable-device-discovery-notifications --no-sandbox http://localhost:9876/?id=82484357 --headless --disable-gpu --remote-debugging-port=9222
[web-server]: serving: C:\BuildAgent_work\1\s\node_modules\karma\static/client.html
[web-server]: serving: C:\BuildAgent_work\1\s\node_modules\karma\static/karma.js
[launcher]: ChromeHeadless have not captured in 60000 ms, killing.
[launcher]: Process ChromeHeadless exited with code 0
[temp-dir]: Cleaning temp dir C:\Windows\ServiceProfiles\NetworkService\AppData\Local\Temp\karma-82484357
[launcher]: Trying to start ChromeHeadless again (1/2).

That is the output with the log level set to debug, set in the karma config as "logLevel: config.LOG_DEBUG". Or do you mean to set this somewhere else?

So i have a windows docker container

I guess this is the issue.  It's not something we can fix in karma-runner.

i haven't tested this just running windows server core in a virtual machine or anything but seems like a strange issue. I see chrome has started but karma just catch attach to it. I put this in the karma chrome launcher issues but the repo doesn't seem very active so i though i would try the root project

Others have reported issues with docker images. I would check the firewall settings to make sure that <IP_ADDRESS> is really localhost. And I would try running a modern karma, 1.7 is old.

unbelievably, this issue is caused by the fonts cache being pretty much empty on the windows docker containers. Chrome seems to be requiring the fonts cache to be populated. im sure there is a specific font it wants but i dont care enough to find out which one


Broken navigation on the "Jobs" page.

The navigation is broken on the jobs page at http://freecodecamp.com/jobs
Visiting the jobs page, the nav bar changes to:

where as the correct nav bar on other pages is:

The Links should be About.
I have fixed this locally, and checked, but have run into a side issue that was apparently pre-existing on the Jobs page nav bar. The spacing between the Links (or corrected to About) and the Sign In button is seems off as seen below:

However it seems to be correct on the other pages like Home or Map pages as seen below:

Solution : The fix can be done by changing the common/app/components/Nav/links.json
{
  "content": "Links", //Change this to About
  "link": "/links" //Change this to about
}]

However I am not sure where to fix the CSS for the spacing part of the secondary issue.

@BerkeleyTrue - Looks like @BerkeleyTrue's changes to the site didn't make it into the Jobs section.

I'll fix on the redesign branch

Closing this issue as it is fixed.


High FDR on dream synthetic dataset 3

I am experiencing a 26% false discovery rate (FDR) on the DREAM dataset 3 for indels only.
I run the tool with default parameters using the "paired variant calling" command from the documentation. For evaluation I used only calls from the output VCF marked as "PASS" and labelled as "Somatic".  Figure 6 from the vardict paper also shows about ~16% FDR on snv+indel combined. The rate seems too high.
Is such FDR what is expected on the DREAM dataset 3 when vardict is run with the default parameters?

Giuseppe;
Thanks for looking into this -- it's always great to have more people working on validations. Here are my validations from running VarDict in bcbio:

DREAM synthetic 4: http://imgur.com/a/gqzwm
DREAM synthetic 3: http://imgur.com/a/qba5k

I'm not running with fully default parameters since I loosen up some to provide better sensitivity than post filter (https://github.com/chapmanb/bcbio-nextgen/blob/master/bcbio/variation/vardict.py#L267) to reduce the introduced false positives from filtering. I could try running a side-by-side comparison of with and without this to get a sense of baseline results without any tweaks if it would help the work you're doing. Thanks again for the discussion and testing.

Thank you Brad. The 26% indel FDR is actually for DREAM set 4 (not 3 as mentioned in my previous post). The sensitivity I get is 62%. In any case, based on your benchmarking, adjusting the p-value threshold (-P parameter) should take care of removing most of the false-positives. I'll give it a try.


Bluetooth: Mesh: Split RPL module

Splits the RPL module to isolate the general RPL features
and the persistent storage features. This will eventually
allow introducing alternate persistent storage schemes
for the replay protection list.
Signed-off-by: Anders Storrø<EMAIL_ADDRESS>

I am a little bit uncertain if this way with the kconfig options will actually work. If this is going into the nrfconnect/sdk-zephyr it will not have access to emds that is in nrf. EMDS has to be in nrf because it is using partition manager. I do not think it will have access to CONFIG_EMDS at least.

LGTM: Doing it this way we do not even need a patch to sdk-zephyr. We can add the changes we want directly to nrf and mesh.


Move on_failure to a handler module

The on_failure was pretty verbose. This PR moves that to a simple handler that should define
:unauthenticated and :unauthorized function instead.
ping @chrismccord

This is a great change @hassox :+1:


Tensorflow has not supported operator [LRN]

mmtoir -f tensorflow -d alexnet -n test.meta --dstNodeName fc8/fc8
Parse file [test.meta] with binary format successfully.
Tensorflow model file [test.meta] loaded successfully.
Tensorflow has not supported operator [LRN] with name [norm1].
Tensorflow has not supported operator [LRN] with name [norm2].
Can anyone help me with this? I'm not seeing the rename_LRN in tensorflow_parser.py, but in tensorflow_frozenparser.py I can see... I don't know if it's the problem, I'm a little bit confused :/

Hi @luisarbezerra, we haven't tested the Alexnet of TensorFlow, then we haven't handled the LRN layer of TensorFlow. I will fix it and ping you if it is finished. Thanks.

Here's the meta file! :) (i
new-model.meta.zip

Hi @luisarbezerra , implemented (but not tested). Please check the newest code. Thanks!

file MMdnn/mmdnn/conversion/tensorflow/tensorflow_parser.py has not rename_LRN,
so it cann't support LRN  operator


[RFC] Replace asyncData with serverPrefetch

What is the motivation for adding / enhancing this feature?
We should replace asyncData with serverPrefetch in the core/pages
This is probably not urgent as serverPrefetch  it's just "asyncData on steroids". However, it probably allows us to simplify and refactor Product.js:asyncData and Category:asyncData that is currently a little bit messy and duplicated with the validateRoute in both: Category and Product
What are the acceptance criteria

The data fetching methods: validateRoute and asyncData unified to a single one,
Data fetching schema works exactly the same way as it's working now - single fetch if SSR request (no additional CSR request), single fetch when CSR request
asyncData methods kept as a backward compatibility
currently, we do have access to ssrContext in the asyncData - by the time of this refactoring we should have the option to add custom middlewares already in place

This task should be done after #2446 merged in, not earlier - to avoid conflicts

Also in this case we should get rid of things like preAsyncData and multiple asyncData invocations for core/theme components modifying each other. Imho we should seek for a more elegant solution to modify asyncData calls.

After product + category page refactor I'm not sure but maybe we should move the SSR data fetching logic all to this?


No syntax errors reported

Development environment used

Z Open Editor version: 4.3.0
Editor Platform

[x] Visual Studio Code

Editor Platform Version: 1.90.1
Operating System: Ubuntu 22.04
Java Version: openjdk version "11.0.23" 2024-04-16
Logs attached: no

Problem Description
Detailed steps for reproducing the problem:

Open a file written in COBOL with any detectable error, for example

       IDENTIFICATION DIVISION.
       PROGRAM-ID. TEST1.
       ENVIRONMENT DIVISION.
       DATA DIVISION.
       PROCEDURE DIVISION.
           DISPLAY "HELLO WORLD!"
           FUNCTION NOTAFUNCTION("").
           STOP RUN.

Observed behavior

Nothing

Screenshot of version 4.3.0

Expected behavior

Errors should show up in the "PROBLEMS" menu of the VSCode terminal, for example: 'NOTAFUNCTION' is not a valid intrinsic or user defined function.
Errors should be highlighted in the editor, for example

Screenshot of version 4.2.0:

Note
I made sure to follow the instructions for resetting the extension's COBOL language server and VSCode before reproducing this behavior.

The setting is not enabled.

Log file:
2024-06-19T17:26:03.282+02:00 INFO : Zowe Explorer was modified for IBM Remote System Explorer API (RSE API) support by IBM Z Open Editor.
2024-06-19T17:26:03.543+02:00 INFO : IBM Z Open Editor extension has (re)started

Thanks for verifying.  Can you change your log level to DEBUG temporarily and re-upload the log after opening it again, INFO isn't providing any insight into what is going wrong.

Logs with DEBUG enbled
v4.2.0
2024-06-19T17:42:53.829+02:00 DEBUG: TelemetryStats after restart: {"lastResetDate":"2024-06-17T22:11:31.164Z","numberOfEvents":202,"numberOfMaxEvents":0}
2024-06-19T17:42:53.831+02:00 DEBUG: UsageReporter: Sending the following telemetry data: {"eventName":"activated","properties":{"editor":"Visual Studio Code"}}
2024-06-19T17:42:53.871+02:00 DEBUG: Zowe CLI Secure Credentials is active. Will initialize Z Open Editor to utilize it.
2024-06-19T17:42:53.884+02:00 DEBUG: Zowe CLI Secure Credentials is successfully activated for Z Open Editor.
2024-06-19T17:42:54.057+02:00 INFO : Zowe Explorer was modified for IBM Remote System Explorer API (RSE API) support by IBM Z Open Editor.
2024-06-19T17:42:54.065+02:00 DEBUG: UsageReporter: Sending the following telemetry data: {"eventName":"licenses/state","properties":{"event.value":"trial-active","editor":"Visual Studio Code"}}
2024-06-19T17:42:54.067+02:00 DEBUG: ZAPP: No ZAPP file was found in /home/ziggy/Documents/Projects/zxplore. User and workspace settings will be used. Started watcher for when a zapp file does get created.
2024-06-19T17:42:54.068+02:00 DEBUG: ZCodeFormat: No zcodeformat file was found in /home/ziggy/Documents/Projects/zxplore. Formatter settings in other workspaces or the default formatter settings will be used. Started watcher for when a zcodeformat file does get created.
2024-06-19T17:42:55.025+02:00 DEBUG: UsageReporter: Sending the following telemetry data: {"eventName":"java/found","properties":{"editor":"Visual Studio Code","platform":"linux","java.majorversion":"11","java.vendor":"openjdk"},"measurements":{"sinceActivated":1.196}}
2024-06-19T17:42:55.026+02:00 DEBUG: JavaFinder: Used the "command -v java" Linux command to locate Java.
2024-06-19T17:42:55.141+02:00 DEBUG: UsageReporter: Sending the following telemetry data: {"eventName":"zowe/profileTypes","properties":{"event.value":"multiple","editor":"Visual Studio Code"}}
2024-06-19T17:42:55.184+02:00 DEBUG: Z Open Editor environment summary:
- Z Open Editor version: 4.2.0
- Editor platform: Visual Studio Code
- Editor platform version: 1.90.1
- Required minimal editor platform version: ^1.81.0
- Editor installation root directory: /snap/code/161/usr/share/code/resources/app
- Editor installation id: e78c9b6f3ab111d158fa8a68a63374503bb689571d769242b2728753b26e2e9a
- Editor workspaces: /home/ziggy/Documents/Projects/zxplore
- Is a valid Java configured: true
  - Java: /usr/lib/jvm/java-11-openjdk-amd64/bin/java
  - Java memory allocation: 512
  - Java startup user settings: Not defined.
- Key user settings:
  - Unreachable Code enabled: true
  - HLASM version used: 3.1
  - HLASM macros file used:
  - HLASM instructions file used: /home/ziggy/.vscode/extensions/ibm.zopeneditor-4.2.0/resources/hlasm-instructions.json
  - HLASM Macro resolution enabled: false
  - HLASM Custom Macro Warnings enabled: true
  - HLASM Custom Macro Reduced Logging enabled: false
  - Enforce Case Sensitive Include File Names: false
  - ZAPP schema used: file:///home/ziggy/.vscode/extensions/ibm.zopeneditor-4.2.0/resources/zapp-schema-1.1.1.json
  - Variables defined by all ZAPP documents in workspace:
  - Found the following ZAPP variable declarations in VS Code settings: []
  - Automatic preprocessor execution setting: off
  - IBM User Build enabled: true
  - Zowe CLI API profile management used: v2 team config in use.
  - Zowe CLI Secure Credentials enabled: true
  - Zowe data providers used: multiple
  - Zowe CLI profiles of types rse, zosmf or ssh found: zosmf(zosmf),rse(rse),ssh(ssh)
  - Zowe CLI files profile name specified in VS Code user settings: Not defined.
  - Zowe CLI SSH profile name specified in VS Code user settings: Not defined.
  - Zowe CLI files profile used for remote include files: rse
  - Zowe CLI SSH profile used: ssh
  - Zowe CLI profiles configured for user build: rse (rse)
  - Zowe Explorer RSE API Mappings File setting: Not defined.
  - Maximum number of parallel downloads over Zowe profiles: 5
  - Include file download strategy: Directly tries to download files from data sets.
  - Zowe Temp-Dir value: Not defined.
  - COBOL Property Groups defined:
  - COBOL include file extensions: .cpy,.copybook,.copy,.cobcopy
  - COBOL disabled problems: false
  - PL/I Property Groups defined:
  - PL/I include file extensions: .inc,.include,.plinc,.pcx,.pci
  - PL/I disabled problems: false
  - HLASM Property Groups defined:
  - HLASM include file extensions: .s,.assemble,.asm,.hlasm,.hla,.copyasm,.macasm,.bms,.mac
  - HLASM disabled problems: false
  - REXX disabled problems: false
  - VS Code Files Associations: *.COBOL*:cobol,*.COB*:cobol,*.COBCOPY*:cobol,*.COPYBOOK*:cobol,*.COPY*:cobol,*.PL1*:pl1,*.PLI*:pl1,*.INC*:pl1,*.INCLUDE*:pl1,*.JCL*:jcl,*.ASM*:hlasm,*.ASSEMBLE*:hlasm,*.HLASM*:hlasm,*.HLA*:hlasm,*.MACLIB*:hlasm,*.EXEC*:rexx,*.REXX*:rexx,*.REXXINC*:rexx,*.rex:rexx,*.rexx:rexx,*.CBL*:cobol
- Zowe Explorer VS Code Extension version: 2.16.2
  - Successfully activated: true
  - Installation path: /home/ziggy/.vscode/extensions/zowe.vscode-extension-for-zowe-2.16.2
  - Log file location: /home/ziggy/.vscode/extensions/zowe.vscode-extension-for-zowe-2.16.2/logs
- Other VS Code extensions visible: vscode.bat(1.0.0),vscode.clojure(1.0.0),vscode.coffeescript(1.0.0),vscode.configuration-editing(1.0.0),vscode.cpp(1.0.0),vscode.csharp(1.0.0),vscode.css(1.0.0),vscode.css-language-features(1.0.0),vscode.dart(1.0.0),vscode.debug-auto-launch(1.0.0),vscode.debug-server-ready(1.0.0),vscode.diff(1.0.0),vscode.docker(1.0.0),vscode.emmet(1.0.0),vscode.extension-editing(1.0.0),vscode.fsharp(1.0.0),vscode.git(1.0.0),vscode.git-base(1.0.0),vscode.github(0.0.1),vscode.github-authentication(0.0.2),vscode.go(1.0.0),vscode.groovy(1.0.0),vscode.grunt(1.0.0),vscode.gulp(1.0.0),vscode.handlebars(1.0.0),vscode.hlsl(1.0.0),vscode.html(1.0.0),vscode.html-language-features(1.0.0),vscode.ini(1.0.0),vscode.ipynb(1.0.0),vscode.jake(1.0.0),vscode.java(1.0.0),vscode.javascript(1.0.0),vscode.json(1.0.0),vscode.json-language-features(1.0.0),vscode.julia(1.0.0),vscode.latex(1.0.0),vscode.less(1.0.0),vscode.log(1.0.0),vscode.lua(1.0.0),vscode.make(1.0.0),vscode.markdown(1.0.0),vscode.markdown-language-features(1.0.0),vscode.markdown-math(1.0.0),vscode.media-preview(1.0.0),vscode.merge-conflict(1.0.0),vscode.microsoft-authentication(0.0.1),ms-vscode.js-debug(1.90.0),ms-vscode.js-debug-companion(1.1.2),ms-vscode.vscode-js-profile-table(1.0.9),vscode.builtin-notebook-renderers(1.0.0),vscode.npm(1.0.1),vscode.objective-c(1.0.0),vscode.perl(1.0.0),vscode.php(1.0.0),vscode.php-language-features(1.0.0),vscode.powershell(1.0.0),vscode.pug(1.0.0),vscode.python(1.0.0),vscode.r(1.0.0),vscode.razor(1.0.0),vscode.references-view(1.0.0),vscode.restructuredtext(1.0.0),vscode.ruby(1.0.0),vscode.rust(1.0.0),vscode.scss(1.0.0),vscode.search-result(1.0.0),vscode.shaderlab(1.0.0),vscode.shellscript(1.0.0),vscode.simple-browser(1.0.0),vscode.sql(1.0.0),vscode.swift(1.0.0),vscode.theme-abyss(1.0.0),vscode.theme-defaults(1.0.0),vscode.theme-kimbie-dark(1.0.0),vscode.theme-monokai(1.0.0),vscode.theme-monokai-dimmed(1.0.0),vscode.theme-quietlight(1.0.0),vscode.theme-red(1.0.0),vscode.vscode-theme-seti(1.0.0),vscode.theme-solarized-dark(1.0.0),vscode.theme-solarized-light(1.0.0),vscode.theme-tomorrow-night-blue(1.0.0),vscode.tunnel-forwarding(1.0.0),vscode.typescript(1.0.0),vscode.typescript-language-features(1.0.0),vscode.vb(1.0.0),vscode.xml(1.0.0),vscode.yaml(1.0.0),adpyke.codesnap(1.3.4),alexcvzz.vscode-sqlite(0.14.1),bradlc.vscode-tailwindcss(0.10.5),charliermarsh.ruff(2024.26.0),dbaeumer.vscode-eslint(3.0.10),eamodio.gitlens(15.1.0),esbenp.prettier-vscode(10.4.0),george-alisson.html-preview-vscode(0.2.5),GitHub.remotehub(0.62.0),github.vscode-github-actions(0.26.3),GitHub.vscode-pull-request-github(0.90.0),Hridoy.rails-snippets(1.0.8),IBM.zopeneditor(4.2.0),marp-team.marp-vscode(2.8.0),me-dutour-mathieu.vscode-github-actions(3.0.1),ms-azure-devops.azure-pipelines(1.228.0),ms-azuretools.vscode-azureresourcegroups(0.9.1),ms-azuretools.vscode-docker(1.29.1),ms-python.debugpy(2024.6.0),ms-python.python(2024.8.1),ms-python.vscode-pylance(2024.6.1),ms-toolsai.jupyter(2024.5.0),ms-toolsai.jupyter-keymap(1.1.2),ms-toolsai.jupyter-renderers(1.0.18),ms-toolsai.vscode-jupyter-cell-tags(0.1.9),ms-toolsai.vscode-jupyter-slideshow(0.1.6),ms-vscode-remote.remote-containers(0.369.0),ms-vscode-remote.remote-ssh(0.112.0),ms-vscode-remote.remote-ssh-edit(0.86.0),ms-vscode.azure-account(0.12.0),ms-vscode.azure-repos(0.38.0),ms-vscode.makefile-tools(0.9.10),ms-vscode.remote-explorer(0.4.3),ms-vscode.remote-repositories(0.40.0),ms-vsliveshare.vsliveshare(1.0.5932),njpwerner.autodocstring(0.6.1),redhat.vscode-yaml(1.15.0),Shopify.ruby-lsp(0.7.5),tomoki1207.pdf(1.2.2),tonybaloney.vscode-pets(1.27.0),Zowe.vscode-extension-for-zowe(2.16.2)
2024-06-19T17:43:21.766+02:00 DEBUG: ZLanguageServer.buildServerOptions: Starting COBOL language server from "language-server".
2024-06-19T17:43:21.793+02:00 DEBUG: ZLanguageServer: Will start a language server with command "/usr/lib/jvm/java-11-openjdk-amd64/bin/java -Declipse.application=com.ibm.wazi.lsp.cobol.core.languageServer,-Dosgi.bundles.defaultStartLevel=4,-Declipse.product=com.ibm.wazi.lsp.cobol.core.languageServerProduct,-Xmx512m,-jar,/home/ziggy/.vscode/extensions/ibm.zopeneditor-4.2.0/language-server/plugins/org.eclipse.equinox.launcher_1.6.400.v20210924-0641.jar,-configuration,/home/ziggy/.vscode/extensions/ibm.zopeneditor-4.2.0/language-server/cobol_config_linux,-data,/tmp/_ws_1718811801793".
2024-06-19T17:43:21.815+02:00 DEBUG: AbstractZLanguageClient.initializeLanguageClient: Started initializing a new COBOL language client with these initialization options: {"enableUnreachableCodeWarnings":true,"publishDiagnostics":true,"includeFileExtensions":[".cpy",".copybook",".copy",".cobcopy"],"compilerOptions":"","commercialFeaturesEnabled":true}.
2024-06-19T17:43:21.826+02:00 DEBUG: AbstractZLanguageClient.startLanguageClient: Language client for COBOL started.
2024-06-19T17:43:23.740+02:00 DEBUG: UsageReporter: Sending the following telemetry data: {"eventName":"document/didOpen","properties":{"editor":"Visual Studio Code","language":"cobol"},"measurements":{"sinceActivated":29.911}}

v4.3.0
2024-06-19T17:47:59.543+02:00 DEBUG: TelemetryStats after restart: {"lastResetDate":"2024-06-17T22:11:31.164Z","numberOfEvents":210,"numberOfMaxEvents":0}
2024-06-19T17:47:59.545+02:00 DEBUG: UsageReporter: Sending the following telemetry data: {"eventName":"activated","properties":{"editor":"Visual Studio Code"}}
2024-06-19T17:47:59.568+02:00 DEBUG: Zowe CLI Secure Credentials is active. Will initialize Z Open Editor to utilize it.
2024-06-19T17:47:59.577+02:00 DEBUG: Zowe CLI Secure Credentials is successfully activated for Z Open Editor.
2024-06-19T17:47:59.683+02:00 INFO : Zowe Explorer was modified for IBM Remote System Explorer API (RSE API) support by IBM Z Open Editor.
2024-06-19T17:47:59.698+02:00 DEBUG: UsageReporter: Sending the following telemetry data: {"eventName":"licenses/state","properties":{"event.value":"trial-active","editor":"Visual Studio Code"}}
2024-06-19T17:47:59.700+02:00 DEBUG: ZAPP: No ZAPP file was found in /home/ziggy/Documents/Projects/zxplore. User and workspace settings will be used. Started watcher for when a zapp file does get created.
2024-06-19T17:47:59.701+02:00 DEBUG: ZCodeFormat: No zcodeformat file was found in /home/ziggy/Documents/Projects/zxplore. Formatter settings in other workspaces or the default formatter settings will be used. Started watcher for when a zcodeformat file does get created.
2024-06-19T17:47:59.946+02:00 DEBUG: Z Open Editor environment summary:
- Z Open Editor version: 4.3.0
- Editor platform: Visual Studio Code
- Editor platform version: 1.90.1
- Required minimal editor platform version: ^1.81.0
- Editor installation root directory: /snap/code/161/usr/share/code/resources/app
- Editor installation id: e78c9b6f3ab111d158fa8a68a63374503bb689571d769242b2728753b26e2e9a
- Editor workspaces: /home/ziggy/Documents/Projects/zxplore
2024-06-19T17:48:00.056+02:00 INFO : IBM Z Open Editor extension has (re)started.

Thank you for the logs.  It looks like your language server isn't starting in the background for some reason.  One thing that stands out to me is that there were no "java/found" event triggers.  This leads me in the direction of the extension not properly finding your java for some reason (not your fault the java version you listed is supported).  Can you verify for me that on the Welcome page that we indicate that we were able to locate your JDK?  If you have disabled the Welcome page you can open it manually from the command pallet with the command IBM Z Open Editor: Welcome

v4.2.0

v4.3.0
The prerequisites are stuck loading.

Thanks for the update.  This makes me think there is a bug in our Java location and validation logic that was introduced this release.  To verify (and hopefully unblock you while we fix the bug) can you set your "zopeneditor.JAVA_HOME" value in your user settings to "/usr/lib/jvm/java-11-openjdk-amd64".  Please let me know that results in us locating the Java and the language server starting as expected.

Yes! That fixes it immediately.
The extension found java, the language server started, and syntax errors started getting reported again.

Wonderful, I'm glad that fixed the issue.  I'm going to tag this issue as a bug so we can get that fixed.  Thanks for the feedback!

@seporterfield if you have time, could you share the developer tools log from a run where you do not specify a path in your zopeneditor.JAVA_HOME setting and your prerequisites fail to load? Developer tools can be opened in Visual Studio Code using the command palette and running >Developer: Toggle Developer Tools. If you could screenshot any errors in your console, that would be great. Sometimes an uncaught error is logged there rather than in the output.

We fixed the issue in v.4.4.0. Can you try again without the setting and let us know?


CNTK performance issue with LSTM

We are having problems with the LSTM example in CNTK, the accuracy is now 0.5, while in MXNET, TF and Keras is around 0.86.
Maybe the inputs to the trainer are not correct...
Any idea?

fixed https://github.com/ilkarman/DeepLearningFrameworks/commit/1611d796d669a812bf6e4593da6d374e5c116c68


Global decorators

If you want to apply some option to all cases where a particular field of a particular type is referenced in an operation (most commonly an input type) we have the for directive option for that.  But you might want to do that not just for a single operation, but globally in your genqlient.yaml, for example because some particular type always wants omitempty or suchlike.  (This came up as a workaround in #187, which is a reasonable thing to want and would require several new features to implement otherwise.)  It might be better to add proper features to avoid the need (e.g. in this case #178), but this could be a good way for schemas that find themselves needing these kinds of workarounds.  This could potentially even subsume features like bind: or a future pointer: always, although there are advantages to having the full power of YAML instead of our weird decorator-syntax.
I think this would look something like
# A list of directives which will be applied to every operation.  Typically
# uses `for` but doesn't have to.  Overridden by directives in the
# operation itself.  The '# ' is omitted [because that's also a
# comment in YAML].
global_directives: |
  @genqlient(omitempty: true)
  @genqlient(for: "MyType.myField", bind: "MyGoType")

One thing that would be useful to get a sense of from people with bigger codebases using genqlient (@Khan or elsewhere): how commonly useful would this be?  Do you see lots of directives that get repeated basically everywhere a certain type is used?

From the Khan side: I agree this is most useful for for:, so I just grepped for that.  We're only using that directive twice in our entire codebase, and for different things each time.  So I think this isn't super-useful for us.
Honestly, except maybe for for: I think having these global configs is too magical for my tastes.  I wouldn't want it to be the case that some types are automatically omitempty and some aren't, and the only way to know is to look at some config file far away from the genqlient definition.

edit: I guess omitempty would apply to all types, that's not so bad.

Ah, interesting.  I guess since Khan got to define the defaults that's not super surprising; it's the users using things like hasura that want this sort of thing.  It's a good point that it's a bit magical, in any case, although I worry that the alternative is just adding config options to do the same things, which is not necessarily any less magical.


Adding Bloom filter to small Parquet file bloats in size X1700

Converting a small, 14 rows/1 string column csv file to Parquet without bloom filter yields a 600B file, adding '.withBloomFilterEnabled(true)' to ParquetWriter then yields a 1049197B file.
It isn't clear what the extra space is used by.
Attached csv and bloated Parquet files.
Reporter: Ze'ev Maor
Original Issue Attachments:

data_index_bloom.parquet
data.csv

Note: This issue was originally created as PARQUET-2122. Please see the migration documentation for further details.

Micah Kornfield / @emkornfield:
I believe the answer is the Bloom filter implementation isn't adaptive, so it simply preallocates all the bytes necessary.  It would certainly be a nice option to have more adaptive data structures that can scale down for smaller files but is probably a decent amount of work to build consensus around this.

Xinli Shang / @shangxinli:
@chenjunjiedada Do you know why?

Ze'ev Maor:
@chenjunjiedada thanks, that worked, though it does seem odd that a MAX size on bloom filter of 1MB would actually result in 1MB used by a Bloom filter on a column with cardinality of just 14 isn't it?

Junjie Chen / @chenjunjiedada:
That's the default size of the bloom filter. Please configure parquet.bloom.filter.max.bytes to fit.


Number is geting rounded when being read by knex

#Environment
Knex version: 0.14.4
Database + version: Mysql 2.15
OS: Linux Debian
@Grimace1975
#Bug
1. Explain what kind of behaviour you are getting and how you think it should do
I am reading all Entrys of a table using this code:
await knex.select("*").from(table);
The Entry for the column "channelid" is<PHONE_NUMBER>54473984 (Bigint(20))
But instead of<PHONE_NUMBER>54473984, knex is returning<PHONE_NUMBER>54474000.
2. Error message
There is no Error Message being provided

For anyone that stumbles across this, you can add
    supportBigNumbers: true,
    bigNumberStrings: true,

to the connection field in the knex config which will not cast the bigint to a js number but instead return it as a string.
So something like this:
const knexConfig = {
  client: 'mysql2',
  connection: {
    host: env.DB_HOST,
    user: env.DB_USER,
    password: env.DB_PASSWORD,
    database: env.DB_DATABASE,
    supportBigNumbers: true,
    bigNumberStrings: true,
  },
};

thanks. I will definitely put that into my code


Upgrade org.postgresql:postgresql 42.2.14 -> 42.2.15

This PR contains the following updates:

Package
Update
Change

org.postgresql:postgresql (source)
patch
42.2.14 -> 42.2.15

Release Notes

pgjdbc/pgjdbc
v42.2.15
Changed

Source release archive shades dependencies (scram) by default. It affects only postgresql-version-src.tar.gz release artifact.

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade PostgreSQL JDBC 42.2.14 -> 42.2.15

See:
- https://github.com/pgjdbc/pgjdbc/blob/master/CHANGELOG.md
- https://github.com/pgjdbc/pgjdbc/compare/REL42.2.14...REL42.2.15


Changedmy dashboard - added accomplishment / proud statement line.html

I think it's correct? 😅

(bump to have a look at the comments in the review)


Show help for FTUE and new features

When users open the sidekick for the first time, or new features have been added, the sidekick should display help content.

Content source: https://www.hlx.live/tools/sidekick/help.json
Example:

:tada: This issue has been resolved in version 5.0.0 :tada:
The release is available on:

GitHub release
v5.0.0

Your semantic-release bot :package::rocket:


Fix the title for .get() in the docs

to: @lelandrichardson
.get() returns a ReactElement and not a ShallowWrapper; the title in the docs indicated the former. Let's fix that!

:+1:


⚠ Bofrost.at has degraded performance

In ecc0b52, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1061 ms

Resolved: Bofrost.at performance has improved in e61c219 after 27 minutes.


Problem running

Hi I have this error trying running the container
robsalasco@agafya:~/Dev$ docker run --name cronicle --hostname localhost -p 3012:3012 intelliops/cronicle:latest
/entrypoint.sh: line 29: syntax error: unterminated quoted string

Good catch, problem is solved now. There was still a typo in there. Testing is still an issue that needs to be worked on.


Non-square thumbnails

add thumbnailSize:transparentBorder:cornerRadius:interpolationQuality which can create non-square thumbnails

Apologies for leaving this open for so long! As you can guess, I no longer use this code in any of my current projects. It looks like this PR needs to be updated before it can be merged. Are you still interested in it? If not, no worries, I understand. Thanks, and I apologize for not responding before!


Solving Issue #3075

Solved typos on #3075

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@CarolynRowe PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


NuGet Framework dependencies don't match compiled code

Just added Nanophone into a 4.5.2 project (which should work according to the nupkg). I think it installed the lib\4.5 dll, but it gives this error in Visual Studio

Decompiling the DLL with IL spy shows
[assembly: TargetFramework(".NETFramework,Version=v4.6.1", FrameworkDisplayName = ".NET Framework 4.6.1")]

I presume this is just a build error, but as an aside - please continue to support the oldest possible framework that's practicable (4.5.xxx) - some of us work in environments with legacy code and OS and .Net installations and customers who simply can't upgrade to the latest and greatest at a whim!

@PhilipDaniels .net 451 is fully supported. You might want to use the latest prerelease though, it's been a while (meaning: lots of commits) since version 1.1. Can you try 1.3.1-alpha instead? They have to be marked prerelease because they're depending on other prerelease packages (notably Consul.net <IP_ADDRESS>-alpha2).


Limit reads based on the configured permissions

Read APIs (read, vread, search) should honor the permissions based on the role the user is in.
Acceptance:

A user can only read resources that match the permission filters for their roles
A user is unable to read a resource that doesn't match the permission filter.

Moving work item tracking to Azure DevOps.


Data Science with Python : Generative Adversarial Networks

Welcome to 'DSWP' Team, good to see you here
This issue will helps readers in giving all the guidance that one needs to learn about gettng started with Generative Adversarial Networks . Tutorial to Generative Adversarial Networks and how it's used effectively with sample code.
To get assigned to this issue, add your serial numbers mentioned in the spreadsheet of "Data Science with Python", the approach one would follow and choice you prefer (Documentation, Audio, Video). You can go with all three or any number of options you're interested to work on.
If you had referred any resources, add them up in "DS Resources". Similarly if you had used datasets, include them in "DS Datasets".
Domain : Artificial Intelligence
Mentors Assigned : Yash Goel / Dhruv Bajaj
Points to Note :
The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Changes should be made inside the Datascience_With_Python/ directory & Datascience_With_Python branch.
Follow Contributing Guidelines & Code of Conduct before start Contributing.
This issue is only for 'GWOC' contributors of 'DSWP' domain.
All the best. Enjoy your open source journey ahead. 😎

Sarmistha Sarna
Issue no: 469
Documentation
Approach: Introduction to GAN, Structure, Variations etc

Issue assigned to @ron352 for documentation.

@prathimacode-hub , Mam
I am M. Ashish Reddy. I would like to do video on this issue.
Batch number: 5
serial number:174.
Approach: A detailed explanation with examples and real life uses

Issue assigned to @ashish-reddy-20-08 for video contribution

Hello, I am Deepthi M with serial number:172, Batch-5. I would like to do audio on this issue.
My approach: a detailed explanation

Issue assigned to @deepthi1107 for audio contribution


display of metrical patterns

In our editions of critical and diplomatic editions, we often have to display metrical symbols.
Here are two examples:

It is disturbing for me that the symbols "–" for long syllables stick together in display. This makes counting them more complicated.
Therefore I request insertion of a tiny amount of blank space between all symbols in such displays of metrical patterns.

I have updated the patterns, it might take up to 24 to be applied for critical editions since we use an exterior file to generate the pattern.

At the moment, I don't see the effect yet in our epigraphic editions. Here's an example from tfc-khmer-epigraphy:

Meanwhile, @met is not being displayed at all in such cases as <div type="canto" n="1" met="jagaddhita"> (from DHARMA_CritEdKakavinParthayajna.xml)

Should we apply @rend="met" to make it be displayed? (I thought the default behavior was to display it in case of <div>....)

I have changed all the system if should be stable enough now for both your points

The display in Kakavin Parthayajna now looks nice, thanks!
The display in DHARMA_INSCIK00419, from which I took the epigraphic example, does not seem to have changed.

No updates have been made on tfc-khmer-repository to apply the new transformation on the files. Once someone does, it will appear

I am confused. There have been commmits to that repository in recent hours and yet the display has not changed.

It should be good now. I have reverted to the initial set up of the workflow that will transform all the files rather than just the updated.


F1 on dataset semeval

In the master branch, the flag 'multi_label' is true on semeval. What's more, the calculation of f1  seems to ignore the case where the pred is positive, but the label is negative. I tried to solve these problems, and finally found that f1 could not reach the value in the paper.

@zyccyz yeah same here on nyt10 dataset cannot reach papers results after 10 epoch as the paper suggested also tried till 100 epochs and still no luck.

@kemalaraz  what was the score after 100 epoch?

This score evaluation method is from SemEval's Github: it removes negative samples, which it called "excluded others".

NYT10's problem see here


Keep note open

How to reproduce:

write something in a note.
switch to another app.
come back to add something to the note.
the application closes the note.

The note must be kept open (as well as the position of the cursor/view in it)

will be fixed

Need more time to explore the problem

The recent update fixed a bit of it. The note stays open when you switch back to the app using the app switcher, but not when you reopen the app from the home screen.

The recent update fixed a bit of it. The note stays open when you switch back to the app using the app switcher, but not when you reopen the app from the home screen (while it's still open, aka still in the app switcher).

yeah I'm working on this issue

Fix will be avaible in future version! Subscribe to beta updates to get updates faster on google play!


CSS flexbox vs CSS grid

Article Checklist
NOTE: (1st time contributors - submit ONLY one article to get familiar with the process, quality, and style-once published, you can then submit any number of articles.)
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.

Grammar Checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical Checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution Guidelines
For first-time contributors and for more details, see Contributing Guidelines
fixes #827

@louisefindlay23 Can check this, please?

@louisefindlay23 Can check this, please?

There is no need to open a new PR - you can work on the article via your forked repo - and once completed open a new PR - this is to prevent overflow on the queue.

There is no need to open a new PR - you can work on the article via your forked repo - and once completed open a new PR - this is to prevent overflow on the queue.

Re-submitted in #1517 @louisefindlay23


Bug in Character Comparison

Hi,
I believe there is a bug when CharacterComparison property is set to true, and CharacterComparisonMarkAddRemove set to false, for sidebyside comparison. The scintilla control may add virtual blank lines to map to the extra lines in the other file, and then the line offsets will no longer match the correct lines in editor. It will highlight lines that are identical.
You can replicate while comparing some xml files, which have one element per line.

Could you paste a screenshot please - it would give somewhat nicer understanding of the bug in question? 🤔


[BUG] Cannot read properties of undefined (reading 'className')

What happened?
not able to see details of offer created
Version
1.5
What platform are you using?
Windows
What ui mode are you using?
GUI
Relevant log output or stacktrace
Cannot read properties of undefined (reading 'className')

URL
#/dashboard/offers/view

Stacktrace
at webpack://@chia/gui/node_modules/@mui/material/Tooltip/Tooltip.js:507:52 popperRef.current.update
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:157:136 Ch
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:176:259 useOpaqueIdentifier
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:271:399 d=b.pendingProps;e=b.memoizedState;e=null!==e?e.element:null;yg(a,b);Cg(b,d,null,c);d=b.memoizedState.element;if(d===e)sh(),b=hi(a,b,c);else{e=b.stateNode;if(f=e.hydrate)kh=rf(b.stateNode.containerInfo.firstChild),jh=b,f=lh=!0;if
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:346 Pj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:277 Pj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:250:137 Pj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:243:162 Lj
at webpack://@chia/gui/node_modules/react-dom/cjs/react-dom.production.min.js:123:114 jg
at webpack://@chia/gui/node_modules/scheduler/cjs/scheduler.production.min.js:18:342 unstable_next

Thanks for the report @harshesp. Please try 1.5.1 as this bug has been addressed in that version.


[NEW ALGORITHM]

Issue will be closed if:

You mention more than one algorithm. You can create a separate issue for each algorithm once the current one is completed.
You propose an algorithm that is already present or has been mentioned in a previous issue.
You create a new issue without completing your previous issue.

Note: These actions will be taken seriously. Failure to follow the guidelines may result in the immediate closure of your issue.

Name:
[NEW ALGORITHM]
Added Maximum Alternating Subsequence Sum
About:
Propose a new algorithm to be added to the repository
This program finds the maximum alternating subsequence sum for a given input array where alternating subsequence sum is difference btw sum of elements at even indices and sum of elements at odd indices within the subsequence

Labels:
new algorithm, gssoc-ext, hacktoberfest-accepted,hacktoberfest, level1

Assignees:

[x] Contributor in GSSoC-ext
[ ] Want to work on it

assigned

no update


add the translate of app.md in zh-CN

You have accidentally committed the changes under vendor/, can you revert them?

i add a new pull request, close this one


Python: adds ZRANGE command

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

please align tests with https://github.com/aws/glide-for-redis/pull/910

@barshaul ready


When two services come up in Freshping, only one gets resolved in PagerDuty

Both HTTP (#11) and SMTP (#10) services went down, probably because of an Internet outage.
FreshPager successfully triggered two incidents in PagerDuty.
Both services came back up at the same time.
FreshPager resolved the HTTP incident with the wrong deduplication key, and never resolved the SMTP incident.

Logs
Sep 19 01:11:54 erebus systemd[1]: Starting FreshPager...
Sep 19 01:11:56 erebus freshpager[29668]: Microsoft.Hosting.Lifetime[14] Now listening on: http://[::]:37374
Sep 19 01:11:56 erebus freshpager[29668]: Microsoft.Hosting.Lifetime[0] Application started. Hosting environment: Production; Content root path: /opt/freshpager
Sep 19 01:11:56 erebus systemd[1]: Started FreshPager.
Sep 25 07:20:31 erebus freshpager[29668]: Program[0] Aldaviva SMTP is down
Sep 25 07:20:31 erebus freshpager[29668]: Program[0] Aldaviva HTTP is down
Sep 25 07:20:31 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.LogicalHandler[100] Start processing HTTP request POST https://events.pagerduty.com/v2/enqueue
Sep 25 07:20:31 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.LogicalHandler[100] Start processing HTTP request POST https://events.pagerduty.com/v2/enqueue
Sep 25 07:20:31 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.ClientHandler[100] Sending HTTP request POST https://events.pagerduty.com/v2/enqueue
Sep 25 07:20:31 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.ClientHandler[100] Sending HTTP request POST https://events.pagerduty.com/v2/enqueue
Sep 25 07:20:34 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.ClientHandler[101] Received HTTP response headers after 3157.3395ms - 202
Sep 25 07:20:34 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.ClientHandler[101] Received HTTP response headers after 3157.3067ms - 202
Sep 25 07:20:35 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.LogicalHandler[101] End processing HTTP request after 3307.7762ms - 202
Sep 25 07:20:35 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.LogicalHandler[101] End processing HTTP request after 3307.8208ms - 202
Sep 25 07:20:35 erebus freshpager[29668]: Program[0] Triggered alert in PagerDuty for Aldaviva SMTP being down, got deduplication key 81e40b63294248c7bb43b07e6ceae4c9
Sep 25 07:20:35 erebus freshpager[29668]: Program[0] Triggered alert in PagerDuty for Aldaviva HTTP being down, got deduplication key f0cd8daed7b54a8494e4f45ba985e54e
Sep 25 07:20:35 erebus freshpager[29668]: Microsoft.AspNetCore.Http.Result.CreatedResult[1] Setting HTTP status code 201.
Sep 25 07:20:35 erebus freshpager[29668]: Microsoft.AspNetCore.Http.Result.CreatedResult[1] Setting HTTP status code 201.
Sep 25 07:23:48 erebus freshpager[29668]: Program[0] Aldaviva SMTP is available
Sep 25 07:23:48 erebus freshpager[29668]: Program[0] Aldaviva HTTP is available
Sep 25 07:23:48 erebus freshpager[29668]: Program[0] No known PagerDuty alerts for service Aldaviva HTTP, not resolving anything
Sep 25 07:23:48 erebus freshpager[29668]: Microsoft.AspNetCore.Http.Result.CreatedResult[1] Setting HTTP status code 201.
Sep 25 07:23:48 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.LogicalHandler[100] Start processing HTTP request POST https://events.pagerduty.com/v2/enqueue
Sep 25 07:23:48 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.ClientHandler[100] Sending HTTP request POST https://events.pagerduty.com/v2/enqueue
Sep 25 07:23:49 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.ClientHandler[101] Received HTTP response headers after 252.854ms - 202
Sep 25 07:23:49 erebus freshpager[29668]: System.Net.Http.HttpClient.Default.LogicalHandler[101] End processing HTTP request after 253.6792ms - 202
Sep 25 07:23:49 erebus freshpager[29668]: Program[0] Resolved PagerDuty alert for Aldaviva SMTP being up, using deduplication key f0cd8daed7b54a8494e4f45ba985e54e
Sep 25 07:23:49 erebus freshpager[29668]: Microsoft.AspNetCore.Http.Result.CreatedResult[1] Setting HTTP status code 201.

The problem is that Freshping only keeps track of at most one deduplication key per PagerDuty service, but each PagerDuty service can map to more than one FreshPing check, so it can't resolve more than one incident for a PagerDuty service.
Linearized execution

FreshPing check C1 goes down
FreshPager triggers an incident on PagerDuty service S1 and stores the deduplication key D1 under the integration key for S1
FreshPing check C2 goes down
FreshPager triggers an incident on PagerDuty service S1 and stores the deduplication key D2 under the integration key for S1, overwriting and erasing D1 (D1 incident can now never be automatically resolved)
FreshPing check C1 comes up
FreshPager looks up the deduplication key D2 based on the integration key for S1, which used to map to D1 but it got overwritten by D2 in step 4. PagerDuty incident for D2 is resolved, so it looks like C2 is up.
FreshPing check C2 comes up
FreshPager fails to look up any deduplication key for the S1 integration key, because D1 was removed in step 6 while resolving the D2 alert.
FreshPager never resolves the D1 alert, so D1 stays open until a human manually resolves it.

It's sort of a race when both services go down at the same time, otherwise they would have shared one deduplication key to start with. To resolve that, we could process requests serially, but each one takes about 3 seconds. Maybe the deduplication keys should be stored with different keys instead.

Trying indexing deduplication keys by Freshping check instead of by PagerDuty integration key. Deployed on two servers, will monitor for failures.


unfederating recipe

When uninstalling a federated ArcGIS server it leaves an unavailable server instance on the portal. It would be nice to have an "unfederating" recipe, that changes the Authentication tier and deletes the server instance on the portal.
Thanks.

Hello @thk70,
We have added an unfederate_server recipe in release v3.8.0:
https://github.com/Esri/arcgis-cookbook/wiki/arcgis‐enterprise-Cookbook-Recipes#unfederate_server
Thanks,
Cameron K.


Bump intl to 0.18.0

Connection with issue(s)

Close #???

Connected to #???
Solution description
Bumps version of intl to 0.18.0 to support Flutter 3.10.0
Screenshots or Videos

To Do

[x] Read contributing guide
[ ] Check the original issue to confirm it is fully satisfied
[ ] Add solution description to help guide reviewers
[ ] Add unit test to verify new or fixed behaviour
[ ] If apply, add documentation to code properties and package readme

CI fails due to issues with GitHub Actions, if Github Status indicates Actions is working again, please retry the CI steps

I see the workflow fails on using flutter commands instead of dart, I’ll update this PR in a bit

@deandreamatias PR has been updated, unless I'm missing something the CI should work now

When pass pipeline I will merge.
For next time, don't need change the version and changelog, I handle this but thanks for do it.


Does Frogbot support GHES

Does Frogbot support Github Enterprise Server?
It is not clear from the documentation and when I tried it (using the JF_GIT_API_ENDPOINT) , I got the following error:
Frogbot
/opt/hostedtoolcache/frogbot/[RELEASE]/x64/frogbot scan-repository
14:56:24 [Info] Frogbot version: 2.19.2
Error: 4 [Error] expected 200 status code while received 406 status code with error:
GET https:///login?return_to=https%3A%2F%2F%2Frepos%2FCICD-Infra%2Ftemp-call-xray-lib-wkfls%2Fcontents%2F.frogbot%3Fref%3Dmain: 406  []
Error: The process '/opt/hostedtoolcache/frogbot/[RELEASE]/x64/frogbot' failed with exit code 1

@esmaila,
Frogbot supports all flavors of GitHub (Enterprise / Cloud / Self-hosted). We will improve the documentation to clarify this.
Will you be able to enable debug logging by adding the JFROG_CLI_LOG_LEVEL variable to your workflow with DEBUG as its value, and then sharing the complete log file with us here?

Frogbot
/opt/hostedtoolcache/frogbot/[RELEASE]/x64/frogbot scan-repository
09:55:34 [Info] Frogbot version: 2.19.2
09:55:34 [Debug] Reading config from file system. Looking for .frogbot/frogbot-config.yml
09:55:34 [Debug] frogbot-config.yml wasn't found in /runner/_work/temp-call-xray-lib-wkfls/temp-call-xray-lib-wkfls/.frogbot/frogbot-config.yml. Searching for it in upstream directories
09:55:34 [Debug] Attempting to download frogbot-config.yml from CICD-Infra/temp-call-xray-lib-wkfls
09:55:34 [Debug] The frogbot-config.yml will be downloaded from main branch
Error: 4 [Error] expected 200 status code while received 406 status code with error:
GET https://git.build.ingka.ikea.com/login?return_to=https%3A%2F%2Fgit.build.ingka.ikea.com%2Frepos%2FCICD-Infra%2Ftemp-call-xray-lib-wkfls%2Fcontents%2F.frogbot%3Fref%3Dmain: 406  []
Error: The process '/opt/hostedtoolcache/frogbot/[RELEASE]/x64/frogbot' failed with exit code 1
I have a package.json at the root level and therefore this shouldn't be required?
It is an npm project

Hey @esmaila, Frogbot usually tries to grab the frogbot-config.yml file from your repository to read its settings, but it's not mandatory, as Frogbot can operate using a configuration based on environment variables. The issue with the failing REST API suggests a possible omission in your configuration, likely related to the endpoint, such as the absence of the /api/v3 suffix. Typically, the endpoint for the GitHub enterprise server appears in this format: http(s)://HOSTNAME/api/v3. Could it be that you're missing the /api/v3 suffix?


add universal models and tests

Add universal models and unit tests for the universal models. Created under @TaylorOshan 's supervision

I set up my own PR just by copying pasting the contents of your universal.py and test_ universal.py
It's still fails but I don't get any of the relative import errors that you are getting, it's actually just a simple rounding error.

I set up my own PR just by copying pasting the contents of your universal.py and test_ universal.py
It's still fails but I don't get any of the relative import errors that you are getting, it's actually just a simple rounding error.

@thoffman1 I can only notice two differences. First, the edits you made to gravity that you then backtracked are registering in an odd way, if the change is no longer exist then it should not say that there is any difference in the files. Second, for some reason it also lists Init Final as somehow having changed even though it's empty. It may just be easier to scrap this PR, create a fresh branch, copy and paste in the two sets in your files, and file a new PR.

@thoffman1 I can only notice two differences. First, the edits you made to gravity that you then backtracked are registering in an odd way, if the change is no longer exist then it should not say that there is any difference in the files. Second, for some reason it also lists Init Final as somehow having changed even though it's empty. It may just be easier to scrap this PR, create a fresh branch, copy and paste in the two sets in your files, and file a new PR.

@TaylorOshan Sounds good, I'll redo the fork and create a new PR!

@TaylorOshan Sounds good, I'll redo the fork and create a new PR!


cart prices not effected

When I am set volume range and  prices then we make a cart then the prices not changed according to range price

I've same problem 😕


Error compiling Javascript

I use chrome, and when I clicked RUN button following error occurs.

Hi @playerpxx, since Flutter 1.9, Flutter Web is included as part of the normal Flutter SDK/package, so you just use the normal package:flutter imports.
If you still have issues once adjusting to those imports, let us know :)


SonarKit podspec 0.6.13 needs to be pushed

0.6.13 has been tagged, but it needs to be pushed to the private cocoa pods repo (the Specs directory currently contains only 0.6.12)

Thanks! 0.6.13 has been pushed and this can now be closed.


Update Add-VpnConnectionTriggerApplication.md

Issue - "fill in the blank" assignments - Description has been updated now.

Docs Build status updates of commit 42b12c6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012r2-ps/vpnclient/Add-VpnConnectionTriggerApplication.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

The text should read "Forces the command to run without asking for user confirmation." per the product documentation.

Docs Build status updates of commit b4b4a00:
:clock10: Pending: waiting for processors (109 builds ahead of you)
:warning: Docs Build is busy, currently there are 109 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

The text should read "Forces the command to run without asking for user confirmation." per the product documentation.

Hi @tseward
Thank you for the Suggestion. Changes are incorporated in the PR. Kindly review. Thank you!

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

File
Status
Preview URL
Details

docset/winserver2012r2-ps/vpnclient/Add-VpnConnectionTriggerApplication.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Looks good!


Fixes storage permissions on API 29.

Issue
The runtime library is unable to write to the Downloads folder on Android 10 (API 29).
Android 10 is a bit of a special case in terms of scoped storage. It's the first version with scoped storage, but it doesn't have the direct file paths feature yet, introduced in Android 11, which re-enables the use of the Java File API.
Fix
This PR fixes this by setting requestLegacyExternalStorage to true and declaring the WRITE_EXTERNAL_STORAGE permission for API 29, so Dropshots can use the Java File API to write to the Downloads folder, or any other media folder.
Closes #32

 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.Jay Shortway seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I see that the testMatchesViewScreenshot test failed on CI, but I have difficulty finding any test output pointing towards what's wrong. I'm used to GitLab CI, so maybe I'm looking in the wrong place.
I have been searching through the CI artifact, and the only thing that stood out to me were these lines in the emulator's logcat:
Canceling events for d5e2d48 com.dropbox.dropshots.test/com.dropbox.dropshots.TestActivity (server) because it is unresponsive
...
Process 2448 exited due to signal 9 (Killed)

Is this a matter of the emulator acting up? Does anyone have any ideas? Thanks in advance.

@rharter Sounds good! Thanks for retriggering CI. Seems like it was indeed flakiness that caused the failure.


active file rename on F2 keypress

From @stushurik :
Idea: active file rename on F2 keypress, I always use F2 in other IDE and intuitively tried to do so in vizhub

Notes from the call:

This requires a refactoring where we move handleRenameIconClick to VZCodeContext, so that it can be made available in useKeyboardShortcuts.
We may need to also keep track of which file is being renamed.


Add integration for Weaver test framework

hello, thanks for this project.
With this PR we are adding support for the weaver framework and I followed the structure of the scalatest module.
I didn't open an issue before as I think this is a small change.

it looks like there is incompatibility between the scala 3 versions. would we be ok to upgrade it to 3.2.2 or even the LTS 3.3.0 one?
sorry I didn't realise I had to pass the 3 version for the compilation command before opening the PR

okay version updated 👍

and I had to update the expected error message in one of the tests, done in 0d1a59a


Update cloud-native-for-java-working-group-proposal.adoc

@kwsutter can you assign this to someone at EF?
even add them as reviewers and watchers of whatever activity is a +1. Thanks!

@kwsutter can you assign this to someone at EF?
even add them as reviewers and watchers of whatever activity is a +1. Thanks!

Hi Amelia,
I will let the EF know about this suggestion, but I don't see them agreeing to it.  Granted this proposal does originate from the EF, but there are some organizations and/or individuals in the MP space that may agree with the direction of a single WG.  To say that it is not "endorsed" by MP is an overstatement.  In the same light, I know there are aspects of the separate WG proposal that are acceptable by some people at the EF.  So, I would rather not get into a game of "MP proposal vs EF proposal".  That's counterproductive.  But, I will let them know about this issue and let them decide.  Thanks!

The requested addition is factually incorrect. The Eclipse Foundation drafted this proposal in collaboration with multiple individuals and organizations within the Eclipse MicroProfile community.
Further, the point of this exercise is to give the MicroProfile community the opportunity to express their opinions. Any statement that the community does not endorse this proposal is premature.

@waynebeaton Can you confirm that CN4J is intended to be more broad than just MicroProfile and Jakarta EE? I had a conference booth conversation with Mike and he mentioned that CN4J could incorporate additional cloud native java projects in the future.

@waynebeaton Can you confirm that CN4J is intended to be more broad than just MicroProfile and Jakarta EE? I had a conference booth conversation with Mike and he mentioned that CN4J could incorporate additional cloud native java projects in the future.

Confirmed. Subject to approval by the steering committee, of course.

Ack. Thx!

On Jan 7, 2020, at 10:30 AM, Wayne Beaton<EMAIL_ADDRESS>wrote:
@waynebeaton https://github.com/waynebeaton Can you confirm that CN4J is intended to be more broad than just MicroProfile and Jakarta EE? I had a conference booth conversation with Mike and he mentioned that CN4J could incorporate additional cloud native java projects in the future.
Confirmed. Subject to approval by the steering committee, of course.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/eclipse/microprofile-sandbox/pull/71?email_source=notifications&email_token=AAAR242G3HIOXZU7KXZH5DTQ4TC3TA5CNFSM4JZGRQSKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIJ2GEQ#issuecomment-571712274, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAR242EONWALTVIXIDNNXDQ4TC3TANCNFSM4JZGRQSA.


Enhancement: Use codecov for collecting test coverage

This PR

[x] uses codecov.io for collecting test coverage

Codecov Report

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

@@           Coverage Diff           @@
##             master   #211   +/-   ##
=======================================
  Coverage          ?   100%
  Complexity        ?     95
=======================================
  Files             ?     10
  Lines             ?    383
  Branches          ?      0
=======================================
  Hits              ?    383
  Misses            ?      0
  Partials          ?      0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update db4b962...c805956. Read the comment docs.


org-roam: Link nodes containing nodes

Consider a node, e.g. list.org, that contains other nodes, like so:
#+TITLE: List of Nodes
:PROPERTIES:
:ID: wfytuhwuyfthyuwft
:END:

* Item
  :PROPERTIES:
  :ID: some-other-id
  :END:

I would like the List of Nodes node to be linked to Item at the schema level. Currently I don't believe this is possible, since node linking relies explicitly on bona fide links, e.g. [[id:some-other-id][label]], which is not present here.
I'd like to extend Org-Roam such that nodes-containing-nodes are recorded as referencing their constituent nodes, and the former are correspondingly recorded as backrefs for the latter.
If there's a native solution for this already, all the better. It's worth asking in Org-Roam forums about this.
Notes

org-roam-reflinks-get might be useful reference here

Currently I don't believe this is possible, since node linking relies explicitly on bona fide links, e.g. [[id:some-other-id][label]], which is not present here.

This appears not to be the case, since the following block:
:PROPERTIES:
:ID:       17B638BE-78B1-41BA-ABE4-8D01FCF800DD
:END:
#+title: Bellman optimality equation

* Bellman update
  :PROPERTIES:
  :ID:       9CD58037-2C9E-4886-BA0F-4A8DD0B217F0
  :END:

  $$Q(s,a) = r(s, a) + \gamma \max_{a^\prime \in A}Q(s^\prime, a^\prime)$$

  In the case of [[id:ED1C15CD-A0D0-445B-A1F6-3C393282D635][Tabular Q-learning]], this is often implemented with a *learning
  rate* \alpha to mitigate against instability arising from simply clobbering
  pre-existing value with new one.

  $$Q(s,a) \leftarrow (1-\alpha)Q(s,a) + \alpha \left(r + \gamma \max_{\alpha^\prime \in A} Q(s^\prime, a^\prime)\right)$$

Evidently gets linked properly, according to the roam UI


add docs for heizou

Also update the checksum for our Prism repo since I was getting "The remote archive doesn't match the expected checksum"

checksum change seems to be causing tests to fail. not entirely sure why

@unleashurgeek can you please take a look at if yarn change ok?

Both the checksum and yarn config should remain unmodified (68f79 is the correct checksum). Unclear why first commit changed this checksum


nano: 2.5.3 -> 2.6.0

Motivation for this change
Update nano to the last version
Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[X] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

I don't see Nano 2.6.0 on http://ftp.gnu.org/gnu/nano/. According to the HN discussion at https://news.ycombinator.com/item?id=11953044, there is some forking situation going on. We should probably stick to the official (GNU) version until this has settled down.

Sorry I just did the pull request and after that I read that. I close it, better to wait what happen.
Thanks.

@mimadrid by the way, it seems to have been clarified by the original maintainers.


Create a settings page for Apinf configuration options

For organizations to host Apinf as a service, they need an easy way to configure the settings like mailgun. Create a settings page to hold current and future configuration options as well as a collection to store the configs.
Research

[x] Look for existing settings/ configuration pages in open-source Meteor projects
[x] Look for configuration page examples in other web applications
[x] Create a design document and list the solutions you found

Design

[x] Add a description of the configuration page to the design document
[x] Create one or more wireframes to demonstrate the intended design (do not skip this step)

Implement

[ ] Create a collection to store configuration settings
[ ] Create one or more templates to hold configuration forms
[ ] Create a schema to define configuration settings
[ ] Use AutoForm to build one or more configuration forms on the settings page(s)

Co-pilot: @brylie

Blocks #502.

@frenchbread You were interested in this task. I have assigned it to you and estimated the size according to your estimation.

@elnzv status report? Also, please provide wireframes before writing code.

Check out how the Dominus game (which is built with Meteor) handles Settings. It seems to have a settings collection, where each setting is stored as a separate document. Settings can then be retrieved using Settings.findOne({name: "settingName"});.

I wonder if this Settings is only related to frontend settings, OR also to Settings Object in ApiBackendSchema that I'm working in #503 and #504 ?
If the answer is Yes to latter one, I think we should co-operate with this one @elnzv

wireframes added

Would it be possible to move the Import button to the top, if the purpose is to populate the fields with settings from existing settings.json? Or should the fields be updated automatically, if there is a settings.json? Or is that file even needed any more?

@bajiat, yes, button is optional. If we still stay with settings.json file - the best approach is to check it on startup and update collection and form with new values.

I think we should plan for grouping, and possibly tabs. I.e. our settings will have natural groupings, such as email authentication, Github application settings, etc.. Not every group of settings will be relevant to all deployments.

Can't access ApiUmbrellaWeb object on startup.
I need to be able to define this somewhere outside the package.
@brylie has already started working on this.

@elnzv We are proposing the two interrelated settings tasks: 327 and 74 for you.

Here is how I think it should work:

if starting without settings.json and there is nothing in Settings collection - show form to enter needed values. Template with form can be rendered on startup before any sync. After submission meteor will look for required values in Settings collection,
if starting with settings.json file parse it and duplicate it to Settings collection (additionally we can show message that now user can edit his settings through UI and start without --settings flag);

Blocking: mailgun dependencies, user can't be registered without mailgun being configured.

Working branch: https://github.com/apinf/api-umbrella-dashboard/tree/feature/admin-settings-page
For now have already this UI for settings page:

TODO/in progress: divide settings into sections with help texts and placeholder examples.

@elnzv status update?

Added sections, new placeholders:

Added logic (formType) to choose "insert" or "update".

Now on startup if application starts without --settings it renders /settings:

if ( Settings.findOne() ) {
        this.next();
      } else {
        this.redirect('/settings');
        this.next();
      }

But still can't pause sync. Idea is somehow to wait after redirect and only when data is added continue with sync.

@elnzv where is your working code for this task? The 'settings' branch has a last commit 5 days ago. I am asking, because I just found the following in the SimpleSchema docs:

A basic schema key is just the name of the key (property) to expect in the objects that will be validated. If necessary, though, you can use string keys with mongo-style dot notation to validate nested arrays and objects.
For example:
MySchema = new SimpleSchema({
   "mailingAddress.street": {
      type: String
  },
   "mailingAddress.city": {
       type: String
   }
});

To indicate the presence of an array, use a $

Basically, you don't need $ in your schema, because that implies an array of objects. So, if you remove the $s, your settings data may validate.

@elnzv you may also be able to set blackbox: true on the settings objecs:

If you have a key with type Object, the properties of the object will be validated as well, so you must define all allowed properties in the schema. If this is not possible or you don't care to validate the object's properties, use the blackbox: true option to skip validation for everything within the object.


Print button should generate a dedicated gift page instead of printing the whole page

print is fixed but when it prints it as a pdf it overflows outside the page. this might be because the display is flex which need to be changed to block.


ref impl: CI + opnode Go base

Part of #119: staging -> main migration
This:

Updates linting docs with Go instructions
Adds Go instructions from CI
Defines the optimistic-specs go module with opnode package
Adds the necessary boilerplate to run the process, shut it down. A.k.a. opnode/cmd
Adds logging command options

Side-note: Not cherry-picking from staging, squashing noise + cleaning up tweaks

@norswap good point, will add a opnode/README.md as part of #132 (where we define the node options and actually gets interesting to run)


Fix serve command

From the Firebase docs, you must use --only in order to run the local function emulator.
See https://firebase.google.com/docs/functions/local-emulator

@jthegedus

@jonjamz Your fork is out of date. I've written comments about this not working and linked to relevant issues in the README - see current canary https://github.com/zeit/next.js/tree/canary/examples/with-firebase-hosting#run-firebase-locally-for-testing
I'll look at this again now and see if anything has changed.

Wow, the local function emulation actually works now! Cheers @jonjamz for bringing this to my attention :smile:
Interestingly it still checks online to see if a Firebase project with that name exists within your account, but it certainly runs local functions with hosting rewrites. And it runs without having to have the Functions already deployed :100:
@jonjamz Once you update to the latest canary the changes that need to occur are:
package.json
+ "preserve": "npm run build-all",
- "serve": "echo \"for details see:\n\thttps://github.com/firebase/firebase-tools/issues/535 \n\thttps://github.com/zeit/next.js/issues/3167\";",
+ "serve": "NODE_ENV=production firebase serve --only functions,hosting",
+ "build-all": "npm run build-public && npm run build-funcs && npm run build-app && npm run copy-deps && npm run install-deps",

``

Ha! I actually just did exactly what you did and came back here to write about it but you already had it figured out 😄. Glad it's all working!

@jonjamz You're still going to need to perform the update to the PR yourself, I don't have write access to your repo.

Fixed the suggestions you mentioned @jthegedus