【sqlsmith_master_release_02】be crash

Steps to reproduce the behavior (Required)
select
  case when ref_2.s_region is not NULL then ref_1.id_smallint else ref_1.id_smallint end
     as c0,
  ref_0.p_size as c1,
  ref_0.p_type as c2,
  ref_4.r_reason_desc as c3,
  ref_0.p_size as c4,
  ref_4.r_reason_sk as c5,
  ref_3.col_binary as c6,
  cast(coalesce(ref_2.s_phone,
    cast(null as VARCHAR)) as VARCHAR) as c7,
  40 as c8,
  ref_2.s_address as c9,
  ref_3.col_timestamp as c10,
  rand() as c11,
  ref_0.p_partkey as c12,
  ref_1.id_int1000 as c13
from
  sqlsmith.hive_ssb_100g_csv_part as ref_0
          left join sqlsmith.es_without_null as ref_1
          on (ref_0.p_category = ref_1.id_char )
        inner join sqlsmith.hive_ssb_100g_csv_supplier as ref_2
        on (ref_0.p_type = ref_2.s_name )
      inner join sqlsmith.hive_external_test as ref_3
      on (ref_1.id_float = ref_3.col_float )
    inner join sqlsmith.tpcds_100g_reason as ref_4
    on (ref_3.col_binary = ref_4.r_reason_id )
where ref_3.col_float is NULL
limit 41;

Expected behavior (Required)
query success
Real behavior (Required)
be crash
could not find method getStackTrace from class (null) with signature (Ljava/lang/Throwable;)Ljava/lang/String;
FileSystem: loadFileSystems failed error:
(unable to get root cause for java.lang.NoClassDefFoundError)
(unable to get stack trace for java.lang.NoClassDefFoundError)
getJNIEnv: getGlobalJNIEnv failed
*** Aborted at<PHONE_NUMBER> (unix time) try "date -d @1656678176" if you are using GNU date ***
PC: @     0x7f636f904641 __strlen_sse2_pminub
*** SIGSEGV (@0x0) received by PID 6891 (TID 0x7f62ec8fc700) from PID 0; stack trace: ***
    @          0x404cad2 google::(anonymous namespace)::FailureSignalHandler()
    @     0x7f6370dd1235 os::Linux::chained_handler()
    @     0x7f6370dd6031 JVM_handle_linux_signal
    @     0x7f6370dc90c8 signalHandler()
    @     0x7f63702805f0 (unknown)
    @     0x7f636f904641 __strlen_sse2_pminub
    @          0x3411694 starrocks::get_hdfs_err_msg()
    @          0x33c5e83 starrocks::HdfsFsCache::get_connection()
    @          0x32dad4c starrocks::HdfsFileSystem::new_random_access_file()
    @          0x32db469 starrocks::HdfsFileSystem::new_random_access_file()
    @          0x350ca59 starrocks::vectorized::HdfsScanner::open()
    @          0x34bef41 starrocks::connector::HiveDataSource::_init_scanner()
    @          0x34c13a3 starrocks::connector::HiveDataSource::open()
    @          0x28e1dd6 starrocks::pipeline::ConnectorChunkSource::prepare()
    @          0x28db169 starrocks::pipeline::ScanOperator::_pickup_morsel()
    @          0x28db3bd starrocks::pipeline::ScanOperator::_try_to_trigger_next_scan()
    @          0x28db4ee starrocks::pipeline::ScanOperator::pull_chunk()
    @          0x290f14a starrocks::pipeline::PipelineDriver::process()
    @          0x2904beb starrocks::pipeline::GlobalDriverExecutor::_worker_thread()
    @          0x2183ca5 starrocks::ThreadPool::dispatch_thread()
    @          0x217f10a starrocks::Thread::supervise_thread()
    @     0x7f6370278e65 start_thread
    @     0x7f636f89388d __clone
    @                0x0 (unknown)

StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

+----------------------+
| current_version()    |
+----------------------+
| MAIN-RELEASE 7e6dc68 |
+----------------------+

https://github.com/StarRocks/starrocks/pull/8201


bug in _g_l_y_f table while reading ttf font

fonttools/Lib/fontTools/ttLib/tables/_g_l_y_f.py", line 925, in draw
coordinates = coordinates + (offset, 0)
TypeError: unsupported operand type(s) for +: 'GlyphCoordinates' and 'tuple'

do you need more info?
could be solved with:
if offset:
    coordinates.extend([(offset, 0)])

thanks

that offset argument of Glyph's draw method, what is it supposed to do? Translating the coordinates (along the x axis only?) by a specified amount?
if that is the case, then line 925 could be changed from:
coordinates = coordinates + (offset, 0)

to:
coordinates.translate((offset, 0))

GlyphCoordinates has a translate method which accepts a (x,y) tuple.

that offset argument of Glyph's draw method, what is it supposed to do? Translating the coordinates (along the x axis only?) by a specified amount?

Yes.  In TrueType, the lsb from hmtx table and the xmin from the glyph are subtracted to result in an offset to be applied to the outline.

if that is the case, then line 925 could be changed from:
coordinates = coordinates + (offset, 0)
to:
coordinates.translate((offset, 0))

Right.  My bad.  Either that, or implement addition for the GlyphCoordinates object.  I think I'll take your fix since it's trivial.

Pushed a fix.  Please test.

thanks!


[Fix] Remove duplicate hintText property

I have made a PR #979 to prevent this type of PR in the future

:+1:

Thanks @patrykkopycinski


Turn into real package

all modules are top-level. This isn't very good for avoiding name collisions, so it should be under a raylib package.
Plan is to put all under a raylib package, with package.d containing the current raylib.d contents. raylib_config.d will move to just raylib/config.d

Now have a package, with deprecated original modules that publicly import the correct ones.


ignore cmake-build-debug generated by CMake in CLion IDE

Reasons for making this change:

The cmake-build-debug directory is typically generated by the CLion IDE (or other IDEs that use CMake) for managing build files in debug mode. Here are some common reasons to ignore it in version control:

Auto-generated files: The cmake-build-debug folder contains generated files from CMake that are used to compile your project in debug mode. These files can be easily regenerated from source, making them unnecessary for version control.

Platform-specific files: The build directory often includes platform-specific binaries, paths, and configurations, which might vary across different development environments (e.g., different operating systems, compilers, or IDE configurations).

Large binaries and object files: The debug build directory frequently holds binaries, object files, and other large intermediate files that can bloat your repository if included in version control. Ignoring these files helps keep your repository lightweight.

Avoiding merge conflicts: Binary files in the build directory can cause unnecessary merge conflicts when multiple people work on the project and each builds locally. Ignoring the build directory reduces the likelihood of these conflicts.

Sensitive configuration information: Some build configurations may include paths or sensitive environment details specific to your local setup. Excluding this directory helps maintain privacy and keeps these configurations local.

Consistency across environments: Different developers may have different setups, so each person's build directory might differ. Ignoring cmake-build-debug encourages each developer to generate their own build files tailored to their local environment.

For these reasons, adding cmake-build-debug to your .gitignore is generally recommended.
Links to documentation supporting these rule changes:
Reference links below

https://intellij-support.jetbrains.com/hc/en-us/community/posts/206020944-What-should-I-gitignore-Can-I-gitignore-cmake-build-debug
https://stackoverflow.com/questions/75274647/which-folders-i-should-put-in-gitignore-file-after-creating-a-project-with-clio

If this is a new template:

Link to application or project’s homepage: TODO

Commented it out like on the Python.gitignore Python.gitignore Pycharm .
I think adding a comment is the best solution instead of keep declining the same PRs over and over again. Thanks.


mempool related url params explained

Documenting rpc-endpoint features developed for rainbow:
useMempool,mempoolRpc, canRevert url parameters to allow proxying txs to mempool and reverting txs
Those are probably not so customer facing features, but mostly for wallet integrators. Should we update our protect button with those new fields?

Incorporated these updates into a larger PR here: https://github.com/flashbots/flashbots-docs/pull/578
If that looks good, we can close this PR in favor of it.

Closing in favor of the linked PR which has been merged


SIG Software Supply Chain Co-chair Nominations

The Software Supply Chain SIG is now opening elections for a new co-chair!
The process will be quite lightweight. Over the next two weeks, please express your interest in becoming the new co-chair of the Software Supply Chain SIG directly under this issue. You may also nominate someone else to be co-chair of the Software Supply Chain SIG but please make sure to check with that person privately before nominating them.
Please include:

Name of the candidate
Their short bio and statement of intent on why they would hold the position
Any other relevant Information

The nomination period will end on Thursday, December 22, 2022.
If we have multiple interested individuals, we will then hold a vote.

🙋🏾 I volunteer.
Brief bio:

Currently Google Engineering Manager for Tekton, a graduated project in the CDF portfolio
Previously Eng Manager for Google Cloud Build
Extensive prior life experience at Goldman Sachs in a variety of roles, including infrastructure and platform engineering and supply chain audits for regulatory compliance

Statement of intent: my goals in co-chairing the SIG include:

deepening and broadening participation across companies and teams
finding areas of OSS collaboration where even competitors can meet and cooperate to the benefit of our industry and the OSS community
recruiting one or more demos for each session, where practitioners share products and feature developments, whether in OSS or in vendor offerings

Relevant Supply Chain SIG participation and contributions:

Established the Supply Chain Maturity Model Workstream under the Supply Chain SIG
I've attended 4 of the 5 SIG meetings since starting my current role in Tekton, including presenting at one meeting and recruiting a Tekton presentation at another.

David Bendory of Google has been unopposed for the position and becomes the new co-chair to the SIG Software Supply Chain.
Thanks a lot for volunteering David!
A pull request is opened to update the readme.


Error free measurement generation

Generating perfect measurement scenarios would assist in algorithm workflow, but navsim/navsim/config.py disallows generation of perfect measurements on line 86
errors = dc.from_dict(data_class=ErrorConfiguration, data=config.get("errors"))

Went ahead and made this a feature in the lastest commit (forgot to link issue). All you need to do is comment out/remove the error you don't want to simulate in your configuration file and that should work.


Visual Studio-style docs for revised IE

Currently the Interaction Engine contains comments in this style:
/**
  * Description of the thing!
  */
  public abstract class Thing : IThing { ... }

Visual Studio doesn't populate the hover tooltip with comments made in this style, so we should document the revised Interaction Engine in this style:
/// <summary>
/// Description of the thing!
/// </summary>
public abstract class Thing : IThing { ... }

This will populate Visual Studio tooltips with the summary information and will be one of our most direct and immediately user-facing source of descriptions and tips for helping developers construct a mental model of the interaction paradigms we use and the layers of abstraction that we provide.
These tooltips should aim to be as concise as possible, so the most useful information for a given task can be provided in the small space of the tooltip as the developer is writing code.

So far this is the case all over leap-gui-interaction.


MyriaL: String literals cannot contain double quotes?

Looking at raco/myrial/parser.py and raco/expression/expression.py, it appears there is no escape syntax for double quotes in string literals.
Wish list: It would be nice to have the choice of single or double quotes a la python.

