[Update Snap] Tenderly

Checklist
All items in the list below needs to be satisfied.

[x] Is the summary of the change documented in this ticket? https://github.com/Tenderly/tenderly-snap/compare/v1.2.2...v1.2.3
[ ] Has a MetaMask Snaps team member reviewed whether the changes need to be vetted?
[ ] If there are changes that need to be vetted, attach a description and the relevant fixes/remediations to this issue.
[ ] The corresponding pull request in this repo has been merged.

This fixes an issue with parsing chain IDs. The new version is 1.2.3


Fix typo in README

README shows account.getChainAddress(1) being called two times in a row and returning two different addresses. Updating 2nd call to account.nextChainAddress(1) to return next address.

It's also possible you mean to to do this:
console.log(account.getChainAddress(0))
// => 1QEj2WQD9vxTzsGEvnmLpvzeLVrpzyKkGt

console.log(account.getChainAddress(1))
// => 1DAi282VN7Ack9o5BqWYkiEsS8Vgx1rLn

account.nextChainAddress(1)

console.log(account.getChainAddress(1))
// => 1CXKM323V3kkrHmZQYPUTftGh9VrAWuAYX

In which case the error was account.nextChainAddress(0) instead of account.nextChainAddress(1) being called in between each call to account.getChainAddress(1)

Well spotted :+1:


Use NonZeroUsize from standard library

miri complains about reference hack. Note that there is still ridiculous amount of UB, refer to #41, but may as well start removing it somehow.

Looks good, thanks!
@bors-servo r+

:pushpin: Commit ff9c8a7 has been approved by SimonSapin

:hourglass: Testing commit ff9c8a7e864dbd30131cffd70c593dd74edb6407 with merge dc22ac0e1d8301b92bb781702034831065076986...

:sunny: Test successful - checks-travis
Approved by: SimonSapin
Pushing dc22ac0e1d8301b92bb781702034831065076986 to master...


Add documentation for onMomentumScrollEnd

Absolutely! Also, FWIW, I don't think that onScrollAnimationEnd is ever called...
https://snack.expo.io/BJyf5OGIW
https://github.com/facebook/react-native/blob/603cc48ceba001827d10231d51b4031c7768eef8/React/Views/RCTScrollView.m#L765

@tomasreimers Yeah... You are right. Thank you!
I think, we also have to do:

Remove onScrollAnimationEnd from documentation and delete all (unused) mentions of it from Android and iOS sources;
Implement onMomentumScrollBegin on iOS (it already should work on Android) and add this to documentation.

I will add it to my plans. (Or a PR is always welcome. 😄)

@shergin
(1) Done: https://github.com/facebook/react-native/pull/15156 (Nothing at FB is someone else's problem, right? ;) )
(2) Already done: https://github.com/facebook/react-native/blob/968aa56ca38b6ad7f08a08081321e799ee015322/ReactAndroid/src/main/java/com/facebook/react/views/scroll/ReactHorizontalScrollView.java#L263

Awesome! Now we just have to mention this in the documentation.

@shergin  https://github.com/facebook/react-native/pull/15158


Fix aggregation error when inserting negative timestamp

Description
Before fix, the result of select count(*) from root.** is always 0, now it will be the right value.

Codecov Report

Merging #11491 (548ae7c) into master (fd3911f) will increase coverage by 0.01%.
Report is 3 commits behind head on master.
The diff coverage is 85.71%.

@@             Coverage Diff              @@
##             master   #11491      +/-   ##
============================================
+ Coverage     48.94%   48.95%   +0.01%
- Complexity    24582    24590       +8
============================================
  Files          2772     2772
  Lines        173261   173297      +36
  Branches      20764    20770       +6
============================================
+ Hits          84801    84836      +35
- Misses        88460    88461       +1

Files
Coverage Δ

...bc/src/main/java/org/apache/iotdb/jdbc/Config.java
100.00% <ø> (ø)

...ueryengine/execution/operator/AggregationUtil.java
84.04% <100.00%> (ø)

.../iotdb/db/storageengine/dataregion/DataRegion.java
50.68% <100.00%> (+0.06%)
:arrow_up:

...a/org/apache/iotdb/commons/conf/IoTDBConstant.java
90.32% <ø> (ø)

...ngine/plan/planner/plan/node/write/InsertNode.java
86.13% <66.66%> (-0.60%)
:arrow_down:

... and 26 files with indirect coverage changes
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Assume user is not overloading the Elm name in imports

I think it's more likely that users will import as:
import Elm
import Elm.Op

than as
import Elm
import Elm.Op as Elm

so I think the examples in the docs should probably reflect this. Unless you're expecting folks to alias the imports?

Good catch!


Switch Mac intel tasks from Cirrus to LUCI

This is a follow up of
https://github.com/flutter/plugins/pull/6637
https://github.com/flutter/plugins/pull/6671
This PR enables these three tasks in LUCI prod and disables them from Cirrus:
darwin_lint_podspecs
build_all_plugins master
build_all_plugins stable

These three tasks have been passing consistently in CI:
https://ci.chromium.org/p/flutter/builders/staging/Mac_x64 lint_podspecs
https://ci.chromium.org/p/flutter/builders/staging/Mac_x64 build_all_plugins master
https://ci.chromium.org/p/flutter/builders/staging/Mac_x64 build_all_plugins stable
(part of https://github.com/flutter/flutter/issues/114373)

Manually land to green the tree.


[Network] az network application-gateway waf-policy managed-rule rule-set add/update: Add allowed value 1.1 for --version

Related command

Description

Resolve #29158
Testing Guide

History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Thank you for your contribution! We will review the pull request and get back to you soon.


envoy mobile stats: remove base_h2 stats

Commit Message: Remove base_h2 cluster stats from stats inclusion list as base_h2 cluster was removed and is no longer used.
Additional Description:
Risk Level: Low
Testing: N/A
Docs Changes:
Release Notes:
Platform Specific Features:

/retest


custom role name is not respected in "Roles" pane?

I'm trying to setup filtering in AI to be able to easy switch between different (but related) projects. I want to use Roles pane for that. It works fine for server events - if I defined a custom role name, I will see it in the "Roles" pane, but it does not work for frontend (JS SDK). Here is how I initialize JS snippet based on official FAQ and some googling:
window.appInsights = appInsights;

appInsights.queue.push(() => {
    appInsights.context.addTelemetryInitializer((envelope) => {
        envelope.tags["ai.device.roleName"] = "today.ya";
        envelope.tags["ai.cloud.role"] = "today.ya";
        envelope.tags["ai.cloud.roleInstance"] ="YAWEB02";
    });
});
appInsights.trackPageView();

but that does not help. Even though "role" is passed through (so I can manually filter by it, as shown on screenshot), it does not appear in the Roles pane. Any ideas how to get it to work?

I'm still using older version of the SDK - ai.0.js - if that matters.
And needless to say, i play with different combinations of ai.device.roleName, ai.cloud.role before just dumping all of them - but neither worked.

I tried the old version SDK with snippet and it seems working fine to me, is this what you are expecting?  Can you please retry?
Here's the snippet I used: https://github.com/microsoft/ApplicationInsights-JS/blob/master/snippet/snippet.html

yes - snippet EXACTLY is the same, the only difference is the bottom part. Instead of
 window.appInsights=appInsights,appInsights.queue&&0===appInsights.queue.length&&appInsights.trackPageView();
I have this:
window.appInsights = appInsights;

appInsights.queue.push(() => {
    appInsights.context.addTelemetryInitializer((envelope) => {
        envelope.tags["ai.device.roleName"] = "today.ya";
        envelope.tags["ai.cloud.role"] = "today.ya";
        envelope.tags["ai.cloud.roleInstance"] ="YAWEB02";
    });
});
appInsights.trackPageView();

not sure if that's the same thing? How did you add role name into that snippet?
And yes - your screenshot is exactly what I'm looking for to get.
Thank you!

Hi I tried both put your code right after the snippet, and remove the line and have your code there and both worked fine on my machine.

@avs099 And do you see Network traffic like this in your browser? The traffic is called "track"

i do:

the weird part is this code works for another AI instance (brand new) for another project. Here I was trying to add new app to existing AI instance. Not sure if that matters but still. Do you have any connections in MS support team where I can file all details? It looks like AI issue to me. Data IS THERE - i can see it - with cloud_RoleName setting - but it's not showing up on "Roles" pane.

let me ask around and get back to you later. :)

Hi @avs099 , the reason role is not appearing is because the current query to get the list of roles only consider "requests, dependencies, exceptions, and performanceCounters tables". If you have only pageViews and no dependencies, the role won't appear. The support team is working on this to get it fixed and the work around for now is to send dependencies.

Thank you.

I don't have an open issue with this, I'll follow it up and get back to you when there's an update.


Bug with importing tensorflow in Streamlit Cloud

https://discuss.streamlit.io/t/unable-to-import-tensorflow/37615

keras<=2.7.0
tensorflow<=2.2.0rc1


Non-Steam Games is Not Working

Hi, the idea of your project is awesome. But i couldnt play with non steam game. I sent the invite and stream started but when the guest trying to connect, it stucks at loading screen.
My non steam game is from Xbox - Microsoft Store, this could be the cause of the problem
EDIT: Yes, the cause of the problem is that i am trying to play Xbox Store game. The other games working perfectly. If you can add this feature it would be great!
Thanks for this tool

Does steam overlay work for said games?

No, steam overlay dont work ever

I'm having a similar issue, the game (in my case the yuzu emulator) is detected as a game and I can invite people, however they are stuck at a black loading screen. I'm on arch linux, this happens in both Vulkan and OpenGl rendering modes.

I'm playing Overcooked from Epic Games Store opened from Steam as non-steam game, with Steam-overlay enabled while overlay is working.
Still, my friend does not receive any invites when I invite them.

well uh am pretty sure Remote Play is only for Steam??

well uh am pretty sure Remote Play is only for Steam??

What do you mean? The repo is about non-steam games. Sry but what are you on about? :'D


Dev 25433

@loaiabdelhalim Pipeline isn't running, you might need to resolve git conflicts

@loaiabdelhalim Pipeline isn't running, you might need to resolve git conflicts

Done


added advertiseFrom field to the channel and annotatioons

You need to also update:

the Deer version of the model
the code the transforms the Owl model to the Deer model
the protocol buffers serialisation and deserialisation to ensure this is stored in C*

LGTM

No tests

Update the LegacyChannelTransformer test to check you are correctly transforming the new field.

See if ChannelWriter has a test class. If it does add a test there. Otherwise LGTM


Custom RBAC Roles for Azure Lighthouse CSP Partners

Is there any future release of Azure Lighthouse that will allow the use of custom RBAC roles. I have a requirement for a Virtual Machine Contributor to be able to snapshot the server prior to applying Windows Updates. This cannot be achieved currently through the existing RBAC roles within Azure Lighthouse.
Thank you
Don Turner

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

