Parsed answer for the second part of day 1 in 2024 is wrong

Hello,
When parsing the text for the second part of the day 1 of this year, we get:
>>> aocd.models.Puzzle(day=1, year=2024).examples
[Example(input_data='3   4\n4   3\n2   5\n1   3\n3   9\n3   3', answer_a='11', answer_b='9', extra=None)]

However, this answer_b should be 31, not 9:

So, for these example lists, the similarity score at the end of this process is 31 (9 + 4 + 0 + 0 + 9 + 9).

Hello,
As said in the repo description:

the goal is to maximize the probability that your parser will somehow find the right result at the instant a new puzzle unlocks.

Then, the parser sometimes fails. Either you can wait for the examples.json file to be updated, or you can manually rectify it by including this (making examples correct until day 5).
"2024/01": [
    {
      "answer_b":   "b_code[-2] "
    }
  ],
  "2024/03": [
    {
      "answer_a":   "a_code[-3] ",
      "answer_b":   "None"
    },
    {
      "input_data": "b_code[-8]",
      "answer_a":   "None",
      "answer_b":   "b_code[-2] "
    }
  ],
  "2024/04":  [
    {
      "input_data": "a_code[-7]",
      "answer_a":   "a_code[-5]",
      "answer_b":   "b_code[-2]"
    }
  ]

…Or we can fix the parser? I’m not familiar with the code, but in the HTML snippet above the expected result is in , but for some reason the parser extracts the last number that’s inside a .

The reference locators have been updated in a recent release
$ aocd 2024 1 -e
                       --- Day 1: Historian Hysteria ---
                      https://adventofcode.com/2024/day/1
------------------------------- Example data 1/1 -------------------------------
3   4
4   3
2   5
1   3
3   9
3   3
--------------------------------------------------------------------------------
answer_a: 11
answer_b: 31
--------------------------------------------------------------------------------


Updated hprompt to accept config args

hprompt now accepts configuration params to customize the following:

toggle the format between "template" and "raw", to make it easier to migrate a prompt to the Helicone format
chain a tagged template literal function, to make it easier to integrate with existing functions, like dedent

I resolved the merge conflicts here:
https://github.com/Helicone/helicone/pull/1427/commits/77ac6b976c9f57538459412d0cc10f949704d486
Basically my thinking is that we should have two functions hprompt and hpromptc this way for a majority of the use cases they can use the short hand notation hprompt but if they want something more custom they can use hpromptc which stands for helicone prompt custom


bug 2002220: rgw: fix misleading log line in rgw health checker

There was a log line that informed that the object store status would
not be updated because the status was deleting erroneously. Move the
line to the correct position.
Signed-off-by: Blaine Gardner<EMAIL_ADDRESS>(cherry picked from commit c8b26e458ccda5268e8dd260990d4cfff814ca51)
(cherry picked from commit c80fb6d6fa40d0479721bb19727a0c0465f27ce3)
This will make debugging similar failures to this BZ easier in the future, as the log line removed is misleading.

Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
[ ] Skip Tests for Docs: Add the flag for skipping the build if this is only a documentation change. See here for the flag.
[ ] Skip Unrelated Tests: Add a flag to run tests for a specific storage provider. See test options.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.
[ ] Pending release notes updated with breaking and/or notable changes, if necessary.
[ ] Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
[ ] Code generation (make codegen) has been run to update object specifications, if necessary.

How about also cherry picking https://github.com/rook/rook/pull/8694 so the commitlint bot will be happy?

Looks like we need to cherry pick https://github.com/rook/rook/pull/8710 for the yaml linting. The object store tests are also failing for the object store, but it doesn't look related to your PR.


Customizing Router examples in docs causes max recursion depth.

Not sure if I'm missing something but the docs are defining getStateForAction as:
getStateForAction(action, state) {
    ...
    return MyApp.router.getStateForAction(action, state);
  }

which leads to that function calling itself over and over again. Am I missing something or is this a typo/mistake?

Note that I'm also using redux so this might have something to do with it? Not too sure.
const appReducer = combineReducers({
  nav: (state, action) => (
    AppNavigator.router.getStateForAction(action, state)
  ),
  ...
});

I think you are missing NavigationActions and navigate. My code for the same is like this and its works fine:
return AppNavigator.router.getStateForAction(NavigationActions.navigate({ routeName: 'route_name' }), state);

Hey @shawngustaw I just did this and my simplest solution was to make a copy of the old router before overwriting/extending:
const oldRouter  = MyApp.router;
MyApp.router = {
  ...MyApp.router,
  getStateForAction(action, state) {
    console.log('getStateForAction is extended!');
    return oldRouter.getStateForAction(action, state);
  }
}


feat: finite-dimensional manifolds are locally compact

... assuming they're modelled over a locally compact field. (Currently, the only such instances in mathlib are $\mathbb{R}$ and $\mathbb{C}$. Once local compactness of the $p$-adic numbers is added to mathlib, this theorem will apply to them also.)

part of the road towards Sard's theorem

The current proof assumes manifolds without boundary: for manifolds with boundary, one has to argue slightly differently. I'm not sure if the current definition of boundary is rigid enough to allow this. (At the moment, it would allow e.g. $E=\mathbb{R}$ and $H=\mathbb{Q}$, which is not locally compact.)
I'm open to moving these results to a new file, about topological properties of (topological or smooth) manifolds.
edit: I have just done so, as I'll probably have more material on this to come.
I've tagged with both geometry and topology. Arguably, this result is at the intersection of both.

Fixed linters errors and force-pushed.
While I was in the area, I removed some extraneous Set. qualifiers in both files (which are slightly discouraged). Let me know if I should split that into a separate commit, or drop it entirely.

Note: I have pushed an update to the lean toolchain because this PR was on a buggy version of the toolchain. WARNING: checking out old commits of this PR using v4.2.0-rc2 or v4.2.0-rc3 can cause lake clean to delete your mathlib folder! If you need to do so, make sure to delete lakefile.olean manually before running any lake commands.

