No clear indication that the file is stored in cache

Hey keeweb devs,
I really love your tool but I just lost some passwords today. The web version of keeweb (using the self hosted version) saves files only in the cache and does not change the local file - which is okay, but it's really dangerous to hide this hint in the settings.
After adding new entries I hit "ctrl+s" to save the file, which caused the status of the file to change to "saved". You can close the tool from that without ever having seen any hint that your local file was never changed. If you clear your cache or upload the file back to, e.g. owncloud, you're not aware that everything you just added to your file was just temporary.
Since it almost caused me to loose a new random 20 characters password today which I only recovered from another application, I think this issue is highly critical.
Thank you for your concern,
skoam

I don't want to think about passwords I created in the last weeks on different devices. This is insane.

Hitting "x" on the recent file list (login view) should also not destroy a complete file (if working with cached files is the only solution). Without a warning at all.

Hello,
what would be your proposal for this, where and when to show the warning?
For me, it's clear that none of web apps can save local files but yes it could be confusing for some users.

Yes, I think it's not actually a problem of the system but convenience-wise it can cause data loss.
I could imagine the following changes:

When saving a file ctrl+s when a local file was opened, trigger downloading the file in addition / instead of saving the cached copy
When removing a cached file by hitting "x", show a clear indication that the cache file will be removed and offer to download the file
Add a hint/popup that states that the web version cannot save local files directly and offer to download the desktop app (there already is a hint for that, but you have to find it in the settings first)

The second option will solve all problems with data loss. To explain the user that he's editing local files, I'll also add an alert when opening such files.

This would be great! Thank you!

I've made a fix, the release is going to happen today, you can preview the changes here: https://beta.keeweb.info/


Consider implementing run locking

Could be a useful feature, although we haven't heard the need for it directly from users yet. Check out how Snowplow implemented here:
https://github.com/snowplow/sql-runner/pull/75

yeah -- this is interesting. We'd probably want to lock at the schema level. That way multiple users can run dbt in their own schemas, but the analytics schema (for instance) can only have one writer at a time. let me think a little more about that

this relies on consul which is not really easily included into an analytics stack. the most interesting approach for this IMO is to use SQL to implement run-locking. on that topic, it'd be interesting to talk about keeping a record of dbt runs in the warehouse.


No players shutdown timeout

Bot option to stop server when no players found during long period of time.
Timeout < 0 = disabled

Message: Server was stopped automatically at Time Date

Cool, but how to set it up for Minecrafters?
Another command or subcommand?

Yup

Done in last commit


[Bugfix] Cherry-pick MAXVALUE bugfixs to branch-2.4

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #12270
Fixes #12231
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

starrocks_be_unittest succeeded.

run starrocks_fe_unittest


Bump deps & fix Protobuf issues

