resolve crash for high resolution images

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/35171
PROPOSAL: https://github.com/Expensify/App/issues/35171#issuecomment-1970625554
Tests

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

Offline tests

QA Steps

Open chat and add attachment.
Load high resolution image and click send attachment.
Open attachment verify attachment loads without any crash.

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

PR Author Checklist

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

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

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

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

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

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

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

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@aimane-chnaif The patch files are not needed and can be deleted.
patches/expo-image+1.10.1+001+applyFill.patch
patches/expo-image+1.10.1+002+TintFix.patch

Ah thanks,  yeah this issue seems to be a dupe. I will test to see if the Android app still has the same OutOfMemory exception -- perhaps this is a slightly different issue.

Here's a high resolution image that used to crash the Android app.

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

@Julesssss looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

The checks all showed green until after I merged...

🚀 Deployed to staging by https://github.com/Julesssss in version: 1.4.56-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.4.56-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


image size

i want to training this network with my own dataset to detect some small object
when I use the image that divided into parts  (part size 256256 ) and gained a mAP as 90%
but when I use the original size image(19201080)to train the network with the same settings but I just gained a mAP as 12%
do anyone know the reason about that? Is it about the data normalization?
Thank you!

https://www.quora.com/To-what-resolution-should-I-resize-my-images-to-use-as-training-dataset-for-deep-learning , please read this , thank you


[playtron-os-files] Update version

to 0.7.0 which includes loading controller drivers.
Also add 'clatd-ipv6-check.service' to be managed by systemd macros.

I did a test build of the source and binary RPMs. The binary RPM installed and had all of the correct files (including the modules-load for the controller drivers + clatd-ipv6-check.service now being correctly enabled during install, disabled during uninstall, etc.).


Updated expected version numbers to fix UnresolvedAssemblyTest failure

Updating version numbers in the expected versions to fix test failure

LGTM


bcachefs-tools: unstable-2023-06-28 -> 1.3.1

Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

@ofborg build bcachefs-tools

We should probably consider updating our linux_testing_bcachefs at some point soon... I had some really bad experiences with our patches-on-linux-6.4-based bcachefs.  Using the upstream nightly (6.7 prerelease) fixed them.

We should probably consider updating our linux_testing_bcachefs at some point soon... I had some really bad experiences with our patches-on-linux-6.4-based bcachefs. Using the upstream nightly (6.7 prerelease) fixed them.

Why even keep linux_testing_bcachefs around at this point, now that bcachefs is in 6.7-rc1? That's probably where Kent will still put interim patches before getting pulled during the merge window—but we don't keep expressions for any other upstream maintainer branches (e.g. like Jens tree for io_uring)... Aside from it being impractical to track too many, maintainer branches are often unstable anyway since developers patch and fix bugs. bcachefs was only special before because it was not upstream and there was appetite to have an expression anyway. With the move upstream, I don't think there's any reason to keep it around.

This conflicted with PR #256638.  Perhaps you want to check the result.


Set and create operations, order of execution

Describe the bug
The available operations don't execute in the specified order.
To Reproduce
In the attached image I have a set of available operations for an image field. I want to use set and create at the same time but it seems that set is always executed after create, no matter if the order is create and set or vice versa.
Expected behavior
Execute operations in the specified order.
Screenshots

Versions (please complete the following information):

Connector: Postgres
Prisma Server: 1.25.0
prisma CLI: prisma/1.27.0 (windows-x64) node-v11.9.0
OS: Windows 10
other dependencies: prisma-client

Hi @AndreiBacescu
We really can't change the order of execution as it will break the existing systems that rely on that order.
In this situation, you should perform two separate calls. That will ensure set is executed before create

@pantharshit00 Performing 2 separate calls is not transactional :/

I am sorry but we really can't change the order here.


False positive: The declaration isn't referenced ( when used with operator is / is! )

void main() {
  print( 1 is _Used );
}

class _Used {} // The declaration '_Used' isn't referenced.

> dart --version
Dart SDK version: 3.0.5 (stable) (Mon Jun 12 18:31:49 2023 +0000) on "linux_x64"

Maybe the analyzer is clever enough to see that if _Used is never referenced anywhere that can create an instace, there can be no instances, and 1 is _Used is always false, so it's not really a use either.
(I assume the original example didn't use 1 as the object, which makes the "always false" deduction trivial.)

@srawlins

@lrhn This makes sense. I was able to fix the issue in my code(wasn't constructing the object I was supposed). The issue here however is that analyzer isn't reflecting the fact that type _Used is actually used in the code.
if (o  as _Used) // no warning
if (o  is _Used) // "declaration _Used isn't referenced"
if (o is! _Used) // "declaration _Used isn't referenced"

Moreover, the warning itself, "declaration isn't referenced" is incorrect since I can clearly see references in the code.

Maybe the analyzer is clever enough to see that if _Used is never referenced anywhere that can create an instace, there can be no instances, and 1 is _Used is always false, so it's not really a use either.

I think this is expected and it isn't doing that? I mean if analyzer can deduce that expression always evaluates to false then it should be saying something like "is _Used is always false", exactly on the line is _Used rather then pointing to a irrelevant location with an incorrect warning.

I'm totally guessing, but maybe the analyzer is seeing that there is no expression with static type _User, and that's what it means by the type not being referenced.
The o as _User expression actually has type _User, so it makes the warning go away, whereas o is _User is just a boolean. Even if it promotes o, you still need to use o afterwards.

No warnings  with as, even if we don't use o anywhere.
o as _Used;
print( 'something else' );

I think analyzer is detecting only redundant is x exps and ignoring all the "always false" ones. I doubt it's even trying to see if is x is always false:
if(1 is int)    // Unnecessary type check; the result is always 'true'.
if(1 is String) // no warning

if(false is bool)   // Unnecessary type check; the result is always 'true'.
if(false is String) // no warning

if(false)       // Dead code. Try removing the code, ...
if(1 is _Used)  // no warning
if(1 is String) // no warning

The documentation actually says this:

Not all references to an element will mark it as “used”:

Assigning a value to a top-level variable (with a standard = assignment, or a null-aware ??= assignment) does not count as using it.
Referring to an element in a doc comment reference does not count as using it.
Referring to a class, mixin, or enum on the right side of an is expression does not count as using it.

The last item states that ... is _Used doesn't count.
It's more interesting that late _Used x; if (false) x = 0 as dynamic; x.member; doesn't count, but it's still correct in that there definitely are no instances of _Used in the program (no constructor invocations, no subclasses), so the class, as a template for object instances, is unused.
I guess the biggest issue is the phrasing, "not referenced" should really be "never instantiated or subtyped".

"not referenced" should really be "never instantiated or subtyped".

This will help.

it's still correct in that there definitely are no instances ... (no constructor invocations, no subclasses), so the class, as a template for object instances, is unused.

Correct, there are no instances but it doesn't implies that type is unused?
import 'package:js/js.dart';

@JS('parse')
external set _parse(String Function(dynamic args) f);

void main() {
  _parse = allowInterop(parse);
}

String parse(dynamic args) {
  if (args is String) {
    return _parseTpl(args);
  }

  if (args is _JsObject && null != args.tplPath) {
    return _parseTpl(_readTpl(args.tplPath!));
  }

  return 'fallback tpl';
}

@JS()
@anonymous
class _JsObject { // warnin: "decl not ref"
  external String? get tplPath;
}

String _parseTpl(String contents) => 'parsed: $contents';
String _readTpl(String path) => 'contents from: $path';

>> parse('literal')
  "parsed: literal"

>> parse({tplPath: 'tpl.html'})
  "parsed: contents from: tpl.html"

>> parse({})
  "fallback tpl"

I know is JsO is bit wonky but hey it works and is much more idiomatic than isJavascriptO/hasProp

I think there are two requests here then:

Change the text from "referenced" to something like "instantiated"
When a private class is annotated with @JS(), count that as a "use".

When a private class is annotated with @JS(), count that as a "use".

Yes.
Also,

Referring to a class, mixin, or enum on the right side of an is expression does not count as using it

If there are more ways to use a private type or invoke its ctor(embedding APIs maybe? Dart_AllocateWithNativeFields) from outside the file/library where it's declared then I'd suggest to start treating Type as used if it appears on the right-side of is exp.


Modifications to compile on C# 5 - forked repository

You might want to create a 'csharp5' branch that contains the code which compiles on C# 5, I needed it, so I figured to make it available.
https://github.com/paviad/Heroes.ReplayParser/tree/csharp5

Thanks for setting that up :) I'm not up for supporting it on my end, but if people ask me I'll send them to your fork


MultiIndex primary key spec removal

Closes #533.
Builds on top of #568. Makes it simpler to handle MultiIndex by removing the pk from the index key specification.
Solved it by collapsing the index key along with an extra element (the raw pk). This makes it possible to keep the old format in the store, as the new (internally generated) index key would be exactly the same as the one corresponding to the previous key spec.
Overall, I'm pretty happy with this implementation. It has some tricks, like shifting the definition of what is a prefix and a sub-prefix. So that prefixes can still be specified over the full index key (which doesn't include the primary key in its spec anymore). Despite (or because of) that, I think the end result is both clear and ergonomic.
This PR also renames K to IK for clarity. So, it also closes (the second part of) #531.
TODO:

Update related documentation / comments. (done)

I don't have the time to dig into this, which it deserves.
If you and Jake agree, feel free to merge.
I feel this is getting to larger changes and should be wrapped into a 0.11 release along with the other pending stuff.
Meaning, go ahead and merge, but we commit to no more 0.10.x releases now. And start all the 0.11 cleanup stuff. (When we get to them). Let's get that utils rename in (finally)


added test case for negative amount with non-scale error #2011

Part of #2011

Hi, as I understand it, this test contradicts to the exercise instructions. The instructions say "If the scale is broken and returning negative amounts of fodder, return an error saying "negative fodder". This test checks for negative value, but expects "non-scale error"