I have a possible fix in https://github.com/uwescience/raco/commit/7df119ec9b47e5181600e58311e4733fc0bb988a
Since python supports this, I only had to change the MyriaL parser


Quick Action: Add XML documentation from parent class/interface

It would be nice if when my cursor is on a public class member that has no XML documentation, if it is determined that the member is either:
a) Overridden from a parent class
b) Part of an implementation of an interface
If either a) or b) has XML documentation attached to give me a Quick Action option to insert that same XML documentation to the currently focused member (and auto-rename any parameter references in the inserted XML doc if it is possible)

Shouldn't CopyCommentsFromBaseCodeRefactoringProvider provide exactly that functionality? Can you give an example, where this does not work?

I was not aware this refactoring was available. Disregard.


Connects to #358. Dependency updates and search UI refactoring.

Key Changes
Dependency updates (see #358 for more details):

Switched vis-network-react module source from no-longer-maintain repo to a forked repo containing upgraded version of vis-network library
Removed individually installed vis-network and vis-data modules since the forked vis-network-react module contains upgraded versions of these libraries
Converted JSON to JavaScript object for the PASS1B-06's network figure (4e) used on homepage since newer version of  vis-network library supports html markup in tooltip
Removed console.log callback handlers from prior vis-network-react implementation

Search UI refactoring:

The React component responsible for rendering the radio buttons for selecting either gene, protein id, or metabolite names on the Search page contains repetitive JSX markup and produces unreliable interactive behavior
This component has been converted into reusable child components with a parent component.

lgtm


How can I get in broker, the host name that user sets in client to send data

Is there any way to get in broker the URL that client use to send data to broker? If it is possible, is that possible to have it in server start?

Hi,
Do you mean the topic or IP. Please share some code pointing out where exactly you need this information.
Best regards
Christian

Hi Christian,
Thank you for your reply. In client we set the option like this
var options = new MqttClientOptionsBuilder()
    .WithTcpServer("broker.hivemq.com", 1883) // Port is optional
    .Build();

How can access that URL in my Broker? I want to know which url client called. I tried to get it from validationContext in MqttServerOptionsBuilder but there is not such property. How can I get that in broker?
Thank you

Mhh sorry but that is currently not supported in the broker. I planned to add endpoints with custom IDs but this is not yet started.

OK. Thank you.

@TaherianuBeac is that support now??

@chkr1011  hello, iss this support now?

No


Test APIs for functionality

Tested.
Original bike API not functional for our needs/abilities.  Found alternate.
Weather API functional.


Improve workflows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (4fb17bf) to head (acef336).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #386   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           92        92
  Lines         2369      2369
  Branches       213       213
=========================================
  Hits          2369      2369

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


bug(cdk-virtual-scroll): Scrolling fast makes the viewport blank until scrolling stops

Reproduction
https://stackblitz.com/edit/angular-qxcwbr
Steps to reproduce:
There are two virtual scroll viewports - the first has the problem. Take the scroll bar with the mouse and scroll down fast. The whole viewport will be blank as long as you scroll.
The second viewport uses a bizarre fix and if you try the same it will magically work.
Expected Behavior
Both of the viewports should scroll without viewport going blank
Actual Behavior
Without the bizarre fix the viewport becomes blank while fast scrolling
Environment

Angular: 10.2.2
CDK/Material: 10.2.7
Browser(s): Chrome Version 86.0.4240.111 (Official Build) (64-bit)
Operating System (e.g. Windows, macOS, Ubuntu): Windows 10

How did you find out about the "fix":

How did you find out about the "fix":

Because it didnt have the problem in another app and after i investigated i saw cdk overlay "fixed" it. And that's how i got to it.

This is actually done intentionally so that we're not constantly rendering new elements while the user is rapidly scrolling which would be detrimental to performance. However, there is probably some room to improve this by showing some kind of lightweight placeholder elements so the user doesn't think its lagging.

@mmalerba The lightweight placeholder idea is a great idea! Hope it can be added as opt in. The only mystery left is the so called fix using the absolute positioned div

@mmalerba The lightweight placeholder idea is a great idea! Hope it can be added as opt in. The only mystery left is the so called fix using the absolute positioned div

This seems to be Chrome related
I was able to reproduce the issue using your StackBlitz demo in Chrome, but in Firefox and Safari it works fine

@mmalerba The lightweight placeholder idea is a great idea! Hope it can be added as opt in. The only mystery left is the so called fix using the absolute positioned div

This seems to be Chrome related
I was able to reproduce the issue using your StackBlitz demo in Chrome, but in Firefox and Safari it works fine

This is actually done intentionally so that we're not constantly rendering new elements while the user is rapidly scrolling which would be detrimental to performance. However, there is probably some room to improve this by showing some kind of lightweight placeholder elements so the user doesn't think its lagging.

It would be great to be able to configure which strategy the virtual scroll uses to deal with this, as it seems many users experience the view to be slower when they see this visual pop-in effect (though I understand the total time spent by the render thread is much lower)

How did you find out about the "fix":

This is a really weird but helpful fix to this issue. Thanks @Harpush !
Also noticed in your example, if I remove the style="position: relative" from the second  box then the blank viewport issue doesn't occur in first box as well.
Here is the sample - https://stackblitz.com/edit/angular-qxcwbr-lyk95e?file=src%2Fapp%2Fcdk-virtual-scroll-context-example.html
Anyways, there must be some kind of explanation for this behaviour and proper workaround will be helpful and mostly useful.

what a bizarre fix indeed but it sure did fix that.. it was driving me crazy. grateful for the share here!
similar to that... any one noticed that the cdk virtual on a dropdown (I have it on a custom data table dropdown - once you scroll the list and make a choice - them reopen the dropdown - the list is blank until you start scrolling the list...  any fixes anyone know of?
as soon as you start scrolling the list shows up

How to fix the "fix" in chrome?
In our senary, we need both the "white screen" effect and the fixed element to lay over the list as a wartermark, but the wartermark erases the white screen effect, whch leads to the list scrolling stuck since before the item being fully rendered, the scroll process seems to be interrupted. but with the white screen effect, even though the cost of rendering item is heavy,  the scrolling is always smoothy and the item will becom white and will finally be renderred after sometime, this is prefered.
scroll with white screen(prefered)
https://user-images.githubusercontent.com/3808838/196651792-d8fd6b67-faa8-4177-be54-4a7dc99064c1.mp4
scroll without white screen( stuck, not prefered, but we do need the fixed wartermark)
https://user-images.githubusercontent.com/3808838/196651975-3a95181e-d060-4fae-a67a-5c4eb1d185d0.mp4

what a bizarre fix indeed but it sure did fix that.. it was driving me crazy. grateful for the share here!
similar to that... any one noticed that the cdk virtual on a dropdown (I have it on a custom data table dropdown - once you scroll the list and make a choice - then reopen the dropdown - the list is blank until you start scrolling the list... any fixes anyone know of?
as soon as you start scrolling the list shows up
doesnt happen everytime but does happen more than 50% of the time on a large list

What is currently working for me in this situation: repeated opening of dropdown presents blank content until re-scrolled, is to call the QueryList first item, call scroll on that item, the set scrolltop of the scroll container to 0. Then wrap it up in a friendly "force it to the bottom of the call stack" setTimeout.
setTimeout(() =>
{
const element = this.optionsList.find((option, index) => index === 0);
element.nativeElement.scrollIntoView();
this.scrollListElement.nativeElement.scrollTop = 0;
}):
When the dropdown opens again, the content is there and I do note a slight appearance of the scroll bar appearing in the top right corner for a brief moment, but the contents is stable and you don't see it actually scroll even if the user scrolled before opening it up the second time.
Note: This was working a year ago with just setting the scrolltop to 0, but as of 9/9/2023 it now requires the above and in that order.

Running Angular v17 the div at the bottom fix is not working - any other recommendations?
Or could we display some sort of a placeholder until the list renders the items until the scrolled location?

Running Angular v17 the div at the bottom fix is not working - any other recommendations? Or could we display some sort of a placeholder until the list renders the items to the scrolled location?

Do you find some solutions for this problem with Angular 17?


feat(presskit): angular logo press kit

Changes

Add a press kit page to angular.io

Preview

Only way to access for now: https://angular-io-presskit.firebaseapp.com/presskit.html

Known issues

Currently not accessible via navigation, this will be fixed with https://github.com/angular/angular.io/issues/1120 which is next.
If there is a place to put the presskit with the current navigation, I can make the addition, however, that is very soon to change. @naomiblack

CLOSES:
https://github.com/angular/angular.io/issues/1121
@naomiblack @petebacondarwin

merged and live!


Layer colour updates

link to demo deployment of your smk based app is
https://smk-clean-bc-rt-b16795-dev.apps.silver.devops.gov.bc.ca
Verify that it looks good, if so merge the pull request which will trigger a prod deployment, otherwise close the pull request without merging
mentions / notifications: @michaelpnelson, @rustyjux

The Vanity URL for the dev deploy is https://smk-clean-bc-api-gov-bc-ca.test.api.gov.bc.ca
If this is a dev deploy,  verify that it looks good, then merge the pull request to trigger prod deployment.
If this is a prod deployment, view your prod deploy at the url above :)
mentions / notifications: @michaelpnelson, @rustyjux

it happened! my unrequested 2c:

the white/red and red/white for fast charging is confusing. taking the regular charging station (red/white) and adding something to it vs inverting would be much easier to interpret
the symbol for a building should be fundamentally different from the other symbols (which are all charging stations)
♥


Added missing getters to fix an EXC_BAD_ACCESS crash in RN 0.23.0

Issue details can be found here: https://github.com/facebook/react-native/issues/6677

Awesome ! Thanks for the contribution @mpretty-homepass  !
I'm going to merge it right now


The cypress npm package is installed, but the Cypress binary is missing

Hello, I've been trying to work out how to fix this issue within the github CI. I've tried several different methods, especially coming from this issue.
Here is my current yml file:
name: 🧪 Apps e2e

on:
  pull_request:
    types: [opened, reopened, synchronize, ready_for_review]
  workflow_dispatch:
    inputs:
      run_on_firefox:
        description: Include Firefox browser
        required: false
        default: false
  push:
    branches:
      - master

concurrency:
  group: ${{ github.workflow }}-${{ github.head_ref || github.ref_name || github.ref }}
  cancel-in-progress: true

jobs:
  runTests:
    name: 🥒 E2e tests (cypress)
    if: github.event.pull_request.draft == false || github.event_name == 'workflow_dispatch'
    runs-on:
      - self-hosted
      - yarn-large
    strategy:
      max-parallel: 2
      fail-fast: false
      matrix:
        browser:
          - name: Chrome
            test_run: pnpm test:functional:run

    steps:
      - name: Checkout
        uses: actions/checkout@v3

      - name: Use latest node and pnpm versions
        uses: ./.github/workflows/composites/lemo
        with:
          working-directory: ./apps
          with-install: true

      - name: Cache Cypress binary
        id: cache-cypress-binary
        uses: actions/cache@v3
        with:
          path: ~/.cache/Cypress # https://docs.cypress.io/guides/references/advanced-installation#Binary-cache
          key: ${{ runner.os }}-cypress-binary-${{ hashFiles('apps/e2e/package.json') }}

      - name: Install System dependencies
        run: |
          sudo ...
          sudo ...

      - name: Run e2e tests for ...
        if: always()
        timeout-minutes: 30
        working-directory: ./apps
        run: ${{matrix.browser.test_run}} ...

Here is the error I'm getting in the CI:
cypress run --browser chrome "--spec=cypress/e2e/app1"

The cypress npm package is installed, but the Cypress binary is missing.

We expected the binary to be installed here: /home/runner/.cache/Cypress/13.9.0/Cypress/Cypress

Reasons it may be missing:

- You're caching 'node_modules' but are not caching this path: /home/runner/.cache/Cypress
- You ran 'npm install' at an earlier build step but did not persist: /home/runner/.cache/Cypress

Properly caching the binary will fix this error and avoid downloading and unzipping Cypress.

Alternatively, you can run 'cypress install' to download the binary again.

@AndyLukac
I don't see you using cypress-io/github-action in your workflow. Did you miss this out from your posting or aren't you using it?
The best advice is to add pnpm cypress install into your workflow. If the Cypress binary is already installed, then it skips the installation. If it is not installed, then it installs it.
You should key off the lock file, not package.json.
You should check if you should be using actions/cache@v4 not v3. You may otherwise get a deprecation warning.

Hi @MikeMcC399,
thanks for your prompt reply.
I managed to figure it out in the end, thanks!

@AndyLukac

I managed to figure it out in the end, thanks!

Well done! I will close your issue now as I understand you have been able to resolve it.


docs: update the "Submitting a Pull Request" section of the "contributing" docs

Removes the requirement to add an entry in the "Unreleased" section of the Changelog document as we will be taking care of the Changelog separately as part of our new release process.
I used the opportunity to tidy up a little bit the text in the "Submitting a Pull Request" section.
@acoulton I think it is better if this reviewed by a native English speaker (I am not) so would be grateful if you could take a look

These changes look good to me


QuickSwap — staking MATIC/QUICK doesn't work after approval (which works)

Unable to unstake on quickswap on the latest iOS release. We tried iphone, ipad, inapp, in chrome, in safari. Nothing works. Same on Android

Works with MetaMask

Stacking works.


copr: Support VECTOR for RowFormatv1

What is changed and how it works?

Issue Number: ref #17290

What's Changed:
When CSV data is loaded via LOAD DATA statement, it turns out that row is encoded using Format v1:
LOAD DATA LOCAL INFILE '/Users/breezewish/Downloads/backup_llama_index_rag_test/test.llama_index_rag_test.0000000010000.csv' INTO TABLE `llama_index_rag_test` FIELDS TERMINATED BY ',' ENCLOSED BY '"' LINES TERMINATED BY '\r\n' IGNORE 1 LINES;

And it causes problems:
mysql> select * from llama_index_rag_test limit 10;
ERROR 1105 (HY000): invalid data type: unsupported data type `20`

This PR make it work.
Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

/retest

/retest

/test idc-jenkins-ci/test

/retest

/ok-to-test


Improve UI of Book Card

Description

Reduced the amount of text to be displayed on each book tile
#148

Affected Dependencies
N/A
How has this been tested?
Using flake8, python manage.py test.
No test fails.
GIF Before

GIF After

Checklist

[✓ ] I have followed the Contribution Guidelines.
[✓ ] My changes do not edit/add any unrequired files.
[✓ ] My changes are covered by tests.

@Sonaljain6431 Unstage settings.py

Ok

Is it ok now?

LGTM, @Sonaljain6431
We can add a issue for tooltip for all the book names and author title wdyt @maneprajakta or maybe this can be taken by the same PR since it's Level 2

@Praful932
Yes we can it will help for getting complete title of book and author name.
I think we should go for another issue for this one.

Do I need to make more changes?

Do I need to make more changes?

Nope, we can merge this :)


feat: navigation (resolves #14)

[x] Mobile navigation
[x] Desktop navigation

@cherylhjli: in regards to the question I asked -- this is when current tab is Practices and focus is on Activities.

@chosww yes, it should show current page when hovering or focusing on another link. I'll update the focus state so it looks less funny.

@chosww I believe all the review comments are addressed now.


RabbitMQ allowing specific workers to be assigned to specific jobs.

This will require adding API keys which will be distributed to users when they want to host their own instances of our workers.


Add Cache Hit Ratio and Index Hit Ratio to Postgres agent

My apologies, I could not get the Vagrant box up and running so this was not tested. If there are instructions beyond what is in the Wiki I would be happy to use them and test.
This PR is to add Cache Hit Ratio and Index Hit Ratio to the Postgres agent.

Found an error in my thinking. Closing this.


KerasLayer with ELMo

Hi, is it possible to load the ELMo module using tensorflow hub and return the full ELMo embeddings? I would like to extract the "elmo" embeddings which returns contextual embeddings for each token. I need to make as_dict=True, but KerasLayer doesn't seem to allow it. I am also trying to avoid using Lambda layers because of issues with serialization.
Code to reproduce
tokens = layers.Input(shape=[None], dtype=tf.string)
seq_lens = layers.Input(shape=[], dtype=tf.int32)

elmo = hub.KerasLayer(
    "https://tfhub.dev/google/elmo/3",
    trainable=True,
    signature="tokens",
)

out = elmo({"tokens": tokens, "sequence_len": seq_lens})
model = keras.Model(inputs=[tokens, seq_lens], outputs=out)
model.compile("adam", loss="sparse_categorical_crossentropy")
model.summary()

Model summary
__________________________________________________________________________________________________
Layer (type)                    Output Shape         Param #     Connected to
==================================================================================================
input_2 (InputLayer)            [(None,)]            0
__________________________________________________________________________________________________
input_1 (InputLayer)            [(None, None)]       0
__________________________________________________________________________________________________
keras_layer (KerasLayer)        (None, 1024)         93600852    input_2[0][0]
                                                                 input_1[0][0]
==================================================================================================
Total params: 93,600,852
Trainable params: 0
Non-trainable params: 93,600,852

Package versions
tensorflow==1.15.0
tensorflow-estimator==1.15.1
tensorflow-hub==0.7.0

trainable=True doesn't work.
Separately, trainable=True doesn't seem to do much. I would expect 4 weights to be trainable, but none of them are.
Thanks

Ah, there is an output_key option 😅
    elmo = hub.KerasLayer("https://tfhub.dev/google/elmo/3",
                          trainable=True,
                          signature="tokens",
                          output_key="elmo")

@tokestermw  according to the official page: https://www.tensorflow.org/hub/api_docs/python/hub/KerasLayer I didn't see any information about the initial params you used (signature, output_key ..). May you tell me where did you find out this use case?

@Zachary4biz In the source code for KerasLayer: https://github.com/tensorflow/hub/blob/master/tensorflow_hub/keras_layer.py#L117


TypeError: 'tuple' object is not callable when using PReLU

Please make sure that the boxes below are checked before you submit your issue. Thank you!

[x] Check that you are up-to-date with the master branch of Keras. You can update with:
pip install git+git://github.com/fchollet/keras.git --upgrade --no-deps

[x] If running on Theano, check that you are up-to-date with the master branch of Theano. You can update with:
pip install git+git://github.com/Theano/Theano.git --upgrade --no-deps

[x] Provide a link to a GitHub Gist of a Python script that can reproduce your issue (or just copy the script here if it is short).

Upon executing the following piece of code:
model = Sequential()
model.add(Dense(512, input_shape=(43,), init='glorot_uniform'))
model.add(PReLU((512,)))
I get the following error
TypeError: 'tuple' object is not callable

Don't know why(512,) in the arguments, just remove it.
Please make good use of document/code and read the definition first.
keras.layers.advanced_activations.PReLU(init='zero', weights=None)


Question: Would there be some problem if one signal handler is triggered during the call of acosw?

https://github.com/hnes/libaco/blob/c941da6bb94a3800bf014722b7a248795f2b7ee2/acosw.S#L65
按照我的理解，这一行之前的: lea     ecx,[esp+0x4]               // esp， 通过将栈顶指针暂时保存在ecx中是作为一个定点的作用吧，因为前面注释里已经说了，signal可能改变栈顶指针esp的值，那这一行又用esp本身地址+8的方式取得to_co的地址，那万一在这之前esp的值由于signal改变了，那此时ecx里保存的值就不是to_co的地址了吧，不知道我这样配合你上面的注释理解得对不对。探讨一下...

Hi @willmafh ;-)

Signals can interrupt processes [see signal(BA_OS)]. Functions called during signal handling have no unusual restrictions on their use of registers. Moreover, if a signal handling function returns, the process resumes its original execution path with registers restored to their original values. Thus, programs and compilers may freely use all registers without the danger of signal handlers changing their values.

-- abi386-4.pdf

So it would be fine as long as we comply with the constraint described in the "Stack Pointer - Miscellaneous - Proof of Correctness" part of README.md:

The stack pointer should always points to the end of the latest allocated stack frame.

P.S.
Here is also a detailed description about the execution of signal handlers:

https://stackoverflow.com/questions/6949025/how-are-asynchronous-signal-handlers-executed-on-linux

I also found the description about the execution of signal handler in the man2 page of sigreturn is wonderful:

If the Linux kernel determines that an unblocked signal is pending
for a process, then, at the next transition back to user mode in that
process (e.g., upon return from a system call or when the process is
rescheduled onto the CPU), it creates a new frame on the user-space
stack where it saves various pieces of process context (processor
status word, registers, signal mask, and signal stack settings).

The kernel also arranges that, during the transition back to user
mode, the signal handler is called, and that, upon return from the
handler, control passes to a piece of user-space code commonly called
the "signal trampoline".  The signal trampoline code in turn calls
sigreturn().

This sigreturn() call undoes everything that was done—changing the
process's signal mask, switching signal stacks (see
sigaltstack(2))—in order to invoke the signal handler.  Using the
information that was earlier saved on the user-space stack
sigreturn() restores the process's signal mask, switches stacks, and
restores the process's context (processor flags and registers,
including the stack pointer and instruction pointer), so that the
process resumes execution at the point where it was interrupted by
the signal.

