[DOC] Italian translation

Italian translation of the README.md and RULES.md files.
Please let me know if there are some other missing documents that I have to update

@feross @Flet any change to get this reviewed?

Thanks @feross! I'll do my best


Interactive learning mode spams the nlu file with unneccesary entities and might even corrupt it

Rasa Core version: 0.13.2
Python version:  3.6.8
Operating system: windows
When exporting data from the interactive learning mode, rasa exports entities that are recognized by pretrained/non trainable extractors(duckling, spacy) to the nlu file and thus confuses extractors that rely on this data, like ner_crf
Example: [Nächsten Dienstag](MISC)[um 11 Uhr](time:2019-02-19T11:00:00.000+01:00)
These informations are not needed for spacy or duckling to improve extraction and only add unnecessary entities to ner_crf
Furthermore it might even corrupt the nlu file if the extractors find overlapping entities like this:
ich brauche [nächste woche freitag](time:2019-02-22T00:00:00.000+01:00)[nächste woche freitag](appointment_positive_preference) einen termin
Suggestions to solve this

Somehow annotate which extractor extracs the entity so others don't wrongly train on them
Don't write entities into the nlu file whichs extractors don't require training
Save the nlu file as json if there are overlapping entities (the json format seems to support them?)

Is this not a duplicate of #1728?

No, they both have similar origins (entities in interactive training mode) but have different consequences and different possible solutions.

@wochinge does your newly created issue cover this?

It does, this issue can be closed.


⚠ Orpheus has degraded performance

In 5c11012, Orpheus ($OPS) experienced degraded performance:

HTTP code: 200
Response time: 1262 ms

Resolved: Orpheus performance has improved in f9a3a1d after 6 minutes.


I want to implement stage1 in chatGPT. What should I do？

📚 The doc issue
How to implement supervised finetune in stage1？
ColossalAl Can it be realized？
What should I do specifically？

Hiii, thanks for your questions!

The model of stage 1 is actually an initial model, which can either be a pretrained model or SFT (supervised fine-tuning) model. The instructGPT paper indicates that they used the SFT model. Therefore, you could use a prompt dataset with label to fine-tune a GPT model with supervised learning in this stage. This model later will be used in sampling dataset for stage 2 by providing model outputs to prompts. For more implementation ideas, here is the guideline of instructGPT section 3 and C.1.

ColossalAI has provided GPT Parallel Training to reduce GPU resources and accelerate your training, you could refer this example to start your journel on using ColossalAI Train GPT with Colossal-AI.

If you want to go through the whole RLHF training in a quick view, it's betther to use pretrained model in stage 1 (since the initial model is not necessarily need to be SFT) and start Stage 2 & 3 with the ColossalAI example here. Otherwise, you can follow our GPT tutorial mentioned above to realize stage 1 or wait for our simple stage 1 example which is under preparation :)

This issue was closed due to inactivity. Thanks.


[WIP]paper update: results of user study

What about the following columns?

Participant background
Tasks solved without hints (Accuracy + some text)
Biggest difficulties
Notes (kind of like your highlights: put additional information that is relevant)

current table


[Feature]: Add 'Forgot Password' Option for Login Details Section

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
GSSOC'23
Describe the solution you'd like

Add Forgot Password Feature in Login Details Section.
In sometimes, User Missing Login Details like Mainly Password Details.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context

I want to work on this Issue. Because of, In every Login Page. Use Forgot Password Feature

Screenshot

I would like to contribute on this issue under GSSOC23


Infrastructure: bump vcpkg hash

Description of the Change
Update dependencies
Benefits
arm64 build
Possible Drawbacks
None

Previous jenkins before merge to trunck has failed. Restarted. Please check @lebdron

"boost::filesystem::status: Function not implemented: "/tmp"" seems to be CI issue as tests pass manually in the same Docker container.


Hide local selector for harvester in login and SideNav

Summary
Previous PR change miss to hide locale selector in SideNav for harvester, the locale selector still appears in login and SideNav within harvester-ui-extension standalone mode.
We need to have @rancher/shell : 3.0.1-rc2 release after this PR merge.

Fixes #
https://github.com/rancher/dashboard/issues/12347
https://github.com/harvester/harvester/issues/6928
Screenshot/Video
After fix (I test this PR change under harvester-ui-extension/node_modules/@rancher/shell)
Login page

SideNav

Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

@torchiaf , @aalves08 , e2e are all passed. Should be ready for review.

@torchiaf @a110605 @aalves08 Just an FYI, we need to move away from anything harvester related in the core dashboard code (there's an issue for this and it's a lot of tech debt). Ideally this feature should come in via the plugin API (IPlugin interface) rather than directly referencing isStandaloneHarvester and the single product name.
Shouldn't need to do anything now, but something to remember for next time

@richard-cox I agree, we have a tech-debt ticket for it https://github.com/rancher/dashboard/issues/12593
@nwmac  Should we tackle it in 2.11 or 2.12 ?


overlapping components with zIndex does not seem to work.

Environment
Environment:
  OS: macOS Sierra 10.12.6
  Node: 8.9.4
  Yarn: Not Found
  npm: 5.7.1
  Watchman: Not Found
  Xcode: Xcode 9.2 Build version 9C40b
  Android Studio: 3.0 AI-171.4443003

Packages: (wanted => installed)
  react: 16.2.0 => 16.2.0
  react-native: 0.52.0 => 0.52.0

Expected Behavior
Adding higher zIndex to a target component should over other components even if they are outside the parent container of the target component.
Actual Behavior
zIndex does not work.
Steps to Reproduce
https://snack.expo.io/@aprilmintacpineda/zindex-issue

I think expo uses the latest version of react-native so this bot is a bit odd not knowing that I added a link to reproduce it on expo.

@aprilmintacpineda expo sdk v25 uses react-native 0.52.0, and you also use that in your environment.
Try to upgrade react-native (https://facebook.github.io/react-native/docs/upgrading.html) and see if the issue persists

@larskarbo
I followed the link you gave me and used react-native-git-upgrade since I was concerned that other options might rewrite files I configured for the external libraries that I use.

I reinstalled the app on my iOS simulator.
the issue still persists.

The issue does not occur on android. (Tested via the expo snack above.)

On iOS, zIndex only works when Views are siblings of each other. Unfortunately what you're trying to achieve here is not supported. Ideally, it would work the same across iOS and Android, but we're not there yet.

@hramos so this is currently a limitation?

Neither. What you describe under Expected Behavior is not how zIndex works. It only affects the order of siblings.

https://github.com/facebook/react-native/issues/698 has additional discussion about this towards the end of the thread.


Documentation: Generate pulldown menu using sphinx toctree

The current content of pulldown menu is hard-coded which means it would be hard to maintain in the future. I take a look at Sphinx and found that it has a built-in function toctree() which provides the table of contents as a HTML list. So I think we can utilise this function to generate our pulldown menu.
I tried my best to make it look similar as before, here is what I got(still a bit different though):

The changes are tested on Chrome, Firefox and Safari, but not IE...
**  Some indentations look different from my text editor. If this idea is acceptable, I will keep improving them later.

@jim-parry I modified your codes of pulldown menu a bit. Could you take a look and give me some comments? :)

This looks a lot cleaner than the hack that Rufnex and I came up with!!
It works well, as far as I can tell :)

Just updated the codes and fixed the indentations. Hopefully it is better now :)

The "Contributing to CodeIgniter" section needs to be in the first column, not the last one.

@andrey I disagree with you. Hongyi's approach has generalized the pulldown menu, based on the sphinx doctree for our user guide. The pulldown menu layout was originally hand-coded to match the pulldown menu style in the CodeIgniter 2 user guide, which was hand/hard-coded too IIRC.
If the pulldown menu style isn't "suitable", I think the solution is to modify the TOC order in the user guide, not to program exceptions.

I'm not saying his approach is wrong.
I'm saying the hacky hard-coded pulldown menu displayed that section where we want it and this one does not. Whether that's due to the TOC itself being in the wrong order is a different question.
Btw, that @andrey guy is not me. :)

Oops re the tag.
What do you think of modifying the TOC order - would that not address the "consistency problem" without having to program an exception?

Works for me, yes. I'm not in favor of making logical exceptions either.

Should the TOC order be changed as part of this PR or a separate one?

Can be done separately after this is merged, yes. I just don't want us to forget about that ... will you do it?

Will do :)

@jim-parry thanks for taking over the doctree rearranging.
@narfbg thanks for cleaning the css.
And thank you all for your time to review it. :)


Ability to perform integration test on Jupyter notebooks

Summary
Ability to perform integration test on Jupyter notebooks
Part of #1218
Test Plan
cargo test

Current dependencies on/for this PR:

main

PR #5076   👈

This comment was auto-generated by Graphite.

I'll add some more tests now that we've the ability to do so!


[feature request]Respond to mouse hover of other panels

This is an amazing plugin(In addition, debug is a very neat and useful feature.). I love it. Thank you! @andymchugh
For embedded grafana's panel, there is a feature called shared xxxx in dashboard Settings.

Imagine if we move the mouse to a specific position (i.e. time) in other panels, it would be great if we could also move the time to that specific time in our plugin.
In fact, if we have this feature, then, https://github.com/andymchugh/andrewbmchugh-flow-panel/issues/42 this is not necessary. because we can build special dots in other panel very easily!
BTW, here is a quick search about this topic:

Access to core events & setting time range from a React ts panel - Grafana Plugin Development - Grafana Labs Community Forums --- the post is a bit older(2020)

Thanks for the feedback. This has been asked for once before in https://github.com/andymchugh/andrewbmchugh-flow-panel/issues/15#issuecomment-2062730610 but back then there was too much other change going on. I agree it would be good to add it. Will have a think...
I don't think it replaces the need for slider highlighting but it would certainly compliment it as a feature.

I don't think it replaces the need for slider highlighting

Thanks for the comments. I thought about it again and I think what you said is right.


Bug: create_instance CFG violation on ARM64EC due to WINRT_IMPL_LINK

