Market List exceeding 2048 char limit in embed

Should be as simple as checking the length of the listings string and splitting into multiple embeds

This has happened so many times in so many plugins. Maybe it's worth having a standard way to handle it? Maybe wrap message.send and then chunk the message?

This has happened so many times in so many plugins. Maybe it's worth having a standard way to handle it? Maybe wrap message.send and then chunk the message?

I could create a universal function for this, but the market plugin is a special case. I'd prefer to not have a listing span across multiple embeds

I could create a universal function for this, but the market plugin is a special case. I'd prefer to not have a listing span across multiple embeds

fair enough

fair enough


RCE_NO_H26X_SCL used in push_frame?

Hello,
As far as I understand, push_frame function takes RTP_FLAGS as a parameter. And RTP_CTX_ENABLE_FLAGS are passed to create_stream function.
However, the only way for the RCE_NO_H26X_SCL flag to take effect, is to pass it to the push_frame function. Passing it when creating the stream does nothing. This is because it is evaluated from the flags variable of the push_frame function:

Is this the intended use? And if so, can I use other RTP_CTX_ENABLE_FLAGS on a per-frame basis?
I was struggling for some time trying to get the RCE_NO_H26X_SCL flag to work.

@wowaser This is probably a bug. I must have gotten confused with all the different flags as they are all just named flags. It should use the enable flags, not the frame flags.


New Feature: Add uuid and ulid methods

New Feature: Add uuid and ulid methods
Allow for the ability to replace selected subjects with Unique Identifiers such as uuid and ulid.
<ulid:01GJCBQBSH99KGER8TR44BP51R>
    a <Person> ;
    <hasID> "kreeves" ;
    <hasOccupation> <Actor> ;
    .

<ulid:01GJCBQBSH99KGER8TR44BP51R>
    a <Person> ;
    <hasID> "mdamon" ;
    <hasOccupation> <Actor> ;
    .

Task

Add the following method argument options:

uuid1, uuid3, uuid4, uuid5, ulid

When using UUID methods, do not throw circular dependency errors.

Have implemented the following UUID methods: uuid1, uuid3, uuid4, uuid5


Fixed adding float values into DynamoDB

Thank you for contributing to LangChain!

[x] PR title: Add float Message into Dynamo DB

community
Example: "community: Chat Message History

[x] PR message:

Description: pushing float values into dynamo db creates error , solved that by converting to str type
Issue: Float values are not getting pushed
Twitter handle: VpkPrasanna

Have added an utility function for str conversion , let me know where to place it happy to do an commit.
This PR is from an discussion of #26543
@hwchase17 @baskaryan @efriis

Need to format the code.
If you have any questions, feel free to ask.

How to verify it locally before i push ? and one more thing the issues shows only formatting but is there any other changes do i need to add because i have added an utility function to fix the issue and it is in the same file do i need to place it in other location or it is fine ?

hi @Undertone0809 i have fixed the issue , can you review it again ?

Hi @VpkPrasanna  / @Undertone0809 -- my team is also affected by this issue. Any update here regarding merging this fix in?

Hi @VpkPrasanna / @Undertone0809 -- my team is also affected by this issue. Any update here regarding merging this fix in?

There is no response / update from the maaintainers

When saving floating point values, boto3+DynamoDB doesn't support the float type at all though (only Decimal types, and will throw otherwise). So, hiding this behind an optional flag only serves to keep the current, errant behavior in place. Is there a good reason for that?
I was also thinking about the str conversion as well, but then I realized it's likely there to prevent floating-point precision errors.

for the same reason that aws doesn't add this conversion to their sdks - it's unexpected behavior for people to store float datatypes, and when you retrieve them, they're Decimal types
I agree this is useful, so I think gating behind an attribute is good. And in the event Dynamo adds this feature to their sdk or service, it won't be a breaking change for folks to turn this off!

Gated behind a coerce_float_to_decimal parameter with default False for now. Suggest adding to docs to assist discoverability.

Thank you @ccurme for merging the PR finally


Controller Middleware not working

I am trying to do something like this in my constructor with no success:
$this->middleware('role:premium.user');
$this->middleware('role:tenant.user')->only('index');
The php artisan route:list command shows the middleware in the routes i need it, but i got a 403 in the routes.
Laravel documentation: https://laravel.com/docs/5.7/controllers#controller-middleware

It doesn’t work like that, see:
https://github.com/jeremykenedy/laravel-roles#middleware

My seeds are ok, now checking arround i notice that if i put this in my constructor:
$this->middleware('role:tenant.user')->except('index','show');
This is working but in the oposite way, this supposedly should allow all functions except "index" and "show", but what this is doing is allowing only "index" and "show".
Funny right? XD

By the way ... to make this work i set \jeremykenedy\LaravelRoles\Middleware\VerifyRole::class, as first in kernel.php, $middlewarePriority section.

I ask what the seed looks like because I see you have a .user at the end of the role, is that part of the seed?

I ask what the seed looks like because I see you have a .user at the end of the role, is that part of the seed?

Yes, thats the slug, check the code:
      if (Role::where('name', '=', 'Tenant User')->first() === null) {
            $userRole = Role::create([
                'name'        => 'Tenant User',
                'slug'        => 'tenant.user',
                'description' => 'Tenant User Role',
                'level'       => 0,
            ]);
        }

Can this cause an issue?

I am ignoring the Level tag, just an FYI, all roles have level 0

Ok so ... i try this in the constructor ...
$this->middleware('role:tenant.user')->except('index','show');
$this->middleware('role:admin')->except('index','create','store','show','edit','update','destroy');
But is only taking the lower permissions, so the admin role can only access what tenant.user can, i have set levels in all ways with out success.


request_contact and request_location optional parameters of KeyboardButton object

Can we wait for this in the nearest time?

should work at least with the latest version:
var k = new ReplyKeyboardMarkup(new []
{
    new KeyboardButton("Location")
    {
        RequestLocation = true
    },
    new KeyboardButton("Contact")
    {
        RequestContact = true
    },
});

await Bot.SendTextMessageAsync(ChatId, "Give me information", replyMarkup: k);

Thanks


removed unnecessary var declaration

var $number was unnecessarily assigned to itself.

Coverage decreased (-0.66%) to 75.86% when pulling 1b244ddc27b0fe51a2cff753da9a1d5c3648bb7d on jamesk322:master into b02ff2e29e7b7a9f496f45bdca9c922a34765856 on brandonwamboldt:master.

:100: Looks good to me. Lol ;-)

Hmmm I dunno, that looks sorta important

Eye of the tiger ;)


Bug in invalid_input_check

The invalid_input_check is not performing accordingly as expected. Example:
If some stations have more observations (for vlinder these are blackglobe obs.) than the stations that do not have these types of observations are always flagged.
To reproduce:
testdatafile = os.path.join(
    str(lib_folder), 'tests', 'test_data',  'testdata_okt_small.csv')

static_data = os.path.join(
    str(lib_folder), 'static_data', 'vlinder_metadata.csv')

# #% Setup dataset
settings = vlinder_toolkit.Settings()
settings.update_settings(input_data_file=testdatafile,
                         input_metadata_file=static_data,
                         output_folder='/home/thoverga/Documents/VLINDER_github/vlinder_toolkit'
                         )

dataset = vlinder_toolkit.Dataset()
dataset.import_data_from_file(coarsen_timeres=True)

Than you can see that 'vlinder01' has no good observations anymore.
(This issue has highest priority)

One suggestion to solve this is to apply a groupby('name') and apply the invalid_input_check per station.

btw, i found that the invalid_input_check is quite slow. @MichielVieijra can you think of a way to improve the speed of this check? You can make an issue for it if you want to postpone it.


 Merely the wrong version.
<EMAIL_ADDRESS>uses<EMAIL_ADDRESS>which requires<EMAIL_ADDRESS>npm uninstall --save-dev liquid-fire
ember install<EMAIL_ADDRESS>

So how do you make it work with 0.20.0?

You'll need to upgrade Ember to 0.13.*.

Unfortunately, liquid-fire 0.20.0 throws compile errors with ember 1.13.x...I guess liquid-fire isn't quite ready for glimmer yet. :/

Unfortunately, liquid-fire 0.20.0 throws compile errors with ember 1.13.x...I guess liquid-fire isn't quite ready for glimmer yet. :/

Liquid fire absolutely does work with Glimmer. Here is our whole test suite passing against canary. If you have a specific bug please report it.

jus got this with ember-cli 0.2.7 and "liquid-fire": "^0.20.2"

Not working for me. I started a new projet with ember 1.13.0-beta.2, ember-cli 0.2.7 and liquid-fire 0.20.2.  I generated a new project with ember-cli and just replace {{outlet}} in app/template/appliction.hbs by {{liquid-outlet}}. I get the following error :

There are multiple different issues being confused in this thread.
@raw1z, your problem is being caused because liquid-fire already shipped an update to match a recent change on Ember canary, and the change on the Ember side has not yet shipped into a beta. I will talk to @rwjblue and figure out when it will ship so we can provide clear guidance on compatibility.

I confirmed that the Ember change will definitely ship in a 1.13 beta, and is already on the beta branch. So you can either run the Ember beta branch directly or wait for the next beta release to ship.

ok thx


Merge release 2020.08.28

Making this manually for now.

wait @jsignell why was this closed? I just went to go open the same PR and saw this here.
Right now I'm working on making saturn stricter like we talked about, but it won't work because main of saturn is using different images than main of this repo.

I have no idea.


Design tokens: fonts

https://taskman.eionet.europa.eu/issues/148554

Will return with clean branch.


Update docs and unify function calls

This PR updates the module documentation and unifies the translate and translated function calls. The desired locale is now the last parameter of the calls.

I've canceled the build since it only affects documentation.
This is ready to be merged.


Feat: Enhanced the UI of the footer by changing it's color

What does this PR do ?
This PR enhances the UI of the footer section by changing it's color.
Fixes #55
Type of change
UI enhancement
How should this be tested ?

[x] Scroll down to the footer section of any of the pages
[x] Find the change in color in the footer section

Screenshot of the change

@SulagnaKundu Congratulations on creating your first pull request! I am merging this PR


The --lang argument for grit patterns list doesn't work

It should filter to only show patterns/workflows that apply for that language.

@mentabot fix this one.

@mentatbot fix this one.


if my outputMessage contain https then i have to show options else no options is this possible?

Hey @ajaysingh7623, could you help elaborate on the issue you're facing? From the screenshot, you're not passing a function to options. If there's some conditional logic you wish to use, then use a function instead of an array. It's similar to how message and path can take in a function as well.

from outputMessage string i will check whether string contains https then i will store in state  and pass that state to options yes and it will help me to redirect to new page.
but if there is no options then there should be no options

You can store your output as a variable, then use a function for options. Then, return an empty array if you don't want it to show up.

