fix optimize kerbero options, remove BE kinit, just use UGI login by newInstanceFromKeytab #29291

Proposed changes
from https://github.com/apache/doris/pull/29291

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


Add simple file mod support

This is a quick one. I often found that when I wanted to test modded asset on OpenKH game engine, I had to modify the original game files and often accidentally losing the source.
This adds mod support. By default it tries to find files in the mod folder. If the file is found, it loads the modded one. If it is not, it is going to fall back to the original file.
Of course you can set any path from the config.yml. I personally set the VFS of CC's DynamicLoader.

Excited to test this when I get home!


multiplier for reconstruction loss in vae

Codecov Report

Merging #457 (d3e96fc) into master (7a3a0fa) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #457   +/-   ##
=======================================
  Coverage   95.35%   95.35%
=======================================
  Files         131      131
  Lines        4633     4633
=======================================
  Hits         4418     4418
  Misses        215      215

Flag
Coverage Δ

unittests
95.35% <ø> (ø)

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

Files Changed
Coverage Δ

domainlab/models/model_diva.py
97.61% <ø> (ø)

domainlab/models/model_hduva.py
95.83% <ø> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Update system message with the date of latest rebuild/redeploy

GitHub seems to lag - commit message with magic word fix didn't close it automatically so have to close it myself


McLighting Mesh ??

Hi Tobias, Reading about the Android app :-) Is it an (great) idea to implement a mesh for all Mclithing esp's ?? ESP8266WiFiMesh or Wifiudpclient/server (#ifdef) or even BROADCAST?
Not sure yet if there need to be a mas of code to be changed (like Server.on /....... etc)
But know for sure many would like to control ALL there McLighting esp's without a Mosquito/mqtt server but from the buildin webpage.
to be cont.....................
Regards Leon

i think if u want to control all of them from the app i dont need to change anything on the esp. iam planning right know to implement more then one device in my app so that i could even call the function to post the http request to all of them. a mesh is more a way i think that all devices communicate with each other but the app can handlr more devices and chamge all of them so the work is done by tje app not the esp so we could keep the code on tje esp smaller and more efficient

When one has a Android phone, yes,  but for iphone there's no app.
LV

yeah but its the same thing, i think there is too much work for the esp to get this working the approach with an rpi running only mqtt is like 20 bucks??? and time beeing maybe we goet someone taking the same approach in creating an iphone app or something.

A big TODO, not an easy implmentation! For now the recommendation is to use MQTT and all clients connect to same topic. Alternatively as mentioned here call multiple rest API from one place (like an app). I've no app building experience.
I think WiFiMesh is a great solution for adhoc wifi network. When I have time I will try to get some examples of painlessMesh working. https://gitlab.com/painlessMesh/painlessMesh/blob/master/examples/webServer/webServer.ino

Never done any mesh networking ever, so ideas welcome here! I will create a separate branch (or even a separate project) and see how this progresses.
As mentioned in https://github.com/toblum/McLighting/issues/140 it will require having a stand-alone web interface (I've no experience in building one), or if others have ideas on how to connect just one node to WiFi and rest to on the mesh.

Hi @LeonVos,
using a mesh network sounds interesting but is not easy to implement in such a way that it works flawlessly for all users. I'd like to have it as easy and stable as possible.
@TechOnlyGamer approach to send the same command to a number of ESPs looks like a easier solution.
I also started working on a alternative web UI that could talk to more than one ESP, but it will take some time to find a nice and stable solution.
Best regards
Tobias

Update: I have something working as of now,

Uses painlessMesh to create mesh network and broadcasts state to every node
Connects to known wifi network predefined in code (no wifimanager, have to create custom webpage for this)
Web interface is borrowed from WS2812FX esp8266 example, completely served on ESP8266
Can do minimal file upload to SPIFFs
Completely Async!
Uses Task Scheduler, no more tickers
No delays in entire code
Has RESTful API (Async WebServer)
Async Websockets (working on it)
Async MQTT (not implemented as of now)

@toblum It does not have the McLighing interface that you have here, but do you want me to create a branch for people to try?

if there is an RestAPI say Hello to my Android App

Hi @debsahu,
that sounds great. Can't wait to try that out. Will see if I can make the webinterface work for that. Is the API mainly the same?
Sure, create a branch as you like.
Regards
Tobias

@toblum Async Websockets and Async MQTT API are the same. RESTful API is working and is same as ws282fx example for now. How do you test WebSockets, i.e. your favorite client?
Currently working on WiFiManager to set WiFi for internet, for now it is hard-coded. Also exporting tickers to Task Scheduler. Work in progress!

@toblum I created develop and experimental. Since this is experimental I will add my code there, not ready for public consumption yet. I cant delete any of your branches, I think only you can do that by going to 'settings'. I also recall that one can restore deleted branches, if needed as well.

Hi @debsahu,
I deleted some branches now. Much better overview. I hope I didn't delete too much.
I was using this chrome extension: https://chrome.google.com/webstore/detail/simple-websocket-client/pfdhoblngboilpfeibdedpjgfnlcodoo It has all the basic features and "just works".
Best regards
Tobias

Anyone here willing to try the Meshy McLighting? https://github.com/toblum/McLighting/tree/experimental/Arduino/MeshyMcLighting
Meshy McLighting Server
WiFi and MQTT settings are still hard coded (Any ideas of making use of something like WiFiManager or something you have come across is appreciated). WiFi channel of the router needs to be manually specified, usually its channel 6 in USA as default.
Meshy McLighting Clients
Do not enable WEBSERVER for now, not being used. WiFi channel must be the same as the WiFi router and Server.
@toblum created a favicon.ico, I dont know if you like it? It is there for me to see if an icon is displayed on chrome, we can change it because I spent a whole 3 min to creating it! And what I need help with is it correctly implemented in the webpages <link rel='shortcut icon' type='image/x-icon' href='favicon.ico' />? I dont see the icon thats why I'm asking.