ID: 81c5a2e6-40bd-f6a0-5936-45064d3f684f
Version Independent ID: eed9e692-5db7-e935-638d-b15b844760cc
Content: Tenants, roles, and users in Azure Lighthouse scenarios - Azure Lighthouse
Content Source: articles/lighthouse/concepts/tenants-users-roles.md
Service: lighthouse
GitHub Login: @JnHs
Microsoft Alias: jenhayes

@andrewsturner783 Thanks for your comment! We will review and provide an update as appropriate.

Hi @andrewsturner783 - Thanks for your question. While I don't have any information to share about future plans, support for custom roles has been suggested in our UserVoice feedback forum: https://feedback.azure.com/forums/922753-azure-lighthouse/suggestions/38901802-add-support-for-custom-roles
The team reviews these regularly to help plan new/enhanced features, so please consider upvoting this suggestion and feel free to submit any other feature requests there.

@JnHs Thanks for the insights. 👍
@andrewsturner783  Responsible feature / product team has confirmed the same that feature ask of custom RBAC roles is in current backlog and right now there is no timeline on the release. But has informed that this scenario will be added to the list of custom RBAC roles requirement. Meanwhile you may upvote the ask in above mentioned UserVoice feedback forum to provide more data to us to prioritize the asks.
At this point it appears that there's no edits or updates needed for the referred Azure documentation so closing this issue for now. If there are further questions regarding this matter or questions or feedback on the referred Azure documentation, please do let us know and we will gladly continue the discussion.

Any update on this feature? The link to the suggestion throws 404.

Apologies for the bad link, our feedback system has changed. I'm not able to provide any update on plans for this, but you can find an equivalent suggestion here: https://feedback.azure.com/d365community/idea/148efdcb-f824-ec11-b6e6-000d3a4f0da0


NameError: name 'x_train' is not defined

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): Custom Code
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Google Colab (GPU)
TensorFlow installed from (source or binary): pip install
TensorFlow version (use command below): 1.15.0
Python version: 3.6

Describe the current behavior
Error when using fit_generator():
ModuleNotFoundError: No module named 'tensorflow_core.compat'
Describe the expected behavior
The model should begin training.
Code to reproduce the issue
from google.colab import drive
drive.mount('/content/drive')

!pip install --quiet tensorflow==2.0.0-rc0
!pip install --quiet neural-structured-learning

from __future__ import absolute_import, division, print_function, unicode_literals

# TensorFlow and tf.keras
import tensorflow as tf
from tensorflow import keras
from tensorflow.keras import datasets, layers, models
from tensorflow.keras.preprocessing.image import ImageDataGenerator

print(tf.__version__)

import numpy as np
import matplotlib.pyplot as plt
import os
import cv2
from tqdm import tqdm
import pathlib
import random

train_data_dir = "/content/drive/My Drive/Resize/train"
train_label_dir = pathlib.Path(train_data_dir)

test_data_dir = "/content/drive/My Drive/Resize/test"
test_label_dir = pathlib.Path(test_data_dir)

CATEGORIES = np.array([item.name for item in train_label_dir.glob('*') if item.name != "LICENSE.txt"])
class_names = CATEGORIES
print(CATEGORIES)

def createdataset(DATADIR, label_dir, CATEGORIES, img_size):
    image_count = len(list(label_dir.glob('*/*.jpg')))
    print(image_count)

    # for category in CATEGORIES:  # do dogs and cats
    #     path = os.path.join(DATADIR,category)  # create path to dogs and cats
    #     for img in os.listdir(path):  # iterate over each image per dogs and cats
    #         img_array = cv2.imread(os.path.join(path,img) ,cv2.IMREAD_GRAYSCALE)  # convert to array
    #     #     break  # we just want one for now so break
    #     # break  #...and one more!

    IMG_SIZE = img_size

    datalist = []

    for category in CATEGORIES:  # do dogs and cats

        path = os.path.join(DATADIR,category)  # create path to dogs and cats
        class_num = np.where(CATEGORIES == category)

        for img in tqdm(os.listdir(path)):  # iterate over each image per dogs and cats
            try:
                img_array = cv2.imread(os.path.join(path,img) ,cv2.IMREAD_COLOR)  # convert to array
                new_array = cv2.resize(img_array, (IMG_SIZE, IMG_SIZE))  # resize to normalize data size
                datalist.append([new_array, class_num])  # add this to our training_data
            except Exception as e:  # in the interest in keeping the output clean...
                pass
            #except OSError as e:
            #    print("OSErrroBad img most likely", e, os.path.join(path,img))
            #except Exception as e:
            #    print("general exception", e, os.path.join(path,img))
    return datalist

training_dataset = createdataset(train_data_dir, train_label_dir, CATEGORIES, 200)
testing_dataset = createdataset(test_data_dir, test_label_dir, CATEGORIES, 200)

print(len(training_dataset))
print(len(testing_dataset))

def dataset(datasets):
    xdata = []
    ylabels = []
    # random.shuffle(datasets)
    for datas,labels in datasets:
        xdata.append(datas)
        ylabels.append(labels)
    return xdata, ylabels

train_images, train_labels = dataset(training_dataset)
test_images, test_labels = dataset(testing_dataset)
print(len(train_images))
print(len(train_labels))
print(len(test_images))
print(len(test_labels))

train_images = np.array(train_images)
test_images = np.array(test_images)
train_labels = np.array(train_labels)
test_labels = np.array(test_labels)

print(train_images.shape)
print(test_images.shape)

# print(train_labels)
# print(test_labels)

train_images = train_images / 255.0
test_images = test_images / 255.0

# plt.figure(figsize=(10,10))
# for i in range(25):
#     plt.subplot(5,5,i+1)
#     plt.xticks([])
#     plt.yticks([])
#     plt.grid(False)
#     plt.imshow(train_images[i], cmap=plt.cm.binary)
#     plt.xlabel(class_names[train_labels[i]])
# plt.show()

epochs = 5
batch_size = 50

datagen = ImageDataGenerator(
    featurewise_center=True,
    featurewise_std_normalization=True,
    rotation_range=20,
    width_shift_range=0.2,
    height_shift_range=0.2,
    horizontal_flip=True)

print("Augmenting the shitty dataset")
datagen.fit(train_images, augment=True, rounds=5)

model = models.Sequential()

model.add(layers.Conv2D(32, (3, 3), input_shape=(200,200,3)))
model.add(layers.BatchNormalization(axis=-1))
model.add(layers.Activation('relu'))
model.add(layers.Conv2D(32, (3, 3)))
model.add(layers.BatchNormalization(axis=-1))
model.add(layers.Activation('relu'))
model.add(layers.MaxPooling2D(pool_size=(2,2)))

model.add(layers.Conv2D(64,(3, 3)))
model.add(layers.BatchNormalization(axis=-1))
model.add(layers.Activation('relu'))
model.add(layers.Conv2D(64, (3, 3)))
model.add(layers.BatchNormalization(axis=-1))
model.add(layers.Activation('relu'))
model.add(layers.MaxPooling2D(pool_size=(2,2)))

model.add(layers.Flatten())

# Fully connected layer
model.add(layers.Dense(512))
model.add(layers.BatchNormalization())
model.add(layers.Activation('relu'))
model.add(layers.Dropout(0.2))
model.add(layers.Dense(4))

model.add(layers.Activation('softmax'))

model.summary()

model.compile(optimizer='adam',
              loss='sparse_categorical_crossentropy',
              metrics=['accuracy'])

model.fit_generator(datagen.flow(x_train, y_train, batch_size=32),
                    steps_per_epoch=len(x_train) / 32, epochs=epochs)

test_loss, test_acc = model.evaluate(test_images,  test_labels, verbose=2)

print('\nTest accuracy:', test_acc)

