Server failing to start - Browse: /Game/FactoryGame/Map/DedicatedserverEntry or LogWorld: BeginTearingDown for /Temp/Untitled_0

Hi there!
Having some issues getting a server instance successfully running.
On first run the server gets to:
[2021.11.26-13.29.26:723][ 0]LogNet: Browse: /Game/FactoryGame/Map/DedicatedserverEntry?name=Host?sessionName=SatisfactoryServer?Visibility=SV_FriendsOnly?loadgame=savefile?listen?bUseIpSockets
If the container is restarted then it'll progress as far as:
[2021.11.26-09.44.19:729][ 0]LogWorld: BeginTearingDown for /Temp/Untitled_0
but no further.
Please find attached the full log from a clean run.
FullLog-Satisfactory-CleanRun.txt

Hey! Could you please provide your full docker run command or docker-compose.yml file, please?

Sure thing:
docker run -d --name=satisfactory-server -h satisfactory-server -e MAXPLAYERS=4 -e PGID=1000 -e PUID=1000 -e STEAMBETA=false -v C:\Satisfactory-Test:/config -p 7777:7777/udp -p 15000:15000/udp -p 15777:15777/udp wolveix/satisfactory-server:latest
I get the same behaviour using ubuntu 20 as well (obvs different volume mount parameters).

I got the same behavior, but actually the server is up and running at this point. I also though it was stuck because I could not connect. However, my issue was that the game did not correctly resolve the hostname of my server. Using the IP address instead worked (IPv4), and I could create a game.

@mdreier this is what I expected to be the case, but hadn't had the time to look into it yet. Several other issues have been opened with a similar situation, and it always resolved to the server functioning fine and the issue being somewhere else.
@psycwolf how are you trying to connect to the server in-game?

I am seeing this too - last log entry being LogWorld: BeginTearingDown for /Temp/Untitled_0.
I'm happy to test more stuff, or open a separate issue if my case is different - I'm commenting here because this issue matches how my log file ends - so I assume it matches what step it's likely failing on.
I am certain my firewalls are fine - I have confirmed by stopping the Satisfactory container and listening on udp port 15777 on the host with nc -u -l <IP_ADDRESS> 15777, then from another machine send it data with nc -u my-satisfactory-host 15777. I see the data arrive fine.
However, when I try to connect to the server in-game, it thinks the server is offline. I have tried both IP and hostname of the server (just in case).
The troubleshooting section on the wiki says that pgrep -f FactoryServer.sh should show pids if the server is successfully running. After execing in (and installing procps in the docker container, to get pgrep), it exits without finding any processes.
I just just stopped my satisfactory container, deleted all the data that would be mounted into /config and started it again.
It stopped at LogWorld: BeginTearingDown for /Temp/Untitled_0 again.

@wolveix and @mdreier Can confirm, despite the logs not progressing, using the ipv4 address I am able to connect to the server. For my purposes I'm pleased with the outcome and consider the issue closed, thanks to you both for your time and input.
I'll leave the issue marked as open for the convenience of yourself (@wolveix) and @JonEllis though please feel free to close the ticket from here on without my input.

Glad to hear it @psycwolf!
@JonEllis I still believe this is a nwtwork issue rather than server issue. Please provide your full run command or Compose file.

The troubleshooting section on the wiki says that pgrep -f FactoryServer.sh should show pids if the server is successfully running. After execing in (and installing procps in the docker container, to get pgrep), it exits without finding any processes.

This won't work with this repo.

Thanks @wolveix, you're probably right about it being my issue!
Also, thanks for confirming that the pgrep won't work for this repo.
I'm starting my container through Ansible, but if I run it directly with docker run, it works fine, so it's not this projects issue.


feat: add support for 0.11.x series of Mill

This bumps the version of mill-scip which brings in support for the newly released 0.11.x series of Mill. This also updates the tests to ensure they are running against both the minimal 0.10.x and 0.11.x versions of Mill.
Test plan
Tests should all be 🍏 .

Thank you!


Add Mac Services support to File/Folder context menus

Summary
While #10213 has the highly useful goal of providing text-based Mac Services, I'd also like to see file/folder-based Mac Services to be available upon context menu clicks of tab and Project Tree files and folders.
Motivation
I'd like to be able to right-click a tab to have the repository of that file be opened in SourceTree (version control app) or use other services (including Automator-created ones) based on a clicked file/folder without leaving Atom.
Describe alternatives you've considered
A package could be designed for a one-off solution, but without integrating with Mac's cross-application Services system, one would be forced to design solutions which can't leverage this Mac-standard means of inter-app communication.
Additional context
I want to do the equivalent to this within Atom file context menus:

https://github.com/cliffrowley/atom-open-in-sourcetree has been since updated, so that particular use case has been addressed, but Services would still be useful.

The issue is still reproducible (has not yet been implemented):

Atom 1.42.0-beta0
Mac 10.15.1 Catalina
Right-click Project files or folders to see if there is a Services menu


Retry silent renew - iframe timeout

This is the first part of two MR. It implements the case to retry silent renew process in case of iframe timeouts. The case with the network timeout (via refresh token) will be done in the second MR.
I have tested it multiple hours with reduce silent renew time (every 30s). Turning off/on network via inspector knob and also physically disconnected/connected the network. Looks really nice.
As suggested previously, i have not added any new property in the settings.

Closes/fixes part of #251
Checklist

[x] This PR makes changes to the public API
[x] I have included links for closing relevant issue numbers

As this might try every 5s a silent renew, we must ensure we do not leak anything. Thus #328 needs to merged first...


Add stax support

@cryptoBeliever @gimre-xymcity @Jaguar0625 Here is what I did for the porting to the new Ledger hardware wallet: Ledger Stax.
For now the SDK is not public, hence the draft and the missing jobs in CI.
But still you can have a look at the code changes and the corresponding screenshots in the tests.
Please do no hesitate to propose changes if some are needed. Specifically regarding the icons. I made them myself by toggling pixels on Gimp and I'm pretty sure someone with more skills can do better :)

Won't fix the clang warning. Was already there and should be address in a separate PR.


Change black to only allow 80 char lines. Reformat sources.

Use a consistent black formatting style across dpctl and numba-dppy.

The changes need to be merged and not squashed.

@diptorupd please move coverage changes to separate PR.
About line width - OK. But now is not a good time for this. CI does not work. As I understand you made it automatically. Could you please repeat it later when CI will work again?

@diptorupd please move coverage changes to separate PR. About line width - OK. But now is not a good time for this. CI does not work. As I understand you made it automatically. Could you please repeat it later when CI will work again?

I can move out the coverage changes to a separate PR, but is just more overhead. As you can see, I want to merge the PR that will preserve the coverage changes as a separate commit.
I do not follow your argument about the CI. Why should a formatting change have anything to do with CI? Moreover, I can see that the public jobs ran successfully. If we wait for internal CI, then unfortunately we have to stop developing and that is unacceptable to me.

I am closing the PR for now. Instead of doing the refactoring for code improvements in this manner, I have decided to develop the changes in my own branch on my fork. It helps me as the changes are incremental and one on top of the other. Once CI is back, I will take the individual commits from my branch and open PRs.


Different Get-SpoSite results

Hello,
I noticed a difference between the result obtained by two Get-SPOSite commands for the same user on the same tenant.
Get-SPOSite -IncludePersonalSite $true -Limit All -Filter "Url -like 'my_URL'" | select Title,Url,ConditionalAccessPolicy
Get-SPOSite -Identity "my_URL" | select Title,Url,ConditionalAccessPolicy
The difference is on the ConditionalAccessPolicy property. It is set to AllowFullAccess with the first command and BlockAccess with the second command.
The right property is given by the second command. But to check the status of this property on all OneDrive with the second command, it is extremely long.
Do I have an incorrect setting? Is it a bug in the powershell module?
Microsoft.Online.Sharepoint.Powershell module version 16.0.8029.0
Thanks

@YohannT Thank you for this feedback
I am not sure why you are getting two different answers with these commands. I want to get you over to support. They have a lot more resources than we do here to dig in and see what is going on. If it is a potential bug then the will be able to document it and take it from there to get a resolution.
Here is a quick video on opening a ticket: https://www.youtube.com/watch?v=puuOy0sEg1Y
If you don't mind please keep us posted here on the resolution and I really want to get this information you are discovering back into the docs.


Failure to build PX4 with cm8jl65 driver

Bug Description
When trying to build for px4_fmu-v2_default with the cm8jl65 driver added to default.cmake, I get an error which prevents the build from finishing (see screenshots). Everything builds fine when I leave default.cmake as-is.
To Reproduce

Clone master
Add the line distance_sensor/cm8jl65 to the drivers section of /Firmware/boards/px4/fmu-v2/default.cmake
Navigate to /Firmware
Run make px4_fmu-v2_default

Expected Behavior
PX4 should build successfully unless there is a flash overflow.
Additional Info
I set up the development environment by following the official instructions on a clean Ubuntu 18.04 VM.
Full Console Output

Hi @jordancoult , I was able to reproduce your issue, thanks for reporting it!  Let me know if PR #13258 solves your issue!

Hi @mcsauder, thanks for the help. I tried your source branch and the build threw a different error:
"Error: PX4 version too low, expected at least v1.9.0
Check the git tag (current tag: 'v1.7.3')"
It seems that your branch has outdated tags. To fix this, I cloned fresh from master and applied your changes and the build was successful.

