chore: Add CGO_ENABLED for konflux operator bundle

If some go is compiled in this image, then do we also need to set CGO_ENABLED to ensure a fips compliant build?
It looks like we do not compile go in this builder stage, and only modify text. But I am not familiar with the operator bundle or why it is using the golang image for the builder stage.

What happens here is that some operator/Makefile targets pull and compile Go-based cli tools like kuttl, yq or operator-sdk. These CLIs are used for generating text files (YAMLs, etc.). Only text files are included in the final stage but CLIs are discarded. Nevertheless, I don't see a reason why not compile with CGO especially if Konflux cluster at some point becomes FedRAMPed.
Let's see if Konflux CI for the operator-bundle will be green and then I'll gladly approve.

No-no, there is some Go compilation, otherwise I would not have to fight with build errors and add ENV GOFLAGS=''. In fact, Tom should know precisely what's used since he had to enable prefetch for these builds https://github.com/stackrox/stackrox/pull/12651


KTOR-6741 Part 1 - Replace "Creating HTTP API's tutorial" with a new tutorial "Handle requests and generate responses"

KTOR-6741
The "Creating HTTP APIs" tutorial will be replaced by two new tutorials:

Handle requests and generate responses. (orig. Understanding Routing and Requests)
Create RESTful APIs that generate JSON. (orig. Content Negotiation and REST)

In this PR:
See preview on staging

[x] Both new topics are added and placed in the navigation
[x]  "Creating HTTP APIs" is removed
[x]  Content is added for "Handle requests and generate responses" tutorial with screenshots for light and dark theme
[x]  An example project is added

The "Create RESTful APIs that generate JSON." topic is currently empty. It will be covered in a separate PR, to be merged to main later with this one.

some screenshots are flying :)


Missing pod name prefix

Hi!
Thanks for creating this! I've been using it daily for a few weeks, and it's great 🥇
Just noticed the prefix-logs section in the readme mention that fblog should prefix logs with pod names when multiple pods are queries for logs. Unfortunately, it doesn't seem to work for me:
When I run
kubectl logs -l app=foo -n bar --follow | ./target/release/fblog -p

with and without -p, it prints regular logs with no pod identifier prefix. Am I missing something, or is the readme not quite right?

-p does not add the prefix but parses it if it is before the json.
with prefix | {"level":"debug","msg":"Trust key rsa-43fe6c3d-6242-11e7-8b0c-02420a000007 found in cache","package":"trust","process":"gatekeeper","store":"cache","time":"2017-07-06T15:21:16Z","version":"0.1.20170620v0125"}

Is rendered like this:
2017-07-06T15:21:16 DEBUG: with prefix | Trust key rsa-43fe6c3d-6242-11e7-8b0c-02420a000007 found in cache

docker-compose for example produces an output like this.

If you want multiple pods you could use this:
stern -n master --tail 100  . | fblog -p
https://github.com/rancher/stern

Cool, thanks for getting back to me!


global.json, dev container dockerfile, and .NET SDK Docker Hub version mismatch

The version of the .NET Core SDK is mismatched between global.json, the dockerfile for the dev container, and what's actually available on Docker Hub.

In global.json the version required is 5.0.100-preview.5.20279.10.
In the dockerfile for the dev container the tag being pulled is 5.0.100-preview.2 which could be easily changed, except...
The latest tag available on Docker Hub appears to be 3.1.301.

This all makes it pretty hard to use the VS Code dev container 😁

Updated the initial issue to point to the correct Docker Hub image. Sorry about that.

I will have a look at this. Thanks for reporting!


[AVM Module Issue]: Fails when no terraform changes and no identity defined

Check for previous/existing GitHub issues

[x] I have checked for previous/existing GitHub issues

