ORCID search should work out of the box

Describe the bug
Out of the box, ORCID search doesn't work. This means that in order to do a person search, the user should change ORCID configurations to have it working.
Mainly, out of the box, a search task is performed from:
https://api.sandbox.orcid.org/v3.0/search?q=paulo&start=0&rows=10

instead of
https://pub.sandbox.orcid.org/v3.0/search?q=paulo&start=0&rows=10

The installer must specify the orcid.api-url setting to use pub.sandbox.orcid.org address (the public address). The public address is currently not considered.
Having api.sandbox.orcid.org as the default, you are required to specify ORCID's settings like client's id an secret. And that forces you to change default ORCID settings. Only then it makes sense to change DSpace ORCID's settings, otherwise (IMHO) the system should work without changes.
To Reproduce
Steps to reproduce the behavior:

Install DSpace 7.5
Create a collection for persons
On my "MyDspace" import a person from external source
Use Orcid and search for a person
You will have an Unmarshall exception for trying to parse a Forbidden HTML page

2023-05-08 16:20:01,373 ERROR 64ed10cd-017d-4207-a5fa-90e47c106514 bcf5c650-544a-4d20-88fb-77963c0db75d org.dspace.app.rest.exception.DSpaceApiExceptionControllerAdvice @ An exception has occurred (status:500)
java.lang.RuntimeException: Unable to unmarshall orcid messagejavax.xml.bind.UnmarshalException
 - with linked exception:
[org.xml.sax.SAXParseException; lineNumber: 1; columnNumber: 1; Content is not allowed in prolog.]

Expected behavior
Without specifying any ORCID's setting, the user should be able to use ORCID public search.

Very similar/likely the same issue as https://github.com/DSpace/DSpace/issues/9333
TL;DR I think the problem is that the DSpace code uses the property where the member API link is expected(orcid.api-url), to make calls that can also be serviced by the public API.

This seems like a very tiny configuration change. If anyone wants to submit a PR we can consider it a bug fix for 7.6.x and 8.0.


Failed to register delegates

When calling delegate_register from the wallet CLI ,
the error is  Failed to register the delegate.
On the XCASH_DPOPS , we can see Invalid message.
validate_data() function exit at line 924
because messages types from wallet CLI to nodes are missing.

@zachhildreth GET / request are rejected, also in  validate_data()
curl http://<IP_ADDRESS>:18283/shareddelegateswebsitegetstatistics

Hi @picatextra
CVSS Score
We have determined that the CVSS score for this bug is a 0
So according to the Bug Bounty official documentation this bug qualifies for a award between $5.00 and 25.00 USD
We have calculated the final amount in USD based on the following criteria

Question
Answer

How long I have spent on the bug
N/A

How often does the bug occur
when the affected route was accessed

Was there any modification to the code to create the bug
NO

Did the user provide a full code fix
NO

Did the bug have to do with multi-threading
NO

Could the tester use a debugger, static analysis or both
both

Are we currently in Alpha, Beta or Live implementation of DPOPS
ALPHA

Although it was noted that after the fix it was found that the delegates and shared delegates website needed fixed as well, and this helped the price.
Based on the above the team has decided that the final awarded amount will be $20.00 USD
Please post a X-CASH Wallet address in the comments, and we will send the payment in a public transaction, post the transaction hash and then close the issue.
Thank you for helping out the DPOPS by finding this bug.

XCA1ka4bXH1bnvdT8vti1S12PfyH8fJ5XDUXyNRusyMRTBfFCawywJyCV5vvG38h71GAv77WotbmwbmbJen5bQpC7GeVx8gSws

Bounty Paid ✅

Date: 2019-03-12
Amount (USD): $20
Amount (XCASH): 681782.98 XCASH
Spot Price: 0.0000293348480972966 USD/XCASH
XCASH Address: XCA1ka4bXH1bnvdT8vti1S12PfyH8fJ5XDUXyNRusyMRTBfFCawywJyCV5vvG38h71GAv77WotbmwbmbJen5bQpC7GeVx8gSws
Tx Hash: 62bd5ffc86a0d2d2fafc92534bff1698af3a36db8c3ea63d0a51d4a34c0952c3
Link: https://explorer.x-cash.org/Transaction?data=62bd5ffc86a0d2d2fafc92534bff1698af3a36db8c3ea63d0a51d4a34c0952c3


XSLT is not valid

Hi
We are having problems getting the above working.
After many tries we used a external tool to apply the XSLT to the XML created by AX and we get an error saying that the XSLT is not valid
can you provide an AIF messages and the final XML generated using the setup above?

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

ID: 6fb2317c-d1ef-fe12-8a2e-3f2183b7c587
Version Independent ID: 4d687d28-145d-384b-cd46-b4858023d7d3
Content: Set up positive pay
Content Source: dynamicsax2012-technet/set-up-positive-pay.md
Product: dynamicsax-2012
GitHub Login: @Khairunj

Thanks for contacting us although I'm sorry you're encountering issues getting the positive pay feature set up. This appears to be more of a support issue than a documentation issue. The topic at this URL, https://docs.microsoft.com/en-us/dynamics365/fin-ops-core/dev-itpro/lifecycle-services/lcs-support, lists a number of options for getting help, including opening a support request and a link for reaching out to the Dynamics community for help. The title of that topic is Get support for Finance and Operations apps or Lifecycle Services (LCS). Thanks again for using our documentation and reaching out to us.


proposed video  metajson for youtube

Checklist

[X] I'm reporting a site feature request
[X] I've verified that I'm running ytb-up version 2022.05.18 (update instructions) or later (specify commit)
[X] I've checked that all provided URLs are playable in a browser with the same IP and same login details
[X] I've searched the bugtracker for similar issues including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue
[X] I've read about sharing account credentials and I'm willing to share it if required

Region
No response
Example URLs
youtube.com
Description
{
"title": "my test title",
"description": "my test description",
"tags": ["test tag1", "test tag2"],
"privacyStatus": "private",
"madeForKids": false,
"embeddable": true,
"license": "creativeCommon",
"publicStatsViewable": true,
"publishAt": "2017-06-01T12:05:00+02:00",
"categoryId": "10",
"recordingdate": "2017-05-21",
"playlistIds":  ["xxxxxxxxxxxxxxxxxx", "yyyyyyyyyyyyyyyyyy"],
"playlistTitles":  ["my test playlist"],
"language":  "fr",
"monetization": {"allowed": true},
"location": {
"latitude": 48.8584,
"longitude": 2.2945
},
"locationDescription":  "Eiffel Tower",
}

is there a way to pass an argument to activate monetization for uploaded video?
"monetization": {"allowed": true}

the title could by default be the specified file name if there is no title is on the command line AND none is in the json file?

{
  "uploadSetting": {
    //你可以为一个账号添加多个代理，这样子一个代理失败会自动切换
    "proxy_option": "socks5://<IP_ADDRESS>:1080",
    //默认的延迟检测 可以不用设置
    "timeout": 200000,
    //是否打开浏览器调试
    "is_open_browser": "True",
    //是否打开调试日志
    "debug": "True",
    "username"<EMAIL_ADDRESS>    "password": "U437P8Is9prmNquVerHJ9%R00a",
    //浏览器可以选择"chromium", "firefox", "webkit"其中之一
    "browser_type": "firefox",
    "channel_cookie_path": "/Users/wenke/github/tiktoka-studio-uploader/offloaddogsboner-cookie.json",
    //三选一
    //"go next after uploading success",
   // "go next after processing success",
   // "go next after copyright check success",
    "wait_policy": "go next after processing success",
    //上传过程自动录制视频 供调试用
    "is_record_video": "True"
  },
  "videos": [
    {
      "video_path": "/Users/wenke/github/tiktoka-studio-uploader/tests/1.mp4",
      "title": "my test title private draft",
      "description": "my test description",
      "thumbnail": "/Users/wenke/github/tiktoka-studio-uploader/tests/1/sp/1-001.jpg",
      //0 -private 1-publish 2-schedule 3-Unlisted 4-public&premiere
      "publish_policy": 0,
      "is_not_for_kid": "True",
          //0 "Autos & Vehicles",
          //1 "Comedy",
          //2 "Education",
          //3 "Entertainment",
          //4 "Film & Animation",
          //5 "Gaming",
          //6 "Howto & Style",
          //7 "Music",
          //8 "News & Politics",
          //9 "Nonprofits & Activism",
          //10 "People & Blogs",
          //11 "Pets & Animals",
          //12 "Science & Technology",
          //13 "Sports",
          //14 "Travel & Events",
      "categories": 10,
      // 默认为1
      // # 0-Allow all comments
      // # 1-Hold potentially inappropriate comments for review
      // # 2-Increase strictness
      // # 3-Hold all comments for review
      // # 4-Disable comments

      "comments_ratings_policy": 1,
      "tags": [
        "Radiation resistance",
        "Turning tiny",
        "Radiation blast"
      ]
    },
    {
      "video_path": "/Users/wenke/github/tiktoka-studio-uploader/tests/1.mp4",
      "age": 29,
      "title": "instant publish my test title",
      "description": "my test description",
      "thumbnail": "/Users/wenke/github/tiktoka-studio-uploader/tests/1/sp/1-001.jpg",
      "publish_policy": 1,

      "is_not_for_kid": "True",
      "categories": 10,
      "comments_ratings_policy": 1,
      "tags": [
        "Radiation resistance",
        "Turning tiny",
        "Radiation blast"
      ]
    },
    {
      "video_path": "/Users/wenke/github/tiktoka-studio-uploader/tests/1.mp4",
      "age": 29,
      "title": "scheduletopublish_at_specific_date my test title",
      "description": "my test description",
      "thumbnail": "/Users/wenke/github/tiktoka-studio-uploader/tests/1/sp/1-001.jpg",
      "publish_policy": 2,
      "release_date": "2023-12-01",
      "release_date_hour": "10:15",
      "is_not_for_kid": "True",
      "categories": 10,
      "comments_ratings_policy": 1,
      "tags": [
        "Radiation resistance",
        "Turning tiny",
        "Radiation blast"
      ]
    },
    {
      "video_path": "/Users/wenke/github/tiktoka-studio-uploader/tests/1.mp4",
      "age": 29,
      "title": "Unlisted my test title",
      "description": "my test description",
      "thumbnail": "/Users/wenke/github/tiktoka-studio-uploader/tests/1/sp/1-001.jpg",
      "publish_policy": 3,
      "release_date": "2023-12-01",
      "release_date_hour": "10:15",
      "is_not_for_kid": "True",
      "categories": 10,
      "comments_ratings_policy": 1,
      "tags": [
        "Radiation resistance",
        "Turning tiny",
        "Radiation blast"
      ]
    },
    {
      "video_path": "/Users/wenke/github/tiktoka-studio-uploader/tests/1.mp4",
      "age": 29,
      "title": "public&premiere my test title",
      "description": "my test description",
      "thumbnail": "/Users/wenke/github/tiktoka-studio-uploader/tests/1/sp/1-001.jpg",
      "publish_policy": 4,
      "release_date": "2023-12-01",
      "release_date_hour": "10:15",
      "is_not_for_kid": "True",
      "categories": 10,
      "comments_ratings_policy": 1,
      "tags": [
        "Radiation resistance",
        "Turning tiny",
        "Radiation blast"
      ]
    }
  ]
}