@enarha It might seem a bit strange but it is all correct. The test in this PR tests that the solution checks the error first and only evaluates the actual result of the function of the error is either nil or is the special ErrScaleMalfunction. See https://github.com/exercism/go/issues/2011#issuecomment-1001638982 for details.

@enarha There was a task to update the concept introduction accordingly but I think there was some issue with merging that change. I will re-open the respective issue and we will make sure the documentation on this will be improved.


fix(Icons): fixing errors with brand icons

Updating stop-color and stop-opacity to stopColor and stopOpacity.

Warnings

:warning:
There are no updates provided to CHANGELOG. Ensure there are no publicly visible changes introduced by this PR.

  Generated by :no_entry_sign: dangerJS


expose domain state?

I'm use handmade script to detect crushed domains, does it possible to expose domain state with this exporter? So i can watch libvirt domain state ?

I will discuss this option with my colleagues if we want to implement this. But I don't see why more metrics is a bad thing, so pull request is always welcome.

I will fix this next week.

Thanks!. I check node-exporter and thinks that libvirt exporter do the same thinkgs -
via command args we choose which metrics groups we want to expose:
domains
pools
volumes
networks
what do you think?

That only makes sense IMO if there is a reason not to collect the data (e.g. overhead)

@stefreak if we expose in/out traffic, cpu,memory, state, iops for each domain and each domain device (i ave nodes with 200 domains) i think that select what export will be useful.
Also for pools.

We've patched this ourselves to get domain state, but I think there is enough users that want this now to warrant a tag and move to GetAllDomainStats()? What could help that happen?


Chore: update spec version, prepare for parachain 1.21.5

Resolves interlay/interbtc-squid#78
Prepare squid for kintsugi testnet reset/release of parachain v 1.21.5

Bump squid version to 0.14.0
Fetched metadata for locally running parachain v 1.21.5.
Combining metadata from kintsugi mainnet and the locally running testnet
Regenerated types, and fixed compile errors
Updated changed event (pallet) names
Minor code clean up in edited files (eg. removing unused imports)

noticed there are some extra changes, but suppose these should be included in the release :)
Correct, db migration schema didn't have all human readable amount fields yet. So I regenerated those as part of this work for us to have a deployable version.


Add app shortcuts

Android app shortcuts seem like a natural way to launch the app and go directly to

Nearby places
Upload from gallery
Take a picture

It should be a matter of configuration rather than large amounts of code - adding a shortcuts.xml and making sure that we can launch directly into the given activities.  I am so-so about "Upload from gallery" and think just taking a photo or going to the nearby list might be the most important.

Cool, I actually had no idea we could do that! What SDK level does that require?
Personally "Upload from gallery" is the option that I use more than 80% of the time in the app, I would be for keeping that in. :)

If the app targets Android 7.1 (API level 25) or higher you can use app shortcuts.  The gradle.properties currently has us set to a minimum of 15, and target of 25, so we are good (though we might want to think about bumping the minimum up as API 15 represents only 0.7% according to Google)
I think one killer feature of app shortcuts is that users can create "pinned shortcuts" - an icon directly on the launcher that jumps directly into the app.  If you use the upload from gallery 80% of the time, this would let you jump there in one tap!
Google have some pretty specific guidance on the icons that should live on the shortcuts.  Someone with more artistic talent that me should create the icons. :-)
Google's documentation - https://developer.android.com/guide/topics/ui/shortcuts.html

Updated the screenshot - icons and starting point for shortcuts.xml are on a branch, should someone else want to pick this up and play around with it - https://github.com/psh/apps-android-commons/tree/app-shortcuts

Sounds great to me. Happy to review if anyone wants to pick this up. :)

@psh I have cloned your repo twice. Both times, all the changes you have done related to app-shortcuts isn't showing on my Android Studio. Is it somehow protected?

@psh, looking into this..

how can i contribute ? i am a beginner with few months of experience with android.

I have left this issue for now.

@deepak-thakur Feel free to fix this issue, see https://github.com/commons-app/apps-android-commons/wiki/Volunteers-welcome! :-)

@nicolas-raoul Can i work on this issue or anyone else has already been assigned ?

@adikul30 Seems like @deepak-thakur is not around for now, so feel free to give it a try, yes :-)

@adikul30 u working on this ? I will like to work on this issue.

@tanvidadu No progress updates from @addshore so feel free to give it a try, yes :-) Please post progress updates every few days, thanks!

@psh i checked your branch for creating apps shortcut. Looks as good starting point. Can I work on your branch ?

@nicolas-raoul Can I take up this issue?

@tanvidadu Any update on this? By the way, you don't really need permission to merge from people's branches (just acknowledging them is nice though) :-)

@madhurgupta10 , sure go ahead !  :+1:
@nicolas-raoul, I haven't worked on this issue since last year  :/

@tanvidadu OK thanks!
@madhurgupta10 It is yours :-)


I believe the dot product for vectors is wrong

Lemme start by saying I am by no means a game developer or math whiz!
But I think the logic for dot product is wrong. It is x1*x2 + y1PLUSy2 whereas I believe it should be x1*x2 + y1*y2 . A * was typo'd as +
https://github.com/replit/kaboom/blob/4925fd04b764686c3f27acfa4bbf89c29801352a/src/math.ts#L99
Unrelated (I've noticed you don't respond to these issues) - I'd really love if the license were switched to MIT. As it stands, I will likely migrate away from kaboom after today to keep my options open before I get too invested. Just a data point for you.
Take care, and thanks for the fun so far!

Just out of curiosity, why do you consider GPL bad for your purpose?

I'd like to keep the option open of selling a game instead of giving it away. My understanding is that if my game uses a GPL library, I'm required to license it under GPL which would allow anyone to legally distribute the game for free, which wouldn't be great for sales.

Nice catch! A typo indeed, fixed in 5d9b32a


Display Top tweets instead of latest when searching hashtag

Describe the problem
Hi, thanks for developing this amazing app.
Something good you could add to Squawker is an option to display Top tweets instead of latest tweets when you search something or click on any trending topic.
Usually, most of the recent tweets are not interesting and sometimes not even related to the reason why the topic is trending.
Describe the solution
There are 2 tabs in the search view to show either people or tweets.
My suggestion is the following: add a third tab to display top tweets as well. I think it should be the default tab when you search.
This is a feature provided in the official X app. I don't know about Twitter APIs but I would be surprised if they don't provide this.
Cheers,