You also could choose to use gdb to see how signal handler is executing during the call of acosw (here is a demo may be helpful). Happy hacking :-)


Update performance test and README to include more fair comparison with pubchempy

This PR makes two small changes -

Updates the compound list in test_performance.py to include only 100 molecules that are known to be translateable using both OPSIN and PubChem. The previous list of compounds had many species that couldn't be resolved using PubChem, resulting in a potentially unfair comparison between the two tools.  Also, the compound list was fairly long - ideally we want to keep the test list as short as possible so as to not spam the PubChem server.

Updates the README to reflect this change, and modifies the language describing the comparison. In particular, the speedup will depend on the size of the input. py2opsin has a non-negligible startup time, and if you are only translating 1 species at a time, py2opsin performance is actually comparable to querying PubChem. The real speedup comes from the batch translations, which is now indicated in the README.

I think this might actually be a limitation of the GitHub runners - they are notoriously weird. On the second run, it passed.

Ah, I spoke too soon - great!


Add Password Generator

Add A Random Password Generator App.
Issue #181
####Description
This is a random password generator app.
User can customize their password by password length and using special symbol, number etc.
Using copy button user can copy the password in clipboard.
####Screenshort

@DhanushNehru  Please review my PR.


⚠ Bofrost.fr has degraded performance