Version
v2.0.230706.1
Summary
I am unsure whether the following is a cppwinrt bug or a compiler bug.
When compiling a program that targets ARM64EC (i.e., with MSVC 2022 17.6.5, with the ARM64 target compiler, with the /arm64EC compiler switch specified) and that has Control Flow Guard enforcement enabled (/guard:cf) and that program invokes WINRT_IMPL_CoCreateInstance, the program fastfails with a CFG violation:
 	ntdll.dll!RtlFailFast2()
 	ntdll.dll!#RtlpHandleInvalidUserCallTarget()
 	ntdll.dll!#LdrpHandleInvalidUserCallTargetEC()
 	repro.exe!winrt::impl::capture_to<IContextCallback,int (__cdecl*)(winrt::guid const &,void *,unsigned int,winrt::guid const &,void * *) noexcept,winrt::guid const &,void * &,unsigned int &>(void * * result, int(*)(const winrt::guid &, void *, unsigned int, const winrt::guid &, void * *) noexcept function, const winrt::guid & <args_0>, void * & <args_1>, unsigned int & <args_2>) Line 2357
 	repro.exe!winrt::capture<IContextCallback,int (__cdecl&)(winrt::guid const &,void *,unsigned int,winrt::guid const &,void * *) noexcept,winrt::guid const &,void * &,unsigned int &>(int(*)(const winrt::guid &, void *, unsigned int, const winrt::guid &, void * *) noexcept <args_0>, const winrt::guid & <args_1>, void * & <args_2>, unsigned int & <args_3>) Line 2596
>	repro.exe!winrt::create_instance<IContextCallback>(const winrt::guid & clsid, unsigned int context, void * outer) Line 6625
 	repro.exe!main() Line 10

This does not occur if the program targets normal ARM64 instead of ARM64EC and does not occur if the program is built without CFG.
I've managed to narrow this down to WINRT_IMPL_LINK's definition of the alternate name. The following program also crashes in a similar way:
#include <Windows.h>
#include <objbase.h>
#include <ctxtcall.h>

extern "C" HRESULT __stdcall MyCoCreateInstance(
    REFCLSID rclsid, LPUNKNOWN pUnkOuter,
    DWORD dwClsContext, REFIID riid, LPVOID* ppv);

#pragma comment(linker, "/alternatename:#MyCoCreateInstance=#CoCreateInstance")

int main()
{
    CoInitializeEx(nullptr, COINIT_MULTITHREADED);
    IContextCallback* ip;
    auto p = &MyCoCreateInstance;
    p(CLSID_ContextSwitcher, nullptr, CLSCTX_ALL, IID_PPV_ARGS(&ip));

}

(note that there's no significance to the specific COM CLSID and IID being used)
However, if instead the alternate name is changed to either:
#pragma comment(linker, "/alternatename:MyCoCreateInstance=#CoCreateInstance")

(i.e., without the leading hash in the first part, but with the hash in prefix for the actual import name) or:
#pragma comment(linker, "/alternatename:MyCoCreateInstance=CoCreateInstance")

(i.e., no hashes at all, identically to what WINRT_IMPL_LINK would normally do on x64) then the program compiles and links successfully, successfully invokes CoCreateInstance through the pointer and does not trigger a CFG violation on ARM64EC.
Reproducible example
#include <Unknwn.h>
#include <ctxtcall.h>
#include <winrt/base.h>

int main()
{
    winrt::init_apartment();
    auto p = winrt::create_instance<IContextCallback>(CLSID_ContextSwitcher);

    return 0;
}

Expected behavior
Programs using winrt::create_instance built for ARM64EC with CFG enabled should not crash with a CFG violation.
Actual behavior
Programs using winrt::create_instance built for ARM64EC with CFG enabled do crash with a CFG violation.
Additional comments
Compiler path:
C:\>where cl
C:\Program Files\Microsoft Visual Studio\2022\Community\VC\Tools\MSVC\14.36.32532\bin\HostARM64\arm64\cl.exe

Compiler command line for repros:
cl /Zi /EHsc /std:c++20 /permissive- /arm64EC /guard:cf repro.cpp ole32.lib

Confirmed this is a known bug. The (internal) reference is DevDiv bug 1846207.

Thanks @kennykerr!

By the way, thanks for the clear repro. That helped triage a similar active bug internally.


I am getting java.sql.SQLFeatureNotSupportedException and I can not see the query transaction.

Describe the bug
I am running APM-agent-java as an agent to my jar file. I am using the JDBC Clickhouse driver to reach my Clickhouse Database. I work with Clojure to do SQL queries into my Database. To create a connection pool to my database I use Hikari CP. My service works perfectly and it reaches the database with no problem, however, the logs of my services are getting out of hand and creating noise in my Filebeat logs.
Steps to reproduce
Steps to reproduce the behavior:

Use this config jdbc:clickhouse//localhost:9999
Then call 'SELECT first_name, last_name, email FROM customer WHERE _id='1234ABC''
Then do return the data to the UI
See error

Expected behavior
I am expecting to see the SQL query in the APM transaction and not seeing the logs about the SQL feature not supported
Debug logs
2020-02-07 17:59:07.919 [XNIO-2 task-7] WARN co.elastic.apm.agent.jdbc.helper.JdbcHelperImpl - Ignored exception
java.sql.SQLFeatureNotSupportedException
	at com.github.housepower.jdbc.wrapper.SQLConnection.getMetaData(SQLConnection.java:69)
	at co.elastic.apm.agent.jdbc.helper.JdbcHelperImpl.getConnectionMetaData(JdbcHelperImpl.java:124)
	at co.elastic.apm.agent.jdbc.helper.JdbcHelperImpl.createJdbcSpan(JdbcHelperImpl.java:74)
	at com.github.housepower.jdbc.statement.ClickHouseStatement.execute(ClickHouseStatement.java:29)
	at com.zaxxer.hikari.pool.PoolBase.isConnectionAlive(PoolBase.java:164)
	at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:185)
	at com.zaxxer.hikari.pool.HikariPool.getConnection(HikariPool.java:161)
	at com.zaxxer.hikari.HikariDataSource.getConnection(HikariDataSource.java:100)
	at sun.reflect.GeneratedMethodAccessor16.invoke(Unknown Source)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at clojure.lang.Reflector.invokeMatchingMethod(Reflector.java:167)
	at clojure.lang.Reflector.invokeNoArgInstanceMember(Reflector.java:438)

I know that using Clickhouse and an open-source adapter that could potentially cause this issue. However, is there a way to turn this off at least in the logs or fix it?

Hi @stLalo , thanks a lot for reporting this.
In this case, apart from the verbose error logging, you should not have have any other side-effects on the application itself, it only affects what the agent is able to monitor at JDBC level. Can you please confirm that ?
We recently had a very similar error with a JDBC driver that does not implement Statement.getUpdateCount() and it has been fixed with https://github.com/elastic/apm-agent-java/pull/1008/ (not yet released, but likely in a few days), thus I feel that fixing this in a very similar way would be a good option to prevent having those recurring error messages every time the agent tries to get connection metadata.

@SylvainJuge Indeed, there are no side-effects in the actual application, per se. It works fine and there are still some APM metrics sent to Elasticsearch.

I found this on my logs today

Thanks for this update.
Agent keeps track of which span is currently active on current thread, most of the time a span is kept active for it's whole duration (while SQL query is executing for example), and thus activated when starting span, and deactivated when ending span.
However, if there is any bug in the agent plugins, like in your case an unexpected exception has been thrown while using JDBC instrumentation, this state is not consistent. In other words, it's very likely a side effect of the issue you've already reported.

let's make sure we deactivate and end the span in a finally block


[TC-77] Docker files need small updates

Docker files need some minor updates.
1) Traffic Ops: centos question is different (centos72.), and some other minor things
2) riak needs search installed : http://trafficcontrol.apache.org/docs/latest/admin/traffic_vault.html#configuring-riak-search
I will try to create a PR once I'm done w 1.8 testing, there may be more.
Author: Jan van Doorn
JIRA Link: https://issues.apache.org/jira/browse/TC-77
Found Version: 1.8.0

@dangogh has taken full control of docker, this is old.


feat: 🎸 add asset required mediators functions

Description
add create, get, remove functions for asset required mediators
Breaking Changes
None
JIRA Link
✅ Closes: DA-1019
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 24.0.0-alpha.8 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 24.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add viridian as an option

https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>1666v1.full.pdf
https://github.com/iqbal-lab-org/viridian

Will revisit when it works for ANY amplicon schema


Coinbase error in streaming trades

Since the new release, it fails to stream trades, at least from Coinbase that I tried.
When we run:
int main(int argc, char **argv) {
  using namespace ccapi;  // NOLINT(build/namespaces)
  SessionOptions sessionOptions;
  SessionConfigs sessionConfigs;
  MyEventHandler eventHandler;
  Session session(sessionOptions, sessionConfigs, &eventHandler);
  std::vector<Subscription> subscriptionList;
  Subscription subscription("coinbase", "BTC-USD", CCAPI_TRADE);
  subscriptionList.push_back(subscription);
  session.subscribe(subscriptionList);
  std::this_thread::sleep_for(std::chrono::seconds(100));
  session.stop();
  std::cout << "Bye" << std::endl;
  return EXIT_SUCCESS;
}