Hi @jordancoult , I have fetched PX4/Firmware 's  upstream tags and pushed them to my fork.  You should no longer see the tag issue.  Thanks for validating the patch!

Hi @jordancoult , I have fetched PX4/Firmware 's  upstream tags and pushed them to my fork.  You should no longer see the tag issue.  Thanks for validating the patch!

Closing. Fixed with #13258!


Revert setuptools-scm

@yzh119 @zhyncs setuptools-scm works poorly with Github actions and we should rather move back to version.txt managed by release-please-action than hack too further. Related PR https://github.com/flashinfer-ai/flashinfer-nightly/pull/6 makes nightly CI works the same as release CI and maybe we can unify them someday.


Couldn't install v2.3.1 from Cocoapods

Hi,
I tried to install via using Cocoapods but the latest version is 1.0.11 there. Is there any update or something needed? Or should I add manually?

I have some problems with cocoapods. You should add it manually or with git.

Ok thanks

Can we please update this ? It would be much easier to manage with cocoa pods, thanks Rene.

Hi, I agree but I never manage to keep unit tests work with cocoapods. If you have any solution for this, feel free to open a PR.

I think this has been solved by someone else through a PR, but it has not been merged back to master.
Can you confirm Rene ?
Thanks for getting back to me, much appreciated. :)
I like the tool, it has helped me very much, thank you.
Regards,

On 16 Dec. 2016, at 11:25 am, René BIGOT<EMAIL_ADDRESS>wrote:
Hi, I agree but I never manage to keep unit tests work with cocoapods. If you have any solution for this, feel free to open a PR.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/renebigot/XlsxReaderWriter/issues/51#issuecomment-267484647, or mute the thread https://github.com/notifications/unsubscribe-auth/ALZiq0xZXLHvZFf9wdKUgwIGMRqa5B6yks5rIdp2gaJpZM4KFkwN.


Dev databinding

Configured build gradle, added new icon, colors and layouts.

@GretaGri @OyaCanli please continue working further. I'm merging the PR :)


use path instead of filepath to express TUF roles

Replaces path/filepath operations with just path operations when dealing with TUF roles because path/filepath is OS-specific (for Windows it will produce a different path, for example).  path functions will adhere to our TUF role format of X/Y/Z
Inspired by discussion in this PR https://github.com/docker/docker/pull/18887#discussion-diff-48414110L40
Signed-off-by: Riyaz Faizullabhoy<EMAIL_ADDRESS>

Wondering if https://github.com/docker/notary/blob/28128c28af78ab176c96017b1f6864dd3ca2cc98/tuf/data/roles.go#L129 should also be replaced with path.Clean instead of filepath.Clean.
Also, https://github.com/docker/notary/blob/db9705dd97509e71adb1c74d466ccf5272b9990a/tuf/client/client.go#L513 calls filepath.Dir - should this also be changed to path.Dir?

This otherwise LGTM!  :)

I think the first filepath.Clean should indeed be path.Clean since we're only referring to the role itself here.  I'll update the PR
My intuition on the filepath.Dir was that it's eventually being used for a json filepath, but this actually seems like we could have some incompatibilities with Windows paths.
I think you're right that we need to change this to path.Dir but I think we would then need to update the dir to have the correct OS separators and the next path.Join to a filepath.Join, right?

Outside of storage structs that specifically operate on the FS, we should probably be using path functions rather than filepath everywhere.
We should update any storage methods that need it to convert to the system specific path.

If we want gun/targets/delegation to be translated to something like C:\notary\..path\to\tuf\gun\targets\delegation.json, we probably need to split by / and re-join with filepath.Join everywhere we do file stuff  :|

@cyli I think we can just use this within the file storage methods: https://golang.org/pkg/path/filepath/#FromSlash
It converts from a / separated path to whatever the appropriate path separator is on the platform.

@endophage I used filepath.FromSlash in the RoleTargetsPath method directly since the function description seems to imply that we should return a valid filepath.
We can handle this in the file storage code instead, if you'd prefer this function to return a / separated path regardless of OS?

@riyazdf it looks like RoleTargetsPath is being used to generate URLs for HTTP requests (at least that's one of its functions) so it might be better to update the comment. Either way, it probably shouldn't be using filepath.FromSlash as that will generate incorrect URLs on Windows.

@endophage that makes a lot of sense, I was confused with the function's description but after more digging I definitely agree with you that it'd be good to update the comment -- this is in my latest change

LGTM!

LGTM.


Noclient

Updates to API services.
Cleanly separate the different authorization modes for users (OIDC, JWT, Certificate). Only one of them can be active at any given time.
Provide ability to deploy external certificates for services and separate root CAs for client authorization.

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "updateservices",
    "component": "trireme-lib",
    "pr-id": "660",
    "commit-sha": "b2cfc657914d2a844b1e587cb7950ac5f44351f7"
  },
  {
    "project": "updateservices",
    "component": "gaia",
    "pr-id": "209",
    "commit-sha": "5f29b2b3e7ed27532a0914e01795816a9e00fa2d"
  },
  {
    "project": "updateservices",
    "component": "enforcerd",
    "pr-id": "858",
    "commit-sha": "b5907bba4008992f46f785b60a54ac3e555a08dd"
  },
  {
    "project": "updateservices",
    "component": "backend",
    "pr-id": "174",
    "commit-sha": "7c4a1c1220bd7f9b691ee3842f92b4d460d7799c"
  },
  {
    "project": "updateservices",
    "component": "apoctl",
    "pr-id": "150",
    "commit-sha": "e6b9ff4413988d4fca0473fa3f1f10eb65b6b4ec"
  }
]

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "updateservices",
    "component": "trireme-lib",
    "pr-id": "660",
    "commit-sha": "cd830697272b8ff845c54d7282ea4844692ffe73"
  },
  {
    "project": "updateservices",
    "component": "gaia",
    "pr-id": "209",
    "commit-sha": "5f29b2b3e7ed27532a0914e01795816a9e00fa2d"
  },
  {
    "project": "updateservices",
    "component": "enforcerd",
    "pr-id": "858",
    "commit-sha": "b5907bba4008992f46f785b60a54ac3e555a08dd"
  },
  {
    "project": "updateservices",
    "component": "backend",
    "pr-id": "174",
    "commit-sha": "7c4a1c1220bd7f9b691ee3842f92b4d460d7799c"
  },
  {
    "project": "updateservices",
    "component": "apoctl",
    "pr-id": "150",
    "commit-sha": "e6b9ff4413988d4fca0473fa3f1f10eb65b6b4ec"
  }
]

/build

/build - automatically fired by gogo with following PRs and commit SHAs v1.0.0
[
  {
    "project": "updateservices",
    "component": "enforcerd",
    "pr-id": "858",
    "commit-sha": "b5907bba4008992f46f785b60a54ac3e555a08dd"
  },
  {
    "project": "updateservices",
    "component": "backend",
    "pr-id": "174",
    "commit-sha": "7c4a1c1220bd7f9b691ee3842f92b4d460d7799c"
  },
  {
    "project": "updateservices",
    "component": "apoctl",
    "pr-id": "150",
    "commit-sha": "e6b9ff4413988d4fca0473fa3f1f10eb65b6b4ec"
  },
  {
    "project": "updateservices",
    "component": "trireme-lib",
    "pr-id": "660",
    "commit-sha": "cd830697272b8ff845c54d7282ea4844692ffe73"
  },
  {
    "project": "updateservices",
    "component": "gaia",
    "pr-id": "209",
    "commit-sha": "1930a20618468c563a1a71a241f9e4740bac094c"
  }
]


uncomment tests for custom type fields

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

Project coverage is 82.18%. Comparing base (939653b) to head (7b864f9).
Report is 9 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1515      +/-   ##
==========================================
- Coverage   84.46%   82.18%   -2.28%
==========================================
  Files          82       82
  Lines        9673     9650      -23
==========================================
- Hits         8170     7931     -239
- Misses       1503     1719     +216


Need to add final script update to InvController for full inventory

just some code in the else block that prompts "inventory full" for a bit with a coroutine before early returning

Actually, might need to change return type to bool so the gameObject is only Destroyed if the inventory was not actually full


Show max of three receipts in report preview

Details
The key used for images was not unique enough in the case of generic receipt images and React was messing up the count and displaying more images than we wanted to.
Fixed Issues
$ https://github.com/Expensify/App/issues/25970
Tests

Request money using the Scan feature
Upload a pdf file so that it uploads a generic image
Repeat the process 5 times
Verify that you only see a maximum of 3 thumbnails in the report preview

[x] Verify that no errors appear in the JS console

Offline tests
N/A
QA Steps
Same as tests

[ ] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/22219519/5583b024-d886-4f90-923b-9c16ba66ec0d

Mobile Web - Chrome
https://github.com/Expensify/App/assets/22219519/ad4eb882-6f10-4bfc-a36f-574356363553

Mobile Web - Safari

Desktop
https://github.com/Expensify/App/assets/22219519/3b336578-354c-4979-ae6b-54632dbcf4cb

iOS
N/A only supports images now

Android
N/A only supports images now

