Using a formatter

🚀 Feature

I am proposing to use black python formatter to decrease number of convention decisions to make and sustain consistency on code convention over repository. To me defaults of black seems great and i think you guys might like it as well.
Black formatter: https://github.com/python/black
Motivation
Using a strong formatter decrease number of convention decisions to make. Reduces frustration on newcomers.
Pitch
Black formatter is so cool. I compare it with yapf from google and autopep8 formatter.
Definitely black has an edge.
Alternatives
Strongest alternatives: yapf from google: https://github.com/google/yapf
and autopep8: https://github.com/hhatto/autopep8
Additional context

I like the idea of formatters in general, but I dislike the rigidity of black (as I understand it it is not very configurable).
The number one concern is whether code formatted with black is actually nice and readable. Do you know any scientific packages that use it? Only the web-world appears to have taken it on.
I guess another way to propose this is to submit a PR with all the code formatted. Then we can see if we like its changes. =)

I'm open to a formatter, and black has come up at CZI recently, but I'm not sure any of our projects actually started using it. @kne42 might know more and I can follow-up.
A PR seeing what the new code would look like would be helpful and might be a good next step if it is not too much work to make. If the new code appears more readable and understandable then I'd be all for it, if we are constantly having to use # fmt: off and # fmt: on to make the code look nice then I'd probably want to pass