I enabled logging to trace, and it outputs:
Event [type = SESSION_STATUS, messageList = [ Message [type = SESSION_CONNECTION_UP, recapType = UNKNOWN, time = 1970-01-01T00:00:00.000000000Z, timeReceived = 2021-02-01T19:46:49.768657000Z, elementList = [ Element [nameValueMap = {CONNECTION=WsConnection [id = 0x7ffff00020c0, url = wss://ws-feed.pro.coinbase.com, instrumentGroup = wss://ws-feed.pro.coinbase.com|TRADE|CONFLATE_GRACE_PERIOD_MILLISECONDS=-1&CONFLATE_INTERVAL_MILLISECONDS=0&MARKET_DEPTH_MAX=1&MARKET_DEPTH_RETURN_UPDATE=0, subscriptionList = [ Subscription [exchange = coinbase, instrument = BTC-USD, field = TRADE, optionMap = {CONFLATE_GRACE_PERIOD_MILLISECONDS=-1, CONFLATE_INTERVAL_MILLISECONDS=0, MARKET_DEPTH_MAX=1, MARKET_DEPTH_RETURN_UPDATE=0}, correlationId = fa37JncCHryDsbzayy4cBWDxS22JjzhM, credential = {}, serviceName = market_data] ], status = OPEN]}] ], correlationIdList = [  ]] ]]

Event [type = SUBSCRIPTION_STATUS, messageList = [ Message [type = ERROR, recapType = UNKNOWN, time = 2021-02-01T19:46:49.841734900Z, timeReceived = 2021-02-01T19:46:49.841734900Z, elementList = [ Element [nameValueMap = {ERROR_MESSAGE=rapidjson internal assertion failure}] ], correlationIdList = [  ]] ]]

Event [type = SUBSCRIPTION_STATUS, messageList = [ Message [type = ERROR, recapType = UNKNOWN, time = 2021-02-01T19:46:51.045225200Z, timeReceived = 2021-02-01T19:46:51.045225200Z, elementList = [ Element [nameValueMap = {ERROR_MESSAGE=rapidjson internal assertion failure}] ], correlationIdList = [  ]] ]]

and continues with the same error message. It points to {ERROR_MESSAGE=rapidjson internal assertion failure}.
Please let me know if this issue can be replicated on your machine. Thank you.

Great. Thank you.

Sorry for the accidental typo.


Bug: Bug with new 0.8.0 'DynamicNodePool.Name' Error:Field validation for 'Name' failed on the 'max' tag

with this config
apiVersion: claudie.io/v1beta1
kind: InputManifest
metadata:
  name: sima-k8s-04
  namespace: default
  labels:
    app.kubernetes.io/part-of: claudie
spec:
  providers:
    - name: hetzner-1
      providerType: hetzner
      secretRef:
        name: hetzner-secret-cloud
        namespace: default

  nodePools:
    dynamic:
      - name: control-hetzner
        providerSpec:
          # Name of the provider instance.
          name: hetzner-1
          # Region of the nodepool.
          region: fsn1
          # Datacenter of the nodepool.
          zone: fsn1-dc14
        count: 1
        # Machine type name.
        serverType: cpx11
        # OS image name.
        image: ubuntu-22.04
    static:
      - name: hertzner-1
        nodes:
          - endpoint: "<IP_ADDRESS>"
            secretRef:
              name: hertzner-key-01
              namespace: default

        labels:
          datacenter: hertzner-1
        annotations:
          node.longhorn.io/default-node-tags: '["hertzner-1"]'

 clusters.
  kubernetes:
    clusters:
      - name: dev-cluster
        version: v1.27.3
        network: <IP_ADDRESS>/24
        pools:
          control:
            - control-hetzner
          compute:
            - hertzner-1

I am getting
Error from server (Forbidden): error when creating "c.yaml": admission webhook "validate-manifest.claudie.io" denied the request: failed to validate nodepools section inside manifest: failed to validate DynamicNodePool "control-hetzner": Key: 'DynamicNodePool.Name' Error:Field validation for 'Name' failed on the 'max' tag

Did not happen with the previous Claudie version.
Thanks!

Hi, we had to put a limit on nodepool names (14 characters) and cluster names (28 characters) in the manifest. Since control-hetzner is 15 characters long you got this error.

Let's check if we can make the errors more descriptive.

@bernardhalas @Despire Hi all! Is it possible to rename existing node pools that has invalid name after update to 0.8.0 without cluster recreation?

Hi @vatroshkin , if the nodepool is dynamic: Add a new nodepool with the correct name (to have sufficient capacity in the cluster). Then remove the old nodepool.
If the nodepool you're talking about is static: Add a new dynamic nodepool (to have sufficient capacity in the cluster). Then remove the wrong static nodepool. Then re-add the static nodepool, now with correct name.
Does this help?

@bernardhalas thanks for reply, I have tried that, but the problem that validation complains about name length for an old nodepool 🥲 - so I can't add a new nodepool with a short name

If it's possibly you could deploy the older version where there was no restriction yet.
This way you could performer the changes @bernardhalas described. and then You should be able to upgrade back to 0.8.

@Despire Hi! Any suggestion how can I do that? When I apply previous release (kubectl apply -f ...), it stays 0.8.0. Thanks!

applying just kubectl apply -f https://github.com/berops/claudie/releases/download/v0.7.5/claudie.yaml should work (just tested it by deploying 0.8.0 then applying the command)


Fix Lint Error: No Coding Skills Needed - Good First Issue

When you first open the project in the Browser, if you check the console you will see several Lint Errors.

Ex:
Warning: Invalid DOM property clip-rule. Did you mean clipRule?

If you continue to read, it will also map you to the element's location. Find the element and fix the error by using the did you mean property.

If you think you can take on this task, please comment below to be assigned to it. Read the contributing.md file and open PRs to the DEVELOPMENT branch, not main.
Thank you and happy Collab! =)

Shall I work on this?

@valanm22 Its all yours! =)
Please read contributing.md
If you need help or have any questions, add them here.
Thank you and Happy Collab!!!!


Optimised code in filemover.py

Changed extensions list of type list into type set for faster lookup.

have you tested the code?

Yes. It seems to be working fine.

Thanks for contributing to the repository. Looking more contribution in future.
Join Py-Contributors Community
Discord  • Telegram  • website
Please share your Hacktober Experience on Discord Channel to Encourage more member to contribute to open source.
Happy Coding


feat(agents): add env var overrides for agent configs

Agent config should be improved to speed up deployments. Currently most config changes have to go through a PR->deploy cycle. Configs should be updatable after deployment
Solution
Agent config should be overridable with env vars
PR Checklist

[x] Added Tests
[ ] Updated Documentation N/A
[x] Updated CHANGELOG.md for the appropriate package

@ltchang2019 I'm pretty sure there are no in-repo docs that have to get updated for this. If I missed anything lmk.

Also we're not calling load_env_overrides anywhere right now. We want to load env overrides on agent config in nomad-base/src/settings/macros.rs below this line. That way everytime agent loads up settings, it tries to override using env vars.

Also we're not calling load_env_overrides anywhere right now. We want to load env overrides on agent config in nomad-base/src/settings/macros.rs below this line. That way everytime agent loads up settings, it tries to override using env vars.

It's called here:
https://github.com/nomad-xyz/rust/blob/115b4b56ac11858124a827fb7df4b07b3020601d/nomad-base/src/settings/macros.rs#L110
The passing integration test is here:
https://github.com/nomad-xyz/rust/blob/cc0e3852f389f13f160b45ab0cd38fd28ed3dee8/agents/kathy/src/settings.rs#L21


[Feature Request] Add support to multiple module loaders

I tried to use jquery-resizable with node.js but it does not recognize jQuery as a global insize jquery-resizable (because it uses the CommonJS approach to include a dependency).
Since that CommonJS and AMD include dependencies in a different way, I adapted jquery-resizable to work with them and created a pull request.

Thanks for the PR  d8ebdf2.


Adding Kensington PowerPointer support

While playing around with the Kensington PowerPointer, I found out that Projecteur now supports way more than what it did in the early days! 🎉
I started a branch for the Kensington PowerPointer, but noticed you wanted it discussed before a PR.
So, here is the discussion issue.
Maybe one point to discuss:
Activation of spotlight?
Right now, any motion of the device (when in "laser pointing" mode) activates the spotlight. I currently have a Projecteur rule that pressing the "laser activation" button toggles the spotlight mode.
The user experience in PowerPoint for the device is that the PowerPoint built-in "laser" pointer appears on screen while the "laser activation" button is pressed. I think that would be the right way of handling it.
Where would such a handling best be added?
(For context: If I remember the Logitech behavior correctly, it only sends relative movement events while the button is pressed. In the "laser pointing mode", sends these events on every orientation change. As this mode is activated by one of the awkward middle positions of a four-position sliding switch, activating/deactivating the mode on the fly is not something you want to be doing continuously while giving a presentation. The "movement-disabled" active position is the other middle position, so switching between them requires some concentration on the fine motor skills…)

Hello Marcel,
thank you for this, for supporting the device out of the box feel free to create a PR!
For the other topic about the activation of the spotlight:
I never used the "laser pointing mode" of the Logitech Spotlight, wasn't even aware of it.
I for myself only want the spotlight shown and "movable" when pressing a button, so currently the
activation of the spotlight on move events works well at least with the Spotlight and some cheaper
imitations I own.
How does that work with the Kensington device? Is the device sending mouse move events all the time?
I think there is surely a possibility to integrate a mode like that, but I would need a better understanding
of the workflow and what the device is sending exactly.

There are two useful modes for the Kensington (besides "off" and "show battery level"):

Dumb presentation: No mouse events are ever sent; only forward/backward and screen blanking is supported
Highlight mode: Mouse events are sent on every movement (i.e., always, if someone holds it in their hand). Pressing/holding the buttons will trigger either drawing commands (for annotation) or the "PowerPoint laser simulation" (where essentially the mouse pointer is replaced by a red dot)

For a reasonable UX with this device, it would be great to have the spotlight only appear if both of the following conditions are met:

The mouse moves and
Ctrl and LeftMouse were triggered

The Ctrl+LeftMouse-Combo is kind of strange. It does trigger the "laser" effect, but does not trigger the Context Menu on a Mac. Ctrl is only briefly held down when the left mouse button is initially pressed; it is released immediately afterward. This activates the "Laser" pointer on PowerPoint, which stays active until the release of the mouse button.
What would be the best way of supporting this:

[ ] have an "automatic" mode, where for this device ID, the spotlight is only active when Ctrl+Mouse1 are held down, or
[ ] add a button "Enable Spotlight on Control+Mouse1, PowerPoint compatibility" (or similar)?

I just noticed that when in "Dumb mode", Projecteur works like a charm, when I long-press the "laser pointer" button: Movement events will only be sent while the button is pressed (and the initial Ctrl-Mouse1-Ctrl combo is conveniently filtered by Projecteur).
So recognizing this sequence was just significantly lowered in my personal priority…


Dev

 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.:x: prernarai sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


[QUESTION] How long does a query stay cached?

How long does a standard query on 5.0.0 stay cached and can this value be controlled trough the GraphQL Server response?

forever)
There is no cache invlidation mechanisms built in
The default behaviour is to run query -> store data in cache -> on query re-run take data from cache -> execute real query behind the scene -> update cache with new data -> update UI
But the real behaviour depends on fetchPolicy


Add mongo-cli types to src/types and add to tsconfig paths

Ask and ye shall receive!

 Add a script for fully setting up a local db with validation, indices, and more sample data


Right arrow wraps below when the window has scrollbar removed by dynamic css

when a window contains scrolling tabs and another element that is resized with css rules such as :
max-height: ~"calc(100vh - 600px)";
which causes the window to gain or lose its scrollbar, the right arrow wraps below and breaks the layout.
this is a repeat of issue 46, https://github.com/mikejacobson/angular-bootstrap-scrolling-tabs/issues/46

For the element that's being resized, is the css rule that's resizing it being applied programmatically or is the rule always present and the trouble occurs when the window is resized?
If the resizing is happening programmatically, so a window resize event is not triggered, then you'll need to hook into the refresh-on attribute.
If, however, the problem occurs on window resize, then I'll need to look into the resize handler. I was able to reproduce it by resizing the window, but I don't want to mess with that code if that's not what's causing your issue.

Its a css rule. not programmatically in js.
I should have been more precise. the rule i gave as an example :  *max-height: ~"calc(100vh - 600px)"; * is typical of the body of a modal which tries to have a footer stay in view at the botom of the screen by restricting the height of the center part (100% of the view minus 600 pixels.)
I'll try to make a fiddle or some other online example.

Ah, ok. Thanks for the clarification.
Yeah, if you could put together a fiddle or something similar, that would be very helpful.


Any examples of DataGrid usage?

I'm currently trying to use the DataGrid control and can't find any examples or documentation on its usage with this library.
When I declare a plain DataGrid and give it an ItemsSource, it works fine: it renders the data with column headers that are auto-chosen based on the property names. But when I set AutoGenerateColumns to false and then add template columns, the AutoGenerateColumns change shows up in the .xaml.cs file but the actual column definitions don't. So, are there any examples that you provide on how to do this? I think this would be a good addition to this "UI_Examples" repo.

Hi, ideally it should be the same like in WPF, but DataGrid does not support custom columns at this moment. I created it just for my internal tool and didn't need that feature yet. I will add that to my backlog http://emptykeys.com/Community/aft/12

Custom columns are implemented in next version (release Monday 1st June) , including simple example. Closing this.


Handle reversal of analyses applied via vulnerability policy

Description

This PR implements the following behaviors:

When a finding has an analysis that was previously applied by a policy, but is no longer covered by any policy, the analysis is reverted entirely
When a policy is deleted (e.g. during reconciliation with bundle), all analyses created by it are reverted

In all cases, entries are added to the respective audit trails:

Additionally, some inconsistencies in the format of analysis comments have been addressed. The format used by policies was slightly different than that used by REST API interactions. The formatting logic is now centralized, so these inconsistencies are less likely to happen again.
Addressed Issue

Closes https://github.com/DependencyTrack/hyades/issues/994
Additional Details

Updated e2e test: https://github.com/DependencyTrack/hyades/pull/996
Checklist

[x] I have read and understand the contributing guidelines
~This PR fixes a defect, and I have provided tests to verify that the fix is effective~
[x] This PR implements an enhancement, and I have provided tests to verify that it works as intended
[x] This PR introduces changes to the database model, and I have added corresponding update logic
~This PR introduces new or alters existing behavior, and I have updated the documentation accordingly~

@VithikaS Thanks for the review! I had to resolve a merge conflict, could you take another look please?


Helm3 unable to template kuberhealthy chart

Describe the bug
Helm3 runs into an error when trying to template the stable kuberhealthy chart. The issue seems to come from the customresourcedefinition.yaml
In the following examples helm calls Helm2 and helm3 calls Helm3.
helm template stable/kuberhealthy/ > helm2-kh.yaml
helm3 template stable/kuberhealthy/ > helm3-kh.yaml
Error: apiVersion "apiextensions.k8s.io/v1beta1" in kuberhealthy/templates/customresourcedefinition.yaml is not available

Steps To Reproduce

Have a Helm3 release installed.
Have a copy of stable Helm3 charts.
Try to run Helm3 template subcommand on kuberhealthy chart

Expected behavior
I am expecting for Helm3 to render chart templates normally without any errors.
It seems the the error is coming from the CRD.yaml; however, the stable external-dns also has a CRD.yaml template using the same "apiextensions.k8s.io/v1beta1" apiVersion and can be templated fine.
helm3 template stable/external-dns/ > helm3-ext-dns.yaml
helm3 template stable/kuberhealthy/ > helm3-kh.yaml
Error: apiVersion "apiextensions.k8s.io/v1beta1" in kuberhealthy/templates/customresourcedefinition.yaml is not available

Screenshots

Versions

Kubernetes Client Version: 1.15.3
Kubernetes Version: 1.11.7
Stable Kuberhealthy Helm chart on master branch
Helm2 Version: 2.14.3
Helm3 Version: 3.0.0-beta.3

Additional context
Helm2 does not run into any errors when rendering templates for stable/kuberhealthy and stable/external-dns.
Helm3 does not run into any errors when rendering templates for stable/external-dns, but does when rendering for stable/kuberhealthy

As part of fixing this, we can probably bring our chart out of the helm/charts/stable repository and into our own.  Helm 3 has the chart museum these days.

we should just rewrite the helm chart for kuberhealthy 2 and helm 3

Replaced by #111


Fix lookahead for optional variables

Closes #347

Checks failing due to nightly ordering issue, will be fixed in a separate pr.


Question/Inquiry

While I use the TrueRNG module in my foundry VTT games, I know that not everybody does.
I'm in the process of building a module of PF1 enhancements, and rather than using the TrueRNG randomize function, what is the best and/or most appropriate "core foundry vtt" function that will work with TrueRNG (when it's installed) and without (when it is not)?

for context, this is the function I'm using (directly against TrueRNG) to simulate a d20 roll.
Math.floor(TrueRNG.GetRandomNumber() * 20) + 1

Found the answer I was looking for from the foundry discord...
let dieRoll = await new Roll('1d20').roll()


Create "Get involved" page

5 sections:
Swipe
Links to download the app, and a short blurb
For organisations
Any org can request help with mapping a project in mapswipe.
How to request, and links to forms/resources
For developers
links to github.com/mapswipe , the API, and a short blurb on how to help
For translators
Link to https://www.transifex.com/mapswipe/mapswipe-app/ where you can volunteer to translate the app (or this website) in your own language.
For socialising: Events
Link to the events page, where people can choose to join.
Sub ticket of #17

There are two GoogleDocs on following topics:

"For MapSwipe requesters"

Requesting a MapSwipe mission
Following the swiping progress
Downloading and using MapSwipe data

"For MapSwipe managers"

Managing MapSwipe missions
Troubleshooting
Sending in-app notifications

Those could be good content drafts.

Mostly done as part of new website. Refined scope for translation #27


Ending hour error

User of posterr under docker, I discovered a problem with the end time announcement, in fact the announced time is two hours off from what it really is.
I imagine this is probably due to a wrong time zone setting. how to remedy this

Are you setting the TZ value for the correct time zone?
Check it [HERE](wikipedia](https://en.wikipedia.org/wiki/List_of_tz_database_time_zones)


ENYO-3163: Fix Panels breadcrumb spotlight issue

Issue
When setting the index of a Panels control (in create time) to > 1, spotlight cannot focus on breadcrumbs, until you transition to another panel.
Fix
Previous method required the breadcrumbs and their container to be rendered to calculate if they were spottable. We can use an alternate method - since we know how many breadcrumbs there are and how many can fit on the screen at a given time, we can calculate which breadcrumbs are spottable.
Enyo-DCO-1.1-Signed-off-by: Jeremy Thomas<EMAIL_ADDRESS>

Closed in favor of https://github.com/enyojs/moonstone/pull/2776.


ImportError: cannot import name 'V1alpha2TensorRTSpec'

Notebook
02_01_fairing_introduction.ipynb
Describe the bug
ImportError: cannot import name 'V1alpha2TensorRTSpec'
Expected behavior
Not to error
Environment (please complete the following information):
Related to this: https://github.com/kubeflow/fairing/issues/521
Workaround:
Use kfserving==<IP_ADDRESS> while ^^ issue is being fixed.
!pip uninstall kfserving —yes
!pip install -Iv kfserving==<IP_ADDRESS>
!pip show kfserving

@tommcm1200 Thanks for reporting the issue. I will have a check on the python library.
to mitigate your issue, please try images below. They are aws built images and I tested they should work well.
      -<PHONE_NUMBER>40.dkr.ecr.us-west-2.amazonaws.com/tensorflow-1.15.2-notebook-cpu:1.0.0
      -<PHONE_NUMBER>40.dkr.ecr.us-west-2.amazonaws.com/tensorflow-1.15.2-notebook-gpu:1.0.0
      -<PHONE_NUMBER>40.dkr.ecr.us-west-2.amazonaws.com/tensorflow-2.1.0-notebook-cpu:1.0.0
      -<PHONE_NUMBER>40.dkr.ecr.us-west-2.amazonaws.com/tensorflow-2.1.0-notebook-gpu:1.0.0


Fix illegal reflective access operation has occurred (Java 9+)

The ExternalJarUtil, used to load database drivers (and maybe script engines) needs to make an illegal reflection to UrlClassLoader#addURL function, we need to find a workaround for this before the illegal reflections are denied

Here's an idea for a possible solution: Add the driver adaptors in an external Jar (maybe using the Service system or something similar) then load them using a custom ClassLoader that extends the URLClassLoader class and redefines addURL (it's protected).
This would let us manage our own ClassLoader, loading the driver JARs without any reflection at all.


Several problems

Hi,
I write from Europe.
I updated the add-on to the latest version (the one with the dubug button).
I use the windows 10 operating system and as a browser google chrome updated to the latest version available.
I have been using this add-on for several months and have never encountered any major problems.
However, in the last few weeks I have encountered some problems:

After completing all the daily daily searches (from Mobile and desktop PC) and the daily quests, the counter remained yellow indicating a negative value. In the first half of December -100 and in the second half of December -50. This problem has been encountered with the 100-point or 150-point quest which must be completed within a certain period (usually 3-4 weeks).
A similar problem with the counter occurred when the quest of "this and that" was not completed correctly (quest where you have to choose between two alternatives 10 times and receive 5 points for each correct answer).
Ex: if the correct answers were 9 out of 10, the add-on's wave remained yellow indicating a value of 5. The "this and that" quest remains available for about a week and the add-on icon remains for the entire period yellow indicating, in the example, the value 5.
For about a week the add-on has not been able to process daily searches, initially only "searches from PC" and then also "searches from mobile devices" and "microsoft edge bonus".
The add-on icon remains blue and you may encounter the following problems:

does not allow you to carry out daily quests that involve user interaction such as "bonus quizzes" or "daily surveys". With the add-on active by opening the hyperlink, the pop-up does not appear at the bottom where you can click. At the moment the add-on is blocked it is possible to complete the daily quests.
As long as the add-on remains active it is not possible to search through the bing search engine, you get the following error: "Ref A: [...] Ref B: [...] Ref C: [...] ". These are however counted by the MSR as searches from PC.

I don't speak English well and I don't know if the names used in Europe to indicate the different activities to be carried out to get the points are the same as those used in the USA.
Thanks for the work done.
Happy Holidays

Hello,
1, please use newest version and post the debug info in #23
2, Updated version should resolve it, I'm working on getting it to be more futureproof in #22


populated model is not updated

This is my model definition
schema = mongoose.Schema
  name: String
  parents: [{type: mongoose.Schema.Types.ObjectId, ref: 'Category'}]
  children: [{type: mongoose.Schema.Types.ObjectId, ref: 'Category'}]

model = mongoose.model('Category', schema)

model.formage =
  is_single: false
  list_populate: ['parents', 'children']
  search: ['parents', 'name']
  list: ['name', 'parents', 'children']

module.exports = model

I have created a "root" category and a "subcategory". The subcategory has "root" as one of its parents. If i change the name of the "root" category to "root name changed", the name is not updated in the populated models. Here are 2 sshots
name changed

populated values

you need to have toString method on the populated Schemas:
schema.methods.toString = function(){
    return this.name;
};


2FOC test procedure needs to be updated

Since there are some of the instruction of the 2FOC test procedure that are outdated we need to update them. This won't block the test but some of the instructions can be misleading to any tester so we need to clarify them well.
cc: @valegagge @fgarini

@MSECode about the PR is there any overlap or duplicate file to manage in the non-legay repo? im referring to https://github.com/icub-tech-iit/electronics-boards/tree/master/2foc/sw

Actually yes, but it is just for giving more details to the tester. Basically, when I show in the test procedure how to check which version of the 2foc you are using, I've also add a link to the logic. Therefore, if needed, one can check exactly if the used version of the 2foc is mounting or not the CAN-H/CAN-L resistors. So yes, we should pay attention to changing in the folders of electronics-boards if they might happen in the future.

Updates needed are ready. In order to move the related PR to ready I'll wait for the test equipment to arrive from the supplier to test if the modifications done are fine.


Error on users.getFollowedChannels

When I use twitchClient.users.getFollowedChannels('ninja'), I get UnhandledPromiseRejectionWarning: StatusCodeError: 400 - {"error":"Bad Request","status":400,"message":"The parameter \"user_id\" was malformed: the value must match the regular expression /^[0-9]*$/"}.

Can this package be changed to accept names too? So if it can't parse user
as an integer, then just translate the username to an ID. And if no user ID
is specified, then use the one associated with the OAuth token that's
currently in use.
On Mon, Jun 18, 2018 at 1:22 AM Daniel Fischer<EMAIL_ADDRESS>wrote:

Closed #16 https://github.com/d-fischer/twitch/issues/16.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/d-fischer/twitch/issues/16#event-1685353341, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAU_hKxoNkuQVsmSf-w_qre6N8GJQ-BUks5t9zj6gaJpZM4UrErE
.


Create a FSharp.Tools NuGet package - closes #408

what else do we need?
/cc @dsyme @granicz

Well, you need FSharp.Compiler.dll, FSharp.Compiler.Server.Shared.dll, FSharp.Compiler.InteractiveSettings.dll, fsiAnyCpu.exe. I also think it's correct to add FSharp.Build.dll and Microsoft.FSharp.Targets - in theory they feel as if they could be in a separate package, but I believe FSharp.Build.dll looks for fsc.exe in the same directory that it resides in by default, so it's appropriate to bundle them.

I think this might still not work out-of-the box if you run this on a clean machine (i.e. Azure VM).
At least I recently tried running FSharp.Compiler.Service.dll on an Azure VM with plain OS and it did not work - it turns out that FSharp.Compiler.Service.dll has dependencies on some MSBUILD stuff (Microsoft.Build.dll and a bunch of others) that are not available on clean install with .NET and have to be installed separately.
Not sure if we can get the MSBUILD stuff from NuGet as a dependency or something...?

ok I added these. I wonder if we should depend on FSharp.Core package, but I assume since we are an app this is not really a good idea and we would need to copy files anyway,

Hmm, Tomas, do you remember what dependencies were those?

Yes, for this I believe you should add all the relevant MSBuild DLLs as local copies.  I think you can just change these references to be <Private>True</Private> and add the DLLs from the directory.

will look at this tomorrow. Now I have to watch Bayern vs. BVB ;-)
2015-04-28 20:06 GMT+02:00 Don Syme<EMAIL_ADDRESS>
Yes, for this I believe you should add all the relevant MSBuild DLLs as
local copies. I think you can just change these references
https://github.com/fsharp/fsharp/blob/master/src/fsharp/FSharp.Compiler/FSharp.Compiler.fsproj#L512
to be True and add the DLLs from the directory.
—
Reply to this email directly or view it on GitHub
https://github.com/fsharp/fsharp/pull/409#issuecomment-97157181.

Yes, first things first @forki !

done

Could you change the name per this comment

done


Remove appcelerator references

Motivation
As per ticket FH-3250 we are no longer supporting Titanium/appcelerator apps. Removing them from ups also.

If the FeedHenry project no longer supports Titanium/appcelerator apps, that does not automatically mean the same is true for the AeroGear community. We do have users, and contributors! that are doing Titanium.

:-1:  on this


Optimize StackArray

Currently, it makes new objects for each element. That's pretty slow.
StackArray should also be able to receive an additional count template N.

If N == 0 (the default), it will simply use dynamic heap memory like normal.
If N > 0, it will allow up to N elements on the stack inside of the object's memory itself. This allows an entire array of a certain amount of elements to be contained on stack memory, for example.

The first option (N == 0) would probably require some template metaprogramming (or specialization)? Or this could be an entirely different class.

Added a new Array<T> for now which can hopefully deprecate StackArray<T>.


Remove ignore:AppVeyor from codecov config

Since this repo migrated to GitHub Actions away from TravisCI and AppVeyor, the mentioned declaration is unnecessary.

Codecov Report

Merging #2540 into master will not change coverage by %.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #2540   +/-   ##
=========================================
  Coverage     80.83%   80.83%
  Complexity     2265     2265
=========================================
  Files           376      376
  Lines          7353     7353
  Branches       1206     1206
=========================================
  Hits           5944     5944
  Misses          810      810
  Partials        599      599

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 953c1a5...1316d65. Read the comment docs.


README: Add example of kernelSpec setting

Great!
I think we should also update this section to prefer jupyter.

I think to install the notebook pip install jupyter is enough.

Awesome thanks!


Code insiders crashes when clicking file menu

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.95.0-insider c85acdad6b679973c79b01a38afa91448c705c1c x64

OS Version:
Distributor ID:	Linuxmint
Description:	Linux Mint 21.3
Release:	21.3
Codename:	virginia

Steps to Reproduce:

Open Code Insiders
Click the File menu link at the top of the main window (it appears to only affect file; all other menus appear fine)
The window closes without prompt or error

This was on a working machine with many development hours on it, so it is not a fresh install and is known as a good environment.

/duplicate of  #230604


MQTT Battery and Aggregate Battery possible?

Is there a way to integrate data from a battery based on MQTT Battery together with batteries based on Serial Battery in Aggregate Battery?

Of course! Take a look at Manuels repository.

🤷🏻♂ which one? Can you link it?

What is wrong with this one you are currently in?

If you aleady have installed dbus-mqtt-battery and dbus-serialbattery they should be both visible in dbus-aggregate-batteries. If not you could try BatteryAggregator.

Linked issue: https://github.com/Dr-Gigavolt/dbus-aggregate-batteries/issues/51


Unable to view pdf image

I have an iOS codebase where vector assets are supplied in pdf format. When I tried to view an asset, the app did not render the pdf. Instead I got an error saying “Cannot display file as text”.

Bug Report Dump (Auto-generated)

Version 1.17.1<PHONE_NUMBER>)
Device: iPhone X (iOS 11.2.5)
TestFlight: false

That’s unfortunately not supported yet. Maybe you can open a pull request to add support?
Sent with GitHawk

@BasThomas  I'll try it the other day.

WIP


Migration path for roles from xsprivileges and xsaccess

Process all roles from xsprivileges and move them to a security descriptor
How to handle different authentication types from xsaccess

Option 1 - Use XSUAA APIs
Reference - https://api.sap.com/api/AuthorizationAPI/resource
Limitations

apiaccess plan is available only through Cloud Foundry
There is no API to create new role templates - one must be predefined in the xs-security.json

Flow

Get credentials for apiaccess plan (through CF)
In the xs-security.json for XSK add a fake role-template and fake scope like so

....
    role-collections:
    - description: XSK Developer
      name: XSK Developer
      role-template-references:
      - $XSAPPNAME.Developer
    - description: XSK Operator
      name: XSK Operator
      role-template-references:
      - $XSAPPNAME.Operator
    role-templates:
    - description: Developer related roles
      name: Developer
      scope-references:
      - $XSAPPNAME.Developer
    - description: Operator related roles
      name: Operator
      scope-references:
      - $XSAPPNAME.Operator
      ----
    - description: XS Privileges scope
      name: XSPrivileges
      ----
      scope-references:
      - "$XSAPPNAME.xsprivileges"
    scopes:
    - description: Developer scope
      name: $XSAPPNAME.Developer
    - description: Operator scope
      name: $XSAPPNAME.Operator
      ----
    - description: XSPrivileges scope
      name: "$XSAPPNAME.xsprivileges"
      ----
....

Obtain a token using the credentials from apiaccess instance
Create a new role from the template

POST https://api.authentication.eu10.hana.ondemand.com/sap/rest/authorization/v2/apps/roles
{
  "appDescription": "This is an example description of an application.",
  "appName": "xsk-xsuaa!t114536",
  "description": "This is the xsprivilege role.",
  "isReadOnly": false,
  "name": "XSPrivilegeRole",
  "roleTemplateAppId": "xsk-xsuaa!t114536",
  "roleTemplateName": "XSPrivileges",
  "scopes": [
    {
      "description": "Display values",
      "name": "xsk-xsuaa!t114536.Privilege"
    }
  ]
}

Create role collection for roles

POST https://api.authentication.eu10.hana.ondemand.com/sap/rest/authorization/v2/rolecollections
{
  "name": "RCFromXSPrivilege",
  "description": "Enables users to view the items.",
  "roleReferences": [
    {
      "roleTemplateAppId": "xsk-xsuaa!t114536",
      "roleTemplateName": "XSPrivileges",
      "name": "XSPrivileges"
    }
  ],
  "isReadOnly": false
}

Option 2 - update the k8s service instance
Use the k8s java client to patch a CR - https://github.com/kubernetes-client/java/blob/master/examples/examples-release-12/src/main/java/io/kubernetes/client/examples/PatchExample.java#L66
Limitations
Both limitations from above are not relevant here

Usable only in Kyma as it depends on ServiceInstance resource

Flow

Find the correct service instance bound to XSK
Change the spec to include the new scope, role-template and role collections
Apply the resource - this will be picked up by the service catalog and sent to XSUAA for processing

@dpanayotov, I think we should go with the first option. I see drawback in the second option, as we will need permissions (serviceaccount?) in the kubernetes cluster to execute updates.

Option 3

Process xsprivileges files
Generate xs-security.json with scopes and roles based on the xsprivileges // TBD JSON or YAML ready for kubectl apply
Display message saying the XSUAA instance needs to be updated with this file content

Pros:

No need for separate XSUAA instance with apiaccess plan
No need for service account for XSK for management of some resources

Cons:

manual step


❓ Is Frame Processors are currently read-only?

Question
I was wondering if possible to implement the camera filters like Snapchat or face beauty filters on  react-native-vision-camera I am aware it is possible to apply and show it in the camera view, but my question is about applying the filters on the video and then saving ita as file (with the filters not only raw video), is  react-native-vision-camera has hardware capture pipeline read and write to the file or it will require third bart libraries like ffmpeg.
What I tried
No response
VisionCamera Version
3.2.2
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Yea, I have this feature working perfectly on my end with an amazingly nice to use API including filters, shaders, etc, but I decided to not integrate that into VisionCamera because it just overcomplicated the codebase a lot and users that don't use these features (probably >90% of users) will suffer under that because of some build issues or larger npm size. See this section in the docs


segmentation fault on vm

Installed neural-style on ubuntu 14.04 LTS virtual machine:
Memory : 3.9 GiB
Processor : i5-3210M CPU @ 2.50GHz x 4
while installing torch7, ./install.sh didn't work so I followed the instructions on https://github.com/torch/torch7/issues/191, and it worked fine.
However, th neural_style.lua -gpu -1 gives me an error:
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:505] Reading dangerously large protocol message.  If the message turns out to be larger than<PHONE_NUMBER> bytes, parsing will be halted for security reasons.  To increase the limit (or to disable these warnings), see CodedInputStream::SetTotalBytesLimit() in google/protobuf/io/coded_stream.h.
[libprotobuf WARNING google/protobuf/io/coded_stream.cc:78] The total number of bytes read was 574671192
Successfully loaded models/VGG_ILSVRC_19_layers.caffemodel
conv1_1: 64 3 3 3
conv1_2: 64 64 3 3
conv2_1: 128 64 3 3
conv2_2: 128 128 3 3
conv3_1: 256 128 3 3
conv3_2: 256 256 3 3
conv3_3: 256 256 3 3
conv3_4: 256 256 3 3
conv4_1: 512 256 3 3
conv4_2: 512 512 3 3
conv4_3: 512 512 3 3
conv4_4: 512 512 3 3
conv5_1: 512 512 3 3
conv5_2: 512 512 3 3
conv5_3: 512 512 3 3
conv5_4: 512 512 3 3
fc6: 1 1 25088 4096
Segmentation fault (core dumped)

