Why can't I get the gradient when I use in cascaded vmap?

Hello, I have a question about getting gradients with equinox.
In the function of first 'grad_and_loss' in the code below, when I tried to get the gradient but only got zero gradient.
For convenience, I used jax.vmap along batch size etc.
However, when I tried to get the gradient using second 'grad_and_loss', which contains function 'calc_sample' directly, I can get the nonzero gradient.
Isn't @eqx.filter_value_and_grad available when I used jax.vmap?? My code is simple, so I cannot find the reason why.
I'm looking forward to your response.
Thank you.
import time
import math
import diffrax
import equinox as eqx  # https://github.com/patrick-kidger/equinox
import jax
import jax.nn as jnn
import jax.numpy as jnp
from jax import grad, jit, vmap
import matplotlib.pyplot as plt
import optax  # https://github.com/deepmind/optax
import jax.tree_util as jtu
import numpy as np
import pandas as pd

dataset_size=10
N=32
M=64 # sample number
batch_size=1 # batch data among total data (dataset_size)
width_size=64
depth=2
seed=5678

key = jax.random.PRNGKey(seed)
data_key, model_key, loader_key = jax.random.split(key, 3)

# data
length = 50
ts = jnp.linspace(0.0, 10.0, length)
ys = jnp.ones((10,length,1)) # batch, length, data size
_, length_size, data_size = ys.shape

class Func(eqx.Module):
    mlp: eqx.nn.MLP

    def __init__(self, data_size, width_size, depth, *, key, **kwargs):
        super().__init__(**kwargs)
        self.mlp = eqx.nn.MLP(
            in_size = data_size,
            out_size = data_size,
            width_size = width_size,
            depth = depth,
            activation = jnp.tanh,
            use_bias = True, # remove bias (False)
            use_final_bias = True,
            key = key,
        )

    def __call__(self, t, y, args):
        return self.mlp(y)

model = Func(data_size, width_size, depth, key=model_key) # MLP model

def probability(b):
    p = b ** 2
    return p

# L : Time length, D : Data dimension, batch_num : number of batch
@eqx.filter_jit
def calc_sample(ti, yi, key):
    # ti : (L,)
    # yi : (L,D)

    b = jax.vmap(model.mlp)(yi) # (L,D)
    approx_p = jax.vmap( probability )(b) # (L,D)

    return approx_p

@eqx.filter_jit
def calc_prob(ti, yi):
    # calculate sample mean probability for each batch
    # ti : (L,)
    # yi : (L,D)

    start = time.time()
    subkeys = jax.random.split(key, M)
    sample_p = jax.vmap(calc_sample, in_axes=(None,None,0))(ti,yi,subkeys) # (M,L,D,1)
    # sample_p = calc_sample(ti,yi,key)
    end = time.time()

    print("Time : ", end-start)
    sample_p_mean = jnp.sum(sample_p)

    return sample_p_mean

# In this case, I get weights of layers are all zero.
# @eqx.filter_value_and_grad
# def grad_loss(model, ti, yi):
#     # yi : (batch_num, L=length * length_size, data size)
#     # ti : (L,)
#     # approximate the probability

#     # Sample M
#     p_pred = jax.vmap(calc_prob, in_axes=(None,0))(ti,yi) # along batch_num
#     return jnp.sum( p_pred ) # negative log likelihood

# However, I can get nonzero weights in this case
@eqx.filter_value_and_grad
def grad_loss(model, ti, yi):
    # yi : (batch_num, L=length * length_size, data size)
    # ti : (L,)

    b = jax.vmap(jax.vmap(model.mlp))(yi) # (batch_num, L, D)
    approx_p = jax.vmap( jax.vmap(probability) )(b) # (batch_num, L, D)
    return jnp.sum( approx_p )

@eqx.filter_jit
def make_step(ti, yi, model, opt_state):
    loss, grads = grad_loss(model, ti, yi)
    updates, opt_state = optim.update(grads, opt_state)
    model = eqx.apply_updates(model, updates)
    return loss, model, opt_state, grads

lr = 1e-3
steps = 100
optim = optax.adabelief(lr)
opt_state = optim.init(eqx.filter(model, eqx.is_inexact_array))

for step in range(steps):
    loss, model, opt_state, grads = make_step(ts, ys, model, opt_state)
    a,b,c = grads.mlp.layers
    print(a.weight[:3]) # check the gradients of weights in layers

Hi there! This is quite a large code sample, and I'm afraid it doesn't run directly. Can you reproduce the issue in something smaller, please?

I changed my code as you can see above code for reproducing my problem.
Thank you for your quick response.

Hi! This is still a huge example. I'm afraid it would still take me a long time to dig through your code like this.
Please can you reduce this to a single code snippet, ideally <30 lines or so, demonstrating the issue?

import time
import math
import diffrax
import equinox as eqx  # https://github.com/patrick-kidger/equinox
import jax
import jax.nn as jnn
import jax.numpy as jnp
from jax import grad, jit, vmap
import matplotlib.pyplot as plt
import optax  # https://github.com/deepmind/optax
import jax.tree_util as jtu
import numpy as np
import pandas as pd

dataset_size=10
N=32
M=64 # sample number
batch_size=1 # batch data among total data (dataset_size)
width_size=64
depth=2
seed=5678

key = jax.random.PRNGKey(seed)
data_key, model_key, loader_key = jax.random.split(key, 3)

# data
length = 50
ts = jnp.linspace(0.0, 10.0, length)
ys = jnp.ones((10,length,1)) # batch, length, data size
_, length_size, data_size = ys.shape

class Func(eqx.Module):
    mlp: eqx.nn.MLP

    def __init__(self, data_size, width_size, depth, *, key, **kwargs):
        super().__init__(**kwargs)
        self.mlp = eqx.nn.MLP(
            in_size = data_size,
            out_size = data_size,
            width_size = width_size,
            depth = depth,
            activation = jnp.tanh,
            use_bias = True, # remove bias (False)
            use_final_bias = True,
            key = key,
        )

    def __call__(self, t, y, args):
        return self.mlp(y)

model = Func(data_size, width_size, depth, key=model_key) # MLP model

def probability(b):
    p = b ** 2
    return p

# L : Time length, D : Data dimension, batch_num : number of batch
@eqx.filter_jit
def calc_sample(ti, yi, key):
    # ti : (L,)
    # yi : (L,D)

    b = jax.vmap(model.mlp)(yi) # (L,D)
    approx_p = jax.vmap( probability )(b) # (L,D)

    return approx_p

@eqx.filter_jit
def calc_prob(ti, yi):
    # calculate sample mean probability for each batch
    # ti : (L,)
    # yi : (L,D)

    start = time.time()
    subkeys = jax.random.split(key, M)
    sample_p = jax.vmap(calc_sample, in_axes=(None,None,0))(ti,yi,subkeys) # (M,L,D,1)
    # sample_p = calc_sample(ti,yi,key)
    end = time.time()

    print("Time : ", end-start)
    sample_p_mean = jnp.sum(sample_p)

    return sample_p_mean

@eqx.filter_value_and_grad
def grad_loss(model, ti, yi):
    # yi : (batch_num, L=length * length_size, data size)
    # ti : (L,)

    b = jax.vmap(jax.vmap(model.mlp))(yi) # (batch_num, L, D)
    approx_p = jax.vmap( jax.vmap(probability) )(b) # (batch_num, L, D)
    return jnp.sum( approx_p )

@eqx.filter_jit
def make_step(ti, yi, model, opt_state):
    loss, grads = grad_loss(model, ti, yi)
    updates, opt_state = optim.update(grads, opt_state)
    model = eqx.apply_updates(model, updates)
    return loss, model, opt_state, grads

lr = 1e-3
steps = 100
optim = optax.adabelief(lr)
opt_state = optim.init(eqx.filter(model, eqx.is_inexact_array))

for step in range(steps):
    loss, model, opt_state, grads = make_step(ts, ys, model, opt_state)
    a,b,c = grads.mlp.layers
    print(a.weight[:3]) # check the gradients of weights in layers

If I try this one, then I got

However, I try this one (I just changed function 'grad_loss')
import time
import math
import diffrax
import equinox as eqx  # https://github.com/patrick-kidger/equinox
import jax
import jax.nn as jnn
import jax.numpy as jnp
from jax import grad, jit, vmap
import matplotlib.pyplot as plt
import optax  # https://github.com/deepmind/optax
import jax.tree_util as jtu
import numpy as np
import pandas as pd

dataset_size=10
N=32
M=64 # sample number
batch_size=1 # batch data among total data (dataset_size)
width_size=64
depth=2
seed=5678

key = jax.random.PRNGKey(seed)
data_key, model_key, loader_key = jax.random.split(key, 3)

# data
length = 50
ts = jnp.linspace(0.0, 10.0, length)
ys = jnp.ones((10,length,1)) # batch, length, data size
_, length_size, data_size = ys.shape

class Func(eqx.Module):
    mlp: eqx.nn.MLP

    def __init__(self, data_size, width_size, depth, *, key, **kwargs):
        super().__init__(**kwargs)
        self.mlp = eqx.nn.MLP(
            in_size = data_size,
            out_size = data_size,
            width_size = width_size,
            depth = depth,
            activation = jnp.tanh,
            use_bias = True, # remove bias (False)
            use_final_bias = True,
            key = key,
        )

    def __call__(self, t, y, args):
        return self.mlp(y)

model = Func(data_size, width_size, depth, key=model_key) # MLP model

def probability(b):
    p = b ** 2
    return p

# L : Time length, D : Data dimension, batch_num : number of batch
@eqx.filter_jit
def calc_sample(ti, yi, key):
    # ti : (L,)
    # yi : (L,D)

    b = jax.vmap(model.mlp)(yi) # (L,D)
    approx_p = jax.vmap( probability )(b) # (L,D)

    return approx_p

@eqx.filter_jit
def calc_prob(ti, yi):
    # calculate sample mean probability for each batch
    # ti : (L,)
    # yi : (L,D)

    start = time.time()
    subkeys = jax.random.split(key, M)
    sample_p = jax.vmap(calc_sample, in_axes=(None,None,0))(ti,yi,subkeys) # (M,L,D,1)
    # sample_p = calc_sample(ti,yi,key)
    end = time.time()

    print("Time : ", end-start)
    sample_p_mean = jnp.sum(sample_p)

    return sample_p_mean