Summarizing my comments from a different thread, I'm open but not thrilled to having an autoformatter. While it'll be great to finally have our codebase cleanly formatted, I don't think it's a proper substitution for people just actually writing well-formatted code.
At the end of the day, readability is something that a simple program cannot quantify; all it's doing is following a strict set of rules. Except styling doesn't consist of rules, it consists of guidelines, which means that following style guides to the letter does not necessarily make your code more readable, sometimes it can make your code less readable.
Ideally, contributors should be following a style guide (cough cough PEP8) and otherwise format sensibly (which, if it isn't clear enough, we can make a guide for too).
However, given the current state of the codebase, I will admit that I have my doubts of letting contributors just go around willy-nilly without some sort of enforcement since I'd prefer not to have to leave 1000 comments on every PR about styling.
What I would most prefer to see is something similar to the @pep8speaks bot that suggests formatting automatically without necessarily forcing it on someone, which IMO would be the best compromise between having some sort of sanity check while letting humans ultimately decide what looks cleanest.
As far as actual autoformatters go, here's my take on them:

black is a hard no from me – I strongly disagree with its formatting conventions
yapf seems very promising in its customizability so I'd definitely be on-board with it
autopep8 is classic and minimalistic but doesn't provide as nice formatting as the other two – it just fixes PEP8 errors, which is fine by me

Thanks to all @jni @sofroniewn @kne42 for your comments.
I like idea of discussing the formatters on a small PR over code.
I will be opening a showcase PR.
Also, i understand the frustration/hesitation against a strong formatter.
black style may not be the most pythonic collection of conventions, however, IMO it increases readability and takes care convention decisions for us.
Its default conventions are also compatible with many many industry-grade conventions.
yapf is really good IMO and on most cases default style is same with black.
I know you guys liked that it is also configurable, i see it as a possible drawback.
Again we might need to make decisions.
Stay tuned till my formatter PR.

just btw I'm super in favour of changing the default line length to 80, like God intended. =P
The problem of review definitely pushes me in favour of the formatter. I have to admit that I have come to regard PEP8speaks as noise, and rarely read it. This is the problem with all auto-suggest tools, including e.g. the github issue template.

@jni I personally find the PEP8speaks very helpful (though I often run pycodestyle because it's faster) as a beginner. I'd say our problem now is that there are many things that it doesn't catch and that I get consistently wrong. The manual approach does mean I'm learning more (and will hopefully get better), but I recognize it's a lot more burden on our reviewers.
I'd say reducing reviewer burden and time would also be a major reason to adopt a formatter, but I'm open to the details

I opened the #282 . Please check details and let me know about your comments.

That's great, thanks @AhmetCanSolak I will check it out. If easy maybe you or @kne42 wants to open a similar PR using the same three files with yapf using some of the configurations that @kne42 likes so we can do side-by-side comparisons.

@sofroniewn it is easy. yapf default is pretty much same with black except putting " for strings for the files in PR. So for side-by-side comparison could make more sense if we have specific configuration in mind.

Looking now at black -S #282 and yapf #283 side-by-side I think a couple of us, (@jni, @AhmetCanSolak, and myself) are leaning towards black -S. @kne42 and @royerloic what are your opinions?
One more thing we'd need to decide is how to enforce the autoformatter. If it is possible, I'd favor an approach where we add a seventh test (like our current 6 cross platform tests) that checked for compliance. If the test fails it will be obvious - there will be a red X - but it would still be possible for us to merge (in the same way that we continue to merge despite the windows / macOSX builds failing). If we wanted to move to a world where you could not merge at all with any failing tests we could also do that in the future. We'd also disable the pep8 bot and its suggestions, but add a description of how to use the autoformatter to our contributing guidelines

I just saw that JupyterHub uses black and uses git pre-commit hooks to ensure that it runs.
https://github.com/jupyterhub/jupyterhub/issues/2416
Their solution involves the package pre-commit to manage git hooks, but we could equally well do it manually. But I think using pre-commit is probably the way to go.
Whatever we do, automating it pre-merge or post-merge is the right way to go. The user should not have to manually run black, and should never be bugged about black. It should Just Work:tm:.

I like the user not having to manually run it, and I have a strong preference towards a pre-commit solution - which if I understand correctly would allow us to see the already formatted code during PR review. If a goal is to streamline PR review I think running the formatting before the review seems very important.
I have no problem with the pre-commit package and don't see any reason to role our own, especially if JupyterHub has been using this for the past few months.
I also don't have a problem with one massive reformatting PR to get us going. People have raised concerns about git blame in this context, but some reading of the black docs leads to a solution involving git hyper-blame.
I'm pretty excited about all this!!

I think running the formatting before the review seems very important.

:100: spot-on point.
I also think that we are early enough in the history of this project that git blame is not a concern. I also think we can generally place the blame on @sofroniewn and have a good chance of being correct. =P
@AhmetCanSolak how do you feel about writing a PR that uses black -S on the whole code base and installs pre-commit hooks as described above?

Hi @jni , yes i can make a PR for black -S

now integrated formatter successfully, im closing this issue.


Rubber banding ship movement

Minecraft v.(1.12.2) JAVA
Forge v.(1.12.2-<IP_ADDRESS>11)
Moving world v.(1.12.2-6.342)
davinci v.(1.12.2-6.340)
The ship, when assembled will fly but rubber bands back to the starting location of assembly. Speed doesn’t seem to affect the issue but I can’t fly the ship more than a few chunks before the rubber banding happens. It will rubber band every time I disassemble.
The rubber banding movement is very similar to a server de-sync rubber band movement.
Bug is affecting a single player survival map crafted ship. It’s a legacy Java map from very early versions of Minecraft. (pre notch selling)
The ship is roughly 2500 blocks including several simple Redstone machines; 7 engine blocks; 44% balloon blocks. Ship has a large amount of storage, crafting benches, etc.
Ship assembles flies, turns, uses fuel, ascends and descends, aligns with no issue.
I have built several ships of similar size on the same map but in prior versions (1.10 if I recall) with no issue. I have also built ships significantly larger in older versions with no issue.
I am running 1 additional mod (Grappling hook mod v10-1.12.2)
What I’ve tried.
Built ships in a fresh creative map in current build with no issues, so I’d assume it’s an issue with my map or the actual ship.
Built basic ship in same map in current build, no issues.
Update: What I’ve tried;
Rebuilt the effected ship in its entirety sans internal works and redstone contraptions. Ship block size is 1932. The clone ship has the exact same issue. Note this clone ship was built ingame and no external tools were used.
I’m leaning to the ships xyz size or some mechanic related to speed causes the issue.
Update: What I've tried;
Copied original ship and associated chunks into a separate creative world with MCedit 2.0. Rebuilt balloon blocks and tested, same issue as original world with one or multiple engines.
Built a test ship in a 50x11x11 bounding box to test if the issue was block amounts; block volume or other. No issues with the block volume ship.
I'm not really sure what else to test. My final plan is to assemble the ship again and test every single block used to see if its a particular block causing the effect.

I am having the exact same issue. My ship is over 7000 blocks and refuses to go anywhere permanently, it keeps snapping back to its original location. Like you said it uses fuel and moves fine but I can't actually sail anywhere with it. It's very strange because I have built and driven other vehicles just fine, they were much smaller however. I'm running 1.12.2 with forge and davinci fully updated.

Same, bruh

Me too! But I can't even sail my ship! It stays there! Only using a bucket of water can I move the water which moves the ship

Update: I've used MCedit2.0 to copy/paste the ship and intervening chunk into a blank creative world. 3 block depth, flat features for testing. Rebuilt Davinci blocks/balloon and tested. Same issue. Tested with single or multiple engines.
Tested simple >100 block ship on same world with no issues. So I'm leaning on this being a block amount or complexity issue. Prior test with a 50x11x11 block ship was all balloon blocks didn't have issues. Not discounting a specific block being the issue.
Current plan is to prep the design for insert into 1.14 world whenever mod is updated to that version.


feat[Spanner]: Add samples for new features

Tests won't pass until https://github.com/googleapis/google-cloud-dotnet/pull/12164 is merged and release. I'm currently running them locally.

@jskeet I've re-requested review as I fixed a copyright year and the previous approval was lost.

The first commit here is from #2588 and will be rebased away once that's merged. We need it in the meantime so these tests run.

@jskeet Here are the dependency updates for Spanner samples and the samples themselves, you had already reviewed one sample but not the other.
And also, when updating the dependencies I had to fix a test. PTAL, thanks!

Failures are unrelated and being tracked in #1623 . Force merging.


Broken link in about section of this repository

The about section of this repo (https://github.com/rucio/documentation, top right) links to rucio.cern.ch/documentation, which is broken. The correct link is https://rucio.github.io/documentation/

The cern webpage works as well, the github.io is just a mirror

You're right, not sure why it didn't seem to work before for some reason, but it works for me as well now

The main url is rucio.cern.ch/documentation
The github.io one works as well, but we should advertise the cern one.


DCPError: Problem does not follow DCP rules. Specifically: The objective is not DCP

Describe the bug
Hi, I am trying to optimise a portfolio with cvxpy.
The function I want to optimise has to maximise return and minimise volatility so it is a standard well known cost function. However, I get the following error
Problem does not follow DCP rules. Specifically:
The objective is not DCP.
because it is not convex.
However I must use that function.
The problem happen exactly with the sign in front of the term PENALTY. If i put + it works with - not.
To Reproduce
def minVarOptC(self, lmbd, r_target):
    numAssets = np.shape(self.rets)[0]
    W = cp.Variable(numAssets)
    penalty=lmbd*cp.pnorm((self.rets @ W - r_target),1)

    risk = cp.quad_form(W, self.covs) - penalty
    prob = cp.Problem(Minimize(risk), [sum(W)==1.,
                                    sum(W[:2])>=min_fi,
                                    sum(W[:2])<=max_fi,
                                    sum(W[2:4])>=min_az,
                                    sum(W[2:4])<=max_az,
                                    sum(W[4:])>=min_cr,
                                    sum(W[4:])<=max_cr,
                                    sum(W[0])>=min_wgb,
                                    sum(W[0])<=max_wgb,
                                    sum(W[1])>=min_emd,
                                    sum(W[1])<=max_emd,
                                    sum(W[2])>=min_we,
                                    sum(W[2])<=max_we,
                                    sum(W[3])>=min_eme,
                                    sum(W[3])<=max_eme,
                                    sum(W[4])>=min_gc,
                                    sum(W[4])<=max_gc,
                                    sum(W[5])>=min_ghy,
                                    sum(W[5])<=max_ghy,
                                    W >= 0.])
    prob.solve()

Version

OS: W10
CVXPY Version: 1.1.1

Can anybody help me to correct the code in order to be able to use such a function?
Thanks
Luigi

This doesn't look standard to me. What is the pnorm from? Return is normally a linear function.

Hi Steve,
the point is that I am trying to optimise the minimum regret because I have a scenario-based situation, so yes probably the way I wrote is wrong. Do you have any suggestion on how to correct it?
Thanks
BR
Luigi

Are you sure the formulation is convex? cp.pnorm((self.rets @ W - r_target),1) is convex, and cp.quad_form(W, self.covs) - penalty is non-DCP assuming lmbd is positive.

yes it was not and indeed I had to modify it, now seems to work.
The norm was an error.
Thanks for putting that in evidence
BR
Luigi


3 位（100 及以上）参考列表 显示粗糙的悬挂缩进

参考列表使用悬挂缩进，并根据研究生办公室证明。 但是，LaTeX 中的当前参考格式似乎使用普通缩进作为参考部分。 我该如何解决这个问题？

目前本模板编译的参考文献格式是正确的悬挂缩进（如下所示）：

参考指南中的参考文献格式：

请问可以更清楚地表述问题吗？
您举得例子看上去是正文中通过enumerate等环境产生的枚举结果，与参考文献无关？

Upon careful thought, I think you are right. The reference format applies the hanging indentation as you just posted. That is correct. However, this is not suppose to be the case according to report from graduate office regarding my pdf submission which is the same as yours.
I think they confused me. I apologize for the post. I think they confused the enumerated list of referencing guide on the word document on page 22 with the referencing format Writing Standards_20220321.docx.
I will obtain a feedback from them and then close this issue if I'm right.

you may set \thesisbibliography[large]{reference} in main.tex to fix the problem

非常感谢。 我很欣赏这个。


[nb_conda_kernels] enabled, 0 kernels found After registering R image sample to Sagemaker Studio

After building, deploying to ECR, attaching to Sagemaker Studio the R image sample and trying to run the kernel I get Kernel Dead error with these logs:
2020-11-26T13:03:33.471-03:00 | + CONDA_DIR=/opt/.sagemakerinternal/conda
-- | --
  | 2020-11-26T13:03:33.471-03:00 | + CONDA_ENV_FILTER=/opt/conda$
  | 2020-11-26T13:03:33.471-03:00 | + command -v python
  | 2020-11-26T13:03:33.471-03:00 | + [ 0 -eq 0 ]
  | 2020-11-26T13:03:33.471-03:00 | + python -c from __future__ import print_function;import sys; print(sys.prefix)
  | 2020-11-26T13:03:33.471-03:00 | + SYSTEM_PYTHON_PREFIX=/opt/conda
  | 2020-11-26T13:03:33.471-03:00 | + export JUPYTER_PATH=/opt/conda/share/jupyter/
  | 2020-11-26T13:03:33.471-03:00 | + [ ! -f /opt/conda/share/jupyter/kernels/python3/kernel.json ]
  | 2020-11-26T13:03:33.471-03:00 | + [ ! -z Custom ]
  | 2020-11-26T13:03:33.471-03:00 | + [ Custom = DLC ]
  | 2020-11-26T13:03:33.471-03:00 | + [ Custom = Studio ]
  | 2020-11-26T13:03:33.471-03:00 | + echo This is not a DLC/Studio image (found Custom), so not adding Python3 kernel.
  | 2020-11-26T13:03:33.471-03:00 | + export PATH=/opt/conda/bin:/bin:/usr/local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/tmp/miniconda3/condabin:/tmp/anaconda3/condabin:/tmp/miniconda2/condabin:/tmp/anaconda2/condabin
  | 2020-11-26T13:03:33.471-03:00 | + /opt/.sagemakerinternal/conda/bin/jupyter-kernelgateway --ip <IP_ADDRESS> --port 8888 --JupyterWebsocketPersonality.list_kernels=True --KernelSpecManager.ensure_native_kernel=False --MultiKernelManager.default_kernel_name= --KernelGatewayApp.kernel_spec_manager_class=nb_conda_kernels.CondaKernelSpecManager --CondaKernelSpecManager.env_filter=/opt/conda$
  | 2020-11-26T13:03:33.471-03:00 | This is not a DLC/Studio image (found Custom), so not adding Python3 kernel.
  | 2020-11-26T13:03:33.471-03:00 | [KernelGatewayApp] [nb_conda_kernels] enabled, 0 kernels found
  | 2020-11-26T13:03:38.782-03:00 | [KernelGatewayApp] Jupyter Kernel Gateway at http://<IP_ADDRESS>:8888
  | 2020-11-26T13:03:48.730-03:00 | [E 201126 16:03:48 web:1792] Uncaught exception POST /api/kernels (<IP_ADDRESS>) HTTPServerRequest(protocol='http', host='<IP_ADDRESS>:32774', method='POST', uri='/api/kernels', version='HTTP/1.1', remote_ip='<IP_ADDRESS>') Traceback (most recent call last): File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/web.py", line 1703, in _execute result = await result File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 742, in run yielded = self.gen.throw(*exc_info) # type: ignore File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/kernel_gateway/services/kernels/handlers.py", line 63, in post yield super(MainKernelHandler, self).post() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 735, in run value = future.result() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 742, in run yielded = self.gen.throw(*exc_info) # type: ignore File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/notebook/services/kernels/handlers.py", line 46, in post kernel_id = yield maybe_future(km.start_kernel(kernel_name=model['name'])) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 735, in run value = future.result() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 742, in run yielded = self.gen.throw(*exc_info) # type: ignore File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/kernel_gateway/services/kernels/manager.py", line 83, in start_kernel kernel_id = yield gen.maybe_future(super(SeedingMappingKernelManager, self).start_kernel(*args, **kwargs)) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 735, in run value = future.result() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 209, in wrapper yielded = next(result) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/notebook/services/kernels/kernelmanager.py", line 168, in start_kernel super(MappingKernelManager, self).start_kernel(**kwargs) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/multikernelmanager.py", line 185, in start_kernel km.start_kernel(**kwargs) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 309, in start_kernel kernel_cmd, kw = self.pre_start_kernel(**kw) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 262, in pre_start_kernel kernel_cmd = self.format_kernel_cmd(extra_arguments=extra_arguments) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 181, in format_kernel_cmd cmd = self.kernel_spec.argv + extra_arguments File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 87, in kernel_spec self._kernel_spec = self.kernel_spec_manager.get_kernel_spec(self.kernel_name) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/nb_conda_kernels/manager.py", line 327, in get_kernel_spec res = super(CondaKernelSpecManager, self).get_kernel_spec(kernel_name) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/kernelspec.py", line 235, in get_kernel_spec raise NoSuchKernel(kernel_name) jupyter_client.kernelspec.NoSuchKernel: No such kernel named hdx-r-kernel
  | 2020-11-26T13:03:48.730-03:00 | [E 201126 16:03:48 log:48] { "Authorization": "token", "Connection": "close", "Host": "<IP_ADDRESS>:32774", "Content-Length": "117", "Content-Type": "application/x-www-form-urlencoded", "Accept-Encoding": "gzip" }
  | 2020-11-26T13:03:53.471-03:00 | [E 201126 16:03:48 log:49] 500 POST /api/kernels (<IP_ADDRESS>) 294.59ms referer=None

Am I doing something wrong?

The relevant error is in this line in the logs you've provided
  | 2020-11-26T13:03:48.730-03:00 | [E 201126 16:03:48 web:1792] Uncaught exception POST /api/kernels (<IP_ADDRESS>) HTTPServerRequest(protocol='http', host='<IP_ADDRESS>:32774', method='POST', uri='/api/kernels', version='HTTP/1.1', remote_ip='<IP_ADDRESS>') Traceback (most recent call last): File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/web.py", line 1703, in _execute result = await result File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 742, in run yielded = self.gen.throw(*exc_info) # type: ignore File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/kernel_gateway/services/kernels/handlers.py", line 63, in post yield super(MainKernelHandler, self).post() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 735, in run value = future.result() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 742, in run yielded = self.gen.throw(*exc_info) # type: ignore File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/notebook/services/kernels/handlers.py", line 46, in post kernel_id = yield maybe_future(km.start_kernel(kernel_name=model['name'])) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 735, in run value = future.result() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 742, in run yielded = self.gen.throw(*exc_info) # type: ignore File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/kernel_gateway/services/kernels/manager.py", line 83, in start_kernel kernel_id = yield gen.maybe_future(super(SeedingMappingKernelManager, self).start_kernel(*args, **kwargs)) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 735, in run value = future.result() File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/tornado/gen.py", line 209, in wrapper yielded = next(result) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/notebook/services/kernels/kernelmanager.py", line 168, in start_kernel super(MappingKernelManager, self).start_kernel(**kwargs) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/multikernelmanager.py", line 185, in start_kernel km.start_kernel(**kwargs) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 309, in start_kernel kernel_cmd, kw = self.pre_start_kernel(**kw) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 262, in pre_start_kernel kernel_cmd = self.format_kernel_cmd(extra_arguments=extra_arguments) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 181, in format_kernel_cmd cmd = self.kernel_spec.argv + extra_arguments File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/manager.py", line 87, in kernel_spec self._kernel_spec = self.kernel_spec_manager.get_kernel_spec(self.kernel_name) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/nb_conda_kernels/manager.py", line 327, in get_kernel_spec res = super(CondaKernelSpecManager, self).get_kernel_spec(kernel_name) File "/opt/.sagemakerinternal/conda/lib/python3.7/site-packages/jupyter_client/kernelspec.py", line 235, in get_kernel_spec raise NoSuchKernel(kernel_name) jupyter_client.kernelspec.NoSuchKernel: No such kernel named hdx-r-kernel

And digging even further this is the relevant error from Jupyter jupyter_client.kernelspec.NoSuchKernel: No such kernel named hdx-r-kernel
It looks like you've provided hdx-r-kernel as the kernel name in your AppImageConfig but the actual kernel name in this image is ir. When Jupyter tries to launch a kernel with that name, it fails because the kernel does not exist in the image.

Changing image name worked for me, thanks. I also had to change r-base version to v4.0.3


feat: iOS SDK for Chimoney API

Description
Develop an iOS SDK for the Chimoney API in Swift. This SDK should provide an easy and intuitive interface for iOS developers to integrate the Chimoney API into their iOS applications.
Requirements:

Design the SDK in a modular way for easy maintenance and updates.
Write a comprehensive README file with setup instructions, usage examples, and API endpoint descriptions.
Write unit tests to verify the functionality of the SDK.

Screenshots
No response
Ready to Work?

[ ] I want to work on this issue

I would like to take this one @phyleria

Sure, please go ahead.

Thanks

Hey @phyleria can u assign it to multiple people . If yes I am interested.

🌟 This Issue is Now a Featured Oruly Task  sponsored by Chimoney! 🚀
Hello there! 🎉 This GitHub issue has been upgraded to a task on Oruly, where you can earn rewards for your Github contributions.
🚀 Task Details:

💡 Title: feat: iOS SDK for Chimoney API
💰 Reward: 10 USD

Why Oruly?

Diverse Task Opportunities: Oruly offers a wide range of tasks encompassing coding, content creation, data analysis, and more, catering to various professional needs.
Global Collaboration: Connect with a worldwide network of professionals. Oruly not only opens doors to global opportunities but also fosters a community where diversity and cooperative progress thrive.
Continuous Learning and Growth: At Oruly, we believe in the continuous professional development of our contributors. Engage with tasks that challenge and refine your skills, supported by tools that offer insights and learning on-the-go.
Rewarding Contributions: We value your time and expertise. Oruly ensures fair compensation and recognition for your work, with transparent and fast payment processes and competitive rewards.

Claiming Your Rewards

Easy Process: Complete the Github Pull Request, as normal, and submit your approved PR link on Oruly. Your reward will be automatically credited to your Oruly wallet after Approval.
Transparent System: All rewards are visible on your dashboard. You can review your earned rewards anytime.
Quick Access: To claim your reward, simply go to the 'Wallet' section on your profile, and select 'Payout'. Your rewards can be transferred to your bank account, mobile money, airtime or used within the platform for services or to fund your own tasks.

Interested in Tackling This Task?

Transform this issue into high-quality code and contribute to exciting projects! 🚀

"Strive not to be a success, but rather to be of value." – Albert Einstein

We're excited to see what you'll build! 🚀🌟


MINOR: Refactored default resource type based auth

More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Refactors the authorization logic to improve readability and maintainability. The main changes include:

Created class to separate already big default method implementation from other interface methods
Separated big method into smaller onse for better readability and maintainability
I ensured that ACL bindings and permission types are handled clearly, and the logic for handling resource patterns (literal and prefix) is optimized

Committer Checklist (excluded from commit message)

[X] Verify design and implementation
[X] Verify test coverage and CI build status
[X] Verify documentation (including upgrade notes)

In general, LGTML. Very good for your first MINOR change.

Hi, request for revbiew @AndrewJSchofield

Update. I made it simpler: Everything is in interface, only one extracted method.
Regarding benchmark, on my laptop ETA is 8 hours XD, let me know if it's necessary @mumrah


Add meaningful RPG code to the repo

Merge meaningful RPG code into this repository.

Per a conversation with @ericboucher, we will move the non-web app parts of RPG to a separate public repository first, then evaluate what components (if any) would make sense living here instead.

RPG code unrelated to the web application now can be found in this repository.


Button V2: Allow content events to pass through

Button V2: Allow content events to pass through

This update updates Button V2 to allow for events (like clicks) to child
components to pass through to the primary Button component.
This is enabled by a new prop, allowContentEventPropogation, which is
true by default.
Buttons should allow for their onClick (or whatever events) to happen, regardless of where you click within the button itself.

Coverage remained the same at 100.0% when pulling dccd4270cfb3aa94f01160b584680748dc0139bd on button-content-allow-clickthrough into c9453924117b2c37ffc4e6d5867332dffc846609 on master.


LHN - Tooltip Appears After FAB Opens and Displays the List When Hovering Over the FAB Icon

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.23-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
1, Sign in to the staging environment of Expensify
2, Click on the Floating Action Button (FAB)
3, Hover over the FAB again; the tooltip continues to be displayed
Expected Result:
The tooltip should not appear after the modal opens
Actual Result:
Even after the modal opens, the tooltip persists when hovering over the FAB
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/63013550-531f-479d-8a16-76aa0e2c58fe
View all open jobs on GitHub

This is a regression of this PR https://github.com/Expensify/App/pull/33247

Both https://github.com/Expensify/App/issues/34185 and this issue are related to the TS migration PR https://github.com/Expensify/App/pull/33247

Fix is being CP'd to staging, cheers to @situchan for the quick fix

Whoops, looks like we just need to issue payment to @situchan for the PR (no C+)

Damn, the Upwork job isn't getting auto-created. @adelekennedy can you please issue payment of $500 to @situchan then close this out?

Upwork is having some trouble but I think I was able to extend you a contract @situchan

payment made!


API for using  file quick open

Feature Request
Add api that lets extension show the file quick open UI to the user and get back the URI of the file that the user selected.
I imagine this API working like the existing showOpenDialog api, expect that will show a the quick open UI instead of the system file dialog (or the simple dialog)
Potential use cases:

In Markdown, add a command that let users insert a link to an image that they select from quick open (#162809)
In Jupyter notebooks, let users add image attachment to notebook using quick open
In JS/TS and other languages, let users import a file that they select from quick open

This was previously requested in #37852, however making extensions implement this themselves will result in inconsistent UX and poor performance
I've tried using the workbench.action.quickOpen command but this command does not return the selected file

This provides the same functionality of the showOpenDialog but it doesn't honor the user's preference of the "file picking experience" that they've chosen since a user can decide to opt in to the simple file dialog.
Aren't we pretty firm that we don't do API that is just a UI difference?
cc @jrieken

@TylerLeonhardt I'd like quick open instead of the normal file browser. I don't think there's any way to show that today
With files.simpleDialog.enable, the UI I see for showOpenDialog  looks like this

I'd instead like

I see... this makes sense to me. Though I also wonder if a mode in the Simple File Dialog makes sense for this case.

I'd like this too. In Dart we have a command to "jump to test file for the current implementation file" (and vice versa).
In some cases, users keep tests in a folder that doesn't match the implementation so we can't easily find it. Being able to trigger the file quick-pick and pre-populate it with a query (based on the current open file basename, but with _test appended) would be really useful.
While they can just open the file picker themselves, it has no context that they might want the test file for the current open file, so they have to type enough characters to get the test file to the top (which in the cases reported to me, might involve a lot of characters).

Probably as a workaround we can just executeCommand("workbench.action.quickOpen", term) and deal with the fallout if the args ever change 🙃

I would really love this as well -- maybe a lightweight way would be to add a vscode.quickOpen (or vscode.quickFileOpen would be better) command that would be wait and return the Uri that was picked.
Right now the files.simpleDialog.enable experience of showOpenDialog is not remotely similar to the quick open experience (even when limited to files only), nor should it be imo -- showOpenDialog is more about the OS experience, so even in "simple" mode it should be more closely aligned with that behavior, while quick open is a rich fuzzy search (and quick open with multiselect would not work).
Additionally files.simpleDialog.enable with showOpenDialog is a user-level preference, not an extension choice either.

@TylerLeonhardt this would be a pretty easy PR if you are open to it -- let me know.

Has there been any progress on this issue

Github Copilot already uses something very similar to this for their file attachments. When will you start playing fair and make all the "proposed APIs" they are already using available to everyone?

@amir-halloul There is no proposed api right now. The feature you're talking about is part of core VS Code https://github.com/microsoft/vscode/blob/5d5976d10cde46dabc31bf264f637b8f6da413a9/src/vs/workbench/contrib/chat/browser/actions/chatContextActions.ts#L839


nuget.config casing

It's not very clear what the correct casing of nuget.config should be:

I've stumbled across various (old) Github issues discussing this, but didn't find (or search long enough) a definitive answer to this
This page mention both nuget.config and Nuget.Config (and not Nuget.config).

Now that .NET is heavily used on case-sensitive platforms, I think it'd make sense to indicate what the official file name is (doesn't prevent tools from being lax though).
Thanks!
PS: In the meantime, I suppose I'll go with nuget.config the same way I have packages.config...

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

ID: 370d9d91-9f35-79aa-3ace-7b77436e988f
Version Independent ID: 3da4591a-3d3d-c07a-6672-97964de13b14
Content: Common NuGet configurations
Content Source: docs/consume-packages/configuring-nuget-behavior.md
Product: nuget
GitHub Login: @karann-msft
Microsoft Alias: karann

This is mostly an old issue that has been resolved since. The preference of names comes from this code
Thus nuget.config is the preferred best practice, followed by NuGet.config, and lastly by NuGet.Config.
If something doesn't work, please let us know in an issue on:
https://github.com/NuGet/Home/issues
Given there are no known issues & this guidance already exists in docs, I'm closing this issue.


refactor: run cargo clippy --fix

Fix the following

https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow
https://rust-lang.github.io/rust-clippy/master/index.html#map_flatten

By default clippy doesn't check test modules. You can check them by running cargo clippy --tests
However, running cargo clippy --fix will check and auto-fix the test modules.

Thank you!


Conditional dependsOn

Bicep version
Bicep CLI version 0.8.9 (33a85174de)
Describe the bug
Similar to the one exlained in #5974 on the last few comment where modules scoped to different scope
dependency creates errors when a module/resource depends on another module with if condition and scoped to different subscription and the condition is false
ERROR: InvalidTemplate - Deployment template validation failed: 'The resource 'Microsoft.Resources/deployments/registerCustomDomainDns' is not defined in the template. Please see https://aka.ms/arm-template for usage details.'.

To Reproduce
below code will generate error because the dependency on the resource siteCustomDomain... for modules registerCustomDnsTxtRecord and registerCustomDnsName
param location string = 'westeurope'
param netFrameworkVersion string = 'v4.0'
@allowed([ 'AllAllowed', 'FtpsOnly', 'Disabled' ])
param ftpsState string = 'Disabled'
param DnsZoneSubscriptionID string = ''
param DnsZoneRG string = ''
param customDomainName string = ''
param dnsZoneName string = ''
param dnsRecordTTL int = 300

resource hostingPlan 'Microsoft.Web/serverfarms@2021-03-01' = {
  name: testPlan
  location: location
  sku: {
    name: B1
    capacity: 1
  }
  properties: {
    reserved: false
  }
}

resource appService 'Microsoft.Web/sites@2021-03-01' = {
  name: webSiteName
  location: location
  identity: {
    type: 'SystemAssigned'
  }
  tags: tags
  properties: {
    serverFarmId: hostingPlan.id
    httpsOnly: true
    siteConfig: {
      minTlsVersion: '1.2'
      http20Enabled: true
      ftpsState: ftpsState
      use32BitWorkerProcess: false
      netFrameworkVersion: netFrameworkVersion
    }
  }
}

module registerCustomDnsName 'dns-record.bicep' =  if(!empty(customDomainName) && !empty(dnsZoneName) ) {
  scope: resourceGroup(DnsZoneSubscriptionID, DnsZoneRG)
  name: 'registerCustomDomainDns'
  params: {
    zoneName: dnsZoneName
    dnsRecord: customDomainName
    ARecord: true
    dnsRecordValue: appService.properties.inboundIpAddress
    TTL: dnsRecordTTL
  }
}

module registerCustomDnsTxtRecord 'dns-record.bicep' =  if(!empty(customDomainName) && !empty(dnsZoneName) ) {
  scope: resourceGroup(DnsZoneSubscriptionID, DnsZoneRG)
  name: 'asuid'
  params: {
    zoneName: dnsZoneName
    dnsRecord: 'asuid.${customDomainName}'
    TxtRecord: true
    dnsRecordValue: appService.properties.customDomainVerificationId
    TTL: dnsRecordTTL
  }
}
var fqdn = '${customDomainName}.${dnsZoneName}'
resource siteCustomDomain 'Microsoft.Web/sites/hostNameBindings@2021-03-01' = if(!empty(customDomainName) && !empty(dnsZoneName) ) {
  name: fqdn
  parent: appService
  properties: {
    azureResourceName: fqdn
    azureResourceType: 'Website'
    customHostNameDnsRecordType: 'A'
    siteName: appService.name
  }
  dependsOn: [ registerCustomDnsName, registerCustomDnsTxtRecord  ]
}

module dns-record.bicep
param zoneName string
param dnsRecord string
param TTL int = 300
param ARecord bool = false
param TxtRecord bool = false
param dnsRecordValue string

resource dnsZone 'Microsoft.Network/dnsZones@2018-05-01' existing = {
  name: zoneName

}

resource dnsARecord 'Microsoft.Network/dnsZones/A@2018-05-01' =  if(ARecord) {
  name: dnsRecord
  parent: dnsZone
  properties: {
    TTL: TTL
    ARecords: [{
        ipv4Address: dnsRecordValue
      }]
  }
}

resource dnsTxtRecord 'Microsoft.Network/dnsZones/TXT@2018-05-01' = if(TxtRecord) {
  name: dnsRecord
  parent: dnsZone
  properties: {
    TTL: TTL
    TXTRecords: [
      {
        value: [
          dnsRecordValue
        ]
      }
    ]
  }
}

@RamBNarayanan
It appears that using an empty string for the RGName Or SubscriptionId is not valid/supported.
Please provide an alternate value for these, other than ''.
These zones must already exist, so consider using the correct RGName and SubscriptionId, using an empty string is not currently supported.
Once you validate this works correctly, I will follow up to ensure there is no way to work around this, however I am guessing there i not.

@brwilkinson Please fill in the SubscriptionId and RGname (REsource Group Name) of your test environment there. Sorry I have left that empty as it is specific for each environment. You may also need to provide the values for param customDomainName string = 'myHost' param dnsZoneName string = 'myDomain.com'

HI @RamBNarayanan just getting back to this one.
I am unable to repro.
Can you confirm these are the param settings that you want me to test with?
param location string = 'westeurope'
param netFrameworkVersion string = 'v4.0'
@allowed([ 'AllAllowed', 'FtpsOnly', 'Disabled' ])
param ftpsState string = 'Disabled'
param DnsZoneSubscriptionID string = '4185fa9b-f470-466a-b3ae-8e6c3314a542'
param DnsZoneRG string = 'AEU1-PE-CTL-RG-D1'
param customDomainName string = ''
param dnsZoneName string = ''
param dnsRecordTTL int = 300

Given this is duplicate, I have moved this over to #5974 will close here.


Random seeds

The idea behind this pull request is, as @obo pointed out, that you don't actually want to choose the random seed while you experimenting, and you don't actually need the random initialization to be random. So the option for specifying the random seed is removed from the configuration and the default random seed is set in the code and not configurable.
Moreover, the tensorflow random seed initialization must take place before the graph is constructed, which is not possible with the current implementation of configuration building.

I don't know, shouln't we have a way to have a really random random seed, after all its called random seed. :-)

Once the new configuration is ready (so that the seed can be set before building graph), we can support a special value 'random' and mix time with username and hostname.
But I don't think that this is worth the hassle with ambiguous value types in the config (int vs. string).

listopadu 2016 19:27:31 SEČ, "Jindřich Libovický"<EMAIL_ADDRESS>napsal:

I don't know, shouln't we have a way to have a really random random
seed, after all its called random seed. :-)

--
Ondrej Bojar<EMAIL_ADDRESS>/<EMAIL_ADDRESS>http://www.cuni.cz/~obo

It's only random seed, not random random seed.. :-)
2016-11-29 18:34 GMT+00:00 Ondrej Bojar<EMAIL_ADDRESS>
Once the new configuration is ready (so that the seed can be set before
building graph), we can support a special value 'random' and mix time with
username and hostname.
But I don't think that this is worth the hassle with ambiguous value types
in the config (int vs. string).

listopadu 2016 19:27:31 SEČ, "Jindřich Libovický" <
<EMAIL_ADDRESS>napsal:

I don't know, shouln't we have a way to have a really random random
seed, after all its called random seed. :-)

