Article correction

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description
There is an article error in the document. I fixed that
Fixes # (issue)

if no issue exists, please create an issue and ask the maintainers about this first

Type of change

Please delete options that are not relevant.

This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions, so we can reproduce.
Please also list any relevant details for your test configuration.

Test A
Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Thank you for showing interest in Appsmith.
Unfortunately, this is a low-quality PR. Please look at our Good First Issues & documentation issues to find relevant issues to improve this project.


Restyle with respect to Brocade style guide

Nice! :+1:

Good work. One minor thing: I think we'll need to change the colour for the highlighted tab (History/Actions/Rules) in the header. Should be something in the Brocade palette that complements the main colour.

Overall I think the third one (red header) probably makes it feel most like a "Brocade" app.

@LindsayHill DZ have picked red one for bwc-ui and blue one for st2web


HADOOP-13327 Output Stream Specification

HADOOP-13327: Add OutputStream + Syncable to the Filesystem Specification

defines what an output stream should do
And what implementations of Syncable MUST do if they declare they support the method.
Consistently declare behaviors in our streams
*Including for some (S3ABlockOutputStream) state tracking: no operations once closed; if an error has occurred, future operations raise it, etc.
With some more utility classes in org.apache.hadoop.fs.impl to aid this

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
0
Docker mode activated.

-1
patch
8
https://github.com/apache/hadoop/pull/564 does not apply to trunk. Rebase required? Wrong Branch? See https://wiki.apache.org/hadoop/HowToContribute for help.

Subsystem
Report/Notes

GITHUB PR
https://github.com/apache/hadoop/pull/564

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-564/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.


Add item for Hybrid Cloud Architecture in context of Hyper scalers also

The intend is to standardize API's specifically service API's  consumed by Edge App providers and that can be unanimously used by hyper scalers as well

Yes, there's no restriction in who can be the consumer of the Edge Cloud APIs


Documentation📄: Update link from GH to docs web site

What is the current documentation state?
In the page at https://github.com/open-horizon/open-horizon.github.io/blob/master/docs/developing/developing_details.md it points to service_def.md with an absolute URL.  This means that a person browsing the documentation web site generated from GitHub will follow that link taking them from the web site to GitHub.  And yet that page also exists in the web site.  Please update the URL to point to the version of the page in the web site: https://open-horizon.github.io/docs/anax/docs/service_def.html
The direct link to GitHub.com occurs three times in developing_details.md.
Where is this stated?
No response
Why do you want to improve the statement?
No response
Proposed Statement
No response
Additional context.
No response

@joewxboy can yopu please assign me 😊

The proper way to resolve this would be to use relative links.  Someone viewing the pages in GitHub, would stay in the github repo. Someone viewing the page from the website would stay on the website.

The proper way to resolve this would be to use relative links. Someone viewing the pages in GitHub, would stay in the github repo. Someone viewing the page from the website would stay on the website.

Ideally, but that would only be possible if the paths were the same.  I don't believe they are in this case.

Right.  Hmm.  Linking between multiple repositories and the website is not optimal.

Closed by #341


Update content to use new Alerts (non-classic) functionality

Alerts (classic) is no longer available from the menu. It is available from within the new Alerts page, however is going to be retired soon, so it would be helpful to update this content to use the new Alerts

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

ID: ecc5e84d-603c-6f88-a941-df33aad0e1d8
Version Independent ID: ae6976bb-aadd-4d40-bb62-57589265f4ff
Content: Monitor published APIs in Azure API Management
Content Source: articles/api-management/api-management-howto-use-azure-monitor.md
Service: api-management
GitHub Login: @vladvino
Microsoft Alias: apimpm

@tonymitchell Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Please assign this issue to @miaojiang.

@mike-urnun-msft Please assign this to @miaojiang.

Thanks for the feedback. It's addressed in this PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/71248


null-safety support

Hello do you have any plans for making this library null-safety?

You will get the Null safe version within this week..
I will get to it soon!

@woogi-kang Thank you  💕 for your request to add the support of null safety to Glassmorphism.
I am happy ☺ to inform you that, the Null safer version is now live and released for use.
You can use the null safe version by the simple steps mentioned below.
Depend on it
Run this command:
With Flutter:
  $ flutter pub add glassmorphism
 Or
dependencies:
glassmorphism: ^2.0.1
Reference :
[Installing  Help](https://pub.dev/packages/glassmorphism/install)

Thank you so much for fast response! We will let you know once we migrate our design to glassmorphism!


Fix weasyprint/pydyf  dependency mismatch that causes an API runtime error (too many arguments passed to pydif function by weasyprint)

When building testrun in the "wild", our computers hit PYPI directly, I'm assuming that Google is probably using a pypi cache internally so the behavior is different.  weasyprint pulls down the latest version of pydyf (0.11.0) which is incompatible with weasyprint 61.2 as specified in testrun's requirements.txt.
Two possible paths for a fix:

specify pydyf==0.8.0 in testrun's requirements.txt
increment testrun's requirements.txt weasyprint requirement from 61.2 to 62.3
I decided to move weasyprint to 62.3, and add pydyf to the requirements.txt and lock it to 0.11.0
We have tested it internally and it resolves the runtime issues with testrun.

Thanks for the pr @caseykelso. An issue that comes up in this PR is the version number for weasyprint. Despite pypi suggests the latest version is 62.3, it seems upto 61.2 is available. It may be due to python version used etc. Check the actions for the error message.

Thanks for the pr @caseykelso. An issue that comes up in this PR is the version number for weasyprint. Despite pypi suggests the latest version is 62.3, it seems upto 61.2 is available. It may be due to python version used etc. Check the actions for the error message.

Hi, does this mean you would prefer to keep the weasyprint at 61.2 in the requirements.txt, and just add a requirement for pydyf to 0.8.0 to resolve the issue?

Thanks for the pr @caseykelso. An issue that comes up in this PR is the version number for weasyprint. Despite pypi suggests the latest version is 62.3, it seems upto 61.2 is available. It may be due to python version used etc. Check the actions for the error message.

Hi, does this mean you would prefer to keep the weasyprint at 61.2 in the requirements.txt, and just add a requirement for pydyf to 0.8.0 to resolve the issue?

That looks to be the only way to get it working unless we can understand why version 62.3 does not seem to be available to us

@caseykelso I have just tried upgrading to 62.3 and it now allows me to. You may have been correct with regards to the Google cache. In version 62.3, the issue with pydyf is resolved so there should be no need to pin it. The issue with weasyprint can be found here: https://github.com/Kozea/WeasyPrint/issues/2200
Could you verify that this PR works for you?

@caseykelso Could you sign the Google CLA as soon as possible so we can get this released in a hotfix? You can do this here: https://cla.developers.google.com/

@caseykelso Could you sign the Google CLA as soon as possible so we can get this released in a hotfix? You can do this here: https://cla.developers.google.com/

No problem, I signed it.

@caseykelso I have just tried upgrading to 62.3 and it now allows me to. It seems that 62.3 relies on Python 3.9 which is not the default python version in ubuntu 20 so that is not a viable upgrade just yet. The issue with weasyprint can be found here: Kozea/WeasyPrint#2200
Could you verify that this PR works for you?

We've already covered this PR in our testing yesterday. weasyprint 62.3 locks the pydyf version to 0.11.0 or greater, and less than 0.12.0, so we'll only get bug patches in theory.
https://github.com/Kozea/WeasyPrint/blob/main/pyproject.toml#L15


Custom blueprint action

I want to handle soft delete/undelete actions in all models. I have overridden destroy action in api/blueprints/destroy.js, hopefully soft delete action is handled now. But I want to handle undelete as well. I have added undelete.js to the same folder as destroy.js, and also I have written a custom route for it in config/routes.js, but in the clinet side I am getting this error "404 (Not Found)", I tried these custom routes but none of them did work:
'GET /undelete/:id': {
blueprint: 'undelete'
}
'GET /:modelIdentity/undelete/:id': {
blueprint: 'undelete'
}
'GET /:model/undelete/:id': {
blueprint: 'undelete'
}
While, when I use the exact name of model in route it works.
Is it possible to write such a route or I have to write a hook?

Just write the code home-slice:
'GET /undelete/:id': {
    controller :'someController',
    action  : 'someAction;
}

@crh3675, your mentioned route is not a blueprint route.


Implement np.mgrid, np.ogrid, & np.r_, np.c_

Numpy has a very convenient np.mgrid which allow to easily create coordinate grids: https://numpy.org/doc/stable/reference/generated/numpy.mgrid.html.
I was expecting jax to have the same, but it seems to be missing:
jnp.mgrid  # raise AttributeError

Thanks! Agreed it would be useful to have a jax-native version of this.

I'd love to solve this. Anyone with a good direction suggestion for me to point to, to solve this issue? @jakevdp @Conchylicultor

To add to the list, np.c_ is missing too: https://numpy.org/doc/stable/reference/generated/numpy.c_.html#numpy.c_

The way to implement these woulc be to create a private class, and override the __getitem__ method to mimic what numpy does. Something like this:
import jax.numpy as jnp

class _Mgrid:
  def __getitem__(self, key):
    if isinstance(key, slice):
      return jnp.arange(key.start or 0, key.stop, key.step or 1)
    else:
      raise NotImplementedError()

mgrid = _Mgrid()
print(mgrid[:5])
# [0 1 2 3 4]

You could also look at the numpy implementation to see how they are implementing it

You could also look at the numpy implementation to see how they are implementing it

But note that if you copy from numpy's implementation rather than writing your own version, the code should go in https://github.com/google/jax/tree/master/jax/_src/third_party for licensing reasons.

And in this case, where there's probably no need to look at NumPy's implementation to implement the API faithfully and well, it's much better to implement from scratch so we don't have to split up the code based on license. In general:

please try not to use code from NumPy
if you do use code from NumPy, it must go in third_party/, and ultimately may need to be rewritten anyway.

Is there an any update for this issue? If not, I would like to progress it :)

I didn't find time for this @minoring
Please go ahead.

Thanks for your response! @VikasHanumegowda

Hi
I'd like to help with this issue if possible
any update?

mgrid has been implemented in #6248, but I don't know of any other work on this

Hi!
I was working on implementing jnp.ogrid.
Maybe you can implement jnp.r_ so that we do not have conflict to each other :)

@minoring Thanks for informing me
Ok, I'm gonna try implementing jnp.r_

Can I take up jnp.s_ if no one else is working on this.

I don't know of anyone working on jnp.s_ – feel free to dive in!

Looks like this issue can be closed since all function have been implemented 🚀
np.s_ was the last remaining function which was added in #7114.

Thanks!


Changing FIP stream on the fly

