XPath filtering issue using bit-is-set() function

Hi!
I've encountered an issue, when using the bit-is-set() XPath function. I know this is probably a libyang issue,
but I used netopeer2-server to reproduce this issue.
My test requires the 2 YANG schemas I've attached, and the following NETCONF commands:
First we put the 2 possible results of bit-is-set() into a leaf-list:
<rpc xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" xmlns:t="urn:com:test"
     message-id="1">
  <edit-config>
    <target>
      <running/>
    </target>
    <config>
      <basics xmlns="urn:com:test">
        <sentence>true</sentence>
        <sentence>false</sentence>
      </basics>
    </config>
  </edit-config>
</rpc>

We set one of the bits:
<rpc xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" xmlns:t2="urn:com:test2"
     message-id="1">
  <edit-config>
    <target>
      <running/>
    </target>
    <config>
      <basics xmlns="urn:com:test">
        <subbasics>
          <bitek xmlns="urn:com:test2">a</bitek>
        </subbasics>
      </basics>
    </config>
  </edit-config>
</rpc>

Then we return one of the elements of the leaf-list (true or false), depending on if the
respective bit (in this case 'a') was set:
<rpc xmlns="urn:ietf:params:xml:ns:netconf:base:1.0" message-id="">
  <get>
    <filter type="xpath" select="/t:basics/sentence[.=string(bit-is-
set(./../t:subbasics/t2:bitek,'a'))]" xmlns:t="urn:com:test" xmlns:t2="urn:com:test2"/>
  </get>
</rpc>

And last <get> RPC always returns 'true'. Shouldn't this return false? Am I doing something
wrong?
I'm using the latest devel of libyang/sysrepo/libnetconf2/netopeer2.
Thanks!
test.tar.gz

It seems the function was comparing on the defined bits in the type instead of those set in the value for some reason. Should work now.

Works good now, thanks!


fails to install - build errors on jupyter lab build on windows 10

Get this error.

I have the same issue.

I also have the same issue.

Same error on windows 11

This is a issue related to windows paths and has been discussed in other issues.


Z-index issue (comment shows above header)

Picture worth a thousand words
https://www.researchhub.com/post/2170/anonymity-in-peer-review

hey @yattias i wanna work on it can you assign it to me ?

@SandipGyawali feel free to work on and open a PR. We usually don't assign Open Source issues

Moved to https://github.com/ResearchHub/issues/issues/3

@SandipGyawali Have you taken a look?


[GITHUB CONFIG] - Setting up projects

Description :

Set up GitHub projects to improve this project visibility

Goal :

Have at least one project to work similar to a sprint

Can use the GitHub documentation as guide


Async loading of quick starts

QuickStartDrawer/catalog should be able to render without quick starts, and once the quick starts have been passed in should let them re-render.

Can now be achieved through the QuickStartContainer component. See the example


Husky + lint-staged for pre-commit checks

Description
Added husky + lint-staged configs for pre-commit checks
Fixes #52
Changes Include

[x] Refactor ⚒ (code change that neither fixes a bug nor adds a feature)
[x] Tooling ⚙ (Changes to the build process or auxiliary tools and libraries such as documentation generation)
[x] Performance ⚡ (code change that improves performance)- [ ] Bug fix (non-breaking change which fixes an issue)

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules
[x] I have checked my code and corrected any misspellings

I've tried to create a fresh pull request as I have made many mistakes in previous one.

LGTM, just include the .husky folder. please test it on your local make a dummy commit and hard reset it after checking if it works.

LGTM, just include the .husky folder. please test it on your local make a dummy commit and hard reset it after checking if it works.

I have made the additions you requested and it is working perfectly fine.
This is what I get when I commit.

don't push the commits just make it on local

don't push the commits just make it on local

Okay fine ! My bad

seems like husky is running let others have a look.

Changes done ! Please do have a look

@all-contributors please add @tarunkishore2303 for code


Support robots.txt

Support robots.txt to get the sitemap url and restrictions.

https://ticket.cdmon.com/es/support/solutions/articles/7000006245-cómo-configurar-el-fichero-robots-txt


python311Packages.fontbakery: 0.11.2 -> 0.12.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.fontbakery is: Tool for checking the quality of font projects
meta.homepage for python311Packages.fontbakery is: https://github.com/googlefonts/fontbakery
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.12.0 with grep in /nix/store/r9nv2hlry3vck050a0ypycvc432ri37r-python3.11-fontbakery-0.12.0
found 0.12.0 in filename of file in /nix/store/r9nv2hlry3vck050a0ypycvc432ri37r-python3.11-fontbakery-0.12.0

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
fontbakery
python311Packages.fontbakery
python312Packages.fontbakery

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/r9nv2hlry3vck050a0ypycvc432ri37r-python3.11-fontbakery-0.12.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.fontbakery https://github.com/r-ryantm/nixpkgs/archive/ca257c817720e1de9db15bc4b34d8336281e6068.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ca257c817720e1de9db15bc4b34d8336281e6068#python311Packages.fontbakery

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/r9nv2hlry3vck050a0ypycvc432ri37r-python3.11-fontbakery-0.12.0
ls -la /nix/store/r9nv2hlry3vck050a0ypycvc432ri37r-python3.11-fontbakery-0.12.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review --extra-nixpkgs-config '{ allowInsecurePredicate = x: true; }' run on x86_64-linux 1

2 packages failed to build:

python312Packages.fontbakery
python312Packages.fontbakery.dist

2 packages built:

fontbakery (python311Packages.fontbakery)
fontbakery.dist (python311Packages.fontbakery.dist)

Maintainer pings
cc @danc86 for testing.

Looks like the update itself is okay, all the package unit tests are passing. passthru.tests is failing because of some changes in the command-line interface.
The check-ufo-sources subcommand got renamed to check-ufo. And the --list-subcommands option no longer exists (not sure if this change was intentional upstream, or a bug). We can fix this in the test script:
diff --git a/pkgs/development/python-modules/fontbakery/tests.nix b/pkgs/development/python-modules/fontbakery/tests.nix
index 4725b1fee3e3..d968d6ec056d 100644
--- a/pkgs/development/python-modules/fontbakery/tests.nix
+++ b/pkgs/development/python-modules/fontbakery/tests.nix
@@ -10,13 +10,10 @@ runCommand "${pname}-tests" { meta.timeout = 5; } ''
   # Check the version matches what we packaged.
   ${fontbakery}/bin/fontbakery --version | grep -q "${version}"

-  # Can it list its own subcommands?
-  ${fontbakery}/bin/fontbakery --list-subcommands >>$out
-
   # Unpack src to get some test fonts.
   tar -xzf ${src} --strip-components=1 ${pname}-${version}/data/test

   # Run some font checks.
-  ${fontbakery}/bin/fontbakery check-ufo-sources --no-progress --no-colors data/test/test.ufo >>$out
+  ${fontbakery}/bin/fontbakery check-ufo --no-progress --no-colors data/test/test.ufo >>$out
   # TODO add more
 ''

This got rolled into PR #305152 so it's on staging already. Will send a new PR to fix the passthru.tests and other broken deps.

Fixes are in PR #307382.


Revisit the callback APIs in OAuth2/OTP/Webauthn, they are quite error prone and easily forgotten.

Web security handlers require the use of HTTP callbacks, the setup is not complex but it has a big developer experience problem.

When a developer forgets a callback the application does start as expected but doesn't work as expected
The handlers require an origin or url in the setup but this is not used until a callback it setup
the verification of the callback path and the origin isn't trivial due to sub routers

For this reason these kind of handlers should incorporate the callback code in the main handler to ensure that applications are not unsecure by a forgotten config.

To illustrate the problem, consider the following application:
    OAuth2Auth oauth2 = OAuth2Auth.create(vertx, new OAuth2Options()
      .setClientId("client-id")
      .setFlow(OAuth2FlowType.AUTH_CODE)
      .setClientSecret("client-secret")
      .setSite("http://localhost:10000"));

    OAuth2AuthHandler oauth2Handler = OAuth2AuthHandler.create(vertx, oauth2, "http://localhost:8080/callback");

    // setup the callback handler for receiving the callback
    oauth2Handler.setupCallback(router.route("/callback"));

    // protect everything under /protected
    router.route("/protected/*").handler(oauth2Handler);

From a developer point of view, there are 2 major concerns here. The creation of the handler requires a URL origin to inform about the callback location: http://localhost:8080/callback. This URL must refer to a route in this application, but in fact it is currently only used to validate the origin of the incoming requests.
Then, there is a setupCallback call that takes a route. The path should match the previously configured URL, but nothing can really be stated as the route can be mounted on a sub router, and we can only assume that the path matches the end of the configured URL.
This issue isn't problematic from a OWASP Top 10 perspective because if the configuration is incorrect, the application will still be secure and deny all incoming requests.
The second issue is the call setupCallback() itself. Given that the handler is created with a builder pattern, a developer may have forgotten to write it and to the human eye, the application looks correct. Given that the callback isn't required all the time, no errors are raised during compile or startup.
The change we should implement is to deprecate for removal all the methods:

OAuth2AuthHandler#setupCallback(Route)
OtpAuthHandler#setupRegisterCallback(Route)
OtpAuthHandler#setupCallback(Route)
WebAuthnHandler#setupCredentialsCreateCallback(Route)
WebAuthnHandler#setupCredentialsGetCallback(Route)
WebAuthnHandler#setupCallback(Route)

Instead, these handlers should have an optional setter for the callback paths. If the paths are defined and not null, then the main handler should process them as if they were on a different route before continuing the router chain.
Example user code:
    OAuth2Auth oauth2 = OAuth2Auth.create(vertx, new OAuth2Options()
      .setClientId("client-id")
      .setFlow(OAuth2FlowType.AUTH_CODE)
      .setClientSecret("client-secret")
      .setSite("http://localhost:10000"));

    OAuth2AuthHandler oauth2Handler = OAuth2AuthHandler
      // instead of a url, an origin should be supplied, as this origin is what is required to perform the right validation
      .create(vertx, oauth2, "http://localhost:8080")
      // we want a callback active, so we specify the path
     .callbackPath("/callback");

    // protect everything under /protected
    router.route("/protected/*").handler(oauth2Handler);

