Make Liquid Drops writable or skip html output

[x] I tried updating to the latest version.

[ ] I can't, there is an issue.

[ ] I Am on Windows
[ ] I Am on Bash on Windows
[x] I Am on Linux
[ ] I Am on OS X

Description
As suggested in issue #448 it would be useful to have writable liquid drops. They could come in handy if you only want to include the asset and need some information about it, but you don't want the html to be generated in place.
Possible implementation
I think it could be implemented in two different ways (just leaving random ideas here):
Option 1) Referencing an asset which hasn't been processed yet will implicitly trigger its inclusion both in the list and the output folder:
Link text
Option 2) Use a specific keyword to add an asset without outputting any html (nohtml keyword used here for the sake of the example):
{% asset img/image1.jpg nohtml %}
If that's not possible
At the moment I have an hacky workaround: create a dummy.html page which won't be shown nor accessible by the users:

{% asset img/image1.jpg %}
{% asset img/image2.jpg %}

This allows me to use the following code, referencing the processed assets in the output folder:
Link to image 1
Link to image 2

I see what you meant earlier now, and what you want to do is already possible, please see the documentation: https://github.com/envygeeks/jekyll-assets#arguments
If the purpose of wanting writable drops, I don't think that's a valid reason to change their behavior.

You'll have to excuse me but I can't understand how can I achieve this looking at the documentation about the argument (I am feeling dumb right now ehehe). Could you make an example please?
Version 2.x used to process all the assets matched by the assets filter, which required some fancy regex writing to prevent a load of useless or unwanted files to be processed. Now, without the assets key in the options all the assets that are not expicitly uset are not processed and this is a great change, I love it! But this change combined with the lack of {% asset_path %} tag creates the need I exposed in this ticket.
I have no problem in keeping using the dummy html page hack, it's just to understand better.

The documentation clearly states to use {% asset yourAsset @path %}.

Sorry didn't mean to close.

Version 2.x used to process all the assets matched by the assets filter in the options

This is now the preload key, and it accepts shell expandables like * (actually that's all we accept) so you can precompile actual assets, but if you have assets that need to be copied we now also have raw_precompile which skips the pipeline and just copies large sets of data (like emoji's from Twemoji npm module.)

I love it! But this change combined with the lack of {% asset_path %} tag creates the need I exposed in this ticket.

{% asset yourAssest.jpg @path %} as the documentation states.

Yep, @path is exactly what I was looking for.
 The maning and use of those arguments weren't clear at all (at least for me).
Also, thank you for the clarification about precompile and raw_precompile. I was wondering where the old asset option had gone and I didn't get the meaning of these two new options.
Maybe the documentation would need some integrations in order to be clearer and provide more examples. One could feel a little overwhelmed by the whole jekyll-assets at first and even somebody who has been using it for quite some time and took his time to experiment and understand deeply how v2 worked, could feel lost with v3. I know one could simply read the source code, but I don't know ruby!
Anyways, excuse my digression, thank you.


Finish screenshots of Making changes with GitHub Desktop

Step 2 (Add new files to the cloned directory) and Step 3 (Make a conflicting change) both are missing screenshots

Added a bunch of screenshots in commit https://github.com/MikeTrizna/github-without-command-line/commit/c68f0542bc134b5d8ea37253306083cbed7f196a


Added repository name as notify variable

In some cases it would be beneficial to have the repository name available for messages as a 5th variable. My specific use-case was while using the slack plugin. I have many repositories, so having the message read like austenc is deploying "repo_name:master" on "staging" would be a lot more helpful. When you have several repositories, it's hard to tell which master branch was pushed up! Perhaps you know of a better way of getting the repository name, but this is what I came up with. If this PR is accepted I'll send one for the slack plugin config file comments too notating the new variable.
Hopefully you find this helpful! Cheers!
Edit: On second thought, one could manually include this info with their message in the config file, this just automates it.
That's what I've done for now, but I'll leave this here in case you deem it useful. Thanks for rocketeer!

Look good. Needs to be targeted at the develop branch though

@Anahkiasen do you want me to re-submit the pull request under the develop branch? Sorry about that.

Yup


Make the tests run faster

Right now, it takes about one second per end-to-end test. Having them be ~instant would be nicer.
Any interest in fixing this, @dpino or @aperezdc ? :-)

OK, I think it makes sense. check is mainly used for running the tests.

I have a patch for this. I will send it later.

Thanks, looking forward to it. :-)


update ember-rdfa-editor from 3.10 to 4.0

Overview
bumps ember-rdfa-editor to the next "major" release. Includes some nice fixes for ember-nodes, so handy for the current tickets that are being worked on.
How to test/reproduce
Check the release notes: https://github.com/lblod/ember-rdfa-editor/releases/tag/v4.0.0
Two most important aspects:

stopEvent and ignoreMutation work differently for ember-nodes. This should just work better and not break anything. But to "test" this, you should see if plugins using ember-nodes still work as expected. (think of key-bindings and the likes)
doc nodespec needs a toDom function. This is done by using docWithConfig. Without this, error will happen.

Challenges/uncertainties
I haven't done bumping of dependency before, so don't make the assumption this is the correct way to do it. Please actually review :)

It seems this update was stopped in https://github.com/lblod/ember-rdfa-editor-lblod-plugins/pull/196
I don't see anything else (plugin-wise) that would need to be fixed before updating though.
What was the reason for the initial revert? Is this actually fixed now?


all services syslog output writing to log file

Changes
docker-compose.yml configured for syslog container
How to test
Start sdx-compose and ensure logs/messages contains all syslog messages (as in console)
Who can test
Anyone but @jjeane

Checked out and confirmed as working.


FIX(SLA): Finished SLA workflow

This PR finishes the DAM workflow and modifies the API for generating the template. Check README.md for examples on how to call the API.

Current coverage is 32.55%

Merging #187 into master will increase coverage by +0.45% as of cb191a9

@@            master    #187   diff @@
======================================
  Files          319     319
  Stmts        11676   11731    +55
  Branches      1472    1474     +2
  Methods          0       0
======================================
+ Hit           3749    3819    +70
- Partial        387     393     +6
+ Missed        7540    7519    -21

Review entire Coverage Diff as of cb191a9

Powered by Codecov. Updated on successful CI builds.

This PR also handles #164

Please @MicheleGuerriero , @szenzaro : Could you have a look at it? If it looks good, please contact @andreaturli or @perezp to merge.

