error: no rules expected the token ;

I seem to be having trouble incorporating the example code into a toy project I am playing with to learn a bit of rust.
Not sure what he cause is, but I'm on 1.64 on fedora if that makes much of a difference
   Compiling axum-server v0.4.2
error: no rules expected the token `;`
  --> /home/fejfighter/.cargo/registry/src/github.com-1ecc6299db9ec823/axum-server-0.4.2/src/handle.rs:94:23
   |
94 |                 biased;
   |                       ^ no rules expected this token in macro call

error: no rules expected the token `;`
   --> /home/fejfighter/.cargo/registry/src/github.com-1ecc6299db9ec823/axum-server-0.4.2/src/server.rs:175:27
    |
175 |                     biased;
    |                           ^ no rules expected this token in macro call

error: no rules expected the token `;`
   --> /home/fejfighter/.cargo/registry/src/github.com-1ecc6299db9ec823/axum-server-0.4.2/src/server.rs:204:35
    |
204 | ...                   biased;
    |                             ^ no rules expected this token in macro call

error: no rules expected the token `;`
   --> /home/fejfighter/.cargo/registry/src/github.com-1ecc6299db9ec823/axum-server-0.4.2/src/server.rs:214:19
    |
214 |             biased;
    |                   ^ no rules expected this token in macro call

error: could not compile `axum-server` due to 4 previous errors

Something is wrong with your tokio::select! I guess. I can't reproduce it. Can you share your Cargo.toml?

Found the problem. It seems like tokio added biased option (which axum-server use) in version 1.4 and you are forcing to use version 1.2.
A slight oversight on my part. I should put tokio dependency to version 1.4 instead of 1.

Awesome, that works for me.
Not 100% sure why I had 1.2, probably from some tutorial/quick start I was looking at.
closing as this seems to fix the issue


feat: Klaviyo flatten WriteParams.RecordData

#1169  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Add fallback if the examine option has been removed by another plugin

https://github.com/donth77/loot-lookup-plugin/blob/bd93c4fc56030d55fd8e0b10a030de9e22df899a/src/main/java/com/lootlookup/LootLookupPlugin.java#L110-L112
Right now if there is an attackable NPC but it has no examine menu entry (menu entry removed by different plugin) then there will be no loot lookup menu entry added.

I think this should achieve the same result without checking for the "Examine" text:
https://github.com/donth77/loot-lookup-plugin/commit/b10af87798e842b276d480373ef293f9d6528ebd#diff-e37addb33b084599af06f9ca8f187542da8cf268727dea17596d33a7717eab5cR116

@CalebBarnes
I think this should achieve the same result without checking for the "Examine" text:
https://github.com/donth77/loot-lookup-plugin/commit/b10af87798e842b276d480373ef293f9d6528ebd#diff-e37addb33b084599af06f9ca8f187542da8cf268727dea17596d33a7717eab5cR115

That is perfect, so it would just add the option as the last item in the list?
  MenuEntry entryToAppendOn = menuEntries[menuEntries.length - 1];

It would behave slightly differently than it does now if you are okay with that.
For example, the option would appear after the examine menu entry for most users, instead of before the examine option like it is now.
And then also if you are right-clicking on a pile of mobs, it would probably add all of the new menu entries to the very bottom but that might even make it easier for the user to select them.
I am totally okay with this solution, just wanted to make sure you know it would change things a little bit!


[Core][OTel] Pass schema URL into tracer

This provides more information as part of the tracer's instrumentation scope.
Also, fixed an attribute name in the rename dict (messaging_bus.destination should be message_bus.destination as that is what is actually being used by the amqp libraries.
Part of #25838

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-core-tracing-opentelemetry


"Security group does not exist" error when deploying to Beanstalk

Deploying a .NET Core application to Linux, using an RDS security group can fail with a "Security group does not exist" error:

Repro steps:

Make an RDS instance on the default vpc
deploy a .NET Core application to Beanstalk, Linux target, and select the RDS instance security group
deploy fails

Notes:

the security group does get created on the account
this does not happen if deploying to a Windows target
this does not happen if referencing an RDS security group where the RDS instance is in a non-default VPC

Is there an update on the ticket ? Any idea of a timeline ?

I am also waiting for this to be resolved. Thank you!

A fix for this has been released with v<IP_ADDRESS> of the Toolkit.


Custom protocol not working on Mac OS X

Describe the bug
No amount of CORS / CSP headers will get WebKit to not complain about trying to request an insecure/untrusted resource.
Steps To Reproduce
use wry::{
  application::{
    accelerator::{Accelerator, SysMods},
    event::{Event, WindowEvent},
    event_loop::{ControlFlow, EventLoop},
    keyboard::KeyCode,
    menu::{CustomMenuItem, MenuBar, MenuItem, MenuItemAttributes, MenuType},
    window::{Window, WindowBuilder},
  },
  http::{header::CONTENT_TYPE, Response},
  webview::WebViewBuilder,
};

fn build_menu_bar_menu() -> (MenuBar, CustomMenuItem) {
  let mut menu_bar_menu = MenuBar::new();
  let mut first_menu = MenuBar::new();
  let mut second_menu = MenuBar::new();
  second_menu.add_native_item(MenuItem::Copy);
  second_menu.add_native_item(MenuItem::Cut);
  second_menu.add_native_item(MenuItem::Paste);
  second_menu.add_native_item(MenuItem::SelectAll);
  let quit_menu_item = first_menu.add_item(MenuItemAttributes::new("Quit").with_accelerators(&Accelerator::new(SysMods::Cmd, KeyCode::KeyQ)));
  menu_bar_menu.add_submenu("", true, first_menu);
  menu_bar_menu.add_submenu("File", true, second_menu);
  return (menu_bar_menu, quit_menu_item);
}

fn init_stdout_logger() {
  let stdout = log4rs::append::console::ConsoleAppender::builder().build();
  let config = log4rs::config::Config::builder()
    .appender(log4rs::config::Appender::builder().build("stdout", Box::new(stdout)))
    .build(log4rs::config::Root::builder().appender("stdout").build(log::LevelFilter::Debug))
    .unwrap();
  log4rs::init_config(config).unwrap();
}

fn main() {
  // init logger
  init_stdout_logger();
  // init event loop
  let event_loop = EventLoop::default();
  let event_loop_proxy = event_loop.create_proxy();
  // init menu
  let (menu_bar_menu, quit_menu_item) = build_menu_bar_menu();
  // init window
  let window = WindowBuilder::new()
    .with_title("wry-poc")
    .with_menu(menu_bar_menu)
    .with_inner_size(wry::application::dpi::PhysicalSize { width: 1280, height: 1024 })
    .build(&event_loop)
    .unwrap();
  // init webview
  let webview = WebViewBuilder::new(window)
    .unwrap()
    .with_url("PUT_HTTPS_NGROK_URL_HERE")
    .unwrap()
    .with_custom_protocol("ipc".into(), move |request| {
      let path = request.uri().path();
      panic!("{}", path);
      let response: Vec<u8> = vec![];
      Response::builder()
        .header(CONTENT_TYPE, "application/octet-stream")
        .body(response.into())
        .map_err(Into::into)
    })
    .with_devtools(true)
    .with_user_agent("wry-poc")
    .build()
    .unwrap();
  // receive events from event_loop_proxy and evaulate them as ipc-response events
  event_loop.run(move |event, _, control_flow| {
    *control_flow = ControlFlow::Wait;
    match event {
      Event::WindowEvent {
        event: WindowEvent::CloseRequested,
        ..
      } => {
        *control_flow = ControlFlow::Exit;
      }
      Event::MenuEvent {
        menu_id,
        origin: MenuType::MenuBar,
        ..
      } => {
        if menu_id == quit_menu_item.clone().id() {
          *control_flow = ControlFlow::Exit;
        }
      }
      _ => (),
    }
  });
}

import express from 'express'
import cors from 'cors'
import ngrok from 'ngrok'

const run = async () => {
  const app = express()
  app.options('*', cors())
  app.use((req, res, next) => {
    console.log({
      method: req.method,
      url: req.url,
      path: req.path,
      headers: req.headers
    })
    //res.set('Content-Security-Policy', `upgrade-insecure-requests; default-src 'self' 'unsafe-inline' https: ipc://test;`)
    res.set('Access-Control-Allow-Origin', 'ipc://test')
    //res.set('Access-Control-Allow-Methods', '*')
    //res.set('Access-Control-Allow-Headers', '*')
    //res.set('Access-Control-Allow-Credentials', '*')
    next()
  })
  app.use(express.static('public'))
  app.listen(3000, () => {
    console.log('listening...')
  })
  const url = await ngrok.connect({
    proto: 'http',
    addr: 3000,
    authtoken: process.env.NGROK_AUTH_TOKEN,
    region: 'us'
  })
  console.log(url)
}

run()

Expected behavior
The request shouldn't get blocked by WebKit and should make it to the rust with_custom_protocol handler.
Screenshots

Platform and Versions (please complete the following information):
OS: Mac OS X 13.3.1
Rustc: rustc 1.69.0 (84c898d65 2023-04-16)
$ uname -a
Darwin Brandons-MacBook-Air.local 22.4.0 Darwin Kernel Version 22.4.0: Mon Mar  6 21:00:41 PST 2023; root:xnu-8796.101.5~3/RELEASE_ARM64_T8103 arm64
$ rustc --version
rustc 1.69.0 (84c898d65 2023-04-16)

Would you want to assign yourself to resolve this bug?

[ ] Yes
[X] No

Additional context
See comments in https://github.com/tauri-apps/wry/issues/938 that got off topic and started looking this as an alternative to "slow" IPC JSON serialization/deserialization.
Error messages:
The page at ... was not allowed to display insecure content from ipc://test
Not allowed to request resource
Fetch API cannot load ipc://test due to access control checks

cc @wusyong

Ok, I misunderstood your case previously. I thought you had the origin of a custom protocol (i.e. with_url("wry://localhost")) and want to send a request to https server.
If it's this problem, the server need to enable ACAO for the URL scheme.
But it's actually reversed. The short answer I would say is custom protocol doesn't work well in this case.
From my previous attempts, you can use <img src="wry:/example.com/icon.jpg" /> to get the assets, but won't work if you use fetch or XHR.
I suggest you either go for custom protocol in your page or host your own localhost server.
Like I said in the former issue, tauri hosts a server for HMR during development. And embed your project into binary, then use custom protocol in release.
We don't use the setup described in this issue.

Thank you very much for the support. To clarify, with_custom_protocol won’t work if you use with_url from an https:// origin
I look forward very much to upcoming non-string based IPC functionality! Thank you for a great product.


Issue15 ci workflow

The workflow to check if the code  is clang formatted, to compile and to run the tests is now in place so we should run it on the develop branch

Yes sure it makes sense !

yes, I think commit 263ac6a (remove double testing) is a misunderstanding. We should test on both mac and ubuntu, so I'd like that commit to be reversed.
I think Tim's point was that it would be nice to avoid code repetition in the two test cases, but they are so different that I don't see that this is possible.

ok sorry for the misunderstanding, I will reverse this commit.
maybe instead of launching each test one by one we could make a file that lists all the test and that will be run on each platform ? this way when adding a new test , we just have to add it to this file ?

ok sorry for the misunderstanding, I will reverse this commit.

No worries :)