Bump for review!

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/tylerkaraszewski in version: 1.3.60-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to staging by https://github.com/tylerkaraszewski in version: 1.3.60-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/luacmartins in version: 1.3.60-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Correction for raw affiliation Independent scholar, Sydney, Australia

Correction needed for raw affiliation Independent scholar, Sydney, Australia
raw_affiliation_name: Independent scholar, Sydney, Australia
new_rors:
previous_rors: 014vqg108
works_examples: W4390330004;W4386390512;W4387342305;W4387979116;W4387980371;W4389444561;W4323981577;W4376287217;W4382864433
contact: 0a847f703fecd9179bc0cb2cba725d66:75599c3d7fb30ae5db529a4cc33a96 @ recherche.gouv.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


image naming and devspace dev

What happened?
When I change the image name in the devspace.yml from default to anything else the devspace dev command fails with Unable to start portforwarding ….
What did you expect to happen instead?
Start in the dev mode.
How can we reproduce the bug? (as minimally and precisely as possible)
Change the imageName default to anything else in all 3 instances:
version: v1beta6

images:
  default:
    image: […]

deployments:
[…]

dev:
  ports:
  - imageName: default
  […]
  sync:
  - imageName: default
  […]

Local Environment:

DevSpace Version: 4.6.0
Operating System: mac
Deployment method: helm

Kubernetes Cluster:

Cloud Provider: google | aws | azure | DevSpace Cloud | DevSpace Cloud Enterprise | other
Kubernetes Version: [use kubectl version]

Anything else we need to know?
Output of devspace dev -i:
Successfully built bfa9d4b229a4
Successfully tagged ???/???:USUQjGI
[info]   Skip image push for ???/???
[done] √ Done processing image '???/???'
[info]   Skipping deployment database
[done] √ Deployed helm chart (Release revision: 0)
[done] √ Successfully deployed ??? with helm
[fatal]  Unable to start portforwarding: Error selecting pod: Couldn't find a running pod with image selector '???/???:USUQjGI'

kubectl get pods
NAME                   READY   STATUS             RESTARTS   AGE
???-6988d9b5dd-x7hhk   0/1     ImagePullBackOff   0          5m11s

(PS replaced my Docker image name with ???)

/kind bug

@tomasc thanks for reporting this issue!

Fixed by #999

@FabianKramm superfast, thanks!


I really do not know what should I have install in visual studio to make it work - what nuget packages?

[Enter feedback here]

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

ID: a8bc803d-3aee-0e1f-ab64-31c5af48e9f9
Version Independent ID: 472fcb4b-39c3-72a3-d2af-4eba68b72f04
Content: Create a New Database Diagram - Visual Database Tools
Content Source: docs/ssms/visual-db-tools/create-a-new-database-diagram-visual-database-tools.md
Product: sql
Technology: ssms
GitHub Login: @markingmyname
Microsoft Alias: maghan

@MaciejAmbroziak -- thank you for your question.  Please consider these resources:

Set Up Database Diagram Designer (Visual Database Tools)
Visual Studio on Q&A
Visual Studio MSDN Forum
Stack Overflow
Reddit

cc @markingmyname for awareness
I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


New version: twpayne.chezmoi version 2.34.2

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110014&drop=dogfoodAlpha

x86 Automatic Validation ended with:

Launching executable file: C:\Users\Validator\AppData\Local\Microsoft\WinGet\Packages\twpayne.chezmoi__DefaultSource\chezmoi.exe
Exception during executable launch operation System.AggregateException: One or more errors occurred. (The specified executable is not a valid application for this OS platform.)

Is this a 64.bit executable?


'dotnet' is not recognized as an internal or external command

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/hello-world-tutorial/install
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

done

done

Closing the issue then, thanks!


complie error in new version

I just update cuda to 8.0,and when i complie the latest version,it will report
cub/cub.cuh: No such file or directory
USE_CUDA = 1
USE_CUDA_PATH = /usr/local/cuda
USE_CUDNN = 1
the makefile is as above

Did you use --recursive flag when git clone the mxnet repo?

yes,i just forget to set cuda lib in .bashrc
Now it fixed.


Maven Enforcer plugin

See https://github.com/JetBrains/jetpad-mapper/pull/129 before start

Plus, I'm thinking about publishing the following policy on the forum:

Starting from now our project uses Maven Enforcer Plugin (http://maven.apache.org/enforcer/maven-enforcer-plugin/). It means we force maven to check that our components don't have same libraries with different versions
among their dependencies.
I also dig through all our POMs and tried to make them a bit more accurate. That's why I want to ask you about the following:

do not insert a dependency in a project with a hardcoded version, keep versions in the "properties" section of
** the current pom if your dependency is extremely specific and absent in other projects;
** "datapad-deps | spring-deps" if they have relation to this level of abstraction;
** "jetpad-mapper/dependencies" if they can be used everywhere;

Let's discuss this as well.


Are db quesries asynchronous?

If I call ctx.run from multiple threads, will the database queries block each other ? For example:
class Main {
  import scala.concurrent.ExecutionContext.Implicits.global
  val ctx: Context = new PostgresJdbcContext[SnakeCase]( ... )

  def main(args: Array[String]): Unit = {
    // will a & b insert into database at the same time?
    val a = Future {ctx.run { .. insert into db .. }}
    val b = Future {ctx.run { .. insert into db .. }}
  }
}

What context should I use? PostgresJdbcContext or PostgresAsyncContext? Documentation does not really explain how do they differ and what is their purpose.
Is the maximum number of parallel queries equal to the size of connection pool?
What about the choice of ExecutionContext and usage of blocking { .. }.

PostgresAsyncContext  use nio-based driver
While PostgresJdbcContext just use the blocking  jdbc driver.
You'd better first do some research with postgresql-async, and if you decide to use it, be careful about this bug

@Josef-Vonasek Yes, you can wrap scala execution context with fixed thread pool with number of threads equals to connection pool. The only restriction is to be careful with transactions since they need to be run in a single thread.
We have gitter channel for general-purpose questions https://gitter.im/getquill/quill


E2E CV blog post part 1

Opening this PR via a branch (not a fork) to trigger proper builds.
Target publish date: May 3rd

[x] Still waiting on Header image corrections from the graphic designer.
[x] Needs commentsUrl for https://discuss.dvc.org/
[x]  There are no grammatical errors, jargon, or obvious stylistic issues. (For reviewer)
[x]  Have images been run through tiny.png?
[x]  Are Oxford commas observed?
[x]  Content Creator has done a complete review for typos/language after taking a break from content
[x]  links checked in the piece
[x]  Have reviewed what the piece looks like on your own machine using yarn develop

Looks good @alex000kim! I agree with the general feedback from @jorgeorpinel that focusing on making it easier to understand and more high-level is the main need. The content looks great.

Looks great! I have a hard time reading when it's not deployed, but I couldn't find anything that needed to be changed!

Thanks, @jendefig!
I think this is the latest deployment: https://build-722c40d9-e91c-4fe2-94ee-a659b81644d3.gtsb.io/blog/e2e-cv-pt1
I can merge first thing tomorrow morning.


c++ Plugins nodehandle is not in global namespace

As stated also by others (http://answers.ros.org/question/109371/remapping-rqt-published-topics/) the topics/services within a C++ rqt-Plugin seem to be in a private namespace, named by the plugin name.
This is not the case with python plugins, where non-global topics are correctly pushed down into the namespace that is given via "ROS_NAMESPACE", when rqt is started.
Even if this is by intention, it should be consistent between C++ and Python.

A rqt C++ plugin is not a ROS node but a ROS nodelet. Simply because roscpp does not support to run more then a single node per process. A nodelet always runs in a namespace (http://wiki.ros.org/nodelet) but can still be remapped.
Since this described behavior is intended and by design I will close this ticket since it can't be changed.


Add AllowedApplicationOrigin environment variable for non url origins

Issue #: N/A
Description of changes:
Adds a new environment variable and logic for handling origins that are not proper URLs (such as those used by Android apps).
The reason for this change is due to android apps sending android:apk-key-hash:some_hash instead of a URL. Currently the origin passed by the client is converted to a URL object which will fail with non-url origins. This behaviour is explained in the webauthn spec here

A web application with a companion native application might allow origin to be an operating system dependent identifier for the native application. For example, such a Relying Party might require that origin exactly equals some element of the list ["https://example.org", "example-os:appid:204ffa1a5af110ac483f131a1bef8a841a7adb0d8d135908bbd964ed05d2653b"].

This PR aims to avoid changing the current behaviour with allowedOrigins and adds this as an extra feature.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

We have tested this by deploying the change to our own staging environment and adding a passkey from an android device

Thanks for this PR!
We are all just back from a very hectic re:Invent week, so bear with us please while we have a closer look.

Pushed a naming nitpick. Also removed the new URL(origin).origin and just did origin now, should also work and the former felt funny looking at it.

Published to npm in v0.12.0


Throw a specific error when wifi is not actived

1/ On Mac, disable wifi
2/ run the example of README
The error return :
(node:27211) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): Error: Could not get SSID
expectation : return a specific error (Error: Wifi disabled. Can get SSID.)

Thanks for reporting. I've published a new release with a fix.


Fully hermetic component downloads

At the moment, we register SHAs for GraalVM itself, but as of the latest GVM releases, not for individual components. Instead, we rely on gu to download those components.
This may introduce hermeticity issues, so we should figure out a better way.
Alternatives under consideration:
1) Download installable JARs and install them manually with gu
This is the approach taken before. It's safe/hermetic and rather optimal for downloads because there's nothing to figure out. On the other hand, when this was one component it was cumbersome, and now that there are like 5 components, it would grow to a point of being unmanageable. Plus, with GVM releases accelerating and users authoring their own components, we'll need to provide a mechanism later anyway with better APIs.
It's also worth noting that gu is under deprecation, further mitigating any benefits to this approach.
2) Offer custom components attr, and wait for Maven transition
Offer an attribute like custom_components, which pulls JAR components or something from other repositories. This way, users could provide their own custom components if needed; how this would work mechanically is unclear. In the transition to deprecate gu, the equivalent assets will mostly be hosted on Maven Central, so we can provide hermetic access via rules_jvm_external.