Hi,
maybe it's already possible, but I'd like to switch FIP channel on the fly without having to open the settings. I'm sure it would be useful! ;)
Thanks for the excellent extension!
marco

Hey!
It's not possible at the moment but it makes total sense. Thanks for raising this idea!

Thanks for the quick reply!
I looked at the code but js is not really my standard bread...
I imagine that after changing the options the app gets restarted with the new stream/new quality.
I'll try to look into it in my spare time, but I don't guarantee anything ;)

You're welcome!
I suspect copy/pasting existing code could work. Have a go at it if it sounds fun to do. Keep me in touch so as I follow back on your attempt.


Cluster Data Replication Problem v.0.12.0-SNAPSHOT

Describe the bug
Cannot replicate data with IoTDB 0.12.0-SNAPSHOT. Lots of "TTransportException: null" exceptions seen on logs.
To Reproduce
Steps to reproduce the behavior:

Download source code

git clone https://github.com/apache/iotdb.git
mv iotdb iotdb-github

Compile with maven

cd iotdb-github/
./mvnw.sh clean package -DskipTests
cp cluster/target/cluster-0.12.0-SNAPSHOT.zip .

Extract zip from target into node1

unzip cluster-0.12.0-SNAPSHOT.zip
mkdir node1
mv lib node1/
mv sbin/ node1/
mv tools/ node1/
mv conf/ node1/

Copy node2 and node3 from node1

cp -rf node1/ node2
cp -rf node1/ node3

Change ports for node2 and node3

sed -i -e 's/6667/6668/g' node2/conf/iotdb-engine.properties
sed -i -e 's/6667/6669/g' node3/conf/iotdb-engine.properties
sed -i -e 's/31999/32000/g' node2/conf/iotdb-env.sh
sed -i -e 's/31999/32001/g' node3/conf/iotdb-env.sh

Start each nodes one by one in seperate terminals

cd node1/sbin
start-server.sh &

cd ../../node2/sbin
start-server.sh -internal_meta_port 9005 -internal_data_port 40012 -cluster_rpc_port 55561  &

cd ../../node3/sbin
start-server.sh -internal_meta_port 9007 -internal_data_port 40014 -cluster_rpc_port 55562  &

Download IoTDB Benchmark tool
git clone https://github.com/thulab/iotdb-benchmark.git
Open with IDE (mine is IntelliJ Ultimate)
Add new maven module as iotdb-0.12
Copy all files from iotdb-0.11 module and replace version in pom file 0.12.0-SNAPSHOT and class package as "cn.edu.tsinghua.iotdb.benchmark.iotdb012"
Replace artifactId as iotdb-0.12 and  name Benchmark IoTDB 0.12 in pom.xml
Replace dependency version as 0.12.0-SNAPSHOT for dependencies; iotdb-jdbc and iotdb-session
Add new version class names in cn.edu.tsinghua.iotdb.benchmark.conf.Constants class;

  public static final String IOTDB012_JDBC_CLASS = "cn.edu.tsinghua.iotdb.benchmark.iotdb012.IoTDB";
  public static final String IOTDB012_SESSION_CLASS = "cn.edu.tsinghua.iotdb.benchmark.iotdb012.IoTDBSession";

Add version 0.12 support in cn.edu.tsinghua.iotdb.benchmark.tsdb.DBFactory class;

            case "0.12.0":
              if(config.getINSERT_MODE().equals(Constants.INSERT_USE_JDBC)) {
                dbClass = Constants.IOTDB012_JDBC_CLASS;
              } else {
                dbClass = Constants.IOTDB012_SESSION_CLASS;
              }
              break;
            case "0.11.0":
            ....

Create dist-dependencies folder in iotdb-0.12 module directory
Create jdbc, service-rpc, session, thrift, tsfile folders into dist-dependencies folder
copy pom.xml and jar files from iotdb-github folder into jdbc, service-rpc, session, thrift, tsfile folders
Manually install SNAPSHOT builds into local maven repository;

cd iotdb-github/dist-dependencies
mvn install:install-file -Dpackaging=pom -Dfile=./pom.xml -DpomFile=./pom.xml
mvn install:install-file -Dfile=./jdbc/target/iotdb-jdbc-0.12.0-SNAPSHOT.jar -Dpackaging=jar -DpomFile=./jdbc/pom.xml
mvn install:install-file -Dfile=./session/target/iotdb-session-0.12.0-SNAPSHOT.jar -Dpackaging=jar -DpomFile=./session/pom.xml
mvn install:install-file -Dfile=./service-rpc/target/service-rpc-0.12.0-SNAPSHOT.jar -Dpackaging=jar -DpomFile=./service-rpc/pom.xml
mvn install:install-file -Dfile=./thrift/target/iotdb-thrift-0.12.0-SNAPSHOT.jar -Dpackaging=jar -DpomFile=./thrift/pom.xml
mvn install:install-file -Dfile=./tsfile/target/tsfile-0.12.0-SNAPSHOT.jar -Dpackaging=jar -DpomFile=./tsfile/pom.xml

Change iotdb-benchmark tool config.properties file as given below, other parameters are left as default;

HOST=<lan ip of your server>
PORT=6667
DB_SWITCH=IoTDB
DB_NAME=plant
BENCHMARK_WORK_MODE=testWithDefaultPath
IS_DELETE_DATA=true
#Whether to create the schema before writing
CREATE_SCHEMA=true
#Write data start time
START_TIME=2021-02-01T00:00:00+03:00
VERSION=0.12.0
OPERATION_PROPORTION=1:0:0:0:0:0:0:0:0
INSERT_MODE=session

maven clean and install iotdb-benchmark tool
Execute test

Expected behavior
I expect data is replicated into all nodes;
cd iotdb-github
cli/sbin/start-cli.sh -p 6667
select s_0 from root.group_1.d_1 LIMIT 3;
<data exists>

cli/sbin/start-cli.sh -p 6668
select s_0 from root.group_1.d_1 LIMIT 3;
<NO data exists>

cli/sbin/start-cli.sh -p 6669
select s_0 from root.group_1.d_1 LIMIT 3;
<NO data exists>

I get some errors which given under "Additional Context" title
I found an FAQ document on this link; https://github.com/apache/iotdb/issues/2270
Translated it into English with Google Translate: https://translate.google.com/translate?sl=auto&tl=en&u=https://github.com/apache/iotdb/issues/2270
According to item one i changed port as 55560 in conf/config.properties file under iotdb-benchmark tool and tried to ingest again,
this time i got Connection Refused exception as given below;
11:39:08.999 [main] ERROR cn.edu.tsinghua.iotdb.benchmark.iotdb012.IoTDBSession - Failed to add session
org.apache.iotdb.rpc.IoTDBConnectionException: org.apache.thrift.transport.TTransportException: java.net.ConnectException: Connection refused
	at org.apache.iotdb.session.SessionConnection.init(SessionConnection.java:91)
	at org.apache.iotdb.session.SessionConnection.<init>(SessionConnection.java:79)
	at org.apache.iotdb.session.Session.constructSessionConnection(Session.java:212)
	at org.apache.iotdb.session.Session.open(Session.java:181)
	at org.apache.iotdb.session.Session.open(Session.java:166)
	at cn.edu.tsinghua.iotdb.benchmark.iotdb012.IoTDBSession.<init>(IoTDBSession.java:38)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at java.base/jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at java.base/jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.base/java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:500)
	at java.base/java.lang.reflect.ReflectAccess.newInstance(ReflectAccess.java:124)
	at java.base/jdk.internal.reflect.ReflectionFactory.newInstance(ReflectionFactory.java:346)
	at java.base/java.lang.Class.newInstance(Class.java:604)
	at cn.edu.tsinghua.iotdb.benchmark.tsdb.DBFactory.getDatabase(DBFactory.java:74)
	at cn.edu.tsinghua.iotdb.benchmark.tsdb.DBWrapper.<init>(DBWrapper.java:33)
	at cn.edu.tsinghua.iotdb.benchmark.App.testWithDefaultPath(App.java:98)
	at cn.edu.tsinghua.iotdb.benchmark.App.main(App.java:68)
Caused by: org.apache.thrift.transport.TTransportException: java.net.ConnectException: Connection refused
	at org.apache.thrift.transport.TSocket.open(TSocket.java:226)
	at org.apache.iotdb.rpc.TElasticFramedTransport.open(TElasticFramedTransport.java:87)
	at org.apache.iotdb.session.SessionConnection.init(SessionConnection.java:89)
	... 16 common frames omitted

Whenever i try to insert using Command Line interface as given below;
cd iotdb-github
cli/sbin/start-cli.sh -p 55560
insert into root.group_d1.d_1(timestamp,value) values(1612126801000,87.74);
# i get data for;
select * from root.group_d1.d_1;

And also i get data by connecting 55561 and 55562 port numbers.
Interestingly when i ingest more data only node1's data folder size is getting larger than node2 and node3 ones.
This means data is actually stored only under node1 data folder.
Ingest Replicated Data with IotDb benchmark tool
At the end of the day i am looking for the correct way to replicate ingest operation to other nodes of cluster with iotdb-benchmark tool.
Desktop (please complete the following information):

OS: Centos Linux
Version: 8 with kernel 4.18.0-240.10.1.el8_3.x86_64

Additional context
Exceptions seen in error log file:
2021-02-15 11:48:51,749 [pool-8-IoTDB-RPC-Client-thread-2] ERROR org.apache.thrift.server.TThreadPoolServer$WorkerProcess:315 - Thrift error occurred during processing of message.
org.apache.thrift.transport.TTransportException: null
	at org.apache.thrift.transport.TIOStreamTransport.read(TIOStreamTransport.java:132)
	at org.apache.thrift.transport.TTransport.readAll(TTransport.java:86)
	at org.apache.iotdb.rpc.TElasticFramedTransport.readFrame(TElasticFramedTransport.java:108)
	at org.apache.iotdb.rpc.TElasticFramedTransport.read(TElasticFramedTransport.java:103)
	at org.apache.thrift.transport.TTransport.readAll(TTransport.java:86)
	at org.apache.thrift.protocol.TBinaryProtocol.readAll(TBinaryProtocol.java:425)
	at org.apache.thrift.protocol.TBinaryProtocol.readI32(TBinaryProtocol.java:321)
	at org.apache.thrift.protocol.TBinaryProtocol.readMessageBegin(TBinaryProtocol.java:225)
	at org.apache.thrift.TBaseProcessor.process(TBaseProcessor.java:27)
	at org.apache.thrift.server.TThreadPoolServer$WorkerProcess.run(TThreadPoolServer.java:310)
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130)
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:630)
	at java.base/java.lang.Thread.run(Thread.java:832)

Thank you very much for your detailed description,