Other info / logs
ERROR:root:Internal Python error in the inspect module.
Below is the traceback from this internal error.

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/interactiveshell.py", line 2882, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-22-ce52d11f71b2>", line 1, in <module>
    model.fit_generator(datagen.flow(x_train, y_train, batch_size=32),
NameError: name 'x_train' is not defined

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/interactiveshell.py", line 1823, in showtraceback
    stb = value._render_traceback_()
AttributeError: 'NameError' object has no attribute '_render_traceback_'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/ultratb.py", line 1132, in get_records
    return _fixed_getinnerframes(etb, number_of_lines_of_context, tb_offset)
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/ultratb.py", line 313, in wrapped
    return f(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/ultratb.py", line 358, in _fixed_getinnerframes
    records = fix_frame_records_filenames(inspect.getinnerframes(etb, context))
  File "/usr/lib/python3.6/inspect.py", line 1490, in getinnerframes
    frameinfo = (tb.tb_frame,) + getframeinfo(tb, context)
  File "/usr/lib/python3.6/inspect.py", line 1448, in getframeinfo
    filename = getsourcefile(frame) or getfile(frame)
  File "/usr/lib/python3.6/inspect.py", line 696, in getsourcefile
    if getattr(getmodule(object, filename), '__loader__', None) is not None:
  File "/usr/lib/python3.6/inspect.py", line 733, in getmodule
    if ismodule(module) and hasattr(module, '__file__'):
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/__init__.py", line 50, in __getattr__
    module = self._load()
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/__init__.py", line 44, in _load
    module = _importlib.import_module(self.__name__)
  File "/usr/lib/python3.6/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 994, in _gcd_import
  File "<frozen importlib._bootstrap>", line 971, in _find_and_load
  File "<frozen importlib._bootstrap>", line 953, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'tensorflow_core.compat'

The real error is
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/IPython/core/interactiveshell.py", line 2882, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-22-ce52d11f71b2>", line 1, in <module>
    model.fit_generator(datagen.flow(x_train, y_train, batch_size=32),
NameError: name 'x_train' is not defined

This points to some lines of code missing in your code sample. You should define values for x_train and y_train

Great that worked.

Hi, I am getting the same error can you tell me how you resolved it ? @jonathanrjpereira


Regression on fetchAndLock long poll (asyncResponseTimeout has no effect)

At some point, asyncResponseTimeout has stopped having effect. It was previously fixed for https://github.com/camunda-community-hub/micronaut-camunda-platform-7/issues/353
Now, with micronaut 3.6.1 and micronaut-camunda-bpm-feature 2.9.0, it is no longer possible to long poll external tasks:
$ date --iso-8601=ns && curl -X POST http://localhost:8080/engine-rest/external-task/fetchAndLock -H "Content-Type: application/json" -d'{"maxTasks": 1, "workerId": "aWorkerId", "asyncResponseTimeout": 2000, "topics": [{"topicName": "myTopic", "lockDuration": 3000}]}' && printf "\n" && date --iso-8601=ns
2022-10-01T15:23:39,356950496+03:00
[]
2022-10-01T15:23:39,384599763+03:00

demo.zip

Closed as invalid.
I had already forgotten that this feature had issues when run with maven with mn:run.

Hi @datakurre ,
thanks for reporting and closing this issue.
It shows that you're still using the integration project 😀

Thanks for maintaining!
That said, I might re-open this in the future, once I have more time to investigate.
The behavior that I'm seeing (on 2.8.1) and right now: It does keep worker waiting for asyncResponseTimeout, but also returns the locked tasks only after asyncResponseTimeout, when it should return them as soon as they have been released. :sweat_smile:
I need to confirm this with a minimal application and different version later next week...


node 20

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: lasith-kg/dispatch-workflow@692cec720c2d07367b512c5f8668e33c3d8f2f13. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

I assume it is https://github.com/lasith-kg/dispatch-workflow/blob/692cec720c2d07367b512c5f8668e33c3d8f2f13/action.yml#L51
since the deps include node20 for the build

No worries. Let's get this supporting Node v20

Hey @davhdavh , I'm having issues with the library I use to release the GitHub Actions release-it. I'll investigate this further over the weekend

No worries, I'm guessing you have a couple of weeks/months more before they actually remove support.

I'm closing this issue. I have tried the action in some personal workflows and did not run into any issues. Thank you @davhdavh! Feel free to give this repository a ⭐ if you had a good experience! It really helps me out


medak error

couldn't run medaka (load the model) when trying to run 12 samples in parallel. Error not happening when running one sample at the time.

now run samples in loop instead of parallel processing.


How to handle the anonymous field on auth plugins

If I want to configure the anonymous field on a jwt plugin, is there a way to handle this in deck?
Its tricky because this field requires the consumer id, which we cannot know until the consumer has been created already.
It would be useful if we could set this field using 'username' in deck, and to have the ability to list some users in the config so that deck will ensure they exist, even when the --skip-consumers flag is used (or maybe have another flag to enable this).
Update:
I see that using the kong admin api I can create a consumer and specify the id so i guess that part should be possible to get working in deck
As for the issue of synching only certain consumers, I have 2 suggestions.

include a parameter to only create consumers and ignore updates or deletions --create-consumers
alternatively, a flag to sync consumers with a particular tag(s). For example, --sync-consumer-tags=anonymous will perform synchronize actions on users with the anonymous tag and ignore others.

As for the issue of synching only certain consumers, I have 2 suggestions.

This is a very common issue that comes up far too often and decK has concept of select-tags.
You can configure decK to perform a sync/diff only for a sub-set of entire Kong configuration.
Please take a look at: https://github.com/hbagdi/deck/blob/master/kong.yaml#L2
You can then manage only a subset of consumers of plugins or any entity you would like.
With this, you can have a state file to manage a single anonymous consumer.
And then the other state file can manage rest of the configuration.

Perfect, select-tags does the trick.
For the anonymous user id, what do you think about decK setting the consumer id directly?
An alternative approach could be to have a new tag that acts like a function to fetch a consumerid based on the consumer username:
_info:
  select_tags:
  - managed_by_deck

services:

  - name: service1
    host: httpbin.org
    path: /
    protocol: http
    port: 80
    connect_timeout: 60000
    read_timeout: 60000
    write_timeout: 60000
    retries: 5
    tags:
    - managed_by_deck
    routes:
    - name: service1_route1
      paths:
      - /
      preserve-host: false
      methods:
      - GET
      protocols:
      - http
      - https
      regex_priority: 0
      strip_path: true
      tags:
      - managed_by_deck
      plugins:
      - name: key-auth
        tags:
        - managed_by_deck
        config:
          anonymous: _consumer_id(anonymous)

consumers:
- username: anonymous
  custom_id: anonymous
  tags:
  - managed_by_deck

Good point about the templating, it could get messy.
Regarding the plugin, it seems strange to me that we need to specify an anonymous consumer at all. When first looking into this functionality I expected the anonymous field to be a boolean, or at least have the option to allow traffic for unauthenticated users.
#29 seems like a good solution if modifying the plugin isn't feasible

https://github.com/Kong/kong/pull/4972 will fix this problem.

The above feature will be included in Kong 1.4 and solve this problem.
I've also started working on #29 as well.
Closing this.

@hbagdi I'm running on version 2.0.0 and can still reproduce this issue:
	while processing event: {Update} failed: 400 Bad Request {"message":"schema violation (config.anonymous: expected a valid UUID)","name":"schema violation","fields":{"config":{"anonymous":"expected a valid UUID"}},"code":2}
	while processing event: {Update} failed: 400 Bad Request {"message":"schema violation (config.anonymous: expected a valid UUID)","name":"schema violation","fields":{"config":{"anonymous":"expected a valid UUID"}},"code":2}

Any suggestions on how to patch this? Will you port your fix also to the 2.x branches?

No. Please upgrade.

I believe I run the last docker version available:
kong                     2.0                 13f14d790da4        12 days ago         144MB
kong                     latest              13f14d790da4        12 days ago         144MB

Please upgrade decK, Kong should be fine.

I run decK from a the official Docker Hub latest image: https://hub.docker.com/r/hbagdi/deck, digest: 9fe912c36ca143dbd110b4f38610e492b987ebfeef6c0df9fa299a4a05f2ba81.
Is the container far behind?

No, that's the latest one. Something got messed up in Kong. I'll investigate.

@hbagdi should I create a ticket at https://github.com/Kong/kong?

Yes, please.

I recreated this problem without DecK and created an issue in the Kong repo. https://github.com/Kong/kong/issues/5551

Thanks @3to1null


require/import packages

Where should I install packages to be accessible by my nodejs kernel ? I don't see any node_modules folder.

@Simobain Did you figure it out?

there's not a super good solution to this atm :/ the kernel's cwd ends up being wherever you started your jupyter process from, which isn't the best. I'd probably like for there to just be a global ~/.jupyter-nodejs folder or something where modules are installed.
Do you have any thoughts?

I agree it would be better to have a "special" node_modules repo for the node js kernel.
And yes i figured out, sorry for the long time without seeing my notification and then your answers ;) !


[add] created by pipenv

Pull Request
Thank you for contributing to @dvcs/gitignore and https://www.gitignore.io.
New or update
Select the appropriate check box for this pull request. This helps when merging to ensure there are no conflicts with other templates or misunderstandings of how thee template list works.
New

[ ] Template - New .gitignore template
[ ] Composition - Template made from smaller templates
[ ] Inheritance - Template similar to an existing template
[ ] Patch - Template extending functionality of existing template

Update

[x] Template - Update existing .gitignore template

Details
Use pipenv and enable PIPENV_VENV_IN_PROJECT option.
The dir name is fixed.
e.g
https://github.com/pypa/pipenv/blob/master/docs/advanced.rst#-configuration-with-environment-variables

Due to change #116 this fails the build because it's checking your change against GitHub's master list.  Can you create a Python.patch file and add this line to that file.  That will allow the build to pass.

OK!
Plese wait a minute until the corresponding.

github.com/github/gitignore is the main list of these templates and the build system checks to make sure that all of our local templates are kept up to date with the changes in that repo.  What that means is that if you want to change one of the templates that is maintained by GitHub without waiting for them to accept your pull request, you need to create a .patch file.  This patch file will apply any additional changes you want.
All you have to do is.

Make a new file called Python.patch
Remove the line you made in Python.gitignore and add it to Python.patch
Re-submit this pull request

The build will be green and I'll merge it as soon as I see it.

Thank you for the advice.
I fiexd now.
Can you check this out?

Hmm...
Python.patch  contents
.venv/ only oneline?
I'm sorry to be so inquisitive.

I see
I am sorry for taking your time.
I hope this in successful!

No problem.  This setup is probably more complicated than it needs to be :)


Custom error messages for helpers?

Thanks for the library!
Is there a reason the helpers (like CheckboxAsString) are not functions I can call to pass extra config (like an error message)? How would I specify my own error message?

There isn't a good reason and I've been meaning to fix this for a while. They also don't work with .transform().
A PR would be much appreciated and should be straightforward.

If I end up pursuing this then I'll be happy to. Thanks!


Sync problem when user elects to "do nothing" when clash occurs

A clash is defined as being when we have a note later than last sync AND there is already one on the server with a different last change date.
However, that sync run will update the client's last sync date, making it more recent than the note in queston's last sync date. So, next sync won't see the clash !!
I though "do nothing" was a valuable addition to possible user responses, Tomboy does no allow such an action, but maybe, now, I see why Tomboy did what it did !

Thinking out loud.
The Tomboy spec has both last-change-date and last-metadata-change date. I have been keeping these in sync and have no indication what the difference is. But, maybe, this is what was intended -
last-change-date records the last time content in the note was changed.
last-metadata-change-date records last time note was played with, eg, by sync engine.
At the moment, when I bring a note back from Backup, I update its last-change-date so it has priority at sync time. A better approach may be to have sync look at the metadata date, leaving the other one for the user to view.
Now, if sync is looking at only metadata change, if user decides to select donothing during a clash, I can update its metadata time to something later than the current sync date and we can be sure that next sync run will again see it as a clash. But user will still see lastchange date relating to content ?
Gee, I wish there was some documentation left around from when the original design work was done on the Tomboy XML spec or sync model !

Nope. metadata value is not the answer. A careful examination of how I believe the Tomboy sync model works has convinced me that 'doNothing' about a sync clash is not possible in every possible combination.
So, I have removed that option and release 0.18 will reflect that change.
Users can choose to download or upload. If they try and avoid the question by closing the dialog, we'll assume download and inform the user that they can recover the local note from Backup if required.


Bitmap Converter

Hello. I use hair segmentation. I want to buy a mask (black & white) for hair coloring and to make changes. I looked at other issues and couldn't find the answer.
OR I want to see the image I see in Surface View as a bitmap
processor.addPacketCallback("output_video"){ packet->

        }

How can I convert the data I listen as a packet to a bitmap?

Hi @farukkyldzx ,
Hair segmentation can be done in hair_segmentation_mobile_gpu.pbtxt calculator usinfg below code snippet.
https://github.com/google/mediapipe/blob/master/mediapipe/graphs/hair_segmentation/hair_segmentation_mobile_gpu.pbtxt#:~:text=node {-,calculator%3A "RecolorCalculator",},-}


Allow query params to be passed through Model.fetch

This is a bit similar to issue #78
It would be great to have a way to pass url params in the fetch method. Many APIs allow you to pass in extra params to get more data. For instance, twitter lets you pass in the include_entities param to get more metadata about hashtags and user mentions.
http://api.twitter.com/version/users/show.json?include_entities=true

Ideally I could add url params to the options object passed into the fetch method
var User = Backbone.Model.extend({
  url:'http://api.twitter.com/version/users/show.json',
});

twitter_user = new User;
twitter_user.fetch({
  data:{
    user_id:'12345',
    include_entities:true
  },
  success:function(){ //do something }
})

instead of overwriting the url property and stringing up a bunch of query params

somehow, i get the same problem, i just overwrite the fetch method like this:
	fetch: function( options, queryObj ){
		var _url = this.url;

		if( queryObj ){
			this.url = this.url + '?' + $.param( queryObj );
		}
		this.fetch( options );

		this.url = _url;
	}

than you can just use fetch with you query params:
    myCollections.fetch( { add: true}, { query: 'query'})


[GOBBLIN-10] Fix_for_#1850_and_#1851

Fix for #1850 and #1851

@abti Can you get someone to look at this?

@kadaan will prioritize

Issue: https://issues.apache.org/jira/browse/GOBBLIN-10
Please update your PR title with following prefix: [GOBBLIN-10]

@kadaan Seems like findbugs is failing for this: https://travis-ci.org/apache/incubator-gobblin/builds/245976111?utm_source=github_status&utm_medium=notification

It looks good to me, so let me know when you update the diff with findbugs resolved and I will merge it in.

@abti Fixed the findbugs issue.

Thanks


Scintilla editor cannot handle DF23 multi-line strings or comments

The DF 23 introduced multi-line strings and multi-line comments is yet to be understood by the Scintilla editor used in the DFRefactor & TestBench programs.

This is on Wil van Antwerpen's ToDo list. He is responsible for the Scintilla stuff also used by The Hammer, but can't give a time estimate for when it can be done.

Fixed! New SciLexer32.dll and SciLexer64.dll have been pushed to the repository.


Add a note "Directory is also know as repo"

line 36

[ x] I have read freeCodeCamp's contribution guidelines.
[x ] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[x ] None of my changes are plagiarized from another source without proper attribution.
[x ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Closed and Reopened this PR to attempt to resolve a specific Travis build failure.


odo app delete removes application components but not services

[kind/bug]
What versions of software are you using?

Operating System: Windows
Output of odo version:

> odo version
odo v1.0.0-beta5 (4d1d5071)

How did you run odo exactly?
> odo.exe create nodejs:latest nodejs1 --git https://github.com/dgolovin/nodejs-ex --ref HEAD --app app --project myproject

> odo.exe push

> odo.exe service create postgresql-persistent --plan default psql --app app --project myproject -w

> odo list
APP     NAME        TYPE       SOURCE                                    STATE
app     nodejs1     nodejs     https://github.com/dgolovin/nodejs-ex     Pushed

> odo service list
NAME     TYPE                      STATUS
psql     postgresql-persistent     ProvisionedAndBound

> odo app delete app -f
This application has following components that will be deleted
component named nodejs1
This application has following service that will be deleted
service named psql of type postgresql-persistent
Deleted application: app from project: myproject

> odo list
 X  There are no components deployed.

> odo service list
NAME     TYPE                      STATUS
psql     postgresql-persistent     ProvisionedAndBound

Actual behavior
Services have not been deprovisioned
Expected behavior
Services have been deprovisioned
Any logs, error output, etc?
N/A

/kind bug
/priority high

This should get resolved once https://github.com/openshift/odo/issues/1793 is implemented.

This should get resolved once #1793 is implemented.

I don't think so. The PR #2125 opened to address #1793 makes DC the owner of Service, Replication Controller, PVC and Secret. But this is not the service (or Provisioned Service as shown on web console) created from Service Catalog using odo service create command. It's the Service started by the DC that we bring up as part of deploying component.
Besides, we had a series of PRs opened to make sure that Provisioned Services don't get bound to a context (.odo/config.yaml). We are now at a stage where odo service {create,delete,list} happens without requirng the command to be executed from a valid component directory.
Do we now want to attach a Provisioned Service to the life-cycle of an app? I understand that the context/component directory and app are different things.
If we delete the service(s) when the app is deleted it causes the future components being created for that app to not be able to attach to the service. User needs to bring up the service again if they want to attach a component to it.

PROs:

if it's a database service, for example, the component won't have to work with old schema and/or data from the leftover service.

CONs:

Time it takes to bring up the service afresh.

Does that fit in with the opinionated approach of the tool?
@kadel @dgolovin @girishramnani

consider --all as an extra flag to remove all the services as well

I think that the important question here is whatever the service is part of the application or not.
If odo is working with the model where services are part of the application, then it should be deleted with it.
If we say that service is not part of the app but stand above it, then we shouldn't label it with application labels and do other changes in odo to reflect this.
Odo currently assumes that service is part of the application and requires that every service is associated with an app.
To make odo behave consistently the odo application delete  shouldn't also remove the services that are in the application. The message even warns the user that services will be deleted.

@kadel TBH, I missed that the odo app delete explicitly warns the user that the service will be removed. Besides that, I agree with your point that at the moment services are tightly bound with the application in that, a service can't be created without project and app flags.
In this scenario, it makes sense to me to delete the service when application is deleted. I guess we could trying setting the app/application as referenceOwner for the service so that kubernetes/OpenShift automatically takes care of this.

In this scenario, it makes sense to me to delete the service when application is deleted. I guess we could trying setting the app/application as referenceOwner for the service so that kubernetes/OpenShift automatically takes care of this.

+1

The changes I've made to fix this issue of deleting a service deployed from Service Catalog when user deletes the app (odo app delete app -f) can be found at https://github.com/openshift/odo/compare/master...dharmit:fix-2113. However, something's off with the permissions and it doesn't let me delete the service binding and service instance. I'm not able to figure out what's wrong. PTAL at the output below.
~ $ odo service create dh-postgresql-apb psql --plan dev -p postgresql_database=mydb -p postgresql_password=password -p postgresql_user=luke -p postgresql_version=10
Deploying service psql of type: dh-postgresql-apb
 ✓  Deploying service [705ms]
 ✓  Service 'psql' was created

Progress of the provisioning will not be reported and might take a long time
You can see the current status by executing 'odo service list'
Optionally, link dh-postgresql-apb to your component by running: 'odo link <component-name>'

~ $ odo service list --app app
NAME     TYPE                  PLAN     STATUS
psql     dh-postgresql-apb     dev      Provisioning
~ $ odo app list
The project 'myproject' has the following applications:
NAME
app

~ $ odo app delete app -f
This application has following service(s) that will be deleted
service named psql of type dh-postgresql-apb
servicebindings.servicecatalog.k8s.io is forbidden: User "developer" cannot deletecollection servicebindings.servicecatalog.k8s.io in the namespace "myproject": no RBAC policy matched
serviceinstances.servicecatalog.k8s.io is forbidden: User "developer" cannot deletecollection serviceinstances.servicecatalog.k8s.io in the namespace "myproject": no RBAC policy matched
 ✗  unable to delete servicebinding,unable to delete serviceinstancervice(s) that will be deleted
service named psql of type dh-postgresql-apb
servicebindings.servicecatalog.k8s.io is forbidden: User "developer" cannot deletecollection servicebindings.servicecatalog.k8s.io in the namespace "myproject": no RBAC policy matched
serviceinstances.servicecatalog.k8s.io is forbidden: User "developer" cannot deletecollection serviceinstances.servicecatalog.k8s.io in the namespace "myproject": no RBAC policy matched
 ✗  unable to delete servicebinding,unable to delete serviceinstance

As far as the changes I've made, it's mainly the occlient.go file that contains the changes that matter. I've mainly done two things:

Added labels to the ServiceBinding that's created as a part of odo service create code
Tried to delete the ServiceBinding and then the ServiceInstance in the same function func (c *Client) Delete(labels map[string]string) error that deletes other resources belonging to the app when we do odo app delete app -f.

Can someone take a look please @kadel @girishramnani @mik-dass @kanchwala-yusuf

Can you try to delete the service using web console or using oc? If that happens successfully something is wrong with our code. We can then further investigate

Yes it works through oc cli and web console. Errors only when using odo.

@girishramnani I figured out the issue. Opened a PR. :man_dancing:


Add link to published paper.

Suggest you also retain the link to the pre-print.

I agree, thanks.


feat(ci): Add a new metric to compute duration of required jobs

What does this PR do?
Add a metric computing required jobs duration

Motivation
It's not easy to compute this duration directly on ci-visibility (as jobs have dependency we must take into account). It will give us a precise information of the minimum CI requested time duration until further updates on the pipeline constraints

Additional Notes
Sorry for the formatting, shipped with the ruff update..

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


Translated French headers in English version into English

Translated French headers in English version into English.  "Comment ça marche?" to "How does it work?", and "Avec qui on travaille?" to "Who do we work with?".

Closes #33


Nhost doesn't sync the metadata from the local database with the remote database for the hasura native queries.

Context for the issue
I have created a few Hasura native queries locally, and local nhost creates the metadata in the nhost/metadata/databases/databases.yaml file. However, Nhost does not create these native queries remotely, even if the deployment is successful. Hasura has announced that native query editing is not supported yet and will be supported in a future version. I thought this might be the reason why Nhost is not creating the native queries. Therefore, I created the native queries manually on the remote. However, Nhost deletes the native queries that are already created whenever a new deployment is done.

@dbarrosop I confirmed that the issue has been gone. Thank you

Thanks for confirming


Issues with running thoughtbot:laptop setup / provisioning laptop

Error: undefined local variable or method `shell_profile' for #Formulary::FormulaNamespacecc13dfc76e452057b82dd3b49090ab6c::Perl:0x007fc21daa4bf0
Updating Homebrew formulae ...
Already up-to-date.
Succeeded in tapping thoughtbot/formulae
Succeeded in tapping homebrew/services
Error: undefined local variable or method shell_profile' for #<Formulary::FormulaNamespacecc13dfc76e452057b82dd3b49090ab6c::Perl:0x007fc21daa4bf0> Please report this bug:     https://github.com/Homebrew/homebrew-bundle/issues/ /usr/local/Library/Taps/homebrew/homebrew-core/Formula/perl.rb:71:in caveats'
/usr/local/Library/Homebrew/formula.rb:1352:in to_hash' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:69:in block in formulae_info'
/usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:69:in map' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:69:in formulae_info'
/usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:14:in formulae' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_dumper.rb:34:in formula_names'
/usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:77:in installed_formulae' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:69:in formula_installed?'
/usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:93:in installed?' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:27:in install_change_state!'
/usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:22:in run' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/brew_installer.rb:10:in install'
/usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/dsl.rb:54:in block in install' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/dsl.rb:36:in each'
/usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/dsl.rb:36:in install' /usr/local/Library/Taps/homebrew/homebrew-bundle/lib/bundle/commands/install.rb:4:in run'
/usr/local/Library/Taps/homebrew/homebrew-bundle/cmd/brew-bundle.rb:60:in <top (required)>' /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require'
/System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/lib/ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in require' /usr/local/Library/Homebrew/brew.rb:23:in require?'
/usr/local/Library/Homebrew/brew.rb:97:in `'
failed

For reference here's the thoughtbot script
https://github.com/thoughtbot/laptop

Should be fixed now, sorry for the pain.


[Request] jump to chapter option

It would come handy in case of novels that are not stacked in volumes and contains large number of chapters

It would look something like that

Do you want this in the reader or when downloading, I am a bit confused about the wording.

Also what app are you using in the second pic, tachi but for ln?

Also what app are you using in the second pic, tachi but for ln?
Here,
https://github.com/LNReader/lnreader

Do you want this in the reader or when downloading, I am a bit confused about the wording.

Both , as a tool to read chapters from in-between when reading/exploring or to download only selected chapters from in-between


feat: Change HasCollisionDetection to be on Component

Description
This change makes it possible to have the collision detection system further down in the tree than on the FlameGame, this enables you to have collision detection on the World component for example.
Today it doesn't work if you have several worlds where the components are overlapping across the worlds, since the hitboxes live on top level.
So now you can use a World like this:
class CollisionDetectionWorld extends World with HasCollisionDetection {}

and all hitboxes added in there will only react with other hitboxes added to that world.
Checklist

[x] I have followed the Contributor Guide when preparing my PR.
[x] I have updated/added tests for ALL new/updated/fixed functionality.
[ ] I have updated/added relevant documentation in docs and added dartdoc comments with ///.
[ ] I have updated/added relevant examples in examples or docs.

Breaking Change?

[ ] Yes, this PR is a breaking change.
[x] No, this PR is not a breaking change.

Related Issues

shipit


Multi-cursor "lorem ipsum" generation is broken

Hi.
On Visual Studio Code, when using multiple cursors and launching the "Lorem Ipsum" generators, i get one prompt per cursor (expected), but only the first value is picked.
Example:
With 4 cursors, i call the generator and insert the values: [2, 3, 4, 5].
The result is that only 2 words are generated, and only in the last cursor position.
I hope my explanation is clear enough, english is not my first language

Hi,
Thanks for reporting it.
Quite clear. This issue was affecting all scripts generating data.
Thank you for reporting it. I just released 1.6.2 with the fix.

Awesome, thank you for the quick reply and quick fix!
Il giorno lun 30 mag 2022 alle ore 18:30 Luis Fontes <
@.***> ha scritto:

Hi,
Thanks for reporting it.
Quite clear. This issue was affecting all scripts generating data.
Thank you for reporting it. I just released 1.6.2 with the fix.
—
Reply to this email directly, view it on GitHub
https://github.com/luisfontes19/vscode-swissknife/issues/7#issuecomment-1141332011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ATYM5O7E3AFQNVWXABDZNRLVMTUK5ANCNFSM5XLBZVPA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Matteo De Carli
Email: @.***
Novamind SAS - Soluzioni informatiche, Siti web, Web Marketing
Via Vittorio Veneto 3, 37063 - Isola della Scala - Verona
Web: www.novamind.it

@matteo-de-carli, sorry edited your comment to hide some personal information from your email reply :)

@matteo-de-carli sorry deleted your comment to hide some personal information from your email reply


Fix SystemMemoryVersion variable missing

Fixes #2950

It was giving me errors in VS, I hate that :)

/azp run


Generic tearoff not working

We're not getting the right type on math.max below:
import 'dart:math' as math;
main() {
  takeFn(math.max/*<int>*/);
}
void takeFn(int fn(int a, int b)) {}

We're getting the following errors:
[error] Type check failed: math.max (<T extends num>(T, T) → T) is not of type (int, int) → int (/Users/vsm/git/dev_compiler/foo.dart, line 7, col 10)
[warning] The argument type '(T, T) → T' cannot be assigned to the parameter type '(int, int) → int' (/Users/vsm/git/dev_compiler/foo.dart, line 7, col 10)
1 error and 1 warning found.

@jmesserly Are we missing logic on tearoffs?

yeah this is basically the unimplemented case in https://github.com/dart-lang/sdk/issues/25175 ... we chatted about it, might be worth adding the downward inference here even though we don't support explicit instantiation yet.
The workaround would be: (x, y) => max(x, y) then inference should figure it out.

https://codereview.chromium.org/1644403005


LC3_EXPORT macro breaks compilation with MSVC

Hi,
This is to report that on #44 a LC3_EXPORT macro was added to aid in managing the ABI of liblc3. However, using this macro prevents compiling the library at all on MSVC, it needs to be changed to __declspec(dllexport) in that case.
Fixing this issue will also enable building shared libraries without further change under MSVC -- previous to that PR, the shared library had an empty ABI and so had no import library generated at link time.

Hi,
Thanks for reporting this.
I am unable to test with MSVC, sorry for that. Can you check that https://github.com/google/liblc3/pull/49 works?
Thanks.


fix: faulty dependency updates in changelog

need to commit earlier so the report is committed before jx changelog is run

/cc ankitm123 tomhobson babadofar


Pokestop constantly out of range.

Expected Behavior
Spin Pokestops.
Actual Behavior
Pokestops are out of range, doesn't spin.
Your config.json (remove your credentials and any other private info)
   "tasks": [
      {
        "type": "HandleSoftBan"
      },
      {
        "type": "SleepSchedule",
        "config": {
          "enabled": false,
          "time": "22:54",
          "duration":"7:46",
          "time_random_offset": "00:24",
          "duration_random_offset": "00:43"
        }
      },
      {
        "type": "CollectLevelUpReward"
      },
      {
        "type": "IncubateEggs",
        "config": {
          "longer_eggs_first": true
        }
      },
      {
        "type": "UpdateLiveStats",
        "config": {
          "enabled": true,
          "min_interval": 10,
          "stats": ["uptime", "stardust_earned", "xp_earned", "xp_per_hour", "stops_visited"],
          "terminal_log": true,
          "terminal_title": true
        }
      },
      {
        "type": "TransferPokemon"
      },
      {
        "type": "NicknamePokemon",
        "config": {
          "enabled": false,
          "nickname_template": "{iv_pct}_{iv_ads}"
        }
      },
      {
        "type": "EvolvePokemon",
        "config": {
        "evolve_all": "Pidgey,Rattata,Zubat,Weedle,Drowzee,Spearow,Venonat,Caterpie,Krabby,Paras,Jigglypuff,Sandshrew,Vulpix,Meowth,Psyduck,Mankey,Tentacool,Ponyta,Slowpoke,Magnemite,Doduo,Seel,Voltorb,Horsea,Goldeen,Staryu,Ekans",
        "first_evolve_by": "cp",
        "evolve_above_cp": 0,
        "evolve_above_iv": 0,
        "logic": "or",
        "evolve_speed": 1,
        "use_lucky_egg": true
        }
      },
      {
        "type": "RecycleItems",
        "config": {
          "min_empty_space": 15,
           "item_filter": {
          "Pokeball": {
            "keep": 20
          },
          "Greatball": {
            "keep": 30
          },
          "Ultraball": {
            "keep": 60
          },
          "Masterball": {
            "keep": 80
          },
          "Potion": {
            "keep": 0
          },
          "Super Potion": {
            "keep": 0
          },
          "Hyper Potion": {
            "keep": 0
          },
          "Max Potion": {
            "keep": 80
          },
          "Revive": {
            "keep": 0
          },
          "Max Revive": {
            "keep": 30
          },
          "Razz Berry": {
            "keep": 30
          }
          }
        }
      },
      {
        "type": "CatchLuredPokemon"
      },
      {
        "type": "SpinFort"
      },
      {
        "type": "MoveToFort",
        "config": {
            "lure_attraction": true,
            "lure_max_distance": 2000
        }
      }
    ],
    "map_object_cache_time": 5,
    "forts": {
      "avoid_circles": true,
      "max_circle_size": 500
    },
    "websocket_server": false,
    "walk": 50,
    "action_wait_min": 1,
    "action_wait_max": 4,
    "debug": false,
    "test": false,
    "health_record": true,
    "location_cache": true,
    "distance_unit": "m",
    "reconnecting_timeout": 15,
    "logging_color": true,
    "catch": {
      "any": {"catch_above_cp": 0, "catch_above_iv": 0, "logic": "or"},
      "// Example of always catching Rattata:": {},
      "// Rattata": { "always_catch" : true }
    },
    "catch_throw_parameters": {
      "excellent_rate": 1,
      "great_rate": 0,
      "nice_rate": 0,
      "normal_rate": 0,
      "spin_success_rate" : 1
    },
     "release": {
    "any": {
      "keep_best_iv": 1
    },
    "Moltres": {
      "keep_best_iv": 3
    },
    "Zapdos": {
      "keep_best_iv": 3
    },
    "Articuno": {
      "keep_best_iv": 3
    },
    "Mewtwo": {
      "keep_best_iv": 3
    },
    "Dragonite": {
      "keep_best_iv": 3
    },
    "Dratini": {
      "keep_best_iv": 3
    },
    "Dragonair": {
      "keep_best_iv": 3
    },
    "Charizard": {
      "keep_best_iv": 3
    },
    "Blastoise": {
      "keep_best_iv": 3
    },
    "Venusaur": {
      "keep_best_iv": 3
    },
    "Nidoking": {
      "keep_best_iv": 3
    },
    "Gengar": {
      "keep_best_iv": 3
    },
    "Jolteon": {
      "keep_best_iv": 3
    },
    "Scyther": {
      "keep_best_iv": 3
    },
    "Snorlax": {
      "keep_best_iv": 3
    }
  },
  "vips": {
    "any": {
      "catch_above_cp": 1200,
      "catch_above_iv": 0.9,
      "logic": "or"
    },
    "Lapras": {},
    "Moltres": {},
    "Zapdos": {},
    "Articuno": {},
    "Mewtwo": {},
    "Dragonite": {},
    "Snorlax": {},
    "Mew": {},
    "Arcanine": {},
    "Vaporeon": {},
    "Gyarados": {},
    "Exeggutor": {},
    "Muk": {},
    "Weezing": {},
    "Flareon": {}
  }
}
your config here

