Plot a subset of points

For huge ARGs, like the sc2ts ARG (1.4 million nodes), we can only display a small subset of nodes in a viz like this. Nevertheless, that can be extremely useful (see the last set of supplementary figures at https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4212v1). It would be really useful to be able to specify a subset of points to plot.
There are also various ways you could imagine creating a subset of points to plot by expanding outwards from a smaller number of nodes (e.g. take 2 samples, expand to find their common ancestor(s) either in a specific tree or in all trees. It is unclear to me if something like this would be built-in to a visualiser, or be a separate routine to create a set of nodes, which then get passed in.

If you wanted to have the whole ARG in the background but to focus on a select set of nodes/edges, something quick like this could work:

I can add a parameter to pass a list of nodes of interest into the drawing function. For the above ARG, I passed subset_nodes=[0,1,2,8,10,11,14,15]. If both the edge's parent and child are in the list, then the edge gets colored. This method doesn't solve the problem when there are too many nodes to coherently display. Most useful if you wanted to focus on a specific region without losing the broader context.
We previously talked about the collapsible force layout for interactively showing a subset of nodes. This method is generally used with trees rather than graphs. If we want to apply this to the ARG, we would need to determine how we want the graph to collapse, especially around recombination nodes which add a unique challenge. Another snag is that I am currently setting the y-axis position of the nodes in Python before sending everything to D3 for plotting. The collapsible force layout would require dynamic positioning so would need to be handled in a different way. I'll continue to look into this!

Just returning to this after thinking about huge ARGs. I wonder if a neat thing would be to pass in the tree sequence and a single node ID, and to be able to right-click (or equivalent) on that node to be able to add connected nodes into the viz, and so on, building up a graph viz by expanding outwards.
How doable might this be @kitchensjn ? Would it help in your case of visualising parts of large tree sequences?

I've been thinking about a single node view similar to as you describe but without the interactivity. The biggest hurdle to work through with the visualization you described is updating the y-position of the nodes once more nodes have been included in the graph; this would all need to be done in JS rather than Python.

So I've taken another stab at this and think I've come up with something. Very much based off of the figures in the sc2ts paper, I've added a draw function where you select a node and a number of degrees of connections and it draws the local connections around that node.

Here, I've selected the recombination node 29/30 and degree 3, so nodes that are above or below in three connections or less. One nice feature from the sc2ts paper that is missing here are the short dashed connections to nodes not included in the subset. I'm working out a way to include those now.
Importantly, this is not any node within three degrees of the focal node, only those that are above and below; this may not be the most convenient way to define this but was easiest to first implement.

Neat. Thanks for this @kitchensjn. I will have a play next week. It would be interesting to see if I can roughly reproduced the sc2ts plots.

Lastest commit includes this new function with some documentation (cedc206).
def draw_node(
    self,
    node,
    width=500,
    height=500,
    degree=1,
    y_axis_labels=True
):
"""Draws a subgraph of the D3ARG using D3.js by sending a custom JSON object to visualizer.js

Parameters
----------
node : int
    Node ID that will be central to the subgraph
width : int
    Width of the force layout graph plot in pixels (default=500)
height : int
    Height of the force layout graph plot in pixels (default=500)
degree : int
    Number of degrees above and below the central node to include in the subgraph (default=1)
y_axis_labels : bool
    Includes labelled y-axis on the left of the figure (default=True)
"""

This function is very similar to the standard draw(), but you need to provide a node ID which will be the center of the subgraph. At the moment, it also doesn't have quite as many optional styling parameters; graphs are always drawn with tree_highlighting=False, y_axis_scale="rank", edge_type="line", variable_edge_width=False, and include_underlink=True.

Also, I added the missing edges

Dashed lines coming into the top of the node are for missing parent edges and below is missing child edges. The number at the end of the dashed line tells you how many edges of that type are missing for each node. This was a lot easier and hopefully cleaner than trying to position a bunch of these dashed edges around the nodes per missing edge.

You can update the styles of individual nodes and edges with the set_node_styles() and set_edge_colors() functions. These are really just wrappers, styles are stored in pandas.DataFrames so the values could also be updated directly if that would be easier for certain scenarios. This is manual process and does require that you know what nodes/edges should be altered.
import msprime
import random
import json
import tskit_arg_visualizer

d3arg = tskit_arg_visualizer.D3ARG.from_ts(ts=ts)

d3arg.set_node_styles([
    {"id":0,"fill":"pink"},
    {"id":7,"fill":"pink"}
])
d3arg.set_edge_colors({
    0:"pink",
    7:"pink"
})

d3arg.draw_node(29, degree=6)

Latest commit (cf69820) adds y_axis_scale parameter to draw_node(). Same as in the original draw(), you can plot using "rank", "time", or "log_time".

Funky. Thanks @kitchensjn !

Another simplification concept that may be relevant to @russcd as well. Here, the top image is the original ARG on a log_time scale and the bottom image is the corresponding discretized time ARG. I started with a set of time bins, grouped all connected nodes that fall into the same time bin into a single node, then updated the edge table to account for all of the newly collapsed nodes. In the bottom image, the collapsed nodes are shown in white and the uncollapsed are the standard green. This collapsed version of the ARG cannot be stored as a tskit.TreeSequence because the collapsed nodes don't play nicely with the edge interval paradigm of tskit (more generally it isn't suitable for calling individual trees), but it does simplify the structure in an algorithmic way. I could see this working interactively as well, say as you zoom in the time bins get smaller so more nodes/connections become visible. My spaghetti code does all of the simplifications in Python given a set of bins then passes it to JS, so binning is a static property. Giving users the ability to Zoom would require this to be done in JS, Pyodide, or on a backend server.

Presumably we can close this issue now, and maybe reopen a different one about drawing "summary" nodes like the version you illustrated in the comment above?

Cool, happy to close and reopen a separate issue for the "summary" nodes as that seems like a big rabbit hole.


Generate derivations with their DataDef and IMod

Is it possible to add a header to a paragraph? I would use it in the derivation sections.
Example,

Are there any instances where we have these headers that are not related to derivations of instance models?

Not in my example

related to #46

Alright, I think we're going to auto-gen the derivation section (Not the derivations themselves, but the header and whatnot, and we can have the contents manually written), so you won't actually need to create that header at this point. If it comes up elsewhere, then we can worry about it.

Your solution sounds good to me Dan, except I think there are examples where there are derivations for general definitions or definitions.  (For example, the general definition in the solar water heating example (SWHS) has a derivation following the general definition for "Simplified rate of change of temperature.")  The reason for the derivation section below the instance model (or other theory or definition) is that the derivation doesn't fit in the description field of the table.  It is definitely like another field for the model, just not in the table.

Yes. I forgot to mention I have derivations in some of my Data Definitions.
Example,

@smiths I think we're on the same page, but I'm not sure exactly what you're trying to get across.
When I said we'd auto-generate the derivations, I meant to be more general (hence no mention of "instance models" or the like), as I figured they'd show up across instance, data def, gen def, etc. and would act exactly as another piece of knowledge to be captured. That way we'd just generate the IM, DD, TM, etc. with the appropriate derivation following it.
This ties into #29 and related issues.

Are there any instances where we have these headers that are not related to derivations of instance models?

I think your earlier comment caused the confusion.

There was a question above on whether the derivation section only followed instance models; I just wanted to point out that this isn't the case.  @njericha just beat me to this explanation, but I'll post my comment anyway.  😄

Related to #409

This is also related to the design issue of having "self-describing" chunks. Right now, we mix the display structure and the more semantic aspects of things too much. We need to 'pull apart' these two aspects.  I couldn't find the issue where this is though.

Derivations can now (commit a13d7ebcb7186173ba3d0fe9ef8a3ce3d2dd8dad) be generated alongside their definitions/models using Attributes.


add dll-app-and-vendor example by @aretecode

What kind of change does this PR introduce?
example

Did you add tests for your changes?
yes

If relevant, link to documentation update:
N/A

Summary
add dll-app-and-vendor example

Does this PR introduce a breaking change?
no

Other information

finally! 😎


How to predict by the trained model

I downloaded the trained model weights, and tried to load them to predict. I picked a image from the dataset (00000322_010.png), labelled pneumothorax, and the predicted probability seemed to be extremely low for all classes. Here is how I ran the prediction:
import cv2
import numpy as np
from keras.models import load_model

model = load_model('./CheXNet-Keras/model.h5') # I saved the model from train.py
model.load_weights('./CheXNet-Keras/brucechou1983_CheXNet_Keras_0.3.0_weights.h5')
im = cv2.imread("./CheXNet-Keras/data/images/00000322_010.png")
im_r = cv2.resize(im, (224, 224))
im_t = im_r.reshape(1, 224, 224, 3).astype('float32') / 255
prediction = model.predict(im_t)

The output of prediction was:
array([[2.0528843e-01, 9.4045536e-04, 3.9193404e-01, 9.2863671e-02,
        1.0580428e-02, 1.1802137e-02, 3.6063469e-03, 1.4762501e-01,
        2.5597181e-02, 1.1413006e-02, 4.9169478e-03, 1.2800613e-02,
        4.7103822e-02, 3.9492457e-05]], dtype=float32)

Any suggestion?

This is a multilabel problem instead of a multiclass problem so the outputs are NOT probability values. You have to set a proper threshold for each of the classes to get the predictions. For example if you want to maximize the f1 score, just loop over the roc curve and find the threshold with largest 2 / ( 1 + 1/tp + fp/tp ).
BTW, I actually don't encourage people to use CheXNet for prediction at this time. CheXNet tells us this task could be done by deep learning. However, if you read the dataset source paper, you know that these labels are extracted by text mining radiological reports. The quality of the label could be considerably low.
Last but not least. ChexNet can't actually do diagnosis because complicated reasons. If you're interested in the detail, Luke's blog is recommended.

@brucechou1983 Does that mean chexnet won't be able to make (decent) predictions / detect with bounding box for foreseeable future?


Balder/mbus transported with headers and body in slime

@vekterli PR
The first commit is what you have review earlier on.
The rest adds more tests for versions and enable the new protocol as of 6.148.
It also corrects a bug in c++ version selection logic.

By the way, our internal version number has been bumped to 6.148.x already. Presumably this won't introduce conflicts as long as none of those internal versions are promoted to release candidates.

I will change this to 149, to make sure we get this tested during upgrade of canary in integration tests.


Forward RTMP Stream to external RTMP Server

Question
Hello, thanks for the project.  I have a question for which I could not find an answer in the docs.
How to forward RTMP stream to external RTMP servers?
Do we need to pipe the stream via ffmpeg to other rtmp servers or rtsp-simple-server has this capability (without ffmpeg)
OBS -> rtsp-simple-server -> google rtmp, twitch rtmp ?
Thank you.

Can the following be used but for RTMP (not RTSP):
paths:
redirected:
source: redirect
sourceRedirect: rtmp://otherurl/otherpath

