undo/redo returning false

Hi, Scott. I'm using the latest version. In an earlier version I had set up 2 buttons like this:
$('#undo-btn').on("click", function(){
  nSQL().extend("<").then(function(response) {
    console.log(response) //<= If this is true, an undo action was done.  If false, nothing was done.
    if(response) {undoing = true};
  });
})

$('#redo-btn').on("click", function(){
  nSQL().extend(">").then(function(response) {
    console.log(response) //<= If this is true, a redo action was done.  If false, nothing was done.
    if(response) {redoing = true};
  });
})

These were working great prior to updating to the latest. I can confirm that changes have been made to rows in the (only) table in the database. This is logging the correct values on update...
nSQL("aniblocks").on('change',function(eventData) {
  console.log('modified: ',eventData.changedRows);
});

I set a breakpoint on nSQL().extend("<").then(function(response) { and took some screenshots for you to peruse.

I wondered if maybe the default settings were undefined so added this just in front of connect()
.config({
  history: true,
  memory: true
})

the onChange listener now is:
nSQL("aniblocks").on('change',function(eventData) {
  getHistory();
  console.log("type: ",eventData.changeType);
  console.log("id: ",eventData.changedRows[0].id);
  console.log('row: ',eventData.changedRows[0].settings.activeSelectors.options[0]);
});

The getHistory function added is:
function getHistory() {
  nSQL().extend("?").then(function(status) {
    console.log(status);
  });
}

Looks like the history status isn't changing.
Here's the log output after insert and 3 subsequent upserts to the same record.

I just checked and noticed that my json was set to ^0.8.1 so assumed it was pulling in the latest. But, to be sure I changed it to ^0.8.21 and still the same results as above.

Just to confirm, the history is working otherwise except the correct true/false value isn't being returned from the extend promise?

It looks like the history isn't being incremented at all. Note the log of [0,0] from the getHistory() function immediately after an update.

Is this something you're unable to replicate? Need more detail?

I've narrowed down the issue to something causing the enable history trigger to become undefined while the datastore is doing it's setup.  This only seems to happen when the memory store is being used without persistence.
Should have this all patched up soon.

Awesome, Scott. Thanks so much.

Just as a heads-up, I added persistent: true to config and still seeing the same thing. I'm adding a copy of my file so you can see how it's set up, etc in case there's something that triggers a clue as to what might be going on. There are export and import functions in there you can ignore.
state-manager.js.zip

It's interesting because when persistent:true is passed in on the "todo" and "draw" demo on the current version, history works as expected but when it's not passed in history stops working.

Do you see anything in that file that I'm doing wrong? I still get the console logs pasted above. Could something be clearing out the history every time it updates?

Hmmm, I wonder if there's a conflict here. In another area of the app, I'm using Dexie to work with indexedDB mainly because I already had it set up to store and retrieve settings and a few other things. Separate tables, etc.
Earlier, when I said it didn't make a difference I had forgotten that my watch listener was turned off so js wasn't being updated in build. Now I see that, when I add persistent:true to config I get an error:

This is strange. Maybe this will be a clue of some kind as to what I might be doing wrong. I noticed this being added to the root dir of the project...

Just pushed an update that resolved the history issue.
That folder you highlighted is the LevelDB store. :).
Also, I think the unexpected JSON input is a separate issue unrelated to this one.

Thanks so much for the update and for clarifying. The history array is now returning accurately and the database is updating! Awesome.
Question: Should eventData.changedRows[0] always returns the latest point in history? Note the 2 console.log's in my on change function. eventData.changedRows[0] always returns the latest point in history but when I query the db for the changed row, I get the current history point. Below is a screenshot of the console.log's highlighted.
NOTE: Just as a FYI, .settings.activeSelectors.options[0] is the field that's changing I used to test.
nSQL("aniblocks").on('change',function(eventData) {

  getHistory();

  switch(eventData.changeType) {
    case 'modified':
      if(eventData.changedRows.length > 0) {
        console.log('row: ',eventData.changedRows[0].settings.activeSelectors.options[0]);
        if( undoing || redoing ) {
          db.query('select').where(['id','=', eventData.changedRows[0].id]).exec()
          .then(function(result,db){
            console.log(result[0].settings.activeSelectors.options[0]);
            animationsMgr.animateFromDB(result[0]);
            resetDoingVars();
          })
        }
      } else {
        // console.log('nothing changed');
      }
    break;
    case 'deleted':
      console.log('deleted');
    break;
    case 'inserted':
      console.log('inserted');
    break;
  }
});

Since this issue was strictly listed as undo/redo returning false I'm closing this and will move that last question to a new issue. Thanks!


Remapping a shortcut doesn't work and disables original shortcut

Microsoft PowerToys version
0.68.1
Installation method
GitHub
Running as admin
Yes
Area(s) with issue?
General
Steps to reproduce
remapped Control C to Alt C
✔ Expected Behavior
Control C would be Alt C
❌ Actual Behavior
Control C wouldn't work anymore and neither Alt C
Other Software
No response

Do you have other things remapped?
/needinfo

Do you have other things remapped?
/needinfo

No I don't have other things remapped

NVM I'm just stupid, I remapped the other way around.

Glad to hear that the problem was resolved.


Crash after call Director::restart()

cocos2d-js crash after the engine restart, it cause by JS_Init(), is it a bug? Besides, why the byte code files generate by new version mozjs-33.dll is much more larger then before

Can you post the logic of your code and some crash informations here ?

I just call Director::getInstance()->restart(); on the onGLFWKeyCallback, and the call stack is showed below.

This operation is safe before v3.3, but caush in the new version v3.5

What's your onGLFWKeyCallback looks like?

void GLViewImpl::onGLFWKeyCallback(GLFWwindow *window, int key, int scancode, int action, int mods)
{
if (GLFW_REPEAT != action)
{
if (GLFW_RELEASE == action)
{
EventKeyboard::KeyCode code = g_keyCodeMap[key];
if (code == EventKeyboard::KeyCode::KEY_F4)
{
Director *director = Director::getInstance();
director->isPaused() ? director->resume() : director->pause();
}
else if (code == EventKeyboard::KeyCode::KEY_F5)
{
Director::getInstance()->resume();
Director::getInstance()->restart();
}
}
	EventKeyboard event(g_keyCodeMap[key], GLFW_PRESS == action);
	auto dispatcher = Director::getInstance()->getEventDispatcher();
	dispatcher->dispatchEvent(&event);
}

if (GLFW_REPEAT != action)
{
    EventKeyboard event(g_keyCodeMap[key], GLFW_PRESS == action);
    auto dispatcher = Director::getInstance()->getEventDispatcher();
    dispatcher->dispatchEvent(&event);
}

if (GLFW_RELEASE != action && g_keyCodeMap[key] == EventKeyboard::KeyCode::KEY_BACKSPACE)
{
    IMEDispatcher::sharedDispatcher()->dispatchDeleteBackward();
}

}

To solve now? I also see the 3.6.1


Total on Table Chart gets cut off on the Dashboard if the Chart is re-sized

The total count from an aggregated Table Chart gets cut off on the Dashboard, after re-sizing it.
How to reproduce the bug

Access Superset.
Create a Table Chart using the Vehicle Sales Dataset.
Set product_line on the Dimensions.
Set SUM(price_each) on the Metrics.
Enable SHOW TOTALS.
Click on Create Chart.
Access the CUSTOMIZE tab.
Change the SUM(price_each) D3 format to Original value.
Save the Chart and add it to a new Dashboard.
Edit the Dashboard, and decrease the Chart height so that a horizontal scroll bar appears.
Increase the height so that all rows are visible.
Save your changes

Expected results
The Total number should display as expected.
Actual results
The Total number is cut off
(will paste screenshot/video here - Github is being problematic here at the moment)
Known Workarounds
Manually re-sizing the Chart so that the number is properly displayed might fix the issue. Reports are mixed.
Environment
(please complete the following information):

browser type and version: Chrome (on PC)
superset version: latest master

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

This seems to be solved by removing the table-layout: fixed from the "Total" wrapper element. Not sure what other problems doing this may cause, but I can't seem to see any at the moment.


Porting blmc_robot solo12 to odri_control_interface

This is an updated version against the latest version of the odri control interface. I've tested it with the blmc_robots/solo12 demo on the real robot.

Why removing the calibration demo?
This would help people know how to calibrate solo12 using the api.

I felt the demo was not buying much more now that the other solo12 demo does the calibration as well. However, it brings a bit of overhead to port the demo. That's why I decided to delete it.
Do you want me to bring it back?

Why removing the calibration demo?
This would help people know how to calibrate solo12 using the api.

I felt the demo was not buying much more now that the other solo12 demo does the calibration as well. However, it brings a bit of overhead to port the demo. That's why I decided to delete it.
Do you want me to bring it back?

Ok no it makes sense.
You should delete the line in the demos/CMakeLists.txt that compiles it then.
Sorry for this.

Ok no it makes sense.
You should delete the line in the demos/CMakeLists.txt that compiles it then.
Sorry for this.

I will fix the CMakeLists.txt as it seems there is a trick.
Every demo need the path to the yaml file then.

I will fix the CMakeLists.txt as it seems there is a trick.
Every demo need the path to the yaml file then.


Can the repository name support UTF-8 names? i.e. Chinese

Can the project name support Chinese?
Many projects are difficult to translate into English, and many people around us are not very good at English. I hope the project name can support Chinese or UTF-8

Can the project name support Chinese?
Many projects are difficult to translate into English, and many people around us are not very good at English. I hope the project name can support Chinese or UTF-8

I have to say there is some technical problem need to be resolved. First, the repository store struct. Now we use repository name as a directory name, a UTF-8 name may cause some problems on different OS.
If you cannot read English document, it's amazing you could work as a programmer nowadays. So I think it's not reasonable. And you can also use many translation sites to do that.

Duplicate: #13295, which was marked as wontfix

https://github.com/go-gitea/gitea/issues/13295


Add link to an explanation of FEEL

I love README.md files, where I can understand the first sentence without googling (and maybe clicking a link). I was not aware of the FEEL language. Google found a good page, which I directly linked.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[release-0.26] Properly handle events without the data field (#1460)

Cherry-pick of https://github.com/knative-sandbox/eventing-kafka-broker/pull/1460
Release Note

Properly handle events without the `data` field.

Docs

/kind bug

/cherry-pick release-0.25


Fix typo

I encountered this error when building the library with arm-none-eabi-gcc 14.2.0:

Error: bad element type for instruction -- `vstrb.u16 q7,[r3],#8'

I found that the above arm gcc compilation error is caused by this typo. (build failed with MVE assembly code)
I can build successfully with MVE intrinsic code after fixing this typo.
(Related to https://github.com/ARM-software/Arm-2D/discussions/56)

Thank you very much.


[Bug]: import requires access to github

Describe the bug
We are migrating from the old kubectl "gavinbunny" implementation to the newer flux bootstrap method, using terraform 1.6,6 and flux provider 1.3.0
I am using this guide
Our resource  block looks like this (mild redactions):
resource "flux_bootstrap_git" "this" {
  embedded_manifests   = true
  image_pull_secret    = var.pull_secret_name
  path                 = var.target_path
  registry             = var.registry
  registry_credentials = "${var.username}:${var.api_key}"
}

However the terraform import fails for us as it tries to access github.com  which we do not have whitelisted on our proxy. The import t is not respecting the embdedded_manifests flag
Error: Getting expected repository files

could not generate install manifests: failed to download manifests.tar.gz
from
https://github.com/fluxcd/flux2/releases/download/v2.3.0/manifests.tar.gz,
error: Get
https://github.com/fluxcd/flux2/releases/download/v2.3.0/manifests.tar.gz":
Forbidden

Adding the github.com  domain to our proxy is not possible.
Is there anyway to work around this? Is it a bug that can be fixed?
I have a horrible alternative of hand editing the state file but I really do not want to do that.
Steps to reproduce

Have an older Flux installation (or remove the flux_bootstrap_git from existing statefile)
Ensure github.com is not reachable
Define the flux_bootstrap resource to use embedded_manifests
Run terraform import

Expected behavior
The resource should be imported into the statefile
Screenshots and recordings
See logs in description
Terraform and provider versions
Terraform 1.6.6 on linux_amd64
Flux provider 1.3.0
Terraform provider configurations
provider "flux" {
kubernetes = {
cluster_ca_certificate = base64decode(data.aws_eks_cluster.cluster.certificate_authority.0.data)
exec = {
api_version = local.cluster_config["aws_cli_client_authentication_api_version"]
args        = local.aws_eks_get_token_command
command     = "aws"
}
host = data.aws_eks_cluster.cluster.endpoint
}
git = {
author_name = var.azuredevops_secret_username_flux_config
branch      = local.cluster_config["azuredevops_git_repo_branch_flux_config"]
http = {
username = var.azuredevops_secret_username_flux_config
password = var.azuredevops_personal_access_token
}
url = "${local.cluster_config["azuredevops_org_service_url"]}/${local.cluster_config["azuredevops_project_name"]}/_git/${local.cluster_config["azuredevops_git_repository_name_flux_config"]}"
}
}
flux_bootstrap_git resource
resource "flux_bootstrap_git" "this" {
embedded_manifests   = true
image_pull_secret    = var.pull_secret_name
path                 = var.target_path
registry             = var.registry
registry_credentials = "${var.username}:${var.api_key}"
}
Flux version
flux: v2.3.0
Additional context
A clean install works fine - the issue is with the import not respecting the embedded_manifests property in the resource definition.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Would you like to implement a fix?
None

See https://registry.terraform.io/providers/fluxcd/flux/latest/docs/resources/bootstrap_git#embedded_manifests

@stefanprodan  Yes, this is the problem!!!
I already do use embedded_manifests  in the resource definition and that works perfectly if Flux is not installed or was installed by previously by the Flux provider's "flux_bootstrap" resource. BUT if Flux was installed by other means (i.e. an older version of the provider where the bootstrap resource was not available) then the migration guide says to use "terraform import" to import the existing configuration into the state file. THIS is the step that is broken.  If I use "terraform import", embedded_manifests is not respected even though it is specified in the resource definition.
This is the crux of the bug report - terraform import of a flux_bootstrap resource needs to respect embedded_manifests which it currently does not do.

The import function can't use the embedded manifests, because those manifests are for the latest version, while the cluster is on an older version.

Hi @stefanprodan , are you saying there’s no migration path if we use embedded manifests? I.e. The import function used either as an import block or CLI tf import won’t work?
if so, any recommendations how to best achieve the migration please?


Dataset selection proposal

I have found two suitable datasets.

Predict Student performance https://archive-beta.ics.uci.edu/ml/datasets/student+performance
Classify origin of wines https://archive-beta.ics.uci.edu/ml/datasets/wine

Reasons: Both of the dataset are relatively clean and easy to work with. The description pages linked above are worth reading (very clear and straight to the point).

Merged into another issue


A function to calculate the gradient from the given positions.

Is there a function to generate a new gradient from a given position. For example:
var gradient = tinygradient([
  '#ff0000',
  '#00ff00',
  '#0000ff'
]);
gradient.getGradient(0.3, 0.8); // new tinygradient

which returns a new tinygradient instance where the gradient starts a 0 and ends at 1 but for the gradient values between 0.3 and 0.8 of the first gradient.

With rgbAt/hsvAt you can get the color at 0.3 and 0.8 and work from here.

Ah, true dat. Thanks.

But actually if the range has a stop at 0.5 that solution isn't going to pass through that stop colour.

But actually if the range has a stop at 0.5 that solution isn't going to pass through that stop colour.

Can you explain better the problem ?


Incorrect Link

In the “template” section, the link of the last sentence is not working.
The problem is that the link refers to a tenant.
Affected sentence:

You can refer to this ARM template example for more context.

Link:

https://paste.microsoft.com/f84d2f83-f6bf-4d24-aa03-175b0c43da32

Error:

The selected user account does not exist in the “Microsoft” tenant and cannot access the “611744cc-cdd0-4d9b-8d87-830df49f7f6b” application (pastebinmsft) in this tenant. The account must first be added to the tenant as an external user. Use another account.

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

ID: 8b1000e8-404c-1173-9b25-a373f8b2770b
Version Independent ID: 197cb6cf-ed51-dec9-2e41-b71c08c960d0
Content: Get high availability and cost savings with Spot Priority Mix for Virtual Machine Scale Sets - Azure Virtual Machine Scale Sets
Content Source: articles/virtual-machine-scale-sets/spot-priority-mix.md
Service: virtual-machine-scale-sets
Sub-service: spot
GitHub Login: @ju-shim
Microsoft Alias: jushiman

@kenoc1
Thanks for the feedback! We will check and update the document as appropriate.

Hi @kenoc1
I tried to open the link. I can able to open it.

if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.

@kenoc1
We are going to close this thread as resolved but if there are any further questions regarding the documentation, please tag me in your reply and we will be happy to continue the conversation.


Emoji should be removed from github.com anchor

macOS 10.12
VS Code 1.14.2
Markdown TOC 1.5.6 (2017-07-25)

Test case:
<!-- TOC updateOnSave:true -->

* [🚲 Hello 🚲](#🚲-hello-🚲)

<!-- /TOC -->

# 🚲 Hello 🚲

...

The generated anchor links to #🚲-hello-🚲, but on GitHub the actual anchor is #-hello-.

This seems to have been fixed in 0.5.7. Should also be autofixed as the semVer range defined in this package is ^0.5.6. Since the fix was provided ~20 days ago in the dependency I assume you (like me) have a version that's older than 20 days. A simple reinstall on the extension page however doesn't do the trick for me to get the dependencies upgraded. Might be a package-lockor similar. Just head over to the directory and force an update
cd ~/.vscode/extensions/AlanWalk.markdown-toc-*
npm i<EMAIL_ADDRESS># restart code completely

You can confirm the dependency version (before or afterwards):
cat ~/.vscode/extensions/AlanWalk.markdown-toc-*/node_modules/anchor-markdown-header/package.json| grep version

This can be closed once 1.6.0 is released.


[Release] sdk/resourcemanager/elastic/armelastic/0.2.0

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run prepare-pipelines

/azp run prepare-pipelines

/azp run prepare-pipelines


optional color coding for ranking changes

as in the original lineup. green up, red down

solved with animation / multi branch of the lineup engine


WIP Add python library reference to the TOC

Docs Build status updates of commit fc0e105:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

api/toc.yml
:white_check_mark:Succeeded
View (qsharp-preview)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Adding custom options when fetching remote images

Could you add a way to add custom option to the HTTP requests made when downloading remote images?
I have some images hosted on internal servers. To fetch these, custom headers must be set.
As far as I can tell, there is no way to provide these headers - apart from monkeypatching shenanigans.
I figure the most versatile implementation would be to enable remoteOptimizedImages.js to return these options with each URL, since they might not be the same for all remote images.

Hi @mmarseu,
Let me think about this. In general, the remote image feature needs more work and I will have a look how this could be implemented.

I just enabled the functionality of query params in remote images. Would that be enough for your use case?

It's always great to see progress but unfortunately no. In my case, I really need to set custom headers for the image request.

Hi @mmarseu,
I created a PR with a proposal for this functionality: #223
If you have time, you could check it out and give your feedback.
Thanks,
Niels


Addition of graphs in ETmodel

Closing of Issue #3935
Goes together with PR #2966 of Etsource

I rebased the branch for this PR, @mabijkerk could you check wether it can be closed?

See my comment in the linked ETSource PR.


domain_unique_error

I cannot add my domain to CE because of the domain_unique_error. This is happening only if I register my site in the cloud beforehand.
The issue is located in the file site.ex lib/plausible on line 88.

Moved to https://github.com/plausible/analytics/issues/4875


Add Ethereum name or address to package.json

Feature Request

Summary

I would like to see an Ethereum name or address added to the ethereum field of the package.json file. This will allow consumers of the project to know a cannonical address to send funds to. For example, I'm building an application that will allow developers to automatically donate ETH to all of their npm dependencies that have Ethereum addresses in their package.json files: https://sustainus.io (not quite ready, alpha soon)

This is a cool idea. I think it'd be nicer if the field was a bit more standardized tho. For example, there could be a donations: {} field with different options that people can choose from.


Time picker format

Description of the problem
While setting time, if the hour is 0, then it should convert to 12 AM
Describe the algorithm of actions that leads to the problem
call set_time(datetime.time(hour=0, minute=22))
Reproducing the problem
my_time = MDTimePicker()
my_time.set_time(datetime.time(hour=0, minute=20))

Screenshots of the problem

Description of Changes
Use of standard library for conversion
Screenshots of the solution to the problem

Code for testing new changes
my_time = MDTimePicker()
my_time.set_time(datetime.time(hour=0, minute=20))

The PR is deprecated due to the new API of the KivyMD library 2.0.0 version


Enable ability to read stream from delta table without duplicates. Only blind commits.

Description

Add an appendOnly:Boolean option to the stream source. When the option applies stream source will skip all commits, except blind Append commits.
To understand motivation, please refer to the motivation part from issue #1490.

Resolves #1490
How was this patch tested?
There is a unit test for this patch.
And also similar changes work in production for the last 2 weeks.
Does this PR introduce any user-facing changes?
Yes, it's introducing a new option for the delta Structure Stream.

Let's discuss on #1490 the use case of these changes

Should be covered by https://github.com/delta-io/delta/pull/1616, feel free to reopen if not.


add input type fastqs to RNA seq

Currently RNA-seq workflows accepts unaligned bams as the only input type.
Add support to accept either fastqs or unaligned bams.

superceded by https://github.com/genome/analysis-workflows/issues/970


StatePac properties in the augmented Swagger modules do not contain the enum values

#79 Found in the 1.0.0-RC1
TapiTopology.swagger
    "AdminStatePac": {
        "description": "Provides state attributes that are applicable to an entity that can be administered. Such an entity also has operational and lifecycle aspects.",
        "properties": {
            "administrativeState": {
                "type": "string",
                "description": "none"
            },
            "operationalState": {
                "type": "string",
                "description": "none"
            },
            "lifecycleState": {
                "type": "string",
                "description": "none"
            }
        }
    },

Tapi.swagger
    "AdminStatePac": {
        "description": "Provides state attributes that are applicable to an entity that can be administered. Such an entity also has operational and lifecycle aspects.",
        "properties": {
            "administrativeState": {
                "enum": [
                    "LOCKED",
                    "UNLOCKED"
                ],
                "type": "string",
                "description": "none"
            },
            "operationalState": {
                "enum": [
                    "DISABLED",
                    "ENABLED"
                ],
                "type": "string",
                "description": "none"
            },
            "lifecycleState": {
                "enum": [
                    "PLANNED",
                    "POTENTIAL",
                    "INSTALLED",
                    "PENDING_REMOVAL"
                ],
                "type": "string",
                "description": "none"
            }
        }
    },

This issue is found in AdminsStatePac, OperationalStatePac and LifecycleStatePac classes. These bugs appear in TapiTopology.swagger, TapiConnectivity.swagger, TapiPathComputation.swagger, TapiVirtualNetwork.swagger and TapiNotification.swagger.

Fixed in #82


Day manipulation is broken

Day manipulation (add and subtract) methods don't work properly after extension. They reset the date to 1970-01-01.
I've created a sandbox to reproduce the issue here.
Version: 2.2.2

This issue is fixed but there is another problem in UTC mode when the day is changed. The timezone returns after day manipulation.


Gas Production Layout

Organization of Gas Production Layout

RRC_gas_ledger_tuples.xlsx
Adding first pass .xlsx from @bpmcdowell11

The initial layout is now coded and ready for testing and for input to working files and notebooks.


Detectron2 build for ppc64le with Pytorch 1.14

Hello,
@hartb please include Detectron2 on the powerai channel. Please find bellow instruction to get this build with Pytorch 1.14
conda config --prepend channels https://public.dhe.ibm.com/ibmdl/export/pub/software/server/ibm-ai/conda-early-access/ conda install pytorch=1.4.0=23447.g18a1a27 cudatoolkit-dev gxx_linux-ppc64le=7 gcc_linux-ppc64le=7
git clone https://github.com/facebookresearch/detectron2.git python -m pip wheel -e detectron2
Thank you,
FM

Working on a conda recipe for this.

I'm afraid I won't be able to finish this recipe. If others are interested in picking it up, please feel free.


Defer generic indexed access when obtaining contextual type within inferential check

I mentioned what currently happens here
fixes https://github.com/microsoft/TypeScript/issues/52575
cc @ahejlsberg

This PR got superseded by https://github.com/microsoft/TypeScript/pull/52611


Error when using setStringValue function

I am using the setStringValue function in my project, but it is returning an error: "Type 'undefined' is not assignable to type 'string'."
let data = ''
data = setStringValue('data', '')

According to the function signature, this should be valid input and should return an empty string as the defaultValue is provided. However, I am getting an error instead.

@4a390141 Fixed, close this issue.


leaflet.locatecontrol is not a module

I installed the leaflet.locatecontrol via npm;
I tried to import to my component at first and it gave an error of 'cannot find module'
Then i installed the types definition, npm install --save @types/leaflet.locatecontrol
Now it says leaflet.locatecontrol is not a module
Can you please help me out?

Looking at https://github.com/DefinitelyTyped/DefinitelyTyped/tree/master/types/leaflet.locatecontrol, I think you want to import locate or use L.control.locate. I can't really help with your specific setup, sorry.


Help: Add a chapter for the bc command

Please fill out this template
Add a chapter for the bc command:
https://github.com/bobbyiliev/101-linux-commands-ebook/blob/main/ebook/en/content/009-the-bc-command.md
Additional context
Make sure to check out the Guidelines.md file on how to contribute!

Hi @bobbyiliev, I would like to work on this. Thanks!


chore: cleanup percent & amount helpers

Codecov Report

:exclamation: No coverage uploaded for pull request base (chore/switch-to-pnpm@df3ad39). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@                   Coverage Diff                   @@
##             chore/switch-to-pnpm      #96   +/-   ##
=======================================================
  Coverage                        ?   64.69%
=======================================================
  Files                           ?       16
  Lines                           ?      422
  Branches                        ?      162
=======================================================
  Hits                            ?      273
  Misses                          ?      149
  Partials                        ?        0

Flag
Coverage Δ

unittests
64.69% <0.00%> (?)

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Update issues.

Hey there,
First of all, thanks a lot for that great idea ! I got some troubles with the plugin for a couple of weeks.
Sometimes, when I click on a placard, the site updates, but most of the time, it doesn't. I tried to uninstall and install back the plugin, click and unclick setting boxes, but the problem is still here.
At the moment, I tried to click on Medium available houses of my home server, Louisoix - Chaos - EU. And if you need any more informations, I'll be happy to help.

It is possible your computer clock may be out of sync. PaissaDB rejects updates from computers who are more than a minute out of sync from the server time.
If the problem persists after ensuring your system clock sync, please send me your dalamud log file (%appdata%\XIVLauncher\dalamud.log) after clicking a placard (email is fine).

Thank you so much, I forced sync the clock and everything works again !


Add 2048 game

Describe the project you want to add with tech stack
I wanted to add 2048 game. The tech stack used are HTML, CSS, JavaScript.

@MohitGupta121  please assign me this issue under gssoc'23

Sorry, I didn't observe that this project was already done. I am closing this issue.


feat: add top level Application.context

This PR adds an Application.context attribute.  It can serve as a natural place for applications to put a top level context (and, as usual, they can spawn children contexts with context.new_child).
it does not yet hook up context.changed signals to anything in the Qt backend.  mostly because I think more consideration of context "scoping" is necessary.  For example, it's not obvious that app.context is the "correct" context to pass to one of the update_from_context methods... it might be a more narrowly scoped child context provided by the user.

cc @lucyleeow, this is (in-part) addressing https://github.com/pyapp-kit/app-model/pull/142#pullrequestreview-1712283260


change default keyboard to.alphabetical

make it a setting to try different keyboards

Now 3 keyboard variations:
Alphabetical
Constants / Vowels
QWERTY
the first 2 remove the enter key in preparation of timer feature


fix(sasjs-create): install dependencies when creating minimal app

Previously we did not have a package.json in the minimal-seed-app project, so there was no need to run an npm install when creating an app using the minimal template.
Since that has been added, we now need to run an install step when creating an app.
This change removes the third parameter to createApp which is an installDependencies flag that is true by default.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add SharedWorker to the library

Details in https://developer.mozilla.org/en-US/docs/Web/API/SharedWorker
This should work
var myWorker = new SharedWorker('worker.js');

@mhegazy what about service worker?

Can you file a new one for these? we need a new lib.serviceworker.d.ts as well.

For some reason, SharedWorker exists in the DOM lib, but not in WebWorker lib.


[wip][sqllab] more robust copy to clipboard

This PR

Closes #6140 by implementing a more robust copy to clip board for SQL lab share query links
Removes the shared query link from the SQL lab menu dropdown (so, share query is only available in the main SQL lab editor)

Fixes share query offset in explore view. This had the wrong offset because we render once without the url, then render again with the url, but the react-bootstrap popover was not updating its position on the 2nd render

before

after

@kristw @michellethomas @graceguo-supercat @mistercrunch

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6180   +/-   ##
=======================================
  Coverage   76.91%   76.91%
=======================================
  Files          47       47
  Lines        9362     9362
=======================================
  Hits         7201     7201
  Misses       2161     2161

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 6ec9e1a...1127de5. Read the comment docs.


health check can be simpler

the check() method always return 0

Codecov Report

Merging #2642 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #2642      +/-   ##
============================================
+ Coverage     50.23%   50.24%   +0.01%
  Complexity     2061     2061
============================================
  Files           413      413
  Lines         12647    12643       -4
  Branches       1294     1292       -2
============================================
  Hits           6353     6353
+ Misses         5845     5843       -2
+ Partials        449      447       -2

Impacted Files
Coverage Δ
Complexity Δ

...lo/configservice/ConfigServiceHealthIndicator.java
100% <100%> (+20%)
3 <1> (ø)
:arrow_down:

...ollo/adminservice/AdminServiceHealthIndicator.java
100% <100%> (+20%)
3 <1> (ø)
: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 f050e44...d8be1e3. Read the comment docs.

Coverage increased (+0.001%) to 53.785% when pulling d8be1e3fcfbe19a070f8b4ebdaed1794ed8d3efb on sunshanpeng:master into f050e44d83264fb728cc0702c8c81c8b0fc674c6 on ctripcorp:master.


Fixes #130

Fixes #130

Hold off on this for a minute. I need to add the border radius and shadow to the listing card too, in order to close #130

Ok should be ready for review

Ok this is ready for another look. I had to add clrSh2 to a few places because the shadows went away. Let me know if I added them incorrectly.

I didn't realize the shadows were missing, those definitely need to be there, since they're a "color" class that is set by the theme.

Cool, I added them in the last push.


Add support for HTTP get method with request body

There some applications using HTTP get request with request body , such as  elasticSearch Request Body Search
Please discuss to support this case or not. Postwoman at now is not support GET request with HTTP request body.
reference

stackoverflow: HTTP GET with request body
stackoverflow: Is this statement correct? HTTP GET method always has no message body

We're using Axios for request transactions and I guess that's not possible with Axios. see https://github.com/axios/axios/issues/2400
And as mentioned in elasticSearch Request Body Search, we can use query parameters with GET request to send data as parameters. Kindly use Parameters section for that.

Well, we could implement it if needed. But it is not a supported behavior according to the HTTP spec. So we don't really want to support it in this app as well.
Here is the spec reference defining that https://tools.ietf.org/html/rfc7231#page-24


add HEAD Mehthod to getObjects route

What kind of change does this PR introduce?
Add a HEAD Method to getObject route so clients can get headers without requesting whole object.
What is the current behavior?
I get a 400 which is ok but a 405 would be better 😂
What is the new behavior?
A HEAD request just like the GET but without the body (see https://github.com/fastify/fastify/pull/2700/)

Hi @buschco, I am worried that people might do a HEAD request and assume that an object exists at that route when it doesn't. For example /bucket/memes/cat-doesnt-exist.jpg will return a 200 which is confusing. If we are adding a HEAD request for getObject, I would rather only return a 200 if the object exists and the user has permissions to download it.

@inian
is this in the behavior of HEAD request in Fastify (or in general) returning 200 even if the GET would result in 4xx?
I could not find evidence for such behavior in the Fastify PR, am I missing something?

Ah I just checked and it does look like Fastify returns the same status code as the GET endpoint rather than returning 200 always which is good.
Can you help check if the docs exported also have the HEAD method documented @buschco?

Unfortunately the HEAD route is not in the generated static/api.json 😕
We could raise an inssue in fastify/swagger or keep it a hidden feature.

Okay, I will merge the PR in. Can you open an issue in fastify/swagger and an issue in our repo so that we remember to add support this in our docs (or ensure this is fixed) after fastify adds support for it? Till then it can be a hidden feature ;)

I raised the issue -> https://github.com/fastify/fastify-swagger/issues/648
thank you 🙂

hmm the tests are failing for some reason. will merge once I debug why thats happening


Change a notation of props from camel to hypen

Would be great with keeping standards of attr, ptSize to size or size-point, point-of-size
any thought @importre

agree with you ;)


Complete README w/ usage examples, etc.

[x] motivation, interoperability notes (methods, JSON)
[x] reference link to https://github.com/aphyr/meangirls
[x] usage examples, for both counters and collections

Decided to use mochawesome to generate Documentation, for now.


Error 0xc0000005

report"0xc0000005"when I first start the NorthstarLauncher. What happen?

Try verifying files through steam or origin, if that does not work try reinstalling Northstar

report"0xc0000005"when I first start the NorthstarLauncher. What happen?

Guessing you on windows 7 x64 too?
The problem seems unsolvable, the only thing I haven't tried yet is to reinstall Microsoft Redistributables

maybe find the problem, but don't know how to fix it
the crash data have mention the Microsoft .NET Framework1.1 but I already uninstall it. Does anyone know how to fix it?

Guessing you on windows 7 x64 too?

Confirmed on Windows 11 x64, as well.

Try verifying files through steam or origin, if that does not work try reinstalling Northstar

I tried this, but this did not fix the issue.

Can confirm after testing it myself. I'm assuming one of the dependencies is using some framework that's not compatible with other windows OS's besides 10. The modified "unpacked" application and launcher don't seem to work correctly on windows 7, 8 or 11 (tested) I also have all my redists on each fully updated.
All produce the "error 0xc0000005" and setting the modified game client application to run in compatibility mode for anything causes a different error from the launcher "check if bob shipped a debug build by accident".

This is not strictly true, I got the server to work on Windows Server 2012 R2 (nice name similarity) just fine and that’s based on win8

the solution is unfortunately disabling ASLR.
the alternative to this is injecting northstar.dll manually into titanfall2.exe before the game window comes up.

Disabling ASLR is not working for me , how do you inject manually?

the solution is unfortunately disabling ASLR. the alternative to this is injecting northstar.dll manually into titanfall2.exe before the game window comes up.

Thank you endlessly, injecting is a bit tricky since you have to do it very fast, before the window shows up, but it actually worked for me on win 7 x64!

the solution is unfortunately disabling ASLR. the alternative to this is injecting northstar.dll manually into titanfall2.exe before the game window comes up.

Thank you endlessly, injecting is a bit tricky since you have to do it very fast, before the window shows up, but it actually worked for me on win 7 x64!

the injection window is between a very small moment after process spawn and game window coming up, for both dedicated and game, since you're running vanilla through launcher this prevents multiple instances and therefore running dedicated and the game at once.
this bypasses #81 as well

" injecting northstar.dll manually into titanfall2.exe before the game window comes up."
How can I do it??

" injecting northstar.dll manually into titanfall2.exe before the game window comes up." How can I do it??

get extreme injector, select titanfall2 process with the game running, close it, drag northstar.dll into the list. run titanfall 2 normally and inject before the game window comes up, should work. this is a workaround mind you, every time you want northstar you need to do the same thing for now. a fix is being worked on

https://i.imgur.com/Zl8p0zA.png
win11 21H2 22000.376

The modified "unpacked" application and launcher don't seem to work correctly on windows 7, 8 or 11 (tested) I also have all my redists on each fully updated.

Not strictly true for 8. I'm (one of the few maniacs still) running Windows 8.1 and I never experienced that error.

The modified "unpacked" application and launcher don't seem to work correctly on windows 7, 8 or 11 (tested) I also have all my redists on each fully updated.

Not strictly true for 8. I'm (one of the few maniacs still) running Windows 8.1 and I never experienced that error.

The modified "unpacked" application and launcher don't seem to work correctly on windows 7, 8 or 11 (tested) I also have all my redists on each fully updated.

Not strictly true for 8. I'm (one of the few maniacs still) running Windows 8.1 and I never experienced that error.

regardless, the exe is a dump not something proper, this is getting ditched in favor of a better launcher see p0358 pr and discord thread

The modified "unpacked" application and launcher don't seem to work correctly on windows 7, 8 or 11 (tested) I also have all my redists on each fully updated.

Not strictly true for 8. I'm (one of the few maniacs still) running Windows 8.1 and I never experienced that error.

regardless, the exe is a dump not something proper, this is getting ditched in favor of a better launcher see p0358 pr and discord thread

So is a better alternative coming?

So is a better alternative coming?

@manotroll, it's still WIP, watch this pull request if you're curious.

@manotroll, it's still WIP, watch this pull request if you're curious.

Should be completely fixed in v1.4.0 now

Let us know if it happens on newer releases


policycoreutils: fix i686-linux compilation error

This adds a patch to quiet a compiler warning which would be harmless except that it breaks the build due to use of -Werror. See http://hydra.nixos.org/build/25151888/nixlog/1

cc @Phreedom

No.If you give %lu to printf, the corresponding argument should always be an unsigned long. Passing a size_t only happens to trigger a warning on i686-linux, but the patch makes better code regardless of platform.

I just changed the patch to what I think is a slightly better one. Instead of casting the argument to unsigned long, leave it a size_t, and change the format argument from %lu to %zu.

tnx!


Add attribute value checks

This PR add two new checks:

Y015: Top-level attribute must not have a default value. (Closes: #21)
Y016: Instance attribute must not have a default value other than "...".

When using type comments, this will only disallow ast.Num and ast.Str nodes to ensure that aliasing still works. When using variable annotations it will disallow all values (except ellipses for instance variables).

Unfortunately, pyflakes has a problem with top-level fields without default value (PyCQA/pyflakes#486). Therefore I am going to merge Y016 and Y015, allow the ellipsis everywhere, but add Y092 for the ellipsis on top-level.

python/typeshed#3411 for necessary fixes to typeshed.

CI should now pass.

@ambv @JelleZijlstra


Response body writer example

Echo's request body and uses middleware to add SHA256 of the response body

CI still failing


Not running code

Hello I am running my code but nothing is shown in the console please can you tell me what is the issue?

Please fix this!

can you tell me what is the issue?

Likely not without more information. What is your code? What do you expect to be printed in the console?


mock().using does not work on method nested inside class property [@automock]

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
We are encountering difficulties when attempting to mock methods that are defined inside existing class properties. The nested property is automatically overwritten with a mocked object, so the actual method inside the property we need to mock is undefined and cannot be accessed (and cannot be mocked, basically).
Steps to reproduce

Define a class with a property that contains a method. e.g.:

class OtherClass {
   public  getSomething: () => string
}

class MyClass {
    readonly property: OtherClass
}

. Attempt to  mock MyClass

let myClass:  jest.Mocked<MyClass>;

const { unit, unitRef } = TestBed.create(ServiceUnderTest)
      .mock(MyClass)
      .using({
         property: {
               getSomething: jest.fn()
          }
       })
 .compile();

myClass = unitRef.get(MyClass)

In the actual test try to mock resolved value getSomething, e.g:

myClass.property.getSomething.mockResolvedValue('mocked')
Expected behavior
The method inside the class property should be successfully mocked. In the above example  trying to do
.getSomething.mockResolvedValue Should not throw an error - getSomething is undefined
Automock version
v2.1.0

Hello @tsahnar,
Please try the following solution:
let myClass:  jest.Mocked<MyClass>;

const propMock = { property: jest.fn() };

const { unit, unitRef } = TestBed.create(ServiceUnderTest)
      .mock(MyClass)
      .using(propMock)
      .compile();

myClass = unitRef.get(MyClass)

You can control the mock (property stub) from propMock const.


Add a new blog post presenting raw image streaming

This blog post details a new feature that was recently added in
Metal3, and how to leverage it for memory constrained deployments.

/assign @knowncitizen
/cc @russellb
/cc @dhellmann
/cc @hardys

Brief description on raw image streaming with examples. Good writing, understandable and well done.

Nice!
/approve
/lgtm


Feat logger

Information

Type
Breaking change

Feat/Chore
No

Description
Logger can be injected to another service.
Usage example
import {Logger} from "@tsed/logger";
import {Injectable} from "@tsed/di";

@Injectable()
export class MyService {
   @Inject()
   logger: Logger;
}

Todos

[x] Tests
[x] Coverage
[x] Example
[x] Documentation

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

v6.5.0
GitHub release

Your semantic-release bot :package::rocket:


[Sage 9] Backslashes in css assets urls on windows

Submit a feature request or bug report

[ ] This is a feature request
[x] This is a bug report
[x] This request isn't a duplicate of an existing issue
[x] I've read the docs and NPM Debugging Guidelines post and followed them (if applicable)
[x] This is not a personal support request that should be posted on the Roots Discourse forums

Replace any X with your information.

What is the current behavior?`
npm run build set images and fonts urls in css with backslashes instead of slashes and it looks like
/wp-content/themes/sage/dist\styles\vendor\fontawesome-webfont.eot
May be it is because I am doing it on Windows?
What is the expected or desired behavior?
Doing normal slashes

Bug report
(delete this section if not applicable)
Please provide steps to reproduce, including full log output:
npm install font-awesome
main.css
`@import '~font-awesome/scss/font-awesome'
`npm run build```
Please describe your local environment:
WordPress version: 4.6
Windows 10
NPM/Node version: 3.10.3 / v6.3.1
Where did the bug happen? Development or remote servers?
Development
X

You shouldn't have to npm install font-awesome with the latest version of Sage 9. Sage 9 already comes with font-awesome.
But for good measure, I just tested this on Windows.
@font-face {
  font-family: 'FontAwesome';
  src: url(/app/themes/sage/dist/vendor/fontawesome-webfont_25a32416.eot);
  src: url(/app/themes/sage/dist/vendor/fontawesome-webfont_25a32416.eot?#iefix&v=4.6.3) format("embedded-opentype"), url(/app/themes/sage/dist/vendor/fontawesome-webfont_e6cf7c6e.woff2) format("woff2"), url(/app/themes/sage/dist/vendor/fontawesome-webfont_c8ddf1e5.woff) format("woff"), url(/app/themes/sage/dist/vendor/fontawesome-webfont_1dc35d25.ttf) format("truetype"), url(/app/themes/sage/dist/vendor/fontawesome-webfont_d7c63908.svg#fontawesomeregular) format("svg");
  font-weight: normal;
  font-style: normal;
}

Looks good to me. So I'm guessing one of the following 3 scenarios is the case here:

You're using an older version of Sage 9, not the latest version.
I've already fixed this locally, in which case it will be pushed within a day or so.
This is an outstanding bug, and my quick test just now simply failed to reproduce it because you and I are doing things differently.

Thoughts?

I `ve downloaded it yesterday, so there is no newer commits.  And yep, sorry for this, I use preinstalled version of fontawesome. btw, same situation with Roboto font I defined in scss
In assets/config.json i have this
"publicPath": "/wp-content/themes/sage",
"devUrl": "http://site.local"

Then i run npm run build and fontawesome fonts now in dist/vendor
But when i look in main scss there is
font-face {
  font-family: 'FontAwesome';
  src: url(\wp-content\themes\sage\dist\vendor/fontawesome-webfont.eot);
  src: url(\wp-content\themes\sage\dist\vendor/fontawesome-webfont.eot) format("embedded-opentype"), url(\wp-content\themes\sage\dist\vendor/fontawesome-webfont.woff2) format("woff2"), url(\wp-content\themes\sage\dist\vendor/fontawesome-webfont.woff) format("woff"), url(\wp-content\themes\sage\dist\vendor/fontawesome-webfont.ttf) format("truetype"), url(\wp-content\themes\sage\dist\vendor/fontawesome-webfont.svg) format("svg");
  font-weight: normal;
  font-style: normal;
}

But browser trying to fetch realy strange url
http://site.local/wp-content/themes/sage/dist/styles/wp-contentthemessageistvendor/fontawesome-webfont_c8ddf1e5.woff
By the way there is my webpack.config.js
/ External dependencies
var webpack = require('webpack'),
    path = require('path'),
    argv = require('minimist')(process.argv.slice(2)),
    qs = require('qs'),
    autoprefixer = require('autoprefixer'),
    Clean = require("clean-webpack-plugin"),
    AssetsPlugin = require('assets-webpack-plugin'),
    ExtractTextPlugin = require('extract-text-webpack-plugin'),
    OptimizeCssAssetsPlugin = require('optimize-css-assets-webpack-plugin'),
    cssnano = require('cssnano');

// Internal dependencies
var config = require('./assets/config');

// Internal variables
var scriptsFilename = (argv.release) ? 'scripts/[name]_[hash].js' : 'scripts/[name].js',
    stylesFilename = (argv.release) ? 'styles/[name]_[hash].css' : 'styles/[name].css',
    sourceMapQueryStr = (argv.release) ? '-sourceMap' : '+sourceMap',
    assets = 'assets/',
    dist = 'dist/',
    jsLoader,
    webpackConfig;

jsLoader = {
  test: /\.js$/,
  exclude: [ /(node_modules|bower_components)(?![/|\\](bootstrap|foundation-sites))/ ],
  loaders: [ 'babel?presets[]='+path.resolve('./node_modules/babel-preset-es2015')+'&cacheDirectory' ]
};

if (argv.watch) { // '--watch' to add monkey-hot
  jsLoader.loaders.unshift('monkey-hot');
}

/**
 * Process AssetsPlugin output and format it
 * for Sage: {"[name].[ext]":"[name]_[hash].[ext]"}
 * @param  {Object} assets passed by processOutput
 * @return {String}        JSON
 */
var assetsPluginProcessOutput = function (assets) {
  var name,
      ext,
      filename,
      results = {};

  for (name in assets) {
    if (assets.hasOwnProperty(name)) {
      for (ext in assets[name]) {
        if (assets[name].hasOwnProperty(ext)) {
          filename = name + '.' + ext;
          results[filename] = path.basename(assets[name][ext]);
        }
      }
    }
  }
  return JSON.stringify(results);
};

/**
 * Loop through webpack entry
 * and add the hot middleware
 * @param  {Object} entry webpack entry
 * @return {Object}       entry with hot middleware
 */
var addHotMiddleware = function (entry) {
  var name,
      results = {},
      hotMiddlewareScript = 'webpack-hot-middleware/client?' + qs.stringify({
        timeout: 20000,
        reload: true
      });

  for (name in entry) {
    if (entry.hasOwnProperty(name)) {
      if (entry[name] instanceof Array !== true) {
        results[name] = [entry[name]];
      } else {
        results[name] = entry[name].slice(0);
      }
      results[name].push(hotMiddlewareScript);
    }
  }
  return results;
};

webpackConfig = {
  context: path.resolve(assets),
  entry: config.entry,
  output: {
    path: path.join(__dirname, dist),
    publicPath: path.join(config.publicPath, dist),
    filename: scriptsFilename
  },
  module: {
    preLoaders: [
      {
        test: /\.js?$/,
        include: path.resolve(assets),
        loader: 'eslint'
      }
    ],
    loaders: [
      jsLoader,
      {
        test: /\.css$/,
        include: path.resolve(assets),
        loader: ExtractTextPlugin.extract('style', [
          'css?' + sourceMapQueryStr,
          'postcss'
        ])
      },
      {
        test: /\.scss$/,
        include: path.resolve(assets),
        loader: ExtractTextPlugin.extract('style', [
          'css?' + sourceMapQueryStr,
          'postcss',
          'resolve-url?' + sourceMapQueryStr,
          'sass?' + sourceMapQueryStr
        ])
      },
      {
        test: /\.(png|jpg|jpeg|gif|svg)(\?.*)?$/,
        include: path.resolve(assets),
        loaders: [
          'file?' + qs.stringify({
            name: '[path][name].[ext]'
          }),
          'image-webpack?' + JSON.stringify({
            bypassOnDebug: true,
            progressive: true,
            optimizationLevel: 7,
            interlaced: true,
            pngquant: {
              quality: "65-90",
              speed: 4
            },
            svgo: {
              removeUnknownsAndDefaults: false,
              cleanupIDs: false
            }
          })
        ]
      },
      {
        test: /\.(ttf|eot)(\?.*)?$/,
        include: path.resolve(assets),
        loader: 'file?' + qs.stringify({
          name: '[path][name]_[md5:hash:hex:8].[ext]'
        })
      },
      {
        test: /\.woff(2)?(\?.*)?$/,
        include: path.resolve(assets),
        loader: 'url?' + qs.stringify({
          limit: 10000,
          mimetype: "application/font-woff",
          name: "[path][name]_[md5:hash:hex:8].[ext]"
        })
      },
      // Use file-loader for node_modules/ assets
      {
        test: /\.(ttf|eot|woff(2)?|png|jpg|jpeg|gif|svg)(\?.*)?$/,
        include: /node_modules/,
        loader: 'file?' + qs.stringify({
          name: 'vendor/[name]_[md5:hash:hex:8].[ext]'
        })
      }
    ]
  },
  resolve: {
    extensions: [ '', '.js', '.json' ],
    modulesDirectories: [
      'node_modules',
      'bower_components'
    ]
  },
  externals: {
    jquery: 'jQuery'
  },
  plugins: [
    new Clean([dist]),
    new ExtractTextPlugin(stylesFilename, {
      allChunks: true,
      disable: (argv.watch === true) // '--watch' disable ExtractTextPlugin
    }),
    new webpack.ProvidePlugin({
      $: 'jquery',
      jQuery: 'jquery',
      'window.jQuery': 'jquery',
      'Tether': 'tether',
      'window.Tether': 'tether'
    }),
    new webpack.DefinePlugin({
      WEBPACK_PUBLIC_PATH: (argv.watch === true) ? JSON.stringify(path.join(config.publicPath, dist)) : false
    })
  ],
  postcss: [
    autoprefixer({
      browsers: [
        'last 2 versions',
        'android 4',
        'opera 12'
      ]
    })
  ],
  eslint: {
    failOnWarning: false,
    failOnError: true
  },
  stats: {
    colors: true
  }
};

// '--watch' to push additional plugins to webpackConfig
if (argv.watch) {
  webpackConfig.entry = addHotMiddleware(webpackConfig.entry);
  webpackConfig.output.pathinfo = true;
  webpackConfig.debug = true;
  webpackConfig.devtool = '#cheap-module-source-map';
  webpackConfig.plugins.push(new webpack.optimize.OccurenceOrderPlugin());
  webpackConfig.plugins.push(new webpack.HotModuleReplacementPlugin());
  webpackConfig.plugins.push(new webpack.NoErrorsPlugin());
}

// '--release' to push additional plugins to webpackConfig
if (argv.release) {
  webpackConfig.plugins.push(new AssetsPlugin({
    path: path.join(__dirname, dist),
    filename: 'assets.json',
    fullPath: false,
    processOutput: assetsPluginProcessOutput
  }));
  webpackConfig.plugins.push(new webpack.optimize.UglifyJsPlugin({
    compress: {
      'drop_debugger': true
    }
  }));
  webpackConfig.plugins.push(new OptimizeCssAssetsPlugin({
    cssProcessor: cssnano,
    cssProcessorOptions: { discardComments: { removeAll: true } },
    canPrint: true
  }));
}

module.exports = webpackConfig;

I`ve made clear test with clean repo
git clone https://github.com/roots/sage.git
cd sage
npm istall
npm run  build

And there is same issue with backslashes

I just ran this with Node 6.3.0 and got similar output in the CSS after running npm run build in Bash Cmder
  src: url(\app\themes\sage\dist\vendor/fontawesome-webfont_25a32416.eot);
It's probably an issue with Webpack, but since we have a kinder gentler qwp6t here, we can use him as a scapegoat

I can also confirm that https://github.com/roots/sage/pull/1703 fixes this
src: url(/app/themes/sage/dist/vendor/fontawesome-webfont_25a32416.eot);

Fro now i replaced native this with laravel-elixir and it warks good

I believe this is fixed now.


SMTP

Area(s)
area:new
Is your change request related to a problem? Please describe.
I would like to see traces of emails being send by our system.  We use a Java library that relies on javax.mail.Transport to send emails via SMTP.
Describe the solution you'd like
I would like to see semantic conventions that I can adhere to when manually wrapping library calls in spans, until https://github.com/open-telemetry/opentelemetry-java-instrumentation/ has implemented support for automatically tracing SMTP requests.
Describe alternatives you've considered
No response
Additional context
No response

if you are interested to send a prototype to https://github.com/open-telemetry/opentelemetry-java-instrumentation, that could help get this ball rolling!


Upgrade to laravel 8

@cloudlog It looks like you created a fork and may have fixed this plugin for Laravel 8. Is that correct?


Lint with shellcheck

Shellcheck reports quite a few warnings on maza.
I haven't investigated to check if they're that serious, but it would be reassuring to have a minimum of those, especially for a script that's going to be run under sudo.

Thanks for the report.
If you read the warnings carefully, they are almost all double-quote notices or an arrays notice.
If you want to make me a pull request with the format improvements I will gladly accept it.
Greetings


Localization: update pyramid_map module in local_pyrami_map

May I know where this module is being using by apollo?
It seems like msf and ndt are still using the older version of the base_map and it's derived classes.


cmd/go: spurious recursive VCS reference using Mercurial

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go 1.9.1, and confirmed issue not present with go 1.9
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
It's Darwin 16.7.0 from macOS 10.12.6 but problem also observed in Linux docker images under Circle CI, so it appears to be platform independent.
What did you do?
I ran go get -v -u -t -d ./... twice on a project which pulls in bitbucket.org/pkg/inflect as a dependency.  When run with a clean tree, it succeeds without error.  When run with the content already checked out, it errors.  Across 385 lines of output from the above, there is one line of difference between the two:
-bitbucket.org/pkg/inflect (download)
+package bitbucket.org/pkg/inflect: directory "/Users/pdp/tmp/go-foo/src/bitbucket.org/pkg/inflect" uses hg, but parent "/Users/pdp/tmp/go-foo/src/bitbucket.org/pkg" uses hg

There is no .hg directory in the parent.  This error appears spurious.
If I switch /usr/local/go from 1.9.1 back to 1.9, then go get -v -u -t -d ./... succeeds.
Thus 1.9.1 appears to be incorrectly determining that the parent is using hg and that this is some kind of conflict.
What did you expect to see?
No error, no difference between the two runs
What did you see instead?
A spurious failure.

I can recreate this by doing
go get bitbucket.org/pkg/inflect
go get -u bitbucket.org/pkg/inflect

The error I get is
package bitbucket.org/pkg/inflect: directory "/home/iant/gopath/src/bitbucket.org/pkg/inflect" uses hg, but parent "/home/iant/gopath/src/bitbucket.org/pkg" uses hg

There is no .hg directory in /home/iant/gopath/src/bitbucket.org/pkg nor in /home/iant/gopath/src/bitbucket.org.  There is a .hg directory in /home/iant/gopath/src/bitbucket.org/pkg/inflect.
CC @rsc

Reopening for 1.9.2.

CL 69670 OK for Go 1.9.2.

go1.9.2 has been packaged and includes:

CL 69670

The release is posted at golang.org/dl.
— golang.org/x/build/cmd/releasebot, Oct 26 21:09:22 UTC


Maddening hex still broken

AutoSave_3.zip
Player.log
still got this error when I used maddening hex in co op mode. non-host uses the maddening hex and it throws this error.

There isn't much else I can do here. Error complains about an attack action. v28 has an small issue with spells so might be it. If not fixed in v29 won't get fixed.

this was on v29, no issues at all on your save.


Version 1.0

Reference Issues/PRs
This PR will close #1 and close #3.
This version contains:
### Added
- codefactor tests
- pypi installing guid
- `Document.ipynb`
- `Examples.ipynb`
- `notebook_check.py`
- `precision` added to `is_prob_dist`
- `payoff` attribute
- `cost` attribute
- `discount` attribute
- `best_policy` method
### Changes
- `make_prob_dist` bug fixed
- `final_dist` bug fixed

Codecov Report

Merging #7 (c1ece00) into master (f39947b) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          128       149   +21
  Branches        22        29    +7
=========================================
+ Hits           128       149   +21

Impacted Files
Coverage Δ

pyrandwalk/pyrandwalk_error.py
100.00% <ø> (ø)

pyrandwalk/pyrandwalk_obj.py
100.00% <100.00%> (ø)

pyrandwalk/pyrandwalk_param.py
100.00% <100.00%> (ø)

pyrandwalk/pyrandwalk_util.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@sadrasabouri Nice 💯
I will review this PR ASAP


"detail": "model not loaded"

"detail": "model not loaded"

post /switch-model API to load the model: https://github.com/josStorer/RWKV-Runner/blob/master/deploy-examples/ChatGPT-Next-Web/setup.sh#L27

谢谢


Convert attributes to camelCase

In order to make API consistent with VDP, attributes should be renamed from under_score to camelCase. This is a violation of PEP-8 in favor of explicity.

76f43151c1560fe360ec4c45b241379b1e1350bb


cargo 0.37.0 rebuilds stuff when it shouldn't need to

Problem
I am packaging some rust packages for Exherbo.
We basically run cargo fetch with network access, then cargo build --release and cargo install --path=. --root=/usr in a sandboxed environment.
Until rust 1.35 and cargo 0.36, it worked as expected.
Since rust 1.36 and cargo 0.37, the cargo install step rebuilds a large part of the deps and bins.
Furthermore, cargo install doesn't seem to use the net.online confirguration from .cargo/config, which is what allowed us to spot this issue, as we hit sandbox violations.
Notes
Output of cargo version:
cargo 1.36.0

Nevermind, that's actually a problem in the crate. the offline error is still valid though, will open another issue


Single line ignore switches

This PR supports using ESLint style single line ignore switches by using tslint:disable-line and tslint:disable-next-line.
This PR resolves #144

I'll see what I can come up with tonight

Looking good to me @patsissons! Let's see if we can simplify things a little using the SourceFile APIs, but if it seems like a bad idea after trying it to you, we can stick with something more like it is now

I'm all for optimized clean code :+1: what you're suggesting sounds pretty reasonable.

👍 Like this new version, much easier to read and understand! Just had one comment above, looks good to me otherwise

Thoughts on tests to cover the start and stop interval creation? Or is that better captured in a new PR?

As far as tests, I think we should be able to move this over to the newer, more visual testing system. (I.e. the files in test/rules. Test case(s) could go in test/rules/_integration/comments. That can be a follow-up though

@patsissons Do you want to do the test PR? Or should I try to knock it out? Just want to make sure we don't duplicate work

I probably wouldn't get to it for a few days, so go ahead. Plus you'll definitely have a better understanding on how to improve things than I will.


Add Spoonacular API

Adding more APIs to help developers from all over the world.
Thank you for taking the time to work on a Pull Request for this project!
To ensure your PR is dealt with swiftly please check the following:

[ ] Your submissions are formatted according to the guidelines in the contributing guide
[ ] Your additions are ordered alphabetically
[ ] Your submission has a useful description
[ ] The description does not end with punctuation
[ ] Each table column should be padded with one space on either side
[ ] You have searched the repository for any relevant issues or pull requests
[ ] Any category you are creating has the minimum requirement of 3 items
[ ] All changes have been squashed into a single commit

Shouldn't the commits should be squashed together into one with a more meaningful description?

F

Hi @RahulBansal123 😄
Thank you for your contribution. ✋
I noticed that there is a conflict in your pull request and you deleted the fork from this repository, so it will not be possible to resolve it. Consider creating a new pull request if you still want to add this APIs to be reviewed and added to our listing.


Dead Link

[Enter feedback here]
This link is dead - https://docs.microsoft.com/en-us/windows/threat-protection/windows-defender-antivirus/windows-defender-antivirus-on-windows-server-2016
Can it be updated to the current location?

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

ID: dfee81e6-dba5-cbd6-4190-3b3aa3fd5f94
Version Independent ID: ad63832b-7b61-01c2-83f8-11a446ddfbb7
Content: Windows Defender Overview for Windows Server
Content Source: WindowsServerDocs/security/windows-defender/windows-defender-overview-windows-server.md
Product: windows-server
Technology: security
GitHub Login: @JasonGerend
Microsoft Alias: jgerend

#assign:xelu86

Thanks again for contacting Microsoft. We've submitted a documentation change based on your feedback. Changes will be live after the next business day. #please-close


[CT-3169] [Bug] .yaml prefix not recognized for packages.yml

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
If I create a valid packages.yaml in my project directory and run dbt deps, it fails to find any dependencies. If I change the file extension to be .yml, things work fine.
Expected Behavior
I would expect .yaml and .yml to be interchangeable (and there seems to be an interface to handle both...? Maybe not used in some cases). If not, at least a warning pointing me to the issue.
Steps To Reproduce

Create new project
Create/copy packages.yaml file
run dbt deps

Relevant log output
{"data": {"log_version": 3, "version": "=1.4.5"}, "info": {"category": "", "code": "A001", "extra": {}, "invocation_id": "f97fdb3c-61cf-49fb-b595-3dc00faaa631", "level": "info", "msg": "Running with dbt=1.4.5", "name": "MainReportVersion", "pid": 63990, "thread": "MainThread", "ts": "2023-09-28T15:46:04.254826Z"}}
{"data": {}, "info": {"category": "", "code": "M013", "extra": {}, "invocation_id": "f97fdb3c-61cf-49fb-b595-3dc00faaa631", "level": "info", "msg": "Warning: No packages were found in packages.yml", "name": "DepsNoPackagesFound", "pid": 63990, "thread": "MainThread", "ts": "2023-09-28T15:46:04.277178Z"}}

Environment
- OS: MacOS 12.7
- Python: 3.10
- dbt: 1.4.5

Which database adapter are you using with dbt?
postgres
Additional Context
No response

Thanks for reaching out @andy-clapson !
Totally reasonable to expect that .yml and .yaml would be interchangable (especially since the YAML folks recommend the .yaml extension).
This is covered by https://github.com/dbt-labs/dbt-core/issues/5002, so I'm going to close this as a duplicate.
If anyone is interested in tackling this effort, there's some guidance here (that covers all of "profiles.yml", "dbt_project.yml", "selectors.yml", "packages.yml"): https://github.com/dbt-labs/dbt-core/issues/5002#issuecomment-1095183224
i.e., the end goal is that all of these would be interchangeable:

profiles.yml
profiles.yaml

dbt_project.yml
dbt_project.yaml

selectors.yml
selectors.yaml

packages.yml
packages.yaml

Thanks for reaching out @andy-clapson !
Totally reasonable to expect that .yml and .yaml would be interchangable (especially since the YAML folks recommend the .yaml extension).
This is covered by #5002, so I'm going to close this as a duplicate.
If anyone is interested in tackling this effort, there's some guidance here (that covers all of "profiles.yml", "dbt_project.yml", "selectors.yml", "packages.yml"): #5002 (comment)
i.e., the end goal is that all of these would be interchangeable:
profiles.yml 	profiles.yaml
dbt_project.yml 	dbt_project.yaml
selectors.yml 	selectors.yaml
packages.yml 	packages.yaml

ah! my searching failed me, but I did try and look for an existing issue - thanks!

Sure thing @andy-clapson ! Thanks again for opening this issue 🏆
You nudged us to review #5002, and then we updated the issue title (for better searchability) and re-categorized it as a handy feature request.


Less jarring indoor/outdoor scene transition

When entering/exiting a building, the game currently freezes for 2-3 seconds and jumps to the new scene. A gentler scene transition would be better.

Player/instructor fades out
Audio cue (fading footsteps)
Scene fades out
Scene fades in
Audio cue (fading footsteps)
Player/instructor fades in

I've added a fade in/fade out; I'll split out a new story to cover the remaining work.

I've added a fade in/fade out; I'll split out a new story to cover the remaining work.


Add an option to set the max_tokens_to_sample parameter

https://docs.anthropic.com/claude/reference/complete_post

This is currently hardcoded to 300 at https://github.com/tomviner/llm-claude/blob/0.2/llm_claude/init.py#L43
What's a good default? Or strategy to set it dynamically?

Released in https://github.com/tomviner/llm-claude/releases/tag/0.3


Add tests to the project

We should have tests in the project to make sure we don't break things. By example, we should have tests to make sure all page renders correctly.
We could even have some integration tests if needed be. But the first draft should contain at least the tests to make sure all pages render.

I'm not sure what you have in mind, but I think a test to check all internal links are properly referenced to the controller on each of the pages would be useful. Currently I find myself testing each new link manually to make sure I don't have errors.

I'm not sure how we would test link, but it's definitely those kinds of things I want to test. I guess for testing links, we could use Selenium or some automated component to test this.

Was the game rules test the last one that you were planning to fix?

It was the last one I had planned to fix, but not the last one to implement. I would like to tests, that the page is well formed, like specific HTML elements are present. Exemple: verify that the navigation menu on the left point to all the right URL like something that would look like : asserThat(element ).linkTo("/cards") (you get the idea). It would be a step higher than just test if the page has rendered correctly.

We now have some unit tests. There are no integration tests but at the moment I don't plan to add that.


composer install error

Hi
i'm getting the following error when running composer:
php composer.phar require league/oauth2-client:
Your requirements could not be resolved to an installable set of packages.
 Problem 1
    - The requested package league/oauth2-client No version set (parsed as 1.0.0) is satisfiable by league/oauth2-client[No version set (parsed as 1.0.0)] but these conflict with your requirements or minimum-stability.

any help appreciated
thanks,
sina

If you're able, can you paste the contents of your composer.json file here? Please do not include any personal or sensitive information when you paste it.
Also, what version of PHP are you using, and is your composer.phar the latest version of Composer?

Thanks for the reply.
I've never used composer before and installed the latest version this morning. This also explains my wrong approach to get it working. I managed to resolved it by using: php composer.phar install under /oauth2-client-master
My php version:
PHP 5.6.30 (cli) (built: May  6 2017 09:35:30)
Copyright (c) 1997-2016 The PHP Group
Zend Engine v2.6.0, Copyright (c) 1998-2016 Zend Technologies
with Zend OPcache v7.0.6-dev, Copyright (c) 1999-2016, by Zend Technologies
with Xdebug v2.5.3, Copyright (c) 2002-2017, by Derick Rethans
Although I tried various entries under require in composer.json, when finally got it working I was using the original.
 You can use Composer to require it as a dependency for your project.
I think you should take a look at a few Composer tutorials to get a good understanding of the purpose of Composer and how it works. Here are a few that I've found helpful:

https://www.youtube.com/watch?v=d_zPOJ8cz_A
https://www.frobiovox.com/posts/2016/08/16/basic-hello-world-with-composer-and-php.html
https://www.sitepoint.com/re-introducing-composer/
https://nomadphp.com/composer-the-right-way/

Once you've grasped the concepts of Composer (it's not hard, but it's a bit different if you're not used to dependency management, like with npm, pip, or gem), then you can require league/oauth2-client as a project dependency simply by typing from your project root:
composer require league/oauth2-client

That will handle download this library into your vendor folder so that you can use it in your project.
But first, go check out some of those tutorials. Good luck!

Very useful - thanks for your help.


Incorrect use-before-declaration error in async IIFE initializer

Bug Report

🔎 Search Terms

dead zone
await tdz
use before declaration await

🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about await

⏯ Playground Link

Playground link with relevant code
💻 Code

const promise = (async () => {
  await null
  promise
})()

Real life code sample affected by the error
  useEffect(() => {
    const request = (async () => {
      try {
        const response = await premiumApi.announcements(client)
        // @ts-expect-error TypeScript bug, assumes "request" is still in dead zone after await
        if (pendingRequest.current === request || pendingRequest.current === undefined) {
          pendingRequest.current = undefined
          setApiResponse(response)
        }
      } catch {
        pendingRequest.current = undefined
      }
    })()
    pendingRequest.current = request
    return () => {
      pendingRequest.current = undefined
    }
  }, [client])

🙁 Actual behavior

"Block-scoped variable 'promise' used before its declaration."
"Variable 'promise' is used before being assigned."

🙂 Expected behavior

No errors related to the promise variable.

This is not about the "'await' has no effect on the type of this expression" warning, this is a feature and is expected despite me disagreeing with it.
The first await to be reached in an async function is the point where the function stops being synchronous and yields a Promise for its result. This means that, immediately after the await, the const promise is no longer in TDZ and is well-defined: it is the not-yet-resolved Promise that was produced by the IIFE's invocation. Now, awaiting the promise would be a deadlock, as the promise would now be waiting for itself to resolve before being able to resolve, but the Promise instance itself is still useful for some purposes like === checks.

I think we can take a simpler fix that is slightly over-pessimistic - the "IIFEs get inlined for CFA" logic shouldn't ever be applied to async functions.

I have similar case with destructure into const bindings with default value,
Playground:
https://www.typescriptlang.org/play?#code/GYVwdgxgLglg9mABAcwKZQCoHc4YBYxjIDOAFAJSIDeAUIvYgE7oiNJWIKoBciAjABpEUHL3AATVMEKpxiAL41FNCAmJRqnMKiEi4iALyIKhgHybVYYnAA2qAHQ24yUl105EAegBUiVAA8AB1RoWWF9HEYAa0RvT0p5BUMUdGxcAiIycgBuGhU1DSgM5ABmZJMDc0trO0dnUiLCUpyvT0RIqOJEaW12ggg8RBgu1GJiUZgAQxsbAE9hPFRhDwAjQnEm4nsgA

I have similar case with destructure into const bindings with default value,
Playground: https://www.typescriptlang.org/play?#code/GYVwdgxgLglg9mABAcwKZQCoHc4YBYxjIDOAFAJSIDeAUIvYgE7oiNJWIKoBciAjABpEUHL3AATVMEKpxiAL41FNCAmJRqnMKiEi4iALyIKhgHybVYYnAA2qAHQ24yUl105EAegBUiVAA8AB1RoWWF9HEYAa0RvT0p5BUMUdGxcAiIycgBuGhU1DSgM5ABmZJMDc0trO0dnUiLCUpyvT0RIqOJEaW12ggg8RBgu1GJiUZgAQxsbAE9hPFRhDwAjQnEm4nsgA

You might want to file a separate issue about that destructuring-specific case (if an issue for that doesn't already exist).

@gabritto is #50199 also fixes my issue?

@gabritto is #50199 also fixes my issue?

I don't think so, the fix was only for async functions and in your case your function is not async. Can you open a new issue for your particular case?

Is it really fixed? I'm still reproducing this issue with any version from 4.8.4 up to 5.4.5 (available in playground), for original code snippet and for destructured snippet from @barak007. I'm facing the same issue, and I don't see any resolution.

Is it really fixed? I'm still reproducing this issue with any version from 4.8.4 up to 5.4.5 (available in playground), for original code snippet and for destructured snippet from @barak007. I'm facing the same issue, and I don't see any resolution.

I think you're right, thanks for pointing that out. The PR seems to have only fixed the problem when strictNullChecks is off.


Bump dependency versions

Hey jamigibbs,
thank you for your awesome template, I am building my personal page ontop of it.
Ruby-sass, a dependency of Jekyll 3.x is deprecated by now. Jekyll 4.0 is stable and utitlizes a new lib for sass compilation. I bumped versions of all dependenciess. At least the demo page renders just finde.

Pagination works as well.

(forgot to create seperate branch for PR)


Visual Studio code does not recognize jest ts

I have installed this package but VScode complains that Jest methods do not exist (describe, it, expect). Any  way to fix this?

Make sure your package.json contains an entry like "@types/jest": "^24.0.15",, reinstall NPM and use 'Typescript: Restart TS Server' option in VSCode. (CMD+SHIFT+P)

And if it still does not work?

Try restarting. This sometimes help.
I also experience VSCode not recognizing types every once in a while though they are definitely installed. Sometimes restarting helps. Sometimes it doesn't. I wish I knew how to reproduce it. This clearly has been an issue for some while.

Actually it somehow magically started working after a while... Thanks :)

@mkolodziejczyk-branch nice to hear!

The only way I was able to fix this, was by adding the tests/ folder to "include" in the tsconfig.json file:
"include": [
  "src/**/*.ts",
  "tests/*.ts"
]

I am having a similar issue. VSC recognises jest types if the project is opened from project root where tsconfig.json is located. If the project is opened from its parent directory, only way to get VSC to recognise the jest types is to include tests in tsconfig.json. Unfortunately this causes the tests to be compiled as well, which is often undesirable.
It would be great if VSC could recognise jest types even if opened in monorepo style.

If you don't want to add the tests folder to the tsconfig.json - why would you? - then you can get rid of the annoying problem messages in VS Code by adding this to the top of the folder
import { describe, test, expect } from '@jest/globals';
And any others you might need. Still hard to understand why @types/jest are not recognised.

@JelsB , I also included tests/**/*.ts but this change the build. Indeed, dist dir contains src and test folder.

I am having a similar issue. VSC recognises jest types if the project is opened from project root where tsconfig.json is located. If the project is opened from its parent directory, only way to get VSC to recognise the jest types is to include tests in tsconfig.json. Unfortunately this causes the tests to be compiled as well, which is often undesirable.

Thank you! This was driving me nuts until I read your post.
I had the following section in my tsconfig.json file, and commenting out "node_modules" fixed the issue...
  "exclude": [
    // "node_modules",
    "**/mocks/*.ts",
    "**/*.spec.ts",
    "**/*.test.ts",
    "**/*.spec.tsx",
    "**/*.test.tsx"
  ],

This may be old, but I figure adding my details may be useful to anyone coming around looking for an answer.
TLDR; Open the project folder directly, instead of opening a mono-repo's root directory
I noticed that VSCode does not find the jest globals when a parent folder of a project is your current "root".  Example below is the structure I had, and while editing test1.test.ts the gobals test and expect are not found by VSCode even though @types/jest is installed.
parent-folder

package

src

tests

test1.test.ts
test2.test.ts

jestconfig.json
package.json
tsconfig.json

test-client

etc...

If I open a new VSCode instance and choose to open the package folder as the root, VSCode does find the jest globals and all looks and works great.
Hope this helps someone 👍

@codingLogan Thanks, that solves the problem for me, but now how do you work on files in another branch off the root?

@codingLogan Thanks, that solves the problem for me, but now how do you work on files in another branch off the root?

Hey @jonahx , in my particular case I opened  up a few instances of VS Code rather than having just one of the whole monorepo.  One was opened up with package as the root, and one with test-client.


Pod install takes a long time

Could you move the lib code to a separate project, since the current project is very big, the first time running pod install takes lots of time, thanks~

+1, Tests are about 132MB. It would be great to seperate the lib into a seperate project.

Yes please. It's a pain to use as a submodule because of the massive size of the test case resource files

Yes please, even with the last version (3.1.0), the "Pod install" takes a loooong time, IT'S NOT NORMAL !!!

same problem

yeap, but I think the whole git repo is getting fetched anyway, so just moving the test files in a new subrepo would not solve the problem. Maybe we would need to split the project in two new repos. can anyone concur?

The problem is that even if we get rid of all the test images, they still live in the git history, so I don't think it would help the time it takes to clone the repo.

Yeap this is what I meant. The whole repo is getting fetched. The only way would be to split this repo.

@ltebean @krutarth @bryanhathaway @gitabdelkarim @benjohnde @cwalcott I have a temporary solution, first download the entire project, then ZXingObjC.podspec ZXingObjC folders and files into your project directory,Last edited Podfile, to set up local files referenced ZXingObjC
pod 'ZXingObjC', :path => '~/youfilepath/ZXingObjC'

You can also refer to the official documentation cocopods Using the files from a folder local to the machine.
This way when you update pod again when you do not need to wait any longer，But the ZXingObjC on local certainly is not good, so we just added the file and then delete, modify podfile is again
pod 'ZXingObjC'

Then update pod again,You will find that it has no need to update ZXingObjC

Also after the first pod install, subsequent ones should be much faster assuming the Pods directory still exists.

After I install the Pod, can I safely delete the large files such as the tests without affecting the functionality of the library? I'm thinking about disk space...

Sure you can. The large files (I guess you mean the resource files for the tests) can be deleted w/o disturbing the normal functionality of the library. They are only required to run all the black box tests.


[Snyk] Security upgrade org.apache.jena:jena-jdbc-core from 4.9.0-SNAPSHOT to 4.10.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

jena-jdbc/jena-jdbc-driver-mem/pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

509/1000  Why? Has a fix available, CVSS 5.9
Improper Input Validation SNYK-JAVA-ORGAPACHECOMMONS-5901530
org.apache.jena:jena-jdbc-core:  4.9.0-SNAPSHOT -> 4.10.0
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Improper Input Validation

Checkmarx One – Scan Summary & Details – 8930c797-70c0-4489-ac50-d74703632a65
New Issues

Severity
Issue
Source File / Package
Checkmarx Insight

CVE-2023-36478
Maven-org.eclipse.jetty:jetty-http-10.0.15
Vulnerable Package

CVE-2023-43646
Npm-get-func-name-2.0.0
Vulnerable Package

CVE-2023-45133
<EMAIL_ADDRESS>Vulnerable Package

CVE-2023-36479
Maven-org.eclipse.jetty:jetty-servlets-10.0.15
Vulnerable Package

CVE-2023-40167
Maven-org.eclipse.jetty:jetty-http-10.0.15
Vulnerable Package

CVE-2023-42503
Maven-org.apache.commons:commons-compress-1.23.0
Vulnerable Package

CVE-2023-44270
Npm-postcss-8.4.21
Vulnerable Package


how to get new refresh token after revoked ?

I trying to get new refresh token after revoked but i am getting 1 instead of refresh token
$dataService = DataService::Configure(array(
     		'auth_mode' => 'oauth2',
	     	'ClientID' => 'xxxxxxxxxxxxxxx',
	     	'ClientSecret' => 'xxxxxxxxx',
	     	//get the refresh token from session or database
	     	'refreshTokenKey' => 'xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx',
	     	'QBORealmID' => '13934156454157',
	     	'baseUrl' => "development"
		));

$OAuth2LoginHelper = $dataService->getOAuth2LoginHelper();

$refreshedAccessTokenObj = $OAuth2LoginHelper->refreshToken();
$revokeResult = $OAuth2LoginHelper->revokeToken($refreshedAccessTokenObj->getAccessToken());

Can you explain your logic, please?
Revoking means that this set of Access+Refresh tokens are void, and can't be used anymore.
Why do you want to void token and get new one?

Hello @ebogdanov
Its necessary for if some user want to disconnect from authorized app, that's why I revoked that token.
I am creating a demo for how to regenerate new refresh token after 101 days/

OK, I see.
I just capture refresh token every time after refreshToken call to solve this.

Yes, you're right.
But documentation says that we need to save every refreshToken, since they "can return new key someday", and old will not be valid after 24 hours.
https://developer.intuit.com/docs/00_quickbooks_online/2_build/10_authentication_and_authorization/10_oauth_2.0#/Refreshing_the_access_token

I tried to get new refreshToken after revoked existing token, I get an error about invalid_grant. RefeshToken is valid for 101 days and accessToken vallid for 24 hours. After run above function I am getting new accessToken on every request. I am storing on refreshToken.

Yes, this is correct API behaviour. You need to reconnect application to get new pair of access+refresh tokens, if your refresh token is expired or revoked.
AccessToken is valid for 3600 seconds, which is 1 hour.
RefreshToken is valid for 100 days if initial accessToken was not used.
Check link, please, it describes all of those cases.

That means I need to reconnect or re-authorize app after 101 days, Right @ebogdanov ?

Yes, during 100 days you should be able to load new access token and refresh token.
If during this was was not done - your keyset have been revoked by Intuit and as a result there is only one solution: to reconnect app, to get new key set.
If you revoked key manually (by code or user disconnected application) you should reconnect app too.

Thanks @ebogdanov


AppId not set when using Helm chart

I tried to deploy Safe-settings on an EKS cluster by using the Helm chart 0.1.0.
helm upgrade --install \
            --username="XXX" \
            --password="XXX" \
            safe-settings-test \
            -f values.yml \
            --set appEnv.APP_ID="\"XXX\"" \
            --set appEnv.PRIVATE_KEY="XXX" \
            --set appEnv.WEBHOOK_SECRET="XXX" \
            --set appEnv.DEPLOYMENT_CONFIG_FILE="deployment-settings.yml" \
            --namespace YYY \
            https://enterprise-repo/safe-settings-0.1.0.tgz

(using XXX for obfuscation)
We got the following error :

<EMAIL_ADDRESS>start /opt/safe-settings
probot run ./index.js

(node:36) UnhandledPromiseRejectionWarning: Error: [@octokit/auth-app] appId option is required
at Object.createAppAuth (/opt/safe-settings/node_modules/@octokit/auth-app/dist-node/index.js:402:11)
at getState (/opt/safe-settings/node_modules/octokit-auth-probot/dist-node/index.js:176:21)
at createProbotAuth (/opt/safe-settings/node_modules/octokit-auth-probot/dist-node/index.js:191:17)
at new Octokit (/opt/safe-settings/node_modules/@octokit/core/dist-node/index.js:113:20)
at new _a (/opt/safe-settings/node_modules/@octokit/core/dist-node/index.js:167:30)
at new OctokitWithDefaults (/opt/safe-settings/node_modules/@octokit/core/dist-node/index.js:143:11)
at new OctokitWithDefaults (/opt/safe-settings/node_modules/@octokit/core/dist-node/index.js:143:11)
at new Probot (/opt/safe-settings/node_modules/probot/lib/probot.js:38:25)
at new ProbotWithDefaults (/opt/safe-settings/node_modules/probot/lib/probot.js:64:17)
at new Server (/opt/safe-settings/node_modules/probot/lib/server/server.js:36:26)
(Use node --trace-warnings ... to show where the warning was created)
I realized that the default docker image version is the 2.0.0 and gave also a try with the 2.0.18, but got the same issue.
It seems that the options are not correctly set but I can clearly see that my pod has the following environments variables : APP_ID, WEBHOOK_SECRET and PRIVATE_KEY with the expected values as defined during helm command execution.
Definitely need some help toe troubleshoot this issue :-)

resolved by https://github.com/decyjphr-org/charts/issues/2?


Rerender is called when component is destroyed

The Error: You can't call rerender on a view being destroyed
The Use Case: Having a service that's receiving a stream of data, collected into an array. This array is binded to ember-collection. When we destroy the component, ie. navigate to another route, the observer of ember-collection still catches changed made from the data stream (WS). It tries to rerender when the component is dead.
I hope someone could tell me that I'm doing something wrong and it's not the library. I patched mine up like so on ember-collection.js.
// ember-collection.js
_needsRevalidate(){
    if (this.isDestroyed || this.isDestroying) {return;}
    if (this._isGlimmer2()) {
      this.rerender();
    } else {
      needsRevalidate(this);
    }
  }

Using Ember 2.10.x

@lukemelia is there a reason we would not want to put the if (this.isDestroyed || this.isDestroying) {return;} check in the official library?


nixos/syncthing: add declarative.extraOptions

Allows setting arbitrary config options through the REST API.
I also refactored the config update script to use the new config endpoints (the old one being deprecated), restart Syncthing only if required, and be more resilient to transient failures.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[X] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

o/ @Lassulus

@ofborg test syncthing syncthing-init

Thanks!


Stop worker daemons when the build is canceled

In #29043, with persistent worker daemons, we are seeing that if a build is canceled while running a worker daemon action (e.g. a compilation) we can get into a situation where the response from the canceled build is still in the communication queue for the worker daemon when the next build executes.  The compile task in the second build then gets the response from the first build and this ends up caching an incorrect result for the compile task.
To combat this, whenever a build is canceled, we immediately stop any worker daemons and remove them from the client manager as we can no longer guarantee that they (or their associated communications) are in a good state.
This only has an effect on persistent worker daemons, as session scoped daemons are stopped anyways when the build session finishes.
Fixes #29043
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

Do you know if we treat a killed client as a build cancellation?

Any time the client disconnects, the daemon triggers a cancellation here.