maybe instead of launching each test one by one we could make a file that lists all the test and that will be run on each platform ? this way when adding a new test , we just have to add it to this file ?

Yes, that sounds very good! And more like what @timspainNERSC had in mind, I think.

Yes, that is more what I was thinking of.
I believe that Catch provides a way of managing individual tests in a test suite. This may be a way of achieving this goal, but I have never used it, so I don't know whether it is appropriate for our situation.

I believe that Catch provides a way of managing individual tests in a test suite. This may be a way of achieving this goal, but I have never used it, so I don't know whether it is appropriate for our situation.

Ok. I suggest we approve and merge now and put up an issue about checking if Catch is a better way to do this.

I will investigate the Catch solution, in the mean time I'm manually adding the tests present in develop to the CI script, if that is ok with you ?

That will be absolutely fine.


New filters / lists

We will have different lists, depending on use cases and what users want to see on the home view

This idea has moved into Inventory


[wallet/desktop] feat: upload-qr component unit test

What was the issue?

missing unit test in the UploadQRCode component.

What's the fix?

Added unit test  UploadQRCode component.

Codecov Report

Merging #1919 (dec4e27) into feature/unit-tests-united-2 (432b98b) will increase coverage by 1.23%.
The diff coverage is n/a.

Additional details and impacted files

@@                       Coverage Diff                       @@
##           feature/unit-tests-united-2    #1919      +/-   ##
===============================================================
+ Coverage                        38.17%   39.40%   +1.23%
===============================================================
  Files                              584      584
  Lines                            13981    13981
  Branches                          2358     2358
===============================================================
+ Hits                              5337     5509     +172
+ Misses                            8582     8421     -161
+ Partials                            62       51      -11

Flag
Coverage Δ

wallet-desktop
39.40% <ø> (+1.23%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...tions/TransactionQRAction/TransactionQRActionTs.ts
32.35% <0.00%> (+32.35%)
:arrow_up:

...mponents/QRCode/ImportQRButton/ImportQRButtonTs.ts
63.63% <0.00%> (+36.36%)
:arrow_up:

src/views/modals/ModalImportQR/ModalImportQRTs.ts
46.15% <0.00%> (+38.46%)
:arrow_up:

...TransactionQRAction/SignedTransactionQRActionTs.ts
40.74% <0.00%> (+40.74%)
:arrow_up:

...QRAction/CosignatureSignedTransactionQRActionTs.ts
40.74% <0.00%> (+40.74%)
:arrow_up:

...mponents/QRCode/QRCodePassword/QRCodePasswordTs.ts
41.93% <0.00%> (+41.93%)
:arrow_up:

...s/modals/ModalWizardDisplay/ModalWizardDisplay.vue
42.85% <0.00%> (+42.85%)
:arrow_up:

...QRCodeActions/ContactQRAction/ContactQRActionTs.ts
44.44% <0.00%> (+44.44%)
:arrow_up:

src/components/RecipientInput/RecipientInputTs.ts
100.00% <0.00%> (+45.00%)
:arrow_up:

...CodeActions/MnemonicQRAction/MnemonicQRActionTs.ts
47.05% <0.00%> (+47.05%)
:arrow_up:

... and 13 more


Improve Quickwit tracing documentation...

the first (main) page contains a lot of implementation detail
there are no e2e tutorial to run quickwit, store stuff to S3, ship OTEL logs to hit, and connect jaeger to it.
All the info is there, but in different places.
unnecessary defaults in all commands.

The first point is partially related to #3009,  there is a "too much information" issue

Just updated the first and last points as they are fixed.


Implement two-part cuts where a middle section is not included

This is very useful for "they talk about doing a thing, they do something else, they do the thing" which happens a lot.
This is mostly a thrimbletrimmer task, because it's not clear how we would implement the UI for this.
From the backend's perspective, we just take 4 numbers instead of 2, and implement some fixed transition (eg. a 1sec fade)
between the first and second parts.

An MVP of this is done, I'll make a new issue covering the unimplemented bits


move docker plugin to a maven profile

otherwise can't build outside VSCT (docker registry points to VSCT one)

validated


Fix bug where using S3 integration with non-existent filepath succeeds

Describe your changes and why you are making these changes
This PR error handles the case when we get the file(s) in a non-existent S3 folder. There are several cases:

S3 folder does not exist, this will prompt an error in server "Given S3 directory path '%s' does not exist"
S3 folder exists but empty, this request will succeed and return an empty tuple.
S3 file does not exist, this is handled already.

The error detection is done by taking off the last character in path(i.e. \) symbol and find match that contains \ as delimiter.
Error message when path to folder does not exist:
Running this block of code:

The error I got from Jupyter notebook:

The error I got from Aqueduct server:

Empty Folder:
The block of code I have run:

The execution result:

Related issue number (if any)
https://linear.app/aqueducthq/issue/ENG-2477/fix-bug-where-using-s3-integration-with-non-existent-filepath-succeeds
Loom demo (if any)
Checklist before requesting a review

[ ] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[ ] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[ ] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

@Fanjia-Yan, could you please clarify which screenshots correspond to which part of the app in your first example? In particular, I'd like to know which one is the error message we see in the Jupyter notebook when running this function.

@Fanjia-Yan, could you please clarify which screenshots correspond to which part of the app in your first example? In particular, I'd like to know which one is the error message we see in the Jupyter notebook when running this function.

@vsreekanti Just Updated.

I have also asked this on linear for this issue, might as well put it here:
"I notice that most errors are threw in aqueduct_executor. Therefore on Jupyter notebook it will only show We couldn't execute the provided query. Please double check your query is correct. While on server it has detailed information such as Given S3 directory path 'hello/' does not exist  or NoSuchKey) when calling the GetObject operation: The specified key does not exist. Are those expected or should we surface those error to SDK user?"

Hm, okay. It seems like we're burying the lede a little bit here. The interesting error message is that the path doesn't exist, but we're just saying we could execute the "query" in the notebook. Not only is query not a good word for reading a file, the error message is not very informative. Could we surface the other error message instead?

Hm, okay. It seems like we're burying the lede a little bit here. The interesting error message is that the path doesn't exist, but we're just saying we could execute the "query" in the notebook. Not only is query not a good word for reading a file, the error message is not very informative. Could we surface the other error message instead?

Agree! I will create a new task on linear on making the error message more informative. As of now, should we get this PR in?

Sorry for the confusion, the error message surfacing works, it just my Jupyter is limiting its word character so it is not showing :)

I need a test for this in data_integration_tests/s3_test.py
You can probably just add to test_s3_basic_fetch_failure(). Maybe we call it test_s3_fetch_failure()

I added that inside test_s3_basic_fetch_failure.

Sweet — great work, @Fanjia-Yan!

After some experimentation, this can be really simple. self.s3.Bucket(self.bucket).objects.filter(Prefix=path) will return empty with non-existing file path. It will return only one entry with key directory_name\ upon empty directory.

@Fanjia-Yan @kenxu95, what's the status of this task?

