Text Summarization NLP

Closes: #64

Title: Text Summarization using NLP
Your Name: Ankan
Idenitfy yourself: GSSOC-24 EXTD

Describe the add-ons or changes you've made 📃
It is Text Summarization Model built using NLP which helps user to summarize long texts. The user just needs to type
the Passage they want and click summarize and within few seconds the model does the job. It has a very suitable UI.
This Model is a Natural Language Processing (NLP) model to summarize long pieces of text into concise summaries
The Backend of the project is built in Python using NLP model Transformer and the frontend is built with Streamlit . The Backend and front is connected with Streamlit in python
Checklist: ☑

[x] My code follows the Contributing Guidelines & Code of Conduct of this project.
[x] This PR does not contain plagiarized content.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] My changes generate no new warnings.

Screenshots 📷

https://github.com/user-attachments/assets/ec30545d-3154-440e-a21c-85fb49884e59

I have built the newer version with streamlit app as per the requirements ,henceforth creating this new pull request.
I hope everything is fine . @Avdhesh-Varshney

I have commited all the changes as per the rules. I think everything is fine now @Avdhesh-Varshney

Don't create a lots of commits, Otherwise it can't be merged into main branch

@Avdhesh-Varshney
I am closing this PR and will be creating a New fresh Pull Request with 1 perfect commit. I think that will be more clear easy to merge.


GPU model pickled state does not work on non-GPU system

Traceback (most recent call last):
  File "/home/jorge/dai-1.6.2-linux-ppc64le/python/lib/python3.6/concurrent/futures/process.py", line 175, in _process_worker
    r = call_item.fn(*call_item.args, **call_item.kwargs)
  File "h2oaicore/models.py", line 904, in h2oaicore.models.MainModel._predict_internal_booster_sub_call
  File "h2oaicore/models.py", line 3495, in h2oaicore.models.XGBoostModel._predict_internal_xgboost
  File "h2oaicore/models.py", line 3595, in h2oaicore.models.XGBoostModel.get_model
  File "/home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/core.py", line 982, in __setstate__
    _check_call(_LIB.XGBoosterLoadModelFromBuffer(handle, ptr, length))
  File "/home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/core.py", line 178, in _check_call
    raise XGBoostError(_LIB.XGBGetLastError())