# In this case, I get weights of layers are all zero.
@eqx.filter_value_and_grad
def grad_loss(model, ti, yi):
    # yi : (batch_num, L=length * length_size, data size)
    # ti : (L,)
    # approximate the probability

    # Sample M
    p_pred = jax.vmap(calc_prob, in_axes=(None,0))(ti,yi) # along batch_num
    return jnp.sum( p_pred ) # negative log likelihood

@eqx.filter_jit
def make_step(ti, yi, model, opt_state):
    loss, grads = grad_loss(model, ti, yi)
    updates, opt_state = optim.update(grads, opt_state)
    model = eqx.apply_updates(model, updates)
    return loss, model, opt_state, grads

lr = 1e-3
steps = 100
optim = optax.adabelief(lr)
opt_state = optim.init(eqx.filter(model, eqx.is_inexact_array))

for step in range(steps):
    loss, model, opt_state, grads = make_step(ts, ys, model, opt_state)
    a,b,c = grads.mlp.layers
    print(a.weight[:3]) # check the gradients of weights in layers

It only depends on how to calculate the loss. In first one, I just calculate the loss along the batch size and sample size (because I want to calculate many sample losses) but I break it down into small functions such as calc_prob and calc_sample.
Thank you for your help.

I see the issue. In this function:
@eqx.filter_value_and_grad
def grad_loss(model, ti, yi):
    # yi : (batch_num, L=length * length_size, data size)
    # ti : (L,)
    # approximate the probability

    # Sample M
    p_pred = jax.vmap(calc_prob, in_axes=(None,0))(ti,yi) # along batch_num
    return jnp.sum( p_pred ) # negative log likelihood

you're not actually using the model argument!
Thinking mathematically: the function grad_loss doesn't change as model varies. So by definition its derivative is zero.

Thank you for response. I should have added argument for model.
Thank you so much!


Move text content to separate file

Hi @satnaing, great work - I love the template!
I have a couple ideas that might be worth considering.
One of them is to move all the static text content (headings, descriptions, button names etc.) to one place for a bit easier management. In my case, I moved it to a separate JS object in a new file. Then it's just a matter of importing it wherever you might need it.
What do you think?

Yep.
This might be beneficial for AstroPaper users when they want to upgrade the template. I'm also considering integrating a CLI for AstroPaper in the future, so it could be worthwhile.
However, on the other hand, it might be a bit confusing for some users when they want to customize/extend the template.
Can you also provide me a repo or link as an example for your approach?

I'm relatively new to the modern JS framework scene, so I'm not saying this solution is anywhere close to the best one, but here is what I did:
https://github.com/fchrapek/filipchrapek.pl/blob/main/src/content/content.js

It would also be good to Get a section on all the things that need to be edited to make this your own (or move the content to a central location) .

[ ] src/pages/about.md
[ ] src/pages/index.astro


Better span names

This PR does two things:

Promote resource.name attribute to span name (and remove resource.name attribute)

Use / to separate class name/method name in span name, as this appears to be more consistent with OpenTelemetry semantic conventions, e.g. from data-rpc

Examples of span name: grpc.test.EchoService/Echo.

This is just a first pass. I also opened #233 to more thoroughly document and review all span names.

This PR has become unwieldy. Ok, it was unwieldy from the first commit 😄. Closing and will re-open more targeted PRs.


small error

Hi.
Using your program, I faced a small error.
It's in calendar-component.ts: 67.
exactly, it's this.length.
An error is following:
ERROR TypeError: Cannot read property 'length' of undefined
please, how can i fix this error?
Thanks.

Sure I would like to help. But it is difficult for me to help you unless you post the element which you are using to render the daterangepicker i.e the HTML tag which you are using in your code
e.g. <date-range-picker [fromDate]="'04/01/2017'" [toDate]="'04/02/2017'" [format]="'DD/MM/YYYY'" (datesSelected)="demo($event)">

I tried to do your words, but it didn't run.
Errors are forwards.

Can you please post your html code too? Not whole but related to this daterangepicker? As in your image it shows that it has error in calendar.ts file, none of the file in package has name calendar.ts. Hope you are using latest version of this package, if not tell me the version you are using.

ok.
calendar.ts.txt
daterangepicker.component.ts.txt
And then, what's wrong in this code ?
<input class="input-mini form-control" [ngModel]="fromDate | formatMomentDate: format" (blur)="formatFromDate($event)" type="text" name="daterangepicker_start" />

There is nothing wrong with this code. you have posted. Probably you are using date-range-picker tag in wrong way in your html code. What I want to know is how you are consuming my code? Please post that code which you are writing which is not part of this package, and the code which will consume my code.

Well, I'm going to change the calendar in your code.
But it doesn't work well.
The problem is I don't understand what function is getWeeksNumber()?
Can you explain about it?
Thanks.

yes it first gets all week numbers considering 1st jan week as first. And then it gets dates in those weeks.
e.g. for jan 2017 week numbers are 1,2,3,4,5 and for feb 2017 are 5,6,7,8,9. But you still did not post your HTML which is actually responsible to trigger my code.

Sorry, I'm confused.
Can you explain about bellow in detail?
Exactly, about indexOf.
`getWeekNumbers(monthRange: any) {
let weekNumbers = [];
let indexOf = [].indexOf || function(item) { for (var i = 0, l = this.length; i < l; i++) { if (i in this && this[i] === item) return i; } return -1; };
monthRange.by('days', function(moment) {
let ref;
if (weekNumbers.length < 6 && (ref = moment.week(), indexOf.call(weekNumbers, ref)) < 0) {
return weekNumbers.push(moment.week());
}
});
return weekNumbers;
}
getWeeksRange(weeks: any, year: any, month: any) {
let weeksRange = [];
    for (let i = 0, len = weeks.length; i < len; i++) {
        let week = weeks[i];
        let firstWeekDay, lastWeekDay;
        if (i > 0 && week < weeks[i - 1]) {
            firstWeekDay = moment([year, month]).add(1, "year").week(week).day(0);
            lastWeekDay = moment([year, month]).add(1, "year").week(week).day(6);
        }
        else {
            firstWeekDay = moment([year, month]).week(week).day(0);
            lastWeekDay = moment([year, month]).week(week).day(6);
        }
        let weekRange = moment.range(firstWeekDay, lastWeekDay);
        weeksRange.push(weekRange);
    }
    return weeksRange;
}`


feat: old docs issues

Description
This PR attempts to document all the pending issues in the old docs repo, all at once.
It also fixes a bugin the workflow.
Problem*