I think this is possible, we just need someone to code it
(I'll be happy to code it when I have some free time)

I have done some research and I didn't find a Twitter/X API query or the appropriate option of the currently implemented Twitter/X API queries that will return the most popular tweets instead of the latest.
@Teskann You say that it is available in the Twitter/X app. I don't use the app.
Is it also available on their WEB site? If so, could you show me?

Yes, it is available on the website
This is the request made for Top Tweets for the trending query Amrabat

URL Query
https://twitter.com/i/api/graphql/-KWrbTBsPifMuLUqqDiU_A/SearchTimeline?variables=%7B%22rawQuery%22%3A%22Amrabat%22%2C%22count%22%3A40%2C%22cursor%22%3A%22DAADDAABCgABGHc3t7WX0C8KAAIYdzeV0Ndx3gAIAAIAAAABCAADAAAAAAgABAAAAAAKAAUYdze4_wAnEAoABhh3N7j-_9jwAAA%22%2C%22querySource%22%3A%22trend_click%22%2C%22product%22%3A%22Latest%22%7D&features=%7B%22responsive_web_graphql_exclude_directive_enabled%22%3Atrue%2C%22verified_phone_label_enabled%22%3Afalse%2C%22creator_subscriptions_tweet_preview_api_enabled%22%3Atrue%2C%22responsive_web_graphql_timeline_navigation_enabled%22%3Atrue%2C%22responsive_web_graphql_skip_user_profile_image_extensions_enabled%22%3Afalse%2C%22c9s_tweet_anatomy_moderator_badge_enabled%22%3Atrue%2C%22tweetypie_unmention_optimization_enabled%22%3Atrue%2C%22responsive_web_edit_tweet_api_enabled%22%3Atrue%2C%22graphql_is_translatable_rweb_tweet_is_translatable_enabled%22%3Atrue%2C%22view_counts_everywhere_api_enabled%22%3Atrue%2C%22longform_notetweets_consumption_enabled%22%3Atrue%2C%22responsive_web_twitter_article_tweet_consumption_enabled%22%3Atrue%2C%22tweet_awards_web_tipping_enabled%22%3Afalse%2C%22freedom_of_speech_not_reach_fetch_enabled%22%3Atrue%2C%22standardized_nudges_misinfo%22%3Atrue%2C%22tweet_with_visibility_results_prefer_gql_limited_actions_policy_enabled%22%3Atrue%2C%22rweb_video_timestamps_enabled%22%3Atrue%2C%22longform_notetweets_rich_text_read_enabled%22%3Atrue%2C%22longform_notetweets_inline_media_enabled%22%3Atrue%2C%22responsive_web_enhance_cards_enabled%22%3Afalse%7D

@j-fbriere are you wanting to tackle this issue?

I see.
It's the "querySource": "trend_click" in the "variables" node of the request.
Thank you very much for the info.
I will implement the feature because I have time to do it.

Ok, thank you for your time on this issue

Hi @j-fbriere thanks for your reply.
I don't think it's about the "querySource":"trend_click". This one might just tell twitter we accessed the data through a click on the trending list.
It rather comes from "product" which is "Latest" in the example above. Indeed, when you search top tweets, this variable becomes "Top". This can also take the value "Media", "People" and "Lists" following what you can see on this screenshot from X website:

Every time you click on another tab, several request are send including one looking like the one @mrhcjones shared. The only difference is this "product" variable.

@Teskann Than you for the information.
In its search page, currently Squawker uses "product": "People" to search users and "product": "Latest" to search tweets.
If I understand correctly the third type of search, the search of trending tweets, should use "product": "Top".
I will try it.

Thanks again. Good luck !

Version v3.7.7 implements this feature.
You'll find the trend search on the third tab of the search screen.


Improves error messaging

Performance loss around 2.5 to 4% parsing example at 12us

This is a propose change for issue https://github.com/naomijub/edn-rs/issues/70

Most of it seems fine :+1:
I just have two things to point out:

(opinion) I think we could destructure instead of using c.1, and only use the index when necessary
It doesn't seem to have a test case for could not be parsed at char count that checks the character index error

This commit adds tests 5d85693. However, destructuring seems to have impacted performance 🤔.
We can check this in another PR


Validate GDB - Feature classes with attachments

Hello! We use several feature classes in our maps that contain attachments (Field locations with field photos, geochem locations with XRD spectrographs, etc). Unfortunately, the Validate Database (v2.7.1) doesn't seem to like attachments (or relationship classes?). Is this a bug with the script, or have I missed a GeMS requirement that specifies no attachments?

Oh, here's a screenshot of the result...

Roger that, That's what I've been doing (running the script after disabling attachments), but just want to make sure you're aware and/or I wasn't missing something obvious. Thanks for the reply though!
The error message should be below...not sure what happened above?

Thanks, now I see it.
Pretty sure I have seen this error before but I can't track the issue down. Have you, by chance, hidden the OBJECTID field in the view properties of that table or changed it's alias?

Negative on the OID. I may have possibly changed the alias on one of these, but I've tried the validation tool with all combinations of FCs with attachments and the script always fails. I'm happy to share my FGDB if you like?

Sure, I would like to take a look.

All right, I see the problem. I presume each of the tables with _ATTACH are created once the Add Attachment tool is run? In those tables, the objectid type field is called ATTACHMENTID. In the other case where your error was reported, the object id was called OBJECTID_1.
I just posted a new release (v2.7.2) with changes to how the object id field is identified, no longer relying on the name. I ran your geodatabase with attachments through the validation tool and it ran correctly. Try it out and let me know.

Awesome. The new script woks perfectly! Thank you.
The one downside, is I now have to add a "_ID" field to the attachment tables to meet Level 3 GeMS compliance haha, but that can be easily automated with the Add Attachments GP tool.
Thanks for your quick response on this!

Matthew A. Massey
Geologic Mapping Section
Kentucky Geological Survey
204 Mining and Mineral Resources Building
Lexington, KY 40506-0107
+1.859.323.0517 (office)
From: Evan Thoms @.>
Sent: Monday, April 18, 2022 8:45 PM
To: usgs/gems-tools-pro @.>
Cc: Matthew Massey @.>; Author @.>
Subject: Re: [usgs/gems-tools-pro] Validate GDB - Feature classes with attachments (Issue #20)
All right, I see the problem. I presume each of the tables with _ATTACH are created once the Add Attachment tool is run? In those tables, the objectid type field is called ATTACHMENTID. In the other case where your error was reported, the object id was called OBJECTID_1.
I just posted a new release (v2.7.2) with changes to how the object id field is identified, no longer relying on the name. I ran your geodatabase with attachments through the validation tool and it ran correctly. Try it out and let me know.

Reply to this email directly, view it on GitHubhttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fusgs%2Fgems-tools-pro%2Fissues%2F20%23issuecomment-1101885894&data=04|01|matthew.massey%40uky.edu|192b053af9f744aa750608da219dcb1f|2b30530b69b64457b818481cb53d42ae|0|0|637859258854127165|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=nAINiFmC%2BPc0nP45P50Be9rXIjUCG4xGHpThrhlv9O8%3D&reserved=0, or unsubscribehttps://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQSB7X77JA2W3SGXZRSNI33VFX6XLANCNFSM5TNW57GA&data=04|01|matthew.massey%40uky.edu|192b053af9f744aa750608da219dcb1f|2b30530b69b64457b818481cb53d42ae|0|0|637859258854127165|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000&sdata=L5tY%2FHZGmTxu8nmZkZvfmVXytx9QJa9DcVOXfeFihTo%3D&reserved=0.
You are receiving this because you authored the thread.Message ID<EMAIL_ADDRESS>

Ah, good point. I just posted yet another release where tables with __ATTACH in the name will be ignored when looking for _ID fields. Not the best logic to go on name, but it does seem unlikely that someone would change the name that Arc assigns to attachment tables.


[BUG] Blazor standalone app can't build release cause of emcc error.

Description
Create blazor standalone app running .NET8, add SkiaSharp.Views.Blazor package and you can't build app in release mode.
Code
<Project Sdk="Microsoft.NET.Sdk.BlazorWebAssembly">

  <PropertyGroup>
    <TargetFramework>net8.0</TargetFramework>
    <Nullable>enable</Nullable>
    <ImplicitUsings>enable</ImplicitUsings>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="SkiaSharp.Views.Blazor" Version="2.88.8" />
  </ItemGroup>

</Project>

Expected Behavior
Debug builds and run fine.
Actual Behavior

No response
Version of SkiaSharp
2.88.8 (Current)
Last Known Good Version of SkiaSharp
Other (Please indicate in the description)
IDE / Editor
Visual Studio (Windows)
Platform / Operating System
Windows
Platform / Operating System Version
No response
Devices
No response
Relevant Screenshots
No response
Relevant Log Output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

This seems to work for me, please attach a repro so I can check:

BlazorApp1.zip


Error code (node:19052)

Error: (node:19052) UnhandledPromiseRejectionWarning: Error: connect ECONNREFUSED <IP_ADDRESS>:3306     at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1191:14)     --------------------     at Protocol._enqueue (mysql-async.js:5807:48)     at Protocol.handshake (mysql-async.js:5714:23)     at PoolConnection.connect (mysql-async.js:720:18)     at Pool.getConnection (mysql-async.js:4577:16)     at Pool.query (mysql-async.js:4731:8)     at Promise (mysql-async.js:4970:12)     at new Promise (<anonymous>)     at execute (mysql-async.js:4967:26)     at Object.global.exports [as callback] (mysql-async.js:4993:5)     at citizen:/scripting/v8/main.js:96:41 Error: (node:19052) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1) Error: (node:19052) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.
But i'v enter the good creditencial

Which version of mysql are you using. With just an error, you cannot be helped.
Mysql, MariaDB, etc. How do I replicate an error and fix it, if I do not know anything about it.

it's just was because i was using a mysql server on another computer , when i move the mysql server on the hosting machine , it was working :)

Okay thanks.


Containers at Hyperspeed - Container Camp London

Part of the transcription pipeline.

@jbenet Did the first of these; is there anything you else you would like me to do for these transcriptions before I ship? I went through the transcript while listening to the video, made manual edits, and then moved the file over here, adding the video file and a short paragraph to the page.

The images need to be added to the same directory as index.md, and then just referenced as is, e.g. ![some img](image.jpg)

@lgierth Should be fixed now.

This LGTM but someone else may want to weigh in on the blog end.  @em-ly?

Closing ancient posts that never got posted.


Support custom contents of states in StatefulButton

Let's expose the whole contents of the StatefulButtons
We can test if stateOne is a React element with https://facebook.github.io/react/docs/react-api.html#isvalidelement and just pass it through if it is, if not we can do what we do now.

Hi @interactivellama !
I'm guessing the context has changed much since this issue was raised. States are now exposed to the user e.g. here and an object is passed in as a prop.
Thus I want to clarify if this statement still applies since a React element isn't used anymore:
We can test if stateOne is a React element with https://facebook.github.io/react/docs/react-api.html#isvalidelement and just pass it through if it is, if not we can do what we do now.

If that's the case, is this issue still valid? Would be glad to do any follow up work :)

This one of the older components. The way this component's props would work now to align with other components in the library would be to pass in a complete <Button/> component into a defaultButton, a selectedButton and a deselectButton and deprecate stateOne, stateTwo, and stateThree.
The top level props would need to still be passed on, but removed the from the public API.
I wouldn't call this one a high priority, but if you'd like to propose completely new props, I'd be open to that as a good initial step for creating a scrum "definition of ready."
We'd want to not break the old props, but we could do a test on if stateOne had a value, and when the old API was detected, it would switch the render to a sub component which would be the current index.jsx just renamed button-stateful/private/deprecated-button-stateful.jsx or something like that..

I just reviewed the markup again and we don't want to pass in three full Button components. I'm going close this for now. If any consumers discover they need their icons on the right or something like that, please re-open.


Found this useful hope it helps

@konklone I found done by a friend of mine using neo4j http://neo4j.com/blog/geospatial-indexing-us-congress-neo4j/  can you check it out since its related to congress

Hi @neilmartis, thanks for sending this, but I'd prefer for inquiries like this you send them by email to<EMAIL_ADDRESS>so as not to unnecessarily email the watchers of this repository.


(PUP-8507) Add function join from stdlib to puppet

In addition to being backwards compatible this version of join() also:

adds support for joining Iterables (such as from reverse_each and
Hash
adds support for joining nested hashes and arrays nested in hashes
improves documentation by including several examples and explains
details about how the join behaves.

CLA signed by all contributors.


= Symbol Breaking Decouple

Using decouple with a Django project to separate passwords from my settings.py file. I've got an element in a .env file, i.e:
SECRET_KEY=myrandompassword124$%=
Which returns:
raise UndefinedValueError('{} not found. Declare it as envvar or define a default value.'.format(option))  decouple.UndefinedValueError: SECRET_KEY not found. Declare it as envvar or define a default value.
I'm guessing the equals sign at the end of the string is throwing decouple for a loop. Any ideas on how to fix? I'd prefer not to have to change the Django key.

same error! why closed???? without solution

If we can’t reproduce it, we can’t help. Feel free to fix it.
Em qui., 25 de fev. de 2021 às 18:49, Qoda<EMAIL_ADDRESS>escreveu:

same error! why closed???? without solution
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/henriquebastos/python-decouple/issues/56#issuecomment-786251196,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAC6ISUB5I3N7JT3JCYT2TTA3AYJANCNFSM4EK7X7IA
.

--
Henrique Bastos http://henriquebastos.net/
Welcome to the Django http://welcometothedjango.com.br/
Muito além da programação!
Twitter http://twitter.com/henriquebastos, Youtube
https://www.youtube.com/henriquebastosnet, Facebook
https://www.facebook.com/henriquebastos, Telegram
http://t.me/henriquebastos
+55 21 99618-6180 https://hbn.link/whatsapp-duvida

my secret key have "=" too!

Hey, we have a solution! The problem is running on WINDOWS! see that: https://gitter.im/NIT-DGPortal-main/Lobby?at=5c5ea1c95095f6660cc3adf7

if you use Windows, then the name of the file .env should be settings.ini

So we altered the docker-compose.yaml env file to settings.ini and all works! windows sucks!

Same problem 3 years later, any solution ?


typescript error

ERROR in node_modules/push.js/index.d.ts(3,20): error TS2714: The expression of an export assignment must be an identifier or qualified name in an ambient context.

I think it was merged already.

Still not right in .08
ERROR in node_modules/push.js/index.d.ts(3,20): error TS2714: The expression of an export assignment must be an identifier or qualified name in an ambient context.

@tony-gutierrez @ukon1990
A PR fixing the issue has been merged. The change would be pushed to npm very soon. I apologise for the inconvenience.

Just released v1.0.9. Should be fixed now.
Closing the issue.


fix(payment): INT-6115 Payment with hosted credit card

What?  INT-6115
Allow to pay with hosted payment
Why?
the payment was failing when Feature_HostedPaymentForm was enabled
Testing / Proof
Video
Siblings PRs
https://github.com/bigcommerce/checkout-sdk-js/pull/1583
bigcommerce/checkout

@vmparra  we already have test with hostedpayment props because HostedCreditCardPaymentMethod inherited from CreditCardPaymentMethod https://github.com/roberbnd/checkout-js/blob/INT-6115-followup/packages/core/src/app/payment/paymentMethod/WorldpayCreditCardPaymentMethod.spec.tsx#L150

@vmparra we already have test with hostedpayment props because HostedCreditCardPaymentMethod inherited from CreditCardPaymentMethod https://github.com/roberbnd/checkout-js/blob/INT-6115-followup/packages/core/src/app/payment/paymentMethod/WorldpayCreditCardPaymentMethod.spec.tsx#L150

Not sure this is entirely true, @roberbnd . I'm sure there must be something we can do to test this particular payment method.

@vmparra could you review, please? ♻

💚 Tested!
QE Details:
Access Worldpay 3DS Purchases now support Hosted Form enable, tested multiple scenarios of 3DS1 and 3DS2, I attach evidence of the test made for this pr:
INT-6115 Payment with hosted credit card.pdf


added support of XGBClassifier

#922

Codecov Report

Merging #928 (102e126) into master (49d8254) will decrease coverage by 0.41%.
The diff coverage is 50.00%.

@@            Coverage Diff             @@
##           master     #928      +/-   ##
==========================================
- Coverage   85.90%   85.49%   -0.42%
==========================================
  Files         141      141
  Lines        9083     8934     -149
  Branches     1515     1516       +1
==========================================
- Hits         7803     7638     -165
- Misses        957      966       +9
- Partials      323      330       +7

Impacted Files
Coverage Δ

dffml/util/testing/consoletest/commands.py
77.12% <50.00%> (-0.37%)
:arrow_down:

tests/df/test_memory.py
83.60% <0.00%> (-10.15%)
:arrow_down:

tests/operation/test_sqlite_query.py
93.33% <0.00%> (-6.67%)
:arrow_down:

tests/util/test_net.py
96.36% <0.00%> (-3.64%)
:arrow_down:

dffml/operation/math.py
80.00% <0.00%> (-1.82%)
:arrow_down:

tests/test_docstrings.py
95.65% <0.00%> (-1.77%)
:arrow_down:

dffml/cli/config.py
57.69% <0.00%> (-1.57%)
:arrow_down:

dffml/service/dev.py
71.21% <0.00%> (-0.96%)
:arrow_down:

dffml/operation/model.py
84.21% <0.00%> (-0.79%)
:arrow_down:

dffml/util/net.py
73.84% <0.00%> (-0.79%)
:arrow_down:

... and 57 more

Continue to review full report at Codecov.

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

The squashed commit message for this PR would be:
model: xgboost: Added XGBClassifier

Fixes: #922

Please rebase master into this then squash and I'll merge

Please rebase master into this then squash and I'll merge

@pdxjohnny I did all those steps and now, It's ready to merge.


Dataset

Description
Added new tripper.dataset module for documenting datasets.
All the main features documented in the module docstring have now been implemented.
The tests currently fails because emmo:hasDimension has been removed from EMMO. How should we deal with this?
For the next PRs, we should:

[ ] Add some kind of session with a user-specific information that should not be stored in the triplestore, including:
- credentials like username and password to external storages
- local root for relative paths
- default storage (e.g. redis) to work against during execution of the user workflow
Exactly how this session should work is still unclear. It will probably be a class with the functions documented in the
top of the dataset module as methods + some additional methods for dealing with the internal state.
[ ] Decide whether we want to move the protocol plugins from DLite to Tripper? It may also be possible to create a new package.
[ ] Update CONTEXT_URL to point to master branch (or to github pages if we copy the context there)
[ ] Create a good example/tutorial
[ ] Document the @context
[ ] Fix the __TODO__'s in the code

Type of change

[ ] Bug fix and code cleanup
[x] New feature
[ ] Documentation update
[ ] Testing

Checklist for the reviewer
This checklist should be used as a help for the reviewer.

[ ] Is the change limited to one issue?
[ ] Does this PR close the issue?
[ ] Is the code easy to read and understand?
[ ] Do all new feature have an accompanying new test?
[ ] Has the documentation been updated as necessary?
[ ] Is the code properly tested?

We have not discussed and agreed that the methods intended for dataset loading and fethcing from a database should be in a different module. Please separate out those methods and write in the moduel documentation that these are likely to be moved out of tripper at a later stage, since tripper is dedicated to the interaction with a triplestore, i.e. det datadocumentation and not the raw data itself.

OK. Will do...

We have not discussed and agreed that the methods intended for dataset loading and fethcing from a database should be in a different module. Please separate out those methods and write in the moduel documentation that these are likely to be moved out of tripper at a later stage, since tripper is dedicated to the interaction with a triplestore, i.e. det datadocumentation and not the raw data itself.

OK. Will do...

done


Missing formula in homebrew

I'm trying to install with homebrew but it's seems to be missing
$ brew install asset-catalog-tinkerer
Error: No available formula with the name "asset-catalog-tinkerer"
==> Searching for a previously deleted formula...
Error: No previously deleted formula found.
==> Searching for similarly named formulae...
Error: No similarly named formulae found.
==> Searching taps...
Error: No formulae found in taps.

You need to be using Homebrew Cask, not just Homebrew.


soca_dirac produces erroneous results when using "hybrid" covariance model

Description
When soca_dirac.x is run with hybrid covariance model, the results are not as expected. Tried running experiment that would compute covariance using dual correlation length scales. The resulting dirac results for SSH were smaller by a factor of about 100 compared with running with single length scales.
Output
Attaching file showing the results using large corr length scales, 10x rossby_radius just for sanity check,  (left), small scale,1x rossby_radius,  (middle) and equally weighted hybrid (right).

turns out dirac works fine, it's just that the output files are being overwritten. Issue to be addressed by https://github.com/JCSDA-internal/oops/issues/1634


Fix issues with Platform Dependent Routine pages

'Platform Dependent Routines' section was empty
'Caller Routines' section was redundant and has been removed
Sub-pages title needed a space (e.g. Routine: ZTBKCVXDPlatform: DSM/VAX)

https://issues.osehra.org/browse/VIVIAN-406
Change-Id: I6bf521d36df0288a605cb0bfcac2f8c58c76c54b

Looks good! Just one fix of a conflict and we'll get this merged.


Simplify social images and add ability to set per-page images

Addressing a few things after v5.2.0 went stable:

Removes the homepage summary layout in favor of the larger image card (makes it more consistent with other pages)
Reuses new thumbnails for Webpack, Parcel, and Vite guides to add their new social images
Updates default social card to be larger for more retina friendly rendering and with new design

Possibly more custom cards to come, but might push that to separate PRs. Ideas welcome on this :).

@julien-deramond Implemented your suggested changes and cleaned up a couple things. Thanks!


Add new primitive: Butterworth filter

Description
A low pass filter (in this example a butterworth filter) for the preprocessing of time series data.
The outcome of such a filter should be similar to the moving aggregations, but the number of samples will not be decreased and therefore might improve the performance of the pipeline.
It takes an array of the data, that should be filtered, as input and returns another filtered array.
What I Did
I started implementing this primitive for testing purposes in the butterworth branch on my fork, which you can check out.
Concretely, I added a Primitive JSON file and a custom function in timeseries_preprocessing.py.
Any feedback on the primitive itself and the implementation would be highly appreciated.

Thanks for the proposal @AlexanderGeiger
Some thoughts and comments:

Perhaps it would be better to split this in two parts and make these First Type (JSON only) function primitives.
We could have:

scipy.signal.butter.ba.json, which points at scipy.signal.butter, has N and Wn and btypeas tunable Hyperparameters and output=ba and analog=False as fixed hyperparameter and which inputs nothing but returns b and a, which will be set as context variables.
scipy.signal.filtfilt.json, which points at scipy.signal.filtfilt, has axis as fixed hyperparameter and padtype and padlen as tunable hyperparameters and which inputs a, b and X and returns X.