--
Ondrej Bojar<EMAIL_ADDRESS>/<EMAIL_ADDRESS>http://www.cuni.cz/~obo
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ufal/neuralmonkey/pull/157#issuecomment-263657127,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABwcs1kOrEnh-7I1oFsqDOBSeEbPeq-bks5rDHAugaJpZM4K_N08
.


Migrate to new Go client library credentials

The oauth2 package is no longer the preferred choice for token sources. Instead, we should use:
https://pkg.go.dev/cloud.google.com/go/auth#Credentials
This means adding a new API option, replacing the internals with auth.Credentials, and directing users away from our current WithTokenSource and related options that use oauth2.
Also we should be using<EMAIL_ADDRESS>This is a companion to https://github.com/GoogleCloudPlatform/cloud-sql-go-connector/issues/904.

cc @codyoss @quartzmo Here's the AlloyDB issue tracking migration to the new auth package.


Add tamlt2704 to Contributors list

Add tamlt2704 to Contributors list

@tamit2704, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added Facebook share and tweet feature to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


force encoding to utf-8

read_text takes the default locale... Now fun fact - when packaging ilastik for OSX and starting via double click, this seems to default to ascii

better safe than sorry ;) Who knows what kind of locales people set...

should I go ahead and merge this? Also could we make a post2 release with it (before Friday)?