I think the content of the PR is already essentially in the library (in greater generality as it doesn't assume boundaryless). To see what is already there, an invaluable tool is loogle: go to https://loogle.lean-lang.org/ and ask for LocallyCompactSpace, ChartedSpace or for LocallyCompactSpace, ModelWithCorners to see if we have already lemmas talking about locally compact manifolds.

I hadn't seen this - this is very nice. Thanks for pointing this out. So far, loogle never worked for me - but now it just did. I guess I'll just be happy I learned a new trick...
With this, the PR whittles down to one lemma of about five lines. I've moved it back.

bors r+
Thanks!


Add Support for Nextjs 15 and React 19

"peerDependencies": {
"next": "^13.0.0 || ^14.0.0 || ^15.0.0",
"react": ">=18.0.0",
"react-dom": ">=18.0.0"
},

Added Next.js v15 to peerDependencies in v1.5.0.
Not sure if defining >=18.0.0 covers canary React 19 releases though.
If you're using React 19, please give new version a try and tell if npm throws any warnings

Perfect, thanks for the update. I will test here to see if everything works correctly with React 19 and let you know.

Hi, regarding react 19, in its RC version it is running normally. TKS


[public report]: seconds counter for deadlines doesn't change

User Impacted
0xb40c994680dd68A63f7ED94608F8B524f6c569de
Discord ID (including number)
LTirrell#4794
Workflow Impacted
Analyze
Feature Impacted
Enter Submission
Issue description
The seconds part of the counter doesn't change (stays at 0). It should either be updated to countdown while the user is on the page, or removed

Blocking
no it is not
Relevant log output
No response

we should hide seconds from these counters, cc @elisemarika

I set a reminder for tomorrow to check out https://metricsdao.xyz/app/market/0x280b3b748Ccc42D5062CE59111fAd08594f51d9F/request/15/timeline so we can confirm if this is still an issue - looks like we show three "time units" and we still have days visible

check this one in the next 20 hours!  https://metricsdao.xyz/app/market/0x280b3b748Ccc42D5062CE59111fAd08594f51d9F/request/15/timeline
I am seeing seconds stuck at 0s - even if it's flakey I think we should remove it and just show minutes

@gnomadic that sounds good to me!


Trigger on CC incorrect

The following example will trigger on CC in ARIA.
It does not on sfizz, unless key=-1 is written explicitly.
Surely it's another case of ARIA where -1 and <not set> have identical semantic.
<region>
sample=*sine
start_locc16=100
start_hicc16=127

I remembered seeing it mentioned explicitely somewhere. It was on Peter Jones' page.

To prevent the player even trying to match the components of a Note On event for a layer, set the note number upper limit to -1. If you want the layer triggered by a MIDI CC event, you must do this.

I don't mind matching ARIA though. I'll test what happens with e.g.
<region>
sample=*sine
start_locc16=100
start_hicc16=127
key=64

and
<region>
sample=*sine
key=64
start_locc16=100
start_hicc16=127

@jpcima you corrected this already no?

I didn't do anything regarding this problem.

My examples mess up with sforzando, I've contacted David about it.
In the meanwhile I'll try to match the behavior by just deactivating CC triggering when a key-type opcode is encountered, and deactivating key-triggering when a CC trigger is encountered.
People mixing and matching those in regions are not likely. The most obvious case would be that there are some key (or CC) opcode in a group or master, and you want to have in your region a CC (or key) trigger. Using the latest opcode in this case thus makes sense to me.

Actually, your example:
<region>
sample=*sine
start_locc1=127
start_hicc1=127

does not disable key triggers in Sforzando. I'll need to update #315.


Would like to extend functions available to all my modules

The functions.sh is a good way to share functions across commands within a rerun module.  I have 20 modules and would like to share functions across all modules.  For instance, I'd like to append logs to a production journal function essentially extending rerun_log.  I'll probably want to do several shared functions down the road.
Would like the ability to add a RERUN _FUNCTION_DIR and drop source into that directory which gets sourced by all my rerun modules.

+1

+1


Edits in README MLP examples, and promote one of them to a file in examples dir.

See #126

Agreed that the examples in the readme do need fixing.
Examples should solve real problems on real datasets, while illustrating basic usage patterns of the library. So we won't merge the toy example.
Otherwise, it seems you are removing a lot of trailing spaces.


How to Include a URL and PDF as Attachments in a Protocol Message with fastapi_poe?

I'm working with the fastapi_poe library to build a bot that includes attachments in the prompt. I need to attach both a URL and a PDF file. However, I'm encountering validation errors. How can I properly structure the Attachment fields to ensure both types are recognized and processed?
Here is a snippet of my current implementation:
import asyncio
import fastapi_poe as fp
async def get_responses(api_key, messages):
async for partial in fp.get_bot_response(messages=messages, bot_name="gpt-3", api_key=api_key):
print(partial.text)
api_key = "YOUR_API_KEY"
message = fp.ProtocolMessage(
role="user",
content="Please create a paragraph based on the provided document.",
attachments=[
fp.Attachment(
url="https://example.com/document",
title="Document",
description="Detailed instructions",
content_type="text/html",
),
fp.Attachment(
url="https://example.com/sample.pdf",
title="Sample PDF",
description="A sample PDF file.",
content_type="application/pdf",
name="sample.pdf"
)
]
)
asyncio.run(get_responses(api_key, [message]))
asyncio.run(get_responses(api_key, [message]))
What modifications are needed to correctly include these attachments? Any help would be greatly appreciated!

Refer to https://github.com/poe-platform/fastapi_poe/issues/89, the API currently doesn't support arbitrary attachments, and the attachments should be uploaded through the user interface. See https://creator.poe.com/docs/server-bots-functional-guides#enabling-file-upload-for-your-bot for details on enabling file uploads for your bot.


Stg 83 Encryption v2 beta release prep

API change check
API changes are not detected in this pull request.

How are live and weekly tests doing?


Caption buttons are disabled for inactive window

When the window loses focus the caption buttons cannot be clicked as in a standard window. You must first click the window to make it active and then the buttons are active. Closing a window takes two clicks.

Fixed in FluentWPF 0.8.0-alpha003
I'll close this issue.


procedure d'annotation

will fix the opensidescan issue : https://github.com/CIDCO-dev/OpenSidescan/issues/158
and will test the procedure once the current training is done

procedure tested

procedure updated with separated train and val dataset


DUPLO-26806 DOC:TF:Node Pool: Add all allowed taints effect in document. DUPLO-26828 DOC:TF: Add example usage for snapshot_window in resource 'duplocloud_ecache_instance'

Overview
Added effect details to nodepool doc
Added ecache snapshot window example
Summary of changes
Added effect details to nodepool doc
This PR does the following:

Added effect details to nodepool doc

Added ecache snapshot window example

Testing performed

[ ] Using unit tests
[ ] Manually, on my local system
[ ✔] Manually, on a remote test system

Describe any breaking changes

...

Task linked: DUPLO-26828 DOC:TF: Add example usage for snapshot_window in resource 'duplocloud_ecache_instance'


Make ReadCountCollection error message more informative

"the input file cannot be null" will often be due to a PoN and case sample using different bed files. This should be noted in the error.

@achevali where does this occur in the code / does it still exist?

@achevali is this still an issue?

Not sure if this is still an issue, closing.


package-lock.json added to .gitignore

This adds the package-lock.json file to .gitignore. package-lock.json is a generated file that is updated by changes in the node module tree or package.json. It is not in source control but because it changes frequently, it appears among other deliberately changed files that are in source control.

would recommend reconsidering.
package-lock.json is one way to guarantee that the versions of installed dependencies on one machine are the same on another. This becomes important when running unit tests in a CI pipeline, or generally avoiding the 'works on my machine' problem. Discovering the distinction between the commands npm ci and npm install is one way to leverage the utility of a package-lock.json file and might persuade you to keep it as a committed file. If updating the version of any dependency is best thought of as a controlled act, you can gain this control through an intentional use of npm install <name>@<version>. Also, if there is no package-lock.json it will render npm ci a useless command.
https://docs.npmjs.com/cli/v7/commands/npm-ci

Thanks for your recommendation, @bdolor . I'll close this request. We'll review and update our dependencies in package.json and then add package-lock.json to this codebase (and do the same for the smk project).


Release 0.4.8

Let's wait for #690 to get merged so this version can be used for the Network changes on Puppeteer side as well.


Create ogr.txt

Istanbul University activate student license

@sahinekber18 I am sorry, the domain ogr.iu.edu.tr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Why dataset can't load alpha168/360 data since 2020-09-26?

🐛 Bug Description

To Reproduce
Steps to reproduce the behavior:

dump_bin latest daily trading data(cn)
test loading data by D, here's code:

fields = ['$adjClose', '$open', '$high', '$low', '$close', '$volume', 'Ref($close, 1)', 'Mean($close, 3)', '$high-$low']
D.features(['SH000300'], fields, start_time='2000-01-01', end_time='2022-12-31', freq='day').tail(300)

output:

That should say qlib can get the latest data, but here's dataset loading result:
import qlib
from qlib.contrib.data.handler import Alpha158

data_handler_config = {
    "start_time": "2008-01-01",
    "end_time": "2023-08-01",
    "fit_start_time": "2008-01-01",
    "fit_end_time": "2014-12-31",
    "instruments": "csi300",
}

if __name__ == "__main__":
    qlib.init()
    h = Alpha158(**data_handler_config)

    # get all the columns of the data
    print(h.get_cols())

    # fetch all the labels
    print(h.fetch(col_set="label"))

    # fetch all the features
    print(h.fetch(col_set="feature"))

Expected Behavior
Dataset load data just like D did.
Screenshot

Environment
Note: User could run cd scripts && python collect_info.py all under project directory to get system information
and paste them here directly.

Qlib version: 0.8.6
Python version: 3.7/3.8
OS (Windows, Linux, MacOS): Linux
Commit number (optional, please provide it if you are using the dev version):

Additional Notes

I found it caused by csi300 problem.


feature: support stream ?

Hello,
I create this issue to continue the discuss started here

For the fun and just for the fun
I made a funny ugly hack to create a fake string from file-descriptor
The sha1 require #str, string.byte(str, i, j) and a concat operation.
See https://github.com/tst2005/lua-fd2string

Yeah that seems somewhat suboptimal...

Copying interface draft:
local state = sha1.new()
state:add(some_input) -- Not necessarily a multiple of block size
state:add(some_input)
...
local hex_sha1 = state.sha1()) -- sha1 of all collected inputs in hex
local bin_sha1 = state.binary() -- sha1 of all collected inputs in binary
state:add(some_input) -- Can keep adding inputs

I don't know if you already know lockbox.
My current standalone sha1sum module built with lockbox
I run your tests and see your optimised sha1 is really faster than my sha1sum/lockbox current solution.

I've heard of lockbox before, looks like it doesn't really try to optimize for performance.

http://lua-users.org/lists/lua-l/2018-11/msg00114.html
Thanks for all work you done.
Rest in pease my friend.


Implementing adldap2 in Laravel 5.2 with system-specific access

Laravel Version: 5.2
Adldap2-Laravel Version: 2.1.3
PHP Version: 5.5.21

Description:
Hi Steve, Can you help as I am confused how to implement role-based authentication on top of ADLDAP2. Basically I can authenticate with LDAP ok, but I need the system to also check to see if the user is in the users table without adding any rows to it.  The user table has the names only of those who can use the system - not the passwords, so I can then control system access within the system itself and not AD groups.
I read how this is possible in V3 and read an earlier post that you could also do this in V2 of adldap and Laravel 5.2, which is exactly what I need. You said that it is possible by disabling password sync and adding the Laravel exists rule to your Username/Password validation. I have tried this and changing the
password_sync to false, but it still writes to the user table.
I'm also confused where to put that Laravel 'exists' rule exactly.  I can't see where/how I add that rule in Laravel 5.2. The only validation I can find in the default Laravel setup is in the Requests folder and relates to updating the user/password. However, this validation is not applied when logging in, only when updating the actual user details. Could you guide me on how this exists rule is applied to the log-in process? I apologise in advance if this is a bit out of scope for the package itself, but it is fundamental for us in being able to use ldap yet retain system-specific access control.
Many thanks