This will also address a problem with routes shading each other. For example:
    router.route("/protected/*")
      .handler(
        OAuth2AuthHandler
          .create(vertx, oauth2, "http://localhost:8080/callback")
          .setupCallback(router.route("/callback"));

This has a problem that due to the order this code is executed, the oauth2 handler is added to the router, and after it creates the callback route which is added after to the router. In order words, the callback is now behind the oauth2 handler which means that to reach it, the callback itself needs to be secure, which is not possible.

Another idea is to create explicit methods in the route, for example:
router.route("/secure/*)
  .authenticationHandler(Oauthe2AuthHandler.create(...))
  .authorizationHandler(AuthorizationHandler.create(...))
  .handler(ctx -> { ... });

Having such granularity allows the route object to properly order the handlers:

authn and if there are any required callbacks, register before itself in the router
authz
user handler(s)

Configuration can be guaranteed to be correct vs:
router.route("/secure/*)
  .authorizationHandler(AuthorizationHandler.create(...))
  .authenticationHandler(Oauthe2AuthHandler.create(...))
  .handler(ctx -> { ... });

Which silently will fail as there's no user yet as authz is tested before authn.
Given that we have 2 distinct interfaces, we could keep the naming short:
router.route("/secure/*)
  .auth(Oauthe2AuthHandler.create(...))
  .auth(AuthorizationHandler.create(...))
  .handler(ctx -> { ... });

As the argument type is distinct, we can still clearly identify the kind and order it correctly in the route.
The only issue is that the ordering is still public, and users that call the order method may put the route still in a bad state.

I've been playing around with the current API and noticed that if we have a marker interface:
interface OrderListener {
  void onOrder(int order);
}

and add those to all callback related handlers we can have a callback action when the handler is added to the router. At this moment, we can mount the callbacks with the same order as the main handler, which will enforce that it will be added right before the main handler itself.
This will ensure that we don't need extra warnings or checks in the handler itself as the callbacks are always mounted in the right order.
What needs to be checked is what happens if the main handler order changes, propagation may or may not work as the order is locked once is added to the router.


fix(dashboard): Charts crashing when cross filter on adhoc column is applied

SUMMARY
When there are charts using different datasets in a dashboard and a cross filter is applied on an adhoc column, the charts that use different dataset than the origin chart are crashing with a db error.
This PR fixes that by excluding filters based on an incompatible adhoc column from the query.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TESTING INSTRUCTIONS

Create a chart that is grouped by an adhoc column (with sql expression) and add it to a dashboard
Add another chart to a dashboard, which should use a different dataset than the first chart
Apply a cross filter by clicking on a series related to the adhoc column in the first chart
Verify that the second chart doesn't crash and is not filtered by the cross filter

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Can we write a test to cover this bug, too?

Test added

Thanks @kgabryje! Looks great


Show Only One Language

As it shows many options for languages in the login page how to remove that option from choosing different languages.

You can remove it in the web interface code.


Install from aur

Added miv here: https://aur.archlinux.org/packages/miv-git/

I merged your pull request. However, I cannot maintain that cite.


[flutter_tools] remove zone level overrides of verbose and daemon logging

Description
Make it possible for all FlutterCommands to be global free, by moving instantiation to inside the Zone context. Additionally, provide VerboseLogger and NotifyLogger (daemon) at the top level and remove from command-specific overrides.
This allows removing a work around where web devices needed to look up directly from the context in non-test code.
Technically the output preferences are still zone injected, but these will be moved soon as they were not being used correctly by the top level command (the injection comes after ArgParser reads the overflow values, causing numerous wrap issues)

Need to check for google3 breakages

This is a breaking change to some google3 APIs, but It gives us a path forward for removing globals on commands.

My change to the daemon test made it flaky

I'll wait to land this until I can put together a g3 patch


Adding track method with argument parameter.

This is the #68 with added unit test.

@richardortiz84 We discussed this PR further in the team, and we are hesitant to expose the args parameter completely, as this could lead to confusion among other customers.
I believe what you were looking for could be achieved by using https://github.com/Leanplum/Leanplum-Android-SDK/blob/master/AndroidSDK/src/com/leanplum/Leanplum.java#L1528 ?
I don't think currencyCode is available in iOS but we can add a similar method for iOS....

@benmarten Calling another event does not achieve what we are trying to do. We have another event that is being used for track purchases where we send in a currency. The track function not having a currency code is a problem because our application takes in currencies other than USD and we need those converted so that we can properly target our audience.

Hi @richardortiz84. I believe what you are looking for is a track method that takes currency code. This is a valid case for non in-app purchase apps. I need to confirm with the team but we will most likely make a new method that will cover this case. If so we will make it. No need to make a PR.

I'm not sure if we made any progress on this request. Let's discuss in todays sprint planning...

We have added a task in our current sprint, to add a new method to support this use case.

Resolved by https://github.com/Leanplum/Leanplum-Android-SDK/pull/77 @richardortiz84


WIP- Handbook bolded titles

Migrated from GitLab: https://gitlab.com/meltano/handbook/-/merge_requests/172
Originally created by @emilylucie on 2022-05-25 15:26:49

updated handbook to include new style guide....

@emilylucie I'm going to merge this since it's a very small change and still seems relevant!


For now, apply the same color scheme as used on edgeryders.eu

Don't put in much effort for this though, as we'll have to change it again.
Our Discourse color scheme is here.
The header background color and page background color should still use the dark gray like now. But other elements should be switched to Discourse-aligned colors, including:

The background color of menu navigation links should be #e45735 on-click and when being the currently active menu item (the latter only if supported by the Middleman framework, we don't want to make it complicated …).

This is actually nonsense. We better already apply something gleaned from the current official NGI platform, with a few modifications. Will make a separate issue for that.


Full screen fix for macOS and iPadOS

HI,
I've noticed that full screen mode wasn't supported on macOS and iPadOS so I made this fix.

Hi @kuglee,
Thanks for your PR 👍
Adding an option to enable isElementFullscreenEnabled is a great addition.
To give developers more control on how the fullscreen player is presented I think it would be a good idea to add a dedicated FullscreenMode option to the YouTubePlayer.Configuration to decide whether the system player (AVPlayerViewController) or the YouTube web fullscreen player should be used.
I will continue the work on the fullscreen-mode feature branch and will let you know when the changes are available with a new release of the YouTubePlayerKit.

Thanks!

@kuglee the changes are now available with the latest release 1.4.0


Android studio Sync Error : no such target '//:-nolaunch_app'

Hello! I'm downeydowney!
i'm on learning mediapipe.
So.. some errors are presented that i couldn't catch it.
please give me advise!
it is error in bazel console on android studio.
i'm on building my graphs which i made
i don't have idea how to fix it.  Thank you for your attention.
Error:Blaze build failed. See Blaze Console for details.
ERROR: Skipping '-nolaunch_app': no such target '//:-nolaunch_app': target '-nolaunch_app' not declared in package '' defined by /data/src/mediapipe/BUILD.bazel
WARNING: Target pattern parsing failed.
ERROR: no such target '//:-nolaunch_app': target '-nolaunch_app' not declared in package '' defined by /data/src/mediapipe/BUILD.bazel
INFO: Elapsed time: 0.084s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (0 packages loaded)
INFO: Build Event Protocol files produced successfully.

Could you try building the helloworld app in your terminal? See https://google.github.io/mediapipe/getting_started/building_examples.html#option-1-build-with-bazel-in-command-line for how to do this. Thanks.

Can you provide more information such as your Bazel version and your OS info?
Could you try building the helloworld app in your terminal? See https://google.github.io/mediapipe/getting_started/building_examples.html#option-1-build-with-bazel-in-command-line for how to do this. Thanks.

thank you for rapid replying!!
i already do that helloworld! and basic solutions!
hand  face pose and so on.
i’m on trying to make new graph using basic solutions!
my os is Linux(ubuntu 18.04)
and bazel version 3.6.?
i have never used android studio.
so.. i lost my way haha
if you need more information, i will reply as soon as possible! thank you
my country time is dawn!
maybe i will reply  6 hour later.. thank you!

It might be a MediaPipe/Android Studio integration issue. My gut feeling is that's a new "feature" of Bazel 3.6.0. You probably can downgrade to use Bazel 3.0.0 to see if it happens again. In the meantime, I can reproduce this issue on my machine.
In fact, we now prefer users to use MediaPipe in Android Studio as an AAR: https://github.com/google/mediapipe/blob/master/docs/getting_started/android_archive_library.md.

I have the same problem with Android Studio, Ubuntu 20.04 and Bazel 3.0.0

Same problem!!

It might be a MediaPipe/Android Studio integration issue. My gut feeling is that's a new "feature" of Bazel 3.6.0. You probably can downgrade to use Bazel 3.0.0 to see if it happens again. In the meantime, I can reproduce this issue on my machine.
In fact, we now prefer users to use MediaPipe in Android Studio as an AAR: https://github.com/google/mediapipe/blob/master/docs/getting_started/android_archive_library.md.

Thank you i will try! downgrade of bazel! Thanks!
if it's getting better, i'll share my result!

This might be caused by https://github.com/bazelbuild/intellij/issues/2084

it happended after bazel sync. if not sync it will not be error.

Same with Linux Android Studio Bazel 3.7

bazel 3.7.2   macOS catalina 10.15.5
I have also encountered this problem
Here is my log
Waiting for target device...
Invoking mobile-install...
Command: /Users/mac/.bazel/bin/bazel mobile-install --tool_tag=ijwb:AndroidStudio --device c016900d:tcp:5037 "--adb_arg=-s " --adb_arg=c016900d --adb /Users/mac/Library/Android/sdk/platform-tools/adb --curses=no --color=yes --progress_in_terminal_title=no -c opt --config=android_arm64 --build_event_binary_file=/var/folders/81/zyx3b9z17jb880v9_w95tr140000gn/T/intellij-bep-99f14cf7-2921-4263-b7f9-a476a4f40a92 --nobuild_event_binary_file_path_conversion -- //mediapipe/examples/android/src/java/com/google/mediapipe/apps/handtrackinggpu:handtrackinggpu --nolaunch_app
Loading:
Loading: 0 packages loaded
ERROR: Skipping '-nolaunch_app': no such target '//:-nolaunch_app': target '-nolaunch_app' not declared in package '' defined by /Users/mac/AndroidStudioProjects/mediapipe/BUILD.bazel
WARNING: Target pattern parsing failed.
ERROR: no such target '//:-nolaunch_app': target '-nolaunch_app' not declared in package '' defined by /Users/mac/AndroidStudioProjects/mediapipe/BUILD.bazel
INFO: Elapsed time: 0.308s
INFO: 0 processes.

FAILED: Build did NOT complete successfully (0 packages loaded)
@jiuqiant jiu

bazel 3.7.2 macOS catalina 10.15.5
我也遇到了这个问题，
这是我的日志
等待目标设备...正在
调用移动安装...
命令：/Users/mac/.bazel/bin/bazel mobile-install --tool_tag = ijwb：AndroidStudio --device c016900d：tcp：5037“ --adb_arg = -s“ --adb_arg = c016900d --adb / Users / mac / Library / Android / sdk / platform-tools / adb --curses = no --color = yes --progress_in_terminal_title = no -c opt --config = android_arm64 --build_event_binary_file = / var / folders / 81 / zyx3b9z17jb880v9_w95tr140000gn / T / intellij-bep-99f14cf7-2921-4263-b7f9-a476a4f40a92 --nobuild_event_binary_file_pathrc_conversion-// mediapipe / examples / android / com / / apps / handtrackinggpu：handtrackinggpu --nolaunch_app
正在加载：
正在加载：已加载0个程序包
错误：正在跳过“ -nolaunch_app”：没有在/ Users / mac / AndroidStudioProjects / mediapipe / BUILD定义的“程序包”中声明的目标“ //：-nolaunch_app”：未声明目标“ -nolaunch_app”。 bazel
警告：目标模式分析失败。
错误：没有这样的目标'//：-nolaunch_app'：未在/ Users / mac / AndroidStudioProjects / mediapipe / BUILD定义的程序包``中声明目标'-nolaunch_app'.bazel
INFO：经过的时间：0.308s
INFO：0个进程。
失败：构建未成功完成（已加载0个软件包）
失败：构建未成功完成（已加载0个软件包）
信息：构建成功生成了事件协议文件。
失败：
@久久九

I downgraded bazel to 3.4.0 , and this problem still occurs after clicking run
Waiting for target device...
Invoking mobile-install...
Command: /Users/mac/bin/bazel mobile-install --tool_tag=ijwb:AndroidStudio --device c016900d:tcp:5037 "--adb_arg=-s " --adb_arg=c016900d --adb /Users/mac/Library/Android/sdk/platform-tools/adb --curses=no --color=yes --progress_in_terminal_title=no -c opt --config=android_arm64 --build_event_binary_file=/var/folders/81/zyx3b9z17jb880v9_w95tr140000gn/T/intellij-bep-7db9a6e8-27be-4ae7-920b-1d8032226c4f --nobuild_event_binary_file_path_conversion -- //mediapipe/examples/android/src/java/com/google/mediapipe/apps/handtrackinggpu:handtrackinggpu --nolaunch_app

Loading:
Loading: 0 packages loaded
ERROR: Skipping '-nolaunch_app': no such target '//:-nolaunch_app': target '-nolaunch_app' not declared in package '' defined by /Users/mac/AndroidStudioProjects/mediapipe/BUILD.bazel
WARNING: Target pattern parsing failed.
ERROR: no such target '//:-nolaunch_app': target '-nolaunch_app' not declared in package '' defined by /Users/mac/AndroidStudioProjects/mediapipe/BUILD.bazel
INFO: Elapsed time: 0.099s
INFO: 0 processes.

FAILED: Build did NOT complete successfully (0 packages loaded)

Meanwhile, Build with Bazel in Command Line can be used for deploying the apps.

Hi @downeydowney, I would request you to check the above with the latest release and let us know if you still see the same.

I experience the same issue. Most likely it's because of extra space added by I would guess AS bazel plugin.
Check this line in output:
Command: /Users/mac/bin/bazel mobile-install --tool_tag=ijwb:AndroidStudio --device c016900d:tcp:5037 "--adb_arg=-s " --adb_arg=c016900d --adb /Users/mac/Library/Android/sdk/platform-tools/adb --curses=no --color=yes --progress_in_terminal_title=no -c opt --config=android_arm64 --build_event_binary_file=/var/folders/81/zyx3b9z17jb880v9_w95tr140000gn/T/intellij-bep-7db9a6e8-27be-4ae7-920b-1d8032226c4f --nobuild_event_binary_file_path_conversion -- //mediapipe/examples/android/src/java/com/google/mediapipe/apps/handtrackinggpu:handtrackinggpu --nolaunch_app
Note that there is a space between -- and //mediapipe/examples/android/src/java/com/google/mediapipe/apps/handtrackinggpu:handtrackinggpu
and probably that is why --nolaunch_app is treated as a target.
Any ideas? Seems like a bug in bazel plugin or so


Uprev to 0.26.0

Codecov Report

Merging #559 (425d07a) into main (ae4cb28) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #559   +/-   ##
=======================================
  Coverage   94.15%   94.15%
=======================================
  Files          95       95
  Lines       12803    12803
  Branches       24       24
=======================================
  Hits        12055    12055
  Misses        742      742
  Partials        6        6

Continue to review full report in Codecov by Sentry.

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


GestureDetector ignores "behavior" property

Tested in dartpad: Flutter 2.5.2 Dart SDK 2.14.3, and on Android emulator:
$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.5.2, on Microsoft Windows [Version 10.0.19042.1237], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[√] Chrome - develop for the web
[√] Visual Studio - develop for Windows (Visual Studio Community 2019 16.8.3)
[√] Android Studio (version 4.0)
[√] IntelliJ IDEA Ultimate Edition (version 2021.2)
[√] VS Code (version 1.60.1)
[√] Connected device (4 available)

Reproduce by running following sample. Expected: Tapping anywhere prints "detector". Actual behavior: Tapping anywhere but the button does nothing. Tapping the button prints "Button".
Change both "opaque" behaviors to "translucent". Expected: Tapping the button prints "detector", "inner detector", and "button". Actual behavior: only "Button" is printed.

import 'package:flutter/material.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark().copyWith(
        scaffoldBackgroundColor: darkBlue,
      ),
      debugShowCheckedModeBanner: false,
      home: GestureDetector(
        behavior: HitTestBehavior.opaque,
        onTap: () => print('detector'),
        child: Scaffold(
          body: GestureDetector(
            behavior: HitTestBehavior.opaque,
            onTap: () => print('inner detector'),
            child: SizedBox.expand(
              child: Center(
                child: MyWidget(),
              ),
            ),
          ),
        ),
      ),
    );
  }
}