cool - as it turned out the issue fixed in this PR would also occur on other OSes if with non-utf8 locales... (also discovered during a course :dagger: )

allright, I'll go ahead an merge this! thx @oeway and @FynnBe for the review :)


TypeError

I keep getting this error:
        [✓] You are on the latest version.
Traceback (most recent call last):
  File "/home/pi/EarnApp-Earning-Monitor/app/main.py", line 120, in <module>
    main()
  File "/home/pi/EarnApp-Earning-Monitor/app/main.py", line 42, in main
    info = AllInformation(config.WEBHOOK_URL, api, graphics)
  File "/home/pi/EarnApp-Earning-Monitor/app/functions.py", line 16, in __init__
    self.get_info()
  File "/home/pi/EarnApp-Earning-Monitor/app/functions.py", line 22, in get_info
    self.devices_info = self.api.get_devices_info()
  File "/home/pi/.local/lib/python3.9/site-packages/pyEarnapp/earnapp.py", line 60, in get_devices_info
    return DevicesInfo(json.loads(response.content), self.report_ip_ban)
  File "/home/pi/.local/lib/python3.9/site-packages/pyEarnapp/models/device.py", line 55, in __init__
    self.total_bandwidth_usage += device.bandwidth_usage
TypeError: unsupported operand type(s) for +=: 'int' and 'NoneType'
You can safely exit this