In 39b3202, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1963 ms

Resolved: Bofrost.fr performance has improved in 3c67ea8.


Font Variation Popup Menu is not functional in Text Inspector

Apple Feedback Assistant ID: FB13434977
DESCRIBE THE BUG:
The font variation popup menu is not functional in the Text Inspector in Final Cut Pro 10.7.

TO REPRODUCE:

Create a new library
Create a new project/timeline
Add a Centered Title
Select the Text Inspector tab in the Inspector
Select any font
Select the font variation popup - you won't be able to select a variation

EXPECTED BEHAVIOUR:

You should be able to select a font variation

SCREENSHOTS:

SPECS:

2021 16-inch MacBook Pro
macOS Sonoma 14.1.2
Final Cut Pro 10.7

Incan confirm this bug here.


bug: virtualScroll doesn't work inside a div

Type: bug
Ionic Version: 2.x
Platform: all
When the virtualScroll  is inside a div the list doesn't show.
GitHub url :
https://github.com/edcampos/ionicVirtualScroll

This is my configuration:
Cordova CLI: 6.4.0
Ionic Framework Version: 3.4.2
Ionic CLI Version: 2.2.2
Ionic App Lib Version: 2.2.1
Ionic App Scripts Version: 1.3.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v8.1.2
Xcode version: Not installed
I'm using Angular 4

Removing the div in contacts.html (src\pages\contacts\contacts.html), the list appear, but I need to put the virtual scroll inside many divs.

Hello! Thanks for opening an issue with us. I am going to close this issue as a duplicate of https://github.com/ionic-team/ionic/issues/10148 as this is caused by the same issue. I will also update that issue with the info that this not only happens when virtualscroll is nested in a menu, but when it is nested in a div.


Use rollup

We need a vanilla benefit require as well as the ability to require from benefit/react as a separate file to get react helpers from.
Example:
import { benefit } from "benefit"

-or-
import { jsx } from "benefit/react"

To support different input/output files, I'm going to experiment with using rollup in place of microbundle

https://github.com/rollup/rollup

You could do different files (index.js vs. react.js) which would allow this.
Both will probably export some of the same utilities, but only benfit/react would export jsx.

I don't think this is needed any longer now that microbundle is bundling multiple entry points 😄


Can't find ttf file for font Noto Sans CJK TC

Can't find ttf file for font Noto Sans CJK TC
Why?

Did you check under all of the sofa cushions?

What are the sofa cushions?


[ERROR: No video formats found]

Checklist

[x] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[ ] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Description
I am using python version of youtube-dl which is made into music bot and after using it for longer time i found several videos without video formats:
https://www.youtube.com/watch?v=l9VFg44H2z8&ab_channel=5FDPVEVO
https://www.youtube.com/watch?v=2Ay1V9uXI7c
https://www.youtube.com/watch?v=nxXcuDAv7Ss
https://www.youtube.com/watch?v=SNhOgS3BHjw&ab_channel=NFVEVO
https://www.youtube.com/watch?v=K533gW3boIY&ab_channel=GEazyMusicVEVO
https://www.youtube.com/watch?v=Q3ilnKA56YY
https://www.youtube.com/watch?v=UDVtMYqUAyw
https://www.youtube.com/watch?v=LwzBeVS8Fi0&ab_channel=XXXTRAKTOR-Topic
https://www.youtube.com/watch?v=jF8J76-ZfeA&ab_channel=IGOCIG
https://www.youtube.com/watch?v=nMDOSLEVdBw
https://www.youtube.com/watch?v=KnumAWWWgUE&ab_channel=PlayboiCarti
https://www.youtube.com/watch?v=a0DbzUe-r4Q&ab_channel=Fazlija
Also i have question: is there any possibility to play video that needs account to confirm age?

Works fine:
PS C:\> youtube-dl 2Ay1V9uXI7c
[youtube] 2Ay1V9uXI7c: Downloading webpage
[youtube] 2Ay1V9uXI7c: Downloading player 449ea0a5
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
[download] Destination: Five Finger Death Punch - Wrong Side of Heaven (Official Audio)-2Ay1V9uXI7c.f248.webm
[download]   2.3% of 17.65MiB at 74.04KiB/s ETA 03:58

i don't know why it doesn't work for me, does it have to do something with country region?

Your system is messed up. It works fine in all countries:
https://unblockvideos.com/youtube-video-restriction-checker?u=#url=2Ay1V9uXI7c
I assume you got IP blocked for abusive downloading. Next time you should put a delay between downloads.

i mean all other videos work, and bot is on cloud hosted platform

Can you try this request:
POST /youtubei/v1/player HTTP/1.1
Host: www.youtube.com
X-Goog-Api-Key: AIzaSyAO_FJ2SlqU8Q4STEHLGCilw_Y9_11qcW8

{
 "videoId": "2Ay1V9uXI7c",
 "context": {
  "client": {
   "clientName": "ANDROID",
   "clientVersion": "17.12.34"
  }
 }
}

What request, i tried that video already and it didn't work for me

what OS are you on

it's on heroku cloud, they say all bots are running on Ubuntu

If you want, you can try this program:
https://github.com/ytdl-org/youtube-dl/files/8403102/2Ay1V9uXI7c.zip
should work on linux. I included the source code. See what output you get.

yo, it's python discord bot on heroku cloud as i said in description, it plays music in discord channels and some urls doesn't work and they need to be fixed

I am trying to help you. I literally wrote a program and cross compiled it for your operating system, just to make troubleshooting easier for you.

some urls doesn't work

Yeah, they do, as I already showed twice.

and they need to be fixed

this is a free project, so no one owes you anything. If you want to fix it, you can do it yourself, or beg someone here to do it.

I am trying to help you. I literally wrote a program and cross compiled it for your operating system, just to make troubleshooting easier for you.

i am sorry but please next time read description properly before helping me
there are same issues with "no video formats found" where people put youtube urls, so i did same
and i didn't mean "they need to be fixed" as i am forcing ytdl team to fix it, i meant it in way it should be noted and rewieved if it's false request or not

I am using python version of youtube-dl which is made into discord music bot and after using it for longer time i found several videos without video formats:

All these links give formats for me. Are you looking for some particular format?
Also, you need the git master to return download links that work at full speed.

Also i have question: is there any possibility to play video that needs account to confirm age?

There has been but apparently the methods used to bypass age verification are no longer valid. Investigation proceeds.

Investigation proceeds.

https://github.com/ReVancedTeam/revanced-suggestions/issues/15#issuecomment-1085480029
There is hope after all.

Also, you need the git master to return download links that work at full speed.
I don't understand that, can you explain it to me?

If you repeatably fail to get a format for some supported YT URL, as usual we need a verbose command-line log to start debugging the issue.

ytdl options:
ytdlopts = {
    'format': 'bestaudio/best',
    'outtmpl': 'downloads/%(extractor)s-%(id)s-%(title)s.%(ext)s',
    'restrictfilenames': True,
    'noplaylist': True,
    'nocheckcertificate': True,
    'ignoreerrors': False,
    'logtostderr': False,
    'quiet': True,
    'no_warnings': True,
    'default_search': 'auto',
    'source_address': '<IP_ADDRESS>',
    'http_persistent': False,
    'cookiefile': 'cookies.txt'
}

and verbose
[debug] Encodings: locale cp1250, fs utf-8, out utf-8, pref cp1250
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.9.9 (CPython) - Windows-10-10.0.19044-SP0
[debug] exe versions: ffmpeg 4.4-essentials_build-www.gyan.dev, ffprobe 4.4-essentials_build-www.gyan.dev
[debug] Proxy map: {}
Cogs Loaded!
Started
{'webpage_url': 'https://www.youtube.com/watch?v=2Ay1V9uXI7c', 'requester': <discord.ext.commands.context.Context object at 0x0000022062593F70>, 'title': 'Five Finger Death Punch - Wrong Side of Heaven (Official Audio)', 'duration': '04m 31s'}
{'webpage_url': 'https://www.youtube.com/watch?v=2Ay1V9uXI7c', 'requester': <discord.ext.commands.context.Context object at 0x0000022062811CD0>, 'title': 'Five Finger Death Punch - Wrong Side of Heaven (Official Audio)', 'duration': '04m 31s'}
{'webpage_url': 'https://www.youtube.com/watch?v=l9VFg44H2z8', 'requester': <discord.ext.commands.context.Context object at 0x0000022062832700>, 'title': 'Five Finger Death Punch - Wash It All Away (Explicit)', 'duration': '03m 59s'}
{'webpage_url': 'https://www.youtube.com/watch?v=UDVtMYqUAyw', 'requester': <discord.ext.commands.context.Context object at 0x0000022062832B80>, 'title': 'Interstellar Main Theme - Extra Extended - Soundtrack by  Hans Zimmer', 'duration': '12m 55s'}
{'webpage_url': 'https://www.youtube.com/watch?v=nMDOSLEVdBw', 'requester': <discord.ext.commands.context.Context object at 0x000002206283FEE0>, 'title': 'Playboi Carti - @ MEH [Official Video]', 'duration': '02m 48s'}
{'webpage_url': 'https://www.youtube.com/watch?v=a0DbzUe-r4Q', 'requester': <discord.ext.commands.context.Context object at 0x000002206283FC40>, 'title': 'FAZLIJA - HELIKOPTER (OFFICIAL VIDEO) prod. by JOKER', 'duration': '02m 54s'}

So somehow videos work again, did u do something to fix it?

No. Perhaps there was a network issue at your hoster, or just https://github.com/ytdl-org/youtube-dl/issues/30805#issuecomment-1086682940.


Add completions-proxy example

Demonstrates listening for completions and chat/completions requests on localhost, forwarding them to openai, then printing streamed responses before passing back to local client.

Vague plan is to leverage local.ai for serving local models and have support for remote providers as well, this example was me playing around with what would be involved in forwarding openai streaming requests.

Hi @tot0
Thank you for the PR and interesting example.
So far the examples in this repo are self contained to demonstrate the use of available OpenAI APIs along with their dedicated READMEs.
The example in this PR lean towards an application built on top, the direction that you wanna go with your plan suggests to me that  having this example in its own repository might be a better choice?

Unsure how soon such a future application will materialize so I just wanted
to push the changes at least and create something that helps me remember I
made them :)
If you don’t think they should live in this repo though that’s reasonable
and I can close this.
On Mon, May 22, 2023 at 21:09 Himanshu Neema @.***>
wrote:

Hi @tot0 https://github.com/tot0
Thank you for the PR and interesting example.
So far the examples in this repo are self contained to demonstrate the use
of available OpenAI APIs along with their dedicated READMEs.
The example in this PR lean towards an application built on top, the
direction that you wanna go with your plan suggests to me that having this
example in its own repository might be a better choice?
—
Reply to this email directly, view it on GitHub
https://github.com/64bit/async-openai/pull/71#issuecomment-1558490166,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM5EOSTKTEATBYP5L3MLNLXHQ2BDANCNFSM6AAAAAAYJ5ZKR4
.
You are receiving this because you were mentioned.Message ID:
@.***>

I'm a bit late here to respond.. apologies.
I don't have any objections  for having it here, however, in spirit of having this example useful for folks new to Rust and new to this repo - do you mind adding:

a README for this example along with a brief overview of what it does
Add ways to actually call the APIs in this example : like curl or Postman. So people can copy paste and run your example.

Above two would make it self documenting where people can actually run it with least friction.


Physical module support

Currently the pogo-pin connector pair is the only physical support between master board and module. Worse, it is only on one side. Therefore, any user interaction with the module usually makes it disconnect. Additionally, Britton pointed out that unsnapping the module is not done "straight up" as it should be, rather from an angle. So, we will have to come up with a way to fix these problems.
A simple fix would be to have our module enclosure walls "come down" and meet the master board.
Another fix could be to use foam on the bottom of the module board opposite to the pogo-pin connector.
Finally, we could investigate using magnets. For example, a south pole magnet on the bottom of the module slot and a north pole magnet on the bottom of the module board (opposite to pogo-pin connector).
Here is an illustration of the issue:

The concern about the wall solution is space. There are only 1.5mm between module slots, so that gives 0.75mm maximum wall thickness on each side of the module. This is doable with my printer, but we will see in practice...


Dont merge - Ci no fail fast

Just creating this to run CI without failfast to collect all failures in one place

@cameron1024 Should we close this one?


Map 'ts' to 'video/mp2t' in forExtension.

Let Mime.forExtension("ts") to return "video/mp2t".

Thank you!
Tagged as 3.1.6.


Changed install link

changed version of installation link for version 3 (most popular) and instalation via bower and npm

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for contributing to the FreeCodeCamp Portuguese Translation!
Contributions like yours help millions of users learn to program in their native language.
Hope you continue to help the translation with more contributions!


Circular progress bar not working in determinate mode

Is that a feature request? Because there is no such thing as a determinate circular progress bar in Android afaik.

This can be very useful feature

I know it may be difficult but +1 on this feature.
I am using this https://github.com/pnikosis/materialish-progress but it is not extending the normal ProgressBar which I don't like.


tqdm required by ramble

==> Error: Module tqdm is not found. Ensure requirements.txt are installed.
Should we be automatically be installing Ramble's package requirements as part of benchpark's setup or let the user get to the setup step of benchpark before hitting this error?

Yes, lets install Ramble's package requirements at benchpark setup, it doesn't make sense to postpone the error.

@slabasan How do we solve this error?

@rfhaque Let me know what you think of this solution. The benchpark script hides the output, so user doesn't necessarily know these packages are getting installed into their python environment. But it solves the problem of user having the needed packages before running ramble.

@pearce8 @slabasan This issue can be bypassed by using the --disable-progress-bar option in ramble

Interestingly, ramble requires tqdm if you do a ramble --help, but knowing we can bypass the tqdm requirement with this flag seems sufficient. I'll update the benchpark instructions.


fix: lint

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


Setup project

added Webapack and jest

Hello @ragangithub
Great work with setting up the project all linters are installed and webpack is acting correctly. The is nothing more to say than it is it is time to merge :1st_place_medal:


explain more about direction

@edvardchen it exists in the doc: https://d2lang.com/tour/layouts/#direction
very light rn, but hopefully makes it clear how to set!

directly in the code? like
direction: right
a -> b

or passing to the command d2 --direction right ...?

yeah directly in the code like that. sorry for the confusion, i'll do this tomorrow

It requires version 0.1.0, quite fresh. But the latest version of d2 on brew is still 0.0.13. Not sure why

@edvardchen shoot, sorry about that, nice catch. also didn't know but someone submitted d2 to homebrew core, so our install script needs to account for this


AttributeError when using cyber_record echo command

Hello! I'm using this tool to analyze record file collected from Apollo 7.  I greatly appreciate the contributions your work has made to the field, and I have found it to be instrumental.
However, I encountered this issue when using the cyber_record info command:
$ cyber_record info -f 8_2.record.00000
Traceback (most recent call last):
  File "/home/sun/.local/bin/cyber_record", line 8, in <module>
    sys.exit(main())
  File "/home/sun/.local/lib/python3.8/site-packages/cyber_record/main.py", line 157, in main
    cyber_record_info(args.file)
  File "/home/sun/.local/lib/python3.8/site-packages/cyber_record/main.py", line 38, in cyber_record_info
    record = Record(record_file)
  File "/home/sun/.local/lib/python3.8/site-packages/cyber_record/record.py", line 104, in __init__
    self._open(f, mode, allow_unindexed)
  File "/home/sun/.local/lib/python3.8/site-packages/cyber_record/record.py", line 298, in _open
    if mode == 'r': self._open_read(f)
  File "/home/sun/.local/lib/python3.8/site-packages/cyber_record/record.py", line 320, in _open_read
    self._reader.start_reading()
  File "/home/sun/.local/lib/python3.8/site-packages/cyber_record/reader.py", line 60, in start_reading
    for single_index in index.indexes:
AttributeError: 'NoneType' object has no attribute 'indexes'

By the way, I'm using Ubuntu 20.04, and the record file was collected from Apollo 7. Given the record file for reproducing this bug: https://drive.google.com/file/d/1W4es2pmQB0RwAKVCmyrVDQhRO1p31fel/view?usp=sharing
How can I solve this issue? Thank you very much!

Record uses indexs to index. If it cannot be found, your record may be damaged.

I tried to use the cyber_record recover command to recover the record file and encountered the same issue:
$ protoc --include_imports --descriptor_set_out=tmp --proto_path=/home/as3r/ADSs/apollo-6 /home/as3r/ADSs/apollo-6/modules/planning/proto/planning.proto
$ cyber_record recover -f 8_2.record.00000 -t /apollo/planning -d tmp -m apollo.planning.ADCTrajectory
WARNING:root:msg_type should be:
WARNING:root:	apollo.planning.EStop
WARNING:root:	apollo.planning.TaskStats
WARNING:root:	apollo.planning.LatencyStats
WARNING:root:	apollo.planning.RSSInfo
WARNING:root:	apollo.planning.ADCTrajectory
Traceback (most recent call last):
  File "/home/as3r/.local/bin/cyber_record", line 8, in <module>
    sys.exit(main())
  File "/home/as3r/.local/lib/python3.8/site-packages/cyber_record/main.py", line 162, in main
    cyber_record_recover(args.file, args.desc_file, args.topic, args.msg_type)
  File "/home/as3r/.local/lib/python3.8/site-packages/cyber_record/main.py", line 123, in cyber_record_recover
    record.recover_index(single_index)
  File "/home/as3r/.local/lib/python3.8/site-packages/cyber_record/record.py", line 210, in recover_index
    for s_index in index.indexes:
AttributeError: 'NoneType' object has no attribute 'indexes'

I got the following error info in Apollo's docker:
[as3r@in-dev-docker:/apollo/recordings]$ cyber_recorder info 8_2.record.00000
record_file:    8_2.record.00000
version:        1.0
duration:       0.000000 Seconds
begin_time:     1969-12-31-19:00:00
end_time:       1969-12-31-19:00:00
size:           0 Bytes
is_complete:    false
message_number: 0
channel_number: 0
E1015 12:40:11.809468 11228 record_file_reader.cc:86] [cyber_recorder]Record file is not complete.
E1015 12:40:11.809871 11228 info.cc:94] [cyber_recorder]read index section of the file fail. file: 8_2.record.00000

Now I suspect that the process used to record the record file exited abnormally. Thank you!


Auto-approving a create request if the PO and at least one TL already have a provisioned project

Describe the issue
Rational/Why
Additional Context
Dependencies
Tests
Definition of done/Acceptance Criteria

Iryna to follow up with Billy and Shelly re this requirement to see if it's still valid.


STEP DELAY Feature

If defined the steps should be delayed with a given timeout. This sometimes make step execution "visible" for people who want to see / follow whats going on. This is often the case in UI tests i make with customers.
Therefore there should be an environment variable to handle a fixed timeout between each step / scenario etc.
The value should be in ms. The runner should wait the given time after each run.
Suggestion is CWT_CUCUMBER_STEP_DELAY

sure, we can do so. but instead of an environment variable I'd prefer extending the program arguments, so that execution goes like this:
./build/bin/box --step-delay 100 ./features/example.feature
or a short version
./build/bin/box -d 100 ./features/example.feature
I can implement this later

Or maybe both. Let me explain.
I am testing with cmake/ctest and using only an option would force me to reconfigure my build tree. Depending due "modularization" of the cmake code this could effect one or all tests (e.g. if you wrap the test call in a cmake makro/function). Calling the test like ctest does on commandline to just change the option is mostly impossible due to adaptions on LD_LIBRARY_PATH (shared library dependencies), Test Setup and so on.
Using the environment variable i just can set it in the environment and rerun the test and everything is fine.


What are the login details??

If I'm being blind then I apologize. But I can't see anywhere in the readme that says what the default username and password is??

It was the default admin and no pass part I was stuck on.It would then ask me to setup my own.On 29 Jul 2023, at 12:35 pm, Kroese @.***> wrote:
During installation it will ask you to choose a  username and password, there are no defaults.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>


Update the team page to reflect the dev-tools re-org

Removes dev-tools peers, adds rustdoc and IDEs teams, changes dev-tools membership, removes style team.
r? @aturon

@nrc my github username is now @jasonwilliams

@jasonwilliams updated; thanks!


segfault on empty reads

Hi @najoshi,
I'm using sickle to clean up some trimming with cutadapt and some of the reads coming from cutadapt can have length zero, which causes sickle to segfault. I put up a file with two reads in it, one of which is empty, as an example here:
https://dl.dropboxusercontent.com/u/2822886/one_empty_read.tar
If you drop the empty read it works fine so I think it is due to the read being empty.
Any shot at a fix?

@najoshi any chance of getting this merged in?

Hi Rory,
As far as I can tell, all this merge would do is add gnu autotools build to
the repo, and it doesn't seem like it actually fixes that problem, just
tests for it. Also, kseq.h is written by Heng Li, not me, so I'm not sure
of the best way to fix it.

Nik.

On Tue, Feb 24, 2015 at 11:58 AM, Rory Kirchner<EMAIL_ADDRESS>wrote:

@najoshi https://github.com/najoshi any chance of getting this merged
in?
—
Reply to this email directly or view it on GitHub
https://github.com/najoshi/sickle/issues/32#issuecomment-75833196.