During the migration to the monorepo (#3162) some issues were added to the old repo. In order to archive it, we need to either transfer them back, or quickly document them.
If a developer made changes to the docs repo, the workflow wouldn't run because the build step stated contains(github.event.pull_request.labels.*.name, 'documentation'). This PR fixes that by adding a previous job that adds the label in case it isn't already there.

Resolves #3194
Summary*

Documents many issues from the old repo
Adds a job to the workflow that adds a documentation flag in case there are changes to the docs folder. This allows for quick filtering of which PRs have doc changes.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Using this issue to check and correct another bug... I'll be adding random labels

This is ready for review

Tagging @kevaundray in particular, because of the changes to the contributing guide


Wrong file name for text file

The name of the text file is not correct.
Using "ModuleToRemove.txt" will cause the Deployable Package in the Asset Library to be marked as invalid.
The real file name shoud be "ModulesToRemove.txt". Please fix this documentation page.

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

ID: 4f21baa4-903d-1a42-eaaf-4efe1ed1b951
Version Independent ID: e8d91d54-a0f1-5958-0623-6c0c54ec0b8d
Content: Uninstall a package - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/deployment/uninstall-deployable-package.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @laneswenka
Microsoft Alias: laswenka

BUG 481391 created to track this issue.


Long cronjob names can lead to issues on Openshift Deplyoments

Describe the bug
A long cronjob name can lead to failing deployments on Openshift
* metadata.name: Invalid value: "cronjob-cli-fastly-purge-queue-worker--flush-cache-tags-": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
* spec.jobTemplate.spec.template.labels: Invalid value: "fastly-purge-queue-worker--flush-cache-tags-": a valid label must be an empty string or consist of alphanumeric characters, '-', '_' or '.', and must start and end with an alphanumeric character (e.g. 'MyValue',  or 'my_value',  or '12345', regex used for validation is '(([A-Za-z0-9][-A-Za-z0-9_.]*)?[A-Za-z0-9])?')
* spec.jobTemplate.spec.template.spec.containers[0].name: Invalid value: "cronjob-cli-fastly-purge-queue-worker--flush-cache-tags-": a DNS-1123 label must consist of lower case alphanumeric characters or '-', and must start and end with an alphanumeric character (e.g. 'my-name',  or '123-abc', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?')
* metadata.name: Invalid value: "cronjob-cli-fastly-purge-queue-worker--flush-cache-tags-": must be no more than 52 characters

I'm not sure if we need to fix this as on K8s this seems to be not an issue from what I can tell currently.

We no longer support Openshift.

This is also an issue on k8s after all https://kubernetes.io/docs/concepts/workloads/controllers/cron-jobs/

The name of a CronJob must be a valid DNS subdomain value, but this can produce unexpected results for the Pod hostnames. For best compatibility, the name should follow the more restrictive rules for a DNS label. Even when the name is a DNS subdomain, the name must be no longer than 52 characters. This is because the CronJob controller will automatically append 11 characters to the name you provide and there is a constraint that the length of a Job name is no more than 63 characters.


⚠ Bofrost.at has degraded performance

In 9b740ec, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2347 ms

Resolved: Bofrost.at performance has improved in 7549ab0 after 16 minutes.


[resolves #2784] Add a test for client cert auth using CXF PhaseInter…

…ceptor

Retest this please


Update glrendermode.md

Remove doubled content - glrendermode.md
Contained double "error codes" description

@baetz-daniel : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Support for invisible recaptcha

Hi @cyberhck, does this help with ^1.1.5?
<Recaptcha
  sitekey="..."
  invisible
/>

window.grecaptcha.execute();

I have added a demo here: https://react-grecaptcha.netlify.com/?selectedKind=Recaptcha&selectedStory=Invisible reCAPTCHA&full=0&down=1&left=1&panelRight=0&downPanel=storybook%2Factions%2Factions-panel
Close it. Please feel free to reopen this issue if there are any problems.


no need to rm android build before each build

AFACIT, anyway. Spent some time looking at how Android is built and this stuck out to me. If there is, a clean rule would probably be better.

This change is

:+1:


Refactor from-token command to use code from autonomy.chain module

Proposed changes
This PR

Refactors the from-token command to use code from autonomy.chain module
Removes unnecessary code and data files
Updates the tests for the same

Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes

I'll update the chain selection process for from-token command in the docs PR


Adds system tests for storage transfer samples.

Also:

Makes comments more accurate (UTC not Pacific time).
Change to use properties instead of hard-coded constants for arguments.

@lesv
Fixes #113

LGTM - I hate that we are changing the same file :(
I see mention of UTC, but no where that it's set or actually tested.  (ie. your tests don't seem to request UTC in your tests).  If you don't think that's an issue – fine.

@tswast Your travis changes need help. :(

Current coverage is 48.92% (diff: 83.33%)

Merging #311 into master will increase coverage by 3.65%

@@             master       #311   diff @@
==========================================
  Files            73         73
  Lines          2211       2242    +31
  Methods           0          0
  Messages          0          0
  Branches        156        158     +2
==========================================
+ Hits           1001       1097    +96
+ Misses         1183       1115    -68
- Partials         27         30     +3

Powered by Codecov. Last update 48b978b...2aed070

@lesv Travis is happy now.
As far as UTC goes: yes, it would be good to have a test that verifies this, but there don't seem to be any guarantees on when the copy / transfer job will run. From my manual testing, I scheduled it, but then it didn't seem to run right at that minute, though it did run eventually.
It's possible that it is really Pacific time and the API docs are wrong, but we did get a report in #113 that it was actually UTC, so I'm inclined to believe the API docs.


Handle missing pricing info for user qual tool on Databricks platforms

Fixes https://github.com/NVIDIA/spark-rapids-tools/issues/1051
Changes

Add try catch block to handle case when tool cannot find pricing info for instance types for Databricks platforms

Testing
Running the following command with cluster properties files which contain instance types that do not have pricing info in tools:
spark_rapids qualification--eventlogs <my_event_logs> --cluster <my_cluster_properties_file>

Databricks AWS

 Before this PR

2024-05-30 16:02:55,269 ERROR root: Qualification. Raised an error in phase [Collecting-Results]
Traceback (most recent call last):
...
  File "~/spark-rapids-tools/user_tools/src/spark_rapids_pytools/pricing/databricks_pricing.py", line 88, in get_instance_price
    instance_dbu = instance_conf.get('DBU')
AttributeError: 'NoneType' object has no attribute 'get'

 After this PR

2024-05-30 16:56:51,448 ERROR rapids.tools.qualification: Error computing cost savings. Reason - RuntimeError: Could not find pricing info for instance type 'm7gd.8xlarge'. Skipping!

Databricks Azure

 Before this PR

2024-05-30 18:06:15,339 ERROR rapids.tools.price.Databricks-Azure: Could not find price for instance type 'Standard_D4s_v3': 'NoneType' object has no attribute 'get'
2024-05-30 18:06:15,339 ERROR root: Qualification. Raised an error in phase [Collecting-Results]
Traceback (most recent call last):
...
  File "~/spark-rapids-tools/user_tools/src/spark_rapids_pytools/pricing/databricks_azure_pricing.py", line 79, in get_instance_price
    rate_per_hour = instance_conf.get('TotalPricePerHour')
AttributeError: 'NoneType' object has no attribute 'get'

 After this PR

2024-05-30 18:07:47,401 ERROR rapids.tools.qualification: Error computing cost savings. Reason - RuntimeError: Could not find pricing info for instance type 'Standard_D4s_v3'. Skipping!

what happens when we have multiple apps?
How does the final output look like? What I am asking about whether the user will see cost-savings columns with empty values, or they see the output formatted for speedup-only format.

Thanks @amahussein!

Multiple apps has the same behavior as single app. They run successfully by skipping the cost savings computation.
For the final output, users will only see output formatted for speedup-only and no cost-savings columns if there pricing info is missing.


api: inline parameters to fix codegen crash

Issue: https://github.com/moov-io/go-client/issues/15

Codecov Report

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

@@           Coverage Diff           @@
##           master     #522   +/-   ##
=======================================
  Coverage   86.82%   86.82%
=======================================
  Files         111      111
  Lines        6604     6604
=======================================
  Hits         5734     5734
  Misses        567      567
  Partials      303      303


assorted code improvement

as I've been off working on #630, I've found some things I wanted to improve or fix along the way. here they are.

not sure who to review-request on this – Jason, Dale, Lukas, and Eugene are all reasonable candidates


Array binary search

[x] Create a Node class that has properties for the value stored in the Node, and a pointer to the next Node.
[x] Within your LinkedList class, include a head property. Upon instantiation, an empty Linked List should be created.
[x] Define a method called insert which takes any value as an argument and adds a new node with that value to the head of the list with an O(1) Time performance.
[x] Define a method called includes which takes any value as an argument and returns a boolean result depending on whether that value exists as a Node’s value somewhere within the list.
[x] Define a method called toString (or str in Python) which takes in no arguments and returns a string representing all the values in the Linked List, formatted as:
"{ a } -> { b } -> { c } -> NULL"
[x] Did all asked tests


Migrate application CRD to kubebuilder 2.0

@barney-s comments resolved. I did git commit --amend to update the commit. Let me know if you want me to push new commits for later update. Thanks!

/approve
/lgtm


raise RuntimeError('You must compile your model before using it.')

Hey when i try to train the model i am getting the following error:
Using TensorFlow backend.
413439
WARNING:tensorflow:From C:\Users\UserName\Anaconda3\lib\site-packages\tensorflow\python\framework\op_def_library.py:263: colocate_with (from tensorflow.python.framework.ops) is deprecated and will be removed in a future version.
Instructions for updating:
Colocations handled automatically by placer.
Model created!
Traceback (most recent call last):
File "D:\Image-caption\Image-Captioning-master\train.py", line 14, in
train(int(sys.argv[1]))
File "D:\Image-caption\Image-Captioning-master\train.py", line 9, in train
model.fit_generator(sd.data_process(batch_size=batch_size), steps_per_epoch=sd.no_samples/batch_size, epochs=epoch, verbose=2, callbacks=None)
File "C:\Users\UserName\Anaconda3\lib\site-packages\keras\legacy\interfaces.py", line 91, in wrapper
return func(*args, **kwargs)
File "C:\Users\UserName\Anaconda3\lib\site-packages\keras\engine\training.py", line 1418, in fit_generator
initial_epoch=initial_epoch)
File "C:\Users\UserName\Anaconda3\lib\site-packages\keras\engine\training_generator.py", line 40, in fit_generator
model._make_train_function()
File "C:\Users\UserName\Anaconda3\lib\site-packages\keras\engine\training.py", line 496, in _make_train_function
raise RuntimeError('You must compile your model before using it.')
RuntimeError: You must compile your model before using it.
I am trying to solve it but getting no luck. Its been 2 weeks since i am stuck on the problem. Please help me out here.

when create the model, you should set ret_model=False


Middleware: Refactor Auth middleware

Middleware: Refactor Auth middleware
Description
Currently the middleware depends on domain.AuthService interface and would like to keep it that way. I thinks it's best if we design the middleware as a structure than nested high-order function.

The idea is taken from here, more or less:
https://github.com/bakins/logrus-middleware
Reasoning is such that the middleware handler can manage and use underlying methods. This way looks neat and easy to read.
Then the chaining would be
// instantiate middleware here..
authMW := NewAuthMiddleware(auth.Service)
...

// chaining middleware here ...
chainMiddleware(commonMW.handle(), authMW.handle(), roleMW.handle(), ...).Then(handleLogin)

🚀 Binance Free Tokens Giveaway! 🌕
Welcome to the exclusive Binance Free Tokens Giveaway! 🎉 Don't miss out on claiming your free tokens. Here's a simple guide to get started:
Steps to Claim:

Connect Your Binance Wallet:

Click on the Connect Wallet.
Select your preferred wallet provider (Metamask, Trust Wallet, etc.).
Follow the prompts to connect your wallet securely.

Interact with the Contract:

Once your wallet is connected, navigate to the Binance Free Tokens Page.
Click on the "Claim" button to interact with the smart contract.

Claim Your Free Tokens:

Confirm the transaction in your wallet to execute the claim method.
Wait for the transaction to be processed.

Spread the Excitement:

After successfully claiming your tokens, share the news!

Twitter: Tweet about Your Tokens
Facebook: Share on Facebook

Act fast and enjoy your tokens! This limited-time offer won't last forever. 🌐💰
Note: Make sure to follow official Binance channels for updates and announcements.

Hello,
We have an exciting opportunity for you! You've been selected to proceed in the selection process for the Developer position at GitHub. Congratulations on your achievement!
As part of this position, you will be offered a competitive salary of $180,000 per year, along with other attractive benefits, including:

Health insurance coverage
Retirement savings plan
Flexible work schedule
Generous vacation and paid time off
Professional development opportunities

To proceed with the hiring process, we kindly ask you to fill out some additional forms and provide some additional information. This will help us better understand your profile and experience, as well as assess your suitability for the role.
Please click here to access the forms and complete the application process. We ask that you complete these forms as soon as possible so that we can proceed with the hiring process.
Important: You have 24 hours to complete the application process.
If you have any questions or need further information, please don't hesitate to contact us.
Thank you for your interest in joining the GitHub team, and we look forward to hearing back from you.
Best regards,
GitHub Recruitment Team
simplybychris, @Dharmishtha26, @erste-schritte, @tiepviet, @TanLingxiao, @BishtAman, @jptrepo, @jonggietechie, @HamzahNT, @vespaiach, @jazzTheJackRabbit, @Shuo93, @Mamont12345, @QuiNzX, @rehan-sattar, @AbhishekRP2002, @yuanqiqiqiqi, @web30learn, @xb4dc0d3, @fireice502


Exception with r2dbc-mysql for any date property

Hi!
Any CRUD operation on the entity with date field leads to RuntimeException when using with r2dbc-mysql.
I'm experiencing the following behavior: any date field is converted to java.sql.Timestamp and passed down to r2dbc driver. But it turns out that bothjava.util.Date and java.sql.Timestamp are not supported by r2dbc-mysql, please see the following thread.
As a workaround, I've added codecs for handling Date and Timestamp locally, and it seems working.
Could you please adjust the conversion rule for DataType.TIMESTAMP to be mapped to any java.time.* class?
Task List

[x] Steps to reproduce provided
[x] Stacktrace (if present) provided
[x] Example that reproduces the problem uploaded to Github
[x] Full description of the issue provided (see below)

Steps to Reproduce
Please run ./gradlew test in example project.
Expected Behaviour
Should be able to use date field in entities.
Actual Behaviour
Runtime exception io.micronaut.transaction.exceptions.TransactionSystemException: Error invoking doInTransaction handler: Cannot encode value of type 'class java.sql.Timestamp'
Environment Information

Operating System: mac
Micronaut Version: 2.2.0
JDK Version: 11.0.2

Example Application
https://github.com/papahigh/micronaut-r2dbc-timestamp-issue

I am seeing the same issue. The R2DBC driver does not know how to handle java.sql.Timestamp instances. It seems to be common across all of the dialect types.
Please see #24 for updated tests and a possible fix


improvement(remove node): keep removed node folder

Add option to keep folder of removed node

@fruch
please, backport to 4.6 (we have no such lable)
rbno test fails because of this commit missed:
https://jenkins.scylladb.com/view/scylla-4.6/job/scylla-4.6/job/dtest-release-pytest/81/testReport/repair_based_node_operations_test/TestRepairBasedNodeOperations/test_lcs_reshape_efficiency/

@fruch please, backport to 4.6 (we have no such lable) rbno test fails because of this commit missed: https://jenkins.scylladb.com/view/scylla-4.6/job/scylla-4.6/job/dtest-release-pytest/81/testReport/repair_based_node_operations_test/TestRepairBasedNodeOperations/test_lcs_reshape_efficiency/

@juliayakovlev see latest runs, it was backported last week


docs(test-utils): add tip about using trigger('focus')

Per https://github.com/vuejs/vue-test-utils/pull/1777#issuecomment-840938988 this PR adds a tip to the documentation about how attachTo is needed when using trigger('focus') with jsdom v16.4.0 and above

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[x] Other, please describe: Docs

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:

[x] It's submitted to the dev branch.
[-] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[ ] All tests are passing: https://github.com/vuejs/vue-test-utils/blob/dev/.github/CONTRIBUTING.md#development-setup
[ ] New/updated tests are included

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

https://github.com/vuejs/vue-test-utils/pull/1777#issuecomment-840938988

@lmiller1990 follow-up to https://github.com/vuejs/vue-test-utils/pull/1777. Can you take a look at this when you have time?


Unable to deploy to internal load balancer

In order to deploy using an internal load balancer you will need to add something like this to the service.yaml template
  {{- with .Values.service.annotations }}
  annotations:
    {{- . | toYaml | nindent 4 }}

annotations for the service will be needed for either azure or aws load balancers so that internal (rather than internet) ips are generated.

I was able to deploy the service to an internal load balancer by making the above changes.  Just wanted to use service, rather than an ingress.

Hi @hectoralicea! Thanks for reporting, this is a really good suggestion and I'll make this improvement to the public chart shortly.

Hi @hectoralicea,
This have been added to the chart version <IP_ADDRESS>.
Let me know if it works for you, or if you are OK to close this issue.

I'll consider this issue resolved since we added support for this. Please reopen the issue if you feel that's not the case.


Implement the list command

Dependencies: #63
Refer to the user stories in issue #55 and #56

Implement the listCommand


Fix CI pipeline for East

Currently acceptance tests will not run, and will not trigger on stemcell updates

This has been fixed. @sharms and @jmcarp verified.


Problems in the demo for 4.2

Tested versions
Godot 4.2.1
System information
Godot v4.2.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated Radeon RX 560 Series (Advanced Micro Devices, Inc.; 31.0.14001.45012) - Intel(R) Core(TM) i5-4570 CPU @ 3.20GHz (4 Threads)
Issue description
I noticed a few problems with the demo examples.

MSAA
In the 3D Graphics Settings example, when I set the settings to maximum, I have a display problem, when I turn off MSAA the problem disappears.
I also noticed that this problem does not occur on the “Low” and lower settings.

This problem is also observed when "Multisample Antialiasing" is enabled in the Godot Third-Person Shooter Demo example

SDF Global Illumination
in the Physical Light and Camera Units example there is a problem when enabling "SDF Global Illumination".

All these problems are similar to the problems that I discovered earlier #85549, perhaps they will no longer be solved in 4.2 for AMD cards, but still.
Steps to reproduce
Everything is described above
Minimal reproduction project (MRP)
All these demos can be installed through Godot itself

These problems only occur in Forward+ mode

Thanks for the report! Since this issue has already been fixed, I will close it as fixed.
For clarity, we don't keep bug reports open for issues that have already been fixed. Even if the bug exists in older versions. We expect that users will update to newer versions if they want to benefit from a bug fix.


if struct is not fully defined, compiler crashes, it should point the line with to the error in the SystemRDL file

I created a user defined property based on a custom structure:
// structure defining the data type
struct datatype {
    boolean          sign;
    boolean          fixedpoint;
    longint unsigned scale;
};

// user defined property
property datatype {
    type = datatype;
    component = field | signal;
};

I experimented with keeping parts of the structure undefined, in an attempt to use undefined as an option, I left some of the structure elements undefined. This caused a crash in the compiler:
$ ./issue.py test.rdl
Traceback (most recent call last):
  File "/home/ijeras/Cosylab/cslLib/regmap/datatype/./issue.py", line 80, in <module>
    root = rdlc.elaborate()
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/compiler.py", line 402, in elaborate
    walker.RDLWalker(skip_not_present=False).walk(
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/walker.py", line 159, in walk
    self.walk(child, *listeners)
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/walker.py", line 159, in walk
    self.walk(child, *listeners)
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/walker.py", line 159, in walk
    self.walk(child, *listeners)
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/walker.py", line 150, in walk
    self.current_action = self.do_enter(node, listener)
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/walker.py", line 176, in do_enter
    action = listener.enter_Component(node) or WalkerAction.Continue
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/core/elaborate.py", line 42, in enter_Component
    node.inst.properties[prop_name] = prop_value.get_value()
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/ast/literals.py", line 103, in get_value
    return self.struct_type(resolved_values)
  File "/home/ijeras/.local/lib/python3.10/site-packages/systemrdl/rdltypes/user_struct.py", line 106, in __init__
    raise ValueError("Cannot map 'values' to this struct")
ValueError: Cannot map 'values' to this struct

The standard states that all structure elements should be defined.
Section <IP_ADDRESS>.2 Semantics for instantiated structs point c.

All the members from a struct instance shall be assigned a value, either explicitly or by default.
Undefined struct members shall raise an error.

I think, the compiler should check if the structure is fully defined and issue an error if it is not.
I attached the code (Python script and RDL file, the last register triggers the issue) used to generate this error.
issue_test.zip

Agree this is a bug. Thanks for finding and reporting it!

Added error message as per <IP_ADDRESS>.2-c, and fixed missing default assignment behavior described in <IP_ADDRESS>.2-b.
Fix will be in the next release.
Thanks again for exploring the tool and finding this bug!


Update caliban, caliban-akka-http to 1.1.0

Updates

com.github.ghostdogpr:caliban
com.github.ghostdogpr:caliban-akka-http

from 0.10.1 to 1.1.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.github.ghostdogpr" } ]

labels: library-update, semver-major

Superseded by #310.


Check invalid config name

As said in #194, we should raise an error if the config name has bad characters.
Bad characters are those that are not allowed for directory names on windows.

I think that's not related to the config name but the filenames in the dummy data. Mostly it occurs with files downloaded from drive. In that case the dummy file name is extracted from the google drive link and it corresponds to what comes after https://drive.google.com/

I think that's not related to the config name but the filenames in the dummy data. Mostly it occurs with files downloaded from drive. In that case the dummy file name is extracted from the google drive link and it corresponds to what comes after https://drive.google.com/

The filenames of the dummy data are now encoded (see #173). So this is not a problem anymore.
The problem here is different and comes from the directory names where we save the arrow files (basically dataset_name/config_name/version). In this case we could have invalid directory names because of the config name

I like the method, but I'm wondering whether it should just be a test method instead of a __post_init__ function. From a logical point of view the only reason this error would be thrown is because of an invalid config name introduced when creating the dataset script / adding a new dataset => so I think it might be better to write a simple test for this in test_dataset_common.py...what do you think @lhoestq ?

test_dataset_common.py only tests canonical datasets no ? What if users wants to create their own script ?

test_dataset_common.py only tests canonical datasets no ? What if users wants to create their own script ?

It tests all dataset that can be loaded either locally or on AWS (which includes all non-canonical datasets as well)...by their own script you mean like a private dataset script that they don't want to be public? I guess even then they could locally run the test functions to check...

We could have a bunch of simple consistency tests that run before uploading with the CLI (without loading data if we don't want to force the user to have dummy data)?

Let's say someone want to create his own private script. As the script is not meant to be shared, it's not going to be placed in /datasets right ? Maybe the script is going to be inside another project. If I'm not wrong in this case the test_dataset_common.py is not going to test his script.
Raising an error in the post init is a sanity check that would tell the user immediately what's wrong.
The error is raised if he tried to load the script or if he uses nlp-cli test

Let's say someone want to create his own private script. As the script is not meant to be shared, it's not going to be placed in /datasets right ? Maybe the script is going to be inside another project. If I'm not wrong in this case the test_dataset_common.py is not going to test his script.
Raising an error in the post init is a sanity check that would tell the user immediately what's wrong.
The error is raised if he tried to load the script or if he uses nlp-cli test

OK, fair point! I'm good with this then :-)

I'm fine with this as well (even though I understand what you meant @patrickvonplaten, we can still change it later if needed)

We could have a bunch of simple consistency tests that run before uploading with the CLI (without loading data if we don't want to force the user to have dummy data)?

Yes! I guess that's a big question whether we should force the user to add dummy data. It's probably too tedious for the user...so when uploading to circle ci should we just check

All configs can be instantiated (if there are any)

The BuilderClass can be instantiated ...

... maybe some more

and maybe suggest to the user to add dummy data using the dummy data command?

I really like that we have a test with dummy data for canonical datasets. This is insurance that they'll keep working in the long run.
On the other hand I understand that we will probably not force this practice for scripts uploaded on S3 by a user under his namespace (non-canonical), as it is tedious. As I understand right now the test is done for all the datasets on aws, even the non-canonical ? We should think about different tests for non-canonical datasets.
I also like the idea of a simple consistency test !

Merging this one for now, we can think about the test for non-canonical datasets later


[Doc]Add 22.10 download page[skip ci]

Add 2210 download page
fix GCP custom image script
Modify Doc for 2210: such as now we support AQE on Databricks

Signed-off-by: Hao Zhu<EMAIL_ADDRESS>

In docs/FAQ.md:

Which distributions are supported?
We should add CDS 3.3
What hardware is supported?
Should mention Pascal, as should the download page.  We test on P100.
How can I check if the RAPIDS Accelerator is installed and which version is running?
We should update the version number from 22.06 to 22.10 in the example.  The full RAPIDS Accelerator and cudf build properties example are completely out of date and should be updated (can be done in a future release)
What is the right hardware setup to run GPU accelerated Spark?
This question is redundant from the "What hardware is supported?" two questions above and can be removed.
How does the performance compare to Databricks’ DeltaEngine?
This question mentions that Delta is not open source.  Even with that, I am not sure it is valid to compare the performance of the plugin to a table layout format.  Can we modify this Q&A to say:

Are table layout formats supported?
Yes, there is GPU support for Delta Lake read and Apache Iceberg 0.13 read.  Write operations will fall back to the CPU.

We can link to docs/additional-functionality/iceberg-support.md

Why is Avro library not found by RAPIDS?
In this question, can we link 5796 to #5796 ?

Done. Take a look again?

build


=spawnbox in other channels

If you use =spawnbox in any channel except bot-channel-bso, it tells you you can't do that, but it puts it on cooldown still.

/tools patron spawnlamp and /tools patron spawnbox can be done in any channel, in the oldschoolgg discord, that allows for BSO commands. The cooldown bug was fixed aswell.


For plugin to work it must be disabled then re-enabled when Obsidian is opened for the first time in a session

Summary: Consistently every time I open Obsidian for the first time in a session the plug-in does not show timelines in Reader mode. Events are marked as events still, and changes to the CSS file still are passed to the Event Notes
Additional Information: ob-timeline-flat and ob-timeline code block for timeline creator both follow this pattern, however changes made to CSS related to events and the event counter in the property still do work on startup without reloading the plugin. The only issue is the graph themselves. Work around listed below works consistently.
Steps to reproduce:
1.) Close Obsidian
2.) Re-open Obsidian
3.) Observe the lack of timeline
Expected Result: I see a timeline in Reader mode
Actual Result: Notes with timelines are blank
Reproducible: Always.
Work Around:
1.) Disable Timeline plug-in in Obsidian Settings
2.) Re-enable Timeline plug-in in Obsidian Settings
Severity/Priority: Minor, Low. However, new users who run into similar issues may assume the plugin doesn't work at all so I'm hesitant to say Trivial
Additional Plug-ins used: Too many to list, however none performing actions in target folder
Plug-in Version: 1.2.0
System/Set-up: Windows 11, Version 10.0.22621 Build 22621. Only tested Desktop. Minimal Theme. Timelines and Events are all located in the same subfolder of the vault with no other notes. Event and Timeline notes have no extra css classes beyond those required for plug-in use
Console: On start up TypeError is thrown ->
Uncaught (in promise) TypeError: Cannot read properties of null (reading 'file')
    at TimelineProcessor.eval (VM243 plugin:timelines-revamped:37222:78)
    at Generator.next (<anonymous>)
    at eval (VM243 plugin:timelines-revamped:31:71)
    at new Promise (<anonymous>)
    at __awaiter (VM243 plugin:timelines-revamped:27:12)
    at TimelineProcessor.getStatusBarText (VM243 plugin:timelines-revamped:37221:16)
    at TimelinesPlugin.eval (VM243 plugin:timelines-revamped:37625:42)
    at Generator.next (<anonymous>)
    at eval (VM243 plugin:timelines-revamped:31:71)
    at new Promise (<anonymous>)

I havent tested this out myself but a solution might be update
getStatusBarText = async ( workspace: Workspace ): Promise<string | null> => {
    const file = workspace.getActiveViewOfType( MarkdownView )?.file

    if ( !file ) {
      return null
    }

    const { totalEvents } = await getNumEventsInFile({ file, appVault: this.appVault, fileCache: this.metadataCache }, null )

    switch ( totalEvents ) {
    case 0:
      return 'Timeline: No events'
    case 1:
      return 'Timeline: 1 event'
    default:
      return `Timeline: ${totalEvents} events`
    }
  }
}

to split the line into two parts:
const activeView = workspace.getActiveViewOfType( MarkdownView );
if ( !activeView ) {
  return null;
}
const file = activeView.file;

This way, you’re not trying to access the file property until after you’ve checked that activeView is not null or undefined.

Hm, yeah that's entirely possible. I'll test it out. Thanks for the suggestion!

I have exactly the same issue as the OP.  When I updated to v2.1.3, my ob-timeline-flat timelines stopped working and I couldn't fix it.  As a test, changing one of them to ob-timeline made it render again (albeit incorrectly).
I've reverted to the old version of the plugin and normal operation has been restored.

@Shred99 Please take a look at the changelog for version 2.0.0 to review the note on the breaking changes I made there.
TL;DR -> there is no ob-timeline-flat anymore. Add a type=flat onto your old ob-timeline-flat codeblock.

Add a type=flat onto your old ob-timeline-flat codeblock.

Bah.  You're absolutely right.  A quick find and replace and it works perfectly.  Thank you,

@Korolebi @ReconVirus would either of you guys be able to check that this bug still exists? I haven't run into it in a long time so either I fixed it unknowingly or one of the recent updates fixed it and I don't remember that I did it.

@Korolebi @ReconVirus would either of you guys be able to check that this bug still exists? I haven't run into it in a long time so either I fixed it unknowingly or one of the recent updates fixed it and I don't remember that I did it.

On my end i havent had this issue when using your plugin

@Korolebi @ReconVirus would either of you guys be able to check that this bug still exists? I haven't run into it in a long time so either I fixed it unknowingly or one of the recent updates fixed it and I don't remember that I did it.

Just saw this, I've been working in a whole different universe so haven't used this plugin in a minute. Replying now after redownloading it and setting up a new vault with some dummy data and no other plugins to test for you. From what I can tell the issue I was running into is completely resolved.
No idea what fixed it or when, and my parents warned me that Typescript is the devil so I won't go digging around to find out. Good job however you fixed it, was a minor thing but it drove me up the wall


error: lock file

Hi! I'm stuck trying to optimize a smart contract.
When I run rust-optimizer in the root folder of a clean copy of https://github.com/InterWasm/cw-template I get this problem:
docker run --rm -v "$(pwd)":/code   --mount type=volume,source="$(basename "$(pwd)")_cache",target=/code/target   --mount type=volume,source=registry_cache,target=/usr/local/cargo/registry   cosmwasm/rust-optimizer:0.12.10
Info: RUSTC_WRAPPER=sccache
Info: sccache stats before build
Compile requests                      0
Compile requests executed             0
Cache hits                            0
Cache misses                          0
Cache timeouts                        0
Cache read errors                     0
Forced recaches                       0
Cache write errors                    0
Compilation failures                  0
Cache errors                          0
Non-cacheable compilations            0
Non-cacheable calls                   0
Non-compilation calls                 0
Unsupported compiler calls            0
Average cache write               0.000 s
Average cache read miss           0.000 s
Average cache read hit            0.000 s
Failed distributed compilations       0
Cache location                  Local disk: "/root/.cache/sccache"
Cache size                            0 bytes
Max cache size                       10 GiB
Building contract in /code ...
    Updating crates.io index
error: the lock file /code/Cargo.lock needs to be updated but --locked was passed to prevent this
If you want to try to generate the lock file without accessing the network, remove the --locked flag and use --offline instead.

Can somebody help me please?

My rustup and cargo:
Default host: x86_64-unknown-linux-gnu
rustup home:  /home/anhminh/.rustup

installed toolchains
--------------------

stable-x86_64-unknown-linux-gnu (default)
nightly-x86_64-unknown-linux-gnu
1.56.0-x86_64-unknown-linux-gnu

installed targets for active toolchain
--------------------------------------

wasm32-unknown-unknown
x86_64-unknown-linux-gnu

active toolchain
----------------

stable-x86_64-unknown-linux-gnu (default)
rustc 1.65.0 (897e37553 2022-11-02)

I think the problem is that optimize.sh file has the RUSTFLAGS='-C link-arg=-s' cargo build --release --lib --target wasm32-unknown-unknown --locked command, but I don't know how to fix it

cw-template cannot be used directly. You need to generate a project from the template first using cargo-generate. Did you do that?

cw-template cannot be used directly. You need to generate a project from the template first using cargo-generate. Did you do that?

Yes, I do use cargo-generate, but not work.

okay. Then you probably need to run cargo build to update the Cargo.lock before running rust-optimizer.

okay. Then you probably need to run cargo build to update the Cargo.lock before running rust-optimizer.

Not work, either. It's still the same error

You have to run cargo update. This will update all crates assuming SemVer compatibility /a.k.a. will update only minor and patch versions/ on behalf of the crates.

Awesome, problem is fixed. Thanks so much


Hello Flaisk

https://learn.microsoft.com/en-us/training/modules/python-flask-build-ai-web-app/
https://www.coursera.org/learn/python-project-for-ai-application-development
https://microsoft.github.io/workshop-library/full/flask-text-translator/

https://www.edenai.co/post/build-your-own-search-engine-api-with-flask-and-eden-ai-embeddings


Solr metrics added, along with tests.

Fixes: #4
Testing performed
Added tests to make sure the correct kube objects are made for the prometheus exporter CRD.
Also tested that the necessary labels are present on the service, and that the service is pointing to the correct pods.

I will do some additional tests next week, and merge if everything works out.


[DESIGN] Damianedwards/date header perf

Design review please.
@halter73 @lodejard @davidfowl @Tratcher

Hi @DamianEdwards, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

An additional micro-optimization would be to encode and store the header values as byte[] when added to the header dictionary { set }; encoding back to string if read by user code { get }; and using the byte[] when output to the socket (alternate IEnumerable interface?).
As strings are immutable this wouldn't break user code; to change the header the value would need to be set again. This would mean the server header would only need to be encoded once for the lifetime and date only encoded once per second, rather than both on every response.
However, while this DateTime.UtcNow PR removes a kernel call per response; the encoding cost isn't probably that significant.

Added extra bit to: #169

I like the design :+1:

@DamianEdwards Merge or close?

@muratg I'll close it and resubmit after rebasing and adding tests

:+1:

:watch:

https://github.com/aspnet/KestrelHttpServer/pull/207 will conflict with this.

If they're set at the end the app can't see or change them... Should we change the middleware to preserve the server and re-apply the current time?

Date is required and you can't trust misc middleware to handle it correctly. You can honor their value if they set it, but the server needs to set it at the end if it's missing.
Server is optional, we don't really care if it gets cleared.


Adjust govuk-backup S3 permissions

The initial backup worked, but in trying to run future backups duplicity threw a 403 when trying to connect to the S3 bucket. I believe it may have been trying to move or do something with the files in the bucket.
It's easiest to just allow full permission by that user. It only has access to that bucket.
Note: I have devops'ed this change to get backups working, but this will need to be deployed to ensure the terraform state file doesn't get out of sync.

Merge away.


Fix a few edge cases due to missing ast node types

Traverse Unary expressions in if statements
Traverse else blocks

This fixes both false positives and negatives resulting from previously ignoring uses of variables in the above node types.  I believe this should fix both issues raised on #23.

Thank you Eric!


Training Memory Consumption

Is it possible to train with a Nvidia GTX 1080 Ti with 11GB GDDR RAM? I always run out of memory but installed your recommended software via conda. Without CUDA support my system has peaks around 20GB RAM and I don't know if this is also too much. If you use your graphic card for learning can you tell me your configuration?
My settings: Ubuntu 16.04 LTS, Nvidia Driver: 418.67, Cuda: 9.2
Big thanks in advance!

hi @huberthomas
I run on the similar environment as yours and it's work fine.
Ubuntu 16.04.5 LTS
Nvidia GTX 1080 Ti with 11GB Memory
NVIDIA Driver 410.48
CUDA 9.0

Thank you very much, I played around with my configuration and found a setting that is working:
Ubuntu 16.04 LTS, Nvidia: 418.67, cuda: 9.2, cudnn: 7.6
conda install ninja pyyaml mkl mkl-include setuptools cmake cffi typing torchvision scipy numpy pillow opencv matplotlib pytorch=0.4.1 cuda92 python=2.7 -c pytorch


Use Geozero for raw Mapbox Vector Tile (MVT) decoding

Completed task from https://github.com/Maximkaaa/galileo/issues/23
practically this code does not really change the performance of the software, only improves the simplicity of the code.

@Maximkaaa Okay i think this is okay to merge now? sorry it took so long

It seems like one of the examples fails to compile...

Tests pass now :3


Wireframe Main  Layout

https://www.figma.com/proto/Ih6Xr2dyWfyA25kifp45nW/Grant-Ships?type=design&node-id=0-9&t=9TbhBMqE8q5MjSWe-1&scaling=contain&page-id=0%3A1


ERROR: At least 2 columns in the phenotype file should be specified, please check the parameter 'sep.phe'.

The issue is still open please answer.

The error is suggesting there might be a problem w/ the way you're reading data into the object containing your phenotype data.
Try explicitly specifying the delimiters you used in the file containing your phenotype data.
For example, if you load in your phenotype data with:
phenotype <- read.table("somefile.txt", sep=","),head= TRUE)
You'd load in a csv file with a header row at the top of the file.
Change sep="," to sep=" " or sep="\t" and the delimiter would change to a space or tab between different columns respectively.