can confirm.
Both docker and non-docker versions are affected.

Will fix it soon.


Selected Product Reduced with tests

Assignee Tasks

[ ] added unit tests (or none needed)
[ ] written relevant docs (or none needed)
[ ] referenced any relevant issues (or none exist)

Guidelines
Please add a description of this Pull Request's motivation, scope, outstanding issues or potential alternatives, reasoning behind the current solution, and any other relevant information for posterity.

Your PR Notes Here
Created the Selected Product Reducer with tests.
Removed the only from describe.only I had previously left on the api product test file products.spec.js

Closes #41


(fix): Handle None in databases data types

Feature or Bugfix

Bugfix

Detail

None is not preserved when inferring strings

Relates

#1880

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 52e41ee142d3dd95897ea29a4087bf71918b9341
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubCodeBuild8756EF16-4rfo0GHQ0u9a
Commit ID: 4b549036e81adc437bd1ff6b04dc7ab59b0ec701
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


C++\error\rightview.cpp

take input:- 1 2 3 4 5 -1 -1 6 7 8 9 -1 -1 10 11 -1 -1 -1 -1 -1 -1 -1 -1
output:- 1 3 5 9 11
this is the right view of a tree..imagine the tree by yourself and find the error
NOTE:- Screenshot of the code and output is required

can you please assign me this issue ??


'Source Provider' & 'Target Provider' drop-down lists are empty while creating mappings

The chosen namespace is konveyer-forklift
Can't create mapping from the new upstream forklift  - 'Source Provider' and the 'Target Provider' drop-down lists are empty though I do have the defined providers.

@rszwajko hi, is this the same problem as the one you identified ?

@polinaag
I assume that the problem occurred in the Plan Wizard, right? Can you provide screenshot of this page (with URL)?
Assuming it's Plan Wizard then the current namespace should be part of the URL. During my tests, empty provider list is (most of the time) caused by namespace differences.  If that's not the case then we need more details about providers.

OK.  The "Add mapping" dialog on the Mappings screen uses default namespace provided at the time of building the plugin which is by default default.
If you want to create a mapping in a different namespace then (currently) the only way is to create(and save) it from the Plan Wizard.
In the near future the Mappings screen will be rewritten to match Console and then it will be possible to pick the namespace explicitly.

thank you , got it

Network mappings screen added in https://github.com/kubev2v/forklift-console-plugin/pull/160
Storage mappings screen added in https://github.com/kubev2v/forklift-console-plugin/pull/186

@yaacov

if we plan to support multiple namespaces then it would be nice to add warning/info near top of the dialog if the namespace was not chosen as the fallback to default might be unexpected
if we plan to limit the namespace choice then please link that issue here

for 2.4.0the fix will be 2 (limit namespace option)
I'll link thefix for the limit namespace here once it's ready

@rszwajko hi, I think this is fixed (works for me), can you check too ?

@yaacov works for me


image from gallery low quality

Hey, I have a funny problem
when I load images from my gallery the resulting image that is being sent is of low quality even if it is a good picture.
my settings are:
}
title: '',
  cancelButtonTitle: 'Cancel',
  takePhotoButtonTitle: 'Take Photo...',
  chooseFromLibraryButtonTitle: 'Choose from Library...',
  cameraType: 'back',
  mediaType: 'photo',
  videoQuality: 'high',
  maxWidth: 800,
  maxHeight: 800,
  quality: 1.0,
  allowsEditing: false,
  noData: false,
  storageOptions: {
    skipBackup: true,
    path: 'images'
  }

am I missing something?
Thanks

Hey @korkinet,
First of all it's a problem on Android or iOS or both ?
To get the best quality of an image don't set maxWidth maxHeight angle