import axios from "axios";
import { ASSETS_URL } from "config/env";
import React, { useState } from "react";
import ChatBot from "react-chatbotify";
function ChatBotify() {
const [sessionHistory, setSessionHistory] = useState({
quote: "",
quoteID: ""
});
const [urlState, setUrlState] = useState("");
const [optionsEnabled, setOptionsEnabled] = useState(false);
console.log(urlState, optionsEnabled, "data");
async function run(params) {
const authToken = localStorage.getItem("authToken");
const payload = {
userinput: params?.userInput,
history: {
quote: sessionHistory?.quote || "",
quoteID: sessionHistory?.quoteID || ""
}
};
const url = "/api/start";
const config = {
headers: {
"Content-Type": "application/json",
Authorization: Bearer ${authToken}
}
};
try {
const response = await axios.post(url, payload, config);
if (response.data) {
const quote = response?.data?.session_history.quote;
const quoteID = response?.data?.session_history.quoteID;
setSessionHistory((prev) => ({
...prev,
quote: quote,
quoteID: quoteID
}));
const outputMessage =
response?.data?.bot_response +
" " +
response?.data?.next_step_recommendation;
const urlMatch = outputMessage.match(/(https?://[^\s]+)/);
if (urlMatch) {
setUrlState(urlMatch[0]);
setOptionsEnabled(true);
} else {
setUrlState("");
setOptionsEnabled(false);
}
params.streamMessage(outputMessage);
} else {
console.error("Unexpected response structure:", response);
return "Unexpected response from server.";
}
} catch (error) {
console.error("Error during Axios request:", error);
return "An error occurred while processing your request.";
}
}
const flow = {
start: {
message: "Hello, I am CPQ ChatBot, How Can I Help You!",
path: "model_loop"
},
model_loop: {
message: (params) => {
run(params);
},
options: optionsEnabled
? [

yes
,
"No"
]
: [],
path: "model_loop"
}
};
const themes = [
{ id: "solid_purple_haze", version: "0.1.0" },
{ id: "minimal_midnight", version: "0.1.0" }
];
return (
<ChatBot
themes={themes}
settings={{
audio: { disabled: false, defaultToggledOn: true, tapToPlay: true },
voice: { disabled: false },
chatHistory: { storageKey: "example_voice" },
userBubble: { dangerouslySetInnerHtml: true },
botBubble: { dangerouslySetInnerHtml: true },
header: {
title: (

IQ BOT

)
},
chatButton: {
icon: ${ASSETS_URL}/images/Hplogo.png
}
}}
flow={flow}
/>
);
}
export default ChatBotify;
i am using this logic but options is not enable when it is getting url link
or there is other approach so,i can make my link enable so it should be clickable and move to new page.
here i used dangerouslySetInnerHtml attriute also
userBubble: { dangerouslySetInnerHtml: true },
botBubble: { dangerouslySetInnerHtml: true },
please help on this @tjtanjin

The options attribute actually takes in an array of string, so a custom component wouldn't work. For your case you can consider using the component attribute (or render if you're on v1) to add your own custom component.
Alternatively, just use strings inside options then handle the opening of links via params.userInput in the next block.

import axios from "axios";
import { ASSETS_URL } from "config/env";
import React, { useState } from "react";
import ChatBot from "react-chatbotify";
function ChatBotify() {
const [sessionHistory, setSessionHistory] = useState({
quote: "",
quoteID: ""
});
const [urlState, setUrlState] = useState("");
const [optionsEnabled, setOptionsEnabled] = useState(false);
console.log(urlState, optionsEnabled, "data");
async function run(params) {
const authToken = localStorage.getItem("authToken");
const payload = {
userinput: params?.userInput,
history: {
quote: sessionHistory?.quote || "",
quoteID: sessionHistory?.quoteID || ""
}
};
const url = "api/start";
const config = {
headers: {
"Content-Type": "application/json",
Authorization: Bearer ${authToken}
}
};
try {
const response = await axios.post(url, payload, config);
if (response.data) {
const quote = response?.data?.session_history.quote;
const quoteID = response?.data?.session_history.quoteID;
setSessionHistory((prev) => ({
...prev,
quote: quote,
quoteID: quoteID
}));
const outputMessage =
response?.data?.bot_response +
" " +
response?.data?.next_step_recommendation;
const urlMatch = outputMessage.match(/https://[^\s)]+/);
if (urlMatch) {
setUrlState(urlMatch[0]);
setOptionsEnabled(true);
} else {
setUrlState("");
setOptionsEnabled(false);
}
params.streamMessage(outputMessage);
} else {
console.error("Unexpected response structure:", response);
return "Unexpected response from server.";
}
} catch (error) {
console.error("Error during Axios request:", error);
return "An error occurred while processing your request.";
}
}
const flow = {
start: {
message: "Hello, I am CPQ ChatBot, How Can I Help You!",
path: "model_loop"
},
model_loop: {
message: (params) => {
run(params);
},
options:
// optionsEnabled
//   ?
[

yes
,
"No"
],
// : [],
path: "model_loop"
}
};
const themes = [
{ id: "solid_purple_haze", version: "0.1.0" },
{ id: "minimal_midnight", version: "0.1.0" }
];
return (
<ChatBot
themes={themes}
settings={{
audio: { disabled: false, defaultToggledOn: true, tapToPlay: true },
voice: { disabled: false },
chatHistory: { storageKey: "example_voice" },
userBubble: { dangerouslySetInnerHtml: true },
botBubble: { dangerouslySetInnerHtml: true },
header: {
title: (

IQ BOT

)
},
chatButton: {
icon: ${ASSETS_URL}/images/Hplogo.png
}
}}
flow={flow}
/>
);
}
export default ChatBotify;
i went render when i get response from api it contain https link then i have show options else options should not be visible

model_loop: {
  message: (params) => {
    run(params);
  },
  path:  optionsEnabled ? "next_loop" : "model_loop",
},
next_loop: {
  // message: "You've selected an option. What would you like to do next?",
  component: (

      {optionsEnabled && (
        <button
          style={{ borderRadius: "12px" }}
          onClick={() => {
            window.open(urlState, "_blank");
          }}
        >
          Yes
        </button>
      )}

),
  path: "model_loop"
}

i have few questions @tjtanjin
1. each block will execute only once?
2.it will support    path:  optionsEnabled ? "next_loop" : "model_loop", this ternary condition,
because i get response from api it contains https: then below that message button should be visible for yes and no so i can move to particular url
is this possible?

Hey @ajaysingh7623, I'm not sure I understand what you're trying to achieve but here's a quick summary:
If you want to use options:
options: () => {
    If (some condition to show options) {
        return ["Yes", "No"];
    } else {
        return [];
    }
},

If you want to use render (component in v2):
render: () => {
    If (some condition to show component) {
        return <MyComponent>;
    }
},

If you want to open up a link when certain options are clicked, this example does it: https://react-chatbotify.com/docs/examples/faq_bot
To answer your further questions:

A block can execute multiple times if it is being repeatedly path-ed to.
Yes you can have conditionals in path.

Hey @ajaysingh7623, I assume the help on discord was sufficient - if not you can continue the discussion there as well 😊


Get Connections

Get All Connections of member likes YouTube, TikTok, Instagram, ...

This is only possible via OAuth2, so can't be implemented for bots.


Remove old mqttAD entities + fixes

Added a function to remove old/renamed mqttAD entities
Fixed some typos
Renamed some entities to align json format and entity name

Fair. I did this as it was causingan issue for you. I could create a button on the website that does this. But I'll leave this off for now.
I want to tweak the json a bit more too. That might mean more tweaks to the names. So maybe don't merge this yet. I will post a discussion point around this on discord.

I've redone this PR. The first two commits are already in master, so not much to look at there. The last commit has a whole raft of updates. including:
Improve setting discoveryTopic
Renamed some entities to align json format and entity name
Added pump modes to json
I am tweaking the WebUI, but that's not in this commit. Nothing too major, just cleaning things up. I was looking at adding a button to clear the ha auto discovery from mqtt. But that really needs, the mqttHaAutoDiscovery function to move into HAAutoDiscovery.cpp - I think that needs a little thought before it's actioned.
But if you're happy with the updated commit, then it's probably a good point to resync.

We should avoid renaming entities if at all possible. It messes up history in HA. I'll ping you on discord to chat once I'm back online.


Create dependabot.yml

Setup dependabot for notifications on updated dependencies.

Add a limit on the number of PRs that can be opened before merging.


Add a duplicate button to duplicate presets

This PR allows users to duplicate export presets on the Export window.

Thanks!


Make Signature monomorphic

Part of #58928 - Only make signatures monomorphic.

@typescript-bot perf test this


Are Determinations Made on Phone Fingerprinting?

I was looking through the algorithm flow charts and did not find this, however when I was told that I had been exposed to someone I don't know how it determined that with the information I supplied.
Mainly I said I was feeling sick > conditions improving > don't know if I was exposed > not tested in last ten days > tested negative in last 1 to 3 months.
I am enrolled in my State's exposure tracking program but the State app or the iPhone didn't alert me.
Does the CDC survey know or is it playing it safe?

When you answer "I don't know", the algorithm follows the same path as if you had answered "Yes". I takes the cautious approach and plays it safe. If you are enrolled in your State's exposure tracking program, the bot does not know this. The bot does not know who you are.

Thank you, @bph9.


feat: introduce release draft

This PR introduces functionality for drafting a release. In the first step of release creation, the user provides a Git tag, and the release is drafted. The flow is also described in Notion.

[!WARNING]
ReleaseDraft will not be saved to the database; it will only be returned as a response (so client can "prefill" release creation form with drafted release). Therefore, I am not completely convinced about the wording. Feel free to suggest a different name; I will sleep on it and think about it more tomorrow.

Drafted release:

user sees if provided git tag already exists or a new tag will be created
user sees if provided git tag is associated with a GitHub release (in the case of an existing tag).
release name and notes are generated

* Wireframes has older naming such as changelog, instead of release notes etc.

To keep the PR smaller, I only added the necessary functionality to the GitHub client and service models. The rest will be implemented in upcoming PRs (service model tests will be added in this PR once PR is approved).

I have realized this PR needs one more update. Assigning back to myself.


[#286] Redesigned Spell Slots / Spell Pips Presentation

#286

I am going to take a different approach to this.
Rather than try to move the pips from the spellbook tab, I'm going to look into making a new Spellbook tab that recovers lots of space and can provide a better summary of spellcasting resources. It will be available alongside the original.


Loading never ends when opening an unpublished entry in editorial workflow

During my work on #129 I've noticed that when opening an unpublished draft the loading will never stop. Here is how the actions are looking in inspector:

So, basically, the action with type: "UNPUBLISHED_ENTRY_REQUEST" is fired but the SUCCESS or FAILURE not.
I can also share a complete state.json from redux store.

After I've added a FAILURE handler, I see that there is an error happening:
TypeError: Cannot read property 'match' of undefined
I assume this should be fixed before this one can be closed.

Fixed here


Clean away remaining Travis related files

This pull request is ready to merge

Merged
c24b3f2eda0235d04865bf258759d46d8a01608d Clean away remaining Travis related files


Running check for one field

I have some Meteor methods that update one field in a document, and for now I'm doing something like this:
if (email) {
  schemaContext.validateOne({email: email}, 'email') || throw new Meteor.Error(500, 'Invalid email');
  // Update email
}

I wanted to just use check lib for validation, but it doesn't work because it requires all fields in the schema to exist (no validateOne variant of that).

Does this work?
check({email: email}, mySchema.pick('email'));

That works, thanks--but I don't get a very useful error message back when it fails.
It would be ideal if using check on a single field returned the error message for that field.

In SimpleSchema 2.0, this can be done with
mySchema.validate({ email: email }, { keys: ['email'] });

@aldeed v2.0 looks epic and I'm pumped that it can be used with just node. Thanks!

In SimpleSchema 2.0, this can be done with
mySchema.validate({ email: email }, { keys: ['email'] });

What about cleaning just 1 field? Can that be done too?

In SimpleSchema 2.0, this can be done with
mySchema.validate({ email: email }, { keys: ['email'] });

What about cleaning just 1 field? Can that be done too?


Support filtering the repositories page list

A continuation work based on original @kevinSawicki filtering branch..
cc'ed @jonanp @atermenji

@jonanp Can we merge this? Or is it still under your review?

You should update your IDE's copyright template ;)

What about it?

The new files here all say Copyright (c) Github 2012

Good catch. Thanks.

@fadils I'm still reviewing and I think I have some issues, I'll give you more details soon.

@jonanp Will wait for the feedbacks.

@jonan @fadils what's the status?

@hzsweers probably need to resolve many conflicts before it can be merged once again. Let's set this to the release after the first one.
But before continueing this thing, was wondering what feedbacks @jonan is going to give. It will be useful to get his feedback, then continue the development.
Meanwhile, if you have some inputs too regarding this, that would be good too.

closed via #796


Cannot get depth image using OpenCV API

Hi there,
The following code provided by @adujardin  can read images in a light manner. But it cannot get depth image. I just wonder whether we can get the depth image using OpenCV APIs.
It is part of https://github.com/stereolabs/zed-python/issues/28, but I opened a new issue since it is a different topic.
import numpy as np
import cv2

cap = cv2.VideoCapture(0) # depending on the ZED ID

while(True):
    ret, frame = cap.read() # Capture SbS frames
    height, width = frame.shape[:2]
    left = frame[0:height,0:int(width*0.5)] # Extract the left img

    # object detection
    .......

# Release the capture
cap.release()
cv2.destroyAllWindows()

Thanks very much.

Hi,
You need to use the ZED SDK if you want to get the depth map. The depth is not computed in the camera but in the ZED SDK.
The only output you have from the camera is the left and right images.
Best,
/OB/

@obraun-sl i also face same issue. Not able to find way to use ZED SDK functions after we are initiating ZED camera with Opencv-python line "cap = cv2.VideoCapture(0)"
Getting error when opening another instance of camera in same code with"zed = zcam.PyZEDCamera()"  for getting depth  with ZED-Python API.
Want to know if any other way to use these depth functions from ZED-Python API, after receiving ZED images using opencv-python.


pixelwise-loss-weight map

System information

TensorFlow version (you are using): 1.14.0
Are you willing to contribute it (Yes/No): Yes

Describe the feature and the current behavior/state.
U-NET (https://www.nature.com/articles/s41592-018-0261-2) has become a popular way to perform semantic segmentation for cell microscopy.  Instance segmentation requires pixelwise-loss weights.  Currently, model.fit_generator supports only val_sample_weights and class_weight--not pixelwise-loss weights.
There have been several work-arounds proposed for this issue, and I have implemented almost everyone.  Most generated errors except
https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/keras-users/ue1S8uAPDKU/PzL7Vb0bBQAJ
It kind of works, but the Jupyter notework keeps issuing warnings like
E tensorflow/core/grappler/optimizers/dependency_optimizer.cc:697] Iteration = 1, topological sort failed with message: The graph couldn't be sorted in topological order.
When I run it, the results change drastically from run to run.
Will this change the current api? How?
It will add more versatility to the api by incorporating an important aspect of computer vision.
Who will benefit with this feature?
Those using keras to implement computer vision programs.
Any Other info.
Thank you kindly for your consideration.

@shinlin77 Will you be able to try this in the latest version of TF? We have made a number of changes to sample weights behavior and we have very few restrictions on sample weights today.
In TF2.2 you are not required to specify sample_weight_mode in compile API.
Closing this issue as the use case is expected to work in TF2.0. Please feel free to ping or reopen if you are facing any issues.

OK, lemme give it a shot. I’ll let you know how it goes. Shin Sent from Mail for Windows 10 From: Pavithra VijaySent: Thursday, April 2, 2020 10:57 AMTo: tensorflow/tensorflowCc: shinlin77; MentionSubject: Re: [tensorflow/tensorflow] pixelwise-loss-weight map (#38060) @shinlin77 Will you be able to try this in the latest version of TF? We have made a number of changes to sample weights behavior and we have very few restrictions on sample weights today.In TF2.2 you are not required to specify sample_weight_mode in compile API.Closing this issue as the use case is expected to work in TF2.0. Please feel free to ping or reopen if you are facing any issues.—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or unsubscribe.

Sample weights, even if you are using custom generators (as far as I know), need to be 1D or 2D.
For pixelwise weighting, the ideal would be having something like @shinlin77 is saying (for one-hot encoded and NCHW: [batch_size, n_classes, height, width]).
This at the moment is not possible (sample weights can be either one weight per sample in batch or one weight per class of sample in batch).


Add chown to COPY command in Dockerfile

In Dockerfile, i add a chown command to copy these files with the node user and group !

What's the point of this change? Can you explain?
Being able to change the container contents at runtime? For debugging, you should be using a different image or no image at all.


On BrickPi, motor output is ttyAMA0:MA not outA or ttyAMA0:outA

Working with @dlech in #ev3dev, we figured this out.

Is it just outA's problem? Shoud outB - outD be changed as well?

Right, sorry it is all of them. And I made a pull request to fix it, https://github.com/rhempel/ev3dev-lang-python/pull/185

Can this be closed? It looks like #185 was merged.


Docs: add new translated blog - Tool, State, and Resource - Chinese Version

Pre-Checklist
Note: please complete ALL items in the following checklist.

[X] I have read through the CONTRIBUTING.md documentation.
[ ] My code has the necessary comments and documentation (if needed).
[ ] I have added relevant tests

Description

Related Issues
#724
New Behavior (screenshots if needed)

A kind reminder that you haven't signed off on your commit(s) yet. Check out our official doc to learn how to do this.

Hi @LauraJW , thanks for your contribution! Please join CNCF's Slack, and join our channel in CNCF Slack!
Here are a few useful things for you to continue your open-source journey:

If you are searching for something else to work on, please see "find an issue".
Check out our new contributor guide and the "Developer Guide".
Read our blog; they are on Medium, too.

We made changes to your PR directly so that your contribution can be merged quicker. We hope you don't mind!
A kind reminder that you haven't signed off on your commit(s) yet. Check out our official doc to learn how to do this. Also, DevStream community follows the conventional commits specification. Please read our doc and update your commit message.


TabbedPageWorkarounds:  warning XA4214: The managed type Microsoft.Maui.Controls.Platform.FragmentContainer exists in multiple assemblies

When building my app with version 1.2.4 of TabbedPageWorkarounds, I get this warning:
C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\33.0.26\tools\Xamarin.Android.Common.targets(1476,3): warning XA4214: The managed type `Microsoft.Maui.Controls.Platform.FragmentContainer` exists in multiple assemblies: PureWeen.Maui.FixesAndWorkarounds, Microsoft.Maui.Controls. Please refactor the managed type names in these assemblies so that they are not identica
l. [D:\Dev\Zitch\ouibank\OuiBank.Maui\OuiBank.Maui.csproj::TargetFramework=net7.0-android33.0]
C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\33.0.26\tools\Xamarin.Android.Common.targets(1476,3): warning XA4214: References to the type `Microsoft.Maui.Controls.Platform.FragmentContainer` will refer to `Microsoft.Maui.Controls.Platform.FragmentContainer, PureWeen.Maui.FixesAndWorkarounds`.

Could you please to rename this class in order to get rid of the warning? Or maybe it's not possible for some reason?
Thanks

👍

Thanks!


Printing HTML with barcode font

I am converting raw HTML to PDF: await PrintingService.Print(html, PrintType.RawHtml);
The css style applied for printing barcodes is style="font-family: 'connectcode39';" . But it is not being reflected in the print.

The font can be recognized in  an external css file
and included in the component


Contribution Guide

Hi Martin,
I really love the app and have been wanting to rebuild it in Flutter for some time! I would love to get involved with this project, do you have a contribution guide and/or roadmap of features that you want developed so that I can help out?
Thanks,
Lockie

Hi @lockieRichter!
I finally manged to setup a contribution guide, updated the readme and created a backlog of features. You can see the open tickets here:
https://github.com/users/shockbytes/projects/1/views/1
And here's the link of the main communication channel on discord:
https://discord.com/channels/824694597728993390/1037365721363136572


update about page: not build with hugo anymore

Hello,
This paragraph need an update right ?
https://github.com/netlify/headlesscms.org/blob/master/content/pages/about.md#how-is-it-made

cc @philhawksworth @sw-yx I bet one of y'all can handle this


Fix duplicate values on colorbar

closes #142 - This PR fixes duplicate values on the colorbar when plotting bias. The colorbar can now be from -0.1 to 0.1 and not has duplicate values.

Can't you have duplicated values with two significant digits? E.g., if you limits are 0.001 and -0.001


Save and Load 'projects'

The IDE should be able to save and load projects
Required Functionality:

The saving function saves the current state of the project including asset- and entity information

Desired functionality:

The project can be placed anywhere on the computer (including paths with spaces or non-ASCII chars)

Adding to this, we should implement some type of "level"-file format which saves entities that are in a level etc.


Wrong documentation samples: http://jscs.info/rule/requireCapitalizedConstructorsNew

The samples should use "requireCapitalizedConstructorsNew" rule instead of "requireCapitalizedConstructors"

Do you want to fix it?


"Or" tag filtering in instant search

Feature Description
If I'm using the instant search and select "Rogue-like", I have the options to narrow my search down by other tags, or exclude some. However, if I wanted to get a list of all games that were "Rogue-like" or "Rogue-lite", there's no option to do so.
Example: having an "or" button, similar to the "exclude" button, would allow users to search all "Rogue-" games by selecting
"Rogue-like" or "Rogue-lite" or "Action Roguelike" or "Roguevania" or "Traditional Roguelike" or "Roguelike Deckbuilder"

In the mean time, sales page does allow switching to any matching for tags.


Need to add separate allowed error for arm/elevator classes

Need to use a separate allowed error for arm/elevator classes because if we set allowed error above 0 on TalonSRXs, when they are within the error the neutral motor output (which would cause arm/elevator to fall).

Fixed my Jay


Please keep updated the all.yaml file in the deploy directory as well

Hello,
I receive notifications about the new releases but seems the all.yaml file not updated in the deploy directory.
Please update that file as well to correspond to the latest version.

Thanks for the heads up!

Thank you for the quick response and fix.


Support restricting workflow_job events by a repository whitelist

Ref https://github.com/actions-runner-controller/actions-runner-controller/issues/1607
Add support for a repository whitelist like we have for the check_run event but for the workflow_job event so we can remove support for all events other than workflow_job

@toast-gear Hey! I'm quite confused. Why do you need it over HRA + RunnerDeployment of repository runners?

No community use case for this as far as we are aware https://github.com/actions-runner-controller/actions-runner-controller/issues/1607#issuecomment-1190144259


fix auth behavior on error

fixes #14
@fdietze Does that make sense? Because I could not really reproduce the problem.
But now we always throw away our local-storage when there is an error. Also we only deserialize to json when the status is 200. Such error now all let the IO fail and that should be recovered in the end.

I'd wait until the error comes up again before merging this.


issue with the question

hey, tired to implement ur prj while doing i kept on getting this problem on too many pages wheni only had 1 , u have any idea why, in ur deployed url its happening

My OpenAI key has expired, you can deploy it yourself with your own key.


Create synchronous / blocking pull

We have a trickle of customer requests for synchronous / blocking pulls.  Whether they are a great idea or not is somewhat irrelevant, customers need them.  The motivations include, but are not limited to:

Some applications use blocking pulls with other Pub/Sub services, re-architecting the application is not feasible.
Some applications just need to pull a few messages and then exit.

After discussing this in https://github.com/googleapis/google-cloud-cpp/discussions/8476 I would like to add one reason to the list:

The ability to control when messages come in. For example to pause.

No time this quarter, but I am hoping to get to it next quarter.


wdio cucumber json report shows screenshot as broken image in html report

"devDependencies": {
"@wdio/allure-reporter": "^5.12.1",
"@wdio/cli": "^5.12.1",
"@wdio/config": "^5.12.1",
"@wdio/cucumber-framework": "^5.12.1",
"@wdio/local-runner": "^5.12.1",
"@wdio/selenium-standalone-service": "^5.12.1",
"@wdio/spec-reporter": "^5.12.1",
"@wdio/sync": "^5.12.1",
"allure-commandline": "^2.9.0",
"multiple-cucumber-html-reporter": "^1.12.0",
"wdio-cucumberjs-json-reporter": "^1.0.8",
"webdriverio": "^5.12.1"
}
}
feature file
Feature: Example feature
Scenario Outline: Get title of website
Given I go on the website ""
Examples:
|website|
|www.google.com|
|www.github.com|
|www.gmail.com|
stepdef file:
var assert = require('assert')
const { Given, When, Then } = require('cucumber')
const cucumberJson = require('wdio-cucumberjs-json-reporter').default
Given(/^I go on the website "([^"]*)"$/, (url) => {
browser.url(url)
cucumberJson.attach(browser.saveScreenshot('Test.png'), 'image/png')
})
wdio.conf
onComplete: function() {
generate({
jsonDir: '.tmp/json/',
reportPath: '.tmp/report/',
});
},
HTML report:

Hi @gtilekar !
Have you tried use takeScreenshot instead saveScreenshot?

It's same with takeScreenshot()

I think image can't be put in step this way.
This is the code in my conf:
afterStep: (uri, feature, scenario, step, result) => {
const cucumber = require('cucumber');
if (result.status === cucumber.Status.FAILED) {
console.log("FAILED");
const cucumberJson = require('wdio-cucumberjs-json-reporter').default;
cucumberJson.attach(browser.takeScreenshot(), 'image/png');
}
},

@robsonpolicarpo Thanks... it works on "wdio-cucumberjs-json-reporter": "^1.0.0"

@gtilekar
This has to do with the fact that you are using an incorrect screenshot method, as @robsonpolicarpo mentioned you should use browser.takeScreenshot(), see the examples https://github.com/wswebcreation/wdio-cucumberjs-json-reporter#attachment and also the docs of WebdriverIO
saveScreenshot doesn't return a string, it only saves it to a location
https://webdriver.io/docs/api/element/saveScreenshot.html
takescreenshot takes a screenshot, doesn't save it but returns a buffer, this is needed for the report
https://webdriver.io/docs/api/webdriver.html#takescreenshot
I'm going to close it for now because this is not an issue with the report


Added QOTD Plugin

Added new slash command plugin qotd.