class MyWidget extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return ElevatedButton(
      onPressed: () => print('Button'),
      child: Text(
        'Hello, World!',
        style: Theme.of(context).textTheme.headline4,
      ),
    );
  }
}

@Lootwig, the documentation isn't quite clear on what they are supposed to do, as I believe that to be the source of confusion in the expected behaviour. There is a proposal to make the documentation clear on what each behaviour is supposed to do, and I believe that issue will be the best place to follow up for more information.
This is the issue in question. Kindly follow up there for more information.
https://github.com/flutter/flutter/issues/74733
As such I will have to close this as a duplicate.
Thank you


cleaned and updated the home page

Signed-off-by: Yash Vardhan<EMAIL_ADDRESS>This PR addresses the issue of cleaning up the home page and updating changes that caused it to look messy.
This PR fixes #53
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

LGTM


Master issue for TAUTO-3

Automatic testing of CIRCLean functionalities with a malicious sample Dataset

Tested on RPi2 Model B v1.1 as well, the 5/27/15 CIRCLean img does NOT working.   Get a 'alsactrl store: save state 1608 error' and then all services start shutting down and app dies.  Any help?

Could you give a try with this image:
https://www.circl.lu/assets/CIRCLean/2015-06-02_CIRCLean.img.gz
sha256: c8e58491b93b5f0aca40e5ece775f6b63693ea87b2b6ed29eb6200e382e93564
It is a test image we never released officially, but working for sure on rpi2. The alsa error is very weird and I did not have that issue recently... You are using a stock unmodified rpi, right?