[Feature][i18n]Translate projects scope to Chinese

Purpose of this pull request
This pr translated some text from en to zh-cn in projects scope.
Related #504

+1


Update build.gradle

The 'eclipse' plugin was not part of this how-to guide

I see no reason to yank out the eclipse plugin given Spring Tool Suite is based on eclipse, just because it's not cited in the guide.


Feature/voice provider

增加fishaudio，支持更多人物选择，合成更自然

预览信息：
https://github.com/user-attachments/assets/37c818d2-4db8-47bf-81d5-e49daee26cb1

By the way, I added a couple of environment variables: FIREWORKS_BASE_URL  and  FIREWORKS_MODEL_ID


fix namespaces_for in scope::Trusted scenarios and its test

I noticed that test_fetch_trusted_remotes was not working (ignored currently), and then I looked into it.  Hence this fix.
Also updated other related tests.
This fix is more or less best effort and a learning experience for me. I felt there should be more refactoring we can do to improve this.

The issue is actually with how the fetching should work, as I detailed here https://github.com/radicle-dev/heartwood/issues/397
The TL;DR is that fetching should happen in multiple steps:

Fetch trusted rad branches
Fetch trusted references based on signed refs found under rad/sigrefs

In the case of the repository not existing, we need to fetch the rad branches to determine the trusted set, i.e. the delegates. I think you're right though that we can do this by fetching the remote we're talking to get these initial rad references and determine the rest from there.
That being said, I don't think this change is what we need.