@merkidemis Any thoughts on the above?

Correct, it should be request.quotes[0].quote.
For additional interest, though I'd have to see if they were possible, I'd use the category to change the bot icon, change the font color, or insert an emoji. I'd also attach the associated image to the feed as well, though it would be cool if it could be set as a Slack or desktop background. Additionally, using the other queries available on that site I would like to be able to specify particular categories per user, or by the admin for certain situations like motivation when we're getting close to the end of the quarter, or inspire during a slow period.

For a quality increase I'd split out that complex line on the .s to individual assignments so I could more accurately detect where data was missing, say if the JSON schema changed.

Let me know when you submit a new pull request with a functioning plugin. I'll take a look again.

I've been unable to test this.

Slack-Stuart will not compile on Windows 10
it compiles under Linux (Ubuntu 15.10), but fails on startup with an error in your markov plugin:
ERROR - Caught Error ReferenceError: Map is not defined
ERROR - Stack ReferenceError: Map is not defined
I cannot run it locally anyway as all incoming http requests are blocked by my ISP
It compiles on heroku.com, but throws an Application Error

All these have been attempted on Node 0.10.36 and Python 2.7.


ADD PRODUCTS to Commerce.js API

@Vivek-kumar21 and @HimaghnaDas this is the most simple task. As you guys are beginners I'm assigning this easy work to you. You have to add data for our Products.jsx Page.
Step 1

Go to this website: https://dashboard.chec.io/

Step 2

Log in using Craftsmen's official Email and Password. Please find it in the description of our WhatsApp Group.

Step 3

After logging in see the sidebar of the page. There you'll find the Products.  After expanding click on the Products.

Step 4

There add products for each category.

This job requires no coding skills. I'll expect you to complete this job by 28th Feb.

@HimaghnaDas @Vivek-kumar21 start working!!!

Finished

Finished

Good @HimaghnaDas @Vivek-kumar21


Display relevant message when network is down

Instead of showing "No data", display something meaningful when the network is not reachable.

Need to handle the scenario where it's just one / some nodes that are down rather than the whole network - could just replicate the same as Console to keep it consistent and as easy as possible


Control.PivotOffset assignment is destructive with existing Rotation or Scale

Tested versions
4.3.1-rc [ff9bc0422], custom build (scons module_mono_enabled=yes debug_symbols=yes ccflags="-fvar-tracking -fvar-tracking-assignments")
System information
Godot v4.3.1.rc.mono (ff9bc0422) - Debian GNU/Linux trixie/sid trixie - X11 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 4060 Laptop GPU - 13th Gen Intel(R) Core(TM) i7-13650HX (20 Threads)
Issue description
Assigning to a Control.PivotOffset will change the Control's transform immediately if a non-default Scale or Rotation is in effect, rather than updating the other properties to keep the Transform constant while PivotOffset changes.
The small attached project lets one interactively compare what currently happens, versus what I expected to happen.
Controls:

Scrollwheel scales the Godot logo
Shift+Scrollwheel rotates the Godot logo
Drag pans it around

Uncheck TestingContainer's "Use workaround" property in the Editor to see the current behavior.
This is the implementation of the workaround for what I believe is more developer-friendly behaviour, which just updates the Control's position to preserve the current transform, in a way I tried to make forwards-compatible:
    private static void ChangePivotOffset(Control control, Vector2 newPivot) {
        /*
         * Scale and Rotation will correctly apply themselves relative to the
         * pivot offset, so they can be freely set without altering other
         * properties.
         *
         * If there is no Scale or Rotation set, then PivotOffset can be set
         * non-destructively; the net transform will not change, and the
         * Control will not move.
         *
         * However, if Scale or Rotation is set, then changing the PivotOffset
         * will move the Control, as Scale, Rotation, and Position values are
         * kept constant, and Scale and Rotation are applied relative to the
         * PivotOffset, leading to different results.
         *
         * This function attempts to change the PivotOffset without moving the
         * Control, or otherwise causing any change in the overall Transform,
         * by adjusting the Position of the Control to compensate for the
         * change in the PivotOffset.
         */
        GD.Print("Control position before:", control.Position);
        Vector2 originPre = control.GetTransform().Origin;
        control.PivotOffset = newPivot;
        Vector2 originPost = control.GetTransform().Origin;
        GD.Print("Origin change:", originPre, " -> ", originPost, " net: ", originPost - originPre);
        GD.Print("Control position after:", control.Position);
        control.Position -= originPost - originPre;
        GD.Print("Control position after correction:", control.Position);
    }

I'm happy to implement this in C++, if someone can show me where to look. There's currently a Control::_edit_set_pivot method which does some adjustment to Position and might be equivalent, but a breakpoint on it never seems to be hit, only Control::set_pivot_offset. I could also try implementing this as a new Control method, if the existing behaviour is likely to be relied on?
Possibly related: #52383
Steps to reproduce

Create a Control, and assign a non-default Scale and/or Rotation
Assign a different value to PivotOffset
Position, Scale, and Rotation are unchanged, but given that Scale and Rotation take effect from the new PivotOffset, the Control's transform has changed.

Minimal reproduction project (MRP)
test-pivot.zip

I consider this correct behavior.
pivot_offset is the raw value used for calculating/composing the given Control's transform, the same as position, rotation, scale are. Changing any of these raw values should not have side effects of changing any other of these raw values. The resulting transform should be the same regardless of the order of assigning values to position/rotation/scale/pivot_offset.
The transform is calculated like:
T // translation
R // rotation
S // scale

Control.get_transform() = T(position) * T(pivot_offset) * R(rotation) * S(scale) * T(-pivot_offset)

Assigning to a Control.PivotOffset will change the Control's transform immediately if a non-default Scale or Rotation is in effect, rather than updating the other properties to keep the Transform constant while PivotOffset changes.

Well, changing position, rotation or scale also changes the Control's transform immediately. Why do you expect the pivot_offset to be different in that aspect and not affect the transform?
And yes, in case R(rotation) * S(scale) is an identity transformation (e.g. rotation = 0; scale = (1, 1)  or rotation = PI; scale = (-1, -1)) then pivot_offset has no effect on the transform indeed as then it's a pure translation by position:
I // identity

If:
R(rotation) * S(scale) = I

Then:
Control.get_transform() = T(position) * T(pivot_offset) * R(rotation) * S(scale) * T(-pivot_offset)
                        = T(position) * T(pivot_offset) * I                      * T(-pivot_offset)
                        = T(position) * T(pivot_offset)                          * T(-pivot_offset)
                        = T(position) * I
                        = T(position)