UFOLibError on write due to public.glyphOrder using unicode rather than utf-8

When glyphsLib reads glyphOrder it then seems to store that data as unicode rather than utf-8. This makes ufo.lib fail to write public.glyphOrder: "ufoLib.UFOLibError: public.glyphOrder is not properly formatted." Was not a problem <2.0.0.dev0. I can work around the problem with this routine, though  it only illustrates the problem, and is not intended as a patch:
    neworder = []
    for name in ufo.lib["public.glyphOrder"]:
        neworder.append(name.encode('utf8'))

I'd say this is not a bug, but rather a feature.
The glyph names in public.glyphOrder are text, not binary data. It makes sense for them to read as unicode type in python 2, which is the same as str type in python 3.
If you want UTF-8 bytes, you encode them with the respective encoding, like you do in the example snippet.

oh wait.. you're saying ufoLib raises an error? That's ufoLib's bug if it does not accept a glyphOrder list of unicode strings..
Can you paste the full traceback?

Unless it's utf-8 ufiLib complains (from what I can tell). Traceback (we wrap calls to glyphsLib within our own psfglyphs2ufo script):
Traceback (most recent call last):
  File "/usr/local/bin/psfglyphs2ufo", line 9, in <module>
    load_entry_point('pysilfont==1.1.3', 'console_scripts', 'psfglyphs2ufo')()
  File "build/bdist.macosx-10.12-intel/egg/silfont/scripts/psfglyphs2ufo.py", line 21, in cmd
  File "build/bdist.macosx-10.12-intel/egg/silfont/core.py", line 505, in execute
  File "build/bdist.macosx-10.12-intel/egg/silfont/scripts/psfglyphs2ufo.py", line 19, in doit
  File "/Tools/glyphsLib/Lib/glyphsLib/__init__.py", line 103, in build_masters
    write_ufo(ufo, master_dir)
  File "/Tools/glyphsLib/Lib/glyphsLib/util.py", line 41, in write_ufo
    ufo.save(out_path)
  File "/Library/Python/2.7/site-packages/defcon/objects/font.py", line 744, in save
    self._saveLib(writer=writer, saveAs=saveAs, progressBar=progressBar)
  File "/Library/Python/2.7/site-packages/defcon/objects/font.py", line 838, in _saveLib
    self.saveLib(writer)
  File "/Library/Python/2.7/site-packages/defcon/objects/font.py", line 855, in saveLib
    writer.writeLib(libCopy)
  File "/Library/Python/2.7/site-packages/ufoLib/__init__.py", line 935, in writeLib
    raise UFOLibError(message)
ufoLib.UFOLibError: public.glyphOrder is not properly formatted.

ouch. i prefer to fix that over at ufoLib, thanks.

it turns out this is the same issue as #236, which should be fixed now. Closing


Fix typo of log file path

Hey!
I got a bit baffled by 'pass' which I believe means 'path' of the log file. Hope it helps!

please, call ./configs/regenerate.sh from the root of repo to regenerate config examples with new comment to fix tests


Add missing closing

Without, it won't work…

Closing, as duplicate of https://github.com/pikapkg/snowpack/pull/126.


bash_completion appears to be already installed if PROFILE variable is set on a system PROFILE file

Operating system and version:
CentOS 7.6.1811
How did you install nvm?
curl -o- https://raw.githubusercontent.com/nvm-sh/nvm/v0.38.0/install.sh | NVM_DIR=/opt/nvm PROFILE=/etc/bashrc bash
What happened?
The output shows => bash_completion source string already in /etc/bashrc where this statement is wrong. This message pops just because the system PROFILE file case is not handled properly.
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 14926  100 14926    0     0  57412      0 --:--:-- --:--:-- --:--:-- 57629
=> Downloading nvm from git to '/opt/nvm'
=> Cloning into '/opt/nvm'...
remote: Enumerating objects: 347, done.
remote: Counting objects: 100% (347/347), done.
remote: Compressing objects: 100% (294/294), done.
remote: Total 347 (delta 38), reused 171 (delta 28), pack-reused 0
Receiving objects: 100% (347/347), 197.06 KiB | 3.86 MiB/s, done.
Resolving deltas: 100% (38/38), done.
* (HEAD detached at FETCH_HEAD)
  master
=> Compressing and cleaning up git repository

=> Appending nvm source string to /etc/bashrc
=> bash_completion source string already in /etc/bashrc
=> Close and reopen your terminal to start using nvm or run the following to use it now:

export NVM_DIR="/opt/nvm"
[ -s "$NVM_DIR/nvm.sh" ] && \. "$NVM_DIR/nvm.sh"  # This loads nvm

What did you expect to happen?
As the vision of nvm is focused "per-user only", the detection function (that decide if the file set in PROFILE variable is a bash or zsh profile file) should detect if the file provided is a user one. If it is a system one, the installation should stop, or a BIG WARNING should pe printed.

Title and description edited as concluded in Pull Request #2569


Visualizations are poor

Describe the bug
Figure out a way to improve the terminals.

Fixed with tmux script.


Update owners.json

@mesosphere-mergebot bump-ee

@mesosphere-mergebot bump-ee

Fix for - Owners.json requires proper owner Jira id for the override functionality to work. https://jira.mesosphere.com/browse/DCOS-40767

@mesosphere-mergebot bump-ee


Geode 9501 microsoft v142

This PR switches to latest Microsoft tools (i.e., compiler and linker). It uses cmake to decide which tools to use based on what's available on the build machine.
Notes:
I have tested locally on my Windows machine. Since I have Visual Studio 2017 and 2019 installed, I have to be careful when setting up the environment. Normally, I launch the developer command prompt that comes with the IDE to set the tools path.  Alternatively, I can select the desired toolchain from a normal command prompt by using the desired generator at cmake config time.
Since the CI uses a separate image for Visual Studio 2017 and 2019, using a developer command prompt isn't required. cmake will find the tools that are installed.


Finish LTPacketFactory ES_DT_Command Packet Class Instantiation

All of the packet classes for packets of type ES_DT_Command need to be generated. This entails

Reading enough bytes to create a BasicCommandRT
Generating the correct class instance based on inspection of the BasicCommandRT's command field (enum ES_Command).

I realized it was easier to generate LTPacketFactory along with the other packet classes (fits better in this module anyway). Now one simply passes a byte string to LTPacketFactory's packet() function to get the correct event/result/return type packet class.


Does APNG support?

I don't think lodepng (the underlying decoder being used) supports APNG. If you want animations I would suggest looking into https://github.com/lvgl/lv_lib_gif.

It looks like APNG is an extension of PNG, so the first frame will probably display fine. However, there is no mechanism I know of to get lodepng to display the next frame.

I use libpng+apng patch to implement APNG in LVGL. https://github.com/guoweilkd/lv_lib_apng
How to add it to lvgl Repo.

Happy to hear that you could make it work!
In this repo are using lodepng as png lib but it seems you are using a different one.
Could you send a PR with the changes? We can merge it into a new branch and if it's better than the current lodepng lib we can replace it with yours.


Better support of human pose/keypoints scenario

Thanks for your great work!
I just start using this tool to annotate a tiny dataset for human pose/keypoints. This is kind of easy to use.
However as long as I copy the keypoints from one frame to other frame, it is super hard for me to make sure which joints is corresponding to which points.
It would be good that it could visualize the order/name for the keypoint based on its order. For example, the first joint is visualized as 1 at the side of the point when I select the points. Or visualize all the order of the points when I select the annoatation.
It would be a great help for me to annotate the keypoint. Espically in the video.
Hopefully it could be a easy fix in the js end.
Best,
Yang

I agree that it may be useful. Alternative approach is to have one point per joint. Thus the point will have a separate label and you can visualize its name.

This would be very useful for creating annotations sets to use with deeplabcut and deepposekit
Also there seems to be a lack of shortcuts/cycling ability to go through keypoints efficiently (ex. in deeplabcut, after you label one keypoint it auto-selects the next label)

The feature has been implemented here https://github.com/cvat-ai/cvat/pull/1
It also was merged to current repository, you need to update CVAT from develop branch if you want to use it.
It is initial implementation and probably will be improved in the future, so, we are glad to get any feedbacks.
Thank you.

@bsekachev just want to know does this branch support annotating points and export to COCO keypoint formats?

@adamsau
It supports annotation points and dumping in CVAT format.
Dumping in COCO format is in progress #4821