Steps to Reproduce
Other Information
OS:
Git Commit: 50cd7bf243eab90b88a2d8bd99e105b39647f331
Python Version: Python 2.7.12
Tried cleaning the logs, changing location, walk speed, nothing has helped.

same issue. Did you solve it?

No, not yet :(

Same issue here constantly out of range

Same issue. Possible kind of softban. App says "Try again later".

same

#3308 #3394 #3475 #3646 #3692 #3711 #3721

I suppose Niantic detect bot by checking
if this IP address travels N km, ban

Nope. Another bot works good. Both have same IP. BTW even if it can't spin
forts, it is still able to catch pokemon.
12 авг. 2016 г. 6:30 PM пользователь "g2384"<EMAIL_ADDRESS>написал:

I suppose Niantic detect bot by checking
if this IP address travels N km, ban
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PokemonGoF/PokemonGo-Bot/issues/3711#issuecomment-239423678,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATxdtb-uqs-vbXU-PqG0PPP3qE-QV-_6ks5qfFljgaJpZM4Ji6To
.

Same here. But the weird thing is that I can still catch pokemons. Doesn't look like a softban.
2016-08-13 04:14:14,126 [  SpinFort] [INFO] [pokestop_out_of_range] Pokestop Wolfe Island Ferry Terminal out of range.
2016-08-13 04:14:14,126 [  SpinFort] [INFO] [pokestop_searching_too_often] Possibly searching too often, take a rest.

Any solution?

You might just got banned.

@youweit what kind of ban?

Please google a bit about "new" softban.
14 авг. 2016 г. 11:20 пользователь "icecubeice"<EMAIL_ADDRESS>написал:

@youweit https://github.com/youweit what kind of ban?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/PokemonGoF/PokemonGo-Bot/issues/3711#issuecomment-239661780,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATxdtW2WxiDL5tsj8kQBAzYM1d7KbJglks5qfs-vgaJpZM4Ji6To
.


RedditFeed missing tons of posts

Like, it only notices maybe one out of 4 or something.

Also it likes to time out which right now crashes SekshiBot entirely :stuck_out_tongue_winking_eye:


Simplify login screen

top "start here" button fires social logins method
bottom metamask button fires web3 method

I think we can close this because it has been implemented in SimpeDApp.
@sarora180673
@RAIReth


Always use initializer lists in constructors.

This has been resolved thanks to KubaO's latest PR.


Random cover art

last.fm cover is empty
Environment

Version used: 2.1.0
Browser name and version: FF 57.0b9 (64-bit)

The extension downloads cover arts from CoverArtArchive, if Last.fm returns nothing. But CoverArtArchive sometimes returns wrong cover arts.


PackageConfig class not accessible when using gradle kotlin-script

The classes that the BintrayExtension contains should be publicly accessible. Otherwise they are not usable from kotlin.
Here is a workaround that requires the usage of the groovy/kotlin interop like here https://github.com/gradle/kotlin-dsl/tree/master/samples/groovy-interop
Create a groovy file named groovy.gradle in your project:
ext.bintrayPackage = { ->
    repo = "myRepoName"
    name = "myProjectName"
    userOrg = "myOrgName"
}

and reference it from your build.kts:
apply {
	from("groovy.gradle")
}

val bintrayPackage: groovy.lang.Closure<Any> by extra

bintray {
	user = ...
	key = ...
	pkg(bintrayPackage)
}

this works for me:
pkg(closureOf<BintrayExtension.PackageConfig>{
...
}

@teyckmans suggestion works in the current version 1.8.0.


Add support for  VEDV (https://github.com/yunielrc/vedv)

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

If you are referring to connecting to the container, then this may be better as a plugin once those are able to create custom file backends (or your use case might be served enough by plugins able to add palette commands + being able to trigger an ssh after the container starts, unsure after a skim.)


Add Information about total number of shares to shares_total_metric

Hey,
I wanted to ask you if it is possible to add this information to the respective metric, it's useful for us, for showing the shares as a pie chart in a Grafana Dashboard for better usage analysis.
Kind Regards

Hi, thanks for the question. Unfortunately it would violate the best-practices used in the Prometheus ecosystem to add this label value, because all values of a label should be aggregatable in a useful way. This specific best-practice is mentioned here.
If you need a total for graphing or alerting either aggregate it on-the-fly with sum() in the query (grouping necessarily using by()) or in case the dataset is too large or you want to simplify graphing by using a recording rule with a similar sum function.
Do not hesitate to ask if you need a more concrete example.

No problem, I totally understand this, thanks for this information, we managed this by agrgregating the metrics :)


Should dispatch return storeState?

Returning just the updated value changes in dispatch doesn't seem right. It is possible for multiple values in the state to have been affected by the action. Does anyone else have any thoughts on this?
And thank you @luisvinicius167 for creating this. It's a pretty good start!

Same opinion, looking at the test case it doesn't seem right.
// Receive new value
newCount := store.Dispatch(increment(10)) // 1+10=11
subCount := store.Dispatch(decrement(10)) // 1-10=-9
fmt.Printf("Your Store state is: %d. Your newCount is: %d. Your subCount is: %d\n", store.GetState("count"), newCount, subCount)	// output: Your Store state is: 1. Your newCount is: 11. Your subCount is: -9
It doesn't change store state at all otherwise the expected results would have been 11 and 11-10=1


[Kaltura] Add support for an uncommon typo in a GDC Vault page

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] Searched the bugtracker for similar pull requests
[x] Read adding new extractor tutorial
[x] Read youtube-dl coding conventions and adjusted the code to meet them
[x] Covered the code with tests (note that PRs without tests will be REJECTED)
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
A few pages on the GDC Vault (https://www.gdcvault.com/play/1025699, https://www.gdcvault.com/play/1025739, possibly more) have a slightly-misformed Kaltura iframe link, where the URL has an unnecessary leading space. The Kaltura extractor isn't durable enough to ignore that so it fails to download the file. Web browsers seem happy to skip it, however.
This allows it to ignore that initial leading space when it exists.
This is a repost of https://github.com/ytdl-org/youtube-dl/pull/21079 which got kinda nuked in the whole DMCA thing.

Does not seem to have any effect:
> py -3.7 .\youtube_dl\__main__.py https://www.gdcvault.com/play/1025699 -v
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.gdcvault.com/play/1025699', '-v']
[debug] Encodings: locale cp1251, fs utf-8, out utf-8, pref cp1251
[debug] youtube-dl version 2021.04.26
[debug] Git HEAD: 61d7f7d
[debug] Python version 3.7.0 (CPython) - Windows-10-10.0.10240-SP0
[debug] exe versions: ffmpeg N-85653-gb4330a0, ffprobe N-85653-gb4330a0, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[GDCVault] 1025699: Downloading webpage
WARNING: [GDCVault] It looks like http://www.gdcvault.com/play/1025699 requires a login. Try specifying a username and password and try again.
WARNING: [GDCVault] Could not login.
ERROR: Unable to extract xml filename; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

You'll need a GDC Vault login to test it, unfortunately; it's joining a similar group because very few of the tests in there still work.

This has nothing to do with GDC Vault login. The video is freely available.


Add option to remove model costs from the output

I'd like to add the checkbox (turned on by default) on whether to display unit costs or not (pts, PL).

Closed by #44


GEODE-7278: convert transport filters

Added unit test for GatewayReceiverConverter.
GatewayReceiverConverter now handles the transport filters.
Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, check Concourse for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

@jmelchio


Implement ignoreAnnotated suppressor like detekt

There are a lot of false positive findings regarding Jetpack Compose. Detekt handles this gently with a ignoreAnnotated suppressor. It's a List of Strings. Code that is annotated with an annotation of this list is ignored.
This would come really handy for following rules:
TOO_MANY_PARAMETERS
LAMBDA_IS_NOT_LAST_PARAMETER
TOO_LONG_FUNCTION
FUNCTION_NAME_INCORRECT_CASE
VARIABLE_NAME_INCORRECT_FORMAT

@WebTiger89 @Supress annotation is not enough for this scenario?

I guess no. This feature is really handy and it might be required for my last suggestion (https://github.com/saveourtool/diktat/issues/1322)
I can imagine that this requires a lot of work but it would improve every rule not only the ones mentioned above.
Just have a look into:
https://github.com/detekt/detekt/issues/4682

To fix this I use, but it looks terrible and is bad practice @SuppressWarnings("UnusedPrivateMember")

https://detekt.dev/docs/introduction/suppressors/#annotation-suppressor
https://github.com/detekt/detekt/pull/4102

I got it, yeah. Actually it is a quick fix for us

We made even more: except the suppression of blocks that are marked with specific annotations, we also aded @Suppress("diktat") that will remove all inspections completely


Python bindings

I've set up the infrastructure to build Python bindings for eos using pybind11 in the python-bindings branch.
There are no real bindings yet, just a simple example with PcaModel. There's a few things that need to be worked out, for example, currently, calling PcaModel::get_mean_at_point in Python causes Python to crash, most likely because the function returns a cv::Vec4f, which is unknown to Python. I want to test if that's solved if OpenCV-python is imported, or if it doesn't work together like that. Otherwise I could try making bindings for the glm vector types.
Any help whatsoever welcome on this.
Once there are actual bindings in the future, it would probably be nice to build them automatically in CMake, so the user doesn't have to run create-python-bindings manually.

I've created a GitHub Project for this!

I found out that it won't work together with OpenCV-python, since they don't expose cv::Mat and the other types to Python, they generate wrapper functions and return NumPy arrays.
However, pybind11 easily allows to expose existing matrix-like types in Python, by supporting the buffer interface. I've implemented that in e0027ba74d766c9978c9519816e9e207efca37d3 for cv::Vec4f and it works great.
One open issue is how to pass images (as NumPy arrays) from Python to C++, but it seems like pybind11 easily supports that case too.

Thanks for this binding. What are the remaining blocking points to expose fitting::fit_shape_and_pose ?

@sfornengo Hi, nothing actually! In fact I just finished that yesterday. It's in the devel branch - see the demo: https://github.com/patrikhuber/eos/blob/devel/python/demo.py
Let me know if you have issues.
There are a few more changes in the devel branch that are not finalised yet, like lots of CMake modernisation stuff - but everything should work fine.

I just try the devel branch and I confirm everything works fine, thanks !
Could we expect render::extract_texture to be exposed too or it is out of your python binding scope ?

Cool!
render::extract_texture is definitely high up on the list too. The main obstacle is that it requires passing and returning of an image (a 3-channel matrix), and Eigen doesn't really support that. There's a few options, I just don't know which is best - let me give it a try today and tomorrow.

This should now work too and there's transparent conversion between cv::Mat and NumPy arrays. See b408cd369dc87c9980d7fdd0e7fb7863a86a7685, dd44ea20baa5482b63df30819aa20de6a4cf535a.

I will close this issue, as the Python bindings are pretty solid now in the devel branch!
Feel free to comment here or open a new issue if anyone wants bindings for something that doesn't work yet.


Ignore non-label columns (recommendation, text classification, Sentence similarity, ...)

For the following scenarios we do not have feature column, hence if the column is not one of the label columns it should be ignored automatically.

Recommendation  =>  not (Label or user or Item) =ignore
Text-classification => not ( Label or text) =ignore
Sentence similarity => not (Label or sentence one or sentence two) =ignore

This is a dupe of https://github.com/dotnet/machinelearning-modelbuilder/issues/2296


Prevent concurrent organisation imports

We should only do one of these at a time, but the cron job runs on all (3)
servers, so can trample over itself.
I suspect this is the cause of duplicate organisations in our database - the
when one instance calls find_or_create_by(slug: slug), it finds nothing, so
creates a new organisation, but another instance has done the same thing a
fraction of a second earlier.
Pattern copied from Transition: https://github.com/alphagov/transition/blob/master/lib/transition/distributed_lock.rb
Merge after: https://github.gds/gds/alphagov-deployment/pull/832

1 minor comment, otherwise looks good to me.


Last time I tried, EV did not automatically appear in the testing ground (when importing a 100% EV testing ground scenario)

I used this scenario

I'm seeing lots of cars now:
#...
- name: 'Electric car #29'
  type: transport_car_using_electricity
  profile: ev_profile_9_3.7_kw
  capacity: 10.0
- name: 'Electric car #32'
  type: transport_car_using_electricity
  profile: ev_profile_6_3.7_kw
  capacity: 10.0
# ...

:+1:


⚠ Bofrost.de has degraded performance

In 5e39d52, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1256 ms

Resolved: Bofrost.de performance has improved in c5f59f5 after 9 minutes.


[11.x] Apply default timezone when casting unix timestamps

Carbon 3 no longer apply default timezone while creating from unix timestamp
Laravel 11 supports both Carbon 2 and 3. I think it should behave the same regardless of installed Carbon version.
We may switch to the new behaviour in the next major version given that we'll drop support for Carbon 2.

@daniser can I confirm with you that you're using a different timezone in php.ini than UTC?

@driesvints yes, both in php.ini and config/app.php.

I confirm this change will make the method behave like in Carbon v2.


feat:scroll to top button

What does this PR do?
add the scroll to top button on the home page so that the user can go directly to the top of the home page
fixes #62
screenshot/recording
https://github.com/gauravsingh1281/Rentalog-Rent-Management-Web-App-For-Landlords/assets/66733074/9cd02012-6dc2-4c7b-a2b9-3d66bd0b1215

Hii @pratikshelar546 thank you for your contribution 😇.


[FEATURE] Make timestamp, changeset, uid and user optional

Is your feature request related to a problem? Please describe.
If I understand correctly, following fields can be empty: timestamp, changeset, uid, user
Describe the solution you'd like
Make them of Option type

@zdila please clarify which object the request refers to

Following structs:
osm_io::osm::{
    model::{
        node::Node,
        relation::Relation,
        way::Way,
    }

Actually there are more fields which should be optional according to https://wiki.openstreetmap.org/wiki/PBF_Format#Definition_of_OSMData_fileblock
...
optional Info info = 4;
...

message Info {
   optional int32 version = 1 [default = -1];
   optional int32 timestamp = 2;
   optional int64 changeset = 3;
   optional int32 uid = 4;
   optional int32 user_sid = 5; // String IDs

   // The visible flag is used to store history information. It indicates that
   // the current object version has been created by a delete operation on the
   // OSM API.
   // When a writer sets this flag, it MUST add a required_features tag with
   // value "HistoricalInformation" to the HeaderBlock.
   // If this flag is not available for some object it MUST be assumed to be
   // true if the file has the required_features tag "HistoricalInformation"
   // set.
   // If visible is set to false, this element has been deleted.
   optional bool visible = 6;
}

Actually if I set numeric values to zero and for user "" then after conversion with osmium I don't see those fields in osm xml file, so maybe this is enough, even if not the most idiomatic way to do it in Rust ;-).

I can create an alternative constructor for core objects (Node, Way, Relation) that accepts these values as optional. @zdila


quantms subworkflows to be built

What subworkflow to be built? such as QC?
DecoyDatabase and OpenmsPeakpicker should be called as single module.

The next subworkflows:

[ ] DatabaseSearchEngines including the following processes: process search_engine_msgf, process search_engine_comet
[ ] index_peptides: Could be a single module that can be reuse multiple times.
[ ] PercolatorPeptideIdBoost including the following processes: extract_percolator_features, percolator
[ ] PSMFDRControl including the following processes: fdr_idpep, idpep, idscoreswitcher_to_qval, consensusid , fdr_consensusid, idfilter
[ ] PhosphoScoring including the following processes: idscoreswitcher_for_luciphor, luciphor
[ ] ProteinInference including idemapper, file_merge , protein_epifany, protein_inference, epi_filter , resolve_conflict
[ ] ProteinQuant including  pro_quant, MsstatsConverter
[ ] pmultiqc should be a single module.

What do you think about this proposal @daichengxin @jpfeuffer ?

I think index_peptides can be the end of the DatabaseSearchEngines subworkflow because all the time will be needed.

DatabaseSearchEngines is ok. And they shared parameters. PercolatorPeptideIdBoost is also ok, they have close relationship. PSMFDRControl seem to have too many modules. idfilter could be a single module that can be reuse in ProteinInference. PhosphoScoring is ok. idemapper and file_merge seem to have no direct relationship with protein inference. idemapper  should be with isobaricanalyzer.  file_merge  can be a single module. ProteinQuant and pmultiqc are ok.

I would do:

[ ] DatabaseSearchEngines including the following processes: process search_engine_msgf, process search_engine_comet and index_peptides (starting from 2.7.1, indexing is done internally per default anyway)
[ ] PSMRescoring including the following processes: extract_percolator_features, percolator, fdr_idpep, idpep
[ ] ConsensusID could also be a single module
[ ] PSMFDRControl including the following processes: idscoreswitcher_to_qval (if IDPEP was chosen), fdr_consensusid (if consensusID was used), idfilter
[ ] PhosphoScoring including the following processes: idscoreswitcher_for_luciphor, luciphor
[ ] IDMapper could be a single module
[ ] FileMergecould be a single module
[ ] ProteinInference including protein_epifany, protein_inference, epi_filter
[ ] ProteinQuant including  resolve_conflict , pro_quant, MsstatsConverter
[ ] pmultiqc should be a single module.

Looks like a great strategy. I will open independent issues.

Isobaricanalyzer  could be with IDmapper ? In LFQ, there is no IDMapper

Yes could be together, in LFQ this happens in the ProteomicsLFQ step.

@daichengxin if everyone agreed with @jpfeuffer https://github.com/bigbio/quantms/issues/8#issuecomment-963098866 we should do a PR by subworkflow.

Done.


Can you explain our homework question?

(1. Is the homework about running matrix operations using different IDE?
2. Is it about running the operation with various methods including Einstein notation? )
and then comparing the execution time?
1st or 2nd? which one do we need to follow? @laserlab

Well, so far I have understood it's both. We perform matrix operations by writing Python code and finding out its execution time. Also, our homework says about "compare" which I didn't get it. Compare with what? Any idea?

First of all you need to do the coding using different method.  for example by looping or ma einstein summation method. And you need to run those code and at last compare the execution time between those process.

https://github.com/ubsuny/CompPhys/blob/main/ReviewPython/EinsteinNotation.ipynb
or you can have a look on this

@poojashresthacode Thanks. This makes sense to me now.

Here is the clarification from the professor.


Add PHP extensions: imagick, pcov and sodium

Tool information

Tool name: imagick, pcov and sodium PHP extensions
imagick on all supported PHP versions, sodium on PHP 7.0 and 7.1 and pcov on PHP 7.1 and newer.
Tool license:

imagick: PHP-3.01
pcov: PHP-3.01
sodium: BSD-2-Clause

Add or update? Add
Desired version: latest versions in ppa:ondrej/php
Approximate size:
If this is an add request:

Brief description of tool:

imagick: PHP extension to modify images using imagemagick library.
pcov: Widely used code coverage driver in PHP. This is an alternative to Xdebug.
sodium: PHP extension providing cryptography functions from libsodium library.

URL for tool's homepage:

imagick: https://github.com/Imagick/imagick
pcov: https://github.com/krakjoe/pcov
sodium: https://github.com/jedisct1/libsodium-php

Provide a basic test case to validate the tool's functionality: php -m

Area for Triage:
PHP
Question, Bug, or Feature?:
Feature
Virtual environments affected

[X] Ubuntu 16.04
[X] Ubuntu 18.04
[X] Ubuntu 20.04
[ ] macOS 10.15
[ ] macOS 11.0
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Can this tool be installed during the build?
Yes
Tool installation time in runtime
~10s
Are you willing to submit a PR?
Yes

Hi!
PR was merged and will be available with the next deployment next week.

@AlenaSviridenko Thanks for the update

Hello @shivammathur,
Could you please check that PHP extensions are in place?
We're looking forward to your reply.

@Darleev
As per the build logs of the PR #3405, extensions should be ok. I will be able to verify this once the changes are deployed.

Is there some delay before this is enabled all over? I've been testing imagick on my pipeline but it's sometimes installed and sometimes not there.
On my beta branch:
Current runner version: '2.278.0'
Operating System
  Ubuntu
  20.04.2
  LTS
Virtual Environment
  Environment: ubuntu-20.04
  Version: 20210517.1
  Included Software: https://github.com/actions/virtual-environments/blob/ubuntu20/20210517.1/images/linux/Ubuntu2004-README.md
  Image Release: https://github.com/actions/virtual-environments/releases/tag/ubuntu20%2F20210517.1

  ---

  ### Step: Verify Imagick
> Run php -r 'echo "imagick is ".(extension_loaded("imagick")?"":"not ")."installed\n";'
> imagick is not installed

On my staging branch (running at the exact same time):
Current runner version: '2.278.0'
Operating System
  Ubuntu
  20.04.2
  LTS
Virtual Environment
  Environment: ubuntu-20.04
  Version: 20210524.1
  Included Software: https://github.com/actions/virtual-environments/blob/ubuntu20/20210524.1/images/linux/Ubuntu2004-README.md
  Image Release: https://github.com/actions/virtual-environments/releases/tag/ubuntu20%2F20210524.1

---

  ### Step: Verify Imagick
> Run php -r 'echo "imagick is ".(extension_loaded("imagick")?"":"not ")."installed\n";'
> imagick is installed

On the staging pipeline with imagick I was able to use and generate my thumbnails correctly, but not on the beta branch.
However, I notice the version number is slightly different 20210524.1 ... should or can we specify a specific version of Ubuntu in the Github Actions to get this working or will this random bug or imagick sometimes not being installed disappear over time?

@Darleev I have verified the extensions.

@tao Ubuntu 20 image version 20210524.1 will be fully deployed in the next few hours. Sorry for the inconvenience.


support for animated shaders

Hi - not sure if this extension allows for animated shader support. I applied the following simple animated shader as a test:
uniform vec2    u_resolution;
uniform float   u_time;

float animationSeconds = 2.0;
float piTimes2 = 3.1415926536*2.0;

void main(void)
{
    float sineVal = sin(piTimes2*(u_time-0.75)/animationSeconds)/2.0+0.5;
    gl_FragColor = vec4(sineVal,0.0,1.0-sineVal,1.0);
}

and the initial color seems to be applied, but I wasn't sure how to test whether the animation was working.

Hi i am facing the same issue . my  animated shader works on the (http://editor.thebookofshaders.com/) editor but not in blender.

Hi - not sure if this extension allows for animated shader support. I applied the following simple animated shader as a test:
uniform vec2    u_resolution;
uniform float   u_time;

float animationSeconds = 2.0;
float piTimes2 = 3.1415926536*2.0;

void main(void)
{
    float sineVal = sin(piTimes2*(u_time-0.75)/animationSeconds)/2.0+0.5;
    gl_FragColor = vec4(sineVal,0.0,1.0-sineVal,1.0);
}

and the initial color seems to be applied, but I wasn't sure how to test whether the animation was working.

this branch is working with animated shaders(https://github.com/mateuszz0000/glslTexture)


Support of ERC 1155 Tokens #2750

Closes  #2750

@vladyslav-iosdev too big to review :) is it done on your side? If it's not exposed and doesn't break anything, we can merge it into master and I can work on it together with the backend.

@vladyslav-iosdev how can I see it in action?

@vladyslav-iosdev how can I see it in action?

tapping on any erc1155 token opens a new flow.

@vladyslav-iosdev too big to review :) is it done on your side? If it's not exposed and doesn't break anything, we can merge it into master and I can work on it together with the backend.

it should't, but i will double check.

Ok, I ran it by making an ERC721 token open up the flow. Seems generally ok. Can you help take a quick look if it doesn't break anything if there are no ERC1155 tokens, then let's just merge it.

Ok, I ran it by making an ERC721 token open up the flow. Seems generally ok. Can you help take a quick look if it doesn't break anything if there are no ERC1155 tokens, then let's just merge it.

ok, np.

Merge if you are ready, or let me know if you want me to merge


Could you provide encrypted sample report for testing?

I have the aggregation service set up, but our system to produce encrypted reports is not ready to go yet. This repo's sampledata directory has a sample report, but it is unencrypted and so only works with the local testing tool only, not with AWS Nitro Enclaves.
Could you provide, either in the repo or in a zip file in this thread, an encrypted sample output.avro and accompanying domain.avrp that we can use to test our AWS aggregation service to make sure everything is running properly?

Hi @JonathanAquino-NextRoll ,
Unfortunately, we can't provide sampledata for avro reports. However, we are planning to publish some guidance on creating these reports. We have shared this with NextRoll and will publish once available.
Thanks!


Multiple definitions of reflex_code_INITIAL when using multiple lexers

My program has two lexers (source attached). The generated source code contains multiple functions with the same signature and external linkage, resulting in this build warning from MSVC linker:
D:\src\yycc\out\build\x64-Debug\PPTokenLexer.yy.cpp.obj : warning LNK4006: "void __cdecl reflex_code_INITIAL(class reflex::Matcher &)" (?reflex_code_INITIAL@@YAXAEAVMatcher@reflex@@@Z) already defined in IdentifierLexer.yy.cpp.obj; second definition ignored
This is the declaration of the generated function causing the warning:
extern void reflex_code_INITIAL(reflex::Matcher&);

Based on what the RE-flex user manual says in section "Combining multiple lexers", I think the intent is that this should work and that it should not be necessary to put the lexers in separate namespaces.
I modified RE-flex to instead give the function internal linkage, which worked. I'm not sure if it is the right approach though, hence bug report rather than pull request.
void Reflex::write_lexer()
{
  // ...

  if (options["matcher"].empty() && !options["fast"].empty())
  {
    for (Start start = 0; start < conditions.size(); ++start)
    {
      if (!options["namespace"].empty())
        write_namespace_open();

      // I changed "extern" to "static" on this line
      *out << "static void reflex_code_" << prefix_opt << conditions[start] << "(reflex::Matcher&);\n";

      if (!options["find"].empty())
        *out << "extern const reflex::Pattern::Pred reflex_pred_" << prefix_opt << conditions[start] << "[];\n";
      if (!options["namespace"].empty())
        write_namespace_close();
    }
    *out << '\n';
  }
  //...

RE-flex version is 3.3.0.
PPTokenLexer.l.txt
IdentifierLexer.l.txt

Thanks for the feedback. I will take a look at this soon to decide if static is permissible with respect to the intended use cases.

Changing to static is permissible if option --tables-file is not used. This option produces a separate source code file with the DFA code and/or tables that are compiled and linked with the generated scanner code. However, changing to static also causes problems with --fast not compiling any longer. So this needs more work. The first step in that direction is:
      if (!options["tables_file"].empty())
      {
        *out << "extern void reflex_code_" << prefix_opt << conditions[start] << "(reflex::Matcher&);\n";
        if (!options["find"].empty())
          *out << "extern const reflex::Pattern::Pred reflex_pred_" << prefix_opt << conditions[start] << "[];\n";
      }
      else
      {
        *out << "static void reflex_code_" << prefix_opt << conditions[start] << "(reflex::Matcher&);\n";
        if (!options["find"].empty())
          *out << "static const reflex::Pattern::Pred reflex_pred_" << prefix_opt << conditions[start] << "[];\n";
      }

Perhaps instead of replacing extern with static, it would help to incorporate the lexer class name in the function name. Currently, it generates:
extern void reflex_code_INITIAL(reflex::Matcher&);

Suppose I have two lexer classes named "PPTokenLexer" and "IdentifierLexer", it could instead generate these for their INITIAL states:
extern void reflex_code_PPTokenLexer_INITIAL(reflex::Matcher&);
extern void reflex_code_IdentifierLexer_INITIAL(reflex::Matcher&);

It seems to already do something along these lines when --flex mode is in use. For example, if two flex lexers were prefixed "pptok" and "id", the current code would (I think) generate these:
extern void reflex_code_pptokINITIAL(reflex::Matcher&);
extern void reflex_code_idINITIAL(reflex::Matcher&);

So perhaps everything could be made consistent by always using the lexer class name and not including the prefix (since the prefix is incorporated in the lexer class name). For example, in my --flex mode example above, these might be generated:
extern void reflex_code_pptokFlexLexer_INITIAL(reflex::Matcher&);
extern void reflex_code_idFlexLexer_INITIAL(reflex::Matcher&);

Renaming should work. But actually, C++ namespaces work fine too. Just add a namespace directive to the lexer file:
%option namespace=pptok

Or you can use
%option prefix=pptok_

to prefix the definitions to get reflex_code_pptok_INITIAL.

Note that the --prefix option is a Flex option. This is a Flex-compatible option that emulates the same behavior as the Flex++ C++ output. This affects a slew of things, including the generated source code filenames.
Strange that in your case the DFA table name lacks the prefix. I'll check it out.
C++ namespaces are preferable. See also https://www.genivia.com/doc/reflex/html/index.html#reflex-multiple

I've committed an update v3.3.1 with a minor change to the reflex tool output for --prefix which apparently in some cases was missing the specified prefix is in the file names and the FSM table name. I don't believe many people use the old Flex-compatible --prefix option with RE/flex ;-)


rolling flutter engine to top of tree Dart sdk results in some tests failing intermittently

Rolling flutter engine to top of tree Dart SDK results in some tests failing intermittently and needs investigation. This is a blocker as flutter would like to roll forward.

When running a flutter app with a local engine I am also seeing random hangs: sometimes after flutter run the Android screen just gets stuck at a black screen when starting the app. Sometimes that app stops responding while scrolling through a list, etc.

Is this a local engine without rolling to top of tree Dart SDK?

Yeah, that's for the local engine as it is currently checked in to flutter/engine.

looks like https://github.com/flutter/engine/pull/3862 should resolve this issue


Compile python reqs, rework docker image testing logic

Automated changes by create-pull-request GitHub action
And then some of my tweaks. By commit is definitely easiest way to review

python reqs
I'll fix message when I squash - ignore a few sqlfluff rules that have bugs in 3.3.0
sqlfluff tweaks - ORDER and WHERE must be on newline.  This is what I would actually like feedback on. I think the logic makes a lot more sense now.

@fvankrieken for the sqlfluff tweaks, were those automated or did you have to do them by hand?

@fvankrieken for the sqlfluff tweaks, were those automated or did you have to do them by hand?

A bit of both, I have my own opinions lol. But mostly automated

@fvankrieken for the sqlfluff tweaks, were those automated or did you have to do them by hand?

A bit of both, I have my own opinions lol. But mostly automated

Haha, cool. I will say it's always nice to separate the automated/manual changes in different commits, but sometimes doesn't doesn't work out that way.

@fvankrieken for the sqlfluff tweaks, were those automated or did you have to do them by hand?

A bit of both, I have my own opinions lol. But mostly automated

Haha, cool. I will say it's always nice to separate the automated/manual changes in different commits, but sometimes doesn't doesn't work out that way.

100%


Git Ignoring Main Entry Point?

In your package.json file, you have lib/index.js as your main entry and the lib/folder is in .gitignore – is that intentional?

Yes. It's intentional.
lib is the build folder which will be built as a pre-publishing step.
If you're developing on the repository, you should run npm run build to compile the code.

OK, good to know! Cheers! :beers:


[Documentation] Create CODEOWNERS group for documentation reviews

https://github.com/orgs/ROCm/teams/rocm-documentation


[BUG] GIT Repo only works if applied on creation of tenant

Hi!
Using GIT repos in IntuneCD Monitor only seems to work if the repo is added while adding a new tenant. Adding a repo and PAT after the tenant is added does not work and will result in the error "Could not clone repository". The logs do not give any meaningful insight to the issue.
On tenants where the GIT repo is configured and working properly, editing the GIT repo URL (in IntuneCD monitor) will break it, even if reverted to the previous setting. The tenant will have to be added again with repo and PAT to fix it.
Running on version 2.1.0.
Have you encountered this before, and do you know of a fix?
Thanks!

What is happening is that an entry for the tenant is created when creating the tenant and a repo is set, this creation in the key vault is not happening when editing a tenant. I can look into changing that behaviour.
As for current tenants editing the URL, are you also updating the PAT to make sure you have a PAT with access to the new repo?

Thank you, it seems the PAT needs to updated at the same time as the new repo, even if the previous PAT would have been valid.


Create new service getOrderdableServiceDetailsByServiceId

[ ] Create new service getOrderdableServiceDetailsByServiceId
[ ] This service must have serviceId in the query. From the serviceId, get the templateId and return the template details irrespective of the status of the service template.

I will take it.


Refactor AwaitLayer to use contexts

Changes
Refactored AwaitLayer to base on Go's contexts instead of channels.
The context returned from AwaitLayer is cancelled when it is time for the given layer.
Test Plan
unit tests
DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

After giving it some thought I begin to have doubts if it is a good idea to signal an event by cancelling a context. It is impossible to distinguish which of the two actually happened: the layer event or parent context cancellation.