Allow to set default catalog and default schema when transform_sql

Description
Currently, wren-core-rs recognizes a qualified table name to generate the corresponding model. The user needs to provide the full name of the model in the query, such as:
SELECT * FROM wren.public.orders

It would be more convenient if the user could simply query with the table name:
SELECT * FROM orders

Solution
We can modify the transform_sql API to apply the catalog and schema of MDL to the DataFusion SessionContext.

It is also related to the V3 validation API.
https://github.com/Canner/wren-engine/blob/8449442a54a7953dee694bf5526f35448da23273/ibis-server/tests/routers/v3/connector/test_postgres.py#L311-L312

@goldmedal Can you assign it to me?

Thanks @himanshu634. If you have any questions about it, feel free to ask me here or leave message.

Hey @goldmedal , I want to understand the project. What module does what work?

Hey @goldmedal , I want to understand the project. What module does what work?
Here's a grammatically corrected version:

Hi @himanshu634, sorry for the delayed response; I had some personal matters to attend to.
This project is transitioning to a Rust implementation. There are many legacy Java modules you can ignore. You only need to focus on the following three modules:

ibis-server: This is a server responsible for data source connections (e.g., Postgres, BigQuery, etc.).
wren-modeling-rs: This Rust module handles data modeling and can translate a semantic SQL based on an MDL (provided by WrenAI web) into a real SQL. For more information about modeling concepts, you can check our documentation.
wren-modeling-py: This is a Python API for wren-modeling-rs. The ibis-server uses it to invoke wren-modeling-rs. You can refer to https://github.com/Canner/wren-engine/pull/710 to see how it is used.

I hope this information is helpful to you.

Hey @goldmedal , I want to understand the project. What module does what work?
Here's a grammatically corrected version:

Hi @himanshu634, sorry for the delayed response; I had some personal matters to attend to.
This project is transitioning to a Rust implementation. There are many legacy Java modules you can ignore. You only need to focus on the following three modules:

ibis-server: This is a server responsible for data source connections (e.g., Postgres, BigQuery, etc.).
wren-modeling-rs: This Rust module handles data modeling and can translate a semantic SQL based on an MDL (provided by WrenAI web) into a real SQL. For more information about modeling concepts, you can check our documentation.
wren-modeling-py: This is a Python API for wren-modeling-rs. The ibis-server uses it to invoke wren-modeling-rs. You can refer to Provide the V3 API of Ibis server for the wren-core rust beta version #710 to see how it is used.

I hope this information is helpful to you.

Thanks.

Hi @himanshu634
Due to our schedule, I finished this work first. I found it may not be so simple for someone unfamiliar with our project and Rust. Maybe it's not a good first issue I think 🤔
Anyway, thank you for trying to contribute to wren-engine.


Cannot parse with Date Format EEE MMM DD hh:mm:ss Z yyyy

I am trying to parse the following date Thu Dec 24 10:48:19 +0000 2020 with this DateFormat("EEE MMM DD hh:mm:ss Z yyyy") but I am getting FormatException (FormatException: Trying to read   from Thu Dec 24 10:48:19 +0000 2020 at position 21)
Is my pattern wrong?
Would Appreciate any help.

I believe it's related to an issue I just filed here: https://github.com/dart-lang/intl/issues/368
EEE now for some reason expects a period at the end of the weekday string.

First of all, I suspect you mean d and not D for day of month instead of day of year.
Unfortunately, time zones and time zone offsets are not yet implemented in this package, which is why Z fails to work for the parser.
Closing this in favor of #19 .


Context switching prevents external keyboard input in Terminal (iPadOS Safari)

Not specific to any extension
VS Code Version: 1.64.2 (GitHub Codespaces)
OS Version: iPadOS 15.3.1

Steps to Reproduce:

Open any GitHub repo in GitHub’s VScode Web Editor (pressing . key on any repo)
Create a Codespace or connect to a Codespace (since Terminal is only accessible within a Codespace/Devcontainer)
Type text in terminal (Command + J to focus it) and it should print as expected
Switch focus to another app with Command + Tab
Switch back to VScode/Codespaces, at which point input is no longer possible without first disconnecting and reconnecting the external keyboard (or perhaps triggering the on-screen keyboard some other way) or triggering input with tap events.

Video reproduction:
https://user-images.githubusercontent.com/65950/156302760-97139ece-5356-4f0a-b965-858e845704b8.MOV
Expected behavior:
If the keyboard focus is on the terminal the active input device should regain focus when OS context switches back to VSCode. This very well could be an iPadOS Safari bug, but if so it seemed important to document it here since I couldn’t find a specific issue covering this unexpected and disruptive behavior.

@rebornix this seems like an ipadOS problem. Thoughts?

@rebornix Is there anything I can do to help debug this?

Merging #151626 here. Some extra info from there:

Return key doesn’t work neither in terminal nor in text editing.
Reloading the page doesn’t help, BUT unplugging and plugging back the keyboard (just slightly pull iPad off the case)
Closing the application also doesn’t help.
Switching to the other app and back - doesn’t help (in another app return key works)

fyi @joaomoreno

/duplicate https://github.com/microsoft/vscode/issues/163448

Help wanted here, to make it clear the team will likely not fix this as those that contribute to the terminal don't have ipads.

This makes vs code tunnels almost unusable! It seems to affect arrow keys and escape as well as enter.
Normal characters as well as control, option, command, and the emoji meta keys all work. Sometimes I have to fiddle with focus until those buttons work: I can't, however, restore the other meta keys without physically unplugging the keyboard.

On-screen keyboard works fine, but not for physical attached one (at least on the offcial Apple one)

There is a workaround I found here:
https://github.com/coder/code-server/discussions/5221#discussioncomment-9629446


20240929.1.0 Windows runner image breaks Python virtualenv

Description
After the update of the Windows runner image to 20240929.1.0 my builds break with
ERROR Failed to create venv. Maybe try installing virtualenv.

The only change since the last successful build is the runner image update.
Platforms affected

[ ] Azure DevOps
[X] GitHub Actions - Standard Runners
[ ] GitHub Actions - Larger Runners

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] Ubuntu 24.04
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] macOS 15
[ ] macOS 15 Arm64
[ ] Windows Server 2019
[X] Windows Server 2022

Image version and build link
Image version: 20240929.1.0
Failed build: https://github.com/KhronosGroup/KTX-Software/actions/runs/11174988721/job/31065570054
Is it regression?
Yes. Latest working image is 20240915.1.0. Latest successful build is https://github.com/KhronosGroup/KTX-Software/actions/runs/11028019847/job/30627418606
Expected behavior
I expected my Python builds to still work.
Actual behavior
The python builds fail with an error that look like virtualenv was removed from the image. However the change logs for 20240929.1.0 show no changes to any Python components.
Repro steps
The easiest way to repro is just to run a build of https://github.com/KhronosGroup/KTX-Software/actions/runs/11174988721/job/31065570054.
More detailed steps.

Set up a Python project with a requirements.txt listing required python packages. I'm using CMake to my Python stuff as part of a larger project but this probably isn't a necessary condition.
As part of the build process for the project do:python3 -m pip install --no-warn-script-location -r ${SOURCE_DIR}/requirements.txt

Set up CI with a step that installs Python:name: Set up Python 3.11
  if: matrix.options.py == 'ON' && matrix.check_mkvk != 'ONLY'
  uses: actions/setup-python@v5
  with:
    python-version: '3.11.4'

Add a build step that does:run: |
  cmake . -B build
  cmake --build build

I am relatively new to Python and did not write the Python parts of our project. Therefore I am not sure what determines that virtualenv should be used when using a Windows runner image. Consequently I am not sure if the error is

because something has changed to trigger use of virtualenv and it never existed and doesn't exist in the image
or because virtualenv was removed from the latest image.

I am surprised, judging from the lack of an existing bug report, that nobody else seems to have encountered this problem.
I have tried to install virtualenv using pipx install virtualenv in a step after the step that installs Python. The installation appears to be successful but I still get the same error.

Hi @MarkCallow , thank you for bringing this issue. We will update you once validated. Thanks .

I used pip instead of pipx to install virtualenv: pip install virtualenv. It works. So it looks like virtualenv was removed from the image and the removal escaped the generator of the change list.

Hi @MarkCallow , is the issue resolved ..?

is the issue resolved

Only in that I've got my build working again. The runner image is broken. A previously supported featured was removed. Given the lack of info in the change list it does not look intentional. If I'm the only one complaining you may not care but I think virtualenv is widely used by Python projects hence my surprise at the lack of other bug reports.

Hi @MarkCallow , thanks for the confirmation , we try to gather the necessary requests from customer , based on the number of requests will consider including in images.
As in  moment we are trying to make the images as light as possible .thanks .closing this issue.

Hi @MarkCallow , thanks for the confirmation , we try to gather the necessary requests from customer , based on the number of requests will consider including in images.

I think you have that backwards. As virtualenv was included in the images previously but has now been removed there will not previously have been any requests to add it.

As in moment we are trying to make the images as light as possible .thanks .closing this issue.

At most this is going to save some kbytes on your image storage and shave a few milliseconds off booting a virtual machine.
There is nothing in my python project that explicitly requests use of a virtualenv. It is the python build module that is requesting it for both of the following commands executed while building my target:
${PYTHON_EXECUTABLE} -m build --sdist --outdir ${DIST_DIR}

${PYTHON_EXECUTABLE} -m build --wheel --outdir ${DIST_DIR}

I suspect both sdist and wheel are widely used in Python projects and therefore need to be built and will thus now encounter the absence of virtualenv.
Please reopen this issue.

I used pip instead of pipx to install virtualenv: pip install virtualenv. It works. So it looks like virtualenv was removed from the image and the removal escaped the generator of the change list.


Update stytch-kotlin naming to stytch-android

Linear Ticket: SDK-1051
Changes:

Update all references to stytch-kotlin to stytch-android

Notes:

Checklist:

[x] I have verified that this change works in the relevant demo app, or N/A
[x] I have added or updated any tests relevant to this change, or N/A
[x] I have updated any relevant README files for this change, or N/A

Ignore these snyk checks. They're looking for an old file from the last time this repo was called stytch-android. We'll get it updated.


Add multiple columns feature, columns property

Fixes #5

This change is

Review status: 0 of 27 files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

demo/index.html, line 36 at r1 (raw file):

      <template>
        <vaadin-form-layout columns="1">
          <vaadin-text-field label="First Name" value="Jane"></vaadin-text-field>