Hi @philbrowne,

The user table has the names only of those who can use the system - not the passwords, so I can then control system access within the system itself and not AD groups.

If you don't need to store their password, then you'll have to create your own solution unfortunately. This is being changed in a future release, but right now the password field is mandatory. However, it shouldn't be too difficult to create. Here's a basic implementation:
// app/Http/Controllers/Auth/LoginController.php

public function login(Request $request)
{
    $username = $request->username;

    if ($user = User::whereUsername($username)->first()) {
        // The user is logging in with a valid
        // email address as it's located
        // in our local table.
        if (Adldap::auth()->attempt($username, $request->password)) {
            // LDAP authentication succeeded, now to log them in locally.
            Auth::login($user);

            return redirect()->to('dashboard');
        }
    }

    return redirect()->to('login')->with('message', 'Login failed');
}

Does this answer your question for authenticating users locally into your application but not storing their password?

Thanks Steve, I can understand your code example -that's very helpful. I think I need to put this in RedirectIfAuthenticated so I will give this a try first. I'm thinking if I have a separate table for system users (which admins can configure) then I can system-authenticate first, and let adldap write to the users table as normal for ldap authentication. Thanks for your help.


Add methods of the checkout recurring

Why are the tests failing?

Why are the tests failing?

I don't know. I was surpriced too. It seems there are some troubles on payex side


About the paper

This is my first contact with fairseq. I want to know if this pytorch-fairseq is the fairseq mentioned in "Convolutional Sequence to Sequence Learning". Thanks!!!

In convolutional sequence to sequence learning, we used a LuaTorch, but this repo contains the PyTorch version. You have found the right place :)


refactor: Pass object to addParam in iDataQueue functions

Part of  #182

Running only the data queue functional tests
$ ./node_modules/mocha/bin/mocha --timeout 3s test/functional/ToolkitFunctional.js

  Toolkit Functional Tests
    DataQueue Functional Tests
      sendToDataQueue
        ✓ sends data to specified DQ using idb transport (86ms)
        ✓ sends data to specified DQ using rest transport (82ms)
        ✓ sends data to specified DQ using ssh transport (730ms)
        ✓ sends data to specified DQ using odbc transport (116ms)
      receiveFromDataQueue
        ✓ receives data from specfied DQ using idb transport (83ms)
        ✓ receives data from specfied DQ using rest transport (61ms)
        ✓ receives data from specfied DQ using ssh transport (652ms)
        ✓ receives data from specfied DQ using odbc transport (80ms)
      clearDataQueue
        ✓ clears the specifed DQ using idb transport (229ms)
        ✓ clears the specifed DQ using rest transport (77ms)
        ✓ clears the specifed DQ using ssh transport (646ms)
        ✓ clears the specifed DQ using odbc transport (82ms)

  12 passing (3s)
``


Autoplay and stop the video by passing a payload

Hello all,
I am new to this Node red with no experience with HTML or CSS.
I am working with this node and i am able to add images and videos on the dashboard.
I want that an image should appear as the background, once a inject button is clicked the first video should be played and stopped after playing it once and then again the image to be shown. Is it possible with this node?
If yes, then please guide me with this.

Hi @meakash
Unfortunately this is not possible with this node. For your goal, you'll probably want to use the ui_template node that comes with the dashboard, but you'll need to write some code there for it to work (possibly HTML, CSS and JS). I suggest you to take a look on the Node-RED forum (https://discourse.nodered.org/), there have been some discussions there already about applications similar to yours.


Fund Ledger Budget Groups Data to Import/Migrate

Fund, Ledger, Budget, Groups, Fiscal Year data needs to be migrated prior to Order data.
Migrate data from a CSV file
Create documentation on what the structure is and why

Status update: Draft CSV file complete. Send to Gabby

File sent to Data Migration vendor.


UI Alignment issues in Error dialogs

We see few UI Issues in error dialogs that needs to be corrected. Attached below some of the sample error screenshots.

@kasturiswain Can you check this ??

This issue is fixed in the latest build.


Add support for sub-commands

Add a SubCommands module.

Support defining the Main command which routes to the sub-commands.
Support registering sub-commands with the Main command.
Must somehow include the Main command's command_name in usage and help output (ex: Usage: maincmd foo [options]).
Support listing all sub-commands.
Support a default Help sub-command:

When given an argument: must load the sub-command and call help
When not given an argument: must list all sub-commands by command_name.

Added in c025b332afe86493b9d416c781decfc3d3f0f27b.


[feature] Discount in installments

Se agrega la row que comunica el descuento en la pantalla de Installments con el rediseño de UI

Codecov Report

Merging #1058 into feature/onetap_complete will increase coverage by 0.25%.
The diff coverage is 54.12%.

@@                     Coverage Diff                     @@
##           feature/onetap_complete    #1058      +/-   ##
===========================================================
+ Coverage                    27.47%   27.73%   +0.25%
===========================================================
  Files                          321      321
  Lines                        15035    15075      +40
  Branches                      1383     1387       +4
===========================================================
+ Hits                          4131     4181      +50
+ Misses                       10383    10365      -18
- Partials                       521      529       +8

Impacted Files
Coverage Δ

...ain/java/com/mercadopago/SecurityCodeActivity.java
51.81% <ø> (ø)
:arrow_up:

...ain/java/com/mercadopago/GuessingCardActivity.java
0% <ø> (ø)
:arrow_up:

...ain/java/com/mercadopago/PaymentVaultActivity.java
0% <0%> (ø)
:arrow_up:

...in/java/com/mercadopago/onetap/OneTapFragment.java
0% <0%> (ø)
:arrow_up:

...rc/main/java/com/mercadopago/CheckoutActivity.java
0% <0%> (ø)
:arrow_up:

...rc/main/java/com/mercadopago/views/AmountView.java
33.33% <0%> (+33.33%)
:arrow_up:

...ain/java/com/mercadopago/InstallmentsActivity.java
44.6% <55%> (-0.24%)
:arrow_down:

...c/main/java/com/mercadopago/CardVaultActivity.java
62.46% <71.23%> (-0.76%)
:arrow_down:

...com/mercadopago/presenters/CardVaultPresenter.java
80.95% <80.89%> (-0.58%)
:arrow_down:

.../mercadopago/presenters/InstallmentsPresenter.java
87.82% <81.48%> (-4.41%)
:arrow_down:

... and 2 more

Continue to review full report at Codecov.

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


Protein check script

Input submission checklist

[ ] created a directory for each system based on the template directory
[ ] named and placed the directory with the following pattern: input_structures/prepared_structures/<set_name>/<system_name>

For each submitted directory:

[ ] filled in system preparation details in the PREPARATION_DETAILS.md file
[ ] added a PDB file with the protein named protein.pdb
[ ] removed any cofactors from the PDB file and placed them in cofactors.sdf
[ ] did not re-protonate the residues, but left protonation state of the protein unchanged
[ ] kept cystallographic waters and metals in the PDB file
[ ] tested the PDB and, if available, cofactors.sdf file using the input validation script
[ ] copied over the ligands SDF file to a file named ligands.sdf and removed any duplicate binding modes or protonation states, keeping the more favorable state

Summary of changes

Licensing agreement

[ ] I agree to release these inputs under the combined MIT and CCBY SA 4.0 licenses of this repository

Hello @IAlibay! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file industry_benchmarks/utils/maint/__init__.py:

Line 1:1: W391 blank line at end of file

In the file industry_benchmarks/utils/maint/check_protein_prep.py:

Line 69:80: E501 line too long (109 > 79 characters)
Line 71:80: E501 line too long (113 > 79 characters)
Line 73:80: E501 line too long (95 > 79 characters)


Bundle a build Dockerfile tool.

If we have not installed docker, how to build docker image .
K3s can bundle a tool to build Docker image?
then we do not need any thirdparty tools go build Docker image from Dockerfile

This is not what you are asking for, but maybe this helps anyway: https://github.com/GoogleContainerTools/kaniko

I still hope that K3s will some day bundle nerdctl instead of ctr: https://github.com/k3s-io/k3s/issues/3279
With nerdctl you would be able to use nerdctl build.


Mediator for the whole multitenant agent - get default-mediator-id Unauthorized

Hi,
I have a mediator for the whole multitenant agent, in which there are no subwallets yet.
The first start of the multitenant agent with --mediator-invitation was successful and mediation was granted.
Subsequent starts of the multitenant agent require the --default-mediator-id parameter.
However, GET {mutlitenantAdminAPI}/mediation/requests returns 401: Unauthorized, therefore I am unable to find out the mediation id to put as the default mediator id.
I am aware of using the Bearer token for the subwallet, but I want to do it for the whole multitenant (the base wallet).
Could you please help me on how to do it?
Thanks

Hmm interesting. The admin API is blocking most api calls for the base wallet in multi-tenancy. Maybe we should make an exception for this specific endpoint?

I found a workaround. To be clear, I have persisted wallets in postgres (for both mediator agent and edge agent).

Start mediator (with --open-mediation)
Start edge agent without --multitenant and --multitenant-admin params. It is now possible to include the --mediator-invitation param to automate the connection and mediation, or connect the agents manually via the REST calls.
 I also created a sub-wallet, created a connection invitation on this sub-wallet, and indeed the routing key was present.

Not sure what to do with the info here -- looks like it could help in a doc -- but am closing this.
Hopefully the ACA-Py-based mediator in the aries-mediator-service repo takes care of this.


Refactored SearchCollection + added tests

Previously, in SearchCollection, we had:
  private List<TaggedSimilarity> similarities;
  private Map<String, RerankerCascade> cascades;

Refactored the second to:
  private List<RerankerCascade> cascades;

By added the ability to "tag" the cascade. This makes the code cleaner. Added associated tests.

hi @edwinzhng @nikhilro can you take a look at this PR?
I spoke to @Peilin-Yang out of band, and he doesn't have the bandwidth right now.
Thanks!


DHO Multitenant

Ability to configure and launch a new DHO with or without ties to the parent DHO

https://notepad.hypha.earth/nz3hUmAzQ1argsF_FzPKww?both#MULTI-TENANT-MVP


Add a fair-mode alternative of OrderedMemoryAwareThreadPoolExecutor.

This is a fair-mode alternative of OrderedMemoryAwareThreadPoolExecutor .
Motivation:

Unfair of OrderedMemoryAwareThreadPoolExecutor
The task executed in OrderedMemoryAwareThreadPoolExecutor is unfair in some situations.
For example, let's say there is only one executor thread that handle the events from the two channels, and events are submitted in sequence:
    Channel A (Event A1) , Channel B (Event B), Channel A (Event A2) , ... , Channel A (Event An)

Then the events maybe executed in this unfair order:
      ----------------------------------------------------> Timeline ------------------------------------------------------>
       Channel A (Event A1) , Channel A (Event A2) , ... , Channel A (Event An), Channel B (Event B)

As we see above, Channel B (Event B) maybe executed unfairly late.
Even more, if there are too much events come in Channel A, and one-by-one closely, then Channel B (Event B) would be waiting for a long while and become "hungry".
Modifications:

Fair of FairOrderedMemoryAwareThreadPoolExecutor
In the same case above ( one executor thread and two channels ) , this implement will guarantee execution order as:
   ----------------------------------------------------> Timeline ------------------------------------------------------>
    Channel A (Event A1) , Channel B (Event B), Channel A (Event A2) , ... , Channel A (Event An)

NOTE: For convenience the case above use one single executor thread, but the fair mechanism is suitable for multiple executor threads situations.
result:
There is a fair-mode alternative of OrderedMemoryAwareThreadPoolExecutor and OrderedDownstreamThreadPoolExecutor now.

AUTOMATED MESSAGE for ADMINS:Please verify and accept the pull request.To accept, say @netkins acceptTo build again, say @netkins buildTo whitelist the author, say @netkins whitelist

@netkins accept

@normanmaurer - 3.x = assigned to you :)