went with option 1


Improved comments

This pull request includes:

[ ] breaking changes
[x] no breaking changes

Changes made to the repository:

Documentation of test results:

Related issues/discussions:

This pull request is no longer up-to-date.
We have, however, incorporated the feedback into our changed codebase.


Store string as tensor type in ONNX

ChainerCI, test this please!

ChainerCI, test this please!

ChainerCI, test this please!


Support for authenticating to private action repos

Using this inside of an organisation where the actions repo are internal or private the scripts are unable to clone the git repo as it expects them to be public.
Any chance we can add a way to specify auth details to push through to git?
I may take a look if/when I get a chance but wanted to get any ideas on implementation first
This is what happens currently
EXEC: git remote add origin https://github.com/my-org/actions.git
EXEC: git -c protocol.version=2 fetch --tags --no-recurse-submodules origin
fatal: could not read Username for 'https://github.com/': No such device or address
ERROR: Command exited with code 128

Also looking for this. Updating the remoteAdd function with:
await exec.exec('git', ['-c', `http.extraHeader=Authorization: Bearer ${github_token}`, 'remote', 'add', 'origin', `${github_prefix}${owner}/${repo}.git`])

Change the error to:
EXEC: git -c http.extraHeader=Authorization: *** remote add origin https://git.drwholdings.com/drw-actions/setup-docker-environment.git
EXEC: git -c protocol.version=2 -c http.extraHeader=Authorization: *** fetch --tags --no-recurse-submodules origin
fatal: unable to access 'https://git.drwholdings.com/drw-actions/setup-docker-environment.git/': The requested URL returned error: 400

But still no success 👎

@danpowell88 FWIW I did figure this out - I ended up making a decent amount of edits to support caching of actions that are stored on an on-prem Github Enterprise Server instance, however I had to setup auth for public github.com too. As long as your github token has the required permissions this should work for you:
- name: Run add action script
  env:
    GITHUB_TOKEN_EXTERNAL: ${{ secrets.DES_DRW_GITHUB_CLOUD_PAT }}
  run: |
    git config --local user.email<EMAIL_ADDRESS>    git config --local user.name "github-actions"

    export GITHUB_TOKEN=$GITHUB_TOKEN_EXTERNAL
    git config --global<EMAIL_ADDRESS>"https://github.com"

    ./script/add-action.sh ${{ inputs.action-author }}/${{ inputs.action-name }}


Convert this to a TypeScript project?

I am learning TypeScript and I love it so far. What do you think about converting this app to a TypeScript project?
@Steffan153 @worachatsun @m11dedhia @lucas-jg @tusharkashyap63 @Chensokheng @DharmarajX24

Yeah you should definitively do it 😉

I love TypeScript


feat(demo): add btc-proof-generator-by-rpc

add "tools/btc-proof-generator-by-rpc" to workspace/members
add docs/demo_CN.md


GH-38300: [Dev][Docs] Update dev/README.md for the current merge_arrow_pr.py

Rationale for this change
dev/README.md is outdated.
What changes are included in this PR?

Don't require arrow remote
Remove ARROW Jira related documents

Are these changes tested?
No.
Are there any user-facing changes?
No.

Closes: #38300

Can we merge this?


Custom Cluster Domains

Is this a BUG REPORT or FEATURE REQUEST?:
FEATURE REQUEST
What happened:
Cluster Domain hardcoded into NATS/Kafka.
What you expected to happen:
To be able to use my custom cluster domain.
How to reproduce it (as minimally and precisely as possible):
Set up kafka trigger on a non-cluster.local cluster. Watch it fail.
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Kubeless version (use kubeless version):
Cloud provider or physical cluster:

Hi @clatko, can you give us more details about the error? How is the domain you are trying to use?  The logs of the Kafka/NATS controller should help. I assume the culprit is this piece:
https://github.com/kubeless/kubeless/blob/master/pkg/utils/event_sender.go#L55
The same should happen for CronJobs since they use a similar address.
cc/ @murali-reddy

Agree with @andresmgot. Issue is related to use of cluster.local when using Kubernetes service backing the function pods.
@clatko Just to be sure, you are referring to the custom domain name that can be specified with --cluster-domain?

thanks @clatko, a PR will be very welcomed

:+1:  on this. I ran into this issue, but couldn't track it down and then went on vacation. Glad someone was able to find it

i have it finished, just trying to run tests and running into issues on osx. hope to have it done before weekend.

PR here - https://github.com/kubeless/kubeless/pull/857

https://github.com/kubeless/kafka-trigger/issues/22
The same issue for kafka-trigger


[Improvement] how to specify Iceberg table location

What would you like to be improved?
For Iceberg table location, here are three different treatment:

For Trino Iceberg connector: supports specifying location when creating an Iceberg table.
For Trino Gravitino connector: does not support specify the location for creating an Iceberg table
For Gravitino Iceberg catalog: the property is reserved and cannot be passed

How should we improve?
We should clarify the usage of location for Iceberg table and unify it.

@FANNG1 @diqiu50 cc

@mchades The Gravitino Trino connector supports  specify the location for creating an Iceberg table.

@mchades The Gravitino Trino connector supports specify the location for creating an Iceberg table.

I cannot find it in the document.

location is not a reserved property in Iceberg, it's a mistake.


Possible v0.1 merge?

I have spent some time working on the following branches:

loading-screen
game-scene
enhance-library

and develop is currently up to date. I think that it is time for a v0.1.
I would appreciate any feedback.

Yeah I think that it is fine. We have to work on the game though - right now all there is a TBD.


Bad Component ID

Test at https://github.com/delaneyj/bar
Added more than 64 components... You'll notice that
type ComponentMapping [3]uint64

Seems right but the ids overflow.
const AZID ComponentID =<PHONE_NUMBER>740992

const BAID ComponentID =<PHONE_NUMBER>9481984

const BBID ComponentID =<PHONE_NUMBER>8963968

const BCID ComponentID =<PHONE_NUMBER>7927936

const BDID ComponentID =<PHONE_NUMBER>75855872

const BEID ComponentID =<PHONE_NUMBER>51711744

const BFID ComponentID =<PHONE_NUMBER>03423488

const BGID ComponentID =<PHONE_NUMBER>606846976

const BHID ComponentID =<PHONE_NUMBER>213693952

const BIID ComponentID =<PHONE_NUMBER>427387904

const BJID ComponentID = -9223372036854775808

const BKID ComponentID = 0

const BLID ComponentID = 0

const BMID ComponentID = 0

const BNID ComponentID = 0

const BOID ComponentID = 0

const BPID ComponentID = 0

Love the idea of this project and have some suggestions if you are pursuing it further

Thanks for cutting these issues, this is awesome feedback and these are all great findings. I'd definitely like to hear any other suggestions you have! I've gone ahead and added you as a collaborator on this repo if you'd like to make the changes you outlined, otherwise I'll get them implemented sometime this week.

I think the generation idea can be VERY powerful but right now arche is orders of magnitude faster.  I think there are approaches that could rival it without the need for archetype graphs but not sure how much you are interested in the topic compared to a fun test.

I'm definitely interested to hear ideas. I can think of some simple optimizations that could probably make this library 10x faster in a benchmark like https://github.com/mlange-42/arche?tab=readme-ov-file#benchmarks but I don't think they'd hold up to real world usage.

Fixed the bug in the issue

Works


[BUG] Output log opens everytime requirements.txt is opened

Describe the bug
The output window for Dependency Analytics takes focus and is opened every time I open my requirements.txt file. It is really breaking my workflow.
I think the reason the output window is being focused is because some of the requests it seems to try to make fail. I am behind a corporate VPN, but have my proxy settings set within my settings.json. This is just speculation.
To Reproduce
Steps to reproduce the behavior:

Create requirements.txt
Get into a position where network requests from Dependency Analytics will fail
Open requirements.txt after previously closing it
Notice output window now has focus

Expected behavior
Never focus the output window even when there are errors
Screenshots

VSCode(please complete the following information):

OS: Win 10 1909
VSCode version: 1.50
Dependency Analytics Version: 0.2.1

@arajkumar Maybe a flag to turn off CA when not required will be helpful?

@tristan957 Thanks for reporting this.
@prashbnair , it is due to the function which we use to log error. connection.console.error causes the output window focus, switching to connection.log.warn fixes the issue.

Thanks for the quick fix!


Topdown photo