It is already documented for position that it is not affected by pivot_offset. But seems like pivot_offset's docs could be clarified as well:
https://github.com/godotengine/godot/blob/38775731e8494a28deddc3febda895679f49fd6e/doc/classes/Control.xml#L1021-L1026

If a new method like set_pivot_preserving_transform tweaking the position accordingly is desired, then a proposal for this should be opened (possibly with a PR implementing it).


#bugfix: Augmentation code fixing

The previously found bug in the augmentation module is fixed.
Besides, shell examples and a small piece of document are provided to assist understanding.

Cool, good job!


Bump cats to 1.0.0-MF

My bad, I forgot I had specified the cats version in the github4s build

Thanks!


Tutorial Outline: An introduction to the yt-napari plugin

This issue is a tutorial overview for one of the tutorials listed in #40
Title
An introduction to the yt-napari plugin
Summary
This tutorial will cover installation and basic usage of yt-napari. Load the same subset of
data from the same file 3 different ways: GUI loader, json file loader, notebook interface
Outline
1. Introduction

installation of napari
installation of yt-napari

2. the GUI loader

describe components

3. JSON file loader

describe how to write a json file
using a json schema
example with autocomplete

4. notebook interface

describe the yt-napari Scene

tutorial requirements

ability to install packages from pip
yt > 4.0

recording and uploaded, awaiting youtube processing

released! https://youtu.be/ta7rBOYNpmg


[CL][Concentrated Liquidity Module]: implement foundations of the module

Background
As part of milestone 2, we should implement the foundations of the new concentrated-liquidity module.
See #3041 for more details
Suggested Design
The following keeper methods:

CreatePool
JoinPool
ExitPool
SwapExactAmountIn
SwapExactAmountOut

Acceptance Criteria

follows twap folder structure
utilizes new pool: https://github.com/osmosis-labs/osmosis/tree/concentrated-liquidity-main/x/gamm/pool-models/concentrated-liquidity
no message server, only keeper methods

@mattverse this is done right?


Support for PostCSS

Hi @nicksp,
I was wondering if the boilerplate supports or intends to support either PostCSS or styled-components instead of plain SASS? Or is there a recipe or a guide to switch one out for the other?
Thank you for putting out this boilerplate!

Hello @oyeanuj
I was kinda busy with my last project, so haven't had a chance to work more on this project, which is unfortunate. BUT...i am ready to give it a more boost soon, and yes, PostCSS and CSS Modules are in my TODO list ;) So stay tuned ;-)
And if you're willing to commit to the project, pull requests are very welcome! Just drop me a line to discuss the topic beforehand.

I'm happy to help with this! I've been trying to add css modules for the last half hour but they are failing silently. Any idea what could be breaking them?


Amazing project. seems those 3 scheme files in starter_script.bat

.\starter_script.bat
python: can't open file '/home/jovyan/work/q_pack/db_pack/schema/secmaster_db_schema_builder.py': [Errno 2] No such file or directory python: can't open file '/home/jovyan/work/q_pack/db_pack/schema/secmaster_db_symbol_loader.py': [Errno 2] No such file or directory python: can't open file '/home/jovyan/work/q_pack/db_pack/schema/risk_db_schema_builder.py': [Errno 2] No such file or directory ...

duplicate submit.


Roaming: type usage cleanups

Signed-off-by: Jérôme Benoit<EMAIL_ADDRESS>

You can basically to the type change to the proper one on your branch, it's a trivial change.

Just did it.


Terminal Integration

The latest ArchWSL release <IP_ADDRESS> supports a couple of terminals "integration". Is there a page where it explains what that does exactly or can you tell us more about it?

When the exe is executed, the process is transferred to the WSL profile of Windows Terminal.
This allows you to open WSL in Windows Terminal instead of the standard console.

When the exe is executed, the process is transferred to the WSL profile of Windows Terminal.
This allows you to open WSL in Windows Terminal instead of the standard console.

There was already a profile in my default Windows Terminal settings. I think Windows Terminal adds it automatically as soon as it detects the WSL is enabled, I just renamed it to ArchWSL:
{
  "guid": "{a5a97cb8-8961-5535-816d-772efe0c6a3f}",
  "hidden": false,
  "name": "ArchWSL",
  "source": "Windows.Terminal.Wsl",
  "icon": "ms-appdata:///roaming/arch.png",
  "colorScheme": "Andromeda"
},

Does the command specified in the release notes (Arch.exe config --default-term <default|wt|flute>) provide anything different form that?

Currently, it is not possible to deal with changing the profile name.


Bugfix/break

Fixes bug:

show end status without any break taken
assert break, which is run before invoking end, fills up remaining required break before end entry

:tada: This PR is included in version 1.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Capture PCM output of apulse

I have a really strange issue, on which I stuck for quiet a long time now.
My goal is to capture the apulse output stream to a file, without the need of alsa.
I run into two issues:

pa_mainloop runs way too fast which causes high cpu load.
audio callbacks which get requested inside data_available_for_stream gets empty or partial empty buffers as result. I debugged this as far as I could, it comes from firefox AudioStream.cpp class, if Audio underruns, the callback buffer gets filled with zeros.
But why does it underrun? I should be able to fifo the pcm audio output and play the fifo at the same time.

Set  in firefox about:config:
security.sandbox.content.level = 0
which will allow the MOZ_DUMP_AUDIO to work.
Run Firefox and dump the audio playback:
APULSE_PLAYBACK_DEVICE=null APULSE_CAPTURE_DEVICE=null MOZ_DUMP_AUDIO=1 LD_LIBRARY_PATH=/path/to/apulse/libs /path/to/firefox/firefox
Produced AudioStream-1.wav in working directory:

As you can see, firefox does fill the callbacks with zeros which is problematic in my case.
I want to fwrite the audio callback buffers into a file. I managed to filter the zeros out, but the high cpu and the underruns still apear A LOT, so its not possible for me right now to fifo the audio and send it into a player at the same time without lags/buffering issues, because of underruns.
Does anyone have some hint?
It should be possible to let apulse act as a capturing layer instead of an alsa layer.