--
Nikhil Joshi
Bioinformatics Analyst/Programmer
UC Davis Bioinformatics Core
http://bioinformatics.ucdavis.edu/
najoshi -at- ucdavis -dot- edu
530.752.2698 (w)

Perhaps an updated copy of kseq.h would do the trick, though this is just a shot in the dark. I haven't looked at the code.
Latest version from htslib: https://github.com/samtools/htslib/blob/develop/htslib/kseq.h

So I think I've fixed it. I took some code from the new kseq.h and put it into this one to deal with the empty read problem. Test it out and let me know if you find more bugs.

Hi @najoshi, good to finally hear from you.
The reason I added a GNU autotools build is that it makes it much easier to add some tests and automatically execute them (as well as build and install the software in a portable and standard way).
Yes, I did not have had a fix yet, just implemented the test to check the issue.
@ucdavis-bioinformatics Maybe you can base your fix upon my #33 PR?

Fix now in #33

Hi @najoshi and @wookietreiber,
Thanks for the responses-- the fix in master works on the test file I posted. #33 does as well.


Does this work with Godot 4 alpha / nightly?

Will this work with Godot 4 as I am already moving some of my code to it. I could not find any information on that.

Hi! Well, not yet, no =/
Godot 4 deprecates the GDNative API in favor of the GDExtension API, which right now does not support registering new scripting languages, which is exactly what this plugin does as a PluginScript.
There is a proposal for getting this functionality into Godot 4, which explains very well the missing API for registering new languages (https://github.com/godotengine/godot-proposals/issues/3927).
I really want to have Lua PluginScript support Godot 4, but it will take some time until there is actual support for doing it, then some more time implementing the new interface and porting code to use the GDExtension API.

lt looks like the mentioned proposal was already implemented. Did they provide all necessary functionalities for lua?

Did they provide all necessary functionalities for lua?

As far as I looked into it, yes it did. Although last time I checked this addition was not in godot-headers yet.

Although last time I checked this addition was not in godot-headers yet.

Is it in now? Any updates on this?

Although last time I checked this addition was not in godot-headers yet.

Is it in now? Any updates on this?

Not exactly on topic, but a potential temporary workaround is simply calling lua.dll (or luajit.dll) from C# with native interop or using MoonSharp. Though, that'd of course require the build of the engine with C# support as I don't think GDScript has such capabilities.

Hi @hendursaga. Yes, the scripting instance extension API is in godot-headers, probably for a long time now.
It should be possible to implement Godot 4.0 support, but I haven't looked into it yet. Like, at all.

Not exactly on topic, but a potential temporary workaround is simply calling lua.dll (or luajit.dll) from C# with native interop or using MoonSharp.

Well, yeah, that's actually a nice way to use Lua in Godot. But it would not be a way to implement Godot objects directly in Lua, which is the point of PluginScripts/Script Language GDExtension.
Using MoonSharp or calling the Lua API directly from the DLL would be more like using Trey2k/lua. Which is totally fine, depending of what you're using Lua for. They're 2 different ways of using Lua, serving potentially different purposes.

Not exactly on topic, but a potential temporary workaround is simply calling lua.dll (or luajit.dll) from C# with native interop or using MoonSharp.

Well, yeah, that's actually a nice way to use Lua in Godot. But it would not be a way to implement Godot objects directly in Lua, which is the point of PluginScripts/Script Language GDExtension.
Using MoonSharp or calling the Lua API directly from the DLL would be more like using Trey2k/lua. Which is totally fine, depending of what you're using Lua for. They're 2 different ways of using Lua, serving potentially different purposes.

Indeed, hence only a potential workaround ^^ Your plugin definitely does a lot more than what lua's dll does by itself.

FYI, I was looking into how porting the PluginScript to GDExtensions would go.
The binding of variants (Godot structures like Vector2, Array and Object) seems to be quite simpler in GDExtensions than in GDNative, but creating a new language needs the implementation of a subclass of ScriptLanguageExtension, which is not very nice using the raw C API.
Thus I'm thinking of a different approach for implementing Lua support in GDExtensions: instead of using LuaJIT's FFI for the whole implementation, we can generate Lua/C bindings for Variants based on the extension_api.json file and possibly use godot-cpp to implement the ScriptLanguageExtension subclass. Later on, if at all, we can generate FFI-based bindings for better JIT support in LuaJIT.
Pros:

Support for PUC Lua and other implementations. Maybe even web support with some WebAssembly implementation of Lua, who knows?
Decoupling the Lua <-> Godot bindings from PluginScript / ScriptLanguageExtension.
This would make it easy enough to create WeaselGames/lua-like functionality using the same code.
Not everybody that uses Lua in Godot needs the whole new scripting language stuff, an extension class that exposes Lua VMs might be enough for lots of cases.

Cons:

Major rewrite 🤦

If anybody has more ideas on the subject, please feel free to bring them to the table! =D

without this plugin, it is really hard for me to make transition to godot 4.


Chat option for Participants and Mentors

We require a chat system enabled with the frontend so that particpants can chat with their mentors and in between their participants

can i use firebase to create the chat?

Unfortunately for these issues this one does not have enough information. I can't just create a chat without having a scope

I would like to thank you for the opportunity but due to lack of information I will not be able to solve this problem.


Declare prettier as dependency of pre-commit hook

Summary
This adds prettier as a dependency of the prettier pre-commit hook.
Related issue :
Closed #640

woops. Looks like I didn't discard the intermediate commits. Hope you squash this?

@sebastian-luna-valero ready for review.


Add search by VO code

Just as we provide search by ECO code on the CTD2 Dashboard, we could add search by VO code in the HIPC Dashboard.  We should support both ":" and "_" in the code, e.g.  VO_0000045 or VO:0000045.

Tested in production, works fine.


Remove from __future__ import absolute_import.

Except in:

docs/writing-bots-guide.md, because bots are supposed to be Python 2
compatible
puppet/zulip_ops/files/zulip-ec2-configure-interfaces, because this
script is still on python2.7

@rht as I've mentioned before, I don't want to remove any this stuff from tools/linter_lib and tools/lint, since those are likely to be split out as a separate open source project with Python 2 support.

Re-added to tools/linter_lib and tools/lint, added note to the commit message.

@timabbott

Merged, after also excluding the changes to lister.py, which probably belongs in linter_lib.


Example program for rendering with D3D using shared textures and OnAcceleratedPaint()

Summary:
A simple proof-of-concept program that hosts an offline CefSharp instance, and initializes it to render via shared D3D11 textures.
It can run in either fullscreen or windowed mode (switch with ALT+ENTER), with or without VSYNC, and with or without external frame trigger.
These are controlled by the FULLSCREEN, VSYNC and EXTERNALTRIGGER consts, found at the top of DXForm.cs.
The program itself is a test application.
It's meant to act as a simple example of how to use the accelerated interfaces, and can also be modified to test and verify new CEF features or alternative implementations.

Thanks for the PR 👍
Can the CefSharp.Direct3D.Example project be renamed to something like CefSharp.WinForms.Direct3D.Example.
At a later point it would be great to add a WPF example.
Thanks

Sure, project is renamed.

Thanks 👍


Set frontmatter when changing icon

Is your feature request related to a problem? Please describe.
At present Iconize will load an icon specified in the icon: property of the frontmatter. This is good, however it only takes effect when the file is edited, when loading a new file, or when restarting the client after selecting "Remove Icon". It doesn't work when editing an .md file outside of Obsidian and reloading, nor if an icon is set with "Change Icon".
When using the "Change Icon" option in the right click menu, the icon is only set in data.json. This means that if you want to use the frontmatter for icons, you need to manually type the name of the icon and cannot use the icon picker.
Describe the solution you'd like
I would like "Change Icon" to set the icon data in the frontmatter if "Use icon in frontmatter" is selected, or for there to be a separate option to have the addon write to frontmatter.
Additionally, it would be an improvement if the icon could be loaded from the frontmatter on file load, instead of only on edit.
Additional context
This would also work well with and apply to #358, essentially moving the two pieces of per file configuration into the frontmatter for those who prefer to have file metadata entirely in the files themselves.

I second this request

I'd like to see this too

Me too! I prefer for the authoritative icon information to stay in the frontmatter rather than in a separate, hidden config file. In general I think keeping note data within the note is better for future-proofing (this file over app philosophy is a key reason I use Obsidian).

This reason is the one thing that is keeping me from using the plugin. I'd really prefer the information to be kept on the files themselves!

Interesting idea. But, is this compatible with the ability to change a folder icon?

In this case maybe an additional file could be created under the selected folder like "_folder_icons.md" or put it under the attachments subdirectory.
By using a plugin like Hide Folders this file could be visually hidden.

Perhaps an integration with the folder-note plugin's frontmatter could also work? Something like a folderIcon: property.


Use the mime crate for parsing the Accept header

Currently our code does some manual parsing, now that #153 is using the mime library for proper parsing, we should replace some of our own code with it in the SerializedData type

Blocked on https://github.com/hyperium/mime/pull/139
mime does not have TOML. (Right now, it is also missing Wasm, but there is a PR for that https://github.com/hyperium/mime/pull/138)
Given the low rate of PR merges on that repo (one merge in 2020, none in 2021), we might just close this. If anyone knows @seanmonstar or any of the other Hyperium devs, maybe we should volunteer as co-maintainers on that repo. It's low activity, and getting more MIME types supported is definitely desirable.

Okay, so on further clarification from @thomastaylor312, I am trying to switch over from strings to mime's types throughout Bindle. The above will not block this PR (though it would still be good to get application/toml into mime.


bootstraptour is down

Is there another page to see the documentation?

Even I wanted to have a look at documentation today. Looks like the domain got expired yesterday.

You can read it in archive.org

Does anyone know if the site is going to come back up or if the product is going away

Does anyone know if the site is going to come back up or if the product is going away

The authors of Tour are still around but don't have the time to maintain it at the moment.
I've created a fork of Tour that fixes many of the issues reported and features requested. My repo is here:
https://github.com/IGreatlyDislikeJavascript/bootstrap-tourist
And you can read more about why here:

Hi everyone! Very sorry about that. The website is back up again.


Terminal windows opened from Windows Explorer via double click cannot join with existing terminal sessions

Note: 📌 Pinned comment: https://github.com/microsoft/terminal/issues/15509#issuecomment-1592021826

Windows Terminal version
1.18.1462.0
Windows build number
10.0.19045.3031
Other Software
Any program that spawns a terminal window
Steps to reproduce
When the default terminal application is set to the Windows Terminal (preview, in this case), programs, such as servers, that open a terminal window will have windows whose tabs cannot join with regular terminal windows.
Expected Behavior
Any instance of Windows Terminal may join into any other window.
Actual Behavior
The spawned terminal window will not be able to join with an existing terminal group. The "Move" popup does not appear, and the tab will not join into the existing window.

My apologies, I do not understand?

Are you running the Terminal as Admin/?
Terminal (stable) windows can't be merged with Terminal Preview windows. There is one report we've had in the past about Preview<->Preview communication not working, though that was a very rare edge case

I do not understand?

Read this

set Windows Terminal as default console
set "windowingBehavior": "useExisting"  in settings.json

Thanks for the suggestion. I tried that, and unfortunately it does not appear to work. The issue still persists :(

Hi @Mersid.

Does this repro if you switch default terminal over to the non-preview version? (make sure windowingBehavior is properly set and you're not running terminal as admin, as mentioned earlier in the thread) We're thinking this may be related to the process model changes we made to the preview version.

Oh no. Ohhh, noooo.
This is definitely broken in 1.18. Instances of Terminal started for -Embedding don't seem to glom with other instances of Terminal.

Hello, @carlos-zamora
The current non-preview version of Windows Terminal (for me) is 1.17.11461.0, and does not support the drag'n'drop feature yet.

Wait @DHowett can you actually repro this?
I've been futzing with this all morning and can't get a repro locally with a dev build[^1]. I can't do it with a Terminal-first cold launch, or a cmd.exe(defterm)-first cold launch. And I can't repro it via #15518 either.
I'm moving this to @DHowett to try and fix. Or close as fixed along the way of 1.19[^2].
[^1]: Admittedly, I did spend a few hours on #15238 and the associated "blow away a dev build to fix a bad defterm registration". But once I got dev build defterm working again, then I haven't had any troubles with this.
[^2]: There's maybe a small chance that #15424 fixed this? but that seems unlikely.

@zadjii-msft If this is the same problem as #15636 then it definitely isn't fixed and I can reproduce it with the latest build (d54ce33afcd5e9875b53e4449ea3d2fd5c1ebc07 / 636be7e514e243810e8944a61f5cf7044d316e05).

I can't reproduce it in 1.19, but I do not know whether that is because it doesn't reproduce on this machine or because it is fixed in 1.19.

tried reproducing #16189 to get some traces to share here but when i tried on my home pc (same setup, just win 11 instead of win 10), it didn't repro. So this is possibly some quirk of win 10 that got fixed on win 11. Sorry, i'm probably not allowed to share traces from my work PC so can't give any more info.

this is possibly some quirk of win 10 that got fixed on win 11.

I've just tried repeating #15636 in a VM and it can be only reproduced on Windows 10 - after installing Windows 11 the issue disappeared.

I'm running Windows 11 (build 22631.3447), use Windows Terminal as default terminal, and I notice if I run a cmd file, it always opens in a new terminal window than say I run "wt.exe". However, if I create a shortcut with wt.exe test.cmd, that terminal window can be reused.
Like DHowett mentioned above, I think the default terminal handler has special logic to put those associated file opening in a separate terminal process, which has "-Embedding" in the launch command line and parented by svchost.exe.
Right now, I basically have to put "wt.exe" in front of every cmd file launch to reuse window, which defeats the whole purpose of setting Windows Terminal as default terminal.


introduce bower.json

This commit introduces a proper bower.json configuration file for Bower package manager.

anyone ?

Looks good to me. Ping @janodvarko ?
Florent

Thanks for the patch!
Honza


toggleLayers update

The 'Toggle all layers' checkbox actually runs counter to how it should. On initial load, if we have layers loaded from the hash, the first 'uncheck' of that checkbox actually does nothing. The jquery checks that were being run in the 'dc.layerlist.toggleAll' worked after that, but since the initial 'loaded' property on the 'Toggle all layers' input was now gone, the toggleAll checkbox would actually be checked when the map layers were hidden, and unchecked when they were displayed.
For this particular use case, document.getElementById is actually a safer bet, hence the new 'allCheckboxes' id.
I also updated the topojson path because it failed on our 'js_vendor' gulp task.

Also, the travis-ci check fails on 'gulp build:prod' due to a 'Maximum call stack size exceeded' error, rather than any karma-run unit test failures.

Heh, funny bug - thanks for working on this @SwampGuzzler! I'll take a look this weekend and will look into solving the travis timeouts as well.

Fixed in https://github.com/cugos/dropchop/pull/257


Implemented new command: spo orgassetslibrary list

Implemented new command spo orgassetslibrary list solving #1041

Coverage decreased (-0.005%) to 99.995% when pulling 92052f7ea02158549d73d84640906c7da118c32d on appieschot:feature/spo-orgassetslibrary-list into 7f42bffb71b637026f2a1e79ebb1679d3308f9a2 on pnp:dev.

Will fix test coverage asap :)

Thanks @appieschot !

Hi Albert (@appieschot), I pushed new ver of the CLI because of the new SPFx 1.9.1 and I had to rebase dev with master to bring the changes to dev. Unfortunately, this caused conflicts for your pull request.
Could you please be so kind to get the latest dev and apply your changes on top? I am really sorry about that disturbance.

@VelinGeorgiev I did a rebase and force push as I did not have merge conflicts after the rebase. Let me know if this would workf or you or if you want me to abandon this PR and create a new one with new changes :)

That is just perfect @appieschot ! We will reviee it shortly.

@VelinGeorgiev I have updated the test and processing of no results. That should also solve the error you had without a library; as the orgAssetsResponse was null it could not be processed correctly. Except the question on the removal of the digest all should be solved :).

Thanks @appieschot !
My opinion is that if it works without then we should not add it until it becomes an issue for someone.  Waldek suggest here to not retrieve digest if not needed https://github.com/pnp/office365-cli/wiki/PR-checklist.
FYI: I had org asset library already when I executed the command and got an error. I will re-test it later.

@VelinGeorgiev ill do another submit to remove the digest then :). Weird thing is that the response you pasted (REQUEST response body) does not contain a library ("IsNull": false). I even added it to the test. Let me know how it plays out!