Sorry, my previous post was wrong,Please help me. Need to get videos from device.
var options = {
  title: 'Select Video', // specify null or empty string to remove the title
  cancelButtonTitle: 'Cancel',
  takePhotoButtonTitle: 'Take Photo...', // specify null or empty string to remove this button
  chooseFromLibraryButtonTitle: 'Choose from Library...', // specify null or empty string to remove this button
  // customButtons: {
  //   'Choose Photo from Facebook': 'fb', // [Button Text] : [String returned upon selection]
  // },
  cameraType: 'back', // 'front' or 'back'
  mediaType: 'video', // 'photo' or 'video'
  videoQuality: 'high', // 'low', 'medium', or 'high'

};
  UIImagePickerManager.showImagePicker(options, (response) => {}

No updates here...closing.


IPv6 support ? Apps being rejected by Apple

Did anyone test this on a IPv6 network.
I think it doesn't work because Apple is blow rejecting apps without IPv6 support. And they can't reach login...
(Using Xamarin to build iOS app)

Some solution here ?

There's nothing in our network stack that relies on any version of IP, whatsoever. If you're experiencing a problem, it's probably not on us (I just tested on my IPv6 network, and everything appears to be in order).
Is it perhaps a firewall issue?

I have no idea. Apple review team is not able to login or register. I am not doing anything special just normal login/register that works on my networks. But they said it's probably something with ipv6. So I have no idea what could be the problem only that the sdk is not IPv6 ready.

Assuming you're using parse-server, is it possible that your host is not configured for IPv6 yet? And if you're not using parse-server, we very strongly encourage you to migrate ASAP.

I was using parse server before but because the sdk is not updated to work with gcm push I had to revert back to using plain old parse.
So we are talking about parse native host.

http://api.parse.com does not support IPv6, and will never support IPv6. If you need IPv6 support, please migrate to parse-server.

Thanks for letting me know.
Will drop this parse all together and migrate to firebase or AWS.  Thanks for the trouble

I suggest Firebase. Good luck!


[4.0] [ClangImporter] Always load a class's members before any categories

Cherry-pick of #8444 to swift-4.0-branch for testing purposes, along with #8468 that disables the problematic assertion I added. I still don't consider this finished until there's a way to turn on that assertion again, but this will at least keep the 4.0 branch from running into the original issue in rdar://problem/30785976.

@swift-ci Please test

LGTM, passed tests! Thanks, I'll merge.


Removed redundant cve suppressions

https://tools.hmcts.net/jira/browse/CMC-1957
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[X] No

This PR is failing dependancy vulnerabilibity checks on pipeline. please could you check the changes and if we need to add latest CVSS warnings. it should not fail dependancy vulnerabilibity at the least

This PR is failing dependancy vulnerabilibity checks on pipeline. please could you check the changes and if we need to add latest CVSS warnings. it should not fail dependancy vulnerabilibity at the least

For some reason the dependency checks pass for me locally. It may be configured incorrectly. I've restored the suppressions for the cves which are causing the build to fail.


Android 4.2.2 not supported?

I've tried to install GmsCore on my old Prestigio with (nearly AOSP) 4.2.2 Android, but installation failed. I didn't looked at logcat cause I remembered that UnifiedNlp isn't officially supported for old platforms anymore, and figured this might be the reason of installation failure. So, is there a way for a user to build GmsCore for old Android versions?

I am using GmsCore on an Android 4.2.2 device without any problem. Have you downloaded the file from http://files.brnmod.rocks/apps/GmsCore/Latest/ ? If yes, have you chosen play-services-core-debug.apk?

No, I used apk I've compiled myself. But will try this one later today, thanks for the hint.

Debug version installs. Apparently, there's an issue with the apk I signed using my signing certificate. Thanks Tentos!


Error opening output file '/dev/.null.bc': Permission denied

Dear WLLVM devs,
I tried to use WLLVM to compile a Debian source package for musl to LLVM bitcode, but I get this error:
checking for C compiler... wllvm
checking whether C compiler works... no; compiler output follows:
error: unable to open output file '/dev/.null.bc': 'Error opening output file '/dev/.null.bc': Permission denied'

Do you have any clue why this is happening? If you are interested, I'm attaching a full build log file.
musl-build-cow.txt
Note I've been able to build about 1000 Debian source packages with WLLM so far.
This was with clang 3.4.2.

There is an environment variable for configure only. See the README. Maybe this is what you need to use.

We build Musl  all the time. We even have a fork @ sri-csl. It is not unusual to have to use the configure only flag when configuring. We create files that the configure script can't rip on.
The flag prevents that.

@ianamason, thanks!
So, you pass the WLLVM_CONFIGURE_ONLY=1 variable when calling musl's configure script like this:
WLLVM_CONFIGURE_ONLY=1 ./configure ...

but then later on you don't use the variable? Can you please c/p the steps you make? I've been trying to automate this within a Debian package build system, but it ends in this:
./tools/install.sh -D -m 644 include/wctype.h /tmp/buildd/musl-1.1.5/debian/tmp/usr/include/x86_64-linux-musl/wctype.h
./tools/install.sh -D -m 644 include/wordexp.h /tmp/buildd/musl-1.1.5/debian/tmp/usr/include/x86_64-linux-musl/wordexp.h
make[1]: Leaving directory '/tmp/buildd/musl-1.1.5'
   dh_install -O--parallel
WARNING:Did not recognize the compiler flag "-dumpmachine"
dh_install: musl-tools missing files (usr/bin/*), aborting
debian/rules:49: recipe for target 'binary' failed
make: *** [binary] Error 2
dpkg-buildpackage: error: fakeroot debian/rules binary gave error exit status 2
E: Failed autobuilding of package

Correct. We do:
WLLVM_CONFIGURE_ONLY=1  CC=wllvm ./configure
make

I rarely do installs, so not sure what the last error is about.  See if you can figure out the exact
line that is failing.

Closeable?

@ianamason, yes, closeable. Thank you for the clarification!


Use non-root location

Update the README to copy the script to ~/.local/bin instead of /usr/local/bin/.
This installs only for the current user and doesn't require root.

I thought about that too when initially writing the script, but for me personally, neither is ~/.local/bin a directory, nor is it included in $PATH by default, therefore I decided to use a location which would be in $PATH by default. This probably depends on the setup though. What are your thoughts on that?

Didn't know this is not a standard but that definitely excludes .local. Judging by this we should use /usr/local/bin but without sudo. Installing there shouldn't require root.

Judging by this output, you do need root permission to write to /usr/local/bin:
$ ls -al /usr/local/ | grep bin
drwxr-xr-x  2 root root 4096 Dez 11 00:57 bin

Weirdly it seemed to work for the person in the thread, I guess this could be distro dependent? Still, if you're up for any other ideas, let me know!


Get rid of crossAttrs

It is a needless abstraction. Just use stdenv.lib.optionalAttrs (stdenv.hostPlatform != buildPlatform) { .. }, or more ideally condition on the individual fields.
CC @bgamari

Hmm I forgot about that. I usually think of getting rid of all splicing, which is the sole reason those exist, but that's a lot harder. Just renaming those to match the others is a good first step.


*: update rust toolchain

Signed-off-by: tabokie<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
update rust toolchain to nightly-2021-01-21
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
No release note

Can you describe the motivation of updating it?

Can you describe the motivation of updating it?

@BusyJay Suggested by this comment, the use of as_chunks method requires latest nightly toolchain. I'm not sure how we do the updates regularly, any hidden hazards here?

@BusyJay Suggested by this comment, the use of as_chunks method requires latest nightly toolchain. I'm not sure how we do the updates regularly, any hidden hazards here?

We usually update it every few months. I'm not objecting it, just curious if it's a routine update or for some specific functionality.

We usually update it every few months. I'm not objecting it, just curious if it's a routine update or for some specific functionality.

Could you wait for the 2021-01-25 toolchain for https://github.com/rust-lang/rust/pull/81245? With the latest fix on cargo (https://github.com/rust-lang/cargo/pull/9077), cargo doc will work for resolver v2

Could you wait for the 2021-01-25 toolchain for https://github.com/rust-lang/rust/pull/81245? With the latest fix on cargo (https://github.com/rust-lang/cargo/pull/9077), cargo doc will work for resolver v2

@sticnarf Done.

@sticnarf Done.

/test

/test

/merge

/merge


How to import permissions module on react native project?

I'm new at react native and typescript.I wanna create permissions module for my react native project and I seperated two links at below Yonah's project and Facebook's permissions project
Additionaly

npm install ,npm start , yarn run build --watch and react-native run-android

command running smoothly but permission module dont use it ,I dont present this module add my project.
And my trying link at below ;
const permission = require('react-native-permissions')
import React from 'react'
import {
StyleSheet,
TouchableHighlight,
Text,
View,
Alert,
AppState,
Platform,
PermissionsAndroid,
} from 'react-native'

interface Props {
names: string[]
}

interface PermissionsState {
status: PermissionStatus
}

type PermissionStatus = 'granted' | 'denied' | 'never_ask_again' | ''

export class permissions extends React.Component<Props, PermissionsState>
{
constructor(props: Props, context: any)
// tslint:disable-next-line:brace-style
{
    super(props, context)

    this.state = {
        status: '',
    }
}

public render()
// tslint:disable-next-line:brace-style
{
    return (<View> a </View>)
}
 }

 async function requestCameraPermission() {
try {
    const granted = await PermissionsAndroid.request(
        PermissionsAndroid.PERMISSIONS.CAMERA,

    )
    if (granted === PermissionsAndroid.RESULTS.GRANTED) {
        // console.log("kamerayı kullanabilirsiniz")
    } else {
        // console.log('kamerayı kullanamazsınız')
    }
  } catch (err) {
    // console.warn(err)
  }
  }

  export default permission

@facebook-github-bot stack-overflow

@facebook-github-bot @mkonicek  thanks for comment I know I asked this question on stackoverflow myself,but dont answer suitable,I just try this platform for chance.

@mkonicek Is it any idea this situation :1st_place_medal:

@facebook-github-bot stack-overflow


Update description for generic update arguments

We now support generic update --set on root level (https://github.com/Azure/azure-cli/pull/22798).
Also add a pointer to az rest examples.

Docs Build status updates of commit a3625cc:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/microsoft-graph-migration.md
:warning:Warning
View
Details

docs-ref-conceptual/microsoft-graph-migration.md

Line 99, Column 1: [Warning: disallowed-heading - See documentation] Heading 'H2 'Feedback'' matches the text of a disallowed Docs template heading.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 22abbc5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/microsoft-graph-migration.md
:white_check_mark:Succeeded
View

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

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

File
Status
Preview URL
Details

docs-ref-conceptual/microsoft-graph-migration.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 391cb29:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/microsoft-graph-migration.md
:white_check_mark:Succeeded
View

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


Question about Hidden State Reinitialization in LSTM for Behavior Cloning

Dear Authors,
Thank you again for this great repo!!! I have a question about the rollout/inference using LSTM with Robomimic. I'm confused about the implementation in bc.py. When config.algo.rnn.horizon is set to 10 (the default value in the config files), the hidden state is reinitialized to zeros during inference every 10 time steps.
If I understand correctly, this means that at timestep 10, the hidden state is reset, and at timestep 11, it would lost information from all previous timesteps (even information from timestep 10) due to the reset. This behavior seems a bit counterintuitive to me. Could you please clarify this? Or perhaps I misunderstood the process?
Thank you in advance! Look forward to your reply!
Best regards,
Pengzhi

Your understanding is correct! We found this to work well empirically. Another option is to framestack the observations, and completely unroll on every single timestep - this is actually what BC-Transformer does. However, we found our current approach to work comparably or better, so that's what we have kept as the main implementation for BC-RNN

Thank you so much for your response! But may I ask whether BC-Transformer's performance is better than BC-RNN in most of the tasks?

We saw that it was usually around the same performance on most tasks.

Got it. Thank you!


Refactor: TemplateFinder()

Main changes are in template_finder.py, the remaining changes are to modify the codebase accordingly.
Use a Template dataclass object to store template information rather than a list (before it was like self._templates[key][0] to get image, [key][1] to get grayscale, etc.)
Deconstruct TemplateFinder class
Use @cache to store all templates in _templates().
Extracted a lot of the logic from template_finder.search() into template_finder._single_template_match(), which is now called from search().
Added new function search_all(), which will need integration into shopper bots as separate PR. This new function is included in template_finder_test.py and test passes.
Modify dataclass for TemplateMatch to include normalized monitor coordinates as attributes so convert_screen_to_monitor and normalize_monitor stuff doesn't need to be used.

Requesting seasoned devs to review. @aeon0 or @VladimirMakaev if you get a chance to review the changes to template_finder.py and give me feedback that'd be appreciated. Thanks!

Really like it 😍 I wanted to do pretty much exactly that refactoring to be able to use multiprocessing more easily on the template searching.

Thanks! Any concerns about thread safety?

The only thing I am a bit worried about is the cached _template() which is not threadsafe. Maybe put a lock around it in get_template() and always call that one.

Closes #683


Add Thanapong Prathumchat to Contributors list

I just want to thank you so much for this tutorial, because during my time as a student, I spent most of my time working alone and feeling scare to contribute to open source project as I don't know what or how to start.
I've been using GitHub for years and never quite understand how contribution process works, so this tutorial helped get my feet wet and I can't wait to contribute to many open source projects out there.

@thanapongp, I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://firstcontributions.herokuapp.com


Honeybadger - Change docs article ownership

Change docs article ownership

Only docs to be moved were for biscuit. I've created PRs to move them to rainbow and kaas


[Feature] Show most recent release for each fork

Releases are available in some forks that include changes introduced in the forks in question.
It would be very useful if in the fork report table there was a link to the latest release available for each of these, if available, for example by also including the date on which they were released (sortable in the table).

I think this would be a nice feature to have. Open for contributions otherwise I'll try to get it in in couple of weeks.

Unfortunately, I won't have a chance to work on it in the upcoming days, but in my opinion, it would be cool to add two columns:

Last fork commit. Clickable with the link to the last commit of the fork.
Last fork release. Clickable with the link to the last release of the fork.


Dataset Docs

Hi,
Thank you for sharing the repository! Could you please point me to where I can find the instructions for the dataset?
Best！

Hi! do you mean instructions to create a custom CACo-like dataset or to download and use the existing dataset?
If you mean the latter, then that is available here.
Can you please clarify this?
Best!

Hi! do you mean instructions to create a custom CACo-like dataset or to download and use the existing dataset? If you mean the latter, then that is available here.
Can you please clarify this?
Best!

Hi! Thank you for getting back to me. I was referring to the instructions for creating a custom CACo-like dataset. Based on what I see, the structure appears to be as follows:
clean_10k_geography/
    - 000000/
        - 20171122T105341_20171122T105612_T31UDQ_2197_48923_8339/
            - B*.tif
            - metadata.json
        - 20171122T105341_20171122T105612_T31UDQ_2197_48923_8339.tif
        - OTHER_SUBFOLDERS/
        - OTHER_SUBFOLDER.tif
    - 000***/

Are there also instructions on what each folder should contain or the meaning behind the folder names? Apologies if this is a basic question :( I'm new to this area as a master's student. Thank you so much for your time and patience!


Add folders or workspaces to cluster catalog

What would you like to be added:
I would like to have folders or workspaces to sort the clusters i have to manage.
Why is this needed:
At the moment i have per customer ~10 different kubernetes clusters, which are all listed, without sorting.
Having the option to add folders or select different local workspaces would be great.
I can't sort them by using Team Spaces, due to company policies, regarding the upload of private keys.
Environment you are Lens application on:

Kubernetes distribution: Not relevant, but k3os/rancher
Desktop OS: Windows 11

Thanks for the request, but I am going to mark this as a duplicate of https://github.com/lensapp/lens/issues/2602


Slow query warning in ArangoDB logs (~900 secs)

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[ ] 3.0
[X] 3.1
[ ] 3.2
[ ] self-compiled devel branch

Mode:

[ ] Cluster
[X] Single-Server

Storage-Engine:

[ ] mmfiles
[ ] rocksdb

On this operating system:

[ ] DCOS on

[ ] AWS
[ ] Azure
[ ] own infrastructure

[ ] Linux

[ ] Debian .deb
[X] Ubuntu .deb
[ ] SUSE   .rpm
[ ] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[ ] MacOS, version:

this is an AQL-related issue:
[X] I'm using graph features
I'm issuing AQL via:

[ ] web interface with this browser:     running on this OS:
[ ] arangosh
[X] this Driver:  Python-arango 3.8.0

Here is the extract from ArangoDB logs
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "666c38b7-3e85-4aff-a6e7-ffae8df70a9d"  RETURN doc', took: 895.588065
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "7b0cdc98-dc65-4825-aae4-3c4f16bf092b"  RETURN doc', took: 896.037055
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "d65147ca-93a7-46c0-a4f5-6b53d608ab51"  RETURN doc', took: 898.017288
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "510562c5-de43-4015-a433-59859665f7da"  RETURN doc', took: 895.589848
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "d6a06276-dcf7-47d0-972e-6f893fbf8f83"  RETURN doc', took: 896.458717
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "060fac41-e76b-4222-b014-ed8acabf36ab"  RETURN doc', took: 894.144532
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "2d5b6e19-822a-452e-95ed-81ad3d498274"  RETURN doc', took: 898.019056
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "dac55d52-0b44-4a99-bee3-448a13fd59ff"  RETURN doc', took: 893.768311
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "060fac41-e76b-4222-b014-ed8acabf36ab"  RETURN doc', took: 898.692032
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "fb7236bd-b3ce-4489-9c5e-047170bee223"  RETURN doc', took: 897.015540
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "5eadd1a7-df89-4bc5-a18d-c1290fe7a367"  RETURN doc', took: 896.928456
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "5a6ac13c-a081-4fca-8988-4e7daccea7bf"  RETURN doc', took: 899.066692
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "ec5a6e76-d20f-4602-8298-ee400de66811"  RETURN doc', took: 895.717443
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "41486780-8f56-4bbc-a177-78c956175e33"  RETURN doc', took: 894.277274
2017-11-27T10:38:44Z [1473] WARNING {queries} slow query: 'FOR doc IN @@collection FILTER doc.`_key` == "2c74f8b9-1fb1-4b8d-8764-3adeec2abb3e"  RETURN doc', took: 897.017139

I've run db._explain("<my aql query>") and it confirmed that it uses the index
Query string:
 FOR doc IN @@collection FILTER doc.`_key` == "dc201805-9170-464c-99d3-7fb8868a1904" RETURN doc

Execution plan:
 Id   NodeType        Est.   Comment
  1   SingletonNode      1   * ROOT
  6   IndexNode          1     - FOR doc IN link   /* primary index scan */
  5   ReturnNode         1       - RETURN doc

Indexes used:
 By   Type      Collection   Unique   Sparse   Selectivity   Fields       Ranges
  6   primary   link         true     false       100.00 %   [ `_key` ]   (doc.`_key` == "dc201805-9170-464c-99d3-7fb8868a1904")

Optimization rules applied:
 Id   RuleName
  1   use-indexes
  2   remove-filter-covered-by-index
  3   remove-unnecessary-calculations-2

The AQL query in question is:
FOR doc IN @@collection FILTER doc._key == "dc201805-9170-464c-99d3-7fb8868a1904" RETURN doc.
We have seen slow query warning in other AQLs ( upto ~ 10 secs execution time ) , but the above AQL is the most frequent and slowest one.
Note that the timings are always almost ~899-900.

We are unable to co-relate this slowness with load/transaction rates. i.e the time when we saw these errors were when there was normal/negligible load on the system and not peak load.

What storage engine are you using? Whats the rest of the load on that collection during these reads? Do you have writes to that collection in a similar timeframe?

Hi @dothebart ,
The engine is the default "mmfiles".
There were very not much writes into the collection when compared with the reads.
In the timeframe under investigation, it was all reads.

Dealing with a similar issue -- were you able to resolve this?

Hi,
can any of you provide a script that will reproduce the issue with a provided dataset?

Hi @falmiski-cr,
Just sending a quick follow up. Can you please provide us the dataset that you are using so that we can reproduce the issue and investigate more?
Thanks.


More control over whitespace in generated templates

Love this project.
I think one feature that would be very helpful here is better control over the whitespace in generated files.
In many cases we end up with weird formatting or indentation in the expanded templates if the templates themselves have indented blocks.
I am not sure what is the best way to tackle this but I think this can be a major problem for whitespace sensitive languages like python, coffeescript, F# etc.

Yeah, this is something that definitely needs improvement.
I am contemplating moving nunjucks as the primary templating solution as it supports better control over whitespace in generated code.


Add boto3-stubs[secretsmanager] as a dependency

fix: https://github.com/ceb10n/pydantic-settings-aws/issues/2

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Add some basic stats to the Airflow integration

Problem
We currently don't have a good way to understand the latency of emitting openlineage events from the airflow integration.
Beyond logging, there also isn't an easy way to aggregate failed attempts of sending the events.
Solution
This PR uses the statsd component that already exists in the Airflow codebase and wraps the section that emits to event with a timer, as well as emit a counter for exceptions in sending the event.
Checklist

[ ] You've signed-off your work
[ ] Your pull request title follows our guidelines
[ ] Your changes are accompanied by tests (if relevant)
[ ] Your change contains a small diff and is self-contained
[ ] You've updated any relevant documentation (if relevant)
[ ] Your comment includes a one-liner for the changelog about the specific purpose of the change (if necessary)
[ ] You've versioned the core OpenLineage model or facets according to SchemaVer (if relevant)
[ ] You've added a header to source files (if relevant)

SPDX-License-Identifier: Apache-2.0
Copyright 2018-2023 contributors to the OpenLineage project

How do we follow up on this? @harels ?

Codecov Report

Merging #1845 (340b6ce) into main (ba6f1d9) will increase coverage by 1.43%.
The diff coverage is 100.00%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@             Coverage Diff              @@
##               main    #1845      +/-   ##
============================================
+ Coverage     80.95%   82.39%   +1.43%
============================================
  Files            90       54      -36
  Lines          3781     3152     -629
  Branches         33        0      -33
============================================
- Hits           3061     2597     -464
+ Misses          681      555     -126
+ Partials         39        0      -39

Files
Coverage Δ

integration/airflow/openlineage/airflow/adapter.py
86.86% <100.00%> (+1.45%)
:arrow_up:

... and 38 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

How do we follow up on this? @harels ?

I've added tests.
Once I get approve from @tatiana we're good to merge I think.

Thanks for adding the tests @JDarDagran!
Would you like me to add them to the Airflow provider OR?

awesome! Great collaboration!


Hook workspace fk up to the correct table in migration

Tickets addressed: AERIE-000
Review: By commit
Merge strategy: Merge (no squash)

Description

There was a typo in migration 9 that added an fk to the parcel table instead of the workspace table.
Verification

I have confirmed that the code is now an exact match to the files updated.
Documentation

Future work

Investigate how this wasn't caught by our automated dbcmp tests.

As long as checks pass, g2g.

The thing is, the checks passed initially somehow 🫠


refactor: VC extension of Credential Subject

closes #1943
Signed-off-by: Dima<EMAIL_ADDRESS>

@kdimak this looks like it solves #1943:

in order to build credential subject of a custom structure, we need to marshal VC and then unmarshal it back into extended VC with the custom credential subject field.
We need a better approach to avoid extra marshalling / unmarshalling of the whole VC.

But as a user I have a request: can we make verifiable.ParseCredential() output a custom credential struct?

But as a user I have a request: can we make verifiable.ParseCredential() output a custom credential struct?
It's complicated. Just because we are using rawCredential structure for the initial VC parsing and then map it to the Credential. So we need to apply "injection points" where we could change the default mapping. Also, it will lead to the API break (of ParseCredential()) as we need to pass a custom credential struct as a parameter I believe. In contrast, we now return created Credential structure.


Re-training a model

Is it possible to re-train a model? If so how?

Yes, it supports re-train the model. If you want to re-train a model, just re-launch the same command line (the code will check if there is already a model in you path. if so, it will load the model and train based on the model).
For more details, can you have a look at lines 137-162 and lines 243-246 in SentenceMatchTrainer.py


fix: default assets when no defaultAssetIdPair pair

Description

Fixes default asset setting on route change
Adds test so it doesn't break again! "If you like then you should put a test on it".
Bumps Cypress to 10.11.0

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[x] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
Closes https://github.com/shapeshift/web/issues/3148
Risk
Small
Testing
Default assets should now populate correctly when navigating to an asset route.
Engineering
☝
Operations
Check UTXOs, Avalanche and Ethereum assets.
Screenshots (if applicable)
N/A

ty for the thourough testing @gomesalexandre!
Made an issue for your edge-case discovery: https://github.com/shapeshift/web/issues/3208


Exchange wrappers should refund extra tokens to msg.sender

What does this mean? Could funds somehow be lost?


Go 1.16 Knative 0.25 and Parallel E2E

Changes

:broom: Splits preexisting Func actions
:broom: Updates to Go 1.16
:broom: Updates to Knative 0.25

/kind chore

@googlebot I consent.

@lkingland just a gentle ping to check the status of bump to Knative 0.25? Is there anything I can help with?

@lkingland haven't you just forgotten to run ./hack/update-codegen.sh and commit results, because it works for me.


Any chance of modifying this to work with lazy.nvim?

Appears to work somewhat, but have to just load the defaults and then set it to disabled in the loader for the module with lazy.nvim.  It still loads and runs, but doesn't run any of the config so just left with the defaults.
I can provide more details if you'd like.

i try with nvim that nvim -u lazy.lua and it work
-- file lazy.lua
local lazypath = vim.fn.stdpath("data") .. "/lazy/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system({
        "git",
        "clone",
        "--filter=blob:none",
        "https://github.com/folke/lazy.nvim.git",
        "--branch=stable", -- latest stable release
        lazypath,
    })
end
vim.opt.rtp:prepend(lazypath)

require("lazy").setup({
    {
        "windwp/windline.nvim",
        config = function()
            require('wlsample.airline')
        end,
    }
}, {})

This is my fault, I was making a stupid mistake in my config.  Thank you for the reply, it helped me get it sorted!


EPIC Implement new prisma init CLI flow

See https://prisma-specs.netlify.com/cli

3 RPCs:

Check if database exists

TODO fill @timsuchanek

Ideally should run on a single connection instead of a pool, so that we don't need to wait 3sec until the pool times out if the connection can't be established.

Related: https://github.com/prisma/prisma2/issues/399

This is implemented in<EMAIL_ADDRESS>please open a new issue if you run into a specific bug/feature request. It is build as per this spec: https://prisma-specs.netlify.com/cli/


feat(Illustrations): add MoneyBack illustration

Overview

PR Checklist

[x] Related to designs: https://www.figma.com/file/O2isc8G1O05RRCXzhd1E6t/%24BG-Lower-Tier-Pricing?type=design&node-id=232-61392&mode=design&t=7HcnNXFaq8oec3et-0
[x] Related to JIRA ticket: FLOW-581
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change
[ ] This PR includes testing instructions tests for the code change
[ ] The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.
PR Links and Envs

Repository
PR Link
PR Env

Monolith
Monolith PR
Monolith Env

Portal
Portal Link
Portal Env

Another Repo
Another Link
Another Env

📬Published Alpha Packages:@codecademy/gamut@55.6.1-alpha.03221a.0@codecademy/gamut-illustrations@0.39.3-alpha.03221a.0@codecademy/gamut-kit@0.6.347-alpha.03221a.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6511c421a1f7c15491b56fa3--gamut-preview.netlify.appDeploy Logs


Validate that async component contains a default export

In case you forget to export async components as default the load hangs forever. This is clearly a programmer error so throwing will help debugging.

Looks good to me. Can you sign the CLA? @lassedamgaard

Merging for now, but let's get it signed too ;-)

!merge


Resize

Great plugin!
However the window resize is not managed to position the arrows.

@mknecht You didn't look into this yet, did you? :+1:

@zkanda nope … my backlog is immense, atm :/


Center UserHeader to match steemit cover picture

Before Fix

After Fix

Fixes # .
https://github.com/busyorg/busy/issues/910
Changes:

add flex property - justify content: center to UserHeader contents

I don't think #910 is a real issue, i prefer aligned to left. Similar than Facebook and Twitter.

@bonustrack alright your call, closing this then :)