Hi There,
Does the dataset have topdown view photo for each floor in all environments?
Or is that a way to use the Habitat simulator to produce the top down photos?
Thanks!

I am wondering if it will require the real images (the original MP3D images) to produce high-quality top down photo, but unlike MP3D it seems that HM3D doesn't provide those viewpoint coordinates / images?

Will be great if you can release all the topdown RGB images for each floor for all environments and specify the scale for each of them :D
Thanks!!!

@YicongHong,
Thanks for your interest. We generated the top-down views using the orthographic camera from habitat-sim. Here is a short snippet to get raw and unscaled maps with a pixel-resolution of 0.05m x 0.05m. You will need access to this function from the metrics script.
You can pass a glb_path to the render_topdown_views function to get the top-down images for each floor.
import os
import cv2
import math
import tqdm
import glob
import imageio
import argparse
import numpy as np
import habitat_sim
import multiprocessing as mp

from metrics import get_floor_navigable_extents

TOPDOWN_WIDTH = 1280

def make_ortho_habitat_configuration(scene_path):
    # simulator configuration
    backend_cfg = habitat_sim.SimulatorConfiguration()
    backend_cfg.scene_id = scene_path

    if habitat_sim.__version__ == "0.1.7":
        # agent configuration
        sensor_cfg = habitat_sim.SensorSpec()
        sensor_cfg.resolution = [4096, 4096]
        # Reference: src/esp/bindings/SensorBindings.cpp
        sensor_cfg.sensor_type = habitat_sim.SensorType.COLOR
        sensor_cfg.sensor_subtype = habitat_sim.SensorSubType.ORTHOGRAPHIC
        sensor_cfg.parameters['far'] = '1000'
        sensor_cfg.parameters['near'] = '0.01'
        sensor_cfg.parameters['fov'] = '90'
        sensor_cfg.parameters['ortho_scale'] = '0.05'
    else:
        sensor_cfg = habitat_sim.CameraSensorSpec()
        sensor_cfg.resolution = [4096, 4096]
        sensor_cfg.sensor_type = habitat_sim.SensorType.COLOR
        sensor_cfg.sensor_subtype = habitat_sim.SensorSubType.ORTHOGRAPHIC
        sensor_cfg.far = 1000.0
        sensor_cfg.near = 0.01
        sensor_cfg.hfov = 90
        sensor_cfg.ortho_scale = 0.05
        sensor_cfg.clear_color = [0., 0., 0., 0.]
    agent_cfg = habitat_sim.agent.AgentConfiguration()
    agent_cfg.sensor_specifications = [sensor_cfg]

    return habitat_sim.Configuration(backend_cfg, [agent_cfg])

def robust_load_ortho_sim(scene_path):
    sim_cfg = make_ortho_habitat_configuration(scene_path)
    hsim = habitat_sim.Simulator(sim_cfg)
    if not hsim.pathfinder.is_loaded:
        navmesh_settings = habitat_sim.NavMeshSettings()
        navmesh_settings.set_defaults()
        hsim.recompute_navmesh(hsim.pathfinder, navmesh_settings)
    return hsim

def get_downward_quaternion():
    """
    Given a unit vector u = ux i + uy j + uz k, and a rotation angle theta
    the corresponding quaternion is as defined follows:
    q = cos(theta/2) + (ux i + uy j + uz k) sin(theta/2)

    To get downward rotation, rotate about either i or k
    since j is upward in habitat.

    By default, the agent faces -Z. To convert this vector to downward,
    rotate by -90 degree along X.

    Output format:
        q - [x, y, z, w] elements of a unit quaternion

    Reference: https://en.wikipedia.org/wiki/Quaternions_and_spatial_rotation
    """
    # -90 degree rotation about x
    q = [-0.7071067, 0.0, 0.0, 0.7071067]
    return q

def render_topdown_views(glb_path):
    sim = robust_load_ortho_sim(glb_path)
    # Get floor extents
    floor_extents = get_floor_navigable_extents(sim)
    floor_extents = sorted(floor_extents, key=lambda x: x['mean'])
    navmesh_vertices = np.array(sim.pathfinder.build_navmesh_vertices())
    floor_images = []
    scene_cent = navmesh_vertices.mean(axis=0).tolist()
    for fext in floor_extents:
        # Get navmesh vertices from current floor
        mask = (
            (navmesh_vertices[:, 1] <= fext['max'] + 0.25) & \
            (navmesh_vertices[:, 1] >= fext['min'] - 0.25)
        )
        fcent = np.median(navmesh_vertices[mask, :], axis=0).tolist() # (3, )
        # Set agent state
        agent_position = [scene_cent[0], fcent[1] + 1.0, scene_cent[2]]
        agent_rotation = get_downward_quaternion()
        agent = sim.get_agent(0)
        new_state = agent.get_state()
        new_state.position = agent_position
        new_state.rotation = agent_rotation
        new_state.sensor_states = {}
        agent.set_state(new_state, True)
        # Get observations
        obs = sim.get_sensor_observations()
        floor_images.append(obs['rgba_camera'])

    # Concatenate images vertically
    floor_images = np.concatenate(floor_images, axis=0)

    sim.close()

    return floor_images

Hi Santhosh, thank you very much for your reply and sharing the code!


Allow en-dash for time ranges.

You'd just need to change the Regex I think.

Why is this is important, though?
As the maintainer of the Tasks plugin, I would caution against this, though.
By having around half a million downloads, the Day Planner plugin has created a kind of de-facto standard for putting dates at the start of time lines.
So I think that changing the format here would have a much larger impact than on just this plugin.

It would effectively require all future plugins that want to use times like this to support en-dashes as well.
Also, any users of the Tasks plugin who have custom searches based on times would need to know to make their own regexes cope with both characters.

Why is this is important, though?

Because that is the typographically correct way to specify ranges in many languages. If you have a grammar checker then it will flag the wrong dash as an error.
It looks bad especially if you're doing it correctly in all other documents

Thanks for the context - much appreciated.
Actually, I'm going to recant on the following:

So I think that changing the format here would have a much larger impact than on just this plugin.

It would effectively require all future plugins that want to use times like this to support en-dashes as well.
Also, any users of the Tasks plugin who have custom searches based on times would need to know to make their own regexes cope with both characters.

It doesn't really require those things.
Users who only use this plugin can take advantage of en-dashes if they wish, if someone volunteers to do the work, and it's merged....
Users of other related plugins are free to to stick with hyphens if they wish.
Adding en-dash support here doesn't force any user to adopt them, and it doesn't force any other developers to add support for them either...


operator: fix typos in comments and tests

This PR fixes spelling and grammar issues in the operator folder.

[ ] Ambient
[ ] Configuration Infrastructure
[x] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[x] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Please check any characteristics that apply to this pull request.

[x] Does not have any user-facing changes. This may include CLI changes, API changes, behavior changes, performance improvements, etc.

Hi @alexandear. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Suggestion - If it's some grammar checking linter/tool you use to find those typos I think a much more beneficial PR would be to add it to the set of CI linters checks we run to future avoid these kinds and fix all current typos in one shot (PR).

@ymesika In general, I agree with the suggestion.
I'm using different tools and self-reading to find typos. Some of them are proprietary and can't be integrated into CI/CD. Also, all spell-checker tools have false positives, so I always re-check their results before creating PR.
istio repo is too big to fix all lint issues in one shot. So, to simplify the review I'm creating medium-large size PRs.
In the next PRs, I'll choose the most appropriate Open Source spell checker and integrate it into the CI/CD process. Anyway, the tool's configuration sometimes needs to be extended with new "false positive" words.

/ok-to-test

@alexandear: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
00bf21ae98f6bb7a4940eb3d93494148cbff3302
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes

@alexandear: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
00bf21ae98f6bb7a4940eb3d93494148cbff3302
link
true
/test release-notes

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-notes


Update Composer dependencies (2021-06-16-00-11)

Visual regression test passed!
View the visual regression test report


fixed AssertionError

@juliohm  I fixed the Assertion Error and it is working fine now, then again was getting ERROR: MethodError: no method matching getindex(::NamedTuple{(:a, :b), Tuple{Vector{Int64}, Vector{Float64}}}, ::Colon, ::Vector{Symbol}) for
(a=rand(Int,3), b=rand(3)) |> ZScore(:b)

I guess the ZScore function expects the input data to be a tabular data structure, like DataFrame or a Table, but the input data in this case is a NamedTuple. Converting into dataframe, I am getting correct output:
julia> DataFrame((a=rand(Int,3), b=rand(3))) |> ZScore(:b)
3×2 DataFrame
 Row │ a                    b
     │ Int64                Float64
─────┼────────────────────────────────
   1 │<PHONE_NUMBER>281385888  -1.12529
   2 │<PHONE_NUMBER>502565167   0.786904
   3 │<PHONE_NUMBER>104395244   0.338386

Still missing tests as suggested in the review.

Still missing tests as suggested in the review.

I am not exactly sure, how to add that.

You can copy the example that was failing in our test/transforms/zscore.jl file and add check the expected result. You can find examples in the file. After you add these, GitHub will run them automatically to make sure that the tests are passing in all platforms.

and add check the expected res