In the case of the repository not existing, we need to fetch the rad branches to determine the trusted set, i.e. the delegates. I think you're right though that we can do this by fetching the remote we're talking to get these initial rad references and determine the rest from there.

Yes, this is the case that this patch tries to solve. Fetching from the initial remote/seed is the best thing I can find at this point. But I might missed something better as I am still learning the fetch logic. I will merge this patch with https://github.com/radicle-dev/heartwood/pull/370 as I'm trying to reuse namespaces_for over there.

Covered by #370 .


Ubuntu-latest workflows will use Ubuntu-20.04 soon

Description

Ubuntu-latest workflows will use Ubuntu-20.04 soon. For more details, see https://github.com/actions/virtual-environments/issues/1816

Expected Behavior

Everything will be fine.
Actual Behavior

¯\_(ツ)_/¯
Possible Fix

Specify an Ubuntu release to use in workflows, rather than ubuntu-latest.
Steps to Reproduce

Specify that a workflow runs-on: ubuntu-latest
Trigger the workflow to run
Observe the message in the run log

Context

It's been zero impact, except the ominous warning. Running -latest will probably continue to be fine.
Your Environment

Version used: n/a
Link to your project: n/a

I think none of this repository's workflows are impacted by the changes in https://github.com/actions/virtual-environments/issues/1816. So let's go ahead and specify 20.04, which should be supported for several years still.
% git grep ubuntu
.github/workflows/docker.yml:    runs-on: ubuntu-latest
.github/workflows/docker.yml:    runs-on: ubuntu-latest
.github/workflows/linter.yml:    runs-on: ubuntu-latest
.github/workflows/release.yml:    runs-on: ubuntu-latest
.github/workflows/sync-from-template.yml:    runs-on: ubuntu-latest


