Provide instance Condense (Header (ShelleyBlock c))

The instance for Header is required by the TracingConstraints in the node top level for all protocols, but it was missing for Shelley.

bors r+

Disclaimer: Condense is a class we use for making test output more readable in consensus. I understand that it is handy to reuse it in cardano-node, but I do not encourage it.

I understand that it is handy to reuse it in cardano-node, but I do not encourage it.

Nor do I!
type TraceConstraints blk =
    ( Condense blk
    , Condense [blk]
    , Condense (Header blk)
    , Condense (HeaderHash blk)
    , Condense (GenTx blk)
    , Condense (TxId (GenTx blk))
    , HasTxs blk
    , HasTxId (GenTx blk)
    , Show (ApplyTxErr blk)
    , Show (GenTx blk)
    , Show (GenTxId blk)
    , Show blk
    , Show (Header blk)
    , Show (TxId (GenTx blk))
    , ToJSON   (TxId (GenTx blk))
    , ToObject (GenTx blk)
    , ToObject (Header blk)
    , ToObject (LedgerError blk)
    , ToObject (ValidationErr (BlockProtocol blk))
    )

Sigh. This would need some time to cut down.


Reorder Configure output

"Configuring..." was displayed with './Configure LIST'.  This reorders
the display of that line to happen after the "targets" LIST, TABLE and
HASH have been checked.

I'm unsure if this should be backported to 1.1.0

Merged.<PHONE_NUMBER>638c39afe1ae1bc893fba5b77524a5
This is not really a bug fix worth a backport, so I concluded I won't


[CP] revert non-zero tessellation optimization

Issue Link
https://github.com/flutter/flutter/issues/138598
Commit Hash
3493f9b
Target
stable
PR Link
https://github.com/flutter/engine/pull/48459
Changelog Description

Revert an incorrect optimization that causes paths to render incorrectly for certain SVGs

Impacted Users
Flutter iOS users
Impact Description
Rendering is incorrect with certain SVGs.
Workaround
Disable Impeller
Risk
low
Test Coverage
yes
Validation Steps
Run the attached app and unit tests on engine CI

@matanlurey can you review this CP request? If it looks good to you, add the "cp: approved" label.

Done.


GitHub Template

@williamFalcon have you considered making this a GitHub Template?
"After you make your repository a template, anyone with access to the repository can generate a new repository with the same directory structure and files."

good suggestion. done!


Incremented version to 2.1.1.

Coming from https://github.com/opensearch-project/opensearch-build/issues/1818 post release activities.

Codecov Report

Merging #688 (ac46868) into 2.1 (fc476e5) will decrease coverage by 31.91%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##                2.1     #688       +/-   ##
=============================================
- Coverage     94.67%   62.76%   -31.92%
=============================================
  Files           280       10      -270
  Lines          7570      658     -6912
  Branches        558      119      -439
=============================================
- Hits           7167      413     -6754
+ Misses          349      192      -157
+ Partials         54       53        -1

Flag
Coverage Δ

query-workbench
62.76% <ø> (ø)

sql-engine
?

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

Impacted Files
Coverage Δ

...ggregation/ExpressionAggregationScriptFactory.java

...nsearch/data/value/OpenSearchExprValueFactory.java

...search/executor/protector/ResourceMonitorPlan.java

...java/org/opensearch/sql/utils/ExpressionUtils.java

...age/script/filter/lucene/relevance/MatchQuery.java

...c/main/java/org/opensearch/sql/sql/SQLService.java

...ression/operator/arthmetic/ArithmeticFunction.java

...ch/sql/opensearch/storage/OpenSearchIndexScan.java

...ensearch/sql/sql/parser/AstAggregationBuilder.java

...pensearch/sql/protocol/response/format/Format.java

... and 260 more

Continue to review full report at Codecov.

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

Pending on org.opensearch:opensearch-ml-client:<IP_ADDRESS>-SNAPSHOT release

Pending on org.opensearch:opensearch-ml-client:<IP_ADDRESS>-SNAPSHOT release

I see it here https://aws.oss.sonatype.org/content/repositories/snapshots/org/opensearch/opensearch-ml-client/<IP_ADDRESS>-SNAPSHOT/

there is still no org.opensearch:opensearch-ml-client:<IP_ADDRESS>-SNAPSHOT?
https://aws.oss.sonatype.org/content/repositories/snapshots/org/opensearch/opensearch-ml-client/<IP_ADDRESS>-SNAPSHOT/

Closing this as 2.2.0 is released and this version will not be out now.
Thanks!


improvement Add more information for show processlist

Proposed changes
Here is the result of show processlist
mysql> show processlist;
+------+------+--------------------+-----------------+-------+---------+------+-------+------+
| Id   | User | Host               | Cluster         | Db    | Command | Time | State | Info |
+------+------+--------------------+-----------------+-------+---------+------+-------+------+
|   84 | root | <IP_ADDRESS>:48794    | default_cluster | demo  | Query   |    0 |       |      |
|   88 | root | <IP_ADDRESS>:56191 | default_cluster |       | Sleep   |  752 |       |      |
|   89 | root | <IP_ADDRESS>:56194 | default_cluster |       | Sleep   |  198 |       |      |
|   90 | root | <IP_ADDRESS>:56196 | default_cluster | tpcds | Sleep   |   31 |       |      |
|   91 | root | <IP_ADDRESS>:56197 | default_cluster | demo  | Sleep   |  749 |       |      |
+------+------+--------------------+-----------------+-------+---------+------+-------+------+

In previous versions, the State and Info columns did not display any content, which was not expected. So I fix this problem and remove the unused Cluster column.
As the same time, I added some additional columns. The final changes are as follows.
mysql> show processlist;
+------------------+------+------+-----------------+---------------------+------------+-------+---------+------+-------+-----------------------------------+------------------+
| CurrentConnected | Id   | User | Host            | LoginTime           | Catalog    | Db    | Command | Time | State | QueryId                           | Info             |
+------------------+------+------+-----------------+---------------------+------------+-------+---------+------+-------+-----------------------------------+------------------+
| Yes              |    0 | root | <IP_ADDRESS>:54318 | 2023-09-07 15:18:53 | internal   |       | Query   |    0 | OK    | 915e90179c434adf-9af2eedb03fd19bd | show processlist |
|                  |    1 | root | <IP_ADDRESS>:54834 | 2023-09-07 15:19:10 | jdbc_mysql | mysql | Sleep   |    3 | EOF   | 3255a97e7e04c69-a4f0b837ce742ff9  | select 1         |
+------------------+------+------+-----------------+---------------------+------------+-------+---------+------+-------+-----------------------------------+------------------+

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buidlall

run buildall

run p0


ERM/ACQ - Update for derived table "finance_invoice_transactions.sql"

Closes #791
Added expense class informations and the external account number extension to the derived table "finance_invoice_transactions.sql".
With this derived table we have the opportunity to view all invoice lines regardless of the workflow status. The invoice lines are linked to the transactions if one exists. The expense classes have been newly added. You can now see what expense class the transactions and invoice lines are.

I added the necessary directive from Metadb to ensure the required table "expense_class__t".

Hi,
One comment, if you can make this quick change before it gets merged: In your comment statement "COMMENT ON COLUMN finance_invoice_transactions.invoice_line_distribution_type IS 'Percentage or amount type of the value property';" I was confused by the word 'property' at the end. Maybe that should be deleted, or else it should be "Property of the value (percentage or amount)" or it can be "Type of value (percentage or amount)".
I approved the pull request but for some reason the checklist shows that 'Query runs without errors' does not have a green checkmark next to it. So just to clarify, it DOES run without any errors.

Thank you @VandanaShah-Cornell . I changed the comment for the column.


Fix grammar in README

Just fix the grammar..

Thanks, but I am not too sure about this and am leaning towards keeping the current implementation. Cheers!


Mod not working on Beta 5

Even though I am subscribed to the mod on steam, it is not allowing me to modify any tracks.

I've been thinking bout hiding the mod and just doing a rewrite and everything. It's not that functional anymore with all the changes to parkitect.


Set minimum version of the curl-sys crate.

This sets the minimum version of curl-sys to the latest to ensure that the correct version of curl-sys is used.
Closes #548

Thanks! I always forget to do this nowadays... I'm going to merge this but I'll want to fix CI in https://github.com/alexcrichton/curl-rust/pull/550 before publishing


fix web thumbnail resizing

Right now when resizing thumbnails on web, the image gets drawn to a canvas that starts the drawing at a y or x value based on the scale of the image. In some cases where images are landscape, this is problematic. See: https://bsky.app/profile/retr0.id/post/3khpgllcczr2j
We should always set the x and y axes to start drawing at (0,0). The height and width of the canvas should be set to the scaled height and width of the image.
Before:

After:

Note still though that the image is a bit too zoomed in. This was an error in the styles. Right now, there's a style applied to the image that sets the height to 200 regardless of platform, while the container is set to 200 on native and 120 on web. I fixed that by just applying flex: 1 to the image so it conforms to the correct bounds on all platforms.
Now they resize correctly and appear correctly on both platforms.

Sweet. Folded into https://github.com/bluesky-social/social-app/pull/2398


feat(SDK): implements the Daydream SDK for VRTK

The pull request implements the Daydream SDK for VRTK, which integrates the Google VR Daydream platform with VRTK. See the readme VRTK/Assets/VRTK/SDK/Daydream/DAYDREAM_README.md for usage documentation and details.

There were the following issues with your Pull Request

Commit: 3539f8c533a42a2b01b9fdff10d0cbccf2a39810

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 9153506f3627ed5c76c180c7f19914ff66b87e94

Commits must be in the following format: %{type}(%{scope}): %{description}

Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore, revert

Commit: cf1753bba3ff440db2942d17579c1f02088a41fb

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 7872c1a67ea04ca9a291fd771fd194ceab204213

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: ed3e317372b8534a4fd4fd90dd2d2b6bdcf45611

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 4ae87521d040998efbbfd5a52b06ec1dec9d2d6f

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 0c6a3be87979df89f815725ed28d8c69dab1939a

Commits must be in the following format: %{type}(%{scope}): %{description}

Commit: 90923941fdb55b40760054bfd48bc1c735a2efb0

Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

There were the following issues with your Pull Request

Commit: c46c10469a97e5f7cab679b6ea60699f95bae618
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com