I tried to work with usleep for eg. the pause state to handle the cpu load which goes crazy. But working with usleep messes up everything even more:
if (paused) {     // client stream is corked. Pause for the bytecnt time because we do not use alsa     size_t bytecnt = MIN(buf_size, frame_count * frame_size);     trace_info_f("PAUSE STATE\n");     pa_usec_t r_usec = pa_bytes_to_usec(bytecnt , &s->ss);     usleep(r_usec); }

I noticed if I use sleep or usleep to prevent high cpu mainloop cycles, firefox stops processing more data which will result in empty callbacks too.
I managed to filter out normal underruns, but I need to find a way to clock/sleep inside the mainloop to keep the cpu usage low without thread blocking firefox internal buffer fill code.

Got it working.


The README file of Event Hubs is confusing

The Event Hub binder sample supports multiple ways to do authentication, connection string, managed identity, and auth file. We should make it clear in the README which properties are required in each scenario. Some enhancements should be added to the readme file.
@lu-cheng please help add more inputs here.

finished. closing the issue


Everything stopped working after update to 2021.5.3

Integration stopped working completely on core version 2021.5.3. Reboot did not help. In log, I see only this multiple times:
2021-05-14 10:06:29 WARNING (MainThread) [homeassistant.helpers.service] Unable to find referenced entities fan.obyvaci_pokoj
2021-05-14 10:07:42 WARNING (MainThread) [homeassistant.helpers.service] Unable to find referenced entities switch.obyvaci_pokoj_night_mode

(obyvaci_pokoj is my Dyson Pure Cool TP04)
dyson_local & dyson_cloud version: 0.12.0

Never ming, it was some kind of problem at my Dyson. Its restart sovled it.


Add video files support

In my op its really a good feature thats missing for some rason.

Great work!


feat: also show unavailable branch-properties AB#6985

@gulfaraz I don't see this change yet on ibf-test..

@jannisvisser neither do I
This merge created v0.55.0 but the test env is on v0.54.0


Tell prisma to force ipv4 even if dns return both

Problem
When the dns return both ipv4 and ipv6 prisma try to connect with ipv6, but i have db port (3306) open only under ipv4
if in connection string you add connect_timeout=300 for a lot of time prisma try to connect and after a timeout prisma fallback in ipv4
Suggested solution
Add an option/env to force prisma to use ipv4
Alternatives
my workaround on ubuntu is add in the end of /etc/gai.conf file:
precedence ::ffff:0:0/96 100

Hi @simonep-subbyx 👋
Is this in context of whitelisting IP addresses while using Prisma Accelerate, Prisma Pulse or are you referring to using Prisma ORM?

Hi @nurul3101
Prisma ORM


Simplifying WDL workflow submission

Description

In an effort to make our WILDS WDL's more accessible to HPC-curious researchers, we are making the input json as minimal as possible:

All inputs other than fastq locations are now optional.
If a reference fasta & gtf are provided, the script will use those, but if not, it will download GRCh38 from NCBI.
To avoid confusion, the script will automatically build a STAR reference directory from the reference fasta/gtf.
To avoid data duplication, it is highly recommended this workflow be run with call caching enabled.

Related Issue

Fixes #5

Testing

Submitted minimal input json via PROOF, works as expected.
Meeting with user on 12/5 to see what they think.

Short-staffed at the moment and need this WDL for a user study. Merging without additional review.


Invalid endpoint format, it should be a valid URL.

when entering the url for me its cdn-upload-aufgldn.aufgeladen.workers.dev it gives out the error Invalid endpoint format, it should be a valid URL. and i dont really understand how cdn-upload-aufgldn.aufgeladen.workers.dev is not a valid url

nvm sorry im retarded i forgot the https

im retarded i forgot the https sorry for opening the issue


Difference between shots and train_shots

I believe num_shots in the code explain number of examples for each class. In the train function the initialization is given as "num_shots = train_shots or num_shots".
Now for 1-shot 5-way example (given),
python -u run_omniglot.py --shots 1 --inner-batch 25 --inner-iters 3 --meta-step 1 --meta-batch 10 --meta-iters 100000 --eval-batch 25 --eval-iters 5 --learning-rate 0.001 --meta-step-final 0 --train-shots 15 --checkpoint ckpt_o15t --transductive
For above case, train-shots = 15 and shots=1, now num_shots in the code would get 15 but it should be 1 as it is one example per class (1-shot, 5-way). Maybe I am missing something, can you please clarify?

For omniglot data, is there a initializer (initial weights) you use to start the training with? As I see in the code, it seems random initialization. Is it true?

In response to your first question: shots is the number of examples per class during evaluation. However, we found that we got better performance on 1-shot and 5-shot classification if we trained the model using more examples per class (e.g. 15). Train shots is thus the number of examples per class used at training time.
Regarding your first question: we use TensorFlow's default random initialization.

Thanks for the response.


feat: names for local plugins

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

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[x] Docs
[ ] Underlying tools
[ ] Other, please describe:

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

[ ] Yes
[x] No

Other information:
This PR tries to simplify getting plugin options when developing local plugins, utilising both thevuePlugins.service field in the package.json and pluginOptions in the vue.config.js.
Using Object instead of an Array for plugin registration, for example:
{
  "vuePlugins": {
    "service": {
      "myCommands": "my-commands.js"
    }
  }
}

key myCommands becomes the plugin name. If that same name is used as key in pluginOptions field, for example:
{
...
  "pluginOptions": {
    "myCommands": {
      "foo": "bar",
      "baz": "quz"
    }
  }
...
}

options for that plugin will become available as the third argument in the PluginAPI, for example:
// my-commands.js
module.exports = (api, options, { foo, baz }) => {
  console.log(foo) // bar
  console.log(baz) // quz
}

Also, this PRs tests now cover registration of local plugins in both cases.

I don't understand why local plugins need a name when you can simply use local:${file}.
To avoid: local:plugins/my-super-cool-plugin.

You also added an unnecessary options to PluginAPI.
Have your read the reasoning I've added this optional options? It's there to provide easier access to plugin options.

To avoid: local:plugins/my-super-cool-plugin.

Yeah, could you please expand on that a bit more? What were the issues you are having and how does this fix it? Etc..

To avoid: local:plugins/my-super-cool-plugin.

Yeah, could you please expand on that a bit more? What were the issues you are having and how does this fix it? Etc..

It provides easier access to plugins options if plugin is named. I found it cumbersome to extract the plugin options from the service when developing plugins. poi has this solved nicely by providing directly plugin options as an argument, so this was inspired by it.
I wasn't having any issues, I just wanted to make things a bit more convenient for plugin developers.
This doesn't have to be merged, it's just my proposal, although PR has some neat tests that cover existing functionality.

It provides easier access to plugins options if plugin is named. I found it cumbersome to extract the plugin options from the service when developing plugins. poi has this solved nicely by providing directly plugin options as an argument, so this was inspired by it.

Yeah, we can provide the pluginOptions as an argument. But, it should be done all kinds of plugins, not only local ones.
It still doesn't change the fact that we don't have to add support for name for local plugins. The code can be changed:
-      const options = pluginOptions[name] || {}
+      const options = pluginOptions[id] || {}

it still doesn't change the fact that we don't have to add support for name for local plugins. The code can be changed

I see your point. I'll update.

@pksunkara Closing this one. I'll make a new PR.


Use MultiTeamAuthorization when making slack api requests without an event

I would like to make a slack API request while using the MultiTeamAuthorization middleware as we are using token rotation, is that possible?

Hey @Moh-inc 👋 API requests are possible without an event for context, even with MultiTeamAuthorization, as long as the enterprise ID and team ID are known. With these IDs you can collect and set a bot token to make requests:
AppConfig config = new AppConfig();
InstallationService installationService = new FileInstallationService(config);

App app = new App();
app.use(new MultiTeamsAuthorization(config, installationService));
app.client().chatPostMessage(ChatPostMessageRequest.builder()
        .token(installationService.findBot("E0123456", "T0123456").getBotAccessToken())
        .channel("C0123456")
        .text("greetings 👾")
        .build());

The enterprise ID can also be null for single workspace teams and a similar method exists to gather the user token with .findInstaller(enterpriseId, teamId, userId).getInstallerUserAccessToken(). Hopefully this helps, but please let me know if you have any other questions!

@zimeg
Follow up question, when making the request, would this run the chatPostMessage through the MultiTeamsAuthorization middleware? I tried tracing the code, but couldn't find an indication that it would run the request through the middleware and hence refresh the token if needed. Another indication of that is that in the above it is passing the token from the installation service.

Great catch! Since this API call doesn't come with a Request from Slack, the middleware isn't being run. From what I can tell, this refresh logic might have to be reimplemented before making the API call to make sure the token is rotated:
https://github.com/slackapi/java-slack-sdk/blob/6a285c35214540f381a6870ddbff5312d9f61b1c/bolt/src/main/java/com/slack/api/bolt/middleware/builtin/MultiTeamsAuthorization.java#L131-L150
There's additional logic to rotate the user token and request for installations also included in that file, as well as error handling and context updates for requests. Feel free to use what you need, though I would suggest checking the status of app.client().authTest(r -> r.token(botToken)).
Thanks for diving in and pointing this out! 👏

@Moh-inc Making such a change could bring behavior changes to the existing code, so we hesitate to make the modification. Also, all Bolt middleware are supposed to be executed only when receiving an incoming request (e.g., Events API, interactivity with buttons and modals) from Slack. Thus, in general, running a middleware without the request data does not work well.
If your goal is to run token rotation logic before making an API call, please reuse only the relevant code within the built-in middleware (meaning the combination of the token rotator untility and installation store) for the use case.
I hope this helps.

Btw in case anyone has the use case where they would like to force refresh tokens, I took this test code from the slack java bolt code base to force refresh the tokens every once in a while
https://github.com/slackapi/java-slack-sdk/blob/6a285c35214540f381a6870ddbff5312d9f61b1c/bolt/src/test/java/test_locally/app/MessageShortcutTest.java#L54


uninitialized constant #Class:Generative::Forwardable

Hi,
I'm seeing this error when running the generative Rake task. We're using Rails 7.0.1 on Ruby 3.1.0, but downgrading both to earlier versions has the same issue.
For me, the fix was just to require 'forwardable' in generative.rb
An error occurred while loading generative.
Failure/Error: extend Forwardable

NameError:
  uninitialized constant #<Class:Generative>::Forwardable

      extend Forwardable
             ^^^^^^^^^^^
# /home/mpd/.rvm/gems/ruby-3.1.0/gems/generative-0.2.4/lib/generative.rb:8:in `singleton class'
# /home/mpd/.rvm/gems/ruby-3.1.0/gems/generative-0.2.4/lib/generative.rb:7:in `<module:Generative>'
# /home/mpd/.rvm/gems/ruby-3.1.0/gems/generative-0.2.4/lib/generative.rb:3:in `<top (required)>'

An error occurred while loading generative.
Failure/Error: extend Forwardable

NameError:
  uninitialized constant #<Class:Generative>::Forwardable
# /home/mpd/.rvm/gems/ruby-2.7.2/gems/generative-0.2.4/lib/generative.rb:8:in `singleton class'
# /home/mpd/.rvm/gems/ruby-2.7.2/gems/generative-0.2.4/lib/generative.rb:7:in `<module:Generative>'
# /home/mpd/.rvm/gems/ruby-2.7.2/gems/generative-0.2.4/lib/generative.rb:3:in `<top (required)>'

@xxx Would you be comfortable offering a PR with that change, perhaps first adding a failing test that "runs rake task and has problem"?

I can make a PR easily enough, but there won't be a failing test, because this problem occurs at load time, so no tests will run.

edit: This has already been fixed in the master branch, over 5 years ago.
This release a new version.


TypeScript examples

Since Vis Network has official TypeScript support now there should be examples in TypeScript to help people get started.

Hi @Thomaash, I would like to contribute in this project, I have created a little Angular project with one example of custom nodes and custom edges, however, I have an error related with include vis-util, vis-data and moment. Should it be necessary to include those dependencies in my project? Those dependencies are included in the vis-network library, could you help me, please?
My repository is https://github.com/jrodva/vis-network-ts and my current error is:

Hi @jrodva,
please, switch to standalone (CSS, Util and Data are bundled, TS declarations are not) or peer (CSS, Util and Data have to be loaded separately) build for examples. The legacy one is kept only for backwards compatibility, has issues and we don't want to encourage people to use it hence https://visjs.github.io/vis-network/examples/network/basic_usage/legacy.html saying: Please don't use this. Eventually it will be deprecated and removed entirely.
Regarding the error it is always necessary to include the dependencies because the declarations reference them. My attempts to bundle the types into a neat single file so far always failed.

PS: You can import like import { DataSet, Network } from "vis-network/standalone"; the /esm/vis-network part is optional now.

Thank you @Thomaash. I will try with those recommendations.

@tomap I kept your instructions(my repository is updated) and I had another different error:

But I solved updating my project to Angular8, because it was necessary at least Typescript 3.5 😄

Now I have the base ready, what are the next steps to include this project like a Typescript example? I can create more examples in this project and add tests if it's necessary.

To be honest I have no idea. All examples so far were simple pages simply placed among examples. There were never any build steps etc.

So after checking it out I found out it loads it's files from the root of the server it's started on not relatively. In other words as it is at the moment it can't be deployed on GH Pages at all not mentioning coexisting with other projects.
Also clicking the add new node button more than once causes errors.
And one last thing I wouldn't call this TypeScript example but rather Angular example. When I opened this I meant pure TypeScript without any framework.

I have just fixed the error with the add new node button, thank you for your Q&A. Now, I got it what you wanted to say, in the future I will create a Typescript project without any framework.

Thanks and just to note I'm not opposed to having Angular examples. As long as people use it's great to have some examples to have them started. It's just not what I meant when I opened this issue.
I'll have to think of a way to include this in the examples. I'd like to do it in a way that people could simply open it without installing and building anything just like with the other examples.

Ok, don't worry. I think that https://stackblitz.com/ is a good option, even it's possible to include easily it in any web https://stackblitz.com/docs#embedding.

I created Typescript examples and raise a pull request: https://github.com/visjs/vis-network/pull/617. I am using React + Typescript and created placeholder to add more examples. Please review and advise.

I have managed to progress with this stuff in Angular 8 as described in #679. Unfortunately there is some problem with Canvas I can't solve without community help

+1

I'm also a TypeScript developer trying to install with NPM:
npm install vis-network
when i try to compile it gave errors about vis-data not being found.
So then I installed it:
npm install vis-data
I'm trying to import like this:
import { Network, DataSet, Node, Edge, IdType } from 'vis-network/standalone';
but I just keep getting compile errors like this:
[INFO] ERROR in [at-loader] ./node_modules/vis-network/declarations/network/Network.d.ts:19:50
[INFO]     TS2307: Cannot find module 'vis-data' or its corresponding type declarations.
[INFO]
[INFO] ERROR in [at-loader] ./node_modules/vis-network/declarations/entry-standalone.d.ts:2:23
[INFO]     TS2307: Cannot find module 'vis-data/esnext' or its corresponding type declarations.
[INFO]
[INFO] ERROR in [at-loader] ./node_modules/vis-network/declarations/entry-standalone.d.ts:4:42
[INFO]     TS2307: Cannot find module 'vis-data/esnext' or its corresponding type declarations.

anyone ever figured out how to get TypeScript actually working ?

Hi @Clay-Ferguson,
this is very offtopic, could you create a separate issue to track this, please?
Thanks.

This looked like the topic for why npm install vis-network doesn't successfully install for TypeScript. Anyway, sure I'll create new topic. thanks!

@Clay-Ferguson What's the link to the new issue?


Provide support for 32 bit raspberry PI and docker on 32 bit systems

Bug description
Cannot generate client on Raspbian with docker.

How to reproduce
Prisma schema

generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider = "postgresql"
  url      = "postgresql://kitcat:admin@postgres:5432/kitcat?sslmode=disable"
}

model GuildSettings {
  id             String   @id
}