Optionally, we would add scipy.signal.butter.zpk in the future if needed.
Doing this, no python code is needed and both primitives can be freely combined with other options.

If we do not make them JSON Only and we build a custom python function, consider:

not sorting the timeseries and not even requiring a time index: a single sequence without time index can be also processed. Assume that it has been already sorted before. This would, enable, for example, using this primitive right after a downsampling made by timeseries_preprocessing.time_series_aggregation, which outputs X and the time index as two different variables.
supporting both numpy array and pandas DataFrame. This is not mandatory, but if possible, it's better if primitive support both types of inputs.
making the output match the input format: if you are given a DataFrame with a time index column, return a DataFrame with a time index column. If you are given a 1d numpy array, return a 1d numpy array.

Thanks for the feedback @csala
I like the first approach and will try to implement the primitive this way.


Correct the spelling of CocoaPods in README

This pull requests corrects the spelling of CocoaPods 🤓
https://github.com/CocoaPods/shared_resources/tree/master/media
One day I’ll make a bot that looks through the READMEs of all Pods, looks to see if it uses “Cocoapods” and PRs “CocoaPods” :D— Ørta (@orta) February 10, 2016

Created with cocoapods-readme.

I almost don't want to merge this out of sheer pride.

No worries hehe


Remove cfg and config from PrintListImages()

There are two unused parameters cfg and config in the PrintListImages() function under pkg/service/list_images.go.
Since these are unused, it would be better to remove them from the function.
/good-first-issue

Hey ! i would like to work on this issue ! @rakshitgondwal

/assign @BalaadityaPatanjali


Readme template: SquadLayerFilter Dead Code?

Description of Issue
The readme template has references to squad layer filter, which I'm unfamiliar with; I tried tracing it back inside of the git history and this looks like dead code/documentation to me?
https://github.com/Team-Silver-Sphere/SquadJS/blob/master/squad-server/templates/readme-template.md#squad-layer-filter
I'm unfamiliar with what this was trying to do ( Build a filtered Layer list for plugins to use?)

I agree, it looks to be dead code, so it can probably be safely removed.


Repackaging - Illusive Platform (MMA to AMA Migration)

Required items, please complete
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

/Package


feat(acls) add admin endpoints to interact with acls

/acls/ to paginate through all acls for all consumers
/acls/:acl_id/consumer to retrieve the Consumer associated with an acl

Issues resolved
Fix #2188
Close #2371

One feature that is missing:
Retrieve ACL group names
I feel having an endpoint GET /aclgroups to return all the group names would be pretty helpful.
The problem I'm facing, it seems there is no straightforward way of doing:
select distinct group from acls;

Is there away around of doing this, or we will need to update the core DB logic? I could take a stab at it if there is another use case for such a query.
IMHO, this would be pretty useful for admin purposes.
Another issue is we don't have group as a resource or entity really so this is not RESTful.
@thibaultcha Thoughts?

@hbagdi Very nice! This is super exciting.
The issue indeed is that there is no way to distinctively select groups. The DAO doesn't support this feature, and cannot, as the limitation comes from Cassandra itself (and the data model chosen for this entity).

An inefficient implementation could be a brute-force approach, which traverses through an entire paginated set of ACLs and figure out the unique groups.

If carefully written this could be fine but it isn't ideal that's for sure. I'd rather not risk it, iterating over entire tables/column families (in Cassandra) is very much a discouraged practice. We have users and customers using those features more or less extensively, so this could cause serious performance issues (potentially slowing down an entire cluster) for those heavy users.

An endpoint such as /acls/:groupname/consumers would be immensely helpful.

If we write a migration to create an index on the group column (and make the appropriate changes in the schema), this could be fine considering the supposedly low cardinality of this attribute. The /aclsgroups endpoint would still be out of reach though.
Ideally, we come up with a better ACL solution once our new data model is completed (we are working on it).

@thibaultcha
I understand the root causes of this not being possible.
Let's push these features for a later time.
I'll open an issue just for tracking purpose.
I've been thinking a little bit about grouping or tagging of APIs/Consumers.
Group or Tag as a Kong entity to make cohorts of APIs and/or Consumers.
Such a feature could be supported by a plugin but then it would create interdependency between plugins, which is not a problem but not an ideal solution as well.
It's also possible to start an experimenting with such a plugin and then move it into the core if there's traction.
Although, supporting it from the core would give the ability to maintain/manage a group of entities with ease.

@thibaultcha Since, this PR is related with other *-auth changes in admin API, do you think we would be able to include this in the upcoming release?

I just had a look and it seems like the acls column family has such an index already for both PostgreSQL and Cassandra. It seems like we could do the /acls/:group/consumers endpoint after all!
However, I'm not sure how well this will play with Lapis and the other endpoint we register: /acls/:acl_id/consumer...

I just gave it a try, and, lucky for us, it seems like Lapis is perfectly able to make the distinction between the two endpoints!
  ["/acls/:acl_group/consumers"] = {
    GET = function(self, dao_factory, helpers)
      return helpers.responses.send_HTTP_OK("/acls/:acl_group/consumers")
    end
  }

$ http ":8001/acls/$(uuid)/consumer/"
HTTP/1.1 404 Not Found
...

{
    "message": "Not found"
}

$ http ":8001/acls/$(uuid)/consumers/"
HTTP/1.1 200 OK
...

{
    "message": "/acls/:acl_group/consumers"
}

I have just opened #3051 to address the failures when using Cassandra for your previous contributions. Mind giving it a look?
Also, FYI, our merging window will be closed this Monday, before the release on Wednesday.

Regarding:

I just gave it a try, and, lucky for us, it seems like Lapis is perfectly able to make the distinction between the two endpoints!
["/acls/:acl_group/consumers"] = {
    GET = function(self, dao_factory, helpers)
      return helpers.responses.send_HTTP_OK("/acls/:acl_group/consumers")
    end
  }

I had tried the same thing and in fact the endpoint was not an issue but processing the request with pagination support.
Since, we can find Acls belonging to a single acl_group but then we need to perform a lookup  for the Consumer of every Acl with acl_group.
In such a case, if a group has a few thousand Consumers then we need pagination support for the endpoint.
A hack comes to mind, where we could internally paginate based on acl_group and not Consumer.
What I mean, internally we do a paginated GET /acls?group=foo. And replace the data in response with a Consumer array instead of an Acl array.
On a request for next page, we lookup Acls and then replace Consumers.

@thibaultcha Good catch with the pagination support.
I was thinking about documenting this but since this is very harmless, we can skip for better end-user understanding and experience.
I've included your patch in the updated commit on this PR.