The default distributed RPC port is 55560. I see that you have found the answer to this question.
For cluster version testing, we generally use the support_ cluster branch[1] for testing, and the master is not used for the cluster version.

Interestingly when I ingest more data only node1's data folder size is getting larger than node2 and node3 ones.
This means data is actually stored only under the node1 data folder.

About node1's data folder size is getting bigger than node2 and node3. Because the default copy is 2 copies, and the data is partitioned according to storage group namee+ time, it is not possible to guarantee the same size of the data on the three nodes. You can use this tool[1] to look at data partitioning.

Lots of "TTransportException: null" exceptions seen on logs.

Yes, there are a large number of errors in the currently distributed version, but it does not affect the normal operation and can be ignored for the time being. However, we welcome to find out the essential cause of this problem.

[1] https://github.com/thulab/iotdb-benchmark/tree/support_cluster
[2] http://iotdb.apache.org/UserGuide/Master/System Tools/NodeTool.html

After today, the RPC ports of the cluster version is the same as those in the stand-alone version, namely 6667.


Enable SQL Command text for non-stored procs in EventSource events for .NET Framework.

When using System.Data.SqlClient, .NET Core provides the command text for all SQL operations but .NET Framework only does so for stored procedures in the EventSource events.
This has made it hard in the past to track down problematic SQL Statement by using the EventSource events, for example when using SQL dependency tracking in Application Insights in .NET Framework applications.
Since Microsoft.Data.SqlClient is a separate package, I think we have an excellent opportunity to change this behavior and bring .NET Framework up to speed with .NET Core, to more easily track SQL statements through the event source, without needing to install an additional profiler.
Would love to get input from @lmolkova and @cijothomas for this change as well, since they are familiar with the Application Insights side of things here.

