Slow message loading

Hi,
some folks reported slow message fetching and all tickets are closed but there is no solution mentioned. I tried both v4 and v5. The bottle neck is looping through paginated messages and takes 8 to 10 sec for 20 messages.
Maybe I'm doing wrong but in my code everything is similar more or less to the example code. Even if I leave everything out in the foreach loop there is no improvement.
Maybe someone has any clue?

Hi,
some folks reported slow message fetching and all tickets are closed but there is no solution mentioned. I tried both v4 and v5. The bottle neck is looping through paginated messages and takes 8 to 10 sec for 20 messages. Maybe I'm doing wrong but in my code everything is similar more or less to the example code. Even if I leave everything out in the foreach loop there is no improvement.
Maybe someone has any clue?

The only way I know is to put the PHP code, in the same hosting you aim.
4 seconds took, now ~ 300ms.
(If it is Gmail/Hotmail, perhaps forwarding an email of your domain and there to use this project)

I made some tests with another imap library which depends on build-in php imap methods and builds some classes upon them. The test setup uses both the simplest example code given by the contributors.
Connecting to the same account and same mailbox, fetching 62 Messages, most of them with one or more attachments, looping through messages and display subject and number of attachments. Results:
PHP7.4.23 Webklex 4.1.2 => 29 sec, DDeboer => 0.7 sec
PHP8.1.6 Webklex 5.3 => 91 sec, DDeboer =>0.7 sec
I repeates this tests several times with no significant differences in result.
So it seems that there is no performance enhancement in version 5.
Maybe someone can explain this or has any idea how to improve webklex' performance because this lib is really awesome in al other aspects.