a=rand(Int,3)
b=rand(3)
t = Table(; a, b)
T = ZScore(:b)
n, c = apply(T, t)
@test isapprox(mean(n.b), 0; atol=1e-6)
@test isapprox(std(n.b), 1; atol=1e-6)
tₒ = revert(T, n, c)
@test Tables.matrix(t) ≈ Tables.matrix(tₒ)

Will in this way work?

That should work 👍🏻
Em sáb., 10 de dez. de 2022 02:42, vickydeka @.***>
escreveu:

and add check the expected res
a=rand(Int,3)
b=rand(3)
t = Table(; a, b)
T = ZScore(:b)
n, c = apply(T, t)
@test isapprox(mean(n.b), 0; atol=1e-6)
@test isapprox(std(n.b), 1; atol=1e-6)
tₒ = revert(T, n, c)
@test Tables.matrix(t) ≈ Tables.matrix(tₒ)
Will in this way work?
—
Reply to this email directly, view it on GitHub
https://github.com/JuliaML/TableTransforms.jl/pull/160#issuecomment-1345147036,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZQW3LCX62O72XX5DZ63QLWMQJ35ANCNFSM6AAAAAASZTR6HA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Please add the example that was failing to the test suite so that the issue is not reintroduced.

It is now added.

Codecov Report

Merging #160 (d072705) into master (976ce71) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head d072705 differs from pull request most recent head fe99a29. Consider uploading reports for the commit fe99a29 to get more accurate results

@@           Coverage Diff           @@
##           master     #160   +/-   ##
=======================================
  Coverage   95.15%   95.16%
=======================================
  Files          27       27
  Lines         888      889    +1
=======================================
+ Hits          845      846    +1
  Misses         43       43

Impacted Files
Coverage Δ

src/transforms.jl
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Thank you @vickydeka for the contribution 💯 The tests are failing for reasons that are not related to your changes.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
19 seconds
clair-scan

✅ Succeeded
12 seconds
inspect-image

✅ Succeeded
11 seconds
deprecated-base-image-check

✅ Succeeded
58 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
8 seconds
label-check

✅ Succeeded
8 seconds
optional-label-check

✅ Succeeded
6 seconds
show-summary

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
17 seconds
clone-repository

✅ Succeeded
1 minute
build-container

✅ Succeeded
12 seconds
inspect-image

✅ Succeeded
19 seconds
clair-scan

✅ Succeeded
58 seconds
clamav-scan

✅ Succeeded
11 seconds
deprecated-base-image-check

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
8 seconds
label-check

✅ Succeeded
8 seconds
optional-label-check

✅ Succeeded
6 seconds
show-summary


Publish ocean to npm

Would love to have it in the Registry.

Done: https://github.com/aframevr/aframe-registry/pull/58


Clarification on consent classification and allowable apps

In the User consent setting section, bullet #2 mentions "All users can only consent to apps that were published by a verified publisher and apps that are registered in your tenant" which seems to indicate that apps registered under Azure Active Directory 'App registrations' should allow consent if the permissions needed by those applications fall within the list of permissions classified as "low impact". In my testing I have not found this to be the case. This could be due to several factors including (1) feature not complete due to being in preview (2) bug (3) documentation misunderstanding / clarity needed. My experience has been that I am still prompted for administrator consent even if the permissions are classified as low impact permissions. I can see this as I have enabled the admin consent workflow. In the below screenshot the application on the right is defined in app registrations and the API permissions are shown. On the left, the consent and permissions configuration is shown which can be correlated to the API permissions in the app registration to the right.

When attempting to access the application the user is still prompted for administrator consent as shown below.

After granting consent the user is no longer prompted. I can also grant consent for the app registration screen and the user does not have to provide consent which is expected behavior.
I would like to better understand how users can consent to "apps that are registered in your tenant" under this scenario. Is it a requirement for permissions classifications to be in effect that the applications be verified no matter if they are internal applications or not? If so that may be the missing clarify needed in the documentation.
The ASP.NET application used to authenticate with the aforementioned app was the application from the Quickstart: Add Microsoft identity platform sign-in to an ASP.NET web app
Thanks

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

ID: c7a6c58c-f887-03a4-8f69-eb91a19e4fb3
Version Independent ID: 4e53d964-7edf-5668-b976-75fcdce4638e
Content: Configure how end-users consent to applications using Azure AD
Content Source: articles/active-directory/manage-apps/configure-user-consent.md
Service: active-directory
Sub-service: app-mgmt
GitHub Login: @kenwith
Microsoft Alias: kenwith

@duwood  Thanks for the question! We are investigating and will update you shortly.

@duwood , Thanks for bringing our attention to this. I can confirm that you observation is correct by reproducing a basic sign-in scenario at my end. I created a single tenant application within my org, but did not grant admin consent to basic sign in permissions. I went on to classify those as low impact and also set the second option from user consent settings. But I still see the admin consent required when signing in where I would expect the user to be able to give consent themselves.

To summarize, the mention of the part "..apps that are registered in your tenant" in the below statement in the document seems to be not working as expected.

All users can only consent to apps that were published by a verified publisher and apps that are registered in your tenant. Users can only consent to the permissions you have classified as "Low impact"

While both consent settings and permission classifications are in preview currently, we appreciate your valuable feedback.
#reassign:@kenwith

GitHub user krish-gh has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#reassign:@krish-gh

GitHub user krish-gh has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