Pulling in all the proposed changes from #3 and attempting to fix the Protobuf issues described there by @crodriguezvega
After adding the third_party/proto files and running proto-update-deps the proto-linter was showing various path issues. Upon changing the paths to the appropriate values (e.g. import "third_party/proto/gogoproto/gogo.proto";) I hit a point where these are the error messages left
Formatting Protobuf files
{"path":"proto/router/v1/query.proto","start_line":15,"start_column":12,"end_line":15,"end_column":12,"type":"COMPILE","message":"method proto.router.v1.QueryService.Params: unknown extension google.api.http"}
{"path":"third_party/proto/google/api/annotations.proto","start_line":28,"start_column":8,"end_line":28,"end_column":8,"type":"COMPILE","message":"unknown extendee type google.protobuf.MethodOptions; resolved to third_party.proto.google.protobuf.MethodOptions which is not defined; consider using a leading dot"}
{"path":"third_party/proto/tendermint/crypto/keys.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/crypto/proof.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/types/types.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/types/validator.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/version/types.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}

If I change the package name of  gogo.proto to third_party.proto.gogoproto; and attempt to run proto-lint then I get errors when trying to resolve the types from gogo.proto
{"path":"proto/router/v1/genesis.proto","start_line":10,"start_column":44,"end_line":10,"end_column":44,"type":"COMPILE","message":"field proto.router.v1.GenesisState.params: unknown extension gogoproto.nullable"}
{"path":"proto/router/v1/genesis.proto","start_line":15,"start_column":5,"end_line":15,"end_column":5,"type":"COMPILE","message":"field proto.router.v1.Params.fee_percentage: unknown extension gogoproto.moretags"}
{"path":"proto/router/v1/genesis.proto","start_line":16,"start_column":5,"end_line":16,"end_column":5,"type":"COMPILE","message":"field proto.router.v1.Params.fee_percentage: unknown extension gogoproto.customtype"}
{"path":"proto/router/v1/genesis.proto","start_line":17,"start_column":5,"end_line":17,"end_column":5,"type":"COMPILE","message":"field proto.router.v1.Params.fee_percentage: unknown extension gogoproto.nullable"}
{"path":"proto/router/v1/query.proto","start_line":15,"start_column":12,"end_line":15,"end_column":12,"type":"COMPILE","message":"method proto.router.v1.QueryService.Params: unknown extension google.api.http"}
{"path":"third_party/proto/google/api/annotations.proto","start_line":28,"start_column":8,"end_line":28,"end_column":8,"type":"COMPILE","message":"unknown extendee type google.protobuf.MethodOptions; resolved to third_party.proto.google.protobuf.MethodOptions which is not defined; consider using a leading dot"}
{"path":"third_party/proto/tendermint/crypto/keys.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/crypto/proof.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/types/types.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/types/validator.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}
{"path":"third_party/proto/tendermint/version/types.proto","start_line":6,"start_column":8,"end_line":6,"end_column":8,"type":"COMPILE","message":"gogoproto/gogo.proto: does not exist"}

A path forward doesn't seem immediately clear to me so some further help on this would be appreciated.

The import paths and package names were not resolving due to the buf.yaml file missing from the root file directory. Adding this file in seems to squash the bugs surrounding make proto-all


Clicking "browse google drive" opens the drive modal behind the results modal, blocking function

See media attachment:

Issue no longer occurs as written

Deployment Verification

[x] Confirmed “Ready for Deploy” status
[x] Media Verifying:


removed hard coded path to /usr/bin

My ruby version located at /usr/bin is version 2.0.0p648. With regards
to rake gem, the build would error on spec.date = Date.today (it
wouldn't find the date class). From what I can tell, rake gem requires
2.3.1p122 along with require 'date' at the top of the gemspec file so
that Date#today is available. With these changes, rake gem works if
you've got your ruby set up using rbenv.

If you saw this error, I think it is a problem in RubyGem.
Which RubyGem version have you used?
[~]$ rbenv shell system
[~]$ ruby -v
ruby 2.0.0p648 (2015-12-16 revision 53162) [universal.x86_64-darwin15]
[~]$ gem -v
<IP_ADDRESS>

Hmm... same as yours.
┌── [ ~/Projects  ]
└─» ruby -v
ruby 2.0.0p648 (2015-12-16 revision 53162) [universal.x86_64-darwin15]
┌── [ ~/Projects  ]
└─» gem -v
<IP_ADDRESS>

Let me see if I can reproduce this before you merge this in.

What do you get when you run:
┌── [ ~/Projects  ]
└─» /usr/bin/ruby  --version
ruby 2.0.0p648 (2015-12-16 revision 53162) [universal.x86_64-darwin15]

I didn't find a reason of your problem yet.
However, I think your problem will be fixed with https://github.com/HipByte/motion-game/commit/5a6ebc9f8d3a05654cfe9322a7d02372e058b15c .
If you have a time, could you please confirm that ?
Thank you.

confirmed. pull request not needed.


Migration Guide v3->v4

Issue Type

[ ] Bug Fix
[ ] Feature
[x] Other

Current/Missing Behaviour
No migration guide to upgrade from v3 to v4.
Expected/Proposed Behaviour
Should have migration guide to upgrade from v3 to v4.
Additional Info
There's no point in having a migration guide unless the new version works. Blocked by #699, #704, #705, 706, #714, #715, #716

Operating System: N/A
Node version: N/A
NW.js version: N/A
Repro link: N/A

Is there a migration guide from v3 to v4?
https://github.com/nwutils/nw-builder/compare/v3.8.10...v4.0.0


How to open DrawerLayout component from button?

I can't find any function etc in DrawerLayout that I could open it with. For example if I wan't to have hamburger menu button which opens the drawer for me. Tell me if more info is needed :) Dragging works fine but having button for opening would be nice too

Have you tried calling this method:
https://github.com/kmagiera/react-native-gesture-handler/blob/master/DrawerLayout.js#L292

Sorry its not documented anywhere. Mainly wanted to match "official" drawerlayout API that has this command.
You need to get a ref to the drawer first in order to call the method. LMK if you need further assistance

Is there any update on this? I see the openDrawer function in the docs, but I'm getting an error saying "openDrawer is not a function"

Nevermind I figured it out!

Nevermind I figured it out!

Could you let me know what you did?

@cyborgium, you have to save a ref to the DrawerLayout and then call the openDrawer method on that. Something like this:
drawer = null;

<DrawerLayout
  ref={ drawer => this.drawer = drawer }
  drawerWidth={200}
  drawerPosition={DrawerLayout.positions.Right}
  drawerType='front'
  drawerBackgroundColor="#ddd"
  renderNavigationView={this.renderDrawer}>
  <View>
    <Text>
      Hello, it's me
    </Text>
  </View>
</DrawerLayout>


Strip whitespace from model URLs

What type of PR is this? (check all applicable)

[ ] Refactor
[ ] Feature
[X] Bug Fix
[ ] Optimization
[ ] Documentation Update
[ ] Community Node Submission

Have you discussed this change with the InvokeAI team?

[X] Yes
[ ] No, because:

Have you updated all relevant documentation?

[X] Yes
[ ] No

Description
This PR strips leading and trailing whitespace from URLs that are entered into either the Web Model Manager import field, or using the TUI.
Related Tickets & Documents
Closes #4536
QA Instructions, Screenshots, Recordings
Try to import a URL with leading or trailing whitespace. Should not work in current main. This PR should fix it.

Should we strip quotations too? Common problem on windows 👀

Should we strip quotations too? Common problem on windows 👀

Single, double or both?


Redirect users without memberships to orgs

https://trello.com/c/0g3MYHja/28-allow-a-orphaned-user-to-login

When a user is no longer associated with any organisation (this is possible if an admin deletes a user's only org) we need to ensure that, once authenticated, they associate themselves with an organisation.
This PR makes sure the orphaned user is redirected to the organisations#new action for this scenario.
The user may still raise support requests while in an orphaned state.

@tommotaylor I think I've covered all your review comments now, I've redirected to the change organisation page with the suggested message.


Feature/add ripgrep

use ripgrep to search for directories in the QuickOpen
Outstanding

[ ] get binaries and set paths for mac and windows (windows seems to be ignored by vcs)
[x] figure out how to handle immense result current cwd returns 4900+ results
which is super laggy ?Flatlist

Future Optimisation

only download Rg based on release version e.g. only mac for mac

Future Optimisation
only download Rg based on release version e.g. only mac for mac

I think we need that in general for the vendored bits, its just a case of being smart about how we do it. We don't want to end up in the same position we had with Oni1, where we kept consistently hitting GitHub API limits that caused CI to fail. I'm not sure if we ever found a real solution to it though.
It obviously needs to work for both local and CI, so hosting stuff isn't especially an option, unless we have both a normal and a CI setup step. A simple JS download script would work fine locally? Or we can go the route I've seen in other tools and have the binaries in their own repos, since I think cloning is always okay from GitHub.

@bryphe there is/was actually quite a lot to this and I was initially hoping to work iteratively on this rather than try and solve all the problems in one PR
The escape bug should now be fixed. Re. the filtering and the input, to be clear I saw that as being a separate piece of work, the current filtering is very basic (and is a placeholder). Actually trying to nail that in this PR as well I think is a pretty big scope creep since its a whole other piece of functionality, my aim here was to just get ripgrep working rather than "finish" the quick open experience

there is/was actually quite a lot to this and I was initially hoping to work iteratively on this rather than try and solve all the problems in one PR

Got it, thanks! I think what would help me is to have the remaining items called out either in a list or as issues (I wasn't sure what was going to be later work, and what was a bug when testing it).

The escape bug should now be fixed.

Sweet 👍

Re. the filtering and the input, to be clear I saw that as being a separate piece of work, the current filtering is very basic (and is a placeholder). Actually trying to nail that in this PR as well I think is a pretty big scope creep since its a whole other piece of functionality (filtering strategy, handling throttling/or debouncing etc), my aim here was to just get ripgrep working rather than "finish" the quick open experience

That makes sense - the filtering is more broadly applicable to the Menu rather than specific to the QuickOpen. Approved - thanks for your work on this!


update to cookies content

Trello card
https://trello.com/b/lRXllOHK/git-website-sprint-board
Context
Our cookies were recently updated to reflect changes needed with adopting VWO and MS Clarity, however Gemma identified there may be some other aspects needing review.
Changes proposed in this pull request
Guidance to review

Need to review whether or not 'cookies' should be capitalised or not, both used at different places in the page but i'm not sure what the right version is? @GayeCord

Thanks Caitlin, the changed to the cookie preference page look good to me. However (and this could be me), the modal page still looks like it has the incorrect info within it and doesn't look to have returned to the 'accept cookies' or 'set cookie preferences' view

@caitlin-reidy I think you've updated the right page now - but the changes to the modal (the acceptance component) are still on this PR - they just need removing!

Have deleted Internet explorer and added safari now @GayeCord

Thanks for the changes to the web browser help section @caitlin-reidy. I've just found a couple of additional errors in the Tracking and Performance section, please can you look at these too:

MUID cookie - the description is missing the leading I
SM cookie - has the wrong description
SRM_B - has the wrong description
FPID - showing as SFPID. The S need to be removed.
Thanks

@GayeCord have corrected those mistakes now


Updates

Updates and old recipe and re-builds for the new texinfo.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Maximum number of calls to same client at same time

Description
I made an app in which when a call comes in to a twilio number, the call is redirected to twilio client (user) which is in my system is mapped to my system's user.
Now the app can login the same user on multiple devices (e.g user is concurrently logged in the iphone and ipad). I am only talking about the iOS side.
When an incoming call comes in, i made the code in the twilio cloud function and through my own custom logic, the number is decoded to get the corresponding user in our system, and then call is redirected to twilio_client (which is mapped against the number).
I have tested with 2 devices at same time, and both iphones were getting calls at same time (which is good).
My question is if there is a limit to that, e.g only 10 devices will get calls for same twilio_client, and should i even be doing this, or make a system in which only 1 login is done at 1 time etc
I basically want to know this limitation, if it even exists or not. I tried searching the documentation but i couldn't find this specific limit.
Steps to Reproduce

name a user client:test_user
get access token on multiple devices at same time
make an outgoing call to client:test_user
all devices (i tested with 2) are getting calls at same time
whoever picks up first will get the call routed

Expected Behavior
I just want to know if there is any limit of concurrent calls to multiple devices at same time to same user
Actual Behavior
I cannot test with alot of iphones, so thats why asking if there is limitation
Versions
twilio-cli version 2.18.0
Voice iOS SDK
6.2.0
Xcode
12.4
iOS Version
14.4, 14.6
iOS Device
6s, 11 pro, 12 pro

Thanks for reaching out @saqibdb and kudos to the nice and clear issue description 👍.
AFAIK there is no such flag to configure the number of clients, with the same identity under the Account SID, to receive the incoming call notification at the same time.
For what it's worth, there is a hard limit of the Voice service by which only the last 40 devices registered with the same identity will receive the incoming call push notification, in case you have more than this number of clients with the same identity at the same time.

@bobiechen-twilio Thats perfect, so 40 is the number on which i put a limit on my system as well
Thanks for the quick response


KittiMOTS detection file does not have rle

Hi. Ocentintas
Thank you for your great work in MOTS.
I have a question about KittiMOTS detection file.
when I run the code python scripts/train.py with KittiMOTS datasets.
I got this error.
ValueError: Length mismatch: Expected axis has 9 elements, new values have 10 elements
I check Kittiloader code and I realize column has 'rle' which is related to mask but detection file which is dowloaded with yours does not have mask representations.
How can I train KITTI MOTS?

Thanks for pointing this out, I've updated the KITTI loader. Now, you will not encounter this error with the latest version of the code.


Add the ability to sync tags from voicenotes

latest version 0.0.10 now syncs tags into note -- need to hit manual sync for existing notes
Cc: @qwertydude


Match headers in Old and New Publish

Context
We're migrating Old Publish to New Publish one piece at a time. The top nav of both apps need to look the same and contain the same info.
Changes proposed in this pull request
Commits provide more detail, but essentially:

Copy the implementation from Old Publish, reworking it to comply with the component-based header we have in this codebase
Ensure this header works across Support, Publish Interface, and the sign in page
Make sure we're linking to Old Publish as the homepage and notifications link, for now

Guidance to review

Should be straightforward to review in a Review app, worth covering sign in page, support, and a publish interface page via one of the personas (manual editing of the URL required to see anything other than the empty root page).

Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Make navigation bar in New Publish the same as Old Publish


Using unescaped unicode literals in source code is not precise enough

The main problem that could occur using unicode literals is that some unicode characters are nearly identical. Moreover, this is made worse by fonts that do not show unicode characters.
Instead I could use '\x006E' and other such escape sequences. These would be more precise, and appear clearly on most developers monitors without having to install an extra font. Of course, what glyph an escape sequence refers to is not obvious, and would have to be looked up by the programmer. But at least we can be assured that the code point is unique.
I think we can use unicode characters in the comments, or in some test cases (for the sake of readability), but we should be careful with the problem of look-alike Unicode characters, and have the actual code points nearby.

I defined the unicode code points as constants in a module called IPAUnicodeConstants. Several other modules were modified to use those constants. Test code still uses unescaped unicode literals in many places, and I don't feel that test code should change.


Wrong scope for nested defs in class methods

class SomeClass
	def self.class_method
		def nested_proc
			p 'in nested_proc'
		end
	end
end

SomeClass.class_method
SomeClass.nested_proc

nested_proc gets defined on SomeClass, not in local scope of class_method.

I think this is the same bug. Just noticed my def statements were creating global functions when I use them in the context of an object - instead of singleton methods like I expected. (You can drop the instance_eval and get the same result with a standard def.)
MRuby
class Test
  def initialize
    self.instance_eval <<EOS
    def eigen_method
      raise "I shouldn't be accessible globally!" unless self.class == Test
      :success
    end
EOS
  end
end
test = Test.new
test.eigen_method
#=> success
eigen_method
#=> RuntimeError: I shouldn't be accessible globally!

MRI
[jared:~/projects/rubium/samples/mruby-edit] irb
irb(main):001:0> class Test
irb(main):002:1>   def initialize
irb(main):003:2>     self.instance_eval <<EOS
irb(main):004:2"     def eigen_method
irb(main):005:2"       raise "I shouldn't be accessible globally!" unless self.class == Test
irb(main):006:2"       :success
irb(main):007:2"     end
irb(main):008:2" EOS
irb(main):009:2>   end
irb(main):010:1> end
=> nil
irb(main):011:0> test = Test.new
=> #<Test:0x007fca4a4247a0>
irb(main):012:0> eigen_method
NameError: undefined local variable or method `eigen_method' for main:Object
	from (irb):12
	from /usr/bin/irb:12:in `<main>'

I fear this is beyond my depth to debug, but I'll give it a shot when I have time (if nobody beats me to it... and I hope they do).

Fix for #2936 went in recently and looks related. I'll try to test this today after work and see if it's fixed.

I guess they weren't the same bug after all.
My case works now
class Test
  def initialize
    self.instance_eval <<EOS
    def eigen_method
      raise "I shouldn't be accessible globally!" unless self.class == Test
      :success
    end
EOS
  end
end
test = Test.new
test.eigen_method
#=> success

eigen_method
#=> NoMethodError: undefined method 'eigen_method' for main

class Test
  def initialize
    def eigen_method
      raise "I shouldn't be accessible globally!" unless self.class == Test
      :success
    end
  end
end
eigen_method
#=> NoMethodError: undefined method 'eigen_method' for main

Origin bug still present
class SomeClass
    def self.class_method
        def nested_proc
            'nested_proc called'
        end
    end
end

SomeClass.class_method
#=> nested_proc
SomeClass.nested_proc
#=> 'nested_proc called'

On MRI the SomeClass.nested_proc call gives an NoMethodError.


Redesign 2021

Hi All,
Just pushing up some of the changes outlined in JuliAnn's Figma files to start the preview/review process.
Quick overview of the changes:

Totally new homepage styling
New favicon
Base fonts changed to: Trade Gothic (headings), Fira Mono (accent), Noto Sans (body)
Code blocks have a different background
Note/Tip modules looks a bit different
Footer on the home page is a copy of the one that will be live on mm.com for the relaunch
Same things for the navigation, but this will be a global change on all docs pages.

Similar to the code blocks and note sections, let us know if you see any other types of content that need a bit of redesign-love.

@emdecr I noticed that the search bar is visible on the home page for the mobile layout, but not the desktop layout. I think this is an oversight on my end with the initial design, can we add the search bar to the home page desktop layout too?

@BenLloydPearson @cwarnermm With my latest push (I don't think the preview site has updated at the time of me writing this comment), we're adding the search back on desktop and mobile, in a slightly different treatment.
JuliAnn's also added some new options for search on the actual docs pages (she'd prefer if the search icon were not in the nav next to to the logo). I'm gonna see what's possible and circle back.

@BenLloydPearson @cwarnermm
I've implemented JuliAnn's proposals for search:

The search bar appears on the homepage for both mobile and desktop
For internal docs pages, the search icon has been removed from the top-left spot in the nav, and moved under the breadcrumb links beside the Edit link

I see a go-ahead/approval from Ben in a comment above, but Carrie obvs let us know if the team has any more pieces of feedback. I remember you mentioning it would be shared with more folks. As JuliAnn is out after tomorrow, I'm not sure any large design-related changes can be implemented, but we can see how we can assist. TY!


Set multiple sparse checkout paths in one command/job

For pipelines with many paths (e.g. core ci), it can be annoying to have so many separate jobs show up in the UI, one for each path. Additionally, sometimes git tries to do garbage collection and it can take a lot longer to run many git sparse-checkout add commands as opposed to a single set command. This PR changes the behavior to parse the yaml paths parameter object into the script, and run a single command to set the sparse checkout paths. I use invoke-expression here because otherwise powershell treats the space-separated string as a single parameter and it all ends up on the same line in the sparse checkout config file.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


extract features

Hi！How to use this code to extract features of other videos, such as ucf101？

Same question!

Hi！How to use this code to extract features of other videos, such as ucf101？

Hi, have you solved this problem? I also want to use this code to extract features of other dataset? How can I start? change the "root" and "split" in extract_features.py? And what others should I do, I just started learning this。

Hi！How to use this code to extract features of other videos, such as ucf101？

Hi, have you solved this problem? I also want to use this code to extract features of other dataset? How can I start? change the "root" and "split" in extract_features.py? And what others should I do, I just started learning this。
have you solved this problem? what should I do？


keep the ratio when using fetchImageWithSize

Is there an option to detect or keep the ratio when using this size one? In that case, the size specified would be the max height and the max width for the compressed image.
For example, if I specify the size to be CGSize(width: 1200, height: 1200), the image will always be cut into 1200 * 1200 square. If the original ratio is 4:3 and 2000:1500, I'd like to have the UIImage to be 1200 * 900. Thanks!

Actually I used fetchFullScreenImage instead which keeps the original ratio and is much faster than fetchOriginalImage

@zhangao0086 First of all i want to say thank you for such a great library! but i want to ask same question, how to keep the ratio with fetchImageWithSize? when i use fetchOriginalImage app would crash but if im using fetchOriginalImage its too small.

(kinda) found the solution. I changed func in fetchOriginalImage in DKAsset from
getImageManager().fetchImageForAsset(self, size: PHImageManagerMaximumSize, options: options) to getImageManager().fetchImageForAsset(self, size:  CGSize(width: 1500, height: 1500), contentMode: .AspectFit) it keeps the ratio with 1500px maximum pixel

I can get it :
asset.fetchOriginalImageWithCompleteBlock({ (image, info) in
let width: CGFloat = image!.size.width
let height: CGFloat = image!.size.height
                        let realWidth: CGFloat = 740  //the max width
                        var realHeight: CGFloat = 0
                        if width < realWidth {
                            realHeight = height
                        } else {
                            realHeight = realWidth * height / width
                        }

                        asset.fetchImageWithSize(CGSizeMake(realWidth, realHeight), completeBlock: { (image, info) in
                            if let uploadImag  = image { // this is the image to keep the ratio

                            }
                        })

                    })

@atongmu518 so basically you get the image twice. I think it may still crash if you select too many photos. For me, it crashed after selecting 6 or more on iphone 6 plus.

@zhiyanshao
another solution is extension the DKAssect, just like @aldolazuardi
extension DKAsset {
    func getImageWithSize(size:CGSize, completeBlock: (image: UIImage?, info: [NSObject : AnyObject]?) -> Void) {
        getImageManager().fetchImageForAsset(self, size:size , contentMode: .AspectFit) { (image, info) in
            completeBlock(image: image, info: info)
        }
    }
}

Hi all, first of all I'm sorry for the late response...
In the latest version, I've optimized for fetching original image. I think I've solved the crash issue.
(You can try to selecting 100 photos :smiley: )
And I've add an API to fetch image with contentModel:
public func fetchImageWithSize(size: CGSize, options: PHImageRequestOptions?, contentMode: PHImageContentMode, completeBlock: (image: UIImage?, info: [NSObject : AnyObject]?) -> Void)

By default,  the fetchImageWithSize method fetching image with .AspectFit.
Let me know if you have any questions :)


Cross-platform support

This refers mostly to the hard-coded "mono " prefix to the invocation of .NET .exe binaries. Those should be avoided under Windows.

I just picked up another bug in Windows: if the project file is in a directory that contains spaces in it (e.g., C:\Code\My Fancy Project\Fancy Project.fsproj), then most functions fail with it (make, FSharpBuildProject, FSharpRunProject). The reason is that fnameescape is used instead of shellescape.

(1) I don't know if it's caused by my lack of mono or something else, but this is frustrating:

After I see that, it loads up the file and syntax highlights it anyways.
(2) Is there a way to just turn off autocompletion, so I can debug this crash?
(3) The logs (enabled by let g:fsharpbinding_debug = 1) seem to suggest it's ignoring my g:fsharp_xbuild_path.
https://gist.github.com/killerswan/d710d16b35e725e145cc


Support batch adding project members

Version: main
UI: dev-c978228
Commit ID: 50733051d3b5c948ccd6006769f7d5a4e684c00e
Env: AWS EC2 Ubuntu 22.04 amd64
Currently, you can only add project members one by one. It's inconvenient when adding multiple project members.

Environment

Walrus version: main e92cde6256262c9671c9d51475f0bee8669f6628
UI version: dev-606a1a9

Test Result: pass


QR codes not updating

I've been experimenting with the samples and also doing my own testing with a Unity-based project using the new XR_MSFTX_scene_marker extension in the preview runtime to detect QR codes.
I'm finding that XRSceneMarkerMSFT objects don't ever seem to update within an XR session. That is, the lastSeenTime and size properties remain constant even if the QR code remains in view (in which case I would expect lastSeenTime to always be recent) and its size is changing over time (in which case I would expect the size property to reflect the changes). This appears to be the case whether the code is first spotted during the session or whether it was an existing code that existed before the session and stuck around.
This can be verified by running the QR sample, resizing a virtual QR code on a screen and observing that the visualization does not change (other than location). When dropping back to the HoloLens shell the system QR code visualization (green outline) immediately becomes responsive to the size changes of the same QR code. Relaunch the sample and observe that the 'most recent' size is now reflected in the visualization but it will not change again (other than location).
Is this intended behaviour or perhaps a bug in the runtime?

Thank you for reporting this issue. These issues should now be fixed in the latest refresh of the runtime: 113.2309.7002. Please check the last seen time with this new version as well as try resizing the marker. These changes should now be getting reflected.


Import the client library rocc-angular

https://github.com/Sage-Bionetworks/rocc-angular

@vpchung Hopefully the instructions given in the client library generated enable you to easily import the library in this app.


Make kubed  subchart of Epinio (small) chart

Part of https://github.com/epinio/epinio/discussions/1190

Let's wait until the work for minio is done: https://github.com/epinio/helm-charts/pull/96 so we don't duplicate efforts.

Moved here: https://github.com/epinio/epinio/issues/1217


Picker Doesn't work in Webapp

The picker doesn't work when you add to home screen a webpage that is a F7 app. (aka as an apple webapp)
    <meta name="apple-mobile-web-app-capable" content="yes">
    <meta name="apple-mobile-web-app-title" content="Webapp Name">
    <meta name="apple-mobile-web-app-status-bar-style" content="black">

It works great through safari and other browsers. I will investigate but I don't have time right now, on a tight timeline for a project.

Thanks man, you rock and your library rocks too.


NOTICE: agave font creator's github username is now "blobject"

I recently changed my github username from "agarick" to "blobject".
Please change your links that reference this repository to:
https://github.com/blobject/agave

Notice placed at top of readme. Will keep it there until the next revision or two.


Hide internal parameter properties in blazor from the IntelliSense

Describe the bug
Is there a way to hide an internal property, which has the [Parameter]attribute, e.g.:
    public class ParentComponentBase : ComponentBase
    {
        [Parameter]
        internal string Value { get; set; }
    }`

Using EditorBrowsable does not seem to work either:
    public class ParentComponentBase : ComponentBase
    {
        [Parameter]
        [EditorBrowsable(EditorBrowsableState.Never)]
        internal string Value { get; set; }
    }

Expected behavior
The internal property is not visible.
Screenshots

Thanks for contacting us, @Stamo-Gochev.
What do you need this for?

@mkArtakMSFT I need this in order to hide an internal setting of a component. The scenario is that a component can be used as a standalone unit, but it can be also integrated in another component, e.g. in a hierarchy. Let's say an editor component can be used in a form, but it can also be used as an editor in a grid component, which requires some internal customizations that are not (currently) relevant to be used in the standalone case.
I think that the EditorBrowsable attribute should handle such scenarios as it serves a more general purpose (and my concrete scenario might not be the best use case). Still, it seems like [Parameter] props cannot be hidden for some reason.

For what it's worth, I've never seen EditorBrowsable respected within the same project or while there's a direct reference to the source. On top of that, [Parameter] properties can be private and they're still visible when writing Blazor, BUT they're going to change that: #8825. That might be the solution you want in the first place.

Thanks @Stamo-Gochev.
This is not something we plan to do.

@Stamo-Gochev The reason is that there are other ways this can be achieved, and we're not sure that [EditorBrowsable] is a desirable way to do it. For example:

You could choose not to declare any [Parameter] property for your internal parameter, and instead receive its value privately by overriding SetParameters on the receiving component (read it directly from the incoming ParameterCollection)
Or, instead of using a parameter for this parent/child communication, have the parent pass an instance of itself to the child so the child can read stuff back from the parent
Or, capture the child using a ref and then make calls to its public methods to perform the private communication between parent and child

I hope one of those is suitable for your use cases!

Thanks @SteveSandersonMS, this makes sense.


undefined method `current' for Neo4j::Transaction:Module Did you mean? current_for

Hi I am using rails

I often getting this error while creating the new user and sometimes some form submission.
Additional information which could be helpful if relevant to your issue

neo4j (7.0.3)
activemodel (>= 4)
activesupport (>= 4)
neo4j-core (>= 6.0.0)
orm_adapter (~> 0.5.0)
devise-neo4j (2.1.1)
devise (>= 3.5.2)
neo4j (>= 3.0.0)
railties (>= 4.2)
neo4j-core (7.1.0)
neo4j-rake_tasks (>= 0.3.0)
neo4j-rake_tasks (0.7.12)
os
rake
ruby-progressbar
rubyzip (>= 1.1.7)
Neo4j database version:
neo4j gem version:
neo4j-core gem version:

It would be good to know the specific version of neo4j-core (check your Gemfile.lock) you are using, but my guess is that if you are using a newer version of neo4j-core with version 7.0.3 of neo4j, that there might be an incompatibility.  If you are using a newer version of neo4j-core, then you might try playing around with older versions if you don't plan on upgrading the neo4j gem


❓ Unable to freeze tensor of type Int64/Float64 into constant layer, try to compile model with truncate_long_and_double enabled?

❓ RuntimeError: [Error thrown at core/conversion/converters/converter_util.cpp:251] Unable to freeze tensor of type Int64/Float64 into constant layer, try to compile model with truncate_long_and_double enabled:

A pre-trained Torch model like Resnet18 was loaded
The model was quantized using pytorch_quantization.quant_modules.initialize()
The quantized model was calibrated
The model was fine-tuned (QAT)
I tried to convert the fine-tuned model to TensorRT using
trt_mod = torch_tensorrt.compile(qat_model,                               inputs=[torch_tensorrt.Input([32, 3, 32, 32])],                               enabled_precisions={torch.int8})
but I encountered the error below:

File "/home/i2027/anaconda3/envs/p/lib/python3.10/site-packages/torch_tensorrt/_compile.py", line 133, in compile
    return torch_tensorrt.ts.compile(
File "/home/i2027/anaconda3/envs/p/lib/python3.10/site-packages/torch_tensorrt/ts/_compiler.py", line 139, in compile
    compiled_cpp_mod = _C.compile_graph(module._c, _parse_compile_spec(spec))
RuntimeError: [Error thrown at core/conversion/converters/converter_util.cpp:251] Unable to freeze tensor of type Int64/Float64 into constant layer, try to compile model with truncate_long_and_double enabled
I have checked the model parameters and all of them were of type float32. I don't know why TorchTensorRT complains about Int64/Float64! Please note that I have managed to convert a simple CNN to TensorRT using the method described above successfully. However, I failed to convert an existing torchvision model using the steps above. I will be grateful for any hint.
Environment

PyTorch Version: 2.0.1+cu118

CPU Architecture: x86

OS: Ubuntu 20.04

How you installed PyTorch: pip

Python version: 3.10.11

CUDA version: 12.1

GPU models and configuration: GeForce GTX 1080 - 12 GB

All pckages versions:

torch==2.0.1+cu118

torch_tensorrt==1.4.0

torchvision==0.15.2+cu118

pytorch_quantization==2.1.2

torchvision==0.15.2+cu118

Additional context
The code is available at https://github.com/panahikhas/TensorRT-QAT/blob/main/torch-tensorrt-QAT.py to reproduce the results.

Thank you very much for your response.
If I add the argument truncate_long_and_double=True the accuracy become 10% from 80%.
I don't know the reason but the problem was from PyCharm. I was working with it and it had errors. But when I used the terminal or VSCode there was no problem and no warning/error about int64.


Fix provider.get_setting choices logic.

Description

choices were ignored because of converter=str in function arguments and first if was if converter is str:.
for https://github.com/elgatito/script.elementum.burst/pull/388
Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Translation

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

I had to revert this change because on Brazilian providers all content only displayed the 720p icon, and only showed the page title in all content.

@icarok99 this change is correct. if you changed code of elementum or burst - then i think you need to fix new code that uses get_setting() incorrectly. or something else, since this change is only for settings not for icons or else.
and obviously you need appropriate changes on burst side, like https://github.com/elgatito/script.elementum.burst/pull/388
https://github.com/elgatito/script.elementum.burst/blob/f894204ac75a6010a33652aaffd64eadb7ba4567/burst/utils.py#L130
anyway, "revert" is not a solution.

@icarok99 this change is correct. if you changed code of elementum or burst - then i think you need to fix new code that uses get_setting() incorrectly. or something else, since this change is only for settings not for icons or else. and obviously you need appropriate changes on burst side, like elgatito/script.elementum.burst#388 https://github.com/elgatito/script.elementum.burst/blob/f894204ac75a6010a33652aaffd64eadb7ba4567/burst/utils.py#L130
anyway, "revert" is not a solution.

The reason for the rollback is when I apply this change https://github.com/elgatito/script.elementum.burst/pull/354

After I apply it, it looks like the image above.

@antonsoroko These are the changes
https://github.com/elgatito/script.elementum.burst/compare/master...icarok99:script.elementum.burst:master

@icarok99 that's a lot of changes. After quick reading i can't say what is wrong.
As I wrote above - create a new issue in burst repo.
Also add your modified providers.json to the diff.

@icarok99 try to apply this change https://github.com/elgatito/plugin.video.elementum/commit/d897e0ddc5a5c0de097265cca091ade45212e22d
maybe it will help you.
(i am not sure if @elgatito made that fix by accident or after reading this conversation)
if it will not help - create a new issue.

@icarok99 let me repeat: please try everything i posted above. and please stop writing here. please create a new issue.
for example, you have not said if you applied related changes to burst - see https://github.com/elgatito/plugin.video.elementum/pull/994#issuecomment-2096259748
i think you have not update your code to use get_setting function correctly.
and please, no screenshots of code, no diffs of 8000 lines, just show the related new code, e.g. your new code in burst that uses get_setting function.
and again - create new issue.

@icarok99 let me repeat: please try everything i posted above. and please stop writing here. please create a new issue.
for example, you have not said if you applied related changes to burst - see #994 (comment)
i think you have not update your code to use get_setting function correctly.
and please, no screenshots of code, no diffs of 8000 lines, just show the related new code, e.g. your new code in burst that uses get_setting function.
and again - create new issue.

The problem was resolved and the post closed


Plugin not working for eclipse in ubuntu 14.04

I installed this plugin for eclipse on my ubuntu 14.04 laptop but didn't find any option to import or start new bazel project.

This should be under "Import Project..." > "Others".


[WIP] move to v3.4.1 for the pause image

issue: https://github.com/containerd/containerd/issues/5013
Wait for https://github.com/kubernetes/kubernetes/pull/98205 to be merged
Signed-off-by: Iceber Gu<EMAIL_ADDRESS>

Let's please wait for a release of k8s (1.21) before we merge this change in. (at least a release candidate)

@dims I feel like we should have a written plan somewhere about the ideal flow of pause updates, previously we qualified these at the very least with dockershim xref: https://github.com/kubernetes/kubernetes/pull/98205#issuecomment-774579069
It's not clear to me why this needs to wait for a k8s release though, it should be independent?
The image is already publicly available to consumers and won't be mutated, so 3.4.1 will be fine as long as it works here.
What's the concern with updating before a k8s release?

Presumably any CI here creating pods tests this change?

@BenTheElder i'd like to get the https://github.com/kubernetes/kubernetes/pull/98205 merged first.
then i'd like to watch periodics for a bit - example : https://testgrid.k8s.io/sig-node-release-blocking#node-kubelet-master
[dims@dims-a01 22:10] ~/Downloads ⟩ rg 'container-runtime="docker"'
tmp-node-e2e-f61ec3da-cos-81-12871-1245-6/kubelet.log
38:Feb 07 01:28:33 tmp-node-e2e-f61ec3da-cos-81-12871-1245-6 kubelet[1861]: I0207 01:28:33.020269    1861 flags.go:59] FLAG: --container-runtime="docker"

tmp-node-e2e-f61ec3da-ubuntu-gke-1804-1-16-v20200330/kubelet.log
37:Feb 07 01:28:35 tmp-node-e2e-f61ec3da-ubuntu-gke-1804-1-16-v20200330 kubelet[3752]: I0207 01:28:35.342940    3752 flags.go:59] FLAG: --container-runtime="docker"

tmp-node-e2e-f61ec3da-cos-85-13310-1209-7/kubelet.log
38:Feb 07 01:28:33 tmp-node-e2e-f61ec3da-cos-85-13310-1209-7 kubelet[1975]: I0207 01:28:33.349476    1975 flags.go:59] FLAG: --container-runtime="docker"

to confirm things are fine, before we make the change here in containerd.

@dims how long do we want to monitor upstream? It's been a few weeks now.

/ok-to-test

@BenTheElder we resurrected the pull-containerd-node-e2e job a couple of days ago. kicking that off. once it goes green, let's go ahead with this.
LGTM

@mikebrow @dims @BenTheElder rebased and it's green.

still tracking some issues with the node e2e tests both here and in k/k. Looks like the test suite passed here but the job is not marked pass/fail. so let's just treat this as 🟢 and move forward.
As long as the windows is a flake, so we may have to rerun it.

I'm prepared to roll forward with this in kind, I don't expect any issues with this image.


Link to CodePlayer.tv

In your readme you point to CodePlayer.tv as an easier way to create the JSON:

Scenario is a JSON file with a list of actions. You could craft scenarios by hands, but that quickly becomes too tedious job for a non-trivial scenario. That's why I've created an editor, to streamline the scenarios production.

But this link doesn't appear to work (even if I go directly to CodePlayer.tv). Is there a location where the editor exists?
This tool is really cool! Thanks for building it!

The service does not exist yet, unfortunately.

OK, thanks for the prompt response!


Change default RSA key size to 2048 bit

Fixes: #8737

I've forced pushed a new commit

BTW, is this a security-related fix per se? I'm thinking of the 1.1.0 and 1.0.2 labels...

You really shouldn't be using 1024 bit keys anymore. People using
genpkey still get 1024 bit keys, while things like genrsa,
dhparam, and req where already changed to 2048 in 1.0.2.

... while things like genrsa, dhparam, and req where already changed to 2048 in 1.0.2.

Thanks, that's justification enough

@kroeckx can this be merged now?

I'll do it if I have some time, but feel free to do it.

Needs a rebase..


DynamoDB config

Is there an existing issue for this?

[X] I have searched the existing issues

YACE version
0.53.0 (docker)
Config file
apiVersion: v1alpha1
discovery:
  jobs:
    - type: AWS/Lambda
      regions:
        - eu-west-3
      period: 300
      length: 300
      metrics:
        - name: Invocations
          statistics: [Sum]
        - name: Errors
          statistics: [Sum]
        - name: Throttles
          statistics: [Sum]
        - name: Duration
          statistics: [Average, Maximum, Minimum, p90]
    - type: AWS/DynamoDB
      dimensionNameRequirements:
        - TableName
        - GlobalSecondaryIndexName
        - Operation
      regions:
        - eu-west-3
      period: 300
      length: 300
      metrics:
        - name: SuccessfulRequestLatency
          statistics: [Average]

Current Behavior
It might not be a bug, but I'm having trouble getting the scraping to work for AWS/DynamoDB (It works fine with: https://github.com/prometheus/cloudwatch_exporter/tree/master and this configuration: https://github.com/prometheus/cloudwatch_exporter/blob/master/examples/DynamoDB.yml).
error :
2023-08-17 16:29:31 {"account":"XXXXXXX","arn":"","error":"expected to discover resources but none were found","job_type":"AWS/DynamoDB","level":"error","msg":"No tagged resources made it through filtering","region":"eu-west-3","time":"2023-08-17T14:29:31Z"}

Expected Behavior
metrics of dynamoDB exported
Steps To Reproduce
No response
Anything else?
No response

+1 I have the same issue, yace wont get DynamoDB metrics.
CLoudwatch exporter with above config works.

@vinbh i have solved for now with that :  (instead of "discovery")
 customNamespace:
    - name: AWS/DynamoDB
      namespace: AWS/DynamoDB
      regions:
        - eu-west-3
      metrics:
        - name: SuccessfulRequestLatency
          statistics:
            - Average
        - name: ThrottledRequests
          statistics:
            - Sum
        ...

@modship  hi, Can you share the whole config yaml? I really need it. Thank you

@modship hi, Can you share the whole config yaml? I really need it. Thank you

im using ansible
yace_exporter_configuration:
  apiVersion: v1alpha1
  discovery:
    jobs:
      - type: AWS/EC2
        regions:
          - eu-west-3
        period: 300
        length: 300
        metrics:
          - name: CPUUtilization
            statistics: [Average]
          - name: NetworkIn
            statistics: [Average, Sum]
          - name: NetworkOut
            statistics: [Average, Sum]
          - name: NetworkPacketsIn
            statistics: [Sum]
          - name: NetworkPacketsOut
            statistics: [Sum]
          - name: DiskReadBytes
            statistics: [Sum]
          - name: DiskWriteBytes
            statistics: [Sum]
          - name: DiskReadOps
            statistics: [Sum]
          - name: DiskWriteOps
            statistics: [Sum]
          - name: StatusCheckFailed
            statistics: [Sum]
          - name: StatusCheckFailed_Instance
            statistics: [Sum]
          - name: StatusCheckFailed_System
            statistics: [Sum]
      - type: AWS/Lambda
        regions:
          - eu-west-3
        period: 300
        length: 300
        metrics:
          - name: Invocations
            statistics: [Sum]
          - name: Errors
            statistics: [Sum]
          - name: Throttles
            statistics: [Sum]
          - name: Duration
            statistics: [Average, Maximum, Minimum, p90]
      - type: AWS/ApiGateway
        regions:
          - eu-west-3
        period: 300
        length: 300
        metrics:
          - name: Latency
            statistics: [Average, Maximum, p95, p99]
          - name: Count
            statistics: [SampleCount, Sum]
          - name: 4xx
            statistics: [Sum]
          - name: 4XXError
            statistics: [Sum]
          - name: 5xx
            statistics: [Sum]
          - name: 5XXError
            statistics: [Sum]
          - name: IntegrationLatency
            statistics: [Average]
  customNamespace:
    - name: Bridge
      namespace: Bridge
      regions:
        - eu-west-3
      metrics:
        - name: bridge-owner-xma-balance
          statistics:
            - Average
          period: 300
          length: 300
        - name: bridge-owner-eth-balance
          statistics:
            - Average
          period: 300
          length: 300
        - name: tDAI-supply-diff
          statistics:
            - Average
          period: 300
          length: 300
        - name: WETH-supply-diff
          statistics:
            - Average
          period: 300
          length: 300
        - name: failed-burnRedeem
          statistics:
            - Average
          period: 300
          length: 300
        - name: failed-lockMint
          statistics:
            - Average
          period: 300
          length: 300
    - name: AWS/DynamoDB
      namespace: AWS/DynamoDB
      regions:
        - eu-west-3
      metrics:
        - name: SuccessfulRequestLatency
          statistics:
            - Average
        - name: ThrottledRequests
          statistics:
            - Sum
        - name: UserErrors
          statistics:
            - Sum
        - name: ConsumedReadCapacityUnits
          statistics:
            - Sum
        - name: ConsumedWriteCapacityUnits
          statistics:
            - Sum
        - name: ProvisionedReadCapacityUnits
          statistics:
            - Average
        - name: ProvisionedWriteCapacityUnits
          statistics:
            - Average
        - name: ReturnedBytes
          statistics:
            - Average
        - name: ReturnedItemCount
          statistics:
            - Sum

@modship  thanks for your response.
below is my configuration.
but it never collects SuccessfulRequestLatency metric.
Do you have any doubts about what the problem might be?
I really need it .
    discovery:
          jobs:
          - type: dynamodb
            regions:
              - ap-northeast-2
            enableMetricData: true
            metrics:
              - name: ThrottledRequests
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ConsumedReadCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ProvisionedReadCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ConsumedWriteCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ProvisionedWriteCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
    customNamespace:
          - name: dynamodb
            namespace: AWS/DynamoDB
            regions:
              - ap-northeast-2
            metrics:
              - name: SuccessfulRequestLatency
                statistics:
                - Average
                period: 60
                length: 60

@modship thanks for your response. below is my configuration. but it never collects SuccessfulRequestLatency metric. Do you have any doubts about what the problem might be? I really need it .
    discovery:
          jobs:
          - type: dynamodb
            regions:
              - ap-northeast-2
            enableMetricData: true
            metrics:
              - name: ThrottledRequests
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ConsumedReadCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ProvisionedReadCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ConsumedWriteCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ProvisionedWriteCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
    customNamespace:
          - name: dynamodb
            namespace: AWS/DynamoDB
            regions:
              - ap-northeast-2
            metrics:
              - name: SuccessfulRequestLatency
                statistics:
                - Average
                period: 60
                length: 60

maybe your indentation for - Average

@modship  This is my final revision. However, the SuccessfulRequestLatency metric is still not being collected. Anyway, thank you for your help!
    discovery:
          jobs:
          - type: dynamodb
            regions:
              - ap-northeast-2
            enableMetricData: true
            metrics:
              - name: ThrottledRequests
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ConsumedReadCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ProvisionedReadCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ConsumedWriteCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
              - name: ProvisionedWriteCapacityUnits
                statistics:
                - Sum
                period: 60
                length: 60
    customNamespace:
          - name: AWS/DynamoDB
            namespace: AWS/DynamoDB
            regions:
              - ap-northeast-2
            metrics:
              - name: SuccessfulRequestLatency
                statistics:
                   - Average
                period: 60
                length: 60


Beta7 - Kestrel does not work on Windows Server 2008 R2 and Windows Server 2012 R2

You end up getting the following error:

This could also happen in other variations of Windows, these are just two that I noticed the issue in.

/cc @loudej @halter73

I believe you need the Visual C++ 2015 redistributable installed for DNX:
https://www.microsoft.com/en-us/download/details.aspx?id=48145

@crozone I think you're right, but shouldn't required. These are redistributable after all.

Looks like libuv.dll references it:

@akoeplinger That's correct. AFAIK it shouldn't be necessary to reference both kernel32.dll and vcruntime140.dll.

@halter73 I thought that unless you statically link the VC++ runtime you need to install the redistributable package?

@akoeplinger Right. We shouldn't need to dynamically link the runtime. I think the same goes for the other redistributables like "api-ms-win-*".

I have installed Visual C++ 2015 redistributable, now I have error: api-ms-win-core-rtlsupport-l1-2-0.dll
System: Windows 7

cc @moozzyk in case he has thoughts on this

Also, @victorhurdugaci does this work with the current bits?

This was fixed in beta-8 I believe. The issue was that the libuv.dll we distributed in dll package was built incorrectly. We now build libuv differently.

Closing. If this still repro's in RC1 or RC2 bits, please reopen.


add requestAnimationFrame and friends

requestAnimationFrame
requestIdleCallback

But maybe the right place is the purescript-js-timers repo.

I think adding them here is probably right - node doesn't have rAF, and a dom dependency -dom suggests browser-y use cases, so it seems appropriate.

Resolved by #91


Fix nested ToolbarSection buttons commands not firing

I was recently tasked w/ cleaning up the toolbar buttons to be less cluttered. Our current design isn't responsive, so the easiest way to clean things up is to move buttons into an expandable button menu. Here is an example of the data that helps drive a ToolbarSection
Toolbar Button:
{
  command: 'Angle',
  type: 'tool',
  text: 'Angle',
  icon: 'angle-left',
  active: false,
},

Nested/Expandable Toolbar Button:
{
  text: 'More',
  icon: 'ellipse-circle',
  active: false,
  buttons: [
    {
      command: 'Bidirectional',
      type: 'tool',
      text: 'Bidirectional',
      icon: 'measure-target',
      active: false,
    },
    {
      command: 'Brush',
      type: 'tool',
      text: 'Brush',
      icon: 'circle',
      active: false,
    },
  ]
}

I encountered a lot of small issues when doing this. While the UI did update to reflect my changes, the buttons in the expanded toolbar menu stopped working. Here's a bit of flow on how all of this currently works:
ohif-core: src\redux\reducers\tools.js
When a button is selected, setToolActive initiates a SET_TOOL_ACTIVE action type. This:

Iterates all of the (top level) buttons in our configuration

Finds the one with a matching command, grabs a reference to it

If the button we found has a type of tool, we again iterate all of the buttons and...

Set all buttons w/ a matching command to active = true
Else, set active = false

Update state

Our buttons and viewport extensions/components then use this updated data to determine if they need to bind new tools for the viewport, update the toolbar, etc.
Problem
We shouldn't need to recursively traverse our toolbar buttons to:

Set an active flag
Activate tools on viewports
Update state

Especially as this logic lives in multiple locations.
Solution
Coming soon. Just wanted to detail this for when I come back to patch it up later.

We now fire commands in a very different way, so this should no longer be an issue. Feel free to re-open if you find out otherwise.


[2022-10-25] Two sum C++

Two Sum
Given an array of integers nums and an integer target, return indices of the two numbers such that they add up to target.
You may assume that each input would have exactly one solution, and you may not use the same element twice.
You can return the answer in any order.
Time Complex & Space Complex
Runtime: 1722 ms, faster than 5.00% of C++ online submissions for Two Sum.
Memory Usage: 10.1 MB, less than 96.60% of C++ online submissions for Two Sum.
참고 링크

[Leet Code](https://leetcode.com/problems/two-sum/)

기타 사항

생각 없이 코딩함.

추가로 PR 제목, Label, Assignees 추가 했습니다 이후 PR 때는 이번 PR에 설정된 거 참고해서 열어주세요


Cell caching (while scrolling) is not working with WindowScroller

Problem: Scrolling with WindowScroller is sluggish, because all cells on screen are re-rendered on every scroll event. There is an internal caching mechanism that should work while scrolling, but not if WindowScroller is used.
defaultCellRangeRenderer checks for cell cache before rendering a cell, but only when isScrolling is true (also something about scaling?). If isScrolling == false, it will always re-render cells.
This is passed in from Grid::_calculateChildrenToRender, which reads isScrolling from Grid.state. Grid.state.isScrolling is only set true, when onScroll is called from div.ReactVirtualized__Grid.
If WindowScroller is used, Grid.state.isScrolling is always set to false.
Proposed solution: allow isScrolling to be overridden from Grid.props. That way any component which renders Grid (and they all pass ...this.props to Grid) can read isScrolling from WindowScroller, and pass that down. E.g.
<WindowScroller>
  {({height, scrollTop, isScrolling}) => (
    <Table
      autoHeight
      height={height}
      scrollTop={overrideScrollTop || scrollTop}
      isScrolling={isScrolling}
      ...
    >
      {columns}
    </Table>
  )}
</WindowScroller>

Will make a pull request.

Good catch. I'm currently having performance issues with the WindowScroller as well.
Until the pull request is merged, a temporary solution for the List might be
import { defaultCellRangeRenderer, List } from 'react-virtualized;
//...

<WindowScroller>
  {({height, scrollTop, isScrolling}) => (
    <List
      autoHeight
      height={height}
      scrollTop={overrideScrollTop || scrollTop}
      defautCellRangeRenderer={params => defaultCellRangeRenderer({ ...params, isScrolling })}
      ...
    >
      {columns}
    </Table>
  )}
</WindowScroller>

The only thing missing from the hack is (Grid.js)
              pointerEvents: isScrolling ? 'none' : '',

Great issue write-up. Thanks for digging in and writing up a proposal.

defaultCellRangeRenderer checks for cell cache before rendering a cell, but only when isScrolling is true (also something about scaling?)

These couple of slides give a little background info on the scaling issue. Basically I'm compressing positions and adjusting offset slightly as a user scrolls- so if I were to cache style (eg cache position) in this case, cells would get misaligned slightly during scroll. It's complicated. ☹
@dfdeagle47 is right that you can temporarily work around this by passing a decorated defaultCellRangeRenderer as the cellRangeRenderer prop. I like your proposed solution though so I don't see any reason we can't move forward with it.

@bvaughn: thanks for the link, that slide deck is golden.

👍

This change has been pushed via 9.5.0. Thanks for your help!

Thank you!

Just a heads-up for anyone who's passing the isScrolling prop to List, but still having problem with rows re-rendering on each scroll: you also need to set isScrollingOptOut prop to true for the List component for the caching mechanism to work properly.


Add: Markdown in Support Ticket Replies

Description
Renders support ticket replies as markdown and sanitizes input when submitting a new reply.
Type of Change

Non breaking change ('update', 'change')

Applicable E2E Tests
N/A
To test
Create support ticket replies with all kinds of tags, including script tags

@Jskobos addressed. And yeah - just doing the markup in the actual descriptions is the scope of this PR


Param web page should say more explicitly why you would want to use parameters - and what will happen to you if you don't ;)

Also compare with Traits, Traitlets, etc

See #313


GraphQL API Queried fields in Lambda event variable

Describe the feature you'd like to request
I'd like to see the fields requested from the GraphQL API when using Lambda-backed data sources generated via @function in the Lambda event variable.
Describe the solution you'd like
For Lambda-backed data sources generated using @function there currently doesn't seem to be any awareness about the fields requested provided to Lambda when invoked.  It would be a great addition to customize the Lambda.  If the client only requested one field of many, for example, to be returned chances are logic could be added to the Lambda code to streamline its runtime and return.
Describe alternatives you've considered
none
Additional context
No response
Is this something that you'd be interested in working on?

[ ] 👋 I may be able to implement this feature request

Would this feature include a breaking change?

[ ] ⚠ This feature might incur a breaking change

Hi @toddaa, Thank you for raising this. We are marking this as a feature request for the team to evaluate further.


Fix issue #388.

Add the delete, list webhook operation.
Changed parameter names and descriptions.

Thank you, I'll modify it.

@rabbah ,could you please review it, I modified according to your comments.

thank you.

@ChrBi is there is a corresponding test for this in blue that we can consider moving?

@rabbah I think there is no test for the git webhook.
The reason for this was, that we were not able to delete the webhook on github with whisk again. So the second test would fail, because the webhook already exists.

@ChrBi since this is supposed to add delete support, we can add a test now?
Also you should review this since I think you authored this first version, if I recall...?

@rabbah Yes, @danielfederschmidt and me did the first version of this. I'll have a look on the changes on this PR.
If I remember right, the support to delete webhooks was already there. But we were not able to identify the webhook to delete, because this information has not been saved in whisk context.

@ChrBi , users need to decide which hook do delete, openwhisk doesn't need to save it.
those 2 docs just FYR.
https://developer.github.com/webhooks/
https://developer.github.com/v3/repos/hooks/

The indentation in the file is not consistent.

@lzbj the feed is specific to a webhook. So when deleting the feed the corresponding hook needs to be deleted.

@ChrBi one option we discussed previously is to store the information in the trigger. So a feed create could return parameters to store on the trigger and provided on future life cycle events.

@rabbah @ChrBi , thanks for your review, I'll first to review the doc mentioned by rabbah, and modify code according to your comments before I submit.

what about we list all the webhooks created by the user, and check if the callback url contains the trigger name, we then delete the  corresponding webhook. In this way, we don’t have to persist the webhook id somewhere.


Patch 1

Added net argument and ulimits argument.
Untested, so I hope that the unit test run.

Thanks for this @timkoers!
Looks there there was a test failure :(  https://travis-ci.org/github/magicmark/composerize/builds/678928371
(not sure why it's not being linked here, will look into that...)

Thanks! That would be great :-)

@timkoers fixed!

Great! There was another option that was missing, well missing..
The network option allows for a shorter command called net, did you implement that too?

Yep! Try it out http://composerize.com/

It doesn't seem to work!
-net host adds
environment:
- 'true'
and --net=host doesn't do anything

@timkoers works for me? try clearing your cache?
https://i.fluffy.cc/XzFHgGN88PBkpRR4jpVL7t3sSXk3KJgD.png

Good call! It's working :)


M61 - All Clear Page - Show WalMart product details

if don not find UPC in spcs and walmart data we show  a text that do not find any thing


Get the month currently displayed

Hi, I want to get the month showing, how can I do it?

Hi;
Where do you need it? PrimeDatePicker or PrimeCalendarView?

In PrimeCalendarView with typePicker=MULTIPLE, I want to create a button when I click on it, the entire day of the month is selected. So, how can i achieve this?

Please upgrade to the version to 3.2.0.
A property named firstDayOfMonthCalendar is provided to deliver information about the first day of the currently visible month. Then, you can filter picked days by its year and month like the following:
val firstDayOfMonth = calendarView.firstDayOfMonthCalendar
val days = calendarView.pickedMultipleDaysList.filter {
    it.year == firstDayOfMonth?.year && it.month == firstDayOfMonth.month
}

Please give an ⭐ if you enjoy the library 😃.

It works very well ⭐😁


Fix sorting for lowercase values

Update the smartSort() function to properly compare lowercase values.
Before:

After:

Good to know that there's a built-in function for this type of sorting. Since "smartSort" was only used in two places I just switched them to use "localeCompare" directly.


Support models without exported ID/primary field

Is your feature request related to a problem? Please describe.
This is a follow up to https://github.com/DataDog/jsonapi/pull/10 .  We have a use-case where models are used for create only so there is no primary attribute/we don't want to expose this to end user. With the current client mode implementation we have to define it if we want to use jsonapi. E.g.
type Example {
	ID string `jsonapi:"primary,random_type"`
}

Describe the solution you'd like
I understand the tag in primary attribute is used for the type. So to work with this, it would be nice to support private/anonymous primary key:
type Example {
	id interface{} `jsonapi:"primary,random_type"`
}

or
type Example {
	string `jsonapi:"primary,random_type"`
}

See attempt at supporting this here: https://github.com/DataDog/jsonapi/pull/46

This experiment for allowing anonymous/private primary field support was successful in implementation, but ultimately we've chosen to not go down this road. encoding/json and other patterns for serde in Go do not typically support anonymous/private fields, so adding this for just primary would be pretty inconsistent behavior.

We may re-open this in the future if there is sufficient need / benefit from supporting private/anonymous primary fields. As an alternative, you could always create 2 structs, one with a primary field which embeds a struct that contains the rest of the fields (no ID on that one), and only expose one to users of your API.


2018-07-01-preview for Microsoft.Web/connections seems to be missing?

The 2018-07-01-preview version for Microsoft.Web/connections seems to be missing from https://github.com/Azure/azure-rest-api-specs/tree/main/specification/web/resource-manager/Microsoft.Web ?
It's documented here: https://docs.microsoft.com/en-us/azure/logic-apps/create-managed-service-identity?tabs=consumption#multi-authentication, 2018-07-01-preview should be available as an API version for using the parameterValueSet property, but doesn't seem to be - it's not seen as a supported version by the terraform azapi provider, which uses this repo as it's source of truth.
Related ticket - https://github.com/Azure/terraform-provider-azapi/issues/130

+1.
Same problem with Microsoft.Web/connections/accessPolicies resource type which is neither documented, nor spec'ed.
I've reached out to the owning team internally.

Hello Mr @jamescrowley 👋
Just digging into this I noticed quite a coincidence ... in the Microsoft.Logic namespace, there is a 2018-07-01-preview version, however, there's no connections resources. It looks like something might have got confused in the documentation here.

haha, hey @danielrbradley 👋 for reference, this worked when disabling azapi's schema validation so I think the documentation is correct, but the specs are not.


[ISSUE] Game crash when using MegaHack's speedhack module.

Basically what the title says. Issue #18 also happens to me.
Reproduction steps:

Install COS
Install MegaHack
Enable both mods
Go into a level
Enable speed hack

Closing this as I only just realized that there are settings for the mod and running it in single threaded mode fixed my issue and the mentioned issue above.


Update failed: An error occurred trying to install this update

Microsoft PowerToys version
0.80.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
Perform auto update from General page.
✔ Expected Behavior
Launch powertoyssetup-0.82.1-x64.exe from LOCALAPPDATA%\Microsoft\PowerToys\Updates and begin installation process.
❌ Actual Behavior
Nothing happened. After manually running PowerToys, its version is still 0.80.1, mentioning that An error occurred trying to install this update. I've checked UpdateLogs, but it has only two lines without any helpful information:
[2024-08-04 11:41:47.905856] [p-17688] [t-15068] [info] update logger is initialized [2024-08-04 11:41:50.472172] [p-3480] [t-10040] [info] update logger is initialized
Other Software
No response

/bugreport

Hi @huaium
Seems that PowerToys 0.80.1 installed on your computer is the user installer but powertoyssetup-0.82.1-x64.exe you are trying to use for the update is the machine installer.
Where does powertoyssetup-0.82.1-x64.exe come from? Have you manually downloaded it? If yes, you should grab the user one: PowerToysUserSetup-0.83.0-x64.exe (we are now on 0.83.0 😉).
/needinfo

Hi @huaium
Seems that PowerToys 0.80.1 installed on your computer is the user installer but powertoyssetup-0.82.1-x64.exe you are trying to use for the update is the machine installer. Where does powertoyssetup-0.82.1-x64.exe come from? Have you manually downloaded it? If yes, you should grab the user one: PowerToysUserSetup-0.83.0-x64.exe (we are now on 0.83.0 😉).
/needinfo

The machine installer is from %LOCALAPPDATA%\Microsoft\PowerToys\Updates, downloaded automatically by PowerToys for updating, which is probably a wrong behavior.


Optimize TestDetector

Right now, test detection is very slow in the not-small examples of https://github.com/SpoonLabs/astor/.

See #74


Exit when outdated version of node is used

Tested with Node 4 and 0.12
$ wcfactory
@wcfactory/cli requires at least version 6.0 of Node, please upgrade

this looks reasonable but @heyMP should review for pushing forward

yep looks good!


Add a path attribute when picking videos so it matches what is returned from images

When you pick images what gets returned is an object similar to this
{ "path": "/storage/emulated/0/Pictures/image-7dda44cb-8d84-4df4-82b6-f23dd581d4d3.jpg",
  "uri": "file:///storage/emulated/0/Pictures/image-7dda44cb-8d84-4df4-82b6-f23dd581d4d3.jpg",
  "height": 2048,
  "width": 1536,
  "isVertical": true }

Videos however only return a content type uri.
{ "uri": "content://media/external/video/media/170" }

What the actual path to the above is this
/storage/emulated/0/Download/stock-footage-close-approach-to-waterfall-by-boat-milford-sound-fjord-south-island-new-zealand.mp4

What would be ideal is videos returning an object like this
{ "path": "/storage/emulated/0/Download/stock-footage-close-approach-to-waterfall-by-boat-milford-sound-fjord-south-island-new-zealand.mp4",
"uri": "content://media/external/video/media/170" }

For what purpose ?
The path is more there for debug than for a real usage (it's not even in README)

the react native upload library I use has an API where It will uploaded based on the absolute file path.
I could live without the path attribute but the uri for video is not close to a file path.
https://github.com/comiclee/react-native-fileupload/
    var obj = {
        uploadUrl: 'http://<IP_ADDRESS>:3000',
        method: 'POST', // default 'POST',support 'POST' and 'PUT'
        headers: {
          'Accept': 'application/json',
        },
        fields: {
            'hello': 'world',
        },
        files: [
          {
            name: 'one', // optional, if none then `filename` is used instead
            filename: 'one.w4a', // require, file name
            filepath: '/xxx/one.w4a', // require, file absoluete path
            filetype: 'audio/x-m4a', // options, if none, will get mimetype from `filepath` extension
          },
        ]
    };

Well first of all i highly recommend you to not use this module since there is no official support since december 2015 and it don't support uri.
Looks like react-native-file-uploader support uri. (no support since 13 jan too :/)
And if you really want to use react-native-fileupload ask for an uri support.


Enable transaction tab, and Exchange rate functionality.

Issue/Task Number: 480-token-stats, 477-transaction-tab,475-exchange-with-rate,474-create-exchange-rate-modal
Overview
Add functionality to view transaction, create rate for tokens and exchange with rate
Changes

Add amount formatter to handle big ints.
Add create rate for token.
Add exchange functionality with token.
Support total supply.
Able to view minted history

Usage

Go to token page, click view history to view minted history.
Go to token detail and create rate.
Transfer with different tokens.

Impact
Deploy as usual.

Why can we set two rates for an exchange pair? It's only one, between token 1 and token 2
Why did you call it rate? This is confusing since it may imply that you can create multiple rate for the same two tokens. Call it Exchange Pair so it matches what we have in the backend, we'll be adding helper tooltips to explain things if needed.
I think we can remove the name from the rate form since we'll be removing it (to confirm with @unnawut)


[Request] Adjust recipe/technology order

It would be cool if you could adjust the recipe order. The smart train should either be directly before the default train or after it, but not first in the rail section.
See this screenshot for explanation:

The same applies to the smart train technology.
Use the "order" tag for this. If you need help with that I can help/do it for you.

You're right it should be ordered like that, and if you know how the sorting system works you are more than welcome to do it, because i do not know how it works :)

Here the PR that fixes this issue: https://github.com/simwir/Shuttle-Train/pull/24

I also created a PR #3 on RailTanker to fix the final ordering, because RailTanker currently uses the wrong order tag and thus would be between the diesel-locomotive and the shuttle-train.


Create a table of contents for RFCs

This adds a table of contents to the RFCs folder, so somebody looking at the folder knows what documents are available. Hopefully this makes it a bit more useful than just plain folder slugs!

Great plan 👍

Yes it should, but I started this before you moved all the docs over. 😉

I have a replacement for this on my iMac at home, but I don't have write access to the docs repo.

Closed as docs have moved.


Pass variables to result function

Thought it would be handy when using a result function to be able to pass the query variables into the function, so they could be used to help generate the mock data.
Had a go at updating the code to make it so and it seems to work OK. Have also added a test around the functionality.

This is cool.

BTW your commit messages are in a mixture of imperative mood and declarative mood :D Most organisations seem to be mandating imperative right now, using that for this repo.

@insidewhy Thanks for this package, it makes working with apollo mocks much easier. Any idea when we might see a release or beta with this PR included?

@ajdean @eli0shin sorry I intended to release 2.0.1 with this two weeks ago, I'll do it today.

@ajdean @eli0shin Released 2.0.1. I think the built-in mock link should be replaced with wildcard-mock-link so I opened this feature request apollo: https://github.com/apollographql/apollo-feature-requests/issues/298
Would appreciate your reactions or comments on that issue. Thanks again for your contributions!


Upgrade to Node.js 4.8.4

There was a critical security vulnerability in all Node.js release lines a while back. It won't affect most of our images, but API in particular needs to be updated.

@orangejulius @dianashk this change will require a rebuild of the base image and all dependant images be pushed to dockerhub, otherwise the changes will roll out at different speeds for different images.

I'm rolling out all the CI infrastructure for building docker images right now. Once we have that in place, we'll be in a good spot to compare old and new docker images, and we can push new versions of the baseimage and libpostal_baseimage. Those should probably be brought under CI as well eventually.


shortcircuiting reconcile loop in ansible

Question
How do you accomplish a scenario like this in ansible? I have a secret(potential containing db credentials) that needs to be up, before my controller starts installing the product. In golang I would assume you would keep throwing an error in the reconcile loop and have the reconcile loop run again. How do you accomplish a similar flow in ansible?
This is what you could have done in the golang code and you can short circuit the reconcile loop.
I understand that for the above case specifically, one can rely on the fact that the pod mounting the secrets would keep crashing till the secret is in place, but the question along the lines of how you would accomplish a broad scenario like this.
Based on the return values, Result and error, the Request may be requeued and the reconcile loop may be triggered again:

// Reconcile successful - don't requeue
return reconcile.Result{}, nil
// Reconcile failed due to error - requeue
return reconcile.Result{}, err
// Requeue for any reason other than error
return reconcile.Result{Requeue: true}, nil

// Reconcile for any reason than error after 5 seconds
return reconcile.Result{RequeueAfter: time.Second*5}, nil

@camilamacedo86 this is more of a question on whether such a behavior even exists in the first place. if not I can take a stab at the code and see what would it take to implement this if it makes sense to implement this,

actually this is what I tried ```- name: Search for secrets
register: secret_list
k8s_facts:
kind: Secret
namespace: '{{ meta.namespace }}'
name: 'missing-secret-xxx'

name: assert that secret was found
assert:
that:

secret_list.resources | length > 0
fail_msg: " {{secret_list | length }} must be greater than 0"
success_msg: "{secret_list | length }} was greater than 0"```

so the loop did error out because the secret was not present but no successive reconcile loop was triggered and I am not sure how to force that.

I have exactly the same problem with ansible operator. If I don't set reconcilePeriod in watches.yml it doesn't ever reconcile. If I set one it is spamming reconcile loops no mater if task failed or succeeded.

Hi @atinsood and @glothriel,
I understand that your scenario is: I'd like to ensure that a task was done before continuing .... Then, in this case, did you try to use Ansible Conditionals. See the Ansible doc Waits for a condition before continuing. Also, I'd like to suggest you check the doc to know how you can use the k8s module as well. It has an example over a task that that wait for the namespace be with state: present and then, state: absent. Also, you can check here some examples that can help you with.
Following a more full explanation over the watch feature since you made few comments about it.
The feature which will trigger the reconcile when a resource is changed is called the watch. See the Memcached example. It has a watch feature here which is implemented for the Memcached and will retrigger the reconcile when any change such as to create, to update and to delete the Memcached resource be faced.
Note that, another way to re-trigger the reconcile is by setting reconcilePeriod which will re-trigger te reconcile after a period, ever x time, which shows been used by @glothriel. To understand the watch feature check the ansible user guide here. And then, to know more about the watch options see the doc
Please, let us know if the above information provided attend your expectations and if we can close this one or if with it you still unable to attend your need.

Thanks for the response @camilamacedo86, it's very clear. I'm aware about the behaviour you described, but it was surprising to me, that the framework doesn't operate in the mode Kubernetes controllers generally tend to follow - triggering reconciliation loops with exponential back-off after consecutive fails to reconcile a resource.

@fabianvf could you also give a hand here and let us know if you see if I am missing something here?

@glothriel if you're failing the playbook (eg, with the fail module), and it's not automatically requeuing with exponential backoff then that's definitely a bug. If you're exiting without a failure that will be viewed as a success though, so double check that the playbook run is marked as failed

Thanks @fabianvf I will try to reproduce on some minimal example this week

Hi @glothriel,
I am closing this one since shows all sorted out. Please, feel free to raise a new issue as a bug if it does not work as described by @fabianvf in your POC/tests.


Feature Request: Add an option to specify host url of openai api key

Background: From where I am living, unfortunately, we don't have direct access to openai network. Even with a vpn setup, It's very difficult to be able to pay for the bills. As a workaround, I turned to use a openai api forwarding service, which demands a specific host rather than api.openai.com.I know this is not relevant to most, but It's the fact for the people living in China.

I'm not really sure about the connection issue with the OpenAI API, but the extension has a "Custom" connection module that you could try.

Tried many times but cannot set custom provider. The document is not clear.

I tried as well but I cannot get it to work. It always says "Connected" at the setup but throws "Connection Error" every time. I don't see any connection request on a server-side so I assume "Connected" doesn't mean much.
Ollama supports OpenAI API and can be used for testing locally. (via http://localhost:11434/v1). Instructions or a bugfix is appreciated.

I guess you have to provide full URL to the chat completion endpoint in custom provider: for OpenAI it will be https://api.openai.com/v1/chat/completions. Also if you don't need an api key (for example for local server), you still have to set it to a nonempty string.


Permission issue on Ubuntu-24.04 when create symlink on mnt drives

cmd /c ver
Microsoft Windows [Version 10.0.22631.3527]
wsl --status
Default Distribution: MyWSL
Default Version: 2
Did the problem occur during installation?

[ ] Yes

What happened?
I am using .bash_aliases and .gitconfig from windows via symlinks like
ln -s /mnt/c/Users/Zohaib/.bash_aliases .bash_aliases
In Ubuntu-24.04 when I open file with nano .bash_aliases it open file with message: [   File '.bash_aliases' is unwriteable   ]
Works fine with sudo permission.
What was expected?
I am using same .bash_aliases, .gitconfig and .ssh/ with on all Ubuntu-22 Distros and it never ask for sudo permission.
Today I install Ubuntu-24.04 for new project and it start asking for sudo permission
Steps to reproduce
wsl -d Ubuntu-22.04
nano /mnt/c/Users/Zohaib/.bash_aliases
ln -s /mnt/c/Users/Zohaib/.bash_aliases .bash_aliases
exit
wsl -d Ubuntu-24.04
ln -s /mnt/c/Users/Zohaib/.bash_aliases .bash_aliases
nano .bash_aliases
-note I am using same user: zohaib on my all Distros.
zohaib@My-WSL:~$ cat /etc/wsl.conf
[boot]
systemd=true
[user]
default=zohaib
[network]
hostname=My-WSL
[automount]
enabled=true
options=metadata,uid=1000,gid=1000,umask=022
Additional information
No response

even can't make a new file in /mnt/c/Users/Zohaib on Ubuntu-24.04
nano /mnt/c/Users/Zohaib/.bash_aliases leads to error:

no. In Ubuntu-24.04 its changed. now there is a new user(ubuntu) and 1000 UID assigned to ubuntu user.
In Ubuntu-22 there was no default(ubuntu) user

So, as you can realize, the permission issues you've seen is due the mismatch of your default user UID and the ones assigned via automount options metadata,uid=1000,gid=1000,umask=022. The fix for the existence of the default user you mention is recent and will be in the next publication to MS Store.

I was having the same issues dealing with docker containers and npm permissions in Ubuntu 24 in WSL. I found a workaround that seems to work in my case. I will write the full process for everyone's convenience.
First, in Ubuntu terminal do  sudo nano /etc/wsl.conf and then add the following.
Any instance of 'YOUR-USERNAME' needs to be replaced with your username, in your case 'zohaib'.
[boot]
systemd=true

[user]
default=YOUR-USERNAME

[automount]
enabled=true
options=metadata,uid=1000,gid=1000,umask=022

Second, sudo nano /etc/passwd, you will see the following at the end of the file:
...
...
ubuntu:x:1000:1000:Ubuntu:/home/ubuntu:/bin/bash
YOUR-USERNAME:x:1002:1002:,,,:/home/YOUR-USERNAME:/bin/bash

Delete the ubuntu line and change yours to become the following
...
...
YOUR-USERNAME:x:1000:1000:,,,:/home/YOUR-USERNAME:/bin/bash

After that you need to change the owner of the home folder since your uid will be 1000 but your home folder will have the uid of 1002, To fix that simply do sudo chown -R $USER /home/$USER
This worked on a clean install, if you face some issues with permissions, you can either fix it using sudo chown -R $USER ./FOLDER to the offending folder or you could do a clean install of Ubuntu to not face these issues again.

I have the operation not permitted error when using pip install . with a local package located in /mnt/d.
I try with Ubuntu 24 and 22, both of them have this problem.
But when I copy the package to ~/folder, the error disappears.
I think that, this error relate to the writable permission.


how to set texture from image

how can i set a texture on a renderable on runtime?
i am trying to change it in code
Texture.builder().setSource(this, Uri.parse("maxresdefault.jpg")).build().thenAccept(texture ->                                     andyRenderable.getMaterial().setTexture(MATERIAL_TEXTURE, texture));
but it seems to set main color of that image as renderable's solid color, not the actual image
this is the picture i expect to be drawn on the model

this is how it actually looks

any help appreciated!

I suspect that the UV mapping for the model is not rendering the texture where you expect it. Assuming you made the model yourself, I suggest you try using this image in your 3D modeling software and seeing if you get the same result. If so, you can look at your UV map and update it accordingly.

@dsternfeld7 thanks, i will try


Removes serial number nonce randomness

Motivation
Removes serial number nonce randomness

This PR will await PR #295 given both PR's touch the DPC struct


Change Device UUID

I'm trying to change my device UUID. I can set my services and characteristics UUIDs, but not my device's.

Here for example BlueLight reports that my device UUID is 5E7EF70B-224B-491B-86DB-2DA3BA8AF437. It does not seem to be random, stopping and restarting my application doesn't change it.
How can I change this using Bleno ? Thanks !

@HippoBaro it is not possible to change the device UUID with bleno.
iOS bases it off the peripherals Bluetooth Address, and it is a randomly generated UUID per iOS device.


[UI] Make the message more digestible  > Environments > Add > pop up window

talked this out with @eabyshev and he suggested to change envID to  envName < @Commander223
Enhancement:
Make the message more digestible  > Environments > Add > pop up window

Fixed


Remove heap dump addon files as well

SAP JVM creates an addon file next to the heap dump file. The addon file includes among other things the command line parameters of the process and the stack traces of the last out of memory errors as well as a class and meta space statistic.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


ISSUE: Documentation does not describe how to use the new sensor.mail_image_url in place of the camera for

Describe the bug
The dedicated card for mail and packages https://github.com/moralmunky/Home-Assistant-Mail-And-Packages-Custom-Card does not seem up to date with latest beta. Documentation is showing info for the latest release which is great but it's not easy for beta testers to actually help beta test without any info. I took a look at the code but was not able to figure out how to configure the new sensor.mail_image_url in place of the custom camera.

That'd be an issue for moralmunky/Home-Assistant-Mail-And-Packages-Custom-Card repo.

That'd be an issue for moralmunky/Home-Assistant-Mail-And-Packages-Custom-Card repo.

THe current version of the card is in step with the latest non-eta release of the integration. The card is not ready for release. Please continue using either the path to the gif.

THe current version of the card is in step with the latest non-eta release of the integration. The card is not ready for release. Please continue using either the path to the gif.


[beta] solid_underline does not mark whitespace

As per title, any error regarding whitespace isn't marked by the solid_underline option (but is by other options e.g. outline).

Also, two questions:

What are the name of the regions that now have to be protected for SublimeLinter to take precedence over GitGutter?

How do I specify an hex/rgb color for the scope(s) in the styles options?

That is a core issue not a problem specific to SublimeLinter.

Thanks, I'll revert to outline for the time being then. I thought it used to work in SL3, but I must have been mistaken.

Feel free to upvote the upstream issue: https://github.com/SublimeTextIssues/Core/issues/137

Nothing we can do here.

@FichteFoll Yep I digged around and I see now how I should be handling custom values. Thanks.


Noise Error

Uncaught TypeError: Cannot set properties of undefined (setting 'noise')
at THREE.Terrain.min.js:7:27
at THREE.Terrain.min.js:7:2257

Hi there. This issue does not contain enough information to respond usefully. Please identify if this is a bug report or support request, what code you tried that resulted in this error, what version of Three.js you're using, anything you attempted to solve the problem, etc.
Also, debugging will be easier if you use the non-minified version.