Compute shader write to vertex buffer question

Hello and first of all let me just thank all of you for making this awesome library!
Now to my question. Is there a way to make a compute shader output to something like a vertex buffer for subsequent rendering? Is there some already available example perhaps, or would someone be willing to give me some pointers on how to achieve that?
Best regards.

IMPORTANT: READ FIRST!
Issue tracker is ONLY used for reporting bugs.
If you have any questions please do not open issue, rather:

If you have quick question, ask on Discord: https://discord.gg/9eMbv7J
Ask in discussions: https://github.com/bkaradzic/bgfx/discussions
Search documentation: https://bkaradzic.github.io/bgfx/

New features should be discussed on:

GitHub Discussions https://github.com/bkaradzic/bgfx/discussions
Discord Chat https://discord.gg/9eMbv7J


The set-output command is deprecated

Describe the bug
When run, the following warning is received. Would you make the suggested change? Thanks!
"The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files."
To Reproduce
Steps to reproduce the behavior:

Run the action

Expected behavior
No warning.
Screenshots
Action Environment (please complete the following information):

OS:  linux

Additional context

@wlwyrick I couldn't reproduce the warning you've reported. Here is a recent run of this action: https://github.com/robinraju/release-downloader/actions/runs/3267845900/jobs/5373546975
Could you please add a screenshot or link to your failed run (if its public)

