Images intermittently disappear on Android

[x] Review the documentation: https://facebook.github.io/react-native
[x] Search for existing issues: https://github.com/facebook/react-native/issues
[x] Use the latest React Native release: https://github.com/facebook/react-native/releases

Images intermittently disappear on Android
Several issues reported in https://github.com/facebook/react-native/issues/13600 which was closed due to being stale but the issue still persists
Using RN 0.55 (can't use 0.56 due to babel 7 and jest mock incompatibilities)
Note / Possible Workaround
In my testing, this issue only occurs when importing images like:
<Image source={require('./my-icon.png')} />

But if I import via:
import myIcon from './my-icon.png';

then it works fine.
See the blog post Quick Tip: Images in React-Native on Android Not Loading for more on this.
Environment
Scanning folders for symlinks in /Users/aaronbrager/[REDACTED]/node_modules (17ms)
Environment:
OS: macOS 10.14
Node: 10.8.0
Yarn: 1.9.4
npm: 6.2.0
Watchman: 4.9.0
Xcode: Xcode 9.4.1 Build version 9F2000
Android Studio: 3.0 AI-171.4443003
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.4 => 0.55.4
[skip envinfo]
Possible cause
fresco has a cache pool for decoded bitmap, with a default size determined by the available runtime memory:
https://github.com/facebook/fresco/blob/3ccf021877571cc66c1cf28ecc82788f83579ad3/imagepipeline/src/main/java/com/facebook/imagepipeline/memory/BitmapCounterProvider.java#L32-L39
When the amount of memory is insufficient it throws an exception:
https://github.com/facebook/fresco/blob/8731cca73e5a4bd44387657dacd1f4562e549854/imagepipeline/src/main/java/com/facebook/imagepipeline/memory/BasePool.java#L237-L245
When this happens, React Native eats the exception and fails silently (see line 71):
https://github.com/facebook/react-native/blob/5c2720b089dd0fb277771b0ac8b0e7788c493e5d/ReactAndroid/src/main/java/com/facebook/react/modules/fresco/SystraceRequestListener.java#L45-L82
Instead of failing silently, React Native should do something (increase memory, free resources, clear cache… not sure) to ensure that the image being rendered appears.
Initial research done by @fatfatson (see https://github.com/facebook/react-native/issues/13600#issuecomment-370754425)

As noted above I can’t update, but the issue I highlighted in the RN code base still appears in the latest build

@hramos is there anyone you know of who worked on the RN/fresco integration who could look at this and at least tell me if I'm onto something?

It doesn't work for me

Hi folks, Anyone have idea to resolve this issue

Hi, I'd like to add some evidence to this issue.
I'm developing an app made with React Native that uses React Navigation to move between views. On one view I have a list of images which are local assets, called with require.
Navigating into the view with the list of images one time was fine, return to the previous view then going again to the list of images made that some of them disappear.
This is the sequence:
View 1 -> Ok
View 2 -> Ok
View 1 -> Ok
View 2 -> Fail
View 1 -> Fail
I was thinking of giving up and even started another version with Flutter, then I review the code that I wrote and found that to go back to the previous view I was doing this from the second view:
this.props.navigation.push('View1') instead of this.props.navigation.goBack()
Once I change it to goBack() the problem disappear. According to the React Navigation documentation the goBack() function hooks into the hardware back button.

On Android, React Navigation hooks in to the hardware back button and fires the goBack() function for you when the user presses it, so it behaves as the user would expect.

My problem was clearly related to the React Navigation implementation, I hope this can help to solve this issue.
This are the dependencies use in my project:
"dependencies": {
    "@sentry/browser": "^4.4.1",
    "firebase": "^5.7.0",
    "lodash": "^4.17.11",
    "react": "16.6.1",
    "react-native": "0.57.7",
    "react-native-device-info": "^0.24.3",
    "react-native-gesture-handler": "^1.0.10",
    "react-native-image-progress": "^1.1.1",
    "react-native-linear-gradient": "^2.5.3",
    "react-navigation": "^3.0.7",
    "uuid": "^3.3.2"
  },
  "devDependencies": {
    "babel-jest": "23.6.0",
    "jest": "23.6.0",
    "metro-react-native-babel-preset": "0.50.0",
    "react-test-renderer": "16.6.1"
  }```

Hello there 👋 this issue has been reported for an old version of React Native. Ideally we'd like everyone to be using 0.59 (see the awesome changes it brought) but we know updating can be a pain. We are going to close this issue because it's from a version before 0.57, which is really old.
But please, if it's actually still an issue with 0.59 please comment below and we can reopen it. Even better, please send us a pull request with a fix 😊

I just ran into this issue using 0.59. In fact, even Text components won't render, likely due to memory issues.


[Bug] Select2 on relationship field (PRO) is squished

Bug report
What I did
I added a CRUD::field('users')->type('relationship') to a field in a setupCreateOperation() function.
What I expected to happen
It would always display like this:

What happened
90% of the time, it displays like this:

What I've already tried to fix it
Nothing, really. It seems to be caused by some sort of race condition. To "fix" it, I refresh until I see it correctly.
Is it a bug in the latest version of Backpack?
Yes, upgraded to 5.0.18. backpack/pro is 1.0.17.
Backpack, Laravel, PHP, DB version
### PHP VERSION:
PHP 8.1.5 (cli) (built: Apr 21 2022 10:15:06) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.5, Copyright (c) Zend Technologies
    with Zend OPcache v8.1.5, Copyright (c), by Zend Technologies
    with Xdebug v3.1.2, Copyright (c) 2002-2021, by Derick Rethans

### LARAVEL VERSION:
v9.11.0@598a8c84d452a66b90a3213b1d67189cc726c728

### BACKPACK VERSION:
5.0.18@06ea421e44c353604ba56669e1fd3ac7a18981df

Damn! Thanks for reporting this @GRardB . I wasn't able to get the same thing though - it shows ok to me. Can you give us a bit of context please?

What browser are you using?
Is it in a page with a lot of fields? If so, it might take a 1-2 seconds to unsquish, does "time fix it"?
Can you give us a screenshot/video of it happening with the full form page, maybe that gives us some more hints?
Any JS errors?

Thanks again. Cheers!

Damn! Thanks for reporting this @GRardB . I wasn't able to get the same thing though - it shows ok to me. Can you give us a bit of context please?

What browser are you using?
Is it in a page with a lot of fields? If so, it might take a 1-2 seconds to unsquish, does "time fix it"?
Can you give us a screenshot/video of it happening with the full form page, maybe that gives us some more hints?
Any JS errors?

Thanks again. Cheers!

Thanks for the quick Response, @tabacitu.

Browser: I'm on the latest Chrome on macOS 12.2.1. I tried reproducing in Firefox and Safari, but those seem to work fine. I tried disabling all my Chrome extensions, but that didn't fix the issue.
Fields: Only two fields in total.
Time: Waiting doesn't seem to fix it, unfortunately.
Screenshot: attached below!
JS Errors: None

@GRardB hi there.
I recall sometime ago we had style="width:100%" hardcoded in the select. It was removed because aparently it was not needed ... but maybe it was there to prevent that "race condition" when it happens (it just not happen that ofter, or happen more for some users than others). I've never seen that error myself ... but hey, here we are and you are getting the error.
Can you go into vendor/backpack/pro/resources/views/fields/relationship/relationship_select.blade.php and add on the <select tag (around line 60) the hardcoded style="width:100%" ? I am not sure if you get the error consistently, if you do it should be easy for you to check if the solution works.
I would test this myself but I am like on hundreds of refreshes and can't get the error to appear on my side not even once :| I am also using Chrome but on windows.
Let me know,
Pedro

@GRardB hi there.
I recall sometime ago we had style="width:100%" hardcoded in the select. It was removed because aparently it was not needed ... but maybe it was there to prevent that "race condition" when it happens (it just not happen that ofter, or happen more for some users than others). I've never seen that error myself ... but hey, here we are and you are getting the error.
Can you go into vendor/backpack/pro/resources/views/fields/relationship/relationship_select.blade.php and add on the <select tag (around line 60) the hardcoded style="width:100%" ? I am not sure if you get the error consistently, if you do it should be easy for you to check if the solution works.
I would test this myself but I am like on hundreds of refreshes and can't get the error to appear on my side not even once :| I am also using Chrome but on windows.
Let me know, Pedro

Hey @pxpm , that does indeed fix the issue!
I did a little more testing, and I'm seeing that the field isn't always race-conditioning to the same width. After a bunch of refreshing, here are some different sizes that I ended up seeing without the fix applied (same window size and everything):

Not sure if that helps, but I figured it was worth mentioning

Hey @GRardB thanks for quick reply and such good explanation.
I guess it's because at the time of draw it calculates the available space and set the width accordingly. Dunno why size differs and other times it fully works. I think I really don't want to go into that rabbit hole, just find the fix for it and be done with.
I will check the best way to fix it, since I think something like this had been reported for other fields that use the select2 plugin, but we never reached a conclusion.
Thanks again for report and confirming the solution.
Cheers

Guys - we just release backpack/pro 1.0.19 which should fix this. We ended up reverting the inline width:100% so that it's non-breaking. No need to publish assets or anything else. After you run a composer update backpack/pro you should be getting 1.0.19 which has this fixed. Can anyone please confirm?
Thanks and sorry 🙏
Cheers!

I can confirm it's fixed! Thanks for the super quick work on this! And no need to apologize 😁

Thanks @GRardB for the help here!
Cheers 🍻


Adding some change to readme file

This is the change I added due to fix needed for a specific  bug created in the request.

Rejected


Redirection from Cloudflare Waiting Room is blocked

The site I'm trying to explore with Puppeteer is protected by a Cloudflare page that analyzes the browser. It appears that there is some problem when using GoToAsync - the Cloudflare waiting room is endlessly updating, but does not block access. As soon as I close the application (= close the CDP session), the redirection is immediately completed successfully. If I open the tab manually and type in the URL, the check is also successful. Either Cloudflare somehow understands that the browser is automated, or there is a problem in Puppeteer. How can we understand this?
In addition, if you create a new tab and load the page, then continue execution from the line IPage page = await PuppeteerHeadful.NewPageAsync();, the check will be successful.
await new BrowserFetcher().DownloadAsync();
PuppeteerHeadful = await Puppeteer.LaunchAsync(new LaunchOptions {
    Headless = false,
    Devtools = false,
    Browser = SupportedBrowser.Chrome,
    DefaultViewport = null,
    Args = new[] {
        "--disable-blink-features=AutomationControlled"
    }
});

IPage page = await PuppeteerHeadful.NewPageAsync();
await page.GoToAsync(url);

URL for example: https://www.vindecoderz.com/EN/check-lookup/WVWMP7AN2BE710202
.NET version: 6.0
PuppeteerSharp version: 14.0.0

There are some conversations upstream about that.

@kblok Found this discussion earlier. I think I have a different problem because I don't see any warnings from Cloudflare.


Make Request Context Accessible To Logger

What am I trying to do?
Using express-errorhandler with concurrency-logger, which needs access to the request context.
log/log.error is exposed in res.locals.
app.use(errorHandler({
  logger: (err, req, res) => res.locals.log
});

What do you think, is this use case too specific? Then I'd just go ahead and write a separate error handler.
The changes are backwards compatible.
@gustavnikolaj  @joelmukuthu

Coverage increased (+0.7%) to 80.702% when pulling 326a89f8fe2c54d74329e3441af60289bf0483a4 on feature/logger-request-context into f83c499a47f784b2036aed9963c02d633eb68949 on master.

Coverage increased (+0.7%) to 80.702% when pulling 97a2f4027d92c1895d8bbf654129d3dda7718ffc on feature/logger-request-context into f83c499a47f784b2036aed9963c02d633eb68949 on master.

Coverage decreased (-0.3%) to 79.661% when pulling bf0d5554271e07c093dee063cea28ad94ad8de55 on feature/logger-request-context into f83c499a47f784b2036aed9963c02d633eb68949 on master.

Coverage decreased (-0.3%) to 79.661% when pulling bf0d5554271e07c093dee063cea28ad94ad8de55 on feature/logger-request-context into f83c499a47f784b2036aed9963c02d633eb68949 on master.

Well, the current usage of logger.error seems to be quite specific to some other usage that someone wanted.  Would you mind doing some extra work so you can use customReporter instead (maybe add a reporter option)? Then (perhaps at a later point) we can find out what logger.error was used for and yank it out.
@papandreou, any thoughts on my ramblings?

Good point, I completely missed that customReporter. Going to update the PR and hook into that :+1:

I agree with @joelmukuthu that customReporter looks more applicable here.
The logger API that we use is based on console, so logger.error logs a message with a severity of "error".

options.reporter is now used to overwrite and customize options.logger with context. Could you revisit the changes?

Why is this better than just using a normal express error handler mounted before this modules error handler?

Because the formatting from express-errorhandler is good and I didn't want to re-implement it, but put the output in context.

I would prefer to separate out the formatting into it's own module, or just as a function that could be used from your module. I think it is a bit much to make this error handler pluggable, as we already have express which is more than capable of providing for this use case :-)

Makes sense and is less inside out :+1: Closing this.


[docker]:Add docker build way.

Problem Summary:
What is changed and how it works?
Proposal: Add docker build way to quick start.
What's Changed:

Add quick start demo on docker.

How it Works:

Build special version by docker compose.

good job.

@Whojohn  The PR need to speed up the progress 💪🏻

@Whojohn The PR need to speed up the progress 💪🏻

Sorry， Too busy to handle my life ,
I forgot it  , i will finish all job before 2022.2.27.

which include:

Docker quick start (All in one dockerfile. Not everyone need dockercompose to split mysql and streamx).
Prebuild docker image in china.net  which  improve experience .


New PR to test SignCLA at 2019-04-01 17:12:36.651290399 -0700 PDT m=+5.683917687

Body of PR

indiv-pass-staging: test succeeded


[Docs] Fix a typo in ${NEW} variable in Post-processing tools for diagnostics section

While reading up I realized there is a typo in: how to use  utils/process-stats-dir.py. The example asks to use OLD and NEW compiler but the documentation uses only ${OLD} compiler in example

None

folks, this is my first (and very tiny) PR; Maintainers please help trigger the PR

⛵

@swift-ci please smoke test and merge

🎉


[backport] tests: move erasure pool testing in lvm_osds

This commit moves the erasure pool creation testing from all_daemons
to lvm_osds so we can decrease the number of osd nodes we spawn so the
OVH Jenkins slaves aren't less overwhelmed when a all_daemons based
scenario is being tested.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>(cherry picked from commit 8476beb5b1f673d8b0925293d9273041c99a9bac)

jenkins test centos-non_container-all_daemons

jenkins test centos-container-all_daemons

jenkins test centos-container-lvm_osds

jenkins test centos-non_container-lvm_osds

jenkins test centos-container-all_daemons

jenkins test centos-container-lvm_osds

jenkins test centos-non_container-lvm_osds

jenkins test centos-non_container-lvm_osds

jenkins test centos-container-lvm_osds

jenkins test centos-container-all_daemons

jenkins test centos-non_container-all_daemons


Can I animate dropdown with animation libraries?

I want to have ability to style dropdown with animations. Is it possible? Currently doesn't work as I cannot change views in renderDropdown func.

Was a workaround ever found for this? Looking at solving the same issue.

pr created to add animation to the dropdown component


Apply unsloth optimizations

⚠ Please check that this feature request hasn't been suggested before.

[X] I searched previous Ideas in Discussions didn't find any similar feature requests.
[X] I searched previous Issues didn't find any similar feature requests.

🔖 Feature description
The project https://github.com/unslothai/unsloth looks very interesting. He claims great speedups for finetuning. He detail the improvements here:
So in GPUs the goal is to saturate the GPU with matrix multiplies instead of data movement. I'll write a more detailed blog but approximately:

1. Flash Attention v2 reduces the time taken by 17% or so

2. RoPE Triton kernels: -7.1%

3. RMS Layernorm in Triton: -3.1%

4. Cross Entropy in Triton: -1%

5. Manual autograd for MLP: -4%

6. Manual QKV autograd: -2%

7. Manual O autograd: -2%

8. Smart cache evictions and reduced data duplications etc: -30%

9. And other tricks in the Max and Pro versions makes it 30x faster

✔ Solution
Would be nice to use their kernels to speedup axolotl
❓ Alternatives
No response
📝 Additional Context
No response
Acknowledgements

[X] My issue title is concise, descriptive, and in title casing.
[X] I have searched the existing issues to make sure this feature has not been requested yet.
[X] I have provided enough information for the maintainers to understand and evaluate this request.

If this is a major request by the OSS community - I'm more than happy to include some of the changes from Unsloth!

I would like to second this request. As I understand it, this is simply free increased efficiency for training with no degradation on accuracy, right? I think this would be a major boost to the Axolotl project.

Yes 0% loss in accuracy - we do actual FLOP reductions via our manual autograd engine. I'm actually working with @casper-hansen and some other Axolotl people to put some methods inside Axolotl!

Legend. Superman has posters of you on his wall. Thanks so much for all of your work!

:)

I tried a few of the optimizations for FFT on Mistral, but I cannot seem to improve it according to the posts. @danielhanchen would be great if you can pitch in with a PR if you have time.
https://github.com/OpenAccess-AI-Collective/axolotl/tree/unsloth_modules

@casper-hansen Oh cool - I'll have a look! Ye I'll try to make a PR to axolotl!!

Hi,
Is there any status on these updates? If I use Axolotl right now, will I benefit from the Unsloth improvements?
Thank you!

@fakerybakery Sorry not yet - I'll take a look at the PR Casper made, but it might take some time

Ok, thank you!

Unsloth is particular interesting if your GPU is not supported by flash attention (e.g., V100).
Unfortunately, as of now, unsloth seems to not have multi-GPU support in the OSS version yet: https://github.com/unslothai/unsloth/issues/107

FYI: gradient checkpointing has been merged: https://github.com/OpenAccess-AI-Collective/axolotl/pull/1528 🎉


Using add-in with existing ArcGIS Pro project at a different scale causes "Spatial Index Grid Size" error

Repro steps:

Create a new Pro project
Zoom into the Monterey Peninsula
Create a line, circle, ring the approximate size of the peninsula
Exit Pro (Save the Project)
Run Pro and open the previously created project
Zoom to world scale
Attempt to create a line at world scale
"Spatial Index Grid Size is Invalid" Error is reported and line can't be created

We have seen this error before in Pro with lpkxs ( https://github.com/Esri/military-features-data/issues/287 ) - but this issue is slightly different, it seems like when you save a Pro project it is setting a spatial index on the GDB, not sure if this is a new issue at Pro 2.3, or we just never noticed previously.
Note: I confirmed that these steps also create the same failure with the Military Symbol Editor (create a Military Line symbol instead of a D&D line)

Need to check on previous version of Pro to see if this is a Pro 2.3 change.

Testing this on 2.4 and I am experiencing some lockups loading the layer package and activating the Map Tool at full extent (I don't see this on my 2.2 machine) - so investigating this

On Pro 2.4.19901 I am seeing an issue where when I try to draw the first graphic, the map zooms to full extent and then when I manually zoom back to the original area, 3 graphics are draw.
@saip can you try this on your latest Pro build?

@saip - @ljuru and I stepped through the repro steps in the original issue on Pro 2.4.19921 on 2 different machines but we are still seeing the lockup issue mentioned previously - can you try this?

Verified that with PR #658 the error is not received - still investigating a possible deadlock issue on 2.4 unrelated to this PR

I was able to reproduce the lockup issue outside of the D&D addin, so I do not believe it is related to our addin and we will pass these repro-steps onto the Pro team (both @dfoll and I repro'd this issue)
Repro Steps:

Open the Pro Package provided in Pro 2.4 (tested on version after 2.4.19921) - "Test-DD-Lockup.ppkx.zip"
Click the Full Extent Button
Click the Insert Tab
The UI behaves strangely, some things work, but the tools are all grayed out and none of the tools can be selected

Core issue logged as https://devtopia.esri.com/ArcGISPro/text/issues/1876

We believe this was fixed in Pro 2.4 final. We believe this only needs to be confirmed as fixed and then listed as an issue addressed

ArcGIS Pro 2.4.0 b1 (19489) + DD build 91

could not reproduce problem with repro steps in the original issue, but could reproduce the problem with the repro steps in this comment

ArcGIS Pro 2.4.0. +DD build 91

certification test updated in this pr

https://solutionsdev.arcgis.com/defense/help/distance-direction/resources/june-2019-distance-and-direction.htm
\...\DefenseSolutions\DistanceAndDirection\Dev\92\

Verified in issues addressed for DD and MT on staging
Verified in PDFs in dev builds of DD 95 and MT 313

this issue may have crept back in for arcgis pro 2.6. have not done a ton of research at this point....info for build that made me think this is <IP_ADDRESS>1 and  ArcGIS.Desktop.Defense built on 24March


bot exits unexpectedly.

I think it's a bug.
2016-08-12 04:39:34,595 [TransferPokemon] [INFO] [pokemon_release] Exchanged Weedle [CP 99] [IV 0.42] for candy.
2016-08-12 04:39:37,177 [       cli] [INFO]
2016-08-12 04:39:37,177 [       cli] [INFO] Ran for 2:35:34
2016-08-12 04:39:37,177 [       cli] [INFO] Total XP Earned: 21660  Average: 8353.90/h
2016-08-12 04:39:37,177 [       cli] [INFO] Travelled 3.48km
2016-08-12 04:39:37,177 [       cli] [INFO] Visited 93 stops
2016-08-12 04:39:37,177 [       cli] [INFO] Encountered 68 pokemon, 66 caught, 68 released, 5 evolved, 0 never seen before
2016-08-12 04:39:37,177 [       cli] [INFO] Threw 79 pokeballs
2016-08-12 04:39:37,177 [       cli] [INFO] Earned 7093 Stardust
2016-08-12 04:39:37,177 [       cli] [INFO]
2016-08-12 04:39:37,177 [       cli] [INFO] Highest CP Pokemon: Exeggcute [CP: 797] [IV: 10/12/12] Potential: 0.76
2016-08-12 04:39:37,177 [       cli] [INFO] Most Perfect Pokemon: Venonat [CP: 490] [IV: 14/14/13] Potential: 0.91
Traceback (most recent call last):
File "pokecli.py", line 559, in
main()
File "pokecli.py", line 96, in main
bot.tick()
File "E:\downloads\PokemonGo-Bot\pokemongo_bot_init_.py", line 468, in tick
if worker.work() == WorkerResult.RUNNING:
File "E:\downloads\PokemonGo-Bot\pokemongo_bot\cell_workers\evolve_pokemon.py", line 38, in work
self._execute_pokemon_evolve(pokemon, cache)
File "E:\downloads\PokemonGo-Bot\pokemongo_bot\cell_workers\evolve_pokemon.py", line 101, in _execute_pokemon_evolve
response_dict = self.api.evolve_pokemon(pokemon_id=pokemon.id)
File "e:\downloads\pokemongo-bot-desktop\gofbot\src\pgoapi\pgoapi\pgoapi.py", line 122, in function
return request.call()
File "E:\downloads\PokemonGo-Bot\pokemongo_bot\api_wrapper.py", line 106, in call
result = self._call()
File "E:\downloads\PokemonGo-Bot\pokemongo_bot\api_wrapper.py", line 63, in _call
return PGoApiRequest.call(self)
File "e:\downloads\pokemongo-bot-desktop\gofbot\src\pgoapi\pgoapi\pgoapi.py", line 220, in call
response = request.request(self._api_endpoint, self._req_method_list, self.get_position())
File "e:\downloads\pokemongo-bot-desktop\gofbot\src\pgoapi\pgoapi\rpc_api.py", line 126, in request
request_proto = self._build_main_request(subrequests, player_position)
File "e:\downloads\pokemongo-bot-desktop\gofbot\src\pgoapi\pgoapi\rpc_api.py", line 216, in _build_main_request
request.auth_info.token.contents = self._auth_provider.get_access_token()
File "D:\p\Python\lib\site-packages\google\protobuf\internal\python_message.py", line 669, in field_setter
new_value = type_checker.CheckValue(new_value)
File "D:\p\Python\lib\site-packages\google\protobuf\internal\type_checkers.py", line 176, in CheckValue
raise TypeError(message)
TypeError: None has type <type 'NoneType'>, but expected one of: (<type 'str'>, <type 'unicode'>)

Please read before you delete the text in the issue template, you're missing key information which would help us to help you

This seems same as #3078
It also happens to me as I posted in #3078


[stable/prometheus-nats-exporter] upgrade to official image

What this PR does / why we need it:

Upgrades the chart to use the official docker image from https://github.com/nats-io/prometheus-nats-exporter/releases
fixed post install notes
fixed some small things in the README
added the other params to the deployment in order to export all metrics (should we made this optional eventually?)

Which issue this PR fixes
none.
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

Great work! Thank you.

/ok-to-test

hmm seems like tests never start... what should I do?

They have been https://prow.k8s.io/view/gcs/kubernetes-jenkins/pr-logs/pull/helm_charts/14528/pull-charts-e2e/1136283679438409728/

/lgtm


Return "Internal Server Error" when an error occurs inside the checker.Check() method in connectivity check handler

Resolved by #6


Skeletal Animation not working with UsdTimelinePlayable example (FTV-725)

Trying to load skeletal animation but why not working? Timeline playing but scene not animated.

@tufeixp which version of the USD package are you using? Are you getting any errors in the console?
I just tried it with the latest USD package version (1.0.3-preview.2) and it looks like we forgot a hard coded path that will need to be adjusted. I've logged it into our bug database for fixing.
In the meantime you should be able to get the sample working with the following steps:

Open the USDTimelinePlayable scene
In the scene hierarchy, select the testRecording GameObject
In the USD asset component, adjust the USD file path to <project_root>\Assets\Samples\USD\1.0.3-preview.2\UsdTimelinePlayable\mesh.usd (changing "preview.1" to "preview.2")

If your problem is something else please provide more details on what you are encountering (ex: console errors, reproduction steps). Thanks!

Yes, I have made the change already. What I'm asking is the skeletal animation playable?

Skeletal animation should be supported with the current USD package. Could you please provide more details on the problem you are having?
Reproduction steps, error logs, screenshots, or specific USD files that are not working would be useful for us to further help you. Thanks!

Try this one:
Taryk.zip
Import it as GameObjects;
Add a playable director component;
Set Bindings;
Drag the timeline cursor => No animation playing.

Are you certain that this file contains animation? I tried opening it in USDview (the official USD file viewer from Pixar) and it doesn't seem to play either.

translation data is varying in the timeline.

Do you see that character moving in the USDview viewport?
For comparison you can try one of the examples provided by Pixar http://graphics.pixar.com/usd/downloads.html. Download the USDSkel example and open HumanFemale.walk.usd. This one will be moving both in USDView and Unity.

@tufeixp Given that it looks like an asset problem, I'm going to close this out until you can confirm it's a Unity-specific bug.
Please reopen it if you can confirm it's a bug on our end!


Use Maps for holding the origins and routes

This guarantees that ties are resolved by insertion order
R: @addyosmani @jeffposnick

LGTM


[ROCm] Fix for the broken --config=rocm build

The following PR/commit breaks the --config=rocm build
https://github.com/tensorflow/tensorflow/pull/30997
It introduces a declaration of the variable "data_layout" , within only the #else section of a #if-else block, and then adds a reference to "data_layout" outside of that #if-else block. Since the ROCm build takes #if path, the declaration for "data_layout" is missing, leading to the compile error on the reference to it.
The fix is add the corresponding declaration for "data_layout" in the #if section.

@tatianashp @whchung @chsigg

The failure in the Linux GPU run does not seem to be related to the changes in this PR

@tatianashp can you please review this PR ?


Save video

We need to save a video of an execution (useful for papers, for replays, etc.).

We did it with OSG :
void bridge::dart_robot_loader::dartRecord(const std::string &directory,
                                           const std::string &prefix,
                                           bool restart,
                                           std::size_t digits){

  boost::lock_guard<boost::mutex> lock(simMutex_);
  osg_viewer_->record(directory,prefix,restart,digits);
}

But I don't find yet a record function inside Magnum's API.

We did it with OSG :

This is recording images per time-step and saving them in a directory, right?
This is already possible with Magnum. You can call the image() function (see here) either inside a Descritptor or after N steps if you do the steps by hand and save them whereever you want using the (save_png_image)(https://github.com/resibots/robot_dart/blob/master/src/examples/cameras.cpp#L81) function.
What we were discussing with @jbmouret was to be able to directly save to a video.

This is recording images per time-step and saving them in a directory, right?

Yes. Then we were transforming after with ffmpeg.
But I understand that you want to directly save to a video.

Yes. Then we were transforming after with ffmpeg.

This is already possible with the current API.

But I understand that you want to directly save to a video.

We will see if we can do this somehow automatically.

Yes, like in pyhexapod, we can stream the image directly to ffmpeg to get a mp4 video directly from the images. The hardest part is to get the picture format right.

see #58

Fixed by #58.


Adding roles, filters and configurations for Physical Switches support

This PR adds roles, filters and configurations for Physical Switches support in the UI.
Depends on:

https://github.com/ManageIQ/manageiq/pull/16948 [Merged]

@agrare, @rodneyhbrown7, this can be reviewed after merging https://github.com/ManageIQ/manageiq/pull/16948.

@agrare, could you take a look on this PR?

To test (to see the changes in the feature tree) this PR has to be tested together with https://github.com/ManageIQ/manageiq-ui-classic/pull/3699
Having done so I see no problems --> merging.


Docker container available?

I'd be happy to run a pre-built Docker container with everything inside to run OmniSharp-server.. would that be feasible ?

Sample Dockerfile to start with in https://github.com/OmniSharp/omnisharp-server/issues/220


Share dialog - record game sometimes shows Play icon and stops

In the Share dialog when I'm recording a thumbnail, the video will start and stop constantly and show a Play icon when it's stopped.
https://user-images.githubusercontent.com/22108954/190701807-804d956c-3f3d-439f-8e05-2cbc20cad41d.mp4

Is the game getting an exception when that happens

Nope

i can't repro this. @Jaqster do you have the game you were sharing?

I'll look at it again tomorrow and see if I can repro...

@Jaqster I can't repro this either. Reopen this issue if you find a repro.


Azure Arc-enabled data services- update secure string variable

Azure Arc-enabled data services- update secure string variable

LGTM as well!


test-branch/execute-job/comment-from-wrong-user-on-pr/6c7e600d-ae09-4756-a2f7-a1ce8eecdcfc

PR for testing 7eb2fd52cbd3c6f89654b47cc20b1f17ff44d739

/canonical/self-hosted-runners/run-workflows 7eb2fd52cbd3c6f89654b47cc20b1f17ff44d739


pebbles could be one of the characters, remotely

it would be cool to have one real person always there. somehow

maybe have her be in front of the pc, ready to intercept about 2-3 times a day for 15 minutes?


Add wgrib2

Fix #29

We are trying to move away from wgrib2 in operations. Who needs to use it?
The reason we are trying to move away from it is because of all it's weird build problems, which create a tremendous amount of extra work and trouble...

A lot of people use wgrib2 for command-line manipulation of grib2.
It's used operationally by various applications.
UFS_UTILS uses the library portion, and though it's moving to g2 we still have to be able to build it.

Yes, I know, it's used in a lot of places. THat makes it hard. So we should discourage future use, if we can...

We still need the ability to install it. And this package.py abstracts away some of that complexity.

I tried to install this from your branch on my Mac with llvm-clang on Monterey and got the following error:
/Users/heinzell/work/jedi-stack/spack-stack-new-joint/spack-stack-develop-20220215/spack/lib/spack/env/clang/gfortran   splat.f90
Undefined symbols for architecture x86_64:
  "_main", referenced from:
     implicit entry/start for main executable
ld: symbol(s) not found for architecture x86_64
collect2: error: ld returned 1 exit status
make[1]: *** [splat.o] Error 1

Too late to figure this out today. Could be one of these pseudo case-sensitive filesystem issues.

Closing until #36 is complete

And I'll give it a try with Clang

Reopening because #36 is not going to happen for now.

I picked this up again today. It's easy to get around the -c flag, but a really weird one is that adding the "usual" gfortran-10+ flag -fallow-argument-mismatch seems to be ignored:
gfortran -fallow-argument-mismatch gdswzd_c.f90
gdswzd_c.f90:233:48:

  233 |              CROT,SROT,XLON,XLAT,YLON,YLAT,AREA)
      |                                                1
Error: Type mismatch in argument 'fill' at (1); passed REAL(8) to REAL(4)

Maybe I've been staring at my screen for too many hours today ...

Fixed with a bit of a hack to find the gfortran libdir, using gfortran -###, when using Clang.
            if self.spec.compiler.name in ['apple-clang', 'clang']:
                if 'gfortran' in self.compiler.fc:
                    output = Executable(self.compiler.fc)('-###', output=str, error=str)
                    libdir = re.search('--libdir=(.+?) ', output).group(1)
                    flags.append('-L{}'.format(libdir))

Ready for review @climbfuji

@kgerheiser I got an error building wgrib2 in the libaec step:
checking for gcc... /Users/heinzell/scratch/spack-stack-package-wgrib2-20220309/spack/lib/spack/env/clang/clang
checking whether the C compiler works... no
configure: error: in `/Users/heinzell/scratch/spack-stack-package-wgrib2-20220309/cache/build_stage/spack-stage-wgrib2-2.0.8-as3xj7ysvzf424mkz2eeiwywk777khc2/spack-src/libaec-1.0.2':
configure: error: C compiler cannot create executables
See `config.log' for more details
make: *** [/Users/heinzell/scratch/spack-stack-package-wgrib2-20220309/cache/build_stage/spack-stage-wgrib2-2.0.8-as3xj7ysvzf424mkz2eeiwywk777khc2/spack-src/lib/libaec.a] Error 77
==> Error: ProcessError: Command exited with status 2:
    'make'

The issue is an unsupported compiler flag I think:
configure:2906: checking whether the C compiler works
configure:2928: /Users/heinzell/scratch/spack-stack-package-wgrib2-20220309/spack/lib/spack/env/clang/clang -I/Users/heinzell/scratch/spack-stack-package-wgrib2-20220309/cache/build_stage/spack-stage-wgrib2-2.0.8-as3xj7ysvzf424mkz2eeiwywk777khc2/spack-src/include -Wmissing-prototypes -Wold-style-definition -Werror=format-security --fast-math -O3 -DGFORTRAN -fopenmp -I/Users/heinzell/scratch/spack-stack-package-wgrib2-20220309/cache/build_stage/spack-stage-wgrib2-2.0.8-as3xj7ysvzf424mkz2eeiwywk777khc2/spack-src/jasper-1.900.1/src/libjasper/include -I/usr/include    conftest.c  >&5
clang-13: error: unsupported option '--fast-math'; did you mean '-ffast-math'?

It's in the top-level makefile, line 286:
ifeq (${COMP_SYS},gnu_linux)
   wCPPFLAGS+=  -Wmissing-prototypes -Wold-style-definition -Werror=format-security --fast-math -O3

If you look a few lines above (240+), you'll see the correct flag in what seems to be the "old configuration method":
# C compile and load commmands
# wCPPFLAGS has the directory of the includes
# wLDFLAGS has the directory/name of the library

ifeq ($(findstring opencc,$(notdir $(CC))),opencc)
   wCPPFLAGS+=-O3   -ffast-math -opencc
   netcdf3CPPFLAGS+=-O3   -ffast-math -opencc
   netcdf4CPPFLAGS+=-O3   -ffast-math -opencc
   hdf5CFLAGS+=-O1   -opencc
endif

I may have an update for your PR ready in a bit, unless I hit more errors further in the build process.


Added possibility to switch off auth on redis

Fixes https://github.com/testcontainers/testcontainers-spring-boot/issues/167

Codecov Report

Merging #169 into develop will decrease coverage by 0.04%.
The diff coverage is 80%.

@@              Coverage Diff              @@
##             develop     #169      +/-   ##
=============================================
- Coverage      84.22%   84.18%   -0.05%
- Complexity       378      380       +2
=============================================
  Files            103      103
  Lines           1249     1252       +3
  Branches          13       15       +2
=============================================
+ Hits            1052     1054       +2
  Misses           190      190
- Partials           7        8       +1

Impacted Files
Coverage Δ
Complexity Δ

...est/redis/EmbeddedRedisBootstrapConfiguration.java
90.9% <100%> (+0.43%)
6 <0> (+1)
:arrow_up:

.../java/com/playtika/test/redis/RedisProperties.java
87.5% <100%> (+1.78%)
3 <1> (+1)
:arrow_up:

...is/EmbeddedRedisClusterBootstrapConfiguration.java
86.95% <50%> (-3.96%)
4 <0> (ø)

Continue to review full report at Codecov.

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

PR needs to be fixed before merging, and in this way it does not working (in order to disable cluster auth we need to modify cluster conf file instead of removing flag from the command, since flag in command does not do anything)


fix wandb lagging at end of ddp training

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] Did you read the contributor guideline?
[x] Did you make sure to update the docs? n/a
[x] Did you write any new necessary tests? n/a

What does this PR do?
Fixes #4619 in a not great way
We need to call wandb.finish() at the end of our code to let wandb know that the multiprocessing job is over. But this is non-trivial with the current setup of progress_bar.py. I tried adding wandb.finish() similar to how tensorboard writers are closed using atexit (see here ) but it doesn't work.
The current solution adds it in fairseq_cli.train but if there is a more elegant solution that uses progress_bar.py I would be happy to change

A better solution is to use wandb service as described in #4619, so I'm closing this PR


Add SkipRoslynAnalyzers parameter to Start-PSBuild

Fix #15632

It is not clear will this block source generators?

It is not clear will this block source generators?

I haven't found any evidence it will.

@xtqqczze Thanks!


HSEARCH-2733 FullTextIndexEventListener doesn't need to be Serializable

https://hibernate.atlassian.net/browse/HSEARCH-2733

Merged, thanks!


[AUTOPATCHER] Patches CVE-2019-5736 for moby-buildx

        Patches CVE-2019-5736 for moby-buildx.
        This patch and PR was automatically generated and needs human verification before merging.

Closing autopatch into 1.0.  This was not intended.


Import target sensors not working. Almost never turn on, and never at sane times.

Describe the bug
Export target sensors work great. Import target sensors no matter what combination of settings never work. Occasionally they turn on at weirdly inappropriate times but mostly not at all. The meter's sensors show all the correct values.
The problem is not new, I've been stuck with this since I started using Home Assistant and this addon. Earlier this year.

Reproduction steps
Create many different sensors on my import meter. Observe that they do not function.
Expected behaviour
I expect the sensors to turn on for the number of hours I set during each day.
Tariff Code
E-1R-AGILE-FLEX-22-11-25-M
Integration Version
v7.4.5
Home Assistant Version
2023.7.2
Fresh Install?
 I wish you'd have raised it sooner so you weren't suffering for so long, as most people to my knowledge have been using this successfully.

Occasionally they turn on at weirdly inappropriate times

Can you clarify this statement further? For instance could you give a day with the times that the integration picked for the rate sensor you've highlighted in your screenshot?
Am I correct in saying that the tariff you specified is what is displayed in the elected meter's current_rate sensor's attributes?
Do your meters have the same mpan but different serial_number values? Again, these can be identified from the associated current_rate sensor.

I wish you'd have raised it sooner so you weren't suffering for so long, as most people to my knowledge have been using this successfully.

No worries, I've been using it successfully to track my energy usage for ages. Only recently found needs for the target sensors. No suffering involved. Thanks for creating it!

Occasionally they turn on at weirdly inappropriate times

Can you clarify this statement further? For instance could you give a day with the times that the integration picked for the rate sensor you've highlighted in your screenshot?

It came on for half an hour at 23:30 a few days ago. Which was a weirdly expensive unit for the day. Sorry I didn't document it.

Am I correct in saying that the tariff you specified is what is displayed in the elected meter's current_rate sensor's attributes?

Yep

Do your meters have the same mpan but different serial_number values? Again, these can be identified from the associated current_rate sensor.

The regular and import meter have the same serial but different mpan. Is that wrong?

I'm not getting anything in the logs. I turned on "debug logging" for the addon earlier. "Download diagnostics" gives me a blank page. Home Assistant main logs don't have anything relevant.

This is interesting. It's on for at least some of the time.

Hi there. I've released a new update (https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/releases/tag/v7.5.2) which included this sensor which I'm hoping will fix your issue.

Hi, I have upgraded. No change yet but I'll give it 24 hours and see if anything new happens.

It turned on for the cheapest half hour last night.

Hi sorry for the late response. This is something I should have spotted earlier, but I believe is the same issue as https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/issues/329 where it's due to the full rate set not being available until the end of the day (and therefore can't calculate the best time). If you update the to time to be equal or below 23:00, then the sensor should start working. See https://github.com/BottlecapDave/HomeAssistant-OctopusEnergy/blob/develop/_docs/setup_target_rate.md#fromto-times for updated docs.

Oooohhhh yes that's done it!
How come the export sensor works? Should I set that to 23:00 to prevent future problems?
Will it definitely be 24 hour or am I losing the last hour?

Never mind, I changed the export sensor to "23:00" and I can see it has properly populated attributes that it didn't have before. I was wrong to say it was working entirely correctly.
Awesome, all working great now. Many thanks for the help and excellent addon.

If you want a "24 hour period", then you can set the from and to times to the same times. So for today it will find the best time between "2023-08-04 23:00 BST" and "2023-08-05 23:00 BST" if both are set to 23:00. Some people set theirs to 16:00 so that their sensors are aligned with the release of the agile rates.

Ahhh now I get it. Thanks again.


Broken gadgets and user scripts due to "ReferenceError: Can't find variable: async"

The primarysources gadgets seems to be broken and causing JavaScript issues for other scripts and gadgets. See https://phabricator.wikimedia.org/T134097 for more information.

Thanks, this was just closed by merging in a pull request with the fix. Sorry for any inconveniences this has caused.


Update setup.py

The extract_loci function call used in PeakGenerator uses the ignore kwarg that wasn't implemented until tangermeme 0.2.3 (possibly 0.2.2? Fails in 0.2.1).

Thanks!


Format document wrong

Hi,
If I have a variable declared in my sqlrpgle than as the same name as a column of my table using format document in vscode it do this:
https://user-images.githubusercontent.com/9719313/219318875-3cbbe102-451b-4787-8b64-17b54f4fbea7.mov
Is it correct?
Many thanks

@pabloto that is the current behaviour. Paste the code here and I will see what I can do, though I don't know what that will be.

Hi Liam, here's a sample.
`**free
Ctl-Opt  DftActGrp(*no)ActGrp(*StgMdl)
StgMdl(*SNGLVL)
Thread(*Concurrent)
Option(*NoUnRef :*SrcStmt :*NoDebugIo)
DatFmt(*Iso) TimFmt(*Iso)
Debug(*Constants)
Expropts(*AlwBlankNum)
AlwNull(*UsrCtl)
DftName(TESTFORM)
Text('Test for format document');
// ____________________________________________________________________________
Dcl-s Ostpc Char(10);
Exec Sql
Set Option Datfmt = *Iso, Timfmt = *Iso, Closqlcsr = *EndMod, Commit = *none;
Ostpc = 'TEST';
Exec Sql
update testupdate
set ostpc = :OStPc
where ococl = '20470' and otito  = 'FCT';
Return;`


Site seems to have stolen your blog ;=(

https://newshimalaya.com/2020/07/04/scaling-linux-services-and-products-earlier-than-accepting-connections/
Btw. thanks for the Blog, I haven't read it yet but it seems to be what I'm looking for.

Thanks! It seems like a bunch of sites automatically ingest popular posts and repost them, oh well! I doubt it's worth chasing them down.


“Running multiple agents” documentation should use --spawn

On some platforms, the documentation for “Running multiple agents” recommends starting multiple buildkite-agents using the same config. This was originally written in 2.x when the default config was name=%hostname-%n (%n was incremented server-side). But now %n does not work anymore (BuildKite Changelog: Agent names no longer support "%n") the default config is name=%hostname-%spawn (%spawn is created client-side https://github.com/buildkite/agent/pull/1377), which requires you to run the agent once with buildkite-agent --spawn=<n>. If you run multiple buildkite-agent processes with the same config, then “there's potential for the different agent processes to compete for a checkout directory” (https://github.com/buildkite/agent/pull/1389). The following 3 docs should be updated to use --spawn instead of running multiple buildkite-agent processes with the same config.

https://buildkite.com/docs/agent/v3/ubuntu#running-multiple-agents
https://buildkite.com/docs/agent/v3/debian#running-multiple-agents
https://buildkite.com/docs/agent/v3/redhat#running-multiple-agents

I suggest fixing this in a slightly different way by using systemd unit file specifiers to give each agent a unique name, rather than whatever --spawn does.
However, the docs are still outdated because they encourage users to copy the original unit file to create a template file - the debian packages already contain the template file now.

Thanks for raising this @yonran! And thanks for adding your thoughts @DavidGregory084. We've updated the docs to recommend the spawn approach for running multiple agents in #2292. 😊


[Network]Do not merge

Hi, @msyyc. The PR has be closed for a long time and it's related branch still exist. Please tell me if you still need this branch or i will delete it in 14 days.


Update MD file on new branch 2017-11-14_11-27-28

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/content1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Support running on supervised installs

The current method of running the library is centered on running the code entirely separate from the install. For a setup like a RPi running HA OS, there is a non-trivial amount of work getting this off the ground and running in a sane way. The library itself supports this, most of the challenge surrounds workspace management
High level goals list
Trying to stick as close as possible to a fresh install of home assistant -

create a general environment management script that will auto set up node
create an addon that can host a "deployment" build of the code
find a way to add maintenance scripts to vscode UI, so typing on terminal isn't required

Notes pile
Environment setup
Information in home folders does not get persisted, and will be reset every time home assistant reboots. This is potentially resolvable via the addon or extension, but it's require granting those components much more access to the system, or potentially additional dependencies.
No data should be persisted there
Accessing node w/ VSCode
The effort of making no changes to the system prevents VSCode from being able to access node by default. This prevents tasks that run nodejs code from operating.
Loading fnm via ~/.zshenv seems to provide access for both new terminal instances, and for above situation.
Open questions

should fnm be installed by default in /homeassistant/.fnm, or /homeassistant/{project}/.fnm?

current answer: /homeassistant/.fnm

is there anything wrong with sourcing node_modules from the dev env? should a deployment build have it's own production node_modules?

current answer: sourcing from dev env for simplicity

V2 notes
V1 targets the MVP to gain full functionality from the core libraries, under a supervised install. There are still big spots for improvement -
HTTP
Creating http servers in this setup can be much more tricky than a more normal bare metal install. The addon requires some additional configuration to properly route inbound traffic, and that needs to be coordinated with the app on the inside
Idea: run type-writer in watch mode
Situations worthy of updating type definitions are somewhat rare. It should be plausible to run the script in a background mode, just waiting for those events to be emitted via the socket. When they occur, the script will automatically rebuild types

Noticed this after setting up tasks, but opening the workspace gives access to the package.json scripts also. This panel does not appear if the workspace is closed.
Guess there are now 2 methods of running the code from the UI now

https://developers.home-assistant.io/docs/add-ons/configuration/#add-on-advanced-options
build for addon should go to the indicated folder

done


Added timeout-option/parameter to start_discovery(...)

I had to re-open this pull-request, because I deleted my first fork of gatt-python. I'm sorry for that.
But I've now implemented blumberg's suggestion.

Would it be an idea to have the default of 10 sec (10000 ms) in the function when the timeout parameter is missing?  So that when it is not explicitly provided, that 10 sec is used; then the explicit option added in the "using" call could be omitted. If acceptable, perhaps an explicit 0 would have it not timeout, meaning that when timeout=0 is provided as parameter input, the discovery keeps on running (which is what I understand happened before this PR).

I agree with @mdxs and go for a default timeout of 10s. It should also be documented what a value of None means for the timeout.


[CF1] Document using different usernames in browser-based SSH

Using a duplicate user works as expected

Thanks for the suggestion! Unfortunately I was unable to get this configuration working in my test environment. After logging into Access as<EMAIL_ADDRESS>the browser-based terminal automatically logs you in as the original user jdoe. There is no option to enter a different username (e.g. john.doe) through the browser when using short-lived certs. How were you able to log in as john.doe?
In any case, I am going to close this PR as this is not a configuration that we officially support.

#13339

Thanks for the suggestion! Unfortunately I was unable to get this configuration working in my test environment. After logging into Access as<EMAIL_ADDRESS>the browser-based terminal automatically logs you in as the original user jdoe. There is no option to enter a different username (e.g. john.doe) through the browser when using short-lived certs. How were you able to log in as john.doe?
In any case, I am going to close this PR as this is not a configuration that we officially support.

By adding another entry to the passwd database that has the same uid/gid as the original user, but a different username.
For all purposes and intent, that other name behaves just like the original user. Linux useradd(8) has the -o option to allow these duplicate users to be created.
This is an ancient thing, for example FreeBSD by default has a toor account (see https://cgit.freebsd.org/src/tree/etc/master.passwd#n2). Purpose of this is so you can set root's shell to a shell that's not in the base system. If at boot you were unable to mount your /usr/local you'd be locked out of root (can't start shell), but toor would let you login.
root::0:0::0:0:Charlie &:/root:/bin/sh
toor:*:0:0::0:0:Bourne-again Superuser:/root:

There's nothing for Cloudflare to support here, support is in the OS 😄, documenting this would make migration easier, otherwise you'd have to go rename existing users to match whatever you get from the Identity Provider. Renaming the user would make things that rely on the username (not the id) like entries in /etc/cron.daily fail.


refactor: new sinsp logger implementation

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup
/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
This removes the g_logger extern variable in favour of a new a more maintainable api. The logger can now the accessed with sinsp::get_logger().
Which issue(s) this PR fixes:

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

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: therealbobo
Once this PR has been reviewed and has the lgtm label, please assign gnosek for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi! Thanks for this PR!
While i like the idea, i have some concerns regarding build time, since everywhere sinsp logger is used must now include sinsp.h :/
Also, i'd just call the method ::logger instead of ::get_logger, since there is no setter.

Yeah, I don't like adding even more responsibilities to the sinsp class :/
If naming is the only issue, can we have e.g. sinsp_logger be a function so we can sinsp_logger()->log(...)?

Didn't thought to do it that way! I'll update the pr :)

LGTM label has been added.  Git tree hash: b72496c22b986bbc17723e9ee6e949319301e36a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, incertum, therealbobo
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,incertum]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Update Pastebin link

I created a disposable account with a disposable e-mail to find the new link :)


Set as public hautelook_alice.fixtures.loader service

When we use AliceBundle with SF 3.X we get a lot of deprecation notice because the  hautelook_alice.fixtures.loader service is private.
Note that the target branch is the 1.X.

Hi @theofidry
Do you think the PR could be reviewed?
Thanks a lot

Sorry I don't think I'll accept it. I'm fine with merging and releasing stuff on 1.x but only if the CI is green which I guess will take its load of work. If you really need this I think moving to a fork makes more sense since 1.x is no longer maintained anyway.


transfer-proxy-token-encryption-aes-key → Length?

Missing information in the documentation for:
https://github.com/eclipse-tractusx/tractusx-edc/blob/5e47d348aee3c72c070d5efb0071c4dc389eeb33/charts/tractusx-connector-azure-vault/values.yaml#L541
What are the requirements for the aes key?

length (128, 256)?
mode (GCM, CBC, ECB, CTR, etc)?
anything else?

openssl enc -aes-256-cbc -k secret -P -md sha1 ?

Hi @jerzy-wachowiak
thanks for raising this the transferProxyTokenEncryptionAesKey is not needed anymore for the runtime to be working.
I think this is just a leftover for the azure helm variant. Will do a PR for removal
Thanks


traffic_ops/experimental - start of a TO 2.0 server

See also notes on the wiki; trying to preserve the git history, hope I did this right.

@PSUdaemon help! I have all the commits here, but it says 0 files changed?

something went wrong, closing.


Pin docker images to version 6

We've been pulling latest, which was fine until we updated the latest tag on dockerhub with a new version of java. Tests that rely on java being where they expect it to be now fail. This PR pins the docker images to a version that has java in the expected location.

looks good

@ebd2 I think this was merged so you can close it. Also don't forget to delete your branch on our fork if you haven't already done so.

Hmmm. I wonder why this didn't close automatically when I pushed. I doubt that I had to rebase, after all. Thanks for catching that @petroav


Delete Account Access Consent should return HTTP 204 No Content

Deleting a consent currently returns a HTTP 200 response with the deleted consent as the payload. This can be observed by running this postman request: https://forgerock-obri.postman.co/workspace/Secure-Banking-Collections~02b51b27-f1d1-4b38-9b77-0e8d085c9a91/request/5812244-aa24ffe4-514f-44a5-b6d0-1c01620fb56e
The OB spec instead says that a HTTP 204 No Content response should be returned, see spec: https://openbankinguk.github.io/read-write-api-site3/v3.1.10/resources-and-data-models/aisp/account-access-consents.html#delete-account-access-consentsconsentid
As IG is reverse proxying IDM for this flow, then I suspect the behaviour we are seeing is how IDM handles deletes. We need to handle the IDM response in IG and convert it to the expected response of the OB API.
We also need to review other consent flows that support deletes to see if they also have this bug.

No other consent flows that we have implemented support deletes.
We will need to be mindful of this bug when implementing VRP payments.


add rolling window batch operation for tf.data.Dataset

Resolve #15044.
implementation
The PR proposes a slide method for Dataset: Groups elements in fixed size blocks by passing a "sliding window" over Dataset. It behaves like batch, in fact, batch(n) == slide(n, n).
I failed to move c++ implementation from core to contrib. Any help will be appreciated.
how to test

[x] add test case.
[ ] pass all tests.

I have checked those failed logs: build broken. And those tests run well on my work machine. Seems unrelated?

Thanks, I'll try to fix the problem when I get back from holiday.
tensorflow/core/kernels/data/slide_dataset_op.cc:208:14: error: 'tensorflow::Status tensorflow::{anonymous}::SlideDatasetOp::Dataset::Iterator::RestoreInternal(tensorflow::OpKernelContext*, tensorflow::IteratorStateReader*)' marked override, but does not override
       Status RestoreInternal(OpKernelContext* ctx,
              ^
In file included from ./tensorflow/core/framework/tensor_shape.h:23:0,
                 from ./tensorflow/core/framework/partial_tensor_shape.h:20,
                 from tensorflow/core/kernels/data/slide_dataset_op.cc:15:
tensorflow/core/kernels/data/slide_dataset_op.cc: In member function 'tensorflow::Status tensorflow::{anonymous}::SlideDatasetOp::Dataset::Iterator::RestoreInternal(tensorflow::OpKernelContext*, tensorflow::IteratorStateReader*)':
tensorflow/core/kernels/data/slide_dataset_op.cc:212:68: error: no matching function for call to 'tensorflow::{anonymous}::SlideDatasetOp::Dataset::Iterator::RestoreParent(tensorflow::OpKernelContext*&, tensorflow::IteratorStateReader*&, std::unique_ptr<tensorflow::IteratorBase>&)'
           TF_RETURN_IF_ERROR(RestoreParent(ctx, reader, input_impl_));
                                                                    ^

Hi, I have merged the latest codes from upstream, and fixed the broken test (at least for Linux CPU). Dose anyone help run all tests again? Thanks.

@drpngx Hi, could you help me run all tests again? 7 days passed, and reviewers seem so busy to reply.

Thank @drpngx for your help, all tests passed.
@mrry @rmlarsen I think all comments have been addressed, and could you take a look? Thank you.

Nagging Assignee @rmlarsen: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Thanks all for the help!

Woohoo!
On Wed, Mar 7, 2018, 2:28 PM Yan Facai (颜发才<EMAIL_ADDRESS>wrote:

Thanks all for the help!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/pull/16123#issuecomment-371307108,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AT_SbQLzaR8DNTO1jk7urSTJ24zTEuxnks5tcF7qgaJpZM4Rd-0O
.

Hi, I have issues getting this working
AttributeError: module 'tensorflow.python.ops.gen_dataset_ops' has no attribute 'slide_dataset'
any suggestions greatly appreciated, thank you

@rombin Could you collect all necessary information and open a issue to report the problem?

I think I have an issue with installation. I have installed tensorflow 1.6 via Conda. But I do not see the sliding.py file in the contrib/data/python/ops folder. Do I somehow need even newer version? I notice you merged 2 days ago??

On 17 Mar 2018, at 23:01, Yan Facai (颜发才<EMAIL_ADDRESS>wrote:
@rombin https://github.com/rombin Could you collect all necessary information and open a issue to report the problem?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tensorflow/tensorflow/pull/16123#issuecomment-373959082, or mute the thread https://github.com/notifications/unsubscribe-auth/AiaZ4s2_pZkXop7EuDuoHMCnK1bOUIuuks5tfZW5gaJpZM4Rd-0O.

Yes, I think the op will be released in the next version, and you can also install tf-nightly to take a try.

tried tf-nightly, works like charm, thank you

On 18 Mar 2018, at 00:25, Yan Facai (颜发才<EMAIL_ADDRESS>wrote:
Yes, I think the op will be released in the next version, and you can also install tf-nightly https://github.com/tensorflow/tensorflow#installation to take a try.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/tensorflow/tensorflow/pull/16123#issuecomment-373962914, or mute the thread https://github.com/notifications/unsubscribe-auth/AiaZ4s1P5doPCjlZ1c27S-HslrwxvpHmks5tfalwgaJpZM4Rd-0O.


feat(autoware_launch): add option to use akima spline at first

Description
launcher PR for https://github.com/autowarefoundation/autoware.universe/pull/2800
NOTE: Akima spline is disabled by default.

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@TakaHoribe Could you approve the PR.


Allow choosing the output format from hex/integer/float tuples

Something like:
--output=(hex | rgb255 | rgb01)

where hex is the default, rgb255 gives you an RGB tuple with integer values (0–255) and rgb01 gives you an RGB tuple with float values (0-1).

Closing as not wanted.


Should I be afraid of non-official build of Ungooled-Chromium?

I know there is a small posibility to download a harmful or tampered release because if you don´t build by your own you have to download someone else`s exe. But, is this well moderated? Really I don´t know how to inspect this kind of code.

For downloading binaries, use the site linked in the github README, here. Those should be trustworthy enough.

For downloading binaries, use the site linked in the github README, here. Those should be trustworthy enough.

For downloading binaries, use the site linked in the github README, here. Those should be trustworthy enough.

Hi mate! I´m new into this browser. Do you recommend installing it via Chocolatey? I read something about this could help me to keep updated ungoogled-chromium without reinstalling

For downloading binaries, use the site linked in the github README, here. Those should be trustworthy enough.

Hi mate! I´m new into this browser. Do you recommend installing it via Chocolatey? I read something about this could help me to keep updated ungoogled-chromium without reinstalling

This is the easiest way to install with auto updates: https://github.com/iklevente/ChromiumForWindows
It uses files from https://github.com/macchrome/winchrome

This is the easiest way to install with auto updates: https://github.com/iklevente/ChromiumForWindows
It uses files from https://github.com/macchrome/winchrome

@farinelli123 chocolatey uses the same binaries as listed here. so you can install from chocolatey.

@farinelli123 chocolatey uses the same binaries as listed here. so you can install from chocolatey.

To somewhat hijack this thread, it'd be fantastic to get official releases from a build server or similar through GitHub. Ungoogled Chromium binary packages are constantly removed from the AUR because there aren't any official builds.

To somewhat hijack this thread, it'd be fantastic to get official releases from a build server or similar through GitHub. Ungoogled Chromium binary packages are constantly removed from the AUR because there aren't any official builds.

Builds are only as trustworthy as the builder and the build tools. But unless people from ungoogled-chromium project trusted them to a certain extent, they would not be linking to those.
I built my own binary fairly recently (this being the first and only time I built it so far) and I think the shift to reproducible build systems would combat this trust issue.

Builds are only as trustworthy as the builder and the build tools. But unless people from ungoogled-chromium project trusted them to a certain extent, they would not be linking to those.
I built my own binary fairly recently (this being the first and only time I built it so far) and I think the shift to reproducible build systems would combat this trust issue.

To somewhat hijack this thread, it'd be fantastic to get official releases from a build server or similar through GitHub. Ungoogled Chromium binary packages are constantly removed from the AUR because there aren't any official builds.

this would be "official" enough for me personally https://software.opensuse.org/download/package?package=ungoogled-chromium&project=home:ungoogled_chromium#manualArch

I hope your question has been answered, otherwise please let us know.
Closing this issue for now.


Cant access Web UI's application tracker link - Spark on Docker

I installed Spark using docker on MAC. I ran into the issue that I wasn't able to access resource manager web ui.
but resolved after adding ports: for i in {10000..10999}; do
VBoxManage modifyvm "boot2docker-vm" --natpf1 "tcp-port$i,tcp,,$i,,$i";
VBoxManage modifyvm "boot2docker-vm" --natpf1 "udp-port$i,udp,,$i,,$i";
done
But I have another issue: In the resource manager web ui for spark , there is another link: application tracker - but I am not able to access it. Any help with this ?

If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
For more information about reporting issues, see CONTRIBUTING.md.
You don't have to include this information if this is a feature request
(This is an automated, informational response)

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@MahalakshmiL ehm, just to prevent confusion, you say you installed "docker on MAC", with that you mean http://github.com/docker/toolbox or the new "Docker for Mac" which is now in beta? https://blog.docker.com/2016/03/docker-for-mac-windows-beta/

Also, I don't have experience with Spark, and a link not working in the control panel may not be an issue with Docker; could you give some more information? What error are you getting then opening that link, what URL does it try to open?
Perhaps you could add the steps needed to reproduce (e.g., is this a custom Dockerfile you built, or is there an image on Docker Hub that you're using?)

Closing as we didn't get a response


add 0.3 second request interval

update the request_records method to insert a small gap between requests, per Slack conversation regarding updating the Meltano schedule that uses this extractor to run every hour (relevant PR)

Great idea, will do!On Aug 8, 2023, at 12:36 PM, Prratek Ramchandani @.***> wrote:
@prratek requested changes on this pull request.
what do you think of making this configurable with a default of 0s? feels cleaner and lets us adjust from just the dagster codebase if we need to
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


problem when running the "use" database command

Description
when running the command use to select a base it gives the error: Error SQL [1064] [42000]: You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'use x' at line 10
DBeaver Version
Comunity version <IP_ADDRESS>404211624
Operating System
Ubuntu Mate
Database and driver
Mysql driver 8.0
Steps to reproduce
Running command "use" in your database
Additional context
No response

Please provide minimal script example and detailed steps to reproduce your issue.

Good morning @E1izabeth, to reproduce just run the use command in your database as shown in print

This problem did not occur in the previous version


[Bug] Proceeding to next step without entering any information

Is there an existing issue for this?

[X] I have searched the existing issues

Bug Description?
When the user attempts to proceed to the next step in the form without entering any information, the application still allows them to move forward. This should be prevented by adding proper form validation.
Steps to Reproduce:
Open the form.
Click on "numbers" without entering any data.
The application proceeds to the next step without any validation.
Proposed Solution:
Implement form validation to prevent moving to the next step unless all mandatory fields are completed.
Add Screenshots
Screencast from 14-10-24 07:26:07 PM +0530.webm
What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[ ] I'm a GSSOC contributor
[ ] I want to work on this issue (Optional)
[ ] I'm willing to provide further clarification or assistance if needed.

I would like to work on this issue under hacktoberfest and gssoc

@Deekshit7483 Go with it.

Hi @Trevisx07 / @Deekshit7483  we already have issue for this #392

@lade6501 okay you can work on it!

@Deekshit7483 No need to do this as he is doing. You can find new issues and comment ensuring that no one took it before.


Enable platform tests in PR/CI.

Update datacollector.PlatformTests to be picked up pattern.
Add deps.json and runtimeconfig.json for datacollector as content to ensure they are
copied to platform tests output directory.

@codito,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


TestTransformers should not verify the number of the sum of transformers

TestTransformers verifies the sum of transformers in Ingress but that is not accurate.
For example, Availabel contour ingress
	}, {
		name: "Available contour ingress",
		instance: servingv1alpha1.KnativeServing{
			Spec: servingv1alpha1.KnativeServingSpec{
				Ingress: &servingv1alpha1.IngressConfigs{
					Contour: servingv1alpha1.ContourIngressConfiguration{
						Enabled: true,
					},
				},
			},
		},
		expected: 0,
	}

This test also passes with Enable: false:
	}, {
		name: "Available contour ingress",
		instance: servingv1alpha1.KnativeServing{
			Spec: servingv1alpha1.KnativeServingSpec{
				Ingress: &servingv1alpha1.IngressConfigs{
					Contour: servingv1alpha1.ContourIngressConfiguration{
						Enabled: false,
					},
				},
			},
		},
		expected: 0,
	}

Because the number of Contour's transformer is 0 atm.

https://github.com/knative/operator/pull/500
This PR added the test functions to verify and compare the resources between two manifests.
We are now able to check if resources match in terms of name, kind, version ... even every attribute.


v0.5.5 Firefox mobile broken

In lists, you can add new bullets/numbers so that the cursor movies down, but when you start typing, it appends the new characters to the end of the last li element.
When you click anywhere in the text editor, it creates a selection menu.
deleting elements skips non-text elements

It looks like this is a bug in Firefox for android. I tested several other html5 editors and they are also broken (but work in Chrome).


digitalWrite not sticky (doesn't hold state after program exits)

I've found that analog writes are sticky (the pwm continutes to be generated after the program quits) however digitalWrites are not sticky, after the program quits they go back to their former state.
Test program:
var pin = 'P8_10';

var b = require('octalbonescript');

b.pinMode(pin, b.OUTPUT, function(x) {
	console.log(x);
});
b.digitalWrite(pin, b.HIGH, function(x) {
	console.log(x);
});

To reproduce, watch the output of P8_10 on a scope or with a meter to see the change when the program exits.

You can write code like this

var pin = 'P8_10';

var b = require('octalbonescript');

b.pinMode(pin, b.OUTPUT, function(x) {
    b.digitalWrite(pin, b.HIGH, function(x) {
        console.log(x);
    });
});

Ah, that makes sense. The Async nature of the calls lead to the pinMode occurring after the digital write I presume.

Yup, that's the case :)


Overhaul Runtime Gizmos

Runtime gizmos are becoming more and more important as we continue developing in VR, especially as we delve into mobile vr where visualization is even more important for debugging and understanding.  The system could be improved in a number of ways:
Optimization:

Prebuild meshes for all default shapes (capsules / cubes / spheres)  Many of these are there already, but we should have full coverage.
Custom vertex shaders for the tricky shapes like capsules to allow us to use a single capsule mesh for all capsule shapes.
Dynamically combine lines into mesh objects at draw time to reduce draw-call overhead.

Migrated to https://github.com/leapmotion/UnityModules/wiki/Feature-Wishlist


Fix Issue 71

While the actual problem for #71 was caused by not specifying an absolute path for the mount point location, it tickled a latent bug in the validation logic where, if the mount point was specified as a string, and the string validation failed, it would then attempt to treat the string like a hash, which did not work. This change makes it so that only strings are checked as strings should be checked, and only hashes are checked as hashes should be checked.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@gregsymons

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.


confluence_version_comment does not work on initial publish

While using the plugin to develop content, I published to my personal space (for the first time), and I noticed that the comment in confluence_version_comment (in this case, the Git commit) was not present. I re-published the same content, though, and it was!

For sure the existing implementation only injects a comment for updates pages. I can take a look to see if the API supports setting a message value for new pages -- it was either missed, which we can fix; or Confluence ignores the option (which we should then update documentation to reflect the limitation). Will investigate.

It looks like _update_page should be getting called for new pages:
https://github.com/sphinx-contrib/confluencebuilder/blob/27547021b019994d70957476d7a681bc905a42d9/sphinxcontrib/confluencebuilder/publisher.py#L739-L741
I think the issue is that the new page does not overwrite page:
https://github.com/sphinx-contrib/confluencebuilder/blob/27547021b019994d70957476d7a681bc905a42d9/sphinxcontrib/confluencebuilder/publisher.py#L688

new page does not overwrite page

Note that this is done purposely in the implementation. The _update_page is solely for processing a page update. The reason for the if page: check is to deal with (1) an initial detected page update and (2) detecting a page update from a non-case matching title after failing to upload an original-detected new page.

Did some testing with Cloud and server installs. It looks, on a Cloud installation, if we do populate a version entry for a new page entry, with a version of 1 with a message field, the page history will be populated with the provided message. Unfortunately, it does not look to be the case for non-Cloud installations. We can queue up a fix for Cloud installations, and add some notes in the configuration documentation to indicate that limitation for non-Cloud instances (until a fix or workaround is found).

Note that I have completed the support request with Atlassian about the injection of a comment for new pages on Confluence Server/DC variants. I have been informed of the following:

[Confluence Server/DC does not] allow a version comment for the initial page creation in the web UI, or via the REST API.
(reference: CA-2218680)

With that in mind, the pull request will be updated to document this limitation for the online documentation. Confluence Cloud versions should still work as expected.

Adjustments have been applied via #762 and merged into the development tree which should improve support for adding a comment for new pages on Confluence Cloud. For users interested in trying this capability before a release is made, the development version of this extension can be installed.

v2.1 is now available on PyPI -- marking as closed.


Research Nostr Blog Data

Find out what kind of data format for sending draft and final blog posts
Find out what we need to do to edit a blog post already sent to Nostr
What relays should we consider using? habla.news and yakihonne.com?
Where is the source code to the relays that are supporting blog posts? (link here)

Reference

Find out what kind of data format for sending draft and final blog posts

Final draft and final blog posts are sent as Markdown Files.
Find out what we need to do to edit a blog post already sent to Nostr
To  edit, all you need to do is click the edit button on a NIP23 client (like habla.news )
What relays should we consider using? habla.news and yakihonne.com?
I like habla.news because it's the first one I used, but yakihonne.com sends long-form posts through the most relays.
Where is the source code to the relays that are supporting blog posts? (link here)

The clients I know of are habla.news, yakihomme, and flycat.club.  I'll need to do more research on the relays.


S3TC support for Godot 4.0 for Android export

Godot version
4.0 alpha 13
System information
Ubuntu 20.04, Vulkan, NVIDIA  GeForce GTX 1070 Ti
Issue description
First mentioned in #63832.
I'm trying to export my VR project for Android to be used on a Meta Quest 2. But when running on the device, it fails to load my texture resources (png, jpg, bmp formats).
The files exported into the apk:

The errors I get in Logcat:
USER ERROR: Unable to open file: res://.godot/imported/icon.bmp-9d1de709bb464c1beb38c8bc584bca25.s3tc.ctex.
at: _load_data (scene/resources/texture.cpp:823) - Condition "f.is_null()" is true. Returning: ERR_CANT_OPEN
USER ERROR: Failed loading resource: res://.godot/imported/icon.bmp-9d1de709bb464c1beb38c8bc584bca25.s3tc.ctex. Make sure resources have been imported by opening the project in the editor at least once.
at: _load (core/io/resource_loader.cpp:222) - Condition "found" is true. Returning: Ref()
USER ERROR: Failed loading resource: res://icon.bmp. Make sure resources have been imported by opening the project in the editor at least once.
at: _load (core/io/resource_loader.cpp:222) - Condition "found" is true. Returning: Ref()
The required .s3tc.ctex files exist in my local .godot/imported folder but they won't get exported into the apk.
Steps to reproduce

Use the latest master build.
Make a basic OpenXR project that uses either png, jpg or bmp files as texture resources.
Export the project for Android, install it to a Meta Quest 2 then launch it.

Minimal reproduction project
This project works in the editor but not on the Quest 2:
godot_4_alpha13_resource_loading_test.zip

Could you test #63918?

I've rebuilt the engine using the latest master branch but It's still looking for the S3TC files:
USER ERROR: Unable to open file: res://.godot/imported/icon.png-487276ed1e3a0c39cad0279d744ee560.s3tc.ctex.
at: _load_data (scene/resources/texture.cpp:823) - Condition "f.is_null()" is true. Returning: ERR_CANT_OPEN
USER ERROR: Failed loading resource: res://.godot/imported/icon.png-487276ed1e3a0c39cad0279d744ee560.s3tc.ctex. Make sure resources have been imported by opening the project in the editor at least once.
at: _load (core/io/resource_loader.cpp:222) - Condition "found" is true. Returning: Ref()
USER ERROR: Failed loading resource: res://icon.png. Make sure resources have been imported by opening the project in the editor at least once.
at: _load (core/io/resource_loader.cpp:222) - Condition "found" is true. Returning: Ref()
USER ERROR: res://Main.tscn:3 - Parse Error: [ext_resource] referenced nonexistent resource at: res://icon.png
at: load (scene/resources/resource_format_text.cpp:486) - res://Main.tscn:3 - Parse Error: [ext_resource] referenced nonexistent resource at: res://icon.png
USER ERROR: Failed loading resource: res://Main.tscn. Make sure resources have been imported by opening the project in the editor at least once.
at: _load (core/io/resource_loader.cpp:222) - Condition "found" is true. Returning: Ref()
USER ERROR: Failed loading scene: res://Main.tscn
at: start (main/main.cpp:2689) - Condition "!scene" is true. Returning: false
Should I change something in the project?

Did you also build the Android export template with this commit?

Reinstalled the alpha 13 export template and the build template. Same errors. Anything else I should do?

Okay, tried with rebuilt export templates too. Same issue persists.

I wasn't supposed to check the 'Custom Build' option, right?

Okay, using the rebuilt export templates and with the 'Use Custom Build' option unchecked I'm not getting these errors anymore. Thanks for everything!


feat(INVT-CPC-895): Get all custom inventory types

JIRA: link to jira ticket
https://champlainsaintlambert.atlassian.net/browse/CPC-895
Context:
This ticket is about adding the rendering of all created inventory types after creating them.
Changes

This affects the inventory
This affects the addition and editing of inventories
Added get all inventory type request
Displays all the default + custom inventory types in the dropdown bar
Search queries now support the custom inventory types as well + displayed when clicked on
All testing required is completed

Before and After UI (Required for UI-impacting PRs)
Before

After

Add Form

Update Form

Linked pull requests (Optional)
https://github.com/cgerard321/champlain_petclinic/pull/505

When you click new type does it open a form or how does it work?

When "New Type" is selected, the textbox will be empty to fill out a custom inventory type. If nothing is entered, an alert will pop up. Once the new inventory type is created, the new type will now be displayed in the list of available types which you can perform adding, editing, and search operations with.


Add v3.0-RC2 support

This PR adds v3.0-RC2 support in the GBFS validator.
I chose to replace the v3.0-RC tests with v3.0-RC2 tests as the v3.0-RC2 changes includes the v3.0-RC changes.
A thorough review would be appreciated as this is my first version update contribution to the validator. Thank you!

LGTM


Add missing break in loop

Looks like has needs to change to hasProperty in build.gradle to fix travis

bors try


Backlog: Filter NER types / Hiding NER values depending on its type

Considering the following result:

Another example:

All of it might be caused by hiding all the entity values by using #E , while it is expected to be hidden only specific ner types: LOC, PER, ORG

Based on #3

Causes an Embedding issues

class CustomSimpleUppercasedEntityFormatter(SimpleUppercasedEntityFormatter):

    def to_string(self, original_value, entity_type):
        assert(isinstance(original_value, Entity) or original_value is None)
        assert(isinstance(entity_type, EntityType))

        if isinstance(original_value, Entity):
            if original_value.Type not in ["PER", "LOC", "GPE", "GEO"]:
                return original_value.Value

        return super(CustomSimpleUppercasedEntityFormatter, self).to_string(original_value=original_value,
                                                                            entity_type=entity_type)


Merge all queries on each specific subscription

Hi,
Actually, if I'm not mistaken, for each subscription, when there is new data available the schema of each listener query is executed against the GraphQL Schema. Am I right ?
This seems rather inefficient.
I think it would be doable to merge all listeners query into one, execute it against the schema and then dispatch only the queried data for each listener.
What do you think ?

This might be doable on a per-client or per-user basis, but we definitely can't do this across multiple users by default because of authorization issues.
Can you give a more specific example of a case where it would be more efficient to merge queries?

Oh ! I forgot about user authentification, and more important, I forgot about query batching/caching.
This is easily solved (i.e. the number of hits to database with each separate subscription being run against the schema) by facebook/dataloader.
Thanks for you quick reply !


Fix isCtrlEnter switch

When CONFIG.newTab = true and two requests are executed: the first with a clamped ctrl, and the second without, then the resulting page will be with the string ".com" at the end. This has now been fixed.

Hello, thank you for the PR!
I could reproduce the bug you mentioned too, and it seems to be fixed with your commit. I am merging the branch.
Have a nice day


feat: Allow to pass optional flags to resource template

In some cases, it might be useful to have the option to pass optional flags to a resource template.
For example, to turn off kubectl default validation when submitting a resource manifest. OpenShift has some strict requirements (and in case of OpenShift 3.11 even discrepancies) in the oapi spec, which prevents resources from being submitted via kubectl
This patch introduces the Template.Spec.Resource.Flags parameter which allows to pass optional flags to a resource template.
Example:
resource:
  action: create
  flags: [
    "--validate=false"  # validation will be turned off
  ]
  manifest: |
    apiVersion: route.openshift.io/v1
    kind: Route
    metadata:
      name: host-route
    spec:
      to:
        kind: Service
        name: service-name

The example references https://github.com/openshift/origin/issues/24060

Could I ask for help with the CI check? I am kinda struggling to understand the reason it failed.

Looks like a network issue when running dep ensure. Doesn't seem like you changed any dependencies, so don't worry about it. @sarabala1979 will re-run the CI when he reviews your PR.

I think that this PR could become more generalized by allowing the user to pass arbitrary arguments to kubectl. If we merge this, other users could request a similar implementation for other flags, which could cause it to get cluttered.
Something like:

  - name: create-route-without-validation
    resource:
      action: create
      flags: ["--validate=false"]  # validation will be turned off

could work.
Will open this as a new issue. @CermakM do you want to own this?
(P.S. Not that this is slightly different from the open https://github.com/argoproj/argo/pull/1834 as that requires executor changes, so it still requires its own PR)

@simster7 This looks reasonable to me and I'll be happy to extend the implementation :) I suppose we therefore want to have just flags, instead of both flags and validate, correct?

Fixes: #1857

Uh, well, I hoped you wouldn't say that :D I'll see what I can do.

@CermakM is this still wanted please?

@alexec Yes it is. I guess I have quite a lot of rebasing to do. :/

@simster7 do you want to continue as reviewer?

Is the failure of Golang CI setup on my side? :thinking:

@simster7 do you want to continue as reviewer?

Yes, sorry this got out of my radar. I'll give another review today

@simster7 you're right, but IMHO it is not Argo who should impose security restrictions, it is the cluster/namespace admin, I.e. if users have the rights to execute a certain command via kubectl, they should be able to do that in Argo as well, shouldn't they?

@simster7 you're right, but IMHO it is not Argo who should impose security restrictions, it is the cluster/namespace admin, I.e. if users have the rights to execute a certain command via kubectl, they should be able to do that in Argo as well, shouldn't they?

You're right, except that Argo its own ServiceAccount with kubectl – not the user's – so with this PR we give free reign to the submitter to use Argo's ServiceAccount. However, @jessesuen brought up a good point by saying that we effectively already do this since users can submit arbitrary containers... so it looks like we're going to go ahead with this.

@simster7 I rebased it once again and pushed it without updated codegen, which causes the conflicts (which, in turn, I assume, have been blocking the merging the whole time). Please, if / when you decide to merge this, generate the codegen as well.
Cheers,
M

Thanks for your work @CermakM! Glad we finally got this merged

Thanks @simster7 !

I'm new to Argo. Will this solve the issue with Openshift resources when used with ArgoCD as well? or will it only affect argo workflows?

I found this option in Application object:
syncPolicy:
  syncOptions:
    - Validate=false


[Snyk] Upgrade firebase from 10.13.1 to 10.13.2

Snyk has created this PR to upgrade firebase from 10.13.1 to 10.13.2.
As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:sparkles: Snyk has automatically assigned this pull request, set who gets assigned.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 7 versions ahead of your current version.
The recommended version was released 22 days ago, on 2024-09-18.

Release notes

Package name: firebase

10.13.2 - 2024-09-18

10.13.2-vertexai-preview.6b035b7d3 - 2024-09-27

10.13.2-vertexai-preview.36cb0178c - 2024-09-25

10.13.2-canary.beaa4dffb - 2024-09-24

10.13.2-canary.47b091324 - 2024-09-20

10.13.2-canary.2561b6306 - 2024-09-18

10.13.2-20240916151535 - 2024-09-16

10.13.1 - 2024-08-29

    from firebase GitHub release notes

Commit messages

Package name: firebase

52e9e4e Version Packages (#8502)
0884f0d Merge main into release
629919e Fix metadata sync issue for cache listeners in multi-tab (#8343)
16d62d4 Remove `referrerPolicy` from Cloudflare Worker `fetch` requests (#8393)
ca4dbcf Set singleRun to true and remove from all karma start scripts (#8484)
3670ab8 Use default concurrency value for lerna run (#8462)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
👩💻 Set who automatically gets assigned
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Upgrade firebase from 10.13.1 to 10.13.2

Upgrade firebase from 10.13.1 to 10.13.2


"extends" schema resolution assumes current schema is at root of file

Example of a schema that currently fails (also checked with http://www.jsonschema2pojo.org)
{
  "definitions": {
    "parent": {
      "type": "object",
      "properties": {
        "propertyOfParent": {
          "type": "string"
        }
      }
    },
    "child": {
      "type": "object",
      "extends": {"$ref": "#/definitions/parent"},
      "properties": {
        "propertyOfChild": {
          "type": "string"
        }
      }
    }
  },
  "type": "object",
  "properties": {
    "child": {"$ref": "#/definitions/child"}
  }
}

Can you please release 0.4.16 quickly. All our schemas that use 'extends' are broken when we upgrade 0.4.14 to 0.4.15.
error:
org.jsonschema2pojo:jsonschema2pojo-maven-plugin:0.4.15:generate failed: Path not present

Hi, I'm just waiting for one more PR to be merged.
It wasn't clear to me that this was a regression introduced in 0.4.15, or
that this change necessarily fixes that problem. Could you provide a schema
that is affected by this? I'd like to confirm that this is now working and
add a test case around it.
It would be helpful if you could mvn install the current snapshot and test
it in your project.
If you are having problems with 0.4.15 I recommend you stick with 0.4.14.
On 23 Oct 2015 10:17 am, "wsalembi"<EMAIL_ADDRESS>wrote:

Can you please release 0.4.16 quickly. All our schemas that use 'extends'
are broken when we upgrade 0.4.14 to 0.4.15.
error:
org.jsonschema2pojo:jsonschema2pojo-maven-plugin:0.4.15:generate failed:
Path not present
—
Reply to this email directly or view it on GitHub
https://github.com/joelittlejohn/jsonschema2pojo/issues/425#issuecomment-150525130
.

My schema is very similar to the new integration test in 0.4.15 (jsonschema2pojo-integration-tests/src/test/resources/schema/extends/extendsSchemaWithinDefinitions.json)
I confirm everything is working with the current snapshot. I tested it.