Would you happen to have any updates?
i guess the problem is this line
while (($next_char = fread($this->stream, 1)) !== false && !in_array($next_char, ["","\n"])) {
it loops for 20,000 times calling fread not sure if this is correct!
which lib do you use now?

I made some tests with another imap library which depends on build-in php imap methods and builds some classes upon them. The test setup uses both the simplest example code given by the contributors. Connecting to the same account and same mailbox, fetching 62 Messages, most of them with one or more attachments, looping through messages and display subject and number of attachments. Results:
PHP7.4.23 Webklex 4.1.2 => 29 sec, DDeboer => 0.7 sec PHP8.1.6 Webklex 5.3 => 91 sec, DDeboer =>0.7 sec
I repeated this tests several times with no significant differences in result.
So it seems that there is no performance enhancement in version 5.
Maybe someone can explain this or has any idea how to improve webklex' performance because this lib is really awesome in al other aspects.

Same issue here. Indeed the nextLine method is the issue. It is extremely slow. For now I changed my application to not fetch message bodies in overviews and limit the number with pagination. It would be great if this could be solved. Maybe through a commercial way? (e.g. paid development).


Not clear how to connect to DevOps

Dev Home version
0.601.297.0
Windows build number
10.0.22621.2506
Other software
OS Build Version: 10.0.22621.2506.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.23
Steps to reproduce the bug
In the description on MS Store this app claims it works with Azure DevOps. Yet, when I try to add an account, there is no extension for DevOps available.
Repro:

Goto Introducing Dev Home
Connect accounts
Add account
Find Extensions

Expected result
Expected: Extension available for DevOps
Actual result
Actual: Only an extension for Github
Included System Information
Physical Memory: 31.85GB (15.06GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.601.297.0_x64__8wekyb3d8bbwe

Duplicate of #521


Frontier comutation and cluster

Hi. Thanks so much for sharing the code. Would you mind letting me know where the codes for computing frontiers and clustering them are? Appreciate it very much.

Thanks for your attention.
Here we compute frontiers:
https://github.com/siyandong/NeuralCoMapping/blob/9edd0cd9db7687752813c50fa88f4dcf44eed42a/env/gibson_api/exploration_env.py#L467
And we downsample them in https://github.com/siyandong/NeuralCoMapping/blob/9edd0cd9db7687752813c50fa88f4dcf44eed42a/utils/map_manager.py.

Thanks for your attention.
Here we compute frontiers:
https://github.com/siyandong/NeuralCoMapping/blob/9edd0cd9db7687752813c50fa88f4dcf44eed42a/env/gibson_api/exploration_env.py#L467
And we downsample them in https://github.com/siyandong/NeuralCoMapping/blob/9edd0cd9db7687752813c50fa88f4dcf44eed42a/utils/map_manager.py.

Thanks very much for your quick reply.


Service Bus GA release updates Java

Add Service bus changes to release notes after SB GA.

/ azp run azure-sdk - ci


TDE in standard edition

I am aware that this is prerelease information, but I have to ask: Is it on purpose that TDE (transparent data encryption) is listes as a feature that is available on Standard Edition? If so, this would be a rather significant change from previous versions :-)

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

ID: d663fffb-d444-59c0-714f-433cf91980c4
Version Independent ID: ee6da2f6-9ab0-792c-8a5d-d3c725779669
Content: Editions and supported features of SQL Server 2019 - SQL Server
Content Source: docs/sql-server/editions-and-components-of-sql-server-version-15.md
Product: sql
Technology: install
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

Closing issue, please refer to https://github.com/MicrosoftDocs/sql-docs/issues/3428.


A Route with indexRoute should not have a mandatory component

In our web project with React Router we use the following route config for our route /orders:
// ordersRoute.js

export default {
  path: 'orders',
  onEnter: ListController.onEnter,
  indexRoute: {
    component: ListController,
  },
  childRoutes: [
    {
      path: ':orderId',
      indexRoute: {
        component: SingleController,
      },
      childRoutes: [
        images,
        chat,
      ],
    },
  ],
};

/orders/ renders ListController, /orders/:orderId renders SingleController.
This pattern does not work when using this lib, failing with  'Failed propTypes: Required prop component was not specified in RouteView.' and rendering nothing.
I'd expect this library to match react router's behaviour of just rendering the indexRoute. Otherwise i'm forced to write it as follows:
// ordersRoute.native.js

const Wrapper = (props) => <View style={{flex: 1}}>{props.children}</View>;

export default <Route
  path='orders'
  component={Wrapper}
  onEnter={ListController.onEnter}>
  <IndexRoute component={ListController} />

  <Route path=':orderId' component={Wrapper} />
</Route>;

Where Wrapper just adds boilerplate.
Is this complaint reasonable? Or was our web route config "bad" to begin with? RR IndexRoutes documentation also supplies component plus an indexRoute component, which makes me thing we have it wrong anyways. But why would it work then without supplying a component?

This should work as expected. #19 doesn't fix this? That was a good catch btw. 😅

Well I'll double check tomorrow, but the proptype warning will be there for sure, which it shouldn't right?

I think I just hit essentially the same issue. I was getting this error:
Failed propType: Required prop `component` was not specified in `RouteView`. Check the render method of `_class`.

which I fixed by changing my routes from something like this:
<Route>
  <StackRoute path="/" component={IndexPage}/>
  <StackRoute path="/foo" component={FooPage}/>
</Route>

to something like this:
<Route component={View}>
  <StackRoute path="/" component={IndexPage}/>
  <StackRoute path="/foo" component={FooPage}/>
</Route>

Ugh. Sorry, I think I misunderstood this when I replied ^. You cannot omit component. So the propType warning is correct. Nested components are passed in children prop to their parents. Without a wrapper component, it doesn't make sense to have an IndexRoute. I'd be surprised if this doesn't spit out errors on the web.
🍺


Knowledge is power (got [JUNK], expecting [data] || CURRENT POS: 60)

warning WARNING: WEM => WAV conversion is a little bit broken right now!
Some WEM formats (especially those that aren't packed in BNK files) convert incorrectly.
Unexpected ID when trying to create format chunk (got [JUNK], expecting [data] || CURRENT POS: 60)

Same sh1t
Unexpected ID when trying to create format chunk (got [cue ], expecting [data] || CURRENT POS: 48)
Unexpected ID when trying to create format chunk (got [LIST], expecting [data] || CURRENT POS: 84)
Unexpected ID when trying to create format chunk (got [JUNK], expecting [data] || CURRENT POS: 122)```


goroutine # [select] waitForUpdate error/crash

Have an instance with 200+ repos and it runs through the repo config and then spits out these errors at the end:
goroutine 555 [select]:
github.com/etsy/hound/searcher.(*Searcher).waitForUpdate(0xc217eb5fb0, 0x61ce6bd0c0)
        /home/hound/hound-repo/src/github.com/etsy/hound/searcher/searcher.go:153 +0xf4
github.com/etsy/hound/searcher.func·001()
        /home/hound/hound-repo/src/github.com/etsy/hound/searcher/searcher.go:419 +0x15c
created by github.com/etsy/hound/searcher.newSearcher
        /home/hound/hound-repo/src/github.com/etsy/hound/searcher/searcher.go:437 +0xa65

goroutine 557 [select]:
github.com/etsy/hound/searcher.(*Searcher).waitForUpdate(0xc208118630, 0x73fcd422c0)
        /home/hound/hound-repo/src/github.com/etsy/hound/searcher/searcher.go:153 +0xf4
github.com/etsy/hound/searcher.func·001()
        /home/hound/hound-repo/src/github.com/etsy/hound/searcher/searcher.go:419 +0x15c
created by github.com/etsy/hound/searcher.newSearcher
        /home/hound/hound-repo/src/github.com/etsy/hound/searcher/searcher.go:437 +0xa65

Any ideas on how to fix this?

Is this the entire output of the crash? All this shows is a couple of goroutines waiting on a channel at the time of the crash.

I found the answer. I was using port 80 and we have an iptable rule to reroute 6080 (Default hound port) to 80 so that's the error I was facing.
Marked as closed.


[BUG] turtleshell throws error when trying to log in or when trying to go to the dex

Describe the bug
Turtleshell throws an error, you can work around it by closing the window.
To Reproduce
Steps to reproduce the behavior:

Go to wallet.turtlenetwork.eu or polarity.exchange
Click on add new account, then select turtleshell
it throws an error.

Expected behavior
Error shouldnt be thrown
Screenshots

Additional context
You can close the windows and use the wallet

@ohager fixed it


Bug: Error code 401, Unauthorized, Authorization for the API is required, but the request has not been authenticated

Describe the Bug
Unauthorized, Authorization for the API is required, but the request has not been authenticated, statusCode: 401
Has been getting this error code in the past two days.
Before, it was error code 403 for about a day.
I am living in Hong Kong, not sure if this is a regional issue.
To Reproduce
In Hong Kong, restart homebridge and the same error appears.
Expected Behavior
connect to Switchbot API
Relevant Log Output
[12/20/2024, 2:20:28 PM] Loaded plugin<EMAIL_ADDRESS>[12/20/2024, 2:20:28 PM] Registering platform<EMAIL_ADDRESS>[12/20/2024, 2:20:28 PM] [SwitchBot] Initializing SwitchBot platform...
[12/20/2024, 2:20:29 PM] [SwitchBot] Unauthorized, Authorization for the API is required, but the request has not been authenticated, statusCode: 401
[12/20/2024, 2:20:30 PM] [SwitchBot] Unauthorized, Authorization for the API is required, but the request has not been authenticated, statusCode: 401
[12/20/2024, 2:20:30 PM] [SwitchBot] Unauthorized, Authorization for the API is required, but the request has not been authenticated, statusCode: 401
[12/20/2024, 2:20:30 PM] [SwitchBot] Unauthorized, Authorization for the API is required, but the request has not been authenticated, statusCode: 401
[12/20/2024, 2:20:31 PM] [SwitchBot] Unauthorized, Authorization for the API is required, but the request has not been authenticated, statusCode: 401

Config for homebridge-switchbot
{
            "name": "SwitchBot",
            "credentials": {
                "token": "",
                "secret": "",
                "notice": "Keep your Token & Secret a secret!"
            },
            "platform": "SwitchBot"
        }

Screenshots
No response
Device and Model
hub mini, curtain 3
Node.js Version
v22.12.0
NPM Version
v10.9.0
Homebridge Version
v1.8.5
Homebridge Switchbot Plugin Version
v4.1.6-beta.11
Homebridge Config UI X Plugin Version
v4.65.2
Operating System
Docker

I am also having same "authorization required", statuscode 403 issue with all of my Switchbot devices, thus cannot be controlled anymore since almost a week now. Please support.

It is working fine now. Thank you for the help.


@dev scenario description cleanup

current output when run as: 'meteor --test'
PASSED cucumber : This scenario will not both on dev and on CI:One-liner ...
updated to: 'meteor --test'
PASSED cucumber : This scenario will run on both dev and CI:One-liner ....

Thank you!


Reimplement OperationQueue similar to Objective-C Foundation

Motivation
The current implementation of OperationQueue work in a different way to Objective-C Foundation.
There are some deadlocks and unexpected behavior but the biggest problem is stopping on semaphore during waiting for other operations in a queue.
https://github.com/apple/swift-corelibs-foundation/blob/master/Foundation/Operation.swift#L441
In application with high loads on OperationQueue it leads to spending time on semaphors and blocking all libdispatch.
Example:

Let's create OperationQueue1 with maxConcurrentOperationCount == 2 and put 1000 operations to this queue.
ER: OperationQueue1 will get 2 threads from pool and use them to do 1000 operations one by one. Any other OperationQueue or DispatchQueue works properly
AR: OperationQueue1 will post 1000 DispatchWorkItem and start waiting on sempahors at 998 threads (probably it will be 64 or other limitation from libdspatch) that block any other OperationQueue2 or DispatchQueue

Changes
First Commit: add tests that fail on old implementation and pass on Objective-C Foundation
(I don't know how to run tests from this repo on Objective-C Foundation, that's why I locally created a separated project for that purpose)
Second Commit: rewrite OperationQueue from scratch and try to copy Objective-C Foundation behavior as much as possible.

Hi @phausler can you look into this?
Looks like all mentioned "specs" already implemented.

Any news on this? I’m heavily using Operations on a server and current queue implementation is very broken in Swift 5.1.*…

The Swift project moved the default branch to main and deleted master branch, so GitHub automatically closed the PR. Please re-create pull request with main branch.
More detail about the branch update - https://forums.swift.org/t/updating-branch-names/40412


Fix broken src attribute of Protonmail image

This has been missed in #278.

Could you please also add the missing </tr><tr> so that it's six elements per line?

Thanks!


Script UI - Script List Component

Script List Component should list System, Organization and Project scripts in a intuitive way. Not sure if "sections" are better than a column with a selector or not.
The number of System scripts could eventually be so large that they clutter the List, so that needs to be considered. System scripts are not editable (unless organization = 1), but I think it should be possible to view/review them by everyone.
"Add" could possibly ask whether it is a "System" or "Custom" script that is requested, which would give a natural way to limit the view of too many system scripts, and inclusion will mean that a system script is visible in the List.
Scripts have a "language" (one of "js", "python" or "ruby") and a "name", which is combined with a colon, e.g. "js:IsSummer", and otherwise the name can use same characters as other identifiers.
Upon Add or Edit of custom script, one will be taken to "Script Edit Page".

"Add" could possibly ask whether it is a "System" or "Custom" script that is requested

What "Add"  means here?

what exacltt "component" mean? Should it be a Grafana panel that can be added to some dashboards?


feat: use security-group module instead of resource

what

use security-group module instead of resource

why

more flexible  than current implementation
bring configuration of security group/rules to one standard

references

CPCO-409

/test all

LGTM

replaced by #64


fix(evm): backend shutdown sequence

@mattsse could you please test this?
at least with central tokio runtime inited in main (my setup), it's no problem to rely on BackendHandler::drop.
if it is necessary to distinguish between "all work finished" vs RapidUnscheduledDisassembly, we can set a flag in poll.

ah, failed test :). the question is - was that ack machinery just for this test to pass, or is it really needed to sync cache to disk before last backend is dropped vs some small time after or when forge quits?
1ms works on my machine, 100ms should be probably enough on cl
index 3a14e0d..ff16eaf 100644
--- a/evm/src/executor/fork/backend.rs
+++ b/evm/src/executor/fork/backend.rs
@@ -550,6 +550,7 @@ mod tests {
             let _ = backend.storage(address, idx.into());
         }
         drop(backend);
+        std::thread::sleep(std::time::Duration::from_millis(1));

         let meta = BlockchainDbMeta {
             cfg_env: Default::default(),

btw, i'm considering moving db from BackendHandler to SharedBackend - it would solve this problem without sleep.

this is faster & deterministic + #1003 will probably refactore this code to use explicit runtime anyway
index 3a14e0d..fd1c95c 100644
--- a/evm/src/executor/fork/backend.rs
+++ b/evm/src/executor/fork/backend.rs
@@ -464,6 +464,7 @@ mod tests {
         providers::{Http, Provider},
         types::Address,
     };
+    use tokio::runtime::Runtime;

     use std::{collections::BTreeSet, convert::TryFrom, path::PathBuf, sync::Arc};

@@ -535,21 +536,24 @@ mod tests {
             chain_id: 1,
         };

-        let backend = fork.spawn_backend(&env);
-
         // some rng contract from etherscan
         let address: Address = "63091244180ae240c87d1f528f5f269134cb07b3".parse().unwrap();
-
-        let idx = U256::from(0u64);
-        let _value = backend.storage(address, idx);
-        let _account = backend.basic(address);
-
-        // fill some slots
         let num_slots = 10u64;
-        for idx in 1..num_slots {
-            let _ = backend.storage(address, idx.into());
-        }
-        drop(backend);
+
+        let runtime = Runtime::new().expect("Failed to start runtime");
+        runtime.block_on(async move {
+            let backend = fork.spawn_backend(&env);
+
+            let idx = U256::from(0u64);
+            let _value = backend.storage(address, idx);
+            let _account = backend.basic(address);
+
+            // fill some slots
+            for idx in 1..num_slots {
+                let _ = backend.storage(address, idx.into());
+            }
+        });
+        drop(runtime);

         let meta = BlockchainDbMeta {
             cfg_env: Default::default(),

isn't flushing the cache completely detached now?

last SharedBackend drop is detected here -> future completes -> cache flush is executed from drop.

isn't flushing the cache completely detached now?

last SharedBackend drop is detected here -> future completes -> cache flush is executed from drop.

right, but remember the BackendHandler is spawned and is running in the background so there's no guarantee that the drop will actually finish before the program exits, that's why we had to sync on last SharedBackend::drop in the first place.

at least with central tokio runtime inited in main (my setup), it's no problem to rely on BackendHandler::drop.

there's no guarantee that the drop will actually finish before the program exits

that was the case with poll. in normal conditions, drop should be pretty close to be guaranteed to complete, but true - i'm not sure about that thread + local runtime.block_on.
let's see how difficult it would be to make several functions openly async, so that we can rely on proper runtime shutdown.

not good, not terrible. started with SharedBackend::spawn_backend (problematic thread without join) and EvmOpts::evm_env as it was cheap to do in one go (was suggested here https://github.com/gakonst/foundry/pull/1003#discussion_r837987628).
ended it as soon as possible, but forge::main is mostly async anyway - with async trait we could get rid of RuntimeOrHandle & cli::utils::block_on completely.

this is faster & deterministic + #1003 will probably refactor this code to use explicit runtime anyway

reverted test to original form that mirrors real usage.

this gets rid of the complicated shutdown sequence by essentially making it async.
so instead of creating a new tokio runtime in a std::thread the backend handler is now spawned as tokio::task and the handler now flushes the cache on drop.
and because it's now async we can control how long it lives via the tokio runtime which is now lifted out and must be created at a more top level location.
 Was the previous impl unsound?

caching worked, but BackendHandler was not (always) able to drop & while trying to fix it in #1199 we figured it would be better to go deeper.
the problem is not critical for foundry, but someone who simulates on more than a few blocks would run out of memory.


Publish your extension to Microsoft Edge Addons website

Microsoft Edge is the second most used browser on the web, making the browsing experience more flexible, offering many more services for users and optimized for Windows devices.
Microsoft Edge supports Chromium extensions, and you can publish your extensions to Microsoft Edge Add-ons website with minimal code changes. Find out more here.
If you have any more queries about porting Chrome extension to Microsoft Edge or anything else, you can contact the relevant teams whose links are given here.
Let me know if this was helpful. I am happy to help you with more details.

This is done. I added URL here:
https://github.com/alex-vv/page-load-time/pull/40/files


CONTRIBUTING

Hi, if you are reading this is because you pretty likely want to participate in the development of this API and curious about how to start.
You are in cpp-sc2 core project, but actually, we have more repositories you can contribute to:

codegen - type IDs codegeneration utility;
commandcenter - original version of CommandCenter bot with integrated cpp-sc2;
docker-sc2 - dockerized StarCraft II Linux client;
scrubber - scrubs type ids from a game client.
blank-bot - starter bot for StarCraft II with integrated cpp-sc2.

What can I do?

[ ] Search for other issues tagged with good first issue or help wanted tag.
[ ] Suggest new features.
[ ] Vote for existing issues (issues with highest number of reactions are likely to be fixed first).
[ ] Fix // FIXME:  comments in the code.
[ ] Improve the documentation of the API.
[ ] Add/Fix/Improve tutorials.
[ ] Add/Fix/Improve new bot examples.
[ ] Add/Fix/Improve tests.
[ ] Help to test latest changes.
[ ] Support this project on Boosty (preferred) or Patreon. Support of the project allows to make releases more often and implement nice new features for developers.

If you have any suggestions, feel free to comment on this thread, or chat me over Discord (the #cpp channel).
What should I know before the start?
Code guidelines
We do our best to conform to the Google C++ Style Guide with the exception that we use four space tabs instead of two space tabs.
Commit guidelines
We follow the semantic versioning approach, this is important so your PR will appear on the Changelog 🎉 ! I guess you expect recognition and we will give it to you.
See standard-version for commit guidelines.
The list of supported commit message prefixes:

feat - a new feature or API improvements;
fix - a bugfix;
perf - performance improvements;
refactor - code refactoring;
test - functional or unit tests;
build - build system improvements,  also includes CI improvements;
docs - documentation improvements;
chore - routine task, e.g. create new release (doesn't appear in the changelog);
style - fix code style, no functional changes (doesn't appear in the changelog.

Before PR

Keep the PR small as possible and one single topic for each.
Write good commit message: describe what was the problem and why the changes required.
Consider implementing a test.

Based on the great guideline in https://github.com/verdaccio/verdaccio/issues/1461.
Hope, someday this issue will be as cool as this one :)

Closed in favor of global README.


Bug(utils): call calculatePermissions will result in SyntaxError

Error:
Calling calculatePermissions(34393292864n) will cause SyntaxError: Cannot convert CREATE_INSTANT_INVITE to a BigInt
Expect:
return ['ADD_REACTIONS', 'CREATE_PUBLIC_THREADS', 'USE_VAD'] without following the order
Test:
expect(calculatePermissions(34393292864n)).to.have.members(['ADD_REACTIONS', 'CREATE_PUBLIC_THREADS', 'USE_VAD'])

I have identified the cause of this
This is the typescript source code

This is what it compiles to

The typescript compiler is dumb
It compile if(Number(permission)) return false to typeof permission !== "number"
if(Number("1")) return 1, therefore it will then run return false, but typeof permission !== "number" return true, therefore it bypass the check

@lts20050703 changing the object key to object value solved it


Problem loading projects & maximum number of 10 projects reached

Describe the bug
I'm unable to do anything on playground v2. I get the following error displayed at the bottom of the screen

And loading my projects fails, leaving me unable to delete any projects.

It's noteworthy that I have only been using playground v1 thus far.
To Reproduce
Steps to reproduce the behavior:
For me, I only have to go to playground for the error to occur. I'm assuming this is due to previous projects cached in my browser, as I don't have the problem in incognito or other browser windows.
Expected behavior
Ability to write Cadence, deploy contracts, execute scripts & send transactions.
Screenshots
See above
Desktop (please complete the following information):

OS: MacOS
Browser: Chrome
Version: Browser - 111.0.5563.64 (Official Build) (arm64) | OS - Ventura 13.2.1

+1

@sisyphusSmiling
Can you try again, we had backend database deployment issues that didn't allow migration of previous user sessions.

@bthaile still having the issue

Sorry for your frustrations, I will be looking into this issue asap! Closing this issue since it's a duplicate of https://github.com/onflow/flow-playground/issues/626


[FEATURE] Scale down Longhorn nodes with Kubernetes

Is your feature request related to a problem? Please describe. Deleting a node from Longhorn outside of the UI involves a number of unique steps
Describe the solution you'd like Simplified process for deleting node during a rolling update of nodes in the cluster, when using kubectl to self-drain the node and delete itself from the cluster, having the ability to delete the node via it's CRD object would provide a universal approach
Describe alternatives you've considered API calls to the Longhorn API

I am thinking we can add a setting to allow automatically delete the node if it's deleted from Kubernetes.

Need to consider the design with #1269

Move out to v1.1.0 due to we need to redesign the feature due to the user may want to retain the disk on the new nodes, e.g. extra EBS volume. See #1269

Retain metadata of disk or not can be implemented as a flag for each disk, with a global default setting.

In order to implement this feature, we will need to promote disk to the first-class object.

The automatic removal of the node feature has been implemented in Longhorn v1.1.0.


Plugin causes homebridge to crash while startup

Describe the bug
In case the internet connection is interrupted, the Homebridge cannot be restarted because the plugin cannot connect to the Tuya API.
To Reproduce

Disconnect the internet connection
Restart homebridge
Watch the homebridge log

Expected behavior
I expected the plugin not to crash
Log
[30/05/2021, 08:45:43] [StarProjector] Sterne: An error occured during connecting to Tuya API
[30/05/2021, 08:45:43] [StarProjector] Error: find() timed out. Is the device powered on and the ID or IP correct?
    at /usr/local/lib/node_modules/homebridge-star-projector/node_modules/tuyapi/index.js:824:13
    at Timeout._onTimeout (/usr/local/lib/node_modules/homebridge-star-projector/node_modules/p-timeout/index.js:25:13)
    at listOnTimeout (internal/timers.js:555:17)
    at processTimers (internal/timers.js:498:7)
[30/05/2021, 08:45:43] [StarProjector] Sterne: Error from socket: connect ECONNREFUSED <IP_ADDRESS>:6668
[30/05/2021, 08:45:53] [StarProjector] Sterne: Error from socket: connect ECONNREFUSED <IP_ADDRESS>:6668
[30/05/2021, 08:45:57] [StarProjector] Sterne: Error from socket: connect ECONNREFUSED <IP_ADDRESS>:6668
[30/05/2021, 08:46:05] [StarProjector] Sterne: Error from socket: connect ECONNREFUSED <IP_ADDRESS>:6668
[30/05/2021, 08:46:21] [StarProjector] Sterne: Error from socket: connect ECONNREFUSED <IP_ADDRESS>:6668
Error: connect ECONNREFUSED <IP_ADDRESS>:6668
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1146:16) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 6668,
  attemptNumber: 6,
  retriesLeft: 0
}
Error: connect ECONNREFUSED <IP_ADDRESS>:6668
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1146:16) {
  errno: -111,
  code: 'ECONNREFUSED',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 6668,
  attemptNumber: 6,
  retriesLeft: 0
}
[30/05/2021, 08:46:21] Got SIGTERM, shutting down Homebridge...

Impact:
Parts of the home automation are no longer available, automations no longer work, and I can't even turn off the light in the dining room, which can be quite annoying ;-). As a result, the Homebridge becomes dependent on an existing internet connection, which contradicts the HomeKit idea.

Thanks @sschuste for reporting this. I have now added a first fix and reworked the whole connection/reconnection logic. Could you install the latest beta version and give a short feedback if the bug is fixed or not?
(sudo) npm i -g homebridge-star-projector@beta

Installed the beta, killed my internet connection, restarted Homebridge and the result is: thumbs up. It now works as expected. Thank you.

Thanks for the feedback. The changes are live now in v1.0.5


Email Referrer when an Assessor adds a Note

JIRA https://dsdmoj.atlassian.net/jira/software/c/projects/CAS2/boards/1347?selectedIssue=CAS2-8
As a referrer
I want to know when a note has been added to a referral I have made
So that I can reply in a timely manner to progress the referral
This uses the EmailNotificationService to send an email to the referrer when a Note is added by an ExternalUser - I don't think we need to specify the role of the External User in this case, because the only other users able to leave a note are Assessors currently, and presumably the referrer should be notified whenever someone who is not a POM leaves a note.
I've also added some CAS2-specific date formatting to the date service, as we display dates differently than the other CAS projects.
unhappy path
For the edge case that a Nomis user does not have an email,
we want to alert the team that the email has failed for this note, but
not let an error bubble up to the user.
We found that all of our existing 200+ users have emails associated
with their accounts, so we are hoping that we are unlikely to encounter
this.

This PR does not yet make use of the just-merged replyToEmailID for us to add a CAS2-specific email. As this feature is still behind a feature flag,I think it's safe to merge, and I will pick up the replyToEmail in the next PR.


[SPARK-7551][DataFrame] support backticks for DataFrame attribute resolution

cc @rxin
It should be better if we could reuse parser and analyzer to do this job or reopen https://github.com/apache/spark/pull/5505 after we can handle joins correctly.

Merged build triggered.

Merged build started.

Test build #32471 has started for   PR 6074 at commit 54c4209.

Merged build triggered.

Merged build started.

Test build #32472 has started for   PR 6074 at commit 399fc8d.

Merged build triggered.

Merged build started.

Test build #32474 has started for   PR 6074 at commit 209263d.

Merged build triggered.

Merged build started.

Test build #32477 has started for   PR 6074 at commit e218d99.

Test build #32471 has finished for   PR 6074 at commit 54c4209.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32471/
Test PASSed.

Test build #32472 has finished for   PR 6074 at commit 399fc8d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32472/
Test PASSed.

Test build #32474 has finished for   PR 6074 at commit 209263d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32474/
Test PASSed.

Test build #32477 has finished for   PR 6074 at commit e218d99.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32477/
Test PASSed.

LGTM otherwise.

Merged build triggered.

Merged build started.

Test build #32585 has started for   PR 6074 at commit 903b319.

Merged build triggered.

Merged build started.

Test build #32586 has started for   PR 6074 at commit 2b86699.

Merged build triggered.

Merged build started.

Test build #32589 has started for   PR 6074 at commit e6f579e.

LGTM.

Test build #32585 has finished for   PR 6074 at commit 903b319.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32585/
Test FAILed.

Test build #32586 has finished for   PR 6074 at commit 2b86699.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32586/
Test PASSed.

Test build #32589 has finished for   PR 6074 at commit e6f579e.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32589/
Test FAILed.

Jenkins, retest this please.

Merged build triggered.

Merged build started.

Test build #32597 has started for   PR 6074 at commit e6f579e.

Test build #32597 has finished for   PR 6074 at commit e6f579e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32597/
Test PASSed.

Thanks. Merging this in.


Release v1.4.0

470d1ff Updates swagger spec

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#12  👈
#11
main

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

Join @pbotros and the rest of your teammates on  Graphite


build_library: Add Vagrant box files

No longer have the OEM install them in the OEM partition only for vm_image_util.sh to move them back out. For now this only applies to the vagrant_virtualbox image, but the goal is to migrate all the Vagrant
images and drop the FS hook.
Box files copied from coreos/coreos-overlay@bd0cd7685ebf71a133c1b70f5ebd5a0b3393ac70 coreos-base/oem-vagrant-virtualbox/files/box.
cc @AlexNPavel

LGTM. I guess the only other change required would be to make this to use the oem-virtualbox package right? At the moment, we still need the vagrant-virtualbox oem-id, but we're in the process of migrating all usage of that to the virtualbox id instead, so we can do that once all those changes are ready (the oem-id and oem-release file are the only differences between the 2 oem packages from what I can tell).


Salvage Info tool too spammy

Remove the failure message for Salvage Info tool

Add some sort of throttling to the requests rather than hiding the message, then figure out why they're consistently failing for some items

#1084


fix error with imagemagick 7 in arch

if it works on at least one machine it is bound to work on more i guess.


Feature request: update search results for opened file immediately

Use case:
Perform search for string abc. String is found 4 times in 2 files (2 entries per file). Open first file. Delete/alter both strings abc.
Do not save file.
When navigating with shortcuts I press go to next entry after deleting both abcs and this is a no-op because there is no such second entry already and next/previous navigation is broken.
Possible solutions:

checkbox "autosave when navigating with narrow if buffer was changed"
exclude current file being edited from search results, save current buffer to tmp file and run search on it and replace search results

What do you think?

checkbox "autosave when navigating with narrow if buffer was changed"

I don't want to do this, but this can be done by user's init.coffee script

auto save file on-did-change timing when narrow-search editor was opened in workspace.

But I want you just hit cmd-s. in the first place.

exclude current file being edited from search results, save current buffer to tmp file and run search on it and replace search results

Yes, understandable. But again I want to avoid this kind of dirty if possible.
If I would go to this direction.

render initial result with ag or rg, since it's fast.
on file change, invalidate entry in modified file and update entry by collecting item by Atom's editor.scan.
This allow scan editor's text which is not yet write out to disk.
But how to do when search option changed?
We have to re-scan whole entry from disk, but when there is modified state file, need to exclude that items and replace it with the result of editor.scan
So I have to manage modified files, and respect it when search option(whole-word, ignore-case) changed.
This definitely introduce complexity to code base, does it worth to do? How many situation user hit this?

Just hit cmd-s to save is first solution.

After thinking , I like *current explicitly save and refresh behavior.
I use narrow-editor item list as kind of TODO list which item is removed when I save buffer.
Let's say removing unused functions scattered across files.
I want removed functions item on narrow-editor is removed on explicit save.
I know there is different opinion, usecase, preference, I'm explaining why I'm not motivated to implement this feature.
Of cource another reason is it need extra dirty work to support this feature.
If it's easy and amount of code I have to add is small, my motivation would be different.

Wow, I will evaluate  this next! I'm not motivated to fix this 👍 👍 👍


Invalid Authentiication

Hello,
This looks very promising, and I can see this working on youtube how-to's ;o)
However, regardless if I try to access Tuya or Ledvance I always get "invalid authentication". I know that my Tuya trial account has expired. Might that be the reason for both failing? Any ideas for how to investigate, what to check or what to do? - Any additional settings to do in iot.tuya or Ledvance?
Thank you!

It only works with your Ledance account.
Maybe this is a similar issue to this and you have to create a new account:
https://github.com/FlagX/ha-ledvance-tuya-resync-localkey/issues/24

No luck after having created two new accounts.
Does it matter that the Ledvance "Time Zone" is set to "Europe/Copenhagen"? - At least for Tuya the region in the app and in the developer portal needs to be the same, not sure if it has an impact here...

You can try to adjust the country code in the const.py file.
https://github.com/FlagX/ha-ledvance-tuya-resync-localkey/blob/main/pyscript_modules/tuya/const.py


zydis: 3.2.1 -> 4.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/zyantific/zydis/releases.
meta.description for zydis is: "Fast and lightweight x86/x86-64 disassembler library"
meta.homepage for zydis is: "https://zydis.re/"
meta.changelog for zydis is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.0.0 with grep in /nix/store/v6ibsyf3bzwvczxxlsgrsq7zgkwihl53-zydis-4.0.0
found 4.0.0 in filename of file in /nix/store/v6ibsyf3bzwvczxxlsgrsq7zgkwihl53-zydis-4.0.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
zydis

Instructions to test this update (click to expand)

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

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v6ibsyf3bzwvczxxlsgrsq7zgkwihl53-zydis-4.0.0
ls -la /nix/store/v6ibsyf3bzwvczxxlsgrsq7zgkwihl53-zydis-4.0.0/bin

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

1 package built:

zydis

Maintainer pings
cc @jbcrail @AndersonTorres for testing.

TODO:

[ ] add changelog link


Implementation of Simulated-Annealing

Added implementation of the search SIMULATED-ANNEALING with J-Unit Tests.

Looks very good. Will need to play with before bringing into the AIMA4e branch. Minor request, could you add a little documentation to the Scheduler class you implemented.
Thanks

Sure. I've added some comments as part of the documentation.


README.md file incorrect

Version
v2.11.0
Reproduction link
https://github.com/ngblaylock/nuxt-sandbox
Steps to reproduce
This is my first time submitting any issues, so please excuse me for any inconsistencies in your workflow.
When I run npx create-nuxt-app <project name> it automatically creates the README.md file. On the first set of instructions, it says to run $ npm run install. From my research, and after much trial and error, I found that it should be npm install (remove run).

What is expected ?
It should install dependencies when cloning a repo from GitHub.
What is actually happening?
It gives me an error saying that it is not a valid command.
Additional comments?
If I am missing anything, please let me know. I am still a newbie, especially to GitHub collaboration.
Thanks for making Nuxt. It is awesome!

This bug report is available on Nuxt community (#c10228)

fixed via https://github.com/nuxt/create-nuxt-app/pull/401


Function pointer overloads

We should provide function pointer overloads where we currently have callback delegates.
Because in net5.0s UnmanagedCallersOnlyAttribute marked methods can be directly called from native code, and this probably has performance implications.

:+1: I have this planned for next time I'm doing binder work


Repair CI (Temporary solution)

CI is currently failing due to https://github.com/pypa/setuptools/pull/3488 implementing PEP660 editable installs in setuptools (starting with version 64)
As a temporary solution the setuptools version is forced to be less than 64

This PR still contains unrelated merge commits that are unnecessary and superfluous.
A proper PR for this fix should consist of a single commit that pins the setuptools version.


1198 index temp locations

closes #1198
DONT merge before we run a reindex on staging to test this.
The holdings_1display that is generated when indexing the custom fixture: https://github.com/pulibrary/bibdata/blob/1198-index-temp-locations/spec/fixtures/marc_to_solr/99124695833506421_custom_holdings.mrx is:
{"22745884920006421"=>
{"location_code"=>"lewis$stacks", "location"=>"Stacks", "library"=>"Lewis Library", "call_number"=>"QB981 . D63 2021", "call_number_browse"=>"QB981 . D63 2021",
"items"=>[
{"holding_id"=>"22745884920006421", "id"=>"23745884910006421", "status_at_load"=>"1",
"barcode"=>"32101104014996", "copy_number"=>"0"},
{"holding_id"=>"22745884920006421", "id"=>"23799884910006421", "status_at_load"=>"1",
"barcode"=>"32101104014996", "copy_number"=>"8"}]},
"lewis$res"=>{"location_code"=>"lewis$res", "current_location"=>"Course Reserve", "current_library"=>"Lewis Library", "call_number"=>"QB981 . D63 2021", "call_number_browse"=>"QB981 . D63 2021",
"items"=>[
{"holding_id"=>"22745884920006421", "id"=>"23898873500006421", "status_at_load"=>"1",
"barcode"=>"32101109060713", "copy_number"=>"2"},
{"holding_id"=>"22845884920006421", "id"=>"23888873500006421", "status_at_load"=>"1",
"barcode"=>"32101109060718", "copy_number"=>"8"},
{"holding_id"=>"22845884920006421", "id"=>"23998873500006421", "status_at_load"=>"1",
"barcode"=>"32101109060718", "copy_number"=>"2"}]},
"22845884920006421"=>{"location_code"=>"firestone$stacks", "location"=>"Stacks", "library"=>"Firestone Library", "call_number"=>"QD981 . D63 2020", "call_number_browse"=>"QD981 . D63 2020",
"items"=>[
{"holding_id"=>"22845884920006421", "id"=>"23645884910006421", "status_at_load"=>"1", "barcode"=>"32101104014998", "copy_number"=>"0"}]}}

This is the custom fixture with holdings_1display https://github.com/pulibrary/bibdata/pull/1734#issue-1052433575 :

This pr will not resolve https://github.com/pulibrary/orangelight/issues/2751 that we closed in the past. This PR will index temporary locations but the availability call should also be updated to return status for the temporary location. I added some notes in the agenda for the orangelight meeting tomorrow 11/23/2021 https://docs.google.com/document/d/1GxpYrcRc6w3kMFU1XU6CEpnumFzvxFrj2GkCq_XvoWw/edit#.

And this is the fixture exporting from Alma with no custom additions.
In staging indexed with no temp location

s
In staging indexed using the pr for the temp locations.

I indexed 3000 temporary locations. I exported the records from the alma sandbox. We reviewed some records with @mzelesky and so far it looks good. some examples: https://catalog-staging.princeton.edu/catalog/99106480053506421 (852 with no 876s)
https://catalog-staging.princeton.edu/catalog/99122643653506421/ (6 holdings but only 3 holdings have items. check the alma sandbox to confirm. one is a temporary res location ) 3 holdings will show up in the record page.

https://catalog-staging.princeton.edu/catalog/99122643653506421/
"holdings_1display":
"{"lewis$res":
{"location_code":"lewis$res","current_location":"Course Reserve","current_library":"Lewis Library","call_number":"HQ767.9 .S534 2020","call_number_browse":"HQ767.9 .S534 2020",
"items":[{"holding_id":"22543249720006421","id":"23543249710006421","status_at_load":"0","barcode":"32101112870454","copy_number":"1"},{"holding_id":"22543249640006421","id":"23543249630006421","status_at_load":"0","barcode":"32101092796398","copy_number":"4"},{"holding_id":"22543249660006421","id":"23543249650006421","status_at_load":"0","barcode":"32101092796380","copy_number":"3"},{"holding_id":"22543249700006421","id":"23543249690006421","status_at_load":"0","barcode":"32101102527726","copy_number":"2"}]},
"22543249620006421":
{"location_code":"lewis$stacks","location":"Stacks","library":"Lewis Library","call_number":"HQ767.9 .S534 2020","call_number_browse":"HQ767.9 .S534 2020",
"items":
[{"holding_id":"22543249620006421","id":"23543249610006421","status_at_load":"1","barcode":"32101092796752","copy_number":"6"}]},
"22543249680006421":
{"location_code":"lewis$stacks","location":"Stacks","library":"Lewis Library","call_number":"HQ767.9 .S534 2020","call_number_browse":"HQ767.9 .S534 2020",
"items":[{"holding_id":"22543249680006421","id":"23543249670006421","status_at_load":"0","barcode":"32101112863095","copy_number":"5"}]}}"

we update the availability response to include the temporary holding id as the temporary_library$temporary_location following the temp locations indexing pattern. The response will look :
example record http://localhost:3000/catalog/99122643653506421
{
"99122643653506421": {
"22543249620006421": {
"on_reserve": "N",
"location": "lewis$stacks",
"label": "Lewis Library - Stacks",
"status_label": "Available",
"copy_number": null,
"cdl": false,
"temp_location": false,
"id": "22543249620006421"
},
"lewis$res": {
"on_reserve": "Y",
"location": "lewis$res",
"label": "Lewis Library - Course Reserve",
"status_label": "Unavailable",
"copy_number": null,
"cdl": false,
"temp_location": true,
"id": "lewis$res"
},
"22543249680006421": {
"on_reserve": "N",
"location": "lewis$stacks",
"label": "Lewis Library - Stacks",
"status_label": "Unavailable",
"copy_number": null,
"cdl": false,
"temp_location": false,
"id": "22543249680006421"
}
}
}

Example record from the alma sandbox:<PHONE_NUMBER>506421 . it has one item not available in lewis$resterm location.

After indexing in staging: https://catalog-staging.princeton.edu/catalog/9960861053506421

The 'Request ' button displays because in bibdata-staging the fulfillment_unit for lewis$resterm is 'General' and not 'Reserves'.
"holdings_1display": "{\"lewis$resterm\":{\"location_code\":\"lewis$resterm\",\"current_location\":\"Term Loan Reserves\",\"current_library\":\"Lewis Library\",\"call_number\":\"QC173.454 .A48 2010\",\"call_number_browse\":\"QC173.454 .A48 2010\",\"items\":[{\"holding_id\":\"22753114530006421\",\"id\":\"23753114520006421\",\"status_at_load\":\"0\",\"barcode\":\"32101078273891\",\"copy_number\":\"1\"}]}}",
https://bibdata-staging.princeton.edu/bibliographic/availability.json?deep=true&bib_ids=9960861053506421
{
"9960861053506421": {
"lewis$resterm": {
"on_reserve": "N",
"location": "lewis$resterm",
"label": "Lewis Library - Term Loan Reserves",
"status_label": "Unavailable",
"copy_number": null,
"cdl": false,
"temp_location": true,
"id": "lewis$resterm"
}
}
}
---------- VS ----------
In production https://catalog.princeton.edu/catalog/9960861053506421 we observe the slow load of the library. In this case the item is in lewis$resterm and available. For confirmation see screenshot from Alma prod.
https://user-images.githubusercontent.com/9905193/152229132-d91cf422-7e9d-4859-b8f0-dc79391f924b.mov

https://bibdata.princeton.edu/bibliographic/availability.json?deep=true&bib_ids=9960861053506421
{
"9960861053506421": {
"fake_id_1": {
"on_reserve": "N",
"location": "lewis$resterm",
"label": "Lewis Library - Term Loan Reserves",
"status_label": "Available",
"copy_number": null,
"cdl": false,
"temp_location": true,
"id": "fake_id_1"
}
}
}

I need to add the second call with deep true to get the availability of all the items. https://bibdata-staging.princeton.edu/bibliographic/99122643653506421/availability.json?deep=true

[ ] also check and update availability.js in orangelight accordingly.

discussion in slack https://pulibrary.slack.com/archives/C02P38XGCMA/p1643908874999159
https://pulibrary.slack.com/archives/C02P38XGCMA/p1643909100404049

additional work in orangelight see https://github.com/pulibrary/orangelight/tree/temp-locations-bibdata-work-1198

99125441517806421 and<PHONE_NUMBER>506421 will be updated. The location had an assignment error both in alma production and alma sandbox. (the alma sandbox was recently copied from alma production). Dont use these two records for QA until they get updated.

Per @mzelesky if temp location (current) location for an item is RES_SHARE$IN_RS_REQ, then ignore the temp location and index the item in the permanent location.


refactor: Use WaitAsync logic for >net6.0

Use the WaitAsync logic for net6.0 and above otherwise, we use a "better" handling, which closes the Task.Delay via a CancellationToken.

Ten attempts and still the windows runner has issues ... also seemingly never with a test itself.

Yeah, it's annoying. Wonder if it's the blame stuff we have in there?
Should we try to remove that?

Yeah, it's annoying. Wonder if it's the blame stuff we have in there?
Should we try to remove that?

Yeah let's try that. We can remove the commit if it is unrelated.

Lol I removed the blame and on 2nd try it worked - I am not sure if this is really related to be honest :D


How to get credit card number before (or) after payment?

Hi @amilamen , I just use this npm. I have a doubt that how to get the card number before or after the payment. I get the exp_Month, exp_Year, and only the last four digit of the card number not full number from stripe charges api. Is there any option to get full card number?
I will wait for your positive reply.
Thanks in advance

Hello @balaji-subramaniyam,
Sorry for late. The card number is entered by the user and it is for security that Stripe only gives you the last digits.
What do you want to do with the full card number?

Thanks for your response. Actually I need the card details to be saved after the successful payment except cvv number.
Otherwise every time the user need to enter the card number to make payment. Is there any possibility that the user can use the existing card for payment?

Hi @balaji-subramaniyam
For this purpose you need to use the stripe api for retrive the card belong a customer. Or save the payment method token in your database belong a customer.
For create payment method: https://stripe.com/docs/api/payment_methods/create
For retrive payment method : https://stripe.com/docs/api/payment_methods/retrieve

Thanks for your response. Actually I need the card details to be saved after the successful payment except cvv number. Otherwise every time the user need to enter the card number to make payment. Is there any possibility that the user can use the existing card for payment?

Hi @balaji-subramaniyam,
I better understand your problem. For actual version of package, you can't retrieve card informations.
I'll make an update of package to allow developer to get these informations.
Stay connected for next version and thank you for your relevant issue.

Thank you @ChafikHadjAbdouRazack and @amilamen  for your reply and suggestions.

@balaji-subramaniyam package has been updated !


Fix nano editor cursor default

When loading the web terminal, by default the cursor blinks in upper left of the screen, not where future text will appear. The positioning of this should be adjusted by default -- I've attached a screenshot to demonstrate where it currently is upon an initial load.

@davelester could you check this in the demo, please?
http://twitter.github.io/cli-guide.js/demo/

It appears this is a chrome-specific bug.

@davelester  please check the demo again please

:shipit: Awesome, it now works in Chrome on a mac

I closed this issue :) thanks


Panic in completion/HirDisplay

Panic context:
> request: textDocument/codeAction CodeActionParams {
    text_document: TextDocumentIdentifier {
        uri: Url {
            scheme: "file",
            username: "",
            password: None,
            host: None,
            port: None,
            path: "/home/matklad/projects/rust-analyzer/crates/rust-analyzer/src/reload.rs",
            query: None,
            fragment: None,
        },
    },
    range: Range {
        start: Position {
            line: 384,
            character: 20,
        },
        end: Position {
            line: 384,
            character: 20,
        },
    },
    context: CodeActionContext {
        diagnostics: [],
        only: None,
    },
    work_done_progress_params: WorkDoneProgressParams {
        work_done_token: None,
    },
    partial_result_params: PartialResultParams {
        partial_result_token: None,
    },
}

thread '<unnamed>' panicked at 'HirDisplay failed when calling Display::fmt!', crates/hir_ty/src/display.rs:233:17
stack backtrace:
   0: std::panicking::begin_panic
   1: <hir_ty::display::HirDisplayWrapper<T> as core::fmt::Display>::fmt
   2: core::fmt::write
             at /rustc/2fd73fabe469357a12c2c974c140f67e7cdd76d0/library/core/src/fmt/mod.rs:1096:17
   3: <hir_ty::types::Ty as hir_ty::display::HirDisplay>::hir_fmt
   4: hir_ty::display::HirDisplay::display_source_code
   5: ide_assists::handlers::add_explicit_type::add_explicit_type
   6: ide_assists::Assist::get
   7: std::panicking::try
   8: ide::Analysis::assists
   9: rust_analyzer::handlers::handle_code_action
  10: <F as threadpool::FnBox>::call_box
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
Panic context:
> request: textDocument/completion CompletionParams {
    text_document_position: TextDocumentPositionParams {
        text_document: TextDocumentIdentifier {
            uri: Url {
                scheme: "file",
                username: "",
                password: None,
                host: None,
                port: None,
                path: "/home/matklad/projects/rust-analyzer/crates/rust-analyzer/src/reload.rs",
                query: None,
                fragment: None,
            },
        },
        position: Position {
            line: 256,
            character: 60,
        },
    },
    work_done_progress_params: WorkDoneProgressParams {
        work_done_token: None,
    },
    partial_result_params: PartialResultParams {
        partial_result_token: None,
    },
    context: Some(
        CompletionContext {
            trigger_kind: TriggerForIncompleteCompletions,
            trigger_character: None,
        },
    ),
}

thread '<unnamed>' panicked at '

Failed to lookup<EMAIL_ADDRESS>in this Semantics.
Make sure to use only query nodes, derived from this instance of Semantics.
root node<EMAIL_ADDRESS>known nodes<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
', crates/hir/src/semantics.rs:634:13
stack backtrace:
   0: rust_begin_unwind
             at /rustc/2fd73fabe469357a12c2c974c140f67e7cdd76d0/library/std/src/panicking.rs:493:5
   1: std::panicking::begin_panic_fmt
             at /rustc/2fd73fabe469357a12c2c974c140f67e7cdd76d0/library/std/src/panicking.rs:435:5
   2: hir::semantics::SemanticsImpl::find_file::{{closure}}
   3: hir::semantics::SemanticsImpl::find_file
   4: hir::semantics::SemanticsImpl::analyze_impl
   5: hir::semantics::SemanticsImpl::scope
   6: ide_db::helpers::import_assets::ImportAssets::search_for
   7: ide_db::helpers::import_assets::ImportAssets::search_for_imports
   8: ide_completion::completions::flyimport::import_on_the_fly
   9: ide_completion::completions
  10: std::panicking::try
  11: ide::Analysis::completions
  12: rust_analyzer::handlers::handle_completion
  13: <F as threadpool::FnBox>::call_box
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Been seeing the display panic on hovering as well, #8077


TypeError: TwitchRequest is not a constructor

Hi, I've followed the documentation as far as possible, but I keep getting the same error.
Maybe you can help me further.
const twitchClient = new TwitchRequest(TwitchClientOptions);
                     ^

TypeError: TwitchRequest is not a constructor
    at Object.<anonymous> (/root/twitchStreamBot/index.js:27:22)
    at Module._compile (internal/modules/cjs/loader.js:1063:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
    at Module.load (internal/modules/cjs/loader.js:928:32)
    at Function.Module._load (internal/modules/cjs/loader.js:769:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
    at internal/main/run_main_module.js:17:47

My code snippet.
const { TwitchRequest } = require("twitchrequest");
const TwitchClientOptions = {
    channels: [],
    client_id: "xxxxxxxxxxxxxxxxxxxx",
    client_secret: "xxxxxxxxxxxxxxxxxxxx",
    interval: 3
};
const twitchClient = new TwitchRequest(TwitchClientOptions);

Hi there Justin!
Sorry for the misunderstanding, the documentation is outdated and I have
now updated it to show the correct usage.
Here is the correct usage:
const { Client } = require("twitchrequest");
const TwitchClientOptions = {
channels: [],
client_id: "xxxxxxxxxxxxxxxxxxxx",
client_secret: "xxxxxxxxxxxxxxxxxxxx",
interval: 3
};
const twitchClient = new Client(TwitchClientOptions);
On Thu, Mar 4, 2021 at 6:26 AM Justin Dittmer<EMAIL_ADDRESS>wrote:

Hi, I've followed the documentation as far as possible, but I keep getting
the same error.
Maybe you can help me further.
const twitchClient = new TwitchRequest(TwitchClientOptions);
^
TypeError: TwitchRequest is not a constructor
at Object. (/root/twitchStreamBot/index.js:27:22)
at Module._compile (internal/modules/cjs/loader.js:1063:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:1092:10)
at Module.load (internal/modules/cjs/loader.js:928:32)
at Function.Module._load (internal/modules/cjs/loader.js:769:14)
at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:72:12)
at internal/main/run_main_module.js:17:47
My code snippet.
const { TwitchRequest } = require("twitchrequest");const TwitchClientOptions = {
channels: [],
client_id: "xxxxxxxxxxxxxxxxxxxx",
client_secret: "xxxxxxxxxxxxxxxxxxxx",
interval: 3};const twitchClient = new TwitchRequest(TwitchClientOptions);
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/destial/twitchrequest/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQEKNHLMFI5J2VOOD2DS27DTB2ZQLANCNFSM4YSCVQIA
.

Hi there Justin!
Sorry for the misunderstanding, the documentation is outdated and I have
now updated it to show the correct usage.
Here is the correct usage:
const TwitchClientOptions = {
    channels: [],
    client_id: "xxxxxxxxxxxxxxxxxxxx",
    client_secret: "xxxxxxxxxxxxxxxxxxxx",
    interval: 3
};
const twitchClient = new Client(TwitchClientOptions);

Thanks, that's how it works.
But I still have one short question. The const { Client } = ... is then more or less generally blocked, right?
Or could I use const Client = .... for something else then too?

const { Client } = ... takes the Client class only from the library, if you want to get all the classes, you can do
const TwitchRequest = require('twitchrequest')
const TwitchClientOptions = {
...
}
const client = new TwitchRequest.Client(TwitchClientOptions);

That way, you have access to TwitchRequest.StreamData and etc.

All right, thank you.


Safe file names

I faced a problem about it.
command = "ffmpeg -i 9. Introduction Testing.ts -qscale:v 3 TEMP/frame%06d.jpg -hide_banner"

This is not solvable for OS. Some trick is going to solve it.
command = "ffmpeg -i \"9. Introduction Testing.ts\" -qscale:v 3 TEMP/frame%06d.jpg -hide_banner"

After this cmd working like
ffmpeg -i "9. Introduction Testing.ts" -qscale:v 3 TEMP/frame%06d.jpg -hide_banner

Tested and work.

This is not better. Please see my below PR:
https://github.com/carykh/jumpcutter/pull/216


Cache hanging while trying to update

I'm using the cache with Redis, everything was fine until today the AddOrUpdate started failing for some reason
This is the log
info: CacheManager.Core.BaseCacheManager[0]
      Cache manager: adding cache handles...
info: CacheReflectionHelper[0]
      Creating handle redis-0 of type CacheManager.Redis.RedisCacheHandle`1[TCacheValue].
info: CacheManager.Redis.RedisConnectionManager[0]
      Trying to connect with the following configuration: 'localhost:6379,allowAdmin=False,connectTimeout=5000,ssl=False,abortConnect=False,connectRetry=10'
info: CacheManager.Core.BaseCacheManager[0]
      Cache manager: adding cache handles...
info: CacheReflectionHelper[0]
      Creating handle redis-1 of type CacheManager.Redis.RedisCacheHandle`1[TCacheValue].
info: CacheManager.Core.BaseCacheManager[0]
      Cache manager: adding cache handles...
info: CacheReflectionHelper[0]
      Creating handle redis-2 of type CacheManager.Redis.RedisCacheHandle`1[TCacheValue].
info: CacheManager.Core.BaseCacheManager[0]
      Cache manager: adding cache handles...
info: CacheReflectionHelper[0]
      Creating handle redis-3 of type CacheManager.Redis.RedisCacheHandle`1[TCacheValue].
info: CacheManager.Core.BaseCacheManager[0]
      Cache manager: adding cache handles...
info: CacheReflectionHelper[0]
      Creating handle redis-4 of type CacheManager.Redis.RedisCacheHandle`1[TCacheValue].
The AttheTouchOfAButton.OptiOp.EventListener service is now running, press Control+C to exit.
info: CacheManager.Redis.RedisCacheHandle[0]
      Loading scripts.
info: CacheManager.Redis.RedisCacheHandle[0]
      DB 4 | Failed to add item 1d5a6466-cbe8-4732-b16b-da7ea156b158 because it exists.

This is how I'm calling the cache operations
_calibrationCache.AddOrUpdate(guid.ToString("D"), calibration, x => calibration);

Just to clarify, my process was hanging bc I configured 1000 retries with 100 msecs timeout, the question would be why is trying to add if the entity is already on Redis

@dariogriffo-fourth AddOrUpdate eventually tries to add the item and will update in case the item exists already. So, the log message Failed to add item 1d5a6466-cbe8-4732-b16b-da7ea156b158 because it exists is totally fine and works as expected.
The log should also eventually have a row for the update though.
I'm not really sure what the issue is your are facing. Little bit hard to tell from the code you showed ;) Looks all fine to me so far. Can you maybe show me more detail or eventually provide code to reproduce the behavior? Thanks, M

That's the problem it was hanging on the retry loop without updating it. I waited for more than 2 minutes and nothing. There was no other interaction with the cache. I'm testing locally under controlled environment. Will see if I can reproduce it. I can add the configuration/setup of the cache also.

@dariogriffo-fourth well, I cannot reproduce this at all with correctly setup systems. Closing it for now.
If you have more information of how to reproduce it, let me know


Ignore build/

This change ignores the build/ directory, created by python setup.py build, with git status, etc.
While here, sort .gitignore entries.

I guess it’s not in there because we don’t build anything so it never came up? Anyways sorting is probably a good idea so thanks. :)


send-request does not accept spaces in URL

Hi,
What is the proper implementation to use SEND-REQUEST with spaces in the base Url that we are trying to query?  We have tried to follow the guidance on how to format the URL, but when we do so, the message comes back as an error.
    <send-request mode="new" response-variable-name="sec-response" timeout="60" ignore-error="true">
        <set-url>@{
            string baseUrl = https://{url}/data/entity?$select=f1,f2,f3,f4&filter=f4 eq 'test';
            return new System.Uri(baseUrl).AbsoluteUri;
        }</set-url>
        <set-method>GET</set-method>
    </send-request>

Thank you.

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

ID: e70c1f08-ac94-5cd1-f1fd-57548c0124da
Version Independent ID: 978fef6b-dea0-55b1-85fd-d7d81c754741
Content: Using API Management service to generate HTTP requests
Content Source: articles/api-management/api-management-sample-send-request.md
Service: api-management
GitHub Login: @adrianhall
Microsoft Alias: adhal

URL needs to be URL-encoded: https://www.w3schools.com/tags/ref_urlencode.asp?sm_au=iVVDMg0TSmrMV6Dm
You need to replace spaces (which are illegal in a URL) with %20
#please-close


Generate jsonbinpack support for TypeScript models

Reason/Context
In Modelina for TypeScript, you can make it generate the models generate basic support for serializing to and from binary with https://github.com/sourcemeta/jsonbinpack, this should be possible through the CLI.
To do this I suggest we add a flag called tsJsonBinPack as a boolean. If true the generator should include the following options:
  presets: [
    {
      preset: TS_COMMON_PRESET,
      options: {
        marshalling: true
      }
    },
    TS_JSONBINPACK_PRESET
  ]

Which should be added here: https://github.com/asyncapi/cli/blob/438de0741e9ef29672b6e3af483b90e796903a7d/src/commands/generate/models.ts#L118 as the code example shows: https://github.com/asyncapi/modelina/blob/master/examples/typescript-generate-jsonbinpack/index.ts
These will be the generated results: https://github.com/asyncapi/modelina/blob/master/examples/typescript-generate-jsonbinpack/snapshots/index.spec.ts.snap
Remember to also add a test here: https://github.com/asyncapi/cli/blob/438de0741e9ef29672b6e3af483b90e796903a7d/test/commands/generate/models.test.ts#L39
Related documentation: https://github.com/asyncapi/modelina/blob/master/docs/languages/TypeScript.md#generate-models-with-jsonbinpack-support
Feel free to reach out if you have any questions or get stuck!

working on this issue.


Initialise table from list of partitions

Currently when a table is created a list of split points can be provided so that the table can be partitioned. This only allows the table to be split on the first dimension. Issue #603 will allow the full partition information of a table to be exported to a file. This means that if a table has been split on dimensions other than the first we can create a new table that is partitioned in the same way. We need to be able to initialise a table from a file containing that exported partition information. We also need the ReinitialiseTable method to be able to accept the file of full partition information. We should continue to support the current methods that allow a table to be split using a file of split points from the first dimension as that will still be a common approach.

Resolved by https://github.com/gchq/sleeper/pull/682


Rename column valid_till_date to valid_until_date

Overview
Renames the field valid_till_date to valid_until_date to avoid using a non-standard abbreviation that only saves 1 letter.
Testing
Will run make pytest-coverage locally.
# To-do list
- [ ] Ensure docs build, unit & integration tests, and test coverage pass locally with `make pytest-coverage` (otherwise the merge queue may reject your PR)
- [ ] For significant ETL changes, ensure the full ETL runs locally
- [ ] For major data coverage & analysis changes, [run data validation tests](https://catalystcoop-pudl.readthedocs.io/en/latest/dev/testing.html#data-validation)
- [ ] If updating analyses or data processing functions: make sure to update or write data validation tests
- [ ] Update the [release notes](../docs/release_notes.rst): reference the PR and related issues.
- [ ] Review the PR yourself and call out any questions or issues you have

Brought this tiny change into #3421


Romain/shading

@flylo @edwardnewett @ravwojdyla

Codecov Report

Merging #43 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #43   +/-   ##
=========================================
  Coverage     36.42%   36.42%
  Complexity       67       67
=========================================
  Files            26       26
  Lines           884      884
  Branches         66       66
=========================================
  Hits            322      322
  Misses          544      544
  Partials         18       18

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 87b813d...80b9009. Read the comment docs.

👍


[189] potential listener LEAK detected, having 1000 listeners already. MOST frequent listener (391):

Error
at c.create in src/vs/base/common/event.ts:902:15
at g.q [as onMarkerChanged] in src/vs/base/common/event.ts:1109:34
at u.c in src/vs/base/common/observableInternal/utils.ts:226:28
at u.addObserver in src/vs/base/common/observableInternal/base.ts:264:9
at p.readObservable in src/vs/base/common/observableInternal/autorun.ts:271:14
at u.read in src/vs/base/common/observableInternal/base.ts:196:18
at h.isPresentAtPosition in src/vs/workbench/contrib/accessibilitySignals/browser/editorTextPropertySignalsContribution.ts:207:9
at p._runFn in src/vs/workbench/contrib/accessibilitySignals/browser/editorTextPropertySignalsContribution.ts:147:42
at p.j in src/vs/base/common/observableInternal/autorun.ts:196:10
at new p in src/vs/base/common/observableInternal/autorun.ts:164:8
at L in src/vs/base/common/observableInternal/autorun.ts:17:9
at d.value in out/vs/workbench/workbench.desktop.main.js:1254:53994
at l.B in src/vs/base/common/event.ts:1211:13
at l.C in src/vs/base/common/event.ts:1222:9
at l.fire in src/vs/base/common/event.ts:1246:9
at d.value in src/vs/editor/browser/widget/codeEditor/codeEditorWidget.ts:1720:38
at l.B in src/vs/base/common/event.ts:1211:13
at l.fire in src/vs/base/common/event.ts:1242:9
at C.s in src/vs/editor/common/viewModelEventDispatcher.ts:64:18
at C.endEmitViewEvents in src/vs/editor/common/viewModelEventDispatcher.ts:109:8
at <anonymous> in src/vs/editor/common/viewModel/viewModelImpl.ts:1111:27
at cb in src/vs/editor/browser/widget/codeEditor/codeEditorWidget.ts:1655:14
at D.U in src/vs/editor/common/viewModel/viewModelImpl.ts:1106:36
at D.restoreCursorState in src/vs/editor/common/viewModel/viewModelImpl.ts:1035:8
at K.restoreViewState in src/vs/editor/browser/widget/codeEditor/codeEditorWidget.ts:999:32
at N.setInput in src/vs/workbench/contrib/files/browser/editors/textFileEditor.ts:139:14
at async o.S in src/vs/workbench/browser/parts/editor/editorPanes.ts:434:4
at async o.L in src/vs/workbench/browser/parts/editor/editorPanes.ts:254:34
at async o.openEditor in src/vs/workbench/browser/parts/editor/editorPanes.ts:131:11
at async in src/vs/workbench/browser/parts/editor/editorGroupView.ts:1269:49
at async d.value in src/vs/workbench/contrib/files/browser/views/explorerView.ts:526:6

Go to Errors Site

Effectively fixed in https://github.com/microsoft/vscode/pull/219567


youtube-dl: backports

Motivation for this change
Backport the last couple youtube-dl releases.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: youtube-dl
Partial log (click to expand)
Successfully installed youtube-dl-2018.10.29
/build/youtube-dl
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/h2cpxac09czjjwiqmgkcvgfw4vx41624-python3.6-youtube-dl-2018.10.29
gzipping man pages under /nix/store/h2cpxac09czjjwiqmgkcvgfw4vx41624-python3.6-youtube-dl-2018.10.29/share/man/
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/h2cpxac09czjjwiqmgkcvgfw4vx41624-python3.6-youtube-dl-2018.10.29/lib  /nix/store/h2cpxac09czjjwiqmgkcvgfw4vx41624-python3.6-youtube-dl-2018.10.29/bin
patching script interpreter paths in /nix/store/h2cpxac09czjjwiqmgkcvgfw4vx41624-python3.6-youtube-dl-2018.10.29
checking for references to /build in /nix/store/h2cpxac09czjjwiqmgkcvgfw4vx41624-python3.6-youtube-dl-2018.10.29...
wrapping `/nix/store/h2cpxac09czjjwiqmgkcvgfw4vx41624-python3.6-youtube-dl-2018.10.29/bin/youtube-dl'...

Success on x86_64-darwin (full log)
Attempted: youtube-dl
Partial log (click to expand)
  The script youtube-dl is installed in '/nix/store/qxm8mmg933h52vaiyfba30iy1xsk4x7n-python3.6-youtube-dl-2018.10.29/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
Successfully installed youtube-dl-2018.10.29
/private/tmp/nix-build-python3.6-youtube-dl-2018.10.29.drv-0/youtube-dl
post-installation fixup
gzipping man pages under /nix/store/qxm8mmg933h52vaiyfba30iy1xsk4x7n-python3.6-youtube-dl-2018.10.29/share/man/
strip is /nix/store/df6k4mgdjxciy0f637lryp7c9ln7n1m3-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/qxm8mmg933h52vaiyfba30iy1xsk4x7n-python3.6-youtube-dl-2018.10.29/lib  /nix/store/qxm8mmg933h52vaiyfba30iy1xsk4x7n-python3.6-youtube-dl-2018.10.29/bin
patching script interpreter paths in /nix/store/qxm8mmg933h52vaiyfba30iy1xsk4x7n-python3.6-youtube-dl-2018.10.29
wrapping `/nix/store/qxm8mmg933h52vaiyfba30iy1xsk4x7n-python3.6-youtube-dl-2018.10.29/bin/youtube-dl'...

Success on x86_64-linux (full log)
Attempted: youtube-dl
Partial log (click to expand)
Successfully installed youtube-dl-2018.10.29
/build/youtube-dl
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/3x6482r180qfwai18lklcax6z4mccf1k-python3.6-youtube-dl-2018.10.29
gzipping man pages under /nix/store/3x6482r180qfwai18lklcax6z4mccf1k-python3.6-youtube-dl-2018.10.29/share/man/
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/3x6482r180qfwai18lklcax6z4mccf1k-python3.6-youtube-dl-2018.10.29/lib  /nix/store/3x6482r180qfwai18lklcax6z4mccf1k-python3.6-youtube-dl-2018.10.29/bin
patching script interpreter paths in /nix/store/3x6482r180qfwai18lklcax6z4mccf1k-python3.6-youtube-dl-2018.10.29
checking for references to /build in /nix/store/3x6482r180qfwai18lklcax6z4mccf1k-python3.6-youtube-dl-2018.10.29...
wrapping `/nix/store/3x6482r180qfwai18lklcax6z4mccf1k-python3.6-youtube-dl-2018.10.29/bin/youtube-dl'...


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fixed failed behat and phpunit tests

Coverage remained the same at 44.701% when pulling 3409e2726df1d8abdfb7426b43540b1a40d45040 on kilip:gh-473 into 2541edcdb0abf0240d147d516b640471308d50fb on cross-solution:file_upload_optimization.


Extract assert_parses_to into a parser utility crate

We have (or will have) multiple combine parsers in the tree:

tx
:find expressions
:where expressions
pull expressions.

These involve a number of utilities for both construction and testing. One such — the first — is the assert_parses_to! macro in query-parser/src/parse.rs.
This macro simplifies some of the boilerplate around running a parser function against input and expecting a certain result.
This ticket tracks extracting this macro into a separate sub-crate, perhaps called "parser_utils", for use elsewhere in the tree.

Will try this out.


Conflict between Audio and VTKVolume panel

https://pyviz-dev.github.io/panel/reference/panes/VTKVolume.html#gallery-vtkvolume

Should VTKVolume check for 3D arrays and Audio check for 1D uint16 arrays?

Seems reasonnable

Fixed with Audio pane check dtype and ndim #738


elastic_exporter metric values are difference from kibana

Hi
i use elasticsearch_exporter to monitor elasticsearch
Cluster topology

01 master
02 data nodes

from kibana values:

Primary
Docs Count
Replicas

2
9473429
1

from elasticsearch_exporter values

Prometheus query
Values

elasticsearch_indices_docs_total{cluster="cluster",index="jaeger-span-2020-02-10",instance="<IP_ADDRESS>:9114",job="elasticsearch_exporter"}
18946858

elasticsearch_export's value is double docs count of kibana, is it right??
why is elasticsearch_exporter have 2 x kibana values
thank you

9473429 should be the count of elasticsearch_indices_docs_primary (only primary documents, without replicas)
elasticsearch_indices_docs_total all primary multiplied with number of replicas -> 9473429 * (1 + 1) = 18946858
just guessing as this would make sense.

thank you

Same issue here, is there a way we can get the number of replicats from the metrics? Or is there a way to filter a metric so it only contains the primary documents?
I'm using release version 1.1.0 and it doesn't report a metric named elasticsearch_indices_docs_primary  or elasticsearch_indices_docs_total.
Only: elasticsearch_indices_docs, and there is no label on it to differentiate primary/replica.
Is it because I'm using v1.1.0? Should I be using a newer version?


if emergency stop button is active initialisation return success

I tried to initialize platform with joystick when emergency stop button was active and even there is an error in communication with motors there is ROS-INFO output about success of initialization.

the problem was just the output. Nothing wrong with the initialization behaviour in the base_drive_chain. Bugfixes in respective PRs referenced above

Thanks for reparing :+1:

Not merged yet ;-)
Von: Denis Štogl<EMAIL_ADDRESS>Gesendet: Dienstag, 10. Februar 2015 11:14
An: ipa320/cob_robots
Cc: Gruhler, Matthias
Betreff: Re: [cob_robots] if emergency stop button is active initialisation return success (#227)
Thanks for reparing [:+1:]
—
Reply to this email directly or view it on GitHubhttps://github.com/ipa320/cob_robots/issues/227#issuecomment-73675261.


Allow paragraphs or lists in body, not a mix of them

Test that should pass (exit code 0):
This is a title

Foo bar baz.

Bar baz foo.

Test that should pass (exit code 0):
This is a title

- Foo bar baz.
- Bar baz foo.

Test that should pass (exit code 0):
This is a title

* Foo bar baz.
* Bar baz foo.

Test that should fail (exit code !== 0):
This is a title

Foo bar baz.
Bar baz foo.

This is actually tricky to implement because of these 2 edge cases (should be exitCode = 0):
This is a title

This is a paragraph that has two sentences, each fits in 1 line.
The 2nd sentence.

(See above how 1st line fits 64 chars, so 2nd sentence has to go in next line.)
Another edge case:
This is a title

This commit msg only has 1 paragraph.
FooBarBaz123456789012345678 is a word that is too long to fit in
the first line of the paragraph.

(First line has less than 64 chars, but next word of 2nd sentence doesn't fit.)

Update: after thinking about it a bit more, it looks like this idea would somehow overlap with this other idea I had for another rule: https://github.com/nblockchain/conventions/blob/2bd08fa28f727583fc64ed954d222936b564134f/commitlint.config.ts#L44
So maybe we want the rule to be called proper-body-wrap.


pigeon classify does not produce output

Hello,
I am following this workflow (https://isoseq.how/classification/workflow.html) for classifying isoforms, and I am having an issue at the pigeon classify step. It has been running for almost 24h but no output is produced.
The reference genome annotation was originally in gff format, so I used AGAT to convert it to gtf. I then sorted and indexed the reference annotation GTF file and sorted the input transcript GFF file. When running the command pigeon classify <sorted.gff> <annotations.gtf> <reference.fa> --num-threads 12 --log-level INFO no output is produced.
To test the command on a smaller dataset, I subset the input transcript GFF and reference annotation GTF files to just one gene and ran pigeon classify, which worked without issues. But if I increase the number of genes even by a few then it gets stuck with no output or log messages being produced.
Any thoughts on what might be causing this?
pigeon 1.0.0 (commit -v1.0.0)

Using:
  pbbam     : 2.2.0 (commit v2.2.0-1-g8c081f6)
  pbcopper  : 2.1.0 (commit v2.1.0)
  boost     : 1.77
  htslib    : 1.14
  zlib      : 1.2.11

Thanks,
Iulia

Carefully watching for developments on this as well, same problem here. I can get it to run on subsets of the full gtf; up to 1499 lines work, but as soon as we use 1500 it resumes hanging forever.
As a work around, can this step be skipped if one wants to look at MASseq data without caring about novel splice isoforms? I understand that there is great value in this but I would accept losing that if it meant I could actually prep the Seurat-compatible file outputs so I can start to compare the single cell RNAseq dataset against some Illumina results I already have.


[Vnet Scale - CNS]: Flattening CIDR ranges for Node NNC to a list

Reason for Change:
*Introduces VNETBlock NCType in CNS
*Reads CIDR IP blocks from VNETBlock NC and updates secondaryIP config accordingly
VNETScale mode NNC example:
https://microsoft-my.sharepoint.com/personal/asn_microsoft_com/_layouts/15/onedrive.aspx?id=%2Fpersonal%2Fasn_microsoft_com%2FDocuments%2FMicrosoft Teams Chat Files%2FSampleVnetScaleNNC.yaml&parent=%2Fpersonal%2Fasn_microsoft_com%2FDocuments%2FMicrosoft Teams Chat Files&ga=1
This PR introduces support for VNET Scale mode in CNS. The NNC will have CIDR blocks for secondary IPs. Also primaryIP will also be a CIDR block where first IP will be used as PrimaryIP for NC and rest of the IPs will be assigned to secondary IP for pods.
Issue Fixed:

Requirements:

[*] uses conventional commit messages

[*] includes documentation
[*] adds unit tests

Notes:

@paulyufan2 @aggarwal0009
please don't "Update branch", it adds commits to your branch that are just noise:

You should rebase instead - conceptually, this means your changes will always happen "after" the latest master (which is our common true "latest" revision), instead of your changes and the concurrent changes from master being intermingled.
Usually, you can use this button:

But because you have mangled your branch history with merges already, you will need to do it manually via the command line, and resolve your conflicts locally.

@paulyufan2 @aggarwal0009 please don't "Update branch", it adds commits to your branch that are just noise:  You should rebase instead - conceptually, this means your changes will always happen "after" the latest master (which is our common true "latest" revision), instead of your changes and the concurrent changes from master being intermingled. Usually, you can use this button:  But because you have mangled your branch history with merges already, you will need to do it manually via the command line, and resolve your conflicts locally.

Updated

blocking for clean commit history

Updated


Secure REST-UI api for XSS vulnerabilities.

Secure REST-UI api for XSS vulnerabilities. REST-UI responses should escapeHTML (remove or escape special HTML chars).

Created Additional SecurityUtil class , to manage HTML string issues. Refactored Identity-Manager and Environment manager to validate inputs.


"ko and go have mismatched GOROOT"  error when trying ko commands.

Hello,
I am referring this document to build knative-serving to get images from source code.
https://github.com/knative/serving/blob/main/DEVELOPMENT.md
I have configured go, ko tools properly.
$ go version
go version go1.17 linux/amd64

$ ko version
0.8.3

In the Deploy knative-serving stage when I am trying below ko command
ko apply --selector knative.dev/crd-install=true -Rf config/core/
after waiting for more than 10 mins I am seeing below error message.
$ ko apply --selector knative.dev/crd-install=true -Rf config/core/
2021/09/06 05:13:23 NOTICE!
ko and go have mismatched GOROOT:
go/build.Default.GOROOT = "go"
$(go env GOROOT) = "/usr/local/go"

Inferring GOROOT="/usr/local/go"

Run this to remove this warning:
export GOROOT=$(go env GOROOT)

For more information see:
google/ko#106
2021/09/06 05:13:23 No matching credentials were found, falling back on anonymous
customresourcedefinition.apiextensions.k8s.io/images.caching.internal.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/certificates.networking.internal.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/configurations.serving.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/clusterdomainclaims.networking.internal.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/domainmappings.serving.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/ingresses.networking.internal.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/metrics.autoscaling.internal.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/podautoscalers.autoscaling.internal.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/revisions.serving.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/routes.serving.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/serverlessservices.networking.internal.knative.dev unchanged
customresourcedefinition.apiextensions.k8s.io/services.serving.knative.dev unchanged
[mayuchau@phoenix61861 serving]$ export GOROOT=$(go env GOROOT)
[mayuchau@phoenix61861 serving]$ ko apply --selector knative.dev/crd-install=true -Rf config/core/
`

as per the recommendation I am exporting GOROOT but again I am seeing same issue.
export GOROOT=$(go env GOROOT)
I have asked this question to knative-serving team as well.
reference:  https://github.com/knative/serving/pull/11900#issuecomment-913449304

I think this is related to/a duplicate of https://github.com/google/ko/issues/305.


Building Samples

Is there any plan to extend the CMake build process to include the samples directory, or is there an easy way to build those samples that I didn't notice?

It looks like the old build files were removed and thought to be replaced by CMake files but it never happened for the samples. Maybe you find something useful here: https://github.com/RigsOfRods/ogre-caelum/commit/793e98d9e310f99ec3918087fbb0ff3f1afe70ec
Pull request adding CMake files for the samples are welcome.

In that case, I'll see what I can do.
On May 14, 2016 4:48 AM, "Hiradur"<EMAIL_ADDRESS>wrote:

It looks like the old build files were removed and thought to be replaced
by CMake files but it never happened for the samples. Maybe you find
something useful here: 793e98d
https://github.com/RigsOfRods/ogre-caelum/commit/793e98d9e310f99ec3918087fbb0ff3f1afe70ec
Pull request adding CMake files for the samples are welcome.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/RigsOfRods/ogre-caelum/issues/2#issuecomment-219208815


Convert MultiListener and subclasses to TypeScript

OK, conversion complete and aqua is looking OK locally. Closing.


Add Gradient Background and remove background image

Remove the gradient image and add a gradient using CSS!
Check out this link for css-matic gradients

I'll do it!

Do ask me anything, if you get stuck anywhere :smiley:

Done #4.

Is this still available?

@Zunaid24 This has been merged!
Btw, I'll be opening new issues soon.
Consider staring the repo, so that coming here back would be handy for you!
Cheers!


fix: OnExit node can retry when retry a workflow (#13184)

Fixes #13184
Motivation
fix retry logic
Modifications

function FormulateRetryWorkflow in workflow/util/util.go file
function isDescendantNodeSucceeded in workflow/util/util.go file

Verification
OnExit node and its parent can be retried when a workflow is retried
yaml: test/e2e/testdata/retry-workflow-with-continueon.yaml

do retrying

done

what if there are multiple onExit nodes?

what if there are multiple onExit nodes?

The logic of multiple onExit nodes and one onExit node is the same, and both depend on the execution status of the parent node of onExit

If the parent node needs to perform a retry, the onExit node will be retried
If the parent node does not need to perform a retry, onExit will not be retried

There are two types of parent nodes that can be retried:

There are no successful child nodes other than the onExit node
In addition to the onExit node, there are other successful child nodes, and all of these nodes are required to retry at the same time

Can you try this workflow:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  name: workflow-exit-handler-demo
spec:
  entrypoint: main
  onExit: exit-handler
  templates:
  - name: main
    steps:
    - - name: fail
        template: fail
  - name: exit-handler
    steps:
    - - name: alarm
        template: alarm
  - name: alarm
    container:
      image: alpine:3.18
      command: [sh, -c]
      args: ["echo alarm"]
  - name: fail
    container:
      image: alpine:3.18
      command: [sh, -c]
      args: ["exit 1"]

Can step alarm in the exit handler be retried after manual retry?

@jswxstw Thank you very much for the example, I found that the alarm was not re-executed after I performed a retry, based on this example I re-modified the code logic, see the latest code submission for details

@jswxstw Thank you very much for the example, I found that the alarm was not re-executed after I performed a retry, based on this example I re-modified the code logic, see the latest code submission for details

There is already a function to determine whether or not node run as exit handler. You should consider using it directly.
https://github.com/argoproj/argo-workflows/blob/ac9cb3ae93bb441bb7ce09b74e99789adf599eae/pkg/apis/workflow/v1alpha1/workflow_types.go#L2384-L2387
However, this function has a bug now: the DisplayName of child nodes in exit handler(if it runs as Steps) misses the .onExit suffix.
I fixed it in #13016 as below, but this requires confirmation from other reviewers.

@jswxstw The function you provided is great, I will modify the code based on it, thanks
Review your change: All children of onExit are also added with the .OnExit suffix. The current onExit child node does not have this suffix, which is why running your example makes it impossible to retry with my pervious code.
I have a question for your changes: how about DAG, does DAG have such a bug as well?
So isExitNode function that I think can do this: recursively determine whether the parent group node contains the .onExit suffix. Then the children node name no need be added this suffix (related pr:https://github.com/argoproj/argo-workflows/pull/13016)

I have a question for your changes: how about DAG, does DAG have such a bug as well?

Good question. DAG do have the same bug.

So isExitNode function that I think can do this: recursively determine whether the parent group node contains the .onExit suffix. Then the children node name no need be added this suffix (related pr:https://github.com/argoproj/argo-workflows/pull/13016)

Good idea! But isExitNode is just the function of NodeStatus, it can not access its root node with boundaryID.

Good idea! But isExitNode is just the function of NodeStatus, it can not access its root node with boundaryID.

boundaryID is parent group node just like this function resetConnectedParentGroupNodes https://github.com/argoproj/argo-workflows/blob/ac9cb3ae93bb441bb7ce09b74e99789adf599eae/workflow/util/util.go#L815

But isExitNode is just the function of NodeStatus, it can not access its root node with boundaryID.

#13016 add a new function IsPartOfExitHandler as below, which applies to exit handler of types such as Steps, DAG, Container, and ContainerSet.
// IsPartOfExitHandler returns whether node is part of exit handler.
func (woc *wfOperationCtx) IsPartOfExitHandler(node wfv1.NodeStatus) bool {
	if node.BoundaryID == "" {
		return node.IsExitNode()
	}
	if boundaryNode, err := woc.wf.Status.Nodes.Get(node.BoundaryID); err == nil {
		return boundaryNode.IsExitNode()
	}
	return false
}

@jswxstw the code has been updated and can be reviewed


Xcode 16/ iOS18 Compatibility check

Description
Hi,
This is just general query to confirm the official Firebase SDK version that's compatible with Xcode 16/iOS18
Reproducing the issue
No response
Firebase SDK Version
10.24.0
Xcode Version
Xcode 16
Installation Method
Swift Package Manager
Firebase Product(s)
Analytics, Crashlytics, In-App Messaging
Targeted Platforms
iOS
Relevant Log Output
No response
If using Swift Package Manager, the project's Package.resolved

Expand Package.resolved snippet

Replace this line with the contents of your Package.resolved.

If using CocoaPods, the project's Podfile.lock

Expand Podfile.lock snippet

Replace this line with the contents of your Podfile.lock!

We expect any recent Firebase release to work with Xcode 16 and most of Firebase has built and run with it from the start. There are some specific issues that have arose that we've addressed and some issues still under investigation. Details at https://github.com/firebase/firebase-ios-sdk/issues?q=is%3Aissue++label%3A"Xcode+16+-+iOS+18"+


Failed to compile on #![feature(error_in_core)]

I added the crate as follows substrate-api-client = "0.11.0".
When I compile, I receive the following error message:
error[E0554]: `#![feature]` may not be used on the stable release channel
  --> /.cargo/registry/src/github.com-1ecc6299db9ec823/substrate-api-client-0.11.0/src/lib.rs:18:12
   |
18 | #![feature(error_in_core)]
   |            ^^^^^^^^^^^^^

Rust version: 1.68.2
Does it mean that the crate requires a nightly compiler version?
It is not mentioned anywhere in the documentation.
Is there any way to avoid this error?

Hello @f-squirrel
Yes, you currently need the nightly version. The rust version is specified in the rust-toolchain.toml file. With the cli cmd rustup show the toolchain should be installed automatically.
Checkout this part of the Readme:  https://github.com/scs/substrate-api-client#prerequisites

We assume this is resolved. Feel free to reopen the issue if not.