The repo is private, so I can't send a link.

I got this warning multiple times and replaced all the set-output commands in the workflow, but still got one remaining. Since the workflow only uses one action, I thought this was what was causing the warning.
If you don't use set-output, then it must be caused by something else.

Hi there,
I made a minimal project to reproduce this issue. (https://github.com/jonz94/release-downloader-issue/actions/runs/3269517029/jobs/5377102326#step:3:16)
And I also found that the issue can be fixed by upgrading @actions/core package to latest v1.10.0 version.

The pr from @jonz94 is merged. You may test it using the commit hash.
- uses: robinraju/release-downloader@62d3e983c99c16718304fb1a69553f01b84b0d40
  with:
    latest: true
    fileName: "foo.zip"


basicfont doesn't support ° (degrees) character

Need to get rid of that character in the output to the screen. It'll be nice to keep in the logged output if possible.

Fixed in #6


Update the UI using the answers in the view model

We had not implemented this because we assumed that it's ok to render the UI without providing an initial value until we implement the initial value to the SDC specification.
However, this already caused issue because the recycler view reuses the questionnaire item views, and if we don't try to set the answer the view might still carry what the user had input for a different question.

thanks yigit :) ptal!

thanks yigit :) ptal!


Move all of Travis-CI steps to GHA

[X] default tests #53
[X] numba tests #54
[x] documentation
[x] artifact publication
[x] coding standards

@kwilcox this last one has been failing for a while and to "fix it" would cause a lot of code churn. So,
1 . do you want to adopt pre-commit for it and fix or;
2. just remove it and don't botther?

Done!


chore(netcdf4-hdf5): Suggested Edits

Suggested edits for netcdf4-hdf5:
Changing:

"is bearing" -> "has borne"
"intermediate service, like [Hyrax]" -> "intermediate service like [Hyrax]"
"promising results" -> "insightful examples"
"things to be aware of" -> "things to be aware of in"
"details have changed, however" -> "details have changed; however, ..."

Thank @jjfrench please merge!


SAP HANA XS Advanced, Connecting to the WebIDE to begin development

Hi Craig
Could we change this paragraph so HXE users do not get confused about the reference to the server landing page for now?
I'm busy building a doc with an assessment of what we need to improve for HXE, and think actually having a server landing page is something we should think of - your thoughts?
R

Hi, Rudi! I'm stepping in here as I'm updating those tutorials. I'm a little lost on what paragraph you mean and I agree a server landing page like the CAL instances have would really help.


fix(keymaps): better up/down keymaps description

j/k are shown correctly but I added a description anyways for consistency

Thanks!


[HOLD for payment 2024-02-12] Card transactions are showing as Cash on the transaction’s preview component

Problem
Card transactions are showing as Cash on the transaction’s preview component:

Assign a >0 dollar limit on an an Expensify Card
Spend a small amount on the card
View the transaction preview:

Expected: The preview should say ‘Card’
Actual: The preview says ‘Cash’
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0197b83f88abe42218
Upwork Job ID:<PHONE_NUMBER>896799744
Last Price Increase: 2024-01-31

Doesn't seem like automation worked on this one, but PR hit prod on Feb 5. So payment is due here @johncschuster

[@mollfpr] The PR that introduced the bug has been identified. Link to the PR:
[@mollfpr] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

I can't find the offending PR.

[@mollfpr] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

The regression step should be good.

[@mollfpr] Determine if we should create a regression test for this bug.
[@mollfpr] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Pre-requisite: account with an imported corporate card, e.g. Discover and a free workspace

In OldDot, Settings > Domains select the imported card and navigate to the Settings tab
Select the free Workspace as the preferred workspace
Update the card so that it creates transactions
In NewDot, verify that the transaction preview shows as a Card

@johncschuster Could you create the payment summary? I'll do a manual request in NewDot, thank you!

Summary
Contributor+ Reviewer: @mollfpr - $500 to be paid via Manual Request

@johncschuster are we good to close this one since the payment is via NewDot?

Yep! I'll get the regression test issue going now. We can close this up!

$500 approved for @mollfpr


[WIP] #261: Add fuzz test

First time working with go 🙏 . With this PR I'm trying to capture expectations and get early feedback.
Looking for pointers to improve and extend this PR - specifically  (but not limited to):

better way to fuzz test these properties
changes required to capture such tests in a mage command


benchmarking besu - optimal hardware specs

Greetings,
We've run some benchmarks for besu (besu/v20.10.1/linux-x86_64/openjdk-java-11)  on the following hardware:
4 nodes (each one a separate VM), ubuntu 18.04, 2 seconds block interval, ~infinite gas limit, IBFT2.0
4 x 32 core CPU (CPU optimized VM), 256GB, NVME SSD 7500 to 15000 IOPS, 3Gbps internal network, accelerated networking on
and
4 x   4 core core CPU(CPU optimized VM) , 64GB, P50 - 7500 IOPS, 250 Mbps, 3Gbps internal network, accelerated networking off
and the results are similar, about 1900 TPS, while the hardware difference is huge
Theoretically  TPS = (GAS Limit ) / (TxGas * BlockTime)
so we were wondering what is the limiting factor and how we can optimise our tests to get more TPS
is there a testing procedure that we can follow to get accurate benchmarking results ?

Transaction processing is still single threaded.  Each TX can be influenced entirely by each previous tx.  There are internal discussions on how to parallelize that step but haven't been implemented yet.

also, is your test harness public?

thanks for the response. what do you mean by test harness ?

The code you are running to (a) set up the System Under Test (the nodes) if automated, and (b) the code that runs the transactions and loads them into the sytem.  and possibly (c) gathers the stats and reports for the test.

I'll have to ask if I can show the code but, in short I do the following:

install manually binary in 4 VMs following your tutorial
transfer the ETH to 10k accounts by a simple script
from as many RPC nodes as possible (I think I tested with 4 separate VMs) I transfer eth between the accounts.
this is not optimized like, auto incrementing the nonce, it's just how you would do it in a real program. I'm thinking for it to be more "real" I should send just the signed transactions.... maybe I'll do that in the next version of the test program.
I look in the blockchain explorer how many blocks were included

The idea is to have as many RPC nodes as possible, because if you ...try too hard.. you would get some TCP errrors address not found. So some auto devops tool like terraform, ansible, chef etc. should be used to create the VMs and destroy them after.... but I didn't reach that part yet.


fix: Trivy output can be modified

Signed-off-by: Jorge Turrado<EMAIL_ADDRESS>
ATM, the output is always trivy.sarif but for PRs should be '' because otherwise the output is not routed to the agent console.
After solving the issue, trivy has detected a vulnerability in one dep, so I bumped it too
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e rabbit*