xgboost.core.XGBoostError: b'[00:50:42] /root/repo/xgboost/include/xgboost/../../src/common/common.h:193: Check failed: n_gpus <= n_available_devices (1 vs. 0) Starting from gpu id: 2, there are only 0 available devices, while n_gpus is set to: 1\n\nStack trace returned 10 entries:\n[bt] (0) /home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/./lib/libxgboost.so(dmlc::StackTrace(unsigned long)+0x94) [0x7ffc82b81664]\n[bt] (1) /home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/./lib/libxgboost.so(dmlc::LogMessageFatal::~LogMessageFatal()+0x4c) [0x7ffc82b81f2c]\n[bt] (2) /home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/./lib/libxgboost.so(xgboost::GPUSet::All(int, int, int)+0x6c0) [0x7ffc82da47a0]\n[bt] (3) /home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/./lib/libxgboost.so(xgboost::obj::RegLossObj<xgboost::obj::LinearSquareLoss>::Configure(std::vector<std::pair<std::string, std::string>, std::allocator<std::pair<std::string, std::string> > > const&)+0x274) [0x7ffc82de03f4]\n[bt] (4) /home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/./lib/libxgboost.so(void xgboost::ObjFunction::Configure<std::_Rb_tree_iterator<std::pair<std::string const, std::string> > >(std::_Rb_tree_iterator<std::pair<std::string const, std::string> >, std::_Rb_tree_iterator<std::pair<std::string const, std::string> >)+0x11c) [0x7ffc82c06b7c]\n[bt] (5) /home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/./lib/libxgboost.so(xgboost::LearnerImpl::Load(dmlc::Stream*)+0xdf4) [0x7ffc82c14614]\n[bt] (6) /home/jorge/dai-1.6.2-linux-ppc64le/cuda-9.0/lib/python3.6/site-packages/xgboost/./lib/libxgboost.so(XGBoosterLoadModelFromBuffer+0x78) [0x7ffc82b8b5f8]\n[bt] (7) /home/jorge/dai-1.6.2-linux-ppc64le/lib/libffi.so.6(+0x7254) [0x7fff97227254]\n[bt] (8) /home/jorge/dai-1.6.2-linux-ppc64le/lib/libffi.so.6(ffi_call+0xe0) [0x7fff97225f50]\n[bt] (9) /home/jorge/dai-1.6.2-linux-ppc64le/python/lib/python3.6/lib-dynload/_ctypes.cpython-36m-powerpc64le-linux-gnu.so(_ctypes_callproc+0x770) [0x7fff96ad7d60]\n\n'
"""

This used to work with xgboost, but does not anymore.  Normally I would be able to load a pickled model, and then before running predict I would be able to change the predictor and model parameters to use no gpus.  But the check is already during pickle loading.
@hcho3

when wanting to use a GPU model on a GPU-less system, I was normally load the pickle state and then do:
    booster.set_param({'n_gpus': 0})
    self.model.kwargs['n_gpus'] = 0
    self.model.kwargs.pop('gpu_id', None)
    model.n_gpus = 0
    model.gpu_id = None
    model.n_jobs = max_threads()

but this is impossible if loading pickled state itself pre-checks everything and fails already.

@trivialfis I think it’s related to #4454. Let me review it

@hcho3  It should help.  But this is not a tested behaviour.  Do you think it's possible to test these mixed environments model serialization on Jenkins?

There are at least 2 ways to do that:

have first part of a test make a pickled object.  Then run a second test in a subprocess for isolation.  Set:

os.putenv('CUDA_VISIBLE_DEVICES', '')
os.environ['CUDA_VISIBLE_DEVICES'] = ''

then go into yet another subprocess and run the test.  For xgboost this requires no imports at top level or else xgboost segfaults due to references in original import.

Run test (as part of test suite) that produces an artifact of a pickle object, and then in isolation run CUDA_VISIBLE_DEVICES= pytest ... for a next isolation test that uses such artifacts.

The latter is simpler but means you have at least 2 stages of tests instead of running them all at once.

Any chance for a quick fix?  Turns out this is critical for us.  I'm happy to apply some quick fix to my own branch. Thanks for help!

@pseudotensor In that case, can you try out the code in #4454? See if this fixes your problem.

yes, checking now.  Thanks.

@pseudotensor

There are at least 2 ways to do that:

Thanks.  Will try to add a test later.

@pseudotensor Any updates?

@hcho3 No, I still get the error.  Will try to fix it.

@pseudotensor @hcho3 Added a test https://github.com/dmlc/xgboost/pull/4454/files#diff-50fcb70048e47cfc99375bad25bf0d4e

Great thanks!  I had trouble merging this PR into my own branch for testing, a few "can't apply" issues, not sure why since everything seems synced properly.  Will try again to test on Tuesday.

closed by #4454 .


dotnetbot sample fails to compile on Ubuntu

From @silentnull:
HI, I downloaded samples dotnetbot code and during compile with latest coreclr in native mode I have follow error: Compiling> [System.Private.CoreLib]System.Runtime.InteropServices.InteropExtensions.MightBeBlittable

https://github.com/dotnet/core/tree/master/samples/dotnetbot

@schellap Could you please take a look at this? People are that are trying to compile hello world are running into this. Do we just need to push updated corert bits somewhere?

Taking a look, surprising because the DotNet/CLI also has Ubuntu CI compiling with these exact bits.

/Cc @brthor
This is the last method compiled successfully in "Hello World" sample app as the toolchain is supposed to.
Compiling> [System.Private.CoreLib]System.Runtime.InteropServices.InteropExtensions.MightBeBlittable
The post compilation linking steps done by CLI fails trying to find the pre-requisite tools/libraries. We need better error messages than "No such file or directory" and bail out early if pre-requisites are not found. I'll keep this issue to track adding support to CLI.


Updated motion permission checks

Add permission checks in motion form (Fixes #1677)
Implement get_allowed_actions in motions JavaScript (Fixes #1668)

ok


takeEvery on channel should close the channel when the task is cancelled

Steps to reproduce the bug/issue

Create a saga generator that yields a takeEvery effect which consumes a channel
Cancel that task.
Notice that the channel has not been closed which can generate memory leaks.

Description of the bug/issue
IMHO I think that it's dangerous to encourage/allow the takeEvery effect to consume channels if the channel is not being closed when the task is cancelled. I could be missing something here, but I think that this could lead to unexpected memory leaks (which is what happened to me BTW :smiley: )
Example
Basically, I would expect these 2 blocks of code to be "equivalent", but they aren't:
export function* saga() {
  const myChannel = yield call(createMyChannel);
  yield takeEvery(myChannel, put);
  yield fork(whatever);
}

function* channelWatcher() {
  const myChannel = yield call(createMyChannel);

  try {
    while (true) {
      const action = yield take(myChannel);
      yield put(action);
    }
  } finally {
    if (yield cancelled()) {
      myChannel.close();
    }
  }
}

export function* saga() {
  yield fork(channelWatcher);
  yield fork(whatever);
}

Is this a bug or this is the expected behaviour? And if this is the expected behaviour: is there an elegant way to close a channel that's being consumed by a takeEvery effect?
Environment information

redux-saga version: 0.16

We cannot close channels automatically, because you hold a reference to the channel and we don't how it is used.
Consider this.
export function* saga() {
  const myChannel = yield call(createMyChannel);
  const ac = yield take(myChannel);
  yield takeEvery(myChannel, put);
  yield fork(whatever);
}

Hi @Andarist ! Thanks a lot for your quick response.
Dah! yep, of course, that makes a lot of sense.
However, I do think that perhaps it would be useful to add another (optional) argument to instruct the effect that the channel can be closed if the task gets cancelled, as I think that in most use cases than would be the desired behaviour. What do you think?
If not, perhaps it would make sense to add a little note in the docs to remind others that if they use takeEvery/takeLatest with a channel, it's their responsibility to close the channel (if they want to) when the task gets cancelled.

That's a good suggestion, would you like to provide a PR for this?

Yep, for sure. I will try to get some time to do that before the end of this week. Thanks!

Closing due to inactivity.  If this is still an issue then please reply and I will reopen and try to help.  Thanks!


invalid opts check is looking at the wrong array

At this line: https://github.com/spruceid/siwe/blob/main/packages/siwe/lib/client.ts#L242
it should be checking invalidOpts.length, not invalidParams.length.

You are absolutely correct. Would you be interested in raising a PR for it?


Improvement in UI for Admin panel

the ui of the Admin panel is outdated. we want to improve it and match the theme of the new home page
the student panel can be accessed by adding /admin to the base url

Hi Rohit, Can I work on this ?

Go Ahead! I have assigned the issue to you @Kabiirk .
Work on a new branch with the feature as the branch name.
When you create a pull request, request review from @rohitagr0310 , @jain-anshika or , @qaidjoharj53

@Kabiirk
The issue has been inactive for an extended period. I am reaching out to notify you that if it remains inactive by the end of this month, we will proceed to close this issue.

Hi @rohitagr0310 ,
I'm done with improving UI for Admin Panel, it looks as follows:

Dashboard

Add Book

All Books

Pending Issue

Issued Books

Manage Students (Have also created Base UI for this, someone else can populate this as per required logic)

History

Let me know if this is fine & I can move forward with the PR.

Looks great 👍


Support Slack Ephemeral Messages

Added support for ephemeral messages in below slack_comm functions
say
send_*
#56

I will look into merging this within the next week or so.

Code looks good. Will need to test locally and will then merge.
Thank you so much for working on this @viveksyngh !

Testing this today

It works! Thank you so much for working on this! :)


Change modal cancel variant

The following changes are implemented
TODO: Summary
Changes in the user interface:
TODO: Add screenshots, recordings or remove this section
Checklist when submitting a final (!draft) PR

[x] Commits are tidied up, squashed if needed and follow guidelines in CONTRIBUTING.md
[x] Code builds
[x] All existing tests pass
[x] All new critical code is covered by tests
[x] PR is linked to the relevant issue(s)
[x] Rebased with the target branch

All looks good just this:

LGTM


.font(.system(size: 50)) not working

I copied the example for setting a font size, made two LaTeX labels with different settings, and they are rendering with the same height.  I did notice a shift in line height.
       LaTeX("\\text{Euler's identity is } e^{i\\pi}+1=0\\text{.}")           .font(.system(size: 50))           .parsingMode(.all)
Using Xcode 15.2

I'm seeing the same issue as well.


data access on Snellius

(Moved discussion from PR #9 to this issue).
Pablo couldn't load the experimental.h5 data on Snellius. I hope this is fixed already, but please check by:

trying again with the same file (I changed permissions)
trying with experimental-copy.h5

If neither works, let me know and I'll reupload it.

It works now. Thanks!


Only restart rsyslog if the configuration changed and if so, do it in background

Add logic to restart rsyslog in metron_agent ctl only if the rsyslog configuration changed.
This avoid unnecesary log restarts, which can cause loss of messages and issues when restarting the metron_service (e.g. monit timeouts if rsyslog restart takes 30s or more). This PR also adds logic to remove the configuration and restart the service if the syslog forwarding is disabled, previously changing syslog_daemon_config.enable from true to false would left
rsyslog running with the latest configuration.
We also configure and restart syslog forwarder in the background.
Restart rsyslog can take more than 30s in some cases. If that happens monit will timeout starting metron_agent, when it could start properly.
For instance when using RELP, if the connection to the remote server is slow of stale, or the remote RELP server is behind a load balancer with all the backends down.

@keymon - we are in the process of splitting this functionality out of loggregator. I suggest making your pull request over at https://github.com/cloudfoundry/syslog-release


Make each keyword a link

This is a great summary of operators, but each name needs to link to its actual documentation.

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

ID: 641bdba6-a2e2-640a-abab-2535734271c5
Version Independent ID: e7973dd0-9d2f-846d-0724-4d44be81cda8
Content: Useful operators in Azure Monitor log queries - Azure Monitor
Content Source: articles/azure-monitor/log-query/useful-operators.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@amp64 Thank you for your feedback, assigning this issue to our document author @bwren to further evaluate the feedback.

Sorry for the horribly late reply. Cleaning up some issues that we missed. We actually retired this article as part of a restructure of the log query content. Articles now in the ADX content that are similar - https://docs.microsoft.com/azure/data-explorer/kusto/query/tutorial?pivots=azuremonitor, https://docs.microsoft.com/azure/data-explorer/kusto/query/best-practices.
#please-close


Load modified .spv shaders from disk

I am trying to load modded spirv shaders into dxvk by adding these codes:
//in dxso_module.cpp
Rc<DxvkShader> DxsoModule::compile(
    const DxsoModuleInfo&     moduleInfo,
    const std::string&        fileName,
    const DxsoAnalysisInfo&   analysis,
    const D3D9ConstantLayout& layout) {

    auto compiler = std::make_unique<DxsoCompiler>(
      fileName, moduleInfo,
      m_header.info(), analysis,
      layout);

    this->runCompiler(*compiler, m_code.iter());	// can this be skipped????
    m_isgn = compiler->isgn();

    m_meta            = compiler->meta();
    m_constants       = compiler->constants();
    m_maxDefinedConst = compiler->maxDefinedConstant();
    m_usedSamplers    = compiler->usedSamplers();
    m_usedRTs         = compiler->usedRTs();

    compiler->finalize();

    char* fna=(char*)fileName.c_str();
    *(int*)&fna[0x10]=0;
    auto kfna="zdk\\"+fileName;
    unsigned int fsz=FileExists_inspvlist((char*)kfna.c_str());
    if(fsz!=0)
    {
        return compiler->compile((char*)kfna.c_str(),fsz);
    }

    return compiler->compile();
  }

//in dxso_compiler.cpp
  Rc<DxvkShader> DxsoCompiler::compile(const char* existfile,uint32_t filesize) {
    DxvkShaderCreateInfo info;
    info.stage = m_programInfo.shaderStage();
    info.bindingCount = m_bindings.size();
    info.bindings = m_bindings.data();
    info.inputMask = m_inputMask;
    info.outputMask = m_outputMask;
    info.pushConstOffset = m_pushConstOffset;
    info.pushConstSize = m_pushConstSize;

    if (m_programInfo.type() == DxsoProgramTypes::PixelShader)
      info.flatShadingInputs = m_ps.flatShadingMask;

    auto vbf=(uint32_t*)ReadAllBytes(existfile,filesize);
    auto rff=SpirvCodeBuffer(filesize>>2,vbf);
    free(vbf);
    return new DxvkShader(info, rff);
  }

Seems it just works, my question is can the runCompiler(*compiler, m_code.iter()) be skipped if we load the shader bytecodes from disk?
BTW, in dx9, seems these:
if (bitfieldExtract((_86 != 0u) ? _104 : spec_state.dword1, 0, 1) != 0u)
    {
        r4 = vec4(textureLod(s0_2d_shadow, vec3(r5.xy, (bitfieldExtract((_86 != 0u) ? _112 : spec_state.dword4, 0, 1) != 0u) ? clamp(r5.z, 0.0, 1.0) : r5.z), r5.w));
    }
    else
    {
        r4 = mix(mix(textureLod(s0_2d, r5.xy, r5.w), textureGather(s0_2d, r5.xy + (vec2(0.99609375) / vec2(textureSize(s0_2d, 0) * ivec2(2)))).zxyw, bvec4(bitfieldExtract((_86 != 0u) ? _112 : spec_state.dword4, 16, 1) != 0u)), vec4(0.0, 0.0, 0.0, 1.0), bvec4(bitfieldExtract((_86 != 0u) ? _93 : spec_state.dword2, 0, 1) != 0u));
    }

can just be replaced by: textureLod(s0_2d, r5.xy, r5.w);	//texture() or textureLod(), not causing errors in my case so far.
also these after ps_main():
ps_main();
/*
    uint _2228 = bitfieldExtract((_86 != 0u) ? _104 : spec_state.dword1, 20, 3);
    uint _2234 = bitfieldExtract((_86 != 0u) ? _93 : spec_state.dword2, 26, 4);
    if (_2228 != 7u)
    {
        float _2272;
        float _2273;
        if (_2234 <= 8u)
        {
            _2272 = float((render_state.alpha_ref << _2234) | (render_state.alpha_ref >> (8u - _2234)));
            _2273 = roundEve
....
*/

can be removed too.
There should be an official implement of loading existing .spv from disk.

Why do you want to replace shaders?
We had this functionality in the past but it was a bit janky and not really useful, so it eventually got removed.

Also why do you want to remove random bits from the shaders? Those are needed. They aren't there for no reason.

People already mod the game shaders with some d3d dll proxy like 3Dmigoto,
it can replace not only the shader bytecodes but also textures, by the hashes. However 3Dmigoto is long dead now.
A fork of this repo:
github.com/NVIDIAGameWorks/dxvk-remix
is going to do the same thing, but that one seems made for shilling their own RTX thingy.
In my case, by modding the game's builtin postprocessing shaders,
I can increase the bloom intensity without putting another bloom filter over it, which is dumb and heavy (like ReShade did).
More important, there are a lot of ps3-era games which are roughly ported to pc in dx9,
their shaders contain tons of nonsense things like:
	def c20, 0.00390625047, 1.52587909e-005, 5.96046519e-008
...

    mad r2.xyz, r2, 255.0, 0.5
    frc r3.xyz, r2
    add r2.xyz, r2, -r3
    dp3 r2.x, r2, c20

that one above trys to emulate the integer operation to get luminance, which is nonsense on pc.
Those roughly-ported shaders also has things like using texldl on rendertarget, and rendertargets on pc do not have lods.

Using texldl makes complete sense actually as it avoids implicit LOD calculation.

DXVK is not a modding tool, so this is out of scope for the project.

It was not made for, but it can be used to. (So does 3Dmigoto, it was originally made for some VR hack.)
What I made is a provement that dxvk can become a "modding tool" by adding 50-lines ish codes.
I'm not going to PR the patch to this repo because I'm not really familiar with cpp codes so my codes of this looks hacky. (you can see it)
Support replacing assets will give more freedom to average users.

The problem is that lots of things in DXVK rely on the exact shaders that DXVK produces (like some of the parts you removed) and if that's not the case things will break in all kinds of subtle and less subtle ways.

The license is very permissive so you are ofc welcome to fork and make your own variant same as dxvk-remix have done if it is something you will want to pursue.

What I made is a provement that dxvk can become a "modding tool" by adding 50-lines ish codes.

Problem is, DXVK doesn't want to be a modding tool. There's plenty of other ways to mod D3D9/10/11 games that don't depend on implementation details.

So we are talking about philosophy.
 Like put a bloom filter over the game's existing bloom filter, like ReShade did.
But why should I double the resources using for that? to accelerate planned obsolescence?
Also why should I endure the lazy porting made by some dirtcheap coders? to make their employer more a happy merchant?
Simple as, Man should NOT be a slave of soulless modern mass production.

Once I created simple vulkan layer to write/read .spv, because I wanted to replace a "broken" shader in an game.
But I failed to figure out which .spv file I should modify :)
Not sure if it was correct approach, but kind-of worked in simple test.


update wrappers for 1.9

Codecov Report

Merging #25 (e4f56c5) into master (00e87cc) will decrease coverage by 3.14%.
The diff coverage is 3.70%.

@@            Coverage Diff             @@
##           master      #25      +/-   ##
==========================================
- Coverage   51.87%   48.72%   -3.15%
==========================================
  Files           6        6
  Lines         426      433       +7
==========================================
- Hits          221      211      -10
- Misses        205      222      +17

Impacted Files
Coverage Δ

src/api/ctypes.jl
100.00% <ø> (ø)

src/api/libucp_api.jl
9.09% <0.00%> (-1.28%)
:arrow_down:

src/api/libucp_common.jl
100.00% <ø> (ø)

src/UCX.jl
78.24% <100.00%> (+0.98%)
:arrow_up:

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 00e87cc...e4f56c5. Read the comment docs.


Improve Sign In/ Sign out process

We use jwt for tracking user's signed in or signed out status. Every time user go to home page we send another request to server where we check the jwt validity and redirect to user dashboard.
Problem:  Always we have to fire two request to server.
Solution: We check the validity of JWT from client itself. And remove redirect from server.
This will also solve #191
Tasks:

[ ] Write jwt validity method in client side.
[ ] Remove the redirection logic from server.

The client can't check the validity of jwt token itself, as she does not know server's secret jwt seed.

The only way to remove extra client-server roundtrip is to pass jwt in cookies.

The issue #191 can be handled by the client, as she only needs to check the expiration time fo the token, not the validity. But in general we will have to switch to cookies, so it will solve #191 automatically.

@Fak3 sorry it is not the validity. It is the expiration time :).

@subhankarb there is still no clear description of the problem from the user perspective. What do they experience here that is problematic? If this is not a user problem what is the technical issue?

@rufuspollock

Problem 1: Refresh problem

go to staging.datapackaged.com.
Login --> Dashboard
Refresh. We are not logged in.

Problem 2: Login Problem.

Go to staging.datapackaged.com/core/{package} .. If i want to login. There is no login button

FIXED.
Fixed with pull req https://github.com/frictionlessdata/dpr-api/pull/235


Empty cropper while uploading .svg files without height/width attributes

Hello,
I recently tried to crop several svg files and got an issue when the exported svg images from illustrator or any svg file that does not contain height/width attribute will render as 0x0px image in cropper.
Here's an sample SVG file :
https://image.flaticon.com/icons/svg/12/12096.svg

Hi firstly update to 8.5.0 as the background of the component was set to black for some reason which made your SVG look entirely black. Next you have to have to give the component a width for the SVG to show - it won't automatically stretch the component out like an image will (that's just a browser thing which I can't change).
So make .ReactCrop fill a container with something like flex, or just give it a fixed width and you will see the SVG. e.g. .ReactCrop { width: 500px } or <ReactCrop style={{ width: 500 }}> or if you render it in a flexbox container then make ReacCrop fill the container with <ReactCrop style={{ flex: 1 }}>


Bars a bit off

the position of the error bars is not in the centre of the expression bars, but slightly shifted downwards (this might be intentional); it kinda makes the graph look a bit wonky
UAUY: Is this easy to fix? If so we should try to have them centered.

I had a look, but it is a border case, I can  move them one pixel up as a patch.


Wrong number of args (2) passed to: core/identity

With this minimal setup on 0.1.1
(ns app.server
  (:require [ring.middleware.transit :refer [wrap-transit]]))

(defn handler [req]
  {:hey "there"})

(def app
  (-> handler
      (wrap-transit)))

This error is being thrown:
clojure.lang.ArityException: Wrong number of args (2) passed to: core/identity
	at clojure.lang.AFn.throwArity(AFn.java:429)
	at clojure.lang.AFn.invoke(AFn.java:36)
	at ring.middleware.transit$wrap_transit$fn__1644.invoke(transit.clj:204)
	at ring.middleware.cors$wrap_cors$fn__1492.invoke(cors.clj:148)
        ...

Thanks for the report. 0.1.2 should fix this.
Ultimately it needs better tests. I will add an integration test & set up Travis to test with and without Aleph.


Recipe/86: time series energy demand zero-shot

pm/#86 Add zero-shot recipe for predicting energy demand time series.

Install library from branch (to be updated)
Download and prepare the data
Construct a Preprocessor
Configure the model
Create and execute forecasting pipeline
Plot the results

See the issue for a couple remaining to do's.

Remaining to do:

Move the plotting function to tsfm-public
Pin the tsfm-public version for pip install

A few comments:

We can remove a few imports: prepare_data_splits and TimeSeriesPreprocessor
Remove comments "# tsp.num_input_channels", " # prediction_channel_indices=tsp.prediction_channel_indices," when loading model
Do we still want the link to the cookbook-dev version?

@fayvor If you want to discuss the plotting in tsfm-public let me know.

Please change the pip install to:
! pip install "tsfm_public[notebooks]<EMAIL_ADDRESS>-U

The final change, to replace plot_predictions() with tsfm_public.toolkit.visualization.plot_predictions, is waiting on this PR in granite-tsfm.

This now uses plot_predictions() from granite-tsfm, but the pip install points at main. Once we have a version of granite-tsfm that contains the new code, we can pin this recipe to that version.

@wgifford @adampingel this is pinned now, so we should be able to merge. Please give it a look.


Add missing debian-stable images

The old ocaml/opam2 never had any image called debian, it was always debian-stable. This also would simplify the special case made for debian in regards to how dockerfile-opam defines the set of tag names.
debian-stable was wildly used for ocaml-ci-scripts. For instance: https://travis-ci.org/github/realworldocaml/mdx/builds/745486303

Let's go with aliasing debian to debian-stable. Pushing multiarch manifest tags is cheap.

Done in #81

See #84 - we should just alias debian-stable to debian in ocaml-ci-scripts


Add dism.exe error handling and EnableAllParentFeatures parameter

When trying to enable a child feature when a parent feature is disabled, xDismFeature fails silently (reporting success). This pull request addresses 2 issues - errors not being handled (exit code not checked) and no ability to pass '/All' parameter to dism.exe.
Consider following example:
Configuration Test {
    Import-DSCResource -ModuleName xDismFeature

    xDismFeature IISParent {
        Name = 'IIS-HealthAndDiagnostics'
        Ensure = 'Absent'
    }

    xDismFeature IISChild {
        Name = 'IIS-HttpLogging'
        Ensure = 'Present'
        DependsOn = '[xDismFeature]IISParent'
    }
}

Test -OutputPath .\Test
Start-DscConfiguration -Path .\Test -Wait -Verbose -Force

dism /online /get-featureinfo /FeatureName:IIS-HttpLogging

The above code runs successfully, but does not install IIS-HttpLogging, because it depends on absent IIS-HealthAndDiagnostics:
VERBOSE: Perform operation 'Invoke CimMethod' with following parameters, ''methodName' = SendConfigurationApply,'className' = MSFT_DSCLocalC
onfigurationManager,'namespaceName' = root/Microsoft/Windows/DesiredStateConfiguration'.
VERBOSE: An LCM method call arrived from computer x with user sid S-1-5-21-4222474-1288197275-456279356-8911.
VERBOSE: [x]: LCM:  [ Start  Set      ]
VERBOSE: [x]: LCM:  [ Start  Resource ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ Start  Test     ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ End    Test     ]  [[xDismFeature]IISParent]  in 1.1720 seconds.
VERBOSE: [x]: LCM:  [ Skip   Set      ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ End    Resource ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ Start  Resource ]  [[xDismFeature]IISChild]
VERBOSE: [x]: LCM:  [ Start  Test     ]  [[xDismFeature]IISChild]
VERBOSE: [x]: LCM:  [ End    Test     ]  [[xDismFeature]IISChild]  in 1.0470 seconds.
VERBOSE: [x]: LCM:  [ Start  Set      ]  [[xDismFeature]IISChild]
VERBOSE: [x]: LCM:  [ End    Set      ]  [[xDismFeature]IISChild]  in 1.8280 seconds.
VERBOSE: [x]: LCM:  [ End    Resource ]  [[xDismFeature]IISChild]
VERBOSE: [x]: LCM:  [ End    Set      ]    in  5.5406 seconds.
VERBOSE: Operation 'Invoke CimMethod' complete.
VERBOSE: Time taken for configuration job to complete is 4.498 seconds

Deployment Image Servicing and Management tool
Version: 6.3.9600.17031

Image Version: 6.3.9600.17031

Feature Information:

Feature Name : IIS-HttpLogging
Display Name : HTTP Logging
Description : Enables logging of Web site activity for this server
Restart Required : Possible
State : Disabled

After applying this pull request, '/All' parameter will be passed to dism.exe by default, which will make the above scenario work (IIS-HttpLogging will be installed). For scenarios where this is not desired (old Windows?), new 'EnableAllParentFeatures' parameter (which defaults to $true) can be set to $false. For the above example, this will result in properly logged exception:
VERBOSE: Perform operation 'Invoke CimMethod' with following parameters, ''methodName' = SendConfigurationApply,'className' = MSFT_DSCLocalC
onfigurationManager,'namespaceName' = root/Microsoft/Windows/DesiredStateConfiguration'.
VERBOSE: An LCM method call arrived from computer x with user sid S-1-5-21-4222474-1288197275-456279356-8911.
VERBOSE: [x]: LCM:  [ Start  Set      ]
VERBOSE: [x]: LCM:  [ Start  Resource ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ Start  Test     ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ End    Test     ]  [[xDismFeature]IISParent]  in 1.5940 seconds.
VERBOSE: [x]: LCM:  [ Start  Set      ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ End    Set      ]  [[xDismFeature]IISParent]  in 3.2340 seconds.
VERBOSE: [x]: LCM:  [ End    Resource ]  [[xDismFeature]IISParent]
VERBOSE: [x]: LCM:  [ Start  Resource ]  [[xDismFeature]IISChild]
VERBOSE: [x]: LCM:  [ Start  Test     ]  [[xDismFeature]IISChild]
VERBOSE: [x]: LCM:  [ End    Test     ]  [[xDismFeature]IISChild]  in 1.1100 seconds.
VERBOSE: [x]: LCM:  [ Start  Set      ]  [[xDismFeature]IISChild]
VERBOSE: [x]: LCM:  [ End    Set      ]  [[xDismFeature]IISChild]  in 1.6400 seconds.
PowerShell DSC resource MSFT_xDismFeature  failed to execute Set-TargetResource functionality with error message: dism.exe failed with
code 50. Output:  Error: 50  The operation is complete but IIS-HttpLogging feature was not enabled. A required parent feature may not be
enabled. You can use the /enable-feature /all option to automatically enable each parent feature from the following list. If the parent
feature(s) are already enabled, refer to the log file for further diagnostics. IIS-HealthAndDiagnostics
    + CategoryInfo          : InvalidOperation: (:) [], CimException
    + FullyQualifiedErrorId : ProviderOperationExecutionFailure
    + PSComputerName        : localhost

The SendConfigurationApply function did not succeed.
    + CategoryInfo          : NotSpecified: (root/Microsoft/...gurationManager:String) [], CimException
    + FullyQualifiedErrorId : MI RESULT 1
    + PSComputerName        : localhost

VERBOSE: Operation 'Invoke CimMethod' complete.
VERBOSE: Time taken for configuration job to complete is 8.121 seconds

Hi @mgr32, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

Powershell ISE is evil - it converted MSFT_xDismFeature.schema.mof to UTF-16 without saying anything, which made build fail. Fixed in two more commits.

It is required that you provide adequate test coverage for the code you change. Contribution Guidelines -Adding test coverage for DSC resources  Please work with us to come up with a plan to cover this code.

Please resolve any merge conflicts.  Instructions per GitHub:
Step 1: From your project repository, check out a new branch and test the changes.
git checkout -b mgr32-dev dev
git pull https://github.com/mgr32/xDismFeature.git dev

Step 2: Merge the changes and update on GitHub.
git checkout dev
git merge --no-ff mgr32-dev
git push origin dev

If there is no update of the pull request within 7 days, we will close the pull request.  After that, if you have an update, feel free to reopen the pull request or a new pull request.

Filed #8 for this
Closing because the pull request has not been updated for feedback given for 21 days.  Once you update the code, please feel free to reopen the pull request.  Thanks, Travis.


Rebuild the Ultraviolet bundle

This change updates the UV bundle to one built with Webpack's production mode, which means sites using Ultraviolet-Static will experience faster load times (on the first visit anyway). In other words, this commit reduces UV's file size for a better user experience.

A compilation approach will be taken with this repository to eliminate including uv.bundle.js in the source code.


Unable to add Markdown Plugin via Reptar Plugin

In version 2.3.0, you added 1638e12 to expose the Markdown instance to add extensions/plugins for it via the plugin API.
So far I made the following plugin: reptar-table-of-contents, which is simply four lines following the example usage in 1638e12:
module.exports = function(Plugin, options) {
  Plugin.markdown.configure(function(md) {
    md.use(require("markdown-it-table-of-contents"), options);
  })
};

In my site (https://github.com/AlahmadiQ8/reptar-site), I intsall reptar-table-of-contents via npm and add the following in my _config .yml:
plugins:
  html-minifier:
    enabled: true
    options:
      collapseWhitespace: true
      collapseBooleanAttributes: true
      removeAttributeQuotes: true
  excerpt:
    enabled: true
  table-of-contents:
    enabled: true
    options:
      includeLevel: [2, 3]

Unfortunately, this still doesn't work. I'm not sure what I'm missing.. Adding [[toc]] in my markup still does not render table of contents.

So I did some debugging...
I was able to get this to work in my local blog by first npm install --save markdown-it-table-of-contents and then inside of my local _plugins directory I copied your plugin file and that worked for me.
I think the issue is that you have not published your npm plugin to npm so when you go to install it with npm it doesn't get installed. https://www.npmjs.com/search?q=reptar-table-of-contents

@hswolff  Thank you...
I only installed it locally as in  "reptar-table-of-contents": "file:../reptar-table-of-contents".


metrics-server logs "unable to fetch pod metrics for pod : no metrics known for pod"

Hey everyone,
I am trying to setup the metrics-server on my microk8s single node cluster. The metrics work for the some pods (see screenshot). Is it container depended if the metrics-server works?
Greetings
Knerd

inspection-report-20200910_130126.tar.gz

I checked the dashboard and only one namespace is affected. The config is attached.
lovingreaper.shop.zip

Apparently deleting the replicasets and recreating them helped. So you can ignore this issue.


missing some imports for sample to run

Thanks for this sample. I tried it and found that some imports are missing - no big deal, but just to avoid people to search for it:
import cv2
from PIL import Image
import numpy as np

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

ID: 754dd7d5-dd8b-4a75-5d79-0ba621a847cd
Version Independent ID: 3cca3535-1c16-2da1-46c6-421cbf808bc3
Content: Run TensorFlow model in Python - Custom Vision Service - Azure Cognitive Services
Content Source: articles/cognitive-services/Custom-Vision-Service/export-model-python.md
Service: cognitive-services
Product: unspecified
GitHub Login: @areddish
Microsoft Alias: areddish

@Supraconductor Thanks for the feedback! I'll get it updated to include the imports.


Add Critical path configuration menu and critical path view

Note that code is hard-coded. A generic way is needed to configure
the menu and command-handler driven by the server.
Alternatively, this feature could be added in a third-party, trace
compass specific front-end extension(s) (i.e. Theia extension/Vscode
extension).
Requires: https://github.com/eclipse-cdt-cloud/tsp-typescript-client/pull/104
Signed-off-by: Bernd Hufmann<EMAIL_ADDRESS>

Just a thought. The time graph entry metadata for the thread status data provider has already the TID. It misses the hostId. Maybe, it's not necessary to retrieve the context if the host Id is provided as well.
We could the have only the menu definition (coming from the server, once implemented) indicate that these parameter are meant to be included query parameters for other data providers.
The only disadvantage is that the hostId is redundant and will increase the size of the data model return from fetchTree().

PoC only, not needed at the moment.


Map not loading

http://www.aba-icc.org/about-the-icc/situations-cases/
And for some reasons Austria pops up below too.

ok weird. Am I going crazy? Why do we have the map on two separate pages?

duh. figured it out. Somehow Austria got some content in it and showed up on the page, and that, because it didn't really have content, screwed up the page. I'll see if I can fix it so that sort of thing won't happen.


Extend Foldable (foldl', null, length)

The Foldable instance was slightly extended for issue for Issue #222 out of boredom. Namely adding foldl', null, length, on both HashMap and HashSet. Due to its usage in HashSet, I assumed the INLINE pragma was appropriate.

Why not also foldl, foldr', and foldMap? While we're at it, we might consider mapM and sequence.

Er.... No, we can't do anything special for mapM or sequence. But the other Foldable methods would be good.

I assume that would require implementing those functions somewhere in the Base.hs files, and since I'm a newbie to Haskell, and with the library overall, I didn't feel comfortable doing that. Sorry for the trouble.

That is true. We'd better do that later.


MAXPLAYERS=x doesn't work

Hello @dasisdormax
I just ran a server with MAXPLAYERS=18 set in casual.conf preset but server is allowing 64 players . it seems this option is not working . How do I set max players ? Also how do we set reserved slots in msm ?
Here is my full casual.conf :
#! /bin/bash
## vim: noet:sw=0:sts=0:ts=4

# (C) 2016-2017 Maximilian Wende<EMAIL_ADDRESS>#
# This file is licensed under the Apache License 2.0. For more information,
# see the LICENSE file or visit: http://www.apache.org/licenses/LICENSE-2.0

#~~~~ GAMETYPE and GAMEMODE: These set the general gamemode to be used
#~~~~     See gamemodes.txt for a list of valid number combinations
#~~~~
#~~~~     Note that many gameplay-related settings will be loaded from
#~~~~     your instance's cs2/cfg/gamemode_<xyz>_server.cfg
__GAMETYPE__=0
__GAMEMODE__=0

#~~~~ MAXPLAYERS: The number of active players allowed, excluding spectators
#~~~~             The default value is gamemode-dependent
__MAXPLAYERS__=18

#~~~~ MAPGROUP: The mapgroup to be played, primarily used on vanilla servers
__MAPGROUP__="mg_asli"

#~~~~ MAPS: A list of single maps to be played. This is used to generate the mapcycle file
#~~~~       If no MAP is given, the first entry will be loaded on server start
__MAPS__="de_mirage de_dust2 de_inferno"

# Customize your GOTV Delay here. The default for competitive is 105 seconds.
# __TV_DELAY__=60

Hi @lostsoul6

I just ran a server with MAXPLAYERS=18 set in casual.conf preset but server is allowing 64 players . it seems this option is not working.

Thanks for reporting, that's indeed a bug. The way the script sets the maxplayers does not work with CS2 anymore.

Also how do we set reserved slots in msm ?

Reserved slots are a feature from Metamod and Sourcemod and (as far as I know) not a built-in feature of the Counter-Strike Server. So, you'll have to check the Metamod/Sourcemod instructions for that.

@dasisdormax any fixes for maxplayers?

Should be fixed with the latest commit. https://github.com/dasisdormax/cs2-multiserver/commit/47f6007497763a7c6f2f12e1901a0e15496020a3


Subscriptions

Hey!
We've seen that there haven't been any updates for quite some time. Does it still work on iOS 13?
Does it support subscriptions still on the latest iOS?
We'd love to use RMStore as it has worked out perfectly for us in the past.

Hello, yes it's work perfectly on iOS 13 (tested on iPhone & iPad).

@istornz awesome! Have you also tested subscriptions? :)

@ForestRingGames Yes I have an auto renewable subscription & consumable product. Both works 👍


Toggle reverse based on scroll direction

Hi,
Is there any examples or guidance on toggling the reverse based on scroll direction (up/down) using Marquee3000?
Here is a site that does it: https://locomotive.ca/en
Similarly, it would be great to see if anybody has been able to adjust the speed temporarily based on current scroll speed, similar to https://index.studio/about.
Thank you,
Elliott

Hi,
Is there any examples or guidance on toggling the reverse based on scroll direction (up/down) using Marquee3000?
Here is a site that does it: https://locomotive.ca/en
Similarly, it would be great to see if anybody has been able to adjust the speed temporarily based on current scroll speed, similar to https://index.studio/about.
Thank you,
Elliott

Hello eliott,
lt's a late answer but I guess you can achieve that but updating the data-reverse property and hitting the update method.
You can easily get the scroll direction or scroll velocity by listening to the scroll event.
cheers


1.2.0 build failed in Android

Steps to Reproduce

Device : Android 13
SDK : Flutter 3.3.10
targetSDK : 33

add dependency 1.2.0
build android

Expected results
build success
Actual results
Build Failed
e: /Users/gimjeongtae/.pub-cache/hosted/pub.dartlang.org/camerawesome-1.2.0/android/src/main/kotlin/com/apparence/camerawesome/cameraX/CameraPermissions.kt: (175, 22): Not enough information to infer type variable T
e: /Users/gimjeongtae/.pub-cache/hosted/pub.dartlang.org/camerawesome-1.2.0/android/src/main/kotlin/com/apparence/camerawesome/cameraX/CameraPermissions.kt: (175, 41): Cannot infer a type for this parameter. Please specify it explicitly.

but 1.1.0 is working well~

Hi, you need to upgrade the kotlin version and gradle version as mentioned on the issue #221.

Upgrade in build.gradle kotlin version to ext.kotlin_version = '1.7.10'
And classpath 'com.android.tools.build:gradle:7.2.0'

Same issue on my end. I'll try what you mentioned

It fixed my problem thanks!

@apalala-dev it work very well thx 👍🏻


feat: add support for python 3.11

Blocked by https://github.com/pytorch/pytorch/issues/86566

this is done for the core package.


LUT as a daughter of XOutput

WIP

All tests are Ok except MTPA one

So you still need PostLUT to setup a LUT?! I would prefer a dedicated setter method or the 'normal' way to setup a simulation.
Still I'm not sure about the LUT attributes (e.g. Phi_dqh_mag) in case a simulation is rerun. I.e. if one e.g. change mesh fineness and do 'LUT.simu.run()'. Then the LUT attributes won't be updated.
OT: Have you checked the load performance with the changes in import.

Hello Sebastian,

So you still need PostLUT to setup a LUT?! I would prefer a dedicated setter method or the 'normal' way to setup and run a simulation.

We continue to investigate the possibility of the LUT as daughter of XOutput an I starting to think about the following behaviour:

Create a LUT of the correct type
Call a method "set_default_simulation(machine, OP_matrix....)"
LUT.simu.run
Rather than creating a LUT object, the PostLUT would clean the non necessary contains of the XOutput/LUT object

Regarding loading performance, it depends on the size of the data stored within the LUT. Anyway we need to carry on the work on h5 save/load when we will have some time.
Best regards,
Pierre

Hello Pierre,
yes I saw the PostLUT cleanup. But I think this is the only benefit of PostLUT.
I think I got your described behaviour implemented in my proposal so far except the set_default_...
Regarding load performance I did some profiling and if I'm correct, the import of classes take the most time (independent of actual size of data). So I will redo this test with your PR to compare.
BTW: Did you see my PR on EEC_2? If you don't like it just close it (without worries :-) ).
Best regards, Sebastian

Hello,

yes I saw the PostLUT cleanup. But I think this is the only benefit of PostLUT.
I think I got your described behaviour implemented in my proposal so far except the set_default_...

Then I will update the code to match this behaviour. (I will maybe not add the set_default_simulation at the end, there would be too many input arguments). Is there any other modification needed to ease your implementation ?

Regarding load performance I did some profiling and if I'm correct, the import of classes take the most time (independent of actual size of data). So I will redo this test with your PR to compare.

We plan to check save/load h5 at the end of the month, all the data/information that you can provide on this topic would be very appreciated to help us decide what we need to optimize (maybe we can open a dedicated issue ?). I don't understand how "the import of classes" can take time, do you have any idea ?

BTW: Did you see my PR on EEC_2? If you don't like it just close it (without worries :-) ).

Sorry for the PR, as you mentioned "without unit test". I thought that it was a Work in progress ^^" It's merged now thank you for this update :) Regarding your other PR, it is still WIP by waiting the merge of this PR ?
Best regards,
Pierre

Hello Pierre,
the set_default_sim. method is of course not mandatory, but could be helpful. Also I would not include all parameters but only the most important ones (e.g. current and speed,). But we can also add such method later.
For my PR, I think you can cancel that for now since you did the most important changes to EEC/LUT for now by yourself.
Nevertheless I may use some of the code to introduce an EEC_PMSM_2 class with my ideas later.
The only thing you could add to LUT methods would be an optional 'machine' argument to get fluxlinkage, inductance, ... for some similar machines (e.g. different length or number of turns). I will take some of the PR's code to introduce the respective functionallity.
For load/save, I can provide the profiling results. But I got some issues using h5 that I may need to investigate first.
Best regards, Sebastian

I have updated the PostLUT to clean the current output and added the set_default_simulation method. I need to run the tests again and MTPA function still need to be updated

The tests are Ok and we will solve the MTPA in another PR


no consistent values for accelerometer (iphone7 vs nexus5)

Hi! I used gyronorm.js to get consistent values for the acceleration accross different devices. The gyronorm function puts out different values than the standard system values, but there's still a big difference between, for example, iPhone7 and Nexus 5 acceleration values.
Here's a fiddle with the code:
https://jsfiddle.net/mo2owLf6/2/
Here's the same code as a demo to check on smartphones:
http://demo.sansho.studio/acc
the gyronorm.js maximum output value is:
iphone: around 90
nexus5: around 40
Is there any chance to get the values consistent?
Thanks!

Hi there,
I dont have access to any of the devices. I test the link on iPhone 6S using Safari.
The values are very similar (72.9 vs 73.1). I think this is acceptable.
Below are some questions/suggestions. But before that I have to say it is a mess how different devices and browsers handle these values. Gyronorm addresses some of these differences; unfortunately not all. You can read about it here https://leanpub.com/gyronormjs
Those cases will need additional assumptions depending on the logic of individual app. This you need to implement on your side.
Here are some questions/suggestion:
Does any of the phones give you correct values? Like I am having with 6S.
Did you try using the acceleration including gravity instead dm.gy instead of dm.y
Can you check if the device is returning correct values outside of browser. With an application like this https://itunes.apple.com/us/app/sensor-kinetics/id579040333?mt=8
I'd suggest to test it with more devices and see what is the consistent value over as many devices as possible. Then you can pinpoint the reason for different values.
Let me know if any of these help.
Cheers,
Doruk

Hi Doruk,
thanks for the quick reply!
I just started working on a list with both values on different devices. Quite interesting!
https://docs.google.com/document/d/1PeSqsbWIpaGbV10OUfobyi-TMK2PDvtcQTNX3gHJGSg/edit?usp=sharing

That is a great list indeed!
I see indeed the devices return different values. I think you can address this in the application logic. Not sure what your use case is; so no concrete suggestions.
Since it is not directly related with Gyronorm I will close those issue.
Feel free to go on posting here and/or re-open it.
And I am very interested about the use case and how/if you find a solution.


Connect new task & edit task to firestore.

ConfigureTaskScreen
new task:
call task service to store the new task to firebase, and the project service to update the project on firebase with the new task (NB: in firebase project.tasks are stored by foreign key (taskId).
edit task:
call task service to store the new task to firebase. No need to update the project in firestore here.

NB: Dependent on #125.


(1.7.10) Builder block missing blocks.

A Builder block was set up with a Shape Card set to "Solid Box" to act as a filler to fill the caves under my base since for some reason there were a rather large amount of them this was done to help with mob spawning near my base however this didnt really decrease mob spawning with a completely lit up surface so i installed a xray mod to see if anything went wrong or some mistake on my end but ive double and triple checked the card and other settings and everything is set up to be a filler correctly but the builder block left two high gaps randomly in the selected area (256x256x60 cube) and other random one block gaps as well in this same area the only fix i found for this is to select the builder to make multiple passes on the gui but outside of using some sort of Xray to see if its actually finished the job properly or not there is no way to tell  since the builder will say its done wether its filled every single block in or not on the first pass and yes the builder block is in a always loaded chunk so that isnt the issue ^^ I apologize if its a feature but it seemed really strange to be a feature so i figured i would report it this bug appeared in for me project ozone 2 reloaded idk how that makes any difference but i thought i would include it I can add pictures if requested of what exactly the builder does when it misses these gaps

One possible issue that comes to mind with using the Builder to fill in
caves, is if while placing things it causes an update that disturbs a
block such as gravel, that could then fall, leaving gaps the Builder
wasn't expecting. I don't know how it calculates the fill work, so could
see it getting out of sync with such changes in the world.

What Brian says is a possibility and not much I can do about that (especially not in 1.7.10 as I no longer work on that versoin)

where would I find a Download for 1.7.10? I actually came here from CurseForge and none of the listed versions say they are for 1.7.10.


Code Quality : Introducing common base for Grid and Details Layout

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Related to #10305

Validation
How did you test these changes?

[x] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[ ] Are there any other steps that were used to validate these changes?

Open app ...
Click settings button ...

@lukeblevins fyi 👀

Closing the PR to redo it with a cleaner version of Files.


Server error for babel-preset-es2015-loose

https://bundle.run/babel-preset-es2015-loose

Looks like it's specific tot his version. Appending @6 or @7 works.

Yes, I have this error on master:
packd error [babel-preset-es2015-loose] Cannot find module 'babel-core' from<EMAIL_ADDRESS>
It looks like the issue comes from babel-preset-es2015-loose@8 itself.

FYI I've just pushed an update so that error messages are visible on the 500 page:

Should make these things a bit easier to track down in future.

Yes, that's great, thanks.
I think we can close this issue. I created an issue on the preset's repo but I don't really except a fix since there's no activity since one year.

Nice tweak, Rich!
Thanks folks.


Can I destroy instances by property rather than id?

 Reading the docs, it seems instances can only be destroyed by id with the destroy() method. Can anyone shed light on if it is possible to destroy by property?
Ideally, I'd like to run something like this:
Model.forge().where({name: 'foo'}).destroy().then(function() {
    // etc
});

Managed to find a solution here: https://joepettit.com/bookshelf-js/
new Model({name: 'foo'}).fetch().then(function(instance) {
    Model.forge().where({id: instance.id}).destroy().then(function() {
        // etc
    });
});


Where can we find the ML metric explanations?

Dear Azure developers,
    Thanks for offering various ML metrics. Is there a centralized place for we to look at the ML metric meaning? Such as balanced_accuracy, weighted_accuracy, macro_AUC, micro_AUC?

Thanks!
Anna

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

ID: 52d75487-5a8d-f735-bc34-8a5d90c52d86
Version Independent ID: b651cdfb-6d3b-8ce9-f665-be07b1f11273
Content: Understand automated ML results - Azure Machine Learning service
Content Source: articles/machine-learning/service/how-to-understand-automated-ml.md
Service: machine-learning
Sub-service: core
GitHub Login: @nilesha
Microsoft Alias: nilesha

@annaymj  You can go through the following classification metrics and there description available in the following document: https://docs.microsoft.com/en-us/azure/machine-learning/service/how-to-understand-automated-ml#classification-metrics.
Hope this helps.

Perfect! Thanks a lot for the quick response!

@annaymj We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Prevent zombie chrome processes with dumb-init, s6, or similar

The Official Puppeteer Docker guidance recommends using dumb-init as an entrypoint to prevent chrome from spinning up and being unable to terminate processes. Adding it here will make the image more robust.

fixed


⚠ Bofrost.fr has degraded performance

In 3b442c6, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1135 ms

Resolved: Bofrost.fr performance has improved in 33aa31d.


correct storage acct limits

Updating to reflect ARM storage account limits as documented at https://docs.microsoft.com/en-us/azure/azure-subscription-service-limits#subscription-limits---azure-resource-manager

@jmutchek-msft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@jmutchek-msft - Thanks for making that fix.
#sign-off


[v3.2.2] Updated Composer helper commands

#38  👈
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @GoodM4ven and the rest of your teammates on  Graphite


[Request] Additonal installation method

I've created an asdf plugin asdf-bin, which can help install multiple versions of bin, which developers might find very handy. So, in the installation section of the README.md, as an alternate/unofficial method you can add the possibility of installing bin using asdf.
## Installing

1. Download bin from the [releases](https://github.com/marcosnils/bin/releases)
2. Run ./bin install github.com/marcosnils/bin so bin is managed by bin itself
3. Run bin ls to make sure bin has been installed correctly. You can now remove the first file you downloaded.
4. Enjoy!

### Alternatively ...

If you're an [`asdf`](https://asdf-vm.com) user, you can simply follow these steps:

  ```console
  $ asdf plugin add bin
  $ asdf install bin latest
  $ asdf global bin latest
  $ bin --version
  bin version 0.16.2
  commit: d1213c11e63483174ed95308c9eb6372bb9b16e9
  built at: 2023-02-21T13:11:19Z
  built by: goreleaser
  .```

has it been a request for this? It shouldn't make a lot of sense for bin users to have multiple version of it.
I prefer no to merge this unless there's a clear need from more users. Thx for the kind request!

If you want have multiple versions of bin, you can check rtx or asdf.
rtx plugin add bin https://github.com/joe733/asdf-bin.git
rtx ls-remote bin
rtx i bin@latest


Add useful information in Checkpoints messages for Partitions

https://github.com/broker-game/git-broker-client/pull/108


Configurable API prefix for kubectl proxy

Makes the API prefix in the proxy more customizable for apps that want to serve static content & serve the API under a different URL. Defaults to /api/ to retain existing functionality.

LGTM. I am curious though; why would an app want to change the api prefix?

For one app I'm working on we're embedding the API proxy so we can take advantage of the service/pod proxying as well as the API proxy. We'd like to use /api for our own internal app's API & /api/kubernetes for the embedded kubernetes API proxy & so need to be able to change the API prefix.

For one app I'm working on we're embedding the API proxy so we can take advantage of the service/pod proxying as well as the API proxy. We'd like to use /api for our own internal app's API & /api/kubernetes for the embedded kubernetes API proxy & so need to be able to change the API prefix.

Nifty. Travis/Shippable is happy, so merging.

Thanks @jlowdermilk!


[ADD ME TO THE ORGANIZATION]

Name: Akash Sharma
Location: India
Interest: Full Stack, App Development, Open Source
Discord username(optional):- AkashSharma#1511
Why do you want to join?

I want to join because I want to contribute some open source and learn from the community. I enjoy collaborating with other programmers.

You can also join our Discord community here
Feel free to check out other cool repositories of the Open Source Community here.

@AkashSharma001 Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


feat(server): implement

This branch is based on the protocol branch and implements the protocol described there.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


code block in listing is broken in "Set up RKE2 for Rancher"

Summary:
It seems like it fails to render a code listing inside a numbered list.
In the article  the is list starting with  To join the rest of the nodes.
Link to the code: docs/rancher/v2.5/en/installation/resources/k8s-tutorials/ha-rke2/
I am not shure if this is a upstream bug in hugo or an error in the markdown code, so I am posting this here first.
Details:
1. To join the rest of the nodes, you need to configure each additional node with the same shared token or the one generated automatically. Here is an example of the configuration file:
```
token: my-shared-secret
server: https://<DNS-DOMAIN>:9345
tls-san:
  - my-kubernetes-domain.com
  - another-kubernetes-domain.com
```
After that you need to run the installer and enable then start rke2
```
curl -sfL https://get.rke2.io | sh -
systemctl enable rke2-server.service
systemctl start rke2-server.service
```

1. Repeat the same command on your third RKE2 server node.

When adding a newline over the code block it gets rendered, but the listing is broken and restarts the counter.
Notes:
Github renders the block, but then restarts the counter:

Closed via https://github.com/rancher/docs/pull/3379


Syntax error for seemingly no reason?

I'm getting this error when it should work? I think I've installed everything properly.

what os?


Add pagination for silences list

What did you do?
So I'm writing a tool called grafana-interacter and I need to be able to query silences with pagination (so, be able to specify limit/offset via query params) and only get a slice of silences and ideally some metadata like limit/offset.
From the OpenAPI docs https://github.com/prometheus/alertmanager/blob/main/api/v2/openapi.yaml#L61 it doesn't seem like it's possible, so it would be lovely if the silences endpoint can provide this.
Something like this would be ideal, but also API breaking (query params won't be breaking, but changing the response format would):
# Query: either this
GET /api/v2/silences?page=0&per_page=50
# or this
GET /api/v2/silences?limit=50&offset=0

# and the response
{
    silences: [], // array
    meta: {
        total: 95, // the total amount of silences
    }
}

What did you expect to see?
What did you see instead? Under which circumstances?
Environment

System information:
insert output of uname -srm here

Alertmanager version:
insert output of alertmanager --version here (repeat for each alertmanager
version in your cluster, if relevant to the issue)

Prometheus version:
insert output of prometheus --version here (repeat for each prometheus
version in your cluster, if relevant to the issue)

Alertmanager configuration file:

insert configuration here

Prometheus configuration file:

insert configuration here (if relevant to the issue)

Logs:

insert Prometheus and Alertmanager logs relevant to the issue here

Pagination is a much needed feature, but as mentioned, it is a breaking change that will affect existing clients. I suspect it we will have to release another API (APIv3) with pagination support as updating APIv2 will be too disruptive.


Jamie adjust petrinet transition matrix rectangle widths

This is something Pascale asked me to look into a while back. Finally figured it out. @mwdchang you may want to make sure I didn't break anything... but it looks good here.
BEFORE

AFTER

@jamiewaese-uncharted there are a couple matrix configurations that look a little odd.

Also note the notation we use is #-rows X #-columns, your before picture had the numbers flipped  ... I am assuming that is just a typo and we don't actually want to calculate the matrix-transpose?

Fixed the issue you noted above, when rows are greater than cols
Removed the labels. They were too small to be useful and added clutter.
Removed the border-radiuses

fyi

@mwdchang I fixed the last errors. Do you mind giving it a review please?


[5.5] Enable Effectful Properties by Default (SE-310)

Cherry-pick of https://github.com/apple/swift/pull/37225 for release/5.5 branch.

@swift-ci please test

@kavon it can be a separate PR, but can you please update the ChangeLog accordingly?


New addition 4

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Adding feature & update readme.md
Why was this change needed? (You can also link to an open issue here)
hello sir,
I have added some more API boxes of the repo datas like
GitHub issues
GitHub closed issues
GitHub pull requests
GitHub closed pull requests]
Other information:
Sir please take  a review of My PR & please merge it.

@dartpain Please take a review of my PR & please merge it

I do appreciate your commit a lot. But I dont think its very necessary as those numbers are already displyed by Github itself.
Thank you so much!
Please check out docs/pages for possible places for contributions.


⚠ GitLab Server has degraded performance

In fe80463, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 5670 ms

Resolved: GitLab Server performance has improved in d3d5e3d.


Add tests for helpers.js

The test failed due to a flaw in the test itself. I'll fix the test and update this PR with the new version once it is available.

wow, this is awesome; don't get a lot of PRs that expand test coverage, thanks @gitname!

 Thanks for merging all of my PRs.


typo in publisher name iceoryx-project-template/src/publisher.cpp

--------------------------------------------------------|
-------------------------------------------------------\ /
constexpr const char APP_NAME[]{"publihser"};
constexpr const char APP_NAME[]{"publisher"};
Required information
irrelevant

@GaryMatthews Thanks for reporting the typo! Would you be interested to create a PR to fix it?


Add validation for StatefulSet queue-name to fail fast.

What type of PR is this?
/kind bug

What this PR does / why we need it:
Add validation for StatefulSet queue-name to fail fast.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Added validation for StatefulSet queue-name to fail fast.

/cc @mimowo

/cherry-pick release-0.9


482 grpcurl third party protos

Description
Adding query.proto for authz and bank modules to support grpcurl queries.
closes: #482

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Review Codecov Report in the comment section below once CI passes

sorry i am mentioning this even, but the feature is not a sub feature of top level feature(docker single) i think

sorry i am mentioning this even, but the feature is not a sub feature of top level feature(docker single) i think

yeah... looks to me like you need to make a minor adjustment on the indentation here


istio-ingressgateway on EC2 instances k8s does not take LoadBlancer for service

Hi there folks , looking for some help,  I trying istio on top of EC2 instances but the load balancer seems like is not providing anything to the istio-ingressgateway:

So read about the problem and I found the recommendation of annotation on the svc in order to get the elb but I'm a little confused if this is for every time I prepare a cluster to istio? and where to dig more about this issue example or docs?
thanks in advance to any chide

Sorry for not helping much here but this is purely a Kubernetes concern that you will need to fix on that end; Istio does not provision load balancers, it just sets Type=LoadBalancer. Your cluster needs to actually provision it; if you are not using a cloud lb you can use something like metallb


youtube playlist

Hi,
I'm trouble with downloading youtube playlist.videos download without audio file.I want 480p quality so I tried only that quality.
here is some of I downloaded youtube play list
https://www.youtube.com/watch?v=tp3Gw-oWs2k&list=PL6gx4Cwl9DGDiJSXfsJTASx9eMq_HlenQ
Here is the code I tried
youtube-dl -citf 135 --playlist-start 1 --playlist-end 23  "https://www.youtube.com/watch?v=tp3Gw-oWs2k&list=PL6gx4Cwl9DGDiJSXfsJTASx9eMq_HlenQ"
please help me.
thank you

videos download without audio file.I want 480p quality so I tried only that quality.

the 135 format only contains the video, if you have ffmpeg installed you can use -f 135+bestaudio to get the video and audio formats and merge them.
If that doesn't solve it, please explain with detail the problem you have and the output you get if you run the command with the --verbose option.

now it's working properly...
thanks alot........ :)


[ELE-527] High memory consumption when running edr

Describe the problem
Seems some of the users running edr are experiencing too high memory usage. Affecting their ability to run in CI / other dockers orchestration, but not so good anyway
Describe the solution
needs to be investigated first
Additional context
Happened in 2 users in support over the last two days:
Jake - see https://elementary-community.slack.com/archives/C02CTC89LAX/p1678895828557069
Oliver - see https://elementary-community.slack.com/archives/C02CTC89LAX/p1678963756105439
From SyncLinear.com | ELE-527

@oravi :
Ok here is my quick analysis for now when working with our CI data set -

On disk html / json files are between 12-13 MB each
send-report starts with around 100MB even before doing anything (is it the Python interpreter? click? imports we are doing? not sure yet but haven't looked it it thoroughly yet)
The total memory in RAM was around 350MB (27x on the report size!)
Expectedly data_monitoring.send_report increments the memory by 250M

Expectedly the heaviest part there was get_all_test_results_db_rowswith 219MB

There it was the run-operation with get_test_results with 196MB
Loading it from strings to dicts using json load added additional 21MB

You can see there that this line -->
logger.debug(f"Output: {output}")
Added alone ~114 MB

Removing it reduced the memory by 1/3 (with my local setup) -

We are facing the same problem on Elementary 0.12.0. What was the solution here?

I've been pinned to v0.6.5 since hitting this issue. I think setting --days-back to 3 (or less than 7) may help, but I haven't had time to come back to this and test any other fixes.

Have the same problem! Tried with the latest version (0.14.1) and could only get 4 last days of data with 5GB RAM in kubernetes. 🙃

I am running into the same problem with elementary-version 0.15.1 (using bigquery adapter). I am looking with last 7 days of data and 8Gi memory OOMs


the dimension of retrieval token is 512 or 768?

Thank you for opening up this amazing project. But the dimension of the retrieval token in the paper is 512, but after reading the code, I believe that the retrieval token used may have a dimension of 768, or is there an additional operation on the class token to map the 512 dimension token for retrieval

Thank you for following our research and asking questions! The dimension of 512 refers primarily to this code:
https://github.com/buptLinfy/ZSE-SBIR/blob/88f87f769208f9de5b08f2d7f0569a8087ee5b4b/model/model.py#L33-L36
As for the retrieval token, there is indeed a small mistake which should be marked as d=768.


Remove duplicate mention of svgus

NOTE: Each discrete (stand-alone) request should be in its own issue.
Put "Remove ____" as Title
[X ] Done
Explain why it should be removed
There are two mentions of svgus obviously
Additional context

@rickyngan thx!


Remove run_to_queue and run from AIService class

I wanted to clean up what it means to be a FrameProcessor vs what it means to be a pipeline. Here's my thesis:

a FrameProcessor consumes a single frame and emits 0 or more frames
a pipeline sends Frames to FrameProcessors

The AIService class muddied this a little bit (because originally it was both a frame processor and a one-step pipeline).
This PR does a few things:

removes the run and run_to_queue methods from the AIService class.
updates the queue_frames method on the Pipeline to consume an iterable rather than a Frame or a List
adds a pipeline parameter to the transport.run method. If a pipeline is provided, the run method will set its queues to point to the transport, run it until it ends, then exit.
adds a say convenience method to the transport, rather than an AIService.
updates the samples & apps to use pipelines in places where they'd used run_to_queue.

I'm approving this solely based on the fact that I knew we'd get to transport.run(pipeline) eventually 😂


URL wasn't passed to FtpSrv from CLI

Starting  from CLI as unprivileged user resulted in a EACCES error. This was a symptom of the URL not being passed correctly to the FtpServ class. Fixed in this patch.

Thanks for the PR. I already have one for this, I just haven't had a chance to pull it in!
https://github.com/trs/ftp-srv/pull/130


Add AccountId type

Let's not merge this until we sort out #352.

I disabled auto-merge for the PR. No reason to make changes to something where we can't agree on the design.


Refactor HelixBrokerStarterTest

Use condition based wait instead of sleep to fix the flakiness of the test
Fix the wrong assert which causes the test to last for more than 1 minute
Use the TestUtils.waitForCondition so that the test can fail when condition cannot be met
Reduce the default interval for TestUtils.waitForCondition to accelerate the test
Remove some un-used class

Codecov Report

Merging #4473 into master will increase coverage by 0.01%.
The diff coverage is n/a.

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


StdLib sprint - Week 13 - 2019

HTTP

[x] Study HTTP/2 flow control principals and netty implementation of that
[x] Verify OAuth2 client authentication
[x] Discuss on Authn and Authz filters, handlers engagement

GRPC

[x] #14494 UnaryBlockingBasicTestCase.testNonBlockingBallerinaClient fails
[x] #14640 Intermittent failure - ballerina-examples/grpc-server-streaming

Message Transformation

[x]  Implement XSLT support
[x]  Add test cases

Scenario Test

[x] Add Kafka BBG in TestGrid dev environment.
[x] Add Kafka BBG in TestGrid production environment.

Data

[ ] Identify the tests/grouping to be added on TestGrid for each database
[ ] Figure out a way to parameterize the tests so that same set of tests can be run on all databases
[ ] Implement the tests on TestGrid

Socket
Messaging

[ ] NATS durable subscription
[ ] Finalizing NATS v2 design
[ ] RabbitMQ writing tests (on TestGrid)
[x] Implement Artemis compiler plugin
[x] Implement JMS compiler plugin
[x] Add examples for Artemis connector

SOAP

[x] Fix issues in existing SOAP client connector. https://github.com/wso2-ballerina/module-soap/issues
[x] Finalize the design of the SOAP connector revamp.
[x] Identify the common SOAP use cases in the production environment.
[x] Study usernameToken Password digest ws-security and ways to implement it from Ballerina.

Task

[x] Update documentation and fix intermittent test failures

Path

[x]  Test Path API implementation in windows and fix failing testcases ctd

Performance

Next week sprint : https://github.com/ballerina-platform/ballerina-lang/issues/14749


update license and notice for 0.9.0 release

Description
Note that roaring bitmap dependency is upgraded to 0.9.23 after 0.9.0 release, so I kept it as 0.9.22 in our 0.9.0 release commit(https://github.com/apache/pinot/commit/0fa158009fc7e48c4b9de29cd61cd9811373014a#diff-a74b4a65ab1f4aed61344787bb9654ba7e835154730653848c88ca11f9965dc0R392).
The NOTICE-binary file is generated by, and I'm not intended to modify/move any section there.
➜ mvn --version
Apache Maven 3.6.2 (40f52333136460af0dc0d7232c0dc0bcf0d9e117; 2019-08-27T10:06:16-05:00)
Maven home: /usr/local/Cellar/maven/3.6.2/libexec
Java version: 11.0.6, vendor: AdoptOpenJDK, runtime: /Library/Java/JavaVirtualMachines/adoptopenjdk-11.jdk/Contents/Home
Default locale: en_US, platform encoding: UTF-8
OS name: "mac os x", version: "10.16", arch: "x86_64", family: "mac"

Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes

Documentation

Codecov Report

Merging #7750 (9d799d8) into master (94d3c5f) will decrease coverage by 16.24%.
The diff coverage is n/a.

:exclamation: Current head 9d799d8 differs from pull request most recent head 8d06979. Consider uploading reports for the commit 8d06979 to get more accurate results

@@              Coverage Diff              @@
##             master    #7750       +/-   ##
=============================================
- Coverage     30.83%   14.59%   -16.25%
- Complexity        0       80       +80
=============================================
  Files          1568     1531       -37
  Lines         80194    78675     -1519
  Branches      11929    11765      -164
=============================================
- Hits          24731    11480    -13251
- Misses        53345    66354    +13009
+ Partials       2118      841     -1277

Flag
Coverage Δ

integration1
?

integration2
?

unittests2
14.59% <ø> (?)

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

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


R0.10: Update Session documentation.

Updates core/public/README.md and Session API doc to address issue #3195.

Can one of the admins verify this patch?


Send failing on Checking and Donation Account

Send max is failing in checking and donation account
Send is failing in checking and donation account
Clicking “confirm and proceed” goes directly to “send unsuccessful”

These 3 issues are not occurring for the test account

@cakesoft-swati I've tried upgrading the app from 160(commit:4be24b3dcfba81998c83098eb82883c966fba49b) to 161 to see whether I'm able to get myself into these issues but unfortunately they didn't replicate on the simulator. Can you please confirm them once by upgrading from the previous build to the recent one and testing these scenarios out?

This is working as expected.


Second initialization fails

Hi
I have a problem with ajax-loaded videos. When i am loading bunch of videos, I am initializing them using videojs function.
For each element (using jQuery.each()):
var newId = 'video-id-'+idCount;
var setup = JSON.parse(el.attr("data-setup"));
el.removeAttr( "data-setup" );
videojs(newId, setup, function() {});
idCount++

And this is working for Youtube videos.
But for video from Vimeo, after first videos are loaded, when I click Load more (it gets new HTML from server and runs code, I wrote above on this HTML), I have an error in console:
TypeError: el is undefined

After some research, I found that Dom.createEl() fails to create element.
 with id is created, but without content.
In short:

When I am initializing one element, it is working
When I am initializing more elements at once (in one jQuery.each() loop) - it is working
When I am trying to initialize additional elements, in second run (no matter if I am running videojs() from console to test, or from another jQuery.each() loop) - it fails

What can I do with this?

@kamil7x I think your issue might be related to the issue I raised here: https://github.com/eXon/videojs-vimeo/issues/65. It's similar because when I try to dynamically create an element and initialise VideoJS using this plugin it breaks. Also switching / adding sources dynamically appears to be an issue.
@hendrathings has pushed an update a few days ago, but it doesn't seem to be merged yet, although @eXon seems to have done an update, so I'm pretty confused right now as to whether or not to use the update on the master branch or to try hendrathings branch instead (since it's not merged yet).
Hopefully they'll let us know :)

+1

This issue is very old, and if I remeber correctly, update worked. I am closing this issue.


Eliminate brake oscillations and grinding / crunching / ripping sound when using Pedal (Honda Pilot Only)

When using the Comma Pedal there are certain situations where the brake makes a very loud grinding / crunching / ripping sound:
a) when OP is engaged and brake is pressed at slow speeds
b) when OP is engaged at standstill and brake is pressed
It was found that when user presses on the brake, brake_pressed does NOT disengage OP until it reaches a threshold value (a dead zone), leading to BOTH the user AND OP applying 2 different brake settings simultaneously.  This leads to an oscillation between the two brake settings rapidly and results in a horrid crunching / grinding / ripping sound.  Additionally the brake lurches and causes an unsafe braking situation.
The fix was to use USER_BRAKE signal (instead of BRAKE_PRESSED) to disengage OP earlier when user depresses the brake pedal and before the oscillation begins.  The logic preserves and covers the brake_pressed scenarios as well.

I’ve confirmed this fix is solid and works great in my 2017 Honda Pilot!!! Finally relief from never being able to touch the brake when OP was engaged and also braking.

The best way I knew how to describe it was that user tries to brake while Openpilot is braking... the brake pedal would resist and would not actually disengage Openpilot.  With this fix, it does disengage cleanly like the Stock system does without Openpilot.
Anyone else experience similar things with or without the pedal interceptor?

I'm going to put it in for the Pilot only, Odyssey seems to have more brake noise so each USER_BRAKE level setting will probably be individual to each car.

Can you guys explain me why this is only a problem when the pedal interceptor is used? I'm not sure why it should be any different than without the pedal interceptor (but clearly it is).

It’s not, he shouldn’t have explained it this way.
It’s actually a problem regardless.  If OP is braking and you press the brake manually it would grind.  This PR resolved it for Pilot.
It now disables OP as Stock would disable immediately instead of grinding and resisting the user.
The pedal makes it way more noticeable for users because it’s higher braking pressure at lower speeds and users go to disengage manually with braking and the physical pedal resists them not disengaging. This PR fixes and disables OP immediately.
At least for Pilots.
--
Joshua F. Rountree
UX Developer
Newsy, The E.W. Scripps Company
312 Walnut Street, Suite 2800, Floor 29th, Office 29W109
Cincinnati, Ohio 45202-4024
(513) 977-3840 (office, M-F, 7a-4p)
(513) 827-7936 (mobile)
Newsy is America’s fastest-growing news network
On Dec 24, 2018, 7:59 PM -0500, rbiasini<EMAIL_ADDRESS>wrote:

Can you guys explain me why this is only a problem when the pedal interceptor is used? I'm not sure why it should be any different than without the pedal interceptor (but clearly it is).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

It should be noted,  this has been reported by a lot of pedal users because they never noticed it before due to lack of high pressure braking where the user intervened.  But it’s common for scary situations to happen that require user to intervene when on pedal which causes them to notice it way more.
--
Joshua F. Rountree
UX Developer
Newsy, The E.W. Scripps Company
312 Walnut Street, Suite 2800, Floor 29th, Office 29W109
Cincinnati, Ohio 45202-4024
(513) 977-3840 (office, M-F, 7a-4p)
(513) 827-7936 (mobile)
Newsy is America’s fastest-growing news network
On Dec 24, 2018, 7:59 PM -0500, rbiasini<EMAIL_ADDRESS>wrote:

Can you guys explain me why this is only a problem when the pedal interceptor is used? I'm not sure why it should be any different than without the pedal interceptor (but clearly it is).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

And they are staying engaged longer at slower speeds such as redlights or stop and go situations.
--
Joshua F. Rountree
UX Developer
Newsy, The E.W. Scripps Company
312 Walnut Street, Suite 2800, Floor 29th, Office 29W109
Cincinnati, Ohio 45202-4024
(513) 977-3840 (office, M-F, 7a-4p)
(513) 827-7936 (mobile)
Newsy is America’s fastest-growing news network
On Dec 24, 2018, 7:59 PM -0500, rbiasini<EMAIL_ADDRESS>wrote:

Can you guys explain me why this is only a problem when the pedal interceptor is used? I'm not sure why it should be any different than without the pedal interceptor (but clearly it is).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Rails 3.1 bundle conflicts

$ bundle install
Fetching source index for http://rubygems.org/
Bundler could not find compatible versions for gem "arel":
  In Gemfile:
    exception_logger (>= 0.1.11) ruby depends on
      arel (~> 1.0.0.rc1) ruby

    rails (= 3.1.2) ruby depends on
      arel (2.2.1)

Note also that meta_where explicitly says not to use it w/ 3.1, and recommends Squeel instead.

@radfahrer +1


update installation instructions

Mainly install all dependencies from Anaconda

Did anyone check this file?  It no longer works.
The spyder line is missing its indentation .. and we all know how python loves indentation :)
Also .. the link from http://qcodes.github.io/Qcodes/start/index.html#installation is broken ...   it seems to contain "useful" descriptive text, but the link destination is broken.


ResolveAssemblyReference task generating warning instead of unifying and including assembly in SuggestedRedirects ouput parameter

Build the following project, using Dev15 RC, or .NET CLI 1.0.0-preview3-004056.  You'll also need a Program.cs file with an entry point.
<Project ToolsVersion="15.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" />
  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net461</TargetFramework>
    <RuntimeIdentifier>win7-x86</RuntimeIdentifier>
  </PropertyGroup>
  <ItemGroup>
    <Compile Include="**\*.cs" />
    <EmbeddedResource Include="**\*.resx" />
  </ItemGroup>
  <ItemGroup>
    <Reference Include="System.Net.Http" />
    <PackageReference Include="Microsoft.NET.Sdk">
      <Version>1.0.0-alpha-20161104-2</Version>
      <PrivateAssets>All</PrivateAssets>
    </PackageReference>
    <PackageReference Include="System.Net.Http">
      <Version>4.1.0</Version>
    </PackageReference>
  </ItemGroup>
  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
</Project>

EXPECTED: No Errors or warnings generated
ACTUAL:

C:\Program Files\dotnet\sdk\1.0.0-preview3-004056\Microsoft.Common.CurrentVersion.targets(1909,5): warning MSB3243: No way to resolve conflict between "System.Net.Http, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" and "System.Net.Http, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a". Choosing "System.Net.Http, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a" arbitrarily.

The Microsoft.NET.Sdk targets set AutoGenerateBindingRedirects to true, so this warning shouldn't be generated.
The verbose log has messages referring to System.Net.Http.dll where it should be referring to other assemblies.  I'm not sure if that is related to this problem or is a separate issue just with the message logging.  For example:

Unified primary reference "System.Security.Cryptography.X509Certificates, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a". (TaskId:10)
Using this version instead of original version "<IP_ADDRESS>" in "C:\Users\daplaist.nuget\packages\system.net.http\4.1.0\ref\net46\System.Net.Http.dll" because AutoUnify is 'true'. (TaskId:10)

Also note that for simplicity the project file has a Reference for System.Net.Http and a PackageReference to the System.Net.Http package.  The more common scenario where this will be hit is for ASP.NET Core applications targeting .NET Framework, which will get the PackageReference transitively from their dependencies.

@nguerrera @ericstj @eerhardt FYI.  This seems like a bug in the ResolveAssemblyReference task but it might actually be elsewhere.

@piotrpMSFT @krwq

This is the behavior of RAR, but we want it to change.  This is the conflict resolution feature we've been discussing.  Today RAR never resolves conflicts between primary references but we want it to do so.

Here is the task I am working on for doing this downlevel (where we also need support) https://github.com/dotnet/standard/blob/master/netstandard/tools/HandlePackageFileConflicts.cs.  This could be done much better (and with better perf, since assemblies are already opened) if it were part of RAR.  @AndyGerlicher, @cdmihai, @dsplaisted we talked about this in a meeting before the holidays.

@dsplaisted is this still needed?

This doesn't repro anymore.  I don't think we made the change in RAR, but conflict resolution handles it in this case.
Here's the repro project converted to current sdk style, FWIW:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>net461</TargetFramework>
    <RuntimeIdentifier>win7-x86</RuntimeIdentifier>
  </PropertyGroup>
  <ItemGroup>
    <Reference Include="System.Net.Http" />
    <PackageReference Include="System.Net.Http">
      <Version>4.1.0</Version>
    </PackageReference>
  </ItemGroup>

</Project>


Tests pass even when there are TypeScript errors

According to https://github.com/facebook/create-react-app/issues/5626, there's a limitation in how Create React App sets out jest, which means type checking isn't being done properly on test files. That means tests could still be passing even if they should break because of TS error.
The workaround seems to be adding tsc before running the tests so the test files goes through the TS type checker and only run the tests if succeeds. Will have to fix some broken test files as a result of this.

Is this why tests are passing CI but failing locally?

No don't think so, that's a separate issue. I think the tests failing locally is something weird with the mocks not working again... not sure why CI likes the workaround but not our machine locally sometimes 🤷
This is more to do with the fact if you add a new prop to a component and you don't update the tests, sometimes it stills passes (when it should fail like in the main app)

Hi everyone! I would love to pick up this issue (and join the development team if I may :) ).

Assigned! Also send me an email (address on my profile)

One other thing I forgot to write in the first description, can you also look at linting the test files too while you are in the area please? I think these are all related enough issues that it'd be good to fix them together. Apologies if that might mean more files to fix 😬

No worries, I've jest setup my dev environment so I can start working.
Do you mean these two lint warnings?

Do you mean these two lint warnings?

Yup! Looks like there aren't too many then 😌


Added Gruvbox theme

Submit a new Plugin to Marketplace
Plugin Github repo URL: https://github.com/OddDev/oddbox
Github releases checklist

[x] a legal package.json file.
[x] a valid CI workflow build action for Github releases. (theme plugin for this).
[x] a release which includes a release zip pkg from a successful build.
[x] a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
[x] a license in the LICENSE file.

Hey :)
I think I'm ready for a review. Thank you!
https://github.com/OddDev/oddbox


Add version: Foldr.Foldr version 3.1.0

Checklist for Pull Requests

[ ] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This is an update for the initial version 3.1.0.  It changes the installer url from:
https://foldr.com/downloads/clients/win/x64/FoldrSetup-x64.msi

To:
https://foldr.com/downloads/clients/win/x64/winget/3.1.0/FoldrSetup-x64.msi


Allow to ignore files

This add #38 support (allow to ignore some storyboards / XIBs).

Rebased and merged locally - cheers for the PR


Synced Lyrics?

I don't know what turning on Synced Lyrics does, but whatever it's supposed to do doesn't look like it's working because when I do enable it, nothing seems to change

http://i.imgur.com/XSxvSMZ.gif
This is what it's supposed to do and appears to be working for me. It should highlight the currently playing lyric. (with some exceptions, it will de sync if you manually seek through the song)
If it can't find the synced lyrics of a song it will fall back to just displaying the regular lyrics though. Generally only happens with obscure songs.

Thanks for clearing that up. The problem seems to be that spotify doesn't have synced lyrics for a chunk of the songs in my library.


Provider pointer helpers

The AWS SDK does this, because most types require a pointer (e.g. pointer to string, pointer to bool etc) please provider simple helper methods e.g.
func String(value string) *string {
   return &value
}

func Bool(value bool) *bool {
    return &value
}

etc.
It would also be really useful to have simple helpers for "safe" values e.g.
func SafeString(ptr *string, default string) string {
    if ptr == nil {
        return default
    }

    return *ptr
}

The IBM Go SDK Core, which underlies the VPC Go SDK, provides these helpers, e.g. StringPtr
// StringPtr returns a pointer to string literal.
func StringPtr(literal string) *string {
	return &literal
}

"Safe" helpers are not provided but could be added with sufficient justification.  Can you provide this?

Thank you for the info. For the "safe" helpers I am often extracting information about vpc objects for display or reference, I don't care what the values are and so, for example, blank string is fine if the value is nil, and avoid me having to handle the nil.
I also sometimes need to extract a value from deep in a structure which might contain nil in the path. Using the softlayer-go library I use this:
https://github.com/softlayer/softlayer-go/blob/master/sl/helpers.go#L143
It would be really useful to have that function, and the OK variant, available in the core go SDK as well. The usage would be, for example, that I want to get a network interface IP address, the Grab method let's me easily get the value, or blank if there is a nil structure somewhere in the hierarchy.
I'd be happy to port and create the helpers and submit and MR

We welcome community contributions!
The "safe" variants seem like they are useful for users, but less so for the SDK itself, which I think explains why these were not included in the Go Core SDK.


Add pypi secrets in GitHub

Please insert the necessary secrets in GitHub so we can build and publish the package to pypi and test.pypi.
The neccesary secrets are
${{ secrets.TEST_PYPI_TOKEN }}
${{ secrets.PYPI_TOKEN }}
${{ secrets.GITHUB_TOKEN }}
I don't know if this github_token is neccesary, or if it's auto available it's at the end of the build script which was available in GitLab.
Please put the secrets in both repositories

@g4challenge @elauser added TEST_PYPI_TOKEN and PYPI_TOKEN in both repositories.  GITHUB_TOKEN should be available out of the box.
Please test and provide feedback.