@Fanjia-Yan @kenxu95, what's the status of this task?

Will work on this and get it in today.


Add v2 update docs

Better viewed as https://github.com/rubocop-hq/rubocop-rspec/blob/add-v2-migration-docs/docs/modules/ROOT/pages/upgrade_to_version_2.adoc
TODO

[x] adjust docs related to the changes from #1063

[x] that Include should be used instead of Patterns
[x] note that Include's merge mode is set to override by default

[x] adjust docs related to the final changes to API from #956

[x] syntax changes

[x] mention the change of ExampleGroups::GROUPS to ExampleGroups#regular and from Examples::EXAMPLES to Examples#regular https://github.com/rubocop-hq/rubocop-rspec/pull/956/files#r516238107

[x]       # call super if you define on_new_investigation in a cop

[x] do not use let(:cop) { described_class.new } in custom cops' specs, as this will break specs (config not loaded)

[ ] add a section on how to configure DSL aliases

Before submitting the PR make sure the following are checked:

[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[-] Added tests.
[x] Updated documentation.
[-] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[x] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

@sl4vr appreciate if you could take a look.

@bquorning Green.
Besides all, this includes information on how to migrate to 2.0 given we decide to merge #956 in its current form.

Thanks for the proofreading, @bquorning !

I believe this PR should be the last one to be merged into release-2.0

Agree :+1:

All review notes applied :heavy_check_mark:

@bquorning Ready for the review. There are changes since the last review, the list is in TODO section of the ticket description.
I've rearranged all the commits.

Eagle eye, @sl4vr, thanks a lot!

Suggestions applied, clarifications made. Thanks for the review, @bquorning !

edge-rubocop is back to green! 💚


[BUG] - Scroll position can change in Safari when switching Tabs ()

NextUI Version
v2.1.3
Describe the bug
When using Tabs, Safari keeps scrolling unless the Tabs component is wrapped in a parent which has fixed height.
Likely, when switching tabs, the origin is unmounted, which causes the page to change overall height and to scroll.
Your Example Website or App
https://codesandbox.io/p/sandbox/kind-dream-99pw35?file=/App.jsx:42,18&utm_medium=sandpack
Steps to Reproduce the Bug or Issue
In Safari when scrolled to the bottom of the page and changing Tabs, the scroll position changes.
Expected behavior
Behaviour as in Chrome, so that scroll position is not changing.
Screenshots or Videos
https://github.com/nextui-org/nextui/assets/13927969/8257742e-e6d3-4b2b-b4bc-f955f5d8bf40
Operating System Version
macOS
Browser
Safari

The reproduced link cannot be opened. Could you please share a new link or repository?

Why not? I just tried from device where I am not logged in and works perfectly.
@jrgarciadev already used the link and worked for him.
Can you give some details what does not work for you?

But just in case, as mentioned, it's fairly trivial, just added some space on top of the example code:
import React from "react";
import { Tabs, Tab, Card, CardBody } from "@nextui-org/react";

export default function App() {
  return (

      <Tabs disableAnimation aria-label="Options">
        <Tab key="photos" title="Photos">
          <Card className="h-[300px]">
            <CardBody>
              Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do
              eiusmod tempor incididunt ut labore et dolore magna aliqua. Ut
              enim ad minim veniam, quis nostrud exercitation ullamco laboris
              nisi ut aliquip ex ea commodo consequat.
            </CardBody>
          </Card>
        </Tab>
        <Tab key="music" title="Music">
          <Card className="h-[300px]">
            <CardBody>
              Ut enim ad minim veniam, quis nostrud exercitation ullamco laboris
              nisi ut aliquip ex ea commodo consequat. Duis aute irure dolor in
              reprehenderit in voluptate velit esse cillum dolore eu fugiat
              nulla pariatur.
            </CardBody>
          </Card>
        </Tab>
        <Tab key="videos" title="Videos">
          <Card className="h-[300px]">
            <CardBody>
              Excepteur sint occaecat cupidatat non proident, sunt in culpa qui
              officia deserunt mollit anim id est laborum.
            </CardBody>
          </Card>
        </Tab>
      </Tabs>

  );
}

I see, the reproduction code above was edited - basically using "disableAnimation" fixes the issue, but has hard transition. If Using with animation it jumps around.

This is a part of the code for the Tabs component.

I added the "disableAnimation" parameter within the  tag in your code, like this:

This disables the animation effect of the Tabs component. Then, I looked into the source code of the Tabs component and found that it references import {LayoutGroup} from "framer-motion";. Perhaps, this is an issue with framer-motion.https://www.framer.com/motion/layout-group

@phev8 This is happening for me in chrome too? Did you do something different for chrome to not have this?

Instead of using disableAnimation, an alternative workaround is to use the Tabs as a controlled component and conditionally render the tab content as a sibling.

It also happens with Chrome and Firefox indeed.

@phev8  As I explored a bit, if you wrap your Tabs in a container with a fix height, it solves the issue. I think it's related to how FramerMotion handles animation.


Add samples and update READMEs

We've closed lots of issues recently requesting features that are already possible. These samples show how to harness the power of MSBuild and from dotnet-watch. This should address some of the commonly asked for features such as

allowing dotnet-watch to launch custom processes (like node or python)
running multiple commands from dotnet-watch (dotnet test + dotnet run)
watching custom files
watching multiple projects

Resolves #274

:up: :date:


rds encrypted storage

Use Case - Is your feature request related to a problem? Please describe.
rds encrypted storage for create db instance and cluster
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "RDS",
            "Effect": "Deny",
            "Action": [
                "rds:CreateDBInstance"
            ],
            "Resource": [
                "*"
            ],
            "Condition": {
                "Bool": {
                    "rds:StorageEncrypted": "false"
                }
            }
        },
        {
            "Sid": "StatementForAurora",
            "Effect": "Deny",
            "Action": [
                "rds:CreateDBCluster"
            ],
            "Resource": [
                "*"
            ],
            "Condition": {
                "Bool": {
                    "rds:StorageEncrypted": "false"
                }
            }
        }
    ]
}

Expected Outcome - Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Affected AWS resource
Mention the AWS resources which is affected
Impact
High: The issue makes a service level improvement which affects all users of AWS
Medium: Single feature which affects a single functionality which is optionally enabled in the AWS service
Low: Niche use case which is particularly affecting the AWS resources if it is configured in a certain way
Supported material
Can be either logs, screenshots or documentation links which provide evidence of need of this issue
I can contribute: Yes/No
If you are able to contribute towards resolving this request.
Additional context
Add any other context or screenshots about the feature request here.
Pull Request number
If a pull request has already been created.

Additional IAM actions
      "rds:CreateDBCluster",
      "rds:CreateDBInstance",
      "rds:RestoreDBClusterFromS3",
      "rds:RestoreDBInstanceFromS3",
      "rds:RestoreDBClusterFromDBSnapshot",
      "rds:RestoreDBClusterToPointInTime"

{
  "Version": "2012-10-17",
  "Statement": [
    {
      "Sid": "RDS",
      "Effect": "Deny",
      "Action": [
        "rds:CreateDBInstance",
        "rds:CreateDBCluster",
        "rds:CreateDBInstance",
        "rds:RestoreDBClusterFromS3",
        "rds:RestoreDBInstanceFromS3",
        "rds:RestoreDBClusterFromDBSnapshot",
        "rds:RestoreDBClusterToPointInTime"
      ],
      "Resource": [
        "*"
      ],
      "Condition": {
        "Bool": {
          "rds:StorageEncrypted": "false"
        }
      }
    }
  ]
}


Code was almost the same, so this PR refers to 3 jiras.
https://issues.redhat.com/browse/ISPN-14597
SINTER SINTERCARD SINTERSTORE
https://issues.redhat.com/browse/ISPN-14598
SMEMBERS
https://issues.redhat.com/browse/ISPN-14595
SCARD

Just some small things, and we're good. We should document the behavior of these commands with other concurrent operations over the same sets. As I guess we don't have the same guarantees as in Redis.

I changed the implementation to use atomic getAll, this way the behaviour should be the same as Redis. Looking at the docs I can't find anything special for the intersect commands implemented here.
please squash the commits if all good. getAll is a separate commit


Error with getting balance

I did all the necessary steps to run the PumpBot but when I try to give the command python3 PumpBot.py
I get this:  Caching all BTC pairs average prices...
           This can take a while. Please , be patient...

            Sucessfully catchedBTCpairs!
            Error with getting balance.