@rosogon I had a look and wrote some minor comments. Overall to me the PR looks fine. Please make sure that the sequence diagram is fully compliant with the service definition (for example it should be POST /seaclouds/commands/agreements if I'm not wrong), since I think this diagram could be really useful for who is going to use the SLA-service! Thanks.

Thanks @MicheleGuerriero . I will make sure the sequence diagrams and the code are aligned.

As this involves the API Proxy in #202  what's the status of this?

@adriannieto : The code in this PR is independent from #202 except in DateTimeAdapter, where you will have a conflict very easy to fix (the right lines are in 202).
I expect we can merge this.

If you want we could finish the review of the #202 and then merge this.

Hi @MicheleGuerriero : I reviewed the sequence diagrams and they match the service except in one minor thing, already fixed.


Fix readme typo

Issue #237

Fix readme typo
Changes

fix-readme-typo

Flags

Related Issues

Issue #
Pull Request #

Fixed typos in the Readme file. @irmerk mam plz review can we have labels here as of Hackoberfest.


Extract Function Enhancements

With the extract function added in https://github.com/Alexandre-Fernandez/astro-i18n/pull/15 (thank you!!!) it's a really good start!
It supports the use case where you are only storing keys. Another use case is where you're storing the actual string of your source language (i.e. English) in your JS/TS files directly.
So instead of a function like this:

{t("index.content")}

You would instead do this:

{t("I'm the title of a page. I have white spaces, punctuation, and all kinds of things that could make escaping difficult!")}

Right now this isn't very robust. For example, if the string of text ends up wrapping like this:

{
      t(
          "I'm the title of a page. I have white spaces, punctuation, and all kinds of things that could make escaping difficult!"
      )
    }

It won't extract.
All this to be said, I appreciate the many different edge cases this could potentially open up. Would you be open to a system that's parsing the AST of an astro file instead of a more brute-force regex method? If so then I could look into opening a PR for it potentially.

Indeed that would be a much less hacky way to do it.
If you want to help out, I'll gladly merge your PR !

@lorenzolewis I fixed the extraction feature for most use cases.
It should always work except if you have ") or ') in your string.
Tell me if you want to improve this further, else I'll close the issue.


Add contributing doc

This documents the general guidelines for PRs and how to get started with a fresh checkout of the repo.

@blampe @breerly

LGTM
cc @webmaven

A belated :+1: from me.


Rdk 34729 2206

with https://github.com/rdkcentral/ThunderInterfaces/pull/173

@npoltorapavlo - Pls rebase and fix the conflicts.


Fix webhook requests for headers with a symbol key

Description
Webhooks fail when there is a header with a symbol as a key:
ShopifyAPITest::Webhooks::WebhookRequestTest#test_with_symbol_headers:
NoMethodError: undefined method `sub' for :clearance:Symbol
Did you mean?  stub

This happens e.g. when using Clearance, which adds :clearance as a header, containing the Clearance session. This means that webhooks will always fail as soon as you include Clearance into the project. Its middleware always adds that header, even if there is no authenticated request.
How has this been tested?
Test case within this PR.

[x] I have added tests that prove my fix is effective or that my feature works.
[x] I have updated the project documentation.
[x] I have added a changelog line.

Seems we got a couple of rubocop failures, could you please address those so we can merge?

@paulomarg Fixed!

Sorry for the delay in getting back to you @phylor, but it seems like we still have one failure when running the sorbet checks (bundle exec srb tc). Can you make sure that's passing so we can merge the PR? Thank you!

Sorry for the delay in getting back to you @phylor, but it seems like we still have one failure when running the sorbet checks (bundle exec srb tc). Can you make sure that's passing so we can merge the PR? Thank you!

No problem, sorry for the failures 🙂 Rebased on main and sorbet is green for me locally.

@paulomarg Sorry for the trouble. We're now getting Rubocop issues not relevant to my changes. Shall I still fix them in this PR?

No need, we'll fix those! Thank you.


Setup tools pip wheel failed with error code -5

I'm having difficulty with the virtual env: virtualenv --no-site-packages -p python3 .
Here's the error:
Enriques-MacBook-Air:qstraderp3 enriqueromualdez$ virtualenv --no-site-packages -p python3 .
Running virtualenv with interpreter /anaconda/bin/python3
Using base prefix '/anaconda'
New python executable in /Users/enriqueromualdez/venv/qstraderp3/bin/python3
Not overwriting existing python script /Users/enriqueromualdez/venv/qstraderp3/bin/python (you must use /Users/enriqueromualdez/venv/qstraderp3/bin/python3)
dyld: Library not loaded: @rpath/libpython3.6m.dylib
Referenced from: /Users/enriqueromualdez/venv/qstraderp3/bin/python3
Reason: image not found
Installing setuptools, pip, wheel...
Complete output from command /Users/enriqueromual...traderp3/bin/python3 - setuptools pip wheel:
dyld: Library not loaded: @rpath/libpython3.6m.dylib
Referenced from: /Users/enriqueromualdez/venv/qstraderp3/bin/python3
Reason: image not found
...Installing setuptools, pip, wheel...done.
Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/virtualenv.py", line 2328, in
main()
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/virtualenv.py", line 713, in main
symlink=options.symlink)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/virtualenv.py", line 945, in create_environment
download=download,
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/virtualenv.py", line 901, in install_wheel
call_subprocess(cmd, show_stdout=False, extra_env=env, stdin=SCRIPT)
File "/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages/virtualenv.py", line 797, in call_subprocess
% (cmd_desc, proc.returncode))
OSError: Command /Users/enriqueromual...traderp3/bin/python3 - setuptools pip wheel failed with error code -5
The initial installation failed, so I reinstalled anaconda, deleted exiting virtualenv, but now I can't seem to get past this step..
Thank you!

Hello all,
I managed to solve this issue by conda installing virtualenv and updating anaconda.
It turns out virtualenv was not automatically installed with Anaconda.


Android example for Instagram

It would be great if we can have a example build for an Android app so that the best practices are followed in implementing other OAuth providers.
Thanks.

To do this the server side code is already implemented. See this part of the code for the auth redirect:
https://github.com/firebase/custom-auth-samples/blob/master/instagram/app.js#L119-L126
And this for the code exchange
https://github.com/firebase/custom-auth-samples/blob/master/instagram/app.js#L133-L148
Basically the Android/iOS native app has to follow this flow:

Generate a random Cryptographic nonce and save it to memory to check later.
Send the user to the /instagram-mobile-redirect?state=<nonce> URL of the server (use the nonce generated in step 1) in a Chrome Custom tab (or equivalent in iOS).
On the browser, the user is redirected to the Instagram sign-in page and is asked to grant access to your app. Once the user has gone through the auth flow (he granted or denied access to the app) he is redirected to a custom scheme URI instagram-sign-in-demo://instagram-mobile-callback?code=<code>&state=<nonce> the app needs to intercept this URI (i.e. in android add an intent filters for incoming links with custom scheme URI) and read the code and state URL parameter values.

PS: you should change the custom scheme on this line to have your own.

Check that the nonce in the state parameter equals the one saved in memory at step 1 to avoid session fixation attacks.
Send an HTTP request to the /instagram-mobile-exchange-code?code=<code>, this will return the Firebase custom auth token that you can use to sign in your user to Firebase.

@bojeil-google in case he has anything else to add or other techniques he wants to discuss.

Seems reasonable. A few recommendations:

consider hashing the nonce when sending it to the instagram auth endpoint and store the plain version on the app.
At end of the flow pass the plain stored version. On the token endpoint, you would hash it and compare it with the one returned from instagram. This makes it harder for an app that intercepts the initial redirect to get the original nonce.

On Android, consider using FDL (Firebase Dynamic Links) links to redirect back to the app for additional security. As custom schemes in android are less secure since they don't have the controlled and regulated ecosystem that Apple has and any app can claim a custom scheme.


Support plugins for the xmodule.CourseField class to avoid monkey patching

Overview
Both of the Course Access Group and the Zoom XBlock repositories monkey-patch CoruseFields (TBD: add links). This is bad design, we need pluggability to make it better.
Solution
The idea is to have a setting for extending the CourseField class:
# cms/env/aws.py
COURSE_FIELDS_MIXINS = ENV_TOKENS.get('COURSE_FIELDS_MIXINS', [])

# common/lib/xmodule/xmodule/course_module.py
class CourseFields(object):
   # should be renamed to something like `BaseCourseFields`
   # whatever course fields we have by default

# Naming is only for demonstration, but this would eventually be the real `CourseFields` class.
NewCourseFields = type("NewCourseFields", [CourseFields] + settings.COURSE_FIELDS_MIXINS, {})

A similar logic already exists in the platform for only for all XBlock types, which include both course and individual XBlocks:
XBLOCK_MIXINS = (
    LmsBlockMixin,
    InheritanceMixin,
    XModuleMixin,
    EditInfoMixin,
    AuthoringMixin,
)

Not needed now since we're using regular Django modules at the moment for simplicity.


⚠ Bofrost.lu has degraded performance

In c787a81, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 884 ms

Resolved: Bofrost.lu performance has improved in 30eb038 after 9 minutes.


Version comparison fails between 0.XX version and 0.XXX version

What version of Hugo are you using (hugo version)?

$ hugo v0.88.0-ACC5EB5B+extended darwin/amd64 or

Does this issue reproduce with the latest release?
No

It seems the solution introduced with https://github.com/gohugoio/hugo/commit/0602135fd44b0cfa0a51b0ec6e451ae58ac95666 does not work with comparison between 0.XX and 0.XXX
1. Good
# Using hugo v0.100.0-27b077544d8efeb85867cb4cfb941747d104f765+extended darwin/amd64 BuildDate=2022-05-31T08:37:12Z VendorInfo=gohugoio
{{ printf "%#v" hugo.Version }}
{{ printf "%#v" (ge hugo.Version "0.104")  }}

👇
"0.100.0"
false

2. Good Also
# Using hugo v0.88.0-ACC5EB5B+extended darwin/amd64 BuildDate=2021-09-02T09:27:28Z VendorInfo=gohugoio
{{ printf "%#v" hugo.Version }}
{{ printf "%#v" (ge hugo.Version "0.89")  }}

👇
"0.88.0"
false

3. Not Good
# hugo v0.88.0-ACC5EB5B+extended darwin/amd64 BuildDate=2021-09-02T09:27:28Z VendorInfo=gohugoio
{{ printf "%#v" hugo.Version }}
{{ printf "%#v" (ge hugo.Version "0.104")  }}

👇
"0.88.0"
true

OK, thinking about it I think this is a bug that was actually fixe back when we reached ... 100. The problem is that we cannot go back and recompile all older versions of Hugo that had this bug, so I'm not sure if this is possible to fix.

Resolved a while ago. Testing with v0.120.4...
{{ lt "0.88" hugo.Version }} → true
{{ lt "0.88.0" hugo.Version }} → true
{{ eq "0.88" hugo.Version }} → false
{{ eq "0.88.0" hugo.Version }} → false
{{ gt "0.88" hugo.Version }} → false
{{ gt "0.88.0" hugo.Version }} → false


Adding New Course: Scala

also add in readme file

@avinash201199  Added in readme file

@avinash201199 please add hactoberfest-accepted label to the PR

hey , there was major bug , due to that I have reverted few pr , please create new pr  to resolve this conflict


KFServing : No IngressNotConfigured and Virtual Services are not getting created

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
Installed a KFserving from samples (sklearn), and when we check the inference service is giving IngressNotConfigured, upon further investigation we found that no Virtual services are created
What did you expect to happen:
Samples should work as per the documentation
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Following are the logs we get from kf-serving controller and similar logs are available in networking-istio
E0106 14:16:45.878293       1 reflector.go:125] knative.dev/pkg/controller/controller.go:456: Failed to list *v1alpha3.VirtualService: v1alpha3.VirtualServiceList.Items: []v1alpha3.VirtualService: v1alpha3.VirtualService.v1alpha3.VirtualService.Spec: unmarshalerDecoder: unknown field "allowOrigins" in v1alpha3.CorsPolicy, error found in #10 byte of ...|"v1"}}]}]}},{"apiVer|..., bigger context ...|.local","port":{"number":8080},"subset":"v1"}}]}]}},{"apiVersion":"networking.istio.io/v1alpha3","ki|... E0106 14:16:46.911249       1 reflector.go:125] knative.dev/pkg/controller/controller.go:456: Failed to list *v1alpha3.VirtualService: v1alpha3.VirtualServiceList.Items: []v1alpha3.VirtualService: v1alpha3.VirtualService.v1alpha3.VirtualService.Spec: unmarshalerDecoder: unknown field "allowOrigins" in v1alpha3.CorsPolicy, error found in #10 byte of ...|"v1"}}]}]}},{"apiVer|..., bigger context ...|.local","port":{"number":8080},"subset":"v1"}}]}]}},{"apiVersion":"networking.istio.io/v1alpha3","ki|... **Environment:**

Istio Version:1.3.1
Knative Version:
KFServing Version: 0.4
Kubeflow version: 1.1
Kfdef:[k8s_istio/istio_dex/gcp_basic_auth/gcp_iap/aws/aws_cognito/ibm] - k8s_istio/istio_dex
Minikube version:
Kubernetes version: (use kubectl version):
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.3", GitCommit:"2e7996e3e2712684bc73f0dec0200d64eec7fe40", GitTreeState:"clean", BuildDate:"2020-05-20T12:52:00Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"windows/amd64"} Server Version: version.Info{Major:"1", Minor:"16+", GitVersion:"v1.16.6-beta.0", GitCommit:"e7f962ba86f4ce7033828210ca3556393c377bcc", GitTreeState:"clean", BuildDate:"2020-01-15T08:18:29Z", GoVersion:"go1.13.5", Compiler:"gc", Platform:"linux/amd64"}
OS (e.g. from /etc/os-release):

We found the rootcause, one of the custom services deployed behind Istio had incorrect definition, however what is interesting to see was the entire cluster and all services were facing the issue.
Closing the issue now.
/close

We found the rootcause, one of the custom services deployed behind Istio had incorrect definition, however what is interesting to see was the entire cluster and all services were facing the issue.
Closing the issue now.
/close


Async methods in domain service

The method CountUsersWithLogin in IUsersCounter is synchronous, how to make it async?
I have introduced IsBrokenAsync to resolve this and it will introduce async methods in the domain model.
This seems to be a bad idea after reading this domain model purity article. Is this the reason behind the method CountUsersWithLogin created as synchronous?
Any thoughts on how to handle this?

Thanks @kgrzybek


changing DLC link

@coveminer Could you review and merge this PR contributed by the community? Thanks.

@coveminer Any updates on this? Thank you.


TypeError: tags.join is not a function

when I  upload the videos more than six hours, it reports error as following :
(node:3864213) UnhandledPromiseRejectionWarning: TypeError: tags.join is not a function
at /home/zch/node_modules/youtube-videos-uploader/dist/upload.js:175:57
at Generator.next ()
at fulfilled (/home/zch/node_modules/youtube-videos-uploader/dist/upload.js: 5:58)
at process._tickCallback (internal/process/next_tick.js:68:7)
(node:3864213) UnhandledPromiseRejectionWarning: Unhandled promise rejection. Th                        is error originated either by throwing inside of an async function without a cat                        ch block, or by rejecting a promise which was not handled with .catch(). (reject                        ion id: 1).It can only upload one vidow as draft,How can I fix this,Thanks for answering.

you need to pass the tags as array of strings
example
tags:["tag1","tag2","tag3"]


[Feature] Live markdown rendering using rich

Simple modifications that live renders response.

#76

Doesn't work on MacOS and Linux for some reason

Doesn't work on MacOS and Linux for some reason

What's the output?

Doesn't work on MacOS and Linux for some reason

What's the output?

Repeated output. It doesn't clear the previous ones

Doesn't work on MacOS and Linux for some reason

What's the output?

Repeated output. It doesn't clear the previous ones

It works well on my linux machine, whether ssh or locally. Maybe you are using an old terminal emulator that doesn't fully support the new features. For compatibility reasons though, we can add an argument for enabling/disabling the use of rich.

Possibly my font. Yes, please reopen this with support for disabling it

@PRO-2684 感谢你提供Markdown格式的代码，但发现有一个问题，搜索 一些问题时，打印更新到一半，忽然又自动全删了，提示无法回答此问题。

@PRO-2684 如果自动全删的问题解决不了，可以考虑加一个命令，可以切换是Markdown格式还是原始格式

Since this PR is merged and can't be reopened, I'll start a new PR.

@acheong08 #138
@wukan1986 #138 现在 bing 尝试撤回时会保留原内容


chore(deps): update dependency react-navigation-animated-switch to ^0.6.0

This PR contains the following updates:

Package
Type
Update
Change

react-navigation-animated-switch
dependencies
minor
^0.3.2 -> ^0.6.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: example/yarn.lock
error An unexpected error occurred<EMAIL_ADDRESS>connect ECONNREFUSED <IP_ADDRESS>:4873".


Added Path extractor and initial pipeline structure

I am pushing the first approach for the path extractor. There are many things missing and some need polishing but I'd rather merge it now to check if the structure is correct. Essentially I've:

Created structure to implement code2vec following sourced.ml style
Demo pipeline using ./submit_pyspark.sh

What is working:

for each UAST, given a max_width and max_length the desired paths are extracted

What is missing:

print the paths (to debug) as part of the pipeline not inside the 'get_paths' function
after creating all paths, gather all distinct paths/nodes and create the token2id and path2id dictionaries (I haven't figured out how the collected paths are returned inside the dataframes for the next step call).

@kafkasl instead of adding a commit each time you modify your PR, good practice is to amend your commit. If you want to use the command line:
git add my_file  # this will add my_file to the staging area, whether it is tracked or not
git add -u  # this will add all unstaged changes in tracked files to the staging area
git add -A  # this will add all untracked files and unstaged changes in tracked files to staging area
# once you have added all your changes:
git commit --amend  # this will amend your commit (you will have possibility to edit the message too)
git push --force

But most IDEs make this simple.
In your case you currently have 6 commits, so to squash them in one:
git rebase -i HEAD~6
# your prompt should be replaced with a text editor. Replace all "pick" preceding commmits with "f" except the first one
git push --force

I reviewed all the changes. A part from the recursion -> iterative thing, the rest shoud be OK. I've removed all unused code and put some functionalities inside a class (although I'd rather go with fp). Next is to build the token2id and path2id dictionaries but I'm not sure how to the paths are passed to the next .link() in the pipeline.

Also, initial unsigned commit is not great, could be something like Add bags of paths extractor and initial pipeline

I add a small fix because the code did not work, and now we are fine.


Emmet Support Requires Word Pattern

VS Code's Emmet support uses the word pattern settings for a language and when no word pattern is set it falls back to a alpha only word pattern, so numbers are picked up.
So if you have Emmet enabled for lang-cfml and use an emmet selector like .row>.col-md-5 the -5 will cause Emmet to fail. You can see details as to what's happening at https://github.com/Microsoft/vscode/issues/32438#issuecomment-322242789 and you can see how I fixed it for Twig at https://github.com/whatwedo/vscode-twig/pull/14/commits/f14f6f668600933ec49e3daa66da449c695dc5f4. I would have posted a Pull Request, but didn't necessarily want to take a different route by adding a .js setup file you might not want when supposedly you can add this word pattern within the cfscript.configuration.json as well.

It looks like this is only an issue with auto-completion suggestions. You can still press Ctrl+Space to force the suggestion or just run the Emmet: Expand Abbreviation command. This is again an issue with mixing languages. It would not be correct to change the CFML "wordPattern" because of this.
One way to address this would be to have *.cfm files default to the HTML language and delegate to the appropriate embedded languages. For example, it should mark CFML as the embedded language when within <cf tags or within ## within <cfoutput>. I'm just not sure how you would go about doing that.

That's right, you can manually expand the support Ramya mentioned in her comments, this is really all about detection based on how VSCode sees the language. Last I heard though there wasn't a way to just delegate to a sub-language which is why even Microsoft's language definition for HTML has duplication with the handlebars wordPattern syntax, but I could be wrong.

The VS Code PHP syntax too doesn't delegate to HTML either and they have set their wordPattern there as well. See: https://github.com/Microsoft/vscode/blob/master/extensions/php/src/phpMain.ts

I wasn't saying to not have a wordPattern, but to not have it change to accommodate Emmet autocompletion. The PHP one you linked also fails autocomplete on your example. It appears to correctly consider - as a word separator.
The issue with the PHP extension and HTML is well documented and a number of developers have chosen to not use VS Code because of it -- see Issue 670 and more generally Issue 1751.
That said, if you look at the their html extension (specifically the server part) they do have a homegrown solution that involves embedding languages.

Hmm, interesting. I see what you mean. But you're right, I don't think we should just be doing anything to accommodate just Emmet compatibility. And it's strange as you point out that PHP uses a different wordPattern than HTML which seems incredibly strange.

PHP and HTML are completely separate languages so it's not that strange that there would be syntactic differences. For example, in HTML5 there is an attribute aria-labelledby which would make sense to be considered one word. In PHP, you might have an expression $z = $x-$y; where - would not be considered part of a word so as to separate $x and $y.

I think you can simply add this to your user settings in VSCode:
"emmet.includeLanguages": {           "cfml": "html"     }


[google_maps_flutter] Expose an "onTilesLoaded" to notify when all tiles have been fully rendered.

Use case
The flutter plugin exposes an onMapCreated() function that states when the map is ready.
However, until all tiles are fully loaded it shows a grid with grey lines, and any effect or any interaction with the map seems useless until tiles are rendered.
These are the use cases I wish to accomplish:

Ensure all tiles are rendered before taking a snapshot.
Running the method takeSnapshot() after onMapCreated() leads to an image with either, partial tiles rendered or none at all.
Zoom-in to a position once the map is first loaded.
The zoom effect is kind of weird if done after onMapCreated() is fired, since it will zoom a grid of grey lines.

Proposal
Each platform already contains a functionality to determine when tiles are fully rendered:

Android - GoogleMap.OnMapLoadedCallback
iOS - GMSMapViewDelegate.mapViewDidFinishTileRendering
Web - Map.tilesloaded

However, AFAIK, there is no such functionality in the flutter plugin.

Hi @FranRiadigos, Thanks for filing the issue. Google maps plugin currently doesn't have such a feature to notify when the tiles are loaded that looks like a valid proposal.

Hi,
I recently got an issue on Crashlytics that might be related to my request.
Please, let me know whether I should open a bug issue about it.
When onMapCreated is fired, I call
controller.animateCamera(CameraUpdate.newLatLngBounds(bounds, padding));

Which produces the following error:
com.google.maps.api.android.lib6.common.apiexception.c: Error using newLatLngBounds(LatLngBounds, int): Map size can't be 0. Most likely, layout has not yet occured for the map view. Either wait until layout has occurred or use newLatLngBounds(LatLngBounds, int, int, int) which allows you to specify the map's dimensions. at com.google.maps.api.android.lib6.common.m.i(:com.google.android.gms.dynamite_mapsdynamite<EMAIL_ADDRESS>(100400-0):0) ...

The GoogleMap widget is rendered using hybrid composition.
AndroidGoogleMapsFlutter.useAndroidViewSurface = true;

Thanks.

@FranRiadigos, Please file that as a separate issue with all the requested details in the issue template and we will take a look at it.

This feature would be really appreaciated, im also working on something that would be useful to know whether or not the map tiles were loaded

Yeah, would be really nice! Any update on when this will be planned? Thanks!

Hey any updates on this issue ?

hey, any updates ?

That would be really nice and helpful. Any updates on this?


Live screenshare PoC

Live sceenshare
Overview
In Real Dev Squad, we want to be able to share our screen over a subdomain, in a controlled fashion. This will be beneficial to us, when we are live on Twitter spaces, or using Discord Stages with a larger audience.

User stories

As a presenter, I should be able to start a video-only screenshare of my desktop or specific window.
As a guest user, I should be able to view the screenshare.

Requirements

Only standard JS/HTML/CSS should be used. No frameworks, since this is for www site
Use the 100ms SDK
Plan out monitoring usage and alerting that will be integrated in future

Acceptance Criteria

[ ] As a guest user, I should be able to open up the public URL, and view the screenshare.
[ ] As the presenter, I should be able share my screen or window.
[ ] If the presenter is not currently sharing, show a default waiting message to the user.

The result of the POC
PS: I currently use my hosted server URL of 100ms SDK and room.
Hosted on codesandbox
CSB link: https://codesandbox.io/s/screenshare-poc-f2ry80?file=/src/index.js
Presenter: https://f2ry80.csb.app/presenter
Guest: https://f2ry80.csb.app/guest
Preview
https://user-images.githubusercontent.com/40385118/209965454-3bf87471-f91c-4dec-abd5-c5f1624973d9.mp4
Integrated locally and hosted on netlify
Branch name: https://github.com/Real-Dev-Squad/website-www/tree/343-live-screenshare-poc
Presenter: https://live-rds-poc.netlify.app/live.html?role=presenter
Guest: https://live-rds-poc.netlify.app/live.html?role=guest
Preview
https://user-images.githubusercontent.com/40385118/209966381-0f493929-69f1-4b1d-8d25-1b90f3e585d6.mp4

Design Link for live screen share -  https://www.figma.com/file/O8QLwPzTuH3RIWoYdxitln/Live-Page?node-id=0%3A1&t=O6Byfdtqg4ByTzpU-1

Requirement document:
https://precodes.notion.site/Requirement-document-for-live-screen-share-feature-e8c0c1eea04947beb44ddb60e609aaec
Use case diagram:
https://www.notion.so/precodes/Requirement-document-for-live-screen-share-feature-e8c0c1eea04947beb44ddb60e609aaec?pvs=4#4a4e033670aa42668240e65e35b30a2a
Sequence diagram:
https://www.notion.so/precodes/Requirement-document-for-live-screen-share-feature-e8c0c1eea04947beb44ddb60e609aaec?pvs=4#d8c7ac652b0946ed8aa5614da09f5a12


Parent style scope not applied to child component root if root comes from slot content

Vue version
3.3.4
Link to minimal reproduction
https://stackblitz.com/edit/vitejs-vite-85scvx?file=src%2FApp.vue,src%2Fcomponents%2FAppLink.vue
Steps to reproduce
Observe the styling of the two links. Only one is correct (has underline).
Use the DOM inspector to compare the two anchors, only one has data-v provided.
What is expected?
Roots of functional components that return only slot content (in this case the native <router-link> with the custom attribute applied) should receive the data-v attribute from the parent.
What is actually happening?
The data-v attribute is not attached to the root, and classes defined and attached in the parent component do not work.
System Info
No response
Any additional comments?
This is the part of <router-link> source code responsible for the difference
return () => {
      const children = slots.default && slots.default(link)
      return props.custom
        ? children
        : h(
            'a',
            {
              'aria-current': link.isExactActive
                ? props.ariaCurrentValue
                : null,
              href: link.href,
              // this would override user added attrs but Vue will still add
              // the listener, so we end up triggering both
              onClick: link.navigate,
              class: elClass.value,
            },
            children
          )
    }

I think this is because slot content can be multiple elements, so slot content from a compiled slot is always wrapped in a Fragment vnode - and the renderer doesn't/can't apply attributes to fragment nodes.
For now, I'm not sure wether this strictly needs to be this way for slots.

Thanks for the reply.
For us it would be nice to have styling for slots with single root elements, so we can style custom router links (we use custom router links in order to append the host name to in-app links).


Error While using this

@xlsdg  Hi, I'm getting this error:
webpack built 2e23409ed345309eab27 in 69ms
error during render : /public/f00cf08ae776e016bbbc.hot-update.json
ReferenceError: document is not defined
    at Object.<anonymous> (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/~/odometer/odometer.js:36:1)
    at Object.h (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/~/odometer/odometer.js:656:2)
    at e (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/bootstrap 2ce16c386878e8895619:19:1)
    at Object.<anonymous> (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/iOdometer.vue:9:1)
    at e (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/bootstrap 2ce16c386878e8895619:19:1)
    at Object.t.exports.render.t (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/src/iOdometer.vue:8:1)
    at e (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/bootstrap 2ce16c386878e8895619:19:1)
    at Object.defineProperty.value (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/src/index.js:1:1)
    at e (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/bootstrap 2ce16c386878e8895619:19:1)
    at Object.defineProperty.value (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/bootstrap 2ce16c386878e8895619:39:1)
    at /home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/bootstrap 2ce16c386878e8895619:39:1
    at n (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/universalModuleDefinition:3:1)
    at Object.<anonymous> (/home/daksh/Code/YTRCP/node_modules/vue-odometer/dist/js/webpack:/webpack/universalModuleDefinition:9:1)
    at Module._compile (module.js:624:30)
    at Module._extensions..js (module.js:635:10)
    at Object.require.extensions.(anonymous function) [as .js] (/home/daksh/Code/YTRCP/node_modules/babel-register/lib/node.js:152:7)
    at Module.load (module.js:545:32)
    at tryModuleLoad (module.js:508:12)
    at Function.Module._load (module.js:500:3)
    at Module.require (module.js:568:17)
    at require (internal/module.js:11:18)
    at r (/home/daksh/Code/YTRCP/node_modules/vue-server-renderer/build.js:7739:16)
    at Object.<anonymous> (webpack:/external "vue-odometer":1:0)
    at __webpack_require__ (webpack:/webpack/bootstrap 8f07a5410bd1a4343c46:25:0)
    at Object.<anonymous> (server-bundle.js:496:72)
    at __webpack_require__ (webpack:/webpack/bootstrap 8f07a5410bd1a4343c46:25:0)
    at Object.<anonymous> (server-bundle.js:425:63)
    at __webpack_require__ (webpack:/webpack/bootstrap 8f07a5410bd1a4343c46:25:0)
    at server-bundle.js:92:18
    at Object.<anonymous> (server-bundle.js:95:10)
    at evaluateModule (/home/daksh/Code/YTRCP/node_modules/vue-server-renderer/build.js:7747:21)
    at /home/daksh/Code/YTRCP/node_modules/vue-server-renderer/build.js:7805:18
    at new Promise (<anonymous>)
    at /home/daksh/Code/YTRCP/node_modules/vue-server-renderer/build.js:7797:14
    at Object.renderToString (/home/daksh/Code/YTRCP/node_modules/vue-server-renderer/build.js:7951:9)
    at render (/home/daksh/Code/YTRCP/main.js:89:12)
    at /home/daksh/Code/YTRCP/main.js:112:27
    at <anonymous>

Hi I have trid this library and got the same errors as you, I have built a complete and fine solution that you can find here: vue-odometer


Loading nib from main bundle fails

Great pod! But I encountered a small problem...
I tried to integrate your calendar view in another app (through cocoa pods) and encountered this exception:
*** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Could not load NIB in bundle: 'NSBundle </private/var/mobile/Containers/Bundle/Application/B235DCB6-6243-4A1D-A82A-EFAD5E27E930/myapp.app> (loaded)' with name 'GLCalendarDayCell''
This is because the .xib is loaded from the main bundle (bundle:nil) at GLCalendarView.m:81
[self.collectionView registerNib:[UINib nibWithNibName:@"GLCalendarDayCell" bundle:nil] forCellWithReuseIdentifier:CELL_REUSE_IDENTIFIER];
Changing this to this line instead, makes it work.
[self.collectionView registerNib:[UINib nibWithNibName:@"GLCalendarDayCell" bundle:[NSBundle bundleForClass:self.class]] forCellWithReuseIdentifier:CELL_REUSE_IDENTIFIER];

Would you be so kind as to update the pod with this fix?
Thanks a lot!

you can use v1.2.1, thanks.


bitwarden-desktop: fix build

See commit messages. While the first commit is mostly pointless given the second, I still think it’s valuable as it’s more likely we’d want to go back to that state if we end up needing dbus-run-session in the future, rather than the current master.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 348862 run on x86_64-linux 1

1 package built:

bitwarden-desktop


Add test cases for AdditiveArithmatic conformance

Here are some tests to check AdditiveArithmatic and SignedNumeric conformance. Please let me know if you need more/any specific edge cases.

Thanks for taking the time to add the tests!


[enhancement] Add missing bower.json.

Hey, maintainer(s) of danleech/simpleicons!
We at VersionEye are working hard to keep up the quality of the bower's registry.
We just finished our initial analysis of the quality of the Bower.io registry:
7530 - registered packages, 224 of them doesnt exists anymore;
We analysed 7306  existing packages and  1070 of them don't have bower.json on the master branch ( that's where a Bower client pulls a data ).
Sadly, your library danleech/simpleicons is one of them.
Can you spare 15 minutes to help us to make Bower better?
Just add a new file bower.json and  change attributes.
{
  "name": "danleech/simpleicons",
  "version": "1.0.0",
  "main": "path/to/main.css",
  "description": "please add it",
  "license": "Eclipse",
  "ignore": [
    ".jshintrc",
    "**/*.txt"
  ],
  "dependencies": {
    "<dependency_name>": "<semantic_version>",
    "<dependency_name>": "<Local_folder>",
    "<dependency_name>": "<package>"
  },
  "devDependencies": {
    "<test-framework-name>": "<version>"
  }
}

Read more about bower.json on the official spefication and nodejs semver library has great examples of proper versioning.
NB! Please validate your bower.json with jsonlint before commiting your updates.
Thank you!
Timo,
twitter: @versioneye
email<EMAIL_ADDRESS>VersionEye - no more legacy software!

Is this something people really want?


About this code：Object.prototype.toString.call(obj)

I have a question about this line of code
https://github.com/alex-cory/use-http/blob/7715b206a1790ad2d44e311e1e3fb13a5dfc02b4/src/useFetch.js#L4
Why not write it like this：
 const isObject = obj => Object.prototype.toString.call(obj) === '[object Object]'

I think you must have a special consideration, so Im very curious.
Could you tell me what your thought if you have time.
THX~

Actually that's just a typo LOL nice catch :)

Do you wanna do a PR? :) Otherwise I can do it

@alex-cory
😃 No problem~
I'll create a PR this afternoon（🇨🇳time zone）


Bug when installing

Hi, Im trying to install python-moshion in my 2.7 versions of Python in windows(sorry, i just dont encountered any documentation about gomoshion in windows, just saw in macOs),
then in prompt Ive just received this error message:

python setup.py install

Traceback (most recent call last):
File "setup.py", line 6, in
from setup_commands import TestCommand, CleanCommand
ImportError: No module named 'setup_commands'
I open up the setup.py but cant understand why this cant run.
can you help me?
cant found install documentation in .zip or web;

i've only been using this on mac - hoping to make it a bit more robust in a month or two


feat: add remote item id to search report responses

Description
Adds the remote item id to search REPORT responses for shared resources.
This is needed for the Web client to correctly resolve into a share from a search result. The Web client used the shareId in the past, however, with https://github.com/owncloud/web/issues/9784 we need to use the actual resource id (= remote item id) for this.
@dschmidt and I also thought about making the property more general by naming it spaceId and exposing it not only for shares but for all search results. But since we only need this for shares in this is the least invasive approach, we decided to go this way. Opinions welcome though.
Related Issue

Fixes https://github.com/owncloud/ocis/issues/9094

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Technical debt
[ ] Tests only (no source changes)

@micbar The Go vulnerability check fails now that I added https://github.com/owncloud/ocis/pull/9095/commits/f594db6067bca61f1f88e31f6e646d695b91e602. Could this be related? Other places in this webdav search service also don't use the protobuf getters 🤷

Thanks for your input!

So if a search result is a file inside for a shared folder, you need to know the resource ID of the shared folder. Right?

Yes.

If it's a file inside of a space that the user is able to access you need the resource id of the space root. Also right?

Yes. The Web client extracts the space id from the file id. I know that's not ideal, but that's how it currently works.

If that is true, should it really be called remoteItemId? I guess this is coming from the graph services remoteItem property in the sharedWithMe/sharedByMe responses?

Yes, the naming comes from the Graph terminology.
There is already a dav property called shareRoot, so we could also stay with that terminology and name the new property shareRootId, as this is the file id matching the share root path.

I added the ones where i know that we use them. Which ones are missing?

@micbar These are missing at a first glance:

oc:shareid
oc:shareroot
oc:activelock
oc:audio
oc:location
oc:public-link-item-type
oc:public-link-permission
oc:public-link-expiration
oc:public-link-share-datetime
oc:public-link-share-owner
oc:trashbin-original-filename
oc:trashbin-original-location
oc:trashbin-delete-datetime

How do we want to continue with this PR? I can make the commit message more descriptive and explain why and how to use this new property. Also, as stated above, I could rename it to shareRootId, to match with the naming convention of shareRoot. Though that might be confusing because it's different to the Graph naming terminology.

@micbar As discussed, I added docs for the missing supported webdav props via https://github.com/owncloud/ocis/pull/9095/commits/783692efdc0942f4192ace97f52c3c005b686eaa and added more detail to the original commit message.


[Alarm] Fix return type of AlarmInfo::type()

Type of type_ field is AlarmType but return type of
type() is int.

lgtm, thanks


Allow build script feedback to the crate compiled

This commit enables the build script for a crate to provide feedback to the
crate itself about how it should be built. This is done through the --cfg
flags of the compiler, and each build script is now allowed to print rustc-cfg
directives to inform Cargo about what --cfg flags it should pass.
All --cfg flags are local to the current crate and are not propagated outwards
to transitive dependencies. The primary use-case that this feature is targeting
is compile-time feature detection for applications like C bindings or C
libraries where the version being targeted may change over time.
Closes #1478

r? @wycats
cc @carllerche

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

:umbrella: The latest upstream changes (presumably #1508) made this pull request unmergeable. Please resolve the merge conflicts.

r? @brson

:pushpin: Commit 61a0ace has been approved by brson

@bors r+

:hourglass: Testing commit 61a0ace with merge f1d6a1a...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-32, cargo-win-64


Android: A Row with 2 children can calculate a size too large for the second child

Summary
On Android, when two children are nested inside a Row, and the childrens' combined content are greater than the Row's width, the second child's calculated width will exceed the parent Row's bounds. It looks like the second child's width will grow to match the Row's width, despite the first child taking up some of that space.
Works as expected on iOS.
Repro
Row {
  Spacer(30.dp)
  Text("This is a very long piece of text that will wrap the screen")
}

Screenshots

Attempted tweaks

Adding Modifier.shrink() or Modifier.flex() to the second child makes it work as expected.
Adding Modifier.grow() to the second child has no effect

I’m looking into this and I regret that I think this behavior is working as expected. I also think that’s surprising, so I’d like to explain how I got here...
The Row has two child components, and as far as it’s concerned they’re symmetric. Assuming the screen is 200.dp wide and vertically scrolling, the row makes two calls to its two children:

spacer.measure(maxWidth=200.dp, height=any), from which the spacer returns 30.dp
text.measure(maxWidth=200.dp, height=any), from which the text returns 200.dp

When I was investigating this, I was surprised at the argument into the 2nd call. Why didn’t the row offer a maxWidth=170.dp, as that’s what’s available?!
But to my surprise the layout doesn’t use a greedy algorithm, hand out space as it traverses through its children. Instead it measures each of them, and then decides how to reconcile the difference if the total doesn’t add up to 200.dp. If the total is lower than 200.dp, it’ll look at the grow modifiers and grow its children accordingly. If the total is greater than 200.dp it’ll look at the shrink modifiers and shrink its children accordingly.
Why offer the same pixels to both children?
This kind of makes sense to me, but if the row has 200.dp to allocate to its two children, why offer both of them the whole 200.dp? Why not start from an even split, like so?

spacer.measure(maxWidth=100.dp, height=any)
text.measure(maxWidth=100.dp, height=any)

I guess we can reject this because it’d break if the spacer was 101.dp.
Similarly, it won’t work in general it starts with 0.dp and lets its children decide everything. Consider:

spacer.measure(width=any, height=any) from which the spacer returns 30.dp
text.measure(width=any, height=any) from which the text returns the full width of the text without wrapping, which might be say, 1000.dp.

In that case the row would need to shrink 30.dp + 1000.dp into 200.dp.
How I sleep at night
Let’s imagine the spacer was on the right.
Row {
  Text("This is a very long piece of text that will wrap the screen")
  Spacer(30.dp)
}

Then the two calls would look like this:

text.measure(maxWidth=200.dp, height=any), from which the text returns 200.dp
spacer.measure(maxWidth=200.dp, height=any), from which the spacer returns 30.dp

This actually makes sense, because it can’t offer 170.dp to the text because it hasn’t measured the spacer is yet.
It’s much more natural to reconcile this case by configuring flex on the text. That’s basically saying, ‘don’t even bother measuring the text, it’ll just take what’s left over’. And that’s what it does.
How I Think About This
Each child of a row is measured independently. When it is measured it is told the target dimensions for the whole row.
The width of a row is the sum of the widths of its children.
These widths may overflow the width of the screen! But to prevent this, configure a child widget to be flexible.
Row {
  Spacer(30.dp)
  Text(
    modifier = Modifier.flex(1.0),
    text = "This is a very long piece of text that will wrap the screen"
  )
}

What About Columns?
Columns are the same. But often our columns scroll vertically, so the screen height is effectively infinite.


Wallet

If I find a valid wallet what to with it?

And how to use when downloaded?

Just run the .exe
Regarding the wallets. Everything will be explained while running the program


New version: LazyStack v0.0.9

Registering package: LazyStack
Repository: https://github.com/mcabbott/LazyStack.jl
Created by: @mcabbott
Version: v0.0.9
Commit: ca71cf1ae0e4295be267296ab5f58520264959c2
Reviewed by: @mcabbott
Reference: https://github.com/mcabbott/LazyStack.jl/commit/ca71cf1ae0e4295be267296ab5f58520264959c2#commitcomment-81947909
Description: 🥞

@mcabbott did you skip a version number on purpose?  What exactly do you want to do? (Please be clear, I can't read minds yet)
[noblock]

Oops, sorry. The intention was to increment, but we messed up & did it twice.
This can be closed, the robot has made https://github.com/JuliaRegistries/General/pull/66899 with the right number.


Alternative Worlds Prompts for Various Tasks and Benchmarks

Thanks. I'll make a PR specifically to add Brier Score.

@RicardoDominguez you can use brier score now. #969


Added local ollama models

Fixes #2432
Same motto as #1513

OLLAMA_URL

That's static. And if it is running in another system, would docker be able to access it?

That's static.

Well, it would be configurable by the user. So not really static.

And if it is running in another system, would docker be able to access it?

I think docker would be able to access it because egress from docker is allowed, but I may be wrong.

How about setting a timeout=2 in the request?

I'm not sure how this relates to my comment, could you elaborate?

There is something strange though, if I see this right. We have base_url in config, which afaik is necessary for local LLMs, for the completion call (and fallback value for the embeddings call).
If we're adding ollama_base_url to config for the models call, will ollama users need to set both, and set them both to the same value?
base_url=ollama_server
ollama_base_url=ollama_server

If they're the same, it seems confusing to need both. An alternative could be to add a config var like is_ollama_server, just to know that it's ollama and we can query it for more info, like models. Or simply name it differently, like ollama_models_url for the specific url if provided. Or maybe... make a special case for ollama for the other API calls too, allowing base_url to not be set, and if the user set ollama_base_url then use it as base_url...

is_ollama_server

That option sounds somewhat cleaner to me.

@neubig wants to show OLLAMA models even when someone uses other models for a quicker switch.

Sorry, just to clarify: you don't need base_url for gpt-4, so you can select it anyway even when you have base_url set, right? In that case, base_url would be pointing to an ollama server, so you could still get the names of the models on the ollama server without removing the option to also select gpt-4.
So basically, I agree with the is_ollama_server setting, or it could be even better if we could check if it's an ollama server without the users having to specify it.

Sorry, just to clarify: you don't need base_url for gpt-4, so you can select it anyway even when you have base_url set, right?
Yes.

So basically, I agree with the is_ollama_server setting, or it could be even better if we could check if it's an ollama server without the users having to specify it.
from the model name, we could know.

@neubig wants to show OLLAMA models even when someone uses other models for a quicker switch.

doesn't it?

just to clarify: you don't need base_url for gpt-4, so you can select it anyway even when you have base_url set, right?

Ah, sorry, I don't think that will work. I believe liteLLM uses base_url if set, it assumes it's an openAI proxy.

I believe

Yes, it does that.

@enyst and @neubig are we waiting for more changes here?

@neubig wants to show OLLAMA models even when someone uses other models for a quicker switch.
doesn't you?

Yeah, that's more-or-less what I was thinking.
But one thing that's a bit confusing here is that if LLM_BASE_URL is set to be an ollama server it might actually make it not possible to use other models like gpt-4o, claude, etc. But I think this is OK for the time being.
A better/more permanent solution might be to use the fact that we are able to configure models separately in config.toml and somehow make it also possible to configure model servers that contain lists of usable models (like ollama).


repair concurrency related ndf in persist package

couldn't we manually trigger the panic by adding an entry to activeFiles in the test? I would prefer that to inserting a runtime.Gosched call into SaveJSON

hmm. That treads into the realm of implementation-specific testing though

closing because the relevant test was removed


Can the icon property of a control such as cardcontrol support the use of fonticon?

Is your feature request related to a problem? Please describe.
I want to customize the icon of the card
Can the icon property of a control such as cardcontrol support the use of fonticon?
Describe the solution you'd like
use the fonticon

You can use Icon="Empty" and manually add a FontIcon to the CardAction.Header or CardControl.Header


No Initial Scrolling

Description
The page scrolls initially when user visits it for the first time, and a back-to-top button is also there.
The page should show random images in the beginning.

Could you elaborate the issue... I couldn't get it

When we visit the site then there is scrolling effect even if there are no images.
The UI looks better it we put random images in this.

https://github.com/Vijaykv5/Image-Searcher/assets/97498433/ba396e94-dc5c-4738-a853-44a4f29e3ab0

Ok I guess we can have it like no Scrolling at first rather than adding a random image @jaypatel523
When initially user goes into the website there is not scrolling effect.

Let me know if @jaypatel523 wants to work on it

I want to work on it. Also Initially if there are some random images then it looks better than empty page.

I want to work on it. Also Initially if there are some random images then it looks better than empty page.

No at first user visits the webpage it should be clean with only header , searchbar and tags.. No need for any other image

Fine. Please let me fix this bug.

I fixed the bug. Can i make pull request now ?

Sure Make sure to an demo also attached with your PR

@jaypatel523 Make sure add a git pull before you add PR

I have make the pull request. By the way, I'm complete beginner in open source contribution so forgive me for my mistakes.

I merged it @jaypatel523 Thank you :)


address minor build warning

Describe your changes
Address this specific warning

Issue or discord link

Testing/validation

Checklist before requesting a review (leave this PR as draft if any part of this list is not done.)

[ ] I have commented my code in hard-to understand areas.
[ ] I have made corresponding changes to README or wiki.
[ ] For front-end changes, I have updated the corresponding English translations.
[ ] I have run yarn run mini-ci locally to validate format and lint.
[ ] If I have added a new library or app, I have updated the deployment scripts to ignore changes as needed

There is also one on the settings page in databasecard

Is there? hasn't popped up for me. I can address in another PR.

I lied, it's on the home page


Update Spring Boot to the latest version

Update the guide to use the most recent Spring Boot version.
Files that require changes are:
initial/build.gradle
initial/pom.xml
complete/build.gradle
complete/pom.xml

Hello, which version do you need to upgrade to? I will be happy to do it.

Hi @zhangwt-cn .  Thank you for the interest.
You can find the latest stable Spring Boot version by finding the CURRENT tag on this page.  Additionally, you could go to start.spring.io and find the latest version that is not a SNAPSHOT or a milestone (M).
As of this moment the latest version of Spring Boot is 3.1.4.

Hi @robertmcnees  I have submitted the modification, please feel free to correct me if there are any questions. #49


Smarter resource selection

Checking in with @yoavweiss on this one; I don’t want to deviate from native anywhere in here.

I'd prefer to leave such "preference" features to extensions. Browsers are not throttling on 2x, and are not planning to AFAIK.

by extensions I mean a "picturefill plugin", not a browser extension

@yoavweiss
There is already one.
But I thought you would make somewhere the cut. Your comment sounded so.
But I will remove this asap

Thanks! :) I'm certainly interested in hearing about experimentations like that, but have no immediate plans to bring that into the native implementation. Sorry if my comment wasn't clear enough in that regard.

Well, I have no 3x device, but I think this is something, that should be tested with a variety of images (perceived perf vs. perceived quality), but I guess the quality improvements aren't that good satisfying so much data transfer.

Reviewed; +1.

@aFarkas Looks like the original branch for this has been deleted. We still looking to get this in or can we close this one out?

I think this is dead. Would still have feedback for the meta issue here: https://github.com/scottjehl/picturefill/issues/497


App doesn't start in airplane mode

The Issue
Steps to reproduce

close lbry
turn on airplane mode
start lbry

Expected behaviour
The app should start up and for example allow me to watch videos I've downloaded to my library
Actual behaviour
I get a screen saying
Oops! Something went wrong
App startup failed. Please check your data connection and try again.

✓ Loaded installation ID.
✓ Loaded local known and followed tags.
❌ Loaded LBC/USD exchange rate.
❌ User authenticated.
❌ Installation registered.
❌ Loaded subscriptions.
❌ Resolved subscriptions.

I know I can find the videos in the downloads folder and watch them from there, so there is a workaround. But I think it would be nice if I could watch my downloaded videos in LBRY even if I'm offline.
System Configuration

Device: Pixel 2 XL
Android version: 10

Anything Else

App version: 0.15.14
LBRY SDK: 0.76.0
Screenshots

Internal Use
Acceptance Criteria

Definition of Done

[ ]  Tested against acceptance criteria
[ ] Tested against the assumptions of the user story
[ ] The project builds without errors
[ ] Unit tests are written and passing
[ ] Tests on devices/browsers listed in the issue have passed
[ ] QA performed & issues resolved
[ ] Refactoring completed
[ ] Any configuration or build changes documented
[ ] Documentation updated
[ ] Peer Code Review performed

Hmm, it doesn't appear to happen every time, but it certainly happens some times.

Thanks for the report. Currently the background process and authentication needs internet to be able to work properly. It will take some time to support an offline scenario.


Speed up iteration over Variables objects

New timing results for for v in variables: pass
                labels
              --------- ------------
                string   34.3±0.7μs
                index    11.2±0.3μs
               integer   34.1±0.6μs

vs the old timing results
                labels
              --------- ------------
                string   63.0±0.3μs
                index     53.1±1μs
               integer    62.2±3μs

Codecov Report

Merging #1180 (73ccb0e) into main (8fc4005) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1180   +/-   ##
=======================================
  Coverage   94.83%   94.83%
=======================================
  Files          82       82
  Lines        8089     8089
=======================================
  Hits         7671     7671
  Misses        418      418

Continue to review full report at Codecov.

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

In this case the speed comes from implementing .__iter__() "by hand" rather than using the one inherited from collections.abc.Sequence which looks like
    def __iter__(self):
        i = 0
        try:
            while True:
                v = self[i]
                yield v
                i += 1
        except IndexError:
            return

(source code).
By doing it "by hand" we can improve the performance in the range case, and even in the case with labels we can bypass a bunch of function calls.


AP-3485: Add specific issue merits question

What
Link to story
Implement the Specific Issue, section 8 question.
Add a DB object to store the responses
Add a new form for validating the answer
Add the route, controller and view
Update the merits task flow and routing
Updated the merits partial to show the response on the CYA, review and merits_report pages
Checklist
Before you ask people to review this PR:

Tests and rubocop should be passing: bundle exec rake
Github should not be reporting conflicts; you should have recently run git rebase main.
There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
The PR description should say what you changed and why, with a link to the JIRA story.
You should have looked at the diff against main and ensured that nothing unexpected is included in your changes.
You should have checked that the commit messages say why the change was made.

Rebased after rubocop changes


Add v1.0.0 test cases

Features

[x] [FEATURE] Ability to use HwAddr to lookup NetworkInterface Name in harvester-installer. #1641 owner:lanfon72 area/installer, enhancement
[x] [FEATURE] maximum CPU in VM creation #1565 owner:lanfon72 area/ui, enhancement
[x] [FEATURE] Support management of node labels  for advanced scheduling of VMs #1416 owner:lanfon72 p2
[x] [FEATURE] Move download kubeconfig icon #1349 owner:lanfon72 p2
[x] [FEATURE] Configuring additional trusted CA certificates #1260 owner:lanfon72 p1
[x] [FEATURE] Doc: Install OS from a USB disk #1200 owner:lanfon72 p1
[x] [FEATURE] backup VM required metadata to the external storage server #988 owner:lanfon72 p2
[x] [FEATURE] Add auto SSL certificate using Let's Encrypt or ability to bring Custom certificate to secure Harvester #761 owner:lanfon72 p1
[x] [FEATURE] Allow customization of TLS parameters #1046 owner:lanfon72, noahgildersleeve p2

Bugs

[x] [BUG] Harvester network controller manager failed to update lock #1722 owner:lanfon72 area/network, blocker, bug
[x] [BUG] When creating a new template from an existing template, the  volume size should be editable #1711 owner:lanfon72 area/ui, bug
[x] [BUG] Fail to use /dev/by-id/xxx link as the install disk in PXE boot installation #1462 owner:lanfon72 area/installer, bug
[x] [BUG] Harvester 1.0.0-rc1 fails to install on NVMe M.2 SSD #1627 owner:lanfon72 p1
[x] [BUG] NVMe drive can be added multiple times in host config in UI #1608 owner:lanfon72 p1
[x] [BUG] VM status doesn't update when host goes down #982 owner:lanfon72 p1
[x] [BUG] Harvester 0.3 does not allow CPU-overprovisioning #1429 owner:lanfon72 p1
[x] [BUG] Memory is being saved as the wrong amount in VMs #1537 owner:lanfon72 p1
[x] [BUG] Error message about invalid backup target doesn't go away #1051 owner:lanfon72 p1
[x] [BUG] Resource quota management not completely implemented in UI #1450 owner:lanfon72 p1
[x] [BUG] Agent node becomes NotReady when one of the controller node fails #1521 owner:lanfon72 p1
[x] [BUG] Nodes in maintenance mode are listed in Node Scheduling selection #1350 owner:lanfon72 p2
[x] [BUG] Backup-target failing for the harvester-ci-testing bucket #1339 owner:lanfon72 p2
[x] [BUG] Support Bundle failed to create due to timeout #1585 owner:lanfon72 p2
[x] [BUG] The password field in the Harvester continue login screen does not offer show/hide toggle #1550 owner:lanfon72 p2
[x] [BUG]  Unable to migrate restore-new vm twice #1086 owner:lanfon72 p2
[x] [BUG] VM scheduling failure not reflected in VM status #1446 owner:lanfon72 p3

Test Case Skipped

[x] [BUG] Node's resource information mismatch to details view #1612 owner:lanfon72 area/ui, bug, regression
[x] [BUG] Harvester v0.3.0 unable to get local issuer certificate with https squashfs url #1431 owner:lanfon72 bug
[x] [BUG] After rebooting all Harvester nodes concurrently some VMs can't be restarted #1263 owner:lanfon72 p1
[x] [BUG] Host overview doesn't show actual available storage from longhorn #1253 owner:lanfon72 p1
[x] [BUG] Installation failed: "Could find partition device path for partition 6" #1583 owner:lanfon72 p1
[x] sysctl in Harvester config does not work #1405 owner:lanfon72 p2
[x] [BUG] VM SSH Key's modifying didn't reflect to user data #1386 owner:lanfon72 p2
[x] [BUG] VM's resource restriction is not match to hosts state #1376 owner:lanfon72 p2
[x] [BUG] VM's function won't work when host is down #1360 owner:lanfon72 p2
[x] [BUG] Lack Edit as YAML button for resource creation #1304 owner:lanfon72 p2
[x] [BUG] Cursor jumps to the end while editing image name #1269 owner:lanfon72 p2
[x] [BUG] The config will keep changing,  so the frontend will keep refreshing the list #1453 owner:lanfon72 p2
[x] [BUG] VM always in starting state when 3 VMs is running #1387 owner:lanfon72, noahgildersleeve p2

Preview of the incoming page changes:


(FACT-2306) Added option to use AWS IMDSv2

Added the option to use AWS IMDSv2.
To use IMDSv2, set the AWS_IMDSv2 environment variable to true.

Just ran into this too!


Crashes on v2.06

Apparently the game crashes sometimes in the latest version. If you experienced this: Any information on what settings you used is appreciated.

Should be fixed in v2.06a


Failed to download mahjong soul log

Online page failed to read any mahjong soul url. Local build says cause by: get mahjong soul log: 503 Service Unavailable. The server is fine for normal players, could be the recent event update?

Apparently, it seems the IP of the server used to download mjsoul logs (at tensoul.herokuapp.com) is banned by mjsoul (login error code 151). I can still download mjsoul logs locally using the same program so it should be the case.

So mjsoul started noticing the akochan public service? How to download the logs locally with the build from resource?

It is a bit complex. Check this https://github.com/Equim-chan/tensoul

I wish I could help, but I'm not good at js and rust. Sadly Akochan couldn't review mjsoul now. Would be appreciated if you plan a fix on this.

Sorry, I have no clue fixing this problem. Retrieving mjsoul logs require a login, so I deployed tensoul on heroku as a helper to login to mjsoul, retrieve the logs, and convert to tenhou's format so that the reviewer can work on it. Currently the IP of heroku is banned from logging in mjsoul, and several months ago I already have migrated it once (us -> eu). Right now both IPs are banned and there is nowhere to migrate again.

I am thinking running locally and use my own credential to log in mjsoul, but I don't know how to deploy tensoul on local and ask the reviewer to use it. Is it doable?

In fact, you don't have to deploy it. You could download the log with command line.
$ node . MAHJONG_SOUL_LOG_ID > tenhou_log.json
$ akochan-reviewer -i tenhou_log.json -a ACTOR -e

Awesome, I am able to download the mjsoul log locally and have akochan review it. Thanks for the help!

See https://github.com/Equim-chan/akochan-reviewer/blob/master/mjsoul.adoc.

9/28 mjsoul update probably failed the tensoul. I switched and your new method works well. Thank you.


napari-trait2d plugin not found in hub

Description
"napari-trait2d" not showing in napari hub after deployment
Steps/Code to Reproduce
I deployed napari-trait2d on pypi (project generated with cookiecutter-napari)
Expected Results
I expected to see the details for the "napari-trait2d" plugin
Actual Results

https://www.napari-hub.org/plugins/napari-trait2d
Plugin is now available, closing.


Redis service not getting connected from sample application at port 5000

Hi,
I have a running Bosh-lite cloud-foundry-instance.To test services,I have written a sample python application to connect to redis service, which has been created as said in the link. https://github.com/cloudfoundry-community/cf-services-contrib-release
my sample application(testredis) looks like
Get Redis credentials
if 'VCAP_SERVICES' in os.environ:
services = json.loads(os.environ['VCAP_SERVICES'])['redis-2.6'][0]
redis_env = services['credentials']
print redis_env
else:
redis_env = dict(hostname='localhost', port=6379, password='')
myredis_host = redis_env['hostname']
myredis_port = int(redis_env['port'])
myredis_pwd  = redis_env['password']
myredis_env  = dict(host=myredis_host, port=myredis_port, password=myredis_pwd)
print myredis_env
Connect to redis
try:
r = redis.Redis(**myredis_env)
r.info()
except redis.ConnectionError as erdesc:
r = None
print "Not connected to Redis... " , erdesc
@app.route('/')
def keys():
if r:
current_hits = r.incr('hits')
return 'Hits: {}\n'.format(current_hits) + 'Available Keys: ' + str(r.keys())
else:
return 'No Redis connection available!'
@app.route('/')
def get_current_values(key):
if r:
current_values = r.lrange(key, 0, -1)
return str(current_values)
else:
abort(503)
@app.route('//')
def add_value(key, s):
if r:
r.rpush(key, s)
return 'Added {} to {}.'.format(s, key)
else:
abort(503)
if name == 'main':
# Run the app, listening on all IPs with our chosen port number
app.run(host='<IP_ADDRESS>', port=port)
The application manifest contains the binding details of redis instance like
applications:

name: testredis
memory: 128MB
disk_quota: 256MB
random-route: true
command: python main.py
services:

myredis

The application has binded successfully with redis and got the details as follows

cf services
Getting services in org vedams / space cfspace as admin...
OK

name      service      plan      bound apps              last operation
mymongo   mongodb      default                           create succeeded
mypgsql   postgresql   default                           create succeeded
myredis   redis        default   testredis2, testredis   create succeeded

cf env testredis
Getting env variables for app testredis2 in org vedams / space cfspace as admin...
OK

System-Provided:
{
"VCAP_SERVICES": {
"redis-2.6": [
{
"credentials": {
"host": "<IP_ADDRESS>",
"hostname": "<IP_ADDRESS>",
"name": "7715b1bc-4d0c-46ae-9159-9016a0dd6756",
"password": "f6aa70e6-2b7b-42bd-8e54-0f7629fbfb89",
"port": 5000
},
"label": "redis-2.6",
"name": "myredis",
"plan": "default",
"tags": [
"key-value",
"nosql"
]
}
]
}
}
{
"VCAP_APPLICATION": {
"application_id": "413e5e12-1fdc-4ecb-b86b-a33d61ca9115",
"application_name": "testredis2",
"application_uris": [
"testredis2-unpenetrant-urease.<IP_ADDRESS>.xip.io"
],
"application_version": "d297bf90-bd11-4e57-bbb5-e9be76778d61",
"limits": {
"disk": 256,
"fds": 16384,
"mem": 128
},
"name": "testredis2",
"space_id": "8e39e3e3-12b7-48de-ae89-5d7af39fd77b",
"space_name": "cfspace",
"uris": [
"testredis2-unpenetrant-urease.<IP_ADDRESS>.xip.io"
],
"users": null,
"version": "d297bf90-bd11-4e57-bbb5-e9be76778d61"
}
}
No user-defined env variables have been set
No running env variables have been set
No staging env variables have been set
The application is running successfully but not connected to redis instance from DEA.
Getting error in app logs like connection refused to redis.
What could be the problem?

You have to make sure access to the services network is allowed by the cf security groups. In the integrations specs this is done here. @Gangabhavani if you get it to work could you adapt the documentation from here to the cf-services-contrib repo and make a PR for it?


Added link to PostgreSQL storage engine

Hi! I made a storage engine for postgres, thought maybe you'd like to link to it.

Thanks @mang :pray:


Adding 'Community forks and extensions' to the README

This adds a section for community forks in the README, addressing #504, and sets us up for reducing feature branch maintainence load.

@all - Please add yours to the list in the README once this is merged.

Awesome pr @oyeanuj, great idea!

Got a new fork at https://github.com/fivethreeo/react-universally/tree/redux-sass-opinionated


Add a note to AWS page

Add a note to the AWS page at line 22 with this content:

If you are using SendGrid through AWS, you cannot change your SendGrid username.

Link to page:

I can work on this

It's this page, correct? content/docs/for-developers/partners/amazon-marketplace.md

https://github.com/sendgrid/docs/pull/4242


Do we have Application level interface in Insight?

hi,
how can one derive application/product quality metrics where an application has 10 services  & each services has multiple components, is there are way we can aggregate the quality metrics related to build, deployment, code quality etc across different sprints ?
Regards,
Manoj.

Yes we do have a concept of Data tagging which helps to tag certain specific type of nodes for application so that we can application type of metrics. This section is undergoing change and it is likely to be available in the next couple of weeks. Please watch for our release update. In order to track this enhancement - we will keep this issue open in GitHub as well.

The interface is ready and there are changes to the agent to do the data tagging during the data creation. The application level hierarchy can be created in the UI by uploading a excel sheet which contains the application level details. Testing is in progress - but you can take a look at the changes which are being made here. Once it is complete it will be merged to the master branch.
https://github.com/CognizantOneDevOps/Insights/commits/DataTagging


Development builds are unable to access the "Vienna sync" keychain element

I noticed that OpenReader syncing does not work anymore when I launch Vienna through Xcode.
Debugging revealed that the app fails to access the server's password through the Keychain: the call to SecKeychainFindGenericPassword returns a -25293 error code (errKCAuthFailed)
The root cause is that there is a problem with the ad-hoc code signature that is applied by Xcode. If I check the built app with codesign -vvvv, I get "invalid Info.plist (plist or signature have been modified)", so macOS allows the app to run, but limits its security credentials.
I can punctually work around the problem by applying codesign -f -s - to the app, but I was unable to automate that: I tried adding a script for forcing the signature, but Xcode will perform frameworks copying and application signing after the script's execution.
The problem is clearly caused by the switch to Sparkle 2 (#1485)
No problem with Deployment builds, thanks to  the procedures described in Release Instructions.md

Have you tried adding that routine to a post-build action under Vienna’s scheme?

I prefer my issue-1492 branch

Closed by #1494


install local/* not found, while search shows it exists

$ helm search web
NAME                	VERSION	DESCRIPTION
local/web           	0.1.0  	Run your application or service on Kubernetes, ...
stable/chronograf   	0.2.0  	Open-source web application written in Go and R...
...

$ helm install local/web --dry-run
Error: file "local/web" not found

According to helm install --help this should work, but it doesn't.

Looks very familiar to https://github.com/kubernetes/helm/issues/2315.

Are you running helm serve? If not, it can't find a server on localhost.

thank you @technosophos - was having the same issue and running helm serve solved it

closing as solved/inactive. Sounds like no local server was running, but the cached index files were still present in ~/.helm/repository/local/index.yaml. If you removed that file, I bet you it would not show up in helm search any more.

Thanks @bacongobbler. I can't remember I ever ran helm serve before, but I guess I must have 👍

For those who stumbled onto a similar issue but in a different context, you will also get the same error if you specify incorrect version using the "--version" option.


Any logged-in user can access each other's statistics.

As long as the user knows the website id, he can access the statistics page, even if he is a normal user accessing the administrator's website page.

I can probably raise a fix for this over the weekend ✨

Beat you to it @hugomd 😄 Fixed in #167

So fast @mikecao ! ⚡


[TI] Update docker images for Sysconfig 1.18.1 update

PR 1 of 2 - dockerfile and version file update for sysconfig

@andy31415 This PR should only have the dockerfile and version changes needed to create a new image


[IMPALA-12291] - impala checks hdfs ranger policy

Related Issue

https://issues.apache.org/jira/projects/IMPALA/issues/IMPALA-12291

Description

When HDFS is ranger enabled, current impala fe doesn't check ranger policy but hadoop inode permission
New feature adds getAvailableAccessLevel() method ranger check code using Hadoop API
Add one more catch in getTExecRequest() method to check AnalysisException

Test

Manual Test done

Distribute impala fe jar for impalad and catalogd
Enable HDFS Ranger Plugin.
Give impala Allow condition for ranger hdfs policy and check insert query - Insert Success
Give impala Deny condition for ranger hdfs policy and check insert query -  Insert Fail

Hi @HalimKim, thanks for your contribution! We don't use pull requests in Apache Impala, we use Gerrit instead: https://gerrit.cloudera.org/. https://cwiki.apache.org/confluence/display/IMPALA/Using+Gerrit+to+submit+and+review+patches has a walkthrough for uploading patches for review.
Thanks, Michael.

@MikaelSmith
Thanks, I just have uploaded the patch on gerrit.
Walkthrough was really helpful.


vue-tsc: Generic Components with await in script setup

I've seen there are other issues with generic components which I've also faced but haven't seen this one mentioned.
Having a generic doesn't allow to use top-level await - I would assume the problem is most likely more general than specifically top-level await, something with tsconfig maybe?
Error: 'await' expressions are only allowed within async functions and at the top levels of modules.

No error

- Operating System: Windows_NT
- Node Version:     v20.1.0
- Nuxt Version:     3.6.5
- Nitro Version:    2.5.2
- Package Manager<EMAIL_ADDRESS>- Builder:          vite
- User Config:      modules, build, tailwindcss, googleFonts, devtools, experimental, alias
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -

"devDependencies": {
      "@nuxt/devtools": "^0.7.0",
      "@nuxt/eslint-config": "^0.1.1",
      "@nuxtjs/eslint-module": "^4.1.0",
      "@nuxtjs/google-fonts": "^3.0.2",
      "@types/node": "^20.4.4",
      "@typescript-eslint/eslint-plugin": "^6.1.0",
      "@typescript-eslint/parser": "^6.1.0",
      "eslint": "^8.45.0",
      "eslint-plugin-nuxt": "^4.0.0",
      "eslint-plugin-vue": "^9.15.1",
      "nuxt-quasar-ui": "^2.0.4",
      "vue-tsc": "^1.8.6"
   },
   "dependencies": {
      "@nuxtjs/tailwindcss": "^6.8.0",
      "@quasar/extras": "^1.16.5",
      "@trpc/client": "^10.35.0",
      "@trpc/server": "^10.35.0",
      "autoprefixer": "^10.4.14",
      "nuxt": "^3.6.5",
      "postcss": "^8.4.27",
      "quasar": "^2.12.3",
      "tailwindcss": "^3.3.3",
      "trpc-nuxt": "^0.10.6",
      "typescript": "^5.1.6",
      "zod": "^3.21.4"
   }

@so1ve this fix works for VSCode, but not for WebStorm. I'm on 2023.2.1 and still get the

Vue:  await  expressions are only allowed within async functions and at the top levels of modules.

error.

I installed the latest @vue/language-server (1.8.8) globally and used this instead of the bundled 1.8.2 version in WebStorm (see screenshot).
Other maybe relevant data:
- Operating System: Darwin (macOS 13.5.1 (22G90))
- Node Version:     v20.5.1
- Nuxt Version:     3.6.5
- Nitro Version:    2.5.2
- Package Manager<EMAIL_ADDRESS>- Builder:          vite
- User Config:      devtools, modules, runtimeConfig, supabase, typescript, ui
- Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>- Build Modules:    -

  "devDependencies": {
    "@iconify-json/mdi": "^1.1.54",
    "@nuxt/devtools": "latest",
    "@nuxthq/ui": "^2.7.0",
    "@nuxtjs/supabase": "^1.0.2",
    "@types/node": "^18.17.6",
    "@vueuse/core": "^10.3.0",
    "@vueuse/nuxt": "^10.3.0",
    "better-supabase-types": "^2.6.1",
    "nuxt": "^3.6.5",
    "supabase": "^1.86.2",
    "typescript": "^5.1.6",
    "vue-tsc": "^1.8.8"
  }

@jovobe Maybe you should report to jetbrains. They have to update volar to include this fix.


AppKitNavigation - Common

This is some basic components, with UIKit is basically the same, I think the back can be AppKit animation and UIKit animation together, AppKit animation generally use a certain property of the animator trigger, for example, view.animator().alphaValue = 1

Thanks! No rush!
Oh, and your comment here:

I think the back can be AppKit animation and UIKit animation together, AppKit animation generally use a certain property of the animator trigger, for example, view.animator().alphaValue = 1

I think property animators may not support the new spring animations UIKit added support for in the most recent UIView.animate function. I could be wrong, though! I didn't dig too deeply into the problem.

@stephencelis The code is ready. You can look at it again.

I think property animators may not support the new spring animations UIKit added support for in the most recent UIView.animate function. I could be wrong, though! I didn't dig too deeply into the problem.

You're right, AppKit's animations aren't as rich as UIKit's, and Apple hasn't added more APIs, so let's keep it that way

It's done.

@stephencelis Hey bro,still here?

@Mx-Iris that's one way of saying "Just a friendly ping"

@Mx-Iris Your experiment has made us reconsider some of the internals, so we've been hard at work making observe and UITransaction more extensible. See this PR for more details: https://github.com/pointfreeco/swift-navigation/pull/212
If you give @mbrandonw and me write access to your fork we can try incorporating the changes once we merge. We're trying to be mindful and careful as we incorporate a new platform like AppKit into the library, so it's going to take us some time to work out some of these new questions and details that come up.
Thanks for your patience, though! We're excited for what you've accomplished so far!

@stephencelis Oh, that's great, I've sent you and @mbrandonw  an invite.

@Mx-Iris that's one way of saying "Just a friendly ping"

☺ Yes, that's what it means

@Mx-Iris Pushed some changes and will discuss with @mbrandonw tomorrow or Monday. Let me know if things look good to you or if I messed anything up 😅

@stephencelis It's okay. No problem.


BUG:Want to Fix Alignment of Eye Icon + Want to Signup Not Saying at same place on different Screen #772

Fixed #772
Before::

After::

hey @mdazfar2 please review this pr

@nishant0708 did you update sync fork?

@nishant0708 In mobile view responsive it looks like weird as compare to the current once please resolve this

hey @mdazfar2 it is fine now please check and let me know changes

@nishant0708 You might not have updated your sync fork. Please update it.

hey @mdazfar2 check it now


Add --use_docker flag support for run_tests.py

Adds --use_docker flag to run_tests.py. If used, all the tests will run under docker.
This is great because it allows running all the unit tests for different languages without the need to install any prerequisites (aside from python and docker). It also allows to propagate all the existing cmdline args provided to the host run_tests.py to a child run_tests.py running inside docker.
As a side effect, run_jenkins.sh can just invoke run_tests.py with  --use_docker flag and it becomes much more transparent.

@nicolasnoble  FYI

Not sure yet if jenkins will be happy with the docker run -t  flag. It's pretty useful when running locally. Let's see.

After fixing some issues, Jenkins seems to be pretty happy about the changes and dockerized tests are running there without any trouble.

ping? I can reassign if needed.


Service Fabric deployments are failing using ServiceFabricDeploy@1 task

Description
We're from 1ES Hosted Pools team in Microsoft. We have several internal customers impacted due to Service Fabric Tasks failing in ADO with the following error. This is happening on Image 20230910.1.0 Windows 2022.
Task fails with
System.Fabric.ImageStore.IImageStore System.Fabric.Management.ImageStore.ImageStoreFactoryProxy.CreateImageStore(System.String, System.String, System.String, Boolean)
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
20230910.1.0
No Public Link.
Is it regression?
Yes. Rollback to previous build works
Expected behavior
Service Fabric Task to succeed.
Actual behavior
Service Fabric Task fails.
Repro steps
Use Pipeline with ServiceFabric Task.

Hey @gakama!
Could you provide more info? It would be nice to drop here relevant yaml snippet with repro for the mentioned issue.
Now new Windows 2022 image (version 20230918) is in progress. I proposing to wait a bit for this image to test it. What do you think about it?

Hey @gakama! Could you provide more info? It would be nice to drop here relevant yaml snippet with repro for the mentioned issue. Now new Windows 2022 image (version 20230918) is in progress. I proposing to wait a bit for this image to test it. What do you think about it?

hi, I'm consumer of 1ES Hosted Pool v1.0.22437501 which was updated recently.
I don't use AzDO task but I can see the same error on basic SF SDK PowerShell script:
+ $success = (Test-ServiceFabricApplicationPackage $PkgPath -Verbose)
+             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Test-ServiceFabricApplicationPackage], MissingMethodException
    + FullyQualifiedErrorId : System.MissingMethodException,Microsoft.ServiceFabric.Powershell.TestApplicationPackage

Hey @ev-ostr! May I ask you to check agent version? Is it 20230918 or 20230910?

Hey @ev-ostr! May I ask you to check agent version? Is it 20230918 or 20230910?

I'm not AzDo admin unfortunately, maybe @gakama could provide this info based on 1es pool version.

20230910.1.0 is the broken one, and rolling back to 20230820.1.0 unblocked us. I don't think 1ES has ingested 20230918 yet.

We are also seeing this issue on 20230910.1.0.

Where in the 1ES portal can I pin to a previous version?  We are using the following base image:
Azure pipelines - windows-2022

@sean-hoyt and others, this is not a support channel for 1ES Hosted Pools, please keep 1ESHP-specific discussion in Teams. This thread is for 1ES and related folks to discuss the underlying agent image issues with the team that manages the agent images.


Allow doc comments for atom_manager!

The atom_manager! macro is really useful, but doesn't allow any doc comments at the moment.
This PR fixes that by passing attributes (like #[doc = "Comment] ≡ /// Comment)
through the macro to the struct definitions.

The clippy complaint is definitively unrelated to this PR. I'll assume that there is no other complaints and "force-merge" this.