Don't add a new commit to resolve the review.
Do a git commit --amend on to your previous commit.
As you now have two commits you'll probably have to git rebase -i HEAD~3 and squash your commits together to make them one commit.

There were the following issues with your Pull Request

Commit: c46c10469a97e5f7cab679b6ea60699f95bae618
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/thestonefox/SteamVR_Unity_Toolkit

This message was auto-generated by https://gitcop.com


Rookout Agent Chart

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:

Hi @unguiculus is it ok now?

Looks good but you need to add a readme and NOTES.txt.

Hey @unguiculus can you merge this pull request?
Is there anything missing?

/ok-to-test

@unguiculus

/assign @unguiculus

/ok-to-test

/lgtm


*: add metrics for checking memory locks

What problem does this PR solve?
There was no metrics about how long checking memory locks takes. Usually it is fast, but it is still useful for diagnosis.
What is changed and how it works?
Track all memory lock checking durations in histograms.
Check List
Tests

Manual test (add detailed scripts or steps below)

Raft Process:

Scheduler-Get:

Coprocessor Details:

Release note

No release note.

/merge

/merge

/test


Class type parameters declaration (indentation)

I have the following class declaration
class BiAdapter<C : RecyclerView.ViewHolder, V1 : C, V2 : C, out A1, out A2>(
    val adapter1: A1,
    val adapter2: A2
) : RecyclerView.Adapter<C>()
    where A1 : RecyclerView.Adapter<V1>, A1 : ComposableAdapter.ViewTypeProvider,
          A2 : RecyclerView.Adapter<V2>, A2 : ComposableAdapter.ViewTypeProvider {

And I get an error while using klintCheck, because the last line has a 10 spaces indentation (to have A2 aligned with A1, and it's done automatically be Intellij
Not sure what is the recommended format in this case, this format feels the most natural, I guess
) : RecyclerView.Adapter<C>() where
A1 : RecyclerView.Adapter<V1>, A1 : ComposableAdapter.ViewTypeProvider,
A2 : RecyclerView.Adapter<V2>, A2 : ComposableAdapter.ViewTypeProvider {

would fix the error, but I'm not sure it's the recommended format.
I remember reading an issue in some github about code style for kotlin, where basically no one would agree on a standard format.
What's the recommendation for this in Klint?

Hi @badoualy.
Thank for you the ticket.
It looks like vertical alignment is a way to go in this case (https://kotlinlang.org/docs/reference/generics.html#upper-bounds). I'll disable indent check inside where clause for now. We'll revisit it as part of (long-overdue) indent rule overhaul.


Add beret to clothesmate

About the PR

Why can you currently only get berets in maints and security? It's dumb having to ask security every single round when i want drip, so i just added it for public use.
Changelog

:cl: Ubaser

tweak: The Clothesmate now dispenses berets.

i love clothesmate bloat


Force Win32 input handling conflicting with Force Nave when open client with project

Build: Force 0.3.9pre2 & Force Nave 1.0.2
When i launch (not released yet) Force Nave, and from it try to launch the Force client, it seams break Win32 API input handling callback and when i try to close Force then it actually closes the Force Nave, because firstly Force Nave was launched, register the Win32 API class with name "Force" and uses the same static function callback that controlling the Force Nave input handling.
Update 16.11.2022:
I digged out the problem and its actually only blocks the input from Force, when i launch the Force from Nave with opening project from command line. If i open Force from Nave without project its not cause this problem.

TODO: Fix that.

Fixed, problem was a lot easily that i think. Basically one flag on editor startup was on reloading or loading sets to incorrect value.


Keyboard types double

Version: Latest Play Store
Device: OnePlus One
Keyboard: Google Keyboard
Basically, when I type on the keyboard, pretty much every character types double except for the period (".").
So if I wanted to type "Hello." It would become "Hheelloo."
Just wanted to point that out.

It also does that on Xiaomi Redmi 3 with Hacker's keyboard.

Same on the samsung galaxy s5 with regular Samsung keyboard.


Add peerDependencies to the package

For eslint, this makes sense since this package cannot be used outside the context of an ESLint environment.
For the two plugins, their inclusion is to support users using npm2, which unlike npm3 and above can fail to resolve certain dependencies specified here if they are also specified in other parts of the parent package. These will be removed from peerDependencies in version 4 of this package.

The need to support npm2 was discovered in https://github.com/brigade/react-waypoint/pull/162

If we're adding eslint as a peerDependency, we should remove it from dependencies.

@steverice good call, done.


[ADP-2868] Use cbor to deserialize transactions when present.

Overview
When listing all transactions, the wallet currently retrieves transaction data from the database by joining several tables. This task is about using the CBOR — when available — to retrieve the transaction content instead. We expect this to increase the running time of this operation.
Comments
Storing the CBOR of transactions in the database is a fairly recent feature — wallets with very old databases may not benefit from the improved running time.
Issue Number
ADP-2868

bors r+

bors r+

bors r+


Bug fix #242 - Long note titles cover the note options button

Closes #242

@taniarascia It would be nice to have tests that accompany bug fixes like these so we know future changes don't cause regressions and so future devs can see undoubtably what the expected behavior is


Connection state

@rushikesh-kulkarni you need to resolve this merge conflict. Pull the v0.13.0 branch in your current branch.

@sharadregoti what's the status of this PR?

@rushikesh-kulkarni resolve the merge conflicts

@YourTechBud why we are not using project field that we are getting from the request url

@YourTechBud why we are not using project field that we are getting from the request url

As I mentioned earlier, project field must be there


TypeError ... r.desktop.username

Hi,
I installed your theme on fresh Arch VM with bspwm. I get this in seat0-greeter.log:
...litarvan/js/app.fc965120.js:1:13146: CONSOLE JS ERROR TypeError: undefined is not an object (evaluating 'r.desktop.username').
LightDM ask me if I want load default thame:

on cancel I get black screen,
as default theme it run antergos theme.

I don't know JS, VUE ect. but I tried change that r.desktop.username to r.user.username, r.desktop, r.user.desktop or "".
It permit to run theme when I click cancel on error message, but without WM selector and when I input password and Enter, I get black screen. Obviously because I can't chose one.

On the other VM I didn't have this error.

On other VM I don't get this error.


[Harvesting - WMS Client] Design client model, operations and configurable options

The client will support:

filtering of remote resources to be listed (title, time window, categories, keywords)
selection of remote resources to be harvested (list of names or filtering by title, time window, categories, keywords
update metadata for previously harvested resources
implement general helth check on remote service

A considerable portion of this work will be shared with issue #175 - I imagine the WMS specific parts of the work to take a small part in the overall task. I plan to include a WmsHarvester in my current PR geonode/geonode#7317

The estimate I provide here is to be considered in addition to the estimate in #175

@meomancer it seems like what is missing here is mainly the implementation of filtering options for the WMS harvester. However, I have doubts regarding @giohappy original list of filter parameters required for WMS harvester:

[ ] title - I already implemented the dataset_title_filter option, which is able to filter datasets by their title.

[ ] keywords and categories - AFAIK WMS does not really have any native notion about these, so not sure what we should do about it. Some WMS servers may support an additional URL that can be fetched for obtaining metadata about served datasets, but AFAIK this is not mandatory. It also means making additional http requests. Personally I would not implement such a feature for now. @giohappy did you have some other idea in mind? Do you agree with not implementing these as filtering options for now?

[ ] time window - IMO implementing temporal filtering for WMS datasets is not really worth it for the moment - it is difficult to correctly parse the WMS temporal dimension format, due to there being several ways to express acceptable TIME values (a list of single values, open ended ranges, interpolating values in between, etc). I also think this would not bring too much value for end users as time-enabled WMS layers usually represent dynamic datasets which can change often. From a catalogue-like application like GeoNode I think it makes more sense to be aware of the whole collection of datasets served by a time-enabled WMS rather than focusing on specific temporal entries inside of it - @giohappy do you agree with postponing implementation of this filter to the future?

Based on @giohappy's feedback we may be able to consider this issue as done, or not. In either case, please see it through @meomancer

hi @giohappy
could you check this ticket please?

I'm fine to limit the filtering to the title only for WMS.

given @giohappy's comment above, and since filtering by title is already implemented, I'm closing this


Update script so it doesn't build everything

Signed-off-by: Erin Schnabel<EMAIL_ADDRESS>

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Alerts

*Issue:*https://github.com/bcgov/entity/issues/21301
Description of changes:
This adds the alerts and covers the issue #21301
It is dependent on the changes to the business store that add access to all of the api values
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the namex license (Apache 2.0).

If there are multiple alerts, when one alert is expanded, top padding and bottom padding are added to this alert so it is separated from others. I guess we don't have many example with multiple alerts, but you can refer to the 'Recent Filing History' section which has the same behavior.

If there are multiple alerts, when one alert is expanded, top padding and bottom padding are added to this alert so it is separated from others. I guess we don't have many example with multiple alerts, but you can refer to the 'Recent Filing History' section which has the same behavior.

/gcbrun

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-12-r41fmuic.web.app

It seems that the alert section will push down the side components.

 It seems that the alert section will push down the side components.

Thanks, fixed

/gcbrun

Temporary Url for review: https://business-dashboard-dev--pr-12-r41fmuic.web.app


design batch queries workflow

When an user executes a long running task (for example, update the whole table, casting a column) the query may take several minutes
The table should be able to work in that scenario, blocking some user actions and reporting to the user that the query is on progress, the same than geocoding and importing.
cc @saleiva @carlostallon @piensaenpixel

Ok, we have an element that could be reused.


fix column initial value for newline

refer to #4868

reset column to 1 and increase line when matching newline character.
fix unit test.

Codecov Report

Merging #5590 (b89ff9f) into master (eabaeb8) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #5590      +/-   ##
============================================
- Coverage     58.71%   58.70%   -0.01%
+ Complexity    27638    27633       -5
============================================
  Files          1323     1323
  Lines        127098   127089       -9
  Branches      23929    23927       -2
============================================
- Hits          74622    74606      -16
- Misses        43204    43209       +5
- Partials       9272     9274       +2

Files
Coverage Δ

.../main/java/com/alibaba/druid/sql/parser/Lexer.java
60.12% <100.00%> (-0.18%)
:arrow_down:

... and 3 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


v<IP_ADDRESS> doesn't build on 32bit

Maybe only these two imports are missing:
--- LowLevel.hs.orig    2001-09-09 03:46:40.000000000 +0200
+++ LowLevel.hs 2021-10-03 08:50:37.760297876 +0200
@@ -27,4 +27,6 @@
 #if WORD_SIZE_IN_BITS != 64
 import Data.Bits (shiftR)
+import Data.Int (Int32)
+import Data.Word (Word32)
 #endif

I believe this is what broke it https://github.com/haskell-unordered-containers/hashable/pull/227
@phadej

Thanks for the report, uploaded https://hackage.haskell.org/package/hashable-<IP_ADDRESS> hopefully fixing the issue.

Thanks for the report, uploaded https://hackage.haskell.org/package/hashable-<IP_ADDRESS> hopefully fixing the issue.

builds here

@hasufell thanks for the confirmation.


Support Safe Area View

Ask your Question
I can not wrap the Modal component by SafeAreaView to handle layout properly. Have anyone ever faced this issue? Please support me! Thank you

Hi @HayTran94
Instead of wrapping Modal, you can wrap its children. Here's a quick example:
<Modal isVisible={isVisible} style={{margin: 0}}>
  <SafeAreaView style={{flex: 1}}>
    <View style={{flex: 1, backgroundColor: 'white'}} />
  </SafeAreaView>
</Modal>

This is what it looks like:

None of these solutions worked for me. I've tried wrapping every single component in safearea view, I've tried using device height and device width. I am still having to use paddingBottom to get my buttons up above the safe are inset.
SOLUTION I found that works for both IOS 8 through IOS 10+. I found SafeAreaView is simply not working for the modal. Instead, I just provide padding bottom to my child component to push it up. Luckily on other Iphone 8 this bottom would be 0, but in IOS 10+ it is 34 or something so it works!.
    <Modal
        isVisible={showTimerSetupModal.show}
        style={styles.timerSetupModal}
        coverScreen={false}
        animationIn={"fadeIn"}
        animationOut={"fadeOut"}
... whatever else you add
      >
       <MyComponent  /> //which is on the bottom of the screen
      </Modal>

Then in <MyComponent /> code
import { useSafeAreaInsets } from "react-native-safe-area-context";
const insets = useSafeAreaInsets();
Style for the top level container is below
<View
      style={[
        styles.container,
        {
          backgroundColor: appTheme.colors.backgroundColor,
          borderBottomColor: appTheme.colors.backgroundColor,
          paddingBottom: insets.bottom,
        },
      ]}
    >
... The rest of the component
</View>

For me the content inside the Modal only respected the SafeAreaView on Android not on iOS, I isolated the issue to paddingTop and the below code worked for me by using the useSafeAreaInsets hook
<Modal visible={modalVisible}>         <SafeAreaView           style={{             flex: 1,             paddingTop: Platform.OS === 'ios' ? insets.top : 0,           }}         >           <Button title="Close" onPress={() => setModalVisible(false)} />         </SafeAreaView>  </Modal>


Loading in Pyodide fails due to missing cryptography package

We are loading bumble in Pyodide in a browser, and running into an issue where it says the cryptography package is not installed.
The installation code for bumble looks like this"
await pyodide.loadPackage('micropip');
await pyodide.runPythonAsync('import micropip; await micropip.install("bumble")')

And the error message is
PythonError: Traceback (most recent call last):
  File "/lib/python311.zip/_pyodide/_base.py", line 540, in eval_code_async
    await CodeRunner(
  File "/lib/python311.zip/_pyodide/_base.py", line 363, in run_async
    coroutine = eval(self.code, globals, locals)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<exec>", line 22, in <module>
  File "/lib/python3.11/site-packages/bumble/device.py", line 158, in <module>
    from .pairing import PairingConfig
  File "/lib/python3.11/site-packages/bumble/pairing.py", line 28, in <module>
    from .smp import (
  File "/lib/python3.11/site-packages/bumble/smp.py", line 59, in <module>
    from . import crypto
  File "/lib/python3.11/site-packages/bumble/crypto.py", line 28, in <module>
    from cryptography.hazmat.primitives.ciphers import Cipher, algorithms, modes
ModuleNotFoundError: The module 'cryptography' is included in the Pyodide distribution, but it is not installed.
You can install it by calling:
  await micropip.install("cryptography") in Python, or
  await pyodide.loadPackage("cryptography") in JavaScript
See https://pyodide.org/en/stable/usage/loading-packages.html for more details.

I can easily add the micropip.install("cryptography") call, but I am wondering if setup.cfg can be updated to avoid the issue in the first place?
https://github.com/google/bumble/blob/5af347b499b26c2d0431f2a00a3c4f44290e6ec2/setup.cfg#L38
Thank you for your time!

You're 100% right. cryptography should be installed as a dependency via setup.cfg. This fell through the cracks as I forgot the examples included an explicit install of the package that was put there during development. I'll push a PR shortly to fix that.

Sorry for the snafu. I had to revert the merged PR. Now I remember why cryptography was installed "manually" and not via a setup.cfg dependency: there's no pure-python wheel for that package, so pyiodide refuses to load it that way. But installing it via micropip.install will find the built-in one.
Maybe there's a way to get pyiodide to do the right thing even when pulled in as a package dependency, but until/unless we find that alternative, we'll stick with the current appoach.

For reference, the error thrown by pyiodide is:
ValueError: Can't find a pure Python 3 wheel for 'cryptography==39'.
See: https://pyodide.org/en/stable/usage/faq.html#micropip-can-t-find-a-pure-python-wheel


Tests: Coverage and enhancement for matrix_test.go

The build fails with Go 1.3 and 1.4 due to deprecation of some of the libraries https://travis-ci.org/klauspost/reedsolomon/builds/147426330 . @klauspost Can you evaluate this?

CI fixed in #31

LGTM. Thanks for the contribution!


Fading toolbar with image and RecyclerView not working

I'm using code from this Gist and this is what I am trying to achieve (a fading ToolBar). However, when I start up my app I get the following error:
   java.lang.NullPointerException: Attempt to write to field 'int android.view.ViewGroup$LayoutParams.height' on a null object reference
This is referring to the following line of code which is also on line 54 of this, as well as here:
headerView.getLayoutParams().height = mFlexibleSpaceImageHeight;

When I comment out that line of code, the error stops appearing, but I don't get the fading effect I want; instead, the RecyclerView appears above the toolbar. The code I used is exactly the same (I've triple checked) as the following:

XML Activity Layout file
Activity
Base Activity
dimens.xml
recycler_header.xml
padding.xml

Here's the problematic scrolling effect:

The image behind the list is inside of the toolbar; that image, along with the toolbar should be in front of the list. The green bar in the other picture where I have scrolled further down the list is the toolbar - as you can see, the intended effect is not being achieved.

Im having the same issue... anyone have a solution?


Add failing spec for invalid tsquery generation

This is a failing test for a bug we ran into: when searching on an unaccented pg_search_scope for a query which includes ‘, an invalid expression is generated:
     ActiveRecord::StatementInvalid:
       PG::SyntaxError: ERROR:  syntax error in tsquery: "' L'Content '"
       : SELECT "with_model_model_with_pg_searches_57799_70174002903340".* FROM "with_model_model_with_pg_searches_57799_70174002903340" INNER JOIN (SELECT "with_model_model_with_pg_searches_57799_70174002903340"."id" AS pg_search_id, (ts_rank((to_tsvector('simple', unaccent(coalesce("with_model_model_with_pg_searches_57799_70174002903340"."title"::text, '')))), (to_tsquery('simple', ''' ' || unaccent('L‘Content') || ' ''')), 0)) AS rank FROM "with_model_model_with_pg_searches_57799_70174002903340" WHERE (((to_tsvector('simple', unaccent(coalesce("with_model_model_with_pg_searches_57799_70174002903340"."title"::text, '')))) @@ (to_tsquery('simple', ''' ' || unaccent('L‘Content') || ' '''))))) AS pg_search_03f80d9a6ae537cac323d6 ON "with_model_model_with_pg_searches_57799_70174002903340"."id" = pg_search_03f80d9a6ae537cac323d6.pg_search_id ORDER BY pg_search_03f80d9a6ae537cac323d6.rank DESC, "with_model_model_with_pg_searches_57799_70174002903340"."id" ASC

It wasn't immediately obvious to me how to fix this (I had a look at normalizer.rb) so I'm posting this here, maybe it's an easy fix for you, or you can give me a hint where to look. In case it's important, the project where this happened is running on Rails 4.2.

So Travis defaults to Postgres 9.1, where this issue doesn’t come up. With Postgres 9.6 the spec also fails on Travis.

So the problem here are left & right single quotation marks (http://www.fileformat.info/info/unicode/char/2018/index.htm, http://www.fileformat.info/info/unicode/char/2019/index.htm) which. This can be worked around by adding these characters to DISALLOWED_TSQUERY_CHARACTERS.
What I think is happening is this:

SELECT ''' ' || unaccent('L‘Content') || ' ''' returns ' L'Content '
this is passed to to_tsquery, which cannot work with that string (because the apostrophe in the string is not properly escaped)

Not sure about this, I guess there is a more proper fix.

+1 This solves the issue for me.

👍 This also fixes the issue for me.

@nertzy Is there anything we can do to help get this merged? Thank you 💛

@nertzy Thank you!

@nertzy can we have a release with this fix?

Now works, thanks!


Pytorch tensor input support

Fixes #206
Overview
Pytorch models currently do not accept inputs of type torch.tensor. This is because when loading torch models, Garden usesmlflow.pyfunc.load_model, which makes a mlflow model that can accept inputs of type np.ndarray and pd.dataframe but not torch.tensor. However, if Garden loads torch models with mlflow.pytorch.load_model, the model accepts inputs of type torch.tensor but not np.ndarray and pd.dataframe.
For the time being, all torch loads have been changed to mlflow.pytorch.load_model and assume torch users will always be using inputs of type torch.tensor.
In the future, we might want to consider adding our own model signatures so users can specify input types of their models.

:books: Documentation preview :books:: https://garden-ai--244.org.readthedocs.build/en/244/

Codecov Report

Merging #244 (c4b22cd) into main (6b769c3) will increase coverage by 0.05%.
The diff coverage is 83.33%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #244      +/-   ##
==========================================
+ Coverage   78.40%   78.45%   +0.05%
==========================================
  Files          46       46
  Lines        3209     3249      +40
==========================================
+ Hits         2516     2549      +33
- Misses        693      700       +7

Files Changed
Coverage Δ

garden_ai/mlmodel.py
72.29% <33.33%> (-0.81%)
:arrow_down:

garden_ai/_model.py
52.63% <73.68%> (+4.57%)
:arrow_up:

tests/conftest.py
95.71% <100.00%> (+0.16%)
:arrow_up:

tests/test_mlmodel.py
100.00% <100.00%> (ø)


Partially Paid / Underpaid

Opencart is not accepting any calls from Mycelium gear with "Partially Paid" and has a status of "Callback response status: 302"
Perhaps we need an additional "Order Statuses" field in the extension settings for underpaid if this is possible?
Right now, underpaid orders remain "Missing orders" in opencart.

Yep, currently the plugin ignores the callbacks sent with underpaid transactions. At least until another transaction is made to the same address with the remaining amount (haven't tested this but if mycelium sends a new callback if the payment is made in more than 1 part it should work).
https://github.com/jaimehgb/Mycelium-Opencart-plugin/blob/master/upload/catalog/controller/payment/mycelium.php#L257
Case 3 (missing) is the one which would handle underpaid orders. I'll fix that tomorrow (hopefully :P )

Transactions still do not seem to be receiving the callbacks when "partially paid". Perhaps this is an issue with mycelium not sending the callback until the status is set to "underpaid"?
Partially paid is set on mycelium for X minutes until order expires, however it would be nice if the partial is considered the underpayment. Allowing customers to top up their balance.
When I check the transaction in mycelium, there is no data in Callback response field.


fix: various typos in README and code

I noticed a few typo errors and wording.
Here is a PR to fix them.

Indeed, I'm sorry, let me fix.

pushed back, please review

Thank you so much @ccoVeille!


[Request] Add n8n

https://github.com/n8n-io/n8n
https://docs.n8n.io/hosting/

It's a very powerful node-based workflow automation tool. Similar to Node-RED, but with a different focus.

Agreed, this is giving me greif trying to add


Typescript migration

Fixes #74

Not stale

From: stale[bot]<EMAIL_ADDRESS>Sent: Monday, July 1, 2019 6:21:21 PM
To: probot/smee-client
Cc: wolfy1339; Author
Subject: Re: [probot/smee-client] Typescript migration (#113)
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/probot/smee-client/pull/113?email_source=notifications&email_token=ABDB6FMQN5K6KQPQLLSTBOTP5J7ODA5CNFSM4HC4VBFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODY7QLEY#issuecomment-507446675, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABDB6FNTIKES5IY2EUXTXZTP5J7ODANCNFSM4HC4VBFA.

Not stale

Ping @JasonEtco

Not stale

Still relevant
On 2020-03-28 2:48 a.m., stale[bot] wrote:

Is this still relevant? If so, what is blocking it? Is there anything
you can do to help move it forward?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/probot/smee-client/pull/113#issuecomment-605405242,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABDB6FOHQQCP4VVKZ6T37ETRJWMU3ANCNFSM4HC4VBFA.

Ping @JasonEtco @tcbyrd

Still relevant
Get Outlook for Androidhttps://aka.ms/ghei36

From: stale[bot]<EMAIL_ADDRESS>Sent: Friday, June 26, 2020 6:41:29 PM
To: probot/smee-client<EMAIL_ADDRESS>Cc: wolfy1339<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [probot/smee-client] Typescript migration (#113)
Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/probot/smee-client/pull/113#issuecomment-650435135, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABDB6FOJGK4CZEWMP4SNR4LRYUPZTANCNFSM4HC4VBFA.

As we don't have good test coverage yet (see #155), did you try it locally if this actually works? 😁


newlines, although desirable, eventually wither

in toaq only definitions on restart during database cleaning

What does this mean?

(The context is that some entries that used to have newlines in them had those newlines stripped, making them not very readable.) The conclusion is that Toadua is probably not the right place to put multi-line definitions in, or at least not a priority item. Closing.


VMSS mount premium disk max share

Hi,
I would like to know what is the maximum instance count on VMSS that could mount the premium disk on the controller. We are expecting to have a large number of users (around 10-100K concurrent users), we would like to know if for example we have 20 instances in VMSS that it will still work. As far as I know, there is a max share limit for the premium disk.

Hello @cenon4dno,
Apologies for the late response. VMSS on Azure supports upto 1000 instances. The template by default limits to 20 instances maximum that you can edit at deployment time (autoscaleVmCountMax) and change the 20 number to whatever you wish. To answer your question, 20 will work fine.


Inspector > Hint > Tooltip; mouse scroll wheel down does not work

Godot version:
v3.2.stable.official
OS/device including version:
Windows 7
Issue description:
When using the mouse scroll wheel to scroll down the inspector properties, there is an area inside the Hint>Tooltip that ignores the scroll wheel "down".  The area does recognize a scroll wheel "up", but not a scroll wheel "down".  See screenshot here: https://i.imgur.com/6cRryj5.png
Steps to reproduce:
Open up Godot, add a button node, click on the button to see the inspector tab, then scroll your mouse wheel down.  It will stop scrolling in that Hint>Tooltip area.
Minimal reproduction project:
n\a

Duplicate of #34622. (The underlying issue is the same.)


No input in terminal

VSCode Version: 1.74.0-insider
Local OS Version: iPadOS 15.6.1 (on iPad Pro 7th gen)
Remote OS Version: Linux 5.14.0-1054-oem 61-Ubuntu x86_64 (running in Ubuntu container in Docker version 20.10.12, build 20.10.12-0ubuntu2~20.04.1 on Ubuntu host)
Remote Extension/Connection Type: Server
Logs:

Steps to Reproduce:

Run a fresh Ubuntu container, update, and install wget and curl
In the container:

docker container run --rm -it --name vscode-server ubuntu /bin/bash

In the container:
apt update
apt upgrade
apt install wget curl

In the container install vscode-server following the steps from the website

wget -O- https://aka.ms/install-vscode-server/setup.sh | sh
code-server

In the iPad Safari browser, authenticate and open the suggested link with the unique machine name. Accept any thrust when prompted. The server will be started. The UI is loaded and the file tree is shown in the explorer UI.

https://insiders.vscode.dev/+ms-vscode.remote-server/whatever-machine/

(Everything up until now works as one expects it should). From the Menu (..the sandwich icon) select Terminal/ New terminal.  A new terminal is opened and the prompt is shown, BUT no cursor is visible, and no keyboard input is accepted.

Does this issue occur when you try this locally?:
Terminal is accessible as expected when the same tunnel is opened in the client loaded in Safari on macOS Ventura 13.0 on MacBook Air.
Does this issue occur when you try this locally and all extensions are disabled?:
The issue is observed in a fresh install, no additional extensions installed yet.

Might be the same as https://github.com/microsoft/vscode/issues/163448

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


[NT]sorry for this Issue: data download failed, would you please recheck these urls or upload this data to other place.

Willingness to contribute
Yes. I can contribute for this issue independently.
Non-technical Request Proposal
PRODUCT_RECOMMENDATION_USER_OBSERVATION_URL = "https://azurefeathrstorage.blob.core.windows.net/public/sample_data/product_recommendation_sample/user_observation_mock_data.csv"
PRODUCT_RECOMMENDATION_USER_PROFILE_URL = "https://azurefeathrstorage.blob.core.windows.net/public/sample_data/product_recommendation_sample/user_profile_mock_data.csv"
PRODUCT_RECOMMENDATION_USER_PURCHASE_HISTORY_URL = "https://azurefeathrstorage.blob.core.windows.net/public/sample_data/product_recommendation_sample/user_purchase_history_mock_data.csv"
PRODUCT_RECOMMENDATION_PRODUCT_DETAIL_URL = "https://azurefeathrstorage.blob.core.windows.net/public/sample_data/product_recommendation_sample/product_detail_mock_data.csv"

I'm having this issue as well. The DNS name azurefeathrstorage.blob.core.windows.net is not resolvable.


chore: update toolchain to 20231213

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
Major intention: to include https://github.com/rust-lang/compiler-team/issues/688
By this toolchain, the binary size with debug info is further cut by ~300MB, to only 800MB now
Checklist

[ ]  I have written the necessary rustdoc comments.
[ ]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

Looks like there is a rustc ICE on this version. Convert to draft


Sub Menu Item Overlapping Issue (Screenshot Attached)

Previous SubMenu window should be closed on the selection of next SubMenu

Foreign Currency > Rates
Types
Should be Close on Click to Store Room

Still not working for seome deep nesting cases

Still not working
https://paste.apache.org/y00xp

@pashminakazi can you look into compiled file CascadingMenuSelectionMouseController.js and see whether there are changes from @yishayw ? https://github.com/apache/royale-asjs/commit/d5556b3a31a73346ea6edb70db514f081322347e and https://github.com/apache/royale-asjs/commit/ef6466d49b95a807658cc9ef3318c49c33d32fdc

d5556b3 changes

ef6466d changes

clearSubmenusOnSameLevel Function

getMenuWithDataProvider Function

You can try debug yourself and see what has happened. @yishayw is on vacation - he will back at the beginning of September.

Thanks for reporting this. Hopefully I'll have time to look at this tomorrow.

Looks like this is not working for menus with xml data. The reason being that the sub menu data provider is recreated each time it's accessed and this cannot be used as an identifier to retrieve the open menu.

@yishayw Thanks issue resolved.


How do you actually use a custom easing function?

I'd like to use a different easing function than the default, but can't figure how you actually pass through a new easing function in a way that Scrollchor will accept without an error.

I believe the current implementation of easing does not provide the correct arguments for the custom easing function, this is one of the things I tried to address in my PR #40. If you need custom easing support right now I suggest trying out my fork of this package, @seinopsys-forks/react-scrollchor, othwerise you can wait until the PR is accepted.

I think it'd be a great idea to make the default easing be something a little more straightforward, less opinionated, something like (easeOutSine)[https://easings.net/#easeOutSine].
For now tho, it would be great to just get an example of how to format for say Linear easing so that it can be passed through the Animation property and work with current code.

I honestly have no clue what sort of arcane magic is powering the implementation in its current form, I know I specifically adjusted my fork because I could not figure this out and wanted to use a more "standard" approach. The PR has sources for two basic easings here: https://github.com/SeinopSys/react-scrollchor/blob/v7/src/easing.ts
These are basically taken verbatim from the jQuery easing library and I'm confident you could plug any of the other easing from it in there and it will work.

@joshuatuscan  @SeinopSys  sorry for the delay!
What I did, for the "deprecated" implementation, was copy and paste the easing function from here: https://github.com/danro/jquery-easing/blob/master/jquery.easing.js although that no longer matters because @SeinopSys  has improved it a lot
Version 7.0.0 of this package had bee release with @SeinopSys PR's who did a wonderful job cleaning this module.
Now it seems that everything is in its place.

I thiks is already solved.
Feel free to reopen it if needed


invalid regex in YAML indent plugin

Steps to reproduce
Run this shell command:
vim -Nu NONE -S <(tee <<'EOF'
    vim9script
    var lines =<< trim END
        # vim: set ft=yaml sw=2 et :

        # START_INDENT
        map1:
        sub1:
        - list item
        map2:
        - another list
        # END_INDENT

        # START_INDENT
        map: &anchor
        map: val
        # END_INDENT

        # START_INDENT
        map: |
        line1
        line2
        # END_INDENT
    END
    lines->setline(1)
    filetype indent on
    &filetype = 'yaml'
    &debug = 'throw'
    normal! gg=G
EOF
)

An error is given:
E678: Invalid character after %[dxouU]

Expected behavior
No error is given.
Version of Vim
9.0 Included patches: 1-353
Environment
Operating system: Ubuntu 20.04.5 LTS
Terminal: XTerm(353)
Value of $TERM: xterm-256color
Shell: GNU bash, version 5.0.17
Additional context
This patch should fix the issue:
diff --git a/runtime/indent/yaml.vim b/runtime/indent/yaml.vim
index d732c37c0..3ed48ffcd 100644
--- a/runtime/indent/yaml.vim
+++ b/runtime/indent/yaml.vim
@@ -54,7 +54,7 @@ let s:c_ns_anchor_name = s:c_ns_anchor_char .. '+'
 let s:c_ns_anchor_property =  '\v\&' .. s:c_ns_anchor_name

 let s:ns_word_char = '\v[[:alnum:]_\-]'
-let s:ns_tag_char  = '\v%(%\x\x|' .. s:ns_word_char .. '|[#/;?:@&=+$.~*''()])'
+let s:ns_tag_char  = '\v%(\x\x|' .. s:ns_word_char .. '|[#/;?:@&=+$.~*''()])'
 let s:c_named_tag_handle     = '\v\!' .. s:ns_word_char .. '+\!'
 let s:c_secondary_tag_handle = '\v\!\!'
 let s:c_primary_tag_handle   = '\v\!'
@@ -63,7 +63,7 @@ let s:c_tag_handle = '\v%(' .. s:c_named_tag_handle.
             \            '|' .. s:c_primary_tag_handle .. ')'
 let s:c_ns_shorthand_tag = '\v' .. s:c_tag_handle .. s:ns_tag_char .. '+'
 let s:c_non_specific_tag = '\v\!'
-let s:ns_uri_char  = '\v%(%\x\x|' .. s:ns_word_char .. '\v|[#/;?:@&=+$,.!~*''()[\]])'
+let s:ns_uri_char  = '\v%(\x\x|' .. s:ns_word_char .. '\v|[#/;?:@&=+$,.!~*''()[\]])'
 let s:c_verbatim_tag = '\v\!\<' .. s:ns_uri_char.. '+\>'
 let s:c_ns_tag_property = '\v' .. s:c_verbatim_tag.
             \               '\v|' .. s:c_ns_shorthand_tag.

I don't understand the regex, so I'm not sure it's correct.  However, \v%\x is not a valid pattern.  Removing the percent character seems to be the simplest fix, and it makes sense.  Also, the YAML indent tests still pass.
BTW, the tests don't give any error unless we set 'debug' to throw.  I wonder whether that should be the default:
diff --git a/runtime/indent/testdir/runtest.vim b/runtime/indent/testdir/runtest.vim
index 6bbd33cac..06f948e3b 100644
--- a/runtime/indent/testdir/runtest.vim
+++ b/runtime/indent/testdir/runtest.vim
@@ -11,6 +11,7 @@ syn on
 set nowrapscan
 set report=9999
 set modeline
+set debug=throw

 au! SwapExists * call HandleSwapExists()
 func HandleSwapExists()

I open the issue here because the maintainer doesn't seem to be active anymore.

last change was made by @lukas-reineke   any comments?

This looks like the same issue I fixed in #5937 somehow it got reverted?
Patch LGTM

Thanks for checking, I'll include it.  I don't see the error when running the tests, even when setting the 'debug' option.

I don't see the error when running the tests, even when setting the 'debug' option.

The tests pass no matter what, but at least an error is printed on the terminal:
$ cd runtime/indent/
$ make clean; make test
rm -f testdir/*.fail testdir/*.out
VIMRUNTIME=.. vim --clean --not-a-term -u testdir/runtest.vim
...
"testdir/yaml.in" 20L, 202BError detected while processing /home/lgc/VCS/vim/runtime/indent/testdir/runtest.vim[88]..function GetYAMLIndent:
line   71:
"testdir/yaml.out" "testdir/yaml.out" [New] 20L, 214B written
Test testdir/yaml.in OK

--- /tmp/without	2022-09-02 13:33:21.749974313 +0200
+++ /tmp/with	2022-09-02 13:33:03.222033503 +0200
@@ -19,5 +19,7 @@
 Test testdir/vim.in OK
 "testdir/xml.in" 32L, 437B"testdir/xml.out" "testdir/xml.out" [New] 32L, 709B written
 Test testdir/xml.in OK
-"testdir/yaml.in" 20L, 202B"testdir/yaml.out" "testdir/yaml.out" [New] 20L, 214B written
+"testdir/yaml.in" 20L, 202BError detected while processing /home/lgc/VCS/vim/runtime/indent/testdir/runtest.vim[88]..function GetYAMLIndent:
+line   71:
+"testdir/yaml.out" "testdir/yaml.out" [New] 20L, 214B written
 Test testdir/yaml.in OK

I don't see the error when running the tests, even when setting the 'debug' option.

The tests pass no matter what, but at least an error is printed on the terminal:
$ cd runtime/indent/
$ make clean; make test
rm -f testdir/*.fail testdir/*.out
VIMRUNTIME=.. vim --clean --not-a-term -u testdir/runtest.vim
...
"testdir/yaml.in" 20L, 202BError detected while processing /home/lgc/VCS/vim/runtime/indent/testdir/runtest.vim[88]..function GetYAMLIndent:
line   71:
"testdir/yaml.out" "testdir/yaml.out" [New] 20L, 214B written
Test testdir/yaml.in OK

--- /tmp/without	2022-09-02 13:33:21.749974313 +0200
+++ /tmp/with	2022-09-02 13:33:03.222033503 +0200
@@ -19,5 +19,7 @@
 Test testdir/vim.in OK
 "testdir/xml.in" 32L, 437B"testdir/xml.out" "testdir/xml.out" [New] 32L, 709B written
 Test testdir/xml.in OK
-"testdir/yaml.in" 20L, 202B"testdir/yaml.out" "testdir/yaml.out" [New] 20L, 214B written
+"testdir/yaml.in" 20L, 202BError detected while processing /home/lgc/VCS/vim/runtime/indent/testdir/runtest.vim[88]..function GetYAMLIndent:
+line   71:
+"testdir/yaml.out" "testdir/yaml.out" [New] 20L, 214B written
 Test testdir/yaml.in OK

I'll put a try/catch around the indenting in the test script and add the
error to the file, so that it gets noticed.
--
The word "leader" is derived from the word "lead", as in the material that
bullets are made out of.  The term "leader" was popularized at about the same
time as the invention of firearms.  It grew out of the observation that the
person in charge of every organization was the person whom everyone wanted to
fill with hot lead.
I don't recomment this; it's just a point of historical interest.
(Scott Adams - The Dilbert principle)
/// Bram Moolenaar -- @.*** -- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


Possible Notification discrepancy

I noticed the following example:
+---------------------- Notification
|              +--------- type        "chat message"
|              |       +- payloadSize 50
|              |       |
n00cchat message00000032{"message":"Hi","from":"nthn","chat_room":"gonuts"}

With the following definition:
Notification    = "n" type payload

type            = text3
payload         = payloadSize payloadData?
payloadSize     = hexUInt8
payloadData     = <byte>{payloadSize}

text3           = text3Size text3Value
text3Size       = hexUInt3
text3Value      = <<byte>{text3Size} as utf8 text>

It looks like the text3 isn't documented in the example. Should be:
+---------------------- Notification
|  +--------------------- text3Size   12
|  |           +--------- type        "chat message"
|  |           |       +- payloadSize 50
|  |           |       |
n00cchat message00000032{"message":"Hi","from":"nthn","chat_room":"gonuts"}

This tripped me up a little when working on a quick and dirty Python implementation: https://github.com/gtaylor/python-gotalk

Good catch. However text3Size has been omitted for the sake of readability. The same is done with the StreamRequest example.


Fixed GIT env vars

Was setting them in an earlier step :/
Signed-off-by: Simon Bennetts<EMAIL_ADDRESS>

Failing job: https://github.com/zapbot/zap-mgmt-scripts/runs/1312125339?check_suite_focus=true


Integration tests for Service Discovery

Add integration tests for Service Discovery.

Service Discovery test cases are being tracked in - https://github.com/rancherlabs/the-ranch/wiki/Rancher-Validation-Test-cases


Docs for local "$refs"

I'd love to use the library, but I can't unless I can easily get local "$refs" working.
If this is supported, it could use some documentation / examples.
See https://json-schema.org/understanding-json-schema/structuring.html#reuse for what I'm talking about.

I greatly appreciate you taking the time to look into this. Regardless of the outcome I think adding a couple examples of using local files as refs to the documentation would be very useful.
Here's a gist with reproducible data:
https://gist.github.com/tryangul/573e4ec857f2d55ed821b0aeddb6631f
Note: the root schema is located in /json_schemas while the sub schemas are located in /json_schemas/_sub_schemas

It looks like the issue is your $ref URLs are relative but there's no base URL to resolve them against. I think you could get it working by defining an $id value for the root schema and parsing it out in the ref resolver. From what I can tell, the current behavior is correct according to the spec, but it's not ideal. How does this work in ruby-json-schema?

This works with 0 configuration in ruby-json-schema. Regarding the lack of $id on the root schema, I believe the spec says in this case the base URI should be the URI in which the schema was found see: http://json-schema.org/latest/json-schema-core.html#rfc.section.8.1
It is also my understanding from the spec and the definition of SHOULD here, this use case should be supported.
I will test out providing an absolute file:/ based URI to the root component later as a suitable workaround.

In this case, we don't know the URI where the schema was found
Is this because json_schemer only accepts ruby hashes? Should it not also accept raw json strings and json files?

I will experiment with generating the an absolute file:/ URI and setting it as the $id on the schema hash

Do you mean that ruby-json-schema reads the schemas from the filesystem with zero config?

Yes, it reads them from the file system with 0 config. The example I provided works with the following ruby-json-schema based script

def validate_json(schema_path, json_path)
  {
    is_valid: JSON::Validator.validate(schema_path, json_path),
    errors: JSON::Validator.fully_validate(schema_path, json_path)
  }.to_json
end

puts validate_json(ARGV[0], ARGV[1])

Note: in this case, it knows where the schema was found as it accepts paths.

Is this because json_schemer only accepts ruby hashes? Should it not also accept raw json strings and json files?

Yeah, right now there's no way to know what the base URI should be for the schema hash. I'd rather not allow raw json, since it's pretty easy for someone to parse it into a hash themselves. I'll look into reading from files though. I'm a little wary of doing it automatically but it makes sense as an optional thing.

I'd rather not allow raw json, since it's pretty easy for someone to parse it into a hash themselves.
Fair, but it seems very weird to implement a JSON spec that doesn't accept JSON.

I got it working with the following https://gist.github.com/tryangul/07f4a46b00cf244f2a95317966fb121e
I still advocate to adding something like this to the docs and or extending the ref resolver to allow for more easily setting this up.
If you're willing, I'd happily make a docs PR and we can move discussion there.

Also, probably for another issue, but while this is working, the output is less than ideal
https://gist.github.com/tryangul/de71c05bf5d3553ecc8e433a113ab815
Is this related to https://github.com/davishmcclurg/json_schemer/issues/1 ?

Glad to hear you got it working. You should try out this branch to see if it behaves like you expect: https://github.com/davishmcclurg/json_schemer/compare/file
A docs PR would be great! Docs are basically nonexistent at this point.
The error output for nested schemas is not good. There's been a lot of discussion about standardizing error output somehow: https://github.com/json-schema-org/json-schema-spec/issues/396 https://github.com/json-schema-org/json-schema-spec/issues/643
I didn't want to implement anything more until that was done, but I haven't kept up with it very well.

I have to say I'm struggling to understand how to use this gem with local files.

I've made a small example reproducing my issue.
I've tested the same nested schema (after expanding it) and it seems to validate fine.
By reading the docs I think this should work as expected, but it still gives me (somewhat cryptic) validation errors.
Am I missing something obvious here?
Once I get this figured out I'll see if I submit a PR for some documentation around this.

@Draiken this looks like another case of using non-string keys (https://github.com/davishmcclurg/json_schemer/issues/13).
Your example works if you change it to:
pp schemer.validate({ 'action' => 'check', 'seatId' => 1 }).to_a

This seems to be one of the main issues people have. I'll think on how to fix it.

Oh I wasn't aware of that.
This is unexpected behavior in my view, since this is dealing with JSON where keys are are always strings and the default behaviour of the JSON gem is to convert symbols to strings when serializing.
I'll see if I can get it working with strings.
BTW, thanks for the gem man.

I am having issues to trying to make external $ref to work, I am trying to make the $id value a relative path, but the issue is that if the $ref value is also relative it returns an error:
URI::BadURIError Exception: both URI are relative

I would like in the docs an example of a local ref working, please advice, and thanks for the hard work!

Just an update for anyone else who runs into this.
If you pass in a Pathname instance pointing to the file on disk, instead of the raw JSON content when calling JSONSchemer.schema, it knows where the original schema lives on disk, and relative $ref attributes work fine.
TL;DR:
This:
JSONSchemer.schema(Pathname.new('/my/awesome/schema.json'))

Instead of:
JSONSchemer.schema(File.read('/my/awesome/schema.json'))

@jimeh I couldn't get this to work. Could you give an example of how you're referencing the files?
schema = JSONSchemer.schema(Pathname.new('broadcasts.json'))

# broadcasts.json
{
  "$schema": "http://json-schema.org/draft-07/schema#",
  "$id": "broadcast.json",
  "name": "Broadcast",
  "type": "object",
  "oneOf": [
    { "$ref": "file://./broadcasts/connections.json" }
  ]
}

# broadcasts/connections.json
{
  "$schema": "http://json-schema.org/draft-07/schema#",
  "$id": "connections.json",
  "name": "Connections",
  "type": "object",
  "properties": {
    "name": { "type": "string" }
  }
}

I tried using file:./path, file://./path, ./path, path and none of them worked.

@Draiken you may need to use an absolute path when creating your Pathname object.

@Draiken try giving Pathname an absolute path like @davishmcclurg suggested. The code I've got passes in an abosolute path by doing something like this:
filename = 'foobar.json'
file_path = File.expand_path(File.join('..', 'schemas', filename), __dir__)
JSONSchemer.schema(Pathname.new(file_path))

Also, the schema refs I've got don't use any scheme:// prefixes, so it's just:
{ "$ref": "./broadcasts/connections.json" }


fix: don't record purl for non conda-forge channels

While trying to reproduce #1230 , I've observed that we record purl for packages even that are not conda-forge but from pytorch.
so by having this example:
[dependencies]
captum = {version="*", channel="pytorch"}
boltons = {version = "*"}

[pypi-dependencies]
captum = { version = "*"}

we need to have purl only for boltons, and empty purls for captum with a warning that pypi will overwrite it.

So I guess the CI error is that because you now check if it is a conda-forge URL by checking its channel, it's actually dummy-channnel so you do not ament the purl.
So with this change you would. not expect the package to have a purl :)


AssertionError  assert num_gpu <= torch.cuda.device_count()

Thank you for your great job.
I run the code on "Quadro RTX 5000", and successfully get the results of video demo and MOT17 MOT20 datasets.
But, I fail to train on MOT17 and MOT20 datasets. These tracking and training are carried on one environment.
When I train, I get an error like this:

And I have checked the informations:

How can I solve this problem?
Looking forward to your reply?

I can successfully run the video demo, but fail in training.

Try changing - d argument to 0 or 1. It will probably work.
Otherwise try reducing the batch size as well but - d would solve it

Thank you very much. I have solved this problem.
But when running the code "python3 tools/train.py -f exps/example/mot/yolox_x_mix_mot20_ch.py -d 0 -b 48 --fp16 -o -c pretrained/yolox_x.pth", I meet another error like this:

How can I solve this problem?

Thank you very much. I have solved this problem. But when running the code "python3 tools/train.py -f exps/example/mot/yolox_x_mix_mot20_ch.py -d 0 -b 48 --fp16 -o -c pretrained/yolox_x.pth", I meet another error like this:    How can I solve this problem?

l have the same question you, how you fix it finally?

This may be the cuda version is not suitable.
I run with python 3.8  torch 1.11.0+cu113 and torchvision==0.12.0

This may be the cuda version is not suitable. I run with python 3.8 torch 1.11.0+cu113 and torchvision==0.12.0
thank you, l have fix it


No reset link in email after customizing PasswordResetSerializer

I wanted to overwrite the default PasswordResetSerializer so I made CustomPasswordSerializer which looks like this -
`class CustomPasswordResetSerializer(PasswordResetSerializer):
def get_email_options(self):
    super().get_email_options()
    return {
        'subject_template_name': 'account/email/password_reset_subject.txt',
        'email_template_name': 'account/password_reset_email.html',
        }`

but from this I got an email somewhat like this -

Here I didn't got the reset link and other items.
So I found a solution in which it was told that put your subject and template file in templates/registration/ So it will automatically overwrite. I tried that but I got this error
`django_1     | ERROR 2021-01-08 21:59:27,915 log<PHONE_NUMBER>41565056 Internal Server Error: /api/auth/password/reset/
django_1     | Traceback (most recent call last):
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/core/handlers/exception.py", line 34, in inner
django_1     |     response = get_response(request)
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/core/handlers/base.py", line 115, in _get_response
django_1     |     response = self.process_exception_by_middleware(e, request)
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/core/handlers/base.py", line 113, in _get_response
django_1     |     response = wrapped_callback(request, *callback_args, **callback_kwargs)
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
django_1     |     return view_func(*args, **kwargs)
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/views/generic/base.py", line 71, in view
django_1     |     return self.dispatch(request, *args, **kwargs)
django_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 505, in dispatch
django_1     |     response = self.handle_exception(exc)
django_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 465, in handle_exception
django_1     |     self.raise_uncaught_exception(exc)
django_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 476, in raise_uncaught_exception
django_1     |     raise exc
django_1     |   File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 502, in dispatch
django_1     |     response = handler(request, *args, **kwargs)
django_1     |   File "/usr/local/lib/python3.7/site-packages/rest_auth/views.py", line 187, in post
django_1     |     serializer.save()
django_1     |   File "/usr/local/lib/python3.7/site-packages/rest_auth/serializers.py", line 186, in save
django_1     |     self.reset_form.save(**opts)
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/contrib/auth/forms.py", line 311, in save
django_1     |     user_email, html_email_template_name=html_email_template_name,
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/contrib/auth/forms.py", line 249, in send_mail
django_1     |     subject = loader.render_to_string(subject_template_name, context)
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/template/loader.py", line 61, in render_to_string
django_1     |     template = get_template(template_name, using=using)
django_1     |   File "/usr/local/lib/python3.7/site-packages/django/template/loader.py", line 19, in get_template
django_1     |     raise TemplateDoesNotExist(template_name, chain=chain)
django_1     | django.template.exceptions.TemplateDoesNotExist: password_reset_subject.txt`
By looking at this error when it calls the save() function of PasswordResetForm it can't find the templates.
Now to be clear, I'm contributing to an opensource org in which there are several apps and there is no TEMPLATE_DIR mentioned so basically there are defaults folder of templates in apps where it will look but it's not working, please help me out.
Please do let me know if you need any more information.
settings.py file here
Thanks

This is more of your template related error, please paste your template code here !

This is more of your template related error, please paste your template code here !

Hello @hamhaingaurav thanks for replying.
`
{% load i18n %}{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Hello from {{ site_name }}!
You're receiving this e-mail because you or someone else has requested a password for your user account at {{ site_domain }}.
It can be safely ignored if you did not request a password reset. Click the link below to reset your password.{% endblocktrans %}
{{ protocol }}://{{ domain }}{% url 'password_reset_confirm' uidb64=uid token=token %}
{% if username %}{% blocktrans %}In case you forgot, your username is {{ username }}.{% endblocktrans %}
{% endif %}{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Thank you for using {{ site_name }}!
{{ site_domain }}{% endblocktrans %}
`
Here is the template, and also I tried to pass context variable like this
`'
extra_email_context': {
            # merge data for all recipients
            'current_site':get_current_site(request),
            'username':user.username
        }`

Here it's giving an error of the request part. Now I'm not getting what to do.
Thanks

Hello @hamhaingaurav thanks for replying.
`
{% load i18n %}{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Hello from {{ site_name }}!
You're receiving this e-mail because you or someone else has requested a password for your user account at {{ site_domain }}.
It can be safely ignored if you did not request a password reset. Click the link below to reset your password.{% endblocktrans %}
{{ protocol }}://{{ domain }}{% url 'password_reset_confirm' uidb64=uid token=token %}
{% if username %}{% blocktrans %}In case you forgot, your username is {{ username }}.{% endblocktrans %}
{% endif %}{% blocktrans with site_name=current_site.name site_domain=current_site.domain %}Thank you for using {{ site_name }}!
{{ site_domain }}{% endblocktrans %}
`
Here is the template, and also I tried to pass context variable like this
`'
extra_email_context': {
            # merge data for all recipients
            'current_site':get_current_site(request),
            'username':user.username
        }`

Here it's giving an error of the request part. Now I'm not getting what to do.
Thanks

UPDATE - It's strange but I solved this issue by simply converting html file to txt. This worked for me.
Thanks

UPDATE - It's strange but I solved this issue by simply converting html file to txt. This worked for me.
Thanks


C.elegans_demo; pseudoembryo0_count.csv access

Hi!
Thank you very much for this ameizing tool!
Im trying to run the example provided in C.elegans_demo.ipynb but I couldn’t find the pseudoembryo0_count.csv file.
I guessed it was reproduced from randomly choosing one represent of a node from 'GSE126954_gene_by_cell_count_matrix.txt'. If so it would be great if you can share the code for this!
Can you maybe assist me with getting pseudoembryo0_count.csv?
Thank you!
Avital

Hi Avital,
Thank you for your interest in our PhyloVelo tool. We have updated the C.elegans demo dataset at https://github.com/kunwang34/PhyloVelo/tree/master/demo/datasets/Celegans.
We hope you enjoy using our tool and find it useful for your research. If you have any questions or feedback, please don’t hesitate to reply.
Best, Kun

Thank you very much!

Thank you very much!


i just can get this work Oauth help please

Hi!.
The other day my oauth was working good on facebook and twitter using visual studio 2015 with ionic and cordova. Then i had a problem and i have to update my android version and CLI and when that happen i cant get it work again and i get a message like this: "Cannot authenticate via a web browser" on both social networks.
scripts:
ng-cordova-oauth - v0.1.4 (2015-09-20)
angular.module('starter2', ['ionic', 'starter2.controllers', 'ngCordovaOauth'])
My plugins:
cordova:
cordova-plugin-inappbrowser 1.1.1-dev "InAppBrowser"
cordova-plugin-whitelist 1.2.0 "Whitelist"
org.apache.cordova.inappbrowser 0.6.0 "InAppBrowser"
bower:
ng-cordova-oauth#0.1.4 extraneous
My Info:
Cordova CLI: 5.3.3
Ionic Version: 1.0.0-rc.2
Ionic CLI Version: 1.7.11
Ionic App Lib Version: 0.6.5
OS:
Node Version: v4.2.1
<EMAIL_ADDRESS>Thanks Alot!!

ooooohh i solved it thanks!!! i missed this line:


ConnectivityError: Unable to retrieve routing table from <neo4j_host>:7688: EOF with neo4j-go-driver 4.4.x & 5.x.x

We met this issue when upgrading neo4j from 4.3 to 4.4 and using driver 4.4.7 with neo4j protocol. We tried to update the driver to version 5.20.0 but the issue doesn't be fixed.
This issue makes system run unstable, it works but sometimes it returns the error Unable to retrieve routing table...
However, the old driver version 4.3.3 works fine (also for 4.3.8).
I assume there is a mistake configurable but I'm not sure. Please give me an idea.
System information:

Neo4j version: Enterprise 4.4.31
Neo4j Mode: HA cluster with 3 members
Operating system: Amazon Linux 2

Hello @fbiville ,
Sorry for my lateness. I sent you the bolt traces log. This log from the server received the request to respond to the result. The log has the prefix bolt_logger, which is log by bolt logger.
bolt_logger.log
While the client gets the routing table, our code returns the error Unable to retrieve the routing table. Then the handshake happens after that. That is a racing condition so that happens with random frequency.
This is the sample code our code is using query cypher.
session := driver.NewSession(neo4j.SessionConfig{AccessMode: neo4j.AccessModeWrite, BoltLogger: boltLogger})
defer session.Close()
result, err := session.Run(query, params)

if err != nil {
	return nil, err
}

I appreciate your support. I'm waiting for your response.

I have applied retry when met this issue around result, err := session.Run(query, params). I think it can work now. However, I feel confused about driver version 4.3.8, which works perfectly for me, with driver version 4.4.0 or above the issue happens. I read the change log of version 4.4.0 but didn't find any point to consider.

You can also session.ExecuteRead or session.ExecuteWrite to leverage the built-in retries (in 4.x, they're called session.ReadTransaction and session.WriteTransaction).
The only case when you cannot use these APIs is when you use CALL {} IN TRANSACTIONS or (in Neo4j 4.x only) LOAD CSV [...] USING PERIODIC COMMIT.

I have the same error, subsequent calls to the driver after this error no longer work, I am forced to restart the driver

I am getting the same error with Go v.1.22.4, driver v.5.20.0 and database v.5.20.0 enterprise
"TransactionExecutionLimit: timeout (exceeded max retry time: 30s) after 6 attempts, last error: ConnectivityError: Unable to retrieve routing table from  :7687: context deadline exceeded"
After the driver returns an error the code closes the connection and opens a new connection for further use.  The code kind of recovers, but the loading of data get delayed, which is very annoying.

@tkandal  did you figure out a solution for fixing the issue yet?

Kind of found a solution.
The config.MaxTransactionRetryTime is default 30 seconds, increased it to 60 seconds and then it seems to work much better.


auth through TiDB

ADD: Auth for spark SQL
MOD: test sandbox
ADD: role-based privilege
ADD: add auth switcher
ADD: get PDAddress from TiDB
ADD: exit when TiAuthorization initialization failed

What problem does this PR solve?
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Code changes

Has exported function/method change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to update the documentation
Need to be included in the release note

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

@birdstorm @sunxiaoguang @zhangyangyu

/cc

/cc birdstorm sunxiaoguang zhangyangyu

/run-all-tests

@sunxiaoguang @birdstorm @zhangyangyu  PTAL

Are database & table names case sensitive?

Are database & table names case sensitive?

Thank you for reminding me, I should change it to case-insensitive same as TiDB.

/run-all-tests

@birdstorm @sunxiaoguang  PTAL

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests

/run-all-tests


Operationalization: Cluster Admin Commands

Provide a way for users to issue the following cluster-wide commands:

rolling restart
upgrade Riak

Done


Camp Harmony confirmation email text (updated and moved from wrong github site)

This is the format for the confirmation email that the PRIMARY registrant receives.  We have not sent emails to "added campers" in the past, so I think it'll be ok.
Dear Judy Depenau,
Thank you for your online registration for Camp Harmony 2023-24! This email confirms the following:
Registration Info
Primary Registrant:
Judy Depenau, Age 31
246 Benton St
Santa, Rosa 95401
707-694-7478,<PHONE_NUMBER>
<EMAIL_ADDRESS>Meals: Vegetarian, No Gluten
Registered for Sun Dec 29th, Mon Dec 30th
Day of arrival: Sun Dec 29 at 2pm
Housing: Economy
Total Fees: $170.00
Campership Donation: $10
No other registrants entered
TOTAL FOR THIS REGISTRATION: $200.00
You have elected to pay by PayPal. Confirmation of your payment will be sent to the registrar as soon as your payment clears.
If you would like to sign up to lead a workshop, you may do so on the [Workshop Signup Thread] https://docs.google.com/forms/d/e/1FAIpQLSfe9LwY1nBqNPan8QE6lcndFJsTJpnTL3fbwK1BKFNsyumEWA/viewform?vc=0&c=0&w=1&flr=0
You have agreed to Camp Harmony's Covid policy [link] and you will need to provide proof of a negative Covid test upon entering Camp.
You will be receiving an email with your housing assignment and additional details by December 4. Thank you for registering for Camp Harmony!

Please edit this with the following in mind:

Please indicate the workshop signup thread, or link to workshop signup instructions
Please note that this form doesn't do renewals, please indicate what you want me to do put instead of that
The system will only send ONE confirmation email out to the primary registrant.  Please indicate how you want to format the email for multiple registrants.

done

The confirmation email should reiterate that each person over 2 needs to bring proof of a negative COVID test, perhaps with the link to the COVID policy,


Add input bam files for nanomonsv.

I am hoping to add new modules to nf-core for nanomonsv
This is an initial commit of test input bams for the nanomonsv parse command.
The data used comes from the original source code repo.
The output generated by the module matches on content the data in the source code repo but the md5 sums are different due to ordering changes (the source code tests just check for entries).

Why'd you put these under delete_me, I don't know what the history of delete_me is anyway 😬


Minor: adb.exe process continues to run after scrcpy is closed

Very minor issue on Win10 (other Windows OSes unconfirmed as I don't have access to them to test) in the grand scheme of things but felt it should be logged anyway.
After closing scrcpy, the adb.exe process it launches continues to run in the background, requiring termination via the Task Manager.
I'm unsure what (if any) information would be helpful in diagnosing this, or even if anything can be done about it (seeing as the exe is an Android one not technically scrcpy) but I can provide any needed details.

This is expected. adb starts a daemon as soon as you use it, which is used for every adb command.
Even without scrcpy, just type adb devices for example, the daemon will stay started.
Killing it would be wrong, since it would break any other parallel usage of adb when you close scrcpy.

If you had to reference this issue many times @rom1v , why not add the command to the main main release so that this is done automatically?

Implemented option --kill-adb-on-close: a3cdf1a6b86ea22786e1f7d09b9c202feabc6949


Add a way to make an ignite plugin generator.

Let's make it easy for people create ignite plugins, by providing them with boilerplate.
ignite plugin new i18n
ignite plugin new MyPlugin

@jamonholmgren wanted this which gives purpose to the new plugin namespace.
ignite plugin list

#665 adds the generator and #728 adds list, search, and info.


In Next Update Be Able To Add Salt And Pepper To Hash

Still in JavaScript Plz =). Your tool is awesome

What do you mean? Do you need HMAC feature?

Sorry, this might sound like a silly question, but what's HMAC?

Never mind, sorry.


[Fix] fix runtime error of "+=" inplace operation in PyTorch 1.10

fix runtime error of "+=" inplace operation in PyTorch 1.10
Closes #7933,
Closes #8221,
Closes #8443,

rpn_head.py has a related issue. https://github.com/open-mmlab/mmdetection/issues/7933#issuecomment-1166368016
Links to issues to be closed: https://github.com/open-mmlab/mmdetection/issues/8221 https://github.com/open-mmlab/mmdetection/issues/8443

rpn_head.py has a related issue. #7933 (comment)
Links to issues to be closed: #8221 #8443

Thanks for your help, have fixed your reported bug


Tree View Stopped Showing Colored Squares for one of my DAGs

Tree View stopped working for one of my DAGs. All I see in the Tree View are empty, uncolored squares. The job is itself running fine for the past 5 days, running every day. The Graph View, the Landing Times, Gantt, and Task Durations all show trends for the past 5 days. Unfortunately, the tree view shows uncolored squares. All of the squares are blank.
I'm running v 1.2 installed via PIP. I should mention that other dags are fine.
Also, my dag is using the new branch operator in 2 places and has about 10 nodes now in it. I don't expect this to be the reason for the Tree View to stop showing data.

Quick thought, did you change your start_date for something that isn't "square" at any point in time? By square I mean 00:00 for a daily job.

default_args = {
    'owner': 'airflow',
    'depends_on_past': False,
    'start_date': datetime(2015, 7, 8),
    'email'<EMAIL_ADDRESS>    'email_on_failure': True,
    'email_on_retry': True,
    'retries': 3,
    'retry_interval': timedelta(seconds=5)
}
dag = DAG('ep_demo', start_date=datetime.now(), default_args=default_args)

Ok, so I'll change the datetime.now() to be rounded to 00:00. I haven't changed this part of the code in a long time, so I am guessing the handling of this changed in the past month or so with respect to the tree view.

@mistercrunch : Is the best way to have have a daily job trigger at a certain time then to use a start_date with 00:00 time, schedule_interval=timedelta(days=1) and make all task in a dag depend on an airflow.operators.TimeSensor? How is the use of sensors affected by the "parallelism" limit of the executor? Can you risk the execution queue being blocked by sensors just waiting for events?

@storpipfugl: yes, and the docs should be more explicit about this
Sensors do take up a slot in your executor while they are running (sensing). You need to make sure you have plenty of slots available to support the max number of tasks that could ever run at the same time.
Sensors are typically cheap in terms of CPU / memory, so you can take that into account when factoring how many slots each worker should have.

Confirmed this resolved my issue. Closing.

Sorry to hijack an already closed issue, but how would you go about doing this for an hourly job?
Unless TimeSensor can sense for dynamic times (x:05 every hour), how would you cause Airflow to sense that 5 minutes have passed?

I think we need a TimeDeltaSensor that senses for a delta after execution_date. It should be very easy to write. Let me give it a shot.

It's in:
https://github.com/airbnb/airflow/pull/237

Thank you. It works like a charm.

Slight changes to the operator:
https://github.com/airbnb/airflow/pull/239


Simplify settings setup for new users

E.g.

It would be great to be able to say: Use the same user settings as User X

or something similar. This could be done e.g. with fractal-web calling two endpoints one after the other: the standard POST, and a new copy-settings-from-another-user PATCH endpoint.

The interface to this looks great!
One question: Will this also include copying over SSH host, SSH username, SSH Private Key Path, SSH Tasks Dir & SSH Jobs Dir in an SSH deployment? Those are the ones where this feature will be most useful for :)

One question: Will this also include copying over SSH host, SSH username, SSH Private Key Path, SSH Tasks Dir & SSH Jobs Dir in an SSH deployment? Those are the ones where this feature will be most useful for :)

Yes!


Update docs-ref-conceptual/install-azure-cli-macos.md: Change python3 to<EMAIL_ADDRESS>

Resolve https://github.com/MicrosoftDocs/azure-docs-cli/issues/2682
Update docs-ref-conceptual/install-azure-cli-macos.md: Change python3 to<EMAIL_ADDRESS>

Docs Build status updates of commit 88b2b0c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/install-azure-cli-macos.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Reapplying log messages clears all content.

If you load a log file, apply highlights / removes and then reapply it it will clear all the content.
I would expect it to reset the styles and reapply the new ones afterwards.

Should be resolved, refactored the vue app since it doesn't accept changing root props


"VM agent isn't currently running" error when configuring VM

When I use lxops to launch and configure a VM, I sometimes get the error "VM agent isn't currently running".
Add an optional wait interval, to apply before configuration, to see whether this might fix the problem.

There are now two wait intervals:

wait-configure: seconds to wait before configuring, defaults to 0.
wait-stop: seconds to wait before snapshot or copy, defaults to 5.

When launching VMs, I set -wait-configure to 20.
This is a workaround.  Ideally there should be a way to check that the VM agent is ready.  Leaving open until I check if this can be done.

There is a way to check that VM agent is running:  Try to do something with it.  Since we exec /bin/sh for scripts, we can run an empty script.  Therefore we can poll for a while, say every second or so, for 20 seconds.

There is a way to poll for the agent.  /bin/sh is one way, /bin/true is another, and InstanceServer.GetInstanceState is yet another.  Check that InstanceState.Processes >= 0.  See https://discuss.linuxcontainers.org/t/is-here-a-way-to-determine-is-lxd-agent-running/16806


auth: remove cached tokens if invalid or corrupted

Shouldn't we preserve the original errors, when clearing the cache succeeds? I think it would be confusing otherwise that an unmarshal error returns file not found.

@n-g So the idea is that if token expired or somehow invalid - we remove the token and return fs.ErrNotExist (because its really not exists anymore). ExchangeToken function would then check the error and do the exchange if no token cached (as it got removed).
I am now changing the structure of the cache file, which would lead to unmarshaling errors if the file contains old structure, so decided that we should not care about corrupted caches and user should not be aware if this.

Shouldn't we preserve the original errors, when clearing the cache succeeds? I think it would be confusing otherwise that an unmarshal error returns file not found.

@n-g So the idea is that if token expired or somehow invalid - we remove the token and return fs.ErrNotExist (because its really not exists anymore). ExchangeToken function would then check the error and do the exchange if no token cached (as it got removed).
I am now changing the structure of the cache file, which would lead to unmarshaling errors if the file contains old structure, so decided that we should not care about corrupted caches and user should not be aware if this.

SG, let's wire ctx context.Context into here, and mention the original error in the debug output at least :)
E.g. https://github.com/namespacelabs/foundation/blob/1bbb17f9116f6b9c697c4eb436401b0f9f052a47/orchestration/prepare.go#L85-L86

SG, let's wire ctx context.Context into here, and mention the original error in the debug output at least :) E.g.
https://github.com/namespacelabs/foundation/blob/1bbb17f9116f6b9c697c4eb436401b0f9f052a47/orchestration/prepare.go#L85-L86

:+1: that is what I also was thinking to do. Thx.


Response::__toString() should not hide exception

https://github.com/Nyholm/psr7/blob/d38874cbc289d25a309cefd04379e24352ced608/src/Stream.php#L98-L109
I found this because of https://github.com/php-http/client-common/issues/186
In my application, I had a 200 with an empty body :exploding_head:
IMHO, doing this is wrong as it hides the root cause.
More over, in PHP 7.4 it's allowed to throw exception.
And for PHP 7.3-, it prefer a fatal error than something wrong.
WDYT?

the catch block should start with this IMHO:
if (\PHP_VERSION_ID >= 70400) {
    throw $e;
}

https://github.com/guzzle/psr7/pull/316

guzzle/psr7#316

Funny that guzzle/psr7 is not PSR-7 compliant: trigger_error() with E_USER_ERROR is a control flow break, which is explicitly forbidden by the PSR. By default, PHP stops execution and a userland error handler would likely throw (Symfony's does).
Anyway, fix proposed in #144 for the issue here.

if (\PHP_VERSION_ID >= 70400) {
    throw $e;
}

Is not PSR-7 compliant either. Not sure why you argue against your own suggestion.

The psr reads as

This method MUST NOT raise an exception in order to conform with PHP's string casting operations.

Which to me means we can interpret that throwing is compliant, when it's technically possible. :)

so you argue a trigger error control flow break is not allowed but throwing an exception is when it explicitly says "MUST NOT raise an exception". Very creative ;)

I think this specific PSR-7 rule is no longer "compatible" with PHP 7.4+

I should have used "fatal error" instead of "control follow break", that'd be more accurate when mentioning E_USER_ERROR, since that's the default behavior, and what psr7 wants to avoid in the first place.
This role is obsoleted by PHP 7.4 for me also.

Me and @Tobion argued about this on Guzzle/psr7. I was against it. The fact that guzzle merged it anyways and @nicolas-grekas shows me a very liberal interpretation of that PSR7 line of definition, I guess I have to say that I was wrong in https://github.com/guzzle/psr7/issues/271
Im happy to merge a fix for this.

The PR I don't get at all and looks wrong to me is https://github.com/guzzle/psr7/pull/243, but https://github.com/guzzle/psr7/pull/271 is fine as it fixes a very WTF situation that was justified by argument that are obsoleted by PHP 7.4.

Note that it would be useful to port #144 to Guzzle if anyone is up to it.


Chris fix CI from PR127

Corrected some CI .yml oversights, also reduced strain on CI runners due to ixia-c traffic tests by reducing # packets, packet rate and increasing timeouts.

Merging myself since it's just fixing some errors on the initial https://github.com/Azure/DASH/pull/127