This could have happened if you changed the value of the quotedCoin in config.json to an invalid value. If its a valid coin (like BTC) then perhaps try changing your endpoint (change it from default to api1 and see if it works. It could be an error with Binance rather than the program.

Tried all those APIs... same error

I have no idea what could be causing this. If you manage to solve  it, please update this issue with the solution.

Have you checked the api key and api secret? I got the same error when I wrote to check the binance spot test api.

I use the production API key and secret from the live environment in Binance,there is no test. (the API keys have no errors)

any solutions?

Unfortunately No. I tried my best, I need help from the owner.

Investing amount for BTC: 0.00526173
Investing amount in USD: 294.95
Coin pair: hive
A BinanceAPI error has occurred. Code = -2010
Account has insufficient balance for requested action.. Please use https://github.com/binance/binance-spot-api-docs/blob/master/errors.md to find greater details on error codes before raising an issue.

I have the following error:
Caching all BTC pairs average prices...
This can take a while. Please, be patient...
Sucessfully cachedBTCpairs!
Error with getting balance.
Press any key to continue . . .

same error as per LaDice and eternaldreamz above.

I looked at the spec https://python-binance.readthedocs.io/en/latest/_modules/binance/client.html#Client.get_asset_balance and compared with the code in PumpBot.py and it looks fine to me. I wonder if there is sth wrong with the Binance API. Alternatively, I am wondering if is there another method the code can call to get the asset (=quotedCoin) balance through the Binance API...or may be a combination that the balance can be derived.....

well their js does actually work, does etics apply if we pythonize it? https://github.com/BinaryDevil/binance-pump-bot

I have the following error:
Caching all BTC pairs average prices...
This can take a while. Please, be patient...
Sucessfully cachedBTCpairs!
Error with getting balance.
Press any key to continue . . .

I really don't know why you're getting an error honestly. I'm testing it on my end and its working fine. Could be an issue with Binance. Have you tried recreating your keys or trying a different endpoint?

Good news fj317. You are spot on!!! I changed the API key and secret and now IT WORKS!!! I believe because I was using the same API keypair (key & secret) in other software API calls, somehow the BINANCE API didn't like it which is a mystery to me unless they are doing it for security purposes.
I am glad is sorted. thank you again fj317.

I'm glad it's fixed for you ! :)

Closing issue since there seems to be a solution for it now


HBASE-28459 HFileOutputFormat2 ClassCastException with s3 magic committer

OutputCommitter needs to be casted to PathOutputCommitter instead of FileOutputCommitter to allow S3 Magic committer as output committer. PathOutputCommitter is only available starting from Hadoop 3 so this change isn't backwards compatible with HBase using Hadoop 2. In this PR, using reflection to call getWorkPath to get around this
Related PR, which makes this change with hadoop3: https://github.com/apache/hbase/pull/5851
@ndimiduk @bbeaudreault @rmdmattingly

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 36s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 18s
branch-2 passed

+1 :green_heart:
compile
0m 32s
branch-2 passed

+1 :green_heart:
checkstyle
0m 12s
branch-2 passed

+1 :green_heart:
spotless
0m 44s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 31s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 55s
the patch passed

+1 :green_heart:
compile
0m 31s
the patch passed

+1 :green_heart:
javac
0m 31s
the patch passed

-0 :warning:
checkstyle
0m 10s
hbase-mapreduce: The patch generated 1 new + 15 unchanged - 0 fixed = 16 total (was 15)

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
10m  9s
Patch does not cause any errors with Hadoop 2.10.2 or 3.3.6.

-1 :x:
spotless
0m 39s
patch has 65 errors when running spotless:check, run spotless:apply to fix.

+1 :green_heart:
spotbugs
0m 37s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

22m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5858

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 71d04062accb 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 1d5774e33c

Default Java
Eclipse Adoptium-11.0.17+8

checkstyle
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/artifact/yetus-general-check/output/diff-checkstyle-hbase-mapreduce.txt

spotless
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/artifact/yetus-general-check/output/patch-spotless.txt

Max. process+thread count
81 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 53s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m 16s
branch-2 passed

+1 :green_heart:
compile
0m 26s
branch-2 passed

+1 :green_heart:
shadedjars
7m  0s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 20s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 19s
the patch passed

+1 :green_heart:
compile
0m 25s
the patch passed

+1 :green_heart:
javac
0m 25s
the patch passed

+1 :green_heart:
shadedjars
6m 55s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

-1 :x:
unit
19m 18s
hbase-mapreduce in the patch failed.

44m 36s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5858

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 470f350d6b71 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 1d5774e33c

Default Java
Eclipse Adoptium-11.0.17+8

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/artifact/yetus-jdk11-hadoop3-check/output/patch-unit-hbase-mapreduce.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/testReport/

Max. process+thread count
2927 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 29s
Docker mode activated.

-0 :warning:
yetus
0m  6s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
4m  4s
branch-2 passed

+1 :green_heart:
compile
0m 23s
branch-2 passed

+1 :green_heart:
shadedjars
7m  0s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 24s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 38s
the patch passed

+1 :green_heart:
compile
0m 27s
the patch passed

+1 :green_heart:
javac
0m 27s
the patch passed

+1 :green_heart:
shadedjars
7m 11s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 17s
the patch passed

_ Other Tests _

-1 :x:
unit
24m  7s
hbase-mapreduce in the patch failed.

49m 30s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5858

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 2c795f47884d 5.4.0-172-generic #190-Ubuntu SMP Fri Feb 2 23:24:22 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 1d5774e33c

Default Java
Temurin-1.8.0_352-b08

unit
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/artifact/yetus-jdk8-hadoop2-check/output/patch-unit-hbase-mapreduce.txt

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/testReport/

Max. process+thread count
3050 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/1/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

Please run mvn spotless:apply to fix the style issue?

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 45s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 33s
branch-2 passed

+1 :green_heart:
compile
0m 39s
branch-2 passed

+1 :green_heart:
checkstyle
0m 15s
branch-2 passed

+1 :green_heart:
spotless
0m 52s
branch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 38s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 33s
the patch passed

+1 :green_heart:
compile
0m 34s
the patch passed

+1 :green_heart:
javac
0m 34s
the patch passed

+1 :green_heart:
checkstyle
0m 12s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
10m 32s
Patch does not cause any errors with Hadoop 2.10.2 or 3.3.6.

+1 :green_heart:
spotless
0m 51s
patch has no errors when running spotless:check.

+1 :green_heart:
spotbugs
0m 48s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 10s
The patch does not generate ASF License warnings.

25m  9s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5858

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti spotless checkstyle compile

uname
Linux 8a8ceeff9501 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / a456bfb142

Default Java
Eclipse Adoptium-11.0.17+8

Max. process+thread count
79 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/console

versions
git=2.34.1 maven=3.8.6 spotbugs=4.7.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 19s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m  5s
branch-2 passed

+1 :green_heart:
compile
0m 21s
branch-2 passed

+1 :green_heart:
shadedjars
5m 25s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 15s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 54s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
shadedjars
5m 43s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 13s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
16m 22s
hbase-mapreduce in the patch passed.

36m 15s

Subsystem
Report/Notes

Docker
ClientAPI=1.45 ServerAPI=1.45 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5858

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c7887d248f82 5.4.0-172-generic #190-Ubuntu SMP Fri Feb 2 23:24:22 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / a456bfb142

Default Java
Temurin-1.8.0_352-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/testReport/

Max. process+thread count
2982 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  6s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m  1s
branch-2 passed

+1 :green_heart:
compile
0m 39s
branch-2 passed

+1 :green_heart:
shadedjars
9m  0s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 24s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 24s
the patch passed

+1 :green_heart:
compile
0m 33s
the patch passed

+1 :green_heart:
javac
0m 33s
the patch passed

+1 :green_heart:
shadedjars
8m 53s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 24s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
21m 30s
hbase-mapreduce in the patch passed.

53m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/5858

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux c1b5a9fb59c6 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / a456bfb142

Default Java
Eclipse Adoptium-11.0.17+8

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/testReport/