@duwood Can you try classifying the permission email (view the signed-in user's email address). This is most likely what is tripping the "you can't consent" logic.

#reassign @psignoret

GitHub user krish-gh has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

@krish-gh If you're still able to reproduce this issue, please share the troubleshooting details of the sign-in attempt where consent was not allowed (click the "..." in the bottom-right of the sign-in page at the "Approval required" prompt and copy and share the correlation ID and timestamp). Alternatively, please open a support case, so that the support engineer can walk you through the troubleshooting steps.

I am going to close this out as a doc issue but feel free to follow up in this thread is something needs updated.
#please-close


Rails.cache.write(key, value, { compress: true }) doesn't work

I have a question.
config.cache_store = :dalli_store, { compress: true }

This works well, but the following doesn't work. The value is not compressed.
Rails.cache.write(key, value, { compress: true })

Is this expected behavior?
I would like you to make the compress option enabled when calling Rails.cache.write.
https://github.com/mperham/dalli/blob/master/lib/dalli/server.rb#L416
-       if @options[:compress] && value.bytesize >= @options[:compression_min_size] &&
+       if (@options[:compress] || options[:compress]) && value.bytesize >= @options[:compression_min_size] &&

Based on the Rails docs here - http://api.rubyonrails.org/classes/ActiveSupport/Cache/Store.html - I think this is an appropriate change.  Can you put together a PR?

Thank you for your comment. I created PR. Please review following.
https://github.com/mperham/dalli/pull/536

PR merged.  Thanks!


Dataimporter

The dispatcher can process the .txt, .csv, .tsv, .josn file now, and it
doesn't decide file format by its suffix, but head row.

@Wentao-Xu,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

                if (cDoubleQuotes == 1 && curIndex != beginIndex)

What this check is doing?

Refers to: samples/DataImporter/GraphEngine.DataImporter/DsvImporter.cs:103 in 7bd8fa9. [](commit_id = 7bd8fa9ded6debaf75c68233c231f1b45531d843, deletion_comment = False)

:clock1:

@Wentao-Xu, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot


i think pagination should be implemented

What would you like to share?
In the view projects page pagination should be there because as the more projects will got added only number of projects should available for a page.
Screen shot

I would love to implement it
Additional information
No response

okay @777Prabhjot Go ahead, implementing pagination is a great idea


htmlWebpackPlugin

URIError: Failed to decode param '/%7B%=o.htmlWebpackPlugin.files.css[css]%20%%7
D'
at decodeURIComponent (native)

can you download latest

this should be resolved in the latest changes. feel free to open the issue again if you still have a problem

@gdi2290
I still have this issue.
and also have this message in the top of windows
ERROR in ./~/zone.js/lib/patch/geolocation.ts
(4,14): error TS2339: Property 'navigator' does not exist on type 'Global'.
ERROR in ./~/zone.js/lib/core.ts
(122,14): error TS2339: Property 'zone' does not exist on type 'Global'.

@gdi2290
I can't reopen this issue.

@gdi2290
ups, i forgot to update index.html. thanks it works


Job Queue Dispatcher - OnAfterHandleRequest

Business Central 19 - dk:
we used subscriber to the publisher OnAfterHandleRequest (codeunit 448 "Job Queue Dispatcher")
It's now obsolete, but I can't use the new OnAfterSuccessHandleRequest publisher at all.
from the code:

I expected I could use new publisher instead of the Obsolete one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Availability update: We will publish a fix for this issue in the next update for release 19.


Correct hyperlink to accounts console

The underlying URL for all the accounts console hyperlinks needs to be updated in this doc
From
https://accounts.cloud.azuredatabricks.net
To
https://accounts.azuredatabricks.net

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

ID: 13716d08-c61a-ffba-47a7-07652d3e9fd1
Version Independent ID: 13716d08-c61a-ffba-47a7-07652d3e9fd1
Content: Manage identities in Unity Catalog - Azure Databricks
Content Source: databricks/data-governance/unity-catalog/manage-identities.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

Thanks for providing feedback that helps improve our documentation. This issue has been resolved and this outdated content has been removed. We are closing the issue.
#please-close

#please-close


Need [NotNull] for all TextWriter  parameters in RazorPage`

RazorPage does some null checks in various property getters.  however most methods do not check their parameters.  the most obvious examples are methods that have TextWriter parameters -- none check the value before using it.
suspect methods that have a RenderAsyncDelegate parameter also need [NotNull] for them.

fixed in commit e9f5605


wrong slave db choice during filesave

Getting problem on this peace of code
        user.picture.save(
                filename,
                ContentFile(content)
                )

problem in
LogEntryManager.log_create
            # save LogEntry to same database instance is using
            db = instance._state.db
            return self.create(**kwargs) if db is None or db == '' else self.using(db).create(**kwargs)

when I'm setting db = None it starts working fine
2018-08-19 02:20:30,728 django.request ERROR Internal Server Error: /rest/v1/my/profile/picture-upload/a
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/django/db/backends/utils.py", line 85, in _execute
    return self.cursor.execute(sql, params)
psycopg2.InternalError: cannot execute INSERT in a read-only transaction

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/django/core/handlers/exception.py", line 35, in inner
    response = get_response(request)
  File "/usr/local/lib/python3.6/dist-packages/django/core/handlers/base.py", line 128, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/usr/local/lib/python3.6/dist-packages/django/core/handlers/base.py", line 126, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/local/lib/python3.6/dist-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/django/views/generic/base.py", line 69, in view
    return self.dispatch(request, *args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/rest_framework/views.py", line 483, in dispatch
    response = self.handle_exception(exc)
  File "/usr/local/lib/python3.6/dist-packages/rest_framework/views.py", line 443, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/usr/local/lib/python3.6/dist-packages/rest_framework/views.py", line 480, in dispatch
    response = handler(request, *args, **kwargs)
  File "/opt/lonje/django/lonje/account/rest_api.py", line 1451, in patch
    data = self.perform_save(request, filename, content)
  File "/opt/lonje/django/lonje/account/rest_api.py", line 1429, in perform_save
    ContentFile(content)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/fields/files.py", line 93, in save
    self.instance.save()
  File "/opt/lonje/django/lonje/account/models.py", line 381, in save
    ret = super(User, self).save(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/django/contrib/auth/base_user.py", line 73, in save
    super().save(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/base.py", line 729, in save
    force_update=force_update, update_fields=update_fields)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/base.py", line 754, in save_base
    update_fields=update_fields,
  File "/usr/local/lib/python3.6/dist-packages/django/dispatch/dispatcher.py", line 178, in send
    for receiver in self._live_receivers(sender)
  File "/usr/local/lib/python3.6/dist-packages/django/dispatch/dispatcher.py", line 178, in <listcomp>
    for receiver in self._live_receivers(sender)
  File "/usr/local/lib/python3.6/dist-packages/auditlog/receivers.py", line 46, in log_update
    changes=json.dumps(changes),
  File "/usr/local/lib/python3.6/dist-packages/auditlog/models.py", line 63, in log_create
    return self.create(**kwargs) if db is None or db == '' else self.using(db).create(**kwargs)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/query.py", line 417, in create
    obj.save(force_insert=True, using=self.db)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/base.py", line 729, in save
    force_update=force_update, update_fields=update_fields)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/base.py", line 759, in save_base
    updated = self._save_table(raw, cls, force_insert, force_update, using, update_fields)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/base.py", line 842, in _save_table
    result = self._do_insert(cls._base_manager, using, fields, update_pk, raw)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/base.py", line 880, in _do_insert
    using=using, raw=raw)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/manager.py", line 82, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/query.py", line 1125, in _insert
    return query.get_compiler(using=using).execute_sql(return_id)
  File "/usr/local/lib/python3.6/dist-packages/django/db/models/sql/compiler.py", line 1283, in execute_sql
    cursor.execute(sql, params)
  File "/usr/local/lib/python3.6/dist-packages/django/db/backends/utils.py", line 68, in execute
    return self._execute_with_wrappers(sql, params, many=False, executor=self._execute)
  File "/usr/local/lib/python3.6/dist-packages/django/db/backends/utils.py", line 77, in _execute_with_wrappers
    return executor(sql, params, many, context)
  File "/usr/local/lib/python3.6/dist-packages/django/db/backends/utils.py", line 85, in _execute
    return self.cursor.execute(sql, params)
  File "/usr/local/lib/python3.6/dist-packages/django/db/utils.py", line 89, in __exit__
    raise dj_exc_value.with_traceback(traceback) from exc_value
  File "/usr/local/lib/python3.6/dist-packages/django/db/backends/utils.py", line 85, in _execute
    return self.cursor.execute(sql, params)
django.db.utils.InternalError: cannot execute INSERT in a read-only transaction

I am closing this issue due to inactivity.


WebAlert to become a BibTask

Originally on 2012-05-09
Currently the WebAlert engine must be scheduled as a cron job. It should be possible to schedule it as a regular BibTask.

Alert engine will have to be re-implemented for Invenio 3 possibly using Elastic percolators.


Uncaught SyntaxError: Unexpected token 'export' (at spin.js:82:1)

I am getting this error immediately on import, which makes the library non-functional.

me too

You should add type="module" when you're loading your script:
<script type="module" src="path/to/spin.js"></script>


improve integration test

This test would have failed in one year, I forgot to pass the tax year. Also removes some redundancy.

Good spot! Thanks!


Update prerequisites version please

Hi @smicyk How are you. Happy 2024!
I'm setting up my groovy-jmeter again, is it possible to bring the prerequisites versions to the following pls?

groovy @4.0.17
gradle @8.5

These are the versions my MacPorts reports currently, which can save me lots of struggle going back to old versions.
UPDATE:
Hmm... I cannot remember why the versions have to be match, but I do remember there has been a time when things were not working when the versions were not matched. Ah, found it https://github.com/smicyk/groovy-jmeter/issues/52#issuecomment-1022662138.
Thanks

but I do remember there has been a time when things were not working when the versions were not matched.

yep, duplicated:
$ groovy example.groovy --help
org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
General error during conversion: Conflicting module versions. Module [groovy-xml is loaded in version 4.0.17 and you are trying to load version 3.0.17

groovy.lang.GroovyRuntimeException: Conflicting module versions. Module [groovy-xml is loaded in version 4.0.17 and you are trying to load version 3.0.17
	at org.codehaus.groovy.runtime.metaclass.MetaClassRegistryImpl$DefaultModuleListener.onModule(MetaClassRegistryImpl.java:524)
. . .

Hi @AntonioSun, Quite fine thanks.
Regarding your request the general rule is that I stick to versions which are used by jmeter. So currently it is groovy 3.x and I'm not sure if it is possible to update it right now without breaking some stuff on jmeter side. I will have a look and see if there are any problems.
I'm pretty sure that you can use gradle 8.5 in your project since the gradle requirement is only valid when you build from source.
The good thing is that there is new version of jmeter 5.6.3 which update gradle builds to 8.5 so I will update groovy-jmeter soon to use that version. Also they have plans to upgrade groovy to 4.x in next major release hopefully it will be soon.
If possible you can try to run in docker either as gradle project or just normal groovy script. Then you don't have to worry about versions. In the examples folder you can find some examples :).
Hope that helps.

Oh, sure, I fully believe in your choice, which is very reasonable and convincing. will do. thx
BTW, I'm closing this for now, but please drop here a line when you've done the update. thx


[wip] Remove Project.LiveStream

When we initially began integrating the LiveStream framework we were receiving some information about live streams along with the Project fetch JSON payload. All that this contained was the live stream eventId, a title, startDate, etc. We then, upon launching the LiveStreamContainerViewController or LiveStreamCountdownViewController, would perform an additional fetch for the LiveStreamEvent which would tell us everything else about the event.
This quickly turned out to be a bad idea and there was just too much information being duplicated between Project.LiveStream and LiveStreamEvent so the decision was taken to remove Project.LiveStream in its entirety and to fetch a list of LiveStreamEvents from a new endpoint based on the Project's ID. We perform this fetch when we land on a project page and once the list is returned we reload the project's subpage cells with any live streams that may have been returned.
The amount of files that this touched is a lot more than I'd anticipated! But it's much simpler this was and it should also prepare us for a lot the work that we would have needed to do for discovery.
Things I've done here:

Removed Project.LiveStream from models and tests that referenced it.
Removed fetching of LiveStreamEvent from LiveStreamEventDetailsViewModel.
Changed all initialisers that previously allowed an optional LiveStreamEvent to now expect this as non-optional.
Updated Koala tracking tests. @mbrandonw you'll want to confirm that I correctly understood how your tests were working around the timeout of fetching live streams and tracking that, etc. I believe it's all still working correctly.
Updated all view model tests that had been referencing Project.LiveStream.
Copied across the changes @mbrandonw had made to LiveStreamEvent for the live stream discovery work. This was necessary to parse the new endpoint @Jwomers had set up for us.

Things to review:

Are the tests still sound?
Should I add some more tests?
Is there a clever way to reduce the amount of emissions when the project page loads now that it's a combineLatest of project, freshProject and liveStreams?

Things still needed:

In testing I wasn't seeing any user subscription info, Firebase paths, OpenTok info, replay URLs, etc. from this endpoint. Will you still be adding those @Jwomers?
May still want to write more tests for ProjectPamphletViewModelTests and confirm that we're still satisfied withProjectPamphletContentDataSourceTests (these should be fine).
Kind've regret not changing LiveStreamEvent.startDate to a TimeInterval as I went through this 😐

These tests will pass once #67 is merged.


Move this sample into ruby-docs-samples

To reduce maintenance and not have to duplicate issues / etc across all Ruby samples that are used in or referenced by our cloud.google.com documentation, this sample application should be moved into ruby-docs-samples
See tracking issue in ruby-docs-samples repo: ...
Many of the issues we file are for things such as "update all Ruby samples to use the latest X.Y.Z version of the ABC library."  This type of maintenance is more difficult with our samples in 2 repositories.  It's easy to forget about this repository ... plus you have to git grep across both.  Yuck!
We also will be making it simpler to run end-to-end tests for all samples found in ruby-docs-samples and having these Rails applications as part of that test pipeline will simplify things greatly!
Before this can be closed and the repo completely moved ...

[ ] Ensure that all documentation is updated to use the Bookshelf sample found in ruby-docs-samples
[ ] Ensure that all interactive tutorials are updated to use the Bookshelf sample found in ruby-docs-samples

Moved to buganizer (id 109937659). Closing.


Write to current file when file not specified

I enjoy using this plugin to quickly add entries to my daily note during the day. However, as a college student, sometimes my days go beyond midnight! Once into the next day, supplying daily=true to the file identification creates the next day's daily note, something that I don't want to happen until I sleep (past midnight) and wake up (the same day).
I noticed that the find and replace functionality didn't need a file identification, instead using the current open file if no identification is provided. Noticing this, we bring in the same logic into the handleWrite() function.
Tested appending on Windows and iOS.

Thanks for the contribution!


Unverified InsecureRequestWarning Python3 SDK call for

Got this warning for python3 when running the sample code
https://github.com/deep-security/automation-center-sdk-samples/blob/master/python/src/computer_status_examples.py
/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/urllib3/connectionpool.py:847: InsecureRequestWarning: Unverified HTTPS request is being made. Adding certificate verification is strongly advised. See: https://urllib3.readthedocs.io/en/latest/advanced-usage.html#ssl-warnings
As Security tool, shouldn't it be fixing ssl request warnings?
Disabling the warning is not a great solution in my opinion
urllib3.disable_warnings(urllib3.exceptions.InsecureRequestWarning)

@kaido89 Thank you for raising this issue. I agree that disabling the warning should not be done by default. I have commented out that line of code in Main.py.
Thanks for using the samples!


[Event Change] CU90 "Purch.-Post" - procedure CheckAndUpdate()

------------------ Change IntegrationEvent Attribute ----------------------
IncludeSender : true, GlobalVarAccess : true
-------------------------------------------------------------------------------

[IntegrationEvent(TRUE, TRUE)]
local procedure OnCheckAndUpdateOnBeforeCheckAssociatedOrderLines(var PurchaseHeader: Record "Purchase Header")
begin
end;

GlobalVarAccess was deprecated in 16.x, there is concept of protected var instead. You can list here the global vars that have to be protected and accessible from extensions.
Please put the code surrounding the place, where you expect a new event call, as a text not as a screenshot.
It helps us finding the right place quickly if you put enough code to identify the unique place.


Dynamically Show the information about the members of ICS.

Presently, the information about the faculty members and other members of the ICS have been hardcoded. The information may change after one sometime(maybe the faculty members change or the students coordinating may change.) and it may be feasible if we show them dynamically.

I would like to propose fetching the information from firebase and rendering the same dynamically.

@riteshsp2000
We will have API for these, isn't it?
Or Firebase?

Why did you close this? 😂
This is a feature that needs to be done at some point


Mapper Lifting with Connectivity Test (Pointcloud to Graph)

Code for loading and generating point clouds is adapted from #34.

Codecov Report
Attention: Patch coverage is 84.34783% with 18 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@497947b). Learn more about missing BASE report.

Files
Patch %
Lines

modules/data/utils/utils.py
56.75%
16 Missing :warning:

...nsforms/liftings/pointcloud2graph/cover_lifting.py
96.96%
2 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage        ?   68.15%
=======================================
  Files           ?       19
  Lines           ?      603
  Branches        ?        0
=======================================
  Hits            ?      411
  Misses          ?      192
  Partials        ?        0

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

Hello @pzajec! Thank you for your submission. As we near the end of the challenge, I am collecting participant info for the purpose of selecting and announcing winners. Please email me (or have one member of your team email me) at<EMAIL_ADDRESS>so I can share access to the voting form. In your email, please include:

your first and last name (as well as any other team members)
the title of the method you implemented
the input domain of the method you implemented
the output domain of the method you implemented
your pull request number (#67)

Before July 12, make sure that your submission respects all Submission Requirements laid out on the challenge page. Any submission that fails to meet this criteria will be automatically disqualified.


Fix access error for torch.mps when using torch==1.13.1 on macOS

What does this PR do?

When I run accelerate using torch==1.13.1 on my macbook, I get:
        elif is_mps_available():
>           torch.mps.empty_cache()
E           AttributeError: module 'torch' has no attribute 'mps'

You can see the torch.mps module and the functions backing torch.mps.empty_cache() and torch.mps.current_allocated_memory() were not added until torch 2.0.0, whereas MPS support was added in 1.12.1:
https://github.com/pytorch/pytorch/blob/v1.13.1/torch/mps/__init__.py
https://github.com/pytorch/pytorch/blob/v1.13.1/torch/_C/__init__.pyi.in
So it is not safe to just use is_mps_available() to know if you can access torch.mps, you actually have to check if it exists before accessing it. There are plenty of other places where is_mps_available() is used and torch.mps is not accessed, so I don't think we should change its definition.
Ideally, accelerate would commit to running at least one 1.13.1 mac CI build, although I know it may not be feasible. The conditions I added appear to cover every access of torch.mps, but without the build, it seems likely there will be additional regressions.
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[x] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a Github issue or the forum? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines, and
here are tips on formatting docstrings.
[ ] Did you write any new necessary tests?

Who can review?

@muellerzr
@amyeroberts

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

Hi @ringohoffman, are you planning to finish the PR ? Otherwise, I can take over !

@SunMarc you are welcome to!

Closing this PR since it has been solved in the PR above.


Remove rack_csrf dependency

dry-web-roda's dry_view roda plugin has a dependency on rack_csrf gem. When using dry-web-roda as an API server, csrf is not needed. Better if this csrf dependency is removed.

Thanks @musaffa — I'll move this into the generated Dry::Web::Roda::Application subclasses instead. I'll try and do this sometime later this week.

An API server may need dry_view roda plugin as well e.g. for mailing purposes. It just don't need csrf and flash as view context options.

API server could also use dry-view with jbuilder or something.


Add the select language, compiler and flags from command

Currently the plugin opens a fuzzy finder for each step, but it would be nicer to have the ability to pass all parameters from command window. This would allow to avoid selecting the same stuff when you changing flags, for example. Neovim supports custom commands completion, so using it would be very convenient.

Hi. I've been thinking about adding this, but I wanted to first prioritize the interactive approach implementation. My rationale was that vim.ui.select|input could be the difference maker between this neovim plugin and the other editor plugins found here. I'll look into it.

This should work now. Let me know how you feel about the current form.
:CECompile compiler=g121 flags=-O2 intel=false

I will also add command completion for the compiler ids in a future commit.

Thank you! I like this feature.


Set the 1.13 in-development version to 1.13.6

DCOS-58190 - Set DC/OS version of 1.13 to 1.13.6-dev

@mesosphere-mergebot label ready for review

@mesosphere-mergebot merge-it


Change bad characters default value to empty list in find-gadget.py

This supersedes Pull Request #20.
This changes the default value of the -b/--bad-chars flag to an empty
list (previously ["00"]).
Prior to this patch, specifying an empty bad characters list was not possible because -b expected at least one argument. Specifying -b "" would yield wrong results. For example:
$ python3 find-gadgets.py -f QuoteDB.exe  -b ""

...
[+] running '/home/kali/.local/bin/rp-lin-x64 -r5 -f QuoteDB.exe --unique --bad-bytes=\x'
...

$ wc -l found-gadgets.txt
1032 found-gadgets.txt

Note that rp-lin-x64 was invoked incorrectly with --bad-bytes=\x and only found 1032 gadgets. However, with this patch applied, 3081 gadgets are found:
$ python3 find-gadgets.py -f QuoteDB.exe

[+] running '/home/kali/.local/bin/rp-lin-x64 -r5 -f QuoteDB.exe --unique'

$ wc -l found-gadgets.txt
3081 found-gadgets.txt

The QuoteDB.exe binary is from https://github.com/bmdyy/quote_db

looks good, appreciate it! merging


Allowing name to be overridden via sky-pages.json

Fixes https://github.com/blackbaud/sky-pages-cli/issues/22

Current coverage is 100% (diff: 100%)

Merging #11 into master will not change coverage

@@           master   #11   diff @@
===================================
  Files          14    14
  Lines         269   272     +3
  Methods        24    24
  Messages        0     0
  Branches       32    33     +1
===================================
+ Hits          269   272     +3
  Misses          0     0
  Partials        0     0

Powered by Codecov. Last update c85f024...c9feb41