gdb test core gives this message:
warning: core file may not match specified executable file.
[New LWP 2807]
Core was generated by `/home/recycli/torch/install/bin/luajit -e package.path="/home/recycli/.luarocks'.
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x08053dd9 in ?? ()

system monitor while executing neural_style.lua :

What could be the problem? Still same thing happens after allocating 5GiB of memory to the virtual machine.

Try passing parameter -image_size 150 and check if it works. 4GB of RAM is barely enough for images around 500px. Also, after that you can try using -optimizer adam and vgg_normalised network for more memory reduction.

Finally found the reason: virtual machine was 32-bit ubuntu.
Upgraded it to 64-bit and now it works well. Thanks for help, drag0.

Wow, nice catch!


Enable color themes to improve contrast for accessibility

ContrastAnalyser determines if the color contrast is sufficient to make text legible for accessibility
For example, the default error message colors on Windows, the red foreground on the black background is insufficient and should be a brighter red according to this tool.  Should be able to define custom colors for Progress, Error, Verbose, etc... and package it as a theme.

I came to discuss:
Set in .bashrc:
Export TERM = putty-256color
or
Export TERM = Eterm-256color
or
Export TERM = st-256color
Set in xshell:
Terminal   --->    terminal type --->  putty-256color
environment:
centos 7
Anyone try it, it works fine, it makes your console color beautiful.
It is recommended to set this to the powershell console default.
I mean: hard coding this setting in powershell is better. Do not let it inherit from BASH.

You can't set things like that in PowerShell, @chuanjiao10 -- those settings depend on your terminal.

@SteveL-MSFT A thought on this: As you know, in FCU, we updated the default console palette to provide better contrast on modern displays. PLUS, we shipped a new tool to allow users to load color "schemes" if they prefer (esp. useful to those who're color-blind/sensitive).
If PowerShell was to fix its palette use to use color[0] as background, etc. rather than color[5], it'd be able to enjoy the palette improvements we've made ;)

@bitcrazed Changing Windows PowerShell to use color[0] is a non-starter for various reasons.  For PowerShell Core 6, we're using color[0] as background color.

The new ANSI rendering proposal will allow setting color definitions which should satisfy this


Bitdefender claims Java.Trojan.GenericGBA.33263 in Fields.class

Hi,
i downloaded Create: Astral a few days ago and Bitdefender keeps deleting that one mod: .minecraft.fabric\processedMods\net_lenni0451_reflect-1.1.0-658b0f614e510689.jar => net/lenni0451/reflect/Fields.class
and claims it is infected with Java.Trojan.GenericGBA.33263..
I uploaded it to virustotal giving me a 0/61 result, but how come bitdefender desperately removes it, even after adding it as an exception..
It's the only mod i ever saw giving me such a warning.
VT Link: https://www.virustotal.com/gui/file/30c9d2f9bea0aac7fe697bfadb2121de257e5936674550d8203798fbef1c1780
Regards

Hi, there are numerous reports of Bitdefender starting to false flag Reflect for no reason. I'm currently not sure how I would fix this. I'll let you know if there is a solution

Hi, there are numerous reports of Bitdefender starting to false flag Reflect for no reason. I'm currently not sure how I would fix this. I'll let you know if there is a solution

You can report a false positive to Bitdefender right here then they'll either fix it or explain why it is not a false positive.

Google also auto flags mods using this as a virus. I use google drive to share a collection of mods and config files privately between a few friends, and google sent me an auto email threatening to ban my drive account because of the false detection.

Google also auto flags mods using this as a virus. I use google drive to share a collection of mods and config files privately between a few friends, and google sent me an auto email threatening to ban my drive account because of the false detection.

It would be good if you could report this to google to review. Maybe they will realize that it is a false flag and fix it

Hi,
i downloaded Create: Astral a few days ago and Bitdefender keeps deleting that one mod: .minecraft.fabric\processedMods\net_lenni0451_reflect-1.1.0-658b0f614e510689.jar => net/lenni0451/reflect/Fields.class
and claims it is infected with Java.Trojan.GenericGBA.33263..
I uploaded it to virustotal giving me a 0/61 result, but how come bitdefender desperately removes it, even after adding it as an exception..
It's the only mod i ever saw giving me such a warning.
VT Link: https://www.virustotal.com/gui/file/30c9d2f9bea0aac7fe697bfadb2121de257e5936674550d8203798fbef1c1780
Regards

It has now been over a week since I wrote the Bitdefender support and I still have'nt gotten a response so there is sadly nothing I can do about this


Rewrite Karavan Designer to use Functional Components and State

Is this process already ongoing?

@stefanhuber yes

Ok is there already a branch for that? If you define how, I might be able to help you with some of the components.
Firstly, thank you for providing such a great tool. Some first thoughts/obvservations, which I made when going through the code base:

It might make sense to create a folder for each component and also add css (instead of one big karavan.css).
Some components are quite big (e.g, RestDesigner), it might be possible to split them into smaller ones
Use 2 spaces for indentation might help (example), it is generally used with javascript
Adding testing to the component folder directly
Starting the development server with npm start takes a long time, vite would improve this drastically
It might makes sense to use storybook for developing components in isolation
Why use react layout components, if the same is possible with css flexbox, just with less code and better performance? (example)
One could create a custom hook in react instead of an additional library (lodash.debounce) for debounding (example)

Greetings,
This library is very useful. I am using this library for one of my personal projects, but the code and logic are currently written in Typescript and I am trying to convert it to JSX but facing some issues because I am not that good in Typescript. It will be very helpful if the JSX version is available for this.
Thanks,
Jasham


[ENHANCEMENT] Required fields are not checked when saving (via yaml save)

Is your enhancement related to a problem? Please describe.

Required fields are not checked when saving (via yaml save).
When we creates a new Harvester cluster, they can create a cluster without a network (from the GUI, via yaml save) which eventually breaks the whole cluster, according to their information it keeps creating and deleting nodes.
Steps to reproduce: Create new cluster, cloud credentials -> Select RKE2 and Harvester node driver -> Type in the cluster name -> Switch to YAML view and click on Create. There is nothing that prevents it.
Describe the solution you'd like

Save via yaml should also validate.
Describe alternatives you've considered

Additional context

Tested the enhancement applied on Harvester v1.3.0-rc1 with Rancher v.2.9-head[^1] but not available on Rancher v2.7.9 and v2.8-head [^2].
I can't confirm Rancher v2.9.0 will ship with Harvester v1.3.0, so move this back to Review.
[^1]: Docker Image: rancher/rancher@sha256:22b31c9b820ab88965f547a1d4a462856e8cea03084857eebb89d18064112a70
[^2]: Docker Image: rancher/rancher@sha256:0352298855f0452b651614a0f0dec4ed8708da3121ea9b325a56e8ca47218be7

@torchiaf Is this in v1.3.0-rc3? Thanks.

According to @torchiaf, the change is merged in https://github.com/rancher/dashboard/pull/10073 (target master code and supposed to be in dev-2.9). This needs to be backported to v2.7 and v2.8.

Verified this enhancement has been applied.
Test Information

Environment: baremetal DL360G9 5 nodes
Harvester Version: v1.3.0-rc4
ui-source Option: Auto
Rancher Versions:

v2.7-bd0bb772b60a447d352789b3d7c4368ffe50eecf-head[^1]
v2.8-19cfbb81afd2689dc055e03193264f9616707922-head[^2]

Verify Steps

Install Harvester with any nodes
Import Harvester into Rancher
Create RKE2 cluster and don't fill in any data, click Edit as YAML
Error message should prompted, and user should not able to switch to yaml editor

[^1]: Docker image: rancher/rancher@sha256:3deedb983e5f80bfc154e20c7d434ffd74a51d8a121c314abbff7b2393f1d027
[^2]: Docker image: rancher/rancher@sha256:e273d4fa864ae3c26511ee613ba516e19aee84bd6634ac936e146ce2d4143b12

Can issue https://github.com/rancher/dashboard/issues/10605 be closed too?


RoomChat: Add "go to live" option when peeking

in order to be able to send a message.
See the following user's feedback:
"When clicking a link from a notification email, I cannot reply to the conversation that the notification comes from."

Done by keeping visible the activity view:


Do not clone deep options in forward

This is a followup to #2713.
In the process of cloning options in forward function, the the route instance is replaced by a copy, so the route instance stored in router.currentRoute is different from the one passed to route async handler.
The proposed change is safe since options variable is not mutated at all.
It is also more performant

Looks good, thanks!


add alias in config-overrides.js

This assumes that the hyva-checkout will be installed by composer and placed in .../vendor/hyva-themes/....
3rd party payment methods can reference core components by using the '@hyva/payments' alias
for example
import Card from '@hyva/payments/components/common/Card';

As per author directions, closing this PR


⚠ Bofrost.de has degraded performance

In 5243f9e, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1119 ms

Resolved: Bofrost.de performance has improved in b1b2bff after 13 minutes.


Why not automatically inject JSFile?

WebViewJavascriptBridge is a great library.
But when I using, I was curious that why not automatically inject JSFile, but match a specific Request URL?
If View instance A need a webview with JSBridge, automatically injection should be more convenient.
https://github.com/marcuswestin/WebViewJavascriptBridge/blob/f40e3d3a280ddf1cefb202a61c7960f8c49a1af8/WebViewJavascriptBridge/WebViewJavascriptBridge.m#L182-L186

I had refactor the WebViewJavascriptBridge .
If you are in trouble.You can try this.
Download Address:
https://github.com/housenkui/WebViewJavascriptBridge
More light and more simple to use. 10 mins will enough to completed your demand.


Refactor BE

Current BE's structure is very complex. It is very difficult to maintain it so that we decide to refactor it.

[
be_refactor.pdf
](url)


Hive module chores

Describe your problem

Vineyard java sdk currently has dependencies that conflict with hive. （Such as guava）

Does the table structure in the vineyard java sdk need to be refactored?
The data for a hive table can be stored in different files under the same folder. However, the hive tables currently stored on the vineyard are actually stored using multiple vineyard tables (one vineyard table per file). Do we need to refactor the vineyard sdk table structure to match the hive table structure?

Optimize the hive module deployment process.

Complete hive documentation.

Please create sub issues for those todos.


more grammar and spelling issues in index.html

I stopped on line 273. I was starting to get confused by the directions and wasn't sure my "grammar and spelling" fixes were accurate.

Great Job.
Could you add the folling to the readme.md file:
Edited by [your name](http://github.com/your nameMerge branch 'main' into look)
This way you get the credit you deserve.
Thanks for your work.  I really appreciate it.
Please enter a commit message to explain why this merge is necessary,
especially if it merges an updated upstream into a topic branch.

Lines starting with '#' will be ignored, and an empty message aborts
the commit.


Add VLC for iOS

Thanks for your contribution! This is a template that can make it easier when submitting your contribution:
Your Project: VLC for iOS
Project URL: _https://github.com/videolan/vlc-ios_
Category: Video
Description: VLC is a free and open source multimedia player
Inclusion requirements:

[x] Has 50 GitHub stargazers or more
[x] Only one project/change is in this pull request
[x] Isn't an archived project
[x] Has more than one contributor
[x] Has unit tests, integration tests or UI tests
[x] Addition in alphabetical order (bottom of category)
[x] Supports iOS 12 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

:heart: Awesome iOS Contributors

已收到您的邮件，我会在24小时内回复您的。——Kevin San.

@WebSnke Thanks for contributing!

@WebSnke Thanks for contributing!

You're welcome. Could you take a look at my other PR too?


Add a formalized backend option to the crypto dependencies

This plugs in a version of curve25519-dalek that uses formalized cryptographic operations from fiat-crypto.
The backend is exposed through the curve25519-fiat package (to be published imminently), which packages certified Rust field arithmetic primitives extracted from Coq.
The curve25519-fiat package is then used in an alternate backend for the dalek suite of libraries linked therein, which should ultimately be exposed as maintained forks.
The key commit adding this backend is https://github.com/huitseeker/curve25519-dalek/commit/5fabb0be26a4e00b1dca415cd298a6acf00c60be
This PR modifies the following behavior:

when compiled (tested ...) with the following options, the crypto and dependent crates use curve operations extracted from Coq proofs of arithmetic correctness,

 cargo build --features="std fiat_u64_backend" --no-default-features

when compiled without those options (as they are by default), the construction is unchanged,

Why this is better:

there are tons of ways to get curve operations wrong in crypto libraries, see a list of recent bugs, and this eliminates this class of possible bugs
the performance of this backend is still super good (~20% downgrade in performance)
this is an option, the default does not modify anything

Why this is worse:

the build is more complicated for the crypto crate

Why forks:

doing it without is much more complicated and less tranparent (we tried)

Future work:

improve the performance of the Rust extracted operations
actually host the necessary forks (or find a way to get rid of them with upstream)

Co-authored-by: Zoe Paraskevopoulou<EMAIL_ADDRESS>Co-authored-by: Lladislas de Naurois<EMAIL_ADDRESS>

Do we agree on the performance penalty this might introduce?

@kchalkias No, which is why this is a non-default build option, while we improve on that.

@kchalkias rebased to reflect the update in dependencies.


@emit does not work?

Parent component:
<hello-world @test="test"></hello-world> methods: {    test(val) {       console.log(val)    } }
Children component:
methods: {   click() { this.$emit('test', 'hello') } }
the parent component test function does not work

sorry, it's working, my mistake


Bug diagnosing error "Object is possibly 'undefined'"

TypeScript Version:  3.4.5

Search Terms: Object is possibly 'undefined'
Code
function Sum(A?: number) {
    const AisUndefined: boolean = typeof A === "undefined";

    if (AisUndefined) {
        return 'is undefined';
    }

    const sum: number = A + 10; //Error here: "Object is possibly 'undefined'"
    return sum;
}

Expected behavior: As I assign the condition typeof A === "undefined" in the constant AisUndefined and make a if statement to return a string in case of A is definitely undefined, in the constant sum, there should be no problem making a sum of A with a number assuming that A is definitely not undefined as the code reach there, after the if statement.
Actual behavior: When I try to make a sum with A and a number, and probably any other operation that takes on that A is not undefined, typescript identifies this as an error, saying that Object is possibly 'undefined', basically ignoring the return inside the if statement.
Note: If I replace the constant AisUndefined by their own condition, like:
if (typeof A === "undefined") {
    return 'is undefined';
}

Therefrom, the error disappears.
Playground Link:  https://www.typescriptlang.org/play//#src=function Sum(A%3F%3A number) {
    const AisUndefined%3A boolean %3D typeof A %3D%3D%3D "undefined"%3B

    if (AisUndefined) {
        return 'is undefined'%3B
    }

    const sum%3A number %3D A %2B 10%3B
    return sum%3B
} (You will have to check the strictNullChecks option to get this error)
Related Issues:

Duplicate of #12184

Yeah, this is normal.  Only direct type guards work; the compiler doesn't track the effect of assigning a type check result to another variable.


[SPARK-8889] [Core]: Fix for OOM for graph creation

Fix for OOM for graph creation

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #38123 has started for   PR 7602 at commit 964dc36.

Merged build finished. Test FAILed.

Test build #38123 has finished for   PR 7602 at commit 964dc36.

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

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #38282 has started for   PR 7602 at commit 72904c7.

thanks @JoshRosen for your review comment.can you please have a look now? thanks!

Test build #38282 has finished for   PR 7602 at commit 72904c7.

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

Merged build finished. Test PASSed.

ping @JoshRosen @andrewor14

Merged build triggered.

Merged build started.

Test build #38587 has started for   PR 7602 at commit 84615f5.

Test build #38587 has finished for   PR 7602 at commit 84615f5.

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

Merged build finished. Test PASSed.

This looks fine, but have you verified whether this actually fixes the issue? I wonder how much memory this actually saves us because I imagine much of the intermediate strings are garbage collected.

@andrewor14 thanks for checking. imo the OOM can only happen if system is running low on memory and/or GC is working 98% of the time and still being able to free < 2%.This can happen if GC reclaims memory but new objects are getting created on heap “simultaneously", can happen in our case strings created in loops and recursive calls.I did test the DAG visualization on few jobs, no OOM/heap issue for me.That said, this issue is a variation on complexity of the job with machine configuration/parallel jobs.
While -Xmx increase is a viable option, this patch is to align with best practices of using optimum object and not entirely depend on GC, as even when the de-scoped objects get eligible for GC, it does not guarantee that they are reclaimed immediately.In addition for just in case scenarios had added SparkException to catch OOM, as some users perceive stacktrace as a system flaw which forgot to anticipate a possible concern, anyhow agreed with @JoshRosen and removed catch.
Please review/approve @andrewor14 @JoshRosen thanks.

I think this actually makes it worse, since it is not only accumulating the result in a StringBuilder, but always converting it to a String as the return value from each recursive call. That's not needed, is it? Without that, I imagine this is a small improvement, but I also do not think this can prevent an OOM. In its current state I definitely don't see how this fixes the issue.

Merged build triggered.

Merged build started.

Test build #38954 has started for   PR 7602 at commit e819695.

Thanks for your comment @srowen . It does not make it worse, as toString() was in earlier code and was always doing it in addition to string objects in recursive calls.However its a valid point you make, that is not needed, updated .If you have a load testing environment where you could execute different complexity of jobs in loops and check profiler with this branch, you will conclude this helps.
imo OOM can never be completely solvable, as a users can run with infinitely tight resource with extremely unreasonable job, but reducing those probabilities is still essential.
Given that, please review , approve @srowen @JoshRosen @andrewor14 thanks!

Test build #38954 has finished for   PR 7602 at commit e819695.

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

Merged build finished. Test FAILed.

I was thinking that it required more memory because, indeed, the same Strings are being constructed and returned, but now a StringBuilder is being build in parallel. I don't know for sure; I didn't measure it. In any event, yes the idea is to only construct the result in a StringBuilder.
Does makeDotSubgraph need to return anything? right now it looks like the recursive calls append to the result, but then return the current result to be appended to itself in the caller? I could be reading it wrong but that doesn't look right.
Also the initial argument starts with "    " appended but I don't think the original version started this way?

Merged build triggered.

Merged build started.

Thanks @srowen for your comment.The recursive calls for cluster child and nodes within makeDotSubgraph internally appends the result of each call and has normal recursion stack flow.makeDotSubgraph has to return value as main method makeDotFile needs it to form the final dot file.There is no additional StringBuilder created by the patch which is not already getting created in original.indent is set to " " in original as well.
What I was also thinking is there is possibility of stack overflow when either there is possible lurking infinite recursion and/or if processing goes beyond recursion depth workable on cluster. But as stack overflow can mask itself as OOM, I was also thinking of increasing –Xss in pom.xml. Or atleast catch OOM error in SparkException and have message making this suggestion to user to configure -Xss. Hence updated patch.
This patch at the very least, makes sure objects are less,and –Xss suggestion which can help in any case.  Given that, please review, approve @srowen @JoshRosen @andrewor14 thanks!

Merged build finished. Test FAILed.

OK, on reading this further, I think there are different problems.
First, indent is increased, but never decreased. That's not how the original worked. indent does not need to be a StringBuilder and shouldn't be.
Second, there's no need to make a new StringBuilder inside each recursive call. I thought the intent was to pass one down through all the recursions. That is what should really drive some memory savings.
And if so, there is no need to return the StringBuilder; the top-level caller already has a reference to it.
Finally, you don't want to catch OutOfMemoryError in general. It's not likely recoverable.

Merged build triggered.

Merged build started.

Good point @srowen  new StringBuilder was as in original code,corrected.Agree that it helps.
OOM catch was suggestive to back my comment on stack recursion/thread stack memory issue.
Please review @srowen  thanks!

Test build #39270 has started for   PR 7602 at commit 345a078.

thanks for your comments @srowen .done.please can you have a look? thanks!

Merged build triggered.

Merged build started.

Test build #39283 has started for   PR 7602 at commit 141def5.

Test build #39270 has finished for   PR 7602 at commit 345a078.

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

Merged build finished. Test PASSed.

Test build #39283 timed out     for PR 7602 at commit 141def5     after a configured wait of 175m.

Merged build finished. Test FAILed.

I suppose the only remaining question is, does it work? I am not sure there is a unit test for it. Does the output appear reasonable? the code looks OK

Test build #1277 has started for   PR 7602 at commit 141def5.

Test build #1277 has finished for   PR 7602 at commit 141def5.

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

The changes look fine, but I'm still skeptical how much of a difference it makes in practice. All of this data inherently takes some space on the driver, and we have tested this with a long-running application and verified that the memory usage is stable over time (could be up to 200 - 300 MB). I think the OOM is just a consequence of the low driver heap size, which used to be 512MB but later increased to 1G by default.

Merged build triggered.

Merged build started.

Test build #39441 has started for   PR 7602 at commit bcef0e1.

Thanks @andrewor14 for your review comments. done. What you say is correct. That said, in stack recursion it is simply bad practice to create more objects than absolutely necessary., as it is loophole for trouble.As its not just heap, even stack overflow caused due to recursion with job depth can camouflage as heap/OOM issue.I did some initial changes, and post great review comments from @srowen  even removed the original extra string builder created on every recursion.Functionality being constant, if you do a load test with different complexity of jobs that can be tested across possible configurations and check profiler, you would see the improvement.
Please review approve @andrewor14  thanks!

Test build #39441 has finished for   PR 7602 at commit bcef0e1.

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

Merged build finished. Test PASSed.

I'd like to merge this; I suppose I'm a little wary to do so as I am not sure there are tests covering this code? is there any easy way to unit test it? anything would be reassuring.

OK, if this doesn't change functionality then I think it's a strict improvement. Admittedly there are currently no tests for the formation of the dotstring, and I should have done that when I wrote this code in the first place. @rekhajoshm would you mind adding a test for the RDDOperationGraph to assert that the string generated is as we expect? If you prefer not to that's also fine, but we'll have to manually test this before we merge it.

Thanks @srowen @andrewor14 I don’t mind for sure, only that I am traveling this month.
Had manually tested the DAG visualization earlier, but will find a slot of time this week when I can do some focused testing.will update.thanks!

Merged build triggered.

Merged build started.

Hi @srowen @andrewor14.This time also attaching the screenshot of manual testing done for dotfile of DAG visualization on this branch.Also added a test for dot file DAG representation.
Please review, hoping to get your approvals on merge. thanks!

Merged build finished. Test FAILed.

That's looking good. The tests passed before and then failed; I don't see the output here for some reason so let me just run tests again.

Test build #1444 has started for   PR 7602 at commit 3fb5271.

Test build #1444 has finished for   PR 7602 at commit 3fb5271.

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

retest this please
@rekhajoshm thanks for adding the test. What I had in mind was more fine-grained test for the RDDOperationGraph itself, actually, but the test you added is also fine. I'll merge this once tests pass.

retest this please

Merged build triggered.

Merged build started.

Test build #40478 has started for   PR 7602 at commit 3fb5271.

Test build #40478 has finished for   PR 7602 at commit 3fb5271.

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

Merged build finished. Test FAILed.

Hm, looks like this is failing the selenium suite. I would recommend not making it a selenium suite but test it on a finer-grained level at the RDDOperationGraph. I'm concerned that it might become a flaky test we have to fix in the future.

Merged build triggered.

Merged build started.

Test build #40579 has started for   PR 7602 at commit eee1733.

Thanks @srowen @andrewor14  updated push with tested UISelenium version which consistently passes (tested more than 20 times). I was curious to know why concise version did not, hence had pushed the concise version earlier to verify with your jenkins and ascertain there is no issue at my end.
[INFO] --- scalatest-maven-plugin:1.0:test (test) @ spark-core_2.10 ---
Discovery starting.
Discovery completed in 2 seconds, 390 milliseconds.
Run starting. Expected test count is: 14
UISeleniumSuite:
- effects of unpersist() / persist() should be reflected
- failed stages should not appear to be active
- spark.ui.killEnabled should properly control kill button display
- jobs page should not display job group name unless some job was submitted in a job group
- job progress bars should handle stage / task failures
- job details page should display useful information for stages that haven't started
- job progress bars / cells reflect skipped stages / tasks
- stages that aren't run appear as 'skipped stages' after a job finishes
- jobs with stages that are skipped should show correct link descriptions on all jobs page
- attaching and detaching a new tab
- kill stage POST/GET response is correct
- stage & job retention
- live UI json application list
- job stages should have expected dotfile under DAG visualization
Run completed in 16 seconds, 283 milliseconds.
Total number of tests run: 14
Suites: completed 2, aborted 0
Tests: succeeded 14, failed 0, canceled 0, ignored 0, pending 0
All tests passed.

Test build #40579 timed out     for PR 7602 at commit eee1733     after a configured wait of 175m.

Merged build finished. Test FAILed.

Selenium suites are flaky in general. It's not surprising that it fails on jenkins but not your local machine, so it doesn't matter how many times it passes locally.

retest this please

Merged build triggered.

Merged build started.

Test build #40599 has started for   PR 7602 at commit eee1733.

Test build #40599 has finished for   PR 7602 at commit eee1733.

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

Merged build finished. Test FAILed.

Test build #1493 has started for   PR 7602 at commit eee1733.

Test build #1493 timed out     for PR 7602 at commit eee1733     after a configured wait of 175m.

Test build #1629 has started for   PR 7602 at commit eee1733.

Test build #1629 has finished for   PR 7602 at commit eee1733.

This patch fails to build.
This patch does not merge cleanly.
This patch adds no public classes.

Test build #1649 has started for   PR 7602 at commit eee1733.

Test build #1649 has finished for   PR 7602 at commit eee1733.

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

Test build #1666 has started for   PR 7602 at commit eee1733.

Test build #1666 has finished for   PR 7602 at commit eee1733.

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

There we go, tests pass. What do you think @andrewor14 ? I'm OK with it.

OK, let's just merge it first and we can always disable the selenium tests if it proves to be flaky. Feel free to merge it.

Thanks @srowen @andrewor14 for the merge.


[no-merge] remove internal dep

/retest

/retest

/retest

/retest

/retest
Fresh platform_ui image released. Test should now fail because PDF status reaches 'Failed' instead of 'Generated'

/retest

/retest


fix failing benchmarks

Pull Request Test Coverage Report for Build 844323900

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.234%

Totals

Change from base Build 819936635:
0.0%

Covered Lines:
2358

Relevant Lines:
2976

💛  - Coveralls

This seems to fix the failing benchmarks. The changes in .gitignore are useful for running benchmarks locally, as explained in https://github.com/tkf/BenchmarkCI.jl.


Port forwarding controls

Is there a way to control port forwarding options?
I'd love to write a command line utility for users to easily set port forward rules (rather than logging into the UI each time)

Update: I ended up writing a quick python + selenium program that I can run from the command line. This has made the editing of port forwarding rules easier.

@bhakta0007 can you link your scripts for port forwarding?

@alan-cugler I just saw your question now. I need to cleanup my script a bit - If it is it still relevant to you - I can do it

here you go: https://github.com/bhakta0007/ng


SingleOrDefault / FirstOrDefault fails for compiled queries

Given the following query:
public class VotesByUserOnPhotoQuery : ICompiledQuery<PhotoMemberCommentVotes>
{
    public int PhotoId { get; set; }

    public int MemberId { get; set; }

    public Expression<Func<IQueryable<PhotoMemberCommentVotes>, PhotoMemberCommentVotes>> QueryIs()
    {
        return query => query.Where(x => x.MemberId == MemberId)
                                .Where(x => x.PhotoId == PhotoId)
                                .SingleOrDefault();

    }
}

This currently throws an exception when no result is returned rather than returning null.
System.InvalidOperationException: Sequence contains no elements
 at System.Linq.Enumerable.Single[TSource](IEnumerable`1 source)
 at Marten.Linq.CompiledQueryExecutor.<ExecuteQueryAsync>d__5`2.MoveNext()
 --- End of stack trace from previous location where exception was thrown ---
 at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
 at HyRes.Web.Modules.Actions.CommentModule.<<-ctor>b__0_0>d.MoveNext() in E:\github\HyRes\src\HyRes.Web\Modules\Actions\CommentModule.cs:line 27
 --- End of stack trace from previous location where exception was thrown ---
 at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
 at Nancy.Routing.DefaultRouteInvoker.<Invoke>d__b.MoveNext()
 --- End of stack trace from previous location where exception was thrown ---
 at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter.ValidateEnd(Task task)
 at Nancy.Routing.DefaultRequestDispatcher.<Dispatch>d__0.MoveNext()
 --- End of stack trace from previous location where exception was thrown ---
 at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
 at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
 at Nancy.NancyEngine.<InvokeRequestLifeCycle>d__9.MoveNext()
 --- End of inner exception stack trace ---
 at Nancy.NancyEngine.InvokeOnErrorHook(NancyContext context, ErrorPipeline pipeline, Exception ex)

I can pick this up tomorrow, unless someone'll beat me to it.

Actually, I wanna take this one tomorrow. Got a refactoring I wanna do as part of this, plus hitting compiled query within the batch query support
Sent from my iPhone

On Apr 7, 2016, at 3:50 PM, nieve<EMAIL_ADDRESS>wrote:
I can pick this up tomorrow, unless someone'll beat me to it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

cool, np.


[MINOR] Remove unused throw exception in SecureHadoopCatalogOperations#testConnection

What changes were proposed in this pull request?
the testConnection throw Exception is redundant in this method, so we should remove it.
Why are the changes needed?
update the code quality
Does this PR introduce any user-facing change?
n/a
How was this patch tested?
n/a

@yuqi1129 please help to review.


Implement stable server API

Need to dig deeper into vibe-d documentation so that it doesn't throw exceptions because optional parameters aren't provided.
The parameters should be equal to the ones from the command line tool / config file.

[ ] Implement API endpoint with certain set of parameters
[ ] Allow different response data depending on http header Accept-Type (application/json, image/png, application/zip)
[ ] For application/json return detailed response messge (e.g. for errors), for image/png and application/zip only return http error codes)