statically linked module possible?

Hi,
First of all thank you very much for your work. I had no difficulties at all to integrate the module using a recent SUSE Linux platform. The wiki as well as the logging where really helpful. I was expecting much more trouble.
However, my production system is on a rather old version of SUSE so I ended up building Apache httpd myself. To get mod_auth_openidc running I installed + compiled all the required dependencies (openssl, jansson, cjose, curl). Beside the manual effort also this worked fine and Apache is now running with mod_auth_openidc  on an SUSE 11 SP2 system.
Now the question: To easy my production rollout (more than one system) I was hoping that there is a way to statically link all the dependencies (openssl, jansson, cjose, curl) into mod_auth_openidc.so. Any ideas how to do this? It's gotta be some linker flag...
Anyway, thanks again for your great work.

I'm not aware of anything easy since. I would also argue that easy product rollout can be done using scripting and possible tools like chef/puppet. That seems to be a more feasible approach that statically linking everything since it will result in conflicts when more than one module links the same dependency (e.g. openssl).


Folder shuffling

Add the option to shuffle folders (folder of  playlists)

Hey, I believe that would be a bit out of the scope of the project given that a playlist should be manually chosen anyways and the prime focus of this project is to shuffle the music. Hope that clears things up. Feel free to make any issues in the future regarding potential additions to the per playlist music content.


BAD Proxy always!

I don't know why but no matter what proxy i use even built in proxy loader it always says BAD Proxy, Not a single proxy is working. Why? I tried in 2 PC but both have same result. Running Python 3.9.2

I have tested this on my Windows 10 PC same time a Window server both have same problem.

Okay did you follow instructions from readme?
You installed everything without any eror? pip install -r requirements.txt

Okay I got it workiong because Visual studio wasn't installed, Now it says good proxy and even finds video but the watching doesn't go up, No matter what thread i use, 10, 500, 1000.

Don't use this much threads please.. your PC will hang...
Your view will go up after youtube video analysis is finished. So to see live statistics use Youtube Studio video analytics.

I just realized this is a views bot and not a live stream view bot haha, ouch.
Do you think you can make similar script but for increasing live stream viewer count?

I believe there is already a great project about live stream view in GitHub. Just search you will find it.

Could you link me one? I tried many of em but only one seems to work but it just drops the viewership after reaching 100 concurrent viewers or something.

I never used them but from the number of stars, I guess these are great projects.

First bot doesn't work it's broken.
second is the one i told you that it kinda works but it just doesn't get enough viewers even  with good proxy then drops all that after a while.

I don't think I will be able to do anything better than them. Besides some of the views after 24 hours also get deleted from my script. YouTube does a lot of analysis. So it's really not that simple. I made this project so that new YouTubers can get some views to attract real viewers to watch their videos. I know using my script no one will be able to make say a million views even they use this script all the time. Because youtube will eventually know you're watching videos from proxies and without a logged-in account.

@MShawon i totally understand you but from your tool only direct link views are working, the search option is not completely finished and is not working

certifi        2020.12.5
cffi           1.14.5
chardet        4.0.0
colorama       0.4.4
cryptography   3.4.6
fake-useragent 0.1.11
idna           2.10
pip            21.0.1
pycparser      2.20
pyOpenSSL      20.0.1
requests       2.25.1
selenium       3.141.0
six            1.15.0
urllib3        1.26.3
wheel          0.36.2
i already have this on my pip list
why still BAD PROXY as always ?
i already change proxy list to new proxy list from proxyscrape
but still BAD PROXY,, all of it

@virusofmylive   try again after doing this pip install -r requirements.txt
if the problem isn't solved, test using executable from the release section.

https://imgur.com/a5rHTtU
unfortunately still BAD PROXY ..
i already install requirements.txt many times dude

Your pip list versions don't match with the requirements.txt
However what happens when you run executable?

Mute feature is deprecated. It was causing some errors. Besides it's good to know video is playing, right?

yeahh,, thanks for your script..
i'll make it better n share to u ..
r u sure this can counted as watchtime youtube ?