I'm a bit confused, didn't was discussed that any form-layout children should be wrapped by a form item?

Comments from Reviewable

Review status: 0 of 27 files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

vaadin-form-layout.html, line 25 at r1 (raw file):

### Spanning Items on Multiple Columns

You can use `vaadin-form-layout-colspan` attribute on the layout items.

In case we're going to support adding any element as direct child of <vaadin-form-layout> (as stated above), a non-standard attribute vaadin-form-layout-colspan is illegal.
Could we limit the support of column spanning to only <vaadin-form-item>'s? Then we could also name the property to much nicer colspan.

vaadin-form-layout.html, line 72 at r1 (raw file):

      }

      :host #layout {

Could we drop :host from the selectors?

vaadin-form-layout.html, line 144 at r1 (raw file):

      }

      _naturalNumberOrOne(n) {

Do we really need to sanitize the value? The type of columns is Number so that should be enough. If a developer types something else than a number, that's his error.

vaadin-form-layout.html, line 162 at r1 (raw file):

       * Update the item widths, column gaps and other styles.
       */
      updateStyles(...args) {

updateStyles handles CSS shims, right? Can we be sure this will get called in the future versions of P2 also?

vaadin-form-layout.html, line 189 at r1 (raw file):

          colspan = Math.min(colspan, this.columns);

          const childRatio = colspan * oneColumnRatio;

const childRatio = colspan / this.columns; does the same, no need for oneColumnRatio

Comments from Reviewable

Review status: 0 of 27 files reviewed at latest revision, 6 unresolved discussions, some commit checks failed.

vaadin-form-layout.html, line 25 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
In case we're going to support adding any element as direct child of <vaadin-form-layout> (as stated above), a non-standard attribute vaadin-form-layout-colspan is illegal.
Could we limit the support of column spanning to only <vaadin-form-item>'s? Then we could also name the property to much nicer colspan.

This is valid HTML, using colspan is allowed on custom elements:
<!DOCTYPE html>
<head><title>Test</title>
<body>
<vaadin-form-layout>
  <vaadin-text-field colspan="1"></vaadin-text-field>
  <paper-input colspan="1"></paper-input>
</vaadin-form-layout>

Maybe limit support of column spanning to any custom elements?

vaadin-form-layout.html, line 144 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Do we really need to sanitize the value? The type of columns is Number so that should be enough. If a developer types something else than a number, that's his error.

Infinity/-Infinity/NaN, negative, fractional numbers and 0 are also Number, but only natural numbers are valid here.
Having an invalid colspan or whatever value does not break the layout in CSS or HTML. Maybe we shouldn’t follow that strictly, but I would like to.

vaadin-form-layout.html, line 162 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
updateStyles handles CSS shims, right? Can we be sure this will get called in the future versions of P2 also?

Right.
It’s a Polymer.Element base class API, not deprecated: https://www.polymer-project.org/2.0/docs/api/elements/Polymer.Element#method-updateStyles
This is a user-targeted API, Polymer does not call it itself.

demo/index.html, line 36 at r1 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
I'm a bit confused, didn't was discussed that any form-layout children should be wrapped by a form item?

Wrapping is not a requirement for having columns. That is why in this PR i haven’t even introduced the form item element yet.
It will come later for having labels on their column.

Comments from Reviewable

Review status: 0 of 27 files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

vaadin-form-layout.html, line 25 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
This is valid HTML, using colspan is allowed on custom elements:
<!DOCTYPE html>
<head><title>Test</title>
<body>
<vaadin-form-layout>
  <vaadin-text-field colspan="1"></vaadin-text-field>
  <paper-input colspan="1"></paper-input>
</vaadin-form-layout>

Maybe limit support of column spanning to any custom elements?

A bit strange limitation IMO. With <vaadin-form-item> it wouldn't need to be an attribute either but a normal property would do just fine.

vaadin-form-layout.html, line 144 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
Infinity/-Infinity/NaN, negative, fractional numbers and 0 are also Number, but only natural numbers are valid here.
Having an invalid colspan or whatever value does not break the layout in CSS or HTML. Maybe we shouldn’t follow that strictly, but I would like to.

This introduces a new practice we haven't had before but ok. It's still a bit odd we sanitize one user input (colspan) but not the other (column gap)

Comments from Reviewable

Review status: 0 of 27 files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

vaadin-form-layout.html, line 25 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
A bit strange limitation IMO. With <vaadin-form-item> it wouldn't need to be an attribute either but a normal property would do just fine.

Makes sense. Although, allowing developers to use colspan on any elements, if non-standard attributes are fine for them, also makes sense to me.
I wouldn’t like to limit spanning only on form items, though. This requirement comes purely from using an attribute, which comes from lack of native custom CSS properties. :-(
I think, it’s OK for the layout element to define styles on content attributes. For ex., same approach used there: https://github.com/PolymerElements/app-layout/blob/2.0-preview/app-toolbar/app-toolbar.html

vaadin-form-layout.html, line 144 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
This introduces a new practice we haven't had before but ok. It's still a bit odd we sanitize one user input (colspan) but not the other (column gap)

column gap is a CSS property. We don’t parse it here, just get the value and use directly in CSS calc. CSS does parsing and type checks in this case.

Comments from Reviewable

vaadin-form-layout.html, line 25 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
Makes sense. Although, allowing developers to use colspan on any elements, if non-standard attributes are fine for them, also makes sense to me.
I wouldn’t like to limit spanning only on form items, though. This requirement comes purely from using an attribute, which comes from lack of native custom CSS properties. :-(
I think, it’s OK for the layout element to define styles on content attributes. For ex., same approach used there: https://github.com/PolymerElements/app-layout/blob/2.0-preview/app-toolbar/app-toolbar.html

Ok, fair enough

Comments from Reviewable

vaadin-form-layout.html, line 144 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
column gap is a CSS property. We don’t parse it here, just get the value and use directly in CSS calc. CSS does parsing and type checks in this case.

You can break the layout by providing an "invalid" value for "column gap" so the CSS parsing isn't really as proof. I'm ok with this if you really want to keep it here.

Comments from Reviewable

Review status: 0 of 27 files reviewed at latest revision, 3 unresolved discussions.

vaadin-form-layout.html, line 72 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
Could we drop :host from the selectors?

Done.

vaadin-form-layout.html, line 189 at r1 (raw file):
Previously, tomivirkki (Tomi Virkki) wrote…
const childRatio = colspan / this.columns; does the same, no need for oneColumnRatio

Done.

Comments from Reviewable

Reviewed 17 of 27 files at r1, 10 of 10 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

Sorry, couldn't find anything else wrong but a typo :man_shrugging:

Reviewed 17 of 27 files at r1, 10 of 10 files at r3.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

vaadin-form-layout.html, line 18 at r3 (raw file):

Supports usign any child elements as layout items.

"using"

Comments from Reviewable

Review status: all files reviewed at latest revision, 3 unresolved discussions.

vaadin-form-layout.html, line 118 at r3 (raw file):

  <script>
    if (!window.Vaadin) window.Vaadin = {};

this line is no needed anymore

Comments from Reviewable

Review status: 26 of 27 files reviewed at latest revision, 13 unresolved discussions.

vaadin-form-layout.html, line 18 at r3 (raw file):

Supports usign any child elements as layout items.

It supports any child element as layout items.

vaadin-form-layout.html, line 20 at r3 (raw file):

Supports usign any child elements as layout items.

By default, makes a two-column layout. The number of columns is customizable

it makes

vaadin-form-layout.html, line 25 at r3 (raw file):

### Spanning Items on Multiple Columns

You can use `vaadin-form-layout-colspan` attribute on the layout items.

layout items does not sound fine, I would use child items or just items

vaadin-form-layout.html, line 36 at r3 (raw file):

### Explicit New Row

Use `<br>` line break element to wrap the layout items on the new row:

Use the ....   ... on a new row

vaadin-form-layout.html, line 56 at r3 (raw file):

@polymerElement vaadin-form-layout
@extends Polymer.Element

Is it necessary? any P2 element extends it by definition

vaadin-form-layout.html, line 147 at r3 (raw file):

        if (typeof n === 'number' && n >= 1 && n < Infinity && n % 1 === 0) return n;
        return 1;
      }

why 2.1 returns 1, shouldn't be better to return 2 ?

demo/common.html, line 25 at r3 (raw file):

<script>
  window.addEventListener('WebComponentsReady', function() {
    window.webComponentsAreReady = true;

Could you add a comment here, that it's used in gemini tests?

demo/index.html, line 36 at r1 (raw file):
Previously, platosha (Anton Platonov) wrote…
Wrapping is not a requirement for having columns. That is why in this PR i haven’t even introduced the form item element yet.
It will come later for having labels on their column.

Ok for not introducing now, but I think we should only support one way when adding the wrapper for avoiding confusion

demo/index.html, line 87 at r3 (raw file):

          addEventListener('WebComponentsReady', function() {
            var layout = document.querySelector('vaadin-form-layout.custom-gap');
            layout.updateStyles({'--vaadin-form-layout-column-gap': '4em'});

Why is this done imperatively instead of in the style block ? I think if it's needed in this way or you want to demonstrate the imperative, the user needs some explanation

test/basic.html, line 11 at r3 (raw file):

<body>
  <test-fixture id="default">

I feel that this file might be renamed, maybe just form-layout-tests ?

test/visual/test.js, line 22 at r3 (raw file):

        gemini.suite(snippet, (snippetSuite) => {
          snippetSuite
            .setUrl('/../../demo/' + demo.name + '.html')

In other components we test visual in .html files for that purpose. I like this way of not double maintaining samples but feel that could be a bit fragile. Not a blocker.

Comments from Reviewable

Review status: 23 of 28 files reviewed at latest revision, 11 unresolved discussions.

vaadin-form-layout.html, line 18 at r3 (raw file):
Previously, Saulis (Sauli Tähkäpää) wrote…
"using"

Done.

vaadin-form-layout.html, line 18 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
It supports any child element as layout items.

Done.

vaadin-form-layout.html, line 20 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
it makes

Done.

vaadin-form-layout.html, line 25 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
layout items does not sound fine, I would use child items or just items

Done.

vaadin-form-layout.html, line 36 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
Use the ....   ... on a new row

Done.

vaadin-form-layout.html, line 56 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
Is it necessary? any P2 element extends it by definition

Done.

vaadin-form-layout.html, line 118 at r3 (raw file):
Previously, limonte (Limon Monte) wrote…
this line is no needed anymore

Done.

vaadin-form-layout.html, line 147 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
why 2.1 returns 1, shouldn't be better to return 2 ?

Done.

demo/common.html, line 25 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
Could you add a comment here, that it's used in gemini tests?

Done.

demo/index.html, line 87 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
Why is this done imperatively instead of in the style block ? I think if it's needed in this way or you want to demonstrate the imperative, the user needs some explanation

Would be nice to use something else, like style="...", but it’s not shimmed.
Due to how custom CSS properties shim work, to change CSS property for an element, users either has to make an imperative call or wrap in another custom element.
With wrapping, the code is much longer (and less descriptive, as IE and Edge don’t display the inner template contents in the code snippet):
        <dom-module id="my-form">
          <template>
            <style>
              vaadin-form-layout {
                --vaadin-form-layout-column-gap: 4em;
              }
            </style>
            <vaadin-form-layout>
              <vaadin-text-field label="First Name" value="Jane"></vaadin-text-field>
              <vaadin-text-field label="Last Name" value="Doe"></vaadin-text-field>
              <vaadin-text-field label="Email"<EMAIL_ADDRESS>            </vaadin-form-layout>
          </template>
          <script>
            HTMLImports.whenReady(function() {
              Polymer({is: 'my-form'});
            });
          </script>
        </dom-module>
        <my-form></my-form>

Commend added.

test/form-layout.html, line 11 at r3 (raw file):
Previously, manolo (Manuel Carrasco Moñino) wrote…
I feel that this file might be renamed, maybe just form-layout-tests ?

Done.

Comments from Reviewable

Reviewed 6 of 6 files at r4.
Review status: all files reviewed at latest revision, 10 unresolved discussions.

Comments from Reviewable


dbeaver/pro#2539 fix connection page NPE

Also, the default port was added on the main page for the Generic Instance. And the Instance field has been increased.

verified


Revert / drop animation?

Hello, does the Rubaxa sortable have a separate revert/drop animation option in addition to "animate: xxx"?

Nohow.


feat: button block ui EXLM-73

Please always provide the Jira Issue your PR is for, as well as test URLs where your change can be observed (before and after):
Jira ID: https://jira.corp.adobe.com/browse/EXLM-347
Test URLs:

Before: https://main--exlm--adobe-experience-league.hlx.page/en/button-samples
After: NA

@nitin-rachabathuni  Could you please generate a preview URL for the page and update the PR description?


Tweak warning messages related to V-Sync on OpenGL

Make warnings print only once per session.
Tweak the message to be less confusing, and mention that the issue most likely stems from a graphics driver limitation.

This closes https://github.com/godotengine/godot/issues/93027.

Thanks!


Documents and demo upgrade

Alaska Airlines Pull Request
Fixes: #16
Summary:
This PR addresses some general updates to include upgrading the demo docs to the new markdown format.
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Refactor HIR validation

Follow-up for #812.

Codecov Report

Merging #813 (8745696) into main (2287baa) will decrease coverage by 0.00%.
The diff coverage is 69.23%.

@@            Coverage Diff             @@
##             main     #813      +/-   ##
==========================================
- Coverage   85.43%   85.43%   -0.01%
==========================================
  Files         319      319
  Lines       29713    29710       -3
==========================================
- Hits        25386    25383       -3
  Misses       4327     4327

Impacted Files
Coverage Δ

lib/hir-mir/src/error.rs
6.25% <ø> (+0.36%)
:arrow_up:

lib/hir/src/analysis/error.rs
1.31% <0.00%> (-0.08%)
:arrow_down:

lib/hir-mir/src/lib.rs
88.09% <100.00%> (-0.15%)
:arrow_down:

.../src/analysis/duplicate_function_name_validator.rs
100.00% <100.00%> (ø)

.../hir/src/analysis/duplicate_type_name_validator.rs
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Dockerfile: confirm microdnf update automatically

microdnf gets stuck when the build runs interactively. Specifying "-y" avoids any prompting and allows the build to complete.

/cc @vthapar @tpantelis

/lgtm

/cherry-pick release-2.11


IQ handler gets not invoked

https://github.com/FluuxIO/go-xmpp/blob/2781563ea7698168d48f87295adf27baf8b6636c/_examples/xmpp_jukebox/xmpp_jukebox.go#L49
I am irritated as my IQ handler never gets called. I already tried changing Packet("message"). to Packet("iq"). but it didn't change the behavior.
Maybe you can check if this example is correct.

Hi, do you have some more context info or code to share ?
Does the issue happen all the time, or after a reconnect ? Maybe the context the lost somehow.

Sorry, I'm quite busy, I'll dig into it once I find time again.
Am October 11, 2019 1:53:31 PM UTC schrieb "Mickaël Rémond"<EMAIL_ADDRESS>
Hi, do you have some more context info or code to share ?
Does the issue happen all the time, or after a reconnect ? Maybe the
context the lost somehow.

Hi,
finally i found time to check again. I don't know if you changed
something regarding this but it is working now. So this issue can be
closed.
Thanks for this lib.
On 11.10.2019 06:53, Mickaël Rémond wrote:

Hi, do you have some more context info or code to share ?
Does the issue happen all the time, or after a reconnect ? Maybe the context the lost somehow.
--
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
https://github.com/FluuxIO/go-xmpp/issues/118#issuecomment-541073961


Better UI for self-served password change

When an account is baked by a password credential, the end user should be able to change their password themselves. This assumes the user have not lost their password, which is different from the password recovery flow covered in #13
The flow would look like this:

the user go to their MAS user management screen
they get asked for their old password, their new password and a confirmation of the new password
their other sessions get (optionally?) all revoked
(optionally) they get an email to warn them their password change

Open questions:

What knobs should we expose to server admins?

Allow password change overall
Notify of password changes by email
Whether other sessions should be signed out

Is the password change email notification useful? If so, we need to set the wording/design
Should we actually revoke sessions? Maybe a middle ground would be to revoke browsers sessions, so sessions in the MAS UI, but not client sessions? That would prevent potential attackers to change sensitive stuff on the account, e.g. the list of emails, delete the account, etc.

Relevant design screens:

Potential entry point for this flow, the account mgmt screen

https://github.com/matrix-org/matrix-authentication-service/issues/2197 is tracking creating the API needed for this.

Here is the final design in Figma.


Add kubeletconfig manifest for maxPods setting

@akrzos is it worth adding this for sno as well?

Good point.  I will add that in the next pr.


Hard coded URL part

Hello,
I am would like ask why URL contains hard-coded "account/sessions/"? Is there any reason to leave it other than back-ward compatibility?

Good point, it should be removed, allowing users of the package to define their own URL structure.


Introduce Arcadia support

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2e32327). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage          ?   61.57%
  Complexity        ?      367
=========================================
  Files             ?       62
  Lines             ?     1598
  Branches          ?       88
=========================================
  Hits              ?      984
  Misses            ?      554
  Partials          ?       60

Impacted Files
Coverage Δ
Complexity Δ

...ure/spark/tools/clusters/ArcadiaComputeDetail.java
0% <0%> (ø)
0 <0> (?)

...zure/spark/tools/job/ArcadiaSparkBatchFactory.java
0% <0%> (ø)
0 <0> (?)

...osoft/azure/spark/tools/job/ArcadiaSparkBatch.java
0% <0%> (ø)
0 <0> (?)

Continue to review full report at Codecov.

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

UT PR is following.


bug/ingest local-output dir not honoroed

Describe the bug
Run a command such as:
python unstructured/ingest/main.py --local-input-path /Users/cragwolfe/Downloads/US11544729-p1.pdf \
--verbose --partition-strategy fast \
--structured-output-dir my-output-dir

using an input file of choice. See that output is not stored in my-output-dir as expected, as also shown in the output:
2023-07-03 14:16:22,583 SpawnPoolWorker-2 INFO     Processing /Users/cragwolfe/Downloads/US11544729-p1.pdf
2023-07-03 14:16:22,583 SpawnPoolWorker-2 DEBUG    Using local partition
2023-07-03 14:16:22,737 SpawnPoolWorker-2 INFO     Wrote /Users/cragwolfe/Downloads/US11544729-p1.pdf.json

Secondary potential issue: when re-running the command still with --verbose, one sees the output:
2023-07-03 14:22:42,869 MainProcess INFO     All docs have structured outputs, nothing to do. Use --reprocess to process all.

but there is nothing in the output to indicate where that file outputs have been "seen".
Expected behavior
The output json file should be stored under my-output-dir/.
Desktop (please complete the following information):

OS: mac, python 3.8.17, on main branch as of commit 6ec177e7 (version 0.7.13-dev0)

CC @ahmetmeleq

Checking this one 👍

any update on this?

Sorry, should've been closed with the merged PR


Allow RV32E to access the same CSRs as RV32I

A fix for issue #440.
Includes fixes for trailing whitespace.

BTW, @Columbus240, I did some archaeology and this idea dates back to an older description of RV32E that indicated the perf counters were excluded.  Now that perf counters are a separate extension, that point is moot.

I thought something like that might come up. I only checked git blame a little.


Preview is not showing like the Demo example.

For some reason my image uploader is not showing the image's preview.
I created my example using react-create-app and just copy and paste the example instalation in my App.js file.
Here is my package.json
{
"name": "teste",
"version": "0.1.0",
"private": true,
"dependencies": {
"react": "^16.4.0",
"react-dom": "^16.4.0",
"react-flip-move": "^3.0.2",
"react-images-upload": "^1.1.99",
"react-scripts": "1.1.4"
},
"scripts": {
"start": "react-scripts start",
"build": "react-scripts build",
"test": "react-scripts test --env=jsdom",
"eject": "react-scripts eject"
}
}
and my App.js:
import React from 'react';
import ImageUploader from 'react-images-upload';
class App extends React.Component {
constructor(props) {
    super(props);
     this.state = { pictures: [] };
     this.onDrop = this.onDrop.bind(this);
}

onDrop(picture) {
    this.setState({
        pictures: this.state.pictures.concat(picture),
    });
}

render() {
    return (
        <ImageUploader
            withIcon={true}
            buttonText='Choose images'
            onChange={this.onDrop}
            imgExtension={['.jpg', '.gif', '.png', '.gif']}
            maxFileSize={5242880}
        />
    );
}

}
export default App;

Ah, the picture's state is working just fine, it's appending the upload picture correctly.

 I didn't read the doc well, it was just add a "withPreview={true}" in the ImageUploader component.
Sorry, i'm closing it.

@AikoRamalho, glad you figured it out. : )


update distributed training sample with latest API

Update distributed training sample with latest API
https://docs.microsoft.com/en-us/python/api/azureml-train-core/azureml.train.estimator.estimator?view=azure-ml-py

@datashinobi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@maxluk @sdgilley - can you give a #sign-off if this is ready to be merged?
Thanks.

#sign-off

@j-martens: I'm sorry - only the author of this article, @maxluk, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


BUG: Fixing the nav bar

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
Hii @arghadipmanna101
The search button and the navbar are overlapping so I want to fix this issue
Can you please assign  this issue to me under gssoc24

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

hello, I'm a GSSOC'24 contributor and I would like to solve this issue for you and recreate the nav-bar @arghadipmanna101


JUnit fails to report results of some tests if used with surefire and parallel option including methods

Example: run mvn test in JUnitParams project with following configuration:
            <plugin>
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-surefire-plugin</artifactId>
                <version>2.18.1</version>

                <configuration>
                    <runOrder>filesystem</runOrder>
                    <parallel>classesAndMethods</parallel>
                    <perCoreThreadCount>false</perCoreThreadCount>
                    <threadCount>16</threadCount>
                    <useUnlimitedThreads>false</useUnlimitedThreads>
                    <threadCountSuites>0</threadCountSuites>
                    <threadCountClasses>0</threadCountClasses>
                    <threadCountMethods>0</threadCountMethods>
                    <parallelOptimized>true</parallelOptimized>
                    <includes>
                        <include>**/SubclassTest.java</include>
                        <include>**/Samples_*</include>
                    </includes>
                </configuration>
            </plugin>

This should give total of 44 tests as of today, but often it will give less than that. Contact me for details about research done so far.
Suspicious places:

org.apache.maven.surefire.junitcore.JUnitCoreRunListener#fillTestCountMap
org.apache.maven.surefire.junitcore.ClassesParallelRunListener#checkIfTestSetCanBeReported
org.apache.maven.surefire.junitcore.TestSet#setAllScheduled

Currently following values of surefire "parallel" option should not be used:

"classesAndMethods"
"methods"
"both"

I did some research about this annoying bug.
With an empirical debugging/testing method I fixed the issue by removing the index from the Utils#uniqueMethodId. I found that the index is sometime not consistent during the run.
I suppose the index in the uniqueMethodId have a good reason to be here, but all the unit tests passes without it and my tests passes as well.
I let you try and test my method and review the causes.
Don't hesitate if you have more questions.

@apierzch and @skyghis - Has there been any advance on this issue and line of research?

Sorry @dankirkd, I no longer use JUnitParams. I now use Junit5 with integrated parameterized tests.


separate player history so it does not get deleted with player list

see #59.

Coverage increased (+0.02%) to 93.512% when pulling 58e507202d2458c74e8cd09e31c71cfe4faa7863 on feature/separateHistory into 2738a06d512e9dea721e3f57dad0630f1a88dece on develop.


Support for Issue credentials V1.0

I am trying to create a credential offer using AFJ
So far so good created the wallet with a public did seed but when attempting to publish the did on the ledger I get the error
"Undefined is not a function"
digging deep into the framework I found the function
const request = await this.indy.buildNymRequest(submitterDid, targetDid, verkey, alias, role || null)
That function is not available in "indy-sdk-react-native"
Can anybody correct me if am wrong or am I doing something wrong?

Are you trying to build an issuer wallet in AFJ?  Do you want to publish the did using the mobile wallet? What's the use case?
We only support the holder (and I think verifier) roles for the indy-sdk for react-native. This will change when we add support for Indy CredX / Indy VDR in the coming months, but would be good to understand the use case first.

Yes I am trying to issue credentials using an AFJ mobile wallet. This is part of the effort to create a fully functional AIP1.0 flow (At least for now) then integrating BLE to the game so we can use in Kiosk and other hardware
If it boils down to indy, it doesn't matter much if it was a mobile or web based I think its a matter of adding the missing wrapper functions to the npm package if it is not there
If it will not be supported then why is the function there in the package

I agree with @MosCD3 that there should be a better handling of this error in aries-framework if it's not going to be supported in mobile environments (i.e. React Native). It does work in Node.js, so that's the reason why it is included in ledger module (the same as other ledger operations such as schema and credential definition registration).
I don't know the complexity of implementing these methods in react-native-indy-sdk but probably for the time being it would be nice to throw an "Unsupported operation" error such as the credential definition and schema case. This in indy-sdk-react-native, of course.

Should work with new shared components, we're not going to solve this for indy sdk


shareIntent is "null" in case app was not opened before clicking on "Share"

Describe the bug
In case the app is NOT opened and I'm doing sharing, then the app just opening WITHOUT shareIntent info (Sharing just URL).
console.log(hasShareIntent, shareIntent, error)
null {"files": null, "text": null} undefined

Additional context

expo version : 49
package version : 0.2.0
target : Android
device : simulator and real device
type : local build and eas build

Facing the same issue. It only works for me when the app is in the background (Android).

it sounds likes a bug in my kotlin implementation, I will make some tests about that.

release 0.2.1 and 1.0.1 resolve the issue


lodash: update declaration to match v4.x

Extracted from #48 since these changes are unrelated to it.

remove _.support
re-organize methods to match docs order
add missing methods
enhances tests to ensure flow coverage is 100%

@marudor I was tinkering with this and kept updating my branch, but I think I'm done with it now. It implements all the missing methods except for chaining support, which I'm still working on in #48 (which I'll rebase if/when this lands).

Wow... Awesome work man!
Once you're done I'm going to add tests and merge it into flow-typed if you are okay with that.

Sure. Hope my tests help give some sort of baseline. I had a local version where the the coverage checker was built into test.js and it used child_process.exec for run coverage on itself.. the recursiveness of it was kind of amusing, but I figured the inline re-definition of JSON.parse it required wasn't worth it.


mixin: Use url filter on Remote Write dashboard

We can modify the value of the url variable using the dropdown at the top of the remote write dashboard.
However it's not yet used in metric queries.
Signed-off-by: Nolwenn Cauchois<EMAIL_ADDRESS>

Thanks!


Data-model update in eos_cli_config_gen for SNMP local interfaces

Change Summary
Data-model update in eos_cli_config_gen for SNMP local interfaces
Change in the data-model for local interfaces for SNMP  as one interface can only be associated to one VRF in EOS.
with this new data model
snmp_server:
  local_interfaces:
    Management1:
      vrf: mgt
    Loopback0:
    Loopback12:
      vrf: test2

eos_cli_config_gen will then render the following which is also the configuration in EOS:
snmp-server vrf mgt local-interface Management1
snmp-server local-interface Loopback0
snmp-server vrf test2 local-interface Loopback12

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes
[ ] Other (please describe):

Related Issue(s)

#507
Component(s) name
Eos_cli_config_gen role

eos template https://github.com/aristanetworks/ansible-avd/blob/devel/ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/eos/snmp-settings.j2
doc template https://github.com/aristanetworks/ansible-avd/blob/devel/ansible_collections/arista/avd/roles/eos_cli_config_gen/templates/documentation/snmp-settings.j2

Proposed changes

How to test

add this to a device structure config
snmp_server:
  local_interfaces:
    Management1:
      vrf: MGMT
    Loopback0:
    Loopback12:
      vrf: Tenant_A_APP_Zone

and run the Eos_cli_config_gen role
and verify the cli config is
snmp-server vrf mgt local-interface Management1
snmp-server local-interface Loopback0
snmp-server vrf test2 local-interface Loopback12

and verify the device markdown documentation
### SNMP Local Interfaces

| Local Interface | VRF |
| --------------- | --- |
| Management1 | MGMT |
| Loopback0 | default |
| Loopback12 | Tenant_A_APP_Zone |

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed (pre-commit, make linting and make sanity-lint).
[x] I have updated molecule CI testing accordingly

Last task is to add data for molecule.
I need help to set up molecule tests when there is no l3ls role (only cli config gen role).
The data is this (in the device structured data)
snmp_server:
  local_interfaces:
    Management1:
      vrf: MGMT
    Loopback0:
    Loopback12:
      vrf: Tenant_A_APP_Zone

eos config is generated by cli config gen role is:
snmp-server vrf MGMT local-interface Management1
snmp-server local-interface Loopback0
snmp-server vrf Tenant_A_APP_Zone local-interface Loopback12

Rebasing to resolve CI issue

Rebasing to resolve CI issue


Updating of avatar leads to error page

When I click on update for the avatar without selecting nay avatar, it redirects me to an error page. Please fix!

Stable now


NIFI-10031 Downgraded Jakarta Mail from 2.0.1 to 1.6.7

Summary
NIFI-10031 Downgrades Jakarta Mail from 2.0.1 to 1.2.2 in nifi-bootstrap to restore compatibility on Java 11 with JAXB 2 runtime dependencies on Jakarta Activation 1.
Additional changes include updating assembly configuration to include Jakarta Activation libraries explicitly in the lib/java11 directory.
These changes to nifi-bootstrap are necessary since NiFi includes the contents of lib/java11 in the bootstrap class path when running on Java 11. Jakarta Mail 2 requires Jakarta Activation 2, which breaks compatibility with JAXB 2 and Jakarta Activation 1.
NIFI-10032 outlines the effort involved with upgrading from JAXB 2 to JAXB 3, which will require changes across the framework.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 8
[X] JDK 11
[ ] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Thanks for the fix @exceptionfactory! I still see the system tests failing, but I think those are due to the fact that NIFI-10001 still needs to be merged. I ran this locally and ran into issues too. I then created a local branch that had this and NIFI-10001's latest commit and things looked MUCH better. So will go ahead and merged this to main. Thanks again!


symlink Big Sur

Is there a workaround for mac symlink restictions with Big Sur OS system files?

works for me, what are you seeing?


Fix storybook typeahead

When we moved to a more "correct" version of search in the MockClient, we needed to add some more resources to the generated mock schema to allow the Storybook stories to function correctly.
This PR:

Adds additional resources to mock structuredefinition.json
Adds additional search parameters to the mock seachparams.json
Changes ResourceInput to use a lookup table to determine which searchParam to use

Coverage increased (+0.002%) to 94.202% when pulling c751663f7ceb0d894ee3d7ef9a01fb7cd9d3d9fd on rahul-fix-storybook-typeahead into 937a7a468586f1cce1312cf140cd1123753f0e6e on main.


Uniqueness of express entity attribute handles

Reproduce:

Create two Express Objects: Product and Review
Add an attribute called "Name" to both objects

The attribute handle names have to be unique. I suspect this will lead to problems soon. It basically means that right now, everything needs to be prefixed by the user (e.g. product_name, review_name).
Prefixes are not ideal, because the Express Object handle may change in the future. A possible solution could be to save the Express Object ID in the AttributeKeys table, just like we do with the akCategoryID.

Yes! I actually thought this was just an oversight about to be fixed in the early versions (when I played with this the last time). A bit surprised to hear this is still an issue.

Hmmm. I have to be honest, looking at it from the approach of how packages deal with data, I'm not sure this will be flexible for package developers.
Currently, for example, topics can be named whatever. What this means is that when I am developing a package, and I name a package tree and then call back for it in the package, I can't be assured I am getting the one I actually want (there is no getByHandle for topics, for example).
So again, when initiating code in a package that may use express, we need to be able to ensure that we can get that entity even without knowing what the id is on that particular system (for one system, it will be entity 23, on another it will be 28). So, let's say I go in and initialize an entity, and then I need to go edit it in an upgrade. If I search by the entity name "name", then I could get the name property for some other item! And the only other option is recursively looking down the entity chain, having to find something that looks like a name on this entity and... it becomes unworkable very quickly.
Taking it back to the original point, "the handle may have to change". Ummm, why? Yes, if something is renamed it's really nice for the handle to be renamed. But the end user doesn't see this, honestly. That changes with the Name property here. So even if something was handled "item_name" and you renamed the name to "Product Name" from "Item Name", the world can still exist.
I think the handle HAS to be able to be unique (which is a major issue I have with topics programmatically, it's a pain, and while I have a note that I'd love to code out a handle for topics, I don't know a good way to do it for existing topics). And I think it has to be able to be relied on.
 But I really, really, really feel like this needs to be cleanly and easily accessible by people coding packages, because that is how upgrades SHOULD be handled in a proper fashion. The way it is described here, removing handles will remove an annoyance, but remove flexibility and usability for the people who are creating extensions. I would be thrilled to be wrong, and have someone tell me an "easy" way to work around this. But there is a reason the boilerplate content most people have consists of "getByHandle", not "getByID". ID can change from install to install. Handle should be more reliable. Something needs to be reliable and unique. OR else the only way to change something is to go in and do every site by hand.

I'd vote for not forcing global uniqueness of attribute handles.
This could simply be resolved with an approach like this:

express objects must have an unique handle (eg product, customer, ...)
attribute objects must have a handle that's unique *only within its parent object`
to get an attribute object, we should force to specify both the object and the container handles (ideally something like getExpressAttributeByHandle($parentHandle, $attributeHandle).

This would simplify attribute handles naming a lot, and could help greatly reducing clashes between attributes defined by different packages

That works. It's an extra step/method but that seems a feasible solution.

This has been merged into feature/9.1.0 and will be a part of 9.1.0


Update the docs due to changes in ABP

ABP got an update and the UI is different. This just documents the new way to add filter lists.

👍


Bluetooth: Host: Move BR/EDR files to host/classic

Add subfolder "subsys/bluetooth/host/classic". Move BR/EDR source files to the subfolder.
Move BR/EDR header files from "include/zephyr/bluetooth" to subfolder "include/zephyr/bluetooth/classic".
Move source files of BR/EDR to subfolder "subsys/bluetooth/host/classic/".
Add the path "include/zephyr/bluetooth/classic/" for the header files of BR/EDR.

Missing migration guide w.r.t the Kconfig rename

Missing migration guide w.r.t the Kconfig rename

Updated migration guide 3.7

@lylezhu2012 there seems to be some conflicts that need resolving.


chore: upgrade to unbuild v3 rc

Description
Unbuild v3 updates to rollup v4 and esbuild 0.24 which helps deduplicate our local dependencies. The main breaking change seems to be that, and rollup.esbuild.target now defaults to esnext, but we've already set the value ourselves so we're clear.

Looks like the js-sourcemap snapshot probably got affected due to transitive deps upgrade. Testing locally it seems to add an additional segment between t() (t( + )), but both still point to the same source, so there's no change in mapping in practice. (Top: before. Bottom: after)


Not getting location data when the user is riding a bike or in a car

Environment

Plugin version: 2.2.0
Platform: Android

ionic (Ionic CLI)       : 3.13.1
global packages:
cordova (Cordova CLI) : 7.1.0

local packages:
@ionic/app-scripts : 3.1.6-201712271536
Cordova Platforms  : android 6.3.0
Ionic Framework    : ionic-angular 3.9.1-201711081842

System:
Android SDK Tools : 26.0.2
Node              : v6.10.2
npm               : 5.5.1
OS                : Windows 7

I am using cordova background geolocation plugin for my ionic app(android). Below is my configuration,
const config: BackgroundGeolocationConfig = {
desiredAccuracy: 0,
stationaryRadius: 5,
distanceFilter: 5,
interval: 10000,
fastestInterval: 5000,
url: "https://1*...1:8200/geo_api/saveBackgroundGeoLocation",
syncUrl: "https://1..*.**1:8200/geo_api/saveBackgroundGeoLocation",
httpHeaders: { "User": user },
maxLocations: 8,
syncThreshold: 4,
startForeground: false,
startOnBoot: true,
debug: true,
stopOnTerminate: false,
stopOnStillActivity: false
};
  this.backgroundGeolocation.configure(config).subscribe(
    (location: BackgroundGeolocationResponse) => {
      console.log(location);
      this.backgroundGeolocation.finish(); // FOR IOS ONLY
    },
    err => {
      console.log(err);
    }
  );

This configuration is working fine when the user is walking or at a slow speed.
But whenever the user is on a bike or in a car location is not being sent to url.
Could, anyone please explain what should be the optimized configuration for getting location data from a fast moving user.

Problem seems to only start after a few minutes for me. Users can restart the app when they get to their destination and it starts recording again but for some reason when they drive the updates stop as well as onStationary stops getting called so i lose track of people.
if you found out anything any help would be great i have been digging for a couple of days already at this.

When the user is going at a faster speed, it becomes difficult to articulate the location.In my case, I increased the values of "desiredAccuracy" and "stationaryRadius" parameters. I put desiredAccuracy at 100 which made sure the device uses "A-GPS".
Also a higher stationaryRadius helps the device to fetch the location.


Does not register broken lillypads.

Version: Quests 3.9.3
we have a quest to collect lillypads and it does not register when you break them.

Thanks for the report. What do you mean by "collect" lillypads? Is this a break-block objective? Deliver-items objective?

Yes break block
On Fri, Apr 24, 2020, 1:15 AM PikaMug<EMAIL_ADDRESS>wrote:

Thanks for the report. What do you mean by "collect" lillypads? Is this a
break-block objective? Deliver-items objective?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PikaMug/Quests/issues/1250#issuecomment-618806228,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHS26BVL2DH27SMCYDIXADROEOALANCNFSM4MPVNE6Q
.

Working fine for me on the latest development build:

What is your MC server version? Do you have any protection plugins like WorldGuard installed that might be interfering? If you're using an anti-cheat or ViaVersion, try diabling those temporarily.

Closing due to lack of response from OP.


Add Redis as a new key-value NoSQL database option

Add Redis as a key-value database option first next to MongoDB in the NoSQL options.
The idea is to add just the dependency and basic configuration to let users/developers add what they need from there.
And in another PR add the option to use it as a cache provider.
And maybe in another one, the option to use it as a message broker next to Apache Kafka.

Increasing the bounty to motivate people as it's a good feature to have

I can work on this issue. Could you assign me?


Fails on Local image?

Our scans seem to be failing when used on an image within the github action
https://github.com/StatCan/kubeflow-containers/runs/1467927330#step:6:24
The error no-longer occurs if we scan an image that has already been pushed to the acr. (But, of course, we'd prefer to scan before pushing)
Is this expected behaviour? Do you have any recommendations for how to address this?

@blairdrummond This seems to be an issue with the underlying tool, Trivy that the action is using. Here are a few possible solutions that I could find:

Try scanning the local image without giving it a label
Set an environment variable TRIVY_TIMEOUT_SEC=360s on the GitHub runner.

Thanks! Sorry, it makes sense that this is an issue with the upstream tool not this action. I will move the issue.


Add Sabrina Fonseca article

An awesome article to design forms from a general point of view

Hey @jorgegalindocruces, I hope you don't mind we reject this PR in favour of #23, which contributes the same post, for 2 reasons: 1) It'll be Iñaki's first contribution and we will have a new folk on board! :tada: :tada: :tada: and 2) although the post has some actionable resources, it is mainly argumentative as the author herself acknowledges.
Thank you for your many contributions anyway! :heart:

Of course!!! 💙
On Nov 10, 2017 6:12 PM, "Manuel Gómez"<EMAIL_ADDRESS>wrote:
Hey @jorgegalindocruces https://github.com/jorgegalindocruces, I hope you
don't mind we reject this PR in favour of #23
https://github.com/folkswhocode/awesome-diversity/pull/23, which
contributes the same post, for 2 reasons: 1) It'll be Iñaki's first
contribution and we will have a new folk on board! 🎉 🎉 🎉 and 2) although
the post has some actionable resources, it is mainly argumentative as the
author herself acknowledges.
Thank you for your many contributions anyway! ❤
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/folkswhocode/awesome-diversity/pull/22#issuecomment-343531620,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGd9sML7Wv910UgvBScILwmYfcZbkFSks5s1IPogaJpZM4QZXQb
.


Examples should have Cargo.toml

Had some issues getting the examples to run as there was no clear Cargo.toml for referencing the correct dependancies and versions etc.
Here's what worked for me, could we add it to the examples folder or the README?
$ cat Cargo.toml
[dependencies]
hyper = "0.13.1"
tokio = { version = "0.2", features = ["full"] }
pretty_env_logger = "0.3.1"

Are you perhaps not aware that examples can be run with cargo run --example [name]?


Question: how to track line number and position in line while parsing?

Can you point me to an example or write one.
While parsing some data I need to know where I am now in a file (line number, position in line).
For example if I'm in the beginning of a new line I want to apply a parser 1 and if I'm not in the line beginning - a parser 2.

paging @Hywan for this, he has very good examples of how to achieve this :)
The idea is to make your own input type, that must implement those traits: https://github.com/Geal/nom/blob/master/src/traits.rs
your input type would contain the line number and position.

Hello @pashinin,
I am writing a blog post about this feature in nom. Until its' published, you can look this PR, https://github.com/tagua-vm/parser/pull/101, or this code directly, https://github.com/tagua-vm/parser/blob/e7aa96322c4abe0a758286a5033dcec90cf28a21/source/tokens.rs#L759-L1189.
@fflorent is working on a generic version of this!

Hello there,
Yup! Here is my version of the same feature:
https://github.com/fflorent/espadon/blob/input_slice/src/input_slice.rs
I am currently focusing on making it work for my case to identify cases where it wouldn't work… But thinking of it, that's not the philosophy of open-source (Release early, release often). I'll make a separate crate soon and will notice you for that, even if that's far from being perfect.
Cheers!
Florent

Hello everyone!
I've just published an unstable version of nom_locate for this purpose. By « unstable », I mean that I need some feedback and that upcoming breaking changes may be brought soon.
Here is the repository:
https://github.com/fflorent/nom_locate/
Feedback welcome (@Hywan already gave me some, thanks to him!).
For example on the vocabulary (I am not very familiar with the parsing one).
Thanks!
Florent

this is awesome :)

Thanks @geal :). I'll probably make a 0.1.0 release with the hope that it'll receive feedback from some users.
Cheers!
Florent

I've published version 0.1.0, as it seems to me to be functional. The only remarkable change is the update to nom 3.1.0.
@pashinin What do you think of this crate? Does it respond to your needs?
Florent

@fflorent Thank you. I'm in a russian forest now) Will test it later =) We may close this issue for now


Cache LLM responses using Redis, for cheap integration testing [on hold]

Before we push more features, we should lay the groundwork for setting up robust tests. As an integration platform, it's essential to have integration tests. But LLM calls are expensive. But don't worry, caching to the rescue! (I hope)
Issues

There are few/no integration tests in LlamaIndexTS
But to do integration tests, we need API calls

"Well. Looks like we have ourselves a genuine conundrum. A quandry if you will. A real life dilemma." – Killa, John Wick 4
Ways to resolve issues

Make it easy to write integration tests
Cache LLM calls using Redis (both embedding and completions)

How we'll implement this

Keep a dump of cached prompts in Redis .rdb file, which can be loaded in during testing

Quick Maths (rough estimate) about Redis store size

1 prompt (~8000 tokens) * 5 characters/token  = 40K characters/prompt * ~2 byte/char (utf-8, including special chars) ≈ 80KB/prompt
1 GB / 80KB ≈ 12K prompts
12K / 25 (# prompts/integration test) ≈ 500 integration tests * 70% (-30% overhead discount, just in case)
divided by 2, since we have to cache {text_chunk: embedding} too
which evaluates to ≈175 integration tests minimum before we hit 1GB.

TL;DR We can use this approach for a while. These estimates are really conservative too.
Edge Cases/Concerns

Multiple LLMs? We can have multiple hash stores
Cost? The first integration test will be expensive, but the rest will be free
Maintenance? Store the DB dump file and load it in
Cloud? Nope, everything is done locally
Scale? See above

To-do List

[ ] Make wrapper class over LLM that caches completions/embeddings
[ ] Test Redis in sandbox
[ ] Implement Redis store, RedisGUI sanity check
[ ] Get Redis to work during integration test
[ ] (Proof of Concept) Write full integration test for src/QueryEngine, and get it running

This is too far into the future, just throw away the code.


Bump regulations-site version

Changes

Bump the version of regulations-site to 2.1.4

Checklist

[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[x] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[x] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Assignee tool :arrow_right:

Agree with @higs4281, it'd be good if setup.py over there was in sync. Not sure it's worth making another release for that, maybe?

I've fixed the setup.py discrepancy.


update getLocalIPAddress

https://ipapi.co/ip start using  Verifying you are human. This may take a few seconds.
logs :
time="2024-04-28 09:32:23.572" level=info msg="publicIpAddress: <!DOCTYPE html><html lang=\"en-US\"><head><title>Just a moment...</title><meta http-equiv=\"Content-Type\" content=\"text/html; charset=UTF-8\"><meta http-equiv=\"X-UA-Compatible\" content=\"IE=Edge\"><meta name=\"robots\" content=\"noi
use https://ifconfig.me/ with User-Agent: curl/7.61.1

we need regional information, not just public IP

I have tested https://ipapi.co/ip, it works

use in same region only not all regions, work in my vps , and not working with my home network

curl ipinfo.io
it seems this more friendly

https://github.com/swanchain/go-computing-provider/pull/54

use : http://ip-api.com/json/ without ip to get the ip and metadata in the same request

thanks for your feedback, we will improve it


热重启或热加载 RtcEngineEventHandler 无效

必须停止再重启启动才能生效，修改代码后就不再生效，不知道是哪里的问题，是不是不支持呀。
开发起来好难受
使用的事例代码 https://docs.agora.io/cn/Interactive Broadcast/start_live_flutter?platform=Flutter

Hello @kids-return , which version of agora_rtc_engine do you use?

If you still face issues, please try upgrading to the new version to see if it works for you.


White-lists and Black-lists

A useful feature for a game server would be to have configuration file white-lists and black-lists for player names (and ip addresses). These lists could have stored, and/or "pre-seeded" configurations for player names.

Closing as completed by #3332 :-)
Still could use a bit of documentation so I submitted #3364 for that


libraries under /opt are not detected

under /opt we put libraries our lambda would require to run .
in the lambda envrionment , we'd had to overwrite the default LD_LIBRARY_PATH to search under the newly library path first : $LAMBDA_TASK_ROOT/lib:$LAMBDA_TASK_ROOT/lib64:$LAMBDA_RUNTIME_DIR:$LAMBDA_RUNTIME_DIR/lib:$LAMBDA_TASK_ROOT:/opt/lib:/lib64:/usr/lib64
when the lambda ran it failed with:
/var/task/main: /usr/lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by /opt/lib/libheif.so.1)
but both were included in our newley created layer with img2lambda:
$ unzip -l lambda-img/layer-1.zip
Archive:  lambda-img/layer-1.zip
Length      Date    Time    Name

13762736  01-25-2019 10:47   lib/libMagickCore-7.Q16HDRI.so.6
3155944  01-25-2019 10:47   lib/libMagickWand-7.Q16HDRI.so.6
10996664  01-25-2019 10:47   lib/libde265.so.0
6577608  01-25-2019 10:47   lib/libheif.so.1
178376  01-25-2019 10:47   lib64/ld-linux-x86-64.so.2
2066416  01-25-2019 10:47   lib64/libc.so.6
19208  01-25-2019 10:47   lib64/libdl.so.2
92736  01-25-2019 10:47   lib64/libgcc_s.so.1
187576  01-25-2019 10:47   lib64/libgomp.so.1
281096  01-25-2019 10:47   lib64/libjpeg.so.62
1465520  01-25-2019 10:47   lib64/libm.so.6
149272  01-25-2019 10:47   lib64/libpthread.so.0
1590808  01-25-2019 10:47   lib64/libstdc++.so.6
so is adding libraries under /opt supported ?

@bciceron same issue to me also I just build https://github.com/jeylabs/aws-lambda-poppler-layer but poppler libs are in /opt/lib64 I cound not execute it without setting LD_LIBRARY_PATH could your please explain how did you fix it?
// cc @asahasrabuddhe


Clarify a few things in Cassandra example.

Fixing Cassandra example.  #8356
Up for any suggestions on how to clean up the paragraph that starts "In Kubernetes, a Service..."  That's still not as clear as I'd like.

@mikedanese Fixed to use kubectl exec, and also updated the docs to use the v3 image (matching the yaml.

@goltermann nice. are you sure the yaml was correct? it doesn't look like the v3 tag is pushed.
https://registry.hub.docker.com/u/kubernetes/cassandra/tags/manage/

@mikedanese They're gcr.io links in the yaml
https://github.com/GoogleCloudPlatform/kubernetes/blob/master/examples/cassandra/cassandra.yaml

@mikedanese Ah, gotcha.  Fixed in the yaml snippets and some of the kubectl outputs.

LGTM


Update the Client Libraries link to replaced the depreciated package

The https://www.nuget.org/packages/Microsoft.Azure.Kusto.Data.NETStandard/ package has been depreciated

@tidusjar : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

@tidusjar Thanks for contacting us with feedback about our product documentation. Depending on the complexity, it could take from a few business days to a few weeks to look further into your question, issue, or suggestion. We'll contact you if we need further information, and you'll receive an email message each time we "@mention" you in a comment.
Thanks again for submitting your feedback. Your suggestions help improve our documentation.

@shsagir : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@shsagir : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@shsagir: I'm sorry - only the author of this article, @orspod, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


python3Packages.geometric: init at <IP_ADDRESS>

Motivation for this change
Geometry optimization code for molecular structures.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build python3Packages.geometric

nixpkgs-review happy, macos 10.15, nixos x86_64 & aarch64

LGTM! :sheep:


How to create Border

how to create border on image gallery ..

Create a div with border, and put jssor slider inside the div.


docs: move Disable Resources docs to its own page

Move the beta Disable Resource documentation to its own page, so it's more visible and discoverable. I'm putting this up now so we can have a stable link to share with users.
Besides the new page, the only other changes are:

Added an explanation of "beta" at the top of the page (@lianmakesthings, lmk if this doesn't sound good!)
The two minor notes about Disable Resources functionality are now formatted as bullet points, instead of tips, because they look too visually distracting in addition to the "beta" tip.

I'm not sure that it makes sense for these docs to be listed under the Tiltfile category, but we can move them later.
I need to do a little bit more work to support adding the "beta" tag to the nav and header, so I'll put that up in a separate PR.

@lizzthabet, this looks great! I think you can go ahead and add the Beta badge to the sidebar as well:

I did a quick edit in Inspector and it looks fine. Surbhi was on Zoom with me and agreed that it's good to ship for now, though we can refine it later by adjusting the colors. (i.e., when we decide to tag more stuff beta or experimental or such in our general docs, and thus want to color code)

  Disabling Resources
  Beta

thanks, @hyu! the sidebar links / headers are also auto-generated, so i need to make template changes to get the pill-tags into those sections. i have this working locally and will put it up for pr separately.


Follow python naming conventions

Is your feature request related to a problem? Please describe.
Describe the solution you'd like
Variables and functions that are currently using camelCase should be changed to snake_case. This would improve the code quality.
The following files should be refactored:
article

[ ] article/views.py

discussion

[ ] discussion/views.py

feed

[ ] feed/serializers.py (this might require a frontend update)
[ ] feed/signals.py
[ ] feed/utils.py
[ ] feed/views.py

notification

[ ] notification/signals.py
[ ] notification/views.py

users

[ ] users/signals.py
[ ] users/views.py

This could be changed in one PR or do it by apps.

@guilleijo thanks for opening this issue !
Can you please refer to the contributing guidelines because you forgot adding this issue to the project board !
https://github.com/divanov11/Mumble/blob/master/Project_Board.md

@guilleijo thanks for opening this issue !
Can you please refer to the contributing guidelines because you forgot adding this issue to the project board !
https://github.com/divanov11/Mumble/blob/master/Project_Board.md

Oh sorry about that, didn't know. Thanks!

@guilleijo thanks for opening this issue !
Can you please refer to the contributing guidelines because you forgot adding this issue to the project board !
https://github.com/divanov11/Mumble/blob/master/Project_Board.md

Oh sorry about that, didn't know. Thanks!

No problem @guilleijo
You can find more instruction in the Mumble Docs Website : https://midouwebdev.gitbook.io/mumble-docs

@guilleijo are you working on this ? Please let us know !

@guilleijo are you working on this ? Please let us know !

Hey @PraveenMalethia, sorry I have not had the time. If anyone wants to work in this can go ahead and do it. It can also be divided by files or apps and multiple PRs can be done.
I'll unassign myself so it's free for taking 👍

I did this refactor


Guidance for automating the peering process across tenants?

I'm trying to automate vnet peering between different tenants, and am currently using service principals to authenticate to Azure.  When trying to peer a vnet in a different tenant, I keep receiving permissions errors, and it appears that service principals cannot be added as guest users in different tenants.  Is there any guidance on how to automate this process so we don't have to manually execute scripts every time we want to peer a vnet in a different tenant?

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 52fe18fe-efc5-3e63-8200-8e20e885b55e
Version Independent ID: 86a4f465-d676-e0e4-7367-2ed9f906340a
Content: Create an Azure virtual network peering - Resource Manager - different subscriptions
Content Source: articles/virtual-network/create-peering-different-subscriptions.md
Service: virtual-network
GitHub Login: @jimdial
Microsoft Alias: jdial;anavin

@NateB2 , We are looking into this query and will update you as soon as possible.

@NateB2 , You can peer virtual networks that exist in two different subscriptions, as long a privileged user of both subscriptions authorizes the peering, and the subscriptions are associated to the same Azure Active Directory tenant. You can use a VPN Gateway to connect virtual networks in subscriptions associated to different Active Directory tenants.
Closing this issue for now. If there are further questions regarding this matter, please reply and we will gladly continue the discussion.

@SubhashVasarapu-MSFT You're saying about "assigning respective RBAC roles to them", but you are not saying how to see AppId from tenant#2 in AAD of tenant#1 (to grant permissions for AppId#2 to do smth in tenant#1). That looks most tricky. The only way I found is here. But to make the URL described in that documentation working I had to modify my AppId with the following steps:
1 set Application ID URI = api:// in Expose an API
2 change to "Accounts in any organizational directory" in Authentication
3 add "Microsoft Graph (1).User.Read" permission in API permissions
4 set "Redirect URIs" = https://www.microsoft.com in Authentication
After that I was able to run the URL from the documentation and my AppId#2 was listed in AAD#1.
However, when I'm trying to set up peering using ServicePrincipal authentication (I'm using Terraform) I got an error  Failure sending request: StatusCode=403 -- Original Error: Code="LinkedAuthorizationFailed" Message="The client has permission to perform action 'Microsoft.Network/virtualNetworks/peer/action' on scope '/subscriptions/<subscr#1>/resourceGroups/dev-rg/providers/Microsoft.Network/virtualNetworks/dev-vnet/virtualNetworkPeerings/dev-vnet-to-dev-vnet-uw2', however the current tenant '<tenant#1>' is not authorized to access linked subscription 'tenant#2'."
Probably you have some known steps what configuration is missing to make this working.


Mahidul

Thanks
What the program do ?

Yes it was good thanks
In what programming language it is written?

Kolkata


cudpp_testrig Stream compact tests ignore the "quiet" flag

Not high priority, but this needs to be fixed.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 25 Jun 2009 at 12:26

Fixed.  Verified.

Original comment by<EMAIL_ADDRESS>on 26 Jun 2009 at 5:23

Changed state: Verified


Dynamic Status Bars support for XMonad.Hooks.StatusBar

Description
This provides dynamic status bars, that react to screen configuration changes. This needs the new interface from #443 and the splitting from #465, so I'm marking the PR as a draft till that's resolved.
Heavily inspired by "XMonad.Hooks.DynamicBars"
Checklist

[x] I've read CONTRIBUTING.md

[x] I tested my changes with xmonad-testing

[x] I updated the CHANGES.md file

[x] I updated the XMonad.Doc.Extending file (if appropriate)

Actually not really correct, as my hooks looks into the event queue directly and discards events. I'll explain more when I get back from a walk. Dne 13. 2. 2021 12:36 napsal uživatel brandon s allbery kf8nh<EMAIL_ADDRESS>@geekosaur commented on this pull request.
In XMonad/Hooks/DynamicStatusBarConfigs.hs:

+newtype ActiveSBs = ASB {getASBs :: [(ScreenId,  StatusBarConfig)]}
+
+instance ExtensionClass ActiveSBs where

initialValue = ASB []

+-- | Given a function to create status bars, 'dynamicSBs'
+-- adds the dynamic status bar capabilities to the config.
+dynamicSBs :: (ScreenId -> IO StatusBarConfig) -> XConfig l -> XConfig l
+dynamicSBs f conf = conf

{ startupHook     = startupHook conf

                 *> setupEventHandler

                 *> cleanAll

                 *> updateSBs f

, logHook         = logHook conf *> logSBs
, handleEventHook = eventHookSBs f <> handleEventHook conf

Correct, you would need to add your own logic to suppress your own event handlers in that case. Note that xmonad has no idea what hooks return All False until the hook has executed, since they can contain arbitrary logic.
—You are receiving this because you commented.Reply to this email directly, view it on GitHub, or unsubscribe.

Alright, I've been using this for a couple of months now, and it's been working great. Except for a minor issue: sometimes, some bars don't get started. Not sure why, it doesn't seem like a problem with the code itself.
Here are the logs from restarting xmonad. For the record, I restarted it like 5 times in a row to get it misbehaving.
XMonad will use build script at "/home/yecinem/.xmonad/build" to recompile.
XMonad recompilation process exited with success!
XMonad is recompiling and replacing itself with another XMonad process because the current process is called "xmonad" but the compiled configuration should be called "xmonad-x86_64-linux"
XMonad will use build script at "/home/yecinem/.xmonad/build" to recompile.
XMonad recompilation process exited with success!
xxmmoobbaarr::  CCaauugghhtt  ssiiggnnaall  1155;;  eexxiittiinngg......

xmonad-x86_64-linux: failed to create OS thread: Resource temporarily unavailable
Bad _NET_DESKTOP with data[0]=-1
Bad _NET_DESKTOP with data[0]=-1
Bad _NET_DESKTOP with data[0]=-1

Alright, I've been using this for a couple of months now, and it's been working great. Except for a minor issue: sometimes, some bars don't get started. Not sure why, it doesn't seem like a problem with the code itself.
Here are the logs from restarting xmonad. For the record, I restarted it like 5 times in a row to get it misbehaving.
XMonad will use build script at "/home/yecinem/.xmonad/build" to recompile.
XMonad recompilation process exited with success!
XMonad is recompiling and replacing itself with another XMonad process because the current process is called "xmonad" but the compiled configuration should be called "xmonad-x86_64-linux"
XMonad will use build script at "/home/yecinem/.xmonad/build" to recompile.
XMonad recompilation process exited with success!
xxmmoobbaarr::  CCaauugghhtt  ssiiggnnaall  1155;;  eexxiittiinngg......

xmonad-x86_64-linux: failed to create OS thread: Resource temporarily unavailable
Bad _NET_DESKTOP with data[0]=-1
Bad _NET_DESKTOP with data[0]=-1
Bad _NET_DESKTOP with data[0]=-1

As we discussed on the IRC, this was because of -rtsopts -with-rtsopts=-N

Overall it looks good. But I have a issue here.
Use-case example:
Two monitors:
laptop 1920x1080
2K display 2560x1440 (external)

And set external monitor to be primary.
Then in the definition of barspawner:
barSpawner :: ScreenId -> IO StatusBarConfig
barSpawner 0 = pure xmobar0  -- two bars on the main screen
barSpawner 1 = pure xmobar1
barSpawner _ = Data.Monoid.mempty -- nothing on the rest of the screens

xmobar0 is used for external display.
But after unplugging the external display, the primary display becomes the laptop. But xmobar0 is for external display.

Thanks for your feedback! For your use case, I think this is out of the scope of this PR (and I guess of xmonad? not sure). We can't control which id is given to which screen. But maybe I'm wrong, I know very little about X.

Thanks for your feedback! For your use case, I think this is out of the scope of this PR (and I guess of xmonad? not sure). We can't control which id is given to which screen. But maybe I'm wrong, I know very little about X.

So I think for my use-case, I need to do a bit of hack. Let me see. :)

As we spoke in the IRC, I mentioned that the commands themselves are used to keep track of what's launched. If someone could give it a proof read, we can merge this tonight :tada:


Remove old YAML library

We need to remove the last few references to gopkg.in/yaml.v2:
cmd/helm/repo.go
28:    	"gopkg.in/yaml.v2"

glide.yaml
12:- package: gopkg.in/yaml.v2

glide.lock
310:- name: gopkg.in/yaml.v2

pkg/lint/rules/template.go
30:    	"gopkg.in/yaml.v2"

pkg/repo/index.go
24:    	"gopkg.in/yaml.v2"

pkg/repo/index_test.go
28:    	"gopkg.in/yaml.v2"

pkg/repo/repo.go
30:    	"gopkg.in/yaml.v2"

These should be replaced with github.com/ghodss/yaml, which exposes mostly the same API.

Due to some recent changes, we're down to just a few references:
⇒  ag yaml.v2
glide.yaml
12:- package: gopkg.in/yaml.v2

glide.lock
366:- name: gopkg.in/yaml.v2

pkg/lint/rules/template.go
30:    	"gopkg.in/yaml.v2"

pkg/repo/index.go
140:// Need both JSON and YAML annotations until we get rid of gopkg.in/yaml.v2


[GUI] [FEATURE] Add provisioned vGpus in VM's devices list

Is your feature request related to a problem? Please describe.

We want to improve the vGpu allocation when provisioning Harvester clusters in Rancher. https://github.com/harvester/harvester/issues/5774
With the new mechanism, the vGpu IDs are selected by the backend once a vGpu device is assigned to a provisioned cluster in Rancher, and saved in harvesterhci.io/deviceAllocationDetails VM's annotation in Harvester.
cc @ibrokethecloud @bk201
Describe the solution you'd like

The UI should build the vGpu list from both source fields:

domain.devices.gpus
harvesterhci.io/deviceAllocationDetails label

Describe alternatives you've considered

Additional context

@torchiaf Could you please update the reproduction steps so that I can be sure I test every scenario before closing this? You mentioned that you changed the way the count in the node driver was determined but I wanted to be sure to update it in every case and make sure the count was correct.

cc @torchiaf Please add a test plan.

Tested with Harvester v1.4-ce5e24e3-head with Rancher v2.9.4 and Rancher v2.10.0
. Validated as working.
Validation Steps

Create at least 4 vGPU devices on Harvester
Import Harvester into Rancher
Create a 3 node RKE2 cluster with assigned vGPUs
Wait for cluster to provision
Click edit on each individual VM in Harvester
Check vGPU tab and validate that each VM has a different vGPU ID

Screenshots
v2.9.4

v2.10.0


Docs: specify yarn 3

And in the volto submodule:

volto/addons/index.md line 335
volto/getting-started/install.md (yikes! this is out of date.)

Originally posted by @stevepiercy in https://github.com/plone/documentation/issues/1458#issuecomment-1449868348

Actually https://6.docs.plone.org/volto/getting-started/install.html is legacy documentation. See https://github.com/plone/volto/issues/3998


Reflect deleteProperty() wiki seems missing

@2shraaf commented on Fri Oct 23 2020
When I go to https://wiki.developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Reflect/deleteProperty, it view Reflect.defineProperty() wiki not deleteProperty

Another user has already reverted the wrong revision on October 29. The page shows the correct content again.
@chrisdavidmills This issue can be closed.

Great stuff; thanks @mikoMK !


Crossplane - Create Wrapper resource for AWS Route53 records

What: Create a wrapper resource for Cloudfront
Who: Pirates Team
Why: Help converting Pirates team resources to Crossplane to help onboarding them as beta users on the Crossplane setup

[x] Create XRD for Cloudfront
[x] Test create a CF in sandbox environment

wip https://github.com/dfds/crossplane-configuration-dfds/pull/32

merged


Return null when image not found on FileUpload component

When FileUpload component does not find the file, it throws an exception. Sometimes files can be deleted for some reasons like updating the server, etc. Very short screen sharing video expressing what I mean: youtube link
Open to any ideas.

@danharrin, of course you're right, it's better to check for the existence of a file than to catch an exception. Made a commit. Please check it.

Thanks!


[Enhancement]: Add Analytics Page

Requirements

Top Nav link to Analytics at /analytics
Sidebar link to analytics at end of Top Pools section of sidebar
retrieve list of most recently used pools (e.g. https://ambindexer.net/gcgo/pool_list?chainId=0x1&poolIdx=420)
filter pool_list results by chainId active in the UI (gcgo issue #6)
retrieve TVL / Volume for each pool (similar to Top Pools cards)
retrieve current pool price and 24h change percentage (similar to Top Pools cards on homepage)
display a row containing the above information and a link to /trade/market route for each pool in pool_list
highlight each row on hover (similar to trade/portfolio tables)
enabled sorting by TVL, Volume, Price, Price Change %
add 'View more' link on homepage (as pictured below)

Figma
https://www.figma.com/proto/0ItLk4nJmtj741R8sn3HYp/ambient?type=design&node-id=6406-152220&t=oYfxxDdr0b3jq5sV-0&scaling=min-zoom&page-id=4359%3A117444&starting-point-node-id=4378%3A105457&show-proto-sidebar=1
Assumptions
No response

NOTE: risk of unacceptable # of infura calls, may need to add cached endpoint for pool stats

@miyuki-eto need some mockups for view more link (top pools) on homepage

NOTE: risk of unacceptable # of infura calls, may need to add cached endpoint for pool stats

I added logic to gatekeep the one automated call. All unguarded calls are made on explicit user action. I did open the gate if the call errors but we'll adjust later if that happens.


Add support for BitTorrent chain

Description
Add support for BitTorrent chain
How to test
Same as other EVM chains
Types of changes

Checklist

[x] Create pull request as draft initially, unless its complete.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] If there is a related Issue, mention it in the description.

If you're adding a new blockchain

[x] I have read the guidelines for adding a new blockchain.

Any good to add this, BitTorrent chain from Tron?

My Public Address to Receive ETH: 0x52B2241F491e5937b8831a158972412497a8C36E
Pay me via Trust Wallet: https://link.trustwallet.com/send?address=0x52B2241F491e5937b8831a158972412497a8C36E&asset=c60
0x52B2241F491e5937b8831a158972412497a8C36E
Sent from my iPhone

Let's revisit this later?


API Gateway integrations incorrectly updated when target changes (AWS Classic)

What happened?
I have a service deployed using pulumi as a lambda fronted with an API Gateway. I want to convert the lambda to use ECS and fargate instead. I update the pulumi files to change the lambda to now be an ECS Task/Service, spin up an ECS cluster, and create a restAPI integration to use HTTP_PROXY, VPC_LINK, and NLB url instead of the initial AWS_PROXY and event handler url.
I expect the Lambda along with any Lambda related resources like permissions and cloudwatch log groups to be deleted, while the ECS related services like task, service, vpc link, NLB, and any security groups, permissions to be created. I also expect the restAPI aws.apigateway.Integration to be updated to instead of using the lambda, use the VPC_LINK and nlb.
Instead what happens is all of the expected resources are created, but the aws.apigateway.integrations are instead destroyed without recreating new integrations using the vpc link. At this point, I can navigate to the AWS console and manually update the method to create the integration. Doing so makes everything function correctly.
Steps to reproduce
Start off with a restAPI configured with resource and method. Create an integration for lambda and deploy your service
new aws.apigateway.Integration(`${methodPath}-${r.verb.toUpperCase()}`, {
                    restApi: restAPI.id,
                    resourceId: parentResource == null ? restAPI.rootResourceId : parentResource.id,
                    httpMethod: method.httpMethod,
                    integrationHttpMethod: "POST",
                    type: "AWS_PROXY",
                    uri: lambda.invokeArn
                })

After that is completed, set up pulumi code to create an ECS cluster and it's associated resources. Create a new integration
for VPC link instead
new aws.apigateway.Integration(`${methodPath}-${r.verb.toUpperCase()}`, {
                    restApi: restAPI.id,
                    resourceId: parentResource == null ? restAPI.rootResourceId : parentResource.id,
                    httpMethod: method.httpMethod,
                    integrationHttpMethod: method.httpMethod,
                    type: "HTTP_PROXY",
                    connectionType: "VPC_LINK",
                    connectionId: vpcLink.id,
                    uri: pulumi.interpolate`http://${nlb.loadBalancer.dnsName}${r.path}`
                })

Deploying this will result in all the resources being created but the integration being deleted. Pulumi up will complete and show that it was updated
+-  ├─ aws:apigateway:Integration                 /-GET                     replaced     [diff: ~connectionId,connectionType,integrationHttpMethod,type,uri]
+-  ├─ aws:apigateway:Integration                 quotes-GET                    replaced     [diff: ~connectionId,connectionType,integrationHttpMethod,type,uri]
+-  ├─ aws:apigateway:Integration                 users-GET                         replaced     [diff: ~connectionId,connectionType,integrationHttpMethod,type,uri]
+-  ├─ aws:apigateway:Integration                 users-POST                        replaced     [diff: ~connectionId,connectionType,type,uri]```

But once you do a pulumi refresh, it will actually attempt to update the stack and show these resources were deleted.

### Expected Behavior

I expected the restAPI method integrations to be updated to point at NLB/VPC_link

### Actual Behavior

aws.apigateway.Integrations using the lambda are destroyed and the ones using the VPC link are not created.

### Versions used

Version      3.16.0

### Additional context

_No response_

### Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Yeh that definitely works. I'm curious is there a way though to do that transition w/o doing a pulumi refresh first? The issue being the pulumi refresh destroys the initial set of integrations and then is created on the pulumi up. The idea being i'm hoping to have a smooth service transition with minimal to no interruption, but maybe that isn't possible

Forgot to add that if I do a refresh and then up, some security groups ingress rules are incorrectly destroyed.

Hi @ewucc - thank you for notifying us of this bug!
I believe this is similar to #2075. You can add a deleteBeforeReplace to your apigatewayIntegration while we work on a fix for this.

Hey @guineveresaenger Just wanted to let you know that I got a chance to finally try this out and it seems I am still having the same issues. Even if I add the delete before replace, instead, all the integrations and methods using lambdas are simply deleted and not replaces with the new ones using ECS


refactor/perf : Use new idle camera disabling native

See the natives documentation for DisableIdleCamera

Can't seem to find it as well :/

Author

I'm going to update the OSS. :)


Fix broken template image support in Tray icon

Between Electron 0.30.x and 0.37.x, the tray icon stopped automatically inverting template images when highlighted. NSImageView normally uses the correct color for template images magically, but I think the addition of event handlers in the container view prevents the image view from determining highlight state.
This PR switches to drawing the image manually. The drawRect function decides whether to use image or alternateImage(pressed image) and then if that image is marked as a template, it fills it with the same color used for the text before drawing it.
I've verified this in a couple tests, not sure if more are needed:

Tray with template image
Tray with non-template image, template pressed image
Tray with non-template image, non-template pressed image

Also checked that "dark menu bar mode" on Mac OS X still works as expected.

[Related to https://github.com/nylas/N1/issues/1824]

Thanks!


"Cotu Ciorii", "Lunca" and "Vizireni" are listed in the drop-down menu

Description:
"Cotu Ciorii", "Lunca" and "Vizireni" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Cotu Ciorii" in the search bar.

Expected results
The location is listed in the drop-down menu as "Cotu Ciorii, Buzău(C.A. Rosetti)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Lunca" in the search bar.

Expected results
The location is listed in the drop-down menu as "Lunca, Buzău(C.A. Rosetti)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Vizireni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Buzău, Buzău(C.A. Rosetti)".