// more left out

Dockerfile

FROM node:15-alpine

WORKDIR /usr/kitcat/bot

# Dependencies for node-gyp
# RUN apk add g++ make python

COPY package*.json ./

RUN npm i

COPY prisma .

RUN npx prisma generate

COPY . .

CMD npx tsc && npm run start

This is run as a service in docker-compose.
When building the images this error gets thrown.
Error: Get config Error: Command failed with Unknown system error -8: /usr/kitcat/bot/node_modules/@prisma/cli/query-engine-linux-musl cli get-config --ignoreEnvVarErrors
Note that I do not use an env file if that matters.
Environment & setup

OS:  Raspbian
Database:  Postgres
Node.js version:  15-alpine (docker)
Prisma version: latest

We don't have precompiled binaries for 32 bit OS like Raspberry Pi OS so this is kind of expected. Since we closed https://github.com/prisma/prisma/issues/861 when we released arm64 binaries I would keep this open to track support for 32 bit PI.

At the current moment, we are not adding 32bit target in the official release pipeline.
But for Pi and ARMv7, I have put together this repository which has precompiled 32 bit engines for these platforms and you can use prisma via this in the meantime: pantharshit00/prisma-rpi-builds
This will build the engine on every stable release.

https://github.com/ImBIOS/prisma-armv7-builds


Fix yaml parameters

On master the yaml paramters are not loaded correctly and the default value is always used

No real preference, as long as we are consistent. Having some way for different airframes to have different configurations would be nice too, without requiring different branches of code. Auto-generated yaml files are one option, or something using rqt-reconfigure, but mostly I'd like it to be  consistent.

@nicovanduijn  I can move them to the dynamic reconfigure

I'm not the biggest fan of dynamic reconfigure in terms of making real-world software, but it does come in handy for debugging. So let's go ahead and have all parameters in dynamic reconfigure

@nicovanduijn  @jkflying  How do you feel about having all the parameters in the dynamic reconfigure and load the vehicle specific configuration from a yaml file in the launch script
<node name="dynparam" pkg="dynamic_reconfigure" type="dynparam" args="load local_planner_node $(find local_planner)/cfg/params_vehicle_1.yaml">
  </node>

I would like that!

Since we are looking at paramters we could transform all these numbers around the codebase into paramters
https://github.com/PX4/avoidance/blob/master/local_planner/src/nodes/star_planner.cpp#L62
https://github.com/PX4/avoidance/blob/master/local_planner/src/nodes/star_planner.cpp#L65
https://github.com/PX4/avoidance/blob/master/local_planner/src/nodes/star_planner.cpp#L71-L72

@nicovanduijn @jkflying are you good with the changes?


Explanations for "Strings can be tricky sometimes!" are incomplete

The main issue is that they do not address the third snippet at all, "a" * 20 vs "a" * 21.
I'm also very curious to know /why/ it's the case that

Strings that are not composed of ASCII letters, digits or underscores, are not interned. This explains why 'wtf!' was not interned due to !.

If it were just non-ASCII, that would make some sense, but what problems could interning a string with an exclamation point cause?

You can read This SO post for details. It covers everything you need to know.

The existence of an SO post doesn't make the explanation section isn't incomplete. The main issue is that it does not address the third snippet at all, "a" * 20 vs "a" * 21.

@jseakle I completely agree with you. Even I noticed this too when I was reading the explanation portion. Then I searched on SO and found the post that I posted in my earlier comment.
Someone really have to improve the explanation for this portion.

+1, I was looking for the same explanation :) Would @satwikkansal be able to add a few words about this? Would a PR make it easier?

Alright guys, really sorry for the ignorance. I've fixed this up in  a0100a8  Let me know if things are still unclear.

@satwikkansal Just submitted a PR about this :) Should I delete it? I think it would be good to mention that is should not be used for value comparison.

If it were just non-ASCII, that would make some sense, but what problems could interning a string with an exclamation point cause?

A look at the source code shows that this is defined here used here to decide if a string should be interned or not.

@sheljohn oops, I didn't knew that, I'm having a look at your PR (y)

Should I delete it? I think it would be good to mention that is should not be used for value comparison.

@sheljohn left a few comments at the PR. I guess maybe we should only go with the typo (fullstop) change for the time being.


Creating navbar

*Hey s for the project I wanna work on this
Creating a navbar

with image
and links

Also making it responsive for various screen
Please assign me this issue I wanna work on it
Under Gssoc'24

@RituKumari98  Can you please assign me this task for Creating a navbar. Please assign me this issue I wanna work on it Under Gssoc'24.

assign this to me please

Hey @RituKumari98
I have worked on the similar project in a hackathon
repo link:-https://github.com/harshcodesit/AIETM-Hackathon-2024
I can create an immersive Navbar

Hello sir @RituKumari98   I would like to work on this issue. Kindly assign this task to me .


Update Line 2

The title just said Continuous Integration whereas the blurb below says Continuous Integration and Continuous Deployment.

<>|<|>
<>|<|>
Why:
Closes [issue link]

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

<>|<|>
<>|<|>
@theblckswan
@blckswan101

Quoi de neuf ?
Le sam. 17 sept. 2022 à 16:09, blckswan-ungtblod @.***>
a écrit :

The title just said Continuous Integration whereas the blurb below says
Continuous Integration and Continuous Deployment.
<>|<|>
<>|<|>
Why:
Closes [issue link]
What's being changed (if available, include any code snippets,
screenshots, or gifs): Check off the following:

I have reviewed my changes in staging (look for the "Automatically
generated comment" and click the links in the "Preview" column to view your
latest changes).
For content changes, I have completed the self-review checklist
https://github.com/github/docs/blob/main/contributing/self-review.md#self-review
.

Writer impact (This section is for GitHub staff members only):

This pull request impacts the contribution experience

I have added the 'writer impact' label
I have added a description and/or a video demo of the changes
below (e.g. a "before and after video")

<>|<|>
<>|<|>
@theblckswan https://github.com/theblckswan
@blckswan101 https://github.com/blckswan101
You can view, comment on, or merge this pull request online at:
https://github.com/github/docs/pull/20677
Commit Summary

5dafe73
https://github.com/github/docs/pull/20677/commits/5dafe7370196452b889d0523c4ae07776e130491
Update Line 2