@Scottmitch sorry... can you tell me where is the code-format configuration file of netty ?

Check http://netty.io/wiki/setting-up-development-environment.html

Am 20.06.2015 um 07:15 schrieb wukezhu<EMAIL_ADDRESS>@Scottmitch sorry... can you tell me where is the code-format configuration file of netty ?
—
Reply to this email directly or view it on GitHub.

@normanmaurer  thank you, i had checked it and it is okey now

@atellwu will review today or tomorrow. Stay tuned.

@normanmaurer  Thank you very much for your patience. Following your suggestions, i have fixed the problems. Please have a check.
Lots of thanks!

@trustin looks good to me... Can you review as well please ?

@trustin Could you have a look for me

@trustin @normanmaurer  Very sorry... I am eager to wait for reply and accidentally clicked the "close and comment" button, so reopen it again.

@trustin ping :)

LGTM. Please cherry-pick once the Javadoc tag issues are fixed.

@atellwu please address @trustin 's comment. After that I will cherry-pick and release 3.10.4.Final. Thanks!

@trustin  @normanmaurer  Thank you very much, it's okey now

@atellwu just one last thing. Please squash your commits into one

@normanmaurer  Done  :-)

@atellwu sorry to be a pita but could you please do me a last favour and update the commit message to match our template:
http://netty.io/wiki/writing-a-commit-message.html

@normanmaurer Please check again ;-)

@atellwu thanks!
Cherry-picked as f93f32712e59efa23c8a0637158ef6c5ea88d9f7


Zcash Sapling Address regexp pattern is incorrect

https://github.com/unstoppabledomains/resolution-go/blob/50dc24df4923ec16c4d94230659aaf7e7c2e373a/keys.go#L166
Zcash Sapling addresses are 78 characters in length.
This middle regexp pattern will be shy by 1 character.  only 77 characters.
I have tested this in your UD interface for adding a sapling address to a crypto domain; if a user adds a valid sapling address, the UI errors and says it is invalid address. But if you delete 1 character from a valid address, then your UD interface accepts the sapling address as valid.
The error is a bug. It appears that the regexp is missing the "1" in the middle pattern, "zs1". Adding the 1 in the pattern will extend the length of the address to 78 characters, which is correct.
So, the corrected regexp should be: "^z([a-zA-Z0-9]){94}$|^zs1([a-zA-Z0-9]){75}$|^t([a-zA-Z0-9]){34}$"
Note: this bug was found by a Ycash contributor. We review Zcash and Ycash related code, as we feel both should be widely supported.

@boyfromcave Thanks for reporting this issue. It should be fixed with our latest version v2.1.0.


MikuMikuModel crash

open the modle will crash in Windows10 1809.

Do you have an Intel HD Graphics GPU?

What kind of a crash is it, may I ask?
Does it show a message box with an error code in it, or does it absurdly crash with no message, and just "MikuMikuModel has stopped working"?

What kind of a crash is it, may I ask?
Does it show a message box with an error code in it, or does it crash with only "MikuMikuModel has stopped working"?

nothing！

Windows 10 has removed the crash dialog in one of the major updates, so when a program is supposed to show "has stopped working" dialog, the program directly closes itself.
Maybe MMM is picking your Intel HD Graphics instead of your NVIDIA GPU. Try running MMM with your NVIDIA?

I closed the intel HD Graphics and still the same as before.

Well, this is weird...
Are you able to load/preview textures or sprites?

Well, this is weird...
Are you able to load/preview textures or sprites?

yes

Well, this is weird...
Are you able to load/preview textures or sprites?

I'm out of clues.
The issue seems to be caused by the new Intel HD Graphics drivers. People also reported the same crash issue to me and they all had Intel HD Graphics. I have one of these iGPUs, but I also have NVIDIA, just like you. Unfortunately, I cannot reproduce this crash, so it is super hard for me to debug it.
In the meantime, I'll be researching the solution for this problem.

Hey, I've done some fixes.  b94c92b
Could you two please test it?
You can download the latest development here:
https://ci.appveyor.com/project/blueskythlikesclouds/mikumikulibrary/build/artifacts

Hey, I've done some fixes. b94c92b
Could you two please test it? @WM86 @Omoinemie
You can download the latest development build here:
https://ci.appveyor.com/project/blueskythlikesclouds/mikumikulibrary/build/artifacts

same as before

doesn't build 1803/1809 have a gpu switching feature

Hey, I've done some fixes. b94c92b
Could you two please test it? @WM86 @Omoinemie
You can download the latest development build here:
https://ci.appveyor.com/project/blueskythlikesclouds/mikumikulibrary/build/artifacts

I hava another PC in 1803 without Intel HD Graphics，but it still crashed when click the obj.bin

Are your drives up to date? I seriously can't think of anything else, the latest commit fixed issues for some people.

I opened a teamviewer remote desktop：ID: 872 939 160 PW：u3ng38

@Omoinemie Do you have any place I can contact you? Like Discord

Are your drives up to date? I seriously can't think of anything else, the latest commit fixed issues for some people.

Discord ID：OmoineMie

I also need your Discord tag number.

I also need your Discord tag number.

For the time being, you could use the version that disables the model viewer:
https://ci.appveyor.com/project/blueskythlikesclouds/mikumikulibrary/builds/20708290/artifacts

@mitsu600 Could you test the latest commit? 4d436b28
I managed to reproduce the crash, and possibly fixed it.

I appreciate your update.
That's excellent! It works perfectly.
Thank you very much.