Hi @debsahu,
I hope I'll find time at the weekend to try it out. So little time at the moment. I'm also working on an alternative UI for McLighting (https://github.com/toblum/McLightingUI). I'm trying to have a first version ready soon.
MeshyMcLighting seems to have a similar API, so it should be possible to build a UI that works on both. Did I see this right? So both together could be the new and improved v3 of McLighting. What do you think?
Regarding Favicon: I like it. But if we choose an icon we should have also bigger versions of it, so that we can provide homescreen icons for mobile devices. I havn't done that so far, because I didn't find a cool icon. Maybe some of the users has also some skill to create one.
All I can say again is that I'm totally impressed with your work. You can do in a short time what would have cost me months and years. Thanks again for your contribution.
Best regards
Tobias

Hi Deb,
this looks great. I like how reliable the sync happens. This is a much better approach than my experiments with websocket calls to multiple nodes. This isn't really reliable as the ESP struggles with many WS connections open.
I see that the master node can also work standalone. I first thought that you always need at least two nodes, but this is great. So this version could easily replace the current implementation.
Regarding power consuption: I don't think it's a problem in most cases, as the strips draw the most power. But is it possible to turn off mesh functionality if you don't need it to save power? I didn't see that.
As the API is almost similar, I will try to make the new UI work with your implementation. Should be easy.
I had to order fresh ESPs and strips to have a meshy setup in parallel to my existing one. I hope they will arrive soon.
Greetings
Tobias

@toblum A few things to note,

WS2812FX + painlessMesh + TaskScheduler causes crashes. Even with setting no delays on loop there is some interference of SDK call espShow() in NeoPixels library. NeoAnimationFX + painlessMesh + TaskScheduler has no issues.
From my numerous experiments, IwIP 1.4 higher bandwidth (default settings on platformio, not default on ArduinoIDE) is reliable and IwIP 2.0 is a mess.
For using McLightingUI (also in the video), ws calls are exactly the same so I replaced ":81" with "/ws", very minimal code change.
I have a Server without Mesh (uses native TaskScheduler and not painlessMesh's stripped down version) working non-stop (300LEDs), still testing on WS2812FX and NeoAnimationFX settings. I am having trouble with mDNS and DNS for some reason. I will iron this out when I have time and create a #define DISABLE_MESH_SETTINGS that can be toggled in definitions.h. Or I can upload what I got and someone can pitch in with creating this definition.

Jep, looks like the mesh was a good brainer of me , @debsahu GOOD JOB.
@toblum, nice job on the new controls.
And some new thing about the wifimanager, checkout ;
https://github.com/kentaylor/WiFiManager#portal-continues-to-exist-after-configuration
Ken has leveled this up, now when on dhcp you get feedback from the wifimanager on connection status and addresses, great idea.

@toblum use IwIPv1.4 Higher-Bandwidth

Also what is the serial output on the client side? It should say connected to AP. On the server Serial output, it should say the WiFi channel that the clients needs compiling, does it match the WiFi channel on the router? I see that in your output you posted it says it's connected to 1 node, so I'm guessing client is connected? Could you change color or something ok the server to trigger a broadcast and see if syncing with client happens?

I had already selected this. But after reflashing multiple times it works now. I don't know what was wrong.
I have now one server with two clients. They all sync, but it takes always about 5 seconds until a change is reflected at the clients. In your video it was a instant reaction. Do you know that behaviour?

Ha yes. My local version is different only in that aspect, sorry I'll upload the changes version once I get to my computer. What you can do is replace all if(!taskSendMessage.isEnabled()) taskSendMessage.enableDelayed(TASK_SECOND * 5); with taskSendMessage.enableIfNot(); taskSendMessage.forceNextIteration(); through out in server sketch.

Ah! That explains it. Works great with the change.
I think that I also have problems with the power. The ESP is getting fairly hot and flashing via USB often leads to restarts of the ESP. That could also be a problem. I have to look for a powered USB hub. That makes setup a bit harder for ma at the moment.
I also noticed that the webpage isn't served very reliable, often the ESP just don't returns anything and the browser keeps loading.

Great job DEB !!! I will try a setup this weekend.

Will mesh be implemented now ?

Most probably not, but you can try the experimental branch. Other avenues of getting the same results are being explored.

@debsahu Hi Debsahu, could you elaborate on what may take it's place? i have a project coming up that would benefit from the mesh style setup.

@aidbish ArtNet or some implementation like that.

Cool, look forward to it.

Hi Debsahu, just a quick question regarding the mesh. i am currently using it till a new version is developed and it does actually work pretty well. what i do a have a problem with is turning them off. for some reason the off doesn't seem to function from the web page.. any ideas?

@aidbish which web UI are you using?

Im using the index.html one from mclighting experimental, couldn't see one in the meshmclighting folder.
it does look different to the standard one though. i even tried the standard one, but it didnt appear to change

@debsahu I am unable to use mDNS with the example provided need a help here


Add documentation to README

I work on several Azure extensions for VS Code (For example https://github.com/Microsoft/vscode-azurefunctions). We're trying to work with proxies and I found this package. The README is a little lacking haha so I had a few questions:

Is this package ready for use and in a reliable state?
Can most extensions ignore configure? It seems like getSystemProxyURI would configure for us
Any chance you could export getProxyAgent? Most of the time we go through an Azure ServiceClient instead of making the request directly. Even though we're not making the request, we can still pass in requestOptions to the constructor, which is where getting just the agent would come in handy.

Closing since it seems like VS Code will support proxies for extensions out of the box: https://github.com/Microsoft/vscode-azure-account/issues/27#issuecomment-441998815


Link not working

Hi, the link to platformio.ini not working

Thanks. Will fix then post on master.

Done.


[Snyk] Security upgrade jest from 23.6.0 to 24.0.0

Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

packages/ffe-chart-donut-react/package.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

589/1000  Why? Has a fix available, CVSS 7.5
Prototype Pollution SNYK-JS-MERGE-1040469
Yes
No Known Exploit

494/1000  Why? Has a fix available, CVSS 5.6
Command Injection SNYK-JS-NODENOTIFIER-1035794
Yes
No Known Exploit

601/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 5.6
Prototype Pollution SNYK-JS-YARGSPARSER-560381
Yes
Proof of Concept

506/1000  Why? Proof of Concept exploit, Has a fix available, CVSS 3.7
Regular Expression Denial of Service (ReDoS) npm:braces:20180219
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Coverage remained the same at 82.009% when pulling 140f67dc04989294118e6140ef955f834fcc9c2b on snyk-fix-e86df886bafcf48e40db2900c93d243b into d743d78bab0f6a4ce0bc199e8e419261247ca8b6 on develop.


add python demo for aliyun dts subscribe

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.wuzhengyi.wzy seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Empty/non-set relations should not return as empty string in the result Json

When a record with a Relation field (e.g. named "relation") has no relation set, the resulting Json returns an empty string:
{
    "id": "2euaei6s2bs9kfx",
    "created": "2022-11-16 21:36:26.035Z",
    "updated": "2022-11-16 21:36:26.035Z",
    "collectionId": "wx8ugpt1m8dk2o5",
    "collectionName": "items",
    "expand": {},
    "name": "Some record name",
    "relation": ""
}

This is a problem when working with freezed and a model Item that expects a model Relation for the "relation" field.
@freezed
class Item with _$Item {
  const factory Item({
    String? id,
    required String name,
    Relation? relation,
  }) = _Item;

  factory Item.fromJson(Map<String, Object?> json) => _$ItemFromJson(json);
}

A null value would be more appropriate and not pose a problem when working with freezed.
Side note: Before that works, I additionally need to copy the full record data from the expand['relation'] field to relation since the latter  would only contain ids, which would neither satisfy the Relation model. So I am questioning whether it makes sense to replace the (list of) id(s) in a relational field when expand=relation is given with the actual data set:
So instead of this (some default field omitted for brevity):
{
    "id": "bvo49tlzn8na3r6",
    "collectionName": "items",
    "expand": {
        "relation": {
            "id": "a74xecr0dl0mgh8",
            "collectionName": "relations",
            "expand": {},
            "name": "A relational record"
        }
    },
    "name": "An Item",
    "relation": "a74xecr0dl0mgh8"
}

have this instead
{
    "id": "bvo49tlzn8na3r6",
    "collectionName": "items",
    "name": "An Item",
    "relation": {
            "id": "a74xecr0dl0mgh8",
            "collectionName": "relations",
            "expand": {},
            "name": "A relational record"
        }
}

PocketBase doesn't have null values and uses zero defaults.
Relations are expanded only under the expand property and the original root-level keys are left untouched.
I'm not sure that I understand your use case, but there are no plans to change the resulting format since it will be too confusing (expand is not always available because it depends on the View API rule of the related collection and if we reuse the same key as the id then you may end up in situation where a single prop will have 2 different types - string and object/array).

Thanks Gani!
What I'm after is a json null for unset relational fields. So "relation": null instead of "relation": "":
{
    "id": "2euaei6s2bs9kfx",
    "created": "2022-11-16 21:36:26.035Z",
    "updated": "2022-11-16 21:36:26.035Z",
    "collectionId": "wx8ugpt1m8dk2o5",
    "collectionName": "items",
    "expand": {},
    "name": "Some record name",
    "relation": null
}

@TomTom101 As mentioned previously, we don't have null and this is deliberate (see the table of the supported values in https://github.com/pocketbase/pocketbase/releases/tag/v0.3.0; there will be more documentation on this in the new docs with the v0.8.0 release later this week).

Now I understand "with the exception of the json type" ;) Thanks!


Update window demo for libadwaita 1.5

See the first example here https://gnome.pages.gitlab.gnome.org/libadwaita/doc/main/class.Window.html

By workbench env you mean, we have to access the window demo from workbench sub-module, right?

Will make PR soon.

@sonnyp is this issue still open? if yes , i would love to give it a try


Extraction of Halo2 in specs is broken for Coq

Should be fixed by #160.

Just checked on main, this is still broken, I get the following errors:
error[HAX0001]: (Coq backend) something is not implemented yet.
                [expr] node app global vcar projector tuple
    --> halo2/src/halo2.rs:5509:19
     |
5509 |         let p_x = point.0;
     |                   ^^^^^^^

error[HAX0001]: (Coq backend) something is not implemented yet.
                [expr] node app global vcar projector tuple
    --> halo2/src/halo2.rs:5485:26
     |
5485 |         let y: FpVesta = points[i].1;
     |                          ^^^^^^^^^^^

error[HAX0001]: (Coq backend) something is not implemented yet.
                [expr] node app global vcar projector tuple
    --> halo2/src/halo2.rs:5484:26
     |
5484 |         let x: FpVesta = points[i].0;
     |                          ^^^^^^^^^^^


Windows Installation?

I tried installing mavlink-router on Windows by implementing linux equivalent commands for Windows but I was unable to do so. Hence, how to install mavlink-router in a Windows PC?
Does mavlink-router support Windows? If not, then I would like to know what is mavlink-router equivalent for Windows.
Thanks.

It probably works on WSL, but not sure anyone tested that

It probably works on WSL, but not sure anyone tested that

I plan to use QGC and MAVSDK on the same computer. It works great on Ubuntu using mavlink-router. But I'm looking to make it work on Windows. Hence, looking for windows install instructions as I read (in mavlink-router vs mavproxy document) that mavlink-router is crossplatform.
WSL wouldn't help me for what I intend to do.
Any install instructions or some ideas would really help.
Thanks

It doesn't work on windows. mavlink-router uses linux-only features, so you will probably need to use other project, develop support for windows, use WSL, or use Linux.

Ok thanks for the information.


build(cypress): Cypress concurrency (only)

SUMMARY
TEST: enable Cypress concurrency without record key.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #12062 (5162bfb) into master (7dac150) will decrease coverage by 11.17%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           master   #12062       +/-   ##
===========================================
- Coverage   63.81%   52.64%   -11.18%
===========================================
  Files         956      438      -518
  Lines       46845    15714    -31131
  Branches     4590     4060      -530
===========================================
- Hits        29895     8272    -21623
+ Misses      16766     7442     -9324
+ Partials      184        0      -184

Flag
Coverage Δ

cypress
52.64% <ø> (?)

javascript
?

python
?

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

Impacted Files
Coverage Δ

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...set-frontend/src/views/CRUD/alert/ExecutionLog.tsx
11.76% <0.00%> (-88.24%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

...rc/dashboard/components/gridComponents/Divider.jsx
13.33% <0.00%> (-86.67%)
:arrow_down:

...ontend/src/dashboard/util/getDashboardFilterKey.ts
14.28% <0.00%> (-85.72%)
:arrow_down:

...end/src/SqlLab/components/ExploreResultsButton.jsx
8.00% <0.00%> (-84.00%)
:arrow_down:

...nd/src/views/CRUD/data/query/QueryPreviewModal.tsx
14.70% <0.00%> (-82.97%)
:arrow_down:

... and 865 more

Continue to review full report at Codecov.

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

Closing in favor of https://github.com/apache/incubator-superset/pull/11750


Minor feedbacks on project team member list

From #1120
Minor feedbacks:

There should be a bottom margin when team member photo move to the next line

The designation and name should either be on a single line or on separate lines. One part of designation on one line and the second on another line looks a bit weird.

Originally posted by @mdshamoon in https://github.com/ColoredCow/portal/issues/1120#issuecomment-950865189

I am having issues, with the avatar placement as when I add new avatars, I am unable to see them updated on the UI. I have fixed the tooltip issue.

This works fine. Closing.


RAM overflow while initializing graph

I have 294 Gb binary .dat file and one machine having 126 Gb of RAM.
When I execute a command
python -m tf_euler \
  --data_dir euler \
  --max_id 177416798  \
  --model graphsage --mode train

memory overflow occurs. What can I do to process  this file in chunks one my machine? Can HDFS help me to do it locally?

I think a single machine of 126GB is not likely fit 294GB of data anyway. So you probably need to distribute your data to multiple machines.

Why is it necessary to put all data in RAM? Why not just read data in chunks? If say GraphSage needs neighborhood information it can get it from disk, this way it can loop through all nodes. As far as I know you already save neighborhood in adjacent list and you can seek to a specific record from file, I guess TFRecord can help or some binary key value storage. So you can efficiently produce minibatches.


getAllData data structure

Thanks for the great script! I used the library from the Google App Script and it works great!
But one thing I noticed is when calling getAllData, the returned data is not in proper json format. Instead of having colons, it has equal signs.
Here is an example return:
[{-LINN2XT9wdCMEQGhZ9C={firstName=myname, yourSalary=200000, position=ioio, salary=100000}, -LINBdoUX4-s793E-cDW={firstName=abcdsad, position=hahahhahha, salary=1223434}, -LINDhN0TNmOcP6Y6_Qq={firstName=yourname, yourSalary=2000000, position=senior leader}}]
Any clue what went wrong?

Hello @yidan ,
This output is only the Logger.log() output format, the real object is of correct JSON format.
Notice how the string are written without being surrounding quota ?


System Property serverPort is ignored

According to the documentation:
java -jar -DserverPort=8080 standalone-hystrix-dashboard-1.5.2-all.jar

should launch the server on port 8080.  However, it starts on the default 7979 instead.  I did some digging into vertx documentation and was able to find a workaround:
Create a hystrix.conf file with the following contents:
{
  "serverPort": 8080
}

Then this will start the server on the port 8080 as desired.
java -jar standalone-hystrix-dashboard-1.5.2-all.jar -conf hystrix.conf

@gcase, i refactored the verticle and forgot to add the system property, just fixed it

@gcase, released a new version following the netflix release, with the correction!


fixing the update page

this PR changes the update page so that:

the date includes the year (#97)
the sidebar shows the updates around the current update, rather than the most recent (#98)
update data is no longer sent if the author is true private
the sidebar should be hidden if the viewer isn't logged in or doesn't follow the author (#79)

this comes along with some general readability/performance improvements — the update page no longer fetches any update data client side!

in the future we might want the highlighted update in the sidebar to have a sticky position (it's fixed to be the third update rn). maybe a scrollable list of updates?


Details of rendering synthetic forms

Hi,
Thanks for releasing the great code! I'm trying to pre-train a Dessurt-like model on your provided pre-training datasets. However, I couldn't find how to render the synthetic forms generated from GPT-2.
Could you explain it or provide the code?

data_sets/synth_form_dataset.py defines a Dataset object which renders the forms on the fly during training. You can use data_sets/test_synth_form_dataset.py to generate and save images offline if you'd like (you'll have to modify the code to write things out).

Thank you for your quick reply.
I was trying to render images with your provided code, but the running might be took a long time.
Could you provide the synthetic images rendered from GPT-2' results in public?

I didn't cache the images, so I don't have any to release, sorry.


Unsupported amino acids should ideally yield warnings, not crash the program

From a usability perspective, I would suggest that sequences with unsupported amino acids be skipped and warnings be issued. The current behavior simply has the program crash with a KeyError. The note on the main page about these amino acids is clear but having the user sanitize their input seems like much more effort than handling this through Prosit.
Here is my quick and dirty patch (just to get the tool running for me). Probably not the best way to handle this but it does rely on utils.peptide_parser() to not duplicate logic.
diff --git a/prosit/tensorize.py b/prosit/tensorize.py
index d4eefdb..d94127f 100644
--- a/prosit/tensorize.py
+++ b/prosit/tensorize.py
@@ -1,4 +1,5 @@
 import collections
+import logging
 import numpy as np

 from . import constants
@@ -45,7 +46,7 @@ def get_precursor_charge_onehot(charges):
     return array

-def get_sequence_integer(sequences):
+def get_sequence_integer(sequences):
     array = np.zeros([len(sequences), MAX_SEQUENCE], dtype=int)
     for i, sequence in enumerate(sequences):
         for j, s in enumerate(utils.peptide_parser(sequence)):
@@ -89,6 +90,14 @@ def csv(df):
     assert "modified_sequence" in df.columns
     assert "collision_energy" in df.columns
     assert "precursor_charge" in df.columns
+
+    n_all = df.shape[0]
+    df = df[df.modified_sequence.apply(is_supported_sequence)]
+    n_supported = df.shape[0]
+    if n_supported < n_all:
+        logging.warning(' {0} / {1} sequences were skipped as they '
+                        'contained unsupported amino acids'.format(n_all - n_supported, n_all))
+
     data = {
         "collision_energy_aligned_normed": get_numbers(df.collision_energy) / 100.0,
         "sequence_integer": get_sequence_integer(df.modified_sequence),
@@ -107,3 +116,12 @@ def csv(df):
     data["masses_pred"] = masses_pred

     return data
+
+
+def is_supported_sequence(seq: str) -> bool:
+    try:
+        _ = [ALPHABET[s] for s in utils.peptide_parser(seq)]
+        return True
+    except KeyError as e:
+        return False
+

PS Good work on developing Prosit! :smile:

If you provide a PR, we are happy to merge it. Bonuspoints if you have a working test case :)

Sorry to chip in. Originally, we design the Prosit pipeline, so that it never changes any user input. We on purpose didn't skip sequences, because input and ouput files would have a different length. I believe that providing malformed input data should throw an error and not a warning. But maybe this premise changed in the Prosit pipeline.
@tkschmidt @LLautenbacher @fburic

I see, that makes sense. If the premise hasn't changed, then I'd only suggest making the error more explicit. In my case, it took me quite some time to figure out what was wrong (and I'm familiar with Python). Then I just decided it was more efficient (and acceptable) for me to fix it like above :)


OpenAPI Validator error message revamp

Description:
OpenAPI validator error messages need to be more descriptive.

Closing since this has been fixed in #19912


Disable the Compare editor 'Next/Previous Change' buttons if there are no differences

Type: Feature Request
When comparing files, there are often times where the files are identical. One of the typical purposes of the Compare editor is to verify there are no changes, not just to view the changes themselves if present.
When comparing the files, if changes are present, they are highlighted and the Next Change/Previous Change buttons are enabled in the toolbar, along with their menu items.

However, if the files are identical, there is no indication at all, and in fact, the buttons are still enabled, making it appear as though there are changes. Therefore, the user must rely on scanning the file overview for any highlighted lines, or, for a more positive confirmation, press the Next Change button (or keyboard shortcut) to verify there are really no differences.

It would be much more helpful, obvious, and intuitive if the editor could indicate there are indeed no differences between the files. This could be accomplished in many ways:

Disable the Next Change/Previous change commands and their buttons
Display the text 'Files are identical' or '0 differences' in the status bar or window title bar, or next to the Next Change/Previous Change icons, or on the Compare editor tab
Show an icon indicating the files match on the Compare editor's tab
Show a notification or confirmation dialog

If the Next Change/Previous Change buttons and their corresponding commands were simply disabled, this would be an obvious indicator that there are in fact no changes to review, but right now, the Compare editor does nothing in this situation, making it very ambiguous, requiring the user to look for some form of confirmation. It is also counterintuitive given that the toolbar buttons and their commands are available to select, yet these commands do nothing in this context.
VS Code version: Code 1.76.1 (5e805b79fcb6ba4c2d23712967df89a089da575b, 2023-03-08T16:32:00.131Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

For reference, here's what the search panel does when there are no matches:

@alexdima I have replicated the issue in latest version.
Version: 1.76.2
Commit: ee2b180d582a7f601fa6ecfdad8d9fd269ab1884
Date: 2023-03-14T17:57:21.103Z
Browser: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Code/1.76.2 Chrome/102.0.5005.196 Electron/19.1.11 Safari/537.36

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


vtu mesh file nodal number reordered

Hello everybody!
I am trying to read the mesh in the vtu format and do some FEA with it.
The FEA process goes smoothly. However, I found that the nodal number of the mesh is reordered.
I was wondering if there is a way to prevent the reordering and just use the given node number instead.
Thank you!

Here is how I read the mesh:
Mesh *mesh = new Mesh(mesh_file1, 0, 0, false);
I have already turned off the refine and fix_orientation.

Hello @ChaoW95,
Is your mesh high-order or straight-sided? Typically MFEM does not reorder the vertices of a straight-sided mesh. It might be useful, if possible, to attach your mesh file here.

Hi @pazner,
I modified the ex2.cpp in the mfem. I used two methods to save the results.
One is the given method:
      ofstream mesh_ofs("displaced");
      mesh_ofs.precision(8);
      mesh->Print(mesh_ofs);

Another one is the ParaViewDataCollection:
    paraview_dc.SetLevelsOfDetail(0);
    paraview_dc.SetDataFormat(VTKFormat::BINARY);
    paraview_dc.SetHighOrderOutput(false);
    paraview_dc.Save();

Those two methods give different mesh numbering.
ex2.zip

Then I try to use the Paraview to check the results. It turns out the result saved by the ParaViewDataCollection has 48 points some of them are duplicate.

The ParaView output is just intended for visualization, not for further analysis. There will be duplicated points because of how MFEM handles the visualization, see https://github.com/mfem/mfem/issues/2906#issuecomment-1074157376

See also #1757


Add tenant to the application subscription UI

Problem
Subscriber user wants to identify the tenant in the application subscription page in the dev portal. The majority of end-user applications utilize the commonly registered applications in the WSO2 devportal. It can be challenging to navigate through each tenant and inspect application subscriptions, particularly for new subscribers or returning subscribers after a period of time as there are number of tenants and APIs
Solution
Provide a way to identify all the subscriptions along with the tenant which are subscribed to an application
Affected Component
APIM
Version
4.0.0
Implementation
No response
Related Issues
https://github.com/wso2/api-manager/issues/2514
Suggested Labels
No response

This is no longer required according to https://github.com/wso2/api-manager/issues/2685


We only need to build bench-tps. No need for other binary

current code build all binaries.
We can build only bench-tps to save time

https://github.com/solana-labs/bench-tps-dos-test/pull/31


add delete for faiss

Fixes #2427
Also ports https://github.com/langchain-ai/langchain/pull/5190
Also fixes float64 parsing by upgrade pickleparser
relates: https://github.com/langchain-ai/langchain/issues/2699

this CI failed with OOM, it should be passed with rerun.

Awesome, thank you!
I'll add a docs page and fix CI later.

Hey @ewfian, I think there's a bug - when I tried to run the example I added I got:
Error: Could not load example examples/src/indexes/vector_stores/faiss_delete.ts: Error: ID undefined not found.
Can you have a look and see if you can repro?

@jacoblee93 Thank you for adding docs, the bug was fixed.

You rock, thanks!


Fix optimistic data IOU delete money request

Details

Related to #35211
There needs to be both backend and frontend changes.
The frontend changes concern the optimistic data.
The addition of the offset parameter in the getLastVisibleAction() function enhances the flexibility and utility of fetching report actions based on their visibility and order. Thanks to that, I keep it DRY. As always, if you can think of any improvements, feel free to tell!
Backend changes are currently under review. Once they are merged, we'll be able to merge this one as well.
Fixed Issues

$ https://github.com/Expensify/App/issues/35211
PROPOSAL:
https://github.com/Expensify/App/issues/35211#issuecomment-1911490753
Tests

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to FAB> Request money>
Request money from a user that has no existing chat
Click on + button> Request Money> Complete the IOU flow
Delete the last created IOU
Navigate back to the conversation
Green line should not appears since there is no new message

Video on how it should look like is in the section MacOS: Chrome / Safari.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/Expensify/App/assets/21549771/d066689d-4362-4075-9de7-f7904a07c91d

MacOS: Desktop

Closing the PR for the reasons mentioned in the issue here


v2<IP_ADDRESS> release

version name changes related to v2<IP_ADDRESS>

@shijingsun-TRI let's wait on releasing this until the Novonix bug is fixed, which I will do today

#659 should be merged with this PR before release


Contraband category mega-issue

Let's collect all of them here so we have a better overview. If you find an item that got the wrong contraband category write a comment and I will add it to the list.

[x] Cat ears should be syndicate contra

[x] Syndicate pAI should be syndicate contra

[x] Chest rig should be syndicate contra

[x] Surplus crates should be contraband

[x] Binary translator key should be science restricted instead of syndicate contra

[x] Laser guns should be all security restricted

[x] Weapons that don't belong to Cybersun or Nanotrasen should be majorly restricted (such as the retro laser blaster, the RPG, or the minigun)

[x] Briefcases should be stealthy again

[x] Hypodart and Hypopen boxes should be syndicate contra

[x] Syndicate implanters should be marked as syndicate contra

[x] Rigged boxing gloves should be stealthy

[x] Riot shields and such should be security restricted

[x] Security radio should be security restricted

[x] RCD should be engineering restricted

[x] Security crates should be properly marked as security restricted.

[x] All security gun safes should be marked as security restricted

[x] Secglasses should be command/security restricted. This is subject to further discussion with the admins. In case #30447 gets merged they should be changed to be security only.

[x] Door remotes should be all command restricted, the armory remote is security and command restricted

[x] Pinpointers should be either command restricted or syndicate contraband

[x] Medal case should be command restricted

[x] Security jetpack should be security restricted

[x] Hot potato should be marked as a syndicate item

[x] Nuclear bomb should be marked as major contraband

[x] Stamps should be either command restricted or security restricted

[x] Ninja gear should be major contraband

[x] Combat gloves should be security/engineering restricted

[x] Syndie hand flag should be syndie contra

[x] Syndicate codeword papers should be syndie contra

[x] CentComm pen should not be syndicate contra

[x] Practice laser rifles and disablers should not be security restricted

[x] Energy dagger pens should only show as contra when toggled on

[x] All giant ass sword type weapons should be upgraded from minor contra to major contra

[ ] #30986

[x] #30931

[x] The advanced magboots should show as command restricted and have a grand theft label.

[ ] The roundstart headset for lawyers should not be considered contraband for them

[x] Grenades should be contraband

[x] Encryption keys, in particular security, command and the master key should be contraband. The respective headsets are already marked as contraband

[x] The fire axe (syndicate weapon) should be syndicate contraband.

[x] The raid suit's helmet should be syndicate contraband instead of departmentally restricted.

[x] Thief undetermined toolbox should be contraband

[x] Thieving beacon should be contraband

[ ] Possibly botany plants and seed (e.g. death nettle, killer tomatos), although these can have some legitimate uses.

[x] Mercenary clothing, maybe sec restricted?

[x] Lethal ammunition should be contraband

[x] Grenades, Rockets, see explosives.yml should be contraband

[x] Advanced circular saw should be contraband

[x] Nukie Plushie should not be contraband

[x] acolyte armor should be contraband

[x] ClothingHeadHelmetERTLeader should not be syndicate contraband

[x] Add Central Command contraband category

[x] The ERT gas mask should be command/CentCom restricted.

[x] Bladed Flatcap should not be contraband (stealth item)

[x] Explorer gas mask should be cargo restricted

[ ] The bandolier should be allowed for the bartender.

[ ] Allow armored vests and double barreled shotguns for bartenders

[ ] The sawn-off shotgun should be restricted to bartender/security

[x] AKMS should be security restricted (found in Bagel armory)

[x] The swat gas mask the HOS starts with should be security restricted, not cargo

[x] The bone helmet should not be security restriced, but something else (I think?)

[x] chitinous armor, bone armor and ironclad II armor should be marked as contraband/restricted

[x] The communications console board should be command restricted or grand theft (check the other theft target machine boards as well)

Please make it so secglasses are SECURITY restricted, not command/security. Also we need some sort of difference between "highly illegal syndicate contraband" and stuff like the syndie flag, like "This is somewhat illegal syndicate contraband, but you should be fine to carry it around" and "This is central command restricted contraband!"

Please make it so secglasses are SECURITY restricted, not command/security.

This is currently being discussed with the admins. A possible solution could be the addition of the special command glasses, which only show the job icon, but not wanted status #30447

Please make it so secglasses are SECURITY restricted, not command/security.

This is currently being discussed with the admins. A possible solution could be the addition of the special command glasses, which only show the job icon, but not wanted status #30447

Would still need to make SecGlasses restricted right? What would be stopping Command from just ditching the command glasses and getting SecGlasses.

Would still need to make SecGlasses restricted right? What would be stopping Command from just ditching the command glasses and getting SecGlasses.

Yes, that would have to be added to that PR.

the fire axe has its own design and description so it isnt a stealth item

The raid suit's helmet (not the actual suit) says it's departmentally restricted instead of syndicate contraband

Merc gear should be sec restricted i belive its currently incorrectly parented as syndie gear

The CE's advanced magboots are listed as engineering restricted, not command restricted
Also they don't have the "This item is a highly valuable target for Syndicate agents!" tag

the acolyte armor should be listed as syndicate or minor contraband, because it is functional armor exclusive to emagging a vendor. there is no other way to get it other than emagging the pietyvendor

Nukie Plushie?

The Nukie Plushie can spawn in the cap's locker, it would be weird to include it as contraband imo

The Nukie Plushie can spawn in the cap's locker, it would be weird to include it as contraband imo

It currently is contraband. That's why I added it. I'll make it more clear

https://github.com/space-wizards/space-station-14/blob/6a549ceee4a411197a39077649be4cb1326d8caf/Resources/Prototypes/Entities/Objects/Weapons/Melee/knife.yml#L139
Bladed flatcap is an improvised weapon but is marked as syndicate contraband

https://github.com/space-wizards/space-station-14/blob/6a549ceee4a411197a39077649be4cb1326d8caf/Resources/Prototypes/Entities/Objects/Weapons/Melee/knife.yml#L139
Bladed flatcap is an improvised weapon but is marked as syndicate contraband

it really SHOULDN'T be marked as contraband, as its a stealthy weapon (last time i checked)

it really SHOULDN'T be marked as contraband, as its a stealthy weapon (last time i checked)

Good point. You can still see the damage it does upon examination though, so it's still kinda obvious?

Good point. You can still see the damage it does upon examination though, so it's still kinda obvious?

you can see a hypopen's reagent counter on inspection, but that doesn't make it immediately obvious contraband. If the weapon has a line stating it's bladed in its description box, however, then its fair game (this is the part I'm not so sure of, because i forget the description)

Probably should be restricted to cargo considering the armor?

Probably should be restricted to cargo considering the armor? (considering its literally better than sec gasmask armor)
Do they have that round-start or is it salvage loot?

explorer masks are just roundstart stuff.

Probably should be restricted to cargo considering the armor? (considering its literally better than sec gasmask armor)

Do they have that round-start or is it salvage loot?

Round start salv. I don’t remember if you can get more, I don’t think so though. Maybe when finding a mining suit crate.

ERT mask = contraband :trollface:

Bandolier is not allowed by the bartender

cleanades marked as contraband

thinking about it... what if people get promoted?

It shouldnt because its a stealth item

Shouldnt it be department restricted? (Security, comes in bagel armoury)

I dont... why??? you can get it from the lottery and its not syndicate related in any way outside of it being accessible from the uplink, and its got no functional usage

thinking about it... what if people get promoted?

I think it is tied to your ID's access, so if that changes with your promotion you should be fine.

It shouldnt because its a stealth item

It got the word "thief" right in its name, so no stealth at all. If you want it to be stealthy, it should be named toolbox instead.

Shouldnt it be department restricted? (Security, comes in bagel armoury)

Corrected it.

cat ears

Because funny, also they were considered contraband before

I thought the thief toolbox was just a normal toolbox namewise till interacted with, mb
But I don't recall the cat ears ever being contraband? (Before the OG pr for this came out and before contraband rules in spacelaw were nuked)

Found these:

So I guess the admin answer is 🤷♂

I have also been told that it's contraband if it's "primarily obtained in the uplink" and I think cat ears are common enough from lotteries to be excluded from that

The swat gasmask that Head of security starts with is marked as cargo restricted

Salvage use normal magboots for their job and can get them out of their locker, but they're restricted to engineering so they appear to be contraband to salvagers

Some armors might need to be marked as contraband:

chitinous armor
bone armor
ironclad II armor.

Salvage use normal magboots for their job and can get them out of their locker, but they're restricted to engineering so they appear to be contraband to salvagers

the eva room also has magboots i think, so it should just be removed or changed to say jobs with external access

I thought normal magboots being contra was fixed already?

galoshes should be jani restricted. Ive seen officers justify using them because they "arent really contra"

x-ray cannons are not security restricted

The HoS’s trench coat is not security restricted

Omega soap should probably be contraband, its syndie soap but better

Please make it so secglasses are SECURITY restricted, not command/security.

This is currently being discussed with the admins. A possible solution could be the addition of the special command glasses, which only show the job icon, but not wanted status #30447

Administrative glasses would be AWESOME, the admin hud isn't flash immune.

Administrative glasses would be AWESOME, the admin hud isn't flash immune.

we already have admin glasses and admin huds

@golinth I asked admins about the omega soap.
I think it is fine without being contraband.

Tesla gun is also not marked as contraband.


fix: basedir for .nfnl file

as title says, basedir is missing and will return XXX/.nfnl.fn.
also notice basedir return will not end with /, so not need string.sub

Nice, thank you! I've also added some tests to check this and prevent it from happening again. Sorry for any issues this has caused!


I am working on a awesome-python-htmx, seeking your feedback

A few of us at PyCon this year got together and brainstormed a new Web Stack that we are calling PyHAT (Python, htmx, ASGI, TailwindCSS). The first thing we set out to do is create awesome-python-htmx; a collection of active tools/libraries/projects in that space.
Your project seems like an obvious thing to include, so I did so. I'd appreciate your feedback if you have any on it's inclusion.
In addition to that, if you could also participate in https://github.com/PyHAT-stack/awesome-python-htmx/discussions/1 that would be greatly appreciated!

Hi @benjamin-kirkbride, thanks for the inclusion! I think that having two different flask extensions for htmx is confusing for the users, so I have been working to replicate all the functionality of htmx-flask into flask-htmx (which has a better package name :D ). Once that is complete, I'll modify the README of the project to redirect users to flask-htmx.
If because of that you think it's not worth to mention htmx-flask on PyHAT,  feel free to remove it.

So it's not obvious to me what the relationship is between flask-htmx, and htmx-flask is at this time. Basically I don't want to drop someone else's project unless they agree with your assessment.
Very happy to reduce the redundancy, and have a more focused list.

I would also love if you could participate in the discussions going on over at https://github.com/PyHAT-stack/awesome-python-htmx/discussions!


Schema update unnecessarily drops/creates related objects when Sensitivity Classification changes are detected

Summary:
When schema compare (correctly) detects that a Sensitivity Classification should be created, dropped, or modified in the target, the generated script takes the unnecessary steps of dropping/recreating all indexes and constraints related to that column. Index rebuilding in particular is problematic in our ability to perform zero-downtime/zero-disruption production releases. Since sensitivity classifications currently cannot be ignored by schema compare, it seems our only recourse is to ensure these classifications are always in sync between the dacpac and target databases, without relying on DacFx to help with this. This is a little painful given our security/ops people apply Azure-recommended classifications directly to the target Azure databases from time to time.

DacFx Version: 150.5282.3
.NET Framework (Windows-only) or .NET Core: .NET 5
Environment (local platform and source/target platforms):

Windows 10
Visual Studio 2019 (v16.11.5)
SQL project with DSP Microsoft.Data.Tools.Schema.Sql.Sql150DatabaseSchemaProvider or Microsoft.Data.Tools.Schema.Sql.SqlAzureV12DatabaseSchemaProvider
.NET 5 console app with Microsoft.SqlServer.DacFx v150.5282.3 installed
Target database SQL Server 2019 or Azure SQL

Steps to Reproduce:
(easiest if you already have a SQL project and sync'd live database to experiment with)

Define a table in a SQL project that contains an index and constraint on some column, build dacpac.
Create a target database (SQL 2019 or Azure SQL) with the same table defined (manually or by deploying the dacpac).
In the target database, add a sensitivity classification (with T-SQL or otherwise) to a column that has an index and constraint.
Generate a schema update script with the following code (or similar):

var source = new SchemaCompareDacpacEndpoint(dacpacPath);
var target = new SchemaCompareDatabaseEndpoint(targetConnectionString);
var comp = new SchemaComparison(source, target);
var script = comp.Compare().GenerateScript(dbName).Script;

Inspect generated script. Notice it correctly drops the classification, but it also unnecessarily drops and recreates any indexes and constraints defined on that column.

Did this occur in prior versions?  If not - which version(s) did it work in?
Yes, same behavior in older versions of DacFx published to NuGet. I'm fairly certain this bug has existed since support for sensitivity classifications was first introduced.

@tmenier We introduced IgnoreSensitivityClassifications DacDeployOption in Microsoft.SqlServer.DacFx 160.6161.0 , you can specify it during Schema Compare too.
var comp = new SchemaComparison(source, target);
comp.Options.IgnoreSensitivityClassifications = true;
var script = comp.Compare().GenerateScript(dbName).Script;

@zijchen  is this shipped in 19.1? can we close it?


ASAN reports memory leaks

I will provide a minimal code snippet to reproduce if this is not a known issue. Please let me know.
=================================================================
==12==ERROR: LeakSanitizer: detected memory leaks

Direct leak of 3456 byte(s) in 24 object(s) allocated from:
    #0 0x7f7322a20dd7 in __interceptor_malloc (/usr/lib/x86_64-linux-gnu/libasan.so.6+0xb2dd7)
    #1 0x5637b50f014d in aeron_alloc aeron-client/src/main/c/aeron_alloc.c:53
    #2 0x5637b512db8b in aeron_image_create aeron-client/src/main/c/aeron_image.c:74
    #3 0x5637b510c5b5 in aeron_client_conductor_on_available_image aeron-client/src/main/c/aeron_client_conductor.c:2184
    #4 0x5637b50fedcf in aeron_client_conductor_on_driver_response aeron-client/src/main/c/aeron_client_conductor.c:282
    #5 0x5637b5123d30 in aeron_broadcast_receiver_receive aeron-client/src/main/c/concurrent/aeron_broadcast_receiver.c:100
    #6 0x5637b51005a8 in aeron_client_conductor_do_work aeron-client/src/main/c/aeron_client_conductor.c:569
    #7 0x5637b50f5b50 in aeron_do_work aeron-client/src/main/c/aeron_client.c:38
    #8 0x5637b50ee3da in agent_main aeron-client/src/main/c/aeron_agent.c:407
    #9 0x7f732254f6da in start_thread /build/glibc-uZu3wS/glibc-2.27/nptl/pthread_create.c:463

Indirect leak of 240 byte(s) in 24 object(s) allocated from:
    #0 0x7f7322a20dd7 in __interceptor_malloc (/usr/lib/x86_64-linux-gnu/libasan.so.6+0xb2dd7)
    #1 0x5637b50f014d in aeron_alloc aeron-client/src/main/c/aeron_alloc.c:53
    #2 0x5637b512dbcb in aeron_image_create aeron-client/src/main/c/aeron_image.c:75
    #3 0x5637b510c5b5 in aeron_client_conductor_on_available_image aeron-client/src/main/c/aeron_client_conductor.c:2184
    #4 0x5637b50fedcf in aeron_client_conductor_on_driver_response aeron-client/src/main/c/aeron_client_conductor.c:282
    #5 0x5637b5123d30 in aeron_broadcast_receiver_receive aeron-client/src/main/c/concurrent/aeron_broadcast_receiver.c:100
    #6 0x5637b51005a8 in aeron_client_conductor_do_work aeron-client/src/main/c/aeron_client_conductor.c:569
    #7 0x5637b50f5b50 in aeron_do_work aeron-client/src/main/c/aeron_client.c:38
    #8 0x5637b50ee3da in agent_main aeron-client/src/main/c/aeron_agent.c:407
    #9 0x7f732254f6da in start_thread /build/glibc-uZu3wS/glibc-2.27/nptl/pthread_create.c:463

SUMMARY: AddressSanitizer: 3696 byte(s) leaked in 48 allocation(s).
================================================================================

Depends on what you are doing and how you are closing down the threads on the client. You will have to coordinate it to shutdown cleanly. Take a look at the various tests we have to see how the tests do this. We do test with sanitize in the low cadence builds.

Please find attached a standalone end-to-end test that reproduces this issue:
aeron_test.txt
It seems like there is an issue when there are multiple subscriptions to a single stream.
AeronTest.OneToOne - No memory leak reported
AeronTest.OneToMany - Memory leak reported as above
Am I missing any coordination that you mentioned in your comment above?

The attached file appears to be truncated.

Github is probably truncating the file (after refusing to attach .cpp files) :|
I opened a merge request that just adds the test to reproduce this issue.

I've posted a patch on the PR.  The subscription hasn't completed closing before the test exits.


Fix error on new version of jsdom

Hi Team!
i got the problem on your new version. So i sent an PR to help you can check it easier! i already disabled that code to make my system run normally. they give me this error
/home/tuan/git/thedragoncode/node_modules/jsdom/lib/jsdom.js:55
throw new RangeError(`Invalid parsingMode option ${JSON.stringify(options.
^
SyntaxError: Unexpected token ILLEGAL
at exports.runInThisContext (vm.js:69:16)
at Module._compile (module.js:432:25)
at Object.Module._extensions..js (module.js:467:10)
at Module.load (module.js:349:32)
at Function.Module._load (module.js:305:12)
at Module.require (module.js:357:17)
at require (module.js:373:17)
at Object. (/home/tuan/git/thedragoncode/test/config/application.js:22:12)
at Module._compile (module.js:449:26)
at Object.Module._extensions..js (module.js:467:10)
please have a look on it and  fix :), thanks

Dupe of #1091, #1087, #1086, and probably a few others for Set.


Add brainfuck language

#77

Pull Request Test Coverage Report for Build 61

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 8.421%

Totals

Change from base Build 60:
0.0%

Covered Lines:
160

Relevant Lines:
1900

💛  - Coveralls


Support 32-bit action IDs (added in v6.43+dev1222)

Main section actions (registered using command_id+gaccel) remain 16-bit (limited by the ACCEL structure from Windows/SWELL using a short).
Other sections (registered using custom_action) were expanded to a 32-bit ID.

:white_check_mark: Build sws 695-master completed (commit https://github.com/reaper-oss/sws/commit/94f990a0f1 by @cfillion)

Windows 64-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-32bit-action-id-94f990a0.exe

Windows 32-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-32bit-action-id-94f990a0.exe

macOS x86 64-bit

build/sws-<IP_ADDRESS>-Darwin-x86_64-32bit-action-id-94f990a0.dmg

macOS x86 32-bit

build/sws-<IP_ADDRESS>-Darwin-i386-32bit-action-id-94f990a0.dmg

macOS ARM 64-bit

build/sws-<IP_ADDRESS>-Darwin-arm64-32bit-action-id-94f990a0.dmg

Linux x86 64-bit

build/sws-<IP_ADDRESS>-Linux-x86_64-32bit-action-id-94f990a0.tar.xz

Linux x86 32-bit

build/sws-<IP_ADDRESS>-Linux-i686-32bit-action-id-94f990a0.tar.xz

Linux ARM 64-bit

build/sws-<IP_ADDRESS>-Linux-aarch64-32bit-action-id-94f990a0.tar.xz

Linux ARM 32-bit

build/sws-<IP_ADDRESS>-Linux-armv7l-32bit-action-id-94f990a0.tar.xz

Removed the preventive change to an unsigned int (the WDL container for g_commands uses a signed int anyway). Fixes these warnings:
..\sws_extension.cpp(391): warning C4018: '>=': signed/unsigned mismatch
..\sws_extension.cpp(391): warning C4018: '<=': signed/unsigned mismatch

...as well as the merge conflict with #1591.

:white_check_mark: Build sws 740-master completed (commit https://github.com/reaper-oss/sws/commit/2a1a894a7f by @cfillion)

Windows 64-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-32bit-action-id-2a1a894a.exe

Windows 32-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-32bit-action-id-2a1a894a.exe

macOS x86 64-bit

build/sws-<IP_ADDRESS>-Darwin-x86_64-32bit-action-id-2a1a894a.dmg

macOS x86 32-bit

build/sws-<IP_ADDRESS>-Darwin-i386-32bit-action-id-2a1a894a.dmg

macOS ARM 64-bit

build/sws-<IP_ADDRESS>-Darwin-arm64-32bit-action-id-2a1a894a.dmg

Linux x86 64-bit

build/sws-<IP_ADDRESS>-Linux-x86_64-32bit-action-id-2a1a894a.tar.xz

Linux x86 32-bit

build/sws-<IP_ADDRESS>-Linux-i686-32bit-action-id-2a1a894a.tar.xz

Linux ARM 64-bit

build/sws-<IP_ADDRESS>-Linux-aarch64-32bit-action-id-2a1a894a.tar.xz

Linux ARM 32-bit

build/sws-<IP_ADDRESS>-Linux-armv7l-32bit-action-id-2a1a894a.tar.xz

:white_check_mark: Build sws 743-master completed (commit https://github.com/reaper-oss/sws/commit/3d8a399091 by @cfillion)

Windows 64-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-32bit-action-id-3d8a3990.exe

Windows 32-bit

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-32bit-action-id-3d8a3990.exe

macOS x86 64-bit

build/sws-<IP_ADDRESS>-Darwin-x86_64-32bit-action-id-3d8a3990.dmg

macOS x86 32-bit

build/sws-<IP_ADDRESS>-Darwin-i386-32bit-action-id-3d8a3990.dmg

macOS ARM 64-bit

build/sws-<IP_ADDRESS>-Darwin-arm64-32bit-action-id-3d8a3990.dmg

Linux x86 64-bit

build/sws-<IP_ADDRESS>-Linux-x86_64-32bit-action-id-3d8a3990.tar.xz

Linux x86 32-bit

build/sws-<IP_ADDRESS>-Linux-i686-32bit-action-id-3d8a3990.tar.xz

Linux ARM 64-bit

build/sws-<IP_ADDRESS>-Linux-aarch64-32bit-action-id-3d8a3990.tar.xz

Linux ARM 32-bit

build/sws-<IP_ADDRESS>-Linux-armv7l-32bit-action-id-3d8a3990.tar.xz


Add compile-time switch for Sequentia RPC definitions

Uses a compile time flag __SEQUENTIA__ for switching between Sequentia and Elements RPC definitions.
Note: The definition of sequentia::createrawtransaction hasn't been updated for the No Fee feature yet. This is more a demonstration and proof-of-concept for how we can extend the protocol without interfering with the Elements codepath.

Closing since this will create extra work if we try to upstream to Elements, and the Elements team has already expressed interest in the No Coin feature (https://github.com/ElementsProject/elements/issues/1302),.


[TEST][FLAKY] frontend/pytorch/test_forward.py::test_forward_nms

https://ci.tlcpack.ai/job/tvm/job/main/19/execution/node/380/log/
Could due to too close ties, perhaps we should construct nms test-cases differently that have no clear ties

cc @masahi @Laurawly

cc @yongwww

@masahi @tqchen I failed to reproduce it on my environment. The flaky issue was introduced due to the accumulation of tiny difference of computations, especially for a large number of input boxes. I will send a pr to loosen the tolerance.

@yongwww maybe setting rng manual seed via torch.manual_seed(...) help.

Still has this problem
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-7824/1/pipeline


Documentation on how to use with vitest shard

Hey,
Thanks for the action - very helpful!
It'd be useful to know how (or if?) this can be used with the multiple outputs from the shard command

Hey @jak,
unfortunately, I don't know the shard feature just yet (haven't been doing a lot of development with vitest recently), so I would have to look deeper into this. I am not sure when I will be able to do so.
However, I am always happy to accept pull-requests that will document how this works with this library, or enable this library to work with the command. 🙂
Sorry I can't be more helpful right now - time is a rare asset 🕐


silently fails to format match expression if arm pat contains comment

For Tock, we run rustfmt with Travis CI, and some PRs (example1 and example2) are passing a rustfmt check while still using tabs for indentation.

No config options (we use defaults only).
Noticed this on nightly-2020-03-06 and nightly-2020-04-10.

I've also tried manually running rustfmt in the relevant crate on my computer and see the same effect. Based on a question on discord, I am opening this issue.
One example (src):
impl Driver for L3gd20Spi<'a> {
    fn command(&self, command_num: usize, data1: usize, data2: usize, _: AppId) -> ReturnCode {
        match command_num {
            0 /* check if present */ => ReturnCode::SUCCESS,
            // Check is sensor is correctly connected
            1 => {
				if self.status.get () == L3gd20Status::Idle {
					self.is_present ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}

			}
			// Power On
            2 => {
				if self.status.get () == L3gd20Status::Idle {
					self.power_on ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
			}
			// Set Scale
            3 => {
				if self.status.get () == L3gd20Status::Idle {
					let scale = data1 as u8;
					self.set_scale (scale);
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
            }
			// Enable High Pass Filter
            4 => {
				if self.status.get () == L3gd20Status::Idle {
					let mode = data1 as u8;
					let divider = data2 as u8;
					self.set_hpf_parameters (mode, divider);
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
			}
			// Set High Pass Filter Mode and Divider
            5 => {
				if self.status.get () == L3gd20Status::Idle {
					let enabled = if data1 == 1 { true } else { false };
					self.enable_hpf (enabled);
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
            }
			// Read XYZ
            6 => {
				if self.status.get () == L3gd20Status::Idle {
					self.read_xyz ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
			}
			// Read Temperature
            7 => {
				if self.status.get () == L3gd20Status::Idle {
					self.read_temperature ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
            }
            // default
            _ => ReturnCode::ENOSUPPORT,
        }
    }

    fn subscribe(
        &self,
        subscribe_num: usize,
        callback: Option<Callback>,
        _app_id: AppId,
    ) -> ReturnCode {
        match subscribe_num {
            0 /* set the one shot callback */ => {
				self.callback.insert (callback);
				ReturnCode::SUCCESS
			},
            // default
            _ => ReturnCode::ENOSUPPORT,
        }
    }
}

Another example (src):
impl Driver for Lsm303dlhcI2C<'a> {
    fn command(&self, command_num: usize, data1: usize, data2: usize, _: AppId) -> ReturnCode {
        match command_num {
            0 /* check if present */ => ReturnCode::SUCCESS,
            // Check is sensor is correctly connected
            1 => {
				if self.state.get () == State::Idle {
					self.is_present ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}

			}
			// Set Accelerometer Power Mode
            2 => {
				if self.state.get () == State::Idle {
                    if let Some (data_rate) = Lsm303dlhcAccelDataRate::from_usize (data1) {
                        self.set_power_mode(data_rate, if data2 != 0 { true } else { false });
                        ReturnCode::SUCCESS
                    }
                    else
                    {
                        ReturnCode::EINVAL
                    }
				}
				else
				{
					ReturnCode::EBUSY
				}
			}
			// Set Accelerometer Scale And Resolution
            3 => {
				if self.state.get () == State::Idle {
					if let Some (scale) = Lsm303dlhcScale::from_usize(data1) {
                        self.set_scale_and_resolution(scale, if data2 != 0 { true } else { false });
                        ReturnCode::SUCCESS
                    }
                    else
                    {
                        ReturnCode::EINVAL
                    }
				}
				else
				{
					ReturnCode::EBUSY
				}
            }
            // Set Magnetometer Temperature Enable and Data Rate
            4 => {
				if self.state.get () == State::Idle {
                    if let Some (data_rate) = Lsm303dlhcMagnetoDataRate::from_usize (data2) {
                        self.set_temperature_and_magneto_data_rate (if data1 != 0 { true } else { false }, data_rate);
                        ReturnCode::SUCCESS
                    }
                    else
                    {
                        ReturnCode::EINVAL
                    }
				}
				else
				{
					ReturnCode::EBUSY
				}
			}
			// Set Magnetometer Range
            5 => {
				if self.state.get () == State::Idle {
					if let Some (range) = Lsm303dlhcRange::from_usize(data1) {
                        self.set_range(range);
                        ReturnCode::SUCCESS
                    }
                    else
                    {
                        ReturnCode::EINVAL
                    }
				}
				else
				{
					ReturnCode::EBUSY
				}
            }
			// Read Acceleration XYZ
            6 => {
				if self.state.get () == State::Idle {
					self.read_acceleration_xyz ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
            }
            // Read Temperature
            7 => {
				if self.state.get () == State::Idle {
					self.read_temperature ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
			}
			8 => {
				if self.state.get () == State::Idle {
					self.read_magnetometer_xyz ();
					ReturnCode::SUCCESS
				}
				else
				{
					ReturnCode::EBUSY
				}
            }
            // default
            _ => ReturnCode::ENOSUPPORT,
        }
    }

    fn subscribe(
        &self,
        subscribe_num: usize,
        callback: Option<Callback>,
        _app_id: AppId,
    ) -> ReturnCode {
        match subscribe_num {
            0 /* set the one shot callback */ => {
				self.callback.insert (callback);
				ReturnCode::SUCCESS
			},
            // default
            _ => ReturnCode::ENOSUPPORT,
        }
    }
}

Thanks for the report @bradjc. Looks like the problem is with the block comment in the match first match arm that's resulting  in rustfmt failing to do any formatting within the match blocks (not just failing to convert the tabs).
This is definitely a bug, and as a temporary workaround you may want to consider changing those two block comments on the 0 arms to inline comments above the arms like was done for the others

I've been bitten by this too. This affects both the match bodies and the match patterns. The following code is unmodified by rustfmt without warnings:
fn parse_term() {
    match x { |
       TokenKind::Name => {
                let qualified_name =1;
            }
            | TokenKind::Suspend
            // foo
            | _ => {}
        }
}

I am working on this issue

I've just run into this, here's a very simple example:
fn main() {
    let i = 0;
    match i {
        0|1
        // Comment!
        |2|3 => {println!("<4");}
        _=>unreachable!(),
    }
}

Removing the comment allows rustfmt to work as expected, but so long as it's there the entire match block seems to be ignored for formatting, even the entirely unrelated _ arm. Is there any update on this case?

On a semi-related note: are there any logging/tracing options that can be enabled to emit a warning in this and similar cases (where where rustfmt silently "gives up")?

@daprilik you can set error_on_unformatted = true.
Trying to format the code snippet from https://github.com/rust-lang/rustfmt/issues/4119#issuecomment-1400933562 produces the following warning:
error[internal]: not formatted because a comment would be lost
 --> <stdin>:3
  |
3 |     match i {
  |
  = note: set `error_on_unformatted = false` to suppress the warning against comments or string literals

warning: rustfmt has failed to format. See previous 1 errors.


Performance question

Hi, when using your node (cocossd) it is detecting fine and I just have a question related to performance. Running NR flow on a raspberry pi and comparing the detection time with node-red-contrib-tfjs-coco-ssd that is much faster, basically detecting the same objects
Do you happen to know what differs? Does your node use the same library? Do you use hw acceleration (using GPU) when it is supported like in the raspberry pi?
Best regards & thank you!
Walter

@krambriw Thank you for your question. As you said, performance is totally different from node-red-contrib-tfjs-coco-ssd because node-red-contrib-tfjs-coco-ssd uses binary code for specific CPUs. node-red-contrib-tfjs-coco-ssd is natural implantation but I implemented my node as pure JavaScript for all of CPUs (Apple M1, AIX, Raspberry Pi Zero).

Thank you for your explanation, fine, understood!
Best regards, Walter


"Run Yarn twice" bug

Our documentation currently recommends running docker-compose build as well as docker-compose run web yarn install. This shouldn't be needed because yarn install is built into the build script.
This ticket requires researching the root cause of the issue and providing a solution that works on all three operating systems: Windows, macOS, and Linux.

I @kalimar : I will like to look into this

@anusha5695 Just starting out with terrastories and hit this problem. I think it's because public/packs is versioned. Removing public/packs and it works like a charm.

Great news. We'll cut another ticket to clean up the docs for people not to run things twice.

@anusha5695 Are you still interested in this ticket? Otherwise I'll be happy to fix it.

@anusha5695 Thanks for considering the ticket. We can definitely find you some more issues in around node modules if you are interested. @wulffeld found a solution and is going to submit a PR.


Flask server in Docker container

In the document, it is stated that all components should be dockerized.

[x] create a Dockerfile to build the Docker Image containing the Flask server
[x] add the component in the docker-compose file
[x] discuss two different solutions for connecting the flask container and the rabbitmq container (i.e., hostname/IP address to use)
[x] test that the deployment works properly (automatically builds the image if not available)

@pgraverdy

I have added another branch named flask_in_docker where I have dockerised the flask app.  I have added the instructions to run it in the readme.  It automatically builds image and runs the app.
Regarding the discussion on two solutions to connect different containers I am adding it in the readme file of this very branch.

Let me know if I should create a PR for this issue.

@pgraverdy
Ways to connect containers:

Communicating through networking: Using network provided by the docker to connect between two containers. This network can be user modified.
By Using Volumes: by writing their files into volumes which can be shared between two containers.

@azlaanmsamad
Regarding the apps in the two containers (flask and rabbitmq) the communication protocol between them is AMQP, so its a networking protocol. There is no shared data/state between them.
Hence the two solutions are

using a dedicated network. between the containers (easy to specify with docker compose) and use the container name defined in the docker compose to communicate (e.g., connect to the rabbitmq server). In this setup, the "IP Address" can be replaced by the container name : if a URL is required for some operation, it would be http://container_name:port/ (e.g., http://rabbitmq_container:15672/ for the dashboard).Docker takes care of the name to IP resolution.
consider that the ports of the application are mapped to ports on the host. Then entering the IP address of the host with the mapped port will connect to the proper service/application (for HTTP protocol, you can also have a proxy server, on the host or as a container in the docker compose, that redirects based on URL paths.

As a side note, sharing volumes between containers (when relevant) has to be carefully done, You just can't access the files containing the data from a DB from another container (it will definitively create inconsistencies in the database). Sharing a volume for read access is fine, not for write access.

@pgraverdy
Okay thank you for the explanation. In case two app containers are using the same database to write files, how do we avoid inconsistency?

@pgraverdy
Okay thank you for the explanation. In case two app containers are using the same database to write files, how do we avoid inconsistency?

@pgraverdy
Should I create a PR for this?


Typo in formula p105

The proof of L'Hôpital Rule page 105 on the second line:
lim x->a (f(x) - f(x)) / (g(x) - g(a))
The difference on the numerator should be f(x) - f(a), not f(x) - f(x), which is obviously 0...
Edit: page number

Also, page 104 in 7.2.1 "However, L'ôpital's Rule ..." is missing its leading "H".


SwaggerResponseExample is not working in version 2.2.0

To work I had to downgrade to version 2.1.1

It's working for me. Can you try 2.2.1, which I just released a minute ago? If it's not working please post your code.

I tested and it's not working in version 2.2.1 too. I am using .net core 2.0 and bogus nuget, but even without bogus it's still not working in these versions (2.2.0/2.2.1).
My repository https://github.com/ThaynaraSantos/api-doc

Hmm I don't have .NET Core 2.0 installed on my dev box as it has issues with some of my existing .NET Core 1.1 solutions at work. I will try run your code under Core 1.1 and see if I can reproduce your issue.

Great, thanks!

What exactly is not working? I've moved your code to my Core 1.1 sample app:
        [HttpGet]
        [Produces("application/json")]
        [SwaggerResponse((int)HttpStatusCode.OK, Type = typeof(IEnumerable<House>))]
        [SwaggerResponseExample((int)HttpStatusCode.OK, typeof(HouseGetExamples))]
        public IEnumerable<House> Get()```

public class HouseGetExamples : IExamplesProvider
{
    public object GetExamples()
    {
        HouseGetExample houseGetExample = new HouseGetExample();

        return new List<House>()
        {
            houseGetExample.GetExamples() as House,
            houseGetExample.GetExamples() as House
        };
    }
}

public class HouseGetExample : IExamplesProvider
{
    public object GetExamples()
    {
        House house = new House {Address = "456 Bridge Street", Color = "Blue", Id = 1};
        return house;
    }
}

![capture](https://user-images.githubusercontent.com/169179/30294022-908da7ae-9733-11e7-9869-5a693fc75ce8.JPG)

Run the project and go to http://localhost:25111/docs
The same response example, in version 2.1.1, is working.

I have reluctantly installed .NET 2.0 SDK on my dev machine just to try reproduce your issue.
I ran your repo - it works fine for me locally. Your repo was using 2.1.1 of my package. I updated your repo to use the latest version of my package - 2.2.1, and it still works fine.
I am testing this by going to /swagger:

I didn't know about the /docs page until you told me about it - what generates that? Is that part of Swashbuckle.AspNetCore? Ah, I see, it's a Redoc page. I've never heard of that. Anyway, that's working fine for me here too:

Hmm. I can't fix it for you if I can't reproduce it.

swagger.json generated in:
Version 2.1.1 (working)

Version 2.2.1 (not_working)

Maybe, the ReDoc script can't read the example in swagger.json after that change.

For your knowledge. (https://github.com/Rebilly/ReDoc)

OK, so it IS working in Swagger-UI, which is where I test it, but not working in ReDoc.
I removed the application/json stuff by request as part of Issue #12.
There is a PR on my .NET Framework version of this project to optionally add it back in again https://github.com/mattfrear/Swashbuckle.Examples/pull/13
When I get a chance to do that I will port that PR to this project and then it should work again for ReDoc.
I have no idea why redoc needs application/json to work and that seems stupid, but I haven't been through their spec to make sense of it, as I only just heard of them.

ReDoc author here.
Just to clarify, ReDoc behaves correctly here according to Swagger 2.0 spec: https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#exampleObject
Example object should be a map from {mime type} to example. So, technically, the output produced by 2.2.1 is invalid Swagger.

Thanks @RomanGotsiy for clarifying. I will revert the change so that my package produces valid Swagger as soon as I get a chance.

Released https://www.nuget.org/packages/Swashbuckle.AspNetCore.Examples/2.2.2

Thank you! It's working fine now.

This seems to be fixed now with Swashbuckle 3.0


Fix for installation error

Fix #6
Remove some reference to serverless plugin project

@kaiquelupo can you approve as well? Once merged this and #8 we can merge everything to https://github.com/twilio-infra-as-code/plugin-twilio-infra


Does the project not support the creation of gateways? The gateway I created failed to log in and the page does not have a header

Overview of the issue
Unable to log in and the page does not have a header
JHipster Version(s)

generator-jhipster-vuejs  1.9.2
generator-jhipster 6.10.3
Browsers and Operating System

win10 chrome

Hi @QAQtutu : which options did you choose, when creating your gateway ?

Microservice gateway > N( Spring WebFlux) > JHipster Registry > JHipster UAA server > MySQL > No
cache > N(Hibernate 2nd level cache) > Maven

I created a React version of the gateway and it works very well. But I only use Vue

Ok, it's because UAA is not supported for Vue.js
See:

https://github.com/jhipster/jhipster-vuejs/issues/529
https://github.com/jhipster/jhipster-vuejs/issues/581
https://github.com/jhipster/jhipster-vuejs/issues/643

Unless someone wants to code this part, it won't be done, so no support.


Processing functions to process l1b and l1c files to l2 wave

First version of a processor to process l1b and l1c files to l2 wave.

Hello @rmarquarlops
Are you ready for a merge ? Can we solve the remaining reviews?


Text only / Numeric only

Is there a way to implement text-only or numeric-only input fields?
Has anyone does this before?
Thanks

Artificial example, does not handle copy/paste
class Program
{
    static void Main(string[] args)
    {
        Application.Init();
        var win = new Window("window");
        Application.Top.Add(win);
        win.Add(new NumbersOnlyTextField(0, 0, 25, ""));
        Application.Run();
    }
}

public class NumbersOnlyTextField: TextField
{
    public NumbersOnlyTextField(string text) : base(text)
    {
    }

    public NumbersOnlyTextField(NStack.ustring text) : base(text)
    {
    }

    public NumbersOnlyTextField(int x, int y, int w, NStack.ustring text) : base(x, y, w, text)
    {
    }

    private readonly int[] _acceptableKeyValues = new int[] { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9' };
    private readonly Key[] _acceptableKeys = new Key[] { Key.Backspace, Key.Delete, Key.CursorLeft, Key.CursorRight, Key.DeleteChar };

    public override bool ProcessKey(KeyEvent kb)
    {
        if (_acceptableKeys.Contains(kb.Key) || _acceptableKeyValues.Contains(kb.KeyValue))
        {
            return base.ProcessKey(kb);
        }
        return true;
    }
}

You could for example pass list of acceptable keys/key-values in the constructor to handle different cases, also you could look at the source of TextField to see what cases should be passed to base class to handle copy/paste, selection, ...

Hi @stanac
Thanks for this great example. This answers a lot of my follow up questions too, so appreciate it.
By any chance, would you know how I might go about underlining fields too? Is that even possible?
Thanks!

Hi @worksofliam I'm not sure what do you mean by "underlining fields". Underlining text label or ... ?

Hey @stanac
Sorry I should have been more specific. Yes, underline for an input TextField
new TextField ("") {
  X = field.Position.X, Y = field.Position.Y,
  Width = field.dataType._Length,
  Height = 1
};

Thanks

I don't think it's possible, I am not sure if any framework has that option in the text input (including basic html). You could change color scheme by overriding Redraw of TextField, if you look at the source of redraw you could see method is setting color scheme, if you set it to something else it could be a kind of highlighting the text in the field.
Also Label has TextColor field which can be set to custom value, if that helps.

Thanks for the info. You did solve my original issue so I will close this for now.
Thanks for your help!


update the template to declare the input args as a struct type

instead of redefining a struct repeatedly create a type to declare the input args.
My motivation was to allow me to make assertions on this call in my tests (ginkgo/gomega in the example below) without needing to redeclare the struct myself
Eventually(func() []listservicesstub.PersistListsCall {
  return elasticStub.PersistListsCalls()
}).Should(ConsistOf(listservicesstub.PersistListsCall{
  Lists: []models.List{newList},
}))

100% backwards compatible and actually results in less generated code

any thoughts about this? would love to get it merged

Perhaps this is no longer relevant?

Hey @hcliff - thank you so much for this. I wonder if the additional types in the package space end up cluttering it too much? What do you think?


Easy cloud.gov deployment

Easier deployment of watchtower as a cloud.gov app.

Created manifest.yml file
Added global entries to config.yaml, since relying on the defaults was not working. All three use Environment variables to allow them to work with any deployment situation

Regarding your comment on the global section defaults. In my original implementation, I intentionally opted for secure defaults. For example I set port to default to 0 to force users to tell watchtower where it should listen instead of assuming a port for their environment. I set cloud_controller_url to an empty string to default to not attempting to talk to a URL that might not be the intended cloud controller.
These defaults are intentional, but your comment makes it clear that these need documenting why they are the way that they are.

These defaults are intentional, but your comment makes it clear that these need documenting why they are the way that they are.

If the defaults don't work, it would be better to not refer to them as defaults and instead refer to them as empty values that must be set. In that case, having them in the config.yaml with blank values would be much more clear to me.
This is maybe a separate issue, but the one default that seems likely to survive, that of refresh_interval didn't actually work for me. When I tried not setting it the app complained that the refresh_interval was less than the minimum interval of 10s

When I tried not setting it the app complained that the refresh_interval was less than the minimum interval of 10s

I have also seen this behavior, although I have only just now created a bug for it (linked).
https://github.com/18F/watchtower/issues/29


move tab left/right does not work

I tried to move tabs left and tight via the command palette, and nothing happens.

It turns out the bug here is the name of this command. It moves the tab to the next pane to the left/right. It doesn't change the order of the tabs within the group of tabs.
It needs a better name.

Fixed in v0.58.0.


Chore(host, client, mpt): Custom error types

Remove all uses of eyre/anyhow from the lib, and remove any panics in fns that return results

I think eyre is the appropriate choice for errors in RSP, since its primary use is as a standalone binary, not a library. For example, we're never going to publish RSP on crates.io. Open to being convinced though.
I agree with getting rid of the MPT panics -- is there a way to use eyre instead of thiserror to better handle the panic?

ooo interesting.. Would. you still be supportive of the FromProofError in rsp-mpt?
I have a (maybe contrived) use case actually for exposing RSP as a lib: you can use RSP to augment light client sync protocols to get execution validity proofs


Gin example cannot use context.

If I understand it properly the gin example does not use the context, because there is no solution to pass the context from gin to the renderer.
I think there would be an alternative solution by using the Render() method of the gin context.

Thanks, you're right!
I'd happily take a PR for an example update if you feel like it.

https://github.com/a-h/templ/pull/466
You mean like this? Or are you thinking on something different?

Closed in https://github.com/a-h/templ/commit/b6e7a779dacbddc4a185023be181fe7349e6b923
Thanks for your contributions!


'unkown shelly-gen1' when using 'shelly TRV flow'

There seems to be a small error in the 'shelly TRV flow' example flow.
I have node-red-contrib-shelly 1.10.1 installed

Ok I will hava a look

Thanks for your work, ... but I still get the same error

You need to install shelly 9.15.

Thanks again.
Sorry, but I need another kick. My node-red palette shows node-red-contrib-tasmota as 0.9.12 being actual.
Then i downloaded 0.9.15.tar.gz from https://github.com/windkh/node-red-contrib-shelly/releases/tag/V9.15.0
renamed it to .tgz and tried to install it in node-red using the ‚upload Module tgz-Datei‘ button.
This gives the attached error message with an empty protocol.
What am i doing wrong (and how can I do it in the right way :-) ?
Greetings Klaus

Am 17.09.2022 um 09:26 schrieb Karl-Heinz Wind @.***>:
You need to install shelly 9.15.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Tasmota?
Just install it from the side bar in node-red: palette verwalten

That was it. Lost in too many open projects :-) Thanks again.

Am 17.09.2022 um 10:21 schrieb Karl-Heinz Wind @.***>:
Tasmota?
Just install it from the side bar in node-red: palette verwalten
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.


Font shows an extra dot and if press enter key also show an extra dot in a textarea component - Internet Explorer

Font shows an extra dot and if press enter key also show an extra dot in a textarea component - Internet Explorer11
like this:

While Internet Explorer has largely been retired, just pointing out that this remains an issue with the version (20H2).  Even on an entirely empty text area:
<textarea style="font-family: text-security-disc;"></textarea>


Requesting 5 dimensions yields a warning the data are being truncated to 3 dimensions

I'm calling the code directly as #1 is a blocker for me right now. In doing so, I requested use of 5 dimensions:
res = convex_hull(qiime2.Metadata(md), pc, 'group_and_timepoint', 5)

This yielded the following warnings:
/Users/dtmcdonald/q2-convexhull/q2_convexhull/convexhull.py:77: Warning: Setting number_of_dimensions to 3.
  warn(
/Users/dtmcdonald/q2-convexhull/q2_convexhull/convexhull.py:84: Warning: PCoA result has 5 dimensions. Truncating to 3 PC's
  warn(

Hi Daniel! Currently it only supports i think two or 3 dimensions. Do you
want me to change this?
On Thu, Apr 20, 2023 at 3:32 PM Daniel McDonald @.***>
wrote:

I'm calling the code directly as #1
https://github.com/dpear/q2-convexhull/issues/1 is a blocker for me
right now. In doing so, I requested use of 5 dimensions:
res = convex_hull(qiime2.Metadata(md), pc, 'group_and_timepoint', 5)
This yielded the following warnings:
/Users/dtmcdonald/q2-convexhull/q2_convexhull/convexhull.py:77: Warning: Setting number_of_dimensions to 3.
warn(
/Users/dtmcdonald/q2-convexhull/q2_convexhull/convexhull.py:84: Warning: PCoA result has 5 dimensions. Truncating to 3 PC's  warn(
—
Reply to this email directly, view it on GitHub
https://github.com/dpear/q2-convexhull/issues/2, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6GMQYSVGFWKJKJYSZXBCDXCG2P5ANCNFSM6AAAAAAXGDMXJ4
.
 And I guess I'm hitting this because of using the code directly rather than through Q2. Thanks!