File Changes
(1 file https://github.com/github/docs/pull/20677/files)

M content/actions/guides/about-continuous-integration.md
https://github.com/github/docs/pull/20677/files#diff-e6058b026bb9e2b5f296659ab80ffb5266583e44082720d624598e5b0d614b0b
(2)

Patch Links:

https://github.com/github/docs/pull/20677.patch
https://github.com/github/docs/pull/20677.diff

—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/20677, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AW5ANHZSER3HKPWUIQCW4RLV6XGKVANCNFSM6AAAAAAQPAVU5Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Store bin size in dict file

Store the bin size in dict file for uploading purposes (it does not make sense to use a different binning size as the one with which the experiment was made)

Is this to store the bin size in GNPS-created experiments?

Yes, if we update that in the GNPS-MS2LDA workflow, then the user doesn't need to indicate that upon uploading into the MS2LDA web app....


Property 'lock' does not exist on type 'ScreenOrientation'.

🔎 Search Terms
N/A
🕗 Version & Regression Information

This changed between versions 5.1.6 and 5.2.2

⏯ Playground Link
https://www.typescriptlang.org/play?#code/O4SwdgJg9sB0DOBjATgU1WWVkgwFwEM8QpMAbKRAawAoAiAB2z2QJDwFoGcBbA5AJ50AlAG4gA
💻 Code
window.screen.orientation.lock("portrait-primary");

🙁 Actual behavior
Property 'lock' does not exist on type 'ScreenOrientation'. ts(2339)

Which is wrong because it does exist per MDN (and was also working before the version update): https://developer.mozilla.org/en-US/docs/Web/API/ScreenOrientation/lock
🙂 Expected behavior
Compiler doesn't complain.
Additional information about the issue
Interestingly enough, unlock is still present on ScreenOrientation and doesn't produce this error.

Duplicate of https://github.com/microsoft/TypeScript-DOM-lib-generator/issues/1615.


Divide Effect Behavior System

Before this change, each SpawnableType had one mega-system to handle all behaviors. I don't this was a good way of handling behaviors because each singular behavior system had to run serially. Making one system for each behavior enables bevy to parallelize the behavior logic.

Remove effect behavior mega-system
Add new private systems for each EffectBehaviorType
Add EffectPlugin that contains all of the effect related ecs stuff

EffectBehaviorPlugin
SpawnEffectPlugin

Add new EffectBehaviorType, FadeOutAndDespawnAfterAnimation

Despawns effects after the animations is complete while also fading out with an exponential decay

Fix bug where full animations were not being played (sorry @LordDeatHunter 😅)

aws --profile shivaji-nonroot s3 cp --dryrun --recursive  s3://assets-thetawave/free_assets/ assets/


too many arguments to return have (string) want ()

when try to run the terraform command to execute terraform code through GOLANG and want to return the id of the resource from the function and use it in other function. Kindly get through the code i am using.
func TestTerraformDataFactory(t *testing.T) {
terraformOptions := &terraform.Options{
TerraformDir: *terraformDir ,
VarFiles: []string{*varFiles},
}
terraform.InitAndApply(t, terraformOptions)
id := terraform.Output(t, terraformOptions, "id")
return id
}
func getDiagnosticSettings() insights.DiagnosticSettingsResource {
ctx := context.Background()
diagsettingsClient:= getDiagsettingsClient()
id := TestTerraformDataFactory()
diagnosticSettings, _ := diagsettingsClient.Get(ctx, id, *diagSettingsName)
return diagnosticSettings
}

@mozehgir

Hi @manya0393 I do not quite get your point. But should this issue belong to terraform rather than go sdk?
In addition, I suppose the code snippet you gave cannot compile. You have a function func TestTerraformDataFactory(t *testing.T) but called it as id := TestTerraformDataFactory().


Interface 'PromiseExtendedConstructor' incorrectly extends interface 'PromiseConstructor'

dexie version 3.0.1
I am getting the following error in dexie.d.ts (53,18):
53:18 Interface 'PromiseExtendedConstructor' incorrectly extends interface 'PromiseConstructor'.
  The types of 'all(...).then(...).then' are incompatible between these types.
    Type '<TResult1 = any, TResult2 = never>(onfulfilled?: ((value: any) => TResult1 | PromiseLike<TResult1>) | null | undefined, onrejected?: ((reason: any) => TResult2 | PromiseLike<TResult2>) | null | undefined) => PromiseExtended<...>' is not assignable to type '{ <TResult1 = any, TResult2 = never>(onfulfilled?: ((value: any) => TResult1 | PromiseLike<TResult1>) | null | undefined, onrejected?: ((reason: any) => TResult2 | PromiseLike<TResult2>) | null | undefined): Promise<...>; <TResult1 = any, TResult2 = never>(onfulfilled?: ((value: any, extraParameters: any) => TResult...'.
      Types of parameters 'onfulfilled' and 'onfulfilled' are incompatible.

@alabaki  have you resolved this error?

What version of typescript?

@dfahlander v3.7.5

@celethil Yes I managed to solve it. It turns out that there was another package in my project that was overwriting the signature of the PromiseConstructor. I just removed that package and it worked. Sorry I forgot to close this issue, will do so now.


Fix oauth SARs for interactive login

Doesn't work unless resource name is plural form
The "group" property is called "resourceAPIGroup" in SAR

"to avoid confusion with the 'groups' field when inlined"[1]

[1] https://docs.openshift.com/container-platform/4.14/rest_api/authorization_apis/subjectaccessreview-authorization-openshift-io-v1.html

Evidence of testing is found on STF-1799

Testing
Deploy from this branch and run smoke tests
$ oc new-project service-telemetry

$ git reflog -1
c5197f2 (HEAD -> csibbitt/STF-1799_fix_interactive_login_sars, origin/csibbitt/STF-1799_fix_interactive_login_sars) HEAD@{0}: commit (amend): Fix oauth SARs for interactive login

$ cd build; ansible-playbook --extra-vars __local_build_enabled=true --extra-vars working_branch="$(git rev-parse --abbrev-ref HEAD)" --extra-vars __service_telemetry_snmptraps_enabled=true --extra-vars __service_telemetry_storage_ephemeral_enabled=true --extra-vars __service_telemetry_observability_strategy=use_redhat -e pull_secret_registry="dummy" -e pull_secret_user="dummy" -e pull_secret_pass="dummy" ./run-ci.yaml

[...]
TASK [stf-run-ci : Show the result of the validate_deployment script] *********************************************************************
ok: [csibbitt-thinkpadp16vgen1.rmtcaon.csb] => {
    "validate_deployment.stdout_lines[-1]": "* [info] CI Build complete. You can now run tests."
}

[...]

PLAY RECAP
**********
csibbitt-thinkpadp16vgen1.rmtcaon.csb : ok=140  changed=38   unreachable=0    failed=0    skipped=32   rescued=0    ignored=1

$ ../tests/smoketest/smoketest.sh
[...]
*** [SUCCESS] Smoke test job completed successfully

Add new "stfviewer" user according to [1]
$ oc get secret htpasswd -ojsonpath={.data.htpasswd} -n openshift-config | base64 --decode > users.htpasswd

$ htpasswd -bB users.htpasswd stfviewer qwerty123

Adding password for user stfviewer

$ oc create secret generic htpasswd --from-file=htpasswd=users.htpasswd --dry-run=client -o yaml -n openshift-config | oc replace -f -

secret/htpasswd replaced

Create a new "stfgetter" role with all of the STF permissions and bind it to the user
$ oc create role stfgetter -n service-telemetry --verb=get --resource=prometheuses.monitoring.rhobs,alertmanagers.monitoring.rhobs,grafanas.grafana.integreatly.org

role.rbac.authorization.k8s.io/stfgetter created

$ oc adm policy add-role-to-user stfgetter stfviewer --role-namespace service-telemetry

role.rbac.authorization.k8s.io/stfgetter added: "stfviewer"

Then I deployed Grafana and enabled dashboarding as described in the docs[2].
Confirmed that I could log in with the "stfviewer" user into these interfaces:

default-prometheus-proxy
default-alertmanager-proxy
default-grafana-route

[1] https://docs.openshift.com/container-platform/4.14/authentication/identity_providers/configuring-htpasswd-identity-provider.html#identity-provider-htpasswd-update-users_configuring-htpasswd-identity-provider
[2] https://docs.redhat.com/en/documentation/red_hat_openstack_platform/17.1/html-single/service_telemetry_framework_1.5/index#dashboards_assembly-advanced-features

Not sure what's wrong with CI, but they are all failing with "Task Set host ip address for crc.testing failed running on host crc" long before they run my code.

recheck

hi, let me do a fix for the dnsmasq for old CRC version

This should help to pass the CI: https://review.rdoproject.org/r/c/config/+/53821

recheck

recheck

recheck

recheck


emacs: Add diff-hl faces based on diff faces

I'm closing this in favor of the templates in the new repo (belak/base16-emacs)


Think about to fix the issues found by adapter checker

I am an automatic service that looks for possible errors in ioBroker and creates an issue for it. The link below leads directly to the test:
https://adapter-check.iobroker.in/?q=https://raw.githubusercontent.com/misanorot/ioBroker.viessmann

[ ] [E802] No topics found in the repository. Please go to "https://github.com/misanorot/ioBroker.viessmann" and press "Manage topics"

Thanks,
your automatic adapter checker.
P.S.: There is a community in Github, which supports the maintenance and further development of adapters. There you will find many experienced developers who are always ready to assist anyone. New developers are always welcome there. For more informations visit: https://github.com/iobroker-community-adapters/info

Done

Thanks, that all bugs have been fixed.


Render and read the asterisk on required fields more accessibly

As per WCAG, meaningful content should not be inserted using CSS pseudo elements. This PR moves the asterisk indicated a required field into a  within the label element. I've also added an aria label to it so that screenreaders will read "required" instead of "star" or "asterisk" when reading it out.
I'm making this as a draft because currently the new asterisk is a separate readout from the label, requiring an extra tab to navigate through, and i'd like to improve that.
UPDATE: upon further research this is how screen readers are expected to work, and so I'm marking this as ready for review.

 Manual a11y review and fix for TSB Form


timeouts and extant callbacks, forced failures and lying stack traces

(edited for clarity)
Test cleanup seems to be a weak spot; I've got some very timing-sensitive tests that -- when they fail -- cause subsequent tests that should pass to fail, with stack traces that point to the wrong issue.  I'm trying to make sure I'm not missing some obvious way of handling these.
Let's say I have these tests in a single suite:
test("A", function(done) {
  this.timeout(1000);
  setTimeout(function() {
    assert(false);
    done();
  }, 2000);
});

test("B", function(done) {
  this.timeout(10000);
  setTimeout(function() {
    done();
  }, 3000);
});

A will timeout and fail.  B will start, and though it would pass, A's timer will fire and A's assertion will cause B to fail (with a stack trace that references A).
The only way I've found to get around this (and the problem exists for any delayed event in a test, whether that test times out or finishes in a separate test branch) is to do something like the following for A:
test("A", function(done) {
  var _runnable = this.runnable();
  this.timeout(1000);
  setTimeout(function() {
    if (!_runnable.timedOut && !_runnable.duration) {
      assert(false);
      done();
    }
  }, 2000);
});

.. is that the preferred method of handling this issue?

Just tested and it's still an issue:
$ cat test.js
var assert = require('assert');

it("A", function(done) {
  this.timeout(1000);
  setTimeout(function() {
    assert(false);
    done();
  }, 2000);
});

it("B", function(done) {
  this.timeout(10000);
  setTimeout(function() {
    done();
  }, 3000);
});

$ ./bin/mocha test.js

  ․․

  0 passing (2s)
  2 failing

  1)  A:
     Error: timeout of 1000ms exceeded. Ensure the done() callback is being called in this test.
      at null.<anonymous> (/Users/danielstjules/git/mocha/lib/runnable.js:170:19)
      at Timer.listOnTimeout [as ontimeout] (timers.js:110:15)

  2)  B:
     Uncaught AssertionError: false == true
      at null._onTimeout (/Users/danielstjules/git/mocha/test.js:6:5)
      at Timer.listOnTimeout [as ontimeout] (timers.js:110:15)

Your solution would work, but I think it's a symptom of a larger a problem.

Blast from the past.  Forgot this was filed, but it's still an issue.
~/tmp $ cat ./test.js
var assert = require('assert');

it("A", function(done) {
  this.timeout(1000);
  setTimeout(function() {
    assert(false);
    done();
  }, 2000);
});

it("B", function(done) {
  this.timeout(10000);
  setTimeout(function() {
    done();
  }, 3000);
});

~/tmp $ ./node_modules/.bin/mocha --version
4.0.1
~/tmp $ node --version
v8.6.0
~/tmp $ ./node_modules/.bin/mocha test.js

  1) A
  2) B

  0 passing (2s)
  2 failing

  1) A:
     Error: Timeout of 1000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves.

  2) B:

      Uncaught AssertionError [ERR_ASSERTION]: false == true
      + expected - actual

      -false
      +true

      at Timeout._onTimeout (test.js:6:5)

~/tmp $

I'm not sure this is actually fixable; in general, it's pretty hard (even moreso the wider the range of environments supported) to programmatically determine which function started a given piece of asynchronous code that threw an exception.

.. which is fine, I just don't want it closed by a bot.

Fair enough! (The bot went a little overkill on old issues as we're updating some labels and it filters based on labels. Sorry about that.)
It occurs to me that this would be a great use case for cancellation semantics -- stopping actions if a timeout is hit... Obviously, some actions have timeouts chosen by the caller of the action, but the lack of a standard way to allow a consumer to control it makes it difficult or impossible to integrate any arbitrary API's cancellability with something like Mocha, as far as I've heard. (This philosophical musing brought to you by my seeing people discuss things like promises but with corrections for common errors and obscure edge cases.)

If it's not clear, I'm not asking for people to focus on this issue -- this is a hard problem (as in "possibly unsolveable at the moment," not "difficult").  I'm 100% behind a decision to close the issue, but it would be nice to document it, and document a work-around (which -- I think -- is something like what showed above).

Adding to this for posterity -- I ran into this issue while designing testing for selenium. If a test times out while an await is pending, mocha will grab its neck and fall over when the promise resolves. I worked around this by wrapping my test functions in this function, based on @aahoughton's workaround. As far as I can tell, this solves the weird async issues entirely, as well as across multiple runs when invoking mocha programatically with cleanReferencesAfterRun turned off.
I don't pretend it's perfect -- it's very possible that there's a way to confound this fix, but it seems to work for my needs.
// if an async returns after the method times out,
// it will cause a "done() called multiple times" error.
// In the context of a running webdriver, this is potentially very bad, since
// we need to be able to quit the driver gracefully under any circumstance.
// This allows methods to time out while an async is pending,
// and will avoid calling done twice in this case.
function asyncSafety ( fn ) {
	return function ( done ) {
		let runnable = this.runnable();

		fn.bind( this )().then( ( res ) => {
			// for successful I think we only need timedOut? not sure though, might have side effects
			// when the duration check is added it will get stuck and never complete on a second run-through
			if( !runnable.timedOut ) {
				done( res );
			}
		}).catch( ( err ) => {
			if( !runnable.timedOut && !runnable.duration ) {
				done( err );
			}
		});
	}
}

Here's how it might be used:
describe( 'foo', function () {
	it( 'should not die on async weirdness', asyncSafety( async function () {
		await someAsyncMethod();
	}));
});

Yeah, this seems like a pretty big structural change as Scott mentions. Plus it hasn't received very much enthusiasm from users in the > decade since it was posted. Closing as wontfix / aged away.
If someone wants something like this, please file a new issue with our new templates that describes in detail what should be changed/improved.
Thanks all!