It seems the issue is finally fixed. I got more people to test the new build, and they all reported that it worked.
Feel free to reopen this issue if it still happens.


Allow templates to be optionally not called

Re: tracker1/gulp-header#11, I noticed this codebase is not the same as gulp-header. It seems that you have changed it quite a bit since forking it. I intend on making the similar changes to my fork of gulp-footer re: tracker1/gulp-header#11. Do you also want a pull to upgrade the codebase to the same gulp-header standard?

@whitneyit I've been meaning to fork gulp-header into a single gulp-header-footer with a configuration parameryer, and having gulp-header and gulp-footer both be wrappers for that... Since they are (were or should have been) more the same than different...


EISDIR error on mantra create

When I try to create a new mantra project, it throws the following error
$ mantra create
create  undefined/
invoke  init
toEnd: could not append to file (code EISDIR): .meteor/packages

That is an incorrect usage of CLI. You need to supply the path to your new app.
Please see the README. We should probably output a friendly error in this case. PR welcomed.

Whoops, missed that. Thank you, it works now!

Made a PR!

Thanks for the PR. Closing this.


dart2js: improve precision of instantiations in global inference.

The current implementation just forwards the type information from the expressions and therefore correctly tracks the instantiated functions.


Setup install_data uses fixed location for etc/ryu, breaks virtualenv (with fix)

To reproduce as a user (Ubuntu 16.04):
sudo apt install python-virtualenv
mkdir ~/faucet-test
cd ~/faucet-test
git clone https://github.com/onfsdn/faucet
virtualenv venv
. venv/bin/activate
pip install ./faucet
# Install fails, don't forget to deactivate the venv
deactivate

Installation will fail with:
error: could not create '/etc/ryu': Permission denied

When in a virtualenv, Ryu uses ./etc/ryu within the virtualenv directory, not the system-wide /etc/ryu. This breaks the ability to test and run Faucet within a virtualenv, requiring all libraries to be installed globally.
This seems to be because /etc/ryu/faucet is specified in the setup.py file source. According to the documentation on writing python setup scripts, if a relative path is used instead, it will default to the installation prefix for the currently used python installation. Removing the beginning '/' indeed causes the default configuration files to be properly located in ./venv/etc/ryu/faucet after running pip install ./faucet again (this is also what Ryu uses in their setup configuration):
(venv) insideof@ubuntu:~/ofworkspace/faucet/faucet⟫ git diff setup.py
diff --git a/setup.py b/setup.py
index 43c91a2..927afdf 100644
--- a/setup.py
+++ b/setup.py
@@ -16,7 +16,7 @@ with open(os.path.join(os.path.dirname(__file__), 'README.rst')) as readme:
         version='1.0',
         packages=['ryu_faucet'],
         package_dir={'ryu_faucet': 'src/ryu_faucet'},