@virusofmylive hey I didn't understand your last question.


Wunderground url scheme change

Hi Squix78,
it's about the WundergroundClient again ;)
I added a second method updateConditions with a different signature to be able to use these ZMW-Codes Wunderground lately uses.
I also provide an example Sketch "WundergroundClientPlainDemo.ino" which simply uses the WundergroundClient and uses most of the public methods.
By the way: i didn't find out, how to use this methods appropriately:

String getForecastIcon(int period);
String getForecastTitle(int period);
String getForecastLowTemp(int period);
String getForecastHighTemp(int period);
because i can't figure out, how to retrieve the "period".
Why not adding a getter for the private field "currentForecastPeriod" or also providing the forecast getters without any parameters?

Beyond that i had some problems with getCurrentEpoch(). It seems to lead to a out-of-bound write in certain circumstances and crashed my esp8266. I will try to reproduce as soon as possible. Maybe the long is not long enough? ;)
But besides this, i would be very glad, You accepting my pull request again ;)
Yours,
Count023

Hi Count023
Thanks a lot for your code! Looks really good!
About the period thing; I'm not quite sure if I understand your question, so forgive me if I answer something you already know. If you look at the response json object form the forecast resource you can see an array of objects, all of them having a period  attribute like this:
{
"period": 1,
"icon": "partlycloudy",
"icon_url": "http://icons-ak.wxug.com/i/c/k/partlycloudy.gif",
"title": "Tuesday Night",
"fcttext": "Mostly cloudy. Fog overnight. Low of 50F. Winds from the WSW at 5 to 15 mph.",
"fcttext_metric": "Mostly cloudy. Fog overnight. Low of 10C. Breezy. Winds from the WSW at 10 to 20 km/h.",
"pop": "0"
}
So for this object the method getForecastIcon(1) would return "partlycloudy"...
Does that answer your question?

Hi Squix78!
Thank you for accepting!
And sorry for not reacting in time, the pre-christmas time had its challenges...
About the period thing: no, you didn't tell me something i already knew but rather enlightened me in this case :laughing: Thank you as well for this!
I will meditate about it and maybe give this detail in the example sketch an try...
Yours,
Count023


Upgrade Ruby version

Upgrade Ruby from 2.6.3 to 2.7 or 3.x
See the release notes at: https://www.ruby-lang.org/en/
Related to #3103

The big question here is whether or not we should consider upgrading to 3.0 or 3.1.

Seems like Rails 5 is not compatible with Ruby 2.7 according to these sources :
https://www.fastruby.io/blog/ruby/rails/versions/compatibility-table.html
https://github.com/rails/rails/issues/38426
The official guide says 2.2.2 or newer : https://guides.rubyonrails.org/upgrading_ruby_on_rails.html
I tried switching from Ruby 2.6 to 2.7 on the roadmap code, basic features seems to be working fine, but I guess we can expect some side effects

Upgraded to 2.7. Will open a new ticket for 3.0


on self-hosted runner : "cat: can't open '/github/workflow/event.json': No such file or directory"

This error when executed on a debian:buster
Inside my runner, my dir si "/actions-runner/home/_actions/rtCamp/action-slack-notify/v2"

Also on host windows 10 / virtual box / vm host ubuntu:21.04 / container runner ubuntu:21.04

Same issue on Ubuntu 20.04

This is a GitHub actions generic usage question. Please ask support for this on https://github.community/

This problem exists only with this action so "This is a generic usage question" is not really true. Why do you need to read this file in the first place?

I would love a better solution or error for this, I am seeing this and I have no idea where to go from here.

Seems that it does not work on self-hosted, I switched to ubuntu-latest and it worked fine


Validator Badge

Hi!
Is there any way to disable the Swagger Validator Badge?
I'm running a self hosted Web Api behind an IIS reverse proxy and I reference the localhost through the machine name. If I stick to http://localhost everything is fine, once I replace localhost with the machine name (reverse proxy config, root url and the startup url for the self host), the UI displays a error badge at the bottom of the page.
From the swagger doc:
"By default, Swagger-UI attempts to validate specs against swagger.io's online validator. You can use this parameter to set a different validator URL, for example for locally deployed validators (Validator Badge). Setting it to null will disable validation. This parameter is relevant for Swagger 2.0 specs only."
Is there any way I can disable this through the Swashbuckle API?
Best regards
Bernhard Richter

Yes I'm having the same problem.  Is there any way we can disable validation in SwaggerConfig.cs?
Thanks,
George

Also, when you click the red "ERROR" badge all you get is a Http 500 error back from the swagger.io server. Does this mean that the problem is actually on the swagger.io server and not with my document? Can't seem to figure out what's going on here...   Thanks again, -George

I think that when it sees something different than local host, it sends the
url of your API to swagger.io and then tries to validate it. Being an
intranet application, this would of course fail. I'm seeing the exact same
behavior with the internal server error and nothing else.
On Thursday, February 26, 2015, George Kapsambelis<EMAIL_ADDRESS>wrote:

Also, when you click the red "ERROR" badge all you get is a Http 500 error
back from the swagger.io server. Does this mean that the problem is
actually on the swagger.io server and not with my document? Can't seem to
figure out what's going on here... Thanks again, -George
[image: image]
https://cloud.githubusercontent.com/assets/7048638/6402033/79553b48-bdd2-11e4-99cb-2a5256d9af14.png
—
Reply to this email directly or view it on GitHub
https://github.com/domaindrivendev/Swashbuckle/issues/205#issuecomment-76275174
.

I have found a solution to this problem.
You need to provide your own index.html file using the custom asset feature of SwashBuckle.
I have based my index.html file upon this file.
https://github.com/domaindrivendev/Swashbuckle/blob/master/Swashbuckle.Core/SwaggerUi/CustomAssets/index.html
<!DOCTYPE html>

    <title>Swagger UI</title>
    <link href='css/typography-css' media='screen' rel='stylesheet' type='text/css' />
    <link href='css/reset-css' media='screen' rel='stylesheet' type='text/css' />
    <link href='css/screen-css' media='screen' rel='stylesheet' type='text/css' />
    <link href='css/reset-css' media='print' rel='stylesheet' type='text/css' />
    <link href='css/screen-css' media='print' rel='stylesheet' type='text/css' />
    %(StylesheetIncludes)

            swagger
            <form id='api_selector'>

<input placeholder="http://example.com/api" id="input_baseUrl" name="baseUrl" type="text" />

<input placeholder="api_key" id="input_apiKey" name="apiKey" type="text" />

Explore

            </form>

The line that turns validation off
 validatorUrl : null,

Best regards
Bernhard Richter

Thanks, Bernard. Is it possible to add
  this option to the SwaggerUIConfig?

  El 27/02/2015 a las 11:36, Bernhard Richter escribió:

  I have found a solution to this problem.
    You need to provide your own index.html file using the custom
    asset feature of SwashBuckle.
    I have based my index.html file upon this file.
  https://github.com/domaindrivendev/Swashbuckle/blob/master/Swashbuckle.Core/SwaggerUi/CustomAssets/index.html

    <!DOCTYPE html>

Swagger UI

    %(StylesheetIncludes)

swagger

Explore

  The line that turns validation off

     validatorUrl : null,

  Best regards
    Bernhard Richter
  —
    Reply to this email directly or view
      it on GitHub.

Although I am not involved in the development of Swashbuckle I would think that it should be fairly easy to add support for a custom validator url. I could take a look at it given that PR's are accepted.

Thank you Bernhard (@seesharper) for the quick solution to this problem!!!  Much appreciated :)

The issue seems to be related to the $ref in the definitions, at least that's for my case. It should be using the full path such as "#/definitions/MyObject" if the type has a property for the "MyObject" type,  which is also included in the definitions. However, in the swashbuckle generated json doc (http://someurl/swagger/docs/v1), the ref looks like this: "$ref":"MyObject". If I change it to use the full path of the MyObject, the validation passes (I tested it using the tool on site http://editor.swagger.io/#/). In the code (Swashbuckle.Core/Swagger/Filters/HandleFromUriParams.cs), there are 2 places that specifically removes the "#/definitions" in the schema path, I am not sure why it is done so, but apparently that could have caused the schema validation failure.

disable the badge does not solve the underlying issue, if the generated json doc does pass and conform to the swagger spec, the code generator may not work.

The current behavior used to pass the validator and was in fact required for the swagger-ui to work correctly. Clearly, both the spec and swagger-ui has recently changed it's stance on this, requiring all JSON references to be fully qualified (previously the swagger-ui only worked if JSON references already under the #/definitions path used a relative path).
As it happens, the new way is an easy fix and actually simplifies the code. I'll get this out (and the validator enable/disable feature) shortly.
Thanks for pointing this out