I reformatted an extra SD card, laid the image down using “dd linux” cmd with the image provided 2015-06-02.
Im getting the same “alsa error” then a msg that all services will shutdown and Pi unit shuts OFF(lights stop blinking).
If I swap in the original SD card with Debian image, I get a login cmd prompt to the Raspberry as usual and unit works fine.
Any recommendations on capturing a log to either forward to you all OR just review.
Regards…
JMC555
From: Raphaël Vinot<EMAIL_ADDRESS>Sent: Friday, July 03, 2015 4:06 AM
To: CIRCL/Circlean
Cc: jmc555
Subject: Re: [Circlean] Master issue for TAUTO-3 (#23)
Could you give a try with this image:
https://www.circl.lu/assets/CIRCLean/2015-06-02_CIRCLean.img.gz
sha256: c8e58491b93b5f0aca40e5ece775f6b63693ea87b2b6ed29eb6200e382e93564
It is a test image we never released officially, but working for sure on rpi2. The alsa error is very weird and I did not have that issue recently... You are using a stock unmodified rpi, right?
—
Reply to this email directly or view it on GitHub https://github.com/CIRCL/Circlean/issues/23#issuecomment-118282104 .  https://github.com/notifications/beacon/AMjO5yiZgkNOkvBkU35W_p0uKZsjQ_4eks5oZkfbgaJpZM4E-Lzm.gif

https://github.com/CIRCL/PyCIRCLean/blob/master/.travis.yml - https://github.com/ytisf/theZoo

Currently we run the PyCIRCLean code against theZoo during the Travis build, but we don't have a way to verify that the files are being identified as malicious. Ideally we'll soon have a way to identify a list of files + how they should be identified and then write tests that confirm the desired behavior.


Azurestack-Cleaning up 2015-06-15 version of swagger for description addition and global parameters

description changes in 2015-06-15 swagger comparing to 2016-12-01 version of swagger, ResourceGroup is a global parameter, response descriptions addition
This checklist is used to make sure that common issues in a pull request are addressed. This will expedite the process of getting your pull request merged and avoid extra work on your part to fix issues discovered during the review process.
PR information

[x] The title of the PR is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For information on cleaning up the commits in your pull request, see this page.
[ ] Except for special cases involving multiple contributors, the PR is started from a fork of the main repository, not a branch.
[ ] If applicable, the PR references the bug/issue that it fixes.
[ ] Swagger files are correctly named (e.g. the api-version in the path should match the api-version in the spec).

Quality of Swagger

[x] I have read the contribution guidelines.
[ ] My spec meets the review criteria:

[ ] The spec conforms to the Swagger 2.0 specification.
[ ] The spec follows the guidelines described in the Swagger checklist document.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.

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

@vishrutshah Thanks I will work on the comments

@vishrutshah I will work on the comments on Thursday , apologies for the delay

@vishrutshah addressed your comments and explained the reason for not fixing few comments. Thanks!

@veronicagg addressed your comments

No modification for Ruby

No modification for NodeJS

 https://github.com/Azure/azure-sdk-for-python/pull/1152


Enhance functionality of the ls command

In #326 @saschalalala mentioned that the ls command does not behave as a normal, unix-style ls. This issue should be used to discuss enhancements to ls.

Well, in my opinion, restic ls should only display the path that one specifies, that means restic -r backup ls <snapshot-id> should display the root directory of the backup. In order to list files of directorys one had to specify this directory, e.g. restic -r backup ls <snapshot-id> path/to/directory. It should, imho, show them in a listed order, similar to the last column of ls -l in Linux and also show hidden files. Just for convenience.

Ok, I think we agree that this is a good idea.

I have two thoughts about this:

If restic -r backup ls <snapshot-id> only shows the top level, I think a recursive command should be added, like in GNU ls.
I think restic -r backup ls should be an alias for restic -r backup ls latest. This would in my opinion be more aligned with how the ls command works.

Good points. If you'd like to work on this and need help, please let me know. Otherwise start hacking, open a PR and then we can discuss your code!

Yes, I'll have a look at it.

Keep in mind this breaks backwards compatibility and also restic ls allows listing multiple snapshots.
While I have no problem with making latest the default snapshot, I think we should be consequent and do the same for all other restic cmds. In fact, if we're going to drop backwards compatibility, moving it from a positional argument to an (multi)flag would be nice as it frees up the args for cmd specific arguments.
I'm also not sure this recurse flag is a good thing. As fd0@ often quotes tar as his role model backup program, tar does not use a -R to recursively list files. Guess we need to make up our minds which gnutool to emulate ^^

Yes, I fully agree. My commits are currently not meant to be a solution for this issue, my goal is to explore what can be done. I think if the command's name is ls, then it should behave like I would expect ls to behave. I'm aware that this breaks backwards compatibility, but I think as long as version 1.0 is not released this is ok to do in a minor version. Maybe it would be best to line out how the cli should work first with some examples.

You raised some interesting points. From my point of view we can modify the ls command, I don't think many people use it.

what is the state here? i would like to help on that, if wanted.
i like the idea but imho would love to keep the old style (listing the whole tree) too, as i often use it with grep to check if a file is part of that snapshot

Ah, there has been some work on ls in the last couple of releases:

We limited it to one snapshot #1941
You can pass an optional path name, in which case restic will only list that path (#1941)
We added --recursive which will list the given path recursively (#1941)
By default, the complete snapshot is listed recursively if no path is given (for backwards compatibility)

Actually, I think all the things we discussed for enhancing ls have already been implemented. Is there anything else, or can we close this issue?

ah ok, nice. so imho there is nothing left here. i guess you could close it 🤔

Cool, I'm going to close this issue for now, please feel free to add further comments. Thanks!


fix(TS): Correct array return types for queryAll and getAll apis

What:
Updated the the return type type definition for the getAll* and queryAll* apis.

Why:
Currently the return type is [HTMLElement] which is an array of 1 element rather than an array of HTMLElements: HTMLElement[]

How:
By updating the .dts file

Checklist:

[ ] Documentation N/A
[ ] Tests N/A
[x] Ready to be merged
[X] Added myself to contributors table

:tada: This PR is included in version 2.6.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


x/tools/cmd/stringer: Gets confused with pointers to C.struct_xxx types.

Save this http://play.golang.org/p/7IHAOsDFqO in foo.go and say:
stringer -type MyT foo.go

Stringer will fail with:
stringer: checking package: foo.go:22:2: invalid operation: pfoo (variable of type *invalid type) has no field or method i

Stringer works ok with this, though: http://play.golang.org/p/uIrtOCI59N
Another similar program that triggers the same behavior (stringer failure) is this: http://play.golang.org/p/P2t3O0zONt

Your program uses cgo. I wasn't sure if stringer officially supported use of cgo or not.
I looked at the source code and I didn't see any "TODO" or similar mentions regarding importing package "C", so it seems it might be supported. The .go files that import "C" are indeed processed (e.g., see here), but that's the only special thing being done.
I guess my point is that it should first be confirmed that the tool is meant to support cgo at this time. If so, this is likely a problem with it (since I imagine it's a situation that's less common and therefore more poorly tested).

stringer doesn't support cgo.  It makes the simplifying assumptions that the initial package consists only of Go source files and that dependencies can be loaded from compiler export data (e.g. $GOROOT/pkg/$GOOS_$GOARCH/fmt.a).
Background:
This issue is more or less a duplicate of "stringer: can't find packages" (https://github.com/golang/go/issues/10249).  Josh closed it and filed the broader issue "go/loader: importers should just work" (https://github.com/golang/go/issues/10276).  I closed that after some changes to go/loader; in passing, I created a CL to make stringer use go/loader (https://go-review.googlesource.com/#/c/8561/) but it was rejected by Rob "until go/loader is tidied up".  go/loader could use more tidying up, though I won't be able to get to this for a while.  In the meantime, stringer still has this bug.


Mit rather than gpl?

Hi!
I just saw this is gpl. That’s fine for surge but I’m working in a set of mit plugins which have things we may want reused and uses this. Any thoughts about swapping this to mit so folks can use it in no gpl contexts
if not I’ll swap airwindows to gpl I guess but I always kinda think infrastructural things like this are better if less contagiously licensed to build community erc
but up to you entirely! I’m happy for my contributions to swap

Done, changed to MIT.


Admin PR - Adding static route entry to ceos startup config

Adding default route entry to the startup config needed in TE-3.2: Traffic Balancing According to Weights
Adding gribi service to the arista_ixia_lag.textproto kne file

@openconfig/featureprofiles-maintainers
@openconfig/featureprofiles-go-reviewers

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 72.664%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1462

Relevant Lines:
2012

💛  - Coveralls

/gcbrun


Make travis run the plugin tests

I detected that travis was not running the tests of the plugins. I've modified travis to run the tests here. Of course, we should wait until all tests are run to make sure that everything runs smoothly.

:scream: good thing you noticed!

Yeah - and I think something is going to fail, cause I'm having problems running the tests locally...

Closing in favor of https://github.com/biocore/qiita/pull/1703


Fixed proxied deactivated state

#86939j6pv

Task linked: CU-86939j6pv Android. Setup style for deactivated meta account


Outputs Mess into Inputs Lines When Cursor is in First Line of Inputs

$ echo III when run a very very long command which
wraps into multiple lines

And Enter Key hit when cursor at the III position, then the output when run a ... will show up the same line of wraps into ... line. The correct way should output at the line below the wraps into .. line.

Wow. This is such an obvious bug. I'm surprised it wasn't caught sooner.
Just pushed the fix. Thanks for the issue.


Automated Package Upgrade

:tada: This PR is included in version 7.7.390 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Generate sitemap at build time

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Problems with fileuploads on "large" harddrives

I can't upload files to my BBS, I get the error message:
"Tyvärr, gränsen för hur full disken får bli har överskridits!"
The disk I'm using is a 3GB disk and the partition NiKom is on is larger than 2GB.
I tried to set DISKFREE to 0 but it made no difference, and it doesn't matter if I make an area where I set the directory to my much smaller Workbench:-partition.
Here's the output from 'info'.
Mounted disks:
Unit	  Size	  Used	  Free Full Errs   Status   Name
PC0:      No disk present
RAM:     7690K    7690       0 100%   0  Read/Write Ram Disk
DH0:      102M   23912  185749  11%   0  Read/Write Workbench
DF0:      No disk present
DH1:    -1148634K   51745 6039596   1%   0  Read/Write Work
Volumes available:
Ram Disk [Mounted]
Work [Mounted]
Workbench [Mounted]

NikTransfer.c:615 - bytesfree=(id->id_NumBlocks - id->id_NumBlocksUsed)*id->id_BytesPerBlock;
This will overflow if you have more than 2GB free space. Between 2 and 4GB available space it will be negative and should always give this error unless DISKFREE is more negative.
A reasonable solution would be to calculating free blocks instead of Bytes and compare that with DISKFREE/BytesPerBlock. This would allow 1024GB partitions (with 512Bytes BytesPerBlock).

Interesting :-)
So, yes, the problem is exactly as @patrikaxelsson points out, the 32 bit variable bytesfree overflows and wraps into negative values.
It seems like the info command is having similar problems? Are there other places where you run into problems? I'm not up to date with AmigaDos issues in the 21st century..
Anyway the fix is easy. We should probably introduce that "0" means don't check at all as well.
As long as the number of bytes free on your partition is in the 2-4GB range you should be able to work around the problem by configuring DISKFREE to INT_MIN (-2147483648) (There is no check in the code that reads the config that will protest against this value.)
How urgent is a fix for you? Would you need a fix to 1.61 (1.62?) or can you wait for 2.0? Or would you prefer running a development version of 2.0 when the fix is in?
Also, out of curiosity.. How do you upload? Are you using an old school terminal program over a serial line? Or are you using a shell/terminal that supports ZModem?

And, I forgot, the reason why it doesn't help to create a file area on a different partition is because it's not the partition of the file area that is checked. It's the partition of the temp upload directory. (You could argue that it's a bit stupid that only this partition is checked, but that's how it currently works..)

Well there are many cases where having a "big" drive can cause problems in AmigaOS, but there are also workarounds for most of them (new scsi.device, newer versions of programs, newer filesystems...). I'm running plain AmigaOS 3.1 without many patches on this machine though.
A fix is not that urgent to me, files on the BBS doesn't exactly feel all that important nowadays :) and the workaround I'm using at the moment is that I filled the disk up with a big file to get less than 2GB free.
I found another issue with file-uploads though, but I will make another issue for that :/

Oh, I forgot to answer how I uploaded, I have tried both with DCTelnet on the Amiga and with SyncTerm on the PC, both have Zmodem support.

@punktniklas:
The places in the code referencing InfoData.id_BytesPerBlock should potentially have similar issues: https://github.com/punktniklas/NiKom/search?utf8=✓&q=id_BytesPerBlock


Wrong result when calling 'multi-line' lambda inside @tf.function

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux, Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.2.0-rc4-8-g2b96f3662b 2.2.0
Python version: 3.7.6
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
When decorating a function using tf.function that uses a lambda that extends to multiple lines,
only the first line is considered. When not decorating the function using tf.function the result is correct.
Describe the expected behavior
tf.function should not alter the function behaviour. In the example below, it prints 1 instead of 0. Removing the tf.function decorator gives the correct result. Moving the lambda into a single line gives again a correct result.
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
import tensorflow as tf

a = (
     lambda y: lambda x: x*y
     - y
    )(1)

@tf.function
def test_lambda():
    tf.print(a(1))

test_lambda()

Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.
The issue was noticed after applying black formatting changed the code output.

I have tried in colab with TF version 2.2 , nightly version and was able to reproduce the issue.Please, find the gist here.Thanks!

This looks like an underlying Python bug:
l = (
     lambda y: lambda x: x*y
     - y
    )

import inspect
print(inspect.getsource(l))

Output:
     lambda y: lambda x: x*y

It looks like the parser misses the second line.
To fix this, we'll need to work around this bug.

Is there any update maybe on this issue? It is concerning that code formatting / linting cannot be safely applied with tf.function

We have confirmed this is caused by a bug in Python, but I think we'll have a workaround in TF 2.4. In the mean time, these practices should help avoid running into this bug:

Avoid placing lambdas inside expressions that could break them across multiple lines, and instead define them on a separate line:

partial = lambda y: lambda x: x*y - y
l = partial(1)

Prefer using normal local functions when the body might be too long to fit on a single line:

def partial(y):
  return lambda x, y: x*y - y
l = partial(1)

Quick update: a more robust fix for this is now in and should be available in tf-nightly and TF 2.4 later this year. The new parsing method should robustly process all cases, and raises an error when it can't determine the proper function with certainty (which should only happen in rare cases).

Fantastic, thanks!
On Tue, Jul 7, 2020, 16:18 Dan Moldovan<EMAIL_ADDRESS>wrote:

Quick update: a more robust fix for this is now in and should be available
in tf-nightly and TF 2.4 later this year. The new parsing method should
robustly process all cases, and raises an error when it can't determine the
proper function with certainty (which should only happen in rare cases).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/39832#issuecomment-654895251,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADTBRJ2PGTOJYCP3B5FQ7MLR2MVBZANCNFSM4NI4CCVQ
.


rPackages, R: R 4.3.3 -> 4.4.0 and bump rPackages

Update R and rPackages.
Hydra: https://hydra.nixos.org/jobset/nixpkgs/r-updates
nixpkgs-review:
7 packages failed to build:
cantor jasp-desktop labplot python311Packages.rpy2 python311Packages.rpy2.dist sage sageWithDoc

41 packages built:
R R.tex diffoscope diffoscope.dist diffoscope.man jetbrains.dataspell ocamlPackages.ocaml-r postgresql12JitPackages.plr postgresql12Packages.plr postgresql13JitPackages.plr postgresql13Packages.plr postgresql14JitPackages.plr postgresql14Packages.plr postgresql15JitPackages.plr postgresql15Packages.plr postgresql16JitPackages.plr postgresql16Packages.plr python311Packages.cnvkit python311Packages.cnvkit.dist python311Packages.radian python311Packages.radian.dist python311Packages.rchitect python311Packages.rchitect.dist python312Packages.cnvkit python312Packages.cnvkit.dist python312Packages.radian python312Packages.radian.dist python312Packages.rchitect python312Packages.rchitect.dist python312Packages.rpy2 python312Packages.rpy2.dist quarto rWrapper radianWrapper rstudio rstudio-server rstudioServerWrapper rstudioWrapper spark spark_3_4 vscode-extensions.reditorsupport.r

jasp-desktop has warnings being elevated to errors, e.g.,:
jaspObject.cpp:508:25: error: format not a string literal and no format arguments [8;;https://gcc.gnu.org/onlinedocs/gcc/Warning-Options.html#index-Wformat-security-Werror=format-security8;;]
  508 |                 Rf_error(("nested key \"" + nestedKeyToString(nestedKey, "$") + "\"does not exist in the options!").c_str());

There doesn't seem to be a newer jasp to bump to, so we could disable format security for the moment. @TomaSajt
Sage is blocked by python3Packages.rpy2, which is failing using 3.11 during testing due to a segfault. This segfault does not happen with python 3.12. Any suggestions @joelmo? Sage builds fine when disabling tests so this is the only problem to resolve for sage.

For jasp-desktop I guess we could just do
env.NIX_CFLAGS_COMPILE = "-Wno-error=format-security";

I think it's fine to include it in this PR.

also, let’s not forget about this: https://github.com/NixOS/nixpkgs/issues/314080

@domenkozar can we enable force pushes to this r-updates branch?

done

The only issue remaining is sage being blocked by rpy2: there's no newer rpy2 to bump to, and sage seems to require python 3.11. Any suggestions? @timokau @collares @7c6f434c

I spent a while yesterday trying to figure out the crash, but didn't have enough time to reach a conclusion. I will try more later, but if I can't figure out something in the next few days I think we should just update to a Sage 10.4beta9 (soon to be RC) and migrate to Python 3.12.
Something goes wrong when R tries to run rpy2's _handler_wrap callback via cffi in tests/Interface/test_embedded.py:test_parse_error_when_evaluating (Python tries to access memory at 0x46 as if it were a frame), but I can't yet explain why Python 3.11 + R 4.4.0 crashes and other combinations don't.

Compiling rpy2 without optimizations, I got the following error:
Exception ignored from cffi callback <function _handler_wrap at 0x6594220>:
Traceback (most recent call last):
  File "/nix/store/pwi3r33c8xhic5011a5kcxp7q4h6ic6q-python3-3.11.9-env/lib/python3.11/site-packages/rpy2/rinterface_lib/_rinterface_capi.py", line 690, in _handler_wrap
AttributeError: 'set' object has no attribute 'rlib'

I can't yet see how openrlib can be a set at _rinterface_capi:690, but I will keep thinking.

I can't yet see how openrlib can be a set at _rinterface_capi:690, but I will keep thinking.

That doesn't seem possible, perhaps the migration to sage 10.4beta9 you suggested is the way to go. If the RC is due soon we probably don't want to spend too much effort on figuring out this python 3.11 bug.

I still need to double-check this, but I think the error goes away if both cffi and rpy2 are compiled with
NIX_COMPILE_CFLAGS = "-O0";
NIX_HARDENING_ENABLE = "";

which makes me think there's some undefined behavior we were very lucky not to hit until now. If you have evidence that this is a rpy2 bug, let me know and I will file an issue upstream.
A workaround which doesn't require changing cffi compile flags is setting the RPY2_CFFI_MODE="ABI" environment flag. I think it might be enough to do it at rpy2 build time, but some extra work is needed to make the tests find libR.so.
Either way, I'll soon investigate upgrading Sage to 10.4beta9, even if just to get a head start at upgrading to rc0.

The error goes away if both cffi and rpy2 are compiled with
NIX_COMPILE_CFLAGS = "-O0";
NIX_HARDENING_ENABLE = "";

which makes me think there's some undefined behavior we were very lucky not to hit until now.
I will try to gather more evidence that this is a rpy2 bug and file an issue upstream. For Sage purposes, I think the following should therefore be acceptable as a workaround while we wait for Sage 10.4 (assuming Sage tests pass, which I haven't checked):
diff --git a/pkgs/applications/science/math/sage/default.nix b/pkgs/applications/science/math/sage/default.nix
index 6624dcc36528..d03cec7e3ef1 100644
--- a/pkgs/applications/science/math/sage/default.nix
+++ b/pkgs/applications/science/math/sage/default.nix
@@ -29,6 +29,18 @@ let
       sage-setup = self.callPackage ./python-modules/sage-setup.nix {
         inherit sage-src;
       };
+
+      # there is some miscompilation happening with -O1, and the fortify hardening requires -O1.
+      # TODO: remove when sage migrates to Python 3.12, since affects 3.11 only for some reason.
+      cffi = super.cffi.overrideAttrs (old: {
+        env.NIX_CFLAGS_COMPILE = (old.env.NIX_CFLAGS_COMPILE or "") + " -O0";
+        hardeningDisable = [ "fortify" ];
+      });
+
+      rpy2 = super.rpy2.overrideAttrs (old: {
+        env.NIX_CFLAGS_COMPILE = (old.env.NIX_CFLAGS_COMPILE or "") + " -O0";
+        hardeningDisable = [ "fortify" ];
+      });
     });
   };

A workaround which doesn't require changing cffi compile flags is setting the RPY2_CFFI_MODE="ABI" environment flag, and I would prefer this latter option if you can get it to work. It might be enough to set the flag at rpy2 build time, but I haven't tested it because some extra work is needed to make the tests find libR.so.
Either way, I'll soon investigate upgrading Sage to 10.4beta9, even if just to get a head start at upgrading to rc0.

@jbedo If I understand correctly, the CI runs at https://github.com/rpy2/rpy2/pull/1051 segfault on the same test (test_embedded_r.py) running under Python 3.12, which is consistent with the undefined behavior explanation.

Filed rpy2/rpy2#1111 just in case.

That's great sluthing, and good to know it isn't resolved with 3.12. I'll spend some time trying to get the ABI mode to work.

Actually, I am not sure the ABI suggestion definitely helps. If you run the minimized tastcase under valgrind --trace-children=yes, both the ABI and the API mode report invalid reads when invoking callbacks. But it might be a good enough workaround while we wait for an upstream fix.

I'm unsure as to the best way forward: this bug really needs to be fixed upstream. Changing ABI/hardening/optimisation really does nothing, we just sidestep the bug during the test case with some luck. Is this any better than disabling tests?  rpy2 is also still broken with the old R 4.3.3 version as you've discovered, so there doesn't seem to be much point to holding the R version bump back waiting for a fix and we may as well pick one of these options and then update rpy2 when an upstream fix is available.

@ofborg build sageWithDoc

R 4.4.1 got released last week, what shall we do, merge this one and re-bump, or directly bump to 4.4.1?

R 4.4.1 got released last week, what shall we do, merge this one and re-bump, or directly bump to 4.4.1?

I'm planning to bump to 4.4.1, just preparing it and checking it. It should be a fairly minor change since we already got 4.4.0 in good shape.

49 packages built:
R R.tex cantor diffoscope diffoscope.dist diffoscope.man jasp-desktop jetbrains.dataspell labplot ocamlPackages.ocaml-r postgresql12JitPackages.plr postgresql12Packages.plr postgresql13JitPackages.plr postgresql13Packages.plr postgresql14JitPackages.plr postgresql14Packages.plr postgresql15JitPackages.plr postgresql15Packages.plr postgresql16JitPackages.plr postgresql16Packages.plr python311Packages.cnvkit python311Packages.cnvkit.dist python311Packages.radian python311Packages.radian.dist python311Packages.rchitect python311Packages.rchitect.dist python311Packages.rpy2 python311Packages.rpy2.dist python312Packages.cnvkit python312Packages.cnvkit.dist python312Packages.radian python312Packages.radian.dist python312Packages.rchitect python312Packages.rchitect.dist python312Packages.rpy2 python312Packages.rpy2.dist quarto rWrapper radianWrapper resorter rstudio rstudio-server rstudioServerWrapper rstudioWrapper sage sageWithDoc
spark spark_3_4 vscode-extensions.reditorsupport.r


BESU_CONFIG_FILE environment variable does not work

Description
Docs says that as a user I should be able to specify configuration file via environment variable BESU_CONFIG_FILE, but looks like I can't: https://github.com/hyperledger/besu/blob/master/besu/src/main/java/org/hyperledger/besu/cli/util/ConfigOptionSearchAndRunHandler.java#L64
I had PR to update docs, but it was closed.
https://github.com/hyperledger/besu-docs/pull/740
Acceptance Criteria

Either docs updated
Or Code updated

Steps to Reproduce (Bug)

Download any version of besu
Create basic config.toml file, let's say with rpc enabled on non default port (e.g. 8555 - make sure it's free)
Run BESU_CONFIG_FILE=/path/to/config.toml besu
Check the logs, it should not state anywhere that RPC server is running
Try BESU_CONFIG_FILE=/path/to/config.toml besu --rpc-http-enabled
Make sure that port didn't change from default 8545
Exit the running besu process
Try running besu --config-file=/path/to/config.toml
Make sure rpc server started on specified port (e.g. 8555)

Expected behavior: Config file can be specified from environment variable as docs says
Actual behavior: Can't specify path to config file via environment variable BESU_CONFIG_FILE as the docs says.
Frequency: 100% of time. It's misdocumented feature IMHO.
Versions (Add all that apply)

Software version: any
Java version: openjdk 11.0.2
OS Name & Version: Debian Linux 9
Kernel Version: 3.10.0
Virtual Machine software & version: -
Docker Version: 1.13.1
Cloud VM, type, size: -

Additional Information

Hey @soulman-is-good ! Thanks for reporting. This should be supported; we'll triage your issue. In the meantime, is using the command line flag a possible workaround? If not, what's your use-case that you need to use the env var?

Hi @RatanRSur , thanks for picking it up.
It is ok for us to use command line argument as a workaround for now.

This is the likely culprit.
https://github.com/hyperledger/besu/blob/main/besu/src/main/java/org/hyperledger/besu/cli/util/ConfigOptionSearchAndRunHandler.java#L61
You can see that Besu only looks to --config-file it needs to also look at the environment variable BESU_CONFIG_FILE


Cache directory and database created even when cache_seed = None

Prior to version 0.2.0, setting use_cache = False in the llm_config would prevent the .cache directory from being created.
The new method of disabling cache is to use seed = None (prior to 0.2.0b4) or cache_seed = None (after 0.2.0b4). However, setting the seed to None does not prevent the .cache folder from being created. In fact the following sqlite database gets initialized:
.cache/None/cache.db
This causes the fix in #593 to be less effective than with prior versions.

I had a similar small discussion -- adding here for context if it helps? https://github.com/microsoft/autogen/pull/600#discussion_r1387311379

how to fix the issue of cache #when autogen is deployed in AWS lambda


This socket has been ended by the other party

Issue Id: dda9a9d9-c965-4e8f-ef83-cbd9fb4e346fVersions - 0.10.6-release-  dfc08dc9edf6607483b193bf934d3badc3fa2e11-  db71ac615ddf9f33b133ff2536f5d33a77d4774eStack Error: This socket has been ended by the other party    at Socket.writeAfterFIN [as write] (net.js:268:12)    at c:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\jrieken.vscode-omnisharp\out\omnisharpServer.js:383:40     at StdioOmnisharpServer._makeNextRequest (c:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\jrieken.vscode-omnisharp\out\omnisharpServer.js:378:16)     at StdioOmnisharpServer.OmnisharpServer._processQueue (c:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\jrieken.vscode-omnisharp\out\omnisharpServer.js:254:14)     at c:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\jrieken.vscode-omnisharp\out\omnisharpServer.js:227:23     at StdioOmnisharpServer.OmnisharpServer.makeRequest (c:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\jrieken.vscode-omnisharp\out\omnisharpServer.js:216:23)     at null._onTimeout (c:\Program Files (x86)\Microsoft VS Code\resources\app\extensions\jrieken.vscode-omnisharp\out\features\dia

+1

on mac os x have this error

This issue was moved to OmniSharp/omnisharp-vscode#209


HTTP redirect option

With FORCE_HTTPS env var, we should enable the following nginx config:
    if ($http_x_forwarded_proto != "https") {
        return 301 https://$host$request_uri;
    }

I started working on this in branch https, however, it does not work correctly on our own cloud v4 clusters due to a different http header being set. We use X-Forwarded-Scheme instead of the more standard X-Forwarded-Proto.

Support for both headers is now supported by the runtime since 7.23.3


Make compatible with<EMAIL_ADDRESS>

congress needs to use this. It's running on<EMAIL_ADDRESS>😞

Do we have an idea of what part of the code needs to change to support Node 0.8?  It doesn't use any ES2015 features, so is this just adding an engine to the package.json?

lab is not compatible. Will need to either downgrade the version or use something else.

since there are no active tests on this project, I feel like we may have to just try it and see if there are any issues..
@musgravejw setting the engines property is just a 'hint', it does not actually affect anything really.

@jackboberg oh, it doesn't affect the node version of the dependencies?


The Razor component type is not always available to the next code blocks

Expected behavior
The component type should be available to the next code blocks.

#!blazor --name HelloWorld

Hello world

#!csharp
var component = new HelloWorld();
component.GetType().Name

Actual behavior
When running in Jupyter, the previous compilation doesn't contain the Razor component type.
This currently works when running in VS Code with both .dib and .ipynb file types.
This also works as expected in the unit tests.

Might be related to issue https://github.com/dotnet/interactive/issues/1378

https://github.com/plbonneville/BlazorInteractive/blob/0c93d590de06127d218778e874577093d4e78889/src/BlazorInteractive/BlazorExtensions.cs#L70

This was a \r\n vs \n issue. Nothing to see here.


New pathTypeFilter to exclude BIOS boot partition

Related: harvester/harvester#1382, harvester/harvester#1241
Solution
Introduce a new pathTypeFilter.
Partition like BIOS boot in GPT scheme should not be provisioned.
So do the parent disk of that partition. Thus, if any partition matches
excluded parttype, both the partition and its parent disk get excluded.

Tested from my custom build. There is no blockdevice exist.

@w13915984028 would be nice if u can submit an enhancement PR of those flaws and assign @weihanglo to review, cheers.


BUG 🐞: Improve Alignment and Resolve Overlapping of Chatbox and Scroll-to-Top Button

Hey @subhadipbhowmik,
Here are the issues i encountered -
Current Behavior :

The chatbox is positioned at the rightmost bottom corner of the website and The scroll-to-top button is placed above the chatbox, creating a "Big Gap" between the two of them (Which i think is not improving the user experience to the fullest).
When the chatbox is opened, it overlaps with the scroll-to-top button, causing usability issues.

Screenshots

This is where the first issue comes : (The big gap between those two)

This is where the Second issue comes : (the overlapping between them)

Steps To Reproduce

Go to the website.
Scroll down to the bottom of the page.
Observe the position of the chatbox at the rightmost bottom corner and the scroll-to-top button above it.
Click on the chatbox to open it and observe the overlap with the scroll-to-top button.

The Expected behavior

The chatbox and scroll-to-top button should be aligned in a visually appealing and efficient manner.
There should be no overlap between the chatbox and the scroll-to-top button when the chatbox is opened.

Proposed Solution :
Below two changes will promise to improve the user experience

Move the scroll-to-top button to the leftmost bottom corner of the website.
Align the scroll-to-top button horizontally with the chatbox.

Please assign this issue to me under GSSoC'24.
I look forward to contributing to this project by implementing the proposed solution.

@subhadipbhowmik


DPL-089 Improve re-racking performance

Danni is planning to run through this, once an issue has been fixed with the Samples Extraction training environment, to verify that the updating Sequencescape step really is the one that takes a long time.
User story
As a user in Sample Management, using the re-racking feature, I would like the time I have to wait for the software to complete tasks. Currently, a single re-racking can take 45 mins - 1 hour. This means I can only get through 4-5 a day, which is not good for the DDD project, where we have hundreds to process.
Who are the primary contacts for this story
Danni, Katy, Eduardo, Andrew
Acceptance criteria
To be considered successful the solution must allow:

[ ] Wait time in the part where Sequencescape is updated with the new plate is reduced from ? to ?

Additional context
This feature has come up recently because it will also be used in the Cardinal pipeline. It is thought that improved tube rack / re-racking functionality could also be useful for the Tree of Life project, and Long Read, as they both use tubes currently.
Sample Management SOP for re-racking -
Samples extraction screen shots for SOP.docx

Speak to Danni to check??

For 2 x 48 tubes, into 1 x 96 tubes. Refer to SOP attached to story to understand what actions are.

Action
Duration

Re-racking - Create a new empty TubeRack
Almost instant

Re-racking - Upload .csv layout for TubeRack
Almost instant

[Reracking] Apply layout into TubeRack
1 min 15 secs

Add a barcode
5-20 secs (but no spinner)

[Reracking] Move barcodes from TubeRack into the Plate
1 min 30 secs

Update TubeRack in Sequencescape
45 secs

From James:

General improvements I spotted when looking at sample extraction:

Background tasks spin up a process per task. This will add significant overhead on anything asynchronous. Rails apps aren’t the fastest to spin up.
There were lots of N+1 query issues, both with the samples extraction database and the SS API calls. These need to be tackled one by one, but typically give major performance boosts

From Danni:

We are using this feature almost daily for our extractions work and over the coming weeks we will be picking up the DDD project which will require this feature. From memory it’s the DDD re-racking that takes 45mins in extraction lims currently so speeding this feature would increase my team throughput massively.


Fix output filename can not begin with '-'

When somebody wants to use a output directory name begin with -, some program (mkdir, file, ...) which invoked by furaffinity-dl will not work.
Some users' name begin with -, this problem is confusing for people who want that directory name is same as user names.
$ ./furaffinity-dl -c cookies.txt -r -o '-zek-' gallery/-zek-/
mkdir: invalid option -- 'z'
Try 'mkdir --help' for more information.

This patch has fixed the problem.
It's worth noting that exiftool doesn't understand the meaning of --, so I had to use temporary file as a hack

Huh odd, I would of expected exiftool to use getopts and thus understand --


Add tag for Cocoapods

Hi, I'm the guy who pushed MTLManagedObjectAdapter to Cocoapods.
It's my fault that I set the deployment_target of iOS to 8.0 and of OS X to 10.9.
as mentioned in #12, I'd like to update/fix the podspec. But by the policy of Cocoapods, if I want to modify a podspec, I have to change its version number. And also there should be a tag which contains the version string.
I would change the version number of the podspec to 1.0-1 which means the same as 1.0 but just a fix to podspec. Hence I need a tag, 1.0-1, which also points to 1.0 commit.
could I request a tag named 1.0-1 and point it to the same commit of 1.0 tag? Thanks.

hi,
I am wondering what the timeline is for this issue? We are working on a project with a deployment target of 7.1 and would like to use MTLManagedObjectAdapter. Thanks!


Add missing translation to Accounts#index

Coverage remained the same at 99.06% when pulling 79c034cc3ffe787f6520b72f35df449c97e45daf on aviav:translation-fix into 7ef3ef8e43819b56806bbaae05b5228c0242efe6 on codevise:master.


Implement a Campaign Mode

[ ] "Campaign" button on main menu
[ ] Multiple campaigns on a Campaign Selection Screen
[ ] Campaign Screen shows all maps in a campaign in order
[ ] Each map unlocks when finishing another map specified by a hash
[ ] Improve Map Highscore UI by showing best coins and moves result under each map in the list row

To implement this feature, several other features need to be implemented:

#10
#7
#5

[ ] Save the texture pack together with the map, override the global texture pack in Campaign mode with the one set inside the map

Campaign mode should have a "Level Selector Maze" that unlocks with campaign progress and allows the player to choose a path / that branches in different directions to allow different campaign trails
Paths in the campaign trail should have labels that are shown to the user (like "Classic Psion 5mx levels", "Tutorial", "...")
The tutorial should be the first campaign trail branch and unlock more tutorials, like "Moving"->"Jumping"->"Ladders"->"Doors"->...
The campaign trail progress is saved in the highscores database per-player, the high score for each map is shown as usual
We need a struct "Game mode" that is either "Tutorial", "Campaign" or "Free Play"


Design proposal: kpt live -- Inventory to ResourceGroup

This PR is a design proposal on enabling kpt live apply to deploy from STDIN and migrate inventory from Kptfile to a standalone ResourceGroup object/file.

@yuwenma seemed like a couple of comments were resolved without answers.  If there is a blanket strategy to resolve it - let me know.  I think it might be good for people to know why those alternatives were provided or not considered.

🚥 ping....

@droot  Updated to '--rg-file`

@yuwenma Thanks for the doc. Approved it.
@droot We can merge it once you approve.

@yuwenma I had a discussion with @phanimarupaka  today about how to treat inventory resource wrt to kpt fn and kpt pkg commands.  Pl. take a look at the summary of the discussion in the last comment. Can you pl. ack the proposal in the comment (below) and would be even better if we can capture this summary in the doc itself somewhere because it impacts the implementation.
https://github.com/GoogleContainerTools/kpt/pull/2576#discussion_r760548405

@droot @yuwenma I added the commit about the rg file interaction with fn commands.

Merging this PR. Thanks for your contribution @yuwenma. @rquitales Please feel free to start implementing this and let us know if you have any questions.
cc @mikebz


HWPV-184: Update to Navigation

@psoleckimoj Have shrunk this PR down, hopefully slightly easier to read through.
Changes in this PR are now:

roleChecking middleware
username-serialiser test
app & app maintenance updates to reflect these
minor linting

@psoleckimoj Thanks! Will try getting it running in maintenance mode with that change first 😄


[core] Organizing Issues

Problem description
So many of the issues are untagged, and it might be overwhelming for new collaborators to see that we have 450-ish issues. The thing is, we don't actually have 450-ish issues. Many of the issues are just discussions, questions, or feature requests. It might be a good idea to tag the issues that we currently have so that both old and new collaborators can see what's really happening in the issues tab. I went through these 450-ish issues and wrote what tag I think they should be under, tell me what you guys think. I also found some issues I thought looked like they could be closed. Thanks!

 I could even implement it if someone from the team says "ok, go ahead"

So many of the issues are untagged

@mpontikes That's true. I do agree that it's can feel intimidating for a new contributor to see all those issue 530+ now as we are speaking. We definitely need to be more organize.
Thanks for doing this first iteration.
Github have just added a new Projects section. I think that we could take advantage of this new feature.
I have started creating one project per component. E.g. with the Snackbar.
Why using this per component approach?
Because our components are quite independent.
We work on one component at the time.
It's great to have a full overview about one component before starting working on it.

And now the issues looks like:

Not labeled issues are rapidly growing...
@oliviertassinari
Could Callemall give me write access for this repo for triaging bugs (label them, close as dups, check is them really happens). You can add me as outside collaborator. And of course, you can check my history activities in audit log
My email<EMAIL_ADDRESS>I'll try to check new issues at least once a week and up to each working day.

@ashl1 Thanks for your interest in helping us. Are you on gitter.im so we can chat about it :).

We have tagged all the linked issues. I'm closing this one.
@ashl1 How did you generate that chart? I would love to see how that looks like 7 months later.


Need cssOptions['position'] to force css files order

Having spent quite much time trying to arrange css files of several bundles in correct order (without success) I finally think that something similar to handling jsOptions['position'] is nesessary for css in \yii\web\View::registerAssetBundle, \yii\web\View::registerCssFile.
I don't mean  POS_HEAD, POS_READY like for js files, but possibility to define number in cssOptions['position'] manually.
Only depends for ordering css files is not enough.
I struggled to arrange this to make custom site.css be in the end - without success:
<link rel="stylesheet" href="/assets/281406ee/css/bootstrap.min.css">
<link rel="stylesheet" href="/css/site.css">
<link rel="stylesheet" href="/assets/864382ef/css/bootstrap-editable.css">
<link rel="stylesheet" href="/assets/6d46f80c/css/bootstrap-popover-x.css">
<link rel="stylesheet" href="/assets/7585d6d9/css/bootstrap-typeahead.css">

All bundles here depend on bootstrap asset, but even if I place registering widgets bundles in main layout it makes no effect.
And explicit calling registerAssetFiles for bundle in main layout results such buggy situation with dependencies:
<link rel="stylesheet" href="/assets/864382ef/css/bootstrap-editable.css">
<link rel="stylesheet" href="/assets/281406ee/css/bootstrap.min.css">

or
<link rel="stylesheet" href="/assets/6d46f80c/css/bootstrap-popover-x.css">
<link rel="stylesheet" href="/assets/281406ee/css/bootstrap.min.css">

What's the problem about letting the bundle containing site.css to depend on other bundles?

The problem is that on some pages we don't need one of the widgets, on some pages - another.
Different layouts are rendered depending on page. And may be in future we'll get rid of some widget.
That is why AppAsset containing site.css depends on bootstrap which may be considered to be something constant.
Anyway, there is something wrong with css handling in bundles, maybe it is interfered by js ordering and ordering of css is done 'as is'.
I finally achieved my goal splitting AppAsset into JsAsset and CssAsset.
And when dependency is not prepended dependent like this:
<link rel="stylesheet" href="/assets/864382ef/css/bootstrap-editable.css">
<link rel="stylesheet" href="/assets/281406ee/css/bootstrap.min.css">

is a separate issue I suppose.

the asset is meant to provide a functionality on the frontend. Sometimes it requires css, js, images and even fonts, for example bootstraps. If you only load the JS then you won't have bootstraps at all.
Also many widgets load their own assets under the hood to work seemlessly, DatePicker will load the required assets when its called in case you don't have them preloaded in your layout. I am working on angular.js assets and plugins and they will behave the same.

@Faryshta
I don't say I load only css or js - I talk about possibility to arrange css files in designed order similar to the feature we have for js.
If you are talking about my phrase

I finally achieved my goal splitting AppAsset into JsAsset and CssAsset.

then - I did not loaded only js or css - I just split assets and loaded both bundles one by one, but result of that split was desired css ordering.

CSS files registration takes dependencies into account the same way JavaScript registration does.

@SDKiller what you want is to load site.css before loading the css files from the other assets?

So in order for your CSS from asset bundle A to be registered before CSS from asset bundle B you need to make sure that B depends on A.

The problem here, of course is that if in some pages you don't need B there should be a way to specify optional dependenct that won't be requested implicitly.

true : need this option to CSS files, like JS. Sometimes need fast include any css...  but without policy of  dependency.

still need position for css assets in 2017, google pagespeed did'nt like a lot of css in head.. can anyone help?

@diss1dent CSS can't be put elsewhere according to HTML spec. What Google suggests is NOT putting it into body.

dependencies of assets is not enough for css source
Currently ,  the default order rule of css file loaded is reversed, its means that:
last register get first position.
This make the style could not work, if there is some override styles then.
I suggest to make css file first register then go to first place.

Won't be implemented in 2.0.


Fix #148

Resolves https://github.com/FriendsOfCake/bootstrap-ui/issues/148

Current coverage is 96.13% (diff: 100%)

Merging #159 into master will not change coverage

@@             master       #159   diff @@
==========================================
  Files            12         12
  Lines           336        336
  Methods          37         37
  Messages          0          0
  Branches          0          0
==========================================
  Hits            323        323
  Misses           13         13
  Partials          0          0

Powered by Codecov. Last update fedc830...c17a2a0


Missing INDO CISD Output Block (as in MOPAC2016) in the new MOPAC v22.0.6

Describe the bug
Running (one of the tests) using MOPAC v22.0.6 will generate an output (co_cisd.txt) that is different from the existing (output file) generated by MOPAC2016 (Version 20.173L) in a way that is lacking important information pertinent to INDO CISD energy values etc.
To Reproduce
Run the test input file with the two aforementioned versions of MOPAC.
Expected behavior
Lack of INDO CISD block of results in the output file generated by MOPAC v22.0.6
Operating system
macOS Ventura 13.3.1 (a) (22E772610a)
Additional context
N/A

This is one of several INDO tests that appears to have been orphaned - it is in the test directory, but I forgot to add it to the list of tests in the CMake script back when the INDO tests were first donated. There are logic barriers that suppress the standard MOPAC output block for INDO calculations, and they may have been added after the old test reference data was generated. I will activate these tests and make some decisions about these barriers. INDO is a spectroscopy model, so many of the standard outputs like heat of formation don't really mean anything and can give erratic and unphysical values. Some of the other observables might make more sense, but they still might not be intentional model outputs.


Implementing jQuery

Hey @KiranPolaki, can I use jQuery cross browser compatible way to handle common DOM tasks? If Yes then please assign this to me. I would like to contribute to this issue and please add the tags of hacktoberfest.

Great idea but that is too much work i guess?

I would be just working with "script.js". I won't change any classes or whatsoever. So it will be fine. Can you please assign this to me? And please mention hacktoberfest tags.

I will be adding tags that's not an issue but i never worked with jquery so i am thinking🤣
. if any issue occurs then i will have to fix it right😅

But i still wanted to add a feature where there will be a button at the end of the game by clicking that we can see all the contributor's who contributed to this project🤔.
So if u are interested to do that i can assign that to you (or) i will do it myself anyways😅.

I will be adding tags that's not an issue but i never worked with jquery so i am thinking🤣 . if any issue occurs then i will have to fix it right😅

If that's the case I'll make a new branch and you can have two branches one with JQuery and other with Vanilla js. If you find it fine then you can merge them 😄

But i still wanted to add a feature where there will be a button at the end of the game by clicking that we can see all the contributor's who contributed to this project🤔. So if u are interested to do that i can assign that to you (or) i will do it myself anyways😅.

Yes please. I would like to work on that.

In the end i have to merge right😅. without that u can't get in hactoberfest

But i still wanted to add a feature where there will be a button at the end of the game by clicking that we can see all the contributor's who contributed to this project🤔. So if u are interested to do that i can assign that to you (or) i will do it myself anyways😅.

Yes please. I would like to work on that.

Sure😀

In the end i have to merge right😅. without that u can't get in hactoberfest. i am good with vanilla js but jQuery is not my thing.

Yep, I get it.

But i still wanted to add a feature where there will be a button at the end of the game by clicking that we can see all the contributor's who contributed to this project🤔. So if u are interested to do that i can assign that to you (or) i will do it myself anyways😅.

Yes please. I would like to work on that.

Sure😀
Thanks! I will start working on it. I need to render "newContributor.html" right?

i though of getting data from Git API's but i think that is too much to ask 😅
For now just apply the same styling as game to new contributor.html and update it and add a button in index.html where this will be redirected to newContributors.html

I haven't worked with Git API😅.
Okay Thanks, I'm working on it then👍

Hey @KiranPolaki I've added "newContributor.html" with styling. Please merge it with tags. And if you need a favicon for "newContributor.html" I will start working on it once this PR gets merged. Have a nice a Day:)

FYI....
i just updated the contributors page😅 u can have a look if u want to know how we use git api's to fetch data and display them

Hey @KiranPolaki that's amazing. It looks far more better than previous one. I was able to understand the code. I had been attending events so I didn't see your message.

Btw, how long does it take to update me on that page? 😅. That looks cool.

Is it in waiting stage? probably they will take 7 days before confirming. in this 7 days ur merge request must not be canceled
Thank you

Yes it is in waiting stage. Thanks for the clarification.

Don't focus on Hacktoberfest it is just the beginning.
Focus on Learning and keep contributing😎

Yeah sure. If you have any other projects then I would like to contribute there as well. Thanks!!


Add request all batches feature to gbt

This patch adds the request all batches command, and does some minor cleanup and improvements to the gbt client functionality in general.

Looks like aws_lc is causing a build issue in CI


GEOMESA-2683 - Converter Metrics - AWS Cloudwatch Reporting

Signed-off-by: Forrest Feaser<EMAIL_ADDRESS>

@forrestfeaser can you run the build/calculate-cqs.sh script and commit the changes? (the script should have been fixed with the last commit to master)


[Feature Request] Hasura Pro Version Option

Hi @elitan ,
For hasura's backend, testing and monitoring are almost non-existent right now. Hasura basic console does not offer the features for us to run regression testing and monitor the traffic. Is it possible to include this feature in the near future?
Thanks

You can use Nhost together with Hasura Pro/Cloud today, although you would have to manage Hasura Cloud separately.

Any hint how to set this up? Could be interesting for us as well!

You can read all about Hasura Cloud here: https://hasura.io/
As soon as we have the connection string available for databases you can connect any service (like Hasura Cloud) to the database.


[TW#17753] gatts_add_char_descr_evt_param is missing characteristic handle?

Im wondering now, since descriptor is strictly connected to characteristic and not to service, why in gatts_add_char_descr_evt_param structure we have uint16_t service_handle but we dont have characteristic handle. Instead we have char_uuid, but what if we have few characteristics with the same UUID in one serviceand each characteristic have CCC descriptor? This is very real situation with ie HID devices. We can have many raport characteristics (0x2a4d).

@Yulong-espressif As promissed here
https://github.com/espressif/esp-idf/issues/1484#issuecomment-357162497
name has been changed, thanks. If there is no plan to add char handle then this issue can be closed.

@chegewara We have fix this bug in the new version of SDK, thanks.

Thanks.


sc-14934 make generate API Key modal responsive

Scope of changes
Fixes SC-14934
Fixes SC-14931
Fixes SC-14913
Type of change

[ ] new feature
[x] bug fix
[ ] documentation
[ ] testing
[ ] technical debt
[ ] other (describe)

Acceptance criteria
https://user-images.githubusercontent.com/54010836/224971032-fc8d6b40-0344-43f8-acb2-9b2e5cf917d9.mov
Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]  Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
[ ] Are there any TODOs in this PR that should be turned into stories?

@masskoder @elysee15 this PR merged in a package.json and yarn.lock into the top level of the repository. Please check to make sure your dependencies are correct and that npm package files do not get committed outside of the directory they're supposed to be in.


rbac: honor useAuthenticated

Please provide a description of this PR:
Sending Principal_FilterState to old proxies will cause NACKs, which is dangerous since proxies cannot get rbac updates
Fixes https://github.com/istio/istio/issues/43785

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

Test name
Commit
Details
Required
Rerun command

gencheck_istio
79c445622e359ed54ad54d3457852675e97ede70
link
true
/test gencheck

lint_istio
79c445622e359ed54ad54d3457852675e97ede70
link
true
/test lint

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


getEnvironmentVariableCollection: not working when reloading the workspace

Re #93601
I was running an extension with below simple setup, which modifies PATH when activated. The activation event for this extension is *.
import * as vscode from 'vscode';
import * as path from 'path';

export function activate(context: vscode.ExtensionContext) {
	const collection = vscode.window.getEnvironmentVariableCollection();
	const workspaceFolders = vscode.workspace.workspaceFolders;

	if (workspaceFolders?.length) {
		collection.append('PATH', ':' + path.join(workspaceFolders[0].uri.fsPath, 'node_modules/.bin'));
	}
}

For newly created Terminal, it works fine. If I create a terminal and then reload the workspace, the terminal will break restored but PATH is then not modified.

Extensions can use getEnvironmentVariableCollection(true) to make sure when the window reloads, the environment variable is already there but it doesn't work if the extension wants to calculate the variable after its' activated, in which case it doesn't want any caching at all.

Extensions can use getEnvironmentVariableCollection(true) to make sure when the window reloads, the environment variable is already there but it doesn't work if the extension wants to calculate the variable after its' activated, in which case it doesn't want any caching at all.

This is by design, using persistent=true will cache everything between reloads so that terminals aren't blocked by ext host and extension activation. An extension should want to reassess/reassign everything when it activates, if the collection is the same then when we do the UX side of this no indicator will pop up saying it's stale.


Crash on use of OCSP_sendreq_nbio() in 3.0.0-alpha12

The triggering code is outlined in https://github.com/openssl/openssl/issues/13873. This crash didn't happen in alpha11. https://github.com/openssl/openssl/issues/13938 was a thing, however.
Note line numbers may not exactly align in GDB outputs below...
(ugdb-x86_64-7.11-912) bt
#0  0x000000082efe8d21 in ASN1_get_object (pp=0x84a3fd178, plength=0x84a3fd180, ptag=0x84a3fd18c, pclass=0x84a3fd188, omax=-1) at crypto/asn1/asn1_lib.c:62
#1  0x000000082eff8751 in asn1_check_tlen (olen=0x84a3fd338, otag=0x0, oclass=0x0, inf=0x84a3fd32e "", cst=0x84a3fd32d "", in=0x84a3fd330, len=-1, exptag=16, expclass=0, opt=0 '\000', ctx=0x84a3fd438) at crypto/asn1/tasn_dec.c:1115
#2  0x000000082eff74ef in asn1_item_embed_d2i (pval=0x84a3fd430, in=0x84a3fd4a0, len=-1, it=0x82f39bed0 <OCSP_RESPONSE_it.local_it>, tag=16, aclass=0, opt=0 '\000', ctx=0x84a3fd438, depth=1) at crypto/asn1/tasn_dec.c:312
#3  0x000000082eff6c24 in ASN1_item_ex_d2i (pval=0x84a3fd430, in=0x84a3fd4a0, len=-1, it=0x82f39bed0 <OCSP_RESPONSE_it.local_it>, tag=-1, aclass=0, opt=0 '\000', ctx=0x84a3fd438) at crypto/asn1/tasn_dec.c:129
#4  0x000000082eff6b43 in ASN1_item_d2i (pval=0x84a3fd430, in=0x84a3fd4a0, len=-1, it=0x82f39bed0 <OCSP_RESPONSE_it.local_it>) at crypto/asn1/tasn_dec.c:119
#5  0x000000082f1aa68d in BIO_mem_d2i (mem=0x0, it=0x82f39bed0 <OCSP_RESPONSE_it.local_it>) at crypto/http/http_client.c:741
#6  0x000000082f1aa608 in OSSL_HTTP_REQ_CTX_sendreq_d2i (rctx=0x84c4832c0, it=0x82f39bed0 <OCSP_RESPONSE_it.local_it>) at crypto/http/http_client.c:791
#7  0x000000082f1e6597 in OCSP_sendreq_nbio (presp=0x84a3fec58, rctx=0x84c4832c0) at crypto/ocsp/ocsp_http.c:45
[...]
(ugdb-x86_64-7.11-912) f 0
#0  0x000000082efe8d21 in ASN1_get_object (pp=0x84a3fd178, plength=0x84a3fd180, ptag=0x84a3fd18c, pclass=0x84a3fd188, omax=-1) at crypto/asn1/asn1_lib.c:62
62          ret = (*p & V_ASN1_CONSTRUCTED);
(ugdb-x86_64-7.11-912) p p
$3 = (const unsigned char *) 0x0
#6  0x000000082f1aa608 in OSSL_HTTP_REQ_CTX_sendreq_d2i (rctx=0x84c4832c0, it=0x82f39bed0 <OCSP_RESPONSE_it.local_it>) at crypto/http/http_client.c:791
791         return BIO_mem_d2i(OSSL_HTTP_REQ_CTX_transfer(rctx), it);

Looks like OSSL_HTTP_REQ_CTX_transfer() fails, and then we're passing NULL as the BIO into BIO_mem_d2i()... which
then blows up several frames down because checking for NULL pointers is totes overrated.

(ugdb-x86_64-7.11-912) f 7
#7  0x000000082f1e6597 in OCSP_sendreq_nbio (presp=0x84a3fec58, rctx=0x84c4832c0) at crypto/ocsp/ocsp_http.c:45
45              OSSL_HTTP_REQ_CTX_sendreq_d2i(rctx, ASN1_ITEM_rptr(OCSP_RESPONSE));
(ugdb-x86_64-7.11-912) p *rctx
$4 = {
  state = 4096,
  readbuf = 0x84ec31400 '\245' <repeats 200 times>...,
  readbuflen = 4096,
  wbio = 0x8387b5b00,
  rbio = 0x8387b5b00,
  mem = 0x83d92f500,
  method_POST = 1,
  expected_ct = 0x0,
  expect_asn1 = 1,
  len_to_send = 226,
  resp_len = 0,
  max_resp_len = 102400,
  max_time = 0,
  redirection_url = 0x0
}

@DDvO do you have any insight on this one?

Thanks @jwalch for your bug report.

then we're passing NULL as the BIO into BIO_mem_d2i()... which then blows up several frames down

Yes, BIO_mem_d2i() (or its caller) did not check for mem == NULL, and
several low-level ASN.1 functions did not check for input len <= 0, either,
resulting in a crash in transmission failure.
I've added checks along the whole backtrace of functions you provided, in #14356 and #14357.

Thanks, @DDvO . Cleaning up the error handling here will definitely help.
It's unclear to me why the BIO_write() within OHS_WRITE state is failing for non-retryable reason (I'm pretty sure that must be what's happening based on the state of the rctx object). I also have some other tests (unrelated to OCSP) failing for the first time when integrating alpha12, which I suspect to be due to some common underlying reason... if I eventually track that to some issue within OpenSSL I can file a new issue for it. For this one we can just operate under the assumption that OSSL_HTTP_REQ_CTX_transfer() is failing for a "valid" reason...