fix: pcss in manifest.json saved as css (fixed #12009)

closed #12009

Description

Additional context

What is the purpose of this pull request?

[x] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the PR Title Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[x] Ideally, include relevant tests that fail without this PR but pass with it.

/ecosystem-ci run

Hmm vite-plugin-ssr seems to be consistently failing with this pr. @brillout maybe you have some ideas why it's happening here. (Note: vite-setup-catalogue is currently failing in main too)

VPS expects all style entries to end with .css, AFAICT rightful so. At least, I certainly don't expect PostCSS files to exist in dist/client/. Therefore I'd argue that all style files should end with .css.

IIUC the output would still be .css. This PR fixes the case when the src in the manifest file points wrongly to, for example, src/style.css instead of src/style.pcss, where the latter is the actual existing file. Is vite-plugin-ssr using the src for something?

VPS doesn't use the src property, so it seems like this PR has some side effects? @faga295

Probably, maybe the assets originName has other uses

nice~this will help a lot

I think any project will surface the unwanted diff, probably doesn’t have
to be a VPS one
On Tue 14. Feb 2023 at 17:59, faga @.***> wrote:

nice~this will help a lot
—
Reply to this email directly, view it on GitHub
https://github.com/vitejs/vite/pull/12030#issuecomment-1430074628, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHVQRRGXTS6DQJIS6JL2UDWXO2VXANCNFSM6AAAAAAUZIHGJU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Seems the originalName of assets chunk only used in generating manifest.json.I really can't think of any other side effects.

@brillout I'm pretty sure this PR doesn't introduce any unwanted change in the manifest, and only changes what's outlined at https://github.com/vitejs/vite/issues/12009#issue-1578883894. The originalName change in this PR is only used in a few places. I'll run vite-plugin-ssr's test suite again just in case, but this PR looks safe to me.

/ecosystem-ci run vite-plugin-ssr

Looks like it's still failing at https://github.com/brillout/vite-plugin-ssr/blob/e0e96a570a2cc2ded9041fe40d4c68d0a99c7edb/vite-plugin-ssr/node/runtime/renderPage/getPageAssets/getManifestEntry.ts#L44, seems like it's indexing using the manifestKeyMap but this PR makes the index undefined? The getManifestKeyMap doesn't look out of the ordinary to me.

Side effect: https://github.com/brillout/vite-pr-12030-reprod/commit/9e6893de2de4c147dadad131bf503ae88c4245b9#diff-3b0435c9efe073e8ab8263bbad459657ba8d6826e4a2e5079b67438cb535a466.
The only diff causing this side effect is this PR: https://github.com/brillout/vite-pr-12030-reprod/commit/f0aec132ebc3beb941bf44c964315698a0f19c9e#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519.

(@bluwy FYI https://github.com/brillout/vite/commit/5af52d1e00fbfb48d308eacf44472b00ea345f2c.)

Indeed, I debug it and I realize that cssAssetName is not always a cssRequest, such as index.html, so the originalName should be cssFileName instead of cssAssetName

/ecosystem-ci run vite-plugin-ssr

Thanks for helping to check that 🙂 Didn't expect a html file in the CSS plugin. I'll tweak the code a bit and run ecosystem-ci again.

/ecosystem-ci run

Seems it still has some problems.

the simplest solution of this issue is add .pcss, .postcss, sss based on ispreprocessor, but not include css.


Threat protection policies

Threat protection policies

doc link: https://docs.wso2.com/display/AM300/Configuring+Threat+Protection+for+the+Gateway


Write intro copy

We need to provide contextual, introductory copy to a few pages, including:

Readme
Wiki Home
Tools Home note:this page needs to be renamed

Wiki Home copy: https://docs.google.com/a/bocoup.com/document/d/1xE3B8tTjvcxFSy3bCTUH5Dn-32FzaRvnmMuTgoUKZrU/edit?usp=sharing

I've got something in the above doc for the wiki home, please feel free to comment, edit, add @iamjessklein, @deathbearbrown, and @pamela-drouin :)

Tools home page (or whatever it will be) https://docs.google.com/a/bocoup.com/document/d/1Wth-FH_219phWzTfnfGjyAfIOvYivUOe4oszy8_iYRg/edit?usp=sharing

So I've just done some short paragraphs for both of these, nothing fancy, but hopefully giving a bit more information and bit more welcoming to someone who may want to contribute or use the tools.

These are all ready for feedback from anyone who wants to give it, @iamjessklein, @deathbearbrown, and @pamela-drouin

README is done, need review on Wiki and Methods home page.

Methods home is updated.

hi I added some comments to these docs

Closing as I think we are good on this front now, moving wiki home over now.


f-string support

hope to have f-string like in python
reference:
https://peps.python.org/pep-0498/
https://realpython.com/python-f-strings/
https://github.com/bicycle1885/Fmt.jl
small example:
a="world"
println(f"hello {a}")    # hello world

b=1.234567
c = f"number {b:.3f}"    # "number 1.234"

d="aaaa"
println(f"{d=}")   # d="aaaa"

This would be nice, though I can't say I want to tackle the actual implementation myself.
Regarding the syntax itself, we currently have {annotations+styles:content:fmt} with the string macro. This is however, rather ugly to use if you just want to format an expression — {:$(expr):fmt}. Upon further rumination, I'm thinking it could be better to split the syntax into {annotations+styles:content} + ${fmt:expr}. I'm not thrilled about the subtle difference between {} and ${}, but I can't think of anything better at this moment.
Having fmt:expr seems like a pointlessly annoying departure from the pythonic syntax of expr:fmt, however I've realised that the colon is actually rather annoying. In python, since : is used to create block forms (defun func():, if x:, etc.) you can safely assume that expr will not itself contain non-string colons. However, this is a very different story in Julia with the various ways we use : (indexing, ranges, etc.).
As I see it, we'll need to pick between:

requiring all :s to be escaped
putting the formatting spec first, so we just need to look for the first colon and then parse as normal

I currently see that second option as the lesser evil.

One could probably actually use a space as delimiter and stop the previous expression wherever the parser says it's done. But that could also be confusing. Like ${1.0 + 2.0 format}.
I'm also not sure if mixing annotation and format directly makes so much sense, as in styled"{style:value:format} but because format mostly applies to single numbers and style mostly applies to a section of a larger string. So separating the two might make sense, as you say. Like styled"{error:The error number is ${.2f:the_number}}"

Another idea, what about ; as the delimiter? It's already used in Julia to end expressions (except in arrays, but they have their own delimiters). So styled"{error:The error number is ${the_number;.2f}}" and styled"{error:The error number is ${1 + 2;.2f}}"

Hmm, on the face of it that seems like it could work nicely, I just have two reservations:

Because the :-syntax isn't used in a "normal Julia code" way, I think that helps signal it's working a bit differently there. I'm not sure if ; makes it clear enough what comes next isn't just another expression
Is it reasonable for someone to want to do ${.2f:arr+=1;arr/2}?

With ; I would actually put the format at the end, matching python. Then the rule would just be, the content can just be a single expression (which makes sense because you want to have one value to interpolate), so it can't have multiple ;s on the top level. It could of course be a begin-end block with internal ;s etc. but that's why you parse this as a julia expression, not a simple string.

@aplavin Yep, we do already have formatting options. The way I'm interpreting this issue though is as a request for a convenient way to mix formatting specifications and string styling.


Upgrade flow dependencies

Closes #???
Description

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d910d27) 39.93% compared to head (d2deeb1) 39.93%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1306   +/-   ##
=======================================
  Coverage   39.93%   39.93%
=======================================
  Files          39       39
  Lines        2161     2161
=======================================
  Hits          863      863
  Misses       1203     1203
  Partials       95       95

Flag
Coverage Δ

unittests
39.93% <ø> (ø)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Support NixOS

There are a couple with running this on NixOS:

locate is not installed by default on NixOS. Even if one wanted to install it, finding files through it might take some time. NixOS' own database can be used for finding the plasma-changeicons command
On Bash return cannot be used for exiting a command. This makes use of return 0 instead

Thanks for your contribution!

I would prefer more universal code

Can you put the respective binary into $PATH? For example by symlinking it into something like ~/.local/bin?
Is there any other mechanism to find binaries in NixOS that is available in other distros as well?
The config is loaded after the BIN_* definitions. You could put it in your config.

Ah I remember the exit/return inconsistency - pushed 2539a4e

No response, closing


Feat/prevent global settings from affecting local settings

Thanks for writing this! After a while of using without any issues I found some undesired behavior for me.
+--------+-----------+-----------------+------+----------+---------+
| Name   | User      | Email           | Sign | Key      | SSH Key |
+--------+-----------+-----------------+------+----------+---------+
| * priv | User Name |<EMAIL_ADDRESS>| ✓    | XXXXXXXX |         |
+--------+-----------+-----------------+------+----------+---------+
| work   | User Name |<EMAIL_ADDRESS> | X    |          |         |
+--------+-----------+-----------------+------+----------+---------+

In an environment where priv is the default profile (set in the global git config),  setting work in a local repo leads to commits being signed with the key of the priv profile. This is undesired behavior for me. After running git switch-user set work in a repo I'd expect commits no to be signed at all.
This PR changes the behavior so that the global profile doesn't affect any local settings. This is done by checking if a global value exists and, instead of unsetting it, overwriting it in the local config.
I also made a change that prevents failures in git --set/--unset from crashing the whole application.
Let me know what you think.
Kind regards
Alexander

You're absolutely right, I didn't take this case into account. Thanks for the contribution!
I'll merge it, clean up some code a bit and make a new release soon :+1:

Great news. Looking forward to the new version and to deleting my fork and using the upstream version again.

Here it is, let me know if it works as expected for you!
https://github.com/cquintana92/git-switch-user/releases/tag/1.1.0

Thank you very much. I just tested it - it seems something was lost in refactoring ;) I created a new PR that restores the behavior described here.


贴的代码有#的话，会被当做标签？

Describe the bug

Steps to reproduce
rt
Screenshots or additional context
No response

Duplicate of #629


Send and TC Enhancement:  Utilizing a new AddressBook structure

Support utilising a new AddressBook structure for where we’re organising different kinds of trusted contacts.
https://github.com/bithyve/hexa/compare/feature/send-flow-refactoring#diff-234c270f5e617d5465e01e5cb081d9fa6d037ce5b152c2cf474beec6908390b6R37
FYI @CypherPoet

Would need to have further discussion in order to understand the required changes.

This has been completed.


NexCloud changes name to nexclipper-agent

We have changed its name and convert the full application to saas agent.

Hi. I've just leave the old nexcloud package and changed its
minDcosReleaseVersion to 999.
Please checkout the PR #2018.
Thank you
2018년 10월 31일 (수) 오후 2:04, Joerg Schad<EMAIL_ADDRESS>작성:

@joerg84 commented on this pull request.
In repo/packages/N/nexclipper-agent/0/resource.json
https://github.com/mesosphere/universe/pull/2018#discussion_r229564831:

@@ -1,18 +1,16 @@
{
"images": {

P.S. I believe that is also causing the one test failure below
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mesosphere/universe/pull/2018#discussion_r229564831,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMmduVUNJGeCQNPndqzZC4B8Uu6Yv345ks5uqS9zgaJpZM4XdrX6
.

I've just left the old nexcloud package and changed its minDcosReleaseVersion to 999.
Please check out the PR.
Thank you

@ssipflow Sorry I saw one more thing, the docker image tag is latest which is unstable. Could we have a dedicated tag here?

Sorry, It is a typo. I've fixed it.

@ssipflow removing is the same as latest, hence will change if you push a new image.
Could you create a new tag (docker tag ...) e.g., with ....:v1.0.0?

@joerg84 OK, we have pushed a new image and added new tag ...:dcos .


Can we create a query for search inventory items where user will have the freedom to choose the table name, column name and value?

Actually I want to create a search query where usr could pass tableName, columnName, and Value for search dynamically in a manner example:-
Future<List<foo>> getItems(tableName, columnName, value)
{
//here I would be able to search using (select * from tableName where  columnName=value)
//can i do something like this?

}

It's hard for moor to generate apis for that because the return type would depend on what you pass as tableName. That said, maybe something like this could work:
Future<List> getItems<V>(String tableName, String columnName, V value) {
  final table = allTables.singleWhere((t) => t.tableName == tableName);
  final column = table.columnsByName[columnName];
  assert(column != null);

  final query = select(table)
    ..where((row) => column.equalsExp(Variable<V>(value)));

  return query.get();
}

it gives error for .columnsByName

Which moor version are you on? That code compiles for me on the latest version.

its moor: 2.4 and moor_flutter: 2.1.1

Ah right, sorry. I ran that on my develop branch. With the latest stable version this is very awkward to use:
Future<List> getItems<V, T extends SqlType<V>>(String tableName, String columnName, V value) {
  final table = allTables.singleWhere((t) => t.$tableName == tableName);
  final column = table.columnsByName[columnName];
  assert(column != null);

  final query = select(table)
    ..where((row) => column.equalsExp(Variable<V, T>(value)));

  return query.get();
}

You would then have to specify the type parameters when using that function, e.g. await getItems<int, IntType>(...).
At least it will get better in moor 3.0

I didn't get the e.g. await getItems<int, IntType>(...). what parameters do
we need to pass? I'm confused, can you help me to resolve this?
On Mon, Feb 24, 2020 at 2:16 PM Simon Binder<EMAIL_ADDRESS>wrote:

Ah right, sorry. I ran that on my develop branch. With the latest stable
version this is very awkward to use:
Future getItems<V, T extends SqlType>(String tableName, String columnName, V value) {
final table = allTables.singleWhere((t) => t.$tableName == tableName);
final column = table.columnsByName[columnName];
assert(column != null);
final query = select(table)
..where((row) => column.equalsExp(Variable<V, T>(value)));
return query.get();
}
You would then have to specify the type parameters when using that
function, e.g. await getItems<int, IntType>(...).
At least it will get better in moor 3.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/simolus3/moor/issues/404?email_source=notifications&email_token=AJBXU6C6TMOPJKKYWGWWQULREOCOBA5CNFSM4KYHQWT2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEMXAFRY#issuecomment-590217927,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJBXU6DDGNWVKPXQQXVJKOTREOCOBANCNFSM4KYHQWTQ
.

Of course. The second type parameter denotes which type a column or variable would have in sql. An instance of that class is also responsible to map Dart values from and to sql. When you use variables, that type needs to be set so that the variable can be mapped to sql.
The mapping is

int -> IntType
double -> RealType
String -> StringType
DateTime -> DateTimeType
Uint8List -> BlobType

In the next breaking moor version we'll remove this. Perhaps, a better way to write this would be
Future<List> getItems(String tableName, String columnName, Variable value) {
  final table = allTables.singleWhere((t) => t.$tableName == tableName);
  final column = table.columnsByName[columnName];
  assert(column != null);

  final query = select(table)
    ..where((row) => column.equalsExp(value));

  return query.get();
}

You could then just use getItems(tableName, columnName, Variable.withInt() instead of that weird IntType thing.

I'm getting the same error again and my calling is something like this
await   appDatabase.getItems("items", "item_type", "EAR").then((t){print("Printing Values for dynamic Table fetching : $t");});

Yeah the type arguments need to be set on the Variable you pass. If you write <String, StringType> in the method definition, those will be type variables and won't point to an actual class.
If you know you're only going to pass strings, you could use
Future<List> getItems(String tableName, String columnName, String value) {
  // ...
    ..where((row) => column.equalsExp(Variable.withString(value))
}

If you want a more general approach you can use the method from my previous comment.

Thank you so much Simon, its working and keep it up!!

One more thing, can I get specific columns like select column1, column2 from tableName.. instead of select * from tableName... while selecting by passing parameters?

This should be possible in the latest moor version, yes. Instead of select, you'll want to use selectOnly, then you'd use addColumns to add the columns you need (you can look those up by their name as well).


Proper Authentication

Currently the app takes name in the beginning to verify the user. devise a way to add some sort of phone or google authentication.

Google authentication might be the best option or we can also use phone number authentication using firebase if that is an option.

Google authentication might be the best option or we can also use phone number authentication using firebase if that is an option.


Enterprise Bot Template Exception with Web Chat

Project
Enterprise Bot Template
Description
In Web Chat, after waking it up with "hi", get the welcome card and "What is your name?" then "Sorry, it looks like something went wrong.".
To Reproduce
Created a new bot from the Enterprise template (<IP_ADDRESS>), works fine in emulator (development and production), but in Web Chat, after waking it up with "hi", get the welcome card and "What is your name?" then immediately (without waiting for input) "Sorry, it looks like something went wrong.".
Expected behavior
As per behaviour in emulator, should wait for input after "What is your name?".
Additional context
I have found this is due to a MissingMemberException exception at the first line of OnboardingDialog.AskForEmail where it retrieves the state:
_state = await _accessor.GetAsync(sc.Context);
One odd thing is that the AskForName method is called before this and successfully gets the (empty) state, but then doesn't actually wait for a response.
Stack trace:
System.MissingMemberException: Property not set and no default provided. at Microsoft.Bot.Builder.BotState.BotStatePropertyAccessor`1.d__7.MoveNext() at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task) at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task) at EnterpriseBot.OnboardingDialog.d__8.MoveNext() in C:\dev\EnterpriseBot\EnterpriseBot\Dialogs\Onboarding\OnboardingDialog.cs:line 62 ...

I'll check this out.

In the short-term, if you change that line to this, it should keep it from breaking. I'd be curious to see how it responds though and if it is able to save the values in state.
 _state = await _accessor.GetAsync(sc.Context, () => new OnboardingState());

I'm also having this error and have tried the change mentioned. It does move the 'Something went wrong error' to appear after I have entered my name rather than when I enter 'Hi'.

I have the same problem. Tried to work around...
If you start the conversation with 'cancel' (if you have dialog interrupt implemented) it goes out to the main dialog and works fine... until you enter any other dialog that tries to get the state.
The problem may related to OnStartAsync override in MainDialog : RouterDialog not being triggered until message is received.
Using ngrok and connecting to the production environment with the emulator - the problem disappears.

How are you testing with webchat? Are you using the test panel in the portal or creating your own?

I was using version 4.1.1 of the components.
Tried to update to 4.1.5 and it's not working at all - messages are crashing the application
Trying to find a reason - created a new project into new environment - [22:46:15]POST201directline.startConversation
[22:46:16]Refresh access token failed with status code: 401
The webchat was the standard one from channels:
<iframe src='https://webchat.botframework.com/embed/Angelique?s=YOUR_SECRET_HERE'  style='min-width: 400px; width: 100%; min-height: 500px;'></iframe>

Maybe this is a different issue, but in connection the the above post:
After executing:
msbot clone services --name "Angelique" --luisAuthoringKey "..." --folder "DeploymentScripts\msbotClone" --location "westus" --appId "1c147b6e-." --appSecret "qmn..."
Infrastructure was created. But when I called:
msbot list --bot Angelique.bot --secret "M14p3..."
the appId was reported as a different one (something starting with "fb3a391e-")
After I changed all appId-s back to the initial one - it all works on Emulator - both local and production
It still not works with webchat or with test panel in Azure.
(web chat is recording issues: There was an error sending this message to your bot: HTTP status code Forbidden)

This defenetly is a different issue - the appId is wrong and I can't change it - I don't have an application with this is.
This is why the comunication fails

I was able to reproduce the issue and I'm debugging now.
@ivlaevski would you mind opening a separate issue for the appId problem? It seems like a bug in the deployment process that we may need to forward to the botbuilder tools team

I'm actively fighting with the original issue now, exactly as described...  Onboarding state is fine in the emulator, but not in webchat.   Happy to help if I can.

I'll take a crack at it right now.  Thanks!

The old webchat used a token that looks different from your secret.  Where does one acquire such a secret?

You go to the channels tab in the Azure Portal and add the Directline channel. It uses that secret

Ok... same token, just looked radically different to me... guess it should.  :)
Tried updating the client side code...  I like how it kicks the bot to start the conversation.
I'm a little leery of whipping up an empty state object at the point were this does, but it seems to get us past the error...  thanks!

It's not keeping the state - there is no error, but no data also.

I will try this approach also... For now what i see is that it keeps the state until browser refresh - I don't know how shall it behaive, but after refresh it is asking for Name - it dosn't remember me.

@ivlaevski that is to be expected. The web chat experience does not persist across sessions unless the driving web page configures a specific user id. If you have more questions about how to extend webchat they have a very active repo with documentation of a lot of scenarios here: https://github.com/Microsoft/botframework-webchat. I'm sure they'd be happy to help with specifics!

These bugs are resolved in the latest version of the Enterprise Template in the marketplace and in the EnterpriseBotSample here: https://github.com/Microsoft/AI/tree/master/templates/Enterprise-Template/src/csharp/EnterpriseBotSample
Closing for now, we can reopen if needed.

@lauren-mills can you point out what is changed in the above mentioned template. As we are using old template and cannot afford to move to this one right now. and we need this fix.
Thanks

@ivlaevski Expanding on @lauren-mills answer, you can generate a unique .userId and cache it in a cookie:

    <script src="https://cdn.botframework.com/botframework-webchat/master/webchat.js"></script>
    <script>

        function getUserId() {
            function getCookieUserId() {
                var name = "cachedUserId=";
                var decodedCookie = decodeURIComponent(document.cookie);
                var ca = decodedCookie.split(';');
                for (var i = 0; i < ca.length; i++) {
                    var c = ca[i];
                    while (c.charAt(0) == ' ') {
                        c = c.substring(1);
                    }
                    if (c.indexOf(name) == 0) {
                        return c.substring(name.length, c.length);
                    }
                }
                return "";
            }
            function setCookie(value, days) {
                var expires = "";
                if (days) {
                    var date = new Date();
                    date.setTime(date.getTime() + (days * 24 * 60 * 60 * 1000));
                    expires = "; expires=" + date.toUTCString();
                }
                document.cookie = "cachedUserId=" + (value || "") + expires + "; path=/";
            }
            function s4() {
                return Math.floor((1 + Math.random()) * 0x10000)
                    .toString(16)
                    .substring(1);
            }

            let cookieUserId = getCookieUserId();
            if (!cookieUserId) {
                cookieUserId = s4() + s4() + '-' + s4() + '-' + s4() + '-' + s4() + '-' + s4() + s4() + s4();
                setCookie(cookieUserId, 365);
            }

            return cookieUserId;
        }

        let dl = window.WebChat.createDirectLine({ secret: 'YOUR_DIRECTLINE_SECRET' });
        let userId = getUserId();

        window.WebChat.renderWebChat({
            userID: userId,
            directLine: dl
        }, document.getElementById('webchat'));

        dl.postActivity({
            from: {
                id: userId,
                name: 'User'
            },
            name: 'startConversation',
            type: 'event',
            value: ''
        })
            .subscribe(function (id) {
                console.log('trigger "startConversation" sent');
            });

    </script>

Better yet would be to have the page the bot is on be rendered from a View, and generate the user.id server side in the View's Model.


[PLAT-675] feat: add danger button

changes: minor
Proposed Changes

PLAT-675

add danger button

Checklist

[ ] I have added or updated unit tests
[ ] I have added or updated integration tests (if appropriate)
[ ] I have added or updated documentation / readme (if appropriate)
[ ] I have verified that my changes have not introduced new lint errors
[ ] I have updated the change log

Testing Instructions

@jrschumacher Please check this out

@aleonov-virtru actually we need to update the Loki snapshots probably.

@aleonov-virtru action items

[ ] Update Loki snapshots to include the new danger state
[ ] Add danger knob to button default story

@aleonov-virtru approved


icon to copy address to clipboard: un-clickable

Want: clicking should copy address to clipboard
Got: clicking selects account for navigation

Yes, we need pay attention to this control

fixed


Provide user interface to add legacy secondary bridge

This is a bit of a hack, but I can't imagine non-hacky way to do that (well, we can start supporting proper configuration fragments like kernel recipes do). SOTA_LEGACY_SECONDARY_INTERFACE variable will not work until https://github.com/advancedtelematic/aktualizr/pull/425 is merged.

@stevana This is actually not too far from what you were asking about earlier!
Where does SOTA_LEGACY_SECONDARY_INTERFACE come from? Is that just to be set in local.conf?

Also, do want to make a similar interface for --secondary-config, or is that not worth it since it's only used for virtual secondaries?

@patrickvacek Yes, the variable is supposed to be set by user to some valid path on device's file system.

@patrickvacek It exports a bitbake variable as a shell environment variable inside a recipe.

This looks good except for my one minor complaint and the fact that getting this to work in practice is difficult. (The latter is not a reasonable ground to delay merging.)

I think this might be breaking oe-selftest.  Does oe-selftest -r updater work for you?

Does oe-selftest -r updater work for you?

It works for me.

oe-selftest clearly hates me :)


upgrade to rails 5.1

Bumps Rails version in Gemfile
Updates test case in build_artifact_controller_spec.rb
Explicitly specifies Rails release when inheriting from ActiveRecord::Migration

rebase "WIP upgrade to rails 5.1" to change the commit message


Switched to calling robot.retimer on SMOOTH paths

This pull request:

Switches to calling robot.retimer on SMOOTH trajectories. This avoids shortcutting trajectories that are already smooth.
Deprecates the default_timelimit parameter to PostProcessPath, which would insidiously mask the timelimit passed into the post-processor at construction time. See #341.

These changes are backwards compatible. However, you may see poor quality SMOOTH trajectory execution from (1) if robot.retimer is set to a post-processing algorithm that stops at every waypoint.

:+1:


Fix commit ref for integration tests workflow

Pull Request (PR) draft checklist - click to expand

[ ] Please review our
contributing documentation
before getting started.
[ ] Populate a descriptive title. For example, instead of "Updated README.md", use a
title such as "Add testing details to the contributor section of the README".
Example PRs: #763
[ ] Populate the body of the pull request with:

A clear description of the change you are proposing.
Links to any issues resolved by this PR with text in the PR description, for
example closes #1. See
GitHub docs - Linking a pull request to an issue.

[ ] Update CHANGELOG.md with details about your change in a section titled
## Unreleased. If such a section does not exist, please create one. Follow
Common Changelog for your additions.
Example PRs: #763
[ ] Update the documentation and/or the README.md with details of changes to the
earthaccess interface, if any. Consider new environment variables, function names,
decorators, etc.

Click the "Ready for review" button at the bottom of the "Conversation" tab in GitHub
once these requirements are fulfilled. Don't worry if you see any test failures in
GitHub at this point!

Pull Request (PR) merge checklist - click to expand
Please do your best to complete these requirements! If you need help with any of these
requirements, you can ping the @nsidc/earthaccess-support team in a comment and we
will help you out!

[ ] Add unit tests for any new features.
[ ] Apply formatting and linting autofixes. You can add a GitHub comment in this Pull
Request containing "pre-commit.ci autofix" to automate this.
[ ] Ensure all automated PR checks (seen at the bottom of the "conversation" tab) pass.
[ ] Get at least one approving review.

📚 Documentation preview 📚: https://earthaccess--870.org.readthedocs.build/en/870/

Ok, integration tests now pass for pull_request, but fail for pull_request_target, but that's okay, because pull_request_target is not using the correct commit. Once this lands, I think this should do the trick, including preventing both pull_request and pull_request_target events attempting to run int. tests.
Only pull_request will run int. tests when PR is from another branch in this repo, and pull_request_target will run int. tests when PR is from a fork.

Thanks guys. I pulled in your suggestion @jhkennedy, so one of you will need to approve again.


Add github action to deploy to feature-staging test apps

What
GH action deploys whatever branch it is on to three new test apps
Why
We want to stop using the beta validation apps for this.
How Tested
tbd

Close in favor of #3168


adds tests for fedora

Was trying to work out if an issue was fedora based. Might be useful for adding other distros as well

Coverage decreased (-8.8%) to 48.591% when pulling c62d16453cffc6824a9ede69b70121c94da9fc61 on timhughes:timhughes-fedora-tests into a605a9eb1332875feb9175563c7778d909ef5514 on dmacvicar:master.

Might pay to have the fedora tests allowed to fail until we can work them.out

I think the failing tests need to be worked out before we merge them. Keeping them around doesn't make much sense.
What do you think, @inercia and @flavio?

Yes, it would be better to fix the tests. However I wonder I long we can go on with this "LXD Russian dolls" situation...
I would like to give a shot at what was suggested here: https://twitter.com/bradrydzewski/status/890900013664141312

is not a bad pr . we could take this pr if we make the fedora tests optional, so they are not running in travis ( since the test are failing, btw to me the failing issue is now related to the same we have on #222),
so to me seems that when we fix #222 (which is not LXC relevant) we will fix also this tests

@timhughes i think this pr can be merged if you implement via make, like make testacc fedoraor something similar syntax. We plan to move ci outside travis

@timhughes  thank you for this PR.
we are moving to Jenkins CI pipelines so i close this PR, but i consider it on #334.
If you wish feel free to contribute there or on other issues.
The plan is to have ci folder where we have then pipelines inside.


Digital Marketing Programs - Links

Update Digital Marketing Programs links.
Display advertising channel samples as a list.
Fix list item in the Paid Search and Display Campaigns section.
Add missing word.

@efreckelton Check out these changes. They should make your links work! 😀

Thanks @LawnGnome!


A cmake re-run after a git commit will cause all source files to re-compile.

Is your enhancement proposal related to a problem? Please describe.
Don't rebuild everything when git revision changes.
Today, if git revision changes and CMake re-runs then all source files are rebuilt, even if there are no changes.
To reproduce:
$ cmake -GNinja -DBOARD=nrf52840dk_nrf52840 -Bbuild samples/hello_world/
Including boilerplate (Zephyr repository): /projects/github/ncs/zephyr/cmake/app/boilerplate.cmake
...
-- Build files have been written to: /projects/github/ncs/zephyr/build
$ ninja -Cbuild
...
[150/150] Linking C executable zephyr/zephyr.elf
Memory region         Used Size  Region Size  %age Used
           FLASH:       16864 B         1 MB      1.61%
            SRAM:        5440 B       256 KB      2.08%
        IDT_LIST:          0 GB         2 KB      0.00%

# Create new commit and re-run CMake:
$ touch new_file.txt; git add new_file.txt; git commit -s -m"new file added"
$ cmake -Bbuild samples/hello_world/
Including boilerplate (Zephyr base (cached)): /projects/github/ncs/zephyr/cmake/app/boilerplate.cmake
....

$ ninja -Cbuild
...
[145/145] Linking C executable zephyr/zephyr.elf
Memory region         Used Size  Region Size  %age Used
           FLASH:       16864 B         1 MB      1.61%
            SRAM:        5440 B       256 KB      2.08%
        IDT_LIST:          0 GB         2 KB      0.00%

As can be seen, all source files are being re-compiled even if they have no changes.
This happens because BUILD_VERSION is added as a compile definition.
Describe the solution you'd like
The BUILD_VERSION should be place in a header file, for example version.h so that only files including this header is rebuilt when the header is updated.
For example like this:
$ ninja -Cbuild
...
[150/150] Linking C executable zephyr/zephyr.elf
Memory region         Used Size  Region Size  %age Used
           FLASH:       16864 B         1 MB      1.61%
            SRAM:        5440 B       256 KB      2.08%
        IDT_LIST:          0 GB         2 KB      0.00%

# Let's touch `version.h` and rebuild:
$ touch build/zephyr/include/generated/version.h
$ ninja -Cbuild
...
[10/10] Linking C executable zephyr/zephyr.elf

as can see, in last case, only source files depending on version.h like version.c and banner.c are being re-built.

@tejlmand Seems like you duplicated this issue, cf https://github.com/zephyrproject-rtos/zephyr/issues/40167

Had some issues with networking / github yesterday.
So most likely that's the cause for the double post.
Thanks for noticing.
Closed.


Allow setting Hydrogen inventory relative to mantle mass

Currently, the user can only set the hydrogen inventory in absolute terms (H_oceans). This becomes somewhat annoying when varying planet mass -- such as when comparing different planets in the same star system -- since it means that the relative amount of hydrogen is implicitly changing.
This PR makes it possible to set the hydrogen inventory in relative terms (H_ppmw) in the same way as we do with nitrogen and sulfur. This is then summed with the absolute amount, although I can imagine that  the user will probably only set one of these parameters to be >0 at any one time.

Thanks Tim! I'll make these changes.


Docs: Font color of method names is unreasonably white on a white background when using dark theme

Describe the Documentation Issue
Hey Praw maintainers, thanks for the great work.
I'm about to use this API and I'm really happy with what I've found so far.
The only sad part is I'll have to read the documentation on light theme. This is because of the issue in the title, pictured below, or directly in the site but turn on dark mode:

Attributes

[X] Yes

Location of the issue
https://praw.readthedocs.io/en/stable/code_overview/reddit_instance.html#praw.Reddit.request
What did you expect to see?
method names a bit easier to read
What did you actually see?
method names hard to read
Proposed Fix
Gotta be a code color somewhere or a css rule to fix it
Operating System/Web Browser
No response
Anything else?
No response

Still looking into this. Just been short on time.


Dont try to generate jiras as part of dependency report

Right now, the dependency report tries to generate Jiras, but we're not using Jira anymore and aren't tracking these through issues at all anymore (instead we use dependabot). This conservatively keeps the report generation around for a while, but gets rid of the Jira component. I anticipate in the long run we will no longer file the report and will instead rely solely on Dependabot.
Part of #21684

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Add a link to the appropriate issue in your description, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

R: @lostluck


project status threshold set to 0.1%

Signed-off-by: Wenqi Qiu<EMAIL_ADDRESS>

Codecov Report

Merging #2 (2da14f2) into main (0586c42) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   36.06%   36.06%
=======================================
  Files          43       43
  Lines       18756    18756
=======================================
  Hits         6764     6764
  Misses      11568    11568
  Partials      424      424


Issue 347- Update DHCP event class

#347
Following this comprehensive Zeek DHCP log

@mr-splunk here is a link for the Infoblox DHCP mapping. In general, within the network activity we have used the src_endpoint to indicate the client and the dst_endpoint to indicate the server so we would be consistent with this here.
Let me look into this!

@mr-splunk here is a link for the Infoblox DHCP mapping. In general, within the network activity we have used the src_endpoint to indicate the client and the dst_endpoint to indicate the server so we would be consistent with this here.
Let me look into this!

Hi Ania, thanks for looking, I have not had the chance to revisit this very much recently. What I can share is some Infoblox sample logs from the "Infoblox NIOS" product, which are in syslog format and what Splunk supports:

Infoblox ACK sample:

<30>Sep 13 10:10:46 <IP_ADDRESS> dhcpd[13613]: DHCPACK on <IP_ADDRESS> to 00:50:56:44:55:66 (WORKSTATION1) via eth1 relay eth1 lease-duration 600 (RENEW) uid 01:00:50:56:44:55:66

Infoblox  EXPIRE sample:

<30>May 5 11:22:11 <IP_ADDRESS> dhcpd[10005]: DHCPEXPIRE on <IP_ADDRESS> to 38:f9:d3:aa:bb:cc

Infoblox RELEASE sample:

<30>May 19 11:22:10 <IP_ADDRESS> dhcpd[10006]: DHCPRELEASE of <IP_ADDRESS> from b8:80:4f:11:22:33 (WORKSTATION2) via eth2 (found) TransID ae3db75d uid 01:b8:80:4f:11:22:33
Let me know if you'd like to sync on how those are currently mapped to OCSF.

Is it possible to instead of 'assigned_ip' and 'requested_ip', just make them server and client? This is what it would look like:
{   "activity_id": 9,   "category_uid": 4,   "class_uid": 4004,   "client": {     "ip": "<IP_ADDRESS>",     "mac": "00:50:56:13:60:56",     "type_id": 0   },   "device": {     "os": {       "name": "Linux",       "type_id": 200     },     "type_id": 1   },   "message": "DHCPEXPIRE on <IP_ADDRESS> to 00:50:56:13:60:56",   "metadata": {     "original_time": "Sep 28 04:45:33",     "product": {       "feature": {         "name": "DHCP"       },       "name": "Infoblox NIOS",       "vendor_name": "Infoblox"     },     "profiles": [       "host"     ],     "uid": "3bbe3345-d7b3-48ae-959b-4bba7a0a2890",     "version": "0.33.0"   },   "server": {     "ip": "<IP_ADDRESS>"   },   "severity_id": 1,   "time":<PHONE_NUMBER>284,   "unmapped": {     "event_data": {       "refEventName": "DHCPEXPIRE"     },     "facility": 3,     "pid": "27785"   } }


Addition in Issue Template

Description:
Hi there!
I want to add  some more points to your issue template like documentation, etc
Please assign this issue to me under Gssoc'23 @avinashkranjan

Like?, Please provide a detailed description of things you want add/implement.

@avinashkranjan Sir, I can see that we have three issue template files currently:-

bug_report.md
feature_request.md
script_addition.md

I want to add a new issue template file here "documentation.md" which will be helpful to guide the contributors working on some documentation-related issues about how they can effectively raise their issues using a pre-defined format.
Demo file:-
Documentation.txt

Go Ahead @Anshika-Jain02


Standardize on one method for detecting keyboard state

Problem
There are two ways for components to detect the keyboard state:

Using the HOC withKeyboardState - this works on native devices, but doesn't have support for mobile web
Using VirtualKeyboard.shouldAssumeIsOpen() - this appears to work on both native devices and mobile web. Originally added in this PR.

Neither option is used consistently, leading to inconsistent behavior in the app.
Solution

Make sure that VirtualKeyboard works for mobile web
Move the functionality from VirtualKeyboard into withKeyboardState
Use withKeyboardState everywhere
Remove VirtualKeyboard

CC @jasperhuangg @parasharrajat
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01e011fc436edaf3d0
Upwork Job ID:<PHONE_NUMBER>075437568
Last Price Increase: 2022-12-30

Sorry @flaviadefaria, this is not a bug, just a cleanup issue, so I am removing your assignement.

All right, here is what I found out. Virtual keyboard exports two methods:

isOpen() - Is always the same as props.isShown on all platforms and it's only used in one place here. That can easily be replaced with props.isShown
shouldAssumeIsOpen() - Is always the same as props.isShown for native devices. It always returns true for mobile web platforms. It's used in 4 places and is almost always used as a proxy for "is this mobile web?" which is also the same as isSmallScreenWidth (usually)

So, I think that all of this can be refactored to just use the two HOCs we currently have for withKeyboardState and withDimensions and we can remove the VirtualKeyboard lib entirely.

Hm, I'm not sure why GH didn't link the PR with this issue https://github.com/Expensify/App/pull/13629 but this was deployed to production, so I'm going to close this.

@thesahindia can you please accept the job and reply here once you have?
https://www.upwork.com/jobs/~01e011fc436edaf3d0

Accepted, thanks!

Paid @thesahindia , closing cuz this shouldn't need regression test steps


Minor documentation fix

The version check issue with pywebview is in Ubuntu 24.04 (Noble), not 22.04 (Jammy).

I've made this typo countless times already, too...

Thanks for pointing this out. I couldn't merge it directly because the repository has signed commits enabled, but I've made the change.


Error:  attribute d: Expected number, "M,0,0".

Hi,
i'm try to use wheelnav js with Vue.js and typescript. Finally i got it running but now when the nevaigation is created, i get the error
Error:  attribute d: Expected number, "M,0,0".
Would be glad if someone has a solution.
Thanks for help :)
Chris

Hi Chris,
What type of raphael do you use?
I suggest this:
<script<EMAIL_ADDRESS>And you can find wheelnav on jsdelivr also: https://www.jsdelivr.com/package/npm/wheelnav

Hi,
currently I'm using 2.2.7. I'll try your suggested version. Thanks for your fast reply.

The following works for me.
In raphael.js, REPLACE:
el.setAttribute(key, Str(attr[key]));
WITH:
var strValue = Str(attr[key]);
if (key === "d" && strValue === "M,0,0") {
strValue = "M0,0";
}
el.setAttribute(key, strValue);

in my case, the arrow line was smaller than the arrow-end triangle,
so I set the min size of the path to be 11
var subPathStr = guidePath.getSubpath(0, Math.max(11, thisLength));


assesment 1

Topics Covered

Streams: Learn about working with streams in your programming projects, gaining insights into efficient data processing and manipulation.
Collections: Explore the world of collections, understanding how to manage and organize data structures effectively in your code.
CSV Parsing and Writing: Master the skills of parsing and writing CSV files, a fundamental aspect of handling tabular data in various applications.

Screenshots
Bowlers with 40+ Wickets for Team CSK

Highest Wicket Taker and Runscorer for Team CSK

Top 3 for Season - Wicket Taker and Runscorer

CSV Written

Drift Video
Watch the Drift Video

Metric
Value

alert_status
:white_check_mark: OK

bugs
1

code_smells
17

reliability_rating
E (at least 1 Blocker Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:white_check_mark: OK

bugs
1

code_smells
17

reliability_rating
E (at least 1 Blocker Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
1

code_smells
18

reliability_rating
E (at least 1 Blocker Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0

Metric
Value

alert_status
:x: ERROR

bugs
0

code_smells
3

reliability_rating
A (0 Bugs)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Small change in  statusmonitor/main.go

Small change in  statusmonitor/main.go

Please check small changes


Add bip70 option to sendTransaction endpoint

Stopped sending null merchantData, cuz the server shouldn't have to bother checking if null. Just set if it needs to be.
Tweaked the parameter names on the options object, all new words in camel case get an uppercase first.


Create reference BIP39 implementation for Android

This was completed as https://github.com/zcash/kotlin-bip39


Generate R docs

#602
To add the generation of R documentation for this project.
Using a small container from dockerhub that has R 3.1 on it, install the packages needed, build the docs, archive the doc files.

I rather have this build separately as well so I'll look into making this it's own jenkinsfile.

Stage looks good. I agree it should be built separately.

Ok so the R documentation is now being generated and being uploaded to S3, the only thing left is we need to decide what kind of url/folder path we would like and also how versioning should be handled.


Automatic signature in all messages sent via Free UK Genealogy systems

Transcriber:
[Name]
[Syndicate] [Role]
[Project]
Free UK Genealogy
Coordinator:
[Name]
[Primary Role] ([All counties]), [Secondary Role]
[Project]
Free UK Genealogy
(FreeREG) Eric B:
[Name]
Place Data Manager, Transcriber
[Project]
Free UK Genealogy

To be discussed at committee for their thoughts

Closing as deployed


Include test sample files, so they are repeatable by others

This issue was raised, because @daspecster was trying to reproduce our successful test results.

Thanks @bjwatson!
For Vision, handling the response for a more complex request like Face Detection might be good to demo here.
For Speech, there are 3 types of requests that are all handled slightly differently. AsyncRecognize, SyncRecognize and StreamingRecognize. The setup for each of those is slightly different and the handling of the responses as well.
I'm not sure how important having these examples would be for new users?
The tests that you guys are working on would also be a great place to see how these are exercised.

@daspecster We aren't planning to do any more work on Vision and Speech here. These smoke tests are temporary, until your google-cloud-python system tests cover us and/or we implement auto-generated testing (which we already have for Java).

@geigerj I updated this PR as you requested offline. PTAL


Adding GitHub Actions

I am using the CPAN Digger to monitor recently uploaded Perl modules that don't have a CI system. This project does not seem to have one.
Would you be interested in a pull-request adding GitHub Actions?

Hey @szabgab, thanks for opening this issue! I'm curious if you had something specific in mind? What sort of CI workflows do you think would be beneficial here? I'm not sure if CI would have a worthwhile impact for this library since its upkeep is fairly lite, but I'm open to the idea.


Select current open pages as launch pages on startup.

Is your feature request related to a problem? Please describe.
Currently there is no way to select the home page as a launch page alongside other ones in the "On startup" settings page.
Describe the solution you'd like

As selecting "add" will open the regular file explorer to select a folder to start at, thus eliminating the option to select Files's "Home Page", I would suggest adding a "Use Current Pages" button (naming can be different ofc) to also eliminate the need to navigate the file explorer again.

Would "Add current pages" be better (appending currently opened pages) or "Use current pages", where the previously added pages in the list are overwritten?

Yes, appending the current pages looks good to me! In any case, you wouldn't overwrite stuff by accident. I like that better.

This feature is already implemented with "Continue where you left off", can you clarify if this is what you are looking for?

No, I think you understood me wrong. I thought of a simpler way to select pages for the "Open a specific page or pages" option, that also included Files's "Home" page, that currently can't be selected there right now.

@crashmit So you are looking for a way to set the home page as one of the launch pages?

Yes that's the main goal.

@crashmit Thanks for clarifying, what I had in mind to solve this was to have a text box where users could enter the path (entering "home" would work as well). There would also be a browse button to open the file picker for easier navigation. What are your thoughts on this solution?

Also dual panes make it a bit more complicated 🤔
What if we just have an "Add home" button, or a context menu on "Add button" with the sidebar items listed?

@nvi9 I don't know if we want to add UI to the settings page to adjust this setting for the dual pane feature. We should definitely support command line arguments for this, and we already support this with the continue where you left off feature. Regarding a button for the home page, it would work, but it seems like a lazy solution.

@crashmit Thanks for clarifying, what I had in mind to solve this was to have a text box where users could enter the path (entering "home" would work as well). There would also be a browse button to open the file picker for easier navigation. What are your thoughts on this solution?

That's where the other part of my suggestion comes in. If you had a "Add current windows" button, you could just add the ones you currently have opened. That way there is no need to enter any file paths or hassle with the win32 explorer. The question here is, if people have the journey "Those windows i have opened right now are really useful / I open them quite a lot, let's add them so my startup pages" . I can totally see if this is too much of an edge case, but perhaps that's something people would use.
Suggestion here: What do you think of adding a polling bot with a dedicated channel to the discord server? I think the "help wanted" or "discussion" tags don't reach as many people as a Discord poll might. In this case, we could post a poll to get the communities opinion? I think approaching this proactively would get more response, than a github issue does. But again, if you think that would be too much effort, I understand that.

@crashmit We can definitely do a poll once we have options to choose between, I think we should continue discussing them a bit to have a polished idea to present when we ask users for their preference. The reason I hesitate to add a button to add the current tabs is because the settings UI shouldn't have access to the open tabs. It's a disconnected UI and I would prefer if it stayed that way.

or a context menu on "Add" button with the sidebar items listed?

Actually I have made some trials with this as well:

This way it would be a bit hidden feature, but maybe a bit different approach would help to solve the add home issue

@nvi9 A menu like that seems like the best idea so far, what if there was an option to browse as well as a submenu to view recent locations?

Sounds good! So "Browse" and "Recent Location"?

I've made this progress so far:

Clicking on "Add" button opens a MenuFlyout and you can either pick a directory with the usual dialog or select an item from the sidebar or from the recently opened directories.
However it has some design issues because items of MenuFlyout control are not bindable for some reason by design. I need a bit more time for investigating a fully MVVM solution in case you don't have suggestions for a better control for this purpose.

@nvi9 I am not sure about the sidebar menu item, but everything else there is looking good. About MVVM, is there an ItemsSource for the MenuFlyout?

@yaichenbaum there is no ItemsSource, so I've tried to create an attached property, but for the first trial without datatemplates. I'll try to work on that a bit more today or tomorrow and try out some things.
I've added the sidebar item because currently it's the only way I can have Home item, browse opens the win32 dialog and recently contains directory entries of StorageApplicationPermissions.MostRecentlyUsedList in this initial implementation.

@nvi9 What about adding the home item to the recent items menu?

@nvi9 A menu like that seems like the best idea so far, what if there was an option to browse as well as a submenu to view recent locations?

Always on the top right?

Btw here is what I've done so far, @yaichenbaum if you think it is worth merging, I can create a PR

Always on the top, right?

Looks good, can you create a PR?

@crashmit This issue should be resolved with #3715.


notificationData is required for FCM

I just found out that 'notificatinData' is required for sending messages through FCM for android. Without that param the response shows no errors but messages don't come up in the android app.
$params['notificationData'] = ['title' => 'this is my title', 'body' => 'this is my notification text'];
$message = new Message('This is an example.', $params);
So I think the documentation here is misleading.
https://github.com/Ph3nol/NotificationPusher/blob/master/doc/gcm-fcm-adapter.md

@punknroll could you please edit it and propose a PR?

Thanks for highlighting this, I've been stuck for hours trying to work out why my notifications weren't being sent properly. The docs are definitely confusing around this.

Closed by https://github.com/Ph3nol/NotificationPusher/commit/012183c89be0dbd154c81fb769606172d0931cb2


C# Custom RichTextEffect Not in Dropdown

Tested versions

Reproducible in v4.3.stable.mono.official [77dcf97d8], v4.1.3.stable.mono.official [f06b6836a]

System information
Godot v4.3.stable.mono - Ubuntu 22.04.4 LTS 22.04 - X11 - Vulkan (Forward+) - dedicated NVIDIA RTX A2000 Laptop GPU - 11th Gen Intel(R) Core(TM) i7-11800H @ 2.30GHz (16 Threads)
Issue description
Creating a Rich text effect by extending the RichTextEffect class, setting the bbcode string and overriding the _ProcesssCustomFX, and the [Tool] and [GlobalClass] annotations does not show up in the drop down in editor, while ones created in GDScript do.  Using InstallEffect() in a C# script to add the effect works correctly.
Steps to reproduce

Load the example project
Open control.tscn
Look at the Custom Effects property of the RichTextLabel
Note that only RichTextGDScript is an option, RichTextTest (written in C#) isn't.
Run the project
Note that after adding manually in the C# script, the custom tag displays correctly.

Minimal reproduction project (MRP)
RichTextEffectsBug.zip

I looked at the provided MRP, and the class RichTextTest is in a file named NewScript.cs. This makes the script invalid, the name of the file must match the name of the class. So rename the file to RichTextTest.cs or the class to NewScript.
This and some other gotchas are documented in the C# basics documentation page.
For attached scripts, we would normally print an error in the console when trying to instantiate the invalid script. But since this script is not attached or instantiated anywhere, it won't show an error.

This is what I get for being lazy when making my MRP (although that doesn't explain why I could use it when running the project)... which does fix the issue (although now I'm wondering what I did wrong in my main project because I did have the name right there)... we can close this one, although I'll be making a documentation update PR later to add C# working examples.


Prevent enhanced addresses from source selection

Pull Request Description
Describe what this PR does / why we need it
Fix the error that arp enhanced addresses will be taken as source IP address by mistake.
Does this pull request fix one issue?

Fix #232
Describe how you did it
Create arp enhanced addresses with scope link
Describe how to verify it
Special notes for reviews

Codecov Report

Merging #235 (6914528) into main (28fee5e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #235   +/-   ##
=======================================
  Coverage   27.41%   27.41%
=======================================
  Files          23       23
  Lines        1674     1674
=======================================
  Hits          459      459
  Misses       1163     1163
  Partials       52       52

Flag
Coverage Δ

unittests
27.41% <ø> (ø)

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

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 28fee5e...6914528. Read the comment docs.


Vacuum takes a long time with no jobs - EMR and S3

I'm writing a lot of data into Delta lake, running on AWS EMR with S3 as storage layer. I'm using EMRFS.
For performance improvements, I'm compacting and vacuuming the table every so often like so:
    spark.read.format("delta").load(s3path)
            .repartition(num_files)
            .write.option("dataChange", "false").format("delta").mode("overwrite").save(s3path)

    t = DeltaTable.forPath(spark, path)
    t.vacuum(24)

It's then deleting 100k's of files from S3. However, the vacuum step takes an extremly long time. During this time, it appears the job is idle, however every ~5-10 minutes there will be a small task that indicates the job is alive and doing something.

I've read through this post https://stackoverflow.com/questions/36524945/spark-long-delay-between-jobs.  But I don't see any options on the delta side to tune any parameters.

It seems this is duplicate of https://github.com/delta-io/delta/issues/395, which is being address by https://github.com/delta-io/delta/pull/416


Intel Driver And Support Assistant v<IP_ADDRESS>

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

Updating and Renaming this manifest since case #1938 is being ignored + the file will still be in the wrong folder
Microsoft Reviewers: Open in CodeFlow

💤bot


Split gulpfile, vendor cleanup

Thanks for the help!  Much appreciated!

@brianpkelley Nice, glad you accepted it!
It was rewritten with nodejs 6.9.4 on board and as you've may noticed it has a lot of whitespace )) for easy-peasy comfortable reading. Though gulp configs look much better when written in coffeescript but let's keep it in ES6 )


Automatically move generated client-go files to repo root

Fixes #219

/retest

@kannon92 Can you checkout this PR locally, try generating the client-go files, and let me know if the fix works?

@kannon92 Can you checkout this PR locally, try generating the client-go files, and let me know if the fix works? It works for me locally when I clone jobset into /tmp folder, delete /tmp/jobset/client-go then run make generate, the client-go files appear in /tmp/jobset/client-go.

Doing this now!

/lgtm
Thanks for fixing this!


Expense - The app crashes when the administrator opens the details of a paid expense

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

Version Number: 9.0.24-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4888153
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause - Internal Team
Action Performed:
Create a new HT administrator account, create a new WS and invite a new HT employee account. For convenience, it is better to perform the steps in two tabs opened simultaneously.
Step:

Open https://staging.new.expensify.com/
Log in under the employee's account
Submit any manual expense in the WS room
Log in under the administrator account
Click on the expense payment button “Pay amount elsewhere”
Quickly click on the expense itself to open the details
Repeat steps 2-6 several times.

Expected Result:
The details of the paid expense should be opened by the adminstrator without crashing the application
Actual Result:
The app crashes when the administrator opens the details of a paid expense
Workaround:
Unknown
Platforms:

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

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

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/user-attachments/assets/fbf1b57b-12d5-42d5-9b40-8a39f3787181
View all open jobs on GitHub

@Christinadobrzyn  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Proposal
Please re-state the problem that we are trying to solve in this issue.
The app crashes when the administrator opens the details of a paid expense
What is the root cause of that problem?
The root cause is policyTagsList has array elements which are not objects with tag prop so here
https://github.com/Expensify/App/blob/6488908562131e5e411275059bb73b578a2cf420/src/components/ReportActionItem/MoneyRequestView.tsx#L219-L220
hasEnabledTags expects all array elements to be in that structure so calls Object.value on tag prop of the array item here
https://github.com/Expensify/App/blob/6488908562131e5e411275059bb73b578a2cf420/src/libs/OptionsListUtils.ts#L1317
So TypeError Cannot Conver null or undefined to object occurs and the app crashes
What changes do you think we should me in order to solve the problem?

We need to implement a safety so change it to
    const policyTagValueList = policyTagList.map(({tags}) => Object.values(tags ?? {})).flat();

What alternative solutions did you explore? (Optional)

I'm curious why the fallback values in this file are set to empty strings.
https://github.com/Expensify/App/blob/9f344833d6abb864d3dc0ccbd578ff4acb20f633/src/components/ReportActionItem/MoneyRequestView.tsx#L687-L698
This change was introduced in this PR, so I'm tagging @kubabutkiewicz and @parasharrajat since they worked on it. Was this change intentional?

There is no way this could be intentional. They tried to give a string fallback in the PR but empty string means fetch the whole collection which is a mistake we often make.

Dupe of https://github.com/Expensify/App/issues/47542


fix: wrong path of README

@ianchn: Thank you for fixing this.
I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.


Migrate Tailor rule config from run script to .tailor.yml

The functionality for filtering rules via .tailor.yml is availble in Tailor v0.6.0

Great :clap: , thanks @adityatrivedi


COVID Check Today - API Sender Onboarding

Problem Statement
COVID Check Today was previously set up as a CSV user. They now would like to be set up as an API user.
What You Need to Know
From User:
Ryan Nelson
Denver Area Manager
Covid Check Today

Hi Sarah,
My apologies for the delayed response. I am working to have backlog numbers for you by next week for CA, CO, NV and FL.
I met with our developers and shared the Programmers Guide with them. They are telling me they need additional information as far as what system/platform they will need to integrate with, and what format you will need the data transfer sent in. When I was working with Ricky, we started down this path, in particular with the excel/csv file layout that you all needed us to send backlog and ongoing records in. Is it possible to revisit this? Also, is there someone on your staff I can connect our developers with for the technical side of things?

To Do

[ ] Set up individual tickets for this API Sender Onboarding epic

I'll share this in standup with the engineers to see who could help me get this email answered.

@rnz6 @sliu1000 looks like this needs to be converted to an "API sender onboarding" ticket. I can fix the ticket details and then I'll reach out to them to request a sender KICKOFF call (cc: @jbiskie @brick-green for visibility). Thanks!

Updated 9/8/2022: BX email replied to Ryan Nelson / COVID CHeck Today:

Hi Ryan!
Thanks so much for reaching out to ReportStream.
I apologize in advance if I'm unaware whether you've yet had this conversation with Sarah before she left on vacation: have you both discussed scheduling a ReportStream Data Sender Kickoff meeting? That would be our first step of engagement. In that Kickoff meeting, we would schedule 30 minutes to go over ReportStream in greater depth as well as detailing the technical onboarding steps for data senders (we would have an engineer join the kickoff meeting to go over the key points in programmer's guide and answering any specific questions you have for getting started with ReportStream).
Does this sound like a good next step for COVID Check Today?
Thanks,
Berni

@sliu1000 FYI, we've converted this ticket from a support ticket to an Epic for "API Onboarding" since that's what the user was asking assistance on.

Update from Sender:
Hi Joel,
Thank you so much for reaching out. I apologize for the delayed response. I have been a bit out of the office last week, this week and next week on paternity leave. I'm working a little bit here and there, and will try and get the task work caught up on this project as soon as possible. It will most likely be mid to end of next week. Thank you for your patience in working with me on all of this.
Ryan

Email to Ryan on 9/20, regarding the sample file he sent on 9/16. His sample file is in keybase.

Overall, the sample looks good. I found a few edits that we'll need to have you make:
The equipment_model_name and test_performed_code should match up with columns B and F of the LIVD table. If the Accula SARS-CoV-2 Test is correct, then use 95409-9 as the code.
For the patient_dob, use "09/09/1970" for the format
On the AOE questions, please leave them blank if the information is not available, do not use "N/A"
When your developers are ready, please have them provide me their keybase.io account name, and I'll send over the API keys with some sample commands so you can start testing against our endpoint.

Ryan is currently out on paternity leave -- @BerniXiongA6 will follow up on this to re-engage when he's back.

@BerniXiongA6 Do we know how much longer he's out on paternity?

@GilmoreA6 what are we waiting on from COVID check today? I see that this is the last email that was sent to them:
ames Gilmore<EMAIL_ADDRESS>Unsubscribe
Wed, May 17, 4:06 PM
to CDCgov/prime-reportstream, me, Comment
Hi Ryan,
I have sent you a keybase message with the x-functions-keys that you will use to send to our staging and production systems
Please find the example curl commands below. The "x-functions-key" values should be replaced with the secrets in the keybase message. You can find information on sending to us in the ReportStream Programmer's guide (https://reportstream.cdc.gov/assets/pdf/ReportStream-Programmers-Guide-v3.0.pdf). Please ensure to never send PII/PHI to our staging environment. Only mock data should be sent to this environment.
Your organization name is "covidchecktoday"
Example curl command staging:
curl -X POST -H “client:covidchecktoday” -H “content-type:text/csv” –data-binary<EMAIL_ADDRESS>-H “x-functions-key:” https://staging.prime.cdc.gov/api/waters
Example curl command production:
curl -X POST -H “client:covidchecktoday” -H “content-type:text/csv” –data-binary<EMAIL_ADDRESS>-H “x-functions-key:” https://prime.cdc.gov/api/waters
Thanks,
James


CORE-4806 skip e2etests locally if cluster unavailable

Skip e2e tests running locally (not Jenkins) if the K8s cluster is not responding, or not booted up.

It looks good, but I don't see where it tells me the message "HTTP RPC Connection is down or not forwarded, skipping the test"?
This is shown when I run the tests from gradle, I don't actually know if they ran or not.
$ ./gradlew e2etest

> Configure project :
********************** CORDA FLOW WORKER BUILD **********************
SDK version: 11
JAVA HOME C:\Program Files\Java\zulu11.45.27
Corda runtime OS release version: <IP_ADDRESS>-SNAPSHOT
Corda API dependency version spec: <IP_ADDRESS>-beta+

BUILD SUCCESSFUL in 2m 27s
355 actionable tasks: 343 executed, 12 up-to-date

Guessing it might be important as a few comments are in the jira about it?

Yes, I was also unable to see the reason reported anywhere.
@pmck91 - I think you wanted to check if there is a way to make this seen using some JUnit5 runner trick?

I have done some research on JUnit5/Gradle integration, in particular around outputting the reason why the test is skipped.
So far I found no way of getting hold of this information.
There is some information here and here.
I think the underlying issue is that in Gradle org.gradle.api.tasks.testing.TestResult contains no information why the test was skipped. If it would be the case it would be possible to add a custom testLogging and output this information.
I feel we have made an improvement in this PR and I will approve and merge it.


PHP_IDE_CONFIG XDebug environment variable, remove "

In the php-fpm section of the docker-compose.yml file is a environment variable set. Xdebug is not working for me unless i change it from
        environment:
            # IMPORTANT: Set the Remote Interpreter entry matching name to `laravel`
            - PHP_IDE_CONFIG="serverName=laravel"

to:
        environment:
            # IMPORTANT: Set the Remote Interpreter entry matching name to `laravel`
            - PHP_IDE_CONFIG=serverName=laravel

i've tested the correctnes of the environment variable via:
docker exec -ti laradock_php-fpm_1 bash
echo $PHP_IDE_CONFIG

@LarryEitel might help with this. Thanks in advanced.

Happy to help. Been traveling for three weeks. Trying to catch up on stuff. Will get to this ASAP. :)

confirmed this issue.

@LarryEitel I created new PR. Waiting for your response.


HBASE-25747 Remove unused getWriteAvailable method in OperationQuota

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 35s
Docker mode activated.


Fix readthedocs build

Description
This pull request fixes the broken docs build by updating the yml file.
Fixes #2311
Type of change
Please check all relevant options.

[x] Improvement (non-breaking)
[ ] Bug fix (non-breaking)
[ ] New feature (non-breaking)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Codecov Report

Merging #2318 (2a33147) into main (337a15f) will decrease coverage by 40.72%.
The diff coverage is n/a.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.

@@             Coverage Diff             @@
##             main    #2318       +/-   ##
===========================================
- Coverage   85.62%   44.91%   -40.72%
===========================================
  Files         324      324
  Lines       29323    29323
  Branches     5405     5405
===========================================
- Hits        25108    13169    -11939
- Misses       2837    15493    +12656
+ Partials     1378      661      -717

see 193 files with indirect coverage changes


Issue when trying to access fashion event...

What happened?

Steps to reproduce
Open Client - login a guest - go to events page, jump into one of the fashion events.

Operating System
Chrome, Windows 11
Wallet
None, played as guest
CPU
i7
GPU
NVIDIA
RAM
32GB
Kernel Version
1.0.0 -2036429826.commit-e96a984
Unity Version
Latest
Realm
Nifty passage
NameTag
juzzy007

Those crashes are being tracked at https://github.com/decentraland/explorer-desktop/issues/179


Add RingCentral auth connector

Keeping the ticket in on-hold status due to the issue with accesing the sandbox (dev portal access works fine, but trying to access the RingCentral account triggers access denied error. Can't get a token without this.

Thanks to Willie's support, the issues with sandbox access were resolved, here is the link to the documentation - https://ampersand.slab.com/posts/ring-central-documentation-43xi0z2f

Moving to Blocked status pending the issue resolution on access to prod.

Removed substitutions to support test env, now the connector is strictly for the prod version


Changes to support template moves

Fixes #82 in conjunction with kubernetes/application-dm-templates#1.
This pull request contains the following changes to support the template moves described in #82:

The template registry documentation will be updated to capture the decisions recorded in #25.
References to the relocated templates in other relocated templates and in kubernetes/deployment-manager/examples will be updated to reflect the change.
Removes the templates directory from this repository.

Just a change to example path, and a comment about the fact that the examples won't work until I get my short path support in. Other than that LGTM.


Don't create a message node for EML attachments unless there is Content-Disposition: inline

This makes sure that we stay consistent with other attachment types.

Seems reasonable, thanks

I had no other changes planned for the near future, so released the changes as v5.0.1 right away


Update README

[x] include a description of the business of mixter
[x] include an link to the evenstorming result (image ?)

Done in slides


Update to the FAQ

Removed customer enrollment form
Added a url to guide customers on regional availability.

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

@suzizuber, note that if this PR is approved, the "en-us" should be removed. Thanks.

#please-close
Closing this PR due to too many conflicts and changes since this was created. Will make the changes in new PR.

Moving change request to PR 225132