Max. process+thread count
2853 (vs. ulimit of 30000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-5858/2/console

versions
git=2.34.1 maven=3.8.6

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fixes widget escaping

CC: @petele

Whoops!
There were 13 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/resources/widgets.md#L90 - '

Excellent. Thank you.

:tada: This has been pushed live to https://developers.google.com/web/


What is the suggested setup to run cla-assistant on our own instance?

We will have to host our own version. What would be the suggested way of achieving this to fulfill most of the following requirements:

Preferable running on AWS (we could check alternatives, so don't hesitate to suggest if not)
Minimal effort for setup/maintain/sync with the master project
Detailed instructions for setting up tools on the suggested platform
Backup support
Paid services are not an issue

In general there are not many requirements and AWS should be fine for hosting (we use a cloud foundry based provider for hosting/ might be less effort on your side). Apart from the application, you also need a mongoDB, which we are hosting on mLab (automatic backup, SSL, ..). In terms of the overall stack, the db would be the most important part as the CLAs are stored there. The app just connects to it.
Other than that, maybe you want to create a fork of the cla-assistant repo and set up your own deployment pipeline from there. We are using travis to build and test and then cf push the application. There is also an option to route log messages to a slack channel (in case you are using it).
Also it would be interesting to know why you need to run your own instance and if there is anything we could change on our side such that you would not need to do it anymore 😄 .

Thanks @thojansen!
We are "forced" to run our own instance because of the #141.. if you can change that part we can send a small thank you basket to your offices in Berlin :) It's right next to ours..

Ok, understood. Please let us check if we can change that.

Closing this issue now in favour of changing our TOS, which is tracked under #141. Still waiting for feedback from our colleagues.

🏇 🎆 🌾 🌴 👍


factor out nsenter, in favor of docker exec

Now that we have docker exec, nsenter seems like needless additional complexity.

nsenter is still there for backward compatibility with people who are on older versions of baseimage-docker, and on older versions of Docker. The entire docker-bash tool will be removed at some point, but that point is not now.

Also, the current docker-bash fails for me entirely.
$ ./tools/docker-bash e8
baseimage-docker-nsenter: cannot open /proc/6819/ns/ipc: Permission denied

Maybe Docker 1.4 has changed sufficiently that nsenter is no longer compatible.

nsenter has now been phased out as far as documentation is concerned. We still support it for backwards compatibility reasons.


Memory leak on IE11

On IE11 with polyfill.io, destroy function doesn't work, so memory is increasing on page navigation.
I currently using esm rollup version, and seams to compile watchers with iterator.
https://github.com/Akryum/vue-apollo/blob/master/dist/vue-apollo.esm.js#L3151
Is there anything I forget to get IE11 working.
I currently try to fix it or find a work around.

Fixed with an another polyfill.


is any pre-trained models will be released in the near future?

particularly the one that u used in evaluating in the paper, which has state-of-the-art performance?

It is not an official implementation


Muted video clips break.

If a video is muted from copyright issues or other reasons, download fails with the following:
C:\Users\Jim\AppData\Roaming\npm\node_modules\pully\lib\preset-manager.js:45
        Error: 500: No valid audio formats found!

Example Clip: https://www.youtube.com/watch?v=W7_4O0GovJ8
Error Message:

Does anyone have any issue with ignoring missing audio streams? In other words, the video will have no audio track.

This might have been fixed in the recent re-write. But I still need to find a video to test.


Conda build and cython problems

I am trying to create a conda package to distribute a python tool. Part of the tool is cythonized, and it works perfectly using python setup.py install.
I create the tar properly but when I try to install it, the package does not contain the .py files that links the python imports and the .so files.
So when I try to import that packages I get a module not found.
The only think I have found around cython and conda is to introduce cython requirement in the build/run section in the meta.yaml, but I don't know why those .py files are not included.
This is my meta.yaml
package:
  name: project
  version: 1.0.0
source:
  path: /home/user/project
requirements:
  build:
    - python >=2.7
    - jinja2
    - numpy
    - scipy
    - matplotlib
    - pysam
    - setuptools
    - h5py
    - cython
  run:
    - python >=2.7
    - jinja2
    - numpy
    - scipy
    - matplotlib
    - pysam >=0.8
    - setuptools
    - h5py
    - cython
build:
  preserve_egg_dir: True
  entry_points:
    - exec_file = project.run_exec:main
about:
  license: GPL3
  summary: "PROJECT"

my setup.py file looks like
from setuptools import setup, find_packages
from distutils.core import Extension
from Cython.Build import cythonize

extensions = [Extension('project.src.norm', ['project/src/norm.pyx'])]

setup(
    name="PROJECT",
    packages=find_packages(),
    version="1.0.0",
    description="PROJECT",
    author='Lab',
    author_email='email',
    url='http://',
    license='LICENSE.txt',
    include_package_data=True,
    entry_points={'console_scripts': ['exec_file = project.run_exec:main']},
    zip_safe=False,
    ext_modules=cythonize(extensions),
    classifiers=[
        'Development Status :: 4 - Beta',
        'Environment :: Console',
        'Intended Audience :: Bioinformaticians',
        'License :: OSI Approved :: BSD License',
        'Operating System :: MacOS',
        'Operating System :: Microsoft :: Windows',
        'Operating System :: POSIX',
        'Programming Language :: Python :: 2.7',
    ]
)

The directory structures is
project/
    setup.py
    __init__.py
    MANIFEST.in
    requirements.txt
    README.md
    info/
        meta.yaml
        build.sh
        bld.bat
    project/
         src/
            norm.pyx
        run_exec.py
    subproject/
        <etc...>

I also tried using python setup.py bdist_conda but the behavior is the same, or it is a conda issue or it is an specific problem on my configuration.
if that is the case I guess is is setup.py....

I'm not super familiar with conda packaging details, but lots of cython-using packages ship with conda so I would try to see what they do.


Deploy

[ ] Docker + distillery
[ ] Edeliver + distillery

References

edeliver
distillery

What are you thinking about having in this container? I think it shouldn't be easy to set up edeliver with docker, as there is nothing about it in the documentation, but I could set up a basic container image, what do you think?

@grufino I think two ways at the moment.

docker + distillery
edeliver + distillery

Hey @zven21,
we had a few talks on deployment in Elixir Taiwan, and one of them was by edeliver maintainer during Ruby X Elixir Conf Taiwan. The general conclusion around edeliver is that edeliver is really hard to debug, it is essentially a bunch of shell scripts. One of our member propose using Ansible to handle build and deployment commands, it is probably the most stable solution. You can find the details in this post:
https://www.cogini.com/blog/deploying-your-phoenix-app-to-digital-ocean-for-beginners/
Cheers

@yuchunc Thank you. :-)


your-account: Change "no changes made" error to success.

This changes the alert error "Error changing settings: No new data
supplied" to "No changes made" as a success bar.

Automated message from Dropbox CLA bot
@brockwhittaker, it looks like you've already signed the Dropbox CLA. Thanks!


Can't install using Nuget

Install-Package : Could not install package 'Nanoid 1.0.0'. You are trying to install this package into a project that targets '.NETFramework,Version=v4.5', but the package does not contain any assembly references or content
files that are compatible with that framework. For more information, contact the package author.

I tried also with different versions, no luck

It‘ll be support target multiple frameworks in version 2.0.

done


Add lower case script letters

Reason: I need to support all characters in https://www.w3.org/2003/entities which includes lower case script letters. As lower case script letters are seldom used, numerical references to Unicode codepoints would suffice, but: due to issue https://github.com/KaTeX/KaTeX/issues/2226 lower case script letters cannot be used this way. If that issue cannot be resolved, it would be nice to have the KaTeX fonts include lower case script letters.

...Actually, if lowercase script letters are from a different source than uppercase script letters, they would be different in style, which does not seem to be good idea. So maybe it is better to add lowercase script letters in the KaTeX fonts, even if https://github.com/KaTeX/KaTeX/issues/2226 is being resolved.

See the ongoing discussion on https://github.com/KaTeX/KaTeX/issues/2226 where the analogous problem for lowercase calligraphic letters is also mentioned.


Security vulnerabilities in many of Conductor dependencies

Just so you know, there are security vulnerabilities in many of the Conductor dependencies. Since our company blocks the download of any that have these problems, I have got to add overrides for all of these in order to allow the build to happen. Below is the block I put in all the build.gradle files. Not all overrides are required in all modules but it is easier to use one block for all of them. Some of them I can exclude because we are not running on AWS. I also have to delete all the dependencies.lock files.
dependencies {
	configurations.all {
		exclude group: 'amazon', module: 'aws-java-sdk'
		exclude group: 'com.thoughtworks.xstream', module: 'xstream'
		exclude group: 'io.netty', module: 'netty-resolver-dns'
		exclude group: 'com.squareup.okhttp', module: 'okhttp'
		resolutionStrategy {
	       	       force 'net.java.dev.jna:jna:+'
		       force 'com.fasterxml.jackson.core:jackson-databind:+'
	               force "com.fasterxml.jackson.core:jackson-core:+"
	               force "com.fasterxml.jackson.jaxrs:jackson-jaxrs-json-provider:+"
			force 'commons-beanutils:commons-beanutils:+'
			force 'commons-collections:commons-collections:+'
			force 'io.netty:netty-common:4.1.48.Final'
			force 'io.netty:netty-transport:4.1.48.Final'
			force 'io.netty:netty-codec:4.1.48.Final'
			force 'io.netty:netty-codec-http:4.1.53.Final'
			force 'io.netty:netty-codec-http2:4.1.48.Final'
			force 'xerces:xercesImpl:2.12.0'
			force 'log4j:log4j:1.2.17-atlassian-1'
			force 'org.apache.httpcomponents:httpclient-cache:+'
			force 'net.java.dev.jna:jna-platform:+'
			force 'com.amazonaws:aws-java-sdk-sqs:+'
			force "io.swagger:swagger-jaxrs:1.6.2"
		}
	}
}

We typically fix vulnerabilities pointed out by GitHub's Dependabot alerts. In this case, it seems like your company's rules are stricter or different. Since you are already running Conductor with these dependencies, it makes sense to upgrade them in the master branch so others could benefit as well. Please create a PR with these upgrades.

Unfortunately our environment is much different than Netflix as we are not using AWS or other components that you use. Testing would be impossible.


Add es6 template strings

Allows cool transformations:
$ echo '{"meal": "pizza"}\n{"meal": "taco"}' | jsonmap '`i love ${this.meal}`'
"i love pizza"
"i love taco"

Do the tests pass on node 0.12 with the template test?

No it doesn't, they are not available on node 0.12, not even with --harmony flag.

I think we should remove the test then (or somehow magically guard it) to avoid too much fragmentation

(sorry for nitpicking this)

No problem, I added a feature test for it.

Uh clever!

Released in 1.4.0


[Feature Request] Define operation and operationMetrics of commitInfo in the protocol

Feature request
Overview
The operation and operationMetrics fields of commitInfo have special meanings across implementations and should be defined in the spec.
Motivation
According to the protocol specification, the commitInfo action allows to store any valid JSON-formatted data. Yet the operation and operationMetrics fields have a special meaning. Implementations use the fields for storing the name of the operation, and metrics on operations, for instance the number of files added or removed, or the number of bytes added or removed. In some implementations the DESCRIBE HISTORY command will crash if they encounter unusual values in these fields.
Current support is:

Databricks and delta-io includes operation and operationMetrics for all commits, but might not populate the exact same fields(?).
delta-rs includes it for the optimize operations only
Delta Standalone has a suggested list of metrics to use, and for instance the Flink connectors uses these.
Trino currently does not include any operation metrics

The implementations already assume that the fields are not arbitrary JSON. For instance, the Databricks runtime will crash on DESCRIBE HISTORY if the operation field is not a string, and I believe delta-io would do the same.
create table some_table as select 1234 as a
# edit delta log to have commitInfo.operation be an array.
describe history some_table
Cannot deserialize value of type `java.lang.String` from Array value (token `JsonToken.START_ARRAY`)
 at [Source: UNKNOWN; line: 1, column: 54] (through reference chain: com.databricks.sql.transaction.tahoe.actions.SingleAction["commitInfo"]->com.databricks.sql.transaction.tahoe.actions.CommitInfo["operation"])

It would be good to define the meaning of the operation field, and of the most common fields included in operationMetrics, such that we get consistency across implementations.
The comments in the DeltaOperationMetrics would be a good starting point for defining operation and operationMetrics in the spec.
I'd be happy to help out writing a draft of this.
Willingness to contribute
The Delta Lake Community encourages new feature contributions. Would you or another member of your organization be willing to contribute an implementation of this feature?

[ ] Yes. I can contribute this feature independently.
[x] Yes. I would be willing to contribute this feature with guidance from the Delta Lake community.
[ ] No. I cannot contribute this feature at this time.

@ryan-johnson-databricks @bart-samwel any thoughts here?

The observations are accurate, and this is just one aspect of a broader problem, that it's actually kind of hard to parse arbitrary JSON. For example, there was a recent bug report https://github.com/delta-io/delta/issues/1548 because one implementation expected "[]", and crashed when another implementation emitted just [].
It's not easily fixed without a spec change and client upgrade, tho.
If somebody is manually parsing the json node by node and row by row, they can theoretically check the type and do smart things... but most implementations just pass a schema to the JSON parser and mismatches in column types (both vs. the read schema, or vs. other rows) throw parsing errors.
In general, the only way to safely allow "arbitrary" content is to define it as a string-string map, so that the structure is fixed and thus easily parsed. But no client today is emitting string-string maps for commitInfo.
Next best alternative is probably to grandfather in the current schema and require all future changes to go in a string-string map?


packit/srpm-build status pending after a failure

When packit/srpm-build fails status says pending. See the attached screenshot and the https://github.com/psss/edd/pull/10 pull request.

Please, wait for the next Monday to let the code move to production.

I'd say that we could redeploy on Wed given all the recent prod issues.


MSC3254: Add "knock" membership to /members filters

I assume it's an oversight that the filters of GET /_matrix/client/r0/rooms/{roomId}/members doesn't include "knock" as a possible membership state.
The response format lists knock as a possible value because it's referencing MemberEvent.

This does look like an oversight in the MSC, and may require another MSC to rectify. I don't think we can add it in the implementation just at the moment I'm afraid.


⚠ Bofrost.lu has degraded performance

In 1cbbf5b, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1043 ms

Resolved: Bofrost.lu performance has improved in ba588ae after 11 minutes.


image appears in directory, but not in davinci resolve

when I use the script/keybind, the image appears in the directory but resolve gives me an error

it says this even though the image IS IN FACT THERE in the directory. I can open the image, it's not corrupted. it just doesn't appear on the timeline or media pool.
when I paste, the image appears in the folder AFTER the error window pops up, I don't know if that has anything to do with it.
even if I change the directory to a different folder, the image still doesn't show up in the timeline, but the images do appear in the directory I chose.
console displays nothing as well
I can still put the image from the folder into resolve, but that's still an extra added step that this plugin is supposed to remove
https://github.com/user-attachments/assets/ce554ecc-9914-486d-b3bd-919dc6bbd5a0
EDIT: forgot to post an image of the text in the notepad

Im clueless on this on right off the bat
i made this a loong time ago and i never use this nowdays,
but everything you show seems 100% correct and there should not be any issues.
The mediapool is a bit wacky at times, for me switching bins and or not being in the clipboard bin does the trick.


Catching PNG image decoding error

Catch error when image decoding fails for PNG/JPG image due to corrupt image
Return undefined in that case and log error

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


chore(deps): upgrade configuration

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-configuration-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4616

Questions ?
Please refer to the Backport tool documentation


bug: Oil buffers cleanup and positioning of cursor

Did you check the docs and existing issues?

[X] I have read the docs
[X] I have searched the existing issues

Neovim version (nvim -v)
nvim -v NVIM v0.9.4 Build type: Debug LuaJIT 2.1.1692716794
Operating system/version
Ubuntu 22.04
Describe the bug
I have two keymaps to toggle oil buffer using oil API (keymaps and functions in the bottom):
require('oil').open(path)

It works. o will toggle oil buffer and open current directory of the buffer. Pressing o again will close oil buffer.
The problem is: if you use buffer cleanup, oil buffer will get cleaned up after 2 seconds. And toggling oil buffer again will not position cursor on last opened file.
If buffer cleanup is disabled than everything works but oil remain active for the whole session (although hidden, lua line shows them without name in bufferline).
Suggestion 1:
Leave everything as it is but add another open function in api that would have a second parameter filename. Filename can than be used to toggle oil buffer and position cursor to that file.
Suggestion 2:
Clean all oil buffers except that last one. That should work to.
Keymaps
vim.keymap.set('n', '<localleader>o', oil_toggle_curr_buf_path, { desc = '[O] Oil current buffer path' })
vim.keymap.set('n', '<localleader>O', oil_toggle_cwd, { desc = '[O] Oil cwd path' })

Functions
local oil_toggle = function (path)
  if vim.bo.filetype == 'oil' then
    -- Close current oil buffer
    require('oil').close()
  else
    -- Open oil buffer
    require('oil').open(path)
  end
end

-- Oil open path in current directory of the buffer
local oil_toggle_curr_buf_path = function ()
  oil_toggle(vim.fn.expand('%:h'))
end

-- Oil open path in current working directory
local oil_toggle_cwd = function ()
  oil_toggle('.')
end
---

### Steps To Reproduce

1. Add these functions and keymaps to neovim config:

# Keymaps

```lua
vim.keymap.set('n', '<localleader>o', oil_toggle_curr_buf_path, { desc = '[O] Oil current buffer path' })
vim.keymap.set('n', '<localleader>O', oil_toggle_cwd, { desc = '[O] Oil cwd path' })

Functions
local oil_toggle = function (path)
  if vim.bo.filetype == 'oil' then
    -- Close current oil buffer
    require('oil').close()
  else
    -- Open oil buffer
    require('oil').open(path)
  end
end

-- Oil open path in current directory of the buffer
local oil_toggle_curr_buf_path = function ()
  oil_toggle(vim.fn.expand('%:h'))
end

-- Oil open path in current working directory
local oil_toggle_cwd = function ()
  oil_toggle('.')
end
---

2. Open some file
3. Wait number of seconds defined in cleanup_delay_ms
4. Press <localleader>o
5. Cursor will not be positioned on current file from which oil was toggled. Unless it is the file in first row offcourse :-)

### Expected Behavior

Toggling oil should position the cursor on last opened file even if all oil buffers are cleaned up.

### Directory structure

_No response_

### Repro

```Lua
I did not check with clean config.
Also did not upload repro.lua.
Please close this bug if this is a problem.

Did you check the bug with a clean config?

[X] I have confirmed that the bug reproduces with nvim -u repro.lua using the repro.lua file above.

I believe the fix you want is even simpler: remove the vim.fn.expand('%:h') because that is the default behavior. When you call open() with no path, it opens the parent directory of the current buffer (plus handles some special buffers, like terminals and netrw). In addition, it will jump your cursor to the file that you were just in. If you pass in a path argument, you lose all of those bonus features.

Thank you !
That works.


[DOC] Update mr_proc documentation for DICOM->BIDS conversion

Changes proposed in this pull request:

Updated Miro diagrams
Updated text about DICOM organization and BIDS conversion

Checklist

[x] PR has an interpretable title with a prefix ([ENH], [BUG], [DOC], [INFRA], [MAINT])
[ ] PR links to Github issue with mention Closes #XXXX
[x] Tests pass
[ ] Code is properly formatted

@michellewang and @nikhil153 is this good to merge?

@surchs yep, forgot that it was already approved. Merging now


[Q] Intentionally Crashing (For Factorized Sweep Grid Search)

Hi there,
My question is this: from within a python script being run from within a wandb sweep, I'd like to be able to send a signal to the server saying that the program temporarily "crashed the agent", but continue to run the agent on different runs in this same sweep. The reason for this is kind of involved and I think not entirely necessary for you to understand, but if you'd like here it is.
I'm trying to create a factorized grid search.  For example, imagine I want to try to test the following grid:
batch_size: [10,15]
optimizer: ['adam', 'adamw', 'sgd']
weight_decay: [.3, .4]
lr: [0.0, 0.1, 0.2]

I understand how to turn that into a config file and run a wandb sweep over it.
But what if I want to run a grid search, and not use all elements of the grid – e.g. I don't care about sgd with weight decay 0 vs .1 vs .2 – I'm not using weight decay when I use sgd.  And likewise with the adam optimizers, I don't care about the learning rate – I've fixed it. Really, the grid then becomes something like this:
batch_size: [10,15]
subset1:
    optimizer: ['adam', 'adamw']
    weight_decay: [.3, .4]
subset2:
    optimizer: ['sgd']
    lr: [0.0, 0.1, 0.2]

I've written a script that will allow me to do this with wandb, but I'm stuck on one problem.  I can figure out whether the given run is one that I should run or should ignore, but I'm not sure what to do next.  Right now, I'm exiting with an error (assert False).  But the problem is that in big sweeps, then it takes time to figure out which programs actually exited with an error (which I'd want to see), and which were "fake" runs that immediately exited.
This is a convoluted way of doing it, I know.  It'd be best if wandb had a solution to this.  If so, would you mind telling me what it is?  If not, would you tell me how I can "crash" my program from within it.  If I can do that, then I can filter my visualizations based on crashed runs and see "true" failed runs. Or is there another workaround you'd recommend?
Thank you for making a great tool!

Hi @abwilf,
Interesting question! I think the tags feature of our runs could work here. If there was a run I would want to crash, I would follow the following pattern:
if to_crash(run.config):
    run.tags = ("intentional", ) # Rename intentional to whatever tag you prefer, this will let you filter such runs out.
    run.finish(exit_code = 1)    # Call finish to make sure tags get updated. exit_code=1 will show a status "Failed" on UI
    exit(1)                      # Exit out of the python process

Please let me know if you have any other questions, hope this helps!

Thank you, Ramit!  Worked like a charm.


Request to bump version of unzipper to eliminate license concerns

Unzipper itself has released latest version 0.11.2 which remove the dependency on buffers
Can exceljs upgrade to use latest version of unzipper, so that we can eliminate the license concerns?
Link related on unzipper:
https://www.npmjs.com/package/unzipper
https://github.com/ZJONSSON/node-unzipper/pull/283
Post related on license concerns:
https://github.com/exceljs/exceljs/issues/2703
https://github.com/exceljs/exceljs/pull/2687
https://github.com/exceljs/exceljs/issues/2686

Hi everyone,
I’ve released  a version with PR Request to bump version of unzipper to eliminate license concerns as a pre-release version: 4.4.1-prerelease.0.
If no issues are reported with this version, I plan to officially release it as 4.4.1 soon.


Stride Support (Replaces Hipchat)

Is there any plans to support Stride once HipChat is transitioned out?

I've been doing some work on a Stride alerter in this fork:
https://github.com/jness/elastalert
I'm waiting to pull request this into upstream, but first wanted to look at some unit test.


server side authorization fails when using oauth2 module

The traceback and steps to reproduce this bug are here: https://gist.github.com/bharadwaj6/e64e1e782f8883e8f42b

Thank you for reporting this issue and appreciate your patience. We've  notified the core team for an update on this issue. We're looking for a response within the next 30 days or the issue may be closed.


[flakey test] Networking VirtualMachineInstance with custom dns [test_id:1779] should have custom resolv.conf

Is this a BUG REPORT or FEATURE REQUEST?:
/kind bug
What happened:
[rfe_id:694][crit:medium][vendor:cnv-qe@redhat.com][level:component]Networking VirtualMachineInstance with custom dns [test_id:1779]should have custom resolv.conf

/root/go/src/kubevirt.io/kubevirt/tests/vmi_networking_test.go:666
 Unexpected error:
     <expect.TimeoutError>:<PHONE_NUMBER>0
     expect: timer expired after 15 seconds
 occurred
 /root/go/src/kubevirt.io/kubevirt/tests/vmi_networking_test.go:691

What you expected to happen:
The k8s-cnao-1.17 lane often fails with the above error

/assign @AlonaKaplan

links to failing jobs -
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3550/pull-kubevirt-e2e-kind-k8s-1.17.0-ipv6/1274404687931510784
https://prow.apps.ovirt.org/view/gcs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/3406/pull-kubevirt-e2e-k8s-1.18/1274979489599721472

				&expect.BExp{R: "nameserver <IP_ADDRESS>"},
				&expect.BExp{R: "\\$"},

i would suggest to add sending \n between those two, because i see in the logs the first one "steals" the prompt when the race occurs.


Render preschools/kindergartens distinctly from schools

Preschools/kindergartens (amenity=kindergarten) are tagged differently than schools, and they play different roles in the community fabric. They're often smaller (many in my area are converted houses or storefronts), and therefore both smaller as destinations themselves and less well-known/important for wayfinding.
As of #689, preschools are rendered the same as all other schools, including universities, which may prove confusing. It would make sense to render them similarly to schools, but a distinct icon could also work. One idea could be to put a "P" inside the school building, similarly to how some of the map samples in #78 differentiate elementary/middle/high schools. I think it would also make sense for them to appear later than schools do (currently z15), perhaps at the same zoom level as doctors offices and parking lots (currently z17), since they're more distinctly local entities rather than well-known landmarks as schools often are.

Just want to note that there is also amenity=childcare. Not sure if merging symbols would be good or bad. Both are focused on little children and at least in my area many facilities to the full age range from infant to preschool.

I think there could be an argument for combining them, I also think the distinction can be somewhat amorphous.


Is get_units() function for depth_frame class not implemented in MATLAB?

Is get_units() function for depth_frame class not implemented in MATLAB? I plan to use this to convert the depth data in terms of meters. Any other way of doing it apart from a computationally expensive loop?

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
<Describe your issue / question / feature request / etc..>

Thanks for your info.
Please have a try the PR https://github.com/IntelRealSense/librealsense/pull/6318 .

Hi.
Will you be needing further help with this? If we don’t hear from you in 7 days, this issue will be closed.
Thanks

Hi
Thanks for your response.
I have been trying to compile the code but I can't get it to work.
I'm new to using Matlab and I'm not sure how packages work.
I did not go through the installation using source code the first time. I
just used the installer.
I tried the tutorials in the github page but to no avail.
Would you be able to help me?
Best regards
Dhong
On Tue, 12 May 2020, 3:11 am RealSenseSupport<EMAIL_ADDRESS>wrote:

Hi.
Will you be needing further help with this? If we don’t hear from you in 7
days, this issue will be closed.
Thanks
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6252#issuecomment-626849476,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2N5JDB6ZLOHRC3VEJLFTDRRA2FDANCNFSM4MJNCKXA
.

@kidhong2 Please refer to the readme document in the link below.
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/matlab

@kidhong2 Any other questions about this? Looking forward to your update. thanks!

Hi
still trying to run compile it.
Can't get through yet.
Sorry. Will let you know when I get it done.
I'm new to using cmake and matlab so there's a learning curve there.
Dhong
On Tue, May 26, 2020 at 4:09 PM RealSense Customer Support <
<EMAIL_ADDRESS>wrote:

@kidhong2 https://github.com/kidhong2 Any other questions about this?
Looking forward to your update. thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6252#issuecomment-633835806,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2N5JGEUHKCI4532QZJQO3RTNPY3ANCNFSM4MJNCKXA
.

@kidhong2 Is there any progress from your side? Thanks!

@kidhong2 https://github.com/IntelRealSense/librealsense/pull/6318 was already merged into the latest librealsense v2.35. You can also have a try with the latest release. Thanks!

Hi
Thank you very much.
This is appreciated.
Dhong
On Wed, Jun 3, 2020 at 3:11 PM RealSense Customer Support <
<EMAIL_ADDRESS>wrote:

@kidhong2 https://github.com/kidhong2 #6318
https://github.com/IntelRealSense/librealsense/pull/6318 was already
merged into the latest librealsense v2.35. You can also have a try with the
latest release. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6252#issuecomment-637966853,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2N5JDWBO4T4U7U3NKWDPLRUXPB7ANCNFSM4MJNCKXA
.

@kidhong2 Any update from your side? Thanks!

@kidhong2 Any chance to try now? If you still can't get chance to try in these two weeks, can we close it at this point? Later if you still get the issue, please feel free to create another one. Thanks!

Hi
Sorry. didn't get the chance to try it.
Please close the ticket.
I will try it sometime in the future and will create another one should I
need to.
Best regards
Dhong
On Thu, Jul 2, 2020 at 2:58 PM RealSenseSupport<EMAIL_ADDRESS>wrote:

@kidhong2 https://github.com/kidhong2 Any chance to try now? If you
still can't get chance to try in these two weeks, can we close it at this
point? Later if you still get the issue, please feel free to create another
one. Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IntelRealSense/librealsense/issues/6252#issuecomment-652791008,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI2N5JDCQTVDLOMTUZR5PPDRZQLGNANCNFSM4MJNCKXA
.

@kidhong2  Thanks for the reply! Close this accordingly.


Can't edit in Rapid from MapRoulette

Description
Can't edit in Rapid from MapRoulette, this is the error I get.
Your connection isn't private
Attackers might be trying to steal your information from mapwith.ai (for example, passwords, messages or credit cards).
NET::ERR_CERT_DATE_INVALID

Screenshots

Version
2.1.1
What browser are you seeing the problem on? What version are you running?
No response
The OS you're using
No response
Steps to reproduce
Go to https://maproulette.org/ and pick a challenge
On the task page change Current Editor to Edit in RapiD
Click Edit
New window will open with the error message
The browser URL at the time you encountered the bug
https://mapwith.ai/rapid#id=w905767344&comment=Fixed building %26 highway intersections %23maproulette https://maproulette.org/browse/challenges/40063&source=&background=EsriWorldImagery&map=19/-35.305809787719824/149.10517662763598
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
No response

It looks like MapRoulette hasn't updated the URL for Rapid. I've opened a PR to fix this

This was due to a bad site cert- since the cert expired, our redirect to rapideditor.org wasn't working. Should be fixed now!


Reorganize toc

move "docker desktop" to the top of the "product manuals" section,
and make it the default page
Add a "Reference documentation" menu item to the TOC, which will be
the "active" menu item when opening the reference section.
move the "Samples home" menu item outside of the "samples" sub-menu,
and add it at the top level as "samples overview", so that the "index"
highlights that menu-item without opening the "samples" sub-menu. The
"samples" submenu contains various samples which are a bit outdated,
so probably shouldn't be that prominent when opening the page.
Moving the menu item also makes the "breadcrumb" navigation more
natural (moving from a sample to "samples overview" navigates back
to the "top" of the samples section.

With this change:

@thaJeztah Thanks for the update. Looks good. The Product manuals card on the homepage still opens the Engine docs.

the Product manuals card on the homepage still opens the Engine docs.

Oh, good catch; perhaps we probably used a "deep link" for that one as we don't have a /manuals/ url or redirect yet. Let me fix that link

updated 👍


Make use of Hugo's 0.56+ module feature.

Need to make sure we don't have the dist folder copied locally before testing this.
Fixes #29142

Maybe this requires go to be available, in which case we can't proceed with this. Adding a new devDependency isn't so good...

@bep: I read the docs, but I'm probably missing something. Is this how modules are supposed to work or do we need Go installed for this too?

This commit works with the unreleased (and unmerged) Hugo version: https://github.com/bep/bootstrap/commit/154417df22aa442b67e593aab0cf301a7f4cfc41

Nice! Can't wait to try it when the next version is out :)
BTW, just thinking out loud for the sake of flexibility. Wouldn't it be more flexible if one didn't have to add all mount points, but only the ones they want to use? I know our use case might not be the most common one, but still. Maybe I'm missing something, I'll play with it when the new version is out :)

didn't have to add all mount points, but only the ones they want to use? I

You only have to add the mounts you use. There are some superfluous mounts there now. (archetypes? i18n?), but I didn't know the project good enough to remove them.

Perfect, thanks! I'll play around with the branch when 0.56.3 is out and let you know if we hit any more issues.

@bep: I just gave this a go and works pretty good. Thanks for the fast fixes!
The only thing I noticed when playing around with SRI generation (I dropped the patch from this branch now though), and I had set another mount to assets/dist in order for me to use resources.Get, then we had another dist folder in publishDir/dist.
I'll make a branch for this later and see if we have any issues there.

@bep: can you checkout this branch and do npm run docs-serve in one cmd and npm run watch-css-main in another? Hugo complains it can't find the file. I'm on Windows but @MartijnCuppens is on macOS and happens there too. Maybe the changes are too many or happen too fast and Hugo doesn't catch them?
If I change an HTML or markdown file and hugo rebuilds the site, then the error goes away.

Confirming the npm scripts run for me, but at localhost:9001 I see file does not exist errors.

No file does not exist errors here. The docs css and js just don't appear in the dom, this a screenshot from the homepage where you should see the Bootstrap css:

Styles on the Netlify preview:

I definitely don't have the issue @MartijnCuppens mentions. I do have the same issue @mdo says above with watch. This doesn't happen for me on Windows without the mount changes.

In fact, I'm able to trigger the file not found error if I modify dist/css/bootstrap.css manually:
C:\Users\xmr\Desktop\bootstrap>npm run docs-serve

><EMAIL_ADDRESS>docs-serve C:\Users\xmr\Desktop\bootstrap
> hugo server --port 9001 --disableFastRender

Building sites …
                   | EN
+------------------+-----+
  Pages            | 105
  Paginator pages  |   0
  Non-page files   |  20
  Static files     | 125
  Processed images |   0
  Aliases          |  16
  Sitemaps         |   1
  Cleaned          |   0

Total in 792 ms
Watching for changes in C:\Users\xmr\Desktop\bootstrap\{dist,site}
Watching for config changes in C:\Users\xmr\Desktop\bootstrap\config.yml
Environment: "development"
Serving pages from memory
Web Server is available at http://localhost:9001/ (bind address <IP_ADDRESS>)
Press Ctrl+C to stop

Change of Static files detected, rebuilding site.
2019-08-02 12:55:25.701 +0300
Syncing \css\bootstrap.css to C:\Users\xmr\Desktop\bootstrap\
ERROR 2019/08/02 12:55:25 file does not exist
ERROR 2019/08/02 12:55:25 file does not exist

I think now we are ready to merge this. I tested it on Windows, no errors when watching.
Huge thanks to @bep for working towards fixing the bugs we found with mounts!

I need an approval @mdo @Johann-S @MartijnCuppens after you guys try this (make sure you do npm i and remove the old docs/4.3/dist folder locally)


Add Support for Undoable Edit Events

Adding an issue to add an event source corresponding the UndoableEditListener.
An UndoableEditListener can be added to the same components as a DocumentListener.


Google Play Music: "Add Album to Queue" and "Play Album Next" don't work in Chromium build

Blocked by tiliado/valacef#11
Hello, I'm testing the Chromium backend. Version 6, on Nuvola Apps 4.9.96.
I'm looking at my Recently Played page.  I click on an album and select from the "dot dot dot" menu the option "Play Album Next" or "Add Album to Queue". Then I look in the queue in the lower right and it's empty.
To confirm this is a bug, I repeated the same steps in Firefox and confirmed albums were added to the queue.
I tried again with the DevTools console open. I saw no errors in the "Console" tab, and no failed requests in the "Network" tab.
These features also works in the WebKit version.

Nuvola will change its rendering engine again (#583). The good news is that it will be implemented via offscreen rendering, which could fix this issue. Keep fingers crossed.

Google Play Music is closing... #660


Style highlight of CSS

CSS codeblocks should be prettier

The official Pug website uses https://codemirror.net/ to show the code snippets.
Markdown-it has  aplugin to use code mirror: https://github.com/TitanSnow/markdown-it-v-codemirror-highlighter I might has well use that


[Runtime] Temporarily update attribute ordering

clang-15 requires extern "C" to be the first attribute and does not allow mixing GNU attributes and standard attributes. Temporarily split and re-order attributes to prevent compilation errors. Avoid updating the public stdlib here (Visibility.h) and instead add the definitions to Config.h, which all impacted headers and up including.
This should be removed once Swift uses at least stable/20221013, which has https://reviews.llvm.org/D137979 and would thus allow us to just move SWIFT_RUNTIME_EXPORT to be the first attribute.
Resolves https://github.com/apple/swift/issues/61468.

@swift-ci please test

https://github.com/apple/swift/pull/62497 doesn't work on main unfortunately. I'll rebase it onto rebranch after fixing the conflicts this PR will cause.

@eeckstein this will have rebranch merge conflicts from this I imagine. I'll fix those + put up the separate fix tomorrow.


feat: add is_between for dask

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

[ ] 💾 Refactor
[x] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #637

Checklist

[x] Code follows style guide (ruff)
[x] Tests added
[ ] Documented the changes

If you have comments or can explain your changes, please do so below.

I will reopen after I'll add a parameterized test for closed arg


Add loadBalancerIP and externalTrafficPolicy support for service

Pull Request
What does this PR do?
Fixes #112 : Add service.loadBalancerIP and service.externalTrafficPolicy optional chart values to configure meilisearch kubernetes service with custom IP address from IaaS.

Thanks for reviewing @alallema. Here are some updates to my original contribution. Please tell me if anything doesn't comply with your recommendations.


Updated course contents not shown

It seems like that edx android app does not refresh itself when course contents is added or updated from the studio (New sections, subsections for example). To force refreshing, the app is needed to be unloaded from task manager or the user enters another course and reenter the updated course. When the app is resides on memory and the user is clicking one course only, there is no way to get the updated contents.
The IOS app shows the updated contents correctly within 1 hour.
(In the manual, the contents in the app is refreshed within 1 hour after the course contents update.)

See #922 for more info on this issue. Closing it in favor of the same issue. Re-open if you think your case is different.