-        data_files=[('/etc/ryu/faucet', ['src/cfg/etc/ryu/faucet/gauge.conf',
+        data_files=[('etc/ryu/faucet', ['src/cfg/etc/ryu/faucet/gauge.conf',
                                          'src/cfg/etc/ryu/faucet/faucet.yaml'])
                     ],
         include_package_data=True,
(venv) insideof@ubuntu:~/ofworkspace/faucet/faucet⟫ cd ..
(venv) insideof@ubuntu:~/ofworkspace/faucet⟫ ll ./venv/etc/ryu/faucet/
total 16
drwxrwxr-x 2 insideof insideof 4096 Aug 17 14:15 ./
drwxrwxr-x 3 insideof insideof 4096 Aug 17 14:15 ../
-rw-rw-r-- 1 insideof insideof 1846 Aug 17 13:59 faucet.yaml
-rw-rw-r-- 1 insideof insideof   28 Aug 17 13:59 gauge.conf
(venv) insideof@ubuntu:~/ofworkspace/faucet⟫

Shall I submit a pull request?

It looks like this is a bigger issue. Faucet does not use sys.prefix when generating the default configuration and log file locations. I'm willing to take this issue on if the team is willing to take a pull request on it.

Hi Leo,
Thanks for looking into this.  Please submit a pull request.
Please include how this was tested in the comment section of the pull
request.
Thanks & Regards
/Shivaram
On Wed, Aug 17, 2016 at 12:33 PM, Leo Scott<EMAIL_ADDRESS>wrote:

It looks like this is a bigger issue. Faucet does not use sys.prefix when
generating the default configuration and log file locations. I'm willing to
take this issue on if the team is willing to take a pull request on it.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/onfsdn/faucet/issues/11#issuecomment-240521904, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABmf84ZszNXgg2_uZe3eqmxGNZJJgMhBks5qg2IOgaJpZM4Jmy05
.

Sorry it's taken me a while to respond to this one. This one took me a while as I was not getting the results I expected at first and I've had to do a lot of research on how the different setup libraries used actually determine where data_files go.
tl;dr: I have a fix ready to submit that only addresses this issue. This is a request for comments on whether I should try to resolve the related issues below that I found while addressing this specific issue, or go ahead and submit the pull request with what I have.
It turns out that Ryu removed the beginning slash from the data_files section so setuptools/distutils can do the proper action when installing configuration, which is to follow the FHS guide for installed software by the system administrator. The ryu.conf file is actually placed in /usr/local/ryu/, not /etc/ryu/, when installed via pip install ryu or installing Ryu as a dependency of Faucet using pip install <faucet path>. When installing in a virtual environment, or another platform that doesn't follow Unix/Linux path conventions (Windows, which prompted the change), setuptools will install the data_files in the proper place for that platform (under sys.prefix). However, oslo_config (used by Ryu for config management) will search a number of paths for loading a configuration file, none of which include the default configuration file, so it seemed somewhat pointless except to resolve the cross-platform incompatibility issue.
When I perform the same change for Faucet's setup.py file (changing /etc/ryu/faucet to etc/ryu/faucet), the example config files are put in /usr/local/etc/ryu/faucet mirroring Ryu's default behavior when installed in this manner and following the FHS guidelines. This creates a couple problems, though:

The example file is no longer at /etc/ryu/faucet/, so documentation might need to be updated to reflect this and this may not reflect the desires of Faucet's development community. This is not technically a breaking change since Faucet doesn't work unless you manually create the log file directories anyways, so manual installation steps are still required (I hope to submit pull requests later to fix this). Also, the documentation recommends using environment variables to specify the location of the config files.
It would be difficult to update the default search path to always find the configuration files considering differences in platforms and libraries. I've tried finding a way to find, programmatically using available libraries, where the files in the data_files section are actually copied to, but this seems to vary depending on which library was used to install.

The fix I have implemented right now only detects if a virtual environment is used and updates the prefix accordingly, both in setup.py and the defaults used in faucet.py and gauge.py. This way, the original issue in this ticket is addressed (only use relative paths and sys.prefix if in virtualenv or pyvenv), but it does not address any cross-platform issues. On the up side, it also doesn't have the issues listed above with allowing setuptools to choose the paths automatically for a global install.


Libcloud 845 delete tags

Support deleting tags without values
ex_delete_tags won't delete a tag without the value - it passes 'None' as a string.
Modified ex_delete_tags to check for None as a passed Value of a tag and if found don't supply Tag.%d.Value to params (and on to EC2 API). See http://docs.aws.amazon.com/AWSEC2/latest/CommandLineReference/ApiReference-cmd-DeleteTags.html For API Reference - Value is not required and if supplied must match for tag to be deleted. By always passing a value, a tag cannot be deleted if you don't know it's value.
Status: done, ready for review
Checklist:

[x ] Code linting (required, can be done after the PR checks)
[x] Tests

👍 thanks @foriegnbody


__WEBPACK_IMPORTED_MODULE_1_nuxt___default.a is not a constructor

(node:6425) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: __WEBPACK_IMPORTED_MODULE_1_nuxt___default.a is not a constructor @pi0

@pi0  Please help

@github1586 Would you please explain more about your error details and usage? (are you using nuxt with express or standalone?)

I'm using nuxt, and I'm going to npm install this again today, and that's it @pi0

@github1586 have you found a solution, I'm having the same problem
@pi0 Using expressjs + nuxt
nuxt = new __WEBPACK_IMPORTED_MODULE_1_nuxt___default.a(config);

/* harmony import */ var __WEBPACK_IMPORTED_MODULE_1_nuxt___default = __webpack_require__.n(__WEBPACK_IMPORTED_MODULE_1_nuxt__);

/* harmony import */ var __WEBPACK_IMPORTED_MODULE_1_nuxt__ = __webpack_require__(17);

/* 17 */
/***/ (function(module, exports) {

module.exports = require("nuxt");

/***/ }),

So it seems, that the error is caused by require("nuxt")(); which is not a constructor

But it should work with the latest template of express!
vue init nuxt-community/express-template express-nuxt
cd express-nuxt
yarn && yarn dev # It works!

If using express-template, would you please update server/index.js to this one?
https://github.com/nuxt-community/express-template/blob/master/template/server/index.js

That does make sense!
It should do the trick, thank you 👍


The swagger api docs endpoint return a blank page cause of ERR_SSL_PROTOCOL_ERROR

see https://github.com/scottie1984/swagger-ui-express/issues/212 for detail

Based on the description of the problem in the linked issue, this is my take on what's happening. I was able to recreate the problem and test a fix.
The Problem
The problem occurs when running the ATT&CK Workbench on a server that is configured to use HTTP and not HTTPS (TLS). The source of the problem is that v1.0.2 (and earlier versions) of the REST API set the upgrade-insecure-requests directive of the Content-Security-Policy header in responses. Setting this directive prompts the browser to convert subsequent requests from HTTP to HTTPS. When the server isn't configured to handle HTTPS, these subsequent requests will fail.
Note that running locally, browsers typically behave differently. The upgrade-insecure-requests directive is ignored for requests sent to localhost. So running the ATT&CK Workbench on a user's machine and connecting to http://localhost avoids this problem, even if the server is running HTTP. The problem only occurs on non-local hosts running HTTP.
Theupgrade-insecure-requests directive is intended to support the migration of a legacy site from HTTP to HTTPS without needing to update every link individually. See CSP: upgrade-insecure-requests
Since this situation (migrating legacy code to HTTPS) doesn't apply to the ATT&CK Workbench, it should be acceptable to remove this directive.
The Fix
The upgrade-insecure-requests directive is set by default by helmet. (helmet is a module that adds a predefined set of security headers to responses to improve application security).
helmet can be configured to enable or disable individual headers and directives. In this case, this code can be used to disable the upgrade-insecure-requests directive, while retaining all other headers and directives.
function disableUpgradeInsecureRequests(app, helmet) {
    const defaultDirectives = helmet.contentSecurityPolicy.getDefaultDirectives();
    delete defaultDirectives['upgrade-insecure-requests'];

    app.use(helmet.contentSecurityPolicy({
        directives: {
            ...defaultDirectives,
        },
    }));
}

Timeline
This is probably a v1.1 fix.

great work and looking forward to v1.2. I am working on integrating the workbench into my xdr product, which communicated over the plain http protocol in a internal overlay network(provide https on gateway instead of every internal web app), so I fixed it in my local distribution.


Pull request/1f4bdfee

Significantly improves update_R_file, old implementation is kept in update_R_file_old. I also removed an unused default parameter in update_R_file. The following comparison is done on a macbook pro.
For small data the difference is negligleble:
N = 10
data = Dict("Scalar" => rand(),
            "Single_element_vector" => [rand()],
            "Vector" => rand(N),
            "Array" => rand(N,3))

@time Stan.update_R_file("testdata0", data)
@time Stan.update_R_file_old("testdata1", data)

gives
0.000550 seconds (200 allocations: 9.859 KB)
0.000582 seconds (629 allocations: 93.781 KB)

But on large data, the speed up is significant.
N = 10000
data = Dict("Scalar" => rand(),
            "Single_element_vector" => [rand()],
            "Vector" => rand(N),
            "Array" => rand(N,3))

@time Stan.update_R_file("testdata0", data)
@time Stan.update_R_file_old("testdata1", data)

gives
 0.033156 seconds (238.07 k allocations: 8.188 MB)
 52.889273 seconds (638.21 k allocations: 56.671 GB, 41.81% gc time)

i.e. More than 1000 x speed up.
Best
Jon

Btw, I checked that the output files are equal by
cmp testdata0 testdata1

Hi Jon, that looks pretty effective!! I'll merge later today.  Thanks for preparing this PR.

Merged! Use Pkg.checkout("Stan") for now to use this feature.


Make a UMD Module and module loadable via npm.

It would wonderful if this could be wrapped in UMD so that you can use this as a script tag, a node module, or AMD/CommonJS require, or ES6/ES2015/Typescript Import.

👍  Really needed.

I have installed this script using `npm install winwheel', but I have noticed that in this way I get old version of the script. I don't quite understand relationship between Github releases and npm, but could this be synced so we can use npm to get latest version?

Hi @kovinet the npm version is not owned by me, someone took a copy of the Winwheel.js and put it up there a long time ago and yes they don't update it so its quite out of date (plus they did not even match the version number with the Github release version). To ensure you can use the latest features please download Winwheel.js from here on Github. Thanks.


Add Kyuubi Hive JDBC Module

Why are the changes needed?

Aiming to make a better supported client for Kyuubi and Spark

Add catalog to getTables meta function for DataLakes (DONE)
Deploy to maven central (PENDING)

How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

Codecov Report

Merging #762 (6f01f8e) into master (ce3147f) will decrease coverage by 0.12%.
The diff coverage is 65.71%.

@@             Coverage Diff              @@
##             master     #762      +/-   ##
============================================
- Coverage     80.50%   80.37%   -0.13%
- Complexity        0        6       +6
============================================
  Files           122      125       +3
  Lines          4749     4784      +35
  Branches        576      579       +3
============================================
+ Hits           3823     3845      +22
- Misses          605      613       +8
- Partials        321      326       +5

Impacted Files
Coverage Δ

...in/scala/org/apache/kyuubi/jdbc/KyuubiDriver.scala
37.50% <37.50%> (ø)

...rg/apache/kyuubi/jdbc/KyuubiDatabaseMetaData.scala
70.58% <70.58%> (ø)

...cala/org/apache/kyuubi/jdbc/KyuubiConnection.scala
80.00% <80.00%> (ø)

.../scala/org/apache/kyuubi/server/KyuubiServer.scala
47.36% <0.00%> (-2.64%)
:arrow_down:

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 ce3147f...6f01f8e. Read the comment docs.

Thanks, will merge it, and open #765 as a follow-up.


Audit deps

Built on top of #810

Updates cli to use "github": "^13.0.0" to remove https-proxy-agent. This is not the latest version (now https://www.npmjs.com/package/@octokit/rest), but is the first version that passes the audit without breaking us.
Updates send and lodash in web-component-tester. I saw there's also #809, which seems like a superset of these changes (though introduces more source changes)

This doesn't fix all audit warnings, but should cover most of the more pressing ones.

Oh no. Don't merge this, but do take a look at #810


MSID: 101103 Version: 1 DOI: 10.1101/2<IP_ADDRESS>6161

MSID: 101103
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>6161
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=101103
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>6161

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/101103v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>6161
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/101103/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@acollings Does this assessment need rewording?

There are also two beta symbols missing from Reviewer 1's comments that we'll need to fix.

Thanks @JGilbert-eLife -- edited accordingly

@fred-atherden We need to do a V2 for this using the bioRxiv v3 preprint at https://api.biorxiv.org/meca_index/elife/10.1101/2<IP_ADDRESS>6161

Doing this now, thanks!


[Doc Improvement][Update extend m365 teams message extension]

updated outlook to Microsoft 365 as per word doc shared by achyut.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
97
22
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 247f230:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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

Learn Build status updates of commit b99c5b0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
98
21
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 0731a90:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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

Learn Build status updates of commit 8c2fcee:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/glossary.md
:warning:Warning
View
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md

Line 144, Column 3: [Warning: bookmark-not-found] Cannot find bookmark '#add-microsoft-365-channel-for-your-bot' in 'm365-apps/extend-m365-teams-message-extension.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
98
21
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 19ba8ba:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/glossary.md
:warning:Warning
View
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md

Line 144, Column 3: [Warning: bookmark-not-found] Cannot find bookmark '#add-microsoft-365-channel-for-your-bot' in 'm365-apps/extend-m365-teams-message-extension.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
98
21
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit ca9082a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
98
21
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 0ef3376:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
98
22
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit c1d893a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
98
22
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit 4981b3b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/get-started/glossary.md
98
22
link
:white_check_mark:

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
98
9
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.

Learn Build status updates of commit ca2dd9b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/assets/images/azure-bot-channel-message-extensions.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/azure-bot-channel-message-extensions-apply.png
:white_check_mark:Succeeded
View

msteams-platform/get-started/glossary.md
:white_check_mark:Succeeded
View

msteams-platform/m365-apps/extend-m365-teams-message-extension.md
:white_check_mark:Succeeded
View

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


Updating AF::Noid

Coverage remained the same at 100.0% when pulling 06d0a392b5797d2a71d317f39014f26cc8636e1a on afnoid into 23ae32d6e08e2aa58dea36d446edb5e3dac664a9 on master.


Improving Grammatical Errors

Improving Grammatical Errors in the readme file

Have you opened a issue first ? @Susmita-Dey is this fine ?

Have you opened a issue first ? @Susmita-Dey is this fine ?

Yes, I have already created an issue and been assigned to this task; you can see it here: https://github.com/Susmita-Dey/Sukoon/issues/83#issue-1330815789

Umm, I think the grammar was fine already. There is no need for a change according to me. The sentences look even more odd after the changes.


[DOC] Incorrect interval in jax.nn.log_softmax

Note that the mathematically correct interval should be
:math:`(-\infty,0)`

https://github.com/jax-ml/jax/blob/6790b90f91b0fbb8d818de8878f99eb3c4a871c2/jax/_src/nn/functions.py#L505
https://github.com/jax-ml/jax/blob/6790b90f91b0fbb8d818de8878f99eb3c4a871c2/jax/_src/scipy/special.py#L2632

I think the documentation should follow the code and not necessarily the mathematically correct definition, but either way the docs are wrong: This function does potentially output the floating point representation of -inf, but it can also output 0 due to rounding. So the interval should either be [-inf, 0] following code or (-inf, 0) following math.
>>> jax.nn.log_softmax(jnp.array([-jnp.inf, 0, 1])) # outputs -inf
Array([       -inf, -1.3132616 , -0.31326163], dtype=float32)

>>> jax.nn.log_softmax(jnp.array([-jnp.inf, -1000, 1])) # outputs 0
Array([  -inf, -1001.,     0.], dtype=float32)

@jakevdp or @mattjj thoughts? I can submit a quick PR to update this if it's a good idea.


Provide Warning dialogue box before allowing Withdrawn status in Projects

Tested on Edna Finnesgard. I changed status to Withdrawn and then could not change it to In Progress after. The "Complete" status change has a pop up warning, which is a helpful warning for permanent changes.
Or, don't make Withdrawn a permanent change and allow changing status back to Upcoming or In Progress.

Added a message in commit.
I can also make the reversal of the declined assessments or withdrawn / completed projects possible as well. Right now, it's permanent since everything is set to a complete status which makes prevents editing, but doing the reversal shouldn't be too difficult to implement.

Let me check out your changes and talk to Shawn & team about ability to reverse. I think once projects are "complete" they should never change (this is where your checks are really useful), but sometimes we do have a decision to decline or a client reinstates their application after withdrawal, so reversal may be helpful. But, wait for me to confirm before any further changes. If this works, I'll close the issue.

I checked with Shawn and I'm including his full answer here along with a summary for action after:
“Generally speaking, I really like the idea of “locking” applications that are withdrawn/declined and projects that are completed, because then those records become a permanent reflection of what actually happened.  However, I because we’re human and because there are multiple people working on the same files, I think there will be lots of times that we’ll want to update some information about a completed project after the fact….for example, at the end of the year when I’m doing my reporting, there’s a huge likelihood that when I run a report for the whole year, the results it spits out won’t be perfect on the first try, and we’ll need to go back into individual records to add things that were forgotten, etc.  I also think that there will be times when we want to re-open a declined or withdrawn application, and if we can’t, that will be a total drag to have to redo everything from scratch.
So after writing this and processing myself through the process of writing this, I guess what I’m saying is that I’d like the ability to mess with things once they’re completed.”
In short, our documentation never really seems to be complete and we have to go back after the fact and update / change / correct / add things, so locking the data sounds like a good idea to prevent it getting messed up, but in reality would result in less complete and less accurate client and project info for Catalyst.
If you can figure out a way to provide a warning dialogue box for when those applications and projects are opened to be careful that any changes will alter the project info and be saved, that would be a helpful reminder.

Assessments, projects, work items, and material items can now be edited and are no longer locked.


ivy

This likely means that the library (ng-zorro-antd) which declares NzInputModule has not been processed correctly by ngcc, or i
s not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking
with the library's authors to see if the library is expected to be compatible with Ivy.

Could you provide a valid reproduction?

For anyone that has this issue, I have had it too and I see that restarting angular server resolves it.
ng serve

Para cualquiera que tenga este problema, yo también lo he tenido y veo que reiniciar el servidor angular lo resuelve.
ng serve

Also works for version 11.0.0


Unpack tool, its image and an example

This PR includes the lean unpack tool which reads all Kubernetes YAMLs in the current directory recursively and prints them to stdout. For additional metadata processing, flags or different binaries and their images can be built.
@kasey I couldn't add you as reviewer, seems you're not member of crossplane-contrib org.
Fixes https://github.com/crossplane/crossplane/issues/1714

Currently, this reads only YAML files. The stream piping each and every file is not feasible because we'd need to invent new file separators for different files and different types of files. I believe for additional Upbound metadata, we'll either need to build another binary or introduce a flag to this one that will read icons, ui-schemas etc directly from the filesystem rather than its stdin. It can use this binary to get all YAMLs or be completely separate.

@hasheddan thanks for the review! All your feedback should be addressed.


New Logger - add_transaction_failure_tracing

New logger add_transaction_failure_tracing  reflected. Resolves #9252
Modified transaction_tracing info based on SME feedback

Change Description

Change Type
Select ONE

[x] Documentation

[ ] Stability bug fix

[ ] Other

[ ] Other - special case

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[x] Documentation Additions

This PR was based on assuming that the release/2.0.x change was meant to be reflected into develop, whereas the original change was what was desired, and the release/2.0.x change was meant to keep from making that change in a minor release.  Closing this PR.


'ionic-app-scripts serve' takes ~90 seconds to compile

Short description of the problem:
"ionic-app-scripts serve" takes ~90 seconds to compile
What behavior are you expecting?
Less than 15 seconds of compile time.
Steps to reproduce:

clone https://github.com/ugurdonmez/kinder-mobile/tree/pages-fix branch
run 'npm install' on project directory
run 'ionic:serve'

Which @ionic/app-scripts version are you using?
@ionic/app-scripts: 1.3.7
Windows_NT: 10.0.14393
node: v6.9.5
npm:  v3.10.10
Other information:
For "ionic run browser", it is even longer than 90 seconds.
package.json:
{
  "name": "ionic-hello-world",
  "author": "Ionic Framework",
  "homepage": "http://ionicframework.com/",
  "private": true,
  "scripts": {
    "ionic:build": "ionic-app-scripts build",
    "ionic:serve": "ionic-app-scripts serve",
    "ionic:run": "ionic run browser"
  },
  "dependencies": {
    "@angular/common": "2.4.8",
    "@angular/compiler": "2.4.8",
    "@angular/compiler-cli": "2.4.8",
    "@angular/core": "2.4.8",
    "@angular/forms": "2.4.8",
    "@angular/http": "2.4.8",
    "@angular/platform-browser": "2.4.8",
    "@angular/platform-browser-dynamic": "2.4.8",
    "@angular/platform-server": "2.4.8",
    "@ionic/storage": "1.1.x",
    "angularfire2": "2.0.0-beta.8",
    "firebase": "3.7.x",
    "ionic-angular": "2.3.x",
    "ionic-native": "^2.2.17",
    "ionicons": "3.0.0",
    "ng2-translate": "^5.0.0",
    "rxjs": "5.0.1",
    "zone.js": "0.7.2",
    "sw-toolbox": "*"
  },
  "devDependencies": {
    "@ionic/app-scripts": "1.3.x",
    "typescript": "2.3.x",
    "cordova": "6.5.x"
  },
  "cordovaPlugins": [
    "cordova-plugin-whitelist",
    "cordova-plugin-console",
    "cordova-plugin-statusbar",
    "cordova-plugin-device",
    "cordova-plugin-splashscreen",
    "ionic-plugin-keyboard",
    "cordova-plugin-camera"
  ],
  "cordovaPlatforms": [
    "ios",
    {
      "platform": "ios",
      "version": "",
      "locator": "ios"
    }
  ],
  "description": "kinder-mobile: An Ionic project"
}

tsconfig.json
{
  "compilerOptions": {
    "allowSyntheticDefaultImports": true,
    "declaration": false,
    "emitDecoratorMetadata": true,
    "experimentalDecorators": true,
    "lib": [
      "dom",
      "es2015"
    ],
    "module": "es2015",
    "moduleResolution": "node",
    "sourceMap": true,
    "target": "es2015"
  },
  "include": [
    "src/**/*.ts"
  ],
  "exclude": [
    "node_modules"
  ],
  "compileOnSave": false,
  "atom": {
    "rewriteTsconfig": false
  }
}

tslint.json:
{
  "rules": {
    "no-duplicate-variable": true,
    "no-unused-variable": [
      false
    ]
  },
  "rulesDirectory": [
    "node_modules/tslint-eslint-rules/dist/rules"
  ]
}

My compile time just decreased to 16 seconds by restarting my computer. The issue is solved.


missing typings folder in npm module

Hi Krispo, thanks for the update. :-)
I've tried the 2.0.0-rc1 version and error occurred when I run ng serve in my local mac.
Here's the error stack:
$home/node_modules/ng2-nvd3/typings/globals/d3/index.d.ts
Cannot read property 'length' of undefined
TypeError: Cannot read property 'length' of undefined
at createSourceFile ($home/node_modules/typescript/lib/typescript.js:14877:109)
It seems that we don't have typings folder in this case. Hope to get your help soon. Thanks again. :-)

Thanks, fixed in rc2!


Animations are not exported in build

Hi, I do export correctly glb with animations from editor, but when i tried with th built project glb has no animations,not sure is an issue but some kind of my error, any hints?

Hi @hierro I downloaded UnityGIT from this link: https://github.com/prefrontalcortex/UnityGLTF.git?path=/UnityGLTF/Assets/UnityGLTF via Package Manager. In Project Settings Export animations is enabled but when I Export is not working... Could you please explain how you exported the animations form Editor?)

Please update to the latest version and open a new issue if the problem persists. Thanks!


Fix to embeddings freq_items and handling of numpy strings

Frequent items sketches were empty due to the performance improvement change to wrapping closest array in numpy. We should check out if there's still a numpy solution and the effect of this on performance, but this at least fixes the bug.

Richard convinced me that we should expand scope to fixing of NumpyView to stop taking pd.DataFrames and only np.ndarrays as contracted by our MyPy signature. I have added that potential change in this PR, but it requires lots of cleanup for test cases.
Richard was excited to do this and I will not stop him! But it is strange that tests like test_merge_single_values_profile_mean are giving slightly different values. Needs investigating.


Revert "feat: add github summary integration"

Reverts lacework/lw-scanner-action#52

Need to revert as this breaks our exit codes which are required to fail pipeline if a policy finding exists.


DEV: SeqDataView.map sliced by slice_record

Addressing comment on https://github.com/cogent3/cogent3/pull/2079#discussion_r1825401297
Summary by Sourcery
Enhance the SeqDataView class by slicing the IndelMap in the 'map' property using slice_record, and update the parent_seq_coords method to use the parent's IndelMap for accurate index retrieval. Additionally, remove the xfail marker from a test, indicating a resolved issue.
Enhancements:

Modify the 'map' property in SeqDataView to return a sliced IndelMap based on slice_record.

Tests:

Remove the xfail marker from the test_coevo_annotated test, indicating the issue with inconsistent length between alignment and map has been resolved.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

7 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.02%) to 92.118%

Files with Coverage Reduction
New Missed Lines
%

src/cogent3/core/new_alignment.py
1
94.76%

Totals

Change from base Build<PHONE_NUMBER>0:
0.02%

Covered Lines:
33575

Relevant Lines:
36448

💛  - Coveralls


[Bug] Can't connect SSL: BAD_SIGNATURE

The problem
Hi, good morning! I tried to test the integration but I am running into this problem when trying to load the end device with SDK Server via hostname.
The integration does not connect with the client. SSL:BAD_SIGNATURE certificate error.
The server loads correctly if I do it from localhost, in fact, I use the API from Python to manage certain things and it is working fine. The problem only occurs from the HASS server.
Thanks in advance for any ideas or suggestions.
Your device(s) with a problem
Ouroboros, Seiren Pro, Blackwidow Chroma, Huntsman Elite, Nari Ultimate, Kraken 7.1 V2, Goliathus Extended (XL)
Firmware version
3.27.04
Integration version with the issue
Latest (0.2.0)
Method of the integration installation
HACS
What version and type of Home Assistant installation do you use
OS 9.0 - Supervisor 2022.09.1 - Home Assistant 2022.10.1
Is there anything useful in the logs?
Config entry 'Chroma' for chroma integration not ready yet: Cannot connect to host desktop-2skaf11.pecceg2.local:54236 ssl:True [[SSL: BAD_SIGNATURE] bad signature

Additional information
No response

The problem continues if I try to access via IP:
Reintentar la instalación: Cannot connect to host <IP_ADDRESS>:54236 ssl:True [[SSL: BAD_SIGNATURE] bad signature (_ssl.c:997)]

Bueno, parece que el problema desaparece cuando actualizo Razer Synapse a la última versión.
Con las siguientes versiones, todo funciona actualmente: {"core":"3.28.01","device":"3.28.01","version":"3.28.01"}
Dejo abierta la sugerencia de agregar al README el hecho de que actualicen Razer Synapse a la última versión disponible para que todo funcione correctamente.
PD: Excelente complemento! Tengo muchas ganas de ver cómo la comunidad se une para agregar cosas como efectos y cosas por el estilo.

Well, it seems that the problem disappears when I update Razer Synapse to the latest version.
With the following versions, everything is currently working: {"core":"3.28.01","device":"3.28.01","version":"3.28.01"}
I leave it open for the suggestion to add to the README the fact that they update Razer Synapse to the latest version available so that everything works correctly.
PS: Excellent complement! I'm looking forward to seeing how the community gets on board to add things like effects and the like.

Hello, @PecceG2,
Thanks for your feedback! 👍🏻
That is strange, there are problems with the previous API versions, but everything could happen. I have definitely tested with some older ones but didn't keep track of the numbers.
I will add to the readme info about the possible problems with the older SDK versions


Number pre appended by 0 fails validation

For eg: for US, number<PHONE_NUMBER> passes the validation. But 0555-555-5555 fails the validation. Also 1555-555-5555 passes the validation, because US's dialing code is 1. Is it possible to do something about it.
Removal of pre 0s seems like an easy fix.

@lokeshagrawal83,
Thanks for using libphonenumber!
The input phone number has to be in right dialling format in order to parse the number right. In this case 0 can not be trimmed off as 1 is the national prefix for the USA as you already mentioned.
I'll close this as it's not an issue with the library.

Please consider joining the discussion group if you'd like to discuss this further.


The action does not support lined template

Then the size of arm template bigger than 4MB, the deployment will be failed. The linked template can resolve this issue.
Could this action support linked template?

I did some trying, and found that this action doesn't support linked templates very well.
GitHub workflow can completed successfully, and deployment of ResourceGroup can be created, but Data Factory resources cannot be created.
I believe that this code deletes these resources.

Actually, I have misunderstood this code.
I think that root case is this line just got resources from master arm template, it ignored linked templates.
I will verify my guessing in my current project. If it's correct, I will submit a pr to fix it .


Illegalize Cargonia

About the PR
require people to be original.
Changelog
:cl: moony

tweak: The game rules have been updated. In short, no more revolutions unless you're an antagonist with administrator permission.

The term used for that in SS13 is "nations", which you might want to add instead of directly addressing "cargonia".

The term used for that in SS13 is "nations", which you might want to add instead of directly addressing "cargonia".

I think this works under the context of SS14 where cargonia is all people really do


Copyrights on example media / audio files?

I am considering re-using the audio files included in:
https://github.com/kivy/kivy/tree/master/examples/audio
but I cannot find an explicit copyright or license statement.
Are those audio files released under the same license as the software?
https://github.com/kivy/kivy/blob/master/LICENSE
Also, basically the same questions for any other media files (such as the fruit_images directory full of jpg media).

The license is stated in the main.py.

All the sounds are from the http://woolyss.com/chipmusic-samples.php
"THE FREESOUND PROJECT", Under Creative Commons Sampling Plus 1.0 License.


Rails/ReflectionClassName gives false positive

Expected behavior
It should not mark an error when a string is given
Actual behavior
Describe here what actually happened.
Steps to reproduce the problem
app/models/spotlight/page.rb:13:33: C: Rails/ReflectionClassName: Use a string has value for a class_name.
    belongs_to :last_edited_by, class_name: Spotlight::Engine.config.user_class, optional: true

However Spotlight::Engine.config.user_class is returning a string.
RuboCop version
$ bundle exec rubocop -v
0.64.0

in my case I'm using a Module Builder, something like
module HasOwner
  module With
    def with(klazz_name)
      Module.new do
        extend ActiveSupport::Concern

        included do
          has_many :xyz,
            as: :owner,
            class_name: klazz_name
        end
      end
    end
  end
end

So that it gets used like
class Thing < AR::Base
  include HasOwner.with('Person')
end

NOTE: the real case is more complex and this is a good solution.
Now, 2 things:

klazz_name is not a constant;
I know it's a String;

The only reason we recommend a string literal is because that's the only case that Rubocop can detect. This to me seems like Rubocop overreaching.

The principle being followed is to prefer false positives, since there are an abundance of ways to address those built in. Conversely, there is no way to get at false negatives.


Compatible script for placing directory on non-system drive

When running cmd from the non-system drive in Windows, the default directory is the user directory (system drive C, usually).
Adding a change dir to the script directory allows compatibility for any folder location even off of the system drive.

That sounds great thanks :D


Remove duplicated docker-sync-strategy

Removed the duplicate DOCKER_SYNC_STRATEGY values from env-example that was unnecessarily added in the Thumbor merge via PR #1373
I note the existing PR #1833, but that one removes the original value which is higher up in the file in the general setup area, and thus more visible and appropriately placed for developers tweaking their config, as opposed to this duplicated entry which appears near the middle of the file in the container configuration options, and you'd have to know about it to change it.
I completed the 3 steps below:

[x] I've read the Contribution Guide.
[x] I've updated the documentation. (refer to this for how to do so).