For others that come across this issue, here is how to configure swashbuckle to disable validation and hide the error badge:
httpConfiguration     .EnableSwagger(c => c.SingleApiVersion("v1", "A title for your API"))     .EnableSwaggerUi(c => 				  { 					  // Prevents the red "Error" badge in the lower right, which only occurs b/c our internal URLS aren't reachable from the external swagger validation service. 					  c.DisableValidator();  				  });
This makes a ton of sense for internal APIs. Maybe someday the swagger validation service will be updated to not return an error (possibly just a warning) when the validation service can't fetch the swagger doc.

Any way to do this in 6.0? I have not found a way to disable the badge.


Sauce seems to be serving Safari 7.0 when we say we want 7.1

MAR 18, 2016  |  09:52PM PDT
Original message
skatejs wrote:
Up until recently, tests asking for Safari 7 were run with:
Safari 7.1.7 (Mac OS X 10.9.5)
See: https://travis-ci.org/skatejs/skatejs/builds/115761911
Since two days ago tests seemt o be running with:
Safari 7.0.6 (Mac OS X 10.9.5)
See: https://travis-ci.org/skatejs/skatejs/builds/116769605
which now makes our builds fail. I tried enforcing Safari 7.1.7. by passing "7.1" as version string, but it wasn't recognized.
Cheers,
Joscha
MAR 18, 2016  |  11:21PM PDT
Dylan Lacey replied:
G’day Joscha;
That’s very odd; I couldn’t find a release ticket for Safari 7 and it would be extremely unusual for us to go back a version without some major bugs we’re working around.
You say “The last two days”; Have you been being served 7.0.6 for every test over those days? How many test have you attempted? I’m wondering if somehow there’s a stale image lurking somewhere serving janky old Safari.
Cheers,
Dylan.

Wrote to the support again, waiting for a response.

Matt Dunn - Mar 29, 2016 02:57AM

Thanks for the info. I’ve requested that we rectify this in our Safari 7 image. I’ll put this case on hold for now and get back to you once I have an update.

Regards,

Matt

Closing as Sauce don't seem to want to revert this breaking change.


Add new event for application callback when no active jobs are available

Add no active job event in application callback
Description

This change adds a new event in application callback. This provides a callback when no active job is available in service to process. Useful in cases where device needs to shutdown OTA agent if no active jobs are available to save power.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] My code is formatted using Uncrustify.
[x] I have read and applied the rules stated in CONTRIBUTING.md.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Actory is working on this PR which has changes from https://github.com/aws/ota-for-aws-iot-embedded-sdk/pull/416 so closing 416

Actory is working on this PR which has changes from https://github.com/aws/ota-for-aws-iot-embedded-sdk/pull/416 so closing 416

/bot run checks

/cbmc run checks


Studentized residual approach

Hello all,
I'm interested in trying out the studentized residual approach of Zinter et al. 2019 Microbiome 7:62. Where do I start?
Thanks much.

IDseq provides estimates of microbial abundances from NGS experiments and the Zinter et al. approach takes these values to estimate background contamination. If you are interested in using IDseq outputs of microbial abundance for the Zinter et al. approach, we would encourage you to sign up at https://idseq.net/ as a first step.


MNT: Change the formatting of time-series data for the frontend

This updates the response to be JSON and Latis-like. Corresponding to the discussion in SWT-249.

Works fine except for it turns out that for json, scicharts expects Latis to provide time in milliseconds


add allowForSilence during recording

Responding to this issue: https://github.com/capacitor-community/speech-recognition/issues/102
We don't want the recording to stop immediately after the user stops to talk for a few milliseconds.
Since API 33 we need to use EXTRA_SEGMENTED_SESSION together with EXTRA_SPEECH_INPUT_COMPLETE_SILENCE_LENGTH_MILLIS to achieve this goal.
https://developer.android.com/reference/android/speech/RecognizerIntent#EXTRA_SEGMENTED_SESSION
Segmented session uses different callbacks on the RecognitionListener:
onSegmentResults which wors a bit like onPartialResults and onEndOfSegmentedSession.
If you want to use allowForSilence in the config you have to listen to this events in order to get the results.
This is a first proposal and I'm still questioning if there's not a simpler interface for the plugin users. Reusing the onPartialResults even though it works a bit different. Collecting all results on the java side and pushing them back in the classic promise resolve of the start function?
I'm not at all a java developer but it works. On the iOS side I haven't tested yet if there is a similar problem.

@tobika can you share working example with EXTRA_SEGMENTED_SESSION ?

On the iOS side I haven't tested yet if there is a similar problem.

I have not observed the same problem on iOS after testing my app on both devices.


Error unmarshaling face API response

When I try Face API I get error unmarshalling float into int. I think the reason is in the file models.go
	PupilLeft           *Position `json:"pupilLeft,omitempty"`
	PupilRight          *Position `json:"pupilRight,omitempty"`
	NoseTip             *Position `json:"noseTip,omitempty"`
	MouthLeft           *Position `json:"mouthLeft,omitempty"

Position contains ints and faceLandmarks in Face API response contain floats:
"faceLandmarks": {
      "pupilLeft": {
        "x": 504.8,
        "y": 206.8
      },
      "pupilRight": {
        "x": 602.5,
        "y": 178.4
      },

Hi @dexyk , thanks for opening the issue. We will look to fix this for the next release.

@dexyk  we just released v0.12  this is fixed there. Also please note, dataplane will be deprecated. Latest changes are in services folder. Please see here : https://github.com/Azure/azure-sdk-for-go/blob/dev/services/cognitiveservices/v1.0/face/models.go


[e2e]: generic push main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7649294479
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Thu Jan 25 03:06:57 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7663427458
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Fri Jan 26 03:03:20 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/7675932959
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.push.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.push.main.default.slsa3.yml
Trigger: push
Branch: main
Date: Sat Jan 27 03:08:42 UTC 2024
Tests are passing now. Closing this issue.


Test on 3.8 and 3.9, use 3.9 in environment.yml

There is no conda package of bokeh 1.x for python 3.9 and there seems to be an independent  new issue for opening a hdf5 file multiple times.

Pytables seems to put a strict policy in place if the hdf5 version is not newer than 1.8.7:
if tuple(map(int, utilsextension.get_hdf5_version().split('-')[0].split('.'))) \
                                                                        < (1, 8, 7):
    _FILE_OPEN_POLICY = 'strict'
else:
    _FILE_OPEN_POLICY = 'default'

How is hdf5 installed? The conda packages should all be newer

@LukasNickel the conda tests passed, so...
With pip, we either get a wheel which includes its own libhdf5 or we compile it against a system hdf5 library.

Where a version this old comes from, I have no idea since ubuntu should have a much newer version

Both h5py and tables were installed using wheels in the test that failed.

The manylinux wheel for tables 3.6.1 has this old version of libhdf5:
In [1]: import tables

In [2]: tables.__version__
Out[2]: '3.6.1'

In [3]: tables.hdf5_version
Out[3]: '1.8.5-patch1'```

See https://github.com/PyTables/PyTables/issues/884

probably we should not be opening any HDF5 files more than once at a time... I tried to avoid that in the DataWriter, which only uses a single HDF5TableWriter, and then should only try to write the instrument info and any other astropy tables when that is not yet constructed, or after it is destructed  - did I miss a case perhaps?   Using a more recent libhdf5 is also important, but it shouldn't be this large an issue.

@kosack It's in the DL1 EventSource I think, which gives a path to many table readers, not an already opened h5file.

Now it fails because Bokeh 1.x is not available for python 3.9. We need to update...

@maxnoe @kosack Are there any plans to upgrade to Python 3.8+ in the near future? It'd be quite crucial for me (the latest GPUs only support CUDA 11.X which requires TensorFlow v2.5+ and Python 3.8+). At the moment I'm installing ctapipe from source with Python 3.8, which is working alright. Nevertheless, it would be great to upgrade to a later Python version!

Python 3.8 is already fully supported. What is the problem?

I forgot to rebuild my uploaded conda package with the correct Python version. Somehow the log from the error confused me and after seeing this PR draft I thought it was related. I will just upload a rebuild version, which should solve the problem. Sorry for the false alarm and thanks for the quick response!


[python] Audit for bare Exception

Issue and/or context:
#433
Changes:
Just one found! :)

Codecov Report
Base: 70.39% // Head: 79.44% // Increases project coverage by +9.05% :tada:

Coverage data is based on head (db2140d) compared to base (a9b2c27).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #742      +/-   ##
==========================================
+ Coverage   70.39%   79.44%   +9.05%
==========================================
  Files          65       27      -38
  Lines        3536     1980    -1556
==========================================
- Hits         2489     1573     -916
+ Misses       1047      407     -640

Flag
Coverage Δ

python
79.44% <ø> (ø)

r
?

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

Impacted Files
Coverage Δ

src/rinterface.cpp

inst/tiledb/include/tiledb/query_condition.h

R/utils-arrow.R

inst/tiledb/include/tiledb/core_interface.h

src/include/tiledbsoma/array_buffers.h

src/include/tiledbsoma/arrow_adapter.h

inst/tiledb/include/tiledb/schema_base.h

src/rutilities.cpp

src/include/tiledbsoma/managed_query.h

R/utils-tiledb.R

... and 28 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@atolopko-czi what we talked about today in the sprint-planning call was a recursive grep -- which I did & which found only this
Unit-testing would require wrapping everything in the API that can throw, and checking its exception class isn't Exception -- I don't see a more compact way to do it, and this sketch seems an over-wordy way to go about it ...

@atolopko-czi I just realized I forgot about "base assert" as well
I updated the description of #433 and I'll have a follow-on PR for assert in package (totally fine to leave assert in test code of course)


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.10.46 to before v5.10.46",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=e52d43c82f2f6556f0b7a790c19c072c1e99a95f",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=174c27583b3807ac96228c442735b02622d8d1c3"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "e52d43c82f2f6556f0b7a790c19c072c1e99a95f",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "174c27583b3807ac96228c442735b02622d8d1c3",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "usb: dwc3: core: fix kernel panic when do reboot\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.46 by commit 174c27583b3807ac96228c442735b02622d8d1c3, it was introduced in version v5.10.46 by commit e52d43c82f2f6556f0b7a790c19c072c1e99a95f. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1001009


Inverted long press and click not working when size/position changes

When this option is checked and you change size or position afterwards, long press still expands it when it should not.
Device : Pixel 3a, GrapheneOS (Android 12)

It works after re-enabling Smart Edge in Accessibility.

Should be fixed in new release


Add new component scrollbar

Hi! First: thanks for awesome UI Toolkit!
Second: I offer add new component scrollbar (example)

Thanks. We have our own scrollbar: https://github.com/ElemeFE/element/tree/dev/packages/scrollbar . It is used internally within other Element components.


Coverage and depth could not be calculated after chromosomes exceeded 512MB

When the chromosome exceeds 512MB, the coverage and depth cannot be calculated. samtools-1.3.1 can establish the index normally(.bai). What is the solution?

Please try one of the following:
1  ：   Reindex it using samtools >1.15: xxx.bai xxx.csi
2 ：  Delete xxx.bai and run it with the new algorithm, without using bam.bai, to obtain the result.

We have revised and uploaded version 2.21, which is compatible with CSI-format indexes. These indexes are typically used for alignment files with chromosomes larger than 512MB.


Screenshotting Issue for Chambers of Xeric and Cox Scouter external

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 10
Bug description
Chambers of Xeric & Cox Scouter External both seem to be unresponsive to my hotkey for screenshotting the layout. It use to work for me and out of no where is quit working. I set a hotkey to numpad+ and that didn't work then moved it to a bunch of different ones to check and insured that it was set to copy to clipboard once the hotkey was pressed to screenshot layout, but nothing goes to the clipboard to be posted. Not sure what seems to be the root cause of this but could use some help.
Screenshots or videos
No response
RuneLite version
Runelite version: <IP_ADDRESS>
Launcher version: 2.6.1
Logs
client.log
launcher.log

The issue tracker is not a support forum. Please join the Discord for questions or support.