I assume there is a cache on the server that gets invalidated later. So I created the org lib and tested immediately, that did not worked. What I will do now is to create the org lib and wait for some time to see it I will get the results. That should be the reason because I have seen that before with tenant properties.

@appieschot  this is what I am getting when no libraries present
(node:9212) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'OrgAssetsLibraries' of null
    at C:\p\office365-cli\dist\o365\spo\commands\orgassetslibrary\orgassetslibrary-list.js:69:39
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:9212) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:9212) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Thats why I added a null check on line 65
 if (orgAssetsResponse === null || orgAssetsResponse.OrgAssetsLibraries === undefined) {
Are you saying that is still failing? Or did you not get the latest push from my side?

That was my bad @appieschot , I got the latest changes and when no lib works just as expected.
When I have library, the command gets the data correctly, but it fails with Cannot read property 'DecodedUrl' of null exception error for me.
Here is what I get:
REQUEST emitting complete https://velingeorgiev-admin.sharepoint.com/_vti_bin/client.svc/ProcessQuery
REQUEST response body
"[\r{\r\"SchemaVersion\":\"<IP_ADDRESS>\",\"LibraryVersion\":\"16.0.19131.12010\",\"ErrorInfo\":null,\"TraceCorrelationId\":\"fac1fa9e-e0cc-1000-077b-61deac0da407\"\r},4,{\r\"IsNull\":false\r},5,{\r\"_ObjectType_\":\"Microsoft.Online.SharePoint.TenantAdministration.Tenant\",\"_ObjectIdentity_\":\"fac1fa9e-e0cc-1000-077b-61deac0da407|908bed80-a04a-4433-b4a0-883d9847d110:a1214787-77d5-4b72-a96d-1c278f72bbb0\\nTenant\",\"AllowCommentsTextOnEmailEnabled\":true,\"AllowDownloadingNonWebViewableFiles\":true,\"AllowedDomainListForSyncClient\":[\r\r],\"AllowEditing\":true,\"AllowGuestUserShareToUsersNotInSiteCollection\":false,\"AllowLimitedAccessOnUnmanagedDevices\":false,\"AllowSelectSGsInODBListInTenant\":null,\"ApplyAppEnforcedRestrictionsToAdHocRecipients\":true,\"BccExternalSharingInvitations\":false,\"BccExternalSharingInvitationsList\":null,\"BlockAccessOnUnmanagedDevices\":false,\"BlockDownloadOfAllFilesForGuests\":false,\"BlockDownloadOfAllFilesOnUnmanagedDevices\":false,\"BlockDownloadOfViewableFilesForGuests\":false,\"BlockDownloadOfViewableFilesOnUnmanagedDevices\":false,\"BlockMacSync\":false,\"CommentsOnFilesDisabled\":false,\"CommentsOnSitePagesDisabled\":false,\"CompatibilityRange\":\"15,15\",\"ConditionalAccessPolicy\":0,\"ConditionalAccessPolicyErrorHelpLink\":\"\",\"ContentTypeSyncSiteTemplatesList\":[\r\r],\"CustomizedExternalSharingServiceUrl\":\"\",\"DefaultLinkPermission\":0,\"DefaultSharingLinkType\":3,\"DisabledWebPartIds\":null,\"DisableReportProblemDialog\":false,\"DisallowInfectedFileDownload\":false,\"DisplayNamesOfFileViewers\":true,\"DisplayStartASiteOption\":true,\"EmailAttestationEnabled\":false,\"EmailAttestationReAuthDays\":30,\"EmailAttestationRequired\":false,\"EnableAIPIntegration\":false,\"EnableAzureADB2BIntegration\":false,\"EnableGuestSignInAcceleration\":false,\"EnableMinimumVersionRequirement\":true,\"EnablePromotedFileHandlers\":true,\"ExcludedFileExtensionsForSyncClient\":[\r\"\"\r],\"ExternalServicesEnabled\":true,\"ExternalUserExpirationRequired\":false,\"ExternalUserExpireInDays\":60,\"FileAnonymousLinkType\":2,\"FilePickerExternalImageSearchEnabled\":true,\"FolderAnonymousLinkType\":2,\"GuestSharingGroupAllowListInTenant\":\"\",\"GuestSharingGroupAllowListInTenantByPrincipalIdentity\":null,\"HideSyncButtonOnODB\":false,\"IPAddressAllowList\":\"\",\"IPAddressEnforcement\":false,\"IPAddressWACTokenLifetime\":15,\"IsHubSitesMultiGeoFlightEnabled\":true,\"IsMultiGeo\":false,\"IsUnmanagedSyncClientForTenantRestricted\":false,\"IsUnmanagedSyncClientRestrictionFlightEnabled\":true,\"LegacyAuthProtocolsEnabled\":true,\"LimitedAccessFileType\":1,\"MarkNewFilesSensitiveByDefault\":0,\"MobileFriendlyUrlEnabledInTenant\":true,\"NoAccessRedirectUrl\":null,\"NotificationsInOneDriveForBusinessEnabled\":true,\"NotificationsInSharePointEnabled\":true,\"NotifyOwnersWhenInvitationsAccepted\":true,\"NotifyOwnersWhenItemsReshared\":true,\"ODBAccessRequests\":0,\"ODBMembersCanShare\":0,\"ODBSharingCapability\":2,\"OfficeClientADALDisabled\":false,\"OneDriveForGuestsEnabled\":false,\"OneDriveStorageQuota\":1048576,\"OptOutOfGrooveBlock\":false,\"OptOutOfGrooveSoftBlock\":false,\"OrgNewsSiteUrl\":null,\"OrphanedPersonalSitesRetentionPeriod\":30,\"OwnerAnonymousNotification\":true,\"PermissiveBrowserFileHandlingOverride\":false,\"PreventExternalUsersFromResharing\":false,\"ProvisionSharedWithEveryoneFolder\":false,\"PublicCdnAllowedFileTypes\":\"CSS,EOT,GIF,ICO,JPEG,JPG,JS,MAP,PNG,SVG,TTF,WOFF\",\"PublicCdnEnabled\":true,\"PublicCdnOrigins\":[\r\r],\"RequireAcceptingAccountMatchInvitedAccount\":false,\"RequireAnonymousLinksExpireInDays\":-1,\"ResourceQuota\":5300,\"ResourceQuotaAllocated\":300,\"RootSiteUrl\":\"https:\\u002f\\u002fvelingeorgiev.sharepoint.com\",\"SearchResolveExactEmailOrUPN\":false,\"SharingAllowedDomainList\":null,\"SharingBlockedDomainList\":null,\"SharingCapability\":2,\"SharingDomainRestrictionMode\":0,\"ShowAllUsersClaim\":false,\"ShowEveryoneClaim\":false,\"ShowEveryoneExceptExternalUsersClaim\":true,\"ShowNGSCDialogForSyncOnODB\":true,\"ShowPeoplePickerSuggestionsForGuestUsers\":false,\"SignInAccelerationDomain\":\"\",\"SocialBarOnSitePagesDisabled\":false,\"SpecialCharactersStateInFileFolderNames\":1,\"StartASiteFormUrl\":null,\"StorageQuota\":1304576,\"StorageQuotaAllocated\":131072000,\"SyncAadB2BManagementPolicy\":false,\"SyncPrivacyProfileProperties\":true,\"UseFindPeopleInPeoplePicker\":false,\"UsePersistentCookiesForExplorerView\":false,\"UserVoiceForFeedbackEnabled\":true,\"WhoCanShareAllowListInTenant\":\"\",\"WhoCanShareAllowListInTenantByPrincipalIdentity\":null\r},6,{\r\"_ObjectType_\":\"Microsoft.SharePoint.Administration.OrgAssets\",\"CentralAssetRepositoryLibraries\":null,\"OrgAssetsLibraries\":{\r\"_ObjectType_\":\"Microsoft.SharePoint.Administration.OrgAssetsLibraryCollection\",\"_Child_Items_\":[\r{\r\"_ObjectType_\":\"Microsoft.SharePoint.Administration.OrgAssetsLibrary\",\"DisplayName\":\"Site Assets\",\"FileType\":null,\"LibraryUrl\":{\r\"_ObjectType_\":\"SP.ResourcePath\",\"DecodedUrl\":\"SiteAssets\"\r},\"ListId\":\"\\/Guid(75f03ba8-c10d-4923-8272-8ed30f08b8f2)\\/\",\"OrgAssetType\":1,\"ThumbnailUrl\":null,\"UniqueId\":\"\\/Guid(00000000-0000-0000-0000-000000000000)\\/\"\r}\r]\r},\"SiteId\":\"\\/Guid(ed45f182-74d9-4b2a-abf7-c39b4a2f0a29)\\/\",\"Url\":{\r\"_ObjectType_\":\"SP.ResourcePath\",\"DecodedUrl\":\"\\u002f\"\r},\"WebId\":\"\\/Guid(98853893-71fd-4270-ab11-57eeaf08561e)\\/\"\r}\r]"
(node:3864) UnhandledPromiseRejectionWarning: TypeError: Cannot read property 'DecodedUrl' of null
    at C:\p\office365-cli\dist\o365\spo\commands\orgassetslibrary\orgassetslibrary-list.js:80:62
    at Array.map (<anonymous>)
    at C:\p\office365-cli\dist\o365\spo\commands\orgassetslibrary\orgassetslibrary-list.js:75:87
    at process._tickCallback (internal/process/next_tick.js:68:7)
(node:3864) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). (rejection id: 1)
(node:3864) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Will compare the response you get with a response I get, might be slight difference between our tenants (or just bad code from my side ..) Hope to figure it out this weekend.