Hello,

use this configuration:

paths:
  redirected:
    runOnPublish: ffmpeg -re -i rtsp://localhost:8554/redirected -c copy -f flv rtmp://otherurl/otherpath

launch the server

use this URL in OBS

rtmp://rtsp-simple-server-ip/redirected

be careful that you need ffmpeg installed too.

Thanks.


Support unknown ua override

Epic: #61
query param "unknown" = default shows default set when user agent is unknown/not supported. otherwise show nothing

Commit #90


Default value of bidders features on dev-docs

Hi, template for bidders on dev-docs (https://github.com/prebid/prebid.github.io/blob/master/dev-docs/bidders.md) needs to be changed. Where bidders mediaType is defined, display type is added as mandatory for every bidder while not all the bidders has display media type, e.g. AdformOpenRTB.

@bretg Can we close this issue?

Am adding a new option to the media_types header: "no-display". This will remove display from bidders that don't support it. Am combining the change with a new safeframes_ok field - will link to this issue when the PR is submitted.

Ok @Pupis - added a way the "no-display" option to prevent it from appearing on the media type list, updated your bidder.md file, and documented it for others.


Add in flipV on geometry support

Fixes #991.

Add in a flip "V" for geometry load. This is a generally useful option.
The option is enabled for the shaderball which currently loads with V flip UVs.
Tested with boombox which and chessboard which already have the appropriate V orientation so don't have this option enabled.

Ideally, it seems the shaderball's UVs should be flipped always but can be handled when this file is cleaned up
in general (e.g. parenting is incorrect).
Results

@kwokcb It's exciting to see textured rendering working in the Web Viewer, but why would this flip only be needed in JavaScript, and only specifically for the shaderball mesh?  If the problem were specific to the shaderball, then I'd expect the same flip to be required in C++.  And if the problem were specific to JavaScript, then I'd expect it to be required for all meshes, not just this one example.

@kwokcb It's exciting to see textured rendering working in the Web Viewer, but why would this flip only be needed in JavaScript, and only specifically for the shaderball mesh? If the problem were specific to the shaderball, then I'd expect the same flip to be required in C++. And if the problem were specific to JavaScript, then I'd expect it to be required for all meshes, not just this one example.

I did some more investigation and all geometry needs to be flipped to be consistent with desktop, but also all textures need to be flipped as well.
Desktop always flips V for geometry and assumes bottom-to-top for shader and texture orientation.
Web (maybe just Three only) flips textures on load (https://threejs.org/docs/#api/en/textures/Texture.flipY) so is inconsistent.
i.e. Seems to boil down to having a consistent orientation convention for:

Shader code
Geometry
Textures


added pull dependencies in bower.json to steps

Omitting bower install will cause the following
...
[21:25:20] Starting 'copy_dependencies'...
[21:25:20] 'copy_dependencies' errored after 574 μs
[21:25:20] Error: ENOENT: no such file or directory, stat 'bower_components/bootstrap/dist/js/bootstrap.min.js'
...

I signed it!

Google Individual Contributor License Agreement
Thanks, your CLA has been submitted.

Thanks! Merged at https://github.com/google/lovefield/commit/4e7ac8d9fe01260204eb9804d6a12d059b1550d7.


Timestamp filter in errorstore does not work correctly

❗ please do not add sensitive information in issues, you can provide extra information via email using issue number as reference ❗
Describe the issue
The timestamp filter option does not work correctly

if you put in a date only it does not select all records for this date
if you put in a complete timestamp it does not filter

in the old blue
Reporter
JJansenvr

To Reproduce
Steps to reproduce the behavior:

Go to an adapter with a error or message store
Click on Display Filters
Scroll down to fill out the timestamp filter field
See no (or no correct) filtering in the list

Screenshots
Configuration
na

Input
na

Environment

FF! 7.6: ibis4-----
Running on srtapp0000 using IBM WebSphere Application Server/<IP_ADDRESS>
Java Version: Java(TM) SE Runtime Environment (<IP_ADDRESS> - pxa6480sr6fp26-20210216_01(SR6 FP26))
Heap size: 839M, total JVM memory: 1240M
Free disk space: 466MB, total disk space: 2GB
Up since: 2022-01-13 10:14:49 (14d)

Additional Environment

Oracle Database 19c Enterprise Edition Release <IP_ADDRESS>.0 - Production Version <IP_ADDRESS>.0] driver [Oracle JDBC driver] driver version [<IP_ADDRESS>.0]

Browser: [e.g. Chrome, Safari, Edge, Firefox]

Additional Context

Workaround: the date or timestamp in the input field is used in a  > way so if you want to find records from the date 2022-01-07 use 2022-01-06 23:59 and you will be able to find the records

This it indeed the way it is implemented

[ ] codebase pipe messagelog uses a different codebase then receiver error/messagestore
[ ] fix the display columns
[ ] total message count
[ ] add end timestamp field
[ ] add delay to search while typing (add debounce)


Limit of concurrent TraceEventSessions?

If I try to create more than 8 concurrent TraceEventSessions on one machine, I get the Exception "Insufficient system resources exist to complete the requested service. (Exception from HRESULT: 0x800705AA)". It doesn't matter if they are created within one process or in multiple processes.
Where does this limit come from and can it be increased?
I found that Windows 10 has a limit of 64 concurrent ETW Sessions (https://docs.microsoft.com/en-us/windows/desktop/etw/event-tracing-sessions), but there are only ~20 sessions active on my machine. There is also a limit of 8 concurrent SystemTrace sessions (https://docs.microsoft.com/en-us/windows/desktop/etw/configuring-and-starting-a-systemtraceprovider-session) and also 8 concurrent private logger sessions (https://docs.microsoft.com/en-us/windows/desktop/etw/configuring-and-starting-a-private-logger-session), but as far as I can see a TraceEventSession does not create any of those.
Code to reproduce this:
var sessions = new List<TraceEventSession>();
try
{
    for (int i = 0; i < 10; i++)
    {
        var session = new TraceEventSession("SessionTest_" + i);
        sessions.Add(session);
        session.EnableProvider("abc"); // throws for the 9th session
    }
}
finally
{
    foreach (var session in sessions)
        session.Dispose();
}

This is not an limitation in TraceEvent.  It is some limitation in the OS but I don't have details.   I will see if I can find out more.

I suspect that there is a limit has to do with the fact that the session is a real time (not file based), but at this point it is only a guess (you can try changing it to create file-based sessions).

I tried, but I get the same error with file-based sessions.

Two other things to try, is to use a real Manifest-based provider.  I would recommend Microsoft-Windows-PowerShell.     Second, try using Microsoft-Windows-PowerShell for the first 5 providers then switch to Microsoft-Windows-ReadyBoost, and see if you get more than 8.   What I am trying to discover is if it is the number of sessions or the number of sessions per provider that is limited.

I heard back from a query I made to ETW experts and the said

Yes, each individual provider can only be enabled to 8 sessions at a time.

You should confirm this by doing the experiment I mentioned above (enabling differnet providers on the differnet sessions and see if that allows you to create more sessions.
I don't have alot of personal experience with it, but you CAN share sessions (attach to existing sessions).   It means that the set of events being selected has to be uniform, but otherwise it should work.  That may be a useful work-around for you.

From MSDN's About Event Tracing (emphasis mine)
MOF (classic) providers:

Use the RegisterTraceGuids and TraceEvent functions to register and write events.
Use MOF classes to define events so that consumers know how to consume them.
Can be enabled by only one trace session at a time.

WPP providers:

Use the RegisterTraceGuids and TraceEvent functions to register and write events.
Have associated TMF files (compiled into a binary's .pdb) containing decoding information inferred from the preprocessor's scan of WPP instrumentation in source code.
Can be enabled by only one trace session at a time.

Manifest-based providers:

Use EventRegister and EventWrite to register and write events.
Use a manifest to define events so that consumers know how to consume them.
Can be enabled by up to eight trace sessions simultaneously.

TraceLogging providers:

Use TraceLoggingRegister and TraceLoggingWrite to register and write events.
Use self-describing events so that the events themselves contain all required information for consuming them.
Can be enabled by up to eight trace sessions simultaneously.

Yes, I think we found the 'root limitation'.

Thanks! That will require us to find a work-around, but at least we know where the limit is coming from.


Automatically generate use case files through Caze

This PR implements a first version of a generator that could generate files dynamically by running:
$ bundle exec caze generate engine:my_engine:my_use_case_module:my_class

The command above will create 2 new files:
# ->  engines/my_engine/lib/my_engine/my_use_case_module/my_class.rb
# ->  engines/my_engine/spec/my_engine/my_use_case_module/my_class_spec.rb

Please note, that I still need to write some simple documentation for this. My intent on opening this PR was to allow reviewers to start reviewing the code upfront.
Also, additionally this adds a sample Dockerfile and docker-compose.yml, so that those using docker-machine can use this gem using docker.
Any ideas on how to improve the code are welcome.
TODO:

[x] Write doc for this generator
[ ] Write help method
[ ] Add the init params as the commands second params
[ ] Add aliases to the command (g for generate)?
[ ] Populate files with template code
[x] Increase test coverage
[ ] Better error handling, what happens when a file already exists, and if a path doesnt exist?

@philss I think we should be able to merge this already, and I can add the rest of the bullet points highlighted here on another PR, what do you think?

What an awesome idea! Keep the good work guys @luizParreira @philss =)


Deploying an agent through the Azure VM extension

Please include or add a pointer to documentation on how to deploy an agent to an Azure virtual machine by using the available extension for both Windows as well as Linux.
In particular how to set up a Linux environment agent that can be used by a yaml pipeline.

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

ID: f64ca92c-b7d5-3142-71fb-306d32a37281
Version Independent ID: dfef40cf-2f50-4c03-bc97-bafa6c305108
Content: Deploy an Azure Pipelines agent on Linux - Azure Pipelines
Content Source: docs/pipelines/agents/v2-linux.md
Product: devops
Technology: devops-cicd-agents
GitHub Login: @steved0x
Microsoft Alias: sdanie

@cveld  Thank you for the feedback, assigning this to the author for further review.
In the meantime, please refer to the Azure VM extensions documentation.


"Also appears in" going off edge of page

https://www.ebi.ac.uk/ols4/ontologies/iao/properties/http%253A%252F%252Fpurl.obolibrary.org%252Fobo%252FIAO_0000589

need to wrap that div

Well it doesn't go off the page anymore...

but really we need to display the first few ...+n with a button to expand the rest


test: fix CI failing on new Mocha unhandledRejection listener

Fixes #526 🦕

@googleapis/yoshi-nodejs what are test(14)  and test(15)`? I noticed this test was stuck for all previous merged PRs as well


"return expression directly" action removes expression.

Description: RA provides the code action "return expression directly" when having a useless let-binding (variable being returned immediately). Performing the action deletes the actual code (binding + expression) instead of the return statement.
Reproduce:

load example code
place cursor on the line of the return statement (line 4; a)
invoke the code action ("return expression directly")

Expected result: the binding of the expression, the ; after it, and the line returning the variable are removed. Not the binding/expression while keeping the return statement (of a now invalid variable).

Version:

extension rust-analyzer 0.1.20200302 from VSCode extension market
rust-analyzer --version -> rust-analyzer f761ed2

cc @kiljacken I believe this is comming from clippy. Could you check if it's a clippy bug, or if we interpret the edit wrongly?
Our own "inline variable" assist works ok with this case though :)

In this case, clippy includes a two part replacement, and we seemingly only apply the first part. I'll look into it at some point during the week.


Update luxtronik2 to 0.3.1

Please update my adapter ioBroker.luxtronik2 to version 0.3.1.
This pull request was created by https://www.iobroker.dev 15f979c.

Test für dev portal


i18n(ru): added translations of nav and ui items

What kind of changes does this PR include?

Added translations of missing items in

nav.ts
ui.ts

Description
This PR adds translations of some words/phrases.

@delucis thank you for your review! Just fixed your comments.


Considerations for LTE-M or NB-IoT devices?

A web-search about "earthquak" detection systems guided me to this project.
Using WLAN comes in some cases also with pain, e.g. installation.
Did you already consider to build also devices on NB-IoT or LTE-M modems?
In the recent time, they get better and better. Maybe a first approach even doesn't require own hardware, there "Multi-sensor cellular IoT prototyping platform" available, which are already equipped with accelerator sensors and a modem. It's not that cheap as the sensor of this project, but it opens also some new possibilities.
One of those platforms is the Nordic Semiconductor - Thingy:91.

Together with a SIM-Card it starts about 150 Euro of invest.
If there is some interest, I would be happy to help. If started a general small demo project, see CoAP/DTSL 1.2 CID for Zephyr. Together with a Eclipse/Californium server, it demonstrates, how reliable and efficient UDP can be.

This is a great idea, and something we have discussed but never done. I agree that LTE-M would be an important addition to OpenEEW as it would allow placement of sensors in areas without the use of residential wifi or ethernet (which is often a challenge).
@boaks are you offering to help with the firmware ? do you have a Nordic Thingy and an ADXL355 (I suppose we could tet this with a cheaper ADXL345 also?)

The Thingy:91 is equipped with two accelerometer, a ADXL372 and a ADXL362.

A proof of concept with a CoAP server would be great. We would need to demonstrate that it could continously receive 31 samples for each of the 3 channels every second.

CoAP will not be the limit there, but if you send about 31 msgs/s, your SIM card will get expensive and the battery will be a very large one. Such use-cases don't fit LTE-M (not NB-IoT).
Usually LTE-M is for devices, which reports "regularly every couple of hours" an "alive" and on an event an "alert". That requires, that the device accumulates the sensor values and pre-process them, reporting only the values either as "less frequent alive" or if the values are indicating an possible earthquake event.

are you offering to help with the firmware ?

Yes, but it depends on the primary approach. As written above, sending 32 msgs/s is in my opinion no mobile IoT use-case.

What about using LoRa network?

continously receive 31 samples for each of the 3 channels every second.

With LoRa?


Added all plugins for Android and iOS

Closes #48

@Sporiff Ok, I've added the last of the integrations here. As per comments on this ticket, there were some inconsistencies I couldnt figure out in terms of platform support. But let's start with getting this out, and we can always create a follow-up task to look into those.


Fixes issue when release name is different from tag name

Reference ticket #267

Codecov Report

Merging #271 (a4254ef) into main (9c3242e) will decrease coverage by 0.27%.
The diff coverage is 88.00%.

@@            Coverage Diff             @@
##             main     #271      +/-   ##
==========================================
- Coverage   91.30%   91.02%   -0.28%
==========================================
  Files          17       17
  Lines        1576     1672      +96
==========================================
+ Hits         1439     1522      +83
- Misses        137      150      +13

Flag
Coverage Δ

unittests
91.02% <88.00%> (-0.28%)
:arrow_down:

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

Impacted Files
Coverage Δ

setup.py
0.00% <0.00%> (ø)

grayskull/base/base_recipe.py
88.09% <83.33%> (-0.32%)
:arrow_down:

grayskull/pypi/pypi.py
94.60% <85.96%> (-1.55%)
:arrow_down:

grayskull/__main__.py
98.21% <100.00%> (+0.06%)
:arrow_up:

grayskull/cli/parser.py
100.00% <100.00%> (ø)

grayskull/utils.py
100.00% <100.00%> (ø)

grayskull/license/discovery.py
98.28% <0.00%> (+1.14%)
:arrow_up:

Continue to review full report at Codecov.

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


when i start train,get error:munmap_chunk(): invalid pointer

i execute :
python run_training.py head512.yaml --gpu 0 --resume
this is yaml:
Here is an example training config file for 512x512 images.
channels: [32, 64, 128, 256, 512, 512, 512, 512]
tensorboard_log_dir: 'runs/stylegan2_head512'
tensorboard_image_interval: 500
checkpoint_dir: 'checkpoints/stylegan2_head512'
checkpoint_interval: 5000
data_dir: ~/head512/new2/
gpu: [0]
then get Error:
munmap_chunk(): invalid pointer
已放弃 (核心已转储)

I have used the pre-trained StyleGAN2 model to realize image editing. Please see the original project for training the model in here. Thanks for your attention.

I have used the pre-trained StyleGAN2 model to realize image editing. Please see the original project for training the model in here. Thanks for your attention.


Should be a way to upload models automatically

System should work with aws S3 or similar. should check that it can upload a dummy file at the beginning and then upload the checkpoints as they are made.
Should probably be a separate, optional set of hparams to pass to pipeline for the basics like bucket_name and access_key, and then get the private keys from a secrets.json
needs to be careful with uploading checkpoints from ../../.checkpoints as the multiple .. cause errors with my existing aws code.
I can add this, just want to see if there are any suggestions or disagreements on how to add this. Will have a look at doing it this eve.

Sounds good! Maybe shove this into the abstract model class? And then we can call from the pipeline?
For referencing directories use pathlib.Path

Up to you - no strong views on this


Help? Extending pages of a Playlist

I'm trying to get all the pages of a Playlist using .extendPages(spotifyAPI). However, a Playlist is not Paginated — it's the PlaylistItems that is. So I can't call extendPages on the result of func playlist( _ playlist: SpotifyURIConvertible, market: String?).
If I were to call .map(\.items) before .extendPages(self) in my code, that converts the resulting publisher to a publisher of PlaylistItems, and I've lost the context of the Playlist publisher. So somehow I need to hang onto the Playlist and stitch together the results of the fully loaded Playlist items.
This is way above my Combine understanding! (As you can see by my work in progress code here, I'm more comfortable with async/await!) Can Peter or anybody else offer some suggestions?  Not sure if it would be worth rolling this solution into the API for others to use in the future …
func playlist(_ uri: SpotifyURIConvertible) async throws -> Playlist<PlaylistItems>?
{
	let result: Playlist<PlaylistItems> = try await playlist(uri, market: Spotify.currentMarket)	// Playlist<PlaylistItems> . Not Paginated, but the PlaylistItems is.
		// IDEA: .map(\.items)		// This would convert to Paginated items, but how do we rebuild as Playlist<PlaylistItems>?
		.extendPages(self)		// above must be Paginated, so this doesn't work.
		.receive(on: RunLoop.main)
		.async() // my utility using `withCheckedThrowingContinuation` to convert to async code
	return result
}

Use SpotifyAPI.playlistitems(_:limit:offset:market:)

After reading your comment more thoroughly, I have developed a solution that should satisfy all of your requirements. You should be able to easily adapt it to work in an async context.
do {

    // https://open.spotify.com/playlist/0DoorcbBIsa7J6NW9FlLio?si=a5d775610b124051
    // 387 songs
    let playlistURI = SpotifyIdentifier(
        id: "0DoorcbBIsa7J6NW9FlLio",
        idCategory: .playlist
    )

    let result: ([PlaylistItemContainer<PlaylistItem>], Playlist<PlaylistItems>) = try spotify.api
        .playlist(playlistURI)
        .flatMap({ playlist in
            spotify.api
                .extendPages(playlist.items)
                .collectAndSortByOffset()
                .zip(Result<Playlist<PlaylistItems>, Error>.success(playlist).publisher)
        })
        // my own utility that waits for the publisher to return a value synchronously
        .waitForSingleValue()!

    // test output

    let playlistItems = result.0
    let playlist = result.1

    print("\nplaylist name: \(playlist.name)")
    print("items count: \(playlistItems.count)\n")

    let itemNames = playlistItems
        .compactMap(\.item?.name)
        .joined(separator: "\n")

    print("--------------------\nplaylist item names:\n\(itemNames)")

} catch let error {
    print("caught error: \(error)")
}

@Peter-Schorn here are the async functions I've come up with. Hopefully this is of use to somebody! Everything seems to work, fingers crossed!

enum AsyncError: Error {
	case finishedWithoutValue
}

extension Publisher where Output : PagingObjectProtocol {
	func asyncPaginated() async throws -> Output {
		try await withCheckedThrowingContinuation { continuation in
			var cancellable: AnyCancellable?
			var accumulatedValues: [Self.Output] = []
			cancellable = self
				.sink { completion in
					switch completion {
					case .finished:
						guard let first = accumulatedValues.first else { continuation.resume(throwing: AsyncError.finishedWithoutValue) ; return }
						guard accumulatedValues.count > 1 else { continuation.resume(with: .success(first)) ; return }	// No point in merging
						let allItems: [some Codable & Hashable] = accumulatedValues.map(\.items).flatMap { $0 }
						let result = PagingObject(href: first.href, items: allItems, limit: allItems.count, next: nil, previous: nil, offset: 0, total: allItems.count)
						continuation.resume(with: .success(result as! Self.Output))
					case let .failure(error):
						continuation.resume(throwing: error)
					}
					cancellable?.cancel()
				} receiveValue: {
					accumulatedValues.append($0)
				}
		}
	}
}

extension Publisher {
	func asyncCursorPaginated<T>() async throws -> Output
	where Output == CursorPagingObject<T>
	{
		try await withCheckedThrowingContinuation { continuation in
			var cancellable: AnyCancellable?
			var accumulatedValues: [Self.Output] = []
			cancellable = self
				.sink { completion in
					switch completion {
					case .finished:
						guard let first = accumulatedValues.first else { continuation.resume(throwing: AsyncError.finishedWithoutValue) ; return }
						guard accumulatedValues.count > 1 else { continuation.resume(with: .success(first)) ; return }	// No point in merging
						let allItems: [some Codable & Hashable] = accumulatedValues.map(\.items).flatMap { $0 }
						let result = CursorPagingObject(href: first.href, items: allItems, limit: allItems.count, next: nil, cursors: nil, total: allItems.count)
						continuation.resume(with: .success(result))
					case let .failure(error):
						continuation.resume(throwing: error)
					}
					cancellable?.cancel()
				} receiveValue: {
					accumulatedValues.append($0)
				}
		}
	}
}
	// Original one, not for paginated, multi-piece content
	// Based on https://medium.com/geekculture/from-combine-to-async-await-c08bf1d15b77
	// But it doesn't let the next pages in since it just calls first()

extension Publisher {
	func async() async throws -> Output {
		try await withCheckedThrowingContinuation { continuation in
			var cancellable: AnyCancellable?
			var finishedWithoutValue = true
			cancellable = first()
				.sink { result in
					switch result {
					case .finished:
						if finishedWithoutValue {
							continuation.resume(throwing: AsyncError.finishedWithoutValue)
						}
					case let .failure(error):
						continuation.resume(throwing: error)
					}
					cancellable?.cancel()
				} receiveValue: { value in
					finishedWithoutValue = false
					continuation.resume(with: .success(value))
				}
		}
	}
}

There is a lot of repetitiveness in your functions. All of them repeat the logic of transforming the publisher into a single async value:
extension Publisher {
	func async() async throws -> Output {
		try await withCheckedThrowingContinuation { continuation in
			var cancellable: AnyCancellable?
			var finishedWithoutValue = true
			cancellable = first()
				.sink { result in
					switch result {
					case .finished:
						if finishedWithoutValue {
							continuation.resume(throwing: AsyncError.finishedWithoutValue)
						}
					case let .failure(error):
						continuation.resume(throwing: error)
					}
					cancellable?.cancel()
				} receiveValue: { value in
					finishedWithoutValue = false
					continuation.resume(with: .success(value))
				}
		}
	}
}

I recommend that you use Apple's implementation instead and create a wrapper around it for cases where you only need (or expect) the first value:
extension Publisher {

    var firstValue: Output? {
        get async throws {
            for try await value in self.values {
                // wait for the first value in the `AsyncSequence`
                // and then return it
                return value
            }
            // if the sequence produces no elements, return `nil`
            // you could change this to `throw AsyncError.finishedWithoutValue`
            // this computed property is already marked as throwing
            return nil
        }
    }

}

This lets Apple take care of the work of consuming the published values and transforming the result into an AsyncThrowingStream.
Then, apply transformations to the stream or the elements it produces, if necessary (instead of using Combine operators). For example:
let playlist = try await spotifyAPI
    .playlist(playlistURI)
    .firstValue

guard let playlist = playlist else {
    fatalError("unexpectedly found nil for `playlist`")
}

let playlistItems = try await spotifyAPI
    .extendPagesConcurrently(playlist.items)
    .collectAndSortByOffset()
    .firstValue


[fix][broker]Cache invalidation due to concurrent access

Fixes #17908
Motivation

ManagedCursorImpl.setReadPosition() will trigger cleaning of caches smaller than the earliest readPosition
In ManagedLedgerFactoryImpl, there is a scheduled task that deletes inactive cache data every second

<strong(High light)>When the cursor updates the readPosition and clears the outdated ledger cache concurrency, an entry's cache is incorrectly deleted.
E.g.

ManagedCursorImpl.setReadPosition
ManagedLedgerFactory.doCacheEviction

get the first entry in the Cache

invalidate position 1:1

delete 1:1 from EntryCache

recycle entry and set entry.timestamp to -1

check: entry.timestamp > maxTimestamp

(High light)pop first entry from EntryCache: expecting 1:1, but really 1:2

So entry 1:2 is incorrectly deleted.
https://github.com/apache/pulsar/blob/0c7a0d144d9e527fbd25a1d738715934fbcf38ab/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/util/RangeCache.java#L195-L208
You can reproduce the problem by runningManagedLedgerBkTest.verifyConcurrentUsage 200～1000 times
Modifications
Use the instruction remove(key) instead of popFirst() to delete the target Entry
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

https://github.com/poorbarcode/pulsar/pull/26

Codecov Report

Merging #18076 (a7b6598) into master (6c65ca0) will increase coverage by 15.90%.
The diff coverage is 57.14%.

@@              Coverage Diff              @@
##             master   #18076       +/-   ##
=============================================
+ Coverage     34.91%   50.81%   +15.90%
- Complexity     5707     8697     +2990
=============================================
  Files           607      607
  Lines         53396    53398        +2
  Branches       5712     5714        +2
=============================================
+ Hits          18644    27136     +8492
+ Misses        32119    23242     -8877
- Partials       2633     3020      +387

Flag
Coverage Δ

unittests
50.81% <57.14%> (+15.90%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../service/SystemTopicBasedTopicPoliciesService.java
61.39% <0.00%> (+9.80%)
:arrow_up:

.../pulsar/broker/stats/BrokerOperabilityMetrics.java
98.21% <ø> (+5.56%)
:arrow_up:

...g/apache/pulsar/compaction/CompactedTopicImpl.java
69.28% <0.00%> (+58.57%)
:arrow_up:

.../org/apache/pulsar/broker/admin/v2/Namespaces.java
54.72% <50.00%> (+46.69%)
:arrow_up:

...apache/pulsar/proxy/server/DirectProxyHandler.java
63.63% <50.00%> (ø)

...broker/delayed/InMemoryDelayedDeliveryTracker.java
65.00% <75.00%> (+65.00%)
:arrow_up:

...pulsar/broker/admin/impl/PersistentTopicsBase.java
51.48% <100.00%> (+40.08%)
:arrow_up:

...rg/apache/pulsar/broker/service/BrokerService.java
58.04% <100.00%> (+10.03%)
:arrow_up:

...e/pulsar/broker/service/EntryBatchIndexesAcks.java
82.14% <0.00%> (-10.72%)
:arrow_down:

.../apache/pulsar/broker/namespace/LookupOptions.java
87.50% <0.00%> (ø)

... and 146 more

Hi @eolivelli
Sorry, I have been too busy these two days to add a test. I created a new pr to make up for it.

https://github.com/apache/pulsar/pull/18120


use git ls-tree instead of find to take advantage of .gitignore

we only care about files in the git repository, so no reason to use find.  This may be faster when a repo may have a lot of code that is in .gitignore because it's loaded from bower or requirements.txt or submodules, or whatever.
It might be useful to create a git submodule for this repo with a repo purely for test cases (I tested this on the current repo, but it's fairly simple), but that's beyond the scope of this pull request.

awesome! sorry for the slow response, thanks for the code, merged.


pkg/controller: stop controller execution on shutdown

Signed-off-by: André Martins<EMAIL_ADDRESS>I wonder how manly flakes could be explain because of this :thinking:

test-me-please

@aanm are there specific flakes which are related to this change?

@ianvernon I can't really tell but basically anything that is caused by a controller that should be shut down but it ran one more time.


Created zdh-thm.txt

@kevinstroeher Unfortunately, zdh-thm.de domain does not exist, thus this pull request can not be satisfied.
$ host zdh-thm.de
Host zdh-thm.de not found: 3(NXDOMAIN)


Burrito failed to wrap up your app

Trying to install via Homebrew, currently getting this error:
Last 15 lines from /Users/mxgrn/Library/Logs/Homebrew/lazyasdf/05.mix:
--> PHASE: :build
---> Generating wrapper metadata file...
error(link): undefined reference to symbol 'dyld_stub_binder'
error: UndefinedSymbolReference
---> Cleaning up...
---> Burrito failed to wrap up your app! Check the logs for more information.
** (RuntimeError) Wrapper build failed
    (burrito 0.8.0) lib/steps/build/pack_and_build.ex:54: Burrito.Steps.Build.PackAndBuild.execute/1
    (burrito 0.8.0) lib/builder/builder.ex:146: anonymous fn/3 in Burrito.Builder.run_phase/2
    (elixir 1.15.7) lib/enum.ex:2510: Enum."-reduce/3-lists^foldl/2-0-"/3
    (elixir 1.15.7) lib/enum.ex:984: Enum."-each/2-lists^foreach/1-0-"/2
    (burrito 0.8.0) lib/builder/builder.ex:95: Burrito.Builder.build/1
    (mix 1.15.7) lib/mix/tasks/release.ex:1064: Mix.Tasks.Release.run_steps/1
    (mix 1.15.7) lib/mix/task.ex:455: anonymous fn/3 in Mix.Task.run_task/5
    (mix 1.15.7) lib/mix/cli.ex:92: Mix.CLI.run_task/2

Need to bump burrito and zig versions. Are you on macOS 14?

Yep


Text inside bottom sheet is just showing one line

you have to wrap your root view in OvergrowScrollView(maxHeight: _) {}


Spec Clarification: Indentation, Multistring, Multimap, and Array Syntax in bru-lang

Hey there, love the idea of writing this as a spec, and the spec itself is looking pretty good!
As a spec writer myself I noted a few things. Since this is a spec based on an existing implementation, I'm guessing you've already thought about these things.
Happy to hammer these out in real time (links in my profile) if that's helpful, or if you write up stuff in here I can wordsmith it into a PR. Happy to help, Bru has been a real game changer for me, so I'd love to give something back.
Tabs versus spaces
You'll see in the Multistring section that indentation starts to matter. I admit that I'm not unbiased on this topic, but at the very least if bru-lang is picking spaces/tabs (or if it's not) it should be stated explicitly.
(When considering the issue, please consider that it's not simply "personal preference", it's actually an accessibility issue.)

[ ] Clarify whether tabs versus spaces matters, and in what ways

Multistring
In the description and examples it's unclear what this string should be interpreted as:
http: {
  body: {
    type: text
    data: '''
      This is a multiline string.
      It can span multiple lines.
    '''
  }
}

My intuitive guess is the data property should be This is a multiline string.\nIt can span multiple lines. as opposed to, for example this \s\sThis is a multiline string.\s\sIt can span multiple lines. or even this \n\s\s\s\s\s\sThis is a multiline string.\n\s\s\s\s\s\sIt can span multiple lines.\n\s\s\s\s
What should happen in this case?
http: {
  body: {
    type: text
    data: '''
Foo
    Bar
        Bizz
    '''
  }
}

Should it be Foo\nBar\n\s\sBizz or Foo\n\s\s\s\sBar\n\s\s\s\s\s\s\s\sBizz or...

[ ] Clarify whether newlines before/after are included
[ ] Clarify that the pre-spaces are removed, e.g. for indentation n strip up to n + 2

Multimap
Mostly the edge cases here involve Multimap with unset values. Is this even allowed? E.g. what about this:
http: {
  query: {
    userId:
  }
}

My intuitive guess would be this would map to JavaScript like:
const multimap = {
  http: {
    query: {
      userId: undefined
    }
  }
}

Arrays
What should a Multiline look like in an array? What about a Multimap?
meta: {
  tags: [
    sanity
    '''
      This string
      is multiline.
    '''
    userId: 1
    regression
  ]
}

Presumably in this case the tags is equivalent to a JSON array like:
[
  "sanity",
  "This string\nis multiline.",
  { "userId": 1 },
  "regression"
]

This should probably just be an example added, to be explicit.

Hey @saibotsivad
We have a call today - 24 Jan 2023 at GMT 5pm on Discord. It's be awesome if you could join it
https://discord.com/invite/xEw7K3BM?event=1199350756269101167

I believe that most of your questions are addressed in the spec.

Tabs versus spaces
You'll see in the Multistring section that indentation starts to matter. I admit that I'm not unbiased on this topic, but at the very least if bru-lang is picking spaces/tabs (or if it's not) it should be stated explicitly.

We do specify that indentation levels are two spaces:
Bru is block-oriented and uses both delimiters and indentation to ensure correctness.
When a new block context is started, the contents of the block MUST be indented to
the next level. Each level of indentation in Bru is signified with two ASCII space
characters (0x20).

Tabs are not supported for indentation, but will be preserved within strings or multistrings.

Multistring

In the description and examples it's unclear what this string should be interpreted as:

I have tried to provide multiple explicit examples for this to reduce this question. If you have some suggestions on how to clarify these examples further, I would appreciate them.

http: {
  body: {
    type: text
    data: '''
Foo
    Bar
        Bizz
    '''
  }
}

This should not parse. The n + 2 indentation is required for multiline strings.

Multimap
Mostly the edge cases here involve Multimap with unset values. Is this even allowed? E.g. what about this:
http: {
  query: {
    userId:
  }
}

This would be the equivalent of { "http": { "query": { "userId": "" } } }.

Arrays
What should a Multiline look like in an array? What about a Multimap?

There are examples for this in the specification.

Excellent answer and response, thank you so much for your time!


use unicode to specify quotes in css file

The quotes tag <q> is modified in the file pygsti_dashboard.css. In that file special characters are used for the quotes, which render incorrectly in the browser:

This PR defines the quotes using unicode, which renders the quotes as intended:

@enielse

@eendebakpt, thanks for this!  I had noticed the quotes were getting misrendered for some time now but never figured this out.  Unfortunately, in order for me to accept your PR you'd need to fill out a contributor license agreement (CLI).  Since this PR only changes several lines, and to save us both paperwork, I've just incorporated your suggested edits into commit 88d74a771031b8da668d6600a39f43250b52ab19 (and credited you there).  I hope this is satisfactory to you.


Use log probability to determine the active pixels, not the elbo

Following PR #408, using log probability to determine active pixels doesn't seem so essential. It's probably better to focus on #157 -- initializing active pixels without an existing catalog.


[FEATURE] ENGLISH DOC

Is your feature request related to a problem? Please describe.
NO
Describe the solution you'd like
ENGLISH DOC
Describe alternatives you've considered
THE PROJECT NEED AN ENGLISH DOC
Additional context
NONE

FINISH


adds release notes regarding RHEL7 installation bug

https://github.com/Shippable/heap/issues/2553

lets link onebox text with http://docs.shippable.com/platform/server/install-onebox/
we should try to link the release notes back to docs and/or blogs


feat: add sourcemap support

Closes #8

Thanks!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


marl::containers::vector fixes

Default copy constructor and copy assignment operators were still being generated, which produce bad implementations due to raw internal allocations.
The tests were only exercising copy and assignment for vectors of different base capacities, which we do have non-default implementations for.
Delete the copy constructor - it discourages use allocators, which is a Bad Thing.
Implement the assignment operator.
Add tests.
Note: Nothing in marl was exercising these broken default constructors.

In general, a type that implements copy assignment also implements copy construction - they go together. Couldn't you implement copy assignment to use the allocator from the assignee?

I could, but that'll behave differently to the other copy constructors that default to marl::Allocator::Default.
I don't consider the marl::containers namespace as a public API, and I prefer to not have the default copy constructor as it forces me to consider memory allocation, which is important for fixing #131.

I could, but that'll behave differently to the other copy constructors that default to marl::Allocator::Default.
I don't consider the marl::containers namespace as a public API, and I prefer to not have the default copy constructor as it forces me to consider memory allocation, which is important for fixing #131.

Just for reference, std::vector has 2 copy constructors: one that takes an 'other' and an 'allocator', and there's the default one that only takes the 'other'. As per #6 in the doc:
6) Copy constructor. Constructs the container with the copy of the contents of other. If alloc is not provided, allocator is obtained as if by calling std::allocator_traits<allocator_type>::select_on_container_copy_construction(other.get_allocator())

So I'm with you on not allowing a default constructor to be generated. Just saying you can write one that does the right thing and uses the allocator from 'other'.
Anyway, your call.

Just saying you can write one that does the right thing and uses the allocator from 'other'.

I'm unsure about wanting to re-use the allocator from other.
All the places where I can imagine we need to copy a marl::containers::vector, we're doing it to make a copy of a public API object. These public objects should all take an allocator themselves, so copying into the allocator of other seems broken.
If I do end up needing cases where I want to copy from other, I can explicitly do this at the call site.
Maybe I'll find I'm barking up the wrong tree, and do end up needing this. However, it's always easier to undelete an API than deleting one that might be in use somewhere.


Mavlink receiver SET_ATTITUDE_TARGET filling for VTOLs

Add MAV_TYPE_VTOL_* to thrust filling
Describe problem solved by the proposed pull request
Thrust received from SET_ATTITUDE_TARGET does not filled for VTOL airframe types.
Test data / coverage
Tested on real UAV and MAVSDK.
Describe your preferred solution
Add VTOL types to thrust filling.

Close due to https://github.com/PX4/Firmware/pull/13141


Nomcom references (in announcement page) is outdated

Describe the issue
The file templates/nomcom/announcements.html only lists 8713; it should also list 9389.
Code of Conduct

[X] I agree to follow the IETF's Code of Conduct

addressed by #6583


react-redux getWrappedInstance doesn't expose members

Hey folks! Trying to add flow-typed to my project. I have a parent component that has a Redux-connected child whose members the parent wants to access. Previously, I called getWrappedInstance on the connected child to get the "real" child so I could access its members. However, the typing of getWrappedInstance seems insufficiently specific to allow access to members of the connected component.
In the past, I just used this:
type EntryConnect = {
  getWrappedInstance: () => Entry,
};

What would be the best way to handle this with flow-typed? Is it possible to update the type definitions in this repo to make getWrappedInstance() work? Alternately, is there some hack I can use to silence Flow in this case while continuing to use flow-typed? (Leaving the definition above in place isn't working.)
Thanks in advance for any help.

I was able to work around this issue by using a version of this suggestion. Here's a link to the workaround if you're curious


[Bug] Comb filtering prevention is always actve when Time.timeScale == 0

Hi! I'm trying this out for my game where I let the player pause the game by setting Time.timeScale to 0 but I still want some sounds (i.e. UI sounds) to be allowed to play more than once while paused. The issue is that comb filtering prevention depends on the frame deltaTime, not the real-time delta between playbacks so when I trigger a sound once while paused, I can't trigger it again until I resume the game.
Here's the origin of the issue:
https://github.com/man572142/Bro_Audio/blob/a89c2c692d712cb6cebf5eaf9a107cb2f8d1eb76/Assets/BroAudio/Core/Scripts/SoundManager/SoundManager.cs#L340
I've fixed the issue locally using Time.realtimeSinceStartupAsDouble and I can open a PR if you allow contributions :)
Thanks!

Thank you for bringing this up! Yes, that's indeed an issue. It shouldn't be affected by the Time.timeScale. I appreciate that you also found a great solution to resolve it. I hope everything is working well after your fix :)
Regarding the PR, while I am very welcoming of any contributions, I have other thoughts on this issue. The design might change completely.
The flaw with the current design is that it uses Coroutine, which means the minimum time duration is one frame (16ms at 60fps and 33ms at 30fps). Unfortunately, this doesn't accomplish the original purpose of this feature and makes the adjustable prevention time in the Preferences meaningless.
I might use async/await to replace that, which will be framerate-independent and not affected by the Time.timeScale. These improvements will be included in the next update. Stay tune and thank you!

Hi! The timeScale issue has been resolved in the latest version. Thanks again for reporting this!
After some research, I discovered that Unity doesn't play audio immediately when we call the playback APIs. Instead, it's played at the end of the frame (specifically, after changing to the audio thread). As a solution, I ended up using a LateUpdate() approach combined with Time.unscaledTime to address this issue. This also improves the overall playback timing. I hope you enjoy the update, thank you!


Add support for Python 3.11 and Numpy 1.24

At some point we'll need to do this, but will have to wait until numba supports both.

Should be possible now as numba latest release supports both.


x/mobile: CGO_CXXFLAGS, CGO_LDFLAGS overwritten for ios

What version of Go are you using (go version)?

$ go version
go version go1.18.2 darwin/amd64

Does this issue reproduce with the latest release?
This bug is in cmd/gomobile, and is reproducible with the latest commit on that.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GOARCH="amd64"
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOVERSION="go1.18.2"
GCCGO="gccgo"
GOAMD64="v1"

What did you do?

I was trying to build an ios framework with gomobile that also includes C dependencies. I need to set CGO_CXXFLAGS and CGO_LDFLAGS to get the build working.
CGO_ENABLED=1 CGO_LDFLAGS="<flags>" CGO_CXXFLAGS="<flags>" gomobile bind -target ios/arm64,iossimulator/amd64 -iosversion 9.3.0 ./cmd/app

What did you expect to see?
gomobile should forward the values of CGO_CXXFLAGS and CGO_LDFLAGS to rest of the build process (gobind, go build, etc.) This does happen when building for Android, but not when targeting ios.
What did you see instead?
The CGO_CXXFLAGS, CGO_LDFLAGS, etc. variables get overwritten by values that are required to introduce system dependencies for ios.
This was causing my build to break.
Ideally, these new values introduced by gomobile should be appended to values already in the environment. I tried out this change that resolves this issue.

Sorry, I was not aware of the bug template. Updated my post above with more details.

@hyangah can you give this a look?


Translating the Comunity Section

As a first step, we could do straight translations. Additionally, as @tesseralis has pointed out it would be cool if we could also add resources in Spanish. I also opened a conversation on Spectrum for discussing which of those resources would be a good fit.
Community Resources

[x] Support @RamirezAlex #207
[x] Courses @jpvg10 #211
[x] Examples @jvelezpo #216
[x] Meetups @RamirezAlex  #210
[ ] Conferences
[ ] Articles @jpmti2016
[x] Podcasts @RamirezAlex #209
[ ] Videos @jpmti2016
[x] External Resources @gonzarodriguezt #243

Tools

[ ] Debugging @mact200590  #219
[ ] Component Workbenches @jmmendivil
[ ] JSX Integrations
[ ] Starter Kits
[x] Routing @hextiandro #214
[ ] Model Management
[x] Data Fetching @OmaritoAtPoly #220
[x] Testing @RamirezAlex #212
[ ] UI Components
[ ] Miscellaneous

I would like to help with Support

I would like to help with Courses

@RamirezAlex, @jvelezpo, @jpvg10 - You got them. Thanks! 🎉

Please review #207 🎉 Support
I would like to continue with Podcasts

Please review #209 Podcasts
I'd like to continue with Meetups

Please review #210 Meetups
I'd like to continue with Testing

#211 Courses ready for review

Please review #212 Testing
I'd like to continue with Conferences

I would like to help with Routing

Welcome @hextiandro! You got it.

I would like to continue with Debugging

Ok, @mact200590, you got it. 🚀

i'd like to colaborate with data fetching

i wish to continue collaborating with UI Components

Hi, @OmaritoAtPoly. I see you are still working in Data Fetching (you created a Draft Pull Request). Once you are done with it you can take another one.

I'm ready to obtain a new task.

I would like to contribute with External Resources.

Hi, @gonzarodriguezt! You got it.

Oh!!! I see I still have Conferences, I will send it soonish. Sorry about that.

I would like to help with Component Workbenches

Welcome, @jmmendivil! You got it!

Component Workbenches is ready for review on #311

@carburo I made 2 PR para videos and articles respectively

I will be translating conferences, since I'm not seeing any translation for this and there's no recent comments for translating it.

Here's the PR for the conferences translation:
https://github.com/reactjs/es.reactjs.org/pull/361

Is the conferences translation made by #361 not deployed? Looking at https://es.reactjs.org/community/conferences.html I noticed that the website is still in English.

@d4vsanchez No, nunca llegó a mezclarse. Casi de forma inmediata tuvo conflictos con la rama principal. Esta página se actualiza con relativa  regularidad y el contenido traducible se limita sobre todo a las fechas. El problema no es traducirla, sino el costo de mantenerla actualizada. A menos que la solución incluya un proceso al menos semiautomático no creo que los beneficios superen los costos.


Allow whitespace in expressions inside template literals

As of now expressions like the following won't get replaced because I'm using spaces  inside the curly braces.
<script jsx>{`
  p {
    color: ${ colors.primary }
  }
`}</script>

It has to be written as ${colors.primary}.
Also note that the README's demo is using that style.
I believe this is the corresponding source: https://github.com/zeit/styled-jsx/blob/45ba8542f93d5caee262ead08a584a25767eacd5/src/babel.js#L108.

@nikvm good catch, can you send a pr to fix this? It should be a matter of replacing using a regular expression (keep the initial for later tho).
I believe that you need a regexp at line 108 which you will use at line 115
new RegExp(`\\$\\{\\s*${e.getSource()}\\s*\\}`)

Sure. I should find some time the coming days.


Feature Request > Ability to get download count on individual files in a repo

It would be great to have a feature to be able to track how many times a particular file in a repo is downloaded. I know this can be done on counting releases downloads but I have a repo where a single .conf file gets pulled daily by users using an update script for the project. Being able to see how many times a day/week/month this file is being requested would be a great way to show live users on the project.

Hi @mitchellkrogza, it sounds like you have a feature request for GitHub.com itself, not for the hub command-line utility that this project and issue tracker is about. You should instead send such requests directly to GitHub Support at<EMAIL_ADDRESS>not to this repo. This project is only about the command-line utility called hub. Thank you!

Thanks @mislav I will email them but I think I figured out a way to track this through my own update script. Will see if I can follow that avenue first to track these downloads. Thanks for your help.

@mitchellkrogza
can you please tell how you were able to do ?


It's for TLS session tickets that we need to validate TP, not the tokens

As being discussed in the I-Ds, when accepting 0-RTT it is necessary to confirm that certain TPs have not changed. As the acceptance (or rejection) of 0-RTT depends on the processing of session tickets, those TPs should be associated to the session tickets.
However, as part of #350, we incorrectly started associating those TPs to NEW_TOKEN tokens.
This PR reverts that, introduces a helper function to be used by applications for proper checking.
Design considerations: QUIC uses TLS as a submodule, sometimes specifying how TLS should be used. Association of TPs to 0-RTT is one such case. From API design standpoint, we try to not hide the TLS context being used by QUIC. Rather, we try to provide helper functions for adjusting how the behavior; see quicly_amend_ptls_context. Following this tradition, we add a new helper function called quicly_build_session_ticket_auth_data which is to be used for binding TPs to session tickets. For actual usage, see https://github.com/h2o/h2o/pull/2354.

Thank you for the reviews!


How to set autoreset on the timer instance?

Hi,
Just came across your library. It find it a bit cumbersome to set certain properties on the timer. e.g autoReset. I need to pass this via the constructor, which is rather annoying when you combine this with e.g. autofac.
In autofac (or any IoC container I guess), I want to register a timer so that any user can get a timer injected.
something like
            builder.RegisterType<SystemTimer>().As<ISignaler>();

Now, any class that requires a timer can have an ISignaler injected. But, now the IoC container is also responsible to define the behavior of the timer. Again, e.g., the autoreset will default to true. Which should be configurable per use of the timer. I can have two classes, where one requires the autoreset to be false, and the other class requires it to be true.
Of course, I can set explicitly define how autofac should instantiate my timer
            builder.RegisterType<SystemTimer>()
                .As<ISignaler>()
                .WithParameter(new Autofac.NamedParameter("autoReset", false));

But this would mean that each class with requires an ISignaler will receive a timer with autoreset disabled.
Of course, I can work around this, but then again I need to boilerplate stuff around the SystemTimer class (e.g. make a factory), which kind of defeats the purpose all together of using this library (then I might as well rely on the .net's System timer instead).
How do you see this? Can you relate? Or have you thought about this too, and can I achieve what I want in a different way?
PS: Best regards from Nuenen ;-)

for this we create two interfaces ( IAutoTimer, IManualTimer ) and timer wrapper ( AutoTimer, ManualTimer ),
public interface IAutoTimer : ISignaler {}
public class AutoTimer : SystemTimer, IAutoTimer {}

public interface IManualTimer : ISignaler {}
public class ManualTimer : SystemTimer, IManualTimer
{
	public ManualTimer(): base( false ) {}
}

then you can register these two and inject the require type of timer.
Services.AddScoped<IAutoTimer, AutoTimer>();
Services.AddScoped<IManualTimer, ManualTimer>();

@RobThree maybe this would be a great addition to this lib ( simple wrapper on all timer types ) because AutoReset is the only property that cant be changed after creating the timer.
This way the DI usage is much easier and the readability is improved ( interface name makes it clear how the timer tick behave without knowing the implementation )


Update 3-graphics.rst

Syntax/grammar fixes for chapter 3 of the tutorial.

Another thing I noticed that may be simply a preview limitation.  The literalinclude commands are not rendering properly.  It shows the block of code.  I'm not familiar enough with RST to decide whether there is a problem with the code.


Fix renovate's config so language filtering works as intended

I cannot find any mention of matchLanguages at
https://docs.renovatebot.com/configuration-options and by my reading of https://docs.renovatebot.com/configuration-options/#matchcategories using matchCategories does what my understanding of the intention of the use of matchLanguages was trying to do.

This is the correct fix, it looks like it was changed 3 weeks ago: https://github.com/renovatebot/renovate/releases/tag/36.0.0

This is the correct fix, it looks like it was changed 3 weeks ago: https://github.com/renovatebot/renovate/releases/tag/36.0.0

I'm surprised they didn't have more of a noisier transition period, and the breaking change is silent?

It is a major version change, which aligns with how they do breaking changes, but (with a very brief search) I haven't found any way to stop renovate-bot itself from auto updating? It might be something managed organisation (google) wide.

It is a major version change, which aligns with how they do breaking changes, but (with a very brief search) I haven't found any way to stop renovate-bot itself from auto updating? It might be something managed organisation (google) wide.

#1489 may help surface future breakage, but potentially only when we touch the config...


AMBARI-23030. Remove light icon in user edit if user role is admin

What changes were proposed in this pull request?
Remove light icon in user edit if user role is admin
How was this patch tested?
Manually, only markup changes
Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/702/
Test FAILed.
Test FAILured.


Console (Scala REPL) API port from sbt-pamflet

Implements #170 . Added some additional tests as suggested.

Hi @sepulkarz,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Lightbend Contributors License Agreement:
http://www.lightbend.com/contribute/cla

Nice.

@sepulkarz Could you let us know when you sign the CLA plz?

@dwijnand Can you have a look at this too?

We should put the following header on all files:
/*
 * Zinc - The incremental compiler for Scala.
 * Copyright 2011 - 2017, Lightbend, Inc.
 * Copyright 2008 - 2010, Mark Harrah
 * This software is released under the terms written in LICENSE.
 */

For port from sbt-pamflet, we should keep the original commit if possible with the --author set to the original author (me).

Thanks for the comments. I will try to adapt the PR according to them.

@eed3si9n Some of the interface files were touched by more than one commit. Also not all of the files were copied, as Logger and F0 are already present in util-interface dependency. Shall I cherry-pick the original 3 commits and delete Logger and F0 in a separate commit?

Yea. I think that would make sense. In this case it's me, so it's fine, but we have to be diligent about crediting the original authors in OSS partly for legal reasons.

Understood. A quick question: CLA mentions "sign and return this Agreement to the Company below by using GitHub", but the page that links to it (http://www.lightbend.com/contribute/cla) says "You do not need to provide Lightbend the signed agreement.". I'm not sure how to proceed.

You just have to verify the CLA via http://www.lightbend.com/contribute/cla/verify once it's signed, instead of physically mailing in the copy to us.

@eed3si9n I think CLA is already signed, otherwise CI wouldn't have kicked in. Can you approve and merge?

@eed3si9n Thanks, I did that. @jvican I think it's not ready to merge yet - I want to do some corrections you guys suggested. I will let you know shortly.

@sepulkarz great work! LGTM and waiting for mentioned 'small corrections'  :)

I made all the suggested changes :). One last tidbit - it seems like the existing test suites don't have a copyright header, so I didn't put one in the new test suite.

@sepulkarz I think it's good, don't worry. Another PR should target putting a copyright header to those test source files :wink:.
@eed3si9n Merge when you agree with the changes. I've already approved this PR.
Thanks again Krzystoff! Hope to see you around :smile:.

LGTM pending Travis.

Merging, Travis is happy.


Refactor UI publishing and polish tests

Signed-off-by: Julien Pivotto<EMAIL_ADDRESS>

looks good, from my point of view test.yaml is not an accurate name as it is publishing also.
Why not splitting the publish job and the tests in two separate files ?

Because of the required relationship between the jobs.

ah yeah ok.
Why not renaming the file then ?

I have renamed it to CI

Alright, more dependencies :)


MaskRCNN and LightTrack

Can you guys provide information on how to switch the CPN and YOLO models with MaskRCNN?

Personally speaking, I think mask-rcnn is not suitable for this system


Remove db constraint on historical user to prevent IntegrityError

Add db_constraint=False to the history_user field
Description
There's a known issue (see link below) where if the user_model has history = HistoricalRecords tracked on it, then an IntegrityError is raised when trying to delete that user. I believe this is because of the order in which Django deletes essentially causes a circular-referencing error within the database.
This change simply removes the db_constraint from this field. Thus, solving the problem. Multiple other users have raised this as the solution in the linked issue.
I was originally hesitate about use db_constraint=False and was considering making it optional, but noticed that this paradigm is already leveraged multiple times in this codebase.
Related Issue
https://github.com/jazzband/django-simple-history/issues/391
How Has This Been Tested?
Added a test to the test suite that covers the bug found in #391.
Also, it seems like myself (as well as multiple other users) are already running this fork in their own production environments without issue.
Screenshots (if appropriate):
n/a
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] I have run the make format command to format my code
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have added my name and/or github handle to AUTHORS.rst
[x] I have added my change to CHANGES.rst
[x] All new and existing tests passed.

@keithhackbarth  thanks for this fix! Looks like it failed tests though.

Only other comment would to make this something that you can opt in to to maintain backward compatibility (and prevent having to run migrations on a minor version change). Happy to make it the default in version 3.0 when I have a few other backward incompatible changes I want to add. What do you think?

Thanks @rossmechanic

Fixed the broken test. Sorry about submitting a broken pull request. This one in particular is a little funky because the core of the issue is foreign key database constraints (MySQL and Postgres). SQLite doesn't enforce this constraint in the same way. The tests always passed on SQLite regardless. To reproduce, I had to use a Postgres test runner.

Based on your comment re: backward compatibility, made it an optional parameter that defaults to original "True" value

@keithhackbarth Thanks for the quick responses here. Going to take a look now. I know - it's been on my list for months to get to adding more DBs to the test suite but haven't had the time as of late.

Just had one comment on the test - mind also adding to the change log

@rossmechanic - Good call on updating the test and changelog. Both are done!


Option to test with system compiled Q

It would be nice to have an option to test qtools with the Q binary in PATH, meaning the Q executable running the tests is the Q executable running the production code.
I attempted this by removing the download code and setting the QBIN_DIR variable producing the error (see this PR for changes):
####################################################
|  Testing:    q_automapper/1-probrcl/
####################################################
Working in /tmp/QTOOLS_TESTS_2Tk/1-probrcl_P4Z
Checking '*log'
sed: can't read *log: No such file or directory
FAIL  Check the following files:
/tmp/QTOOLS_TESTS_2Tk/1-probrcl_P4Z/diffs.txt
/tmp/QTOOLS_TESTS_2Tk/1-probrcl_P4Z/stdout.txt

Will look at this.
It probably failed due to the variable being assigned a relative rather than absolute path.

Solved in devel.


After compiling I'm getting Payload File does not exist errors

Hey all!
I've compiled FFmpeg using both the BuildFFmpeg.bat and following the guide here: https://trac.ffmpeg.org/wiki/CompilationGuide/WinRT
When I open the FFmpegWin10 project and attempt to build I get the following errors:
Error		Payload file 'C:\Users\joshu\source\repos\FFmpegInterop\ffmpeg\Build\Windows10\x64\bin\swscale-4.dll' does not exist.	MediaPlayerCS
Error		Payload file 'C:\Users\joshu\source\repos\FFmpegInterop\ffmpeg\Build\Windows10\x64\bin\swresample-2.dll' does not exist.	MediaPlayerCS
Error		Payload file 'C:\Users\joshu\source\repos\FFmpegInterop\ffmpeg\Build\Windows10\x64\bin\avutil-55.dll' does not exist.	MediaPlayerCS
Error		Payload file 'C:\Users\joshu\source\repos\FFmpegInterop\ffmpeg\Build\Windows10\x64\bin\avformat-57.dll' does not exist.	MediaPlayerCS
Error		Payload file 'C:\Users\joshu\source\repos\FFmpegInterop\ffmpeg\Build\Windows10\x64\bin\avfilter-6.dll' does not exist.	MediaPlayerCS
Error		Payload file 'C:\Users\joshu\source\repos\FFmpegInterop\ffmpeg\Build\Windows10\x64\bin\avdevice-57.dll' does not exist.	MediaPlayerCS
Error		Payload file 'C:\Users\joshu\source\repos\FFmpegInterop\ffmpeg\Build\Windows10\x64\bin\avcodec-57.dll' does not exist.	MediaPlayerCS

Those files do exist in my Build folder, however, they are all named one increment (version?) higher. So "avcodec-57.dll" is "avcodec-58.dll" in the correct destination. And ideas?

If I manually change all the file names, the project will build but I am unable to open any media with the example player.
Thanks for any help! Apologies if my terminology is vague or incorrect. I'm quite new to development.
UPDATE
I was able to get it working with FFmpegInteropX so it seems the issue was not with my compiled FFmpeg.

Glad to hear that you could get it working. Closing this issue.

Oh lol, only now I realized that this issue was not opened in FFmpegInteropX repo. Not really my responsibility to close issues here...

Hey! Sorry for not being clear. Yeah it seems the issue is with FFmpegInterlop as I did not have any issues using the same compiled FFmpeg with FFmpegInterlopx. Thanks a ton for your work on these projects!!


iFrameResize inside iframeSizer.min.js file

Hello David,
Is it fine to use the iFrameResize(null, '#myIframe') in the bottom of iframeSizer.min.js file?

Only if you load the file after the iFrame tag in your html.


Un-register all the bot commands when the bot shuts down.

When testing on production, the testing bots tend to clog up the command list when they're offline, causing confusion to users. This PR unregisters the slash/application commands whenever the bot is shut down, so that end-users don't have any confusion.

Note: It takes about a minute or two for the bot to completely shut down now, as commands are removed one-by-one. Does not work with Visual Studio Code debug mode for some reason.

IMO, this should only apply to non-prod versions of the bot. Given that the prod bot does important logging, we should minimise its downtime as much as possible.
We also definitely need to resolve the VSCode debug mode issues before merging since most (if not all) of the Chiya team use it to varying degrees.

I agree that this branch should not be merged, but should be kept around for local testing to unload the slash commands when needed as it takes way too long to completely shut off or reload the bot.


traffic_signals for pedastrians are missing (1.9.2)

highway=traffic_signals - "Required. Indicates traffic signals for cars." (http://wiki.openstreetmap.org/wiki/Tag:highway%3Dtraffic_signals)
crossing=traffic_signals - "Required. Indicates traffic signals for pedestrians." (http://wiki.openstreetmap.org/wiki/Tag:highway%3Dtraffic_signals)
"crossing=traffic_signals
Position this tag where the crossing-traffic (pedestrian, bicycles) have their own traffic lights.
Mostly near highway=traffic_signals." (http://wiki.openstreetmap.org/wiki/Key:crossing)

We have a preset for highway=traffic_signals already that can be set on junction nodes.
We have a generic preset for highway=crossing that can be set on junction nodes or ways.  It displays as "Street Crossing" and allows users to pick the "Type" from any of the commonly used crossing=* values (top values currently are: uncontrolled, traffic_signals, zebra, unmarked).  I don't like crossing=traffic_signals because it breaks duck tagging but people can set it here if they really want to.
We have a specific preset for highway=crossing + crossing=zebra that displays as "Pedestrian Crosswalk".  We discussed it extensively on #2918 and I'm not planning to change it anytime soon as there is no better alternative at this time.

because it breaks duck tagging

Nobody except for maybe English speakers support duck tagging in database.
http://wiki.openstreetmap.org/w/index.php?title=Duck_tagging&action=history
User Cartinus or next guy from http://wiki.openstreetmap.org/wiki/Category:Users_against_tag_voting
represent only marginal views (even when counting only wiki users), but force WAY too hard in OSM.
Refer to actual policies, not random Richard phrases.
https://wiki.openstreetmap.org/wiki/Proposal_process
http://wiki.openstreetmap.org/wiki/Good_practice#Document_your_custom-tags
And no @systemed or Richard Fairhurst, an journalist, cartographer and canal person, is nohow experienced in this question. After 10 years in OSM this guy wrote 0 POI catalogues based on OSM data. I don't see why his statements about how to define new tags should be respected.

I don't like crossing=traffic_signals because it breaks duck tagging but people can set it here if they really want to.

Why they don't want to tag pedestrian crossings?!
crossing=traffic_signals was defined/described as "pedestrian crossing" at two pages

And no @systemed or Richard Fairhurst, an journalist, cartographer and canal person, is nohow experienced in this question. After 10 years in OSM this guy wrote 0 POI catalogues based on OSM data. I don't see why his statements about how to define new tags should be respected.

The currency of any open project is measured in meaningful contributions and RichardF has earned more of it than anybody.
Sorry @d1g I'm banning you from the iD issue tracker for a while.  You're becoming a pest and the rest of us here have important work to do.


Template for description of a engineering drawing case

There is a need of template (exemplary description and drawing) of selected case as example of solving and reporting method.

The code for this template can be found in this Jupyter:
https://cocalc.com/projects/cdcf3856-3e8b-4d1c-bcb2-5c42ba749d83/files/UCZESTNICY/Jakub Szydłowski/[Template] ED/guide_engeneering_drawing_PL.ipynb


fix linux compilation

So I got it to compile and run on my machine by fixing the command line handling and the c++14 function handling.
I do have a few problems though. the fonts shaders audio and textures folders all need to be copied to the executable folder or the program won't run. I also wasnt able yet to play any of my songs, but the program runs and I can type into the window.

actually I got it working now!

When I try to load a K-Shoot Map I get the following error
# >  optirun ./Main B.B.K.K.B.K.K./bbkk.ksh
[Main][15:48:26][Warning] Failed to load embeded fallback font
[Main][15:48:26][Info] FPS cap set to 60
[Main][15:48:26][Info] Starting task "Application Setup"
[Main][15:48:26][Warning] Failed to open file for reading Main.cfg: 13
[Main][15:48:26][Info] Starting task "Audio Init"
[Main][15:48:26][Info] Finished task "Audio Init" in  330 ms
[Main][15:48:26][Info] Starting task "GL Init"
[Main][15:48:26][Info] Finished task "GL Init" in  67 ms
[Main][15:48:26][Info] Finished task "Application Setup" in  810 ms
[Main][15:48:26][Info] Starting task "Load Beatmap"
[Main][15:48:26][Info] Starting task "Load KShootMap"
terminate called after throwing an instance of 'std::length_error'
  what():  basic_string::_M_replace_aux

Just running the game without a beatmap results in an more or less empty window

# >  optirun ./Main
[Main][15:50:46][Warning] Failed to load embeded fallback font
[Main][15:50:46][Info] FPS cap set to 60
[Main][15:50:46][Info] Starting task "Application Setup"
[Main][15:50:46][Warning] Failed to open file for reading Main.cfg: 13
[Main][15:50:46][Info] Starting task "Audio Init"
[Main][15:50:52][Info] Finished task "Audio Init" in  6213 ms
[Main][15:50:52][Info] Starting task "GL Init"
[Main][15:50:52][Info] Finished task "GL Init" in  48 ms
[Main][15:50:52][Info] Finished task "Application Setup" in  6670 ms
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/font.vs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35676]] = proj
[Main][15:50:52][Info] Uniform [1, loc=1, Unknown [35676]] = world
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/font.fs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35666]] = color
[Main][15:50:52][Info] Uniform [1, loc=1, Unknown [35678]] = mainTex
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/guiTex.vs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35676]] = proj
[Main][15:50:52][Info] Uniform [1, loc=1, Unknown [35676]] = world
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/guiTex.fs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35666]] = color
[Main][15:50:52][Info] Uniform [1, loc=1, Unknown [35678]] = mainTex
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/guiColor.vs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35676]] = proj
[Main][15:50:52][Info] Uniform [1, loc=1, Unknown [35676]] = world
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/guiColor.fs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35666]] = color
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/diffFrame.vs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35676]] = proj
[Main][15:50:52][Info] Uniform [1, loc=1, Unknown [35676]] = world
[Main][15:50:52][Info] Listing shader uniforms for /build/unnamed-sdvx-clone/bin/shaders/diffFrame.fs
[Main][15:50:52][Info] Uniform [0, loc=0, Unknown [35678]] = frame
[Main][15:50:52][Info] Uniform [1, loc=1, Unknown [35678]] = jacket
[Main][15:50:52][Info] Uniform [2, loc=2, Unknown [5126]] = selected
[Main][15:50:52][Info] Uniform [3, loc=3, Unknown [5126]] = time
[Main][15:51:11][Info] Starting task "Application Cleanup"
[Main][15:51:11][Info] Cleaned up 2 resource(s) of N8Graphics7MeshResE
[Main][15:51:11][Info] Cleaned up 9 resource(s) of N8Graphics10TextureResE
[Main][15:51:11][Info] Cleaned up 8 resource(s) of N8Graphics9ShaderResE
[Main][15:51:11][Info] Cleaned up 1 resource(s) of N8Graphics7FontResE
[Main][15:51:11][Info] Cleaned up 4 resource(s) of N8Graphics11MaterialResE
[Main][15:51:11][Warning] Failed to open file for writing Main.cfg: 13
[Main][15:51:11][Info] Finished task "Application Cleanup" in  61 ms
SIGKILL received


Have a default mode (dark:light)

Would be cool to have a default theme mode, to prevent initial flashing

I'm working on an update to use system defaults — it was supposed to work that way but the JS override (at least the way I've written it now) causes the browser to completely ignore user settings, which is a bummer

this is now fixed as of #36


Release 2.0 is Incompatible with KeePass 2.45 or later

Have you ever thinked about releasing in PLGX Format (https://keepass.info/help/v2_dev/plg_index.html#plgx)  instead  of a bare DLL?

Have exit  KeePaas 2.46 and delete KeePass.exe.config inside program directory as inside it references still to KeePass 2.43 after relaunch of KeePass 2.46 is was created newly with correct references inside


where is the path of jupyter on ubuntu when set ENV["JUPYTER"]?

I don't know how to find the path of jupyter on ubuntu ? I  installed jupyter using navie pip

Run which jupyter at the terminal


Update spring-cloud-netflix.adoc

Remove unintended white-space
Specify imagesdir

@rahmanusta Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@rahmanusta Thank you for signing the Contributor License Agreement!

Current coverage is 74.17% (diff: 100%)

Merging #1578 into master will decrease coverage by 0.05%

@@             master      #1578   diff @@
==========================================
  Files           193        193
  Lines          5937       5937
  Methods           0          0
  Messages          0          0
  Branches        893        893
==========================================
- Hits           4407       4404     -3
- Misses         1206       1208     +2
- Partials        324        325     +1

Powered by Codecov. Last update c380039...f26a555


3.0.2 contains BC breaks!

For some reason 3.0.2 was released with BC breaks - 3.0.1 is allowed to be installed on PHP v8, but 3.0.2 restricts this back down to v7 and its minor versions again. I don't know if its a goal of the project to follow semantic versioning or not but if so this shouldn't have happened.

That looks like an erroneous tag. We'll release v3.0.3 and delete v3.0.2.


proposal: net/http: add Request.CookiesNamed

I'd like to propose to expose (by wrapping) the readCookies method on http.Request as CookiesNamed that takes a single argument which is the filter for the cookie name.
The function signature would be:
func (r *Request) CookiesNamed(name string) []*Cookie

... and the implementation would be as simple as something like this:
// CookiesNamed parses and returns the named HTTP cookies sent with the request
// or an empty slice if none matched.
func (r *Request) CookiesNamed(name string) []*Cookie {
	return readCookies(r.Header, name)
}

Problem statement
Modern browsers may send multiple cookies with the same name if they are present in the cookie storage and are allowed to send. See RFC 6265.
However, there is no convenient way so far to get a slice of Cookies that matches a specific name.
Today, there is only r.Cookies() to get all cookies from the Cookie header and r.Cookie(name) to get the "first" cookie from the Cookie header that matches the given name.
The proposed r.CookiesNamed(name) would solve that by providing a simple way to just read all cookies with a given name.
Proposed Implementation
I have a draft PR ready here, which I'd like to contribute for this proposal: https://github.com/golang/go/pull/61473
Edits

After https://github.com/golang/go/issues/61472#issuecomment-1644204061 I've changed the function name from ReadCookies to CookiesNamed.

If we add this, perhaps call it CookiesNamed, to keep the cookie-fetching functions lexically adjacent?
This doesn't seem unreasonable; it's currently possible to get this information by using req.Cookies and looking for matches, but that does require parsing all cookies rather than just the ones of interest.

If we add this, perhaps call it CookiesNamed, to keep the cookie-fetching functions lexically adjacent?

Sounds good. I've renamed it in the proposal and in https://github.com/golang/go/pull/61473

We recently wrote a routine to remove a cookie by name https://github.com/zalando/skipper/pull/2410
func removeCookie(request *http.Request, name string) bool {
	cookies := request.Cookies()
	hasCookie := false
	for _, c := range cookies {
		if c.Name == name {
			hasCookie = true
			break
		}
	}

	if hasCookie {
		request.Header.Del("Cookie")
		for _, c := range cookies {
			if c.Name != name {
				request.AddCookie(c)
			}
		}
	}
	return hasCookie
}

This would be a bit simpler if we could get all cookies except cookie we'd like to remove.
Another example is selecting cookies having common prefix.
With that in mind and considering that api for trivial case to get single cookie is there I am wondering if it would make sense to support predicate instead of exact match:
func (r *Request) CookiesByName(filter func(string) bool) []*Cookie

though it is not a common pattern, I could only find https://pkg.go.dev/reflect#Value.FieldByNameFunc and https://pkg.go.dev/go/ast#Filter

@AlexanderYastrebov I totally see your point. However, I'm wondering where to draw the line between what the library should provide in terms of filter functionality and what it shouldn't.
For example, if a function like the one you provided would exist, why isn't there one where I could filter for e.g. the Domain or Path or even the cookie value.
I think that the proposed CookiesNamed provides a very low-maintenance / low-risk API for a standard use case in addition to Cookie(name string) - that is getting ALL cookies with a specific name. Actually, CookiesNamed could be considered as a "confusion-free" replacement of Cookie(name string) which returns "just" one Cookie with the given name. However, there may be more than just one.
The current Cookie(name string) kinda suggest otherwise, which is wrong.
IMHO, the existing Cookies(name string) should behave like the proposed CookiesNamed(name string) and the former shouldn't exist, because it's just a source for errors.

The Cookies method returns all cookies. That's nice and simple, and you can filter out whatever you need from that list using whatever criteria you want.
A proliferation of cookie-fetching-and-filtering methods doesn't seem like a good idea. As @timofurrer says, where do we draw the line?
I think the argument in favor of CookiesNamed is fairly compelling, though: It's common to want the cookie with a specific name, but the Cookie method only returns a subset of data associated with that name. Adding a way to get all of it seems reasonable.

Another thought: should we wait to see if the iterator proposal lands and add a cookie iterator instead?

Another thought: should we wait to see if the iterator proposal lands and add a cookie iterator instead?

@carlmjohnson I generally like the idea. However, I still think that the proposed CookiesNamed method is a standard-enough use case to have it's own method that works in the simplest way possible.
IMHO, the cookie iterator could be another proposal to solve problems like the one @AlexanderYastrebov described in https://github.com/golang/go/issues/61472#issuecomment-1646239566. WDYT?

@neild has there been any progress on the proposal status? Is there anything I can assist with to get this over the line?

Since this one missed Go 1.12 I've retargeted the PR to Go 1.22. Any chance of getting this in?

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group
In net/http, add a new Request method CookiesNamed, so that the full set of cookie methods is:
func (r *Request) Cookies() []*Cookie // all cookies
func (r *Request) Cookie(name string) (*Cookie, error) // first with name, or ErrNoCookie
func (r *Request) CookiesNamed(name string) []*Cookie // all with name

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group
In net/http, add a new Request method CookiesNamed, so that the full set of cookie methods is:
func (r *Request) Cookies() []*Cookie // all cookies
func (r *Request) Cookie(name string) (*Cookie, error) // first with name, or ErrNoCookie
func (r *Request) CookiesNamed(name string) []*Cookie // all with name

@timofurrer thanks for the patience and for the hard work! I've just added some comments to your change and after addressing I shall review again and then if gucci, I shall approve; we are almost there and thank you!

Reverted in CL https://go-review.googlesource.com/c/go/+/571277. @timofurrer once the revert lands, please prepare your CL again rebased from the latest master.

Actually nevermind, @dmitshur helped us send a fix to add the api update in commit https://github.com/golang/go/commit/364687bda969d8bb70b1f50b5cce617bbdb094b0. Thank you Dmitri!


Offline installation

OS version: Ubuntu 18.04
Editor version : 1.54.3
Programming language: python
TabNine extension version: 3.2.13

I am connecting to the server via ssh without internet access. And so I can't install tabnine.
A also tried to download vsix and install tabnine from vsix. But it is also not working(
Can I somehow install tabnine on such server?

Hi,
Please contact us to<EMAIL_ADDRESS>and we will try to find a solution