Thanks @stebet This will be a huge value add to customers to get full sql without additional steps, which are sometimes non-trivial (https://docs.microsoft.com/en-us/azure/azure-monitor/app/asp-net-dependencies#advanced-sql-tracking-to-get-full-sql-query)
If this is approved, we need corresponding changes in application insights side as well. Also we want to enable full sql under a feature flag only so as to let users turn it off. (https://github.com/microsoft/ApplicationInsights-aspnetcore/issues/980)
cc: @SergeyKanzhelev @Dmitry-Matveev who would know a lot more of additional historical information about sql as well.

I actually got the SQL command to show up in Application Insights without any changes needed there when I tested this change locally. Don't know if you'd want to change anything other than the feature flag on the Application Insights side of things?
Example from me repro app:
{
    "name": "Microsoft.ApplicationInsights.Dev.RemoteDependency",
    "time": "2019-10-03T09:55:06.9817477Z",
    "tags": {
        "ai.cloud.roleInstance": "*redacted_machine_name*",
        "ai.user.id": "Yrl8e",
        "ai.operation.id": "f6040a3049d9eb43b6d5e729f8cf7b1c",
        "ai.operation.parentId": "|f6040a3049d9eb43b6d5e729f8cf7b1c.3272d3e70294844b.",
        "ai.operation.name": "GET Home/Index",
        "ai.location.ip": "::1",
        "ai.internal.sdkVersion": "rddf:2.11.0-21563",
        "ai.internal.nodeName": "*redacted_machine_name*"
    },
    "data": {
        "baseType": "RemoteDependencyData",
        "baseData": {
            "ver": 2,
            "name": "SELECT name FROM sys.tables",
            "id": "|f6040a3049d9eb43b6d5e729f8cf7b1c.5deeaba979d30045.",
            "data": "SELECT name FROM sys.tables",
            "duration": "00:00:00.0270801",
            "success": true,
            "type": "SQL",
            "target": "*redacted_server_name* | *redacted_db_name*",
            "properties": {
                "DeveloperMode": "true",
                "_MS.ProcessedByMetricExtractors": "(Name:'Dependencies', Ver:'1.1')"
            }
        }
    }
}

My understanding was that this was intentionally limited to SP name only to minimize the for additional security review and exposing any configuration setting.
I'm all for this change. I'd let code owners to decide whether they are willing to expose this information to the customers and improve SQL issues troubleshooting and manageability experiences.

Agreed, especially since this is the default in .NET Core, and moving code over to this new library is a conscious decision requiring code changes, so there is an opportunity here to make this change i.m.o.

we had an agreement with SQL team (@David-Engel) around the instrumentation of Microsoft.SqlClient with DiagnosticSource - it brings other benefits such as end-to-end correlation and flexibility of using raw SQL commands in the payload. AFAIK this in the backlog.

@stebet @cijothomas
Would grab your attention to issue #52 if you can confirm if we can close that issue.
Thanks!

I agree. This seems like a good change in general and for parity between the two targets (netcore/netfx).


Add a way to store rendered "frames"

Is your feature request related to a problem? Please describe.
I want to support both, opengl and vulkan renderers of imgui. However, the opengl renderer is relying on having an object with a limited lifetime passed to the "render" function. At the same time, for Vulkan it is better to separate the "drawing" and "presentation" stages, due to the nature of vulkan. So I was sort of trying to support both APIs by separating the stages for opengl renderer too. However, this would require storing the imgui::Ui object with a limited lifetime which I can't guarantee. In other words, I'd have to own the data in my structure. With current imgui-rs API it is impossible to obtain the imgui draw state (vertex buffers and others) and own them. We only can get a reference to the current frame's data and guarantee its lifetime.
Another problem is that now it is impossible to, let's say,  "draw" a bunch of frames of the imgui, then store these states, and later render those either to a frame buffer of an API of choice or dump to a file, or render without any API to a file, you would be able to do anything with it. The only option now is to render immediately at the same place where the imgui::Ui object is obtained.
Describe the solution you'd like
I'd like to have a way to obtain an owned copy of imgui::DrawData, namely vertex and index buffers, so that I am not limited by the lifetime of a single frame but can postpone the rendering and split it into stages and actually can do whatever I want with this data at any time later.
Currently, the imgui::Context::render returns only a reference to imgui::DrawData that can't be owned. Due to the limitations of imgui::Context, it is also not possible to store it directly as it is neither copyable nor clonable. The only solution now, which is dirty here, is to store an Rc<RefCell<imgui::Context>> that is ugly and not really a solution.
Describe alternatives you've considered
Can't think of any alternatives.
Additional context
imgui::DrawData implements neither Copy nor Clone.

Sorry for the slow reply!
This would be a good thing to support. Main issue is we can't trivially add #[derive(Clone)] to the DrawData as its private members contain pointers to imgui::sys::ImDrawList (and to imgui::sys::ImGuiViewport in the docking branch)
There was a similar request to (de)serialize the draw data in #606 which would work in this case - the solution there was to basically have a duplicate set of structs:
https://github.com/rmccrystal/imgui-rs-serialize/blob/a3e2288ee48a718153fd327041b7780fa5a32dae/src/draw_data.rs
We could provide these in imgui-rs itself (maybe as DrawDataOwned), but that has some downsides (like do renderers need to take both imgui::DrawData and imgui::DrawDataOwned?)
I suspect possibly having something like imgui-rs-serialize as a separate crate might be the best solution, given this is a somewhat uncommon use case - any thoughts?

Hello, and first of all, thank you for writing this library!
I stumbled across this issue as I am also working on integrating imgui-rs into a multithreaded renderer (Bevy), and also need a means of cloning and temporarily storing an ImDrawData object so that it can be passed to the render thread. In Bevy, the only time in which the CPU and the renderer are synchronising is during the 'Extract' phase, which is supposed to be as cheap as possible. So, it would be advantageous if the clone didn't depend on third party serialisation libraries here. The solution would also need to be able to expose a DrawData object, since, as you've pointed out in your reply above, many ImGui renderers already operate on this object.
When looking into potential avenues here, I came across this issue in the ImGui repo and the use case feels very similar to https://github.com/ocornut/imgui/issues/2646. So I thought I'd try prototyping a solution, which you can find here:
https://github.com/jbrd/imgui-rs/commit/a6454917989daba33f6bb7432fba64ac46df758a
It'd be great to get your thoughts on this. Essentially, the idea is to introduce an OwnedDrawData struct that holds a cheaply-cloned ImDrawData structure that has been allocated and cloned on the heap. It leverages ImDrawList::CloneOutput for cloning the underlying draw lists, and all allocations are made via ImGui, which has a nice consequence of them being tracked in the IO object's allocation metrics. The OwnedDrawData struct provides an accessor to the underlying DrawData object, so that it can still be passed to renderers without modifying them.
Would something like this be possible to provide?

That sounds pretty much perfect - it's consistent with the upstream library, plus it solves the concern of having two objects for renderers to deal with, and avoids having to keep the duplicated structs up to date :partying_face:

Excellent, thanks both! I'll try to get this tidied up and into a PR over the new few days...


Adding language switcher to alternative panels

I was wondering if there was a simple way supported by this package to allow adding only the language switcher to other panels. I have two panels, admin where the translations are also managed, and the user dashboard panel. I seem to only be able to load the plugin on the dashboard panel with the translations management page included.
Any easy way to only load the switcher, or to deny viewing the management page / removing the management page from the sidebar?

good idea we can try to do it on the next version.

Use Language Switcher
You can use the language switcher in your panel by using the following plugin:
$panel->plugin(\TomatoPHP\FilamentTranslations\FilamentTranslationsPlugin::make())

NOTE Your auth user table must have lang filed on the table to make this switch work fine.
from v1.0.9

Use Language Switcher
You can use the language switcher in your panel by using the following plugin:
$panel->plugin(\TomatoPHP\FilamentTranslations\FilamentTranslationsPlugin::make())

NOTE Your auth user table must have lang filed on the table to make this switch work fine.
from v1.0.9

Yes, but this also comes with the translations page. That's where the problem lies.

Yap! That adds it, unfortunately it seems actually switching doesnt work.
When i switch from english to dutch, it'll reload the page but not switch the translation or change the locale in the database.

Do you add 'lang' to your user table?

Yes, the lang column was added

can you let me see your panel provider?

<?php

namespace App\Providers\Filament;

use App\Filament\Dashboard\Pages\Tenancy\EditTeamProfile;
use App\Filament\Dashboard\Pages\Tenancy\RegisterTeam;
use App\Http\Middleware\ApplyTenantScopes;
use App\Models\Team;
use Filament\Http\Middleware\Authenticate;
use Filament\Http\Middleware\DisableBladeIconComponents;
use Filament\Http\Middleware\DispatchServingFilamentEvent;
use Filament\Pages;
use Filament\Panel;
use Filament\PanelProvider;
use Filament\Support\Colors\Color;
use Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse;
use Illuminate\Cookie\Middleware\EncryptCookies;
use Illuminate\Foundation\Http\Middleware\VerifyCsrfToken;
use Illuminate\Routing\Middleware\SubstituteBindings;
use Illuminate\Session\Middleware\AuthenticateSession;
use Illuminate\Session\Middleware\StartSession;
use Illuminate\View\Middleware\ShareErrorsFromSession;

class DashboardPanelProvider extends PanelProvider
{
    public function panel(Panel $panel): Panel
    {
        return $panel
            ->id('dashboard')
            ->path('dashboard')
            ->login()
            ->registration(env('APP_ENV') !== 'production' ? Pages\Auth\Register::class : null)
            ->profile()
            ->tenant(Team::class, ownershipRelationship: 'team', slugAttribute: 'slug')
            ->tenantRegistration(RegisterTeam::class)
            ->tenantProfile(EditTeamProfile::class)
            ->tenantMiddleware([
                ApplyTenantScopes::class,
            ], true)
            ->favicon(asset('images/favicon/favicon.ico'))
            ->colors([
                'primary' => Color::Blue,
            ])
            ->discoverResources(in: app_path('Filament/Dashboard/Resources'), for: 'App\\Filament\\Dashboard\\Resources')
            ->discoverPages(in: app_path('Filament/Dashboard/Pages'), for: 'App\\Filament\\Dashboard\\Pages')
            ->pages([
                Pages\Dashboard::class,
            ])
            ->discoverWidgets(in: app_path('Filament/Dashboard/Widgets'), for: 'App\\Filament\\Dashboard\\Widgets')
            ->widgets([
            ])

            ->plugin(\TomatoPHP\FilamentTranslations\FilamentTranslationsSwitcherPlugin::make())
            ->middleware([
                EncryptCookies::class,
                AddQueuedCookiesToResponse::class,
                StartSession::class,
                AuthenticateSession::class,
                ShareErrorsFromSession::class,
                VerifyCsrfToken::class,
                SubstituteBindings::class,
                DisableBladeIconComponents::class,
                DispatchServingFilamentEvent::class,
            ])
            ->authMiddleware([
                Authenticate::class,
            ]);
    }
}

As you can see i apply only the switcher, but the issue is there regardless of having both plugins or only the switcher, this is on version 1.0.9

i have same issue with the language switcher, switching does not make any effect

yes i found the problem and i working on fix it ASAP

it's working now you can check the last version
https://github.com/tomatophp/filament-translations/releases/tag/v1.0.10

@MiataBoy thanks for supporting me with your panel provider.


[TableView] Allow focus to set on both header and the subsequent dialog action button with keyboard

🙋 Feature Request
Allow keyboard navigation to focus on table view column as well as a dialog action button if there is one in the column header.
🤔 Expected Behavior

When using keyboard navigation, user can first focus on the column header before focus next sets to the dialog action button within the column header.
User can still select the column header if the table allows column sorting
Voice over announcement on the column header still work

😯 Current Behavior
Using keyboard navigation, the focus jumps directly to the column's action button if there is one, and user does not have a chance to focus on the column header itself.
💁 Possible Solution
🔦 Context
For tables, there could be some use cases where where we want to include a dialog trigger with an action button to display a small dialog for more information about the column.
Currently with this behavior, the keyboard focus sets on the action button, but there is no way for user to use keyboard to focus on the column header itself.
Where when you tab into focus on the table, you can see when you use arrow keys on the table column, the focus immediately sets to the dialog's action button instead of the column header.
I've created a sandbox in the example section below which can hopefully give a better illustration on the request.
💻 Examples

I have a sample code here for reference:
https://codesandbox.io/s/zen-fermi-zbo48?file=/src/App.js

🧢 Your Company/Team
Adobe AEP
🎁 Tracking Ticket (optional)

Can't have nested interactive elements, as the whole column header is clickable to sort. Closing out issue - will work with team to find alternative place for column header info


:lady_beetle: Bug - Better missed error handling

Problem
When the elm-land client JS code connects to the server and there's a compiler error, in some cases the message for the compiler error is sent to the client before the client is ready to handle it. The client code detects the situation and tries to fix it by refreshing the page if there is no error or compiler output after a timeout, but the problem can happen again after the refresh.  For me, this meant up to 6 or so refreshes before the error appeared sometimes.
Solution
Rather than the server sending messages to the client as soon as it connects, it waits for the client to send an `elm:client-ready event.
Notes
This was done in a few stages, so the first commit just replaces the refresh with a message that triggers a re-send of the error. I also made what I thought was a minor code tidying change while I was in there that I will call out below in case it was a mistake.

Oh no! Editor autoformat strikes again!  I will get this down to a reasonable diff and update it.

I fixed all of the auto-format changes, and I don't think I introduced any fresh mistakes in the process,


docker version of juicer

A docker version of juicer would be helpful for all

https://github.com/theaidenlab/Juicer-Docker
https://github.com/ENCODE-DCC/hic-pipeline
On Fri, Oct 8, 2021 at 10:34 PM seqyuan @.***> wrote:

A docker version of juicer would be helpful for all
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aidenlab/juicer/issues/244, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK2EWZ44EKCPYGVJL3DM2DUF6S5FANCNFSM5FUYAEXA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--
Neva Cherniavsky Durand, Ph.D. | she, her, hers
Senior Scientist |  Gene Regulation Observatory
Broad Institute of MIT and Harvard

In the README, there is no mention of the Juicer Docker images, hosted here I believe: https://hub.docker.com/r/aidenlab/juicer/tags
And the last release was six years ago which seems outdated
Similarly the link to the ENCODE images is: https://hub.docker.com/r/encodedcc/hic-pipeline/tags
Perhaps adding this information in the README would be useful :-)


lbfgs untargeted attack support

See #1000 for detail description and  code to reproduce error.

Selecting the second label as a target may not always be the best thing to do in the untargeted setting. Would you be able to instead modify the loss in the untargeted case so that rather than maximizing the score of the target class, the attack minimizes the score of the correct class until the input is misclassified? This would likely be more generally applicable

@HaojieYuan did you get a chance to look at this?

@npapernot  Sorry for the delay, I've been working on other stuff recently.
I will try to figure it out next weekend.

@npapernot
Can I have my name on the Authors list in README.md if this PR is merged?
I think I've already satisfied the requirments:
The following authors contributed 100 lines or more (ordered according to the GitHub contributors page)

:laughing: it will be super cool for me~
HaojieYuan (University of Science and Technology of China)


env not working?

Using the latest Portainer.
I read this page and followed the instructions: https://www.portainer.io/blog/using-env-files-in-stacks-with-portainer
But I get this error when trying to deploy a stack?

What did I do wrong?

@YouveGotMeowxy first thing I would try is a minor formatting adjustment. Instead of
env_file:
   - 'stack.env'

change it to
env_file: stack.env
As a general recommendation, I would suggest using full paths wherever possible to alleviate issues. For example, my portainer env file is at /var/data/config/portainer/portainer.env so my compose file contains env_file: /var/data/config/portainer/portainer.env.

Hi @jdenson
tyvm for replying. Apparently, no one else seems to give a hoot. :-p
I am using portainer's env feature to add the variables:

I clicked the "load from .env file" button, selected the .env file, it filled in all the env stuff in the portainer fields, and then after reading the link in my above post, I was under the impression that I didn't need to manually create the .env file anywhere else, and portainer would do it for me?
Maybe you could whip up a quick test container and add some env variables using portainer's env feature, and see if it happens to you as well?
p.s. I did try the format adjustment you suggested, but unfortunately had the same result. I also tried both format styles with and w/out the quotes.

@jdenson ty again for helping. :)
Just out of curiosity, have you read the link in my OP? It gives the impression (at least) that Portainer's supposed to create it for you, like it does the usual docker_compose.yml (DC) when you use it to create a stack.
Normally you cant tell which folder Portainer is going to create in it's subdir for it's DC, since it creates them as a number. In order to find it, you have to either just click through them looking for it, or look at the number in the browser's location bar. Seems like a weird thing to have to do that, then create or copy the env to it in order for the stack to be successfully created.

I'm wondering if I need to mount the env as a volume first, and then use the path of the mounted env file. hmmm.
shouts through bullhorn Can we get a Portainer employee up in here?! lol

I'll spin up a portainer instance in a bit and double check, but as far as I'm aware there isn't any situation where portainer will generate an actual env file. I would recommend creating the file and placing it in a known location, then change your config to env_file: /path/to/filename for the time being.

You've mentioned you're using the latest version of Portainer but you haven't specified whether you're using CE (2.11.1) or BE (2.12.2). I'm assuming CE.
If you define environment variables via the UI and want to include them all in your stack without referencing them individually, you can add the stack.env reference to the service in your compose file. You do not upload a file called stack.env - the file is created automatically by Portainer.
Here's an example nginx stack I just spun up to test this:

Note the stack.env reference in the service, and the two environment variables (MY_ENV_1 and MY_ENV_2) I have defined.
When I deploy the stack, if I now check the nginx container it generated I can see those environment variables exist and are set:

Can you try the above and see if you still have issues?

Hi @jamescarppe 🌝
Yes, I am using CE, on Swarm.
Your steps are exactly what i did, but the only difference might be that i clicked "load from env file", selected an env file that i already had loaded with many names:values, and portainer then populated the fields in the ui.
After that i cannot deploy because of the error shown in the screenshot.

This currently doesn't work with Swarm, as you've surmised - I suspect this is because Swarm uses docker stack deploy to deploy whereas Standalone uses docker-compose, and there'll be some syntax and capability differences there. We're talking with the team internally around this, but in the meantime you can individually define each environment variable in your compose file (once you've populated the fields from your file) to get them into your stack:
environment:
  - MY_ENV_1=$MY_ENV_1
  - MY_ENV_2=$MY_ENV_2

It's less than ideal especially when you have a lot of vars, but it'd at least get you running.

Not sure I understand the point it not being supported in docker stack deploy, if I do docker stack deploy outside of Portainer with a reference to an absolute path to a .env file (in a compose file), it works just fine, but as soon as I put the same compose into a new stack in Portainer, it fails with an error with it unable to find the absolute path.  Using CE 2.14 myself. I'm not that well versed in the swamp of containers so just trying to get an understanding :)

I'm also a bit confused with the same issue as @Slipehr. When using docker stack deploy -c stack.yml test and the stack.yml contains:
version: '3.8'

services:
  test:
    image: ubuntu
    command: env
    env_file: stack.env

And there is a stack.env file containing e.g.:
MYVAR1=something
FOO=bar

It succeeds and prints in the log:
<EMAIL_ADDRESS>  | PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
<EMAIL_ADDRESS>  | HOSTNAME=9194cb8346cc
<EMAIL_ADDRESS>  | FOO=bar
<EMAIL_ADDRESS>  | MYVAR1=something
<EMAIL_ADDRESS>  | HOME=/root

So it should be possible to do in Portainer, too I think.
@jamescarppe I didn't see your note until at first sight. It may be easier to see if it used the same formatting as e.g. the note about "compose file is not changed when environment variables are used" (the box with blue side bar and blue info icon).

I might be over-simplifying this, but at the surface, since Portainer already creates a compose.yml file in a numbered folder, why not also add a whatever.env file there as well, and add the entered env vars to it, and use that when handling that stack's actions?

i am running into the exact same problem after reading the exact same article. I have a compose file that references somename.env file. I load the somename.env file from my machine to Portainer UI (CE latest) and I get an error saying that env file does not exist.

This is driving me absolutely nuts.
The docker-compose file works outside of Portainer, specifying the .env file
When attempting to launch with Portainer, there is an error that the .env file cannot be found.
If you add the full path of the .env file, you get a similar message, but it seems to be looking for the .env in
/data/compose/
NOT in the specified full path.
If we look in this document
https://www.portainer.io/blog/using-env-files-in-stacks-with-portainer
There are a couple of options offered-

importing the environment variables file. This is not suitable because the file exists on the Server that Portianer is running on- it is not local, and not shared or accessible via GUI
Specifying the full path. This does not appear to work because you are assuming that the stack is built inside Portainer

So, how can we reference a .env file that exists external to Portainer?

 I just encountered the exact issue but it turns out i had to change 2 rows of env_file


Operation name dynamically genarated

Hello Team,
During WSDL import in Azure API Management (APIM),  dynamic operation ID values are generated, which are displayed as ‘Name’ on the operation Frontend UI. Currently, there is no way to control this before or during the import.
However, our official documentation states that the following:
"The following fields are filled automatically with information from the SOAP API: Display name, Name, Description."
This implies that the field name is automatically imported from the import. Since we know that this is not what happens, and that they are dynamically genarated, can we have the documentation updated?”
Thank you in advance!

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

ID: 2f393748-f909-a584-a15f-b64e45955d79
Version Independent ID: afb5cb8d-b707-e24e-0683-d295bab506e2
Content: Import SOAP API to Azure API Management
Content Source: articles/api-management/import-soap-api.md
Service: api-management
GitHub Login: @dlepow
Microsoft Alias: danlep

@leaarone
Thanks for your feedback! We will investigate and update as appropriate.

@leaarone - Thanks for pointing this out. I'll submit a doc update shortly with the correction.


Add relations cache busting

@patrixr Here's an implementation of an option to bust related caches on POST, PUT and DELETE calls (as per proposed in #13). It piggybacks on the existing clearCache helper (that way, if enabled, the related caches will always be busted) and is enabled via a clearRelatedCache middleware option.
What it does basically is:

Gather the related models by inspecting the updated model's relations to other models.
It then goes looking for components that have the updated model as a relation. We go two levels deep in the components to cover use cases such as a relation in a component that is then included in a dynamic zone.
It then gathers the models where the related components are used.
Finally, it constructs cache keys for the related models and adds them to the existing rexps array to be busted alongside the updated model's cache.

It works well, I've tested it in a working Strapi installation. I'm not 100% happy with the structure, there's some repeated code. I've tried to spin off the functionality in a separate helper but it became too convoluted and hard to follow. At least the way it is now it's easy to follow albeit a little clumsy.
Let me know what you think! I can write tests if you agree with the approach, I'd have to add dummy models in the strapi instance in the tests, as it lacks the strapi.models, strapi.plugins and strapi.components objects needed to gather the related models.

It then goes looking for components that have the updated model as a relation. We go two levels deep in the components to cover use cases such as a relation in a component that is then included in a dynamic zone.

Great choice :wink:

It then goes looking for components that have the updated model as a relation. We go two levels deep in the components to cover use cases such as a relation in a component that is then included in a dynamic zone.

Great choice :wink:

It then goes looking for components that have the updated model as a relation. We go two levels deep in the components to cover use cases such as a relation in a component that is then included in a dynamic zone.

Great choice 😉

I rely heavily on relations embedded in components so I already had such use cases to test against. @derrickmehaffy it's actually you that gave me the insight yo use relations that way :)

It then goes looking for components that have the updated model as a relation. We go two levels deep in the components to cover use cases such as a relation in a component that is then included in a dynamic zone.

Great choice 😉

I rely heavily on relations embedded in components so I already had such use cases to test against. @derrickmehaffy it's actually you that gave me the insight yo use relations that way :)

Hi @julesrenaud
Thanks for this ! I think the approach would work nicely, but I think maybe we should move a bit of that code out to a helper, maybe something like:
        if (clearRelatedCache) {
          const relatedModels = getRelatedModels(model); // move the population code to its own function

          // prep related models to have their whole cache busted
          _.each(relatedModels, (relatedModel, key) => {
              const relatedModelSlug = relatedModel.kind === 'collectionType'
                ? relatedModel.collectionName
                : key;

              rexps.push(new RegExp(`^/${relatedModelSlug}`))  // Bust all entries from models that have relations with that model
          })
        }

This would:

Keep the cache busting / cache key code in the same file and function
Extract the relational aspect of the code to its own file/helper, which can maintain the current step-by-step approach that you're using

Regarding tests, do add any mocks needed. I am aware it is quite a hacky setup currently, we're not (yet) striving for perfection but adding some basic tests to smoke out regression will go a long way :)
Cheers 🍺  !

Hi @julesrenaud
Thanks for this ! I think the approach would work nicely, but I think maybe we should move a bit of that code out to a helper, maybe something like:
        if (clearRelatedCache) {
          const relatedModels = getRelatedModels(model); // move the population code to its own function

          // prep related models to have their whole cache busted
          _.each(relatedModels, (relatedModel, key) => {
              const relatedModelSlug = relatedModel.kind === 'collectionType'
                ? relatedModel.collectionName
                : key;

              rexps.push(new RegExp(`^/${relatedModelSlug}`))  // Bust all entries from models that have relations with that model
          })
        }

This would:

Keep the cache busting / cache key code in the same file and function
Extract the relational aspect of the code to its own file/helper, which can maintain the current step-by-step approach that you're using

Regarding tests, do add any mocks needed. I am aware it is quite a hacky setup currently, we're not (yet) striving for perfection but adding some basic tests to smoke out regression will go a long way :)
Cheers 🍺  !

@patrixr agreed, I'll go ahead and implement these changes

@patrixr agreed, I'll go ahead and implement these changes

@patrixr @derrickmehaffy the populating code has been moved to a separate helper as suggested. I've retained for...in and for...of loops in the helper to leverage execution termination. The strapi object in the tests has also been mocked to match a real-use data structure. I've added the researcher collection type which is directly related to the academy model and is related to the homepage model via components (2-level deep). The tests are successful on my end, the cache is busted for the related model in both cases.

@patrixr @derrickmehaffy the populating code has been moved to a separate helper as suggested. I've retained for...in and for...of loops in the helper to leverage execution termination. The strapi object in the tests has also been mocked to match a real-use data structure. I've added the researcher collection type which is directly related to the academy model and is related to the homepage model via components (2-level deep). The tests are successful on my end, the cache is busted for the related model in both cases.

@julesrenaud
Looks good to me ! Well done on the specs !
Few small things:

Could you bump the version to 1.4.0 ? (for the autopublish when we merge)
Could you edit the README to mention this feature ? I would do it myself but since my understanding of the logic is only rudimentary I think you'd do a better job at describing it

Thanks ! 🤜 🤛

@julesrenaud
Looks good to me ! Well done on the specs !
Few small things:

Could you bump the version to 1.4.0 ? (for the autopublish when we merge)
Could you edit the README to mention this feature ? I would do it myself but since my understanding of the logic is only rudimentary I think you'd do a better job at describing it

Thanks ! 🤜 🤛

@patrixr there you go!

@patrixr there you go!

Great ! Thank you (again) for the contribution :)
Version 1.4.0 is now on npm

Great ! Thank you (again) for the contribution :)
Version 1.4.0 is now on npm


SPARK-4660: Use correct class loader in JavaSerializer (copy of PR #3840...

... by Piotr Kolaczkowski)

Test build #25794 has started for   PR 4113 at commit a5e84ca.

This patch merges cleanly.

Test build #25794 has finished for   PR 4113 at commit a5e84ca.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/25794/
Test PASSed.

Thanks - I'll pull this into master and the older branches. You can close the other PR's.


Cannot clear Messages while query is running

Issue Type: Feature Request
In SSMS it is allowed to clear messages while query is running. Please allow the same in Azure Data Studio.
Azure Data Studio version: azuredatastudio-insiders 1.14.0-insider (c70e7794ebe1bdff82aafc149ae104ba587bd1a6, 2019-11-05T08:55:52.327Z)
OS version: Windows_NT x64 10.0.18362

Thanks for the feedback @GrzegorzLyp. Are you requesting that the notifications in the bottom right should clear once a query is run?

I have some tools that produce some information about e.g. progress via "print" command in T-SQL. In SSMS I am allowed to clear this messages as the query is in progress. In Azure Data Studio such messages take a lot of RAM and I want to be able to clear them before query if complete.


[Android] APK (release) working fine on emulator and device, published AAB in close testing fails.

Description
Tried building the demo variant of my app in release mode. Dragged the file to the simulator (works), tested on a physical device (Pixel 6) working fine. But when I generate an app bundle, and publish it for close testing, the app fails to pass the login screen. (Google approved as well). Just wondering what could cause this sort of mix up.
Version
0.67.3
Output of npx react-native info
System:
OS: macOS 12.0
CPU: (8) x64 Apple M1 Pro
Memory: 34.02 MB / 16.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 16.13.1 - ~/.nvm/versions/node/v16.13.1/bin/node
Yarn: Not Found
npm: 8.1.4 - ~/.nvm/versions/node/v16.13.1/bin/npm
Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman
Managers:
CocoaPods: 1.11.2 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 21.0.1, iOS 15.0, macOS 12.0, tvOS 15.0, watchOS 8.0
Android SDK: Not Found
IDEs:
Android Studio: 2020.3 AI-203.7717.56.2031.7784292
Xcode: 13.1/13A1030d - /usr/bin/xcodebuild
Languages:
Java: 11.0.10 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: ^17.0.2 => 17.0.2
react-native: ^0.67.3 => 0.67.3
Steps to reproduce
cd android && ./gradlew bundleRelease for app build builds.
Snack, code example, screenshot, or link to a repository
    signingConfigs {
        debug {
            storeFile file('debug.keystore')
            storePassword 'android'
            keyAlias 'androiddebugkey'
            keyPassword 'android'
        }
         release {
            if (project.hasProperty('MYAPP_UPLOAD_STORE_FILE')) {
                storeFile file(MYAPP_UPLOAD_STORE_FILE)
                storePassword MYAPP_UPLOAD_STORE_PASSWORD
                keyAlias MYAPP_UPLOAD_KEY_ALIAS
                keyPassword MYAPP_UPLOAD_KEY_PASSWORD
            }
        }
    }

and publish it for close testing, the app fails to pass the login screen

Could you clarify what's the failure here?

Every component I written works when I build locally. On local the app is pointing to the right to env db, gcp authentication is working... etc, whenever I generate a signed APK and download it to the simulator or my physical device, it works as expected. However, every time i generate aab (appbundle) to upload to the google play store (for closed testing). liek 10% of my components fail (certain pickers, keyboard offset for certains screens are wrong) etc. How can I test if its my code or if its something that has to do with generated app bundle?


Errors in Numberscope while developing should be traceable

When developing a visualizer and you get an error, instead of going to the console, it goes to a dialog popup.  Which is the desired behaviour we added.  Except that it doesn't say useful info like what line caused the error.  So it's more difficult to debug now.  Is there a way to either let the error info go to the console also, with a line number, or something else?

I second the motion to have a debug mode that just lets the error occur rather than catching it. What I do at the moment is comment out the following lines in main.ts as needed:
app.config.errorHandler = e => {
    window.alert(alertMessage(e))
}

Closing as a duplicate of #387, which is resolved.


WIP Distribute GAS to voters

Close #1348 .

Take a look at neo-project/neo#2049 also, this will require updating a number of tests that you touch here anyway.

    wallet_test.go:209:
        	Error Trace:	wallet_test.go:209
        	Error:      	Not equal:
        	            	expected: 0
        	            	actual  : 1
        	Test:       	TestClaimGas

Why does it happen?

@roman-khimov I am investigating. It happens with -race flag. Most likely, the problem is that we can't determine height when claim tx will be persisted, but this height is needed to calculate expected amount of GAS claimed.


Release Google.Analytics.Admin.V1Alpha version 2.0.0-alpha18

Changes in this release:
New features

Change netstandard2.1 target to netstandard2.0 (commit 82bea85)

Created release for Google.Analytics.Admin.V1Alpha-2.0.0-alpha18


⚠ Processors has degraded performance

In 77cd2dd, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 38 ms

Resolved: Processors performance has improved in fcc9f50.


Default values for all methods

train_val_test_split needs default values for stratify_cols and for train test val split sizes. Check rest of methods for other default values.

A fix for this will be released in 0.17a


Feature request: utility functions to allow stopping meta-gradient propagation

Hi there, it would be really useful if higher's API allowed blocking gradients at a point in the graph
Use Cases:

When working with Actor Critic models, I'd want to be able to allow gradients to flow through the critic when training it, but block them when using the critic to train the actor. It would be useful if i could make a detached copy of the critic for the second part.

When working with meta-gradients that don't go over the whole training loop (eg only use meta-gradients on K unrolled steps), it would be useful if we could detach the models so gradients don't flow past K.

Proposed API
The simplest and most intuitive way in my opinion is to make the differentiable optimizers use with pytorch's existing .state_dict() and .load_state_dict() , so we can call them on pytorch's normal modules and optimziers like:
nn_module_model.load_state_dict(functional_model.state_dict())
normal_optimizer.load_state_dict(differentiable_optimizer.state_dict())

To continue training with meta gradients, we can then re-create the higher versions of these.
The functional models already do this, but the differentiable optimizers don't. I've hacked together a patch that works for my use case (2-step unrolling), but I doubt would work in general.
An extension for convenience
Lastly, a more convenient (but less intuitive) extension to this idea would be to have the old .detach() and .detach_() methods for torch tensors apply to the Functional models. it didn't amke sense for nn.Modules since they weren't differentiable, but it would be so convenient to for Functional models since they (their parameters) are now differentiable too:

detach() should create a fresh copy of the whole functional model that is cut from the computation graph, which is really useful for the actor critic case:

# critic needs to be trained and backprop its meta gradients
critic_loss = F.mse(
    fmodel_critic(state,action),
    target_q_value
)

# actor also needs to be trained and backprop its meta-gradients, but critic used in training it should NOT backprop policy's loss into its own parameters
fmodel_critic_for_training_actor = fmodel_critic.detach() # copy of fmodel_critic that doesn't alter original's params
actor_loss = - fmodel_critic_for_training_actor(
    state,
    fmodel_actor(state)
)
# backpropagation and updates now ignore the detached copy
diffopt_critic.step(critic_loss)
diffopt_actor.step(actor_loss)

detach_() should detach all param tensors in place. This would also be really useful for the and more efficient for the K step unroll use case since we don't need to deep copy back to a pytorch nn.Module model, then copy again to a functional model to continue. With this .detach_() version we can just do that in place :

with  higher.innerloop_ctx(model,optim) as (fmodel,diffoptim):
    for i in range(num_steps): # training loop
        for k in range(meta_gradient_steps): # inner loop i need to backprop through
            ....#get losses
            train_step(fmodel,loss,diffoptim)
        ... # do meta updates
        fmodel = fmodel.detach_() # block backprop here by detaching all params in palce

Of course, these are just for convenience and efficiency since the state_dict methods could functionally do the same things, albeit slower

Thanks, this is super. Just FYI, I am on leave all of November, and most of December. If this is urgent, please flag this here. I will try and find time when I return early December to implement an appropriate and robust state_dict method for patched modules and differentiable optimizers which does the right thing.
In the meantime, if you fancy having a stab at doing this yourself, we welcome contributions and it would be very helpful indeed.

Hi !
I could also really use those type of utility function !
Because for now the only working way, i found, to stop the meta-gradient propagation (and save of weights of the K last states) with an arbitrary K is to perform a complete copy of the model / optimizer states and gradients before re-instantiation of fmodel and diffoptim.
I also tried to directly alter the fast_weight memory of the fmodel to remove states from the memory but it didn't stop the memory to ever increase with each iteration until saturation of the GPU...

@AntoineHX I saw no speedup in doing this detach version over just copying (in my experiments with 1-step meta-gradients it was actually slower), and I'm still not sure why.
I'll investigate this again in about a month to figure out why, but for now I'm just getting by with the explicit copy version
The "memory leak" is caused by the autograd graph for old iterations being accidentally held onto by these persistent tensors:

DiffOpt state
DiffOpt param groups
Fast model weights

So you need to detach both of them after you've finished the loops on optimization and meta-optimization.
If you'd like to try it, I made this .detach_() method for the DifferentiableOptimizer base class:
    def detach_(self):
        """Removes all params from their compute graph in place."""
        # detach param groups
        for group in self.param_groups:
            for k, v in group.items():
                if isinstance(v,_torch.Tensor):
                    v.detach_().requires_grad_()

        # detach state
        for state_dict in self.state:
            for k,v_dict in state_dict.items():
                if isinstance(k,_torch.Tensor): k.detach_().requires_grad_()
                for k2,v2 in v_dict.items():
                    if isinstance(v2,_torch.Tensor):
                        v2.detach_().requires_grad_()

@ihexx Thanks for the quick reply !
I tried to use your function but for me, it didn't work as intended. After using .detach_(), at the end of next inner iteration, the diffopt.step(loss) get a

RuntimeError: Trying to backward through the graph a second time, but the buffers have already been freed. Specify retain_graph=True when calling backward the first time.

maybe it's because i had to change your function so that v2.detach_().requires_grad_() to v2=v2.detach().requires_grad_()  so that it isn't performed in-place (because views cannot be detached in-place).
I don't really have the time right now to dig any deeper so I'll believe you that it doesn't get any faster than the copy version !

@AntoineHX @ihexx We will look at this issue next week. In the meantime, in #15 we fixed some memory leak issues from #6. Could you please let us know if you are still seeing memory leaks after pulling a fresh version of the repo (and updating higher in your env if you pip installed it)?

Hello. I've returned from leave and allocated some time to look into this issue over the next two weeks. Hopefully we'll make some progress and report back, or come back to you with questions.

@ihexx Note the patched modules already support this, i.e. you can do something like
module = ...
fmodule = higher.monkeypatch(module)
with torch.nograd():
    for p in module.parameters():
        p.add_(1)  # modify original params so that fmodule params are different
module.load_state_dict(fmodule.state_dict())  # restore params from fmodule

Does this fit your needs on the module side of things?
I'll look at the optimizers later this week or early next week as there are other bugs that have higher priority.

Hi,
I am wondering are there any updates on the state_dict() for the optimizers?


Old web page of unitaryhack can be mistaken for the new unitaryhack.dev page

This repository is for the 2021 edition of unitaryhack: unitaryfund.github.io/unitaryhack/
If I google unitaryhack the first page that I get is this one instead of the new one. This can be confusing to participants, especially since the year is not very visible in the 2021 page.
Any idea how fix this problem? E.g. adding a big text comment, automatically redirect, temporarily close, etc.
@natestemen, @nathanshammah

I see there being two courses of action:

Remove the GitHub pages deployment setup (this can be done easily on GitHub under the settings for this repo), and archive[^1] the repo. This is definitely the easiest way, with no future management required.
Change the GitHub pages deployment so that this website is deployed to a subdomain of unitaryhack.dev such as 2021.unitaryhack.dev for archival purposes. This should be straightforward following the docs. Once the new site is deployed to a URL that indicates it is not the canonical location to find information on the current unitaryhack, the repo can be archived[^2] so no future changes can be made (which adds another indication this is not up to date). This solution should hopefully require no maintenance, but it could.

I think both options are fine, but I do like keeping webpages up. I'm happy to take care of this, but if we decide on option 2, I will need access to the UF domains so I can add the appropriate DNS records.
[^1]: This is what Xanadu does. See e.g., 2021 qhack: https://github.com/XanaduAI/QHack2021 and 2022 qhack: https://github.com/XanaduAI/QHack.
[^2]: I couldn't find much online as to whether GitHub pages continues to run once the repo has been archived, but this community post indicates that it does.

Agreed, a banner here across all pages is a good start.
I also think putting this site under a subdomain of unitaryhack.dev is quite simple as per the docs I shared in the previous comment.

To get started, I opened a PR for the readme #59.

If someone (@francespoblete @natestemen or @WrathfulSpatula for example) could add a banner in the old website, is also a good quick fix to follow up with.

@natestemen, I like your idea of a subdomain on the new website (under /2021).

Looks like this is addressed, thanks @natestemen!


Make history table topic more helpful

Helpfulness rating is very low.

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

ID: 390f9258-6f7c-dbe8-3e79-c83446ba6192
Version Independent ID: 38896138-0566-d481-2c91-8158f6d8f274
Content: Custom Migrations History Table - EF Core
Content Source: entity-framework/core/managing-schemas/migrations/history-table.md
Product: entity-framework
GitHub Login: @bricelam
Microsoft Alias: bricelam

Closing this issue: on 10/10/18 SkyRay showed 1k page views and 0 ratings for the last 30 days. The number of ratings is not enough to draw reliable conclusions; when we do the next sweep for high-traffic/low ratings docs, we can reopen this one if this doc meets those criteria.


arch() is not recognized as Pest test function

arch() is not recognized as Pest test. Therefore it can't be fired up form IDE separately via mouse or keyboard shortcut.
Screenshots

Desktop:
PhpStorm 2023.3.4
Build #PS-233.14475.35, built on February 13, 2024
Runtime version: 17.0.10+1-b1087.17 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o.
Windows 10.0
GC: G1 Young Generation, G1 Old Generation
Memory: 2048M
Cores: 8
Registry:
debugger.new.tool.window.layout=true
run.processes.with.pty=TRUE
Non-Bundled Plugins:
Dummy Text Generator (1.2.0)
Batch Scripts Support (1.0.13)
com.intellij.ideolog (<IP_ADDRESS>)
com.intellij.properties (233.11799.172)
String Manipulation (9.12.0)
ru.adelf.idea.dotenv (2024.1)
de.espend.idea.php.annotation (9.4.0)
com.laravel_idea.plugin (<IP_ADDRESS>)

@pestretsov was a new release ever created since the merge of https://github.com/JetBrains/pest-intellij/pull/510 ?

@olivernybroe Unfortunately, no. #510 and the following changes will be available in PhpStorm 2024.1


Don't use Ansi Colors if not supported.

We are using this tool in an environment that doesn't support ANSI Color code. So we have made a small change to enable them only when the environment supports them.

We referred to this website to construct the support_ansi_colors function
https://bixense.com/clicolors/

I have this issue as well so would love to see this merged ☝

I added a test coverage for utils/color. It's the first time I'm using pytest, so please tell me if I can improve the test.
I made and tried it on an Ubuntu machine. But else, I am using detect-secrets on Windows. I never tried to run the code test on Windows though.

The pytest implementation looks good. I just suggested possibly a few function name changes to describe what the test is doing. Thanks for your contribution!

Thanks for your contribution! I'd wait until we push #528 so we can further test this against Windows before formally approving it.

Closing and re-opening this PR to run the additional Windows checks from #528.


BF: RU Update

I assume you're aware, but most if not all of these Pokemon have existing entries in RU. Also, please put the newline back.

Sorry, looks like I forgot to remove the old sets when I added the new ones. Done, and put the newline back too.


Not able to use net20 library in net461 project

Description
I'm having a problem after upgrading Paket to 3.4.0. Our project is net461, and we have framework: net461 at the top of our paket.dependencies. However, we use a package which only exists for net20. We used to have the reference to this project in paket.references, but added a reference to the dll manually. The new version of paket breaks this though. It removes our manually added reference, but does not add a reference to the dll, probably as its net20 and not net461. We tried a workaround by removing it from paket.references and adding it manually, but this breaks on our build server as we're using paket.targets which runs restore directly against the paket.references file and, as we removed the reference, won't download the package. Preferable, we would like paket to add this to our project even though a specific version doesn't exist for net461 (which works automatically for libraries >=net40 it seems). Is there a way to force this behavior? Or other workarounds which doesn't involve changing paket.targets or our build server?
Repro steps
Run paket install in repo https://github.com/simendsjo/paket-net20-library-problem
Expected behavior
net20 library is referenced in net461 project
Actual behavior
net20 library is not referenced
Known workarounds
None
Related information

Operating system: Windows 10
Paket: 3.4.0
.NET: 4.6.1

Yesterday someone else reported something similar (but closed it again) . But this is such a basic feature, how could we break it and didn't notice in integration tests?  This will be first thing after my vacation.

ok found 5min while packing my stuff for vacation. it seems this case actually never really worked. There is a lot going on (so there is much do to after vacation), but could you please test latest alpha? I think I found the cause and hacked a bit around it.

Problem was that EPPlus has a readme.txt file in the lib folder. The penalty for the "empty" lib folder was not high enough...

So there is a pr for this?

Currently it is part of https://github.com/fsprojects/Paket/pull/1785 as I was working on that part of the codebase anyway and it was a good test case. Sorry. I think you could cherry pick the last commit if you want to release this (the fix was the increment of the constant). I try to get the PR to a mergable state next week. Figuring out why the build was red just took way too much time :)

cherry pick the last two commits

done.

fixes are in latest alpha. I will create proper release later today


Arash's current email address?

Hi Arash--my apologies for writing you here, I did not know where else to look. We are trying to find your current email address (VLDB Slog resubmission). If you can please email me<EMAIL_ADDRESS>so we can make an account for you, I would appreciate it.

Hi Kris. I emailed you from my personal email address.


User token vs application/client token

Hello,
This isn't an issue, just a question regarding some vocabulary in the OAuth2 knowledge space. But I believe this is the most knowledgeable place where my question could be answered. At least I hope someone can clarify this for me.
As we all know the OAuth2 framework specifies the issuance of JWT tokens either using the client credentials flow, which places client/application claims on the token, and the other flows which do place user claims on the token. This distinction is enforced by the spec of JWT: https://tools.ietf.org/html/rfc7523#section-3

The JWT MUST contain a "sub" (subject) claim identifying the
principal that is the subject of the JWT.  Two cases need to be
differentiated:

    A.  For the authorization grant, the subject typically
        identifies an authorized accessor for which the access token
        is being requested (i.e., the resource owner or an
        authorized delegate), but in some cases, may be a
        pseudonymous identifier or other value denoting an anonymous
        user.

    B.  For client authentication, the subject MUST be the
        "client_id" of the OAuth client.

But there doesn't seem to be the same enforcement of distinction between types of identity claims on the token with regards to when the resource-server will receive that token. In certain situations you can, at the resource-server level, distinguish whether the token was issued for a user or a non-user client (for ex. identityserver fills the subject with the guid of the user vs the client_id, as the spec says), but there doesn't seem to be a clear distinction between the two.
Would it be wrong or introduce confusion, or any other negative effect that you see, to make a distinction between the two token types? Calling one a so-called user token and the other an application/client token? It makes sense to me to name them differently, since in some circumstances you will likely want to distinguish between the two. The most likely scenario that comes to mind is an audit scenario, where you want to know if the request token was made on behalf of a particular user or just a generic client.
I would really appreciate some feedback from you knowledgeable ladies and gentlemen. Thank you and have a good one.

Well - the spec you are citing above is confusing. It is about using JWTs as client secrets.
In IS we made the decision for the JWTs we issue that sub always refers to a human. We use client_id to identify a client. This way we minimize chances they are parsed incorrectly.

Well - the spec you are citing above is confusing. It is about using JWTs as client secrets.
In IS we made the decision for the JWTs we issue that sub always refers to a human. We use client_id to identify a client. This way we minimize chances they are parsed incorrectly (or mixed up).

So, you made the explicit choice to not overload the subject field with either human identifier or a client identifier? And instead use one, the subject, for human identifier and the client_id as a client identifier.
So, I guess you would agree that you see no issue in distinguishing between the two types of tokens? One a "user token" and the other an "application/client token"?

our access tokens always have a client_id claim to identify the client. In addition - IF a user is present - there is also a sub claim.

Thank you for the information. This helped a lot.


How to specify the state of traffic lights in the scenario?

I am running a scenario using the scenario test runner, and I wonder how to specify the state of traffic lights in the TIER IV Scenario Format Version 2.0. It seems that the scenario simulator v2 does not follow the OpenScenario v1.2 specification.
Below are my scenario and map files, which I used to test traffic light behaviors. The scenario includes a simple ego routing story and sets all traffic lights to red. (I have checked the syntax against other scenario files, though there might be an error.) I expected the ego car to stop at the junction because of the traffic light, but it ignores it and reaches the destination.

test_scenario_map.zip

The code below is the part related to traffic lights in the scenario file.
RoadNetwork:
  LogicFile:
    filepath: lanelet2_map.osm
  SceneGraphFile:
    filepath: point_cloud.pcd
  TrafficSignals:
      TrafficSignalController:
      - name: test_signals
        Phase:
        - duration: INF
          name: Stop
          TrafficSignalState:
          - state: red solidOn circle
            trafficSignalId: '494'
          - state: red solidOn circle
            trafficSignalId: '520'
          - state: red solidOn circle
            trafficSignalId: '586'
          - state: red solidOn circle
            trafficSignalId: '553'
Storyboard:
    Init:
      Actions:
        GlobalAction:
        - InfrastructureAction:
            TrafficSignalAction:
              TrafficSignalControllerAction:
                phase: Stop
                trafficSignalControllerRef: test_signals

@hcn1519
Hi, thank you for reporting!
Since the traffic light message type differs depending on the version of autoware, it is necessary to appropriately switch the traffic light topic output from the scenario_simulator_v2 side.
You can switch it by switching the architecture_type of scenario_test_runner.
For older Autoware, awf/universe is appropriate, and for newer Autoware, awf/universe/20230906 is appropriate.
If nothing is specified, awf/universe will be used, so the traffic light will not connect to new autoware.
We are aware of this issue and are working with #1232 .

It seems that the scenario simulator v2 does not follow the OpenScenario v1.2 specification.

The user guide provides a scenario for specifying a traffic light using OpenDRIVE as an example.
However, in reality, as shown below, this is just an example, and it is not stipulated in the standard that it must be followed.
description of TrafficSignalState::state

State of the signal, e.g. the visual information "off;off;on". The number of values in the state depends on the traffic signal in the roadnetwork , e.g. type and subtype in OpenDRIVE

Autoware uses lanelet2 as its map format instead of OpenDRIVE, so please understand that although it is within the scope of the OpenSCENARIO standard, the specifications are different from the scenario in the user guide, which assumes OpenDRIVE.

Thank you for your answer @HansRobo
I test to use awf/universe/20230906, and it works. Also, I appreciate your additional answer regarding specification.

If you are still having problems please reopen this or create a new issue.


⚠ Bofrost.si has degraded performance

In 8bcfa82, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 2373 ms

Resolved: Bofrost.si performance has improved in 12a4658.


OpenAI API Error: 400 - Bad Request

Describe the bug:
When trying to generate a commit message it says the request is too long
Steps to reproduce:
aicommits --all                                                                                ─╯
┌   aicommits
│
◇  Detected 3 staged files:
     dist/app.js
     dist/app.js.map
     src/app.ts
│
◇  Changes analyzed
│
└  ✖ OpenAI API Error: 400 - Bad Request

{
  "error": {
    "message": "This model's maximum context length is 4097 tokens. However, you requested 4103 tokens (3903 in the messages, 200 in the completion). Please reduce the length of the messages or completion.",
    "type": "invalid_request_error",
    "param": "messages",
    "code": "context_length_exceeded"
  }
}

Expected behavior:
it works

Hi, I understand your concern, but it appears that you are not using our product, but rather https://github.com/Nutlope/aicommits. Our product solves this problem by segmenting text content. We welcome you to try it out: https://plugins.jetbrains.com/plugin/21289-aicommit 😸


💡[Feature]: Want to add HOUSE PRICE PREDICTION

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Bases on surrounding house price predictions
Use Case
Easy to understand about price
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I have worked on this model . Please assign me this project, My model has high accuracy and other ML metrics with reference to this aspect


Change visibility of config file constant

https://github.com/ink8bit/deby/blob/9fd51eae37fd4987b78bd0c660855a7ef19b62cf/src/config/mod.rs#L28
pub(crate) const CONFIG_FILE_NAME: &str = ".debyrc";

should be private


📜 0.2.4 release

this branch intends to release a new version of Viceroy.
since our last version, we have merged #69 and #70. the former is a bug-fix worth releasing soon, so let's get it out there!
i'm following the instructions added here, so upon approval I'll push a tag and publish the crates.

One note: the release process notes don't seem to include publishing to crates.io, and we have not yet published 0.2.3 there. Would you mind addressing that as well?

@aturon, i've retroactively published the 0.2.3 crates (see below) as well as introducing some extra documentation about publishing in ad6489a. :sparkles:

https://crates.io/crates/viceroy/versions
https://crates.io/crates/viceroy-lib/versions


Fixes catalog adds prefixes or suffixes to avoid mutual influence

Proposed changes

es catalog adds prefixes or suffixes to avoid mutual influence
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run clickbench


Github Releases and Tags are not up to date

Hi Jean,
I am performing a review of NeuNorm for JOSS under the ticket (https://github.com/openjournals/joss-reviews/issues/815)
I notice that the version submitted to JOSS is 1.4.0, but I can't seem to find either a Github or tag for this release (or the newer 1.4.1) that is on PyPI.
I'll create separate issues here for any other comments and suggestions if that's ok ?
Cheers
Stu

Sorry about that. I just created the 1.4.1 tag. Bad habits I have of creating pypi and conda versions without creating a tag first. I will try not to do it again.


[Docs] Need to include Localization support extension guides

Is your suggestion related to a missing or misleading document? Please describe.

We used to have documentation on extending the localization support in the Login Portals & My Account / Console.

https://is.docs.wso2.com/en/6.1.0/references/extend/localization-support-in-identity-server/#!

But in the latest in progress IS 7.0 docs, we don't have this documented.
Describe the improvement

Need to validate the previous content and include the guides.
⚠ Need to specially make sure that the approach is U2 safe (specially in Console / My Account)

This doc implementation is hold since there is an issue with the customization. Issue created [1]
[1] https://github.com/wso2/product-is/issues/19529

This doc is now available: https://is.docs.wso2.com/en/latest/guides/branding/localization/


Create Project in a diferent domain - GAM Install

Hi, If i have a GSuite domain A.com, is it posible to create the GAM Project in the GAM instalation process in the GCP domain of B.com?

Please move this to the discussion group:
https://groups.google.com/forum/#!forum/google-apps-manager


Feature: Add Point manually

It would be nice to add points manually. This way it would be possible to keep the measurement running when hitting another point. For example whole site duration or multiple tasks. Or you could finish a point by its label. For example
Performance::point("site:start");
//run code
Performance::point("cache");
//run code
Performance::finish(); // finish cache point
//run code
Performance::finish(); // finish site:start

Or

Performance::finish("cache");
Performance::finish("site:start");

It's a good idea. I will add this functionality in version 2.3.1. Note that if you use this multiple point function the measurement are less accurate. This is because you will measure your task and also the performance tool.
For more info over the multiple point function, see example 12 in de example folder on git.

Code and documentation is updated. See https://github.com/bvanhoekelen/performance/wiki/Multiple-points and https://github.com/bvanhoekelen/performance/releases/tag/v2.3.1


Add missing import cirq

Fix notebook, this is failing in the site import pipeline.

Done.

Thanks!


Create a helper function for creating stoplights

This pull request creates a Stoplight helper function that acts as an alias for Stoplight::Light.new. This is purely for convenience and doesn't add any new functionality.
# Before
Stoplight::Light.new('example') { nil }.run
# After
Stoplight('example') { nil }.run

I got the idea from circuit_b. I'm not convinced that Stoplight is the right name for this function. Some other options include:

stoplight: This seems like it would often conflict with a local variable.
Stoplight.light: This has the benefit of existing within the stoplight module, but it's still a little verbose.

I'm also not convinced this is the right functionality. I wish I could create a function that would run the light after any fluent builders. For example:
magical_function('much') { :result }
# => :result
magical_function('many') { :fluent }.with_timeout(3)
# => :fluent

But I'm pretty sure that's impossible. So maybe this Stoplight function could be a shortcut for running a stoplight. You would have to switch to Stoplight::Light.new for the fluent builders, but you'd be able to write this:
Stoplight('such') { :wow }
# => :wow

I'm not sure. That's probably not enough of a win.

Hey @camdez, can you look this over?

We discussed this in person and agreed that it seems reasonable.


How can we track changes done with set method?

I am using set method at multiple places for updating fields.
This doesnot track updates when made using 'set' method.
How can I achieve this ?

You can't today since it's one that doesn't invoke callbacks and is atomic.


enable inference for RAMB36E1

I'm going to put off adding support for 8/16/32-bit widths for later, since it's not trivial and not needed right now.
@litghost What value do you suggest for min bits?

@litghost What value do you suggest for min bits?

Let's leave it alone.

Have you tested the relevant circuits in symbiflow?  E.g. murax, picosoc and scalable proc?  Which of these now instance the larger primitive?

@HackerFoo Please report hardware testing status for:

[ ] Scalable proc
[ ] Murax
[ ] Picosoc

I haven't tried scalable proc yet
Murax works, but doesn't use 36k blocks
PicoSoC does use them, but doesn't work with or without them, so I need to find a working configuration.

top_bram_n3 also doesn't work without 36k RAM inference.
rx:'054BB395' pat:'054BB395' V
rx:'06D2F54C' pat:'06D2F54C' V
Got 5 consecutive matches. In sync.
MISMATCH! (rx='4C736D44', pat='A3C391EB'), Sync lost!
rx:'A3C391EB' pat:'A3C391EB' V
rx:'0150BAB0' pat:'0150BAB0' V
rx:'0060ED28' pat:'0060ED28' V
rx:'054BB395' pat:'054BB395' V
rx:'06D2F54C' pat:'06D2F54C' V
Got 5 consecutive matches. In sync.
MISMATCH! (rx='4C736D44', pat='A3C391EB'), Sync lost!
rx:'A3C391EB' pat:'A3C391EB' V
rx:'0150BAB0' pat:'0150BAB0' V

top_bram_n3 also doesn't work without 36k RAM inference.

Does top_bram_n1?

top_bram_n1 fails too.


Investigate XA usage

I did some more tests.
I am assuming non xa behavior, and it behaves as if it was supporting it.
specifically those tests should fail (and do not):

sending and inserting in the same tx (required/required)
receiving and inserting in the same tx (required/required)

there should be an error saying that you can't enlist 2 non-xa resource managers in the same tx
if there is no error, it is either:

we are not running in the same tx (I don't think that is the case)
both resource managers are xa
one resource manager is xa, and nayarana enables LRCO under the cover

it is probably one of the last 2 situations. if so we should have the tx logs somewhere (e.g. data\tx-object-store) and a recovery manager?? and that would make the application stateful. we need to investigate that.
the default behavior for quarkus should assume the app to be stateless (12 factors, microservice, ...)
Originally posted by @vsevel in https://github.com/quarkiverse/quarkus-ironjacamar/issues/25#issuecomment-1705939888

Actually that's not an IronJacamar issue, but a transaction question. Being discussed in https://quarkusio.zulipchat.com/#narrow/stream/187030-users/topic/jms.20with.20quarkus-jca


Fix automated tests by constraining pytest versions

https://github.com/mitodl/pylti/issues/58

Coverage remained the same at 100.0% when pulling dc5b4b13b305f43ebd7def9ebe6ca1fbf6a32456 on nikolas:fix-test into<PHONE_NUMBER>42b08cce4bf2a13bb20f212fcc8f98 on mitodl:master.

Coverage remained the same at 100.0% when pulling 108fb2790d20a021490f3a31da6088a6599d4e01 on nikolas:fix-test into<PHONE_NUMBER>42b08cce4bf2a13bb20f212fcc8f98 on mitodl:master.

@pwilkins this can be merged, and should be done soon, as tests are currently failing on master.

@nikolas thanks for your contribution:
I am going to close this because the problem has been fixed on master.


SQLLAB AutoComplete: add space after double quotation mark

autocomplete feature in sqllab add space after selection it works nice when there is not any ", but when the name is within a double quotation marks, the desired behaviour is adding space after double quotation.
Environment
(please complete the following information):

browser type and version: Chrome Version 95.0.4638.69 (Official Build) (64-bit)
superset version:
python version: 3.8.12
node.js version: node -v

This does indeed still seem to be happening on current-era Superset. Curious if @yousoph @eschutho have heard any complaints about this one.

Can you provide some more details, how to reproduce the issue?

The repro steps are:

type a double quote
start typing a column for it to auto complete
click/tab to let autocomplete happen
observe that the space is inside the quotes, rather than outside.

Here's a GIF to illustrate

I'd like to help fix this bug. Can you guide me?

@AnalogAnt that would be great... what kind of guidance do you need? I have no idea what's causing the issue, but if you can track it down in the code, I'm happy to help you through the PR process. Please feel free to reach out on the #contributing channel on Superset Slack if you want to talk about that process, or start a thread in #debug-together if you want help with the diagnostic parts.

I need help identifying the part of the code to change for the bug fix. Being new to open source, I could use some guidance.

I can't reproduce this in 3.1.0 or HEAD.

Same here... seems we're good now!


Update jquery.socialfeed.js

Mostly syntactical fixes, along with adding missing semicolons and brackets for better readability and safer processing.

Thank you!

Once I get to the Point where I find some use for it, I will add some more useful stuff
Andy

Am 10.02.2015 um 23:42 schrieb Pavel Kucherbaev<EMAIL_ADDRESS>Thank you!
—
Reply to this email directly or view it on GitHub.

Hey!
New message, please read http://www.hellousablog.com/watched.php?lmsjl
Andy Fuchs