@appieschot , the response with the data is there it just might not return the exact same json schema, this is what can differ depending on the tenant setup I guess.

@VelinGeorgiev found the bug; it fails because you have not set a thumbnail URL (and I did), hence it cannot find a decodedurl (as the object is null). That definitely makes sense and I had no test case for it so I didn't catch it during my tests. Will update code accordingly.

Great stuff! Thank you @appieschot !

All changes should be reflected in latest commit.

Thank you @appieschot ! We will review shortly.

Thank you Albert @appieschot ! Merged manually 🚀


Add support for ST 🎆

Hey I added a theme for ST (the simple terminal) ! what do you think ?

@pineapplegiant (sorry for the mention) what do you think ? is there anything I should improve ?


Remove attachMediaStream and reattachMediaStream

https://github.com/webrtc/adapter/pull/334

Removed.


fix: set domain to (0, 1) when no data is present

Fixes https://github.com/bqplot/bqplot/issues/1614
If no data is present, a scale has a domain of (0, 1) but if we remove the data afterwards (set it to an empty array) , we set it to (-inf, +inf) which causes rendering issues and make the bqplot figure unable to restore itself.
It is more consistent to behave like no data was passed in, and use (0, 1) for the domain.

I hope i remembered the magic phrase correctly:
By the power vested in me by all code repositories, I call upon the almighty 'update galata references' phoenix. Arise from the ashes of the old screenshots and blaze a trail of new visuals across our Github sky!

Looks like the phoenix burst into flames before it could try to fly.
I suspsect the galata bot will only work with the main branch from now on. I guess since it's now using JupyterLab 4 and the new galata.

We need to upstream it to bqscales https://github.com/bqplot/bqscales/issues/84


Bug Report - Cannot move Azure policy assignment from subscription level to management group level

Describe the bug
We would like to move a policy assignment from the subscription level to the management group level.  The definition location of the policy is at the management group level.  Every time the steps below are performed and a pull is initiated the policy assignment is changed back to its initial state.
Steps to reproduce

Change scope in policy assignment json file.
Move the policy assignment json file up to the management group folder.

Screenshots

Hi @MM-MikeM, I understand your desired outcome and plan of action.
So to move a policy assignment from subscription ‘x’ to management group ‘y’ you have updated the scope value to reflect the desired management group and moved the file itself to that management group.
Suggested approach
A change in the steps here would help with the outcome you are looking for.
Instead of moving the file, create a new file (empty) at the management group scope and copy the content from source policy assignment and paste into new empty file, update the scope and delete the policy file at subscription scope.
Performing the steps above will ensure the deletion is handled and the new policy assignment gets deployed.
What is happening?

Create branch
Find file for the existing policy assignment at subscription scope
Update scope section in file to point towards new mgmt scope
Move file
Create Pull Request
AzOps runs validation of the suggested change
Once PR is merged to main
AzOps runs push job and creates the new policy assignment at desired mgmt scope
AzOps runs pull job and finds that there is (still) a policy assignment at subscription scope ‘x’ so it brings that over as code into main.

Why?
The reason we end up with both a file at the subscription scope ‘x’ and management group scope ‘y’ is because we “moved” a file in Git. Git is smart and does not report that the file is deleted but instead it is renamed. The trigger for AzOps to perform deletions of policy assignments are triggered when the git diff results match a deletion.
So in short the current “behavior” is excepted based on the current pipeline configuration and AzOps module configuration.

I wonder it it make sense now to separate rename operation here to handle this scenario more effectively.
https://github.com/Azure/AzOps/blob/ac67ba5f784a178dbba1c2bf4444251c89fb823e/src/functions/Invoke-AzOpsPush.ps1#L178

Agree that if we deterministically can catch renames and from statements the move experience could be improved, by adding logic to Invoke-AzOpsPush.ps1 and validate-deploy.yml. Labeling this issue as a future enhancement to investigate further.
https://github.com/Azure/AzOps-Accelerator/blob/2a75754ff908ed4997e39862d092a43dd084af71/.github/actions/validate-deploy/action.yml#L27


Implement EnhancedUserType in ImmutableType and MutableType in Hibernate 6.2

The ImmutableType and MutableType should implement the EnhancedUserType in Hibernate 6.2.
If this issue is implemented, it will have to add support for EnhancedUserType for all Types that now extend ImmutableType and MutableType.

This issue will be integrated when *ALL  Types that now extend ImmutableType and MutableType will implement the fromStringValue from EnhancedUserType.
I won't have the time to implement it for all Types, but if this is important to you, then you should check out this branch and continue overriding the fromStringValue for each Type that extends ImmutableType and MutableType.
When all the work is done, I will integrate it.

I can try and make it a weekend project, any objections? :)
@vladmihalcea @piotrp

Please go ahead. I wanted to try doing it, but it became larger than I have time for.
I think that toString/fromStringValue should be implemented by delegating them to appropriate types, which would leave toSqlLiteral. That too could be delegated, but unfortunately UserType doesn't provide enough information to do this.
    @Override
    public String toSqlLiteral(T value) {
        // Type interface provides dialect and wrapper to toSqlLiteral, but UserType doesn't...
        return getJdbcTypeDescriptor().getJdbcLiteralFormatter(getJavaTypeDescriptor()).toJdbcLiteral(value, Dialect, WrapperOptions);
    }

    @Override
    public String toString(T value) throws HibernateException {
        return getJavaTypeDescriptor().toString(value);
    }

    @Override
    public T fromStringValue(CharSequence sequence) throws HibernateException {
        return getJavaTypeDescriptor().fromString(sequence);
    }

Probably the best way to handle it is to see how the Hibernate core Types implement them.
Most Hibernate core Types use the JDBC and Java descriptors, but implementing all that in the Hibernate 6.2 module is going to be a lot of work.

I am puzzled, the original issue was there due to absence of toSqlLiteral signature in the UserType interface. However I'd still like to cover with tests all the new methods for the types affected. Any chance you got an insight what the method fromStringValue could used for in Hibernate (i.e what should be covered with the test)?
From what I could gather - it's used to work with discriminator column and discriminator values, but the column must be of type: String, Char or Integer which kinda doesn't go well with all the UserTypes.

I am puzzled, the original issue was there due to absence of toSqlLiteral signature in the UserType interface.

The UserType interface has no toSqlLiteral method.

Any chance you got an insight what the method fromStringValue could used for in Hibernate (i.e what should be covered with the test)?

The fromStringValue is from the EnhancedUserType, and according to the JavaDoc, it's goal is to:

Consume the given string representation back into this types java form.

The EnhancedUserType interface has changed in Hibernate 6, and since it's been 4 years since I quit working on Hibernate, I have no idea why the core devs changed it and for what purpose.
You will have to see in the Hibernate core course code what it's used for and then you'll see what's the best way to implement it.

Only the ImmutableType should extend the EnhancedUserType, so I created a DescriptorImmutableType to make this transition easier.
However, previously the YearMonth Types were extending the ``MutableType, and that's not correct since the YearMonth` is immutable.