Yes, the pagination in such a case might be tricky and might require custom usage of the DAO (the "CRUD helpers" are, after all, just helpers for the most common tasks.
Maybe we should leave this out of 0.11.2 and I am thinking of simply taking this PR as is for now.

Yes, let's take this PR as for 0.11.2.
Let me know if we need to update anything.
On Nov 26, 2017 1:11 PM, "Thibault Charbonnier"<EMAIL_ADDRESS>wrote:

Yes, the pagination in such a case might be tricky and might require
custom usage of the DAO (the "CRUD helpers" are, after all, just helpers
for the most common tasks.
Maybe we should leave this out of 0.11.2 and I am thinking of simply
taking this PR as is for now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kong/kong/pull/3039#issuecomment-347038624, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AEeiOfWExGljB5rVkFGMW77gx0iKUT6Xks5s6dPqgaJpZM4Qi3Zw
.

Manually merged with some minor modifications - thank you very much for your work!

Hi everyone,
I've seen all the discussion about the endpoint /acls/:acl_group/consumers.
Anyone have any news on this?
I need a lot to list all consumers from a group.
Thanks!


Text Wrap for Song Title & Artist

[ IMPORTANT: Please remove or replace everything between two square brackets ( [ ] ) before posting your issue. ]
Is your feature request related to a problem? Please describe.

The song title & artist takes up a whole screen. Small scale just make the song title & artist miniscule and thus not very noticeable and pleasant to the eye.

Describe the solution you'd like

An option to wrap the text (song title & artist)

Describe alternatives you've considered

Levers to change the height and the width of the text respectively instead of only one scale lever for both width and height

Additional context
Nothing.

Hi i scond this demand if possible. Or at least is it a way to separate cover and song info scale bar in settings ?
Because so far if you narrow trhe size of song info cover picture begins to be to small to be usefull or pretty :)


Removing "Job Submission" for applications

Not so much a universal issue as more or a process alignment issue.  Can we remove the Internal Submission function without wrecking the overall functionality of the Career Portal?
Current behavior
All applications from the career portal are automatically logged as an Job Submission.
Desired behavior
All applications are loaded as a candidate and flagged as a Web Response on the appropriate Job but no Job Submission is completed.

Hey Royce,
With web responses, they are Job Submissions with the status of "New Lead" where in the application they are displayed as web responses.  They are the same entity, but in the application they should display separately.
Are you seeing different behavior?

No - that is the behavior we are seeing.  However, this breaks or process and associated reporting metrics.  We are looking to see if we can remove the Job Submission component, but retain the rest of the functionality.  The main reason is we only add candidates as a job submission when they are fully vetted and what is happening is any Portal Applicant is showing Job Submission dates that are a week or more older when they should be showing as the current date (does this make sense?)

I gotcha, currently with the public API used within the Career Portal, it uses a single apply function that creates a candidate and adds them to the job as you are able to see.
The functionality that you desire would require extending the app with the normal REST API to add a candidate manually then add a note to a job and candidate to possibly track what job they applied to.
Does this sound like what you are looking for?  The biggest challenge would be to keep the association to the job they applied to, as a Job Submission establishes that relationship between the Job and Candidate.
If you wanted to query the existing data in your system, and exclude when your web responses were added in your search, you can utilize the Job Submission History to determine when a Job submission was changed to a status that was not New Lead.

Okay, that might work provided we can associate the note made to the specific job query...will investigate.


Cannot connect to the socket server

⚠ Please verify that this bug has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
It says "Cannot connect to the socket server. [Error: websocket error] Reconnecting..." but i don't have reverse proxy setup
🐻 Uptime-Kuma Version
Not sure installed it 3 days ago
💻 Operating System and Arch
Windows 10 Pro
🌐 Browser
Brave
🐋 Docker Version
No response
🟩 NodeJS Version
No response

I'm seeing the same problem and am also NOT using a reverse proxy.
Docker Version: 20.10.22, build 3a2c30b
Docker Compose: v2.14.1
Host OS: Ubuntu 20.04 LTS
Uptime-Kuma Version: "latest" tag as of 6 days ago.
docker logs -f uptime-kuma produces no additional output when I attempt to access the dashboard.
In my case, the Uptime Kuma host is not internet connected. I don't believe this should be problematic, but thought it worth noting.

Can anyone advise how I can go about extracting more log information? I even entered a bash environment within the uptime-kuma container and ran node server/server.js --verbose but I'm neither seeing anything relevant to the error in the web interface, nor indeed any log activity when I reload the page.

I am experiencing the same problem.
"Cannot connect to the socket server."
I do not use a reverse proxy.


Create a grid for the Worksheet Page

At the moment all of the worksheets are listed one by one down the page. I originally pictured it as a grid with around 3 worksheets in a row. Initially I tried to implement this with Jinja2, but it went wrong and I couldn't get the syntax right. I think a more straightforward way to make this happen would be with JQuery.
If Jquery will not work I will go back to doing it with Jinja2, because I believe it was just that the syntax was getting complicated, but complicated syntax can be overcome it is just that the code will not be pretty.

As of July 16 this is done


More metrics about DNS queries sent by Armeria

We currently only record the time taken for DNS queries. A user wants to learn more, such as:

the number of DNS queries
the number of connection attempts that required a domain name resolution

with some tags for aggregation, e.g.

Success or failure
DNS error code
UDP or TCP
Query type (A, AAAA, CNAME, ...)

Please let us know if you have anything to add.

Related: https://github.com/line/armeria/issues/1921

We could also allow the customization of meter tags.

Working in progress.

@trustin It looks to me we will have to implement DnsQueryLifecycleObserver which will be responsible for capturing metrics about DNS queries. We will need a custom DnsQueryLifecycleObserverFactory that returns a new DnsQueryLifecycleObserver that is responsible for capturing metrics. Is there a standard way in armeria code base that we use to capture metrics ? Also how are we going to expose the captured metrics for DnsNameResolverBuilder?
You have mentioned in the description for the ticket above that "record the time taken for DNS queries". Can you please direct me to the class that is doing this?

@amitvc We use Micrometer to record metrics. Your DnsQueryLifecycleObserver will require a MeterRegistry at its construction time, which could be acquired from ClientFactory (see ClientFactory.meterRegistry()).
We currently collect the time taken for DNS queries in (Default)RequestLogBuilder.session() via ClientConnectionTimings. You could record the metrics for the DNS query duration there, again using MeterRegistry from the ClientFactory.
I hope this makes sense. Please feel free to let me know if you need more help. :smile:

@trustin ClientConnectionTimings records timings for each connection.

the number of DNS queries
Are we measuring the total number of DNS queries[different DnsRecordType] made within scope of single client request?

the number of connection attempts that required a domain name resolution
This seems like a global metric. We want to measure number of connection attempts that require DNS resolution. Is this a metric that is required to be kept for a client pool? Not sure about this one.

@amitvc

the number of DNS queries - Are we measuring the total number of DNS queries[different DnsRecordType] made within scope of single client request?

I think it's OK to have the counters in the scope of ClientFactory, i.e. no need to track individual requests.

the number of connection attempts that required a domain name resolution - This seems like a global metric. We want to measure number of connection attempts that require DNS resolution. Is this a metric that is required to be kept for a client pool? Not sure about this one.

Probably OK to keep it in the scope of ClientFactory as well? A user could compare it with the total number of requests sent. What do you think?

@trustin I am trying to understand how clients use the ClientFactory in real-world scenarios. Do they use HttpClientFactory implementation to create and manage httpClients as and when they are needed?  I see httpClientFactory has HttpClientDelegate which has the ClientConnectionTimingsBuilder that I need. I am trying to figure out how DnsQueryLifecycleObserver into play here.

@amitvc ClientConnectionTimingsBuilder is for request-scoped timings, so you don't need to touch that. All we need to do here is to implement DnsQueryLifecycleObserver to record the metrics into MeterRegistry of ClientFactory.

@trustin I have created a draft PR for this https://github.com/line/armeria/pull/2880
This is not complete but wanted to get your opinion if I am heading in right direction

What I can confirm that my DefaultDnsQueryLifecycleObserver callbacks are being called correctly. But I need ClientConnectionTimings like facility to capture actual timings for dns queries.


[angular] do major refactoring

better organize angular code, create directives, break controllers and services in different files, create all the needed services

comit https://github.com/inmarelibero/FriendsMappin/commit/ddf510a8c633fa8e5622d2a8d4ee249c9f3b1bf2 is refenced


Add the ability to upload .docx files to the NJ AI Assistant

Context
With the changes in this branch, users of the NJ AI assistant will now be able to upload .docx files when using the AI assistant.
This branch also includes some minor improvements to the repository:

Basic unit tests around uploads as well as new scripts for running these tests separately for the frontend and backend
Updates to the NJ README to give some information on running the code + tests locally

Future Additions
Noting that in a future branch we should continue to add more unit tests + add the ability to run tests through github actions

is this in prod yet?


Pin sqlite3 version to 1.*

This commit addresses the CI failure.
It is Rails 7.2 that allows sqlite3 version 2.0. Previous versions of Rails are conventionally configured to require sqlite3 "~> 1.4" (meaning it excludes 2.*), which the development dependency can naturally follow.
cf.
https://github.com/rails/rails/blob/v<IP_ADDRESS>/activerecord/lib/active_record/connection_adapters/sqlite3_adapter.rb#L14
This stops "bundle exec rake" from causing such failure like this, copied from GitHub Action's log:
>> BUNDLE_GEMFILE=/home/runner/work/public_activity/public_activity/gemfiles/rails_7.1.gemfile bundle exec rake
/opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundler/rubygems_integration.rb:237:in `block (2 levels) in replace_gem': can't activate sqlite3 (~> 1.4), already activated sqlite3-2.0.1-x86_64-linux-gnu. Make sure all dependencies are added to Gemfile. (Gem::LoadError)
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/gems/3.3.0/gems/activerecord-<IP_ADDRESS>/lib/active_record/connection_adapters/sqlite3_adapter.rb:14:in `<top (required)>'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundled_gems.rb:74:in `require'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundled_gems.rb:74:in `block (2 levels) in replace_require'
	from /home/runner/work/public_activity/public_activity/test/test_helper.rb:26:in `<top (required)>'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundled_gems.rb:74:in `require'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundled_gems.rb:74:in `block (2 levels) in replace_require'
	from /home/runner/work/public_activity/public_activity/test/test_activist.rb:3:in `<top (required)>'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundled_gems.rb:74:in `require'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/3.3.0/bundled_gems.rb:74:in `block (2 levels) in replace_require'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/gems/3.3.0/gems/rake-13.2.1/lib/rake/rake_test_loader.rb:21:in `block in <main>'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/gems/3.3.0/gems/rake-13.2.1/lib/rake/rake_test_loader.rb:6:in `select'
	from /opt/hostedtoolcache/Ruby/3.3.1/x64/lib/ruby/gems/3.3.0/gems/rake-13.2.1/lib/rake/rake_test_loader.rb:6:in `<main>'
rake aborted!
Command failed with status (1)
/opt/hostedtoolcache/Ruby/3.3.1/x64/bin/bundle:25:in `load'
/opt/hostedtoolcache/Ruby/3.3.1/x64/bin/bundle:25:in `<main>'
Tasks: TOP => default => test
(See full trace by running task with --trace)
Error: Process completed with exit code 1.

Source: https://github.com/public-activity/public_activity/actions/runs/9204987054/job/25319683065

Thank you @sato11 !


Update cav25

Which conference does this PR update?

CAV 2025

Related URL

https://conferences.i-cav.org/2025/

please append the new one instead of overwriting the previous one.


WinSDK: repair build on case insensitive file systems

The Windows SDK includes WinSock2.h as winsock2.h in WS2tcpip.h which
breaks on case sensitive file systems such as ext4.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge

@swift-ci please smoke test and merge


mac-20 is dead.

Type of Request
bug
Infrastructure Environment
LUCI, Github, Cocoon scheduler, Autosubmit, etc...
What is happening?
mac-20 is dead.

Steps to reproduce
Step 1:
Step 2:
..
Step n:
Expected results
I expect to see X when Y is finished.

mac-20 is up and running fine, I can login remotely from my end.
flutter-devicelab-mac-20:~ swarming$ uptime
11:06  up  6:02, 3 users, load averages: 1.22 1.43 1.48

mac-20 is out of sync with Salt master, run command salt-call state.apply to re-sync again.


feat/android_show_consent_dialog

added showConsentDialog() for Android

Verified to run this on both Android and iOS, on Android, the consent dialog is shown, on iOS, the error message is printed in the log console
Added a subtask for the doc update

We have to update CHANGELOG.md actually - for all features added to the plugin - could you do that?

Sure, then do we have a new release?

You can put a placeholder for now - x.x.x

You can put a placeholder for now - x.x.x

Ok, updated CHANGELOG, please review.


Mirror api

I believe this is really good start (thanks, @lebauce), but it would require more work on downloading part: it is not great that downloading happens while mirror update call is running. Probably there should be some separate API to control download progress/status.
I would move this to 0.10 so that 0.9 would happen sooner or later.

Concerning the download progress/status challenge:
I am working on an API which allows queuing of tasks and each task can update its own status/progress. The idea is all API calls which need a write lock will be queued into this queue and processed one by one. A separate API call can then be made to check the progress of a task.
An experimental and unfinished implementation you can find here https://github.com/sliverc/aptly/tree/queuing_api
Open tasks I am currently working on:

Writers on API should not block out readers (rewriting of locking needed)
Rewriting of api PUT, DELETE, POST calls to use queue

This is an idea on how to approach the issue of long API calls which would make it possible to implement this mirror api PR more easily.

I am quite eager for a mirrors api, is this generally safe to use? Should updating a mirror via this API be avoided or is it simply a sub-optimal interface to the long running process?

I didn't check the code, I think the biggest issue mirror update, as it might take considerable amount of time, and blocking in API call doesn't make sense.

Closing in favor of #573


how to use VsCode  to debug ionic2? the first  step  setting

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
hi . i am trying debug ionic2  via VS Code  the latest version(ver 1.8.1)
follow the tutorial .
http://www.damirscorner.com/blog/posts/20161122-DebuggingIonic2AppsInChromeFromVisualStudioCode.html
and   there is a file  launch.json  which need  config it.
but this ionic2 rc4,  one more  configuration option  as below. how do i  config it ?
{
"version": "0.2.0",
"configurations": [
{
"name": "Launch Chrome against localhost, with sourcemaps",
"type": "chrome",
"request": "launch",
"url": "http://localhost:8100",
"sourceMaps": true,
"webRoot": "${workspaceRoot}"
},
  {             "name": "Attach to Chrome, with sourcemaps",             "type": "chrome",             "request": "attach",             "port": 8100,             "sourceMaps": true,             "webRoot": "${workspaceRoot}"         }
]
}
the Bond style is the  problem section,how do i setting it ?  (sourcemaps  already setting it.)

Hi
This is mine.
Debugger for Chrome extension
Google Chrome shortcut Target (Windows)
"C:\Program Files (x86)\Google\Chrome\Application\chrome.exe" --remote-debugging-port=9222

launch.json
{
    "name": "Attach to Chrome, with sourcemaps",
    "type": "chrome",
    "request": "attach",
    "port": 9222,
    "sourceMaps": true,
    "webRoot": "${workspaceRoot}/www",
    "url":"http://localhost:8100/"
}

You also need to use @ionic/app-scripts version 47 or greater.
hth
Tim

Hi @WillLeeBebi,
Please post this question in the Ionic Framework forum. We don't take questions on Github unfortunately. We use issues as a way to track work.
Thanks,
Dan


tests: Add alignment test

This test fails on ARM currently

Thanks! Do you know how to fix it or should I look into it?

Apparently it works with the arm asm removed, and built with generic.

Also, for another data point, if you modify that test to use d instead of d[1:] in the call to Hash and update data to remove the leading 00 (aka make it aligned), the test will pass.

OK, thanks. I've verified that it indeed failed the test (on Raspberry Pi 3) and removed ARM assembly until someone sends a PR with a fixed version. (I've created https://github.com/dchest/siphash/tree/arm-asm branch that points to the last commit before removal).

I'll take a look. Having a repro will definitely help unearth the issue.

@rakitzis thank you!

It's here:
	AND.S	$3,R10,R8
	BNE hashloopunaligned

In Go code, that is effectively:
if val&0x03 != 0 {
	hashloopunaligned()
}

I'm sure the problem is more subtle but I need to sit with the code when I can stop thinking about other things for a little while... it's tricky to get into the right mindset!

To clarify, I didn't mean to imply the root of the issue is there.  I was responding to @Kunde21 about the check that jumps to the check (although for the non 128-bit version).  That definitely is not the root cause of the issue for 32-bit and 64-bit.

@davecgh That's present in Hash, but missing from Hash128.

OK so that might be it, but it's best to sit with the test cases and make sure there is 100% coverage. Unaligned is considerably trickier to get right.

Answering my own question: it seems armv5 (supported by Go) requires unaligned accesses to be synthesized in software.


Add documentation for KubeOps operation "delete"

Change Overview
KubeOps was introduced in #1102 but documentation for operation "delete" is missing from "Kanister Functions".
Pull request type
Please check the type of change your PR introduces:

[ ] :construction: Work in Progress
[ ] :rainbow: Refactoring (no functional changes, no api changes)
[ ] :hamster: Trivial/Minor
[ ] :bug: Bugfix
[ ] :sunflower: Feature
[x] :world_map: Documentation
[ ] :robot: Test

Issues

#1102

Test Plan

[x] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

Resulting docs:

This LGTM :+1:. While you are here :wink:, can you also help us to add the Outputs subsection to describe the function outputs? Looking at the code, these should be the relevant fields :point_right:  https://github.com/kanisterio/kanister/blob/c71b5aa161e589364a40d5514d03e7052af0030f/pkg/apis/cr/v1alpha1/types.go#L58-L75. Thanks!


Milestone 3: Build React-Redux App

Project implementations:

Developed a React application using Redux for state management.
Created a homepage with a list of items that can be filtered based on user input.
Integrated an API of choice to retrieve numeric data for the categories.
Designed and implemented a details page to display detailed data for a selected category.
Created an About page with more details about the app.
Implemented navigation between the homepage and details page.
Ensured code adheres to HTML, CSS, and JavaScript best practices.
Added appropriate documentation for the project in a professional manner.
Ensured there are no linter errors in the codebase.
Followed the correct Gitflow process for version control.

API:

Used the Jikan API.
Selected the Financial Modeling Prep API for obtaining stock data.
Integrated the API into the application to retrieve numeric values based on parameters.

[ ] Hello @BenMKT please refrain from making changes after you request a review as it will alter the review and cause confusion I noticed you added 2 new commits.


A11y_Model Builder_Forecasting_Data_AI4D: (msedgewebview2/document 'Daytona Greeter Sample') Chromium components should be scanned with a web-based scanner.

GitHubTags
#WCAG1.3.1; #A11ySev2; #A11yTCS; #A11yWCAG2.1; #DesktopApp; #Win11; #Visual Studio Client; #SH_ModelBuilder_Win32_May2023; #Info and Relationships; #Win32; #Benchmark; #AI4D; #A11yAuto;
Environment Details
Application Name: Microsoft Visual Studio Enterprise 2022 (64-bit) - Current
Version 17.6.1
Microsoft Windows 11 Enterprise 10.0.22621 Build 22621
Repro steps

Launch the Visual Studio with at least one of the .NET workload.
Tab till enable ML.NET Model Builder (Tools -> Options -> Environment -> Preview Features) and select it.
Create a new .NET project or open an existing .NET project.
Open context menu(Right-click) of the project in the solution explorer.
Tab till Add and then tab till Machine Learning Model and press enter.
Model Builder dialog will open. Tab till Scenario tab.
Select Scenario screen will appear.
Tab till "Forecasting" and press enter.
Setup environemnt screen will appear, Tab till Local CPU tile and select it.
Tab till Next button and press enter.
Data screen will appear.
Turn on AI4D
Verify whether we are getting AI bug like chromium components should be scanned with a web-based scanner or not.

Issue observed throughout the application.
The following accessibility issue needs investigation.
App
msedgewebview2
Element path
document 'Daytona Greeter Sample'
Issue Details
Chromium components should be scanned with a web-based scanner. WCAG 1.3.1
How To Fix
Use a web-based scanner (for example, Accessibility Insights for Web) to scan this component.
This accessibility issue was found using Accessibility Insights for Windows, a tool that helps debug and find accessibility issues earlier. Get more information and download this tool at https://aka.ms/AccessibilityInsights.
Attachment

@zewditu
As per the discussion with SME, it is not a bug. Hence closing this bug.
#Regressed:06-14-23; #Closed;


Changelog.md not helpful

We did some work to auto update the changelog with every release.
After our first release the changelog information is poor, this needs fixing.

I have addressed this issue in my PR: https://github.com/holidayextras/ui-toolkit/pull/87


New patient page

was client_new, but I changed nomeclature to patient

Ben, just fixed the merge conflicts that came up after merging #27 for you on this branch. Let me know if anything is looking screwy.


Add Mock for $cordovaLocalNotification

Add the $cordovaLocalNotification plugin to the ngCordovaMocks module.

If you could create a pull request with the mocks for this; that'd be helpful.  Since there are a number of plugins we support, we rely on individual contributions to keep the project going.   Otherwise it'd be one of those "When we're able to get to it" sort of things, which may be different than your time table.


Version missmatch between zed and repository

Just a question, why is in Zed the version of the nix extension 0.1.0 and the latest version in the repository is 0.0.4 ?

The version in the repository is 0.1.0
https://github.com/zed-extensions/nix/blob/f17bcfae1df66b71a19b31dc22d1ea5098de9514/extension.toml#L4
The version in the zed extension store is 0.1.0

There are some recent changes in the repository which have not yet been released. But all it takes is a version bump here and then a PR to the zed-industries/extensions repo to publish the updated version.


[Feature Request] Add css snippet inline or as a tooltip to autocomplete list

Thank you for this extension, it was very much needed for VS 2022.
Would it be possible to add the actual css for the official tailwind classes to the auto complete as follows?

This is an example from the official tailwind extension for VS Code.
Thank you!

This is definitely a needed feature. I added it into the extension, but I did change make the CSS show up as a description instead of a suffix (as shown in this gif):

Thanks for the suggestion!

Thank you very much for implementing this so quickly! After updating the
extension to the latest version, I saw the auto-complete trigger briefly
but then an Object reference not set to an instance of an object dialog
popped up (and pops up on each subsequent key press)
[image: image.png]
Happy to provide any other details if needed.
On Tue, Aug 8, 2023 at 8:57 PM Theron Wang @.***> wrote:

This is definitely a needed feature. I added it into the extension, but I
did change make the CSS show up as a description instead of a suffix (as
shown in this gif):
[image: IntelliSense-Demo-1]
https://user-images.githubusercontent.com/65189291/259274852-8da1be72-0918-447b-aafb-5ecac09e97f9.gif
Thanks for the suggestion!
—
Reply to this email directly, view it on GitHub
https://github.com/theron-wang/VS2022-Editor-Support-for-Tailwind-CSS/issues/7#issuecomment-1670543012,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALUFCN4KUCPZDK3CPSNDADXULVBTANCNFSM6AAAAAA3HEZ3PQ
.
You are receiving this because you authored the thread.Message ID:
<theron-wang/VS2022-Editor-Support-for-Tailwind-CSS/issues/7/1670543012@
github.com>

Sorry about that, not sure what happened with the image.


Removed incompatibility with django3.0+

Fixed the six ImportError
Updated django-crispy-forms

Also import six from django.utils in crud_tag.py


Alerts are rendered under FormScreen overlay

Description
I've tried to show a notification in onSubmit of useFormScreenPlugin and it rendered below the overlay
Steps to reproduce
const cms = useCMS()
const [values, form] = useForm({
  // ...
  onSubmit() {
    cms.warn('Not implemented!')
  }
})

Expected result
Alert should be rendered on top of overlay
Actual result

Environment
Tinacms Package Versions

├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>│ ├─┬<EMAIL_ADDRESS>

Tools

Tool
Version

browser
Chromium  85.0.4183.121 and Firefox 81.0

os
Archlinux

node
14.2.0

npm
6.14.4

It should be made optional to show it as an overlay @ncphillips . Can I work on this ?

Update: This issue will be fixed by #1519 which should be released today
@prakhargurunani Do you mean that it should be possible to turn off the overlay for Form ScreenPlugins?

@prakhargurunani Do you mean that it should be possible to turn off the overlay for Form ScreenPlugins?

Yes

Great idea! Feel free to dig in, but let's make a separate issue for that 🙂


Exposing signature validation errors?

Currently whatever error causes the SAML response signature validation to fail, the same error message is returned.
Would you be open to exposing these somehow? (Either in more detailed error messages, or via a property on the SAML object and method on the strategy to read it), or is hiding the specific error a deliberate decision to make attacks harder?

(specifically, would a pull request adding this be welcome?)

I may be able to help with this; I've just spent all day with the Node debugger attached to a process in my production environment hunting down an invalid signature. Looking through the code, in most cases it should be as simple as just throwing an error instead of just returning false. In others - those which use xml-crypto to validate signatures - it may be as easy as check the return code then throwing sig. validationErrors (an array of error messages collected in xml-crypto when validating signatures).

@ccampanale Just yesterday I realized a new version of passport-saml that dealt with this. Instead of returning "Invalid Signature" in 6 different places, each place now gets a unique error message. Does that meet your needs, o are their places where you want to pass along more detail as returned by the XML parser?

@markstos I didn't even realize that! Let me bump our package refs and take a look. I'll let you know. Thanks!

👍 Sorry I never got around to making a PR 🙈

Bumped the version and tested. Still seeing our same issues but I'm pretty sure the issues are with our IDP. The new error messages are nice as they let you know where exactly in saml.js the issue is occurring but there are some errors from xml-crypto that could be relayed as well as I mentioned above. (Not necessarily parser level errors, though that may be helpful in some cases?) It's not a huge deal but I could see how it may save folks from needing to do a deep dive with the debugger as I have the past couple of days.

@ccampanale If you'd like to make a PR to pass along the detailed failures from xml-crypto, I'm up for merging that as well. XML parsing has historically had some security flaws, so if XML is parsing sometimes fails in strange ways, it could be useful to log details for forensics as well as to debug IdP connections.

Yeah, since I'm still fighting the good fight with my organization's IDP and can't get our production deployment to work I may very well do that. As you said, if nothing else it may save others a good deal of time troubleshooting strange failures and/or from a forensics collection perspective.

I just about pulled my hair out with a recent integration, only to eventually realize that the profile field mapping between them and us wasn't as expected. It's increasingly tempting to consider using Okta or OneLogin for their "Service Provider" offering.


WIP: Unify the two versions of PackageDescription into a single one using availability annotations instead of compile conditionals

Replace the compile-time conditionals in PackageDescription with availability annotations.  This will let us build a single PackageDescription, which will make the build scripts easier (with the goal of being able to build it entirely from the package itself).
Motivation:
This is preparatory work for https://github.com/apple/swift-package-manager/pull/3431 and will also make it easier for clients of libSwiftPM, since only one version of PackageDescription needs to be built.
Modifications:

replace compile-time conditionals with availability annotations
adjust a unit test to account for a different error message now when using 4.2 and trying to use ints for SwiftLanguageVersion

Result:
This builds PackageDescription libraries that aren't dependent on the compile time condition.  This is a WIP because:

the build scripts still need to be adjusted
the manifest loader can be adjusted to not specialize on PackageDescription version
we need to figure out if the unsafeFlags are going to be a problem for the PackageDescription for clients

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please smoke test

Error doesn't look related.  Rerunning.

@swift-ci please smoke test linux

@neonichu Do you have any concerns about this change?


New

This is a...

[ ] Feature Request
[ ] Bug Report

Problem:
Proposed Solution:
Page to Update:
https://kubernetes.io/...

/close


Create a Space Exploration Hub

Pull Request Details:

Title: Create a Space Exploration Hub
PR Link: #27
Issue: #26

Next Steps:
I'm ready for feedback and any necessary revisions. Please review at your earliest convenience, and feel free to provide comments or suggest improvements.

Hi Paras,
Thank you for your pull request. I appreciate your contribution to our project.
I have reviewed your changes and everything looks good. Please be patient, and I'll get back to you as soon as possible.
If you have any questions or need further assistance, don't hesitate to ask.
Best regards,
Jayesh


style: add very_good_analysis

Status
READY
Breaking Changes
YES

final argMatcher = ArgMatcher(matcher, null, capture); to final argMatcher = ArgMatcher(matcher, null, capture: capture);

Description
Add very_good_analysis in mocktail and mocktail_image_network
Type of Change

[ ] ✨ New feature (non-breaking change which adds functionality)
[ ] 🛠 Bug fix (non-breaking change which fixes an issue)
[x] ❌ Breaking change (fix or feature that would cause existing functionality to change)
[x] 🧹 Code refactor
[ ] ✅ Build configuration change
[ ] 📝 Documentation
[x] 🗑 Chore

This is a duplicate of #154

Yes, but the PR has not been updated for 23 days for the CI pass.

Yes, but the PR has not been updated for 23 days for the CI pass.

That's because the latest version of Very Good Analysis introduces breaking changes and also forces an upgrade to Dart 2.18

I reverted the part that created a breaking changes in the package.


Auto-discovery and Auto-connecting among the peer nodes is not working.

We have launched network with 3 nodes(Validators), With the following power (n1-->1, n2-->10, n3-->10).
n1 is the first node, n2 is launched by adding n1 as a persistent peer, n3 is launched by adding n1 as a persistent peer.
After few blocks we stopped n1 and n2 was not able to connect with n3.
In cosmos don't we have any auto nodes discovery feature?
What will be the solution for this problem?
Is there any suggestive node topology for the system?
How to use seed node here and does it solve the problem?

Can someone comment on this

Hello, the setup you mentioned should work, namely nodes n2 and n3 should be able to learn each other addresses by having n1 as a common peer. There is no need to setup a seed node in this case, as (regular) nodes should periodically exchanging know peer addresses using the PEX protocol. At the same time, this exchange is only performed when needed, so it is not automatic that n2 and n3 will know each other's address in a short time interval.
For a better diagnosis it would be good to have access to the configuration files adopted by the nodes.

Can someone comment on this

Please note that we don't provide real-time support for issues. Also, as @cason says, we at the very least need:

The configuration files for your nodes
Log outputs showing where the problem occurs

@thanethomson I didnt ask any realtime support, the issue I was facing was a basic one and I guess its your responsibility to guide us and take your product to next level. Sorry, if it sounded rude but you just think it is a common feature which is not working, I also have experience with other distributed systems and didnt face this problem till now.

@cason Should I provide config.toml for p2p configuration or what configuration files do you exactly mean?

In cosmos don't we have any auto nodes discovery feature?

CometBFT nodes are able to learn the addresses of other nodes running in the same network. So, like I mentioned, nodes n2 and n3 in your setup should eventually know the address of each other. This process is not immediate, though.

What will be the solution for this problem?

I don't see a problem in your setup. From your description, the 3-node network has an initial topology n2 -> n1 <- n3. Then n1 is killed. In this particular case, there are no guarantees that n2 and n3 will find a way to connect to each other. Notice that the voting power of the validators does not matter here.

Is there any suggestive node topology for the system?

What are the system requirements? For a small network, manually configuring the addresses of all other nodes at each node would be the recommended setup. This would lead to a fully-connected network topology. While this is not the recommended setup to in production networks, this is the most obvious setup for a 3-node network.

Optimistically closing this as "can't repro".
@aramakam3505 please reopen and we'd be glad to continue the conversation here!


NL core Administration Agreement does not yet refer to nl-core-Patient.

NL core Administration Agreement does not yet refer to nl-core-Patient.
https://github.com/Nictiz/Nictiz-R4-zib2020/blob/MedicationProfiles/resources/nl-core/nl-core-AdministrationAgreement.xml

Duplicate(?, actually 'specialization') of #183

Fixed.