Issue Type?
Bug
(Optional) Module Version
0.1.5
(Optional) Correlation Id
No response
Description
Minimal terraform config (for me tag configuration was required to achieve desired state as we have implemented tag inheritance using policies, but I guess you could skip that if that's not the case):
terraform {
  required_providers {
    azurerm = {
      source  = "hashicorp/azurerm"
      version = "~> 3"
    }
  }
}

provider "azurerm" {
  features {}
}

locals {
  location = "swedencentral"
}

data "azurerm_subscription" "current" {}

module "naming" {
  source  = "Azure/naming/azurerm"
  version = "~> 0.4"
}

resource "azurerm_resource_group" "main" {
  location = local.location
  name     = module.naming.resource_group.name_unique

  tags = data.azurerm_subscription.current.tags
}

module "redis" {
  source  = "Azure/avm-res-cache-redis/azurerm"
  version = "~> 0.1"

  location            = azurerm_resource_group.main.location
  name                = module.naming.redis_cache.name_unique
  resource_group_name = azurerm_resource_group.main.name

  capacity         = 0
  enable_telemetry = false
  sku_name         = "Basic"
  tags             = azurerm_resource_group.main.tags
  zones            = null
}

terraform apply, first run:
Success
`terraform applyplResult, second run:
╷
│ Error: Unsupported attribute
│
│   on .terraform\modules\redis\outputs.tf line 25, in output "system_assigned_mi_principal_id":
│   25:   value       = jsondecode(data.azapi_resource.this.output).identity.principalId
│     ├────────────────
│     │ data.azapi_resource.this.output is "{"id":"/subscriptions/xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx/resourceGroups/rg-g6lz/providers/Microsoft.Cache/Redis/redis-xxxx","location":"Sweden Central","name":"redis-xxxx","properties":{"accessKeys":null,"enableNonSslPort":false,"hostName":"redis-g6lz.redis.cache.windows.net","instances":[{"isMaster":true,"isPrimary":true,"sslPort":15000}],"linkedServers":[],"minimumTlsVersion":"1.2","port":6379,"provisioningState":"Succeeded","publicNetworkAccess":"Enabled","redisConfiguration":{"maxclients":"256","maxfragmentationmemory-reserved":"30","maxmemory-delta":"30","maxmemory-policy":"volatile-lru","maxmemory-reserved":"30","preferred-data-persistence-auth-method":"SAS"},"redisVersion":"6.0","sku":{"capacity":0,"family":"C","name":"Basic"},"sslPort":6380,"updateChannel":"Stable"},"tags":{"myTag":"myValue"},"type":"Microsoft.Cache/Redis"}"
│
│ This object does not have an attribute named "identity".
╵
I guess output.tf should not expect identity to be configured as it's optional.

@christerengman  - thanks for submitting this. I will look to address this as part of a v4 support release this week.  Thanks for submitting this issue.


loading text file and accessing data using tensorflow

ubuntu 14.0.04
python 3.5
I have a text file of size (20480,8) with all float values. I want the data in 4th column in to one array. I am able to do it using python as
file_pathname1 = os.path.join(os.path.expanduser('~'),'TF','1st_test', '<IP_ADDRESS>.09.13')
x= np.loadtext(file_pathname1)
y= x[:,4]
#print(np.shape(x))
print(np.shape(y))
I get the size of the y as (20480,)
but I am trying to copy the same as a tensor. How to access the data
file_pathname1 = os.path.join(os.path.expanduser('~'),'TF','1st_test', '<IP_ADDRESS>.09.13')
y = tf.read_file(file_pathname1)
sess = tf.Session()
sess.run(y)
print(y.get_shape())
I cant understand why the loaded file is empty array  because the output is ()
once loaded how to copy the column in to array ?

for solution
x1 = tf.constant(y,name = 'x1')
model = tf.initialize_all_variables()
with tf.Session() as session:
session.run(model)
print(np.shape(session.run(x1)))


Not working in tf 2.0?

Hi,
I was trying to use cyclic lr in tf2.0. Below is my test code:
import tensorflow as tf
print(tf.__version__)
from debug import clr

mnist = tf.keras.datasets.mnist
(x_train, y_train), (_, _) = mnist.load_data()
x_train = x_train.reshape(x_train.shape[0], 28, 28, 1)
x_train = x_train / 255.0
y_train = y_train.astype('int32')

buffer_size = 5000
batch_size = 100
ds = tf.data.Dataset.from_tensor_slices((x_train, y_train)).shuffle(buffer_size)
ds = ds.batch(batch_size)

class My_Model(tf.keras.Model):
  def __init__(self):
    super(My_Model, self).__init__()
    self.flatten = tf.keras.layers.Flatten()
    self.fc1 = tf.keras.layers.Dense(10)

  def call(self, x, training=True):
    x = self.flatten(x)
    x = self.fc1(x)
    return x

def loss(logits, labels):
  return tf.reduce_mean(
      tf.nn.sparse_softmax_cross_entropy_with_logits(
          logits=logits, labels=labels))

step_counter = tf.Variable(0,  trainable = False, dtype=tf.int64)

learning_rate = clr.cyclic_learning_rate(step_counter,
                              learning_rate=0.01,
                              max_lr=0.1,
                              step_size=100,
                              gamma=0.99994,
                              mode='triangular',
                              name=None)

opt = tf.optimizers.Adam(learning_rate=learning_rate())

model = My_Model()

epochs = 2
for epoch in range(2):
  for batch, (images, labels) in enumerate(ds):
    with tf.GradientTape() as tape:
      logits = model(images, training=True)
      loss_value = loss(logits, labels)
      # print('loss', loss_value.numpy())
    grads = tape.gradient(loss_value, model.variables)
    opt.apply_gradients(zip(grads, model.trainable_variables))
    print('LR {}'.format(opt.learning_rate.numpy()))
    step_counter.assign_add(1)
    print('Step Counter  {}'.format(step_counter.numpy()))

For some reason, the learning rate is not being updated? Is there something wrong in the way it is being used?
Thank you,
Best,
SK

Hi SK,
I didn't test it -yet-, but yes, probably the current code is not compatible with tf2.0.
In addition, it is worth noting that github has been giving me security alerts about the tensorflow version that I am using in this code, therefore, use it with caution.
I will try to fix this as soon as I can.

Hi,
Thank you so much for the reply. I think I got your code running on tf2.0. However, I did not test it yet. May be you can use this as a base and build on it. Below is the code:
import tensorflow as tf
from tensorflow.python.framework import ops
from tensorflow.python.ops import math_ops
from tensorflow.python.keras.optimizer_v2.learning_rate_schedule import LearningRateSchedule

class cyclic_learning_rate(LearningRateSchedule):
    def __init__(self,   learning_rate=0.01,
                         max_lr=0.1,
                         step_size=20.,
                         gamma=0.99994,
                         mode='triangular',
                         name=None):
        super(cyclic_learning_rate, self).__init__()
        self.learning_rate = learning_rate
        self.max_lr = max_lr
        self.step_size = step_size
        self.gamma = gamma
        self.mode = mode
        self.name = name

    def __call__(self, step):
        with ops.name_scope_v2(self.name or "CyclicLearningRate") as name:
            self.learning_rate = ops.convert_to_tensor(self.learning_rate, name="learning_rate")
            dtype = self.learning_rate.dtype
            global_step = math_ops.cast(step, dtype)
            self.step_size = math_ops.cast(self.step_size, dtype)

            double_step = math_ops.multiply(2., self.step_size)
            global_div_double_step = math_ops.divide(global_step, double_step)
            cycle = math_ops.floor(math_ops.add(1., global_div_double_step))
            # computing: x = abs( global_step / step_size – 2 * cycle + 1 )
            double_cycle = math_ops.multiply(2., cycle)
            global_div_step = math_ops.divide(global_step, self.step_size)
            tmp = math_ops.subtract(global_div_step, double_cycle)
            x = math_ops.abs(math_ops.add(1., tmp))
            # computing: clr = learning_rate + ( max_lr – learning_rate ) * max( 0, 1 - x )
            a1 = math_ops.maximum(0., math_ops.subtract(1., x))
            a2 = math_ops.subtract(self.max_lr, self.learning_rate)
            clr = math_ops.multiply(a1, a2)
            if self.mode == 'triangular2':
                clr = math_ops.divide(clr, math_ops.cast(math_ops.pow(2, math_ops.cast(cycle-1, tf.int32)), tf.float32))
            if self.mode == 'exp_range':
                clr = math_ops.multiply(math_ops.pow(self.gamma, global_step), clr)
            return math_ops.add(clr, self.learning_rate, name=name)

    def get_config(self):
        return {
            "initial_learning_rate": self.learning_rate,
            "max_learning_rate": self.max_lr,
            "step_size": self.step_size,
            "gamma": self.gamma,
            "mode": self.mode,
            "name": self.name
        }


Fix[#2267]: domain part in email regex

This PR is for correct the email regex. Now there aren't errors with subdomains or hyphen as separetor.
Also it solves issues #2154, #2193

@colinhacks anything I can help with to merge this PR?

or @scotttrinh?


Scepter Crash

MC 1.12.2
Cyclic 1.10.7
Single player
Although the crash seems to kick me out of the internal server, doesn't fully crash the client (for some reason when the game stopped, it put me in the Multiplayer server selection screen), and doesn't produce a crash report like last time. Copied and pasted what I thought was the relevant parts from the log.
Describe problem (what you were doing / what happened):
Was using scepter to make a vanilla farm on a skyblock. Wanted to place planks below dirt, right clicked and saw the scepter was empty, hence the first message in the log. I placed 32 planks in the first slot, and used it a couple of times. and then switched my tool to break some dirt blocks. When I scrolled with the mouse wheel across the hotbar, I it crashed when it cycled past the Scepter (which should've still had oak planks left in it).
Crash log (if any): https://hastebin.com/ojumofoqay.php

Added a fix for it. Glad youre enjoying the tool! https://minecraft.curseforge.com/projects/cyclic/files/25118470000000000000000


CHANGED DOWN SIDE OF PAGE #MAJOR CHNAGES ADDED NEW CSS - by mohit

i have changed odd looking panel in downside ,added some new css like card ,row ,etc!

There are some conflicts

resolved

@mbcse thanks for contributing.

My contribution is comming invalid ,can you help me with that

@mbcse Yup since they are no longer accepting the PRs


fix(addon-doc): change activeItemIndex to 0

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

What is the current behavior?
Have some problem with routable dialog demo.
After reloading page content inside 'example' tab doesn't appear
https://github.com/Tinkoff/taiga-ui/pull/3369/files#diff-8d0c448dfc1d841f3bd066d2cdd249dd9f028b4d84ff367ab0cb6cc7b22584c0R20
Closes #
What is the new behavior?
Reloading page for routable dialog demo works as expected
Does this PR introduce a breaking change?

[ ] Yes
[x] No

https://user-images.githubusercontent.com/38430641/218305748-64d4aa01-51bd-43f8-83f8-07b2925af839.mov


An error occurred when I tried to run the make_onnx_model.py to convert MiDas 3.1 to onnx

RuntimeError: Error(s) in loading state_dict for MidasNet_preprocessing:

same here, any idea how to fix it?


Runplan version multiple times in log when multiprocessing

08:36:44 | Reading model components...
           Runplan version: 1
           Runplan version: 1
           Number of model points: 100
           Multiprocessing on 4 cores
           Calculation of ca. 25 model points per core
08:36:45 | Starting calculations...
           Runplan version: 1
           Runplan version: 1

Fixed in db335312f90eac98b04adf5040e8c09c99f1815c


Database Name Missing From MongoDB Connection URL

Database name should be part of the connection string to maintain a consistent user experience.
When database user is first created. Database name is a mandatory field:
Error: Failed to render chart: exit status 1: Error: unable to build kubernetes objects from release manifest: error validating "": error validating data: ValidationError(AtlasDatabaseUser.spec.roles[0]): missing required field "databaseName" in com.mongodb.atlas.v1.AtlasDatabaseUser.spec.roles

But the resultant connection string doesn't contain the database name:
<EMAIL_ADDRESS>Neither can the the database user create a another database, resulting in error:
pymongo.errors.OperationFailure: not authorized on mymongodb to execute command { find: "todo", filter: { done: "no" }, limit: 1, singleBatch: true, lsid: { id: UUID("10abc4cd-adb1-43d1-99ae-66776eaac296") }, $clusterTime: { clusterTime: Timestamp(1621847098, 1), si _<IP_ADDRESS> - - [24/May/2021:09:05:07 +0000] "GET / HTTP/1.1" 500 290 "-" "kube-probe/1.19"

Hi-
Database users and "Database Name" work a little differently on Atlas than in regular old MongoDB. In Atlas all scram/sha users are created in the "admin" database (since db-users are scoped to project level, by default users are shared by all db clusters in an Atlas project).  So here, it's actually trying to connect to a db called "default".

Thanks for the update @jasonmimick  let me close the issue ticket.


Proto spatial

OK! Achieved:

acquired county boundaries for everywhere including PR and VI
computed centroids for all counties
created centroid points for arizona counties in map

However:
4. had to use a simpler projection and/or just Arizona counties (not sure which was needed in the end) to get the projection not to error out on something about a stream.
So next steps (for someone else - @mwernimont ?) are:
5. try again to use the custom projection for the county points (i basically just gave up)
6. progressively add back in states until we find the limits of the custom projection
i think the problem may be something to do with some counties having extreme coordinates that weren't expected by the projection, but i'm really not sure. in the meantime, enjoy the purple circles!

requesting reviews from both of you because @mwernimont is the custom projection expert and @lindsaycarr is likely to want to use this PR ASAP

I'm going to merge this because I really want that centroid calculation stuff, and the projection issues might take longer than we want for this prototype. I'll probably go ahead and just try to make this in R as a quicker step. Comments on the projection issues should still go here but I've recorded that as issue #9 so that we don't forget.


Camel case all preferred prefixes

This PR updates the preferred prefixes to be always camel case (even if the original label was not camel cased, e.g. for imported terms).

(I am merging now so that https://github.com/incf-nidash/nidm/pull/416 can use the updated code for testing).


[Merge] CL:0000717 'fusimotor neuron' into CL:0008037 'gamma motor neuron'

@dosumis (cc @shawntanzk )
CL term (ID and Label)
CL:0000717 'fusimotor neuron'
Reason for deprecation
Currently, CL:0000717 'fusimotor neuron' and CL:0008037 'gamma motor neuron' are siblings.
The former is defined as "A subset of motor neurons that innervates intrafusal muscle fibers and has small diameter axons.".
The latter as "A lower motor neuron that innervates only intrafusal muscle fibers. (...)".
Wikipedia says "A gamma motor neuron (γ motor neuron), also called gamma motoneuron, or fusimotor neuron...".
They really sound the same, so please consider merging CL:0000717 'fusimotor neuron' into CL:0008037 'gamma motor neuron' (the latter has children and is better axiomatised).
Term to be merged with (ID and Label)
CL:0008037 'gamma motor neuron'
Thanks!
Paola

I'll handle this :) thanks for spotting this @paola-scibite and sorry for adding gamma motor neuron without checking properly first >.<


Fix COBS implementation

[x] rewrite COBS implementation to fix encode/decode of buffers larger than 254 bytes
[x] replace old (erroneous) COBS encode/decode

Waiting for @oliverlee to review pr

I can try to look at it on the next few days. No promises since I don't
have a computer.
On Mar 23, 2017 13:00, "Mick van Gelderen"<EMAIL_ADDRESS>wrote:

Waiting for @oliverlee https://github.com/oliverlee to review pr
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oliverlee/phobos/issues/130#issuecomment-288696841,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AC0-J3Qy2G6NLvwFWQ6rs4-ke8p7NDePks5rol7BgaJpZM4MXyU7
.


Don't run rust tests when only docs change

I'm not sure if there are cases where this would be wrong, but it would be great to avoid running tests when we're just changing dev portal and root-dir files like CONTRIBUTING.md, ISSUES.md, etc.
Right now I'm waiting 15+ minutes for the test suite to run on content that changes CONTRIBUTING.md.

More over I am not sure if we can skip the review process for doc only / .md files. I am not sure if the usual process we do for code is the best for fixing text.

@bmwill - did you say you addressed this?  We can close it out if so.

Done!


Issue #1430: Clean up compiler warnings

This PR takes care of all main and test source compiler warnings for the api, core, core-spi-test and impl modules.
I recommend focusing the review on the main source changes as they are the one that could have an impact I missed.
It also makes any new warning in any of these module an error thus preventing any to be introduced by accident.

I have flag some cases (but there are more) where we cast to a rawtype. If we want to prevent to use any rawtype, you have to use the cast to wildcard - cast to new type trick. I can survive either way.
Otherwise, LGTM and awesome work!

I believe that the cast is the real issue but sometimes we cannot work around it. I certainly do not mind the use of raw types in tests. Maybe I could track the places where they are used in source.
However, the build is failing for another reason. Looks like on the CI environment, we get warnings about use of internal APIs, Unsafe namely and that is something you can't suppress. So the sad bit is that I have no other option than to turn off the error mode from the impl module.

It seems you can try this  javac -XDignore.symbol.file. Another solution would be to fail only in Jenkins using a plugin looking at the warnings. And keeping only the -Xlint:unchecked

Not really fond of using -XDignore.symbol.file. So instead keeping the unchecked instruction, but not the error on warnings.
Pushed an update version that fixes some variable extraction I did which are not legal in Mockito.

test this please


Fix shape inference error for newly version Tensorflow in ctc_label_dense_to_sparse

To make example/image_ocr.py work with Tensorflow 0.11.

possible fix for #3768

LGTM, thanks.


Automated recreation of tables around upgrades

Problem Statement
[Raising feature request for consideration/refinement following a brief discussion with @seut ]
Storage compatibility is currently only maintained for the previous major release of CrateDB, that is tables that were created prior to upgrading to CrateDB 2.x will not function with 3.x and must be recreated before moving to 3.x and so on for tables created prior to 3.x when upgrading to 4.x, etc.
Recreating the tables is a manual process for each table requiring planning, expertise, additional disk space, and potential downtime. This may make people delay upgrades and it becomes a maintenance cost consideration for organizations that may have policies mandating the deployment of upgrades on a certain frequency.
Possible Solutions
Some ideas:

a command that would abstract away the steps involved, something like "RECREATE TABLE ", the mechanism could be optimized over time and perhaps it could work on a shard at a time (minimizing space requirements) and even keep them read-only as the process progresses (minimizing downtime).
if the process above can be local and lightweight enough perhaps a flag to enable that would recreate shards (?) automatically the first time they are accessed on the new version?

Considered Alternatives
No response

cr8 has a reindex command which automates the process somewhat.  Although I wouldn't recommend to use it on production without prior testing - e.g. it doesn't take extra disk space into consideration.
I think for most production scenarios the roll-over with partitions as described here: https://crate.io/docs/crate/reference/en/5.0/admin/system-information.html#avoiding-reindex-using-partitioned-tables is a good fit for larger tables.

I recently had to reindex a large cluster (taking the INSERT INTO SELECT approach), here are some observations:

Reindexing large tables (1 TB) as a whole is not feasible, the process needs to be done on partiton level. Otherwise, it takes too long, and there is no clear state in case of failures.
For such large tables, creating a complete copy is sometimes not even possible due to disk size constraints. It would be great to have the ability to reindexing a partition, and then directly replacing the old one with the new one, without having to build up an entire copy of the table first.
When running the INSERT INTO SELECT statements from a script (e.g. in a loop over multiple tables/partitions), I always lost connectivity with a "connection reset" error after a few hours. A way to run those statements asynchronosly in the background, and checking progress/success in a system table, would be preferred (similar to https://github.com/crate/crate/issues/11748).
Speed of INSERT INTO SELECT was a concern (overload protection being too protective?). The longer the command ran, the slower it got, down to ~ 1300 rows/s on a three-node CR2 cluster that was otherwise idle.


Updated sample runs in ex30 to remove warnings

Updated sample runs in ex30 to remove warnings

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#2881
@brendankeith
@tzanio
@tzanio + @psocratis
3/10/22
⌛due 3/24/22
⌛due 3/31/22

PR Checklist

[ ] Code builds.
[ ] Code passes make style.
[ ] All sample runs at the top of the example source file work.
[ ] (LLNL only) After merging: Update internal tests to include the new features.

This PR is now under review (see the table in the PR description). To help with the review process, please do not force push to the branch.

at the bottom of ex30p.cpp the code
   sol_sock.precision(8);
   sol_sock << "parallel " << num_procs << " " << myid << "\n";
   sol_sock << "mesh\n" << pmesh << flush;

should go in an if(visualization) statement

Merged in next for testing...


Text in some console window is white on pale grey - almost ureadable

I've just installed the latest version from Extension Warehouse - 3.1.4 (which is now a later version than on SketchUcation - 3.0.2).
I'm using SU 2019 Pro on Mac Mojave.
When I click on the three-bar menu top right in the RubyConsole+ window, the pop up dialogue is white text on a pale grey background, and I can't read any of it until I hover the mouse over a part of it.
Looks like this when no mouse-over:

or like this with the mouse over the top line - revealing 'New' in black. Same for other input boxes.

Is there any way I can patch this, or do I need to wait for an update?

It is an inconsistency how different operating systems and versions implement CSS system colors. As it seems, Chromium on macOS Mojave now messes the colors up or interpretes the usage purpose of each color differently.
You should be able to patch it in ae_console/css/console.css. In the dialog, right-click and open the developer tools. In the Chromium developer tools, activate the inspector tool (cursor-in-a-box icon) and select the text element to find out the relevant CSS rules where you can change the color in console.css.
Possibly the rule for body needs to be changed from color: WindowText; to color: black; (or a similar rule)
https://github.com/Aerilius/sketchup-console-plus/blob/master/src/ae_console/css/console.css#L26
This is not something I am quickly going to update, without well thinking and taking a decision first.
I would prefer the dialogs to have consistent, native colors matching the platform. It is a compromise that it does not (anymore) work equally well on all platforms. A platform switch is not a good solution. Maybe I have to bite into the apple and hard-code specific colors.****


not relying on topic labels in the taxonomy graph

related to https://github.com/buda-base/xmltoldmigration/issues/91
The taxonomy graph currently contains labels for topics (ex: bdrT3CN2023). This shouldn't be the case in the future. I'm not sure how the code is organized, but lds-pdi sends the labels for the topics in the search results, for example:
http://purl.bdrc.io/lib/topicAllAssociations?I_LIM=500&R_RES=bdr%3AT2423
we should just make sure that the labels for the topics are not coming from the outline graph but from a query that lds-pdi would run to get them.

all good, the taxonomy tree is fetched by https://github.com/buda-base/lds-queries/blob/master/library/Taxtree.arq which doesn't care about the graph, so no problem


Enhancements in the Throttle decorator

Description:
Following PR #14546, a suggestion was made to synchronize the data polling to HA with the effective refresh in the NetAtmo cloud.
More generically, sensors polling data from 'the cloud' may wish to get the freshest data, whilst avoiding to execute a remote call too often. In order to do so, they would need to re-adjust the timing of their throttled update() method.
The proposal here is to enhance the Throttle() decorator, so that the wrapped method/function could be immediately throttled (i.e. the first effective call will not happen before min_time seconds have passed).
Beside that, the other commit improves the behavior of the throttling by being independent from the time spent for the actuall call to method.
This is NOT a breaking change, but nevertheless it touches one of the core classes of Home Assistant, therefore I appreciate an adequate review. By the way, it has been inspiring to look at this code :-)
Related issue (if applicable): N/A
Pull request in home-assistant.github.io with documentation (if applicable): N/A
Example entry for configuration.yaml (if applicable): N/A
Checklist:

[x] The code change is tested and works locally.
[ ] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[ ] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[ ] New dependencies have been added to the REQUIREMENTS variable (example).
[ ] New dependencies are only imported inside functions that use them (example).
[ ] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Going to close this PR as the new parameter is not necessary and the other change will cause worker pool starvation.


Death Protection Poppet Witchery not working

Soooo I tried performing the Rite of Infusion on myself using the Spirit of Otherwhere and NORMALLY you need some kind of means to prevent the death that follows if you're not prepared. So I made a death protection poppet ( the item you need to prevent that death) and I still killed me and didn't give me the infusion. So i made ANOTHER death protection poppet (very expensive btw lol) and tried it again this time leaving it on a poppet shelf aaaand it still killed me...a resolution would be awesome otherwise we can't enjoy that part of the mod lol, thanks!

Is this in SSP or on the official server?

This is on the official server

Some time ago we disabled some Witcher functions to prevent grieving and pvp.  This may be working as intended.  Could you replicate this in single player to verify?

Yes it works in single player. Maybe you could at least enable the spirit of otherwhere on the server? it's main use is for teleportation; It gets hectic having to recall to the spawn point and travel all the way back to your base lol but it's entirely up to you guys; thank you for the help.

Some things aren't disabled but simply not compatible with the custom jars we use to run the server with plug-ins for anti grief and the like.  Alternatively their are several other mods for teleportation. I'll check into it.


Add print terminal to page depending on if printing is supported.

Screenshots for reference:

@eyurko @seanlip PTAL, switched to new branch as the other was getting somewhat cluttered.

LGTM. Thanks!


Sprockets error when using asset-url

First off, thanks for an amazing tool. jekyll-assets is fun to work with and has been very useful.
I am running into trouble using asset-url(). Each time I use it in a .scss file, I get the following error:
$ bundle exec jekyll build  --config _config.yml,_config_dev.yml --trace
Configuration file: _config.yml
Configuration file: _config_dev.yml
            Source: src
       Destination: site
      Generating...
/Users/cgansen/.rbenv/versions/1.9.3-p385/lib/ruby/gems/1.9.1/gems/sprockets-sass-1.3.1/lib/sprockets/sass/functions.rb:128:in `sprockets_context': undefined method `[]' for nil:NilClass (NoMethodError)

(the full stacktrace is available at https://gist.github.com/cgansen/686da5682d802beb7fe4)
When I remove the asset-url(filename) call from my .scss file, everything builds normally.
Any tips? Anything else I can provide to help debug this issue?
Here's a gist with relevant environment data (bundle versions, etc): https://gist.github.com/cgansen/e7c2bba76e85c19c4401

I found that this issue went away when I removed an errant gem 'jeykll-sass' in my Gemfile. False alarm!

:D Sorry I had no time to take a look. Glad you found the reason. :+1:


About the training of Camvid.

Hi, thanks for your excellent work.
I have a question, the Camvid dataset is small (i,e. the images are few), do you share your training tricks or tips for Camvid?
And I see you that your max-epoch is 1000 on Camvid, don't you? Wow, it's too many epochs.
Thanks!!!

Adam is more stable in the low-resolution image like CamVid dataset and class weight are employed in the loss function just as ENet.
Actually, it does not need to set the epoch to 1000 on camvid, the model can saturate about 500~800 epochs, I just set the same as cityscapes, you can try different epoch.


feature/api-upgrades-and-other-changes

PR Overview
This PR will address the following Issue/Feature: Issue #46
This PR will result in the following new package version: v0.9.0

There are a number of intermediate model removals as well as fields being removed. Therefore, this should be categorized as a breaking change.
Please detail what change(s) this PR introduces and any additional information that should be known during the review of this PR:
🚨 Breaking Changes 🚨

In October 2023 the Intercom API upgraded from 2.9 to 2.10 which resulted in the the connector schema receiving updates. These updates have downstream impacts on the data models within this package. The following changes are a result of the Intercom API and connector upgrades:

Breaking changes within the dbt_intercom_source package. Please refer to the relevant Intercom Source PR for more details around the source package breaking changes.
Removal of the _fivetran_deleted field within the following end models:

intercom__company_enhanced
intercom__company_metrics
intercom__contact_enhanced

Removal of the following intermediate models as filtering on the new _fivetran_active field achieves the same result:

int_intercom__latest_contact
int_intercom__latest_conversation
int_intercom__latest_conversation_contact
int_intercom__latest_conversation_part

Feature Updates

The _fivetran_active, _fivetran_start, and _fivetran_end fields have been added to all *_history staging models as well as the following end models to replace the deprecated _fivetran_deleted field:

intercom__company_enhanced
intercom__company_metrics
intercom__contact_enhanced

Dependency Updates

Removal of the dbt_expectations dependency.

PR Checklist
Basic Validation
Please acknowledge that you have successfully performed the following commands locally:

[X] dbt compile
[X] dbt run –full-refresh
[X] dbt run
[X] dbt test
[X] dbt run –vars (tested all variables being turned on and then off)

Before marking this PR as "ready for review" the following have been applied:

[X] The appropriate issue has been linked and tagged
[X] You are assigned to the corresponding issue and this PR
[X] BuildKite integration tests are passing

Detailed Validation
Please acknowledge that the following validation checks have been performed prior to marking this PR as "ready for review":

[X] You have validated these changes and assure this PR will address the respective Issue/Feature.
[X] You are reasonably confident these changes will not impact any other components of this package or any dependent packages.
[X] You have provided details below around the validation steps performed to gain confidence in these changes.

Please see the full validation steps linked in the Hex notebook within the Height ticket.
Standard Updates
Please acknowledge that your PR contains the following standard updates:

Package versioning has been appropriately indexed in the following locations:

[X] indexed within dbt_project.yml
[X] indexed within integration_tests/dbt_project.yml

[X] CHANGELOG has individual entries for each respective change in this PR

[X] README updates have been applied (if applicable)

[n/a] DECISIONLOG updates have been updated (if applicable)
[X] Appropriate yml documentation has been added (if applicable)

dbt Docs
Please acknowledge that after the above were all completed the below were applied to your branch:

[X] docs were regenerated (unless this PR does not include any code or yml updates)

If you had to summarize this PR in an emoji, which would it be?

ℹ

Thanks @fivetran-jamie! Changes have been applied and responses provided. Let me know if there are any other questions.


feat: add fs/append-file

adds file system append file utility to Stdlib
Description

What is the purpose of this pull request?
This pull request brings the node js file system append-file utility to Stdlib

This pull request:
is aligned with the purpose of achievinng feature parity with Node.js fs package. It Brings file system's powerful append file utility to Stdlib expanding its existing fs utilities.
Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

Implementation details
-- async version
var join = require("path").join;
var fpath = join(__dirname, "examples", "fixtures", "file.txt");
appendFile(fpath, "beep boop\n", onAppend);
function onAppend(error) {
if (error) {
throw error;
}
}
-- sync version
var appendFile = require( '@stdlib/fs/append-file' );
// Explicitly handle the error...
try{
appendFile.sync( '/path/to/file.txt', 'beep boop\n' );
} catch(err) {
throw err
}
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[ ] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Thanks, @HRIDYANSHU054, for working on this. This PR is currently failing CI. Those failures need to be addressed before the PR can be reviewed.

Hi @kgryte , Thanks for checking this PR. I m currently working on solving these issues. Majority of them have arised because the fixtures directory was somehow not pushed to github.

@kgryte can you please have a look, I have made changes to tackle the previous PR issues

Corrected some more things. @kgryte can you once again check this

@kgryte can you please approve the remaining workflows. I am interested in this would like to tackle any other issue that arises or any change that is required.

@kgryte all the issues that have been highlighted thus far during the CI checks have been solved. Can you again approve the remaining workflows so that if there are any other errors then they could also be solved and this pr can thus be reviewed. On a sidenote really thanks for your patience✨.

@kgryte can you please have a look

@kgryte please check

@kgryte please rerun the CI

@kgryte please can you again check it

@kgryte please check these changes

@kgryte please have a look , a Lot of CI check issues have been solved and now only the lint check is remaining to be solved

@kgryte please check

Finally, this has passed all checks. @kgryte please can you review it now.

@kgryte this is now clear for review. Kindly check

@kgryte please review my PR

@kgryte the errors that were coming during the CI checks have all been solved. Please review this PR and tell me if any change is needed.

@HRIDYANSHU054
Would you mind checking the box in the OP indicating that you've read the contributing guidelines? This needs to be checked in order to confirm that you agree to licensing terms and the Developer's Certificate of Origin.

@kgryte thanks for going through this and reviewing it. I don't why the splitting has happened in only some functions and not others even though I have configured my ESlint to highlight these issues if they occur. Also can I address this after 5th april as I am currenlty having fromm today my mid semester examinations.

@kgryte the changes you highlighted have been implemented, certain things like the new benchmarking approach strongly needs to be reviewed

@kgryte can you approve these remaining workflows and then review the changes

Hi ✋ @kgryte just hoping you could further review these changes, and anything more that you want me to change or add I will work on that also.

hi @kgryte could you review this

@kgryte could you review this PR

Thanks @Planeshifter looking forward to learning and contribute more to stdlib node's environment


Improve how we're downloading the AWS Sentinel Connector lambda layer

Summary
Some Docker image builds in this repo are using the AWS Sentinel Connector lambda layer:

Layer download in the workflow
Layer source

The issue we're facing is that when a new version of the layer is published, AWS deleted any previous, unused versions.  Since our use in the Docker image is not detected, the referenced layer is deleted, and subsequent builds fail.
Potential fixes

Clone the cds-snc/aws-sentinel-connector-layer at a specific tag and build the layer code ourselves as part of the docker image build.
Consume a built artifact or Docker image that's created as part of the cds-snc/aws-sentinel-connector-layer workflows.

Related

cds-snc/aws-sentinel-connector-layer#7

aws-sentinel-connector now has a docker build step that builds and push the image for it to be consumed by other projects. Security-tools is now consuming this docker image to acquire the required python connector.


Remove tac command usage which require coreutils

Replace tac command with tail -f
Added .gitignore

/cc @Prajyot-Parab @cs-zhang @sajauddin


[V3] Incorrect iOS Documentation

currently the installation documentation for iOS is confusing & does not work
https://wix.github.io/react-native-notifications/docs/getting-started
here is just missing a line for the AppDelegate.m
here I need additional add #import "RNNotifications.h"
and
calling the constructor https://wix.github.io/react-native-notifications/docs/getting-started#register-for-notification-events needs also to call super(props)
https://wix.github.io/react-native-notifications/docs/installation-ios
here in step one I should link a file which has another path
`+ pod 'ReactNativeNotifications', :podspec => '../node_modules/react-native-notifications/ReactNativeNotifications.podspec'

Is the documentation up to date yet?

This is the correct pod spec. The one that yogevbd provided was wrong.
pod 'react-native-notifications', :podspec => '../node_modules/react-native-notifications/react-native-notifications.podspec'


manual upload cannot correctly automatically continue after reconnect to internet

If user has more than 1 manual upload task (e.g. 4), and by uploading the first task, internet is cut off. After reconnect to the internet, the first task will be automatically continued and will be finished normally. But the rest of the waiting task cannot be started. The user has to start them manually.

fixed


Any way to get the mime type of the picked item ?

Thanks for the lib.
Any one having any solution to get the mime type of the selected media item. ?

@eshajari Not sure I get you mean, this would depend on how you encode the image on your side.
For example, when the didFinishPicking callback is fired you can do.
let jpegData = items.singlePhoto?.image.jpegData(compressionQuality: 0.8)

doing so you would upload it using a image/jpeg MIME type.
let pngData = items.singlePhoto?.image.pngData()

This would be image/png for instance.
Hope this helps :)

@eshajari can we close this or are you still experiencing an issue?

Closing this for lack of activity, fee free to reopen if needed :)


speed comparison

As mentioned in the paper, the inductive inference time of GLNN is compared to other inference acceleration methods of GNN on 10 randomly chosen nodes, but the code does not include these experiments. So could you please provide some more details about how the inference time is measured and compared ?

Below are the functions I wrote for time measurement. Note that the results could be different from our paper as they depend on your hardware.
def sage_inference(model, feats, dataloader):
    for step, (input_nodes, output_nodes, blocks) in enumerate(dataloader):
        model(blocks, feats[input_nodes])

def sage_measure(conf, model_name, L, prune_budget=0.5, fan_out=None):
    conf = copy.copy(conf)

    conf['model_name'] = model_name
    if model_name == 'SAGE':
        get_model = lambda conf: Model(conf).encoder
    elif model_name == 'SAGEQuantize':
        get_model = lambda conf: torch.quantization.quantize_dynamic(Model(conf).encoder)
    elif model_name == 'SAGEPrune':
        conf['hidden_dim'] = int(conf['hidden_dim'] * prune_budget)
        get_model = lambda conf: Model(conf).encoder
    gnn_dur = []
    for l in range(1, L+1):
        conf['num_layers'] = l
        set_seed(0)
        model = get_model(conf)
        sampler = dgl.dataloading.MultiLayerNeighborSampler([fan_out] * model.num_layers)
        model.eval()
        with torch.no_grad():
            dur = []
            for exp in range(num_exp):
                set_seed(exp)
                idx = torch.randperm(g.num_nodes())[0]
                dataloader = dgl.dataloading.NodeDataLoader(g, idx, sampler, batch_size=batch_size, shuffle=False, drop_last=False)
                t0 = benchmark.Timer(
                    stmt='sage_inference(model, feats, dataloader)',
                    setup='from __main__ import sage_inference',
                    globals={'model': model, 'feats': feats, 'dataloader': dataloader})
                measurement = t0.blocked_autorange(min_run_time=0.2)
                dur += [measurement.median * 1000]
            gnn_dur += [np.mean(dur)]
    return gnn_dur

Thx for your quick reply!


Issues including external scripts for use as webpack external dependencies

Getting a server-side error message trying to render a react component that depends on React-d3.js.
Error:

Error while loading "~/scripts/externs/react-d3.min.js": ReferenceError: document is not defined
    at Script Document [7]:9:29125 -> 3_array=function(list){return d3_arraySlice.call(list)},d3_document=document,d
    at Script Document [7]:13:26437
    at s (Script Document [7]:6:559)
    at Script Document [7]:6:610
    at ./areachart (Script Document [7]:6:9775)
    at s (Script Document [7]:6:559)
    at e (Script Document [7]:6:726)
    at Script Document [7]:6:743
    at Script Document [7]:6:272
    at Script Document [7]:6:277
Line: 9
Column:29125

Scenario:

Webpack bundle including a react component that requires 'd3-react' library.

'use strict';
var React = require('react'),
    BarChart = require('react-d3').BarChart;
var Test = React.createClass({
  render: function() {
    var barData = [
      {label: 'A', value: 5},
      {label: 'B', value: 6},
      {label: 'F', value: 7}
    ];
    return (
      <div>
        pretend im useful: {this.props.data}
        <BarChart data={barData} width={500} height={200} fill={'#3182bd'} title='Bar Chart' />
      </div>
    );
  }
});
module.exports = Test;

Webpack entrypoint including Test.jsx

module.exports = {
  Test: require('expose?Test!charts/Test')
};

Webpack config specifying externals including 'React' and 'react-d3'.

externals: {
  // Don't bundle these with the server-side bundle. React.NET needs these to be globals
  react: 'React',
  'react-d3': 'react-d3'
},

Server-side ReactConfiguration including 'react-d3' script.

            React.ReactSiteConfiguration
                .Configuration
                .AddScript("~/scripts/externs/react-d3.min.js")
                .AddScript("~/scripts/c/server/Charts.js");

Forgot to mention, I'm using the V8 Engine. I'll try with MSIE as well.

I'm not familiar with Webpack, or with React.net.  However, I can tell you by the error message that you're missing jsdom, which is a d3 dependency.  In theory, d3 should pull that in if you're using NPM.  However, I believe that there may be some issues with jsdom and Windows, since it may require separate compilation.
Check out the jsdom installation instructions for Windows from the jsdom team here: http://www.steveworkman.com/node-js/2012/installing-jsdom-on-windows/
If you still have problems after you've installed jsdom (and you also may need to npm cache clean and reinstall react-d3 after you install jsdom).
Ideally, we'll eventually not need the part of d3 that's pulling in jsdom, but that's probably a few minor version releases away.

Thanks! Sorry actually i meant to post this to the React.NET project :)
but this may be helpful too.

Out of curiousity, what's the part of d3 that uses JSDOM that you want to get rid of... maybe I can help.

@andycmaj Unfortunately, it's a pretty big job: we need to be able to reproduce the SVG axes that are rendered by d3 in pure React.  All the other chart components are in pure React -- d3 is only used most of the time for path and scale calculations.  I'm planning on tackling this for v0.5 (and we're just about at 0.2 now).
But there are a lots of other contributions you're welcome to do right now.  Are you familiar with d3?


Update changelog for 2.0.1

Description
Update changelog for the new version of the CLI

We should try and get #680 in for this.

Closing as outdated


[qwt] Fix missing feature dependency

All features are tested successfully in the following triplet:
x86-windows
x64-windows
x64-windows-static


Neighbors

In a similar vein as #63 , the k-mer logic currently doesn't check neighbors, which drops reads that have a mismatch. Probably best to have a queue of positions to evaluate. Place the exact matches first (on either strand), check those to set a bench mark of best. Then check 1 neighbors of those positions.

No longer relevant in the full PHMM.


Fix uncertainty calculation in StereoMeanCombiner

This fixes a problem with the uncertainties calculated by StereoMeanCombiner where the mapping of telescope events to array events did not work correctly. This resulted in many nan values in e.g. the RandomForestRegressor_energy_uncert column.
In doing so, I added a new module (ctapipe.vectorization) for the helper functions I implemented. This module also contains a faster (numba) alternative to np.unique, ctapipe.vectorization.get_subarray_index.
This is part of #2497, but I think this should be fixed regardless of the other content of that PR.

The coverage here is too low because we cannot get a valid coverage reported for the numba jitted  code, the actual coverage is pretty close to 100 %.


Added ALBERT Model

Added ALBERT Model in ppt and jpeg files.

Any other models you want to make note of? Also is it possible to add a description/comparison between each model?

At the moment, there are many ALBERT models in SQuAD, so there is no other model I would like to add.
On the GLUE benchmark, T5 has been ranked first for a while and I am interested.
T5 stands for "Text-to-Text Transfer Transformer".
I'd like to see the progress a little more about whether it is worth mentioning.

What about Reformer from https://www.youtube.com/watch?v=rNG_hpSyZcE ?

Thanks for the suggestion.
I understand that Reformer is a model that learns with less memory.
I think it's a good idea to chunk close vector words with LSH(Local Sensitive Hash) Attention.
Reformer can learn with less memory than Transformer, so it may be a basic technology for future learning models.
I don't know the exact accuracy, so I'll try it in my native Japanese first.
It is important that languages without spaces such as Chinese and Japanese have high accuracy.
(There is no problem because Reformer's TOKENIZE is SentencePiece.)

SO what about the others in https://gluebenchmark.com/leaderboard and https://rajpurkar.github.io/SQuAD-explorer/
Are they just permutations of currently included models within this repo?
Also what about the other benchmarks in https://www.tensorflow.org/datasets/catalog/overview ("text section")?

No.
Reformer has not yet entered SQuAD or GLUE rankings.
Reformer is described as "The Efficient Transformer", so I think it will replace Transformer or be used with Transformer.

@artisanbaggio I was referring to other entries in the benchmark.

OK, The model I am interested in in SQuAD and GLUE is the "T5" model as the first answer.


Add a generic node iterator.

Use the node iterator to traverse symbols/nodes in write_config(),
write_min_config(), write_autoconfs()...
This ensures that the order of symbols is consistent between config
files and headers and that symbols don't get shuffled around between
configuration changes.
A nice side effect is that comments are now included in the generated
header files and minimal configs.
I did run the test suite against linux kernel 4.18.3 and it seems to be failing, but not because of this change. Please let me know if you need the logs.

Hi ulfalizer,
if you have already a patch that guarantees the order of symbols in "min_config" and header files that's great :)
The issue I was trying to fix is that we're having generated header files and "minimal" configs under revision control and the symbols were shuffled around randomly when the Kconfig changed.
Anyway, I did push changes that should satisfy all if not most of your requests. That being said, I'll be happy with only the ordering patch as that should be enough to fix the issues we're having.
Thanks for thins great library!

@pinkfluid
Yep... finishing up a fix now (it's basically just replacing for sym in self._defined_syms_set: with for sym in self.defined_syms, but I want to add some tests as well).
Some kind of tree walking helper would definitely be nice to have still, and having the comments in the headers might be nice too. I'll look into a bit later.

That's great, thank you.

Just one more comment, node_iter() can probably be changed to use the "defined_syms" when skip_duplicates=False.

@pinkfluid
Might want to visit other MenuNodes besides symbols though.
A MenuNode can represent a symbol, a choice, a menu, or a comment. skip_duplicates would only apply to symbols, because it's usually what you want when writing output.

Pushed a fix for the ordering issue and made a new 10.0.1 release.
I noticed that write_config() could be simplified a bit too: https://github.com/ulfalizer/Kconfiglib/commit/db6cb68b74bc3a9f88f802a0f32613f95df7b854. Maybe that could be integrated here as well.
Gotta work on some work-related things now, but I'll get back to the traversal helper later. :)

Hi, the new version seems to work perfectly. The order seems to be preserved in both minconfig an config.h! Thank you!

I added a variant of the node iterator now as well: https://github.com/ulfalizer/Kconfiglib/commit/7dae98803a6fc5d08041d1387e2e0d83fc0eb0ed
I did some reorganization prior, and it's a bit hard to write docs without knowing all the Kconfig details, so didn't want to force you to update the PR.

I was already happy with the ordering fix, but this looks great :)
I didn't really take a look deep inside the node class, but could this evolve to some sort of a public API?
It's just a thought, but you could use the iterator to generate customized files. For example, some projects may need to  add support for other languages besides C or generate a C structure instead of a header file...
I think its time we discard this pull request :)

@pinkfluid
The MenuNode class and the new Kconfig.node_iter() generator are already public APIs. You can access the top menu node via Kconfig.top_node and follow next and list pointers, etc., as well. Symbols and choices also have a public nodes property for accessing their menu nodes.
See the print_tree.py example for example.
Alright, closing the issue. Thanks!

Basically, anything that doesn't have a _ prefix is a public API. Public functions have docstrings too, while internal functions use "doc comments" instead.

When I said "public API", I was thinking more in the lines of creating an API for writing configurations using a generator. Something you would use instead of write_config()/write_min_config(). This could be useful, for example, for export the configuration as a JSON or INI files, and probably other things like:

adding support for non-C/shell/make languages
kconfiglib could be used for generating configuration files (imagine having a menuconfig interface for configuring a web server?)
writing to stdout/sockets

These are just ideas, so its up to you to decide if its useful or not :) In any case, I'm rather happy with kconfiglib in its current state.
This is an example of such iterator that I scrapped together. Below you can find examples that export the current config in JSON and INI formats.
import kconfiglib

class WriteItem:
    MENU = 1
    COMMENT = 2
    BOOL = 3
    TRISTATE = 4
    STRING = 5
    INT = 6
    HEX = 7

    def __init__(self, wtype, name, value):
        self.type = wtype
        self.name = name
        self.value = value

def write_items(self, unique_syms=False, minimal=False):
    for node in self.node_iter(unique_syms):
        item = node.item

        if isinstance(item, kconfiglib.Symbol):
            if minimal:
                # Skip symbols that cannot be changed. Only check
                # non-choice symbols, as selects don't affect choice
                # symbols.
                if not item.choice and \
                   item.visibility <= kconfiglib.expr_value(item.rev_dep):
                    continueFor example

                # Skip symbols whose value matches their default
                if item.str_value == item._str_default():
                    continue

                # Skip symbols that would be selected by default in a
                # choice, unless the choice is optional or the symbol type
                # isn't bool (it might be possible to set the choice mode
                # to n or the symbol to m in those cases).
                if item.choice and \
                   not item.choice.is_optional and \
                   item.choice._get_selection_from_defaults() is node and \
                   item.orig_type == BOOL and \
                   item.tri_value == 2:
                    continue

            item_name = "{}{}".format(self.config_prefix, item.name)
            if item.orig_type == kconfiglib.BOOL:
                yield WriteItem(WriteItem.BOOL, item_name, item.str_value == "y")

            elif item.orig_type == kconfiglib.TRISTATE:
                if item.str_value == "m":
                    yield WriteItem(
                            WriteItem.TRISTATE,
                            "{}_MODULE".format(item_name),
                            True)
                else:
                    yield WriteItem(
                            WriteItem.TRISTATE,
                            item_name,
                            item.str_value == "y")

            elif item.orig_type == kconfiglib.STRING:
                yield WriteItem(WriteItem.STRING, item_name, item.str_value)
            elif item.orig_type == kconfiglib.INT:
                yield WriteItem(WriteItem.INT, item_name, int(item.str_value) if item.str_value else 0)
            elif item.orig_type == kconfiglib.HEX:
                yield WriteItem(WriteItem.HEX, item_name, int(item.str_value, 16) if item.str_value else 0)
            else:
                continue
        elif not kconfiglib.expr_value(node.dep):
            pass
        elif item == kconfiglib.COMMENT:
            if not kconfiglib.expr_value(node.dep): continue
            yield WriteItem(WriteItem.COMMENT, None, node.prompt[0])
        elif item == kconfiglib.MENU and kconfiglib.expr_value(node.visibility):
            if not kconfiglib.expr_value(node.dep): continue
            yield WriteItem(WriteItem.MENU, None, node.prompt[0])
        else:
            # Skip everything else
            pass

kconf = kconfiglib.Kconfig("/home/mitja/git/device/core/kconfig/Kconfig")
kconf.load_config("/home/mitja/git/device/vendor/plume/kconfig/targets/VFE_BCM52")

"""
Dump config in JSON format
"""
dot = False

print("{")
for w in write_items(kconf, minimal=False):
    if dot: print(",\n", end="")
    dot = True

    if w.type in (WriteItem.HEX, WriteItem.INT):
        print('     "%s": %d' % (w.name, w.value), end='')
    elif w.type == WriteItem.STRING:
        print('     "%s": "%s"' % (w.name, w.value), end='')
    elif w.type in (WriteItem.BOOL, WriteItem.TRISTATE):
        print('     "%s": %s' % (w.name, "true" if w.value else "false"), end='')
    elif w.type in (WriteItem.COMMENT, WriteItem.MENU):
        print('\n     "__COMMENT__": "{}"'.format(w.value), end='')
    else:
        dot = False

print("\n}")

"""
Generate an INI file
"""
kconf.config_prefix = ""
for w in write_items(kconf, minimal=False):
    if w.type in (WriteItem.HEX, WriteItem.INT):
        print('%s=%d' % (w.name, w.value))
    elif w.type == WriteItem.STRING:
        print('%s="%s"' % (w.name, w.value))
    elif w.type in (WriteItem.BOOL, WriteItem.TRISTATE):
        print('%s=%s' % (w.name, "true" if w.value else "false"))
    elif w.type in (WriteItem.COMMENT, WriteItem.MENU):
        print('\n\n[{}]'.format(w.value))

The tricky part is coming up with something general I think. Always the risk that you make something too limited if you make it too magic.
I think I prefer the approach of adding small helpers whenever you find that you keep doing the same thing over and over. Just gotta discover those helpers.

It was just an idea, I don't have any immediate use of it right now :) I'll just push it to my fork, so I have a backup in case I need this in the future.
Thanks again.


add packaging example

This PR adds an example specfile for RPM packaging. It is meant as blue-print for distributors.

force pushed the PR again. Added gpg signature to all commits.


Won't work inside NAT

The node connection setup protocol is as follows:
--> Client sends it's local ip:port information
--> Server reads that information and updates the DHT
Thus the connection ot that client will always fail if it's inside a subnet.
The ip information that is kept is so bad that my local computer sends it's loopback address  i.e. "<IP_ADDRESS>:port"  to the server

Hey @mesudip, Did you do any code updates to solve this issue? If you did please send me a pull. If not let me know and I'll try to sort it out ASAP.
I've actually only used this DHT locally. I'd be happy to add this functionality.

The node connection setup protocol is as follows:
--> Client sends it's local ip:port information
--> Server reads that information and updates the DHT
Thus the connection ot that client will always fail if it's inside a subnet.
The ip information that is kept is so bad that my local computer sends it's loopback address i.e. "<IP_ADDRESS>:port" to the server

can you please tell me which file i have to run first iam intresting to work on this project

google brought me here... why have peer 1 send its own contact information to peer2? peer2 will know what the ip of peer1 is as soon as peer1 connects... probably still won't help you with nat traversal, because if peer1 is behind a nat peer3 won't be able to get through even if it got working contact information from peer2...


MFRC522 support with esp-open-sdk

Hello,
since this project does noch relay on Arduino, I thought it might be possible to convert the MFRC522 lib to make in compatible to the esp-open-sdk.
Or is there maybe already an option to do this.
Thanks for any help.

Hello. MFRC522 lib here is imported from public domain arduino lib - see sources and copyright notice on the top of the file - https://github.com/devicehive/esp8266-firmware/blob/develop/firmware-src/sources/devices/mfrc522.c
So I think that is not hard to fork this lib to any other project.


⚠ Turnos 24x7 has degraded performance

In 00d073f, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 1316 ms

Resolved: Turnos 24x7 performance has improved in 31de2a7.


Add Lombok without any impact in the fatJar

https://github.com/rzwitserloot/lombok/issues/1602

Added


Cannot set an ID on Paragraph component

Attempting to set an id on a Paragraph will silently fail and be ignored.
It seems that this is because our component is wrapping react-markdown, which is silently ignoring any provided ID.  See https://github.com/rexxars/react-markdown/issues/242
See also #503

The only thing we could do here, as far as I could see, is wrap with an additional div.
I would prefer to remove markdown support altogether in #503.
@stevesims ?


fix: save logo caption and description when uploading app

Currently the information about the image uploaded together with the app was ignored. This adds support to add caption and description when creating a new app.

I can't actually get this to work when testing, the frontend doesn't seem to add the caption or description to the json payload within app.images that the backend reads, only an empty array.
It seems that this code always overwrites the images array as imageFile.length is never > 0 (it can be either empty array or a single image object):
https://github.com/dhis2/app-store/blob/master/client/src/components/form/UploadAppFormStepper.jsx#L303

yes, tested locally to merge with latest master and then it works.

I can't actually get this to work when testing, the frontend doesn't seem to add the caption or description to the json payload within app.images that the backend reads, only an empty array.
It seems that this code always overwrites the images array as imageFile.length is never > 0 (it can be either empty array or a single image object):
However in the master branch this seems to be fixed as it only checks for imageFile not being falsy
instead of imageFile.length

yes, tested locally to merge with latest master and then it works.

@Birkbjo would be nice with a unit test for this as well


Clean up geocoder index generation

Continuing on the path of #18

Closed in favor of https://github.com/3nprob/headway/pull/1


Update MRU from 3.4 to 3.9.

I found an annoying bug in MRU 3.4, open the MRU window, navigate the cursor to any recent file, press the t, it supposed to open that file in new tab, but in MRU 3.4 it says an error:
Error detected while processing function <SNR>86_MRU_Select_File_Cmd[21]..<SNR>86_MRU_Window_Edit_File[14]..<SNR>86_MRU_Open_File_In_Tab:
E16: Invalid range: 999tabnew /Users/tonni/.vim_runtime/.git/config
Press ENTER or type command to continue

Here is my Vim and OS version:
Vim:
➜  .vim_runtime git:(master) vim --version
VIM - Vi IMproved 7.4 (2013 Aug 10, compiled Oct  5 2016 16:04:34)
Included patches: 1-898
Compiled by<EMAIL_ADDRESS>
OS: macOS 10.12.2 (16C67)
MRU 3.9 has fixed this problem, and therefore I created this PR for updating MRU.
Please have a look, thanks.

@HouCoder already updated by the plugin updater, but thanks a lot for the PR


[kafka_consumer] pytest migration

What does this PR do?
Almost ready, might clean up a couple of things, and need to make sure the other tests ~ain't~ aren't broken.
Motivation
pytest+tox ❤
Review checklist:

[x] Gitlab enabled via list in tasks.py
[x] test dir renamed to tests
[x] setup.py changes
[x] new __about__.py, tox.ini, requirements-dev.txt, MANIFEST.in
[x] AgentCheck import
[x] __path__ = __import__('pkgutil').extend_path(__path__, __name__) in datadog_checks/__init__.py
[x] remove the flavor from .travis.yaml
[x] add to README.md

ugh - looks like the networking is messed up in gitlab. This might be painful.

@masci / @truthbk / @ofek this PR was unnecessarily painful to parse what happened when I read through the commit changelog: https://github.com/DataDog/integrations-core/commits/master/kafka_consumer/datadog_checks/kafka_consumer/kafka_consumer.py
In particular, because it was a normal merge rather than a rebase, I couldn't look at only the latest commits on the file to see what happened. Git intermingles the commits from this branch with the existing commits, so I had to unravel it to figure out what changed vs what got overriden by a backmerge from master.
Additionally, all these merge commits just clutter up the final commit log and make the diff's unnecessarily large--again, merging by rebasing could fix this.
Just my $0.02

@jeffwidman thanks for the feedback, this was actually my bad, I accidentally clicked the wrong green button and it wasn't the only case of git history cluttering unfortunately. Shortly after I disabled on github any merge option than "squash and merge" so I hope from now on things will be better :)

@masci I've done this too.
The problem I have with squash and merge is that I typically batch my commits into logical, discrete change sets that are intended to be kept as separate units for easier code-spelunking in the future. If you require squashing, then I have to submit them as separate PRs.
So I generally prefer to allow both squash-and-merge plus rebase-and-merge and then trust the person with commit rights to be intelligent about which they choose.

@jeffwidman thanks for the advice, I'll raise the question around meaningful commit histories for PRs internally with the team. If we get good at it "rebase and merge" definitely makes sense.


Error installing on Linux due to pyodbc needing unixodbc

Describe the bug
When installing qualyspy on Linux, sql functionality does not work if unixodbc is not installed.
Desktop (please complete the following information):

OS: Linux

Will add better driver detection - try/except in importing pyodbc, etc. new db_type parameter in db_connect should also open up to other databases


Chapter 5 help pages

Similarly to chapter 4 these pages need some psuedo code solutions instead of actual answers.
Additionally this chapter is where we really dropped off adding any additional reading resources so we should at least try and add some

Are you currently working on this? I am in Chapter 5 making some other small improvements and started reorganizing some of the resources, specifically around section 3. Just realized you recently merged the updates to chapter 4 resources and want to make sure I'm not stepping on any toes.

I am but go ahead and make any changes you need to, I'm fine rebasing my pr


[BUG] FileNotFoundError:  Notion sync fails with error reading taskrc file.

Describe the bug
tw_notion_sync fails with FileNotFoundError, Apparently due to not being able to read the configuration file appropriately.
I installed the application and tried to sync with a page in Notion. The only changes done to .taskrc is I have applied a theme.
To Reproduce
Steps to reproduce the behaviour:

Install and try using tw_notion_sync with a modified .taskrc (I have a theme applied)

Expected Behavior
Sync should work fine.
📸 Screenshots
Logs:
Traceback (most recent call last):
  File "/xxxxxx/code/venvs/task-sync/bin/tw_notion_sync", line 8, in <module>
    sys.exit(main())
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskwarrior_syncall/scripts/tw_notion_sync.py", line 157, in main
    tw_side = TaskWarriorSide(tags=tw_tags, project=tw_project)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskwarrior_syncall/taskwarrior_side.py", line 53, in __init__
    self._tw = TaskWarrior(marshal=True, config_filename=config_file)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskw/warrior.py", line 438, in __init__
    super(TaskWarriorShellout, self).__init__(config_filename)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskw/warrior.py", line 62, in __init__
    self.config = TaskWarriorBase.load_config(config_filename)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskw/warrior.py", line 180, in load_config
    return TaskRc(config_filename, overrides=overrides)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskw/taskrc.py", line 58, in __init__
    config = self._read(self.path)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskw/taskrc.py", line 106, in _read
    TaskRc(right.strip())
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskw/taskrc.py", line 58, in __init__
    config = self._read(self.path)
  File "/xxxxxx/code/venvs/task-sync/lib/python3.9/site-packages/taskw/taskrc.py", line 96, in _read
    with codecs.open(path, 'r', 'utf8') as config_file:
  File<EMAIL_ADDRESS>line 905, in open
    file = builtins.open(filename, mode, buffering)
FileNotFoundError: [Errno 2] No such file or directory: 'dark-yellow-green.theme'

Versions
OS Version (cat /etc/*-release; uname -a)
Darwin MacBook-Pro.local 21.6.0 Darwin Kernel Version 21.6.0: Wed Aug 10 14:25:27 PDT 2022; root:xnu-8020.141.5~2/RELEASE_X86_64 x86_64
Python version (python --version; python3 --version)
Python 3.9.13
--version of your executable
tw_notion_sync, version 1.2.1
Additional context
Add any other context about the problem here.

Just realized that this might be an issue with taskw. Will update the details.

Hi @csgcode , sorry for the late reply.
Correct, this is an issue with taskw and it's a duplicate of https://github.com/bergercookie/syncall/issues/57. Feel free to open an issue with taskw instead :)

Pending PR in taskw that fixes this: https://github.com/ralphbean/taskw/pull/151


Generic sysprep

Hi Ben,
This pull request extends the factory with the ability to create a genericly sysprepped image (one without specifying a unattend.xml during the sysprep run). This is required for later processing with Virtual Machine Manager or PDT as they inject a unattend.xml themselves.
This pull request is based on the cleanup request I sent you earlier and I hope you are willing to merge this in your tree.
Thanks & BR
Chris

This fixes #2.

I'll open a new pull request off of master.


Requirements for hypertext links

Criterion: “The project MAY use hypertext links to non-project material as documentation, as long as the linked-to information is available and meets the requirements.”
Comment: What requirements does this refer to?

This refers to the documentation requirement above, "The project MUST provide basic documentation for the software in some media (such as text or video) that includes....".
But there's really no need, we can just shorten this to "The project MAY use hypertext links to non-project material as documentation."


[deno] node_stream_zip isn't working

Unable to parse /tmp/mvn-deps-5ODZYX/org/springframework/boot/spring-boot/1.5.1.RELEASE/spring-boot-1.5.1.RELEASE.jar. Skipping. RangeError [ERR_OUT_OF_RANGE]: The value of "length" is out of range. It must be >= 0. Received -38
    at __node_internal_ (ext:deno_node/internal/fs/utils.mjs:713:13)
    at Object.read (ext:deno_node/_fs/_fs_read.ts:57:3)
    at FsRead.read (file:///mnt/work/CycloneDX/cdxgen/node_modules/.deno/node-stream-zip@1.15.0/node_modules/node-stream-zip/node_stream_zip.js:979:16)
    at FsRead.readCallback (file:///mnt/work/CycloneDX/cdxgen/node_modules/.deno/node-stream-zip@1.15.0/node_modules/node-stream-zip/node_stream_zip.js:998:18)
    at ext:deno_node/_fs/_fs_read.ts:72:7
    at eventLoopTick (ext:core/01_core.js:63:7) {
  code: "ERR_OUT_OF_RANGE",
  name: "RangeError"
}

Needs to be rewritten using<EMAIL_ADDRESS>

https://github.com/denoland/deno/issues/22238

Working fine with deno 1.46.1
https://github.com/denoland/deno/issues/22238#issuecomment-2310976759


GraphQL server+client example

This is a feature idea.
Add:

example realistic (-ish) server which exposes a GraphQL endpoint.
example realistic UI module which consumes that.
use Apollo libraries in both.

Justification:
For a number of years, REST APIs have been the standard most enterprise data system projects aim for. While this has been an improvement over many past practices, it is also been found to not scale very well in terms of system size. In particular, it often leads to manual work composing together different API aspects so that they can be consumed efficiently from a user interface.
GraphQL solves this problem, by raising the level of abstraction.

Server work is underway on this, it's not in use in the user interface yet.


kChildIdMask maybe error

https://github.com/openthread/openthread/blob/c9f23ccddaaa08f4c5cc2b8a9160eac1e2b4a2e9/src/core/thread/network_diagnostic_tlvs.hpp#L1036
is kChildIdMask 0x1ff ?

@nykh2010 Thanks for catching this issue, I just created https://github.com/openthread/openthread/pull/7531 to fix it.


Only use default parameter values for path parameters (#450)

This change allows endpoint functions to be defined with parameters with defaults, but for those defaults to be ignored if the function parameter subsequently appears as a path parameter. The default value is only operative if the parameter is a query parameter.
This change includes a new set of tests for this functionality.

WIP: this approach is too aggressive as it ignores all Path() specifications. Will fix later.

As far as I can see this should be mergeable. The travis runs are raising errors with incorrectly sorted imports in masses of other files unrelated to this commit, so I'm not sure what's going on there.

Install locally with flit as described in the doc
Le lun. 26 août 2019 à 9:13 AM, jonathanunderwood<EMAIL_ADDRESS>a écrit :

As far as I can see this should be mergeable. The travis runs are raising
errors with incorrectly sorted imports in masses of other files unrelated
to this commit, so I'm not sure what's going on there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tiangolo/fastapi/pull/464?email_source=notifications&email_token=AAINSPXBZIY3HT32XIEBNETQGN7BNA5CNFSM4IPI77S2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5DPYRQ#issuecomment-524745798,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAINSPUAYBDIVXS5M2Z4QFLQGN7BNANCNFSM4IPI77SQ
.

OK, fixed the import order issue. All should be good now.

OK, fixed the import order issue. All should be good now.

been bitten by it before too ;) https://github.com/tiangolo/fastapi/issues/38#issuecomment-464840923

@tiangolo any feedback on this one?

Thanks for the help here @euri10 ! :bowing_man:
@dmontagu if you have time, could you give this a quick review?

All suggested formatting changes have been added.

MyPy tests are now failing in code unrelated to this change, which is strange.

Thanks for the format changes!
Yeah, the same mypy error just happened in a couple of other PRs. It's related to some changes in a new version of mypy. I just rebased your branch from master, as it has those fixes (from these recent PRs). Let's see if that fixes it.

Thanks for your contribution @jonathanunderwood ! :cake: :tada: :rocket:

Thanks for taking the time to review this change, much appreciated.


[Bug] Debug version run failed

Checklist

[x] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.

Describe the bug
[TM][DEBUG] void turbomind::cublasMMWrapper::Gemm(cublasOperation_t, cublasOperation_t, int, int, int, const void*, int, const void*, int, void*, int)
[TM][DEBUG] void turbomind::cublasMMWrapper::Gemm(cublasOperation_t, cublasOperation_t, int, int, int, const void*, int, const void*, int, void*, int, float, float)
terminate called after throwing an instance of 'std::runtime_error'
  what():  [TM][ERROR] CUDA runtime error: misaligned address /data/lmdeploy/lmdeploy/src/turbomind/utils/cublasMMWrapper.cc:303

I compiled the debug version, and replace the so file.
cp ./lib/_turbomind.cpython-38-x86_64-linux-gnu.so /opt/conda/lib/python3.8/site-packages/lmdeploy/lib/_turbomind.cpython
-38-x86_64-linux-gnu.so

But the run failed.
Any advice? thank you!
Reproduction
debug version
#!/bin/sh

cmake ..

@AllentDan Could you help check it?

@AllentDan Could you help check it?

Yes, I reproduced the error.

May update the latest version

@AllentDan Hi this problem still occurred using the latest main branch.
terminate called after throwing an instance of 'std::runtime_error'
  what():  [TM][ERROR] CUDA runtime error: misaligned address /opt/lmdeploy/src/turbomind/models/llama/unified_attention_layer.cc:353

Aborted (core dumped)

cmake .. \
-DCMAKE_BUILD_TYPE=Debug \
-DCMAKE_EXPORT_COMPILE_COMMANDS=1 \
-DCMAKE_INSTALL_PREFIX=/opt/tritonserver \
-DBUILD_PY_FFI=ON \
-DBUILD_MULTI_GPU=ON \
-DBUILD_CUTLASS_MOE=OFF \
-DBUILD_CUTLASS_MIXED_GEMM=OFF \
-DCMAKE_CUDA_FLAGS="-lineinfo" \
-DUSE_NVTX=ON

I build using -DCMAKE_BUILD_TYPE=Debug, is that ok ?

What model are you using?

What model are you using?

vicuna 13B, I post it here.  https://github.com/InternLM/lmdeploy/issues/2399#issuecomment-2320739338

Sorry, I didn't get the issue.
What's the motivation? Are you suggesting lmdeploy perform vicuna inference on V100 failed?

Sorry, I didn't get the issue. What's the motivation? Are you suggesting lmdeploy perform vicuna inference on V100 failed?

I first get this issue on V100 a few months ago, and last week I get it again.
So I post another issue to describe it more clearly.
The issue is: when compiling lmdeploy using Debug mode, it crashed when running inference.

OK. I get it.
Let me try to verify the target built with DEBUG mode


Support percentiles for aggregated metrics

<>
Per this StackOverflow answer, it’s not possible to do percentiles on aggregated metrics sent through AppInsights.
https://stackoverflow.com/questions/58124268/how-to-do-percentiles-on-custom-metrics-in-azure-appinsights
The request is to support this in some form, since it seems like a significant miss relative to other platforms like Prometheus.
Is there any workaround other than sending telemetry for every metric measurement (since that won’t scale at all)?
I would love not to have to set up Prometheus/Grafana infrastructure to support this.  Thanks!

@vgorbenko Is this something in Metrics roadmap.. ?

Any indication of how close this might be?
For high-volume scenarios it makes AppInsights unusable for metrics (since simple averages won't cut it for production monitoring).  If there's a solution AppInsights provides here that I'm missing please let me know (our plan is to track aggregate metrics for billions of events/day).

@andyvig This is not planned for 2019. I will check and report back the plan for next semester. (2020).
I also know that its possible for you to write custom aggregator and plug into rest of metrics pipeline if you want to do percentlies. Its not documented, but if you want to take a look, heres where to start looking:
https://github.com/microsoft/ApplicationInsights-dotnet/blob/develop/src/Microsoft.ApplicationInsights/Metrics/Extensibility/MetricSeriesAggregatorBase.cs

Thanks @cijothomas, how would we then query that on the Log Analytics side?  Does the percentile function support aggregate data?
I'm looking for something similar to this operation in Prometheus:
"To calculate the 90th percentile of request durations over the last 10m"
histogram_quantile(0.9, rate(http_request_duration_seconds_bucket[10m]))
From https://prometheus.io/docs/prometheus/latest/querying/functions/#histogram_quantile

Don't think there exists any native support as schema dont have anything for storing percentiles.https://github.com/microsoft/ApplicationInsights-dotnet/blob/develop/src/Microsoft.ApplicationInsights/Extensibility/Implementation/External/DataPoint_types.cs
You'd need to store quantiles as customProps, and do custom queries to get them, as Analytics wont understand customProps.
@SergeyKanzhelev even if one authors own aggregator, any way to store quantiles (.1,.5..9 etc) in schema?

Any news / roadmap item / documentation / customer guidance of

publishing metrics as histograms to AppInsights
with the goal of using percentiles in Queries/Views/Alerts

to make AppInsights a good fit for SLOs?

No work is planned to add support for this in ApplicationInsights SDK.
The Metrics support in OpenTelemetry is coming by end of 2021 (nov 2021) - https://github.com/open-telemetry/opentelemetry-dotnet/issues/1501.
After the OpenTelemetry part is shipped, there'd be a supported way to export metrics to ApplicationInsights, but no solid dates for this. Also no solid date for supporting percentiles/histogram in ApplicationInsights.

@cijothomas Checking in here...
This still a "maybe sometime in the future but all dates unknown" situation or is there any more definition around if/when this might be supported?  Thanks.

No firm dates that I can share. (the feature requires not just SDK support, but backends/UI etc.). From SDK side, this will likely come via OpenTelemetry route, and not from this repo.

We still don't have a clear statement, if and how this will come.
If this feature is not coming in the Azure Monitor / AppInsights backend and various SDKs, there should be some guidance published, how these technologies could be used if someone wants to follow SRE best practices:

SRE SLO Aggregations
SRE Monitoring Calculations

Hi Ricardo, Azure Managed Prometheus (Preview) was announced last month and is available with Azure Managed Grafana integration. This is compatible with Prom Client.
https://learn.microsoft.com/azure/azure-monitor/essentials/prometheus-metrics-overview
Additionally we are working on supporting percentiles via the OpenTelemetry histogram API. Unfortunately this work requires some major changes in how our backend works and thus any release is 6+ months out.
CC: @vishiy

Thanks a lot for clarification and details around workarounds/other possibilities.

@mattmccleary can you provide an update on this? Thx

@mattmccleary any update on percentile tracking support?
I'm trying to use 'Azure.Monitor.OpenTelemetry.Exporter' to collect and report on our application latency. Looks like currently it tracks things like max value, but it's not very useful for practical purposes since max value can be influenced by a lot of external factors and doesn't necessarily provide an accurate view of how the app is doing. Ideally we want to track the 99-th percentile of this latency value, but I can't figure out how to do that or if it's supported at all.


type list

'At the moment we don't supply a type list for the IMPORT if IMPORT_FROM_ORA is used. Only in the JDBC case. Background is the assumption that types don't map well between JDBC and ORA connection. But they'd like to challenge this assumption. Could you create  an experimental branch where you remove the case distinction and try adding the types to the IMPORT statement'

we made an experimental branch providing a typelist which solves the #1 issue here, namely conversion to varchar instead of to decimal. next steps are to provide a switch and provide a clean implementation.


Support querying multiple repositories sequentially

OK, I backed out the filenamify change and will raise it again as a follow-up. I'll open an issue detailing the problems.


Deployment Tracker Service is discontinued

Please refer to the following document for details and migration instructions https://github.com/IBM-Bluemix/cf-deployment-tracker-client-node/wiki/migrating-to-the-new-metrics-tracker-client.

This was fixed by removing the deployment tracker


Godot 3 supported?

Does this project work with Godot 3?

any help needed? I can do Python )

Amazing, will try it out the second I am home. Thank you so much for the effort!

You rock!


Improved: carousel time in product carousel (#444)

On mobile devices if you swipe the carousals, it takes time to get stationary.
Experience should be smooth while swiping the carousal
Reduced the, animationDuration and rewindDuration to 2000 from 3000
Related Issues

Closes #444
Short Description and Why It's Useful

Screenshots of Visual Changes before/after (If There Are Any)

IMPORTANT NOTICE - Remember to update CHANGELOG.md with description of your change
Contribution and Currently Important Rules Acceptance

[ ] I read and followed contribution rules

@AishwaryShrivastav please add changelog


pub shuffle

fix #292
please don't merge until removing the mintable tokens query

closing, part of this code was used in a different pull


STORM-969: HDFS Bolt can end up in an unrecoverable state

A few notes about this PR:

I updated the storm-hdfs pom.xml to align with other external modules.  Most significant change was probably going from hdfs version 2.2 to ${hadoop.version} (i.e. currently 2.6)

Many errors are recovered by forcing a file rotation which opens a new, valid File.  So the rotation now occurs either according to rotation policy or when a serious error happens.  Work could probably be done to reopen the same file name to reduce the number of rotations.

Added unittests with MiniDFSCluster

@arunmahadevan since you've made changes to HdfsBolt as part of STORM-837 can you take look at this PR check if its already handled by your changes.

@harshach the changes I made were in the trident implementation (HDFSState) which is independent of this. Anyways I reviewed the changes.
Overall it appears that the exceptions are now handled individually at write, sync and rotate phases. But I see a few issues with the change.

the tuples are acked only on sync - if the tuple rates are low, they will never be acked and keep timing out and same tuples will be replayed again and again.
there is an attempt to sync even when the write fails with an IOException. Since write already threw an IOException, chances are high that the sync would also fail with IOException.

I think it might be simpler to keep the existing logic and just rotate the file whenever we see an IOException (or maybe after a few times we repeatedly hit the IOException) and completely fail by propagating the exception up if the situation does not improve after a few rotations.
Also I see that the existing implementation acks the tuples before actually syncing them to disk, which might result in data loss. I think we should change this to ack only after an hsync and have a sync policy that considers both count and time based thresholds.

@arunmahadevan Thank you for the feedback!  I have added a tick tuple feature to address your first point (I am already using this locally, I forgot to include it in this PR).
You are right that I attempt to sync even when the write fails.  I agree that this is unlikely to succeed, but it seems worthwhile to try to "save" as many tuples as possible.
With this tick tuple change I think the code is very close to your recommendation.  If the bolt is in a bad state the tick tuples will periodically try to rotate the file and all errors will be reported up through this.collector.reportError(e)

Thank you @harshach and @arunmahadevan, I will incorporate this feedback soon.

@harshach @arunmahadevan Changes made based on feedback, PR also rebased and squashed to one commit.  Additional feedback welcome, thank you!

+1 overall the patch looks good once the [comment] (https://github.com/apache/storm/pull/664#commitcomment-13003095) is addressed

@arunmahadevan Our approach is to set the tick tuple frequency to be half of the message timeout setting for the topology.  Can the bolt get access to that topology setting? prepare() passes a TopologyContext to the bolt but I don't see a way to get configuration information out of it.  However, I am not that familiar with TopologyContext.
If that's not possible, the bolt could just set a flush frequency of 15 seconds since the storm timeout default is 30 seconds and assume that anyone changing the timeout setting should also be adjusting the flush frequency as well

The value of topology.message.timeout.secs can be read from the conf that gets passed as the first parameter to doPrepare

@arunmahadevan So it can!  PR updated.

@arunmahadevan @harshach Do you have any further comments about my updated PR? Thanks!

Looks good to me.

@harshach (or other commiters) do you have feedback about this PR?
Anecdotally, this has been very useful to us in production. We had an HDFS restart, which created the exact situation I tested with (failed writes due to HDFS safe mode) but the bolt recovered without a topology restart.

+1

Thanks @dossett merged into master.


Typescript: project should include a definition file for files with .vue extension

Projects using typescript should include a definition file:
vue-shim.d.ts
declare module "*.vue" {
  import Vue from 'vue'
  export default Vue
}

this allows importing .vue modules inside .ts files ( for unit tests for instance)

Let’s see what @kevinmarrec thinks about this :)

It's mentionned in documentation : https://typescript.nuxtjs.org/guide/setup.html#configuration

Documentation ... who has time to read them ? 😁😜
More seriously, why not add that file directly when typescript is selected ?

Oh yeah nvm it should be indeed added by CNA :)

Adding it to the repo will indeed be helpful as now doing it with documentation gives error:
Parsing error: Imports within a `declare module` body must always be `import type` or `import typeof`

...and trying to modify piece from docs to be:
declare module '*.vue' {
    import type Vue from 'vue';
    export default Vue;
}

...gives following error:
Parsing error: Only declares and type imports are allowed inside declare module

@Reeywhaar
This PR adds it : https://github.com/nuxt/create-nuxt-app/pull/449
And about your error, I never had this one, I think you have a mistake in your configuration, please open a new issue on TypeScript repo with reproduction so I can find a fix for you.

@Reeywhaar I think it's babel error, you may didn't look at : https://typescript.nuxtjs.org/guide/lint.html#configuration (see Warning)


disregard the commented code

Hey, thanks for your hard work. I am very happy with this extension.
As a suggestion, it would be great if the extension will disregard comment code.
For instance, if I try to run
-- select count(id) from users;
select * from users;

I will get a syntax error. Same thing if the comment is in this fomat: /*  select count(id) from users;  */
I also tried to select what I want to be executed before doing so with no success.
Thanks!
Sergio

Fixed in 2.2.0 version